added reverse transformation from 3D to 2D

Added reverse transformation with visualization of 3D bounding box points to 2D image plane.

Had to set p1 and p2 as tuples like below. Otherwise, the program threw the SystemError: new style getargs format but argument is not a tuple
p1 = tuple(int(x) for x in img_points[line[0]][0])
p2 = tuple(int(x) for x in img_points[line[1]][0])

Now, I can run the code example, but it does not how me the color and point cloud images. The program gets frozen as soon as I run it.

The error you got was due to older opencv version. I updated the requirements.txt and not it should be fixed.
As for the freezing, does the app run correctly if you comment out 2D visualization in main.py (lines 159 and 160)? I would need some more information to try to reproduce this.

I see. The packages are now updated based on the requirements.txt.
I still face the issue with and without the two lines (159 and 160 in main.py)

This is what I get... It freezes every time. Sometimes the color window pops up, but not the point cloud window.

I use OAK-D-POE-PRO and the below is the versions of FW and depthai

so, I tried the code with other cameras (OAK-D-POE) and it worked fine. The issue was my OAK-D-POE-PRO camera... have to do factory reset maybe.

I have three cameras (1 OAK-D-POE-PRO and 2 OAK-D-POE) and now only 1 OAK-D-POE camera works with this code. Very confusing.

Based on the camera stream I would say that the problem is the setting in which you are testing the code. The current approach has some limitations and the main one is that the region of interest has to consist only of ground (a plane) and a box. Check the images in README.md  to get an idea of what is the assumed box position by the algorithem and other limitations of the approach.

I figured out my issue. After doing "pip3 install -r requirements.txt" with the requirement file in box measurement dir, the latest version of depthai (depthai==<IP_ADDRESS>) gets overwritten by the old one, depthai==<IP_ADDRESS> which is specified in the file. With the certain version of depthai and the latest FW, it gave me difficult time to use the stereo sensors.
After upgrading the depthai version to the latest, I was able to avoid the issues I wrote above. It would be a good idea to update the depthai version in requirements.txt

Other than that, I don't have any issues. Thank you for you effort implementing the feature very quickly. Much appreciate it.

Thanks for the feedback @miknai, I updated the requirements.


Game crash while using gadgets

Minecraft Version: 1.20.1
Mod Version: 0.1.3
Using Optifine: No
Issue: Game frequently crashes when using gadgets, or even scrolling past them on the toolbar.
This is the error log left after one of the crashes.

These kinds of issues usually are memory related. Weird. Is there copy data on the gadgets ? Is it any particular gadget causing it, or all of them?

No that should be plenty - it could possibly be driver or video card or your java install related?
Are you using the 64 bit version of Java? What kind of video card do you have?

Java Version: 17.0.2 64bit
card is a GTX 1080.

So its really Java crashing, NOT minecraft, but of course, Minecraft runs in Java.  It looks like an issue with your graphics card, because the file thats crashing is nvoglv64.dll - which is nVidia related.
Could you try updating your graphics card drivers?

Yeah, looks like a new driver did come out yesterday. I'm downloading it now.

Thanks! If that doesn't work I'm mostly out of ideas, except maybe doing a clean install of your drivers. I'll keep this bug report open in case others have the same issue, but no ones reported anything like this yet so its pretty weird :).

That's a negative. As far as I know, none of my mods do anything to the rendering engine. That is unless Create does something to it for all it's fancy effects.

Gotcha.  Well building gadgets is calling a native to minecraft rendering call, and its the minecraft piece thats crashing, oddly enough.
Let me know how the drive update goes! :) THANKS!

As an update it looks like a few more people are having this issue now. Will keep you updated. Still seems to be a driver problem and in addition we’ve found a way to duplicate the crash in vanilla - apparently reloading textures in the end with f3-t does it? May need to go in and out of the end a couple times.

Hmm, guess that’s a good news/ bad news scenario. I did have the game crash once after updating the video driver before going to bed.
I installed a pending Windows update overnight, but I haven’t been home again yet today to do any further troubleshooting.
For what it’s worth, all the crashes I’ve had have been in the overworld. Also, I am using a 64 bit texture pack. If it’s reproducible in vanilla, I’m assuming neither would matter.

Yea the current best guess by folks smarter than me is that this is an nvidia driver bug that is obscure in vanilla minecraft, but used more in BG2 :).

I am having the same issue.
Minecraft Version: 1.20.1
Mod Version: 0.1.2
Java: OpenJDK 64-Bit Server VM Zulu17.42+19-CA
Video: NVIDIA GeForce RTX 4080
Using Optifine: No
I was on driver version 536.99 (8/8) and upgraded to the latest version 537.13 (8/22) and the issue remained.
I wasn't able to get the standard building gadget to crash, but I could get the exchanger to crash pretty reliably.  In particular, if I tried exchanging multiple blocks before they were all done, I could get it to crash when I started the 4th exchange while the first one was wrapping up.  Every once in awhile, just looking at something would cause the client to crash.
I know this could be a video driver issue, but I am not seeing this with many other 1.20.1 mods.  Something is different about this mod on 1.20.1 that is triggering it.  I haven't used 1.19.x with the mod, but I used it extensively in 1.16.5 and it never crashed.

Thanks! We’re definitely lookin into it but we’ve had success causing the exact same crash in the end with vanilla.
Related to:
https://bugs.mojang.com/browse/MC-263607
If you go into the end - kill the dragon - and do f3-A it seems to crash with the same problem.
Do you mind trying that in vanilla? Just curious if it happens no need.
Bg2 calls a vanilla method that’s rarely used in vanilla to do its rendering. If we can’t fix it I may have to find another way to render.

I've been able to make it back to Minecraft. After trying the vanilla end suggestion, I am even more intrigued.
After killing the dragon, I flew around the end for 40 minutes basically spamming F3-A to no avail. I upped the render distance. I also tried leaving and re-entering the end multiple times, even respawning the dragon and going at it while reloading.
No matter how fast (or slow) I reloaded the chunks, I could not get the game to crash. Sadly, I could get modded Minecraft to crash in under 30 seconds with the gadgets.

Thanks for checking - thats the problem with this bug - i've been told by some very smart folks on forgecraft that theres nothing I can do to prevent/fix it - its really just openGL and an NVidia problem.
It just so happens that BG2 is exasperating it.  Hopefully we get it sorted via other means soon

I just pushed v0.1.6 that addressed a memory leak and may have had an impact on this bug. We can't know for sure, because of how hard it is to reproduce, but would you mind grabbing the latest bg2 and seeing if it makes this happen less by chance?
https://legacy.curseforge.com/minecraft/mc-mods/building-gadgets/files/4723411
(I just pushed it to curseforge so if its not available yet give it 5 minutes)

So far so good... I updated to 0.1.6 and have been using both the Building Gadget and the Exchanging Gadget with great success for awhile now... Not a single crash and I have my fingers crossed!  Thanks!

Woot! Let me know if you get any crashes at all. The expectation is that you may crash less but it’s not expected to entirely resolve the issue. That said I’m crossing my fingers 🎉

Same results for me. Seems to be going stable for now. Big thumbs up.

Wow glad to hear.

Been running with this version of the mod for a day now and we have used the building and exchanging gadgets extensively as well.  We haven't seen a single crash!  Thanks again!

Awesome. I’ll close this for now. If it starts happening again please let me know.

Same problem and now installed the update, fill report back if it happens again. With quick testing didn't seem to crash now.

Cool :)


DUPLO-25454: TF: S3: Replication: Add validation for rule name

Overview
Add validation for rule name for terraform code
Summary of changes
This PR does the following:

Validation function added which will validate name

Task linked: DUPLO-25454 TF: S3: Replication: Add validation for rule name


fix(badge): dpo tokens

What I did

use dpo status tokens in badge
deprecate old statuses and alias them to dpo statuses

@marionnegp Sounds good, do you want to handle that?

@coreyvickery, yep! I can update the Figma component by tomorrow.
@bennypowers, I'll work on updating the "Variants" section of the docs today. I'll go ahead and use this same branch.

I've updated images in the docs and added back a length token for a min width of 32px, which includes padding.


[Feature]: Remove the warning Node.js 12 actions are deprecated

Contact Details
No response
What happened?
The following warning was reported in the unit test execution workflow: Node.js 12 actions are deprecated.
Solution
Please update the following actions to use Node.js 16: actions/checkout@v2.
For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/
Alternatives
A clear and concise description of any alternative solutions or features you've considered

Fixed in PR #7


Broken prefixed flex

The code below is broken by clean-css:
.foo {
    -ms-flex: 1 1 0px;
    flex: 1 1 0px;
}

The expected output is:
.foo{-ms-flex:1 1 0px;flex:1 1 0}

-ms-flex requires a unit to be given for the 3rd argument even if the value is zero, while flex accepts just 0.
But the actual result which breaks layout in IE 10 is like this:
.foo{-ms-flex:1 1 0;flex:1 1 0px}

Good catch! It's already fixed on 3.3 branch, I'll release it as 3.3.8 later on.


[MM-62196] Add support for downgrades using the MSI

Summary
An issue was noted while testing the MSI and trying to install an older version over top. An error would be thrown and the MSI would technically install, but would miss the main EXE file causing the app to not work at all.
This PR makes a couple template changes to actually support downgrading properly (as of the installer that this is released in) and fixes the error and missing EXE.
Ticket Link
https://mattermost.atlassian.net/browse/MM-62196
Add support for downgrading using the MSI installer

@yasserfaraazkhan Just to speed things along, I'm going to merge without QA approval here and cut a new dot release that you can test. Feel free to test this fix there.

Cherry pick is scheduled.

/cherry-pick release-5.9

Cherry pick is scheduled.


Disable logs

Good day. Can I somehow disable logs in development mode? It's very hard to find the right information in a big count of vanity logs.

I tried to do something like this in Rails:
file: config/environments/development.rb

Vanity.configure do |config|
    config.templates_path = 'app/views/vanity'
    config.logger  = Logger.new('/dev/null')
 end

but it did not help

