RFC: Repository

Summary
This is an "internal" interface that should be used by Store / Model implementors.
Motivation
Now that we landed IdentityMap I would like to discuss the path forward. I believe the next step should be an interface that would provide a building block for Store / Model implementors and so the proposed interface, Repository, is not meant as a public interface but as a primitive to build on. It is heavily inspired by lots of methods existing in ember-orbit. It also takes some inspiration from ember-data's RecordData interface.
Detailed design
All write operations are async. Read operations on relationships have both sync and async versions.
import { Schema, QueryOrExpression } from '@orbit/data';
import MemorySource from '@orbit/memory';
import IdentityMap from '@orbit/identity-map';

export interface Factory<Identity, Model> {
  create(identity: Identity, repository: Repository): Model;
}

export interface RepositorySettings<Identity, Model> {
  source: MemorySource;
  factory: Factory<Identity, Model>;
  identityMap?: IdentityMap<Identity, Model>;
}

export interface Repository<Identity, Model> {
  schema: Schema;

  // Attribute
  getAttr(identity: Identity, attribute: string): unknown;
  setAttr(identity: Identity, attribute: string, value: unknown): Promise<void>;

  // HasMany Relationship
  getHasMany(identity: Identity, relationship: string): Model[];
  asyncGetHasMany(identity: Identity, relationship: string): Promise<Model[]>;

  addToHasMany(identity: Identity, relationship: string, record: Identity): Promise<void>;
  removeFromHasMany(identity: Identity, relationship: string, record: Identity): Promise<void>;
  setHasMany(identity: Identity, relationship: string, records: Identity[]): Promise<void>;

  // HasOne Relationship
  getHasOne(identity: Identity, relationship: string): Model | null;
  asyncGetHasOne(identity: Identity, relationship: string): Promise<Model | null>;

  setHasOne(identity: Identity, relationship: string, record: Identity): Promise<void>;

  // Unload / Remove
  unloadRecord(identity: Identity): void;
  removeRecord(identity: Identity): Promise<void>;

  // Query
  query(queryOrExpression: QueryOrExpression, options?: object, id?: string): Model[] | Model | null;
  asyncQuery(queryOrExpression: QueryOrExpression, options?: object, id?: string): Promise<Model[] | Model | null>;
  liveQuery(queryOrExpression: QueryOrExpression, options?: object, id?: string): Observable<Model[] | Model | null>;
}

export interface Observable<T> {
  subscribe(callbackFn: (value: T) => void): Subscription;
}

export interface Subscription {
  unsubscribe(): void;
}

Drawbacks
Probably the most controversial part of the proposal is the Observable  interface for liveQuery. I still strongly believe that in this case Observable is the right primitive. It is what is used in Apollo for live queries for example.
Alternatives
Leave all the work to each Store / Model implementation.

I think another approach would be to implement identityMap aware finders on the store and pass the store to records (similar to how ember-orbit is implemented today)


Social Media Fields Added

These need to be global settings

@johnlewissims
I am still not seeing social media icons on articles. Is it an asset problem or something that you need to add in?

I imagined these links being "share" links. Like they integrate with the platforms to share the article. If this is too much then we can just kill it and maybe its a "copy" link button. If that is too much than I can figure out a design solution.

I bet that the assets are missing. See if this asset loads on your machine...
/wp-content/uploads/2022/07/facebook2x.png
@brendon-avalos

@johnlewissims I got them to load. Let me know how you feel about making them "share" page links instead of just links to overall social media.

Ahhh now I see the issue. Ok, added that to the most recent commit


Getting error: cannot communicate with server: Post http://localhost/v2/snaps/amass:

I'm getting this error when trying to install amass through snap

Found the solution. If you are using kali run systemctl start snapd.service after installing snapd


CRI v1alpha2

For https://github.com/kubernetes-incubator/cri-tools/issues/236.
Update CRI to v1alpha2.

/lgtm

Will merge after test passes.


fix(unplugin-vue-i18n): check for vue compiler in directive transform hook

resolves #420

closing this, it is a bug in @vitejs/plugin-vue, compiler not being initialized correctly: https://github.com/vitejs/vite-plugin-vue/pull/475

@kazupon @BobbieGoede looks like Vite Vue plugin PR will took some time, I guess we can use the protection in this PR to avoid the error in Nuxt calling Vite warmup.

I agree we probably need to skip transform if the compiler is unset, it seems like both devtools (Overlay.vue) and nuxt (nuxt-root.vue) trigger this in some sort of race condition. I'm not sure if it's something we should/can fix at their end.


[INFINITY-2713] Update the JRE and JCE to 1.8_152

I noticed a few more references to JRE/JCE artifacts in test config files and documentation, does it matter if those aren't changed?
./cli/commands/testdata/responses/cosmos/1.10/enterprise/describe-no-package-versions.json:152:          "jre-tar-gz": "https://downloads.mesosphere.com/java/jre-8u131-linux-x64.tar.gz",
./cli/commands/testdata/responses/cosmos/1.10/enterprise/describe.json:152:          "jre-tar-gz": "https://downloads.mesosphere.com/java/jre-8u131-linux-x64.tar.gz",
./cli/commands/testdata/responses/cosmos/1.10/enterprise/update.json:152:          "jre-tar-gz": "https://downloads.mesosphere.com/java/jre-8u131-linux-x64.tar.gz",
./cli/commands/testdata/responses/cosmos/1.10/open/describe.json:152:          "jre-tar-gz": "https://downloads.mesosphere.com/java/jre-8u131-linux-x64.tar.gz",
./frameworks/hdfs/docs/api-reference.md:462:					value: "https://downloads.mesosphere.com/java/jre-8u112-linux-x64-jce-unlimited.tar.gz",
./frameworks/hdfs/docs/managing.md:755:              "value": "https://downloads.mesosphere.com/java/jre-8u112-linux-x64-jce-unlimited.tar.gz",
./frameworks/kafka/docs/kerberos.md:175:    wget https://downloads.mesosphere.com/java/jre-8u131-linux-x64-jce-unlimited.tar.gz && \
./frameworks/kafka/docs/kerberos.md:176:    tar -xzvf jre-8u131-linux-x64-jce-unlimited.tar.gz && \
./frameworks/kafka/docs/kerberos.md:177:    rm jre-8u131-linux-x64-jce-unlimited.tar.gz

@mpereira Good catch.
@elezar why are we referencing a JRE in the kerb docs?
@NimaVaziri why are we referencing a JRE in those two HDFS docs?

The reference in the Kerberos docs are because the standard docker image described does not include the JCE and it needs to be downloaded.

The references to the version have been moved from the Kafka kerberos docs.

@elezar plz spawn a ticket to investigate that.

Done.

@elezar But what ticket tho


java example missing

As java examples missing
https://www.pulumi.com/registry/packages/rancher2/#example
not sure if pulumi supports rancher with java, can you pl. support

Yes, Java should be supported already. It looks like something went wrong with the release process for those docs, so I've added this to the backlog to get fixed.


Critical source map generation issues in 2.18.16

OS: MacOS
Scout-App Version: 2.18.16
I discovered Scout-App today, and it's great! It does exactly what I need, and nothing else, which is more than I can say for my previous automatic SASS compiler. However, there is one rather serious problem, one somewhat odd problem, and one related request I'd like to make.

The source map filename is generated incorrectly, which breaks the map entirely. Scout-App will generate airspace.css and airspace.map, but then refer to the map within the CSS file as /*# sourceMappingURL=airspace.css.map */. Scout-App should be generating the map's filename as airspace.css.map.

Those comments that Scout-App generates in Development mode on every single rule are being generated wrong. Here's an example of what I get in v2.18.16:

/* line 238, ../../../../../Users/<username-redacted>/dev/workspace/wagtail-airspace/airspace/static/airspace/scss/_mixins.scss */

