Rust 1.60 docker update

update docker image to rust 1.60
use cargo-chef to improve image building performance
removed redundant binary target definition (due to https://github.com/LukeMathWalker/cargo-chef/issues/128)

If we're going to bump the Rust version in the docker file, shouldn't we also do it for CI?

@adlerjohn it looks like CI automatically uses the latest rust version: https://github.com/FuelLabs/sway/runs/6236542491?check_suite_focus=true#step:3:15

Do we document anywhere what this Dockerfile is for or how we use it? I've never used Docker and am currently unaware of how we use this in the Fuel ecosystem.

I'm in the exact same boat as Mitch around Docker, unfortunately. Would love to read up on how it's used in Fuel if such a document exists?

Do we document anywhere what this Dockerfile is for or how we use it?

I don't think we document it, but this is the base image that our Github runners use when running our CI. So this change makes our CI use Rust 1.60.

All I know is that it creates a docker image with a compiled version of forc. @sezna is this currently being used?


Envoy Filter Only Send one Message per Stream

Title: Envoy Filter Only Send one Message Per Stream
Description:
We are using envoy filters
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
# ...
    patch:
      operation: MERGE
      value:
        typed_per_filter_config:
          envoy.filters.http.ext_proc:
            "@type": "type.googleapis.com/envoy.extensions.filters.http.ext_proc.v3.ExtProcPerRoute"

That filter calls our GRPC server, which measures some metrics, including the number of messages per stream.
When we load test our GRPC server locally, using GRPCurl, for example, we can see the number of messages in the same stream is around 10k for a 5-second load test. But when we measure the stream size of the actual GRPC calls from the Envoy filter, all are size 2.  Also, the number of EOFs matches the number of calls to the GRPC server. So, there are only two messages in each stream, one of which is an EOF.
It looks like we are not using the best of the streaming capability during these calls, which can reuse the same GRPC request to send multiple messages, creating new streams for each message instead. If we do that properly, we may considerably reduce the latency around network protocols shaking hands.

As a reference, here is a simple comparison of performance from the same GRPC Server interacting with the same GRPClient:
Both tests use GRPCurl as a client.

I have tested different combinations of number of clients process, but the results are quite similar.
The stream approach is much faster than the single message approach.


TS doesn't watch files for changes

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.85.2
OS Version: All

Steps to Reproduce:

Move a file somewhere or rename it
See how Typescript doesn't react to this change at all

You constantly have to reload VSCode window for changes to show.

Do you have tsconfig.json/jsconfig.json? Do you have files or include? Do you move file to location that matches include?

@IllusionMH here's reproduction repo:
https://github.com/aleksey-hoffman/sigma-file-manager/tree/v2
Reproduction steps:

Go to src/router/routes.ts
Rename the import path to something else, e.g. '@/pages/homeupdated';
Rename the directory to new name `homeupdated and see the error still showing


[Adjust Rule] SIM109 - Suggest tuple instead of list

Desired change

Rule(s): SIM109
Adjustment: Suggest rewriting to a tuple instead of a list

Explanation
See rationale here
Example
This change would update the error message as follows:
- SIM109 Use 'a in [b, c]' instead of 'a == b or a == c'
+ SIM109 Use 'a in (b, c)' instead of 'a == b or a == c'

@mxr Thank you for the suggestion! I've added it in flake8-simplify==0.14.4 :-)


java.lang.NoSuchFieldException: bv

Server Version: git-Paper-196 (MC: 1.16.3)
Plugin Version: BetterChairs version 0.11.6

Describe the bug
Sometimes the plugin will print an error in console, not really what causes it though:
[23:32:31 WARN]: BetterChairs could not apply protections to a Chair at Location{world=CraftWorld{name=world},x=-253.0,y=62.0,z=-255.0,pitch=0.0,yaw=0.0} (java.lang.NoSuchFieldException: bv)
To Reproduce
Steps to reproduce the behavior:
Not entirely sure

Thanks for your report!
I know what is causing the error and am going to take a look as soon as I've got some time.

The bug should now be fixed. A new plugin version should be available within the next days ^^

The bug should now be fixed. A new plugin version should be available within the next days ^^

Thank you! :)


Testing, please ignore. ajfnyw

This is a bug bounty test. Please do not approve this! ajfnyw

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


Add DN name to accept_result struct

Previously, it was possible to get DN using dn_callback passed to
credentials object. This callback is invoked on every handshake (if
'client_auth' is set to true) and accepts two parameters, subject and
issuer. But there is no way for the user of the tls library to connect
particular client connection with the DN string. Also, it's not the best
way to obtain the DN string because the handshake is delayed until the
first read or write attempt is made.
This commit solves the issue by

adding a DN field to the accept_result struct
propagating DN name to the accept_result
forcing TLS handshake after the connection is established and before
the accept_result is returned in order to get DN field populated

In order for this mechanism to work the user must use 'seastar::tls::listen'
method directly and not 'wrap_server'. Also, the user have to pass
the credentials with client-auth set to 'REQUIRED' to the 'listen'
function.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

If the handshake is more eager, does this make an exhaustion attack more easy?

I'm curious as well. Normally TLS exhaustion attack will actually require a handshake anyway. Here we're just starting the handshake as soon as connection is established. Without this the attacker can open a lot of connection without doing the handshake. So in my point of view this looks like reduction of the attack surface.

Are there concerns over when the set_sockopt can be called? I.e., are there scenarios where socket options must be set before the handshake?

I don't think that there are any concerns regarding this.


2.33.0 results in error 401: Props Authentication Token not found

Describe the bug
With jfrog/jfrog-cli-v2-jf:latest the following:
jf config add artifacts.my-company.de --artifactory-url=https://artifacts.my-company.de/artifactory --user=some-user --access-token=foobar
jf config show
jf rt ping

results in:
[Error] server response: 401 Unauthorized
{
  "errors": [
    {
      "status": 401,
      "message": "Props Authentication Token not found"
    }
  ]
}

When using jfrog/jfrog-cli-v2-jf:2.32.0 everything is fine.
Current behavior
See description
Reproduction steps
No response
Expected behavior
No response
JFrog CLI version
2.33.0
Operating system type and version
Official docker image
JFrog Artifactory version
EnterpriseX license 7.33.12 rev 73312900
JFrog Xray version
No response

I can confirm this bug:
jf config add<EMAIL_ADDRESS>--access-token=XYZ --url=https://artifactory.mycompany.com/ --artifactory-url=https://artifactory.mycompany.com/artifactory --distribution-url=https://artifactory.mycompany.com/distribution --xray-url=https://artifactory.mycompany.com/xray --interactive=false --overwrite=true

jf docker push artifactory.mycompany.com/docker-registry/image-name:image-tag

[Error] failed while attempting to get Artifactory version:
server response: 401 Unauthorized
{
  "errors": [
    {
      "status": 401,
      "message": "Props Authentication Token not found"
    }
  ]
}

In version 2.32 this works.

Hi @g3n35i5 and @famod,
Thanks for reporting this issue.
Our apologies for the inconvenience caused by this.
Are you using a reference token or a full JWT token? Also, can you please test the same command without the user field?

Hi @g3n35i5 and @famod,
API key should be used as a password and not as an access token.
To avoid breaking users that used it as access-token we made it work now with a warning.
JFrog CLI 2.34.0
Looking forward to your feedback on it.

@famod 2.34 released a few minutes ago.
Glad it helped!

@sverdlov93
I'm encountering the error, but 2.33 hasn't hit the apt repo yet. Any timeline when this will released?

@mjkl-gh
Do you mean 2.34?
Also, did you try using it as a password(as it should be used)?
jf some command --user="your username" --password="your API key"

@mjkl-gh Do you mean 2.34? Also, did you try using it as a password(as it should be used)? --user="your username" --password="your API key"

Yes we changed it in our script, but I was wondering about the release of 2.34 at the same time.

I believe it will appear soon. It sometimes takes some time for new versions to appear on some tool providers.


Separate MIN_MATCH into properly separated values.

STD_MIN_MATCH is 3, and is used to stay compatible with the deflate standards.
WANT_MIN_MATCH is set to 4, and is used to indicate to the code what the minimum match length we intend to accept is.

Having WAN_MIN_MATCH set to 4 also means that we now skip the last round of longest_match in the sliding window. This is good because we currently hash 4 bytes, so checking for a match when there is only 3 bytes left is pointless.
The performance difference with this PR appears to be better (as it should), but the difference is slight (around +0.05-0.10% faster), so it is within the margin of error.
This PR also removes the partial, dead, broken and untested support for STD_MIN_MATCH != 3.
As far as I understand, setting this to something else would cause the output data to deviate from the deflate standard and produce an incompatible output stream, unless the tree tables are also regenerated to match the changed offset. And STD_MAX_MATCH would probably have to be reduced by the same number as well. I am not 100% certain of this, but I think it is easier to experiment with this kind of thing now that the variables are split.
I am not 100% certain about the correctness, but the current iteration makes sense to me and the compressed output is identical to pre-PR in my tests.

Actually we will probably need that code for fast-zlib so..

Needs rebase.

I think adding MIN() does not belong in this PR, but I agree those should probably be converted in another PR.


pinentry: remove multiple outputs

Motivation for this change
Previously pinentry was a package with multiple outputs. This caused lots of rebuilds and always needs the dependencies of all flavors to build. Fixes #133156 #124753.
This PR refactors the different pinentry flavors into separate package builds and keeps the old interface for compatibility. As the attribute name pinentry would be misleading now it was renamed to pinentryFlavors. The old multiple outputs based interface is aliased for compatibility since this was not only used in the gnupg NixOS module but also the corresponding home-manager module.
Things done

[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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages 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/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 133542 at 010aa08a run on x86_64-linux 1

10 packages built successfully:

kwalletcli
nixos-install-tools
passExtensions.pass-tomb
pinentry-emacs (pinentry-curses)
pinentry-gnome
pinentry-gtk2
pinentry-qt
python38Packages.trezor_agent
trezor_agent (python39Packages.trezor_agent)
tomb

6 suggestions:

warning: missing-patch-comment
Consider adding a comment explaining the purpose of this patch on the line preceeding.
Near pkgs/tools/security/pinentry/default.nix:48:9:
   |
48 |         ./autoconf-ar.patch
   |         ^

warning: build-tools-in-build-inputs
sudo is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/os-specific/linux/tomb/default.nix:16:3:
   |
16 |   buildInputs = [ sudo zsh pinentry-curses ];
   |   ^

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

warning: unclear-gpl
gpl3 is a deprecated license, please check if project uses gpl3Plus or gpl3Only and change meta.license accordingly.
Near pkgs/os-specific/linux/tomb/default.nix:40:5:
   |
40 |     license     = licenses.gpl3;
   |     ^

warning: missing-patch-comment
Consider adding a comment explaining the purpose of this patch on the line preceeding.
Near pkgs/tools/security/pinentry/default.nix:50:9:
   |
50 |         (fetchpatch {
   |         ^

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/os-specific/linux/tomb/default.nix:29:3:
   |
29 |   installPhase = ''
   |   ^

Looks like a sensible approach to me. I think we can leave out the compatibility hack if we also have a corresponding home-manager PR.

Thanks for the feedback. I've updated the PR and resolved the conflict with master (the qt variant does not build on aarch64-darwin).
I will open a home-manager PR later and remove the alias.

@fpletz may I ask what's the status here? :)

Closing as dead.

It's still a good idea, as it improves build closure size and reduces rebuilds due to unrelated changes in everything that depends on pinentry.

Reopening and working again on this PR since it makes more sense than #277221.

Reopening and working again on this PR since it makes more sense than #277221.

We should have opened a new one instead of reviving a 3 year old PR that has been closed since a few months. :/

Please cherry-pick 945f9fe, too

Pinentry with gtk2 support will stil be available.

Should we use mkDefault for all DEs, rather than somewhat arbitrarily deciding that some of them don't need it?

Not opposed to that. My thought there was that for some DEs like Gnome the users' choice is pretty unambiguous while for others like sway the user could conceivably want something else than our default.

I've rebased the PR against master to fix the failing ofborg eval.
What's the plan to continue this? We should move forward with this and make pinentry a lot smaller.

@SuperSandro2000 ofborg eval still fails. What's you take on the remaining issue with multiple DEs enabled? I'm currently in favor of going back to the old way.

Eval is now fixed. goldwarden was using the plain pinentry attribute.

What's your take on the remaining issue with multiple DEs enabled? I'm currently in favor of going back to the old way.

How do you want to do that?
We could set custom priorities to mimic the old behavior.

I just did a quick rebase to fix the merge conflict.
Can we finish this? I don't think there where any points left.

Goldwarden is a GTK app, shouldn't it be using gnome3 pinentry?

I will steal this image!

@ofborg eval

I've just noticed that we also need to add a pinentry to plasma6 and I just did that.

Here goes nothing 🎉

This broke eval:

       error: undefined variable 'pinentryFlavor'

       at /nix/store/ngy0l9sly1qqg049584bwbiz9bb5x1mv-nixpkgs-src/nixos/modules/services/security/yubikey-agent.nix:40:49:

           39|     # yubikey-agent package
           40|     systemd.user.services.yubikey-agent = mkIf (pinentryFlavor != null) {
             |                                                 ^
           41|       path = [ config.programs.gnupg.agent.pinentryPackage ];
❯

Fix in https://github.com/NixOS/nixpkgs/pull/294771.


Legend Setting

To whom it may concern,
This is really a great visualization library in C#
and I wonder if there is any new update on "Legend Function".
Legend, a rectangular showing the title of each Lines on the Figure.
Thank you and looking forward to your precious reply.
Best regard

@alex555176 a legend feature is not currently implemented and would be somewhat hard to add with the current code, but I'm working on version 3 of ScottPlot and I'll make a note right now to be sure support for a corner legend gets added into the next release. Thanks for the suggestion!

@swharden
Understood, and we are indeed looking forward on your latest version 3 ScottPlot.

ScottPlot v3 is pretty far along now, so a legend will be supported pretty soon...

I just released ScottPlot v3 and the cookbook demonstrates how to create a plot with a legend:
var plt = new ScottPlot.Plot(600, 400);
plt.PlotScatter(dataXs, dataSin, label: "first");
plt.PlotScatter(dataXs, dataCos, label: "second");
plt.AxisAuto();
plt.Legend();
plt.SaveFig("legendDemo.png");


IP Range for MS SQL?

Best practices are that database servers should not be available to the entire Internet, so we have enabled the MS SQL firewall and we allowlist IPs that can connect.
What are the IP ranges that Viva Goals would use when connecting to our database?

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

ID: 7074f9af-0e5c-279a-8ea1-c2d1efc5e37d
Version Independent ID: 7074f9af-0e5c-279a-8ea1-c2d1efc5e37d
Content: Microsoft SQL Server integration
Content Source: Viva/goals/ms-sql-server-integration.md
Service: viva
Sub-service: viva-goals
GitHub Login: @RaSanders-MSFT
Microsoft Alias: rasanders

Hi @arlogilbert, thank you for your feedback.
These IP addresses should be included here but not yet:
https://learn.microsoft.com/microsoft-365/enterprise/urls-and-ip-address-ranges
Because all Microsoft 365 endpoints data displayed in this article is autogenerated by the Microsoft 365 endpoints web service, the content team cannot resolve issues directly related to this data.
We recommend that you contact your organization’s TAM (Technical Account Manager) to escalate your inquiry to the engineering team associated with the endpoint in question. If you do not have a TAM assigned to your organization, please contact Microsoft Support.
@RaSanders-MSFT, maybe you can provide additional information. Thanks!

Similar to https://github.com/MicrosoftDocs/Viva/issues/490.


Support for usb controller

Would it be possible to add a support for usb controller?

I am not willing to implement this.
It will be easy to attach RC Controller to SITL using proxy FC https://github.com/iNavFlight/inav/pull/9365#event-12632397812 soon.

Alright, great


Rname registry to kcl-registry

Name the registry a special kcl-registry instead of a common registry to reduce the impact on developers' local.

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[ ] Y

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[ ] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

If we reach an agreement I will continue to submit the same PR for kcl-lang/cli

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 39.812%

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
3085

Relevant Lines:
7749

💛  - Coveralls


Add Variable to allow setting a custom Publish Name

Here is the second PR to allow a custom published name in MQTT. I did not add change log data as I figured you would need to do that after the PR is merged and you cut a release. If you want me to include that let me know.

Sweet thanks! Forgot to mention, can you open issues for this change and the other one? Then I can reference in the changelog when I publish the release tomorrow

Sure thing

#157


Create ApiCoinSelection data type.

Context
In order to support users that wish to use a hardware wallet, we would like to expose our coin selection functionality as a new API function selectCoins. This function will take a list of payments as its input, and produce a coin selection as its output.
Decision
Provide the following API datatypes: (Values of ApiCoinSelection will be provided as output from selectCoins.)
data ApiCoinSelection (n :: NetworkDiscriminant) = ApiCoinSelection
    { inputs :: !(NonEmpty (ApiCoinSelectionInput n))
    , outputs :: !(NonEmpty (AddressAmount n))
    } deriving (Eq, Generic, Show)

data ApiCoinSelectionInput (n :: NetworkDiscriminant) = ApiCoinSelectionInput
    { id :: !(ApiT (Hash "Tx"))
    , index :: !(ApiT Word32)
    , address :: !(ApiT Address, Proxy n)
    , amount :: !(Quantity "lovelace" Natural)
    } deriving (Eq, Generic, Show)

The above types should correspond to the following Swagger definitions:
  ApiCoinSelection: &ApiCoinSelection
    type: object
    required:
      - inputs
      - outputs
    properties:
      inputs: *transactionResolvedInputs
      outputs: *transactionOutputs

x-transactionResolvedInputs: &transactionResolvedInputs
  description: A list of transaction inputs
  type: array
  minItems: 0
  items:
    type: object
    required:
      - id
      - index
      - address
      - amount
    properties:
      address: *addressId
      amount: *transactionAmount
      id: *transactionId
      index:
        type: integer
        minimum: 0

Acceptance Criteria

[x] There must be an API type ApiCoinSelection that meets the definition provided above.
[x] There must be an API type ApiCoinSelectionInput that meets the definition provided above.
[x] There must be roundtrip and golden JSON serialization tests for the above types.

Development

[x] Create the ApiCoinSelection type.
[x] Create the ApiCoinSelectionInput type.
[x] Create the corresponding Swagger definitions.

QA

[x] Create roundtrip and golden JSON serialization tests.

PRs

Number
Base

#1152
master

Same remark as #1135, the QA could deserve some more love :)


removed unused line

$service = new Google_Service_Drive($client);

This line is not seems to be necessary here.

Agree


Create an endpoint that shows agora internals (for debugging)

Currently it is very hard to debug complex problems like: https://github.com/bosagora/agora/issues/1552 in our production environment.
It would be helpful to have a http endpoint that exposes the most important internals of the Agora instance to help debugging. This includes:

last 10 known preimages for each validators and their corresponding heights
last 5-10 SCP messages received/sent
latest block header/maybe transaction summary
summary of transactions in the transaction pool
what candidate a node would propose, if it was its turn
what nodes we are connected to
which nodes are banned
...

Some of these information is already available through other endpoints, but it would be good to see them all on the same endpoint.
Some information like know preimages are also known, if we ssh'd into the machine and read the sqlite db, but that is very cumbersome to do.
Some information like SCP messages are only available in the logs, if we set the level to Trace.
Some information like currently banned nodes are in its own file.

What I was actually thinking about is expose an UNIX socket that would allow one to directly interact with the nodes. This would also allow us to control nodes, e.g. by setting log level and co. But all things consider, going the HTTP route, with a control interface, might be much simpler to implement.

For the moment this should be added to the admin interface.
One example is that we can now dynamically reconfigure loggers: https://github.com/bosagora/agora/blob/2809791954133da4c352d612d7347ee667e65312/source/agora/api/Admin.d#L30-L42

last 10 known preimages for each validators and their corresponding heights

Already part of the API: https://github.com/bosagora/agora/blob/2809791954133da4c352d612d7347ee667e65312/source/agora/api/FullNode.d#L246

last 5-10 SCP messages received/sent

Currently not very useful, but will keep it in mind for the future.

latest block header/maybe transaction summary
summary of transactions in the transaction pool

We already have getBlocksFrom, but for transactions, we need an endpoint to dump the tx pool / enrollment pool.

what candidate a node would propose, if it was its turn

I think we're better off just logging it for now.

what nodes we are connected to
which nodes are banned

Yup.

Some information like SCP messages are only available in the logs, if we set the level to Trace.

I think the logger reconfigure mentioned above helps with this.

last 10 known preimages for each validators and their corresponding heights

Already part of the API:
https://github.com/bosagora/agora/blob/2809791954133da4c352d612d7347ee667e65312/source/agora/api/FullNode.d#L246

last 5-10 SCP messages received/sent

Currently not very useful, but will keep it in mind for the future.

latest block header/maybe transaction summary
summary of transactions in the transaction pool

We already have getBlocksFrom, but for transactions, we need an endpoint to dump the tx pool / enrollment pool.

what candidate a node would propose, if it was its turn

I think we're better off just logging it for now.

what nodes we are connected to
which nodes are banned

Yup.

Some information like SCP messages are only available in the logs, if we set the level to Trace.

I think the logger reconfigure mentioned above helps with this.

I think it would be nice to have all that information show up as a result of 1 HTTP/REST call, so we don't have to reconfigure/scavenge the logfiles. We currently have 5 validators, logging into each one of those machines and trying to find relevant log messages will still take a lot of time versus just opening up 5 browser tabs. We might not be able to find the exact reason of the problem we are trying to  debug, but we would hopefully at least know if it is SCP/Network/Something else

I think it would be nice to have all that information show up as a result of 1 HTTP/REST call, so we don't have to reconfigure/scavenge the logfiles.

For that, maybe a EKL stack would be better suited ?


[1.17 - 20w48a] Snapshot release

Is there a way to get the snapshot release somewhere?
This mod is incredible we really want it on the server!

The Snapshot Version is in the Minecraft Launcher
I dont believe there are mod version for Snapshots (could be though)

There's a lot I mostly make servers up to date, lots of mods are up to latest snapshot, not all tho


drivers/flash/qspi_nor_flash: Fix qspi_sfdp_read qspi_unlock on err

The commit fixes situation where failed call to ANOMALY_122_INIT
would call qspi_unlock on lock that have not yet been obtained.
Signed-off-by: Dominik Ermel<EMAIL_ADDRESS>

good catch

The PR has been surpassed by https://github.com/zephyrproject-rtos/zephyr/pull/40017


Add screenshot name to test report

In the case of a screenshot test taking multiple screenshots per test it would be useful to have the name of the screenshot displayed in the test report. This PR adds this small change.

I fixed code formatting for you 😃. As soon as the build passes, I'll merge this PR. Thanks!

I updated the code formatting for you, once the build passes, I'll merge this PR. Thanks!


⚠ K-Thorne has degraded performance

In 2d20903, K-Thorne (https://k-thorne.com) experienced degraded performance:

HTTP code: 200
Response time: 13178 ms

Resolved: K-Thorne performance has improved in 6df05c4.


HHH-12104 @Where clause is ignored when eagerly loading ManyToOne association

https://hibernate.atlassian.net/browse/HHH-12104
I'll need to do a lot more testing on this, but this is what I have so far.

Closing because I don't think this is a safe fix.


YouTube embed is not working with URL that contains "-" (Hyphen)

Hi,
YouTube embed is not working with URL that contains "-" (Hyphen).
Notice that even on Semantic website, the video is not working.
http://semantic-ui.com/modules/embed.html#/definition
Could this be resolved?
Thanks!

The video in SUI was taken down unfortunately. I have this fixed in next, but have been holding off for releasing everything as a new minor version 2.2

Thanks for the reply jiukic. Do you know when v2.2 will be released? Our company is using Semantic UI in the production at the moment and we're experiencing handful of YouTube related problems due to this specific bug.

I'll try to bundle up 2.2 in the next couple weeks.
You might consider using the next branch as an NPM dependency to get any changes now.

Additionally if your'e experiencing any other bugs with embed don't hesitate to report (with test cases), this is generally a component that I have had less feedback on.

Is this issue resolved? Thanks.

Yes in unreleased 2.2 please hold

When would this be available?
ᐧ
On Thu, May 19, 2016 at 11:50 PM, Jack Lukic<EMAIL_ADDRESS>wrote:

Yes in unreleased 2.2 please hold
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/Semantic-Org/Semantic-UI/issues/3888#issuecomment-220530708


Use more ES6 syntaxes in the real world example

Convert the async example files to using fat arrows, spread operators and stateless components.
This is a follow up to what I started in #1942 and #1945
References #1800

Thanks!
The example is still working and code changes reflect those already merged in  #1942 and #1945

I tweaked this do don't worry!

@gaearon I don't. I'm really happy to get some advice so I can make my next PRs better suited to the conventions.


bug: fix the Release CI to run on all

Make sure the Release CI for this repo runs on all files, not only on diff

partially fixed with #14 but still missing dependecy between checkov in release CI and semantic releases


Add API Docs

Update SPI info
Update CI
Add DocC bundle
Fix the imports

Codecov Report

Merging #85 (f874308) into main (d07f434) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head f874308 differs from pull request most recent head 673e897. Consider uploading reports for the commit 673e897 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #85   +/-   ##
=======================================
  Coverage   84.15%   84.15%
=======================================
  Files           7        7
  Lines         303      303
=======================================
  Hits          255      255
  Misses         48       48

Flag
Coverage Δ

unittests
84.15% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

Sources/FluentSQLiteDriver/Exports.swift
100.00% <ø> (ø)

...FluentSQLiteDriver/FluentSQLiteConfiguration.swift
100.00% <ø> (ø)

...rces/FluentSQLiteDriver/FluentSQLiteDatabase.swift
82.80% <ø> (ø)

...ources/FluentSQLiteDriver/FluentSQLiteDriver.swift
64.51% <ø> (ø)

...rces/FluentSQLiteDriver/SQLiteError+Database.swift
87.50% <ø> (ø)

...ources/FluentSQLiteDriver/SQLiteRow+Database.swift
92.85% <ø> (ø)


Spring Data Cosmos MultiDatabase sample is not working

Query/Question
I tried the multidatabase samples on the folder which is new spring data cosmos release version 3.4.0, but it does not work. can you please provide a complete sample with pom.xml ?
The sampleapplication run method never switch the databases always called the first database in the property file.
https://github.com/Azure/azure-sdk-for-java/tree/master/sdk/cosmos/azure-spring-data-cosmos/src/samples/java/com/azure/spring/data/cosmos/multidatasource
https://github.com/Azure/azure-sdk-for-java/tree/master/sdk/cosmos/azure-spring-data-cosmos
based on azure-spring-boot-starter-cosmos 3.1.0.

com.azure.spring
azure-spring-boot-starter-cosmos
3.1.0

Why is this not a Bug or a feature Request?
A clear explanation of why is this not a bug or a feature request?
Setup (please complete the following information if applicable):

OS: mac os 11.15.7
IDE : Intellij
Version of the Library used

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ ] Query Added
[ ] Setup information Added

@chenrujun  - I believe you are already looking at this issue here - https://github.com/microsoft/spring-data-cosmosdb/issues/554
Please close the issue on spring-data-cosmosdb repo, and let's work it out here.

sure, thanks very much.


terraform 0.12 syntax problem

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Terraform Version

Affected Resource(s)

artifactory_permission_target

Terraform Configuration Files

resource "artifactory_permission_target" "deploy" {
  name = "t012_Deploy"

  repo = {
    includes_pattern = ["**"]

    repositories = ["libs-dev", "libs-prod"]

    actions = {
      users = [
        {
          name        = "deployuser1"
          permissions = ["read", "annotate", "write", "delete"]
        },
        {
          name        = "deployuser2"
          permissions = ["read", "annotate", "write", "delete"]
        },
      ]

      groups = [
        {
          name        = "deploygroup1"
          permissions = ["read", "annotate", "write", "delete"]
        },
      ]
    }
  }

Debug Output

Panic Output

Expected Behavior
permission should be created just as with terraform 0.11
Actual Behavior

terraform plan (v0.12) stops with the error message below, but i'm not sure if this provider is 0.12 compatible. what would be the right syntax of the repo block for 0.12?
Error: Unsupported argument
on permission.tf line 4, in resource "artifactory_permission_target" "deploy":
4:   repo = {
An argument named "repo" is not expected here. Did you mean to define a block
of type "repo"?
Steps to Reproduce

access to artifctory pro
terraform plan

Important Factoids

References

#0000

So the provider will build for 0.12 in the next release which is currently being tested but you may also need to run terraform 0.12upgrade. See https://www.terraform.io/upgrade-guides/0-12.html. I don't know if there are any provider side changes we can do to fix this

i did run terraform 0.12upgrade but that hasn't touch the repo block. i was hoping you can paste a valid repo block (or fix/extend the documentation) in relation to the 0.12 release. do you have an ETA for the release?

@dextarr I've been working quite heavily with TF 0.12 and this provider recently. It looks like there is a syntax error in you TF that is causing it. There has been a subtle change in TF to the way it works out map/blocks which means your declaration no longer works the way it used to, see here for more info.
You config should now look something like:
resource "artifactory_permission_target" "deploy" {
  name = "t012_Deploy"

  repo {
    includes_pattern = ["**"]

    repositories = ["libs-dev", "libs-prod"]

    actions {
      users = [
        {
          name        = "deployuser1"
          permissions = ["read", "annotate", "write", "delete"]
        },
        {
          name        = "deployuser2"
          permissions = ["read", "annotate", "write", "delete"]
        },
      ]

      groups = [
        {
          name        = "deploygroup1"
          permissions = ["read", "annotate", "write", "delete"]
        },
      ]
    }
  }

@jamestoyer: actually i tried your suggestion before created the issue, but its not working.
using this exact tf file... //hint: no need to replace the dummy provider-variables until the file get parsed correctly, which is not the case :(
provider "artifactory" {
  url     = "what"
  api_key = "ever"
}

resource "artifactory_permission_target" "deploy" {
  name = "t012_Deploy"

  repo {
    includes_pattern = ["**"]

    repositories = ["libs-dev", "libs-prod"]

    actions {
      users = [
        {
          name        = "deployuser1"
          permissions = ["read", "annotate", "write", "delete"]
        },
        {
          name        = "deployuser2"
          permissions = ["read", "annotate", "write", "delete"]
        },
      ]
      groups = [
        {
          name        = "deploygroup1"
          permissions = ["read", "annotate", "write", "delete"]
        },
      ]
    }
  }
}

i still have error messages for the inner blocks during terraform plan:
Error: Unsupported argument

  on main.tf line 15, in resource "artifactory_permission_target" "deploy":
  15:       users = [

An argument named "users" is not expected here. Did you mean to define a block
of type "users"?

Error: Unsupported argument

  on main.tf line 25, in resource "artifactory_permission_target" "deploy":
  25:       groups = [

An argument named "groups" is not expected here. Did you mean to define a
block of type "groups"?

suggestions?

I believe that's also wrapped up into "breaking" changes in the 0.12 API.
Could you try:
provider "artifactory" {
  url     = "what"
  api_key = "ever"
}

resource "artifactory_permission_target" "deploy" {
  name = "t012_Deploy"

  repo {
    includes_pattern = ["**"]

    repositories = ["libs-dev", "libs-prod"]

    actions {
      users {
        name        = "deployuser1"
        permissions = ["read", "annotate", "write", "delete"]
      }

      users {
        name        = "deployuser2"
        permissions = ["read", "annotate", "write", "delete"]
      }

      groups {
        name        = "deploygroup1"
        permissions = ["read", "annotate", "write", "delete"]
      }
    }
  }
}

If that works, we can look at changing the names to make more sense. I'm generally using it in a dynamic block so I believe the below is syntactically the same as:
resource "artifactory_permission_target" "foo" {
  name = var.name

  repo {
    includes_pattern = var.includes_pattern
    excludes_pattern = var.excludes_pattern
    repositories     = var.repositories

    actions {
      dynamic "groups" {
        for_each = var.groups
        content {
          name        = groups.value.name
          permissions = groups.value.permissions
        }
      }
      dynamic "users" {
        for_each = var.users
        content {
          name        = users.value.name
          permissions = users.value.permissions
        }
      }
    }
  }
}

yes, now it's working as expected, thanks @jamestoyer!

I'm going to close this out. I've opened #48 to track changing the names


Support Addressable: add, getByName, Update and delete by id

🚀 Feature Request
Add missing Addressable functionality:

Creates a new Addressable and returns the ID of the new item if successful.
Get Addressable for  specified ID
Get Addressable for the specified name
Update the Addressable data
Delete  Addressable by specified ID

@difince @mkbhanda - I would make this enhancement a low priority given that Addressable is being removed with Hanoi.

Because Addressable are going to be remove in Hanoi, I am deleting this issue directly


[CLI] Get repos from git remotes

Clear and concise description of the problem
Instead of having  to add the repo as arguments to every repo-related command, it would be nice if the CLI could determine the repo itself using the git remotes.
Suggested solution
See above.
Alternative
No response
Additional context
No response
Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Do you have something like and autodetect in mind which uses the git remote and searches the woodpecker db by that to know which project you are in?


Add fetch reason string to storage and deltastorage apis

Refer: https://github.com/microsoft/FluidFramework/issues/6483
Api changes to add fetch reason to storage api calls so that server can record in telemetry and it can then help in debugging different scenarios.

ping


Add remotecheck check target

It should include a group of tests that have no dependencies on local paths so that it can be used to verify remote installations.

This should run under a chroot environment after packaging is complete. Docker would work too but it seems overweight.


Use make invocation in periodic for consistency

Calling a make target in periodic instead of script to be consistent with other prowjobs.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold


Project name conflict

Please note there is already a similar OpenOrienteering project named CupCalculator (not actively developed, but maintained):
https://github.com/OpenOrienteering/cupcalculator
http://www.openorienteering.org/apps/cupcalculator/

Thank you.


Remove version: foxglove.studio version 1.19.0

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/foxglove/studio/releases/download/v1.19.0/foxglove-studio-1.19.0-win-arm64.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 720)


change: change all tracing levels to trace to reduce verbosity

Seems like a pretty straightforward PR. Merging


[DO NOT MERGE] Auto PR to update dependencies of old_pilot_repo

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/pilot-presubmit.sh
fa601c4046a6be679bb337a44961050c47f2f453
link
/test pilot-presubmit

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


Validate TLD in email addresses

What is the context of this PR?
Validates the Top Level Domain (TLD) in email addresses. These changes match the TLD validation that Notify currently uses. Although the domain isn't checked for validity, it will stop single character domains and non-alpha characters.
How to review
Use the test_confirmation_email schema and test some emails with an invalid TLD e.g<EMAIL_ADDRESS>or<EMAIL_ADDRESS>Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

Validation of TLD working 👍. Just waiting on clarification about the double error message.


Update Composer dependencies (2024-01-15-00-18)

Visual regression test passed!
View the visual regression test report


Fix issue of "host" configuration not work correctly on Appium.

Solution: handle webdriverio specs correctly

Thank you!


Is there a way to set a custom content for the ganttchart extrainfo for task?

I want to set a html content, eg.:

column:
value

is this possible?

Currently, there is no plan to do incorporate this feature.


Fix go to declaration on Windows

The previous regex don't match with Windows style path and the go to feature wasn't working.
This one ensures that the path is captured until :line_number:

Good catch, thanks!


Incubation

Merge in PacketDPDKOperator.

@engebret Can you do me a favor and bump the version number of the toolkit to 2.8.0? It will save me from having to remember to do it afterwards. Thanks!


Create CI/CD for deploying multiple instances of the Next.js app

We need to setup a job to deploy multiple instances of the web app in our pipeline. Each web app instance should connect to a different sanity dataset, representing a different satellite.

Checked it and it looks good. Same approach as for Sanity. Perhaps optimize at a later stage.


directly support additional imputation tools

Overview
This tool was written with the expectation of results files from the imputation server and minimac. There isn't any intrinsic reason why this can't be used as a sliding filter method for the output of other imputation tools; but how to interface with this tool is debatable.
Quick hack
If you prepare files that look like minimac's output info.gz files, you can use it directly. This is obviously a bit of a pain.
The info.gz format still follows this outdated documentation. In short, you need column 1 (variant ID), column 2 (0-coded allele), column 3 (1-coded allele), column 5 (minor allele frequency), and column 7 (imputation rsq).
The names of the files can be specified using -i [file1] [file2] [...], so that's not a blocker. Though it supports gzipped files, the files don't actually have to be gzipped, and don't have to have a gzip suffix (I think).
Adding support for other imputation formats
I think the easiest and most generalizable way to do this would be to directly support a five column, whitespace-delimited format with just the entries enumerated above. It would be pretty easy to implement; but I don't know whether there's interest at this time.

thinking about an htslib-based vcf extraction option. it could be configurable with the INFO tag corresponding to r2, maf, and imputed (indicator flag).

support for reading directly from vcf as with the above INFO tags has been added as of 1.2.0. I'll keep this open for a while in case additional needs arise.


Numerous features and bugfixes

Features
Having more assertions:
-> isSubset (resolve lrills#6)
-> equalRaw (resolve lrills#11)
-> matchRegexRaw (resolve lrills#11)
-> matchSnapshotRaw (resolve lrills#11)
-> contains, expanded with count value (resolve lrills#52)
-> contains, expanded with any boolean (resolve lrills#74)
-> failedTemplate (resolve lrills#39, resolve lrills#82)
Added support for test-output (resolve lrills#51)
Added support to validate multiple templates (resolve lrills#38, resolve lrills#54)
Added support to use checksum validation for release and install (resolve lrills#60)
Added support to test dependent charts (resolve lrills#65)
Added support for Helm V3, including the option to render helm2 & helm3 charts (resolves lrills#87, resolves lrills#98)
Fixes
Fixed templates in subdirectories fail to load (fixed lrills#44)
During development also converted to git modules (fixes lrills#78, fixes lrills#79, fixes lrills#80)
Fixed support capabilities (refactered rendingen of the charts) (fixed lrills#88)
Fixed escaping . in e.g. annotations (fixes lrills#95)
Update to latest Helm2 library to support deepclone (fixes lrills#96)
Fixed fixed download version (fixes lrills#97)
Additional
For the time being a latest version can be retrieved from https://github.com/quintush/ (closes lrills#102)

Closed, as it conflicts with the updated quality checks.
Will reopen a new pullrequest, once the quality checks are compatible again.


Lint project source code

All set, @lukebjerring!

Merged into master as a56b66951ab92ab1e4594f47d05bc97b8b7f20a1. Unfortunately I didn't have permission to push to the fork or I would have updated this PR directly.
@jugglinmike creating branches directly in this repo would make this sort of thing easier, even if the "allow contributors to modify" bit were set on the PR.

We'll face the same friction for contributors who don't have "write" privileges. I'm happy to rebase whenever reviewers have bandwidth :)

The friction is for reviewers though, who will have write access. Anyway, not a big problem.


Add support for SameSite=None to 3.x

Backport #414 to the 3.x branch.
All PRs:

[x] Has tests
[x] Documentation updated

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I think this CLA bot is busted.

Successfully registered gem: secure_headers (3.7.4)


Some application distributes your library, without your license copy?

Hello
When I download some application from here in here and install it, your library is found in its installation directory, but your license copy is missing.
The Shared Library.zip
How do I know that is your library?
I hover the mouse pointer over the library file on the Windows system, and it shows in the field of the "Company":

The OpenSSL Project, http://wwww.openssl.org/

It contains the website link to your homepage, which is the trace back link and the unique identification that the library belongs to you.
According to your license terms:

Redistributions in binary form must reproduce the above copyright
notice, this list of conditions and the following disclaimer in
the documentation and/or other materials provided with the
distribution.

I can't find your license copy neither in their binary nor their EULA, so I report it to you.
The following applications share the same issue:
Download here from here
Download here from here
Download here from here
Download here from here

Thanks, we will followup with them.

Thanks for accepting my report.
Sorry but I forgot a point, I can't find the license phrase in their software, as well.
That is, I can not find the following acknowledgment from their ads, and About info.
And they include the Windows specific code, so SSLeay License, Point 4 also applies.
According to your license, Point 3

All advertising materials mentioning features or use of this
software must display the following acknowledgment:
"This product includes software developed by the OpenSSL Project
for use in the OpenSSL Toolkit. (http://www.openssl.org/)"

And SSLeay License, Point 3 and 4

All advertising materials mentioning features or use of this software
must display the following acknowledgement:
"This product includes cryptographic software written by
Eric Young (eay@cryptsoft.com)"
The word 'cryptographic' can be left out if the rouines from the library
being used are not cryptographic related :-).
If you include any Windows specific code (or a derivative thereof) from
the apps directory (application code) you must include an acknowledgement:
"This product includes software written by Tim Hudson (tjh@cryptsoft.com)"


Create a guide to exporting Gantt charts

A step-by-step walk-through or at least some clear guidelines for how to generate a Gantt chart for attachment to AIOs/reports would be helpful for everyone. Most students seem to be using InstaGantt, so specific instructions for that tool would be good. Some general instructions would be nice for groups not using InstaGantt.
One potential issue with providing instructions for a single tool is it might be seen as a form of endorsement of that tool. Solutions would be to create instructions for other tools or to keep the guide general.

I'm thinking that next year I'll just introduce asana and the instagantt plugin. I don't mind endorsing something, as long as they are free to use other tools they prefer. I'm seeing more and more cool uses of asana. It also has a trello style kankan board.
Just start with InstaGantt and we can the students in class which software they are using. If it's half/half we can add instructions for others.

There doesn't seem to be an export option with InstaGantt that preserves the current date indicator. I'm not sure what to recommend. Taking a screenshot shows the current date but limits the date range shown and is fairly low-resolution.

Page is up: https://moorepants.github.io/eme185/pages/gantt-charts.html

If this looks good, should we make an announcement for future AIOs/reports?

Not sure if this warrants re-opening the issue or creating a separate one, but I just found Elegantt which looks somewhat similar to InstaGantt but integrated with Trello rather than Asana.


Add japanese translation for download page

@azuchi Thank you for translating!  Based on your experience, is there anything you'd suggest changing in order to make translation easier or better for other translators in the future?

@harding Thank you for review! For translation,  CONTRIBUTING.md is written to use Transifex,  but I do not know it is working.

Unfortunately Transifex made some changes that basically broke our workflow so that kind of fell apart.

Oh, I see. So it may be better to update CONTRIBUTING.md according to the current. Some people are requesting to join in Transifex.


No code suggestion appears. Error: Language server extension command failed.  Exception: Error: command 'python.intellicode.loadLanguageServerExtension' not found

Never managed to make it work since 1 year. And nothing changed.
code 1.90.0-1
pylance 2024.6.1
intellicode 1.3.1
python 3.12.4-1

Sorry, something went wrong activating IntelliCode support for Python. Please check the "Python" and "VS IntelliCode" output windows for details.

Output: Python
2024-07-07 12:59:58.057 [info] Default formatter is set to null for workspace /home/user/ws/project
2024-07-07 12:59:58.064 [info] Python interpreter path: ./.venv/bin/python
2024-07-07 12:59:58.089 [info] > pyenv which python
2024-07-07 12:59:58.089 [info] cwd: .
2024-07-07 12:59:58.411 [info] Starting Pylance language server.
2024-07-07 12:59:58.419 [info] Send text to terminal: source /home/user/ws/project/.venv/bin/activate
2024-07-07 13:00:05.372 [info] Discover tests for workspace name: project - uri: /home/user/ws/project

output: VS Intellicode
Acquiring model
Acquiring model 'intellisense-members-lstm-pylance' for python
Acquiring model 'intellisense-members-lstm-pylance' for python
vs-intellicode-python was passed a model: {}.
Found bundled model for python
Activating Python extension
Language server extension command failed. Exception: Error: command 'python.intellicode.loadLanguageServerExtension' not found
    at v.n (vscode-file://vscode-app/usr/lib/code/out/vs/workbench/workbench.desktop.main.js:1637:23867)
    at v.executeCommand (vscode-file://vscode-app/usr/lib/code/out/vs/workbench/workbench.desktop.main.js:1637:23796)
Error while activating Python: {}

This should be a bug in intellicode: https://github.com/MicrosoftDocs/intellicode/issues/509.
Looking at intellicode's source, my only guess is that you don't have python.languageServer set to 'Default'.

Their code might have a bug in the order in which they load the extensions. If python.languageServer is Default, they will end up loading Pylance before they attempt to call python.intellicode.loadLanguageServerExtension so then the command should work. If it's not set to Default, I don't think it's guaranteed Pylance will load soon enough.

This should be a bug in intellicode: MicrosoftDocs/intellicode#509.
Looking at intellicode's source, my only guess is that you don't have python.languageServer set to 'Default'.

Their code might have a bug in the order in which they load the extensions. If python.languageServer is Default, they will end up loading Pylance before they attempt to call python.intellicode.loadLanguageServerExtension so then the command should work. If it's not set to Default, I don't think it's guaranteed Pylance will load soon enough.

Set in 'Default': still same issue but thanks for trying


Faceted plot panel doesn't occupy 100% width and have browser's horizontal scroll bar when deployed on QA

Works fine in my dev instance, but not on QA.
I don't think screenshots are required ;-)

The root cause of this has manifested as weird looking layouts if a visualization is loaded when the app first loads in a browser. The cause is multiple versions of material-ui's classname generator (used by makeStyles). This is causing classname conflicts since the two versions (ie, instances) have a unique id generator that both start at the same number.

Faceted plot panel now occupies 100% width and browser's horizontal scroll bar. Passes QA.


Allow checks to manually specify in their configuration which defaults to use

What does this PR do?
This PR essentially allows checks to manually specify the configurations they want to use, rather than making that an option in the base class.
Motivation
Review checklist

[x] PR has a meaningful title or PR has the no-changelog label attached
[x] Feature or bugfix has tests
[x] Git history is clean
[x] If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo

Additional Notes

Codecov Report

Merging #2145 into master will decrease coverage by 17.88%.
The diff coverage is 100%.


kwargs for downloading and parsing upon Article object creation

 Issue by Technerder
Tue Dec 10 00:49:03 2019
Originally opened as https://github.com/codelucas/newspaper/issues/756

Do kwargs for automatically downloading and parsing an article exist within the constructor of an Article?

newspaper.article()  function


Common actions list

+1 card, +2 actions
Council room? - however this would require implementing something that puts back the next turn actions after the player's next turn
Etc

Done in v0.10.0!


Use WebdriverIOs driver setup mechanism

Context
This is my attempt to drive #94 ahead. I believe your branch was very close and mostly needed to set the browserVersion.
Proposed Changes

Remove npm packges for chromedriver and wdio-chromedriver-service
Use WebdriverIO's setup mechanism

Do not download ChromeDriver

Remove unused utilities

Questions

Would you rather review on a call?
What do we need to do to support Chromium/ChromeDriver v114? The pipeline fails for v1.83.
Should we make the peer dependency on webdriverio explicitly ^8.14.0?
Want a separate PR to update dependencies? It looks like that was folded into #94 as well but I didn't include it to try to simplify the change.
Would you prefer to keep the ServiceCapability interface?

Can we update the CI tests to run on VS Code v1.84 minimum? This should hopefully resolve the problem. I think we need to set the minimum VS Code version to the one that runs on Chromium 115 and make this a breaking change.

Will do.
I'm curious, if you did need to support older versions would you be able to keep wdio-chromedriver-service and it'd override the core implementation?

Yeah .. maybe we can include a section with some information about it, e.g. adding chromedriver to the package.json and add it as service. Not sure how straightforward it is.

Ah, 1.85 is looking for Chromedriver 114. The VS Code manifests confirm that it uses v118 in 1.86.0.

💚
Can we add some documentation about the versioning?

I've updated the docs. The pipeline failure is a timeout at line 20578.

This should close #101.
It no longer closes #97 (update deps). Let me know which deps and I'll get a PR up for that too.

Released as part of v6.0.0!


Removed static array from functions strlwc, strcrop to make them thread-safe.

Hi Klaus,
It seems that functions strlwc and strcrop are not thread-safe. Let's analyze function
static char * strlwc(char * s)
{
  static char l[ASCIILINESZ+1];
  int i ;

  if (s==NULL) return NULL ;
  memset(l, 0, ASCIILINESZ+1);
  i=0 ;
  while (s[i] && i<ASCIILINESZ) {
    l[i] = (char)tolower((int)s[i]);
    i++ ;
  }
  l[ASCIILINESZ]=(char)0;
  return l ;
}

This function can be called concurrently (at the same time) by more than one thread. The flow for every thread is as follow:
wsman_send_request
    |->wsmc_handler(cl, request, NULL);
        |->init_curl_transport(cl);
            |->iniparser_getint(ini, "client:curlopt_nosignal", 0)
                |->iniparser_getstring(d, key, INI_INVALID_KEY)
                    |->lc_key = strdup(strlwc(key))

As you can see it may happen that two threads in the same time can call function strlwc so they can modify static array called "l" simultaneously. So the output can be not that what we would expect. For example thread1 passes "Key1" and thread2 "Key2". We should get for thread1 "key1" and for thread2 "key2". But when we return static allocated string we can get something different.
What do you think about this? Am I wrong or not?

Thanks a lot !
My local branch only had a workaround for this problem, not a real fix.
Will be in 2.6.2, to be released soon.


Add extra notebook checks from bokeh

E.g. for broken links.
See e.g. https://github.com/bokeh/datashader/blob/master/examples/nb

Was done in ff571186a41748686c077ca3fb27f890a5f09aae


[ChatGPT] ...I always get API limit reach, but my API limit is not reach and I see no usage

Extension
https://www.raycast.com/abielzulio/chatgpt
Raycast Version
1.80.0
macOS Version
14.6.1
Description
upon asking a ChatGPT question I get API limit reached

Steps To Reproduce

Simply ask any questions
get the error

Current Behaviour
No response
Expected Behaviour
not getting the error when providing the API key

@jifbrodeur, are you using the Azure OpenAI service or the OpenAI Platform?

OpenAI

Based on the error message, I believe this is not an issue from this extension. It says that you need to upgrade the billing/plan to pay-as-you-go.
Maybe you can check whether you got rate limited from OpenAI. I suggest you to check some possibilities that can happen with your situation. Some ideas you might consider checking:
Check if you are in a free tier, if so then you probably have been sending more than 200 messages today.
Ref:

If you are on free trial, check if it hasn't been expired yet.
Try to generate a new token.
Try contacting OpenAI support, they might reply within 24h on working days (Mon to Fri).
If you are on Azure and in a trial period, make sure that your subscription is still active and usable.
Try your API key on different platforms (e.g., MacGPT, ChatGPTx, etc.), see if the same error happens.

Hi, I created a new projet API key, gave it all rights, I am on a paid subscription for over 6 months, and I get the same error message. I don't feel very comfortable to put my API key in a lot of other application, is there a log that I could check to see the handshake between the extension and OpenAI?
Thanks

Hi, I created a new projet API key, gave it all rights, I am on a paid subscription for over 6 months, and I get the same error message. I don't feel very comfortable to put my API key in a lot of other application, is there a log that I could check to see the handshake between the extension and OpenAI?
Thanks

This is just my personal opinion. I always use different API keys for each app so I can control them whenever needed. If I want to check the usage for each API key, I just go to https://platform.openai.com/organization/usage/activity, or at the project level at https://platform.openai.com/usage/activity. Feel free to try this approach if you’re interested.

Hi, the approach you suggest is exactly what I did and I see no activity from the extension.
Thanks for your help and suggestion.


fix: unescape params

What to fix:
failed to query when target value is url
I got a column whose value is url, for example column name is url and the value is https://www.cnblogs.com/lyzg/p/5125934.html
When I do the query like this: client.SupaClient.DB.From(DB_NAME).Select("*").Filter("url", "in", fmt.Sprintf(("%s"), url))
It can't query any record out.
I found it's because supabase-go encode the query params before send the request.

This actually caused to fail some other uses cases:
https://github.com/nedpals/supabase-go/issues/28
https://github.com/nedpals/supabase-go/issues/26


Version 1.1.1 is not on Cocoapods

Thanks for an awesome library
Could whoever is entitled to do so bump the Cocoapods version to 1.1.1
Thanks in advance

hi @nrmtrifork
thanks for the report.
the reason why i've not pushed it to the spec is I got the following pod-spec-lint error:
CompileSwift normal i386 /Users/damiankolakowski/Desktop/swifter/Sources/String+SHA1.swift

swifter/Sources/String+SHA1.swift:31:25: error: integer literal '3285377520' overflows when stored into 'Int'
        var h4 = UInt32(0xC3D2E1F0)
                        ^
/Users/damiankolakowski/Desktop/swifter/Sources/String+SHA1.swift:29:25: error: integer literal '2562383102' overflows when stored into 'Int'
        var h2 = UInt32(0x98BADCFE)
                        ^
/Users/damiankolakowski/Desktop/swifter/Sources/String+SHA1.swift:28:25: error: integer literal '4023233417' overflows when stored into 'Int'
        var h1 = UInt32(0xEFCDAB89)
                        ^

** BUILD FAILED **

i am checking what's wrong :)
best
dk

@nrmtrifork I solved the problem ( https://github.com/glock45/swifter/issues/101 ) with the overflows and pushed version 1.1.2 to cocoa.


Mutation-based generator of arbitrary executable documents

For a given valid schema, use a byte string typically provided by cargo-fuzz as a source of entropy to deterministically generate a valid executable document:
pub fn arbitrary_valid_executable_document(
    schema: &Valid<Schema>,
    arbitrary_bytes: &[u8],
) -> Valid<executable::ExecutableDocument>

Instead of using the arbitrary crate, parts of it are forked for reasons documented at the top of the new entropy.rs file.
At a high level, it creates a minimal operation then keeps adding selections to it until entropy is exhausted. As a result, providing a longer byte string tends to create larger documents. Hopefully this helps fuzzers better explore the space of possible documents, compared to apollo-smith which uses code like u.int_in_range(1..=5) to decide how many items to generate regardless of remaining entropy.
With this approach I believe we could enumerate all possible valid documents up to a certain "size" with bounded amounts of entropy. We’re not quite there yet for at least a few reasons:

All generated field selections get a unique alias, so field merging is trivial. We can easily sometimes re-use response keys, but maintaining "fields can merge" validity is tricky.
Every time the generator decides create a named fragment spread selection it creates a new named fragment definition. If re-using fragments we need to be careful not to introduce cycles but this may accidentally be already solved: for borrow-checking reasons we remove fragments from the map while modifying them, and this may be exactly the subset that would introduce a cycle if reused.
String values are generated like names (limited character set, and typically short). I think this is ok for the purpose of testing query planner as string values in executable documents are typically not meaningful for planning purpose. (It’s a different story for generating subgraph schemas with federation directives)

This PR is draft because I’m unsure if this generator should live inside apollo-rs. I think eventually yes, but at first we’ll likely want to tweak it a lot and going through a crates.io release for every change would be a significant obstacle.


Check that all JARs and POMs have corresponding metadata

MavenMetadataValidator should check if all packaged JAR files have corresponding Maven metadata.
If not, "warn" or at least print "info".
In case there is a POM without metadata, "fail" the check.

Good, please also update the class-level comment which describes what the validator does.

Good, please also update the class-level comment which describes what the validator does.

The comment was already outdated, but sure, I will update it.

Thank you.


Convert to using our own file streams to download FTP files

Currently, we're using a ftp.get() function to download files. To get better progress status and to support file resume, we need to write our own pipes/stream to download the files.

Done. Now we need to figure out rate limiting and file resume.


fix(firebase_analytics): allow logEvent to accept items inside parameters

Description
Before this PR: logEvent did not accept items inside parameters
After this PR: logEvent can accept items inside parameters
Reason for this PR: On my project, our backend can send all data for analytics event, including items - this PR extends checks in _assertParameterTypesAreCorrect to allow this behavior
Related Issues

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (melos run analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

@Lyokone, I saw that you can merge MR's - can you please help with this MR?


How to emulate simple=true when calling jfr2flame directly?

Until now, I have been using only CPU profiling, with a flamegraph output.
I have now enabled cpu and alloc, which forced me to switch to JFR output and to generate flamegraphs myself.
I managed to get it working using java -cp async-profiler-2.8.3-linux-x64/build/converter.jar jfr2flame ..., but the resulting CPU flamegraph gives me classes with their FQN, which makes the chart hard to read. I'd rather see just their class names in there.
With the flamegraph output earlier, I was able to do that by supplying the option simple=true. However, scanning through the jfr2flame source code, I was not able to find its equivalent. Any advice?

There was no such option before.
Now I added --simple.

Thank you! I will close this issue as soon as there is an async-profiler release with this change included.

Async-profiler 2.9 includes the mentioned feature.


Rename PartiallySignedTransaction to Psbt

Last release we added a type alias for Psbt, now lets just rename the type and be done with it.
Includes re-export at the crate root because bitcoin::Psbt is clear and obvious.

In 26e160d760de3dc2850569dc825362c117e7eb7d you change the CHANGELOG.md to rename Psbt in old changelog entries. We shouldn't do this.

My bad, I thought I checked out the changelog after sed'ing, obviously not. Clearly didn't review the diff before pushing either.


feat: Support casting of Term

This is a proposal to cast Terms back to particular types of terms.
I was mainly after the feature of being able to cast Terms as Optional<Subject>s; and being able to construct an Optional<Triple> from arbitrary Terms.
If you're on board with this feature I can add documentation and then mark this as ready for review.

Awesome - I'll address the suggestions in a few hrs

The failing test should be fixed now @Tpt

@Tpt I'm on board with all of these ideas - if you have time today feel free to make the changes so that I don't slow things down in getting this merged; otherwise I can take a look at things tomorrow.

I've applied those changes @Tpt - feel free to make changes directly to this PR if you have further thoughts/suggestions so I don't slow things down further.

Thank you! Will do so (I am quite buzy at the moment so it might not be today sadly)

Hey @jeswr I am not allowed to edit your branch so I open #807 that is based on yours (the first commit is the squash of yours and the second is some simplifications)


Navigation menu

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #WRLDBAEMS-37-Navigation Menu
Open Issues,

Scrollbar shows up for wide monitor -- Fixed
In Mobile View, On scroll to the end, it shows background without content --Fixed
In Tab view, Custom scroll needs to be added

Test URLs:

Before: https://main--world-bank--aemsites.hlx.live/
After: https://navigation-menu--world-bank--aemsites.hlx.live/

@singhpritam18 Noticed a few minor issues:

When I resize the page and come back, the hamburger menu is automatically open. @naina66406 Maybe this is something from your implementation. Can you collaborate with Padmashree on this?
There is a horizontal scroll on lower desktop resolutions, this is a minor issue. But we may have to check if this can be fixed by wrapping text one below the other.
No cursor: pointer on the "Who we are" items.
the language selector dropdown is not toggling on clicking arrow button. I am okay if you want to handle this in a follow-up PR.
Closing hamburger should also happen on pressing esc button. Not urgent and can be a follow-up PR.
Scrollbar styles missing for tablet.
Increase the touch area for "who we are" section in mobile and tablet.

Overall it seems to be in good shape. 1st issue should be fixed as this is major. Rest of them can be handled in a follow up PR.
Note: will add code review in code diffs.

Hi @thomsebastin

When I resize the page and come back, the hamburger menu is automatically open. @naina66406 Maybe this is something from your implementation. Can you collaborate with Padmashree on this? - Done
There is a horizontal scroll on lower desktop resolutions, this is a minor issue. But we may have to check if this can be fixed by wrapping text one below the other. - Need to check with you
No cursor: pointer on the "Who we are" items. -- Done
the language selector dropdown is not toggling on clicking arrow button. I am okay if you want to handle this in a follow-up PR. -- Will work on that
Closing hamburger should also happen on pressing esc button. Not urgent and can be a follow-up PR. -- Will Work on this
Scrollbar styles missing for tablet. --  Need to check with you
Increase the touch area for "who we are" section in mobile and tablet. -- Done


Php 8.1 support for Windows

Describe the feature you would like to see
Hello, I want to use laravel-firebase in Windows environment with Php 8.1. Unfortunatelly, your package requires lcobucci/jwt 4.2-4.3 which requires ext-sodium which is not available on windows php 8.1 (only 7).
Can you make your package compatible with windows php 8 as well?

No problem! 🙌


Authentication failed on git push

Hi,
we want to do an empty commit with some build information during a TFS build and I tried to set up the write access using "Enable Git Remote Access" and "Restore Git Remote" from your tools.
I enabled OAuth token support in the build settings. When I try to push my commit to the repository I receive the error
fatal: Authentication failed for 'https://OAuth:********@my-tfs-server:443/myRepo/_git/myRepo/'

I tried "git push" as command line task and inside a powershell script, same behaviour.
Any ideas where to find further information on the issue?
Thank you for your help.
We use TFS 2015 (Version 14.102.25423.0)

Not the right place to ask for help?

Getting the same error, did you ever find a fix?

No, I abandoned this solution and created a PowerShell script we now use to tag our builds. So no empty commits but annotated tags. We used a .netrc file for autologin to be able to push. I hope this is a helpful hint ;)

Any updates? Im getting the same error.


Get License Information for Dependency - SDK-Style

License Type
License Text

idk how this might work, i know of these locations where Licenses might be:

NuGet-Property: PackageLicenseExpression
NuGet-Property: PackageLicenseUrl
LICENSE file embedded in NuGet

To download a NuGet Package maybe use this command:
dotnet add package <NAME> -v -n -f <TARGET FRAMEWORK> --package-directory <PACKAGE_DIRECTORY>
On a dummy project and use PACKAGE_DIRECTORY as a point to refer to the downloaded package
More information see here

But what's with a possible nuget.config?
For that I might need to add this dummy project in the actual source structure...
But I also want to be non-invasive 🤔

OH!
Maybe dotnet restore --packages <PACKAGE_DIRECTORY> might work?
This'll definitely utilize everything that there is (nuget.config, cached packages on the system, ...) and will be offered for the .NET (Core) CLI
Does this work non-SDK-Style projects too?

Yes, dotnet restore works. Following needs to be done for downloading packages:

Non-SDK-Style: nuget restore -Force -OutputDirectory <FOLDER> -Recursive
SDK-Style: dotnet restore --packages <FOLDER> --force /p:DisableImplicitNuGetFallbackFolder=true

There are some catches, though:

Structure:

dotnet restore: Package-Name (lower-case) --> Version --> All data (nupkg, nuspec, license, ...)
nuget restore: Package-Name + Version (i.e. Autofac.4.9.4) --> Some data (nupkg, license, ...)

Getting the License-Text should also be "two-stepped"

Is there a License already embedded in the NuGet-Package?
Download from the URL-Source provided

Getting the License-Type should be done, by:

Has the nuspec a "License Expression"?
...
Get it from the internal/manually provided URL to Type Mapping
Get it from the License-Text (Keywords, see #10)

Yes, dotnet restore works. Following needs to be done for downloading packages:

Non-SDK-Style: nuget restore -Force -OutputDirectory <FOLDER> -Recursive
SDK-Style: dotnet restore --packages <FOLDER> --force /p:DisableImplicitNuGetFallbackFolder=true

There are some catches, though:

Structure:

dotnet restore: Package-Name (lower-case) --> Version --> All data (nupkg, nuspec, license, ...)
nuget restore: Package-Name + Version (i.e. Autofac.4.9.4) --> Some data (nupkg, license, ...)

Problem with using restore is just that every package will be downloaded, even though we might just need a few, depending on the cache (see #1) and additionally restore might take pretty long in some cases (idk why)
Maybe looking for the package in the package file-package-sources

%USERPROFILE%/dotnet/packages (or sth similar)
packages folder in case of non SDK style
...

And then downloading directly might be a better option
Or really just using dotnet add or something on a dummy project (inside the obj folder maybe) for packages that are really needed might be the better option

Maybe there's a package that can do this? 🤔

These are the ways licenses can be specified (as far as i found out so far):
1:

license tag with file attribute (/package/metadata/license[@file])
license file in downloaded content
get raw license text from specified file
get license type from raw license text (#17 )

2:

license tag with expression attribute (/package/metadata/license[@expression])
licenseUrl tag (/package/metadata/licenseUrl)
no license file in downloaded content
get raw license text from url (licenseUrl tag)
get license type from license tag

3:

license tag with expression attribute (/package/metadata/license[@expression])
licenseUrl tag (/package/metadata/licenseUrl)
license file in downloaded content
get raw license text from license file
get license type from license tag

Best way to do it would be to have a number of sources:

nuspec
1.1) 'license'-element (license-type or raw license-text)
1.2) 'licenseUrl'-element (license-url -> raw license-text)
any file that contains 'license' (raw license-text)
mapping
3.1) internal mapping license-url -> license-type (license-type)
3.2) external mapping license-url -> license-type (license-type)
3.3) get license-type from raw license-text (license-type)

Other TODOs:

[ ] Change PackageReference to have a LicenseInformation object? 🤔


Add default Optim to DPO example

Comparison here: https://wandb.ai/huggingface/huggingface/reports/RMSProp-DPO--Vmlldzo1Mzc2NDQ3

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Hi @natolambert, thanks for the PR. There is a lot of metric in the wandb report; which are the ones we should look at?

Mostly this PR is just one of many things to get closer to the original implementation. Thanks for sharing the screenshot (I didn't realize it was private).
@vwxyzjn the goal of this was to show there is not a big difference. We'll have more signal soon as we scale it.


springfox-swagger 3.0.0 - SNAPSHOT - JSON schema not being generated as it was before te update

Please take the time to search the repository, if your question has already been asked or answered.
-3.0.0-SNAPSHOT

SpringBoot 2.2.7 Webflux
JsonView

I'm using SpringFox swagger in SpringBoot with JsonView support
My docket configuration is:
final List<ResponseMessage> globalResponses = Arrays.asList(
                new ResponseMessageBuilder().code(400)
                                            .message(ErrorMessages.INVALID_REQUEST)
                                            .responseModel(new ModelRef("InvalidDataResponse"))
                                            .build(),
                new ResponseMessageBuilder().code(401)
                                            .message(ErrorMessages.UNAUTHENTICATED)
                                            .responseModel(new ModelRef("UnauthenticatedResponse"))
                                            .build(),
                new ResponseMessageBuilder().code(403)
                                            .message(ErrorMessages.FORBIDDEN)
                                            .responseModel(new ModelRef("ForbiddenResponse"))
                                            .build(),
                new ResponseMessageBuilder().code(500)
                                            .message(ErrorMessages.INTERNAL_SERVER_ERROR)
                                            .responseModel(new ModelRef("InternalServerError"))
                                            .build()
        );

        return new Docket(DocumentationType.SWAGGER_2)
                .select()
                .apis(RequestHandlerSelectors.any())
                .paths(PathSelectors.any())
                .build()
                .globalResponseMessage(RequestMethod.GET, globalResponses)
                .globalResponseMessage(RequestMethod.POST, globalResponses)
                .globalResponseMessage(RequestMethod.DELETE, globalResponses)
                .globalResponseMessage(RequestMethod.PATCH, globalResponses)
                .globalResponseMessage(RequestMethod.PUT, globalResponses)
                .useDefaultResponseMessages(false)
                .genericModelSubstitutes(ResponseEntity.class)
                .alternateTypeRules(
                        newRule(typeResolver.resolve(
                                Mono.class,
                                typeResolver.resolve(ResponseEntity.class, WildcardType.class)),
                                typeResolver.resolve(WildcardType.class))
                )
                .additionalModels(
                        typeResolver.resolve(InvalidDataResponse.class),
                        typeResolver.resolve(UnauthenticatedResponse.class),
                        typeResolver.resolve(ForbiddenResponse.class),
                        typeResolver.resolve(NotFoundResponse.class),
                        typeResolver.resolve(InternalServerResponse.class)
                )
                .host(baseUrl);

And for the following endpoint/annotations:
    @ApiOperation(
            value = "List all projects"
    )
    @ApiResponses(value = {
            @ApiResponse(
                    code = 200,
                    message = SuccessMessages.CONTENT_READY
            )
    })
    @GetMapping(path = "/projects", produces = MediaType.APPLICATION_JSON_VALUE)
    @ResponseBody
    @JsonView({View.List.class})
    Mono<ResponseEntity<SuccessResponse<PaginatedListResult<ProjectModel>>>> getAll(

Now i get the schema of :
"responses": {
          "200": {
            "description": "content ready",
            "schema": {
              "$ref": "#/definitions/Error-ModelName{namespace='reactor.core.publisher', name='Mono«ResponseEntity«SuccessResponse«PaginatedListResult«Project»»»»'}"
            }
          }
        }

And the model definition is exported as :
"SuccessResponse«PaginatedListResult«Project»»View": {
      "type": "object",
      "properties": {
        "message": {
          "type": "string",
          "example": "content ready!"
        },
        "result": {
          "example": "null",
          "$ref": "#/definitions/PaginatedListResult«Project»View"
        },
        "status": {
          "type": "boolean",
          "example": true
        }
      },
      "title": "SuccessResponse«PaginatedListResult«Project»»View",
      "description": "Success response model"
    }

Before the last update this was working with the schema being generated similar to what is shown below
"responses": {
          "200": {
            "description": "content ready",
            "schema": {
              "$ref": "#/definitions/Mono«ResponseEntity«SuccessResponse«PaginatedListResult«Project»»»»"
            }
          }
        }

....

"Mono«ResponseEntity«SuccessResponse«PaginatedListResult«Project»»»»": {
      "type": "object",
      "properties": {
        "message": {
          "type": "string",
          "example": "content ready!"
        },
        "result": {
          "example": "null",
          "$ref": "#/definitions/PaginatedListResult«Project»View"
        },
        "status": {
          "type": "boolean",
          "example": true
        }
      },
      "title": "SuccessResponse«PaginatedListResult«Project»»View",
      "description": "Success response model"
    }`

What should i do different to fix this up ?

I'm having the same issue... Any workaround?

That API has been deprecated, see here for an example of how to do this going forward using Docket#globalResponses() instead. Keep in mind that API is still being polished before the release.

It should still work however. I'll try and fix that :) Thanks for reporting

It should still work however. I'll try and fix that :) Thanks for reporting

For me also it's breaking.

Hi,
With the latest approach of going with globalResponses instead of globalResponseMessage, how can we have custom response models which we used to create with ModelRef.

@Shivakumarmurthy see 12

Hi,
With the latest approach of going with globalResponses instead of globalResponseMessage, how can we have custom response models which we used to create with ModelRef.
new ModelRef("ErrorResponse500")

There is a sad example with DTO class:
new ResponseBuilder()
        .code(code)
        .description(description)
        .representation(MediaType.ALL)
        .apply(r -> r.model(m ->
                m.referenceModel(ref ->
                        ref.key(k ->
                                k.qualifiedModelName(q ->
                                        q.namespace("com.example.api").name("ErrorDto"))))))
        .build();

Hi,
With the latest approach of going with globalResponses instead of globalResponseMessage, how can we have custom response models which we used to create with ModelRef.
new ModelRef("ErrorResponse500")

There is a sad example with DTO class:
new ResponseBuilder()
        .code(code)
        .description(description)
        .representation(MediaType.ALL)
        .apply(r -> r.model(m ->
                m.referenceModel(ref ->
                        ref.key(k ->
                                k.qualifiedModelName(q ->
                                        q.namespace("com.example.api").name("ErrorDto"))))))
        .build();


Home.html Page needs a pop up.

I want a small initial pop with some image and a link to hacktoberfest on the home page, make it attractive yet not irritating.

Where do you want the link to redirect to? I can take this.

Want to work with this issue. assign me this task


feat: style blog

closes https://github.com/deer/fresh_blog/issues/12
I've finally learned enough in order to actually sort this out.

I'll create another PR to pull the non-style changes out of this.

For anyone who encounters this, the way to style the primary part of the blog is in your tailwind.config.ts:

export default {
  darkMode: "class",
  content: [
    "{routes,islands,components}/**/*.{ts,tsx}",
  ],
  theme: {
    extend: {
      colors: {
        light: {
          background: colors.white,
          foreground: colors.black,
          mutedBackground: colors.gray[200],
          mutedForeground: colors.gray[700],
        },
        dark: {
          background: colors.black,
          foreground: colors.white,
          mutedBackground: colors.pink[700],
          mutedForeground: colors.gray[400],
        },
      },
    },
  },
  safelist: safelist,
} satisfies Config;

If you want to style the generated markdown, then you'll need to add something like this to your styles.css:
@layer base {
  #markdown-body[data-color-mode="dark"] {
    --color-canvas-default: #000000;
    --color-fg-default: #ffffff;
  }
}

This particular example is forcing the markdown in dark mode to match the background/foreground in my theme.