Doc typos in book chapter 4.6 Git on the Server - Smart HTTP ?

Followed the instructions at https://git-scm.com/book/tr/v2/Git-on-the-Server-Smart-HTTP and noticed it didn't work unless I made two changes:

Instead of ScriptAlias /git/ /usr/libexec/git-core/git-http-backend/, I had to use ScriptAlias /git/ /usr/lib/git-core/git-http-backend/. Not sure what libexec there is, no such path on my system, but lib instead of libexec worked. (note that other parts of the doc page do refer to lib instead of libexec, so assuming this one instance is a typo?)

If I used htdigest -c /opt/git/.htpasswd "Git Access" schacon to create the git user password file, attempting to access would always cause an invalid password error in Apache logs (Apache 2.4, Linux Mint 18 server, serving over https). Instead I had to create the git user password file with htpasswd -c /etc/apache2/passwd.git <user> (found that from ftp://www.kernel.org/pub/software/scm/git/docs/howto/setup-git-server-over-http.html), after which the password authentication would work correctly.

After these two modifications, I'm able to run git clones over https on my web server.
One suggestion to improve would be to add a note and a config option to how to make both fetches and pushes operate behind password protection, not just pushes (with fetches being open to all).

This issue tracker is dedicated to the git-scm.com website.
The issue you have raised is related to the Progit book. Can you copy this issue to https://github.com/progit/progit2/issues ?
Thanks

Oh, thanks! I was led here via the footer on that page, which led me to understand that this would have been the tracker for its issues. I'll make this issue over at the other tracker.

It's complicated because the git-scm.com website embeds the book content. But updates to the book content need to go through that repo.
Possibly pages that show book content should have a more clear explanation in the footer.


test of zoe1 offer robustness

Test of a fix for https://github.com/Agoric/agoric-sdk/issues/8387
This verifies that https://github.com/Agoric/agoric-sdk/pull/8453 performs as expected in an upgrade of synthetic agoric-3 chain.
TODO

[ ] restart vat-admin

closing because further work should be in agoric-sdk


This is not how servermate works.

If you had any sort of competence, you would know that servermate doesn't listen to bots. Nor do most good discord bots.
Please, delete this repo. It is also against servermate's TOS to attempt to automatically bump using [user]bots.

If you had any sort of competence, you would know that servermate doesn't listen to bots. Nor do most good discord bots.
Please, delete this repo. It is also against servermate's TOS to attempt to automatically bump using [user]bots.

This is meant for a user token, so a selfbot

Servermate has in-built selfbot detection. Also, on top of breaking
servermate's TOS, its also against Discord's TOS to selfbot.
On Sat, 26 Dec 2020 at 16:08, Motzumoto<EMAIL_ADDRESS>wrote:

If you had any sort of competence, you would know that servermate doesn't
listen to bots. Nor do most good discord bots.
Please, delete this repo. It is also against servermate's TOS to attempt
to automatically bump using [user]bots.
This is meant for a user token, so a selfbot
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Shqrp/AutoBump/issues/2#issuecomment-751370104, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMWEWQYM73DV2LKJTDYIGILSWYDAXANCNFSM4PIATEYQ
.

Servermate has in-built selfbot detection. Also, on top of breaking servermate's TOS, its also against Discord's TOS to selfbot.
then why are you here


Create a CONTRIBUTING.md definition

Welcome users
Tell them how to setup the environment

Either clone a specific repo and then run generate.py
Or pip install generalpackager then run Packager().create_blank_locally() or something

Mention the Apache 2 license
Small PR guide
Code style
Unittests
Principles such as DRY

Probably missed something

For what is worth, I could get going by just cloning the repo and running pip install -e ., which is great :slightly_smiling_face:
Out of curiosity, what would generate.py do?

Nice, that's good to know, and definitely something that'd be nice to continue supporting! I mainly had

https://github.com/ManderaGeneral/generalpackager/issues/80

in mind. generate.py would replace pre-commit install and possibly do more, but writing this made me think


Visualization of distribution changes of variables after applying IPW

I suggest adding build-in visualization of how confounding variables' distributions become homogeneous across treatment groups after applying the weight by inverse propensity.

Yes, that's a great idea. Would you like to contribute this?

I would be happy to contribute, though I am not sure how it is planned to integrate visualisation module into the library.

@ErikHambardzumyan I just pushed a new interpret API for visualizations and other interpretations in dowhy. Intepreting the causal estimate was another popular issue that was coming up, and I decided to have a single API that combines textual content and visualizations for understanding operations in DoWhy.
So now you can create a new interpreter under interpreters/confounder_distribution_intepreter.py (feel free to choose another name :) )
Here's how the new interpret API works. A user can interpret a CausalModel,  CausalEstimate or CausalRefutation object. The commands will look like:
m = CausalModel(...)
m.interpret() #show causal graph, explain which variables are confounders, etc.
est = m.estimate_effect(...)
est.interpret() # show a textual interpretation of what the estimate value means
r = m.refute_estimate(...)
r.interpret() # custom interpreters for each refutation method, TODO

I believe your proposed visualization for IPW will be an interpretation for the CausalEstimate object. The signature for interpret is interpret(method_name:str, **kwargs).
A quick note on file structure:

/interpreter.py contains the base class that every interpreter should inherit from. It processes the optional kwargs arguments.
There are two additional base classes that inherit from Interpreter: TextualInterpreter and VisualInterpreter, both in the interpreters directory. You may choose to inherit from them, although right now they are a bit empty.
For convenience, I've added two example interpreters:

TextualEffectInterpreter: Returns a string that interprets the ATE estimate value in words.
PropensityBalanceInterpreter: Shows the change in standardized mean differences between treatment and control (balance plot) for propensity score stratification.

To create your own visualizer, you can:

Create a new file in interpreters folder. Make sure to name the file as class_name.py with underscores, and name the interpreter class within as, ClassName. This helps in automatically invoking your interpreter when user types interpret(method_name="class_name")
The only other requirement is that your interpreter class inherits from Interpreter or VisualInterpreter and that it implements two methods: __init__ and interpret.
Feel free to use interpreters/propensity_balance_interpreter as a template and build on it. You will see that you can specify the supported estimators for your visualization as a class variable (SUPPORTED_ESTIMATORS) and that the IPW estimator can be accessed as self.estimate.estimator. Not all visualizations will apply to every estimate.

If everything works, you should be able to test your interpreter as:
est.interpret(method_name="confounder_distribution_interpreter", ....)

Let me know if you have any questions. Thanks for starting this discussion and look forward to your contribution!


React 18 version which has useSyncExternalStore()

Do you have a branch of the repository which has a version of React which has the new useSyncExternalStore functionality?  I’m trying to use the use-sync-external-store package but can’t seem to get the shim to not be used yet.
The warning:
use-sync-external-store.development.js:52 Warning: You are using an outdated, pre-release alpha of React 18 that does not support useSyncExternalStore. The use-sync-external-store shim will not work correctly. Upgrade to a newer pre-release.

I’ve tried installing multiple versions of React @next and @experimental and clearing node_modules/yarn cache clean, but haven’t figured it out yet.

Simply, I don't think it's released yet.
Here's a link to my experiment with React 17: https://twitter.com/dai_shi/status/1434131745700417540

That was my suspicion, thanks! I’ve also noticed tearing getting worse when the shim is used, so hopefully this gets sorted out.

It seems like useSyncExternalStore is shipped in the experimental branches. It’s tricky to get it all working because it seems like you have to use exact versions or some random React alpha is requested instead.
diff --git a/package.json b/package.json
index 8122d02..662f7dc 100644
--- a/package.json
+++ b/package.json
@@ -40,7 +40,7 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "@apollo/client": "^3.4.10",
+    "@apollo/client": "/Users/briankim/Work/apollo-client/dist/apollo-client-3.5.0-beta.13.tgz",
     "@react-rxjs/core": "^0.8.3",
     "@simplux/core": "^0.17.0",
     "@simplux/react": "^0.17.0",
@@ -49,8 +49,8 @@
     "effector-react": "^22.0.4",
     "graphql": "^15.5.2",
     "jotai": "^1.3.2",
-    "react": "^18.0.0-alpha-1314299c7-20210901",
-    "react-dom": "^18.0.0-alpha-1314299c7-20210901",
+    "react": "^0.0.0-experimental-67222f044-20210915",
+    "react-dom": "^0.0.0-experimental-67222f044-20210915",
     "react-hooks-global-state": "^1.0.2",
     "react-redux": "^7.2.5",
     "react-tracked": "^1.7.4",

This got me a useSyncExternalStore() version (ignore the Apollo Client stuff).

Closing as resolved! Thanks!


Intersecting SNPs

Hi
I am seeing lot of warnings that says "SNP out of order has been skipped" while using "find_intersecting_snps.py". Can you please explain what the warning means and is it okay to ignore?
Thank you

What do your SNP files look like? Are they sorted by position?

They were sorted by position. The problem went away when SNP positions do not contain "reference skips" regions.


Documented GEM_HOME

Without setting a dedicated Ruby environment via the GEM_HOME
environment, the development environment is likely to break
the regular crowbar setup on the admin node it is installed
to by installing gems to the system paths. This commit adds
instructions on using GEM_HOME to avoid this. That way a cloud
can be re-used for other purposes that do not involve the crowbar
dev environment.
Note: mkcloud's devsetup step does not use GEM_HOME either so it would probably be a good idea to adapt that as well.

Just execute bundle with the --path option, but this documentation is anyway outdated.

True, but that's not documented either :-)
What else is outdated? I'd rather fix this (or even remove it entirely) than leave stale documentation around for people to get confused...

LGTM, always up for not installing gems into the system +1

@jgrassler overall I wouldn't say it is outdated, to me it looks still valid, but it could have been updated after merging https://github.com/SUSE-Cloud/automation/pull/1027
so this change here lgtm
+1
@rsalevsky after that can you update the docs to fit the devsetup step?


BUG: res_transformers.power1 contains NaN for neutral

Describe the bug
The real value of res_transformers.power1 is NaN for the neutral branch.
To Reproduce
Minimal Working Example to understand the problem
en = ElectricalNetwork.from_catalogue(name="LVFeeder38211", load_point_name="Winter")
for name, branch in en.branches.items():
    if name == 'Transformer':
        branch.parameters.max_power = 160. * 1e3 * 1.1

en.solve_load_flow(auth=auth)
print(en.res_transformers.power1.isna().sum())

Expected behavior
The real value of res_transformers.power1 is not NaN for the neutral branch.
Version details
roseau-load-flow 0.6.0
System Information
------------------
python               3.10.8.final.0
os                   linux
os_name              posix
machine              x86_64
Installed Dependencies
----------------------
pandas               2.1.0
numpy                1.23.5
geopandas            0.13.2
shapely              2.0.1
regex                2022.10.31
pint                 0.22
requests             2.31.0

@YassineAbdelouadoud @alihamdan will take a look at this bug and at #158. Thanks for the report. :+1:

Hello Yassine, thanks for the report.
This is working as expected here. The transformer is Dyn, it has the phases "abc" on the primary side and "abcn" on the secondary side. Since the neutral doesn't exist on the primary, it has the values in the results set to nan. If you don't care that a value does not exist, I suggest filling it with 0 (like in en.res_transformers.fillna(0)).
Note that this could only happen with transformers as the phases on the primary and secondary are not guaranteed to be the same, while lines and switches would always have the same phases on both sides.


