Frame independent movement adjusted by setting velocity

delta Time added, movement (hopefully) frame-indepentent from now on?

feel free to check and test if U have time, I don't wanna F this shit up

I will have some comments and will request some changes to be done, but I don't want to spoil the fun :D It is a very big and important (relative to the current size of the project) change, I will probably want to use it as an example to talk about a couple of important principles related to various gamedev topics. So please don't merge this in for now :)
But from a brief look it should work! If anyone has some spare time, please check the Files changed tab above and review the change, and/or test that locally and provide feedback to Ania.
Nevertheless, great job! It was a tough and tricky one :)

Oke, it just dawned on me that I could have handled the whole delta time thingy differently. Sooo, correct me if I'm wrong cuz that can be just a misguided notion, but I probably should've just created a global variable dTime instead?
then update it in the Main function once the clock restarts and use as needed inside class functions?
That way I wouldn't have to change the signatures and calls of all Update() functions in all classes derived from Entity? I would just need to add one variable and everything should work smoothly...
that is my working theory at least, I don't wanna change anything just yet, in case I would just meddle unnecessarily

well, I would expect the frame rate control to work, i reset the clock every time the entire frame update process begins and during that time i use the duration of the previous frame as dt(assuming that most frames would have similar length) after sorting everything out, I reset dt to 0 and the add the elsapsed time to it. This way I know how long the entire process took. and then the loop begins again and I check if dt is greater than desired frame length. If so, I repeat the entire process described above, otherwise I just add the time to dt.
I would expect this logic to regulate frame rate, and that, I suppose, it does.
the strange part comes in later:
changing frameDur to a greater value for some reason speeds up all movement and similarily, decreasing frameDur slows things down impossibly. Now, why would that be if I multiply all velocities by dt anyway, and should expect movement speed to remain consistent regardless of the number of frames and their refresh rate?
I'm probably missing something obvious here or have overcomplicated things, but what am I doing wrong? also, I hope this description or whatevs is clear enough, Idk how to explain myself more clearly considering I'm not even sure what the issue is lmao
thx

The attempted solution doesn't work because you are discarding the remaining time that elapsed - namely, you accumulate the elapsed time into dt but once it reaches that magical value for 60fps, you just do dt = 0.f. In case a lag or hitch happens, the dt can be so high that stuff will appear to teleport on screen or smth :D
Also, you count elapsed time incorrectly.  You should either do dt = elapsedTime or dt += reset.
And you introduced global dt, which is a no-no :D
I suggest the following:

the desired timeframe duration can be calculated instead of writing a magical value

constexpr float desiredFramerate = 60.f;
constexpr float desiredDt = 1 / desiredFramerate;

At the beginning of while (window.isOpen()) loop we accumulate the elapsed time
We then have an inner while loop, that checks if the accumulated time is > than desiredDt
So we enter this inner while only when the accumulated time is larger than our desired framerate. But! We don't zero out the elapsed time inside, instead we deduct desiredDt

 elapsedTime -= desiredDt;

In the update function, we simply pass desiredDt. That means that we will have a fixed delta - no matter what we will always pass the exact same value. Thanks to that, no matter if we have 10 or 999 fps, update will always recieve the same delta time, and thanks to the inner while loop that is subtracting dt from elapsed time, in case of a hitch or lag we simply update the game few times in a row, instead of doing one big update with giant delta that could mess things up :p

Approving review as I don't want to force you to use my solution just because - if yours works then it works :) I would say it's ready for testing!


Unable to explicitly target Aura bundles in .forceignore

I'm trying to be very specific in my forceignore (so I don't pull down things from sratch orgs that I don't want - e.g. all the bits created if I create a community declaratively), but in doing so I've realised I can't specifically target Aura bundles, and as such I open up some dangerous wildcarding.
For example;
Version: sfdx-cli/7.48.0-a71f8c6899 linux-x64 node-v10.15.3
I can target everything else with filename.extension type syntax (e.g Default_Navigation.navigationMenu), but for Aura I have to just supply the name of the directory of the component (e.g. loginForm - which is one of the components auto generated by Salesforce).
If I use loginForm though this also means anything else starting with loginForm to be ignored (e.g. loginForm.cls, or loginFormUpdated.page).
I might be missing some trick to specifically target Aura bundles, but my GoogleFu is failing me.

@toddhalfpenny I would recommend posting your question in the SFDX Trailblazer community


Use ADS name for telemetry log

Also updating text to match latest from VS Code
https://github.com/microsoft/vscode/blob/main/src/vs/platform/telemetry/common/telemetryLogAppender.ts#L27

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.009%) to 42.328%

Files with Coverage Reduction
New Missed Lines
%

src/sql/workbench/services/connection/browser/connectionDialogWidget.ts
1
73.05%

Totals

Change from base Build<PHONE_NUMBER>:
0.009%

Covered Lines:
29330

Relevant Lines:
64481

💛  - Coveralls


Add more faq entries.

Can you change
XP is earned by talking in all channel except the bots category in this server. It is awarded once per minute, with a random value of 1-10. Leveling up is determined from multiplying your level by 50. I.e. a person of level 5 needs 250 XP. To check your xp and level, do the b!rank command or b!leaderboard for all.
to
XP is earned by talking in all channels except the "bots" category in this server. It is awarded once per minute, with a random value of 1-10. Levelling up is determined from multiplying your current level by 50. I.e. a person of level 5 needs 250 XP. To check your XP and level, run the b!rank command or b!leaderboard for all.
Just grammar and spelling fixes.

Can you change
XP is earned by talking in all channel except the bots category in this server. It is awarded once per minute, with a random value of 1-10. Leveling up is determined from multiplying your level by 50. I.e. a person of level 5 needs 250 XP. To check your xp and level, do the b!rank command or b!leaderboard for all.
to
XP is earned by talking in all channels except the "bots" category in this server. It is awarded once per minute, with a random value of 1-10. Levelling up is determined from multiplying your current level by 50. I.e. a person of level 5 needs 250 XP. To check your XP and level, run the b!rank command or b!leaderboard for all.
Just grammar and spelling fixes.

done

done


hri_face_body_matcher: 2.0.3-1 in 'humble/distribution.yaml' [bloom]

Import package(s) in repository hri_face_body_matcher 2.0.3-1:

upstream repository: https://github.com/ros4hri/hri_face_body_matcher.git
release repository: https://github.com/ros4hri/hri_face_body_matcher-release.git
distro file: humble/distribution.yaml
bloom version: 0.10.7
previous version for package: null

hri_face_body_matcher
hri_face_body_matcher is a ROS4HRI -compliant node that attempts to match people faces to their detected bodies.
Check here for the complete CHANGELOG.

Holding for Humble sync

I'm not sure if we should accept this because of the license. Asking...

Hey @severin-lemaignan, we can't accept that license on the buildfarm. If you can change it to a common open source license, we can accept it.

Given that there's been no activity here for a while, I'm going to close this out.  Feel free to reopen once the LICENSE situation has been sorted.

@clalancette -- following up in #43548 as I can not re-open this PR and github does not let me update it by pushing to the same branch.

hi @tfoote @audrow -- license updated to Apache 2 in hri_face_body_matcher 2.1.0. New PR replacing this one: #43549


Legacy Facebook adapter should have configurable API version

The api version is set in an internal constant, but should be settable (and possibly updated to LTS version of the API)

possible fix in https://github.com/howdyai/botkit/pull/1683


Finish splitting tests for different metrics sources

What does this PR do?

Management and prometheus (openmetrics) plugin tests are completely separate now.
Motivation

Paying down tech debt :)
Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached
[ ] If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

Current dependencies on/for this PR:

master

PR #13957   👈

PR #13960

This comment was auto-generated by Graphite.


Install cryptography

The dependency on cryptography was added to SROS in https://github.com/ros2/sros2/pull/124
CI scripts were updated in https://github.com/ros2/ci/pull/306/

Q: Should I also update the Fedora instructions or does rosdep cover that? (there is a rule for it)


[Improvement]  Exit on OutOfMemoryError

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

What would you like to be improved?
At present, if OutOfMemoryError is encountered, server will not exit.
How should we improve?
Add -XX:+ExitOnOutOfMemoryError and -XX:+CrashOnOutOfMemoryError to JVM argments.
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

PTAL @jerqi @zuston @advancedxy

PTAL @jerqi @zuston @advancedxy

Do we need dump the memory when OOM occurs。

PTAL @jerqi @zuston @advancedxy

Do we need dump the memory when OOM occurs？

The dump file maybe too big and disk space may be not enough to store it. So users should specify the path to store it.

The bin/start-shuffle-server.sh may handle that as a cli argument or env variable.

Good idea. This should be supported.

Requesting permission to contribute to this feature. . Can you please advise on how I can get involved? Thanks
I assign this issue to you.

@AdarshRawat1 You can refer to https://github.com/apache/incubator-uniffle/blob/master/CONTRIBUTING.md
If you have any question, you can ask me.

Could you kindly provide me with additional resources and guidance [files to change, Tests to perform], please?

files to change
https://github.com/apache/incubator-uniffle/blob/master/bin/start-coordinator.sh
https://github.com/apache/incubator-uniffle/blob/master/bin/start-shuffle-server.sh
2.test to perform
You build package and unzip it, set environment variable (JAVA_HOME and HADOOP_HOME) and run the cmd
sh bin/start-shuffle-server.sh and sh bin/start-coordinator.sh. If the process start successfully, the change is ok.

@AdarshRawat1 Do you need any other help from me?

@AdarshRawat1 Do you need any other help from me?

@jerqi I’m currently occupied with my mid-term exams. Once I’m done with them, I’ll address this issue. I would appreciate your valuable guidance at that time. Thank you.

@AdarshRawat1 Do you want to continue this issue?

@jerqi I deeply regret the delay and will commence work on it today. I look forward to receiving your guidance throughout the process.

@jerqi I apologize, but I'm unable to work on this issue.


I cannot use the keystroke "f" inside the editor

I cannot use the keystroke "f" inside the editor at all - so if I type any word  with "f" it is ignored and not displayed

Thank you Greg followed your clear steps turned out to be me creating a shortcut F I for flutter inspector to be the issue - changed it and problem (I caused) solved! Thank you!


CAMEL-19853 - Camel-Thymeleaf: Fix Camel Catalog test and review the …

…syntax
Description

Target

[x] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[x] I checked that each commit in the pull request has a meaningful subject line and body.

[x] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

/component-tests camel-thymeleaf


Author Name Hyperlinked to HydroShare Account of Resource Creator

I have created a resource and re-assigned to a different author who is not a HydroShare user. However, that author's name is hyperlinked to my HydroShare account. The hyperlink on the author's name should be removed.