There are two things wrong, here. First, that uses .. to go all the way up to the root of my filesystem, then digs back down into the same folder from which it started. That's probably an os-related pathname bug. Second, the URLs for the referenced files should be as short and relative as possible, so that they're not specific to the development machine of any single dev on a project. Assuming these comments are even useful (isn't this what the map file is for?), they should look like this:
/* line 238, ../scss/_mixins.scss */

Please add the /*# sourceMappingURL=airspace.css.map */ line to the Production mode output. It's an utterly trivial number of bytes, but it allows developers to use the production CSS in development without losing the map functionality. This is important for developers who use Docker, as we generate the image file for our docker instances with the intent to use the same exact files for both dev and prod.

@coredumperror

Bug: Super easy fix and I can't believe it's been like that for so long. It's just changing the .map to .css.map on line 134 of scout-files/_scripts/app.js. So you could modify that locally in your version of Scout-App.

https://github.com/scout-app/scout-app/commit/79f4d83491f3206d16618a570e4143e2ad3933bb?diff=unified#diff-bb96692b2139a473d0c4345ef340cb0aR134
(this code has already been merged to master)

Bug: This is caused by having the context of where the "current working directory" based on where Scout-App.exe is running from. I've played around with this some and can't find any simple fix for this (other than major rewrites to Scout-App). So my approach here is to change the general settings that are passed to Node-Sass. The new settings will just not create a comment above every CSS Rule (since they have the problem you have highlighted). Instead we'll keep the comment at the end of the file and link to a .css.map file next to the .css. The map file will include a copy of all referenced source files so the browser should still be able to map the rules to the original files. The map file will also include the paths to all the source files, but these will be relative to the project folder.

(Code changes for this can be seen in the same link above).

Feature Request: There used to be a restriction where source maps could not be created unless you were in nested or expanded mode. This is no longer the case. I'd like to change this so that there is a checkbox to enabled outputting source maps (enabled by default) and the dropdown for output style that always shows all 4 options. Then remove the Dev/Prod options for environment. This will require a decent amount of code changes and will also require some new text to be translated to the other culture codes (french, german, etc.).

(No code changes for this yet, will take more time)

Glad to see a prompt response! I'll see if I can figure out how to build Scout-App from the master branch, so I can share the fix with my colleagues.

Running Scout-App locally for development

@coredumperror
All bugs fixed. Feature request implemented into master branch. Will be in the next release of Scout-App.

Code changes: https://github.com/scout-app/scout-app/compare/79f4d83491f3206d16618a570e4143e2ad3933bb...55eff28169d8db6cfaeafa1ed237fea50be91c9c

Awesome, thanks!

@coredumperror

Bug: Super easy fix and I can't believe it's been like that for so long. It's just changing the .map to .css.map on line 134 of scout-files/_scripts/app.js. So you could modify that locally in your version of Scout-App.

79f4d83?diff=unified#diff-bb96692b2139a473d0c4345ef340cb0aR134
(this code has already been merged to master)

Bug: This is caused by having the context of where the "current working directory" based on where Scout-App.exe is running from. I've played around with this some and can't find any simple fix for this (other than major rewrites to Scout-App). So my approach here is to change the general settings that are passed to Node-Sass. The new settings will just not create a comment above every CSS Rule (since they have the problem you have highlighted). Instead we'll keep the comment at the end of the file and link to a .css.map file next to the .css. The map file will include a copy of all referenced source files so the browser should still be able to map the rules to the original files. The map file will also include the paths to all the source files, but these will be relative to the project folder.

(Code changes for this can be seen in the same link above).

Feature Request: There used to be a restriction where source maps could not be created unless you were in nested or expanded mode. This is no longer the case. I'd like to change this so that there is a checkbox to enabled outputting source maps (enabled by default) and the dropdown for output style that always shows all 4 options. Then remove the Dev/Prod options for environment. This will require a decent amount of code changes and will also require some new text to be translated to the other culture codes (french, german, etc.).

(No code changes for this yet, will take more time)

Thank you so much for pointing out the bug.. I am really looking forward to use Scout as my default compiler but I was really confused when I couldn't see the correct map in inspection


Release of version <IP_ADDRESS> of xExchange

Releasing version <IP_ADDRESS>

This change is

Codecov Report

Merging #422 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #422   +/-   ##
=======================================
  Coverage   43.33%   43.33%
=======================================
  Files          40       40
  Lines        4938     4938
=======================================
  Hits         2140     2140
  Misses       2798     2798

Impacted Files
Coverage Δ

...chTransportService/MSFT_xExchTransportService.psm1
100% <ø> (ø)
:arrow_up:

...ortService/MSFT_xExchFrontendTransportService.psm1
100% <100%> (ø)
:arrow_up:

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 40c7920...1ff1dda. Read the comment docs.


CSG geometry bug

Godot version:
Godot 3.2 alpha3
OS/device including version:
Win7 Nvidia
Issue description:
Geometry (or rendering?) bug showing visible faces with two CSG boxes, the second child one in subtraction operation. There should be no diagonal faces visible, since this area is "subtracted" by the second child CSG box.
Steps to reproduce:
Add CSG box node, change height, width and depth to your liking,
Add another CSG box as child of the first, set Operation to "Subtraction", change height, width and depth to see the bug (visible diagonal faces).

Does this happen in earlier versions of Godot as well?

For the record, this seems different than #21125.

@name-here Yes, can confirm in 3.1 as well. This bug appears when the diagonal edges of both boxes align.


pragma support

I find this helpful for switching between React or Inferno during development.

@mstijak does this even work with React?

@trueadm
Let me explain how this works. In my app there is a large number of components. Each component uses JSX in the render method and that gets translated into React.createElement or Inferno.createBlueprint calls. Switching from React to Inferno requires changing import statements on top of each component file, which is a real pain. React already supports pragma option which allows you to replace React.createElement calls with something else. In this pull request I added the same for Inferno and now it's possible to set that both target the same name.
["babel-plugin-inferno", { "pragma" : "Lib" }],
or
["babel-plugin-transform-react-jsx", { "pragma": "VDOM.createElement" }],

React.createElement -> Lib.createElement
React.Component -> Lib.Component
Inferno.createBlueprint -> Lib.createBlueprint
Inferno.Component -> Lib.Component

On top of each file
import Lib from './Lib'
//import React from 'react'; //not required anymore
//import Inferno from 'inferno'; //not required anymore

And Lib.js
export React as Lib from 'react';
//or
export Inferno as Lib from 'inferno';
// some details missing here

That's the general picture. Some details are missing.
If there is a better way I would be very glad to hear about it.

@mstijak makes sense! :) merged

cool feature :+1:


Uncaught TypeError: Cannot read properties of undefined (reading 'errorTextureDatas')     at _MmdAsyncTextureLoader._addErrorTextureReferenceCount

米雪儿炼狱_by_卡拉彼丘_13e60d28cf440b5c753247fdea0f393b.zip

Related to issue #9. I will release the 0.23.0 version now, so please test it again

Related to issue #9. I will release the 0.23.0 version now, so please test it again

0.23.0 resolved.


Connect faces with database or datasheet to recognize.

could you please provide a sample of the functionality you wish to include in the project

Functionality like Connect name database with a particular person so when they are  in front of camera again there name shows , i am talking about multiple faces

Please assign me this issue ?

ok


Detect duplicate genus

Moved the two files for detecting duplicate genus records from the AttachmentLinks branch into their own branch.

No files from this branch should be merged to dev (they both have been condensed to be included in DuplicateSearch.py and test_DuplicateSearch.py  from the Duplicate Search branch)


Including icon Fails with "could not find attachment"

When attempting to publish a file that contained  as csv file I had errors due to the attachment not found.
I'm running through docker, only way it succeeded was when I stopped including the csv file as include and pasted it in directly into the table contents.
CSV Format looked like
"Team","Member or Group"
"Team1","icon:user[] BurritoBoy"
"Team1","icon:user[] TacoMan"

I tried to ensure it had underscores in the name, but it still failed, I've also tried adjusting with ./_file and include::_file to ensure path issues weren't the problem.
[format="csv",options="header",cols="20,40",seperator=,]
[caption="Response Team "]
.members
|===================================================
include::./_response-team.csv[]
|===================================================

Exception in thread "main" java.lang.RuntimeException: Could not find attachment
        at org.sahli.asciidoc.confluence.publisher.client.ConfluencePublisher.fileInputStream(ConfluencePublisher.java:254)
        at org.sahli.asciidoc.confluence.publisher.client.ConfluencePublisher.addOrUpdateAttachment(ConfluencePublisher.java:197)
        at org.sahli.asciidoc.confluence.publisher.client.ConfluencePublisher.lambda$addAttachments$9(ConfluencePublisher.java:192)
        at java.util.HashMap.forEach(HashMap.java:1289)
        at org.sahli.asciidoc.confluence.publisher.client.ConfluencePublisher.addAttachments(ConfluencePublisher.java:192)
        at org.sahli.asciidoc.confluence.publisher.client.ConfluencePublisher.lambda$startPublishingUnderAncestorId$1(ConfluencePublisher.java:127)
        at java.util.ArrayList.forEach(ArrayList.java:1257)
        at org.sahli.asciidoc.confluence.publisher.client.ConfluencePublisher.startPublishingUnderAncestorId(ConfluencePublisher.java:123)
        at org.sahli.asciidoc.confluence.publisher.client.ConfluencePublisher.publish(ConfluencePublisher.java:80)
        at org.sahli.asciidoc.confluence.publisher.cli.AsciidocConfluencePublisherCommandLineClient.main(AsciidocConfluencePublisherCommandLineClient.java:87)
Caused by: java.io.FileNotFoundException: /tmp/confluence-publisher1621229107102861561/assets/3f70f6e9d48eba115943f9083b309afbdedcb9b30c285b6e472c765f760fd28e/user (No such file or directory)
        at java.io.FileInputStream.open0(Native Method)
        at java.io.FileInputStream.open(FileInputStream.java:195)
        at java.io.FileInputStream.<init>(FileInputStream.java:138)
        at org.sahli.a

Update to the issue, i found it it's failing due to the icon:user[]. If I set to :icon: font  it's supposed to use font-awesome icons, but apparently these are failing right now as it's trying to attach them.
Is there any fix for this?

Hi @sheldonhull, thank your for your issue. Icons are currently not yet supported by the Confluence Publisher. Unfortunately, the renderer is not distinguishing between an icon and an image and therefore renders an attachment link for icons, too. Subsequently, the upload fails as the attachment does not exist as a file.
I will have a look into it asap to see how to at least ignore icon directives so that the publishing does no longer fail (but then will ignore the icon). Do you want to open a feature request for actually supporting icons? Not sure though whether that would be feasible within Confluence.

I can live without them. It just adds some nice eye candy to the presentation to make it look better. Would it be possible to include in confluence by rendered the png or svg in the conversion process to an embedded image instead? I could open a request for that.

I think if feasible, icons can be a nice feature.
+1 from me :)

Has there been any progress on this subject? I'm encountering that precise bug and can work on it (but for that, I have to know if there is already some work done)

Hi @Riduidel, thank you for your support, a pull request would be highly appreciated. At least me, I am not currently working on this. Thank you!

Hi, there! Is there any solution for the issue with font icons?
I'll be grateful for any answer, thank you.

AAACLg0ODAoPeNqNVEtv4jAQvudXRNpbpUSEx6FIOQBxW3ZZiCB0V1WllXEG8DbYke3A8u/XdUgVQ
yg9ZvLN+HuM/e1BUHdGlNvuuEHQ73X73Y4bR4nbbgU9ZwFiD2IchcPH+8T7vXzuej9eXp68YSv45
UwoASYhOeYwxTsIE7RIxtNHhwh+SP3a33D0XnntuxHsIeM5CIdwtvYxUXQPoRIF6KaC0FUGVlEB3
v0hOAOWYiH9abFbgZith3i34nwOO65gsAGmZBhUbNC/nIpjhBWEcefJWelzqIDPWz/OtjmXRYv2X
yqwnwueFkT57x8e4cLmbCD1QnX0UoKQoRc4EUgiaK4oZ2ECUrlZeay75sLNs2JDmZtWR8oPCfWZG
wHAtjzXgIo0SqmZiKYJmsfz8QI5aI+zApuq6fqJKVPAMCPnNpk4LPW6kBWgkZb+kQAzzzS2g6Dnt
e69Tqvsr4SOskIqEFOeggz1v4zrHbr0yLJR8rU64FpQpVtBy1mZxM4CnHC9Faf8tKMnTF1AiXORF
ixyQaWto3RZ+ncWLXtMg6EnKZZRpmQNb2R8tnJXFulCfXmXLry7TrHBWn2HNVyH8WYxj9AzmsxiN
L/R88Xg6rA1lVs4QpO5titxhplJcCY2mFFZLutAZVhKipm15/VhJx36YVqyN8YP7IaGC1+lwnJ7Q
5pJpNmxk5hP3qovutY8Pi4E2WIJ59esnr1p+T6eD67teBVCHf+ga+ho4/4D9YItZDAsAhQ5qQ6pA
SJ+SA7YG9zthbLxRoBBEwIURQr5Zy1B8PonepyLz3UhL7kMVEs=X02q6


Refs #29784 -- Switched to https:// links where available.

Changed Python f-string link to use intersphinx.
Normalized docs.python.org URLs to omit the version.
https://code.djangoproject.com/ticket/29784

I've done this same change on a number of different repositories now. I automated this change using the script:
https://github.com/jdufresne/commit-messages/blob/master/scripts/checkhttp.py
At a high level, this script does the following:

