Multiple ddns updates

Finally a script that works flawless! Thanks for that!
Only thing that is missing is a multi domain config file that can be used. I have 8 domains but I don't want 8 scripts ;-)
Is that something that can be considered ? If so, I can offer myself as beta tester ;-)
Keep up the good work !

https://github.com/timetoexpire/cloudflare-ddns-updater

Thanks for your help but I can need some more because the documentation is rather cryptic. I have installed your forked script and it works for 1 domain. But I'm stuck how to update
domain1.org
domain2.co.uk
.....
These have all different parameters but no clue how to proceed with a config-file. Is this possible or do I need to create a script for each domain ? Your mail function is very useful but... can you configure it (when to sent an email) and how can I test it. I think the info is in the source code, but that is not my cup of tea (I hardly can read it)

With config file it would go somthing like. This
-zone_identifier="xyz45678901234567890123456789001"
-record_name="domain1.org"
-zone_identifier="abc45678901234567890123456789002"
d542e0aeab6d68ad70e43e871ca0c953
-record_name="domain2.co.uk"

As diffent domains use diffrent zone identifier.
The thing got to make shore that API Zone Resources (auth/_key) for that is allow to edit the record for both domain1.org and domain2.co.uk. If the have diffrent then have do something like
-zone_identifier="xyz45678901234567890123456789001"
-auth_key="123456789012345679456446"
-record_name="domain1.org"
-zone_identifier="abc45678901234567890123456789002"
-auth_key="546833542486404054803540"
-record_name="domain2.co.uk"

You can use white space.
-zone_identifier="xyz45678901234567890123456789001"
  -auth_key="123456789012345679456446"
  -record_name="domain1.org"

-zone_identifier="abc45678901234567890123456789002"
  -auth_key="546833542486404054803540"
  -record_name="domain2.co.uk"

I would recomed that edit the config_file using Linux. If edit in using Windows need to remove \CR, this has fix in next version coming latter on this mounth.

Do you have a working example of a config file ? Or is it the above in a file (eg. cf_ddns.ini) and configured in your script header like depicted below?
proxy=false                                                       # Set the proxy to true or false
slackuri=""                                                        # URI for Slack WebHook "https://hooks.slack.com/services/xxxxx"
config_file="/install/dns-updater/cf_ddns.ini"     # set path and configuration-file name
email_username=

That what that look like in config_file
-proxy="true"

-proxy="false"

-slackuri="https://hooks.slack.com/services/12345678"

You can not call config_file with in config_file. You can only have 1 config_file running. You set within the script (.sh) or bash line
# cloudflare_template.sh -config_file="/install/dns-updater/cf_ddns.ini"

<EMAIL_ADDRESS>-email_password="abcdfsd"
-email_smtp="smtp.gmail.com"
<EMAIL_ADDRESS>-email_fromName="Cloudflare 20220126"
<EMAIL_ADDRESS>-email_toName="Hello There"

Thanks for the help. I will post the results soon ;-)

Could go in script at bottom and remove the # on
echo "paramter :${cf_setting_parameter_array[*]}"

echo "file :${cf_setting_file_array[*]}"

You need to change as per your account at Cloudflare.
Here video of how it all done. You need to enter your own details which get from cloudflare/slack. Remove the -ipset my IP dosn't change that quickly ;)
https://youtu.be/e1N5XLlAxc8
It not best video ever but would help understand. Hopefuly get idea. The strange error of invailed parameter option been defined. But hopeful the update will be out this mounth https://github.com/timetoexpire/cloudflare-ddns-updater/discussions/2
-auth_email=""
-slackuri="https://hooks.slack.com/services/XXX/YYY/ZZZ"
-zone_identifier="zi-tte"
-#="tte"
  -auth_method="token"
  -proxy="false"
  -proxy="true"
  -auth_key="ak-tte"
  -ipset="<IP_ADDRESS>"
  -record_name="test3.timetoexpire.co.uk"
  -sleep=10
  -ipcheck=
  -record_name="test3.timetoexpire.co.uk"
  -ipset="<IP_ADDRESS>"
  -record_name="test4.timetoexpire.co.uk"
  -sleep=20

-zone_identifier="zi-rp2"
-#="rp2"
  -proxy="true"
  -auth_key="ak-rp2"
  -ipset="<IP_ADDRESS>"
  -record_name="test1.rp2021.co.uk"
  -sleep=10
  -ipset="<IP_ADDRESS>"
  -record_name="test2.rp2021.co.uk"
  -sleep=20
  -rsleep=10

-zone_identifier="zi-tte"
-#="tte"
-auth_key="ak-tte"
  -ipset="<IP_ADDRESS>"
  -record_name="test3.timetoexpire.co.uk"
  -sleep=4
  -ipset="<IP_ADDRESS>"
  -record_name="test4.timetoexpire.co.uk"
  -sleep=20
  -rsleep=10

-zone_identifier="zi-rp2"
-#="rp2"
  -proxy="true"
  -auth_key="ak-rp2"
  -ipset="<IP_ADDRESS>"
  -record_name="test1.rp2021.co.uk"
  -sleep=10
  -ipset="<IP_ADDRESS>"
  -record_name="test2.rp2021.co.uk"
  -sleep=10

-zone_identifier="zi-rp2"
-#="rp2"
-auth_key="ak-rp2"
-ipcheck=""
-record_name="test1.rp2021.co.uk"

Thanks for your effort so far. Decided to create individual scripts per A record and call these via 1 batch in cron. It is very desirable to have this in 1 script but in case of a less easy setup, documentation is king. For people that are not aware of slackuri ( Like me ), this is a second dimension in this solution which is not pointed out.
Good (process) documentation and good examples are really an added-value to your effort. Love to use it, but I'm looking forward to a good manual

First thanks for trying, and I have been able gain a little insight from your trials and tribulations.
The docments are already being updated. I not just the wording, but how it is formated/presented is also important.
You comments have been duly noted and hopeful you would choose to use my solution at a later date.
Documentation is a weak spot of mine personally as I have dyslexia, but I always try to improve. https://youtu.be/PEnJbjBuxnw ;-)


Update Readme.md

PR Description 📜
Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below
Fixes #3314

