Update LMP Pricing Data Table Format

Description
Present the LMP Pricing Data Availability Table in the same green checkmark and red X fashion as the methods availability table.
Before

After

Unrelated test failure with self = <json.decoder.JSONDecoder object at 0x7f95a30a63d0> [493](https://github.com/kmax12/gridstatus/actions/runs/3726071149/jobs/6319276936#step:5:494) s = 'ï»¿<!DOCTYPE html>\r\n\r\n\r\n    <title>An internal server error has occurred. Please check the URL and ...coration:none;font-size:14px;">Go Back to the Homepage\r\n
\r\n
\r\n\t\r\n\r\n' [494](https://github.com/kmax12/gridstatus/actions/runs/3726071149/jobs/6319276936#step:5:495)

wanted to get #97 merged in before returning to this. This is good but we just need to make sure Ercot still works after merging in main and makes sense to just mention somewhere on this page (maybe below table?) that Ercot uses get_spp instead of get_lmp

This is good but we just need to make sure Ercot still works after merging in main. Also i think it makes sense to just mention somewhere on this page (maybe below table?) that Ercot uses get_spp instead of get_lmp

Do you foresee the other ISOs getting the get_spp method, thus making this a separate table (or more columns)?

no, I think just ercot has spp. everyone else uses lmp

OK. I'll review the detection code and add a note right after the table.

The LMP doc could use a fresh coat of paint I think.

We are currently adding Locational Marginal Price (LMP). Even though each BA offers different markets, but you can query them with a standardized API

What does BA stand for here?

Ready for review

What does BA stand for here?

Stands for balancing authority. Basically, the organization responsible for balancing supply/demand on the grid. I'm fairly confident it is save to say that all the ISOs and RTOs are BAs. However, there are definitely more BAs that aren't ISOs or RTOs.
Not sure it's usage is necessary/helpful here

Stands for balancing authority.

Ah, OK - thanks.


Talent as a switch condition?

Would it be possible to get talent as a switch condition?  This would make it great on wotlk beta!
Making different rotations for retribution and protection that switch when I switch talents would be great!

With Wrath, some of the code that is currently available in Retail will be unlocked again (though it will have to be modified.
Specifically, in Retail, you have multiple talents - so you actually get a tab for each talent, and a separate set of rotations for each spec.
You won't need to have switch conditions for talents.

Should be resolved as of 1.4.0 beta1


Stream memory leak

Streams are not clearing some object created every time a then() is called on them, leaving 2 Promises 1 Deferred and 1 Stream in memory, even after calling detachStream or unlink.
I traced the issue to be the then(f) of the _end_promise of the Stream, which is never removed from its  _update array.
When forcefully clearing the _update array those objects are garbage-collected as expected but I guess that could lead to other bugs.
This is the gist of my test https://gist.github.com/alebianco/5a4ab56a6934bf3e2ed2
I'm compiling for JS and did my profiling in Chrome.
I will try and propose a pull-request to fix this but if you have any suggestion on how this is supposed to work, it would be appreciated.

thanks for the heads up on this.

On this topic, in general how are streams and promises cleaned up, memory wise. E.g. if you add a then(...) to a promise, and the parent promise object cluster gets disposed of, do you have to take care of unlinking your then(..) addition?
For example, with events there's a 'once' method that cleans up the listener after the event fires once. Is this necessary with promises?

Promises maintain their connection upstream by design, since they're geared to catch multiple resolves (and throw an error in that case).
It looks like I just need to do some housecleaning for the _end array when the stream is detached.

I do have a doubt about the fact that promises are upstream-linked. How do you handle throwaway promises?
something like:
function connect():Promise<Bool> {
  var d = new DeferredPromise<Bool>();
  // do connection stuff and resolve
  return d;
}

// in another class
connect().then(doStuff);

the use of "doStuff" will create a linkage that will block the garbage collector to free some objects but i can't figure out a sensible way of unlinking the promise generated by then() so I've no way (that I can think of) of taking care of it ...
Am I missing something obvious?

The promise object is permanently bound to the callback by design.  Stream has a method of unbinding, and is better for situations where you might want to listen to a global emitter that will cause the async to never be gc'ed (like setTimeout).
Could you give some more examples of where you run into problems?  I'm making a mostly philosophical argument here, if there's something that's causing practical problems, I'll find a way to get rid of them.

I gave it a shot at fixing the leak in the Stream class. So far seems ok but my test was in no way comprehensive. Care to pitch in and review it?

Sure, I"ll take a look


API endpoint broken for reverse proxy

Description
Unable to use behind a reverse proxy because angular constant of endpoints start with / and there is no  tag.
Steps to reproduce the issue:

Serve portainer behind a subfolder with a reverse proxy /portainer/
Try any endpoint api they 404 because the want to connect to /api instead of /portainer/api

Technical details:

Portainer version: 1.11.0
Portainer Docker image tag (latest/arm/windows...): latest
Target Docker version (the host/cluster you manage): host
Platform (windows/linux): linux

Could you please provide release date?

I configured HAProxy in my LB machine to switch traffic with /portainer to portainer container.
What steps do I need to do in portainer machine to host it with /portainer?
I pulled the latest docker image but still I css, JS files do not have /portainer in root url.
I get a BIN file if I access with '/portainer'/

@sungiri88 1.11.1 is now live on Dockerhub.

@deviantony
Installed the latest version now.
When i hit portainer with /portainer, I get a BIN file to download.
What settings should I do ?

@sungiri88 your reverse proxy configuration might be incorrect. Have a look at https://portainer.readthedocs.io/en/stable/faq.html#how-can-i-configure-my-reverse-proxy-to-serve-portainer for some inspiration.

@deviantony
See the below config, where HA Proxy rule as path_beg /portainer. In the FAQ link, you have used domain name without any relative path.
frontend www-http
bind    *:80
stats   enable
mode    http
option  http-keep-alive
acl portainer   path_beg /portainer

use_backend     portainer       if portainer

backend portainer
stats   enable
option  forwardfor
option  http-keep-alive
server  portainer    <IP_ADDRESS>:9000 check

@sungiri88 example in the doc was provided by one of our users. I won't be able to help you more than that, you'll have to try it yourself. Sorry, keep us posted though.

using that kind of conf for nginx :
upstream portainer {
    server ADDRESS:PORT;
}

server {
  listen 80;

  location /portainer/ {
      proxy_http_version 1.1;
      proxy_set_header Connection "";
      proxy_pass http://portainer/;
  }
  location /portainer/api/websocket/ {
      proxy_set_header Upgrade $http_upgrade;
      proxy_set_header Connection "upgrade";
      proxy_http_version 1.1;
      proxy_pass http://portainer/api/websocket/;
  }
}

When I hit http://localhost/portainer it works like a charm.

@radamanth What version of portainer are you using? When I try with 1.12.1 and the exact nginx config above, I still receive a 404 for localhost/portainer/api/settings

@michaelmsm89 Can't remember, I was testing it on virtual box, And I dont have  it anymore.
I didn't try the API with that conf anyway, just the Portainer UI

@radamanth No worries, thanks for the reply. I am just trying the UI too, but it calls it's own API on startup which is when the problem happens.

@michaelmsm89 I had the same problem. This config works:
upstream portainer {
  server foo:9000;
}
server {
...
  location /api/ {
    proxy_http_version 1.1;
    proxy_set_header   Connection "";
    proxy_pass         http://portainer;
  }

  location / {
    proxy_http_version 1.1;
    proxy_set_header   Connection "";
    proxy_pass         http://portainer/portainer/;
  }

  location /api/websocket/ {
    proxy_set_header Upgrade $http_upgrade;
    proxy_set_header Connection "upgrade";
    proxy_http_version 1.1;
    proxy_pass http://portainer/api/websocket/;
  }
}

Note it happens to move the default path from /portainer to /

I had a similar problem using a reverse proxy (nginx) for ssl offloads.  My example allows me to use portainer on standard ports (https://portainer.example.com) in my internal network. Here is my config that hands off to portainer (<IP_ADDRESS>) on port 9000.
Without the second block /api/websocket/ the some of the xhr requests errored (500 Internal Server Error). [I have not tried to assess why, eg the http upgrade]
server {
    listen   80;
    server_name portainer.example.com;
    location / {
      return 301 https://$server_name$request_uri;
    }
}

server {
    listen   443 ssl;

    server_name portainer.example.com;
    keepalive_timeout    70;

    ssl_certificate      /etc/nginx/ssl/portainer.example.com/cert.pem;
    ssl_certificate_key  /etc/nginx/ssl/portainer.example.com/privkey.pem;

    access_log  /var/log/nginx/portainer.example.com.access.log  main;

    location / {
        proxy_pass http://<IP_ADDRESS>:9000;
        proxy_http_version 1.1;
        proxy_set_header Connection '';
        proxy_set_header Host $host;
    }

    location /api/websocket/ {
        proxy_set_header Upgrade $http_upgrade;
        proxy_set_header Connection "upgrade";
        proxy_http_version 1.1;
        proxy_pass http://<IP_ADDRESS>:9000/api/websocket/;
    }
}

using that kind of conf for nginx :
**upstream portainer {
    server ADDRESS:PORT;
}**

server {
  listen 80;

  location /portainer/ {
      proxy_http_version 1.1;
      proxy_set_header Connection "";
      proxy_pass http://portainer/;
  }
  location /portainer/api/websocket/ {
      proxy_set_header Upgrade $http_upgrade;
      proxy_set_header Connection "upgrade";
      proxy_http_version 1.1;
      proxy_pass http://portainer/api/websocket/;
  }
}

When I hit http://localhost/portainer it works like a charm.

Thank A lot !!! the Upstream is the one way find for solve redirection to subpath of domaine :)

using that kind of conf for nginx :
upstream portainer {
    server ADDRESS:PORT;
}

server {
  listen 80;

  location /portainer/ {
      proxy_http_version 1.1;
      proxy_set_header Connection "";
      proxy_pass http://portainer/;
  }
  location /portainer/api/websocket/ {
      proxy_set_header Upgrade $http_upgrade;
      proxy_set_header Connection "upgrade";
      proxy_http_version 1.1;
      proxy_pass http://portainer/api/websocket/;
  }
}

When I hit http://localhost/portainer it works like a charm.

Thank A lot !!! the Upstream is the one way find for solve redirection to subpath of domaine over reverse nginx :)

WOrk Fine!


Initial attempt at weight Breakdown (issue #507)

@whoburg initial breakdown class ready for review. Should be fully functional. Let me know if you want to tweak how the diagram looks, I could possibly make it colored, change fonts/sizes, dash some lines, etc.
current problems:
improper indent warning from pylint on continued lines

Can one of the admins verify this patch?

test this please

@mayork, I'm going to rebase and force push to eliminate all the commits related to a merge gone wrong. You'll need to update your local branch -- I'll post again with instructions on how to do that

ok, rebased and force pushed (and I have a backup of the branch in case there is any issue)
anyone (including @mayork ) that has a local checkout of this branch needs to do the following:
(stash any uncommitted changes)
git checkout master
git branch -D breakdown
git checkout breakdown
git pull origin breakdown

@mayork, it looks like your text editor is still set up to insert tabs instead of 4 spaces -- google how to set the tabstop and expandtab so that tab means 4 spaces for python files.

@mayork I did an initial code review with inline comments. You can add additional commits to address them. I think there are some ways to further simplify this - but you can start with the comments so far.

disregard the large chunk of deleted lines in the above commits...I was getting a merge error when I tried to commit, I think this was an artifact of moving the branch around. It's fixed now.

@whoburg addressed all of the corrections and am up and running with pylint

@whoburg talked with @bqpd about the graphics...he thought SVG was the way to go. If you concur I'll start on that.

Can one of the admins verify this patch?

@whoburg

this is my current diagram...thoughts?

Can one of the admins verify this patch?

test this please

retest this please

verify this please (is that how I do this)

Looks like there's merge conflicts?

@bqpd not sure how to fix a merge conflict...also, what I just committed doesn't pass pylint because of improper indent on a continued line. What is the protocol for that? I couldn't figure it our from the config file

retest this please

@mayork can you update the very first comment to give an overview of the current status? What's there now is out of date.

ok failing unit tests because svgwrite is not on the Jenkins machines. @bqpd any strategy thoughts here? Should part of this be moved to interactive?

err, is breakdown.py the only changed code?

I think this should inherit from ConstraintSet and have a method which imports the svg drawing logic from interactive, like Model and ControlPanel.

retest this please

@bqpd @whoburg I still am not sure what exactly needs to be moved to svg.py inside of interactive....do I need to move every method that touches any part of the drawing object (i.e. dwgrecurese and drawsegment)? I have tried a few things and can't something which works
when I try and do this:
from .interactive.svg import importsvgwrite
from svgwrite import cm
i get this error
"ValueError: Attempted relative import in non-package"
I just pushed with the svg.py file I made inside interactive and an updated breakdown.py...the svg.py has some commented out areas which reflect a previous attempt to get things working

Try from gpkit.interactive.svg import ...
I don't fully understand why the relative import isn't working here - @bqpd might know.
In any case, try to follow the pattern currently used by gpkit/constraints/costed's controlpanel method, which imports modelcontrolpanel from interactive.

I think the problem is that this isn't being imported as part of the module, and so the relative import won't work.

@bqpd could we meet to look at this? I tried a few other things...if I try to do from gpkit.interactive.svg import _____ i get the error:
importerror: no module named svg

I think for now you should just import svgwrite in breakdown.py; when the breakdown class is feature-complete, I can sit down with you and we can figure out how to integrate it into gpkit.

@bqpd I think that's the point we're at, woody was hoping to merge it into master and close this branch out this week/next week

Okay! For this to be in GPkit, Breakdown should probably be a ConstraintSet which generates its constraints on init, and its name should end with ConstraintSet. Then should be a function in interactive which accepts a Breakdown and does all the plotting; much of what are attributes of self now will be instead gotten from the passed Breakdown.

I'll take a stab at starting this.

Ready for review by @mayork

@bqpd @whoburg I think this is ready for final review...I moved all the unit tests to t_constraints, all unit tests pass on my machine. Also fixed some diagram drawing problems that came up after moving things to interactive
not sure where the branch conflicts are...i can't find them

@mayork the conflicts are probably from some PRs recently merged to master, I can handle that. Breakdown still doesn't match the gpkit code style in some significant ways; let's meet up in person and go over the code before final review by @whoburg.

@bqpd sounds good, when would you like to meet?

retest this please

Ready to merge once TODOs are cleaned up and bd.py is removed. There should potentially be a test that the correct SVG is produced?

@whoburg @bqpd took care of all of the TODOs, just need to remove bd.py
Also, what kind of test did you want to verify the correct SVG is produced? Should I just save a sample file into gpkit somewhere?

Awesome! Reviewing...

@bqpd just pushed a new version with your changes

@whoburg I looked over this again and think it's ready for review...do you want any sort of test method? I'm not entirely sure what that would/should look like

just rebased this with @mayork, force pushing

retest this please

this is the diagram the test method draws...seems potentially useful to have somewehre

Nice, this looks good.

@whoburg this is ready for a final review. I think we should disable the remaining pylint warnings, but I didn't want to do that before it was reviewed

retest this please

@mayork I'd like to do  this review in two stages:

high-level comments/discussion (now)
a more detailed line-by-line review by me or @bqpd

For now, the high-level comments/discussion:

I want us to try using this on the 1682 model and see how it goes. Could you please work with @mjburton11 to try this out on a gpkit-models branch. My suspicion is that it won't make sense to specify a single weight breakdown input dict in one location -- rather, it will be more natural to have multiple Breakdown objects (wing, fuselage, etc) that are local to their submodels/components -- but not sure how this will really play out.

Related to the above, does it work for one of the leaves of Breakdown A to itself be a Breakdown?

Further related to the above, should we implement it that way, i.e. each Breakdown is only one level deep, its leaves always being either a Variable or a Breakdown?

Can we switch the font to math font?

I'll get with Mike and set up a 1682 breakdown.

Have not tested that...however, if it doesn't work should be an easy feature to add simply by appending the constraints from the other breakdown

What do you see as the benefit of that method? I'm not sure I fully understand what you're going for

Changing font is possible...I looked into it before, it was a while ago so I don't recall why I didn't do it. I'm not sure on math type, I'm guessing there's a way to do it but I need to investigate

The benefit of 3 would be a cleaner implementation in that the data structures and parsing would match up. To parse an input dict, for all values in a level of the dict you would a) create a variable, if the value is a string or Variable, or b) create a Breakdown, if the value is a dict.
Additional benefit is that it would be always possible to draw the breakdown diagram associated with a subset of a Breakdown, e.g. Aircraft has a Breakdown, but its Wing is also a breakdown.
Start by addressing 1 and I suspect 3 may end up being related.

