Add documentation and example for on-expire attribute

Thanks @jpmckearin !

Hey guys, looking for some help with the on-expire attr.
When I get an expiration event I reset the response to null like in the example:
$scope.response = null
After this, in both the demo, and in my code, if a user attempts another recaptcha, there is an error in the console:
cannot read property "then" of null...
Do you guys have any tips on how you handle the "expire" event?
@jpmckearin @iambrosi

@trickpattyFH20 There is no need to vcRecaptchaService.reload() when the timeout callback is called, it is just to notify the developer of the timeout so they can display messages or do whatever.  Internally, the now invalid captcha response is cleared out of model and if the captcha is within a form control it will mark the form as invalid.  Google internally resets the captcha.

@TheSharpieOne the functionality I was looking to achieve was the have a new empty typeahead shown after a timeout on the re-captcha key on google's end. no sense in making a user refresh the page to get a new typeahead when I can do it asynchronously in the timeout callback

@trickpattyFH20 I see, you wanted a brand-new recaptcha instance rather than reusing the old one (by reuse, I mean "unchecking" the checkbox and allowing the user to "check" the same checkbox again).  But really, vcRecaptchaService.reload() is not a "reload", it just calls recaptcha's reset(), which resets the same checkbox rather than creating a new instance.
Also, just fyi, without this module, or any additional code, the functionality you want happens automatically in recaptcha's code... give it a try.
Just don't want to make people think they have to use the callback to reset recaptcha, because they don't.  Resetting the captcha happens automatically, asynchronously, and without the need to reload/refresh the page.  The callback is just a convenience to let the developer be notified that the valid recaptcha has timed out.

ah ok, I didn't know the vanilla recaptcha code included that functionality I just used this module for the convenience of it, thank you! @TheSharpieOne


Implicit multiplication bug

If you have the following formula "3x+1" and you disable the implicitMultiplication, I'll have, not an exception, but an error telling you that you have too much operators !!

Current version of source throw exception.
@Test
public void testImplicitMultiplicationOffVariable() {
Expression e = new ExpressionBuilder("3x+1").variable("x").implicitMultiplication(false).build();
        e.evaluate();
    }
this is the exception thrown:

java.lang.IllegalArgumentException: No value has been set for the setVariable 'x'.

Thanks for the update @bvedrenne .  It seems like that exception is only being thrown because the variable x has not been assigned any value.  Your test could be rewritten as:
@Test
public void testImplicitMultiplicationOffVariable() {
Expression e = new ExpressionBuilder("3x+1").variable("x").implicitMultiplication(false).build().setVariable("x", 2);
e.evaluate();
}

This would assign x the arbitrary value of 2.  However, since implicit multiplication is off, an IllegalArgumentException is still thrown, but this time it says that there is an invalid number of arguments passed to the function, which is a more accurate description of the exception thrown.
Correct me if I'm wrong, but it seems to me like that was the only issue with the code.  Now, the correct exception is being thrown.

Thanks for the update @bvedrenne and @shuvamc019 for your answer.
To my mind, I don't think throwing an exception that says there is an invalid number of arguments passed to the function, because we don't use a function (exp4j function) in our formula 3x+1.
The more accurate to me is to throw an UnknownFunctionOrVariableException. What do you think ?


New messages add to the bottom of the list (I provided the solution, can you please implement?)

can you add the below method to be able to pass in an index so that the messages load to the back of the list?
addAll(int index, ArrayList chatMessages){
this.chatMessages.addAll(index, chatMessages);
notifyDataSetChanged();
}
(beautiful UI by the way, thanks so much)

Here is my ugly ugly solution but it is what it is until the implementation of the code above:

create this object:
private List<List> messagesGroupsByPage = = new ArrayList<List>();
add the messages to the group
messagesGroupsByPage.add(pageGroup-1, response.body().getMessages());
clear the actual list before readding
chatView.clearMessages();
add
for (int i1 = 0; i1<pageGroup ; i1++){
chatView.addMessages(chatMessageArrayListConverter(messagesGroupsByPage.get(pageGroup-1-i1)));
}
scroll to position and:
pageNumber++

timigod, again we are all thankful and will be more thankful for the improvement

Hi @ousanmazCT, can you please clarify what exactly you're trying to achieve. I'm having a hard time understanding the problem

Thanks. This is much clearer.
I think what we should be doing in this case is creating a new list from the old and new lists and passing it to ChatView because I can see how mutating the internal list in this way can introduce bugs. I think adding a method to swap the lists would create the flexibility to achieve this.
What do you think?

I think that is fine but my only concern would be the UI. When we reset the list, and when I say reset I mean replacing the old list with a new one, the UI will want to reset itself as well.
If UI wants to reset, instead of the scroll bar staying where it is, it is going to reset and go all the way to the begining, therefore not a smooth experience.
This is the ugly code I implemented at the moment. I am replacing the old list with a new one and it is resetting the UI, making the scroll bar go all the way down to the beginning.
This is why I believe adding the new set of messages to the end of the messagesList (of ChatView) is a better option.
What do you think? I don't know if the current code will accommodate appending the new messages to the end of the list.

Let's try it out and evaluate the options.
We'll most likely have to switch to RecyclerView so we can achieve a smoother experience.
Can you please create a pull request with the proposed fix and the we'll go from there.


Fix #1554 - issue with onedrive site storage quota

Type

[x] Bug Fix
[ ] New Feature
[ ] Sample

Related Issues?
Fixes #1554
What is in this Pull Request ?
Fix the one drive storage quota issue.
We need to get the storage quota level of the personal site itself and not the max quota.
@KoenZomers - need your help here :)

Can we optimize it more to retrieve the site ?
I had to add 1024 * 1024 , not sure why, but only after that it worked. Seems a bit flaky , need something more robust, any ideas ?

Yikes, fetching ALL OneDrive sites in order to do this? There must hopefully be a more efficient way of achieving this. How can this be reproduced anyway? I couldn't repro it before. Is the catch here to set it to something less than the tenant configured maximum?

Yeah, the filter with URL didnt work, so had to fetch all sites :(
To repro this, you can just do:
Set-PnPUserOneDriveQuota -Account<EMAIL_ADDRESS>-Quota<PHONE_NUMBER> -QuotaWarning<PHONE_NUMBER>

and then use the above code.

@KoenZomers  - managed to figure out an efficient way to filter the site for a specific user. Looks good to me now :)


Rtl support

Is your feature request related to a problem? Please describe.
In Rtl direction lane drag has problem.

Hey bro! I forked the project to support RTL, feel free to use :)

Hey bro! I forked the project to support RTL, feel free to use :)
react-trello-rtl

@itaybarnoy
Thank you very much...


Social Card content adjustments

https://presentations.cncf.io/presentations/keynote-cloud-native-101-motor-city-edition/
Can we drop the "- CNCF Presentations" text since we only want the title? Can we also put the presenter's names on there underneath the title in smaller text? Thanks!

I have already dropped the "- CNCF Presentations" but some of the old images had been cached. I've just purged the cache so you shouldn't see them anymore.
It's not currently possible to add the presenters' names beneath the title. There is, however, a version 2 of the plugin in the works which may provide this functionality. I'll keep an eye out for that.


'isCacheFirst' not working

Hi,
in my previous PR I've implemented the isCacheFirst functionality, however it's not working as expected. I haven't tested the implementation well enough. This time I did some tests with debugging and it seems to work correct. Let me know what you think, here's my PR:
https://github.com/kosukesaigusa/geoflutterfire_plus/pull/203
PS.
Sorry for the inconvenience, I promise to test my code better next time

@Kurczaak
Thank you for creating a PR again!
Now it's released in pub.dev!
https://pub.dev/packages/geoflutterfire_plus/changelog


Lua Variables for BPM & Character/Puchichara metadata

