Share code location

Often it's necessary to share a particular location in the solution (Project > File > Line) with the coleague so I don't have to type the path and he doesn't have to follow it. It would be great if there was a way to do that. E.g. you put cursor on the line of code, press hotkey and the path is copied into the clipboard. You send it as text to anyone who is working on the same project. He has the same solution opened, presses a hotkey, inserts the path and VS opens the requested file on the requested line.
Similar to how bookmarks are saved and navigated through, just with the ability to share them.
Not sure if similar functionality already exists, but I couldn't find it.

Or copy and set "method location",
Copy "current location" like "NameSpeceOfProject.Class1.Method1(int,int):100:20" ("100" is the line in the method, "20" is the position in the row).
And possibility to set this "location".


exec /usr/local/bin/text-generation-launcher: input/output error

System Info
Running on a dual 3090 system with Ubuntu 22.04 and nvidia docker
using:
--model-id meta-llama/Llama-2-7b-chat-hf --huggingface-hub-cache /root/.cache/huggingface/hub  --trust-remote-code --quantize bitsandbytes-nf4 --max-input-length 4095 --max-total-tokens 4096
Information

[X] Docker
[ ] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction
I am using my fork and docker-compose file to run these models.  The only real difference the is the docker-compose file and the scripts I have added to the repo.  I am using the latest docker image and I have never had issues before
https://github.com/mallorbc/text-generation-inference

clone
configure the command to use any llama 7B model
docker-compose up -d

It will give the error
Expected behavior
It has always worked before.  Is the new image broken?

This happens if I run this as well:
docker run -it --gpus all ghcr.io/huggingface/text-generation-inference:latest bash

Another docker image I run frequently does not do this, leading me to think its not the hardware.  At the same time, when used on an A100 system I do not see issues....

I have reinstalled my entire OS and I still get this error

Ok so I think the issue is with docker itself or my disk.  I have for the longest time used docker where the images are located on another disk.  When I moved docker and the images back to the root disk, it seems to be fixed.
When I run fsck, the disk seems fine so idk, maybe a recent update introduced a bug.
Closing this issue then because it seems unrelated to this software.


[No Ticket] Remove unused locales

Purpose
Save developers time when adding or modifying language.
Summary of Changes

Remove unused locales!

Reviewer Checklist

[ ] meets requirements
[ ] easy to understand
[ ] DRY
[ ] testable and includes test(s)
[ ] changes described in CHANGELOG.md

sorry, mea culpa :(


added ids as argument and simplified api

In this PR:

allow filtering based on object IDs in download
added a couple helper functions in the client
swapped out some CLI arguments for simpler to understand options. Also added "library function equivalent" functions in download.api to get equivalent pandas dataframes in another program. To make this as easy as possible, it is also possible now to configure a "global" client, which avoids having to pass this object into every function call.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

56 of 177 (31.64%)  changed or added relevant lines in 14 files are covered.
4 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.8%) to 56.355%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/elabftwcontrol/_logging.py
7
8
87.5%

src/elabftwcontrol/upload/parsers.py
0
1
0.0%

src/elabftwcontrol/upload/planning.py
0
1
0.0%

src/elabftwcontrol/cli.py
0
3
0.0%

src/elabftwcontrol/download/transformers.py
3
12
25.0%

src/elabftwcontrol/client.py
8
24
33.33%

src/elabftwcontrol/global_config.py
14
38
36.84%

src/elabftwcontrol/download/api.py
15
81
18.52%

Files with Coverage Reduction
New Missed Lines
%

src/elabftwcontrol/cli.py
1
0.0%

src/elabftwcontrol/download/api.py
1
30.51%

src/elabftwcontrol/client.py
2
33.61%

Totals

Change from base Build<PHONE_NUMBER>:
-0.8%

Covered Lines:
1942

Relevant Lines:
3446

💛  - Coveralls


cameraVelocity property in /replay/render/ endpoint

ReplayAPI Enhancement Request - As it stands, there is only absolute camera coordinates available as a property. Adding a position/rotation velocity property would allow a lot more combustibility in terms of custom geometry as well as greatly helping live broadcast applications for an esports use case.
An acceleration property might also be useful in order to allow even finer control

After a bit of research, I've found that current in game set ups have the camera able to move relative to it's current rotational frame of reference. Being able to choose between that and a world frame of reference would be great as well (similar to the difference between FPS cam and overhead cam)


Icon for custom action is not showing

I already configure all the things for the custom action
this is the configuration for the icon
register_instance_option :link_icon do
'icon-envelope'
end
but still it is not showing in the admin panel
please help me out this issue

You need to mark it as a member action to make it interact with each record.
https://github.com/sferik/rails_admin/blob/master/lib/rails_admin/config/actions/base.rb#L71-L74


[controlPlane] Failed to bring up Control Plane: [Failed to verify healthcheck: Failed to check https://localhost:6443/healthz for service [kube-apiserver] on host [<IP_ADDRESS>]: Get https://localhost:6443/healthz: dial tcp: lookup localhost on <IP_ADDRESS>:53: no such host, log: I0916 04:57:33.012159 1 log.go:172] http: TLS handshake error from <IP_ADDRESS>:34039: EOF]

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

Result:

Other details that may be helpful:
Environment information

Installation option (single install/HA):

sudo docker run -d --restart=unless-stopped -v /root/rancher:/var/lib/rancher/ -p 80:80 -p 443:443 rancher/rancher:stable

clear all your environment
and
just wait more moment


Pkg: Implement spacing system

Base unit 4px

Include named spacing system with logical steps up from the base unit
Create function
Add helpers
Add documentation on site

Related to #748

Looking at:
https://github.com/SAP/techne/commit/a8524e7b84c41ccb362fc7e9447058463512ddf6#diff-dc446cd5ee41bbcf85b1778fa17c006aR18
$tn-spacing: (
   base: $tn-spacing--base,    //4
   xs: $tn-spacing--base * 2,  //8
   s: $tn-spacing--base * 4,   //12
   reg: $tn-spacing--base * 6, //24 (should match bottom margin default)
   m: $tn-spacing--base * 8,   //32
   l: $tn-spacing--base * 10,  //40
   xl: $tn-spacing--base * 25, //100
   xxl: $tn-spacing--base * 37 //148
   ) !default;

I recommend avoiding using literal identifiers for the spacing. The spacing system is about proportions: a < 2a < 3a, 4a > 2a, etc.
The developers cannot calculate the proportions using a literal system, they would have to learn then by memory, not by logic.

The system used on the design is simpler and logical, illustrated bellow

I will recommend using only one notation: a, 2a, 3a, 4a, etc for the spacing.

Having 2 systems available, means the components will get developed, ones with one system other with the other, in the worst case a mix of both. That is not consistent and the developer have to remember what each system is.

When using the same notation as the design, we don't have to "map" one system to the other, they just correspond exactly: 2a = tn-space(2)

Only one spacing notation, less margin for error.
This can be simpler and effective.

The function does support number arguments that multiply as you describe.  That, however, should be the secondary use case. The design system should adhere to a modular scale that will not always be simple increments of 4, esp. when you get to large numbers like 100. Therefore the "keywords" should be used more often than the numbers.
For example, the margins under block elements like p, table, ul will be 24px. Margins under form groups may be 40px, with margins under sections being 100px. These are fundamental measurements that will be repeated over and over again and bring a vertical rhythm to the site. You will see these core measurements regularly in the design so they are called out for that reason. These are more appropriate for defining how containers stack on top of each other. For the input example you show, the number arg is supported if fine-tuning needs to happen.
Having a controlled spacing hierarchy is as important as a controlled font hierarchy. We want to encourage common sizes and discourage arbitrary measurements. Again, the keywords are preferred, the numbers are available as a backup knowing that there will be cases where other dimensions are needed. There's no mapping needed, but yes, the devs may need a cheatsheet until they get used to it, just as they will need one for the fonts and colors.
Does that make sense? This system will continue to evolve so we can see how it works as it is applied.

@xak
I understand the need to have common spacing, but that's shouldn't increase the learning curve.
PLEASE NOTE: The spacing is given from the Visual Design, been able to map 1:1 that with the development is an increase in productivity.
Again:
2a = tn-space(2)
3a = tn-space(3)
4a = tn-space(4)

it is easier than
2a = tn-space(xs)
3a = tn-space(3) // there is not 3a literal equivalent
4a = tn-space(s)

That, right there exemplify the problems with the literal identifiers, very clearly.
This is contradictory on https://github.com/SAP/techne/issues/816#issuecomment-315369486

There's no mapping needed, but yes, the devs may need a cheatsheet until they get used to it.

If the developer need a cheatsheet, that's mapping right there. The design proportions notation are not mapped directly to the literal identifiers.
Other point
There is no difference using always tn-space(6) all the time than tn-space(reg).
Even more problematic the developer have to remember that p, table, ul have tn-space(reg)

Thanks for your input, we'll see how it goes. I've had good luck with this set up in the past. If it proves burdensome, the number system is already in place to there's no real problem.
Again, the design should be using common sizes — 8, 12, 24, 40 — not random increments of 4. The same as the fonts, a dev should NEVER hardcode a font-size, they should ALWAYS use the approved sizes. This is the same, cementing the common sizes in code to encourage their use. The team will soon enough learn the hierarchy just as they will the fonts simply by looking at them.
It'll work, trust me.

@xak
Your solution is not considering the pitfalls that I mentioned on https://github.com/SAP/techne/issues/816#issuecomment-315376728
They are real and they need to be addressed.
There is no valid technical reason for using literal spacing names, your arguments doesn't give a logic rationale behind.
This issues needs to be addressed before we need to change it all again like happened with the colors - https://github.com/SAP/techne/issues/765
A technical solution/architecture must be proved in a POC before integrating in into the library. Otherwise it just stays there and we have to drag the wrong solution because it is too late to change it.
Please review this again.

I haven't had a chance to talk with @LeoT7508 yet about this. It's really more his call so I'll get back to you.

I spoke with @LeoT7508 about this and he is fine with this notation esp. when we defining spacing for containers stacking on top of each other. That is the best use case ... labels have base bottom margins, cards will have reg between them and under them, sections will have xl bottom margins, etc. The hierarchy establishes a vertical rhythm.
An established hierarchy with easy-to-use names will also be easier for theming. If every spacing unit is defined in relation to a base unit, then the only modification possible is to change the base unit and the spacing will respond radically. The alternative then is to change each instance. If bottom margins are coded as tn-space(5) in multiple places and then a team decides they prefer a theme with a 24px bottom margin, each instance of the tn-space(5) must be modified to (6).
With a map of named spacing units, they can fine tune their theme more easily by changing the map itself.
$tn-spacing: (
    base: $tn-spacing--base,    //4
    xs: $tn-spacing--base * 2,  //8
    s: $tn-spacing--base * 3,   //12
    reg: $tn-spacing--base * 6, //24 (custom value)
    m: $tn-spacing--base * 8,   //40
    l: $tn-spacing--base * 10,  //100
    xl: $tn-spacing--base * 25, //148
);

If they want a 7px base unit, no problem. They can do that and still have better control over the end values ...
//spacing
$tn-spacing--base: 7px;
$tn-spacing: (
    base: $tn-spacing--base,    //7
    xs: $tn-spacing--base * 2,  //14
    s: $tn-spacing--base * 3,   //21
    reg: $tn-spacing--base * 4, //28
    m: $tn-spacing--base * 6,   //42
    l: $tn-spacing--base * 12,  //84
    xl: $tn-spacing--base * 20, //140
);

If we implement only a multiplication system (3a), the above change would require a huge number of overrides in the CSS for theming. We cannot guarantee that a theme will always use 5a for the reg spacing. The Techne default will use 5a, the examples above use 6a and 4a respectively.
Again, the spacing keywords are the preferred method and the design will reflect that. I see no problem with the overlap of tn-space(5) and tn-space(reg). I can reference colors in CSS with all of these methods ...
color: red;
color: #00ff00;
color: rgb(0,0,255);

The named colors are a convenience just like ours here. There are only 140 colors that are named but I still have access to any rgb, hsl, or hex value I desire.
I cannot promise this will not change — the design system is still evolving — but it is manageable.

The only modification intended, tested and documented on the spacing feature is: only have to change the base value as described on https://github.com/SAP/techne/wiki/Spacing-Design-specs
Everything else, right now, is out of scope. And only waste development resources.
This feature's implementation as it is now: is focused on using the spacing feature from only SCSS.
That doesn't take in count: the main use of the library is CSS. CSS usage is the main focus focus of the library.
The visual rhythm is already implicit in the documented solution.
With all the above,
this solution is only worth the while when the following criteria has been meet:

The spacing feature with the 2 modes as described on: https://github.com/SAP/techne/wiki/Spacing-Design-specs have a CSS solution, not only SCSS.
The visuals, which is the design step from where the final spacing comes out: are provided in the proposed literal notation.
The components developed have a healthy 80-20 ratio: 80% literal values, 20% numeral values.

@xak
Can we agree to the above mentioned 3 points as acceptance criteria?

I don't know what you mean by your last two comments but I'll create a --compact version of each component even though it will be confusing.
That presents two types of Cards (and other components) to choose from — regular and compact — so we may see a mix of the two when teams implement based on nothing more than a personal preference. That will not be a correct use of the compact library when the true use case is more of a theming concern. This also means that teams using compact globally will need to add an extra class to every component they use.

New issue created to address the card itself. This can be closed.

@joseegm @xak
I agree with Zack on this one, we should make the theming global, not per component or from a visual side things will look broken if they have different components with different spacing on one page.

The feature is designed to be uses, per screen and details had been posted already on: https://github.com/SAP/techne/issues/827#issuecomment-316636672
Posting it here again for context:
The Spacing feature will be used mainly by CSS, not using the SCSS.
All components have to be implemented to adapt to a spacing class that can be present in a higher element in the DOM tree. The spacing options can be set at any level on the component hierarchy and the components bellow that will use the compact space.
The solution was first described at the request https://github.com/SAP/techne/issues/748
Was also designed at https://github.com/SAP/techne/issues/763, https://github.com/SAP/techne/issues/751
Example:

That's is already implemented on
https://techne-alpha.firebaseapp.com/components/structure/spacing

@xak The points you don't understand:

The final visuals will be represented using a similar notation as:

80% of the spacing should be set using the literal values and not the numerical ones.

I'll put this back to "in process". The feature implementation doesn't work as designed.

Feedback from the Backoffice developer indicates that the solution as design is sufficient for an MVP. No further changes are needed at this time. As we gather feedback from alpha and beta releases we can revisit this.
It is more important to focus on the default Techne theme now. Advanced features can be discussed holistically at a later time.


配置对外IP怎么配置？

我自己搭建的一个Druid集群，有5个节点，你这个exporter是每个节点都需要安装么？安装完成后，对外的metrics是默认可以通过各个IP/metrics访问么？ 我个人只了解一点点Python，问题要是很蠢的话，请不要笑话，谢谢

I tried to translate into english (assuming that your comment is not spam and this is a real user), and as far as I got you are asking if this exporter needs to be deployed/installed on all nodes and if it provides metrics for each node. Yes to both questions :)


