HHH-17164 - Proper, first-class soft-delete support

HHH-17164 - Proper, first-class soft-delete support
https://hibernate.atlassian.net/browse/HHH-17164

It's nice to be able to define these at different levels.

I am wondering though if we really want to allow nulls in the database for these mappings.  Not speaking about how common or uncommon such a strategy is, but more thinking about the queries needed to handle soft-deletes.
A sneaky aspect, that affects this, which I had not considered initially is that soft-deletes will require behavior very similar to @NotFound mappings.  Or at least might, depending on how we want to treat many-to-one references to a soft-deleted row.  Specifically whether we want to manage the "extra" referential integrity ourselves and force nulling the fk value or if we want to consider data like this as valid:
customer {
    id = 1,
    removed=true,
    ...
}

order {
    id = 1,
    cust_fk = 1,
    ...
}

It's nice to be able to define these at different levels.

Even if we have a single annotation, it would still be possible to define this at different levels (package + class + attribute). Due to our defaulting in the annotation, we can easily detect that e.g.
@SoftDelete(converter = MyConverter.class)
package com.mycompany.entity;

...
@SoftDelete(column = "removed")
public class MyEntity {
   ...
}

for MyEntity we also use the MyConverter, because the converter used in the @SoftDelete annotation at the entity level is the default one. I acknowledge that this might be confusing if package level configurations can't be overridden so that defaults are used again for specific entities, but then I'd argue that this would be an unrealistic case anyway. Because why would anyone want to have different database representations for this soft deleted flag across a model? The only reasonable override is IMO the column name, like in the example above.

I am wondering though if we really want to allow nulls in the database for these mappings. Not speaking about how common or uncommon such a strategy is, but more thinking about the queries needed to handle soft-deletes.

I'd say let's disallow the column to be nullable. It's IMO unreasonable to allow a third state for this type of column. Allowing null wouldn't buy the user any storage space anyway, so I think for our first class support we should simply limit ourselves to reasonable uses. If people cry up, we can still add null support later if we want to.

"Valid" insofar as loading that data and treating the corresponding Order#customer as null

I agree that to-one associations will/should behave as if all associations to types that are soft-deletable are annotated with @NotFound(IGNORE). In fact, I think we could even model it this way i.e. set the not found action in the runtime model to IGNORE if the target type is soft-deletable.

I understand we can do all these things you mention.  However, the other aspect to splitting or not is separation of concerns.  Personally I am not a fan of uber-annotations just to consolidate into a single one.  I'd argue we have 2 distinct pieces of configuration info -

How to handle the boolean values
Describe the column.

Now, given the the neutering of @SoftDeleteColumn from my last pushes, I can see moving back to a single annotation also.  When more was configurable for the column I liked the separation for that reason.  But right now there are 3 values configurable there -

name
length of char-based columns
nullability

I can definitely see an argument to dropping nullability as I mentioned above.  The need to craft predicates like deleted is null or deleted = ? is clearly sub-optimal.  At least as an initial offering, anyway.  If people request allowing nullability, we can reconsider.
And, is length really useful?  Personally I've never seen this as anything more than 1 in the real world ('Y'/'N', 'T'/'F', ...).
If we remove those 2, we now have just the column-name, which is even more persuasive in terms of having one annotation.  But if we are allowing configuration of multiple aspects of the column, I still prefer keeping them split.  I could be persuaded by quorum though :D
You do bring up a good point about "overrides" from the package level.  At the moment I treat that as a complete replacement - it effectively ignores the package-level annotation when you apply one more locally (class or attribute).  There is definitely a discussion to be had though wrt allowing override of just parts (which is irrespective of single v. multiple annotations btw).
Lastly, given the ability to define this at the package level, I have also been thinking about the ability to disable soft-deletes at the class and attribute levels (akin to converter).  I think it is forced to require users to use different Java packages if they want some to be soft-deleted and others not.  I also am not convinced that annotating the package necessarily means you want collections soft-deletable.  Something like
/**
 * Enable for the package...
 */
@SoftDelete(converter = MyConverter.class)
package com.mycompany.entity;

package com.mycompany.entity;

/**
 * Disable for this particular entity
 */
@Entity
@SoftDelete(enabled=false)
class TheEntity {
    ...
    /**
     * Disable for this collection
     */
    @ElementCollection
    @SoftDelete(enabled=false)
    Collection<String> someStrings;
}

Thanks for the feedback!

Looks great. Just a minor comment about the lines that CodeQL complains about, it seems the JDBC based tests try to verify that the tables contain the soft-delete columns. You also added a few tests that verify this by running HQL/find operations and checking the emitted SQL. I prefer the latter also because it would get rid of the CodeQL warning :)

Honestly not too worried about CodeQL warnings on tests.  But I can change the few that still use that.  However there are lots of tests using HQL, load, etc.  The schema verify ones are just attempting to check the exported schema in a number of cases - writing HQL, load, etc operations to test all such cases is simply not feasible nor desirable.

Great work!

Thanks :)

Applied


Allow to blacklist classes and packages

Make it possible to blacklist classes and packages via annotation, something like
@BlacklistPackage("java.util")
public class TestClass {
    ...
}

Possibilities:

