Unable to access data inside alert section of aggregation rule

I have been trying to set up elastalert monitoring on my ELK stack. For the beginning I want to set up a simple rule which will generate a notification if any disk on the file system has reached 80% usage. The rule seems to be working correctly but in the alert section I am not able to pass the data to python script. The uncommented command in the alert section gives following error.
ERROR:root:Error while running alert command: Error formatting command: 'system.filesystem.mount_point'
Here is my rule file. Please excuse the formatting of the yaml.
name: Metricbeat high FS percentage
type: metric_aggregation

es_host: localhost
es_port: 9200

index: metricbeat-*

buffer_time:
minutes: 1

metric_agg_key: system.filesystem.used.pct
metric_agg_type: max
query_key: beat.name.keyword
doc_type: metricsets

bucket_interval:
  minutes: 1

realert:
  minutes: 2

sync_bucket_interval: true
#allow_buffer_time_overlap: true
#use_run_every_query_size: true

max_threshold: 0.8

filter:
- query:
    query_string:
      query: "system.filesystem.device_name: dev"
      analyze_wildcard: true
- term:
    metricset.name: filesystem

# (Required)
# The alert is use when a match is found

alert:
  - debug
  - command
command: ["/home/ubuntu/sendToSlack.py","beat-name","%(beat.name.keyword)s","used_pc","%(system.filesystem.used.pct_max)s","mount_point","%(system.filesystem.mount_point)s"]
 # command: ["/home/ubuntu/sendToSlack.py","--beat-name","{match[beat.name.keyword]}","--mount_point","{match[system.filesystem.mount_point]}"]
 # command: ["/home/ubuntu/sendToSlack.py","--beat-name","{match[beat][name]}","--mount_point","{match[system][filesystem][mount_point]}"]
 #pipe_match_json: true
 #- command:
 #    command: ["/home/ubuntu/sendToSlack.py","%(system.filesystem.used.bytes)s"]

Some observations: On testing the rule file using the command python -m elastalert.test_rule rules/high_fs.yaml I get the output
Successfully loaded Metricbeat high FS percentage

Got 149161 hits from the last 1 day

Available terms in first hit:

tags
beat.hostname
beat.name
beat.version
type
@timestamp
system.filesystem.available
system.filesystem.files
system.filesystem.mount_point
system.filesystem.free_files
system.filesystem.free
system.filesystem.device_name
system.filesystem.used.bytes
system.filesystem.used.pct
system.filesystem.total
host
@version
metricset.rtt
metricset.name
metricset.module

I should be able to access any of the fields mentioned above. When I run this rule using  python -m elastalert.elastalert --verbose --rule rules/high_fs.yaml a list is printed on the screen
@timestamp: 2017-10-18T17:15:00Z
beat.name.keyword: my_server_name
num_hits: 98
num_matches: 5
system.filesystem.used.pct_max: 0.823400020599

I am able to access all the key value pairs in this list. Anything thats outside the list fails with the above mentioned formatting error. Been stuck over this for long. Any help is appreciated.
Its quite possible that my understanding is wrong or I am missing something obvious.

I was able to get a workaround on this. Not sure if its the right way. I changes the type to any and then wrote my own query. Here is how one of my rules file look currently.
Name: High CPU percentage
type: any

es_host: localhost
es_port: 9200

index: metricbeat-*
query_key:
  - beat.name

filter:
- range:
    system.cpu.total_norm_pct:
      from: 0.9
      to: 10.0

realert:
  minutes: 60

alert:
- command:
    command: ["/home/ubuntu/slackScripts/sendCPUDetails.py","{match[beat][name]}","{match[system][cpu][total_norm_pct]}"]
new_style_string_format: true

Sorry, this isn't really called out in the docs at all. Some rule types and options (metric_aggregation, use_count_query, use_terms_query, flatline) do not contain full field data because they are either based on numbers or can be trigged by absence of data.
I'll add a section to the docs and call this out in the alert_text_args/command bits.

Thanks for your reply, Yeah It will be helpful if its mentioned in the doc.

@shshnkTrxcn I ran into same problem.  However, I'm unable to understand how you workaround.  Can you clarify what you mean by "wrote my own query" and how changing "type" to "all" helped.

@ftc13184
"Wrote my own query"
filter:
- range:
    system.cpu.total_norm_pct:
      from: 0.9
      to: 10.0

With that query and type: any, any document showing .9>10 cpu will trigger an alert.


shriyadindi-upr

Unmerged PR

They are the PRs, just open the link

i can see your other PRs separately in this repo..
clear me if those contains this ?

i will review them now i think you created many separate

I've first created a single repo containing all the PRs but then many of them have created a separate pr for each unmerged pr so I thought that is the way and i created them separately

yeah its fine

will review them

@aslams2020 okay


Inconsistent Execution of instrument_bb_coverage Function A

t appears that the instrument_bb_coverage function is not triggered every time a basic block is executed. I added the following code at the beginning of instrument_bb_coverage:
dr_fprintf(log_file, "bb called %d\n", bb_count);
bb_count++;
if (bb_count == 100000) bb_count = 0;

Upon reviewing the log_file, I noticed that the output increases rapidly when fuzzing begins. However, over time, it slows down and eventually stops at different points for different applications. Could you please help me understand why this might be happening?
Thank you for your assistance!

That sounds correct. instrument_bb_coverage is not meant to run each time a basic block is executed, but rather each time a basic block is translated. In DR, my understanding is that this generally occurs the first time a basic block is executed. instrument_bb_coverage then writes assembly instructions to the basic block (using instrlist_meta_preinsert) and those assembly instructions are executed each time a basic block executes.

Thanks for your help! However, I also noticed some repeated addresses in the log file, though they don't appear too often. How should I understand this?

That sounds correct. instrument_bb_coverage is not meant to run each time a basic block is executed, but rather each time a basic block is translated. In DR, my understanding is that this generally occurs the first time a basic block is executed. instrument_bb_coverage then writes assembly instructions to the basic block (using instrlist_meta_preinsert) and those assembly instructions are executed each time a basic block executes.

Thanks for your help! However, I also noticed some repeated addresses in the log file, though they don't appear too often. How should I understand this?

Can you explain what you mean by "repeated addresses"?

I added the following code at the beginning of instrument_bb_coverage, intending for the offset of each executed basic block to be logged in log_file (a file_t object initialized in dr_main).
dr_fprintf(log_file, "bb called %d\n", bb_count); bb_count++; if (bb_count == 100000) bb_count = 0;
However, over time, the log_file stops increasing. You explained that instrument_bb_coverage is only called the first time a block is translated, yet I observe some addresses appearing more than once. Why are these blocks translated multiple times? Thank you!