Scans all text files for strings that look like a URL starting with http://
Tries fetching the same URL but with https://
Check the final destination is a 200 response with protocol https:// (to avoid redirects back to http://)
If it looks good, replace all occurrences

After running the script, I analyze the results locally using a visual diff tool (I like meld). I undo the changes for tests that look to be intentionally using an http:// URL (Django has a lot of these). I also try to remove en/latest/ paths from readthedocs that didn't previously exist. I manually test a bunch of URLs especially any that look suspicious with a wildly different path. I re-analyze the results using GitHub's diff tool (I find looking at a diff in multiple ways can help emphasize different differences).
There are classes of URLs we can immediately be confident with:

https://docs.djangoproject.com
https://code.djangoproject.com
https://docs.python.org
https://bugs.python.org
https://blog.python.org
https://github.com
https://www.sqlite.org
https://stackoverflow.com

Perhaps some others too.
So, in summary, a script has confirmed that all these changes result in https:// URLs that 200 and several have been manually tested.
Please feel free to re-run the script to verify or do additional manual testing.

Thanks @jdufresne. Good explanation. Sounds fine. (I will give it a run and review.)
@IgnatiusFlo I mainly CC-d you here so you knew what was coming and didn't repeat effort. If you focus on the remainder of #29784 when you have time, that would be super. Thanks!

Can the last commit message start with 'Fixed'?

This PR only fixes one aspect of the ticket. The ticket is asking for a full audit of the external links on the settings docs. Links using https:// is just one aspect of this audit. From the ticket, see the text:

Some, like https://www.owasp.org/index.php/HTTPOnly should be replaced with a link to more up to date information.

This PR isn't addressing that concern. So I think changing this to "Fixed" would close the ticket prematurely and skip over some intended changes desired by the ticket.
I decided to handle https:// links separately because it is a global change, not localized to just one page of the docs.

Rebased.

Of course, sorry for my poor comment.
However, I'm not sure about the feasability of a global audit. It may be a rather huge task, and wonder if we should ask the reporter to open tickets for the links he noticed. It's a bit like "Audit the documentation for grammar constructs...".

My thought was the 80:20 would be feasible, and good enough. (Plus, I think @IgnatiusFlo volunteered, so it should t hang too log.)


Made codes more readable, prettier

I tried to make the codes become more standard, readable and prettier. Although it could be better yet.

Do you know why Travis check fails?

Do you know why Travis check fails?

This seems fine even with our current style guide. It also doesn’t look like there are a lot of functional changes in here that would affect performance. I do know that @Nyholm likes LOC as a measurement for this library so I am not sure extra linebreaks are going to get merged in.

@Zegnat Thanks for reviewing. However, your nice package could be nicer :wink:

Hey.
Thank you for the PR and for the review.
As @Zegnat says, This does not bring any functional changes and "pretty" is highly subjective.
I appreciate this contribution but I see no reason to merge at this point.


Can't build the repo when i run cargo build

