Feature/agency timezone

Adds agency_timezone and some other improvements:

No more None-to-None as route long name
Ignore deleted relations
Searches ways as well as nodes in order to find a timezone for a relation

You will notice that the number of agencies are way smaller than previous script, the reason is ignoring non-transport routes for finding agencies.

No more None-to-None as route long name

What would be used now?

Ignore deleted relations

what do you mean by this? Is this about the cross-border thing?

What would be used now?

Nothing. Long names are optional when the short name is provided.

what do you mean by this? Is this about the cross-border issue?

OSM files contain historical data which are sometimes tagged deleted. We don't need them.

@mehdisadeghi I just ran the script. Quick questions

what is the purpose of the random characters in the filename?
doesnt that make it more complicate to use in an automatic way? lets say run the script programatically, get the output file and store it in firebase/aws?

what is the purpose of the random characters in the filename?

I have not yet any option to accept the filename as an input parameter and I wanted to make sure that every time that I run the program I get a new file and won't get an error that the same file already exists. So this was for my own convenience to be changed later.

doesnt that make it more complicate to use in an automatic way? lets say run the script programatically, get the output file and store it in firebase/aws?

This makes sense. My initial idea was to write the zip file to standard output when no file name was provided. However I ran into some issues writing a zip file so I decided to just write it in a file and improve it later. There is a limitation here, at the moment I can only write the zip file into a real file on disk and not in a stream (unless I use an external library to write zip files to streams).

There is a limitation here, at the moment I can only write the zip file into a real file on disk and not in a stream (unless I use an external library to write zip files to streams).

I see. For the time being I am ok with living with that limitation. For the future I guess that providing creating a stream that can be passed upwards would be ideal (thinking of a AWS lambda function or similar)


[Proposal] Support ClusterSpec Propagation Feature in TF 1.14

Goals
Since Tensorflow 1.14, TensorFlow supports Cluster Propagation feature which "allows TensorFlow workers
to be booted independently of each other, and with no knowledge about others". This essentially allows us to add/remove workers on-the-fly. Specifically, this makes two features possible:

Worker Failover: If a worker fails (e.g., OOM) or is evicted (e.g., not enough resource), the training continues. Later once the failed worker restarts, it can join the training job dynamically without interupting the training process.
Scale Workers Up/Down: During the training, we can dynamically add/reduce the number of workers on-the-fly based on the needs. This is particularly helpful for online learning -- use more workers during peak time while less during spare time.

The goal of this proposal is to allow tf-operator to support this.
Current Issues
In order to support the new features, there are a couple of issues that need to be solved:

Support sparse ClusterSpec mentioned here.
Support manually scaling up/down workers.
Status update logic needs to be changed (e.g., failed workers are not supposed to result in training failure).

Implementation Details
The work can be devided into the following tasks:

In TFJobSpec, add a boolean variable AllowDynamicWorker.
When AllowDynamicWorker == true, reconcile TFJobs every single time. Change needs to be made here.
When AllowDynamicWorker == true, use sparse form in TF_CONFIG here.
Handle the cases where worker index is larger than replicas here. When AllowDynamicWorker == true, implement scale-down logic, i.e., remove workers from the one has largest index until the number of workers equals replicas, here. The same change need to be done for service here and here.
Change the status update logic here.

LGTM. It is a helpful feature.
/cc @richardsliu @johnugeorge

@gaocegege The first PR is here https://github.com/kubeflow/tf-operator/pull/1142 . Looks like I'm not allowed to add you as the reviewer. Can you take a look when you get a chance? Thanks.

I like the feature, sounds good!
Is current dynamic worker also supported in all training mode, e.g. allreduce, parameter server or even sync/async training?
I am also interested in implementing it because my graduation thesis is exactly to implement the autoscaling and ps/worker location aware scheduling controller based on tf-operator which had a lot of limit at the time.
You can reference to my thesis
http://www.scitepress.org/DigitalLibrary/Link.aspx?doi=10.5220/0007707605690577
and our implementation called DRAGON
https://github.com/NTHU-LSALAB/DRAGON
So I would also like to know more about your implement details or maybe we can work on this together. Thanks 😊

I like the feature, sounds good!
Is current dynamic worker also supported in all training mode, e.g. allreduce, parameter server or even sync/async training?
I am also interested in implementing it because my graduation thesis is exactly to implement the autoscaling and ps/worker location aware scheduling controller based on tf-operator which had a lot of limit at the time.
You can reference to my thesis
http://www.scitepress.org/DigitalLibrary/Link.aspx?doi=10.5220/0007707605690577
and our implementation called DRAGON
https://github.com/NTHU-LSALAB/DRAGON
So I would also like to know more about your implement details or maybe we can work on this together. Thanks 😊

Cool! I'll submit a PR, which has been done internally, implementing in a very naive way based on the 4th item in the Implementation Details. The PR can already meet our needs for now. I think we can work together on top of it later making the functionality more sophisticated.

Interesting feature 👍

Closed in https://github.com/kubeflow/tf-operator/pull/1149


License and role requirements

Microsoft Dataverse User security role should be updated to Basic user security role

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

ID: dfab9669-0748-7c43-8ded-06f97bcd2a4d
Version Independent ID: dfab9669-0748-7c43-8ded-06f97bcd2a4d
Content: Scan business cards (Dynamics 365 Sales)
Content Source: ce/sales/scan-business-cards.md
Service: dynamics-365-sales
GitHub Login: @lavanyakr01
Microsoft Alias: lavanyakr

Hi @gilmeyer
Sorry for the delay in addressing this issue. I've updated the role requirement to Any primary sales role. As all primary sales roles have the permissions to read/write to contact and lead, they should also be able to scan the business cards.
Hope this update resolves this issue. I'm closing the issue for now. Please feel free to reopen if you feel the issue is not addressed adequately.


why breaking change on audio events?

change from audiostop to audio-stop
what useful value is that?

I'm confused. The dashes have always been there. The very first commit has them: https://github.com/rhasspy/wyoming/blob/892e01dea82ccb804d93a25dc8eef74a2d4fca37/wyoming/audio.py#L11
How are you using Wyoming?

weird indeed. I was looking at what I thought was 1.5 (master), but using 1.3 and couldnt get the is_type to work.  at that point I wasn't using the class libs. so this was probably my fault. transcribe however was different. so I ass-umed.

sorry, didn't answer the second question..making a new asr.

nodejs didn't have trouble, just needed to explicitly send a newline.  then the server side responded. no info before that

closing, not an issue


Perl port of 96_Word.

I have rewritten the game in what I understand to be fairly mainstream idiomatic Perl. The copyright statement is what I usually use, and I understand it to apply only to this port, and not to the original BASIC.

Excellent thank you!!


post_processing the results

Hello @HYPJUDY ,
just wondering, why you are taking the max of only the scores of the positive classes and not including the background class in this line in the post_process method? Is this because, you are filtering out the low confidence detections in this line? But, the config.filter_conf_threshold is set to very small (0.10). In that case, aren't you saving a lot of potentially false positive detections in the result file?
Can you please clarify?
Thanks

You're right to some extent.
The post_process function is used for finding probable "action". It's assumed the inupts are action proposals.
https://github.com/HYPJUDY/Decouple-SSAD/blob/9a78cea6be27f9cc7cd512e027f00526be5ecad8/operations.py#L369-L384
Before post_process, there are two steps to filter potential backgroud proposals,

config.filter_neg_threshold is mainly used for filtering background. It's set to 0.98 and can filter more than 60% proposals out, as mentioned in
https://github.com/HYPJUDY/Decouple-SSAD/blob/9a78cea6be27f9cc7cd512e027f00526be5ecad8/operations.py#L545-L546
config.filter_conf_threshold is auxiliary as mentioned in
https://github.com/HYPJUDY/Decouple-SSAD/blob/9a78cea6be27f9cc7cd512e027f00526be5ecad8/operations.py#L575-L577

By the way, the performance of action detection evaluated by mAP could benefit from more action proposals. That's how this process works.
https://github.com/HYPJUDY/Decouple-SSAD/blob/9a78cea6be27f9cc7cd512e027f00526be5ecad8/operations.py#L392
But too much proposals would be of no help, as mentioned in
https://github.com/HYPJUDY/Decouple-SSAD/blob/9a78cea6be27f9cc7cd512e027f00526be5ecad8/operations.py#L410
So parameters tuning is needed for different cases. You could try to set different parameters or remove these processes to see the effect. In most cases, I adopt the same parameters as SSAD.

Ok, I understand. Thank you very much for your as usual through and detailed feedback! I highly appreciate it.

