Post text limit and "View More" functionality

env: https://staging.fightpandemics.work/feed
For signed in users
Expected: The posts are limited to 25 words --> User clicks "View More" --> Post shows all text
Fix:

Limit posts to 25 words
When "View More" is clicked --> expand post to show rest of text

For users unregistered or not logged in
Expected: The posts are limited to 25 words --> User clicks "View More" --> User is directed to sign up page
Fix:

When "View More" is clicked --> user is directed to sign up page

Issue seen in: Desktop and Mobile

@doheekim93 Actually we do need this. We just need to be consistent and only show 25 words right @Naraujo13?

The idea is to have consistency in how long the posts are.

Not sure how it is done in the frontend but the /posts endpoint is limiting already based on wether the user is logged in or not.
If the user is logged the backend returns full content, otherwise it limits to 120 char (Twitter limit I think, we talked about this being easier to do a while agora @manualzuru).

From Figma:

I think this is why the text is not limited to 25 words for logged in users? Like @Naraujo13 said, the text IS limited for unregistered/not logged in users though, so we can make it consistent between all users if we want to.

Not sure ai got that. You mean limiting for logged users also?

@manualzuru Do you want to limit text for logged in users as well?

@Naraujo13 Let's streamline it, so posts are always defaulted to 25 words regardless of whether the user is logged in or not. I will update the ticket!

But defaulting to 25 words is against the requirements...
I'm not seeing the problem here.
Wouldn't it be:
If the user is not logged, he sees only 25 words, with no view more. This was what is said at the requirements, to direct them to signing up.
If he is logged he sees 25 words with the view more, being able to see the whole post.
Is that it @manualzuru ?

Oh I see now the description @doheekim93, that is exactly what I thought. The backend already does that, we should add to the frontend to always show 25 words and the view more and then, based on wether the user is logged or not, we redirect him to sign-in/sign-up or expand the post

On the frontend we should have consistency that's for sure. So I'd say that yes we need to show X amount of characters always (they can click on view more).

At the backend it is currently done by char

@BMariscal isn't this also being taken care of by the big CRUD PR that is pending? Thought I saw this functionality on the demo video, we should assign it to you and link this issue on that PR

@idjevm Yes, I think that was taken care by the big CRUD PR. However, there are posts that are less than the char limit that shows "View More", like this post here:

@doheekim93  Please take a look at these changes: https://streamable.com/pajksg
Now the user can click on the div itself (the title and description section) to access the post's page and can share the post's link from the feed.

That looks so good!!! Definitely better UX this way. Thanks, @BMariscal!

Since this functionality has already been implemented by @BMariscal, let's close this issue, and open any new issue to iterate and update if needed.


How to delete multiple rows

I want to delete id:1 and id:2 at the same time, instead of one by one.
Something like this:
const del_data = [{"id":1, "name":"tom"},{"id":2,"name":"kelly"}]

// table data :
// [{"id":1, "name":"tom"},{"id":2,"name":"kelly"},{"id":3, "name":"siro"}]

del_data.map((item) => {
   deleteData('searchTable', { id: item.id })
})

You code seems Ok, only it needs to be handled with async await if you are performing read just after delete.
As in example Delete Data


opt out of certain checking

match-sorter: 2.2.0
node: v8.9.3
npm (or yarn) version: 5.7.1

Problem description:
Is there a way to opt out of certain checking. Like the use case I have, I do not want CASE ACRONYM, ACRONYM  & SIMPLE MATCH being used for comparison. So any items that match these 3 things are omitted from final result.

This is not supported, but I'd happily accept a tested, documented, working PR for it :+1:

Sure. Let me give it a shot. Will circle back in few days with a rough design.

I think this is the kind of API I'd like:
matchSorter(otherThings, 'app', {excludeRankings: [matchSorter.rankings.STRING_CASE_ACRONYM, matchSorter.rankings.ACRONYM, matchSorter.rankings.MATCHES]})

Though it just occurred to me that your purposes might be better served by simply doing:
matchSorter(otherThings, 'app', {threshold: matchSorter.rankings.CONTAINS})

This means that if it doesn't at least match well enough as the contains ranking then it wont be considered a match. Does that work for you?

Yup matchSorter(otherThings, 'app', {threshold: matchSorter.rankings.CONTAINS}) works perfectly.
I somehow was not able to make sense of what rankings meant. Your explanation here makes perfect sense.


Make Credentials a generic map instead of a list of hard-coded properties

Closes: #167
Signed-off-by: Doug Davis<EMAIL_ADDRESS>

do we need to be worried about logging this struct at any point? I know some times there are 'redacted objects' that when they are asked to print themselves they do not leak secrets. Does kube proper have something? Not really related to the pr. a thought.

I guess we could consider masking the values of all items and just log the keys - I don't think that would be a security risk.  Can consider that for a follow-on PR I guess.
rebased.

Service catalog doesn't need to talk to the deployed instance using this binding, so it doesn't matter what's in the creds, and we don't need to parse it. The service catalog needs to be a dumb pipe from the broker to user of a service.

I think we'll need to at least parse it enough to get the top-level json keys since those will become the keys for the secrets - I think

For now, all we need to do is shuffle the blob from one place to the other. Once the schematization happens, we might need more internal inspection, but I'm not sure that will be true.

I think we'll need to at least parse it enough to get the top-level json keys since those will become the keys for the secrets - I think

That is accurate; keys will become secret/configmap keys.


Homebridge MyQ  unconfigured parameters

I did everything step by step, and copy pasted the config.json from this site.  then when I try to add the device on the iOS app devices, it says unconfigured parameter. I spent 22 hours reading and trying please help. thank you.  I am willing to pay for someone helping get this work.
"platforms": [
   {
     "platform": "MyQ",
     "name": "MyQ",
     "MyQApplicationId": "NWknvuBd7LoFHfXmKNMBcgajXtZEgKUh4V7WNzMidrpUUluDpVYVZx+xT4PCM5Kx",
     "UserAgent": "Chamberlain/2793 (iPhone; iOS 9.3; Scale/2.00)",
     "BrandId": "2",
     "culture": "en",
     "username"<EMAIL_ADDRESS>     "password": “******”,
     "loglevel":"0"
   }
],

My iPhone is on iOS 10 public beta, but I just copy pasted the config.son file anyways

Can you post your entire config file?
If that is your config file, it is incomplete and invalid.
You can see a sample for this platform here: MyQ config-sample.json
PROTIP: to aleviate timesucks, validate your json: json validator
I am not sure why the author decided to add an extra platform, you can use this, based on what you provided:
{
    "bridge": {
        "name": "Homebridge",
        "username": "CC:22:3D:E3:CE:AA",
        "port": 51826,
        "pin": "031-45-154"
    },
    "description": "I will not get frustrated. I will not get frustrated. I will not get frustrated.",
    "platforms": [
        {
            "platform": "MyQ",
            "name": "MyQ",
            "MyQApplicationId": "NWknvuBd7LoFHfXmKNMBcgajXtZEgKUh4V7WNzMidrpUUluDpVYVZx+xT4PCM5Kx",
            "UserAgent": "Chamberlain/2793 (iPhone; iOS 9.3; Scale/2.00)",
            "BrandId": "2",
            "culture": "en",
            "username"<EMAIL_ADDRESS>            "password": "******",
            "loglevel": "0"
        }
    ],
    "accessories": []
}

config.json.txt
thats my config file man. Thanks so much buddy.  I will try your config now.

i used your configuration file with just the username and password modified and then when I try to add device.   I get this issue (look picture).  Do I have to change any of the user agent/ ip. if so how do i change it.   i have a member in my family with disability who needs to be able to open door without taking out phone.  I am new to all this coding stuff.  but I think I am close to getting it to work.

Oh also I am on the latest iOS 10 beta, if thats the problem. I could use a iOS 9.3 device, in fact let me try that.

I'm also using the beta on the iphone, this issue is most likely not related to the OS.
After upgrading to the new beta I had an issue where my screen was black, this may not be the same issue that you're having, but its worth a try. In IOS 10 you can uninstall the stock apps, and reinstall from the app store, although you'll have to recreate your home, its worth a shot.
If that is not it, it is probably plugin specific, as I'm not using it, I'll be of little help.
You may want to post on the github repo for that plugin: homebridge-myq
Homebridge logs to two places:

/var/log/homebridge.log
/var/log/homebridge.err

I suggest you look in them to see if anything jumps out at you, this will also help the people on that repo help you easier.
PROTIP: unix has the tail command, it makes it really nice to see logs in real time.
Assuming you have homebridge running, from that server run one of the following commands to see what is getting logged live.
tail -f /var/log/homebridge.log
tail -f /var/log/homebridge.err

I looked at that repo, it looks like it is retired. I suggest you try the plugin referenced.
Unfortunately, even this repo looks pretty stale, however its maintained by @nfarina who also maintains the repo we're on.
Good luck!

Project Retired

Already an existing project that does this called "Liftmaster": https://github.com/nfarina/homebridge-liftmaster

your original problem was because you used the wrong kind of quote marks, I spotted it straight away.. those are inverted quote marks, you need to use straight up and down quote marks.  yes homebridge needs better config.json error handling, but its all we've got.
"username"<EMAIL_ADDRESS>"password": “”,

hey guys, let me try it when I get home. I will let you guys know how it goes

I had issues getting the MyQ plugin to work, however the Liftmaster plugin works perfectly with the Chamberlain MyQ. Liftmaster and Chamberlain are the same company and make the same product, branded differently. Works perfect for me on iOS 10 beta, Watch OS 3 Beta, MacOS Sierra.

Hey treeiler, to use the lift master, do I need to be using the lift master app on my phone? or make a new login on lift master?   also can you please give me your JSON?  thanks mate.

I am having trouble understanding the fundamentals of whats happening. I mean I really don't want to learn it actually.  The guy at wal-mart told me this was home kit compatible. EVEN on their demo poster that walmart had it said home kit. I own ecobee3. and let me tell you THAT WAS SO EASY to setup, like i expect from apple, i am not too good at tech stuff.
Anyways, I must get this working, because walmart denied me a return policy.  And my family member cannot be pulling out their phone to open garage, they need siri.
So to the point look at this JSON,  I modified Nothing
config.json.txt

and I presume I add the device to a lift master account?
Thanks.

The guy at Walmart was wrong. It's not HomeKit compatible. HomeBridge is a brilliant, amazing hack but it's not an Apple product.
Your family members better have another way to get in the house because hacks have a way of not working suddenly and without warning.
Sent from my iPhone

