Expose ContentPluginProps?

Hey, cool project! I'm just now starting to create some custom plugins, but I definitely feel I'm going about it awkwardly. I basically just copied code for an existing ory-editor-plugins component and have been trying to get it to run. One issue I found is the ContentPluginProps type is not accessible in ory-editor-core. Can that be exposed so I can import it?
Or am I going about this the wrong way?

Hey, the ContentPluginProps  are just a type (in the context of flow). Importing them won't really do anything because we don't expose / compile types yet. Other than that, you're going at it the right way!

Ah, got it. I took out the part defining the argument type and that got me rolling again. Thanks for your help!
For starters I'm adding the option for an Image to be a link. I think I can probably throw together a pull request when I'm done.

Awesome! :)


macOS Support

Loved the size & simplicity of the framework, and noticed it wasn't too far off from being able to support Mac in a very similar way. Sense the Mac is a bit farther behind on being able to do these kinds of animations easily like on iOS, this proves pretty useful.
Tried my best to not change any behavior, while keeping the code the same as much as possible.

Hey Sam — so cool that you’re submitting this! I’m out of town on vacation until Tuesday. I’ll review as soon as I get back.

Also, please make sure to use the same indentation as in the original source files.

Thanks for kicking this off. I've merged macOS support to master.


Add types and minor code refactors to half of eolearn.features

Got tired of thinking about area managers and eopatch managers, so I did some cleaning

Codecov Report
Base: 83.70% // Head: 83.73% // Increases project coverage by +0.02% :tada:

Coverage data is based on head (23609a6) compared to base (d843b30).
Patch coverage: 94.87% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #513      +/-   ##
===========================================
+ Coverage    83.70%   83.73%   +0.02%
===========================================
  Files           60       60
  Lines         5657     5661       +4
===========================================
+ Hits          4735     4740       +5
+ Misses         922      921       -1

Impacted Files
Coverage Δ

features/eolearn/features/feature_manipulation.py
96.39% <87.50%> (+0.06%)
:arrow_up:

features/eolearn/features/bands_extraction.py
100.00% <100.00%> (ø)

features/eolearn/features/blob.py
100.00% <100.00%> (ø)

features/eolearn/features/clustering.py
98.24% <100.00%> (-0.06%)
:arrow_down:

core/eolearn/core/extra/ray.py
95.12% <0.00%> (+2.43%)
:arrow_up:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


IstioIngress should supply default values for Content-Security-Policy, X-Content-Type-Options, and X-XSS-Protection headers

I recently ran a 3rd party scanner on Bookinfo Productpage and got warnings about lack of Content-Security-Policy, X-Content-Type-Options, and X-XSS-Protection headers.
I noticed an SAP blogger suggested these and a few other headers in https://blogs.sap.com/2019/02/11/kubernetes-security-secure-by-default-headers-with-envoy-and-istio/ and proposed using EnvoyFilter to add them.
Would it make sense to configure these as defaults for Istio installs?
cc @GregHanson

I do not think we should do this. This is far too opinionated. We provide APIs that expose knobs to tweak response headers.

Would it make sense to provide this as a sample?

Seems reasonable if you want to add one

Closing as there is no plans to add these. Users can add them  through existing APIs if they want


Sink Feedback for Carrier is incorrect

When I sank the carrier, the image for the Destroyer showed as sunk (image attached below). I can also see that there are no other possibilities for another ship to be vertical at the end of the carrier, where the submarine and carrier meet.
When I sink a ship, the correct ship should be marked as sunk

I think that there is another ship that is vertical along the right wall and that is why it says the destroyer is destroyed and not the carrier


Fix:Move to top hover is not visible on hover.

Description

Hover on move to top button was not visible and at the bottom the button was not visible due to white color.
I had changed the color of Move to top button on both normal and on hover.
Related Issues

Closes #455

Type of PR