This very probably would produce worse results.
If I remember correctly, the backgroud class indicating by score_0 usually have very high scores (e.g., bigger than 0.95). That's why the config.filter_neg_threshold is set to 0.98. So even if there's an action class in one proposal, its score can be smaller than background class.
You can set the config.save_predict_result to True (it's by default) and see the full list predict result to get a glimpse of the scores distribution.
https://github.com/HYPJUDY/Decouple-SSAD/blob/9a78cea6be27f9cc7cd512e027f00526be5ecad8/operations.py#L544-L546
I'm not sure. If you have need findings, welcome feedback : )

Thanks a lot for the clarification.

I didn't do further experiments on ActivityNet so I don't know.

you mean, you used the same thresholds for ActivityNet?

I mean, I don't know what's the proper parameters because I didn't do complete experiments on ActivityNet.

Ok, I understand. Thanks again for your nice feedback.


How to reproduce result with the pretrained weights

Thank you for your work.
I want to know how to reproduce the [Pitch modified Violin] result which is  presented in the demo page.

Pretrained weight could be found here https://github.com/sweetcocoa/ddsp-pytorch/tree/models
Autoencoder.reconstruction() might be the one you are looking for.
https://github.com/sweetcocoa/ddsp-pytorch/blob/master/train/network/autoencoder/autoencoder.py
modifying f0 will be affect the output's f0.
Please understand that I'm not sure if all my comments are correct because I made this repository a very long time ago.


Add support for meilisearch

Add support for Meilisearch
https://docs.meilisearch.com/
Description
This PR is related to #847
I will add document for this container soon

Hi @cytopia could you do a review please


chore: Standardize CI with other projects

@ghislainbourgeois, should we close this PR? or should we merge it?

It might be easier to still merge this one to finish the work of moving the smaller libs to util, and move to the newer mongo at a slower pace. Unless it is quick and easy to change to the newer lib, I can check with Gulsum tomorrow.

@ghislainbourgeois, the license-check action fails because several files do not have copyright and license headers.


Remove redundant pkg folder in a project generated by helidon CLI

resolves #764
Signed-off-by: aserkes<EMAIL_ADDRESS>

/oca-checked

/trigger


Permission to take down tafawa balewa profile.

tafawa balewa was the first and only prime minister and not a president.

can you add it to his bio instead?


xe-guest-utilities: init at 6.2.0

Things done:

[x] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

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

More
cc @benwbooth
Notes:
I'm using NixOS at work, where we use Citrix Xen as our VM Host. The xe-guest-utilities allows me to query the Xen host for IP addresses of VMs, a crucial feature for me, which is why I went through the trouble of packaging this.
I based this off of the xe-guest-utilities ArchLinux package. Citrix seems to require a login to download the files, so ArchLinux keeps a copy on their servers. This expression downloads the source from the Arch Linux servers, I'm not sure if that's a problem or if there's a better place to put the source files.
I did not add the services to the nixos/modules/virtualisation/xen-domU.nix and did not add the module to the module-list.nix. Since this is a new module I think it should be completely opt-in, at least until it gets more testing.
I've tested it so far on my NixOS VM, and I am able to query for IP addresses now.
I had trouble running the nox-review wip command. I'm not sure if this is because I already made the commit before I ran the command. It says this:
==> We're in a git repo, trying to fetch it
error: The following untracked working tree files would be overwritten by checkout:

and then proceeds to list all the files in my local nixpkgs tree. This is my first package submission so I'm still learning the Nix way of doing things.
Please let me know if there's any corrections to be made or problems. Thanks!

I did not add the services to the nixos/modules/virtualisation/xen-domU.nix and did not add the module to the module-list.nix

I think it's ok to add this module to module-list.nix because otherwise this service module can't be used easily. It isn't enabled by default so this shouldn't really matter.
Thanks! The code looks ok to me. If you add this module to module-list.nix and squash your commits, I think we can merge this and maybe later enabled this by default for Xen domUs. :+1:

Thanks for all the feedback! Please let me know if there's anything else I need to fix.


[bug] Solve Recommendation Issue in Trending

We're falling back to DB's Recommendation for given Article if suggestions are not generated. We need to figure out better way to handle this for Trending articles.

issue fixed : https://github.com/fafadiatech/newscout_web/pull/257


Add function to move cursor when navigating using arrow buttons / keys / search bar

Clicking arrows buttons, arrow keys, and search bar items should trigger cursor to activate and animate.

Up/down keyboard support added


[Suggestion] Make a filter for curations you're Subscribed to

It would be an easy to use Quick filter, and it would help filter curations

done in https://github.com/Dri0m/flashpoint-submission-system/commit/440a03f3395ac4a1a8497d3a8bf50a82c1d3b285


New icon for the project

Hi,
I'm the author of passwe a webextension that is doing the same thing as browserpass but with a bit less features. I have had a lot of fun creating this add-on but I plan on deprecating it as I prefer to use browserpass now.
Would you like browserpass to get passwe's icon?

If you like this icon better than your current one, I will create a PR.
@NicolasPetton is the author of the icon and agrees on giving it to browserpass.

Hey, thanks for the offer! 🙂 I like both the current and this one.
On the one hand I already have two other browser extensions installed that also use fingerprint in their icons, on the other hand it shouldn't stop us from using a better icon if we like it 😉
@erayd which one do you like more?

@maximbaz It's a nice icon, but I think I would prefer to avoid a fingerprint icon, as it implies biometrics, which isn't really anything to do with browserpass. As such, I think the current icon is better suited to the purpose of this extension.
With that said, I'd be quite happy for the project to have a new icon. A padlock is rather generic - something that more obviously indicates passwords, or a password manager, would be useful and improve the user experience.

OK, agreed to prefer the current icon over the icon with fingerprint in it. Thanks again @DamienCassou and @NicolasPetton for the offer!
We can keep this issue open for now to see if there are other suggestions for the icon, it would be good indeed to have a new icon to distinguish v2 from v3.

Interesting idea, but having ID card somehow doesn't click with me...
On that site I noticed a couple of interesting icons that give a hint of forms and passwords:

one

two

I wanted to say that spending $2 for the license is manageable, but even the purchased license specifically prohibits using the icon as part of a logo 😕

I had another idea to use the letter P (for pass), but make the descender into a key. Something like this:

Obviously someone with better skills than me would have to pretty it up. Although it seems like maybe you guys want the icon to suggest the idea of passwords much more directly, rather than relying on a metaphor with physical objects.

Nice job @richlewis42 👍 Please do attach the SVGs, I would like to play around and see how these icons look in the browser. I'll most probably keep the current blue color (#5f8dd3), because black looks bad in browsers with dark theme while blue seems alright in both light and dark themes, but I can change the color myself 😉

A friendly ping @richlewis42 🙂

ah my bad, the machine the file is on died and I need to get the hd into its replacement - now is as good a time as any to do that, will get back later this eve

Well I got it sorted finally:
pass-logo.tar.gz

Thanks a lot @lewisacidic!
I definitely want a new logo for v3, and my favorite is capital P. I have a couple of suggestions, in the current form it looks like this:

While it looks cool on the "chrome://extensions" page where I can see a larger icon, on the top panel I can barely recognize that it is a key (and I have a 4k screen), I'm afraid the majority of folks will think that it's a plain letter P without realizing that there is much more in the logo 😉
Given that .png icon has to be a square size (128x128 px) anyway, can you try to make use of the available space on the sides and make "key" parts of the icon more... expressive?
My very crude demo of what I mean is below (please don't laugh 😛), I made the curve of the P as well as the left side thicker, it looks ugly when looking close:

But when looking in a browser, especially in the top panel, I think it's more recognizable as a key:

Sorry for the delay, lots going on on my end atm!! So I'm really not a designer, but I gave it another go, making it thicker, the key part longer, and a bit more pronounced.  Thoughts?

pass-logo-candidate.tar.gz

I like it! I added it to Browserpass v3, which is now available for testing! If you have time, please try it out, and tell me if you like this icon too (any other feedback is welcome as well).

I will close this since I don't think I'm going back to the lock icon, but if anyone has more ideas or feedback, please share 😉

And once again, thanks a lot everyone who participated, and especially @ajgrf for the idea and @lewisacidic for the implementation 🎉

Remember to check out https://github.com/browserpass/browserpass/issues/334 !


Sentence deduplication output

Hi,
I started to use datatrove for deduplication. If I managed to understand the minhash_deduplication script, I've got difficulties understanding the outputs of sentence_deduplication.py.
All I obtain are 'intermediate', 'sent_dups' and 'sent_sigs' folders.
1/ 'sent_sigs' is supposed to contain a signature for each document. I've got 15 docs, and only 9 output folders in here, with 3 c4_sig files in each that I can't read.
2/ 'sent_dups' contains also 9 folders, where I've got 2 c4_dup files in each. what does these files contain extactly?
3/ where is the output of SentenceDedupFilter? The final stats seem to be okay :
" Stats: {total: 15, doc_len: 259 [min=33, max=106, 64.75±35/doc], removed_sentences: 32 [min=2, max=5, 2.91±1/doc], original_sentences: 36 [min=2, max=5, 3.27±1/doc]}"
but I can't exactly figure out how since I can't find any new version of the documents with the removed_sentences.
Could you provide any help on that?
Thanks

Hi, can you share the script you used? normally you would add a Writer after the filter and choose yourself where to save the final output

Yes sure!
I used this script: https://github.com/huggingface/datatrove/blob/b5443d2b8ef473262bc97b3d7717a217b6eaf1f3/examples/sentence_deduplication.py
That I modified like this:
`from datatrove.executor.base import PipelineExecutor
from datatrove.executor.local import LocalPipelineExecutor
from datatrove.pipeline.dedup import SentenceDedupFilter, SentenceDedupSignature, SentenceFindDedups
from datatrove.pipeline.dedup.sentence_dedup import SentDedupConfig
from datatrove.pipeline.extractors import Trafilatura
from datatrove.pipeline.filters import GopherQualityFilter, LanguageFilter
from datatrove.pipeline.readers import JsonlReader, WarcReader
from datatrove.pipeline.writers.jsonl import JsonlWriter
from datatrove.utils.typeshelper import Languages
from datatrove.pipeline.writers.disk_base import DiskWriter
"""
example on how to use sentence-deduplication. sentence-deduplication implements deduplication as in:
https://jmlr.org/papers/v21/20-074.html
'To deduplicate the data set, we discarded all but one of any three-sentence span
occurring more than once in the data set.'
to run deduplication we need to run three different pipelines,
pipeline 1:
implements usual extraction + quality filtering, it ends with SentenceDedupSignature, preprended by a writer.
pipeline 2:
implements only SentenceFindDedups
pipeline 3:
implements SentenceDedupFilter prepended by a reader of the same writer-kind used during stage 1. after the
SentenceDedupFilter.
"""
modify sentence dedup hyper params here
sent_dedup_config = SentDedupConfig(
n_sentences=2,
split_sentences=True,  # set to False to split on \n instead
only_dedup_in_index=True,
min_doc_words=1,
)
FINDER_WORKERS = 10  # this will speed up/parallelize step 2
##1. create a signature for each sentence in each doc
def run_example():
pipeline_1 = [
#WarcReader(data_folder="warc/", limit=1000),
JsonlReader(data_folder="./", paths_file="path_file.txt"),
#Trafilatura(),
#GopherQualityFilter(min_stop_words=0),
#LanguageFilter(language_threshold=0.5, languages=(Languages.english,)),
JsonlWriter("sd_out/intermediate/"),
SentenceDedupSignature(output_folder="sd_out/sent_sigs/", config=sent_dedup_config, language=Languages.french, finder_workers=FINDER_WORKERS),
]
#2. reads all the signatures and loads them to check for duplicates.
pipeline_2 = [SentenceFindDedups(data_folder="sd_out/sent_sigs/", output_folder="sd_out/sent_dups/", config=sent_dedup_config)]
#3. reads a documentpipeline and removes duplicated sentences found before
pipeline_3 = [
JsonlReader(data_folder="sd_out/intermediate/"),
SentenceDedupFilter(data_folder="sd_out/sent_dups/", config=sent_dedup_config, language=Languages.french),
]
executor_1: PipelineExecutor = LocalPipelineExecutor(pipeline=pipeline_1, workers=4, tasks=4)

executor_2: PipelineExecutor = LocalPipelineExecutor(pipeline=pipeline_2, workers=1, tasks=FINDER_WORKERS)

executor_3: PipelineExecutor = LocalPipelineExecutor(pipeline=pipeline_3, workers=4, tasks=4)

print(executor_1.run())
print(executor_2.run())
print(executor_3.run())

if name == "main":
run_example()
`

You're indeed missing a writer after the filter:
pipeline_3 = [
JsonlReader(data_folder="sd_out/intermediate/"),
SentenceDedupFilter(data_folder="sd_out/sent_dups/", config=sent_dedup_config, language=Languages.french),
JsonlWriter(data_folder="sd_out/final_output/")
]

Thank you! Now I've got the keys to understand the deduplication process 🙏

Suggestion: maybe this should be added to the original script https://github.com/huggingface/datatrove/blob/b5443d2b8ef473262bc97b3d7717a217b6eaf1f3/examples/sentence_deduplication.py as it is missing there too!


placement of code is not shown on screen

hello,
this is an odd thing i have always come across with nape. whilst i donst think its a bug or anything like that, maybe you could help with it.
so i am using kha, a haxe framework, the building and rending of shapes etc is fine, got it perfectly working. but any collision with another shapes seems completely off.
here is the code i am using for say the ball & the player
Player.hx
package char;

import kha.graphics2.Graphics;
using kha.graphics2.GraphicsExtension;
import kha.Color;
import kha.Assets;
import kha.math.Vector2;

import kha2d.Animation;
import kha2d.Sprite;
import kha.math.Vector2;

import nape.phys.Body;
import nape.phys.BodyType;
import nape.shape.Polygon;
import nape.geom.Vec2;
import nape.space.Space;

import ll.Timer;

class Player {
  public var up:Bool;
  public var down:Bool;
  public var left:Bool;
  public var right:Bool;

  public var x:Int;
  public var y:Int;
  public var xVel = 0.0;
  public var yVel = 0.0;
  public var speed = 60;
  public var friction = 6.0;

  public var body:Body;

  public function new(space:Space){
    body = new Body();
    body.space = space;
    body.shapes.add(new Polygon(Polygon.rect(128, 128, 64, 64)));
  }

  public function render(graphics:Graphics){
    graphics.color = Color.Green;
    graphics.drawRect(body.position.x, body.position.y, 64, 64);
  }
}

Ball.hx
package char;

import kha.graphics2.Graphics;
using kha.graphics2.GraphicsExtension;
import kha.Color;
import kha.Assets;
import kha.math.Vector2;

import nape.phys.Body;
import nape.phys.BodyType;
import nape.shape.Circle;
import nape.geom.Vec2;
import nape.space.Space;

class Ball {
  public var body:Body;
  public function new(space:Space){
    body = new Body();
    body.space = space;
    body.shapes.add(new Circle(32));
    body.position.x = 256;
    body.position.y = 256;
  }

  public function render(graphics:Graphics){
    graphics.color = Color.Cyan;
    graphics.drawCircle(body.position.x, body.position.y, 32);
  }
}

i also have a GIF to show exactly what i mean, but when the player is hitting or going near the ball, it is not even near the rendered shapes but starts to move, its odd.

it would be great if you or someone else knew the problem. im asking around with some kha folk too ;)
so will close and comment if i find the problem beforehand
thanks

fixed. really simple and stupid mistake that was moving the physics lower ground away by say 128, 128 on the x, y. and the block was staying at 0,0
doh!!!


New AJAX "Propose a Name" popup form for show_obs page

Depends on #1345

Coverage: 93.469% (-0.02%) from 93.493% when pulling 87fa00c82b70a8f66f9403e9646bfbc8d85e1ee1 on nimmo-ajax-propose-name into 7461da3d8f1b41329f8619c55b881bad11ea9e13 on main.


Prometheus unable to scrape operator metrics

We have deployed Keda and prometheus and several other services to our cluster but are unable to see any keda metrics in prometheus. We are interested particularly in "keda_scaler_errors" and "keda_scaler_errors_total" but neither seem to be available in prometheus.
When running curl against the metrics endpoint of the keda operator, we are able to see a list of metric events that appear to have valid data but that don't get picked up in prometheus. Other services' metric data is available in prometheus successfully, only keda seems to be not working for us.
Example:
curl XXX.XX.XX.120:8080/metrics
returns:
keda_scaler_errors{metric="s0-prometheus-nginx_ingress_controller_requests",namespace="<MY_NAMESPACE>",scaledObject="<MY_SCALED_OBJECT>",scaler="prometheusScaler",scalerIndex="0"} 41846
keda_scaler_errors{metric="s0-prometheus-nginx_ingress_controller_requests",namespace="<MY_NAMESPACE>",scaledObject="<MY_SCALED_OBJECT>",scaler="prometheusScaler",scalerIndex="0"} 12
keda_scaler_errors{metric="s0-prometheus-nginx_ingress_controller_requests",namespace="<MY_NAMESPACE>",scaledObject="<MY_SCALED_OBJECT>",scaler="prometheusScaler",scalerIndex="0"} 12
# HELP keda_scaler_errors_total Total number of errors for all scalers
# TYPE keda_scaler_errors_total counter
keda_scaler_errors_total 41876

Expected Behavior
Keda operator metrics that are available on :8080/metrics should be scraped by prometheus.
Actual Behavior
Keda Metric data is not available in promethus.
Steps to Reproduce the Problem

Enable and deploy promethus metrics and annotations in Keda helm chart
Actual configuration:

 podAnnotations:
   keda:
     prometheus.io/port: "8080"
     prometheus.io/scrape: "true"
     prometheus.io/scheme: http
     metrics.dynatrace.com/scrape: "true"
     metrics.dynatrace.com/port: "8080"
   webhooks:
     prometheus.io/port: "8080"
     prometheus.io/scrape: "true"
     prometheus.io/scheme: http
     metrics.dynatrace.com/scrape: "true"
     metrics.dynatrace.com/port: "8080"

TRUNCATED CONFIG

 webhooks:
   enabled: true

 prometheus:
   operator:
     enabled: true
     port: 8080
     path: /metrics
   webhooks:
     enabled: true
     port: 8080
     path: /metrics

Check to confirm metrics have data via curl <KEDA_OPERATOR_POD_IP>:8080/metrics
Search any keda metric in promethus and confirm data is not available.

Specifications

KEDA Version: 2.10.1
Platform & Version: AKS
Kubernetes Version: 1.24.9

@JorTurFer Do you have an idea?

This was a configuration issue. PodMonitor or ServiceMonitor must be explicitly enabled like so
prometheus:
  operator:
    enabled: true
    port: 8001
    path: /metrics
    serviceMonitor:
      enabled: true
  webhooks:
    enabled: true
    port: 8002
    path: /metrics
    serviceMonitor:
      enabled: true


Upload local model feature

Fixed a bug of entering non integer while selecting existing model that causes to crash the CLI.
Also added a new feature that let the user to load a local file from full path beside selecting from existing model files.

Merged build triggered. Test Failed.

Merged build started. Test Failed.

Merged build finished. Test Failed.


demographicsCohort ignores ageGroup names

cdm <- CohortConstructor::mockCohortConstructor()
#> Note: method with signature 'DBIConnection#Id' chosen for function 'dbExistsTable',
#>  target signature 'duckdb_connection#Id'.
#>  "duckdb_connection#ANY" would also be valid

demographics <- CohortConstructor::demographicsCohort(
  cdm = cdm,
  name = "demographics",
  ageRange = list("name1" = c(0, 19), "name2" = c(20, Inf))
)
#> ! cohort columns will be reordered to match the expected order:
#>   cohort_definition_id, subject_id, cohort_start_date, and cohort_end_date.
#> ℹ Building new trimmed cohort
#> Adding demographics information
#> Creating initial cohort
#> Trim age
#> ✔ Cohort trimmed

CohortConstructor::settings(demographics)
#> # A tibble: 2 × 3
#>   cohort_definition_id cohort_name    age_range
#>                  <int> <chr>          <chr>
#> 1                    1 demographics_1 0_19
#> 2                    2 demographics_2 20_Inf

Created on 2024-10-17 with reprex v2.0.2

So would you want two settings?

age_name (name of list, or age_range as is now if no name provided)
age_range (as is)

yep happy with that


Fix bit-depth

Description
Fix waveform generation for custom bit-depths (4, 16, 32, bits)
Acceptance Criteria

[ ] wave form generaiton should be correct for 4-bit, 16-bit and 32-bit

Can I take this up


analytics: Remove outdated comment in counts.py

Automated message from Dropbox CLA bot
@rishig, it looks like you've already signed the Dropbox CLA. Thanks!

Merged, thanks @rishig!


Install the Siteimprove CMS plugin

We would like to have the Siteimprove CMS plugin installed, to e.g. enable us to go to the correct page in Sanity from an issue, with a click on one button. We've had that from AEM, so currently it's not working for Sanity. https://help.siteimprove.com/support/solutions/articles/80000863887-how-to-install-the-siteimprove-cms-plugin

Unfortunately Sanity is not on the list for available CMS plugins on the SiteImprove page, so I don't think that is possible. AEM on the other hand is a premium plugin collaborator @meols

OK - thanks for checking @wenche


Bar and box plots always use the 'default' matplotlib colormap, even the style is changed

I set the matplotlib style to "fivethirtyeight", which changes many settings in the axes, colors, etc.
This is what I get when doing a simple line plot:
pd.DataFrame(dict(a=[1, 2, 3], b=[5, 10, 2])).plot.line()

The style matches what I was expecting.
(hist, area and other type of plots I tried work as well)
But when I want to get a bar plot, I get this:
pd.DataFrame(dict(a=[1, 2, 3], b=[5, 10, 2])).plot.bar()

Most elements of the style seem to be here, but the colors are the default matplotlib ones, not the ones in "fivethirtyeight" (and the same goes with a boxplot)

Any clue about what might be causing this?
These are the lines I used to style my plots:
import matplotlib.pyplot as plt
import matplotlib
matplotlib.style.use('fivethirtyeight')

%matplotlib inline

COLORS = [(0.3333333333333333, 0.6588235294117647, 0.40784313725490196),
 (0.2980392156862745, 0.4470588235294118, 0.6901960784313725),
 (0.7686274509803922, 0.3058823529411765, 0.3215686274509804),
 (0.5058823529411764, 0.4470588235294118, 0.6980392156862745),
 (0.8, 0.7254901960784313, 0.4549019607843137),
 (0.39215686274509803, 0.7098039215686275, 0.803921568627451),
 (0.33999999999999997, 0.86, 0.57920000000000016)]

plt.rcParams['figure.figsize'] = (14.0, 7.0)
plt.rcParams['axes.prop_cycle'] = plt.cycler('color', COLORS)

For boxplots at least, the problem might come from this:
https://github.com/pydata/pandas/blob/03ee0c19c99583da915139fbaec855e66322f5f3/pandas/tools/plotting.py#L2668
(it's using a custom _get_colors that doesn't pass a colormap)

The bar plot issue should be solved in master (see issue https://github.com/pydata/pandas/issues/11614 and PR https://github.com/pydata/pandas/pull/11865/files). Are you able to test that?
The boxplots issue also seems OK when I try it with master, but probably it would be good to add a test for that as well.

Great, thanks Joris!

It turns out that this issue also arises when using matplotlib alone, without pandas. I filed an issue on the matplotlib repo referencing this one.

Just tested, and the issue is solved on master. Closing this issue. Thanks!


Added maven configuration for DocService

Added protobuf-maven-plugin configuration needed to expose services documentation to DocService.

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

Incorporated your observations.

:+1:


qa: update coordinator to latest master

This will fix the spurious failures to store a signature.

Updated to include https://github.com/revault/coordinator/pull/49.

re-ACK 6528f52560af9b6ea8cfd0e70e877ea3b3d84c05


Attempting to Alter the Duty Cycle w/o changing any other values

Arduino IDE 1.8.19
RP2040_PWM - v1.0.4
Using Example - PWM_DynamicFreq
I'd like to modify the Duty Cycle w/o altering the Frequency.  My goal is to change the speed of a DC Motor
in a small project.  When I change the Duty Cycle in the example without changing the frequency, the Duty Cycle
remains the same as it was before the "SetPWM" call.  I looked at your code to see if I was using the wrong call
but do not see a specific call for altering only the Duty Cycle.  If I alter the Frequency in your example by any
amount, the Duty Cycle is changed as I specify in the SetPWM call.  If I do not alter the Frequency, the Duty Cycle
remains unchanged.
Can you show me the proper way to alter the Duty Cycle w/o changing any other values?

Hi @AKLitman
Thanks for opening the issue which I can consider as a real bug or an enhancement request.
I'll fix and create a new release soon to let you alter the DC on-the-fly.

Hi @AKLitman
The new RP2040_PWM releases v1.0.5 has just been published. Your contribution is noted in Contributions and Thanks
Please try the new example PWM_DynamicDutyCycle which demonstrate how to change dutyCycle and keep same frequency on-the-fly.
Best Regards,

Releases v1.0.5

Permit changing dutyCycle and keep same frequency on-the-fly. Check Attempting to Alter the Duty Cycle w/o changing any other values #3
Add example PWM_DynamicDutyCycle to demonstrate new feature
Add support for more boards using arduino-pico core, such as ADAFRUIT_ITSYBITSY_RP2040, CYTRON_MAKER_NANO_RP2040, SPARKFUN_PROMICRO_RP2040, CHALLENGER_2040_WIFI_RP2040, ILABS_2040_RPICO32_RP2040, MELOPERO_SHAKE_RP2040, SOLDERPARTY_RP2040_STAMP, UPESY_RP2040_DEVKIT, WIZNET_5100S_EVB_PICO, etc.
Display compiler informational warning messages only when _PWM_LOGLEVEL_ > 3

Thank you.  I will download and try this out.  Your code is very straightforward and I found the library easy to install and use.  This new feature will greatly enhance its use for speed control on the Cytron Maker Pi board.  Austin

UPDATE:  The new Feature works perfectly.  My Cytron Maker Pi board responds exactly as intended now.  Many thanks for being very responsive and accommodating.  Austin


💡 [REQUEST] - Make CSS Modular ( MentorLogin.page.js )

Create a separate CSS file named MentorLogin.css
Redesign the CSS and add some good properties.
Please keep class name meaningful

Concept: Modular CSS
For Help refer : https://github.com/anmode/grabtern-frontend/commit/28db8d25a0bdac3670a984348e273f96197c37c7

Hey, I think I can work on this issue.


The Ginger API calls are crashing.

This issue is regarded to the app crash I investigated for a while.
Today, after debugging the whole app including the encryption, I found out that the problem was happening due to an API call error which needed the "libflutter.so" registrant to handle API calls.
What I did was, disable the encryption and rules for calls to test if it works, then too thereafter the app crashed.
And while investigating the logcat, I found that the "libflutter.so" registrant didn't support newer Android versions having x86 architecture, and arm64-v8a. While older versions including armeabi-v7a with Android 8.1.0 or older.
This registrant is deprecated for newer Android versions, so I am finding for a fix which may "temporarily" suspend this issue as most of the phones are arm64-v8a.
If anyone finds a relevant fix, please feel free to comment.
Peace.

This is the logcat error, which seemed to do an infinite loop and apparently crashing the application:
dalvik.system.PathClassLoader[DexPathList[[zip file "/data/app/com.abacate.mob-1/base.apk"],nativeLibraryDirectories=[/data/app/com.abacate.mob-1/lib/x86, /vendor/lib, /system/lib]]] couldn't find "libflutter.so"```


mobile_ui_fix: Made the Keyboard Shortcuts and Setting modals more responsive.

The Keyboard Shortcuts Modal and Textarea in Settings/User Profile was overflowing in small width devices.
This commit fixes that issue by adding appropriate media query and changes to existing CSS

Partially Fixed #16817 (2/3)
Testing plan:
Ran it locally.
GIFs or screenshots:
Keyboard Shortcuts Before (375 x 667):

Keyboard Shortcuts After (375 x 667):

Keyboard Shortcuts Before (534 x 330):

Keyboard Shortcuts After (534 x 330):

Keyboard Shortcuts Before (438 x 669):

Keyboard Shortcuts After (438 x 669):

Settings Textarea Before

Settings Textarea After

It would be great if somebody could second me on this, but I feel the commit messages can be improved.
Here's a link to the documentation - https://zulip.readthedocs.io/en/latest/contributing/version-control.html.
I think you have 2 different commits for the 2 changes with the prettier changes squashed into the appropriate commit.

Yeah, it'd be better to have these two fixes in two different commits, and prefix the commit summary with something like "settings modal:" and "keyboard shortcuts modal:"

Sorry, @timabbott I accidentally push the latest changes and it automatically merged and closed this PR.
I have created another PR!


how to appStartWar with war generated in subproject?

i added the gretty app to one of my subprojects where the .war is generated.
when i execute the task appRunWar, i see that tomcat is starting but i see no deployment of any war file or whatsoever. the url that is produced (http://localhost:8080/ecaas_tarif-browser-web-core) produces a 404. Is there any way to see what is going on behind the scenes? as the url is wrong (or nothing is deployed), i guess there are some errors which are somehow hidden.

16:14:12: Executing task 'appRunWar'...

Configure project :team_tarif-browser-web-core
Inferred project: team_tarifbrowser, version: 3.1.0-dev.5.uncommitted+ECPC.3744.war.deployment.dc83a11

Configure project :
Property named 'artifactrepoUsername' not set.
Property named 'artifactrepoPassword' not set.

Task :team_tarif-browser-common:schemaGen_ch-forum_datenaustausch-invoice400 UP-TO-DATE
Task :team_tarif-browser-common:schemaGen_ch-forum_datenaustausch-invoice410 UP-TO-DATE
Task :team_tarif-browser-common:schemaGen_com-adcubum-xsd-eclaim-common UP-TO-DATE
Task :team_tarif-browser-common:schemaGen_com-adcubum-xsd-eclaim-invoice UP-TO-DATE
Task :team_tarif-browser-common:schemaGen_syriusRuleDef.xsd UP-TO-DATE
Task :team_tarif-browser-common:compileJava UP-TO-DATE
Task :team_tarif-browser-common:processResources UP-TO-DATE
Task :team_tarif-browser-common:classes UP-TO-DATE
Task :team_tarif-browser-common:jar UP-TO-DATE
Task :team_tarif-browser-core-common:compileJava UP-TO-DATE
Task :team_tarif-browser-core-common:processResources NO-SOURCE
Task :team_tarif-browser-core-common:classes UP-TO-DATE
Task :team_tarif-browser-core-common:jar UP-TO-DATE

Task :team_tarif-browser-core-domain:compileJava

Task :team_tarif-browser-core-domain:processResources UP-TO-DATE
Task :team_tarif-browser-core-domain:classes
Task :team_tarif-browser-core-domain:jar UP-TO-DATE
Note: Hibernate JPA 2 Static-Metamodel Generator 5.4.9.Final
Note: C:\Users\gnah\IdeaProjects\tarifbrowser\core-domain\src\main\java\com\adcubum\mediframe\browser\core\repositories\DefaultRepositoryFactory.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Task :team_tarif-browser-core:compileJava UP-TO-DATE
Task :team_tarif-browser-core:processResources UP-TO-DATE
Task :team_tarif-browser-core:classes UP-TO-DATE
Task :team_tarif-browser-core:jar UP-TO-DATE
Task :team_tarif-browser-web-common:compileJava UP-TO-DATE
Task :team_tarif-browser-web-common:processResources NO-SOURCE
Task :team_tarif-browser-web-common:classes UP-TO-DATE
Task :team_tarif-browser-web-common:jar UP-TO-DATE
Task :team_tarif-browser-web-core:compileJava UP-TO-DATE
Task :team_tarif-browser-web-core:processResources UP-TO-DATE
Task :team_tarif-browser-web-core:classes UP-TO-DATE
Task :team_tarif-browser-web-core:war UP-TO-DATE
Task :team_tarif-browser-web-core:prepareArchiveWebApp UP-TO-DATE
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.codehaus.groovy.vmplugin.v7.Java7$1 (file:/C:/Users/gnah/.gradle/caches/modules-2/files-2.1/org.codehaus.groovy/groovy/2.5.8/2f1e8ea55e625fe51e85ef35eb067f1d9c61772d/groovy-2.5.8.jar) to constructor java.lang.invoke.MethodHandles$Lookup(java.lang.Class,int)
WARNING: Please consider reporting this to the maintainers of org.codehaus.groovy.vmplugin.v7.Java7$1
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
Dez. 10, 2019 4:14:17 NACHM. org.apache.coyote.AbstractProtocol init
INFORMATION: Initializing ProtocolHandler ["http-nio-8080"]
Dez. 10, 2019 4:14:25 NACHM. org.apache.tomcat.util.net.NioSelectorPool getSharedSelector
INFORMATION: Using a shared selector for servlet write/read
Dez. 10, 2019 4:14:25 NACHM. org.apache.catalina.core.StandardService startInternal
INFORMATION: Starting service [Tomcat]
Dez. 10, 2019 4:14:25 NACHM. org.apache.catalina.core.StandardEngine startInternal
INFORMATION: Starting Servlet Engine: Apache Tomcat/9.0.13
Dez. 10, 2019 4:14:25 NACHM. org.apache.catalina.startup.ContextConfig getDefaultWebXmlFragment
INFORMATION: No global web.xml found
Dez. 10, 2019 4:14:27 NACHM. org.apache.jasper.servlet.TldScanner scanJars
INFORMATION: At least one JAR was scanned for TLDs yet contained no TLDs. Enable debug logging for this logger for a complete list of JARs that were scanned but no TLDs were found in them. Skipping unneeded JARs during scanning can improve startup time and JSP compilation time.
Dez. 10, 2019 4:14:27 NACHM. org.apache.catalina.core.ApplicationContext log
INFORMATION: No Spring WebApplicationInitializer types detected on classpath
Dez. 10, 2019 4:14:30 NACHM. org.apache.catalina.util.SessionIdGeneratorBase createSecureRandom
WARNUNG: Creation of SecureRandom instance for session ID generation using [SHA1PRNG] took [2,947] milliseconds.
Dez. 10, 2019 4:14:30 NACHM. org.apache.coyote.AbstractProtocol start
INFORMATION: Starting ProtocolHandler ["http-nio-8080"]
16:14:30 INFO  Tomcat 9.0.13 started and listening on port 8080
16:14:30 INFO  team_tarif-browser-web-core runs at:
16:14:30 INFO    http://localhost:8080/team_tarif-browser-web-core

Task :team_tarif-browser-web-core:appRunWar
Press any key to stop the server.

Do you have a servlet mapped to path “” or “/“ or have a default servlet or welcome file?

this was an error on my side.
the .war file which was generated did not contain all files, so no .html was shown.
unfortunately, i forgot to close that issue


Coral Dual TPU not detected

Description
I have a Lenovo P340 Tiny. I replaced Intel AX210NGW with Coral Dual TPU E-key. Followed the install instruction and don't see even one apex device detected.
PC: https://pcsupport.lenovo.com/us/en/products/workstations/thinkstation-p-series-workstations/thinkstation-p340-tiny
OS: Ubuntu 20.04.3 LTS
Coral: Dual TPU E-Key
Click to expand!
Issue Type
Build/Install
Operating System
Ubuntu
Coral Device
M.2 Accelerator with dual Edge TPU
Other Devices
No response
Programming Language
No response
Relevant Log Output
simon@docker:~$ lsmod | grep apex
simon@docker:~$

simon@docker:~$ ls /dev/apex_0
ls: cannot access '/dev/apex_0': No such file or directory

simon@docker:~$ sudo lspci
00:00.0 Host bridge: Intel Corporation Device 9b33 (rev 05)
00:01.0 PCI bridge: Intel Corporation Xeon E3-1200 v5/E3-1500 v5/6th Gen Core Processor PCIe Controller (x16) (rev 05)
00:01.1 PCI bridge: Intel Corporation Xeon E3-1200 v5/E3-1500 v5/6th Gen Core Processor PCIe Controller (x8) (rev 05)
00:04.0 Signal processing controller: Intel Corporation Xeon E3-1200 v5/E3-1500 v5/6th Gen Core Processor Thermal Subsystem (rev 05)
00:08.0 System peripheral: Intel Corporation Xeon E3-1200 v5/v6 / E3-1500 v5 / 6th/7th/8th Gen Core Processor Gaussian Mixture Model
00:12.0 Signal processing controller: Intel Corporation Comet Lake PCH Thermal Controller
00:14.0 USB controller: Intel Corporation Comet Lake USB 3.1 xHCI Host Controller
00:14.2 RAM memory: Intel Corporation Comet Lake PCH Shared SRAM
00:16.0 Communication controller: Intel Corporation Comet Lake HECI Controller
00:16.3 Serial controller: Intel Corporation Device 06e3
00:17.0 SATA controller: Intel Corporation Device 06d2
00:1b.0 PCI bridge: Intel Corporation Comet Lake PCI Express Root Port #17 (rev f0)
00:1b.4 PCI bridge: Intel Corporation Comet Lake PCI Express Root Port #21 (rev f0)
00:1d.0 PCI bridge: Intel Corporation Comet Lake PCI Express Root Port #9 (rev f0)
00:1f.0 ISA bridge: Intel Corporation Device 0687
00:1f.3 Audio device: Intel Corporation Comet Lake PCH cAVS
00:1f.4 SMBus: Intel Corporation Comet Lake PCH SMBus Controller
00:1f.5 Serial bus controller [0c80]: Intel Corporation Comet Lake PCH SPI Controller
00:1f.6 Ethernet controller: Intel Corporation Ethernet Connection (11) I219-LM
02:00.0 VGA compatible controller: NVIDIA Corporation GP107GL [Quadro P1000] (rev a1)
02:00.1 Audio device: NVIDIA Corporation GP107GL High Definition Audio Controller (rev a1)
03:00.0 SATA controller: Samsung Electronics Co Ltd Device a801 (rev 01)
04:00.0 Non-Volatile memory controller: Samsung Electronics Co Ltd NVMe SSD Controller SM981/PM981/PM983

hi mate, did you ever fix this ?

nope. Replaced the Lenovo with a different NUC, yet still not able to use both TPUs, only a single one, since it's apparently super rare to have two ECIe x1 lanes on in a E-Key connector.


AH Smart Contracts

@athei and team are for now building something quick that they can control in https://contracts.polkadot.io/
This should be the staging ground. The main AH Contract tutorials should ideally live here.

 It can be easily moved.


TestServiceExportedInOneCluster test flake

https://prow.istio.io/view/gs/istio-prow/logs/integ-pilot-multicluster-tests_istio_postsubmit/1415399944373669888
https://prow.istio.io/view/gs/istio-prow/logs/integ-pilot-multicluster-tests_istio_postsubmit/1412820797625470976

I suspect something must not be getting updated/pushed when we updateXDS. Perhaps the issue is that it's only clearing the EDS cache for the cluster where the ServiceExport was created? Maybe we need to clear EDS for the service in all clusters?

https://prow.istio.io/view/gs/istio-prow/pr-logs/pull/istio_istio/34086/integ-pilot-multicluster-tests_istio/1417269147036815360

https://prow.istio.io/view/gs/istio-prow/pr-logs/pull/istio_istio/34258/integ-pilot-multicluster-tests_istio/1418407700542263296
Looks like https://github.com/istio/istio/pull/34143 is not the fix

fyi https://prow.istio.io/view/gs/istio-prow/logs/integ-pilot-multicluster-tests_istio_postsubmit/1425152917559578624 is still occuring

Quick update ... still appears to be happening after a variety of recent updates to the MCS logic: https://prow.istio.io/view/gs/istio-prow/logs/integ-pilot-multicluster-tests_istio_postsubmit/1447618309238296576

not stale


Blur on Sierra and newer cursors

The sierra and newer cursors are very broken. On white screens only the black part appears and more. Also the blur specifically on the link one.

@THEVERYLOL30 what is your resolution screen?

This is cursors only for 4K (3840 x 2160 pixel). Please read description.


Added ControlsTimer class with example usage in mazda+hyundai

Increase the readability of DT_CTRL based duration timers in carports. Also, remove redundant self.frame+=1 in every carcontroller. See examples for Mazda and Hyundai.

Not sure why process replay is failing. The code should be good.

I like the direction this is going in. Gonna move to draft until I have some time to look at it more.

Let me know if you have any questions about it

I added some unit tests in case you make changes


Create BBEdit gitignore

Thanks, but can you provide some explanation, links to relevant documentation etc? Sometimes with IDEs the *.<ide>proj files are recommended to be kept, for example. And is this something that would be useful to many people? Or is it a niche tool?

BBEdit is a popular text editor for Mac OS X. I don't know what you're looking for, but looking at the files for some of the other editors, I don't see anything unusual in my submission.

See our contributing guildelines. For example, in order of decreasing preference:

is there any documentation telling you to leave this file out of version control or otherwise?
does the file contain user-specific paths or confidential details that make it an obvious candidate for ignoring?
are there examples of this file that people have checked in inadvertently?
are there examples of people adding this rule to their .gitignores in existing projects?


LineLengthTransformer handle the case when file has no trailing newline

Please add expected test cases yourself, not sure how should those look like
Fixes #1412

Duplicate of #1413


c1xx : fatal error C1083: Cannot open source file: '..\src\binding.cc': No such file or directory [E:..\build\binding.vcxproj]

Node Version:
Platform:
Compiler:
Module:

Verbose output (from npm or node-gyp):

E:\Web>node-gyp build
gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| win32 | x64
gyp info spawn C:\Program Files (x86)\MSBuild\14.0\bin\msbuild.exe
gyp info spawn args [ 'build/binding.sln',
gyp info spawn args   '/clp:Verbosity=minimal',
gyp info spawn args   '/nologo',
gyp info spawn args   '/p:Configuration=Release;Platform=x64' ]
Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
  binding.cc
  win_delay_load_hook.cc
c1xx : fatal error C1083: Cannot open source file: '..\src\binding.cc': No such file or directory [E:\Quarterly\Quarterly-Web\build\binding.vcxproj]
gyp ERR! build error
gyp ERR! stack Error: `C:\Program Files (x86)\MSBuild\14.0\bin\msbuild.exe` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (C:\..\AppData\Roaming\npm\node_modules\node-gyp\lib\build.js:258:23)
gyp ERR! stack     at emitTwo (events.js:125:13)
gyp ERR! stack     at ChildProcess.emit (events.js:213:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:197:12)
gyp ERR! System Windows_NT 6.3.9600
gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\...\\AppData\\Roaming\\npm\\node_modules\\node-gyp\\bin\\node-gyp.js" "build"
gyp ERR! cwd E:\Quarterly\Quarterly-Web
gyp ERR! node -v v8.1.2
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok

Where can I get this binding,cc file!?

What exactly are you trying to do?

I am trying to install node-gyp in windows-8.1.It was require binding.cc file which i have declared in binding.gyp file.
My question is : should i create empty binding.cc file?

Can you explain what you are trying to accomplish?  node-gyp is a build tool for C and C++ code.

I am setting up one app which have front-end in ionic and backend in node.I was trying to set up front-end in windows.When i was trying to execute 'ionic serve' ,It throws an error that Error in your gulp script. Then,I have installed npm build-tools. After all these process,Trying to execute 'node-gyp' which gives me this error.

I don't think the issue is with node-gyp.  As I said, it's only a build tool.  What's more, there is no gulp script, that's probably from ionic.  Perhaps try contacting that project.
You shouldn't need to deal with node-gyp directly, npm install normally takes care of running it when necessary.  Good luck.

I have the same problem exaxtly

Please provide the command that you typed and the full error log.

In case someone gets here.
https://stackoverflow.com/questions/44357010/node-gyp-build-fatal-error-c1083

what a helpful support person
didnt answer question, closed issue, person left confused.


[2022 M4] Improve battery model (non-constant discharge rates)

This issue is to discuss and track the progress on Improving the battery model being used at the time of writing.
Discussion:
The Tethys model currently uses the gazebo battery plugin that implements a linear battery model. The idea is to improve the this plugin and its use by the Tethys model in order to make a more accurate use of the battery charge by adding ways to have non-constant discharge rates.

Dynamic addition and removal of power consumers.

After discussing this will be done through components, as follows:

A plugin can sign up as a battery consumer by creating this component with the value of the power load.
This power load will be added to the battery model.
Make sure the power load is removed when the plugin is not there anymore.

Add topics support

Add topics support

See https://github.com/gazebosim/gz-sim/pull/1811


File handle leak

In nl.isaac.dotcms.searcher.util.SearchableAttributesUtil there is a call to IOUtils.toByteArray(file.getInputStream())
In 2.2 of commons-io the stream is not closed. I believe in 2.6 it is.
This leaks file handles.

Thanks, we already noticed a leakage in one of our systems but we hadn't found the cause yet. We'll pick this up.

Picked up in https://github.com/isaaceindhoven/isaac-dotcms-advancedsearch/tree/feature/issue-3_file-leackage

@marcus-leake : Can you check whether the latest master version works for you? I've tested it locally and the number of open files now stays constant when I use the advanced search.


allow pages to stop from navigating away, e.g. if there are unsaved state changes

Hello,
I am going to start a web project with Flutter, for another project use the Fluro library, however, I could not solve the following problem:
If a user is on a page that contains a form and is making modifications is said form, I would like that if he tries to exit without having saved the modifications (either by putting another url, with the browser's backbutton, ...) it would show him an alertdialog giving him the possibility of canceling the navigation to the new url or, on the contrary, of canceling the modifications and staying on the same page where he is.
The question is, can I do this with getx?
getx has any mechanism by which this functionality can be achieved?
Please, if this functionality is possible with getx could you give me some indications.
Thank you so much for all the help.
Regards,
Jose

@Josua2012 this types of questions is related more to flutter framework not GetX package, I recomend sites like StackOverflow as most proably this question is already answer there.
Non the less, what you are searching for is in the library from "dart:html", which contains the class "window" which has all (i think) methods used to work with DOM objects.
import 'dart:html' as html;
html.window.onBeforeUnload.listen((event) async{
  // do something
});

@maares
But shouldn't a library like GetX that have routing support incorporate this feature?
Thanks.
Regards,
Jose

@Josua2012 I´m a novice on flutter really, I´m not aware that navigation 2.0 it´s own methods to detect for instance when a tab is closed, please send link to learn from it.
On my opinion it could be implemented on GetX, but bare in mind web is relatively new. GetX as mentions on the read me section is more than state management, but that does not mean it will re-implement in a more convenient way in advance all available features that already exists on flutter framework libraries.
Think it more as a road-map, as this library implemented GetConnect hopefully in the future it will continue to implement more functionality in order to make our development easier.


3461: styling fixes

This PR updates govuk_frontend_toolkit / elements / templates to the latest versions. As part of this it untangles the work that had previously been committed into app/assets/stylesheets/elements as this would have been wiped out during the upgrade.
Finally, it upgrades govuk-lint to 1.0.0 and makes the subsequent amends.

LGTM


python tools/download_imagenet_weights.py

Traceback (most recent call last):
File "/home/zq/anaconda3/lib/python3.6/site-packages/urllib3/connection.py", line 141, in _new_conn
(self.host, self.port), self.timeout, **extra_kw)
File "/home/zq/anaconda3/lib/python3.6/site-packages/urllib3/util/connection.py", line 83, in create_connection
raise err
File "/home/zq/anaconda3/lib/python3.6/site-packages/urllib3/util/connection.py", line 73, in create_connection
sock.connect(sa)
TimeoutError: [Errno 110] Connection timed out
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/zq/anaconda3/lib/python3.6/site-packages/urllib3/connectionpool.py", line 601, in urlopen
chunked=chunked)
File "/home/zq/anaconda3/lib/python3.6/site-packages/urllib3/connectionpool.py", line 346, in _make_request
self._validate_conn(conn)
File "/home/zq/anaconda3/lib/python3.6/site-packages/urllib3/connectionpool.py", line 850, in _validate_conn
conn.connect()
File "/home/zq/anaconda3/lib/python3.6/site-packages/urllib3/connection.py", line 284, in connect
conn = self._new_conn()
File "/home/zq/anaconda3/lib/python3.6/site-packages/urllib3/connection.py", line 150, in _new_conn
self, "Failed to establish a new connection: %s" % e)
urllib3.exceptions.NewConnectionError: <urllib3.connection.VerifiedHTTPSConnection object at 0x7f42d27027f0>: Failed to establish a new connection: [Errno 110] Connection timed out
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/zq/anaconda3/lib/python3.6/site-packages/requests/adapters.py", line 440, in send
timeout=timeout
File "/home/zq/anaconda3/lib/python3.6/site-packages/urllib3/connectionpool.py", line 639, in urlopen
_stacktrace=sys.exc_info()[2])
File "/home/zq/anaconda3/lib/python3.6/site-packages/urllib3/util/retry.py", line 388, in increment
raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='docs.google.com', port=443): Max retries exceeded with url: /uc?export=download&id=1wHSvusQ1CiEMc5Nx5R8adqoHQjIDWXl1 (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f42d27027f0>: Failed to establish a new connection: [Errno 110] Connection timed out',))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "tools/download_imagenet_weights.py", line 91, in
main()
File "tools/download_imagenet_weights.py", line 86, in main
download_file_from_google_drive(file_id, destination)
File "tools/download_imagenet_weights.py", line 51, in download_file_from_google_drive
response = session.get(URL, params={'id': id}, stream=True)
File "/home/zq/anaconda3/lib/python3.6/site-packages/requests/sessions.py", line 521, in get
return self.request('GET', url, **kwargs)
File "/home/zq/anaconda3/lib/python3.6/site-packages/requests/sessions.py", line 508, in request
resp = self.send(prep, **send_kwargs)
File "/home/zq/anaconda3/lib/python3.6/site-packages/requests/sessions.py", line 618, in send
r = adapter.send(request, **kwargs)
File "/home/zq/anaconda3/lib/python3.6/site-packages/requests/adapters.py", line 508, in send
raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='docs.google.com', port=443): Max retries exceeded with url: /uc?export=download&id=1wHSvusQ1CiEMc5Nx5R8adqoHQjIDWXl1 (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f42d27027f0>: Failed to establish a new connection: [Errno 110] Connection timed out',))

did you make it? @zq130320339

I met the same problem .have you solved it？


e_os.h: Include wspiapi.h to improve Windows backward compatibility

This patch is a trivial one-liner.
According to getaddrinfo function (ws2tcpip.h) - Win32 apps | Microsoft Docs:

The getaddrinfo function was added to the Ws2_32.dll on Windows XP and later. To execute an application that uses this function on earlier versions of Windows, then you need to include the Ws2tcpip.h and Wspiapi.h files. When the Wspiapi.h include file is added, the getaddrinfo function is defined to the WspiapiGetAddrInfo inline function in the Wspiapi.h file. At runtime, the WspiapiGetAddrInfo function is implemented in such a way that if the Ws2_32.dll or the Wship6.dll (the file containing getaddrinfo in the IPv6 Technology Preview for Windows 2000) does not include getaddrinfo, then a version of  getaddrinfo is implemented inline based on code in the Wspiapi.h header file. This inline code will be used on older Windows platforms that do not natively support the getaddrinfo function.

Current functionalities remain unchanged, since the exported getaddrinfo and freeaddrinfo symbols are still used on higher versions of Windows through dynamic loading. This patch enables inline fallback implementations on Windows 2000 and Windows 9x operating systems, which would otherwise generate an error message running applications linked to OpenSSL:

It does not solve all Windows 9x compatibility problems, but this error will go away and programs will run.

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

This pull request is ready to merge

ping @openssl : this PR seems to be "ready to merge"

When I tried to merge it, I've got a following error message:
Rewrite c2ff48c9323efc65d6f1871a44a43c65f31aa675 (1/1) (0 seconds passed, remaining 0 predicted)    Commit author<EMAIL_ADDRESS>has no CLA, and this is a non-trivial commit
msg filter failed: /home/dbelyavs/ossl_team/tools/review-tools/gitaddrev --prnum=14550 --reviewer=@t8m --reviewer=@paulidale <EMAIL_ADDRESS>
@mattcaswell, could you please take a look?

@beldmit  gitaddrev fails because no CLA has been submitted and the CLA: trivial marker is missing. Since @Prcuvu already stated in the GitHub thread that he considers it a trivial fix, all that needs to be done is to amend the commit message as follows
e_os.h: Include wspiapi.h to improve Windows backward

CLA: trivial

and the reviewers need to agree that it's a trivial change.

Well... There is a hole in our processes as we din't get the CLA-related hold earlier.

Ok, this time the bot added the hold: cla required label. Not sure why it didn't do it when the pr was created.

I agree this is trivial fix.
@Prcuvu could you please amend the commit message?

CLA marker added.
@t8m @beldmit @mspncp I apologize for all the inconvenience.

@paulidale Do you agree with CLA: trivial? If so, please merge. I see your agreement crossed out but I suppose this is due to the confusion whether this was submitted with CLA: trivial or not.

CLA marker added.
@t8m @beldmit @mspncp I apologize for all the inconvenience.

No problem!

@openssl : ready to merge ?

Agreed trivial.  I won't be able to merge this until mid next week though.

I'll try to merge it when the repo is unfrozen

Merged. Thanks both for your contribution and your patience!


how do I get the app id?

How can I get the app id

https://discord.com/developers/applications APPLICATION ID under your app


fix: reduce motion for accessibility setting

Fixes Issue
fix PR https://github.com/EddieHubCommunity/BioDrop/pull/9172
closes #9139

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Thanks for testing 👍


Request follow caret

Should fix #101, but I'm not sure if I implemented it correctly.

Updated in light of your comment.

👍

Since that's now a public method... we should add some javadoc to explain
its usage in the method and class javadoc
On Oct 6, 2016 5:18 PM, "Tomas Mikula"<EMAIL_ADDRESS>wrote:

Merged #378 https://github.com/TomasMikula/RichTextFX/pull/378.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TomasMikula/RichTextFX/pull/378#event-815582392, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIBfbYQgpR3gDBTSNEd5jABlbgbWXx8sks5qxY_JgaJpZM4KPf8j
.

I amended the commit, but it seems to have shoved the two commits into one. Thus, requestFollowCaret isn't actually public in this PR. I'll need to open another one to correct that issue.


HHH-10282 Short-hand enum syntax support for HQL

Thanks for the review @sebersole!

Regarding the enum work in particular, I wonder if there is a way we can clean that up so that it is handled once as part of the normal visitation of those nodes instead of having to deal with it across the various predicates. As it is, if we add new predicates we have to remember that we might need to account for this enum handling. Not saying its doable nor that this needs to change. Just saying it is something I'd like to consider.

I don't think this is possible because we need to resolve the type of the non-literal side to dermine if the parse tree could be interpreted as enum literal. We might be able to pull this off for qualified literals, but since multiple enums with the same simple names might exists, we'd run into issues there again. I know it's not beautiful, but such a context sensitiveness probably never is :(

Yes, the "problem" is that we are trying something here that is "out of place" in terms of design.  Its really the next phase where interpretation/guessing of types is supposed to be done.  Things like this done out of place never seem clean.
I wonder if we could design an SQM node that is "delayed" in part of it resolution.  Something for us to think about

Great! We should have done this a long time ago but much better late than never! Thanks!


Feature Request: extend the /show command

Could you please extend the /show command to return the values of Basic Move, Current Move, Basic Speed and Fright Check.
/show move would return
Basic Move / Current Move
/show speed would return
Basic Speed
/show fright would return
Fright Check

Done.   BTW, you could already do Fright Check with either:
/show frightcheck
/show "fright check"

but for you, our loyal customer, I added
/show fright

It will be in the next release.


Fixes Intersector.intersectRayTriangle failures #3147

Why the epsilon? We already have MathUtils.FLOAT_ROUNDING_ERROR as well as MathUtils.isZero and MathUtils.isEqual.

Because I'm still new to libgdx and I didn't know about MathUtils...
Fell free to remove my epsilon and use your MathUtils.isZero instead.

Could you please update your PR to use the existing Libgdx API and then squash the commit? Thanks!

Brand new PR (from a fresh fork): https://github.com/libgdx/libgdx/pull/3301


Which algorithm used by coinbase..?

May I know which algorithm used by coinbase..? Using createAccountAddress() function we generate account address right.?

Yes, that's correct. Not sure what you mean by what algorithm we use.

I mean when we call createAccountAddress() it return one address so it's a random generated string or u guys are using any algorithm for that.?
I hope now you got it what exactly I am mean.

Yes, addresses are generated randomly.

You can find the technical details on how the visible address is derived from the ECDSA keypair here: https://en.bitcoin.it/wiki/Technical_background_of_version_1_Bitcoin_addresses


Restructuring Code/Performance Updates

Changes:

Some function name and variable changes (probably needs review)
Static arrays used throughout where appropriate
Improved type stability
Stability derivatives calculated using ForwardDiff, instead of the custom dual numbers implemented in sderiv
Separated stability derivative calculations and core vortex lattice method calculations
5x speedup and 5x reduction in allocations.
Added an experimental function to construct panels based on an existing grid.
Updated documentation
All tests pass.

I have not changed the theoretical foundation yet.
Future work:

fix remaining stability derivatives (the ones that didn't match in the previous version of VLM)
visualization (both a plotting package and ParaView would be nice)
change theoretical foundation to be more appropriate for large deflections
unsteady extension?

I'm going to merge, but just had a couple questions/comments.
Why are these lines needed (and others like it):
Base.eltype(::Panel{TF}) where TF = TF
How did you determine when inlining was needed?  Does it really help much?
I like putting the stability derivatives separate with AD.  The downside is that if you want derivatives of the stability derivatives you'll have to use nested AD, which can be slow (though I haven't profiled it in this case).

Those eltype definition lines are in order to be able to extract type information from the structs.  It's useful when combined with promote_type to find the correct type of float to use in functions.  I suppose I could have used a custom function instead of overloading eltype, but I think overloading eltype is more standard.

I put inline on all the internal functions of the performance critical functions.  It helps with performance when using static arrays.

With regards to AD, you were basically implementing your own AD already, so I wouldn't expect nested AD to be any slower than when using the previous version, especially since the version of AD in ForwardDiff is more optimized for performance.

I've never needed promote_type in order to make my functions AD compatible. You must have a different approach from what I normally do.  I wasn't implementing AD, just symbolic derivatives, but you're probably right that the performance won't be much different.


Dear author, I did not find the fps data of VITA in your paper. Can you tell us the fps in different sizes?

Dear author, I did not find the fps data of VITA in your paper. Can you tell us the fps in different sizes?

@sukjunhwang

I am very sorry for the late reply. Using V100 GPU with a ResNet-50 backbone, it runs at 46.1 FPS.


Global Find/Replace with Regex not working

Environment data
dotnet --info output:
.NET Command Line Tools (1.0.0-preview2-003121)

Product Information:
 Version:            1.0.0-preview2-003121
 Commit SHA-1 hash:  1e9d529bc5

Runtime Environment:
 OS Name:     Mac OS X
 OS Version:  10.11
 OS Platform: Darwin
 RID:         osx.10.11-x64

VS Code version:1.3
C# Extension version:1.2.2
Steps to reproduce
Do a global search with regex turned on and with capture groups eg (this) then in the replace enter $1 which will replace what it finds with the first capture group
Expected  behavior
Replace with the value of $1 not the actual string of $1
Actual behavior
Replaces all found items with the string of the capture group

This issue was moved to Microsoft/vscode#9223

@jchannon This is an issue with VS Code and not the C# Extension. I have moved the issue to them.


Create NFT descriptor contract for reliquary

Explore if we can use maturing nfts

first descriptor done https://github.com/beethovenxfi/beethovenx-token/pull/27
need proper images


modelnet40 scale and norm

Hi HuguesTHOMAS,
Thanks for sharing with us such a great work which could help us a lot with a research we're doing!
I'd like to combine objects from KITTI dataset and existing point clouds from ModelNet40.
I wonder, how did you normalize and scale the point clouds? Is it spherical?
Additionally, is it necessary to have the normals in order to train the network or just x, y, and z?
Thanks a lot!
Carmi

Hi @CarmiShimon,
I am not sure to understand your question. IF you are talking about the models from ModelNet40, I use the data from PointNet article, where all objects are already scaled to fit in a unit sphere. If you are talking about the input point cloud in the network, they are not normalized, I take the data as-is and the network convolutions are scaled accordingly with the subsampling size (first_subsmpling_dl in the code).
The network does not need normals, but it can use them if you want to.
Best,
Hugues


Calling out Pod / Deployment inline vs PV/PVC more clearly

This is a really helpful page, thank you for thoughtfully putting this together!
I think that this comment on the page is a little out of date:
"Note: inline azureFile volume can only access secret in the same namespace as pod, to specify a different secret namespace, please use below persistent volume example instead."

I think it should be reworded to say that "inline azureFile volume can only access secret in the default namespace". This is based on my reading of a technical discussion here:
https://github.com/Azure/AKS/issues/2108
So it may be helpful to also mention that using kind: Deployment instead of kind: Pod means you should go ahead and manage the Azure file share access with PV/PVC as well, regardless of the namespace. As per Github user andyzhangx, these inline definitions for kind: Pod assume the 'default' namespace. Specifically I base that statement on this reply:
https://github.com/Azure/AKS/issues/2108#issuecomment-776508224
I almost have to question the value of having an inline example on the page at all, given that many deployments are not in the default namespace. The PV/PVC example is indeed more complicated to understand, but it's important to understand because of the move to the CSI driver for azureFile.

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

ID: d634c63a-6194-86a6-b522-3310812c81e8
Version Independent ID: fa86da1e-9c86-88fe-170c-5e6aa9230b5f
Content: Manually create Azure Files share - Azure Kubernetes Service
Content Source: articles/aks/azure-files-volume.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Thanks for the contribution, @jeff-r-koyaltech! Based on testing and our understanding of recent Kubernetes updates, this is no longer an issue and inline volume mounting can be performed successfully from non-default namespaces. If you feel this has been closed erroneously, please feel free to reopen and tag me.
#please-close


[BUG] why thre are two same selector when register a new client

Bug
when i register a new client，there are two same selector data

and in database, there are two records

after debug, i find if using nacos as registration center, and deploy more than one admin instance(for example two). when i start a client and to register, both admin server receive nacos change event and start to register meta data as follow:
org.apache.shenyu.admin.service.impl.SelectorServiceImpl#registerDefault(org.apache.shenyu.register.common.dto.MetaDataRegisterDTO, java.lang.String, java.lang.String)
below code, both admin instance with not find selectorDO and will create two same selector data
String contextPath = ContextPathUtils.buildContextPath(dto.getContextPath(), dto.getAppName());
        SelectorDO selectorDO = findByNameAndPluginName(contextPath, pluginName);
        if (Objects.isNull(selectorDO)) {
            SelectorDTO selectorDTO = SelectorUtil.buildSelectorDTO(contextPath, pluginMapper.selectByName(pluginName).getId());
            selectorDTO.setHandle(selectorHandler);
            return registerDefault(selectorDTO);
        }

that cause lot of warn
AbstractRetryTask [shenyu-shared_wheel_timer-2] Failed to execute task:/o:divide,retried：1 ,total retries:-1,cause:nested exception is org.apache.ibatis.exceptions.TooManyResultsException: Expected one result (or null) to be returned by selectOne(), but found: 2

version pls?

version pls?

2.5.0

version pls?

2.5.0，a little bit old, but i see the newest code org.apache.shenyu.admin.service.register.AbstractShenyuClientRegisterServiceImpl#register may has the same problem if more than one shenyu admin instance receive meta data register nacos change event at the same time

hi, if u have time to handle this problem, u can make a pr to resolve it. 😊


redux-form

This was probably considered before, so I'm sorry if it's an answered question. What's the main argument for using this custom redux-fields over redux-form? It seems the latter is universal too, and has a few more features (doing my research now since I need array fields).
EDIT: i'm talking specifically about v6 of redux-form

redux-fields is simple abstraction without bugs. It's stable, fast, predictable. I would never add anything with hundreds weird issues into Este. https://medium.com/@steida/why-validation-libraries-suck-b63b5ff70df5 My friend will soon extract from Este and open source it.


Improve how analysis definitions are created

TL;DR: Frontend is now taking care of creating analysis definitions when a layer or multiple layers are saved, but this could leave the app in an invalid state unless we implement some error handling or we make this operations atomic by moving some of the logic to the backend.
Right now, the frontend takes care of creating analysis definitions so that layers that have a source point to an existing analysis definition. This is done in two places right now:

Whenever a layerDefinitionModel is synced (something that happens right after the layerDefinitionModel is saved), the frontend checks if the analysis definition needs to be created, in which case, it sends a POST request to the backend (to api/v3/viz/VIS_ID/analyses). Code is here.

Right before the layerDefinitionsCollection is saved, the frontend checks if any of the layerDefinitionModels point to an analysis definition that doesn't exist yet. If that's the case, the above mentioned request is triggered. Only when all analysis definitions have been created successfully, the layerDefinitions are saved in bulk. Code is here.

@viddo raised some concerns about this, as some of this requests could fail and leave the app in an invalid state, or we'd need to add some logic to handle these cases. For that reason we talked about making this operations atomic.
Also, if a map has multiple layers, and new analysis definitions are need for all of them, we're now triggering one request for each of them (instead of saving them in bulk as we do with layers).
It's working fine right now so we can wait until ☝ become actual problems, but opening this issue to discuss about how we can go about this.
For instance, analysis definitions could be sent together with the layer information? Instead of doing:
{
  ...
  "options": {
    ...
    "source": "b0"
    ...
  },
  ...
}

The frontend could send:
{
  ...
  "options": {
    ...
    "source": {
      "id": "b0",
      "type": "source",
      "params": {
        "query": "SELECT * FROM untitled_table_116"
      },
      "options": {
        "table_name": "untitled_table_116"
      }
    },
    ...
  },
  ...
}

It'd then be the backend the one that determines if the analysis definition already exists or needs to be created, every time a layer is saved...
@viddo @juanignaciosl @xavijam @javierarce @javisantana thoughts?

I recall discussing about keeping analyses outside layers for a reason, but my memory is really bad.
I think that we kept them separated because, from a conceptual point of view, an analyses doesn't require a layer to exist, and moving them inside would couple them tightly.
Deleting analyses might be an issue. When should backend delete an analysis? Sending an explicit DELETE doesn't fit the analyses-inside-layers very well, but not doing it doesn't provide enough information unless backend can know for sure that all analyses of a layer are always sent. Currently we're not even storing the layer id of an analysis, and we should.
These are my first, quick thoughts. I would really love rewinding to the conversations we had about this a couple of months ago, maybe somebody else can. I can't tell whether this is a clear win or not. It feels to me like changing some problems with others, but I might be wrong. You (frontenders) are the ones who are actually struggling with this, so it's up to you.

Thx for taking the time to write the issue, @alonsogarciapablo

@juanignaciosl I __think that we kept them separated because, from a conceptual point of view, an analyses doesn't require a layer to exist, and moving them inside would couple them tightly.

The first part is correct, and arguably the second one, too.
That said, while we could argue back and forth regarding the concepts and how they should be modeled, the main issue (at least from my point-of-view) is the lack of control of state mutations/lack of error-handling. When a state change involves more than one object, each save/state change could lead to erroneous state, and "rolling back" by doing more requests (which themselves could fail…) is far from ideal.  It's becoming even more obvious with #7878 (drag'n'drop nodes to create a new layer), since that involves both creating new layer/analysis and updating existing layer(s).
It would be good to have a conversation and decide how we want to solve this.

As of #8042 this afternoon I managed to pull out the implicit side-effects of listeners to a common place, so it's slightly better controlled), but proper error-handling/recovery is still missing. I'd like to have a conversation about this promptly to address this issue.


object

does this return one object? or a object for each row..
from the code i think its one big object.. correct?

It returns an array, with each row of data an object.
Amazing. thanks for this work max. I've successfully used your package! keep it up!
Now onto inserting the stuff into mongo!


[Q] use Keyestudio Electromagnet Module

will this module work as alternative to the proposed circuit?
https://www.keyestudio.com/products/keyestudio-electromagnet-module-for-arduino-diy-projects

Good question! In short: I'm not sure, but it's worth a shot.
The longer answer is:

The product states it draws 0.3A (and, somewhat contradictory, 3W, implying 0.6A), which nominally should be safe to pull from the Flipper (which has a fuse set for 1.2A on the 5V pin). That said, it's possible there would be transient draws larger than this, or that flipping back and forth could cause the fuse to blow somehow anyway. Hence, do so with caution, and possibly start by just testing feasibility powering it externally and only using the F0 to enable/disable the electromagnet.
To match the standard speed of swiping a card, the app plays data back with a 200-700us clock. If the inductance of this electromagnet is too high, it's possible it won't be able to switch back and forth fast enough. I'm not an electrical engineer, but if my memory of undergrad circuits classes serve me, the time constant here should be L/R (inductance over resistance), neglecting any wonky effects from the ferrite core (retentivity/hysteresis/etc). My uninformed guesstimates would put resistance at ~10ohms, inductance at ~1-100mH, putting the time constant on the order of milliseconds, not microseconds. Hard to say without an actual datasheet (and someone with more EE knowledge than me) or just testing it, but that might make it too slow.

Either way, curious to see what comes of you trying it! Keen for updates if you decide to give it a shot.

thank you for the tips, i'll make some tests with a CC1101 module first.


[MM-19888]: Add documentation for silent installation of MSI

Summary
Added documentation for silent installation of MSI.
Ticket Link
Fixes: #3220
JIRA: https://mattermost.atlassian.net/browse/MM-19888

Hello @haardikdharma10,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Hi @cwarnermm, thank you for your time and suggestions. I have made the necessary changes.

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @jfrerich @emilyacook

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @jfrerich @emilyacook

@deanwhillier, ping for review and welcome hope you had a great holidays

@deanwhillier, ping for review and welcome hope you had a great holidays

Sincere apologies for not getting to this, I somehow missed seeing the review request. Will get on it asap.

Sincere apologies for not getting to this, I somehow missed seeing the review request. Will get on it asap.

@metanerd I see other pages deploy correctly to github pages, but this one is failing, is there something we can do to fix it?

@metanerd I see other pages deploy correctly to github pages, but this one is failing, is there something we can do to fix it?

@metanerd I see other pages deploy correctly to github pages, but this one is failing, is there something we can do to fix it?

I will have a look

@metanerd I see other pages deploy correctly to github pages, but this one is failing, is there something we can do to fix it?

I will have a look

@metanerd I see other pages deploy correctly to github pages, but this one is failing, is there something we can do to fix it?

This is because the PR is from a forked repo and the Github pages refers to our preview server which doesn't spin up previews from forked repos. :)


Total survey time tracking without timer on the screen

Are you requesting a feature, reporting a bug or asking a question?
Asking a question
What is the current behavior?
Cannot track the total time of a survey, without displaying a timer on the screen.
What is the expected behavior?
Would like to know how long a survey takes without showing it on the screen as a timer.
How would you reproduce the current behavior (if this is a bug)?
Provide the test code and the tested page URL (if applicable)
Tested page URL:
Test code
your_code_here

Specify your

browser:
browser version:
surveyjs platform (angular or react or jquery or knockout or vue):
surveyjs version:

Survey model has the timeSpent property - the time in seconds end-user spends on the survey. Does it fit your needs?

@tsv2013  for some reason the timeSpent always returns 0 for me.

@vihansri Do you call survey.startTimer()?

Thank you,
Andrew


Add FreeBSD support

Had this as a dependency on a FreeBSD system so I added a config.h. The end result of my efforts worked so you can consider this at least lightly tested (i.e. It Works On My System).

Thanks! My github notifications seem to be not working so took a while to spot this.


Folder Renaming Stalls

Type: Bug
When I rename a folder the IDE often stalls with a spinning wheel on the status below.  The folder in this case has one file inside, and one more folder with one file inside that so hardly any nesting.
If I then close the VS Code and re-open the folder is indeed renamed.   So the renaming is happening but vs-code is unable to end some sort of loop.
 In the newly created extension, the extension development host editor opened by f5 appears 100%, and then I found that if I enable the extension I wrote before, this will also happen. I don’t know how to solve it.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Display git build info (FINERACT-883)

Display fineracts git build info actuator/info endpoint
Description
Describe the changes made and why they were made. Ignore if these details are present on the associated Jira ticket
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[ ] Commit message starts with the issue number from https://issues.apache.org/jira/projects/FINERACT/. Ex: FINERACT-646 Pockets API.

[ ] Coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions have been followed.

[ ] API documentation at https://github.com/apache/fineract/blob/develop/api-docs/apiLive.htm has been updated with details of any API changes.

[ ] Integration tests have been created/updated for verifying the changes made.

[ ] All Integrations tests are passing with the new commits.

[ ] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the list.)