Use GLOB-like-Patterns, like java.util.** or java/util/**
Also add @WhitelistPackage similar to the path restrictions (should that overpower the Blacklist?)
Should java.lang always be allowed?

Resolved, current implementation:

GLOB-like Syntax java.util**
Everything is whitelisted by default, so the blacklist can again be overpowered by the whitelist
Only in effect when AJTS is active and the class is being loaded (the first time)


Add settings.json and extensions.json to vscode extensions

extensions.json can be used for "workspace recommendations": https://code.visualstudio.com/Docs/editor/extension-gallery#_workspace-recommended-extensions

@Gama11 Sorry but we have decided that those are too generic to associate them with vscode. This could only be applicable if there was a way to determine that those files are under the vscode folder.

Quoting @robertohuertasm:

I'm including tsconfig.json for the moment. settings.json and tasks.json are too generic to apply a vscode icon to them.

tasks.json is still part of the list though?

@Gama11 You are seeing the tree and missing the forest.

I'm more leaning towards completely remove the vscode icon to all these json files than to associate them all with it.  If we want to be consistent, and provided we don't have a way to detect the file tree position of the file, that would be for me the only alternative.

@robertohuertasm Yes, I would agree with that. The main reason behind this PR was that the inconsistent icon usage in my .vscode folders bothered me... If the files use the json or vscode icon doesn't matter that much to me in the end.

@Gama11 Welcome to my "O.C.D. Club".

@JimiC do you have any objections on using json icons for launch.json and tasks.json?

None.

@Gama11 are you willing to create a new PR with those files out?
As a side note, eventually, tsconfig.json and jsconfig.json are going to have their own icon but we can keep them at the moment.


Bicep updates: Add containers to storage, clean up

Repoman Generation Results
Repoman pushed changes to remotes for the following projects:
Project: todo-csharp-cosmos-sql
Remote: azure-samples-staging
Branch: pr/1082
You can initialize this project with:
azd init -t Azure-Samples/todo-csharp-cosmos-sql -b pr/1082

View Changes | Compare Changes

Project: todo-csharp-sql-swa-func
Remote: azure-samples-staging
Branch: pr/1082
You can initialize this project with:
azd init -t Azure-Samples/todo-csharp-sql-swa-func -b pr/1082

View Changes | Compare Changes

Project: todo-csharp-sql
Remote: azure-samples-staging
Branch: pr/1082
You can initialize this project with:
azd init -t Azure-Samples/todo-csharp-sql -b pr/1082

View Changes | Compare Changes

Project: todo-java-mongo-aca
Remote: azure-samples-staging
Branch: pr/1082
You can initialize this project with:
azd init -t Azure-Samples/todo-java-mongo-aca -b pr/1082

View Changes | Compare Changes

Project: todo-java-mongo
Remote: azure-samples-staging
Branch: pr/1082
You can initialize this project with:
azd init -t Azure-Samples/todo-java-mongo -b pr/1082

View Changes | Compare Changes

Project: todo-nodejs-mongo-aca
Remote: azure-samples-staging
Branch: pr/1082
You can initialize this project with:
azd init -t Azure-Samples/todo-nodejs-mongo-aca -b pr/1082

View Changes | Compare Changes

Project: todo-nodejs-mongo-swa-func
Remote: azure-samples-staging
Branch: pr/1082
You can initialize this project with:
azd init -t Azure-Samples/todo-nodejs-mongo-swa-func -b pr/1082

View Changes | Compare Changes

Project: todo-nodejs-mongo
Remote: azure-samples-staging
Branch: pr/1082
You can initialize this project with:
azd init -t Azure-Samples/todo-nodejs-mongo -b pr/1082

View Changes | Compare Changes

Project: todo-python-mongo-aca
Remote: azure-samples-staging
Branch: pr/1082
You can initialize this project with:
azd init -t Azure-Samples/todo-python-mongo-aca -b pr/1082

View Changes | Compare Changes

Project: todo-python-mongo-swa-func
Remote: azure-samples-staging
Branch: pr/1082
You can initialize this project with:
azd init -t Azure-Samples/todo-python-mongo-swa-func -b pr/1082

View Changes | Compare Changes

Project: todo-python-mongo
Remote: azure-samples-staging
Branch: pr/1082
You can initialize this project with:
azd init -t Azure-Samples/todo-python-mongo -b pr/1082

View Changes | Compare Changes


Hanger more info wont display

when in the main hanger page, no module info is shown it merely states default

text.

This panel is going to be replaced - temporary holding panel for now.

cool cool


[AKS]Leave only one default role for Automatic clusters

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

General Guidelines

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[x] My extension version conforms to the Extension version schema

For new extensions:

[x] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

AKS

Live Test : https://dev.azure.com/msazure/CloudNativeCompute/_build/results?buildId=93531457&view=results

Live test passed today
https://dev.azure.com/msazure/CloudNativeCompute/_build/results?buildId=93602522&view=logs&j=b162b355-d59d-5864-ce0f-0a70f12dd28b.


Beta 20170112 updates

This PR adds links and doc updates to the release notes, updates the version and build strings, regenerates the sql diagrams, and makes a few edits to the type casting additions.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/cd323dd4d4bcbd664d56a0aea7faedf97d5bf4b9/

LGTM


final pipeline adjustments

I made some code adjustments so that it would run on my machine.

Merge complete. Thank you Tim!


NPR Morning Edition feed is down

The pyther feed looks good, but it's not pulling over to the podcast feed.

Hi @snarffalita. Can you provide a little more context here? The feeds appear to be getting updated and they are reflected in my podcast app.

Apologies, I am new to github and wasn't sure what would be helpful. I'm
using Podcast Republic and it says Morning Edition has not been updated
since yesterday, but pyther is showing content today.
On Wed, Dec 13, 2023, 11:56 AM Matthew Gyurgyik @.***>
wrote:

Hi @snarffalita https://github.com/snarffalita. Can you provide a
little more context here? The feeds appear to be getting updated and they
are reflected in my podcast app.
—
Reply to this email directly, view it on GitHub
https://github.com/pyther/podfeed/issues/10#issuecomment-1854345723, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BDDS72EWYOUPYEGDFNBBF3LYJHM2XAVCNFSM6AAAAABATPLBO2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJUGM2DKNZSGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for the additional context. I've identified the issue. The parser (in the code) looks for the data-play-all attribute in the website html and generates the feed based on that. For morning edition, the a data-play-all attribute is not generated for the current day. Therefore the morning edition will always be a day behind.
Looking at the parser code and the current structure of the NPR web pages, the parser needs to be reworked. Unfortunately, I won't have any time to work on that any time soon. While I would like say I'll be able to work on this within the next few weeks, realistically I probably won't have a chance to work on this for many months. Hopefully, someone else might be able to take a stab at this.

In theory, this should be fixed in https://github.com/pyther/podfeed/commit/5f4c7487fd036ee40d50ccd81b045d108fe0cd99. I changed the code to parse the current stories from the program page and the full episodes from previous days.

The last two days have appeared after airing. This appears fixed. Thanks for reporting the issue and let me know if there are any other problems.


MD004 flagged on unordered sublist

I have a list ordered in the same format below but I get the MD004 error.  Even if I copy the sublist example in the MD004, I still get the error.
* Item 1
  + Item 2
    - Item 3
  + Item 4
* Item 4
  + Item 5

That is correct behavior for the default MD004 configuration of “consistent“. For the example above to work you need to have changed that to “sublist”. Can you please confirm that you’ve done so and show the configuration you used?


ENH/API/IO: integrate with MNE-Python object system

Now that there is the new shiny issue tracker here is the NEO counter-part issue to:
https://github.com/mne-tools/mne-python/issues/870
Looking forward to discussing and tackling this ;-)

The matching pull request in MNE-Python is now closed.


KAFKA-6424: QueryableStateIntegrationTest#queryOnRebalance should accept raw text

Remove to .toLowerCase in word count stream
Add method to read text from file, and move text to resources file
Minor cleanup: change argument order in assertEquals

Committer Checklist (excluded from commit message)

[x] Verify design and implementation
[x] Verify test coverage and CI build status
[x] Verify documentation (including upgrade notes)

Build error is related:
Unknown license: /home/jenkins/jenkins-slave/workspace/kafka-pr-jdk9-scala2.12/streams/src/test/resources/data/sampleText.txt

We cannot just a file without license header. Please move back to in-memory. Thanks.

Thanks. I missed that on the output. I'm moving it back to in-memory.

My intention was to allow the test to read from an external file.
File has bigger capacity which can test related code at higher scale.

@tedyu Would the file path/name be hardcoded? Or how to pass in the file name? We need the in-memory fall-back as we cannot just add a file without license header.
An alternative would be, to create the file at startup and read it back during the test.

Does data file have to contain license header ?
Hardcoding file path is fine - if the file doesn't exist, we can fall back to the current text.
Thanks

If we want to have the text in a file and the file must have a license I could just skip the license while reading, or parse the license lines, adding them to the word count stream, and adjust the test to accommodate this change.

Either way is fine.
Probably skipping the header is less work for you.
Thanks

I've done a few things to try to address your comments. I removed the header skipping logic and now default to a list of strings that is enough to run the tests.
I documented the method to make what it is doing clear, and changed the default file name to resources/QueryableStateIntegrationTest/inputValues.txt which I think is bit more obvious.

Retest this please.

Retest this please.

Merged to trunk
Thanks for the PR @h314to !

Great! Thanks for the helpful reviews.


Create Forked properties

What does this PR change?

Does the PR depend on any other PRs or Issues? If yes, please list them.

Checklist
I confirm, that I have...

[ ] Read and followed the contributing guide in CONTRIBUTING.md
[ ] Added tests for this PR
[ ] Formatted the code using npm run format (if applicable)
[ ] Updated documentation on the Paralus docs site (if applicable)
[ ] Updated CHANGELOG.md

Marking as invalid an closing.


Adds support for the new macOS MTGA client

This PR reads client data from different paths based on the system it is running on.
Actually supports the same systems as MTGA client: Windows and macOS.
Needs testing on Windows since I only own a Mac.

Thanks so much! You rock :)


Change border colours of the transactions

Change the border of income and expense transactions to '#B6CEFC80' when in dark mode.

Hey Ankit Chowdhury,
Hope you are doing well. I have check the issue and interested on fix it. Please assign the issue to me.
Thanks,
Shabbir

@Sabbirfeni sure

Thanks, I am working on it.


docs: Add diagram and postman collection for credit accounts

Description
Add diagram and postman collection for credit accounts
Fixes # ORV2-1935 (partial)
Type of change

[X] Documentation

How Has This Been Tested?
N/A
Checklist

[X] I have read the CONTRIBUTING doc
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works (N/A)
[X] New and existing unit tests pass locally with my changes
[X] Any dependent changes have already been accepted and merged

Thanks for the PR!
Deployments, as required, will be available below:

Frontend
Vehicles
Dops
TPS-Migration

Please create PRs in draft mode.  Mark as ready to enable:

Analysis Workflow

After merge, new images are promoted to:

Merge Workflow

Closing PR, will add more documentation and open another PR later


Does the @icon item have to be a URL address?

I have written a configuration with the @icon item as a base64-encoded image like this:
new UserscriptPlugin({
    headers: {
    // ...
    icon: 'data:image/x-icon;base64,...',
    // ...
    }
})

There is an error message like this:
[webpack-cli] HookWebpackError: Validation groups: main
An instance of Headers has failed the validation:
 - property icon has failed the following constraints: icon must be a URL address

    at makeWebpackError (.\node_modules\.pnpm\webpack@5.88.0_webpack-cli@5.1.4\node_modules\webpack\lib\HookWebpackError.js:48:9)
    at .\node_modules\.pnpm\webpack@5.88.0_webpack-cli@5.1.4\node_modules\webpack\lib\Compilation.js:3068:12
    at eval (eval at create (.\node_modules\.pnpm\tapable@2.2.1\node_modules\tapable\lib\HookCodeFactory.js:33:10), <anonymous>:34:1)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
-- inner error --
Error: Validation groups: main
An instance of Headers has failed the validation:
 - property icon has failed the following constraints: icon must be a URL address

    at ValidateHeaders.validateHeaders (.\node_modules\.pnpm\webpack-userscript@3.2.2_webpack@5.88.0\node_modules\webpack-userscript\dist\features\validate-headers\impl.js:45:23)
    at .\node_modules\.pnpm\webpack-userscript@3.2.2_webpack@5.88.0\node_modules\webpack-userscript\dist\features\validate-headers\impl.js:17:73
    at _next0 (eval at create (.\node_modules\.pnpm\tapable@2.2.1\node_modules\tapable\lib\HookCodeFactory.js:75:10), <anonymous>:54:16)
    at eval (eval at create (.\node_modules\.pnpm\tapable@2.2.1\node_modules\tapable\lib\HookCodeFactory.js:75:10), <anonymous>:79:1)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
caused by plugins in Compilation.hooks.processAssets
Error: Validation groups: main
An instance of Headers has failed the validation:
 - property icon has failed the following constraints: icon must be a URL address

Does the @icon item have to be a URL address?

Does the @ICON item have to be a URL address?

The truth is, I have no idea.
I found no clue about the use of @icon in data URI format, based on docs of the following 4 userscript engines.
TamperMonkey: userscript headers
GreaseMonkey: metadata block
GreasyFork: meta keys
ViolentMonkey: metadata block
According to tampermonkey, @icon, @iconURL and @defaulticon are substitutes of each other. Noticing the name @iconURL, that is why I defined @icon to be an URL address.
If you found any docs or use cases of @icon in data URI format, please let me know, thank you!
:)

I also learned this usage from other scripts, but I found some issues about it:
https://github.com/Tampermonkey/tampermonkey/issues/1500
https://github.com/violentmonkey/violentmonkey/issues/1130
I have only used Tampermonkey, I hope this is helpful.


Controller and Info text display bug in arcade

Describe the bug
When I clicked the download button, after the download was completed, the display of Controller and Info of Arcade changed from Chinese to English. Including the blocks in Controller and Info

Fixed in /beta


Gym and ALE

Hi, the environments here use Atari of Gym. Are they totally same as Atari of ALE?

Could you ask this on gym repo?


Fix WithArguments when used with IFactory bindings including Components

Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[x] No compiler errors or warnings

Pull request type

Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Issue Number

Issue Number: #196
Does this introduce a breaking change?

[x] Yes
[x] No

Depends on the definition of "breaking":
Some unnecessary WithArguments use which was previously valid, might now produce errors about unused injected values. However, this is very unlikely to be a major issue for anyone, and points out actual problems (values being unused) in any case.
Other information

This fix has been in use in Yousician for a long time already without issues, only now took the time to prepare a PR out of it.

On which Unity version has this been tested?

[ ] 2020.4 LTS
[ ] 2020.3
[ ] 2020.2
[ ] 2020.1
[x] 2019.4 LTS
[ ] 2019.3
[ ] 2019.2
[ ] 2019.1
[ ] 2018.4 LTS

Scripting backend:

[x] Mono
[x] IL2CPP

Note: Every pull request is tested on the Continuous Integration (CI) system to confirm that it works in Unity.
Ideally, the pull request will pass ("be green"). This means that all tests pass and there are no errors. However, it is not uncommon for the CI infrastructure itself to fail on specific platforms or for so-called "flaky" tests to fail ("be red").  Each CI failure must be manually inspected to determine the cause.
CI starts automatically when you open a pull request, but only Releasers/Collaborators can restart a CI run. If you believe CI is giving a false negative, ask a Releaser to restart the tests.

@sbergen: regarding whether or not it's a breaking change: imho code for which the behavior changes due to the applied code changes relies on a bug (/broken contract), and the changes attempt to fix this bug. So if you ask me, this could be dealt with as being not a breaking change.
On the one hand, it looks like the added test cases don't cover all methods that got adjusted, on the other hand, just by looking at the code, all adjusted methods should work fine if any of them does.
On the other hand, since I'm not familiar with this part of Extenject yet, I don't know if there are any other parts in the code where the same fix might need to be applied.
I'd be fine with merging this Pull Request as it is right now.

Merged
Thanks!

@sbergen Yousician is an awesome music app! Should/can it be added to the apps that use (x)enject list?
You can all things you want to tell about Yousician it here. And it will be added to the list!


Makefile:58: recipe for target 'bmdcapture' failed

Can't seem to get any of it to build correctly. I followed the instructions on the Wiki (Including insalling all the applications from source).
Ubuntu 18.04 LTS
Flattened and rebuilt the machine a few times trying get it working. This is the log.
/home/a/Downloads/bmdtools-master# make SDK_PATH=/home/a/Downloads/BMSDK/Linux/include/
g++ -o bmdcapture bmdcapture.cpp modes.cpp /home/a/Downloads/BMSDK/Linux/include//DeckLinkAPIDispatch.cpp pkg-config --cflags libavcodec libavformat libswscale libavutil -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -Wno-multichar -I /home/a/Downloads/BMSDK/Linux/include/ -fno-rtti -g pkg-config --libs libavcodec libavformat libswscale libavutil -lm -ldl -lpthread
bmdcapture.cpp: In function ‘void avpacket_queue_flush(AVPacketQueue*)’:
bmdcapture.cpp:95:33: warning: ‘void av_free_packet(AVPacket*)’ is deprecated [-Wdeprecated-declarations]
av_free_packet(&pkt->pkt);
^
In file included from /usr/local/include/libavformat/avformat.h:258:0,
from bmdcapture.cpp:37:
/usr/local/include/libavcodec/avcodec.h:3908:6: note: declared here
void av_free_packet(AVPacket pkt);
^~~~~~~~~~~~~~
bmdcapture.cpp: In function ‘int avpacket_queue_put(AVPacketQueue, AVPacket*)’:
bmdcapture.cpp:117:47: warning: ‘int av_dup_packet(AVPacket*)’ is deprecated [-Wdeprecated-declarations]
if (pkt != &flush_pkt && av_dup_packet(pkt) < 0) {
^
In file included from /usr/local/include/libavformat/avformat.h:258:0,
from bmdcapture.cpp:37:
/usr/local/include/libavcodec/avcodec.h:3899:5: note: declared here
int av_dup_packet(AVPacket pkt);
^~~~~~~~~~~~~
modes.cpp: In function ‘void print_display_mode(IDeckLinkDisplayMode, int)’:
modes.cpp:38:5: error: ‘BMDDisplayModeSupport’ was not declared in this scope
BMDDisplayModeSupport displayModeSupport;
^~~~~~~~~~~~~~~~~~~~~
modes.cpp:38:5: note: suggested alternative: ‘bmdDisplayModeSupports3D’
BMDDisplayModeSupport displayModeSupport;
^~~~~~~~~~~~~~~~~~~~~
bmdDisplayModeSupports3D
Makefile:58: recipe for target 'bmdcapture' failed
make: *** [bmdcapture] Error 1

Managed to resolve it, the build will fail if you use SDK 11.2. SDK 10.11.4 is the newest build that will work

I was traveling I guess the codebase needs to be updated.

this affects me also


Add infrastructure setters and validations

@jordan2175 could you please review this PR

Thanks for the pull requests.  I will give you public credit in the readme and the individual files if you would like, but not on the copyright line.  I need things to be very un-encumbered to make it easy for people to know what they can and can not do with this code.

@jordan2175 OK, it wasn't clear what I was supposed to put there, so just tried to follow the pattern

So if I accept this pull request and immediately change it and push an update, are you okay with that?

@jaysneg Please verify on this pull request if you are okay with me accepting the pull request but then removing your copyright line.

Please verify on this pull request if you are okay with me accepting the pull request but then removing your copyright line.

ok


plugin works if opening a file from command line, doesn't work if opening a file from vim

Hi Andrew,
First of all, many thanks for your effort building and maintaining this project.
I've played a bit a few months ago and just came back to it and realized that is not working if I open a file from vim.

I start vim in my terminal and open a file from vim using :e /path/to/file

this case Switch plugin won't work - eg. not changing 'foo' to 'bar', seemingly has no effect at all

I start vim in my terminal passing /path/to/file as an argument vim /path/to/file

Switch plugin works as intended

I tried with this minimal 4 lines .vimrc
     call plug#begin('~/.vim/plugged')
        Plug 'AndrewRadev/switch.vim'
     call plug#end()
       let b:switch_custom_definitions = [ ['foo', 'bar', 'baz'] ]

Thanks,
Szilveszter
vim: 8.0.494 compiled from source using ./configure --enable-python3interp=dynamic --enable-cscope --enable-rubyinterp --without-x --disable-gui && make && sudo make install
Switch: latest version ddc069787c98ab90580667062908750257f43a34

I've found if I am using
let g:switch_custom_definitions
instead of
let b:switch_custom_definitions
then Switch works fine even if I open the file from vim.

I think I understand the problem. The custom definitions you've created with b:switch_custom_definitions are buffer-local. That's what the b: prefix means. You can find more information on the topic with :help b:.
Basically, if you wanted to only define switches for a particular filetype, you would use the b: variant. I guess I hadn't explained it well enough in the documentation, so I added some more information about it: https://github.com/AndrewRadev/switch.vim/blob/f9826833a84d600811d4214059e30e2892c24dbf/doc/switch.txt#L162-L170
Does this solve your issue?

Yes, that is perfect. Thanks a lot!


Trying to play something

When I type something like "!m q https://soundcloud.com/illeniumofficial/the-chainsmokers-dont-let-me-down-illenium-remix" It says "Queued",  "Started" and finished within 3 seconds. If I check the console it shows me this "Buffering errored: The system can not find the file specified".
Hope you know the problem!

you didn't setup ffmpeg properly. Follow the guide carefully.

I am 100% that I did set it up properly.

You didn't. Open cmd/terminal and type ffmpeg. It will say command not found or something similar. Try restarting your PC.

I even tried that what you said and then it just showed me all the commands. But I fixed it now by downloading the same version of ffmpeg as I used with Redbot and now it works.


hilo.hiloapi - Request for access token failed

Since now more than 24h, it look like my system is failing to get the new token from Hilo. No change on my homeassistant.
What I see in the logs:
2021-08-18 13:07:41 WARNING (SyncWorker_2) [custom_components.hilo.hiloapi] Request for access token failed.
It's the same after restarting. Is it just me ? The app still work fine, and no password changes

You can reset your Hilo password, that should work.
Le mer. 18 août 2021, à 13 h 20, macgaver @.***> a
écrit :

Since now more than 24h, it look like my system is failing to get the new
token from Hilo. No change on my homeassistant.
What I see in the logs:
2021-08-18 13:07:41 WARNING (SyncWorker_2)
[custom_components.hilo.hiloapi] Request for access token failed.
It's the same after restarting. Is it just me ? The app still work fine,
and no password changes
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/francispoisson/hilo/issues/23, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACRNQSCD2QAICNTNRF2Z3FLT5PTULANCNFSM5CMPVYKQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Indeed, problem fixed.  Not sure why my password needed a reset but that did the trick !
Thanks for the quick answer

The day Hilo will provide an official API, it will be easier to monitor
what's going on...
Le mer. 18 août 2021, à 14 h 58, macgaver @.***> a
écrit :

Indeed, problem fixed. Not sure why my password needed a reset but that
did the trick !
Thanks for the quick answer
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/francispoisson/hilo/issues/23#issuecomment-901354266,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACRNQSDXSG3C3GMY2YOUK53T5P7G7ANCNFSM5CMPVYKQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.


terraform.so: undefined symbol: tree_sitter_terraform

Describe the bug
Hi, I'm trying to install this parser, using https://github.com/nvim-treesitter/nvim-treesitter#adding-parsers directions. The installation seems to be ok, but I get this error when opening a terraform file:
Error detected while processing CursorHold Autocommands for "<buffer=1>":
E5108: Error executing lua Failed to load parser: uv_dlsym: /home/cova/.local/share/lunarvim/site/pack/packer/start/nvim-treesitter/parser/terraform.so: undefined symbol: tree_sitter_terraform
stack traceback:
        [C]: in function '_ts_add_language'
        /usr/share/nvim/runtime/lua/vim/treesitter/language.lua:33: in function 'require_language'
        /usr/share/nvim/runtime/lua/vim/treesitter.lua:38: in function '_create_parser'
        /usr/share/nvim/runtime/lua/vim/treesitter.lua:93: in function 'get_parser'
        ...ext-commentstring/lua/ts_context_commentstring/utils.lua:124: in function 'get_node_at_cursor_start_of_line'
        ...-commentstring/lua/ts_context_commentstring/internal.lua:116: in function 'calculate_commentstring'
        ...-commentstring/lua/ts_context_commentstring/internal.lua:145: in function 'update_commentstring'
        [string ":lua"]:1: in main chunk

To Reproduce
I just followed the directions described above. My configuration for this plugin:
local parser_config = require "nvim-treesitter.parsers".get_parser_configs()
parser_config.terraform = {
  install_info = {
    url = "/home/cova/repos/github/utils/tree-sitter-hcl/", -- local path or git repo
    files = {
      "src/parser.c",
      "src/scanner.cc",
    }
  },
    filetype = "terraform", -- if filetype does not agrees with parser name
}

Then I just ran TSInstall terraform (also tried from grammar)
I'm pretty sure that I'm doing something wrong, but can't see what :)
I'm running lunarvim (master) with neovim 0.6.1
Any suggestions where to look?
Thanks!

Yep, indeed! I was tricked because I was searching for "terraform" in the list of supported files :)
sorry for the noise and thanks for the quick answer!

Hey @cova-fe,
no worries, the setup is a little confusing with hcl vs terraform, actually i think one could probably write a terraform parser ( adding the missing keywords ) using this as a base and make highlighting and everything a little more comfortable in nvim-treesitter. Maybe in a future version!


Status bar color is not updated

Issue Type: Bug

Install ReUI extensions > status bar color remains "purple"

Update settings.json file to change statusbar.background (let's say black) and statusbar.foreground (white) > status bar color remains "purple"

VS Code version: Code 1.44.0 (2aae1f26c72891c399f860409176fe435a154b13, 2020-04-08T08:23:56.137Z)
OS version: Darwin x64 19.4.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 x 2600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
2, 3, 3

Memory (System)
16.00GB (0.12GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

markdown-all-in-one
yzh
2.8.0

(1 theme extensions excluded)

When no folder is open the setting is statusBar.noFolderBackground

Thanks @gjsjohnmurray
Seems to be the right track !
But I still have the purple status bar with the configuration below:
{
    "workbench.colorTheme": "ReUI Default Bordered (react)",
    "workbench.colorCustomizations": {
        "statusBar.noFolderbackground": "#282c34",
        "statusBar.noFolderforeground": "#ddd",
    }
}

Do you have an idea of what I am missing ?

@ant-1 Background and foreground need to be capitalized:
{
    "workbench.colorTheme": "ReUI Default Bordered (react)",
    "workbench.colorCustomizations": {
        "statusBar.noFolderBackground": "#282c34",
        "statusBar.noFolderForeground": "#ddd",
    }
}

My bad, you are totally right !
Thanks @RMacfarlane


Images no longer working

Looks like the unsplash endpoint has depreciated, and the package no longer gives valid image urls. Any other options that we can use to fix this?

Yes, #74 fixes this I believe.

Released in https://pub.dev/packages/faker/versions/2.2.0


Help im stuck trying to update my Remarkable 2 according to https://github.com/ddvk/remarkable-update

I have done everything on this page. is there any way to force update without using python.
This is error on the device.

So i referred to this (https://github.com/ddvk/remarkable-update#usage)
-Step 3 is where i get stuck i cannot find that (file/folder/conf)
I cannot find the ./updates ANYWHERE?
I then approach Via the CLI and nothing works for me.

Step 2 looks like

Then im stuck here!

I'm very new to this and I have spent two days looking around at constant threads i cant get the automatic update to return back after reverted back to original according to:
sh -c "$(wget https://raw.githubusercontent.com/ddvk/remarkable-hacks/master/patch.sh -O-)" _ purge
Nothing works for me. I would really appreciate some insight.

Try this https://github.com/Jayy001/codexctl

So I downloaded the Zip file opened it. When I opened it a command window came up with command and immediately closes command window.  This is what shows i had to record a video and slow it down to show you.

Then i continue with the commands Tapped into the remarkable like this:
Building from source
git clone https://github.com/Jayy001/codexctl.git
cd codexctl
pip install requests loguru
This error Shows...

Am I doing it wrong or missing something?

At this point!!!! I'm ready to surrender my screen to you. lol Thank you for your time and efforts with me.

What do you advise next?

oh dear :)
I'm afraid a little familiarity with command line is required here :)
The zip contains an executable which you start after opening a shell/CLI.
When you run it with ./codexctl --help you get a list of things you can do with it.
See section "Usage" for examples but instead of python codexctl.py you just write ./codexctl
The second way "run from source" is not needed. You'll need software development tools like git and python. The error you get is because you don't have git.
(and to take a screenshot in Windows, I think it's the Print key? ;-) Or use Greenshot)


Added examples for export

Type of changes

[ ] Bug fix
[ ] New feature
[ ] Documentation / docstrings
[ ] Tests
[x] Other

Checklist

[x] I've run the latest black with default args on new code.
[ ] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[ ] I've added tests for new code.
[x] I accept that @willmcgugan may be pedantic in the code review.

Description
Added examples for export features provided by console.

What I meant was informing the user what the script is doing.
For instance:
Wrote "html_export.html"

Otherwise you would have no idea that it created some files.


Extended configuration mode and bug-fix

Hello,
I started adapting the library for a project that has the following requirements:

setting credentials if they do not exist
once the network is set, do not start the AP at startup in case it cannot connect to the network
For example, in case of a power failure, the router will start more difficult and the application will be stuck in AP mode
option to restore the default values (from the web page or button)

I also modified:

the names of the defines so that there is a lower risk of being equal to defines from other libraries
defines with values 0/1 that can be added in platformio,ini with build_flags
option to completely remove serial port calls

Kind regards,
Bogdan

Hi, thank you for your valuable contribution!
If you feel like it, also take a look at this variant of the library that uses ESPAsyncWebServer as base webserver.
From the tests I have carried out on some of my projects I have noticed that the performance of the library in question is better than the one included with the core.
https://github.com/cotestatnt/async-esp-fs-webserver

Hi, thank you for your valuable contribution! If you feel like it, also take a look at this variant of the library that uses ESPAsyncWebServer as base webserver. From the tests I have carried out on some of my projects I have noticed that the performance of the library in question is better than the one included with the core.
https://github.com/cotestatnt/async-esp-fs-webserver

Thanks for the suggestion, I've seen that library, but I think this one is more suitable for my project.
The device I work on measures the energy to monitor a pump over time.
The web server part is only used for initial configuration and some troubleshooting - in the local network.
I think, it will never be the case to have more clients connected simultaneously.
However, the project includes several libraries and need ota, so I have to reduce the size as much as possible - even on Esp32
I still have some changes that I would like to make, maybe you can help me :)
Please let me know how you prefer to communicate for these.
Thank you,
Bogdan

Good morning @BogyS
For next changes it's the same for me: you can open an issue or if you already have some working code, make a pull request as you have already done.
As for your project, keep in mind that you can use a partition scheme that reserves more space for the firmware (for example, I recently developed a project with a firmware of almost 1.6Mbytes and I selected "Huge app").
For firmware updates, consider that you can also do it from the browser without adding other libraries. In the "async" library for example, I integrated this functionality directly into the /setup page.
As soon as I have some time I'll do it in this library too


Logout REST api call for the controller feature request

Hi
Exposing deis auth:logout through the controller API.

Deis' controller uses token-based authentication now, so there is no server-side session or state that is maintained. When we made that change, we got rid of the logout REST endpoint. You can still do deis logout, but it just destroys the local copy of the token and doesn't make any network calls.
@daemonza is there a need for the logout endpoint if the server isn't storing session state?

ping @daemonza

Yes, so we don't really use the cmd line deis utility that much. We are busy with a web interface through which we are deploying applications from docker repo's. The problem I have is, if I want to "expire" a user's token. Without the API call, how would I be able to do it.

In Django REST Framework's implementation, there is no way to expire a token by default. It is persistent, meaning that every time you log in, you are given the same auth token that was generated for you on the first registration/login. See http://www.django-rest-framework.org/api-guide/authentication/#tokenauthentication
If we are to add token expiration features, we should follow https://developers.facebook.com/blog/post/2011/05/13/how-to--handle-expired-access-tokens/ as there are a few cases where we need to expire the token (such as when we change the user's password).

ping @daemonza. Are you looking for token expiration, or was this explanation good for you?

closing as a duplicate of #2175.

yup thanks @bacongobbler


home-manager switch failing

Same error message as #647
But
home-manager switch
trace: warning: The option `qt.useGtkTheme' defined in `/home/daniel/.config/nixpkgs/nix-dotfiles/config/xsession/xsession.nix' has been changed to `qt.platformTheme' that has a different type. Please read `qt.platformTheme' documentation and update your configuration accordingly.
error: while evaluating the attribute 'buildCommand' of the derivation 'home-manager-generation' at /nix/store/h9jr0w5m541iilmwsm1sn8xw5nz1giij-nixos-18.09.2436.395a543f360/nixos/pkgs/stdenv/generic/make-derivation.nix:177:11:
while evaluating the attribute 'text' of the derivation 'activation-script' at /nix/store/h9jr0w5m541iilmwsm1sn8xw5nz1giij-nixos-18.09.2436.395a543f360/nixos/pkgs/stdenv/generic/make-derivation.nix:177:11:
while evaluating 'mkCmd' at /nix/store/ri57zbyl9cpi1qk7ay7s2gzqbm5rvg55-master.tar.gz/modules/home-environment.nix:347:17, called from undefined position:
while evaluating the attribute 'data' at /nix/store/ri57zbyl9cpi1qk7ay7s2gzqbm5rvg55-master.tar.gz/modules/lib/dag.nix:96:41:
while evaluating the attribute 'data' at /nix/store/ri57zbyl9cpi1qk7ay7s2gzqbm5rvg55-master.tar.gz/modules/lib/dag.nix:89:11:
while evaluating the attribute 'data' at /nix/store/ri57zbyl9cpi1qk7ay7s2gzqbm5rvg55-master.tar.gz/modules/lib/dag.nix:115:12:
while evaluating the attribute 'passAsFile' of the derivation 'home-manager-path' at /nix/store/h9jr0w5m541iilmwsm1sn8xw5nz1giij-nixos-18.09.2436.395a543f360/nixos/pkgs/stdenv/generic/make-derivation.nix:177:11:
while evaluating the attribute 'buildCommand' of the derivation 'home-manager-manpages' at /nix/store/h9jr0w5m541iilmwsm1sn8xw5nz1giij-nixos-18.09.2436.395a543f360/nixos/pkgs/stdenv/generic/make-derivation.nix:177:11:
while evaluating the attribute 'buildCommand' of the derivation 'manual-olinkdb' at /nix/store/h9jr0w5m541iilmwsm1sn8xw5nz1giij-nixos-18.09.2436.395a543f360/nixos/pkgs/stdenv/generic/make-derivation.nix:177:11:
while evaluating the attribute 'buildCommand' of the derivation 'home-manager-manual-combined' at /nix/store/h9jr0w5m541iilmwsm1sn8xw5nz1giij-nixos-18.09.2436.395a543f360/nixos/pkgs/stdenv/generic/make-derivation.nix:177:11:
while evaluating the attribute 'buildCommand' of the derivation 'generated-docbook' at /nix/store/h9jr0w5m541iilmwsm1sn8xw5nz1giij-nixos-18.09.2436.395a543f360/nixos/pkgs/stdenv/generic/make-derivation.nix:177:11:
while evaluating the attribute 'buildCommand' of the derivation 'options-db.xml' at /nix/store/h9jr0w5m541iilmwsm1sn8xw5nz1giij-nixos-18.09.2436.395a543f360/nixos/pkgs/stdenv/generic/make-derivation.nix:177:11:
while evaluating 'genRelatedPackages' at /nix/store/ri57zbyl9cpi1qk7ay7s2gzqbm5rvg55-master.tar.gz/doc/default.nix:27:24, called from /nix/store/ri57zbyl9cpi1qk7ay7s2gzqbm5rvg55-master.tar.gz/doc/default.nix:54:68:
while evaluating anonymous function at /nix/store/ri57zbyl9cpi1qk7ay7s2gzqbm5rvg55-master.tar.gz/doc/default.nix:45:57, called from undefined position:
while evaluating 'describe' at /nix/store/ri57zbyl9cpi1qk7ay7s2gzqbm5rvg55-master.tar.gz/doc/default.nix:32:18, called from /nix/store/ri57zbyl9cpi1qk7ay7s2gzqbm5rvg55-master.tar.gz/doc/default.nix:45:60:
while evaluating 'attrByPath' at /nix/store/h9jr0w5m541iilmwsm1sn8xw5nz1giij-nixos-18.09.2436.395a543f360/nixos/lib/attrsets.nix:24:35, called from /nix/store/ri57zbyl9cpi1qk7ay7s2gzqbm5rvg55-master.tar.gz/doc/default.nix:36:38:
Invalid package attribute path `qgnomeplatform'

nix-channel --list
home-manager https://github.com/rycee/home-manager/archive/release-18.09.tar.gz

Hmm,
while evaluating 'mkCmd' at /nix/store/ri57zbyl9cpi1qk7ay7s2gzqbm5rvg55-master.tar.gz/modules/home-environment.nix:347:17

perhaps you have some Home Manager master version sneaking in somewhere?

This turned out to be
programs.home-manager = {
    enable = true;
    path = "https://github.com/rycee/home-manager/archive/release-18.09.tar.gz";
  };

path being set to master


Deprecated link in CodeLabs in "Write Your First Flutter App, part 2"

In https://codelabs.developers.google.com/codelabs/first-flutter-app-pt2#2 there is a deprecated link that needs to be updated.

Hi @stasinato
This is a duplicate of https://github.com/flutter/flutter/issues/86258


infobutton not working in 1.8.1?

Seems that infobutton is not working in 1.8.1:
dialog --button1text "Open MSC" --button2text "Cancel" --infobuttontext "More Info"
#echo $?
case $? in
  0)
  	echo "Opening MSC"
  	open -a "/Applications/Managed Software Center.app"
  	;;
  2)
  	echo "Pressed Cancel"
  	;;
  3)
  	echo "More Info requested"
  	;;
  *)
  	echo "Something else happened"
  	;;
esac

the behaviour changed in 1.8.0 so by default the info button performs --infobuttonaction but does not exit  https://github.com/bartreardon/Dialog/wiki/Buttons-and-button-behaviour#the-more-information-button
if you want the infobutton to exit dialog with exit code 3, add --quitoninfo to your command.

OK, I got it working on 1.8.1 as you suggested...false alarm.

No problemo 👍🙂


HTML5: Add (multi-)touch events, fix inverted scrolling

Adds touch events
Reimplements mouse events without the GLUT interface to fix #7871

Quick question, would this be providing multi-touch functionality into desktop platforms? Would it be possible to extend the multi-touch to desktop? These days there's many tablets running Windows 10 with multi-touch, also at work we use multi-touch on Linux and the lack of the multi-touch in Godot is the only item left to convince the people at my job to give a chance to Godot.

@samvila No, this only touches files related to HTML5 export.
Adding multi-touch support to the Windows platform should be possible though.
I remember punto saying that there's some commented code that would implement multi-touch support for Windows. If someone takes the time to uncomment it and give it a check, it shouldn't be too hard to get it working


Reproducer for Case file conditional event fails other cases

The tests will be failing till this issue is fixed - https://issues.jboss.org/browse/JBPM-5541

reproducer included in https://github.com/droolsjbpm/jbpm/pull/724 though it's still not decided if that is going to be solved or how.


Plugin

Coverage decreased (-16.4%) to 20.274% when pulling c0ae7c774957a5965890e3df5d34f59aa084601e on plugin into 53dee0efab37b4d19a84880074d8945a9769641b on master.

Coverage decreased (-9.01%) to 27.632% when pulling 89a82660c3968e4d0cab1993d3f8bbf5547a34eb on plugin into 53dee0efab37b4d19a84880074d8945a9769641b on master.

Coverage decreased (-4.1%) to 32.5% when pulling 68773d2388f7b6f5bb50febfc06daf2ccca8fa25 on plugin into 53dee0efab37b4d19a84880074d8945a9769641b on master.

Coverage increased (+7.03%) to 43.675% when pulling 58d442bf6091b566dde5c689be8d9a868e80e12b on plugin into 53dee0efab37b4d19a84880074d8945a9769641b on master.


Add validation in github actions, including artifact collection

Seems like you had a flaky failure. Retriggered the job and documented here: https://github.com/submariner-io/submariner/issues/532#issuecomment-621947343

Sorry I'm not familiar with github actions.  Can you provide some detail in the PR description as to the purpose and how it's related to the travis jobs?

Ok, based on the conversation we had the other day during the team meeting, do you guys think we could merge this? :)


Grammer error in Projected CRS in .wtk

Excessive double quotes before every GEOGCS both in <https://raw.githubusercontent.com/USGS-Astrogeology/GDAL_scripts/master/OGC_IAU2000_WKT_v2/IAU2000_v2.wkt> and <https://raw.githubusercontent.com/USGS-Astrogeology/GDAL_scripts/master/OGC_IAU2000_WKT_v2/IAU2009_v2.wkt>.
According to "Geographic information - Well-known text representation of coordinate reference systems", the sole required separator between <crs name> and <geogcs> is <wtk separator>, which is comma, but in the two WTKs above the separator is ',"' (comma and double quote).
For example, in IAU2000_v2.wkt the actual text is
19910,PROJCS["Mercury_Equidistant_Cylindrical","GEOGCS["Mercury 2000" ...
where it should be
19910,PROJCS["Mercury_Equidistant_Cylindrical",GEOGCS["Mercury 2000" ...

Pull requests are always welcome! Would you min submitting one with the
desired change?
Thanks,
Jay
On Monday, September 12, 2016, yamamoto-mitsuo<EMAIL_ADDRESS>wrote:

Excessive double quotes before every GEOGCS both in <https://raw.
githubusercontent.com/USGS-Astrogeology/GDAL_scripts/
master/OGC_IAU2000_WKT_v2/IAU2000_v2.wkt> and <https://raw.
githubusercontent.com/USGS-Astrogeology/GDAL_scripts/
master/OGC_IAU2000_WKT_v2/IAU2009_v2.wkt>.
According to "Geographic information - Well-known text representation of
coordinate reference systems
http://docs.opengeospatial.org/is/12-063r5/12-063r5.html", the sole
required separator between  and  is ,
which is comma, but in the two WTKs above the separator is ',"' (comma and
double quote).
For example, in IAU2000_v2.wkt the actual text is
19910,PROJCS["Mercury_Equidistant_Cylindrical","GEOGCS["Mercury 2000" ...
where it should be
19910,PROJCS["Mercury_Equidistant_Cylindrical",GEOGCS["Mercury 2000" ...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/USGS-Astrogeology/GDAL_scripts/issues/8, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ACAgfFZvZkP68gfjrPBlSBwca3dLSIvrks5qpPrHgaJpZM4J6Ss4
.

Let me know if this fixes the issue.
-Trent
On Mon, Sep 12, 2016 at 6:36 AM, jlaura<EMAIL_ADDRESS>wrote:

Pull requests are always welcome! Would you min submitting one with the
desired change?
Thanks,
Jay
On Monday, September 12, 2016, yamamoto-mitsuo<EMAIL_ADDRESS>wrote:

Excessive double quotes before every GEOGCS both in <https://raw.
githubusercontent.com/USGS-Astrogeology/GDAL_scripts/
master/OGC_IAU2000_WKT_v2/IAU2000_v2.wkt> and <https://raw.
githubusercontent.com/USGS-Astrogeology/GDAL_scripts/
master/OGC_IAU2000_WKT_v2/IAU2009_v2.wkt>.
According to "Geographic information - Well-known text representation of
coordinate reference systems
http://docs.opengeospatial.org/is/12-063r5/12-063r5.html", the sole
required separator between  and  is ,
which is comma, but in the two WTKs above the separator is ',"' (comma
and
double quote).
For example, in IAU2000_v2.wkt the actual text is
19910,PROJCS["Mercury_Equidistant_Cylindrical","GEOGCS["Mercury 2000"
...
where it should be
19910,PROJCS["Mercury_Equidistant_Cylindrical",GEOGCS["Mercury 2000" ...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/USGS-Astrogeology/GDAL_scripts/issues/8, or mute
the
thread
<https://github.com/notifications/unsubscribe-auth/
ACAgfFZvZkP68gfjrPBlSBwca3dLSIvrks5qpPrHgaJpZM4J6Ss4>

.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/USGS-Astrogeology/GDAL_scripts/issues/8#issuecomment-246348925,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC86YtUDJI3kZWWzRx9A9rOc7_C29Giaks5qpVVEgaJpZM4J6Ss4
.

I believe the issue is resolved.
Thank you for your responses. Thank you very much for creating and promoting the standard.
--
Yamamoto, Mitsuo
Lunar and Planetary Exploration Data Analysis Group
Institute of Space and Astronautical Science
Japan Aerospace Exploration Agency


[Bug]: Github login is not going through on v1.47 postgres

Is there an existing issue for this?

[X] I have searched the existing issues

Description
On an instance pointing to postgres db, when I login with github user credentials, post setting up github sso, it fails with redirect uri error.
Steps To Reproduce

Pull v1.47 locally, and in the admin settings, point to a postgres db
Setup Github SSO
Try to login with the github credentials
The redirect uri error screen pops up

Public Sample App
No response
Environment
Production
Severity
High (Blocker to building or releasing)
Issue video log
No response
Version
v1.47

Tried this a couple of times more and not facing this issue any more - seems to be some configuration issue - hence closing it out.


Add Rules.UseDecorateeReuseForDecorators

This actually should be the default from the start, but at least we can add the rule to apply to all container decorators without explicit Reuse specified.

related to #139

fixed


vscode-graphql, LSP, IDE Roadmap 2022

The next chapter now that 2021/2020 LSP Roadmap is complete
This applies to graphql-language-service-server, vscode-graphql which will soon be merged with this monorepo, and many other IDE plugins who often utilize graphql-language-service-cli via python, lua, java, etc.
Suggestions are welcome in the comments, if you want to nominate any bugs, feature requests here or in vscode-graphql, or create an RFC and suggest one, please feel free to!
🐛 fixing bugs

[ ] better error handling & logging

[ ] https://github.com/graphql/graphiql/pull/2161

[x] SDL files just don't work...
[x] graphql-config in package.json doesn't work
[x] graphql-config in .ts/tsx files doesn't work (graphql-config upgrade)
[x] #2068
[x] https://github.com/graphql/graphiql/issues/1894 (trigger characters implemented but should be revisted)
[x] /* GraphiQL */ tag doesn't work for highlighting in vscode-graphql (link to PR)
[x] https://github.com/graphql/graphiql/issues/2212
[ ] https://github.com/graphql/graphiql/issues/2018
[ ] workspaces support related issues - load graphql-config from workspace root? vscode-graphql may need proper handling of workspaces + graphql-config projects as well.
[ ] dotenv issues (currently the workaround is to import dotenv directly in graphql config files)
[ ] Windows paths, replace URL().basename approach with vscode-uri for vscode uri string parsing
[ ] https://github.com/graphql/graphiql/issues/2182
[ ] !!! https://github.com/graphql/graphiql/issues/2172
[ ] !!! Parsing bug from getDocuments when documents is used (there are many duplicates)
if a user provides documents in config, then that is what triggers graphqlConfig.getDocuments() currently. we also call this excessively. there are many duplicates of this