Ahhh, I found solution:
Vanity::Adapters::ActiveRecordAdapter::VanityExperiment.logger = Logger.new('/dev/null')


Increased max rows to return in CSV export

Coverage increased (+0.0009%) to 95.268% when pulling b2ee8aceec9fad8cb3aaa4c54ad9369cfe6956f2 on cody-csv-max-rows into c5a4ea90e9183d985bc7c2fe3903f38832be2f64 on main.


[Editor] Support resizing editors with the keyboard (bug 1854340)

/botio-linux preview

One quick observation is that it doesn't seem possible to undo (with Ctrl+Z) the resizing done via the keyboard without moving focus away from the resizer elements first, and it's not clear to me if this is intentional behaviour or not?

One quick observation is that it doesn't seem possible to undo (with Ctrl+Z) the resizing done via the keyboard without moving focus away from the resizer elements first, and it's not clear to me if this is intentional behaviour or not?
Edit: This seems to apply to e.g. removing the editor (with Delete) as well.

Yes it's on purpose: when we're entering in resize-with-keyboard mode we can just resize.
The a11y team checked how resizers can be controlled with the keyboard in other softwares and didn't find anything.
So we're more or less experimenting something here, hence the current behavior could be changed in the future.

/botio integrationtest


use browser package to make pub serve work in chrome

I also updated the sdk version, for using caret syntax we need at least 1.6 or something. But just to be save, I used 1.13.0.
Making sure that development is happening on at least 1.13.0 will not hurt anyone I guess.

lgtm

Thanks!


Tmux color scheme?

Would it be possible to get a tmux color scheme, similar to this one based on the vim-gotham theme? https://github.com/shaneog/tmux-colors-gotham

Sure, this is also already a planned port project 😉
You can find more information in the epic issue #18.
You'll also be notified as soon as the specific backlog issue is created so you can track the progress or contribute ideas for this port project.

🚢 Shipped with plugin release version 🏷 0.1.1.


fix: read events directly from resp and not logs

read events directly from resp and not logs

LGTM

addressed via another RPR #228


Network details page error (browser console)

Description
When bringing up a stack composed of two containers and two external networks, the information shown by portainer is not consistent. The details page of one of the networks shows the list of connected nodes. The other does not. The console in the browser thwrows an error.
Steps to reproduce the issue:

Create a directory and paste this docker-compose.yml file:

version: '2'

services:
  gitea:
    container_name: gitea
    image: gitea/gitea
    restart: always
    networks:
      default:
      builds:
        aliases: [ "gitea" ]
    volumes: [ "./data_gitea:/data" ]

  drone-server:
    container_name: drone-server
    depends_on: [ gitea ]
    image: drone/drone
    restart: always
    volumes: [ "./data_drone:/var/lib/drone" ]
    environment:
      - DRONE_NETWORK=net-build

networks:
  default:
    external:
      name: net-ci
  builds:
    external:
      name: net-build

Create the networks:

docker network create net-ci
docker network create net-build

Bring the stack up:

docker-compose up -d

Check the details page of the two containers and the two networks:

[gitea]
net-build 	<IP_ADDRESS> 	<IP_ADDRESS>
net-ci 	<IP_ADDRESS> 	<IP_ADDRESS>

[drone-server]
net-ci 	- 	-

[net-build]
gitea 	<IP_ADDRESS>/16 	-

[net-ci]
- 	- 	-

Check the details page through docker inspect:

[gitea]
"Networks": {
    "net-build": {
        "Gateway": "<IP_ADDRESS>",
        "IPAddress": "<IP_ADDRESS>",
    },
    "net-ci": {
        "Gateway": "<IP_ADDRESS>",
        "IPAddress": "<IP_ADDRESS>",
    }

[drone-server]
"Networks": {
    "net-ci": {
        "Gateway": "",
        "IPAddress": "",
    }
}

[net-build]
"Containers": {
    "1f3ae99c3020933f80704978b74624d9b39c1edfb92b72f3d538d1fbaad4f56d": {
        "Name": "gitea",
        "IPv4Address": "<IP_ADDRESS>/16",
    }
}

[net-ci]
"Containers": {
    "1f3ae99c3020933f80704978b74624d9b39c1edfb92b72f3d538d1fbaad4f56d": {
        "Name": "gitea",
        "IPv4Address": "<IP_ADDRESS>/16",
    }
}

On top of that, when browsing the details page of net-ci the console shows this error:
Error: c is undefined  app.f319a0fb.js:3:22017
j/<@http://localhost:9000/js/app.f319a0fb.js:44:7761
j@http://localhost:9000/js/app.f319a0fb.js:44:7714
k/<@http://localhost:9000/js/app.f319a0fb.js:44:8244
m/</z[b]/B<@http://localhost:9000/js/app.f319a0fb.js:11:26476
h/<@http://localhost:9000/js/app.f319a0fb.js:3:28671
$eval@http://localhost:9000/js/app.f319a0fb.js:4:3995
$digest@http://localhost:9000/js/app.f319a0fb.js:4:2528
$apply@http://localhost:9000/js/app.f319a0fb.js:4:4291
i@http://localhost:9000/js/app.f319a0fb.js:3:11616
u@http://localhost:9000/js/app.f319a0fb.js:3:13832
nb/</y.onload@http://localhost:9000/js/app.f319a0fb.js:3:14308

f/< http://localhost:9000/js/app.f319a0fb.js:3:22017
cb/this.$get</< http://localhost:9000/js/app.f319a0fb.js:3:7927
h/< http://localhost:9000/js/app.f319a0fb.js:3:28754
$eval http://localhost:9000/js/app.f319a0fb.js:4:3995
$digest http://localhost:9000/js/app.f319a0fb.js:4:2528
$apply http://localhost:9000/js/app.f319a0fb.js:4:4291
i http://localhost:9000/js/app.f319a0fb.js:3:11616
u http://localhost:9000/js/app.f319a0fb.js:3:13832
nb/</y.onload http://localhost:9000/js/app.f319a0fb.js:3:14308

Since drone-server seems not to have any assigned IP in none of the nets, I don't expect it to be shown in the details page of any of these. However, gitea should be shown in the details of net-ci just as it is shown in the details page of net-build. I believe that getting rid of the error would allow the page to load completely, and the info should already be there.
Technical details:

Portainer version: 1.15.0
Target Docker version (the host/cluster you manage):

Client:
 Version:      17.09.0-ce
 API version:  1.32
 Go version:   go1.8.3
 Git commit:   afdb6d4
 Built:        Tue Sep 26 22:40:09 2017
 OS/Arch:      windows/amd64

Server:
 Version:      17.09.0-ce
 API version:  1.32 (minimum version 1.12)
 Go version:   go1.8.3
 Git commit:   afdb6d4
 Built:        Tue Sep 26 22:45:38 2017
 OS/Arch:      linux/amd64
 Experimental: true

Platform (windows/linux): Windows 10
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer):

version: '2'

services:
  portainer:
    image: portainer/portainer
    container_name: "portainer-app"
    restart: always
    ports: [ "<IP_ADDRESS>:9000:9000" ]
    volumes: [ "//var/run/docker.sock:/var/run/docker.sock" ]
    networks: [ "net" ]
    command: --host=unix:///var/run/docker.sock

  watchtower:
    image: v2tec/watchtower
    container_name: "portainer-watchtower"
    restart: always
    volumes: [ "//var/run/docker.sock:/var/run/docker.sock" ]
    networks: [ "net" ]
    command: --cleanup portainer-app portainer-watchtower

networks:
  net:
    driver: bridge

Browser: Firefox Version 56.0.1 (64-bit)

Note that this stack is not working correctly, the drone-server service is not configured correctly:
drone-server    | time="2017-10-25T11:25:56Z" level=fatal msg="DRONE_HOST is not properly configured"
drone-server    | time="2017-10-25T11:25:57Z" level=fatal msg="DRONE_HOST is not properly configured"
drone-server    | time="2017-10-25T11:25:57Z" level=fatal msg="DRONE_HOST is not properly configured"
drone-server    | time="2017-10-25T11:25:58Z" level=fatal msg="DRONE_HOST is not properly configured"
drone-server    | time="2017-10-25T11:25:59Z" level=fatal msg="DRONE_HOST is not properly configured"
drone-server exited with code 1

Due to the restart policy being set to always (restart: always) the container keep restarting in bad state.
Note that we can reproduce this issue using the following:

version: '2'

services:
  gitea:
    container_name: gitea
    image: gitea/gitea
    restart: always
    networks:
      default:
      builds:
        aliases: [ "gitea" ]
    volumes: [ "./data_gitea:/data" ]

networks:
  default:
    external:
      name: net-ci
  builds:
    external:
      name: net-build

Note that this stack is not working correctly, the drone-server service is not configured correctly
Due to the restart policy being set to always (restart: always) the container keep restarting in bad state.