If the target process restarts for any reason, code is going to need to be reinstrumented (during a fuzzing session this occurs if maximum number of persistent iterations has been reached, on crashes, hangs etc.) so translation will occur again. It's also possible that DynamoRIO sometimes translates basic blocks multiple times for whatever reason (e.g. if it suspects a basic block has changed, as a part of some optimization?), but I'm not that familiar with DR internals. Such questions I can better answer for TinyInst, which I developed myself (TinyInst will not translate a basic block twice unless the target process restarts)

Amazing! You're the author of TinyInst! Thank you so much for your help!


Incorrect Crash Message

If you accidentally define a biome twice under CustomBiomes in WorldConfig.ini, an incorrect crash message is produced.
e.g if Ocean Island is designated id 75 and 614, the following crash report is generated
net.minecraftforge.fml.common.LoaderExceptionModCrash: Caught exception from Open Terrain Generator (openterraingenerator) Caused by: java.lang.RuntimeException: Could not allocate the requested id 614 for biome Ocean Island. All available id's under 256 have been allocated
even though all id's under 256 have not been allocated. Which could be misleading for new pack makers n stuff.
Thanks,
snarp

Hey @Virtualistik, thanks for the feedback! This is fixed with v7 dynamic biome id's, which retires custombiomes (it only still exists to support legacy worlds).


Shared Mailbox Licensing

O365 admin portal does not show shared mailbox user accounts, licensed or not.
I could not find any current instructions on how to add a license to a shared mailbox.

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

ID: fd8b6d7d-1a95-d263-595e-78649069275f
Version Independent ID: bacc64b9-916b-bb3d-a885-46fe5d096826
Content: Remove license from shared mailbox - Microsoft 365 admin
Content Source: microsoft-365/admin/email/remove-license-from-shared-mailbox.md
Service: o365-administration
GitHub Login: @SKjerland
Microsoft Alias: sharik

@DanMan32 Thank you for your feedback. I was able to test this on my lab and can view the details under users-->Active users. All mailboxes are listed here and usually shared mailboxes that are not licensed show up as unlicensed.

Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @DanMan32 Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


feat(OCPADVISOR-72): Render upgrade risks banner

Implements https://issues.redhat.com/browse/OCPADVISOR-72.
Mocked at https://www.sketch.com/s/c5c07e06-f4a1-4c64-9cdf-8b12d8ac9ad5/a/8ylYyDw.
This makes the cluster view render the upgrade risks banner which indicates whether there are any upgrade risks for this cluster.

Must render the warning alert if there is at least one upgrade risks.
Must render the success green alert if the are no upgrade risks for this cluster or the cluster is managed.
Must render the warning alert "not currently available" if back end returns 503.
Must not render the alert in any other cases.

How to test
The same as for https://github.com/RedHatInsights/ocp-advisor-frontend/pull/547#issue-1665942399: try to run the PR together with the mock service. For different clusters (scenarios), you should get different results.
Screenshots
Has some risks

Has no risks/is a managed cluster

Service is down (503 status code)

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.16 :tada:

Comparison is base (f9e3031) 87.82% compared to head (1510695) 87.98%.

:exclamation: Current head 1510695 differs from pull request most recent head bf30d60. Consider uploading reports for the commit bf30d60 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #549      +/-   ##
==========================================
+ Coverage   87.82%   87.98%   +0.16%
==========================================
  Files          31       32       +1
  Lines        1265     1282      +17
  Branches      486      499      +13
==========================================
+ Hits         1111     1128      +17
  Misses        154      154

Impacted Files
Coverage Δ

src/Components/Cluster/Cluster.js
100.00% <100.00%> (ø)

.../Components/UpgradeRisksAlert/UpgradeRisksAlert.js
100.00% <100.00%> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

LGTM


@squoosh/cli does not support GIF

When I try to convert from GIF I get this error
Binary blob has an unsupported format
Other formats work fine (jpg, png)
Node v16.13.0
origImage = await s3.getObject({ Bucket: dstBucket, Key: inputImgPath }).promise();
image = imagePool.ingestImage( origImage.Body);
await image.decoded;
await image.encode({ });
const outBuffer = (await image.encodedWith.avif).binary;

Yeah you are absolutely fine. The CLI doesn’t support GIF. In the web app we can rely on the browser to decode GIFs. In Node we’d have to ship a decoder of our own and I completely forgot about this format. If anyone wants to add a wasm GIF decoder, that’d be welcome.

Unfortunately, due to a few people leaving the team, and staffing issues resulting from the current economic climate (ugh), the CLI and libsquoosh packages are no longer actively maintained. I know that sucks, but there simply isn't the time & people to work on this. If anyone from the community wants to fork it, you have my blessing. The squoosh.app web app will continue to be supported and improved.


datafable.com is offline

This impacts rendering of http://peterdesmet.com/posts/analyzing-gbif-data-licenses.html
It appears this is very recent https://web.archive.org/web/20230320104459/http://peterdesmet.com/posts/analyzing-gbif-data-licenses.html vs https://web.archive.org/web/20230405182853/http://peterdesmet.com/posts/analyzing-gbif-data-licenses.html
In Chromium-based browsers, the page renders, just with some blank spaces where visualizations were. In Firefox, rendering seems to hang waiting on requests to datafable.com

Thanks for reporting. The domain datafable.com was recently discontinued. I've updated the links to datafable.github.io and the charts load again now.


Convert to use Shared AppVeyor Code and Add CodeCov.io

I'll work on this over the weekend.

This was completed in https://github.com/PowerShell/xCertificate/pull/48


UX meeting agenda 17th August 2016

Horizontal panel UX.
Debug UX
https://github.com/Microsoft/vscode/issues/10437
https://github.com/Microsoft/vscode/issues/9740
https://github.com/Microsoft/vscode/issues/1931

@stevencl fyi Isidor is still on vacation.

#10546 Tabs for integrated terminal? (see my last comment in this issue)

@bpasero here are the icons.
Simple Explorer Icons.zip

Thanks!


Added Registration page

need to go back through and add navbar and more styling and condense the html file

sgtm


Update download scripts

Description

Bug fix: daily files for 2021 weren't being saved since date.max() resolved to 12-31-2020 instead of 01-DD-2021.
Increase marker-size in geo-scatterplot.

Fixes # (issue)
#13
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Screenshots

Checklist:

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

Sorry, I accidentally closed this from the GitHub App for Android.

Sorry, I accidentally closed this from the GitHub App for Android.


Minor change to Advanced example...

This is a very minor change. In the first example you had two top-bar-left classes. It seems more logical to have one left and one right as that example which will then match the example below that.

Makes sense to me, thanks!


Vuetify3

Is there an update planned for Vuetify3 ?