💯 new features
when adding changes to getAutocompleteSuggestions  and other parts of graphql-language-service-interface, monaco-graphql will benefit!

[ ] minimal schema-less support for all language features possible

[ ] schemaless validation rules
[ ] minimal schemaless completion
[ ] https://github.com/graphql/graphiql/issues/2174

[ ] 💥  breaking change: new major version of interface modules that accepts object parameters to make it easier to deprecate and provide alternatives such as documentAST input and new config and APIs
[ ] 💥  breaking change: graphql-language-service-interface, -types,, -parser and -utils should all be merged into the single graphql-language-service.

[ ] leave some waypoint for direct users of the -interface methods and other packages, for example a warning on install for the final release of these packages as they are deprecated. otherwise they will miss out on new features. a great example of one of our direct users of getAutocompleteSuggestions is postwoman!

[ ] allow disabling validation in the language server or vscode-graphql: https://github.com/graphql/vscode-graphql/issues/280
[ ] support insertText for completion of object and list fields https://github.com/graphql/graphiql/issues/587
[ ] support vue, svelte for syntax highlighting at least
[ ] autocomplete field names for types/interface when using implements
[ ] jump to source field name on implemented interface fields
[ ] jump to definition for query fields and arguments
[ ] getDiagnostics should allow an option for schema validation, or detect whether schema types are present to provide additional schema validation rules, or gather errors from graphql-js validateSchema
[ ] graphql-language-service-server should send the lsp client instructive notifications about graphql-config - when it's detected, and when it's not, or when there are issues.

