translators/stack-lock: Use nix-community/all-cabal-json

Some things I'm not sure of:

Do we additionally want to use all-cabal-json for the top-level
file? This would have the drawback of yielding outdated data for a
developer of a Haskell project who wants to use dream2nix to
manage their project. I'm not sure if dream2nix is really
designed for that use though.
Do we want to continue using all-cabal-hashes in the stack-lock
translator? We end up copying a Cabal file from all-cabal-hashes
in the builder anyway. Though I'm not sure why we don't simply use
the Cabal file from the source tree.

Some things I'm not sure of:
1. Do we additionally want to use `all-cabal-json` for the top-level
   file? This would have the drawback of yielding outdated data for a
   developer of a Haskell project who wants to use `dream2nix` to
   manage their project. I'm not sure if `dream2nix` is really
   designed for that use though.

2. Do we want to continue using `all-cabal-hashes` in the `stack-lock`
   translator? We end up copying a Cabal file from `all-cabal-hashes`
   in the builder anyway. Though I'm not sure why we don't simply use
   the Cabal file from the source tree.

I think for the top-level package we should only use the actual cabal file from the source. I think using dream2nix on in-development projects should be supported and will be one of the major use-cases. Though for information about transitive deps we still need the cabal files from all-cabal-json (see answer 2.).

The cabal file from the source tree will only provide us with the dependency names of direct dependencies. In order to find transitive dependencies we need to consult the cabal files of our direct deps. Therefore we need either all-cabal-hashes or all-cabal-json in the translator as well.

I actually forgot about the fact that the builder uses the cabal files from all-cabal-hashes as well to replace the one coming from the package. Though it should only do that for dependencies. Not for our top-level package. If that is not the case, we should fix it.
Also I think, eventually we should only use one of the all-cabal-... repos. I think we should update all-cabal-json to also contain the cabal files. Then we can get rid of all-cabal-hashes.

Merging for now as this is already a huge improvement. Feel free to do further improvements in another PR.


Missing all of these "C# editing support, including Syntax Highlighting, IntelliSense, Go to Definition, Find All References"

I don't see the C# editing support, including Syntax Highlighting, IntelliSense, Go to Definition, Find All References.

@mahadreamz Sorry you are having trouble. Can you please share the OmniSharp Log from the Output pane?

Thanks for the reply. I don't see any log in there. I have installed this extension for my C# project. When I expected this language support such as Go to definition, Find all references, e.t.c they all not appearing when I right the code area or even thru the command palette. I'm not sure what I'm missing.

@mahadreamz This seems odd. There should be a log in the OmniSharp output window. Have you opened a folder that contains a C# project or solution?

@JoeRobich It has nothing.. Just to make sure I'm giving you correct info.. I did not try to work on omnisharp project.. I'm saying the issue I see is while using this extension in my c# solution.

@mahadreamz I would have expected there to be logging such as this:

Since you have opened a folder containing your C# solution but do not see any logging for OmniSharp in the Output pane, I can only assume it might be an installation failure. Could you try reinstalling the extension to see if that resolves the issue?

Thank you.. I see this..

