Add start looking ahead ar doc

Q
A

Is bugfix?
no

New feature?
yes

Breaks BC?
no

Tests pass?
yes

Fixed issues
comma-separated list of tickets # fixed by the PR, if any

Merged. Thanks!

Thank you @samdark
Hi @cebe , Depends of your comment here, I ping you, After accept start-looking-ahead by @samdark, we can say all start-*.md was added to Arabic guide. Thanks.

I also noted that you have used
 tags instead of  for inline code blocks, this results in the text to be broken into lines.

Thank you @cebe for interested, about your question I can say:

I use dir="rtl" to make display text from right to left in github, I read how I can make text dispay from right and I found I need to set direction and work like Html player.
In yii website, its not necessary, Thats true:D
Sure I will, and I will replace all div in all translation page to span.
About using
 I think its used in mistake...Since I use  tag most time...I will find
 tag where is it used and replace it.

If I need to work in these changes, Would you like to I create a new branch or I use WIP arabic guide branch?
Many Thanks

Please noted, I remember why I using div nested of span, If I set span the direction will not work truth, since its inline, and div is a block...so that the direction will be work in block element only....so that how would you like to fix issue? would you like to keep it in div or using inline style like display:block inside span...?

Thank you!

I merged and deployed https://github.com/yiisoft-contrib/yiiframework.com/pull/411, please check it on the website and report if there is anything broken.

Sure I will, Thank you :D

I found some issue:

Main Grid Need Reflection from left to right in main page. for example: "المقدمة" most be in first column from right.

Internal links issue. for example: https://www.yiiframework.com/doc/guide/2.0/guide/db-active-record.md - this issue received since I set anchor tag like this in Arabic guide action, I use this way to keep Arabic guide reference to updated English guide...so that what do you like to I do with this?

Search in Guide for Arabic dose not return any results like search "تثبيت".

I fix some issue too in this branch - Fixing Arabic Guide WIP issues.
Note: Im try to find
 tag in our Readme, I not found where it, if you can reference me to page. thanks.

1 and 2 should be fixed by https://github.com/yiisoft-contrib/yiiframework.com/commit/10381ae21d21245ad652db7fad4d66cc9846074e now.
3 could be caused by Arabic language not being set up in elasticsearch. added a comment here: https://github.com/yiisoft-contrib/yiiframework.com/issues/254#issuecomment-446928814

Note: Im try to find
 tag in our Readme, I not found where it, if you can reference me to page. thanks.

https://www.yiiframework.com/doc/guide/2.0/ar/start-installation#verifying-installation
below the image.
start-installation.md line 173 and following.

Thank you @cebe, Its Perfect!
I fixing related issue too, and I pull #16963  to Yii.
If you have any note, issue in any time I will be happy to doing my best to improve this community.
My next step is share this update and try to got Arabic Yii Developer to contribute Arabic guide, and I/WE will continue work in translation.
Many Thanks cebe!


[Request] command to close all opened channels

Currently there is no way to close all channels that are opened without some hacking. It would be nice to have something like:
lncli closechannel -all
If you want to close all channels: @djseeds:
to close all channels...
lncli listchannels | jq '.[][]' | jq -r '.channel_point' | tr : ' ' | xargs -n2 lncli closechannel
to close only inactive channels...
lncli listchannels | jq '.[][]' | jq -c -r 'select(.active == false) | .channel_point' | tr : ' ' | xargs -n2 lncli closechannel --force

there is no documentation about the command, please provide one here


Communication tower on varadero sometimes does not show on the proper spot on tacmap

i have literally zero idea how this can happen, i've also not seen it happen on another map yet
happened on round 2044 and some other round i played 9 days ago

what is wrong with me this is literally wrong


change special characters to common chars

change special characters to common chars.
now all special chars form ASCII are picked, even those we have problems to type in with a keyboard.

maybe implement two charsets of special chars. (s and S)


Fix deleting ogre2 when created (and deleted) in a thread

🦟 Bug fix
Summary
I ran into a crash in gz-sim on exit after unloading the ogre2 render engine. It was found that it only happens if ogre-next is built with EGL support and the engine is created / deleted in a thread. The fix is to make sure EGL returns to its original state, specifically by calling eglReleaseThread() after deleting ogre root.
The fix this PR make EGL calls which requires including and linking against the EGL library, hence the extra cmake code. The proper fix may be to do this inside ogre.
To Test:
Run the INTEGRATION_load_unload test
Checklist

[x] Signed all commits for DCO
[x] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Not sure if this should've been merged with the homebrew failures. The stable branch is green https://build.osrfoundation.org/view/ign-garden/job/ignition_rendering-ci-gz-rendering7-homebrew-amd64/

testing changes in https://github.com/gazebosim/gz-rendering/pull/841 to see if it fixes the crash on macOS


Extensions notebooks

Polish, cleanup, and minor fixes to BA and webdev extensions notebooks.

All changes made here have been reviewed.


Fix integration with Artisan not allowing access to retainers

No longer sets the plugin to enabled during Artisan execution if retainers are not available.
Disables plugin after finishing Artisan execution to allow normal access to retainers.

Thank you!


Need to save phone number somehow

Add "question" to wufoo form to ask for number. Patterns will just save the number in that field instead of asking for it?

I added a phone number question/field to end of wufoo form. Patterns fills this out before submitting the form

https://github.com/smartchicago/kimball/commit/35a58d4695b6d75eb3d279aa3eea2ab343b22d77

The SMS sign-up Wufoo form asks for a phone number, but this is not being asked to the user (since they are texting and we already know their number). Adding this question allows us to capture the phone number and send to Patterns.
Closing since this is completed.


[SPARK-4808] Remove Spillable minimum threshold and sampling

Seeing some problems that this PR could address so reviving this thread.
@lawlerd the configurable count would help because if it is known that the individual objects would be large, the sampling could be set to be done more frequently. So if sampling every 32 times is too passive then a more aggressive option can be configured, say sampling every 5 times.

Hey @lawlerd this seems to be superseded by #4420. Would you mind closing this PR? Please follow the latest discussion there.


Admin access can't add people

Not sure if this is only a problem for adding brand new people, but if you don't have system access you get a something went wrong popup even though it finds the person on the screen.

It is this line that is preventing it.
https://github.com/ucdavis/Sloth/blob/master/Sloth.Web/Controllers/UsersController.cs#L86

Need to pass the team in the route values here:
https://github.com/ucdavis/Sloth/blob/8af9709518ae375c75461d2bb1fc66eeb1512f51/Sloth.Web/Views/Teams/Details.cshtml#L404-L415


Receipt verification failing during review

I've been trying to debug and get through this myself, but am at the point of needing some assistance. I'm using the InAppStore plugin to make In-App Purchases in one of my games. But I keep getting rejected by the iOS app store review for the following reason:

We found that your in-app purchase products exhibited one or more bugs when reviewed on iPad running iOS 15.1 on Wi-Fi.
Specifically, we still received an error message when we tried to make an in-app purchase.
Next Steps
When validating receipts on your server, your server needs to be able to handle a production-signed app getting its receipts from Apple’s test environment. The recommended approach is for your production server to always validate receipts against the production App Store first. If validation fails with the error code "Sandbox receipt used in production," you should validate against the test environment instead.

I'm using the most basic implementation that is presented in the documentation/examples and am able to complete purchases using TestFlight without hitting this error, so this is only happening during review when they're using the production signed app to make purchases on the sandbox. I'm at a loss for what could be going wrong here. Please help.

@jebentier you should probably request additional data from AppStore Review team to see if they can provide you with anything on the issue.
I would also recommend you to double check you InApp Purchases settings if they are available and have all required metadata.
As far as I can tell purchases should also be sent to review, so issue might be related to that.