[ ] vscode-graphql should be configured to handle these notifications accordingly

[ ] additional language support - vscode-graphql grammars and in graphql-language-service-server for completion, diagnostics, etc

this is a complicated feature but it deserves prioritization!

[ ] TODO track down any new spec features that need to be implemented
[ ] investigate usage of notebooks API

🤝 merge vscode-graphql with graphiql monorepo

[ ] create issue menu & forms with options for reporting IDE vs web bugs/enhancement requests
[ ] migrate issues
[ ] migrate PRs
[ ] migrate github actions & other automation

🌀 refactors

[ ] refactor the language server to use graphql-tools and graphql-config completely, and replace parseDocument entirely with CodeFileLoader or a custom loader if needed.
[ ] refactor the extension client to make it easier to build behavior around graphql-config

💻 e2e suite for vscode-graphql
using https://github.com/microsoft/vscode-extension-samples/tree/main/lsp-sample as a fantastic point of reference:

[ ] test harness
[ ] diagnostics
[ ] completion
[ ] all supported schema config combinations
[ ] test configuration options
[ ] run in CI
[ ] test extension client behaviors

⚙ automation

[x] fix open-vsx publish for vscode-graphql
[ ] deploy previews for vscode-graphql
[ ] canary releases to graphql.vscode-graphql-insiders (vsce also seems to support pre-release flag now?)