@JoeRobich  I updated the path variable (https://github.com/OmniSharp/omnisharp-vscode/issues/32#issuecomment-241223579) and it worked. Omnisharp started successfully. Please correct me if you think otherwise.

@mahadreamz  Very odd that you would have to manually set the omnisharp.path. Did your solution load properly with your change?

@JoeRobich  yes it did.


Bundle size is more

:tada: This issue has been resolved in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Reduces 25% of bundle size<EMAIL_ADDRESS>


Update version to 0.16

Platforms Impacted

[ ] iOS
[ ] macOS

Description of changes
Updates to release 0.16

d9ae100360c29ab66860ee2030ba1b4e1ee028fe
4193ed501c65c115f33d2595cc0495e4346e16d6
d85d4eec15339ee2b4c84ee5f055bfa4dfcf9532
9a33e4a9e2a96980d5731361c5a4827c25d2f9cb
b697a48e88fba0f62700e0a3a92eb981fc553498

Binary change
(how is our binary size impacted -- see https://github.com/microsoft/fluentui-apple/wiki/Size-Comparison)
Verification
(how the change was tested, including both manual and automated tests)

Visual Verification

Before
After

Screenshot or description before this change
Screenshot or description with this change

Pull request checklist
This PR has considered:

[ ] Light and Dark appearances
[ ] iOS supported versions (all major versions greater than or equal current target deployment version)
[ ] VoiceOver and Keyboard Accessibility
[ ] Internationalization and Right to Left layouts
[ ] Different resolutions (1x, 2x, 3x)
[ ] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)
[ ] iPad Pointer interaction
[ ] SwiftUI consumption (validation or new demo scenarios needed)
[ ] Objective-C exposure (provide it only if needed)

Microsoft Reviewers: Open in CodeFlow

Might need a separate PR to fix linting issues?


Connection quality should be measured

Part of the opt-in to metrics is that the user can be given a sense of connection quality.
We can likely use mlab for this, which already has built a native JS speed test client:
https://github.com/m-lab/ndt-javascript-client
We would run that through the proxy, and report experienced speeds. (or even bucket it into 'high latency', 'Sufficient for browsing', 'Video capable!')
should work with mlab to figure out how to tag our connections so that they don't mistake these tests as being the real speed of the proxy. Should also ask them about the impact of a cloud-front to protect our user IPs from ending up in mlab's database. - in particular how much that would degrade their test results.
--Will

I doubt the networking team will have time for it in this sprint but I think this is an interesting idea.
The only thing I would say is that reminds me a little of the "uproxy should block ads!"-type suggestions. We've decided against those in the past because there already exist many browser extensions to that. In this case there probably aren't any extensions that do this but I think the same reasoning might apply.

I think this could be considered a duplicate of:
https://github.com/uProxy/uproxy/issues/1242


Filter motion type on camera/doorbell camera events

This is obviously not a bug, but an enhancement request.  I have no idea what data is provided by the camera/doorbell events, but if it does have motion types, e.g. motion, person, package, etc. I was wondering if it would be possible to filter by motion type at some point.  I'd like to only send motion events to Homekit that my nest doorbell comera identifies as a person.  I am using this to create a Homekit automation to turn on my outside lights.  It seems I cannot select specific motion types in the automation.

The SDM API supports three types of camera events: Motion, person, and sound.  The first two are reported to Homebridge as motion.  The third is not reported.
As far as I can tell there is no way to tell Homebridge that the motion is caused by a person, just that there was motion https://developers.homebridge.io/#/service/MotionSensor. If you know of a way please let me know.

The SDM API supports three types of camera events: Motion, person, and sound.  The first two are reported to Homebridge as motion.  The third is not reported.
As far as I can tell there is no way to tell Homebridge that the motion is caused by a person, just that there was motion https://developers.homebridge.io/#/service/MotionSensor. If you know of a way please let me know.

I figured Nest was just sending a motion event and HomeKit determined the type. Worth a shot.  Thanks for the response. This can be closed as far as I'm concerned.


Gateway-required Vnet Integration

Couple of questions specific to Gateway-required Vnet Integration

Is the Azure Web App connection to the Azure resources behind the Gateway-required Vnet Integration using a public or private IP?

Can you Global Vnet Peer the Vnet which has the Gateway-required Vnet Integration, so that the Web App has access to resources in that globally peered Vnet?

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

ID: a7a98803-1438-b1b5-f543-7dd88bc4294e
Version Independent ID: 37ff1d0f-ed8e-5e4d-1f4c-1b9f6cffb938
Content: Integrate app with Azure Virtual Network - Azure App Service
Content Source: articles/app-service/web-sites-integrate-with-vnet.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

Hi @hiteshvadgama,
Thank you for the detailed question. We are currently investigating and will update you when we have accurate information to provide.

Hello @hiteshvadgama , thanks for your patience here are the answers to your question:

Public. You can provision the gateway and see the IP on it.

You can even connect directly to that VNet in another region.    There are limitations though.  The feature won’t work with ExpressRoute.  If you have ExpressRoute on the VNet or in any peered VNet, it won’t work.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


is it possible to use my actual navbar with sidr?

this is the site under construction:
http://thelittleman.es/
In big screen you can see the left menu correctly, I want it to collapse when its seen on mobile and to bring the navbar in a mobile facebook style. How can I do it? I´ve tried whithout success.

Thanks @robertbossaert and @momekh


how to add google pay feature in initPaymentSheet useStripe()

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

https://stripe.com/docs/payments/accept-a-payment?platform=react-native&ui=payment-sheet#react-native-google-pay


Code formatting

I'd appreciate if you could provide a way to reproduce the existing code formatting style of probeye, maybe in a CONTRIBUTING.md file (e.g. here). Otherwise, I expect the pull requests from collaborators (including me) to be either manually (=inconsistently) formatted or the diff to be cluttered with format changes.
black is a common choice in python projects, but -- from my testing -- differs from the existing style (I assume pycharm). One way could be to integrate black into pycharm. Providing a way to reproduce the pycharm formatting with open source tools would also be fine.

I'm with @TTitscher :).
How about if we tried to write up @aklawonn's personal code formatting rules with one or more of the existing code checking and formatting packages? So things would be consistent and @aklawonn could still do the formatting manually and @TTitscher would have the option to do it automatically.
Maybe at this stage the formatting does not need to be enforced so it will not scare away contributors. (On the long run I'm in favor of having a bot automatically reformatting any non-compliant code and not allowing merging or pushing to the main if the formatting rules are not met.)


inlay hints not working for stm32xx hals.

rust-analyzer does not provide inlay hints when working with stm32xx hals. Very similar to issue #6504. Perhaps the token limit needs to be bumped again? I provide more detail for what my issue in #6029. I think my comment may have been misplaced which is why I am opening up this issue.

Not sure why 2 of these got created. See #7258

Not sure why 2 of these got created. See #7258


doc(README): update roadmap and contributing guides

update readme with project roadmap and contributing guides
add troubleshooting doc info

Signed-off-by: Akhil Mohan<EMAIL_ADDRESS>

Codecov Report

Merging #607 (b727186) into master (b2102cc) will decrease coverage by 0.10%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #607      +/-   ##
==========================================
- Coverage   47.71%   47.60%   -0.11%
==========================================
  Files          79       79
  Lines        3720     3720
==========================================
- Hits         1775     1771       -4
- Misses       1786     1789       +3
- Partials      159      160       +1

Impacted Files
Coverage Δ

cmd/ndm_daemonset/probe/udevprobe.go
61.74% <0.00%> (-2.19%)
:arrow_down:

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 b2102cc...b727186. Read the comment docs.


Feature/simple oauth

@morehavoc I have created a new PR so that you can review my changes to the OAuth flow.
I attempted to implement some of the ideas we discussed previously. What I discovered, however, is that JSRuntime is essentially never ready for calls until after a view has been initialized and rendered. Since we are using the JS IdentityManager, this means we simply can't "pre-load" authentication.
I added code in MapView that will trigger oauth initialization, and optionally trigger a login on first view render. However, you can also still call the methods you created in OAuthAuthentication directly if you are doing other work and don't want to show a map.
I created a Server OAuth sample to verify that the current pattern works for both Wasm and Server. The AddScoped method in DI works well for both, because there is only one effective scope in blazor WASM.
Unfortunately, I hit some roadblocks with a MAUI implementation. https://github.com/dotnet/maui/issues/2702 documents how OAuth is essentially broken for MAUI Windows/WinUI. Interestingly Morton from Esri commented on this with a workaround, probably what the ArcGIS .NET SDK team is using. I think it might make sense to have a MAUI-specific implementation in the future that uses the ArcGIS solution. But in the meantime, I'm comfortable with adding OAuth support minus MAUI as long as we document it as such.

@morehavoc I did another serious refactor, moving everything into AuthenticationManager to allow for handling both ApiKey and OAuth together, and support the idea of GetCurrentToken. If you have a chance to re-review this, please do.


add only tag for ruby sdk

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

add for ruby sdk generation
Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-go

Automation for azure-sdk-for-python

Can one of the admins verify this patch?


[FEATURE REQ] Include SessionId in metadata bindings for ServiceBusTrigger on .NET 5 Isolated

Library or service name.
I'm using Microsoft.Azure.Functions.Worker.ServiceBusTrigger on .NET 5 Isolated.
Is your feature request related to a problem? Please describe.
As far as I can tell, there is no way to get the session id of a queue session. Creating feature request based off stackoverflow answer here.

Related feature request but doesn't quite fit the requirements here https://github.com/Azure/azure-sdk-for-net/issues/24358. took over 2 years for this one so I wouldn't be holding my breath

Yes its frustrating that the sessionId can't be retreived.

Related https://github.com/Azure/azure-sdk-for-net/issues/27120


Update body_controller.h for xy-axis control

Add xy control posibilities.

@tarasborov thanks for sending the PR. Just wondering, are you able to just call translate function once and pass x, y, z rather than calling translate every axis update?

@tarasborov thanks for sending the PR. Just wondering, are you able to just call translate function once and pass x, y, z rather than calling translate every axis update?

@grassjelly You are right, the translation function should be call once. I have commited changes. Thanks.

thanks @tarasborov


Program freezes when trying to save plotly.express.box to a PNG via write_image

I get this issue where the program freezes when trying to save a plotly.express.box via write_image into a .PNG file.
from PIL import Image, ImageFont, ImageDraw
import pandas as pd
import numpy as np
import plotly.express as px
from plotly.io import write_image
confluency_df = pd.DataFrame([[f"Directory 0", np.random.uniform()*100] for i in range(10)], columns=["directory", "confluency (%)"])
plotly_fig = px.box(
        confluency_df,
        x="directory",
        y="confluency (%)",
        boxmode="overlay",
        points="all",
    )
plotly_fig
plotly_fig.write_image("src/static/results/confluency_graph.png")

The strange thing is it seems to freeze the whole program without throwing an error. All steps work fine up to the last line above.
I am using the following:
plotly==5.8.0
kaleido==0.2.1

Thanks for your interest in Kaleido. We are currently working on an overhaul that might address your issue - we hope to have news in a few weeks and will post an update then. Thanks - @gvwilson


Prevent use of :: for dosbatch comments by default

This patch is to disable (in fact highlight as an error) comments using the :: idiom in batch files as it can use problems on modern Windows systems. For anyone still working with good old MS-DOS (or equivalents) where the idiom still works fine you can set a variable in the vimrc file to support them as before.
Rolled in some miscellaneous fix-ups to the doc and syntax file.

I'll include it, thanks.


Introduce CannotLead

Before when a node had an outdated (or conversely not-yet-active) it had no way to indicate that it was a leader but that it couldn't lead before the key was wrong. We now distinguish these two cases, although we don't yet take advantage of this. We should: both PBFT and TPraos should check if the key matches the delegation state, and if not, say that they cannot lead, rather than produce an invalid block.

bors merge


Ensure correct raiden version is pulled for test harness

Currently, CircleCI does not pull the correct raiden version for our test-harness, resulting in its failure.
Example:
https://circleci.com/gh/raiden-network/scenario-player/2369?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

We should now be running against the latest develop commit, instead of what's pinned in pyproject.toml.


[Question] What is the purpose of FileStream::Mode::OVERLAPPED

From #413 there is a change to the DownloadTaskCurl:
_fsMd5 = FileUtils::getInstance()->openFileStream(_checksumFileName, FileStream::Mode::OVERLAPPED);
I did notice the O_TRUNC flag is missing from FileStream::Mode::OVERLAPPED, but what is the difference in behavior between the FileStream::Mode::WRITE and FileStream::Mode::OVERLAPPED?
The reason for asking this is that I need to ensure that the VFS implementation of FileStream that I'm using replicates the behavior of the FileStream::Mode::OVERLAPPED option.

The mode WRITE will always trucate file size to zero when open.
The OVERLAP behavior same with 'APPEND'+'SEEK TO SET', it's useful to seralize and update md5 state realtime and resume file download at breakpoint

Thank you for the explanation!


docker-receive: Fix converting AUFS whiteouts

Is there any reasonable way around this? CAP_SYS_ADMIN is a massive sledgehammer.

We could add the list of opaque directories to the layer configuration, and then insert a layer after the given layer at mount time which has the OverlayFS whiteout dir.

Okay, let's do it this way for now and then we can sort it out when we have a Go package that can write out squashfs filesystems.


Helm Improvements

lgtm

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


Extract compressed pixel data without profile

Question about an existing feature
What are you trying to achieve?
Our service creates PDF files with embedded images. We'd like to embed 3 images with the same color profile in a PDF, without embedding the color profile 3 times in the PDF.
We first use Sharp to convert all images to a specific color profile: U.S. Web Coated (SWOP) v2. Next, our PDF generation service embeds the ICC profile, reads the 3 images into Buffers using readFile, and tags them with the ICC profile embedded in the PDF. This all works like a charm.
However, since we're using readFile to get the file buffer, it will also include the embedded ICC profile. Since our goal is to reuse the profile for all 3 images, we'd like to extract the compressed pixel data (without the ICC profile).
Sharp allows us to access the uncompressed pixel data using raw, but we'd like to access the compressed pixel data as it would allow us to embed the image stream as-is, without any reprocessing. Sharp's toBuffer converts to the device-independent sRGB colour space, and withMetadata allows to convert to a specific ICC profile.
Is there any way we can access the already compressed pixel data for usage in PDF files where we manually control which ICC profile is to be (re)used for the image streams?
Please provide sample image(s) that help explain this question
colorspace-cmyk-swop.jpg.zip

The forthcoming v0.32.0 will include a new ignoreIcc option (commit https://github.com/lovell/sharp/commit/42d2f07e447b928eedf5d42edc6fac4ec7278ee7) for use on the input side that might help, although you'll need to add a second pipeline to ignore/remove the profile added by the first pipeline.
If the concern is filesize related, perhaps you could use a smaller CMYK profile e.g. https://github.com/saucecontrol/Compact-ICC-Profiles#cmyk is close to SWOP.

I've investigated further now the ignoreIcc option is available. I'm not able to re-open this issue so please let me know if you prefer me to open a new issue.
It seems the ignoreIcc option is only intended to not take into account the embedded profile in the processing pipeline.
I'm looking for a way to access the compressed pixel data of a given image without any image processing. For example fogra39.jpg is an image in CMYK color space with Color profile Coated FOGRA39.
We have 3 images with the same profile embedded and are looking to embed them in a PDF file. This PDF is created with 3rd party software that allows to embed the color profile (FOGRA39) once, and reference the same profile in 3 ICC based XObject image streams. This currently works, but the image streams contain the profile as well, since we simply each file into a buffer (also using the 3rd party software).
I thought to use Sharp's toBuffer method in combination with the ignoreIcc option to access the original image data without any color profile attached. Essentially stripping out the color profile without reprocessing any image data.
Would that be possible with the ignoreIcc option?

A decode/encode roundtrip will always occur.
To both ignore the input CMYK profile, ensure everything remains CMYK and output a profile-less CMYK image, try something like:
await sharp('fogra39.jpg', { ignoreIcc: true })
  .pipelineColourspace('cmyk')
  .toColourspace('cmyk')
  .toFile('out.jpg');

Thank you for the clarification. In your example, a new file is created that uses a Generic CMYK profile.
My goal is to separate the ICC profile data to embed it in a PDF, and reuse it multiple times by adding profile-less image images to the PDF that reference the already attached profile.
Is there any way I can get the (reencoded) file from your example, but without having the profile attached in the metadata?

In your example, a new file is created that uses a Generic CMYK profile.

Here's the metadata of the output when I tried this locally, which describes a profile-less CMYK JPEG image, so it works for me.
 {
  format: 'jpeg',
  width: 2889,
  height: 2063,
  space: 'cmyk',
  channels: 4,
  depth: 'uchar',
  density: 72,
  chromaSubsampling: '4:4:4:4',
  isProgressive: false,
  hasProfile: false,
  hasAlpha: false
}


Script stops after website can't be reached.

Sometimes Volafile gets a little downtime, servers are restarting i think. This make this script crasch, should'nt there be a "self-restart" feature? If the server is down, the script waits for 3-4~ minutes and restart itself. I know zero coding, but i have tried to implement this on my own with while True, unfortunatly without success.
[0908/190442.285:ERROR:gpu_process_transport_factory.cc(1007)] Lost UI shared co
ntext.
DevTools listening on ws://<IP_ADDRESS>:12660/devtools/browser/e7bfa122-d0e3-4596-9
394-88ddcef5a10c
?[31m[-] The Website might be offline or another error occurred: Message:
?[0m
F:\volafile-downloader-master\volafile-downloader>

Hi Kristoffer, I will try to implement it this weekend :)

Awesome! Are you by any chance, adding the possibility to enter password protected rooms also?

Done, closing this issue


Modal not closing when switch from ExpressReceiver to AwsLambdaReceiver

Description
Describe your issue here.
I have a Slack app, which takes input from the modal, sends data to another API, and after checking the response, sends it back to the slack app. I used ExpressReceiver with the below code to handle submit an activity,
module.exports = async function handleSubmit({
    client, body, ack,
}) {
    const stateValues = body.view.state.values;
    const title = stateValues['input-title'].input.value.trim();
    const text = stateValues['input-text'].input.value.trim();
    const tags = stateValues['input-tags'].input.value.trim();

    const errors = validateFormData(title, text, tags);

    if (Object.entries(errors).length > 0) {
        await ack({
            response_action: 'errors',
            errors,
        });
    } else {
        await ack();

        try {
            const response = await postQuestion(title, text, tags);
            const message = getSuccessMessageBody(response.data);
            message.response_type = "ephemeral";
            await postViaResponseUrl(command.response_url, message);

        } catch (error) {
            const message = getFailMessageBody();
            await postViaResponseUrl(command.response_url, message);

        }
    }
};

When I change it to AwsLambdaReceiver, all the other functionality works, but the model does not close.
Could someone please help me with this?
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] example code related
[ ] testing related
[x] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[ ] I've searched for any related issues and avoided creating a duplicate issue.

Reproducible in:
package version: 3.4.0
node version: 12.13.0
OS version(s): Windows
Expected result:
Modal should close
Actual result:
Shows error on modal
Attachments:

Hi @amar391, thanks for asking the question!
In this scenario, I guess your own code like postQuestion can take longer than 3 seconds. With the AWS Lambda receiver, your Bolt app works with the processBeforeResponse: true mode. This means that your listener function must complete within 3 seconds. You can find discussions / questions on this topic:

https://github.com/slackapi/bolt-js/issues/361
https://github.com/slackapi/bolt-js/issues?q=is%3Aissue+processBeforeResponse

Our recommendation is to move the time-consuming parts to another AWS Lambda function call. Refer to my comment here: https://github.com/slackapi/bolt-js/issues/914#issuecomment-870079306 for more details.
I hope this was helpful to you.

@seratch  I will give this a try. Thanks for your quick help


No data, just "loading" in FireFox 47.0

I have a web app that is using jQuery, jQueryUI and jstree.
In my app, I make a REST call to retrieve data and then wish to draw the results in an existing jstree instance.
The API I am calling is
$("#entitiesTree").jstree(true).settings.core.data = treeNodes;
$("#entitiesTree").jstree(true).refresh();

The good news, it works perfectly in the latest Chrome and the latest Edge browsers ... however, in the latest FireFox, I get the "Loading ..." spinner ... and no tree.  I have validated that the data has been returned and the above two API calls are being made ... but that is as far as it goes.  No errors are being reported in the developer tools console and I am stuck on what to try next.

Does this work for you: http://jsfiddle.net/DGAF4/1415/ (as it does for me in Firefox)?
I can't help further, as I need a demo of the problem - the above fiddle was as close as I could get, and it works properly in all browsers for me.
Please provide a demo (or modify the above fiddle) and I will reopen the issue. If you can't provide a demo, nor create a similar fiddle where I can see the problem I really can't help.
By the way - try recreating the tree and not modifying the settings at runtime. What you are doing is not officially supported and has never been tested (modifying the settings at runtime). The better way to do what you need would be to recreate the tree with the new data OR use core.data as a function which always returns a variable that you modify (instead of touching the internal data key).
Best regards,
Ivan

Howdy Ivan,
I opened the fiddle supplied above ... and it too doesn't work on my FireFox.  All I see is the Loading message.  However, from your post you spurred me on to try more research ... I went over to a colleagues machine and opened up his FireFox and the jsfiddle worked perfectly.  Obviously this means that there is something screwy in my environment.  I will dig deeper and see if I can't find out what it might be as I try and run virgin browsers without plugins or customizations.   If it turns out to be something that others might run into, I'll post it back to this issue in case others may stumble upon it.
Again, I want to thank you for the awesome jstree.  It is working perfectly for me (on the other browsers) and is exactly what I needed ...
Neil

I am having this exact issue as well (even with the fiddle above) .. just loading wheel but no data using chrome 52.0
works fine on firefox.
Any help with this ?

I just tried with the latest Chrome (52) and it works fine, but sometimes the loading spinner takes too long to clear. Please try this fiddle and report any issues you may see in the dev console:
http://jsfiddle.net/DGAF4/1505/

I had this issue in firefox. The problem was with worker.
It is start working when i disable him.
jstree({
  core: {
     data: [],
     worker: false
  }
});

Setting worker to false fixed my issue with MS EDGE combined with callback method to set data. See here: http://jsfiddle.net/rpv9js35/

jstree({
core: {
data: [],
worker: false
}
});
Worked for me. Thanks a ton!


[Improvement] update llm prepare scripts

PR types

Improvement
PR changes

Models
Description

更新 llm 安装环境的脚本，从而实现ce 脚本的稳定运行。

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Admin interface not properly showing items

Hi, why do the admin table show items this way?

I copied the app to my local proyect and edited the view to mark_safe this strings to properly show, but, that's not really a good solution, what cain I do?, im using django 2.0.2.

I believe a new release should fix this, we merged a PR recently https://github.com/jjkester/django-auditlog/pull/167 but we haven't done a new PyPI release. @jjkester do you think we could do another minor release?

@kuipumu Can you verify the current repository version works for you?
You can install the current version by running the command below:
pip install --upgrade git+https://github.com/jjkester/django-auditlog.git@master

To go back to the distributed version you can simply uninstall django-auditlog, and then reinstall it by just providing the name instead of the repository.

I upgraded from the current repository and im still getting the same result. I don't know if there is some kind of cache in python, but it's the same. The only way it works is by that workaround I made.

Ahh.. I think the issue is that mark_safe wasn't applied after the format_html,
Try to edit mixins.py file and change LogEntryAdminMixin class to have mark_safe(format_html(...)) and let me know if the urls appear properly

@audiolion Yes, that was the problem, just has the fix that was pulled to the master branch, @jjkester that upgrade from the master branch worked, the problem was that I was in the wrong virtualenv. Thanks.

I will push a release with the fixes soon, hopefully this week, then it would be safer to go back to the packaged version of auditlog.


Update README.md by adding the software description

added software description

DO keep pull requests small so they can be easily reviewed.
DO make sure your contribution fits the openess and scaleability for future extensions
DO make sure unit tests pass.
DO make sure any public APIs are XML documented.
DO make sure not to introduce any compiler warnings.
AVOID making significant changes to the driver's overall architecture.
Delete the above section and the instructions in the sections below before submitting.
Type of request

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Refactoring
[x] Documentation or documentation changes

Related Issue(s)

Concept

Description of the change

Description of current state
Description of target state
Estimated investment / approximate investment

Initiator
name / company / position / created at
Technical information
Platform / target area

DB Changes
Technical information
known side-effects
other
DB Changes (if so: incl. scripts with definitions and testdata)
How has this Tested?

Screenshots, Uploads, other documents (if appropriate):

Checklist:

[ ] My code follows the code style of this project.

[x] I have read the CONTRIBUTING docs.
[x] I have added/updated necessary documentation (if appropriate).

git -c diff.mnemonicprefix=false -c core.quotepath=false -c credential.helper=sourcetree push -v --tags origin refs/heads/master:refs/heads/master
Pushing to https://github.com/aposin/openinsuranceplatform.git
remote: Permission to aposin/openinsuranceplatform.git denied to MarRothm.
fatal: unable to access 'https://github.com/aposin/openinsuranceplatform.git/': The requested URL returned error: 403
Completed with errors, see above

Am 30.08.2018 um 14:59 schrieb holzerma<EMAIL_ADDRESS>Merged #6 https://github.com/aposin/openinsuranceplatform/pull/6 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/aposin/openinsuranceplatform/pull/6#event-1818133088, or mute the thread https://github.com/notifications/unsubscribe-auth/AlKwjSz7A1zeitCcnmbNLi_kAfpLnzuJks5uV-GegaJpZM4WRg9W.


HIVE-23266: Remove QueryWrapper from ObjectStore

@dengzhhu653 Thanks so much for the feedback.

LGTM, +1. Thanks @belugabehr

@dengzhhu653 Really, thanks so much for the review.  Greatly appreciated.  Happy to get this class moving in the right direction.


bobcat: 5.05.00 -> 5.09.01

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/bobcat/versions.
meta.description for bobcat is: "Brokken's Own Base Classes And Templates"
meta.homepage for bobcat is: "https://fbb-git.gitlab.io/bobcat/"
meta.changelog for bobcat is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 5.09.01 with grep in /nix/store/k8xjy5bfvwl3kyby6bf9mvkqmcip844k-bobcat-5.09.01

found 5.09.01 in filename of file in /nix/store/k8xjy5bfvwl3kyby6bf9mvkqmcip844k-bobcat-5.09.01

directory tree listing: https://gist.github.com/32996ccdcd8fa3ec1cde684094ba1707

du listing: https://gist.github.com/36d6d1d72daf64776f6b45f14616ec07

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

9 total rebuild path(s)

3 package rebuild(s)

3 x86_64-linux rebuild(s)
3 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
3 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
bisoncpp
bobcat
flexcpp

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/k8xjy5bfvwl3kyby6bf9mvkqmcip844k-bobcat-5.09.01 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A bobcat https://github.com/r-ryantm/nixpkgs/archive/bc0ec9d2fff9ccaacf5f45db1651d38f6f2bbbc0.tar.gz

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

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

bisoncpp
bobcat
flexcpp

Maintainer pings

Result of nixpkgs-review pr 133400 at bc0ec9d2 run on x86_64-linux 1

3 packages built successfully:

bisoncpp
bobcat
flexcpp

5 suggestions:

warning: missing-phase-hooks
buildPhase should probably contain runHook preBuild and runHook postBuild.
Near pkgs/development/libraries/bobcat/default.nix:29:3:
   |
29 |   buildPhase = ''
   |   ^

warning: unclear-gpl
gpl3 is a deprecated license, please check if project uses gpl3Plus or gpl3Only and change meta.license accordingly.
Near pkgs/development/libraries/bobcat/default.nix:41:5:
   |
41 |     license = licenses.gpl3;
   |     ^

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/development/libraries/bobcat/default.nix:34:3:
   |
34 |   installPhase = ''
   |   ^

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/development/libraries/bobcat/default.nix:38:3:
   |
38 |   meta = with lib; {
   |   ^

warning: build-tools-in-build-inputs
util-linux is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/development/libraries/bobcat/default.nix:17:3:
   |
17 |   buildInputs = [ libmilter libX11 openssl readline util-linux ];
   |   ^


[filesystems]Support Tencent COSN (#842)

[filesystems]
Purpose
Support Tencent COSN
Documentation
(Does this change introduce a new feature)
change paimon/docs/content/filesystems/overview.md
add       paimon/docs/content/filesystems/cosn.md

@daixiaoyu, could you add the test cases for loader of Tencent COSN support?

@SteNicholas Okay, I'll try.

@daixiaoyu, any comments or updates for above review comments?

This can be close , Tencent COS  can use  S3 FileSystem


HBASE-20557 Backport HBASE-17215 to branch-1

The second backport of HBASE-20555

ran mvn clean install -DskipTest and executed unit test TestHFileCleaner successfully, pinged on HBASE-20557

Can you fix the import order checkstyle warning?

I tried to used the hbase_eclipse_formatter.xml and followed the setting in hbase-checkstyle/src/main/resources/hbase/checkstyle.xml, submitted a new patch for reorganizing the imports. However, it's not working as expected and I don't know what is missing from the checkstyle, you can also see those non-changed classes MergeNormalizationPlan and SplitNormalizationPlan is importing correctly as well but failed with the same issue, I'm wondered if I'm hitting a bug or something for checkstyle, I need more time to look into it but if you can point out the problem that would be helpful

  [ERROR] /local/home/wutaklon/workspace-tmp/hbase/hbase-server/src/main/java/org/apache/hadoop/hbase/master/normalizer/MergeNormalizationPlan.java:27: Wrong order for 'java.io.IOException' import. [ImportOrder]
  [ERROR] /local/home/wutaklon/workspace-tmp/hbase/hbase-server/src/main/java/org/apache/hadoop/hbase/master/normalizer/SplitNormalizationPlan.java:27: Wrong order for 'java.io.IOException' import. [ImportOrder]
...
[ERROR] /local/home/wutaklon/workspace-tmp/hbase/hbase-server/src/main/java/org/apache/hadoop/hbase/master/cleaner/HFileCleaner.java:33: Wrong order for 'java.io.IOException' import. [ImportOrder]

I figured it out at the end that checkstyle of the IDE in hbase_eclipse_formatter.xml and checkstyle while running maven-checkstyle-plugin with conf in hbase-checkstyle/src/main/resources/hbase/checkstyle.xml does not match.
with the following the imports order checkstyle will pass

import com.google.common.annotations.VisibleForTesting;
import java.io.IOException;
import java.util.ArrayList;
import java.util.List;
import java.util.Map;
import java.util.concurrent.BlockingQueue;
import java.util.concurrent.LinkedBlockingQueue;
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import org.apache.hadoop.conf.Configuration;
import org.apache.hadoop.fs.FileStatus;
import org.apache.hadoop.fs.FileSystem;
import org.apache.hadoop.fs.Path;
import org.apache.hadoop.hbase.Stoppable;
import org.apache.hadoop.hbase.classification.InterfaceAudience;
import org.apache.hadoop.hbase.conf.ConfigurationObserver;
import org.apache.hadoop.hbase.io.HFileLink;
import org.apache.hadoop.hbase.regionserver.StoreFileInfo;

So, I'm wondered if we should have an separate JIRA (I will open it and fix it) to fix and sync between these two setting files (with HBASE-19262 and HBASE-19552) still does not work well, so we need to backport it branch-1 and also fix the problem we found here.
For this changes, I personally see the current commit matched what  hbase_eclipse_formatter.xml setup, so, it's safe to commit. but if you insist we need to fix it within this PR, please let me know if we should fix it here instead of a new JIRA/PR.

Let's update the order related to the checkstyle.xml for now and open a JIRA to discuss this mismatch.

sure, I have done that by updated the commit and patch on the HBASE-20557, the latest auto-build show the checkstyle has been passed.

any update ? do I need to change anything else?

merged. @busbey how can I close the PR? It looks like I don't have the permissions required.

this has been merged as https://github.com/apache/hbase/commit/a52c6bf2f3f742ae4bec29c5d10dfc339c1e3f0d


Labels we need

Github allows custom issue labels. I've created an "additional factor" label. This is for issues that contain ideas for new things the bot should consider. What other labels would be helpful for this project? Are there any defaults that aren't needed?

NO JOKE LABELS, PLEASE.
That aside, we'd also want a "building feature" label or something to discuss features we're building into the bot.

How about "in progress"?

That's the whole project, though :P

But, like, specific things that are currently being coded by someone. This would prevent duplicates of the same feature.

After all, isn't that what you meant when you said "building feature"? I was just paraphrasing.

We could keep track on a separate issue continuously maintained.

Wouldn't it make more sense to use a label though? That way you can see, directly on an issue, whether it's already being worked on.

Maybe both?

That seems a bit overcomplicated. Let's get someone else's opinion.
(I'm looking at you @Thepuzzlegame @Zaerospi)

uhhh...I really don't know. Just the word "maintaining" sounds like word though. Maybe a wiki page, or a TODO file in the repo that says who is doing what? Because then everyone can edit it, and it won't flood the issues.

*work, not word

Everyone can edit an issue post too :P

Oh. I'm a n00b obviously.

What would be the benefit of having the labels and a dedicated issue? You can view all the issues with a certain label anyway.

True, forgot about that.  Let's just have individual labels.

How about a "planning" label, for issues like this one?

Individual labels would allow discussion about that specific feature in one thread, so individual.

What do you mean by "individual labels"?

planning label sounds nice.

But is it helpful?

Sorry, I meant individual issues. I mean planning sounds like a good word to label these "issues" with.

Ah, I see what you mean. Okay...

Seems pretty well taken care of. Closing to get this out of the way. If anyone still wants to add to this, feel free to re-open.


Split up contributing docs

I took what @j08lue added in #60 and pulled in content from https://github.com/NASA-IMPACT/veda-stac-ingestor/blob/main/API_usage.md
I think this organization makes the pieces a little more granular.

Thank you @j08lue!!!


FLOPs in paper

Hi,
Thanks for your work.
I wonder How do you calculate your GFLOPs in your paper?
I tried some open-source work https://github.com/Lyken17/pytorch-OpCounter  and  https://github.com/sovrasov/flops-counter.pytorch   But the result is quite different from its in paper,  could you please share this part ?

I used the THOP, example code:
import model as models
from thop import profile

model = models.__dict__['inception_iccv'](pretrained=False, num_classes=51)

flops, params = profile(model, input_size=(1, 3, 256, 128))
print(flops/1e9, params/1000/1000)

I used the THOP, example code:
import model as models
from thop import profile

model = models.__dict__['inception_iccv'](pretrained=False, num_classes=51)

flops, params = profile(model, input_size=(1, 3, 256, 128))
print(flops/1e9, params/1000/1000)

Thanks for reply,  I have tried your solution, but got the results:

from model.inception_iccv import inception_iccv
from thop import profile
import torch
with torch.cuda.device(2):
    model = inception_iccv(pretrained=False,num_classes=51).cuda().eval()
    input = torch.randn(1,3,256,128).cuda()
    flops, params = profile(model, inputs=(input,))
    print(flops/1e9, params/1000/1000)

#p is fine, but GFLOPs still not right.
I am new to THOP, any suggestion?

The above codes are exactly the codes I used in my experiments, while I have never seen these warnings.
Maybe due to the version of PyTorch or THOP? I used PyTorch 0.4 when preparing the table.


Merge development into master branch for publishing Capri RC-01

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

@leonwanghui @xing-yang @wisererik
Hi All,
Some suggestions for future:

dev to master merge -- always we let each feature owners review mandatory for such PRs
instead of one bulk merge to master we do incremental merge/seperate PRs
Please share your views.

I actually prefer to get rid of development branch.  All changes should be merged to Master directly.  Then cut a stable branch from master at release time.  After that, we need to cherry-pick each change to stable branch.
Even for each regular PR merge (non release related), we should ask PR author to squash all those merge commits to make commits clean.
Let's start this after Capri release.

I agree with @skdwriting and @xing-yang, the initial thought why we split development branch is because we want to make our master branch is always stable so any code change would not impact master branch which is exposed to end users. But since we have published several stable releases (v0.5.3 up to now), we can be ok to remove it then.


Update figure panels to show celltyping in processed object

While writing legends, I realized that some of our workflow figures may not properly reflect the workflow. The overview workflow (Fig 2B) and all panels that stem from it make it look like cell type annotation is not included in the processed object. I would expect a cell type annotation arrow feeding into the SCE Output Data box, right?
The other panels that also use this base image are:

additional-modalities-cite.svg (Fig S2a, but note also #62)
additional-modalities-multiplexed.svg (Fig S2b, but note also #62)

Tagging @allyhawkins @dvenprasad

I think I have mixed feelings about this change. I think we talked about it when Deepa was putting these together and I'm not sure how you would show it unless you extended the line between the two arrows pointing down to the reports, but I'm not sure that really clears things up?
I do want to note that we have a separate cell type annotation workflow in Fig. 4 that shows cell types get incorporated into the processed object.

I do want to note that we have a separate cell type annotation workflow in Fig. 4 that shows cell types get incorporated into the processed object.

Yeah I did notice this!
I drafted up the legend to emphasize that the cell type annotations are in the processed object, which may cover our bases if the design is too tricky with arrows? See: https://github.com/AlexsLemonade/ScPCA-manuscript/pull/36

I drafted up the legend to emphasize that the cell type annotations are in the processed object, which may cover our bases if the design is too tricky with arrows? See: https://github.com/AlexsLemonade/ScPCA-manuscript/pull/36

I think this is fine. Also in the section of the paper that refers to this figure we don't really talk about cell typing, just that we do it. There is a whole other section on cell typing and describing the cell type workflow that will come with Fig. 4, so my vote would be to leave it.


Refactor linker and delete some unused code

Use more efficient solutions than DefAnalyzer, and shuffle linker step order to remove need for ScalarType/AggregateType.
Considering that not using DefAnalyzer results in more efficient code in less lines of code (not even counting DefAnalyzer itself), this seems good.

I think this is a sound change; I think the way it import/export type checking used to work was because of the way the deduplicator used to work. Running this after dedup loses some of the type information for error reporting, but because it's significantly faster then that might be a win (and we might be able to recover that information another way).


hdf5 missing from $NIX_LD_LIBRARY_PATH

cannot find -lhdf5: No such file or directory
Steps To Reproduce
Steps to reproduce the behavior:

install ldf5 and uv into your nixos system packages
run: uv pip install tables==3.6.1

Expected behavior
It should be able to load ldf5 library from the list of libaries available in $NIX_LD_LIBRARY_PATH and install the python package.
Actual
error: Failed to download distributions
  Caused by: Failed to fetch wheel: tables==3.6.1
  Caused by: Failed to build: `tables==3.6.1`
  Caused by: Build backend failed to build wheel through `build_wheel()` with exit status: 1
--- stdout:
* Using Python 3.10.14 (main, Mar 19 2024, 21:46:16) [GCC 13.2.0]
* USE_PKGCONFIG: False
.. ERROR:: Could not find a local HDF5 installation.
   You may need to explicitly state where your local HDF5 headers and
   library can be found by setting the ``HDF5_DIR`` environment
   variable or by using the ``--hdf5`` command-line option.
--- stderr:
<string>:18: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
<string>:21: DeprecationWarning: dep_util is Deprecated. Use functions from setuptools instead.
/nix/store/bgcaxhhxswzvmxjbbgvvaximm5hwghz1-binutils-2.41/bin/ld: cannot find -lhdf5: No such file or directory
collect2: error: ld returned 1 exit status

Notify maintainers
@markuskowa

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
root in /etc/nixos on master λ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux <IP_ADDRESS>-microsoft-standard-WSL2, NixOS, 24.11 (Vicuña), 24.11.20240605.437e3a2`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.2`
 - channels(root): `"nixos, nixos-wsl"`
 - nixpkgs: `/nix/store/4vk4vsnh808b6jlzzkhipkxry9lsrs5r-source`

Add a :+1: reaction to issues you find important.

I do not think that is considered a bug, but expected behavior:
Putting hfd5 in environment.systemPackages exposes the executables from <store-path>/bin of the package system wide, but not its libraries.
I have never tried to run pip or uv under nix, but it sounds like you rather want to use a nix shell for that purpose.

Okay, so then what is the standard way of exposing libraries in nixos? when I install a package system wide in python for example, I can use that package system wide also without a virtual environment.
But this is C++ land, I'm a bit of a newbie on how nix does c++ stuff.


Add supports for Symfony 4

Coverage decreased (-3.03%) to 96.97% when pulling f60fb9fafba3562e43a1b970cf0d2d2280dd88ea on mcg-web:symfony_4 into cd76661539b17617c4e23999a891c1936cc162da on overblog:master.

Coverage remained the same at 100.0% when pulling 86834ab6353f3e89e11362f79a58b0358ac7feb2 on mcg-web:symfony_4 into cd76661539b17617c4e23999a891c1936cc162da on overblog:master.

Coverage remained the same at 100.0% when pulling 22d3b159d392431bee94d9bd682657cb5e11fa57 on mcg-web:symfony_4 into cd76661539b17617c4e23999a891c1936cc162da on overblog:master.

Coverage remained the same at 100.0% when pulling e91486f010e90fb78eac35e5c54f92ef67f62f88 on mcg-web:symfony_4 into cd76661539b17617c4e23999a891c1936cc162da on overblog:master.


Version Packages

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and publish to npm yourself or setup this action to publish automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
<EMAIL_ADDRESS>Patch Changes

5618624: Lazy get database client for batch loader

<EMAIL_ADDRESS>Patch Changes

Updated dependencies [5618624]

<EMAIL_ADDRESS>

The following preview packages were published:

Name
Version
Install Command

@frontside/backstage-plugin-batch-loader
0.1.2-changeset-release-main.1
yarn add @frontside/backstage-plugin-batch-loader@changeset-release-main

Generated by @thefrontside/actions


problem in documents for flutter push notifications

I use the document from the link:
https://docs.webengage.com/docs/flutter-push-notifications
It has two problems:
The first one in the Doc has 2 titles "Integrating WebEngage with your Flutter Android App". And I'm not sure which part to use.
Because service is not set in the manifest in the second part of the service.
After I used the first part, the program encountered the following error:
/Users/mahanabedini/AndroidStudioProjects/doki-app-new26/android/app/src/debug/AndroidManifest.xml Error: 	android:exported needs to be explicitly specified for element <service#es.mhholding.doki_app.MyFirebaseMessagingService>. Apps targeting Android 12 and higher are required to specify an explicit value for android:exported` when the corresponding component has an intent filter defined. See https://developer.android.com/guide/topics/manifest/activity-element#exported for details.
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:processDebugMainManifest'.

Manifest merger failed with multiple errors, see logs

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 7s
Exception: Gradle task assembleDebug failed with exit code 1`
update:
and is think step5 in part one must change like this in kotlin:
`import com.google.firebase.messaging.FirebaseMessagingService
import com.google.firebase.messaging.RemoteMessage
import com.webengage.sdk.android.WebEngage
class MyFirebaseMessagingService : FirebaseMessagingService() {
override fun onMessageReceived(remoteMessage: RemoteMessage) {
val data = remoteMessage.data
if (data != null) {
if (data.containsKey("source") && "webengage" == data["source"]) {
WebEngage.get().receive(data)
}
}
}
    override fun onNewToken(s: String) {
    super.onNewToken(s)
    WebEngage.get().setRegistrationID(s)
}

}`
I would appreciate it if you updated the document.

@mahanabd

For your first point - The first one in the Doc has 2 titles "Integrating WebEngage with your Flutter Android App".
If you will check the docs it is written
Integrating WebEngage with your Flutter Android App (FCM not integrated) and
Integrating WebEngage with your Flutter Android App (FCM integrated)

So if you are using firebase in your project then follow 2nd one else follow the 1st one.
for error :- please add exported = "true" in MyFirebaseMessagingService in Manifest file
so the code will look like below in Manifest file of android
<service android:name=".MyFirebaseMessagingService"
            android:exported="true">
            <intent-filter>
                <action android:name="com.google.firebase.MESSAGING_EVENT" />
            </intent-filter>
</service>

And We will update our docs for the same and the below kotlin code you have mentioned.
Thank You 👍

Due to the lack of activity, the issue is closed.


Update README.md

Hi, following your instuctions I got this error:
Download https://deno.land/dfirestore/mod.ts
An unsupported media type was attempted to be imported as a module.
  Specifier: https://deno.land/dfirestore/mod.ts
  MediaType: Unknown

changing it to https://deno.land/x/dfirestore/mod.ts fixed it

@DavidBonelo thanks for the first PR to the repo and an important fix!

@DavidBonelo thanks for the first PR to the repo and the fix!


[RaisedButton] Fixes #5168, disabled when use containerElement

[ ] PR has tests / docs demo, and is linted.
[x] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[ ] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

reference to react-router. we must use containerElement to create the link, So I how to pass the test case.
Some confusing about href and with out href props. when disabled it with href return <span> and with out href return <button disabled>.

The RaisedButton  component was migrated on the next branch.
We are now focusing on releasing out the next branch.
We plan to stop the support of the migrated components.
I'm gonna close this PR. Thanks for your time.
Would be great if you check out the new component.
All the buttons were merged under a <Button /> component. Any PR is welcomed.


Fix bug in Holoscan 2.0 code path of TensorToVideoBufferOp

The primary purpose of this MR is to fix a broken code path for Holoscan 2.0 within TensorToVideoBufferOp
Specifically, there was a location where a Tensor was created instead of the expected shared_ptr<Tensor>.
This MR also fixes some mistakes in the h264 application README files.
I confirmed that compilation of h264_endoscopy_tool_tracking  is fixed by this change.

Thanks for fixing this Greg!


AdaTAD License question

Does the Apache License apply to all models in the repo, and specifically does it apply to the Adatad model?
Thanks!!

The License applies to all the models in the repo.


Issue in CycleGan-Pix2Pix while calling  Discriminator()

While calling Discriminator() from models.py in cycleGan and pix2pix, I get a syntax error response as ;
Traceback (most recent call last):
  File "cyclegan.py", line 15, in <module>
    from models import *
  File "./PyTorch-GAN/implementations/cyclegan/models.py", line 167
    *discriminator_block(64, 128, 2, True),
    ^
SyntaxError: invalid syntax

This happens with both python3 and python2.7.
Looks like dereferencing does not work and I could not find a way to make it work.
Any bits of advice?
Thanks

That's strange. I'm using Python 3.6.3 and it's working fine for me. What 3.x.x version are you using?
Not sure if this will work (since it's also unpacking a list) but you could try:
class Discriminator(nn.Module):
    def __init__(self, in_channels=3):
        super(Discriminator, self).__init__()

        def discriminator_block(in_filters, out_filters, stride, normalize):
            """Returns layers of each discriminator block"""
            layers = [nn.Conv2d(in_filters, out_filters, 3, stride, 1)]
            if normalize:
                layers.append(nn.InstanceNorm2d(out_filters))
            layers.append(nn.LeakyReLU(0.2, inplace=True))
            return layers

        layers = []
        layers.extend(discriminator_block(in_cannels, 64, 2, False))
        layers.extend(discriminator_block(64, 128, 2, True))
        layers.extend(discriminator_block(128, 256, 2, True))
        layers.extend(discriminator_block(256, 512 , 2, True))
        layers.append(nn.Conv2d(512, 1, 3, 1, 1))
        self.model = nn.Sequential(*layers)

    def forward(self, img):
        return self.model(img)

If not you might have to add the layers one by one.

works like a charm. Two small issues
(i) There is a typo in the above code. Instead of "in_cannels"  it should be "in_channels"
(ii) As you are using stride=2 for 4 times so I think line#54 in "cyclegan.py" needs to be changed as
patch_h, patch_w = int(opt.img_height / 2**4), int(opt.img_width / 2**4)
By the way, just wanted to confirm, the code also works for python2.7 too (with a few simple edits). I will try to run an experiment to see if the results also make sense with python2.7 :smile:
Thanks again


adopt flake8 maccabe complexity metric

🚀 Pull Request
Description
This PR introduces the flake8 mccabe algorithm to provide a metric on iris code complexity.
This empirical metric can then form part of the discussion between a PR author and reviewer/s with regards to the complexity of code being proposed.
According to McCabe, code with a score greater than 10 is deemed too complex. Currently iris has a mccabe score of 50.
Adopting the maccabe metric will allow us to target and reduce complexity within iris, but also be aware of any potential rise in complexity.
As per flake8, the complexity warnings/errors issued by mccabe can be explicitly ignored, if so deemed suitable and appropriate.

Consult Iris pull request check list

Bumping the complexity down to 1 makes interesting reading...

Interesting! Is it possible to get some examples from the iris codebase along with their complexity scores? Reading the wikipedia link @rcomer provided:

Cyclomatic complexity may also be applied to individual functions, modules, methods or classes within a program.

Do we know what level this tool applies? From the output shown on the project page it certainly does per-function analysis

Great question...
It's early days, and this is all new to us. I'd happily opt for just rolling with it on a PR by PR basis, making in-situ judgement calls, and watching out for patterns on how this affects our workflow.
If we take this all under the remit of the Peloton and review on occasion how were fairing, that seems like a heathy and reasonable approach to take.
What say ye?

From @bjlittle’s posted output, there is only one offender with a score over 30.  So if something else goes over 50, that would indicate a genuine problem I think.

@rcomer sure, for now. But I hope over time we are able to drop the threshold and then it will become more pertinent to think about "accepting failures". I'm happy to go with @bjlittle's approach of rolling with the punches for now; personally I'll be pushing for # noqa if it comes up.

have to dual with the reviewer

Sounds much more collaborating and less adversarial than "have a duel"!

I'm also hoping that I can always play the "reduce complexity card" without jumping through hoops, otherwise then it becomes a author/reviewer discussion i.e., a strong signal that it might be harder to get the pull-request merged without very good reason.

👍 There are so many things we all agree we want, but it's often hard to agree what that looks like. So having an objective vehicle for such things is a positive step, like black does for readable code style. As you say, there will still be discussions at times, but hopefully fewer.


Add test case for ResizableCapacityLinkedBlockingQueue

Code classes that need to develop unit tests:

cn.hippo4j.common.executor.support.ResizableCapacityLinkedBlockingQueue

Please refer to the manual for the first contribution: guide
Unit test development refers to the following two examples:

cn.hippo4j.core.executor.DynamicThreadPoolExecutorTest
cn.hippo4j.core.executor.ExtensibleThreadPoolExecutorTest

Please assign it to me.


chore: replace #eval by run_cmd in Funlike/Basic

This is the only #eval used in Mathlib.

bors merge

bors d+

bors r+


Support for Server Sent Events

Does the new LWS support eventsource connections to backend services? The previous version was working fine.  On version
$ ws --version
2.0.0-pre.4

eventsource (aka Server Sent Events with accept: 'text/event-stream',) seem to always return empty responses.  When using -v to see the output, the data field is empty (see all the way at the end)...
{ 'socket-data': { id: 5, bytesRead: 730, bytesWritten: 0 } }
{ 'rewrite-proxy-req':
   { req:
      Url {
        protocol: 'http:',
        slashes: true,
        auth: null,
        host: '**myhostname**:8080',
        port: '8080',
        hostname: '**myhostname**',
        hash: null,
        search: '?&streamId=13d783492c76a9be5fbb34bdaeb936c9691fd893fd67a11ed99051c55c855259',
        query: '&streamId=13d783492c76a9be5fbb34bdaeb936c9691fd893fd67a11ed99051c55c855259',
        pathname: '/path/to/stream',
        path: '/path/to/stream?&streamId=13d783492c76a9be5fbb34bdaeb936c9691fd893fd67a11ed99051c55c855259',
        href: 'http://**myhostname**:8080/path/to/stream?&streamId=13d783492c76a9be5fbb34bdaeb936c9691fd893fd67a11ed99051c55c855259',
        id: 3,
        method: 'GET',
        headers:
         { host: '**myhostname**:8080',
           connection: 'keep-alive',
           accept: 'text/event-stream',
           'cache-control': 'no-cache',
           'user-agent': 'Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3071.115 Safari/537.36',
           dnt: '1',
           referer: 'http://**myhostname**/path/to/app/',
           'accept-encoding': 'gzip, deflate',
           'accept-language': 'en-US,en;q=0.8',
           cookie: 'JSESSIONID=15A2;and_other_cookies..' },
        rejectUnauthorized: false },
     data: '' } }

See the last line has empty data: data: ''.
But when I hit my downstream service directly I get periodic events as expected. Is there a new middleware I need to configure/enable to turn on SSE for rewrite requests?
I haven't tried websockets so it may or may not behave the same way.

hi.. will look into this now.. does it work with the latest version (2.0.0-pre2.3)? Your version is a little old now..
npm install -g local-web-server@next

could you send me a simple test case please to help me reproduce?

still the same issue with 2.0.0-pre2.3. I'll see if I can make something later tonight that you can use. My real backend is a Java service that implements SSE over servlets, but any service that implements the SSE spec should work. Basically something like a simple express.js server that pushes some data on an interval like one of these:
https://github.com/fmvilas/sse/blob/develop/examples/server.js (probably the simplest example)
https://github.com/trquoccuong/Server-Sent-Events-Nodejs
Some other links I just dug (for the curious, and for reference later)...
https://developer.mozilla.org/en-US/docs/Web/API/Server-sent_events
https://github.com/schwarzkopfb/sse-broadcast
http://bayn.es/real-time-web-apps-with-server-sent-events-pt-2/
https://technology.amis.nl/2017/02/19/node-js-application-using-sse-server-sent-events-to-push-updates-read-from-kafka-topic-to-simple-html-client-application/

thanks, looking into it now..

this is fixed in v2.0.0-pre2.4, please reinstall
npm install -g local-web-server@next

thanks again, let me know if you find anything else!

Fantastic.  Works great... now -v verbose output shows:
        statusCode: 200,
        statusMessage: 'OK',
        url: '' },
     data: [ SSE event-stream ] } }

👍


[Bug]: Scrolling Malfunction on Brave Browser

Describe the bug
This issue only seems to be occurring on Brave Browser. (Works fine on Firefox)
ISSUE:
When scrolling up on the website, window is automatically taken scrolled down to the bottom.
This makes it impossible to stay or read any particular section.
The below attached gif would describe the issue clearly.

To Reproduce

Open Brave Browser
Open any page of the NSCC website, expect the home page.

Expected Behavior
No response
Screenshot/ Video

Desktop (please complete the following information)
Device - Lenovo Thinkbook
OS -  Linux Ubuntu 22.04
Browser - Brave
Additional context
No response

I would like to work futher on this issue, if possible.
@mnnkhndlwl @pradeeptosarkar

go ahead @SAYUK09

go ahead @SAYUK09

Done. Here's the PR.
https://github.com/NSCC-BPIT/NSCC-BPIT-Website/pull/423

resolved


[CS] Fixes #2487 Add RemoveCommentedCodeFixer

@TomasVotruba it uses regex to detect commented code.

All checks have passed 🎉 @TomasVotruba it is ready for review.


Adding ELGs with zwarn==4

For some high-z ELGs, the [OII] emission features are robust but the redrock can not robustly distinguish the features as [OII] or Ha or [OIII]. This yields a small detachi2 for the first best fit with zwarn==4.
If only systems with zwarn==0 are included, we will lose a fraction of those high-z galaxies.
It will be great if ELGs with zwarn==4 can also be included in the catalog.
It will increase the sample size by about ~20%.
Ting-Wen

Done in https://github.com/desihub/fastspecfit/commit/9019bc9108f4f1d47c738fee3a01910dd91cb3af.


Support for stopping container

Just directly use kill SIGKILL to stop the container for temporary.
And modify "stop command of hyperctl" like "kill" which can stop pod or container.

lgtm


TO: Add option to skip over HTTPS Certificate creation

We have our own certificate generate from Ansible playbook. Would be nice if we could skip this section. I tried to control-C and I got in a endless loop..

I'd second this request, I'd like to be able to do the Database config via a file, non-interactively as well


cmx_3600 adapter incorrectly processing three-clip transitions

I have a case of a three part Transition in a CMX 3600 EDL. It looks like so:
Clip A -> Dissolve -> Clip B -> Dissolve -> Clip C
Here's the EDL:
TITLE:   TRANSITION_TEST_2
FCM: NON-DROP FRAME
001  ABC0000. V     C        01:00:06:18 01:00:08:00 01:04:11:17 01:04:12:23
002  ABC0010. V     C        01:00:06:15 01:00:08:18 01:04:12:23 01:04:15:02
003  ABC0010. V     C        01:00:08:18 01:00:08:18 01:04:15:02 01:04:15:02
003  ABC0020. V     D    035 01:00:06:22 01:00:10:07 01:04:15:02 01:04:18:11
* BLEND, DISSOLVE
004  ABC0020. V     C        01:00:10:07 01:00:10:07 01:04:18:11 01:04:18:11
004  ABC0030. V     D    064 01:00:06:10 01:00:09:22 01:04:18:11 01:04:21:23
* BLEND, DISSOLVE
005  ABC0040. V     C        01:00:08:14 01:00:12:14 01:04:21:23 01:04:25:23

In that example, ABC0020 should have a duration of 96, however OTIO it showing it to have 50 frames.
There appears to be something specifically wrong here, in the logic that derives what is the "previous clip":
https://github.com/PixarAnimationStudios/OpenTimelineIO/blob/84af5e5a30e4d14cf3434d6d055631e115ac1ee5/src/py-opentimelineio/opentimelineio/adapters/cmx_3600.py#L706-L717

Turns out, the issue is actually in how the adapter is adjusting the duration of the clip after the transition. Specifically, in how it's defining the "next clip."
https://github.com/PixarAnimationStudios/OpenTimelineIO/blob/84af5e5a30e4d14cf3434d6d055631e115ac1ee5/src/py-opentimelineio/opentimelineio/adapters/cmx_3600.py#L731-L749
To explain, here's an EDL with a simple dissolve:
001  ABC0010. V     C        01:00:06:15 01:00:08:18 01:04:12:23 01:04:15:02
002  ABC0010. V     C        01:00:08:18 01:00:08:18 01:04:15:02 01:04:15:02
002  ABC0020. V     D    035 01:00:06:22 01:00:10:07 01:04:15:02 01:04:18:11
* BLEND, DISSOLVE

In this case, when the adapter gets to line three, it hits the above code. It adds the Transition and adds half of the transition's runtime to the "next clip", which is also line 3.
However, in the case of an A->B->C transition, the adapter gets confused. It sees line 3 and line 4 as continuous and thus, it adds the Transition length to line 4, which is a 0-length transition marker.
001  ABC0010. V     C        01:00:06:15 01:00:08:18 01:04:12:23 01:04:15:02
002  ABC0010. V     C        01:00:08:18 01:00:08:18 01:04:15:02 01:04:15:02
002  ABC0020. V     D    035 01:00:06:22 01:00:10:07 01:04:15:02 01:04:18:11
* BLEND, DISSOLVE
003  ABC0020. V     C        01:00:10:07 01:00:10:07 01:04:18:11 01:04:18:11
003  ABC0030. V     D    064 01:00:06:10 01:00:09:22 01:04:18:11 01:04:21:23
* BLEND, DISSOLVE

In the next loop, the adapter then removes line 4 (because it's just a transition marker) and we lose half the duration of the transition.
I fixed it by tweaking the "next_clip" logic so that, rather than assuming the next clip is "whatever's next," it defines the next clip as "the next clip with a duration."
def _get_next_clip(start_index, track):
    # Iterate over the following clips and return the first "real" one
    for index in range(start_index+1, len(track)):
        clip = track[index]
        if clip.duration().value > 0:
            return clip

    return None

(This also led me to simplify the loop in _expand_transitions() and remove the iter().)
This works for me and all the tests are passing now, however the entirety of _expand_transitions() feels a little overcomplicated to me, and could possibly be simplified. But in attempting to simplify it, the test suite started to fail, so I don't think I fully grok the entirety of what it's doing.


Create Template for Generating Stele TV style Squad Lists

Stele TV creates squad lists as basic text files.   A sample of the output:
(skill) Ship Name - Points
Upgrades
Upgrades
Shields (*) Hull (H)
The template should generate out a text file that displays this information.  There should be one for both Left aligned and right aligned versions.  It also needs an option Unique Pilot Identifier to track particular ships that are the same skill and ship type.

This is now implemented and working.


LB rule with selector removed when any labelled service removed

Rancher Versions:
Server: 1.4.1

Rancher/lb-service-haproxy:v0.5.9

Docker Version:
1.12.6
OS and where are the hosts located? (cloud, bare metal, etc):
openstack, centos7
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Rancher HA with external DB (galera)
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:

Balancer is used like global service (not deployed for specific stack/service)
LB service -> Add Selector Rule -> https,443 and target: loadbalancer.https=true
start two services with label 'loadbalancer.https: true' add lb_config

    lb_config:
      port_rules:
      - hostname: first.example.com
        target_port: 8080

    lb_config:
      port_rules:
      - hostname: second.example.com
        target_port: 8080

remove one of services, check rules on LB.
Results:
Selector rule is removed with service, expected is that this rule will stay here (at least until all services with this label not removed).

+1 same here

+1 same issue.
Rancher Version:

Server: 1.4.2
rancher/lb-service-haproxy:v0.5.9

Docker Version:

1.12.6

Same for 1.5.1 and rancher/lb-service-haproxy:v0.6.2, Docker v1.13.1

@VAdamec It would help me if you could put your full compose files for the load balancers and services to it'd be easier to reproduce and ensure we're doing the same steps.
Could anyone give more details on how you are adding/removing services? Was it through the UI or compose? How are you checking the rules?
Are there additional rules or just 1 selector rule on the LB?
I assume your 2 services were started using CLI/rancher-compose as there is no UI support for adding in port rules to services being targeted by selectors.
I removed the service through the UI.
I am unable to reproduce in v1.5.3 using lb-service-haproxy:v0.6.2
Steps I did:

I started a load balancer using selector rules with 2 services being picked up by doing "Add Stack and inputting the following compose"

docker-compose.yml
version: '2'
services:
  lb1:
    image: rancher/lb-service-haproxy:<version>
    ports:
    - 81
  web1:
    image: nginx
    labels:
      foo: bar
  web2:
    image: nginx
    labels:
        foo: bar

rancher-compose.yml
version: '2'
services:
  lb1:
    scale: 1
    lb_config:
      port_rules:
      - source_port: 81
        # Target any service that has foo=bar as a label
        selector: foo=bar
        protocol: http
    health_check:
      port: 42
      interval: 2000
      unhealthy_threshold: 3
      healthy_threshold: 2
      response_timeout: 2000
  # web1 and web2 are targeted with the same source port but with the different hostname and path rules
  web1:
    scale: 1
    lb_config:
      port_rules:
      - target_port: 80
        hostname: test.com
  web2:
    scale: 1
    lb_config:
      port_rules:
      - target_port: 80
        hostname: example.com/test

From the UI, I can see the load balancer and 2 services. I check the yaml of the stack by clicking on "View Config" from the dropdown of the stack. I also check the LB that the HAProxy is programmed correctly for both services by execing into the LB container and viewing /etc/haproxy/haproxy.cfg

From the UI, I delete one of the services. I re-check the yaml of the stack and the

selector: foo=bar

part of the lb_config is still in.
I check the HAProxy and confirm that there is still 1 service being targetted.
I tried these steps with a load balancer with a specific scale as well a load balacner with a global scale. Both times, after deleting one of 2 services, the port rules on the load balancer still remained.

I'll attach full files, but there is nothing special. Selector rule was added manually via UI, all stack/service changes are done via rancher-compose and checked in UI (or test by app test suite and rancher-cli). But I have same result If I remove stack/service via UI.

Where or how are you checking that the selector rule still exists on the LB?
I just tried deleting the stack in the UI and my selector rule is still on the LB.

Also good to mention, LB are not part of app stack, it's separate service. So If you need to test it you need to separate LB and web1/2 services to docker/rancher config files.
Idea behind this usage is quite simple, in corporate company you cannot just simple open all ports on all servers/ports to all clients, so you select special separate range just for LB services, which has special firewall rules. Other hosts are not used for publishing external ports. So all 80/443 communication are setup in one stack/LB service - one selector rule for all. Same approach is used for example with Traefic.

Yes, I had tried it where the LB is not part of the app stack and it's a separate service and tried deleting the app separately. I am still able to see the LB selector rule on my LB.

LB compose files: https://gist.github.com/VAdamec/315c57273626fede1a1ce19a36b5e054

I made completely fresh environment, add LB config and add service, with this setup https://gist.github.com/VAdamec/f5a89dd36b76d535bad69abe6a12df0a
and yes, rule stay even if I delete last service which use it. So question is what's different. I'll remove and create LB in production environment to see If this helps or not.

Ok so that helps, I removed LB config and create exactly same configuration and no issue with selector rule now. Maybe some issue during upgrading between Rancher version?

I'm going to request this be re-opened. I'm experiencing the same thing (and it seems others at the top of the thread are too) and I don't really consider "just start over" to be a sufficient resolution to the bug that likely exists.
Here are my semi-anonymized docker-compose/rancher-compose files for the load balancer that is problematic: https://gist.github.com/mattdodge/a0755d3b42538314449b4183d7993d33
Like @VAdamec, this stack is separate from the stacks that the load balancer points to. So the bug that is happening is if I were to delete the service 855f7312-ad55-4908-a8bc-6e5f3245f519/image-1 then the rule for that service and the selector rule (rule.proxyable=instance) would both be deleted. The other service rule remains. The desired behavior would be for both the other service and the selector rule to exist on the load balancer.
If this doesn't get reopened I can create another issue. If I do that, I will link it to this one so at least there is some trail.

How are you launching services/deleting them? As I mentioned earlier I was unable to reproduce but I need clearer steps so I can follow exactly what you guys are doing.
For launching, are you using UI, compose/CLI or catalog?
Are you launching the LB or services first?
For deleting, are you using the UI or CLI?

Launching services via an API call (POST to /stacks endpoint). Deleting them via the UI for now. I think in all cases the load balancer was set up before the services, then the rules were added to the load balancer once the services were started.

After upgrade to 1.6.0 I'm get this issue back (rancher/lb-service-haproxy:v0.7.1)

Same here; Rancher v1.6.2

We're experiencing the issue as well in Rancher v1.6.x.

@deniseschannon We are seeing the same problem, it's easy to reproduce. I tested with rancher 1.6.4 on local machine and tried to make it as simple as possible:

created "Default" stack from ui
created "nginx" service from ui (name "nginx", image "nginx")
created load balancer "lb" from ui with 1 rule (port 9000, target "nginx", target port 80)
checked load balancer configuration via "Upgrade/Edit" button in the ui (my rule is there)
deleted "nginx" service
checked load balancer configuration again (no rules)
created "nginx" service again
checked load balancer configuration again (still no rules)

@stszap this issue is talking about selector rules, which indirectly target containers that have a label.
You're taking about rules directly pointed at a service, which are correctly (intentionally) removed if the service is removed instead of leaving garbage references pointing to nonexistent services lying around.  They are linked by ID anyway, not name, so adding a new service with the same name wouldn't be associated with the old rule even if it still existed.

@vincent99 Oh, thank you for clarification.

I believe this is related to #9118 and fixed in 1.6.6


pgsql-query-exporter fails to log into pgsql on dalco-staging

From the logs of this pgsql-query-exporter prometheus exporter it seems that the password is wrong. This is a bug.

we will close this ticket once we are sure the changes work in dalco-staging

we will close this ticket once we are sure the changes work in dalco-staging

It requires releae to dalco production. As soon as we release to dalco production, we will verify that the fix works correctly and close this issue

After the release, this issue has been fixed
2023-09-29 07:38:37,431 - DEBUG - query-exporter - connected to database "postgres_staging"

In order to see DEBUG logs, I enabled debug logs on pgsql-query-exporter service by introducing command: ["-L", "DEBUG"] command in the pgsql-query-exporter service
https://github.com/albertodonato/query-exporter#debugging--logs

thanks a lot


feat: upgrade standard to 16

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

19 of 19 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.366%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
54

Relevant Lines:
55

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 19 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.366%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
54

Relevant Lines:
55

💛  - Coveralls


Fine-4 & OpenStack Mitaka

Hi,
I'm facing an issue when trying to add a Mitaka-based OpenStack environment as Cloud Provider, where Fine-4-based ManageIQ fails to do so.
What we can see in the network flows, is:

ManageIQ tries to authenticate with 'admin' project before authenticating with domain scope.
Mitaka-based OpenStack responds 'Project does not exist'
ManageIQ stops trying, so provider is not added.

With other, newer OpenStack versions:

ManageIQ tries to authenticate with 'admin' project before authenticating with domain scope.
OpenStack responds 'The request you have made requires authentication'
ManageIQ continues trying with domain-scoped authentication, and provider is added successfully.

Where is this logic being handled for Fine-4?
Thanks!

@gianpietro1 sorry for the late response.
Can you open this question here: http://talk.manageiq.org/
You'll probably get a better response to your question there.
If you could also close this issue.

@gianpietro1 can you close this issue.

Sorry @JPrause I missed the messages! closing this right away.

@gianpietro1 not a problem. Thanks for your help! :+1:


AC-coverage error

🐛 Description
Some unknown error, while calculation the 15min bins for ac_coverage
🔢 To reproduce
input:
{
  "name": "ac1/342/342e032a.ac1",
  "type": "ac1"
}

✌🏽 Expected behaviour
Raise/catch expected exceptions for retry or move to known succeed state if this is expected.
📷 Screenshots
If applicable
💻 Traceback
| 2023-11-09T08:44:30.614+01:00 | START RequestId: 544bd636-facf-4078-9bf8-1dd9e3d57e2f Version: $LATEST
-- | -- | --
  | 2023-11-09T08:44:30.614+01:00 | [DEBUG] level1b.pre-process.handler: setting up postgres before pre-processing of 342cf799.ac2
  | 2023-11-09T08:44:31.006+01:00 | [DEBUG] level1b.pre-process.handler: checking attitude coverage for 342cf799.ac2
  | 2023-11-09T08:44:31.176+01:00 | 'DataFrame' object has no attribute 'stw_bin': AttributeError
  | 2023-11-09T08:44:31.176+01:00 | Traceback (most recent call last):
  | 2023-11-09T08:44:31.176+01:00 | File "/var/task/odincal_handler.py", line 159, in preprocess_handler
  | 2023-11-09T08:44:31.176+01:00 | assert_acfile_has_data_coverage(ac_file, backend, pg_string)
  | 2023-11-09T08:44:31.176+01:00 | File "/var/task/ac_coverage.py", line 85, in assert_acfile_has_data_coverage
  | 2023-11-09T08:44:31.176+01:00 | start = df.stw_bin.min()
  | 2023-11-09T08:44:31.176+01:00 | File "/var/task/pandas/core/generic.py", line 5067, in getattr
  | 2023-11-09T08:44:31.176+01:00 | return object.getattribute(self, name)
  | 2023-11-09T08:44:31.176+01:00 | AttributeError: 'DataFrame' object has no attribute 'stw_bin'
  | 2023-11-09T08:44:31.177+01:00 | END RequestId: 544bd636-facf-4078-9bf8-1dd9e3d57e2f
  | 2023-11-09T08:44:31.177+01:00 | REPORT RequestId: 544bd636-facf-4078-9bf8-1dd9e3d57e2f Duration: 563.26 ms Billed Duration: 564 ms Memory Size: 2048 MB Max Memory Used: 219 MB

Solved in level0 - pipeline


InteractiveElementExampleScene is in SDK Package and cant be edited

Describe the bug
I'm interested in the Interactive Element feature described here:
https://docs.microsoft.com/en-us/windows/mixed-reality/mrtk-unity/features/experimental/interactive-element?view=mrtkunity-2021-05
Unfortunately, it's not possible to view the InteractiveElementExampleScene because it's in the SDK and not in the Examples Package Manager package.

To reproduce
Steps to reproduce the behavior:

Create a new Unity project
Import MRTK 2.7
Search for InteractiveElementExampleScene and attempt to open it.
See read-only error

Expected behavior
The scene should be included in the Examples package so that it could be opened and edited.
Screenshots
Above
Your setup (please complete the following information)

Unity Version 2018.3.18f1
MRTK Version v2.7

Target platform (please complete the following information)

OpenXR

Additional context
None

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all-new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visit https://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


fix removing whitespace

fix https://github.com/vattle/sqlboiler/issues/136

This may leave blank lines, but gofmt should clean them up so I merged to get you going again. Thanks for the PR.


columnName of ForeignKeyReference is not used for PrimaryKey

DBFlow Version: 4.2.3
Hi, I just upgraded from an older version to v4.2.3 and have an issue with the code generation. I could reproduce it in a sample project as well.
Here are some classes which are referenced by a ForeignKey (which is also a PrimaryKey) and the column is renamed in ForeigKeyReference
@Table(database = AppDatabase.class)
public class AntHill extends BaseModel {
    @PrimaryKey
    public String hillId;
}

@Table(database = AppDatabase.class)
public class Ants extends BaseModel {
    @PrimaryKey
    public String antId;

    @PrimaryKey
    @ForeignKey(tableClass = AntHill.class, references = {
            @ForeignKeyReference(columnName = "hillIdRef", foreignKeyColumnName = "hillId")
    })
    public String hillId;
}

The Project won't compile as it is because there is an error in the Ants_Table.java.
error: cannot find symbol variable hillId
Caused by that generated method in the Ants_Table.java:
@Override
public final OperatorGroup getPrimaryConditionClause(Ants model) {
  OperatorGroup clause = OperatorGroup.clause();
  clause.and(antId.eq(model.antId));
  clause.and(hillId.eq(model.hillId));
  return clause;
}

It should be clause.and(hillIdRef.eq(model.hillId)); if I'm not wrong.

ill look into it. thanks for report!

in develop for 4.2.4. you can try it via 904a66738e


The HttpEventCollectorSender does not read the encrypted configuration from the quarkus profile

The configurations for splunk (url, token) are encrypted in the yaml file in quarkus application using jasypt encrpytion (See https://docs.quarkiverse.io/quarkus-config-extensions/dev/jasypt.html).  However, the encrypted values are not decrypted in HttpEventCollectorSender   .  It takes the part ENC(...) directly.
However, the encrypted value could be read normally using @ConfigProperty() or using ConfigProvider.getConfig().getConfigValue().

Interesting, I didn't know about this extension. However I think this may have to be reported to https://github.com/quarkiverse/quarkus-config-extensions ?
I don't see it mentioned in the doc or an integration-test for Quarkus extensions @ConfigItem, which are a bit different from regular @ConfigProperty(my.secret)

@rquinio1A , thanks for your reply. I think so. I debugged in the code a little bit and created a new issue there. See https://github.com/quarkiverse/quarkus-config-extensions/issues/231
I will close this ticket.


Weird behavior

BUG REPORT INFORMATION
I have two container images where are built from the same process,
Container 1 consists of base image + binary A, EntryPoint: /bin/sh -c /usr/local/bin/binaryA
Container 2 consists of the same base image + binary B, EntryPoint: /bin/sh -s /usr/local/bin/BinaryB
Container 1 works fine, containers 2 return "127" error code, which is "Not found"
I use "docker exec -it container1 /bin/sh" and in the container,  BinaryA is there in /usr/local/bin and I can execute it in the container.
I use "docker run -it container2 /bin/sh" and in the container , BinaryB is also there in /usr/local/bin BUT I CAN NOT execute in the container and I got a "/bin/sh: ./BinaryB: not Found" , BinaryB has 'x" attribute set and I can even use vi to open it.
Other observation, if in /bin folder, I run "cp a b", "a" can run, but "b" can not,("applet not found") error.
Really confused here and can any one help me?
Thanks
Henry
Steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Additional information you deem important (e.g. issue happens only occasionally):
Output of docker version:
1.9.1
(paste your output here)

**Output of `docker info`:**
Containers: 15
Images: 29
Server Version: 1.9.1
Storage Driver: aufs
   Root Dir: /var/lib/docker/aufs
   Backing Filesystem: extfs
   Dirs: 59
   Dirperm1 supported: false
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.13.0-24-generic
Operating System: Ubuntu 14.04 LTS
CPUs: 2
Total Memory: 3.839 GiB

(paste your output here)

**Additional environment details (AWS, VirtualBox, physical, etc.):**

Can you put together a reproducible testcase?

tonistiigi:
I am trying run kubemark in a kubernetes environment.  It doesn't have an easy repro.
Sorry about it.
Henry

"applet not found" is normally a busybox error. You do not however give enough information to be able to help. Basically you have a non executable binary, common causes are that it is missing shared libraries for example. If it is actually busybox, it looks at the name of the executable, so it matters what the bonary is called. Try renaming it to busybox.

I am going to close this as it is not a bug in Docker, but with some more information I expect we can work out what the issues is.


[FLINK-20347] Rework YARN documentation page

What is the purpose of the change
This PR depends on https://github.com/apache/flink/pull/14223 (that's why I wrote it into a new page).
The Flink on YARN documentation page, completely rewritten from scratch, with the following principles / audiences in mind.
1st section: “Getting Started”
Beginners guide in session mode for having a cluster running in less than 5 minutes.
This is meant for somebody who has a YARN cluster, and wants to quickly try out Flink.
2nd section: "Deployment Modes Supported"
Intermediate level for quick reference on the different modes.
This is meant for somebody who knows Flink already a bit (typical config params), but needs a reference for the different deployment modes.
3rd section "Reference"
special topics for Flink on YARN, advanced usage. Description of behaviors.
This is meant for somebody productionizing Flink on YARN.

Did you leave Accessing the logs out on purpose?

Yes, because that's not something Flink specific, it's general YARN knowledge. Maybe I find an opportunity to mention it somewhere, but not as a full section.
Thank you for your review. I'll rebase the change to the new doc structure, then address your comments.

I pushed an updated version of this PR.

Thanks a lot for the through review. I've addressed all comments.

Thanks a lot for your reviews! I'll merge the PR soon.


[BUG] remove telemetry

Please make sure you've tried the latest version of Fritter first, as the F-Droid version is sometimes out of date!

Describe the bug
Telemetry sentry_flutter found & is currently being used as a logging tool.
Why does it need to be there for a privacy centric client app?
// main.dart

var sentryOptions = SentryOptions(dsn: 'https://d29f676b4a1d4a21bbad5896841d89bf@o856922.ingest.sentry.io/5820282');
  sentryOptions.sendDefaultPii = false;
  sentryOptions.attachStacktrace = true;

  var sentryClient = SentryClient(sentryOptions);
  var sentryHub = Hub(sentryOptions);
  sentryHub.bindClient(sentryClient);

To Reproduce
n/a
Screenshots
n/a
Device
Pixel 6, Android 12, en, v2.16.3

The app description says: "Fritter uses Sentry for opt-in error reporting, which is not enabled by default."

This feature seems to be opt-out, not opt-in, as it is enabled by default. At least the checkbox is checked when you install the app.

The usage of Sentry is opt-in. It's not enabled by default - when the first error happens, the user is presented with a dialog asking whether they want to turn it on or not, or even submit the error once without enabling Sentry permanently.


Adds ability to unset decorations

Signed-off-by: Hal Gentz<EMAIL_ADDRESS>

CI failure is unrelated:

E: Failed to fetch http://apt.llvm.org/precise/pool/main/l/llvm-toolchain-snapshot/clang-5.0_5.0~svn299990-1~exp1_amd64.deb  Connection failed

bors retry