Mark the task you have completed ✅
<!----Please delete options that are not relevant. In order to tick the check box just but x ins
- [ ] I follow [CONTRIBUTING GUIDELINE](https://github.com/kunjgit/GameZone/blob/main/.github/CONTRIBUTING_GUIDELINE.md) & [CODE OF CONDUCT](https://github.com/kunjgit/GameZone/blob/main/.github/CODE_OF_CONDUCT.md) of this project.
- [ ] I have performed a self-review of my own code or work.
- [ ] I have commented my code, particularly in hard-to-understand areas.
- [ ] My changes generates no new warnings.
- [ ] I have followed proper naming convention showed in [CONTRIBUTING GUIDELINE](https://github.com/kunjgit/GameZone/blob/main/.github/CONTRIBUTING_GUIDELINE.md)
- [ ] I have added screenshot for website preview in assets/images
- [ ] I have added entries for my game in main README.md
- [ ] I have added README.md in my folder
- [ ] I have added working video of the game in README.md (optional)
- [ ] I have specified the respective issue number for which I have requested the new game.

<hr>

## Add your screenshots(Optional) 📸

---

## Thank you soo much for contributing to our repository 💗

Thank you @msifran ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Hey @msifran,
You can't work on a closed issue 👀!
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗

Thank you  @msifran , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Markers not visible with when sticky header enabled

Probably need to move the header marker into the <Timebar />

Fixed


Remove programatic height

When you upload a JSON file the text height is zero, so this sets textarea heights to zero.
While it works great while live editing, a loaded JSON file when edited gets set to zero.

If this is used somewhere (I see there's a different spot that uses a similar function), I can change it to just set min-height so this error goes away.  But I figured remove the issue if the code isn't doing anything.


MPSL and New Compilation

For 5.59 (upgrading from 5.58), mpsl is now included in mp-5.x?  No need to clone the separate repo mpsl any longer? That seems to be behind a login wall, now, anyhow.
If I have the repo mpsl cloned previously, do I need to delete that to make sure mp-5.x uses the mpsl in its own tree?

yes, mpsl is inside MP right now (as well as the other component, mpdm). you can delete the mpsl repo if you want. you may need to run config.sh again if you have an old version.

OK, thanks.


Loading Screen with Thingsboard Icon

I can't seem to disable or change the loading screen which has the thingsboard icon. Is there a way to do so?

Hello @vsood97.
The ThingsBoard loading indicator consists of two assets.

assets/images/thingsboard_center.svg
assets/images/thingsboard_outer.svg

You can substitute these assets with your own or completely modify the screen by altering the TbProgressIndicator class.


Textangular issues

I have no idea where else to post this, looks like fraywing:text-anuglar on github is the original project and its not clear who maintains it on meteor.
http://meteorpad.com/pad/eedqTbamxnyHKcAK6/Textangular
Textangular gives the following error when included.
WARNING: Tried to load angular more than once.

yeah, I didn't find a good package too.. you can try creating your own package following this steps http://angular-meteor.com/tutorials/angular1/3rd-party-libraries

Thanks Shinrox, looks like I have some learning to do :)

@netanelgilad I think you have created one as well right?

yes I think you can also use my package here: https://github.com/netanelgilad/meteor-textAngular.
If you have problems with it, don't hesitate to open an issue


Correction for raw affiliation Lumière University Lyon 2, Géography Research Institut (IRG-UMR 5600)

Correction needed for raw affiliation Lumière University Lyon 2, Géography Research Institut (IRG-UMR 5600)
raw_affiliation_name: Lumière University Lyon 2, Géography Research Institut (IRG-UMR 5600)
new_rors: 029brtt94;03rth4p18;0256r1e04
previous_rors: 029brtt94;03rth4p18
works_examples: W106313651
contact: 2d49689be43d39c3d99a1d43eb9724cb:c920bf9e587f413cafc4dbd18e1b @ ens-lyon.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


Move Zipkin exporter to another repo.

Only the zipkin exporter depends on thrift, and the thrift dependency has caused us problem like https://github.com/census-instrumentation/opencensus-go/issues/1092. By moving the zipkin exporter to a separate repo we should be able to remove thrift dependency from OC-Go core library.

We need to decide where the exporters should be located before this can happen. /cc @rghetia

Done with v0.21.0 release.


API endpoints login , logout and user-info added with the corresponding modules needed in requirements file

Summary by Sourcery
Introduce new API endpoints for login, logout, and user-info to manage user authentication and sessions. Enhance security by implementing JWT token decoding and verification with AWS Cognito. Update session management with secure cookie settings and session lifetime configuration. Update the requirements file to include necessary dependencies for these features.
New Features:

Add login, logout, and user-info API endpoints to handle user authentication and session management.

Enhancements:

Implement JWT token decoding and verification using AWS Cognito JWKS for secure authentication.
Introduce session management improvements with secure cookie settings and session lifetime configuration.

Build:

Update requirements.txt to include new dependencies: pyjwt, python-dotenv, requests, and cryptography.

@sourcery-ai review

@sourcery-ai review

@sourcery-ai review

@sourcery-ai review

@sourcery-ai review


Patch deployment diff

Includes #196
This PR gives us a stable server for now. It ignores diffs in Owners and Kinds of deployments. See also https://github.com/opentable/sous-gdm-alpha/commit/6b93f9915aaa0b7082736f7ed14f08674c3adaf3 and https://github.com/opentable/sous-gdm-alpha/commit/67c7b73175171e881b7723347e2f52ef0bbf501a
Now we have the server stabilised, we need to address two issues:

Ensure owners get written to Singularity (they don't at present)
Ensure Kind is always written in the manifests (at present it is always blank)

Once those are done, we can revert https://github.com/opentable/sous/commit/3e339acf441078f58b3992aa1964da5afd5b5d8c

Current coverage is 59.23% (diff: 70.00%)

Merging #198 into master will decrease coverage by <.01%

@@             master       #198   diff @@
==========================================
  Files           119        119
  Lines          6277       6332    +55
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           3719       3751    +32
- Misses         2243       2256    +13
- Partials        315        325    +10

Powered by Codecov. Last update 55a5764...3e339ac


Optionally specify sub directory on SourceRef

Git repos often have docs, notes, test scripts, etc. included in them. At the moment updating the docs inside a repo causes the tf-controller to perform a plan. If it were possible to specify a sub directory to monitor inside the sourceRef, similar to Flux Kustomize, then the plan would only be performed when required.
sourceRef:
  kind: GitRepository
  name: foo
  namespace: bar
  path: foo/bar

I must have missed that in the docs. The CrossNamespaceSourceReference does not mention a path element.

yep, both KS and TF sourceRef do not have that path field.
path only exists at the spec-level of both resources.
If you'd like to exclude some dirs or files. you'd be doing so at a Source definition.


docs(website): shorten "scikit-learn" to "sklearn"

"scikit-learn" rendered across two lines, elongating the box and causing inconsistency:

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.53%. Comparing base (0421c66) to head (f3187e5).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #113   +/-   ##
=======================================
  Coverage   84.53%   84.53%
=======================================
  Files          24       24
  Lines        1862     1862
=======================================
  Hits         1574     1574
  Misses        288      288

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Issuing credits to Evan

Issue 10000 credits to @gnomevan.

 Follow up
On Sunday, 24-Mar-24 06:33:46 UTC by Gov4Git dev
Issued 10000 credits to member @gnomevan.


Add pairs argument to Spanner Results

The pair method argument allows rows and types to be returned as a nested Array of pairs rather than a Hash. This will be useful for when users have rows that have duplicate names. The values can be
represented in a data structure that allows for duplicate keys, while being familiar to Rubyists.

@quartzmo @vkedia This is an attempt at handling #1449. What do you think?

@quartzmo @vkedia What do you think about separating these structures into separate methods? I'm thinking Results#rows could yield a Hash, and Results#row_pairs could yield an array of array pairs. Similarly, Results#types could return a Hash, and Results#row_pairs could return an array of array pairs. Thoughts?

yeah that sounds reasonable to me. But I would still prefer if the rows method threw an error if there are duplicates

I like the separate methods  Results#rows and Results#row_pairs.


New logging facility

Right now every message that the bridge writes is dumped into stderr or stdout. This should be replaced with a more structured format at some point.

This is now using the new logging facility.  Closing this issue.

This work spanned multiple commits - 52cf743d2c5e13e86597cb480cbca158b2bb5806, 53bf3f7e3177f8dc126412baedd7335c0059dc01, and 006533a88101b24474a13e4cb4b88ad2b6f48367.


OpenFPGA Stack: Project Formation

Project Documentation & Links
This issue tracks the new project "OpenFPGA Stack".
Project Name: OpenFPGA Stack
Project Repository: https://github.com/OFS
Application PR: {LINK TO PR/TBD}
Project Checklist

[x] TAC Vote June 11, 2024
[x] New Project Created In LFX
[ ] Legal Submission

[ ] Project formation form submitted by project
[ ] Technical Charter Draft Approved
[ ] Contribution Agreement Approved
[ ] Trademark Docs Approved

[ ] Board Vote  Date
[ ] Project made active in PCC by notifying the formation team
[ ] Merge PR & Confirm Website Is Accurate
[ ] Add ‘thelinuxfoundation’ as an owner of the GitHub org
[ ] Transfer any domains
[ ] Onboard to LFX/Community Metrics/Insights
[ ] Enforce CLA on repo(s)
[ ] Accept/adopt LF code of conduct
[ ] Enable security vulnerability reporting process

Followed up with intel on PR and legal submission.

Followed up again with Intel on tasks.

Will regroup with stakeholders in August.

Followed up with Intel on 8/5 and 8/13 without a response.

Check back in on Aug 23

@wendallkaters next steps are to reach out to Intel on next steps.


fix : update useless parameters

🔗 Linked issue

❓ Type of change

[x] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

in this link ( https://nuxt.com/docs/guide/directory-structure/pages#special-metadata ) ,


June 2019 Release Plan

This plan captures our work in June. This is a 4 week iteration, and we will ship our June Update in in early July. We'll continue work we started in the previous iteration, and we started to pick up items from our roadmap.
Endgame

June ~24~ 27 Code freeze for the endgame
July ~1st~ TBD Endgame done

The endgame details for this iteration are tracked in  #2905
Plan Items

[x] Consolidate Job Submission Experience @Binyang2014 @debuggy @sunqinzheng
issues: #2621
pr: #2867 #2885 #2887

[x] @wangdian team wise storage support

[x] New dedicated VC @mzmssg @sunqinzheng (backend ready)
issues: #2790
~pr: #2960 #2923 #2921~

[x] [Dedicated vc] Mapping available gpu nodes to vc #2991
[x] doc

[x] Python SDK and CLI @mydmdm
issues: #2808 ~ PR #3018
[x] Enable VSCode job protocol v2 support  #2866 @yiyione
pr: #2913
[x] ~Dockerfile Generator plugin~
[x] Duration in the Job list should follow the same format as Job details #2972

Engineering Improvements

Refine default plugin hosting and code location @abuccts

Bug fixes

[x] "View job" button on vc page doesn't work as expected #2870
[x] Need hidden hint when window width isn't enough for all information #2929

Deprecated

Submit job v1
Marketplace plugin

Pending Items

Job submitting wizard
Replace the "Document" side bar with the "help" icon. #2908
Hide "0" node GPUs from Available Nodes Chart #2909
webportal can't show legacy job log #2917

Deferred

[x] AAD and user management #2898 #2897 #1663 @ydye @sunqinzheng
pr: #2973 #2950

[x] Group management plugin @fenghuajia

Released: https://github.com/microsoft/pai/releases/tag/v0.14.0


Unable to start in HTTPS in docker container

Which version of the Azurite was used?
Latest version found in DockerHub
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
DockerHub
What problem was encountered?
Trying to use HTTPS and the cert and key generated using mkcert mentioned in the readme.md. When execute with --cert and --key, encounter the following error:
Exit due to unhandled error: ENOENT: no such file or directory, open '<IP_ADDRESS>.pem'
When try to run with only --cert param, it can start the container.
Steps to reproduce the issue?
Executing the command:
docker run -p 10000:10000 mcr.microsoft.com/azure-storage/azurite azurite-blob --blobHost <IP_ADDRESS> --blobPort 10000 --cert "<IP_ADDRESS>.pem" --key "<IP_ADDRESS>-key.pem"
Have you found a mitigation/solution?
No

Hi,
what operating system are you using docker in, and what is the location of your cert and key files?

Hi,
I am using Mac and the location of my cert and key files are in the download directory. I make sure i was in the download directory when i run the docker run command

@bendanye
The cert file is on you local machine, to make docker can access the cert file, you need to map the folder contains the files to docker. (in following sample, I map local folder  c:/azurite to /workspace on docker )
The command work for on my machine is ("c:/azurite" contains the cert file):
docker run -p 10000:10000 -v c:/azurite:/workspace   mcr.microsoft.com/azure-storage/azurite azurite-blob --blobHost <IP_ADDRESS>  --cert /workspace/<IP_ADDRESS>.pem --key /workspace/<IP_ADDRESS>-key.pem

@bendanye
I will close the issue as not receive your responds for 7+ days.
Feel free to contact us again if you need any further assistance on Azurite.


Added missing steps and clarified some sections

Additions:

modifications to android/settings.gradle
package importation in MainActivity.java

Clarifications:

dependencies to android/app/build.gradle
module registration in MainActivity.java

@saidimu thanks for clarifying this.


Fix PreferencesAppController applicationOpenURL:

Replace "root=Tweaks" with "root=[tweaksLabel stringByAddingPercentEscapesUsingEncoding:NSUTF8StringEncoding]"
Fix for issue #22

Close, but doesn't work for all tweaks.  Need to revert to the ugly method.


Add Whispers tab for keeping track of whispers

I was bored and my buddies and I felt that it would be nice to be able to see the whispers you get like in this app. That way you can keep track of the responses you get if you are advertising a group. Not sure if this is something you want to merge or it just becomes my own fork. Needs a little cleanup at the moment.

@iamameme let me know the status of this


Fixed missing partial for cluster drift analysis

https://bugzilla.redhat.com/show_bug.cgi?id=1332806

Please, make a small spec.


Move boot fatjar into own project

This PR moves the Spring Boot fat jar repackaging into it's own project (genie-app) so that genie-web can continue to be published as a regular "thin" jar and be recompiled into other web applications while simultaneously keeping the convenience of the executable boot jar for OSS deployment.
Database fields have also been moved from TEXT to VARCHAR type to improve performance.

Coverage increased (+0.02%) to 93.111% when pulling ef4f6465219760c7ed5b41d114d0827fd205f0d8 on tgianos:develop into c940874434bdb4d310250afb30cdecc58f14637e on Netflix:develop.


SymbolLoader - additional condition

The middleware for parsing flash symbols sometimes fails if resource has not been loaded and iterator function also fails at this point
else if (data.nodeName && data.nodeName === 'IMG') {
or at this
_ShapesCache2.default.add(resource.name, data);
when we pass empty object as shape data
In order to have able to continue loading flow even when loading fails (for example we wanna retry loading of this resource) please provide some additional conditions for this function to prevent empty object accces.
My solution is
if (!data){
next();
}else if (url.search(/.shapes.(json|txt)$/i) > -1) {
_ShapesCache2.default.add(resource.name, data);
} else if (data.nodeName && data.nodeName === 'IMG') {
// Add individual images to the texture cache by their
// short symbol name, not the URL
PIXI.Texture.addTextureToCache(PIXI.Texture.fromFrame(url), resource.name);
}
next();

Sounds good. Would you be interested in putting together a PR for this?

I'm on it! https://github.com/jiborobot/pixi-animate/pull/53


Release afs2-model 2.1.13

The release reason is about #33
Keep the module name afs to give old users and apps, and let they can fix only the installing package name.
Keep deprecate function to remind users, these functions will be disappeared in 2.2

I got to merge the feature to develop-branch, and create another PR from develop to master.


Add lock warpper for resource change.

Codecov Report

Merging #294 (ea16c73) into master (a8a0adb) will decrease coverage by 0.04%.
The diff coverage is 34.78%.

@@            Coverage Diff             @@
##           master     #294      +/-   ##
==========================================
- Coverage   66.46%   66.41%   -0.05%
==========================================
  Files          81       82       +1
  Lines        5976     5982       +6
==========================================
+ Hits         3972     3973       +1
- Misses       1461     1465       +4
- Partials      543      544       +1

Impacted Files
Coverage Δ

common/context.go
73.60% <0.00%> (-5.04%)
:arrow_down:

service/locker.go
0.00% <0.00%> (ø)

task/task.go
0.00% <0.00%> (ø)

api/api.go
58.82% <50.00%> (-0.56%)
:arrow_down:

plugin/default/lock/empty_lock.go
80.00% <100.00%> (-3.34%)
:arrow_down:

server/admin_server.go
84.14% <100.00%> (ø)

Continue to review full report at Codecov.

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


Fixed password reset

Is this tested? I tried it just now on a stage account but got no email. How do I test it so that it works?

Test it on live, we don't think stage has the feature
On Sat, Sep 11, 2021, 14:21 Lukas Gustavsson @.***>
wrote:

Is this tested? I tried it just now on a stage account but got no email.
How do I test it so that it works?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/fsek/App2/pull/78#issuecomment-917398334, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AOK3WVO6AWHCHBJIXR34N7TUBNCVFANCNFSM5DZKSQDQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Bruh the git app is weird. Where is the approve button


Glyphs are different sizes?

I just installed lsd and nerd fonts hack and I get glyphs for most file types but they are different and wrong sizes:

I didn't know if I should post here or on the nerd fonts page but since colorls behaves differently and had the folder glyphs in the right size I'm writing here.
I haven't installed a custom font since XFree86 had a font server so don't really know what I'm doing.
Installed fonts-hack-ttf on Pop OS! 22.04 and have also installed nerd fonts hack from their git repo in .local/share/fonts/NerdFonts/
It seems that that was the only way to get all glyphs working.
I've tried lsd in gnome terminal and guake and it looks the same.

c icon based on the c language logo. They are explicitly meant to have different icons.

c: https://github.com/Peltoche/lsd/blob/2ffb59cc9eab759d1eea23c217f19ac93ff0971f/src/icon.rs#L245
h: https://github.com/Peltoche/lsd/blob/2ffb59cc9eab759d1eea23c217f19ac93ff0971f/src/icon.rs#L313

That wasn't my question. If you look at the screenshot you'll see that the
icon for cpp is a "chinese" symbol.
On Wed, 22 Jun 2022, 18.19 Abin Simon, @.***> wrote:

c icon based on the c language logo. They are explicitly meant to have
different icons.

c:
https://github.com/Peltoche/lsd/blob/2ffb59cc9eab759d1eea23c217f19ac93ff0971f/src/icon.rs#L245
h:
https://github.com/Peltoche/lsd/blob/2ffb59cc9eab759d1eea23c217f19ac93ff0971f/src/icon.rs#L313

—
Reply to this email directly, view it on GitHub
https://github.com/Peltoche/lsd/issues/678#issuecomment-1163331477, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVWEIEF63H34JK6GMP6MNTVQM4KZANCNFSM5ZQRUWEQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

The font you are using does not have nerd font glyphs patched in properly and is instead having some other glyph at the location where we are expecting to have a cpp logo glyph. Ideally if you have a properly patched font, this would not happen.

I have it working now by selecting the font in the terminal app preferences.
Don't know why only some or any icons were present when it was set to
system font. Before I installed nerd fonts or font awesome there were no
icons, when set to system font.
On Wed, 22 Jun 2022, 20.43 Abin Simon, @.***> wrote:

The font you are using does not have nerd font glyphs patched in properly
and is instead having some other glyph at the location where we are
expecting to have a cpp logo glyph. Ideally if you have a properly patched
font, this would not happen.
—
Reply to this email directly, view it on GitHub
https://github.com/Peltoche/lsd/issues/678#issuecomment-1163481122, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVWEIBLVWD4G34MKLITODTVQNNEZANCNFSM5ZQRUWEQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Stop passing Strings/StringIO objects to the run function of IRVElection.

This is not a huge deal, but I think passing StringIO objects to the run __init__  function was a mistake. We should be passing a struct of some kind. This would separate the parsing the string from the IRV algorithm.

run in IRVElection currently takes no arguments; I presume you mean during init of the object?
I might also favor keeping it this way, for future maintenance as easy as possible.  Right now, the actual computing of the results (i.e. IRVElection'' in irv.py) can function independently of the Wildcat Connection parser.  If in the future WC changes/people switch to something else, someone can write a new parser to the ballot format without needing to touch or (beyond minimally) understand irv.py.  My worry, then, is that moving the parser will remove this ability to use IRVElection`` as a blackbox.
Of course, it likely depends on the exact implementation, and maybe things like website integration make this a nonissue.

I do mean init, thank you.
I was thinking we have a separate object that represents a parsed election, which would be passed to IRVElection. The production pipeline would then look something like:
parser = WildcatConenctionCSV()
parsed_election: ParsedElection = WildcatConenctionCSV(input_csv)
IRVElection(parsed_election).run()

And the testing pipeline would be
parser = TestingParser()
parsed_election: ParsedElection = TestingParser(test_csv)
IRVElection(parsed_election).run()

Passing around strings is also just generally a bad idea for readability and understanding code, because strings contain no data structure to easily get information. When we first made this, I was under the impression that we would need to be taking files from the PHP Slivka Server, which may or may not have a wacky way to passing information to a python program (files on disk), however, in my recent iterations of the web app, I have found that to be not the case.
This also should be last on our list of priorities lmao.

After a year and a half, this has been resolved by @mnorman23 in #20. Thanks Norman!


Bug Report(?): [file_contents,mods,introMod/]

If you are playing a downloaded version of the game, what operating system are you using?
Windows (x86), Windows (x86_64), Linux, or macOS? Specify below.
Windows x86_64
What version of the game are you using? Look in the bottom left corner of the main menu.
KE 1.8 (Latest Stable)
Have you identified any steps to reproduce the bug? If so, please describe them below. Use images if possible.
It seems very random. Some songs I add work just fine, while others appear with this bug. (I know it doesn't happen because of a song not having vocals. Some songs with just insts work while others don't).
Please describe your issue. Provide extensive detail and images if possible.
Whenever I try to play some songs I added in Freeplay, this error shows up.

Pressing OK just closes the game. I don't really think there is much else to explain about it. Looking in the source to find some sort of trace message didn't get me anywhere. Any help for this bug would be greatly appreciated. Thanks!

Hey I am having this too. Can you please tell me what version are these charts made in (1.8 or an older version)? If the reason is what I am thinking we may be able to solve this ourselves.

Having this issue too, the chart I ported in was from 1.7.

Having this issue too, the chart I ported in was from 1.7.

My problem was solved by going into the chart via debug menu and resaving it. I assume this changes the format slightly so 1.8 can handle it. Hope you guys work this out!


new(tests): add tests for contains/icontains

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area tests

Does this PR require a change in the driver versions?
No

What this PR does / why we need it:
A file that covers another operator.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [LucaGuerra]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.18.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,LucaGuerra]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 0976daee99b393da2b96b85b22e7e5a6ec8b3689

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,LucaGuerra]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Added Gen AI Based Modern Quiz App

Aim:
To develop a quiz application powered by Generative AI to evaluate and assess users' skills dynamically. The AI will generate personalized quiz questions and assess responses in real-time, providing a more interactive and adaptive learning experience.
Summary:
This project aims to create a Generative AI-based quiz application that intelligently generates quiz questions based on user input or selected topics. Leveraging Gemini LLM, the app will evaluate the users' answers, offering tailored feedback and suggestions for improvement. The application will be built using Next.js for a fast, scalable, and responsive user interface, ensuring a seamless and interactive experience. This platform can be utilized for skill assessment, education, or training purposes, enabling users to enhance their knowledge dynamically.
Related issue:
#593
Screenshots

mention the issue no

mention the issue no

I've already mentioned bro.. It's 593


[Bug] When the authentication fails, the stacktrace window overlays the buttons

When the authentication fails and you open the stacktrace window (If the error is large), the stacktrace window overlays the "Copy to clipboard" and "Ok" buttons.

If you close the stacktrace window again, the buttons are displyed again.
Not really a big thing, but this could be a bit nicer, I guess :D

Thanks for reporting this. I fixed the layout by adding a tab control. Dealing min and max height of the stack traces turned out to be a little bit clunky. Let me know that you think about the solution.

Do you have a pre-release or should I check it from the latest source?


How do you handle when you delete storyboards?

I deleted storyboard(s) from my project and Storyboards.swift is causing the build to fail. How do you fix the issue where you don't go manually and comment-out Storyboards.Board from all of the code?

Build failed? Why do not provide the errors...
Did you regenerate the file using after deleting your storyboard?

sorry for the late response, got side tracked doing something else. anyways, here is the sample error. I'm using the build script to generate the storyboards.swift but since I deleted a storyboard it is throwing an error above. If I delete the storyboards.swift I would end-up commenting all of the references in my project for Storyboards.xxxx which is kinda cumbersome.

If you regenerate the Storyboards.swift file, you have BadgeViewController not in the same compilation target or not in your project but in your file system
You can specify the folder to look for storyboards files

Got it by manually regenerating the file from the terminal. I initially thought the build script can handle such task. Thanks by the way!


Secure access by default (TLS) vs using a certificate

Hello
I am currently using XRDP server and by default on xrdp.ini it's set to
crypt_level=high
ssl_protocols=TLSv1.2, TLSv1.3
#tls_ciphers=HIGH
Settingd are mainly by default.
No certificate is specified. However, is still secure to access a server setup like this? can a packet sniffer capture the traffic being sent? Is the information being transferred encrypted or decrypted?
Thanks

There would be two points. xrdp supports two types of encryption. One is classic RDP  encryption that is not very secure now (opportunistic encryption). The other one is TLS that is introduced to replace classic RDP encryption.
You seem to be mixing and misunderstanding these two encryptions.
This is a configuration for classic RDP encryption. This doesn't affect with TLS at all. You can forget this if you want to use TLS only.

low: traffic only client -> server will be encrypted with 40bit RC4
medium: 40bit RC4 for both directions
high: 128bit RC4 for both directions

 crypt_level=high

These are configurations for TLS.
ssl_protocols=TLSv1.2, TLSv1.3
#tls_ciphers=HIGH

You also must prepare a TLS certificate and private key or you cannot use TLS. Specify the place with the following parameters.
certificate=
key_file=

One more important point is security_layer. This controls which classic RDP encryption or TLS to use. negotiate is the default value.

negotiate: negotiate encryption with the client (whichever available TLS or RDP)
rdp: use only classic RDP connections (not very secure)
tls: use only TLS, xrdp doesn't accept connections if misconfigured a certificate

security_layer=negotiate

So if you haven't configured certificates, classic RDP encryption is used instead of TLS because security_layer should be set to negotiate by default.
To ensure TLS, set security_layer=tls.


In :def handle storing to vim9 autoload export.

Fix #14606.
Problem occurs when import autoload ./.../autoload/.... The autoload in the specified path causes the use of an autoload_prefix which combines with the import autoload to create trouble.
In generate_store_var() case dest_script use ISN_STOREEXPORT, when needed, instead of ISN_STORES. When executing ISN_STOREEXPORT, check for autoload_prefix.
Reviewers - there may be better ways to handle this problem.

@yegappan I hope you look at this. The notes in the related issue may be of interest.

@yegappan After saying that val would have been looked up as l, I realized I didn't explore this. Sure enough, both before and after this PR, things are screwy. And it looks like the s: prefix in the :def might be optional, something else to check.
I put this PR into draft and will take a look tomorrow. I'm fuzzy on what should happen. ???
The following has code to changes xxfoo, but foo gets changed
let s:foo = 'init'
let s:xxfoo = 'init'
def F()
    xxfoo = 'NEW VALUE'
enddef
call F()
echo ' '
echo printf('after F(): s:foo: %s, s:xxfoo: %s', s:foo, s:xxfoo)

outputs
after F(): s:foo: NEW VALUE, s:xxfoo: init

Updated PR to distinguish lhs->lhs_dest between dest_script and dest_script_v9 which simplifies the compile/execute logic.
Add test for :def in legacy script accesses the correct script variables.

@yegappan Here's the "final" (we'll see) changes for review
https://github.com/vim/vim/compare/a8a9a9e71f06026b66962695f6e7fbcb85ba9eb5..bf526e0039daccdbeebed969e68116eb93569824

thanks all!


add rule E3017 to check when a prop is required based on a value

Issue #, if available:
#1744
Description of changes:

Add rule E3017 to validate when a property is required based on another property having a certain value

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Think investigating similar rules/requests will be helpful:
CloudWatch, DynamoDB, EC2, Lambda, RDS, RDS, Route53, Route53, Route53, StepFunctions
Something to note is that quite a few of them are the opposite: a certain property cannot be specified based on the value of another property

For tracking after this PR is merged I will evaluate if these rules can be deleted and moved to this rule.
https://github.com/aws-cloudformation/cfn-python-lint/blob/master/src/cfnlint/rules/resources/rds/AuroraScalingConfiguration.py
https://github.com/aws-cloudformation/cfn-python-lint/blob/master/src/cfnlint/rules/resources/rds/AuroraDBInstanceProperties.py
https://github.com/aws-cloudformation/cfn-python-lint/blob/master/src/cfnlint/rules/resources/route53/HealthCheck.py

@PatMyron I'm going to look at another rule for doing the opposite check because I'm finding use cases where it could be valuable.


Problem with collapsed state of fixedAccordion

Investigate issue found by ManageIQ and referenced in this PR https://github.com/ManageIQ/manageiq-ui-classic/pull/1984
Looks like angular-bootstrap isn’t adding the collapsed class to the  inside .panel-title

Fixed by #634


Substream init fails when a contract creation block cannot be found

Error: getting Ethereum Mainnet contract creating block: empty result from response {0 No transactions found []}
The contract was created using a create2factory

Looks like the contract was deployed, but nobody executed any functions on it yet.

the contract is a proxy implementation that is called via fallback.


Improvement idea: add an additional button to column-review.html

There should be two actions at the bottom of the page, one for save and continue editing and one for save and close.

This is running into a problem with the django framework (I think), where it won't allow a Save unless all fields are completed.  I'm going to pause this until we have a solution for it.

This is on hold until the team has agreed on the design, see issue #397.

@signekb is doing this currently.

This is duplicate of #397


feat(module:all): Add RTL support to all ng-zorro-antd

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #1762
What is the new behavior?
All components will be displayed correctly in RTL direction.
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

Hi @saeedrahimi
thanks for your pull request!
most components have been refactored now, it is safe to rebase and finish RTL support now  :)

Hi @saeedrahimi
how is everything going, we have released 9.0 version and make RTL to our 9.1 milestone
if you need any help, please let us know, thanks a lot for your contributions!

Hi @vthinkxie, @saeedrahimi
I can help you to complete RTL support from two to four weeks, if there isn't any problem tell me to start the process.
thanks

@saeedrahimi @HDaghash @hdm91
Thank for your contribution, it looks sufficiently great now. We would like to mention your contribution in the release log, so could you please tell us your twitter IDs?

@saeedrahimi @HDaghash @hdm91
Thank for your contribution, it looks sufficiently great now. We would like to mention your contribution in the release log, so could you please tell us your twitter IDs?

I'm glad I could help.
twitter ID: saeedr_ir

@saeedrahimi @HDaghash @hdm91
Thank for your contribution, it looks sufficiently great now. We would like to mention your contribution in the release log, so could you please tell us your twitter IDs?

Thank you guys for making this happen
Glad to hear
Twitter :@HDagash

@saeedrahimi @HDaghash @hdm91
Thank for your contribution, it looks sufficiently great now. We would like to mention your contribution in the release log, so could you please tell us your twitter IDs?

Someone missed to mention.
I want to thank @hmdnikoo for all his contributions. he kindly changed most of the components to fix a bug on RTL change.

@saeedrahimi @HDaghash @hdm91
Thank for your contribution, it looks sufficiently great now. We would like to mention your contribution in the release log, so could you please tell us your twitter IDs?

Someone missed to mention.
I want to thank @hmdnikoo for all his contributions. he kindly changed most of the components to fix a bug on RTL change.

Certainly. We value all your contribution very much. Please also provide his Twitter id. By the way, do you have @hdm91 's Twitter id?

@saeedrahimi @HDaghash @hdm91
Thank for your contribution, it looks sufficiently great now. We would like to mention your contribution in the release log, so could you please tell us your twitter IDs?

Someone missed to mention.
I want to thank @hmdnikoo for all his contributions. he kindly changed most of the components to fix a bug on RTL change.

Certainly. We value all your contribution very much. Please also provide his Twitter id. By the way, do you have @hdm91 's Twitter id?

Hi guys , thanks for your consideration and helping to make RTL feature on great ng zorro , I wish I would have enough time to help you in future
my twitter ID :@hdm9170


Feature - History display

Is your feature request related to a problem? Please describe.
I would like to be able to see which items in a watchlist have already been watched.
Describe the solution you'd like
Users can view history (all items hidden from watchlists due to watched status)

Done


Local search as in minimap

If it is possible to do easily at all, would be great if local search will open like minimap on the right. With synced highlighting in the actual buffer.

I guess you want the search to open in a micro buffer pane (as a split) instead right?
I do plan to have a go at it but I will see how it goes.


Functions that lack test in QuTiP.

These are some modules in QuTiP for which there are no tests written yet. I will try to write them since they are very useful to define what an operation should do. This facilitates the development of new data types.

[ ] Pow (PR ready for review)
[ ] Expm
[ ] expect
[ ] expect_super
[ ] norm
[ ] Eigen
[ ] reshape
[ ] ptrace

I don't know all of them, but off the top of my head, the eigensolvers, ptrace and expect definitely have tests.

Hey Jake, thanks for pointing out that those test exist as at first I did not look beyond the tests in tests/core/data/! Indeed there is a test_expect.py (and similar for eigensolver and ptrace) in QuTiP but these seem to be testing Qobj level functionality. I was looking for tests similar to those in test_mathematics that I can inherit from and easily employ to test my own specialisations. I think that extending test_mathematics.py to provide tests for the above specialisations would be very useful. This allows QuTiP to define what the specialisations should do in a very concrete way and it helps future developers to develop new data layers.
From the discussion at yesterdays GSoC meeting, it seems that @hodgestar, @Ericgig and @quantshah agree although they suggested to extend test_mathematics in a different file for organization purposes as test_mathematics is already quite big.


Release 0.23.0

The release candidate looks good on tierstaging and tier4. Lets just merge the final release commit and then start deploying!
cc: @ElvinEfendi

Merging now. All checks are green. Watch me get pinged anyway :-).