Thanks @acao !


✨ Add Page to Select an Organisation(s)

User Need
As an employee who wants access to the MoJ GitHub Organisations
I want to be able to choose which organisations I would like to join
so that I can get the correct access to complete my work
Value
To be able to configure my access to the MoJ organisations so I can ensure I get the correct access I need.
Functional Requirements:

User can select MoJ / MoJAS (either or both)
User can not continue without selecting at least one organisation

Non-Functional Requirements:

Use checkboxes
Use error message for validation i.e. when user doesn't select an org, this should be obvious to the user that they need to select at least one org

Acceptance Criteria:

[x] After submitting an email address that is in the allow list, the user is presented with a page to select an organisation (MoJ / MoJAS)
[x] User cannot continue until at least one organisation is selected

Notes:

Ensure if they select both that both org names are accessible from the form data.


unheadVueComposablesImports ts error

Environment
ts 4.9
node 18
Reproduction

Describe the bug
When I add it, it gives ts error like on the screen
Additional context
No response
Logs
No response

Hey @productdevbook, thanks for the issue.
Looks like it was using a format for an earlier version of unplugin-auto-import. I've pushed up a fix to use the new format, it's available in 1.0.15.


Make phone numbers tel: links

https://www.gaintap.com/archives/clickable-website-phone-number-and-call-tracking-guide/