Parse fuzzy scrutiny dates

The latest taxonomic scrutiny date data found in the CoL is treated there as text and has various forms of partial dates.
We would like to parse all these into a structured form, either a LocalDate as we have now in the data model Taxon.accordingToDate or a new FuzzyDate class that:

has 3 year, month, day int fields
requires a year, but allows fuzzy dates by having a null day or even a unknown month (in that case also unknown day)
offers a getTemporalAccessor() method that returns either a Year, YearMonth or LocalDate instance to interact with the java API.

If the LocalDate class is used we need to point a year to the 1st January of that year and likewise for months only. This is not ideal but an ok start.

see https://github.com/Sp2000/colplus-backend/blob/master/colplus-parser/src/test/resources/dates.txt with 100 example values

@ayco-at-naturalis any preference yet?

Oops only now see your question ... again I have opted for the "simple" solution b/c it hooks most easily into existing parsing code ... FuzzyDate or something is something much grander that would necessitate a complete rethink of date parsing (also for DwCA)

So we would have multiple date parsers for a LocalDate and apply them by context?
I am not sure if that is feasable, ultimately we will move some resources away from ACEF into DwCA. We are also likely facing similar dates with dwc terms. I think we better have just a single date parser.

if we can get this new parser to handle all dates we encounter in other dwc or acef date fields then lets use that instead of the GBIF wrapped one.

btw, here are gbif date parser and tests:
https://github.com/gbif/parsers/tree/master/src/main/java/org/gbif/common/parsers/date
https://github.com/gbif/parsers/tree/master/src/test/java/org/gbif/common/parsers/date

maybe we should inventory all distinct date types we see in our data

Yes, I think we should make an inventory of all formats and then sequentially apply those (long to short, or in some configurable sequence to avoid wasting time on improbably formats given some data source) to any date string to be parsed

Looking at the data that doesnt seem too difficult, we only have very few terms which are dates.
I am dumping all distinct values for these:

ACEF:LTSDate
dwc:namePublishedInYear
dc:modified

I cannot get this one from the db, please check the archives or assembly db for data:

ACEF SourceDatabase.txt ReleaseDate

Thats all there is apart from References.
But thats a special beast as its embedded in the CSL and we can deal with that later I reckon.
So maybe your AcefDateParser can be extended and become THE date parser, ditching the GBIF beast

Just writing ... yes ... I think the current gbif date parsing library is difficult to hook into if another date format pops up in the data, and probably more complex than needed ... we just need a list of "allowed date formates" and sequence over it.

dates.tar.gz

ideally all dates are ISO dates, but we need to adapt to whatever we see

excellent!
Before closing this we need to be sure we really want "2016-03-01" in our API instead of "March 2016". The artificially created precision is misleading. For now its sufficient, but we should keep an issue open (maybe new one that links to this) to improve the fuzzy date storage

Some imports now fail with IllegalArgumentException
	at org.col.admin.task.importer.acef.AcefDateParser.parse(AcefDateParser.java:48)

The parsers should never throw any other runtime exception but an UnparsableException

btw, the new DateParser should live in the parser module