On Aug 1, 2016, at 4:41 PM, azulon1<EMAIL_ADDRESS>wrote:
I am having trouble understanding the fundamentals of whats happening. I mean I really don't want to learn it actually. The guy at wal-mart told me this was home kit compatible. EVEN on their demo poster that walmart had it said home kit. I own ecobee3. and let me tell you THAT WAS SO EASY to setup, like i expect from apple, i am not too good at tech stuff.
Anyways, I must get this working, because walmart denied me a return policy. And my family member cannot be pulling out their phone to open garage, they need siri.
So to the point look at this JSON, I modified Nothing
config.json.txt
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

yea man, I really fear that, but I don't know any other way. she is in a wheelchair, the only other way is that I get a smart lock on t he front door, but that has stairs you know.  So the way we have been doing it is I come having to help her in all the time, or uses siri to call me or another person.   Also I am not too lofty on money, if walmart won't take this back, then I am going to have to make this one work.
I didn't mean to inconvenience, but I have no other option.

Just wanted to make sure that you understood that even after you get it working, it's likely to be a less than 100% reliable solution so she definitely needs alternative ways to open the garage door.
Sent from my iPhone

On Aug 1, 2016, at 5:18 PM, azulon1<EMAIL_ADDRESS>wrote:
yea man, I really fear that, but I don't know any other way. she is in a wheelchair, the only other way is that I get a smart lock on t he front door, but that has stairs you know. So the way we have been doing it is I come having to help her in all the time, or uses siri to call me or another person. Also I am not too lofty on money, if walmart won't take this back, then I am going to have to make this one work.
I didn't mean to inconvenience, but I have no other option.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Well can you help me with lift master please

I don't have one so have no personal experience. Assuming you installed the appropriate modules, there's a couple of things that look wrong in your config.json. First, you don't need the platform section since you are installing the LiftMaster accessory. Next, the username and password values in the accessory section need to be in double quotes. Your config.json shows some kind of special character as the current delimiter.
You sudo installed Homebridge-liftmaster?

On Aug 1, 2016, at 17:35, azulon1<EMAIL_ADDRESS>wrote:
Well can you help me with lift master please
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

yes sir I have

Fix the config.sys issues and post the revised it along with the messages you get when you start Homebridge

On Aug 2, 2016, at 20:22, azulon1<EMAIL_ADDRESS>wrote:
yes sir I have
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

hey yo I will I am trying to google all these things.  I have a question. Is it alright that maybe we just add each other and chat through this.  Like I said I would definitely pay etc.

Join homebridgeteam on Slack and go to the LiftMaster channel

On Aug 2, 2016, at 20:27, azulon1<EMAIL_ADDRESS>wrote:
hey yo I will I am trying to google all these things. I have a question. Is it alright that maybe we just add each other and chat through this. Like I said I would definitely pay etc.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

i am on the slack, I am new to slack. so i wrote stuff on there, let me know if your getting it.

You all are using the home ridge-lift master plugin right? Not the MyQ
plugin...(Duplicate project and outdated)
On Tuesday, August 2, 2016, tannebil<EMAIL_ADDRESS>wrote:

I don't have one so have no personal experience. Assuming you installed
the appropriate modules, there's a couple of things that look wrong in your
config.json. First, you don't need the platform section since you are
installing the LiftMaster accessory. Next, the username and password values
in the accessory section need to be in double quotes. Your config.json
shows some kind of special character as the current delimiter.
You sudo installed Homebridge-liftmaster?

On Aug 1, 2016, at 17:35, azulon1<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
Well can you help me with lift master please
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nfarina/homebridge/issues/722#issuecomment-237093635,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APh0ort6srgsM90MWwyeHhqdY9p8atqEks5qb-EwgaJpZM4JXMth
.

yea, I was advised to use the lift master.  so the myq is newer?  Would you help us with creating a config.son. I don't know if I have to look up IPS or anything , I am new to that. if you could tell me what the config fi


Implementing unit tests for Inventory

I've implemented some unit tests for Inventory. I ended up making quite a few changes, including breaking the dependency Inventory had on Player. Game still works as before and all tests pass. I think this version is cleaner because now Player creates its own default inventory.
I'm just kind of blundering into this, so this isn't a helpful direction to go, please let me know.

Thanks! I'm trying to finish up some stuff I'm working on, then I'll take a look at this (if I can find the time tonight). Definitely appreciated and I'm pretty sure we had to break that dependency anyway so we can have non-player inventories, like for displaying a container's contents. Although I haven't looked too closely at that yet, so don't quote me on anything :D
Looking forward to seeing what you come up with next - want to make it official and add a post in the contributor intro forum? :-)
http://board.movingblocks.net/viewforum.php?f=7

Alright, midnight here and still not done. Tomorrow it is! I see more stuff making it in for you anyway, so pop open a thread in the forum sometime to help discuss and we'll get the code sorted out tomorrow at some point. And thanks again for the initiative!


added bash script to update TGF to latest

Pull Request Test Coverage Report for Build 808

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 43.243%

Totals

Change from base Build 806:
0.0%

Covered Lines:
416

Relevant Lines:
962

💛  - Coveralls

Nice! This is so cool

Works on Windows Subsystem for Linux now, thanks 👍


U/kvncampos 5348 UI enchancement

Closes #5348
What's Changed

Added UI enhancement for orderable columns.
UP/DOWN arrows now display sorting order.
Functionality:

First click: UP arrow, ASC sort (e.g., path/?sort=name).

Second click: DOWN arrow, DESC sort (e.g., path/?sort=-name).

Third click: Default view, no sort (e.g., path/).

Screenshots

Added new js file  to project-static/js
Included new code in templates/inc/javascript.html
Added custom-data-attribute: data-column-name for js code

Does this need to be rebased to the next branch?

@kvncampos can you rebase this to next? If there are still test failures after the rebase I'll work on getting them fixed


Limit scene broadcast publications when paused

I don't think the scenebroadcaster needs to send frequent messages when paused.
Signed-off-by: Nate Koenig<EMAIL_ADDRESS>

I haven't tried this yet, but have you checked that manipulating models in the GUI while paused still works?

I was able to move the models through the GUI when paused. On the other hand, setting the model pose through ign service did not move the model since the scene broadcaster is not publishing the new state to the gui. The model moves only after sim is unpaused.

The Physics system removes the pose change component during Update. The SceneBroadcaster's PostUpdate then misses the OneTimeChange state.
I've add a PreUpdate function to the SceneBroadcaster that captures the change event in: 79c6a4392edb43dd77e549c94346ee31d945cb55. Moving an entity from the CLI should now trigger a broadcast event.

I've add a PreUpdate function to the SceneBroadcaster that captures the change event

It looks like that broke some things. The problem of checking for changes on pre-update is that systems are adding their changes during that call, and we can't guarantee the order in which systems run. Try for example moving the SceneBroadcaster above the Physics and UserCommands systems on the SDF, and you'll see how the GUI is not working properly anymore, even the component inspector fails to show any data sometimes.

Before I go any further with this PR, I want to make sure the general direction is aligned with our goals. @chapulina , I not opposed to declining this if you think there is a different or better way to improve performance while paused.

Not publishing periodic changes while paused seems reasonable to me. We just need to adjust the GUI's expectations to match that. There may also be other downstream consumers of the state message who would be affected. I worry about such a big change in a released version, so I would suggest we target this at Edifice.

Not publishing periodic changes while paused seems reasonable to me. We just need to adjust the GUI's expectations to match that. There may also be other downstream consumers of the state message who would be affected. I worry about such a big change in a released version, so I would suggest we target this at Edifice.

I think I've addressed the problems in 17bb8f178c5cb97e8c741c5a4cdd10efda90b35a


Add support for USDT tracepoints in VAST

:notebook_with_decorative_cover: Description
See extensive description in the new header file itself.
:memo: Checklist

[ ] All user-facing changes have changelog entries.
[ ] The changes are reflected on docs.tenzir.com/vast, if necessary.
[x] The PR description contains instructions for the reviewer, if necessary.

:dart: Review Instructions
Review all commits at once.
To verify that release builds now contain trace points by default:

Download the CI artifact on a linux system
Use sudo perf buildid-cache -a /path/to/libvast.so to add the binary to the buildid cache
Use sudo perf probe --list | grep vast to see if the tracepoint exist
(optionally)
Use e.g. sudo perf probe sdt_vast:chunk_make and perf record -e sdt_vast:chunk_make -- vast start to record the samples

This requires Folly to be added to our Open-Source Licenses document. Do you want me to make this change?


Button not working

On the '/panel/Link/edit?do_delete=xxx there is a 'Articles' button, the URL is: /rapyd-demo/filter - that URL does not exists in a new panel

@rabol would be great to send a pull request to fix that


Weird behavoir when running tests on live mode

Dear Catalyst Maintainers,
Before I tell you about my issue, let me describe my environment:
Environment

Operating System: ubuntu 16.04
Python Version: 2.7
Python Bitness: 64
How did you install Catalyst pip
Python packages: pip
alembic==0.9.6
bcolz==0.12.1
boto3==1.5.1
botocore==1.8.15
Bottleneck==1.2.1
ccxt==1.10.400
certifi==2017.11.5
chardet==3.0.4
click==6.7
contextlib2==0.5.5
cyordereddict==1.0.0
Cython==0.27.3
decorator==4.1.2
docutils==0.14
empyrical==0.2.1
enigma-catalyst==0.4.0
futures==3.2.0
idna==2.6
int-date==0.1.8
intervaltree==2.1.0
jmespath==0.9.3
Logbook==1.1.0
lru-dict==1.1.6
Mako==1.0.7
MarkupSafe==1.0
multipledispatch==0.4.9
networkx==2.0
numexpr==2.6.4
numpy==1.13.3
pandas==0.19.2
pandas-datareader==0.5.0
patsy==0.4.1
pkg-resources==0.0.0
python-dateutil==2.6.1
python-editor==1.0.3
pytz==2017.3
requests==2.18.4
requests-file==1.4.2
requests-ftp==0.3.1
s3transfer==0.1.12
scipy==1.0.0
six==1.11.0
sortedcontainers==1.5.9
SQLAlchemy==1.1.15
statsmodels==0.8.0
stockstats==0.2.0
tables==3.4.2
toolz==0.9.0
urllib3==1.22

Now that you know a little about me, let me tell you about the issue I am
having:
Description of Issue
When testing your code on live mode it will save the settings in some file so you can continue where you left of. When your are testing your code this becauses problematic. I was running code that would use all your cash to buy but when i was running i live mode the cash was diffrent than the cash i really had.
This will not only happen to your cash but anything, like position and currency pair (for example the new code was running on the currency pair etc_btc but it still displayed the currency pair ltc_btc)