I would like to be able to enhance Backgrounds with more values, one especially being BPM.
In addition, it would be cool to grab the metadata of currently equipped Characters/Puchicharas & change how the background displays (i.e. you've equipped an epic character & have some colors of the background change to blue, or have the background write out your character's/puchichara's name on a display board)

BPM added

Rarity metadata added, if the puchichara/character names are deemed important in the future please add a new ticket (to avoid having too much long/big tickets that stay openned)


Problem running V2

I updated acos, camviewer, agt, and mantisapi:

acos-<IP_ADDRESS>_dev.deb
agt-<IP_ADDRESS>_dev.deb
camviewer-<IP_ADDRESS>_dev.deb
mantisapi-<IP_ADDRESS>_dev.deb

Then I ran start.sh:

ssh<EMAIL_ADDRESS>"ps aux | grep acos |grep -v grep"
ubuntu    2778  128  3.2 7145868 131352 ?      Sl   06:28   0:03 acosd
ssh<EMAIL_ADDRESS>"ps aux | grep acos |grep -v grep"
ubuntu    2771  107  3.2 7159132 133424 ?      Sl   06:28   0:04 acosd
ssh<EMAIL_ADDRESS>"ps aux | grep acos |grep -v grep"
ubuntu    2769  100  3.2 7145900 133632 ?      Sl   06:28   0:05 acosd
ssh<EMAIL_ADDRESS>"ps aux | grep acos |grep -v grep"
ubuntu    2767  117  3.3 7147052 138488 ?      Sl   06:28   0:05 acosd
ssh<EMAIL_ADDRESS>"ps aux | grep acos |grep -v grep"
ubuntu    2714  112  3.5 7145900 145636 ?      Sl   06:28   0:06 acosd
ssh<EMAIL_ADDRESS>"ps aux | grep acos |grep -v grep"
ubuntu    2728  127  3.4 7145932 140344 ?      Sl   06:28   0:07 acosd
ssh<EMAIL_ADDRESS>"ps aux | grep acos |grep -v grep"
ubuntu    2747  140  2.6 7103924 109108 ?      Sl   06:28   0:09 acosd
ssh<EMAIL_ADDRESS>"ps aux | grep acos |grep -v grep"
ubuntu    2747  127  3.6 7149692 149680 ?      Sl   06:28   0:08 acosd
ssh<EMAIL_ADDRESS>"ps aux | grep acos |grep -v grep"
ubuntu    2725  124  3.4 7145868 140024 ?      Sl   06:28   0:09 acosd
ssh<EMAIL_ADDRESS>"ps aux | grep acos |grep -v grep"
ubuntu    2848 94.0  2.3 6651160 96288 ?       Sl   06:28   0:08 acosd
Then ./V2
===============================================
Welcome to V2, the Aqueti Visual Cortex
Command server listening on port 9999
using .//camera_1181053543
Command server listening on port 10000
Command server listening on port 10001
Command server listening on port 10002
Command server listening on port 10003
Command server listening on port 10004
Command server listening on port 10005
Command server listening on port 10006
Command server listening on port 10007
Command server listening on port 10008
Command server listening on port 10009
Command server listening on port 10010
Command server listening on port 10011
Command server listening on port 10012
Command server listening on port 10013
Command server listening on port 10014
Microcamera metrics:
Currently receiving data: false
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
Microcamera metrics:
Currently receiving data: false
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
mcam<PHONE_NUMBER>: 	received: 0	   added: 0
Then ./Nous:
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
Corrupt JPEG data: premature end of data segment
From nohup:
ERROR: ArgusConsumerBase.cpp::mainLoop: 226: new frame time should be greater than previous max time. max time:<PHONE_NUMBER>704277248 timestamp:<PHONE_NUMBER>571429
ERROR:
ERROR:
ERROR: ArgusConsumerBase.cpp::mainLoop: 226: new frame time should be greater than previous max time. max time:<PHONE_NUMBER>709726720 timestamp:<PHONE_NUMBER>570252
ERROR:
ERROR: ArgusConsumerBase.cpp::mainLoop: 226: new frame time should be greater than previous max time. max time:<PHONE_NUMBER>709713664 timestamp:<PHONE_NUMBER>573733
ERROR:
ERROR: ArgusConsumerBase.cpp::mainLoop: 226: new frame time should be greater than previous max time. max time:<PHONE_NUMBER>704252928 timestamp:<PHONE_NUMBER>605318
ERROR:
ERROR: ArgusConsumerBase.cpp::mainLoop: 226: new frame time should be greater than previous max time. max time:<PHONE_NUMBER>742930176 timestamp:<PHONE_NUMBER>604744
ERROR:
ERROR: ArgusConsumerBase.cpp::mainLoop: 226: new frame time should be greater than previous max time. max time:<PHONE_NUMBER>704277248 timestamp:<PHONE_NUMBER>604419
And nohup report after I ran Nous, we can stream live video through nous:
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():38: Received add connection interrupt from <IP_ADDRESS>:9002
INFO: DataTransmitter.cpp::addConnection:104: Connecting to <IP_ADDRESS>:9002
Command server listening on port 10010
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():164: Setting tiling filter for <IP_ADDRESS>:9002 to 2
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
/home/ubuntu/bin/moveFocusMotor.py 1 50 &
im initializing here
['1', '50']
I will move motor 1 50 steps forward
im initializing here
setting sec per step at 0.003
Double coil steps
0.003  sec per step
/home/ubuntu/bin/moveFocusMotor.py 1 50 &
im initializing here
['1', '50']
I will move motor 1 50 steps forward
im initializing here
setting sec per step at 0.003
Double coil steps
0.003  sec per step
/home/ubuntu/bin/moveFocusMotor.py 1 50 &
im initializing here
['1', '50']
I will move motor 1 50 steps forward
im initializing here
setting sec per step at 0.003
Double coil steps
0.003  sec per step
/home/ubuntu/bin/moveFocusMotor.py 1 50 &
im initializing here
['1', '50']
I will move motor 1 50 steps forward
im initializing here
setting sec per step at 0.003
Double coil steps
0.003  sec per step
/home/ubuntu/bin/moveFocusMotor.py 1 50 &
im initializing here
['1', '50']
I will move motor 1 50 steps forward
im initializing here
setting sec per step at 0.003
Double coil steps
0.003  sec per step
/home/ubuntu/bin/moveFocusMotor.py 1 50 &
im initializing here
['1', '50']
I will move motor 1 50 steps forward
im initializing here
setting sec per step at 0.003
Double coil steps
0.003  sec per step
/home/ubuntu/bin/moveFocusMotor.py 1 50 &
im initializing here
['1', '50']
I will move motor 1 50 steps forward
im initializing here
setting sec per step at 0.003
Double coil steps
0.003  sec per step
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
/home/ubuntu/bin/moveFocusMotor.py 2 50 &
im initializing here
['2', '50']
I will move motor 2 50 steps forward
im initializing here
setting sec per step at 0.003
Double coil steps
0.003  sec per step
/home/ubuntu/bin/moveFocusMotor.py 2 50 &
im initializing here
['2', '50']
I will move motor 2 50 steps forward
im initializing here
setting sec per step at 0.003
Double coil steps
0.003  sec per step
/home/ubuntu/bin/moveFocusMotor.py 2 50 &
im initializing here
['2', '50']
I will move motor 2 50 steps forward
im initializing here
setting sec per step at 0.003
Double coil steps
0.003  sec per step
/home/ubuntu/bin/moveFocusMotor.py 2 50 &
im initializing here
['2', '50']
I will move motor 2 50 steps forward
im initializing here
setting sec per step at 0.003
Double coil steps
0.003  sec per step
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
/home/ubuntu/bin/moveFocusMotor.py 1 50 &
im initializing here
['1', '50']
I will move motor 1 50 steps forward
im initializing here
setting sec per step at 0.003
Double coil steps
0.003  sec per step
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
/home/ubuntu/bin/moveFocusMotor.py 1 50 &
im initializing here
['1', '50']
I will move motor 1 50 steps forward
im initializing here
setting sec per step at 0.003
Double coil steps
0.003  sec per step
/home/ubuntu/bin/moveFocusMotor.py 1 50 &
im initializing here
['1', '50']
I will move motor 1 50 steps forward
im initializing here
setting sec per step at 0.003
Double coil steps
0.003  sec per step
INFO: DataTransmitter.cpp::operator():146: Stopping streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
INFO: DataTransmitter.cpp::operator():131: Starting streaming to <IP_ADDRESS>:9002 from<PHONE_NUMBER>
/home/ubuntu/bin/moveFocusMotor.py 2 50 &
im initializing here
['2', '50']
I will move motor 2 50 steps forward
im initializing here
setting sec per step at 0.003
Double coil steps
0.003  sec per step
/home/ubuntu/bin/moveFocusMotor.py 2 50 &
im initializing here
['2', '50']
I will move motor 2 50 steps forward
im initializing here
setting sec per step at 0.003
Double coil steps
0.003  sec per step
/home/ubuntu/bin/moveFocusMotor.py 2 50 &
im initializing here
['2', '50']
I will move motor 2 50 steps forward
im initializing here
setting sec per step at 0.003
Double coil steps
0.003  sec per step

I believe this issue is resolved, correct?


Add this package to mason.nvim registry

I am using AstroNvim which uses mason.nvim as a Lsp package manager.
Can this package be registered to the mason registry so that I can install it through mason?

@ArbazIrshad I don't think that makes much sense. Mason is for installing packages, not plugins. This is a plugin, so should be installed with a plugin manager like packer. It doesn't ship any binaries or anything like that. I believe this is likely a misunderstanding.

Closing this out since my comment should answer this question and I've heard nothing back

@akinsho I can't install this in astrovim due to it not existing on mason.nvim.

@bit-void as I mentioned before, this is a neovim plugin, not a binary or some other type of tool that mason.nvim would handle. mason.nvim does not install plugins like this. I think you as the OP did have misunderstood how this plugin works and how the neovim ecosystem generally works. Please raise an issue in astrovim's repository about any issues relating to using it with this plugin. As I mentioned, what you are describing doesn't really make sense.

@akinsho You're probably right, just struggling. For any future people that stumble on this, I found a convenient snippet on astrovim's wiki on how to configure this plugin: https://astronvim.github.io/Recipes/advanced_lsp


Typescript - Make settings available on created user manager

Hi,
I'm trying to add unit tests for a factory that creates UserManager objects. In the Javascript world, I may call directly userManager.settings.xyz to fetch the value of a given setting. It would be nice if I could do the same in TypeScript.
Thanks in advance!

I don't follow what you're asking to be done. Can you elaborate?


Is it possible to show more than text?

I'm hoping to find a way to show images in the lower left corner if they are chosen in ProPresenter. I'm absolutely loving what you have done so far, it is a great asset to our church.

This is not planned to be implemented at the moment.


Allow changing the container environment at runtime

It is sometimes useful to be able to use the production environment in a debug build to test something. Currently, the environment is set to .development automatically for debug builds, and to .production for release builds, which I think is a great default, but it would be neat if the user of the library could override that manually if they want to.
If the idea is to replicate the CloudKit framework more closely, then this change could be provided at compile time with a flag different from DEBUG, which would take the place of the entitlement in the original framework.
Updated by Eric:

[x] Dynamic environment selection via initializer
[x] Dynamic environment selection via configuration

Added to CKWSContainer.Configuration in 1c3d317d486419eaab30a679056ad7c853ac1fa0
Closing out.


Fixed e2e tests for alloydb module

Fixed issue with AlloyDB e2e tests. I had to add ignore_changes due to a bug in the API (not yet managed in the provider) causing an attribute not being set and plan never aligned with infrastructure state.

I applicable, I acknowledge that I have:

[x] Read the contributing guide
[x] Ran terraform fmt on all modified files
[x] Regenerated the relevant README.md files using tools/tfdoc.py
[x] Made sure all relevant tests pass

E2E run


Plugin Included

Theme must not include plugin. A theme can recommend plugins but not include those plugins in the theme code.
Check maera/includes/plugins/timber-library

Note that I'd say this is OK because this actually a framework/library.  It'd also be necessary to do if the Timber Library is required for the theme to function because core WP doesn't have a dependency system.  I'm just getting into the review though and will follow up if necessary.

Required plugins have been removed. While it's still not optimal of course to require a plugin in this instance that is needed. But they've been removed as embedded plugins which keeps things cleaner and resolves some odd glitches with Timber not being detected as already active, resulting in a loop.


On MacOSX, Mac M Hardware (ARM), a segmentation fault happened with YDF when pyarrow is installed

Setup : MacOSX 13 or 14, Mac M hardware
Prerequisite : Install miniforge3
% conda create --name ydfpandasissue
% conda activate ydfpandasissue
% conda install python=3.10
% conda install pandas
% pip install ydf-0.2.0-cp310-cp310-macosx_13_0_arm64.whl

When running this program (ydf_test.py), it works.
import ydf
import pandas as pd
import numpy as np

dataset = {
    "x1": np.array([0, 0, 0, 1, 1, 1]),
    "x2": np.array([1, 1, 0, 0, 1, 1]),
    "y": np.array([0, 0, 0, 0, 1, 1]),
}

model = ydf.CartLearner(label="y", min_examples=1, task=ydf.Task.CLASSIFICATION).train(dataset)
print(model.describe())

Now install pyarrow from conda or pip the result is the same:  it fails
Only the error message is different.
% conda install pyarrow
% python ydf_test.py
zsh: segmentation fault  python ydf_test.py

% conda uninstall pyarrow
% pip install pyarrow
% python ydf_test.py
libc++abi: terminating due to uncaught exception of type std::__1::system_error: mutex lock failed: Invalid argument
zsh: abort      python ydf_test.py

Note that pyarrow is mandatory when we work on big tabular dataset stored in parquet files.

Thank you for the detailed report, I will have a look

Similar issue happened with tensorflow_decision_forests.
After installing tensorflow and tensorflow_decision_forests from pip (as tfdf for ARM on conda is not available), in the same config as above, the following error happened (here python terminal):
Python 3.10.13 | packaged by conda-forge | (main, Dec 23 2023, 15:35:25) [Clang 16.0.6 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import tensorflow_decision_forests as tfdf
>>> import ydf
[mutex.cc : 453] RAW: Lock blocking 0x600001892898   @

I had the same issue, but i failed to make the connection to ydf.
As a temporary workaround, i switched to fastparquet, which is the other library pandas supports to read parquet files. This one works fine for me.

But the issue is still there when importing tensorflow or tensorflow_decision_forests.
We have utils libs importing tensorflow so it makes it crash with :
libc++abi: terminating due to uncaught exception of type std::__1::system_error: mutex lock failed: Invalid argument
zsh: abort      python ydf_test.py

To give some preliminary findings from the crash logs:

A protobuf incompatibility is the source of the crash. Both (conda-installed) Pyarrow and (pip-installed) YDF depend on protobuf.
AFAICT, pyarrow links dynamically against libprotobuf 25.2. Unfortunately, there's a symbol overlap, where libprotobuf calls ydf. ydf has protobuf24.3 statically linked. Since the two versions don't match, there's a crash
There has to be an easy way to prevent this mess during compilation (suggestions anyone?) - I imagine one would just have to instruct ydf not to expose protobuf symbols that confuse libprotobuf
Very dirty solution (attached, UNTESTED very experimental): If you compile ydf with protobuf 25.2 it actually seems to work. But we obviously cannot keep ydf in sync with every protobuf that's out there. ydf-0.2.0-cp310-cp310-macosx_14_0_arm64.whl.zip
I might also be that this is a conda-specific type of issue. I'd strongly prefer not maintaining a conda package alongside a pip package at this point though
I'll have to look into TF-DF separately

Nice, we manage package consistency with conda but inside a conda env we can also install packages with pip when needed. Tomorrow I will try by using pip only to check.
Thanks for your help

Hi,
For the issue with pyarrow, thanks to your indication it's resolved just by forcing protobuf to 4.24.3, even installing protobuf with conda is ok and now it works.
The strange thing is that, even if ydf has protobuf24.3 statically linked, pip install the very last 4.25.3 version. There is no strict requirement to force the protobuf version to 4.24.3 when installing ydf from pip, just protobuf>=3.14, maybe it should be modified ?
Anyway, by doing it manually, it works now.
Not the same issue for TF-DF, it still crash, so I cannot even use model.to_tensorflow_saved_model(path) function.

I believe this issue has finally been solved for good with version 0.7.0. Closing this but feel free to re-open if there are still issues.


Tmp dir location

This allows for custom locations of tmp dirs for usage on clusters such as npk where each node has its own storage and if you don't use that the main filesystem will be hammered. Also includes the fix that I already made a pull request for that fixes argparse.

--tempDir whould be --tmpDir, since that how it's spelled on Unix ;-)
While an explicit override is useful, it should use the TMPDIR
environment variable by default.  This should be set on our
clusters.  If it is use TMPDIR, we need to investigate why it's not
set on the cluster.
ifiddes<EMAIL_ADDRESS>writes:

This allows for custom locations of tmp dirs for usage on clusters such as npk
where each node has its own storage and if you don't use that the main
filesystem will be hammered. Also includes the fix that I already made a pull
request for that fixes argparse.
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
You can merge this Pull Request by running
git pull https://github.com/ifiddes/jobTree tmp_dir_location
Or view, comment on, or merge it at:
https://github.com/benedictpaten/jobTree/pull/25
Commit Summary
• don't want to pass parser here, it doesn't work
• allows user to set tmp dir location
File Changes
• M src/jobTreeRun.py (8)
• M src/jobTreeSlave.py (34)
Patch Links:
• https://github.com/benedictpaten/jobTree/pull/25.patch
• https://github.com/benedictpaten/jobTree/pull/25.diff
—
Reply to this email directly or view it on GitHub.*

https://i.imgur.com/c7NJRa2h.jpg
On Mon, Jan 5, 2015 at 9:56 PM, Mark Diekhans<EMAIL_ADDRESS>wrote:

--tempDir whould be --tmpDir, since that how it's spelled on Unix ;-)
While an explicit override is useful, it should use the TMPDIR
environment variable by default. This should be set on our
clusters. If it is use TMPDIR, we need to investigate why it's not
set on the cluster.
ifiddes<EMAIL_ADDRESS>writes:

This allows for custom locations of tmp dirs for usage on clusters such
as npk
where each node has its own storage and if you don't use that the main
filesystem will be hammered. Also includes the fix that I already made a
pull
request for that fixes argparse.

━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

You can merge this Pull Request by running
git pull https://github.com/ifiddes/jobTree tmp_dir_location
Or view, comment on, or merge it at:
https://github.com/benedictpaten/jobTree/pull/25
Commit Summary
• don't want to pass parser here, it doesn't work
• allows user to set tmp dir location
File Changes
• M src/jobTreeRun.py (8)
• M src/jobTreeSlave.py (34)
Patch Links:
• https://github.com/benedictpaten/jobTree/pull/25.patch
• https://github.com/benedictpaten/jobTree/pull/25.diff
—
Reply to this email directly or view it on GitHub.*

—
Reply to this email directly or view it on GitHub
https://github.com/benedictpaten/jobTree/pull/25#issuecomment-68829517.

Both spellings? no, that is just adding unnecessary arguments.  Sorry the
unix inventors wanted to save one extra character.
Both --tmpDir and TMPDIR env, yes, it should be a hierarchy
 --tmpDir overrides TMPDIR, defaults to /tmp

Dent Earl<EMAIL_ADDRESS>writes:

https://i.imgur.com/c7NJRa2h.jpg
On Mon, Jan 5, 2015 at 9:56 PM, Mark Diekhans<EMAIL_ADDRESS>wrote:

--tempDir whould be --tmpDir, since that how it's spelled on Unix ;-)
While an explicit override is useful, it should use the TMPDIR
environment variable by default. This should be set on our
clusters. If it is use TMPDIR, we need to investigate why it's not
set on the cluster.
ifiddes<EMAIL_ADDRESS>writes:

This allows for custom locations of tmp dirs for usage on clusters such
as npk
where each node has its own storage and if you don't use that the main
filesystem will be hammered. Also includes the fix that I already made a
pull
request for that fixes argparse.

━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

You can merge this Pull Request by running
git pull https://github.com/ifiddes/jobTree tmp_dir_location
Or view, comment on, or merge it at:
https://github.com/benedictpaten/jobTree/pull/25
Commit Summary
• don't want to pass parser here, it doesn't work
• allows user to set tmp dir location
File Changes
• M src/jobTreeRun.py (8)
• M src/jobTreeSlave.py (34)
Patch Links:
• https://github.com/benedictpaten/jobTree/pull/25.patch
• https://github.com/benedictpaten/jobTree/pull/25.diff
—
Reply to this email directly or view it on GitHub.*

—
Reply to this email directly or view it on GitHub
https://github.com/benedictpaten/jobTree/pull/25#issuecomment-68829517.

—
Reply to this email directly or view it on GitHub.*

I have added a commit to fix 'temp' to 'tmp'.
I don't know who is in charge of npk (behind the cancer firewall), but that cluster does not have $TMPDIR set:
Ians-MacBook-Air:~ ifiddes$ ssh npk
Last login: Tue Jan  6 08:46:05 2015 from <IP_ADDRESS>
Rocks 6.1 (Emerald Boa)
Profile built 20:37 12-Feb-2014
Kickstarted 13:12 12-Feb-2014
-bash-4.1$ echo $TMPDIR
-bash-4.1$ ssh kkr18u56
Last login: Tue Jan  6 08:46:34 2015 from npk.local
Rocks Compute Node
Rocks 6.1 (Emerald Boa)
Profile built 15:22 04-Mar-2014
Kickstarted 15:52 04-Mar-2014
-bash-4.1$ echo $TMPDIR
-bash-4.1$
And the attached filesystem (depot4) is NFS. So if I submit a few hundred jobs that all are writing to depot4 it slows to a crawl. Each node has a local 256gb hard drive (/scratch/) but this is not set as $TMPDIR on the node. I don't think it would be straight forward to have each jobTree job set it before proceeding with the job, would it?

Mail<EMAIL_ADDRESS>to request it, you can CC me...
ifiddes<EMAIL_ADDRESS>writes:

I have added a commit to fix 'temp' to 'tmp'.
I don't know who is in charge of npk (behind the cancer firewall), but that
cluster does not have $TMPDIR set:
Ians-MacBook-Air:~ ifiddes$ ssh npk
Last login: Tue Jan 6 08:46:05 2015 from <IP_ADDRESS>
Rocks 6.1 (Emerald Boa)
Profile built 20:37 12-Feb-2014
Kickstarted 13:12 12-Feb-2014
-bash-4.1$ echo $TMPDIR
-bash-4.1$ ssh kkr18u56
Last login: Tue Jan 6 08:46:34 2015 from npk.local
Rocks Compute Node
Rocks 6.1 (Emerald Boa)
Profile built 15:22 04-Mar-2014
Kickstarted 15:52 04-Mar-2014
-bash-4.1$ echo $TMPDIR
-bash-4.1$
And the attached filesystem (depot4) is NFS. So if I submit a few hundred jobs
that all are writing to depot4 it slows to a crawl. Each node has a local 256gb
hard drive (/scratch/) but this is not set as $TMPDIR on the node. I don't
think it would be straight forward to have each jobTree job set it before
proceeding with the job, would it?
—
Reply to this email directly or view it on GitHub.*

Both spellings? no, that is just adding unnecessary arguments.

... not sure if srs or trolling >.>
Dent
On Tue, Jan 6, 2015 at 7:56 AM, Mark Diekhans<EMAIL_ADDRESS>wrote:

Both spellings? no, that is just adding unnecessary arguments. Sorry the
unix inventors wanted to save one extra character.
Both --tmpDir and TMPDIR env, yes, it should be a hierarchy
--tmpDir overrides TMPDIR, defaults to /tmp
Dent Earl<EMAIL_ADDRESS>writes:

https://i.imgur.com/c7NJRa2h.jpg
On Mon, Jan 5, 2015 at 9:56 PM, Mark Diekhans<EMAIL_ADDRESS>

wrote:

--tempDir whould be --tmpDir, since that how it's spelled on Unix ;-)
While an explicit override is useful, it should use the TMPDIR
environment variable by default. This should be set on our
clusters. If it is use TMPDIR, we need to investigate why it's not
set on the cluster.
ifiddes<EMAIL_ADDRESS>writes:

This allows for custom locations of tmp dirs for usage on clusters
such
as npk
where each node has its own storage and if you don't use that the
main
filesystem will be hammered. Also includes the fix that I already
made a
pull
request for that fixes argparse.

━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

You can merge this Pull Request by running
git pull https://github.com/ifiddes/jobTree tmp_dir_location
Or view, comment on, or merge it at:
https://github.com/benedictpaten/jobTree/pull/25
Commit Summary
• don't want to pass parser here, it doesn't work
• allows user to set tmp dir location
File Changes
• M src/jobTreeRun.py (8)
• M src/jobTreeSlave.py (34)
Patch Links:
• https://github.com/benedictpaten/jobTree/pull/25.patch
• https://github.com/benedictpaten/jobTree/pull/25.diff
—
Reply to this email directly or view it on GitHub.*

—
Reply to this email directly or view it on GitHub
<
https://github.com/benedictpaten/jobTree/pull/25#issuecomment-68829517>.

—
Reply to this email directly or view it on GitHub.*

—
Reply to this email directly or view it on GitHub
https://github.com/benedictpaten/jobTree/pull/25#issuecomment-68883798.

It's simple to add extra aliases to argparse options, and might be worth doing.
I'm not sure about the /scratch/tmp default, though, as not everyone necessarily has a /scratch mount.

R.E. the temp vs tmp debate: when you request a temporary dir from a Target in scriptTree, the method is called getGlobalTempDir()
I agree that setting a default value may be a bad idea.

mktemp (a coreutil) spells it like "temp" as well :) I really don't think it matters.
I think /tmp is the only sane default (if we can't get anything from env variables). It happens to be terrible for our machines because we have , but it's there on every unix system.
I think the common environment variables that people check are TMPDIR, TMP, and TEMP.
PS: should I be merging these pull requests? I have commit access.

It's not a rush to merge, I got it working on npk by just having cluster admin set the tmpdir variable. When I have a chance I will remove the default setting
Ian Fiddes

On Feb 11, 2015, at 12:16 PM, Joel Armstrong<EMAIL_ADDRESS>wrote:
mktemp (a coreutil) spells it like "temp" as well :) I really don't think it matters.
I think /tmp is the only sane default (if we can't get anything from env variables). It happens to be terrible for our machines because we have , but it's there on every unix system.
I think the common environment variables that people check are TMPDIR, TMP, and TEMP.
PS: should I be merging these pull requests? I have commit access.
—
Reply to this email directly or view it on GitHub.


maxvote and mintvote wrong in config

both seem to be wrong

Actually the amounts are correct just the variable names are a bit confusing. Fixed.


URL rejected: No host part in the URL

Hey Katy,
Is there any way that we can get around this error?
https://github.com/flamedfury/metadata-library/actions/runs/7178793959/job/19547671628
~fLaMEd

Hey hey! The problem here is that the meta tag for the OG image is not absolute and so it fails when attempting to download. I can look into if bookmark-action can evaluate if the image can be downloaded and skip it if not

Have I messed my configuration up somewhere?
https://github.com/flamedfury/metadata-library/actions/runs/7236168216
Add bookmark
Error getting metadata for https://rknight.me/assets/ogi/please-expose-your-rss.png: Must scrape an HTML page


Add option to drop shard that will drop series without data

Sometimes when a user drops a shard they would like to have any series that have no data in other shards to be cleared out. Add something to the query language to support this and have the drop shard action potentially update the metastore by dropping the series name.

DROP SHARD is no longer relevant in 0.9


How to limit tile requests while mouse drag in the navigator ?

Hi, I found that many tiles are requested when I drag the red frame of the navigator . Is there any way to reduce these requests during the dragging process? by the way, I can reduce the requests on onCanvasClick of  the navigator by setting panTo to true. but I can not use the same way to solve the onCanvasDrag of the navigator.

The tile loading is because you're exposing new areas of the image, so it wants to load the tiles for those areas.
You tried adding true to the panBy in the navigator onCanvasDrag and it didn't help? You could also add it to the applyConstraints to be thorough.
Another thing to try is to initialize your viewer with imageLoaderLimit: 1... that way it only tries to load one tile at a time, which should reduce the number of tiles it's loading.
I suppose beyond that it could be interesting to consider adding a "don't load new tiles" option you could turn on when the user starts dragging and turn off again when the user stops. This might not be a good UX, though, since you'll expose blank or very blurry areas.

Thank you very much for your answer. I am very interested in your suggestion to add "don't load new tiles" options,but I don't  know how to use it ,Looking forward to your help.

I have an other question about the event of unloaded.When the tiles are released, what operation is done on the image loaded on the tile, is it released? if the image is released, why is the memory not coming down?I found in the project that when I quickly drag the red area in the navigator, the memory will rise wildly and cannot fall. Setting the value of maxImageCacheCount doesn't seem to work either.

I believe we're releasing the images fully when we unload a tile. It's up to the browser when to garbage collect and reclaim the memory, though, so it won't necessarily happen immediately. Of course if you can find a place where we aren't releasing the images, I'd love to know about it!
As for adding the ability to pause tile loading, you'd have to write that patch yourself. I'm thinking adding pause and resume functions to OpenSeadragon.ImageLoader:
https://github.com/openseadragon/openseadragon/blob/master/src/imageloader.js
... Might be a good API.

Thank you for your answer. Since I am not very familiar with Javascript, I would like to ask for your help on whether you can force the memory to be freed after the tile is unloaded.

It doesn't look like it's possible to force garbage collection to happen in JavaScript:
https://stackoverflow.com/questions/8032928/can-i-trigger-javascripts-garbage-collection/12658212
It's entirely up to the browser. It should definitely do it before it runs out of memory though.
Are you encountering problems with low memory situations, or are you just concerned about keeping things clean?


Remove check-query-cache experimental flag

Description

References

fixes #1007
Review Checklist

[X] I have clicked on "allow edits by maintainers".
[ ] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[ ] The correct base branch is being used, if not main
[ ] I have added tests to validate that the change in functionality is working as expected

Remove this line too please
https://github.com/openfga/openfga/blob/3b3e86d7887d53bb2db128c4d2bf2733563d108d/pkg/server/server.go#L48

I've removed this in 82e370f6fd7d0a7ad56a337f4208dd4bb78f2e6b
I've left things like this (to possible reuse in future experimental feature flags):
https://github.com/Siddhant-K-code/openfga/blob/a34f909153d67f88725b587c01baec9f5f96cffa/pkg/server/server.go#L93
https://github.com/Siddhant-K-code/openfga/blob/a34f909153d67f88725b587c01baec9f5f96cffa/pkg/server/server.go#L198
https://github.com/Siddhant-K-code/openfga/blob/a34f909153d67f88725b587c01baec9f5f96cffa/pkg/server/server.go#L261


[WIP] [BEAM-2930] Side input support for Flink portable streaming.

Support for streaming side inputs. Please note:

Currently only works with the iterable access pattern and tested with Python pipeline (multimap should be a small change in the side input handler only).
View materialization is done in operator state, different from the non-portable runner, which does it by adding additional operators. I modified the side input handler to allow for incremental state materialization. With that comes the need to identify when the state is ready, which in a first, perhaps naive attempt is just based on the side input watermark.
Due to the existing support classes side inputs are identified with PCollectionView. While working on it I realized that we can probably get rid of most PCollectionView usages from the DoFnOperator since it is really just a wrapper around the string ID.
It should be possible to refactor the DoFnOperator to separate out everything that is common between old and new runner into a new base class (state and side input handling) and pull up the "DoFnRunner" business into a subclass. There are subtle aspects in how state, timers and side inputs are handled correctly that should not be duplicated but also not mingled with legacy code going forward.
FlinkExecutableStageContext assumes that a state handler can be constructed just based on RuntimeContext (and ExecutableStage). That is true in batch when using broadcast variables. But here we need a side input handler that is supplied by the operator. I got around this by using a static registry for now, we should come up with a clean abstraction.

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---
---

R: @bsidhom @angoenka

@lukecwik thanks for taking a look. I wonder how droppably late can be evaluated in this case: The side input handler won't have received an input and the window is "ready" based on the watermark? In the old runner we have a separate combine operator, here I'm trying to accumulate the iterable in state.
Regarding refactor, my goal is to get this functional correct in first pass. Hope to then discuss/work with @aljoscha on how to extract common base operator for both runners.

@tweise You need to use the WindowMappingFn to figure out what side input is being asked for. If the side input isn't ready you should store the elements into state and set a timer. The simplest pushback solution would be to set a processing time timer that would then read those elements and check if they are ready, if so process them otherwise keep them in state. You could do better with map state or timers with payloads and you could also do better if you could use a watermark based timer based upon the side input PCollection.
The windowing strategy on the side input PCollection tells you the allowed lateness. Once the watermark + allowed lateness of the side input PCollection surpasses it, the window becomes ready.
This is how the existing implementation worked using PushbackSideInputDoFnRunner and the ReadyCheckingSideInputReader.

@lukecwik thanks and correct, pushback is already handled. What I was missing is the allowed lateness and then return empty side input iterable even when no element was previously received.

Early/repeated firings provide inconsistent side input values between bundles. Side input access is meant to provide the latest firing at the time when the bundle starts processing and it is up to the runner to try to make the latency between side input firing being materialized and visible to the consumer to be minimal but there is no latency requirement other then best effort.
Only side inputs that have a single firing can be reasoned concretely.

OK, then the idea of building up state as Iterable as shown in this PR is definitely not working.
Repeated firing isn't idempotent at least with Flink. If pipeline reverts to a checkpoint then change in order of arrival between (updating) side input and main input can lead to different result even for the same bundle.

The PR is ready for review. I made the changes to use GBK to materialize the side input, which adheres to the windowing strategy and is equivalent to what the old runner does. @lukecwik @angoenka

Thank you for your work @tweise, I will also have a look.


[feature request] Drop AAAA on IPv4-only Network

There's still a lot IPv4-only network or the IPv6 connectivity is super worse than IPv4,
So we need an option to drop all AAAA response and try to request A response if trying to get some DNS record.
Please add an options for users to drop all AAAA if user specified: do-ipv6: no or you might want to add a new config option.
Thanks.

Apply AAAA-filter patch in contrib/aaaa-filter-iterator.patch.

Try this configuration - it drops all AAAA records in response messages.

server:
private-address: ::/0

server:
private-address: ::/0

Just want to say THANK YOU for this!  It seems to have worked on pfSense Unbound DNS Resolver...and seemed to result in a massive speedup web browsing on my IPv4-only ISP not having devices trying to use IPv6 responses.


FutureWarning: Possible nested set at position 3 in jsinterp.py

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.11.11 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Unrestricted
Provide a description that is worded well enough to be understood
Downloading https://youtube.com/watch?v=jzRrUPQgrpc leads to a FutureWarning on Python 3.10, Arch Linux. I can only reproduce this on that specific YouTube URL, suggesting that yt-dlp hit a parsing edge case.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', 'https://youtube.com/watch?v=jzRrUPQgrpc']
[debug] User config "/home/$USER/.config/yt-dlp/config": ['-f', 'mp4[height>=720]', '--restrict-filenames']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.11.11 [8b64402]
[debug] Python 3.10.8 (CPython x86_64 64bit) - Linux-5.15.81-1-lts-x86_64-with-glibc2.36 (OpenSSL 3.0.7 1 Nov 2022, glibc 2.36)
[debug] exe versions: ffmpeg 5.1.2 (setts), ffprobe 5.1.2
[debug] Optional libraries: certifi-2022.09.24, sqlite3-2.6.0
[debug] Proxy map: {}
[debug] Loaded 1723 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.11.11, Current version: 2022.11.11
yt-dlp is up to date (2022.11.11)
[debug] [youtube] Extracting URL: https://youtube.com/watch?v=jzRrUPQgrpc
[youtube] jzRrUPQgrpc: Downloading webpage
[youtube] jzRrUPQgrpc: Downloading android player API JSON
[debug] Loading youtube-nsig.72d3c60a from cache
/usr/lib/python3.10/site-packages/yt_dlp/jsinterp.py:346: FutureWarning: Possible nested set at position 3
  inner = re.compile(inner[1:], flags=flags)
[debug] [youtube] Decrypted nsig 1GNC6u6XPSblJeLSe => HtSIq-cfYPC48A
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec:vp9.2, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), channels, acodec, lang, proto, filesize, fs_approx, tbr, vbr, abr, asr, vext, aext, hasaud, id
[info] jzRrUPQgrpc: Downloading 1 format(s): 22
[debug] Invoking http downloader on "https://rr2---sn-u5a3u5a3-h5oel.googlevideo.com/videoplayback?expire=1670540362&ei=6heSY4nPKJD1xgLJwrboCA&ip=<IP_ADDRESS>&id=o-AETkKx6duUr2V7PNK9VUoMLJFeuXrvzEh9MeHosUwdFV&itag=22&source=youtube&requiressl=yes&mh=M5&mm=31%2C29&mn=sn-u5a3u5a3-h5oel%2Csn-aigl6nzk&ms=au%2Crdu&mv=m&mvi=2&pl=20&initcwndbps=1433750&spc=SFxXNgfyIVYxZK50wZ0EkDNGEY3QeDI&vprv=1&svpuc=1&mime=video%2Fmp4&cnr=14&ratebypass=yes&dur=822.543&lmt=1670511366146582&mt=1670518404&fvip=4&fexp=24001373%2C24007246&c=ANDROID&txp=3318224&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cspc%2Cvprv%2Csvpuc%2Cmime%2Ccnr%2Cratebypass%2Cdur%2Clmt&sig=AOq0QJ8wRgIhAJHT4sZWnxCzFKTZiY-eXYsEwJigIcawkC6t59h-wApyAiEAm7ZF_keoLiPjiDn8LfFY8at-hMO9jNr0SGdOI-v1LMw%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRAIgJyAONHJKm3_u5LidpOeyNSZwg5AJbxBFt16_GfEHHZkCICCSa_VMSNgOJHlWzOLqFqUn4wujtOZhHbFTrR93Aj-f"
[download] Destination: Verifying_AI_Black_Boxes_-_Computerphile-[jzRrUPQgrpc].mp4
[download] 100% of  137.98MiB in 00:00:09 at 15.33MiB/s

It's happening due to slight difference in how regex works b/w JS and Python. Despite the warning, everything's actually working fine

What about the following:

FutureWarning is now emitted if a regular expression contains character set constructs that will change semantically in the future, such as nested sets and set operations. (Contributed by Serhiy Storchaka in bpo-30349.)

Is https://docs.python.org/dev/whatsnew/3.7.html incorrectly flagging this up, or is there a risk that this may break in a future version of Python?
Thanks for the help.

Sorry for commenting on a closed issue but is there a flag that hides this message? --no-warning doesn't seem to have any effect on it.

It's controlled by an env var https://docs.python.org/3/using/cmdline.html#envvar-PYTHONWARNINGS

Setting the environment variable works if you've used pip install or similar, but does not work if you're using a version bundled by PyInstaller or py2exe. Those tell Python to ignore the usual environment variables during initialization.


Increase code line length from 80 to 100 characters

(Let's not forget to create a release once this is merged into main so that the publish action runs)


Steam.requestInternetServerList() outright crashes and doesn't return an integer.

Simply running Steam.requestInternetServerList() results in a slightly delayed crash without any error messages.
No crash should occur and an integer id should be returned to lookup the request results.
Ran on Windows 10 version 21H2.
Godot 4 Beta 13 pre-built.
Steamwork 1.55 module.

Hey there!  Hmm, I will check it out and see what I can find.  Thanks for reporting the issue!

Just wanted to give you quick note that I will finally have time to solve this!  I have a project launching on Thursday so I've been in crunch-mode.  I should have something to report back this coming weekend!  Sorry for the delay.

Hmm, yeah, this function and a few others should return the server_list_request but do not.  They store them internally but related functions ask for this to be passed as an argument.  I think I had planned on using these like some other internally stored ID's but whatever plan I had clearly failed.  Haha.
This whole class will need updated to actually return the ID's and either ask for them as arguments in related functions or default to the internally stored value if nothing is passed to them.  Relatively easy fix!
I will get this patched up and pushed to repo relatively soon.  Thanks for reporting this!

OK.  This has currently been fixed in my local copy.  After I test compile it, I will push to repo and then issue new pre-compiles from it.  Should fix those functions and have them return the server_list_request.
Thanks again for reporting this!


Tweak organisation summary breadcrumbs

Add correct breadcrumb for org pages
Correctly indicate "Feedback" link in header as selected

:+1:


New customer is not added to the list of customers

Describe the bug
When adding a new customer he is not listed when creating a new offer. The app must be reloaded for the customer to be shown.
To Reproduce
Steps to reproduce the behavior:

Add a new customer
Click "New Offer"
Fill the content of the forms
Search the customer list for your new entry (won't find it)

Expected behavior
A new customer should be directly available.

 Test Create Customer Feature

 Test Create Customer Feature

https://github.com/qbicsoftware/qoffer-2-portlet/pull/195

https://github.com/qbicsoftware/qoffer-2-portlet/pull/195

The list is replaced so the data provider is not informed. Change in customerEvent.register call

The list is replaced so the data provider is not informed. Change in customerEvent.register call


Following something through variables

I am trying to track movements by year - so seeing how people move between places by their graduation year.  Currently the only way I can seem to do it is alluvial GradYear var1 GradYear v2 etc etc.  Ideally I would like to dosomething like  by GradYear: alluvial var1 var2 etc, but have all the gradyears on the graph.

Can you give an example with code or graph please.
Sure!
The code I am currently using is
. alluvial GradYear PGY12  PGY34 GradYear PGY56  PGY78 GradYear PGY910 which gives me
the attached graph.
Example.pdf

What I would like is for the GradYear to follow through PGY12 PGY34 etc, so I don't have to stick the extra GradYears in to see the breakup.  Hope this helps.

Thank you for the detailed example! I can now see what you are trying to do and it is good to see that the command is at least working for all these layers :)
You have a very specific implementation request so I have to think about a generic implementation to be able to by() the diagram.
It might even be better to split each year into different Alluvials and then use graph combine to stack them on top of each other. Otherwise the way it is done now, over inflates the Metro category.

Thank you for the detailed example! I can now see what you are trying to do and it is good to see that the command is at least working for all these layers :)
You have a very specific implementation request so I have to think about a generic implementation to be able to by() the diagram.
It might even be better to split each year into different Alluvials and then use graph combine to stack them on top of each other. Otherwise the way it is done now, over inflates the Metro category.

Thank you for the detailed example! I can now see what you are trying to do and it is good to see that the command is at least working for all these layers :)
You have a very specific implementation request so I have to think about a generic implementation to be able to by() the diagram.
It might even be better to split each year into different Alluvials and then use graph combine to stack them on top of each other. Otherwise the way it is done now, over inflates the Metro category.