CmdDrawIndexed not drawing as expected

I've been following the Vulkan Tutorial and I implemented an index buffer and noticed that it does not draw my vertices correctly. It seems to be drawing in the reverse order.
My vertices
private Vertex[] vertices = new Vertex[]
{
	new Vertex(new float[] { -0.5f, -0.5f }, new float[] { 1.0f, 1.0f , 1.0f }),
	new Vertex(new float[] {  0.5f, -0.5f }, new float[] { 255/255f, 0.0f , 100/255f }),
	new Vertex(new float[] {  0.5f,  0.5f }, new float[] { 0.0f, 1.0f , 1.0f }),
};

My indices
private readonly int[] indices = new[]
{
	0, 1, 2
};

Everything draws correctly when using CmdDraw, but it wont draw properly (with the correct front face) when I use CmdDrawIndexed unless i reverse my indices.
Here is the full code:
Command Buffers
Initialize
Pipeline Creation
Vertex/Index Buffer Creation
I'm not sure why it's not working, I couldn't find any obvious errors with my code, so I decided to post here.

The indices array in your post differs from that in your code. Which one are you using?

The ones I posted, the ones in my code is the reversed order to try to make sense of what may be happening, but I was only able to get one triangle rendered with that method.

I think the problem is in this line:
commandBuffers[i].CmdBindIndexBuffer(indexBuffer, 0, IndexType.Uint16);
You are using an int array for indices. The IndexType should be Int32 in this case. You could also change the array to ushort[].


#62 Add File menu item to custom menu

Adds blank "File" menu item to the custom menu.
Submenu items and related functionality to be added through issues #63-#65.

Signed-off-by: John M. Horan<EMAIL_ADDRESS>

LGTM 👍


Simplify runtime/builder image update

Currently, some uses of the cilium-runtime and cilium-builder still need to be updated manually after updating the images (see https://docs.cilium.io/en/latest/contributing/development/images/#update-cilium-builder-and-cilium-runtime-images, step 9). This PR simplifies this process by changing the respective scripts in images/scripts/ to update these users as well. This way the manual update step is no longer necessary.
See individual commits for details.

test-me-please

test-runtime

PR #15327 was used to test the image update and verify the correct images are used (for both race and non-race CI jobs). Marking as ready for review.


No dependencies to install. No python 2.7 issue (probably)

OS Kali Linux 2016.2
on both (BeeLogger and Umbrella) there is a problem showed as:
wine: cannot find '/root/.wine/drive_c/Python27/python.exe'
Traceback (most recent call last):
File "./bee.py", line 184, in
main()
File "./bee.py", line 139, in main
os.rename('dist/k.exe', 'dist/' + name)
OSError: [Errno 2] No such file or directory
I tried many times to install/reinstall. As I found I have Python 3.4 sucessfully installed in .wine with VeilEvasion which makes the dependencies install automatically. On start of Umbrella and BeeLogger after selection of dependencies installation nothing happens. What to do? I really tried before asking.

@sisilana BeeLogger use Python 2.7 not 3.4, do you have installed dependencies correct without cancel ?

Try reclone now i think that are fixed.


Numerics.go validators are "undefined"

Getting "undefined" error when trying to use validators from the numerics.go.
I tried for:
IsNonNegative
InRange

Can you provide an example of usage that gives error?

Here you go:
package main

import (
	"fmt"
	validator "github.com/asaskevich/govalidator"
)

func main() {
	// Works
	fmt.Println(validator.IsNumeric("1"))

	// Generates compile error
	// undefined: govalidator.IsNegative
	fmt.Println(validator.IsNegative(-1))
}

Can you try to run go get -u github.com/asaskevich/govalidator and try again?

It's working now!


Russian translation: improvements of helper-array

Q
A

Is bugfix?
yes (doc)

New feature?
no

Breaks BC?
no

Tests pass?
--

Fixed issues
--

Merged. Thank you!


Issue while specifying resources in task

when we are adding resource in the task and trying to print w.to_yaml , we are having this issue

Could you please show us your code to reproduce this?

The error suggests that the line
resources=self.resources.build()

uses a list type for resources rather than a Resources object. Any chance the task accidentally received
Task(..., resources=[Resources(...), Resources(...)])

or something like that? The resources field expected a single Resources object

i have specified resources in this way

yes yes before i passed as lists, now passed as in object so it is working now properly.


Adds contributing guidelines with steps

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
docs update

Why was this change needed? (You can also link to an open issue here)
Fixed: #578

Other information:
This Pull Request aims to enhance our contributing guidelines and includes necessary code to improve the contributor experience. The updated guidelines provide clearer instructions and expectations for potential contributors, making it easier for them to participate effectively in this project. The code changes ensure the seamless integration of these guidelines into our repository.

@dartpain Please review the PR.

@dartpain Done :)

Thank you!

@dartpain Thank you too.


Tree view uses the default Date() format for journal notes where title is not wrapped in quotes

Please select the area your bug applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[ ] Workspace.  VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup.  Dendron's Lookup Command
[x] Views.  Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema.  Dendron Schemas
[ ] Pod.  Data import from / export to Dendron
[ ] Publish.  External Site Publish
[ ] Markdown.  Markdown features, such as syntax support and features

Describe the bug
If the title of a journal note doesn't have quotes (title: 2022-07-23), the tree renders the note's name using default Date() format:

To Reproduce
Steps to reproduce the behavior:

Create a journal note
Remove quotes from the title
Check the tree view to see the note's name change

Expected behavior
I expect the note's title in the tree view to stay the same regardless of the quotes around the title.
Desktop (please complete the following information)

Versions (all that apply to bug):

Dendron Extension: 0.104.1

Additional context
I noticed the above because GitJournal strips quotes from the title when editing notes.

Thanks @aleksey-rowan - I can repro.


Task20,21,22,23,24

20.Read the excel (consists of Lower Letters), and type into website (Vajrang.org - Uppercase Task) and get the result and write into one excel
21.Find the temperature of few cities and print the results into Excel
22.Read the excel (consists of Lower Letters), and type into website (Vajrang.org - Uppercase Task) (by using Add data row add the data into the excel)
23.Data Manipulation: Assign set of string (eg: abcdefg), and check the (length, substring, index, Last letters and Remove) 24.Assign input value (eg: UiPath Studio) , and replace the word with any other word by using Replace function

{
"NoFile":  [
{
"Your organization requires your project to have an Automation Hub URL defined.":  "Use "Project Settings" to add the missing information. "
},
{
"Dependency package UiPath.Mail.Activities is not used.":  "Remove unused packages in order to improve process execution time. "
}
],
"Main.xaml":  [
{
"Log Message not used in workflow Main.":  "Using Log Message activities is important both during development and after release. They help understanding the flow path which is very useful for maintenance and, when an error occurs, to easily identify the source and the cause. "
},
{
"WriteLine activity usage at Write Line.":  "Write Line activity can be replaced with tracepoints during debug. They should be removed before publishing. Consider also changing WriteLine to Log Message if it is really needed for debugging purposes. "
},
{
"Variable input doesn\u0027t have DataTable prefix.":  "DataTable Variable should start with a dt_ prefix. "
},
{
"Activity Write Line has a default name.":  "Check if it could be easier to understand what the activity does by adding a more descriptive title. "
},
{
"Activity Assign has a default name.":  "Check if it could be easier to understand what the activity does by adding a more descriptive title. "
}
]
}

{
"NoFile":  [
{
"Your organization requires your project to have an Automation Hub URL defined.":  "Use "Project Settings" to add the missing information. "
},
{
"Dependency package UiPath.Mail.Activities is not used.":  "Remove unused packages in order to improve process execution time. "
}
],
"Main.xaml":  [
{
"Log Message not used in workflow Main.":  "Using Log Message activities is important both during development and after release. They help understanding the flow path which is very useful for maintenance and, when an error occurs, to easily identify the source and the cause. "
},
{
"WriteLine activity usage at Write results in uipath.":  "Write Line activity can be replaced with tracepoints during debug. They should be removed before publishing. Consider also changing WriteLine to Log Message if it is really needed for debugging purposes. "
},
{
"Variable temp doesn\u0027t have DataTable prefix.":  "DataTable Variable should start with a dt_ prefix. "
}
]
}

{
"NoFile":  [
{
"Your organization requires your project to have an Automation Hub URL defined.":  "Use "Project Settings" to add the missing information. "
},
{
"Dependency package UiPath.Mail.Activities is not used.":  "Remove unused packages in order to improve process execution time. "
}
],
"Main.xaml":  [
{
"Log Message not used in workflow Main.":  "Using Log Message activities is important both during development and after release. They help understanding the flow path which is very useful for maintenance and, when an error occurs, to easily identify the source and the cause. "
},
{
"Variable index doesn\u0027t have DataTable prefix.":  "DataTable Variable should start with a dt_ prefix. "
},
{
"Variable build doesn\u0027t have DataTable prefix.":  "DataTable Variable should start with a dt_ prefix. "
},
{
"Activity Assign has a default name.":  "Check if it could be easier to understand what the activity does by adding a more descriptive title. "
}
]
}