error: failed to run custom build command for nanos_sdk v0.1.0 (https://github.com/LedgerHQ/ledger-nanos-sdk.git#427e686d)
Caused by:
process didn't exit successfully: /Users/kudiuser/Documents/self_learn/ledger_shenanigans/rust-app/target/debug/build/nanos_sdk-4d7c9822da2f9333/build-script-build (exit status: 101)
--- stderr
thread 'main' panicked at 'failed: Os { code: 2, kind: NotFound, message: "No such file or directory" }', /Users/kudiuser/.cargo/git/checkouts/ledger-nanos-sdk-f77eb1a6021b7ee5/427e686/build.rs:17:10
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...
error: build failed

Did you install an arm-none-eabi-gcc toolchain ?

I believe you can install it using brew install --cask gcc-arm-embedded.


Conan and Vcpkg packages are out of date

Both the Conan and Vcpkg Flux packages are git snapshots from some time ago.
It would be great to get these updated for the new Flux release.

(I guess this is only a "good first issue" if you're already familiar with Conor or Vcpkg, but...)

@tcbrindle Hi, I am also trying to get newer version of flux to xrepo (atleast for time being my own xrepo then to official one). I do have a question. Will the tags be now kept as is? I can't find release tags before 0.4.0. The previous version for xrepo is for commit 8434e8e0fd131cebc3aed0d845530335bf7fb5e1 tagged 2023.08.17 which is similar to vcpkg and connan which have the clang 18 crash issue present.
If the tags releases will be kept as, is it would be easier to write versions.

Hi @Sid911,

Will the tags be now kept as is? I can't find release tags before 0.4.0

Yes, the tags will be kept as-is from now on. Future releases of Flux will use semantic versioning, tagged appropriately. (0.4.0 was the first "official" numbered release).
I hope this helps, thanks for helping to get Flux packaged

For conan it appears only minimal efforts are required to the recipes:
config.yml:
  versions:
+  "0.4.0":
+    folder: all
    "cci.20240115":
      folder: all

all/conandata.yml:
   sources:
+   "0.4.0":
+     url: "https://codeload.github.com/tcbrindle/flux/zip/43c09af5cc2e89e32b98ddc3507247f593ec2642"
+     sha256: "9d8891b2ab9f826b301678b17011d48ab6c85e534e065f66269ae08537be68d5"
    "cci.20240115":
      url: "https://codeload.github.com/tcbrindle/flux/zip/e942a678ed3b46c7f7ffeebe47eed5c5bec005b2"
      sha256: "3832fb160417d14cfb3636c8edab04b2d0cd6c230b4d474a18bb1389fa1c3b8f"

I've tried this on my local system and it builds correctly and flux works. Need someone to submit that to conan center index. Who's maintaining flux package for conan?

ping @valgur (original PR)

The Conan update looks like it has propagated through (https://conan.io/center/recipes/flux?version=0.4.0), thanks very much @SidneyCogdill!

Conan Center current has some issues that can cause users to miss v0.4.0: https://github.com/conan-io/conan-center-index/issues/25863#issuecomment-2461921683
Therefore it might be helpful to directly mention conan tutorials in README.md:
conan remote update --url https://center2.conan.io conancenter

[requires]
flux/0.4.0
[generators]
CMakeDeps
CMakeToolchain
[layout]
cmake_layout

and warn that flux/[*] wildcard will miss the latest version due to the cci name.
and also add a direct link to the latest version https://conan.io/center/recipes/flux?version=0.4.0

The vcpkg package has now been updated as well, thanks again @SidneyCogdill!


strike total clear duration is not using both days

The duration of the last log per day on both strikes and raids was not added to the time.


Update the documentation for "Get started with nuxt" now that volar takeover and the typescript plugin are no longer needed.

Describe the feature
After helping a colleague to get started with nuxt3, we noticed that just using the base volar extension is the only thing needed.
I am only 90% sure that this is true, but im making the issue and a PR with the assumption that it is true.
Additional information

[X] Would you be willing to help implement this feature?
[ ] Could this feature be implemented as a module?

Final checks

[X] Read the contribution guide.
[X] Check existing discussions and issues.

This is currently the Vue documentation.
https://vuejs.org/guide/typescript/overview.html#volar-takeover-mode
cc: @johnsoncodehk

I updated Volar to 1.8.3 and I see this message if I don't enable the takeover mode.

Tested this again.
Running with node v18.12.1, Vue Language Features (Volar) 1.8.3, nuxt version 3.3.2. I have NOT disabled the builtin language features. I am not getting any issues at all. Given that this is working would point towards that atleast "TypeScript Vue Plugin" is no longer needed.
@prashantpalikhe Where did you see that issue?
@danielroe Maybe take over is still a good suggestion if it does indeed provide better performance (didnt see any signs of this either)?

Will need to dive in to confirm. Maybe we can also remove the default shim in that case ...

The situation has not changed, you still need one of takeover mode / ts plugin / shim, otherwise you will get an error when importing .vue in .ts.

@johnsoncodehk not sure I trust that given that it is working fine in all my projects with the setup I specified before. Can you provide a project where it is not working for you with the setup I use?

You can try https://github.com/johnsoncodehk/volar-starter

@SebbeJohansson

Where did you see that issue?

cmd + shift + p -> Show running extensions -> Volar
Without the takeover mode, I see the error I mentioned above.

Okay i did some further testing! I think this was more a case of me not understanding the vue/volar documentation. I understood it as if Typescript wouldnt work within vue components, not the other way around. Specifically "To get Vue SFCs and TypeScript working together" confused me. The fact that I am able to do most things within vue SFCs without takeover mode confused me.
@johnsoncodehk, I can confirm that I see the same errors from the typescript language package:

If I understand it correctly, I can still get most builtin nuxt stuff to work correctly, but typescript is still not working correctly in the background?
Example where i can use a "typescript feature":

As you can see all imports and functions i need to use, work without takeover mode.
@prashantpalikhe weird... I am not seeing that behaviour.
Given that apparently not everything works (but enough does for me) without takeover mode, it 100% makes sense to keep that instruction. I will close this issue.


Static IP to handle device reboots

Is your content request related to a problem? Please describe.
While I enjoy setting up mynode to connect/interact with my devices (different wallets) – simple and easy experience – I don't really want to do it over and over again.
Every so often the mynode has to reboot. For an update, for a change in setting. Or maybe I have to do some fan cleaning. When this happens... I have noticed that the IP address used doesn't stay constant.
ii) VPN settings need to be updated
iii) Onion links maybe stop working as well
Why is the IP changing? Not sure as not a network software dev (and it doesn't say assigned on the router).
Describe the solution you'd like
A few options could be possible ranking from h

Have the mynode self-assign/request an IP to the router so I don't have to think about
Setup a staticIP form the router using DHCP (basic instructions -> pointing with links for where/how to do it primarily. outlining the issue and ways around it so that folks don't have to think and dig, it is just there).
Something else that requires little rework.

Additional context
Add any other context or screenshots about the feature request here.

I understand that can be annoying, but the best way to solve it is not to set a static IP via the device. That would require lots of other settings, like DNS servers, etc...
The best way to handle this is to setup a static "host" within your router so a given device will always get the same IP. That way, it learns all the other info DHCP provides, like DNS servers.

Thanks for sharing Sounds good. Basically what I think I did (assigning the device an IP on the route). Not sure the language of "host" but I digress!
Maybe then the feature request changes to something of the following. Just trying to help get to a "don't make me think model".

A guide for those using the mynode (still requires a technical person)
Any solution that simplifies a node for the every-day person (e.g., no idea how, onion links that don't change and don't have latency/fail issues).

Best,
Jared


Animation refactory

A rewrite of the animation lifecycle that allows batching of animation frames when many charts are present on one page.  Greatly improves performance for the scenario, and provides a better internal api for animations.

+1 Totally on board with this

So... where is everyone on this?

I'm on board, but I helped write it so I'll defer to everyone else. :stuck_out_tongue:

Ditto
On Thu, Apr 30, 2015 at 7:45 AM Evert Timberg<EMAIL_ADDRESS>wrote:

I'm on board, but I helped write it so I'll defer to everyone else. [image:
:stuck_out_tongue:]
—
Reply to this email directly or view it on GitHub
https://github.com/nnnick/Chart.js/pull/1068#issuecomment-97797345.

@nnnick @derekperkins @jakesyl @fntneves We need a second nomination here please

+1 It works awesome for us in production

Just to one more :+1: , I have like 50 charts all on the same page, and their all updating at about 60fps now!

Solid

http://www.quickmeme.com/img/54/54504eb08de8bea51ecf9ccc75e21a07767a171264fdee210e3d22ad8cd334c9.jpg

:+1: :100:


[Bug]: Settings not applying (PhpStorm)

What happened?
Some issues still persist despite my previous bug report on wsl +JBC issues. Application, project and file settings customizations are not applied despite updating to v2.1.5.
IDE version
24.3 Preview
Plugin version
2.1.4
What operating system are you using?
Windows
Relevant log output
Please contact me on discord again for relevant logs.

After spending some time debugging, I have to restart wsl projects in order for changes to work.


--audio-width problem with transcode-video on CentOS

Hi, recently I encountered a problem with transcoding video on CentOS 7. The input file contains audio track in Dolby format. I wanted the output to have only stereo audio AAC format, so I added --audio-width main=stereo into the command. The output is as expected when I tested in my local MacOS computer. However the output file when I run the command on CentOS still have audio track with 6 channels, when I inspect the output file with ffprobe, it comes like this:
{
"index": 1,
"codec_name": "aac",
"codec_long_name": "AAC (Advanced Audio Coding)",
"profile": "LC",
"codec_type": "audio",
"codec_time_base": "1/48000",
"codec_tag_string": "mp4a",
"codec_tag": "0x6134706d",
"sample_fmt": "fltp",
"sample_rate": "48000",
"channels": 6,
"bits_per_sample": 0,
"r_frame_rate": "0/0",
"avg_frame_rate": "0/0",
"time_base": "1/48000",
"start_pts": -440,
"start_time": "-0.009167",
"duration_ts": 20020800,
"duration": "417.100000",
"bit_rate": "384516",
"max_bit_rate": "384516",
"nb_frames": "19552",
"disposition": {
"default": 1,
"dub": 0,
"original": 0,
"comment": 0,
"lyrics": 0,
"karaoke": 0,
"forced": 0,
"hearing_impaired": 0,
"visual_impaired": 0,
"clean_effects": 0,
"attached_pic": 0,
"timed_thumbnails": 0
},
"tags": {
"creation_time": "2018-08-28T04:42:02.000000Z",
"language": "eng",
"handler_name": "Surround"
}
}
]
Any ideas where I could have been using it incorrectly? Thanks in advance
P.S: I use latest HandBrake version, following this guide: https://handbrake.fr/docs/en/latest/developer/install-dependencies-centos.html

Tested on Ubuntu, seems that this problem only happens on Cent

@nobitataj Interesting. It seems the audio mixdown defaults are different on CentOS than any other platform.
You could try forcing the audio mixdown on CentOS by adding --handbrake-option mixdown=dpl2 to your command line. The dpl2 indicates that you want a stereo mix in Dolby Prologic II format.
Let me know if that works. And thanks for opening this issue!

@donmelton perfect
{
"index": 1,
"codec_name": "aac",
"codec_long_name": "AAC (Advanced Audio Coding)",
"profile": "LC",
"codec_type": "audio",
"codec_time_base": "1/48000",
"codec_tag_string": "mp4a",
"codec_tag": "0x6134706d",
"sample_fmt": "fltp",
"sample_rate": "48000",
"channels": 2,
"channel_layout": "stereo",
"bits_per_sample": 0,
"r_frame_rate": "0/0",
"avg_frame_rate": "0/0",
"time_base": "1/48000",
"start_pts": 0,
"start_time": "0.000000",
"duration_ts": 135420912,
"duration": "2821.269000",
"bit_rate": "158904",
"max_bit_rate": "158904",
"nb_frames": "132248",
"disposition": {
"default": 1,
"dub": 0,
"original": 0,
"comment": 0,
"lyrics": 0,
"karaoke": 0,
"forced": 0,
"hearing_impaired": 0,
"visual_impaired": 0,
"clean_effects": 0,
"attached_pic": 0,
"timed_thumbnails": 0
},
"tags": {
"creation_time": "2018-08-29T01:50:23.000000Z",
"language": "eng",
"handler_name": "Stereo"
}
},
{
"index": 2,
"codec_name": "ac3",
"codec_long_name": "ATSC A/52A (AC-3)",
"codec_type": "audio",
"codec_time_base": "1/48000",
"codec_tag_string": "ac-3",
"codec_tag": "0x332d6361",
"sample_fmt": "fltp",
"sample_rate": "48000",
"channels": 2,
"channel_layout": "stereo",
"bits_per_sample": 0,
"dmix_mode": "-1",
"ltrt_cmixlev": "-1.000000",
"ltrt_surmixlev": "-1.000000",
"loro_cmixlev": "-1.000000",
"loro_surmixlev": "-1.000000",
"r_frame_rate": "0/0",
"avg_frame_rate": "0/0",
"time_base": "1/48000",
"start_pts": 0,
"start_time": "0.000000",
"duration_ts": 135421152,
"duration": "2821.274000",
"bit_rate": "224000",
"nb_frames": "88165",
"disposition": {
"default": 0,
"dub": 0,
"original": 0,
"comment": 0,
"lyrics": 0,
"karaoke": 0,
"forced": 0,
"hearing_impaired": 0,
"visual_impaired": 0,
"clean_effects": 0,
"attached_pic": 0,
"timed_thumbnails": 0
},
"tags": {
"creation_time": "2018-08-29T01:50:23.000000Z",
"language": "eng",
"handler_name": "Stereo"
},
"side_data_list": [
{
"side_data_type": "Audio Service Type"
}
]
}
Tested on Mac and Cent, let me try on Ubuntu

Ubuntu verified working, thanks a lot @donmelton

@nobitataj Excellent! I'll close this now but feel free to comment here or open other useful issues like this one.


AVRO-2865: Remove maven 2 support

Make sure you have checked all steps below.
Jira

[X] My PR addresses the following Avro Jira issues and references them in the PR title. For example, "AVRO-1234: My Avro PR"

https://issues.apache.org/jira/browse/AVRO-2865
In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

Tests

[X] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[X] My commits all reference Jira issues in their subject lines. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain Javadoc that explain what it does

@iemejia What do you think -- is this CVE a reason to wait for 1.10.0 RC2 ?
On the one hand, the plexus-util jar with the vulnerability will only be on the machine building avro specific records, and XML injection could only be done from the pom.xml sitting right there in front of the user running maven...
... on the other hand, a lot of build machines are CI (jenkins) and automatic/expensive shared resources.

It makes sense to do a RC2 we should not let security vulnerabilities exposed. Thanks for the PR @RyanSkraba


dns/bind: update template paths to 9.18

The package update in https://github.com/opnsense/plugins/pull/3192 did not update new version's changes to the configuration file path changes:
9.16 - https://cgit.freebsd.org/ports/tree/dns/bind916/pkg-plist
9.18 - https://cgit.freebsd.org/ports/tree/dns/bind918/pkg-plist
TLDR: /usr/local/etc/named/master is now /usr/local/etc/named/primary and /usr/local/etc/named/slave is now /usr/local/etc/named/secondary

I did not increase the version number in the Makefile since I wasn't sure if that was needed in this case.

Is this really necessary when using a fixed path or do you encounter an error when running the devel version

I was working on fixing another issue, and the devel version wouldn't run specifically because these files are no longer in the right place:
zone "localhost"        { type master; file "/usr/local/etc/namedb/primary/localhost-forward.db"; };
zone "127.in-addr.arpa" { type master; file "/usr/local/etc/namedb/primary/localhost-reverse.db"; };
zone "0.ip6.arpa"       { type master; file "/usr/local/etc/namedb/primary/localhost-reverse.db"; };

Agree here with moving all. The only issue I see here is that eventually GUI should also follow the official bind nomenclature, which is tricky because it is also used in JavaScript and could subtly break while doing it. For now this PR is best...

Agree here with moving all. The only issue I see here is that eventually GUI should also follow the official bind nomenclature, which is tricky because it is also used in JavaScript and could subtly break while doing it. For now this PR is best...

I can take a stab at that in another PR in the next few days, the next release will be in January I'm assuming? I am using almost every feature in this plugin at this point so would be relatively easy for me to test.

Merged, thanks!

This is also in the API endpoint URLs. We can switch those for a major release, but as I said it's not unlikely we will miss something doing so. 23.1 is not that far away, but it should be attempted for sure if @mimugmail agrees.
Cheers,
Franco

We definitely need to update the API endpoint in acme-client: https://github.com/opnsense/plugins/issues/3103


Annotation - Some checkboxes have an empty N dictionary

it aims to fix #14021;
the N dict is empty here so just create a default one;
it implies that the checked checkbox has no appearance so create a default one too in order to print it.
in the pdf in the issue, a checked box is not printed because it has no default appearance so we need to guess its appearance from its state.

/botio test

/botio-linux test

The regression for issue12294 isn't a regression but an improvement !
I compared preview in firefox and in chrome and we've the same.

/botio test

/botio makeref

/botio-linux makeref


Exception by using filter identity_plugin_has_role

If I use the filter has_role like this:
{{ "admin" | identity_plugin_has_role }}

I receive the following exception:
undefined method `has_role' for #<Liquid::Strainer:0x007fac932171f8>

Please tell me what is wrong?
Thanks in advance!

Seems to be a ruby issue or somewhat related to ruby:
It works if I use ruby 2.0.0. The issue returns starting with 2.1.2.


When spring.liquibase.clear-checksums property of spring boot is set, when the target database is empty, it throws exception, and SpringLiquibase.afterPropertiesSet doesn't run the changelogs

Search first

[X] I searched and no similar issues were found

Description
So I want to submit two bugs, but didn't want to open two issues because two can be reproduced from the same repo that I will provide
Steps To Reproduce
First bug: When spring.liquibase.clear-chekcsums is set to true, initializing on an empty database throws exception in spring boot application on application startup
Steps:

Clone repo: https://github.com/fatihdogmus/afterpropertiesset-bug
Run docker compose up -d. This will initialize a bitnami alpine 15.4 postgres image and bind the data to ~/DockerVolumes/liquibase-bug, since we need persistent storage for the next bug
Run the application
You will see exception: encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'liquibase' defined in class path resource [org/springframework/boot/autoconfigure/liquibase/LiquibaseAutoConfiguration$LiquibaseConfiguration.class]: liquibase.exception.DatabaseException: liquibase.exception.DatabaseException: ERROR: relation "public.databasechangelog" does not exist Position: 8 [Failed SQL: (0) UPDATE public.databasechangelog SET MD5SUM = NULL]

Second bug: This is a bit trickier. After DB initialization, when we want to reset the database programatically with SpringLiquibase, by first dropping everything, the changes are not run when spring.liquibase.clear-chekcsums is set to true
Steps:

Clone the same repo and get DB working with docker. I tried with empty database, so it would be wise to remove anything from previous DB.
First, set spring.liquibase.clear-chekcsums to false, otherwise the application doesn't start
Stop the application, set spring.liquibase.clear-chekcsums to true and run the application. After this, the DB is in correct state.
Send a GET request to http://localhost:8087/resetDatabase, which is this code:

@RestController
public class ResetDatabaseController {

    private final SpringLiquibase springLiquibase;

    public ResetDatabaseController(final SpringLiquibase springLiquibase) {
        this.springLiquibase = springLiquibase;
    }

    @GetMapping("/resetDatabase")
    public String resetDatabase() throws LiquibaseException {
        springLiquibase.setClearCheckSums(false);
        springLiquibase.setDropFirst(true);

        springLiquibase.afterPropertiesSet();

        springLiquibase.setClearCheckSums(true);
        springLiquibase.setDropFirst(false);

        return "Database has been reset successfully";
    }
}

You will see Database is up to date, no changesets to execute command in the logs, after all the objects are dropped.
And the database won't be updated.

Strangely, this bug doesn't happen if the application is started with spring.liquibase.clear-chekcsums set to false. I don't know what the problem is, I set it false on my code anyway so I don't know why it would behave differently.
Both problems didn't happen before we migrated to spring boot 3.2.0 from 3.1.4
Expected/Desired Behavior
For first bug to not throw exceptions and second bug to run the changes.
Liquibase Version
4.24.0
Database Vendor & Version
PostgreSQL 15.4
Liquibase Integration
Spring Boot
Liquibase Extensions
No response
OS and/or Infrastructure Type/Provider
Macos
Additional Context
No response
Are you willing to submit a PR?

[x] I'm willing to submit a PR (Thank you!)

I can provide a PR, but I don't know what the problem is. I've looked at the source code, but couldn't find the problem, since I'm not very familiar with the code base. If you can find the problem and direct me to it, I can submit a PR.

Hello @fatihdogmus !
I believe that this may be related to clearChecksums command refactoring, done in PR https://github.com/liquibase/liquibase/pull/4628 and included in version 4.24.0 . Does the problem goes away if you force liquibase version 4.23.2 ?
If this is true then after the refactoring we may be throwing an exception that is not being handled correctly by SpringLiquibase class.

Hey,
Since spring boot 3.2.0 comes with 4.24.0, when I try to set version to 4.23.2 with spring boot 3.2.0, it gives the following error:
java.lang.NoClassDefFoundError: liquibase/UpdateSummaryOutputEnum 	at java.base/java.lang.Class.getDeclaredFields0(Native Method) ~[na:na] 	at java.base/java.lang.Class.privateGetDeclaredFields(Class.java:3473) ~[na:na] 	at java.base/java.lang.Class.getDeclaredField(Class.java:2780) ~[na:na] 	at org.springframework.boot.context.properties.bind.DefaultBindConstructorProvider$Constructors.isInnerClass(DefaultBindConstructorProvider.java:144) ~[spring-boot-3.2.0.jar:3.2.0] 	at org.springframework.boot.context.properties.bind.DefaultBindConstructorProvider$Constructors.getCandidateConstructors(DefaultBindConstructorProvider.java:134) ~[spring-boot-3.2.0.jar:3.2.0] 	at org.springframework.boot.context.properties.bind.DefaultBindConstructorProvider$Constructors.getConstructors(DefaultBindConstructorProvider.java:103) ~[spring-boot-3.2.0.jar:3.2.0] 	at org.springframework.boot.context.properties.bind.DefaultBindConstructorProvider.getBindConstructor(DefaultBindConstructorProvider.java:55) ~[spring-boot-3.2.0.jar:3.2.0] 	at org.springframework.boot.context.properties.ConfigurationPropertiesBean.deduceBindMethod(ConfigurationPropertiesBean.java:298) ~[spring-boot-3.2.0.jar:3.2.0] 	at org.springframework.boot.context.properties.ConfigurationPropertiesBeanRegistrar.createBeanDefinition(ConfigurationPropertiesBeanRegistrar.java:93) ~[spring-boot-3.2.0.jar:3.2.0] 	at org.springframework.boot.context.properties.ConfigurationPropertiesBeanRegistrar.registerBeanDefinition(ConfigurationPropertiesBeanRegistrar.java:89) ~[spring-boot-3.2.0.jar:3.2.0] 	at org.springframework.boot.context.properties.ConfigurationPropertiesBeanRegistrar.register(ConfigurationPropertiesBeanRegistrar.java:61) ~[spring-boot-3.2.0.jar:3.2.0] 	at org.springframework.boot.context.properties.ConfigurationPropertiesBeanRegistrar.register(ConfigurationPropertiesBeanRegistrar.java:55) ~[spring-boot-3.2.0.jar:3.2.0] 	at java.base/java.lang.Iterable.forEach(Iterable.java:75) ~[na:na] 	at org.springframework.boot.context.properties.EnableConfigurationPropertiesRegistrar.registerBeanDefinitions(EnableConfigurationPropertiesRegistrar.java:49) ~[spring-boot-3.2.0.jar:3.2.0] 	at org.springframework.context.annotation.ImportBeanDefinitionRegistrar.registerBeanDefinitions(ImportBeanDefinitionRegistrar.java:86) ~[spring-context-6.1.1.jar:6.1.1] 	at org.springframework.context.annotation.ConfigurationClassBeanDefinitionReader.lambda$loadBeanDefinitionsFromRegistrars$1(ConfigurationClassBeanDefinitionReader.java:376) ~[spring-context-6.1.1.jar:6.1.1] 	at java.base/java.util.LinkedHashMap.forEach(LinkedHashMap.java:986) ~[na:na] 	at org.springframework.context.annotation.ConfigurationClassBeanDefinitionReader.loadBeanDefinitionsFromRegistrars(ConfigurationClassBeanDefinitionReader.java:375) ~[spring-context-6.1.1.jar:6.1.1] 	at org.springframework.context.annotation.ConfigurationClassBeanDefinitionReader.loadBeanDefinitionsForConfigurationClass(ConfigurationClassBeanDefinitionReader.java:148) ~[spring-context-6.1.1.jar:6.1.1] 	at org.springframework.context.annotation.ConfigurationClassBeanDefinitionReader.loadBeanDefinitions(ConfigurationClassBeanDefinitionReader.java:120) ~[spring-context-6.1.1.jar:6.1.1] 	at org.springframework.context.annotation.ConfigurationClassPostProcessor.processConfigBeanDefinitions(ConfigurationClassPostProcessor.java:428) ~[spring-context-6.1.1.jar:6.1.1] 	at org.springframework.context.annotation.ConfigurationClassPostProcessor.postProcessBeanDefinitionRegistry(ConfigurationClassPostProcessor.java:289) ~[spring-context-6.1.1.jar:6.1.1] 	at org.springframework.context.support.PostProcessorRegistrationDelegate.invokeBeanDefinitionRegistryPostProcessors(PostProcessorRegistrationDelegate.java:349) ~[spring-context-6.1.1.jar:6.1.1] 	at org.springframework.context.support.PostProcessorRegistrationDelegate.invokeBeanFactoryPostProcessors(PostProcessorRegistrationDelegate.java:118) ~[spring-context-6.1.1.jar:6.1.1] 	at org.springframework.context.support.AbstractApplicationContext.invokeBeanFactoryPostProcessors(AbstractApplicationContext.java:775) ~[spring-context-6.1.1.jar:6.1.1] 	at org.springframework.context.support.AbstractApplicationContext.refresh(AbstractApplicationContext.java:597) ~[spring-context-6.1.1.jar:6.1.1] 	at org.springframework.boot.web.servlet.context.ServletWebServerApplicationContext.refresh(ServletWebServerApplicationContext.java:146) ~[spring-boot-3.2.0.jar:3.2.0] 	at org.springframework.boot.SpringApplication.refresh(SpringApplication.java:753) ~[spring-boot-3.2.0.jar:3.2.0] 	at org.springframework.boot.SpringApplication.refreshContext(SpringApplication.java:455) ~[spring-boot-3.2.0.jar:3.2.0] 	at org.springframework.boot.SpringApplication.run(SpringApplication.java:323) ~[spring-boot-3.2.0.jar:3.2.0] 	at org.springframework.boot.SpringApplication.run(SpringApplication.java:1342) ~[spring-boot-3.2.0.jar:3.2.0] 	at org.springframework.boot.SpringApplication.run(SpringApplication.java:1331) ~[spring-boot-3.2.0.jar:3.2.0] 	at spring.liquibase.afterpropertiessetbug.AfterpropertiessetBugApplication.main(AfterpropertiessetBugApplication.java:10) ~[classes/:na] Caused by: java.lang.ClassNotFoundException: liquibase.UpdateSummaryOutputEnum 	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:641) ~[na:na] 	at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:188) ~[na:na] 	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:526) ~[na:na] 	... 33 common frames omitted
it seems it is trying to find an enum that doesn't exist in older versions.
When I set the spring boot version to 3.1.6, and liquibase version to 4.23.2, the first bug goes away, but the second one, where it doesn't run the changelogs, still persists. And the workaround of setting the clear checksums to false also works.
Weirdly enough, when the application starts from an emty database and clear checksums is true, the reset database works, but when I restart the application it doesn't.

hey, any update on the issue? we are still having the same problem and having difficulty in developmen

Hi, any update on this issue?
we are encountering similar issues when issuing liquibase clear-checksums from command line against an empty database.
This issue happens on both 4.25.0 and 4.25.1, but does not happen on 4.23.0.
Liquibase Version: 4.25.0
Starting Liquibase at 13:57:44 (version 4.25.0 #3966 built at 2023-11-10 23:07:29 UTC)
Liquibase Version: 4.25.0
Liquibase Open Source 4.25.0 by Liquibase

Unexpected error running Liquibase: ERROR: relation "public.databasechangelog" does not exist
  Position: 8 [Failed SQL: (0) UPDATE public.databasechangelog SET MD5SUM = NULL]

For more information, please use the --log-level flag

Liquibase Version: 4.25.1
Starting Liquibase at 07:54:58 (version 4.25.1 #690 built at 2023-12-18 16:29+0000)
Liquibase Version: 4.25.1
Liquibase Open Source 4.25.1 by Liquibase
ERROR: Exception Details
ERROR: Exception Primary Class:  PSQLException
ERROR: Exception Primary Reason: ERROR: relation "public.databasechangelog" does not exist
  Position: 8
ERROR: Exception Primary Source: PostgreSQL 12.16

Unexpected error running Liquibase: ERROR: relation "public.databasechangelog" does not exist
  Position: 8 [Failed SQL: (0) UPDATE public.databasechangelog SET MD5SUM = NULL]

Liquibase Version: 4.23.0
Starting Liquibase at 13:16:33 (version 4.23.0 #10709 built at 2023
Liquibase Version: 4.23.0
Liquibase Open Source 4.23.0 by Liquibase
Liquibase command clear-checksums" was executed successfully.

Thanks @xys1836 ! It was somewhere between 4.23.2 and 4.24.0 that it was broken. Let me see what we can do.


Map not reshreshed after push new styles

Issue description
I'm using a toggle button to display or not poi on the map.
For that purpose I have a toggle code in my component:
  toggleBusiness() {
    if (this.businessDisplayed) {
      this.styles.push({
        featureType: 'poi.business',
        elementType: 'labels',
        stylers: [
          { visibility: 'off' }
        ]
        });
    } else {
      const indexToRemove = this.styles.findIndex(s => s.featureType === 'poi.business');
      this.styles.splice(indexToRemove, 1);
    }
    this.businessDisplayed = !this.businessDisplayed;
  }

This code is not working, the map style is not applied.
If I'm using this code:

  toggleBusiness() {
    if (this.businessDisplayed) {
      this.styles = [{
        featureType: 'poi.business',
        elementType: 'labels',
        stylers: [
          { visibility: 'off' }
        ]
        }];
    } else {
      this.styles = [];
    }
    this.businessDisplayed = !this.businessDisplayed;
  }

But it clears my predefined styles ...
I tried by adding a MarkForCheck() in case of. But without success.
My detectionStrategy is the initial one, no OnPush
Steps to reproduce and a minimal demo of the problem
https://stackblitz.com/edit/angular-google-maps-demo-z5utkm?file=app%2Fapp.component.ts
Expected/desired behavior
Styles are refreshed even if we use push on styles
angular-google-maps, Angular, & any other relevant dependency versions
Other information

use ChangeDetectorRef.detectChanges


vendor-specific strings in HTTP headers

The current distribution-spec contains vendor-specific strings like Docker in HTTP headers. Though it's not clear to me what we should do about it, because simple string substitutions would be incompatible changes. So I'm simply listing them below, without creating a PR.

Docker-Content-Digest

https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/changes.md
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#digest-header
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#existing-manifests
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#existing-layers
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#completed-upload
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#cross-repository-blob-mount
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#on-success-ok-3
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#on-success-created
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#on-success-ok-4
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#on-success-temporary-redirect
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#on-success-accepted-1
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#on-success-upload-complete

Docker-Distribution-API-Version

https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#api-version-check

Docker-Upload-UUID

https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#uploading-the-layer
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#upload-progress
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#chunked-upload
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#on-success-created-1
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#on-success-accepted-2
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#on-success-created-2
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#on-success-upload-progress
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#on-success-data-accepted
https://github.com/opencontainers/distribution-spec/blob/59d293a11d862b2aad0c984dc6516f0401830045/spec.md#on-success-chunk-accepted

I suggest we indicate in the specification that to support legacy implementations of the distribution spec the following http headers MUST be supported as alias names for the corresponding OCI header names:
Docker-Content-Digest == OCI-Content-Digest
...

@mikebrow I like option 1.
What about the Docker-Distribution-Api-Version header? Currently it SHOULD be set to registry/2.0 but it'd personally like to see something with semantic versioning like:
OCI-Distribution-Api-Version: registry/v2.0.0
OCI-Distribution-Api-Version: aws-ecr/v3.5.1
OCI-Distribution-Api-Version: google/v0.8.2

Below is a list of all the Docker headers mapped to OCI:

Docker
OCI

Docker-Distribution-Api-Version
OCI-Distribution-Api-Version

Docker-Content-Digest
OCI-Content-Digest

Docker-Upload-UUID
OCI-Upload-ID

@dongsupark That's a question for the maintainers. I would love to see this change made before the v1.0 though.

@dongsupark i think this have been worked at, but could you review again to see how far it is from being closed?

At meeting, we concluded Docker-Content-Digest is the last thing that needs to be worked out in order to close this issue. This discussion has been centered around making that specific header optional.

As discussed on the call today, we'd like to include every Docker prefixed header as optional, and not replace any with OCI prefixed headers for 1.0.0.
The follow steps need to be done to close this issue:
Docker-Distribution-API-Version - needs to be documented that it is OPTIONAL
Docker-Content-Digest - needs to be made clear in the tables that it is OPTIONAL

Resolved by #206


HHVM incompatiblity with 1.4

Fatal error: Parameter $callbacks is variadic and has a type constraint (callable); variadic params with type constraints are not supported in /home/travis/build/phpbench/phpbench/vendor/lstrojny/functional-php/src/Functional/Partition.php on line 41

As encountered in: https://travis-ci.org/phpbench/phpbench/jobs/177463602#L402
1.2.4 works fine

This is the upstream bug for that issue: https://github.com/facebook/hhvm/issues/6954

Closing


Toggle UI

I have trouble making my clipboard working over ssh (complicated and not related to this issue). I found that selecting text with shift is a very easy and working all the way solution, the only problem is that to copy file text only, I have to hide

Line numbers
Indentation signs |
Linting errors
more..

Would be awesome to have a possibility to toggle all UI options off and on. Ideally with Shift down

please use discussions in the LazyVim repo instead


prebuild binaries for windows

Hi,
i develop node.js apps with VSCode / Webstorm. I want to use better-sqlite3 and i'm pleased that you added prebuild support. In the releases section you said
Prebuilt binaries are only made for Linux and Mac OSX, not Windows (but again, building from source should work fine on Windows too).

I just tried building it myself and heres what problems i encouter:

i had to install python and set the path correctly
now it wants me to install visual studion

gyp ERR! stack Error: Could not find any Visual Studio installation to use
gyp ERR! stack     at VisualStudioFinder.fail (C:\Users\D068725\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\lib\find-visualstudio.js:121:47)
gyp ERR! stack     at C:\Users\D068725\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\lib\find-visualstudio.js:74:16
gyp ERR! stack     at VisualStudioFinder.findVisualStudio2013 (C:\Users\D068725\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\lib\find-visualstudio.js:351:14)
gyp ERR! stack     at C:\Users\D068725\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\lib\find-visualstudio.js:70:14
gyp ERR! stack     at C:\Users\D068725\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\lib\find-visualstudio.js:372:16
gyp ERR! stack     at C:\Users\D068725\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\lib\util.js:54:7
gyp ERR! stack     at C:\Users\D068725\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\lib\util.js:33:16
gyp ERR! stack     at ChildProcess.exithandler (child_process.js:297:5)
gyp ERR! stack     at ChildProcess.emit (events.js:200:13)
gyp ERR! stack     at maybeClose (internal/child_process.js:1021:16)

Not sure which problems come up next.
I don't know how big VS is now, when i last tried it it was a pretty big installation and a lot of components were installed and it was not so easy to get rid of them all.
So for me it is not easy to install this binary at all. Is there an easier solution for this?
Is there no way to also add prebuild binaries for windows? This really is a limitation for me, as we have a mixed setup of win/linux/mac in our team.
Best regards
Samuel

So for me it is not easy to install this binary at all. Is there an easier solution for this?

I'm not a Windows user, but the docs say windows-build-tools is enough?
npm install --global --production windows-build-tools

Option 1: https://github.com/nodejs/node-gyp#on-windows
It probably doesn't get easier than a single command.

Thanks @Prinzhorn, will give it a try.

Somehow installing windows build tools did not work on my machine correctly, also i think its too much to ask if you just want to use a dependency, to install some additional build tools or programming languages like python. This was already mentioned here:
https://github.com/JoshuaWise/better-sqlite3/issues/88
Guess i can not use it right now. Is there anything i can help with to get prebuilds on windows running?

This is now available in v7.1.4.


Refactor (Routes and Filters)

I've been stumbling though the code and found out that before and after are part of the the HttpMethod enum. I believe this was made to treat filters as routes, but I don't think this brings in easy coding or any other advantage. The code ends up doing things like
if (filterTarget instanceof FilterImpl) {

Is there any other reason why  Routes and Filters should be treated as one?

Is this fixed?

I never proposed a change, no one seemed to care =P

I'm trying to care now (cleaning up our issues).
Looking at HttpMethods, before/after definitely seem out of place (not being, well, HttpMethods), but could you give me a bit more context to your example? When do you need to do instanceof?
Also, Routes and Filters are already two different interfaces, where are they treated as one?

here is where you need the instanceof
and here is where they are treated as one (see add route and add filter do the same thing, they just put a different HTTP method/name)


[AutoPR cdn/resource-manager] [Hub Generated] Review request for Microsoft.Cdn to add version 2025-01-01

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4865

(message created by the CI based on PR content)

Closed initial issue


Fix inconsistency in import alias

This PR addresses the inconsistent import alias, instead of contextily being imported as ctx or cx it is now only imported as cx. This is in line with the GeoPandas documentation.

@darribas Do you have a preference on this?


python3Packages.zigpy-zigate: propagate build inputs

Motivation for this change
I didn't notice zigpy-zigate didn't have propagatedBuildInputs before. It's other dependencies previously happened to be resolved by its fellow zigpy-* modules, but recent update added a unique dependency on pyusb, causing my home-assistant to fail at loading the component.
Things done
Changed zigpy-zigate to propagate build inputs.

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

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

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

Oh yeah, I missed that. Also I'm personally a zigbee2mqtt user :)


How to Deploy an ERC-721 NFT Smart Contract with Remix IDE

Topic Suggestion
How to Deploy an ERC-721 NFT Smart Contract
Writing sample(s):

https://www.section.io/engineering-education/an-introduction-to-blockchain-architecture/
https://www.section.io/engineering-education/an-introduction-to-defi/

Proposal Submission
Proposed title of the article
How to Deploy an ERC-721 NFT Smart Contract
Proposed article introduction
[Blockchain] This article will provide a step by step guide to deploying an NFT with Remix IDE and Interplanetary File System
Key takeaways

How to deploy an ERC-721 token
Writing the Smart Contract with Remix to deploy the token
Utilizing IPFS as a decentralized cloud in storing your digital artwork
Utilising Metamask test networks
Deploying and Compiling an ERC-721 token

Article quality
This article will be succinct towards ensuring beginners in the blockchain space can easily understand how to deploy an NFT token, because it seems to have been made complex and strange to new entrants in this tech space.
References
N/A

Thank you for submitting your topic suggestion. There seems to be many articles that already cover this topic. Can you tell me how yours will build upon them and be unique to add value to our readers?

I have checked through them, mine is a variation of that of quicknode, but it will show how to mint an NFT on the OpenSea market place and using pinafa cloud, coupled with a process on the test opens market place to buy and sell. It basically covers the whole process end-to-end.

Topic approved but please ensure your article is in-depth and unique enough to add value for our readers.


Add support for table reflection and statement compilation

Hi,
Is it possible to add support for SQLAlchemy's table reflection and statement compilation features?
Here is an example:
table = CHTable(
table_name,
metadata,
engines.Memory(),
autoload=True,
schema=schema_name,
keep_existing=keep_existing,
extend_existing=extend_existing,
)
stmnt = (
select(
table.c.id,
table.c.name
)
.outerjoin(second_table, second_table.id == table.id)
)
Thanks!

Of course pull requests are always welcome but in general SQLAlchemy support is lower priority than many of the other outstanding enhancement requests.  (Honestly part of the reason is that SQLAlchemy code and documentation is extremely challenging to work with and test.)


Watching problem

Watching did not work, because it rebuild all files, not only changed file. Other functions like gulp-livereload runs a number of all files times. What is that? Why it is like that? Why watch cannot detect only changed file and rebuild it???

You already opened an issue about this #1331
Please stop opening duplicate issues.


Is this project still alive?

Very useful PRs (https://github.com/vectart/ember-cli-pace/pull/40) have not yet been merged...

I have a feeling the answer is: no :(
@devotox since you worked on #40, are you interested in maintaining a fork?


Reissue Certificate Emails

Scope of changes
This PR creates three new email templates as well as the context data and helper functions to send them. The email templates are:

"ExpiresAdminNotification": the email sent to the TRISA admins when a certificate is about to expire so they can review the certs and ensure that they should be reissued.
"ReissueReminder": the email sent to all verified contacts associated with the VASP letting them know that their certificates will expire and be automatically reissued soon.
"ReissueStarted": the email sent to the same contact that will receive the certs (only one contact -- usually the technical contact) that contains the whisper link to the PKCS12 password.

Each of these emails has:

Templates (both text and html) in the templates directory
A subject for the email
A context Data structure to populate the templates
An email builder function to create the email from the templates and the contex
A client send method to quickly send the email using send grid

Type of change

[ ] bug fix
[x] new feature
[ ] documentation
[ ] other (describe)

Acceptance criteria
Grammar and consistency are important for these emails. Also please check that we're sending to the right contacts and including the right information in the email template!
Author checklist

[x] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[x]  I have recompiled and included new protocol buffers to reflect changes I made
[x]  I have added new test fixtures as needed to support added tests
[ ]   Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[x]  I have moved the associated Shortcut story to "Ready for Review"

Reviewer(s) checklist

[x] QA expires_admin_notification email for grammar and spelling (including subject)
[x] QA reissuance_remdiner email for grammar and spelling (including subject)
[x] QA reissuance_started email for grammar and spelling (including subject)
[ ] Tests seem consistent
[x] Data in contexts is appropriate
[x] Emails make sense generally

@rebeccabilbro the Admin UI tests are failing but I'm going to go ahead and merge; it doesn't appear to be anything related to this PR.


gltfpack: Keep image names and unused named images

Discussed in https://github.com/zeux/meshoptimizer/discussions/682

Originally posted by enesaltinkaya April 27, 2024
Hi,
Blender has a new features regarding images while exporting GLTF/GLB;

export_unused_images (boolean, (optional)) – Unused images, Export images not assigned to any material

When using this feature, one needs to know names to be able to distinguish between images.
Similar to -kn can we have a -kin "keep image names" for example?
The second image below named "test_Albedo" is not connected to any materials. Both images are packed in .blend file.
This could be useful when one wants to carry over additinonal images with glb, like heightmaps/bump maps.
And once again thanks a lot for this awesome library!
before

after

Fixed by https://github.com/zeux/meshoptimizer/commit/ae4d14ec3031d8ab9a077325d546e417ee856115 (which should be sufficient since unused images are not skipped atm).


splitting osdq into independent component

This merge is to split the current profiler into osdq-core and osdq-desktop as discussed over email.
I have already created new repository for osdq-desktop under arrahtec and renamed profiler as osdq-core.
So we have now two repositories, osdq-core and osdq-dekstop.
The version is 6.1.8-SNAPSHOT, once this is fully tested I will drop SNAPSHOT and push into maven as 6.1.8.
https://github.com/arrahtec/osdq-desktop will now main project which is dependent on osdq-core.
_Please note I have also dropped 4 proprietary jdbc driver as well as rowset.jar. So before releasing next osdq-desktop to user on sourceforge we need to inform users about downloading drivers themselves, we can provide respective company url for those driver on our site. Also we need to change the code dependent on rowset.jar as its no longer available.
Once you merge, you need to clone osdq-desktop and this project. Run mvn install on core first (this will install core in local m2 repository. And then you can work with osdq-desktop. We can setup call if required.

split into independent projects


whoops, a bit oversimplified

returned []byte needs to have some kind of free function, or going back to a callback like before
onJson(function of []byte)

Just wanted to pop in and say this has to be the best name for a JSON serializer ever. I knew exactly your perspective when naming this thing I'm super cereal!


[Bug] upgrade jetty due to CVE-2024-8184

Search before asking

[X] I searched the issues and found no similar issues.

Linkis Component
linkis-dist, linkis-public-enhancements
Steps to reproduce
CVE-2024-8184
Expected behavior
secure dependencies
Your environment

Linkis version used: 1.1.2
Environment name and version:

cdh-5.14.2
hdp-3.1.5
hive-2.1.1
spark-3.2.1
scala-2.12.2
jdk 1.8.0_121
....

Anything else
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

CONTRIBUTING.md


Azure SQL Database: Clarify ALTER DATABASE with ##MS_DatabaseManager## role

As it is recommended to use ##MS_DatabaseManager## over the dbmanager role, could this paragraph be modified to take that into account?

To alter a database a login must be either the server admin login (created when the Azure SQL Database logical server was provisioned), the Azure AD admin of the server, a member of the dbmanager database role in master, a member of the db_owner database role in the current database, or dbo of the database.

It would also be helpful to clarify the Permissions of fixed server roles where it says that the server level role provides ALTER ANY DATABASE on the server level but under the previous heading it says

Members of the ##MS_DatabaseManager## role don't necessarily have permission to access databases that they don't own.

How should the permissions be set up if a user is granted ##MS_DatabaseManager## on the server level and the user should be able to run e.g. ALTER DATABASE [db] SET ALLOW_SNAPSHOT_ISOLATION ON on existing databases?

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

ID: 1e24f87c-7a50-8dbe-a818-c603c979c5e4
Version Independent ID: 07401c95-1053-03ff-abc6-b6b18c5f3fef
Content: ALTER DATABASE (Transact-SQL) - SQL Server
Content Source: docs/t-sql/statements/alter-database-transact-sql.md
Product: sql
Technology: t-sql
GitHub Login: @WilliamDAssafMSFT
Microsoft Alias: wiassaf

@kimjamia -- Kim, thank you for your feedback.  Please consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

@WilliamDAssafMSFT -- William, please look into this issue.

Hi @kimjamia, thanks for the question.
Members of the  ##MS_DatabaseManager## fixed server role can create and delete databases. Members of the ##MS_DatabaseManager## role don't necessarily have any permissions inside databases, but they can become the owner of a database they create. Then, they would gain permissions inherent to the database owner. You can change the owner of a database, and then the login only in the ##MS_DatabaseManager## role would not be able to connect as the dbo user.
The sentence there in the ALTER DATABASE Permissions heading does not need to change to address this new role specifically. Hopefully that helps you understand this new and limited server role.
More information:

https://techcommunity.microsoft.com/t5/azure-sql-blog/new-server-roles-for-azure-sql-database-and-sql-server-2022-in/ba-p/3428433:

##MS_DatabaseManager##
Members of the ##MS_DatabaseManager## fixed server role can create and delete databases. A member of the ##MS_DatabaseManager## role that creates a database, becomes the owner of that database, which allows that user to connect to that database as the dbo user. The dbo user has all database permissions in the database. Members of the ##MS_DatabaseManager## role don't necessarily have permission to access databases that they don't own.
For Azure SQL Database, it is recommended to use this server role instead of the special dbmanager database level role that exists in master (Azure SQL DB only) and was previously used to reach the same goals but required a User account in master-database.

https://learn.microsoft.com/en-us/sql/relational-databases/security/authentication-access/server-level-roles?view=sql-server-ver16#fixed-server-level-roles-introduced-in-sql-server-2022

If you have any further questions, this may not be the best forum. I recommend Microsoft Q&A (SQL Server), Microsoft Q&A (Azure SQL), SQL Server Feedback, Microsoft Support, or your support partners.


don't known how to describe this,look the image

this will only happened on LSP
file type:go
linter:gopls
nvim:0.3.5
when a go file has a error.
when open it and change buffer to it,this will happen

I can't read Chinese in screenshot, so I can't help much there. Have you tried using git pull to get the latest version and trying it again? It might be related to #2475, which was already merged.

Can you translate the error message from Chinese to English, for the benefit of others, or reproduce it with the locale settings set to English?

@w0rp

@w0rp
this can be repeated very easily
use golang's lsp linter gopls and config it right
create two .go files in a dir under gopath
write some code in the two .go files
open both two files,and write some wrong code in one file.
now use :bnext or :bprev to change buffer.
this problem will happen

That looks like the problem that was already fixed in #2475. Maybe @andreypopp can have a look.

@w0rp
the using linters gopls gofmt golint govet
situation 1 use linter gopls
there is a 100% to triger this problem when switch buffer between file with errors and file has no error
situation 2 delete linter gopls
there is still a low chance about 10% to triger this problem when switch fastly between file with errors and file has no error

Okay, git pull and try it again. I made it grab the lines from the buffer to get the number of lines, and constrain the highlight ranges by the number of lines. I would update the tests, but none of the tests that were written are run by Travis CI anyway, because the Docker image hasn't been updated to also test with a version of NeoVim that's new enough.

@w0rp nothing changed
did i config the ale right?
here is the images.

@chenjie199234 wasn't able to repro with this exact version of neovim and set of linters. Any chance you can have a docker container with repro?

Right, I'm git reverting it until it works in that case.

Now I've removed support for it. I'll reopen the issue for adding support for it until it works again.


use the same rendererName variable when calculating isNative

Coverage remained the same at 74.487% when pulling b4a1b18ad4420849cd0eea16a7b0c70309bb4498 on igorko:play into 80b658fc7e9f125659b9a7b1a6284c8fba2a244e on mediaelement:master.


Mouse pointer is active on inactive screen

Microsoft PowerToys version
0.70.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Mouse Without Borders
Steps to reproduce
When the mouse cursor is on one machine, the mouse pointer on the other machine is active. It doesn't move, but any tooltip under the mouse, at the top centre of the screen, is activated. E.g. if there is a browser open with multiple tabs active, the tab in the middle shows its tab preview.
✔ Expected Behavior
On the previous version (not in PowerToys) the mouse did not activate the tooltip when not on screen.
❌ Actual Behavior
Mouse does activate tooltips, If over a browser, the tooltip is a distraction.
Other Software
Tooltip on Chrome Version 113.0.5672.127 (Official Build) (64-bit) is the one that showed this, but it's true of anything that responds to the presence of the mouse.

Maybe related to this - when I'm using a fullscreen RDP session on my primary PC and move the mouse off-monitor to another PC via MWB, the mouse transitions to the other PC, but the keyboard input stays bound to the fullscreen RDP session. The "known issues" section here says that this "hide mouse at screen edge" option should be a workaround for the keyboard problem, but that's not the case for me. Maybe the mouse-hiding option isn't working properly? That would explain this bug too.

Are you still experiencing this issue? /needinfo


Replace bool array by more compact vector

Related issue: #4062.

This fails with a -1 index as noted in #4062. Also, a vector is not any more compact than a stack array because a vector is just an array with more c++ isms around it

The old code caused an illegal memory r/w operation with index -1, while the new code causes an exception. That's a clear improvement, even if it does not fix your issue.
And std::vector<bool> "is a possibly space-efficient specialization of std::vector for the type bool", see documentation.

it only throws an exception if you use c++ exceptions which are often disabled due to perf and general usability problems (which is why we do it). A bool vector could be a bitmask underneath, but if it is, you take a perf hit instead - probably not a great tradeoff for a small stack var - especially since the compiler can optimize bool[] any way it sees fit


multiple download results windows bug

multiple download results windows when many batch downloads in one time or from multiple sources.
When some of downloads are finished, results window with current results may appear.
After compleating all downloads results window with true results also appears.
looks like this bug appeared in last grabber version(s)

It seems to come from the new "packing" system when using asynchronous downloads. When there are a lot of already existing images, there will be a race condition causing the download to be finished when it's actually not.


Software not hosted under ORNL GH organization are not listed any more

ADIOS, QMCPACK, Cabana, ArborX, Kokkos, etc. are not listed any more in the catalog.

I see ArborX, QMCPACK and Kokkos listed, but not ADIOS and Cabana.

Yes that is the correct site. Are ADIOS and Cabana under the ORNL GitHub organization? If not, you will need to let me know where they are and I can add them explicitly.

They are not under ORNL.  They were listed at some point in the past.  You will find multiple hit if you search for them in this repo.  The repo are "ornladios/ADIOS2" and "ECP-copa/Cabana".
"arborx/Arborx" and "kokkos/kokkos" have the same issue (were added in the past, used to be listed but are not showing anymore)

@jarrah42 May also want to add adamantine-sim/adamantine

"arborx/Arborx" and "kokkos/kokkos" have the same issue (were added in the past, used to be listed but are not showing anymore)

ArborX and Kokkos are showing up.

You're right, I just checked and only Cabana is not showing from the one I had initially reported.

ADIOS2 shows up, but not ADIOS (ornladios/ADIOS). Dunno if both should be there.

ADIOS2 is the right thing.

So is it only Cabana that is missing?

So is it only Cabana that is missing?

And adamantine (missing/new)


slither.io client ?

hello,
will you make a slither.io client ? Because maybe i will do one and some help would be great :)
greats,
david

I am in the situation..
www.github.com/masonburdette/slither-client

@MasonBurdette to make it public just allows noobs to get free bots. This will destroy the game like agar.io is destroyed. I will help you, but only if you don't publish it.

@MasonBurdette is there a wiki already or not ? cannot find one

@agarioBotDev
I am not making slither.io client and not planning to do any.
I made agario-client for learning, don't see anything new i can learn making slither client so i'm not interested in it.

@agarioBotDev skype?  i code agar bots too (without agario-client) and an working in slither.

@MasonBurdette yes, my site is: d-bots.com, but i will not make my skype public - just write an email to<EMAIL_ADDRESS>then i can give you my skype.


Ticket/1.6.x/10490

If we do not find a macaddress when running ifconfig -a (case is
described in #10490 and is reproduceable if you deconfigure all
interfaces despite lo0) we get the error:
Could not retrieve macaddress: private method split' called for
nil:NilClass

Return nil as a macaddress if no macaddress can be found. The same
behaviour is already used for macaddresses on windows.

This is a great patch Stefen. Can I ask 1 simple request though ... since the bug was introduced through Virtuozzo interfaces can we add another fixture & example for that particular case as well?
I know logically what you have done should work - but I wouldn't like to see a potential change in the future to cause a regression on this particular ticket. As unlikely as it is, consider it an insurance policy :-).
I found a good sample here:
http://serverfault.com/questions/214633/change-the-order-of-ip-addresses-returned-by-ifconfig

Ken, thanks for the example ifconfig output.
Also, the tests did pass even if you do not patch macaddress.rb, because the split error is caught in Facter::Util::Resolution#value and the macaddress will be nil anyways. So the new patch now also adds a test for util/macaddress.rb.


Backstage Addon

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@5herlocked  please provide feedback

Note that this is already a working version. What is needed is, like explained in the .md file, the removal of manually created external dependencies (e.g. RDS), and use cdk created instead. I suggest you first insure this version works for you too following the instructions and then evolve it with improvements. This is a dev branch we can use it to iterate.

I can confirm, the blueprint as it is now, works after manually creating resources.


GenesisConfig for Next_Length_multi & Target_Block_Size

This PR Fixes #178.
It intrudces optional set up for those 2 storage values,
if not provided the default ones will be in use

Thanks. Can you add some docs to the new pub items?

Yes


page-object API should be more similar to watir API

I am pretty sure I am not the only one that has experience with watir API, and gets confused because page-object gem uses different names for some page elements.
Examples:

HTMLwatirpage-object

tdtdcell

hiddenhiddenhidden_field

imgimgimage

aalink

lililist_item

ololordered_list

ppparagraph

input type="radio"radioradio_button

selectselectselect_list

tdtdcell

textareatextareatext_area

ululunordered_list

I am all for making the API human-friendly, that is what watir tried to do in the beginning. After a few years we have noticed that people are getting confused with the API that sometimes uses the same names as HTML, and sometimes does not.
I would suggest that you leave the existing API as is and to create aliases for the people that are familiar with HTML and watir APIs.
Example: we should be able to access unordered list with both unordered_list and ul.
For more people thinking page-object API should be more similar to watir API, take a look at Alister's blog post (and comments): http://watirmelon.com/2012/06/04/roll-your-own-page-objects/

class TestPageObjectAccessor
include PageObject
in_frame(:id=>'ts_frame_01')do |frame|
text_field(:first_name,:id=>'ts_first_name',:frame=>frame)
end
def perform_text_actions
self.your_name="Raja" ; sleep 3
end
end
I cant able to create keyword for this code.@bret could you please help me out ?

I don't know what you mean by "keyword", but you haven't defined #your_name anywhere... You have a #first_name element defined though


Rename Algorithm enum cases to PascalCase

enum Algorithm {
    ED25519 = "ed25519",
    SECP256K1 = "secp256k1"
}

should be
enum Algorithm {
    Ed25519 = "ed25519",
    Secp256k1 = "secp256k1"
}

for consistency

Probably non of our users is using the symbols Algorithm.ED25519 and  Algorithm.SECP256K1 directly. However, the type is exported via @iov/tendermint-types, so this should be considered a breaking change. Thus we don't include it in 0.6.x.
I just verified that Algorithm is the only enum left with SCREAM_CASE value names.

Hi @webmaster128 I would like to take this up. Can you point me in the right direction

Hey @lexcorp16, you are very much welcome, thanks!
You can build this TypeScript project by running yarn build in the repo root. We use lerna to build all the sub-packages from there. The file to be changed is https://github.com/iov-one/iov-core/blob/master/packages/iov-tendermint-types/src/keys.ts. After that, all the places where we use Algorithm.ED25519 or  Algorithm.SECP256K1 must be updated. If you can build the whole project after the change, you are probably done.

Thanks a lot @webmaster128 the PR is here https://github.com/iov-one/iov-core/pull/388


Cache index names for 30s

This should reduce load on elasticsearch, as we don't need to fetch
the real name of every alias on every query.  On the other hand,
getting the index name of an alias is a cheap operation, so it may not
save much.

Trello card

I don't think indexing will be affected (because that doesn't use this code), but querying will temporarily break if you delete the old indices under 30 seconds after the alias switches over, yes.

I don't think indexing will be affected (because that doesn't use this code), but querying will temporarily break if you delete the old indices under 30 seconds after the alias switches over, yes.

How temporary is the break? I think it worth the small risk if it's just that 30 seconds, or is the break longer?

It's just that 30s, as after that the redis key will expire and it'll be fetched by querying elasticsearch again.

querying will temporarily break if you delete the old indices under 30 seconds after the alias switches over

Perhaps we could update the cached index aliases as part of the reindexing job. It would be ideal if we could prevent user requests from failing.

Going to close this for now, because it would be good to do some performance testing, which I don't have time for at the moment.


Properly initialize state block

Fixes the loading screen in Guild 2 when D9VK gets compiled with MSVC.
Might also fix the main menu with the constant opt branch.
Also fixes the main menu issues with the constant-opt branch according to Riesi.

Cheers. I'm very surprised that none of the compilers, MSVC, Clang, MinGW/GCC gave me a warning about this :-/


This should fix the null pointer exception that we are getting in Fea…

…tureLoader in MyNextClassMainActivity

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

support/src/main/java/edu/byu/support/feature/FeatureLoader.java, line 91 at r1 (raw file):

	public static Feature getFeatureById(String featureId) {
		if (allFeatures == null) {

When would allFeatures be null? Do we have another problem here?

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

a discussion (no related file):
Previously, romrell4 (Eric Romrell) wrote…
Test the onDestroy pattern one more time

Done

Comments from Reviewable

Reviewed 2 of 2 files at r4.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Create a proper tutorial for Neon

Create a step-by-step tutorial that will run users through the basic functionalities of Neon. This can replace the blank test.

done 69fd5f59a85741871da04ec230325c6a3caf6270
also in the wiki