merge with #18


Create LearnLingo.md

📝 Description
Add a brief description.
💣 Is this a breaking change (Yes/No):

[ ] Yes
[x] No

🚧 How to migrate?
Don't need.
📝 Additional Information
lol

Dear @Achyut22 , we found you wrote code in the prompt instead of commend in nature language. If you want to create a translator, you can just write "Translate the following sentences to English. "

Dear @Achyut22 , we found you wrote code in the prompt instead of commend in nature language. If you want to create a translator, you can just write "Translate the following sentences to English. "

Changes Made please check


ci: configure bors

Just for the moment seeing how to interact with bors, will experiment with CI setup in #2710 once I understand it.

bors try

try
Timed out.

I think our branch specification is missing the trying and staging branches used by bors:
on:
  push:
    branches:
      - main
  pull_request:
  workflow_dispatch:

Yes, that was the conclusion that I'd come to too. I'll push that first and then see what a try gives, before perhaps moving part of the pipeline to only run on bors.

bors try

bors try

bors is probably still waiting for something, causing https://github.com/PyO3/pyo3/pull/2771#issuecomment-1323797551
bors try-

bors try-


Icon for reset

I can't seem to get the icon for reset to appear on the right within the textfield when I begin to type.

Could you attach an example UI? Can't do anything with this info.