^ There it is


Includes-what-you-use not works in VS 2022

#include <iostream>
#include <vector>

int main()
{
	std::cout << "Hello World!\n";
}

Even for this simple file, Includes-what-you-use still cannot delete the vector header.
I use the default setting, right-click on the cpp file and click Run Include-What-You-Use.
The result is it only formats my code, the header is still there.

What does the output window say? I launched the IWYU on the clear VS2022 without any predefined parameters, it works fine.

Hi Agrael1. Thanks for your reply. This bug is hard to hit, I opened another ConsoleApplication, this time Include-What-You-Use can delete the  header. And when Include-What-You-Use fails, there is nothing in the output window.

Another thing is, there is only Include-What-You-Use in my right-click menu. I don't have Include Formatter and Trial and Error Include Removal. Is this a bug? Is there a way to debug this extension? So, maybe I can fix it.

Another thing is, there is only Include-What-You-Use in my right-click menu. I don't have Include Formatter and Trial and Error Include Removal. Is this a bug? Is there a way to debug this extension? So, maybe I can fix it.

the format and other options are in code window, you need to right click anywhere on the code. For the format you need to select headers you wish to format, and right click in the selection


Add gunzip to fetch with assorted tests

Add gunzip to fetch with assorted tests (not a gzip file, default behaviour of fetch, correct size of output for a valid file)