I am implementing the "fuzzy date" logic on a different branch.  What's left in the master branch should probably restored to the original code until I'm ready to merge the branches (although that will cause import problems, because that's why we have this issue)

I wonder whether a one-size-fits-all parsing (as currently happens) is a good thing. Data quality would probably improve if we would/could make the date parser data source-aware, maybe even field-aware. For example so that we can successfully (or more easily) parse dwc published-in dates like "2007a". Also we know that some data sources use american dates (mdy) while other use european dates (dmy), so it doesn't make sense to check for ambiguity (and it may actually be bad).
In other words, it's probably a good thing if "inserters" and "interpreters" can configure their own parser, per data source type (ACEF, DWCA) and possibly even per field.
At least it probably pays the make date parsing more configurable.

thats true, I just find it hard to know how to configure it. Maybe passing at least a preferred & expected date format (through static builder pattern?) that is tried before any of the others would be good

Finished but not merged back to master


Certificate provisioning problems with Istio 1.9

I'm running into problems with a non-functional net-http01/Knative setup (0.23 and 0.24) after upgrading from Istio 1.8 to 1.9.
When spinning up a new Knative service, the http URL never changes to https and the net-http01 log contains a number of entries like this (eventually followed by rate limit errors):
{"severity":"ERROR","timestamp":"2021-08-04T12:40:51.615383808Z","logger":"net-http01","caller":"ordermanager/ordermanager.go:240","message":"Encountered an error waiting for challenges: acme: authorization error for HOSTNAME: 403 urn:ietf:params:acme:error:unauthorized: Invalid response from http://HOSTNAME/.well-known/acme-challenge/FOO [IP_ADDRESS]: 503.","commit":"b44a7db","knative.dev/controller":"knative.dev.net-http01.pkg.reconciler.certificate.Reconciler","knative.dev/kind":"networking.internal.knative.dev.Certificate","knative.dev/traceid":"d5729d7a-3feb-4d2b-a836-31fb47cafaa9","knative.dev/key":"default/route-935d4abc-27ab-4be5-94fa-7be9f3fc6e8f","stacktrace":"knative.dev/net-http01/pkg/ordermanager.(*impl).initiateNewOrder.func2\n\tknative.dev/net-http01/pkg/ordermanager/ordermanager.go:240"}

When accessing the new service's acme-challenge URL with curl, it does indeed return a 503 status code.
Do you have any idea what might be causing this, or suggestions for further investigation? Thanks!
Update: the same problem occurs with a fresh install of Istio 1.9/Knative 0.24 on Kubernetes 1.20.

This is not a bug in net-http01, but I'm documenting the fix here just in case other people run into it.
The problem occurs with Istio 1.9 on a cluster with global STRICT mTLS. As described above, requests to the acme-challenge URL will receive a 503 response. This can be fixed by setting the entire knative-serving namespace to PERMISSIVE mTLS, as described here: https://knative.dev/development/admin/install/installing-istio/#using-istio-mtls-feature
This was not required prior to Istio 1.9, and I'm not 100% sure why it broke now. The most likely reason seems to be a change in how port-level mTLS configuration is interpreted, which is something that a few services in knative-serving use.


Restructure per-feature headers

please make 1.1 branch for future patches and further minor releases ... then merge


CRAN package should not have newer version from github indexed

In some cases, Rdocumentation index a newer version of a package that comes from github, if the package is a CRAN package, we should not index versions from github

This is actually already done, see https://github.com/datacamp/RDocumentation-lambda-worker/blob/master/functions/update_github_packages/index.js#L198


Instrument gRPC coroutines library with log statements

It would be beneficial to users if they were able to enable log output when debugging issues with coroutines. One of the main reason this was avoided initially was because we didn’t want to tie users to a particular logging framework. Additionally we didn’t want to introduce any overhead into the library.
Using something like googles flogger would provide performant logging and allow users to include whichever backend matches their current logging framework.

This conversation from grpc-java sheds light on somethings that need to be considered when implementing this.
https://github.com/grpc/grpc-java/issues/1577


Add Wox

Microsoft Reviewers: Open in CodeFlow

Change to add latest version V1.4.1196, and I submitted it to Windows Defender.

thank you


Crashes when running UWP

This project immediately crashes when running UWP with the error:
{Windows.UI.Xaml.UnhandledExceptionEventArgs}
It is in the file App.g.i.cs, line 53. There is no stack trace. I am using Visual Studio 2017.
Any ideas?

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

ID: ec31ee56-e3b9-a70a-a4ef-80fdf58e7249
Version Independent ID: 766b5774-50bf-f43f-800d-48b5c6c66509
Content: Xamarin.Forms Master-Detail Page - Xamarin
Content Source: docs/xamarin-forms/app-fundamentals/navigation/master-detail-page.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Confirmed that this is crashing.

The quick fix is to add the Win2d.uwp NuGet package to the UWP project. Then the sample will run.
Early next week I'll fix this issue across all projects that require it.

Thanks

I've bumped all of our samples to use the latest build of Forms 3.6. On UWP, this will cause a transitive restore of Win2d.uwp, due to being a dependency of Forms.


Make more Stylish and  responsive Design

Complete the following REQUIRED checkboxes:

[ ] I have thoroughly read and understand The Odin Project Contributing Guide
[ ] The title of this issue follows the location for request: brief description of request format, e.g. Foundations exercises: Add exercise for XYZ

The following checkbox is OPTIONAL:

[ ] I would like to be assigned this issue to work on it

1. Description of the Feature Request:

2. Acceptance Criteria:

3. Additional Information:

I don't understand what you're requesting, these are basic exercises for building CSS skills, what should be "more stylish"?

Closing due to inactivity and unclear intentions


Alternative bit packer not using SIMD and not using C

SIMD is not portable as is on ARM. Also this would open the door to compile tantivy to web assembly.

I apparently forgot to close this issue. it is already possible to compile tantivy without SIMD by using the  --no-default-features flag


Pretrained checkpoints

Hi,
Thank you for maintaining the repo this nicely. However, would it be possible to provide some pretrained checkpoints for 'VISDA-C' or 'Office-31' datasets?
Thanks and Best Regards,
Mirza.

Hi,
Thank you for maintaining the repo this nicely. However, would it be possible to provide some pretrained checkpoints for 'VISDA-C' or 'Office-31' datasets?
Thanks and Best Regards, Mirza.

Hi, Mirza
Sorry for the late reply due to the busy period for CVPR rebuttal. Maybe one week later, I would like to upload the pretrained checkpoints. I will notify you once they are okay for downloading.
Best,
Tim

@jmiemirza pls refer https://github.com/tim-learn/SHOT/blob/master/pretrained-models.md for the pre-trained source models.


Fix related external link bug

When updating an artefact related external links were being 're-saved' with each update effectively doubling them each time. Destroying them was also broken. This PR fixes the bug and adds some tests.
Zendesk Ticket

Graham has kindly begun my induction into the hallowed halls of rails users by explaining in detail how this works - the only puzzle piece missing is Mongoid's problem.
As a result I think this is good to go.


tfds에서 데이터 관리 어떻게 하는지

https://github.com/tensorflow/datasets
현재 keras-retinanet에서는 pulbic dataset을 직접 다운받아서 경로를 지정해줘야 하는데
tfds에서는 어떻게 그걸 해주는지

We are trying to make the downloading process be encapsulated as "tfds" package does. But 'tfds' gives *.tfrecord file. We are looking for other ways to get image-extension too.

@minus31 thanks for your advice.
Then how about [this](colab.research.google.com/github/visionNoob/Tensorflow_Tutorial/blob/master/Day_1/Q1 - Score Prediction.ipynb)?
I don't wanna give up tf.record. cuz it's very efficient.


[Lint] Change eslint configuration for new styles

Description
The bots (sonatype)  and other ci are throwing semicolon errors which is not fixed by eslint --fix, Either we should choose to remove the use of semicolons completely, or to use semicolons everywhere required. We can't stay in between since it will create linting issues for other CIs.
Next, moving to good practices. There should be no spacing required before colons when declaring objects or writing key: value pairs. Infact, when we autoformat the code within the IDE(vscode/intelliJ) it formats the code by removing all the spaces before colons(":").
Thus made two changes in the eslint config file to

Add semicolon as required (We can also choose to remove semicolons from all places as the other alternative)
Remove whitespace before colons (To get in sync with latest practices)

Now when you run eslint --fix then it will autoformat the code to follow the rules and will not throw error or become burden on developer to add semicolons or remove extra whitespaces.

Codecov Report

Merging #4577 (15a23cf) into master (e23ff24) will decrease coverage by 7.16%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4577      +/-   ##
==========================================
- Coverage   33.28%   26.12%   -7.17%
==========================================
  Files          60       60
  Lines        5053     5053
==========================================
- Hits         1682     1320     -362
- Misses       2935     3409     +474
+ Partials      436      324     -112

Flag
Coverage Δ

gointegrationtests
?

unittests
26.12% <ø> (ø)

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

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/start.go
1.11% <0.00%> (-58.52%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
2.50% <0.00%> (-47.50%)
:arrow_down:

mesheryctl/internal/cli/root/system/status.go
0.00% <0.00%> (-47.44%)
:arrow_down:

mesheryctl/internal/cli/root/system/update.go
1.05% <0.00%> (-34.74%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
0.86% <0.00%> (-27.59%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.39% <0.00%> (-22.38%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
3.29% <0.00%> (-20.88%)
:arrow_down:

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 e299608...15a23cf. Read the comment docs.

@Abhishek-kumar09 @sudo-NithishKarthik, what do we want to do about this bot? Keep it or remove it?

This bot is not helping us. It is better to be rid of it.

This bot is not helping us. It is better to be rid of it.

Agree with removing it since it doesn't follow react syntax. maybe it need more customization.

Thank you, gents. I gave this bot the 👢

Thank you @Abhishek-kumar09 .
Merge conflicts cropped up ^^

@sudo-NithishKarthik
I have created a new PR for this, Actually, there were merge conflicts raised and there were problems at some points choosing between the changes, so decided not to take risk.  https://github.com/meshery/meshery/pull/4627


Bugs in the UESRPG DnD 5e sheet

I have a question concerning the UESRPG character sheet for D&D 5e the sheet seems to have a few bugs in it.

When entering attributes it does not calculate modifiers
When you change your level it does not modify your proficiency modifier
It does not calculate Saves, Skills, Spell Save DC or Spell Attack modifier
I am also not sure if the if the expertise or jack of all trades feature works or not

The description of the sheet says that it is based off of the 5e sheet by roll20 if that would help speed up the debugging of the sheet .I would like to use the sheet for an upcoming game but understand if there is not enough time to check it out and fix it.

Issue #5153 also refers to the same problems that I am encountering


Add Docstrings

Add doc-strings to functions, improving documentation

Will close this and create a master project to cover all documentation-related work.


Adding git fetch

I'm not sure that this is a good practice, but I use to optionally add a git fetch before the git status:
https://github.com/Ircama/clink/blob/ircama_git_status/examples/git_status.lua#L15
https://github.com/Ircama/clink/blob/ircama_git_status/examples/git_status.lua#L42
This also runs in flexprompt.lua. Would you include it?

Doesn't git fetch make network calls, and potentially change the local state?
Is this a widespread practice?  I can see how it might be beneficial in some workflows.  I'm a little surprised and am not sure yet whether it's something to build into flexprompt.
With that said, I don't think I would recommend contacting remote servers every time I press Enter, nor changing my local workspace state just because I pressed Enter.  That seems a bit aggressive.
In any case, if git fetch will be executed at every prompt, then it might be nice to use a coroutine and io.popenyield() so that it can run in the background instead of blocking the prompt while waiting for network IO.
Or maybe it's better to have a separate prompt filter, maybe priority 1, that checks if the current directory is a git repo, and runs git fetch if so.  The prompt coroutines run in the background, but they run in series, one at a time (consecutively, not concurrently).  So, a separate prompt filter at priority 1 could use a coroutine and still guarantee that git fetch finishes before the built-in {git} module runs.
What are your thoughts?

With that said, I don't think I would recommend contacting remote servers every time I press Enter, nor changing my local workspace state just because I pressed Enter. That seems a bit aggressive.

I agree with this. If something like this is added, I hope there will be an option to turn it off (or even better, leave it off by default)

Thanks for the feedbacks.
What I reported before is something temporary and I agree that it might be a bit aggressive fetching data from a remote server on every Enter.
The implementation of a separate prompt looks like a good point, exploiting async processing (setting also the mentioned high priority), maybe performing the operation periodically and possibly based on rules (e.g., not more than once a day).

maybe performing the operation periodically and possibly based on rules (e.g., not more than once a day).

Yes, I was wondering if the bigger picture goal might be something like "somehow keep somewhat up to date automatically, without any extra user action".  That does make sense.  Personally I prefer being explicit, since every once in a while an automatic-but-potentially-destructive operation might actually have a painful side effect.  I've "helped" myself in painful ways enough times over the years that I try to be wary.  😉

git fetch (not git pull) is generally safe and does not change your own local branches. It just updates the local copy of the remote repository, so that git status reports appropriate tracking. Otherwise, especially when you are doing limited/slow coding contribution over a branch linked to an upstream repository which is being frequently updated by the maintainers, git status risks reporting improper up-to-date information and you might work on a code that produces potential merge conflicts that you discover while performing the PR. In any case, the practice of setting up a periodical auto-update is arguable and should be made optional, but there are many use cases where activating this option is definitely helpful. Notice for instance that GitHub Desktop - the official GitHub GUI desktop client - has it active by default (File > Options > Advanced > Periodically fetch...), but also allows disabling it (I guess that the background fetching is internally set to occur every hour and the interval is not configurable). Also other IDE popular tools provide this feature.
By thinking about how to implement it with clink, a very simple way might be to monitor the .git\FETCH_HEAD file modification time whenever Enter is pressed and do a git fetch if the date is older than a predefined interval (comparing it with the current time). Anyway, I do not think to be able to implement it: perhaps the features of the LuaFileSystem library are useful for this purpose, but clink does not integrate it currently.

git fetch (not git pull) is generally safe and does not change your own local branches.

Ok, thanks, it wasn't clear to me from the documentation whether it can change the local workspace, but it had sounded like it could.

Notice for instance that GitHub Desktop - the official GitHub GUI desktop client - has it active by default (File > Options > Advanced > Periodically fetch...), but also allows disabling it

Ok, thanks, that precedent satisfies me plenty well.  :)  I'll add a flexprompt.settings.git_fetch_interval:  negative number means never, the units will be in minutes, and the default will be 60.  Sound good?

By thinking about how to implement it with clink, a very simple way might be to monitor the .git\FETCH_HEAD file modification time whenever Enter is pressed and do a git fetch if the date is older than a predefined interval (comparing it with the current time). Anyway, I do not think to be able to implement it: perhaps the features of the LuaFileSystem library are useful for this purpose, but clink does not integrate it currently.

Throttling it sounds reasonable.  It seems reasonable to simply remember repos that flexprompt has fetched in the current session, and avoid fetching them again until at least X minutes later.  That's easy to implement with a table, and avoids needing to fuss with getting file modification times.

Also, I'll look at adding an os.touch() and adding some additional fields to the "extrainfo" table returned by os.globdirs() and os.globfiles().
Thanks for the link to LuaFileSystem.  I could include it, but most of what it provides is already available.  And lfs.attributes() is inefficient for iterating in a directory; in contrast, Clink's os.globfiles() is much more efficient except that Clink currently doesn't actually return most of the fields -- adding them there will run faster.
If LuaFileSystem and some other libraries are commonly used/required, then it may make sense to include them even if they aren't necessarily as efficient or whatever.  Thoughts?

Also, I'll look at adding an os.touch()

Yes, sure. Anyway, git fetch already touches .git\FETCH_HEAD, so, to throttle fetching, its comparison with the current time could be enough and this implicitly supports different repos without an internal table.

so it will simply be a number of minutes and the default will be nil). Sound good?

That would be perfect, setting it disabled by default, thanks!

If LuaFileSystem and some other libraries are commonly used/required, then it may make sense to include them even if they aren't necessarily as efficient or whatever. Thoughts?

LuaFileSystem was just an example and I was simply considering lfs.attributes(), because I wouldn't be capable of doing this otherwise. Exploiting os.globfiles() would be even a better and more compact alternative.

Yes, sure. Anyway, git fetch already touches .git\FETCH_HEAD, so, to throttle fetching, its comparison with the current time could be enough and this implicitly supports different repos without an internal table.

Of course.  But as you noted, there is no API in Clink for getting a file's timestamp.  While I've added one, I'd like to avoid making flexprompt require the absolute latest version of Clink.  And so the alternative approach seems like it should suffice.
Also, while it's possible to make tools directly access git internal data, it makes the tools (more) susceptible to failure when git enhances its internal data schema.  So if it's not crucial to make assumptions about git's current implementation details, then I would prefer not to.  (Sometimes it is crucial -- e.g. in some cases directly accessing git internals is necessary to work around performance problems in git -- but that's not the case here, I don't think.)

Sorry @chrisant996, this morning I did not realize you have already implemented the feature, which works perfectly. It's fantastic, thank you, I appreciate a lot this new product capability; flexprompt is excellent!

@Ircama @pukkandan this is somewhat related:
The clink-gizmos repo now includes an autopull.lua script.
The script has some configurable settings (via clink set).  When configured with a list of directories, it will automatically periodically run git pull in each of the directories.  This can be handy for updating repos that contain Lua scripts, or any other repos that the user wants to keep up-to-date.
If any errors occur during the git pull, it logs them in the clink.log file and reports them when the next prompt is displayed.


autogen-agentchat[gemini] - RAG error

What happened?
Hi, I have just started to configure autogen-agentchat[gemini]~=0.2 to use Gemini 1.5 Flash for function tool calling. However, when the Gemini Flash trigger to call the tool (which is a RAG retrieval), it gives the following error:
Error: Expected requested number of results to be a int, got 5.0
This is despite that I have defined the n_result to be 3 (integer) in my function. Clearly my code in the function was being overridden not sure by the LLM or some codes.
I did not encounter this error when I was using autogen-agent[mistral].
Please help.
Thanks & Best Regards.
What did you expect to happen?
Hi, I would have expect the program to run smoothly without error like what I have experienced using Mistral instead of Gemini.
How can we reproduce it (as minimally and precisely as possible)?
This is the error message :

EXECUTING FUNCTION retrieve_content...
Initiator (to chat_manager):

Initiator (to chat_manager):
***** Response from calling tool (410) *****
Error: Expected requested number of results to be a int, got 5.0

Next speaker: Coordinator
Coordinator (to chat_manager):
***** Suggested tool call (76): retrieve_content *****
Arguments:
{"message": "Tender requirements for vendor submission", "n_results": 5.0}

My function is as follows:
def retrieve_content(
message: Annotated[
str,
"Refined message which keeps the original meaning and can be used to retrieve content for code generation and question answering.",
],
n_results: Annotated[int, "number of results"] = 3,
) -> str:
tenderowner.n_results = n_results  # Set the number of results to be retrieved.
_context = {"problem": message, "n_results": n_results}
ret_msg = tenderowner.message_generator(tenderowner, None, _context)
return ret_msg or message
AutoGen version
0.3.0
Which package was this bug in
AgentChat
Model used
Cloud hosted Gemini 1.5 Flash
Python version
No response
Operating system
No response
Any additional info you think would be helpful for fixing this bug
No response

From what I gathered, the error arises due to the behaviour of certain LLM's.


Wording mistake in ARO Networking Concepts

Regarding URL:
https://docs.microsoft.com/en-us/azure/openshift/concepts-networking
"Ingress Visibility - Set the API visibility when running the az aro create command.
"Public" routes will default to a public Standard Load Balancer (this can be changed).
"Private" routes will default to internal load balancer (this can be changed)."
Should be :
"Ingress Visibility - Set the Ingress visibility when running the az aro create command.

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

ID: 8e00b226-866a-91c0-8796-280b264734f8
Version Independent ID: 09ba08c2-7cd9-cc8f-7c4d-fec7a8e1e87d
Content: Concepts - Networking diagram for Azure Red Hat on OpenShift 4
Content Source: articles/openshift/concepts-networking.md
Service: azure-redhat-openshift
GitHub Login: @sakthi-vetrivel
Microsoft Alias: suvetriv

Thank you! I made a pull request to fix this. https://github.com/MicrosoftDocs/azure-docs-pr/pull/177044


chore(quote): change slot names to dash-case

Related Ticket(s)
Refs #4230.
Description
Changes slot names of <dds-quote> from pascal case to dash-case to align the naming pattern to other components.
Changelog
Changed

The slot names of <dds-quote> from pascal cse to dash-case.

@asudoh one thing though, the documentation isn't updated, and can the slots get documented too?

Great catches @jeffchew - Updated.

@jeffchew Thank you for catching missing source-copy slot content! Fixed.


About experiment

Hi, If it's convenient, could you share the script of YCSB in Caliper with me？

Our Caliper is heavily optimized for our own usage and tightly coupled to our environment.
As it may contain confidential information, pls understand that we are not convenient to disclose it.

Hi, we take some efforts to strip away the platform-specific artifacts from our own Caliper.
We have released it in https://github.com/RUAN0007/caliper-min.
You can refer to YCSB, Smallbank, and another Custom workloads under benchmark/ directory of the above repo.

Thank you very much, it will help me a lot.


Feature Request: Searchfield in sidebar

Would it be possible to add a correctly styles and align searchfield to the sidebar?

Is this feature request related to a specific issue? And can you describe a solution you'd like?

Closed due to inactivity


Releaser fixes for archives.

https://goreleaser.com/deprecations/#archivesreplacements

Codecov Report

Merging #1595 (f7538b6) into master (94dfb5d) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##           master    #1595   +/-   ##
=======================================
  Coverage   78.49%   78.49%
=======================================
  Files         280      280
  Lines        7906     7906
=======================================
  Hits         6206     6206
  Misses       1329     1329
  Partials      371      371


Disable arm7 builds (#7037)

backport #7037

We should probably remove unworking binaries from downlaods and github releases as well


How to add click event when any markdown word clicked.

Markwon version: 4.3.1

I am using library to add mentions and highlight them in messages.
For that i have create custom plugin, custom extension, custom delimiter processor and custom node.
I am able to highlight mention in message using foreground span.
Example:-
Message: Hello @Bhaumik1406@, How are you ?
After Setting markdown : Hello @Bhaumik1406 , How are you?
Now i want to have click listener on that highlighted mention.

Hello @gbhaumikkumar14 !
Well, add a LinkSpan with special URL that you can identify and provide own LinkResolver:
final Markwon markwon = Markwon.builder(this)
        .usePlugin(new AbstractMarkwonPlugin() {
            @Override
            public void configureConfiguration(@NonNull MarkwonConfiguration.Builder builder) {
                builder.linkResolver(new LinkResolverDef() {
                    @Override
                    public void resolve(@NonNull View view, @NonNull String link) {
                        if (/*my link*/) {
                            // handle you link
                            return;
                        }
                        // else pass to default implementation
                        super.resolve(view, link);
                    }
                });
            }
        })
        .build();


[FORMATTING] BigQuery: Doesn't handle escaped quotes in raw strings

Input data
Reproduces on https://sql-formatter-org.github.io/sql-formatter/
Which SQL and options did you provide as input?
Language: BigQuery
SELECT
    regexp_extract(event_properties, r'Email": \'(.*?@.*?)\'')
FROM test

Actual Output
Parse error: Unexpected "@.*?)\'') " at line 2 column 50. SQL dialect used: "bigquery".

Thanks for reporting.


⚠ Bofrost.fr has degraded performance

In 1d1ca5f, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1031 ms

Resolved: Bofrost.fr performance has improved in 41ff499 after 6 minutes.


✨[Feature Request]: Preloader

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I would like to add a preloader during loading loading of page
Use Case
Adding a preloader will enhance the UI of the site
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

Hi @Jagpreet153,
Thank you for bringing up this feature request! I'm happy to assign this issue to you. Adding a preloader will definitely enhance the UI of the site. If you need any assistance or have any questions, feel free to reach out. Looking forward to your contribution!
Best,
Sanika Chavan.
[https://www.linkedin.com/in/sanika-chavan-52457b236]

@sanika391 its two week this issue created no pr added please assign this issue to me i will resolve it

please  i assign this issue to me

Hi aikansh008 ,
Thank you for bringing up this feature request! I'm happy to assign this issue to you. Adding a preloader will definitely enhance the UI of the site. If you need any assistance or have any questions, feel free to reach out. Looking forward to your contribution!
Best,
Sanika Chavan.
[https://www.linkedin.com/in/sanika-chavan-52457b236]


Create an automated way to clean up swarming bot caches

It happens caches are polluted between builds on the same bot. Existing way to clean up cache is manually ssh to the bot. This has been an expensive process and it is unreliable and getting more expensive as new security requirements require 2 approvals to ssh the machines.
We should have an automated way to do the clean up.

Closing as obsolete.


Only 100 videos per show are processed

You are only getting the first page of videos for a show. For example "fang-des-lebens-der-gefaehrlichste-job-alaskas" has 122 videos. Since the new api always need pagination, you should check for total amount of pages and get them too.
You could fix it like this (sorry i was to lazy to issue a pull request):
new code:
def get_videos_api_request(showid, token, page):
try:
req = get(SHOW_INFO_URL.format(showid, page), headers={"Authorization": "Bearer " + token})
except Exception as e:
logger.critical("Connection error: {0}".format(str(e)))
return False
if req.status_code != 200:
    logger.error("This show does not exist.")
    return False

data = req.json()
if "errors" in data:
    logger.error("This show does not exist.")
    return False

return data

Replacement for lines 83-96 in dmax.py:
data = get_videos_api_request(showid, token, 1)
if data["meta"]["totalPages"] > 1:
    logger.info("More than 100 videos, need to get more pages")
    for i in range(1, data["meta"]["totalPages"]):
        more_data = get_videos_api_request(showid, token, i+1)
        data["data"].extend(more_data["data"])

Thank you very much!


How can I render QR Code on a PDF?

I need to render a QR code on a PDF which my end-user can download. FOr creating the pdf I am using react-pdf/renderer.
Here is my code -
import React from "react";
import {Page, Text, View, Document, StyleSheet, Image} from "@react-pdf/renderer";
import { QRCode } from "react-qr-svg";

export function PdfDocument(props) {
    return (
        <Document>
        <Page size="A4">
          <View>
            <Text>Section #1</Text>
          </View>
          <View>
            <Text>Section #2</Text>
            <QRCode
                bgColor="#FFFFFF"
                fgColor="#000000"
                level="Q"
                style={{ width: 256 }}
                value="some text"
            />
          </View>
        </Page>
      </Document>
    );
}

The above gives me the following error -
Error: Invalid element of type rect passed to PDF renderer
What I am doing wrong? How can I render a QR code on the fly onto a PDF?

why closed? did you solved?

Wrong repository, that question belongs in react-pdf and/or react-qr-svg

Have you solved this issue ? i'm facing this problem now

HI @ChengLam123
I solved it, but I guess there is no native way to solve it. I kind of did a dirty hack.
I laid out the HTML code and used a Qr Code generator (like this one), to render a QRcode in the HTML itself. Later, I converted the HTML to pdf.

You can also use toDataUrl to get a png base64 string, and then put that inside a pdf
https://github.com/react-native-community/react-native-svg/issues/855#issuecomment-538351261

Also here: https://github.com/magicismight/react-native-svg-example/blob/master/examples/Svg.js#L153-L185


when you yotta link, build dependencies are not properly invalidated

(the cmake files aren't touched, because yotta detects that the generated files would be identical, and CMake doesn't seem to pick up the fact the directory has moved)
This could be fixed by reading the source of links and using absolute paths to reference dependency CMakeLists (then the absolute path will change when the module is linked).

http://jira-dev0.cambridge.arm.com/browse/IOTSFW-905


Support for readline behavior

When using the shield cli it would be super nice if we could use readline functionality like when we do backspace or left arrow, it will really goes back instead of adding [[D^[[D^[[D

Have you tried the Web UI?

Not yet. Will try :) Properly less people use CLI than Web UI.


Update workflows to use the latest yggdrasil version 0.5.4

this one of them, please check the other workflows if they need to be fixed too
https://github.com/threefoldtech/tfgrid-sdk-ts/blob/e2580a57b73eccf122b824082e99a124b4611737/.github/workflows/grid_client_tests.yml#L44

I checked the other workflows, and only the grid client test is the one using the tggdrasil.
I updated the workflow with the latest version from their site: https://github.com/yggdrasil-network/yggdrasil-go/releases/tag/v0.5.4.
PR: https://github.com/threefoldtech/tfgrid-sdk-ts/pull/1710
Worfklow tested here: https://github.com/threefoldtech/tfgrid-sdk-ts/actions/runs/7395403959/job/20118505842

Verified,
And the last run of the workflow was successful using the latest yggdrasil version 0.5.4: https://github.com/threefoldtech/tfgrid-sdk-ts/actions/runs/7484206776


Validate custom components service.yaml

Proposed change

We need to be validating custom component services.yaml to avoid things like this from happening. Built in components already get validated by hassfest.
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New integration (thank you!)
[ ] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[X] Code quality improvements to existing code or addition of tests

Checklist

[X] The code change is tested and works locally.
[X] Local tests pass. Your PR cannot be merged unless tests pass
[X] There is no commented out code in this PR.
[X] I have followed the development checklist
[X] The code has been formatted using Black (black --fast homeassistant tests)
[X] Tests have been added to verify that the new code works.

@balloob Can you give me a pointer on what kind of test you'd like to see? Is there an existing mock for custom components?

Patch _load_services_file to return invalid services description, pass a Mock() in for the integration that indicates it's not a built-in integration and make sure that you return an empty dict.

ping


bitcoind 27.0 fails during build/install on aarch64-darwin

Describe the bug
nix profile install nixpkgs#bitcoind fails while compiling bitcoin
The same thing happens with nix profile install nixpkgs#bitcoin
Steps To Reproduce
Steps to reproduce the behavior:

nix profile install nixpkgs#bitcoind

or:

nix profile install nixpkgs#bitcoin

Expected behavior
bitcoind is successfully installed.
Additional context
Here is the output:
error: builder for '/nix/store/x3s1ljagibs78b9fpdxa5bdip8cjaqpl-bitcoind-27.0.drv' failed with exit code 2;
       last 10 log lines:
       > 2024-04-22T04:48:47.088571Z [test] [wallet/bdb.cpp:315] [Verify] Using wallet /private/tmp/nix-build-bitcoind-27.0.drv-0/test_common_Bitcoin Core/c9ed87809f413c480df00f73c4737a803456f06b006825530fe30abe43b74d32/receive-requests-0/wallet.dat
       > 2024-04-22T04:48:47.096066Z [test] [wallet/bdb.cpp:161] [Open] BerkeleyEnvironment::Open: LogDir=/private/tmp/nix-build-bitcoind-27.0.drv-0/test_common_Bitcoin Core/c9ed87809f413c480df00f73c4737a803456f06b006825530fe30abe43b74d32/receive-requests-0/database ErrorFile=/private/tmp/nix-build-bitcoind-27.0.drv-0/test_common_Bitcoin Core/c9ed87809f413c480df00f73c4737a803456f06b006825530fe30abe43b74d32/receive-requests-0/db.log
       > 2024-04-22T04:48:47.096397Z [test] [wallet/bdb.cpp:189] [Open] BerkeleyEnvironment::Open: Error -30974 opening database environment: DB_RUNRECOVERY: Fatal error, run database recovery
       > make[3]: *** [Makefile:22947: wallet/test/wallet_tests.cpp.test] Error 1
       > make[3]: Leaving directory '/private/tmp/nix-build-bitcoind-27.0.drv-0/bitcoin-27.0/src'
       > make[2]: *** [Makefile:20856: check-am] Error 2
       > make[2]: Leaving directory '/private/tmp/nix-build-bitcoind-27.0.drv-0/bitcoin-27.0/src'
       > make[1]: *** [Makefile:20521: check-recursive] Error 1
       > make[1]: Leaving directory '/private/tmp/nix-build-bitcoind-27.0.drv-0/bitcoin-27.0/src'
       > make: *** [Makefile:813: check-recursive] Error 1
       For full logs, run 'nix log /nix/store/x3s1ljagibs78b9fpdxa5bdip8cjaqpl-bitcoind-27.0.drv'.

Notify maintainers
@prusnak @fanquake @roconnor
Metadata
 - system: `"aarch64-darwin"`
 - host os: `Darwin 22.6.0, macOS 13.6.6`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.18.1`
 - channels(root): `"nixpkgs"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

Fix in https://github.com/NixOS/nixpkgs/pull/306043

@Thanks @prusnak
On the same system that I encountered the error on, I checkout out the master branch with your fix and then ran:
nix profile install .#bitcoind

The installation worked and bitcoind is runnable. I'll be doing more testing later today/tomorrow.


Add BackgroundCommand.communicate()

Ease blocking interaction with a BackgroundCommand object, in the style of Popen.communicate()
Tested on paramiko and local target, but not android.

@setrofim I'm not sure to understand, communicate() is a new method of BackgroundCommand, and is implemented by all its subclasses. ConnectionBase is left untouched in the operation.

No worries, GitHub is indeed not so smart in its folding. It only shows the previous "def ..." line in the hunk header, instead of doing something based on indentation

There are a few things to cleanup (docstring etc) so if you are happy with the API as it stands I will proceed with:

adding docstrings
renaming asyn.parallel into asyn.concurrently
remove the .concurrent attribute, as it is not that useful in practice I think (it allows running the function concurrently, only blocking to get the result when it is awaited. The "normal" couroutine behavior is to not do anything until the return value is awaited). asyn.concurrently is usually better as it makes it obvious what runs concurrently and avoid any kind of "task leak".


Better to keep AWS CLI version aligned

What would you like to be added:
Keep the AWS CLI version aligned.
Why is this needed:
Recently introduced AWS CLI v2 into build scripts at #1066 and fix the path issue later on #1102.
However, there are still 2 more scripts will try to setup AWS CLI v1 if user try to run log-collector-script with SSM,

https://github.com/awslabs/amazon-eks-ami/blob/v20221112/log-collector-script/linux/eks-ssm-content.json#L27-L36
https://github.com/awslabs/amazon-eks-ami/blob/v20221112/log-collector-script/windows/eks-ssm-content.json#L25-L34

Which might cause inconsistent AWS CLI output if there's some $PATH order issue.
After checking commit history, the scripts were introduced 3 years ago

https://github.com/awslabs/amazon-eks-ami/pull/354/files

Discussion:

Do we need to install AWS CLI if we had already have it installed while baking AMI? maybe we should remove it?

Do we need to install AWS CLI if we had already have it installed while baking AMI? maybe we should remove it?

The log collector script is intended to be minimally useful on custom AMI's, which may not already have the AWS CLI. Feel free to PR the script to install v2 instead of v1.

Will do 👍


core/blobSupport

This is intended to make it in to the 2.11 release branch, do not merge it to 2.10 pls

This is intended to make it in to the 2.11 release branch, do not merge it to 2.10 pls

I merged changes in release/2.11 into this branch to get the CI building again


Using Volumes with 2019

What is the suggested way to use a volume at path: /var/opt/mssql/.  Since 2019 operates as a non-root user by default I am getting the following error:
2019-12-07 21:52:16.89 Server      Setup step is copying system data file 'C:\templatedata\master.mdf' to '/var/opt/mssql/data/master.mdf'.
2019-12-07 21:52:16.94 Server      ERROR: Setup FAILED copying system data file 'C:\templatedata\master.mdf' to '/var/opt/mssql/data/master.mdf':  2(The system cannot find the file specified.)
ERROR: BootstrapSystemDataDirectories() failure (HRESULT 0x80070002)

Compose file is like so:
version: "3.7"
services:
  sqlserver_2019:
    image: mcr.microsoft.com/mssql/server:2019-GA-ubuntu-16.04
    hostname: sqlserver_2019
    ports:
      - 1433:1433
    environment:
      - ACCEPT_EULA=Y
      - MSSQL_SA_PASSWORD=YourStrong!Passw0rd
    volumes:
      - type: volume
        source: data
        target: /var/opt/mssql/data
volumes:
  data:
    external:
      name: sqlserver_2019

Hello,
same issue here,
Any news?

Same here

Same issue here.

What is the suggested way to use a volume at path: /var/opt/mssql/. Since 2019 operates as a non-root user by default I am getting the following error:
2019-12-07 21:52:16.89 Server      Setup step is copying system data file 'C:\templatedata\master.mdf' to '/var/opt/mssql/data/master.mdf'.
2019-12-07 21:52:16.94 Server      ERROR: Setup FAILED copying system data file 'C:\templatedata\master.mdf' to '/var/opt/mssql/data/master.mdf':  2(The system cannot find the file specified.)
ERROR: BootstrapSystemDataDirectories() failure (HRESULT 0x80070002)

Compose file is like so:
version: "3.7"
services:
  sqlserver_2019:
    image: mcr.microsoft.com/mssql/server:2019-GA-ubuntu-16.04
    hostname: sqlserver_2019
    ports:
      - 1433:1433
    environment:
      - ACCEPT_EULA=Y
      - MSSQL_SA_PASSWORD=YourStrong!Passw0rd
    volumes:
      - type: volume
        source: data
        target: /var/opt/mssql/data
volumes:
  data:
    external:
      name: sqlserver_2019

Hello there,
I think there was a little problem when declaring the volume name at the end. Here you have the YML syntax:
version: "3.7"
services:
  sqlserver_2019:
    image: mcr.microsoft.com/mssql/server:2019-CU2-ubuntu-16.04
    hostname: sqlserver_2019
    ports:
      - 1433:1433
    environment:
      - ACCEPT_EULA=Y
      - MSSQL_SA_PASSWORD=YourStrong!Passw0rd
    volumes:
      - type: volume
        source: data
        target: /var/opt/mssql
volumes:
  data:

Testing the volume by creating a new database called "VolTest":
[dba mastery] $  sqlcmd -S localhost -USA -P'YourStrong!Passw0rd'
1> create database VolTest;
2> go
1> select name from sys.databases;
2> go
name
------------------------------
master
tempdb
model
msdb
VolTest

A few points to remark here:

You don't need to use docker-compose to create a single container, it works best for a compose solution :)
The /var/opt/mssql is the path where both MDF and LDF files are created, you can create one volume for each one if needed.

Cheers,

What if you don't want to use a named volume? I don't really like Docker telling me where my data should be, I like my data to be in the same folder where I have the project. So, in my case:
docker -v
Docker version 19.03.11, build 42e35e61f3

docker-compose.yml:
version: '3.6'
services:

  db:
    restart: always
    image: mcr.microsoft.com/mssql/server:latest
    environment:
      - ACCEPT_EULA=Y
      - SA_PASSWORD=BLAH
      - MSSQL_PID=Express
    ports:
        - 1433:1433
    volumes:
      - ./data:/var/opt/mssql/data
    logging:
      driver: "json-file"
      options:
        max-size: "500k"
        max-file: "5"

I get this same dreaded error, so I can't start up my DB:
db_1  | 2020-06-02 02:18:32.94 Server      Setup step is copying system data file 'C:\templatedata\master.mdf' to '/var/opt/mssql/data/master.mdf'.
2020-06-02 02:18:33.14 Server      ERROR: Setup FAILED copying system data file 'C:\templatedata\master.mdf' to '/var/opt/mssql/data/master.mdf':  2(The system cannot find the file specified.)
db_1  | ERROR: BootstrapSystemDataDirectories() failure (HRESULT 0x80070002)

What is the solution then?

Well I found the solution: Run as root -> https://github.com/microsoft/mssql-docker/issues/13#issuecomment-641904197

I found the solution in spanish
https://www.eiximenis.dev/posts/2020-06-26-sql-server-docker-no-se-ejecuta-en-root/

Okay, so Marcozm's link worked for me.
Basically, I have a Dockerfile that changes the user to root by running this command "USER root" and now everything works fine.
By default, I believe it runs under the user "mssql".

Unfortunately sometimes running a container as root is not an option - for example in an Openshift or Kubernetes cluster. For anyone googling and stumbling on this issue like I did - it's a permissions issue. By default /var/opt/mssql/data  and /var/opt/sqlsever don't the necessary permissions and owners.
Since I was using a custom Dockerfile anyway, adding the following lines solved my issue. Some of them might be redundant / unnecessary though.
USER root

RUN mkdir -p -m 777 /var/opt/mssql && chgrp -R 0 /var/opt/mssql
RUN mkdir -p -m 777 /var/opt/mssql/data && chgrp -R 0 /var/opt/mssql/data
RUN mkdir -p -m 777 /var/opt/sqlserver && chgrp -R 0 /var/opt/sqlserver

RUN chmod -R 777 /var/opt/mssql /var/opt/sqlserver /var/opt/mssql/data
RUN chown -R mssql:0 /var/opt/mssql /var/opt/sqlserver /var/opt/mssql/data

USER mssql

SQL Server 2019 by default runs as a non root container.  Here is a sample dockerfile where we create a custom container https://github.com/microsoft/mssql-docker/blob/master/linux/preview/SLES/dockerfile you can see we give permissions and ownership to the mssql user and group ( which is the default user/group) that runs the SQL 2019 containers. Recommendation is always to run the container as non root.

I found the solution in spanish
https://www.eiximenis.dev/posts/2020-06-26-sql-server-docker-no-se-ejecuta-en-root/

This worked for me

Just add user: root
  sql:
    image: "mcr.microsoft.com/mssql/server"
    user: root
    environment:
      - SA_PASSWORD=${SQLPASS}
      - ACCEPT_EULA="Y"
    ports:
      - 1433:1433
    volumes:
      - /home/moti/polytex/pm8/data:/var/opt/mssql/data
      - /home/moti/polytex/pm8/log:/var/opt/mssql/log
      - /home/moti/polytex/pm8/secrets:/var/opt/mssql/secrets

Do I have to create the 'mssql' user explicitly in the host?

@nicholasyin the user sa is built in.

I solved this problem by making the entire /var/opt/mssql folder a volume instead of just the data subdirectory.

If you need to run your container as non root user, see what is your current user on system with coomand:
id
uid=502(myuser) gid=20(mygroup)
Because in container default user is mssql with guid 10001
then issue commnad
sudo chown -R 10001:mygroup /var/opt/mssql/
in my case:
sudo chown -R 10001:20 /var/opt/mssql/
in microsoft docs is written: chown -R 10001:0 /var/opt/mssql/ but it is not working unless you want to run ms sserver as a root with commnad sudo docker run .... (not recommanded)
And works with it! (MacOs)

With SELinux turned on, you can use those free commands to allow container to have access to volume:

sudo chcon -Rh svirt_sandbox_file_t /path/to/volume/directory
sudo chmod -R 777 /path/to/volume/directory
sudo stat -c %a /patch/to/volume/directory

It is good to keep those always at hand.

On Windows, using the root user does not work.
I get this error
This program has encountered a fatal error and cannot continue running at Mon Jan 16 11:55:07 2023
The following diagnostic information is available:

         Reason: 0x00000006
        Message: Termination of \SystemRoot\system32\AppLoader.exe was due to fatal error 0xC0000001
        Address: 0x3fffa5888b7f
    Stack Trace:
                 file://package4/windows/system32/sqlpal.dll+0x000000000000E16F
                 file://package4/windows/system32/sqlpal.dll+0x000000000000B879
                 file://package4/windows/system32/sqlpal.dll+0x0000000000088DEF
                 file://package4/windows/system32/sqlpal.dll+0x0000000000088B7F
                 file://package4/windows/system32/sqlpal.dll+0x0000000000086FE8
                 file://package4/windows/system32/sqlpal.dll+0x0000000000003D1F
                 file://package4/windows/system32/sqlpal.dll+0x0000000000202428
                 file:///windows/system32/AppLoader.exe+0x000000000000371F
                 file:///windows/system32/AppLoader.exe+0x0000000000003869
                 file:///windows/system32/AppLoader.exe+0x0000000000003889
                 file:///windows/system32/AppLoader.exe+0x000000000000A8E8
                 file:///Windows/SYSTEM32/KERNEL32.DLL+0x0000000000014414
                 file:///windows/system32/ntdll.dll+0x0000000000075541
        Modules:
                 file://package4/windows/system32/sqlpal.dll=E59EEA3917F70A1E2DB0DFD3370D69501
                 file:///windows/system32/AppLoader.exe=D7AA0798438350C25E1958B896A98C9E1
                 file:///Windows/SYSTEM32/KERNEL32.DLL=C715300FB2664729A6126A3F591E6F302
                 file:///windows/system32/ntdll.dll=45137AA3F9814512B3123991067EEE6E2
        Process: 10 - sqlservr
         Thread: 40 (application thread 0x74)
    Instance Id: c96842c8-bfbd-405f-8a90-02b24e06f42d
       Crash Id: b5185085-14b7-4590-9ce1-8c0b15d92d06
    Build stamp: f708684a2cfcb51177273c54f975ed8c62029cbc89aa962bf7d6b956f01a0c27
   Distribution: Ubuntu 20.04.5 LTS
     Processors: 8
   Total Memory:<PHONE_NUMBER> bytes
      Timestamp: Mon Jan 16 11:55:07 2023

When using this docker compose
version: '3.9'
services:
  mssql:
    image: mcr.microsoft.com/mssql/server:2019-latest
    user: root
    ports:
      - 1433:1433
    environment:
      - ACCEPT_EULA=Y
      - MSSQL_SA_PASSWORD=<YourStrong!Passw0rd>
    volumes:
      - ./mssql:/var/opt/mssql

I had to map the /var/opt/mssql/data directory directly to make it work:
version: '3.9'
services:
  mssql:
    image: mcr.microsoft.com/mssql/server:2019-latest
    ports:
      - 1435:1433
    environment:
      - ACCEPT_EULA=Y
      - MSSQL_SA_PASSWORD=<YourStrong!Passw0rd>
    volumes:
      - ./mssqldata:/var/opt/mssql/data

Comment peut on connaitre la version de sql server sur laquelle on travaille par un script sql ?

Peeps - it works out of the box.
MSSql needs three folders to get this working. Here's a picture of my docker instance:

So there's two ways to do this.

just define the "parent" folder (aka /var/opt/mssql) and this will be stored in a single volume
define all three folders data / log / secrets (aka /var/opt/mssql/data, /var/opt/mssql/log ...) and each one has it's own volume

I prefer to keep things simple, so here's mine:
version: '3.9'
services:
  mssql:
    image: mcr.microsoft.com/mssql/server ## Latest SQL SERVER. use whatever you want/need
    ports:
      - 1433:1433
    environment:
      - ACCEPT_EULA=Y
      - MSSQL_SA_PASSWORD=<YourStrong!Passw0rd>
    volumes:
      - mssql-db-vol:/var/opt/mssql
volumes:
  mssql-db-vol:

notice how i DID NOT specify user: root? Good! Notice how I used a simple docker volume (which it controls where the data is stored, etc) with the volumes: section at the bottom of the file and then did  - mssql-db-vol:/var/opt/mssql to specify the link.
You could also use "bind mounts" which is a direct folder on your computer like (note: i've not tested this .. i'm just trying to use my memory for how this might work):
version: '3.9'
services:
  mssql:
    image: mcr.microsoft.com/mssql/server ## Latest SQL SERVER. use whatever you want/need
    ports:
      - 1433:1433
    environment:
      - ACCEPT_EULA=Y
      - MSSQL_SA_PASSWORD=<YourStrong!Passw0rd>
    volumes:
      - ${PWD}:/var/opt/mssql

Notice how there is no volumes: section? and the ${PWD} will be the CURRENT DIRECTORY where you run the docker compose command, from.
Enjoy peeps!


Add sounds from 2023.11.09 folder

I have a series of dated folders in which I collected clips, but kept procrastinating.  This is one of them.


unknown users in Server Leaderboard

📢 Bug reported by Totopizza<PHONE_NUMBER>55561729)
What is the title of your bugreport?
unknown users in Server Leaderboard
How can you reproduce the bug?
https://cdn.discordapp.com/attachments/408470179052257292/588655150705541120/unknown.png
Which result would normally have to be?
https://cdn.discordapp.com/attachments/408470179052257292/588655300194598916/unknown.png
Comments:
❌ Nickkel: Couldn't be reproduced
❗✅ Dadi: could be reproduced (forwarded to monkeyyy11)

Could not reproduce it again :/


Can't using "Mo-js" in framework7 :confused:

This is a :

[x] bug
I can't using http://mojs.io/ in framework7 :disappointed:
I dont know why? :confused:
Pleace help me. :face_with_head_bandage:

Anybody using mo.js with framework7?

Hi @cuduy197!
Could you give me more info for the issue? Do you see any error?

Nothing error when I import mo-is in
but mo-js does work on my app (using framework7 with vue + webpack2) ?
    document.addEventListener('click', function(e) {       const coords = {         x: e.pageX,         y: e.pageY       };       burst         .tune(coords)         .replay();     });

Thanks for the information. It is hard to say what's wrong right now. If you can create a simple demo on CodePen or JsBin would be great.

@cuduy197, mo.js works ok with Vue.js (so it should work ok with Framework7 too). Here a simple port of an example from the tutorials ported to Vue.js.
Use Vue.js events (like: <button @click="yourMethod"></button>. Vue.js is data-driven, so it's better to think in the Vue's way when porting mo.js code to an app. Better start with something simple like porting an tutorial's example to Vue.js.

@eltonmesquita nice one! So it definitely works with Vue.js wondering what's up with Framework7.

@eltonmesquita did you try with Framework7 ?
I add mo-js like you in component, but :confused:  not thing happened ?

Thanks!
I'll have to try it with framework 7 but it should work as it's running inside a webview. I'll look at it later.

@cuduy197 was right, nothing is shown when running mojs in framework7. But it's a simple z-index issue. I don't know why, but the app always overlays the mojs elements. The simple fix is to just add z-index: 0 to the app container and everything works as it should. Hope it solves your issues @cuduy197 .

Awesome, this makes sense. You can have access to the DOM element of the Burst (burst.el) so can increase zIndex by:
burst.el.style.zIndex = 2;

Should work just fine. Thanks for your help @eltonmesquita!

Glad I can help @legomushroom!
Weirdly enough increasing the mojs element z-indexmakes no difference, even though there's no z-index on the app div and mojs elements are added after the app element. The only fix seems to be declaring a low z-index directly to the app element.
It makes no sense to me as this shouldn't happen. Maybe it's a webview issue? Or something related to transform/Css 3d context and Safari Mobile?

Humm, that's strange, are you setting it in dev tools or can you confirm that it is set in dev tools?

Tried both and the results are the same, the elements still get under the app container. I'm trying to run this on Android, but it's a bit trickier than iOS.

@eltonmesquita Thanks you so much, but where to add z-index: 0 ?
Can you give me example ?

burst.el.style.zIndex = 99999;
WILL FIX THIX :tada:


Improve ordering for scheduler queues and add debug comamnd

It also adds a task-manager:queues command, so that you can run:
bin/console messenger:consume $(bin/console task-manager:queues)


Is there a way to keep the server starting without getting a catalog?

In registry-ui v0.10.3, I think the server tries to get the catalog from the registry the moment it starts, and if it fails, the server stops working.
Is there any way to make the server continue to start itself even if access to the registry fails at the moment of startup?
The assumption is that the system will start with no data, expecting it to be acquired at the next acquisition timing.
The above question was asked because we wanted to let the service continue to start up even if it was not yet ready to access the registry when we tried to start it up in the following order.

Start registry.
Start token authentication server.
Start registry-ui
Start reverse proxy.

I rely on translation tools. I'm sorry if there is a strange sentence.

Yes, it tries to get the catalog the moment it starts. It can't get it instantly so this runs in background. However, if it fails immediately because the registry is not available it fails to start completely.
It is possible to add a config option to not fail app/web server startup with unreachable registry and then try to do that as usual - refresh the catalog on interval.


Upgrade commons-codec:commons-codec from 1.11 to 1.15

Task Description
We need to upgrade commons-codec:commons-codec from 1.11 to 1.15.
Tasks
The following tasks will need to be carried out:

[ ] Update the version in the strongbox-parent project.
[ ] Apply any required code changes, or dependency excludes.
[ ] Test and confirm that there are no issues with the core projects (strongbox, strongbox-web-integration-tests, etc) after upgrading.

Task Relationships
This task:

Relates to: #1847

Help

Check our article on how to upgrade dependencies and plugins.
Our chat
Points of contact:

@carlspring
@sbespalov
@steve-todorov

I will look into this upgrade.

@DLambros91 are you from GHC? :)

@steve-todorov I am not, just an interested open-source contributor. If being from GHC is a requirement, you may ignore the previous message

Thanks for your reply! The GraceHopperOSD labeled issues are only for participants in the Grace Hopper Celebration. If nobody takes this task today you can work on it tomorrow. :)

Hello! I am with GHC. I will take this issue.

Great to have you, @steffisbootcampdrive!
Feel free to pick it up and good luck! :)

Hello, just checking in. I looked through all of the parent folders and files and I was unable to locate a specifically commons-codec version 1.11 to update and test. I did locate commons-beanutils. Commons-codec may be a dependency located within another dependency package, perhaps commons-beanutils, I see another person has https://github.com/strongbox/strongbox/issues/1847. Would it be alright for me to work on this one as I have located this? Is there any additional information specifically for commons-codec available regarding this project specifically, I will search for this, but any assistance in locating it within the code to then update would be appreciated. Also, what is the deadline for this?

Hi @steffisbootcampdrive ,
You are welcome to pick this up. What I'd do, if I were you, would be to do a:
mvn dependency:tree > dep.tree

@steffisbootcampdrive ,
Any updates on this?

commons-codec is a transitive dependency only brought in from org.apache.httpcomponents:httpclient. It is currently (with the httpclient already upgraded to 4.5.13) at version 1.13.

Thanks! Could you please try overriding the one from the org.apache.httpcomponents:httpclient and use the latest version of the commons-codec? I remember there was some vulnerability being reported by snyk.io which is why I raised this, (if I recall correctly, and it was over a month ago).

I can do that if you really want to, but apart from Snyk listing this as a minor vulnerability of Information Exposure in the Base32 codec, this is already fixed in 1.13.

Okay. In this case we can close this task. Thanks for looking into it! :)

@flaix ,
Feel free to join our chat channel where we could discuss some other tasks that you might be interested in help out with.


General improvements

Merging this PR will:

Remove accidentally staged artifacts
Configure gitignore
Upgrade build system to .NET Core
Replace writeline / readline dialogs with a single JSON config
Fix mixed async / blocking methods
Upgrade libraries to supported versions

wrong pr


Can't find VLC when VLC is installed

I am running Linuxmint 21.3 virginia, I have the mod and VLC installed but the mod can't find it

Is VLC installed as a flatpak?

VLC was not, but prism was. after I reinstalled prism as a native application it all worked fine


refactor to template-only components

Having a JavaScript class, which extends from ember-bootstrap/bs-form/element/control, does not seem to be necessary as all. Components does not seem using the backing JavaScript class at all. The components do not use this in their templates.
Closes #348

Tests for Ember 3.16 is failing. It is failing on master as well. I feel it is time to drop support rather investigating what is going wrong.


External links opening in a new tab

Issue: Solving issue #20
External links for GitHub and Discord on the website currently open in the same browser tab, which can disrupt the user experience by navigating away from the site.
Changes Made:
Updated the HTML anchor tags for GitHub and Discord icons to include target="_blank" to ensure that these links open in a new browser tab.
Added rel="noopener noreferrer" to improve security and performance when opening links in a new tab.
Affected Files:
Modified the HTML file containing the links to GitHub and Discord.
Testing:
Clicked on the GitHub icon and verified that it opens in a new tab.
Clicked on the Discord icon and verified that it opens in a new tab.
Additional Notes:
These changes ensure that users can easily return to the website after visiting GitHub or Discord.

@Shinyzenith could you please review this pr and provide me feedback incase there are any changes to me made?

Thank you for your contribution.


prevent SOCKET leak in BindListenPort()

the call to CloseSocket() is placed after the WSAGetLastError(), because
a CloseSocket() can trigger an error also, which we don't want for the
logging in this two cases

needs to be rebased after cherry-picking
43f510d37d680ca4347878d2fb6f8b97b54e7611
eaedb59e0570558a528eac52f7fd89639911496e

all good now.


cgltf_image::buffer_view::data is null

I'm loading some glb files and cgltf_image::buffer_view::data is null:

I'm able to get the data by doing:
void* data = texture->image->buffer_view->buffer->data + texture->image->buffer_view->offset;

But this feels like not the intended way of doing this.

Ah, I saw that comment, but still miss-interpreted the purpose of the cgltf_buffer_view::data field. I feel that the naming is a bit confusing, but understand that altering it is a breaking change, so is not possible.
But I did actually end up writing a function identical to cgltf_buffer_view_data() myself, so maybe exposing it is a good idea.


Leaderboard: Fix code benchmarks

Currently CoIR and MTEB(Code) do not work with the new leaderboard, as they contain a completely different set of languages from the rest, and this confuses the Gradio elements that have been told that the only list of choices is the languages that are present in the Multilingual benchmark.
This issue is also a question on whether this is something we would like to have on the main leaderboard.
@KennethEnevoldsen @Muennighoff @isaac-chung

I think we should support them as well.
Can we aggregate all the languages across all benchmarks, or is it more complicated than that?

I'll look into it.

Turns out it was pretty easy to fix: #1441


Invoking Neo.Contract.IsPayable fails

I ran
my_hash = GetExecutingScriptHash()
Notify(my_hash.IsPayable)

and consistently received a warning message:
[W 181004 11:17:07 InteropService:537] method Neo.Contract.GetIsPayable not found

Just installed neo-boa version 0.5.4 and this works as expected now


[Logging v2] - certificate based options are not properly configured in the ClusterOutput of the rancher-logging app

What kind of request is this:
bug
Steps to reproduce:

Configure Elasticsearch/Kibana backend with certificate based auth.

https://www.elastic.co/guide/en/elastic-stack-get-started/current/get-started-docker.html#get-started-docker-tls

Deploy the rancher-logging chart from the Dashboard setting using the username, password and certificates.

Result:

The ClusterOutput generated by the install doesn't include anything about the username, password or certificates.

  spec:
    elasticsearch:
      host: <host>
      index_name: test3
      port: 9200
      scheme: https

Other details that may be helpful:
If we navigate to the installed app in the Dashboard I see the username/password and certs shown in the Chart Options form.

The secret rancher-logging-elasticsearch is created for certs and password in the cattle-logging-system namespace with keys: client_cert, client_key, client_key_pass, password
Environment information

Rancher version: master-head 08/10/2020 9b4685f29
Installation option: HA k8s

Related:
#28295
#27935

Will re-test when the new UI refactor is made for Apps v2, part of Dashboard testing.


Slack: Use display_name instead of name, bidirectional

Referencing https://github.com/42wim/matterircd/issues/150
Presently, matterircd displays the users's name instead of display name in the name list, as well as in "@" mentions name = myname, should use display_name = mycoolnickname
e.g. on an '@' mention from matterircd to slack:
@myname, should use @mycoolnickname
When a user on the Slack interface (web and app) "@" mentions another user, matterircd displays the user's id, e.g. <@W012A3CDE> message
Reference: https://api.slack.com/methods/users.list

@christian-blades-cb


Skia4Delphi support

I've added a beta version of this new engine:
https://github.com/viniciusfbb/skia4delphi
Some svg files are not correctly rendered (for example cowboy and Butterfly...)
FixedColor don't work, GrayScale is not supported and the rendering area is not "proportional"...
I've added the new factory to the Benchmark and using a complex svg like lion.svg the engine is very fast...
Feedbacks are welcome...

Sounds good.  A drawback is that skia svg does not support the style and class attribute https://bugs.chromium.org/p/skia/issues/detail?id=12251&q=style&can=2.  It does not look like such support is coming soon (low priority).

There are appear to be a number of serious issues in skia svg support (see https://bugs.chromium.org/p/skia/issues/list?q=svg&can=2&colspec=ID Type Status Priority M Area Owner Summary Opened) for details:

improper (locale dependent) parsing of floats
rendering issues
lack of style/class support

Also, although the ski svg includes accessing the svg elements via DOM. the Delphi wrapper does not include the relevant parts.
My suggestion is to leave out of the master for now (create a branch) and get back to it when it is more mature.   Other parts of skia are great but not the svg support.

Yes, I agree with you, I will revert to previous version of the repo and make a branch... probably at the end of the week... Now I'm very busy...

Today I've committed a new version compatible with SKIA4Delphi 3.0 but many SVG files are not correctly rendered if includes "DOCTYPE" section, like explained here

Problems for other set of icons, here

Updated support to SKIA4Delphi 3.0.3 and fixed some rendering problems.

Does Skia4Delphi cause any deployment dependencies (for Windows and other platforms)?

Actually SKIA4Delphi support is available only for VCL version of SVGIconImageList. You must deploy the proper .dll ok SKIA with your project, as explained in SKIA project.

thanks, was hoping they could statically link somehow

As in .NET one uses Microsoft C++ for such cases, I'd expect to be able to use C++Builder to make some shim for Delphi to talk to (to handle the name manging etc. internally automatically)

@carloBarazzetta Improved Ethea compatibility by adding GrayScale property to TSkSvgBrush: v3.1.0

Thakyou @viniciusfbb! We need only a final implementation, ApplyFixedColorToRootOnly, very useful as explained here:
https://github.com/EtheaDev/SVGIconImageList/issues/161
In TSkSvgBrush could be ApplyOverrideColorToRootOnly...

Thakyou @viniciusfbb! We need only a final implementation, ApplyFixedColorToRootOnly, very useful as explained here: #161 In TSkSvgBrush could be ApplyOverrideColorToRootOnly...

Right! In the next release we will add this.

I have some problem with SKIA4Delphi in FMX: I have implemented the TFmxImageSkiaSVG class in the FMX.ImageSkiaSVG unit to render an SVG icon in FMX using SKIA, using the Bitmap.SkiaDraw helper... I don't know why the bitmaps are never visible using my SVGIconImageList component (SVGIconImageListDemoFMX.dpr demo) and is only visible when clicking on the next icon using my TSVGIconImage component (SVGIconImageDemoFMX demo).
If someone can help me, remember to activate {$ DEFINE Skia_SVGEngine} in SVGIconImageList.inc instead of Image32_SVGEngine... to activate SKIA support.
I'm not an FMX platform expert and I'm probably missing out something...
Thank's

@carloBarazzetta I looked at the code and I suggest some changes:
1.
Change the line:
https://github.com/EtheaDev/SVGIconImageList/blob/e451a1ded23d48ba0bd3d80335cd36d886f1af08/Source/FMX.ImageSkiaSVG.pas#L109
to:
  FSvg.Source := Source;

2.
To improve performance, remove the line:
https://github.com/EtheaDev/SVGIconImageList/blob/e451a1ded23d48ba0bd3d80335cd36d886f1af08/Source/FMX.ImageSkiaSVG.pas#L152
And remove the , False from the line:
https://github.com/EtheaDev/SVGIconImageList/blob/e451a1ded23d48ba0bd3d80335cd36d886f1af08/Source/FMX.ImageSkiaSVG.pas#L155
3.
At line:
https://github.com/EtheaDev/SVGIconImageList/blob/e451a1ded23d48ba0bd3d80335cd36d886f1af08/Source/FMX.SVGIconImageList.pas#L259-L265
I suggest to change the block (BeginScene, Clear and EndScene) to each implementation of "PaintToBitmap" because the "PaintToBitmap" method of Skia4Delphi svg engine, can't use it.
4.
At line:
https://github.com/EtheaDev/SVGIconImageList/blob/e451a1ded23d48ba0bd3d80335cd36d886f1af08/Source/FMX.SVGIconImageList.pas#L293-L294
To improve performance, I suggest change it to:
  LBitmap.SetSize(LBitmapWidth, LBitmapHeight);

Thank you very much Vinicius, now works perfectly!

@carloBarazzetta, we decided not to directly add ApplyFixedColorToRootOnly to our main code, as our idea is to keep the components short as much as possible. However in our new version released today, we improved the TSkSvgBrush code to allow for more customizations.
I made an example code of how to create a descendant of TSkSvgBrush with this new property. The code works for both, FMX and Vcl:
unit Unit1;

interface

uses
  System.UITypes, Skia, Skia.FMX; // Or Skia.Vcl in case of Vcl

type
  { TSkSvgBrushEx }

  TSkSvgBrushEx = class(TSkSvgBrush)
  strict private
    FOverrideRootColor: TAlphaColor;
    procedure SetOverrideRootColor(const AValue: TAlphaColor);
  strict protected
    procedure DoAssign(ASource: TSkSvgBrush); override;
    function MakeDOM: ISkSVGDOM; override;
  public
    function Equals(AObject: TObject): Boolean; override;
  published
    property OverrideRootColor: TAlphaColor read FOverrideRootColor write SetOverrideRootColor default TAlphaColors.Null;
  end;

implementation

uses
  System.SysUtils;

{$IF CompilerVersion < 29}
type
  { TSkFormatSettingsHelper }

  TSkFormatSettingsHelper = record helper for TFormatSettings
  strict private
    class var
      FInvariant: TFormatSettings;
    class constructor Create;
  public
    class property Invariant: TFormatSettings read FInvariant;
  end;

class constructor TSkFormatSettingsHelper.Create;
begin
  FInvariant := TFormatSettings.Create('en-US');
end;
{$ENDIF}

{ TSkSvgBrushEx }

procedure TSkSvgBrushEx.DoAssign(ASource: TSkSvgBrush);
begin
  if ASource is TSkSvgBrushEx then
    FOverrideRootColor := TSkSvgBrushEx(ASource).FOverrideRootColor
  else
    FOverrideRootColor := TAlphaColors.Null;
  inherited;
end;

function TSkSvgBrushEx.Equals(AObject: TObject): Boolean;
begin
  Result := (AObject is TSkSvgBrushEx) and (FOverrideRootColor = TSkSvgBrushEx(AObject).FOverrideRootColor) and inherited;
end;

function TSkSvgBrushEx.MakeDOM: ISkSVGDOM;
var
  LAlphaColorRec: TAlphaColorRec;
  LNewColor: string;
begin
  Result := inherited;
  if FOverrideRootColor <> TAlphaColors.Null then
  begin
    LAlphaColorRec := TAlphaColorRec(FOverrideRootColor);
    LNewColor := Format('rgb(%d,%d,%d)', [LAlphaColorRec.R, LAlphaColorRec.G, LAlphaColorRec.B]);
    if Result.Root.TrySetAttribute('fill', LNewColor) and Result.Root.TrySetAttribute('color', LNewColor) and (LAlphaColorRec.A <> High(LAlphaColorRec.A)) then
      Result.Root.TrySetAttribute('opacity', FloatToStr(LAlphaColorRec.A / High(LAlphaColorRec.A), TFormatSettings.Invariant));
  end;
end;

procedure TSkSvgBrushEx.SetOverrideRootColor(const AValue: TAlphaColor);
begin
  if FOverrideRootColor <> AValue then
  begin
    FOverrideRootColor := AValue;
    RecreateDOM;
    if HasContent then
      DoChanged;
  end;
end;

end.

Thank's to @viniciusfbb I've completed the support for SKIA4Delphi, adding ApplyFixedColorToRootOnly.


SAM demo supporting multiple runtimes + models

Gradio SAM demo offering four prompting modes: point-prompted, box-prompted, mixed point and box prompts, and automatic full image segmentation.
Supports:

PyTorch, ONNX, and TensorRT runtimes
Toggling between l0, l1, l2, xl0, and xl1 models
Experimentation with full image segmentation parameters

Script included to generate ONNX models and TensorRT engines.

Awesome demo!!!


NGen executed for ImageResizer during setup

https://github.com/microsoft/PowerToys/blob/eb15cdde1b57f219e7ae30c7550a22490ac5693d/installer/PowerToysSetup/Product.wxs#L473
This must be removed after porting to .NET Core
Log from 0.29.3 installation
ExecNetFx:  Microsoft (R) CLR Native Image Generator - Version 4.8.4084.0
ExecNetFx:  Copyright (c) Microsoft Corporation.  All rights reserved.
ExecNetFx:  Uninstalling assembly C:\Program Files\PowerToys\modules\ImageResizer\ImageResizer.exe because of an error during compilation: Failed to load the runtime. (Exception from HRESULT: 0x80131700).
ExecNetFx:  Failed to load the runtime. (Exception from HRESULT: 0x80131700)
ExecNetFx:  Error 0xffffffff: Command line returned an error.
ExecNetFx:  failed to execute Ngen command (with error 0xffffffff): C:\WINDOWS\Microsoft.NET\Framework\v4.0.30319\ngen.exe install "C:\Program Files\PowerToys\modules\ImageResizer\ImageResizer.exe", continuing anyway
ExecNetFx:  Microsoft (R) CLR Native Image Generator - Version 4.8.4084.0
ExecNetFx:  Copyright (c) Microsoft Corporation.  All rights reserved.
ExecNetFx:  Uninstalling assembly C:\Program Files\PowerToys\modules\ImageResizer\ImageResizer.exe because of an error during compilation: Failed to load the runtime. (Exception from HRESULT: 0x80131700).
ExecNetFx:  Failed to load the runtime. (Exception from HRESULT: 0x80131700)
ExecNetFx:  Error 0xffffffff: Command line returned an error.
ExecNetFx:  failed to execute Ngen command (with error 0xffffffff): C:\WINDOWS\Microsoft.NET\Framework64\v4.0.30319\ngen.exe install "C:\Program Files\PowerToys\modules\ImageResizer\ImageResizer.exe", continuing anyway
ExecNetFx:  Microsoft (R) CLR Native Image Generator - Version 4.8.4084.0
ExecNetFx:  Copyright (c) Microsoft Corporation.  All rights reserved.
ExecNetFx:  Microsoft (R) CLR Native Image Generator - Version 4.8.4084.0
ExecNetFx:  Copyright (c) Microsoft Corporation.  All rights reserved.

Great news!  This was resolved in the newly released 0.33 version of PowerToys!  Head to https://aka.ms/installpowertoys to grab latest.


supress sourceMapUrl in output file as it is sent as a header.

Fix for Sourcemap error with root files 3.10

thanks


Add an entry point that runs both dispatcher threads.

Partner to #64

Codecov Report

Merging #65 (89f8f99) into master (3852c0d) will decrease coverage by 0.03%.
The diff coverage is 13.15%.

@@            Coverage Diff             @@
##           master      #65      +/-   ##
==========================================
- Coverage   44.92%   44.88%   -0.04%
==========================================
  Files          38       39       +1
  Lines        4474     4491      +17
==========================================
+ Hits         2010     2016       +6
- Misses       2464     2475      +11

Impacted Files
Coverage Δ

assemblyline_core/dispatching/__main__.py
0.00% <0.00%> (ø)

assemblyline_core/dispatching/run_files.py
74.54% <0.00%> (ø)

assemblyline_core/dispatching/run_submissions.py
72.30% <0.00%> (ø)

assemblyline_core/expiry/run_expiry.py
69.30% <0.00%> (ø)

assemblyline_core/ingester/run_ingest.py
78.37% <0.00%> (ø)

assemblyline_core/ingester/run_internal.py
77.14% <0.00%> (ø)

assemblyline_core/workflow/run_workflow.py
0.00% <0.00%> (ø)

assemblyline_core/alerter/processing.py
76.25% <100.00%> (ø)

assemblyline_core/dispatching/client.py
72.93% <100.00%> (+0.91%)
:arrow_up:

... and 3 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 ed3e3b6...89f8f99. Read the comment docs.


replace tempdir with tempfile

Replace tempdir development dependency with tempfile.
Fixes https://github.com/sigstore/sigstore-rs/issues/23
Note well, depends on https://github.com/sigstore/sigstore-rs/pull/24

I'll merge this PR once https://github.com/sigstore/sigstore-rs/pull/24 is into main


Example Commit Mode w/ Configuration: Closing comma in JSON leads to parsing error when running github action

The closing comma after the label_extractor array is causing a parsing error when running an action with this setup taken from your Readme Examples. Testet on Enterprise Github 3.12
jobs:
  release:
    if: startsWith(github.ref, 'refs/tags/')
    runs-on: ubuntu-latest
    steps:
      - name: Build Changelog
        uses: mikepenz/release-changelog-builder-action@v5
        with:
          mode: "COMMIT"
          configurationJson: |
            {
              "template": "#{{CHANGELOG}}",
              "categories": [
                {
                    "title": "## Feature",
                    "labels": ["feat", "feature"]
                },
                {
                    "title": "## Fix",
                    "labels": ["fix", "bug"]
                },
                {
                    "title": "## Other",
                    "labels": []
                }
              ],
              "label_extractor": [
                {
                  "pattern": "^(build|chore|ci|docs|feat|fix|perf|refactor|revert|style|test){1}(\\([\\w\\-\\.]+\\))?(!)?: ([\\w ])+([\\s\\S]*)",
                  "target": "$1"
                }
              ],
            }
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}```

Thank you!

Thank you for the report. We will address the README.


Migrate off of the textDocumentNotebook API proposal

👋  Hi, I work on APIs for VS Code
The VS Code Dot Net Interactive notebooks extension currently has access to the textDocumentNotebook api proposal. We do not plan to finalize this proposal and instead would like to remove to simplify our code. See https://github.com/microsoft/vscode/issues/147414 for details
Can you please check to see if your extension is currently using this API. If it is, please look into migrating off of it so we can remove this API from our code

We're not using the textDocumentNotebook APIs and they can be safely removed.


Bug

https://github.com/researchmm/CKDN/blob/d5de075cea770b10d6354f011337008c8cf76787/ckdn.py#L205
The target should be removed.

Thanks for your suggestions. The 'target' here is not used in the following and should be removed.


Is the application sufficiently accessible?

While the intent is for the application to be accessible, and have followed the guidelines, it needs to be thoroughly tested. Specifically around the handling of the popup menus in the banner.

#21


Shortcut for unsubscribe

It’d be cool to have a keyboard shortcut for unsubscribing from the currently shown issue.

Browser
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/41.0.2272.118 Safari/537.36

@kkaefer is m for mute what you want?

Yep.

It's not listed on the Keyboard Shortcuts page though

Yeah, I think this is what bit me in https://github.com/bkeepers/github-notifications/issues/81, which I'd forgotten about. Thanks for the bump!


chore(maintenance): migrate idempotency utility to biome

Summary
Changes

Please provide a summary of what's being changed

This PR migrates the linting and formatting of the Idempotency utility to use Biome instead of a combination of Eslint and Prettier.

Please add the issue number below, if no issue is present the PR might get blocked and not be reviewed

Issue number: closes #2796

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Review & merge this after #2791.