Question, do I need to add @VaadinServletConfiguration for widgetset = "org.vaadin.resetbuttonfortextfield.DemoWidgetSet"

If you have named your widgetset like that


#how to stream video from esp32 cam to tkinter python with link http://<IP_ADDRESS>:80/webcam ?

i tried using cv2 but it can't read the format of this url http://<IP_ADDRESS>:80/webcam, i also tried curl but it doesn't work, although on chrome browser it works fine. Do you have any solution for my problem? Thank you very much!

Start the webcam webserver
>>> import webcam
...
...
PWD: UajaoH5S
Start server 80
Try - http://<IP_ADDRESS>/login/UajaoH5S

Now, on a linux computer, perform the following:
$ cd /tmp

$ curl http://<IP_ADDRESS>/login/UajaoH5S
OK!

$ curl http://<IP_ADDRESS>/live --output live.mp4
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 17.4M    0 17.4M    0     0   196k      0 --:--:--  0:01:30 --:--:--  214k^C

Note: <ctrl>+c to kill curl
You will now have a file called live.mp4, which contains a video file that you can watch on celluloid.
$ celluloid live.mp4

This is probably the simplest way to capture video from an esp32-cam.
Take note of the URL! It's http://xx.xx.xx.xx/live, not../webcam, as you stated.
You can skip the login step if you disable authentication.
I've never used openCV, so I can't help you there.