Close in favor of #6176, that PR has the right test


Materialize Generator

I'm a contributor to the new http://materializecss.com/ Material Design framework. I'm also helping a University Ruby on Rails class that wants to use the framework. Is there anyway I could help or advise an install for Materialize? Having simple form run for this framework would be really helpful for us.

Thank you for the suggestion/ We don't plan to add any new generator to Simple Form. In fact I already thought many times to remove the currently included generators to new gems.

Do you have any recommendations on how we can use simple form but still customize it some way?

All Simple Form configurations for frameworks are just wrappers. I believe you will have to create wrappers for the framework components in your Simple Form initializer.

Cheers! So, will it be a good practice to make a separate gem with css_framework-specific generator and load it as dependency ot smth like that?


Could you please provide the Canny Control dateset?

Hi
You can just run the canny algorithm in the opencv package on the satellite images to create the dataset.
Thanks

With default parameters?

All default parameters except the 1st threshold set to 100 and the 2nd threshold set to 200.

Probably worth throwing a script in to do that so people can replicate results.

Could you please provide the Canny dateset?The effect I generated myself is not good.Thanks!

Below I'm sharing a script to load the Canny dataset. After reading the satellite image, I run the canny detector algorithm defined in the ControlNet repo:
import json
import cv2
import numpy as np
from ..ControlNet.annotator.canny import CannyDetector
from ..ControlNet.annotator.util import HWC3
from torch.utils.data import Dataset