FYI, today i do have a pretty similar problem. And just found out, if the currency of your IAP item not match the region you choosed your sandbox tester. It could cause an error. You better choose "US" as the base currency. Because they test your items with US sandbox account, if you choose different currency, there is chances that they will have error when try to make a purchase. BTW, better note for them that they should try again if the purchase failed.

I’m having the exact same problem with my game failing app review. The plugin works fine for me testing in Sandbox, and I use the exact same implementation in another game successfully, so I’m not sure where to start. Has anyone found a solution?


Co-interior Pairs

Enter calculator name -
Co-interior Pairs
Describe calculator functionality -
The Co-interior Pairs Calculator takes the value of one angle of the co-interior pair and calculates the value of the other angle.
Under which category you wish to place it -

[ ] Computer Science
[x] Mathematics
[ ] Physics
[ ] Chemistry
[ ] Biology
[ ] Art
[ ] Finance
[ ] Scientific
[ ] Health
[ ] Games/Sports
[x] Let project owner decide
[ ] Suggest a new category - (Answer here)

Hey @SarthakKeshari , @Vedansh-Keshari!
Can I work on this issue?

Hey @hubsMIT1,
Assigned.
You may contribute your code under the folder path -
calc_for_everything\calculators\src\mathematical\coInteriorPairs
Issue completion deadline - 31st Oct 11:00:00 pm IST
NOTE - In case the issue completion deadline is not met, the issue will be assigned to a new participant


I am unable to clone the repository .. why?

I use this command git clone git@github.com:kujason/avod.git --recurse-submodules to clone the repo but I get this error
Cloning into 'avod'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

why?

Same here...

This is probably because you don't have an ssh key set up. You can clone using HTTPS instead:
git clone https://github.com/kujason/avod.git
git submodule update --init --recursive

Thanks for the quick response!
The first command works, but the second command failed with the following info.
fatal: clone of 'git@github.com:kujason/wavedata.git' into submodule path '/Users/Shasha/Documents/avod/wavedata' failed
Failed to clone 'wavedata'. Retry scheduled
Cloning into '/Users/Shasha/Documents/avod/wavedata'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.
Any suggestions? Thanks!

Try changing the url for the wavedata submodule in .gitmodules to the HTTPS version:
https://github.com/kujason/wavedata.git

It works! Thanks for the quick and helpful response.


x/pkgsite: package removal request for https://pkg.go.dev/github.com/googlecloudPlatform/gcsfuse

What is the path of the package that you would like to have removed?
Path: https://pkg.go.dev/github.com/googlecloudPlatform/gcsfuse
Are you the owner of this package?
I am one of the Admin of https://github.com/GoogleCloudPlatform/gcsfuse repository.
What is the reason that you could not retract this package instead?
We see that both the below packages point to our repo: https://github.com/GoogleCloudPlatform/gcsfuse.

(Not updated from 2020, to be removed) https://pkg.go.dev/github.com/googlecloudPlatform/gcsfuse
(Updated with release in our repo) https://pkg.go.dev/github.com/googlecloudplatform/gcsfuse/tools/build_gcsfuse?tab=versions

We don't want two packages to point to our repo and hence we want to remove #1.

Documentation at https://pkg.go.dev/github.com/googlecloudPlatform/gcsfuse has been removed.


Quercetin predictions

@tdrl this is the feature branch for today's predictions on quercentin.  It includes a python script that facilitates chunk-by-chunk prediction on an input compounds file and extraction of a subset of relevant proteins from the outputs.  In the test folder, I added the inputs and outputs specific to quercentin.

Removing this breanch; will recommit primary script separately


Fails to find Simon & Garfunkel lyrics even though they are on Musixmatch

Not a single Simon & Garfunkel song has worked for me so far. I looked up the last one to fail and it was on Musixmatch.
Just a guess, but might it be that tidal-lyrics fails to properly escape the '&' character and thus matching fails? Maybe some strategic spot needs a call to a UrlEscape method, or similar, to fix this for all special characters?

I'll look on it rn

It seems that Musixmatch doesn't understand & when the url is encoded (see here https://www.musixmatch.com/search/The Sound of Silence (Electric Version)* - Simon %26 Garfunkel). Even if you type Simon & Garfunkel in the search and then refresh, it will break. So yeah I'll have to fix that..

So, you actually have to not url escape it for it to work. smirk
That is super veird :)

Now it works

Fabulous! Will be eagerly awaiting a new build or release :)