oh, looks like the problem was with the authentication when using with opencv or something similar not web (i use it on mac maybe there is some difference), now it works for me, thanks he is very much about this module!


No VS Code task provider for 'nil'

Hey guys!
I create a simple tasks.json in .vscode directory in the root folder.
{
    "tasks": [
        {
            "name": "Echo example",
            "cmd": "echo 'Current file = ${file}'"
        },
    ]
}

:OverseerToggle, Open are empty.
:OverseerInfo returns this message No VS Code task provider for 'nil' .
Minimal configuration: require('overseer').setup()

I read the docs, but gotta no clue of I'm missing.
Ty,

This is because you're missing the type. You need to have either "type": "shell" or "type": "process", according to the documentation on custom tasks. Also I believe "name" is deprecated; they seem to use "label" now.

@stevearc ty! worked with this tasks.json:
{
    "version": "2.0.0",
    "tasks": [
        {
            "label": "echo_example",
            "type": "shell",
            "command": "echo 'Current file = ${file}'"
        }
    ],
    "group": "test",
    "presentation": {
        "reveal": "always",
        "panel": "new"
    }
}

Btw, Is possible to store these jsons in the nvim config dir, for common tasks? Instead of .vscode of each directory?

Nope, the tasks.json format is currently per-directory, just like how it work in VS Code. If you want general tasks to always be available, you'll have to register them with lua. If you'd really prefer to have general tasks defined this way, I can probably add an API call to load a tasks.json file from an arbitrary location