Show failed task collection logs

In tasks page, the client shall present the logs of a failed task collection to the use.

This would be resolved by 21ec492


Connecting to Postgresql with SSL

Hi,
Does Cadence support connecting to PostgreSQL using SSL? Can you please help me point to the documentation for required config options?
Looks like there is no support:
https://github.com/uber/cadence/blob/master/common/persistence/sql/sqlplugin/postgres/plugin.go
var errTLSNotImplemented = errors.New("tls for postgres has not been implemented")
Thanks,
Ram

Hi @rphadake ,it's completed in https://github.com/uber/cadence/pull/3488
thanks to @nadilas
You can try it out in the master docker image.
@meiliang86 I think we will include that in next minor release.


Re-export mlir_sys

To close https://github.com/raviqqe/melior/issues/583

@edg-l Do you think this is a good change?
I feel like this is the responsibility of the package manager (i.e. Cargo.) But at the same time, I've never hit problems @cospectrum described in the original issue.

I dont have an opinion on this, but with cargo you can have multiple versions with package renaming

@edg-l Yeah, that's what I've suggested in #583 too.
@cospectrum How critical is this change to your workflow?

Well, that's quite important for me. Every time I run cargo update, for example, the workspace has a chance to not compile anymore, and cargo will not prevent it. To fix the compile error, I should find melior version in Cargo.lock, then find out exact version of its mlir_sys somewhere (crates.io or lock file), and then I should downgrade my own mlir_sys in Cargo.toml manually.

I think PR will not cause problems at all. But it will solve one problem for me!


[WIP] Verification tests deployment test

Superseded by https://github.com/chipsalliance/Cores-VeeR-EL2/pull/78


No longer works with Manifest V3

I've noticed a sharp decrease in usability recently on this, and tried to switch it to manifest v3. Now the js has stopped working altogether. Any solutions / suggestions? It looks like you can no longer run persistent js on chrome now.

I've got it mostly working on Manifest V3: https://github.com/cspotcode/spaces
It's ported to a service worker as dictated by manifest v3.  Not sure if the service worker going idle and terminating will break things.  As far as I can tell, browsers restart the service worker on demand whenever you open the extension's popup or interact with it.
I was gonna teach the extension to understand tab groups, but I dunno if I'll have time for that; Edge has this mostly built-in with Workspaces, so I might just use those.

I'm a developer of TabXpert extension. It has a similar session tracking approach as in Spaces, no wonder, because Spaces encouraged me to write TabXpert. I recently implemented an import from Spaces. Also, TabXpert MV3 compatible and supports Chrome tab groups, sync with Cloud and many other features.

@cspotcode
Thanks for your porting, Sincerely.
I had been concerned until I saw your post.

@cspotcode
I try to use your repo, and I found some bugs about active space name in race condition and can't trig spaces-switch
to specific closed sessions correctly.
(repo: https://github.com/alieonsido/spaces/tree/ManivestV3Checker , newest branch is ManifestV3checker)
So that, I had rewritten some functions by LLM and sent pull request.
However, I'm not a front-end engineer, although I fixed wrong spaces-switch cases, but active space ID in race condition bug haven't been fixed, maybe added some redundant & dirty codes.
More details are in pull request script.
Sincerely, thanks for your work & help, your work helps me a lot.
I really love Spaces Project.

I've stopped working on Spaces because I discovered Edge Workspaces. They do everything I need and are built-in to the browser, and I haven't hit the occasional syncing and tab duplication bugs that seem inevitable with extension-based solutions.  Though to be fair, I never tried any paid options.
https://www.youtube.com/watch?v=1kJ59DzjNOU
Anyway, I'm posting this because I don't want anyone to waste time sending me pull requests, but feel free to take my code for your own fork.


BlueOcean support?

What feature do you want to see added?
Is there a chance to get the CSS being inlined when using the BlueOcean interface?
Upstream changes
No response

This was already requested here and here, but since BlueOcean is - for all intents and purposes - dead, I will not work on this feature myself. Pull Requests implementing this, are - of course - welcome, as long as it won't affect current users (read: there needs to be some kind of opt-in, for most flexibility on the ThemeElement level, but a global toggle would also work)


why  exp is special?

The code as blow:
   if json then
      local expiry_claim = opts.expiry_claim or "expires_in"
      local ttl = json[expiry_claim]
      if expiry_claim ~= "exp" then --https://tools.ietf.org/html/rfc7662#section-2.2
        ttl = ttl - ngx.time()
      end
      openidc_cache_set("introspection", access_token, cjson.encode(json), ttl)
    end

I check https://tools.ietf.org/html/rfc7662#section-2.2 link, that's as blow:
exp
      OPTIONAL.  Integer timestamp, measured in the number of seconds
      since January 1 1970 UTC, indicating when this token will expire,
      as defined in JWT [RFC7519].

The exp is integer timestamp, isn't seconds from now.

the code actually sets the ttl to the absolute timestamp first; if then the claim name is not equal to exp then it is assumed that this is a non-standard type of introspection and it will assume that the value is a relative timestamp; so exp is not the special case, it is all other values

Thank you for you reply.  I'm English is not good. I'm confused that 'exp' is standard and it is a  number of seconds since January 1 1970 UTC. Not a absolute timestamp.  keycloak is corrected. So I think if 'expiry_claim' == 'exp' need execute 'ttl = ttl - ngx.time()'.
Sent from my OnePlus ONEPLUS A3010 using FastHub

The number of seconds since January 1 1970 is considered to be an absolute timestamp. A relative timestamp would be measured in seconds e.g. 600 seconds "from now". That's the spec, that's how lua-resty-openidc does it and I believe that's what Keycloak does as well reading: http://www.keycloak.org/docs/2.4/authorization_services_guide/topics/service/protection/token-introspection.html (search for "exp").
If you want different behavior you'll need to run a customized version of lua-resty-openidc.

Thanks, I understand.
Sent from my OnePlus ONEPLUS A3010 using FastHub

After re-reading the code I think it is the wrong way around...

I change it and run locally, code as below:
-- cache the results
    if json then
      if json.active then
        local expiry_claim = opts.expiry_claim or "expires_in"
        local ttl = json[expiry_claim]
--        if expiry_claim ~= "exp" then --https://tools.ietf.org/html/rfc7662#section-2.2
          ttl = ttl - ngx.time()
--        end
        openidc_cache_set("introspection", access_token, cjson.encode(json), ttl)
      else
        err = "invalid token"
      end
    end

It's looks fine.


make text and code box fixed height and allow scrolling

again this is just an idea.
The idea is that both the left text and the right text box have a fixed height, so that the reader can always see the navigation menu at the bottom.
Moreover decoupling the two views allows to scroll in the code box without the text moving with your scrolling.
We could map the common navigation key (key {up,down}, page {up, down}) to currently focussed box and by default to left text box.

Actually I already had that in the initial version because I found it to be nice in the golang tour. But it effectively reduced the visual space for the content on all screens. And it had issues on mobile. And some users complained about it. And removing made the dlang-tour not look like a golang clone :-)
I'll leave the issue open for now so if everyone wants that we might think about it again.