What did you expect to happen?
I expected the bot to use the cash I had instead of getting a cash amount from the saved file

What happened instead?
The cash amount that my bot had was not simmulair with the cash amount I really had

Here is how you can reproduce this issue on your machine:
Reproduction Steps

Run some bot
Chance the bot a bit like currency pair or use some cash you have to buy something
Run the bot again. Some values now wont match up
.

What steps have you taken to resolve this already?
version = '0.06' NAMESPACE = 'rsi_trade - v{}'.format(version)
I change the version everytime i changed my code
Anything else?
I dont know if this is the expected behavoir but what i can say is that its very anoying when you dont kow what is happening. Maybe its good to have some kind of clean cach function? Or a live testing mode? Or just a varaible that u can adjust where u say "dont save my progress"
Sincerely,
`Vincent

Sorry, I don't doubt the issue but I don't have a good enough idea of what's going on to replicate it. Which values do not match between frames. In live mode, the algo state is saved to disk. To reset it, run:
catalyst clean-algo -n {namespace of your algo}

The values that didnt match for me were cash, position and currency pair. I dont think this is a real issue since you just told me how to clean it. (Also two of the clean commands didnt work for me, but i looked at it quite quickly a while ago)
I think there should be some information about the cleaning in de docs but maybe its just me but i was really confused why my values were diffrent until i saw in the example code a file is saved every time you run life mode. I got quite some problems with expected values so in develpment i think its nice to always clean it. (I prefer not the develop in live but as a new user it is nice to test it out) (i am not just new to catalyst but trading in general)
In general alot of the confusion i had with catalyst came from not having enough information from the docs. I will be looking in how to contribute to the docs.

I tried cleaning up the algo, but the positions are still wrong. The algorithm crashed after making an order and the order got filled, but when I restart it, it says I have 0 positions, even after cleanup.

@caioaao do you have the stack trace related with this incident? Something specific must have happened. I did resolve an issue with placing orders with Poloniex last week (some inconsistency with how it's presenting the order_id). This is resolved in the "develop" branch, but we did not issued a release yet. The stack trace will allow me to confirm that it's the same issue.

I think it was just an API instability or backpressure (idk how many requests/sec are being made to poloniex api when a buy happens). anyway, here's the stack trace when it blew up:
[2017-12-29 23:19:00.993179] WARNING: exchange_blotter: check open orders attempt 0: Request failed: 'Exchange'
[2017-12-29 23:19:07.512328] WARNING: exchange_blotter: check open orders attempt 1: Request failed: 'Exchange'
[2017-12-29 23:19:12.658269] WARNING: exchange_blotter: check open orders attempt 2: Request failed: 'Exchange'
[2017-12-29 23:19:19.220624] WARNING: exchange_blotter: check open orders attempt 3: Request failed: 'Exchange'
[2017-12-29 23:19:24.323487] WARNING: exchange_blotter: check open orders attempt 4: Request failed: 'Exchange'
[2017-12-29 23:19:30.453291] WARNING: exchange_blotter: check open orders attempt 5: Request failed: 'Exchange'
Traceback (most recent call last):
File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
"__main__", fname, loader, pkg_name)
File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/usr/local/lib/python2.7/dist-packages/moneyprinter/strategies/bbands.py", line 185, in <module>
live=True)
File "/usr/local/lib/python2.7/dist-packages/catalyst/utils/run_algo.py", line 563, in run_algorithm
stats_output=stats_output
File "/usr/local/lib/python2.7/dist-packages/catalyst/utils/run_algo.py", line 346, in _run
overwrite_sim_params=False,
File "/usr/local/lib/python2.7/dist-packages/catalyst/algorithm.py", line 724, in run
for perf in self.get_generator():
File "/usr/local/lib/python2.7/dist-packages/catalyst/gens/tradesimulation.py", line 224, in transform
for capital_change_packet in every_bar(dt):
File "/usr/local/lib/python2.7/dist-packages/catalyst/gens/tradesimulation.py", line 122, in every_bar
blotter.get_transactions(current_data)
File "/usr/local/lib/python2.7/dist-packages/catalyst/exchange/exchange_blotter.py", line 297, in get_transactions
return self.get_exchange_transactions()
File "/usr/local/lib/python2.7/dist-packages/catalyst/exchange/exchange_blotter.py", line 283, in get_exchange_transactions
return self.get_exchange_transactions(attempt_index + 1)
File "/usr/local/lib/python2.7/dist-packages/catalyst/exchange/exchange_blotter.py", line 283, in get_exchange_transactions
return self.get_exchange_transactions(attempt_index + 1)
File "/usr/local/lib/python2.7/dist-packages/catalyst/exchange/exchange_blotter.py", line 283, in get_exchange_transactions
return self.get_exchange_transactions(attempt_index + 1)
File "/usr/local/lib/python2.7/dist-packages/catalyst/exchange/exchange_blotter.py", line 283, in get_exchange_transactions
return self.get_exchange_transactions(attempt_index + 1)
File "/usr/local/lib/python2.7/dist-packages/catalyst/exchange/exchange_blotter.py", line 283, in get_exchange_transactions
return self.get_exchange_transactions(attempt_index + 1)
File "/usr/local/lib/python2.7/dist-packages/catalyst/exchange/exchange_blotter.py", line 289, in get_exchange_transactions
error=e
catalyst.exchange.exchange_errors.ExchangePortfolioDataError: Unable to retrieve portfolio data: order-status, giving up after 5 attempts: Request failed: 'Exchange'
No newer events found at the moment. Retry.

the weirdness happens afterwards, when I restart:
[2017-12-29 23:30:49.753261] INFO: run_algo: running algo in live-trading mode
[2017-12-29 23:30:52.598118] INFO: run_algo: base currency available in the account: [redacted] usdt
[2017-12-29 23:30:52.600201] INFO: exchange_algorithm: initialized trading algorithm in live mode
[2017-12-29 23:30:52.901352] INFO: exchange_algorithm: got totals from exchanges, cash: [redacted] positions: 0.0
[2017-12-29 23:30:57.104007] INFO: exchange_algorithm: statistics for the last 10 minutes:
starting_cash ending_cash portfolio_value pnl long_exposure short_exposure orders transactions
period_close
2017-12-29 23:30:00+00:00 [redacted] [redacted] [redacted] 0.0 0 0 0 0

the result was the same before and after a cleanup. there was actually a positive balance of XRP that the algorithm couldn't see

Yes, this actually looks like the issue which is resolved in the develop
branch. It placed the order just fine, but the retries happen after when
trying to fetch the order_status afterwards to confirm the transaction.
There is a specific difference with the poloniex data which I did not catch
testing with Binance and Bittrex. You can work around this issue by testing
with the develop branch. We’ll make sure to push a release on Tuesday.
On Sat, Dec 30, 2017 at 5:04 AM Caio Oliveira<EMAIL_ADDRESS>wrote:

I think it was just an API instability or backpressure (idk how many
requests/sec are being made to poloniex api when a buy happens). anyway,
here's the stack trace when it blew up:
[2017-12-29 23:19:00.993179] WARNING: exchange_blotter: check open orders attempt 0: Request failed: 'Exchange'
[2017-12-29 23:19:07.512328] WARNING: exchange_blotter: check open orders attempt 1: Request failed: 'Exchange'
[2017-12-29 23:19:12.658269] WARNING: exchange_blotter: check open orders attempt 2: Request failed: 'Exchange'
[2017-12-29 23:19:19.220624] WARNING: exchange_blotter: check open orders attempt 3: Request failed: 'Exchange'
[2017-12-29 23:19:24.323487] WARNING: exchange_blotter: check open orders attempt 4: Request failed: 'Exchange'
[2017-12-29 23:19:30.453291] WARNING: exchange_blotter: check open orders attempt 5: Request failed: 'Exchange'
Traceback (most recent call last):
File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/usr/local/lib/python2.7/dist-packages/moneyprinter/strategies/bbands.py", line 185, in
live=True)
File "/usr/local/lib/python2.7/dist-packages/catalyst/utils/run_algo.py", line 563, in run_algorithm
stats_output=stats_output
File "/usr/local/lib/python2.7/dist-packages/catalyst/utils/run_algo.py", line 346, in _run
overwrite_sim_params=False,
File "/usr/local/lib/python2.7/dist-packages/catalyst/algorithm.py", line 724, in run
for perf in self.get_generator():
File "/usr/local/lib/python2.7/dist-packages/catalyst/gens/tradesimulation.py", line 224, in transform
for capital_change_packet in every_bar(dt):
File "/usr/local/lib/python2.7/dist-packages/catalyst/gens/tradesimulation.py", line 122, in every_bar
blotter.get_transactions(current_data)
File "/usr/local/lib/python2.7/dist-packages/catalyst/exchange/exchange_blotter.py", line 297, in get_transactions
return self.get_exchange_transactions()
File "/usr/local/lib/python2.7/dist-packages/catalyst/exchange/exchange_blotter.py", line 283, in get_exchange_transactions
return self.get_exchange_transactions(attempt_index + 1)
File "/usr/local/lib/python2.7/dist-packages/catalyst/exchange/exchange_blotter.py", line 283, in get_exchange_transactions
return self.get_exchange_transactions(attempt_index + 1)
File "/usr/local/lib/python2.7/dist-packages/catalyst/exchange/exchange_blotter.py", line 283, in get_exchange_transactions
return self.get_exchange_transactions(attempt_index + 1)
File "/usr/local/lib/python2.7/dist-packages/catalyst/exchange/exchange_blotter.py", line 283, in get_exchange_transactions
return self.get_exchange_transactions(attempt_index + 1)
File "/usr/local/lib/python2.7/dist-packages/catalyst/exchange/exchange_blotter.py", line 283, in get_exchange_transactions
return self.get_exchange_transactions(attempt_index + 1)
File "/usr/local/lib/python2.7/dist-packages/catalyst/exchange/exchange_blotter.py", line 289, in get_exchange_transactions
error=e
catalyst.exchange.exchange_errors.ExchangePortfolioDataError: Unable to retrieve portfolio data: order-status, giving up after 5 attempts: Request failed: 'Exchange'
No newer events found at the moment. Retry.
the weirdness happens afterwards, when I restart:
[2017-12-29 23:30:49.753261] INFO: run_algo: running algo in live-trading mode
[2017-12-29 23:30:52.598118] INFO: run_algo: base currency available in the account: [redacted] usdt
[2017-12-29 23:30:52.600201] INFO: exchange_algorithm: initialized trading algorithm in live mode
[2017-12-29 23:30:52.901352] INFO: exchange_algorithm: got totals from exchanges, cash: [redacted] positions: 0.0
[2017-12-29 23:30:57.104007] INFO: exchange_algorithm: statistics for the last 10 minutes:
starting_cash ending_cash portfolio_value pnl long_exposure short_exposure orders transactions
period_close
2017-12-29 23:30:00+00:00 [redacted] [redacted] [redacted] 0.0 0 0 0 0
the result was the same before and after a cleanup. there was actually a
positive balance of XRP that the algorithm couldn't see
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/enigmampc/catalyst/issues/112#issuecomment-354545235,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABZ-Qg1WIQetn2qQlt8Zh_gJUbQrG3Syks5tFjTOgaJpZM4RIKfV
.

@fredfortier great, thanks! btw, how "safe" is the develop branch? I mean, is there someone running live algos with it?

It’s not a stable branch. We’re trying not to push breaking in changes but
that’s our main development branch.
On Sat, Dec 30, 2017 at 6:01 AM Caio Oliveira<EMAIL_ADDRESS>wrote:

@fredfortier https://github.com/fredfortier great, thanks! btw, how
"safe" is the develop branch? I mean, is there someone running live algos
with it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/enigmampc/catalyst/issues/112#issuecomment-354547728,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABZ-QlgSRr0-xQabqH22zm7PGIbqjh9Wks5tFkI_gaJpZM4RIKfV
.


test: Add remaining tests for Event and Attendee model and presenter

Fixes #121

Complete all cases of tests for Event and Attendee Repository, Presenter and Model

@niranjan94 Codacy gives a lot of false positives. This affects its rating of code that might be perfectly OK

@mejariamol Please approve

@niranjan94 Please merge


Internal/External VIP

What if I have no "external" vip and it's all inside my internal network?

It's a good question - I cannot instantly think a reason why this would not work but the devil is in details and I haven't tested this myself. IPVS/LVS does not support the scenario where VIP and real server are on the same host or real servers are using VIP but that that should not be needed here, HAProxy is used for communication between masters and otherwise you should be able to use keepalived in DR mode for handling incoming connections outside of the OCP cluster.
However, this would require you to try this out and try to find the working setup, perhaps you could start by creating the initial configuration with the playbook, then removing parts that are needed only in the two NIC setup (e.g., in iptables) and work it from there.


7# add forms

Dear Reviewer!

In this PR I followed the instructions in Blog app - Add forms,
I implemented the following:

Create a post functionality
Add a comment functionality
Add a like functionality

@MattGomb @Lameck1 - I approved it manually. No need to request another review

@Lameck1, @enniopo, thank you for the clarification, and the approval!


fix tags when baseURL = "/"

This PR resolves #16
I've seen this trick used in other themes, such as Terminal, and it's worked well for me. Just tested on my own site, deployed to two different domains, with baseURL = "/", and it works great.

Thanks!


GitHub API limits cause packages to fail restoring

Description
Fetching gist dependencies fails when GitHub API reaches the call limit.
Repro steps
Build enough times on a cloud CI to start receiving errors.
Actual behavior
See this build https://ci.appveyor.com/project/tpluscode78631/nancy-routing-uritemplates/build/1.0.18. Paket fails to restore.
Known workarounds
There is a similar issue with JSPM, where it downloads packages directly from GitHub. On Travis it's possible to add an encrypted environment variable with a personal ACCESS_TOKEN

I've just found this in the docs: Using a GitHub authentication key from environment variable.
Closing


T-Display S3

Wondering if it would be possible for support to be added for the T-Dispaly S3 1.9in listed here: https://www.amazon.com/LILYGO-T-Display-S3-ESP32-S3-Display-Development/dp/B0BRTT727Z
Any suggestions would be appreciated!
I managed to get if flashed but cannot get it to run or even connect to esp via putty

did you make anythink


[Android]border-radius and border-width and border-color error

if border-radius less than border-width，border-radius not take effect。
if border-color has alpha, the color of Corner is dark than border。
here is code
<template>

</template>

Weex may not render as expected if you need a smooth transition between different border-radius or border-width on adjoining sides.

As it's said in the documentation, this is the issue of Android system itself, and there is not much we can do.
Anyway, I will look into the border-width and border-radius problem, but this issue may not be fixed. As for border-color: rgba(0,0,0,.5);, please don't use alpha, there is nothing we can do to combine alpha, border-width, border-radius in Android View system.


Client throws TypeError, doesn't load on test chain (networkid 10101)

Client doesn't work on the test chain (networkid 10101).  I get this error in my Firebug console:
TypeError: this.contractAddress[this.networkId] is undefined
http://localhost:8080/app.js
Line 46740

This error goes away if I check out commit 5869ee47e6fed09500cf793e48a1df4304df87b7, so it was introduced in the last couple commits.

i think this is fixed now.  can you confirm?  i'm using augur-contracts, so it should be aware of any network id in there.

It's throwing a different error now:
TypeError: result.toNumber is not a function
http://localhost:8080/app.js
Line 44870

(Not sure if the original error is fixed or not -- it may be "underneath" this error...)

Fixed in 3c5a5a0adc4d73a857c17a1d2c7ec2c66cd528b9.


style(frontend): Change icons for the Hero back and menu buttons

Motivation
We update the icons for the Back button and the Context Menu in the Hero.

@peterpeterparker about the [OISY-62], that is mostly because Jira auto-link the PRs to the ticket, so it is easier to follow-up in Jira itself. It is mostly for our (my) own internal management of tasks

It is mostly for our (my) own internal management of tasks

I meant prominently. If for example it's linked in the description, for sure I'm fine with it. I just meant that so expressively, in the PR title, feels really corporate and not open source.

@peterpeterparker I didn't know it would work in the description too! Changed!

[OISY-62]


Validation Bulk Fixes

Docs Build status updates of commit 3534430:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-services/latest/messaging-eventhubs-checkpointstore-blob-readme.md
:warning:Warning
View (azure-java-stable)
Details

docs-ref-services/latest/messaging-eventhubs-readme.md
:warning:Warning
View (azure-java-stable)
Details

docs-ref-services/latest/messaging-eventhubs-checkpointstore-blob-readme.md

Line 2, Column 1: [Warning: ms-prod-missing - See documentation] Missing attribute: 'ms.prod'. If you specify 'ms.technology', you must also specify 'ms.prod'.
Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

docs-ref-services/latest/messaging-eventhubs-readme.md

Line 2, Column 1: [Warning: ms-prod-missing - See documentation] Missing attribute: 'ms.prod'. If you specify 'ms.technology', you must also specify 'ms.prod'.
Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit c416d57:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-services/latest/messaging-eventhubs-checkpointstore-blob-readme.md
:white_check_mark:Succeeded
View (azure-java-stable)

docs-ref-services/latest/messaging-eventhubs-readme.md
:white_check_mark:Succeeded
View (azure-java-stable)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Global effort to fix build validation errors
@conniey
The Content & Learning team is fixing build validation errors on docs.microsoft.com for the rest of H2. This will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. Please review within five business days and merge or comment in the PR with any changes you’d like to see. Thank you!


Switch to eth_utils.HasXXXLogger

What was wrong?
How was it fixed?
Cute Animal Picture

Seems like this ought to block on https://github.com/ethereum/eth-utils/issues/159 because the logging check  has a significant EVM performance impact.

Looks like this doesn't play nicely with abc.ABC.  Ends up with a metaclass error that I think needs to be resolved at the eth-utils level.  Might opt to re-work it using brian's descriptor approach which resolves both abc.ABC and typing.Generic.


[Feature] Install KubeVela on cluster with kubeconfig file

Is your feature request related to a problem? Please describe.
vela CLI can only install the KubeVela on default cluster which is present in Kubernetes configuration file ~/.kube/config
Describe the solution you'd like
--kubeconfig Flag to provide custom Kubernetes configuration file in vela install
vela install  --kubeconfig   kubeconfig-file

@kethavathsivanaik you can set the environment.
export KUBECONFIG=<your kube config file path>
vela install

@kethavathsivanaik There is no more feedback, I will close this issue, thanks!


[building-rollup] dynamic imports broken

I just updated to 0.7.1 and got an issue with __dynamicImport__ is not defined when I run my app.
The same thing can be reproduced in the open-wc repo:

cd to packages/building-rollup
npm run build:modern
npm run build:start
open demo webpage
click lazy load button

Sorry about that, something got mixed up in a merge. Fixing it now!

cool, thanks for fixing it so fast


ARROW-11245: [C++][Gandiva] Add support for LLVM 11.1

It's not released yet but RC1 is available.

@github-actions crossbow submit test-debian-10-cpp

@github-actions crossbow submit test-debian-10-cpp

@github-actions crossbow submit -g linux

@github-actions crossbow submit -g linux

+1

+1


Is there a HighPass or Motion Detector available for GPUImage3

Hello,
As an avid user of GPUImage, I am finally making the move to GPUImage3. Getting started and I haven't found the HighPass, LowPass or MotionDetection filters. Are they available or easy to make by subclassing other things?

Fyi, I took HighPass filter from MetalPetal framework.

@BradLarson Funny meeting you here. I remember when GPUImage went from v1 to v2. Glad to see you are still doing the fun stuff.


[Enhancement]: Improve Styling of Pagination Buttons Section

Is your enhancement request related to a problem? Please describe.➕
GSSOC'23
Issue Description

In Pagination section, Improve Enhancement.
Suppose, When Click on Next And Previous Buttons. That selected Button & It's Pagination Number Background Color is Not Much Attractive.

Describe the solution you'd like
Solution steps:

In Pagination Number Digits, Replace Previous Color to Pure white Color.
It happens Only on Either Click on selected Pagination Number OR Click on Previous or Next Buttons.

Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @NarendraKoya999 , You can't have another issue before completing the previous one 😀
you already have the following 1 open issues 👀 ! :
#1438

Hey @NarendraKoya999 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇

Hey @NarendraKoya999 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹


onMonthChanged is not being triggered

Environment

Flutter version: 3.16.5
Dart version: 3.2.3
Library version: 0.3.15+35

Description
I recently updated to version 0.3.15+35 from 0.2.3+11, which, according to the changelog, includes a fix for the onMonthChanged callback not being triggered. However, I am still experiencing this issue in my application.
Expected Behavior
The onMonthChanged callback should be triggered as documented when changing to a different month.
Actual Behavior
The onMonthChanged callback is not being triggered upon changing months.
Code Snippet
Calendar(
  hideBottomBar: true,
  hideTodayIcon: true,
  startOnMonday: true,
  events: sReserva.hasData && sReserva.data != null
      ? sReserva.data
      : {},
  isExpandable: false,
  isExpanded: true,
  defaultDayColor: Colors.black,
  selectedColor: Colors.blue.shade800,
  todayColor: Colors.black,
  eventListBuilder: (BuildContext context,
      List<NeatCleanCalendarEvent> _selectesdEvents) {
    return SizedBox();
  },
  onDateSelected: (date) async {
    _bloc.inDay.add(date);
    await _bloc.findReserve(date);
    await _bloc.isReserve(date);
  },
  onMonthChanged: (date) async {
    await _bloc.findDays(date); // It won't trigger
  },
  locale:
      localizationDelegate.currentLocale.toString(),
  weekDays: [
    translate("SEMANA.SEG"),
    translate("SEMANA.TER"),
    translate("SEMANA.QUA"),
    translate("SEMANA.QUI"),
    translate("SEMANA.SEX"),
    translate("SEMANA.SAB"),
    translate("SEMANA.DOM"),
  ],
  dayOfWeekStyle: TextStyle(
      color: Colors.black,
      fontWeight: FontWeight.w800,
      fontSize: 11),
  );

Same issue here and I'm using the same version

Same here. Methods nextMonth, nextWeek, previousMonth, previousWeek don't trigger the callbacks
  void nextMonth() {     setState(() {       _selectedDate = Utils.nextMonth(_selectedDate);       var firstDateOfNewMonth = Utils.firstDayOfMonth(_selectedDate);       var lastDateOfNewMonth = Utils.lastDayOfMonth(_selectedDate);       updateSelectedRange(firstDateOfNewMonth, lastDateOfNewMonth);       selectedMonthsDays = _daysInMonth(_selectedDate);       var monthFormat =           DateFormat('MMMM yyyy', widget.locale).format(_selectedDate);       displayMonth =           '${monthFormat[0].toUpperCase()}${monthFormat.substring(1)}';       _selectedEvents = eventsMap?[DateTime(               _selectedDate.year, _selectedDate.month, _selectedDate.day)] ??           [];     });   }

I can confirm, that onMonthChanged currently only triggers, when the user selects a day in the calendar view, that belongs to another month.
Selecting a new date by using the date picker or changing the month by using the arrow buttons doesn't trigger.

@rwbr this issue can be closed, it was fixed with the last release


[Event Request] Table 7320 "Warehouse Shipment Header" - OnBeforeValidateShipmentDate

In order to support overriding the Confirm dialog when changing the Shipment Date on a Warehouse Shipment Header, it would be helpful to have an IsHandled-event publisher on the trigger.
Current code:
field(45; "Shipment Date"; Date)
{
    Caption = 'Shipment Date';

    trigger OnValidate()
    var
        WhseShptLine: Record "Warehouse Shipment Line";
    begin
        if "Shipment Date" <> xRec."Shipment Date" then begin
            WhseShptLine.SetRange("No.", "No.");
            if not WhseShptLine.IsEmpty() then
                if Confirm(
                     StrSubstNo(Text008, FieldCaption("Shipment Date")), false)
                then
                    WhseShptLine.ModifyAll("Shipment Date", "Shipment Date");
        end;
    end;
}

Proposed change:
field(45; "Shipment Date"; Date)
{
    Caption = 'Shipment Date';

    trigger OnValidate()
    var
        WhseShptLine: Record "Warehouse Shipment Line";
+       IsHandled: Boolean;
    begin
+       IsHandled := false;
+       OnBeforeValidateShipmentDate(Rec, xRec, IsHandled);
+       if IsHandled then
+           exit;

        if "Shipment Date" <> xRec."Shipment Date" then begin
            WhseShptLine.SetRange("No.", "No.");
            if not WhseShptLine.IsEmpty() then
                if Confirm(
                     StrSubstNo(Text008, FieldCaption("Shipment Date")), false)
                then
                    WhseShptLine.ModifyAll("Shipment Date", "Shipment Date");
        end;
    end;
}

And the proposed event publisher:
[IntegrationEvent(false, false)]
local procedure OnBeforeValidateShipmentDate(var WarehouseShipmentHeader: Record "Warehouse Shipment Header"; xWarehouseShipmentHeader: Record "Warehouse Shipment Header"; var IsHandled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Canary builds

Issue Type

[ ] Bug Report
[x] Feature Request
[ ] Other

Current/Missing Behaviour
Only release builds are downloaded by checking nwjs.io/versions.
Expected/Proposed Behaviour

Support nightly builds via dl.nwjs.io/live-build.
Add options.channel default value of "release", other options being "canary".

Additional Info

Package version: N/A
Operating System: N/A
Node version: N/A
NW.js version: N/A
Repro link: N/A
...

See https://github.com/nwutils/nw-builder/commit/9a92688d848a83a8317196c254009a6c1eeffa44


Try GitHub Actions as alternative to Travis CI

The builds on PR #391 have been getting stuck. I found out why here: https://travis-ci.com/github/beijbom/coralnet/requests
"Job rejected" "Owner beijbom does not have enough credits."
I've heard that Travis CI introduced significant limitations to their free plan in November 2020, so it's probably because of that. Specifically, it seems the free plan gives 10,000 credits (lifetime, in our case starting November 2020), and 10 credits are used per build minute.
GitHub Actions may be worth a try instead. It's probably nicer to have CI rolled right into GitHub anyway, rather than living in a separate service.

Completed in PR #393. Now using GitHub Actions as our CI.


Relevance of Arcus.Scripting.All PowerShell module

Is your feature request related to a problem? Please describe.
Currently, we have two .All PowerShell modules in our solution: 1 that collects all the Azure Storage-related modules, and 1 that collects all our modules. The problem with the second one is that it is harder to maintain and by time less relevant as there may be no scenarios that requires all our PowerShell modules.
Describe the solution you'd like
We can remove the Arcus.Scripting.All PowerShell module from our solution and provide a deprecation message in the PowerShell gallery saying that it will no longer be maintained and that references to more concrete modules is necessary.

Currently, this module is also not up-to-date as it does not contain all our PowerShell modules (like Arcus.Scripting.IntegrationAccount and Arcus.Scripting.AppService).

I'm in favor of removing the '.All' modules.  It is indeed true that there is no real use-case where you would want to pull in all modules.


Allow Application Security Groups to be Applied to VM's

The Gyro implementation currently has a class gyro.azure.network.ApplicationSecurityGroupResource. This resource can be used when creating gyro.azure.network.NetworkSecurityGroupRuleResource.
However, I can't find a way to associate the ApplicationSecurityGroup with a VM or Network Interface through Gyro.
The workflow is explained here using the Azure console:
https://docs.microsoft.com/en-us/azure/virtual-network/tutorial-filter-network-traffic#associate-network-interfaces-to-an-asg
This feature would be valuable to ensuring that our security group rules are grouped by function.

I've created an issue in Azure to help point us to the API in the Java SDK that can help us accomplish this.

It looks like Application Security groups are added to the Network Interfaces associated with the VM. The following is copied from the response I received from an Azure support ticket.
https://gist.github.com/tloisel1/ce2547aed4fa9399a5432f77f8f6bf05#file-azure-application-security-group-L141-L142


OpenSpiealGraph link added to the documentation

In the documentations i added a link to OpenSpielGraph, a react sigma web application where you can see OpenSpiel c++ and python files as an interactive graph.

Hi @Max-Antonio, can you sign the CLA?
See the above.. click on "Details" and follow the instructions.


Updat for pyomicron v2.0.7

Checklist

[x] Used a personal fork of the feedstock to propose changes
[n/a ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ x Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender in


Update maintainers for vsphere-cpi

I believe the maintainers list for the vsphere-cpi is out of date. This should be updated:
https://github.com/vmware-tanzu/community-edition/blob/5b8537e2c64b902553b57956af0ba650b18b2387/addons/packages/vsphere-cpi/metadata.yaml#L10-L13
Perhaps it should be sync'd with this member list?
https://github.com/orgs/vmware-tanzu/teams/pkg-vsphere-cpi-owners/members
Not sure if that list is up to date, but it looks closer to me...
cc @vmware-tanzu/pkg-vsphere-cpi-owners

Resolved based on #2756


Improvements for Aperture

The magnification is obsolete and does not provide any extra information. This is a reminder to myself to remove it at a later date.

Also I might split the hierarchy so that RadiallySymmetricAperture is a class. This would avoid applying null rotations to CircularAperture and AnnularAperture which happens at the moment.

We probably don't want to be applying rotations to the arrays very often, for speed


Bump my packages

Motivation for this change
Bumping some of my packages with update.nix.

ptcollab: 0.4.0 -> 0.4.1
libsidplayfp: 2.1.1 -> 2.2.0
sidplayfp: 2.1.1 -> 2.2.0
punes: unstable-2021-04-25 -> unstable-2021-06-05

Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 127061 run on x86_64-linux 1

7 packages built:

audacious
gjay
libsidplayfp
ptcollab
punes
qmplay2
sidplayfp

Most binaries run. The binaries in qmplay2 crash with the following error, unrelated to bumps & reproducible on master:
Inconsistency detected by ld.so: dl-open.c: 913: _dl_open: Assertion `_dl_debug_initialize (0, args.nsid)->r_state == RT_CONSISTENT' failed!