Our guidelines for code reviews is at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide

@xurror can you locally run and fix ./gradlew rat ? Some license header is missing.
BTW this has nothing to do with FINERACT-905, confusion.
PS please ignore the failed Build - Summary, and only look at Travis (see FINERACT-907).

PS please ignore the failed Build - Summary, and only look at Travis (see FINERACT-907).
Was wondering where it came from. Thanks for the notice

@xurror can you locally run and fix ./gradlew rat ? Some license header is missing.
Looking into it. It's kind of weird rat-report.txt is getting checked for license.

./gradlew rat  passes locally.

Oh now this interesting. Seems .git directory cannot be found here since it is a file which is automatically ignored

It's hard for me to figure out how to pull this off.
I'm thinking to set the option failOnNoDotGitFound to false. @vorburger, @awasum do you think we can go with that.

@xurror  i saw the build, in your case following test case is failing:
org.apache.fineract.integrationtests.AccountingScenarioIntegrationTest > checkPeriodicAccrualAccountingFlow FAILED
java.lang.AssertionError at AccountingScenarioIntegrationTest.java:588
For now may be you want to ignore the the testcase . But definitely it is not a long time solution.
It appears to me a concurrency issue on Loan entity object and is not limited to integration tests.

@xurror re. test failure, follow #782 to find FINERACT-899 and note #774, I think if you rebase this, that will disappear.