{
"NoFile":  [
{
"Your organization requires your project to have an Automation Hub URL defined.":  "Use "Project Settings" to add the missing information. "
},
{
"Dependency package UiPath.Excel.Activities is not used.":  "Remove unused packages in order to improve process execution time. "
},
{
"Dependency package UiPath.Mail.Activities is not used.":  "Remove unused packages in order to improve process execution time. "
}
],
"Main.xaml":  [
{
"Log Message not used in workflow Main.":  "Using Log Message activities is important both during development and after release. They help understanding the flow path which is very useful for maintenance and, when an error occurs, to easily identify the source and the cause. "
},
{
"Variable firstval is not used.":  "Remove variables that are not used. "
},
{
"Variable value is not used.":  "Remove variables that are not used. "
},
{
"Variable replacevalue is not used.":  "Remove variables that are not used. "
},
{
"WriteLine activity usage at Displaying Output data in output panel.":  "Write Line activity can be replaced with tracepoints during debug. They should be removed before publishing. Consider also changing WriteLine to Log Message if it is really needed for debugging purposes. "
},
{
"WriteLine activity usage at Displaying string length.":  "Write Line activity can be replaced with tracepoints during debug. They should be removed before publishing. Consider also changing WriteLine to Log Message if it is really needed for debugging purposes. "
},
{
"WriteLine activity usage at Displaying index of string.":  "Write Line activity can be replaced with tracepoints during debug. They should be removed before publishing. Consider also changing WriteLine to Log Message if it is really needed for debugging purposes. "
},
{
"WriteLine activity usage at Remove value Output.":  "Write Line activity can be replaced with tracepoints during debug. They should be removed before publishing. Consider also changing WriteLine to Log Message if it is really needed for debugging purposes. "
},
{
"Sequence display name is defined many times. Current allowed threshold is 1. Code contains 2.":  "Names of activities should give a clear indication of how they are being used. Default names are usually too generic. Please avoid duplicate names. "
},
{
"Activity Comment Out has a default name.":  "Check if it could be easier to understand what the activity does by adding a more descriptive title. "
}
]
}

@Mohdabid161
Task -20
Code verified.
Task -21
Code verified.
Task -22
Code verified. Default activity name

Task -23
Code verified.
Task -24
Code verified.


Add EstakingWithdrawReward: Rewards per validator can be claimed

elysd query distribution rewards elys12tzylat4udvjj56uuhu3vj2n4vgp7cf9fwna9w elysvaloper12tzylat4udvjj56uuhu3vj2n4vgp7cf9pwcqcs
rewards:
- amount: "1128.960000000000000000"
  denom: ueden
- amount: "4515.840000000000000000"
  denom: uedenb
- amount: "88200.000000000000000000"
  denom: uelys

  elysd tx wasm exec --from validator --keyring-backend test --gas auto --gas-adjustment=1.3 --fees 100000uelys -b sync -y "elys1nc5tatafv6eyq7llkr2gv50ff9e22mnf70qgjlv737ktmt4eswrqau4f4q" '{"estaking_withdraw_reward": {"validator_address": "elysvaloper12tzylat4udvjj56uuhu3vj2n4vgp7cf9pwcqcs"}}'

rewards:
- amount: "3.528000000000000000"
  denom: ueden
- amount: "14.112000000000000000"
  denom: uedenb
- amount: "88200.000000000000000000"
  denom: uelys

Wondering why we have uelys rewards here

Executing fine on localnet w/ snapshot data:


Update README.md

Fix broken link
https://docs.dolthub.com/tutorials/installation/ -> https://docs.dolthub.com/getting-started/installation

Codecov Report

Merging #162 (2c4a95d) into master (be7f9bc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #162   +/-   ##
=======================================
  Coverage   42.63%   42.63%
=======================================
  Files          23       23
  Lines         971      971
=======================================
  Hits          414      414
  Misses        557      557

Continue to review full report at Codecov.

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


lndclient: use Features map instead the deprecated GlobalFeatures in the NodeUpdate

Note: first commit to be updated once LND master is tagged.

Should this be opened against the lnd-12-0 branch if it requires lnd v0.12.0?
We came up with the following plan for handling lnd compatibility:

master is always backward compatible with the last major version (so currently lnd v0.11.0)
we create branches for all minor versions and future major versions (there are the branches lnd-11-1 and lnd-12-0) and merge PRs to those branches, if the features require that version to work.
we rebase the branches if needed and use tags to track versions that we depend on in other projects

I'm not sure when exactly we should merge the branches back. Probably we could merge the lnd-11-1 as soon as lnd v0.12.0 is out of RC.
@carlaKC is that also how you remember our Keybase convo? Or did I remember something incorrectly?

Should this be opened against the lnd-12-0 branch if it requires lnd v0.12.0?
We came up with the following plan for handling lnd compatibility:
* master is always backward compatible with the last major version (so currently `lnd v0.11.0`)

* we create branches for all minor versions and future major versions (there are the branches `lnd-11-1` and `lnd-12-0`) and merge PRs to those branches, if the features **require** that version to work.

* we rebase the branches if needed and use tags to track versions that we depend on in other projects

I'm not sure when exactly we should merge the branches back. Probably we could merge the lnd-11-1 as soon as lnd v0.12.0 is out of RC.
@carlaKC is that also how you remember our Keybase convo? Or did I remember something incorrectly?

SGTM, I'll reopen this against lnd-12-0 just wasn't aware of this.

SGTM, I'll reopen this against lnd-12-0 just wasn't aware of this policy.

We should write it down somewhere, since it's only documented in a closed PR rn 😅
Ran into this the other day and it turns out you don't actually need to close and reopen, you can switch base branch.

Ran into this the other day and it turns out you don't actually need to close and reopen, you can switch base branch.

Oooooh, nice! TIL, thank you!


"/" route run twice!

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?
go1.6.2-2

What operating system and processor architecture are you using (go env)?
arch linux/amd64

What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.

package main
import (
"fmt"
"net/http"
)
func main() {
http.HandleFunc("/", indexRoute)
http.ListenAndServe(":8000", nil)
}
func indexRoute(w http.ResponseWriter, r *http.Request) {
fmt.Println("aa")
}

I've checked another routes but they were OK (like "/about")

What did you expect to see?
"aa"

What did you see instead?
"aa"
"aa"

This is your browser hitting /favicon.ico.
I am going to close this issue as it is not a bug. We don't use the issue tracker for asking questions , please see htttps://golang.org/wiki/Questions for questions.

Thank you


[dashboard][s #59]: Fix URL

Fixed URL Mapping

Coverage remained the same at 82.54% when pulling 06f5150557c6350ae7b3496fb61e38fcd5b26f84 on kittdotech:master into 15d3ff6a4d621f11614d82a12b4d8d7863ba6628 on frictionlessdata:master.


Beast options and resources in config

Currently only the dynamic variables are passed to beast from the config.
beast:
  - "ePopSize=0.3"
  - "Date.dateFormat=Y"
  - "Date.delimiter=|"
  - "mutationRate=1.0"
  - "mcmc.threads=6"
  - "mcmc.chainLength=10000"
  - "treelog.logEvery=1000"
  - "tracelog.logEvery=1000"

However we should also pass the cli options and resources e.g.
beast:
  beast:
     - "-beagle"
     - "-beagle_GPU"
  dynamic-variables:
      - "ePopSize=0.3"
      - "Date.dateFormat=Y"
      - "Date.delimiter=|"
      - "mutationRate=1.0"
      - "mcmc.threads=6"
      - "mcmc.chainLength=10000"
      - "treelog.logEvery=1000"
      - "tracelog.logEvery=1000"
   resources:     # Only used for submission to an HPC queue
    time: "20:00:00"
    mem: "4G"
    cpus: 1

Closed by #38


Generated function group maintenance view, #305

lets try this, just serialize the include, seems to work
the solution is not perfect as it looks like there are diffs for VARCL, but at least the syntax error is fixed
see https://github.com/larshp/abapGit/issues/305 and https://github.com/larshp/abapGit/issues/435

merging


Pfsync

Add special defined type for pfsync interfaces
bsd::network::interface::pfsync

This new type makes use of the parent interface of
bsd::network::interface, setting the value of the syncdev
parameter to the parent.

README.md updated, and spec tests added for the new defined
type and the corresponding hostname_if function.

This defined type is a bit special compared to the others,
since, when no parameters given, it explicitly sets the defaults

i.e. just create a pfsync0 interface, without parameters will end
up with a /etc/hostname.pfsync0 with:

-syncdev -syncpeer maxupd 128 -defer
up

What is default anyways is taken from the pfsync manual page.

This has the advantage that interfaces are easier to reconfigure.
Instead of ensuring its absense, and re-creating it, a parameter
can just be removed, and it will be reset to the default value.

This is a two commit PR, the second one enables Travis-CI runs for Puppet 3.8.3,
which is just available for OpenBSD 5.8-current.

Oh this is great!  Top to bottom an implementation of a new interface type!  This all looks good save for the couple minor notes I mentioned above.

both well spotted ;)
should be fixed now


packet size mismatch dual, 1262, 1256 when launch P40P

roslaunch hesai_lidar p40p.launch
Crashed directly after printing multiple lines of
packet size mismatch dual, 1262, 1256

[hesai_lidar-2] process has died [pid 5515, exit code -11, cmd /home/dev/ros_ws/devel/lib/hesai_lidar/hesai_lidar_node __name:=hesai_lidar __log:=/home/dev/.ros/log/4907e4fc-f863-11e8-9149-7085c2535d50/hesai_lidar-2.log].
log file: /home/dev/.ros/log/4907e4fc-f863-11e8-9149-7085c2535d50/hesai_lidar-2*.log

Please use: https://github.com/HesaiTechnology/Pandar40p_ros

@willdzeng


Allow the server to start without the database

Closes #152
Tasks done:

Fix .gitignore and build.sbt

Refactor existing layers

Remove schema creation from SlickItemRepository

Integrate Flyway with ZIO

Add auto-migration of DB schema at app startup

Make package generic

Fix build.sbt formatting

Why would you want the service to run without the database?


[release-1.19] Gateway API cherrypicks

This PR cherry picks three fixes for gateway api from master that didn't have a chance to reach 1.19.0.
Closes #46932.

@jwendell: 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

release-notes_istio_release-1.19
3fc4e321dd52da84aebf1ac1a4f2b0017207a135
link
true
/test release-notes

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.

/test release-notes


Amazing new feature

Please pull this in!

Codecov Report

Merging #4 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master     #4   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          14     14
  Lines         615    615
=====================================
  Hits          615    615

Continue to review full report at Codecov.

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


add postMessage to oauth

for https://github.com/RoamJS/query-builder/pull/248

