Possibility to change style in real-time

Hi,
first of all I want to thank you for your support. I want also to congratulate about this component which is very good.
I wirte you to ask if it would be possible for the next version to supply a scss style file where configure the style in real-time.
In particular I would like to use this component with Nebular framework where it is possible to change the style using a selector.
If I have the scss style file for this calendar component I can change the background color, the date selected color etc...
Again, thanks a million in advance for your help

You could overwrite the picker's default css style in your own scss file. The picker default scss file is in the source code.
The css selector would be found in your browser's dev tools.

That's cool, thanks a million.
If I may suggest something, I think it would be great to add the picket style documentation to the very good one at Ng Date Time Picker
That is very well done and maybe you can add at the end a Style section.
Also, I am trying to open the picker from a modal window but it appears behind it instead of in front.
Do you have any suggestion on how to fix it?
Again thanks a million


Properly escape release attachment URL

Make sure file attachments on a release get a properly escaped URL when
linking.
Fixes #6506

Codecov Report

Merging #6512 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6512      +/-   ##
==========================================
+ Coverage   40.26%   40.26%   +<.01%
==========================================
  Files         403      403
  Lines       54069    54069
==========================================
+ Hits        21770    21772       +2
+ Misses      29282    29280       -2
  Partials     3017     3017

Impacted Files
Coverage Δ

models/repo_list.go
67.89% <0%> (+1.05%)
: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 9bf20f4...7ee1c71. Read the comment docs.

@mrsdizzie merged. Please backport :)


Example code not working

Hello. I have been trying to  test your example code so i add the search function to my app but it seems the example code isn't working

Can you please highlight the exact error you're facing?
Here's a live web demo of the package and it seems to be working fine,
Live Web Demo

After running the code and trying to search it doesn't show any results as above.
Here is my firstore structure

The code
FirestoreSearchScaffold(
      firestoreCollectionName: 'files',
      searchBy: 'name',
      scaffoldBody: const Center(child: Text('Firestore Search')),
      dataListFromSnapshot: DataModel().dataListFromSnapshot,
      builder: (context, snapshot) {
        if (snapshot.hasData) {
          final List<DataModel>? dataList = snapshot.data;

          return ListView.builder(
              itemCount: dataList?.length ?? 0,
              itemBuilder: (context, index) {
                final DataModel data = dataList![index];

                return Column(
                  mainAxisSize: MainAxisSize.min,
                  mainAxisAlignment: MainAxisAlignment.center,
                  crossAxisAlignment: CrossAxisAlignment.start,
                  children: [
                    Padding(
                      padding: const EdgeInsets.all(8.0),
                      child: Text(
                        '${data.name}',
                        style: Theme.of(context).textTheme.headline6,
                      ),
                    ),
                    Padding(
                      padding: const EdgeInsets.only(
                          bottom: 8.0, left: 8.0, right: 8.0),
                      child: Text('${data.name}',
                          style: Theme.of(context).textTheme.bodyText1),
                    )
                  ],
                );
              });
        }

        if (snapshot.connectionState == ConnectionState.done) {
          if (!snapshot.hasData) {
            return const Center(
              child: Text('No Results Returned'),
            );
          }
        }
        return const Center(
          child: CircularProgressIndicator(),
        );
      },
    );

The Model
class DataModel {
  final String? name;
  final String? link;
  // final String? framework;
  // final String? tool;

  DataModel({this.name, this.link});

  //Create a method to convert QuerySnapshot from Cloud Firestore to a list of objects of this DataModel
  //This function in essential to the working of FirestoreSearchScaffold

  List<DataModel> dataListFromSnapshot(QuerySnapshot querySnapshot) {
    return querySnapshot.docs.map((snapshot) {
      final Map<String, dynamic> dataMap =
          snapshot.data() as Map<String, dynamic>;

      return DataModel(
        name: dataMap['name'],
        link: dataMap['link'],
      );
      // framework: dataMap['framework'],
      // tool: dataMap['tool']);
    }).toList();
  }
}

I'm using these
firebase_core: ^1.10.6
cloud_firestore: ^2.2.2
firestore_search: ^0.1.7

Have you configured firebase for your project?

yes, Firebase is configured and everything firebase related is working.
I'm just having issues with the search.

on your web example, can I have a few search terms in your database so I can  see

The issue might be that your configured firebase project does not have any
entries in the cloudfirestore.
Lemme share a screenshot of the collection so you can replicate it.
On Thu, 23 Dec 2021, 10:39 PM Tatenda, @.***> wrote:

[image: image]
https://user-images.githubusercontent.com/40611131/147274860-c7cca4d1-ebc0-401b-94ce-8fbd01fa5ac2.png
on your web example, can I have a few search terms in your database so I
can see
—
Reply to this email directly, view it on GitHub
https://github.com/asadamatic/firestore_search/issues/14#issuecomment-1000443998,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI6ZBMPQW4J7JNCJHWW5U5LUSNNEHANCNFSM5KUWJV3Q
.
You are receiving this because you commented.Message ID:
@.***>

Thanks. The issue was with my firestore entries structure


Repo name change / packagist