As a follow up: personally I would prefer this design. I just didn't get it right. Maybe we could give this one another try when Bootstrap #255 has been removed. They only main complaints with the first version was that too much space was "wasted" especially on mobile. So if the footer is kept quite small this should be the way to go. Also a very possible fix for #187 because the user directly knows what to do knows using this visual aid.


Periodically update node spec

Docker doesn't detect resource (CPU/Mem) change on the host so it cannot inform Swarm of such change. Swarm needs to updates node spec with some interval. It also updates node labels.
Fix #1281, #1949.
Signed-off-by: Dong Chen<EMAIL_ADDRESS>

LGTM

Is a resource change something there should be an event for? This isn't the container changing cpussets, its the host cpusets changing, right?

@MHBauer Yes it's the host resource (CPU/Mem) or label change. Ideally Docker should emit events for change. But Docker doesn't track CPU/Mem in manifest. So it doesn't know there is a change.

LGTM, label update is also included which will help me a lot.

LGTM


refactor: generic input for util::migrate

Commenting so that some other PRs are the least recently updated. They are used as test fixtures.


borzzzenko_api_methods

Codecov Report

Merging #15 into master will increase coverage by 7.02%.
The diff coverage is 46.87%.

@@             Coverage Diff              @@
##             master      #15      +/-   ##
============================================
+ Coverage     41.42%   48.45%   +7.02%
- Complexity       10       18       +8
============================================
  Files             8        8
  Lines            70       97      +27
  Branches          1        6       +5
============================================
+ Hits             29       47      +18
- Misses           41       50       +9

Impacted Files
Coverage Δ
Complexity Δ

...m/team3/collections/Controllers/ApiController.java
5.00% <0.00%> (-9.29%)
1.00 <0.00> (ø)

.../collections/Database/CollectionsDataOperator.java
100.00% <100.00%> (+44.44%)
9.00 <7.00> (+8.00)

Continue to review full report at Codecov.

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


feat(IAM Identity): Add support for IAM enterprise

PR summary
Added support for IAM enterprise feature. Enterprise admins can create templates for account settings and trusted profiles and assign them to accounts and account groups in the enterprise.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

Current vs new behavior
SDK adopters will be able to work with IAM enterprise feature.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
API definition: Staging
Test information:
Integration Tests:
Ran 490 of 490 Specs in 814.586 seconds
SUCCESS! -- 490 Passed | 0 Failed | 0 Pending | 0 Skipped
PASS
ok      github.com/IBM/platform-services-go-sdk/iamidentityv1   817.700s

Examples:
Ran 473 of 473 Specs in 759.903 seconds
SUCCESS! -- 473 Passed | 0 Failed | 0 Pending | 0 Skipped
PASS
ok      github.com/IBM/platform-services-go-sdk/iamidentityv1   763.477s

@michaelbeck Looks like your PR build had some linter errors. Based on the messages, I'm guessing that you defined the same function in both the integration tests and examples code.  Unfortunately, when we run the linter we use --build-tags=integration,examples which means all the code inside the service package directory will be "seen" by the linter and since all the code is part of the same package, the linter detects that the functions are actually defined twice in the two different source files.  I guess the easiest solution is to rename the functions in one of the files so they're unique.

BTW, each SDK has some detailed developer instructions in the update_service.md file (linked from CONTRIBUTING.md), but specifically... you can run make all in your local sandbox and it will run the unit test and linter steps, similar to an automated Travis run.  That way you can head off errors like this before pushing your changes to remote, etc.

BTW, each SDK has some detailed developer instructions in the update_service.md file (linked from CONTRIBUTING.md), but specifically... you can run make all in your local sandbox and it will run the unit test and linter steps, similar to an automated Travis run.

I tried to respect what was written in the readmes there. But neither go test nor go test ./...seemed to run any meaningful test, so I ignored them. Sorry :-(

:tada: This PR is included in version 0.42.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Unexpected handler pages/api/[collection]/lib/worker.js in next-payload

Link to reproduction
https://github.com/Stupidism/sunmer/tree/feature/add-payload-integration
Describe the Bug
Looks like payload has some new handlers, but next-payload doesn't support?
error uncaughtException: Error: Cannot find module '/Users/sun/Documents/personal/sunmer/apps/steps/.next/server/pages/api/[collection]/lib/worker.js'

Repro repo: https://github.com/Stupidism/sunmer/tree/feature/add-payload-integration
To Reproduce
git clone git@github.com:Stupidism/sunmer.git
git checkout feature/add-payload-integration
pnpm install
nx run steps:serve
open http://localhost:4200/admin

Payload Version
2.1.1
Adapters and Plugins
none

Hey @Stupidism I think this is a separate issue.
In my experience, whenever I see "worker.js"-related issues, it's an internal Nextjs thing that is happening as a result of your code crashing for other reasons.
I see that this.config.db is not a function - could it be related to that? Regardless, we will move this issue to the next-payload repo and then see what we can do.


tabula-java: Init at 1.04

Motivation for this change
Add the CLI tool for tabula, tabula-java.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I was confused because I'm already in the maintainers file here, but I used the wrong name. I'll fix that.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110905 run on x86_64-darwin 1

1 package built:

tabula-java


Fix Control resizing wrongly after "change type" in editor

Fixes #78779
This bug has been bothering me for a while now, so I decided to tackle it once and for all. Yes it's a hacky fix, but since nobody has stepped up with a proper solution so far, I'd argue this is better than keeping this bug around for even longer.
Similar (six year old!) hacks are also found right below where I did my change, for making other node types work properly with this editor feature. So maybe this method is flawed in some way?
Note that this does NOT fix the undo issues that come with this (so undoing a "change type" may still cause the control to end up wrongly sized). There are similar issues with the editor showing outdated sizes, or undo/redo not reverting this, so it's likely a more fundamental issue at play.
Background
See this very helpful comment for some details.
The PR #78009 made it so that Control updates its size cache in a lot more places, including when entering the tree (NOTIFICATION_THEME_CHANGED).
As I understand it, the editor's "replace by" feature (used by "change type") works by first calling Node::replace_by and then re-applying same-named properties. However, since replace_by also adds the node to the scene tree, the size is recomputed through _size_changed().
I'm not exactly sure why it ends up using the parent viewport's size in this case, instead of the project settings size, but this is the best fix I could come up with.
Tried changing Control::get_parent_anchorable_rect logic a bunch, but it doesn't seem to be the culprit from my understanding. Don't want to remove/delay the problematic _size_changed() call since it seems to have fixed a bunch of bugs.

This is funny.
https://github.com/godotengine/godot/blob/1d47561319938e10cb53d202ceaeca102511a31e/scene/gui/control.cpp#L663-L667
The condition fails, because edited_scene_root is null, because it's currently being replaced by another node. A fix could be setting the properties after the root was set as edited scene, but not sure how doable is that and it's probably not safe.


Some  awesome notifications warnings when running the app

Platform
Android 9
Version
0.9.0
Flutter SDK
3.16.5
Steps to reproduce
When the program is run, I encounter the warnings that I posted in the log, but this does not make the program not run or work.
Code Sample
No response
Logs
Pub\Cache\hosted\pub.dev\awesome_notifications-0.9.0\android\src\main\java\me\carda\awesome_notifications\AwesomeNotificationsPlugin.java:22: warning: [deprecation] Registrar in PluginRegistry has been deprecated
import io.flutter.plugin.common.PluginRegistry.Registrar;
                                              ^
Pub\Cache\hosted\pub.dev\awesome_notifications-0.9.0\android\src\main\java\me\carda\awesome_notifications\FlutterBitmapUtils.java:12: warning: [deprecation] FlutterMain in io.flutter.view has been deprecated
import io.flutter.view.FlutterMain;
                      ^
Pub\Cache\hosted\pub.dev\awesome_notifications-0.9.0\android\src\main\java\me\carda\awesome_notifications\AwesomeNotificationsPlugin.java:112: warning: [deprecation] Registrar in PluginRegistry has been deprecated
    public static void registerWith(Registrar registrar) {
                                    ^
Pub\Cache\hosted\pub.dev\awesome_notifications-0.9.0\android\src\main\java\me\carda\awesome_notifications\DartBackgroundExecutor.java:74: warning: [deprecation] PluginRegistrantCallback in PluginRegistry has been deprecated
    private static io.flutter.plugin.common.PluginRegistry.PluginRegistrantCallback
                                                          ^
Pub\Cache\hosted\pub.dev\awesome_notifications-0.9.0\android\src\main\java\me\carda\awesome_notifications\DartBackgroundExecutor.java:86: warning: [deprecation] PluginRegistrantCallback in PluginRegistry has been deprecated
            io.flutter.plugin.common.PluginRegistry.PluginRegistrantCallback callback) {
                                                   ^
Pub\Cache\hosted\pub.dev\awesome_notifications-0.9.0\android\src\main\java\me\carda\awesome_notifications\FlutterBitmapUtils.java:46: warning: [deprecation] FlutterMain in io.flutter.view has been deprecated
                String assetLookupKey = FlutterMain.getLookupKeyForAsset(bitmapPath);

Flutter Doctor
[√] Flutter (Channel stable, 3.16.5, on Microsoft Windows [Version 10.0.19045.3803], locale en-US)
    • Flutter version 3.16.5 on channel stable at D:\sdk\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 78666c8dc5 (3 weeks ago), 2023-12-19 16:14:14 -0800
    • Engine revision 3f3e560236
    • Dart version 3.2.3
    • DevTools version 2.28.4

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at D:\sdk\android
    • Platform android-34, build-tools 34.0.0
    • Java binary at: C:\Program Files\Android\Android Studio1\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[!] Visual Studio - develop Windows apps (Visual Studio Enterprise 2022 17.2.6)
    • Visual Studio at D:\Microsoft Visual Studio\Enterprise
    • Visual Studio Enterprise 2022 version 17.2.32630.192
    X Visual Studio is missing necessary components. Please re-run the Visual Studio installer for the "Desktop development with C++" workload, and include these components:
        MSVC v142 - VS 2019 C++ x64/x86 build tools
         - If there are multiple build tool versions available, install the latest
        C++ CMake tools for Windows
        Windows 10 SDK

[√] Android Studio (version 2021.2)
    • Android Studio at D:\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)

[√] Android Studio (version 2022.2)
    • Android Studio at C:\Program Files\Android\Android Studio1
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)