https://www.dropbox.com/s/m9fbvv0hnhbnyg3/tidal-lyrics-1.4.0 Setup.exe?dl=0
Here is a windows build (took a while to upload, my internet is slow :( )

And it's working. Simply outstanding speed of providing a fix.
Major kudos!


Add error handling

Added Error Messages to wm.c.
Please let me know if there is something to fix!
P.S. I'm going to try to create the keyboard navigation (and some shortcuts) too :)

Okay... Let me know if this is better!
I added a cleanup, fixed the typo and added a return EXIT_FAILURE and EXIT_SUCCESS
(Don't mind the random deletion of shortcuts.c and shortcuts.h its not done yet but I accidentaly pushed it too 😅)

That should be it then. Thank you very much for the reviews!

Merged, thanks!


Update 011-the-help-command.md

What type of PR is this? (check all applicable)

[ ] ♻ Refactor
[ ] ✨ New Chapter
[ ] 🐛 Bug Fix/Typo
[ ] 👷 Optimization
[x] 📝 Documentation Update
[ ] 🚩 Other

Description
Added some examples for "help" command for better understanding of the syntax of "help" command.
Related Tickets & Documents
None
Added to documentation?

[x] 📜 readme
[ ] 🙅 no documentation needed

[optional] What gif best describes this PR or how it makes you feel?

Hey, I added the backticks as requested.

Hi, sorry for the typo, corrected it now


Navbar added

PR Description 📜
I have added the navbar
The navbar is properly aligned and user can navigate to the corresponding sections
Fixes #3892
I request the admin @kunjgit to label this under gssoc and merge the pr

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

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


Port bitcoin secp256k1 release 0.3.1 fixes

Use volatile flag in all _cmov functions
Port wycheproof test suit from bitcoin secp256k1

@Naviabheeman you need to rebase with master branch to fix ci.


Do not generate liquid ores near fissures

I've added noise_params to the ore def for liquid ores, that matches watershed's fissures with a noise_threshold of 0.08, to avoid endless water/lava/acid flowings in fissures. I've also duplicated these 3 ores, with inverted scale of noise_params to generate them on both side of the fault.
But there can still be liquids flows, but much more rarely (for example if they sink into a small cave that leads to a fault). I followed 3 fissures on more than 1000 nodes down and have not seen any liquid in 2 of them. The third was filled by lava because of a volcano. I may try to fix this. But I think it takes part of the game, as long as that's not the case for most of them.

Im my point of view it's enough, thank you @Gael-de-Sailly for this improvment.
if volcano inside fissures are rare enough, we can just work this this. (Because clearly, 90% of the flowing liquids into fissures has been removed with this PR)

6aa375d


⚠ AGEG website (prod) has degraded performance

In eea3d3d, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 2241 ms

Resolved: AGEG website (prod) performance has improved in a7341b0.


Inpaint not working on SDXL using sdxl_gen_img.py

When I try to generate images using:
"python sdxl_gen_img.py --outdir "outputs" --ckpt stabilityai/stable-diffusion-xl-base-1.0  --steps 40 --strength 0.8 --prompt "A picture of a chair" --network_module networks.lora --network_weights test_output/test_output-step00001500.safetensors --image_path "inpaint-images/input/input0.jpeg" --mask_path "inpaint-images/input/input0-mask.jpg""
My images get weirdly distorted all over. --fp16 returns black images, but that apparently has something to do with the VAE. using --bf16 or the standard float32 as above, does not resolve the issue. I will add the mask and the init_image below, as well as the generated image.

Okay, thanks! I went back to rewriting the Diffusers training code for it to implement more functionality.
On another note; I would love to get in touch with you regarding a business query! You can reach out to me at<EMAIL_ADDRESS>much appreciated!


fix #89 Add individual paste buffers to cursors

With this commit you can use multiple cursors to yank/paste and they will use
their own block of text as expected. In Visual mode operations y and Y are
supported. In Normal mode, all operations that change unname register are
supported (dcsx) but only few motions are supported (e.g. cw, ciw, caw,
ci(, and so on, but no support for ctx or c/pattern<cr>).
For best experience, use these options in your ~/.vimrc:
let g:multi_cursor_normal_maps = {'d': 1, 'c': 1}
let g:multi_cursor_exit_from_insert_mode = 0
let g:multi_cursor_exit_from_visual_mode = 0

This hopefully fixes #89. These are my first baby steps in VimL, so somebody experienced should definitely check that. @eapache Could you please try/review this?

More tests could be added for commands with motions, such as ciw, 'da(', I just need to figure out how to override global options in unit tests.

I'm a bit busy at the moment but I will try to find time to review this sometime in the next week, this is definitely a feature I want!

Me too!

So I think this approach will generally work, and I'm not sure what other approaches you tried, but if I were to try writing this I think I would approach it slightly more simply:

after each cursor operation, save the contents of the register
before each cursor operation, restore the contents of the register

This would do unnecessary work on operations that don't copy/paste any data, but I think it would be simpler and easier to make work with e.g. the more complex motions. Thoughts?

That was exactly my first approach :) Unfortunately I had to give it up. Here is why. You see, if we pay no attention to what cursor operation we're saving register after, operations such as b, w or any other movements in Normal mode would overwrite paste buffer of all cursors: remember that vim has only one unnamed register, thus recording its contents to all paste buffers during operation that does not change unnamed register will copy that same block of text N times which is not what we want. We have to save register only during replay of csdxy operations because that's the only moment of time when we know the value of @" for each cursor, after that it's lost (except for the last cursor).
But I didn't give up that easily. Next, I though, ok, let's improve that approch: why not check if current value of unnamed register is exactly equal to the value of any of paste buffers? If unnamed register is already present somewhere, then we must be just moving around, right? Wrong. Imagine the following case: first we put 3 cursors on the second column of words one bellow another and yank them:
send send
send receive
send skip

If we start moving after that, it's good, imagine last cursor yanked send (@" = send), which is present in at least one paste buffer. But now we put our cursors on the first column of send words and yank them. It doesn't work because we have one send already stored in one of our paste buffers.
 I don't even know if it works with remappings: what if we nnoremap h c, will c be detected as a "register changing operation"? What we need is some kind of notification from vim about type of operation that is about to happen. Sadly, I'm not aware of such API, are you? Thoughts? Am I missing something?

Hmmm... you're right it is much trickier than I thought at first glance!

why not check if current value of unnamed register is exactly equal to the value of any of paste buffers?

My gut says there's still an approach here that works. What about if, for each operation:

save the current register to a temporary variable
set the current register to the contents on the buffer (if any)
run the operation
save the register to the buffer
restore the register from the temporary variable only if it didn't change during the operation

I think that covers all the cases nicely?

I've been desperately trying to grasp the conditon: only if it didn't change during the operation and failed. Instead I decided to just go and implement it without that condition. It doesn't work because register gets copied into all cursor buffers during movement operations.
Just so we are clear, let me put that into pseudocode (R = register, T = temporary, B = paste buffer):

T=R
if (B) R=B
operation
B=R
R=T (what's the pseudocode for only if it didn't change during the operation?)

Hmm, yes, that condition was supposed to deal with the problem you noted, but I did a very poor job of explaining. As I edited my original post to add though, I think that condition is entirely unnecessary if you make the second step unconditional as well. Working pseudocode should go something like this:
T=R
R=B
operation
B=R
R=T

Note that R=B should run even if B is empty. Also note that this means we would 'lose' the existing contents of the register when entering multiple-cursor mode; I think that can be remedied by initializing new cursor buffers to the contents of the register, rather than just an empty string.

I'm afraid it still doesn't work. As soon as I disable if strridx(s:paste_buffer_supported_operations, s:char[0]) >= 0 check, it breaks the tests:
Multiple Cursors op pending & exit from insert|visual mode
  #paste buffer normal caw then p (FAILED - 1)
  #paste buffer normal C then ABC then p
  #paste buffer normal daw then P (FAILED - 2)
  #paste buffer normal D then P (FAILED - 3)
  #paste buffer normal s then p

I did the following changes according to your description: http://sprunge.us/WjDA?diff
Am I missing something?

If fact, I am. I forgot about this check: if s:from_mode ==# 'n' && s:char ==? 'p'. Now tests pass! Hooray and glory to @eapache :)
I've left a couple of comments in the last commit. Please review it all once again to check whether I didn't forget any leftovers or misleading comments or anything like that. And thank you for the simple idea, you rock! :)

I was thinking originally that both save and restore could happen regardless of mode... It's a bit odd to me that right now you save in visual mode but only restore in normal mode...

Hmmm... I should probably run some performance/latency tests and see if it makes any difference with and without mode checks.

Hmmm... I should probably run some performance/latency tests and see if it makes any difference with and without mode checks.

I've noticed a few times some latency timers built-in to the code, but I don't know how to use them.
My only real concern here is that operations in visual mode do multiple saves in a row without a restore, meaning that T gets overwritten several times and the original contents of the buffer gets lost.
One potential performance win, now I think about it, is that the original value of R (saved and restored from T all the time) should basically never change at all when in multi-cursor mode. You might not have to save/restore it on each operation, but only when entering/leaving multi-cursor mode (you would still have to save/restore the per-cursor buffer on each operation).

Apart from the concern about multiple-possible-saves-without-a-restore in visual mode, this looks really good - the other issues are just "nice-to-have", and since this ticket is getting quite long I suggest you start a new ticket to work on them (if you want) once this one has been submitted.

My only real concern here is that operations in visual mode do multiple saves in a row without a restore, meaning that T gets overwritten several times and the original contents of the buffer gets lost.

I moved saving/restoring of T to init/exit functions, please check if I chose the right place. If I understand it right, the problem should be gone. But even before that I tried to reproduce it and I couldn't. I created N cursors, and tried to move in Visual mode, change selection. The value of unnamed register before and after multiple cursors mode was the same. Isn't it restore which is called in Visual mode? save is only called in Normal mode, see process_user_input.
There are other glitches that I've noticed:

if you have something in @" and you start N cursors, then pastes, I expect all N cursors to paste the contents of @". It pastes something else now.
undo does something to the cursors that reverses some of them. If you yank, paste, see the output, then undo and paste again, output will be different.

I agree that work on those should be done in a separate ticket. If you feel like the work here is done, let's wrap up and merge this.

The undo problem will probably be complicated and certainly can wait. The initial paste problem is unexpected, so I'd kind of like to track that down before we merge this (we should have a test for it as well).

I've added test to check that unnamed register is pasted into 3 cursors. I didn't have to fix anything, looks like I accidentially fixed it in  839aed9.

Cool, this looks awesome, thanks for all your work!

It's been a pleasure working with you :)


E2: Found nested #ifdef/#ifndef pre-processor directive error

@TomyLobo If I recall correctly, you have made those pre-processor directives?
If so, can you make them "nestable"?
I have just noticed my "deprecated" E2 library doesn't validate/run because of this (see this).
And also I also noticed I can't test multiple functions as in:
#ifdef function1() & !function2()
# Stuff...
#endif

I remember that I tried to add this feature, but failed. I don't think it's easily feasible.
https://github.com/wiremod/wire/blob/master/lua/entities/gmod_wire_expression2/base/preprocessor.lua#L368

you'd need to add kind of a stack there
i think there's a stack class somewhere in wiremod that you can use

I haven't seen Stack that comes with Wire, yet..
Anyways, there is one that comes builtin.
I had one in pure Lua (just can't remember where I have saved it..). But, I guess builtin Stack should be good to go.

I had one in pure Lua (just can't remember where I have saved it..). But, I guess builtin Stack should be good to go.

It is in pure Lua, source

yeah you could use that i guess...
@Divran or is that a bad idea due to people trying to use E2 without gmod?

E2 can't be used without gmod anyway, so that doesn't matter. I'd still not use garry's (probably bad) class though. I'd probably just use a regular lua table as a stack.

I'd still not use garry's (probably bad) class though. I'd probably just use a regular lua table as a stack.

Well I mean, it's not bad in implementation terms, but the methods are pretty much just glorified table operations.

I'd still prefer seeing "local bar = foo.Pop()" to "local bar = foo[#foo] foo[#foo] = nil" everywhere.

Hmm, according to PiL, it's much simpler than that:
https://www.lua.org/pil/19.2.html
"With those two functions, it is straightforward to implement stacks[...]. We can initialize such structures as a = {}. A push operation is equivalent to table.insert(a, x); a pop operation is equivalent to table.remove(a)."
> a={1,2,3,4,5}
> function p(a) for x in ipairs(a) do print(x) end end
> p(a)
1
2
3
4
5
> table.insert(a,6)
> p(a)
1
2
3
4
5
6
> return table.remove(a)
6
> return table.remove(a)
5
> return table.remove(a)
4
> return table.remove(a)
3
> return table.remove(a)
2
> return table.remove(a)
1
> return table.remove(a)
>


818 multipath

The Problem
Closes #818
The Solution
Implements #754
Checklist

[ ] Test manually on test-ui using remote chains.
[ ] Run load tests.
[ ] Update documentation if needed.
[ ] Update CHANGELOG.md.

Removed the allowUnlimitedContractSize flag since it is no longer needed https://github.com/connext/nxtp/pull/916/commits/78f897a7eb066a094f5d090e26c6c364babf3f5b


Update milmove-cypress

Jira ticket for this change
Summary
Is there anything you would like reviewers to give additional scrutiny?
this article explains more about the approach used.
Setup to Run Your Code

💻 You will need to use three separate terminals to test this locally.
Terminal 1
Start the Storybook locally.
make storybook

Terminal 2
Start the UI locally.
make client_run

Terminal 3
Start the Go server locally.
make server_run

Additional steps

Access the
Login as a

Verification Steps for Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

Frontend

[ ] User facing changes have been reviewed by design.
[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] There are no new console errors in the browser devtools
[ ] There are no new console errors in the test output
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.

Backend

[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Database
Any new migrations/schema changes:

[ ] Follows our guidelines for Zero-Downtime Deploys
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

Screenshots

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against d09012ad8f0874f363928650b3fc9e3f5eaf060c

Hey @Ryan-Koch it turned out that I didn't need to update Cypress in my PRs. so let me know if you'd like any extra eyes on this, but there shouldn't be any duplicated work between our PRs. 👍🏽

Hey @Ryan-Koch it turned out that I didn't need to update Cypress in my PRs. so let me know if you'd like any extra eyes on this, but there shouldn't be any duplicated work between our PRs. 👍🏽

Oh nice, that makes things a bit less complicated for us. I'll let you know when I have this ready for a look. I've been looking into some extra test flakiness that seems to be happening with the new version.


IndexOutOfBoundsException when registering component types

log
code
It looks like trying to register too many component types might an IOOB - maybe the reassignment isn't working properly?

Found the actual problem line! if (this.raw2Types.length < rawId) doesn't work because arrays zero-index; it needs to be <= instead


Build Google Search Algorithm with Python.

Sir, Kindly assign this issue to me.

@Niketkumardheeryan sir may I work on this??

@Niketkumardheeryan can you please assign this issue to me? I saw its up for grab. I will use beautifulsoup4 ,request and response library


[RFC] More Consistent Notion of "Infinite Width" Values

Discussion originated in https://github.com/freechipsproject/chisel3/pull/857. The motivating example is bit extraction for literals of unspecified width. For example:
for (i <- 0 until 4) {
  io.out(i) := io.in(i) && 5.U(i)
}

Here I'm using the literal 5.U as a numerical value, I don't care about the width. However, without the special-case support in #857 this would result in an out-of-bounds indexing error. A more consistent notion would be helpful because the existing implementation that works for literals does not work for other Chisel values like Wires, Registers, nor Ports.
Sketch of Proposal
As discussed in the Chisel developers meeting on July 27th, we could create a new Chisel API (and corresponding FIRRTL primop) that does something like padToInfinity or padUnbounded. The full implications of such an operation are not clear, but at least in the case of bit select, the argument would be treated as if it were zero- or sign-extended to infinity. I imagine we need to think through exactly how this influences width inference (if at all).
PR Template

Type of issue: feature request

Impact: API modification*
It's hard for me to see how we do this in a principled way without some sort of API modification. The extend of it remains to be seen.

Development Phase: request

This issue has been around for a while now, with no real resolution, and keeps popping up at the dev meetings. I'm going to try to summarize all the positions here so we can try to make some progress on this.
There are several possible resolutions:

Leave as-is, and codify the hotpatch in #857.  This makes literals behavior inconsistent with non-literal behavior: specifically chisel frontend const-props literal bit extracts in a way that's slightly different from firrtl, in that it won't error out when attempting to index an out-of-bounds bit. I don't think this is a good idea because it introduces edge cases and blows holes in peoples' conceptual models in order to (bluntly speaking) save a few characters in an uncommon use case. 🗑🔥
Revert #857 (so making the chisel behavior match the firrtl behavior) with no additional changes. Basically, users will need to add special case logic in their code for handling out-of-bound extracts, such as padding their data to the position they want to extract from. Main disadvantage is some localized duplication of code, both the pad and extract need to have the same argument passed in. Argument for this would be in the absence of the special case const-prop in chisel frontend (which is arguably a bug), would anyone really have pushed for special support for this use case? There could be a runtime deprecation phase. 🤔
Add an operator for an out of bounds bit extraction like in #869. Eliminates the disadvantage of duplicating the bit position to extract across pad and extract by combining both into one operator, but creates another operator that may not see a whole lot of use (or will it?). Meeting minutes indicate agreement on this, though a name is still required. Suggestions included padExtract, i (probably a bad idea), extract, adding a pad keyword-argument to the existing extract operator, and iPad ("infinity pad"). This would break existing code (like the above option, and we could also have runtime deprecations), but offers a cleaner alternative. 🤔
Add a pad-to-infinity operator, which can be composed with bit extract. Elegant, but turns out to be an implementation nightmare. 😔

@jackkoenig @albert-magyar

Resolution: add padExtract operator, preserve existing behavior with deprecation warning until at least 3.3.


Update README.md to fix hub URL

Issue #, if available: N/A
Description of changes: Fix the hub URL in the readme
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hi @SandersAaronD , Thank you for the contribution


Homebridge logs - LG AC12BQ

Model: lg-AC12BQ

[6/16/2020, 13:37:54] [LgThinqAirConditioner] HeatingThresholdCharacteristic: Failed to set state 19.5 Error: Request failed with status code 400
[6/16/2020, 13:37:54] [LgThinqAirConditioner] CoolingThresholdCharacteristic: Failed to set state 22 Error: Request failed with status code 400

[6/9/2020, 15:42:06] [LgThinqAirConditioner] SwingModeCharacteristic: Error parsing state Error: Unsupported API value: 4
[6/9/2020, 15:42:06] [LgThinqAirConditioner] TargetHeaterCoolerStateCharacteristic: Error parsing state Error: Unsupported API value: 6
[6/9/2020, 15:42:36] [LgThinqAirConditioner] SwingModeCharacteristic: Error parsing state Error: Unsupported API value: 4
[6/9/2020, 15:42:36] [LgThinqAirConditioner] TargetHeaterCoolerStateCharacteristic: Error parsing state Error: Unsupported API value: 6
[6/9/2020, 15:43:06] [LgThinqAirConditioner] SwingModeCharacteristic: Error parsing state Error: Unsupported API value: 4
[6/9/2020, 15:43:06] [LgThinqAirConditioner] TargetHeaterCoolerStateCharacteristic: Error parsing state Error: Unsupported API value: 6
[6/9/2020, 15:43:36] [LgThinqAirConditioner] SwingModeCharacteristic: Error parsing state Error: Unsupported API value: 4
[6/9/2020, 15:43:36] [LgThinqAirConditioner] TargetHeaterCoolerStateCharacteristic: Error parsing state Error: Unsupported API value: 6

Thanks! Could you also provide:

What "model" is listed in the Home app's accessory settings?
What "mode" is your A/C unit on when you see this "Unsupported API value: 6" error?


landing page should be created

it is important to have a good landing page.

#17 Hero page has been created which is nothing but landing page.


make test breaks for Julia 1.10.7

Running make test for the lts version of julia, i.e. 1.10.7 yields the following error:
$ make test
julia -e 'import Pkg; Pkg.activate("."); Pkg.test(test_args = split("") .|> string)'
  Activating project at `~/git/ReactiveBayes/ReactiveMP.jl`
     Testing ReactiveMP
┌ Warning: Could not use exact versions of packages in manifest, re-resolving
└ @ Pkg.Operations ~/.julia/juliaup/julia-1.10.7+0.aarch64.apple.darwin14/share/julia/stdlib/v1.10/Pkg/src/Operations.jl:1822
ERROR: Unsatisfiable requirements detected for package REPL [3fa0cd96]:
 REPL [3fa0cd96] log:
 ├─possible versions are: 1.10.7 or uninstalled (package in sysimage!)                                                                                └─restricted to versions 1.11.0-1 by project [349aa928] — no versions left
   └─project [349aa928] log:
     ├─possible versions are: 0.0.0 or uninstalled                                                                                └─project [349aa928] is fixed to version 0.0.0
Stacktrace
[...]

In Project.toml, I see that julia 1.10 is required but at the same time, REPL 1.11 is required.
Should the julia requirement be pushed to 1.11 or is it the expectation that if you run julia 1.10, then you need to (somehow) have REPL 1.11 running?
make test runs for the current release version of Julia, i.e. 1.11.2.

I think we should downgrade minimum version for REPL to 1.10. Good catch!

Lets see if this helps https://github.com/ReactiveBayes/ReactiveMP.jl/commit/825eba36dcaf09c9f0cfe9df3f85af2ee57dee3c
https://github.com/ReactiveBayes/ReactiveMP.jl/actions/runs/12370007652


Caching for API views

There is a lot of performance to be gained from partial caching of API views. Most of the records in our database rarely change, if ever. Once the API views are DRY, caching should be added following the rails "Russian Doll" caching pattern.
Blocked by https://github.com/aosdict/yacs/issues/27

Blocked by #27

After some experimentation, I've decided that more testing will be needed to push this into production. Moving to General Release


To update flash encryption document for flashing pre-encrypted firmware (IDFGH-3006)

In Flash Encryption document:
https://docs.espressif.com/projects/esp-idf/en/latest/esp32/security/flash-encryption.html#using-host-generated-flash-encryption-key
The "Using Host Generated Flash Encryption Key" section mentioned:
It is possible to pregenerate the flash encryption key on the host computer and burn it into the ESP32’s eFuse key block.
This allows data to be pre-encrypted on the host and flashed to the ESP32 without needing a plaintext flash update.
However, the document does not mention how to flash pre-encrypted firmware at all.
I think this needs update.

@AxelLin Thanks for your suggestion in the documentation,  we will evaluate the suggestion and provide feedback. Thanks.

Hi @AxelLin,
Thanks for pointing this out and sorry for the delay in getting back to you.

However, the document does not mention how to flash pre-encrypted firmware at all.

For simplicity, we try not to descrbe every possible way to work with flash encryption. Instead, we choose some recommended workflows for Development & Production and then document these. The workflow described at the link can be summarised as:

Before first boot, pre-load the encryption key to efuse
Configure with "Development" mode
Flash plaintext binaries, the device will encrypt itself on first boot from the plaintext files and using the pre-loaded key.
Use the encrypted-flash commands for any additional flash updates.

This is the recommended workflow for using a pre-generated key for development.
ESP-IDF v3.x actually did have documentation about how to encrypt the files on the host and then flash them:
https://docs.espressif.com/projects/esp-idf/en/v3.3.1/security/flash-encryption.html#reflashing-with-pregenerated-key
This was removed in V4.0 and replaced with the "Development mode" workflow. We made this decision for two reasons:

It's simpler, the build system can do everything instead of requiring manual steps.
We found some users chose to deploy pre-encrypted binaries in production and shared the same key across all devices. We can't prevent this, but it's considered to be a security risk so it's discouraged.

(The old workflows still all work, but as I said we don't want document every possible workflow - it's too complex.)
We still want to make sure that ESP-IDF users have all of the tools they need for their product development. Can you explain why you need to flash a pre-encrypted binary, instead of using Development mode?

Hi @projectgus
I understand the risk of pre-encrypted with the same key.
However, I'd like to compare the production speed difference at the moment.
IMHO, if it's actually possible to flash pre-encrypted firmware it should
be well documented. Otherwise, people just try to dig old discussion threads
and may find out-of-date information.
Yet another issue might be related to pre-encrypted firmware:
Is it possible to do OTA with pre-encrypted firmware?
From the information on the discussion forum I understand your point is the
firmware should be in a safe place. However, it's not a perfect world.
In some case, the customer asks to allow manually upload firmware for OTA.
We would prefer to use pre-encrypted firmware rather than unencrypted firmware
for customers in such case. (Although I still don't know how to OTA pre-encrypted firmware at this moment. Is it possible?)

Hi @projectgus and @AxelLin -
Just wanted to chime in that for us in production, we're encrypting with the host PC.  We found that it was much faster to encrypt with the PC to the target than it was for the target to do the in-place encryption.  Our production flow is thus:

Generate a random flash key for the unit, and record it on the functional tester
Generate a random signing key for the unit, and record it on the functional tester
Burn in the following eFuses:

BLK1, enabling write protection only
BLK2, enabling write protection only
CRYPT_CONFIG 15
CRYPT_CNT 1, enabling write protection
ABS_DONE_0 1, enabling secure boot

Generate per-unit encrypted image and write into unit
Run functional test in unit

When the functional test completes successfully, the firmware in the unit then burns the eFuses for read protection on BLK1 and BLK2

This was a lot faster than doing the in-place encryption.  I don't have the numbers in front of me, but just wanted to chime in that it's a valid way of doing it, and IMHO better than doing the in-place encryption that's done with the example since it ends up with a faster functional test in production.
- Tim

Hi @tim-nordell-nimbelink @AxelLin ,
Thanks Tim for updating with your process and sorry to both of you for the slow turnaround.
You're right that we should document a process for pre-encrypting firmware. We're having some internal discussion about providing better support for this kind of factory workflow (while still discouraging sharing flash encryption keys). It may be some time before something is ready, unfortunately, but this issue will be updated when it is.

Although I still don't know how to OTA pre-encrypted firmware at this moment. Is it possible?)

Not supported out of the box with the current OTA APIs, unfortunately. We also plan to support a way to do encrypted OTA of this type (probably using a separate key that's protected by flash encryption inside the device), but no ETA for this I'm afraid.

Hi @projectgus -
For us, we're set for our production workflow.  However, the following may help other folks move into production.
Maybe some kind of tool that could be configured where you can give it a list of partitions, images for those partitions, including NVS encrypted partitions?  Then it could automate the generation and burning actions on the PC.  It should probably optionally save the keys utilized on the local PC, and such.  If a full tool had been available I probably would have used it, but instead I have an external bash shell script that is calling into espefuse.py (for setting keys into the eFuses), espsecure.py (for pre-encrypting the partitions with the PC, and signing the bootloader with the unique key), esptool.py (for flashing), and nvs_partition_gen.py (for encrypting the default NVS partition correctly) to accomplish the PC side manipulation required.
There are a lot of moving pieces within the flow with the current tools inside the ESP-IDF.  If a single tool combined all of those (even if it invokes external tools on your behalf) such that you could provide as parameters:

(optional) Folder to save encryption keys and digest keys into
(optional) Ability to defer marking keys as not readable
List of binaries to program partitions from.
Flags for each binary indicating how it needs to be mutated for flashing.  E.g. unencrypted, encrypted, and inputs for nvs_partition_gen.py for generating a default NVS partition

Maybe somehow partitions.csv could be incorporated into the tool for sourcing flags for how to mutate each partition as it's flashed?
Of course, some of the above is different with "secure boot v2" (namely the digest key part is effectively an known value to everyone) a similar process could probably be used.

Tim

I would also note that deferring marking the eFuse keys as non-readable should be done even with the "normal" flow until it's been programmed successfully (just as the last step within the automated process), just in case it gets interrupted for whatever reason.

Tim

Hello,
If we are using flash protection with development option with pre-generated key, why we are using "encrypted-flash" option?
I don't provide the key here!

Hi @ahmedwahdan,
I saw you posted the same question on the forum, I've replied to you there: https://esp32.com/viewtopic.php?f=2&t=20046&p=73741#p73741

Hi,
I generated a key and burned it to device.
When i enabled flash encryption, set it to development mode and run the command: idf.py flash monitor
I get errors that i burned regular plaintext(not encrypted).
When i run the command "idf.py encrypted-flash monitor" then it run ok.
Then I tried to run the flash encryption in production,
I changed in the menu to production and run the command "idf.py flash monitor".
But i got errors (running without ending):
rst:0x10 (RTCWDT_RTC_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
flash read err, 1000
ets_main.c 371
ets Jun  8 2016 00:22:57
rst:0x10 (RTCWDT_RTC_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
flash read err, 1000
ets_main.c 371
ets Jun  8 2016 00:22:57
rst:0x10 (RTCWDT_RTC_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
flash read err, 1000
ets_main.c 371
ets Jun  8 2016 00:22:57
What do i miss?
I follow this document:
https://docs.espressif.com/projects/esp-idf/en/release-v4.2/esp32/security/flash-encryption.html#flash-encryption-algorithm

@roysG I don't work for Espressif any more so I'm afraid I can't help you. As this issue has been closed, you might not get a reply - suggest maybe posting on https://esp32.com or opening a new issue to make sure someone sees it. :handshake:


ci(lint): add shell linter - Differential ShellCheck

Differential ShellCheck is a GitHub action that performs differential ShellCheck scans on shell scripts changed via PR and reports results directly in PR.
I see that your script is in great shape, but  I think that you might find the differential-shellcheck action useful. It is able to produce reports in SARIF format. GitHub understands this format and is able to display it nicely as a PR comment, and on the Files Changed tab, please see below.

Documentation is available at @redhat-plumbers-in-action/differential-shellcheck. Let me know If you are missing some feature or option. I'm always happy to extend functionality.

@jamacku : thank you for offering an idea to improve our workflow CI process! ❤
In the past, I have used ludeeus/action-shellcheck for personal shell-based GitHub CLI extensions I built and maintain.  That said, I can see why a project might adopt Differential ShellCheck:

"needed some way to verify incoming Pull Requests without getting warnings and errors about already merged and for years working code."

Given we don't have a linting process for the repository, which is largely powered by a shell script, and several key features below, I think this is something that would benefit the workflows here:

Basic shell scripting linting / testing via koalaman/shellcheck
SARIF support - warnings are visible in the Changed files tab of the Pull-Request and as comment alerts on Pull-Requests
Results displayed as Job Summaries - example

@williammartin : Any concerns or thoughts about how we might adopt this?


coveralls.io + github.com OAuth credentials need to be updated

coveralls.io isn't showing actual source code diffs for coverage reports. The exact message is:

SOURCE NOT AVAILABLE
The owner of this repo needs to re-authorize with github; their OAuth credentials are no longer valid so the file cannot be pulled from the github API.

See, for example: https://coveralls.io/builds/5735092/source?filename=src%2Funiforms%2Fmod.rs

Ugh.
I think it's ok now.

Unfortunately this still seems to be a problem as of a couple days ago:
https://coveralls.io/builds/6510103/source?filename=src%2Fsampler_object.rs


add genelist to oncoplot/oncostrip

Please add the ability for genes to take a list, rather than naming each gene of interest, when graphing oncoprints. Thanks!

The list works with oncoplot!


Implement the Observer pattern on TransactionEntry

Currently, the TransactionEntry has a direct dependency to the EventRepository,  as the TransactionEntry.commit process needs to invoke some methods from the EventRepository. This issue aims to improve the relationship between these classes with the implementation of the Observer pattern.
With this new approach we'll obtain two important improvements:

TransactionEntry will not need to know anything about the EventRepository class.
The implementation of new features are supported by transactions will be really simple to connect with the transaction changes (reservation, committing, rejection, etc.)

Closing this issue as it is moved to https://github.com/minos-framework/minos-python/issues/63


Leniently Comparing Coordinates

✨ Feature Request

Could we have some functionality in the public API that allows users to compare coordinates leniently?
Motivation

I wanted to check two cubes were ordered consistently, so put this in my script:
assert cube1.coord('realization') == cube2.coord('realization')

the assertion failed because one of the cubes had been saved as NetCDF and the other had not, so the var_names clashed.  Since Iris v3, we have functionality under the arithmetic hood to make a lenient comparison of coordinates.  It would be great if user code could directly access this functionality, so I could make a more useful assertion.

@rcomer Oohhhh lovely use case.
Seems totally possible to me. We'd need to think very carefully about how we control such lenient behaviour. Also, this is a nudge for us to perhaps start to think about other areas of iris that would benefit from lenient behaviour e.g., merge and concatenate are obvious contenders.
But this use case touches on a space that is rather powerful (and possibly surprisingly dangerous), as lenient coordinate equivalence is a BIG deal, because it occurs in many different places within iris. We'd need to think about how best to control such behaviour 🤔

I wanted this again today, so knocked this together:
def coords_equal(coord1, coord2) -> bool:
    """
    Lenient comparison of coordinates.

    """
    if not np.array_equal(coord1.points, coord2.points):
        return False

    if (coord1.has_bounds() and coord2.has_bounds() and
            not np.array_equal(coord1.bounds, coord2.bounds)):
        return False

    return coord1.metadata.equal(coord2.metadata, lenient=True)

It seems to work for my current example at least.


chore: improve patrol tests

Use patrol test variants instead of for loops.

Improve network error test by always re-enabling the network after tests, even on failure.

[ ] Tests pass

LGTM 👍


add brackets to separate the aguments in error message

I.e., a message such as
The operation + cannot accept the arguments: 30 and more
should instead be
the operation + cannot accept the arguments: [30] [and more]

this is PR #292

Fix in master


fix: CVE-2023-44487

Provides a fix for CVE-2023-44487

/lgtm
/approve

/test lvm-operator-e2e-aws-sno

/jira refresh

@jakobmoellerdev this will need to be backported to all release (4.12-4.14)

/cherry-pick release-4.14


Claude Dev - API Request Fail Loop

Hi Saoud,
Just fyi I am a beginner in GitHub, VS Code, and AI. So apologies if this is a novice issue!
I'm a stuck in a loop with the same API Requested Failed 529 Error (as shown in the attached image) and I've been retrying for about 10-15 mins. I am left with only 1 choice, which is to Start New Task since I cannot type in the prompt field (it's grayed out and not selectable) or somehow cancel the last prompt. I prefer to not Start a New Task.
What should I do?
Thank you!

Hi there! 529 indicates the server is down, this tends to happen sometimes with Anthropic. I suggest going to settings > API Provider and switching to OpenRouter in the meantime (they have higher rate limits too!)

Thank you for the quick reply!
Sorry another novice question. Do I need to pay for another API key from OpenRouter? (I'm Tier 2 in Anthropic)

Yes but if you want a free model in the meantime then I suggest Google Gemini, their flash model offers 15 request per minute for free


Publish to npm

This should be made publicly available either by overtaking the meteor-desktop npm package name, or by republishing under a new name or scope like @meteor-community/meteor-desktop.

I will try to get a new version out by the end of the month either way.


The Server receives data after stoped

Hello,
On new version v<IP_ADDRESS>, the server receives data after the server stopped as bellow (maybe it must work like that) :
Listener IP   : [<IP_ADDRESS>]
Listener Port : [9000]
Use SSL       : [y/N]? N
Server started
Command [? for help]: [SimpleTcp.Server] starting data receiver for: <IP_ADDRESS>:58177
[<IP_ADDRESS>:58177] client connected
[SimpleTcp.Server] data receiver started for client <IP_ADDRESS>:58177
[<IP_ADDRESS>:58177]: test
Command [? for help]: stop
[SimpleTcp.Server] stopped listening
[SimpleTcp.Server] stopped
Command [? for help]: [<IP_ADDRESS>:58177]: Hello!
[<IP_ADDRESS>:58177]: Test
Also on the client-side when I try to execute the disconnect command  ( ClientTest sample) it hangs on  as below :
Server IP   : [<IP_ADDRESS>]
Server Port : [9000]
Use SSL     : [y/N]? N
[SimpleTcp.Client] initializing client
[SimpleTcp.Client] connecting to <IP_ADDRESS>:9000
[SimpleTcp.Client] attempting connection to <IP_ADDRESS>:9000
[SimpleTcp.Client] connected to <IP_ADDRESS>:9000
*** Server <IP_ADDRESS>:9000 connected
Command [? for help]: send
Data: [Hello!] test
Command [? for help]: send
Data: [Hello!]
Command [? for help]: send
Data: [Hello!] Test
Command [? for help]: [<IP_ADDRESS>:9000] Test
Command [? for help]: disconnect
[SimpleTcp.Client] disconnecting from <IP_ADDRESS>:9000

Hi, stopping the server does not tear down existing connections; it only prevents new connections from being accepted.  So the behavior you experience is correct, you can still send the server data after stopping the server.
What framework/runtime and operating system are you using?  I'm not experiencing the hang on disconnect issue using the ClientTest and ServerTest project.

I just tested again and the only issue I could find was an uncaught AggregateException when running .NET Core 3.1 or above in the client DataReceiver but it wasn't affecting operation.  I updated the library to v<IP_ADDRESS> to address that, however, I have been unable to reproduce the issue with hanging on disconnect.
Please let me know what operating system and framework/runtime you are using and if possible please provide sample code to reproduce.  Cheers

I compiled on VS 2019 and .Net framework 4.8 on windows 10, 64 bit Pro version.
On my tests I did not change anything on your code.
I will do some more tests and I will come back.
Regards

Just tested on 4.8 and it worked on my end.  You may notice up to one second in delay between disconnecting and disconnected as I had to modify the internal DataReadAsync since Stream.ReadAsync doesn't obey CancellationTokenSource cancellation.

Hi Joel,
I did some test on different .net versions and on the version v<IP_ADDRESS> it hangs on my PC.

After downloading the new version <IP_ADDRESS> I do not face the same problem again but I thing there is a new issue, the client on .net 4.8, 4.6.1 and .net core 3.1 do not receives the data from server.

Regards

On .Net 5.0 it works

Hi @hakikpaci I have a fix, but I've asked @tinohager to weigh in with an opinion as well.  Please refer to https://github.com/jchristn/SuperSimpleTcp/issues/123 - closing this one out since it's a duplicate.  Thanks!

Also please revert to <IP_ADDRESS>


Add GDPR cookies notice support

@z3by I'm using https://github.com/apertureless/vue-cookie-law . Also, I pinned the version to 1.12.0 because there seems to be a bug with 1.13.x and SSR.


fix(breadcrumbs): key prop

closes  https://github.com/uber-web/baseui/issues/969
fixes https://medium.com/@robinpokorny/index-as-a-key-is-an-anti-pattern-e0349aece318

merging as is, screener was a false positive


WIP: general meta changes

added fullname to the title when on person#show

added social media meta tags to:

news#show
people#show
resources#show

added function called
set_social_meta title: '', description: '', image: ''


[SPARK-10165][SQL] Await child resolution in ResolveFunctions

Currently, we eagerly attempt to resolve functions, even before their children are resolved.  However, this is not valid in cases where we need to know the types of the input arguments (i.e. when resolving Hive UDFs).
As a fix, this PR delays function resolution until the functions children are resolved.  This change also necessitates a change to the way we resolve aggregate expressions that are not in aggregate operators (e.g., in HAVING or ORDER BY clauses).  Specifically, we can't assume that these misplaced functions will be resolved, allowing us to differentiate aggregate functions from normal functions.  To compensate for this change we now attempt to resolve these unresolved expressions in the context of the aggregate operator, before checking to see if any aggregate expressions are present.

Merged build triggered.

Merged build started.

Test build #41397 has started for   PR 8371 at commit 5b9b4ae.

Test build #41397 has finished for   PR 8371 at commit 5b9b4ae.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41397/
Test FAILed.

I think this breaks having group resolution ...

Merged build triggered.

Merged build started.

Test build #41432 has started for   PR 8371 at commit cff8b1d.

Test build #41432 has finished for   PR 8371 at commit cff8b1d.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41432/
Test FAILed.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #41439 has started for   PR 8371 at commit 827688e.

Test build #41439 has finished for   PR 8371 at commit 827688e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41439/
Test FAILed.

Merged build triggered.

Merged build started.

Merged build triggered.

Merged build started.

Test build #41468 has started for   PR 8371 at commit f00e1e9.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41467/
Test FAILed.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #41470 has started for   PR 8371 at commit a52d305.

Test build #41468 has finished for   PR 8371 at commit f00e1e9.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41468/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #41483 has started for   PR 8371 at commit ab80fad.

Test build #41470 has finished for   PR 8371 at commit a52d305.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41470/
Test FAILed.

Test build #41483 has finished for   PR 8371 at commit ab80fad.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41483/
Test PASSed.


Clone Function

Is it possible to add clone functionality? So whenever user has already fill the form, the filled form will be cloned.

hmm i think the clone function could easily be done by creating a custom event. Like focusout() on the last field to create a new set?


All of, or a subset of these colors should be white/black with transparency

#25328

VSCode Version: Code - Insiders 1.12.0-insider (48ae6e3027be560d45dd1ebfbe090d6ff77af7f0, 2017-04-26T06:32:53.505Z)
OS Version: Linux x64 4.10.0-19-generic
Extensions:

Extension
Author
Version

EditorConfig
EditorConfig
0.9.3

lorem-ipsum
Tyriar
1.0.0

sort-lines
Tyriar
1.2.0

theme-sapphire
Tyriar
0.1.0

vscode-svgviewer
cssho
1.4.0

tslint
eg2
0.12.0

git-project-manager
felipecaputo
1.3.2

md-navigate
jrieken
0.0.1

vscode-scss
mrmlnc
0.6.2

vetur
octref
0.6.3

seti-icons
qinjia
0.1.3

I went through all the colors for the generator and put the following into the bucket for using white (on dark) and black (on light) with transparency. These values should apply nicely pretty much everywhere as the color is just a lighter or darker version of their background color.
"listActiveSelectionBackground": "",
"listDropBackground": "",
"listFocusAndSelectionBackground": "",
"listFocusBackground": "",
"listHoverBackground": "",
"listInactiveFocusBackground": "",
"listInactiveSelectionBackground": "",
"editorLineHighlight": "",
"editorLineHighlightBorder": "",
"editorRangeHighlight": "", // Highlight on rows on find
"editorDragAndDropBackground": "", // DND overlay
"activityBarDragAndDropBackground": "",
"panelActiveTitleBorder": "",
"sideBarSectionHeaderBackground": "", // eg. Debug collapsible sections
"statusBarItemActiveBackground": "", // Mouse down
"statusBarItemHoverBackground": "",
"pickerGroupBorder": "",

@Tyriar I do not agree with the tree colors, we use a very strong blue when the tree has focus and a grey otherwise. I am not seeing how that works across themes that have a strong base color which is different?

Actually this may work..

@Tyriar I am dealing with the tree/list in https://github.com/Microsoft/vscode/issues/25417
I pushed fixes for
Imho pickerGroupBorder cannot use transparency, it is a strong blue color, nothing we have on the background.
Moving to @aeschli for the remaining editor colors.

@bpasero pickerGroupBorder on Dark+ is a gray?

Good point, I wanted to review all workbench colors for May to see if they could be derived and I will revisit this one 👍

@bpasero it may be good, at least for the time being, to keep derived colors in VS Code strictly to transparency and let tools built on top handle the complexity.

Role-based colors: Define a bunch of color roles such as selection, focus, hover, strong background, editor background and set colors based on that:
selection
  > sidebar selection background
strong background
  > sidebar background

I'm leaning towards using the last one in the theme generator if I can manage to categorize the majority of the colors in a nice way. But yeah, I think using the generator for experimentation would be best until we land on something solid.

closing, no action planed here


wrong closing parenthesis in CFE_MSG_CMD_HDR_INIT macro

The CFE_MSG_CMD_HDR_INIT macro's parentheses are not properly balanced.
The CFE_MSG_CMD_HDR_INIT macro contains a ')' instead of a '}' in line 70 as of Latest commit [f9112b4] on Jun 7, 2022
File: nasa/cFE/modules/msg/option_inc/default_cfe_msg_hdr_priext.h
Simple inspection shows the problem.
Opening and closing parentheses should be balanced.
line 70:  .SystemId = {0, 0) \
Matthias Bark, Boeing

There is also an unbalanced (extra) round bracket/parenthesis:


When will it be Updated?

When will it be updated to the latest version of Arangodb

Unfortunately, both of C# driver are maintained much, I asked Arangodb for official c# driver, maybe you can also request at their slack

Please note that there is the community effort here:
https://github.com/Actify-Inc/arangodb-net-standard
to create a base transport for higher layer driver.

Unfortunately, both of C# driver are maintained much, I asked Arangodb for official c# driver, maybe you can also request at their slack

Yeah guess I won’t be using Arango...

That's a shame. This driver still works, just wish that it is more actively maintained, or a more official one by arangodb. The arrangodb-net-standard is looking good as replacement once they have some more features ready.


Question

Hello,
would you be interested in a PR to expose more styling options (especially the fontsize of the text) ?
also - do you think you will ever work on the Android version?
thanks!

Hi @nonotest! Sorry for late reply. Yes I am interested in a PR to expose more styling options.
Regarding Android version, I am working on it and may take sometime to release it.


Does it work for draft genomes?

I got the following output:
○ Warning: gff file /home/user/Desktop/40_S40_L001.gff contains information for more than 1 sequence. File will be skipped.
○ Warning: gff file /home/user/Desktop/29_S29_L001.gff contains information for more than 1 sequence. File will be skipped.
Please share a sample locus_annotation_table for draft genomes.

Thanks for the quick response.
Regards,
Tanbir

No problem! I actually replied the day you asked, but somehow comment wasn't sent...
I’ll add this feature to the to-do list
Thank you for trying anyway!
Best,
Artyom


[Bug] Microscopy never tracks measurement

Describe the Bug
When I add measurement, I expect the track measurement dialog to show but it doesn't.
There is currently no way to save the measurement for microscopy.
Steps to Reproduce

Open a wsi microscopy
Add a measurement

The current behavior
Measurement tracking dialog not showing and there is no way to track
The expected behavior
Measurement tracking dialog should show
OS
macOS 12.6
Node version
16.14
Browser
Chrome Version 114.0.5735.133 (Official Build) (arm64)

I am happy to take this on if I can get some direction.

Same behavior here  (os: Debian GNU/Linux 11,  browser firefox 114.0.2 (64 bits) node 18.16.0

Bump. Any updates?

Regrettably, the mentioned feature is not supported at present and is not included in our current plans. However, we would be delighted to review your pull request if you would like to contribute.


Extract and update license and include in download tree.

Description of the feature
[ ] Extract the content of the license, update the variables and push it to the respective data model directory
[ ] Include in the download zip tree in the database
Parent #149

[ ] This should also apply to the public view of a project:

Closing this in favor of #788


.asCallback is not a function

Environment
Knex version: 0.20.11, was not present in 0.20.10
Database + version: mariadb
OS: ubuntu16
Bug
(node:1505) UnhandledPromiseRejectionWarning: TypeError: self.knex.select(...).from(...).where(...).andWhereRaw(...).then(...).asCallback is not a function
at /home/shodan/shodan/node_modules/connect-session-knex/index.js:264:5
at processTicksAndRejections (internal/process/task_queues.js:93:5)
Seems like bluebird was replaced with native promises somewhere, and asCallback was lost.

That's a bug in connect-session-knex:  https://github.com/llambda/connect-session-knex/issues/61
I suppose, this dependency update should be rolled back.

@jehy Bluebird was completely removed from knex codebase and we are not supporting its custom methods anymore. Still need to update parts of documentation, but there are no plans to support these functions going forward.

Bluebird was not part of an official knex api since 0.18.0: https://github.com/knex/knex/blob/master/UPGRADING.md

I think we should add link to that document near to changelogs to documentation page. I didn't see any mentions about it there.

@elhigu Good idea, will do.


Add ability to replay (previous track)

Add replay command to move to the next track. Suggested command:
spty player replay

Fix / implements this in 0.2.0


V1.2a change person to student

Codecov Report

Merging #53 (402e0c8) into dev (ac4c437) will decrease coverage by 0.12%.
The diff coverage is 78.84%.

@@             Coverage Diff              @@
##                dev      #53      +/-   ##
============================================
- Coverage     71.45%   71.33%   -0.13%
- Complexity      377      385       +8
============================================
  Files            68       70       +2
  Lines          1156     1193      +37
  Branches        112      117       +5
============================================
+ Hits            826      851      +25
- Misses          302      308       +6
- Partials         28       34       +6

Impacted Files
Coverage Δ

...main/java/seedu/address/commons/core/Messages.java
0.00% <ø> (ø)

.../seedu/address/logic/parser/FindCommandParser.java
100.00% <ø> (ø)

...main/java/seedu/address/model/student/Address.java
80.00% <ø> (ø)

...c/main/java/seedu/address/model/student/Email.java
80.00% <ø> (ø)

...rc/main/java/seedu/address/model/student/Name.java
80.00% <ø> (ø)

...c/main/java/seedu/address/model/student/Phone.java
80.00% <ø> (ø)

.../java/seedu/address/model/util/SampleDataUtil.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/MainWindow.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/StudentCard.java
0.00% <0.00%> (ø)

...c/main/java/seedu/address/ui/StudentListPanel.java
0.00% <0.00%> (ø)

... and 28 more

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 ac4c437...402e0c8. Read the comment docs.