@whoburg more detailed answer to 4 from above. new fonts and styles can be defined as a new CSS style...I'm not venturing outside my knowledge base but from what I know/can tell from some quick web research CSS has limited support for math, so I don't think we can go display math font. How big of a problem is that? I think it would be nice to be able to do, but I don't think it's that big of an issue

I would be surprised if the svg package does not support latex math. But I've been surprised before.
In other news, looks like we have some conflicts; I'll attempt a rebase.

@mjburton11 @whoburg 16.82 gas_male breakdowns. the code to make them is below.
    #create the centbreakdown
    BDcent = {'Center': {'Fuel': sol('W_{fuel-tot}'), 'Payload': sol('W_{pay}'),
                                'Engine': sol('W_{eng-tot}'), 'Avionics': sol('W_{avionics}'),
                                'Skid': sol('W_{skid}')}}
    centbd = Breakdown(BDcent, "lbf")
    mcent = Model(centbd.root, centbd)
    centsol = mcent.solve()
    centbd.make_diagram(centsol,  "16.82_cent_weight_breakdown.svg", 7, 20)

    #create the fuselage breakdown
    BDfuse = {'Fuselage': {'Ribs': [sol('m_{rib}')*9.8*units('m/s^2'), "N"],
                           'Skin': [sol('m_{fuse}')*9.8*units('m/s^2'), "N"]}}
    fusebd = Breakdown(BDfuse, "lbf")
    mfuse = Model(fusebd.root, fusebd)
    fusesol = mfuse.solve()
    fusebd.make_diagram(fusesol, "16.82_fuse_weight_breakdown.svg", 7, 12)

    #create the wing breakdown
    BDwing = {'Wing': {'Caps': [sol('m_{cap}')*9.8*units('m/s^2'), "N"],
                       'Skin':[sol('m_{skin}')*9.8*units('m/s^2'), "N"]}}
    wingbd = Breakdown(BDwing, "lbf")
    mwing = Model(wingbd.root, wingbd)
    wingsol = mwing.solve()
    wingbd.make_diagram(wingsol, "16.82_wing_weight_breakdown.svg", 7, 12)

    #create the total breakdown
    total = dict(BDwing.items() + BDfuse.items() + BDcent.items())
    BDtotal = {'Total': total}
    fullbd = Breakdown(BDtotal, "lbf")
    mfull = Model(fullbd.root, fullbd)
    bdsol = mfull.solve(solver="mosek")
    fullbd.make_diagram(bdsol, "16.82_weight_breakdown.svg", 10, 22)

@whoburg you cannot give a Breakdown another Breakdown as an arugment. However, I think the way it currently works is still close to your vision.
Breakdown is really just a plotting tool. The user supplies a Breakdown with a nested input dict which defines the breakdown. Breakdown provides a convenient method to sum weights in this input dict, but that does not need to be used if the values for all the weights are already known. If the user wants multiple breakdowns like I made above, you can just make multiple sub dicts and combine them before making a Breakdown. What are your thoughts on this? I don't think its too user intrusive.

I still feel like this is dying to be a ConstraintSet that can be included as a model building block. The drawing functionality is a natural use for process_result.
In the current form I'm worried that models will get out of sync from their corresponding Breakdown.
I think all the work done here on drawing will be super useful, and I don't think defining the recursive class will be too bad. I can take a shot at this at some point if it's confusing what I'm suggesting here.

What do you want the input to the building block constraint set to be?

Also, I can adjust line styles etc. is there a preference on what they should look like?

I would think there would be two main inputs:

Arguments
---------
root (varkey):
    top-level variable to break down
leaves (iterable):
    list of (variables or other breakdowns) that sum to form root

The resulting constraint pseudocode is something like
class Breakdown(Model):
    def __init__(self, **kwargs):
        constraints = [root >= sum(getattr(x, "root", x) for x in leaves)]
        constraints.extend([x for x in leaves if hasattr(x, "root")])  # add sub-breakdown constraints
        self.root = root
        Model.__init__(root, constraints, **kwargs)

... and then the only hard part is drawing. Unless I'm missing something hard?
On style, I think we should try to make them look like the latex examples posted above.

