[Feature request]: Is there any possibility of seeing a gem that we can use in our rails or jekyll apps ?

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!
svcho, @GETandSELECT, @bmgmonteiro, @antx-code, @yangwao, @hackfisher, @LmingXie, @wadang, @AAinslie, @gracelee8, @DomHudson, @qicny, @edno, @ohtasoji, @djakaitis, @jiangying000, @kael4n, @wenlilearn, @dwearden, @nous-, @ast3150, @zhuwenger, @7Hd, @halis, @EXO-AL, @Thibaut120094, @leechor, @brunoocasali, @alexisweill, @YazaiHu, @tompham12345, @ucwong, @macoymejia, @BRKalow, @matteofigus, @adotsloan, @samsonq, @ibitebyt3s, @tensaix2j, @debragail, @halitogunc, @FreedomBen, @bellflower2015, @loongmxbt, @z7737546436, @clarknelson, @ghostbar, @kyroskoh, @sinsunsan, @shea256

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!
deterralba, @hu9osaez, @Isacco-B, @JoKrk, @CookieMonster-alt, @ziwon, @baixiaoyu2997, @ChakibHi, @Hadeso-0, @Biku-220, @Allemeshov, @macaframalamama, @a12jmx, @pwnedzen, @vojin7, @AntiKnot, @moontools-night0wl, @trevormil, @swetankkk, @tina1998612, @jiin-yang, @ccolorado, @Houshawany, @kiki-zjq, @AdetunjiTejumade, @henryhan-lab, @zbndm, @anhhung04, @UMUTBAYGUT, @infodavid, @arttet, @kursataktas, @MKS-01, @SeanChao, @JoachimFrydenlund, @Shawn-stross, @M-progr, @didy-kpn, @jahanshahub, @agnosticlines, @alitonia, @Elvis-Dodti, @yxxyun, @bheema-bhx, @Meshugah, @FelipeNystrom, @tolga0, @TimmyKwesi, @nkrishang, @luvchurchill

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!
ogonzal87, @nihalmpatel, @icygual, @densy07, @TarekJor, @tamara10, @Klexur, @micromaomao, @skyeg3, @atf1206, @monuelo, @zzc1308, @ombran, @SkyScraperTwc, @iddan, @abhijpes, @mattmelwoll, @OmarMohamedDev, @iluwatar, @raosuj, @syegres, @chardy, @vinnaykulkarni, @JamesWuChina, @kmanna, @Gguishan, @adad184, @chadfaling, @jackson-sandland, @benharold, @bitome, @tamablockchain, @prasanthkuna, @AiolosLiu, @trangmaiq, @metaer, @ericgopak, @vineeth12345, @alloncheung, @ergatea, @gustavocoding, @cjdao, @rifqifatih, @AHL96, @ppxu, @rwielinga, @fooock, @NagyGeorgeScripps, @vishalk90, @Kyborg2011


I found out no skins doesn't work

Describe your bug here. Mind you, if you are modding, please try testing your bug in a clean version of the engine instead, this ensures the bug is actually caused by the engine itself. Also, if you're using an older version of JSE, please try the latest action build in the Actions tab or the latest release build. The issue has probably been fixed since that version. Oh yeah, and also be sure to check the pinned 'JS Engine: Known Issues' issue, to double check if your issue hasn't already been found!
nothing really to explain the image explains everything

Command Prompt/Terminal/Crash logs (if existing)
No response
Have you identified any steps to reproduce the bug? If so, please describe them below in as much detail as possible. Use images if possible.
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
I only added scripts
If you use 1.12.0 or earlier, did you have Optimized Chart Loading turned on?
None
Did you check for any similar issues to what you're reporting? Check, then come back here. If there is a similar issue, then do not report the issue, otherwise it will be marked as a duplicate.
Yes

is this the latest version/latest action build?

is this the latest version/latest action build?

no?

Latest commit closes this issue as completed.

Also forgot, when making #290 you basically made a duplicate of this issue so lol


Performance regression: GrpcUnary-1MB-GrpcCore

A performance regression has been detected for the following scenarios:

Scenario
Environment
Date
Old RPS
New RPS
Change
Deviation

GrpcUnary-1MB-GrpcCore
Linux, h2c, KestrelSockets
2019-10-19 09:52:20Z
236
230
-2.54 %
-2.54 σ

Before versions:
Microsoft.AspNetCore.App 5.0.0-alpha1.19472.2
Microsoft.NetCore.App 5.0.0-alpha1.19514.1
After versions:
Microsoft.AspNetCore.App 5.0.0-alpha1.19472.2
Microsoft.NetCore.App 5.0.0-alpha1.19515.17
Commits:
Microsoft.NetCore.App / Core FX
https://github.com/dotnet/corefx/compare/a434a52ae3f1175bc1cad8b1a02b0921ef3b1f55...a434a52ae3f1175bc1cad8b1a02b0921ef3b1f55
Microsoft.NetCore.App / Core CLR
https://github.com/dotnet/coreclr/compare/b415b57a15b0c6ba77e63df901823bb46b8aafda...535984200521b3e8dd1338bbc4d50b1617fb077a
Logs

@JamesNK ?

FYI @JamesNK the first thing to do is try to repro the regression by running the benchmark with the two sets of version detected. Also check on the charts that the regression is still active (not some noise)

I'm guessing this is a rogue test. The only difference between this (GrpcUnary-1MB-GrpcCore) and GrpcUnary-1MB-GrpcNetClient is the client. If there was really a regression then they both would have regressed.

Recovered


change clearTime in order to fix issue 3757

Fixes #3757
Changes proposed in this pull request:

datetime misses kind and ticks causing Equals() to fail later

Thank you for the pull request! Are you up for making the unit test for this fix?
That's easy! Just copy over another test case to this case (N3757.cs), replace the file texts that point to the issue, and add the code to test the issue, ensuring you use Assert. to count something as working or not. Some useful methods are:

Assert.True
Assert.AreEqual
Assert.Throws

You can search other test cases for how they are used.
Here's the last fixed issue's test case file you could base on: N3733.cs on Bridge tag 17.4.0
Looking forward to your test case!

Hi Fabrício,
Thank you for checking it. I will provide unit test.

Hi Fabrício,
I provided the test - it looks correct to me and it compiles. I followed Bridge.Test wiki page but was not able to run Batch3. It failed in JS with "Bridge is not defined" so I added bridge.js to index.html but then it complained about Bridge.ClientTest.

Hmm interesting, thanks for the feedback.
I have just tested a fresh clone of bridge repo and built Bridge.Dev.sln. It built successfully.
Then I scrolled down to the Tests folder, and set the Runner project as Startup Project (right click, and choose Set as StartUp Project).
Then voila. The tests page run when you build. You may build without debugging (in VS Debug > Start Without Debugging). I ensured to remove any local NuGet repositories so it pulled all from the web.
But you might of course want to route the generated Bridge Nuget packages to a local NuGet repo and cleanup/rebuild the project -- so that it grabs your built packages when linking the test projects.
To grab the built NuGet packages, just locate any .nupkg files within your repo after a successful build, then copy them over the local folder where you set your local NuGet repository; then next NuGet restore should pull them off.
You can use the 'find' tool if you have either cygwin or git bash shell. Here's what I get in the fresh repo after the first build:
find ./ -name '*.nupkg' -not -path './packages/*'
./Compiler/Build/bin/Release/Bridge.Clean.17.4.0.nupkg
./Compiler/Build/bin/Release/Bridge.Min.17.4.0.nupkg
./Compiler/Contract/bin/Release/Bridge.Contract.17.4.0.nupkg
./Compiler/Translator/bin/Release/Bridge.Compiler.17.4.0.nupkg
./PostBuild/bin/Release/Bridge.17.4.0.nupkg
./PostBuild/bin/Release/Bridge.Core.17.4.0.nupkg

Remember you need to remove the outdated copies of the packages within the packages/ directory besides replacing the files in the local nuget repo.
This way you will run all tests integrated. You can then set a filter to show just your test. If the linked bridge version is still old, you'd notice your test failing (as it should without the fix!).
Hope this helps and does not sound too much way overcomplicated, just tried to cover most possible issues. Let us know if you give this a try and could have the tests running altogether. Would be nice to get feedback to improve the process.

Fabrício,
thanks for explaining how to run those tests. Your comment would nicely fit into some kind of 'How to run bridge tests' section in readme.md :) Anyway - my test fails without Date.js fix. With fix applied it passes correctly (and all the other pass as well).

Thanks for the feedback, wouldn't make much sense publishing this step-thru if it didn't actually work for you. As we frequently do this here, it may look like obvious to us and I didn't even think there was nothing about how to run it. But it should indeed be written somewhere.
I've just added this wiki entry for running the tests. I believe it should be pretty easy to follow and cover some pitfalls for people developing Bridge too. Suggestions, as always, are welcome!
Bridge wiki page: Build & run Bridge self Tests

Just a reminder, this pull request can potentially fix issue #3773, so check it up when reviewing the pull request.

Fabrício,
Your wiki entry for building tests is detailed and helpful. I'm convinced that since now it will be clear for anybody on how to run tests. Thank you!


Fix Graph if there are no files

In the Webview, if there are no files, the graph.
Steps to reproduce the behavior:

Go to an empty directory
Open File Makeup Viewer

Instead, It should display something like "There are no files that match your query"

Hi, I want to make a little contribution. I've fix this bug, but I don't know if the fix is inline with the desired behavior, so I record a little demo, if you think it's good, then I can make a pull request. I didn't implement FileDeleteEvent from VS Code API for the solution, and only use basic condition checking.
https://github.com/user-attachments/assets/56d53233-de08-4359-a0b1-ca829ddbbf56

This looks great! If you could also hide the dropdown menu if there are no directories, that would be great, so it only has the Header and the "No files were found" message. Thank you very much!


Remove one region to make navigation for blind users mor efficient, make audio notifications work always and everywhere ;-)

