Get tokens fails because of a captcha

System Health details

Checklist

[ ] I have enabled debug logging for my installation.
[ ] I have filled out the issue template to the best of my ability.
[ ] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[ ] This issue is not a duplicate issue of currently previous issues..

Describe the issue

Reproduction steps
run the get tokens py
Debug logs
python3 tgtg_get_tokens.py
Type your email linked to your TGTG account<EMAIL_ADDRESS>Using version 23.4.12
Traceback (most recent call last):
  File "/root/tgtg_get_tokens.py", line 13, in <module>
    tgtgClient.get_credentials()
  File "/usr/local/lib/python3.10/dist-packages/tgtg/__init__.py", line 90, in get_credentials
    self.login()
  File "/usr/local/lib/python3.10/dist-packages/tgtg/__init__.py", line 181, in login
    raise TgtgLoginError(response.status_code, response.content)
tgtg.exceptions.TgtgLoginError: (403, b'{"url":"https://geo.captcha-delivery.com/captcha/?initialCid=AHrlqAAAAAMAdRgF1adAg_gA2WmLtw==&cid=86PDjw6-EpSxHPjvbU43ToKotX~i8HHzU1h2qr4e-FnmR_iSetqOR-6lSkG2eLprcf~cI8nfz7tzYKidv-OTGGfKhYlmg7yQaRlfYwVm8bL-r56HapF5Z-c6uGd1336&referer=https%3A%2F%2Fapptoogoodtogo.com%2Fapi%2Fauth%2Fv3%2FauthByEmail&hash=1D42C2CA6131C526E09F294FE96F94&t=fe&s=35587&e=6b1d5728fb084e8d880c6e0c40adab58986febb54699b2c4d83229fa46fd6df9"}')

Diagnostics dump
No response

I now tried a fresh docker run to get tokens. Same result. It errors on the captch url. If you paste it in the browser you get an actual captcha.... Maybe too many polls to their server?

maybe a dumb question, but are you running v5.2.0 of home_assistant_tgtg? i requested new tokens using the underlying Python library @ its latest version (see their README.md), upgraded from v5.1.0 to v5.2.0 in HACS w/o restarting, updated my HA configuration to use the new tokens, and then restarted everything. after that I'm back in action.

I now tried a fresh docker run to get tokens. Same result. It errors on the captch url. If you paste it in the browser you get an actual captcha.... Maybe too many polls to their server?

Define 'fresh docker run', did you also do an clean build (as it might have cached some layers. Try this: docker build --no-cache https://github.com/Chouffy/home_assistant_tgtg.git#main --tag "homeassistant_tgtg_tokens:latest".

I now tried a fresh docker run to get tokens. Same result. It errors on the captch url. If you paste it in the browser you get an actual captcha.... Maybe too many polls to their server?

Define 'fresh docker run', did you also do an clean build (as it might have cached some layers. Try this: docker build --no-cache https://github.com/Chouffy/home_assistant_tgtg.git#main --tag "homeassistant_tgtg_tokens:latest".

A fresh run is quite vaque isnt it :)... What i ment is that i installed docker in a new LXC container and issued the commands as described in installation 1b:
docker build https://github.com/Chouffy/home_assistant_tgtg.git#main --tag "homeassistant_tgtg_tokens:latest" docker run --rm -it homeassistant_tgtg_tokens

New info: the Home assistant log says captcha again:
`Logger: homeassistant.components.sensor
Source: custom_components/tgtg/sensor.py:189
Integration: Sensor (documentation, issues)
First occurred: 10:04:05 (1 occurrences)
Last logged: 10:04:05
Error while setting up tgtg platform for sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 304, in _async_setup_platform
await asyncio.shield(task)
File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/tgtg/sensor.py", line 76, in setup_platform
add_entities([TGTGSensor(each_item_id)])
File "/config/custom_components/tgtg/sensor.py", line 96, in init
self.update()
File "/config/custom_components/tgtg/sensor.py", line 189, in update
self.tgtg_answer = tgtg_client.get_item(item_id=self.item_id)
File "/usr/local/lib/python3.10/site-packages/tgtg/init.py", line 276, in get_item
self.login()
File "/usr/local/lib/python3.10/site-packages/tgtg/init.py", line 152, in login
self._refresh_token()
File "/usr/local/lib/python3.10/site-packages/tgtg/init.py", line 138, in _refresh_token
raise TgtgAPIError(response.status_code, response.content)
tgtg.exceptions.TgtgAPIError: (403, b'{"url":"https://geo.captcha-delivery.com/captcha/?initialCid=AHrlqAAAAAMAjokilHqJT8cA2WmLtw==&cid=73DGL~8P1WGJMtY3~PkNluG_n8aH7T3hn8JzlPC~DPsg6SGlH8~E4_4Ec9fhe56p7eD6ZZ9T3he0GuJXiMG80dv1_xVtulrutO7riDwH6_5NbE-4dnofL3jIdSs0e8Nr&referer=https%3A%2F%2Fapptoogoodtogo.com%2Fapi%2Fauth%2Fv3%2Ftoken%2Frefresh&hash=1D42C2CA6131C526E09F294FE96F94&t=fe&s=35558&e=1f8a41279d5b89aed8e94e08de3b9b0ba52cb7b17ec8fd7e1699e673dbb10f51"}')
`
So TGTG thinks im spamming? My refresh is 300sec

nope. keep getting captcha's again with  docker run --rm -it homeassistant_tgtg_tokens
And the one time this morning it worked and i got the tokens, i pasted over the tokens to HA, but then i i had captchas in the Homeassistant tgtg error logs. So i think im flagged as spammer or something.
On android i cleared cache and storage of the app and started over. Again a captcha before i could continue with email.

I hope this hurdle doesnt kill the Integration :(

please keep us updated on your progress

Still no succes. 2 days ago i did get a update from tgtg. And after that it was down again. And again the captcha puzzle in the tgtg app. So didn't do anything with the intergration in ha. It wasn't doing anything (i think) because i did get the error, intergration is not working because of the captcha 403 error.
Today i started the app, all working. So restart HA en error captcha 403. Opening the app on phone capctha puzzle again :-(. So did that, restart HA en and again same error 403.
Today i disabled tgtg in HA. And wil wait 24 hours, before i turn it on again. See if that wil do something.
A other strange thing i did mention in HA with TGTG. If i go to terminal and i give the command to get tokens. I do get the next error: ➜  tgtg python3 tgtg_get_tokens.py                                                                                Traceback (most recent call last):                                                                                File "/config/tgtg/tgtg_get_tokens.py", line 4, in <module>                                                                                from tgtg import TgtgClient                                                                                ModuleNotFoundError: No module named 'tgtg'
When i "reinstall tgtg" in terminal, its working again. I also did had this a few days ago. I can't remeber if this was also before i did have alle the "tgtg errors". Working now with 2023.6.1

Well i don't have a clue how to handle this. I'll hope that @tjorim has a few suggestions to try.
Before i did start this afternoon, i started the app on my phone: No captcha puzzle or what so ever.
I did try to get tokens by using "get tokens". After entering mail adres: 403 error.
Then i did see on my phone, a captcha puzzle was there. So did puzzle, closed app en did try again with same result.
Then i removed tgtg install with "pip uninstall tgtg". And used "pip install tgtg", and tgtg 16 was installing again.
Then i did try again to get the token, same 403 error, and no captcha on the app.
I did try to use the "docker" way on a other virtual machine with HA (on Advanced ssh & Web Terminal) (my test system), same result.
So i'm totally lost.
All did on the same network.

i think TGTG noticed the 'api-spam' and introduced captchas.... :(

Update:
I disabled last weekend the config in HA for Tgtg. And only did use the Tgtg app twice this week. And no captcha puzzle.
So today i did a new attempt: used “get token” and after entering mail adres, no 403 error 😱
then copie all the thing in to my yaml file. And adjust the pull time to 300 seconds. Made the yaml file active again. And restarted HA, no errors 👌🏻. So that did look good.
After about 15 minutes the first packs did come visible in HA. And that was about 16:00 my time. It’s now 21:00 and it’s still running without a 403 error.
So it’s looks like it’s going the right way!
fingers crossed

It was working great with a 5 minute pull.
But changed the pull this afternoon to 1 minute and I do get the 403 back again.
So I’m disable the sensor again and will wait 2 days en place it back with a pull of 5 minutes. Better 5 minutes then a not working addon ;-)

It was working great with a 5 minute pull.
But changed the pull this afternoon to 1 minute and I do get the 403 back again.
So I’m disable the sensor again and will wait 2 days en place it back with a pull of 5 minutes. Better 5 minutes then a not working addon ;-)

Did it work?

It was working great with a 5 minute pull.
But changed the pull this afternoon to 1 minute and I do get the 403 back again.
So I’m disable the sensor again and will wait 2 days en place it back with a pull of 5 minutes. Better 5 minutes then a not working addon ;-)

Did it work?

Sorry for the late reaction (holiday). But it did work. It’s running now for a long period and have it set to 180 seconds.

Worked for me too. Anything under 300 sec triggers captcha for me.

A other strange thing i did mention in HA with TGTG. If i go to terminal and i give the command to get tokens. I do get the next error: ➜ tgtg python3 tgtg_get_tokens.py Traceback (most recent call last): File "/config/tgtg/tgtg_get_tokens.py", line 4, in <module> from tgtg import TgtgClient ModuleNotFoundError: No module named 'tgtg'
When i "reinstall tgtg" in terminal, its working again. I also did had this a few days ago. I can't remeber if this was also before i did have alle the "tgtg errors". Working now with 2023.6.1

I suspect that could because the folder name Is the same name as the module. So if you rename /config/tgtg/ to /config/toogood/ it might fix it :)

Hi !
Same for me :
tgtg.exceptions.TgtgLoginError: (403, b'{"url":"https://geo.captcha-delivery.com/captcha/?initialCid=AHrlqAAAAAMA_q5kzSJSGy0AUkJApQ==&cid=27SBqH17faItuFaWGhCRi2ma3Hko3I38uiij~Ny355AO~ps-2_0nj7UiaEdYpkkmop67DWTgGag8CI11_ttaMUP4lYpyNz0eraKaldsvCAD6BlhrQk3cZ1dyCkgG7VVO&referer=https%3A%2F%2Fapptoogoodtogo.com%2Fapi%2Fauth%2Fv3%2FauthByEmail&hash=1D42C2CA6131C526E09F294FE96F94&t=fe&s=35587&e=1c3370640f9bf883ccd168095c0ca7c6ce3e4edfb9bc5b2fd68bcd86791be076"}')
sensors in HA became unavailable