[√] Connected device (6 available)
    • WAS LX1A (mobile) • 2XJDU17C14002494 • android-arm64  • Android 8.0.0 (API 26)
    • SM G960N (mobile) • emulator-5554    • android-x64    • Android 9 (API 28)
    • Windows (desktop) • windows          • windows-x64    • Microsoft Windows [Version 10.0.19045.3803]
    • Chrome (web)      • chrome           • web-javascript • Google Chrome 120.0.6099.199
    • Edge (web)        • edge             • web-javascript • Microsoft Edge 120.0.2210.121

[√] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.
PS D:\workspace\flutter\zireh> flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.16.5, on Microsoft Windows [Version 10.0.19045.3803], locale en-US)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[√] Chrome - develop for the web
[!] Visual Studio - develop Windows apps (Visual Studio Enterprise 2022 17.2.6)
    X Visual Studio is missing necessary components. Please re-run the Visual Studio installer for the "Desktop development with C++" workload, and include these components:
        MSVC v142 - VS 2019 C++ x64/x86 build tools
         - If there are multiple build tool versions available, install the latest
        C++ CMake tools for Windows
        Windows 10 SDK
[√] Android Studio (version 2021.2)
[√] Android Studio (version 2022.2)
[√] Connected device (6 available)
[√] Network resources

Problem still persists.
https://api.flutter.dev/javadoc/io/flutter/plugin/common/PluginRegistry.PluginRegistrantCallback.html
awesome_notifications 0.10.0
Flutter 3.24.3
/home/tommy/.pub-cache/hosted/pub.dev/awesome_notifications-0.10.0/android/src/main/java/me/carda/awesome_notifications/DartBackgroundExecutor.java:74: warning: [deprecation] PluginRegistrantCallback in PluginRegistry has been deprecated
    private static io.flutter.plugin.common.PluginRegistry.PluginRegistrantCallback
                                                          ^
