[Bug]: (Install workspace error)

Describe the bug
Write a clear and concise description of what the bug is.
Steps to reproduce the behavior

Go to 'terminal'
Run 'install-workspace'

Expected behavior
Idk what's wrong
Actual behavior
Idk what's wrong
Screenshots
Add screenshots to help explain your problem, if applicable.
Output
PS C:\Users\Admin\Downloads\fire-stone-hosting-master (1)> install-workspace
install-workspace : The term 'install-workspace' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a
path was included, verify that the path is correct and try again.
At line:1 char:1

install-workspace

  + CategoryInfo          : ObjectNotFound: (install-workspace:String) [], CommandNotFoundException
  + FullyQualifiedErrorId : CommandNotFoundException

@KendallDoesCoding  I think you are just trying to run install-workspace instead of npm run install-workspace

Yes

@Arturo-Lopez , @codyseibert Getting this error when I run npm run install-workspace
0 verbose cli [
0 verbose cli   'C:\Program Files\nodejs\node.exe',
0 verbose cli   'C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js',
0 verbose cli   'run',
0 verbose cli   'install-workspace'
0 verbose cli ]
1 info using<EMAIL_ADDRESS>2 info using<EMAIL_ADDRESS>3 timing npm:load:whichnode Completed in 0ms
4 timing config:load:defaults Completed in 1ms
5 timing config:load:file:C:\Program Files\nodejs\node_modules\npm\npmrc Completed in 3ms
6 timing config:load:builtin Completed in 3ms
7 timing config:load:cli Completed in 2ms
8 timing config:load:env Completed in 0ms
9 timing config:load:file:C:\Users\Admin\Downloads\fire-stone-hosting-master (1).npmrc Completed in 0ms
10 timing config:load:project Completed in 2ms
11 timing config:load:file:C:\Users\Admin.npmrc Completed in 1ms
12 timing config:load:user Completed in 1ms
13 timing config:load:file:C:\Users\Admin\AppData\Roaming\npm\etc\npmrc Completed in 0ms
14 timing config:load:global Completed in 0ms
15 timing config:load:validate Completed in 0ms
16 timing config:load:credentials Completed in 1ms
17 timing config:load:setEnvs Completed in 1ms
18 timing config:load Completed in 11ms
19 timing npm:load:configload Completed in 12ms
20 timing npm:load:setTitle Completed in 0ms
21 timing npm:load:setupLog Completed in 1ms
22 timing config:load:flatten Completed in 3ms
23 timing npm:load:cleanupLog Completed in 2ms
24 timing npm:load:configScope Completed in 0ms
25 timing npm:load:projectScope Completed in 0ms
26 timing npm:load Completed in 20ms
27 timing command:run-script Completed in 2ms
28 verbose stack Error: ENOENT: no such file or directory, open 'C:\Users\Admin\Downloads\fire-stone-hosting-master (1)\package.json'
29 verbose cwd C:\Users\Admin\Downloads\fire-stone-hosting-master (1)
30 verbose Windows_NT 10.0.19044
31 verbose argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "run" "install-workspace"
32 verbose node v16.13.1
33 verbose npm  v8.1.2
34 error code ENOENT
35 error syscall open
36 error path C:\Users\Admin\Downloads\fire-stone-hosting-master (1)/package.json
37 error errno -4058
38 error enoent ENOENT: no such file or directory, open 'C:\Users\Admin\Downloads\fire-stone-hosting-master (1)\package.json'
39 error enoent This is related to npm not being able to find a file.
40 verbose exit -4058

@KendallDoesCoding Can you try with lastest commit? delete all node_modules in your project folder and then run npm run install-workspace in the root folder

I had no issues installing deps

I had no issues installing deps

Yeah, its just my laptop and pc ig.

@KendallDoesCoding Can you try with lastest commit? delete all node_modules in your project folder and then run npm run install-workspace in the root folder

Thanks, Will do this later in the day.

this sounds like an issue specific to your machine, closing

Yes. That's true...And you guys gave me the help I needed, just gotten
around to doing it yet.
On Sun, 19 Dec 2021 at 21:38, Cody Seibert @.***> wrote:

this sounds like an issue specific to your machine, closing
—
Reply to this email directly, view it on GitHub
https://github.com/codyseibert/fire-stone-hosting/issues/73#issuecomment-997417755,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUGJUTPS4QWV7A5YGID5CH3URX7QPANCNFSM5KEDXHDA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

@codyseibert It gave me the options in gitpod too, but it worked!!!


Crest /corporation/id/loyaltystore/ endpoint docs

exposed via /npccorps/ but kept separate to keep docs formatting consistent.

Thanks! Cherry-picked as 8c9732877784b4d2fe8a937e9227aba15edf2601.


Fix select_related

Coverage remained the same at 61.458% when pulling 69937ffad24010e2c4883caba8634401b1102f01 on fix/typo into 857351855d4961c80740723a73deb4688220a1af on master.


Implemented change request (click tracker) and styling

Great change to not hard code localhost in client files.
Nice implementation of the click tracker. Thanks a bunch for taking this on. A clean addition to the project. Same thing with the dark mode toggle. Handel'd elegantly. See what I did there?

Looks great! Thanks for the awesome contributions Colleen :grin:


Warcaster substituted cantrips used with sentinel do not trigger movement speed reduction.

Base Character Build:
Soul blade warlock, dip into fighter for bonus feats and polearm master fighting style, take sentinel feat and warcaster feat, use any type of polearm as a weapon.
Expected Behavior: Enemy gets within 2 of you, you get an opportunity attack from polearm master, and can substitute eldritch blast as a reaction. Enemy gets knocked back per connecting eldritch blast, and gets locked in place by sentinel since they were hit by an opportunity attack.
Actual Behavior: Enemy gets within 2 of you, you get an opportunity attack from polearm master and can substitute and eldritch blast as a reaction.  Enemy gets knocked back per connecting eldritch blast, then sentinel movement lock does not trigger.

won't fix.


Can't assert a DBI

Hi!
I am somewhat new to Lua but I need to run a few scripts from inside a luatex. I have a luadbi installed from a package for my distribution but does not seem to be able to run it even following test example:
[serge@veles latex]$ lua5.3
Lua 5.3.6  Copyright (C) 1994-2020 Lua.org, PUC-Rio
> DBI = require "DBI"
> assert.is_not_nil(DBI)
stdin:1: attempt to index a function value (global 'assert')
stack traceback:
        stdin:1: in main chunk
        [C]: in ?
>

I went to look at the code and tried the following:
> local name_to_module = {
    MySQL = 'dbd.mysql',
    PostgreSQL = 'dbd.postgresql',
    SQLite3 = 'dbd.sqlite3',
    DB2 = 'dbd.db2',
    Oracle = 'dbd.oracle',
    ODBC = 'dbd.odbc'
}
> name_to_module[MySQL]
stdin:1: attempt to index a nil value (global 'name_to_module')
stack traceback:
        stdin:1: in main chunk
        [C]: in ?
> name_to_module = {
    MySQL = 'dbd.mysql',
    PostgreSQL = 'dbd.postgresql',
    SQLite3 = 'dbd.sqlite3',
    DB2 = 'dbd.db2',
    Oracle = 'dbd.oracle',
    ODBC = 'dbd.odbc'
}
> name_to_module[MySQL]
nil
>

How come this does not work if it is taken from DBI.lua? Is there anything I am doing incorrectly?

Your first example, 'assert' is normally a built in Lua function. Try:

DBI = require "DBI"
assert(DBI)

The code you are see calling assert.is_not_nil is likely part of the test suite, which overrides assert with the luassert library to simplify test cases. If you are new to Lua and/or LuaDBI I recommend that you do not do that now.
Second example, you are missing either single or double quotes, try:

name_to_module['MySQL']

Also make sure your database driver modules are installed as well (dbd.mysql, etc. The exact package name varies with your distribution.)

Thanks, it seems to work now...


MacOS uses different selecors then windows

I assume linux uses different ones too could be to do with the user agent

google changed the css selector depending on what os is using meet, pre using "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/97.0.4691.0 Safari/537.36" keeps it the same from Windows to Mac (to linux, I havnt testeed yet)


Update what-s-new-in-tls-ssl-schannel-ssp-overview.md

Worked with EE Wayne McIntyre and found that this is the behavior and needs to be updated for the public.

Very well, looks OK to me so far. Would you like to consider adjusting the added sentence to be a colored Note blob, similar in appearance to this previous one?

[!NOTE]
The client-side implementation of RFC 5077 was added in Windows 8.

(Depending on how you feel about this, I may have a suggestion for a slight grammar adjustment to your added sentence, as well.)

@joshern-ms I can work with you to merge this update, but as a Microsoft employee, please use the private repo, https://github.com/MicrosoftDocs/windowsserverdocs-pr in the future. You will have more options from the writing team by going through there.
Do you want the text to appear as a note or simply as a section of text?

Hi Elizabeth,
Please elaborate on using the private repro. I went to the link and it says 404 error. I want to ensure I follow the correct process going forward. The text can appear as a note. See below. Thanks!
NOTE: If System Mapper is enabled by the client application and SendTrustedIssuers is configured, System Mapper will add CN=NT Authority to the issuers list.
Regards,
Jose Hernandez
T3 Support Engineer | Cloud & Infrastructure Solutions | Directory Services (On-Prem AD)
Customer Service & Support | Microsoft Corporation
Next OOF: May 11th -15th training, May 18th out of office
•+1<PHONE_NUMBER> | • jose.m.hernandez<EMAIL_ADDRESS>| •Mon-Fri 12pm-8pm CT
Manager: Steve Stoker | • robert.stoker<EMAIL_ADDRESS>| •+1<PHONE_NUMBER>
You may also call +1<PHONE_NUMBER> and ask for the next available engineer.
<EMAIL_ADDRESS>From: Elizabeth Ross<EMAIL_ADDRESS>Sent: Friday, May 15, 2020 4:20 PM
To: MicrosoftDocs/windowsserverdocs<EMAIL_ADDRESS>Cc: Jose Hernandez<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windowsserverdocs] Update what-s-new-in-tls-ssl-schannel-ssp-overview.md (#4313)
@joshern-mshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjoshern-ms&data=02|01|jose.m.hernandez%40microsoft.com|058c36c448194b8d2d3b08d7f915bfe7|72f988bf86f141af91ab2d7cd011db47|1|0|637251744134407745&sdata=krg5yZ1ZC4%2B2bsocNugBJSTuuqcaUKtoN652hZZhFUo%3D&reserved=0 I can work with you to merge this update, but as a Microsoft employee, please use the private repo, https://github.com/MicrosoftDocs/windowsserverdocs-prhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindowsserverdocs-pr&data=02|01|jose.m.hernandez%40microsoft.com|058c36c448194b8d2d3b08d7f915bfe7|72f988bf86f141af91ab2d7cd011db47|1|0|637251744134417745&sdata=rWH%2BQQ4OS8MW4Erll7g8b3qQlYkVejvVKDrg9%2BiP2kE%3D&reserved=0 in the future. You will have more options from the writing team by going through there.
Do you want the text to appear as a note or simply as a section of text?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindowsserverdocs%2Fpull%2F4313%23issuecomment-629492755&data=02|01|jose.m.hernandez%40microsoft.com|058c36c448194b8d2d3b08d7f915bfe7|72f988bf86f141af91ab2d7cd011db47|1|0|637251744134427738&sdata=RYcHHx4KIvYIjWa4veJHIzr55DfYhmr2RMedISP1srw%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAPQJ434GUL3G3AM4BW4BDFDRRWWYVANCNFSM4M4SHG2Q&data=02|01|jose.m.hernandez%40microsoft.com|058c36c448194b8d2d3b08d7f915bfe7|72f988bf86f141af91ab2d7cd011db47|1|0|637251744134427738&sdata=QKihRhu%2FOvOwEOXQg7pGu6HdLXTRpuoE1voyKeIXZo8%3D&reserved=0.

@joshern-ms  Yes, the 404 error is normal, I see it too :) It just means that you have to sign in to your GitHub account before you can access the repo. The repo needs to make sure you have the right credentials...after that, you can use the instructions here: https://github.com/MicrosoftDocs/windowsserverdocs-pr/blob/master/Contributor-guide/github-browser-updates.md (if you want to edit, but don't want to use a code editor) or here: https://github.com/MicrosoftDocs/windowsserverdocs-pr/blob/master/Contributor-guide/edit-existing-using-github.md (to edit using Visual Studio Code). Hope this helps! Thank you for all of your effort so far, it's greatly appreciated!