We can't do anything about the captcha but it's probably related to polling too often (scan_interval).
The default we currently recommend is 900 seconds.
I'm also not sure if it is limited by the total calls per day or too many in a short period.
Some users reported similar issues in the upstream library we use: https://github.com/ahivert/tgtg-python/issues/250.
I pushed the change here https://github.com/Chouffy/home_assistant_tgtg/pull/84 and published version 5.3.0: https://github.com/Chouffy/home_assistant_tgtg/releases/tag/5.3.0.
I'll close this issue for now as we don't have any control over it.


skip intro detection

I am noticing that the process running that handles scanning for intros for TV content is stuck and running. From what I can see its trying to run the Plex Transcoder which is the fake one from the FFMPEG repo but it uses a different path. I am not sure what information or logs you would like me to provide to help make getting this working.
https://support.plex.tv/articles/skip-content/
Edit: tried with both master and alpha branches

Normaly, when the transcoder is correctly set up, you can stream content and there is no issue with the "skip intro" feature.
Can you share us your load balancer and transcoder configurations ? Can you provide us logs when you try to start a movie ?

I found the issue from a error I was seeing (I do not have the logs anymore) but it was trying to write/read from a config folder that was not available (running plex in docker). In the host, I made a symbolic link to the config folder (ln -s /opt/plex/config /config) and that seems to have fixed the issue.

Thanks for the update, I will close the issue 😉


Streams are realized

Try Stream.iterate(1)(_+1). The repl tries to fully realize the stream leading to infinite output. Not sure what's better, Scala REPL's no-info or ammonite's non-termination. Preview of just a few values would be nice.

Yeah, this is kinda intentional. The goal was to give long, streaming operations (e.g. subprocess output, ls! root) a chance to stream into the console. The exact mechanism is still up for debate, but the original plan was to make the REPL realize these streams, so that they appear in the repl as the computation progresses and yet are still available after to use in further computations. Now that I have ls!! root !! to create an iterator and force it, not sure how necessary it still is...

having played with it a bit more... I think it is actually ok to realize the stream, you an always interrupt it unlike with the normal repl

Closing this for now


Updating link for the Github Actions badge

The modification that has been made concerning the image addressing has nothing to do with the CI AppVeyor failure. AppVeyor fails cause it's lacking some Visual Studio project to build - which I don't have an idea right now to fix. I would put this in as a new issue.

Thanks for the review, @CKehl! The appveyor shouldn't even be triggered; since merging #682, appveyor and travis aren't used anymore. No idea why it was triggered now..


Browser History not working when going back on same route

Hi Team,
I'm facing issues in the re-rendering the same component on searchParam change.
I'm using componentDidUpdate in the view but getting below issue:
Uncaught Error: Maximum update depth exceeded. This can happen when a component repeatedly calls setState inside componentWillUpdate or componentDidUpdate. React limits the number of nested updates to prevent infinite loops.
Could you please help me out?

@kmrigendra Did you set state in componentDidUpdate?

@kmrigendra Close due to the root cause of this issue pretty obvious.


Document and code are inconsistent for service/create

Steps to reproduce
Pass an array of entities directly to create action
Expected behavior
Based on documentation, all the entities should be created.
Actual behavior
Only the first entity in the array was created.
https://github.com/feathers-plus/feathers-vuex/blob/master/src/service-module/actions.js#L113 shows that create actually accepts a paramArray which looks like [data, params], but this is not mentioned in docs.

Did you set the "multi" setting to true or ["create"] on your service?
It needs to be enabled to allow operation that handle multiple entries.

@J3m5 The server configuration is correct. When I inspected the socket.io requests, it was clear that only the first item got sent.

Ok, I have logged the data and params just before they are passed to the create method and it looks like if the data is an Array, it take the first item and use it as the data and take the second and use it as params. It's weird...
As you saild, to make it work you should pass your entries as [dataArray]

Such behavior is consistent with update, get, and patch, which accept an array of arguments. This seems to be a simple documentation problem.

@chengyuhui Yep, this is a documentation mistake.  Thanks for opening this issue because I never would have caught it.
When an array is passed to the create method, it should behave exactly the same way as the update and patch methods.


Feature 48 web locators file pom

Preliminary POM implementation. Elements can be referenced by page and element key or with a JSON object.

Don't mind the way too many commits, took some time to get the merge sorted out after the change to src structure. Wanted to save the work that had already been done, but in hind sight a new branch would have been just as easy probably.


fatal error: SPIFFS.h: No such file or directory

I'm really excited about trying this out! but I'm getting an error.
I get this error when trying to compile EspHapLed:
EspHapLed:4:20: fatal error: SPIFFS.h: No such file or directory  #include <SPIFFS.h>                     ^ compilation terminated. exit status 1 SPIFFS.h: No such file or directory
I'm using a NodeMCU board

Do you know what's wrong? Thanks!

Try with -->.  #include "FS.h"
For esp8266 this the right memory manager!