@stevearc Today is my third day using overseer in my config, I believe I will abandon the tasks.json, I enjoyed defining them with lua.


Use cirq.decompose protocols instead of Composite[Gate/Operation]

Fixes #945

Automerge: started

Automerge done.


pull from upstream.

📝 Changed routes:

/404
/

(view full size)
(view full size)

1 deleted route:

/

Commit 84e2a2b38275deb71327f83710361486a32fd369 (https://running-page-pwb19kvqm-lizhimiao.vercel.app).


Remove extra fields from DTOs

What Changed?
Removed extra fields in DTOs and cleaned up comments.

There are a few more you can remove:
Media#localVersion
Media#


Network related fixes

Yes, the name of the idrac-mgmt.
On Thu, Feb 22, 2024 at 11:34 AM Balazs Nemeth @.***>
wrote:

@.**** commented on this pull request.
In clusterConfigSchema.yml
https://github.com/bn222/cluster-deployment-automation/pull/107#discussion_r1499026848
:

@@ -17,6 +17,7 @@ clusters:
type: "<vm / physical / bf>"
ip: ""
node: ""

 bmc_name: "optional. Could be used instrzad of bmc_ip"

Is this the dns name?
—
Reply to this email directly, view it on GitHub
https://github.com/bn222/cluster-deployment-automation/pull/107#discussion_r1499026848,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASOCVMMPAOKVTVVVF26O42LYU4NJ5AVCNFSM6AAAAABDUU3PJ6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQOJVGQ4TONBWGI
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

Yes, the name of the idrac-mgmt.
…
On Thu, Feb 22, 2024 at 11:34 AM Balazs Nemeth @.> wrote: @.* commented on this pull request. ------------------------------ In clusterConfigSchema.yml <#107 (comment)> : > @@ -17,6 +17,7 @@ clusters: type: "<vm / physical / bf>" ip: "" node: "" + bmc_name: "optional. Could be used instrzad of bmc_ip" Is this the dns name? — Reply to this email directly, view it on GitHub <#107 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASOCVMMPAOKVTVVVF26O42LYU4NJ5AVCNFSM6AAAAABDUU3PJ6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQOJVGQ4TONBWGI . You are receiving this because you authored the thread.Message ID: @.*** com>

have you tried setting the ip to the name?

Yes, you could do that as well - but I think it's not really intuitive
(e.g. the "ip" in workers or masters config must be an ip, but the bmc_ip
could then be a name).
Now, if you do not like adding this config option, I am ok dropping the
patch, as I have a workaround
On Thu, Feb 22, 2024 at 12:01 PM Balazs Nemeth @.***>
wrote:

Yes, the name of the idrac-mgmt.
… <#m_4869489240398863898_>
On Thu, Feb 22, 2024 at 11:34 AM Balazs Nemeth @.> wrote: @.**
commented on this pull request. ------------------------------ In
clusterConfigSchema.yml <#107 (comment)
https://github.com/bn222/cluster-deployment-automation/pull/107#discussion_r1499026848>
: > @@ -17,6 +17,7 @@ clusters: type: "<vm / physical / bf>" ip: "" node:
"" + bmc_name: "optional. Could be used instrzad of bmc_ip" Is this the dns
name? — Reply to this email directly, view it on GitHub <#107 (comment)
https://github.com/bn222/cluster-deployment-automation/pull/107#discussion_r1499026848>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASOCVMMPAOKVTVVVF26O42LYU4NJ5AVCNFSM6AAAAABDUU3PJ6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQOJVGQ4TONBWGI
. You are receiving this because you authored the thread.Message ID: @.**
com>
have you tried setting the ip to the name?
—
Reply to this email directly, view it on GitHub
https://github.com/bn222/cluster-deployment-automation/pull/107#issuecomment-1959203332,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASOCVMPF3J6IRCAINBWVIWDYU4QQBAVCNFSM6AAAAABDUU3PJ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJZGIYDGMZTGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

@Zinkelburger can you please test if this PR works for our use cases?

@Zinkelburger can you please test if this PR works for our use cases?

Yes, I can do that today

@bn222 I was able to successfully deploy the changes

@Zinkelburger can you provide feedback on the code?

@Zinkelburger ?

LGTM, thanks for testing @Zinkelburger

Need to be careful since ensure_linked_bot_bridge also restarts libvirtd. Are we ok with this?

Aah I see, apologies for the oversight. Reverted: https://github.com/bn222/cluster-deployment-automation/pull/134


Fix typo in documentation what-is-new/index.md

Changed EntityTypeConfiguraiton to EntityTypeConfiguration

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

:warning: Validation status: warnings

File
Status
Preview URL
Details

entity-framework/core/miscellaneous/1x-2x-upgrade.md
:warning:Warning

Details

entity-framework/core/what-is-new/index.md
:white_check_mark:Succeeded

entity-framework/core/miscellaneous/1x-2x-upgrade.md

Line 33: [Warning] Illegal link: [implement  *IDesignTimeDbContextFactory<TContext>*](configuring-dbcontext.md#use-idesigntimedbcontextfactory) -- missing bookmark. The file core/miscellaneous/configuring-dbcontext.md doesn't contain a bookmark named use-idesigntimedbcontextfactory.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Thanks @bascoder for the fix.


Update Welsh link to match start page

This updates the Welsh link on the 'Sorry, this service is only available in England' page, so it matches the link on the start page.

Looks good to me - feel free to merge


undefined method `configure' for AppleMusic:Module

I have added gem in rails 6 app.
& created a file in config/initializers named apple_music.rb with settings
AppleMusic.configure do |config|
  config.secret_key_path = ENV['APPLE_MUSIC_SECRET_KEY_PATH']
  config.team_id         = ENV['APPLE_MUSIC_TEAM_ID']
  config.music_id        = ENV['APPLE_MUSIC_MUSIC_ID']
  config.storefront      = 'us'
end

but gives error saying
undefined method `configure' for AppleMusic:Module

yep, it doesn't work:
[1] Vydia(main)> AppleMusic.configure do |config|
[1] Vydia(main)*   config.secret_key_path = './AuthKey_MUSIC_ID.p8' # or ENV['APPLE_MUSIC_SECRET_KEY_PATH']
[1] Vydia(main)*   config.team_id         = 'YOUR TEAM_ID'          # or ENV['APPLE_MUSIC_TEAM_ID']
[1] Vydia(main)*   config.music_id        = 'YOUR MUSIC_ID'         # or ENV['APPLE_MUSIC_MUSIC_ID']
[1] Vydia(main)*   config.storefront      = 'jp'                    # or ENV['APPLE_MUSIC_STOREFRONT'] ('us' by default)
[1] Vydia(main)* end
NoMethodError: undefined method `configure' for AppleMusic:Module
from (pry):1:in `__pry__'

I think this is because a release has not been made, it does work on master:
gem "apple_music", git: "git://github.com/yhirano55/apple_music.git"

@yhirano55 are you planning on publishing a release for this gem?

I'm sorry to have kept you waiting. I will publish it.

I've released v0.2.0.


Enter isn't captured in the WebView

After further testing, 'Enter' seems to not be captured within the WebView when the user is currently focused on another app outside of the overlay (you essentially have to be on the Desktop for it work).

The issue is known to be an issue with the WebView object.


forking broken

Describe the bug
when clicking "fork" the spinner load forever, but nothing happens.
Link to the blitz that caused the error
https://stackblitz.com/run?file=src/app/breadcrumb-custom-root-example.ts
Steps to reproduce

go to https://stackblitz.com/run?file=src/app/breadcrumb-custom-root-example.ts
click "fork"

Expected behavior
it should create a fork
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots

Platform
Browser name  = Chrome
Full version  = 97.0.4692.99
Major version = 97
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/97.0.4692.99 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 70202393,
  "usedJSHeapSize": 64069493,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = 5df97b5e61f923919ea927c406a4806b792de8c6

Additional context
No response

probably a duplicate of #569

Closing b/c duplicate of #569


Feature/xinnanli/aff bug

Critical Changes
Changes
Better handles affiliation null pointer exception by displaying beautified and specified error messages
Issues Closed
Fixes #578
New Metadata
Deleted Metadata
Testing Notes

Create accounts without record types before deploying HEDA
Deploy HEDA
Create a contact and use one of the testing account as primary field (Business org etc).
Change the primary field to another account without record type/ uncheck primary checkbox on the new affiliation created.
Make sure the proper error message is shown.
Create another contact and use another account as primary field (Business or etc).
Go to setting page and modify the affiliation account mappings to not matching existing record types.
Change the primary field to another account without record type/ uncheck primary checkbox on the new affiliation created.
Make sure the proper error message is shown.

Issue found (with some others that @xinnanlisf  found):
In a contact
I had a primary affiliation to a business organization record type account
In related list->I Edited that affiliation and choose an account without a record type (and left primary checked) and pressed Save button- Got an error.
TDTM_Affiliation: execution of AfterUpdate caused by: System.NullPointerException: Argument cannot be null. Class.AFFL_AccChange_TDTM.run: line 96, column 1 Class.TDTM_TriggerHandler.runClass: line 145, column 1 Class.TDTM_TriggerHandler.run: line 75, column 1 Class.TDTM_Global_API.run: line 61, column 1 Trigger.TDTM_Affiliation: line 33, column 1

We also talked about this concern:
If we are putting a new enforcement in place for recordtypes on those primary fields, it's possible that existing integrations will break if a customer has things set up incorrectly. I'm an advocate for having a HEDA setting for "Enforce record type mappings when populating mapped fields". Even if we check it by default during the upgrade, we have a path do disable it for customers to get back up and running until they fix their integration. I see it as similar to the "enforce picklist values" feature in platform...good to have but can't force it on orgs.
Some ideas:
enforce record type mappings when populating mapped fields- for trial orgs -default checked to enforce
for brand new install/orgs- default checked to enforce,
Existing heda orgs- defaulted to not checked to enforce
**all have the options to modify this setting in heda settings

Hey @lparrott, this most part of this PR should be ready for review now. I might need to work on 1 or 2 unit tests thou. Thank you! @annejackson  feel free to start testing if you want. :)

@lparrott should be good again!

@lparrott i made some changes so we check if contact field is not null at the very beginning of the code. Could we take a review please? Thank you!

@xinnanlisf The code looks fine as long as it works as expected in QA!

I sent @xinnanlisf  some messages on this one with some screenshots. Thanks @juliesnow  for the changes/note help!

@lparrott this PR should be ready for review now. We are handling scenario when contact primary field in mappings mismatch.

Found some issues on this, let @xinnanlisf know.

Hey @lparrott, I made some changes to error handling in last two commit. Mind taking a look?

@xinnanlisf Great solution with using if(SUBSTRINGS.keySet().contains(STATUS)) to get around that error you were seeing, this all looks fine to me.

Hey @ceiroa & @lparrott, I just made some polishment so that we don't hardcode anymore. (or less hardcoded) Let me know if that makes sense.

This looks great @xinnanlisf!! Thanks for all the hard work. All my scenarios have passed. @ceiroa  and @lparrott let me know when one of you is done checking the polishments then I will merge.

@xinnanlisf @annejackson Makes sense to me!

Merging!

@juliesnow I moved the second paragraph and added a sub bullet. Let me know if it makes sense. Thank you!

I like the sub-bullet, it looks good with all the release notes together.

Made some minor changes to release note based on @juliesnow's feedback.

Updated release notes based on findings.


[feat]: Implement better bounty relay evaluation.

Finish the implementation (jsanmi/pricing):

Take into account wallet configuration (gas price adjustment)
Take into account how gas is computed in different chains (depends on #25)
Make sure the verification logic is taken into account on the cost estimation.
Decide whether to make use of the evaluation class or otherwise.
Implement 'strong incentives'

Logic implemented via #29. This logic must be tested in testnet.


Understanding ta.vp indicator (Volume Profile). bug?

For testing & understanding purposes I've created a DataFrame and run vp on the first 10 rows which is the minimum width required by the indicator.
I understand as on vp.py close series is evaluated and through the series.diff(1) in the signed_series function current close is compared to the next close and then assigned a positive or negative value, which then results in either pos_volume or neg_volume for the vp itself.
As I understand, the total volume in a given price range, should be the same as the total vp volume should return. Is this correct?
I have checked Issues #74 and #185 looking for an already answered similar question.
Here is the df.head(10) :

And here is df.ta.vp() where you can see Volume data loss on 2nd and 3rd rows:

As you can see, when close prices are equal on consecutive rows, issues arise as
df.ta.vp()['total_Volume'].sum() == df.Volume.sum() evaluates to False
What am I missing or maybe not understangid about vp?

Hello @argcast,
Yeah that does look like an issue. Something I would have to find some time to dig into more when I get a chance.
I found these links for VP which may help as its actual source is hard to find.

akhilman - vol-poorofile
Fr3d0's Volume Profile Visible Range - TV version 4
reddit algotrading

If you have any ideas on how to fix it, that would be helpful. 😎
Apologies for the late reply.
Kind Regards
KJ

Hey @twopirllc ,
sorry for the late reply. I have been working on it with a simple fix. Here is my approach:
Since the issue happens only when open and closing prices are equal, current vp.py does not know wether if it is pos_Volume or neg_Volume happening. Neither do we and that means we cannot directly sum it to either of them.
That's why I modified the code a little bit adding a new neut_volume (neutral volume) that computes those cases.
Let me explain better:
import pandas as pd
import pandas_ta as ta
import yfinance as yf

df = yf.download('RVP', start="2023-03-01")

As we can see, we have found one case where open == close and upon calling df.ta.vp()we can see we have data loss:

While if we consider neut_vol (neutral volume) as what is happening in those cases we can avoid data loss:

and VP charting should consider that "neutral" volume too

PS: I'm not sure wether if "neutral" should be the correct naming for this volume
I've created a PR #670  with modified vp.py so you can take a look at it and make any necessary adjustments.
I hope this helps 🙂
Albert.


Syntax Bug: Closing parenthesis after inline module

I came across a nasty bug just now. I'm not sure if it's just this specific setup though. Some code of mine compiled fine but didn't work the way I expected. When I checked the JS Code, I found it was missing everything after a certain point (Still, it compiled fine).
I eventually found one closing parenthesis too much after a module definition.
See the following example:
open Ws

let wss = Server.make({port: 82})
let address = wss->Server.address
let log = msg => Js.log(`> Server: ${msg}`)
log(`Running on: ${address.address}:${address.port->string_of_int} (${address.family})`)

module ClientSet = {
  module T = Belt.Id.MakeComparable({
    type t = Client.t;
    let cmp = (a, b) => {
      compare(a->Client.getUniqueId, b->Client.getUniqueId);
    };
  });

  let empty = Belt.Set.make(~id=module (T));

}); // this ")" here, this can even be "))))))))))))))))" and it still compiles without complaint

Js.log("test"); // Is omitted from the compiled JS

Addition: When I run the file through the file through napkinscript.exe -parse ns -print ns, everything after the module closing } gets deleted from it