Oh now this interesting. Seems .git directory cannot be found here since it is a file which is automatically ignored

I don't think the .git/ is a file which is automatically ignored, but I suspect there are several situations where it's not present... this project certainly should still build even if there is no .git/ directory present.  This could happen if someone builds from the Release ZIP, and a few other situations I can imagine...

I'm thinking to set the option failOnNoDotGitFound to false. @vorburger, @awasum do you think we can go with that.

... therefore something like that would probably be acceptable / good idea; see FINERACT-911.
However I do not find an option failOnNoDotGitFound so I suspect this was a typo?

@xurror re. test failure, follow #782 to find FINERACT-899 and note #774, I think if you rebase this, that will disappear.

Oh now this interesting. Seems .git directory cannot be found here since it is a file which is automatically ignored

I don't think the .git/ is a file which is automatically ignored, but I suspect there are several situations where it's not present... this project certainly should still build even if there is no .git/ directory present. This could happen if someone builds from the Release ZIP, and a few other situations I can imagine...

I'm thinking to set the option failOnNoDotGitFound to false. @vorburger, @awasum do you think we can go with that.

... therefore something like that would probably be acceptable / good idea; see FINERACT-911.
However I do not find an option failOnNoDotGitFound so I suspect this was a typo?

Oops, typo. It's failOnNoGitDirectory.