Hello @PaulMenheere
Vue datatable url sync support vue 2.6, vue 2.7 and vue 3.0 with the help of vue-demi.
Have you an issue with vue 3 ? You can check the Vue3 example: https://github.com/socotecio/vue-datatable-url-sync/tree/main/vue3-example


OAI not stopping

Collecting data from DLESE
http://uc.dls.ucar.edu/dds_oai_server/services/oaiDataProvider/oai_explorer.jsp
this says 7316 records
http://uc.dls.ucar.edu/oai?verb=ListIdentifiers&metadataPrefix=gmd
Server count now at 300k, even though entire site only has 36k records
http://uc.dls.ucar.edu/oai?verb=ListRecords&metadataPrefix=oai_dc

not really sure what is going on. Maybe the DLESE server?

The Mercury OAI has no resumption token, and 271 records.
Does not stop.
https://mercury.ornl.gov/oai/provider?verb=ListIdentifiers&metadataPrefix=fgdc
https://mercury.ornl.gov/oai/oaisearch.do

OAI.patch.txt
patch

Patch applied with https://github.com/Esri/geoportal-server-harvester/commit/40cdc65aa34a576d79be86bf64501317530e9813

Verified.


Adjust timestamp rounding if needed

Take a look whether  edgedb/edgedb#4885 is relevant for Dart.

All Dart date/time types use microsecond precision, so I don't think any changes to decoding/encoding are needed.

Good. I thought all newer languages use nanoseconds, but good for us.


Maintained fork

First of all, thank you @endclothing guys for maintaining this package after Jimdo.
Sadly it looks like this repository is not maintained anymore (last activity was in Feb.), so I forked it to https://github.com/lkaemmerling/prometheus_client_php. I just released v1.0.3 with a few minor fixes (most important is the removed Guzzle requirement).
I plan to continue to maintain the project, so I would recommend moving to this version. I use the Prometheus client myself in several projects, so you can expect to have an active maintainer with me! (Just as a side note, i work mostly for Open Source stuff @hetznercloud, so i will have time to maintain it)
Maybe @endclothing, if you don't plan to maintain it again, could you add a note to the readme?

How did you remove the Guzzle dependency?
In the PushGateway.php in your repo it still clearly uses it?

Yeah you are right Thank you!
I moved it to require-dev, because if was needed within the tests (but phpstorm didn't find the usage within the PushGateway). I will takle this next week when I am back from vacation. My first idea would be to remove the pushgateway support in the first place and add an additional package which's allows usage of the pushgateway, because not everyone needs the pushgateway support.

I'd like to ask you to keep the PushGateway support, though doing so in a separate package would also be okay.
I'll see if I can see to do something on this next week, my work is probably okay with me dedicating some time to this as we're using it in a lot of projects.

Hey @L3tum,
i "moved" the whole push-gateway support into a dedicated package: https://github.com/LKaemmerling/prometheus_push_gateway_php
Could you please test too? I plan to release a version (1.0.0 of the Push Gateway and 2.0.0 of the Prometheus client [because the removing is a breaking change]) within the next few days, when i don't encounter any other problems within my projects.

@LKaemmerling Ah, cool! I'll test that when I have some time. Looks good though.
In the meantime I also forked your repo 😄 https://github.com/L3tum/prometheus_client_php
I've done some pretty extensive changes over the weekend to make it more streamline to what a project at my company would look like. The next step would be to throw out all those test cases and make some good ones that don't require spinning up 5 containers. Maybe I'll do a PR for your repo once I'm done

Feel free to open a PR :) As i said, i plan to maintain an easy usable fork with as few as needed dependencies :) Maybe your use case might help others too, so feel free to contribute everything and i will try to find a good path between specific "use-cases improvements" and improvements that will help other too :)

Just for the record (cc @L3tum) i just published v2.0.0 of the fork without the PushGateway support. The Pushgateway client was also released as v1.0.0
composer require lkaemmerling/prometheus_client_php

composer require lkaemmerling/prometheus_push_gateway_php