Yes. However, I kept it like that (didn't add the DRONE_HOST environment variable on purpose) to reflect that even though a network is listed by docker inspect, it is not connected because it is missing an IP. I don't know if portainer handles this case explicitly or it is an unknown feature.

Yeah, managed to solve it, just wanted to report the issue with the stack.


Binds files instead of values on Input type="file"

Hi, It seems that for Input component of type file, the "bind:files" should be used instead of the "bind:values" to be able to retrieve the files and not the name of the files.

Published as v3.2.8, thanks @VirgileD


refactor(bootstrap): move bootstrap templates to ui-bootstrap folder

I think formly.validation-message.ts should not be in core templates, it should be in ui-bootstrap folder, because its a project specific implementation, and has nothing to do with core functionality.

Except that that looks great to me 👍


⚠ Bofrost.de has degraded performance

In c606de2, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1312 ms

Resolved: Bofrost.de performance has improved in c679ec3 after 6 minutes.


fix for allowing None -> null in executemany

Maybe not the ideal solution to this problem. But this is what allowed me to use "executemany" function with None mapping to null entries. Otherwise it will break.

Maybe not the ideal solution to this problem. But this is what allowed me to use "executemany" function with None mapping to null entries. Otherwise it will break.

Thank you for your contribution. I assume your contribution would be still applicable and useful but could you please add some test to verify the functionality and avoid future regressions of the same problem?

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Dan Zezula seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Closing due to the sunset of the project. Please check active maintained alternative: https://pypi.org/project/hdbcli/


Operation.WaitForCompletion should trace an activity for the entire duration

Though we have DiagnosticScope for REST calls, it would be advantageous to trace, measure, and adjust total polling times for Operation<T> by starting and stopping an activity for the entire duration of WaitForCompletionAsync.
public static async ValueTask<Response<TResult>> DefaultWaitForCompletionAsync<TResult>(this Operation<TResult> operation, TimeSpan pollingInterval, CancellationToken cancellationToken)
    where TResult : notnull
{
    EventSource.SetCurrentThreadActivityId(Guid.NewGuid(), out Guid oldActivityId);
    try
    {
        while (true)
        {
            await operation.UpdateStatusAsync(cancellationToken).ConfigureAwait(false);
            if (operation.HasCompleted)
            {
                return Response.FromValue(operation.Value, operation.GetRawResponse());
            }

            await Task.Delay(pollingInterval, cancellationToken).ConfigureAwait(false);
        }
    }
    finally
    {
        Eventsource.SetCurrentThreadActivityId(oldActivityId);
    }
}

cc @kinelski

@lmolkova :  Your thoughts on whether this fits our OTel strategy would be appreciated.

yes, this aligns with out distributed tracing policy https://azure.github.io/azure-sdk/general_implementation.html#distributed-tracing
We should create one span for public methods that result in 1+ server calls which includes WaitForCompletion. We'd end up with nested UpdateStatusAsync spans under WaitForCompletion, but we have means to suppress them.

@jsquire I am moving this to Azure.Core project? Are you tagging OTel issues with anything special?

Hi @heaths, we deeply appreciate your input into this project. Regrettably, this issue has remained inactive for over 2 years, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support.


Refactoring link pager c

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
✔

Restore pageCssClass, firstPageCssClass, lastPageCssClass, nextPageCssClass, prevPageCssClass, activePageCssClass, disabledPageCssClass short methods

@Gerych1984 please resolve conflicts.


BUG: fix common arg passing

#43 didn't quite fix the issue.
I must have left changes locally (i.e. not pushed up all changes to github).  CI pytest failures are being hidden by the use of the pipe.
Issues

https://github.com/MetOffice/improver_suite/issues/2253

The test coverage has decreased from 84% to 78%...

Ignore this.  Due to ignoring logging integration tests (which don't work in CI right now).


Add support for encrypted XML-based config

Currently only desktop CLR supports this pending EncryptedXml being checked in to Core CLR.
/cc @glennc @blowdart @victorhurdugaci

Hi @GrabYourPitchforks, I'm your friendly neighborhood Microsoft Open Technologies, Inc. Pull Request Bot (You can call me MSOTBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Levi Broderick). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla2.msopentech.com.

TTYL, MSOTBOT;

:shipit: this looks pretty nice and neat!

Looks good :shipit:


🔧 [Refcator] Improvements for Send GDD Form

🔧 Refactor ticket
The layout of the send Gdd form must be improved. The feedback for validation must be improved.

Validation messages below the field.
focus on first field that shows an error after submit
Input text should be black instead of grey

Maximal length for email, ...


Stop transpiling import to require

Unlike packager, webpack has now native support for import and leaving it that way will not only simplify HMR setup but also enable possible code optimisations.

We need to figure out how to make babel-preset-react-native updatable and configurable w/o forking for it to make sense long term.

https://github.com/babel/babel/blob/master/packages/babel-preset-es2015/src/index.js#L42

Going to work on this one

Implemented in @haul-bundler/babel-preset-react-native


DOC: Fix convergence message for remez

Correctly print the error message so as to describe the failure of convergence. Removes the "Design may still be correct" from the error message. Fixes #4569.

Sorry, I had accidentally changed sprintf to printf. Fixed it now. Thanks @rgommers .

Can you squash the two commits and remove [ci skip] from the commit message? This is executable code, so the tests should be run. [ci skip] is really for when you touch docstrings or .txt files.

Oops, I didn't think about that since it was just a change in error message, sorry about that. I squashed the two commits and removed [skip ci] from the commit message. I also renamed the message to MAINT:... instead of DOC:....

Thanks, looks good now. MAINT is right. This can be merged when the testrun has completed.

Merged, thanks @maniteja123


Controls not showing up in HomeKit

Firstly, thank you so much for rewriting the integration! Like others, I can't get the arming/disarming to work on the previous iteration with my Honeywell Vista-20p board. It's working great with this integration and everything is looking good in HA.
The issue I'm seeing is that nearly all the controls are not being exposed into HomeKit. The only control visible in HomeKit is the switch to enable or disable the panel Chime.
None of the sensors show, and neither does the control to actually enable or disable the alarm itself.

Hi, I'm sorry but I don't use HomeKit so don't have much advice for you.  But I'm not aware of anything specific to the integration that would be preventing it.  I believe there are other users have been  using it with HomeKit successfully.  E.g. someone mentions it working for them in #51.

Hey @Stuart-Richards, I ran into this as well.
I'm assuming you're using the HomeKit Bridge integration with GUI config. I was able to use Siri to control and it showed up if I navigated directly to the bridge in HomeKit (Home Settings > Bridges > HASS Bridge), but didn't show up in the room.
I set up a second integration entry with YAML/manual configuration and included the alarm control panel there and everything functions as expected.

@gadgetguyj, Thank you!
Yes, this pointed me in the right direction. I uninstalled and reinstalled the integration. Then a new Bridge showed up. I'm wondering if that new Bridge did show up originally and maybe I dismissed it not realizing it's significance.....  When I added the new Bridge to HomeKit, I see the Alarm control. Just the control, not any zones, but that's certainly good enough for me to use it! I only did the set up in the GUI, didn't create any YAML entries this time. But it's definitely working OK.
Curious why it would show up as a whole new Bridge, rather than be exposed as another Accessory within the existing Bridge, but I'll take it!
Thank you @gadgetguyj  and @ufodone!


I want to use filters in category page So I can limit the products visible in the category page

Context
After opening specific category I want to use filters for narrowing the list of the products to make it easier to find the ones, that I am interested in.
Dependencies

Blocked by #4
Blocked by #7

Acceptance criteria

I can see a list of available filters for the current category.
I can filter by [...] (to split by each filters to another stories?).
When I use a filter then products list gets updated (to extract to another story?).
I can use a few filters at once (to extract to another story?).
I can filter by nested field (not a field of top level object).

Done:

Getting filters from Shopware.
Displaying available filters.

To do:

Applying filters to URL and getting filtered data from Shopware.


Weird routing issue in a very specific scenario

Describe the bug
We've been looking at a very weird issue in YARP's routing almost the entire day and we still haven't quite pinpointed yet what the problem is, but we can consistently reproduce the issue in our test environments.
The problem stems from the fact that we have two routes, which are mostly identical. Here is route1:
{
   "RouteId": "route1",
   "Match": {
     "Path": "/api/{**catch-all}",
   },
   "ClusterId": "api",
   "CorsPolicy": "default",
   "Transforms": [
     {
       "PathRemovePrefix": "/api"
     },
     {
       "X-Forwarded": "Off"
     }
   ]
}

And route 2:
{
  "RouteId": "route2",
  "Match": {
    "Path": "/api/{**catch-all}",
    "Headers": [
       {
         "Name": "Cookie",
         "Values": [
           ".AspNetCore.Cookies"
         ],
         "Mode": "Contains"
        }
      ]
    },
    "ClusterId": "api",
    "AuthorizationPolicy": "CookieAuthentication",
    "CorsPolicy": "default",
    "Transforms": [
       {
         "PathRemovePrefix": "/api"
       },
       {
         "X-Forwarded": "Off"
       }
    ]
}

As you can see both routes are nearly identical, but route2 has an additional match based on Headers. This has been working fine for us so far while running on Windows and mostly using HTTP/1.1. We are now trying to migrate this to run on Linux containers on Kubernetes with HTTP/2 and that's where we are seeing something interesting. In particular route2 never seems to be chosen at least not while sending requests from Chrome and Edge. Sending a request with cURL using the exact same headers though seems to work just fine, which is what makes this such a weird issue.
 So far we haven't been able to figure out what makes these requests different.
To Reproduce
We've tried to create a small repro, but so far haven't been able to. Even with a minimal .NET 6 project with YARP added to it and putting that into a Docker container and doing a request with cURL works just fine.
Further technical details

Include the version of the packages you are using: 1.0.0 on .NET 6.0.1
The platform (Linux/macOS/Windows): Linux (containers, alpine)

To clarify the only reel difference seems to be in the client being used, ie. cURL vs Chrome. Other than that both requests seem to be identical.

Based on offline discussion, @jmezach was able to reproduce the same client-dependent weirdness when there are only these 2 routes in the config.
@Tratcher is there any debug logging they could enable to root cause it?

We've enabled Trace logging for the Microsoft source to see if that gives us any additional pointers to go on. Here is the logging for a Chrome requests:
06 Jan 2022 10:00:29.351
 Request matched endpoint '"route1"'
06 Jan 2022 10:00:29.348
 Endpoint '"route1"' with route pattern '"/api/{**catch-all}"' is valid for the request path '"/api/user"'
06 Jan 2022 10:00:29.348
 Endpoint '"route2"' with route pattern '"/api/{**catch-all}"' is valid for the request path '"/api/user"'
06 Jan 2022 10:00:29.341
 2 candidate(s) found for the request path '"/api/user"'
06 Jan 2022 10:00:29.311
 All hosts are allowed.
06 Jan 2022 10:00:29.311
 Request starting HTTP/2 GET https://services.apollo.zeus.corp/api/user - -

And the cURL requests give me the following logging:
06 Jan 2022 10:11:05.168
 Request matched endpoint '"route2"'
06 Jan 2022 10:11:05.168
 Endpoint '"route1"' with route pattern '"/api/{**catch-all}"' is valid for the request path '"/api/user"'
06 Jan 2022 10:11:05.168
 Endpoint '"route2"' with route pattern '"/api/{**catch-all}"' is valid for the request path '"/api/user"'
06 Jan 2022 10:11:05.167
 2 candidate(s) found for the request path '"/api/user"'
06 Jan 2022 10:11:05.162
 All hosts are allowed.
06 Jan 2022 10:11:05.162
 Request starting HTTP/2 GET https://services.apollo.zeus.corp/api/user - -

As you can see the logs are nearly identical, except that in the first scenario it selects route1 and in the second scenario it takes route2. To be honest, this is starting to drive my crazy ;).

As this thing was driving me crazy we ended up building our own Yarp version with some additional logging in the HeaderMatcherPolicy class. As it turns out it seems that for the Chrome requests the value of the Cookie header is treated as multiple values, while for our cURL requests this does not seem to be the case.
Specifically we've added an else branch to this else if and added a log statement there. For our Chrome requests we hit that else branch, while for the cURL requests we do not.
This is probably not related to YARP, but more of an issue downstream in the runtime somewhere. Maybe @davidfowl has a clue of what's going on here?

That is weird, I wonder what is the difference of the request on the wire. Can you check it out in Wireshark or Fiddler easily?

Triage: It seems the problem is in format of multiple values -- either 1 header with comma-separated values or multiple headers.
We do not currently support multi-value headers matching in the policy regardless of the format. However, this shows it is not clear and super confusing.
@Tratcher will add his comments about matching Cookie headers on top of that (similar to query matching).
We should likely figure out design for multi-value headers matching. We will decide on 1.1 vs. later based on complexity.

You're hitting two issues:

https://github.com/dotnet/aspnetcore/issues/26461, Cookie is supposed to be a single header, but HTTP/2 allows clients to split it and Kestrel doesn't put it back together.
HeaderMatcherPolicy isn't designed for multi-value headers (either multiple headers or comma separated values).
https://github.com/microsoft/reverse-proxy/blob/d9f186a7175dcfb532fe8c57a19082966192ee84/src/ReverseProxy/Routing/HeaderMatcherPolicy.cs#L86-L88

1 will likely be addressed in 7.0. 2 would be expanded functionality in YARP. The interesting question there is what would it take?
Exists should already work.
https://github.com/microsoft/reverse-proxy/blob/d9f186a7175dcfb532fe8c57a19082966192ee84/src/ReverseProxy/Routing/HeaderMatcherPolicy.cs#L77-L85
For each of the others I think we'd want to enumerate each header, as well as split on commas, and then apply the match condition to the individual values. This is a low slower and more expensive, so it might be opt-in.
@jmezach as for workarounds you have two options.
A) use middleware to reformat the cookie headers to a single header
B) use a custom matcher policy specific for cookies. E.g. something like QueryParameterMatcherPolicy that uses the parsed cookie collection rather than sniffing the raw headers.
https://github.com/microsoft/reverse-proxy/blob/main/src/ReverseProxy/Routing/QueryParameterMatcherPolicy.cs