To be clear - "Elizabeth Ogata", the author's name, is linked to the profile page for the "iUTAH Data Manager" HydroShare user. "Elizableth Ogata" is not a HydroShare user.
@AmberSJones - I'm assuming that when you originally created this resource, you did it under the "iUTAH Data Manager's" account and that you then removed "iUTAH Data Manager" as the author and then added "Elizabeth Ogata" and "Michelle Baker" - neither of which is a HydroShare user.  Is this correct?

Yes, @horsburgh is correct. Thanks for clarifying.

@AmberSJones - @mauriel133 and I discussed this, and here's our approach for a fix. We had made a decision earlier that we would not allow people to modify the URL of the HydroShare user's profile. So, the immediate fix for you is to add a new author that has the name you want (HydroShare user or not) and then delete the iUTAH Data Manager user as an author.  Don't try to modify the iUTAH Data Manger author by changing its name or editing it.
Now, there is still an issue that we will solve by modifying the edit button on an Author that is a HydroShare user. First, we will allow people to add HydroShare users as authors, which will take a snapshot of the information in their profile and paste it into the Author metadata. The edit button will not allow them to edit the author metadata for an existing HydroShare user, but it will allow them to click a button to update the author metadata to the most recent information in the user's profile.  So, the workflow is edit your profile, then update your author information.

@horsburgh This sounds good, and we will work to modify the author info on the datasets that we have added. I'll leave the issue open until you address the question of not allowing the edit button to change the author metadata for an existing HydroShare user- unless you have that documented elsewhere.

No change to system needed.  This is a user education issue.  Need to think about messaging.


~/.docker/config.json getting reset

This is incorrect docker/config.json:
{
  "detachKeys": "ctrl-],q"
}

Correct one has auths, but sometimes it seems to disappear. Could be due to home-env update(?).

Error you get is:
denied: requested access to the resource is denied
Build: ERROR TRAP
Directory: /var/tmp/containers-20180228.190533-178


I want to match pattern from file log ( tail, json format )

Describe the bug

I want to match pattern ( json format )
but I found log from td-agent.log 'pattern not match'
so, I can't forward another Fluentd
To Reproduce

Expected behavior
I create json file on my local machine. example ->  (path : /var/log/resources.log )
I wrote my json log at this file
and, I set Fluentd conf. @type tail, format  json, tag log_test
but I can't match this tag..
How can i match this tag..??
Absolutely I want to use '@type tail'
I want this sentence ' log_test : {~~~json format data} '
Your Configuration
<source>
        @type tail
        path /castletown.server/log/resources.json
        pos_file /var/log/mysql_log.pos
        tag log_test
        emit_unmatched_lines true

        <parse>
        @type json
        </parse>
</source>

<match log_test>
        @type stdout
</match>

Your Error Log
2019-07-20 06:27:56 +0000 [info]: gem 'fluent-plugin-s3' version '1.1.10'
2019-07-20 06:27:56 +0000 [info]: gem 'fluent-plugin-td' version '1.0.0'
2019-07-20 06:27:56 +0000 [info]: gem 'fluent-plugin-td-monitoring' version '0.2.4'
2019-07-20 06:27:56 +0000 [info]: gem 'fluent-plugin-webhdfs' version '1.2.3'
2019-07-20 06:27:56 +0000 [info]: gem 'fluentd' version '1.4.2'
2019-07-20 06:27:56 +0000 [info]: adding match pattern="log_test" type="stdout"
2019-07-20 06:27:56 +0000 [info]: adding source type="tail"
2019-07-20 06:27:56 +0000 [info]: #0 starting fluentd worker pid=793 ppid=788 worker=0
2019-07-20 06:27:56 +0000 [info]: #0 following tail of /castletown.server/log/resources.json
2019-07-20 06:27:56 +0000 [info]: #0 fluentd worker is now running worker=0
2019-07-20 06:28:21 +0000 [warn]: #0 pattern not match: "{'log_type': 1, 'buddy_key': '5d24589be297cd0030bac238', 'date_time': '2019-07-20 06:28:21'}"

Additional context

Unfortunately, your log is not json. json doesn't use single quotation for string.
See json spec for more detail: https://www.json.org/
BTW, please follow contribition guide. Use mailing list first: https://github.com/fluent/fluentd/blob/master/CONTRIBUTING.md#got-a-question-or-problem

Thank you for your reply.
I'll check again Tanks.

Thank you so much. I solve this problem. I appreciate it..
I'm sorry I wasted your time. I will close issue.


Chrome: Gui tries to reconnect to owner "Lost Connection"

for me on the master i cannot connect with chrome to the owner server anymore: https://satvm02.researchstudio.at/owner
the top menu open which says "Lost connection – make sure your internet connection is working, then click “reconnect”."
However reconnect doesnt work, the owner server runs fine however
On matchat however things are working fine still
THIS ISSUES SHOULD BE CHECKED BEFORE MERGING THE MASTER INTO MATCHAT!

can anybody confirm this behavior?

maybe this could be a proxy problem since https://satvm02.researchstudio.at:8082/owner/ and https://satvm02.researchstudio.at:8081/owner are working fine

the problem seems to be one of the commits between Feb 7, 2017 (matchat) and Mar 7, 2017 (master). Since the problem is also there on integration-test when i deploy the master version there. However it disappears when the matchat version is deployed on integration-test


Wrong version shown

This morning when running "winget upgrade", I received the following list:
PS C:\Users\W> winget upgrade
Name            Id              Version Available Source
Mozilla Firefox Mozilla.Firefox 86.0    103.0     winget
NAPS2           Cyanfish.NAPS2  Unknown 6.1.2     winget
GOG GALAXY      GOG.Galaxy      Unknown <IP_ADDRESS>  winget
This is odd as the version of FF I have installed on my computer is current 104.0

For now, I will just ignore it (as I do with the apps showing unknown versions) but thought that others might be seeing similar issues.

Duplicate of #2380


Add script for generating test OPERA RTC services using public S3 bucket