Hey @L3tum,
would you volunteer to maintain the new fork together with me (under https://github.com/PromPHP)?


:badarith error

In the logs I keep getting a :badarith error randomly between 2 to 30 seconds, the stacktrace is (reformatted for readability):
** (ErlangError) erlang error:
[
  reason: :badarith,
  mfa: {Wobserver.Web.Client, :websocket_handle, 3},
  stacktrace: [
    {Wobserver.System.Scheduler, :"-utilization/0-fun-0-", 1, [file: 'lib/wobserver/system/scheduler.ex', line: 37]},
    {Enum, :"-map/2-lists^map/1-0-", 2, [file: 'lib/enum.ex', line: 1229]},
    {Wobserver.System, :overview, 0, [file: 'lib/wobserver/system.ex', line: 54]},
    {Wobserver.Web.Client, :client_handle, 2, [file: 'lib/wobserver/web/client.ex', line: 47]},
    {Wobserver.Web.Client, :websocket_handle, 3, [file: 'lib/wobserver/web/client.ex', line: 5]},
    {:cowboy_websocket, :handler_call, 7, [file: 'c:/Users/<user>/Projects/my_server/deps/cowboy/src/cowboy_websocket.erl', line: 588]},
    {:cowboy_protocol, :execute, 4, [file: 'c:/Users/<user>/Projects/my_server/deps/cowboy/src/cowboy_protocol.erl', line: 442]}
  ],
  msg: {:text, "{\"command\":\"system\",\"data\":null}"},
  req: [
    socket: #Port<0.89433>,
    transport: :ranch_tcp,
    connection: :keepalive,
    pid: #PID<0.1515.0>,
    method: "GET",
    version: :"HTTP/1.1",
    peer: {{10, 1, 2, 158}, 64429},
    host: "<user>",
    host_info: :undefined,
    port: 80,
    path: "/observer/ws",
    path_info: :undefined,
    qs: "",
    qs_vals: :undefined,
    bindings: [],
    headers: [
      {"host", "<user>"},
      {"user-agent", "Mozilla/5.0 (Windows NT 10.0; WOW64; rv:51.0) Gecko/20100101 Firefox/51.0"},
      {"accept", "text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8"},
      {"accept-language", "en-US,en;q=0.5"}, {"accept-encoding", "gzip, deflate"},
      {"sec-websocket-version", "13"}, {"origin", "http://<user>"},
      {"sec-websocket-extensions", "permessage-deflate"},
      {"sec-websocket-key", "S82iJ2lUI+jBYrl9P6icQQ=="},
      {"cookie", "_my_server_key=snip"},
      {"connection", "keep-alive, Upgrade"},
      {"pragma", "no-cache"},
      {"cache-control", "no-cache"},
      {"upgrade", "websocket"}
    ],
    p_headers: [
      {"sec-websocket-extensions", [{"permessage-deflate", []}]},
      {"upgrade", ["websocket"]},
      {"connection", ["keep-alive", "upgrade"]}
    ],
    cookies: :undefined,
    meta: [websocket_version: 13, websocket_compress: false],
    body_state: :waiting,
    buffer: "",
    multipart: :undefined,
    resp_compress: false,
    resp_state: :done,
    resp_headers: [],
    resp_body: "",
    onresponse: :already_called
  ],
  state: %{proxy: nil, state: %{}}
]
  (cowboy) c:/Users/<user>/Projects/my_server/deps/cowboy/src/cowboy_protocol.erl:442: :cowboy_protocol.execute/4

Specifically it is complaining about the division on line 37 in the lib/wobserver/system/scheduler.ex file, the whole function is:
  def utilization do
    ensure_started()

    case last_utilization() do
      false ->
        get_utilization()
        |> Enum.map(fn {_, u, t} ->u / t end)
      last ->
        get_utilization()
        |> Enum.zip(last)
        |> Enum.map(fn {{_, u0, t0}, {_, u1, t1}} -> (u1 - u0) / (t1 - t0) end)
    end
  end

I also see the error happen on the other Enum.map line as well, specifically it seems t can be 0 in the first Enum.map and t1-t0 can be 0 in the other Enum.map at times, checks probably need to be added to do an alternative action instead.

Rookie mistake.... I remember thinking: I need to right a 0 check for this.
Will fix.

Rookie mistake.... I remember thinking: I need to right a 0 check for this.

Heh, we've all done those plenty.  ^.^

Fixed in release v0.1.7.


Fix for setupRefund bug (Issue #19)

Hey I fixed the bug was preventing the refund exchange from getting thru, cuz we're using bitcore and need payment channel code to work.
It seems to be something to do with how Transaction.change was being used, so I used Transaction.to in place of it, calculating the fees manually.
I tested it on code communicating with BitcoinJ and it successfully did the refund exchange.
Wasn't able to figure out how to do gulp lint etc. on it, as there was no lint target in the gulp file, but I don't understand gulp very well. If anybody wants to help me out with that, I'll run the unit tests and stuff.

Can you check the status of this with the latest merge?

It looks like the latest merge covers the issues I was addressing, but more cleanly. Next time!

Actually, I take it back. The issue is setupRefund expects the commitmentTx to have an output, and it doesn't. It should be added in processFunding, I think. Something like:
Consumer.prototype.processFunding = function(utxo) {
$.checkArgument(_.isObject(utxo), 'Can only process an array of objects or an object');
this.commitmentTx.from(utxo);
this.commitmentTx.to(this.commitmentTx.address, utxo.amount - this.commitmentTx.getFee());
};
Reopening to get attention on this.

I take that back. There's still the issue that setupRefund expects the commitmentTx to have an output, and it doesn't. I think it should be in processFunding:
Consumer.prototype.processFunding = function(utxo) {
$.checkArgument(_.isObject(utxo), 'Can only process an array of objects or an object');
this.commitmentTx.from(utxo);
this.commitmentTx.to(this.commitmentTx.address, utxo.amount - this.commitmentTx.getFee());
};
commitmentTx.address is a P2SH 2-of-2 address with both provider's and consumer's public keys.
Reopening this to get attention on it. I'll make another pull request if it makes sense.


What to ingore in andorid/ directory

Hi, thanks for developing this tool.
In the ReadMe you mention that it's good to ignore some files in the resoures/ directory. I think it's a good advice, but it seems to me that much bigger benefit would come from ignoring resources that are copied to the android/ directory. Unfortunately being a noob at Android development I'm not sure which paths I should ignore. They seem to be scattered in various sub-directories. It seems to me that these patterns should be added to .gitignore:
android/app/res/mipmap/*
android/app/src/main/res/drawable-*/splash.png
android/app/src/main/res/drawable/splash.png
android/app/src/main/res/mipmap-*/*.png

Am I right? Or is there a good reason to keep them in version control? I'd love some guidance here.
Bonus points for same advice for iOS. I have not ventured there yet, but expect a similar question.

Thanks for the question. In Capacitor you want to check in your full native project. We're dropping Cordova support in 1.0 so this question won't be relevant and I don't have a good answer for you.


Issues templates

I suggest adding issue templates to simplify the resolution of questions or problems. You can view the templates here.
After submission, labels will be automatically created and descriptions will be formatted more detailedly. Here is an example: click.
Unfortunately, my knowledge of the English language does not allow me to guarantee the complete absence of errors or inaccuracies in this text. If you find any shortcomings, please make your corrections. The editor has a built-in tutorial that will help you create your own templates.

Sure, why not. Cheers!


Labels don't work

Can't seem to get the labels to show. Searching the DOM I can't find the label: '<i class="map-icon-parking"></i>' that was set.

@kartikluke, can you give source code - i had some problems with labels

hi,
label is not shown for me neither and I get this error too:
Uncaught TypeError: undefined is not a function (program):1
rT                                              (program):1
(anonymous function)                            maps.gstatic.com/maps-api-v3/api/js/20/3/main.js:15
(anonymous function)                            maps.gstatic.com/maps-api-v3/api/js/20/3/main.js:45

I'm using the package in Meteor
thanks,
tinto

Same issue here

Had the same problem initially. You need to make sure too include the css styling of the marker-label and marker-icon classes in. This should be made in your main css file:
.marker-label,
.marker-icon {
    z-index: 99;
    position: absolute;
    display: block;
    margin-top: -50px;
    margin-left: -25px;
    width: 50px;
    height: 50px;
    font-size: 30px !important;
    text-align: center;
    color: #FFFFFF;
    white-space: nowrap;
}

and styled to preference
Hope this helps!

Fixed in v3.0.0

@tintoverano, How exactly did you get the package to work in meteor?  I didn't see any packages in atmosphere or did I miss it?


build error in windows 7 x64

"C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" install font-manager
Exit code: 1
Standard error:
npm WARN package.json<EMAIL_ADDRESS>No description
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:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\lib\build.js:276:23)
gyp ERR! stack     at emitTwo (events.js:87:13)
gyp ERR! stack     at ChildProcess.emit (events.js:172:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)
gyp ERR! System Windows_NT 6.1.7601
gyp ERR! command "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" "rebuild"
gyp ERR! cwd E:\workspace\MintReader2\node_modules\font-manager
gyp ERR! node -v v4.4.3
gyp ERR! node-gyp -v v3.3.1
gyp ERR! not ok
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "install" "font-manager"
npm ERR! node v4.4.3
npm ERR! npm  v2.15.1
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the font-manager package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs font-manager
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!
npm ERR!     npm owner ls font-manager
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     E:\workspace\MintReader2\npm-debug.log
Standard output:

<EMAIL_ADDRESS>install E:\workspace\MintReader2\node_modules\font-manager
node-gyp rebuild

E:\workspace\MintReader2\node_modules\font-manager>if not defined npm_config_node_gyp (node "C:\Program Files\nodejs\node_modules\npm\bin\node-gyp-bin....\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "" rebuild )
Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
E:\workspace\MintReader2\node_modules\font-manager\build\fontmanager.vcxproj(20,3): error MSB4019: The imported project "E:\Microsoft.Cpp.Default.props" was not found. Confirm that the path in the  declaration is correct, and that the file exists on disk.

Facing same error. Any solution for this ?

After installing vc++ and adding path resolved the issue.

As I was having VS2017 installed in my machine , Installed VC++ as per below.
https://stackoverflow.com/questions/33154696/msbuild-error-the-builds-tools-for-v140-platform-toolset-v140-cannot-be-f

Set Environment variable VCTargetsPath
VCTargetsPath to C:\Program Files (x86)\Microsoft Visual Studio\2017\BuildTools\Common7\IDE\VC\VCTargets


Is there possible for one app register itself as multiple services in the service registry?

Is your feature request related to a problem? Please describe.
Is there possible for one app register itself as multiple services in the service registry? That will be transparent to the clients. The clients can still use the old service name to access the service.
Describe the solution you'd like
Is there any config can achieve this?
Describe alternatives you've considered
Is there possible to use an utility class to register services?
Thanks in advance. Any help wouble be greateful :)

ping :)

Not automatically. You can use the ServiceRegisitry bean to do it manually.


Flaky crash on vm-reload on test corelib_2/bigint_parse_radix_test

This flaky crash is occurring repeatedly on the linux-release-reload bots. Output with stack trace is at the bottom of this message.
The test is corelib_2/bigint_parse_radix_test. I will mark the test as crashing flakily.  It crashes on builder https://luci-milo.appspot.com/buildbot/client.dart/vm-linux-release-x64-reload-be/
A related test was crashing flakily, but seems to have resolved itself: #31660
https://github.com/dart-lang/sdk/issues/31660
Error with stacktrace from
https://luci-milo.appspot.com/buildbot/client.dart/vm-linux-release-x64-reload-be/11911
AILED: none-vm-checked release_x64 corelib_2/bigint_parse_radix_test
Expected: Pass Timeout
Actual: Crash
--- Command "vm" (took 05.000897s):
DART_CONFIGURATION=ReleaseX64 out/ReleaseX64/dart --no-preview-dart-2 --enable_asserts --enable_type_checks --hot-reload-test-mode --no_intrinsify --sync-async --ignore-unrecognized-flags --packages=/b/build/slave/vm-linux-release-x64-reload-be/build/sdk/.packages /b/build/slave/vm-linux-release-x64-reload-be/build/sdk/tests/corelib_2/bigint_parse_radix_test.dart
exit code:
-6
stderr:
===== CRASH =====
version=2.1.0-edge.56957ff17161f426684bb32c281b2c23d1df0398 (Tue Sep 4 08:54:59 2018 +0000) on "linux_x64"
si_signo=Segmentation fault(11), si_code=1, si_addr=0x2a43
Dumping native stack trace for thread 1e79
[0x00007fadc1f2fbdc] RegExp.matcher
[0x00007fadc1f2fbdc] RegExp.matcher
[0x00007fadc5540da3] [Stub] InvokeDartCode
[0x00007fadc68e5f39] dart::DartEntry::InvokeFunction(dart::Function const&, dart::Array const&, dart::Array const&, unsigned long)
[0x00007fadc6a77b8a] dart::IRRegExpMacroAssembler::Execute(dart::RegExp const&, dart::String const&, dart::Smi const&, bool, dart::Zone*)
[0x00007fadc684d155] Unknown symbol
[0x00007fadc684cdb2] dart::BootstrapNatives::DN_RegExp_ExecuteMatch(_Dart_NativeArguments*)
[0x00007fadc5540a59] [Stub] CallBootstrapNative
[0x00007fadc1f0fe35] _RegExp._ExecuteMatch
[0x00007fadc55413c0] [Stub] DeoptimizeLazyFromReturn
[0x00007fadc1f0a316] _BigIntImpl.parse
[0x00007fadc1f09e8f] BigInt.parse
[0x00007fadbbd5918f] testParseRadix.testParse
[0x00007fadc33e8a64] _Closure.call
[0x00007fadc1f00850] testParseRadix
[0x00007fadc33fef3f] main
[0x00007fadc33fee18] main
[0x00007fadc33f42fb] _Closure.call
[0x00007fadc33fdf04] _startIsolate.
[0x00007fadc33cfd7e] _Closure.call
[0x00007fadc33fdb6e] _RawReceivePortImpl._handleMessage
[0x00007fadc5540da3] [Stub] InvokeDartCode
[0x00007fadc68e5f39] dart::DartEntry::InvokeFunction(dart::Function const&, dart::Array const&, dart::Array const&, unsigned long)
[0x00007fadc68e96f3] dart::DartLibraryCalls::HandleMessage(dart::Object const&, dart::Instance const&)
[0x00007fadc691387b] dart::IsolateMessageHandler::HandleMessage(dart::Message*)
[0x00007fadc694419a] dart::MessageHandler::HandleMessages(dart::MonitorLocker*, bool, bool)
[0x00007fadc694490c] dart::MessageHandler::TaskCallback()
[0x00007fadc6ae297d] dart::ThreadPool::Worker::Loop()
[0x00007fadc6ae2814] dart::ThreadPool::Worker::Main(unsigned long)
[0x00007fadc69efcc9] Unknown symbol
-- End of DumpStackTrace
--- Re-run this test:
python tools/test.py -m release --checked --no-preview-dart-2 --hot-reload corelib_2/bigint_parse_radix_test

This test is already marked as timing out flakily: #31659


feat: scheduler add limit count of filter parent func

Description

Scheduler add limit count of filter parent func.
Default number of filter count is 5.
User can config filter_parent_count in manager console.

Related Issue

Motivation and Context

Screenshots (if appropriate)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[x] Code compiles correctly.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report

Merging #1090 (887bc22) into main (fe8fb6f) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1090      +/-   ##
==========================================
+ Coverage   45.36%   45.45%   +0.08%
==========================================
  Files         130      130
  Lines        9909     9914       +5
==========================================
+ Hits         4495     4506      +11
+ Misses       4889     4883       -6
  Partials      525      525

Flag
Coverage Δ

Object-compatibility-e2etests
?

e2etests
?

unittests
45.45% <100.00%> (+0.08%)
:arrow_up:

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

Impacted Files
Coverage Δ

scheduler/scheduler/scheduler.go
89.05% <100.00%> (+1.92%)
:arrow_up:

cdn/supervisor/progress/progress.go
97.01% <0.00%> (+5.97%)
:arrow_up:


Number of Group Members

User Story
As a user, I would like to have the App indicate the number of group members after I added all of them, so that I can confirm my selections.
Detailed Description
Provide additional details and context.
Acceptance Criteria

[x] Given [Condition], then [Expected Result]
[x] Given [Condition], then [Expected Result]
[x] Given [Condition], then [Expected Result]

Can be added at anytime. Not necessary at this moment. issue closed.

Can be added at anytime. Not necessary at this moment. issue closed.

Removed feature


Add running a workload at the end of deploy

This will e.g generate metrics for ceilometer to show.

Feel free to merge any time, but I'm curious if you have any sort of output showing what it launches? I'd be mostly curious what projects, networks (subnets), servers, images and volumes it loads. If you don't have time to run this again I'll give it a shot later today perhaps. Thanks!

In this case, it's not super satisfying. It complains at the end of its ansible run that a floating ip is not available.
However, a workload named
workload_instance_0 is created, an image named workload_image_0, a network workload_internal_net_0, etc.

Awesome!

Thank you for the review.


Split up T3K perplexity tests into separate jobs for each model

The T3K perplexity test workflow should launch a separate job for each model and send notifications to each owner.

This is has been in main for over a week now 👍


Settings page goes white blank

🔍 Is there already an issue for your problem?

[X] I have checked older issues, open and closed

📝 Description
Settings page won't load up, showing white blank page instead. Unable to interact with app after that, but can reload the page. What I've noticed, there's exception in Settings.tsx and the app is calling github.com API and fails.
Console shows exception:
Settings.tsx:310 Uncaught TypeError: Cannot read properties of null (reading 'login')
Call to github ends with 401 Unauthorized.
Request URL: https://api.github.com/
Request Method: GET
Status Code: 401 Unauthorized

🪜 Steps To Reproduce

Log in to EGH with token. (TBH I am not sure you can log in, I'm already logged in and can't logout. I assume logout button is somewhere in the settings.)
Click settings ⚙ icon.
Page not loading, app becomes white blank.

Gitify Version
5.5.0
Operating System
macOS
GitHub Account
GitHub Enterprise
📸 Screenshots

@setchy Thanks a ton for solving it. Is there any dev build I could use to enter the settings?

v5.6.0 is hot off the press. LMK how that goes for you

Fixed. Thank you. 🙇


Add Yes and No button

Add api to trigger yes and no options from outside the component. I'm new to react native, all ideas are welcome!

I second that idea. I'm actually currently working on it, might send a PR to the mods.

Any advancements gentlemen?

I made some modifications to the library and released it with this app. If you like it, I could look into doing a fork of the latest build and modifying it so it is generic.

It'll be great a PR @Naoto-Ida :)

OK, I'll try and get something out this week.

Any update?

It's possible to use refs to trigger the options yup and nope.
You can check in this repo:
https://github.com/VctrySam/Tinder/blob/master/App/components/home.js#L114
Tip: use _forceLeftSwipe and _forceRightSwipe instead of _goToNextCard

Sorry guys, haven't had time to commit this and it was done when I just started with RN so it's very sloppy.
I had difficulty isolating the buttons from the lib as it was rendered inside of the card container...
I think @vitorebatista's approach is better.

@vitorebatista yes you right, but what if someone wants to animate card and trigger yup/nope function ?


Makefile etc

So, I've bodged the makefile from your previous incarnation of the MQTT software and I can now successfully compile and re-compile it.
However, once it's logged onto my MQTT server and subscribed to the test messages it just sits there doing nothing!
If I send an MQTT message to one of it's topics, it doesn;t respond and if I send any serial data it doesn't respond either.
I think that this has great potential though.
I've thought a lot about it and when you get it further developed it would be great to have a consistent set of messages:- something like
Send
MQTT::
response
OK
or

or
SUBSCRIBE:
response
OK
or
FAIL
When the wifi has connected and the MQTT server has connected maybe a simple message like
READY
I was thinking about how you might connect a newly installed ESP module and perhaps if GPIO2 is held to ground at power up, you could generate a WiFi AP with the SSID as the MAC number of the module.
A simple config web page would then give the following:-
AP SSID and password.
MQTT server URL and port number
MQTT server user and password
An outgoing data topic if required
An incoming data topic if required
Display the units MAC number
This would allow anyone to connect to a new unit with a phone or tablet to configure it.
I can see applications for this in home automation. My friend Peter Scargill and I are designing home automation devices and we've been using NRF24L01 radio modules and ATMEL RF processors with a lightweight mesh.
However, I think that ESP modules with MQTT software is far more useful.
I'd be happy to give you a few mains control units to play with when we get the PCBs finished and integrated with your MQTT software
Regards
Aidan
PS This is the current version of my mains switching unit with a 10 amp relay and a DS1802 temperature sensor

hello Aidan
This library design for native ESP8266 SoC, used for an application run stand alone in this SoC, not for serial. Of course, you can use this library for your application run with other MCU and communicate via serial port.
I have done other library for Web configuration, and can be integrated with this, but i want to make this library clearly

Ah ok! Many thanks for th info
Regards
Aidan
Sent from my iPad
On 3 Jan 2015, at 11:15, "tuanpmt" <notifications<EMAIL_ADDRESS>wrote:
hello Aidan
This library design for native ESP8266 SoC, used for an application run stand alone in this SoC, not for serial. Of course, you can use this library for your application run with other MCU and communicate via serial port.
I have done other library for Web configuration, and can be integrated with this, but i want to make this library clearly
—
Reply to this email directly or view it on GitHubhttps://github.com/tuanpmt/esp_mqtt/issues/5#issuecomment-68589899.


0xB0aF00891c2E0bD32ea37b6E56c030D12a6cF21c

https://github.com/celo-org/celo-monorepo/blob/c1f72b8918d8d6f1da2d988afd410bbd0ebad96a/packages/protocol/contracts/common/Accounts.sol#L266

<EMAIL_ADDRESS>

It is unclear what this issue is about.


RUN "Please describe what I am doing." error

Traceback (most recent call last):
File "/root/miniconda3/envs/videollm/lib/python3.10/site-packages/gradio/queueing.py", line 532, in process_events
response = await route_utils.call_process_api(
File "/root/miniconda3/envs/videollm/lib/python3.10/site-packages/gradio/route_utils.py", line 276, in call_process_api
output = await app.get_blocks().process_api(
File "/root/miniconda3/envs/videollm/lib/python3.10/site-packages/gradio/blocks.py", line 1928, in process_api
result = await self.call_function(
File "/root/miniconda3/envs/videollm/lib/python3.10/site-packages/gradio/blocks.py", line 1526, in call_function
prediction = await utils.async_iteration(iterator)
File "/root/miniconda3/envs/videollm/lib/python3.10/site-packages/gradio/utils.py", line 656, in async_iteration
return await iterator.anext()
File "/root/miniconda3/envs/videollm/lib/python3.10/site-packages/gradio/utils.py", line 649, in anext
return await anyio.to_thread.run_sync(
File "/root/miniconda3/envs/videollm/lib/python3.10/site-packages/anyio/to_thread.py", line 56, in run_sync
return await get_async_backend().run_sync_in_worker_thread(
File "/root/miniconda3/envs/videollm/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 2177, in run_sync_in_worker_thread
return await future
File "/root/miniconda3/envs/videollm/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 859, in run
result = context.run(func, *args)
File "/root/miniconda3/envs/videollm/lib/python3.10/site-packages/gradio/utils.py", line 632, in run_sync_iterator_async
return next(iterator)
File "/root/miniconda3/envs/videollm/lib/python3.10/site-packages/gradio/utils.py", line 815, in gen_wrapper
response = next(iterator)
File "/data3/2024-projects/26_video_gen/videollm-online/demo/app.py", line 89, in gr_liveinfer_queue_refresher_change
history[-1][1] += f'\n{response}'
IndexError: list index out of range

Could you describe your operation

thanks!, Exactly the opposite order

Thank you. Or you can make the video playing, then input query. If you want to have faster inference, cli is recommended


Drop PHP 7.3 and use PHP Cs Fixer

Codecov Report

Merging #53 (f0bce15) into main (0687a01) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main      #53      +/-   ##
============================================
- Coverage     97.84%   97.83%   -0.01%
  Complexity      371      371
============================================
  Files            23       23
  Lines           974      972       -2
============================================
- Hits            953      951       -2
  Misses           21       21

Impacted Files
Coverage Δ

src/Config/Config.php
100.00% <ø> (ø)

src/File/Finder.php
100.00% <ø> (ø)

src/Sniff/BlankEOFSniff.php
100.00% <ø> (ø)

src/Sniff/DelimiterSpacingSniff.php
100.00% <ø> (ø)

src/Sniff/EmptyLinesSniff.php
100.00% <ø> (ø)

src/Sniff/OperatorSpacingSniff.php
100.00% <ø> (ø)

src/Sniff/PunctuationSpacingSniff.php
100.00% <ø> (ø)

src/Sniff/TrailingCommaSingleLineSniff.php
100.00% <ø> (ø)

src/Sniff/TrailingSpaceSniff.php
100.00% <ø> (ø)

src/Command/TwigCsFixerCommand.php
96.96% <100.00%> (ø)

... and 13 more

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


[beta] PoC: replace Sandpack with react-runner

PoC for https://github.com/reactjs/reactjs.org/issues/4338
Preview
@gaearon It's quite rough but should work already, haven't checked why theme is not applied for CodeMirror but it works here

I think this looks very promising. How much work is it to get this styled so that the editor works and we can compare 1:1? I see some sandboxes on https://beta-reactjs-org-n2pucbumf-fbopensource.vercel.app/learn/updating-objects-in-state are a bit borked.

@gaearon There are some visual regressions, as previously we target the to root document but now we use a shadow DOM instead, I think it should be super easy to fix, if we decided to make the switch

I don't think we can decide whether to make a switch until we have a good idea of how it works end-to-end, including all the styling, having the code highlighting hooked up, etc. I know it's a bunch of work but it's hard to compare otherwise.

@gaearon I totally understand, I'll try to fix the styling issues first, it works well in my demo though, and the benefit is obvious, the demo is part of our app, so it's super fast, and can reflect user's input with no delay, no bundling, only compiling the JSX syntax

@gaearon tweaked the styling a bit, though there are still a lot of work to do

A few quick thoughts.
Overall approach
I think it's pretty clear this is the right direction for the tradeoffs we have today. I was hoping we can introduce a mode like this to Sandpack in longer term. The instant feedback is hard to beat. The network waterfalls and other inefficiencies in Sandpack today are unfortunate because our use case is very limited. I think it's plausible the Sandpack package will evolve to support this model as opt-in. I'd like to hear more from their perspective and I'm not saying we should necessarily switch packages to yours. But it seems clear an approach like this is promising, whether through this package or something else.
Production vs Development
One thing that seems unfortunate with this approach is that errors are minified.
E.g. try adding {{}} into some JSX. You'll get:

Minified React error #31; visit https://reactjs.org/docs/error-decoder.html?invariant=31&args[]=object with keys {} for the full message or use the non-minified dev environment for full errors and additional helpful warnings.

But since we're on the docs site, it seems confusing. It also means we'd miss console.errors. (I don't think Sandpack currently displays them, but ideally, I think we'd like to do that for key warning etc.)
One option would be to make the docs site use a development build of React. Seems bad.
Another option would be to load development build of React, but just for sandboxes. Seems okay-ish. Though also feels a bit silly if we just want to render a few examples. I'm not sure about the best approach.
There could be some middle ground. Like use prod build until dev builds loads lazily. Though might be confusing if a sandbox has an error initially.
Fast Refresh
It's not great that sandboxes lose state on edits. Sandpack implements a simple Fast Refresh integration to avoid that. I think you can't do this very easily because you don't have Babel though and can't run the Fast Refresh transform. Maybe some simplified version could be written. It's thorny with multi-file edits though.
Maybe it's ok that it doesn't work. But feels like a regression.

@gaearon Thanks for your thorough feedbacks, here are my rough answers:
Production vs Development
One possible solution is map error message back to decoded one, since the data will be stored in the same app, I think it should not be that hard
Fast Refresh
I haven't tested yet but the transpiler React Runner uses is sucrase, which supports react-hot-loader, not sure if that would work in this case, but again I think that's not a big problem as we can also use Babel to compile the code, then we can use the plugin ecosystem of Babel. The only reason I chose sucrase is the much smaller bundle size, but if we want provide more versatile functionalists, we can use Babel in this case and lazy load it in a worker like some other solutions do
Server Components
This website it built with Next.js which also supports Server Components, probably we can just use it? Or for this case we can just other solutions like Sandpack or CodeSandbox or StackBlitz, we are not bonded to use only one tool for all scenarios
Third Party Libs/Transforms
If we do need babel plugins, we can use Babel as the transplier instead of sucrase, BTW it's works well with styled-components and emotion without any babel plugins

Closing as per comment in the issue, but thank you very much, @nihgwu for taking the time to contribute! 👍


Map stops requesting new tiles when async vector tile source is used.

I've created a map with 2 layers: a TileLayer with the standard XYZ source and a VectorTileLayer with my own VectorTile source. The data for the vector tile is pulled using an XHR request. When the map contains just the TileLayer, it works fine, but when I add the vector layer, the map retrieves an initial set of tiles, but after that, no new tiles are retrieved, despite zooming and/or panning the map.
I've noticed the problem only occurs when the VectorTile source's tileLoadFunction contains the XHR request, i.e, this code will break the map:
tileLoadFunction = (tile, url) =>
{
    tile.setState(TileState.LOADING);

    var request = new XMLHttpRequest();
    request.open("GET", url, true);
    request.responseType = "json";
    request.setRequestHeader("Content-Type", "application/json");

    request.onload = function(e)
    {
        tile.setState(TileState.EMPTY);
    }

    request.send();
}

However, if I simply do this, the map works fine:
tileLoadFunction = (tile, url) =>
{
    tile.setState(TileState.EMPTY);
}

I've confirmed that the onload method for the tile is being fired and that no errors are being generated in the tileLoadFunction.

Please refer to the documentation for a proper tileLoadFunction for vector tile layers. See https://openlayers.org/en/latest/apidoc/module-ol_source_VectorTile-VectorTile.html.
You should not be using setState() except for load errors. Instead, just call setFeatures() on the tile when you have parsed the features.

Unfortunately, my original approach didn't have the calls to setState, but that didn't work so I stripped out all the code and added the setState method calls in order to test (I wasn't sure if I had to set the tile state or not). If I revert back to my original code, I still get the same issue, it still stops retrieving tiles:
tileLoadFunction = (tile, url) =>
{
    var request = new XMLHttpRequest();

    request.open("GET", url, true);
    request.responseType = "json";
    request.setRequestHeader("Content-Type", "application/json");

    request.onload = function(e)
    {
        if(request.response != null)
    {
            var labels = request.response;
            var features = [];

            for(var i = 0; i < labels.length; i++)
            {
                var labelPoint = new Point([labels[i].x, labels[i].y]);

                var label = new Feature();
                label.setGeometry(labelPoint);
                label.set("name", labels[i].label);
                label.set("type", labels[i].type)

                features.push(label);
            }

            tile.setFeatures(features);
        }
    }

   request.send();
}

Ah okay, no I see where I have it wrong now, some of my request responses were coming back as null, and in the code above I'm only setting features on the tile if the request response isn't null.
Once I moved the tile.setFeatures(features) call outside the if(request.response != null) block, it works fine now.
Thanks!


[Bug] Falls into endless loop with Firebase user object.

Deepmerge fails to merge two Firebase user objects.

If you have never worked with Firebase before, it might be annoying for you to reproduce and debug. Also it's difficult for me to make an easy to reproduce setup because you need to be authenticated through firebase...
I'm so sorry!
I'd love the library to detect a Firebase user object and just overwrite the entire thing.
The way it's integrated into my app now is I'm merging all kind of things, but only the Firebase user object bugs. I have no way to let deepmerge know when it's merging the firebase user object without going through some rather untrivial changes. So i'd be great if deepmerge could detect this on its own.
Do you know what makes it special or different from other JS objects? Thakns!

I'm going to guess that the objects have cycles, like
const root = {}
root.child = {
  parent: root
}

deepmerge doesn't have any code to detect/avoid cycles.
You could pass in a custom isMergeableObject that would return false for any objects seen before, that would probably do the trick.

You could also pass in a custom isMergeableObject to return false for Firebase objects.


Make all user registration fields optional except for email and password

You heard it ;)

Or remove the fields:
social (array[social], optional): Social login information
addresses {
street (string, optional),
city (string, optional),
state (string, optional),
zip (string, optional),
}
location {
type (string, optional),
}


OAuth provider returns 403 instead of 401 for expired token

Description:
The HSReplay OAuth provider is returning 403 instead of 401 for an expired token. This means that automatic token renewal is not working as the code only checks for 401 and 400 errors in the NSError extension. Is there a way to work around it without having to explicitly try to refresh the token in case of an unexpected error?
OAuth Provider? (Twitter, Github, ..):
HSReplay.net
OAuth Version:

[ ] Version 1
[x] Version 2

OS (Please fill the version) :

[ ] iOS :
[x] OSX : 10.15.7
[ ] TVOS :
[ ] WatchOS  :

Installation method:

[x] Carthage
[ ] CocoaPods
[ ] Swift Package Manager
[ ] Manually

Library version:

[ ] head
[ ] v2.1.0
[ ] v2.0.0
[x] v1.4.1
[ ] other: (Please fill in the version you are using.)

Xcode version:

[ ] 11.4 (Swift 5.2)

[ ] 11.x (Swift 5.1)

[ ] 10.x (Swift 5.0)

[x] other: 12.1

[ ] objective c

add a new case in code (PR)
https://github.com/OAuthSwift/OAuthSwift/blob/master/Sources/NSError%2BOAuthSwift.swift#L70


Update BSK

Task/Issue URL: https://app.asana.com/0/1108686900785972/1208264562025859/f
Description:
Update BSK reference to latest one including changes from the release fork 201.0.0-1.

Internal references:
Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

Doing it as part of merging in release branch which failed due to conflicts


Sequelize

Please Review ClientForm.jsx & server.js
You can choose to check the form, the logic, or both
Relevant Info:

Used Sequelize to connect and send requests to my database.
POST request is sent to the database locally so it could result in an error for reviewers since there's no db dump file

Check Form Submission

Please open dev tools and verify that the inputted data from the client form prints to the console.

When I submit the form locally the console prints the data and sends it to the database on submit.

Check Logic / Give Suggestions to Improve Legibility / Revise code to Follow Best Practices

Is the logic sound?
Is it readable?
Does it follow best practices?

@margaretgeary Hi Margaret! Could you review the PR and tell me your thoughts?

Feel free to re-request a review once you make edits and I can approve.

Your code looks well-organized and was easy to read and follow the logic. Love that you were able to utilize Sequelize. I look forward to seeing what more you will accomplish!

Thank you so much again! You taught me something new today in regard to the package-lock.json, so I appreciate it!

Looks great! I only have a few comments on removing console.logs/commented lines. Great job otherwise!

Thanks for reviewing the code! It looks a lot cleaner now without the extra comments.

Feel free to re-request a review once you make edits and I can approve.

I added some additional comments to further clarify the code. With that, I think I was able to implement Sequelize sufficiently, so I'll go ahead and merge this in!

Summary of Pushed up Files

ClientForm.jsx
AdminListEntries.jsx
server.js
package.json
package-lock.json
.gitignore


icon moving when clicking the "contributors page"

Bug Description
when you click the contributors' page, the icons are moving for a while
Steps to Reproduce

go on 4c.rocks
click the contributors' button

Assign this to me.

This Issue has been fixed @FrancescoXX by yours truly. 😁😁. Closing this now.