Result of nixpkgs-review pr 127061 run on x86_64-darwin 1

1 package marked as broken and skipped:

ptcollab

2 packages built:

libsidplayfp
sidplayfp

Thank you!


ctx cannot be resolved

Just started with the basic tutorial
The compiler keep complaining:"ctx cannot be resolved

"

env: ubuntu 20.04, jdk1.8, with eclipse

It's probably problem with your cache/corrupted jar archive/literally everything, but it's not quite connected to Javalin itself. Are you able to reproduce this issue in other project/other IDE like IntelliJ?

My guess is that the project is set to language level below 8

Might be closed I think


How to search from the last ID?

Any method to search from the last ID registered in the database onwards?

Hi @alexv96 ,
no there is no such thing. You have to save that ID somewhere and do it yourself :)
Best regards,


Integrate marketplace tests into npm test

The marketplace tests added in #364 aren't yet integrated into npm test, which means they're invisible to CI. Filing this to make more visible.
@tyleryasaka, not sure if you were planning to work on this

@cuongdo Yeah I'll look into this. I would rather that we wait until we decide whether we're using the Truffle tests or the custom test helper. Then we can start migrating all our tests to that format and make sure npm test runs those tests.
I'll be looking into it.

Note that the OpenZeppelin token tests in contracts/test/openzeppelin are all imported from OpenZeppelin. Because OpenZeppelin is still gradually evolving their tests, it'd be nice to keep those tests in their original form (Truffle tests) to make merging test changes trivial.