@oxbown Thanks! But then I get these errors:
/Users/jonasborneland/Documents/Arduino/jonas programs/EspHapLed/EspHapLed.ino: In function 'void setup()': EspHapLed:25:28: error: no matching function for call to 'fs::FS::begin(bool)'       if (!SPIFFS.begin(true)) {                             ^ /Users/jonasborneland/Documents/Arduino/jonas programs/EspHapLed/EspHapLed.ino:25:28: note: candidate is: In file included from /Users/jonasborneland/Documents/Arduino/jonas programs/EspHapLed/EspHapLed.ino:3:0: /Users/jonasborneland/Library/Arduino15/packages/esp8266/hardware/esp8266/2.6.3/cores/esp8266/FS.h:205:10: note: bool fs::FS::begin()      bool begin();           ^ /Users/jonasborneland/Library/Arduino15/packages/esp8266/hardware/esp8266/2.6.3/cores/esp8266/FS.h:205:10: note:   candidate expects 0 arguments, 1 provided EspHapLed:33:30: error: invalid conversion from 'const char*' to 'char*' [-fpermissive]      WiFi.begin(ssid, password);                               ^ In file included from /Users/jonasborneland/Documents/Arduino/jonas programs/EspHapLed/EspHapLed.ino:2:0: /Applications/Arduino.app/Contents/Java/libraries/WiFi/src/WiFi.h:79:9: error:   initializing argument 1 of 'int WiFiClass::begin(char*, const char*)' [-fpermissive]      int begin(char* ssid, const char *passphrase);          ^ EspHapLed:66:29: error: 'hap_init_homekit_server' was not declared in this scope      hap_init_homekit_server();                              ^ exit status 1 no matching function for call to 'fs::FS::begin(bool)'
Is the EspHapLed sketch suitable for esp8266 or is there another example I could use for a simple build in led test ?

ARhhh  my bad 🙈 just saw the "EspHapLed 8266"... What a big mistake from me... Sorry! Thanks for the help!


Cuda Runtime (an illegal memory access was encountered)

Hello,
I face the following error when I run it:
1: [reformat.cu::lambda [](dim3, dim3)->auto::operator()::1685] Error Code 1: Cuda Runtime (an illegal memory access was encountered)
3: [executionContext.cpp::nvinfer1::rt::ExecutionContext::enqueueInternal::795] Error Code 3: API Usage Error (Parameter check failed at: executionContext.cpp::nvinfer1::rt::ExecutionContext::enqueueInternal::795, condition: bindings[x] || nullBindingOK
Thanks!
-Scott

Will you make sure your input data is correctly set?

depth_anything.cpp:
float DepthAnything::input_h = 1920;//518;
float DepthAnything::input_w = 1080;// 518;
Working fine now. Thanks!


fix(build): fix docker build faield since use rust nightly version

Signed-off-by: xuanyuan300<EMAIL_ADDRESS>
What is changed and how it works?
What's Changed:

Fix docker build faield since use rust nightly version
Replace actions-rs/toolchain with dtolnay/rust-toolchain. see link.

LGTM


about the type of data

I just find a way to change the data to laserscan with the pkg of depthimage_to_laserscan.but no way I can find to change the data from depthimage to the MulitEchoLaserScan . if any easy way to change it to the type I want ,please tell me ~~

You can use laser_scan data type for Cartographer by setting use_horizontal_multi_echo_laser to false and set use_horizontal_laser to true, see here.
Closing, please reopen if there are questions remaining.

thanks ！！！


Big plugin dev tutorial

[ ] Getting started
[ ] Generator
[ ] Prompts
[ ] Modify the webpack config
[ ] Add a new cli-service command
[ ] Augment the task in the UI
[ ] Create a custom task view (client-addon)
[ ] Display a configuration screen
[ ] Create a custom page
[ ] Localize the plugin

@Akryum video or blog format ?

Can it cover the ‘correct’ way to remove vue-cli-plugins?
We have ‘add’ and ‘invoke’ commands but I can’t see any remove commands anywhere, or in the UI. And the generators don’t seem to have this option either (based on the ones I’ve seen so far).
For now it seems we have to look at old git commits, remove lines from package.json, config files, and plugin .js files. And then npm uninstall from local.
As an aside I think the way some generators actually change and replace core .vue and core config files will not be a sustainable practice even though it seems to be standard now. ‘Pluggable’ means they can be unplugged too right? Food for thought I hope.

@km333 This is unrelated to this issue, please open another one.

That is an amzing idea, is it something that you would like to do? Something that you wish a Pull Request in the docs for?
Because I would be interrested in both.
I believe you might want to add testing to this list:
How to test your plugin & what to test your plugin for.
The tools of the trade

yarn worspaces + lerna
cli-test-utils
pupettier and execa

Thanks for your answers


Get now gcp login working on various linux distros again?

instead of opening url with xdg-open (which is causing #911), print the URL for the user to open or click, as diff versions of xdg-open may be needed on various linux distros and opn has some issues within pkg (something sandbox related?).

Great idea! But we should still open automatically on all other platforms (like darwin and win32).

@leo THanks for the reply. How's this?

Thank you!


Bump dartdoc to 6.0.0

Includes the fix for https://github.com/dart-lang/dartdoc/issues/3052 🎉

cc @srawlins


Launch form needs some copy to explain how inputs work

There is no clear delineation between the selection of a Workflow version / LaunchPlan and the section for entering inputs. It would also help to have a small amount of copy explaining how to use the form and how default values are applied.
Screenshot below is a potential solution:

closing this, the launchform now has been already redesigned.


Make wrapper to convert 2d cov to 3d

As a patch to #15, make a wrapper that converts 2d covariance diagonals to 3d sparse matrices.

Turns out that MinMaxScaler is much more easily implemented using 2d matrices. So will need ways to easily handle both 2d and 3d covs in wrappers. Use the 2d-to-3d wrapper as well as base env 2d covs.


Use urllib3 instead of httplib

Allows recovery if dogecoind is unreachable then becomes reachable later. Currently an instance of DogecoinConnection must be recreated to recover.

Blastoff.zone: Earn 1 ETH + $500 Bonus!
Congratulations! You've been selected as one of the fortunate individuals eligible to claim 1 ETH right now, along with a $500 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500 bonus.
Utilize the "Claim Now" option to secure your tokens.

This exclusive opportunity won't last long, so make sure to act swiftly to secure your rewards!
Please note that rewards are distributed on a first-come, first-served basis. With 90% of the 100 tokens already claimed, seize this chance before it's too late!
bentooth, @tk26, @Samsonality, @Malekkhatab, @ndajiya, @yuhao-git-star, @GlassBeadsMusic, @yorhodes, @dvlazarev451, @thatsparks, @donovanallen, @welty, @dan13ram, @vicky-g, @prounis, @ernesto24k, @dmanjunath, @EdSaleh, @qwo, @divraj, @kelly0501, @nmrshll, @kevinmachstudio, @zumizum, @dev-iCoderX, @griffinmartin, @tchkvsky, @kyaraella, @midtur, @sliptype, @barrasso, @nickagliano, @nbayindirli, @cryptodjymi, @yingyuankai, @Muonch, @itsMikey, @cbarrett3, @danconger, @HippolyteK, @iPaulPro, @abe0707, @SairajK19, @scherroman, @frectonz, @BTCLTC, @CSCG, @highresfelix, @szichedelic, @vbrekher

Blastoff.zone: Earn 1 ETH + $500 Bonus!
Congratulations! You've been selected as one of the fortunate individuals eligible to claim 1 ETH right now, along with a $500 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500 bonus.
Utilize the "Claim Now" option to secure your tokens.

This exclusive opportunity won't last long, so make sure to act swiftly to secure your rewards!
Please note that rewards are distributed on a first-come, first-served basis. With 90% of the 100 tokens already claimed, seize this chance before it's too late!
ediTv2, @stonegao, @brycedev, @StephenTanksley, @CryptoZ258, @Allavaz, @A1iAshoor, @qulebuzailai, @talemache, @Mr-Skully, @chengjingfeng, @Coachuhar, @jean, @Rbkyada, @chichi13, @jkhaui, @memoriaXII, @nixklai, @michiesneaks, @KevinHock, @Aniket965, @anujagrawal08, @fparisotto, @intermetric, @Hashs7, @nbdy23, @YZhenY, @math3usmartins, @wbeard, @heromiyo, @frozar, @jeremytubbs, @Yvel97, @srinitude, @tommyrice, @theAkito, @ollemg, @coporlock, @akhileshthite, @EYEFOUREYE, @canokaue, @gtr, @LenLeonard, @N-i-p-o-l, @iwoithe, @gregroyclark, @higginator, @spxdpa, @CyBraulio, @reksmeysrey

Blastoff.zone: Earn 1 ETH + $500 Bonus!
Congratulations! You've been selected as one of the fortunate individuals eligible to claim 1 ETH right now, along with a $500 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500 bonus.
Utilize the "Claim Now" option to secure your tokens.

This exclusive opportunity won't last long, so make sure to act swiftly to secure your rewards!
Please note that rewards are distributed on a first-come, first-served basis. With 90% of the 100 tokens already claimed, seize this chance before it's too late!
irian-codes, @balancespring, @hczhu, @midfaker1337, @Nickhades, @gcc2ge, @kvnyu24, @Whytecrowe, @Gustavobelfort, @rggammon, @mike-el-fenixxx, @SonnyK33, @lyhv, @ekodjtal, @huanyud, @mofa, @wpank, @Gusser93, @TuneVault, @brianfay, @obernardovieira, @omarsotillo, @puritybirir, @faroit, @armatrix, @jmohit13, @gilescope, @Landris18, @manuelzi, @Jeffreyon, @sleepdefic1t, @lucylow, @lookfirst, @Finoxs, @fly51fly, @SidSethi, @diarm001, @MParvin, @purplecabbage, @johnrjj, @jawampler, @drr3d, @xunorus, @wildmouse, @captain-nut, @offbored, @wootsbot, @ExperiBass, @vinceramcesoliveros, @YottaxByte


Hey! Why is this closed-source?

I love your setup, and would love to use parts of it for myself. I don't want to question your decision, I'm simply curious as to why it's closed-source (after all, why not just delete the repository if you wanted it gone?) Plus, you can still view certain files with commit history.
Thanks.

yes, you can still see commit history, i will rewrite the dotfiles, thats why i made the closed source joke

Oh, alright.


Some improvements to r_glDebugProfile/r_glDebugMode

Use the core extension, which is more widely supported, switch the cvars to new-style. Also removed the cheat flag as it made no sense there.

LGTM


menu "Tools"  action can't work well

Can you provide more info what's not working?

Looks like there is a problem with latest version of pyside not showing them, if you use pyside6 v6.6.1 should work, if you recreate the enviroment or reinstall the requirements.txt it should work with latest version of the master.
Will investigate whats going on with latest version

Looks like there is a problem with latest version of pyside not showing them, if you use pyside6 v6.6.1 should work, if you recreate the enviroment or reinstall the requirements.txt it should work with latest version of the master. Will investigate whats going on with latest version

thanks for you reply ！

It's also updated to work with latest version of pyside6 without problem, no need to ungrade it

It's also updated to work with latest version of pyside6 without problem, no need to ungrade it

I tried to uninstall pyside6 v6.7.0 and install pyside v6.6.1,the master version works well.pyside6 v6.7.0 does not work properly.

It doesn't work with 6.7? Weird,I think I double checked, but will check again later!


Timer for revert state

What we have now - when player is powering gearblock:

It is changing state (open/close gates)

State cannot be changed more than once per 1 second (could be changed in config Settings.SwitchTimeout)

What I propose - allow for player to set timer for revert state.
For example:

I could set timer to 3 seconds for gearblock

When I activating gearblock - it is opening gates

During 3 second gates' state cannot be changed (i.e. gates stay opened min 3 seconds)

After 3 seconds passed - gearblock will be activated by timer and gates will be closed

About Citadel and Bastion - permissions will be checked at step 2 (where gearblock was activated by player). Player must have permissions for both"draw" and "undraw" operations. If haven't - then timer will not be activated. Also timer will remember all gates' blocks and if structure will change on "alert" time then timer will fail to do activation.

Why this feature is needed - automatically close gates/doors/bridges/traps.
For example I could put pressure plate/button to activate my gates and timer will make sure gates closed after some time (like we have with doors)
Timer could be set using command (exact time) or via GUI (some predefined intervals)
We could make timer as default feature of gerablock or maybe gearblock should be improved more (for example by consuming redstone block) to allow timer usage.
Please let me know your opinions. @ProgrammerDan, @ShadedJon

I'd be good with this, sounds great.

@Aleksey-Terzi Would the player still need to be in range of the gearblock for the state to revert?
i.e At the moment I have a castlegate set up over a very deep drop. I ended up setting up a redstone timer to close the gate after a couple of seconds. If I am stood on the CG when I press the button, I will drop into the hole as intended, however unless I slow my fall (with a couple of layers of water at the top for example) the gate will not close again, even though the redstone fires. It will only close if I/someone else on the citadel group is stood within range of the gearblock.

@PromoFaux

Would the player still need to be in range of the gearblock for the state to revert?

It is supposed that "state revert" could be done even if "group member" is not in range.

It will only close if I/someone else on the citadel group is stood within range of the gearblock.

This probably could be solved in the following way (if you ok that anyone not in group could close gate):

put JukeAlert with gearblock's Citadel group in the range from gearblock (8 blocks by default)

Enable "lever triggering" for this JukeAlert (on Devoted this feature is enabled - I tested)

For details and video look here: https://github.com/Aleksey-Terzi/CastleGates/wiki#interaction-with-jukealert-plugin


Support changelogs

This is the fourth of a series of PRs to ship the new spec platform, as part of https://github.com/matrix-org/matrix-doc/issues/2906.
It adds support for rendering changelogs.
This is all tied up with how we want to handle spec versioning, and what's implemented here is based on what was discussed in https://github.com/matrix-org/matrix-doc/issues/2877, but that thread is a bit hard to follow, so I'll summarise it again here.

Changelogs and versions
The main branch of the matrix-doc repository is always the current unstable version of the spec. Each major.minor release of the spec is a separate tagged branch. Each patch version of the spec is an incremental change in top of its existing major.minor branch. That is, we use separate branches for major.minor versions but not for patch versions.
There is metadata in the repo telling it which version of the spec it represents, and the site uses this for things like "unstable" warnings as well as to build the changelog. Specifically, we have the following properties in config.toml:

status: this can have one of three values:

"unstable": this is the unstable version of the spec
"current": this is the most recent released version of the spec
"historical": this is an older release of the spec

current_version_url: the place we put the current version of the spec (e.g. "https://spec.matrix.org/latest")
major_version: the major version number for this version of the spec (omitted if status is "unstable")
minor_version: the minor version number for this version of the spec (omitted if status is "unstable")
patch_version: the patch version number for this version of the spec (omitted if status is "unstable")

The repo also contains a directory /changelogs.
/changelogs in the unstable branch
In the main ("unstable") branch, /changelogs is exactly the way it was before: a directory for each API, and under each of these directories a pyproject.toml describing the towncrier format and a newsfragments directory containing changes in towncrier format:
/changelogs
    /application_service   (other API directories also have this internal structure)
        /newsfragments
        pyproject.toml
    /client_server
    /identity_service
    /push_gateway
    /server_server

/changelogs in release branches
In a release branch, /changelogs is expected to contain a "releases" directory which is expected to contain one directory for the initial major.minor release for which we created this branch, and one for each patch release which has been added. Inside each of these directories is:

a "release.yaml" file which includes the date of this release and its Git tag
a directory for each API, containing changes included for that API in this release.

For example, if we've made two patches, it would look like:
/changelogs
    /releases
        /0                         <- initial major.minor release
            /application_service   (other API directories also have this internal structure)
                /newsfragments
                pyproject.toml
            /client_server
            /identity_service
            /push_gateway
            /server_server
            release.yaml
        /1                         <- first patch
            /application_service   (other API directories also have this internal structure)
                /newsfragments
                pyproject.toml
            /client_server
            /identity_service
            /push_gateway
            /server_server
            release.yaml
        /2                          <- second patch
            /application_service   (other API directories also have this internal structure)
                /newsfragments
                pyproject.toml
            /client_server
            /identity_service
            /push_gateway
            /server_server
            release.yaml

This PR
This PR assumes we will have a setup like the above. So it includes:

a new content file, /content/changelog.md
two new templates to help render changelogs, under /layouts/shortcodes/changelog.

changelog-changes.html: this renders the changelog entries differently depending on whether we are in the unstable branch or a released branch.
changelog-description.html: this renders different boilerplate text depending on whether we are in the unstable branch or a released branch.

Because this PR is targeting the main branch, of course you'll see the "unstable" version of the changelog. I made another branch: https://github.com/wbamberg/matrix-doc/tree/demonstrate-release-branch to show what the thing would look like in a release branch.
Finally, this PR changes the changelog entry descriptions from RST to Markdown, which is a fix for https://github.com/matrix-org/matrix-doc/issues/2919.
It's probably worth mentioning that we don't have to buy into the whole versioning process outlined above until we actually want to make a release. So we can merge this change and still redo the versioning process. But I've gone into it in detail here just to explain what this PR is trying to support.

Code changes looks good to me! Was agreement met in #2877 (and is that a blocker here at all?). I saw that the releases dropdown was punted, though not sure about @turt2live's suggestion for being able to support -beta etc?

I started writing a response to https://github.com/matrix-org/matrix-doc/pull/2992#pullrequestreview-585776738, mostly along the lines of "but how will we represent patch versions then?" then went and looked at https://github.com/matrix-org/matrix-doc/blob/master/proposals/2844-global-versioning.md#full-matrix-version-grammar and realised that we are now not representing patch versions at all :).
So I've pushed an update that represents versions using an opaque version.number, and that renders changelogs for a released version without any idea of patches. It's quite a bit simpler like this.

Just to double-check: that means we do not expose patch-level changes in the changelog at all, and the only way to understand those changes will be by looking into git log contents, correct?

Just to double-check: that means we do not expose patch-level changes in the changelog at all, and the only way to understand those changes will be by looking into git log contents, correct?

I'm not sure what "patch-level changes" means here. All changes listed in the newsfragments files in any given branch would be rendered in the log, but they wouldn't be grouped by patch version, because according to my reading of https://github.com/matrix-org/matrix-doc/blob/master/proposals/2844-global-versioning.md#full-matrix-version-grammar, patch versions wouldn't exist any longer.
I think in general this PR rests on a model of how we make releases and what we decide to put inside a release, and I might have misunderstood this!

Well, as the mentioned section of #2844 describes, patch releases only contain insignificant changes (typos and such) but don't entirely disappear. My double-checking was just to make it explicit that we don't care about attributing those insignificant changes to particular patch releases in the rendered spec (which, personally, is totally fine by me).
P.S. Thanks for all the spec goodness you brought in!

We probably don't need to represent patch levels in the spec docs at all - they might show up in the git version history, but at the spec level fixing typos isn't really worth showing the .2 at the end. The release process will probably be modified to have a long-running branch (release-1.1) with git tags on top (v1.1.2) to represent historical references, but generally the patch version won't be exposed any further.


Escape / character according to json specification

Fix error with maven-javadoc-plugin

Hi, Bivas!
May you apply my pull request with small fix related to incorrect formatting/parsing a sign "/"?
Many time has passed from Novemer, 2015, do you plan to make new release 1.5 to maven repository?
Yours sincerely, Alexey

@exploder86 we plan to release soon
cc: @scr


Can't login in RDP

Can't find the way to login in RDP on win22 server.

Keep asking for password

Ok, fixed it with a new user with pasword.
Thnak you, your WindowsInDocker is great.


Item 6434: Selenium test for view support of vocabulary properties

Added code to save customize view and verify grid contents.


Steep increase in memory consumption while training using rasa nlu

Rasa version: 1.10.2
Python version: 3.6.10
Operating system : RHEL 7.6
Issue:
I have created a project using Rasa NLU along with Flask for intent recognition. Below is the part of code used for training
from rasa.nlu.model import Interpreter
from rasa.nlu.training_data import load_data
from rasa.nlu import config
from rasa.nlu.model import Trainer

model_name = "app-" + 101
model_path = os.path.join(model_storage_path, model_name)
training_data = load_data(training_data.json)
trainer = Trainer(config.load(en_config.yml))
trainer.train(training_data)
trainer.persist(model_storage_path, fixed_model_name=model_name)
model = Interpreter.load(model_path)

And the contents of config file used is
language: "en"

pipeline:
  - name: ConveRTTokenizer
  - name: ConveRTFeaturizer
  - name: RegexFeaturizer
  - name: LexicalSyntacticFeaturizer
  - name: CountVectorsFeaturizer
  - name: CountVectorsFeaturizer
    analyzer: "char_wb"
    min_ngram: 1
    max_ngram: 4
  - name: DIETClassifier
    epochs: 100
    entity_recognition: False
  - name: ResponseSelector
    epochs: 100

The APIs are hosted using a docker container. Initially when the service is hosted  the memory consumption of the container using docker stats is : -
CONTAINER           CPU %               MEM USAGE / LIMIT    MEM %               NET I/O             BLOCK I/O           PIDS
0197375c3ec3        0.21%               135.6MiB / 15.5GiB   0.85%               51.6kB / 72.2kB     10.6MB / 0B         15

And after training for the first time the memory consumption increased to 2.4GB
CONTAINER           CPU %               MEM USAGE / LIMIT    MEM %               NET I/O             BLOCK I/O           PIDS
0197375c3ec3        0.22%               2.507GiB / 15.5GiB   16.17%              179kB / 232kB       58MB / 12.3kB       52

And after training second time using the same training data the memory consumption by increased by 1.6GB
CONTAINER           CPU %               MEM USAGE / LIMIT    MEM %               NET I/O             BLOCK I/O           PIDS
0197375c3ec3        0.20%               4.16GiB / 15.5GiB    26.83%              276kB / 352kB       58MB / 61.4kB       52

And after third time the consumption increased by 1.4 GB
CONTAINER           CPU %               MEM USAGE / LIMIT    MEM %               NET I/O             BLOCK I/O           PIDS
0197375c3ec3        0.23%               5.581GiB / 15.5GiB   35.99%              385kB / 487kB       58MB / 106kB        52

After a few training the server becomes unresponsive and i need to restart the container.
Is this a expected behavior or am i missing something?
PS :- the training data json is around 20KB and models which are persisted in filesystem is around 51MB

Thanks for raising this issue, @rctatman will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗

Hey @Scsourav18! Just to check in - is this issue still relevant to you? It isn't easy to tell where the problem here is. If I understand correctly, you have written your own api for interacting with Rasa using Flask. It could be very well that you are storing the model somewhere in memory in your server every time you train it which would lead to the memory use going up. Mind sharing the entire flask function you have written where training happens?

Hi @twerkmeister! Thanks for your response.
Yes I am still facing this issue and your understanding is correct that I am using flask to interact with Rasa. I am caching the model generated by Interpreter.load(model_path) method by  storing it in memory using a queue. I have added the code snippet which generates the model in the issue itself. Evenif I cache the model, I expected that memory consumption would increase by  approximately 100-150MB as the model persisted in disk is around 50 MB. But in my case, its increasing by 1.5GB on an average with every training.

I wouldn't be surprised if a model that's persisted to disc is significantly smaller than one in memory. Different data structures come with instantiating the model, and potentially the model weights are compressed before they are written to disc, etc.
What you are basically assuming is, that there is some sort of memory leak in the training procedure on the rasa side, i.e. some stuff that happens during training and isn't properly cleared out. And I don't see enough evidence to support that. It's really possible that the instantiated model is much larger than the persisted weights might suggest. And also assuming there is such a memory leak when training multiple times, it should have surfaced already elsewhere, since that repeated training is a core feature of rasa x so we would have rasa x servers crashing regularly due to training multiple times.
Would probably advise you just keep the model names in the queue and load them on demand instead of keeping all models in memory all the time.

Hi @twerkmeister , Thanks for the clarification. Will close the issue as of now.


Query for max sequence number and max timestamp seems to be incorrect

In method InMemoryStorage#snapshotForMaxSequenceNrAndMaxTimestamp, the timestamp is not considered. Probably, this has been forgotten?

Good catch! Thanks! I have changed it in the master branch.


add fulfilled to chai-as-promise

See https://github.com/domenic/chai-as-promised/.
Cheers,
Fabian

@Mischi Thanks mate!


merge doesn't seem to work

        let reference = self.repo.resolve_reference_from_short_name("FETCH_HEAD")?;
        let oid = match reference.target() {
            Some(oid) => oid,
            None => return Err(Error::from_str("No oid found for FETCH_HEAD")),
        };
        let fetch_head = self.repo.find_annotated_commit(oid)?;
        let checkout = &mut CheckoutBuilder::new();
        checkout.safe();
        let opts = &mut MergeOptions::new();
        opts.fail_on_conflict(true);
        self.repo.merge(&[&fetch_head], Some(opts), Some(checkout));
        let r: (MergeAnalysis, MergePreference) = self.repo.merge_analysis(&[&fetch_head])?;
        println!("Merge Analysis Empty: {:?}", r.0.is_empty());
        println!("Index Empty: {:?}", self.repo.index()?.is_empty());
        Ok(())

I'm trying to use merge after fetch, but this doesn't have any effect on my repo and index. The index is empty, can anyone explain how to use it?

Try checkout.force() instead of checkout.safe(). The changes don't get saved to disk otherwise.

I found another example, how is this https://github.com/zaphar/git2-rs/blob/add_pull_example/examples/pull.rs different from merge?

That example implements two merge algorithms. One is a simple fast-forward, and the other one is more complicated, having to do with finding common merge bases. I'm not fully knowledgeable of what it does, but https://github.com/zaphar/git2-rs/blob/add_pull_example/examples/pull.rs is a more complete implementation of what you have.


TRY INSTANCING!!

https://github.com/SaschaWillems/Vulkan/blob/master/examples/instancing/instancing.cpp


Handle Hubzilla Likes better

We don't currently translate Hubzilla Like activities to mf2 HTML very well. They have both object and inReplyTo, which arguably they shouldn't, but probably isn't technically wrong. We convert them to a combined reply and like as mf2, which webmention receivers like the WordPress plugin (cc @dshanske) interpret as a reply over a like. (Which is probably right!)
For AS2 Likes, we should prioritize the like and remove the reply when converting to AS1 and/or mf2.
Example: https://authorship.studio/activity/143b8b1d-0bf8-46ab-b0bf-065e689120cd . Pruned AS2:
{
   "type" : "Like"
   "id" : "https://authorship.studio/activity/143b8b1d-0bf8-46ab-b0bf-065e689120cd",
   "actor" : "https://authorship.studio/channel/scott",
   "inReplyTo" : "https://fed.brid.gy/r/https://snarfed.org/2024-04-07_52743",
   "cc" : ["https://authorship.studio/followers/scott"],
   "object" : {
      "type" : "Note",
      "id" : "https://fed.brid.gy/r/https://snarfed.org/2024-04-07_52743",
      "actor" : "https://fed.brid.gy/snarfed.org",
      "url" : "https://fed.brid.gy/r/https://snarfed.org/2024-04-07_52743"
      "attributedTo" : "https://fed.brid.gy/snarfed.org",
      "content" : "One of our cats purrs so loudly that it’s a running joke in our family. It makes her fur vibrate. We have to turn the TV volume up when she sits with us. Once, she was in another room, and I heard her purr through the wall. No joke.",
      "published" : "2024-04-07T16:07:49Z",
      "to" : ["https://www.w3.org/ns/activitystreams#Public"],
   },
   "published" : "2024-04-07T16:31:35Z",
   "to" : ["https://www.w3.org/ns/activitystreams#Public"],
}

HTML from https://ap.brid.gy/convert/web/https://authorship.studio/activity/143b8b1d-0bf8-46ab-b0bf-065e689120cd :

  https://ap.brid.gy/convert/web/https://authorship.studio/activity/143b8b1d-0bf8-46ab-b0bf-065e689120cd
  <time class="dt-published" datetime="2024-04-07T16:31:35+00:00">2024-04-07T16:31:35+00:00</time>

    <data class="p-uid" value="https://authorship.studio/channel/scott"></data>
    Scott M. Stolz
    scott

  Likes a post by
    <data class="p-uid" value="https://snarfed.org/"></data>
    snarfed.org

One of our cats purrs so loudly that it’s a running joke in our family. It makes her fur vibrate. We have to turn the TV volume up when she sits with us. Once, she was in another room, and I heard her purr through the wall. No joke.

  https://snarfed.org/2024-04-07_52743
  <time class="dt-published" datetime="2024-04-07T16:07:49+00:00">2024-04-07T16:07:49+00:00</time>

  snarfed.org/2024-04-07_527...

  One of our cats purrs so loudly that it’s a running joke in our family. It makes her fur vibrate. We have to turn the TV volume up when she sits with us. Once, she was in another room, and I heard her purr through the wall. No joke.

Parsed mf2 (pruned):
{
  "items": [
    {
      "type": ["h-entry"],
      "properties": {
        "uid": ["https://ap.brid.gy/convert/web/https://authorship.studio/activity/143b8b1d-0bf8-46ab-b0bf-065e689120cd"],
        "name": ["Likes a post by snarfed.org One of our cats purrs so loudly that it\u2019s a running joke in our family. It makes her fur vibrate. We have to turn the TV volume up when she sits with us. Once, she was in another room, and I heard her purr through the wall. No joke."],
        "in-reply-to": ["https://snarfed.org/2024-04-07_52743"],
        "published": ["2024-04-07T16:31:35+00:00"],
        "content": [{
            "html": "Likes a post by   \n  <data class=\"p-uid\" value=\"https://snarfed.org/\"></data>\n  snarfed.org\n  \n  \n  \nOne of our cats purrs so loudly that it’s a running joke in our family. It makes her fur vibrate. We have to turn the TV volume up when she sits with us. Once, she was in another room, and I heard her purr through the wall. No joke.",
            "value": "Likes a post by snarfed.org One of our cats purrs so loudly that it\u2019s a running joke in our family. It makes her fur vibrate. We have to turn the TV volume up when she sits with us. Once, she was in another room, and I heard her purr through the wall. No joke."
          }
        ],
        "like-of": [{
            "type": ["h-cite"],
            "properties": {
              "uid": ["https://snarfed.org/2024-04-07_52743"],
              "name": ["One of our cats purrs so loudly that it\u2019s a running joke in our family. It makes her fur vibrate. We have to turn the TV volume up when she sits with us. Once, she was in another room, and I heard her purr through the wall. No joke."],
              "url": ["https://snarfed.org/2024-04-07_52743"],
              "published": ["2024-04-07T16:07:49+00:00"],
              "content": [{
                  "html": "One of our cats purrs so loudly that it’s a running joke in our family. It makes her fur vibrate. We have to turn the TV volume up when she sits with us. Once, she was in another room, and I heard her purr through the wall. No joke.",
                  "value": "One of our cats purrs so loudly that it\u2019s a running joke in our family. It makes her fur vibrate. We have to turn the TV volume up when she sits with us. Once, she was in another room, and I heard her purr through the wall. No joke."
                }]
            },
            "value": "https://snarfed.org/2024-04-07_52743"
          }
        ]
      },
        "author": [{
            "type": ["h-card"],
            "properties": {
              "uid": ["https://authorship.studio/channel/scott"],
              "name": ["Scott M. Stolz"],
              "nickname": ["scott"],
              "url": ["https://authorship.studio/channel/scott"],
              "photo": [{
                  "value": "https://authorship.studio/photo/profile/l/4?rev=1701617226",
                }]
            },
            "value": "Scott M. Stolz"
          }],
    }
  ],
}

It's probably best to go by the "type" : "Like" here instead of prioritising one interpretation by available properties.

Good point!

Oh yeah, I remember now. The difficulty here is that mf2 itself is basically untyped, at least for all of the types we care about here. They're all h-entry, and they have like-of, repost-of, in-reply-to, etc properties, which we preserve and translate from the source AS2 regardless of its type.
So the brute force way to fix this is, for each AS2 type we'd strip all properties that don't apply to it. Eg for a Like, we'd strip inReplyTo. Doable, but it'd be verbose, error-prone, and kind of painful. Sigh.

I took the minimal approach here and just removed iin-reply-to from just likes, shares, etc.


Banking System Software

Prerequisites

[X] I am a GSSoC-22 Participant
[X] I checked to make sure that this issue has not already been filed

Description
I will be adding a C++ program inside c++ folder
The goal is to create a banking software or application, which could maintain data & provide a user-friendly interface for retrieving customer related details using OOPs concept in c++. The application facilitated the addition of details of new customers, getting basic detail of transactions like deposit, withdrawal.etc . It also has features as opening, modifying and deleting an account, password protection in the form of login through account number.
So please assign it to me.
Tech Stack
C++

Please assign it to me @abhijeet007rocks8 @heet-kumar


Identify strategy to support missing fields in the Submission Portal

During GSP workshop, several users wanted to know what to do when required fields are not available / missing / confidential.
User need - Medium
Feasibility - unknown
Implement Missing Fields values (discussed before)
Requires broader discussion. @turbomam stated that supporting missing fields is problematic for non-text fields.

+1 by Champions during beta testing of submission portal

Blocked by GSC decision: https://github.com/microbiomedata/issues/discussions/743

Note, go/no-go date.
IF GSC has not resolved https://github.com/GenomicsStandardsConsortium/mixs/discussions/828 by Nov 15th, we will begin a squad to work on our own solution.

Nov 15th is passed.  @mslarae13 how do you want to handle this issue?


Privsep overhead: document, warn, shortcut when using -u root

Privsep operations used within per-connection code (such as when using -i on Linux, #221) adds a serious bottleneck and considerable latency to connection handling.

[x] Add information on which configuration options add per-connection privsep overhead to manual page
[x] As an initial mitigation, in debug mode, warn about configurations that cause per-connection privsep overhead
[x] Add option to shortcut privsep code entirely, i.e. when using -u root, use a fast path that bypasses the client/server socket machinery
[ ] Refactor socket handling to transport only a master socket across the fork, and then request client sockets dynamically as needed later - OR - centralize information about how many sockets will be needed in a nicer way

Implemented and tested, closing issue.


Problem with jupyter distance_to_object Notebook Intel Real Sense

Hi, I'm trying to run this example with the guide https://github.com/IntelRealSense/librealsense/blob/jupyter/notebooks/distance_to_object.ipynb
Perfectly arrive in In [4] and Out [4]
but when arriving at the lines In [5]:
colorizer = rs.colorizer ()
colorized_depth = np.asanyarray (colorizer.colorize (depth_frame) .get_data ())
plt.imshow (colorized_depth)
throws me an error: np.asanyarray (colorizer.colorize (depth_frame) .get_data ()) RuntimeError: null pointer passed for argument "frame"
I'm stuck there and I don't know what to do, maybe you can help me?
@dorodnic i need your help pleaseee

Required Info

Camera Model
{ D435 }

Firmware Version
(Open RealSense Viewer

Operating System & Version
(Ubuntu 16

Kernel Version (Linux Only)
4.14.13)

Platform
UPboard.

SDK Version

Language
python }

Segment
{Robot/Smartphone/VR/AR/others }

Hello @dorodnic , you may have some idea how to solve it, I tried to config.enable_stream and run pipeline with parameter config but I still get the same error.

I couldn't reproduce your issue, but I just opened #4787 to try and fix some other issues. Could you see if the following binder link to my fork works for you?

HI @candrescn,
Did you try the update from @lramati and see how it works?

HI @candrescn,
I will close this if no additional question.


⚠ Bofrost.at has degraded performance

In 64b6789, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2054 ms

Resolved: Bofrost.at performance has improved in 656790d after 310 days, 17 hours, 19 minutes.


Address-book

Description
Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.
Fixes #829

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[x] I have added the project meta data in the PR template.
[ ] I have created the requirements.txt file if needed.

Project Metadata
If there is no-file/nothing to fill the below fields with, then type: none
Example: If no requirements.txt needed/present, then type none in Requirments.
Category:

[ ] Calculators
[ ] AI/ML
[ ] Scrappers
[ ] Social_Media
[ ] PDF
[ ] Image_Processing
[ ] Video_Processing
[ ] Games
[ ] Networking
[ ] OS_Utilities
[ ] Automation
[ ] Cryptography
[ ] Computer_Vision
[ ] Fun
[x] Others

Title: Address-book
Folder: Address-book
Requirements: None
Script: addressbook.py
Arguments: none
Contributor: sameeksharl
Description: This project is to create an Address book in which the user can add a new contact, edit and delete existing contact and view all the contacts.

@kaustubhgupta its deleted only if we click on 'view all' again after deleting right??
I didn't get this issue ? could u pls tell me again so that i can rectify ??

Could u pls check now??? @kaustubhgupta

Thank you for your response!!
On Sat, 22 May 2021, 8:44 am Kaustubh Gupta, @.***>
wrote:

@.**** approved this pull request.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/avinashkranjan/Amazing-Python-Scripts/pull/1081#pullrequestreview-666146985,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOIEAAISIZFQWV75DJFONBTTO4OR3ANCNFSM45HPL4UQ
.

@kaustubhgupta  @avinashkranjan could u pls merge the pr??
as @avinashkranjan also said LGTM

@kaustubhgupta @avinashkranjan could u pls merge the pr??
as @avinashkranjan also said LGTM

PA (avinash) is yet to review the PR. He will merge the PR soon or on the last date of GSSOC. Kindly have patience. You will get your points for PR.

Thanks for the info !!! not about points, just a verification .Wanted make sure it is reviewed coz some other project’s mentors are not verifying either its been a month . Sent from Mail for Windows 10 From: Kaustubh GuptaSent: 26 May 2021 14:55To: avinashkranjan/Amazing-Python-ScriptsCc: sameeksharl; AuthorSubject: Re: [avinashkranjan/Amazing-Python-Scripts] Address-book<EMAIL_ADDRESS>@avinashkranjan could u pls merge the pr??as @avinashkranjan also said LGTMPA (avinash) is yet to review the PR. He will merge the PR soon or on the last date of GSSOC. Kindly have patience. You will get your points for PR.—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.


Regression_1

Regression 1

[ ] Look at the data. Choose a feature, and plot its relationship with the target.

Check out the opacity argument with plotly graphs and dense points.

[ ] Use scikit-learn for linear regression with one feature. You can follow the 5-step process from Jake VanderPlas.

Perfect.

[ ] Define a function to make new predictions and explain the model coefficient.

How can you make this function more versatile? What if you want to know about another feature?

[ ] Organize and comment your code.

Good job. Put # comments right above the code they describe.

Stretch Goals

[ ] Do linear regression with two or more features.

Excellent!
Same as above, how can you make this function more versatile?


fix : change for opencv 4.10.0 resolve exception

Added OpenCV 4.8 or higher version installation guide for ROS2 env. Closing in commit 2204b2b

Added OpenCV 4.8 or higher version installation guide for ROS2 env. Closing in commit 2204b2b


Script Disables Other Normal Features?

Installing this script on Greasemonkey results in the disabling of the dotted menu next to Youtube video titles that typically lets me save them to Queue or Watch Later. It also prevents me from viewing new responses to my comments.

Try use Tampermonkey instead of Greasemonkey

Tampermonkey is not working either

Try out the latest version. Hopefully, it should be fixed for most popups.


Introduce "DatabaseTransformers" [DRAFT]

See https://github.com/packbackbooks/questions/pull/9523 for context.
This branch experiments with fixing different gotchas that seem to frequently pop up while running ZDT migrations.

I opened an issue here: https://github.com/Daursu/laravel-zero-downtime-migration/issues/43 in hopes that we can get rid of our fork. In the mean-time, I'm closing the PR b/c this shows up in my "Created" PRs list, and I'm trying to get better about pruning things that distract me :laughing:


Remove old/irrelevant note

There's a website for Clippy lints now.
But I think this basic information is not the concern of this crate within the current more mature Rust ecosystem.

Can you please elaborate on your reasoning for this change?
That bullet point links to rustc documentation, and has nothing to do with Clippy.

Closing is fine.
The sentence I proposed to remove from your contribution guidelines, I think is unclear. It refers to ‘lints’. The term ‘lints’ is usually used very differently in contribution guidelines, namely in relation to code QA requirements. But you seem to means this in a very particular sense of implementing custom lints for procedural macros (?). I don’t know why that comment would be important to take note of for maud contribution, even if I give it thought.
Clarifying could be useful.


Rearrange dashboard

I placed several div tags dynamically with add_widget API.
Each DIV tag has a close button along with one data table.
I want whenever close button is clicked, widget should be removed from dashboard which i successfully did but it leaved  blank space on the dashboard instead dashboard rearrange all DIV tags.

https://github.com/troolee/gridstack.js#remove_widgetel-detach_node
This should help.


Add cross namespace verification in Kubernetes CRD

What does this PR do?
It filters out ServersTransport and TLSOptions from namespaces that are
not in the current IngressRoute namespace and when the option
allowCrossNamespace is not set.
Here's a chart representing the support for each Ingress Route Types (irt):

irt
ServersTransport
TLSOptions

HTTP
X
X

TCP

X

UDP

Since TLSStore is not used, its behavior has not changed.
Motivation
Remove the access of resources from routes that should not access them.
Fixes #8415.
More

[X] Added/updated tests
[ ] Added/updated documentation

We should wait for PR #8431 to be merged, before merging this PR.


add shell script

A PR request should consist:
Issue Reference: #16
Changes proposed:

Added terminal commands for encoder and decoder
Added usage section to Readme.md

@mention: @Aniq55

@inishchith what is the use of .encoder.py.swp I think you shouldn't have added it. Can you please remove it from your commit? or better, simply delete it on your system, add the changes and send another commit. I'll squash and merge.

@Aniq55 couldn't find on local machine(even in hidden) . so had to delete if here  . i hope it's fine.


Add state saving callbacks for plugins

Add state saving callbacks for plugins

Done here: https://github.com/flutter/engine/pull/13280


A few comments

Hello,
I am using sleek for a few days now and I like your software really a lot.
Despite its 0.xx version code I'm impressed by the stability and the features it has.
However I have a few remarks but all of them are minor:

The shortcut Strg+b (to show/hide filter pane) is missing in the Shortcuts help

A shortcut to disable the filters is missing
Could you shorten the links in the tasks to improve readability
It would be nice, if you could add some other schemes to the URI-detection like e.g. file:// which would allow to link files
The tabindex could be improved sometimes. E.g. if you are in the filter line a tab doesn't bring you to the task pane
Generaly I would like to navigate through the tasks and edit them with my keyboard
Could you support hidden tasks (h:1) as other todo.txt apps do?
And the best would be, if you could support notes in tasks as the notes add on for todo.txt-cli (https://github.com/mgarrido/todo.txt-cli/tree/note/todo.actions.d) does.

Thanks for your good work and keep on with sleek.
BR
Stefan

@SteOtt Thank you for your feedback, it's highly appreciated. All your points are valid and while I will work on the minor ones soon, please let me add the note function to my backlog. It would be an interesting addition, but up until v1.0 I committed myself to only work on technical depths. If the v1.0 milestone is reached, its' features again.

The shortcut Strg+b (to show/hide filter pane) is missing in the Shortcuts help
A shortcut to disable the filters is missing
Could you shorten the links in the tasks to improve readability

These three will be included in the 0.9.0-Beta
Also the tabindex improvements will be part of the v0.9.x. The other points are material for 1.0 and beyond.

I adapted your suggestions to my backlog and hope you're fine with me closing this issue.

Hi @SteOtt , the hidden todos feature has been implemented in v0.9.8 and needs testing: https://github.com/ransome1/sleek/issues/76#issuecomment-808787906


nixos/zfs: ZED config module

Motivation for this change
Continue work from #64306
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@Infinisil Is it good ? I can squash the commits if it's ready to be merged

@Infinisil I squashed all into one commit


feat: RPC call to get pre funded accounts in Madara

Wallets need an RPC where they can fetch funded accounts on a devnet. It helps them easily fund new accounts for users to play with. Katana and starknet-devent already support this. It's predeployed_accounts for devnet/devnet-rs and /katana_predeployedAccounts for katana.
We need to do the same for Madara. We can call it madara_predeployedAccounts. We already log the pre funded accounts when the user starts the node in --dev mode. You can see it here. Ideally, we can move this information to constants or a JSON config and use it for both - logs and for this API.
BONUS: This isn't necessary, but ideally if we can fetch the balance of the native token for each of these accounts it would be a nice feature.

Would be interested to work on this issue :)
Still learning the codebase though

Awesome! Feel free to message on the Madara group if you any doubts. Assigned to you @Juul-Mc-Goa

Hey @Juul-Mc-Goa any progress on this ? Let us know if you need any help

@apoorvsadana Maybe I'll give it a try, before I do I need to know exactly what data is returned by madara_predeployedAccounts, all I see printed is this information

@Juul-Mc-Goa can you write to me on telegram for payment please.
@timothee_delabrouille


Update dependencies to reduce Docker image size for GitHub Codespaces

[x] Local Docker image build is currently 6.09 GB.
[x] GitHub Codespace builds and the demo notebooks run correctly.
After this pull request is merged, I will tag and release this version.

Version 0.3.1 is now tagged and released.


bunny cold start takes 2-3 times longer than stock discord

after noticicing a large increase in cold start launch time, i tested both stock discord and bunny (with safe mode enabled to prevent any plugins messing with the results) and found that stock discord averaged around 2.1 seconds for first cached render and 2.5 seconds TTI, while bunny averaged around 6.6 seconds for first cahced render and 6.9 seconds TTI.

I realise I have not provided enough details, if any more details are needed, let me know

Bunny starts slower (especially on Android) mostly because it has to initialize all JS modules in advance (which Discord does lazily). Without doing that, we wouldn't know which modules to patch or to use. I have been working on improving this by "caching" modules we need, and it has been available on https://github.com/pyoncord/Bunny/tree/dev, making start-up slightly faster after the first boot of the Discord version. There's nothing can be done for cold start, unfortunately.


[BUG] -

Testdeck Version
0.0.1
Testdeck Package

@testdeck/core

NodeJS Version

12.10.0

Mocha / Jasmine / Jest Version

Mocha: latest
@types/mocha: latest

Operating System

all

Actual Behaviour
Existing decorator logic no longer works with Typescript 3.8.3 and breaks all tests.

Initial analysis:

suite decorator function (as returned by makeSuiteFunction) will return the actual decorator which is then never called

I had changed decorator return result from function to () => {}

Casting the return type to any does the trick, however, this cannot be the solution here.
For the moment we consider this fixed but we might want to fix our typings.


Lage demo applikasjon

Hi!
During our work on integrating Fhi.HelseId into our code I made a basic demo app.
It has an Angular frontend and is based on dotnet new angular.
@robinbaxon has seen this demo app in action and helped us with how logout should be called.
I have just pushed this to my Github here: https://github.com/mmalvik/Fhi-HelseId-Demo
I do not know whether you want the demo app in this repository or a separate one.
Feel free to copy my demo app and use it any way you want 👍


Next Stack Club meeting:  tomorrow (Fri. April 5), usual 9AM PST time

Stack Club friends,
Our next Stack Club meeting will be tomorrow, Fri. April 5th, at 9AM PST. We don't have any specific walk-throughs.  We will start with brief statements from all participants as to what they've been working on, and whether there were any problems they encountered recently - and we will continue hacking for the rest of the time slot.  Still, if anyone has a notebook which they would like to share - we certainly can accommodate it!
See you tomorrow, on http://stanford.zoom.us/j/946390572
Phil, Greg & Alex

We will start in 5 min.  See you at the Stack Club zoom!  --Greg


Package gogs-0.11.19-1500937836.643c85e9.centos7.x86_64.rpm is not signed

CentOS7
[root@git ~]# yum update
Loaded plugins: fastestmirror
Loading mirror speeds from cached hostfile
 * base: mirror.daniel-jost.net
 * epel: mirror.wiuwiu.de
 * extras: mirror.daniel-jost.net
 * updates: mirror.daniel-jost.net
Resolving Dependencies
--> Running transaction check
---> Package gogs.x86_64 0:0.11.19-1500397851.dc10594.centos7 will be updated
---> Package gogs.x86_64 0:0.11.19-1500937836.643c85e9.centos7 will be an update
--> Finished Dependency Resolution

Dependencies Resolved

====================================================================================================================================
 Package                Arch                     Version                                               Repository              Size
====================================================================================================================================
Updating:
 gogs                   x86_64                   0.11.19-1500937836.643c85e9.centos7                   gogs                    24 M

Transaction Summary
====================================================================================================================================
Upgrade  1 Package

Total size: 24 M
Is this ok [y/d/N]: y
Downloading packages:

Package gogs-0.11.19-1500937836.643c85e9.centos7.x86_64.rpm is not signed
[root@git ~]#

Why is it not signed?

I am also getting this.

I also got this error. I replaced /etc/yum.repos.d/gogs.repo with the current version from packager.io (direct link to the .repo file), ran yum update again and it worked.
For your reference, my old version of gogs.repo looked like this:
[gogs]
name=Repository for pkgr/gogs application.
baseurl=https://rpm.packager.io/gh/pkgr/gogs/centos7/pkgr
enabled=1

mine looks like this
[gogs]
name=Repository for pkgr/gogs application.
baseurl=https://rpm.packager.io/gh/pkgr/gogs/centos7/pkgr
gpgcheck=1
enabled=1

current remote has gpgcheck=0
I know that I can disable gpgcheck but that's not the point.
Why isn't it signed?

Hi... thanks for the feedback... but we don't take any issues/requests about packager.io here.. because this repository does not have control over it, it is maintained by third-party. You may send an email to the admin of packager.io.


Add Sidekiq Scheduler.

Resolves #1079
Description
Add new gem sidekiq-scheduler.
Type of change

New feature (non-breaking change which adds functionality)

Can you provide instructions of to configure the sidekiq-scheduler?

Hey @jash-kothari this is great! Could you also add in the a config file for this that runs the ReminderDeadlineJob each day at midnight?

This looks good! What do you think, @seanmarcia ?
I saw some files without a blank line at the end, but I believe that's not a problem...

Looks good, thanks so much!


[5.1][test] Mark long collection tests as such

Cherry-picked from #26066.
The FlattenCollection.swift.gyb and LazyFilterCollection.swift.gyb tests are taking 1000+ seconds to execute, even with optimizations. It should be possible to shorten these, but meanwhile, mark them as long tests to speed up regular testing.
rdar://problem/52872766
rdar://problem/52873170

@swift-ci test


⚠ Bofrost.be has degraded performance

In 98721e3, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1682 ms

Resolved: Bofrost.be performance has improved in 9115a7b.


Fixed example for 'Custom validators'

added missing comma
added missing module - datetime

Thanks for finding that!


Make the addon independent of the project window configuration

Hello!
I've been using this addon recently and I want to thank @KoBeWi for this useful tool.
However, I found an unwanted behaviour that I think could be an obstacle for new users.
The problem is that is very difficult to use the addon when your project have the configuration for pixel art. When you work in this type of games, you use to change the strech mode to "viewport", use a very low resolution and disable the "resizable" option. Well, as the addon is an scene that you run, this settings are applied too. This can produce the following situation:
https://github.com/KoBeWi/Godot-Spritesheet-Generator/assets/93356841/1be61c16-1326-4148-81b4-fa3fcbe583f5
As you can see, with this settings it's impossible to see the rest of options showed in the panel, you can't even save your sprite sheet (which is the problem I deal the first time I used the addon). I know this is described in the README file, but I think this can be really annoying for your workflow in certain situations. If you are a new user who wants to use this addon because you found it in the asset library, you will need to change your settings every time or try to figure out how to set a different window configuration for the addon.
I don't mention the stand-alone version because many new users won't even know how to reach this GitHub repository.  I know it can seem like an unimportant problem and it focus on a small group of people, but it could be very valuable for the usability of this addon. My argument is that if it is possible to avoid problems for new users, then we should do it.
The way I solved this was the following:
func _ready():
	# Addon window configuration code
	get_window().size=Vector2(1100,700) #Or any other common resolution
	get_window().content_scale_mode=Window.CONTENT_SCALE_MODE_DISABLED
	get_window().unresizable=false

I used this code in the script attached to "SpriteSheetGenerator.tscn". I don't know if it is the best solution and it is very obvious, but I put it in case it helps.
Anyway, that's all. I would like to thank you again KoBeWi and also sorry for the bad english! :P

Hi, my plan right now is to switch to stand-alone completely (the progress is here). It's possible to upload a whole projects to assetlib, so that's probably what I'll do once I finish the rework. I'm not sure yet if I want to keep the addon version.

Oh, that sounds interesting. I honestly don't know what the difference is, but I hope this means you can have a different window setup for the tools you have in your projects. Thanks for your answer!


Is not getLRSObject a spec violation?

I am not sure when the change was made, but the extended vars added by getLRSObject do not work in a conformant LRS.  I do not understand the logic of grabbing every parameter from the querystring and adding them back to all LRS calls.  Our LRS correctly rejects calls to the get the State document, for example, because additional parameters used on the content launch line are not allowed in calls to State API.

Will look at this today, good catch.

Looks like this one from August:
https://github.com/adlnet/xAPIWrapper/commit/cee119319995d573f59c8d1174311c30af99341e#diff-1b2039be21a4454c52d6b497832996eb

I believe this is the current offending code in getLRSObject():
if (Object.keys(qsVars).length !== 0) {
lrs.extended = qsVars;
}


Use HTTP statuses for errors and add 422

I'm sorry but no new errors are being added to this package. The predefined ones are there as convenience. It's expected that users define their own bespoke errors as needed.

That's pretty clear. Thanks @asim. I close this PR.
BTW, this package should not be used in examples. This can be confusing for newcomers that can be tempted to use it everywhere in their projects and send PRs again :)


Godot physics RigidBody size is overriden by the physics engine

Godot version:
3.2.3
OS/device including version:
Windows 10 - GLES3
Issue description:
Rigid bodies after changing physic engine from default to godot physics don't save size on application/game start - giant boxes appear.
(Additionally move and slide with kinematic characters seams buggy - I can fly normal but when I walk on some surface, I get stuck sometimes (In my game it is critical error - because I need to use constant_linear_velocity but it is not supported in default physics engine))
Steps to reproduce:
Change physics engine to godot physics.
Make rigid body - with collision box and mesh instance.
add Camera and some surface so box won't escape camera field of view
run project
Minimal reproduction project:
Test.zip

Looknig at the forum post, I see @TwistedTwigleg said:

What is strange with this, in my opinion, is that jumping even works with this solution! I thought move_and_slide_with_snap made jumping hard because the snapping code would force the character to stick to the ground, but I guess this is not the case.

I would have expected this as well. Are these docs no longer accurate? https://docs.godotengine.org/en/stable/classes/class_kinematicbody.html#class-kinematicbody-method-move-and-slide-with-snap:

As long as the snap vector is in contact with the ground, the body will remain attached to the surface. This means you must disable snap in order to jump, for example. You can do this by setting snap to (0, 0, 0) or by using move_and_slide instead.

Erm I meant rigid body does not save size + My player character is kinematic body when I use move_and_slide without snap. After I changed from default("bullet") physics engine to "godot physics" physics engine

Sorry, commented on the wrong issue :bow:

ok, no problem @rcorre

There is a warning in the editor notifying that this will happen:

That been said, I'm not sure there is an open issue to address this in Godot physics; so I suggest keeping it open.

(Additionally move and slide with kinematic characters seams buggy - I can fly normal but when I walk on some surface, I get stuck sometimes (In my game it is critical error - because I need to use constant_linear_velocity but it is not supported in default physics engine))

Please only report one issue per issue report. Assuming it's not a duplicate of an existing issue, please open a separate issue for each issue you're experiencing.

I tested it with changing size of collision shape and that seams right - above.

Closing as duplicate of #5734.

Closing as duplicate of #5734.

https://github.com/godotengine/godot/issues/42973 👌


Should we keep examples for each resource in the README?

@jsmartt suggested and I totally agree that maybe we should keep only a few examples in the README and add links to the examples files.

Maybe we do not need to put detailed like it is right now. But, a little description would be interesting, mainly if there is some use case that is a bit different from the usual use case of the resources.
Regarding the README update, since it is a document that is not exactly directed to developers of this project itself, but for end-users, I think we just need to update it prior to releases, since the most use cases are available in the examples.

I think the README is actually the first thing any developer will look at when wanting to use this cookbook. I think we should document simple things there first, and point off to examples as a secondary resource for the complex, unusual, or special cases @tmiotto mentioned.
One advantage to this is that people skimming the cookbook can quickly see how it works, and some real code without having to navigate through an examples directory. Having no examples in the README might come off as either lazy or that this thing is really complicated, so we couldn't fit it all in this file.
Every time I use a new cookbook, the README is the first place I go, and it makes me very happy when I can get all the usage information right there in 1 spot. (The examples folder is an added bonus on top, for complicated examples).

Yeah... I think it would be good to have all the basic info in one place too.

Created #19 to track the update to the previous resources without examples.
Also, I suggest to require the README update as a PR acceptance criteria, what do you guys think?

I totally agree with this :+1:

closing because #19 now exists to track this


MSC2475: API versioning

Rendered

I'm still a bit unclear about what you're proposing here.
I think you're saying that https://github.com/matrix-org/matrix-doc/pull/2438#discussion_r388548634 is wrong (which I agree with: we should be using v1 there)?
some examples would help.

I'm not saying anything is wrong per-say, just that we need to figure this out.

gah, I put extra lines between the examples to avoid exactly that problem. Will give them titles I guess.

@turt2live You could also do:

The spec currently has /v1/profile, /v1/get_event, and /v2/send_message.

An MSC to add a new endpoint, /delete_message is made. It would be /v1/delete_message because it's the first time we've seen it.

...perhaps.
Either way, this MSC looks pretty sane to me and @richvdh has agreed to it as well, so:
@mscbot fcp merge

... I'm not yet convinced myself that the MSC is the way to go, and I see no reason to rush this through.
@mscbot concern Needs more thinking time before we rush into things

@mscbot fcp cancel
This needs to be reviewed for accuracy and relevance (by me).

@mscbot fcp cancel
This needs to be reviewed for accuracy and relevance (by me).

This MSC is no longer relevant.


How to get KB ID and simplifying the publish script

Hello Running the code I had some issue
Here is alternative simplified code to publish a kb.
`
import http.client, os, urllib.parse, json, time, sys

# Represents the various elements used to create HTTP request path for QnA Maker operations.
#Replace this with a valid subscription key.
# User host = '<your-resource-name>.cognitiveservices.azure.com'
host = '<your-resource-name>.cognitiveservices.azure.com'
subscription_key = '<QnA-Key>'
# you can use the below to dynamically get the kb id
kb_id = '<KB-ID>'
publish_kb_method = '/qnamaker/v4.0/knowledgebases/' + kb_id

try:
    headers = {
        'Ocp-Apim-Subscription-Key': subscription_key
    }

    conn = http.client.HTTPSConnection(host)
    conn.request ("POST", publish_kb_method, None, headers)

    response = conn.getresponse ()

    # Note status code 200 means success.
    print(response.status)

except :
    print ("Unexpected error:", sys.exc_info()[0])
    print ("Unexpected error:", sys.exc_info()[1])

`
Also please see the following how to get the kb id using python, in case you wanted to get he kb ids dynamically. note this is just getting 1 id, for more than one you will need to create an iteration.
`
import http.client, os, urllib.parse, json, time, sys

# Represents the various elements used to create HTTP request path for QnA Maker operations.
# Replace this with a valid subscription key.
# User host = '<your-resource-name>.cognitiveservices.azure.com'
host = '<your-resource-name>.cognitiveservices.azure.com'
subscription_key = '<QnA-Key>'
get_kb_method = '/qnamaker/v4.0/knowledgebases/'

try:
    headers = {
        'Ocp-Apim-Subscription-Key': subscription_key,
        'Content-Type': 'application/json'
    }

    conn = http.client.HTTPSConnection(host)
    conn.request ("GET", get_kb_method, None, headers)

    response = conn.getresponse()
    data = response.read().decode("UTF-8")
    result = None
    if len(data) > 0:
        result = json.loads(data)
        #print(json.dumps(result, sort_keys=True, indent=2))

    KB_id = result["knowledgebases"][0]["id"]
   # Note status code 200 means success.
   print(response.status)
   print(KB_id)

except :
    print ("Unexpected error:", sys.exc_info()[0])
    print ("Unexpected error:", sys.exc_info()[1])

`

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

ID: d9915bd5-e369-2db1-d0c2-95ba405cab49
Version Independent ID: 59c4a94a-ea00-4ef8-9554-fb4bfea416f3
Content: Quickstart: Create knowledge base - REST, Python - QnA Maker - Azure Cognitive Services
Content Source: articles/cognitive-services/QnAMaker/Quickstarts/create-new-kb-python.md
Service: cognitive-services
Sub-service: qna-maker
GitHub Login: @nerajput1607
Microsoft Alias: nerajput

@islamtg Thank you for your feedback . We will investigate and update the thread further.

thanks
Kind regards,
Islam El-Ghazali
Cloud Solution Architect (CSA) - Data & AI
Customer Success Unit (CSU)
[A close up of a logo  Description automatically generated][Linkedin]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.linkedin.com%2Fin%2Fislamelghazali&data=02|01|iselghaz%40microsoft.com|00e882d8135a45bef6e208d7d5976bcb|72f988bf86f141af91ab2d7cd011db47|1|0|637212718651166229&sdata=N6UsVb3EOTgFInD%2BAoWNuanzwUgWQIeXzZjTU1DL9pM%3D&reserved=0
Office: Remote
Work (Cell): +1<PHONE_NUMBER>
Email: iselghaz<EMAIL_ADDRESS>From: AmanpreetSingh-MSFT<EMAIL_ADDRESS>Sent: 28 February 2021 00:44
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Islam El-Ghazali<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] How to get KB ID and simplifying the publish script (#71365)
@islamtghttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fislamtg&data=04|01|iselghaz%40microsoft.com|fd0159b8e7a7474108ac08d8dbabe0e3|72f988bf86f141af91ab2d7cd011db47|1|0|637500878558234706|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=UQfOgEMtMsiE1nQ%2BnfQyiZj6PuRoVdW1u%2Fk9%2FQIZ7cQ%3D&reserved=0 Thank you for your feedback . We will investigate and update the thread further.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F71365%23issuecomment-787400290&data=04|01|iselghaz%40microsoft.com|fd0159b8e7a7474108ac08d8dbabe0e3|72f988bf86f141af91ab2d7cd011db47|1|0|637500878558244697|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=90g7gQSjpS6rxbnZ3M4%2FLXy8m7SCqAboCzRJ6X0juik%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAPSFDFEYPXV3FN6I476XDF3TBHJ2ZANCNFSM4YKSL3YA&data=04|01|iselghaz%40microsoft.com|fd0159b8e7a7474108ac08d8dbabe0e3|72f988bf86f141af91ab2d7cd011db47|1|0|637500878558244697|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=tBIJn0Um699eP2eSIHLiHLx0cZBVUopx%2Fu3tGWENulE%3D&reserved=0.

@islamtg Thanks for posting the helpful snippets. I have tagged our content team to review and add it to the documentation for REST calls if required. Similar snippets to manage a KB are available for the SDK in this document.
@nerajput1607 FYI.

#reassign:DishaAgarwal

Is this closed? @mrbullwinkle

We won't be doing this for now. Can we archive this?

We can close this out for now, but I will create an internal work-item and add it to the backlog with this issue for reference as potential improvements to the existing scripts/examples.  @islamtg  thanks for bringing this to our attention.

#please-close


Employee edits already created agreement objectives

Implemented:

Given I have an existing agreement
When I change the objectives on the agreement
Then the changes are saved on the agreement

Once again can be merged as user not yet taken to new index page after login.

Given I have an existing agreement
When I change the objectives on the agreement
Then the changes are saved on the agreement

open sesame


Images are not removed

Hello,
first I want to thank for this nice library, but I have one issue:
I have a list of images, which upon clicking on them I load into the image viewer. When loading the second image, it seems that the first one does not get removed. Having a look inot the debugger tools of the browser, it reveals that with every imageViewer.load a new image is added (I am only loading the image with one url, so no hi res image url present).
The images are added with the classes iv-image iv-small-image. Looking into the source code of iv-viewer in the method _loadImages, I can see that only the images with the following classes are removed from the dom iv-snap-image, iv-large-image. I guess thats the reason why the images stay there. Removing the snap image works as expected.
Any advice?
Thx
Markus

I will fix this. You can also raise a PR if you know the fix.

Hello,
thx for this. Well it depends: I don't know exactly how this iv-small-image is working in the new version as it seems that it was not there in the old (pre es6) one. There it seems also only the iv-large-image was there, even when no hi res image was loaded.
BR
Markus

Fixed in 2.0.0-beta5. Let me know if you find any other issue.

Works. Thanks for fixing this.
BR
MArkus