fixed release-prerequisites

Closes/fixes/resolves issue(s)?
What was added/changed/fixed?
Related issue(s)? [Optional]
Others [Optional]
Checklist (after created PR)

[ ] Added reviewers
[ ] Added assignee(s)
[ ] Added label(s)
[ ] Added to project
[ ] Added to milestone

Thinking about it, this might actually be more suitable to include when we have our next release, just so that it can be tested at the same time. WDYT?

Thinking about it, this might actually be more suitable to include when we have our next release, just so that it can be tested at the same time. WDYT?

See respond 👉 https://github.com/fredrikhgrelland/terraform-nomad-postgres/pull/61#issuecomment-733519565

Thinking about it, this might actually be more suitable to include when we have our next release, just so that it can be tested at the same time. WDYT?

See respond 👉 fredrikhgrelland/terraform-nomad-postgres#61 (comment)

See answer 👉 https://github.com/fredrikhgrelland/terraform-nomad-postgres/pull/61#issuecomment-733524853

Thinking about it, this might actually be more suitable to include when we have our next release, just so that it can be tested at the same time. WDYT?

See respond 👉 fredrikhgrelland/terraform-nomad-postgres#61 (comment)

See answer 👉 fredrikhgrelland/terraform-nomad-postgres#61 (comment)

See reaction 👉 https://github.com/fredrikhgrelland/terraform-nomad-postgres/pull/61#issuecomment-733524853

@claesgill @zhenik
this has now run successfully in hive, so I think we are good to merge this now
https://github.com/fredrikhgrelland/terraform-nomad-hive/actions/runs/382715027


Not found (404) alters status code on HEAD request

Something strange popped up. On my site I don't have a '/ping' route. When I run cURL on it to fetch the status code, I get this:
➜  server  curl -k -s -o /dev/null -w "%{http_code}" https://my.server/ping
404%

➜  server  curl -k -s -o /dev/null -I -w "%{http_code}" https://my.server/ping
200%

The difference here being that the first request happens with a GET request and the second one with a HEAD request. Am I missing something or is this a bug?
I have a custom $app->notFound() handler that returns a pretty 404 webpage and returns the 404 status code. It obviously doesn't get called for a HEAD request.

Closing this one out as there has been no reply on it for awhile.


debounce on input change, but immediate on blur

Hi!
It would be great to have the same functionality as in angular 1: https://docs.angularjs.org/api/ng/directive/ngModelOptions That is: ng-model-options="{ updateOn: 'default blur', debounce: { 'default': 500, 'blur': 0 } }"
The use case is that we want a debounce delay when typing (on e.g. input fields) but an immediate update on blur.

+1

well debounce does not know about events
updateTrigger is very trigger happy..
the way I see it is there can be another behavior combining the two.. but its kinda not dry
and not separated.. definitly doable though..
plugin?


Provides financial terms.

I needed something like this to create events, so now it can be used to generate this:
ExportLapseRequested
PolicyCardIssued
ReversalAlertCompleted
ATMCompletionExpired
AlertClaimUpdated
DeclineBreachIssued
AccessDeclinePending
ATMTransactionPending

Add it to the existing Finance provider?
https://github.com/datafaker-net/datafaker/blob/main/src/main/java/net/datafaker/providers/base/Finance.java

Other than that, it looks good to me.


Fixes/Improvements

Dopedia List Articles Fix
Added SEO Options on i18n
Fixed broken links in some articles
Disable Dogepedia article  "dogecoin-developers" (can be enable)

Hello,
it will take a bit to review this as there are a lot of files that have been changed. But why removing the developers article? I proposed a new version, simplified and more generic, here: https://github.com/dogecoinfoundation/dogecoin.com/pull/109
I think the article should be kept but modified.

I will submit a new PR to replace this one.


Unable to connect to MongoDB

I am trying to connect to MongoDB Atlas.  Are certain ports blocked?
Steps to reproduce the behavior:
from pymongo import MongoClient
MONGO_URL = "xxx"
client = MongoClient(MONGO_URL)
client.db.find({})
Results in ServerSelectionTimeoutError on port 27017

Hi, we're taking a look at this right now.

@benstrum - Couple of questions:

Are you trying to connect to a MongoDB installed elsewhere from the notebook?
Which notebook are you trying to test?

Hi, I had created my own notebook where I was going to pull a dataset out
of one of my existing mongo servers.  To reproduce the issue, you can
easily create a free mongo db at their site (
https://www.mongodb.com/cloud/atlas/register) and then try to connect.  My
guess is that port 27017 is blocked on aws' end.
-Ben
On Fri, Jan 21, 2022 at 7:59 PM swagulkarni @.***>
wrote:

@benstrum https://github.com/benstrum - Couple of questions:

Are you trying to connect to a MongoDB installed elsewhere from the
notebook?
Which notebook are you trying to test?

—
Reply to this email directly, view it on GitHub
https://github.com/aws/studio-lab-examples/issues/51#issuecomment-1018778469,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEXEG665XA4XVXKXQ6ASJ2LUXGUKTANCNFSM5MGVVI4Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

How are you managing your connection to your Mongo instance? Can you validate that the connection works with the same code outside of SMSL, for example from your laptop?

Yes I'm able to connect locally and from Google colabs.  Have you tried
making a connection?  As i mentioned, you can create a free account in
mongo atlas   Also, can you verify that the port is open?
On Thu, Jan 27, 2022, 10:41 PM Emily Webber @.***>
wrote:

How are you managing your connection to your Mongo instance? Can you
validate that the connection works with the same code outside of SMSL, for
example from your laptop?
—
Reply to this email directly, view it on GitHub
https://github.com/aws/studio-lab-examples/issues/51#issuecomment-1023664179,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEXEG63A4ZZCPKPSZWE2BRDUYG3Z3ANCNFSM5MGVVI4Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

This issue also exists for outbound traffic on port 3306.  We have a notebook that connects to a mysql RDS instance that is open to the world (academic use with fake data) works great from EC2 or other db tools but the same code when run from a Jupyter Lab notebook fails to connect.  Running curl -v telnet://.cluster-xxxxxx.us-west-2.rds.amazonaws.com:3306 times out when run from a terminal session on the Jupyter Lab notebook but not my local instance.  Are there any outbound network restrictions?  Port 443 seems to work but not 22 or 3306?

So I found a work around if you're willing to use a reverse TCP proxy using NGINX.  It looks like the notebooks will support outbound traffic on port 80.  I setup a reverse proxy with the following config in nginx.conf
stream {
server {
listen     80;
#TCP traffic will be forwarded to the specified server
proxy_pass .cluster-xxxxx.us-west-2.rds.amazonaws.com:3306;
}
}
Then updated my notebook to connect as follows:
try:
mydb = mysql.connector.connect(
host="ec2-xx-xx-xx-xx.us-west-2.compute.amazonaws.com", # URL or hostname of the database in Oregon
user="dbuser",
password="",
port="80",
database='XXX')
It would be great if the Jupyter Studio Lab environment would allow outbound 3306, 27017, etc . .
Maybe if we are nice to @EmilyWebber she can help us fix it ;)

@darrenkraker It never hurts ... haha just kidding! We'll try to get some clarity on which ports are open / closed for SMSL straight away.

We're having the same issue with it blocking Neo4j ports.  Apparently 80, 443 and 53 are open.  Keep in mind this isn't incoming traffic it's block but outgoing, which is pretty strict....
I've tried to configure to use those other ports but AWS Linux seems to have things running on them already, so I'm getting bind failures.

Hi @EmilyWebber ,
I just wanted to chime in here.  It is now close to 1 year since @benofben wrote the excellent note above that describes the issue in detail.
I just spent about 1 hour trying to figure out why a connection that works from other AWS systems and Google Colab doesn't work in Studio Lab.  I then luckily stumbled on this thread which saved me from another several hours of searching for a solution.  I now know that SageMaker Studio Lab only supports a limited set of ports externally, but what is being done about this?
Any chance we will get access other ports or is Studio Lab not a viable solution for those of us that need to access ports other than 80, 443 and 53?
Thanks in advance for your time!

Hi, these ports were purposely closed for security reasons.
My best recommendation if you need these ports now is to migrate to SageMaker Studio.  It requires an AWS account. There is a free tier that you can take advantage of.


Update Decimal128 from mongo-java-driver

At some point since this class was originally copied from the mongo java driver codebase, it was amended to extend Number and implement Comparable. Let's take advantage of that.
Resolves #95

Thanks for the contribution!


setLocation with "io.appium.java_client.Location" throws "Error: longitude must be provided"