I will also go ahead and accept this pull request, so you won't have to re-do it.


Remove BenchmarkMetricRouter.js and related logic

There is no need to have BenchmarkMetricRouter.js. It causes complicated logic and unnecessary data in database. Most importantly, we need to maintain/update this file whenever we have a new perf test. (See PR #122, PR #189, PR #159). Therefore, we should remove BenchmarkMetricRouter.js and related logic.

Hi Lan,
I am currently doing the cleaning up removing the BenchmarkMetricRouter.js . file, and having a problem of deciding the keywords in those 5 bencharkNames:

The front 4 benchmarkNames all go to LibertyDTThroughput(needs to be renamed to be keyword 1)
The last 1 benchmarkName(DayTrader30) goes to DayTrader30(need to renamed to be keyword 2)

While in order to use the containing check keyword method in the benchmarkName, I think it must need the keywords to be unique and also could not contained one by one another.
I have checked with Piyush that the metrics under LibertyDTThroughput and DayTrader30 are different and could not be combined. Any good idea for deciding the keyword 1 and keyword 2 here?  @llxia @piyush286 ? Thanks in advance for the guidance!!
    "LibertyDayTrader3": {
        "17dev-4way-LargeThreadPool": "LibertyDTThroughput",
        "17dev-4way-LargeThreadPoolwarm": "LibertyDTThroughput"
    },
    "LibertyDayTrader7": {
        "17dev-4way-LargeThreadPool": "LibertyDTThroughput",
        "17dev-4way-LargeThreadPoolwarm": "LibertyDTThroughput"
    },
    "CleanedLibertyThroughput-DayTrader3":{
        "9dev-4way-LargeThreadPool": "LibertyDTThroughput",
        "9dev-4way-LargeThreadPoolwarm": "LibertyDTThroughput",
        "17dev-4way-LargeThreadPool": "LibertyDTThroughput",
        "17dev-4way-LargeThreadPoolwarm": "LibertyDTThroughput"
    },
    "CleanedLibertyThroughput-DayTrader7":{
        "9dev-4way-LargeThreadPool": "LibertyDTThroughput",
        "9dev-4way-LargeThreadPoolwarm": "LibertyDTThroughput",
        "17dev-4way-LargeThreadPool": "LibertyDTThroughput",
        "17dev-4way-LargeThreadPoolwarm": "LibertyDTThroughput"
    },

    "DayTrader30": {
        "WAS855gen1GB_4way": "DayTrader30",
        "WAS855gen1GB_4waywarm": "DayTrader30",
        "WAS9000gen1GB_4way": "DayTrader30",
        "WAS9000gen1GB_4waywarm": "DayTrader30"
    },

Except for LibertyDTThroughput and DayTrader30, do we have any other exceptional case?

Except for LibertyDTThroughput and DayTrader30, do we have any other exceptional case?
Those two are the only remaining cases now.

I feel that we can do something for LibertyDTThroughput and DayTrader30 parsing logic as some of the logic is similar. For example, outerRegex is the same and Throughput can use the same regex to parse. For others, we can use |, but it may need some testing to verify.
For simplicity, we can leverage the fact Object.keys() preserves the insertion order for strings. Move DayTrader30  logic before LibertyDTThroughput  (rename to DayTrader) logic in BenchmarkMetric.js, so we can match DayTrader30 first if it occurs.
In short,

keyword 1 =DayTrader and keyword 2 = DayTrader30
In BenchmarkMetric.js,  move DayTrader30 logic before DayTrader

For new perf test, the naming convention should be <benchmark>-<extra>. For example,
dacapo-eclipse
dacapo-avrora
dacapo-fop
https://github.com/AdoptOpenJDK/openjdk-tests/blob/master/perf/dacapo/playlist.xml

PR#210 is merged. Close this issue

@llxia Thanks for helping finalize the naming convention for perf tests today. I've added some new notes along with some quotes from everywhere to summarize things in one place. Feel free to update this note if something doesn't look right or we missed talking about.
Naming Convention for Perf Tests & Parser Key
Convention:
<BenchmarkFolderName>-<extra> (not <BenchmarkFolderName>_<extra>)
<BenchmarkFolderName>: Name of the benchmark folder, which contains all the benchmark files such as playlist.xml and build.xml. Example: (Folders such as dacapo, liberty found under openjdk-tests/perf folder).
<extra>: Variant or other info. This can also be used to make the name longer if you wan the parser key to be longer as well. If you 2 words, then separate them with _ and not - (example: baremetal_throughput).
Example:
pingperf-quarkus-baremetal_throughput
dacapo-eclipse
dacapo-avrora
Note: Not all open perf tests are following it. They need to be updated.
Related code:
https://github.com/AdoptOpenJDK/openjdk-test-tools/blob/f47cb6ef79330827ebb39d51c587c2615ee3a2fd/TestResultSummaryService/parsers/Utils.js#L4-L7
Preference for Naming Parser Key
https://github.com/AdoptOpenJDK/openjdk-test-tools/pull/207#discussion_r405663954

Having a short name will cause matching issues in the future as we add more and more data. Unless this needs to be shared with other benchmark tests, I would prefer to keep the full name.

Hence, we should give longer names on the parser side so that not all test case can match that. Test name will always be longer than the parser key name since we check whether the parser key is a substring of the test name (i.e. testName.includes(key)).
Order of Parser Key in BenchmarkMetric.js Matters:
https://github.com/AdoptOpenJDK/openjdk-test-tools/issues/190#issuecomment-594587515

we can leverage the fact Object.keys() preserves the insertion order for strings. Move DayTrader30 logic before LibertyDTThroughput (rename to DayTrader) logic in BenchmarkMetric.js, so we can match DayTrader30 first if it occurs.
In short,
• keyword 1 =DayTrader and keyword 2 = DayTrader30
• In BenchmarkMetric.js, move DayTrader30 logic before DayTrader

Old: Since we use<BenchmarkFolderName>-<extra> instead of <benchmark>-<extra>.
https://github.com/AdoptOpenJDK/openjdk-test-tools/issues/190#issuecomment-595300853
Related PRs:
Main Clean up: https://github.com/AdoptOpenJDK/openjdk-test-tools/pull/210
Quarkus Parsers: https://github.com/AdoptOpenJDK/openjdk-test-tools/pull/207


Create PRIVACY_POLICY.md to keep consistency across all projects

Is your feature request related to a problem? Please describe.
Register form on all AnitaB.Org Open source applications need to have common Privacy Policy. At the moment developers getting the content from https://anitab.org/privacy-policy/.
Describe the solution you'd like
If we have PRIVACY_POLICY.md in the /documentation repo, developers can use .md parser in their project so that any changes made to this policy will automatically apply to that project. This ensures the policy is up to date and consistent across all AnitaB.Org Open Source projects.
Describe alternatives you've considered
NA
Additional context
I believe the content of the Privacy Policy of the AnitaB above can be used across all projects as indicated in the screenshot below.

@isabelcosta , can you please see if this issue is valid? Thanks 😉

can i work on this issue?


Explore using ccache for CI pipeline

ccache could reduce the load and runtime of our pipeline a lot, since most files don't change in a pull request. How much can be gained needs to be tested, since use of templates makes ccache less efficient because there are fewer compilation units overall and changes are more likely.

time measurements with ccache on my workstation using 4 processes:

3:52 min with empty cache
3:22 with change io.h, included almost everywhere in our code, basically only the dependencies (boost, etc) are cached
2:50 with change to time_series.h, used on many places, some unittests and other cpp files are cached
0:52 with change to date.h, included in only a few files
0:51 with change to parameters.h, only used in one model

With higher parallelism, the use of the cache goes down. With 16 processes:

1:58 empty cache
1:45 with changing io.h
0:49 with changing parameters.h

Observations:

even when only a handful of files need to be compiled as in the parameters.h case, linking still takes a long time e.g. because linking of unittest executable needs to be done if only one translation unit changed. Maybe separating the unittests into smaller packets could reduce times here?
with ccache, parallel build is worth much less in terms of runtime. Overall resource consumption is still reduced.
very reliable gains come form caching dependencies like boost, which almost never change. but these gains could be achieved by using conan package manager as well (see #187)

Miscellaneous notes on ccache:

integrate into cmake build by setting variable CMAKE_CXX_COMPILER_LAUNCHER=ccache
cache size can be configured and should be big enough to hold a few versions at the same time so that, e.g., two pull requests and the nightly build that all build different versions don't interfere with each other's cache.
By default, the cache includes the working directory. this should be disabled, so that builds in different places in the repository (e.g. on in pycode and one in cpp) can use the same cache

@dabele Could you upload your configurations to a branch?


Enable setting ThreadId for Telegram notifications

See https://github.com/grafana/grafana/pull/79546.

Please add

tests here and here
documentation https://github.com/prometheus/alertmanager/blob/fe287e504eb16d0509293caead2c715034913007/docs/configuration.md#L1071-L1100

Please add

tests here and here
documentation here

Also, please update the description of this PR with more information. Link to another PR is not a really good description. Thanks!

Done 👍
The tests looked OK to me, but let me know if you think you need more cases.

Any updates?

@gotjosh @simonpasquier can you please review the PR?

This is a duplicate of an existing pull request, https://github.com/prometheus/alertmanager/pull/3638, which has been pending review since October 16, 2023, nearing four months. Several maintainers have been tagged for attention. It is disappointing to see this duplicate receiving more attention.

@yuri-tceretian This is a duplicate of an existing pull request, #3638, which has been pending review since October 16, 2023, nearing four months. Several maintainers have been tagged for attention. It is disappointing to see this duplicate receiving more attention.

If you mean https://github.com/prometheus/alertmanager/pull/3560, this is not duplicate, message_thread_id inclusion is different compared to reply_to_message_id, even though the idea is the same (sending message to a certain topic), message_thread is the intended way to sending a message to a certain topic, reply_to_message, as the fields name says, are more targeted to replying to a certain message ID rather than sending messages.

@yuri-tceretian This is a duplicate of an existing pull request, #3638, which has been pending review since October 16, 2023, nearing four months. Several maintainers have been tagged for attention. It is disappointing to see this duplicate receiving more attention.

If you mean #3560, this is not duplicate, message_thread_id inclusion is different compared to reply_to_message_id, even though the idea is the same (sending message to a certain topic), message_thread is the intended way to sending a message to a certain topic, reply_to_message, as the fields name says, are more targeted to replying to a certain message ID rather than sending messages.

Thank you, @raphielscape, for explaining the differences between PR #3560 and the current request. Your insight into how message_thread_id differs from reply_to_message_id is very helpful. Considering both PRs are focused on enhancing Telegram-related functionalities, it might be beneficial if the maintainers could review and possibly merge them both 🚀

I updated MessageThreadID as int32, and also updated my branch with the upstream one.

hi guys, just a quick question.
any plan for merging this PR and releasing the new feature?

Please rebase and sign. See https://github.com/prometheus/alertmanager/pull/3638/checks?check_run_id=21162278935

Please rebase and sign. See https://github.com/prometheus/alertmanager/pull/3638/checks?check_run_id=21162278935

I followed the instructions on the link you shared, please let me know if there is anything else I need to do 💐

@th0th I don't think you rebased correctly as there are 73 commits in this PR. You also still need to sign off the commits with --signoff. Did you use git rebase?

@th0th I don't think you rebased correctly as there are now 73 commits in this PR. Did you use git rebase? You also still need to sign off all of your commits with --signoff.

@th0th I don't think you rebased correctly as there are now 73 commits in this PR. Did you use git rebase? You also still need to sign off all of your commits with --signoff.

I followed the exact instructions on the link, but I think the commit history got messed up since I previously merged the upstream branch to my own branch. And now I don't know how to fix 😅

You can squash all the commits into a single commit using git rebase and then sign the commit using git commit --amend --no-edit --signoff.
Another option is to delete the branch, create a new branch with the same name, add the modifications, commit them, and then force push the branch to GitHub and it will replace the old branch.

I rebased, and it reduced the number of commits but there are still some commits that are shown in this PR, but not mine. No idea why.

Can you share the commands you ran to rebase?

Can you also run go mod tidy as otherwise it doesn't seem to compile:
../../../../../../pkg/mod/github.com/go-openapi/spec@v0.20.9/header.go:21:2: missing go.sum entry for module providing package github.com/go-openapi/jsonpointer (imported by github.com/go-openapi/analysis); to add:
	go get<EMAIL_ADDRESS>../../../../../../pkg/mod/github.com/go-openapi/spec@v0.20.9/ref.go:25:2: missing go.sum entry for module providing package github.com/go-openapi/jsonreference (imported by github.com/go-openapi/spec); to add:
	go get<EMAIL_ADDRESS>

I rebased, and it reduced the number of commits but there are still some commits that are shown in this PR, but not mine. No idea why.

Can you share the commands you ran to rebase?

I did this:
git remote add upstream https://github.com/prometheus/alertmanager.git
git rebase upstream/main

I rebased, and it reduced the number of commits but there are still some commits that are shown in this PR, but not mine. No idea why.

Can you share the commands you ran to rebase?

I did this:
git remote add upstream https://github.com/prometheus/alertmanager.git
git rebase upstream/main

Did you merge main before the rebase? If so, that might have caused the issue. I would just squash all 16 commits in this branch to a single commit using git rebase and then we can go from there. There is a nice tutorial on how to do that here.

come on :))) merge this