Hi,
I am a blind user of Mattermost and I am working with Linux and the Orca screen reader. I was using Mattermost with the messaging software Pidgin and the purple plugin for Mattermost, because my first steps with the Mattermost desktop app or with the browser version were not that satisfactorily. To be honest navigating the different regions and doing simple tasks like selecting a channel, searching a user and so on are far less effizient then using Pidgin with the purple plugin. This is very sad, because I'd really like to use Mattermost via the Desktop app or in the browser. Pidgin and the purple plugin has its limitations and some Mattermost features are just not implemented and can not be used with this setup.
So today I gave the newest desktop app version 5.2.2-rc2 another try to see if things have improved. Also I invested more time to understand how things are working with my screen reading software.
The first impression is that efficiency is still a problem IMHO. Switching throug all regions with F6 or shift F6 is much slower then switching windows with Alt + Tab or Strg + Tab in a native application for example. Also picking the right channel and searching for users to write direct messages took me some time to understand because it was not very intuitive :-(. On the other hand understand how the app behaves might get better if I use the app more ofthen...
But there are some more basic problems that need to be resolved first IMHO that would help me a lot using the app:

I still see unlabeled buttons and grafics inside the app and the browser version of our Mattermost installation, currently version Mattermost enterprise edition 7.5. I do not know if those buttons do come from a installed module or are located in the default app, however such things should be fixed.
I've configured desktop notifications to get an acoustical signal for new messages. Unfortunatly this signal is not played every time. For example if I have focused a channel and new messages arive, no signal is played. Also if I am not in the app and do works outside the app I see new unread messages when returning into the app, but no signal was played. Having propper acoustical signals for new messages always and everywhere is esential IMHO for blind users, at least if they have been activated and are wanted.
When navigating through the regions with F6 an empty region is read by my screen reader. My software is telling me something like "login successfull complementary input region". Wht is this region and can it be dimmed or is it necessary to be also read out when cycling through the regions?

In conclusion the Mattermost desktop app is accessible, yes, but working with the app is not that efficent like working with some other native messaging apps, e.g. pidgin. This is not related to Mattermost itself, its related to browserbased or electron apps, they are always not that easy to use and not that efficient, no matter which screen reader is used and on which operating system this apps are executed. However, if the basic things are working, e.g. that acoustical signals are played everywhen and everywhere, and if searching for users and channels could be made more easy, I'd use the app.
How can I help to improve the app and make things more accessible? Is it OK to open issues on github or are there persons who are working for Mattermost who can be contacted for accessibility related issues?
Cheers and thanks,
Scoepp

Hi @sonor3000,
I'm Laney, Senior Product Manager for the Suite Users Team here at Mattermost. Our team has been working hard on accessibility and screen reader improvements recently. We appreciate the feedback you've provided.
If you're available and open to it, we're also conducting a usability study to learn more about how to make the user experience friendly for people with a diverse range of abilities. We're trying to make Mattermost the best it can be, so your honest thoughts and experience would be valuable to us in carrying out these sessions.
What will I be doing in a usability study?
We'll start the session by asking a few questions about you, your background, and your experience with Mattermost. Then we'll ask you to share your screen and complete a series of tasks using a testing environment. Finally, we'll gather general feedback about your experience once the study has been completed.
How long is a session?
One hour
When and where does the study take place?
To participate, please sign up for a time slot using the calendar link below:
Click here to Signup for the Accessibility Usability Testing
If you prefer, you can also forward me some times and dates that work for you, along with your email address, and I can send you a calendar invite.
If you have any questions, please feel free to reach out to<EMAIL_ADDRESS>We appreciate your time and look forward to learning from you.
Best,
Suite Users Team @ Mattermost

Hi @laneycs
thank you very much for your reply and sorry for the late followup. There was less time during the last weeks and I wanted also to use the Mattermost app a bit more to get more expirienced with it and to see if the problems I have are related to the app, my setup or just because the app is new to me.

If you're available and open to it, we're also conducting a usability study to learn more about how to make the user experience friendly for people with a diverse range of abilities. We're trying to make Mattermost the best it can be, so your honest thoughts and experience would be valuable to us in carrying out these sessions.

Yes, we can make such a session in January after holiday. I'll get back to you and we will find a good time slot.
Now, after working a bit more with the current stable release of the Mattermost app, some things are more clear to me. I am able to navigate the different regions, write to channels, search and select users for direct messages and so on. Allthough I still think this is not that efficient then using a native chat app like Pidgin on Linux I am able to do the things I need to do with the app.
Unfortunately for one big thing I've still found no solution, and this is indeed a showstopper for me: The sound  / desktop notification for new messages is not always played. That means I am not aware if new messages arive when I have not focused the app or when I am using other programs or working in the textbased console, which is the case most of the time. The problem might be related to my Linux setup where I am working in the pure textterminal most of the time and just using the graphical environment only if necessary, e.g. if I want to use the Mattermost app, but on the other hand Pidgin is playing sound notification perfectly no matter if I am using the terminal, the pure real textbased console or a graphical program.
Maybe I have to use the Mattermost app e.g. on Windows or Mac to see if the problem with the sound notifications is also a problem on other operating systems or if it is really releated to my very uncommon Linux setup.
Cheers and have a nice holidy time!
Chris

This one unfortunately got lost - I'm going to move to the main repo since there's some feedback for the whole system in here.


Oldest Supported Rust Compiler Version?

Hi, what is the oldest version of the rust compiler that is supported by this project?

cargo-msrv after updating to the latest versions of all dependencies gives me 1.53.0.


To support Retain Message Feature

Retain Message are also a feature of the MQTT 3.1.1 protocol that is not supported in our current release and we plan to discuss implementation solutions for this protocol feature.

CouId I fix this ?

CouId I fix this ?

yes, welcome ping @ShannonDing


Fix filenames to prevent error on git checkout on windows

Description
Currently json files in test_utils contains character : which makes git on windows complain about invalid file name. We get below error:
cannot checkout to invalid path 'src/test_utils/hash_to_curve_vectors/BLS12381G1_XMD:SHA-256_SSWU_NU_.json'; class=Checkout (20)
This error happens because : is reserved character on NTFS file system.
What does this PR do?
This PR is renaming the problematic filename to be NTFS compatible and consequently solving the error. It also resolves some imports that prevent the testing of the changes.

@claravanstaden Can you please review it?


Ignore unresolvable configuration instead of segfaulting

This is a suggestion rather than fixing a bug.
Currently, chimera raises a segfault when it found an unresolvable declaration in the configuration file. However, sometimes a chimera project could have different configurations depending on the platform, and the project could see a configuration that is not invalid on some platforms.
This can be resolved by the chimera project (e.g., using different configurations per platform), but I think it would be still safe to ignore unresolvable declarations in the configuration file with warnings.

This is a topic that @mkoval and I debated quite a bit.
At the core of the issue is that if you are not very diligent, this is a very easy way for small typos in your configuration to go undetected.  Chimera will simply ignore the definition and move on with its default behavior.  Early in development this led to us accidentally missing entire modules worth of configuration.
On the other hand, if we are selectively compiling modules, it is also very inconvenient to run a preprocessor stage on the YAML file to dynamically control what modules are present or absent.  This can itself introduce errors by incorrectly generating configuration in the first place.
At the end of the day, this is probably a situation where --although I really hate to do it-- we should have a flag to switch the desired behavior between warnings and errors.  Since we are using clang, I wonder if there is an existing -W flag that we can co-opt.

At the end of the day, this is probably a situation where --although I really hate to do it-- we should have a flag to switch the desired behavior between warnings and errors. Since we are using clang, I wonder if there is an existing -W flag that we can co-opt.

Maybe we can specify this policy in the configuration file?

Yes, perhaps we can add a strict config and matching command line option.

Sounds good to me!

Yes, perhaps we can add a strict config and matching command line option.

Added command line option -strict, but not for config yet.
What would be the best location for strcit config? I found arguments in some example configs something like:
arguments:
  - "-extra-arg"
  - "-I/usr/lib/clang/3.6/include"

I guess adding -strict to the arguments tag should work, but I wasn't able to find code does the job.

@jslee02 I think you want to implement a block like the following (this is how bindingName is resolved from multiple locations):
https://github.com/personalrobotics/chimera/blob/887dbbc7f30491933f3b004eca136db15677c738/src/configuration.cpp#L407

I think you want to implement a block like this in the CompiledConfiguration constructor (this is how bindingName is resolved from multiple locations):

Sounds good to me. Do you think we should implement for parsing arguments tag, which I believe it's not implemented, and parse -strict from there? Or we create a new root tag something like following?
options:
  strict: True
  # add more options

I think we should use a new root tag, the arguments are really for clang
options, not chimera ones.
On Fri, Apr 3, 2020, 22:32 Jeongseok Lee<EMAIL_ADDRESS>wrote:

I think you want to implement a block like this in the
CompiledConfiguration constructor (this is how bindingName is resolved
from multiple locations):
Sounds good to me. Do you think we should implement for parsing arguments
tag, which I believe it's not implemented, and parse -strict from there?
Or we create a new root tag something like following?
options:
strict: True
add more options
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/personalrobotics/chimera/pull/225#issuecomment-608959115,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVM4K6QXVAFEXAIZNSTE6LRK2L43ANCNFSM4F3NM44A
.

I think we should use a new root tag, the arguments are really for clang options, not chimera ones.

Done in https://github.com/personalrobotics/chimera/pull/225/commits/2640cea5600bcb025e2c8d03e6c9725b7eb61420

bump :smile:


Loopback broken: Packets sent locally to loopback address are now dropped

Previously, packets sent locally to the the loopback address <IP_ADDRESS> were returned to the sender (as expected).
The commit 68ef8f06a5082ee3baa4128c4fe8e84832f22a57 broke this behaviour, by adding an is_invalid_packet() function which results in dropping packets sent locally from the loopback address.
This issue breaks the WIP MQTT over sockets PR #5854, which relies on the loopback address for a socket server restart.
Possible solution: pass the is_loopback flag from the process_data() function on to the net_conn_input() function into the is_invalid_packet() call so that the loopback packets are not dropped.

The commit 68ef8f0 broke this behaviour

This commit broke the behavior, because it was buggy. 4dd5867a79b8919c89e1f19544480446af51d6d8 fixed it. Can you please retest with the latest master?
If you look at the cumulative change, the check added makes a good sense, in particular, it triggers only if source and dest port numbers are the same - which should happen very rarely in the real life on its own, and never when src addr is also a local addr. If we with @rmstoi miss something there, we'd be glad to know.

@pfalcon, yes, verified, fixed with 4dd5867.
Thanks.


Release datastore 1.7.0

This pull request was generated using releasetool.
Closes #5680.

Not in this PR, no. Maybe for the next release.
On Wed, Jul 25, 2018 at 10:05 AM Tres Seaver<EMAIL_ADDRESS>wrote:

@tseaver commented on this pull request.
In datastore/CHANGELOG.md
https://github.com/GoogleCloudPlatform/google-cloud-python/pull/5683#discussion_r205188590
:

@@ -4,6 +4,25 @@

+## 1.7.0
+
+### Implementation Changes
+
+- Do not pass 'offset' once the query iterator has a cursor (#5503)
+
+### Documentation
+
+- minor fix to datastore example (#5452)
+- Add example showing explicit unicode for text values in entities. (#5263)
+
+### Internal / Testing Changes
+
+- Add Test runs for Python 3.7 and remove 3.4 (#5295)
Should we remove the classifier in setup.py as well?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-python/pull/5683#discussion_r205188590,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUcwjN_epPRZwa7SdjJC7EcmC0-S-Oks5uKKVSgaJpZM4VdFBS
.

CI failure due to subsequent merges to master.  We can ignore it, or else rebase.

@theacodes I'm not sure how the process works with merging this kind of PR and generating the appropriate tag.

@tseaver https://github.com/GoogleCloudPlatform/releasetool if you wanna give it a shot, you just need to run releasetool tag once this is merged.

@theacodes

you just need to run releasetool tag once this is merged.

I couldn't make that work:  no PR showed up in the list, and when I manually typed the PR, it crapped out:
> Please pick one of the following PRs:

Which one do you want to tag and release?: 5683
Traceback (most recent call last):
  File "../releasetool/venv/bin/releasetool", line 11, in <module>
    load_entry_point('gcp-releasetool', 'console_scripts', 'releasetool')()
  File "/home/tseaver/projects/agendaless/Google/src/releasetool/venv/lib/python3.6/site-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/home/tseaver/projects/agendaless/Google/src/releasetool/venv/lib/python3.6/site-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/home/tseaver/projects/agendaless/Google/src/releasetool/venv/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/tseaver/projects/agendaless/Google/src/releasetool/venv/lib/python3.6/site-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/tseaver/projects/agendaless/Google/src/releasetool/venv/lib/python3.6/site-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/home/tseaver/projects/agendaless/Google/src/releasetool/releasetool/main.py", line 54, in tag
    return releasetool.commands.tag.python.tag()
  File "/home/tseaver/projects/agendaless/Google/src/releasetool/releasetool/commands/tag/python.py", line 160, in tag
    determine_release_pr(ctx)
  File "/home/tseaver/projects/agendaless/Google/src/releasetool/releasetool/commands/tag/python.py", line 65, in determine_release_pr
    ctx.release_pr = pulls[pull_idx-1]
IndexError: list index out of range

So, I tagged datastore-1.7.0 and pushed it manually.


route

hello
tell me where you can read about configuring routed  (html5history)  need a client-side implementation, rather than the server

http://html5doctor.com/history-api/

This is understandable, but I was interested in another time ... as an example
https://www.npmjs.com/package/vue-route

@wormen I'll assume English is not a native language for you as I'm not understanding the question correctly, sorry :(

@Mat-Moo in what place it is not clear?

that's a decision that I needed to
http://michaeljcalkins.com/2015/07/getting-started-with-vuejs-creating-routes-with-vue-router/
http://vuejs.github.io/vue-router/installation.html

@wormen
There are people in the community who speak russian, so don't be afraid to duplicate your messages in russian. Although you should probably ask questions in vuejs/Discussion or vue gitter.

В комьюнити есть люди, разговаривающие по-русски, там что не стесняйтесь дублировать сообщения на русском. Тем не менее, вопросы лучше размещать в vuejs/Discussion или vue gitter.

@simplesmiler :+1:  thanks :)


Low entropy after enclave boot (reading from /dev/random blocks)

Hi,
I just faced the problem, a brand new enclave has zero entropy:
cat /proc/sys/kernel/random/entropy_avail
0

which causing block when reading from /dev/random early after boot.
What about adding some entropy from NSM RNG at enclave boot stage ?
(in nitro-cli run-enclave? )
(or at lower level:  I just submitted issue there - https://github.com/aws/aws-nitro-enclaves-sdk-bootstrap/issues/5)
Another question: which preferred workaround to use right now?

There a few possible solutions:

Call aws_nitro_enclaves_library_seed_entropy explicitly during application start, and on a timer afterwards.
Use RDSEED CPU instruction to seed entropy, rngd or similar can help with this.
Set random.trust_cpu=on in the kernel commandline (only works on Linux kernels > 4.19).

It might be possible to integrate calls to generate entropy and seed entropy into the provided init process of the enclave, but might not be the best option.

Hi @petreeftime , is there an easy way to call aws_nitro_enclaves_library_seed_entropy from the command line?

I attempted to use solution 1 to supply the system with entropy but it doesn't have the expected effect. Here's what I did:

Run a simple executable that calls aws_nitro_enclaves_library_seed_entropy(1024) periodically.
Look at /proc/sys/kernel/random/entropy_avail, which is still 0.
Start a separate application that reads from /dev/urandom; the read call blocks.

What am I missing here? For what it's worth, I'm running my enclave in debug mode (in case that makes a difference) and here are the last four log messages from startup:
[    1.024374] nsm: loading out-of-tree module taints kernel.
[    1.025030] nsm: module verification failed: signature and/or required key missing - tainting kernel
[    1.033770] random: entropy-init: uninitialized urandom read (32 bytes read)
[    1.034594] random: entropy-init: uninitialized urandom read (32 bytes read)


Bump web dependencies

Bootstrap was bumped from 3.3.7 to 4.4.1;

Alongside with this we got basically a new, fresh admin panel :^)

JQuery was bumped from 2.1.4 to 3.5.0 - THIS IS IMPORTANT: JQuery v2.x no longer receives updates.

Looks great now!


[Fix][UI]: fix re-login problem in new tab

Tips

Thanks very much for contributing to Apache DolphinScheduler.
Please review https://dolphinscheduler.apache.org/en-us/community/index.html before opening a pull request.

What is the purpose of the pull request
fix re-login problem in new tab
Brief change log
fix re-login conditional logic in dolphinscheduler-ui/src/js/module/io/index.js
the new re-login condition is (!sIdCookie || (sessionId && sessionId !== sIdCookie)) && requstUrl !== 'login'
assume requstUrl !== 'login':

sIdCookie is undefined or null: require re-login
sessionId && sessionId !== sIdCookie: require re-login when sessionId changed

attention that sessionId may be null but sIdCookie is non-empty

Verify this pull request
(Please pick either of the following options)
This change added tests and can be verified as follows:
Before fix:

After fix:

Codecov Report

Merging #4162 (06fbccb) into dev (62c2e85) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #4162      +/-   ##
============================================
- Coverage     41.37%   41.33%   -0.04%
+ Complexity     3115     3111       -4
============================================
  Files           467      467
  Lines         22168    22168
  Branches       2720     2720
============================================
- Hits           9172     9164       -8
- Misses        12113    12122       +9
+ Partials        883      882       -1

Impacted Files
Coverage Δ
Complexity Δ

...inscheduler/service/zk/CuratorZookeeperClient.java
56.09% <0.00%> (-7.32%)
6.00% <0.00%> (-1.00%)

...e/dolphinscheduler/remote/NettyRemotingClient.java
50.00% <0.00%> (-2.78%)
9.00% <0.00%> (-2.00%)

...dolphinscheduler/remote/future/ResponseFuture.java
81.35% <0.00%> (-1.70%)
18.00% <0.00%> (-1.00%)

Continue to review full report at Codecov.

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

In the past, the same browser can open multiple tabs, but there will be a problem. For example, the task of a certain tab is changed, and the task of another tab will not be updated synchronously. Is there a better way to support multiple tabs? And if a certain tab is changed, other tabs can be updated simultaneously

@break60 Thanks for reply.
The action to require re-login in new tab for logged-in user is not a good user interaction experience, so I submit this pr to solve the problem.
For the state synchronization in multiple tabs, this problem appears in most front-end projects. In essence, each tab is independent of each other, and http request is also stateless. To solve this problem, listening event of localStorage may be possible. Specifically,

Chage localStorage when some state changed in a certain tab
Add event listener for key storage in another tab

This means that you can trigger a page refresh, or whatever you want to do

The sample code is as follow:

The example effect is as follow:

The action to require re-login in new tab for logged-in user is not a good user interaction experience, so I submit this pr to solve the problem.
For the state synchronization in multiple tabs, this problem appears in most front-end projects. In essence, each tab is independent of each other, and http request is also stateless. To solve this problem, listening event of localStorage may be possible. Specifically,

Chage localStorage when some state changed in a certain tab

Add event listener for key storage in another tab

This means that you can trigger a page refresh, or whatever you want to do

The sample code is as follow:

Very good, can it be added to the current pr

I will add it tomorrow

this pr make a new problem!!!
when you edit dag node , update a sql and click ok(not save the dag)
switch tab to copy a sql, and switch back,
the sql change is lost !

this pr make a new problem!!!
when you edit dag node ,
eg:

update a sql and click ok(not save the dag)
switch tab to copy something
switch to ds dag edit tab
now open the node param ,found that the sql change is lost !

Can you create a new issue?

@geosmart @break60
This is a problem caused by state synchronization in new tab. For state synchronization, we will refresh page in new opened page.
If we don't do this, the issue the task of a certain tab is changed, and the task of another tab will not be updated synchronously referenced by @break60 will occur.

@geosmart @break60
This is a problem caused by state synchronization in new tab. For state synchronization, we will refresh page in new opened page.
If we don't do this, the issue the task of a certain tab is changed, and the task of another tab will not be updated synchronously referenced by @break60 will occur.

I know the visibilitychange can fix the sync problem
but the scenario of dag editor can not refresh to make the data loss
the dag design is usauly need to switch tab

This merged pr is based on visibilitychange


there is a SIGINIFICANT problem in the title of your README.md

the last word of your title "RestoreFormer++: Towards Real-World Blind Face Restoration from Undegraded Key-Value Paris", do you mean "Pairs"?

Thanks for your reminder. The typo has been revised.


Platform detection logic is flawed

"win" is also in "darwin", so the installation is requiring "pywin32" on macOS
https://github.com/Preston-Landers/concurrent-log-handler/blob/master/setup.py#L61
A safer way to detect the platform would be:
if sys.platform.startswith("win"):
    ...

Thanks for the PR!


Generate Assessment Plan from SSP

Is your feature request related to a problem? Please describe.
According to understood OSCAL model process flows, the SSP is the precursor model to the Assessment Plan. This issue is aligned with determining how to generate an Assessment Plan from the SSP. This potentially includes executing Lula Validations to determine contents of the Assessment Plan (and downstream Assessment Results?)
Describe the solution you'd like

Given an SSP
When lula generate -f <ssp.yaml> <flags>
Then an Assessment Plan is created

Describe alternatives you've considered
There may be another OSCAL model flow, this is possibly unclear? This issue conflicts with #518 where an Assessment Plan is documented as an output of a Component Definition. Which is true? Which do we want to go with if both could be true?
Additional context
Outstanding Question - do we need to run lula validate under the hood as part of this process? Relates to #518.
Additional point of discussion - How many commands do we want to have/users need to memorize to create OSCAL docs?
Expected Deliverable

[ ] Discussion and fully scoped issue set on the details of the implementation

Likely shares an overlap with #514 and #518.

Likely shares an overlap with https://github.com/defenseunicorns/lula/issues/514 and https://github.com/defenseunicorns/lula/issues/518.

Agree, called out #518 in the description but #514 is probably there as well - seems like the deliverable on this would either scope or close those issues as they relate to the specific assess command (https://github.com/defenseunicorns/lula/issues/514 - do we even want to create another command?) and the inputs as a component-definition (https://github.com/defenseunicorns/lula/issues/518 - should that be the input or should it be an SSP?)


[WIP] Extract ansible tower

the provider is for now at https://github.com/durandom/manageiq-providers-ansible_tower
before merging I have to point it to a proper manageiq repo
@miq-bot assign @Fryguy
@miq-bot add_labels wip, pluggable providers, providers/ansible_tower

The dependency on ansible_tower_client should probably move to the plugin as well.

@blomquisg What's your opinion on the timing of this move?  I can do it immediately, but it affects your team the most with the ongoing effort.

What's your opinion on the timing of this move?

Since we're moving a bunch of models around right now with respect to ConfigurationManager vs. AutomationManager it's probably easier to just to leave it in place for now until we get the modeling sorted.
Once that's out of the way, timing should matter less.


520 Error When Navigating to Database Section in NuxtHub Account

I'm encountering a 520 error when navigating to the database section in the NuxtHub account .
This issue occurs specifically when attempting to perform a database query via the Drizzle API.
Steps to reproduce the behavior:

Go to the NuxtHub account and select a project.
Navigate to the database section.
See the 520 error stacktrace.txt

Expected behavior
Navigating to the database section and performing a query should complete successfully without encountering a 520 error.

Can you confirm this happens all the time?

@atinux it happned earlier today, it seems fixed now


Crashing - idna.py - UnicodeError: label empty or too long

While scrolling down within the mitmproxy window using the arrow key it crashes. Repeatable multiple times when using mitmproxy to intercept the following site http://www.buddharadio.com.au
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/console/__init__.py", line 478, in run
    self.loop.run()
  File "/usr/local/lib/python2.7/dist-packages/urwid/main_loop.py", line 278, in run
    self._run()
  File "/usr/local/lib/python2.7/dist-packages/urwid/main_loop.py", line 375, in _run
    self.event_loop.run()
  File "/usr/local/lib/python2.7/dist-packages/urwid/main_loop.py", line 678, in run
    self._loop()
  File "/usr/local/lib/python2.7/dist-packages/urwid/main_loop.py", line 715, in _loop
    self._watch_files[fd]()
  File "/usr/local/lib/python2.7/dist-packages/urwid/raw_display.py", line 392, in <lambda>
    event_loop, callback, self.get_available_raw_input())
  File "/usr/local/lib/python2.7/dist-packages/urwid/raw_display.py", line 492, in parse_input
    callback(processed, processed_codes)
  File "/usr/local/lib/python2.7/dist-packages/urwid/main_loop.py", line 399, in _update
    self.process_input(keys)
  File "/usr/local/lib/python2.7/dist-packages/urwid/main_loop.py", line 496, in process_input
    event, button, col, row, focus=True ):
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/console/window.py", line 32, in mouse_event
    self.keypress(args[0], "down")
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/console/window.py", line 38, in keypress
    k = super(self.__class__, self).keypress(size, k)
  File "/usr/local/lib/python2.7/dist-packages/urwid/container.py", line 1128, in keypress
    return self.body.keypress( (maxcol, remaining), key )
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/console/flowlist.py", line 358, in keypress
    return urwid.ListBox.keypress(self, size, key)
  File "/usr/local/lib/python2.7/dist-packages/urwid/listbox.py", line 1001, in keypress
    return actual_key(self._keypress_down((maxcol, maxrow)))
  File "/usr/local/lib/python2.7/dist-packages/urwid/listbox.py", line 1119, in _keypress_down
    widget, pos = self.body.get_next(pos)
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/console/flowlist.py", line 260, in get_next
    f = ConnectionItem(self.master, self.state, f, False) if f else None
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/console/flowlist.py", line 104, in __init__
    w = self.get_text()
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/console/flowlist.py", line 111, in get_text
    hostheader = self.master.showhost
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/console/common.py", line 375, in format_flow
    req_url = f.request.pretty_url(hostheader=hostheader),
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/protocol/http.py", line 607, in pretty_url
    self.pretty_host(hostheader),
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/protocol/http.py", line 599, in pretty_host
    return host.encode("idna")
  File "/usr/lib/python2.7/encodings/idna.py", line 164, in encode
    result.append(ToASCII(label))
  File "/usr/lib/python2.7/encodings/idna.py", line 73, in ToASCII
    raise UnicodeError("label empty or too long")
UnicodeError: label empty or too long

mitmproxy has crashed!

Thanks for the report! This has been fixed in  7890450.


Auto grab MakeMKV app_key

NOT AN ISSUE
I've created a little script that will grab the latest app_key from the MakeMKV Forum and update the MakeMKV config file with it.
mmkvcode
All instructions needed are on that page / README file.
Feel free to try it out, Any issues let me know either here or create an issue on the repo 👍

Cool idea, but I really feel that if you are using the shareware features(Blu-ray ripping) of MakeMKV, you should be purchasing the registration key. You don't need a key for the freeware functions (DVD ripping)
The key in the MKV Forum is intended for beta testers of MakeMKV, not to extend the 30 day demo period indefinitely. If you are using the beta version with ARM and contributing feedback to MakeMKV then by all means use this script to make that process easier for you.

I don't think it's an issue to automate it while they provide a free key, but I'd rather see people buy a key if MakeMKV is useful.  I purchased a key and would encourage others to do the same.  (I would have used open source software for this piece, but I couldn't find anything even remotely comparable).


DOC: stats: complete references and links for descriptive stats

Reference issue
None. No code modification, only documentation.
What does this implement/fix?
This PR completes references and cross-links for descriptive stats functions.

p.s. please don't forget to add some skip instructions for the CI 😃 ([skip actions] [skip circle], we need Azure and CircleCI for doc PRs).

Sorry, but I don’t understand what changes I need to make to solve the current warning.

Ooops can you re commit without the skip circle? My bad I meant to do skip cirrus.

So we don't forget, I created an issue about the Sphinx warning: https://github.com/scipy/scipy/issues/17738


A very serious bug

if we use rublog clear, we will lost .git dir forever!!!
I try to find a way to protect the information of git......

Maybe we should mark rublog clear as a dangerous option


[HOLD][$1000] Web - Workspace - An error is visible when go back from Connect manually

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to URL https://staging.new.expensify.com/
Log in with any account
Go to Settings -> Workspaces -> Connect bank account -> Connect manually
Click "Back" NOT entering any data

Expected Result:
There should NOT be any error
Actual Result:
An error is visible
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/203867325-9ecc2079-5206-4ee0-b2dc-6a8cf69b1442.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~016000f297c530dd31
Upwork Job ID:<PHONE_NUMBER>529708032

Proposal
When clicking back button, the input loses focus and trigger the error validation.
https://github.com/Expensify/App/blob/39b2f9924060c7381b2ff37563a5af88360b0d9d/src/pages/ReimbursementAccount/BankAccountManualStep.js#L77
Solution
Add preventDefault when click back will do the trick.
-  onBackButtonPress={() => BankAccounts.setBankAccountSubStep(null)}
+ onBackButtonPress={(e) => { BankAccounts.setBankAccountSubStep(null); e.preventDefault(); }}

We can also add onLongPress event too incase we don't want the input to loose focus when long click.
It will also happen if we click other button like: Get Assistance, Close. We can also do the same.
Result:
https://user-images.githubusercontent.com/16502320/203968166-7c1a5c52-d4e8-442e-9957-9b76d1929c95.mov

This behaviour is present at every page that is using Form component. If we wanna fix this then we should be fixing this for all the pages.

Proposal
Problem
We're not preventing default in the HeaderWithBackButton component.
Solution
Following is a generic solution that works across all pages.
diff --git a/src/components/HeaderWithCloseButton.js b/src/components/HeaderWithCloseButton.js
index 27ab5dbbe..3a2a94336 100755
--- a/src/components/HeaderWithCloseButton.js
+++ b/src/components/HeaderWithCloseButton.js
@@ -202,7 +202,10 @@ class HeaderWithCloseButton extends Component {
                         && (
                         <Tooltip text={this.props.translate('common.close')}>
                             <Pressable
-                                onPress={this.props.onCloseButtonPress}
+                                onPress={(e) => {
+                                    e.preventDefault();
+                                    this.props.onCloseButtonPress(e);
+                                }}
                                 style={[styles.touchableButtonImage, styles.mr0]}
                                 accessibilityRole="button"
                                 accessibilityLabel={this.props.translate('common.close')}

Updated Proposal
Problem
The issue is that clicking on the x button triggers the onBlur handler of the Form input element. Since onBlur has a greater priority than onClick handler, onBlur gets called first. As such, adding an e.preventDefault in the onClick handler is of no use.
Fix
We need to not fire validation in the onBlur handler that is attached to the Form component given:

Events related target is clickable having a tab index of less than 1.

Thus the following changes:
diff --git a/src/components/Form.js b/src/components/Form.js
index 4734381bc..99b8abe3b 100644
--- a/src/components/Form.js
+++ b/src/components/Form.js
@@ -217,7 +217,10 @@ class Form extends React.Component {
                 ref: node => this.inputRefs[inputID] = node,
                 value: this.state.inputValues[inputID],
                 errorText: this.state.errors[inputID] || fieldErrorMessage,
-                onBlur: () => {
+                onBlur: (e) => {
+                    if ('tabIndex' in e.relatedTarget && e.relatedTarget.tabIndex < 1) {
+                        return e.preventDefault();
+                    }
                     this.setTouchedInput(inputID);
                     this.validate(this.state.inputValues);
                 },

Result

Proposal
We encountered a similar issue on https://github.com/Expensify/App/issues/12715 and fixed it by calling preventDefault on onMouseDown event. We can do a similar thing on back button. Since this issue is only occurring on back button. No need to change Form component I think.

@Delgee onMouseDown has a greater precedence than the onBlur event. We need to preventDefault in a handler that has a greater precedence than the onBlur event. As such, your solution works as well. But it's not generic. We'll have to do it on every Form with Back button. What I've purposed is generic and will work for all the forms.

@allroundexperts I think validating input on focus loss is correct behavior. And we should be able to validate input even if the focus is lost because of a click outside of Form element. So we should fix this issue on every page this issue happening instead of removing validation when clicked outside Form element. Also using e.relatedTarget.tabIndex is a bad idea tabIndex is just working on the specific cases it is not related to Form element at all.

@Delgee We are validating the input if the focus is lost because of click outside the Form. It's just for specific elements (like href), that when clicked, we don't want to validate. Also, TabIndex is just a single way to do it. As mentioned in proposal, we can use id / class as well to pinpoint this further.
Also, If we change this on every page, I'm sure this will come up again when new pages get added.

@allroundexperts Yes, we just don't want to validate in specific cases, not general cases. So we should fix this issue in those specific cases.

Proposal
Problem
As mentioned before onBlur is fired before onClick , hence the error shows when we click back button
Solution
We can handle mouse event for the form and only validate when mouse is inside the form , this will prevent any validation when we click outside the Form
diff --git a/src/components/Form.js b/src/components/Form.js
index 4734381bc9..6a47329fa9 100644
--- a/src/components/Form.js
+++ b/src/components/Form.js
@@ -75,6 +75,7 @@ class Form extends React.Component {
         this.state = {
             errors: {},
             inputValues: {},
+            focused: false,
         };

         this.inputRefs = {};
@@ -83,6 +84,7 @@ class Form extends React.Component {
         this.setTouchedInput = this.setTouchedInput.bind(this);
         this.validate = this.validate.bind(this);
         this.submit = this.submit.bind(this);
+        this.toggleFocus = this.toggleFocus.bind(this);
     }

     /**
@@ -128,6 +130,10 @@ class Form extends React.Component {
         this.props.onSubmit(this.state.inputValues);
     }

+    toggleFocus() {
+        this.setState(prev => ({focused: !prev.focused}));
+    }
+
     /**
      * @param {Object} values - An object containing the value of each inputID, e.g. {inputID1: value1, inputID2: value2}
      * @returns {Object} - An object containing the errors for each inputID, e.g. {inputID1: error1, inputID2: error2}
@@ -218,6 +224,9 @@ class Form extends React.Component {
                 value: this.state.inputValues[inputID],
                 errorText: this.state.errors[inputID] || fieldErrorMessage,
                 onBlur: () => {
+                    if (!this.state.focused) {
+                        return;
+                    }
                     this.setTouchedInput(inputID);
                     this.validate(this.state.inputValues);
                 },
@@ -249,6 +258,8 @@ class Form extends React.Component {
                     style={[styles.w100, styles.flex1]}
                     contentContainerStyle={styles.flexGrow1}
                     keyboardShouldPersistTaps="handled"
+                    onMouseEnter={this.toggleFocus}
+                    onMouseLeave={this.toggleFocus}
                 >
                     <View style={[this.props.style]}>
                         {this.childrenWrapperWithProps(this.props.children)}

@flodnv, can you please confirm if we wanna fix this issue

Thank you for the discussion everyone. For now let's hold on that discussion, see you there!

Commented in the linked thread. I think we should close this issue. Taking it off hold in any case because the discussion there is wrapping up.


Handle File Input selection dialog.

How would I handle the file selection on the .click the input element is created dynamically.  Should I add code handling to deal with a test injection?  Is there a way I can globally trap the file selection event and give it a file/files list?  There is more code in place for setting the input type filter, which it seems can only be set once.
const uploadInput = document.createElement('input');
uploadInput.type = 'file';
uploadInput.addEventListener('change', evt => {
  ...
});
uploadInput.click();

You can use elementHandle.uploadFile instead of clicking the input. The file dialog behavior is different across operating systems, so we have this more direct way of handling it.

So, I have to write a custom if/trap in my code to handle the test case.

@JoelEinbinder In my case, the elementHandle.uploadFile adds the path in the input box however, doesn't calls the services from backend.  Once i navigate to the file & select it, the content of the file is read & displayed on the UI.
I am facing issues while replicating this in puppeteer.
@tracker1 Did you find any solution/ workaround?

@parasvora TBH, I don't remember specifically...  IIRC I added a reference to an object and used that to call the .uploadFile instead of the click event... I'm no longer working on, or have access to the source, or would be able to give more info.

@parasvora can you please share a URL of the website so that we can take a look?


Add rust library links

Adding some links to steer people towards the rust libraries.

Thank you, @cholcombe973!

No problem :)


Feature Request: add support for climate devices

Hello,
I trying to install zigbee2mqtt addon but im still having some error. Its only for testing purpose!
Thanka

Hi, can you provide some more details? Are you able to get the docker logs from the plugin page (tab next to configuration) ?

To be honest, I am wondering what you are doing there. This addon is publishing the entities you already have in Home Assistant as a matter bridge to other matter clients (like Alexa).
why do you need zigbee2mqtt in there?
wouldn’t it be more useful to have zigbee2mqtt in home assistant, have the entities in there and publish them via HA to this plugin?

Hi,
I want to use the climate device also.
[15:38:05.112] [Matterbridge] Installing plugin matterbridge-zigbee2mqtt...
[15:38:05.118] [Matterbridge] Failed to start child process: spawn sudo ENOENT
[15:38:05.118] [Matterbridge] Error installing plugin matterbridge-zigbee2mqtt
[15:38:05.120] [Matterbridge] Child process stdio streams have closed
with code -2
[15:38:10.151] [Matterbridge] WebSocket client disconnected
[15:38:10.318] [Matterbridge] WebSocketServer client connected
[15:38:10.337] [Matterbridge] WebSocket received message => Connected
to WebSocket: ws://<IP_ADDRESS>:8284
[15:38:14.374] [Matterbridge] WebSocket client disconnected
Le lun. 10 juin 2024, 1 h 20 a.m., Tobias Glatthar @.***>
a écrit :

To be honest, I am wondering what you are doing there. This addon is
publishing the entities you already have in Home Assistant as a matter
bridge to other matter clients (like Alexa).
why do you need zigbee2mqtt in there?
wouldn’t it be more useful to have zigbee2mqtt in home assistant, have the
entities in there and publish them via HA to this plugin?
—
Reply to this email directly, view it on GitHub
https://github.com/t0bst4r/matterbridge-home-assistant-addon/issues/36#issuecomment-2157290640,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXKJDQFP5ETU7QTFAC4QDLTZGUZSBAVCNFSM6AAAAABJBHVIIWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJXGI4TANRUGA
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

Okay umm, i see. You are using the home assistant addon and you are trying to add the zigbee matterbridge plugin using the webui of matterbridge. Am i right?
The docker image I am building for this addon does not include any other plugin than mine. Even when the UI of matterbridge allows adding plugins which are already installed, it does not allow installing plugins which are missing.
i am not sure if I should add those matterbridge plugins to the addon. I should rather add support for the entity types of home assistant which are missing.
Since I don’t have any climate devices, I’d need help for that. What attributes does a entity has? What service calls are needed in home assistant?
as long as not all entity types are supported, unwould be fine adding the zigbee plugin (without any support from my side) . But i don’t think it’s a good solution.

Yes you are right. I'm trying to put some zigbee baseboard heater on Matter!
There is the attribute:
hvac_modes:

heat
min_temp: 5
max_temp: 30
target_temp_step: 0.5
current_temperature: 21
temperature: 5
hvac_action: idle
friendly_name: Z2M - T - Chambre Garçons
supported_features: 385

The only service need is the set target temperature.  To put them at off,
their need to be set at 5 degres.
There is the model: https://www.zigbee2mqtt.io/devices/HT402.html
Le lun. 10 juin 2024, 4 h 11 p.m., Tobias Glatthar @.***>
a écrit :

Okay umm, i see. You are using the home assistant addon and you are trying
to add the zigbee matterbridge plugin using the webui of matterbridge. Am i
right?
The docker image I am building for this addon does not include any other
plugin than mine. Even when the UI of matterbridge allows adding
plugins which are already installed, it does not allow installing
plugins which are missing.
i am not sure if I should add those matterbridge plugins to the addon. I
should rather add support for the entity types of home assistant which are
missing.
Since I don’t have any climate devices, I’d need help for that. What
attributes does a entity has? What service calls are needed in home
assistant?
as long as not all entity types are supported, unwould be fine adding the
zigbee plugin (without any support from my side) . But i don’t think it’s a
good solution.
—
Reply to this email directly, view it on GitHub
https://github.com/t0bst4r/matterbridge-home-assistant-addon/issues/36#issuecomment-2159198998,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXKJDQHQRWZWBCP6PVGTOMLZGYB6XAVCNFSM6AAAAABJBHVIIWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJZGE4TQOJZHA
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

Those are the attributes from home assistant right? So it is connected to HA? Just to make sure.
Can you post the detailed service call in yaml format of how you set the temperature in home assistant (e.g. in automations) ?
Could you provide a list of all service calls available for this kind of entity?
Could you provide the entity_id of one of those devices?

Yes. Those are connect directly to zigbee2mqtt.

service: climate.set_temperature
data_template:
temperature: 20
entity_id: climate.z2m_t_chambre_garcons

It's the only service call available for this model. It's also returning
the energy mesurement but I dont need it.

climate.z2m_t_chambre_garcons

Le lun. 10 juin 2024, 5 h 43 p.m., Tobias Glatthar @.***>
a écrit :

Those are the attributes from home assistant right? So it is
connected to HA? Just to make sure.
Can you post the detailed service call in yaml format of how you
set the temperature in home assistant (e.g. in automations) ?
Could you provide a list of all service calls available for this
kind of entity?
Could you provide the entity_id of one of those devices?

—
Reply to this email directly, view it on GitHub
https://github.com/t0bst4r/matterbridge-home-assistant-addon/issues/36#issuecomment-2159338190,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXKJDQFSS7MZGJAHOLRSF43ZGYMWPAVCNFSM6AAAAABJBHVIIWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJZGMZTQMJZGA
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

Thank you. I’ll give it a try in the next couple of days and then come back to you to test it 🙂


Popup spawned from EditorPlugin has size (0,0)

Godot version:
10cab25
Issue description:
Sometime between the build specified and one from about a week ago (sorry, don't know the exact commit!), the popup in my plugin now seems to spawn into the editor with an incorrect size.  It's launched nearly immediately after activating the plugin, using popup_centered().  No min size set, default rect size of (800,480).  Perhaps one of the recent tweaks to make sure certain UI elements don't fly out of bounds are responsible?

Steps to reproduce:
Make an EditorPlugin spawn a popup of a certain expected size.  (Toss some controls on it to see what's going on maybe)

duplicate? https://github.com/godotengine/godot/issues/18940

@AlexHolly they certainly seem like they could be related....


fix: accept 0% as tax rate

This is to make it possible to set 0% tax rate while creating a tax
to fix https://github.com/getlago/lago-go-client/issues/119

Thank you very much for this fix! It will be released with the next version, probably today.


[Feature] add auto merge background task

Issue
#297
We need to add an auto merge background task with cron expression scheduler.
Proposed Changes

add cron task for auto merge with cron library.
add related unit tests

Hi @roseduan, please take a look. Thanks. 😊
I have one problem when writing unit test for auto merge.
I don't know how to check whether merge action has been operated without an internal flag.
I think I should add such a unit test for case coverage.
Best Regards

Hi @roseduan, please take a look. Thanks. 😊
I have one problem when writing unit test for auto merge.
I don't know how to check whether merge action has been operated without an internal flag.
I think I should add such a unit test for case coverage.
Best Regards

Thanks for your PR.
What does 'internal flag' mean?
Merge operation will discard all invalid data, and rewrite all valid data into new data files.
So maybe you can check the valid keys, for example, you put 10 keys, and delete 5 keys, after merge, all keys num will be 5.

Hi @roseduan, please take a look. Thanks. 😊
I have one problem when writing unit test for auto merge.
I don't know how to check whether merge action has been operated without an internal flag.
I think I should add such a unit test for case coverage.
Best Regards

Thanks for your PR. What does 'internal flag' mean?
Merge operation will discard all invalid data, and rewrite all valid data into new data files. So maybe you can check the valid keys, for example, you put 10 keys, and delete 5 keys, after merge, all keys num will be 5.

Hi @roseduan , thanks very much for your warm suggestion.
I'm sorry for lack of explanation for 'internal flag'.
Previously I wanted to add a flag for merge actions and write unit test with it. But it makes no sense for production code.
I'll use keys' total count for unit test.
Happy New Year! 🎉
Best Regards

Hi @roseduan, please take a look. Thanks. 😊
I have one problem when writing unit test for auto merge.
I don't know how to check whether merge action has been operated without an internal flag.
I think I should add such a unit test for case coverage.
Best Regards

Thanks for your PR. What does 'internal flag' mean?
Merge operation will discard all invalid data, and rewrite all valid data into new data files. So maybe you can check the valid keys, for example, you put 10 keys, and delete 5 keys, after merge, all keys num will be 5.

Hi @roseduan , please take a look.
I have use datafile's key total count to make unit test.
It works now! Thanks very much for your suggestion.

Thanks, the CI job timeout is 10 minutes, please fix the unit tests.

Thanks, the CI job timeout is 10 minutes, please fix the unit tests.
There are some other failures

Hi @roseduan , sorry for forgot adding cleanup code for my unit tests.
I have ran this commit in my own repo's GitHub Actions.
Please take a look.
Thanks very much,
Best Regards

Thanks for your great work, looking forward to your next PR!

Thanks for your great work, looking forward to your next PR!

Thanks for your warm help.
I'm satisfied to make some contribution. 😊
Best Regards


fauxpasapp 0.1.3

Hi @mackoj ,
i think you shared a previous version of the step check out the file changes section header:
steps/fauxpasapp/0.1.1/step.yml
Also the changes are missing, check out the previous PR's conversation:
https://github.com/bitrise-io/bitrise-steplib/pull/405

I made a mistake and I forget to update the version number.

Hi @mackoj ,
can you please check again, what @viktorbenei wrote about is_always_run  and run_if  in this conversation?
Usually is_always_run : true and run_if : true is used by notification steps. If you set that inputs to true, your setp will run even if previous step (like git-clone) fails.

I have made this modification last Friday I will update this PR soon.

Il will re-share thanks


Feature support - parameterized scopes

This would be important to our project.  Would it be very hard to add? If not perhaps I will do a PR.
FYI the project is http://ruby-hyperloop.org  See issue hyper-model/issues/9

@catmando I think the problem with parameterized scopes might be the discovery of the parameters and default type casting of those parameters needed to exercise the scope. Although if the definition of the scopes were provided to the generator from the model then you wouldn't need to discover them, you would just need to have some way to validate them.
Alternatively, you could build your own argument list for the inputs for a query (or delete mutation) including your customized inputs arguments for parameterized scopes (not sure the argument list is currently setup correctly for the override on the master branch but it's only a couple lines of code to get it to work) and then override the resolve on the model with your own resolve to include use of your parameterized scope arguments.
Alternatively, I'm a few days away from providing a global resolve wrapper capability (along with some other nuggets on the relayimprovements branch including firming up the argument override) that may be useful for this purpose.
If you want to give it a whirl feel free to submit a PR. It may be an option on the default arguments for queries and delete mutations that could be used by others.

@catmando Took a look at hyperloop, some pretty cool stuff. Will have to carve out some time to take it on a test drive since my next direction was to research react with graphql. Looks like ya'll have already implemented to the nth degree some of the things I was interested in discovering.

@catmando I've done a little exploratory work with implementing scopes with parameters and here is what I've found.

I've been unable to find a proper arity for scope methods because of the way that they are assembled in rails.

https://github.com/rails/rails/blob/4cc1c14493e7a9bd188ee13c237366f6bc2e13f4/activerecord/lib/active_record/scoping/named.rb#L158
They all end up having a -1 arity to accept any number of parameters, even if defined as a lambda that should only accept a specific number of parameters. Even though the arity is -1 they then fail if not given the proper number of parameters. If you know of a way to retrieve the proper number of parameters from a scope method that would be helpful.

Since I'm unable to even determine the number of params at this point it's still a little loose, too loose for what I'd like to accomplish. I suppose I could tighten it up to the point where it would depend on the definition being sent, but without a way to verify the user definition it would still be loose.

I'm not giving up, just trying to find a way to tighten it a bit. It would be easier if the scope were defined as a class method instead of being wrapped in the scope declaration. Class methods have a specific arity, so then I would at least be able to verify the number of parameters defined.

In that branch, given a scope method on a model:
scope :find_lazy, -> (id) { where(id: id) }

and a graphql_query macro definition
graphql_query scope_methods: [:find_lazy]

you would be able to send a qraphql query to that model such as:
query{ model(scopes: [{scope: find_lazy, arguments: ["8"]}])
 {
    items {
      edges {
        node {
          item_id
        }
      }
    }
  }
}

if you attempted to send the wrong number of parameters to the query
query{ model(scopes: [{scope: find_lazy, arguments: ["8", "9"]}])

you would receive a GraphQL:ExecutionError in the form:
{
  "data": {
    "model": null
  },
  "errors": [
    {
      "message": "scope method: Model.find_lazy arguments: [\"8\", \"9\"] error: wrong number of arguments (2 for 1)",
      "locations": [
        {
          "line": 2,
          "column": 3
        }
      ],
      "path": [
        "model"
      ]
    }
  ]
}

since I'm not able to determine the number and types of arguments for the scope, right now the parameters are limited to string types that need to be coercible in the scope method. In this simple case, they are.

I've merged this capability into the master branch #26

Well you bit me too it... but I have been thinking hard with some other of the hyperloop core team, and one question has come up: how do you chain scopes?
I.e. some_user.todos.active.due_after('5/1/2018') etc.

since the query is not materialized until it is converted into an array until later in the process it is as simple as applying the scope methods iteratively on the query object.
The scopes are entered as an array to the query in the form [{scope:, arguments:},{:scope, :arguments}...]  and in the resolve that array is iterated and each scope is validated (to ensure the method is valid, even though it is constrained by the ScopeEnum), then applied.
https://github.com/geneeblack/graphql_model_mapper/blob/master/lib/graphql_model_mapper/resolve.rb#L28
The query is tested for correctness applying a limit(0) before it is sent down the resolve chain
https://github.com/geneeblack/graphql_model_mapper/blob/master/lib/graphql_model_mapper/resolve.rb#L125
where it is later materialized in its entirety by GraphQL

We had kind of different thought, that I wanted to run by you (let me know if there is a better way to discuss than adding stuff here...)
For each Model class the following Type would be generated:
type MyModelCollection {
  all: [MyModelCollection]
  scope(method: String, params: [String] = []): MyModelCollection
  count: Int
  # any other standard scope instance methods
  }

now i can say:
type MyOtherModel ... {
  my_models: MyModelCollection # i.e. in AR this was has_many :my_models
  ...
}

and in the graph
  MyOtherModel(id: 123) {
    my_models { scope('some_scope', 1, 2, 'blah') { scope('another_scope') { all } } ... }
  }

So instead of passing in your scope in the input arguments you are adding them as output selection elements (with arguments)? Would each of these scope elements then return a separate set of records? Or are you placing them in the selection to be used as filters for the  all output?
I think maybe what you are wanting is access to scopes in nested has_many relationships. i.e. to be able to filter the nested relationship based on the scope value available with the nested model. Would adding the input arguments to the has_many relationships/types get you there as well? Similar to the paging elements already available? (first,last,after,before)
I'd have to see how that would affect the resolve flow and the impact it would have, although right now the optional limits being set on the nested records is already having a similar effect on the nested relationship selection. I would be hesitant to add them as sibling elements since that would allow two different sets of records to be retrieved, one for the all element and one for scoped element. Unless that is not what you are intending. (your third example shows all being nested under scope, whereas your first example shows them as sibling elements?).

I think the above example would return something like this:
my_other_model { my_models { scope { scope {all: [.... the actual collection ...] } }  }

I've not tried this, its just a paper experiment, but it would seem to allow direct translation of some AR expression like MyOtherModel.find(123).my_models.some_scope.another_scope to a query.

This is roughly how we do it in hyperloop... although we are not using graphql (yet) the idea is the same.  Client passes a request tree to the server, and then server walks down the tree, generating the response.
WIthout more graphQL experience I don't know if what I am suggesting is practical.  It would require that the scope's resolver be able to return or save in some context the intermediate AR expressions, like we do in hyperloops tree walker.

The resolve tree does indeed pass an object context (query) which could be modified by applying scope, limit... further down the chain, essentially treating it as a separate query that already has its references set to the parent record. Currently it imposes a limit in this way for the child records. I don't think it would be too much effort to include the same selection criteria that is currently available in the parent root type  on the child collections (has_many references). Originally I hesitated to do that because of the n+1 query implications. But with paging information already being set I've already crossed that bridge and am doing n+1 queries to filter the child records by those limits.
Let me see if I can put something together, assuming that nested record filtering is the feature you are wanting.

hmmm... but if you can somehow just pass each AR scope result down as the context then there should not be n+1 queries.  In my simple example there would only be two AR queries executed:  find(123) and then my_models.some_scope.another_scope.all

Imposing the child limits forces a separate query to be made (unless those limits are imposed on the original collected array rather than being sent back to the server).
Imposing it on the original returned array has its own set of issues depending on the size of that array (which of course can't be determined until that array query is already materialized).  GraphQL does indeed use the resultant array to set its (first,last,before,after) inputs as well as it's pageInfo information. I opted to let the server do that work in my :flat representation (even though it doesn't do everything that GraphQL's Relay structure performs). GraphQL's relay structure pretty much requires it to work on the returned array since it needs to set some internal values to be filtered upon (cursor).
That being said, applying a scope would require revisiting the server since I don't think I'd be able to apply the scope to a previously returned array.
In your example (with one nesting) the scope would be applied to each subsequent child nesting for each parent. i.e. the reference passed to each child nesting is its relationship to the parent, child_parent_id = parent_id. Each scope would be a result of applying the scope to that reference..  child_parent_id = parent_id AND scope, so for each parent record there would be an n+1 query performed for its child relationships. Unless you think applying the scope can be done in a different way.
Otherwise, if you had those scopes set on different has_many references and they were already preset on the collections
has_many :all_items
has_many :scoped_items -> {where some condition}

then it would not take the additional query hit since those records would be materialized on the original query in the left join of those items

I guess what I am not being clear on mainly because I don't even know what is possible...
but if you look at those type definitions, the only field that is an array is all field.
Maybe this doesn't work the way I think it does in GraphQL?

Let me look into it more to determine exactly what the implications are, I may be being too skeptical at this point and only thinking about my :flat representation using lists. I'll have to see how GraphQL will treat it in it's GraphQL::Relay::ConnectionType class.
Assuming the base need is to impose filtering on child collections with the primary concern being able to apply scope to those child relations.

yeah, I think your current structure of passing the chain of scopes as a param to the outer field call is fine... I was just curious if you think what we came up would work.

well, I did get it working the way I was thinking about it, but it does cause n+1 queries. Let me stew on it a bit to see if I can approach it in a different way.

what I have right now is in the parameterized_scopes_2 branch, it's a bit messy but essentially it is applying the same selection availability on the nested has_many relationship as is available on the root item, i.e.
{
  post(item_ids: [37, 322]) {
    items {
      edges {
        node {
          item_id
          comments(scopes: {scope: has_content}) {
            items {
              edges {
                node {
                  item_id
                  content
                }
              }
            }
          }
        }
      }
    }
  }
}

The queries that emerge from that are:
  Post Load (0.6ms)  SELECT `posts`.* FROM `posts` WHERE (`posts`.`deleted_at` IS NULL) AND (`posts`.id in (37,322)) LIMIT 100
  Comment Load (0.5ms)  SELECT `comments`.* FROM `comments` WHERE `comments`.`job_id` IN (37, 322) AND (`comments`.`deleted_at` IS NULL)
  Comment Load (0.6ms)  SELECT `comments`.* FROM `comments` WHERE `comments`.`job_id` = 37 AND (`comments`.`deleted_at` IS NULL) AND (comments.content > '') LIMIT 10
  Comment Load (0.6ms)  SELECT `comments`.* FROM `comments` WHERE `comments`.`job_id` = 322 AND (`comments`.`deleted_at` IS NULL) AND (comments.content > '') LIMIT 10
  Completed 200 OK in 29.3ms (Views: 1.0ms | ActiveRecord: 2.7ms)

I need to trace back to where the original comments selection is being done correctly (as part of the included tables on the original query) and be able to apply the scope there rather than waiting until later to apply the scope.


Describe handling inside SARIF model for developers + change model to handle context situations

Situation
http://docs.oasis-open.org/sarif/sarif/v2.1.0/sarif-v2.1.0.html
describes properties and objects with following terminolgy:

The key words “MUST”, “MUST NOT”, “REQUIRED”, “SHALL”, “SHALL NOT”, “SHOULD”, “SHOULD NOT”, “RECOMMENDED”, “NOT RECOMMENDED”, “MAY”, and “OPTIONAL”

Currently there is no usage of must,must not, required inside the 2.1.0 specificiation, but
all others are used in same class in different context situations!
E.g. a driver property inside a SARIF model is a ToolComponent and it is important that it contains rules (shall), but on the contrary a Taxonomy is also a ToolComponent object ... but... there it is NOT allowed to contain rules (or notifications) (see https://docs.oasis-open.org/sarif/sarif/v2.1.0/os/sarif-v2.1.0-os.html#_Toc34317558)

If the toolComponent describes a standard taxonomy (for example, the Common Weakness Enumeration [CWE™]), it SHALL NOT contain rules (§3.19.23) or notifications (§3.19.24).

This means it is very difficult to handle the situation inside same java class for different situations (in this case ToolComponent.java)
In our SARIF java model we do currently provide

shall by defining the object as an immutable  instance. E.g. List inside driver is never null.
This means in our Jackson JSON output we have always at least an empty array.
So far so good, but providing taxonomies and using same class does result in a an empty rules array for taxonomies,
what is extreme confusing, because taxonomies are not allowed to contain rules by specification!
may by allowing the object or the property to be null.
shouldand may by allowing the object or the property to be null.

It is also not documented how our SARIF java model structure is defined - e.g. what kind of base classes exists, how we map properties etc.
Wanted

Clear specification for SARIF java model
Clean and valid way to handle the the situation when have shall and shall not in same class

Solution

A specification inside SecHub technical documentation about the concept how we handle the SARIF situations described inside this
issue (in a short form). So every developer extending the SARIF java model understand how to handle different situations. Also the relationships and
the kind of objects etc. should be described (e.g. SarifObject as base class etc.)
We will use

fixed and immutable objects for MUST and REQUIRED(future only, currently not used ins specification)
In this case it should not be optional.
SHALL, SHALL NOT  -> XOptional
MAY -> XOptional
OPTIONAL -> XOptional

Open : XOptional: is it optional or null ? Using always optional in every api call is often very cumbersome. And even optional can be null at some situations.

If we are successful with #1303 this issue becomes obsolte because we could generate all model stuff there.


Improve code quality and fix bug

Changes

Change SelectAppointmentParser to use the 1 index parse index of parseIndexes

Reduce method length of parse in FindCommandParser and improve slap

Extracted out if else block in parseCommand in AddressBookParser

Fixed a bug where you can do commands like mark patients 1, delete all 1 and it'll work as if the patients/all isn't there

Codecov Report

Merging #279 (5b7963b) into master (6f2ec6c) will decrease coverage by 0.36%.
The diff coverage is 52.63%.

@@             Coverage Diff              @@
##             master     #279      +/-   ##
============================================
- Coverage     70.74%   70.38%   -0.37%
- Complexity      814      815       +1
============================================
  Files           111      111
  Lines          2738     2745       +7
  Branches        365      356       -9
============================================
- Hits           1937     1932       -5
- Misses          691      703      +12
  Partials        110      110

Impacted Files
Coverage Δ

.../seedu/address/logic/parser/AddressBookParser.java
54.23% <40.90%> (+0.39%)
:arrow_up:

.../seedu/address/logic/parser/FindCommandParser.java
98.11% <90.90%> (+0.03%)
:arrow_up:

...s/logic/parser/SelectAppointmentCommandParser.java
100.00% <100.00%> (ø)

...in/java/seedu/address/logic/parser/ParserUtil.java
81.25% <0.00%> (-14.07%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Month view

I have a problem with the month view. The calendar shows 6 weeks but if the first day is sunday, i can´t see the first day in this month.
I also attach you an image.
Do you know what i did wrong?
Best regards

I don't understand your problem.
On your image the first day of the week is Monday, if you want to begin the week to Sunday you have to do self.calendar.calendarAppearance.calendar.firstWeekday = 1; Check the Example project or the README.
Also the calendar always show 6 weeks (because some months need 6 weeks for be display) and it's too complex to manae a calendar with a dynamic height.

Ok, I explain you.
When i put these lines in my code:
NSLocale *locale = [NSLocale localeWithLocaleIdentifier:@"es_ES"];
self.calendar.calendarAppearance.calendar.locale = locale;
The calendar show wrong the days of the month, but if I change this line to @"en_EN", the calendar works perfectly. Do you have any idea?
Thanks!

I see the bug, I don't know why it happen but from what I understand about NSCalendar when you use firstWeekday it overrides locale and locale is not used to translate the days or the month, so I only can advise don't set locale.

I just release a new version.
Check the Example project.
This bug should be gone.


Performance (Memory): Add eviction to Memoize functionality

We aggressively memoize expensive computations using a very simple Memoize.make helper function:
https://github.com/revery-ui/revery/blob/ed0f2d92c4b0e0f84f61ef35aebac5690fc76e4b/src/Core/Revery_Core.re#L32
Some examples of items we currently 'memoize' today:

Geometry: https://github.com/revery-ui/revery/blob/7b8ccf7e9644d22359d512331bdcb4b63762fecd/src/UI/Assets.re#L15
Font glyph rendering: https://github.com/revery-ui/revery/blob/3516ca529eb0e85e7c1063578f3567c48be1ab59/src/UI/FontRenderer.re#L7
Font shaping: https://github.com/revery-ui/revery/blob/3516ca529eb0e85e7c1063578f3567c48be1ab59/src/UI/FontRenderer.re#L44

For very expensive operations that aren't meant to be done frame-over-frame, this is helpful - and keeps us from wasting cycles.
However, it also causes our memory usage to balloon - and eventually, we really don't need to keep some of these items around.
We should implement some sort of eviction policy so that we can remove items that aren't being used.
One option for this would be some sort of least-recently-used eviction policy - it looks like there are some LRU cache implementations like:  lru-cache
The work here would be to investigate some sort of eviction policy to be used to augment (or, perhaps an lru cache could replace entirely) our Memoize.make functionality.

One of the most interesting memoization techniques that I've learnt in JavaScript is the usage of WeakMap to store the result from expensive computations in a map where the keys are weakly hold. The advantage of weak maps is that once the key K is not hold anymore anywhere in the program, any key/value pair in the weak map with key === K is released as well, without any manual handling required.
The upside compared to lru-cache or other strong memoization techniques is that weak memoization can be very effective, allocating and liberating memory in a very "elastic" way. The main downside compared to lru-cache is that the costs in memory have no hard upper limit, so if new values keep arriving to a memoized function, new results keep being allocated, so overuse of weak memoization can lead to too much allocation of data that might end up not being very actively needed during the program execution.
But I still think weak memoization might be a good fit for the kind of programs built with a React-based renderer like Revery, and a language based on immutability like Reason, because references are not generally hold for too long. With the usual disclaimer of "use with care when it makes sense" 😄
I just found out that WeakMaps in JavaScript were actually built on the idea of something called "ephemeron" (see this great example by Brendan Eich on Twitter), which were discovered in 1997 (paper) and also seems were added to OCaml in 4.03! 🎉  (see JaneSt blog post where ephemerons are explicitly mentioned).
@bryphe I have no idea about ephemerons in OCaml tbh 😅  but it seems something worth exploring, maybe for just a small set of the cases mentioned above?

The OCaml docs about ephemeron are great too: https://caml.inria.fr/pub/docs/manual-ocaml/libref/Ephemeron.html

Wow, nice find @jchavarri ! You weren't kidding - the description for the Ephemeron module's use case reads exactly like the problem we're trying to solve. Seems promising!
We might have to evaluate the way we're using the keys (we're using transient tuples as keys - probably not an ideal approach in general, but especially not in the context of this module).
In some cases we're using keys like (fontFace: FontKit.fk_face, fontSize: int, glyphId: int) to memoize renderGlyph - I wonder how that would work in the context of a WeakMap - the fontFace would likely be long-lived, so it might mean that the values are long-lived too? We'd potentially have to tag it with another key (perhaps a render node instance?) to tie it into some shorter-lived lifecycle 🤔

I believe the key is evaluated as the whole thing, i.e. the lifetime of a key (a, b) is independent of the lifetime of (a, b) individually. This is good because we don't have to worry about things living too long, but might cause issues too since we have to start storing that value somewhere so that the key keeps existing. (All this is speculation though)

I'd be curious to look into this if no one else is already poking around


NPE in HttpRequestTask.doInBackground()

Stacktrace:
Caused by java.lang.NullPointerException
       at org.apache.harmony.xnet.provider.jsse.OpenSSLSocketFactoryImpl.createSocket(OpenSSLSocketFactoryImpl.java:94)
       at libcore.net.http.HttpConnection.setupSecureSocket(HttpConnection.java:197)
       at libcore.net.http.HttpsURLConnectionImpl$HttpsEngine.makeSslConnection(HttpsURLConnectionImpl.java:478)
       at libcore.net.http.HttpsURLConnectionImpl$HttpsEngine.connect(HttpsURLConnectionImpl.java:433)
       at libcore.net.http.HttpEngine.sendSocketRequest(HttpEngine.java:289)
       at libcore.net.http.HttpEngine.sendRequest(HttpEngine.java:239)
       at libcore.net.http.HttpURLConnectionImpl.connect(HttpURLConnectionImpl.java:80)
       at libcore.net.http.HttpURLConnectionImpl.getOutputStream(HttpURLConnectionImpl.java:188)
       at libcore.net.http.HttpsURLConnectionImpl.getOutputStream(HttpsURLConnectionImpl.java:281)
       at com.smartdevicelink.util.HttpRequestTask.doInBackground(SourceFile:88)
       at com.smartdevicelink.util.HttpRequestTask.doInBackground(SourceFile:16)

Occurs at https://github.com/smartdevicelink/sdl_android/blob/master/sdl_android_lib/src/com/smartdevicelink/util/HttpRequestTask.java#L88
NPE happens deep into getOutputStream() call. Unsure of the cause.

http://stackoverflow.com/questions/20817383/url-openstream-or-urlconnection-getinputstream-raise-nullpointerexception-on

Possibly call .connect() before the IF statement?


Make sure all the PhantomJS processes are killed

This fixed my issue: https://github.com/site5/phantoman/issues/20
Let me know if you see things to improve, I'm especially interested why you used register_shutdown_function().

This doesn't seems to resolve my issue. I'm closing this PR now, hopefully I find another solution

@TrafeX Ah that's too bad. I'll take a look soon and see what I can come up with. Thanks!


ISO8601 formats don't handle fractional seconds

Given a timestamp as a DateTimeTz value:
timestamp.format(ISO8601.IsoDateTimeFormat(null, "YYYY-MM-DDThh:mm:ss.sssZ"))
does not work since it wants to try to parse the fractional part as seconds.

This also goes for the DateTime class.

@j0bro, @rwslinkman, you need to use 'S' for fractional part:
timestamp.format(ISO8601.IsoDateTimeFormat(null, "YYYY-MM-DDThh:mm:ss.SSSZ"))

I am afraid that is not the case. They should be lowercase and I inspected
the source, it is not even trying to parse the fraction. What happens is
that it tries to match it against the seconds part, which only succeeds if
one provides two s's after the period, otherwise it fails.
Op zo 19 apr. 2020 17:10 schreef Mikhail Reznichenko <
<EMAIL_ADDRESS>
@j0bro https://github.com/j0bro, @rwslinkman
https://github.com/rwslinkman, you need to use 'S' for fractional part:
timestamp.format(ISO8601.IsoDateTimeFormat(null,
"YYYY-MM-DDThh:mm:ss.SSSZ"))
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/korlibs/klock/issues/102#issuecomment-616157775, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABHU5LESJNPIP2DJNWWMKOTRNMH53ANCNFSM4MEVWLKQ
.

@j0bro it works correctly with 'S' for milliseconds

Ah, I missed your mention about ISO8601 format, sorry. For ISO8601 it's a bug indeed

@j0bro I fixed this bug in #106

@rwslinkman you should use S for milliseconds with DateTime formatting/parsing:
PatternDateFormat("dd MMM yyyy HH:mm:ss.SSS")
If you still need to use s for milliseconds, please create a new issue.


PR to yciabaud's PR – recursion!

do not filter out extra keys from 'update' or 'scrape' events, in WS and HTTP
remove requestHandler option
expose Swarm object for easy overriding
test: remove UDP request handler test -- not possible
test: change request handler test to new approach

Oh thank you for this changes, you are right this is totally fixing my issues and this looks less intrusive in the basecode.
I am merging this and will update my implementation soon.

Great!


More Extensive Documentation

Hi there,
I am thinking of using Enzyme for a larger project, but many things are unclear to me as the documentation is so short and really only covers the basics of the autodiff function. I watched your JuliaCon presentation from last year, and some of the points discussed there already helped me, but they should probably be included in some (updated) form in the documentation, like

GPU usage with KernelGradients
(GPU usage with CUDA.jl?)
How does Enzyme.jl work with the FFI
How does Enzyme.jl work together with ChainRules (it says in the talk that it is possible, but clunky)

@vchuravy can we copy in your presentation notebook into the docs somehow?

I think these [or the successor functionality] are now reasonably well documented


command python.setInterpreter not found

Hello All ,
I would like to ask if someone can solve the issue with python + VS Code issue. The issue is that the python extension does not work properly on Mac OS X Ver 10.15 (and 10.14 as well)
When ever I try to set the python version I get the following error...
"command python.setInterpreter not found"
It does not recognize the python interpreter that are installed.
I did try to uninstall it and reinstall the extension with no lack so far.
The same vscode version with the same plugin on ubuntu 18.04 works good. My filling is that issue is related to case insensitive system (macosx). Can some please give some help.
It is prevent me from using vscode on macosx.
Vscode info :
Version: 1.41.0
Commit: 9579eda04fdb3a9bba2750f15193e5fafe16b959
Date: 2019-12-11T17:58:38.338Z
Electron: 6.1.5
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Darwin x64 19.0.0

Hi @mksoft-systems,
Can you share a gif of the steps you're taking before you get this error? Also, are you waiting for the extension to finish loading before trying to set the python version?

The step is very simple.
There is no python ver on the left buttom like in linux.
The plugin is done installing.
The vscode is not recognize the python.
Please fix this error on mac osx 10.14+ , I am not the only one who have it.
הורד את TypeApp עבור Android
ב-16 בדצמ׳ 2019, 23:54, ב-Kim-Adeline Miguel<EMAIL_ADDRESS>23:54 כתב:

Hi @mksoft-systems,
Can you share a gif of the steps you're taking before you get this
error? Also, are you waiting for the extension to finish loading before
trying to set the python version?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/microsoft/vscode-python/issues/9131#issuecomment-566261580

--
test text sig

It works on macOS 10.14+, always has, so there's something else going on in your setup:

Do you have a python file open? And if you have a python file open, are you waiting for the extension to finish loading before trying to set the python version?
Can you share a recording of what's happening starting from when you open VS Code?

I can see , but at my mac it never did.
And there is more like me.
הורד את TypeApp עבור Android
ב-17 בדצמ׳ 2019, 18:14, ב-Kim-Adeline Miguel<EMAIL_ADDRESS>18:14 כתב:

It works on macOS 10.14+, always has, so there's something else going
on in your setup:

Do you have a python file open? And if you have a python file open, are
you waiting for the extension to finish loading before trying to set
the python version?
Can you share a recording of what's happening starting from when you
open VS Code?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/microsoft/vscode-python/issues/9131#issuecomment-566629276

--
test text sig

Because we have not heard back with the information we requested, we are closing this issue for now. If you are able to provide the info later on then we will be happy to re-open this issue to pick up where we left off.


Fix Yslow error: The following resources have identical contents, but…

… are served from different URLs: https://cdn.auth0.com/js/lock/10.3/lock.min.js
When running my site through YSlow I get the following error:
The following resources have identical contents, but are served from different URLs. Serve these resources from a consistent URL to save 1 request(s) and 681.6KiB.
https://cdn.auth0.com/js/lock/10.3/lock.min.js
https://cdn.auth0.com/js/lock/10.3/lock.min.js?ver=4.7
SSO is enabled.  The SSO template seems to include Lock manually before its inline JS code. I just removed this . This fixed the issue for me, but further testing may be required.

Is this still an issue?

I'm not sure. I stopped using SSO because of this issue.