The problem
In Java client 9.1.0, this (https://github.com/appium/java-client/commit/4623e1138ca19c388ac2f636a30a4320c00b7fb5) issue is solved but will be released in the next version.
After updating selenide-appium:7.2.0 and even 7.1.0, we are facing to run the test while setting the location before starting the test.
My question is, how do I handle this issue because in 9.1.0  the setlocation (import org.openqa.selenium.html5.Location;
) is depriciated.
Details
If necessary, describe the problem you have been experiencing in more detail.
Tell us about your environment

Selenide-Appium Version: 7.2.0 or 7.1.0
IOS\Android\WEB version:
Selenium Version:
OS Version:

Code To Reproduce Issue [ Good To Have ]
Please remember that with sample code it's easier to reproduce the bug.

As the documentation says use io.appium.java_client.Location instead of org.openqa.selenium.html5.Location

@umerhamdan10 It seems your question is related to Appium, not Selenide.


feat(backend): 跨业务监控验证

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


access: may-start-trybots

Hi, I'd like to request access for the may-start-trybots privilege, so I can request trybots on the viewcore (x/debug) repo.
Thanks!
Here is the group URL: https://go-review.googlesource.com/#/admin/groups/1030,members
My Gerrit email address is<EMAIL_ADDRESS>

cc @golang/release

@jordanlewis: since the trybots are relatively sensitive infrastructure, we try to keep the set of people who can trigger them low. We're happy to expand the set to include frequent contributors for whom it'll be a big time-saver, but so far it looks like you've only sent a CL or two. For a low volume of changes, we hope that relying on your reviewers to trigger trybots should be acceptable.
I'm going to close this issue. Feel free to reapply if you find that not being able to trigger trybots is a major impediment to your work.


Add Feature BaseCharacter Attack

[x] Stop Moving When Attacking
[x] Delete Collider in Projectile
[x] Move to enemy and if Character is enough close to enemy, Attack enemy

https://github.com/SR-PLUM/SidheRigel/pull/17/commits/92137da0154b2e577988785175160e66e2c39950


Investigate the blog example

The blog is failing because it try to access to filesystem.
Need to investigate and fix.

This is likely coming from here https://github.com/opennextjs/opennextjs-cloudflare/blob/main/examples/vercel-blog-starter/src/lib/api.ts

I believe this is because there is no cache handler serving the seed files anymore so it's falling back to on-demand ISR for the routes when they should be SSG'd at build time. Part of the reason I added the seed file stuff in the cache handler in main was because I had the same problem with my blog.

Thanks @conico974 and @james-elicx.
I think re-introducing cache would solve that, we can hopefully come to that this week.


example added: extending a Image classification with Object detection

Hi, it looks like there are some key issues in this PR—please correct me if I'm wrong:

The PR proposes to remove all files/content from the Keras website (keras-team/keras-io) and adds one file with your example (classification_to_detection.py). Maybe you created an empty folder and added your file to it, which may be causing git to interpret that you're removing nearly everything (I've checked your repo: https://github.com/krn-sharma/keras-io). You should only have the "green" diff:

The example appears to have multiple grammar and spelling errors. You can resolve some of them by copying and pasting the Markdown text into a free Google Doc and then clicking on Tools > Spelling and grammar* > Spelling and grammar check (wait for a few seconds for the file to be auto-saved in your Google Drive first). Then, I'd suggest that you have someone you know review the text before you commit and push the changes to your repo. It can take hours for reviewers and contributors to check the doc, especially if it contains many basic grammatical errors (such as this sentence: Convoultion implemention is much more fast than tradition sliding window object detection because it predict the output of all window in just one inference.)

Here are some helpful Google guides: writing style: https://developers.google.com/style, "a"/"the" (articles): https://developers.google.com/style/articles, sentence-case capitalization: https://developers.google.com/style/capitalization + anything under Language and grammar. Then, check the material under Punctuation and Formatting. And remember the General principles (https://developers.google.com/style/tone). Microsoft also has some awesome material: https://docs.microsoft.com/en-us/style-guide/welcome/
👍
I hope this helps!

@8bitmp3 Thanks for your feedback. I will review the example.
Documentation for contribution states that to submit only .py file in PR that's why I deleted other files from my fork. Maybe I shouldn't have forked but instead created empty repo.
I am not sure as I am new to open source contribution but I will check it.
Thank you for taking out your time to review my PR

No worries.

Documentation for contribution states that to submit only .py file in PR that's why I deleted other files from my fork. Maybe I shouldn't have forked but instead created empty repo.

Yes, by adding a file to the forked repo you don't have to delete everything else—that's how PRs work. Note that the contributing page (https://keras.io/examples/#adding-a-new-code-example) says: "New examples are added via Pull Requests to the keras.io repository. They must be submitted as a .py file that follows a specific format."

@8bitmp3 thanks for the help with the PRs -- much appreciated!

Are you still planning on working on this PR? Should I close it?


Remove unnecessary usage of StoredState to track exporter installation status

Currently we use StoredState in the charm to track whether the exporter has been installed. But this is unnecessary as we could just use Path.exists to check whether the relevant files (/etc/systemd/system/hardware-exporter/.service and /etc/hardware-exporter-config.yaml) exist.
Limitations of StoredState

As part of this PR to add smartctl exporter support for the charm, we are removing the usage of StoredState to track whether an exporter has been installed.


Add react-devtools only as dev dependency

I'm compiling electron at every install for no reason.

I agree with you. This dep should be dropped. It’s here because historically a react quirk prevented the library from working without it. But it should not be the case anymore. Coule tou investigate a bit to determine if the dep can be safely dropped?


Roll engine c23690f0d751..c06432da47bd (1 commits)

https://github.com/flutter/engine/compare/c23690f0d751...c06432da47bd
git log c23690f0d751..c06432da47bd --no-merges --oneline
c06432da4 Delay the removal of the onDrawListener. (flutter/engine#6052)
The AutoRoll server is located here: https://flutter-engine-flutter-roll.skia.org
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

@rmistry @liyuqian : New roll has been created by https://flutter-engine-flutter-roll.skia.org

Trybots failed. These were the failed builds: https://flutter-dashboard.appspot.com/build.html

Commit queue failed; closing this roll.


Set Host header at serialization, not at beginning

This should be simlar to how TransferEncoding is added only if ContentLength or TransferEncoding aren't already present.

@seanmonstar thanks for the discussion on IRC about this and for filing the issue. For anyone else following, this makes setting client::Request headers from a Headers work (with 0 overhead) without worrying about any state that hyper had in the original headers map (because there won't be any!).

I took a quick look at this. So what you're saying is move the Host specific code from here to somewhere in here, right? If so, how are we  going to have access to the Request.url() from the second function?
Also, in the Client code we're creating a new Headers instance place the Host first and extend with the remaining headers. I assume that we do that so that the Host header appears first. Do we care about the order of the remaining headers? If not maybe a combination of swap_remove and push could work without iterating.

Hm, I no longer think this should be done (doing just before serialization), since as you pointed out, the information isn't available that far down the pipeline.

How about this part?

Also, in the Client code we're creating a new Headers instance place the Host first and extend with the remaining headers. I assume that we do that so that the Host header appears first. Do we care about the order of the remaining headers? If not maybe a combination of swap_remove and push could work without iterating.

It would take a few additions in Headers's and VecMap's implementations to expose those Vec methods. I know we're not gaining much but just asking.

That the host is set first in the client implementation is for 2 reasons, I suppose:

To allow an explicitly set Host header to override the default.
RFC7230 recommends making the Host header the first one after the start line.

However, there are some downsides to how this is currently done. It is a little wasteful, as it copies the Headers struct unnecessarily. It also means that if someone did set a Host header, it is not necessarily put to the front, and so may not be printed first.
Perhaps a better solution would be:

Don't create a new Headers struct to copy in to, but instead check if there is no Host header, and if not, add it.
In hyper::http::h1::parse, adjust the ClientTransaction::encode function to wrap the Headers in a HostFirstHeaders<'a>(&'a Headers) adapter. This fmt::Display adapter could always write the Host header first, and then skip the Host in the iter part.


Anyone please help to build miniblink49 Lite branch

Today I tried to assemble miniblink49 Lite branch.
2 projects (ots, miniblink) failed to compile...
I use newest VS 2022 Preview.
If you're experienced C++ developer please try to repeat my way... and explain the bugs you meet...
My goal is:
transform MiniBlink into some useful uwp web browser for old sweet Lumia phones (Windows 10 Mobile os)
Best wishes,
MediaExplorer

Sorry, I just saw the message.
What error did you encounter when compiling?

The lite branch has been discontinued, please compile with the master branch

Sorry, I just saw the message. What error did you encounter when compiling?

Thank you. I decided to cancel all my MiniBlink49's plans. I closed this issue. Sorry


Update PandExo to V1.4

Kevin and I have pushed the V1.4 PandExo release. It includes several updates to the website front end, fixed broken links, updates to code, exomast queries and bunch of other stuff.
Make sure you download the new pandeia reference data (v1p4) because the old data is not compatible with the new code.
Also, you will want to add the exoctk google analytics before you install (so make sure you clone and install from git instead of pip)
Lastly, make sure you test both the HST and the JWST calculations. Specifically, make sure that calling a planet through ExoMAST and inputting in fixed values both work and yield similar results for the same planet. Kevin and I have tested locally but we should double check.

Great! What is the approximate timeline for this change to be implemented? I am getting increasing # of emails.

This task is priority #1.  Let's get the website updated yesterday!

Sorry for letting this slip through the cracks.  I'll get on it!

I should ping stargate for this once its ready to be tested

@kevin218 @natashabatalha I believe I have the new version of pandexo deployed on tlexo now!
Thank you for the guidance @natashabatalha, it was quite helpful!

Great!! I don't have access to tlexo. @kevin218 or @hrwakeford can you make sure both Webby and HST are working?

Update:  I tried to deploy pandexo version 1.4 to plexo (note that we can't actually test the deployment on tlexo for various reasons), however we got a 502 error when trying to access the service.  Furthermore, we don't actually have read access to the web server error logs, so it is not apparent what the problem may be.
We will have to ask IT to show us what is going on in the error logs and/or try to get things running locally and see if any obvious errors appear.

Should we close this issue or is there more to do?

I think we can close it. I'll open up a PandExo issue to remind myself to fix the last exo.mast thing.


bug: Bug in Tab Navigation - (back button not visible)

Type: bug
Ionic Version: 1.x
Platform: all
Hi all,
in the current ionic verion 1.3.1 the back navigation seems to be broken. The back button is not visible on subpages of an tab after some navigation steps.

With version 1.3.0 the back button "< About" is shown correctly:
CodePen 1.3.0
https://codepen.io/DrJonnie/pen/yJARyg

Hello! Thanks for opening an issue with us! I am going to be closing this issue as a duplicate of https://github.com/driftyco/ionic/issues/6875. Thanks for using Ionic!


(FACT-3079) Caching custom facts not working propperly

Used the following custom fact to test:
Facter.add('yum_package_updates') do
  confine osfamily: 'RedHat'
  setcode do
    yum_updates = ["a","b","c","d"]
    yum_updates
  end
end

Facter.add('yum_updates') do
  confine osfamily: 'RedHat'
  setcode do
    Facter.value(:yum_package_updates).length
  end
end


docs: updated style_guide

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[ ] Feature
[ ] Refactoring
[x] Documentation related to #7106

Context

[ ] Closes #<issue_number>
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality

Optional

[ ] I've tested it at </ksm/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Community participation

[ ] Are you at KodaDot Ecosystem Telegram?

Screenshot 📸

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary

🤖 Generated by Copilot at ec90a29
Updated STYLE_GUIDE.md to fix errors and enhance readability.

🤖 Generated by Copilot at ec90a29

Style guide refined
sentence, example, format
Cut like autumn leaves

pay 10 usd

😍 Perfect, I’ve sent the payout
💵 $10 @ 4.25 USD/DOT ~ 2.353 $DOT
🧗 13rFRPVKjJzQXVC8ZqHZv5YMmwmk4MU7z4HeYk218hEMpQXH
🔗 0x27e11c7b0e13c2027faa9fc9b688a601194ed26fc93bd47c40e66cc790148651
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Any plans to port Flump to Haxe Starling?

I have a version that works with Haxe Starling. However it is unmaintained and uses a completely different API, specific for my own purposes, that I don’t think most starling/flump users would like. It was pretty experimental and I’m not sure if it is much worth putting it up on github.

I should mention that I have a Kha port that is much better. I could possibly put it up on Github if there is enough interest.

That would be great!
On Sat, 23 Nov 2019, 00:02 Jack Lee<EMAIL_ADDRESS>wrote:

I should mention that I have a Kha port that is much better. I could
possibly put it up on Github if there is enough interest.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jackwlee01/pixi-flump-runtime/issues/32?email_source=notifications&email_token=AAMILYTJDKTPGJYZ65WL4L3QVBJGHA5CNFSM4JQRXPXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE675PY#issuecomment-557711039,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMILYXWJZN46BJNGGVHGFTQVBJGHANCNFSM4JQRXPXA
.


FIxes #126   Self-signed certificate issue

Currently our app doesn't support servers with self-signed certificates.
This PR fixes that.
When a user tries to connect a server with a self-signed certificate, a dialog box appears asking for permission. If permission is granted the app loads the server. Also, that server is added to the list of CertifiedURL  in app's backend, so the user doesn't face the dialog box in future if he tries to connect the same server but only initially once.

Any suggestions on improving this PR are welcome :smile:

This looks good. @timabbott is it possible for you to cross check it? As of now, I don't have access to a Linux machine so I'm not sure whether this fix will work or not :thinking: .

@Lplenka there is lot of code shared between pref.js and domain.js. We could reuse it.

@Lplenka there is lot of code shared between pref.js and domain.js. We could reuse it.

Yes, they share a lot in common, only the HTML elements they use with are different. You can suggest me the changes, I will add them. But most probably, in the long run we won't be having a pref.js since the app will support multipe servers so server switching related files will be deleted eventually.

To test this, I'd recommend getting a cheap VM and installing the Zulip production environment following  http://zulip.readthedocs.io/en/latest/prod-install.html, using the SSL certificate documentation.
I think @feorlen would probably be able to cross-test this pretty quickly, since she's pretty practiced with the installer (though I think she's on a trip this week, so may be slow to reply) :)

Sure, I can have a look when I'm home early next week. I'm away without a computer at the moment.

This partially fixes the self-signed cert problem. If a user enters a server which is self-signed, app will ask for permission to load that URL -

@Lplenka merging this. Next step would be select the appropriate certificate from keychain using select-client-certificate.

@Lplenka merging this. Next step would be select the appropriate certificate from keychain using select-client-certificate.

Yes, surely will work on that.

I tried to test this, following the directions in the zulip-electron README.md (since now it's been merged to master) but install failed. I've never installed the desktop app before, so I'm not sure what should happen. This is on a ubuntu VM that previously was running the webapp (dev), so I would think it would have all the expected stuff.
Do you still need someone to verify this against a running server?
Here's what happened:
first attempt at npm install fails here:
npm ERR! Linux 3.13.0-24-generic
npm ERR! argv "/usr/local/nvm/versions/node/v6.6.0/bin/node" "/usr/local/nvm/versions/node/v6.6.0/bin/npm" "install"
npm ERR! node v6.6.0
npm ERR! npm  v3.10.3
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>postinstall: `node install.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'node install.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the electron package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node install.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs electron
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls electron
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/feorlen/zulip-electron/npm-debug.log

Here's what's in the debug log:
0 info it worked if it ends with ok
1 verbose cli [ '/usr/local/nvm/versions/node/v6.6.0/bin/node',
1 verbose cli   '/usr/local/nvm/versions/node/v6.6.0/bin/npm',
1 verbose cli   'start' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prestart', 'start', 'poststart' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 silly lifecycle<EMAIL_ADDRESS>no script for prestart, continuing
7 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>8 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
9 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/local/nvm/versions/node/v6.6.
0/lib/node_modules/npm/bin/node-gyp-bin:/home/feorlen/zulip-electron/node_module
s/.bin:/usr/local/nvm/versions/node/v6.6.0/bin:/usr/local/nvm/versions/node/v6.6
.0/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/
usr/local/games
10 verbose lifecycle<EMAIL_ADDRESS>CWD: /home/feorlen/zulip-electron
11 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'electron ./app/main' ]
12 info lifecycle<EMAIL_ADDRESS>Failed to exec start script
13 verbose stack Error<EMAIL_ADDRESS>start: `electron ./app/main`
13 verbose stack spawn ENOENT
13 verbose stack     at ChildProcess.<anonymous> (/usr/local/nvm/versions/node/v
6.6.0/lib/node_modules/npm/lib/utils/spawn.js:33:16)
13 verbose stack     at emitTwo (events.js:106:13)
13 verbose stack     at ChildProcess.emit (events.js:191:7)
13 verbose stack     at maybeClose (internal/child_process.js:877:16)
13 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_proc
ess.js:226:5)
14 verbose pkgid<EMAIL_ADDRESS>15 verbose cwd /home/feorlen/zulip-electron
16 error Linux 3.13.0-24-generic
17 error argv "/usr/local/nvm/versions/node/v6.6.0/bin/node" "/usr/local/nvm/ver
sions/node/v6.6.0/bin/npm" "start"
18 error node v6.6.0
19 error npm  v3.10.3
20 error file sh
21 error code ELIFECYCLE
22 error errno ENOENT
23 error syscall spawn
24 error<EMAIL_ADDRESS>start: `electron ./app/main`
24 error spawn ENOENT
25 error Failed at the<EMAIL_ADDRESS>start script 'electron ./app/main'.
25 error Make sure you have the latest version of node.js and npm installed.
25 error If you do, this is most likely a problem with the zulip package,
25 error not with npm itself.
25 error Tell the author that this fails on your system:
25 error     electron ./app/main
25 error You can get information on how to open an issue for this project with:
25 error     npm bugs zulip
25 error Or if that isn't available, you can get their info via:
25 error     npm owner ls zulip
25 error There is likely additional logging output above.
26 verbose exit [ 1, true ]

After running provision on the Zulip install already there, repeating npm install also shows this error:
feorlen-VirtualBox[feorlen]:~/zulip-electron$ npm install

><EMAIL_ADDRESS>postinstall /home/feorlen/zulip-electron/node_modules/electron
> node install.js

/home/feorlen/zulip-electron/node_modules/electron/install.js:46
  throw err
  ^

Error: ENOENT: no such file or directory, lstat '/home/feorlen/zulip-electron/node_modules/electron/dist/resources'
    at Error (native)

@feorlen Thanks for your help on this.

Do you still need someone to verify this against a running server?

It would be better if we get positive reviews from all those you are using the self-certified zulip server. So please test it.
Please upgrade  nodejs  to 6.9.4 or more recent.
You should remove node_modules using command rm -rf node_modules && rm -rf app/node_modules and try a fresh install after upgrading nodejs.
Here is a discussion on a similar problem.

Hi @Lplenka I'm trying to follow the directions in the conversation you linked, but I'm confused. Is there another way to install nodejs 6.9.4 without first building nvm from source? apt-get on ubuntu only installs 6.6.0.
feorlen-VirtualBox[feorlen]:~/zulip-electron$ sudo apt-get install npm
Reading package lists... Done
Building dependency tree
Reading state information... Done
npm is already the newest version.
0 upgraded, 0 newly installed, 0 to remove and 2 not upgraded.
feorlen-VirtualBox[feorlen]:~/zulip-electron$ which npm
/usr/local/nvm/versions/node/v6.6.0/bin/npm

When I try to build/install nvm using the install script from its repo, I get a message that nvm is already installed and I should update it directly from git into /usr/local/nvm.
=> nvm is already installed in /usr/local/nvm, trying to update using git
=> Failed to update nvm, run 'git fetch' in /usr/local/nvm yourself.

Why would I need a git repository for anything in /usr/local? Is this normal?

@Lplenka I'll do that, as updating v6.10.2 is giving me the same error as when I tried v6.9.4. See you in chat!

I don't understand why node gave me so much trouble, but @Lplenka and I got it working. I'm prompted to confirm the self-signed cert. Yay!

@feorlen Thanks! :smile:


feat(client): consolidate excessive amount of notes in a given tier

Backport of #4922
We had some people reporting hitting tx too large problems, so decided to attempt a backport.
However, I'm not sure how I feel about landing it, as it feels like a big change to just add to a minor update to already matured 0.3.

I'd say let's not do anything major on 0.3 anymore.


Stunprod

About the PR

Added the sound of blows for a stanprod like a stunbaton.
The stampede can be thrown like a stunbaton.
Added a standpod with a medium-capacity battery. 4 punches to knock you down. Charge for 10 strokes.
The recipe for creating a stunprod with a medium-capacity battery.

Why / Balance

Stunprod is not used by players.
A stundard stunprod requires 5 strokes. The battery is running out fast.
Stunprod has no impact sound.
Stunprod does not have a StaminaDamageOnCollide component, so it cannot be thrown at an enemy.
I hope that after the update, this item will be used frequently by players.

Technical details

Media

https://github.com/space-wizards/space-station-14/assets/154531074/8e1bf2a6-d27e-4cc0-9e91-dbc7fed740ae

[X]

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

The team will need to discuss the balancing impact on this.

i think you should replace the glue with an igniter, since thats on the sprite and we have those now
would also be cool if you could take the cell off and recharge that separately

i think you should replace the glue with an igniter, since thats on the sprite and we have those now
would also be cool if you could take the cell off and recharge that separately

This is my first PR, so I followed the example. A good offer for deconstruction, but I will stop for now, because on another github my changes caused errors in YAML Linter / YAML Linter (pull_request) and
Build & Test Debug / build (ubuntu-latest) (pull_request).


Added stageName

Adds a 'curStage' variable accessible in Lua.

Why??
function onCreatePost()
     debugPrint(getPropertyFromClass('PlayState', 'curStage')
end

Why??
function onCreatePost()
     debugPrint(getPropertyFromClass('PlayState', 'curStage')
end

bruhhhh why didn't you tell me on my issue

actually songName is a thing, so why not keep this

Because songName is useful unlike curStage

songName doesn't even work

songName doesn't even work

It doesn't ?

Yes it does lmao

I at first thought another variable like stageName was redundant, but maybe this would give inexperienced modders the ability to do some stage shenanigans. (curSong currently influences the stage in source, and the stage influences stage elements if they aren't null.) Aren't luascripts still able to change any variable though?

I at first thought another variable like stageName was redundant, but maybe this would give inexperienced modders the ability to do some stage shenanigans. (curSong currently influences the stage in source, and the stage influences stage elements if they aren't null.) Aren't luascripts still able to change any variable though?

Yeah I think they can change basically anything, but the value might be changed again by the game.

 Setting curStage works without any hiccups. (things could be null depending on wether functions using curStage are called before or after)


fairseq_moe_docker_slurm (Remove Slurm pinning)

Removed Slurm pinning, we are using torch.distributed.launch with slurm (srun) starting 1 process per node. The slurm pinning is for 8 slurm tasks and so is unoptimal for a single slurm (srun) process.

Looks good to me.  @garvct did you test the impact on performance by chance?


TEST - IGNORE

Description

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Deploy preview for calico ready!
Built with commit 9636d98fa60b604206bebc7cef17f79647439aee
https://deploy-preview-2226--calico.netlify.com


FEAT Develop the AggJoiner and AggTarget

This meta-issue is a roadmap for the developments of the AggJoiner and AggTarget estimators, currently implemented in #600.
We need to merge the following PRs before tackling any of the developments below.

[x] #733
[x] #600

This is not a personal roadmap. Anyone is welcome to contribute! 🙂

[ ] Add add key argument
As discussed in https://github.com/skrub-data/skrub/discussions/751
[ ] Add MultiAggJoiner and MultiJoiner
Joiner and AggJoiner currently accept multiple aux tables, but this creates complexity because some parameters can be iterable of iterable of str. Instead, we need to create Multi{AggJoiner, Joiner} to handle multi tables, and only accept a single table otherwise.
[ ] Separate count from the other operations
The count operation is currently applied on all categorical columns, which create duplicates. Also, we want to use it even if we only have numerical columns. So, we need to treat this operation separately.
[ ] Add column specific operations
Selected operations are applied over all categorical or numerical columns. We could let the user select columns on which to execute specific operations. We could either achieve this with a column-operation mapping keyword argument, or a list of tuples, inspired by TableVectorizer's column_specific_transformers.
[ ] Support polars.LazyFrame
Lazyframes don't have the __dataframe__ attribute. A duck-typing like lazy_df.__dataframe__ = None is enough to get it working, but we might want a less hacky solution.
[ ] Support value_counts and hist operations for Polars
[ ] Add more operations for both Polars and Pandas dataframes/lazyframes:

[ ] topk(k)
[ ] unique-count
[ ] quantile(q)
[ ] first
[ ] last

[ ] Support fuzzy join
This could be implemented using the fuzzy_join function or composing the Joiner class.
[ ] Enable screening
When running multiple aggregation operations over features, screening would enable us to select the columns correlating with the target before joining them.

Any other ideas?

I'm working on adding the key argument, and the MultiAggJoiner :)

Hey! Would having Bayesian means be useful? It seems to me the spirit of skrub is to provide good defaults to users. One thing that can happen with AggTarget and AggJoiner out of the box is to do aggregates on small groups, which can lead to overfitting

shrinking towards the overall aggregate across groups sounds like a useful option to add

I guess it only applies for some of the aggregation operations

I guess it only applies for some of the aggregation operations

My initial understand of skrub is that it should provide good defaults to users. It's nice to have AggJoiner and AggTarget be able to compute a variety of statistics. But in practice (e.g. Kaggle) it's more or less sufficient to compute the mean. So if these classes are going to be used as part of TableVectorizer, maybe the default could be a mean that shrinks towards the overall mean. I think this would be a good pit of success for most users.

Hi @MaxHalford, that sounds like a good idea, and something that is already performed in scikit-learn TargetEncoder. We should create a small benchmark to explore this idea in AggJoiner and AggTarget.

I'd be interested to work on screening when I'm done with the multi joiners. I think it's a feature that might also be useful for other estimators than the Joiners, if you think that's the case we can open a new issue on this topic

as discussed with @TheooJ , the AggTarget does not implement cross-fitting (see eg the target encoder doc) which can cause serious overfitting of the downstream estimator. moreover shrinking/smoothing is probably important when there are some values with few matches in the joining column

As discussed with @Vincent-Maladiere & @jeromedockes, here are the next steps for the AggJoiner & AggTarget as well as some elements of design to figure out:

Remove _pandas, _polars & _namespace files, only used in AggJoiner and AggTarget. To do so, RFC the aggregate func with the dispatch mechanism

Which operations will be supported ? For now, it would be simpler to work without considering datetimes / embeddings
Do we want first / last operations (note that for now we don't handle datetimes) ?
Do we want hist and value counts ?

RFC AggTarget as a generalization of the TargetEncoder (with cross-fitting), but with more aggregation options
Do we want to specialize estimators to apply one operation to all selected columns ? Issue: groupby + join happen multiple times

Do we want the numeric / categorical operations split ?
Do we want columns specific operations (like the TableVectorizer's column_specific_transformers) ?


preconnect is supported by Chrome 46+

Support is enabled on all platforms, see: https://www.chromestatus.com/feature/5560623895150592

Thanks!


Added vscode editor plugin

VSCode plugins are not yet publicly available but will be soon. This plugin adds XO linting to the editor.

Nice!

No problem. Happy to help!


Support run tests on windows

Although there is no plan to write collector for windows, it should ensure tests are runnable on windows.
Atleast support mingw.

fixed
3352973fbbc63dc799a6c169df2478705b7a62ab
3410ca76fbf604025550cd3f21928028563aceac
194980aec56fbc4d9155c96094330ac7a6a83ca6
a9da4b8bd11264820985737d005a93e84e9b2f90


Tearsheet main body is taking 20% margin from right side

20% margin is appearing from right side of main body under tearsheet.

We are using the latest version(0.35.3  as of now) of @carbon/ibm-cloud-cognitive  and consuming the tearsheet in the following way.
Importing the following in .scss file.
@import '~@carbon/ibm-cloud-cognitive/scss/index';

and
import { Tearsheet } from '@carbon/ibm-cloud-cognitive'

Can someone please look into this issue asap?

@dcwarwick FYI

@dcwarwick
Thanks for looking into this issue.

Yes we are using the carbon.  "carbon-components": "^10.32.1",
  "carbon-components-react": "^7.32.1",
  "carbon-icons": "^7.0.7",

It'll be available in our demo. I'll send you the URL once it'll reach there in demo environment.

@dcwarwick
We fixed this by overriding css in our own component. I'm closing this issue as it's not happening now.
Thanks for your help.


[Create a Project] UPD location field to country dropdown

UPD this field in “create a project” to either a country drop down or a search-and-select based off of where the project originated.
Q: Potentially expand this issue to allowing users to filter projects by country.

is this different than what just went live? or can we close this?

Looks like we can close!


Add logic to fetch APP_UUID from the applicaiton

The has a global/namespaced scope with focus purely on the experiments. Any metadata we'd like to add towards the metrics for any given experiment run can be added via the context labels on the chaosengine CR - these are converted to metrics filters/labels


fix: add left margin to align with content

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #32
Test URLs:

Before: https://main--choice--hlxsites.hlx.page/en-us/explore/
After: https://32-deal-teaser-alignment--choice--hlxsites.hlx.page/en-us/explore/

Deoesn't seem to be aligend on the right side


Add a switch to start or stop charging

I'm assuming a "charge" switch would work for that. Otherwise a pair of buttons? Uses charging_service.StartSession and StopSession I think.

Fixed in https://github.com/borski/ha-lucidmotors/commit/3f31c0c2da5a076a4949072e26084326303d2779


Documentation update

Updated documentation to warn the user about non-markdown files being added to the public build, and providing a ignore pattern as a workaround. Please feel free to edit or let me know if I need to change any wording!

make sure you run npm run format to fix the format errors but the rest looks good :)

whoops sorry about that! Just ran it now :)


newbie in need of help

Good day, I am very new with Ark Server Tools, and Linux entirely. My admin had highly suggested using Linux and ark server tools on our new server (upgraded from our old windows one), and we just got it set up and going except we don't have our server configured to do our auto restarts, auto updates, and restart countdown timers. Unfortunately, things have happened suddenly where he is unable to get online to continue to help me configure this, and I have absolutely no idea what I am doing. I see in the config file the example but i'm not all that sure how to work it. for update warning messages it says %d is a replacement for the number, do I add a number here and how do I make it use that number timer for a countdown before restarting? Sorry I am really confused. All I need is 15 minute countdown for restarts, auto start the server on crashes, auto updates, and a daily restart at 4am est.

In the arkmanager,cfg file there is a line like this
arkwarnminutes="60"                                                 # number of minutes to warn players when using update --warn
Change the arkwarnminutes to what every you like.
The warning message should be ok unmodified.
As for the 4AM restart, you need to edit crontab in linux and add arkmanager restart @all

I almost forgot, you have to add a cronjob for the update to work also. Mine checks for update every 4 hrs and looks like this
0 0,4,8,12,16,20 * * * /home/xxxxx/bin/arkmanager update --warn --saveworld --backup --update-mods @all
Here is the man page for crontab in linux
http://man7.org/linux/man-pages/man5/crontab.5.html


requestBody content schema always uses first in list

Issue workflow progress

Progress of the issue based on the
Contributor Workflow

[ ] 1. The issue provides a reproduction available on
Github,
Stackblitz
or
CodeSandbox

Make sure to fork this template and run yarn generate in the terminal.
Please make sure Mesh package versions under package.json matches yours.

[ ] 2. A failing test has been provided
[ ] 3. A local solution has been provided
[ ] 4. A pull request is pending review

Describe the bug
When multiple content types are defined in the requestBody, the first is always used and forwarded.
More info and test available here:
https://github.com/Urigo/graphql-mesh/pull/6039
To Reproduce Steps to reproduce the behavior:
https://github.com/Urigo/graphql-mesh/pull/6039
Expected behavior
Expect JSON to take precedence if available.
Environment:

OS:
@graphql-mesh/...:
NodeJS:

Additional context

Thanks for the PR! Available in the latest release!


SPFx solution with multiple extensions doesn't display all extensions after deployment

Category

[x] Bug

Describe the bug
I have an SPFx solution that started out as 1 web part, simple HelloWorld using Yeoman. I deployed that to my tenant app catalog as version <IP_ADDRESS>. I have a communication site that I added this application to. The site has 1 regular SharePoint list and the out of the box shared documents doclib. I added the HelloWorld web part to the home page.
Then I ran the Yeoman generator again and added a list view command set extension. ExtensionOne should only appear on generic lists and only in the command bar. It is a HelloWorld extension. I packaged this up as version <IP_ADDRESS> and deployed it to my tenant app catalog. I updated the app on my site. At this point the extension shows up on my generic SharePoint list correctly but the web part disappears from the home page.
Finally, I ran the Yeoman generator again and added a second list view command set extension. ExtensionTwo should only appear on document libraries and can appear on both the command bar and the context menu. It is once again a HelloWorld extension. I packaged this up as version <IP_ADDRESS> and deployed it to my tenant app catalog. I updated the app one more time on my site. Now only ExtensionOne shows up. The original web part is still missing from my home page and ExtensionTwo does now show up at all.
If I delete the app from my site and from my tenant app catalog, empty out the trash bins, empty out the second level trash bins, deploy again, and add the app to my site again, then everything works as expected.
Steps to reproduce

Run Yeoman generator to create HelloWorld web part
Package and deploy to tenant app catalog.
Add app to site.
Add web part to a page.
Run Yeoman generator again and add list view command set extension.
Update app version.
Package and deploy to tenant app catalog.
Update app on site.
Run Yeoman generator again and add another list view command set extension.
Update app version.
Add new extension to elements.xml
Package and deploy to tenant app catalog.
Update app on site.

Expected behavior
I expect my web part to not disappear after first update. I also expect to see my second extension after my second update. It is not feasible to have to completely wipe the app from SharePoint and do a fresh deployment every single time we add a new extension. As shown here, this should be possible.
Developer environment

OS: Mac 10.15.3
Framework: Node 8.17.0
Browser(s): Chrome Version 80.0.3987.149
Tooling: VS Code 1.43.2, SPFx v1.10.0
SharePoint Online
Out of the box communication site

Additional context
I added my code here. This code includes both extensions and the web part. The only things I changed that were not a part of the HelloWorld Yeoman generated files are as follows:

package-solution.json: changed the version number
elements.xml: added the second extension since Yeoman doesn't automatically do that, made the second extension RegistrationId=101 and Location=ClientSideExtension.ListViewCommandSet as described here
ExtensionOneCommandSet.manifest.json in each file: changed the text in the buttons so I know which button belongs to which extension

Thanks for your contribution! Sharing is caring.

As we have seen that many times in our projects too, I could share some of the solutions we found for getting around that.
First, the missing Webpart in the picker:
The yeoman generator does not add a feature specification for Webparts to the package-solution.json. We have seen such Webparts working many times, but sometimes they are missing (for whatever reason). What we found is, that when you add a feature for your Webpart to the package-solution.json then it works every time.
The object to add looks like this:
     {
        "id": "6eebd686-ad73-44c5-b09b-5ca3db55ff92",
        "title": "A title that will be visible in the website features catalog",
        "description": "A description that will be visible in the website features catalog",
        "version": "<IP_ADDRESS>"
      }

The value of "id" must match the id of the Webpart that is defined in the Webpart.manifest.json.
It seems, increasing the value of "version" to something higher than "<IP_ADDRESS>" helps too, especially if you already had the solution installed without the added feature definition.
After that you'll need to create a new package, upload it and update the installed app on the site, like you did before.
Second, the missing extension:
You did everything we are also doing in that case with one exception. If the newly added extension does not show up you might need to deactivate and reactivate the feature corresponding to the extension under website features. That should do the trick.
It helps to have keywords like "Webpart" and "Extension" in the titles of your features to better distinguish between them when you need to reactivate one.
Hope this helps.

Thank you, @cwdata. I'll try these ideas out today!

Hi @cwdata @andrewconnell , I tried out the ideas listed above. Here are my findings:
Regarding the missing webpart in the picker, adding this entry to the package-solution.json file does add an entry to the site features list. I can then turn that feature off and on and can then see the webpart in the picker again.
However, this causes an issue in the app catalog under the column "App Package Error Message." The error is "Component ID {0} exists more than once in current package." From my testing, if the solution is already deployed before I add this to the package-solution.json file, the error doesn't really make a difference. But if I do a fresh deployment of the app with this object in the file, then the app never works correctly and no webparts show up in my picker and no extensions show up where they should.
Regarding the missing extension, turning the extension feature off/on definitely does the trick. Thanks for that suggestion. However, our extensions will be installed on potentially thousands of sites. Therefore we cannot do this process manually. Our next step was to try doing this using PowerShell. But then we hit a roadblock there when enabling the feature.
Apparently you cannot currently enable a site feature that is disabled. There are open bugs on that here and here for example. I tried this with PnP and with the o365 cli tool, same result on both.
Our current best solution is to do a tenant wide deployment and add code to our webparts and extensions so that they only work on the sites that we want them to. We are planning on checking the site property bag for X property and if it does not exist then they will show an error.
While this doesn't really resolve the original bug, if there is something we are doing incorrectly or could be doing better, please do share your ideas! Thank you!

Hi @erwinkehrt
that error you get in the app catalog is a known bug (https://github.com/SharePoint/sp-dev-docs/issues/3802)
Currently upgrading a solution in the app catalog is not reliable, and regarding the open issue, won't be in the foreseeable future. You need to delete the package (also from the recycle bin) and upload it again. That's not dependent on having the Webpart feature listed in the package-solution.json. We have seen that error on packages without such a feature too.
Fresh deployments with a feature for the Webpart in the Json file work very well for us, as well as upgrades using the app catalog procedure mentioned above. I'm wondering what could make the difference here.
Regarding the management of the feature: We always deactivated and reactivated the features through the GUI. I have not tried to automate that yet but yes that is indeed a huge problem if that doesn't work either.
If a tenant wide deployment of SPFx solutions that are meant to run only on one or a couple of sites is the only solution, then the load on SharePoint will increase heavily. Hopefully that's going to increase the priority to fix some of the bugs :-)

Hi @cwdata and whoever might be having this issue in the future. We opened a support ticket for this and were able to find a proper solution to this specific problem.
In regards to the web part disappearing, the solution that @cwdata left on March 27th fixes that.
In regards to the new extensions not displaying after being added to a solution that is already deployed, the trick is updating the version number in the feature itself in the package-solution.json file. This can be a different number than the version number of the solution, but you might as well keep them the same, point is it must be updated when a new extension is added to the solution. Please see image below as an example.

Once the solution with a new extension and updated feature version number is deployed, assuming this is not a tenant wide deployment, you need to update the installed app on the site in question. For clarity, this is done by going to the site contents, clicking on the ellipsis of the already installed application, click on details. That will take you to a page that should look something like this.

The "ADD IT" button will be available at this point. Once you add the latest version your extension will be visible. No need to turn the feature off and on.
If you need to do this programmatically like we do, you can use the sharepoint-pnp PowerShell command found here. This is the equivalent of going through the GUI and grabbing the latest application for your site.
The support tech told us they will probably update the official MS documentation to account for this use case; we will see.
Hope this helps!


Example how to set the fields in XAML

I'm missing an example of how to set the fields in XAML code. I would like to define myself a Thickness-property in my App.xaml file.

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

ID: 9654762b-f3d4-6259-2b8e-874d9185be19
Version Independent ID: 52145687-0078-79c2-2235-b24d9c5822c1
Content: Thickness Struct (Windows.UI.Xaml) - UWP app developer
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.ui.xaml.thickness.yml&version=GBlive&_a=contents
Product: windows
Git Login: wdg-dev-content
Microsoft Alias: wdg-dev-content

I will follow up and turn this into a feature request in the Microsoft/microsoft-ui-xaml repo.

Now tracked as a feature proposal on microsoft-ui-xaml.


[FEATURE] Show template values

Is your feature request related to a problem? Please describe.
I need a way to get defined values for a specific template.
It happened to me that I created a project with "zh" while the supposed language meant to be "zh_cn".
$ memsource project template create -f value -c uid --name "SOMETHING" --template-id SOMENUMBER --target-langs ja fr zh_cn
This API call gives that information:
https://cloud.memsource.com/web/docs/api#operation/getProjectTemplate
Describe the solution you'd like
$ memsource project template show --template-id SOMENUMBER -f value -c targetLangs
Describe alternatives you've considered
Additional context

Assigning to @ludekjanda

The feature has been added with this commit: https://github.com/unofficial-memsource/memsource-cli-client/commit/7bd00d1377e6a01d23f29641cb759f59430dc8ff
Wiki link: https://github.com/unofficial-memsource/memsource-cli-client/wiki/Memsource-Template#template-show


Send delivery KPIs (fixes #130)

This PRs sends additional metrics to Datadog to measure the performance of teams. It's based on the  blog post from Stelligent.
Each Job  provides

Lead Time
Cycle Time
Feedback Time
MTTR
MTBF

From https://github.com/stelligent/pipeline-dashboard/blob/master/README.md#metric-details

Looks awesome!! @sjenriquez any chance for a review soon?

@pgarbe Thanks for the PR, will take a look shortly! cc: @miketheman

How close is this to hitting master?

@sjenriquez @pgarbe Thoughts on next steps?

@irabinovitch This PR is mostly there, we can finish it up internally and get it merged.

Thanks again for this @pgarbe! I've done the requested cleanups and merged this in #132 🙇


Feature/printable: Add black and white theme and print mode

Adding support of print  …

Class option print allow to format content for printing
Class option nocolor allow to format content in black and white
Add zds logo for no-color mode
Black and white theme for no-color mode

Base (color):
test.pdf
No-color and print mode:
test-nocolor.pdf

I do not see how to factor that.
If you find a solution I can implement it.


TraceId is N/A in Logback AsyncAppender with TraceIdMDCPatternLogbackLayout

Please answer these questions before submitting your issue.

Why do you submit this issue?
[x] Question or discussion
[x] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?
When I try to print TraceId in log with apm-toolkit-logback-1.x by #1241 #567 demo,
I find it worked will with ConsoleAppender, but always print N/A  with  AsyncAppender .

<!-- logbak.xml -->
<configuration scan="true" scanPeriod=" 5 seconds">
    <appender name="STDOUT" class="ch.qos.logback.core.ConsoleAppender">
        <encoder class="ch.qos.logback.core.encoder.LayoutWrappingEncoder">
            <layout class="org.apache.skywalking.apm.toolkit.log.logback.v1.x.mdc.TraceIdMDCPatternLogbackLayout">
                <pattern>%d{yyyy-MM-dd HH:mm:ss.SSS} [%X{tid}] [%thread] %-5level %logger{56} --- %msg%n</pattern>
            </layout>
        </encoder>
    </appender>

    <appender name="ASYNC_PRINT" class="ch.qos.logback.classic.AsyncAppender">
        <discardingThreshold>0</discardingThreshold>
        <queueSize>1024</queueSize>
        <neverBlock>true</neverBlock>
        <appender-ref ref="STDOUT"/>
    </appender>

    <root level="DEBUG">
        <!-- [TID:N/A] -->
        <appender-ref ref="ASYNC_PRINT"/>

        <!-- [TID:25.118.15585145542260003] -->
        <appender-ref ref="STDOUT"/>
    </root>
</configuration>

Bug

Which version of SkyWalking, OS and JRE?
SkyWalking version "6.1"
java version "1.8.0_161"
OS version  "Widows 10"

Which company or project?
Spring Boot 2.1.X project

What happen?
N/A  instead of TraceID in log.
And I don't know if need other setting or coding or it's a bug.

Because you need to agent toolkit activation to support. We don't support async append, you need to build a new activation to do that

@wu-sheng Can we make it to a help wanted issue, because we also need it.

@zhaoyuguang create a new one if you want. You can label an issue.

@zhaoyuguang create a new one if you want. You can label an issue.

OK


Utilizing bUnits parser and comparer

Hi @SimonCropp et al,
I have been experimenting with Verify.Bunit and have some ideas for improving it, such that it leverages the pretty printer and differ built into bUnit. This ensures that bUnit's blazor:xxx attributes are removed and ignored, and a diff is performed using AngleSharp.Diffing with the same configuration that bUnits "inline snapshot testing" method MarkupMatchces uses.
My attempt at a VerifyBunit type looks like this, with lots of copy paste from the current implementation:
using AngleSharp;
using AngleSharp.Diffing.Core;
using AngleSharp.Dom;
using AngleSharp.Text;
using Bunit.Diffing;
using Bunit.Rendering;
using Microsoft.AspNetCore.Components;
using System.Runtime.CompilerServices;

namespace VerifyTests;

public static class VerifyBunit
{
    private const string BunitFileExtension = "html";

    public static bool Initialized { get; private set; }

    public static void Initialize(bool verifyMarkupOnly = true)
    {
        if (Initialized)
        {
            throw new("Already Initialized");
        }

        Initialized = true;

        InnerVerifier.ThrowIfVerifyHasBeenRun();

        VerifierSettings.RegisterFileConverter<IRenderedFragment>(verifyMarkupOnly ? ConvertFragmentMarkupOnly : ConvertFragment);
        VerifierSettings.AddExtraSettings(settings => settings.Converters.Add(new RenderedFragmentConverter()));
        VerifierSettings.RegisterStringComparer(BunitFileExtension, BunitMarkupComparer);
    }

    private static Task<CompareResult> BunitMarkupComparer(string received, string verified, IReadOnlyDictionary<string, object> context)
    {
        using var parser = new BunitHtmlParser();
        var receivedNodes = parser.Parse(received);
        var verifiedNodes = parser.Parse(verified);
        var diffs = receivedNodes.CompareTo(verifiedNodes);

        var result = diffs.Count == 0
            ? CompareResult.Equal
              : CompareResult.NotEqual(CreateDiffMessage(received, verified, diffs));

        return Task.FromResult(result);
    }

    private static string CreateDiffMessage(string received, string verified, IReadOnlyList<IDiff> diffs)
    {
        var builder = StringBuilderPool.Obtain();
        builder.AppendLine();
        builder.AppendLine("HTML comparison failed. The following errors were found:");

        for (int i = 0; i < diffs.Count; i++)
        {
            builder.Append($"  {i + 1}: ");
            builder.AppendLine(diffs[i] switch
            {
                NodeDiff diff when diff.Target == DiffTarget.Text && diff.Control.Path.Equals(diff.Test.Path, StringComparison.Ordinal)
                    => $"The text in {diff.Control.Path} is different.",
                NodeDiff diff when diff.Target == DiffTarget.Text
                    => $"The expected {NodeName(diff.Control)} at {diff.Control.Path} and the actual {NodeName(diff.Test)} at {diff.Test.Path} is different.",
                NodeDiff diff when diff.Control.Path.Equals(diff.Test.Path, StringComparison.Ordinal)
                    => $"The {NodeName(diff.Control)}s at {diff.Control.Path} are different.",
                NodeDiff diff => $"The expected {NodeName(diff.Control)} at {diff.Control.Path} and the actual {NodeName(diff.Test)} at {diff.Test.Path} are different.",
                AttrDiff diff when diff.Control.Path.Equals(diff.Test.Path, StringComparison.Ordinal)
                    => $"The values of the attributes at {diff.Control.Path} are different.",
                AttrDiff diff => $"The value of the attribute {diff.Control.Path} and actual attribute {diff.Test.Path} are different.",
                MissingNodeDiff diff => $"The {NodeName(diff.Control)} at {diff.Control.Path} is missing.",
                MissingAttrDiff diff => $"The attribute at {diff.Control.Path} is missing.",
                UnexpectedNodeDiff diff => $"The {NodeName(diff.Test)} at {diff.Test.Path} was not expected.",
                UnexpectedAttrDiff diff => $"The attribute at {diff.Test.Path} was not expected.",
                _ => throw new SwitchExpressionException($"Unknown diff type detected: {diffs[i].GetType()}"),
            });
        }

        builder.AppendLine();
        builder.AppendLine("Actual HTML:");
        builder.AppendLine();
        builder.AppendLine(received);
        builder.AppendLine();
        builder.AppendLine("Expected HTML:");
        builder.AppendLine();
        builder.AppendLine(verified);

        return builder.ToPool();

        static string NodeName(ComparisonSource source) => source.Node.NodeType.ToString().ToLowerInvariant();
    }

    private static ConversionResult ConvertFragmentMarkupOnly(IRenderedFragment fragment, IReadOnlyDictionary<string, object> context)
    {
        var markup = fragment.Nodes.ToHtml(DiffMarkupFormatter.Instance).Trim();
        return new ConversionResult(null, BunitFileExtension, markup);
    }

    private static ConversionResult ConvertFragment(IRenderedFragment fragment, IReadOnlyDictionary<string, object> context)
    {
        var markup = fragment.Nodes.ToHtml(DiffMarkupFormatter.Instance).Trim();
        var allNodes = fragment.Nodes.Select(x => x.GetDescendantsAndSelf().Count()).Sum();
        var info = new ComponentInfo(GetInstance(fragment), allNodes);
        return new ConversionResult(info, BunitFileExtension, markup);
    }

    private static IComponent? GetInstance(IRenderedFragment fragment)
    {
        var type = fragment.GetType();
        if (!type.IsGenericType)
        {
            return null;
        }

        var renderComponentInterface = type
            .GetInterfaces()
            .SingleOrDefault(_ =>
                _.IsGenericType &&
                _.GetGenericTypeDefinition() == typeof(IRenderedComponentBase<>));

        if (renderComponentInterface == null)
        {
            return null;
        }

        var instanceProperty = renderComponentInterface.GetProperty("Instance")!;
        return (IComponent)instanceProperty.GetValue(fragment)!;
    }

    private sealed class ComponentInfo
    {
        public string? Component { get; }

        public IComponent? Instance { get; }

        public int NodeCount { get; }

        public ComponentInfo(IComponent? instance, int nodeCount)
        {
            Component = instance?.GetType().FullName;
            Instance = instance;
            NodeCount = nodeCount;
        }
    }

    private sealed class RenderedFragmentConverter : WriteOnlyJsonConverter<IRenderedFragment>
    {
        public override void Write(VerifyJsonWriter writer, IRenderedFragment fragment)
        {
            writer.WriteStartObject();

            writer.WriteMember(fragment, GetInstance(fragment), "Instance");

            writer.WriteMember(fragment, fragment.Markup, "Markup");

            writer.WriteEndObject();
        }
    }
}

The above code compiles with the following package references:
Verify.Xunit
bunit

A few questions/comments:

RenderedFragmentConverter does not seem to be used?
It seems redundant to convert fragment.Nodes to markup just to convert them back to nodes again in the comparer. But as far as I can tell there is no way around this with Verify, right?
As mentioned, this utilizes the differ built-in to bUnit. That has the advantage of avoiding tests breaking if bUnit internally makes changes, and it handles some of the special attributes bUnit adds to the output.
This defaults to using the semantic comparer (based on Anglesharp.Diffing) that bUnit comes with, which has been configured to ignore bUnits attribute in the markup, and it includes a custom node matcher that is needed in some circumstances.
Ideally, users should not be asserting against the internal state of a component. Even though the component is a public thing, accessing its parameters and other public properties is not considered good practice in frontend testing, since you are interacting and accessing the component in a way that the end user will not. The end user will look at the markup only. Thus, by default, I suggest only verifying the produced markup.
I did attempt to set BunitFileExtension to bunit, such that the compare would only be used with it, avoiding any conflicts with non-IRenderedFragment compares that may be registered. But it does not seem to be possible.

I am not super familiar with Verify so I wanted to dump my suggestion here instead of opening a PR.

RenderedFragmentConverter does not seem to be used?

Used here https://github.com/VerifyTests/Verify.Blazor/blob/main/src/Verify.Bunit/VerifyBunit.cs#L28

It seems redundant to convert fragment.Nodes to markup just to convert them back to nodes again in the comparer. But as far as I can tell there is no way around this with Verify, right?

unfortunately no. since comparers are designed to work on the rendered result.  but perf wise it is usually not an issue since the comparer on files if the text diff resolves to notequal

As mentioned, this utilizes the differ built-in to bUnit. That has the advantage of avoiding tests breaking if bUnit internally makes changes, and it handles some of the special attributes bUnit adds to the output.

Yep i like this approach

This defaults to using the semantic comparer (based on Anglesharp.Diffing) that bUnit comes with, which has been configured to ignore bUnits attribute in the markup, and it includes a custom node matcher that is needed in some circumstances.

I like it. was not aware buint had a comparer

Ideally, users should not be asserting against the internal state of a component. Even though the component is a public thing, accessing its parameters and other public properties is not considered good practice in frontend testing, since you are interacting and accessing the component in a way that the end user will not. The end user will look at the markup only. Thus, by default, I suggest only verifying the produced markup.

Not sure i agree with that. component state can be helpful in a snapshot. the rendered result can somewhat obscure the state, and also sometime omit some useful info. But i am happy to have a option to exclude component state .

I did attempt to set BunitFileExtension to bunit, such that the compare would only be used with it, avoiding any conflicts with non-IRenderedFragment compares that may be registered. But it does not seem to be possible.

yeah that would be difficult. Verify is aware of "what extensions are text" and treats them differently. but it should be possible. I will take a look at it

I am not super familiar with Verify so I wanted to dump my suggestion here instead of opening a PR.

Based on the above i think a few smaller PRs would work better. so we can discuss the specifics of each individual change
Alternatively. I can do a PR to the bunit codebase that moves this project into yours, and make u the owner of the Verify.BUnit nuget. Obviously still happy to help u out, but it would mean you can take Verify.BUnit any direction u want.
For context when i created Verify.BUnit, i was primarily working in blazor. Now i am back on full time backend work, so not doing any blazor.


ValueError: Layer #0 (named "efficientnetb0" in the current model) was found to correspond to layer efficientnet-b0 in the save file. However the new layer efficientnetb0 expects 312 weights, but the saved weights have 309 elements.

Hi, I am trying to train on a TPU with Kaggle, save the weights, and then predict on a GPU.  The problem is that whenever I save my model's weights and try to load them up again I get an error like this:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-30-44f514f0c1e1> in <module>
      4     test_ids_ds = TEST.map(lambda image, idnum: idnum).unbatch()
      5
----> 6     models = [get_model(model, EFFICIENTNET_MODELS[model]) for model in EFFICIENTNET_MODELS]
      7     predictions = [model.predict(test_images_ds) for model in models]
      8     results = pd.DataFrame(sum(predictions) / len(predictions))

<ipython-input-30-44f514f0c1e1> in <listcomp>(.0)
      4     test_ids_ds = TEST.map(lambda image, idnum: idnum).unbatch()
      5
----> 6     models = [get_model(model, EFFICIENTNET_MODELS[model]) for model in EFFICIENTNET_MODELS]
      7     predictions = [model.predict(test_images_ds) for model in models]
      8     results = pd.DataFrame(sum(predictions) / len(predictions))

<ipython-input-29-6bdf6c1e54c8> in get_model(model_name, weight_path)
    100                           metrics=[tf.keras.metrics.AUC(name='AUC', multi_label=True)])
    101
--> 102             model.load_weights(weight_path)
    103
    104             return model

/opt/conda/lib/python3.7/site-packages/tensorflow/python/keras/engine/training.py in load_weights(self, filepath, by_name, skip_mismatch, options)
   2209             f, self.layers, skip_mismatch=skip_mismatch)
   2210       else:
-> 2211         hdf5_format.load_weights_from_hdf5_group(f, self.layers)
   2212
   2213   def _updated_config(self):

/opt/conda/lib/python3.7/site-packages/tensorflow/python/keras/saving/hdf5_format.py in load_weights_from_hdf5_group(f, layers)
    704                        str(len(symbolic_weights)) +
    705                        ' weights, but the saved weights have ' +
--> 706                        str(len(weight_values)) + ' elements.')
    707     weight_value_tuples += zip(symbolic_weights, weight_values)
    708   K.batch_set_value(weight_value_tuples)

ValueError: Layer #0 (named "efficientnetb0" in the current model) was found to correspond to layer efficientnet-b0 in the save file. However the new layer efficientnetb0 expects 312 weights, but the saved weights have 309 elements.

I have tried using another EfficientNet model to see if it was just the one set of weights, but I get the same error, with an expected amount of weights being 3 more than provided in the weight file.  Here is the code I use to load my weights:
EFFICIENTNET_MODELS = {'b0': '../input/ranzcr-weights/efficientnetb0.h5',
                                                'b1': '../input/ranzcr-weights/efficientnetb1.h5',
                                                'b2': '../input/ranzcr-weights/efficientnetb2.h5',
                                                'b3': '../input/ranzcr-weights/efficientnetb3.h5',
                                                'b4': '../input/ranzcr-weights/efficientnetb4.h5',
                                                'b5': '../input/ranzcr-weights/efficientnetb5.h5'}

def get_model(model_name, weight_path):
        with strategy.scope():
            if model_name == 'b0':
                EfficientNet = tf.keras.applications.EfficientNetB0(
                    include_top=False,
                    weights=None,
                    input_shape=(*IMG_SIZE,3)
                )
            if model_name == 'b1':
                EfficientNet = tf.keras.applications.EfficientNetB1(
                    include_top=False,
                    weights=None,
                    input_shape=(*IMG_SIZE,3)
                )
            if model_name == 'b2':
                EfficientNet = tf.keras.applications.EfficientNetB2(
                    include_top=False,
                    weights=None,
                    input_shape=(*IMG_SIZE,3)
                )
            if model_name == 'b3':
                EfficientNet = tf.keras.applications.EfficientNetB3(
                    include_top=False,
                    weights=None,
                    input_shape=(*IMG_SIZE,3)
                )
            if model_name == 'b4':
                EfficientNet = tf.keras.applications.EfficientNetB4(
                    include_top=False,
                    weights=None,
                    input_shape=(*IMG_SIZE,3)
                )
            if model_name == 'b5':
                EfficientNet = tf.keras.applications.EfficientNetB5(
                    include_top=False,
                    weights=None,
                    input_shape=(*IMG_SIZE,3)
                )

            EfficientNet.trainable = True

            model = Sequential([EfficientNet,
                                Flatten(),
                                Dense(11, activation='sigmoid')])

            model.compile(optimizer='adam',
                          loss='binary_crossentropy',
                          metrics=[tf.keras.metrics.AUC(name='AUC', multi_label=True)])

            model.load_weights(weight_path)

            return model

models = [get_model(model, EFFICIENTNET_MODELS[model]) for model in EFFICIENTNET_MODELS]

For saving the weights, I simply set up the model in the same way and then train and save the weights.  Is this issue in my code or is it an issue with EfficientNet?  How can I fix this?

Don't save the weights, I had the same problem. Also Dont save it in .h5 format also. Save the model as a tensorflow checkpoint model. model.save("path",include_optimizer=False)`

Don't save the weights, I had the same problem. Also Dont save it in .h5 format also. Save the model as a tensorflow checkpoint model. model.save("path",include_optimizer=False)`


Set compare build id

Hi, Richard.
In our release pipeline, the change may need to be rolled out for enterprise clients. In this case, I wish the work items could be getting from a specific build to the current one instead of the last successful one.
Is your feature request related to a problem? Please describe.
Considering work items get from Azure API get work items between builds
 GET https://dev.azure.com/{organization}/{project}/_apis/build/workitems?fromBuildId={fromBuildId}&toBuildId={toBuildId}&api-version=6.0-preview.2

In log file
Getting the current build details
##[debug]System.StageName=__default
Getting items associated the builds since the last successful build to the stage '__default'
Comparing 132961 against 132961
Ignore compare against self
Comparing 132960 against 132961
Considering build

When I set check stage == true, it will be compared with the last successful build
But the last successful build may not run in the same branch as the current one.
Describe the solution you'd like
Could I set {fromBuildId} in the build pipeline?
Additional context
Also, I noticed the advanced Name of the stage (override) -- System.StageName
In multiple-stage pipeline could I set both {fromBuildId} and {System.StageName} ?
Thanks for your help,
Joanna.

I can't see a technical reason to not add this feature. My only concern would be the operational one if it bit a bit of a manual overhead to keep updating the new fromBuildId parameter.

+1 for this feature as a bit of safety net. A couple of times we've had dev's switch branches and the release notes go haywire as a result. It would be useful after that happens to be able specify what build to compare against (otherwise continue to default to current behaviour).

The people have spoken, I will push this feature up the backlog

I have implemented an override for both multistage YAML builds and UI based release. The new parameter (on the Advanced menu) accepts an build or release ID e.g:
overrideBuildReleaseId  = 1234

For releases this value is the release ID of the 'last good release' to compare against, for multi-stage YAML it is the build ID.
Note that

If this value is not specified, the default, then the good build/release logic is used
If this value cannot be parsed to an int as entered, then the task will exit with an error
If this specified release/build is not found then the task will exit with an error
No check is made as to the success for failure if the specified build/release - it is assumed the administrator know what they are doing.

The new version is currently in test but will be released as 3.21.x


Webserver static dir can be configured through configuration

The idea is to make it possible to configure the webserver static dir.
The variable __dirname points to the directory where the SlackBot.js file resides which is usually inside node_modules.
var controller = Botkit.slackbot({
      webserver: {
        static_dir: __dirname + '/public'
      }
    });

An alternative (which actually seems better) is to add a new parameter to the setupWebserver function.

Seems like it solves #136

Thanks @fabito, can you also update the docs?

Sure.
First I will move the configuration parameter to the setupWebserver function.
I was thinking on adding a config param which would hold the existing port plus the static_dir.
Something like this:
var config = {
    port: 8080,
    static_dir: __dirname + '/public'
};
controller.setupWebServer(config, cp);

If backward compatibility is an issue, we could check whether config is an object or a number and handle both cases.
What do you think ?

Seems good to me. @benbrown what do you think?


Added more UE4 blueprint support

Added blueprint support for JoinRequest() and Respond, added latent
blueprint node for getting Discord profile images

I will try to fix the line endings here, sorry


Question related to the project?

Hey! I'm facing issues to setup database locally cause I never use SQL database before ''m only use mongodb But I know mySQL syntax. Will you tell me step by step how do I setup db locally?

Okay, so,

Install MySQL to your local machine.
Check if you are able to connect to the mysql client with your credentials or not.
If you are able to connect to it then create a .env file in the project folder and set it up in the same format as in .env.example. Add your personal mysql credentials to their respective places in your .env file.
After that just follow the instructions mentioned in guidelines written.

If you have trouble running the .sql scripts then just open up your mysql client and then run the two scripts with their respective file paths like
source <file path>

@priyanshuSharma-WebDev did this help?

i have some trouble with running the backend ( i think it has something to do with the env). What would be the host value in the .env for example?

Great to hear that, though it can also be 'localhost'

@priyanshuSharma-WebDev should I close the issue then?

Yeah! It's work, Now I'm ready to familiarize myself with codebase.

One more thing! Have you ever try to make a real time collaboration webapp ex: google docs cause I want to make something like this?

Yeah! It's work, Now I'm ready to familiarize myself with codebase.

Great, I am glad to hear that and am looking forward to some really cool contributions from your side 😄.

One more thing! Have you ever try to make a real time collaboration webapp ex: google docs cause I want to make something like this?

I haven't done anything like that up till now, though I have worked a little with Web Sockets for a chat application


Preview

Preview #1274
Open
bgoonz wants to merge 9 commits into master from preview
+4 −15
Conversation 0
Commits 9
Checks 2
Files changed 5
Conversation
@bgoonz
Owner
bgoonz commented now
No description provided.
bgoonz and others added 9 commits 11 minutes ago
@bgoonz
Merge pull request #1273 from bgoonz/master
8b5da7f
@stackbit-projects
bookmarks.md: added
cfdf217
@stackbit-projects
dependencies.md: added
2b54b88
@stackbit-projects
free-certifications.md: added
a8efe6f
@stackbit-projects
sitemap-nov.md: added
c2afc0c
@stackbit-projects
site-metadata.json: changed
ab4be61
@stackbit-projects
site-metadata.json: changed
e449f0a
@stackbit-projects
site-metadata.json: changed
07c44d2


Cassandra Sink for Cassandra 3.x?

Hello,
When trying to use the Cassandra Sink, I'm getting this error (according to others) because I am using Cassandra 3.x.
"com.datastax.driver.core.exceptions.InvalidQueryException: unconfigured table schema_keyspaces"
Apparently as of Cassandra 3.x the table "system_schema.keyspaces" is used.
Do you have plans for a new or updated Cassandra Sink to allow for use with Cassandra 3.x?
Thank you!

Hello,
Just for information, if you don't already try this: Just replacing the cassandra driver in lib folder of stratio-ingestion with cassandra3's one: cassandra-driver-core-3.0.1-shaded.jar worked for me


Merge in crawler

Merge the private crawler repo into the public sycamore repository.
Three main steps:

Prepare crawler repo by moving all files into a subdirectory
Create the merge of unrelated histories between the two repos
Fix the resulting merge so that it works; including some bugfixes that were missed in the crawler repo.

Fixing issue and re-creating pull request to minimize history complications.


SPM error: Package.swift has no Package.swift manifest for version 0.7.0

I can't add Firebase-UI with SPM.
Environment

Xcode: 13.2.1
Mac: M1 Pro
Firebase SDK version: 8.11
FirebaseUI version: 12.1.1

Steps to reproduce:

Add Firebase-UI with SPM: https://github.com/firebase/FirebaseUI-iOS

Observed Results:
Package.swift has no Package.swift manifest for version 0.7.0 in https://github.com/firebase/FirebaseUI-iOS

Hi,
I see the same issue and not sure where 0.7.0 is coming from.
But if you set the Package Repository Rule setting in "Package Dependency" as "Version : [Up to Next Major] 12.1.1 > 13.0.0,  It should resolved the issue. Please try.

Hey @kfujino7, it worked thanks!


add flush support

Fixes #20 and it also fixes the issue with inputs like http where each received payload will always yield one or many json lines but may not contain a delimiter on the last one. These inputs need to call flush after each request to make sure any buffers are emptied.
This codec will now be consistent with the line codec which also support the flush method in the same way.
Also, for reference, this flush "semantic" was discussed in elastic/logstash#8830

could this be considered a breaking change? are there other places where we've done this already?

@jsvd this is exactly the same idea as with the line codec, any/all delimited content codec should support the flush method for situation such as logstash-plugins/logstash-input-http#81 or https://github.com/logstash-plugins/logstash-input-udp/commit/ecf93e1a82f820fef7166517aa2926e206d8abfb#diff-f55c7d32f8cd66e4be73265d1b1d2342R122
In essence, this is not a breaking change but a fix to a potential data loss or corruption problem with inputs such as udp or http.

Thanks @jsvd
bumped version to 3.0.6, will merge and publish.


Split package into "vec" and "vec-lib".

I also fixed some incidental things noted in the commit messages.

Fantastic!  Thanks for putting this together!


Issue497 update oceanpy markdowns v0.8.1

Towards https://github.com/oceanprotocol/ocean.py/issues/497.
Changes proposed in this PR:

Update ocean.py markdowns to v0.8.1

In this PR, I attempted checking out the v0.8.1 tag before generating the new markdowns. But, this yielded a URL with HEAD as the branch name.
source: https://github.com/oceanprotocol/ocean.py/blob/HEAD/README.md

Seeing this, I realized that using the branch name (main) to generate these source URLs will send users to documentation from the latest commit on the main branch which may or may not be released.
Using the tag name would be better because the generated source URLs will always lead to the matching, released documentation.
I created #31 to implement a fix and I'm going to mark this issue as a draft until #31 has been fixed and I can regenerate the markdowns with a tag instead of branch name.


Update code example in step 63 of calorie counter project to show NaN

Describe the Issue
In step 63, we show the number constructor with an example checking for a valid number.
But we also mention, NaN so it would be nice to add to the example.
updated description code example for step 63
The `Number` constructor is a function that converts a value to a number. If the value cannot be converted, it returns `NaN` which stands for "Not a Number".

Here is an example:

```js
Number('10'); // returns the number 10
Number('abc'); // returns NaN
```

Affected Page
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/learn-form-validation-by-building-a-calorie-counter/step-63
Your code
see explanation above
Expected behavior
see explanation above
Screenshots
No response
System

Device: [e.g. iPhone 6, Laptop]
OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 22]

Additional context
No response

This has been opened for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors.
If you would like to contribute and have not read the contributors docs, please do so here: https://contribute.freecodecamp.org/#/
If you have any issues with contributing, be sure to join us on the contributors channel, or on the contributors sub-forum
Files that need to be changed
https://github.com/freeCodeCamp/freeCodeCamp/blob/main/curriculum/challenges/english/15-javascript-algorithms-and-data-structures-22/learn-form-validation-by-building-a-calorie-counter/63c9bce376ca4f09c15a3768.md
Happy coding


Disabled serialization for JSON type columns

The default migration uses the JSON type for databases (such as Postgres) that support it. The concern no longer attempts to invalidly serialize the tokens field if it is backed by a JSON column. This fixes the TypeError encountered when the JSON field is nil (as it is by default). The tests have also been updated to test the JSON column type if it is available. I have also bumped the postgresql version on Travis CI to 9.3 so JSON datatypes would be available to test.
This also resolves #271 by explicitly checking if the tokens field is nil before attempting to delete expired tokens.

This is great @colavitam. Thanks for this PR!


Improvement for issue #1303

Refactor OpenAPIV3Parser to make it more clear
Fixes #1303

@mih-kopylov Thanks a lot! This is included in #1380 which adds some minor updates in #1380 for compatibility


⚠ Minecraft has degraded performance

In 1fd9eeb, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1061 ms

Resolved: Minecraft performance has improved in 45e7a87.


[Request for External] codeunit 90 "Purch.-Post" OnPostItemJnlLineOnAfterPrepareItemJnlLine

Describe the request
Can you include QtyToBeInvoiced as parameter in event OnPostItemJnlLineOnAfterPrepareItemJnlLine in codeunit 90 "Purch.-Post"?
Additional context
Need it in addition to QtyToBeReceived
Internal work item: AB#561094

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 28412.


Get entitydetail list

New method for get all EntityDetails in local-graph-repository (Janusgraph).

Hi @Mirovan , can we close this pr?

Hi @Mirovan , can we close this pr?

Hi, I think yes.

Closing as agreed above


Is there Android Auto compatibility?

It would be very useful and cool, if not could you add it?

Not at the moment, but I'm keeping this open as a suggestion I might add in the future


Trim dependency graph by removing unused stuff

Which issue this PR addresses:
Fixes none, it is maintanence PR reducing noise in dependency graph
What this PR does / why we need it:
Removes old unused replace directives.
Test plan for issue:

Is there any documentation that needs to be updated for this PR?

@petrkotas any reason /hold is still needed?

@bennerv Uli wanted to merge after release, jut to be extra safe.

I'm good to merge + remove hold label if you are now.  I think we're well past release + changes here.


Update/Add Miniconda to windows-2022 image

Tool name
Miniconda
Tool license
same as what's on windows-latest
Add or update?

[X] Add
[ ] Update

Desired version
same as what's on windows-latest
Approximate size
No response
Brief description of tool
Miniconda is missing from the windows-2022 image
URL for tool's homepage
No response
Provide a basic test case to validate the tool's functionality.
No response
Virtual environments affected

[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11
[ ] Windows Server 2016
[ ] Windows Server 2019
[X] Windows Server 2022

Can this tool be installed during the build?
No response
Tool installation time in runtime
No response
Are you willing to submit a PR?
No response

Hi @MichaelGoulding!
Could you please clarify — do you use conda in AzureDevOps or GH actions? I'm asking since there is a popular action for GH to install and prepare conda environment in runtime https://github.com/marketplace/actions/setup-miniconda

@MichaelGoulding the changes have been merged. The new image will be deployed next week. I'll let you know once the deployment is finished

Should be finished as per https://github.com/actions/virtual-environments/commit/1fc7895bd9e469460c143cabef1e85a848b7a39c#diff-a1f67f5c4a583bf90c239743379ad9473fbf1fa9d11c5cb7739c8012404dcc7a


Problem using navbar fixed-top

If my template has a  structure, then the subsequent  is hidden behind it. In order to  align , I need to specify ? Or is there a more universal solution?
For example, if I use the solution suggested by me, then when I increase any element related to , I should also change the margin-top from 6rem to large.
How right?

Our docs covers this: https://getbootstrap.com/docs/4.1/components/navbar/#placement.