@cuongdo Why are we running OpenZeppelin tests? Did we not decide not to modify the OpenZeppelin contract code? (If we're not modifying the OpenZeppelin contracts, we have no need for the tests.)

We have some additions to the contracts to allow possible future token migrations (like Augur's) :
https://github.com/OriginProtocol/origin-js/pull/371
This restricts burning of tokens to the contract owner (OpenZeppelin allows any token holder to burn tokens):
https://github.com/OriginProtocol/origin-js/pull/345/commits/8add1d7985f6aa0ca838e66215bd030d0dbb5d1a#diff-a45ccc6cc0733f203d0983e4629180caR30
Also, the OpenZeppelin token code was created in a different build environment, and their test environment doesn't test with the set of token contracts ours inherits from.
In short, there are enough differences that I'd be worried about the token code atrophying in our environment without OpenZeppelin tests running regularly.
However, they don't necessarily have to run in every invocation of npm test. They can just run during CI, for example.

Resolved by https://github.com/OriginProtocol/origin-js/pull/380


Monitor Summary - Hostname or IP Address

Describe the bug
When opening a monitor, there is a field called
Monitor Summary
There is a field called hostname or ip
The following information is displayed in this field
[object Object]
To Reproduce
Steps to reproduce the behavior:
1 - Go to a monitor page
2 - Look at the monitor summary field
Expected behavior
A clear and concise description of what you expected to happen.
I believe the IP or hostname of what is being monitored should appear?
Screenshots

This is fixed and will be released today: https://github.com/OneUptime/oneuptime/commit/a6cb80f64053999b06678f355b21c0d85faa1044


Provide an alternative way to retrieve available public ips from a subnet

Hello there !
I've been lurking about the issue about getting the first free available ip address from phpipam (https://github.com/phpipam/phpipam/issues/2960) and I was wondering if there were some alternative solution.
Basically, I'd like to implement a new datasource, let's say subnet_free_addresses and would expose the available IP addresses by the following process:

fetch a given subnet as CIDR format
cast the CIDR address as a list of IP addresses
fetch the list of addresses of the given subnet
remove all used address from the list computed in step 2

Having a PHPIPAM endpoint that would retrieve a subnet available ips would drastically simplify that process as it would remove all CIDR computations.

Once that data source would be available, we could write our modules like so :

data "phpipam_subnet_free_addresses" "free_addresses" {
  subnet_id = 12
}

resource "vmware_vm" "my_vms" {
  count = 3
  // .... //
  ipv4_addr = data.phpipam_subnet_free_addresses.free_addresses[count.index].ipv4_addr
}

It feels to me like it is a good alternative to the first_free_address endpoint usage which does not work in our case. What do you think about it ?
If the solution seems convenient to you, I'd like to suggest a pull request that would implement such solution.
Bye !

I ended up with something like this seems fully functional. I'm quite new to phpipam so I don't know if there are some edge cases where it won't work but it fits out internal needs for now.
Let's say I need to get 10 free IP addresses :
data "phpipam_subnet" "my_subnet" {
  subnet_address = "<IP_ADDRESS>"
  subnet_mask    = 24
}

data "phpipam_nth_free_addresses" "ten_addresses" {
  number    = 10
  subnet_id = data.phpipam_subnet.my_subnet.subnet_id
}

output "some_free_addresses" {
  value = data.phpipam_nth_free_addresses.ten_addresses
}

# reuse the ten_addresses source the way you want when spinning VMs (don't forget to use the `phpipam_address` resource for each IP address !)

I ended by adding a number property because retrieving all the free ip addresses of a subnet can be quite complex and irrelevant (thus I guess that anyone should be able to compute the number of desired IPs).
I'm cleaning my work then I'll create a pull request !

Hi @julien-jean
The task with multiple IPs has been a long time ago for me.
I can't remember all the nuances of the phpipam problem with receiving free_address in loop.
In my case, previously, I needed to get the last IP address in a loop from different subnets, not from the same subnet.
I was able to solve this with terraforms using loops and explicit use of subnet data source indexes.
I don't think my code would be readable here.
Feel free to create your Pull Request. Maybe new feature will be useful to other users of this terraform provider.

@julien-jean Will you provide some pull request about this issue or I can close this ?

Closed because of inactivity


Unclear documentation for Azure App Service certificate imports

The note at the top of this article about Azure App Service certificates is not intuitive.
It says that for Key Vaults using RBAC, you have to assign the Key Vault Secrets and Key Vault Reader role to the "Microsoft Azure App Service", however, it is not clear that this is referring to an Enterprise Application (Object ID = f5596fd3-a1d7-4db0-bdd3-f2fef4db0126), not the managed identity of the App Service.
Additionally, that is not the correct identity for Azure Government. The correct one for Azure Gov is "Fairfax Azure WebApps" (Object ID = 4e29c68d-0819-41df-bf24-c4c17e47a6b8). This should be clarified, and the correct apps for each Azure cloud should be specified.

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

ID: 44b9a84f-6765-67dc-bb7b-04303e070c03
Version Independent ID: 32fc20a0-4321-c74a-34aa-909f43a72db0
Content: Grant permission to applications to access an Azure key vault using Azure RBAC
Content Source: articles/key-vault/general/rbac-guide.md
Service: key-vault
Sub-service: general
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@bguidinger
Thanks for your feedback! We will investigate and update as appropriate.

@msmbaldwin
Can you please check and add your comments on this doc update request as applicable.

@bguidinger
I'm going to assign this to the document author so they can take a look at it accordingly

@bguidinger "Microsoft Azure App Service" is the identity used on RBAC blade. For detail information of App Service you will need to refer to their documentation.

@jlichwa that is not true for Azure Government where it i should be the "Fairfax Azure WebApps" identity. The documentation needs to be updated to include this, as well as the necessary Identities for the other national clouds.

@bguidinger For other clouds you will need to refer to App Service documentation. We can explicitly say public cloud?

Thanks @jlichwa. For anyone else who comes across this issue in the future, this is documented on the following Azure App Service docs page now: https://learn.microsoft.com/en-us/azure/app-service/configure-ssl-certificate?tabs=apex#authorize-app-service-to-read-from-the-vault.


SaveAsImage add chart watermark

What problem does this feature solve?
I need to add some customization on the chart (e.g. watermark), that needs to happen only when the chart's saveAsImage toolbox button is clicked.
What does the proposed API look like?
I believe this can be fixed with 2 different ways.
One way is to add a backgroundImage option is saveAsImage chart option.
https://ecomfe.github.io/echarts-doc/public/en/option.html#toolbox.feature.saveAsImage
this backgroundImage will be added as background to the chart only in the exported image
produced when the chart's saveAsImage toolbox button is clicked.
The other way I can think of is to add two events on the chart called saveAsImageBefore/After (like restore event exists)
https://ecomfe.github.io/echarts-doc/public/en/api.html#events.restore
saveAsImageBefore will have some code to draw the watermark, and saveAsImageAfter will hide the watermark.

I think you should make a custom toolbox feature and in onclick, add watermark with graphic and get the image using getDataURL.

Yes that sounds like a reasonable solution.
Thank you for your help !

Hey @MINDoSOFT,
Did you manage to get this working? I'm trying to combine the Echarts example code to generate a save with watermark functionality.
The onclick event looks something like:
function () {
          var ecModel = this.ecModel.setOption({
            graphic : [
              {
                  type: 'group',
                  rotation: Math.PI / 4,
                  bounding: 'raw',
                  right: '10%',
                  bottom: '10%',
                  z: 100,
                  children: [
                      {
                          type: 'rect',
                          left: 'center',
                          top: 'center',
                          z: 100,
                          shape: {
                              width: 400,
                              height: 50
                          },
                          style: {
                              fill: 'rgba(0,0,0,0.3)'
                          }
                      },
                      {
                          type: 'text',
                          left: 'center',
                          top: 'center',
                          z: 100,
                          style: {
                              fill: '#fff',
                              text: 'ECHARTS LINE CHART',
                              font: 'bold 26px sans-serif'
                          }
                      }
                  ]
              }
          ]
      });
    debugger;
        const model = this.model;
        const api = this.api;
        const title = model.get('name') || ecModel.get('title.0.text') || 'echarts';
        const isSvg = api.getZr().painter.getType() === 'svg';
        const type = isSvg ? 'svg' : model.get('type', true) || 'png';
        const url = api.getDataURL({
            type: type,
            backgroundColor: model.get('backgroundColor', true)
                || ecModel.get('backgroundColor') || '#fff',
            connectedBackgroundColor: model.get('connectedBackgroundColor'),
            excludeComponents: model.get('excludeComponents'),
            pixelRatio: model.get('pixelRatio')
        });
        const browser = env.browser;
        // Chrome, Firefox, New Edge
        if (isFunction(MouseEvent) && (browser.newEdge || (!browser.ie && !browser.edge))) {
            const $a = document.createElement('a');
            $a.download = title + '.' + type;
            $a.target = '_blank';
            $a.href = url;
            const evt = new MouseEvent('click', {
                // some micro front-end framework， window maybe is a Proxy
                view: document.defaultView,
                bubbles: true,
                cancelable: false
            });
            $a.dispatchEvent(evt);
        }
    }

I'm getting the error :
echarts-en.min.js:45 Uncaught DOMException: Failed to execute 'toDataURL' on 'HTMLCanvasElement': Tainted canvases may not be exported.
From this bit:
  const url = api.getDataURL({
            type: type,
            backgroundColor: model.get('backgroundColor', true)
                || ecModel.get('backgroundColor') || '#fff',
            connectedBackgroundColor: model.get('connectedBackgroundColor'),
            excludeComponents: model.get('excludeComponents'),
            pixelRatio: model.get('pixelRatio')
        });

In case anyone comes across this issue and get a Tainted canvases may not be exported. error message from the browser's console: the error may also come from the image used in the custom feature's icon field!


feat: auto build and release

This PR supported auto build and release

:baby_chick: Download canary assets:
nexus--canary.6.4438897565.zip

:rocket: PR was released in v0.0.2 :rocket:


fix(getters/shrex): removing implicit context assumption

Overview
Prevents infinite processing during availability calls

Codecov Report

Merging #1740 (2a23181) into shrex-blocksync (b211aa8) will decrease coverage by 0.11%.
The diff coverage is 0.00%.

@@                 Coverage Diff                 @@
##           shrex-blocksync    #1740      +/-   ##
===================================================
- Coverage            56.78%   56.68%   -0.11%
===================================================
  Files                  236      237       +1
  Lines                15500    15508       +8
===================================================
- Hits                  8802     8790      -12
- Misses                5776     5797      +21
+ Partials               922      921       -1

Impacted Files
Coverage Δ

share/getters/shrex.go
0.00% <0.00%> (ø)

core/listener.go
47.88% <0.00%> (-4.23%)
:arrow_down:

share/eds/byzantine/bad_encoding.go
66.00% <0.00%> (-3.00%)
:arrow_down:

share/eds/byzantine/pb/share.pb.go
32.88% <0.00%> (-1.97%)
:arrow_down:

core/config.go
100.00% <0.00%> (ø)

libs/header/sync/sync_head.go
66.15% <0.00%> (+0.50%)
:arrow_up:

libs/header/sync/sync.go
68.15% <0.00%> (+0.66%)
:arrow_up:

share/ipld/get.go
94.62% <0.00%> (+2.47%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


On 2023.6 _MSC_VER recognized incorrectly

I'm trying to compile a project on Power Tools 2023.6 and I see that _MSC_VER recognized incorrectly. It is recognized as 1910 when in fact it's 1936, which causing to take incorrect header files from libraries.

Hi @SlavaC1,
Thank you for reporting this issue
To continue investigate this problem, please provide us full log from output

First activate Verbose mode

Kind regards,
Marina

Attaching the log file.
log.txt
Also the header with decisions looks like this:
#if _MSC_VER < 1900
	#include "xlocnum_hack_vs2010-2013.h"
#elif _MSC_VER == 1900
	#include "xlocnum_hack_vs2015.h"
#elif _MSC_VER >= 1910 && _MSC_VER < 1914
	#include "xlocnum_hack_vs2017.h"
#elif _MSC_VER == 1914 // VS 2017 version 15.7
	#include "xlocnum_hack_vs2017_15_7.h"
#elif _MSC_VER >= 1915 && _MSC_VER < 1920 // VS 2017 version 15.8 and higher
	#include "xlocnum_hack_vs2017_15_8.h"
#elif _MSC_VER >= 1920 && _MSC_VER < 1926 // VS 2019 version 16.0 to 16.6
	#include "xlocnum_hack_vs2019.h"
#elif _MSC_VER >= 1927 && _MSC_VER < 1929 // VS 2019 version 16.7 to 16.9
	#include "xlocnum_hack_vs2019_16_7.h"
#elif _MSC_VER >= 1929 && _MSC_VER < 1931 // VS 2019 version 16.10 to 16.11, 1930 is VS 2022
	#include "xlocnum_hack_vs2019_16_10.h"
#elif _MSC_VER >= 1931 && _MSC_VER < 1938 // VS 2022 version 17.1
	#include "xlocnum_hack_vs2022_17_1.h"

Do you have the same problem on Clang Power Tools v2023.5.0 (don't forget to disable auto update after reinstalling this version)?
What version of Visual Studio you using?

Same issue with 2023.5.0

Thank you for quick reply

@SlavaC1, you can bypass this problem by changing -fms-compatibility-version value (for example in 19.36)

Thanks a lot! Now it's working. Not very intuitive though...


Don't scroll on initial load

Hi, thanks for the great addon!
With FastBoot, the user is able to scroll down the static HTML version of the app.
When the app rehydrates, ember-router-scroll resets scrolling, producing jerky behavior.

I've found the isFirstLoad property. It is read here, but it does not set up to prevent scrolling:
https://github.com/DockYard/ember-router-scroll/blob/d6cc035e45a3e85c0ea97ffe98d4276f3e558437/addon/index.js#L146-L148

This ugly hack resolves the issue for me:
    this.routerScroll.set('preserveScrollPosition', true);

    later(this, function () {
        this.routerScroll.set('preserveScrollPosition', false);
    }, 1000);

...in application route's beforeModel hook.

@lolmaus 👍  Do you see an improvement that could be added here?  Thx for the issue btw!


loading state is not reset if callback receives empty array

In this example "Loading" text is never hidden from drop down. A workaround is to pass [0]

Hi @SlyNet,
This has been resolved as of version 2.7.2.
Thanks


The program stops after agent posted

Hi,
Thanks for your excellent work!
I would like to know what would be an interesting scenario when the number of agents is 1000. I just change the agent_num to 1000 in config.yaml.
The program executes normally when the agent chats with other agents or enters the recommender system. But when the agent posts on the social network, the terminal gets stuck, like this:
NFO:12700:Linda Davis is going to social media.
INFO:12700:Linda Davis is posting.
INFO:12700:Linda Davis posted: "Hey everyone! Looking for some great movie recommendations, haven't watched anything recently but excited to hear your suggestions!"

When I CTRL+C, the error is :
Traceback (most recent call last):
  File "D:\YuLan-Rec-main\YuLan-Rec-main\simulator.py", line 1220, in <module>
    main()
  File "D:\code\YuLan-Rec-main\YuLan-Rec-main\simulator.py", line 1211, in main
    message = recagent.round()
  File "D:\code\YuLan-Rec-main\YuLan-Rec-main\simulator.py", line 828, in round
    msgs = self.one_step(i)
  File "D:\code\YuLan-Rec-main\YuLan-Rec-main\simulator.py", line 752, in one_step
    self.agents[i].memory.add_memory(
  File "D:\code\YuLan-Rec-main\YuLan-Rec-main\agents\recagent_memory.py", line 867, in add_memory
    self.save_context(
  File "D:\code\YuLan-Rec-main\YuLan-Rec-main\agents\recagent_memory.py", line 903, in save_context
    stm_memory_list = self.sensoryMemory.add_ssm(obs)
  File "D:\code\YuLan-Rec-main\YuLan-Rec-main\agents\recagent_memory.py", line 177, in add_ssm
    return self.dump_shortTerm_list()
  File "D:\code\YuLan-Rec-main\YuLan-Rec-main\agents\recagent_memory.py", line 149, in dump_shortTerm_list
    result = LLMChain(llm=self.llm, prompt=prompt).run({})
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\langchain\chains\base.py", line 501, in run
    return self(args[0], callbacks=callbacks, tags=tags, metadata=metadata)[
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\langchain\chains\base.py", line 306, in __call__
    raise e
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\langchain\chains\base.py", line 300, in __call__
    self._call(inputs, run_manager=run_manager)
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\langchain\chains\llm.py", line 93, in _call
    response = self.generate([inputs], run_manager=run_manager)
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\langchain\chains\llm.py", line 103, in generate
    return self.llm.generate_prompt(
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\langchain\chat_models\base.py", line 469, in generate_prompt
    return self.generate(prompt_messages, stop=stop, callbacks=callbacks, **kwargs)
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\langchain\chat_models\base.py", line 359, in generate
    raise e
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\langchain\chat_models\base.py", line 349, in generate
    self._generate_with_cache(
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\langchain\chat_models\base.py", line 501, in _generate_with_cache
    return self._generate(
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\langchain\chat_models\openai.py", line 360, in _generate
    response = self.completion_with_retry(
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\langchain\chat_models\openai.py", line 299, in completion_with_retry
    return _completion_with_retry(**kwargs)
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\tenacity\__init__.py", line 289, in wrapped_f
    return self(f, *args, **kw)
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\tenacity\__init__.py", line 379, in __call__
    do = self.iter(retry_state=retry_state)
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\tenacity\__init__.py", line 314, in iter
    return fut.result()
  File "D:\applications\anaconda3\envs\recagents\lib\concurrent\futures\_base.py", line 439, in result
    return self.__get_result()
  File "D:\applications\anaconda3\envs\recagents\lib\concurrent\futures\_base.py", line 391, in __get_result
    raise self._exception
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\tenacity\__init__.py", line 382, in __call__
    result = fn(*args, **kwargs)
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\langchain\chat_models\openai.py", line 297, in _completion_with_retry
    return self.client.create(**kwargs)
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\openai\api_resources\chat_completion.py", line 25, in create
    return super().create(*args, **kwargs)
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\openai\api_resources\abstract\engine_api_resource.py", line 153, in create
    response, _, api_key = requestor.request(
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\openai\api_requestor.py", line 288, in request
    result = self.request_raw(
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\openai\api_requestor.py", line 596, in request_raw
    result = _thread_context.session.request(
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\requests\sessions.py", line 589, in request
    resp = self.send(prep, **send_kwargs)
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\requests\sessions.py", line 703, in send
    r = adapter.send(request, **kwargs)
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\requests\adapters.py", line 486, in send
    resp = conn.urlopen(
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\urllib3\connectionpool.py", line 714, in urlopen
    httplib_response = self._make_request(
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\urllib3\connectionpool.py", line 466, in _make_request
    six.raise_from(e, None)
  File "<string>", line 3, in raise_from
  File "D:\applications\anaconda3\envs\recagents\lib\site-packages\urllib3\connectionpool.py", line 461, in _make_request
    httplib_response = conn.getresponse()
  File "D:\applications\anaconda3\envs\recagents\lib\http\client.py", line 1377, in getresponse
    response.begin()
  File "D:\applications\anaconda3\envs\recagents\lib\http\client.py", line 320, in begin
    version, status, reason = self._read_status()
  File "D:\applications\anaconda3\envs\recagents\lib\http\client.py", line 281, in _read_status
    line = str(self.fp.readline(_MAXLINE + 1), "iso-8859-1")
  File "D:\applications\anaconda3\envs\recagents\lib\socket.py", line 704, in readinto
    return self._sock.recv_into(b)
  File "D:\applications\anaconda3\envs\recagents\lib\ssl.py", line 1275, in recv_into
    return self.read(nbytes, buffer)
  File "D:\applications\anaconda3\envs\recagents\lib\ssl.py", line 1133, in read
    return self._sslobj.read(len, buffer)

The code I'm using is up to date, and I've tried changing the active_agent_threshold to 1000 in config.yaml, as well as falling back to the version before this change #3 , but neither works.

I tried with an agent count of 500 and the program is fine. It took seven minutes to get the post to 10 agents.
Is this because when the number of agents reaches 1,000, "Linda Davis" has a lot of friends, so they will all see the post and save it to their memories, resulting in a long time?

@LIUQI-creat Hi, I believe you are correct. From the screenshot above, it appears that it stopped during the self.agents[i].memory.add_memory process, which may be time-consuming due to adding memory.


retry policy: check retry conditions before remaining retry count

This commit reorders some of the checks in the HTTP retry state
implementation. Previously, it would check if there are remaining
retries before checking if a retry would be attempted based on the retry
policy and event that's occuring. This means that shouldRetry might
return NoRetryLimitExceeded for cases that should not be retried anyway.
This bubbles up to the router and causes a response flag that might be
confusing to end users. In particular this happens if the final retry
attempt succeeds.
Signed-off-by: Michael Puncel<EMAIL_ADDRESS>For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
[Optional Fixes #Issue]
[Optional Deprecated:]

I caught this case while working on #6228. In the hedging case we examine the output of shouldRetry() to know if we should cancel all other outstanding upstream requests and return a result downstream. However even if we get "good" response headers back, the router was seeing RetryStatus::NoRetryLimitExceeded instead of RetryStatus::No which resulted in not sending the response downstream.
I believe this is a good change to make even for master, today there might be a case where you see say a 200 response code but a "retry limit exceeded" response flag.


Update expected-lite to 0.6.1

(Yes, this only changes the version number; see https://github.com/martinmoene/expected-lite/releases/tag/v0.6.1)

Coverage remained the same at 59.323% when pulling 1f3d33d7045f313b2b60c57a52dd7682857c0651 on feature/bump-expected-lite into ca8ca498153f2868a57e71eb1b7843d1feafbdbd on master.


feat: Initial text + icon table

Closes #130

Trying to find a way to accomplish this consistent spacing via CSS, so that devs would not need to encapsulate text+icon in any special wrappers...but it seems that CSS does not have the ability to recognize plain text.  So targeting scenarios where the icon comes before/after text, and applying the appropriate spacing, is proving more involved than I'd hoped.
After listing out the investigation below, I'm tempted to say that continuing with the current approach of "Devs just need to insert a   between an icon and adjacent text" may actually be the most sustainable approach.
Alternative approaches:

Add props to the <Icon /> component that accept text and handle the spacing

Ex: <Icon name='info' textRight="This text would display to the right of the icon with a margin between" />
Ex: <Icon name='info' textLeft="This text would display to the left of the icon with a margin between" />
Pros

Makes it easy to create both the icon and it's accompanying text in a single component

Cons

May not be the most intuitive approach so we would want to call this out in our documentation

Create an helper/wrapper component for text that appears next to an icon, which we could then target via CSS

Ex. `<>This text appears next to an icon<>
Pros

Takes care of the need for devs to be responsible for inserting the space between icon/text

Cons

Devs still have to know to wrap icon-adjacent text in the special wrapper in order to get the proper spacing

Integrate an icon prop into components that commonly display icons

Ex. Link w/ icon: `Download font

This would automatically insert the named icon after the link text with appropriate styling and spacing.

Examples from CF.gov of text + icon

https://www.consumerfinance.gov/owning-a-home/

Lots of download links
One H3 with icon before text

https://www.consumerfinance.gov/consumer-tools/educator-tools/

More headers w/ icons

https://www.consumerfinance.gov/activity-log/


Improve logging

Closes #17

Added log4j implementation of slf4j.
Added a log4j2.properties file for logging configuration.
Added a new env DEBUG_LEVEL that can be set in the YAML file.

Signed-off-by: Shishir Halaharvi<EMAIL_ADDRESS>

@dinogun Should be held off until minikube PR is merged and additional changes are made.

@chandrams Do you have any suggestions on the log messages you need for testing?

@dinogun @chandrams Should the logs be saved in a file, or should they be directed to STDOUT?
They would be persistent in a file, but anyone who wants to get that log file would need to run kubectl cp, while the logs can be easily viewed by using kubectl logs -f for STDOUT.
Should this be an option for the user?

The k8s best practice is stdout, no files please.

For reference:

When DEBUG_LEVEL is set to INFO, or no env is set(by default) INFO_LOG.txt

When DEBUG_LEVEL is set to DEBUG: DEBUG_LOG.txt

Info provided in info log is more, we shouldn't be printing recommendations for idle apps. We can have a logic to monitor only active applications. Have a thread monitoring the idle ones and move them to active when they become active.

Do we need to log the Auth token and Monitoring agent end point? Would there be any security issue? We can anyways print this when Debug is on.

Memory is in KB or MB? Can we print this too.

Can we print the original requests and limits when INFO is on?

Can the sampling interval be configurable? I think this can be done if kruize monitors apps in parallel.

In the info log, it prints "No previous recommendations available" - include the app name here.

If an app is scaled, does kruize give recommendations for all scaled applications. We need to test this scenario.

If a pod is removed, does kruize stop monitoring and logging this.

What error messages are logged when an error occurs? Does this msg "com.kruize.main.Kruize - End of initialization phase" indicate kruize has been successfully initialized.

Should we log recommendations each time kruize monitors and generates recommendations? Can we have it log every 5th or 'X'th time it generated the recommendations. This would reduce the size of the log. Atleast in INFO level we don't need this, might be DEBUG can give these details if required.

1. Info provided in info log is more, we shouldn't be printing recommendations for idle apps. We can have a logic to monitor only active applications. Have a thread monitoring the idle ones and move them to active when they become active.

Created an issue #58. But this will take some time.

2. Do we need to log the Auth token and Monitoring agent end point? Would there be any security issue? We can anyways print this when Debug is on.

True, security would be a major concern. I think that Auth token shouldn't be printed even in debug mode, but a little conflicted about the endpoint URL. Makes debugging easy, but still poses a security risk. I would welcome your thoughts on how to approach this @chandrams @dinogun

3. Memory is in KB or MB? Can we print this too.

Will make the change.

4. Can we print the original requests and limits when INFO is on?

Will make the change .

5. Can the sampling interval be configurable? I think this can be done if kruize monitors apps in parallel.

Should be done in #58, but again, looks like it'll take quite a while until that's done.

6. In the info log, it prints "No previous recommendations available" - include the app name here.

Will make the change.

7. If an app is scaled, does kruize give recommendations for all scaled applications. We need to test this scenario.

Will test and print the logs.

8. If a pod is removed, does kruize stop monitoring and logging this.

Will test and print the logs.

9. What error messages are logged when an error occurs? Does this msg "com.kruize.main.Kruize - End of initialization phase" indicate kruize has been successfully initialized.

Will give the logs when initialization fails.

Removed printing auth token.
Printing memory in MB now.
Bug with original values results in them being obtained multiple times as of now. This should be resolved once #44 gets solved, following which original values will also be made available in INFO.

Errors when initialization fails:
initializationError3.txt
initializationError2.txt
initializationError.txt

Logs when acmeair pod was deleted after sometime:
podDeleteInfo.txt
deletePodDebug.txt
No errors were printed or additional messages were printed in this case and the same earlier recommendations were shown. Do you want anything specific in the logs for this case showing that the app was killed?

Logs look fine. Should we reduce the frequency for logging recommendations for a deleted app?

@chandrams Yes, deleted apps shouldn't log as frequently. But right now, there is no convenient way of checking if the application was killed. Will need to add that in a future PR.

@dinogun - I have this PR, few changes requested will need further work. For now, I think we can go ahead and merge the changes.

Can we make this small in the logs
Timestamp  LOG_TYPE FUNCTION_NAME pod_name: ACTIVITY_INFO

This would translate as follows
07:16:43.653 [Thread-4] RECO  com.kruize.analysis.AnalysisImpl - acmeair-sample-57c4bdc97-nq7pn: CPU Limit = 1.2
07:16:43.653 [Thread-4] RECO  com.kruize.analysis.AnalysisImpl - acmeair-sample-57c4bdc97-nq7pn: CPU Requests = 1.1
07:16:43.654 [Thread-4] RECO  com.kruize.analysis.AnalysisImpl - acmeair-sample-57c4bdc97-nq7pn: Memory Limit = 222.12 MB
07:16:43.654 [Thread-4] RECO  com.kruize.analysis.AnalysisImpl - acmeair-sample-57c4bdc97-nq7pn: Memory Requests =<PHONE_NUMBER>5734863 MB

Makes it a lot more easier to read. Also note RECO tag instead of INFO for recommendations. This can be a different log level, so that should make it easy to turn on just the recommendations for debugging if needed.

@dinogun Changed the log messages as in the above comment. Currently though, the custom RECO tag is not possible because slf4j doesn't support it. Right now, as we are using Eclipse Jetty, which depends on slf4j, it is a little difficult to migrate to other logging libraries like logback or log4j.
In the future, as Kruize will need an API endpoint, we should use OpenLiberty or other better webservers rather than Jetty. That would enable us to completely switch to log4j, and use custom tags.

@dinogun Updated README.

Also changed the env from DEBUG_LEVEL to LOGGING_LEVEL. Is this OK or should I roll it back?


Test helper rotation: update TLS certs  on all hosts

When a test helper is deployed by the rotation script a fresh TLS cert is generated using the Digital Ocean API. The cert is deployed on the new host.
The rotation script can be tweaked to [attempt to] update the cert on all existing test helper hosts instead.

We should move the test helper rotation from the custom script into the new devops pattern. See: https://docs.ooni.org/backend/ooniapi/services/


better handling of enum struct variants and a bit more testing

@nimily @JoeHowarth after we merged the previous PR for enums with struct variants I realized it was actually blowing up with our client code - I am not sure how I ever convinced myself it was fine. This removed the hardcoded optionality of the struct variant and tests both cases.

This looks fine to me, but @nimily knows the codebase better than I do
@kchaliki can you fix the merge conflicts please?


refactor(prettier): clean up object::print_object_properties

Current dependencies on/for this PR:

main

PR #1573   👈

This stack of pull requests is managed by Graphite.

Merge activity

Nov 28, 11:30 PM: @Boshen merged this pull request with Graphite.


File and directory names

Hello! 👋
Could generify be used to replace the names of directories  and files in a template as well? Something along the lines of:
given template:
template
|-@foo@
  |-bar.js // console.log('__bar__')
  |-<EMAIL_ADDRESS>// console.log('__baz__')

and data:
const data: {
  foo: 'hello',
  bar: 'world',
  baz: 'renamed'
}

output:
foo
|-bar.js // console.log('world')
|-renamed.js // console.log('renamed')

Thanks 😄

Happy to contribute a PR if this is a feature you would like to support!


Typo in Readme

You have a typo in How it works...

Thank you @danzen for pointing this out! ☺ I just fixed it!


Add a call helper to call JS functions in a more performance way

Goal
Allow to call JS functions directly.
Performance
This improve performance of calling functions directly, rather than passing them through eval:
Warming up --------------------------------------
                eval     9.062k i/100ms
                call    12.708k i/100ms
Calculating -------------------------------------
                eval     99.386k (± 7.2%) i/s -    498.410k in   5.042353s
                call    138.059k (± 4.8%) i/s -    698.940k in   5.074396s

Limitations
No support for:

memory limitation
complex error processing

Nice, this looks really good to me!  I am merging it in!
I want to do a release but would really love https://github.com/discourse/mini_racer/issues/75 fixed first, any chance you can have a look?

Hey @SamSaffron not at the moment I'm afraid :(
Our next steps would probably target this which has a major performance impact when eval or call return objects.

sure, anything to make mini racer faster is totally welcome!