Hello, since the recent change of name of the repo (d11wtq/boris to borisrepl/boris), the packagist reference still points to https://github.com/d11wtq/boris.git.
Unless some composer cache exists on the server or on the local machine for d11wtq/boris package, it is now impossible to get it via composer, unless we had a specific repositories section in the composer.json file.
*** [err<EMAIL_ADDRESS>- Installing d11wtq/boris (v1.0.8)
*** [err<EMAIL_ADDRESS>Downloading
*** [err<EMAIL_ADDRESS>Failed to download d11wtq/boris from dist: The "https://api.github.com/repos/d11wtq/boris/zipball/125dd4e5752639af7678a22ea597115646d89c6e" file could not be downloaded (HTTP/1.1 404 Not Found)

Can you refresh or update the package reference in https://packagist.org so that it now points to https://github.com/d11wtq/boris.git ?

:+1: it needs to point to https://github.com/borisrepl/boris.git.

Yes, messes with the final repo.
Thx @GrahamCampbell

This must be a caching issue. The source in packagist is already set correctly.

 It really annoys me how broken it is. It caches branch aliases wrong too.

@d11wtq Yeah, it's caching...

@d11wtq please help, I'm getting this in my eb-commandprocessor.log when trying to deploy my app to AWS Elastic Beantstalk. (It's a laravel 4.2 app)
 - Installing d11wtq/boris (v1.0.8)
    Downloading: connection...

  [Composer\Downloader\TransportException]
  The "https://api.github.com/repos/d11wtq/boris/zipball/125dd4e5752639af7678a22ea597115646d89c6e" file could not be downloaded (HTTP/1.1 404 Not Found)

I've tried the additions to my composer.json file but it didn't make a difference.
Many thanks,

please help, I'm getting this in my eb-commandprocessor.log when trying to deploy my app to AWS Elastic Beanstalk. (It's a laravel 4.2 app)

You need to make sure your lock file is updated, and your composer cache is cleared.


CompileJava with wrong package,  build successful

I had a class named MissionUploadResource with wrong package, like:

Usage:

When i use gradle to run task "compileJava", the result is "BUILD SUCCESSFUL".
I think there should report a error.

This isn't a bug. It's the expected behavior shared by both Gradle and IntelliJ. The red squiggly is simply telling you that by convention the file path and package name should match but it's not required.


Added a large middle finger

please consider for hactober2024

Please dont create such PRs, also you didnt create any issue first
and this PR is not at all tolerated and entertained


Fix e2e tests that were broke introducing wait-on

Codecov Report

Merging #150 (7e2828e) into main (77041b2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #150   +/-   ##
=======================================
  Coverage   54.70%   54.70%
=======================================
  Files          82       82
  Lines        2210     2210
=======================================
  Hits         1209     1209
  Misses       1001     1001

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/78932

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/96802

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/97242


Can go to file function ignore .. and . in the begin

I am dealing with project which write in c and use Makefile + GCC to compile.
Sometimes GCC gives error file path like ./aaa/bbb/ccc.c, which can not access in current go to file function even file is just at this place.
And subfolder has it's Makefile use common header file in super folder.
So  GCC gives error file path like ../common/common.h, which can not access in current go to file function.
So I suggest ignore  .. and ., even ../../././ in the begining of go to file  path

/duplicate https://github.com/microsoft/vscode/issues/133438

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Update Root Changelog

Update root changelog

Not stale. ( unless it has been done in another PR )


How to integrate the code ?

Your plugin is very powerful but for some begginers that's not easy...
I have this not working:
ERROR:
#####################
[Error] TypeError: undefined is not a function (evaluating '$('div#clock').countdown')
global code (index.php, line 780)
HTLM:
#####################

Some text...

SCRIPT:
#####################
$('div#clock').countdown('2020/10/10', function(event) {
  var $this = $(this).html(event.strftime(''
    + '%w weeks '
    + '%d days '
    + '%H hr '
    + '%M min '
    + '%S sec'));
});

Probably because you didn't imported the code correctly in your HTML file.


revision 6a3ff018b1 (2 weeks ago) • 2018-10-26 01:30:21 -0400 breaks fcharts 0.0.9

Flutter 0.10.1 • channel dev • https://github.com/flutter/flutter.git
Framework • revision 6a3ff018b1 (2 weeks ago) • 2018-10-26 01:30:21 -0400
Compiler message:
file:///dev/pub.dartlang.org/fcharts-0.0.9/lib/src/bar/data.dart:14:7: Error: Inferred type argument '#lib1::ChartDrawable<dynamic, #lib1::ChartTouch>'
violates the corresponding type variable bound of 'ChartData' in the supertype 'ChartData' of class 'BarGraphData'.
Try specifying type arguments explicitly so that they conform to the bounds.
class BarGraphData implements ChartData {
^
file:///dev/flutter/.pub-cache/hosted/pub.dartlang.org/fcharts-0.0.9/lib/src/chart_data.dart:4:26: Context: Bound of this variable is violated.
abstract class ChartData<T extends ChartDrawable<T, ChartTouch>> {
^
Compiler terminated unexpectedly.
Please help!

Downgrade Flutter to v0.10.1 resolve the issue but for long term, I think should should future version versions of Flutter. Thanks!

Thanks for reporting!
Fixed with f996a1ed6141ae37d19df17205600abe70977057


Appstudio update hacbs-test-project-mje5

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-mje5-on-pull-request-
creating pipelinerun hacbs-test-project-mje5-on-pull-request- in namespace rhtap-demo-odcg-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-mje5-on-pull-request-
creating pipelinerun hacbs-test-project-mje5-on-pull-request- in namespace rhtap-demo-xwyu-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-mje5-on-pull-request-
creating pipelinerun hacbs-test-project-mje5-on-pull-request- in namespace rhtap-demo-htqk-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


Trying to exclude number of paths from breaking command

Hi,
I'm new to this tool which I find very astonishing :-).
I want to compare and check for breaking changes but I need to exclude some of my paths.
I tried --match-path and --filter-extension but it doesn't work for me.
Can someone help?

Hi  Afik.
Could you please add instructions to replicate the issue you encountered:

the two files you compared
the cmd-line you run
the expected output

Alternatively, you are welcome to contact me on slack: https://join.slack.com/t/oasdiff/shared_invite/zt-1wvo7wois-ttncNBmyjyRXqBzyg~P6oA


added environment variables for selection of datacenters

added environment variables

NEXMO_REST_API_URL
NEXMO_API_URL
to set datacenter according to:
https://developer.nexmo.com/client-sdk/configuration/configure-data-center

Coverage increased (+0.1%) to 100.0% when pulling 815dbb8ec4de3c60e3bb6a44f16416c9a7fb423f on sbickmann:master into 01bc00cb3954a6fe7d21abffcc8ff790ff5cd684 on Nexmo:master.

I think we have support for changing URLs in the python lib, but @superdiana can you take a look please?

We have directives for overriding API url's in the readme. https://github.com/Nexmo/nexmo-python#overriding-api-urls

Hi, sorry @superdiana for that additional question.
We wanted to switch the api server to e.g. https://api-eu-1.nexmo.com for data privacy reasons needed in german /eu markets.
Just switching client.api_host does not switch the hard coded Authentication Server.
see: https://github.com/Nexmo/nexmo-python/blob/master/src/nexmo/init.py#L119
Doing another trick, I am able to workaround it... So I created the Merge Request :D
But maybe you can tell me the Authentication should not be done within the eu api endpoint (which is currently possible).
Thanx for a short answer to that :)

@sbickmann you have a good point. I will reopen this issue and workaround your PR for an optimum solution.

You can also extend the class

class NexmoClient(nexmo.Client):
    def __init__(....):
        super().__init__(....)
        api_server = BasicAuthenticatedServer(
            "mycustomurl",
            user_agent=user_agent,
            api_key=self.api_key,
            api_secret=self.api_secret,
        )
        self.application_v2 = ApplicationV2(api_server)```

And rewrite what you need to rewrite and from there create your personalised instance of the class.Is this the sort of workaround you already explored with?

@superdiana
Yes this kind of workaround was what we did also.
But as there are environment variables for other things like NEXMO_API_KEY or more, my thoughts were to apply the same scheme and add it for the api host.
If you want me to change anything, I can do it.
Closing would be also ok for me if you have own plans ... since the workarounds do their job currently :)
So how to proceed with this request?
Kind regards

The PR is a very good option for those who want to integrate it to their personal forks. For the moment, this is an optional step and it won't be included in the official SDK. Needless to say we are working on improvements and exciting upcoming changes.  I will add additional instructions to our README for the time being. Again, we really appreciate and encourage contributions to our SDK. Thank you for your awesome work!


Leak fixes

Fixes various use-after-free and nullptr dereferencing bugs that happen when the model file can't be found.
Fixes #4

This seems to be stable now under the various situations where a model file can be missing:

The module data directory is missing at start-up
The module data directory exists but is empty at start-up
The filename stored in the settings points to a model file that doesn't exist at start-up (inside and outside the module directory)
Switching from a working model to a broken one (ie select a non-default model when the default one is missing, and then restore defaults)
Switching to a working model path from any of the above states

OBS reports no memory leaks at exit.

Thanks - works for me, no reported leaks :smile:


Facebook : Fix Javascript and New Domains

As Facebook made a new user interface, I corrected the phishlet to match the new login forms. Like instagram, I added some js to delay the login and send the password unencoded before the real login.

Hello, This phishlet does not capture cookie tokens, only username and password. After re-entering it writes "Acces denied" and blocks the phishing page. Where is the solution?

Hello, This phishlet does not capture cookie tokens, only username and password. After re-entering it writes "Acces denied" and blocks the phishing page. Where is the solution?

You probably made too many login attempts so your ip is temporarily banned, but I can confirm the phishlet works well

UPDATE : Facebook seems to have a fraud detection system like all big websites targeting by IP address, so I would recommend to not use IP of VPS of famous hosters. Instead, I would recommend residencial proxies.

I had similar issues stuck on the first sign in page that's the old phishlets.... attually I got the latest .yaml off the author y'all can reach out to him on icq @mrgretzky to get issues fixed if needed.


webview.src doesn't work before body.onload

Minimal repro: https://gist.github.com/mkscrg/a8748a986c3040b58d65
This creates a webview but doesn't correctly set the src attribute:
<body>
  <script>
var loadWebview = function() {
var webview = document.createElement('webview');
webview.id = 'splash';
webview.style.height = '100%';
webview.style.width = '100%';
webview.src = 'webview.html';
document.body.appendChild(webview);
};
loadWebview();
  </script>
</body>

Workarounds include:

calling the function after full page load, e.g. <body onload="loadWebview()">
including the webview tag (and src attribute) explicitly in HTML

webview doesn't have any of its attributes until document.readyState == 'interactive'. This means src, preload, getURL, etc. will all be undefined, and if set, their values will be overwritten.


[REQUEST] Add custom sources for games.

Is your feature request related to a problem? Please describe.
Not at all.
Describe the solution you'd like
I would love a way to add custom sources for games.
Whatever source you guys use for this software, it barely scrape anything relative or new.
Like, Hellblade II <--  there are tons of safe, secure releases for that game.   yet it does not show up in Hydra :D
would love a way to add sources myself, so i can get relevant and new games added too :D
Describe alternatives you've considered
No response
Additional context
No response

First, do you mean Hellblade does not show up at all? Or that is shows with no download options?
Second, the option to add download sources is almost ready and will be on next release

Oh! i probably should've said.
So, i just took hellblade II as a random game, but i do find the game itself on hydra yea.  there's just no available source to download from.
but there are tons of sources available for it. ( trusted sources )
hence why i was wondering about the download sources thingy :D
But you answered that for me in your post there! :D

Done in 2.0


Using custom request methods like PROPFIND

Hi there,
is it possible to use request methods like PROPFIND? I tried using request() with method: "PROPFIND", method: :propfind and method: 'PROPFIND', but everything fails with a FunctionClauseError for :httpc.request/5.
Best regards,
CK

Oh. It works when setting `config :tesla, adapter: Tesla.Adapter.Hackney'. Which one? All of them! ;P

Sorry for the noise


solo: Better method of detecting Spotify network

Still sucks that we have to do this, but this way is more reliable than
the old way.

:+1:


Base url - prioritize meta tag value over the config of StorefrontModule

Currenlty the withConfig method of StorefrontModule takes precedence over the value from meta tag. But it should be the other way around.
What's more, it would be good to insert a placeholder meta tag in our index.html, so we can do easy search/replace when we deploy. For example:
<meta name="occ-backend-base-url" content="OCC_BACKEND_BASE_URL_VALUE" />

It's a following ticket of GH-1148

QA steps:
base url from meta tag works

[ ] configure some valid (but different than usual) base url in the content of the meta tag in index.html:<meta name="occ-backend-base-url" content="OCC_BACKEND_BASE_URL_VALUE" />

[ ] run app yarn start and verify that calls are made to different base url than usual


feat: add websocket service

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Wrong results when using Passphrase

w = Wallet(mnemonic)
This creates the correct bitcoin keys and derivations, if I check it agains the Samourai BIP44 Mnemonic Code Converter. On path m/44'/0'/0'/0/x, where x is any number, the keys match 100%.
However, as soon as I try:
w = Wallet(mnemonic, passphrase="blah")
The results no longer match Samourai's Code Converter with the same passphrase. They are totally different.
I think there must be an error or mistake in how the passphrase is added into the derivation (assuming that Samourai is doing it correctly), because the results should match.

Nevermind, I was reading passphrases from a txt file, and it turns out that it added some spaces, which meant the passphrases did not match what I put into Samourai to test. Now it works fine.


Function to go from model_ds to gwf/sim and backwards

I would like a function to create a simulation object (sim) from a model_ds object. The ds_model is easy to store and backup, while the gwf/sim is needed to use the plot functions of flopy. I can also imagine plenty of usecases for the reversed. Probably a lot of code is already part of the package. Thanks!

Good idea!

These are already in "nlmod/mfpackages"


Fix the gain methods of the ZI HDAWG8

The range of the HDAWG8 device specifies the final output range of the device channels. For example a range of 3.0 means that the device has output range of -1.5 to +1.5 V. This corresponds to a gain of 1.5. In general gain = range /2
This PR fixes a bug in the ZI HDAWG8 driver where the gain and range where confused.

Just a comment on that, since I've seen it passing by. Actually the range coincide with the gain, but under the assumption that your load is high impedance. If the load is 50 Ohm, the voltage is divided by two, since also the HDAWG output impedance is 50 Ohm.

@YakBizzarro Thanks for the info. I will add this to the documentation for our code.


Compactions cause the configured storage.tsdb.retention.size to be exceeded (and risk of running out of disk space)

Excerpt / TL;DR
The set limit of storage to be used (storage.tsdb.retention.size) is violated during compaction which poses a the risk of  running out of disk space.  The reason is that Prometheus does not take into account the required space for the temporary blocks. See Proposal section for two suggested approaches to fix this and allow Prometheus to reliably avoid exceeding it's allowance.
Issue
The storage.tsdb.retention.size option (https://prometheus.io/docs/prometheus/latest/storage/#operational-aspects) was a great addition to allow keeping metrics in relation to the available storage. While this can be a provisioned volume from some cloud provider or a LVM volume or just the amount of space you can give to Prometheus and then have it make the most of the available space (https://www.robustperception.io/configuring-prometheus-storage-retention/).
While there were some issues with WAL files and head_chunks not counted against the configured limit, those are now all taken into account when determining the current storage size thus reducing confusion for users and the risk of running out of disk space. Since checkpoints and first level of compactions happen over relatively small time intervals of two hours the peak is relatively small and usually just about keeping some sensible head room for - just as written in the documentation:

So the minimum requirement for the disk is the peak space taken by the wal (the WAL and Checkpoint) and chunks_head (m-mapped Head chunks) directory combined (peaks every 2 hours)

There also is good documentation on how to calculate the space requirements regarding the amount of samples ingested per time interval and their average size via built-in metrics:

https://www.robustperception.io/how-much-disk-space-do-prometheus-blocks-use/
https://www.robustperception.io/how-much-space-does-the-wal-take-up/

Even though this all helps to anticipate how man samples and time series can be stored and over which retention times using the available space, more critical to the risk of running out of space are the compactions over longer periods covering up to 31 days of samples by default  (https://prometheus.io/docs/prometheus/latest/storage/#compaction). The leveled compaction time ranges and the compaction is nicely explained at leveled compaction: https://ganeshvernekar.com/blog/prometheus-tsdb-compaction-and-retention/#condition-2-preset-time-ranges.
And there also is documentation at https://www.robustperception.io/configuring-prometheus-storage-retention/ explaining the requirement to keep enough space for such compactions:

As of Prometheus 2.7 and 2.8 there's new flags and options. 2.7 introduced an option for size-based retention with --storage.tsdb.retention.size, that is specifying a maximum amount of disk space used by blocks. This is still a bit best effort though as it does not (yet) include the space taken by the WAL or blocks being populated by compaction. To be safe allow for space for the WAL and one maximum size block (which is the smaller of 10% of the retention time and a month).

This is simply because the compaction only marks blocks as Deletable (https://github.com/prometheus/prometheus/blob/e51a17b50159d397544d7f62775ee9ceaf48cfc8/tsdb/compact.go#L71), but requires the additional storage to first write the merged blocks.
Coming to the real issue for me, this is not easy to convert into an actual amount of space to subtract from the total available space when setting storage.tsdb.retention.size aiming to maximize the storage retention time and making the most use of the space there is, see https://github.com/prometheus/prometheus/issues/10287. And even if I would use the referenced metrics to calculate the size a directory holding 31d worth of samples has now, this clearly is changing over time with time series churn or just more time series being ingested. And while the metrics are there in Prometheus, it's still not something that works without manual interventions and adjustments. This might be the cause for the issues like https://github.com/prometheus/prometheus/issues/10890.
To me, the current complexity in setting up retention by storage size beats the purpose of having a set it and forget it setting which translates to: Only use X GB of storage and make the most of it.
If I have to calculate the ingest rate and bytes per month and continuously monitor and adjust for that ... I could just end up converting this to a suitable retention TIME instead beating the purpose of having a retention size setting in the first place.
Proposal
I therefore propose to add functionality to not exceed storage.tsdb.retention.size during compactions.

Maybe this could be done by extending on the existing compaction planning to calculate a worse-case of storage required to write the new merged blocks. This value could be added to the "used storage" just like the current size of the WAL was added via https://github.com/prometheus/prometheus/pull/5886.
Another approach would be to make the compaction have a smaller footprint altogether by continuously removing the blocks from the previous compaction level and not having all block of two compaction levels exist in parallel. This would then define an upper bound for the amount of additional storage required - something similar to the documented space requirements for WAL and their checkpoints.

By tackling this issue, it would be come MUCH less likely to run Prometheus into out of space ... which still happens when currently using retention.size feature (https://github.com/prometheus/prometheus/issues/8884).

Hi, it seems your detailed report has not received much attention over the past 16 months.
I might suggest that you put a 2-line summary of what you are suggesting at the top.

Hi, it seems your detailed report has not received much attention over the past 16 months. I might suggest that you put a 2-line summary of what you are suggesting at the top.

@bboreham Thanks a for taking a look at this matured issue.
I added the suggested short version to the top and rephrased the title. After reading it again: One might lightly argue a certain style of doing things / implementation is a bug, I actually believe this issue here is a bug and not a feature.
The expectation of the user when setting storage.tsdb.retention.size actually is, that this is the automatically maintained limit (or very close to the limit). Storage based retention happens for a reason (most likely because the volume has a certain size).

Hello from the bug scrub!
We think this is a relevant issue to fix and would welcome PRs.
A mitigation of the issue would be to simply update the docs that additional disk usage is expected for those things. @machine424 has committed to clarify the documentation.


fix: camera controls ignored props

Fixes https://github.com/Tresjs/cientos/issues/384
Adds support for mouseButtons and touches prop.
Adds default mouseButtons and touches prop values based on https://yomotsu.github.io/camera-controls/examples/config.html
Adds support for partial overrides of mouseButtons and touches.

Feel free to point out at any mistakes and discrepancies from conventions. I have a strong React background, and some aspects of Vue are kinda new to me 😜

I agree with you, I would do it @whereiswolf (maybe you can export it in the defaultExpose ¿?)

@JaimeTorrealba quick summary of the recent changes:

Updated CameraControls docs with "User input config" section.
Added logic to check the camera type, to set the default values in the same way as camera-controls does.
Added CameraControlsClass re-export of camera-controls (if you have a better name suggestion please lmk 😄 )

If anything's unclear or should be changed, I'm happy to work on that 😉


switching between profiles

Hey! Great App!
I have one issue with it though. I use two profiles. (gist with config) One for ms teams which I use exclusively for University and always want to use push to talk with, and one for private use in discord, in which I always wnat to have an open mic (or toggle).
Now my issue is that the program only switches profile when I close the app i.e. end the task and then open the other app. Not when it's minimized. This is an issue for me since I want both always running in the background. Would it be possible to change when the target window becomes active, instead of when it starts?
Alternatively I would be fine with the possibility to set a hotkey to switch profiles.
Do you think you could implement either of these features?
Thanks for your work and I hope you can help me :)

Now my issue is that the program only switches profile when I close the app i.e. end the task and then open the other app. Not when it's minimized. This is an issue for me since I want both always running in the background. Would it be possible to change when the target window becomes active, instead of when it starts?

The linked app feature actually checks if a window exists for that app, not if the app is running or not.
So when the discord or teams window is closed (even if the app is running in the background) it should switch back to the default profile, however, in the case of teams, that's not working because the teams window is never actually hidden, instead the window is just minimized and the taskbar icon is hidden.
to fix this I'll just add an extra check so if the window is minimized, it will switch back to the default profile.

Commit 1cf7f8d should fix it, You can test it out in the development build.


viz tries to convert to number for a calculated (db function) column which is string

Superset version
0.25.5
Expected results
Table View shows my query data for the following clickhouse (sqlalchemy) query
SELECT EventID AS EventID,
       COUNT(*) AS count,
       dictGetString('events', 'Name', toUInt64(EventID)) AS EventName,
       sum(Value) AS TotalValue,
FROM production.fact_game
WHERE EventID == 1000
GROUP BY EventID
ORDER BY TotalValue DESC
LIMIT 50000;

Actual results
Unable to parse string "EventNameABC" at position 0

is displayed with the following stacktrage
Traceback (most recent call last):
  File "pandas/_libs/src/inference.pyx", line 1125, in pandas._libs.lib.maybe_convert_numeric
ValueError: Unable to parse string "EventNameABC"

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/superset/viz.py", line 364, in get_df_payload
    df = self.get_df(query_obj)
  File "/usr/local/lib/python3.6/site-packages/superset/viz.py", line 187, in get_df
    self.df_metrics_to_num(df, query_obj.get('metrics') or [])
  File "/usr/local/lib/python3.6/site-packages/superset/viz.py", line 199, in df_metrics_to_num
    df[col] = pd.to_numeric(df[col])
  File "/usr/local/lib/python3.6/site-packages/pandas/core/tools/numeric.py", line 133, in to_numeric
    coerce_numeric=coerce_numeric)
  File "pandas/_libs/src/inference.pyx", line 1158, in pandas._libs.lib.maybe_convert_numeric
ValueError: Unable to parse string "EventNameABC" at position 0

Potential cause
could be related to the following PR/FIX https://github.com/apache/incubator-superset/pull/4726

Maybe unrelated, but does the query really work in other clients? To me the EventName    column does not look like an aggregation function (dictGetString). I.e., doesn't it need to be in the group by clause?
I'm not experienced with Clickhouse, though...

it was not changed since we upgraded from a 0.23.x version... so yes...
the function is no aggregation function, it's simply a lookup in an inmemory map, translating an id to a name

maybe fixed with https://github.com/apache/incubator-superset/pull/5176 ?


schema for Dataset.id

I'm pretty sure I have read a thread between @Zethson and @davidsebfischer on this, but cannot find it.
A couple of questions:

Have we settled on a schema for Dataset.id?
As this most likely involves using Dataset.organ, how do we deal with organ names (coming from ontology) that contain spaces? just replace space by underscore?
Once we have a harmonised schema, can we just create the id automatically based on the attributes set in the constructor (like self.year, self.organ, self.organism et.?)

Maybe you were thinking of this thread?
https://github.com/theislab/sfaira/issues/57

nah, something else I think

This is how I am currently generating the ID:
https://github.com/theislab/sfaira/pull/89/files#diff-4321091ee1ec2fd54ffe5c2eb77ac003bcebc543c805fccc3988a0d4896db3c4R129
We should absolutely standardize this 110%!

nah, something else I think

We were discussing this on the sfaira_team_cookie mattermost channel. Maybe you need to scroll up a little bit.

Adding to what we talked about last time, Datasethas an attribute _directory_formatted_id which has all the slashes etc removed, can be keep idmaximally humanly readable, ie keep the full DOI, and use the transformation as in _directory_formatted_id if we use this as file names (eg. for cache files). The loader .pyfile can be the same ID entirely without the DOI, in this case we do not have any formatting issues anyway.
Secondly, organ and protocol are used when defining the ID, these are not necessarily single words anymore now that we use ontologies for that, so we have to take care when transforming these. Maybe just take ontology name and strip it from special characters?

I started implementing this. two sections of the ID that are a little tricky @davidsebfischer :

we use the first author of the paper in the ID, however firstauthor is not a metadata field in our sfaira datasets (and hence makes autogeneration of the ID difficult. I suggest we either use the lastauthor of the paper in the ID or make the firstauthor also a metadata field
the way we currently implement the numbering of the datasets in the ID of the DatasetGroupManyFiles (001 to 00x) assumes that there is no other dataloader from that paper. do you think this is a valid assumption

1. we use the first author of the paper in the ID, however firstauthor is not a metadata field in our sfaira datasets (and hence makes autogeneration of the ID difficult. I suggest we either use the lastauthor of the paper in the ID or make the firstauthor also a metadata field

.author is usually first author right now for the ones that I recently saw, if not we should change it to that. Moving forward, .author could also be a list of all authors. We could just use the first one if that s the case. In any case I'd use .author.

2. the way we currently implement the numbering of the datasets in the ID of the DatasetGroupManyFiles (001 to 00x) assumes that there is no other dataloader from that paper. do you think this is a valid assumption

It's fine if the numbering only holds for one sub group, mostly necessary to make otherwise identifcal IDs unique.

That's kind of how I implemented it.
Random code snippets of it:
    author = sfaira_questionary(function='text',
                                    question='Author(s):',
                                    default='Einstein, Albert; Hawking, Stephen')
        self.template_attributes.author = author.split(';') if ';' in author else author

        first_author = author[0] if isinstance(author, list) else author
        try:
            first_author_lastname = first_author.split(',')[0]
        except KeyError:
            print('[bold yellow] First author was not in the expected format. Using full first author for the id.')
            first_author_lastname = first_author

Not relevant to implementation but I would suggest you will get more unique dataset names by using first author rather than last author which are probably easier to remember.

Not relevant to implementation but I would suggest you will get more unique dataset names by using first author rather than last author which are probably easier to remember.

exactly my thoughts!

yes, agreed. how about this: self.author should be either the first author or a list of authors. if it's a list we just use the first member of that list.
I think for now self.author is still mainly last authors (I think this field used to be called self.lab), but I'll change that

I am adopting some of these rules in #102 already.

Are there defaults if .organis not set and .obs_key_organ is set instead? We could just set it to "x" in these cases.

Default is 'NA'.


There is something wrong with xcode 8.0(beta 3),expect to update as soon as possible!

Thanks!

I have the same error problem, related to #issue105. I think it is the Xcode8 problem, hoping Apple will fix it in the next version.

Duplicate of #140


Filename too long

Tried to clone the repository. Some png filenaming is too long for my windows pc. I am installing to an already nested file system. I don't want to change my file system for this interesting app. I have had similar issues in the past. Most repos are cloning okay for me. This filename you have is particularly long.
The file ends with ...[NEXUS_5_NOTNIGHT,fontScale=1.5].png
Theres three others like that one. Is it possible to reduce the file naming?

Argh, Windows file name restrictions...
Those files are automatically named via Paparazzi when generating and validating snapshots, so there doesn't look to be a super easy way to change them.
If you ignore those files, the rest of the app should run fine, the only thing that wouldn't work is the tests verifying those screenshots. I don't personally develop on Windows, so hopefully the rest does work!

Those files are automatically named via Paparazzi when generating and validating snapshots, so there doesn't look to be a super easy way to change them.
If you ignore those files, the rest of the app should run fine, the only thing that wouldn't work is the tests verifying those screenshots. I don't personally develop on Windows, so hopefully the rest does work!

Great answer. I will try forking again and see if I can integrate the app without those files. Maybe remove paparazzi entirely.


Install and invalid char :

Just trying to install and build but getting an invalid char ':' error.
go get -u https://github.com/asticode/go-astilectron-bundler.git
go get https:/github.com/asticode/go-astilectron-bundler.git: malformed module path "https:/github.com/asticode/go-astilectron-bundler.git": invalid char ':'

Looks like there's a bad path within a module but not sure.

The second line you pasted is weird :
https:/github.com/asticode/go-astilectron-bundler.git
It should be https://github.com/asticode/go-astilectron-bundler.git.
My guess is you should run go get -u github.com/asticode/go-astilectron-bundler instead of go get -u https://github.com/asticode/go-astilectron-bundler.git.

The second line you pasted is weird :
https:/github.com/asticode/go-astilectron-bundler.git
It should be https://github.com/asticode/go-astilectron-bundler.git.
My guess is you should run go get -u github.com/asticode/go-astilectron-bundler instead of go get -u https://github.com/asticode/go-astilectron-bundler.git.


testnet readme fixes

This PR improves the testnet readme with a few small fixes

[x] a typo is fixed
[x] some explanatory text is added about path_name
[x] replaced the potentially confusing {src_port} and {dst_port} with transfer
[x] a bit more explanatory text

Thanks for working on this! This relayer is amazing.

Also fixes an issue where
rly tx link {{src_chain_id}} {{dst_chain_id}}
needs to be
rly tx link {{path_name}}

Thank you very much for these fixes @nylira! Glad you like it!


RecyclerView IllegalArgumentException

I am using your library and instead of listview I am using RecyclerView.
I am getting the exception:
lang.IllegalArgumentException: Scrapped or attached views may not be recycled. isScrap:false isAttached:true
Will it work for recyclerview? if yes, can please provide me an example

Probably it is not a very convinient place, but you can find next notes in wiki:

...
Where coverView is a view from the list that will be animated.
Note, that it must not be a list item's root layout, since UnfoldableView will try to remove it from it's parent, and ListView and RecyclerView will not behave correctly in this case.

So you just need to wrap your "cover" view in the list into e.g. FrameLayout.


Unpacking zip file

I have downloaded the .zip file two times and i cannot open or extract it. Do you recommend me another software to extract the file or there is any issue with it? I am running Ubuntu 22.04.01 LTS full updated. Thanks for your attention! :)

Install p7zip and p7zip-full

thanks a lot pal but isn't working, the command extract only the wallpapers leaving behind empty folders and no .sh files. i will keep researching but if you manage it first i'll appreciate your help. I really felt in love with this theme :100:

You can use git clone, or download the tar.gz files on release page


refactor(fw,tests): fix address and hash types

🗒 Description

Creates the common/base_types.py file, which includes basic types other complex types use, such as Address, Hash, and other types that do not have any other dependency.
Completely removes to_address, to_hash and to_hash_bytes in favor of using the Address and Hash classes directly.
Adds Address to the list of types an Op callable can receive, and, instead of treating the Address as bytecode, it automatically detects the type and adds the appropriate Op.PUSH20.
FixedSizeBytes types can now be compared to strings and integers
Removes Op.PUSH20 across all tests when an Address type is used as a parameter of an Op.

Nice thing about this change is that it produces zero fixture changes, and slightly reduces lines of code.
TODO: Update docs.
🔗 Related Issues
Fixes #387
✅ Checklist

[x] All: Set appropriate labels for the changes.
[ ] All: Considered squashing commits to improve commit history.
[ ] All: Added an entry to CHANGELOG.md.
[ ] All: Considered updating the online docs in the ./docs/ directory.
[x] Tests: Included the type and version of evm t8n tool used to locally execute test cases:  e.g., ref with commit hash or geth 1.13.1-stable-3f40e65.
[ ] Tests: Ran mkdocs serve locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.

Not sure what docs should be added, the only thing I can think of adding the usage of the Address type when initializing accounts, within pre & post states etc here:
https://github.com/ethereum/execution-spec-tests/blob/main/docs/writing_tests/writing_a_new_test.md?plain=1#L103-L206
This PR got me thinking: I wonder if we should force all addresses to be of the Address type, pre and post to be Alloc types, and storage to be Storage. If we check for this when filling, it might catch some an unintentional additions.


[kn-admin] Improve domain set command

When I tried to use kn-admin domain set to config a custom domain, I found the below points that we can improve to provide a better user experience:

For the --selector description, we can add an example like: app=prod to help user understand how to use this flag, and also we can tell user the flag can be used multiple times with different selectors.
For the below codes, I talked with @zhanggbj and she agreed that we should check the error returned from splitByEqualSign function, and tell user the detail error message if the given selector is invalid. Also @zhanggbj suggests moving the selector parsing codes into PreRunE phrase.
https://github.com/knative/client-contrib/blob/f207c1f9a74266bebeb5a2a32f2a51cf8f9cd87f/plugins/admin/pkg/command/domain/set.go#L62
When I give the wrong flag value or miss the value, the command will output the error message, but also the whole usage will be printed. I think printing usage is a little verbose for user,  actually the user can use --help to show the usage details. I tried knative command and no such behavior, I think we'd better keep the same with kn command.
Here is an example for our domain command:

☕15:25 ➜ ~/Workspace/Code/github.com/client-contrib/plugins/admin/cmd [master ✘] ./kn-admin domain set -d test.com --selector
Error: flag needs an argument: --selector
Usage:
  kn admin domain set [flags]

Flags:
  -d, --custom-domain string   desired custom domain
  -h, --help                   help for set
      --selector strings       domain selector

Global Flags:
      --config string   config file (default is $HOME/.config/kn/plugins/admin.yaml)

@chaozbj Thanks for opening this!
for 2, I think we do need a precheck in  PreRunE for value of --selector and give an suggestion if the input is wrong, but maybe not moving whole splitByEqualSign to PreRunE

@chaozbj Thanks for opening this!
for 2, I think we do need a precheck in PreRunE for value of --selector and give an suggestion if the input is wrong, but maybe not moving whole splitByEqualSign to PreRunE

Ok, thanks, I will be careful when I am doing these changes.

@zhanggbj I made changes for this issue and created PR in my repo: https://github.com/chaozbj/client-contrib/pull/3, the changes are simple, please help me have a look when you are free.
I didn't create a formal PR in this repo since I think there are conflicts between our changes and github may can not automatically merge it. So I want to wait your changes to merge first, and then submit a formal PR for my changes.
Any problems, please let me know, thanks a lot!

/assign
I would like to work on it

@chaozbj  would you please also handle the comments for domain subcommand and some other wording as suggested here?
https://github.com/knative/client-contrib/pull/40#discussion_r444649879
https://github.com/knative/client-contrib/pull/40#discussion_r444652039
https://github.com/knative/client-contrib/pull/40#discussion_r444657724
https://github.com/knative/client-contrib/pull/40#discussion_r444660533

@zhanggbj ok, I will handle them.

The codes have been merged and would like to close.
/close


Incorrect object schema during migration

Goals
Inspecting the new schema during a migration and checking if a newly added primary key is marked as primary key.
Expected Results
Inspecting the realm object which has a newly added primary key to return correct data - e.g. "isPrimary" to be YES like below:
Subject {
		name {
			type = string;
			objectClassName = (null);
			linkOriginPropertyName = (null);
			indexed = YES;
			isPrimary = YES;
			array = NO;
			optional = NO;
		}
	}

Actual Results
isPrimary is incorrectly set to NO. Once the migration is over, inspecting the schema correctly recognizes the new primary key.
Steps to Reproduce
Add a new primary key and inspect migration.newSchema during a migration.
Version of Realm and Tooling
Realm framework version:  RealmSwift 3.1.1
Xcode version: 9.2
iOS/OSX version: 11.0
Dependency manager + version: CP 1.3.1

Primary keys are disabled during migrations because they can have some very surprising effects (e.g. you can accidentally merge all of your objects into a single object when adding a new PK column if you do anything at all before initializing the values of that column), and this is reflected in the schema reported during the migration as the schema object is the actual thing that we work off of internally and not a separate view.

Thanks, this explains the behavior, though it'd be helpful if that was stated in the migration docs.


Image 404s on logos page

Description of the bug:

The CDK8s logos page doesn't seem to render any of the images. Possibly some issue with the way mkdocs is resolving image paths.
Reproduction Steps:

Visit https://cdk8s.io/docs/latest/assets/logos/readme.html

This is :bug: Bug Report

Looks like the issue fixed itself.


Remove 2nd payload

Hello,
I am using fluent-plugin-loggly-syslog on a AWS EKS cluster in fluentd daemonset in order to forward the logs of my workload to Loggly. But apart from json: { ... } containing the information I want there is a 2nd payload which I do not need.
I'd like to remove this 2nd payload containing syslog: { severity: "...", appName: "...", priority: "...", facility: "...", timestamp: "..." }
Is there a way to filter it out?
Regards,
Sener

Hi @skoban-rmp those fields are automatically added by Loggly's backend because this plugin uses the Loggly syslog collection endpoint to send data to Loggly. You can switch to Loggly's HTTP endpoints instead to have these syslog values not show up.
Another option is fluent-bit which has a built-in HTTP output plugin that supports Loggly's HTTP bulk endpoint natively  (when you specify the json_lines format) without requiring any plugins. There is a configuration example in the AWS Firelens example repo and in this customer's Kubernetes configuration example repo.


corrected numer value for variance on RHS of eqn

The latex code in Bayes.Rmd also contains some errors or inconsistencies (w.r.t use of preceding 0.) in the line 299 around "\mbox{E}(\theta \mid Y=450) &\approx .285 "
...where numbers are plugged in to calculate B,: Here you use 0.110 and not 0.111. A tiny error but still confusing to a learner.
Also the bayes.Rmd says: var(theta | y) can be shown to be ... = 0.026.
IMHO the standard deviation sd (theta | y) should be the sqare root of this: sqrt(0.026) = 0.1612.
but at the end of the paragraph below you say the 95% credible interval:
this turns out to be: .285 ± 0.052.

I though confidence intervals at the 95% level are approx. SDs wide. Are 95%-level credible-intervals two variances wide?
Shouldn't this be two SDs wide: 2* 0.162 = 0.3225 - but this would still be meaningless (because the E(Theta | Y) = 0.285 < 0.3225
Ahh I see the video uses the correct values var = 0.00069, => sqrt(var) = 0.026

fixed thank you!


Assign uperf server port as an option

Signed-off-by: Sai Sindhur Malleni<EMAIL_ADDRESS>Description
This is needed for fixing an issue with running
multiple pairs using NodePort.
Related PR: https://github.com/cloud-bulldozer/benchmark-operator/pull/755

/rerun all


code is incomplete

There seems to be some necessary code missing in openselfsup.utils such as Registry etc.

Thanks a lot, I didn't actually configure the environment, sorry about that


Support for multiple CPIs (mvp)

This PR provides basic support for multiple CPIs in a single BOSH director.
The high level concept is outlined here: https://github.com/cloudfoundry/bosh-notes/blob/master/multi-cpi.md
@cppforlife supported me with implementation details and reviews of some code parts.
Some notable points:

The heart of the PR is the newly introduced CpiConfig which stores the multiple CPI endpoint configurations. CloudFactory then looks up the cpi config for a given AZ and returns an instantiated Cloud. To achieve this lookup, the cloud factory uses the latest cpi config and the deployment's cloud config (which contains the AZ -> CPI relation).
Every cloud interaction thus needs an up-to-date deployment.cloud_config. In some cases, i.e. while updating the deployment, deployment.cloud_config did not contain the most recent CloudConfig but an older one, because this association was only set and persisted after updating the deployment. I had to do some ugly hacks to set deployment.cloud_config before every cloud interaction. I guess this needs some improvements.
I added only very basic integration tests. The branch is built in BOSH's concourse and is successful. I never ran BATS tests though. Do we need some more integration tests or BATS tests?
I was having a hard time completely understanding the code parts of stemcell compilation. I’m not quite sure I changed it the right way, so I’d appreciate if you could look at this more carefully.
For stemcells, we extended the unique constraint to the new cpi attribute. But since postgresql seems to completely ignore the uniqueness constraint if one of the fields is NULL (which is the case for the cpiattribute when not using cpi config), I added a second validation. I assume this validation could fail if there is a stemcell with name=x, version=y, cpi=z in DB and we add one with name=x, version=y, cpi=nil. Is there a better way to do this? A custom validator? Ignore this case?
To be able to support the cpi config, each CPI needs to be modified to also be able to read the configuration from the context hash. We implemented this small change for the openstack cpi.
We tested this in our lab and were successfully able to stretch deployments over multiple openstacks. I will file the PR there as soon as this PR is merged and a new version of the bosh_cpi gem is ready for downstream use. (cc @voelzmo)

since postgresql seems to completely ignore the uniqueness constraint if one of the fields is NULL

@MatthiasWinzeler should be we use "" instead of NULL then. i would like us to have db enforced constraint is possible.

@cppforlife @MatthiasWinzeler question about the context hash handling in the CPIs, e.g. cloud_properties['openstack'].merge!(context['openstack']). I like that the global CPI configuration is now dynamic, but we seem to have this weird mix of dynamic values vs static values in the cpi.json file.
Since the expectation is for all cloud.properties.openstack values to be pulled at runtime from the context, I'd like to push back against the merge call and go back to overriding: cloud_properties['openstack'] = context['openstack']. Having to add properties to spec files and ERB templates seems like a waste when its only use is to provide default values. I think it would be easier as a developer to have default values in the code instead and make everything dynamic. I'd imagine that users don't look at the CPI spec file for defaults, but would use the bosh.io CPI docs instead.
Big thanks for the PR btw 👍

not everything, blobstore and nats still stay in the cpi for the moment. will pull them out of cpi completely at some point.
Sent from my iPhone

On Oct 14, 2016, at 3:33 PM, Lyle Franklin<EMAIL_ADDRESS>wrote:
@cppforlife @MatthiasWinzeler question about the context hash handling in the CPIs, e.g. cloud_properties['openstack'].merge!(context['openstack']). I like that the global CPI configuration is now dynamic, but we seem to have this weird mix of dynamic values vs static values in the cpi.json file.
Since the expectation is for all cloud.properties.openstack values to be pulled at runtime from the context, I'd like to push back against the merge call and go back to overriding: cloud_properties['openstack'] = context['openstack']. Having to add properties to spec files and ERB templates seems like a waste when its only use is to provide default values. I think it would be easier as a developer to have default values in the code instead and make everything dynamic. I'd imagine that users don't look at the CPI spec file for defaults, but would use the bosh.io CPI docs instead.
Big thanks for the PR btw 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@cppforlife agreed that cloud.properties.{registry,agent,blobstore} should come from cpi.json for now. But I would like for cloud.properties.openstack to come exclusively from dynamic values by removing the merge call.

@cppforlife @ljfranklin We decided for merge instead of a simple replacement because of properties like use_dhcp. Users mostly don't really care about those and don't set them in their manifest, because "it just works".
I'd like to keep it this way without compromising the idea that defaults in a boshrelase are exactly in one place: the spec file. The current way isn't ideal, I know, because now CPI configuration is coming from two places instead of just one (if you're using this new feature). Seemed like the lesser evil to me, though.

@voelzmo I agree that all defaults should ideally go in a single place. But a CPI can have default properties for vm_type.cloud_properties or disk_pool.cloud_properties that cannot live in the spec file. So you now have some defaults in your code (e.g. AWS disk_pools.cloud_properties.type) and some in the spec file (e.g. use_dhcp). So the "all defaults in one place" rule for CPIs has to be defaults defined in the code. Defaults in the code are also much easier to unit test as compared to ERB rendering.

@MatthiasWinzeler In the cpi_config you used for manually testing the feature, were your properties for each CPI constructed like this?
- name: blah
  type: aws
  properties:
    openstack:
      some-prop: foo
      some-other-prop: bar
...

If you could provide an example cpi_config based on the one you used, that would be great too.

@cppforlife Good idea, I will change the default to "" then.
@tjvman My cpi config looked like this (omitted the third cpi for better readability):
cpis:
- name: openstack1
  type: openstack
  properties:
    openstack:
      auth_url: https://api.openstack1:5000/v2.0/tokens
      tenant: openstack1
      username: openstack1-user
      api_key: openstack1-password
      connection_options:
        ssl_verify_peer: true
        read_timeout: 300
        write_timeout: 300
      config_drive: disk
      use_dhcp: false
      default_key_name: cloudfoundry
      private_key: /home/ubuntu/bosh-workspace/ssh/cloudfoundry
      default_security_groups: []
      human_readable_vm_names: true
      endpoint_type: publicURL
      state_timeout: 300
      boot_from_volume: false
      stemcell_public_visibility: false
      wait_resource_poll_interval: 5
      ignore_server_availability_zone: false
- name: openstack2
  type: openstack
  properties:
    openstack:
      auth_url: https://api.openstack2:5000/v2.0/tokens
      tenant: openstack2
      username: openstack2-user
      api_key: openstack2-password
      connection_options:
        ssl_verify_peer: true
        read_timeout: 300
        write_timeout: 300
      config_drive: disk
      use_dhcp: false
      default_key_name: cloudfoundry
      private_key: /home/ubuntu/bosh-workspace/ssh/cloudfoundry
      default_security_groups: []
      human_readable_vm_names: true
      endpoint_type: publicURL
      state_timeout: 300
      boot_from_volume: false
      stemcell_public_visibility: false
      wait_resource_poll_interval: 5
      ignore_server_availability_zone: false

Some of the properties are just defaults of the openstack cpi. They are now picked up by default (see @voelzmo 's comment above) and could most likely be omitted.

@MatthiasWinzeler @voelzmo Thanks for this PR! We've been looking over the code and doing local testing and it's looking good so far. There are a couple issues that we'd like to bring to your attention before we finalize the review, though.

When using cpi_config, if you run bosh task x --debug then the logs will contain any credentials that were passed in the request['context'] to the CPI; redact creds in the context  before sending them to debug logs. See bosh_cpi/lib/cloud/external_cpi.rb:84.

Always using the newest cloud_config when performing CPI calls breaks bosh recreate functionality (may break other things). We would like to preserve the versioning functionality for the CloudConfig, but from your initial PR comments it seemed like always using the latest CloudConfig was necessary. Can you give us some more context on why the newest CloudConfig has to be used so we can work together and figure out a solution that allows both the new PR and desired pre-existing functionality to work?

@cppforlife @MatthiasWinzeler I realize this thought is coming a little late, but do we actually need a separate cpi-config file and command or could we just add those properties to the cloud-config?
The distinction is more or less an implementation detail as the cloud config is already full of properties that will be passed to the CPI. I'm worried users not familiar with the internals of BOSH won't understand why some IaaS properties need to go in cloud-config and others in cpi-config. I'd imagine this also simplifies the multi-cpi implementation and solves the issue mentioned by @tjvman where multi-cpi could break bosh recreate. Let me know if I'm missing something.

thinking behind separation was that cpi config isnt versioned just like cloud config. its likely that you will have to update/shift configuration over time for your cpi (cred change, etc) and i dont think we want users to have to redeploy for that.
Sent from my iPhone

On Oct 26, 2016, at 7:44 AM, Lyle Franklin<EMAIL_ADDRESS>wrote:
@cppforlife @MatthiasWinzeler I realize this thought is coming a little late, but do we actually need a separate cpi-config file and command or could we just add those properties to the cloud-config?
The distinction is more or less an implementation detail as the cloud config is already full of properties that will be passed to the CPI. I'm worried users not familiar with the internals of BOSH won't understand why some IaaS properties need to go in cloud-config and others in cpi-config. I'd imagine this also simplifies the multi-cpi implementation and solves the issue mentioned by @tjvman where multi-cpi could break bosh recreate. Let me know if I'm missing something.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@ljfranklin @cppforlife Furthermore, cpi-config contains credentials and should potentially handled differently by admins and also the director than the rest of the cloud-config.

Thanks for reviewing the changes!
I pushed most of the requested changes and will complete the remaining open points in the coming days.

Always using the newest cloud_config when performing CPI calls breaks bosh recreate functionality (may break other things). We would like to preserve the versioning functionality for the CloudConfig, but from your initial PR comments it seemed like always using the latest CloudConfig was necessary. Can you give us some more context on why the newest CloudConfig has to be used so we can work together and figure out a solution that allows both the new PR and desired pre-existing functionality to work?

It's best explained when looking at this integration test which tests the switch from non-cpi-config to cpi-config:

Deploy without cpi-config
Create cpi-config
Update cloud-config (add CPIs to AZs)
Deploy

My assumption was that this works out of the box. Without my ugly fix, this did not work - BOSH used the older cloud config while deploying, so it did not know which AZ uses which CPI and thus deployed to the director cpi.
You can easily reproduce this by removing my hack and run the integration test above.
Maybe this is some other bug?

Pushed some last improvements - it's ready now for another review - and merged current develop. integration-1.9-postgres has some broken specs, but it's the same on develop.

@MatthiasWinzeler we are planning to take a look at the changes next week.

@MatthiasWinzeler Thanks for making those changes. We looked into how the change to when the deployment model gets updated, and we believe we've identified the source of the bug as well as a way to fix it.
With that in mind, we intend to merge this PR as it currently stands and make our change to how deployment.cloud_config works on top of that.

@MatthiasWinzeler the cpi-config and update-cpi-config has also been ported over to the go bosh cli

Nice work @MatthiasWinzeler !  Thanks tons for contributing this!


Utils from binary release WPS-CMake 4.1.0 are no relocatable

Tested with: wps-cmake-4.1.0-basic_nesting-serial-x64-linux-release.tar.xz
The executables under util directory doesn't define the RPATH on the binary dynamic section.
For example util/calc_ecmwf_p.exe:
$  objdump -p util/calc_ecmwf_p.exe
....
Dynamic Section:
  NEEDED               libgfortran.so.5
  NEEDED               libm.so.6
  NEEDED               libgcc_s.so.1
  NEEDED               libc.so.6
....

[some output ommited]
It can be fixed like:
$ patchelf --force-rpath --set-rpath "\$ORIGIN/../.libs" --replace-needed libgfortran.so.5 libgfortran-2e0d59d6.so.5.0.0 util/calc_ecmwf_p.exe
$ objdump -p util/calc_ecmwf_p.exe
.....
Dynamic Section:
  RPATH                $ORIGIN/../.libs
  NEEDED               libgfortran-2e0d59d6.so.5.0.0
  NEEDED               libm.so.6
  NEEDED               libgcc_s.so.1
  NEEDED               libc.so.6

the fixing includes also to point the right SO.
Regards

Thanks for the report. This is fixed now in the re-released 4.1.0.


Headless service listener behavior confusion

Bug Description
as https://istio.io/latest/docs/ops/configuration/traffic-management/traffic-routing/#headless-services say.
if service is headless, listener will build POD_IP:PORT for these pod.
apiVersion: v1
kind: Service
metadata:
  name: tunnel-internal-svc
spec:
  clusterIP: None
  clusterIPs:
  - None
  ports:
  - name: default
    port: 9996
    protocol: TCP
    targetPort: 9996
  selector:
    app: tunnel
  sessionAffinity: None
  type: ClusterIP
status:

it's works.  build some listener like
<IP_ADDRESS>_9996
<IP_ADDRESS>_9996

but i modify name to http
apiVersion: v1
kind: Service
metadata:
  name: tunnel-internal-svc
spec:
  clusterIP: None
  clusterIPs:
  - None
  ports:
  - name: http
    port: 9996
    protocol: TCP
    targetPort: 9996
  selector:
    app: tunnel
  sessionAffinity: None
  type: ClusterIP
status:

it's not work. listener like
<IP_ADDRESS>_9996

it's behavior make us confusion …… maybe add more doc for this condition
Version
$ istioctl version
client version: 1.9.5
control plane version: 1.9.5
data plane version: none

Additional Information
No response

relating issue https://github.com/istio/istio/issues/5992

@yanickxia can you upgrade to 1.12？ AFAIK, 1.9 is EOL.

@zirain  i try repeat unittest in v1.12, it's still exist.

Yeah the doc is a bit misleading here - its hard to describe complex behavior in a doc :-) https://istio.io/latest/docs/ops/configuration/traffic-management/traffic-routing/#http describes this a bit though - HTTP is header based.

For tcp, you can find the listeners are very identical, the diff is the address IIUC. For http, the host is different, so different http instances share one listener if they use one same port.

Base this condition, make something case difficult to understand .

two headless service selector same pod (headless-svc-1, headless-svc-2)
enable automatic protocol selection
headless service port name is unkonwn (such as tunnel, xxx, yyy)

when you try curl to some headless pod
curl <IP_ADDRESS>:9996

with miss HOST, this request will return 404.
but add Host maybe connect succeed。because Route Match one of (headless-svc-1, headless-svc-2). it's Undefined Behavior.

otherwise if modify port name to tcp, it's meaning disable http filter. it's request will be ok. everything go to TCP_PROXY

That's an interesting found, can you show the config dump of both listener and route for this service.

Ok. @hzxuzhonghu , upload dump files:

service.yaml:  headless service
listener.json: dump listener json
route.json: dump route json

dump.zip

I think latest Istio (not yet released) makes the without-host work


[DEV] cleanup images older than 90 days

github says they do this, but they don't.
ublue started doing this weekly in an effort to reduce wasted disk space as a courtesy since we're using this storage for free :)

https://github.com/secureblue/secureblue/pull/738


Update article.md

Code snippets for TextView  and Button were not available on documentation page.

Nice catch! Thank you @AmitGurbani


install a systemd user service unit

add a systemd --user unit/service file, so that one can run waybar as a
--user systemd service.
this feature is automatically enabled if systemd is found, but can be disabled
with -Dsystemd=disabled

I'm not sure about WantedBy=sway-session.target since waybar might be used for other wayland compositors too. People have suggested wayland-session.target on the internet, so that might be a good option.
(does Waybar run on other wayland compositors?)

Thanks!
Yeah 'wayland-session.target' is better since waybar run on all wlroots compatible compositor

updated to wayland-session.target

Nice, thanks ❤

it just dawned on me that I didn't account for the possibility that waybar is not installed in /usr/bin/waybar. Not sure if that's expected, but if it is, I'll send a patch to make the service an .in file.


[Frontend] UI to answer the questions in Q&A Section

Is your feature request related to a problem? Please describe.
Currently, we only have questions list in Q&A section of the website, so user won't be able to answer to those questions.
Describe the solution you'd like
Add an ui where any user can answer the question. These questions should get saved using the post answer api

I want to work on this issue.

@TejasNasre  please go ahead, happy learning :)

Please Specify The Endpoints

Please Specify The Endpoints

its already present in the issue description, please check

Hey @Kajol-Kumari
I'm interested to work on the issue


Package missing in search from within editor settings

Any idea why the https://atom.io/packages/language-nesc package doesn't show up in the search results within the editor settings?

It installs just fine via apm:

The search field you're looking for is on the Install tab now. The field you're editing in your screenshot is more accurately a filter, filtering the packages that are already installed.

DOH!...thanks :+1:


 When I try to add a Hawck script, I get the following error:
$ hawck-add ~/.config/hawck/scripts/example.hwk
lua5.3: ./LLib/Keymap.lua:261: Unable to find keymaps directory.
stack traceback:
	[C]: in function 'error'
	./LLib/Keymap.lua:261: in function 'Keymap.getall'
	./LLib/Keymap.lua:281: in function 'Keymap.new'
	./LLib/kbd.lua:36: in main chunk
	[C]: in function 'require'
	./LLib/Hawck.lua:35: in main chunk
	[C]: in function 'require'
	./init.lua:2: in main chunk
	[C]: in function 'require'
	[C]: in ?

The two directories it's looking through don't exist though. I tried adding another directory after this line. I found some keymaps in "/usr/share/ibus/keymaps" (including the "us" keymap, which is the one I'm interested in; I'm using an unmodified "us" layout), so that's the path I added. Now it gives me this error:
$ hawck-add ~/.config/hawck/scripts/example.hwk
lua5.3: ./LLib/Keymap.lua:284: No such keymap: no
stack traceback:
	[C]: in function 'error'
	./LLib/Keymap.lua:284: in function 'Keymap.new'
	./LLib/kbd.lua:36: in main chunk
	[C]: in function 'require'
	./LLib/Hawck.lua:35: in main chunk
	[C]: in function 'require'
	./init.lua:2: in main chunk
	[C]: in function 'require'
	[C]: in ?

Looks like it's checking for Norwegian, but it's not in that directory. Not sure if there's another place on Fedora 32 that has all the keymaps. Is the above behavior of crashing when not finding Norwegian a bug?

Did you edit ~/.local/share/hawck/cfg.lua?
$ cat ~/.local/share/hawck/cfg.lua                                                                                local config = {
  keymap = "us", -- You want to change this line
  disable_on_err = false,
  eval_keyup = true,
  eval_keydown = true,
  eval_repeat = true,
  disabled = false,
  notify_on_err = false,
}
return config

Also, at least on Debian/Ubuntu you need to install a package in order to get the console keymaps, I don't know if that's the case on Fedora as I've never used it. But that's one thing you could look into.
You definitely haven't done anything stupid, none of this is standardized, so it's a bit of a pain to make it work across many distros. I think from now on I'll just bundle the console keymaps together with Hawck to keep this issue from popping up, I'll reference this issue when I get that done.

I think that on Fedora keymaps are located at /lib/kbd/keymaps/.

@silenc3r I see a lot of .map.gz files inside /lib/kbd/keymaps/xkb. Are these the keymap files?

@elmehalawi If adding that keymap directory works on Fedora, then a pull request for it would be welcome.
Sorry for the late response.


Fan is listed in Home-App but cant control

My Fan cant be controlled. I have the new V2 one.
Here are my confg.json and the Terminal Protocoll:
`{
"bridge": {
"name": "Homebridge",
"username": "CC:22:3D:E3:CE:30",
"port": 51826,
"pin": "031-45-156"
},
"platforms": [{
"platform": "NelloPlatform",
"name": "nello.io",
"username": "t",
"password": "...",
"clientId": "...",
"lockTimeout": 5000,
"locationUpdateInterval": 3600000,
"exposeReachability": true,
"motionSensor": true,
"motionTimeout": 5000,
"alwaysOpenSwitch": false,
"homekitUser": "home bridge"
}, {
"platform": "TplinkSmarthome",
"name": "TplinkSmarthome"
},
{
"platform": "DysonPlatform",
"name": "DysonPlatform",
"email": "Email",
"password": "PW",
"country": "DE",
"accessories": [{
"ip": "<IP_ADDRESS>",
"displayName": "Dyson",
"serialNumber": "A1A-EU-KKB2XXXA"
}]
}]

}`
[2019-4-27 23:46:44] [DysonPlatform] Use device password from account
[2019-4-27 23:46:44] [DysonPlatform] Dyson IP:<IP_ADDRESS> Serial Number:DYSON-A1A-EU-SERIAL-527
[2019-4-27 23:46:44] [DysonPlatform] Device serial number format valids
[2019-4-27 23:46:44] [DysonPlatform] Device cached. Try to update this
[2019-4-27 23:46:44] [DysonPlatform] Init Sensor for Dyson
[2019-4-27 23:46:44] [DysonPlatform] Init Fan State for Dyson
[2019-4-27 23:46:44] [DysonPlatform] Auto switch found. Remove this now
[2019-4-27 23:46:44] [DysonPlatform] Night mode button is added
[2019-4-27 23:46:44] [DysonPlatform] Heat Available. Add Heat button and jet control
HAP Warning: Characteristic 000000BF-0000-1000-8000-0026BB765291 not in required or optional characteristics for service 000000B7-0000-1000-8000-0026BB765291. Adding anyway.
[2019-4-27 23:46:44] [DysonPlatform] Jet Focus mode button is added
[2019-4-27 23:46:44] [nello.io] Updated webhook for door with ID 12e56d12-14ba-4f7f-bfec-60393b985c7c to https://nello-socket.alexdev.de/callback/hak-wiisaza-libcirdi-uka-kolmerhi-9279.
I noticed that die Serial is probably in the wrong format in the Terminal Text. Serial Numer: DYSON-Serial-527. Where come the 527 from? I dont have that serial number in the old format.
Is there maybe the problem?
Can someone help me pls? Thanks!

the word "DYSON" and "527" were just derived automatically from you account info so it should be fine.
Did you see something like "Connected to . subscribe now" in the log?

I think it was the Port 1883 that were blocked by the Router. The Problem is solved. Thanks.


Upgrade to Zig 0.13

This change is part of the following stack:

#19 ◀

Change managed by git-spice.


sdcnet

	mask_prefix = "gtFine" 	colormask_prefix = "gtFineColor" 	rgb_prefix = "leftImg8bit" 	sequence_prefix = "leftImg8bit_sequence" 	split = "train"

What is 'leftImg8bit_sequence' in the ./sdcnet/sdc_aug.py

The file is leftImg8bit_sequence_trainvaltest.zip available from cityscapes.
30-frame snippets (17Hz) surrounding each left 8-bit image (-19 | +10) from the train, val, and test sets (150000 images)


Code issue

When I run the feature_pu_model.py I get the following error:
Traceback (most recent call last):
File "feature_pu_model.py", line 11, in
from utils.data_utils import DataPrepare
ModuleNotFoundError: No module named 'utils.data_utils'
Could you please help. Thank you.

change the file name such like uttl, and change the code utils to uttl too.


Router

Upgrade to ui-router and rewrite all route controllers as components.

Pull Request Test Coverage Report for Build 779

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.007%) to 54.239%

Totals

Change from base Build 772:
-0.007%

Covered Lines:
2815

Relevant Lines:
4858

💛  - Coveralls


Tooltip rebuild child widget on mouse reentering Flutter window

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Clone the example project below.
Build on a desktop platform (confirmed on Linux and Windows), then move the mouse cursor in and out of that window.
This does not happen on the web platform.
Expected results
Text always remaining the same after the initial delay of one second.
Actual results
The FutureBuilder's connection state briefly flashes between waiting and done whenever the mouse enters the window. The Future is not run again, so this is purely within the builder function in the FutureBuilder.
This is apparently triggered by the Tooltip widget. The FutureBuilder is not responsible, it also happens with other widgets (the FutureBuilder just makes it easy to see). It seems like it's re-evaluating the child widget again when you leave/enter the window. I added a print statement to demonstrate this in the example project.
Step to reproduce on other widgets:

open performance overlay
leave and enter window
the performance overlay will move by one frame

Code sample
https://github.com/anlumo/future_flash/
Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.10.3, on Ubuntu 23.04 6.2.0-20-generic, locale en_US.UTF-8)
[!] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    ✗ No Java Development Kit (JDK) found; You must have the environment variable JAVA_HOME
      set and the java binary in your PATH. You can download the JDK from
      https://www.oracle.com/technetwork/java/javase/downloads/.
[✓] Chrome - develop for the web
[✓] Linux toolchain - develop for Linux desktop
[!] Android Studio (not installed)
[✓] VS Code (version 1.78.2)
[✓] Connected device (2 available)
[✓] Network resources

! Doctor found issues in 2 categories.

Replicable on Linux using bspwm for window manager
Moving window focus using keyboard doesn't trigger this problem

This looks like a duplicate of https://github.com/flutter/flutter/issues/117627. Feel free to reopen if that's not the case or if the issue isn't fixed on the master branch.


Defaultdict with set does not work

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
I cannot define an attribute with a defaultdict using a set. list works perfectly fine.
Example Code
from collections import defaultdict
from typing import DefaultDict, Set

from pydantic import BaseModel

class Test_Set(BaseModel):
    test_dict: DefaultDict[str, Set] = defaultdict(set)

# pydantic.errors.PydanticSchemaGenerationError: Unable to infer a default factory for with keys of type typing.Set.
# Only tuple, set, dict, list, bool, int, float, frozenset, str are supported, other types require an explicit default factory set using `DefaultDict[..., Annotated[..., Field(default_factory=...)]]

Python, Pydantic & OS Version
pydantic version: 2.1.
pydantic-core version: 2.4.0
pydantic-core build: profile=release pgo=true mimalloc=true
install path: /venv/lib/python3.11/site-packages/pydantic
python version: 3.11.0rc1 (main, Aug 12 2022, 10:02:14) [GCC 11.2.0]
platform: Linux-5.19.0-46-generic-x86_64-with-glibc2.35
optional deps. installed: ['email-validator', 'typing-extensions']

Selected Assignee: @samuelcolvin

Thanks @orfisko  for reporting this issue 🙏
I've created a PR to fix it.

@hramezani you're welcome, thanks for picking it up this quickly!

Thanks @orfisko for reporting.


Make the requirement about "pandoc" is configurable

Hi,
I am using stlink for several months,  I buit it on Cygwin, but after recent upgrades, I cannot able to build it out successfully. Seems that "pandoc" is required to generate the manpages.
Is it possilbe to make it configurable? For example, "cmake MANPAGES=NONE".
Or if it cannot find the "pandoc", it will not generate the manpges, just as it checks for the "GTK".
Thanks & Regards,
Gerry

You are right, currently the Release target will not generate the man-pages with pandoc. The Debug target will. I will patch it so we have cmake option like: STLINK_ENABLE_DOC (like nanomsg does: https://github.com/nanomsg/nanomsg/blob/master/CMakeLists.txt#L90-L97)

Seems that it is fixed, right?
I built it on debian 8.6,  though I have not install pandoc, it can be built out successfully.
I also verified this on Cygwin.

Yes. The fix is not clean, but it now works without pandoc installed.
2016-09-29 15:10 GMT+02:00 gerryqd<EMAIL_ADDRESS>
Seems that it is fixed, right?
I built it on debian 8.6, though I have not install pandoc, it can be
built out successfully.
I also verified this on Cygwin.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/texane/stlink/issues/479#issuecomment-250461296, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEeH8EquMoiOteW3DsRuw-_CgBt4Xraks5qu7jFgaJpZM4KCsU4
.

@gerryqd could you try on latest master?

Related to #487, fixed in commit https://github.com/texane/stlink/commit/68b0f3bddc3c4aaffe34caa6a3201029edd8ad56.


Audit review fixes

In addition to the comments in the review, there are a couple additional minor issues I noticed.

There are a number of locations where "emitted" is spelled "emmitted", which we should fix.
This comment should be removed.


Please explain what a query string for this method can look like

this documentation page should maybe explain what the query-string can actually be, or point to another page that explains the possible query syntax. I am new to this function I have no idea what a query against a datalake-file could actually be or what syntax it could have. there is no chance I will randomly guess a working query, so there should be more information about the query string beyond it beeing a string and it beeing a query
Thank you :)
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 08afb360-f71f-5667-d695-546e531313ac
Version Independent ID: 3ca63d38-4be0-5e83-3100-265697dece59
Content: DataLakeFileClient.Query(String, DataLakeQueryOptions, CancellationToken) Method (Azure.Storage.Files.DataLake) - Azure for .NET Developers
Content Source: xml/Azure.Storage.Files.DataLake/DataLakeFileClient.xml
Service: azure
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

@BitBender1 Apologies for the late reply. Thanks for the feedback. We have created a PR Azure/azure-docs-sdk-dotnet#2141 to address this issue. It is currently pending on review.


Error in FirebaseError.h

Hi, can anyone help me because I keep getting this error in Arduino IDE regarding a certain code in FirebaseError.h. I've tried to look at it in the header file but I found nothing was wrong.
firebase-arduino-master\src/FirebaseError.h:6:1: error: expected ',' or ';' before 'enum'
6 | enum FIREBASE_ERROR_CODES {
| ^~~~
exit status 1
Error compiling for board NodeMCU 1.0 (ESP-12E Module).
These are the errors that I've received in Arduino. I don't know what is the problem

Have you found the solution for the error ?


DNSMonitor doesn't work with ClusterServersConfig

I noticed that Redisson isn't detecting DNS changes when using the ClusterServersConfig with AWS Elasticache Redis in cluster mode. After debugging it looks like this line is never hit https://github.com/redisson/redisson/blob/master/redisson/src/main/java/org/redisson/connection/MasterSlaveConnectionManager.java#L289
I am using a CNAME DNS record that points to the Configuration endpoint of the Redis cluster and then testing destroying/recreating the cluster and pointing the CNAME at the new configuration endpoint.
Is this expected? What's the expected behavior here?

@aoskotsky-amplify  which  version of redission

@aoskotsky-amplify I have solved an issue with DNS resolving in last version. Have you tried it?

I tried it with version 3.5.7

Oh, sorry cluster is not supported yet. Which Redis cloud vendor do you use?

DNS resolving for Cluster hasn't been implemented. I'm going to implement it in next version

@aoskotsky-amplify  I'm still thinking how to do it in best way. In Redis cluster there are no single node used to read configuration. Redisson is always trying to read cluster config from any node in cluster since every node has the same configuration. Cluster config usually use ip rather than domains. This why I decided to postpone dns resolving implementation for cluster setup. What do you think?

AWS Elasticache provides a domain name to use. IPs can change as nodes go up and down so I need to use the domain name that Elasticache gives.
Why not use the original node address thats provided in the config and then resolve it to a list of IPs each time it's needed? This could be cached if performance is an issue but it would be very useful if Redisson eventually picked up the new IPs when a DNS change happens for a AWS Redis cluster.

@mrniko does Redisson cache InetSocketAddress? see https://github.com/lettuce-io/lettuce-core/issues/144

@aoskotsky-amplify I got you. I'll research that case. Thank you!

@johnou no, it doesn't. all InetSocketAddress bound to client object. If client goes down or its IP has been updated then this client is shutdown.

Done

Is this problem solved?  I tryed ,but did not work,  because of netty cached dns and will not update dns relation.

@WuKongQC
Which netty version do you use? Could you provide netty dns resolver log?

version info     redisson:3.13.6      netty: 4.1.52.Final
redis config:
redisson:
# 集群节点，格式：redis://ip:port
address:
- redis://dcluster-redis:6380
first, set hosts
<IP_ADDRESS> dcluster-redis
redisson log:
[DEBUG] 2020-10-15 16:53:50.125 [] [] [nioEventLoopGroup-5-5] ClusterConnectionManager.lambda$updateClusterState$10[453]:cluster nodes state got from /<IP_ADDRESS>:6380:
than, changed hosts
<IP_ADDRESS> dcluster-redis
in redisson log, it use old IP  :
[DEBUG] 2020-10-15 16:55:20.503 [] [] [nioEventLoopGroup-5-5] ClusterConnectionManager.lambda$updateClusterState$10[453]:cluster nodes state got from /<IP_ADDRESS>:6380:
no netty log, I debug into netty and find netty used cached dns relation.
dns code in class:  io.netty.resolver.dns.DnsNameResolver  doResolveAll

NoopDnsCnameCache.INSTANCE is used by default as resolveCache instance

You need to adjust TTL returned by the DNS server

not through DNS server,   I set domain to  ip relation in operate system hosts file .
I added one line code as follow, when create RedissonClient Bean.   solved this problem.
//class org.redisson.config
config.setAddressResolverGroupFactory((var1, var2)->DefaultAddressResolverGroup.INSTANCE);

@WuKongQC do you use a security manager? by default if you set one, java will ignore the ttl and return the first resolved ip(s). To work around this you can set the ttl when bootstrapping your application but it must happen before the first socket connection..
public class Main {

    static {
        java.security.Security.setProperty("networkaddress.cache.ttl", "10");
        java.security.Security.setProperty("networkaddress.cache.negative.ttl", "10");
    }

    ...
}

https://docs.oracle.com/javase/7/docs/technotes/guides/net/properties.html
networkaddress.cache.ttl
Specified in java.security to indicate the caching policy for successful name lookups from the name service.. The value is specified as integer to indicate the number of seconds to cache the successful lookup.
A value of -1 indicates "cache forever". The default behavior is to cache forever when a security manager is installed, and to cache for an implementation specific period of time, when a security manager is not installed.

I have tryed this method, does not work.
the root cause is that netty bootstrap's dns cache never expire, that is what i saw in netty source code.

@WuKongQC
Probably it's correct behaviour for dns entries stored in file. I would recommend to use MaraDNS instead https://maradns.samiam.org/


Windows agent image build failure

Description
Error for window agent image build failure
azure-arm.image: Downloading package from https://msedgedriver.azureedge.net/131.0.2903.112/edgedriver_win64.zip to D:\temp\edgedriver_win64.zip...
azure-arm.image: Package downloaded in 0.08 seconds
azure-arm.image: Expand Microsoft Edge WebDriver archive...
azure-arm.image: Expand archive 'D:\temp\edgedriver_win64.zip' to 'C:\SeleniumWebDrivers\EdgeDriver' directory
==> azure-arm.image: Signature thumbprint do not match expected.
==> azure-arm.image: At C:\Program Files\WindowsPowerShell\Modules\ImageHelpers\InstallHelpers.ps1:997 char:9
==> azure-arm.image: +         throw "Signature thumbprint do not match expected."
==> azure-arm.image: +         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
==> azure-arm.image:     + CategoryInfo          : OperationStopped: (Signature thumb...match expected.:String) [], RuntimeException
==> azure-arm.image:     + FullyQualifiedErrorId : Signature thumbprint do not match expected.
==> azure-arm.image:
Platforms affected

[X] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] Ubuntu 24.04
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] macOS 15
[ ] macOS 15 Arm64
[ ] Windows Server 2019
[X] Windows Server 2022
[ ] Windows Server 2025

Image version and build link
Image 20241215.1.0
Is it regression?
N/A
Expected behavior
Image build completes successfully
Actual behavior
error pasted in description.
Repro steps
NA

Hi @jaswant-dnv , Thank you for bringing this issue to our attention. We are looking into this issue and will update you on this issue after investigating.

Hi @jaswant-dnv,  EdgeDriver signature has been updated, Could you please retry by forking latest code from main branch.
Closing this issue,Thank you.


Added ability to remove entities from the world

Exist = false may not be performant way to remove enitites.
Imagine a game when there are bullets, lots of small short lived objects. If they are all stored in the world the game would run very slowly after a while not to mention memory footprint.

Good to be able to remove Entitys.
Some thoughts:

would it be faster to use a map, instead of a list? (we could benchmark this? ) - not really related to the code you just wrote;
would it be faster if we did some "garbage collection" of our own? (i.e. delete all non-exiting elements every xth frame? or every yth garbage item?)

Note that removing one element (each time you need it) might be costlier than not doing it on-the-fly.
For now, it's nice that we can actually remove entities :-).

I agree. Anything else seems like a premature optimisation. Lets wait until we have #40 happening with tests before we do that.

I'm wondering. Would it make sense to write a test for this? I figure we will do it eventually, why not start now?
Thoughts @EtienneBruines @matiwinnetou?

Not sure what we would be testing. Benchmark would make more sense?

@paked @EtienneBruines this is core code so test would make sense (to catch a regression). As far as benchmarking, I dunno, perhaps it is easier to operate on a vector then to recreate arrays like crazy. Map I am not sure why, do we need a key value map?

@EtienneBruines stupid me, map will be perfect for this use case, but only if we merge uuid generator for entities first. Keys will be entity ids

Sounds good to me
On 10:29, Mon, 02/11/2015 Mateusz Szczap<EMAIL_ADDRESS>wrote:

@EtienneBruines https://github.com/EtienneBruines stupid me, map will
be perfect for this use case, but only if we merge uuid generator for
entities first. Keys will be entity ids
—
Reply to this email directly or view it on GitHub
https://github.com/paked/engi/pull/48#issuecomment-152876237.

@paked @EtienneBruines review new code please?

Code LGTM. Haven't had the time to test it yet, but I assume it works ;-).

:+1: for adding in a mutex.

I would disagree, it only makes sense if we make a decision to multithread. A lot of UI engines are single thread only and this is a general question if we want to answer ourself if we want the engine to use the power of multiple cpus at the higher complexity to reason about the code and deal with whole topic of potential nightmare of concurrency bugs.

I do get your point as to "worry about it when we start to multithread", since we have to look at a lot of code to make that happen.

@matiwinnetou good point. Thinking about it we actually don't use any concurrent features at the moment. Until we do, I do not see the point of adding this in.

LGTM.

Thanks for the contribution


Need to expose all modules based on extension

I need to expose all modules based on extension when combining bundle. I have tried this, but it didn't work (got me [Error: write after end] for each file).
.on('file', function (file, id) {
      if (id.substr(id.length - 4, 4) === 'nunj') b.require(id)
})

Is there any other way to hook into pipeline where b.require can be called for each file?
P.S. need this to overcome a problem with nunjucksify

@w0rm It looks like you solved your problem another way, but are you still interested in what you originally asked about, being able to expose all modules with a certain file extension?


errors with nightly zig to "build --release" on windows

I get the following 2 errors when trying to build any of the release type (fast, small, safe) with zig-master.
I tried with:

zig-windows-x86_64-0.14.0-dev.1457
zig-windows-x86_64-0.14.0-dev.1417

C:\Users\user\AppData\Local\zig\p\12203578c4326bd2088018f5d0f5ad395b6f4ea8228ceec99b60274c4329e832aa5c\src\civetweb\civetweb.c:714:9: error: 'MUST_IMPLEMENT_CLOCK_GETTIME' macro redefined
#define MUST_IMPLEMENT_CLOCK_GETTIME
        ^
<command line>:9:9: note: previous definition is here
C:\Users\user\AppData\Local\zig\p\12203578c4326bd2088018f5d0f5ad395b6f4ea8228ceec99b60274c4329e832aa5c\src\civetweb\civetweb.c:22173:20: error: expansion of date or time macro is not reproducible
  const char *bd = __DATE__;

can be reproduced with:
zig build run_minimal --release=safe

There is no problem with zig-0.13 to "zig build --release" on windows, nor is there any problem with zig-master to just "zig build".
The issue only occurs when using zig-master to "build --release" on windows.

This seems to be a master bug
I'll try a minimal reproduction and report upstream

I solved the problem by modifying the code in civetweb.c. I'm not sure if it's the best practice, but it does work.
Code modification at civetweb.c:714:
#if !defined(WIN_PTHREADS_TIME_H)
#ifndef MUST_IMPLEMENT_CLOCK_GETTIME
#define MUST_IMPLEMENT_CLOCK_GETTIME
#endif
#endif

Code modification at civetweb.c:22173:
The compiler complained about not being able to return a the certainty value, so we just need to provide a simple string.
const char *bd = "Sep 13 2024";

Maybe this issue should be raised directly to civetweb