We've added a middleware which basically boils down to the following:
if (context.Request.Headers.TryGetValue(HeaderNames.Cookie, out var requestHeaderValues))
{
    // Only do something if there is more than one value
    if (requestHeaderValues.Count > 1)
    {
        // This might not be the most efficient way of doing this, but let's see if it works first
        context.Request.Headers[HeaderNames.Cookie] = requestHeaderValues.ToString();
    }
}

That seems to resolve our immediate issue for now. We've added some #if's so that when we'll eventually upgrade to .NET 7 we'll be triggered to re-evaluate whether this is still needed or not.
One thing we were wondering though is whether this is the most optimal way to do this. Obviously we're allocating a string here. Maybe there's a more performant way of doing this that we're not aware of?
@Tratcher Thanks for the quick response by the way.

Note cookies should be ; separated, not the default , separated that StringValues.ToString() gives you.
https://datatracker.ietf.org/doc/html/rfc6265#section-4.2.1
The perf should be OK unless you're expecting many/large cookies. You could switch to a StringBuilder.

The behavior that makes some intuitive sense to me would such that multiple request values act as OR.

Exact: Matches if any of the request values exactly matches any of the expected values
Prefix: Matches if any of the request values starts with any of the expected values
Contains: Matches if any of the request values contain any of the expected values
NotContains: Matches if none of the request values contain none of the expected values

Preudo-code:
var tryMatchMode = matchMode == NotContains ? Contains : matchMode;

foreach (var requestValue in requestHeaderValues)
{
    foreach (var expectedValue in expectedHeaderValues)
    {
        if (TryMatch(requestValue, expectedValue, tryMatchMode, comparison))
        {
            return matchMode != NotContains;
        }
    }
}
return matchMode == NotContains;

Examples:

Exact, Foo, new[] { "abc", "def" }
would match
Foo: ab
Foo: def

Contains, Foo, new[] { "def", "ab" }
would match
Foo: bar
Foo: abc

Note: everything here most likely applies to QueryParameterMatcherPolicy as well, as the logic is practically the same.
@Tratcher thoughts?

Foo: ab, def is equivalent to Foo: ab \r\nFoo: def but requires additional parsing. Since that parsing comes at some overhead cost, matching against multi-value headers should be opt-in. How would we indicate that in the API?
This might not apply to queries since those come pre-parsed.

That most likely also means keeping in mind which separator to split on based on the name.

IHeaderDictionary.GetCommaSeparatedValues handles the necessary parsing, including comma splitting and quoted strings. The only header that should need to be special cased is Cookie, if we wanted to support that.
https://github.com/dotnet/aspnetcore/blob/ea2b6d0af43aeaba39243d59d96eedd85089a1a2/src/Http/Http.Abstractions/src/Extensions/HeaderDictionaryExtensions.cs#L42

The only header that should need to be special cased is Cookie, if we wanted to support that.

Cookies is what I had in mind.
Looking at the signature string[] GetCommaSeparatedValues, this doesn't sound ideal - we should be able to avoid substring allocations here without too much trouble.

Looking at the signature string[] GetCommaSeparatedValues, this doesn't sound ideal - we should be able to avoid substring allocations here without too much trouble.

Yeah, but you might want to use it to write the first version and tests before optimizing 😁.


Unable to set Thumb Radius and Thumb Color from XML or Java

Wiki states that :
custom:thumbRadius / setThumbRadius(dim) can be used for thumb radius customisation and
custom:thumbColorNormal / setThumbColorNormal(int) can be used for thumb colour customisation
But I am unable to set any of these from XML or Java code.
I looked into the attrs.xml file in res/values, it doesn't even have these attributes. Neither does the RangeBar class have the stated methods.

+1


Issue 178/2.x capabilities

Remaining work:

[ ] Write e2e-network test for RAFT 2 orgs and Fabric 2.0
[ ] Handle endorsement policy as optional for Fabric 2.0
[ ] Handle --init-required flag for Fabric 2.0


Write upgrade script 1.2 > 1.3 (Old 194)

In gitlab by @A-Lawrence on Aug 31, 2015, 01:09
Write the upgrade script for data conversion between version 1.2. and 1.3
Sadly, this means once we rollout 1.3 there's no going back (so it needs to be error free).

In gitlab by @A-Lawrence on Aug 31, 2015, 01:15
Removed composer pre-install commands.  Also added KohanaUpgrade to ./artisan.  Resolves issue #488535 (194).
ÔåÆ <<cset 56d0f3be571f>>
ALawrence66 - 2015-01-06T19:59:58.345474+00:00

In gitlab by @A-Lawrence on Aug 31, 2015, 01:15
More Kohana upgrade work. Addresses issue #488535 (194).
ÔåÆ <<cset 070742123f34>>
ALawrence66 - 2015-01-05T00:27:52.631128+00:00

In gitlab by @A-Lawrence on Aug 31, 2015, 01:15
Added model dependency builder - sort of addresses issue #488535 (194).  Changed some migrations to make a bit more sense.  Modified timeline action keys and added postmaster migrations (and templates).  Partially addresses issue #191.  Also implemented a blade database compiler and guzzle (as pre-req of the Laravel mail classes).  Finally organised the pre/post commands properly too.
ÔåÆ <>
ALawrence66 - 2015-01-01T17:59:48.202600+00:00


setup/tools

https://github.com/Scony/godot-gdscript-toolkit
https://github.com/Scony/godot-gdscript-toolkit/blob/master/editor-plugins/vscode/README.md

added to doc https://github.com/uralys/fox/blob/master/docs/tips/gamedev.md#other-godot-tools


Update mPDF and include it via composer