@dvargas92495 https://www.loom.com/share/ef9c4c5367d9465ab28c46e944bb9821


maxconnection=x is ignored or wrong checked

With version 1.1.3 "maxconnections=20" don´t connect to the network when maxconnections is smaler then 75. By using Version 1.1.4 i can set maxconnection=25 - but in reality it has more then 50 connections and still counting. So the maxconnection-settings is borked and ignored in the last two releases.

Use it in conjunction with listen=0 and see if that works.

in my case i like to use it as server/daemon, not as client only.
So i think it must work in client AND server/daemon-mode ?

Is this still the case with version v1.1.5?

maxconnections is generelly working again, but the behavior is not the same as by other daemons:
setting maxconnections=25 used at least 26-28 connection on windows or linux.
Normaly 10 connections are reserved typicly by using this setting. So my normal behavior
is by setting maxconnections=25, that the daemon uses between 8-15 connection avg.
10 connections are usuable reserved for new incomming and outgoing connections.

Ahhh, I see. That is good information to know. Expect a changeset to match other daemon's behavior for v<IP_ADDRESS>


Question: How to keep WindowTitleBrush Background Color after loosing focus

Hallo i have a question. Is it possible to keep WindowTitleBrush Background Color after loosing focus ?
i have project a window, where i can show new metrowindow, but if i show new Metrowindow mainwindow looses WindowTitleBrush Background Color and switch it to system color. Current i set color ot Titlebrush WindowTitleBrush="#1BBC9B" like that, but after looses it going to be gray or current windows system color. How can i define or programm my color ?

@Torchok19081986 You can set the NonActiveWindowTitleBrush property to the same brush
https://github.com/MahApps/MahApps.Metro/blob/b7348bd42bf34a34a52e1613e62a486b6b54850f/src/MahApps.Metro/Controls/MetroWindow.cs#L101


[parametric] Ensure W3C tracestate propagation always occurs when it is a configured propagator

Description
Implements system-tests for the deliverable: Other vendors' TraceState support.
The tests assert that the W3C tracestate header is always propagated when the following conditions are met:

The tracecontext propagator is configured (either by default or by the user)
The trace-id in the traceparent request header matches the trace-id of the extracted trace context.

This covers the edge case where another propagator is configured to run first (e.g. Datadog when the configuration is set to Datadog,tracecontext) and the Datadog propagator extracts a trace context. In this situation, the tracestate will be propagated if the trace-id extracted by the tracecontext propagator matches the trace-id extracted by the Datadog propagator.
Motivation
We want to interoperate with other systems that use W3C Trace Context, so we must passthrough the tracestate header when we're continuing an incoming traceparent trace context.
Workflow

⚠⚠ Create your PR as draft
Follow the style guidelines of this project (See how to easily lint the code)
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Once your PR is reviewed, you can merge it! :heart:
Reviewer checklist

[ ] Check what scenarios are modified. If needed, add the relevant label (run-parametric-scenario, run-profiling-scenario...). If this PR modifies any system-tests internal, then add the run-all-scenarios label (more info).
[ ] CI is green

[ ] If not, failing jobs are not related to this change (and you are 100% sure about this statement)

[ ] if any of build-some-image label is present

is the image labl have been updated ?
just before merging, locally build and push the image to hub.docker.com

[ ] if a scenario is added (or removed), add (or remove) it in system-test-dasboard nightly

There are some tests failures

Hi Roberto, I've fixed the failing parametric tests but there is one unrelated test run that is failing

Per peer feedback, I've cherry-picked commits 83a0fdbe6164e451786c09515bffa920c84421c9 through a0897ccee3d9651661b07c9c035260b9117ce557 from https://github.com/DataDog/system-tests/pull/1807 so that we are all working on one PR for these deliverables. @robertomonteromiguel @cbeauchesne please feel free to review again if necessary.

Per peer feedback, I've cherry-picked commits https://github.com/DataDog/system-tests/commit/83a0fdbe6164e451786c09515bffa920c84421c9 through https://github.com/DataDog/system-tests/commit/a0897ccee3d9651661b07c9c035260b9117ce557 from https://github.com/DataDog/system-tests/pull/1807 so that we are all working on one PR for these deliverables

That's way better, thanks! 👍
But I still think we need some dedicate branch and automation to have a consolidated branch to test agains -- againt it might be too late for the re:invent thing.
For example, your branch does not seem to be rebased on main so lakes the 128-bit tid generation by default (#1814) check and won't include span link test from @bwoebi (#1811)


Update-Dbainstance

This version: 12.0.5557
trying to install this version: https://support.microsoft.com/en-us/help/4482967/cumulative-update-16-for-sql-server-2014-sp2
Works when manually installing, fails on Update-DbaIntance with error
internal\functions\Get-SqlServerUpdate.ps1:117:                            Stop-Function -Message "Current SP version $currentMajorVersion$currentSP is not the latest
available. Make sure to upgade to latest SP level before applying latest CU." -Continue

this should work, eh? Please note that upgrade is also a typo.

any chance for the -Debug output? I don't have any 2014s around :(

dang no :( can it be faked?

what was the original version? I'll try to mock something up

that was it - 12.0.5557 then i was trynna update to CU 16

oh okay, I gotchu

Mocked the exact scenario and it works when targeting CU16 specifically. Did you run it simply as -Type CumulativeUpdate without specifying a version? Could you provide a command line?

yes my apologies, i did indeed specify a version. I do wish i could have copy/pasted but i couldnt. i will close this as non repro and get much more info if it happens again. thank you for looking into it.


Generate without Card Backs

I want to generate and print my cards without the card backs on them (I don't really want them, so printing them would be a waste of expensive ink). I know I can set the printer up to only print specific pages, in this case the ones that have the card fronts on them, but for a lot of pages this is a pain. Is there anyway I can generate a PDF without the backs at all?

There's a setting for this: Page settings -> Arrangement -> Front side only


Initial work for scoverage 2.0

scoverage 2.x makes a few changes to the structure of the scoverage project, including the artifacts, and also will allow for Scala 3 support once https://github.com/lampepfl/dotty/pull/13880 gets merged. I wasn't super familiar with how mill was using it, so this was just me learning a bit and updating some of the things I saw. However, it introduces a few questions that I'd love to get your thoughts on.
This is a breaking change that won't easily allow for users to use 1.x anymore. Are you ok with that? Once scoverage bumps to 2.x it will be pretty much impossible to support both 1.x and 2.x for integrations since the structure is quite different, even some of the package names, method signatures etc. In the sbt plugin once I officially bump I'll error out if someone manually tries to use 1.x in the 2.x series of the plugin. What are you thoughts on something like that in Mill?
I'll ask the other question in the parts of the code to make them clearer.
I'll leave this as a draft for now but will iterate on it since there is a few things we'll need to update if you're ok with the 2.x approach listed above.

[ ] Add some messages to make it clear to the user that once on 2.x they need to use 2.x
[ ] Add in the logic to determine if the user is using Scala 2 or 3. If 3, then just set the -coverage flag and the datadir
[ ] Add some stuff to the tests to cover both the Scala 2 and Scala 3 usecase

This is a breaking change that won't easily allow for users to use 1.x anymore. Are you ok with that? Once scoverage bumps to 2.x it will be pretty much impossible to support both 1.x and 2.x for integrations since the structure is quite different, even some of the package names, method signatures etc. In the sbt plugin once I officially bump I'll error out if someone manually tries to use 1.x in the 2.x series of the plugin. What are you thoughts on something like that in Mill?

There is a major difference between the sbt plugin for Scoverage and this mill plugin. The sbt plugin is external to sbt and users can probably use the older versions supporting scoverage 1.x for a very long time. As our scoverage plugin is developed in tree, we in general only support compatibility to the exact same mill version. (This is different for plugins developed outside of the tree.)
So my question is, do we loose something with Scoverage 2.x? E.g. support for older Scala versions? If that should be the case (which I assume to be true), I'd say we should try a bit harder to keep support for Scoverage 1.x in the plugin. We already load scoverage jars in the worker, so it should be pretty easy to have some logic switch and use a different worker module depending on the actual used Scoverage version.

So my question is, do we loose something with Scoverage 2.x? E.g. support for older Scala versions?

Yea, this is exactly the case. In the 2.x series we'll no longer publish for 2.11.

@ckipp01 Let me know if you need further guidance in creating a new worker module for Scoverage 2.x. Or just ping me in our Gitter channel.

@ckipp01 Let me know if you need further guidance in creating a new worker module for Scoverage 2.x. Or just ping me in our Gitter channel.

Sounds good. It'll probably be a bit before I get around to it. Once I get the green light on the direction in the pr to Dotty I'll probably swing back around to this, but I'll let you know.

@ckipp01 Are you planning to get back to this? I may be interested in picking it up.

@ckipp01 Are you planning to get back to this? I may be interested in picking it up.

Hey @kierendavies! I was planning on it, but no idea if it'd be soon or not. Seeing that scoverage 2.0.0 is release, by all means pick this up if you'd like. Feel free to ping me to review the scoverage parts or anything and I'll be happy to give some insight. If you do pick it up, just let me know so we don't duplicate work.

Is anybody working on this? @ckipp01 or @kierendavies ?

Not me. I don't really have any plans in the short-term to get back to this.

As there is nobody currently working on this and we decided to keep the old worker but also as there are merge conflicts in this PR, I decided to re-start this work in a new branch and pull request:

#2010

I heavily used this PR to identify the required changes. Thank you Chris (@ckipp01) for creating this PR!
Closing this PR as it is superseded by #2010.


Add GammaMM notebook

This PR adds an example of inference in the Gamma Mixture model.

Codecov Report

Merging #81 (74ec87c) into master (4a7f1ad) will decrease coverage by 1.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
- Coverage   74.52%   73.46%   -1.07%
==========================================
  Files         172      170       -2
  Lines        5449     5253     -196
==========================================
- Hits         4061     3859     -202
- Misses       1388     1394       +6

Impacted Files
Coverage Δ

src/macrohelpers.jl
58.69% <0.00%> (-19.31%)
:arrow_down:

src/ReactiveMP.jl
50.00% <0.00%> (-16.67%)
:arrow_down:

src/rule.jl
54.62% <0.00%> (-8.79%)
:arrow_down:

src/score.jl
88.57% <0.00%> (-7.09%)
:arrow_down:

src/approximations/gausshermite.jl
37.50% <0.00%> (-6.95%)
:arrow_down:

src/nodes/bifm.jl
72.72% <0.00%> (-5.33%)
:arrow_down:

src/algebra/companion_matrix.jl
42.10% <0.00%> (-3.35%)
:arrow_down:

src/variables/constant.jl
47.22% <0.00%> (-2.78%)
:arrow_down:

src/variables/data.jl
58.13% <0.00%> (-2.74%)
:arrow_down:

src/rocket.jl
76.27% <0.00%> (-1.86%)
:arrow_down:

... and 32 more

Continue to review full report at Codecov.

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

as_constraint and bs_constraint variables in the model specification are not used - it is probably better to remove them. Other than that IMO is ready to merge.