@vorburger, this passes now. Hope the you OK with the actual changes


Allow specifying excluded paths

fixes #30

This package is no longer being maintained, users should migrate to the opentelemetry-api package.
Closing.


I need help with an error.

Whenever I start up HiyaCFW on my Ds it says "Erorr! Failed to read Launcher's title.tmd". Is there any posssible fix?

Check if the file title/00030017/484e41xx/content/title.tmd in your SD card is 520 bytes long

Not a helper's issue. Closing...


Allow Keptn CLI to use a proxy for accessing the Keptn API

Some of our users can't access the Keptn API directly but have to go through a Proxy. If they want to use the Keptn CLI as their choice of automation tool they currently cant use it as the Keptn API doesnt allow to specify a proxy to be used.
One option would be to extend the keptn auth command with an optional proxy setting, e.g: proxy url, proxy user, proxy password

Might be helpful: https://stackoverflow.com/questions/14661511/setting-up-proxy-for-http-client

It would be really great if this was solved. I think the problem is in cli/cmd/check_endpoint_status.go There is a custom transport object and no proxy setting so setting environment variable does not work. I think I cloud make it work with something like this (adding the proxy line).
var client = http.Client{
Transport: &http.Transport{
TLSClientConfig: &tls.Config{InsecureSkipVerify: true},
Proxy: http.ProxyFromEnvironment,
},
}