[X] (#455)

Screenshots / videos (if applicable)

Checklist

[X] I have gone through the contributing guide
[X] I have updated my branch and synced it with project main branch before making this PR
[X] I have performed a self-review of my code
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

Additional context:

@Anjaliavv51 please review it and merge it.

@gyanshankar1708 implement the effect to all hmtl pages and also UI of the button color should match webpage color's

@Anjaliavv51  I had changed to all the pages.
Please review it nad merge it.

@Anjaliavv51 I had changed to all the pages. Please review it nad merge it.

color is not changes still.
look in ur deployment in below link
https://deploy-preview-458--rococo-mooncake-0d89f7.netlify.app/

@Anjaliavv51  I am closing this PR. I had solved #468. after merging that I'll work on this.


ARROW-9259: [Format] Add language indicating that unsigned dictionary indices are supported but that signed integers are preferred

This does not alter the format metadata in any way but has implications for the reference implementations (e.g. C++ currently rejects unsigned integer indices).

+1, vote result https://lists.apache.org/thread.html/rf239738d3ce23878ade6eb87d0e69f5ce685dd335c45a06d9c05c43a%40<dev.arrow.apache.org>


Request for help with installation

Hi,
i'm using a mac ( for the first time) and i'm having some problems with the installation of basis.
I've downloaded the project's folder and runned the command cmake "CMakeLists.txt" and "make" from the terminal following the READ.me instructions.
At the end i get the message [100%] Built target basisu.
After that if i try the command basisu i get the error "command not found".
Am i missing something?
Thank you

It should be in the bin_osx directory, so if you change to that directory then you can run it with ./basisu. If you want to install it, you can use make install which will then install it to /usr/local/bin and you should then be able to run basisu

Yes, look in the bin_osx directly. Let me know if you have any other problems.

Thank you both


Enhancing Local LLM Use with Proxy Configuration

Thank you very much for your work, which allows me to use LLMs conveniently locally. However, I have a small issue here: my network environment requires the use of a proxy to access GitHub. It would be very helpful for me if you could add the function of configuring a proxy in the software. Thank you.

I also need proxy to access huggingface .

Thank you very much for your work, which allows me to use LLMs conveniently locally. However, I have a small issue here: my network environment requires the use of a proxy to access GitHub. It would be very helpful for me if you could add the function of configuring a proxy in the software. Thank you.

没必要吧。 自己直接从 hugginface 的 mirror 上下载gguf 模型放到目录里面就行了


Unhandled Exception

This mod looks so cool. Can't wait for the fix for this

Hi mgroth; I know this is a super late reply, so I apologize for that.
Question:  Were you able to install Memoria and extract the assets successfully?  I think that hasn't happened, which is why you received this error.
(Side note:  We plan on implementing an "installer" in our next release)


Can we provide a documented guarantee that the Params.IoStatus field is always valid?

The page says:

Note that the completion parameters structure contains valid information only if the driver has formatted the request by calling one of the WdfIoTargetFormatXxx methods.

However, WDF requests can also be formatted using WdfRequestWdmFormatUsingStackLocation, in which case I believe the IoStatus field in the completion params is valid (and only that field). This should be documented. Perhaps we should say that field is always valid.
@Shyamal-V FYI.

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

ID: fb9e1e59-835a-5619-30ad-ea16c001bf6c
Version Independent ID: 5e688c84-1338-f55d-c579-028043e62d18
Content: EVT_WDF_REQUEST_COMPLETION_ROUTINE (wdfrequest.h) - Windows drivers
Content Source: wdk-ddi-src/content/wdfrequest/nc-wdfrequest-evt_wdf_request_completion_routine.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @tedhudek
Microsoft Alias: tedhudek

Thanks for the feedback @rahul-ramadas and apologies for the delay.
Spun up #874 to address.  Could you and @Shyamal-V please review?

Thanks for getting the ball rolling on this @rahul-ramadas
I just merged the fix - should be on the live site within the hour.


Verify migration

This PR has two parts:

A manual script to verify the DB migration process. This script verifies that the service provider can create a DB from scratch with the required view for retrieving triggers, as well as add the view to an existing DB. It only needs to be run once (I've run it).
Cloudant was being very, very slow as I was developing this script, and I noticed that when this happens, the situation can arise where the database migration is kicked off before the DB has actually been created. For this I have moved Database initialization to inside the main function.

@bjustin-ibm do you want to merge this PR or close it?

There is a conflict now

@csantanapr I have rebased this work and all tests still pass.


purge-cluster: remove unneeded tasks

This removes tasks from the end of the purge-cluster playbook that are not needed anymore. This was noticed because in rhcs we install ceph-common on clients nodes and that package does not include ceph.target so the playbook would fail.  Removing the packages should stop that service anyway if it is running, eventually we'll want to not install ceph-base upstream on client nodes either as it's not meant to be installed standalone.
There is also some cleanup on sysv and upstart tasks.

jenkins test jewel-ansible2.2-purge_cluster

jenkins test jewel-ansible2.2-centos7_cluster

jenkins test kraken-ansible2.2-docker_cluster

jenkins test kraken-ansible2.2-xenial_cluster

jenkins test jewel-ansible2.2-centos7_cluster

@ktdreyer thoughts on backporting this to stable-2.2? We do not have a BZ for it currently but I imagine we will soon.

sure, backport it

We seem to be having issues pulling from docker hub, perhaps a side effect of this scheduled maintence: https://status.docker.com/pages/maintenance/533c6539221ae15e3f000031/58fe66db1e525b480e00008f
Regardless, this PR does not introduce any changes to our docker tasks so I think it's safe to merge.


[get_started.md] Fix typos in MMDetection 3.x

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Iwant to achieve through this PR.
Modification
The Code error occurred in strp 2 of verify the installation in MMdetection 3.x get_started.md.
Fix the Code of yolov3_mobilenetv2_8xb24-ms-416-300e_coco.py to yolov3_mobilenetv2_8xb24-320-300e_coco.py.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMCls.
The documentation has been modified accordingly, like docstring or example tutorials.

@Zheng-LinXiao Thanks for the report of the typos.


[FEATURE] Create NodeClient or HLRC or RemoteClusterClient implementation in SDK Client and deprecate it

Is your feature request related to a problem?
To speed migration work for AD Plugin (and other existing plugins) we will create a client that can use the existing (NodeClient) API calls from those plugins.
What solution would you like?
Create a new getter in SDKClient that returns an instance extending NodeClient that can be used by REST Handlers.  Add deprecation annotations and javadocs clearly stating that it is provided for backwards compatibility and will eventually be removed.
What alternatives have you considered?
See #294
Do you have any additional context?
See https://github.com/opensearch-project/OpenSearch/issues/5424 regarding eventual deprecation.

Existing plugin implementations use this signature:
protected RestChannelConsumer prepareRequest(RestRequest request, NodeClient client) throws IOException { ... }

Proposed implementation:

Add another level of inheritance between BaseExtensionRestHandler and the individual implementations.
Include an abstract prepareRequest() method with the above signature.
Implement the necessary bits to translate from the routes() to handle the request/response handling similar to how is done in the handleRequest() method of BaseRestHandler.

So instead of this, we will just return the REST response and the rest should be taken care by high level rest client

I don't think that's such a simple replacement.  client.execute() still has to do something to execute the transport; the arguments include 3 arguments: the action (a constant, we can deal with), the request (that the action needs), and an action listener for the response.  (In this case it's created by a private method.)
So whatever client we create still has to pass the action identifier, and the request, to be executed by another class.

Looking further into this:

RemoteClusterAwareClient looks like it could work if we had designed Extensions from scratch to run as a cluster.  Since we didn't, it ends up being pretty heavyweight for this purpose.
The client implementation as a wrapper is a good example of future development we may want to do to wrap RestClient calls in a HLRC-syntax-compatible wrapper.
HLRC is still probably the way to go with some internal tweaks.

Raised a draft PR for @dbwiddis POC work for converting AD plugin to AD Extension using High Level Rest Client: https://github.com/opensearch-project/anomaly-detection/pull/773


ember-drag-sort broken in Firefox prior to Quantum

It does not look like ember-drag-sort supports older versions of Firefox.
This is with 55.0.3 > https://cl.ly/2D0a300x2D2i

I tried reproducing the issue in BrowserStack and failed. Tried Firefox 55 and 54 on WIndows 10.
Though there's an error visible in the console: Permission denied to access property "type", originating from jQuery, sorting works correctly from the user perspective.
To further investigate this issue, I need a way to reproduce it. Please try reproducing it on BrowserStack.

@lolmaus I am unable to reproduce this issue in BrowserStack.


fix(authentication-service): handle the unnecessary KEYCLOAK env variable

Description
Handle the unnecessary KEYCLOAK env variable
Fixes #2011
Type of change
Please delete options that are not relevant.

[x] 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 not work as expected)
[ ] Intermediate change (work in progress)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[ ] Performed a self-review of my own code
[ ] npm test passes on your machine
[ ] New tests added or existing tests modified to cover all changes
[ ] Code conforms with the style guide
[ ] API Documentation in code was updated
[ ] Any dependent changes have been merged and published in downstream modules

CI failing @arpit1503khanna

Fixed in another PR https://github.com/sourcefuse/loopback4-microservice-catalog/pull/2016


The HTTP request is not available through methods decorated

Describe the bug
The HTTP request is not available through methods decorated .
Expected behavior
The request object should be available everywhere, as this helps us build multi-agent chatbots. The current implementation makes that impossible.

Thanks for the feedback @ahmadmayahi. Though I should like to ask that before you create bug reports you take a minute or two to see whether a similar issue already exists.
In this case, it's a dup of #1213. Happy to hear your thoughts on my proposal there -- or possibly even your PR towards that end!


Add free(::XMLNode) and free(::XMLElement)

Adds a manual solution to #17 - how to deal with finalizers, gc, and whether or not we need reference counting for child elements is still an open question but I don't think there's any harm in adding a manual free method to the exposed API here.
cc @robertfeldt you had done something similar in your branch. I've made it work for XMLNode as well, because why not.

I added unlink functionality too, since I'm in need of it. cc @lindahua or @garborg or anyone else have any feedback? If not I'll merge and tag fairly soon since I need this or something equivalent. Might be interesting to look at the coverage here and add some more tests. My package will indirectly test this functionality, but would take some work to separate what it does out into standalone tests.


No detection of known IOC IP in access logs

🐛 Bug Report
app.py does not detect any known IOC IPs from the indicators.py file.
To Reproduce
Steps to reproduce the behavior:

Downloaded Pulse events and access logs from device to ./logs2/ directory.
Run python3 app.py --path logs2/
app.py found no indicators.
Replaced all IPs in the logs2/pulse.access log with one of the known IOC IPs from indicators.py.

Expected behavior
Expecting app.py to find a known IOC from indicators.py in the pulse.access log.
 This does not mean there was no compromise.
No output file was specified. If you would like an outfile, rerun and specify --raw, --json, or --csv

Manually check for known IOC IP in pulse.access log
~/check-your-pulse# grep -m 5 -ri <IP_ADDRESS> ./
Binary file ./src/__pycache__/indicators.cpython-37.pyc matches
./src/indicators.py:    "<IP_ADDRESS>",
./logs2/pulse.access:2019-04-18 06:10:05 - psa1 - [<IP_ADDRESS>(unresolved)] UserID(My_Realm)[] - Password realm restrictions successfully passed for UserID/My_Realm
./logs2/pulse.access:2019-04-18 06:10:05 - psa1 - [<IP_ADDRESS>(unresolved)] UserID(My_Realm)[] - Primary authentication successful for UserID/MyDomain from <IP_ADDRESS>(unresolved)
./logs2/pulse.access:2019-04-18 06:10:05 - psa1 - [<IP_ADDRESS>(unresolved)] UserID(My_Realm)[] - Password realm restrictions successfully passed for UserID/My_Realm
./logs2/pulse.access:2019-04-18 06:10:08 - psa1 - [<IP_ADDRESS>(unresolved)] UserID(My_Realm)[] - Secondary authentication successful for UserID/RSA Authentication from <IP_ADDRESS>(unresolved)
./logs2/pulse.access:2019-04-18 06:10:11 - psa2 - [<IP_ADDRESS>(unresolved)] UserID(My_Realm)[] - Host Checker policy 'RegCheck' passed on host <IP_ADDRESS>(unresolved)  for user 'UserID'.

Thank you for leaving an issue! We are troubleshooting now.

Please test with this branch and let us know if it works!

Same result: check-your-pulse did not find any activity based on the IOCs. This does not mean there was no compromise.  I cloned the branch:issue3 and made sure my local copy had the change you made, it does.

Same result: check-your-pulse did not find any activity based on the IOCs. This does not mean there was no compromise. I cloned the branch:issue3 and made sure my local copy had the change you made, it does.

This is a pickle, we assumed the issue was with the [IP(message)] formatting in the IP. We will continue troubleshooting on our end.

It is strange.  The only thing I can think of is we use a Pulse cluster so the log is slightly different than if it were a single standalone.

It is strange. The only thing I can think of is we use a Pulse cluster so the log is slightly different than if it were a single standalone.

I believe I've found the issue, it does have to do with the log formatting. I should have a solution for you in 30 minutes or so for testing.

Ok, new push to issue3 which is matching on the strings in your initial post, give it a shot!

Yahtzee! looks good now.  Much appreciated!

Just glad I could help!

Glad you created this.  Luckily I have a year's worth of logs to parse!  One feature request: some logic to run the parser even if one or the other of events/access logs are missing.  We only syslog our access logs off so I had to pull a fresh one to drop into the logs folder in order for the access log to parse.

Glad you created this. Luckily I have a year's worth of logs to parse! One feature request: some logic to run the parser even if one or the other of events/access logs are missing. We only syslog our access logs off so I had to pull a fresh one to drop into the logs folder in order for the access log to parse.

Interesting, we will make note of that. For your immediate purposes, check-your-pulse just looks for a .events file existing in that folder, so you can drop a blank one there. Just note that the fidelity of the IOCs is decreased with less data to enumerate.

Perfect! thanks again.


fix: service methods have changed since '/file' war remove from base

at least the RCPTT part is not needed anymore...


gradle plugin artifacts not signed

Which version and edition of Flyway are you using?
10.15.2 or any other version.
If this is not the latest version, can you reproduce the issue with the latest one as well? (Many bugs are fixed in newer releases and upgrading will often resolve the issue)
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Gradle plugin
Which database are you using? (Type & version)
Not applicable
Which operating system are you using?
Not applicable
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
Added the gradle plugin.
What did you expect to see?
Successful verification of the gradle plugin with the release signature that's used for the flyway library which passes the verification thanks to:
<trusted-key id="F79157DB93697AA32CD4C46CC485C5A843FADB15" group="org.flywaydb"/>

What did you see instead?
Error resolving plugin [id: 'org.flywaydb.flyway', version: '10.15.2']
> Dependency verification failed for configuration 'detachedConfiguration1':
    - On artifact org.flywaydb.flyway.gradle.plugin-10.15.2.pom (org.flywaydb.flyway:org.flywaydb.flyway.gradle.plugin:10.15.2) in repository 'maven': checksum is missing from verification metadata.

Workaround is to update verification-metadata.xml with the checksum instead each time the gradle plugin is updated. Would be great if it was signed instead, as the flyway library has the infrastructure to do that already.
https://repo1.maven.org/maven2/org/flywaydb/flyway-core/10.15.2/ correctly contains .asc files for all artifacts related to the flyway library, while equivalent signatures seem to be lacking in the gradle plugin case.

Nah, parts of the artifacts are not signed, see here for example:
https://plugins.gradle.org/m2/gradle/plugin/org/flywaydb/gradle-plugin-publishing/10.18.1/


update julia install doc

cc @iblis17

Preview link is here:
http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-15609/1/install/index.html?platform=Linux&language=Julia&processor=CPU
That code block display is weird.

@mxnet-label-bot add [Julia, Doc, pr-work-in-progress]

Could you add a section about "install the latest release" ?... via
]add MXNet

http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-15609/4/install/index.html?platform=Linux&language=Julia&processor=CPU

looks fine.
Could you rebase to re-trigger CI again?


fix(package): move Marked to required dependencies

marked is necessary to run this project, without it developer will see the following error in console:
ERROR in ./~/markdown-to-html-pipe/src/markdown-to-html.pipe.js
Module not found: Error: Can't resolve 'marked' in '/Users/danielkucal/Applications/Angular2/danielkucal.com/node_modules/markdown-to-html-pipe/src'
 @ ./~/markdown-to-html-pipe/src/markdown-to-html.pipe.js 3:13-30
 @ ./~/markdown-to-html-pipe/index.js
 @ ./src/app/app.module.ts
 @ ./src/main.ts
 @ multi webpack-dev-server/client?http://localhost:4200 ./src/main.ts

@DanielKucal First of all: thank you for your pull request!
Actually this behavior is intended. You just have to add marked as a dependency by yourself (the same way as you do with Angular; npm install --save marked).
The benefit is that you can upgrade to newer (future) versions of marked without possible duplicate installs (if you would upgrade to a future version of 0.4.0, you would also have 0.3.x installed). It is likely, that markdown-to-html-pipe won't break for marked updates as it uses only a small bit of its interface.
Do you agree with that?
9110e718122b3392f98fc6f7cca73c049b8690bd updates README.md accordingly.

Well, I don't think it's something that should be trying to be resolved in an npm module. There are tools like npm dedupe and npm tries to avoid duplications and maintain flat dependencies structure itself for this purpose...
Moreover, version ^0.3.6 implies that all 0.X.X versions are compatible with this project, so it's a bad example with 0.4.0 (would be correct for ~0.3.6).
Although correcting the installation command can be enough, I find it a bit confusing for new users.
By the way, thanks for this great pipe! It's really useful :)

Actually 0. versions are an exception of this rule: ^0.3.6 will be handled similar to ~1.0.0 (^0.x would work like the 1.0-caret.
The version used by this plugin should always be the same as the Angular application uses. Otherwise the wrong marked would be configured by the user as we don‘t provide an API for configurations.
That would be much more frustating than a missing import.

I see, you are right... Have you tried it with marked set to *?

@DanielKucal sorry for the late response. I thought about it a bit and I guess ^0.x would be the best solution. If you change your PR accordingly, I will merge it.

I have updated the PR

Merged! Thank you very much for your feedback and contribution!


Object(...) is not a function error in TypeScript 3.9.7 and Ionic v3(Angular v6) environments

Please consider that it is a translator. 🙏
Hello, I'm Korean DatoCMS user
I'm leaving an issue because I have a question about DataCMS.
Which version of TypeScript is supported by the datocms-structured-text-to-html-string library provided by DatoCMS?
The version of TypeScript I'm using now is version 3.9.7.
The framework uses Ionic v3 (Angular v6).
The validate method of datocms-structured-text-utils was used to determine that it was a valid dast grammar. If you use the render method, the error message Object(..) is not a function is output.
Attached is a photo that proves that it is valid when validating with error message below.

Please contact me if you need more information.
It works in Angular v10 & TypeScript 4.0.3 environments.
I look forward to hearing from you. Have a nice day :)

Hi! Can you please paste here the exact structure/data that you're passing to the datocms-structured-text-to-html-string render method please? Angular/TypeScript should not be an issue here, so it must be something else.

Hi, Here's the data.
    "schema": "dast",
    "document": {
        "type": "root",
        "children": [
            {
                "type": "paragraph",
                "children": [
                    {
                        "type": "span",
                        "value": "비즈넵 서비스 긴급 점검 중!"
                    }
                ]
            },
            {
                "type": "list",
                "style": "bulleted",
                "children": [
                    {
                        "type": "listItem",
                        "children": [
                            {
                                "type": "paragraph",
                                "children": [
                                    {
                                        "type": "span",
                                        "value": "작업일시: 2021.06.11 오후 2시 ~2021.06.12 오후 2시"
                                    }
                                ]
                            }
                        ]
                    },
                    {
                        "type": "listItem",
                        "children": [
                            {
                                "type": "paragraph",
                                "children": [
                                    {
                                        "type": "span",
                                        "value": "작업사항: 서비스 개선"
                                    }
                                ]
                            }
                        ]
                    }
                ]
            },
            {
                "type": "paragraph",
                "children": [
                    {
                        "url": "https://www.notion.so/bznav/cd055a3c2d8b40579e2867d92b2f3e75",
                        "type": "link",
                        "children": [
                            {
                                "type": "span",
                                "value": "자세한 사항 확인하기"
                            }
                        ]
                    }
                ]
            }
        ]
    }
}```

Mh, I see no problems here: https://codepen.io/stefanoverna/pen/KKmMxEe

Oh... The same error occurs when stored in a separate variable. :(
      schema: 'dast',
      document: {
        type: 'root',
        children: [
          {
            type: 'paragraph',
            children: [
              {
                type: 'span',
                value: '비즈넵 서비스 긴급 점검 중!'
              }
            ]
          },
          {
            type: 'list',
            style: 'bulleted',
            children: [
              {
                type: 'listItem',
                children: [
                  {
                    type: 'paragraph',
                    children: [
                      {
                        type: 'span',
                        value: '작업일시: 2021.06.11 오후 2시 ~2021.06.12 오후 2시'
                      }
                    ]
                  }
                ]
              },
              {
                type: 'listItem',
                children: [
                  {
                    type: 'paragraph',
                    children: [
                      {
                        type: 'span',
                        value: '작업사항: 서비스 개선'
                      }
                    ]
                  }
                ]
              }
            ]
          },
          {
            type: 'paragraph',
            children: [
              {
                url: 'https://www.notion.so/bznav/cd055a3c2d8b40579e2867d92b2f3e75',
                type: 'link',
                children: [
                  {
                    type: 'span',
                    value: '자세한 사항 확인하기'
                  }
                ]
              }
            ]
          }
        ]
      }
    };

    const result = render(data);
    console.log(result);

What version of datocms-structured-text-to-html-string are you using?

I am using 1.1.4 version. The same error occurs after repeated install and install several times.

@stefanoverna Are you following this issue internally? I wish I could get some help...🥲

Are you able to share a small repository reproducing the issue? I really don't know what could be the cause honestly  🙄

I think it'll be difficult because it's a company code.. 😢  Could it be related to tsconfig?

it does not need to be the company's code, just a small chunk of code that reproduces the issue (codepen?). my hands are kinda tied otherwise, I'm sorry

I tried to set and test the stackblitz according to our project configuration as much as possible, but it works well, and I wonder why it doesn't work well for us. That's why I talked about tsconfig.
https://stackblitz.com/edit/ionic-2dwkxk?file=pages/home/home.ts

More) I debugged it with Chrome devtool and found that child is undefined in the 42nd row of render.js. However, if you look at the node in the right toolbar 'Scope', you can see that there is a child in the node.

It seems to me that children is undefined because you stopped the code execution before the ternary expression was evaluated 100%.

Oh, I understand. I'll try to figure it out more internally in this. Please don't close the issue. 🙏

@stefanoverna This issue has been resolved! The array-flatten module was installed in the project and it was solved. Thank you very much for your help. 👍 🙏


missing tests - base

[ ] loops anonymous
[ ] loops named
[ ] tap
[ ] flags
[ ] unwinding, wrapping, multiple buffers, stream-stuff
[ ] manual push & pop
[ ] skipping
[ ] offsets
[ ] queueJobs


Verify ledger at hard fork

At the time of the upcoming hard fork to accommodate Snapps and other changes, the structure of accounts will have changed.
We'll want some automated way to verify that the ledger contents have not changed, modulo the changed structure.

package verification tool does this


Removes 'Live Tracer'

I believe it's not working anymore.

https://github.com/Igglybuff/awesome-piracy/pull/524#issuecomment-848969254

https://github.com/Igglybuff/awesome-piracy/pull/524#issuecomment-848969254


feat: oci genai

Closes #1049
📑 Description
Adds Oracle Cloud Infrastructure (OCI) GenAI service as a backend for k8sgpt.
✅ Checks

[] My pull request adheres to the code style of this project
[] My code requires changes to the documentation
[] I have updated the documentation as required
[] All the tests have passed

ℹ Additional Information

CompartmentId is added as a new parameter for OCI GenAI.

Thanks for this contribution! Would you please add an example for this in the README under the
tab and/or a docs contribution also? https://github.com/k8sgpt-ai/docs/issues

Hi Alex,
I'll create a docs PR for this feature.

semantic checks seem to be passing now, had to force push the branch.

@AlexsJones FYI https://github.com/k8sgpt-ai/docs/pull/105

More of an FYI, but if you want this to work in the operator there need to be some changes

@k8sgpt-ai/k8sgpt-maintainers  please review

I haven't done any actual testing, but the code looks fine. I asked a question I don't know much about OCI.

Let me know if you would like to do any OCI testing, I can assist with this.

More of an FYI, but if you want this to work in the operator there needs to be an addition in the supported backends

Is this something I update in this repo, or in the operator repo? Having the operator support could be good 👍


Support for Server-Sent Events (SSE)

For our project we would like to use SSE. Currently, code to activate SSE for LiveReload 3.0.0 is quite simple:

if (LiveReload) {
	LiveReload.connector.disconnect();
}

var source = new EventSource(url);

source.addEventListener ('reload', function (evt) {
	var files;
	try { files = JSON.parse (evt.data); } catch (e) { return; }

	if (LiveReload) {
		files.forEach (function (file) {
			LiveReload.performReload ({path: file})
		});
	} else {
		location.reload ();
	}
});

source.addEventListener ('alert', function (evt) {
	console.log ('alert event', evt);
	if (LiveReload) {
		LiveReload.performAlert (evt.data);
	}
});

Unfortunately, this not works with minified version nor version 3.0.1. I would like to make it works without LiveReload.connector.disconnect() if that's fine for you

I'm not very up-to-date with this. This would function as a replacement for websockets, right?
If so, how would you want to signal you want to use SSE instead? Or would livereload 'listen' to both at the same time?

Unfortunately, this not works with minified version nor version 3.0.1.

You mean this doesn't work with something that happened with 3.x (as in: it did work with 2.x)?
Or what is problematic with version 3.0.1 and/or minification?

I would like to make it works without LiveReload.connector.disconnect() if that's fine for you

You mean you currently need the first 3 lines to stop the websocket? In that case, it is connected to my first question I guess?

I'm not very up-to-date with this. This would function as a replacement for websockets, right?
If so, how would you want to signal you want to use SSE instead? Or would livereload 'listen' to both at the same time?

Actually, I'm preparing the pull request for this. As a LiveReload user, you can choose between WebSockets and EventSource (SSE). WebSockets will be default choice.
From my point of view WebSocket is just a transport to send reload and alert commands. SSE will be additional transport to send those events. I'm planning to add transport property to the Options object and in case transport = sse, different connector will start and emit reload or alert events. Options already contains host, port and path, so transport will be enough.

You mean this doesn't work with something that happened with 3.x (as in: it did work with 2.x)?
Or what is problematic with version 3.0.1 and/or minification?

Not sure about problem in minified version, but I've fixed issue with 3.0.1 in #88

You mean you currently need the first 3 lines to stop the websocket? In that case, it is connected to my first question I guess?

With my patch, disconnect() will not be needed anymore, so no problem with minified version.

I'm fully comprehending what the issues with the minified version are, but if you can/could solve them: awesome.
The idea of multiple transports sounds nice! Looking forward to your PR!


switching from address to wallet selects the same btc wallet and account, the account shouldn't be the same by default

maybe it’s better to hide the address and wallet segment if there's nothing to do in the wallet segment (e.g only one wallet of an asset type with only one accoun)

maybe it’s better to hide the address and wallets segment if there's nothing to do in the wallet segment (e.g only one wallet of an asset type or only one account for the wallet to be selected)

I thinks we should show only address segment if have one wallet and one account

maybe it’s better to hide the address and wallets segment if there's nothing to do in the wallet segment (e.g only one wallet of an asset type or only one account for the wallet to be selected)

I thinks we should show only address segment if have one wallet and one account

that is current, but no point showing the segment title in that case, just the content is fine

maybe it’s better to hide the address and wallets segment if there's nothing to do in the wallet segment (e.g only one wallet of an asset type or only one account for the wallet to be selected)

I thinks we should show only address segment if have one wallet and one account

that is current, but no point showing the segment title in that case, just the content is fine

I mean we will only show send to address with no segment if there is only one wallet and that wallet has only 1 account

correct


How to configure FluentValidation.resources.dll

How to configure / use FluentValidation.resources.dll ? is there any example ?

There's nothing to configure...this is just the standard localization mechanism available in all .NET apps.
So long as the appropriate resource file for the language you want to use is located in the bin directory (eg bin\de\FluentValidation.resources.dll) you can use those localized messages in your application:
If you're not sure how .NET localization and satellite assemblies work, I'd suggest taking a look at some of the .NET documentation. Here's an introduction:
https://blogs.msdn.microsoft.com/global_developer/2011/07/22/introduction-to-satellite-assemblies/
https://msdn.microsoft.com/en-us/library/1ztca10y(v=vs.85).aspx


Which dll/so?

Could anyone update the README so that it's clear what is needed?
Where to get the dll/so ? A help channel is comming.

You need grab them from internet.

is simply not enough

The most painless way for Linux/ OS X users is this
Run: wget http://pgoapi.com/pgoencrypt.tar.gz && tar -xf pgoencrypt.tar.gz && cd pgoencrypt/src/ && make && mv libencrypt.so encrypt.so
Then copy encrypt.so into your bots main folder, done

We can't provide them directly, however, users in the slack channel can help you here.

ok heres the full way for mac users:

in terminal:

brew install wget

cd PokemonGo-Bot

wget http://pgoapi.com/pgoencrypt.tar.gz && tar -xf pgoencrypt.tar.gz && cd pgoencrypt/src/ && make && mv libencrypt.so encrypt.so

search in finder for file encrypt.so and copy it into the PokemonGo-Bot folder

start bot, should work

##Here's a more optimized command
cd into PokemonGo-Bot folder
cd PokemonGo-Bot
This command is optional if wget is already installed:
brew install wget
Then run
wget http://pgoapi.com/pgoencrypt.tar.gz && tar -xf pgoencrypt.tar.gz && cd pgoencrypt/src/ && make && mv libencrypt.so ../../encrypt.so
###Bot should work

how do i do this for windows?


Build failure due to "peer not authenticated"

I tried to build with gradlew following instructions in the README but failed here:
FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring project ':app'.
> Could not resolve all dependencies for configuration ':app:_developmentDebugCompile'.
   > Could not resolve com.github.PhilJay:MPAndroidChart:v2.1.3.
     Required by:
         gnucash-android:app:unspecified
      > Could not resolve com.github.PhilJay:MPAndroidChart:v2.1.3.
         > Could not get resource 'https://jitpack.io/com/github/PhilJay/MPAndroidChart/v2.1.3/MPAndroidChart-v2.1.3.pom'.
            > Could not GET 'https://jitpack.io/com/github/PhilJay/MPAndroidChart/v2.1.3/MPAndroidChart-v2.1.3.pom'.
               > peer not authenticated
   > Could not resolve com.google.android.gms:play-services-drive:7.0.0.
     Required by:
         gnucash-android:app:unspecified
      > Could not resolve com.google.android.gms:play-services-drive:7.0.0.
         > Could not get resource 'https://jitpack.io/com/google/android/gms/play-services-drive/7.0.0/play-services-drive-7.0.0.pom'.
            > Could not GET 'https://jitpack.io/com/google/android/gms/play-services-drive/7.0.0/play-services-drive-7.0.0.pom'.
               > peer not authenticated

By Google, I found a few quick suggestions:

Make sure you are not being blocked by a firewall or something. Try with sudo or something
Upgrade to the latest JDK

Must be the JDK, any chance to build the package with only Debian/Ubuntu packaged tools ?

I'm using Ubuntu's OpenJDK 8 and before version 7, so this shouldn't be a problem.

openjdk-8-jdk here, 8u91-b14-0ubuntu4~14.04
Building as root makes no difference.

@strk do you still experience build issues?

Yes, but different error now, so could be unrelated.
New error is (as of bc9159cf9f800fcd4c1ff895c6a7ab732ec16091):
FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring project ':app'.
> Could not resolve all dependencies for configuration
> ':app:_developmentDebugCompile'.
   > Could not find com.google.android.gms:play-services-drive:8.3.0.
     Searched in the following locations:
         http://dl.bintray.com/populov/maven/com/google/android/gms/play-services-drive/8.3.0/play-services-drive-8.3.0.pom
         http://dl.bintray.com/populov/maven/com/google/android/gms/play-services-drive/8.3.0/play-services-drive-8.3.0.jar
         https://maven.fabric.io/public/com/google/android/gms/play-services-drive/8.3.0/play-services-drive-8.3.0.pom
         https://maven.fabric.io/public/com/google/android/gms/play-services-drive/8.3.0/play-services-drive-8.3.0.jar
         https://jitpack.io/com/google/android/gms/play-services-drive/8.3.0/play-services-drive-8.3.0.pom
         https://jitpack.io/com/google/android/gms/play-services-drive/8.3.0/play-services-drive-8.3.0.jar
         http://JRAF.org/static/maven/2/com/google/android/gms/play-services-drive/8.3.0/play-services-drive-8.3.0.pom
         http://JRAF.org/static/maven/2/com/google/android/gms/play-services-drive/8.3.0/play-services-drive-8.3.0.jar
         https://jcenter.bintray.com/com/google/android/gms/play-services-drive/8.3.0/play-services-drive-8.3.0.pom
         https://jcenter.bintray.com/com/google/android/gms/play-services-drive/8.3.0/play-services-drive-8.3.0.jar
         https://repo1.maven.org/maven2/com/google/android/gms/play-services-drive/8.3.0/play-services-drive-8.3.0.pom
         https://repo1.maven.org/maven2/com/google/android/gms/play-services-drive/8.3.0/play-services-drive-8.3.0.jar
         file:/usr/src/gnucash/gnucash-android/app/libs/play-services-drive-8.3.0.jar
         file:/usr/src/gnucash/gnucash-android/app/libs/play-services-drive.jar
         file:/usr/src/android/android-sdk-linux/extras/android/m2repository/com/google/android/gms/play-services-drive/8.3.0/play-services-drive-8.3.0.pom
         file:/usr/src/android/android-sdk-linux/extras/android/m2repository/com/google/android/gms/play-services-drive/8.3.0/play-services-drive-8.3.0.jar
     Required by:
         gnucash-android:app:unspecified

Seems like you're missing the Google Repository component in the SDK?

@strk if this issue is now fixed, could we close this?

I don't have a build environment anymore so cannot verify.
Feel free to close.


How can I get 6 (lan, lot) coordinates of a hexagon of a cell grid?

I didn't find such a method in both CellGrid & ISEA3H classes, I believe there should be an opportunity to do this (e.g., for drawing/visualizing hexagons).

Hi!
There is actually a library called geogrid.js that seems to do what you want.  The geogrid library computes the centres of the grid cells and (you are right) not the coordinates of the six points defining the boundary.  However, the library geogrid.js can be used in the browser to estimate the coordinates of these six points and to visualize the result information.  If this should not solve your problem, feel welcome to explain your use case in more detail (maybe via e-mail).
Best regards


Render Custom Header example has an error

When selecting a month from the month dropdown menu, it results in a NaN error.
{months.map(option => (
                    <option key={option} value={option}>
                      {option}
                    </option>
                  ))}
</select>

This fixes the error:
{months.map((option, i) => (
                              <option key={option} value={i}>
                                {option}
                              </option>
                            ))}

https://github.com/Hacker0x01/react-datepicker/blob/master/docs-site/src/examples/render_custom_header.jsx

I add this code and it works but now it does not load the month of the actual date.
Did this happen to you?

Fixed the custom header issue in this PR:
https://github.com/Hacker0x01/react-datepicker/pull/1856
Waiting for it to be merged.

Fixed by #1856

I still have a problem. I just copy/ pasted the code from the example, but the app crashed when I open the datepicker. And it shows an error:
TypeError: Object(...) is not a function
...
value={months[getMonth(date)]}


I think the paragraph explaining the example is incorrect??

[Enter feedback here]
In this paragraph: https://docs.microsoft.com/en-us/sql/relational-databases/import-export/use-native-format-to-import-or-export-data-sql-server?view=sql-server-ver15#sample-non-xml-format-file
In the text it say the "c" qualifier is used, but it is using the "-n" option in the actual example. Did I miss something?

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

ID: 328e38b9-9bc3-b8d0-6c35-523adc294fa3
Version Independent ID: f8a555b0-d2ae-c8bc-842e-0b160afb4490
Content: Use native format to import & export data - SQL Server
Content Source: docs/relational-databases/import-export/use-native-format-to-import-or-export-data-sql-server.md
Product: sql
Technology: data-movement
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

@cosmoKenney -- Ken, thank you for your feedback.
@MashaMSFT -- Masha, please look into this conundrum.

Hi @cosmoKenney , this change has been processed and is now live.
Thanks for improving our documentation! Hope you have a wonderful rest of your day :)
Masha from the SQL Docs team


Update to streams 1.0

Added throws declaration on main methods, because otherwise sbt does not recognize them as main clases.
Merge after 1.0 artifacts are in repositories.

LGTM :-)


Changed currency number format

Locale string is more readable. Displays to 3dp. This should be sufficient. If not, the could be a condition that this is only for multiplier > 1.
NOT TESTED.

I agree that this is valuable, we probably have too many decimals nowadays


M3-3925: Make Firewall breadcrumb editable

Description
Edit Firewall's label in the breadcrumb of the detail view. This will be the only way to edit a Firewall's label.
Type of Change

Non breaking change ('update', 'change')

This works well!
The only minor point I would raise is that if you block the request as @Jskobos described to simulate a request failure, edit the Firewall name, and click the checkmark, it looks like it accepts it when it really doesn't (i.e., the new name looks like it goes through, but if you navigate away from the Firewalls tab and then back, the original name shows). Not sure if there's an easy visual to indicate that the change won't go through, though.


Process finished with exit code 1 in watch mode

Process finished with exit code 1 in watch mode when we  when we make a mistake in the script or viewScript
build started...
✘ [ERROR] Expected ";" but found "new"

    src/viewScript.js:5:27:
      5 │     const accordion = test new;
        │                            ~~~
        ╵                            ;

/Users/front-it/www/gutenbergblocks/wp-content/plugins/gutenberg-blocks/vite-plugin-gutenberg-blocks/node_modules/esbuild/lib/main.js:1649
  let error = new Error(`${text}${summary}`);
              ^

Error: Build failed with 1 error:
src/viewScript.js:5:27: ERROR: Expected ";" but found "new"
    at failureErrorWithLog (/Users/front-it/www/gutenbergblocks/wp-content/plugins/gutenberg-blocks/vite-plugin-gutenberg-blocks/node_modules/esbuild/lib/main.js:1649:15)
    at /Users/front-it/www/gutenbergblocks/wp-content/plugins/gutenberg-blocks/vite-plugin-gutenberg-blocks/node_modules/esbuild/lib/main.js:1058:25
    at runOnEndCallbacks (/Users/front-it/www/gutenbergblocks/wp-content/plugins/gutenberg-blocks/vite-plugin-gutenberg-blocks/node_modules/esbuild/lib/main.js:1484:45)
    at buildResponseToResult (/Users/front-it/www/gutenbergblocks/wp-content/plugins/gutenberg-blocks/vite-plugin-gutenberg-blocks/node_modules/esbuild/lib/main.js:1056:7)
    at /Users/front-it/www/gutenbergblocks/wp-content/plugins/gutenberg-blocks/vite-plugin-gutenberg-blocks/node_modules/esbuild/lib/main.js:1085:16
    at responseCallbacks.<computed> (/Users/front-it/www/gutenbergblocks/wp-content/plugins/gutenberg-blocks/vite-plugin-gutenberg-blocks/node_modules/esbuild/lib/main.js:703:9)
    at handleIncomingPacket (/Users/front-it/www/gutenbergblocks/wp-content/plugins/gutenberg-blocks/vite-plugin-gutenberg-blocks/node_modules/esbuild/lib/main.js:762:9)
    at Socket.readFromStdout (/Users/front-it/www/gutenbergblocks/wp-content/plugins/gutenberg-blocks/vite-plugin-gutenberg-blocks/node_modules/esbuild/lib/main.js:679:7)
    at Socket.emit (node:events:390:28)
    at addChunk (node:internal/streams/readable:315:12) {
  errors: [
    {
      detail: undefined,
      id: '',
      location: {
        column: 27,
        file: 'src/viewScript.js',
        length: 3,
        line: 5,
        lineText: '    const accordion = test new;',
        namespace: '',
        suggestion: ';'
      },
      notes: [],
      pluginName: '',
      text: 'Expected ";" but found "new"'
    }
  ],
  warnings: []
}

Process finished with exit code 1

Thanks for the feedback.
In recent versions, the watch script will no longer exit when encountering a syntax error.


7.1.1 not published to galaxy

SUMMARY
Version 7.1.1 is not published to galaxy
ISSUE TYPE

Bug Report

sadly same issue with the v7.2.0 version

All missed versions were publish into ansible-galaxy


CheckTx/DeliverTx responses

CheckTx data is empty and DeliverTx data is the TxID. They should both just be empty now that tendermint returns the tx hash.
Note the TxID from basecoin is the hash of the sign bytes (ie. without the signature) while the tendermint txid is the hash of the whole tx

Is this true for abci in general or just basecoin?
Does it make sense to return other values in delivertx?
For example, I had a CreateEscrow tx (as plugin) that generated a new escrow and auto-generated an address for it, which was returned in DeliverTx (the DeliverTx.Data was the key to look up the escrow or use it).
Was that a bad design? (I am used to the server generating IDs more than the clients)

If you are only talking about SendTx, then I am fine.  If it is a general change, then it needs more discussion.

For 0.7, I will clear up the DeliverTx return data to be something useful or empty, no random hex digits.


Podman fails to parse port ranges in a Dockerfile

It seems some podman versions fail to parse the Dockerfile if an EXPOSE is set to a port range. These are just hints anyway, so disable the port range.

Also added the ability to specify the rpc port range that samba will use (these were the ports in question). I noticed the default range is also used by sshd, which may prevent the container from starting. The problem with these ports is samba uses them as needed (and just picks ones which are available). They aren't available to the container unless we reserve them, but if we reserve them all, then the container may not start. So instead we can specify a smaller range of ports, and explicitly reserve just those.


add response builder tests

This PR adds more tests in oder to cover response builder functions.
Closes: #11

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


Update _index.md

👷 Deploy Preview for gladia processing.

Name
Link

🔨 Latest commit
b4755ba168b30402a901f5b1481fa0cee12ea128

🔍 Latest deploy log
https://app.netlify.com/sites/gladia/deploys/66c5e2373c1cbc00094218c7


Unstable compaction test; failed due to timeout

Jenkins failure, https://builds.apache.org/job/CouchDB/job/master/222/
  Compact after lowering attachment size limit
    couchdb_attachments_tests:544: should_compact_after_lowering_attachment_size_limit...*failed*
in function couchdb_attachments_tests:wait_compaction/3 (test/couchdb_attachments_tests.erl, line 586)
in call from couchdb_attachments_tests:compact_db/1 (test/couchdb_attachments_tests.erl, line 573)
in call from couchdb_attachments_tests:'-should_compact_after_lowering_attachment_size_limit/1-fun-1-'/1 (test/couchdb_attachments_tests.erl, line 550)
**error:{assertion_failed,[{module,couchdb_attachments_tests},
                   {line,573},
                   {reason,"Timeout waiting for database database compaction"}]}
  output:<<"">>

Also, "database database compaction"? Heh.

For reference apache/couchdb#1232

FYI This is happening a lot recently in Travis and Jenkins. It'll be nice to fix this one up.

Closing in favour of #1442


Make istiod respect nodeSelector

Fix https://github.com/istio/istio/issues/36110

@kebe7jun: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
92ffc8c85c41ada548e1625e058f36d072444ad0
link
true
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/cherry-pick release-1.11

@jacob-delgado: once the present PR merges, I will cherry-pick it on top of release-1.11 in a new PR and assign it to you.

In response to this:

/cherry-pick release-1.11

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/cherry-pick release-1.12

@jacob-delgado: once the present PR merges, I will cherry-pick it on top of release-1.12 in a new PR and assign it to you.

In response to this:

/cherry-pick release-1.12

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@istio/wg-environments-maintainers PTAL.

@jacob-delgado: new pull request created: #36226

In response to this:

/cherry-pick release-1.11

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@jacob-delgado: new pull request created: #36227

In response to this:

/cherry-pick release-1.12

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Use ids rather than id for a name

id clashes with the id from Predule so it gives warnings when you compile this code.

I had missed that one. :+1: Thanks.


Comments: Create a Signup Page

Link to chapter - http://serverless-stack.com/chapters/create-a-signup-page.html

This issue was moved to https://discourse.serverless-stack.com/t/comments-create-a-signup-page/65


Remove caching ability

Hey guys,
With this particular container, is there any way to remove the caching ability? I have a cache already on pihole, so dual cache seems a bit redundant.
I don't intend to have this container external, just for my internal network, would this be suitable or would another container (e.g. gists/dnscrypt-proxy) be more suitable?
Thanks!

unbound needs a bit of cache to work, as it is a recursive resolver, configured to gets its information starting from root servers.
The cache in clients is not redundant. Clients caches usually only see and store the final response, not intermediary ones.
If this is just for your internal network, you'd better use dnscrypt-proxy.


Update reference solution endpoint to include flags whether current user can delete or evaluate

Coverage increased (+0.1%) to 73.706% when pulling 47f596f17cd60afd0e1d3e914d9ba407c917f00a on exercise-rights-tuning into bb4a968e56bce2226c695602b95ccb1adb1180b8 on master.

Not sure if we really settled on this approach... It somehow smells... Personally I would engage new permissions endpoint as soon as possible, including this.

I agree, adding a general permissions endpoint is really simple and cleaner than reimplementing this for every entity.

Well, since we have no permissions endpoint yet, and this needs to be done soon, the proposed solution was the effective and efficient way, how to implement it. And it is much better than re-implementing ACL verifications at client side.
Furthermore, nice implementation of the security endpoint will require much more work. I do not see the "really simple". Straightforward maybe, but it will take some doing.

An API endpoint that takes an array of structures that map 1:1 to ACL calls is really not complicated. However, I agree that using this efficiently from the web-app might be difficult.

But we do not want an API that resolves ACL calls. That would require to reimplement business logic at client side. You need also a mapping between which ACL functions call which endpoints to find out, whether API can call DELETE /exercise/, for instance.
I have already enlisted a topic for tomorrows discussion regarding the endpoint. But before we have it, providing the canXXX flags in the response for each object is the best way to go in my opinion.

Well, we already have endpoints such as /assignments/<id>/can-submit that serve a similar purpose. Adding another way of checking permissions doesn't seem wise to me. I agree that in the web app, you should ideally only know the endpoints that you need to call and adding some privilege identifiers is not optimal.
For now, I propose to add a simple endpoint that checks if the user can update/delete a ref. solution and be done with it. Later, we can try to come up with a more general solution.

Well, if this should be temporary solution, why not to include it in web application directly as for every other role checks? If we do not know the final solution for this it seems logical...

Hints were moved into a separate decorator. However, it is still a temporary solution. I will rewrite the decorator to use security endpoint (once it is implemented). Furthermore, the flags are also temporary as we need to analyze, which permission hints are likely to be needed.

Coverage increased (+0.2%) to 73.692% when pulling 22b515e72fae756d4e176e565699cfa5ab9dfd03 on exercise-rights-tuning into 05212e5938dcc999b1df0cdf3c2691bdc2fdec37 on master.

Coverage increased (+0.2%) to 73.692% when pulling 026473dd8bbc87e5a733a12472b8b7122bb1447e on exercise-rights-tuning into 05212e5938dcc999b1df0cdf3c2691bdc2fdec37 on master.

Coverage decreased (-0.08%) to 73.456% when pulling 59d1db4be596a9319d4993715527d4be462ff136 on exercise-rights-tuning into 05212e5938dcc999b1df0cdf3c2691bdc2fdec37 on master.

Coverage decreased (-0.09%) to 73.448% when pulling ae931be739ee54d60c649d9d5e101efeb6ab79f2 on exercise-rights-tuning into 05212e5938dcc999b1df0cdf3c2691bdc2fdec37 on master.


Refine the hash search widgets/section

A "Reset" button is needed
Label the 'Output' boxes so the student knows what they're reading about
The search advice box needs to blank before new information is printed.
Steps should be provided to explain what is needed, and in what order.

Lets consider this widget-only refinement, and any search-based refinement will be for other issues.

Numbers '1' and '4' fixed in commit 2d6c604.
2 and 3 weren't really relevant any more.


tsm1: Replace goroutine merge with k-way merge

Previously replaced WalkKeys implementation for a considerable
improvement to startup time

During a run of megacheck the following issues were discovered:
/tmp/132448102/src/github.com/influxdata/influxdb/tsdb/index/inmem/inmem_test.go:127:29: func (*seriesFileWrapper).reopen is unused (U1000)


Ability for shards to use a custom material

I saw something like this mentioned in the commits but I don't see it available in the plugin, basically just a material slot which if found not null at time of shard generation will be used on all of the shards.

This can be achieved by overriding the material in the shards scene. Is that an option for you?

This can be achieved by overriding the material in the shards scene. Is that an option for you?

Assuming you mean just manually editing the shard GLBS material; I mean that's a basic Godot feature, but that's kinda a pain compared to just having an exported attribute as part of your destructibles plugin haha. Obviously up to you though, I just think that's a worthwhile QOL update.

Just to make sure that we are talking about the same thing: You can open the import dialog for the shard scene, go to the materials tab and set the "use external" to your material. Still, I can see this being inconvenient if you have to do it for a lot of scenes.

Just to make sure that we are talking about the same thing: You can open the import dialog for the shard scene, go to the materials tab and set the "use external" to your material. Still, I can see this being inconvenient if you have to do it for a lot of scenes.

Yup, that's I'm referring too. By allowing this too you could update materials for specific items or scenes instead of remaking the shard scene for multiple colors and whatnot.


switch to yaml?

 Plus it's picky about quotes and commas and things.
Maybe YAML is a better option?

:beers: if we switch to yaml. I am more opinionated about preferring it these days.

Would we want to switch for yaml for all the things? Including the steve.ini file?

Also, @codersquid and I talked on IRC about this. Instead of switching wholesale to yaml, we want to add support for yaml and additionally change the default to do things in yaml rather than json. We need to support json so that we don't have to go redo all the things we've already done.

YAML is supposed to be a superset of JSON. So start with the current routines that read in JSON and replace those, making sure the resulting data structures are compatible through testing.
There are some caveats with 4 byte unicode encoded files (but you are unlikely to have those), with YAML recognising some more types of data ( "No" for False, and "00:10:02" for 602 ). Also PyYAML has some problems with properly recognising corner cases of floats (I never encountered these myself, but found an item on stackoverflow by someone that ran into them), I fixed those in my ruamel.YAML (I started that because PyYAML is stuck on YAML 1.1 and not 1.2 and development seems stalled).
So start with replacing normal JSON usage, load them with the ``jaml.load(fp, Loader=yaml.SafeLoader) option that should be relatively painless. It is relatively easy to support both .ini and YAML files for reading. I do so in my AppConfig but that is based on using ConfigObj and configparser/ConfigParser in the standard lib is a little different.
You can continue to have the application write JSON files, you should be able to read those back in. Direct advantage of changing to YAML is that:
{
"a": 1,
"b": 2,
}
becomes valid input. JSON chokes on the trailing , and I have had that multiple times editing out some stuff, including the last dict item from JSON files. And that you can use # end of line comments to clarify/document your files.
At a later stage you could consider making the writing out YAML an configuration/commandline option. If those files are for human perusal, you could consider basing them on the RoundTripParser in ruamel.yaml, which allows you to include explanatory comments in the output.

I created some routines to sync a directory containing JSON files with a directory containing YAML (with the same basename - extension). You provide a last-synced timestamp to prevent syncing when a file on both sides has been updated.
For that I would like to:

convert steve.ini to steve.yaml so I can write the last-synced timestamp somewhere without losing comment information
add a command to steve-cmd that syncs the json subdir with a new yaml subdir

After that the following workflow would be possble

create project
fetch json
sync json and yaml
repeat

update .json or .yaml files (but not the same file on both sides)
sync json and yaml

upload json files to richard.

Once that works I could have the webeditor and all the other commands work with the YAML files, further reducing JSON to the format that the YAML is converted to just before uploading.


NOT Working

Hi, is this still working? Nothing happen for me after installation.

I'm pretty sure this project was abandoned a while ago :/

I recommend GooseMod if you're still interested in anything here, and maybe use Discord PTB for the account switcher functionality since that's going being added soon to Discord

Hi there, yes the project is currently dead because I had no time to work on it, I do plan on giving the torch to someone else so they can continue with it.
Currently, I'm a student and also have a job so I don't have much free time, i've worked on it a bit from time to time tho, and so far I got it to work again and also was able to successfully turn it into a great extension where you can enable and disable functionalities!

Hi there, yes the project is currently dead because I had no time to work on it, I do plan on giving the torch to someone else so they can continue with it.
Currently, I'm a student and also have a job so I don't have much free time, i've worked on it a bit from time to time tho, and so far I got it to work again and also was able to successfully turn it into a great extension where you can enable and disable functionalities!

Hi there, yes the project is currently dead because I had no time to work on it, I do plan on giving the torch to someone else so they can continue with it.
Currently, I'm a student and also have a job so I don't have much free time, i've worked on it a bit from time to time tho, and so far I got it to work again and also was able to successfully turn it into a great extension where you can enable and disable functionalities!

Hi I feel sorry that I can't help. However, I'am looking forward to the extension!!!

DiscordPlus+ is now back!

DiscordPlus+ is now back!


Advent of code ends before December ends

Description

Taken on the 26th

Steps to Reproduce

Look at the bot after AOC
Expected Behaviour

normal status or something
Actual Behaviour

countdown since aoc of the day that didn't happen
Known Impacted Platforms

[x] Web
[x] Desktop
[x] Android App
[x] iOS App

Possible Solutions

More complex date checkers that can actually lock features to specific times of months, actually wouldn't be that hard to implement. @in_timeframe(start=..., end=...)
Or just have a checker in the event itself to check if its 1 to 25 or 26 to 31.
Would you like to implement a fix?
Note: For high-priority or critical bugs, fixes may be implemented by staff.

[x] I'd like to implement the bug fix
[ ] Anyone can implement the bug fix

Looking at the current code it does seem to currently try and end at the right time https://github.com/python-discord/sir-lancebot/blob/532a8d53fca6c06a90d9bcb4862a36062750d359/bot/exts/events/advent_of_code/_helpers.py#L536-L539
Is the issue just that it isn't reset to what it originally was after this while loop?

Is the issue just that it isn't reset to what it originally was after this while loop?

I think so, as someone restarted the deployment and the status reset.

Alright, that should be fairly simple to fix, just adding a line after the while loop to reset it. Would you like me to assign you?
Just need to think of a way of avoiding hardcoding the activity text across the two files, could have something like a get_default_activity function in bot/bot.py, or just put the text as a constant in constants.py, not sure.


hup

Sorry, didn't intend to open this on the main repo, trying to get something working in a fork


How to set AllowAdditionalProperties to specified model

Hello, i have a small question about how to set an AllowAdditionalProperties to some model, as an examble i have a model1 and model2
in model1 i will have something like "public list models2 {get; set;}"
how to set the AllowAdditionalProperties just to the model2 without model1?

This is only possible if model1 and model2 are different schemas so that you can set this for one schema...


[Feature Request] 快捷键打开菜单

Checklist

[X] I searched exising issues and no one else requests similar feature.
[X] I think that 25%+ users are positive towards this feature.

Describe the feature
todo
Additional context
No response

https://blog.csdn.net/qq_43750882/article/details/110143664

https://blog.csdn.net/qq_43750882/article/details/110144285

logConsole.setAccelerator(KeyCombination.valueOf("SHORTCUT+o"));

KeyCombination ctrl_v = new KeyCodeCombination(KeyCode.V, KeyCombination.CONTROL_DOWN);

KeyCombination shift_c = new KeyCodeCombination(KeyCode.C, KeyCombination.SHIFT_DOWN);

KeyCombination alt_v = new KeyCodeCombination(KeyCode.V, KeyCombination.ALT_DOWN);

KeyEvent.java中 isShortcutDown()、isControlDown()、isAltDown()、isShiftDown()、isMetaDown()

KeyCombination ctrl_alt_c = new KeyCodeCombination(KeyCode.C, KeyCombination.ALT_DOWN, KeyCombination.CONTROL_DOWN);

https://openjfx.cn/javadoc/16/javafx.graphics/javafx/scene/input/KeyCombination.html
https://link.csdn.net/?target=https%3A%2F%2Fopenjfx.cn%2Fjavadoc%2F16%2Fjavafx.graphics%2Fjavafx%2Fscene%2Finput%2FKeyCode.html


Database for change_history exists

This is not an issue or a bug, but rather a suggestion to think about.
I don't want to give my deployer user any grants outside of the database that it is supposed to work with, meaning also no rights to create a database. But snowchange script executes a "CREATE DATABASE IF NOT EXISTS" for creating the CHANGE_HISTORY table, which then is followed by error. In my view, we don't want a new db for change_history in any case, so I would rather get an error, when the defined db doesn't exist.
But I understand that other users might find it an useful feature, so my suggestion would be to change the script so, that db creation right is not needed by default.
So far I can easily leave out the part of the code where this is tried in my local env.

Hey @nugic, please check out issue #15 which is the same. I'll close this as a duplicate.

Hey @nugic, please check out issue #15 which is the same. I'll close this as a duplicate.


How to get docker-slim work inside docker container?

On my mac book
>system_profiler SPSoftwareDataType
Software:

    System Software Overview:

      System Version: macOS 10.14.5
      Kernel Version: Darwin 18.6.0

# Then run docker
>docker run -it -v /var/run/docker.sock:/var/run/docker.sock docker:18.09.6 ash
/ #
/ # cat /etc/*release
3.9.4
NAME="Alpine Linux"
ID=alpine
VERSION_ID=3.9.4
PRETTY_NAME="Alpine Linux v3.9"
HOME_URL="https://alpinelinux.org/"
BUG_REPORT_URL="https://bugs.alpinelinux.org/"
/ #
/ # cd /usr/local/bin
/usr/local/bin #
/usr/local/bin # wget https://downloads.dockerslim.com/releases/1.25.0/dist_mac.zip
Connecting to downloads.dockerslim.com (<IP_ADDRESS>:443)
dist_mac.zip         100% |*******************************| 4835k  0:00:00 ETA
/usr/local/bin #
/usr/local/bin # unzip -x dist_mac.zip
Archive:  dist_mac.zip
   creating: dist_mac/
  inflating: dist_mac/docker-slim
  inflating: dist_mac/docker-slim-sensor
/usr/local/bin #
/usr/local/bin # ls -l
total 174244
-rwxr-xr-x    1 1000     1000      28007480 May  4 02:42 containerd
-rwxr-xr-x    1 1000     1000       4964704 May  4 02:42 containerd-shim
-rwxr-xr-x    1 1000     1000      15686552 May  4 02:42 ctr
-rwxr-xr-x    1 1000     1000      50693656 May  4 02:42 docker
-rwxrwxr-x    1 root     root           933 May 11 00:23 docker-entrypoint.sh
-rwxr-xr-x    1 1000     1000        764144 May  4 02:42 docker-init
-rwxr-xr-x    1 1000     1000       2837280 May  4 02:42 docker-proxy
-rwxr-xr-x    1 root     root      10170712 Jun 11 01:33 docker-slim
-rwxr-xr-x    1 root     root       3367744 Jun 11 01:33 docker-slim-sensor
-rwxr-xr-x    1 1000     1000      54339296 May  4 02:42 dockerd
-rwxrwxr-x    1 root     root           587 May 11 00:23 modprobe
-rwxr-xr-x    1 1000     1000       7565160 May  4 02:42 runc
/usr/local/bin #
/usr/local/bin # docker images myip
REPOSITORY          TAG                 IMAGE ID            CREATED             SIZE
myip                1.0                 c76a0f8b3880        6 days ago          7.08MB
/usr/local/bin #
/usr/local/bin # docker-slim build --http-probe myip:1.0
/usr/local/bin/docker-slim: line 1: ����: not found
/usr/local/bin/docker-slim: line 1: [�: not found
/usr/local/bin/docker-slim: line 2: can't create [__gosymtab__TEXT@M[@M[__gopclntab__TEXT@M[�#/@M[�__DATA��ؿ����__nl_symbol_ptr__DATA����#__noptrdata__DATA: nonexistent directory
/usr/local/bin/docker-slim: line 2: hH__PAGEZEROx__TEXT����__text__TEXTl_@�__rodata__TEXT�o@Ȑ�o@__symbol_stub1__TEXT�__typelink__TEXT@[�=@[__itablink__TEXT�: not found
/usr/local/bin/docker-slim: line 3: : not found
/usr/local/bin/docker-slim: line 4: �: not found
/usr/local/bin/docker-slim: line 3: syntax error: unexpected end of file (expecting ")")
/usr/local/bin #

### How to get docker-slim work inside docker container?

@leo18945 running docker-slim in a container will be supported in the next release actually. Right now docker-slim needs to be able to mount a local directory to add its sensor to the test container it creates and to get the dynamic analysis artifacts back from that container. This doesn't work as-is in the current version. The next release will address this limitation by adding two additional modes to exchange data (with volumes or using copy commands).

Ok, thanks. I am waiting for the next release.


Add chat

depends on this issue.

Closing because we won't tackle this in the foreseeable future.


Detecting region ID

@jackah2 halp! If you have a minute I'm sure this is a super easy fix, trying to merge in the work Sam did and get this thing to compile. I tried adding -api on the end of spigot and experimental (instead of R0.1) in pom.xml but have no idea what's going on with the configuration remapping bit. Building attempts give errors probably related to missing libraries that it can't get because of the broken dependencies.

from @samhum "that part of the Pom is to unobfuscate the spigot functions to get nbt tags. Try running the buildtools jar and there is some additional instructions if that doesn't work: https://www.spigotmc.org/wiki/buildtools/"
Did this, plopped it in the directory but same result:
[ERROR] Failed to execute goal on project ContainerTracker: Could not resolve dependencies for project com.emicb:ContainerTracker:jar:1.3.1: The following artifacts could not be resolved: org.spigotmc:spigot:jar:remapped-mojang:1.20.4-R0.1-SNAPSHOT (absent): org.spigotmc:spigot:jar:remapped-mojang:1.20.4-R0.1-SNAPSHOT was not found in https://hub.spigotmc.org/nexus/content/repositories/snapshots/ during a previous attempt. This failure was cached in the local repository and resolution is not reattempted until the update interval of spigotmc-repo has elapsed or updates are forced -> [Help 1]
Tried renaming the buildtools output jar to 1.20.4-R0.1-SNAPSHOT but no luck, I'm guessing I need to tell it how to integrate this locally generated copy somehow?

Regular build attempt gives "import org.apache.commons.lang.StringUtils" as missing, out of time for now


Make link to report clickable in terminal

Closes #4455

Hi,
I'm afraid the problem is way more complicated than what you've tried (I've already tried what you've done and gave up for now).
The issue here is that there isn't A terminal.
What you've done works on YOUR terminal.
It doesn't work on others and worse, breaks what currently works: clickage on iTerm2 and can be copy-pasted into a browser.
We would need a matrix with the before and after result:

on various Linux terminals, from zsh to the default one in the distro
on MacOS: default and iTerm2
on Windows
in IntelliJ's embedded terminal

I don't know if JLine could help with this.


[NAV][ODataV4] Cannot Delete/Patch: Error in query syntax

Hello i have been trying to debug this for the past 3 business days and i havent been able to figure out what is wrong.
I call delete and i get Error in query syntax as soon as i pass some parameters.
While looking at the metadata i get the following keys:
string Item_No
string Sales_Type
string Sales_Code
date Starting_Date
string Currency_Code
string Variant_Code
string Unit_of_Measure_Code
decimal Minimum_Quantity

So i call it like this:
delete https://nav.domain.com:7048/NAV/ODataV4/Company('TESTJL')/SalesPrices('C-MONITORING','Customer','752','0001-01-01','','','UN',0)
And I also tried like that:
delete https://nav.domain.com:7048/NAV/ODataV4/Company('TESTJL')/SalesPrices(Item_No='CONSUL%20SENIOR',Sales_Type='Customer',Sales_Code='1537',Starting_Date='',Currency_Code='',Variant_Code='',Unit_of_Measure_Code='UN',Minimum_Quantity=0)
But i always get Bad Request - Error in query syntax.
We are using Dynamics NAV 2018 v. 11.0.20783.0
PS: SalesPrices is a Page of Object ID 7002 with "All Tenants" checked.

Sorry, this forum is for extensibility requests for V16/V17, not for reporting bugs or requesting hotfixes. Please create request via technical support to get help.

Thank you for reaching out. Your request could be reporting a product bug not related to the AL extensibility.
Please reach out to the relevant support channels as specified in Product Support
On premises - https://support.microsoft.com/en-us/supportforbusiness/productselection?sapId=93d37907-ad94-d591-22e9-593cfa09dd3f
On line – read more here https://docs.microsoft.com/en-ca/dynamics365/business-central/dev-itpro/administration/manage-technical-support or contact your partner
If you are a partner open a support incident through Partner Center - https://partner.microsoft.com/en-us/pcv/dashboard/overview


don't print env variable values in the logs (some are sensitive)

Happy new year friends!!! 👏🏼 🍾 😄
While testing out v1.9.0, I realized I made a mistake in https://github.com/weaveworks/kured/pull/464 PR that prints in the log the value of the Environment Variables that are being bound to command flags, since some of these are sensitive (like KURED_NOTIFY_URL) we don't want their values to end up in the logs.
This PR addresses it, I think this is relatively important to merge in and perhaps worth cutting a v1.9.1 since currently it leaks sensitive details to the logs.
apologies for not catching this sooner.
/cc @ckotzbauer @evrardjp @jackfrancis (just tagging you folks for awareness as you've had the chance to review the original PR)

Thanks @ckotzbauer & @jackfrancis for taking a look and approving 🙏🏼  do you think we can go ahead and merge this?


Upgrade jwt auth

Description

Upgrade jwt auth to v2.0.0
Related Issue

Motivation and Context

solves underlying dependency vulnerability
How Has This Been Tested?

manual extension upload using npx<EMAIL_ADDRESS>Screenshots (if appropriate):
Types of changes

[ ] 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)
[x] Dependency update

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Closes #55


How do you install this?

How do I actually install and use this library. I've downloaded it and ran CMAKE but it gives me an error about not knowing where Qt is. When I give it the directory it still complains about a missing cmake file. Is there an example of this file or should it already be in one of the sub directories.
Should the Qt Directory be the root of Qt or the root of the specific version that I want to use (inside the Qt dir)?
Sorry for the dumb issue but we all start somewhere... :)

Point cmake to a path similar to C:/Qt/5.9/msvc2015_64
Depending on your compiler.

How do I actually install and use this library. I've downloaded it and ran
CMAKE but it gives me an error about not knowing where Qt is. When I give
it the directory it still complains about a missing cmake file. Is there an
example of this file or should it already be in one of the sub directories.
Should the Qt Directory be the root of Qt or the root of the specific
version that I want to use (inside the Qt dir)?
Sorry for the dumb issue but we all start somewhere... :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/paceholder/nodeeditor/issues/123, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGxqOt66Z22Z3cHfcxr00yOUCTdql__Cks5steh4gaJpZM4P9mS9
.

Which OS are you using?
On linux I usually do this:
export CMAKE_PREFIX_PATH=/your/qt/installation/path/qt/5.9.1:$CMAKE_PREFIX_PATH
git clone _address_of_this_git_repository_
cd nodeeditor
mkdir build && cd build
cmake .. && make

The first line changes the environment variable that points CMake where to look for the libraries.

Did it work for you ?

Sorry not had a chance to experiment with it. I think that russelltg is on the right lines, I'll update this when I get a chance to try it again (probably saturday)

No hurry, just wanted to make sure that it works for you.

No I still can't get it to install. As far as I know I have all the settings right but I am useless when it comes to CMAKE...
https://imgur.com/a/wJnjt

The variable CMAKE_INSTALL_PREFIX should point to the Qt's installation directory.
All the rest Qt5xxx_DIR things are then not necessary.

Alternatively, you could define all necessary Qt5xxx_DIR variables.

![image](https://user-images.githubusercontent.com/322952/31853623-da4adb38-b68b-11e7-94e5-bd0475fb222f.png)

In my case they point to qt/installation/path/cmake/Qt5

Ok, so i got it building the path for me ended up being D:/Qt/5.8/msvc2015_64/lib/cmake/Qt5.
With the sln I get 3594 warnings (using Visual Studio 2015 Enterprise), is that expected? By default I also got 6 build errors to do with LNK1112 which I fixed with this, after that I got 7 errors for MSB8020 which has todo with CMAKE setting the wrong build target for visual studio. That's basically where I am now. Visual studio can't build the project because it thinks it needs to use a different compiler, I've gone through the settings and looked but can't  find anything wrong. Platform Toolset is set to v140 which is correct and other settings are correct as well.

The project is being build with MSVC, here is the continuous integration config: https://github.com/paceholder/nodeeditor/blob/master/.appveyor.yml
To make it work I needed to spend time on wrong target settings as well.
I can't give any decent hints as I use Linux + command line tools.
it seems that CMake and Visual Studio is a crappy combination.
Update

I ran the build process again and now I'm getting the original LNK1112 error again.   I would probably switch to CLian but it's not quite there yet.

it seems that CMake and Visual Studio is a crappy combination.

CMAKE and Windows is a bad combination (in my experience), it would be better if you could just use apt-get for all the libraries like you can on Linux. :(

can't even uninstall the old version of VS

If you need to make any changes to VS it takes ages. Installing the newest versions of VS takes at least 40 min, usually 2-3 hours with a few packages selected...

Take a look at QtCreator, I used it a couple of years ago. Since then it got just better. It eats CMakeLists.txt as a project file.

I had a look at it when I first installed Qt (a few months ago), It looked quite limited as far as debugging and multi project solutions go. There as a VS plugin for the Qt gui creation tool so i've been using that with VS for actual coding.

@crener

It looked quite limited as far as debugging and multi project solutions go.

care to say what seems to be missing for you ? in my experience QtCreator >>> VS

The question was how can I get this to compile under Visual Studio, let's keep that topic because it is permanent and nodeeditor can compile under VS (I've done it plenty of times).
To me, it seems Qt5_Dir should be set to D:/Qt/5.8/msvc2015_32 (can't remember if its that or msvc2015) if you're using the 32 bit generators for visual studio, which it seeems like you are. If you only have 64-bit Qt installed, then you'll have to use the 64 bit generators for CMake. Delete your current build directory (or CMakeCache.txt) and then re-run cmake and when you select the generator, select Visual Studio 14 2015 Win64. Then pass the D:/Qt/5.8/msvc2015_64 directory as Qt5_Dir.
That works for me, tell me if you have any issues :).

Hi, I am trying to build this on debian linux (9.2/stretch).
After apt-get install qtbase5-dev I did like you proposed:
mkdir build && cd build
export CMAKE_PREFIX_PATH=/usr/include/x86_64-linux-gnu/qt5
cmake .. && make

I get the following error:

CMake Error at /usr/lib/x86_64-linux-gnu/cmake/Qt5Core/Qt5CoreMacros.cmake:63 (file):
file attempted to create a directory:
/home/user/workspace/nodeeditor/resources into a source directory.
Call Stack (most recent call first):
/usr/lib/x86_64-linux-gnu/cmake/Qt5Core/Qt5CoreMacros.cmake:211 (qt5_make_output_file)
/usr/lib/x86_64-linux-gnu/cmake/Qt5Core/Qt5CoreMacros.cmake:282 (_QT5_PARSE_QRC_FILE)
CMakeLists.txt:49 (qt5_add_resources)
-- Configuring incomplete, errors occurred!
See also "/home/user/workspace/nodeeditor/CMakeFiles/CMakeOutput.log".

Do I need more than qtbase5-dev? Do I point it to the right directory?

Hi @phzs,
I guess when you install it on Debian with apt-get the Qt libs end up in standard linux library paths, i.e. /usr/lib.
Thus, there is no need to define CMAKE_PREFIX_PATH. CMake should check the standard paths without extra instructions.
I googled the same issue as in your listing but did not find any decent solution.
Which version of Qt are you using?
Thanks
Dmitry

Hi @paceholder,
thanks for the quick reply :)
You are right, it compiled cleanly after unsetting the variable (and deleting CMakeCache.txt). Many thanks!


fix: add description to __schema introspection result

Newer Graphiql versions' introspection queries include description to the __schema as the following:
query IntrospectionQuery {
  __schema {
    description <------ required
    queryType {
      name
    }
    mutationType {
      name
    }
    subscriptionType {
      name
    }
    types {
      ...FullType
    }
    directives {
      ...
    }
  }
}
...

Introspection queries through newer versions of graphiql result in:
“Unknown field \“description\” on type \“__Schema\“. Did you mean \“subscriptionType\“?”

Official implementation reference:
https://github.com/graphql/graphql-js/blob/main/src/type/introspection.ts

thanks 🙂


[eslint-plugin-lit-a11y] Remove dependency on intl-list-format

Expected behavior
Installing eslint-plugin-lit-a11y should use a small disk footprint
Actual Behavior
eslint-plugin-lit-a11y depends on intl-list-format which is 9.1MB on disk, currently the largest dependency in all node_modules of Chrome DevTools. It appears that it was only used for Node 10 compatibility (https://github.com/open-wc/open-wc/blob/4f55bf52fde479d9b2e2ed61cea78b29bd97082a/packages/eslint-plugin-lit-a11y/lib/rules/role-has-required-aria-attrs.js#L15)
Since Node 10 is EOL, can this large dependency be removed?
Additional context
Happens on the latest version (1.0.1).

@thepassle can we reopen this issue please?

For sure. Sorry this is taking so long, we're in prerelease mode for open-wc because we're waiting for lit to release the new version, but its taken a bit longer than anticipated

Thanks for fixing this! It removed about 60k lines from our node_modules :tada: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/3268293/


[WIP] Add additional webhookrhobsreceiver for fleet mode

What type of PR is this?
feature
What this PR does / why we need it?
This PR add additional webhookrhobsreceiver for handling alerts on RHOBS cluster to send SL for actionable alerts.
Which Jira/Github issue(s) this PR fixes?
Fixes # OSD-15393
Pre-checks (if applicable):

[ ] Tested latest changes against a cluster
[ ] Ran make generate command locally to validate code changes
[ ] Included documentation changes with PR

Codecov Report

Merging #50 (2ae615e) into master (367f21b) will decrease coverage by 7.28%.
The diff coverage is 12.28%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master      #50      +/-   ##
==========================================
- Coverage   38.94%   31.67%   -7.28%
==========================================
  Files          14       16       +2
  Lines         701      862     +161
==========================================
  Hits          273      273
- Misses        412      573     +161
  Partials       16       16

Impacted Files
Coverage Δ

pkg/handlers/webhookreceiver.go
77.46% <ø> (+11.43%)
:arrow_up:

pkg/handlers/webhookrhobsreceiver.go
0.00% <0.00%> (ø)

pkg/handlers/helper.go
30.43% <30.43%> (ø)

/retest

I'm going to lgtm this to push it along and get an image created. In my own personal tests this seems to be working. We can address anything else we find in subsequent PRs.
/lgtm

/approve