that should be a pretty easy refactor on making the breakdown constraint set the way you want...I think most of the current code can actually still be used.
As far as the drawings do we want to actually make them tin latex? I'm rather amateur with latex so I'm not sure how hard it would be to make the switch.  I know @bqpd suggested svg, but there might be advantages to latex (namely simpler printing of math symbols)

Yeah you probably do want to make them in latex, though might be worth
doing svg as well for faster interactive display. Latex is just too slow
for interactive use.
On Sep 13, 2016 6:25 PM, "mayork"<EMAIL_ADDRESS>wrote:
that should be a pretty easy refactor on making the breakdown constraint
set the way you want...I think most of the current code can actually still
be used.
As far as the drawings do we want to actually make them tin latex? I'm
rather amateur with latex so I'm not sure how hard it would be to make the
switch. I know @bqpd https://github.com/bqpd suggested svg, but there
might be advantages to latex (namely simpler printing of math symbols)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hoburg/gpkit/pull/522#issuecomment-246845317, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABagGL38VWn3FpxyBFSjJ6a9fB2rBlFNks5qpyK_gaJpZM4HiJyG
.

yeah tough call there. Possibly useful: http://tex.stackexchange.com/questions/51757/how-can-i-use-tikz-to-make-standalone-svg-graphics


just publish the src directory, no need to copy the output

We can just have npm include the src directory, instead of needing to copy it to the dist folder.
This also allows the contracts to be imported like:
import {IProtocolRewards} from<EMAIL_ADDRESS>

hype. thanks!


[Port to foxy] Add support for namespace

Signed-off-by: Homalozoa<EMAIL_ADDRESS>

Hi @paeunt
foxy-devel branch, currently, is only used to create the packages. As the master currently is for Foxy, I usually sync master with foxy-devel before creating the packages.
So, I would close this PR, and would work in #3 .
Thanks!!

Hi. But the newest version is galactic, perhaps master branch should be port to galactic, and leave foxy-devel for foxy version. What do you think so?

Hi @paeunt

Hi. But the newest version is galactic, perhaps the master branch should be port to galactic, and leave foxy-devel for foxy version. What do you think so?

Yes, you are right. As soon as master starts to point to Galactic (probably soon), foxy-devel will contain the development for Foxy (only bug fixes).
Meanwhile, only PRs to master would count.

I am closing this now that #3 has been merged


"All references complete" email, is being sent to referee

In gitlab by @A-Lawrence on Nov 25, 2016, 16:41
The wrong email is being sent to referees - it seems the "All your references have been submitted" is being sent to the referees.

In gitlab by @atoff on Nov 28, 2016, 19:07
Mentioned in merge request !34

In gitlab by @atoff on Nov 29, 2016, 00:39
Status changed to closed by commit 65cec157347d88c5d1d1dc44cfc126076c656f21


Added the Ability to Create Events Using Notation

Thanks for the great tool!
Just a simple addition that allows the addition of events using notation.
Example from the README file:
[inboxStateMachine addEventWithNotation:@"Deleted->UnDelete->Read"];

Which adds an event named 'UnDelete' for transitions from the Deleted state to the Read state.
I found this useful and more visual in my opinion.
The user can comma separate the source states:
[inboxStateMachine addEventWithNotation:@"Read,Deleted->Mark as Unread->Read"];

Added a few tests to accomodate the new changes.
Hope everything is up to standard.
Cheers,
Hisham

Not sure why Travis CI keeps failing with:
The command "gem install cocoapods" failed and exited with 1 during .


fix: datepicker issues in property pane

Fixes https://github.com/appsmithorg/appsmith/issues/23936

Media
https://github.com/appsmithorg/appsmith/assets/13763558/cdb8c883-ec4e-4aff-8b45-a11ef9d7a493
Type of change

Bug fix (non-breaking change which fixes an issue)

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/build-deploy-preview env=release skip-tests=true

/ok-to-test

This iss has been verified in the DP. Looks good.
https://github.com/appsmithorg/appsmith/assets/39921438/192478ae-7996-400f-8a91-89e29ea8dc4e

/ok-to-test


Adding angular acceleration to the Link class

Signed-off-by: Carlos Agüero<EMAIL_ADDRESS>🎉 New feature
Summary

This pull request extends the Link class adding the ability to get the angular acceleration.
Test it

Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

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

added test in 73afcaf. Marking this PR as ready

Can you just update the tutorial here?

yep updated. da78b81

The UNIT_SdfGenerator_TEST test failure looks unrelated and ABI report looks good..

The UNIT_SdfGenerator_TEST test failure looks unrelated

I shoved a fix for that into #1286 , because that test was behaving differently for different platforms. It's showing up now since the ign-common 4.5.0 release.

This pull request has been mentioned on Gazebo Community. There might be relevant details there:
https://community.gazebosim.org/t/new-ignition-releases-2022-01-24-citadel-edifice-fortress/1241/1


Inconsistent names on button for Adding taints and node labels in a custom cluster

#3876
Updated node label button text to "Add Label". Now consistent with the Add Taint button text.