/home/tommy/.pub-cache/hosted/pub.dev/awesome_notifications-0.10.0/android/src/main/java/me/carda/awesome_notifications/DartBackgroundExecutor.java:86: warning: [deprecation] PluginRegistrantCallback in PluginRegistry has been deprecated
            io.flutter.plugin.common.PluginRegistry.PluginRegistrantCallback callback) {
                                                   ^


Does express router supports matrix patameters like jax-rs framework.

Hello Express Team,
Does express router supports parasing matrix patameters like jax-rs framework.
example :
/books/2011;author=mkyong
here authoer-mkyong is called as matrix parameter.
Can I natively parsed this in Express.js
Jax-rs framerork url
http://www.mkyong.com/webservices/jax-rs/jax-rs-matrixparam-example/

@kevinburkeshyp
@davidyang

What's the problem with using the standard /books/2011?author=mkyong?

Express does not support that format. For queries about Express, join us at https://gitter.im/expressjs/express.

Using matrix parameters is more suited to our Use Case.
No problem is using /books/2011?author=mkyong?
From: Hage Yaapa<EMAIL_ADDRESS>Sent: Wednesday, May 25, 2016 5:48 PM
To: expressjs/expressjs.com
Cc: Atul Shimpi; Author
Subject: Re: [expressjs/expressjs.com] Does express router supports matrix patameters like jax-rs framework. (#648)
What's the problem with using the standard /books/2011?author=mkyong?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHubhttps://github.com/expressjs/expressjs.com/issues/648#issuecomment-221556402
DISCLAIMER
This e-mail may contain privileged and confidential information which is the property of Persistent Systems Ltd. It is intended only for the use of the individual or entity to which it is addressed. If you are not the intended recipient, you are not authorized to read, retain, copy, print, distribute or use this message. If you have received this communication in error, please notify the sender and delete all copies of this message. Persistent Systems Ltd. does not accept any liability for virus infected mails.

You might find http://expressjs.com/en/4x/api.html#req.params useful.

Hi @atulatgithub, the support for matrix parameters in Express routing was only ever brought up one time before in https://github.com/expressjs/express/issues/2532 and it ended with the user never making a pull request to even get an implementation going (see https://github.com/expressjs/express/issues/2532#issuecomment-73341441). You're welcome to start one!

@dougwilson
I recently had to use matrix URIs at my internship. Seeing that there wasn't much support out there for Matrix URIs in Node, I've developed an npm module (matrix-parser) for this purpose.
Its fully functional (although not optimized, but I'm working on it). Since I'm not so experienced with JS, I'd appreciate if you could have a look at the module and give me feedback.
I've written lots of tests to cover side-cases, you can see their descriptions to understand what rules my module follows for parsing MU.
If you think that this is something you could use, please let me know. I'd like contribute to express as well :)
Here's the repo

Hello Rahav,
Thanks for the efforst u have taken to implement this.
Will see in coming weekend. Right now I am really tied.
From: Raghav Dua<EMAIL_ADDRESS>Sent: Saturday, June 18, 2016 4:05 PM
To: expressjs/expressjs.com
Cc: Atul Shimpi; Mention
Subject: Re: [expressjs/expressjs.com] Does express router supports matrix patameters like jax-rs framework. (#648)
@dougwilsonhttps://github.com/dougwilson
I recently had to use matrix URIs at my internship. Seeing that there wasn't much support out there for Matrix URIs in Node, I've developed an npm module (matrix-parser) for this purpose.
Its fully functional (although not optimized, but I'm working on it). Since I'm not so experienced with JS, I'd appreciate if you could have a look at the module and give me feedback.
I've written lots of tests to cover side-cases, you can see their descriptions to understand what rules my module follows for parsing MU.
If you think that this is something you could use, please let me know. I'd like contribute to express as well :)
Here's the repohttps://github.com/duaraghav8/matrix-parser
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/expressjs/expressjs.com/issues/648#issuecomment-226934666, or mute the threadhttps://github.com/notifications/unsubscribe/AHVNnfl5XsnpqEfDc94y83HsEfSh-Macks5qM8nDgaJpZM4ImeBa.
DISCLAIMER
This e-mail may contain privileged and confidential information which is the property of Persistent Systems Ltd. It is intended only for the use of the individual or entity to which it is addressed. If you are not the intended recipient, you are not authorized to read, retain, copy, print, distribute or use this message. If you have received this communication in error, please notify the sender and delete all copies of this message. Persistent Systems Ltd. does not accept any liability for virus infected mails.

Hi Atul,
Sure, let me know what you think:)
On Sat, Jun 18, 2016 at 4:50 PM, Atul Shimpi, Pune, India <
<EMAIL_ADDRESS>wrote:

Hello Rahav,
Thanks for the efforst u have taken to implement this.
Will see in coming weekend. Right now I am really tied.
From: Raghav Dua<EMAIL_ADDRESS>Sent: Saturday, June 18, 2016 4:05 PM
To: expressjs/expressjs.com
Cc: Atul Shimpi; Mention
Subject: Re: [expressjs/expressjs.com] Does express router supports
matrix patameters like jax-rs framework. (#648)
@dougwilsonhttps://github.com/dougwilson
I recently had to use matrix URIs at my internship. Seeing that there
wasn't much support out there for Matrix URIs in Node, I've developed an
npm module (matrix-parser) for this purpose.
Its fully functional (although not optimized, but I'm working on it).
Since I'm not so experienced with JS, I'd appreciate if you could have a
look at the module and give me feedback.
I've written lots of tests to cover side-cases, you can see their
descriptions to understand what rules my module follows for parsing MU.
If you think that this is something you could use, please let me know. I'd
like contribute to express as well :)
Here's the repohttps://github.com/duaraghav8/matrix-parser
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<
https://github.com/expressjs/expressjs.com/issues/648#issuecomment-226934666>,
or mute the thread<
https://github.com/notifications/unsubscribe/AHVNnfl5XsnpqEfDc94y83HsEfSh-Macks5qM8nDgaJpZM4ImeBa>.
DISCLAIMER
This e-mail may contain privileged and confidential information which is
the property of Persistent Systems Ltd. It is intended only for the use of
the individual or entity to which it is addressed. If you are not the
intended recipient, you are not authorized to read, retain, copy, print,
distribute or use this message. If you have received this communication in
error, please notify the sender and delete all copies of this message.
Persistent Systems Ltd. does not accept any liability for virus infected
mails.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/expressjs/expressjs.com/issues/648#issuecomment-226936446,
or mute the thread
https://github.com/notifications/unsubscribe/AMKtCpXBD4YRR8TAr1p5osXUwdRe5xrhks5qM9RwgaJpZM4ImeBa
.

--
Raghav Dua
+91-9818245320, +91-9627003616
http://www.projectbigoh.com/ http://www.projectbigoh.com/

https://in.linkedin.com/in/raghavdua https://github.com/duaraghav8 **
https://www.twitter.com/dotFoxInc*


Add only strong parent to RootBlock

[ ] Add only strong parent to RootBlock (Currently iterate through all parents)
[ ] Evaluate if adding RootBlock in BlockAccepted will lock acceptance

file: pkg/protocol/engine/engine.go

Add only strong parent to RootBlock (Currently iterate through all parents)

I think as long as we solidify and propagate weight through all parent types, root blocks should contain all types of parents as it is implemented now.


add decimal coordinate for each country

I've added the decimal latitude and longitude of each country so it can be used as data source for http://jvectormap.com/

Looks like you used camelcase for this,  could you please use Ruby style snake case.

My bad, renamed.

@jabawack81 we just did a reasonable amount of restructuring data,  any chance you could move this over to the new data format? Each country has a lib/data/countries/US.yaml one file per country

Here is a rake task I used on a few PR's
task :pull_new_keys_from_old_file do
  require 'yaml'
  countries = YAML.load_file File.join(File.dirname(__FILE__),  'countries.yaml')
  country_list = []
  countries.each do |cc, _data|
    keys = [ 'min_longitude',
            'min_latitude',
            'max_longitude',
            'max_latitude'
           ]
    country_data = YAML.load_file File.join(File.dirname(__FILE__), 'lib', 'data','countries', "#{cc}.yaml")
    keys.each do |key|
      country_data[cc][key] = countries[cc][key]
    end
    File.open(File.join(File.dirname(__FILE__), 'lib', 'data', 'countries', "#{cc}.yaml"), 'w') do |h|
      h.write(country_data.to_yaml)
    end
  end
end

Just put the keys you want to pull in the keys array, and have your old countries file at the root of the rep

thi PR can be deleted I've made a new one with the new structure.
Jaba


How to remove chart outline?

Hi guys, is there any way to remove the gray chart outline on a chart?
My chart actually looks like this:

And I want it to looks like that:

@tgeselle Have you resolved this issue?


Clarifications for "Training your own Object Detector"

Regarding: https://github.com/justadudewhohacks/tfjs-tiny-yolov2#training-your-own-object-detector
Hi! Thanks for putting this together. I wanted to see if I was able to train a custom model given your existing instructions but I was hoping you might be able to update the documentation with some more explicit instructions if they were readily available.
For example, is the /train folder supposed to contain scripts to help with the training process? I see npm start references a server.js which doesn't seem to be checked in.
Ran into some similar stumbling points trying to reproduce the face training example -- e.g. train.html in that folder references scripts that aren't defined or in the location that is specified as far as I could tell.
I'd love to have some way to have a stand-alone baseline training process that works end-to-end easily from just checking out the code as-is (small set of images/bounding boxes as a sample training set) -- then i could swap out the images and bounding boxes used as input -- especially if there was a script to help with the Computing Box Anchors step. Not sure how far the current state is from that though. I'm having a hard time getting the training process started without wiring up the missing bits.
Another clarification: Will the same process work for sizes < 32 if that value is changed (e.g. to 8)?

@chasemaier  Have you train your dataset success?

hi similarly, the self training module seems to be missing files, any chance we can fill those in ?


Update create-sharepoint-sites-and-add-users-with-powershell.md

Added a note to the document stating that the resource quota works only on classic sites and not on modern sites.

@JohanFreelancer9 Please copy review

issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/3126

@JoeDavies-MSFT Please merge this PR. Thanks.


Support implicit create from group actor announcements

@pmusaraj This adds support for when a Group actor Announces a Note directly (i.e. an "implicit" create), which is an approach used by some other implementors. We already support implicit Create from Persons.
{
  type: "Announce",
  object: "https://community.nodebb.org/posts/1",
  actor: "https://community.nodebb.org/category/2"
  ...
}

After reviewing the NodeBB implementation, we actually send the whole Note object in the object property, my apologies for the confusion.

No worries, that will also work with this change. Thanks for the check 👍


Very complex data

Data required may be a very deep object, flat schema would be so much less pain!

Agree, created https://github.com/moodysalem/react-tournament-bracket/issues/9
Would love a contribution


ZS-91 Adjust RTU device simulator for use with Cucumber.

All is well.

All is well.

All is well.

All is well.


http OPTIONS call returns 500 error on 0.37.2

Context
I'm using Flask with Zappa, and after upgrading to 0.37.2 (from 0.36.0) the OPTIONS call to my API will return a 500 'Internal server error'.  The error is from Cloudfront, and I don't even see the request even hitting the lambda (judging by the logs).
When I reverted to 0.36.0 and redeploy my API, it works again.  It also works locally, if I call my API against localhost from starting flask locally.  So this appears to be a regression from 0.37+
Expected Behavior
OPTIONS call against an endpoint should return 200
Actual Behavior
500, internal server error from cloudfront
Possible Fix
Reverting back to using 0.36.0 and redeploy
Steps to Reproduce

Have an Flask API endpoint deployed with Zappa 0.36.0, at say http://my.api.com/dev/api/test
Upgrade zappa to 0.37.2
zappa update dev
OPTIONS http://my.api.com/dev/api/test

Your Environment

Zappa: 0.37.2
Python: 2.7 with virtualenv
OSX El Capitan (10.11.6)

Dang, was hoping this was fixed.
Can you tell me what's in your OPTIONS body? Is there nothing?
We need to specifically handle the OPTIONS call in both body and non-body scenarios. See: https://en.wikipedia.org/wiki/Hypertext_Transfer_Protocol#Summary_table
I suppose we should add a line to check if there is a body there, I imagine that's the error? Can you give me the zappa tail?
Related: https://github.com/Miserlou/Zappa/issues/696

Do you have cors: true in your zappa_settings?

Yup, I do have cors: true set.  And the OPTIONS response body is just empty

Removing this key fixes the 500 for me.
Tell me - do you actually use the Zappa/Amazon provided CORS features? Should this feature be deprecated in favor of application-layer CORS?
Pinging @collingreen @mathom

You can leave cors: true but set binary_support: false - they don't appear to be playing nicely together.

Can't say I use it myself. App level CORS seems more useful for a project like Zappa. Just my 2 cents though.

Turning cors off worked, and yeah apparently I did not need to use this setting.

@d3ming I was also facing the same issue and it got resolve by setting up the binary_search to false but at the same time it raise an issue with image upload, So in this I need to have binary_search to true.
Let me know what should I do in this situation as I am having cors enable at django side for my rest api and using these api in angular js for hybrid mobile app. Though I tried to disable the preflight option request in angularjs but its really hard and bad hack to do it as its automatically get added by chrome.
Please let me know how should I deal with this situation.
Thanks.

You have two avenues: a) try to fix simultaneous CORS + Binary Support in Zappa itself, thereby fixing your problem and everybody else's or b) disable cors and then add in your own CORS support at your own application layer.
I strongly suggest a!

Hey @Miserlou, You are absolutely right, as of now I used option b and It would be my honor to contribute to zappa so I am would like to implement the feature a  and would raise PR soon.
Thanks you very much. 👍

Any update on this?

This was one of the issues affecting us, so I dug around trying to find the solution. I found an answer on an unrelated project here: https://github.com/awslabs/aws-serverless-express/issues/58
I tried that out, adding the contentHandling property to the OPTIONS methods, and it seems to fix the problem. This is the first time I have had to dig this far into API Gateway, so if anyone sees or knows any issue with this approach, let me know, but it seems to solve it in all the cases I tried.

"Tell me - do you actually use the Zappa/Amazon provided CORS features? Should this feature be deprecated in favor of application-layer CORS?"
I don't see an option of using application level CORS with API Key. When API Key is set all requests, including OPTIONS return 403 when x-api-key key is omitted. Naturally browser will not supply x-api-key when performing CORS checks.
Although I do think API Key being incompatible with application-layer CORS is a separate issue, @Miserlou do you agree? If so I'll open a new one.

I completely agree with @IlyaSukhanov, we need the cors feature for this very reason as we also use AWS API Keys.


Investigate event hubs live test failures in build

Mike mentioned that you will look at the event hubs failing tests.

https://github.com/Azure/azure-sdk-for-js/issues/1751

closing in favor of #1751


NSConcreteMutableData being sent instead of bytes

The problem
Provide a clear and concise description of what the problem is.
I've created a basic GattServer on Mac, and instead of receiving bytes, or bytearray in the write_request_func.
Reproduction
Provide instructions or code samples that demonstrate the issue if possible>
    def _write(self, characteristic: BlessGATTCharacteristic, value: Any, **kwargs):
        characteristic.value = value
        if isinstance(value, bytearray) or isinstance(value, bytes):
            logger.debug(f'Char. {characteristic.uuid}, value set to: {value.hex().upper()}')  #  This never hits

Expected behavior
Provide a clear and concise description of what you expected to happen.
It would be preferable to receive a device agnostic value, or at least provide a way to get from NSConcreteMutableData to bytes
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. Windows, macOS, Linux]

Additional context
Add any other context about the problem here.

n/m value.str() has the desired effect.

Thanks for this. I think this is helpful to know since it still may be worth while to add some checks to still validate os agnostic values here. Will look into this.


Sparse Matrix Support

See the fwildclusterboot issue.

@s3alfisc I'm interested in writing a PR for this. Is the issue that the input to wildboottest will be sparse matrix which isn't supported right now?


MINOR: bug fixes to ducktape services

Here's a (mostly successful) run with these changes:
http://testing.confluent.io/confluent-kafka-branch-builder-system-test-results/?prefix=2016-06-27--001.1467080884--alexlod--ducktape-fixes--ad85493/
At least one of the failed tests is failing in trunk, too:
http://testing.confluent.io/confluent-kafka-branch-builder-system-test-results/?prefix=2016-06-28--001.1467090978--alexlod--ducktape-fixes--ad85493/
The contribution is my original work and I license the work to the project under the project's open source license.

one minor question, but looks good to me

I just added another commit with an additional minor change: changing zk_set_acl in kafka.py to a boolean, not a string boolean.

LGTM
ping @ijuma or @ewencp for final review/commit

LGTM, we have a test run that is currently only exhibiting other known transient failures.

Any ideas why this was closed? I assume I should make a new PR?

@alexlod, it was merged.

Oh... great! Thanks @ijuma, @granders, @ewencp for reviewing.


[ITensors] More tests for Index

Description
Couple tests for stuff not already covered.
How Has This Been Tested?
Tests pass locally.
Checklist:

[x] My code follows the style guidelines of this project. Please run using JuliaFormatter; format(".") in the base directory of the repository (~/.julia/dev/ITensors) to format your code according to our style guidelines.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have added tests that verify the behavior of the changes I made.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] Any dependent changes have been merged and published in downstream modules.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@9d9f562). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 05cfe65 differs from pull request most recent head 3b2cbd5. Consider uploading reports for the commit 3b2cbd5 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main    #1084   +/-   ##
=======================================
  Coverage        ?   43.85%
=======================================
  Files           ?       94
  Lines           ?     9957
  Branches        ?        0
=======================================
  Hits            ?     4367
  Misses          ?     5590
  Partials        ?        0

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

Thanks again!


problems.showCurrentInStatus showing incomplete messages

Issue Type: Bug
OS: Mac
#85943
When making the window small enough so the "line #, col #" in the status bar disappears, the resulting blank space cuts into the part of the status bar that shows the problem and it looks like the message is incorrect.

Version: 1.41.0-insider
Commit: 97855786a014be2440751b038b373c3726e11fe8
Date: 2019-12-03T05:35:54.992Z
Electron: 6.1.5
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Darwin x64 18.7.0

This should be fixed by #85638


add InvalidMessageException tests

All but the last suggestion, that i had a question about, was applied.


The method 'push' was called on null.

Describe the bug
Error when trying to navigate throught screen using Get.to.
Expected behavior
Navigate to another screen.
Flutter Version:
Flutter (Channel master, 1.18.0-11.1.pre, on Microsoft Windows [versÃ£o 10.0.18363.836], locale pt-BR)
• Flutter version 1.18.0-11.1.pre at C:\src\flutter
• Framework revision c2b7342ca4 (13 days ago), 2020-05-06 23:16:03 +0800
• Engine revision 33d2367950
• Dart version 2.9.0 (build 2.9.0-5.0.dev 9c94f08410)
[√] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
• Android SDK at C:\Users\Dih\AppData\Local\Android\sdk
• Platform android-29, build-tools 29.0.3
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe
[√] Android Studio (version 3.6)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin version 45.1.1
• Dart plugin version 192.8052
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
[√] VS Code (version 1.45.1)
• VS Code at C:\Users\Dih\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.10.2
[√] Connected device (3 available)
• LG K430    • LGK4305DKZJJKN • android-arm    • Android 6.0 (API 23)
• Web Server • web-server     • web-javascript • Flutter Tools
• Chrome     • chrome         • web-javascript • Google Chrome 81.0.4044.138
• No issues found!
Get Version:
2.5.6
Describe on which device you found the bug:
LGK10 - Android 6.0.
Minimal reproduce code
Get.to(ScreenTwo());
Error:
The following NoSuchMethodError was thrown while handling a gesture:
I/flutter (18746): The method 'push' was called on null.
I/flutter (18746): Receiver: null
I/flutter (18746): Tried calling: push<dynamic>(Instance of 'GetRouteBase<dynamic>')
I/flutter (18746): When the exception was thrown, this was the stack:
I/flutter (18746): #0      Object.noSuchMethod (dart:core-patch/object_patch.dart:53:5)
I/flutter (18746): #1      Get.to (package:get/src/get_main.dart:56:41)
I/flutter (18746): #2      ScreenOne.build.<anonymous closure> (package:get_example/main.dart:47:23)
I/flutter (18746): #3      _InkResponseState._handleTap (package:flutter/src/material/ink_well.dart:772:19)
I/flutter (18746): #4      _InkResponseState.build.<anonymous closure> (package:flutter/src/material/ink_well.dart:855:36)
I/flutter (18746): #5      GestureRecognizer.invokeCallback (package:flutter/src/gestures/recognizer.dart:182:24)
I/flutter (18746): #6      TapGestureRecognizer.handleTapUp (package:flutter/src/gestures/tap.dart:522:11)
I/flutter (18746): #7      BaseTapGestureRecognizer._checkUp (package:flutter/src/gestures/tap.dart:282:5)
I/flutter (18746): #8      BaseTapGestureRecognizer.handlePrimaryPointer (package:flutter/src/gestures/tap.dart:217:7)
I/flutter (18746): #9      PrimaryPointerGestureRecognizer.handleEvent (package:flutter/src/gestures/recognizer.dart:475:9)
I/flutter (18746): #10     PointerRouter._dispatch (package:flutter/src/gestures/pointer_router.dart:76:12)
I/flutter (18746): #11     PointerRouter._dispatchEventToRoutes.<anonymous closure> (package:flutter/src/gestures/pointer_router.dart:122:9)
I/flutter (18746): #12     _LinkedHashMapMixin.forEach (dart:collection-patch/compact_hash.dart:379:8)
I/flutter (18746): #13     PointerRouter._dispatchEventToRoutes (package:flutter/src/gestures/pointer_router.dart:120:18)
I/flutter (18746): #14     PointerRouter.route (package:flutter/src/gestures/pointer_router.dart:106:7)
I/flutter (18746): #15     GestureBinding.handleEvent (package:flutter/src/gestures/binding.dart:218:19)
I/flutter (18746): #16     GestureBinding.dispatchEvent (package:flutter/src/gestures/binding.dart:198:22)
I/flutter (18746): #17     GestureBinding._handlePointerEvent (package:flutter/src/gestures/binding.dart:156:7)
I/flutter (18746): #18     GestureBinding._flushPointerEventQueue (package:flutter/src/gestures/binding.dart:102:7)
I/flutter (18746): #19     GestureBinding._handlePointerDataPacket (package:flutter/src/gestures/binding.dart:86:7)
I/flutter (18746): #23     _invoke1 (dart:ui/hooks.dart:291:10)
I/flutter (18746): #24     _dispatchPointerDataPacket (dart:ui/hooks.dart:200:5)
I/flutter (18746): (elided 3 frames from dart:async)
I/flutter (18746): Handler: "onTap"
I/flutter (18746): Recognizer:
I/flutter (18746):   TapGestureRecognizer#11ca8
I/flutter (18746): ════════════════════════════════════════════════════════════════════════════════════════════════════

I think you forgot to add the GetMaterialApp.
add Get before your app stuff.
Before: MaterialApp
Then: GetMaterialApp.

Closing this because it is the absence of GetMaterialApp.
If you face problems, do not hesitate to open another issue, and if you have questions about it, you can ask here that I answer.


Playlist privacy setting.

API version 6.0 is missing an option to make new created playlist private.

Playlist privacy set in another call, not on creation.


android build fails 1.8.3

Using cordova  6.3.0,on ionic run android it shows this error:
FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring root project 'android'.
> Could not resolve all dependencies for configuration ':_debugCompile'.

> Could not find any matches for com.google.android.gms:play-services-gcm:9.0.2+ as no versions of com.google.android.gms:play-services-gcm are available.
     Searched in the following locations:
         https://repo1.maven.org/maven2/com/google/android/gms/play-services-gcm/maven-metadata.xml

         https://repo1.maven.org/maven2/com/google/android/gms/play-services-gcm/
         https://jcenter.bintray.com/com/google/android/gms/play-services-gcm/maven-metadata.xml

         https://jcenter.bintray.com/com/google/android/gms/play-services-gcm/
         file:/home/ubuntu/Descargas/android-sdk-linux/extras/android/m2repository/com/google/android/gms/play-services-gcm/maven-metadata.xml
         file:/home/ubuntu/Descargas/android-sdk-linux/extras/android/m2repository/com/google/android/gms/play-services-gcm/
     Required by:
         :android:unspecified

@JulianGarciaCastillo update your local android build environment https://github.com/phonegap/phonegap-plugin-push/blob/master/docs/INSTALLATION.md#compilation

Perfect! Thank you, I just needed it to install Android Support Repository on sdk manager.


temporary speed change

Hello again!
When navigating, dynamic control of cursor speed would be useful, i.e. keys which increase or decrease the cursor_speed value while being pressed.
From an implementation point of view, it would probably be much simpler to just introduce "fast" and "slow" variants of the cursor movement commands, i.e. cursor_speed, 'cursor_speed_slow, cursor_speed_fast` and the same for left, right, up, down.

I second this.

I've been meaning to add configurable pointer acceleration to account for the gap between macoscopic and fine movement. I'm not opposed  to also adding some kind of explicit speed toggle, but I wonder if the former might be sufficient.

To make this solution feature complete, there should also be kinetic movement (as present in trackpoint hardware as well as qmk and uhk), so that as soon as a movement key is pressed the pointer gets accelerated according to some quadratic curve. Short presses thus cause slow and precise movements while longer presses can cover distances. However, as this is probably a bit involved, I suggested incorporating full mouse control in keyd, see my other comment at https://github.com/rvaiya/keyd/issues/158

so that as soon as a movement key is pressed the pointer gets accelerated according to some quadratic curve

Indeed. This is what I meant by 'pointer acceleration'. There is already similar logic for scrolling (scroll velocity increases with time). The older version even had something called 'scroll flinging' which was fully inertial scrolling to which you could impart an impulse by tapping the scroll key for a second or third time, but I ended up removing it after realizing it wasn't very useful for most scrolling tasks, and probably counter productive in most cases.

Pointer acceleration is now enabled by default in the latest release. I've picked default which feel reasonable on my machine, but you may want to play with the acceleration and max_speed variables. If you still feel the need for dedicated keys after using this for a bit let me know :).

Thank you very much, have been using it for about a week now and it works like a charm! On my trackpoint-less keyboards I still miss a “quick move” possibility (for covering mid-range distances for which a hint-mode jump would take too long) but this is definitely not urgent, nor critical. If you decide and find the time to implement it: How about a generic action, which would disable acceleration and sets speed to a given argument while the corresponding activation key is held down? This way one could either slow down or speed up movement.
Another option would be to just multiply speed, acceleration and max_speed by some factor, but this seems less useful to me.

I'm closing this for now. Feel free to reopen it if you feel the issue is unresolved.

Again I'm very late with a reply, sorry for this. The acceleration feature works like a charm (as was to be expected) and I'm actually using it quite often. Thanks again, I really can't stress enough how useful warp/keyd are for me! They solved all the input problems I had been struggling with for years, and the longer I use them, the more I get the impression that all the feature requests I'm posting here are just desperate attempts to convince myself that there can't be software which is almost perfect (another such attempt is my inclusion of the word 'almost').


feat: port CategoryTheory.Limits.Preserves.Shapes.Products

[x] depends on: #2564

This PR/issue depends on:

leanprover-community/mathlib4#2564
By Dependent Issues (🤖). Happy coding!

This PR/issue depends on:

leanprover-community/mathlib4#2564
By Dependent Issues (🤖). Happy coding!

bors r+

bors r+


net: openthread: ACK timeout and RX timestamp

A couple of improvements:

Translate IEEE802154_HW_TX_RX_ACK capability and forward it to OpenThread,
Set mTimestamp field in OpenThread RX frame,
Implement these features for nRF5 radio driver

hal_nordic SHA updated, removing DNM.


ADBDEV-3048 FAKE DON'T MEGRE!!!

Here are some reminders before you submit the pull request

[ ] Add tests for the change
[ ] Document changes
[ ] Communicate in the mailing list if needed
[ ] Pass make installcheck
[ ] Review a PR in return to support the community

@enp, is this still needed?

@enp, is this still needed?

No, let's close


Add Feature to Draw a Green Box Around Detected Barcodes

Hello,
Thank you for the amazing barcode scanner library! I have a feature request: Would it be possible to add functionality that draws a green box around detected barcodes during the scanning process? This would make it visually clear which barcode has been detected, similar to the example in the attached image.
I believe this feature could greatly enhance the usability and visual feedback of your library. Thank you for considering this!
Best regards,

Hi @fatihydev - This feature is now available in version 1.4.0. Please refer to the example for further details.


Curl Output

What is this supposed to do? Nothing seems to happen after the re
$ curl -X POST \
→      -H "Content-Type: application/json" \
→      -d '{"typeScale": [5,4,3,2,1,0.5] }' \
→      https://tachyons.pub

<!doctype html>

Redirecting (302)

https://zeit.co/johno/tachyons-publish-api/bvyenznlao?redirect=1

I think the server might've never unfrozen after Zeit now's outage on Friday. I just gave the server a kick so it should be working again. Could you give it another try and let me know if it continues to not work?

Ok. Returns a url. https://s3-us-west-1.amazonaws.com/tachyons-pub/0.13.0/111e6389b4cc3b97dd4f2dcf07693226/index.html. So what just happened? I missed the magic i think. What is this doing exactly?

Apologies that none of this is documented yet. It's pretty alpha software since I haven't had the time to formalize it after completing this proof of concept (that's nearly ready for a true beta aside from the lack of docs).
This project is the result of a couple hack sessions with the end goal of dynamically generating Tachyons css from a configuration. Hypothetically this allows (will allow) someone to generate their own custom project with different media queries, colors, typescales etc. with accompanying documentation.
The api itself provides a microservice that allows someone to make an api request to generate and host the docs/css in an S3 bucket. So, you can come up with your own configuration (a structured JSON object) and then POST it to the endpoint.

Take a look at compositor.io. seems they are basically doing the same thing to generate a github static site...


Discuss url naming convention.

Given a label say training the qhub urls would be:
Less verbose option (make the jhub link primary):
jupyter: training.qhub.dev
staging jupyter: staging.training.qhub.dev
kubessh: shell.training.qhub.dev
staging kubessh: staging.shell.training.qhub.dev
OR we could keep what we currently have but add an extra landing page
qhub landing page that links to the others: training.qhub.dev
jupyter: jupyter.training.qhub.dev
staging jupyter: staging.jupyter.training.qhub.dev
kubessh: shell.training.qhub.dev
staging kubessh: staging.shell.training.qhub.dev
The second would also give us a good way to list available intergrations like grafana etc.
@costrouc @aktech @tylerpotts @prasunanand what do y'all think

I think this naming convention works fine.
We also discussed the naming convention for Quansight managed qhub github repos in today's standup. We're thinking:
qhub-<project>-<cloud>-<deployment>
Datum's prod deployment github would then be:
qhub-datum-gcp-prod

So are you saying the prod/dev env would have different github repos? That sounds like it would be hard to keep in sync.

I agree with @dharhas .
@tylerpotts, Sorry  I missed the standup today. I am unable to understand the naming convention . I have a few questions. I would like to discuss in the next standup.

Issue moved to Quansight/quansight-internal-support #69 via ZenHub

Moved this issue here since this is an internal IT question


Text formatting (case transformation) processing order option. PowerRename.

Description of the new feature / enhancement
I propose including a choice when text formatting (case transformation) is applied: before the search/replace is applied (a new option); or after the search/replace is applied (the current behaviour).
Scenario when this would be used?
Consider current behaviour of using the text formatting "Capitalize Each Word" choice ...
Without any values in the Search for nor Replace with the string "rob the dog" is transformed to "Rob The Dog", as expected.
With - Search for = \s; Use regular expressions = ticked; Match all occurrences = ticked - then the string becomes "Robthedog". That is, the search/replace is processed then the text formatting is processed. This is the current behaviour.
However, it would be desirable to have the option to have the text formatting processed then the search/replace be processed. That is, to end up with "RobTheDog". Furthermore, it strikes me this ought be the default option, if there were the two options.
Supporting information
If the proposal to have both options is convincing then the remaining decision would be how to present this in the UI. Choices here would include:

A simple additional toggle under "Text formatting"; or
The text formatting section duplicated at the top. Then the two sections renamed "Before search/replace text formatting" and "After search/replace text formatting".

Duplicate of #687

This should be a existing feature, try these options out:

@franky920920 and @crutkas. My original post was predicated on the text formatting options @franky920920 circles being an existing feature. I made this clear, for example, with "Consider current behaviour of using the text formatting ....". What is circled, and what already exists, are: lowercase, Uppercase, Titlecase, "Capitalize Each Word", (and Enumerate Items). What is not circled is the feature I mention: the processing order that any of those text formatting options is applied. This is a feature nowhere mentioned in #687.
Can I suggest you read, rather than skim, my original post then reopen the thread?

I think this adds a lot of complexity.  I really need to better understand the scenarios in which this would be leveraged versus just having a bit more crazy regex.

How about the scenario I mention? I'm constantly having a need for it. And, as mentioned, it is likely to be the more common scenario than the current behaviour.


Fatal error: Can't use library

Hey guys,
I've just installed you lib via composer as per documentation.
I've got:

opentracing/opentracing (1.0-beta1)
jukylin/jaeger-php (v2.0.0-beta)

Now, when I use it I see the following PHP Fatal error:
Xdebug: Fatal error: Declaration of Jaeger\Jaeger::inject(OpenTracing\SpanContext $spanContext, $format, &$carrier) must be compatible with OpenTracing\Tracer::inject(OpenTracing\SpanContext $spanContext, $format, OpenTracing\Propagators\Writer $carrier) in /srv/vendor/jukylin/jaeger-php/src/Jaeger/Jaeger.php on line 13.

It looks like package versions are not compatible. Can you please let me know if this is a known issue?
Thanks

I get the problem too ...

Sorry , you can use  opentracing/opentracing (1.0.0-beta2)

Not sure about that... tried to replace thewhole dir and it didn’t help :(
On Mon, 19 Mar 2018 at 17:21, 742161455<EMAIL_ADDRESS>wrote:

Sorry , you can use opentracing/opentracing (1.0.0-beta2)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jukylin/jaeger-php/issues/17#issuecomment-374112328,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH0rP-TSyIZg1ZUDRH4yhJZBr1By-aBOks5tf06DgaJpZM4Svi32
.

What's your installation step.

I install it via composer, if I use 1.0.0-beta2 like this:
"require": {
        "opentracing/opentracing":"1.0.0-beta2",
        "jukylin/jaeger-php" : "v2.0.0-beta"
	}

I'll get an error:
- jukylin/jaeger-php v2.0.0-beta requires opentracing/opentracing 1.0-beta1 -> satisfiable by opentracing/opentracing[1.0-beta1]
If I just replace the opentracing vendor folder with contents from 1.0.0-beta2 I still see the same error message.
P.S: I can't use   "minimum-stability": "dev", on prod, that's not an option.

fix bug, but  you have to use "minimum-stability": "dev", It's not a stable.

Hey @jukylin, thanks for fixing it!


Add custom discord widget to landing page

Yeah, I'd quote someone,

it's like you have a basic flat and you put Versace chair into the room.

Here starts the idea to have a better discord widget so people could easier find us!
Spec

fits design of our page
logo / users / online users / join button
@ins0mn1a will sketch something!

Ref

Discord API
https://github.com/sveenxx/discord-widget
https://devbest.com/threads/custom-discord-widget.92305/

We will avoid a lot of small unnecessary requests tho!

I can do this. I'll need the guild id of the server to be able to fetch data relevant to the discord server, can you please provide me with that?
The steps to find the guild id is:

In Discord, open your User Settings by clicking the Settings Cog next to your user name on the bottom.
Go to Appearance and enable Developer Mode under the Advanced section, then close User Settings.
Open your Discord server, right-click on the server name, then select Copy ID
Paste the Server ID on your text editor for later
If desired, you can disable Developer Mode now.

The API call that we'll use is: GET/guilds/{guild.id}/preview
https://discord.com/developers/docs/resources/guild#get-guild-preview

I guess it's this one?
840514076538830888
seems matching with widget https://github.com/kodadot/nft-gallery/blob/main/components/landing/Landing.vue#L72

@atharva3010 any progress on this?


Java classpaths are ignored when specified as a javac arg

When specifying a classpath for a java project like so:
let g:neomake_java_javac_args = ['-cp', 'some:classpath:here']
It appears that Neomake is making the arguments silently vanish here because the classpath is failing to expand and this results in an empty string
I've tried escaping the : as \: but that still seems to fail
Another way I've gotten around this is patching neomake locally by replacing the expansion with something that checks for an empty result and returns the plain string like so
function! s:expandIfValid(arg) abort
        if expand(a:arg) != ""
            return expand(a:arg)
        else
            return a:arg
        endif
endfunction
call map(args, 's:expandIfValid(v:val)')

but this is a suboptimal solution. Any help would be appreciated!

It is a bug，I will fix it.

Hmm, I might be missing the point, but :echo expand('some:classpath:here') looks on in Vim?!

@DonnieWest fixed, but for classpath, you should use g:neomake_java_javac_classpath.
for example let g:neomake_java_javac_classpath = '~/test:./target/classes in linux, in windows, you should use let g:neomake_java_javac_classpath = "~\test;.\target\classes"

@wsdjeg that works quite well! Thank you so much! Looking forward to having it merged into neomake proper :)
@blueyed well, apparently some other symbol is causing the expansion to fail. This definitely explains why my efforts on my end have thus far failed

@wsdjeg
Just check the existing sh.vader file, which is the test by itself.
You need Vader also, of course: https://github.com/junegunn/vader.vim/
Tests can then be run in Vim by itself using :Vader - although I do not use this often myself.
The Makefile just wraps it for non-interactive use.


Generate PDF of Rd files and out it to the repo

Hi @mkbouaziz ,
Do you already have some kind of build process to create the pdf and inlcude it to the repo?
Best, Philip

Hi @pmig ,
The command is:
R CMD Rd2pdf OrientR/
and some packages have to be implemented before, like:
1-LaTeX
2-sudo apt-get install texinfo

it happens that Rd2pdf is incompatible with R 3.2.3 as described here:
You receive the error :
! LaTeX Error: File `inconsolata.sty' not found.

http://markmail.org/search/?q=list%3Aorg.r-project.r-devel+inconsolata.sty#query:list%3Aorg.r-project.r-devel inconsolata.sty+page:1+mid:arxwlgn5jkvzpibd+state:results


onbeforeunload callbacks do not allow network requests

Which version of PhantomJS are you using? Tip: run phantomjs --version.

2.1.1

What steps will reproduce the problem?

Register a beforeunload callback containing a network request.
Load the page in phantom
Close the page
The callback triggers but the network request is not made

Example:

window.addEventListener('beforeunload', function () {
   $.post('/foo', {});
});

Which operating system are you using?

OSX

Did you use binary PhantomJS or did you compile it from source?

Installed via homebrew

Please provide any additional information below.

It works in Chrome/Firefox/Safari.

This is most likely due to the fact that $.post('/foo', {}); is an asynchronous operation, and that the page closes before the operation is finished. You need to replace the asynchronous operation with a synchronous one:
$.ajax({
    async: false,
    data: {},
    type: "post",
    url: "/foo"
});

Using JQuery synchronous requests is our current work-around. However async: false is deprecated as of 1.8 and is a hack anyway. What we really want is the proposed beacon standard.
As far as I can tell, sending xhr requests as the page unloads is a gray area in the browser spec. Chrome and Safari let the request through. Firefox lets it through but trigger the 'fail' callback. IE aborts the request and Phantom appears to not even make the request.
In this gray space, it would be nice if PhantomJS adopted the more lenient behavior of Chrome/Safari.

I just tested this with the latest Firefox, Chrome and Safari on Mac and only Chrome actually sends the HTTP request if it's async. Firefox and Safari both don't even send the request.

That's interesting because I definitely saw the requests reach my server from Firefox/Safari. I was running the server locally, so it's possible that it only worked because of the low latency.

Looks like you are right. I tested it a bit more now. Before I tested using PHP, now I'm testing with nodejs. Firefox does request the URL, I'm guessing PHP was too slow and didn't start before the request was cancelled. I still can't get it to work with Safari (both local and remote). The only time Safari seems to request the url is when I exit Safari.
require('http').createServer(function(req, res) {
  console.log(req.url);

  res.statusCode = 200;
  res.end(`
<!doctype html>

<meta charset=utf-8>
<title>beforeunload ajax test</title>

link

  `)
}).listen(8090);

Quick question: are you doing this in the main execution context? Or with a real web page?

@erikdubbelboer I too am using a node backend. I also noticed that Safari only made the request when closing the tab, but at the time I just assumed it was because the event was not firing on refresh.
@Vitallium It's a real webpage. I tested Chrome/ Firefox manually and using webdriverio (where their behavior matched). I only tested phantom using webdriverio.

What about this issue? I'm forcing it too. Are there any other way or a hook to handle the onbeforeunload event through the webdriver and phantomjs?

@FelipeeDev @Gerg Did you guys managed to solve this one?
I'm facing it myself.


Support for out-of-source builds

The utility assumes that the .c files and the git repository are in the same tree as the .gcno files (-r option).
With cmake, it is quite easy (and common) to build utilities in a separate "build" directory.
I think a --source-root option would make sense.

huh, so am I correct in saying that if you have:
-project_root
|-src
|-other_stuff
|-build
    |-compiler_1

you won’t run into any issues?

I'd say that I have to test it, but I think that is correct: it would work if I "-r project_root"

Yes, this is correct, having both src and build directories under a common parent root directory works fine.

I'm also trying to use this with CMake on Travis.  The problem I'm getting is that the exclude folders for the coverage reports and the source files is the same.  I keep getting these matches for source files on CMake internal files (feature_tests.c, CompilerId*).  I can use some specific rules but it seems like this is depending on CMake internals.
Any way we can have exclusion patterns specifically for source file discovery?

I've found better functionality and support on codecov.io and have moved there. The bash script works quite well, and is easier to use than coveralls.

Would it be easy to just have two arguments, one for the build tree and another for the source tree?


remove dashboard operations table header

The operations section of dashboard has a double header. The panel header that reads "Operations" and a table header that reads "Operation Name   |   Switch". The table header is redundant and can be removed.

OK, cool, thanks!
Although this page is going to be converted semantic UI shortly.

But.... hmm, can you re-PR to develop branch?

But.... hmm, can you re-PR to develop branch?

https://github.com/gogits/gogs/pull/1584


[pyarrow] Should Table.from_pandas default to preserve_index=False?

I spent a bit of time tracking down why an __index_level_0__ columns was being written to our Parquet files. Eventually we found that it was coming from the from_pandas method on Table. There is an optional parameter preserve_index which defaults to True. Setting it to False dropped the index on transfer and led to drastically smaller file sizes (4MB -> 1MB, 2M rows).
table = pa.Table.from_pandas(df, preserve_index=False)

https://github.com/apache/arrow/blob/8c9890c3731688683024a14c12cdca34c2901077/python/pyarrow/table.pxi#L949-L950
Would it make sense to not preserve the index by default? In many cases (but maybe not all?) the index is not desired and bloats Parquet files. Presumably it would have a similar impact in-memory in the Arrow format.

It really depends on the user. We selected the defaults to faithfully store the data represented by the pandas DataFrame, so my view is that we should leave the default as is. This issue could probably be better highlighted in the Python documentation, though
We could possibly implement an optimization for the case of simple RangeIndex (similar to https://issues.apache.org/jira/browse/ARROW-1639) to reduce the storage footprint


[KO] Update translation

@Dalgona, @malkoG, @devleoper, @riseshia review please.
FYI. I use this command for diff:
git diff f8baebe..bcb0548 -- lessons/

Also, includes this review.

💯


Ansible verion doesn't recognize package

Doesn't look like package is a known parameter
[chernand@chernand openshift-ha-load-balancer]$ ansible --version
ansible 1.9.6
  configured module search path = None
[chernand@chernand openshift-ha-load-balancer]$ ansible-playbook -i ansible.hosts lb-ha-setup.yml
ERROR: package is not a legal parameter in an Ansible task or handler

Thanks for the report. The package module (https://docs.ansible.com/ansible/package_module.html) was introduced in Ansible 2.0 and since OCP 3.4+ comes with Ansible 2.2 or newer I'm going to close this issue. A quick fix for older Ansible versions would be use the yum module instead.


Not working with forwardRef wrapped components

https://github.com/storybooks/babel-plugin-react-docgen/blob/fa170793df1d9065ba4bd25238627a72e55fbcb6/src/index.js#L49
This seems to exit early when a component is generated in a forwardRef call:
const Component = React.forwardRef(() => {
  return
Hello

});

And as such does not attach any docgen data. This seems to stem from actualNameHandler.

Looks like v4 of react-docgen support this: https://github.com/reactjs/react-docgen/pull/324

Just tested this plugin using react-docgen@4 and can confirm a component wrapped in forwardRef works as expected if this is commented out:
if (!exportName) {
  // return;
}

So if we can fix actualNameHandler to recognise forwardRefs this should work.

any chance you can issue a PR for this? If so I can test and release an upgrade

Yes, I'm just working one up, will be with you shortly.

Added: https://github.com/storybooks/babel-plugin-react-docgen/pull/68

fixed in v3.0.0, will integrate into storybook.

Do you want me to create a PR to update to v3 here:
https://github.com/storybooks/storybook/blob/next/app/react/package.json


DT tooling and DT API for RANGES property

As requested in https://github.com/zephyrproject-rtos/zephyr/pull/37668#pullrequestreview-729799792, this adds support for parsing and extracting the DT RANGES property with the associated macros.
The ranges property is used standalone in PCIe DT bindings, As described in IEEE Std 1275-1994, to describe the PCI I/O and memory regions.

Seems doctree doesn't like #address-cells and #size-cells in doc, is there a way to do this correctly ?

Seems doctree doesn't like #address-cells and #size-cells in doc, is there a way to do this correctly ?

You need to indent the code examples so doxygen recognizes them as preformatted code blocks. See the other headers for examples.

@galak is the API ok so I can add tests in order to merge #37688 ?

Following the rework, I completely cleaned the devicetree/ranges.h header and the related tests.

Thanks @superna9999 . I'll get back to you this week; I need some time to digest.

I'll try to start the stable API process, but, should I start it for this PR or should I open a separate PR for 9685ca8 ?
And do I need to also do the same process for da0cfd6 ?

I think that in this case, it is OK to do a single stable API change process for both commits, since they are part of the same PR, which is making one logical change ("use DT_NUM_RANGES instead of DT_PROP_LEN for the ranges property") to the API.

The minimal should be to exclude dma-ranges like interrupt, reg and ranges like in:

My opinion is that this is acceptable to include this while we're breaking things for regular ranges, and then defer dma-ranges support to potential future work.
If you do, you will need to update the stable API change request etc.

We should probably update release-notes as part of this PR.

@galak @mbolivar-nordic done

@mbolivar-nordic @galak Ping.
Are the release-notes and dma-ranges changes ok ?

@mbolivar-nordic @galak Ping.
Are the release-notes and dma-ranges changes ok ?

LGTM.

Looks like a merge conflict in release notes.

@galak Fixed


Update tapir-akka-http-server, ... to 1.1.0

Updates

com.softwaremill.sttp.tapir:tapir-akka-http-server
com.softwaremill.sttp.tapir:tapir-enumeratum
com.softwaremill.sttp.tapir:tapir-json-circe
com.softwaremill.sttp.tapir:tapir-swagger-ui-bundle

from 1.0.6 to 1.1.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.tapir" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.softwaremill.sttp.tapir" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #377.


Add Jest to Test this project

This PR adds a dependency on Jest (and friends) and setups the first tests. It also adds a github workflow to run these tests as CI.
The tests as shown using jest (instead of npm run test, because that runs jest --silent suppressing the console logs).
the backport action
    given a payload for a PR without backport label
      ✓ can be run without impact (20 ms)
    given a payload for a PR with backport label
      and backport.sh returns exit code 1
        ✓ comments on failure (9 ms)
      and backport.sh returns exit code 0
        ✓ creates a pull request and requests reviewers (4 ms)

closes #6

@pihme Please have a look after the holidays.
This turned out to be a larger PR than I initially wanted, but it is what it is. I think it might be best to read the commits individually. If you would like to sync on this please let me know.