This script allows us to generate test services using data staged in a public S3 bucket rather than in EDC. It is for the use of the ASF team so that we can publish fully functional services for testing purposes (we can't see EDC source rasters when the services are published to our image servers).

This PR should not be acted upon until the redo-url-file has been merged to opera-egis-updates, and that branch has been merged to develop.

This effort was addressed in https://github.com/ASFHyP3/gis-services/pull/157


CloudController still chooses the DEA which is almost full to do app staging if the DEA number is less than 5

Issue
CC chooses the almost full DEA to do staging, and that causes "Not enough memory resources available" error.
Context
According to this https://github.com/cloudfoundry/cloud_controller_ng/blob/master/lib/cloud_controller/dea/pool.rb#L101 , CloudController always picks up the top X (X is the max value between 5 and DEA Num * percentage_of_top_stagers) DEAs with max available memory, and select one randomly from the top X DEAs, but for a CloudFoundry deployment with less than 5 DEAs, even if one DEA is almost full, it's still possible to be picked up, and if two CloudControllers picks up the same full DEA at the same time, that causes staging error with "Not enough memory resources available".

@JimmyMa - I should be submitting a PR today for this.

https://github.com/cloudfoundry/cloud_controller_ng/pull/603 has been merged.

Hey @JimmyMa ,
Thanks for submitting this issue. We are going to close it now since it looks like the PRs that fix this  have been merged. Feel free to reopen if you feel like we haven't addressed the issue.
Thanks,
CAPI Community Pair - @ericpromislow & @michaelxupiv


Is it a reshaping of the gt density maps needed?

Hi, thank you for your work.
It seems that during the data loading, the gt density maps should be in the folder gt_density_map_adaptive_512_512_object_VarV2
However, this folder does not exist in the original dataset. I guess that they are the reshaped original density maps. Did you pad the density masks? What kind of padding did you use? Did you reshape density maps in some other ways?
Thank you

Yes, that's correct! If you want to train the model from scratch, refer to LOCA for all the necessary steps to generate density maps. LOCA uses adaptively changed Gaussians based on the width and height of the exemplars in the training stage. However, for evaluation, everything should work properly with the original density maps. I haven't checked, but I'm fairly certain it should.

Yes, that's correct! If you want to train the model from scratch, refer to LOCA for all the necessary steps to generate density maps. LOCA uses adaptively changed Gaussians based on the width and height of the exemplars in the training stage. However, for evaluation, everything should work properly with the original density maps. I haven't checked, but I'm fairly certain it should.

Thank you for your answer.
I think that it is needed not only for training, but also for other scripta such as fscd_0shot_clip.sh
Btw, I will follow the code of LOCA, thank you again.


Open appropriate Dexcom app

Fixes https://github.com/loudnate/Loop/issues/20

Congrats on your first Swift code! I'm ready to merge when Travis finishes.


Fix memory leaks of jest-worker

Summary
We noticed some closure related memory leaks in the current implementation of jest-worker with v8. The arguments and result of the task of each worker can't be garbage collected until a new task is assigned to that worker. If the arguments/result sizes are large, it's gonna introduce a huge memory increase.
This PR ensures that args and task are correctly scoped so they won't be claimed by other closures, and references can be cleaned once they become unneeded.
Test plan

We have an app that passes image buffers as the argument of tasks which are pretty large, and we are seeing a huge memory leak after the tasks have finished (ref: vercel/next.js#22925). Here's a screenshot for the memory dump:

The memory usage increased from 112MB to 145MB (+29%) after processing the task, and most of the new allocated memory was taken by the arguments and result of that task, which should have been recollected after it ends.
With this fix we are no longer to reproduce this leak with the same app:

Thank you for adding the test @jeysal, it looks great!

Great spot and thanks again!

@MichaReiser FYI

@shuding would a pre-release of this help? If not urgent a stable release should be out this month (🤞🤞🤞)

@SimenB Yep we're good with a pre-release, thanks for your help!

<EMAIL_ADDRESS>is available now 👍


Error while trying to do brew install

go_mod_cache/bin missing error.

Working in the latest release 0.1.0.

Working in the latest release 0.1.0.


why publiush a .net core web project will contain many Microsoft.VisualStudio.Web.CodeGeneration.xxx dll

I found that when I publish a asp.net core 2.2 Web project ,in publish folder contain many startwith Microsoft.VisualStudio.Web.CodeGeneration.xxx.dll,like this:

I think these dll is in VS 2019's  Analyzers,

but is these dll need to use in runtime?How Can I pushlish the project don't contain they?because I like the pushlish less files to server

Thanks for contacting us, @kugarliyifan.
@pranavkm can you please confirm this is caused by https://github.com/dotnet/sdk/issues/1212, which is blocked on NuGet?


How do interface SCIP to Pyomo ?

I read a lot in internet about this topic  but it is not clear how to do it.
Do you have any tutorial to solve the next problem:
WARNING: Could not locate the 'scipampl' executable, which is required for
solver scip
ApplicationError: No executable found for solver 'scip'

Hi!
The installation of a optimization programming solver (such as SCIP) goes beyond the scope of the pycity_scheduling package.
However, did you already study the SCIP webpage?
https://www.scipopt.org/
You also need a valid license to run SCIP.
If you are a member of a non-commercial or academic institution, it is free of charge.
Moreover, the repository also includes a Dockerfile witht the reference commands for setting up SCIP on a linux-based machine.
https://github.com/ElsevierSoftwareX/SOFTX-D-20-00087/blob/master/gitlab-utils/Dockerfile


chore: update git ignore

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

/check-enforce override

/check-enforcer override

/check-enforcer override

/check-enforcer override

/check-enforcer override


add STREAM receipts

RFC
STREAM implementation
Web monetization spec (Pending)
STREAM receipts allow web monetized sites to verify received payment without having access to ILP packets.
The web-monetization-id header is no longer necessary for SPSP queries.
~I was able to test a staging build of the coil extension with web-monetization-demo (with this patch) while port forwarding 7768 to Coil's staging connector.~
You can test a staging or prod build of the coil extension with BTP_ENDPOINT=btp+ws://localhost:3000 with web-monetization-demo:
// run web-monetization-demo with
node_modules/.bin/lerna run server:dev --stream
node_modules/.bin/lerna run verifier:dev --stream
node_modules/.bin/lerna run client:dev --stream

@wilsonianb
I am still getting my head around receipts and kicking the tires on this.
I fixed some bugs in @web-monetization/react and added a webpack macro so that you can override the btp endpoint to connect to @web-monetization/demo which uses ilp-plugin-mini-accounts.
So you may want to rebase.
To build extension to connect to the demo:
export BTP_ENDPOINT=btp+ws://localhost:3000
yarn dev-chrome-prod

Rebased and made the receipt verifier a standalone service in web-monetization-demo

Rebased and force pushed due to yarn.lock conflicts


⚠ Bofrost.de has degraded performance

In 71838f5, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1828 ms

Resolved: Bofrost.de performance has improved in b157185.


FTPS support

With the help of the double-bag-ftps dependencie I developed the option to use FTPS explicite.
You just have to add the "ftp_secure: true" option in your _config.yml.
If it's not set it defaults to false so nothing changes, in the default settings standard ftp works as usual.
I added this functionality because I didn't want to send my plain password over the internet.
Unfortunately I have never before coded something in Ruby so I hope there are no newbie errors.
Also I'm not sure how to develop the tests in Ruby and we should probably add double-bag-ftps dependencie as an dependencie to the project.
If someone gives me some hints I will glady try to do this stuff.
Other than that I'm using the ftps functionlity right now and didn't encounter any problems yet.

hi again, sorry for the long absence of communication
maybe someone who actually codes in ruby could go forward with this? I saw your remarks and they are all good, seeing that I have no knowledge of ruby whatsoever I was having trouble doing all the testing and even simple compilation with rake ...
When I tested it, it kind of worked but as you said in the other topic the implementation is flawed, also  because I have no real idea about syntax and build process in ruby
ps; I’m also still under ubuntu 12.04 which doesn't help having a recent ruby version environment that works well, will update soon though


update()

I'm using this library for a simple REST service in deno.  I need to await r.db("test").table("test").update(...), but update() does not seem to be implemented yet.  Is this something planned for the near future?

I haven't really planned anything for this library as I have no usecase anymore. I will see how simple this would be to implement.

Thank you in advance.  This was the only library I found that I could easily use in deno.  I was not able to get any of the other node libraries to import.  I would take on support of this repository, but unfortunately I'm still learning deno, Typescript, and rethinkdb, so it may be a while before I can contribute anything useful.


feat(ominauth): support json-formatted values in omniauth callback.

Fixes #221
This is potentially a breaking change.  For example, nil values used to be rendered as an empty string, but will now render as undefined.
We needed it because we have some values in there that are arrays.

:+1:


broken GSM download

When I download GSM supplementary files by:
gsm = cast(GSM, GEOparse.get_GEO("GSM1944823", destdir="/tmp"))
files = gsm.download_supplementary_files("/tmp", False<EMAIL_ADDRESS>
I get the following error
13-Feb-2018 18:02:51 DEBUG utils - Directory /tmp/Supp_GSM1944823_MG_UKJ_30_190214_1HS_brain already exists. Skipping.
13-Feb-2018 18:02:51 INFO utils - Downloading NONE to /tmp/Supp_GSM1944823_MG_UKJ_30_190214_1HS_brain/NONE
Traceback (most recent call last):
  File "<input>", line 1, in <module>
  File "/home/antonkulaga/rna-seq/containers/geoparse/env/lib/python3.6/site-packages/GEOparse/GEOTypes.py", line 443, in download_supplementary_files
    utils.download_from_url(metavalue[0], download_path)
  File "/home/antonkulaga/rna-seq/containers/geoparse/env/lib/python3.6/site-packages/GEOparse/utils.py", line 114, in download_from_url
    destination_path))
  File "/home/antonkulaga/rna-seq/containers/geoparse/env/lib/python3.6/site-packages/wgetter.py", line 272, in download
    url = opener.open(link)
  File "/usr/lib/python3.6/urllib/request.py", line 511, in open
    req = Request(fullurl, data)
  File "/usr/lib/python3.6/urllib/request.py", line 329, in __init__
    self.full_url = url
  File "/usr/lib/python3.6/urllib/request.py", line 355, in full_url
    self._parse()
  File "/usr/lib/python3.6/urllib/request.py", line 384, in _parse
    raise ValueError("unknown url type: %r" % self.full_url)
ValueError: unknown url type: 'NONE'

Hi,
The problem here is that the entries do not have supplementary files ie. The list of supplementary files is composed of one entry: 'None'. I see you do not want to download SRA so this is not an issue for you.

@guma44 in such case the behavior is not consistent. If for some GSM there are no supplementary files, then it should download nothing and return an empty array of paths.

And this is exactly what try-except block does. As can be seen, no one can guarantee that user will not put some nonsense in the metadata. Thus, the code tries to download it but if it cannot, it adds nothing to the return dictionary. Thus, as a result, is an error logging message and empty dictionary. I can actually change Exception to ValueError to be more strict on what error is caught.

It is not clear why it does not download at least sra when I tell to download supplements. At least sra does exist in GSM1944808
import os
import re
from pprint import pprint
from typing import *

import GEOparse
from GEOparse import *
from GEOparse import utils
from functional import *

gsm = cast(GSM, GEOparse.get_GEO("GSM1944808", destdir="/tmp"))

filetype = 'sra'
keep_sra = True
fastq_dump_options = {
	'skip-technical': None,
	'clip': None,
	'split-files': None,
	'readids': None,
	'read-filter': 'pass',
	'dumpbase': None,
	'gzip': None
	}
sra_kwargs = {
    "keep_sra": keep_sra,
    'filetype': filetype,
    "fastq_dump_options": fastq_dump_options
}
directory_path = "~/test"
gsm.download_supplementary_files(directory_path, True<EMAIL_ADDRESS>sra_kwargs)

I get
Downloading NONE to /pipelines/text/Supp_GSM1944808_MG_UKJ_15_190214_1HS_brain/NONE

while I at least expect sra to be downloaded as I sad download_sra=True

SRA, in this case, is not listed in the supplementary files. It is just listed as a relation but not as a supplementary file. This is a separate issue and it would be worth to check relations for that but for now, this functionality does not exist. Overall, it does not say why it did not download SRA because there is no  SRA unfortunately :/.

I've tried to download SRA with
path =<EMAIL_ADDRESS>"/pipelines/test", **sra_kwargs)

The sra was downloaded (and saved as Supp_GSM1944808_MG_UKJ_15_190214_1HS_brain ) but path is []
I think the function should return the path to downloaded file(s) instead of an empty array

Yes, that is the bug. download_SRA is working but not as expected. I will fix it.


The dataframe generation takes along time

Issue description
When we want to train a model on a large-scale dataset (cem500k) the data frame generation takes a long time (having no idea if finish at all).
Current behavior

These are  the last log message I get:

2021-11-08 16:01:42.007 | INFO     | ivadomed.config_manager:_display_differing_keys:150 - Adding the following keys to the configuration file
2021-11-08 16:01:42.007 | INFO     | ivadomed.config_manager:deep_dict_compare:43 -     lr_scheduler: base_lr: 1e-05
2021-11-08 16:01:42.007 | INFO     | ivadomed.config_manager:deep_dict_compare:43 -     lr_scheduler: max_lr: 0.01
2021-11-08 16:01:42.007 | INFO     | ivadomed.config_manager:_display_differing_keys:152 -
2021-11-08 16:01:42.007 | INFO     | ivadomed.utils:get_command:356 - No CLI argument given for command: ( --train | --test | --segment ). Will check config file for command...
2021-11-08 16:01:42.007 | INFO     | ivadomed.utils:get_path_output:371 - CLI flag --path-output not used to specify output directory. Will check config file for directory...
2021-11-08 16:01:42.007 | INFO     | ivadomed.utils:get_path_data:383 - CLI flag --path-data not used to specify BIDS data directory. Will check config file for directory...
2021-11-08 16:01:42.008 | INFO     | main:set_output_path:198 - Output path already exists: out_cem500k_BIDS
2021-11-08 16:01:42.570 | INFO     | ivadomed.utils:define_device:135 - Using GPU ID 0
2021-11-08 16:01:42.570 | INFO     | ivadomed.utils:display_selected_model_spec:145 - Selected architecture: Unet, with the following parameters:
2021-11-08 16:01:42.571 | INFO     | ivadomed.utils:display_selected_model_spec:148 -   dropout_rate: 0.2
2021-11-08 16:01:42.571 | INFO     | ivadomed.utils:display_selected_model_spec:148 -   bn_momentum: 0.1
2021-11-08 16:01:42.571 | INFO     | ivadomed.utils:display_selected_model_spec:148 -   depth: 4
2021-11-08 16:01:42.571 | INFO     | ivadomed.utils:display_selected_model_spec:148 -   is_2d: True
2021-11-08 16:01:42.571 | INFO     | ivadomed.utils:display_selected_model_spec:148 -   final_activation: sigmoid
2021-11-08 16:01:42.571 | INFO     | ivadomed.utils:display_selected_model_spec:148 -   length_2D: [256, 256]
2021-11-08 16:01:42.571 | INFO     | ivadomed.utils:display_selected_model_spec:148 -   stride_2D: [244, 244]
2021-11-08 16:01:42.571 | INFO     | ivadomed.utils:display_selected_model_spec:148 -   folder_name: cem500k_BIDS
2021-11-08 16:01:42.572 | INFO     | ivadomed.utils:display_selected_model_spec:148 -   in_channel: 1
2021-11-08 16:01:42.572 | INFO     | ivadomed.utils:display_selected_model_spec:148 -   out_channel: 3