Minor comment, when updating your PR from master make sure to rebase instead of merge. This will avoid additional commits once merged back to master (we don't squash).


bad export of transparent rectangle annotation

Reported by Kai Shi on FEX page on Feb 3, 2015:
I added a rectangle box in the figure and change the transparency of the box to 0.3. However, when I export the figure into latex. The transparency of the box is still 100%. With and without the -transparent option shows the same results. The code of generating the box is shown below:
annotation(hCCs, 'rectangle',rectSize, 'FaceColor','r', 'FaceAlpha',0.3, 'LineStyle','none');

The default export renderer (which is painters for vectorized formats such as EPS or PDF) does not support transparency (same in Matlab figures). I modified it so that if the figure contains any patches or transparent annotations, then the default renderer will become OpenGL. This should also solve issue #44.

This causes me problems. My OpenGL implementation is fckd up because of Haswell GMA+Radeon and Ubuntu, so I switch Matlab to use software OpenGL rendering. But the software renderer shows to be quite bad - even less featureful than the painters renderer. So I'd like to still have an option to forbid the use of opengl entirely.

I'm sorry, my bad. I noticed that when I specify the -painters option, then the opengl is not being used. Sorry for false alarm...


[e2e]: go workflow_dispatch main config-noldflags slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/3834588854
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.workflow_dispatch.main.config-noldflags.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.workflow_dispatch.main.config-noldflags.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Wed Jan  4 02:21:07 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/3843250927
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.workflow_dispatch.main.config-noldflags.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.workflow_dispatch.main.config-noldflags.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Thu Jan  5 02:23:32 UTC 2023
Tests are passing now. Closing this issue.


feature: implement oldValue in jcache

Solves #3511

Thanks for suggested pr. According to your changes old values are sent for any listener even if old value wasn't requested. This may increase pub/sub traffic for customers who don't need it.

Thanks for the feedback, that makes sense. I'm happy to give it a shot. Let me know if you have any pointers or similar concepts already I can look at for guidance.

You need to introduce oldvaluelistneres counter like with name generated like in getTimeoutSetName() method. Use incr command in registerCacheEntryListener method and decr in deregisterCacheEntryListener method. Don't send old value if counter value == false.

Looks like your message crossed my draft implementation. I think we're on the same page. I have implemented for updated only, but will complete the other listeners if you think I'm on the right track here.

I added few comments. Implementation looks right

I think there is a problem in the listener logic. When the first listener registers without requiring the old value it will instantiate a codec that doesn't expect it, and then another listener registers requiring the old value, then the first listener will start receiving it as well, and its codec will need to be instantiated as well. This will only result in an error when sync methods are used I think. I will try to produce a test case and look into how to solve it. Let me know if you have any suggestions.

I think codec needs to be modified and data with 0 length should be sent when we don't need old value.

I have updated the PR with a suggested solution. In the end, a signed short value of -1 indicates that there is no old value in the message and the decoder can skip trying to decode it. Let me know if that's sort of what you had in mind.

Thanks much for this contribution!


Add README for OpenCV_Server

What?
Add a comprehensive readme for the OpenCV_Server repo

While reading about how python projects structure their README file, I found this library for generating documentation for the code.
I created an issue for it #20. I think we should talk to Marius or Georgio about if this is something that we should focus on. Might be a huge diversion and not provide a real value. Might be very easy

This repository shows how to setup a template project with useful tools: https://github.com/rochacbruno/python-project-template

Resources:
Used this as a starting point:
https://www.makeareadme.com/

Update 04/01/2024 - End of day status
Almost done with the readme. Created a PR that is not ready to be merged (#21).
Missing parts:

[ ] Describe venv usage
[ ] Setup the main.py file so that it can be used by multiple methods
[ ] Does not talk about the python module structure (maybe remove it)

Will work with the missing part in the weekend. For now, this works. See PR #21


Payment gateway

What are the supported payment gateways (providers)?
Do you support Omnipay?

badaso support all providers
but available module is for midtrans for now https://github.com/uasoft-indonesia/badaso-midtrans-module
omnipay is also interesting package, it's already has many existing payment gateway, we will looking for that to integrate with badaso soon
you can also help us to integrate it with badaso


Updating 70.climbing_stairs.cpp

Updating the solution with a more easily understandable code according to the video solution

File(s) Modified: climbing_stairs.cpp
Language(s) Used: C++
Submission URL: _https://leetcode.com/submissions/detail/816998657/_

I didn't quite understand, should I keep both the solutions in the file including the previous one by just modifying the classname?
Thanks,

Yep that's correct,  e.g SolutionIterative, SolutionRecursive for a file that contains both an iterative and recursive approach.

That's all, @aniruddhakj. Thanks for contributing.

That's all, @aniruddhakj. Thanks for contributing!

Anytime!


gitRepoConfig usage

Hello,
Can you please tell me how to use gitRepoConfig {}, where do i place this?
gradle.properties
build.gradle
settings.gradle
Thanks!

Its actually gitPublishConfig, nvm closing


Swift 4.1 Cannot override with a stored property '...Constraint'

class AssetVideoScrollView: UIScrollView {
    private var widthConstraint: NSLayoutConstraint?

and the same error with
  @IBDesignable public class TrimmerView: AVAssetTimeSelector {
    private var leftConstraint: NSLayoutConstraint?
    private var rightConstraint: NSLayoutConstraint?

You can check it in https://github.com/Yummypets/YPImagePicker/commit/4fea39d9c3688a1ea8e36ae83dcce59a507c3e3d

Those are stored properties, private moreover, you are not supposed to override them.

But they are in your library https://github.com/prynt/PryntTrimmerView/blob/master/PryntTrimmerView/Classes/Trimmer/PryntTrimmerView.swift
The compiler warning on them.

But they are in your library https://github.com/prynt/PryntTrimmerView/blob/master/PryntTrimmerView/Classes/Trimmer/PryntTrimmerView.swift

The only thing I'm overriding in this class are the methods to setup the subviews and the constraints:
setupSubviews and constrainAssetPreview.

The compiler warning on them.

Where ? In your project or in this library ?


ignore timeToLive when null

Adds NullValueHandling.Ignore for the TimeToLive wrapper property on both EventItem and FullItem.
Addresses issue #314

Hey @all-contributors, please add @philip-reed for code and issues.

@all-contributors please add @philip-reed for issues, tests and code


Improve AtomicWrite performance with pthread_mutex and os_unfair_lock

Use os_unfair_lock if it is available, otherwise it will fall back to  pthread_mutex.
This should give a nice performance boost according to the tests in this article:
https://www.cocoawithlove.com/blog/2016/06/02/threads-and-mutexes.html

Why is the wrapper called AtomicWrite instead of Atomic?
The documentation was referring to Atomic


Temperature reading not adjusting to temperature change

Hi there!
Thanks a lot for creating the Node!
It would be just the perfect thing for me if it would show the right temperature.
Unfortunately it only shows a reading of -242.02. (important note here that it is the same temp as a friend of mine worked out and got stuck with doing it via a python program and also was this temperature mentioned in another Issue-Discussion on this Node)
I also tried holding the PT100 into 100 degrees water which also didn't change the value at all.
I have a RaspberryPi 3b+ with a MAX31865 with a PT100 attached.
The Node is configured as follows: 0,0,100,430,3,CustomName
Jumpers on the MAX31865 are set as on the Picture.

Thanks a lot in advance for helping me out if you got the time!

Hey there,
A reading of -242.02 sounds like there is a reading, but CS going wrong, or a configuration error in the board settings (resistance or wires).
Let's figure out what's happening. Could you please take the following steps:

Regarding the wiring, let's start with connecting board CS to Pi SPI0 CE0 (BCM 8): https://pinout.xyz/pinout/pin24_gpio8 - I am assuming the other wiring is correct.
ls -l /dev/spidev* should give you an indication of the bus digits. With a single board there should be only one device listed.
Create a workflow in Node-RED with an input trigger connected to the MAX31865 and a debug output to see if anything happens when you trigger.

Otherwise we should setup a screen sharing call, so I can look with you.

Thank you very much! Unfortunately the thing just started running. I was about to swap the MAX out when it started working properly. As far as I remember I did not do anything besides a couple restarts. I have it now running in 2 wire config. Hopefully this thread helps someone reading it. Thanks for your time and have a great day!

I'm having the same problem and can't get it fixed.. My PT100 has 2 wire's.
With "ls -l /dev/spidev*" my output is:
crw-rw---- 1 root spi 153, 0 Mar 24 15:01 /dev/spidev0.0
crw-rw---- 1 root spi 153, 1 Mar 24 15:01 /dev/spidev0.1


lucide.eot file missing in lucide-static

Prerequisites

Version: V. 0.244.0
Are you running from source/main: no
Are you using a released build: lucide-static, import using parceljs
Operating system: Unix
Bits: 64 bits

Step to reproduce

npm install lucide-static
Go to node_modules/lucide-static/font/.
See that lucide.eot is missing.
Try to @import "~lucide-static/font/Lucide.css";
Getting error

Actual behavior

npm install lucide-static
Go to node_modules/lucide-static/font/.
Find the file lucide.eot
Try to @import "~lucide-static/font/Lucide.css";
File found, so no error occurs

Any message or error
@parcel/resolver-default: Cannot load file '../node_modules/lucide-static/font/lucide.eot' in './src'.
Resources

No resources

Comment
Hello together, other packages like those from @fontsource only serve woff2 and woff files in their css, isn't that enough? Or is it possible to provide a css file with only the icon classes and their unicode content, so that you can add the rest yourself? Thank you.

@schmidt-oliver Thanks for pointing this out!

I found out that you are using fontcustom based on ruby and python to create the font files. There is an interesting node package called svgtofont. Eventually it is a great alternative. I will take a look and try it out.

Thanks for pointing out. Currenty I'm nog really happy with fontcustom. It's also not very well maintained. We currently have some open issues with the font. We really need stable hex values for the icons. We currently have some open issues about it: #514, #926 and #538. So a custom icon-codepoints map for the font bundler is a must.
I was looking at fantasticon  but maybe svgtofont is also a good one to look at.

Thank you and good to know. I will take this into account in my tests.

Hey, I checked out the library svgtofont. The custom icon-codepoints wouldn't be a problem here. But there is a huge problem with native shapes.  Eventually I missed something. Take a look if you want at my test repo.
I will check out fantasticicon next.

@schmidt-oliver Thanks for digging into this. I think this is not really a problem. Ok, will try to explain why.
When we used fontcustom the icons looked crap as well, not because of the antive shapes but because of the stroke-widths we use in the icons.
We fixed this by outlining all the icons and use this to create the font.  See script: build:outline-icons in root package.json, the package and scripts are located under "tools/outline-svg".
This will create a directory called "outlined" in the root with all the icons with outlined vectors. And those we can use to make the fonts.

@ericfennis I noticed that we missed to adjust the Github workflow for new releases. Can you please adapt it like the one for lucide-static? Thank you.
https://github.com/lucide-icons/lucide/blob/4cc4468d2f2832be31670f67216a4f73abd3f591/.github/workflows/release.yml#L188-L189C8

You're right, I thought we did this already. Yess I will.
I will reopen this one, to make sure we don't forget this one!


feat: repack-init

Summary
This PR introduces repack-init, a utility npm package that automates the process of integrating the Repack bundler into React Native projects.
Here's a summary of the functionality repack-init provides:

Package Manager Detection: Identifies the package manager (npm or yarn) being used in the project.
React Native Verification: Checks whether React Native is installed in the project, and what version is being used.
Dependency Addition: Utilizes the established package manager to add necessary dependencies to the React-Native project.
Configuration of React Native Config: Checks for the existence of react-native.config.js and adds or modifies the necessary fields (commands).
Webpack Configuration: Creates and configures webpack.config.js based on user-provided flags (--mjs or --cjs).
iOS Modification: Updates the project.pbxproj file in the iOS directory.
Android Modification: Updates the build.gradle file in the Android directory based on specific React Native version requirements.

This tool aims to simplify the setup of Repack in React Native projects, ensuring developers can quickly start using Repack's features.
Limitations:
Minimum required node version: 16
Usage
Usage: @callstack/repack-init [options]

Options:
  -e, --entry    Path to main entry point for the React-Native project
                                                  [string] [default: "index.js"]
  -f, --format   Format of the webpack.config file
                               [string] [choices: "mjs", "cjs"] [default: "mjs"]
  -v, --verbose  Enables verbose logging              [boolean] [default: false]
      --version  Show version number                                   [boolean]
      --help     Show help                                             [boolean]

Example outputs
✔  Dependencies installed
✔  Created webpack.config.mjs from template
✔  Created react-native.config.js
✔  Added "webpack-bundle" as bundleCommand to android/app/build.gradle
✔  Added "webpack-bundle" as BUNDLE_COMMAND to build phase shellScript in ios/test1.xcodeproj/project.pbxproj

🎉 Setup complete. Thanks for using Re.Pack!

maybe it's worth checking this lib for pbxproj parsing instead of relying on indexOf


Fix build failure in _SynchronizationShims module

When Foundation adds an import Synchronization to its module interface, we end up with build failures while building the benchmarks project due to the following issue in the _SynchronizationShims module:
/home/build-user/swift-nightly-install/usr/lib/swift/shims/_SynchronizationShims.h:26:19: error: a type specifier is required for all declarations
24 |
25 | static inline __swift_uint32_t _swift_stdlib_gettid() {
26 |   static __thread tid = 0;
   |                   `- error: a type specifier is required for all declarations
27 |
28 |   if (tid == 0) {

I'm uncertain why we don't see this failure on mainline, but this fixes the error so that Foundation can successfully add the Synchronization import without breaking downstream clients that will need to compile this clang module.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test macOS

@swift-ci please smoke test macOS


3D subcameras refactoring

Ok I think I get it.  Just one question: How do you integrate the WebVR and the VRDeviceOrientationCamera ?

Will look at tomorrow, but the fundamental concept is: Camera class does the actual rendering.  Different Camera subclasses only do the aiming through events:  keyboard, mouse, touch, and / or orientation.
For those 2, the only thing that should be still in the class are the event handling.  If a stereogram WebVR comes along someday, there would be nothing to do.  An Oculus should still work with a Gamepad.  It is just a gamepadcamera with rendering for an Oculus.

Yep I think I captured it..let me know your thoughts.
Do you mind documenting this powerful function in the camera doc?


Indicate required status on website

@nicolasvanermen suggested to indicate on the website (https://ices-tools-dev.github.io/esas/tables/) which fields are required. I think that is a good idea, I'll see if I can adapt to layout (https://github.com/ices-tools-dev/esas/blob/main/_layouts/tables.html) to do that automatically.

Fixed with 142d94994403f8d12983c18bbf32ef8bc5c81dfe


[sdl2-mixer] Enable native MIDI support on macOS

Enable native MIDI support for sdl2-mixer on macOS, like it is done on Windows.

I was just looking for this exact feature for https://github.com/GPSnoopy/BelgianChocolateDuke3D :-D

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: jgiannuzzi sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Hi @PhoebeHui, I made the required change.
I tested the feature locally on macOS Catalina 10.15.3.
I also checked that MIDI was still working on Windows 10 1909 64bit.

Need to test features.

Hey @PhoebeHui and @JackBoosY, do you need anything from me before this can be merged?

Just waiting for merge this PR.

Thanks y'all! This is awesome !


swayrbar: 0.4.0 -> 0.4.2

Semi-automatic update generated by nixpkgs-update tools.
meta.description for swayrbar is: Status command for sway's swaybar implementing the swaybar-protocol
meta.homepage for swayrbar is: https://git.sr.ht/~tsdh/swayr#a-idswayrbarswayrbara
Updates performed

Rust version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.4.2 with grep in /nix/store/22ya3w4qf4b30m6clhzb1z1rkzwcvika-swayrbar-0.4.2
found 0.4.2 in filename of file in /nix/store/22ya3w4qf4b30m6clhzb1z1rkzwcvika-swayrbar-0.4.2

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
swayrbar

Instructions to test this update (click to expand)

Build yourself:
nix-build -A swayrbar https://github.com/r-ryantm/nixpkgs/archive/d7a38923f90100d4d157b0eae99bc92aaee66e14.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/d7a38923f90100d4d157b0eae99bc92aaee66e14#swayrbar

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/22ya3w4qf4b30m6clhzb1z1rkzwcvika-swayrbar-0.4.2
ls -la /nix/store/22ya3w4qf4b30m6clhzb1z1rkzwcvika-swayrbar-0.4.2/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @SebTM for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.
Semi-automatic update generated by nixpkgs-update tools.
meta.description for swayrbar is: Status command for sway's swaybar implementing the swaybar-protocol
meta.homepage for swayrbar is: https://git.sr.ht/~tsdh/swayr#a-idswayrbarswayrbara
Updates performed

Rust version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.4.2 with grep in /nix/store/22ya3w4qf4b30m6clhzb1z1rkzwcvika-swayrbar-0.4.2
found 0.4.2 in filename of file in /nix/store/22ya3w4qf4b30m6clhzb1z1rkzwcvika-swayrbar-0.4.2

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
swayrbar

Instructions to test this update (click to expand)

Build yourself:
nix-build -A swayrbar https://github.com/r-ryantm/nixpkgs/archive/d7a38923f90100d4d157b0eae99bc92aaee66e14.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/d7a38923f90100d4d157b0eae99bc92aaee66e14#swayrbar

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/22ya3w4qf4b30m6clhzb1z1rkzwcvika-swayrbar-0.4.2
ls -la /nix/store/22ya3w4qf4b30m6clhzb1z1rkzwcvika-swayrbar-0.4.2/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @SebTM for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.
[result] Success updating swayrbar 0.4.0 -> 0.4.2

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 358643

x86_64-linux

:white_check_mark: 1 package built:

swayrbar

Hey, I do not use sway anymore atm. so I can't really test beside nixpkgs-review. Could you do me a favor and remove me as maintainer?


v3.12.2

Remove v1 assets and code

Pull Request Test Coverage Report for Build 2569

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 70.057%

Totals

Change from base Build 2564:
0.0%

Covered Lines:
2233

Relevant Lines:
2986

💛  - Coveralls


Used by filecache repo, should return all executor IDs when register …

…a new executor
Filecache repo added a new feature to register an executor ID and return all the active executor IDs.
Modify RapidsDriverPlugin.receive to return real message instead of null for filecache messages.

First merge the filecache commit, change is:
override def handleMsg(msg: FileCacheLocalityMsg): Unit =

==>>
override def handleMsg(msg: FileCacheLocalityMsg): AnyRef

After filecache commit is merged, then this PR can pass compile.

build


[CORE] Prefer to use Gluten plan in fallback policy

What changes were proposed in this pull request?
If the vanilla Spark plan also contains ColumnarToRow and the number of ColumnarToRow is not smaller than Gluten ColumnarToRow, we should better to prefer to use Gluten plan. For example,
Scan Parquet
     |
ColumnarToRow

This pr adds a new config spark.gluten.sql.columnar.fallback.preferColumnar and enables it by default.
How was this patch tested?
Add tests

cc @PHILO-HE @zhouyuan thank you

cc @PHILO-HE any more comments ? thank you


Fix grammar and typos in README

@Arihant25 Your changes look quite good to me. Thanks a lot for all these fixes.
Merging your PR.


Conjur Buildpack doesn't fail when retrieval of secrets fails

Summary
deployed an app that had conjur secrets, saw in logs it said it was injecting the vars, then failed due to conjur being down. Since the app certainly relies on conjur for environment variables we need the push to fail with an error or the app needs to fall to startup so that we are aware of this issue and don't have to dig in logs to findout.
Steps to Reproduce
Steps to reproduce the behavior:

push an app with conjur secrets.yml and buiildpack
check logs that it failed to retrieve creds due to conjur connectivity/down issue
see that app reports as up and started (no visible errors other than in logs)

Expected Results
app would fail to start/faill to push because it couldn't contact conjur
Actual Results (including error logs, if applicable)
app starts up without env vars that are required.
Reproducible

[ X] Always
[ ] Sometimes
[ ] Non-Reproducible

Version/Tag number
conjur_buildpack               1          true      false    conjur_buildpack-2.1.4.zip
and a java app running springboot
Environment setup
PCF on premise

@mkkeffeler thanks for filing this. I'm checking with our contacts at VMWare to see if they have any insight into why the cf push process continues after a non-final buildpack specified in the app manifest or with the -b flag of the cf push command exits on a non-zero code.
From reviewing the app manifest and cf push documentation, it does not appear that you can indicate in your app manifest or with the cf push command that you want the push to fail if any specified buildpacks exit on an error code.
One way to manage this that occurs to me is to have your app startup process fail if the required secrets are not present in the app env - I'm not sure whether that's workable for your org, but I have low confidence that CF provides a solution for critical non-final buildpacks. I'll update this issue when I hear more on this.

yeah, had considered that as well. we need somethiing that doesnt require a developer know what to do, and rather they just push with their manifest and that manifest will fail it. whatever that looks liike

@mkkeffeler we've explored a few options with VMWare and there doesn't seem to be a mechanism to force cf push to fail unless it fails in the compile phase of the language buildpack (e.g. not this buildpack). I don't know of another way to enforce this without application changes, unfortunately - it's a limitation of the system that this buildpack is running in.
We could explore options for providing other tooling that would verify that the buildpack completed its run and/or that secrets were appropriately injected. If you're interested in pursuing this route, I'd suggest working with your account executive to file an official enhancement request.
Since at this time I'm not aware of any changes we could make to the buildpack itself that would change this behavior, I'm going to close this issue for now.


Date Subscription Renew

Is there any way to get the date the subscription will renew after you bought it? I can see a TransactionDate but not when it will renew.
Would like to see a PurchasedItem.renewDate or something similar.
Or is it something I am missing?

@RobertBrunhage Um.. Currently, I've no idea how to get it. I'll look for this one. Also I found this in stackoverflow.

I know that the FlutterPayments pub has a renewDate variable, will try to see if I can understand it there and comment back here if I found anything!

Here he created the variable purchaseExpires variable: https://github.com/delightfulgoods/flutter_payments/blob/master/lib/src/purchase.dart

@RobertBrunhage Oh. Yes that is supported from ios 11.0.2. I can manage this in ios using this one. I may take some time cause I am working on something else now. I'll try this on the weekend.

If it helps it seems he gets this:

And then have this function that he adds it on

Thanks mate, looking forward to it! Will you look into android also? Testing it out on android atm :)

@RobertBrunhage Yes I've seen it. I may get subscriptionPeriod parameter but I won't transcode it like he did. That may be done manually with developer.

Yeah that would be great also, I just need a way to get the period of the subscription.

@RobertBrunhage I've just implemented the call for subscriptionPeriod. I've read through the docs and found that android and ios are giving different types of variables for this. I may manage to be the same later but currently, I've just bridged the parameter that is sent from each platform. Also, flutter_payments plugin only had subscriptionPeriod sent from android when I looked in the code. Just remind yourself.
Here is general guide for getting the period for each platform.

Android - subscriptionPeriodAndroid will give you a period in milisecs.
IOS - subscriptionPeriodNumberIOS will give you the unit number and subscriptionPeriodUnitIOS will give you the unit in String like DAY or WEEK or MONTH or YEAR.

Awesome! Just another question, how do you check if a subscription is active?

Also tried print(items[0].subscriptionPeriodAndroid); which returns P1W.

@RobertBrunhage You can refer here but I think only way is to check the receipt.

@RobertBrunhage Oops what's that? Let me check.

@RobertBrunhage Sorry I was wrong. Maybe I read the wrong article. Here you may know what P1W stands for.

Ahh okay, thanks!
Still having problems understanding how to check if a subscription is active though. I store the date on my server on when the subscription will end so that I only check the IAP when that date is reached. Then I want to check again if it is renewed or not to add the new date.

@RobertBrunhage I think you should check that with rest apis in each platform. Currently, you may follow here for android. And follow here for ios.

Closing this for now since this is no more an issue.

@RobertBrunhage There seems to be no native way to check subscription status, but I do have this working in theory which might be useful to you. I may do a PR to add this as a convenience method.
This assumes there is only one product and it is a subscription.
  getHistory() async {
    await FlutterInappPurchase.prepare;

    if (Platform.isIOS) {
      var history = await FlutterInappPurchase.getPurchaseHistory();
      FlutterInappPurchase.endConnection;

      history.sort((a, b) => (a.transactionDate.millisecondsSinceEpoch)
          .compareTo(b.transactionDate.millisecondsSinceEpoch));

      final transactionDate = history.last.transactionDate;
      int minutes = DateTime.now().difference(transactionDate).inMinutes;
      bool isSubscribed = minutes <= 5;

      print(
        'we ${isSubscribed? "are" : "are not"} subscribed, sub is ${minutes}m old',
      );

      return isSubscribed;
    } else if (Platform.isAndroid) {
      var history = await FlutterInappPurchase.getPurchaseHistory();
      var purchases = await FlutterInappPurchase.getAvailablePurchases();
      FlutterInappPurchase.endConnection;

      for (var purchase in purchases) {
        if (purchase.productId == "when_coin_premium") {
          print('subscribed!');
          return true;
        }
      }
      print('not subscribed');
      return false;
    }
  }

@lukepighetti Yeah, I would love to see that as a convenience method!

@lukepighetti Review this commit. I've made changes to your method.

That's fine with me! :)

@lukepighetti  @hyochan @RobertBrunhage Did you found any way to check that any subscription is active at the time or not? If it is cancelled from play store even then we can have that response properly.
Can you suggest me how can i achieve this subscription date expiry or active subscription ?
This is most important feature reuired for subscription part. But, still there is not so much solutions available. Can we have any workaround solutions for this?
Thanks.

@lukepighetti @hyochan @RobertBrunhage Did you found any way to check that any subscription is active at the time or not? If it is cancelled from play store even then we can have that response properly.
Can you suggest me how can i achieve this subscription date expiry or active subscription ?
This is most important feature reuired for subscription part. But, still there is not so much solutions available. Can we have any workaround solutions for this?
Thanks.

I went with RevenueCat for subscriptions instead, worked wonders :)


[WIP] experiment: use cached packages

just some experiments to guide solver to use some already "cached" packages present on the systems.
related to https://github.com/prefix-dev/pixi/discussions/1354
Quo vadis?
By hypothetically having "boltons_21.0.0" in cache/pkgs, we can guide solver when adding boltons="*" to reuse already cached pkg, thus adding boltons==21.0.0

Is this still something you would like to work on?

Is this still something you would like to work on?

yep! just don't have enough time to get into this


Structural changes

Summary
Reordering repo and simplifying the config
Details
removing default values from config.py and keeping all configs in config.yaml
moving all source files from the main dir to 'src'

Hi @BenSiv
Thanks for your contribution!
Cleaning up the configs is a good idea.
@Jerakin was already working on moving everything to a separate src folder in MR #147.


feat: refactor binary interactions in test framework

Consolidates test framework interaction with binaries into a binary pkg

@bharathkkb What's the status of this PR?

@morgante I de prioritized this for some feature work. I will pick this up soon.


Umbraco 7.2.2 requests en.js language file

After upgrading to Umbraco v7.2.2, I got a bunch of errors when viewing/saving any data-type that is using Archetype.
404 (not found): /App_plugins/Archetype/langs/en.js

I'm not sure what has actually changed in v7.2.2, but my guess would be that the default culture for a back-office user has been switched from "en-GB" to generic "en".
I resolved this issue by making a copy of the en-gb.js and renaming it en.js.
Is it worth adding a default en.js to the Archetype distribution?

Yep.  It was reported last night.  Trying to find time for a release.
https://our.umbraco.org/projects/backoffice-extensions/archetype/sound-off!/61678-Archetype-Shows-404-Errors-with-Umbraco-722

Thanks @kgiszewski - I can throw over a PR if that helps?
(No rush from me on a release) :+1:

Thx, but it'll be just as easy if I do it.

Alright guys, I was finally able to repro (browser cache was tripping me up, d'oh!) and know the cause now. Please have a look at the issue: http://issues.umbraco.org/issue/U4-6307
For now the easy fix for you is to provide extra languages (en.js, dk.js, en_us.js.. yes, underscore, not dash.., etc.).

@nul800sebastiaan Thx for tracking it down!  I'll try to get a patch out soon.

Thanks @nul800sebastiaan - makes sense!

v1.7.1 fixed it


Version Bump

Can we get a version bump so we can pull the latest changes from bower?  I'm using the expire-callback in a project and would like to not have to manage working from my fork.

Created a new release 0.4.0. Also adding the ability to test out examples on a local test server (Node.js). Documentation for the updates was added as well. Thanks for the request @ryanblanks

Closing after bumping version.


Camera calibration and convert between camera coords and real world coords with realsense camera

Required Info

Camera Model
{ L515}

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
Linux (Ubuntu 20.04)

Kernel Version (Linux Only)
5.15

Platform
PC

SDK Version
}

Language
python }

Segment
{Robot }

Issue Description
Currently, I'm trying to dive understand related to the camera coordinates system to real word camera system. And this is my detail problem. Hope I could have the more information from guys.

Theory:
I follow this pinhole model:

According to the equation as below:

I use the equation as below for the coords transformation :

Hence, this is my detail what I do:
import pyrealsense2 as rs

pipeline = rs.pipeline()
pipe_profile = pipeline.start()
frames = pipeline.wait_for_frames()
depth_frame = frames.get_depth_frame()
color_frame = frames.get_color_frame()

depth_intrin = depth_frame.profile.as_video_stream_profile().intrinsics
print("depth intrinsics:\n",depth_intrin)
color_intrin = color_frame.profile.as_video_stream_profile().intrinsics
print("color intrinsics:\n",color_intrin)
depth_to_color_extrin = depth_frame.profile.get_extrinsics_to(color_frame.profile)
print("Extrin Camera:\n",depth_to_color_extrin)

I obtain extrinsic matrix & intrinsic matrix camera based on the code as above.
Then, I'm trying to convert to the real word as above:
with tvec_targer2cam , r_target2cam from extrinsic  matrix camera
and cameraMatrix_realsense from intrinsic matrix
cameraMatrix_realsense = np.array([[901.353, 0.0, 654.587],
                                   [0.0, 901.834, 351.407],
                                   [0.0, 0.0, 1.0]])
tvec_targer2cam = tvec_targer2cam.reshape(3,1)

r_target2cam_inv = np.linalg.inv(r_target2cam)

# Example point in camera frame
u =770
v=440
scale=1

suv = scale*np.array([u,v,1]).reshape(3,1)

# ([Xi,Yi])/scale = Min x [Xc,Yc,Zc]
point_cam_coords = np.dot(np.linalg.inv(cameraMatrix_realsense),suv)
# Pinhole model [Xc,Yc,Zc] = R[Xw,Yw,Zw] + t

point_real_coords = point_cam_coords - tvec_targer2cam

point_real_coords = np.dot(r_target2cam_inv,point_real_coords)
print("point_real_coords:\n",point_real_coords)

So, I test with u,v I obtain when I plot the image with plt() and move the mouse to get the pixel u,v.
The output of point_real_coords  is:
x =0.12844626
y =0.10895989
z =1.00277352

For the information provided as above. What's the unit of x,y,z output?

Hi @LeVHoangduc  The L515 uses the coordinate system of the D435i camera model that is described at the link below.
https://github.com/IntelRealSense/librealsense/blob/master/doc/d435i.md
In this system, X is horizontal, Y is vertical and Z is depth (pointing forwards from the front of the camera).  The distance unit of the real-world values is meters.

Hi @LeVHoangduc  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


add initial unit test

This is an initial unit test for the metdata builder. It loads a sample DLL using the hosting API and then runs the store_wrapper_method_ref  function.

@lucaspimentel I added the iterators code. It converts ms-style enums to c++ for/range loops. I also added a second test for a method in a separate DLL.


Astra DB vector store imports from newly-named package

WARNING: Do not merge until llama_index/#11056 is merged (and the newly-named package published!)
This aligns the import statement for AstraDBVectorStore in the two example notebooks to reflect the package renaming that the PR referenced above implements.
It should be merged once the llama_index integration package is published as llama-index-vector-stores-astra-db on PyPI.

You are abslutely right, thank you for spotting that!
I cannot imagine how I could have gotten that wrong. Sorry about the confusion, fixed now.


feat: support journald appender

This fixes #76

@tisonkun Why am I being mentioned here?  I'm sorry but I'm entirely missing the context :thinking:

@swsnr I'm using your code from systemd-journal-logger.rs and here to share patches or ideas on these two TODOs in your code:

https://github.com/swsnr/systemd-journal-logger.rs/blob/7560406e380adb0ea79dd044e70098e9c4017b66/src/fields.rs#L74
https://github.com/swsnr/systemd-journal-logger.rs/blob/7560406e380adb0ea79dd044e70098e9c4017b66/src/fields.rs#L48

Perhaps open an issue/PR at systemd-journal-logger.rs a better choice .. Just found quoted code here can be easier to convey the background.

Ah, thanks. That's the background I was missing.  I'll take a look 🙏
If you can find the time to make a PR that be appreciated a lot 😊


[ECS] Parallel scan does not work

Yesterday I've tried parallel scan end it ended with error like this:
> git diff origin/develop... --name-only --diff-filter=ACMR -- '*.php' | xargs -r ecs check

Warning: substr() expects parameter 1 to be string, object given in .../vendor/hoa/protocol/Protocol.php on line 170

Warning: is_dir() expects parameter 1 to be a valid path, object given in .../vendor/hoa/protocol/Protocol.php on line 171

Fatal error: Uncaught TypeError: Argument 1 passed to ECSPrefix20210923\React\Promise\Promise::settle() must implement interface ECSPrefix20210923\React\Promise\ExtendedPromiseInterface, instance of ECSPrefix20210923\React\Socket\Connection given, called in .../vendor/symplify/easy-coding-standard/vendor/react/promise/src/Promise.php on line 203 and defined in .../vendor/symplify/easy-coding-standard/vendor/react/promise/src/Promise.php:139
Stack trace:
#0 .../vendor/symplify/easy-coding-standard/vendor/react/promise/src/Promise.php(203): ECSPrefix20210923\React\Promise\Promise->settle(Object(ECSPrefix20210923\React\Socket\Connection))
#1 .../vendor/symplify/easy-coding-standard/vendor/react/socket/src/TcpConnector.php(73): ECSPrefix20210923\React\Promise\Promise::ECSPrefix20210923\React\Promise\{closure}(Object(ECSPrefix20210923\React\Socket\Connection))
#2 .../vendor/sympl in .../vendor/symplify/easy-coding-standard/vendor/react/promise/src/Promise.php on line 139

Runtime: PHP 7.4.22 on macOS 11.6, ECS 9.4.65

In https://github.com/symplify/symplify/pull/3726/files we removed the dependency on symfony/http-kernel and caching removal in general. The container is build per config on run and clearing cache is not neccesary.
This issue should be resolved now.

Thank you @TomasVotruba, I'll check it since I am currently working on migrating to PHPStan 1.0 and Rector 0.12 🙂


Browser API link for v2.0.0 is not working

I've been trying to access the browerAPI link, but it doesn't exist for v2.0.0.
Although Link for v1.0.1 is working fine

Oh yeah, I haven't published v2 to NPM, I got busy with school.
The last time I touched this project, I was rewriting it in TypeScript, I'll see if I still have the code somewhere and publish it by at least tomorrow.

I've justed pushed and publish version 2.0.0, the module can be accessed in the browser using<EMAIL_ADDRESS>let me know if you encounter any issues, cheers.


Silence run status if all hooks are marked quiet

Fixes #357

Thanks for the pull request.
As I suggested in #357, I think a global quiet option would be better for this, as it saves you from having to explicitly mark all hooks as quiet. I've gone ahead and done that in cefbadc.

Thanks! :+1:


remove usage of dictionary

reduce dictionary usage from tabular data resource

For posterity, what would this help with? Couldn't find much in the description, so my best guess is that this would help us with conversions to and from the Tabular data resource helping with data frame sharing (https://github.com/dotnet/interactive/issues/2060). Is that accurate?


DO machines not deleting from provider when I delete cluster

Rancher versions: 2.0 master 1/29
Steps to Reproduce:

Create a RKE cluster with Digital Ocean machines
Wait for it to become active
Delete cluster

Results:  The DO machine are not deleting from the provider and are stuck "Waiting on controller.cattle.io/machine-controller"

E0130 17:59:30.890497       1 generic_controller.go:191] ClusterController cluster-5g9qh [cluster-provisioner-controller] failed with : Operation cannot be fulfilled on clusters.management.cattle.io "cluster-5g9qh": the object has been modified; please apply your changes to the latest version and try again, [mgmt-cluster-rbac-remove] failed with : Operation cannot be fulfilled on clusters.management.cattle.io "cluster-5g9qh": the object has been modified; please apply your changes to the latest version and try again, [cluster-scoped-gc] failed with : Operation cannot be fulfilled on clusters.management.cattle.io "cluster-5g9qh": the object has been modified; please apply your changes to the latest version and try again
time="2018-01-30T17:59:30Z" level=info msg="Deleting cluster [cluster-5g9qh]"
time="2018-01-30T17:59:30Z" level=info msg="Deleted cluster [cluster-5g9qh]"
E0130 17:59:30.958139       1 generic_controller.go:191] ClusterController cluster-5g9qh [mgmt-cluster-rbac-remove] failed with : Operation cannot be fulfilled on clusters.management.cattle.io "cluster-5g9qh": the object has been modified; please apply your changes to the latest version and try again
E0130 17:59:30.986920       1 generic_controller.go:191] ClusterController cluster-5g9qh [cluster-stats] failed with : Operation cannot be fulfilled on clusters.management.cattle.io "cluster-5g9qh": the object has been modified; please apply your changes to the latest version and try again
E0130 17:59:31.014272       1 generic_controller.go:191] ClusterController cluster-5g9qh [cluster-stats] failed with : Operation cannot be fulfilled on clusters.management.cattle.io "cluster-5g9qh": StorageError: invalid object, Code: 4, Key: /registry/management.cattle.io/clusters/cluster-5g9qh, ResourceVersion: 0, AdditionalErrorMsg: Precondition failed: UID in precondition: 40a77b85-0548-11e8-919b-0242ac110002, UID in object meta:
E0130 17:59:35.931095       1 generic_controller.go:191] MachineController cluster-5g9qh/machine-2dqvc [machine-controller] failed with : cluster.management.cattle.io "cluster-5g9qh" not found

Version - 2.0 master 2/3
Verified fixed


Update nerdicons.lua

Added icons for D and Haxe source files

I've changed version 1.2.3 ⟶ 1.2.4


Invalid URL: "http://www.."

Hello,
Please take a look at the address bar in the attached image.

All emails point to the wrong website. The URLs in the emails always contain double dots i.e. "**http://www..**TLD". I have checked the entire backend if I entered the wrong URL somewhere at some point but not so. May I add that the /storage/logs/system.log, environment.php and app.php indicate the correct URL i.e. https://www.TLD. How come the contents of the emails get altered?
OctoberCMS build version: 434

@micadeyeye take a look at https://github.com/rainlab/user-plugin/blob/master/components/ResetPassword.php#L145 and do some debugging in that method to see why it's generating the wrong URL.

Closing as it has been over a month since any activity on this occurred and we are trying to figure out what issues are still relevant. If this is still something that you would like to see through to fruition please respond and we can get the ball rolling.


Fibonacci_series.cpp

code regarding Fibonacci series generator using cpp by number of terms

https://github.com/PummyKumari123/programming-4/blob/main/fibonacci.-cpp.zip
Fibonacci program


Kinesis - Fix split_shards behaviour

Fixes split_shards behaviour to create two new shards, instead of the existing behaviour of reusing one
Adds error scenarios, and fixes existing error messages
Fixes some erroneous Response-attributes when describing a stream
All new tests were validated against AWS.
Closes #1405
Closes #1475

Codecov Report

Merging #4540 (c1986b3) into master (3353677) will increase coverage by 0.01%.
The diff coverage is 93.93%.

@@            Coverage Diff             @@
##           master    #4540      +/-   ##
==========================================
+ Coverage   95.14%   95.16%   +0.01%
==========================================
  Files         514      515       +1
  Lines       58480    58485       +5
==========================================
+ Hits        55643    55655      +12
+ Misses       2837     2830       -7

Flag
Coverage Δ

servertests
47.95% <42.42%> (-0.05%)
:arrow_down:

test_responses
32.29% <12.12%> (+<0.01%)
:arrow_up:

unittests
95.06% <93.93%> (+0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

moto/kinesis/models.py
90.65% <92.85%> (+1.76%)
:arrow_up:

moto/kinesis/exceptions.py
92.00% <100.00%> (+1.52%)
:arrow_up:

moto/s3/models.py
94.49% <0.00%> (-0.08%)
:arrow_down:

moto/iot/utils.py
100.00% <0.00%> (ø)

moto/cognitoidp/responses.py
98.59% <0.00%> (+<0.01%)
:arrow_up:

moto/cognitoidp/models.py
94.91% <0.00%> (+<0.01%)
:arrow_up:

moto/cloudformation/parsing.py
97.20% <0.00%> (+0.01%)
:arrow_up:

moto/iot/models.py
91.74% <0.00%> (+0.36%)
:arrow_up:

Continue to review full report at Codecov.

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


Tock 2.0 Alpha1

This updates tock-2.0-dev to master and irons out a few remaining wrinkles I found, particularly with the imix test app. It also adds an additional memory test, which walks to the end of the heap to find the end of the readable region. Finally, it updates the MPU test that checks the end of RAM to note that it should fail, since there are grants there.
This PR was tested by running the imix test application, mpu_flash_end, mpu_heap_end, and mpu_ram_end.
Note that in this state libtock-c does not uniformly follow one aspect of the Tock 2.0 calling conventions, particularly returning EBADVAL if the return type of a system call does not match what is expected. Some modules do, others don't. The plan is to clean this up before when we do a release.

bors r+


tests: Update anomaly logging to use new config

This PR is dependent on Suricata #4100 and provides updates per the configuration changes in that PR.
Both PRs are required for successful tests.

Continued in #116


script to update version and commit sha

A small script (Python 3 only) that makes it easier to update commit SHA hashes for googleapis and gapic-generator, and bump the version. Less manual edits, less editing errors. Requires #449 to be merged first.
$ python3 update-versions.py
usage: update-versions.py [-h] [--fetch-googleapis] [--fetch-gapic-generator]
                          [version]

Prepare new artman release.

positional arguments:
  version               Set version (MAJOR.MINOR.PATCH).

optional arguments:
  -h, --help            show this help message and exit
  --fetch-googleapis    Fetch the latest googleapis commit SHA from GitHub and
                        store in Dockerfile.
  --fetch-gapic-generator
                        Fetch the latest gapic-generator commit SHA from
                        GitHub and store in Dockerfile.

@garrettjonesgoogle @lukesneeringer ping on this please :)


[SIDE-1007] C# LDK Completion

This PR implements the remainder of the services in the C# LDK and makes way for its publication.

This is impressive work. However I feel unqualified to review it. I recall hearing that @dmorrison-olive has some C# experience. Do we need to find a second person?


[BUG] I were starting server and this happened

Describe the bug
A clear and concise description of what the bug is.
The bag says ":x: Failed to start! Reconfigure the bot with /configure and try again. See /help if the problem persists." when i restarted the server bug stayed and I don't know what to do.
A clear and concise description of what you expected to happen.
Idk what happened but I hope you will read it and help me I am creating a server for myself fr so I hope the answer will be soon as possible.

(just to let you know if that could help, I am macOS user)

I'm seeing a lot of Aternos failed to start the server. in the logs. That can mean one of the following:

It's a temporary issue on their end
You misconfigured the bot
They changed something security related, breaking the bot in the process.

Maybe the 3 one because I tried it on many laptops and it still did not work even on them, also I used different VPNs and browsers.

Actually, I can see in the log it only happens with one server (probably your discord server). So it's most likely nr 1 or 2. Did you start the server manually and accept EULA before using the bot? Read /help, it's stated in there.

Yes i accept it and tried to restart server again and there is still this problem :(

@AUUUUUUUUUGH  Try this:

Kick the bot
Invite the bot
Run your Aternos Server
config the bot
try again!

Closing because I believe the actual problem OP is facing now is  #44 Reopen if I'm wrong.


Improve testability of the Real interpreter

There is an issue in how we test actions. Fundamentally, we rely on the fact that the Real runner is generating git calls correctly. This could be neglected in theory, but there is another issue, which forces us to generate "textual" representation of what our command is doing, in multiple places, which leads to code repetition.
As a solution to this, I've created 1 data type for every type of the command we can execute, and implemeted the way to render it to the "textual" representation.
This could be used by Real runner directly, but this still suffers from a different issue. We give Real runner to much access, by adding MonadIO to the context.
There is a solution however. We can create a custom Monad which can provide all needed functions for Real to proceed. Going this way, we can hide how we "render" and "execute" git calls from the Real runner, which implies that it has no way to run arbitrary commands at all.
#299
The contribution:

[ ] updates all affected documentation
[ ] provides commits messages which comply with the CONTRIBUTING.md > Committing the changes rules
[ ] updates the completion scripts if requires
[ ] complies with all requirements from README.md > Hands-on development notes

@bees-hive/elegant-git-maintainers, please review.

@teggotic please resolve conflicts here

@teggotic please resolve conflicts here

Sure, @extsoft please check


"Snake" touch input

The mouse-like behaviour isn't optimal.
Users would like to drag the primary panel and have the secondary follow based on the direction dragged.
The "cursor" should be hidden when all touch has been released.

I believe a good UX can be achieved by hiding the (current) cursor initially, only showing the animated "hand". If a true mouse or keyboard event is initiated the cursor becomes visible. Just to be safe it should be hidden by any final touchend event.


Angular- Table view- Link tab and cluster tab does not seem to work

When I click on link tab, I see the same thing as Nodes tab

Issue was that the way the values were being accessed in the html from the typescript.  Have fixed it to access the correct values and just made a push.  Should be visible soon.


Devices names not showing

Hi, I'm using this library to get list of devices connected to the router... it is giving me the list of IP addresses and mac addresses but not the device or at least vendor names.. I understand it is router specific also that some routers do not allow this information. But can you please add some methods to get device or vendor names? Thanks

I'm not currently aware of any free services that provide an api to lookup mac address to vendor name. If you know of any please let me know and I can add it in :)

I'll look up any such services but there are a ton of free apps that can give out vendor names.. for example https://play.google.com/store/apps/details?id=com.wifirouter.whousemywifi  ...
And Eshareserver https://play.google.com/store/apps/details?id=com.eshare.clientv2  gives out device names too

ok here is a free service https://macvendors.com/api

Hey, so here https://macvendors.co/api  is the link to free API. It's available in json to lookup vendor name using Json and it's free and unlimited. Hope you can add this to your library so it'll do the entire job. Thanks


Refresh of the webwallet

First refresh commit
Deleting files become useless
Set the toolbar color to gray.
Add gh-pages for install in github website
Delete unnecessary files
Update package.json files
"InputGroupButton" component has been deprecated
'hushjs' & 'bs58check' are assigned a value but never used.
Deletion of many variables assigned but never used.
Replace '==' with '==='
'info_data' is assigned but never used
Unexpected string concatenation of literals
Update 'package.json' file
Replacement of the "CardBlock" component with "CardBody". (deprecied)
Deleting a double declaration of variable 'i'
Deleting a double declaration of variable 'c_pk_wif'
Modification of the data presentation visible in the tables.
Change the width of the columns to display the addresses in full.
Fixed a bug : 43 addresses are created from the same chain
Add svg icon
Little modification of the presentation
Removing dependency on 'react-icons'
Add Hush logo in SVG
Adding links on icons
Integration of the 'hushjs' library
Removed variable 'bigi' because it is never used.
Removed variable 'int64buffer' because it is never used.
Delete the 'zbufferutils' variable declaration that is duplicated.
Remove the double declaration of the variable 'j'.
Remove the double declaration of the variable 'i'.
Replacing a '==' with '===' in an if
Deleted of 'gh-pages' dependencies
Deleted of useless parameter
Update 'package-lock.json' file.
Deleted of 'registerServiceWorker'
Reduce the size of the contacts icons
Deleting a useless function
Changing the API path
Update all dependencies.
Graphical update of the 'contacts' page
Open new tab to explorer

@gilardh wow!!! Thanks so much for working on this


fix(types): Add create method to EmailAddress typings

Type of change

[x] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[ ] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[x] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[ ] @clerk/backend
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/fastify
[ ] @clerk/chrome-extension
[ ] gatsby-plugin-clerk
[ ] build/tooling/chore

Description

[x] npm test runs as expected.
[x] npm run build runs as expected.

While updating the Clerk beta docs, I noticed that the EmailAddress resource typings didn't have the documented/implemented create method. This PR aims to rectify this.

I noticed this issue exists on the PhoneNumberResource type as well (missing the create method).
Is this intentional or would it be best advised to add to the PhoneNumberResource type as well?

@crutchcorn thank you for the contribution! ❤

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


Sets default caching TTL to 300 seconds

This PR changes the TTL for caching stats to a more sane default: 300 seconds.
Reasoning behind this:
https://developer.toon.eu/developer-journey

you are allowed to consume 60 calls per minute and 500 calls per day per display

I've been contacted by Quby regarding this issue and they would highly appriciate action around this part.
I'm looking into a solution on the Home Assistant side of things as well, nevertheless, from a implementation / quick win perspective, this is the fastest way to prevent further issues.
Ref: #31

I am questioning if this is needed. the toonapilib has worked with https://github.com/krocat/ToonHA for the last year. without any errors. It still functions without any errors. and the update time was 30 seconds.

@scheric Whatever the issue and how we are addressing is, is something else. This is not a solution, this an aid for the wounds for now.
There is a proper rate-limiting needed since that is what is imposed and part of the policy of Quby (and thus the Developer API). Now more people start using it, it becomes a larger problem at a magnitude at which they are considering banning.
I know you like to keep hold to the custom component as the solution, nevertheless, the fact is, the custom component is capable of hitting a 30 second update period as well. Which is, again, against the policy by Quby.

@costastf Thanks for merging this in, could you tag it so I can up the dependency? I'll try to get it into the beta or a hotfix release (I can at least ask right? 👍 )
#33 In interesting as well and a followup. One of the main issues of hammering is the lib ignoring the 429 response (which is handled as invalid credentials by the lib actually, which is incorrect).

That is actually a valid point. But if quby people reached out and asked
for a bumping of the refresh window, i guess it is only polite to do so. On
the other hand maybe the library should stay with 30 seconds and the
component should have a configurable timeout that starts with 300.
On Mon, Mar 18, 2019 at 3:37 PM scheric<EMAIL_ADDRESS>wrote:

I am questioning if this is needed. the toonapilib has worked with
https://github.com/krocat/ToonHA for the last year. without any errors.
It still functions without any errors. and the update time was 30
seconds.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/costastf/toonapilib/pull/32#issuecomment-473936660,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEbFa-cJ1UHKHS9UbYaBTB_2VDDMc0Qwks5vX6SogaJpZM4b5k1l
.

@frenck v3.2.2 is out with the 300 timoute

Hi @costastf,
We (Quby) did try to contact you directly on the email mentioned here https://toonapilib.readthedocs.io/en/latest/authors.html regarding this topic. Today we tried @frenck and he got back to us right away. Nevertheless, thank you both for taking this up.
Nikolas from Quby

Hi Nikolas, I have not received any email from your, just checked my spam also. Please double check that you actually sent something. You can always reach out directly here.

component should have a configurable timeout that starts with 300

@costastf I absolutely agree with you on that and that is something I'm going to provide. This change, however, which take off the large part of the weight instantly. The change in the component requires a little more effort and time to achieve. I would suggest on reverting this (or maybe even remove the TTL) later on.

:+1:  Some time this week I will also look into implementing the backoff for the 429 code.

I will actually remove the caching and apply a rate limiter with back off since there are concrete values for it now. Some time this week...


Add custom server requests

This mirrors existing custom client requests for the server.

Oh this is a much better idea! Thank you so much


SA1623: Property standard text for ru-RU culture is wrong

For ru-RU culture property standart text should be Возвращает instead of Получает (Возвращает или задает insted of Получает или задает and so on).

@ds1709 The internal check was inconclusive. In my review of their response, I notice that the English version describes the behavior of accessing the property from the caller's perspective, as opposed to describing the behavior from its own perspective. The response made it sound like Получает is a better fit for that. 😕


Books search and books fetching using google books api functionality added

The following changes have been made:
A new search bar has been added to the books section.
A new function has been created to fetch the books from the Google Books API.
The views have been updated to display the search results.
Book cards made responsive and added good enough CSS to display books.
These are the changes done :

Tip : Replace my personal generated API KEY with your api key for better user experience in future.
Please review the changes and let me know if you have any questions.
Thank you for your time.

@M-A-N-I-K Remove your api key and make PR

@rohansx without api key search functionality will not work..you sure i should do that?

@M-A-N-I-K I'll add mine


Enable LTO in release build process

What problem does this PR solve?
Issue Number: ref #4909
Problem Summary:
What is changed and how it works?

new feature flag thinlto in ./tiflash version.
clean useless modules: reduce binary size from 329M to 293M.
inline function invoke automatically.

Check List
Tests

[ ] Unit test
[ ] Integration test
[x] Manual test (add detailed scripts or steps below)
[ ] No code

Test release build under x86 and arm64
docker run --rm -it -m 40G -v `pwd`:/home/jenkins/agent/workspace/tiflash-build-common/tiflash hub.pingcap.net/tiflash/tiflash-llvm-base:amd64 sh
sh-4.2# NPROC=12 /home/jenkins/agent/workspace/tiflash-build-common/tiflash/release-centos7-llvm/scripts/build-release.sh

sh-4.2# ./tiflash version
TiFlash
Release Version: v6.1.0-alpha-186-g7eb89fa-dirty
Edition:         Community
Git Commit Hash: 7eb89fac9cc36b789fb0aaf449ac95e1b5897de2
Git Branch:      lto
UTC Build Time:  2022-05-18 09:43:31
Enable Features: jemalloc avx avx512 unwind thinlto
Profile:         RELWITHDEBINFO
...

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

But I suggest merging this PR after code freeze for 6.1. Leave the uncertainty behind it.

/hold

/merge

/merge


can't get  to work with link object passed to :field attribute?

I'm passing in a Prismic link object to  via :field which looks like this
id:"Xg4BSxAAACQAAj9C" isBroken:false lang:"en-gb" link_type:"Document" slug:"our-products" tags:Array[0] type:"section_overview" uid:"products"
But it's simply creating an anchor tag with no href value in the dom? Am I passing the wrong type of object?

 My link resolver wasn't working for some reason... I've moved into app/prismic instead of specifying path in nuxt.config and all fine now.


PlayerView: Coin amounts (both player and coin pot) not appearing on player views, but appearing on decider.

desktop, chrome. This screen is in the brainstorm phase, but it is true from the beginning of the game.
I am not sure if amounts are begin pulled correctly from the cms. Labels are a bit confusing (since a player does not gain more coins each round automatically...which is how it is labeled in the cms. I left this at zero, which might be why this is happening). If this simply means the coin amount players are STARTING with, that might solve it. Lets investigate the cms setup and make sure it is correctly assigning and pushing coin values. Once I have this confirmed, I will change the values and can test again.
This is the player screen

This is the decider screen

@jpailthorpe So for sure the decider and player do not automatically get a bonus amount of coins as round 2 and 3 start? This was present in the unity version.

I would like to test not getting more coins, and see how it plays out with areal life test through 3 rounds. Reason being, if player's get an auto "refresh" of coins at the start of each round it breaks the rationale for players to want to get more coins. Therefore, if a player uses all these coins and ends up in a situation in which they have less than 50 coins they simply cannot buy more time. This puts them at a disadvantage, yet at the same time, could drive motivation to acquire more coins in the next round. This is from a game design rationale.
In addition, a player would have to realistically not acquire any coins at all AND buy time 3 different times (since every player starts the game with 200 coins) for them to end up in a position where they can no longer buy time. This is absolutely possible, but my gut feeling is it will be rare, and players who do end up in this position might not be utilizing their coins/time strategically.
After testing these assumptions, we can return to this to evaluate. I will make a note of this to add to my ux test questions though!


Chat Functionality

while fetching response if open new chat, then the previous chat response show in new chat
new chat text should be always keep on screen, user don't need to scroll

Prem---Self-Sovereign-AI-Infrastructure.webm

@hardik-rl can you please check this one too


Javascript client don't return title of page

Hi !
I have a problem with the client, I can't get the title of a page :/ despite the configurations. For information, I use a template from Docusaurus v2.
Version of client : 4.17.0
Index settings
attributesForFaceting: [
        "type",
        "lang",
        "language",
        "version",
        "docusaurus_tag",
        "title",
      ],
      attributesToRetrieve: [
        "hierarchy",
        "content",
        "anchor",
        "url",
        "url_without_anchor",
        "type",
        "title",
      ],

Client request options
attributesToRetrieve: [
    'version',
    'url',
    'title'
]

This doesn't seem to be opened in the right repository. Can you open an issue in docusaurus please?

Docusaurus repo ?


Add settings page option for manual override time period

When you manually turn a light on/off, it disables any rule evaluation for a certain period of time, by default maybe 2 hours?  This time should be configurable in the settings page.

Resolved in daf038c5254342e4c1d7d2b029b6a1db9a2d3741.


Updating readme link to events module on npm

The current Gozala module instead of the old shtylman module

:+1:

thanks, merged this in from another duplicate PR.