1. Why is this change necessary?
mPDF is currently directly bundled into the Shopware repository in version 6.0.0, which is at least 2,5 years old (judging from the tag on GitHub). It was last updated with Shopware 5.1.0 (a4ea89a7fd51d152c9869032f0f290ca1383d20e) and is shipped with custom fonts and a minor config change (see 9f921df4db75d3a6b9ee59695dae0fe20c997f4e). Some changes where made afterwards to backport specific fixes for PHP 7 compatibility (see b560f1110ca07bb3de475d1fd7584e1eb31f82e3 and df075d6a020547cf7ab3440065c5a5a245999ca8).
Unfortunately there is still a problem with PHP 7 and SVG usage on documents (a lot of people use a vector graphic to achieve sharply printed logos on their PDFs) when the server is configured to show E_DEPRECATED errors: PHP Deprecated:  Methods with the same name as their class will not be constructors in a future version of PHP; SVG has a deprecated constructor in  /engine/Library/Mpdf/classes/svg.php on line 33
Also since mPDF version 6.1.0 errors are thrown as exceptions which leads to better usability in the backend when there is not enough disk space or the file permissions are wrong (see https://github.com/mpdf/mpdf/commit/2d02c208b8d7b15c4208517375b9a10b0ea7cefa). The improved output for order documents is pending PR #1250.
2. What does this change do, exactly?
Update mPDF to version 6.1.4 (the current stable release), which has much better compatibility. Also it includes mPDF via composer to clean up the source tree and ease future updates.
We checked carefully for this to not introduce any backwards incompatibility. We first diffed a4ea89a7fd51d152c9869032f0f290ca1383d20e against a original mPDF 6.0.0 zip from http://www.mpdf1.com to make sure there are no hidden changes. We then reapplied all configuration options in a composer compatible way. The exact same 5 fonts are available (all fonts shipped with the mPDF distribution are disabled, as before) and even the backupSIPFont option is still deactivated (even though probably nobody knows what it actually does nowadays).
We also did something very similar for FPDF with #1009.
3. Describe each step to reproduce the issue or behaviour.
Just create an order document with a regular Shopware 5.3 instance, save it and then create one with the PR applied. They will both look 100 % the same, pixel perfect. Fortunately mPDF is not under a lot of development, so this was the expected result
4. Please link to the relevant issues (if any).
n/a
5. Which documentation changes (if any) need to be made because of this PR?
None
6. Checklist

[X] I have written tests and verified that they fail without my change
[X] I have squashed any insignificant commits
[X] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-19759.
Please use this issue to track the state of your pull request.

thanks for your contribution @samuelvogel :+1:

Hi @samuelvogel!
FYI: We'll have to move your PR to Shopware 5.4 - a lot of plugins rely on something like require_once(Shopware()->DocPath() . 'engine/Library/Mpdf/mpdf.php'); (including some Viison plugins) which will break since the mpdf.php-file will have been moved to vendor/shopware. We cannot deliver such a breaking change in a patch release, sorry. 😞

Just add a new file to engine/Library/Mpdf/mpdf.php which is empty and the problem is fixed :D

This sound like one of those workarounds which tend to stay around forever 😆

Looks like this PR is in Shopware 5.4 now, so we are happy 👍
As for our plugins: We have already scoped the FPDF includes to Shopware versions < 5.3 and as soon as a Shopware release includes this change, we will do the same for mPDF.

I really hate to tell you that it is not likely that this change will make it into the 5.4, it is just to short a time for us to really test this through (though you really did an awesome job with that already)!
But I will make sure this is one of the first things being merged for 5.5, you shouldn't have to wait to long for this to hit a release! 💙
Again, thank you so much for that awesome PR (in record time!), I hope we can include it rather sooner than later!

You should the paragraph about this in the Shopware 5.4 upgrade docs then 😢: https://developers.shopware.com/developers-guide/shopware-5-upgrade-guide-for-developers/#mpdf

I am not sure what you mean, mPDF still has been updated to 6.1.4 and the class has been registered in the composer autoloader (although it wasn't required using composer). So the paragraph seems to be correct?

Ok, nevermind, as mPDF was actually updated with Shopware 5.4, most of the benefits of this PR (no PHP 7 errors with SVGs anymore and improved error message) will be shipped with it. And since the require() calls are already gone, the move to a composer dependency (and update to version 7) we did with #1495 will be less breaking.


KeyError: "MaskRCNN: 'UniFormer is not in the backbone registry'"

Thanks for your question. You may not install the environment correctly. Please check the issue here.

As there is no more activity, I am closing the issue, don't hesitate to reopen it if necessary.


Newbrasileirao

woking versions in branchs.
1 - mocked version
2 - monbo version
Main now is working in progress. A new version is comming


Cleanup Redundant JMX Exporter Rules

Currently, many JMX -> Prom exporter's rules are duplicated. This has led to bloating up the config files. This PR removes all such redundant rules.

Were you able to run a performance test to see if there were any resource savings after this change?

I measured the memory being used by java.util.regex.Pattern objects before and after, there's a ~ 33% reduction:
Before:

Total: 64.82K
After:

Total: 43.32K
However, I doubt there's be any significant impact on the container's memory usage considering how little memory these compiled regexps occupy.

@soumitra-st

Did you validate this change (number and name of metrics) using the unit tests and running on an actual cluster?

These have been validated using the unit tests. It'll be very hard, if not impossible, to validate that each of the metric is being exported with the correct names and labels on an actual cluster, as we cannot export all metrics at once (as we cannot execute all the code paths on which these metrics lie on an actual cluster).

Did you find any JMX metric that were not exported to Prometheus?

No, all metrics are being exported.

@soumitra-st I deployed the updated rules on a cluster. I can see metric coming in, especially those for whose the regexps have been deleted.

@Jackie-Jiang @soumitra-st Just a note: Till now, the metric DEDUP_PRIMARY_KEYS_COUNT comes up (incorrectly) as: pinot_server_${partitionId}_Value{database="dedupPrimaryKeysCount",         // table="dedupPrimaryKeysCount.myTable",tableType="REALTIME",}. I have removed the regex that does it and now it is exported with the right name and labels. The reason is that there is a dynamic partitionId in the metric name, so I don't think anyone would've used this metric in either any dashboard or an alert.
Ref:
      if (serverGauge == ServerGauge.DEDUP_PRIMARY_KEYS_COUNT) {
        //this gauge is currently exported as: `pinot_server_${partitionId}_Value{database="dedupPrimaryKeysCount",
        // table="dedupPrimaryKeysCount.myTable",tableType="REALTIME",}`. We add an explicit test for it to maintain
        // backward compatibility. todo: ServerGauge.DEDUP_PRIMARY_KEYS_COUNT should be moved to
        //  gaugesThatAcceptPartition. It should be exported as:
        //  `pinot_server_dedupPrimaryKeysCount_Value{partition="3", table="myTable",tableType="REALTIME",}`
        addPartitionGaugeWithLabels(serverGauge, TABLE_NAME_WITH_TYPE);
        assertGaugeExportedCorrectly(String.valueOf(3),
            List.of(ExportedLabelKeys.DATABASE, serverGauge.getGaugeName(), ExportedLabelKeys.TABLE,
                "dedupPrimaryKeysCount.myTable", ExportedLabelKeys.TABLETYPE, ExportedLabelValues.TABLETYPE_REALTIME),
            EXPORTED_METRIC_PREFIX);

Wondering why you don't need to explicitly specify pattern for other gauges?

@Jackie-Jiang We do have patterns for gauges too, ex:
- pattern: "\"org\\.apache\\.pinot\\.common\\.metrics\"<type=\"ServerMetrics\", name=\"pinot\\.server\\.(realtimeSegmentNumPartitions|luceneIndexingDelayMs|luceneIndexingDelayDocs)\\.(([^.]+)\\.)?([^.]*)\"><>(\\w+)"
  name: "pinot_server_$1_$5"
  cache: true
  labels:
    database: "$3"
    table: "$2$4"

In this, the gauge name comes before the table name. Similarly, we have regexps that match all the gauges.

Is our test good enough to catch all metrics?

It iterates through all the metrics (using data providers) so it is good enough.

Also, do you see a way to make them consistent?

We'll mostly have to fix the gauge naming convention and then change the corresponding regexps.

Can we add comments similar to the broker config to other components?

Sure. Should've marked this as WIP, as I have been pushing changes as-and-when they were done.

@Jackie-Jiang Have addressed the comments.

I think you've fixed some wrong metric rule. Can you add them to the PR description?

@Jackie-Jiang Updated the description with the updated metric rules.


Discrepancy in the number of data sources

Just wanted to draw your attention to this section of the document here.  In the section you state the following, whilst presenting only two.
There are three different sources of data that you can use with Azure OpenAI on your data.

Blobs in an Azure storage container that you provide
Local files uploaded using the Azure OpenAI Studio

Thank you

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

ID: 6317af39-eff1-77b4-8680-9fc06f0d1fdd
Version Independent ID: 60284391-262c-807a-fb8f-9d7422b1926d
Content: Using your data with Azure OpenAI Service - Azure OpenAI
Content Source: articles/ai-services/openai/concepts/use-your-data.md
Service: azure-ai-openai
GitHub Login: @aahill
Microsoft Alias: aahi

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

@michaelspinks
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


SCSS function intellisense insertion point

VSCode Version: Code - Insiders 1.11.0-insider (a3b1076f079c150d921a7b91d91c6783fa04450a, 2017-03-06T07:04:45.166Z)
OS Version: Darwin x64 16.4.0
Extensions:

Extension
Author
Version

code-settings-sync
Shan
2.5.1

sort-lines
Tyriar
1.2.0

html-css-class-completion
Zignd
1.0.3

Bookmarks
alefragnani
0.12.0

project-manager
alefragnani
0.13.3

nord-visual-studio-code
arcticicestudio
0.1.1

path-intellisense
christian-kohler
1.2.0

vscode-eslint
dbaeumer
1.2.7

python
donjayamanne
0.5.9

tslint
eg2
0.8.1

copy-relative-path
enochc
0.1.1

Angular2
johnpapa
2.0.3

vscode-icon-theme
jtlowe
1.5.0

theme-karyfoundation-themes
karyfoundation
11.0.0

restructuredtext
lextudio
0.0.21

HTMLHint
mkaufman
0.3.3

vscode-autoprefixer
mrmlnc
2.0.0

vscode-csscomb
mrmlnc
4.0.1

vscode-stylefmt
mrmlnc
2.2.3

stylelint
shinnn
0.23.0

darcula-extended
smlombardi
3.3.2

slime
smlombardi
1.2.0

theme-tesla
smlombardi
6.0.0

vscode-todo-highlight
wayou
0.4.5

change-case
wmaurer
1.0.0

Probably affects CSS as well.
Steps to Reproduce:

Type a scss function, such as transform: scaleX(), allowing Intellisense to complete the scaleX part for you
the cursor/insertion point is placed after the parens instead of between them.  This means to type a value, you need to place the cursor inside the parens manually.  See gif:

@aeschli
I can repro even with all extensions disabled.


Bump the version to v0.14.0

Coverage remained the same at 100.0% when pulling 4245c9ddbd6e74ffdddebccbff6514d7cf900cf4 on prepare-v0.14.0 into b3c0bdf3135bbb66ae5abc6eb13d07abd0b58055 on master.


[Release] sdk/resourcemanager/liftrqumulo/armqumulo/2.0.0

https://github.com/Azure/sdk-release-request/issues/5324

The breaking changes are introduced and approved in this PR.

/azp run go - armqumulo - ci

The breaking changes are introduced and approved in this PR.

@JeffreyRichter , please approve this major version release by adding label of Approved Major Version, thanks.


Fixes onmessage type

Was using OnMessage in a WebView and noticed it is wrongly typed. This fixes it.

I'm not sure this is correct when reading the docs here: https://facebook.github.io/react-native/docs/webview#onmessage
But I think that this line: https://github.com/reasonml-community/bs-react-native/blob/master/src/RNEvent.rei#L14 should just be a string.
I might be wrong as I haven't done any react-native lately this is just from reading the docs.

I have tested this out and I think that the type on OnMessage is incorrect. I believe this should be the type: https://github.com/reasonml-community/bs-react-native/blob/master/src/RNEvent.rei#L50

I have tested this out and I think that the type on OnMessage is incorrect.
I think the type should be: https://github.com/reasonml-community/bs-react-native/blob/master/src/RNEvent.rei#L50
And then you can use OnMessage like this:
          onMessage={
            event =>
              send(
                OnMessageReceived(
                  event->RNEvent.nativeEvent,
                ),
              )
          }

If you would have the same function and have it as it is then you get the type back but everything on it comes back as undefined. But with this PR I get values on it.
But of course I could be wrong but with the current implementation it is not working and it doesn't work with changing data on the NativeEvent to string because everything comes back as undefined.


avconv subprocess does not respect youtube_dl --proxy setting

Proxy logs and lsof show that the avconv subprocess is not using the proxy configured with --proxy.
(venv)17:45:53+nlevitt@desktop-nlevitt:youtube-dl$ youtube-dl --verbose --no-check-certificate --proxy http://localhost:8000 --write-sub 'http://video.pbs.org/video/2365392401/'
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--verbose', '--no-check-certificate', '--proxy', 'http://localhost:8000', '--write-sub', 'http://video.pbs.org/video/2365392401/']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.06.25
[debug] Git HEAD: 533f67d
[debug] Python version 3.4.0 - Linux-3.13.0-55-generic-x86_64-with-Ubuntu-14.04-trusty
[debug] exe versions: avconv 9.18-6, avprobe 9.18-6, rtmpdump 2.4
[debug] Proxy map: {'http': 'http://localhost:8000', 'https': 'http://localhost:8000'}
[PBS]<PHONE_NUMBER>: Downloading JSON metadata
[PBS]<PHONE_NUMBER>: Downloading recommended_encoding video url info
[PBS]<PHONE_NUMBER>: Downloading m3u8 information
[PBS]<PHONE_NUMBER>: Downloading alternate_encoding video url info
[debug] Invoking downloader on 'http://ga.video.cdn.pbs.org/videos/msu-commencements/5f84513b-5eff-4123-a9e8-ca7c5dab7334/164449/hd-mezzanine-16x9/0884b587_msuco-141213-amcommencement-16x9-hls-2500k.m3u8'
[download] Destination: Baccalaureate Degree Commencement, Fall 2014-2365392401.mp4
WARNING: Your copy of avconv is outdated, update avconv to version 10-0 or newer if you encounter any errors.
avconv version 9.18-6:9.18-0ubuntu<IP_ADDRESS>, Copyright (c) 2000-2014 the Libav developers
  built on Mar 16 2015 13:19:10 with gcc 4.8 (Ubuntu 4.8.2-19ubuntu1)
Input #0, hls,applehttp, from 'http://ga.video.cdn.pbs.org/videos/msu-commencements/5f84513b-5eff-4123-a9e8-ca7c5dab7334/164449/hd-mezzanine-16x9/0884b587_msuco-141213-amcommencement-16x9-hls-2500k.m3u8':
  Duration: 01:44:57.00, start: 10.000000, bitrate: N/A
    Stream #0.0: Video: h264 (Main), yuv420p, 1280x720 [PAR 1:1 DAR 16:9], 29.97 fps, 29.97 tbr, 90k tbn, 59.94 tbc
    Stream #0.1: Audio: aac, 44100 Hz, mono, fltp
Output #0, mp4, to 'Baccalaureate Degree Commencement, Fall 2014-2365392401.mp4.part':
  Metadata:
    encoder         : Lavf54.20.4
    Stream #0.0: Video: libx264, yuv420p, 1280x720 [PAR 1:1 DAR 16:9], q=2-31, 90k tbn, 90k tbc
    Stream #0.1: Audio: aac, 44100 Hz, mono
Stream mapping:
  Stream #0:0 -> #0:0 (copy)
  Stream #0:1 -> #0:1 (copy)
Press ctrl-c to stop encoding
Non-monotonous DTS in output stream 0:1; previous: 0, current: 0; changing to 1. This may result in incorrect timestamps in the output file.
^Came=21229 fps=1662 q=-1.0 size=  239222kB time=708.26 bitrate=2766.9kbits/s
ERROR: Interrupted by user

I couldn't find an option for setting the proxy in avconv/ffmpeg, do you know how to do it?
Using an environment variable probably works, could you try running export http_proxy=<PROXY> before running youtube-dl?

The environment variable works! Thanks, that's great. Maybe youtube-dl can set it when running avconv?

Implemented quite some time ago.


Suffix testnet specific client executables with cli

Let's keep the convention on...

I had a discussion with simon on this
I find the cli very ugly. And not used anywhere else
Mongod and mongocli?
Sshd and sshcli?
Psqld and psqlcli?
I keep it for wasmd as that is familiar, but I prefer nicer names for our testnet. Do you see a big issue with the names? The seem nicer to me

If the choice is intentional, no need to revert. I thought it got slipped somehow.


Richer tab-completion

Current tab-completion just shows available method names, without giving any info on the method signatures.
Ideally, you'd get a full-signature of each method in the tab-completion box. However, if the scalac presentation compiler doesn't support that (?) then it'd be good to get Ammonite like behavior working, where once the method name is complete, then another push of tab, lists the signature for that method.

Note that the BeakerX Scala kernel supports this feature. Maybe some parts of their implementation could be reused.


chore(master): release 1.1.12

:robot: I have created a release beep boop
1.1.12 (2023-06-07)
Miscellaneous

deps: update dependency pestphp/pest to v2.6.3 (cd1f5a7)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-cycle-event-sourcing/releases/tag/v1.1.12 :sunflower:


Update ApplicableCharges.json

Updated the configs for Additional Charges

Closing this PR as new PR is raised for the same https://github.com/odisha-muktasoft/mukta-mdms-data/pull/309


Fix for default renderer

Reviewed 1 of 1 files at r2.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from the review on Reviewable.io


LightGBM installation error_MacOSX10.14_Cmake

gcc -v
Configured with: --prefix=/Library/Developer/CommandLineTools/usr --with-gxx-include-dir=/Library/Developer/CommandLineTools/SDKs/MacOSX10.14.sdk/usr/include/c++/4.2.1
Apple LLVM version 10.0.0 (clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin18.2.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin
Issue:
I followed the installation guide and on 'cmake ..' step got following error:
############
-- The C compiler identification is Clang 4.0.1
-- The CXX compiler identification is Clang 4.0.1
-- Check for working C compiler: /anaconda3/bin/x86_64-apple-darwin13.4.0-clang
-- Check for working C compiler: /anaconda3/bin/x86_64-apple-darwin13.4.0-clang -- broken
CMake Error at /anaconda3/share/cmake-3.12/Modules/CMakeTestCCompiler.cmake:52 (message):
The C compiler
"/anaconda3/bin/x86_64-apple-darwin13.4.0-clang"

is not able to compile a simple test program.
It fails with the following output:
Change Dir: /Users/adityajuvali/LightGBM/build/CMakeFiles/CMakeTmp

Run Build Command:"/anaconda3/bin/make" "cmTC_4d700/fast"
/anaconda3/bin/make -f CMakeFiles/cmTC_4d700.dir/build.make CMakeFiles/cmTC_4d700.dir/build
make[1]: Entering directory '/Users/adityajuvali/LightGBM/build/CMakeFiles/CMakeTmp'
Building C object CMakeFiles/cmTC_4d700.dir/testCCompiler.c.o
/anaconda3/bin/x86_64-apple-darwin13.4.0-clang   -march=core2 -mtune=haswell -mssse3 -ftree-vectorize -fPIC -fPIE -fstack-protector-strong -O2 -pipe  -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX10.14.sdk   -o CMakeFiles/cmTC_4d700.dir/testCCompiler.c.o   -c /Users/adityajuvali/LightGBM/build/CMakeFiles/CMakeTmp/testCCompiler.c
Linking C executable cmTC_4d700
/anaconda3/bin/cmake -E cmake_link_script CMakeFiles/cmTC_4d700.dir/link.txt --verbose=1
/anaconda3/bin/x86_64-apple-darwin13.4.0-clang -march=core2 -mtune=haswell -mssse3 -ftree-vectorize -fPIC -fPIE -fstack-protector-strong -O2 -pipe  -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX10.14.sdk -Wl,-search_paths_first -Wl,-headerpad_max_install_names -Wl,-pie -Wl,-headerpad_max_install_names -Wl,-dead_strip_dylibs  CMakeFiles/cmTC_4d700.dir/testCCompiler.c.o  -o cmTC_4d700
ld: warning: ignoring file /Library/Developer/CommandLineTools/SDKs/MacOSX10.14.sdk/usr/lib/libSystem.tbd, file was built for unsupported file format ( 0x2D 0x2D 0x2D 0x20 0x21 0x74 0x61 0x70 0x69 0x2D 0x74 0x62 0x64 0x2D 0x76 0x33 ) which is not the architecture being linked (x86_64): /Library/Developer/CommandLineTools/SDKs/MacOSX10.14.sdk/usr/lib/libSystem.tbd
ld: dynamic main executables must link with libSystem.dylib for architecture x86_64
clang-4.0: error: linker command failed with exit code 1 (use -v to see invocation)
make[1]: *** [CMakeFiles/cmTC_4d700.dir/build.make:87: cmTC_4d700] Error 1
make[1]: Leaving directory '/Users/adityajuvali/LightGBM/build/CMakeFiles/CMakeTmp'
make: *** [Makefile:121: cmTC_4d700/fast] Error 2

CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
CMakeLists.txt:7 (PROJECT)
-- Configuring incomplete, errors occurred!
#######
Please help. I am not able install even other models due to this same error.

Hi @AdityaJuvali !
As it can be observed from your logs, CMake cannot find you system Clang compiler. Instead, it uses some compiler from Anaconda installation.
-- The C compiler identification is Clang 4.0.1
-- The CXX compiler identification is Clang 4.0.1
-- Check for working C compiler: /anaconda3/bin/x86_64-apple-darwin13.4.0-clang
-- Check for working C compiler: /anaconda3/bin/x86_64-apple-darwin13.4.0-clang -- broken

Try to specify right compilers by the following command:
export CXX=g++ CC=gcc

Or just remove /anaconda3/bin from your PATH variable.

Thank you for the supportive response. I tried this and got another error. It's anaconda again. Could you please help on changing the PATH variable that you mentioned. I am a beginner Data science student and haven't handled MacOSX before.
Error:
-- The C compiler identification is AppleClang <IP_ADDRESS>01044
-- The CXX compiler identification is AppleClang <IP_ADDRESS>01044
-- Check for working C compiler: /Library/Developer/CommandLineTools/usr/bin/gcc
-- Check for working C compiler: /Library/Developer/CommandLineTools/usr/bin/gcc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /Library/Developer/CommandLineTools/usr/bin/g++
-- Check for working CXX compiler: /Library/Developer/CommandLineTools/usr/bin/g++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Error at /anaconda3/share/cmake-3.12/Modules/FindPackageHandleStandardArgs.cmake:137 (message):
Could NOT find OpenMP_C (missing: OpenMP_C_FLAGS OpenMP_C_LIB_NAMES)
Call Stack (most recent call first):
/anaconda3/share/cmake-3.12/Modules/FindPackageHandleStandardArgs.cmake:378 (_FPHSA_FAILURE_MESSAGE)
/anaconda3/share/cmake-3.12/Modules/FindOpenMP.cmake:466 (find_package_handle_standard_args)
CMakeLists.txt:78 (find_package)
-- Configuring incomplete, errors occurred!

@AdityaJuvali Now everything OK with your gcc (Apple Clang actually)! No need to do anything else, like PATH modification.
The current error says you don't have OpenMP on your machine.
Please follow the installation guide (step 2) to install it:
https://lightgbm.readthedocs.io/en/latest/Installation-Guide.html#apple-clang

I have OpenMP on my pc. Please check below:
brew install libomp
Updating Homebrew...
==> Auto-updated Homebrew!
Updated 1 tap (homebrew/core).
==> New Formulae
postgresql@10
==> Updated Formulae
easyengine
Warning: libomp 7.0.0 is already installed and up-to-date
To reinstall 7.0.0, run brew reinstall libomp

It seems that OpenMP is installed correctly.
Are you still getting the following error?
Could NOT find OpenMP_C (missing: OpenMP_C_FLAGS OpenMP_C_LIB_NAMES)

It seems that OpenMP is installed correctly.
Are you still getting the following error?
Could NOT find OpenMP_C (missing: OpenMP_C_FLAGS OpenMP_C_LIB_NAMES)

Yes. I uninstalled everything and followed all your steps. I am getting the same error.

Then try to temporary remove all conda-associated items from your PATH variable.

Then try to temporary remove all conda-associated items from your PATH variable and follow installation guide from the scratch.

you mean, should I uninstall anaconda3?
I don't know how to remove PATH temporarily, please give reference or steps.
Thank you so much for your help. Appreciated. 👍

you mean, should I uninstall anaconda3?

I suppose it's unneeded. Just try to remove all things connected to anaconda from PATH:
https://blog.just2us.com/2011/05/setting-path-variable-in-mac-permanently/

I removed the variable 'anaconda3/bin' from the PATH. It moved ahead but again gave error as:
CMake Error at /usr/local/Cellar/cmake/3.13.1/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:137 (message):
Could NOT find OpenMP_C (missing: OpenMP_C_FLAGS OpenMP_C_LIB_NAMES)
Call Stack (most recent call first):
/usr/local/Cellar/cmake/3.13.1/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:378 (_FPHSA_FAILURE_MESSAGE)
/usr/local/Cellar/cmake/3.13.1/share/cmake/Modules/FindOpenMP.cmake:473 (find_package_handle_standard_args)
CMakeLists.txt:78 (find_package)

Try to brew reinstall libomp

I tried that too. No progress. Please help!

Please provide the full consequence of actions you've performed and the error message.
Also try the following command: brew link openmp

brew link openmp gives following error:
$ brew link openmp
Error: No such keg: /usr/local/Cellar/openmp

Sorry, I've mistyped. Correct command is brew link libomp.

In my environment (macOS 10.14.1), installation error happens.
❯ cmake ..
-- The C compiler identification is AppleClang <IP_ADDRESS>01145
-- The CXX compiler identification is AppleClang <IP_ADDRESS>01145
-- Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc
-- Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++
-- Check for working CXX compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Error at /usr/local/Cellar/cmake/3.13.2/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:137 (message):
  Could NOT find OpenMP_C (missing: OpenMP_C_FLAGS OpenMP_C_LIB_NAMES)
Call Stack (most recent call first):
  /usr/local/Cellar/cmake/3.13.2/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:378 (_FPHSA_FAILURE_MESSAGE)
  /usr/local/Cellar/cmake/3.13.2/share/cmake/Modules/FindOpenMP.cmake:473 (find_package_handle_standard_args)
  CMakeLists.txt:78 (find_package)

-- Configuring incomplete, errors occurred!
See also "/Users/henry/.ghq/github.com/henry0312/LightGBM/build/CMakeFiles/CMakeOutput.log".

Exactly the same error. Did you get success thereafter?

Seems to be that this is Mojave-specific issue, because on Azure Pipelines everything OK with CMake 3.13.2 and libomp 7.0.0:
2018-12-17T05:58:37.6588020Z ==> Downloading https://homebrew.bintray.com/bottles/libomp-7.0.0.high_sierra.bottle.tar.gz
2018-12-17T05:58:38.6903780Z ==> Pouring libomp-7.0.0.high_sierra.bottle.tar.gz
2018-12-17T05:58:40.0097150Z ==> Caveats
2018-12-17T05:58:40.0098320Z On Apple Clang, you need to add several options to use OpenMP's front end
2018-12-17T05:58:40.0098450Z instead of the standard driver option. This usually looks like
2018-12-17T05:58:40.0099000Z   -Xpreprocessor -fopenmp -lomp
2018-12-17T05:58:40.0099050Z
2018-12-17T05:58:40.0099110Z You might need to make sure the lib and include directories are discoverable
2018-12-17T05:58:40.0099270Z if /usr/local is not searched:
2018-12-17T05:58:40.0099310Z
2018-12-17T05:58:40.0099810Z   -L/usr/local/opt/libomp/lib -I/usr/local/opt/libomp/include
2018-12-17T05:58:40.0099940Z
2018-12-17T05:58:40.0100010Z For CMake, the following flags will cause the OpenMP::OpenMP_CXX target to
2018-12-17T05:58:40.0100730Z be set up correctly:
2018-12-17T05:58:40.0101480Z   -DOpenMP_CXX_FLAGS="-Xpreprocessor -fopenmp -I/usr/local/opt/libomp/include" -DOpenMP_CXX_LIB_NAMES="omp" -DOpenMP_omp_LIBRARY=/usr/local/opt/libomp/lib/libomp.dylib
2018-12-17T05:58:40.0101790Z ==> Summary
2018-12-17T05:58:40.0102460Z 🍺  /usr/local/Cellar/libomp/7.0.0: 12 files, 1.2MB
2018-12-17T05:58:42.2522760Z ==> Upgrading 1 outdated package:
2018-12-17T05:58:42.2524200Z cmake 3.13.1 -> 3.13.2
2018-12-17T05:58:42.2524290Z ==> Upgrading cmake
2018-12-17T05:58:42.7773500Z ==> Downloading https://homebrew.bintray.com/bottles/cmake-3.13.2.high_sierra.bottle.tar.gz
2018-12-17T05:58:44.8503970Z ==> Pouring cmake-3.13.2.high_sierra.bottle.tar.gz
2018-12-17T05:58:52.5900040Z ==> Caveats
2018-12-17T05:58:52.5900390Z Emacs Lisp files have been installed to:
2018-12-17T05:58:52.5902670Z   /usr/local/share/emacs/site-lisp/cmake
2018-12-17T05:58:52.5902790Z ==> Summary
2018-12-17T05:58:52.5903440Z 🍺  /usr/local/Cellar/cmake/3.13.2: 5,542 files, 50.6MB
2018-12-17T05:59:03.3040880Z Solving environment: ...working... done

2018-12-17T06:00:35.5450690Z -- The C compiler identification is AppleClang <IP_ADDRESS>20041
2018-12-17T06:00:35.6472470Z -- The CXX compiler identification is AppleClang <IP_ADDRESS>20041
2018-12-17T06:00:35.6588080Z -- Check for working C compiler: /Applications/Xcode_8.3.1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc
2018-12-17T06:00:36.6639250Z -- Check for working C compiler: /Applications/Xcode_8.3.1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc -- works
2018-12-17T06:00:36.6639930Z -- Detecting C compiler ABI info
2018-12-17T06:00:36.6640500Z -- Detecting C compiler ABI info - done
2018-12-17T06:00:36.6640990Z -- Detecting C compile features
2018-12-17T06:00:36.9180440Z -- Detecting C compile features - done
2018-12-17T06:00:36.9243210Z -- Check for working CXX compiler: /Applications/Xcode_8.3.1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++
2018-12-17T06:00:37.0678040Z -- Check for working CXX compiler: /Applications/Xcode_8.3.1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++ -- works
2018-12-17T06:00:37.0971140Z -- Detecting CXX compiler ABI info
2018-12-17T06:00:37.2393120Z -- Detecting CXX compiler ABI info - done
2018-12-17T06:00:37.2495630Z -- Detecting CXX compile features
2018-12-17T06:00:38.0855010Z -- Detecting CXX compile features - done
2018-12-17T06:00:39.0040450Z -- Found OpenMP_C: -Xclang -fopenmp (found version "3.1")
2018-12-17T06:00:39.1810780Z -- Found OpenMP_CXX: -Xclang -fopenmp (found version "3.1")
2018-12-17T06:00:39.1816250Z -- Found OpenMP: TRUE (found version "3.1")
2018-12-17T06:00:39.1890090Z -- Configuring done
2018-12-17T06:00:39.2274230Z -- Generating done
2018-12-17T06:00:39.2394380Z -- Build files have been written to: /Users/vsts/agent/2.144.0/work/1/s/build

Google says that PyTorch users encountered with the same issue last month.
@henry0312 Do you have any guesses what can be wrong?

Xcode version doesn't matter too. Just run it with Xcode 10.0:
-- The C compiler identification is AppleClang <IP_ADDRESS>01145
-- The CXX compiler identification is AppleClang <IP_ADDRESS>01145
-- Check for working C compiler: /Applications/Xcode_10.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc
-- Check for working C compiler: /Applications/Xcode_10.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /Applications/Xcode_10.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++
-- Check for working CXX compiler: /Applications/Xcode_10.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found OpenMP_C: -Xclang -fopenmp (found version "3.1")
-- Found OpenMP_CXX: -Xclang -fopenmp (found version "3.1")
-- Found OpenMP: TRUE (found version "3.1")
-- Configuring done
-- Generating done
-- Build files have been written to: /Users/vsts/agent/2.144.0/work/1/s/build

https://lightgbm-ci.visualstudio.com/lightgbm-ci/_build/results?buildId=1120 (MacOS regular log).

CMake can't find OpenMP on Mojave (#18520) · Issues · CMake / CMake · GitLab

@henry0312 Can you please try gcc on Mojave? Is everything OK with it?

Can you please try gcc on Mojave? Is everything OK with it?

Yes, there is no problem with gcc-8.

UPD2: Or can we temporary overcome this issue by manual editing LD_LIBRARY_PATH, like it was done here?

If I really try to fix CMakeLists.txt, this issue may be solved.

@henry0312 The current OpenMP part of CMakeLists.txt is quite simple and elegant:
https://github.com/Microsoft/LightGBM/blob/2323cb3befa0515dd266d4a1393a37c833c11b83/CMakeLists.txt#L77-L86
https://github.com/Microsoft/LightGBM/blob/2323cb3befa0515dd266d4a1393a37c833c11b83/CMakeLists.txt#L212-L217
If I'm not mistaken, a hotfix from our side can be some path hardcoding only. I suppose that this issue should be fixed from CMake side actually.

I suppose that this issue should be fixed from CMake side actually.

I hope so, however, the issue doen't seem a priority.

from https://gitlab.kitware.com/cmake/cmake/issues/18520.

@henry0312 Yeah, you're right (unfortunately)!
Hey, what about proposing a PR to their repo? Or at least leave a comment with patch in the issue.

what about proposing a PR to their repo?

Yes, I'll do.
However, firstly, I must look up the cause and then fix the issue, because I think CMake doesn't accept dirty work-around.

Oh Great work! The cmake step worked finally but 'make -j4' threw an error. The results for cmake are-
-- The C compiler identification is AppleClang <IP_ADDRESS>01044
-- The CXX compiler identification is AppleClang <IP_ADDRESS>01044
-- Check for working C compiler: /Library/Developer/CommandLineTools/usr/bin/gcc
-- Check for working C compiler: /Library/Developer/CommandLineTools/usr/bin/gcc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /Library/Developer/CommandLineTools/usr/bin/g++
-- Check for working CXX compiler: /Library/Developer/CommandLineTools/usr/bin/g++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found OpenMP_C: -Xpreprocessor -fopenmp -I/usr/local/opt/libomp/include (found version "3.1")
-- Found OpenMP_CXX: -Xpreprocessor -fopenmp -I/usr/local/opt/libomp/include (found version "3.1")
-- Found OpenMP: TRUE (found version "3.1")
-- Configuring done
-- Generating done
-- Build files have been written to: /Users/adityajuvali/LightGBM/build

Make errors-
https://docs.google.com/document/d/1ruD9w-rycWlJJGx71alkfOX-I8mDIEapfbOQzx1ldpc/edit?usp=sharing

@AdityaJuvali I'm sorry I forgot I fixed CMakeLists.txt.
Can you edit CMakeLists.txt like this:
diff --git a/CMakeLists.txt b/CMakeLists.txt
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -211,8 +211,8 @@ endif(USE_MPI)

 if(USE_OPENMP)
     if(CMAKE_CXX_COMPILER_ID STREQUAL "AppleClang")
-        TARGET_LINK_LIBRARIES(lightgbm ${OpenMP_libomp_LIBRARY})
-        TARGET_LINK_LIBRARIES(_lightgbm ${OpenMP_libomp_LIBRARY})
+        TARGET_LINK_LIBRARIES(lightgbm OpenMP::OpenMP_CXX)
+        TARGET_LINK_LIBRARIES(_lightgbm OpenMP::OpenMP_CXX)
     endif()
 endif(USE_OPENMP)

and then re-try cmake and make.


master - Undefined method httpBuildQuery

Hello. I think due to some recent change to AbstractProvider, the httpBuildQuery method doesn't exist anymore in master. As a result, this Google provider no longer works.
Thanks,
Patrick

Merged into #5.


[GHES] GitHubTokenExchangeError: Internal Server Error

Type: Bug
Logging in to a self-hosted GitHub Enterprise server from the GitHub sidebar results in "Having trouble logging in? Would you like to try a different way?" and finally "Error signing in to GitHub Enterprise".
Logs
2022-12-09 14:31:11.399 [info] Reading sessions from keychain...
2022-12-09 14:31:11.406 [info] Getting sessions for read:user,repo,user:email,workflow...
2022-12-09 14:31:11.406 [info] Got 0 sessions for read:user,repo,user:email,workflow...
2022-12-09 14:31:11.437 [info] Getting sessions for read:user,repo,user:email...
2022-12-09 14:31:11.437 [info] Got 0 sessions for read:user,repo,user:email...
2022-12-09 14:31:11.454 [info] Getting sessions for read:user,repo,user:email,workflow...
2022-12-09 14:31:11.454 [info] Got 0 sessions for read:user,repo,user:email,workflow...
2022-12-09 14:31:17.161 [info] Logging in for the following scopes: read:user repo user:email workflow
2022-12-09 14:31:17.170 [info] Trying without local server... (read:user repo user:email workflow)
2022-12-09 14:31:27.736 [info] Exchanging code for token...
2022-12-09 14:31:28.467 [error] GitHubTokenExchangeError: Internal Server Error
	at t.GitHubServer.u (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/github-authentication/dist/extension.js:2:1309015)
	at process.processTicksAndRejections (node:internal/process/task_queues:96:5)
	at async /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/github-authentication/dist/extension.js:2:1304423
2022-12-09 14:31:36.464 [info] Trying with local server... (read:user repo user:email workflow)
2022-12-09 14:31:38.721 [info] Exchanging code for token...
2022-12-09 14:31:39.081 [error] GitHubTokenExchangeError: Internal Server Error
	at t.GitHubServer.u (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/github-authentication/dist/extension.js:2:1309015)
	at process.processTicksAndRejections (node:internal/process/task_queues:96:5)
	at async /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/github-authentication/dist/extension.js:2:1305597
2022-12-09 14:31:46.451 [info] Trying device code flow... (read:user repo user:email workflow)
2022-12-09 14:31:47.176 [error] Error: Failed to get one-time code: Cookies must be enabled to use GitHub.
	at t.GitHubServer.p (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/github-authentication/dist/extension.js:2:1305870)
	at process.processTicksAndRejections (node:internal/process/task_queues:96:5)
	at async t.GitHubServer.login (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/github-authentication/dist/extension.js:2:1303463)
	at async t.GitHubAuthenticationProvider.createSession (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/github-authentication/dist/extension.js:2:1299643)

Extension version: 0.57.2022120809
VS Code version: Code 1.74.0 (Universal) (5235c6bb189b60b01b1f49062f4ffa42384f8c91, 2022-12-05T16:43:37.5

I see a similar error

Extension version - 0.56.0
VS Code version - 1.74.1
GHES version - 3.4.9


White space at right and botom of screen

I'm using your module in portrait mode with two different monitors.  One is a 4k monitor, the other a 1920x1080 monitor.
Both are displaying the same white space at the right side and bottom of screen.
When I use the wallpaper module, I do not have this issue.
I am also using the fast-css module for formatting, so maybe the white color is coming from that, but I don't have the same issue with the wallpaper module

I'm not seeing this effect on my installs so I think you're probably right about this being caused by a conflict with another module. I'm not familiar with the fast-css module, but if you're using it to change the css/formatting of your mirror that does seem like the thing likeliest to cause the bug you describe.
It's hard to give advice without knowing what your config.js looks like (and what other modules you have installed), but you could try starting with only the WallberryTheme module enabled, and then enable your other modules one-by-one to see if you can isolate which modules are introducing the conflict.

I'm not seeing this effect on my installs so I think you're probably right about this being caused by a conflict with another module. I'm not familiar with the fast-css module, but if you're using it to change the css/formatting of your mirror that does seem like the thing likeliest to cause the bug you describe.
It's hard to give advice without knowing what your config.js looks like (and what other modules you have installed), but you could try starting with only the WallberryTheme module enabled, and then enable your other modules one-by-one to see if you can isolate which modules are introducing the conflict.


actions: use load_data to get the old value in flashback

Signed-off-by: JmPotato<EMAIL_ADDRESS>What is changed and how it works?
Issue Number: ref #13303.
What's Changed:
Use `load_data` to get the old value in flashback to prevent unnecessary `seek_write`.

Check List
Tests

Unit test
Integration test

Release note
None.

/cc @Connor1996 @HuSharp

/merge


Update README.md

added CF deprecation notice

Removed references to CF. Also fixed a few CLI terminology things.
And we'll do a rewrite for Code Engine under a different issue and PR.