Expected behavior

I think after some reasonable time of the data frame preparation, it should go to the dataset loading and data augmentation.
Steps to reproduce

You can reproduce the whole experiment using this bash file:
in summary,

extract the cme500k dataset (on the Ivadomed root is a good place).
run the raw_to_BIDS.py scripts to generate the BIDS-like structured cem500k dataset with 3 targets (2 classes) and the target directory with some needed JSON files.
Train using this config file.

Environment
System description

Windows workstation with WSL and Linux ComputeCanada node (186G or 191000M	2 x Intel Gold 6148 Skylake @ 2.4 GHz	1 x NVMe SSD 1.6T	4 x NVidia V100SXM2 (16G memory), connected via NVLink)
Installed packages

Exactly as described on the installation documents

 Output of pip freeze
absl-py==0.15.0+computecanada
alembic==1.2.1+computecanada
bids-validator==1.8.4
cachetools==4.2.4+computecanada
certifi==2021.10.8+computecanada
charset-normalizer==2.0.7+computecanada
click==8.0.3+computecanada
cloudpickle==2.0.0+computecanada
configparser==5.0.2+computecanada
csv-diff==1.1
cycler==0.11.0
databricks-cli==0.9.0+computecanada
dictdiffer==0.9.0+computecanada
docker==5.0.3
docopt==0.6.2+computecanada
entrypoints==0.3+computecanada
Flask==2.0.2
flatbuffers==20190709135844+computecanada
gitdb==4.0.9+computecanada
gitpython==3.1.24+computecanada
google-auth==2.3.3+computecanada
google-auth-oauthlib==0.4.6+computecanada
grpcio==1.38.1+computecanada
gunicorn==20.1.0+computecanada
h5py==2.10.0+computecanada
idna==3.3+computecanada
imageio==2.10.1
importlib-metadata==4.8.1+computecanada
itsdangerous==2.0.1+computecanada
jinja2==3.0.2+computecanada
joblib==1.1.0+computecanada
kiwisolver==1.3.1+computecanada
loguru==0.5.3+computecanada
mako==1.1.5+computecanada
Markdown==3.3.4+computecanada
MarkupSafe==2.0.1+computecanada
matplotlib==3.4.2+computecanada
mlflow==1.21.0
networkx==2.6.3+computecanada
nibabel==3.2.1+computecanada
num2words==0.5.10+computecanada
numpy==1.21.2+computecanada
oauthlib==3.1.1+computecanada
onnxruntime==1.8.1+computecanada
packaging==21.2+computecanada
pandas==1.3.0+computecanada
patsy==0.5.2
pillow==8.4.0+computecanada
prometheus-client==0.12.0
prometheus-flask-exporter==0.18.5
protobuf==3.19.1+computecanada
pyasn1==0.4.8+computecanada
pyasn1-modules==0.2.8+computecanada
pybids==0.13.2
pyparsing==2.4.7+computecanada
python-dateutil==2.8.2+computecanada
python-editor==1.0.4+computecanada
pytz==2021.3+computecanada
pywavelets==1.1.1+computecanada
PyYAML==5.4.1+computecanada
querystring-parser==1.2.4+computecanada
requests==2.26.0+computecanada
requests-oauthlib==1.3.0+computecanada
rsa==4.7.2+computecanada
scikit-image==0.18.2+computecanada
scikit-learn==0.24.2+computecanada
scipy==1.7.1+computecanada
seaborn==0.11.2+computecanada
six==1.16.0+computecanada
smmap==5.0.0+computecanada
SQLAlchemy==1.3.20+computecanada
sqlparse==0.4.2
tabulate==0.8.9+computecanada
tensorboard==2.7.0
tensorboard-data-server==0.6.1+computecanada
tensorboard-plugin-wit==1.8.0+computecanada
threadpoolctl==3.0.0+computecanada
tifffile==2021.11.2
torch==1.8.1+cu111
torchvision==0.9.1+cu111
tqdm==4.62.3
typing-extensions==<IP_ADDRESS>+computecanada
urllib3==1.26.7+computecanada
websocket-client==1.2.1
werkzeug==2.0.2+computecanada
zipp==3.6.0+computecanada

Thank you @MosyMosy for sharing the code and files. We never tried running the dataframe on such a large dataset!
If I understand correctly, after 24h, the command is still running and the bids_dataframe.csv file is still not in the path_output, correct?
As a first step, would you be able to also share a few images (~5) from the original cme500k dataset so I could reproduce your pipeline without downloading the whole 16 Gig?
I'll try to take a look at the data and config file tomorrow.

Hi @mariehbourget, Thanks for your consideration.
Yes, so it is going to be 40 hours by now. I am suspicious of the "bids_dataframe.py" line 110.
It is simply a directory containing some images without any structure and metadata. you can download this sample.
Thank you.

I was able to reproduce your pipeline on my side and start the training without errors (with only 5 subjects).
This may be a little bit complex to debug because the error is not directly in ivadomed. The error you got above comes from sqlalchemy which is a dependency or pybids that ivadomed uses to index the files in the dataset and create the dataframe. To reduce the numbers of variables, you can:

Remove participants.tsv, participants.json, samples.tsv and samples.json from your dataset (they are not required if you don't need the metadata from them).
In the sidecar json file of each image file (from sample_metadata.json), you can remove everything except the "PixelSize" field.

I don't know if that would be enough to fix the error and speed-up the process though. I would also recommend to try and load much smaller datasets and increase progressively the size to see what happens. Once the dataframe is created, you may ran into other issues with such a large dataset (see this issue for example).
There are a few other things that I saw but that are probably unrelated to you issue:

In BIDS, the <label> following an entity in the filename (ex: sample-<label>) must be alphanumeric only. So a name like sample-0AO4EJXNKJLS_0_0124_0_448 is not good because of the underscores. In the dataframe, the sample_id will be cut after the first underscore. Currently, it is working because we force pybids to index microscopy files, but it won't in the future (when microscopy is officially included in BIDS).
Are the files all .tiff? If yes, I would recommend renaming to .tif (one f) to follow BIDS and to only use .tif in the extensions field in the config file.
You are using a PixelSize : [1, 1] (in um) in the sample_metadata.json file, is it the real resolution of the images? With the Resample : [0.0001, 0.0001] (in mm) in the config file, it means that after the dataframe is created, every image will be resampled from a resolution of 1um/pixel to a resolution of 0.1 um/pixel before training start. (It may be intentional as 0.1 um/pixel is our current resolution for SEM, but I thought I should mention it.)


Esbuild ouput target does not match projects tsconfig target

I've discovered this issue when using optional chaining in my typescript code which is set to transpile down to ES2015 code in my tsconfig. However esbuild does not use this target so the actual output code is ES2020 or whatever default target esbuild has which includes optional chaining in the js bundle. This causes errors when invoke my lambda because Node 12 does not support optional chaining.
So a potential solution could be to read the projects tsconfig target and pass it into esbuild or allow for a target to be set via some other config.
Here's a repo I created with the issue recreated: https://github.com/SyedRFarhan/sst-esbuild-target-mismatch
Run this with Node 12 and invoke the function to recreate my issue.

@SyedRFarhan Thanks for reporting this. Can you share the step by step commands I need to run after git cloning your repo to reproduce the issue?

Set your environment to Node 12
npm run start
Make a get request to the lambda endpoint
An error should be printed to the terminal regarding the use of optional chaining syntax

Thanks @SyedRFarhan
Can you confirm this is the error you are getting on your side?
{ name: 'SyntaxError',
  message: 'Unexpected token .',
  stack:
   '/Users/frank/Sites/fwang/sst-esbuild-target-mismatch/.build/src/lambda.js:16
     return someObject?.anything ?? "does not exist";
                       ^

   SyntaxError: Unexpected token .
       at Module._compile (internal/modules/cjs/loader.js:723:23)
       at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
       at Module.load (internal/modules/cjs/loader.js:653:32)
       at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
       at Function.Module._load (internal/modules/cjs/loader.js:585:3)
       at Module.require (internal/modules/cjs/loader.js:692:17)
       at require (internal/modules/cjs/helpers.js:25:18)
       at getHandler (/Users/frank/Sites/fwang/sst-esbuild-target-mismatch/node_modules/@serverless-stack/cli/assets/lambda-invoke/bootstrap.js:87:15)
       at start (/Users/frank/Sites/fwang/sst-esbuild-target-mismatch/node_modules/@serverless-stack/cli/assets/lambda-invoke/bootstrap.js:32:15)
       at Object.<anonymous> (/Users/frank/Sites/fwang/sst-esbuild-target-mismatch/node_modules/@serverless-stack/cli/assets/lambda-invoke/bootstrap.js:26:1)'
}

@fwang
Yes, that's the error.

@SyedRFarhan Just had a look at this. This is a bug on our side. We are not explicitly setting the esbuild target right now. That said, I don't think esbuild uses the target set in the tsconfig.json. To ensure compatibility, we'll set the target globally to ES2015 for now.
https://github.com/evanw/esbuild/issues/277

Another quick fix is to use NodeJs 14.x instead of 10 or 12 (if possible) in the function runtime and in the dev environment. With this runtime, the new operators work in the handler at runtime

@ryanmalesic Yeah it does. Just tested this as well.

@SyedRFarhan This is fixed in release v0.9.5

Thanks for looking into this!


Use ApiError, ApiResult

[x] Update init/handle/query in contracts to serialize ApiError rather than a raw string for error messages
[x] Replace {Init,Handle,Query}Result with ApiResult customizations
[x] Ensure schemas are properly generated
[x] Refine testing workflow
[x] Simplify wasm bindings
[x] Update CHANGELOG
[ ] Regenerate test contracts for vm (after approval)

@webmaster128 ready for review now.
This works to extend rich Results with ApiError or Error all throughout the stack.

I like the idea of a minimal cosmwasm-schema package to avoid copying code to each repo when updating it.
Can you propose a better wording for the changelog,  as I am unsure what is misleading?

Okay, made last requested changes and prepared for merge.
Awaiting approval.


[Feature][ST-Engine] Output job summary metrics at the end of the job

Is there any progress? At present, st-engine can print the metric at the end of the task, but it is very rough and needs to optimize the output

@FWLamb
Is there any progress? At present, st-engine can print the metric at the end of the task, but it is very rough and needs to optimize the output

@FWLamb Is there any progress? At present, st-engine can print the metric at the end of the task, but it is very rough and needs to optimize the output

Sorry, I'm busy these days and haven't looked carefully. If someone wants to participate in this, they can be assigned; I take time to do this on weekends. Do you have any suggestions for the output content? I don't have a good idea about which metrics to output.

Move to #4013


Tests working

ARuleMiner is now Miner.
Powerup and Info fields added to Miner.
FPGrowth now efficiently saves metrics when an FPTree is a list and it is sure to be generating only frequent itemsets.
Tests run smoothly.
Benchmark.

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


docs: remove containerRuntime from examples as it was dropped

Fixes #N/A
Description
How was this change tested?
Does this change impact docs?

[x] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

There are notes after each of the places where you cut containerRuntime that say: "You can specify the container runtime to be either dockerd or containerd. By default, containerd is used." Should you remove those notes if you can no longer specify using anything other than containerd?

Fixed with a note about spec.containerRuntime being dropped

Thanks for pushing this update through @lordz-md! Nice work 🎉

Thanks for pushing this update through @lordz-md! Nice work 🎉

Thanks @jonathan-innis.


test: Change tests according to image controller always deletes component image repository

Description
This PR changes e2e tests expectations regarding Component image repository lifecycle. With new behavior, Image Controller always deletes image repository of the Component, when the Component gets deleted.
Annotation image.redhat.com/delete-image-repo is deprecated and will have no effect.
Issue ticket number and link
STONEBLD-1340
Checklist:

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

/retest

/test redhat-appstudio-e2e

/test redhat-appstudio-e2e


Unable to download JSON metadata: HTTP Error 403: Forbidden [pluralsight]

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.07.23. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.07.23

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

C:\Users\oTaKuGeRi\Desktop\Youtube-dl>youtube-dl -v https://www.pluralsight.com/courses/networking-comptia-a-plus-220-901
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.pluralsight.com/courses/networking-comptia-a-plus-220-901']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2017.07.23
[debug] Python version 3.4.4 - Windows-10-10.0.14393
[debug] exe versions: none
[debug] Proxy map: {}
[pluralsight:course] networking-comptia-a-plus-220-901: Downloading course JSON
ERROR: Unable to download JSON metadata: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpu6zpd7mk\build\youtube_dl\extractor\common.py", line 502, in _request_webpage
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpu6zpd7mk\build\youtube_dl\YoutubeDL.py", line 2166, in urlopen
File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default
C:\Users\oTaKuGeRi\Desktop\Youtube-dl>youtube-dl -U
youtube-dl is up-to-date (2017.07.23)
C:\Users\oTaKuGeRi\Desktop\Youtube-dl>python
Python 3.6.2 (v3.6.2:5fd33b5, Jul  8 2017, 04:57:36) [MSC v.1900 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.

For some reason I can't login to Pluralsight Offline app as well and getting "Error Occurred", however I can login to web with no problem.
UPDATE: I was able to download properly now after using a VPN connection (Hotspot VPN) as well as login to offline app, I will be closing this ticket for now.

Probably implemented the CAPTCHA to access via JSON, since you do not have access to the dialog it refuses access.

Yes, the problem all started after I got the CAPTCHA page prior to accessing the my Pluralsight Dashbpard, I tried doing the https://github.com/rg3/youtube-dl#how-do-i-pass-cookies-to-youtube-dl, but it never work for me.
Checked https://github.com/rg3/youtube-dl/issues/154 and so far no update on it since 2015
After a week since I encountered the problem, somehow I can access/download again and the CAPTCHA page is now gone.


Omit scrobbling support from release

Out of respect for SomaFM's wishes, I'll omit Last.fm scrobbling support from the release others will be able to download. Scrobbling will still be possible if you build the app yourself and provide your own Last.fm API keys.

Hi. What the reason to hide last.fm from release version?
Is this feature works?
Just found this line: https://github.com/cheshire137/huxleyfm/blob/c038541b913948888f582bf803b1a5fcdd2f724d/settings/settings.html#L2
Can you specify, there should I put API key in the app?

What the reason to hide last.fm from release version?

I thought the discussion took place in this repo, but it was actually in another repository of mine for listening to SomaFM. Check out the thread starting at https://github.com/cheshire137/soma-chrome/issues/39#issuecomment-362104481 for some backstory on not supporting Last.fm scrobbling with SomaFM players. For context, @rustyhodge works at SomaFM.

Is this feature works?

It should work okay, yeah, it just requires some manual work to set a config variable. I think these docs are up to date, but let me know if they don't work for you! https://github.com/cheshire137/huxleyfm#how-to-develop

@cheshire137 thanks for the explanation.
BTW, I'm also switching to open alternative of lastfm:

Libre.fm
https://musicbrainz.org/

Now I'm choosing which is suits better my need. Would you mind to consider to support these services?


Get List from List<TinyCsvParser.Mapping.CsvMappingResult>

I have the following simple code:

What is the recommended way of getting at the underlying type?
Right now my result is:
List<TinyCsvParser.Mapping.CsvMappingResult<ShippingRecord>> but I am after List<ShippingRecord>.
I know I can loop and create a new list and return that, but is that the recommended way here? There's no easier way?

csvParser
// Read the File:
.ReadFromFile(csvFilePath, Encoding.UTF8)
// Get the Valid Results:
.Where(x => x.IsValid)
// And get the populated Entities:
.Select(x => x.Result)
// Turn it into a List:
.ToList();


Supporting something like array.insert?

Hi there!
I am fairly new to MST/keystone and trying to wrap my head around the philosophy, features, implementation details.
I have found many issues around arrays/ordering in MST/keystone repos (adding-replacing gotchas, problems around safeReference, etc.), however I did not find something like .insert(index, newElementArray) as part of the API.

Is there an approach/canonicial way to mimic .insert like behaviour? What are the gotchas?
If there is no canonical way: why, what is the main issue?
If there are only dirty workarounds: as far as I understand keystone handles references explicitly, so my question is if this information can be used to support something like .insert across the app state.

Eg.:
const array = [0,1,5,6]
array.insert(2, [2,3,4]) // +3 length
console.log(array) // [0,1,2,3,4,5,6]

Patch/inverse patch info (merged to be more concise)
[op, path, value]:
replace, 2, 5->2
replace, 3, 6->3
add, 4, 4
add, 5, 5
add, 6, 6
And I do not know about the implementation how references can / should work in all cases, but if an identity is resolved through an explicit ID/address, then this shift/insert does not seem to be an issue (for references pointing to indices after the added part), meanwhile, in case of path-based identity, we could just update the observers not to observe /2 /3 anymore, but /5 and /6 instead (just adding the same +3 everywhere). Maybe both are semantically valid insertion types, .insertKeepingPathRef vs .insertKeepingIdRef. Furthermore, if path based identity is lazily used/calculated, then there is no update/change needed for observers (of this array) at all.
Unfortunately, I am not familiar with array/model semantics of keystone, so I felt I had to raise this question, sorry if it misses some points.

you may use the standard Javascript splice to do that insert
splice(2, 0, 2, 3, 4) would be the equivalent to your insert
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/splice

@xaviergonz Oh great, thanks. Somehow I though it is necessary to keep the same number of elements (deleted/replaced) in MST/keystone environment, I did not know why I had this assumption.


Mention dry-initializer on external dependencies page

This change addresses https://github.com/dry-rb/dry-validation/issues/637 by adding a dry-initializer reference to the external dependencies page.

Thanks!


Bltsd v2 build folder temp DLLs

Continues
https://github.com/JDSherbert/Sherbert-Engine/pull/22
From the previous PR addressed significant new control for pathing of the output files.
Please see the previous before Merging!
Problem
The following addresses the change of changing the default x64 folder for Debug and releases of builds to be instead addressed in the " Build " folder in the solutionDir path.
Note (Allows the default git ignore to ignore the entire build folder from the commit).
Solution
Such as  SolutionDir -> Build -> x64 -> Debug /Release
As previously SolutionDir -> x64 -> Debug /Release
Solves
Output Directory specification
Git Ignore Folder path for " Build " (Next Update).

Addressed Merging Error, the wanted to overwrite the previous PR to this PR for the Output Directory.

Merged and Addressed https://github.com/JDSherbert/Sherbert-Engine/pull/24#issuecomment-1923685104 working as intended.


Use new fs.watch API for faster polling

I did a quick set of measurements and I'm seeing things update a few hundred milliseconds faster on average.
There are some downsides: http://nodejs.org/api/fs.html#fs_caveats

@shz Any additional thoughts on the downsides? Do we care about a hundred milliseconds? Is there a way to fall back to the old method?

The benefit is pretty sweet, from what I've experienced.  Feels much better being that much closer to instant, especially when making small CSS changes.  That said, I've been thinking about this and I think I want to recant a bit.  Having issues on networked filesystems seems pretty severe, as those aren't too uncommon.
I think the way to go is probably to have the fs.watch approach be the default, but also support a fallback to fs.watchFile via a user-controlled option.  If that sounds good, I'll implement and update the PR.

If you have a graceful way to do that, and test it, please do!

bump

One more bump before I close this.

Hey, sorry about the radio silence, been a bit swamped.  I've put aside some time this weekend though, hoping to have this PR finished up on Monday.

I'm going to fire up a new PR for this, had to make significant changes and a rebase, was easiest to just use a clean slate...


Request for documentation and design-decisions

This seems like a really interesting library, especially the type-checking part. Would it be possible to answer the following questions:

What was the original intended purpose for this library?
Has the PG SQL parser been implemented from scratch in Haskell? Why not bind to the existing PG bindings?
Why does it say that it's still in pre-alpha stage? What is not working currently?
Is the project being actively maintained? What is its future vision?

@saurabhnanda It works wonderfully. It's a bit lacking in documentation but I used http://jakewheat.github.io/hssqlppp/latest/ and a bit of reading through the source code itself to use it in my project https://github.com/abhin4v/ringo/ to generate SQL.

What was the original intended purpose for this library?

I had some code which was a prototype for an application development
system, and it used mainly plpgsql, with a lot of code generation done
within plpgsql using string concatenation, and the maintainability was
getting worse and worse. I originally wrote hssqlppp to help with
this.

Has the PG SQL parser been implemented from scratch in Haskell?

Yes.

Why not bind to the existing PG bindings?

I had no idea these existed until now. But I think the level of
detail captured, and the coverage of different SQL statements is not
nearly enough for what I want.

Why does it say that it's still in pre-alpha stage? What is not working currently?

I mean pre-alpha with respect to the stability of the interface.This
is still in flux and changes a lot between major releases. There are
also some smallish gaps in the parser, and quite a lot of gaps in the
type checker (including some big regressions wrt typechecking with ddl
statements).
If you are just parsing and type checking queries and dml then I don't
think you will come across any issues or these gaps.

Is the project being actively maintained? What is its future vision?

It is being actively maintained, but only very slowly and with some
latency. It is in more development internally in Sqream, and all the
changes to the hssqlppp code are released on github.
The future vision at the moment is to

improve the implementation of dialect handling

support more of ansi sql, port the parsing code from
a newer project which does parsing only (simple-sql-parser) - it is
much better written

I am also looking at replacing parsec with megaparsec or maybe
uu-parsinglib or possibly something else (but still a haskell parser
combinator)

improve the typechecking, especially some of the implementation is a
mess

improve the error messages

improve the quality of the code and the code documentation

There is a more detailed todo full of things which I never get round
to:
https://github.com/JakeWheat/hssqlppp/blob/master/todo

abhin4v, can you give some more details on what is lacking in the documentation, maybe I can work on some of this

I had no idea these existed until now. But I think the level of detail captured, and the coverage of different SQL statements is not nearly enough for what I want.

In what way? Wouldn't PG's internal query parser always be up-to-date and comprehensive?

Yes, I think you are right that it is comprehensive for select statements. But when I originally wrote hssqlppp, I already was trying to parse, insert/update/delete, ddl and create function statement bodies also.
Maybe if I had seen this at the time I would have tried to use it (and extend the c code to cover the extra kinds of statements), but I wouldn't like to be maintaining a fork of some postgres code in c, or to require postgres in order to use hssqlppp. In addition, extending the parser in c to support new syntax or other SQL dialects isn't something that I would like to try and do, if it is even feasible.


Update Flow to 0.32

Motivation: Better libs needed in an upcoming PR.

Current coverage is 87.61% (diff: 100%)

Merging #2606 into master will not change coverage

@@             master      #2606   diff @@
==========================================
  Files           240        240
  Lines         18054      18054
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits          15818      15818
  Misses         1985       1985
  Partials        251        251

Diff Coverage
File Path

•••••••••• 100%
js/noms/src/put-cache.js

•••••••••• 100%
js/noms/src/fetch.js

•••••••••• 100%
js/noms/src/struct-test.js

Powered by Codecov. Last update e28cda9...1fde918


Fix issue 17037 - std.concurrency has random segfaults

https://issues.dlang.org/show_bug.cgi?id=17037
Setting scheduler to null once it has been set is bad news.
This is the solution I found given the current design / implementation.

Seems to cause a deadlock now, closing until I debug it.


Not working?

Ok, so ive checked my sound and its not my laptop, whenever i press a key it doesnt make any sound. It still shows the key being pressed though... its not working and ive been trying to get it work all day. I even restarted my laptop : ( please fix this :(

You may have to clear your browser cache.

I cleared my browser cache, but the sound isn't working... any suggest?

sound works with all sites

yeah not working for me ether even after the cash clear

The new Google Chrome update has made my website incompatible with the new changes (https://developers.google.com/web/updates/2017/09/autoplay-policy-changes). I will try and fix them shortly. If you want to use the site while this is still broken, try using a different browser.

Update: this issue should now be fixed on the newest version of the Google Chrome browser.


Only show reset password success message if username is valid

Currently we show success message even if a username does not exist in the db.

I'm ok with that.
I don't think it's a big deal, but e.g. datadog, says something like, 'an email has been sent, if a user with that email exists'.
Basically, just prevents discovering whether a user has a login, from that form.

I've reworded the message


Cards that don't work as headers or footers

So far:

Logbook Card - It works fine as a middle card with light text on black background

When using the glance card with footer or footer-open, the title picks up the color from the left bar.

I get this with a lot of HACS cards. Seems to be a common issue.


fix: handle bad response for helium height

Why
Diagnostics fail entirely if unable to get Helium blockchain height.
How
Return -1 if request fails.
It's not clear why the request is failing in the first place.
curl to the same endpoint works from miners that are actively failing.
Tried disabling cache on the request, but no impact.
References
Related to: https://github.com/NebraLtd/hm-diag/issues/223

I highly suspect that the Helium API is not reporting the blockchain height currently atm. A lot of things in the Explorer are not loading right now and I see the same behaviour on my SC units

Maybe we should grab the blockchain height from the miner over JSON RPC instead of from Helium API?
Or compare the two or something

Now seems to be a good time to try this out. Are we ready to merge to testnet?

Now seems to be a good time to try this out. Are we ready to merge to testnet?

Should be good to merge soon, just testing this out locally to make sure it's all good. Balena is giving me trouble again with updating device but as soon as it goes through I'll run the tests

Yep Balena is having issues again.. Lot's of devices showing as Heartbeat only

I've tested all the changes on my local dev miner, everything seems to be operational.
This is the diagnostics page with all containers running and API issues:

This is the diagnostics page with the helium-miner container down and API issues:

Let's start testing it then?

Let's start testing it then?

I'll try to get another code review on the changes just to be sure it's good to push, as I reviewed the changes Marvin had made but no one has reviewed the changes I pushed after


bump bazel-toolchain to fix sysroot on Mac

Motivation

productivity++
Test plan

See included automated tests.

Will rebase with https://github.com/sorbet/bazel-toolchain/pull/6 once that merges, since that has an actual explanation of the problem and is more permanent than the commit I used here.

I'm guessing it is to fix the no LC_RPATH's found compilation error? If true, I just tried locally and it didn't seem to resolve the issue. Do we need to https://github.com/sorbet/bazel-toolchain/pull/6 to be merged first?

Oh, and d) what version of OS X you are running on and what /usr/bin/ld -v says.

Ahh bazel clean --expunge did the trick 🎉
I'm seeing a few warnings:
-macosx_version_min has been renamed to -macos_version_min
-macosx_version_min has been renamed to -macos_version_min
ld: warning: ignoring duplicate libraries: '-lm', '-lpthread'

But Sorbet does indeed build and no more no LC_RPATH's found.
Thanks! ❤

I changed this to pull in sorbet/bazel-toolchain#7, which reverts sorbet/bazel-toolchain#6 and solves the issue in IMHO a better way.  The new approach is more obviously correct and also more compatible with upstream, should we choose to take that route.
This also builds on Sonoma for me, running tests in -c dbg and --config=buildfarm-sanitized-mac.


Supabase Storage .getPublicUrl is not working correctly

Bug report
Describe the bug
When calling the getPublicUrl in swift, it is not reutrning the correct format for the public image.
I am calling it like this -
let profileUrl: URL = try await client.storage
                .from(id: "bucket2")
                .getPublicUrl(path: "1-profile.jpg")

and it returns back this
INCORRECT - https://PROJECT_ID.supabase.co/object/public/1-profile.jpg?
but the correct format should be
CORRECT - https://PROJECT_ID.supabase.co/storage/v1/object/public/bucket2/1-profile.jpg
This cause the data to now load.
To Reproduce

create a bucket called bucket2 that is public
upload an image called 1-profile.jpg to it
run the snippet above

Expected behavior
the correct url should allow you to view the file
System information

OS: macOS
Version of supabase-swift: 3.0.0

I'm currently getting the same issue. When I paste the url in the browser I get this error message:
{"message":"No API key found in request","hint":"No apikey request header or url param was found."}


Document using next to stop executing a lane

Documenting the research findings for #5913

I just noticed, this should also be updated https://github.com/fastlane/docs, where we also have an Advanced.md. Currently those files are not synced automatically, as we probably want to remove all remaining docs from the main repo in the future

@mfurtak can you also copy that to docs.fastlane.tools or should I do it?

Yes, I'll edit it there as well


try healthz api when readyz api is forbidden

see problem https://github.com/open-cluster-management/backlog/issues/3186

/assign @deads2k

/assign @pmorie

/lgtm
/approve


Add RcodeZero to DNS providers

We do support CDS and CDNSKEY on RcodeZero. DELETE options and _dsboot is not implemented yet.

Sweet! Do you also have plans to consume such records at the registry?

Thanks!

Sweet! Do you also have plans to consume such records at the registry?

As far as I know this is currently not planned.


String concatenation in yml

Can I perform string concatenation in yml file.
(https://stackoverflow.com/questions/53184056/string-concatenate-in-yml-file-and-use-with-aspnetcore-2-1)

Steeltoe.Common has several methods for working with Property Placeholders, though one was added in the last several weeks and hasn't made it out to nuget.org yet, and we expect to add another method or two, probably in Steeltoe Configuration, to streamline the overall experience before we release 2.2.
In 2.1 and below, you can already manually resolve placeholders with code like this:
string result = PropertyPlaceholderHelper.ResolvePlaceholders("foo=${foo}", someIConfiguration)

With the recent addition, you can resolve all placeholders in your IConfiguration, though you still have to discretely add the results of placeholder resolution as another config provider, something like
config.AddInMemoryCollection(
               PropertyPlaceholderHelper.GetResolvedConfigurationPlaceholders(config.Build()));

Which you can see demonstrated in the eShopOnContainers Sample
The code is here
The tests are here

I think the above is a resolution, please re-open if you have further questions


Proposal: Introduce ComboBoxDropdownBackground Property in WinUI 3

Description:
The current ComboBox control in WinUI 3 obliges developers to override the entire ControlTemplate to modify the background color of the dropdown container. This process is cumbersome and potentially error-prone, especially when the modification is as straightforward as a simple visual change.
Use Case:
The ability to easily customize the background color of the entire dropdown container is important for:
Branding: To align the application's ComboBox with corporate or design guidelines.
User Experience: To offer enhanced visual cues and improve usability.
Flexibility: To simplify theming and styling across different scenarios and layouts.
Proposed Solution:
Introduce a new XAML property for the ComboBox in WinUI 3, named ComboBoxDropdownBackground. This property will target the background color of the entire dropdown container, making it easier for developers to apply quick visual changes without dealing with ControlTemplates.

Alternatives Considered:
Global Styling: Would affect all ComboBox instances, and therefore isn't a desirable solution for every use case.
Third-party Libraries: Using an external library to achieve this might introduce unnecessary overhead or conflicts with existing features.
Overriding ControlTemplate: This approach seems overkill for such a simple visual modification and could introduce unwanted side effects.
Additional Context:
Simplifying the way for developers to make this particular modification can streamline the development process. It will also make the framework more approachable for newcomers and reduce the risk of errors that can arise from meddling with complex ControlTemplates.

This is already possible using lightweight styling without touching the control template.
The two brushes that you need to modify are:
    <SolidColorBrush x:Key="ComboBoxDropDownBackground" Color="Pink"/>
    <SolidColorBrush x:Key="ComboBoxDropDownBorderBrush" Color="Red"/>

@DHancock  thank you.  Im now closing the proposal.


Error -2145844819 when submitting *.clickonce but only signs .exe when submitting *.zip

When we submit our *.clickonce file we get a 500 error (full error below). However, if we submit that same file using the .zip extension it signs only the *.exe files.
I am inclined to believe the issue lies in either our folder structure or in how our filelist.txt file is configured. Is there any further guidance available on the structure of the ClickOnce publish folder that is submitted and/or the filelist.txt structure?
Our folder is structured as follows:

Root

Application Files (folder)

Application Version (folder)

dlls
pdbs
applicationName.exe.config
applicationName.exe
applicationName.exe.manifest
applicationName.ico

setup.exe
applicationName.exe
applicationName.manifest

Our filelist.txt reads like this:
**/Setup.exe
**/applicationName.application
**/applicationName.exe.manifest
This is the command being passed to the deployments server:
C:\SignService\src\SignClient\bin\Debug\netcoreapp3.1\SignClient.exe 'Sign'  --baseDirectory 'C:\Temp\ToSign' --input '**/*.{appxbundle,appinstaller,zip,nupkg,clickonce}' --config 'C:\SignService\scripts\appsettings.json' --filelist 'C:\SignService\scripts\filelist.txt' --user '***' --secret '***' --name 'applicationName' --description 'applicationName Desktop' --descriptionUrl 'https://www.applicationName.com/support' --output 'C:\Temp\ToSignCompleted'

Error message:
C:\SignService\src\SignClient\bin\Debug\netcoreapp3.1\SignClient.exe : Error signing 'C:\Temp\ToSign\SignClient.clickonce'
At line:1 char:1
+ C:\SignService\src\SignClient\bin\Debug\netcoreapp3.1\SignClient.exe  ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (Error signing '...ient.clickonce':String) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError

Server returned non Ok response: 500 Internal Server Error
Exception: System.AggregateException: One or more errors occurred. (Response status code does not indicate success: 500 (Internal Server Error).)
 ---> System.Net.Http.HttpRequestException: Response status code does not indicate success: 500 (Internal Server Error).
   at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
   at SignClient.SignCommand.<>c__DisplayClass3_1.<Sign>b__3(FileInfo input) in C:\SignService\src\SignClient\SignCommand.cs:line 325
   at System.Threading.Tasks.Parallel.<>c__DisplayClass33_0`2.<ForEachWorker>b__0(Int32 i)
   at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`1.<ForWorker>b__1(RangeWorker& currentWorker, Int32 timeout, Boolean& replicationDelegateYieldedBeforeCompletion)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw(Exception source)
   at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`1.<ForWorker>b__1(RangeWorker& currentWorker, Int32 timeout, Boolean& replicationDelegateYieldedBeforeCompletion)
   at System.Threading.Tasks.TaskReplicator.Replica`1.ExecuteAction(Boolean& yieldedBeforeCompletion)
   at System.Threading.Tasks.TaskReplicator.Replica.Execute()
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.TaskReplicator.Run[TState](ReplicatableUserAction`1 action, ParallelOptions options, Boolean stopOnFirstFailure)
   at System.Threading.Tasks.Parallel.ForWorker[TLocal](Int32 fromInclusive, Int32 toExclusive, ParallelOptions parallelOptions, Action`1 body, Action`2 bodyWithState, Func`4 bodyWithLocal, Func`1 localInit, Action`1 localFinally)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw(Exception source)
   at System.Threading.Tasks.Parallel.ThrowSingleCancellationExceptionOrOtherException(ICollection exceptions, CancellationToken cancelToken, Exception otherException)
   at System.Threading.Tasks.Parallel.ForWorker[TLocal](Int32 fromInclusive, Int32 toExclusive, ParallelOptions parallelOptions, Action`1 body, Action`2 bodyWithState, Func`4 bodyWithLocal, Func`1 localInit, Action`1 localFinally)
   at System.Threading.Tasks.Parallel.ForEachWorker[TSource,TLocal](IList`1 list, ParallelOptions parallelOptions, Action`1 body, Action`2 bodyWithState, Action`3 bodyWithStateAndIndex, Func`4 bodyWithStateAndLocal, Func`5 bodyWithEverything, Func`1
localInit, Action`1 localFinally)
   at System.Threading.Tasks.Parallel.ForEachWorker[TSource,TLocal](IEnumerable`1 source, ParallelOptions parallelOptions, Action`1 body, Action`2 bodyWithState, Action`3 bodyWithStateAndIndex, Func`4 bodyWithStateAndLocal, Func`5 bodyWithEverything,
Func`1 localInit, Action`1 localFinally)
   at System.Threading.Tasks.Parallel.ForEach[TSource](IEnumerable`1 source, ParallelOptions parallelOptions, Action`1 body)
   at SignClient.SignCommand.Sign(CommandOption configFile, CommandOption inputFile, CommandOption baseDirectory, CommandOption outputFile, CommandOption fileList, CommandOption clientSecret, CommandOption username, CommandOption name, CommandOption
description, CommandOption descriptionUrl, CommandOption maxConcurrency, CommandOption loggingLevel) in C:\SignService\src\SignClient\SignCommand.cs:line 262

Can I ask what your solution was to this problem? We seem to be having the same issue with our setup and we haven't been able to solve it yet.

Can I ask what your solution was to this problem? We seem to be having the same issue with our setup and we haven't been able to solve it yet.

You will have to forgive me as it appears I left the error codes out of my comments.  In hindsight that was a terrible oversight.
I will only really be able to provide you with the updates we made to overcome the various errors we encountered in the process.
One issue was that we kept submitting our archive file as a .zip file and overlooked the fact that we needed to update the file extension to .clickonce before shipping it to SignService.  We added an Archive files task to our job, used the archive type of zip but created a .clickonce file.
Task type: Archive Files
Root folder to archive:
$(build.artifactstagingdirectory)/publish/app.publish
Archive Type:
zip
Archive file to create:
$(Build.ArtifactStagingDirectory)/$(SignClientStaging)/$(Build.BuildId).clickonce
As you can see from above, we are zipping the app.publish folder.  In our case this is:
C:\agent\_work\1\a\publish\app.publish
You can adjust it to the appropriate _work directory.
In the final Authenticode (a PowerShell task) we found the need to explicitly reference the full path to SignClient.exe.  In our case this variable (SignClientPath) was set to:
C:\SignService\src\SignClient\bin\Debug\netcoreapp3.1\SignClient.exe
We also needed to add the .clickonce extension to our input values in the Authenticode.
Here is our final PowerShell task for signing with Authenticode.
Two things to note:

I have broken syntax and added line breaks to make it easier to read but they would need to be removed
The single and double quotes are literals and are all part of the syntax that should not be removed

Task type: PowerShell
Type: Inline
Script:
$(SignClientPath) 'Sign'
--baseDirectory '$(Build.ArtifactStagingDirectory)\$(SignClientStaging)'
--input '**/*.{appxbundle,appinstaller,zip,nupkg,clickonce}'
--config '$(SignServiceScripts)\appsettings.json'
--user '$(SignServiceUser)'
--secret '$(SignServiceSecret)'
--name "Your Application Name"
--description "Company Name"
--descriptionUrl 'The URL used to deploy the app'
--filelist '$(SignServiceScripts)\filelist.txt'
--output '$(Build.ArtifactStagingDirectory)\$(SignClientShipping)'

We also had to make adjustments to our filelist.txt file in C:\SignService\scripts as we did not initially include the ApplicationName.exe.deploy:
**/setup.exe
**/ApplicationName.exe
**/ApplicationName.manifest
**/ApplicationName.application
**/ApplicationName.exe.deploy


Can the "parameters" type be "NSString *"?

- (NSURLRequest *)requestBySerializingRequest:(NSURLRequest *)request
                               withParameters:(id)parameters
                                        error:(NSError *__autoreleasing *)error

while "parameters" was converted into JSON or encrypted with AES.
the following assignment query = AFQueryStringFromParametersWithEncoding(parameters, self.stringEncoding); will take a wrong URL string.
I don't know how many kinds of parameters. I'm using [parameters isKindOfClass[NSDictionary class]] || [parameters isKindOfClass[NSArray class]]  to avoid these two situations.
Is there any else?
Thanks for all your help.

Excuse me. Why not make it possible to support a string?  @kcharwood

Any news on this?

The parameters are most often key/value pairs so if you pass in a dictionary it is converted to a key/value pair parameter string. @{@"name":@"chau", @"date":@"September 15, 2015"} becomes: @"&name=chau&date=September 15, 2015"
Is there something different that you want to accomplish?

thanks for your answer.
but there is a problem: the parameters must be encrypted with AES in our server.
just like this: 1E0977332BDDFF784EC2C2F34D67A9F3961F2318C6ECA27F6A24248706EE0962
I think there is lots of coders were bothering with this problem.

:+1:

Can you use the setQueryStringSerializationWithBlock method on AFURLRequestSerializer to set create your on query parameter serialization logic? That seems like the best place for it.

@kcharwood Thanks for owner's answer. It has fix my issue. Thanks again.


LSP - Codelens and Intellisense?

The only meaningful output I get from the language server is #268
Is there something I'm doing wrong to not get Intellisense and Codelens support?
My config:
{
  "terraform.codelens.enabled": true,
  "terraform.format": {
    "enabled": true,
    "formatOnSave": true,
    "ignoreExtensionsOnSave": [
      ".tfsmurf"
    ]
  },
  "terraform.indexing": {
    "enabled": false,
    "liveIndexing": false,
  },
  "terraform.languageServer": {
    "enabled": true,
    "installCommonProviders": true,
    "pathToBinary": "/Users/redacted/.vscode/extensions/mauve.terraform-1.4.0/lspbin",
    "args": []
  },
  "terraform.lintPath": "/usr/local/bin/tflint",
  "terraform.path": "/usr/local/bin/terraform",
  "[terraform]": {
    "editor.formatOnSave": true
  }
}

Still open? Would have thought this wouldn't be a difficult fix?

We just released v2.0.0-rc.1 of the extension. The main features include:

Added syntax support for 0.12
Added terraform-ls usage by default (currently on 0.3.0, which offers basic provider code completion)

You can find additional information and specifics in the release notes and CHANGELOG.
With this release we expect that many of the prior issues and PRs are no longer relevant or have been addressed, and are therefore being closed. If you feel the action taken on an issue or PR is in error, please comment as such and we can figure out the appropriate way to address it.
We plan to add the final 2.0.0 release to the marketplace soon, but are actively seeking your feedback now on the release candidates. You can download the .vsix from the releases page and manually install it in VS Code to try it out.


A project containing no imports shouldn't fail to load unintelligibly in VS

If a project has no imports, then the XAML rules needed by the project are not brought in and we fail to load the project and show an error about a missing the "BrowseObject" catalog. This may require a CPS fix? We should either just load successfully (like the old project system) or fail with a more intelligible error about missing critical targets or something. Since csproj is editable now, it's more likely for someone to get into this state.

Tagging @jviau and @lifengl. Per #1012 we now crash VS in this case. Can we pursue some fix for this for RTM?

@srivatsn this is a CPS issue. We won't be able to load the project, as without the XAML rules CPS has no idea how to interpret the project file. The best we could do is a descriptive error. Maybe we can load the .csproj file but have an empty tree with a message saying that no xaml rules were loaded.
I opened a bug in our database to track this.

Thanks i'll keep this issue open as a tracking item.

@srivatsn is there an issue tracking the 2nd point @adrianvmsft pointed out? ProjectLockFileWatcher.GetProjectLockFilePathAsync crashes VS if BaseIntermediateOutputPath is not defined in the project or set to an empty string. For example, shared projects do not have that property set, so trying to load one right now crashes VS due to this.

I think that was #1012 but got duped against this one. So I guess we could either reactivate it or have this bug (but in that case we should remove the Tracking tag).

@srivatsn @davkean Just a reminder that the crash reported above (Issue 2) needs to be fixed on the roslyn project system side (the CPS side of the fixes are in, but they won't prevent the crash)

@adrianvmsft Tracking via: https://github.com/dotnet/roslyn-project-system/issues/1129


chore: update penumbra deps to 0.80.3

We were dependent on a non release penumbra tag waiting for new release with fixes to be made, this new release includes fixes.

Closing this PR as being stale. Please open a new PR to bump to a more recent version of penumbra.


[doc] Updates docs for cleanly upgrading to release/v4.4 ESP-IDF SDK.

Problem
Fix #13602
Change overview
Update docs with how to cleanly upgrade an existing ESP-IDF to the latest version.
Update tag to use release/v4.4.
Testing
Doc only change.  CI.

@turon Please restyle :)

/rebase


Redirects for Website Section RSS Feeds

 Get RSS Feed set up for Google Publisher Center and Google News for all Brand Sites


cdc: always ignore sequence tables

What problem does this PR solve?

Issue Number: close #4552
What is changed and how it works?
Always ignore sequence tables
Check List
Tests

Unit test
Integration test

Related changes

Need to cherry-pick to the release branch

Release note
Fix a bug that sequence should not be replicated even if force-replication is true.

[2022-02-10T13:34:22.486Z] === FAIL: pkg/p2p TestClientSendAnomalies (unknown)

10T13:34:22.487Z] assert: mock: I don't know what to return because the method call was unexpected.

[2022-02-10T13:34:22.487Z] 	Either do Mock.On("Flush").Return(...) first, or remove the Flush() call.

[2022-02-10T13:34:22.487Z] 	This method was unexpected:

[2022-02-10T13:34:22.487Z] 		Flush()

[2022-02-10T13:34:22.487Z]

[2022-02-10T13:34:22.487Z] 	at: [client_test.go:45 client.go:319 client.go:211 client.go:199 errgroup.go:57 asm_amd64.s:1371]

/run-integration-tests

/run-integration-test

/run-integration-test

/merge

/run-kafka-integration-test

Codecov Report

Merging #4563 (ca63d36) into master (9607554) will increase coverage by 0.0496%.
The diff coverage is 59.2169%.

Flag
Coverage Δ

cdc
60.2698% <54.6572%> (+0.3476%)
:arrow_up:

dm
51.8310% <61.2598%> (-0.1979%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #4563        +/-   ##
================================================
+ Coverage   55.6402%   55.6898%   +0.0496%
================================================
  Files           494        507        +13
  Lines         61283      63042      +1759
================================================
+ Hits          34098      35108      +1010
- Misses        23750      24435       +685
- Partials       3435       3499        +64

/merge

/run-kafka-integration-test

/run-kafka-integration-test

/run-leak-test

/run-integration-test
/run-dm-integration-test

/run-integration-test

/run-integration-test

/run-integration-test

/run-integration-test

/run-integration-test

/run-integration-test

/run-integration-test /tidb=pr/32434

/run-leak-test
/run-integration-test
/run-dm-integration-test

/run-verify

/run-leak-test


New version: Microsoft.Edge version 126.0.2592.87

Checklist for Pull Requests

[ ] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

/azp run

@hackean-msft are you able to kick this off?

/azp run

https://github.com/microsoft/winget-pkgs/issues/142854


DSA <IP_ADDRESS> character based counting problems in strings

Description
Concatenated string with uncommon characters of two strings
Task

[ ] Audio
[ ] Video
[ ] Documentation

Note

While requesting the issue to be assigned, please mention your NAME & BATCH NUMBER.
Mention the type of contribution you want to make.
Kindly refer to the Issue Guidelines.
Do not use the assign command for this issue. Follow the above procedure.
In case you see that the issue has already been assigned, try to request a different task within the same issue.
Changes should be made inside the DSA/directory & DSA branch.
Name of the file should be the same as the issue.
This issue is only for 'GWOC' contributors of the 'DSA' Domain.

Domain
DSA
Type of Contribution
Audio, Video, Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

I am ready to work on the documentation part of this issue. Please assign me this.
Name: Pranshu Kashyap
Batch: DSA Batch 5
Type of Contribution: Documentation

@goelesha Please assign this issue to @pranshukas

Please do assign this to me.

Please do assign this to me.


Removed anchor tag

Removed anchor tag which hyperlinked most of the content

Docs Build status updates of commit 9c9709d:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-conceptual/azps-4.6.1/uninstall-az-ps.md
:warning:Warning
View (azps-4.6.1)
Details

docs-conceptual/azps-4.6.1/uninstall-az-ps.md

Line 2, Column 1: [Warning-ms-prod-or-service-missing] One of the following attributes is required: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.
Line 2, Column 8: [Suggestion-duplicate-titles] Title 'Uninstall Azure PowerShell' is duplicated in: 'docs-conceptual/azps-4.6.1/uninstall-az-ps.md(2,8), docs-conceptual/azurermps-6.13.0/uninstall-azurerm-ps.md(2,8)'. Titles must be unique within a docset.
Line 3, Column 14: [Suggestion-duplicate-descriptions] Attribute 'description' with value 'How to perform a complete uninstall of Azure PowerShell' is duplicated in 'docs-conceptual/azps-0.10.0/uninstall-az-ps.md(3,14)', 'docs-conceptual/azps-1.8.0/uninstall-az-ps.md(3,14)', 'docs-conceptual/azps-2.8.0/uninstall-az-ps.md(3,14)', 'docs-conceptual/azps-3.8.0/uninstall-az-ps.md(3,14)', 'docs-conceptual/azps-4.5.0/uninstall-az-ps.md(3,14)'.... NOTE: This Suggestion will become a Warning in late October, 2020.
Line 9, Column 1: [Suggestion-duplicate-h1s] H1 'Uninstall the Azure PowerShell module' is duplicated with other articles: 'docs-conceptual/azps-4.6.1/uninstall-az-ps.md(9,1), docs-conceptual/azurermps-6.13.0/uninstall-azurerm-ps.md(9,1)'. First level headings must be unique within a docset. NOTE: This Suggestion will become a Warning in late September, 2020.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

@piyushmutha Thank you for submitting a PR to correct this issue! We received an issue for it and our CXP team submitted a PR prior to this one that we've merged and it should go live soon. #1562


Port value for Zipkin Tracing should be not defaulted

ZipkinTracerBuilder created based on config, as specified in https://helidon.io/docs/latest/#/mp/tracing/02_zipkin, defaults port to 9411, if not specified.
As the config for host allows for FQDN, port should not be defaulted. I am running into issue pushing/sending Traces to OCI APM service since the collection/ingestion endpoint for the same does not require/specify port https://docs.oracle.com/en-us/iaas/application-performance-monitoring/doc/configure-open-source-tracing-systems.html#APMGN-GUID-B5EDE254-C854-436D-B844-B986A4E077AA (which will be case when user is using Zipkin as service, rather than hosting it).

In theory this should work by specifying a port number in the config that matches the default port number for the protocol (80 for http or 443 for https). But apparently Arjav ran into a service that rejects the HTTP request in this case -- maybe because the service was not expecting a port number in the Host: header.
We do not want to change the default behavior because that would break compatibility. A couple options instead:

Interpret -1 for port to mean do not use a port number in the request URL.
Add a new property, like noPort to indicate no port number

The first option is appealing because it's providing a work-around for a bug in another product without inventing something new. And it is consistent with URL.getPort()

Please specify port that is valid for your address - if you specify https://zipkin.myserver.com, the port should be 443. Using default port for a protocol is not supported, as Zipkin does not use 80 or 443 by default (in docker/k8s installations) - so we decided to use the default port of zipkin installation (which is indeed 9411).
TL;DR:
The default is correct - Zipkin uses 9411 as the default port. If you need to use a different port, please specify it explicitly.

Closing as this is the designed and intended behavior.

I think there might still be in issue here. We should make sure specifying "-1" as a port number translates to "no port number" and we should make sure that specifying a collector URL with no port number preserves the URL with no port number. Specifically, this code looks suspicious: https://github.com/oracle/helidon/blob/799ebd22ceae7019b7a903b482c35635264615e3/tracing/tracing/src/main/java/io/helidon/tracing/TracerBuilder.java#L163

I create #3966 for the issue described in the previous comment.


Incorrect password entered 10 times, max incorrect password limit of 10 reached

Incorrect password entered 10 times, max incorrect password limit of 10 reached

@mrbardia72  - what is the issue?


refactor: Use single settings.gradle for flank projects

Fixes #1112
Test Plan

How do we know the code works?

All github actions should work.

The changes here will affect the incoming windows workflow #770 . which one should be merged first?

The changes here will affect the incoming windows workflow #770 . which one should be merged first?

I think that this PR should be merged first, however Janek is off until Friday, maybe someone could take care of it?

I think that this PR should be merged first, however Janek is off until Friday

Feel free to finish it without me ;).

I think that this PR should be merged first, however Janek is off until Friday

Feel free to finish it without me ;).

I will, not worries :D

Integration tests fixed