Hi @mpolonen, in https://github.com/keptn/keptn/issues/6343#issuecomment-1020998889 we updated the documentation to explain how to enable a proxy via env var. We offer two env var: HTTP_PROXY and HTTPS_PROXY.


Correcting E2e

-Making changes to run on debian
-Adding change for rerun
-Adding change for validating co
-Updating README.md
Signed-off-by: Shilpi Das<EMAIL_ADDRESS>

@pravin-dsilva @yussufsh  Please review this. This code is same as our internal repo

/approve

/lgtm


About LyCORIS

First of all, thanks you guys implement this kind of useful tools!
But I want to point out some problems or some questions:

LyCORIS is not LyCoris.
I recommend you guys to implement some kind of "plugin system" like kohya(which allowed custom addon network) or HCP-Diffusion(which allowed custom add on layers). So "support new algorithm" can be done by community easily.
If you guys have plugin API, let me know and I can just support your framework in LyCORIS directly. (But actually you can also just import my package and patch the layer you want to patch directly XD)

First of all, thanks you guys implement this kind of useful tools! But I want to point out some problems or some questions:

LyCORIS is not LyCoris.
I recommend you guys to implement some kind of "plugin system" like kohya(which allowed custom addon network) or HCP-Diffusion(which allowed custom add on layers). So "support new algorithm" can be done by community easily.
If you guys have plugin API, let me know and I can just support your framework in LyCORIS directly. (But actually you can also just import my package and patch the layer you want to patch directly XD)

Thank you very much for your feedback!

Regarding the spelling, we'll make sure to correct that.
2&3. Your suggestion about implementing a "plugin system" is a great idea.  It would be great to collaborate with LyCORIS in supporting our framework. In the meantime, we'll explore the possibility of importing your package and patching the necessary layers directly. Once again, thank you for your valuable feedback, and we look forward to further discussions and collaborations.