Shorter version
module C = {
  module T = Fun({
      foo(a+c, b+c)
  })
}
)
Js.log("test"); // Is omitted from the compiled JS

Issue: parseImplementation, and I guess parseInterface should check when last token is not eof, and act accordingly.

Thanks for the report @jsiebern! This turned out to be a one-liner.
@cristianoc turns out I forgot an eatBreadcrumb, giving the wrong parsing context during the error.
Proves again that I should implement your suggestion of make eatBreadcrumb also take a grammar, so it's easier to check that the calls are matching


Make ClientID configurable

I had problems to add rscp2mqtt to my FHEM installation via the build-in MQTT2 server. Finally I could do so after changing your mosquito connection string, which generates at every start a new random ClientID, to this fixed string (line 1621 in RscpMqttMain.cpp):
mosq = mosquitto_new("rscp2mqtt", true, NULL);
It would be fine, if it is possible to set a fixed ClientID in the config file.

Hi docolli,
release 3.9 is available. Now it's possible to configure the parameter MQTT_CLIENT_ID in the config file.
Best regards
Thomas

Thank you!
I did an implementation of this by myself after my request. I am not a good coder, so I compared what you did and how my solution looks like to learn.
In my solution I did not need to use an extra flag fix_mqtt_client_id to have an argument for the necessary if-condition when connection is done.
Here is how I did it:
    strcpy(cfg.mqtt_host, "");
    cfg.mqtt_port = MQTT_PORT_DEFAULT;
    strcpy(cfg.mqtt_cid, "");
    cfg.mqtt_auth = false;
    cfg.mqtt_qos = 0;

        // MQTT connection
        if (!mosq) {
            if (cfg.mqtt_cid=="") {
				mosq = mosquitto_new(NULL, true, NULL);
			} else {
				mosq = mosquitto_new(cfg.mqtt_cid, true, NULL);
			}
            logMessage(cfg.logfile, (char *)__FILE__, __LINE__, (char *)"Connecting to broker %s:%u\n", cfg.mqtt_host, cfg.mqtt_port);

Hi docolli,
correct, the flag fix_mqtt_client_id isn't really necessary. The comparison with an empty character string would be sufficient.
But your implementation isn't okay. If you like to compare two strings, you have to use strcmp in C, not "=="!
Best regards
Thomas

Thanks for this hint!
However, it did compile without any error and I think, when I tested it without ClientID configured in the config file, the ClientID was generated randomly, so the code worked.
As I said, I am not very experienced in coding, so I checked on the web now and found this, at least for C++ the "==" is also possible to compare strings and leads to the same machine code: https://stackoverflow.com/questions/9158894/differences-between-c-string-and-compare


Content Updated #107

Changed the Crispy logo with SILK Logo and also updated the bottom of the website.

I have used the same logo that is in the header.
Before

After

I am going to change the "Developed By" to SILK Corp instead of WE.

Change SILKCorp in Footer to SILK Corp


Some property/type URIs are not compacted

~Two~ Some bflc properties are missing in the context because they are missing in the BFLC vocabulary so that they were not added to the context with f6fab69b5d0896714fd0ee5368834c944b299de7:

http://id.loc.gov/ontologies/bflc/consolidates
http://id.loc.gov/ontologies/bflc/relatorMatchKey
http://id.loc.gov/ontologies/bflc/procInfo
http://id.loc.gov/ontologies/bflc/profile

Another property that is not compacted is http://id.loc.gov/ontologies/bibframe/relatedTo. I think it wasn't added because the python script does not seem to cover properties that are of type owl:SymmetricProperty. Thus, the following four properties are also missing (but I am not sure whether they are actually used in the dataset): bf:issuedWith, bf:otherEdition, bf:otherPhysicalFormat, bf:hasEquivalent,

bf:reltedTo was added to the context. bf:otherPhysicalFormat and bf:hasEquivalent are not used in the dataset. The following are still missing in the context:

bf:issuedWith
bf:otherEdition
http://id.loc.gov/ontologies/bflc/consolidates
http://id.loc.gov/ontologies/bflc/relatorMatchKey
http://id.loc.gov/ontologies/bflc/procInfo
http://id.loc.gov/ontologies/bflc/profile

Also, there are lots of properties not compacted, which actually should be as they are in the context. Examples: arrangement.http://id.loc.gov/ontologies/bflc/appliesTo, http://id.loc.gov/ontologies/bibframe/projection. See the whole list in the index profile in Kibana by filtering with "http".

Won't fix. It makes sense to still have some examples in the data that don't work that well.


Add labels to support kubectl --prune with Helm

This is required to be able to use helm template with kubectl apply --prune in a multi-revision case where gateways are managed separately.

@ostromart: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
0681dccd896a0dc5ae18fe43be4e6a6a25afc61e
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
0681dccd896a0dc5ae18fe43be4e6a6a25afc61e
link
/test release-notes_istio

lint_istio
0681dccd896a0dc5ae18fe43be4e6a6a25afc61e
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
0681dccd896a0dc5ae18fe43be4e6a6a25afc61e
link
/test release-notes_istio

lint_istio
0681dccd896a0dc5ae18fe43be4e6a6a25afc61e
link
/test lint_istio

gencheck_istio
0681dccd896a0dc5ae18fe43be4e6a6a25afc61e
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
0681dccd896a0dc5ae18fe43be4e6a6a25afc61e
link
/test release-notes_istio

lint_istio
0681dccd896a0dc5ae18fe43be4e6a6a25afc61e
link
/test lint_istio

gencheck_istio
0681dccd896a0dc5ae18fe43be4e6a6a25afc61e
link
/test gencheck_istio

unit-tests_istio
0681dccd896a0dc5ae18fe43be4e6a6a25afc61e
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart @nrjpoddar I am really wishing we had upgrade tests for the helm workflows before introducing irrevocable changes such as labels.
Cheers,
-steve

@ostromart: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
0681dccd896a0dc5ae18fe43be4e6a6a25afc61e
link
/test lint_istio

unit-tests_istio
0681dccd896a0dc5ae18fe43be4e6a6a25afc61e
link
/test unit-tests_istio

gencheck_istio
c7749891f3ea5d7775b3e442ab4ded10108d59a9
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
0681dccd896a0dc5ae18fe43be4e6a6a25afc61e
link
/test unit-tests_istio

gencheck_istio
c7749891f3ea5d7775b3e442ab4ded10108d59a9
link
/test gencheck_istio

lint_istio
c7749891f3ea5d7775b3e442ab4ded10108d59a9
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
c7749891f3ea5d7775b3e442ab4ded10108d59a9
link
/test lint_istio

gencheck_istio
57a5d361d548e47a53f0aacce82c187881e56ea0
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
57a5d361d548e47a53f0aacce82c187881e56ea0
link
/test gencheck_istio

unit-tests_istio
57a5d361d548e47a53f0aacce82c187881e56ea0
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart @nrjpoddar I am really wishing we had upgrade tests for the helm workflows before introducing irrevocable changes such as labels.
Cheers,
-steve

Agree about testing.
I chatted with @ostromart and confirmed that Helm based canary upgrades would only need the "istio.io/revision" label as Helm performs pruning via its own mechanism. The new ownership label is needed for "kubectl" pruning afaict.

@ostromart: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio
2de512a44b0b9fcb3f188a80bac1b48f78673099
link
/test integ-telemetry-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio
2de512a44b0b9fcb3f188a80bac1b48f78673099
link
/test integ-telemetry-k8s-tests_istio

unit-tests_istio
2de512a44b0b9fcb3f188a80bac1b48f78673099
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
f57e7c79b9c13089cb926c2bc9948010afdbc2e9
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@ostromart: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-test_istio
f57e7c79b9c13089cb926c2bc9948010afdbc2e9
link
/test release-test_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

/test release-test_istio

Fixed https://github.com/istio/istio/pull/28039.


Show ReciprocalSearch when searchColumns is empty

The general search field can still be used.

Closes https://github.com/foam-framework/foam2/issues/1488

How do I specify I don't want any search border from the model then?
You can tell if the user set searchColumns to [] or it's just blank like this:
foam.nanos.menu.Menu.model_.searchColumns;
undefined
foam.nanos.cron.Cron.model_.searchColumns;
(2) ["id", "description"]

I added the ability to do that through a flag on DAOController last week.
https://github.com/foam-framework/foam2/blob/master/src/foam/comics/DAOController.js#L90
Either solution could solve the problem. One advantage of having a flag on the view makes it possible to include the filters on one page but hide them in another if we wanted to. Having the preference set on the model means it would have to be the same everywhere, wouldn't it? I'm open to whatever solution we think is best though.

I think that setting the searchColumns should be an option on the controller as well in case you don't want the defaults and we should treat searchColumns: [] an an explicit request for now searchColumns.

I think that setting the searchColumns should be an option on the controller as well...

Did you mean model instead of controller? Assuming you did, should an empty array for searchColumns mean the general search field is still available, or should the entire sidebar be hidden?

I meant controller. It's already an option to set it on the model. I should be able to set it on the controller if I don't want to use what the model specifies.

Opened a ticket for the proposed behaviour here: https://github.com/foam-framework/foam2/issues/1544
Closing this since it doesn't make sense anymore. I'll open a new PR to implement https://github.com/foam-framework/foam2/issues/1544 when I get around to it.


test_plugins[plugins-docker-default] fails because of non-existing repo

It looks like @retr0h has removed https://github.com/retr0h/ansible-plugins repository today/yesterday, which lead to failing the test test_plugins[plugins-docker-default], which tries cloning it internally.
Issue Type

Bug report

Molecule and Ansible details
Cloning into '/home/travis/.gilt/clone/retr0h.ansible-plugins'...
remote: Invalid username or password.
fatal: Authentication failed for 'https://github.com/retr0h/ansible-plugins.git/'

Desired Behavior
Test test_plugins[plugins-docker-default] pass
Actual Behaviour
Test test_plugins[plugins-docker-default] fails

cc @gundalow

Hey @retr0h, any way to restore that? We can move the repo if you don't want it under your account but need a source to figure out the test.

Log: https://travis-ci.com/ansible/molecule/jobs/158736934#L1074-L1078