class Dataset(Dataset):
    def __init__(self, prompt_path, location_embeds_path):
        self.data = json.load(open(prompt_path, "rt"))
        self.apply_canny = CannyDetector()

    def __len__(self):
        return len(self.data)

    def __getitem__(self, idx):
        item = self.data[idx]

        target_filename = item["target"]
        prompt = item["prompt"]

        target = cv2.imread("../" + target_filename)

        # Do not forget that OpenCV read images in BGR order.
        target = cv2.cvtColor(target, cv2.COLOR_BGR2RGB)

        # Apply Canny Edge Algorithm
        source = self.apply_canny(target, 100, 200)
        source = HWC3(source)

        # Normalize source images to [0, 1].
        source = source.astype(np.float32) / 255.0

        # Normalize target images to [-1, 1].
        target = (target.astype(np.float32) / 127.5) - 1.0

        return dict(
            jpg=target, txt=prompt, hint=source
        )

Below I'm sharing a script to load the Canny dataset. After reading the satellite image, I run the canny detector algorithm defined in the ControlNet repo (which uses opencv):
import json
import cv2
import numpy as np
from ..ControlNet.annotator.canny import CannyDetector
from ..ControlNet.annotator.util import HWC3
from torch.utils.data import Dataset

class Dataset(Dataset):
    def __init__(self, prompt_path):
        self.data = json.load(open(prompt_path, "rt"))
        self.apply_canny = CannyDetector()

    def __len__(self):
        return len(self.data)

    def __getitem__(self, idx):
        item = self.data[idx]

        target_filename = item["target"]
        prompt = item["prompt"]

        target = cv2.imread("../" + target_filename)

        # Do not forget that OpenCV read images in BGR order.
        target = cv2.cvtColor(target, cv2.COLOR_BGR2RGB)

        # Apply Canny Edge Algorithm
        source = self.apply_canny(target, 100, 200)
        source = HWC3(source)

        # Normalize source images to [0, 1].
        source = source.astype(np.float32) / 255.0

        # Normalize target images to [-1, 1].
        target = (target.astype(np.float32) / 127.5) - 1.0

        return dict(
            jpg=target, txt=prompt, hint=source
        )

Thanks for the code, I will try it out and thanks again for this great work.


Synchronize edx.org course exports to S3