Hi @th0th! 👋 Do you need help with the rebase?

Hi @th0th! 👋 Do you need help with the rebase?

I do indeed 🙃

OK. I ran the following:
git fetch
git rebase origin/main // here origin is git@github.com:prometheus/alertmanager.git
git rebase -i HEAD~16

I then did the following:
pick 9f7f92cd Enable setting ThreadId for telegram notifications
s 7dceb6fa Fix the typo
s 0274e497 Update tests
s 5ec55ae7 Update documentation
s 15682c4a Update notifiers_test.go
s a75dd888 Update notifiers_test.go
s b6017dd0 Use int32 instead of int64 for MessageThreadID
s 3cecaae6 Bump github.com/go-openapi/swag from 0.22.4 to 0.22.7 (#3655)
d b29d9af6 Support UTF-8 label matchers: Add metrics to matchers compat package (#3658)
d c8eca8d9 Change compat metrics to counters (#3686)
s 3c958067 Bump github.com/go-openapi/errors from 0.20.4 to 0.21.0
s aeb3cdbd Bump github.com/go-openapi/swag from 0.22.7 to 0.22.9
s 37426b93 Bump eslint from 8.35.0 to 8.56.0 in /ui/react-app (#3692)
d 084e0d0c Do not register compat metrics in amtool (#3713)
d 70540688 Remove metrics from compat package (#3714)
s ef146bce Fix the typo

This removes the unrelated commits and squashes the others into a single commit. I also removed the commit messages of all other commits except the first commit. You can see here that this leaves just one commit ahead of main:
git log master..th0th/main --oneline
fa78ab4b (HEAD -> th0th/main) Enable setting ThreadId for telegram notifications
cb9724db (origin/main, origin/HEAD, main) notify/webhook: Fix crash on errors when url_file is used (#3798) (#3800)

pick 9f7f92cd Enable setting ThreadId for telegram notifications
s 7dceb6fa Fix the typo
s 0274e497 Update tests
s 5ec55ae7 Update documentation
s 15682c4a Update notifiers_test.go
s a75dd888 Update notifiers_test.go
s b6017dd0 Use int32 instead of int64 for MessageThreadID
s 3cecaae6 Bump github.com/go-openapi/swag from 0.22.4 to 0.22.7 (#3655)
d b29d9af6 Support UTF-8 label matchers: Add metrics to matchers compat package (#3658)
d c8eca8d9 Change compat metrics to counters (#3686)
s 3c958067 Bump github.com/go-openapi/errors from 0.20.4 to 0.21.0
s aeb3cdbd Bump github.com/go-openapi/swag from 0.22.7 to 0.22.9
s 37426b93 Bump eslint from 8.35.0 to 8.56.0 in /ui/react-app (#3692)
d 084e0d0c Do not register compat metrics in amtool (#3713)
d 70540688 Remove metrics from compat package (#3714)
s ef146bce Fix the typo

Thanks a lot @grobinson-grafana. But when I run git rebase -i HEAD~16, and put in the input, it says:
➜  alertmanager git:(main) git rebase -i HEAD~16

error: could not parse '9f7f92cd'
error: invalid line 1: pick 9f7f92cd Enable setting ThreadId for telegram notifications
error: could not parse '7dceb6fa'
error: invalid line 2: s 7dceb6fa Fix the typo
error: could not parse '0274e497'
error: invalid line 3: s 0274e497 Update tests
error: could not parse '5ec55ae7'
error: invalid line 4: s 5ec55ae7 Update documentation
error: could not parse '15682c4a'
error: invalid line 5: s 15682c4a Update notifiers_test.go
error: could not parse 'a75dd888'
error: invalid line 6: s a75dd888 Update notifiers_test.go
error: could not parse 'b6017dd0'
error: invalid line 7: s b6017dd0 Use int32 instead of int64 for MessageThreadID
error: could not parse '3cecaae6'
error: invalid line 8: s 3cecaae6 Bump github.com/go-openapi/swag from 0.22.4 to 0.22.7 (#3655)
error: could not parse 'b29d9af6'
error: invalid line 9: d b29d9af6 Support UTF-8 label matchers: Add metrics to matchers compat package (#3658)
error: could not parse 'c8eca8d9'
error: invalid line 10: d c8eca8d9 Change compat metrics to counters (#3686)
error: could not parse '3c958067'
error: invalid line 11: s 3c958067 Bump github.com/go-openapi/errors from 0.20.4 to 0.21.0
error: could not parse 'aeb3cdbd'
error: invalid line 12: s aeb3cdbd Bump github.com/go-openapi/swag from 0.22.7 to 0.22.9
error: could not parse '37426b93'
error: invalid line 13: s 37426b93 Bump eslint from 8.35.0 to 8.56.0 in /ui/react-app (#3692)
error: could not parse '084e0d0c'
error: invalid line 14: d 084e0d0c Do not register compat metrics in amtool (#3713)
error: could not parse '70540688'
error: invalid line 15: d 70540688 Remove metrics from compat package (#3714)
error: could not parse 'ef146bce'
error: invalid line 16: s ef146bce Fix the typo
You can fix this with 'git rebase --edit-todo' and then run 'git rebase --continue'.
Or you can abort the rebase with 'git rebase --abort'.

What am I doing wrong? :(

OK. I ran the following:
git fetch
git rebase origin/main // here origin is git@github.com:prometheus/alertmanager.git
git rebase -i HEAD~16

I then did the following:
pick 9f7f92cd Enable setting ThreadId for telegram notifications
s 7dceb6fa Fix the typo
s 0274e497 Update tests
s 5ec55ae7 Update documentation
s 15682c4a Update notifiers_test.go
s a75dd888 Update notifiers_test.go
s b6017dd0 Use int32 instead of int64 for MessageThreadID
s 3cecaae6 Bump github.com/go-openapi/swag from 0.22.4 to 0.22.7 (#3655)
d b29d9af6 Support UTF-8 label matchers: Add metrics to matchers compat package (#3658)
d c8eca8d9 Change compat metrics to counters (#3686)
s 3c958067 Bump github.com/go-openapi/errors from 0.20.4 to 0.21.0
s aeb3cdbd Bump github.com/go-openapi/swag from 0.22.7 to 0.22.9
s 37426b93 Bump eslint from 8.35.0 to 8.56.0 in /ui/react-app (#3692)
d 084e0d0c Do not register compat metrics in amtool (#3713)
d 70540688 Remove metrics from compat package (#3714)
s ef146bce Fix the typo

This removes the unrelated commits and squashes the others into a single commit. I also removed the commit messages of all other commits except the first commit. You can see here that this leaves just one commit ahead of main:
git log master..th0th/main --oneline
fa78ab4b (HEAD -> th0th/main) Enable setting ThreadId for telegram notifications
cb9724db (origin/main, origin/HEAD, main) notify/webhook: Fix crash on errors when url_file is used (#3798) (#3800)

Thanks a lot @grobinson-grafana. But when I run git rebase -i HEAD~16, and put in the input, it says:
➜  alertmanager git:(main) git rebase -i HEAD~16

error: could not parse '9f7f92cd'
error: invalid line 1: pick 9f7f92cd Enable setting ThreadId for telegram notifications
error: could not parse '7dceb6fa'
error: invalid line 2: s 7dceb6fa Fix the typo
error: could not parse '0274e497'
error: invalid line 3: s 0274e497 Update tests
error: could not parse '5ec55ae7'
error: invalid line 4: s 5ec55ae7 Update documentation
error: could not parse '15682c4a'
error: invalid line 5: s 15682c4a Update notifiers_test.go
error: could not parse 'a75dd888'
error: invalid line 6: s a75dd888 Update notifiers_test.go
error: could not parse 'b6017dd0'
error: invalid line 7: s b6017dd0 Use int32 instead of int64 for MessageThreadID
error: could not parse '3cecaae6'
error: invalid line 8: s 3cecaae6 Bump github.com/go-openapi/swag from 0.22.4 to 0.22.7 (#3655)
error: could not parse 'b29d9af6'
error: invalid line 9: d b29d9af6 Support UTF-8 label matchers: Add metrics to matchers compat package (#3658)
error: could not parse 'c8eca8d9'
error: invalid line 10: d c8eca8d9 Change compat metrics to counters (#3686)
error: could not parse '3c958067'
error: invalid line 11: s 3c958067 Bump github.com/go-openapi/errors from 0.20.4 to 0.21.0
error: could not parse 'aeb3cdbd'
error: invalid line 12: s aeb3cdbd Bump github.com/go-openapi/swag from 0.22.7 to 0.22.9
error: could not parse '37426b93'
error: invalid line 13: s 37426b93 Bump eslint from 8.35.0 to 8.56.0 in /ui/react-app (#3692)
error: could not parse '084e0d0c'
error: invalid line 14: d 084e0d0c Do not register compat metrics in amtool (#3713)
error: could not parse '70540688'
error: invalid line 15: d 70540688 Remove metrics from compat package (#3714)
error: could not parse 'ef146bce'
error: invalid line 16: s ef146bce Fix the typo
You can fix this with 'git rebase --edit-todo' and then run 'git rebase --continue'.
Or you can abort the rebase with 'git rebase --abort'.

What am I doing wrong? :(

What am I doing wrong? :(

You should be able to edit what is there when you run git rebase -i HEAD~16 – you don't need to copy/paste what I wrote. Basically you should change all pick to s (for squash) with the following exceptions:

9f7f92cd should be kept as pick because this is the first commit
b29d9af6 should be d (for drop)
c8eca8d9 should be d (for drop)
084e0d0c should be d (for drop)
70540688  should be d (for drop)

What am I doing wrong? :(

You should be able to edit what is there when you run git rebase -i HEAD~16 – you don't need to copy/paste what I wrote. Basically you should change all pick to s (for squash) with the following exceptions:

9f7f92cd should be kept as pick because this is the first commit
b29d9af6 should be d (for drop)
c8eca8d9 should be d (for drop)
084e0d0c should be d (for drop)
70540688  should be d (for drop)

I think I managed to rebase, can you please confirm @grobinson-grafana? 💐

I think I managed to rebase, can you please confirm @grobinson-grafana? 💐

Looks good to me! @gotjosh can you help with CircleCI?

Thanks Josh! @th0th there are some failing tests (please see CI / Test https://github.com/prometheus/alertmanager/actions/runs/8892559543/job/24416875284?pr=3638). You can also run make test before committing the fix to make sure there are no other failing tests.

Thank you very much for your contribution!

@th0th Thanks also for the contribution! I would appreciate it if you could test it and make sure it works as I cannot seem to create a Telegram supergroup or topics (perhaps I just don't know how to do it?).

OK, I managed to create a supergroup and then create topics.
receivers:
  - name: telegram
    telegram_configs:
      - bot_token: xxx
        chat_id: -1002096584164
        message_thread_id: 15
route:
  receiver: telegram
  group_wait: 15s
  group_interval: 30s

ts=2024-04-30T15:59:38.854Z caller=dispatch.go:164 level=debug component=dispatcher msg="Received alert" alert=[3fff2c2][active]
ts=2024-04-30T15:59:53.855Z caller=dispatch.go:516 level=debug component=dispatcher aggrGroup={}:{} msg=flushing alerts=[[3fff2c2][active]]
ts=2024-04-30T15:59:54.064Z caller=telegram.go:101 level=debug integration=telegram msg="Telegram message successfully published" message_id=18 chat_id=-1002096584164
ts=2024-04-30T15:59:54.064Z caller=notify.go:863 level=debug component=dispatcher receiver=telegram integration=telegram[0] aggrGroup={}:{} alerts=[[3fff2c2][active]] msg="Notify success" attempts=1 duration=208.804917ms
...
ts=2024-04-30T16:04:53.853Z caller=dispatch.go:516 level=debug component=dispatcher aggrGroup={}:{} msg=flushing alerts=[[3fff2c2][resolved]]
ts=2024-04-30T16:04:53.941Z caller=telegram.go:101 level=debug integration=telegram msg="Telegram message successfully published" message_id=20 chat_id=-1002096584164
ts=2024-04-30T16:04:53.942Z caller=notify.go:863 level=debug component=dispatcher receiver=telegram integration=telegram[0] aggrGroup={}:{} alerts=[[3fff2c2][resolved]] msg="Notify success" attempts=1 duration=88.678291ms

Hi! When are you going to make a new release with that MR?

Hi! When are you going to make a new release with that MR?

https://github.com/prometheus/alertmanager/discussions/3243#discussioncomment-10254485


Background Dataset for Training

Hello,
I'm new to rcnn, and trying to train a model to detect only one classification (+ background?).
I have prepared the dataset for my classification, but how should I handle the background class? Do I have to create a random set of background dataset? and why is it necessary? is it just acting as negative images?
Thank you,
Tamir.

You can prepare background windows by using UoI. if UoI less than 0.5 then you can consider them as background windows.
It is necessary because some of proposals will be discarded as negative windows during test phase.


Add WKT Support to Geo

https://www.elastic.co/guide/en/elasticsearch/reference/current/release-notes-6.2.0.html
Two issues to consider:

Add WKT Support to GeoBoundingBoxQueryBuilder - https://github.com/elastic/elasticsearch/pull/27692
Add Well Known Text (WKT) Parsing Support to ShapeBuilders  - https://github.com/elastic/elasticsearch/pull/27417

Closing this as #3377 has been merged into 6.3, to go into 6.x


Cancelling a request through browser or Postman doesn't cancel api asych call with CancellationToken

Hi,
I have an app with just  this middleware components:
app.UseDeveloperExceptionPage();
app.UseMvc();
My controller action is like this:
public async Task<IActionResult> GetAsync(int delay, CancellationToken token)
        {
            _logger.LogInformation($" <== {DateTime.Now.ToString("yyyy-MM-dd-HH:mm:ss:FFF")} GetAsync  start");
            await Task.Delay(delay, token);
            _logger.LogInformation($" <== {DateTime.Now.ToString("yyyy-MM-dd-HH:mm:ss:FFF")} GetAsync  stop");
            return Ok("Request finish succefully");
        }

the request is with a delay of 5 seconds
and in the middle I hit cancel request from Postam or the browser.
this is the log:
WebApplicationCore.Controllers.ValuesController:Information:  <== 2018-10-04-22:00:04:615 GetAsync  start
WebApplicationCore.Controllers.ValuesController:Information:  <==2018-10-04-22:00:09:628 GetAsync  stop
shouldn't cancel the api call?
thanks

What server are you using? IIS & Kestrel, Kestrel standalone, HttpSys? Where are you running it? Locally in Visual Studio, remotely in Azure, etc.?
Does it work if you specify a longer timeout (10s, 60s)? TCP disconnects are not as immediate as you might think.

The MVC model binder binds to RequestAborted https://github.com/aspnet/Mvc/blob/release/2.2/src/Microsoft.AspNetCore.Mvc.Core/ModelBinding/Binders/CancellationTokenModelBinder.cs#L28
Is this this fired on disconnect?

That's the idea.

Just confirming, I confirm this every time cancellation comes up.

Hi @Tratcher , sorry for not including those details in my post initially.
I am running it locally in Visualstudio.
Indeed, for an async call with 120s delay, if we cancel the request in postman the request is cancelled before finishing the delay of 120s. Missed that TCP disconnects are not immediately.
Thank you!


Errors while dragging layer, with leaflet Canvas renderer

I've encountered some strange behavior while dragging the layer. That error occurs only when I use canvas renderer on the leaflet map.
But it works fine with preferCanvas: true property though.
Here is the link: https://jsfiddle.net/etLsox8j/1/

@AndriyBroda can you please check if the PR #1076 fix your issue. You can get the src from here

@Falke-Design perfect, that fix works, nicely done 🙌


Update Images in README file

The Webpage images available in the README file are outdated. Update the file as per the current state of the project.

@arpitghura i will complete it ps assign it  to me...

Hello @Yash-Var Welcome to the community. I am assigning you the issue.

@arpitghura i have seen the readme and find that the image are already updated...

@arpitghura i have seen the readme and find that the image are already updated...

They are not. The Homepage Image is updated but rest of all are not updated.

okay @arpitghura i will completed .....

Hey @Yash-Var Are you working on the issue?

@arpitghura yes sorry for tgr delay i will completed it today..

hey @arpitghura if the work is still in progress, then I would gladly work on this.

@arpitghura are you still working on it ?

@arpitghura please assign it to me. I'll update it

Hey, @Preeti8021 assigning this issue to you. Go ahead.
Meanwhile @anushkaaa34 you can look at some other issues.
Thank you

Thank you!!


How Do You Compile This Makefile in Unix

I have been trying to install this in my server but it doesn't work.
I have tried to Make the file by typing Make. the Makefile runs and it gives me an error.
Is this the right way to build this project in my server?
Thank you
V.

Hi Jan-Willem,
I guess I don’t know how to compile this source code.
I went to my UNIX server and at the root of the directory when I uploaded the contents of the zip file and typed; Make.
Then I got an error. But I’m not familiarized with compiling and making Python code.
Any help is appreciated,
Victor
From: Jan-Willem Gmelig Meyling<EMAIL_ADDRESS>Sent: Friday, December 26, 2014 6:45 PM
To: driftyco/graphite
Cc: vanleurth
Subject: Re: [graphite] How Do You Compile This Makefile in Unix (#13)
Whats the error you get? You should have python installed and added to your path I think..
Kind regards,
Jan-Willem
Op 27 dec. 2014 om 02:19 heeft vanleurth<EMAIL_ADDRESS>mailto:notifications<EMAIL_ADDRESS><EMAIL_ADDRESS>het volgende geschreven:
I have been trying to install this in my server but it doesn't work.
I have tried to Make the file by typing Make. the Makefile runs and it gives me an error.
Is this the right way to build this project in my server?
Thank you
V.
—
Reply to this email directly or view it on GitHubhttps://github.com/driftyco/graphite/issues/13.
—
Reply to this email directly or view it on GitHub https://github.com/driftyco/graphite/issues/13#issuecomment-68165603 .  https://github.com/notifications/beacon/AAKwtm4ZIuSqLzRtNdirEFhvEktrQyD0ks5nbgaVgaJpZM4DMd7i.gif


RequestOptions Add Header method

The Encryption SDK, internal compute services, and even customers may want to add customer headers so they can correlate their request to Cosmos operations. The RequestOptions.Properties is meant for the client context and is not meant for adding headers.
The suggested public API
public class RequestOptions
{
     public Func<Header> AddHeaders { get; set; }
}

ItemRequestOptions other = (ItemRequestOptions )this.MemberwiseClone();
Func originalHeaderFunc = other.AddHeaders;
other.AddHeaders = (header) =>
{
originalHeader(header);
AddEncryptionHeader(header);
};


Is it possible to use cargo-check?

https://github.com/rsolomo/cargo-check
Cargo check is a wrapper of cargo rustc -- -Zno-trans

It seems like we could just change our cargo call as appropriate rather than using cargo-check. However, testing it locally, I am having some issues building a module that builds fine with cargo build.
I probably won't have time in the near future to dig into this further, but a working PR would likely get merged.

cargo rustc requires to provide crate name or bin name via --lib or --bin, so this plugin may have to change a little bit. I tested it locally, it seems that all works well in my project.

+1

+1

As of several months ago, this linter uses -Zno-trans. As far as I know, using cargo-check would not provide any additional benefit.

I should have specified I meant +1 to this comment:

cargo rustc requires to provide crate name or bin name via --lib or --bin, so this plugin may have to change a little bit. I tested it locally, it seems that all works well in my project.

I had to fork the plugin and manually add --lib to verify it still works as intended for a Rust library, just get it to work again and it seems to behave exactly as expected.
There should be a check that adds --lib or -bin NAME to the cargo rustc call. Maybe simply allowing the args setting to be used in that call is enough, users can specify the flag on their own.

Ah, that makes sense. Adding the ability to pass arguments through sounds good.

This now supports cargo check with Rust 1.5. cargo-check must be installed and use-cargo-check must be enabled.


Updates to supported cars

Only Touring has Honda sensing on 15 and 16 CR-V
2018 Hatch Supported
Odyssey steers down to 0, not 12

Thanks, fixed internally. Will be in next release.


kmscon: unstable-2018-09-07 -> 9.0.0

Description of changes
This updates kmscon to the latest release tag on the repository, after 3 years(!) without an update in Nixpkgs.
NOTE: This PR depends on  #171069
Notifying maintainer; @omasanori

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

As noted in #178748, updating meson appears to have created a small variety of build errors due to the fortify hardening. It seems that meson is removing the -O2 flag from the hardening, causing the compiler to reject it due to the fortify hardening requiring the optimization flag.

Was made aware that someone else made a meson PR, so retargetting this on top of that.

Rebased on top of the meson changes and edited the PR message accordingly.
I didn't see anything in the Nixpkgs manual about the case of depending on another PR being merged, so my (current) assumption is that when the meson change is merged, the commits this PR has will be seen as already included and thus won't have duplicates, but I could very much be wrong.
Please let me know if I should just rebase out the commits this depends on.

Rebased out the dependent commits.

Pulled in a patch from after the most recent release, and added a compiler flag which disables a specific warning, as those warnings (treated as errors) prevented compilation. Also as noted in the previous review, removed cmake as an extraneous dependency. See the recently mentioned issue for the issue behind the compiler flag.

Incidentally included the auto-formatting my editor did for lines 1/2, 10/11, and 33/34, which I'm going to leave in since I personally prefer the expanded form. If I should revert that shift, please let me know.

Incidentally included the auto-formatting my editor did for lines 1/2, 10/11, and 33/34, which I'm going to leave in since I personally prefer the expanded form. If I should revert that shift, please let me know.

It is acceptable to me.
By the way, what kind of tests have you done? Just completed a build or running the kmscon service with this? While I will do some tests this weekend anyways, I'd know what is succeeded on your machine.

For those who want to more easily test the changes using flakes, I've made a branch on my nixpkgs fork that combines the meson and kmscon changes: https://github.com/Infinidoge/nixpkgs/tree/combined/kmscon
(github:Infinidoge/nixpkgs/combined/kmscon)

Seems to be running just fine on my machine, which is good:

Since the relative feature changes were small, it's a bit hard to test them specifically, however I am setting up a compose key to try using that under kmscon.

I had tested underline, italic and multilingual text with emacs-nox, noto-fonts and noto-fonts-cjk-sans and confirmed that it works. Thanks @Infinidoge !

Since the dependent PR has been merged, is there anything that needs to be done before this can be merged?

@ofborg eval

Also, it seems that the meson changes have made their way into master, should I go ahead and retarget to there, since this change itself doesn't cause a mass rebuild?

Oh wait it already is targetting master, nevermind


[SMALLFIX] Simplified equals implementation of MostAvailableFirstPolicy

Simplified the equals implementation of the MostAvailableFirstPolicy class.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9645/
Test PASSed.

LGTM


add tasks and plans directories

Hey folks,
Whenever any of my customers/prospects beginning using PE and the control repo, adding tasks and plans is always a bit of a mystery, especially when there's no template directories or files like there are for roles and profiles.
Given the maturity of tasks and plans at this point, we should really include something in the template control repo.
The adhoc directory within site-modules contains a tasks and plans directory.
In the tasks directory I've included a skeleton for a nix and windows based task with some basic task metadata (environment + task description). I've also included a plan with a description and targets param.

Awesome! Thanks for the suggestions, I've accepted them now - ready for second review.

fwiw, bolt has bolt plan new proj_name::plan_name with the optional -pp flag.  You get a YAML plan without the -pp flag and you get a puppet language plan with the -pp flag.  I'd argue the plan examples should just be the same one from those commands.
The long term goal is to be able to run projects directly in PE and forgo the need to put them in a control-repo or module but this seems fine in the interim.


Compatiblity with API v6

Hello! Just wanted to update this plugin. I've added simple compatibility with Dalamaud API level 6.

Updated DalamudPackager
Added Punchline in plugin definition
Removed all context menu codes (hook, settings front-end) to prevent proper crashes

Now I try it with this pull request here. Hope you'll find the time to update the plugin. Otherwise if you want and - of course - you agree @r00telement, I also can publish the update, that wouldn't be a problem. Just need to know in this case. ^^'
Fix #9

Hello! Just wanted to update this plugin. I've added simple compatibility with Dalamaud API level 6.

Updated DalamudPackager
Added Punchline in plugin definition

Now I try it with this pull request here. Hope you'll find the time to update the plugin. Otherwise if you want and - of course - you agree, I also can publish the update, that wouldn't be a problem. Just need to know in this case. ^^'
Fix #9

I would like it, if you could do it.

I would like it, if you could do it.

Yes, I really would like to, but I have to wait for the OK of the maintainer... Without I'm not allowed probably. The rules for plugins mentoined in pinned messages in the Goat Place Discord server tellst to wait until the maintainer has the time to update their plugins theirself. Just don't public own builds.
Well, I could try, but I don't want to be the guy that don't respect rules.

@Pilzinsel64 My apologies for the delay! I'm unfortunately not able to continue maintaining this plugin these days, so to that end, I give you permission to publish updates for PlayerTags to Dalamud from your own fork if you wish to do so! Cheers.

Btw, around the time I quit FF14, Dalamud implemented its own context menu stuff, so you should be able to delete those files in PlayerTags and just use the Dalamud impl.

Oh, thank you very much for your response @r00telement! And also thank you for your openness and understanding. And don't worry for the delay. You wasn't able to have an idea about that. ❤
So, that means, the next days I will push the current state to the the offical repo for testing, just to have a working version public. And after that I implement the new context menu.
Btw, I didn't knew that the new context menu is implemented. :D I saw the pull request for the context menu, but I thought it was closed and not merged. Wil check that. ^^


The new 1.0.08 version generates 409 errors.

telegram.0 | 2017-10-05 21:44:34.342 | error | polling_error:ETELEGRAM, Error: ETELEGRAM: 409 Conflict: terminated by other long poll or webhook
-- | -- | -- | --

Every second a line like above!

Please test https://github.com/Haba1234/ioBroker.telegram
I'm trying to fix this error

Ok, I reloaded the adapter from git and the problems do not show up anymore!

super 👍


new cards 💳

Added:

diners club
maestro
visa electron

Closing this for now until there is demand for support of these types of cards.


Rox 25519: Backport changes to preserve and periodically prune orphaned node CVEs

Description

Backported changes in #12283 , #12372 and #12459
In 4.6, we generate _vtproto.pb.go files for protos but that is not generated in 4.5. So manually changed the calls into  _vtproto.pb.go files to use matching functions in _pb.go files present in 4.5.
In 4.6, we autogenerate PruneMany function in the store implementations but 4.5 does not do that. So, manually implemented the PruneNodeCVEs function to use a similar logic to PruneMany.

User-facing documentation

[X] CHANGELOG update is not needed
[X] documentation is not needed

Testing and quality

[ ] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[ ] CI results are inspected

Automated testing

[X] added unit tests
[X] modified existing tests

How I validated my change

Unit tests and by repeating manual tests from the attached 4.6 PRs.

/retest


Submit Competition Code: CycleGAN Related

In this PR we are submitting our code implementation for the competition.
In general, we implement an improved CycleGAN with TinyMS & MindSpore. Specifically, we guide the image translation task between different domains by constructing new regularizier.
Please notice the code implementation we submited here does not represent the final version. In the future we will make more improvements based on it.

 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.

@bravotty Thank you for your submission! Please sign our Contributor License Agreement before we can accept your contribution.

@bravotty Thank you for your submission! Please sign our Contributor License Agreement before we can accept your contribution.

Thank you for reminding me. I have signed the CLA.

Maybe you need to be familiar with the tinyms directory architecture, put your core code into the tinyms/tinyms module directory, and put the main.py and other executable code into the tinyms/tests/st directory, create a new directory named competition under this directory, then place the city_cyclegan.py execution code, which can avoid repetitive code and help other developers use it quickly.


fix false positive under valgrind

valgrind gets confused by a non-aligned pointer load; use an aligned load
so it's quietened.
Signed-off-by: John Levon<EMAIL_ADDRESS>

Fixes #347

This was not a false positive - see #363


Add containerId to the README example of ScrollTo Element function

Added containerId prop to the scrollTo example for clearer documentation (wasted some time on it myself before I read the source code and realized what I should do :( )
I know this is part of the Link props and should be obvious, but apparently it isn't that obvious...

Good catch, thanks for the PR! :)


Object loaded from URDF does not move

I am trying to create a new task where the robot has to open the bottle cap.
I loaded the bottle data from a URDF, obtained from the PartNet-Mobility Dataset:
loader = self._scene.create_urdf_loader()
loader.fix_root_link = True
loader.load_multiple_collisions_from_file = True
loader.scale = 0.08
self.bottle = loader.load(str('dataset/3596/mobility.urdf'))
self.bottle.set_pose(sapien.Pose(p=[-0.064, 0.052, 0.02], q=[1, 0, 0, 0]))

However, the bottle/cap is unmovable (e.g., when the panda's arm touches the bottle, the bottle does not move; when the panda grasps the cap and lifts it in the air, the cap does not detach/move).
How can I make the bottle/cap movable (non-static)?
I've tried setting the loader's fix_root_link to False, but the bottle would start flying when I moved the robot arm without even touching the bottle.

I was able to make the bottle cap movable by adapting mani_skill2/envs/misc/turn_faucet.py from ManiSkill2.


masterdata manage translations

Can we add a short readme how you initialize and configure the translations?

Can we add a short readme how you initialize and configure the translations?
I think for now it should be used/tested on masterdata. It needs some fine tuning to be more developer friendly for using in other apps.


Repository page ScrollViewer position

Describe the bug
As the title
To Reproduce
Steps to reproduce the behavior:

Go to repository page
Scroll down

Expected behavior
I don't know where to make it scrollable.
Screenshots (optional)
Version

Windows: Win11 22523.rs_prerelease
FluentHub: unversioning

Additional context

first suggestion

second:

third(out dated)

I have now inserted a script that disables vertical scrolling in WebView.

Ok, the idea is getting that down while scrolling? That's actually my idea

wherea would you like?

second idea is good?

I thought in the first one, but if u like the second one, we can use it

YES. go with first one


_post/0000-01-02-YOUR-CNTB0C064.md

layout: slide
title: “Welcome to our second slide!”
Your test
Use the left arrow to go back!

created a branch, created a file and made a commit, and opened a pull request


Blog images don't scale down on mobile

Blog images remain full size on mobile on both blog index and blog post pages. That seems sub-optimal.

Fixed by using an inline style to set the max-width to 100% and the max-height to 400px. This will get pushed out at some point with other fixes.

Got pushed with https://github.com/xbmc/kodi-tv/commit/e7f1afac121b5bd88603bf749c5bdac4ccf846b4.


Remove flaky checks in test_delete_force

We can't reliably tell if timeline should be active or not, and activity logic can be changed in the future. This PR disables checks for was_active in test_delete_force, to fix flaky test fails.

This resolves #2512


"F3" vs "CTL-G" in Windows builds

QT overrides the default hotkey for "Find Next" on windows to "F3" but that is a default debugger hotkey in Binary Ninja. We need to either resolve this with a different default setting or changing the debugger hotkeys on windows, either way.

I can't reproduce this now, closing until I can get a more reliable repro.


Mocker server does not support modifying handlers

In my tests:

Stub server supports altering handlers.
Mock server does not support altering handlers.

Same handlers, when used as initial handlers, work just fine with mock server. Trait and methods exist but they seem to have no effect at runtime.

fixed


I am looking forward to using it with OBS.

I couldn't get my heart rate so I'll share.
It's a great app and I really appreciate it.
Watch model…Versa2
Firmware version number…<IP_ADDRESS>
The type of phone … Android 12
Fitbit app version number…3.72.fitbit-mobile-37268001-489675908

Did you follow the troubleshooting steps? Do you get an error anywhere? I need more info to help you debug.

I followed the troubleshooting steps. Heart rate data cannot be acquired and remains 0.

Darn. I was hoping you hadn't followed the troubleshooting steps yet. If you've already gone through those, there's likely a configuration issue with either your firewall or your router and I'm not knowledgeable enough to help with either of those...


Update 06_Generics.md

Fixed minor typo at the bottom of the page. to compiler -> the compiler

Thanks! :)


Docs - grammatical mistake

File: themes/default/content/registry/packages/aws/api-docs/organizations/account/_index.md
Section: "Create a Account Resource"
Should read "Create an Account Resource"

@cnunciato what repo is the code in for this particular docs gen?

It's actually in pulumi/pulumi: https://github.com/pulumi/pulumi/blob/master/pkg/codegen/docs/templates/resource.tmpl#L16


ARROW-4141: [Ruby] Add support for creating schema from raw Ruby objects

The followings should be implemented by follow-up works:

Arrow::TimestampDataType.new(unit: ...)
Arrow::Time32DataType.new(unit: ...)
Arrow::Time64DataType.new(unit: ...)
Arrow::DecimalDataType.new(precision: ..., scale: ...)
Arrow::SparseUnionDataType.new(fields: ..., type_codes: ...)
Arrow::DenseUnionDataType.new(fields: ..., type_codes: ...)
Arrow::DictionaryDataType.new(fields: ..., type_codes: ...)

Codecov Report

Merging #3293 into master will decrease coverage by 20.03%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #3293       +/-   ##
===========================================
- Coverage    88.5%   68.47%   -20.04%
===========================================
  Files         539      366      -173
  Lines       72968    48609    -24359
===========================================
- Hits        64583    33285    -31298
- Misses       8278    15324     +7046
+ Partials      107        0      -107

Impacted Files
Coverage Δ

cpp/src/arrow/util/memory.h
0% <0%> (-100%)
:arrow_down:

cpp/src/parquet/hasher.h
0% <0%> (-100%)
:arrow_down:

cpp/src/gandiva/to_date_holder.h
0% <0%> (-100%)
:arrow_down:

cpp/src/arrow/io/hdfs-internal.h
0% <0%> (-100%)
:arrow_down:

cpp/src/arrow/util/sse-util.h
0% <0%> (-100%)
:arrow_down:

cpp/src/arrow/pretty_print.h
0% <0%> (-100%)
:arrow_down:

cpp/src/parquet/murmur3.h
0% <0%> (-100%)
:arrow_down:

cpp/src/arrow/util/int-util.cc
0.39% <0%> (-99.21%)
:arrow_down:

cpp/src/arrow/ipc/json-simple.cc
0.41% <0%> (-98.76%)
:arrow_down:

cpp/src/arrow/test-util.h
0% <0%> (-98.1%)
:arrow_down:

... and 330 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9376d85...d251ba9. Read the comment docs.


Error message (1)

In a previous version of the JsonPreprocessor, this code
"par${...}am2" : "123"
caused:
'Invalid parameter format in line: "par${...}am2" : "123"'
In current version this changed to:
''NoneType' object is not subscriptable'
The first error message is the better one.
Addendum:
Finally this has to be a naming convention issue.

Solved by https://github.com/test-fullautomation/python-jsonpreprocessor/issues/270
Issue can be closed.


[gRPC] service with package doesn't work with service version changes

Description:
gRPC Service with package doesn't work with service version changes. With service version changes, service package derived like,
<package_name>:<package_version>.
In gRPC service, service methods should be defined like below,
<package_name>.<service_name>/<method_name>
and registered in the system.
So with the version changes, service methods are derived like,
<package_name>:<package_version>.<service_name>/<method_name>
This is not correct format.
We need to remove package version when registering service method.
Steps to reproduce:
Register service with package. We cannot invoke service method with the client.
Affected Versions:
0.975.0

Closing this issue. since this is fixed with #9221


fix: always set DOCKER_CONFIG unless manually set and no base64 config provided

Fixes #392

CI seems to have issue with privilege and remounting, not sure why it applies to my test and not others. Looking into it.
Also realized we'll have to copy over config.json or change DOCKER_CONFIG to the remounted path if applicable.


Send 422 instead of 400 on cwl or wdl parse errors

Addresses https://ucsc-cgl.atlassian.net/browse/DOCK-1322

NextflowHandler.java doesn't throw any exception on a parsing error unlike WDLHandler.java and CWLHandler.java; this seems odd. Also not sure that Dockstore does Galaxy descriptor parsing...

@denis-yuen NextflowHandler.java and LanguagePluginHandler.java don't throw any exceptions on a parsing error unlike WDLHandler.java and CWLHandler.java; this seems odd.

Galaxy parsing is done in a separate repo


Swap alt fee tokens into the native token prior to fee burn

As described in #4320, the initial implementation of multi-asset fee support burns the alt fee tokens. This is incorrect, since the chain should only be minting/burning native assets. The correct behavior is that the fees paid in alt fee tokens should be converted to the native token.  However, since the DEX is batched, we need some plumbing to collect fee amounts, include them in batch swaps, and then handle the resulting fee payments. This overlaps with work we'd thought about (but haven't done, and deferred) as part of #3666 and we should think about them at the same time.

However, since the DEX is batched, we need some plumbing to collect fee amounts, include them in batch swaps, and then handle the resulting fee payments.

Should we add an additional fee multiplier to fees paid in the non-native asset to account for the additional work of swapping?

I don't think so, since that cost isn't really attributable to a psecific transaction.
I'm also not sure it's necessary. In this V0 design, there's no connection between the different prices, all of which are fixed -- so the values of the different fee tokens could float around and cause pricing disparities. To address this, I think that the fixed fees for alternative gas tokens should be significantly higher (10x). This way, users can come into the chain and use it, but they're incentivized to use the native fee token which will protect their privacy.

The issue here is primarily that we don't currently have any plumbing set up to move the values of the fees around and then swap them, and doing that will require some design work. Maybe we don't get to doing this pre-mainnet.

Sounds like we need to discuss & clarify which part of this ticket is in scope for V1 - @hdevalence when you say "maybe we don't get to doing this pre-mainnet", do you mean this ticket generally? Will burning the alt fee tokens cause issues for users?

what does it mean if this ticket is tagged as `v1  and removed from the current sprint, specifically in the context of the upcoming code freeze?


Chinese character GBK code does not display

Which version of PhantomJS are you using? Tip: run phantomjs --version.
2.1.1

What steps will reproduce the problem?

phantomjs png.js http://kx.cq.gov.cn/zfxx/gsgg/2017-9/37423.html tu.png

Which operating system are you using?
windows7

Did you use binary PhantomJS or did you compile it from source?

Please provide any additional information below.

png.js：
"use strict"
var page = require('webpage').create();
var args = require('system').args;
var url =  args[1];
var filename = args[2];
page.open(url, function (status) {
if (status !== 'success') {
phantom.outputEncoding="gbk";
console.log('Unable to load ' + url + ' !');
phantom.exit();
} else {
window.setTimeout(function () {
phantom.outputEncoding="gbk";
page.render(filename);
phantom.outputEncoding="gbk";
console.log('finish:', filename);
phantom.exit();
}, 1000);
}
});

Thanks for the report! Unfortunately, the test URL doesn't work anywhere, hence it is impossible to troubleshoot the issue.


aespipe: 2.4e -> 2.4f

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/aespipe/versions.
meta.description for aespipe is: '"AES encrypting or decrypting pipe"'.
meta.homepage for aespipe is: '"http://loop-aes.sourceforge.net/aespipe.README"

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 2.4f with grep in /nix/store/fnc1fwvl01d0p5fy9cyl6jm2i0sh382w-aespipe-2.4f

directory tree listing: https://gist.github.com/6d67b7b2fec4d12bf14367971ad3e769

du listing: https://gist.github.com/6213acca3f7d9981af783b88a7efadf0

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
aespipe

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fnc1fwvl01d0p5fy9cyl6jm2i0sh382w-aespipe-2.4f \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A aespipe https://github.com/r-ryantm/nixpkgs/archive/89832d2343a6d644572d62f267acd67541b161ef.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fnc1fwvl01d0p5fy9cyl6jm2i0sh382w-aespipe-2.4f
ls -la /nix/store/fnc1fwvl01d0p5fy9cyl6jm2i0sh382w-aespipe-2.4f/bin

cc @cillianderoiste for testing.

Changelog from archive:

aespipe-v2.4f     October 2 2019
- Fixed configure script mis-detection of "labels need underline
prefix" on Fedora. Fix from Jiri Hladky.
- Added configure script autodetection for x86/amd64 -fno-pie compile
and -no-pie linker flags. These are needed when compiler/linker
produces position independent executable (PIE) by default.


Adding menu bar

Codecov Report

Merging #103 (d721286) into master (a5f89fe) will increase coverage by 0.16%.
The diff coverage is 98.90%.

@@            Coverage Diff             @@
##           master     #103      +/-   ##
==========================================
+ Coverage   79.92%   80.08%   +0.16%
==========================================
  Files         116      117       +1
  Lines       10827    10916      +89
  Branches     1131     1147      +16
==========================================
+ Hits         8653     8742      +89
  Misses       2174     2174

Flag
Coverage Δ

unittests
80.08% <98.90%> (+0.16%)
:arrow_up:

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

Impacted Files
Coverage Δ

packages/lib/components/index.ts
12.24% <0.00%> (+1.82%)
:arrow_up:

packages/lib/components/menu-bar/menu-bar.tsx
100.00% <100.00%> (ø)

...ackages/lib/components/menu-button/menu-button.tsx
100.00% <100.00%> (ø)

packages/lib/components/menu/menu-overlay.tsx
98.09% <100.00%> (+0.03%)
:arrow_up:

packages/lib/components/menu/menu.tsx
96.77% <100.00%> (+0.03%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a5f89fe...d721286. Read the comment docs.


kubectl get events should highlight errors/warnings in a different color

Feature request: kubectl get events should highlight errors/warnings in a different color

Let's discuss this in https://github.com/hidetatz/kubecolor/issues/117


Be sure to detach one-shot timer watcher

See https://github.com/fluent/fluentd/pull/1059/files#r75269945

I've pushed change for comment @tagomoris

LGTM.

Thanks!


How do I perform Batch Inference for TSR?

I have a pdf and have converted it to the images. Now I want to pass it through the TETR for Table Structure Recognition.
Is there a way to run Inference with batch size > 1 for TSR?
Can it be done using the inference.py or I need to run in the eval mode?

Hi,
Batched inference is supported by default in the Transformers library as shown here. Just pass multiple images to the image processor.


Discuss: Apply insetStyle to outer view

I think insetStyle should be applied to outer view in ScreenWithoutScrolling same as in ScreenWithScrolling.
Then styles defined in inner preset like paddingBottom specific for app design won't overrided by insetStyle. Because for example in most cases we want our design paddingBottom: 20 + safe paddingBottom: 34 = 54  for iPhone 12 and 20 + 0 for iPhone SE and so on.

But after this, it's harder to style when we need some content under StatusBar for example, because safe paddingTop applied to the outer view and shift our inner content view

Now I think it's a possibly wrong idea. And also find out that when behavior={isIos ? "padding" : undefined} outer paddingBotom not applied, for behavior="height" - paddingBottom applied.

@pronevich After a month, do you still think this is the wrong approach? I could get others to weigh in if you think it's worthwhile.


NetworkError: 401 Unauthorized on basic auth protected sites

Hi, I like your addon, but I have some issue with it on a basic auth protected site.
When I try to connect to this site, firefox asks me for the credentials, which I enter correctly.
The site loads, but without all the font resources, which are also protected by this login. The .js and .css resources are working.
I can see lots of NetworkError: 401 Unauthorized - %URL_OF_RESOURCE% messages in firebug for resources like .woff, .eot, and .ttf.
When I try to access the font resources directly (after I logged in), it works too.
A quick search (http://davidwalsh.name/cdn-fonts) revealed that the Access-Control-Allow-Origin header maybe need to be injected into the response header for the fonts too, which is currently not happening.
Is this 401 error a problem within your addon, or firefox?

The injection of the CORS headers into the response is post-processing, it only makes Firefox believe that the server has sent the necessary headers to bypass the security restrictions. It should not affect the http transaction itself.
When the request is rejected because of the same origin policy, the web console gives a security type message such as follows:
"Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at %URL%. This can be fixed by moving the resource to the same domain or enabling CORS."
The errors 401 suggest the server does not see you as an authenticated user when you try to access the fonts, which doesn't seem related to CORS.
Anyway, questions:

Are the fonts files on the same server and dir/subdir as the html/css/js files, protected by the same .htpasswd ? I mean literally the same .htpasswd, not a copy in a different folder.
If you login to access the html and then access a font directly, does it work right away or do you have to login again ?
Is the server running IIS ?

Thank you for the answer.
I have a feeling that the company has a misconfiguration in their remote server. Nothing you can fix, I first thought it could be related to CORS, but my mind changed.

Are the fonts files on the same server and dir/subdir as the html/css/js files, protected by the same .htpasswd ? I mean literally the same .htpasswd, not a copy in a different folder.

I only manage the server that hosts the html, the one that is protected is the remote one which delivers the fonts, css, and js resources. So I can't say anything about the remote's internals and can't answer the question about the htpasswd file. The font files are in different subfolders than css/html/js.
I was wrong about the the .js resources. It only worked because the js resources were loaded directly in head by a script tag. I just tried to load the same resources using javascript with $.get(), and now I will get a 401 status error for this too and a user defined response body which has is 404 page, but if I call the same url directly, I can fetch the resource just fine. The fonts that are failing with 401 gets loaded by a external css file from the same domain, port and protocol were the css file is hosted. I included this external css in a link tag in head.
The only thing different is that all failed resources which gets loaded by $.get()  or by the external css have this 401 status and a 'WWW-Authenticate: Basic realm="..."' header in the response. The realm matches the one that is shown in the credential dialog.
With your addon disabled I get the 401 access denied error and the "Cross-Origin Request Blocked" warning in console for the fonts and js, if I enable your addon the "Cross-Origin Request Blocked" warning goes away, but the 401 stays. When your addon is enabled I can see in the response header how your addon is inserting the Access-Control-Allow-Origin header for both, the js and the fonts.

If you login to access the html and then access a font directly, does it work right away or do you have to login again ?

I just need to login once to access the html, then I can access the fonts or js directly without a password question.

Is the server running IIS ?

No, both are running Apache 2.4.

So I've been experimenting with http basic auth and cross domain requests.
Both Firefox and Chrome do not send the "Authorize: Basic xxx" header that carries the credentials when fetching a font from another domain with a css font-face rule. So even with the proper CORS headers, the request will still fail with an error 401. In fact, when setting the style directly in the html document, I am never prompted for credentials to access the font file.
This is not something that can be solved with a cors header override.
Javascript fetch.
You should be able to send credentials along the request. The jquery docs should provide some answers.
Css web fonts.
The only solution I can think of would be to use the extension "Modify Headers", available here:
https://addons.mozilla.org/en-US/firefox/addon/modify-headers/
and inject the "Authorize" header. You can find its value by looking at the request for the css file.
This should likely apply to javascript fetch calls as well.
I don't think there is much more I can do.

For the record, the fonts fetching behavior seems consistent with the spec:

When fetching, user agents must use "Anonymous" mode [...]

Depending on the implementation, forcing the addition of an Authorize header via extensions might not work, and in any case wouldn't be standard behavior as I interpret it.
I haven't found any rationale for this behavior, it could be an oversight.

It's unlikely the server would care though. I tested the injection of the "Authorize" header and got access to the font, so it's at least possible.

Thank you for your hint with the modify header addon, adding the Authorization header did the trick.


Mention in the documentation that it reads environment variables from an interactive login shell

On my mac, this module does not load .bashrc as the README suggests.  It took me a few hours to track this down, but the -ilc in /bin/bash -ilc env (used in index.js) makes bash act like an interactive login shell, which means that it only reads from .bash_profile. I assumed (wrongly) that a script within a node GUI app would be considered a non-login shell, hence I expected it to load from .bashrc.
zsh seems to behave differently.  When I run  /bin/zsh -ilc env on my mac, it loads variables from .zshenv, .zprofile, and .zshrc. Weird.
Anyway, I think it would be helpful for other users to understand that, in bash, this module only loads variables from files that are sourced in an interactive login shell, ie .bash_profile.
PS: Note that this is complicated by the confusing bash/zsh situation on Mac at the moment.  For instance, if your default shell is bash, but then you run /bin/zsh -ilc env it will include bash environment variables, presumably because the parent shell is bash... confusing. Also, it seems that you have to log out after running chsh -s /bin/bash (as opposed to simply closing a terminal window) to fully change the default shell. Also, just changing the "shells open with command" in Terminal preferences (as some articles suggest) doesn't work. Wow, exciting morning.

I have mentioned this in the readme. PR welcome if you think it could be written better/clearer.


Automate the tracking of Dark Visitors

The idea is to scrape the content of Dark Visitors using a bot and generate PRs for this project. A bit like dependabot.

@cdransf: please could you add a "help wanted" label to this issue? You never know, someone might automate this for us.

@glyn added! I don't see a feed on the site, so this may be manual (I'm signed up for their emails).

The API only offers a subset of list so far. e.g. It does not include uncategorized ones. Fortunately, the site itself does not prevent any bots at all at this point.
I managed to add this feature to my own repo with a python script and commit hooks. It caches the response in a sqlite database as well.
If such added dependencies are fine with you, I can create a PR to help this. But I need to understand your preferences in terms of:

How often, where, and how it should be executed;
What categories of bots should be disallowed by default;
Should it modify files other than the robots.txt;

and any other ideas you might have.
Many thanks for this repo!

@ChenghaoMou that would be great! Perhaps daily to update robots.json? Pushes containing updates to robots.json will then generated an updated version of table-of-bot-metrics.md and robots.txt.

In the past (3 months ago) I wrote a little script to generate robots.txt from DarkVisitor web page. If it can be helpfull :)
https://git.dryusdan.fr/Dryusdan/darkvisitor-useragent-scrapper
The only need is to make a .env in the root of repository folder with
darkvisitors_token=<API token not used>
allow_categories=["Archiver","AI Search Crawler","Developer Helper","Fetcher","Search Engine Crawler","Uncategorized"]
custom_deny_agents=["AhrefsSiteAudit","Mail.RU_Bot","Twitterbot"]

allow_categories is the category we allowed bot (like Google bot), other will be write in robots.txt

@cdransf Here is the workflow working on my fork: run.
code
Based on the documentation, the API/token offered still does not include all the information rn.

Agent types include AI Assistant, AI Data Scraper, AI Search Crawler, and Undocumented AI Agent.

vs what is listed on the webpage

[
"AI Assistants",
"AI Data Scrapers",
"AI Search Crawlers",
"Archivers",
"Developer Helpers",
"Fetchers",
"Intelligence Gatherers",
"Scrapers",
"Search Engine Crawlers",
"SEO Crawlers",
"Uncategorized",
"Undocumented AI Agents"
]

If it looks good, I can create a PR shortly.

This is great! A couple followup questions:

can releases also be automated when there's a change, for people who follow the releases web feed?
maybe the data source should be mentioned in the README and/or the FAQ? Just to make it clear that the list is not maintained manually anymore
overengineering idea: offer a GitHub Action that can inject the robots.txt from this repo into static websites

* maybe the data source should be mentioned in the `README` and/or the FAQ? Just to make it clear that the list is not maintained manually anymore

Isn't Dark Visitors a subset of our robots.txt? If so, the rest is manually maintained.

I probably misunderstood this :sweat_smile: I was under the impression that the robot.txt was now fully generated from DarkVisitors data, looking at the latest commits on main (the full robots.txt is committed by the cron job). But I haven't looked at exactly what the scraper does


Fix CI builds: Update package.lock

Github actions uses 'npm ci' to build which does not update package.lock which is out of date with package.json causing 'npm ci' to exit leading to CI build failures.

I don't understand the npm system very well. So in the very rare times I've updated package.json I've never committed package-lock. json because I don't want to break things for other devs.
Question: in general is it necessary and safe to commit package-lock.json as well as package.json?

Yes, package-lock.json is intended to be committed every time package.json is changed (normally via npm install). See https://docs.npmjs.com/cli/v10/configuring-npm/package-lock-json


feat(backup): add .Status.PluginMetadata field

Closes #5777
Release Notes
The Backup status now contains a PluginMetadata field that can be used to store plugin-specific information.
Note for the reviewers

The PR does not have backport enabled due to the plugins not being in GA. If we wish to make the plugins available to previous CNPG releases we should backport this.

/ok-to-merge


docs: fix typo in $http docs 'Arguments' section

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Docs update (fix typo)

What is the current behavior? (You can also link to an open issue here)

The details describing the transformResponse property of the config param for $http included a link with a typo: "See Overriding the Default TransformationjqLiks" instead of "See Overriding the Default Transformations".

What is the new behavior (if this is a feature change)?
This commit fixes the typo to read "See Overriding the Default Transformations"

Does this PR introduce a breaking change?
No.

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

Backported to v1.5.x (d6ec5d0) and v1.4.x (aec25f1).


Seek to beginning of config file after looking for verbose key

https://github.com/NOAA-OWP/LGAR-C/blob/567adc75bea3ed3580427592f198638b0a8a32ef/src/lgar.cxx#L184-L206
If verbose is not the first key in the config file, lgar will likely not correctly interpret a configuration file. This is because the code looks for the verbose key in all lines of the config and never seeks back to the beginning of the file before parsing the config file for parameters.

Ill submit a fix for this in the morning. Just wanted to document it!


More data passthrough, new generic benchmerk/viewer

Fixes to primitive shapes to align with size/shapes from MJX
Passthrough of texture and lighting data from MJX
Adds a generic viewer and benchrmark tool

Thanks!


Update _index.md

Missing $ in frot of the subscriptionID variable in code example in two places.

@kristopherjturner thank you. Since these are shell commands and not PoSH, there is no need for the $ sign


Pawelka/linux tests

/cc @BrennanConroy @halter73

How do you build on Linux? I tried using CMake but hitting some errors

@BrennanConroy - I will be adding building instructions to README but in general you do this:

build casablanca as per https://github.com/Microsoft/cpprestsdk/wiki/How-to-build-for-Linux (note I built version 2.6.0 but 2.7.0 is now available which I have not tried yet)
create a subfolder in the SignalR root repo (e.g. build)
run  CXX=g++-4.8 cmake .. -DCMAKE_BUILD_TYPE=Release -DCPPREST_INCLUDE_DIR=~/source/casablanca/Release/include/ -DCPPREST_LIB_DIR=~/source/casablanca/Release/build.release/Binaries (adjust values of CPPREST_INCLUDE_DIR and CPPREST_LIB_DIR accordingly)
make
This will build the lib and the tests. The binaries will be in the bin subfolder.

There are some issues with the tests:

there are 3 tests that are flakey and fail quite often (especially when compiled in Debug)
more often than not (again especially in the debug) - all tests appear to have finished but the process does not exit. Looks like gtest is waiting for some threads to finish
My Linux box is a quite old laptop so there might be more (or fewer?) issues on faster boxes

Thanks for looking at this.

@moozzyk I haven't looked at this super carefully, but it seems good to me. Do you want someone else to try running this? I have an Arch Linux VM which might show us something new.

@halter73 - Thanks for taking a look. It would be great to try it in a different setup I tried so far (keeping in mind that Linux support is experimental at this point - see my comment about tests).

I'm hitting an error with the make_unique on line 27 of connection_implementation.cpp
Sent from my HTC
----- Reply message -----
From: "Pawel Kadluczka"<EMAIL_ADDRESS>To: "aspnet/SignalR-Client-Cpp"<EMAIL_ADDRESS>Cc: "Brennan Conroy"<EMAIL_ADDRESS>Subject: [SignalR-Client-Cpp] Pawelka/linux tests (#132)
Date: Fri, Dec 11, 2015 7:34 PM
@halter73https://github.com/halter73 - Thanks for taking a look. It would be great to try it in a different setup I tried so far (keeping in mind that Linux support is experimental at this point - see my comment about tests).

Reply to this email directly or view it on GitHubhttps://github.com/aspnet/SignalR-Client-Cpp/pull/132#issuecomment-164105765.

Weird - I did not see it. I will try to git clean my tree and rebuild. I will also check if the include is not missing since I had to do make_unique on my own.

@BrennanConroy - I fixed the issue with make_unique and moved to cpprestsdk 2.7.0.
I also wanted to try the client on CentOs but could not build casablanca - it could not find the correct version of (suprise) OpenSSL

Looks good, seeing these 2 errors
[  FAILED  ] connection_impl_start.connection_state_is_connecting_when_connection_is_being_started
[  FAILED  ] connection_impl_reconnect.reconnect_cancelled_if_connection_dropped_during_start_and_start_failed

Thanks @BrennanConroy - I filed a bug for this: https://github.com/aspnet/SignalR-Client-Cpp/issues/136


Bump com.azure:azure-ai-openai from 1.0.0-beta.8 to 1.0.0-beta.9

This updates to the latest version of Azure OpenAI Java SDK 1.0.0-beta.9 - see https://github.com/Azure/azure-sdk-for-java/releases?q=1.0.0-beta.9&expanded=true
There are some issues in the deployments and tests, and I believe they are caused by:

new tests that seem to have been added by @agoncal -> I've added some new deployments named text-embedding-ada-002, dall-e-3, gpt-35-turbo-instruct so they can be tested. Some of them are not available in eastus so there's no way to test them at the moment, without starting a second instance in swedencentral.
quotas that seem a lot smaller than before, that's why I added some Thread.sleep(60_000); so we don't run out of quota quickly.

@langchain4j I believe this upgrade is important: I was having some bugs (in another project) with beta-8 and they are solved with beta-9.

@jdubois are all tests passing for you? I have 10 failing tests...
I guess the most critical is AzureOpenAiStreamingChatModelIT.should_use_json_format:
Status code 400, "{
  "error": {
    "message": "'messages' must contain the word 'json' in some form, to use 'response_format' of type 'json_object'.",
    "type": "invalid_request_error",
    "param": "messages",
    "code": null
  }
}

I have commented on it above
It works fine on 1.0.0-beta.8 but fails on 1.0.0-beta.9

@langchain4j for the AzureOpenAiStreamingChatModelIT.should_use_json_format I cannot reproduce, then I have many other errors - those seem to be mostly linked to rate limiting, but I'm first having a look at them.
Here's a screenshot showing that AzureOpenAiStreamingChatModelIT.should_use_json_format works for me, and yes that's annoying!

Hi @jdubois, I can't see AzureOpenAiStreamingChatModelIT.should_use_json_format on the screenshot, only AzureOpenAiChatModelIT.should_use_json_format (without streaming)

Sorry @langchain4j , and I know the issue.
This is caused by https://github.com/Azure/azure-sdk-for-java/pull/40687 which results in encoding the message in Base64.
This has 2 consequences for us:

It makes GTP-4o bad at reasoning (it understands Base64 so it will answer something, but it won't be able to do good reasoning with it. As easy way to test it is with this prompt: "Maria's father has 4 daughters: Spring, Autumn, Winter. What is the name of the fourth daughter?"
For the JSON output, there's a check that the "JSON" keyword is in the message's content, and as it's in Base64 it can't find it... That's why it fails!

I believe this release is too bad to use, and that we should rather wait for the next one.

1.0.0-beta.10 should be released this week, and will fix this issue, so let's close this.

@jdubois do you know when this release will happen exactly? (I plan to release tomorrow)

@langchain4j I don’t have a more precise date, pinging @JonathanGiles as he will know better than me

@langchain4j @jdubois
Beta.10 is released in maven now.

Thanks!! I’ll try to do the PR tomorrow @langchain4j