User Story
As an engineer working on MIT Open I would like to be able to integrate course contents from edx.org along with our other Open edX installations.
Description/Context
IRX is maintaining a Google Cloud Storage bucket with the .tar.gz course bundles for courses being hosted on edx.org. All of our other Open edX installations have pipelines configured to load course bundles into date-stamped paths in S3 on a nightly basis. In order to reduce the level of development effort we would like to replicate that interface by synchronizing the GCS course exports to S3 using the same path formats. The pipeline code that manages that export is located at https://github.com/mitodl/ol-data-platform/blob/main/src/ol_orchestrate/ops/open_edx.py#L482-L576
The bucket in GCS that has the course bundles is [simeon-mitx-course-tarballs](gs://simeon-mitx-course-tarballs)
Acceptance Criteria

[ ] There is a pipeline that replicates the course exports from edx.org from GCS to S3
[ ] The pipeline is triggered by a sensor that monitors for changes/new contents in the source GCS bucket

The pipeline code that manages the export is located here - https://github.com/mitodl/ol-data-platform/blob/main/src/ol_orchestrate/ops/open_edx.py#L630-L704

Looked into using our existing bigquery resource and mainly just changing the last line in yielding a storage client, however it appears that we have to create a separate resource (which was mainly copy/paste). Ran into a permissions issues using the existing creds to read from GCS and am working with Jon to see what permissions need to be tweaked on their end to resolve it.

Chatted with Jon and it seems like this is not something he can do himself and is contacting IST to create a custom role for this to work.

Checked in with Jon on Monday and he still hadn't heard anything. Will ping him again today to see if there are any updates.

We now have access to the GCS bucket.

Have most of the pipeline code written except for connecting the output from one op (downloading the files from GCS) to the input into the other op (uploading the assets to S3) which I will be working on today and then testing out on QA.

Ran into some issues with the dagster container builds after adding the gcs pipeline. Tobias ended up making some changes to how we're building containers for pipeline use in dagster which ended up being a blocker, and now we're back to testing the pipeline on QA.

Resolved some bugs in the pipeline code the last being a subfolder needed to be created before downloading the GCS bucket files. After resolving that, trying now to determine what S3 bucket to upload the data to. Appears that we have a bunch of different buckets specified, however not all of them actually have been created. Trying to determine why those buckets haven't been created and whether we want to stick to that way of doing things or whether we want to upload those files to an existing bucket within a subfolder.

Sync on QA went through and course tarballs have been uploaded to QA bucket. Working on the following things now:

Verify that the sensor is working whereby the dagster pipeline is triggered whenever a file is added to the GCS bucket
Check with Matt on what permissions the open app requires for that bucket and add those in pulumi

Still struggling to verify whether the sensor is functional. It doesn't show up in dagster and running dagit throws an exception. The sensor is meant to make a call to a job however we have things setup a bit differently and aren't using the job decorator. Tried a few different ways using to_job based on Dagster docs to resolve the issue, however haven't had much luck so far. Continuing to look through the docs and try a few different things to get the sensor working.

Chatted with Tobias about it and he mentioned the newly introduced dagster definitions feature that helped resolve the issue. Change was merged and will be verified on QA before deploying it to production on Monday.


Create 3.0.0-dev.2

This PR contains:

Build changelog from news files

However, trying to upload it to TestPyPI gives me the following error:
$ twine-3.6 --version
twine version 1.13.0 (pkginfo: 1.4.2, requests: 2.22.0, setuptools: 40.5.0,
requests-toolbelt: 0.9.1, tqdm: 4.31.1)

$ tox -e prepare-dist
[...]
prepare-dist run-test: commands[1] | twine check 'dist/*'
Checking dist/semver-3.0.0.dev2-py3-none-any.whl: PASSED
Checking dist/semver-3.0.0.dev2.tar.gz: PASSED

$ twine-3.6 upload --repository testpypi dist/*
Uploading distributions to https://test.pypi.org/legacy/
Uploading semver-3.0.0.dev2-py3-none-any.whl
100%|█████████████████████████████████████████████| 24.2k/24.2k [00:01<00:00, 14.3kB/s]
NOTE: Try --verbose to see response content.
HTTPError: 400 Client Error: 'attr: semver.__about__.__author_email__' is an invalid value for
Author-email.
Error: Use a valid email address See https://packaging.python.org/specifications/core-metadata
for more information. for url: https://test.pypi.org/legacy/

I also tried it with latest twine (version 3.2.0) without success.
@tlaferriere Thomas, do you know how this can be solved? Seems twine or (Test)PyPI doesn't understand our new setup.cfg. :cry:  Seems this is not really supported... :thinking:

Looks like it is using an old version of setuptools. I think I saw version >=43.x somewhere in the docs about the setup.cfg only support. I don't know if you could update it to something more recent?

Nope. I checked that already and added a little change in tox.ini to make sure I have the latest setuptools:
diff --git i/tox.ini w/tox.ini
index 73fbfc5..d6c17df 100644
--- i/tox.ini
+++ w/tox.ini
@@ -95,7 +95,9 @@ basepython = python3
 deps =
     wheel
     twine
+    setuptools >= 43
 commands =
+    # just to make sure we see what's installed:
+    pip list
     python3 setup.py sdist bdist_wheel
     twine check dist/*

when I run it
$ rm -rf build/ dist/
$ tox -e prepare-dist
[...]
prepare-dist run-test: commands[0] | pip list
[...]
semver             3.0.0.dev2
setuptools         50.3.2
[...]
prepare-dist run-test: commands[2] | twine check 'dist/*'
Checking dist/semver-3.0.0.dev2-py3-none-any.whl: PASSED
Checking dist/semver-3.0.0.dev2.tar.gz: PASSED

As you can see, I have even setuptools version >50. When I extract the METADATA from the wheel file, I get:
$ cat /tmp/semver3.0.0-dev.2/semver-3.0.0.dev2.dist-info/METADATA
[...]
Summary: attr: semver.__about__.__description__
Home-page: https://github.com/python-semver/python-semver
Author: attr: semver.__about__.__author__
Author-email: attr: semver.__about__.__author_email__
Maintainer: attr: semver.__about__.__maintainer__
Maintainer-email: attr: semver.__about__.__maintainer_email__
[...]

Either this is a bug in setuptools or we did something wrong. :smile:

Interesting... after a closer look, the Version in the above METAFILE has replaced the version from semver.__about__.__version__ but not the others variables. :thinking:

I think I've found the culprit. According to the table in Metadata section of the setuptools documentation:

Key
Aliases
Type
Minimum Version
Notes

name

str

version

attr:, file:, str
39.2.0

url
home-page
str

download_url
download-url
str

project_urls

dict
38.3.0

author

str

author_email
author-email
str

maintainer

str

maintainer_email
maintainer-email
str

That means, the attr: key is only allowed in version (where it works), but not for author and the other keys. Only strings are allowed. :disappointed:
Seems I need to replace them with the real values.

Success! :tada: After I've updated the fields above, I could upload the files to TestPyPI: https://test.pypi.org/project/semver/3.0.0.dev2/

Oh no, such a specific limitation. I wish we could use attr in more places. I think I'll go read the issues over in the setuptools repo to see if anyone has suggested this already.

That would be great. 👍I think this is such a nice and elegant feature.
For the moment, it's hard-coded, but if you find a better solution we can always publish a next dev release. 😉


Using settings to reach very low bitrate

Hi,
I am trying to perform encoding with very low bitrates (~50kbps) on a 854x480 video and it seems the encoder setting are not well taken into account since the generated video is a 170kbps one.
I tried to modify the QP parameters, wouldn't change anything on the bitrate though.
Are there any settings I could use to achieve very low bitrate encoding ??

what is the current parameter you are using?

no more info. if have the same issue again, please submit a new one.


🪸 My schema requests

Can be implemented after: https://github.com/aiven/klaw/issues/350

Layout

Notes about layout

we don't do the total numbers
the filter / search elements are in one line, same as in the other "My / My team requests"
we don't do "edit"
we add a select AND a column for request type (same as all other tables!)
"Type" may be "Request type" like in the approvals table (added a comment in Figma to clarify)

if it should be "Type", this needs to be updated in Approvals table, too, to be consistent

"Date requested" should be "Requested on" like in Approvals (see slack discussion)

Questions

should we add a "View details" link like we did in approvals table (table rows are not clickable)
is the workflow matching the approvals table -> opening modal -> click delete -> delete confirmation modals opens?

Todos
I'll write down the todos as complete as I can, please update them to make one todo a good PR scope that fits your workflow! Also the Todos may change dependent on the answer to the above questions.

[ ] add API definition and types for /getSchemaRequests
[ ] add API definition and types for /deleteSchemaRequests
[ ] add table with schema requests
[ ] add pagination with functionality
[ ] add search functionality
[ ] add filter functionality (Environment, Type, Status)
-> ⚠ Environments for schemas come from getSchemaRegistryEnvironments
[ ] add Toggle for "My requests" (may need refinement with design)
[ ] add details view flow
[ ] add delete functionality from details view and from quick action button
[ ] usability and accessibility test and optimisation if needed


refactor httputil error stream handling

Coverage decreased (-0.1%) to 50.334% when pulling aec394a6c0f8dfdff5e28cf3f4d0fada3a354131 on nobodyiam:refactor-httputil-error-handling-merge into f9cb281d457d35d294d97343980b3c29bb5f8f55 on ctripcorp:master.

Codecov Report

Merging #883 into master will decrease coverage by 0.11%.
The diff coverage is 78.57%.

@@             Coverage Diff              @@
##             master     #883      +/-   ##
============================================
- Coverage     46.92%   46.81%   -0.12%
+ Complexity     1544     1542       -2
============================================
  Files           352      352
  Lines          9715     9717       +2
  Branches        962      963       +1
============================================
- Hits           4559     4549      -10
- Misses         4813     4826      +13
+ Partials        343      342       -1

Impacted Files
Coverage Δ
Complexity Δ

...com/ctrip/framework/apollo/util/http/HttpUtil.java
78.43% <78.57%> (-7.29%)
7 <0> (ø)

...lo/exceptions/ApolloConfigStatusCodeException.java
0% <0%> (-100%)
0% <0%> (-2%)

...ework/apollo/internals/RemoteConfigRepository.java
83.97% <0%> (-3.21%)
21% <0%> (ø)

...ervice/service/ReleaseMessageServiceWithCache.java
84.52% <0%> (-1.2%)
23% <0%> (-1%)

.../apollo/internals/RemoteConfigLongPollService.java
77.91% <0%> (+1.22%)
28% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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


Show boostdep data in library/release detail and report.

fixes #1336
If a libraryversion does not have any known dependencies in the version in question or the prior version, stats are not shown.  There is the case where if a library goes from 0 to n dependencies or vice-versa, that stat will not be shown.  If this is a necessary feature, more work will need to be done to facilitate it.  We are only keeping a many-to-many relationship of LibraryVersions to Libraries so we cannot distinguish between if there are no dependencies or if we were not able to fetch them through the boostdep github action.

library detail page

I chose green for removed dependencies and red for added.
The green is a bit light, I'll leave it for someone else to choose the exact color you want, if any.

release page

release report

I think I would have made the opposite color choice (green for added, red for removed). I'll let others chime in on specific shades.

I understand why you did the color choices that way, that's fine, it expresses what's needed. I think it would also be helpful to put a "(*New)" in smaller text next to the word "Dependencies" and put a preceding asterisk next to each new dependency for color blind users and for more clarity.

Can we have the added/removed fonts smaller? like an h4 instead of h3


lldb running on an ios device is using the _dyld_get_all_image_infos()

SPI call to to find its own shared cache's UUID.  On newer sytems we
need to use the a new SPI which will return the UUID directly.
rdar://problem/36625871
git-svn-id: https://llvm.org/svn/llvm-project/lldb/trunk@324437 91177308-0d34-0410-b5e6-96231b3b80d8
(cherry picked from commit 63f24576c2deffa02857dd7d2efdcf21df958a66)

@swift-ci please test

Hmm, this has no test. I don't feel comfortable merging this as it's changing a fairly subtle detail and chances are this will break in the future.

Reading the commit, this API should exist on 10.12 and beyond, which is, among others, the version we currently use. It should be possible to test this change on High Sierra, e.g.

This function is only built when lldb is built on iOS - it's guarded by #ifdef's.  The only way to test it would be to intercept a call to dlsym() to interpose our own copy of their SPI function, when built on iOS.

Then we should consider adding a unittest for GetSharedCacheUUID and run that on configuration where this is supported. We seriously can't keep leaking changes untested.

Our testing for iOS is concentrated on our customer use scenario -- lldb running on the mac, debugserver and the inferior process running on the device.  Looking into testing lldb on the device is something we should do in the future, but it is a much lower priority workflow.

We really should have regression testing for anything that changes functionality. Tomorrow somebody can show up, remove this code, and we're going to ship with a broken functionality in the wild. I apologize if this sounds pedantic, but not testing this feature right now is just pushing the problem away a little bit, as any other similar feature will hit the same "we can't test it". In other words, I'd rather try to solve this problem now than postpone it.

It's a good discussion to have.  Testing new areas of lldb, new corners and environments that we have not been able to test before, is for the good of the project.  But that's not where we are today.  And while this does impose the requirement that people work with some care, and there is the chance of regressions, lldb has worked pretty well for the past decade.  We can do better, we will do better, but lldb was designed with one approach to testing in mind, and it will take a lot of time and design consideration to build on that.

I think we agree on our shared goals here. But I think testing should be one of our highest, if not the highest priority right now. As I understand it we have a few weeks to land this change. I think we should take the time to figure out how to test this now, to put ourselves in a better position going forward.

I think having a discussion about how to test the on-device lldb is a good discussion to have.  I don't think changing the requirements for lldb's testing to say that a patch needs to have infrastructure in place that does not exist is appropriate.

I agree that it's a good time to hash out on-device testing for lldb. So long as this landing this isn't an emergency, I'd ask that we flesh that out and come up with a test plan before checking in more uncovered code. I realize this is a new requirement, but still think that it's appropriate, as most of the team has been working on lldb test infrastructure and the work is paying off.

Let's have the discussion about what it would take to test this then. We can then make an informed decision. As I said earlier, I'd really like every functional change to be covered by a test case.

On-device lldb testing is far, far down the list of testing priorities that we already have.  There's no way I can reasonably allocate any time working on testing on-device lldb ahead of the tasks we already have planned.

Per our offline conversation, we'll work on getting build/test infrastructure for this going in parallel. Thanks Jason!


GEODE-7345: break dependency on SocketCreator

GEODE-7345
In preparation to put TcpServer and TcpClient into their own module, this PR breaks dependencies from those classes, to classes in geode-core.
In the nascent geode-tcp-server module, replace usages of the (broad) SocketCreator class (from geode-core) with usages of a new (narrow) TcpSocketCreator interface.
This means injecting (a reference to an object implementing that interface) everywhere we were formerly creating an instance of SocketCreator. You'll see many examples of TcpClient construction taking a TcpSocket creator ref now.
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[x] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

This branch had an approved PR a week or so ago. But I've fiddled with it a lot since then, to get the stress test passing, so thought it was appropriate to ask for a fresh sign-off.


Create derryfield.txt

URL for our Computer Science courses is https://www.derryfield.org/courses/upper-school/us-steam-x/

@MrYoungDS First stage of verification passed, official domain for The Derryfield School is derryfield.org as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

Thanks!
On Mon, Sep 16, 2024 at 6:24 AM Philip Torchinsky @.***>
wrote:

@MrYoungDS https://github.com/MrYoungDS Thank you for your patience
while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes
to propagate through our system.
—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/23413#issuecomment-2352531932,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2FO6AXHF6CGRYBAQNCBHLTZW2WXJAVCNFSM6AAAAABN7CG2SOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJSGUZTCOJTGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Upper School Mathematics Department, STEAM-X Department
The Derryfield School, Manchester, NH


Improve decrypt with PGPSecretKeyRingCollection

But I've found another issue. I propose to discuss the following code
fun decryptWithResult(
    srcInputStream: InputStream,
    pgpSecretKeyRingCollection: PGPSecretKeyRingCollection,
    protector: SecretKeyRingProtector
  ): DecryptionResult {
    srcInputStream.use { srcStream ->
      val destOutputStream = ByteArrayOutputStream()
      destOutputStream.use { outStream ->
        try {
          val decryptionStream = PGPainless.decryptAndOrVerify()
            .onInputStream(srcStream)
            .withOptions(
              ConsumerOptions()
                .addDecryptionKeys(pgpSecretKeyRingCollection, protector)
            )

          decryptionStream.use { it.copyTo(outStream) }
          return DecryptionResult.withDecrypted(
            content = destOutputStream,
            filename = decryptionStream.result.fileName
          )
        } catch (e: Exception) {
          return DecryptionResult.withError(
            processDecryptionException(e)
          )
        }
      }
    }
  }

where

srcInputStream: InputStream - our -----BEGIN PGP MESSAGE-----...-----END PGP MESSAGE-----
pgpSecretKeyRingCollection: PGPSecretKeyRingCollection - contains private keys that can be used to decrypt a message from srcInputStream(let's imagine it contains 2 keys: A and B)
protector: SecretKeyRingProtector - handles fetching passphrase for pgpSecretKeyRingCollection(let's imagine it contains just one passphrase for B)

expected results
If we try to decrypt a message that was encrypted for 2 keys(A and B) and just pass only one passphrase for B we should be able to decrypt that message. Because for at least one suitable key we have a passphrase
actual results
Due to a logic one by one PGPainless takes the first key from the ring collection(A in our case), asks protector to provide a passphrase for A, and throws an exception. But we have key B and a passphrase for this key.
Originally posted by @DenBond7 in https://github.com/pgpainless/pgpainless/issues/183#issuecomment-919771807

I changed the decryption logic, such that PGPainless will

first try to decrypt any password based encrypted session keys
then try any public key encrypted session keys where the decryption key has a passphrase available
then try any public key encrypted sesison keys where the passphrase is missing which causes callbacks to fire

This should also work with wildcard encrypted data.

Thank you! I've checked - your changes fix an issue.

One more question. Do you have a plan to post a new update(0.2.13)?

I'm doing that right now :P

Done.


I can use the client to send TopologyRequestCommand with retries

Is your feature request related to a problem? Please describe.
As you can read in https://github.com/zeebe-io/zeebe-client-csharp/issues/69 we want to send requests with retries.
With https://github.com/zeebe-io/zeebe-client-csharp/pull/172 we implemented the first step to send commands with retries.
Describe the solution you'd like
Here you can find a reference implementation for the CancelWorkflowInstanceCommand.

Is this still needed, I would like to do this if it is?

Hey @askalik absolutely !
What to do:

The related interface needs to implement IFinalCommandWithRetryStep, can be seen in ICancelProcessInstanceCommandStep1
The implementation just needs to delegate to the AsyncWaitStrategy https://github.com/camunda-community-hub/zeebe-client-csharp/blob/master/Client/Impl/Commands/CancelProcessInstanceCommand.cs#L42
You pass it from the ZeebeClient to the related command, can be seen here https://github.com/camunda-community-hub/zeebe-client-csharp/blob/master/Client/ZeebeClient.cs#L149

Thanks :tada:

Question for @Zelldon should I take the others as well that are to implement the same across the platform?
Seems like it wouldn’t be that much additional effort?
I am open to do it as C# is a language I am fairly comfortable with.  Just new-ish to open source contributing.

This is on PR #329


draw.io bug - saving the file acts differently when in a group

This problem only occurs with the vscode draw.io extension. The vscode draw.io extension uses the draw.io export functionality when saving and is effected by this bug: https://github.com/jgraph/drawio/issues/3516

Upstream drawio bug is fixed. It may still take a while before the vscode extension updates to the fixed version though.

It's been a year, is this still an issue with the vscode extension?

It's been a year, is this still an issue with the vscode extension?

Yeah, still an issue in version 1.6.6 (the latest). It's been over a year since the last release of the vscode extension. https://marketplace.visualstudio.com/items?itemName=hediet.vscode-drawio
The draw.io offline application is generally more stable. It's not quite as convenient as vscode extension, but it only takes a few clicks (every time you launch it) to get the StateSmith plugin working: https://github.com/StateSmith/StateSmith-drawio-plugin/wiki/Use-with-app-or-website


implementations of visualizing xmodel

Hi, lutzroeder:
We really appreciate your work about Netron, which is widely used by us. I tried to optimize the visualization of xmodel. Please have a look at this patch, could these changes be merged into your main branch ?
New Features:

Visualize the subgraphs for the .xmodel files.
Display the attributes of tensors.
Change the color.

Some Examples:

https://www.xilinx.com/bin/public/openDownload?filename=pt_face-quality_80_60_61.68M_1.4.zip
Download the zip file from this link, please test the PointsQuality_int.xmodel, which is an uncompiled xmodel.
https://www.xilinx.com/bin/public/openDownload?filename=face-quality_pt-vck190-r1.4.0.tar.gz
Please test the face-quality_pt.xmodel, which is a compiled xmodel.

The xmodel specific UI changes are not applied or unified to all other supported formats. Maybe this should be broken up into separate changes starting with something simple like data types. For example, the argument and data type change is not following the int32, uint32 notation used for all other formats, sidestepping TensorType which is used for all other formats. It is unclear or not documented how xint differs from regular int.


The readme doesn't mention the type of currency for the donation address

I guess no one will send coins blindly, but I just wanted to point out the information is missing.

@petko10 if it's not stated then it's bitcoin.


better visual cues / consistency to directionality in radial view

The radial view one gets when selecting a node could do a better job of positioning the connected nodes such that the directionality follows the "inbound at top, outbound at bottom" approach we use to lay out the main graphs.
Follow-up to #875, which says

Concepts involving re-arranging the nodes were unsuccessful, as it lead to very unbalanced "sides" of the diagram which lead to difficult to read diagrams.

I suggest an approach where the list of connections is sorted to have inbound first, bidirectional second, and outbound last. We then place needs nodes on n_nodes equidistant positions on the radial circle by iterating through the list and placing the first node at the top, the 2nd node to its left, the third to its right, the fourth to the left of the 2nd node, the fifth to the right of the 3rd node, etc.
Add some special casing for small numbers of nodes, e.g. always create at least 4 positions - top, bottom, left, right - and place nodes in their "best free match" positions.

@fbarl, @miklosp and I thought about this a bit. I propose a radial view with directionality grouping as follows :

I don't know how many rows should be the maximum, but this layout still creates problems when the spacing is too tight.

What do you think?

Looks fine, and makes better use of the space than the existing view. BUT...
It looks like the circle has been divided into four quadrants, top - for inbound, bottom for outbound and left/right for bidirectional. That reserves way too much space for bidirectional links, which are rare. I reckon the segment size needs to be somewhat dynamic, perhaps 0-180 degrees, proportional to the ratios of the number of nodes in each directionality bucket.

this layout still creates problems when the spacing is too tight.

every layout will have some upper bound on the number of nodes in can show.

every layout will have some upper bound on the number of nodes in can show.

Perhaps there's an opportunity do design a more graceful way for nodes to overlap.
What if (in the case where there are 50+ connections) nodes could stack horizontally and show only one label (cycling through each node's name on hover).

Dealing with lots of nodes is (or, rather, should be) a separate issue.

What do you think?

I like the radial layout with directionality grouping as in the screenshot above, although I also agree with @rade that it should be split in 3 quadrants rather than 4.
One problem I see though is that such a layout does consume a bit more space, which we'd have to figure out how to deal with in the context of Weave Cloud (Workload Summary graph).

Perhaps there's an opportunity do design a more graceful way for nodes to overlap.

I just opened #3293 to discuss this separately, but I just wanted to point out that we already have a notion of stacked nodes, which we could potentially use here, although the problem is that some topologies already display all their nodes as stacked, so it would be mixing of use cases :/

it should be split in 3 quadrants rather than 4.

Umm, you cannot divide a circle into 3 quadrants ;)
What I was suggesting is that the four segments are sized in proportion to the number of nodes they need to contain, with the additional constraint that no segment exceeds 180 degrees (or perhaps just shy of that, e.g. 135).

you cannot divide a circle into 3 quadrants

:)
I think it is more clear when data senders are centred on 0°, receivers on 180° and bidirectionals on 90° (we can choose only one side right or left).
Here, the sender and receivers are skewed by -20° and limited to fan out by 90° and the bidirectionals are centred on 90°.

This whole valence thing looks much more complicated and takes too much space. Let's drop it, organise the single orbit with senders on top and receivers on bottom, and focus on improving the grouping of nodes to fix this problem.

If we want to keep things simple, I suggest we stick to the single orbit, with four segments located as follows:

300° - 60° inbound
60° - 120° bidirectional
120° - 240° outbound
240° - 300° bidirectional

This gives us 120° each for inbound, outbound, and bidirectional.

Though skewing looks weird and I'd rather have bidrectionals centered at 90° and 135°.

135°? Do you mean 270°

This gives us 120° each for inbound, outbound, and bidirectional.

I got the impression that there are fewer bidirectionals. If so, we could allocated a smaller portion to bidirectionals?

135°? Do you mean 270°

Oops. Yes.

I got the impression that there are fewer bidirectionals. If so, we could allocated a smaller portion to bidirectionals?

There typically are indeed fewer bidirectionals. But sometimes there aren't. For example, in the Nodes topology usually everything is bidirectional. Hence if we go with fixed segment sizes I would rather stick to 120° each. OTOH, if we go for variable segment sizes, we could adjust the size between 90% and 150%. But that is more work.

The goal should be to show summary in the workload summary view. If it gets too crowded with lots of nodes, then you can have a hover to show how many nodes and then clicking it would bring up a modal (or something)  to show the fuller picture before going to Explore which shows that in context with the rest of the cluster.

The goal should be to show summary in the workload summary view

This is the scope oss repo. There is no 'workload summary'

If it gets too crowded with lots of nodes

Dealing with lots of nodes is covered in #3293.

Oops - I didn't notice scope repo - lost track - never mind.

Time passed, and I'm not a big fan of the radial view anymore. Instead, I propose 2 simple steps to resolve this issue for now :

We do what @rade originally said, though I don't think that always creating at least 4 positions is necessary - we could simply turn groups of 3 to the most appropriate angle.
To summarise, while maintaining equal radial distribution of nodes (as we already have), group inbound connections towards the top, bidirectionals to the sides and outbound connections to the bottom. Here are some examples :
→

Make highlights more aggressive as in #3293

Fine by me, on the grounds that it is definitely an improvement on the status quo.


added a course for linux

Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add Resource(s) | Remove Resource(s) | Add info | Improve Repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

getting Travis CI - build Failed
how can i solve this?


Feature request: bind arguments to a sol::function

I'm not sure if this is possible yet, but I would like to be able to create a sol::function or sol::protected_function from another instance with bounded arguments.
Example:
sol::state lua;
lua.open_libraries(sol::lib::base);

int i = 0;
lua.set_function("f", [&i](int n) { i = n; });

lua["f"](1);
REQUIRE(i == 1); // ok

sol::function f = lua["f"];
f(2);
REQUIRE(i == 2); // ok

// new stuff
sol::function b = sol::bind(f, 3); // similar to std::bind
b();
REQUIRE(i == 3);

This would require me to make sol::function heavier than it needs to be. By a lot.
Use std::function<...>, and std::bind, plus a lambda.
I don't want to introduce a sol::bound_function either, just to support this because it's another type that will ruin things.

You can also stuff it into lua to act as the "bind", that will keep f alive.
![https://i.imgur.com/DfVampm.png](It works)
#define SOL_CHECK_ARGUMENTS 1
#include <sol.hpp>

int main() {

	sol::state lua;

	int i = 0;
	lua.set_function("f", [&i](int n) { i = n; });

	sol::function sf = lua["f"];
	lua["g"] = [f = std::move(sf)]() -> void {
		f(3);
	};

	sol::function g = lua["g"];
	g();
	assert(i == 3);

	return 0;
}

Thanks, I have written a generic lua function so I can bind dynamically.
#define SOL_CHECK_ARGUMENTS 1
#include <sol.hpp>

int main() {

	sol::state lua;
	lua.open_libraries(sol::lib::base, sol::lib::table);

	lua.script(R"(function bind(f, ...)
                          local params = {...}
                          return function()
                              return f(table.unpack(params))
                          end
                      end
                   )");

	int i = 0;
	lua.set_function("f", [&i](int n) { i = n; });

	sol::function sf = lua["f"];
	sol::function g = lua["bind"](sf, 3);

	g();
	assert(i == 3);

	return 0;
}


Get Node.js runtime for e2e tests dynamically

Close #78

Unfortunately this naive approach isn't trivial to get working. Therefore, I'll be closing this and leave finding a suitable solution for later. If anyone has a good idea for how to fix #204, please feel free to contribute it :slightly_smiling_face:


Update keyfactor-go-client to v1.3.3

This incorporates a patch for the v1.3.2 release of keyfactor-go-client, fixing the issue regarding exporting security roles in KFUtil.

I am unsure if I tested this release change correctly. When I download a zip of the v1.3.3 keyfactor-go-client release and paste it into the vendor directory, I the security roles are correctly exported. However, when I run 'go get<EMAIL_ADDRESS>and allow Goland to create a vendor file automatically, none of the changes from v1.3.3 seem to load. I went ahead and created this pull request anyways, but I am curious if there is something I'm doing wrong to cause the keyfactor-go-client to not update.

I am unsure if I tested this release change correctly. When I download a zip of the v1.3.3 keyfactor-go-client release and paste it into the kfutil vendor directory, I can confirm the v1.3.3 works because the security roles are correctly exported. However, when I run 'go get<EMAIL_ADDRESS>and allow Goland to create a vendor file automatically, none of the changes from the new v1.3.3 seem to load. I went ahead and created this pull request anyways, but I am curious if there is something I'm doing wrong to cause kftil to not load the revised keyfactor-go-client v1.3.3?

I don't know what the issue was here but I made these changes to the go client https://github.com/Keyfactor/keyfactor-go-client/pull/19 to get export to work for me.


Some capabilities are missing in hello message(status).

Hi @michalvasko
After connecting with netconf cli i exchange the hello capabilities by using status cmd but below capabilities are missing can you see this and also tell why port is different every time when i perform call home feature.
Following capabilities not present in Hello capabilities exchange message "
Discard change operation
Lock and un-lock operations
Confirmed commit Capability
Cancel commit operation
URL capability

status
Current NETCONF session:
ID          : 2
Host        : ::ffff:<IP_ADDRESS>
Port        : 47628
Transport   : SSH
Capabilities:
urn:ietf:params:netconf:base:1.0
urn:ietf:params:netconf:base:1.1
urn:ietf:params:netconf:capability:writable-running:1.0
urn:ietf:params:netconf:capability:candidate:1.0
urn:ietf:params:netconf:capability:rollback-on-error:1.0
urn:ietf:params:netconf:capability:validate:1.1
urn:ietf:params:netconf:capability:startup:1.0
urn:ietf:params:netconf:capability:xpath:1.0
urn:ietf:params:netconf:capability:with-defaults:1.0?basic-mode=explicit&also-supported=report-all,report-all-tagged,trim,explicit
urn:ietf:params:netconf:capability:notification:1.0
urn:ietf:params:netconf:capability:interleave:1.0
urn:ietf:params:xml:ns:yang:ietf-yang-metadata?module=ietf-yang-metadata&revision=2016-08-05
urn:ietf:params:xml:ns:yang:1?module=yang&revision=2017-02-20
urn:ietf:params:xml:ns:yang:ietf-inet-types?module=ietf-inet-types&revision=2013-07-15
urn:ietf:params:xml:ns:yang:ietf-yang-types?module=ietf-yang-types&revision=2013-07-15
urn:ietf:params:netconf:capability:yang-library:1.1?revision=2019-01-04&content-id=61
urn:sysrepo:plugind?module=sysrepo-plugind&revision=2020-12-10
urn:ietf:params:xml:ns:yang:ietf-netconf-acm?module=ietf-netconf-acm&revision=2018-02-14
urn:ietf:params:xml:ns:netconf:base:1.0?module=ietf-netconf&revision=2013-09-29&features=writable-running,candidate,rollback-on-error,validate,startup,url,xpath
urn:ietf:params:xml:ns:yang:ietf-netconf-with-defaults?module=ietf-netconf-with-defaults&revision=2011-06-01
urn:ietf:params:xml:ns:yang:ietf-netconf-notifications?module=ietf-netconf-notifications&revision=2012-02-06
urn:ietf:params:xml:ns:yang:ietf-netconf-monitoring?module=ietf-netconf-monitoring&revision=2010-10-04
urn:ietf:params:xml:ns:netconf:notification:1.0?module=notifications&revision=2008-07-14
urn:ietf:params:xml:ns:netmod:notification?module=nc-notifications&revision=2008-07-14
urn:ietf:params:xml:ns:yang:ietf-x509-cert-to-name?module=ietf-x509-cert-to-name&revision=2014-12-10
urn:ietf:params:xml:ns:yang:iana-crypt-hash?module=iana-crypt-hash&revision=2014-08-06
urn:notification_history?module=notification_history
urn:ietf:params:xml:ns:yang:iana-if-type?module=iana-if-type&revision=2017-01-19

why port is different every time when i perform call home feature

The printed port is not the one the client was listening on but rather the one the session is actually established on with the server. That is auto-assigned by the kernel and differs for every connection and every client.

Following capabilities not present in Hello capabilities exchange message "
Discard change operation

What capability do you have in mind, can you reference some specification?

Lock and un-lock operations

Same problem, I am not aware of any specific capability for these operations.

Confirmed commit Capability

Unless you are using a recent version of netopeer2 v2, this capability is actually not supported.

Cancel commit operation

Has no specific capability and support for this operation is indicated by the confirmed-commit capability.

URL capability

This is an optional capability that is disabled if you compile netopeer2 without having the devel package of libcurl installed.