Can't enable OIDC using Helm installation

I want to enable OIDC using the Helm Chart. After some search in the issues here, I found that amundsen is providing special oidc images. But there is no image for amundsen-search image. I also verified this on docker hub.
So, I tried to use the image amundsendev/amundsen-search:2.11.1. But with this image I get the following error during startup.
Traceback (most recent call last):
  File "search_service/search_wsgi.py", line 15, in <module>
    application = create_app(config_module_class=config_module_class)
  File "/app/search_service/__init__.py", line 55, in create_app
    importlib.import_module(FLASK_APP_MODULE_NAME),
  File "/usr/local/lib/python3.7/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1006, in _gcd_import
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 965, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'flaskoidc'

This is the configuration that I'm using:
frontEnd:
  image: amundsendev/amundsen-frontend-oidc
  imageTag: 3.12.0
metadata:
  image: amundsendev/amundsen-metadata-oidc
  imageTag: 3.9.0
search:
  image: amundsendev/amundsen-search
  imageTag: 2.11.1

oidc:
  enabled: true
  configs:
    FLASK_OIDC_PROVIDER_NAME: auth0
    FLASK_OIDC_CLIENT_ID: "..."
    FLASK_OIDC_CLIENT_SECRET: "..."
    FLASK_OIDC_CONFIG_URL: "https://my-domain.auth0.com/.well-known/openid-configuration"

Expected Behavior
I would expect that there is oidc-image for the search component.
Current Behavior
search service can not start with oidc enabled due to the error described above.
Steps to Reproduce

clone the amundsen repository containing the helm chart
override the values in values.yaml with the values from above
run `helm install amundsen

Context
I want to use Amundsen on GKE with OIDC. Currently, this is not possible.
Your Environment

Amundsen version used: See yaml configuration
Deployment (k8s or native): K8s

Even I am facing the same issue. Have you found the fix ?

No, couldn't find any solution. It looks like that Amundsen is not really interested to run on kubernetes.

Why isn't there an official docker image of the search component that includes oidc? At least, this is available for metadata and Frontend. Why not for search?

I am also curious why is oidc necessary for the metadata/search services? I would have thought that we only require oidc for the frontend which is the service that handles requests from users.

@raman-nbg the reason there is no search oidc image is that no one implemented this. 🤷  Happy to review your PR if you like to give it a go, and implement the same logic for search.
@benrifkind the reason some organizations require the OIDC for search/metadata is that they want to expose the metadata and search endpoints to be used in other applications, hence securing those endpoints.


Change getLocalisedTables method to public.

I have found that the getLocalisedTables() method is incredibly useful for some of the project specific oddities that we've been adding. Can we please make it public so that I don't have to duplicate the logic elsewhere?

I guess! Will need to be a new minor version though. (4.1).
Need to set up some more branches before we merge this.

We have a 4.0 branch already - good to merge?


Update Custom API throwing error message

While trying to update the Custom API in Mobile service, it is throwing error message that error in creating the API.
We are able to do the update in the azure portal successfully.
PFB the screen shot.

This issue is reproducible in the latest build.

This issue is fixed and working as expected in the latest build.


3d visualization on low level zoom

Issue Type
[ ] Question
[ ] Bug
[X ] Improvement
[ ] Build system related
[ ] Performance
[ ] Documentation
Description
Add new visualization 3d on low level zoom.
When you have low zoom values, rather than seeing a flat representation with polar deformations, show a sphere (orthografic projection).
In orthographic projection, geodesic line is  straight line, and other lines are arcs
rectangular projection

orthografic projection

great idea, but it's not trivial. I'd suggest you look into NASA's world wind for android

It's even beyond "not trivial", it's unfeasible if we stick to our slippy map tiles.
Of course we could warp tiles, but anyway we wouldn't be able to display the poles as we don't have tiles beyond N85 and S85.

you could have a background color (white, or other color that can be set), so that at the poles over 85 ° you can see the background, or you could create tiles with uniform color over 85°.
I think that the complication is warping tiles.

yeah. if you want this, try out nasa worldwind for android.


Code restructuration

Let's merge MacroDatabase and MacroContainer into one single class MacroContainer
Let's the library from C++17 standard library to check if directory exists and list files inside folders
Let's create a class MacroLoader from MacroContainer, that will offer method to load this container from a file or a folder.
let's rename filesystem.cpp/.hpp into macrosearch.cpp/.hpp, as it will only contain function related to the search of macros inside folders.


[QUESTION] Conflict with mobile navbar..

Hey!
In mobile, when the timeline element appear on screen during scrolling, my navbar is bugging and i cannot find why..
Can you help me please? :/
Here is my site:
www.oliviercastroperrier.com

Maybe you can use Stackoverflow for your problem and add more information about your problem.

I will, but i asked here because i found out that when i disable the animation (animate = false), everything is fine..


[Feature Request] A prop to change the position of the tooltip

Hi, i'd just like to start off by saying thank you for this package! It's by far one of the best I've tried.
I think a useful feature would be to be able to change the position of the tooltip. By default it is at the top, but a prop to change the position would be useful.

Actually, there is an existing position prop available to do just what you're asking!
Here's an example:
<VueCustomTooltip label="This is the tooltip text" position="is-right">a tooltip</VueCustomTooltip>

The allowed values for the position prop are shown below:
'is-top' // default
'is-right'
'is-bottom'
'is-left'

Oh wow, thanks! Turns out i'm just blind and completely missed the bit in the readme 😆


Duplicated imports when running layer playground

🐛 The bug
Getting a whole bunch of duplicated imports from nuxt/scripts when running nuxi dev .playground.
A target app that's using this layer, shows no logs of duplicated imports. So this only happens when running a playground.
Attached a reproduction containing an empty nuxt layer, with only Nuxt Scripts installed as a module.
🛠 To reproduce
https://stackblitz.com/github/onderwijsin/onderwijsloket-common?file=README.md
🌈 Expected behavior
No duplicated imports :)
ℹ Additional context
WARN  Duplicated imports "useScript", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used  10:58:21 AM

[10:58:21 AM]  WARN  Duplicated imports "useScriptEventPage", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptTriggerConsent", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptTriggerElement", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptClarity", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptCloudflareWebAnalytics", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

 WARN  Duplicated imports "useScriptCrisp", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used    10:58:21 AM

[10:58:21 AM]  WARN  Duplicated imports "useScriptFathomAnalytics", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptGoogleAdsense", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptGoogleAnalytics", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptGoogleMaps", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptGoogleTagManager", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

 WARN  Duplicated imports "useScriptHotjar", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used  10:58:21 AM

[10:58:21 AM]  WARN  Duplicated imports "useScriptIntercom", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptLemonSqueezy", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptMatomoAnalytics", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptMetaPixel", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

 WARN  Duplicated imports "useScriptNpm", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used        10:58:21 AM

[10:58:21 AM]  WARN  Duplicated imports "useScriptPlausibleAnalytics", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptSegment", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

 WARN  Duplicated imports "useScriptStripe", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used  10:58:21 AM

[10:58:21 AM]  WARN  Duplicated imports "useScriptVimeoPlayer", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

 WARN  Duplicated imports "useScriptXPixel", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used 10:58:21 AM

[10:58:21 AM]  WARN  Duplicated imports "useScriptYouTubePlayer", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

ℹ Running with compatibility version 4                                                                                10:58:21 AM

 WARN  Duplicated imports "useScript", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used  10:58:21 AM

[10:58:21 AM]  WARN  Duplicated imports "useScriptEventPage", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptTriggerConsent", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptTriggerElement", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptClarity", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptCloudflareWebAnalytics", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

 WARN  Duplicated imports "useScriptCrisp", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used    10:58:21 AM

[10:58:21 AM]  WARN  Duplicated imports "useScriptFathomAnalytics", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptGoogleAdsense", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptGoogleAnalytics", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptGoogleMaps", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptGoogleTagManager", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

 WARN  Duplicated imports "useScriptHotjar", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used  10:58:21 AM

[10:58:21 AM]  WARN  Duplicated imports "useScriptIntercom", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptLemonSqueezy", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptMatomoAnalytics", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptMetaPixel", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

 WARN  Duplicated imports "useScriptNpm", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used        10:58:21 AM

[10:58:21 AM]  WARN  Duplicated imports "useScriptPlausibleAnalytics", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptSegment", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

 WARN  Duplicated imports "useScriptStripe", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used  10:58:21 AM

[10:58:21 AM]  WARN  Duplicated imports "useScriptVimeoPlayer", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

 WARN  Duplicated imports "useScriptXPixel", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used 10:58:21 AM

[10:58:21 AM]  WARN  Duplicated imports "useScriptYouTubePlayer", the one from "#app/composables/script-stubs" has been ignored and<EMAIL_ADDRESS>is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptPlausibleAnalytics", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/plausible-analytics" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptCloudflareWebAnalytics", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/cloudflare-web-analytics" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptFathomAnalytics", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/fathom-analytics" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptMatomoAnalytics", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/matomo-analytics" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptSegment", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/segment" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptMetaPixel", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/meta-pixel" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptXPixel", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/x-pixel" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptGoogleAdsense", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/google-adsense" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptIntercom", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/intercom" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptHotjar", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/hotjar" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptClarity", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/clarity" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptStripe", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/stripe" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptLemonSqueezy", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/lemon-squeezy" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptVimeoPlayer", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/vimeo-player" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptYouTubePlayer", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/youtube-player" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptGoogleMaps", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/google-maps" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptCrisp", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/crisp" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptNpm", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/npm" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptGoogleTagManager", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/google-tag-manager" is used

[10:58:21 AM]  WARN  Duplicated imports "useScriptGoogleAnalytics", the one from<EMAIL_ADDRESS>has been ignored and "/Users/remihuigen/local-projects/clients-onderwijsloket/onderwijsloket-common/node_modules/@nuxt/scripts/dist/runtime/registry/google-analytics" is used

UPDATE: Just noticed that as soon as I remove @nuxt/scripts from the target apps devDependencies (which should be fine, since it's installed as a dependency of the layer), the duplicated imports warnings occur in the target app as well...


Packets In-Order?

Hi,
I've wrote a simple test where a quic server simply echos back the data that a quic client has sent and closes the connection.
In detail, the server

Gets a session in Secure state
Accepts a stream
Reads a block of data
Writes back the data
Closes the stream
Closes the session

The client

Opens the session
Opens the stream
Writes some test data
Reads back that data
Closes the stream
Closes the session

There seem to be a race condition, when the client reads back the data (step 4). Sometimes the Read returns a PeerGoingAway error before the data was read. Could it be, that the close packet is received before the data packet?

Hi @phifty,
thanks for testing. What you're seeing can be caused by two things: First, as you already suggested, you could see a reordering of packets. Second, even with perfect in-order delivery, if your application is reading too slowly from the stream, Stream.Read will return an error as soon as the CONNECTION_CLOSE was processed.
I looked at Google's QUIC implementation, and as far as I can see there's no special treatment of PeerGoingAways, as compared to other CONNECTION_CLOSE reasons. That means that as soon as Chrome does the same: as soon as it receives a CONNECTION_CLOSE, it immediately closes the connection (not sure about the error returned from the stream though, I'm not that familiar with their code).
I think what we're seeing here, is a race condition in the protocol itself. In the h2quic package it doesn't occur because the server only closes initiates a closing of the connection after the DefaultIdleTimeout (30s), i.e. when there's no network activity for 30 seconds. This seems to work reasonably well for H2 over QUIC.
At the moment, there is an ongoing discussion in the QUIC WG about a clean connection teardown, unfortunately spread over a larger number of issues and mailing list threads. I don't think that we're close to a consensus here, so it's a good time to jump into the discussion!

https://github.com/quicwg/base-drafts/issues/330
https://github.com/quicwg/base-drafts/issues/353
https://github.com/quicwg/base-drafts/issues/347
https://mailarchive.ietf.org/arch/msg/quic/oxQF6Ax07bfsagWjxWVG5IekXFU
https://mailarchive.ietf.org/arch/msg/quic/awD1plcs68V7VfPTRXNrD43xHJ0

I seem to be having a similar problem, but with v0.21.0. Because neither SendStream.Write() nor SendStream.Close() block and wait for the final STREAM frame with FIN set to be acknowledged, there doesn't seem to be any way to guarantee that the peer has gotten all of the data before closing the connection, resulting in a seemingly unresolvable data race. I can't even manually acknowledge it because I'd just get the exact same problem the other direction. Am I just missing something?

@DeedleFake Sounds like you re-discovered the Two Generals Problem: https://en.wikipedia.org/wiki/Two_Generals'_Problem.
That said, in the future it might be a better idea to open a new issue than trying to resurrect a discussion that had been dead for more than 3 years.

I know what the Two Generals Problem is and this isn't it. This is a race between writing a small packet and closing the session. I've been going through a decently large chunk of quic-go's codebase and it seems that it's being caused by my calling Listener.Close(), which in turn closes all of the accepted Sessions, sending CONNECTION_CLOSE frames to all of the peers and breaking any existing streams with untransmitted data. Since neither Write() nor Close() block for streams if the data written is small enough to fit into a packet, there's no way to guarantee that all of the data has been sent before the session is closed. It's a data race, and, as far as I can tell, a bug in quic-go. Sometimes it works because of convenient timing and sometimes it doesn't, the classic staple of a data race.
Sorry about reviving this issue, but I thought that it might not have been fixed after all so I thought it made sense. If you think that what I detailed above is actually a bug, I'll file a new one.

which in turn closes all of the accepted Sessions, sending CONNECTION_CLOSE frames to all of the peers and breaking any existing streams with untransmitted data. Since neither Write() nor Close() block for streams if the data written is small enough to fit into a packet, there's no way to guarantee that all of the data has been sent before the session is closed.

That's true, but even if we guaranteed that the data was sent out, there'd be no guarantee that it would be received (the packet could be lost).
If you want to make sure that data was actually received by the peer, you can't close the connection before you haven't received an explicit signal from the peer.

Because I have no guarantee that the latest bytes will even attempt to be sent, how can I reliably signal the peer? Isn't the whole point of QUIC to make data transmission reliable enough for dependable usage via timeouts and retransmission, like in TCP? I'm calling SendStream.Write() but the bytes that I'm passing to it never get sent to the other end at all unless I just wait for an arbitrary amount of time via time.Sleep(). Then they get sent. That seems like incorrect behavior to me.
I seem to have no way at all to know if the library has actually attempted to send the bytes that I gave it or not, which means that I can't ever know when to close the connection. I can safely close a stream, which results in the remote end getting the bytes and returning io.EOF from its read operation, but closing it still doesn't guarantee that any attempt has been made to send them, or will be made before the session is closed. In fact, the example code and the usage of various quic-go components in the http3 package never seem to close any Sessions or Listeners unless there's an unrecoverable error.

I'm calling SendStream.Write() but the bytes that I'm passing to it never get sent to the other end at all unless I just wait for an arbitrary, but long enough, amount of time via time.Sleep().

Same in TCP. Packets get lost sometimes. Retransmissions might get lost as well. Really, this is no different than TCP.
As I said before, if you want to close the session from the sender's side, you'll need a signal from the receiver that it received all the data. That part is not hard to implement. If you're hoping that QUIC will offer you a solution to the Two Generals Problem, you're out of luck though.

I literally can't send a signal because the library doesn't even try to send the bytes that I give it. This is not the Two Generals Problem. The problem here is entirely contained to one machine. It has nothing to do with network conditions or lost packets.
https://github.com/lucas-clemente/quic-go/blob/af3e1008ad94c64d5a60d1d7b2ddc579bca4ca79/send_stream.go#L116-L120
In order to save a frame, the library will hold excess bytes that will fit into a single frame so that, if the stream is closed, it can send those last bytes with the FIN bit set instead of having to send an empty STREAM frame with the bit set. It will attempt to send them if SendStream.Close() is called, but there is a race condition that exists between that call and closing the session that the stream is part of, meaning that that frame might be lost completely before even getting put out onto the network at all. I can't send a signal back because the same problem exists on both ends because both ends are using this library. The only way that I can get the library to reliably attempt to send data is by either passing a large enough amount to Write() that it can't hold on to it to save a frame later, or to close the connection, which does reliably send a CONNECTION_CLOSE frame which, unfortunately, is not exactly great for carrying custom data that isn't an error code.
The following workaround does work:
Sender's end:
fakeFunctionToIndicateSendingData()

_, err = s.Read(make([]byte, 1))
if err != nil {
  var apperr *quic.ApplicationError
  if errors.As(err, &apperr) {
    if apperr.ErrorCode == quic.ApplicationErrorCode(quic.NoError) {
      return // Cleanup is deferred.
    }
  }
  fakeFunctionToIndicateErrorHandling()
}

Receiver's end:
fakeFunctionToIndicateReadingTheDataAndClosingTheStream()

err = session.CloseWithError(quic.ApplicationErrorCode(quic.NoError), quic.NoError.Message())
fakeFunctionToIndicateErrorHandling()

Is this the recommended pattern? Because nothing else seems to work because the library literally won't send the data that I give it.
And no, Go's TCP implementation does not have this problem. I tested it with net.TCPConn and it had no issues at all, even if I called SetNoDelay(false). No matter what timing I used, if I called Write() before calling Close() it would always transmit the data before closing the connection.

Alright, it's not 3:00 A.M. anymore. Let me see if I can clarify a bit. Sorry for getting so frustrated.
Here's the actual problem I'm having: If I want to write something and then close just a stream, it works just fine. If I write something to a stream, close the stream, and then also close the session, on the other hand, it doesn't necessarily process the bytes that I wrote before the close in the runloop, resulting in them never getting sent. Is my only option to deal with this to close the session from whichever end last received data, and is this expected behavior or should a clean close flush any remaining packets before closing?
As a followup, is the correct, clean way to close a session to call Session.CloseWithError() and give it something specific to the application-layer protocol that I'm actually implementing? There doesn't seem to be any consistent way to trigger a call to session.closeLocal(nil) without closing the associated Listener, but that only works on the server's side and, obviously, I can only do that if I want to actually close the Listener.


[Bug] spec.service.type gone in v1beta1?

Describe the bug
I am trying to deploy Grafana with LoadBalancer as a service type but it says spec.service.type and spec.service.ports are invalid
Version
v5.0.0-rc0
To Reproduce
Use this deployment:
apiVersion: grafana.integreatly.org/v1beta1
kind: Grafana
metadata:
  name: grafana
  labels:
    dashboards: "grafana"
spec:
  service:
    type: LoadBalancer
    ports:
      - name: grafana80
        port: 80
        protocol: TCP
        targetPort: grafana-http
  config:
    log:
      mode: "console"
    auth:
      disable_login_form: "false"
    security:
      admin_user: root
      admin_password: secret

Expected behavior
It should create a service which exposes port 80 via the LoadBalancer
Runtime (please complete the following information):

OS: Ubuntu 22.04 LTS
Grafana Operator Version v5.0.0-rc0
Environment: Kubernetes 1.25.6
Deployment type: deployed

By looking through the CRD I found it now needs to be spec.service.spec.type / ...ports.
It'd be nice if this was added to the documentation as well.


Entity interfaces and hierarchy

This PR introduces interface Entity and abstract base classes that implement these interfaces.
This PR is preparation for:

introducing EntityMeta, which will be based on AbstractEntity
an experiment with buildable Repository.

@armiol, PTAL.

@armiol, PTAL.

@alexander-yevsyukov LGTM.


Added spiner actovity indicator

Description:
Los WebView tienen diferentes métodos visuales de loading

Linked issues:

How to test:
We need to test:

Aurora Screen
-News and Bulletins

All should have the same loading animation its a Spinner with large size

the active indicator does not appear in the center of the screen


updated coverage to ignore random files

Pull Request Test Coverage Report for Build 54

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+40.7%) to 79.751%

Totals

Change from base Build 51:
40.7%

Covered Lines:
449

Relevant Lines:
563

💛  - Coveralls


codegen+runtime: Module resolution fixes/ basic formatters for Array/Set/Dictionary iterators

@jntrnr Just noticed this had conflicts, fixed now.


fix(project): Bump sdk-core and axios for CVE-2024-39338

PR summary

Bump sdk-core and axios for CVE-2024-39338
This also adds @ibm-cloud/sdk-test-utilities as a dev dependency per the sdk-core release notes for v5: https://github.com/IBM/node-sdk-core/releases/tag/v5.0.0
Fixes: N/A
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

Vulnerable dependency version
What is the new behavior?

Patched dependency consumed
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

:tada: This PR is included in version 1.1.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Weaponmaster and Deadly Blow

Are there plans to implement Weaponmaster and Deadly Blow with attacks? It would be great if you could select the attacks they apply to like how combat skill levels are implemented.
Something similar to this:

The +3DC is pretty straight forward.  Need to discuss with @phBalance about the conditional part as I recall concerns about how conditional defenses are implemented.

Maybe when the attack roll is being made they can check a box.

@phBalance you OK with conditional attack bonuses in the same theme as conditional defenses?

I'm working on an alternate (better?) way of dealing with powers that only apply to a subset of attacks.  Attacks with checkboxes are converted to custom adders.  This should allow people to use custom adders in HD, which we can parse in FoundryVTT.

Initial support for WeaponMaster.  Have not reviewed costs.

Example of Deadly Blow conditional attack bonus:


HSL to HUSL?

I noticed the website has a hue picker, and it looks like it's displaying HSL values along with a hex. Is this slider input being used to produce HUSL at all? It wasn't clear to me beyond the visual colour space (chroma?) changing and the dot for colour moving through it. I wanted to use this library but it didn't seem like it was friendly for stepping through the hues, there doesn't seem to be documentation on it's value range? When trying the Lua version out I was getting hue values all over the place as I moved through the hues incrementally:
Progressively moving along the hue range from 0 to 359.5
6.7535548210144
636.54663085938
1390.5018310547
3776.076171875
6971.8715820313
5463.1713867188
1969.6057128906
336.37377929688

husl/huslp to rgb function seems to convert the colour fine. But I guess there isn't a programmatic way to cycle through the hue like with HSL? Perhaps a function could be used for this?

Hi Brennan,
The H, S, L sliders on the homepage represent HUSL values, not HSL values, which might be the source of confusion here. There is some further confusion with the issue of stepping through the hues. To step through the hues with HUSL, you would need to do something like this (pseudocode):
for (i=0; i<10; i++) { huslToRgb(i * 36, 50, 50); }

The numbers you provided seem to be incorrect, but I'm not sure how exactly you generated them. If I understand you correctly they are hue values? But you generated them by stepping though other hue values? Could you explain that?

This is a test snippet for the above results in a games lua engine,log() is just to print to the games console and log file.
log("hex to huslp1: " .. tostring(HUSL:getHUSL().hex_to_husl('#ea0064')))
log("hex to huslp2: " .. tostring(HUSL:getHUSL().hex_to_husl('#ef001d')))
log("hex to huslp3: " .. tostring(HUSL:getHUSL().hex_to_husl('#c85300')))
log("hex to huslp4: " .. tostring(HUSL:getHUSL().hex_to_husl('#8f7400')))
log("hex to huslp5: " .. tostring(HUSL:getHUSL().hex_to_husl('#388800')))
log("hex to huslp6: " .. tostring(HUSL:getHUSL().hex_to_husl('#006ffd')))
log("hex to huslp7: " .. tostring(HUSL:getHUSL().hex_to_husl('#d900bd')))
log("hex to huslp8: " .. tostring(HUSL:getHUSL().hex_to_husl('#ea0066')))

You can see that I sampled the hex code from the slider as I stepped through it(not perfect steps). I did this since using hue values of 0-360 which I would have assumed HUSL would use was always producing red. No idea why the number is so large, the colours are correct when using hex_to_husl.
I would apply the colours like so:
colour = Vector3( HUSL:getHUSL().huslp_to_rgb( HUSL:getHUSL().hex_to_huslp('#39a467')) )
If I understand correctly, huslp is similar to husl but slightly better choice? If I could step through the hues with a loop like I could with HSL that'd be great :)
The only change I've made to the class is to wrap it in:
if not _G.HUSL then
    _G.HUSL = {}

    function HUSL:getHUSL()
        -- Rest of code here
    end
end

Hi Brennan, I just did the following test to confirm that HUSL generated correct numbers for the hex colors you provided:
Lua 5.2.4  Copyright (C) 1994-2015 Lua.org, PUC-Rio
> husl = require 'husl'
> = husl.hex_to_husl('#ea0064')
0.11786819154934	99.999999999999	49.999461028217
> = husl.hex_to_husl('#ef001d')
11.109833607508	100	50.03227766649
> = husl.hex_to_husl('#c85300')
24.26885039576	100	50.060878557424
> = husl.hex_to_husl('#8f7400')
65.904978665339	100	49.895842364182
> = husl.hex_to_husl('#388800')
121.68214034739	100	50.035253204047
> = husl.hex_to_husl('#006ffd')
258.3664850745<PHONE_NUMBER>0001	50.046382454934
> = husl.hex_to_husl('#d900bd')
319.34071134768	99.999999999998	50.013097245283
> = husl.hex_to_husl('#ea0066')
359.58763950098	99.999999999999	50.046245526067

I'm not sure where those big numbers are coming from, but they don't seem to be coming from HUSL. If you still believe the problem is with HUSL, please give me the shortest snippet you can that would demonstrate it (just one color, no wrapper functions etc.).
I may be able to help you further if you tell me what you are trying to use HUSL for. Usually using HUSL involves generating HUSL coordinates (stepping through hues is a common example) and converting the HUSL color to RGB or hex in order to display it.

I was just after a lua class/functions to handle colour manipulation better, specifically HSL and I stumbled upon HUSL which seemed it might be a better option.
All I plan to do is take a number value that is divided by the hue range 0-360 and returns a table/array of colours, so 4 would return 4 colours using the hue values 0, 90, 180, 270. The hex conversion here was to troubleshoot why hue values between 0-360 were all returning red rgb, from memory the hex -> husl -> rgb conversion resulted in the correct colour but those odd numbers for the hue channel.
Apart from the wrapper function the actual code is being used for modding a game that uses Lua, it might be the code is conflicting somewhere with it? I've setup this REPL example which is how I remember implementing your class(not at home right now). It seems to function correctly.
I'll close this as the problem seems to be due to my Lua environment and not your code.

If anyone comes across this issue, the problem was due to my Lua environment using degrees for input and return values for the math functions, whereas the lua implementation uses radians.


L10N : Updated locale files with latest translation.

Pinging @milamberspace to have a look.

@barunhalder1234 I thinks you don't use the Transifex website to update the translations?
For example on Transifex (for 4.6/4.7 resources) the Japanese translation for the key label.ha.enabled (HA Enable in US string) is:
高可用性有効
(translate in Google Translator: Enable high availability)
But in your PR the same key is translated with:
高可用性を有効にする
(translate in Google Translator: We want to enable high availability)
(curiously the current japanese string on transifex seems better than your update string...)
The issue is that the transifex Japanese / Chinese resources files aren't up-to-date, so next time, when we update the resources files from transifex to the git repo, we lost all your updated strings.
The right way to translate the Web UI of CloudStack is to use the Transifex project [1] and use the script inside the cloudstack project to download and prepare the resources files (unicode/native conversion and alphabetical order) [2]
To deal with this issue, you can update the transifex JA and CH resources files with the Transifex client. (See this script [3]) I can do this if you prefer.
[1] https://www.transifex.com/ke4qqq/CloudStack_UI/
[2] https://cwiki.apache.org/confluence/display/CLOUDSTACK/Update+L10N+files+from+Transifex+to+git+repo
[3] https://github.com/apache/cloudstack/blob/master/tools/transifex/sync-transifex-ui.sh
cc @remibergsma


Issue with browser formatting validations

Hi,
Wondering whether you can help me. My users are reporting an issue with the formatting that I was not aware of.
I am initialising the formatting of some inputs as follow:
jQuery(document).ready(function() { $('.decimal9FormatCssClass').autoNumeric({ mNum: 10, mDec: 2, vMin: '-9999999.99', vMax: '9999999.99' }); });
So you can see that the quantity of digits are 10. However, this is not the case with modern browsers like IE11 where apparently the length is validated with the localisation formats.
The browser automatically add the thousand commas separators according to the regional settings of the client machine. For example, 999,999.99. So the maximum value the users can add is 999,999.99 instead of 9,999,999.99 as defined in the above initialization.
If I add 9,999,999.99, then I get a hint over the input: "You must enter text with 10 or few characters"
Also, for negative numbers, a significant reduction as the minimum that they can add is only -99,999.99 instead of -9,999,999.99 .
I do apologise if I am missing something really obvious. Can you suggest how can we omit the commas being counted???

Jorge,
What version of autoNumeric are you using? I do not recognize the "mNum"
option?
Is the max length of the input define to 10?
Please post a jsFiddle so I can review.
Bob
On 5/17/2015 11:25 AM, jorgechoypernia wrote:

Hi,
Wondering whether you can help me. My users are reporting an issue
with the formatting that I was not aware of.
I am initialising the formatting of some inputs as follow:
jQuery(document).ready(function() {
$('.decimal9FormatCssClass').autoNumeric({ mNum: 10, mDec: 2, vMin:
'-9999999.99', vMax: '9999999.99' }); });
So you can see that the quantity of digits are 10. However, this is
not the case with modern browsers like IE11 where apparently the
length is validated with the localisation formats.
The browser automatically add the thousand commas separators according
to the regional settings of the client machine. For example,
999,999.99. So the maximum value the users can add is 999,999.99
instead of 9,999,999.99 as defined in the above initialization.
If I add 9,999,999.99, then I get a hint over the input: "You must
enter text with 10 or few characters"
Also, for negative numbers, a significant reduction as the minimum
that they can add is only -99,999.99 instead of -9,999,999.99 .
I do apologise if I am missing something really obvious. Can you
suggest how can we omit the commas being counted???
—
Reply to this email directly or view it on GitHub
https://github.com/BobKnothe/autoNumeric/issues/144.

Jorge,
This sounds like your length on the input is set a ten!
As I said in my earlier email I am not sure what this mNum is - if you
are using the latest version 1.9.36 you could change the following:
jQuery(document).ready(function() {
$('.decimal9FormatCssClass').autoNumeric({vMin: '-9999999.99', vMax:
'9999999.99' }); });
mNum: 10 ??
mDec: 2 - redundant the min/max 2 decimal places is already define
I am not aware of any browser that automatically inserts commas. The
comma is the default thousand separator in autoNumeric and will be
inserted unless you change the "aSep: ',' to "aSep: ''. if you want
comma's then the following is true:
vMin: '-9999999.99' =>    -9,999,999.99     character length is 13
vMax: '9999999.99' =>    9,999,999.00       character length is 12
So you maxlength attribute needs to be 13 - smaller will not let the
user enter the max amount

Bob
On 5/17/2015 11:25 AM, jorgechoypernia wrote:

Hi,
Wondering whether you can help me. My users are reporting an issue
with the formatting that I was not aware of.
I am initialising the formatting of some inputs as follow:
jQuery(document).ready(function() {
$('.decimal9FormatCssClass').autoNumeric({ mNum: 10, mDec: 2, vMin:
'-9999999.99', vMax: '9999999.99' }); });
So you can see that the quantity of digits are 10. However, this is
not the case with modern browsers like IE11 where apparently the
length is validated with the localisation formats.
The browser automatically add the thousand commas separators according
to the regional settings of the client machine. For example,
999,999.99. So the maximum value the users can add is 999,999.99
instead of 9,999,999.99 as defined in the above initialization.
If I add 9,999,999.99, then I get a hint over the input: "You must
enter text with 10 or few characters"
Also, for negative numbers, a significant reduction as the minimum
that they can add is only -99,999.99 instead of -9,999,999.99 .
I do apologise if I am missing something really obvious. Can you
suggest how can we omit the commas being counted???
—
Reply to this email directly or view it on GitHub
https://github.com/BobKnothe/autoNumeric/issues/144.

Hi Bob,
Thanks very much for the quick response.
I took the mNum parameter from a very old example/system developed years ago developed from one of my collegues. Maybe it was withdrawn in the new releases of autonumeric???
My fault, I should be following the most recent documentation. I had also a look of this parameter from the follwoing site: http://prgassist.blogspot.de/2011/04/jquery-plugin-autonumeric.html.
You are right... The browser does not add the commas, I think is the default from the autonumeric library. I am going to increase the vMax and vMin values as suggested.
Thanks Bob.

Jorge,
Wow that post is 4 years old and covered version 1.6.2
Major updates on version 1.7 & 1.8.
Just make sure the input max length is equal to or greater than the formatted min/max value including currency sign.
Good luck.
Bob
Robert (Bob) Knothe

On May 17, 2015, at 6:02 PM, jorgechoypernia<EMAIL_ADDRESS>wrote:
Hi Bob,
Thanks very much for the quick response.
I took the mNum parameter from a very old example/system developed years ago developed from one of my collegues. Maybe it was withdrawn in the new releases of autonumeric???
My fault, I should be following the most recent documentation. I had also a look of this parameter from the follwoing site: http://prgassist.blogspot.de/2011/04/jquery-plugin-autonumeric.html.
You are right... The browser does not add the commas, I think is the default from the autonumeric library. I am going to increase the vMax and vMin values as suggested.
Thanks Bob.
—
Reply to this email directly or view it on GitHub.


Create custom notice shortcode styling for premium tier features

Made a new style for a "Premium Tier Feature" callout (referred to as a "notice" shortcode in this Hugo theme).
My instinct says this should live in a custom .css somewhere instead of just appending it to the theme's .css, in the event that we ever update the theme and accidentally blow away this custom style. But I couldn't quickly figure out where that should be - i.e. where Hugo would know to look for it - and didn't want to spend too much time on this.
If we want to merge this, I can then use this notice style to add a note to the "Preview IP Filtering" feature we're documenting in #210 and denote that it's an enterprise-only feature.
I did validate that it works locally - here's what it looks like:

I like this a lot!
One slight tweak to the language:
- This is an enterprise-only feature
+ This feature is only available to [Tugboat Enterprise](https://www.tugboat.qa/enterprise) or On-Premise subscribers.

Agreed. This is great!
I know this is currently proof-of-concept, but can we not lead with a question in this callout, and go straight into "This feature is only available..."


Add option to trucnate UDP messages (default true)

This PR adds a default parameter to readDatagram(into:bufSize) called truncate that tells whether or not to truncate messages that are too long.
Description
See above.
Motivation and Context
Since UDP sockets are by nature unreliable, it makes little sense to force users to make their buffer large enough, since they may not care about getting all the data anyway. However, the ability to be alerted to a buffer that is too small is useful. This PR attempts to have the best of both worlds, allowing the error to be ignored by default but enabling the user to catch such issues if desired.
How Has This Been Tested?
Existing tests ensure that normal UDP communication still works. I have also added to the testReadWriteUDP() test to ensure that by default a small buffer does not throw an error.
Checklist:

[x] I have submitted a CLA form
[x] If applicable, I have updated the documentation accordingly.
[x] If applicable, I have added tests to cover my changes.

If desired, this could be extended to TCP as well (of course, with truncate = false by default). Making it a default parameter means that it would not be an API breaking change, and would allow users to choose a max size for their buffer while still being able to process data.
Perhaps in that case a different name than truncate would be better, since we would not actually be truncating the data but instead just leaving it in the buffer for next time... Let me know your thoughts.

Great idea. Go ahead and extend it for TCP/UNIX but with truncate defaulted to false. For UDP, change the default to true.


Allow products to be placed in multiple categories

This will allow us to use multiple category trees to represent different taxonomies like brands and collections.

Please look at how the selection widget works in Spree. We need an autocomplete field with a backing API to avoid exporting the full list that could very well consist of thousands of items.


Query anchors from api.tx.anchorBn254.anchorList mutate the state object with anchors

Implemented on polkadot-bridge-api
SRC


platform channel invokeMethod from background service in android

Hello everyone,
I am trying to figure out how to call platformChannel.invokemethod("mymethod","some argument"); from running background service in android, i thought of passing MethodChannel instance created in MainActivity.java to service via Bundle but then i realised for this to work data that must be sent to service via intent should implement parcelable or serializable  but none of this would work since implementation of MethodChannel class is final, please guide me with right approach of accessing method Channel instance from service.
Maybe this is not an issue i just don't know how to do it
Thanks in advance!

Hi @shivanshtalwar0
This issue doesn't seem to describe a bug or a feature request. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you

Seemed like a perfect question to me, I don't know why it was closed. I've seen people asking non-framework-related issues here a well.

I'm facing a similar issue.

@pedromassangocode please open this issue

Hi @tirth-stasis
Sorry this does not seems to be a valid issue. See https://github.com/flutter/flutter/issues/62738#issuecomment-668021592 for more context. In short, here is not a place for asking questions.
Thank you

This is a framework issue. Platform channels don't work in the background.

How did you conclude that it does not seem to be a valid issue? I already saw the comment you mentioned :)

If you know a way to do platform channel communication in the background, I would appreciate that. I tried searching everywhere I'm not able to find any documentation for this.


(#872) Update to support Cake 2.0.0

Fixes #872

All the changes that are contained within this PR have already been completed, so this PR is no longer required.


chore(main): release core-js 1.5.13-experimental

:robot: I have created a release beep boop
1.5.13-experimental (2024-11-13)
🧹 Chore

release main (99afb8e)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

core-js-v1.5.13-experimental
:sunflower:


send chunking

Does this actually help with large transfers? I noticed that transfers fail in nonblocking mode if they are larger than the socket can handle. I've implemented nonblocking sending in my fork after trying something similar to this PR, which didn't work because the socket still was not ready to receive more data.

Right now the error occurs anytime more than ~260KB is returned in a response. Our error pages are ~500KB so they are failing to load entirely.
If you think there's a better way to implement this that we can do in a patch or minor fix let's do that. The overhead of chunking bytes with this solution is not ideal.
If not, we can postpone your solution for a later 1.x or 2.x and get this fix in sooner than later as it's a fairly critical issue.
Error
Server error: respond(Transport.StreamError.send("There was a problem while sending data.", Socket failed with code 0 ("Undefined error: 0") [sendFailedToSendAllBytes]))

Line Throwing Error
https://github.com/vapor/socks/blob/b42c03d0afdf0f6547efed3b274de4e8f393579b/Sources/SocksCore/TCPSocket.swift#L81
Bytes Sent vs. Length
bytes sent: 261676
bytes in data: 508903

When this does get merged, how do we go about getting these changes? Just vapor build --clean?

@mcdappdev yep
vapor clean && vapor build

@andreasley you're right. I'm also seeing failures above 2MB. Closing in favor of #101.


Add UnitsNetSetup to hold global static state

I built on your recent QuantityInfoLookup and added UnitsNetSetup to gather global state in a single place as a singleton, with the possibility of passing instances of it to static methods later to override the defaults.
This way, state is no longer scattered among multiple classes that depend on each other in non-obvious ways and where initialization order has caused issues up until now.
Changes

Add UnitsNetSetup to hold global state as a singleton property Default that controls default state initialization.

Add properties for all "services" that depend on global state: UnitConverter, UnitAbbreviationsCache, UnitParser, QuantityParser

Forward all other Default singleton properties from these services to UnitsNetSetup.Default and mark obsolete
Add some TODOs where it seems functionality is missing

Moved to https://github.com/angularsen/UnitsNet/pull/1267


New interactive page has mobile format on web

I'm on Chrome - web browser on laptop.
When I go to https://processcode.publiccode.net/ and click on the "see this process code" link to 1A - I get a page that is shrunk to a mobile device size. See attached.

I'm also not a fan of the URL it now displays: https://processcode.publiccode.net/1au46-getting-oriented-all-breakpoints.html

The first issue has to do with the breakpoint of 1400 being too wide. @angelplasma can you make this a more appropriate size?
Not sure what to do about the URL other than manually rename the generated file... @amydarling what URL would you prefer to see?

1a hadn't been updated yet. I've updated it now. There are still some layout anomalies that will likely require some post-export code mods.
NOTE: I'm currently working in "Anima export 6" (to avoid any experiments Max was running in 5)
So: 1a and 1b are now relatively responsive. Trying (with limited success) to figure out best system before updating all layouts.


Enable loginWithPopup to optionally accept an existing popup window

This change was recently added to allow passing in an existing popup window to loginWithPopup allowing customization of additional windowFeatures in the window.open specification.
However, Auth0Provider appears to not have been updated and does not allow passing in this new configuration.
It'd be great to get this interface updated so we can use this new feature in auth0-spa-js!

My mistake - it is updated but not in the release I was using!

Hey @lukexor - going to do a release in about a week, which will have this in it


Cause tracker bug in DropItemEvent.Pre

Steps to reproduce:
1.) Listen for DropItemEvent.Pre
2.) Call event.getDroppedItems().clear() in listener
3.) Spawn mob using /summon and kill it by hand
4.) The following happens: https://pastebin.com/6RFGQ2FU
Also I should note that I tried adding to event.getDroppedItems(), but that has no effect.
Tested on SpongeVanilla 1.11.2-6.0.0-BETA-239.

I can fix this, however, please copy the correct log, since this one has artificial new lines added in several places, making it difficult to read.

I'm sorry, I copied the log from the windows console.. bad idea.
Here's the log from latest.log: https://pastebin.com/9XuZwrn8

Any updates on this? I don't want to be rude, but an essential plugin feature depends on this issue being resolved :)

There is another bug:
When cancelling DropItemEvent.Dispense the item drops anyway.
And when cancelling DropItemEvent (top-level):

The item isn't dropped if I drop it with Q
The item is dropped if I drop it through the inventory

This is still happening in 1.12 (bleeding branch)
The first bug:
The problem is with DropItemEvent.Pre only when it doesn't involve a player, rather that the whole event. As a proof try use /kill. There would be no stacktrace, but why ? When a player drops something, https://github.com/SpongePowered/SpongeCommon/blob/bleeding/src/main/java/org/spongepowered/common/entity/EntityUtil.java#L981 is called, but there is a check to handle an empty list https://github.com/SpongePowered/SpongeCommon/blob/bleeding/src/main/java/org/spongepowered/common/entity/EntityUtil.java#L1008. Now, spot the difference with https://github.com/SpongePowered/SpongeCommon/blob/bleeding/src/main/java/org/spongepowered/common/entity/EntityUtil.java#L932.
Fixed by changing that line to: item = event.isCancelled() || dropEvent.getDroppedItems().isEmpty() ? null : ItemStackUtil.fromSnapshotToNative(dropEvent.getDroppedItems().get(0));

any update?
cancel event still cause on cursor item disappear

@ImMorpheus can you split this up into multiple issues for whatever is still an issue on latest? kinda hard to keep track whats still relevant here

Done. https://github.com/SpongePowered/SpongeCommon/issues/1879


feat: label support ExcludeLabels

[x] add test case
[x] update Doc
[x] update Helper

/hold

[ ] add test case
[ ] update Doc
[ ] update Helper

Codecov Report

Merging #177 (1806201) into master (a8af690) will decrease coverage by 0.32%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master     #177      +/-   ##
==========================================
- Coverage   85.90%   85.58%   -0.33%
==========================================
  Files          11       11
  Lines         972      978       +6
==========================================
+ Hits          835      837       +2
- Misses         77       79       +2
- Partials       60       62       +2

Flag
Coverage Δ

unittests
85.58% <50.00%> (-0.33%)
:arrow_down:

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

Impacted Files
Coverage Δ

internal/pkg/externalplugins/config.go
100.00% <ø> (ø)

internal/pkg/externalplugins/label/label.go
82.40% <50.00%> (-2.89%)
:arrow_down:

Continue to review full report at Codecov.

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

/hold cancel

/assign @Mini256
PTAL~ Thanks!

@Mini256  PTAL~

/lgtm

/merge


Create centralized method for adding tags to records

Assumption: all modules that have json schemas, and which use tags, format them according to https://github.com/folio-org/raml/blob/master/schemas/tags.schema (agreements and licenses allow a simpler formatting when posting)
Question: should we treat tags-in-tags-storage as reference data to be set up separately beforehand and then validated, or as extradata to be created along with the tagged records? (We want to avoid duplication.)
The migration tools already allow adding tags to organizations: https://github.com/FOLIO-FSE/folio_migration_tools/issues/297

I think I'd prefer using it as ref-data @branchedelac .
Or, if we do it like extradata, batchposter should check for existence before posting.

I agree! Handling it as ref data makes more sense.
Even if the tags API will protest if you try to add a second tag with the same label as an existing one, handling this as extradata/putting the responsibility on the BatchPoster could become quite... noisy.


Fix ESLint errors for environment variable provider

Lnting for https://github.com/microsoft/vscode-python/pull/15377

Codecov Report

Merging #15383 (834c68c) into main (6267d38) will not change coverage.
The diff coverage is 75%.

@@          Coverage Diff           @@
##            main   #15383   +/-   ##
======================================
  Coverage     65%      65%
======================================
  Files        560      560
  Lines      26850    26850
  Branches    3876     3876
======================================
  Hits       17495    17495
  Misses      8635     8635
  Partials     720      720

Impacted Files
Coverage Δ

...t/common/variables/environmentVariablesProvider.ts
93% <75%> (ø)


Set a Clear Guidelines on Products

with launch of our expanded community moderation tools, we'll need to set a clear guideline on what products belong on OpenHunt.co, and make sure all the moderators know what the community is looking for.
Obviously this is going to be an ongoing discussion throughout the life of Open Hunt, but it just needs something simple and clear to kick things off

Referenced by #64

I haven't thought deeply about this but I wanted to break the ice on this discussion (and my own commenting on this project). The basics could be:

New tech product (hardware or software).
Recently and substantially updated tech product.

Things that I see on PH that I think are worth discussing as to whether we would like to encourage them here are "Email newsletters", "Y Combinators Holiday Shopping Recommendations", etc.
I think the question is whether we want submissions that are actual technology or is just being technology related good enough?
I could see going the actual technology route as being a nice differentiator from PH.

I agree on actual technology route.
We should write up a draft for this. I think it's blocking adding new moderators because we don't have clear guidelines yet on what they should be moderating.
@wuliwong want to take a stab at it?  thanks!

@jacquescrocker Sure, things are going to be a little slow until after New Year's, I have to use my non-family time for my "real" job until then.  :)

Do post up a draft, even if it's on a napkin, so we can chip in and build on it together. :+1:


bug(linter) incorrect fixer in no_null

see https://github.com/oxc-project/oxlint-ecosystem-ci/actions/runs/10518058484/job/29218160739?pr=25
return null!

is being fixed to
return !;

which is invalid TS

another one:
  x Expected a semicolon or an implicit semicolon after a statement, but found none
    ,-[packages/backend/server/src/core/quota/service.ts:73:19]
 72 |         } catch {}
 73 |         return  as unknown as typeof quota & {
    :                   ^
 74 |           feature: QuotaConfig;
    `----
  help: Try insert a semicolon here

Re-opening this because @Dunqing's PR only handled the first mentioned problem, not the second.

I'll look at this - i think i will make the fixer a lot more concervative as to whether it fixes or doesn't.

Re-opening this because @Dunqing's PR only handled the first mentioned problem, not the second.

https://github.com/oxc-project/oxc/blob/05d25e2aa371013443615441a0bc79d5e17ee67e/crates/oxc_linter/src/rules/unicorn/no_null.rs#L383
It's also resolved

still broken 😞
although maybe this is a different issue.
const newDecorations = enabled ?
	this._debugService.getModel().getBreakpoints().map(breakpoint => {
		const parsed = CellUri.parse(breakpoint.uri);
		if (!parsed || parsed.notebook.toString() !== this._notebookEditor.textModel!.uri.toString()) {
			return null;
		}

		const options: INotebookCellDecorationOptions = {
			overviewRuler: {
				color: debugIconBreakpointForeground,
				includeOutput: false,
				modelRanges: [new Range(breakpoint.lineNumber, 0, breakpoint.lineNumber, 0)],
				position: NotebookOverviewRulerLane.Left
			}
		};
		return { handle: parsed.handle, options };
	}).filter(x => !!x) as INotebookDeltaDecoration[]
	: [];

is being fixed to:
const newDecorations = enabled ?

	: [];

@mysteryven do you want to fix this?

I'm wondering if we should make this a suggestion 🤔

actually i'm not sure why our fixer works this way
https://github.com/sindresorhus/eslint-plugin-unicorn/blob/204d31c9c2a92496a59635d9f42a532e1dc46f90/rules/no-null.js#L87-L122
unicorn:

if its the arg of a loose equals, suggests replacing with undefined
suggests deleteing the  null literal if it's the argument of a return stmt
suggests deleting it if it's not a const var decl
suggests replacing with undefined

mysteryven do you want to fix this?

Oh, please excuse the delay of two days, I am always AFK these days due to something in my life. I'll take a look today!

no worries,
i think this one is fixed now https://github.com/oxc-project/oxc/commit/d6b84ec44e874fb5084b510cb9c0dbbc0334c5df


DataType JSON auto convert to String

To reproduce the problem, I created a dataset from the abc table
create table public.abc
(
  source_id varchar(255) not null,
  json jsonb,
  constraint pk_source_id primary key(source_id)
);

insert into abc (source_id, json) VALUES (1, '{"key": "value"}');

then I checked that my column type was json.

and when I get an object for handlebars, I get very upset that I get a string instead of an object.

and I can't do {{this.json.key}} because there is a string in the json
{"source_id":"1","json":"{'key': 'value'}"}

After digging a bit in the source code, I found that this is related to the lack of support for complex data types as can be seen here:
https://github.com/apache/superset/blob/419d12281612bd0cd1bdca121efbb39f02d7cb1e/superset/utils/core.py#L158
and here:
https://github.com/apache/superset/blob/419d12281612bd0cd1bdca121efbb39f02d7cb1e/superset/result_set.py#L155
And is mentioned in related issue, here: https://github.com/apache/superset/issues/19388

would be great this to work!


Joint Controllers not working properly when joint limits are specified

Environment

OS Version:

Distributor ID:	Ubuntu
Description:	Ubuntu 22.04.1 LTS
Release:	22.04
Codename:	jammy

Source build  - branch ign-gazebo6

Description

Expected behaviour: In Velocity or Force mode joint position limits should be respected and the joints should be able to recover if the commanded velocity results in hitting the position limits.
Actual behavior: When using joints with position limits, which is often the case in robotics, the Velocity mode joint controller can't recover when it hits the limit --> model joint_controller_demo while the Force mode joint controller is stalling - accumulates the error in the integral component (unless we specify manually some small value - absolute values for i_max and i_min ) which means the anti-windup clamping method default parameters are too high - at least there should be some "good guess" explanation for the users of ignition::math::PID and directions on how it works with the transfer function of the joint. The only situation in which Force mode joint controller works without the problem is when the sequential commanded velocities are of similar absolute value with opposite sign.

Steps to reproduce

git clone https://github.com/gazebosim/gz-sim.git
cd gz-sim && mkdir build && cd build && cmake ..
make && sudo make install
bash -c "echo 'export IGN_CONFIG_PATH=/usr/local/share/ignition' >> ~/.bashrc"
source ~/.bashrc

Go to ~/.../gz-sim/examples/worlds/joint_controller.sdf and for model joint_controller_demo and its joint j1 under the <axis> section add following:

<limit>
  <upper>0.75</upper>
  <lower>-0.75</lower>
  <effort>100</effort>
  <velocity>10</velocity>
</limit>

Do the same for joint j1 for model joint_controller_demo_2.

Enforce the changes globally on your system with make && sudo make install.
Run the example ign gazebo joint_controller.sdf -v
Test the Velocity mode joint controller with:

ign topic -t "/model/joint_controller_demo/joint/j1/cmd_vel" -m ignition.msgs.Double -p "data: -5.0"
joint j1 of model joint_controller_demo will not move

Test the Force mode joint controller with:

ign topic -t "/model/joint_controller_demo_2/joint/j1/cmd_vel" -m ignition.msgs.Double -p "data: 5.0"
then after 2 seconds execute: ign topic -t "/model/joint_controller_demo_2/joint/j1/cmd_vel" -m ignition.msgs.Double -p "data: -0.5"
joint j1 of model joint_controller_demo_2 will stall and start moving after > 30 seconds

Output
https://user-images.githubusercontent.com/26708124/187866697-2105180b-34f3-4f01-bb76-95c55b530baa.mp4

Not sure if its strictly related, but I had a similar weird issue. When using the component JointVelocityCmd and setting the velocity close to the velocity limit, the command was completely ignored.
Using JointVelocityReset didnt have this issue. Moreover, multiplying with 0.99 solved the issue before sending the command (99% of the velocity limit) but multiplying with 99.9% didn't, implying there might be some rounding error somewhere ?
I would expect that if the command is above the limit that a warning is printed if its ignored, and that even if the command is equal or less to the limit it would be executed. Even potentially capping it to the limit and printing a warning, not just ignoring it.

Hi,
I seem to have a similar problem with joint trajectory controller - if the joint ever reaches the joint limit, it becomes unrecoverable - all the next joint trajectory commands will not move the joint anymore. I can prevent that by setting joint limits in the urdf to e.g. negative value so the model itself will get in self-collision and physically prevent the joint from reaching the limit, but that obviously seems quite hacky. Anyone encountered that as well and figured out a reasonable solution?
My setup:
Ubuntu 20.04
ROS2 Humble
Plugin for control: name='ign_ros2_control::IgnitionROS2ControlPlugin' filename='ign_ros2_control-system'

I was able to reproduce the problem and it looks like it's a bug in DART. The following patch to the dart/constraint/JointConstraint.cpp on the release-6.13 branch fixes the issue for me.
diff --git a/dart/constraint/JointConstraint.cpp b/dart/constraint/JointConstraint.cpp
index faa072d8f21a..bf032f0ffb2a 100644
--- a/dart/constraint/JointConstraint.cpp
+++ b/dart/constraint/JointConstraint.cpp
@@ -238,7 +238,7 @@ void JointConstraint::update()
         // the position error.

         const double C1 = mErrorAllowance * A1;
-        double bouncing_vel = -std::max(B1, C1) / timeStep;
+        double bouncing_vel = -std::min(B1, C1) / timeStep;
         assert(bouncing_vel >= 0);
         bouncing_vel = std::min(bouncing_vel, mMaxErrorReductionVelocity);

@@ -280,7 +280,7 @@ void JointConstraint::update()
         // the position error.

         const double C2 = mErrorAllowance * A2;
-        double bouncing_vel = -std::min(B2, C2) / timeStep;
+        double bouncing_vel = -std::max(B2, C2) / timeStep;
         assert(bouncing_vel <= 0);
         bouncing_vel = std::max(bouncing_vel, -mMaxErrorReductionVelocity);

Without this change bouncing_vel is always 0 and does not contribute to fixing the constraint violation. When the joint reaches the position limit, the desired velocity change is computed as
mDesiredVelocityChange[i] = bouncing_vel - velocities[i];, but since both bouncing_vel and velcoties[i] will be 0 at that point, the desired change is also 0.

https://github.com/dartsim/dart/pull/1774 has been merged upstream. This will be available for Harmonic at some point, but will not be fixed for Fortress and Garden. @j-rivero Can we get a release of dart-6.13?

Has this been fixed in Harmonic? I'm using ROS2 Humble and updated my gazebo to Harmonic in hope to fix this problem, yet it still exists. I'm using the gz-harmonic=1.0.0-1~jammy and ros-humble-ros-gzharmonic=0.244.12-3jammy apt packages.

Yes, a new version of DART (6.13.1) is now available which should fix this problem, so make sure you've updated DART on your system.

I have DART(6.13.2) yet still having the same issue.
I am using gz::sim::components::JointVelocityCmd to set a velocity for a specific joint. Joint becomes unresponsive and unrecoverable once it reaches its limits. Getting the same error using JointTrajectoryController.

I tested it locally following the reproduction steps in the PR description (changing ign to gz) and I found that the velocity mode (item (5) in the list) works without any issues. The force mode (item (6)) did not work at first and behaved similar to what was described in (6), but changing the integral gain in the SDF to 0 fixed it for me. Alternatively, setting the i_max and i_min to 0.001 and -0.001 respectively also fixed it. So I think the issue with (6) is integral windup, not an issue with DART.
@ufrhaidar can you try to reproduce the steps above? Or do you have an example that can demonstrate your issue?

I am not using the JointController plugin, but a similar customized version of it that reads the velocity command from a different source (not a gazebo topic) and follows the same logic as JointController's Velocity mode.
Both plugins set a JointVelocityCmd as the last step in the PreUpdate function.
https://github.com/gazebosim/gz-sim/blob/6af74459f4b0f5a2a33fa112e24ac50633a90d3b/src/systems/joint_controller/JointController.cc#L347
I assumed the DART issue was shared between them both (which it is), but it looks like there might be other issues happening after.
@azeey thanks for your help. I'll workaround the issue for now on my end. Will look into the issue furthermore when I have more time.

@azeey the integral gain issue you mentioned is possibly caused by the PID in gz-math not having an anti-windup. As you reach limit, the integral error keeps increasing.

@azeey
if you were to use this sdf example.sdf.txt and start the sim
gz sim -v 4 -r example.sdf.txt

Then wait for the joint to reach the limit and command a negative velocity
gz topic -t /model/joint_controller_demo/joint/j1/cmd_vel -m gz.msgs.Double -p "data: -0.1"

does it not get stuck?
@livanov93 @ahcorde any updates on this issue?

I was able to reproduce the issue as well. It seems whether it gets stuck at the joint limit depends on the commanded velocity. In the example.sdf.txt file from @ufrhaidar , if we change the <initial_velocity> parameter to 0.2, the joint does not get stuck.

I was able to reproduce the issue as well. It seems whether it gets stuck at the joint limit depends on the commanded velocity. In the example.sdf.txt file from @ufrhaidar , if we change the <initial_velocity> parameter to 0.2, the joint does not get stuck.

@azeey Yes that was interesting. If you command it before the <initial_velocity> reaches the limit, it works fine, otherwise it gets stuck.
When I was looking at it last time, i noticed that it was getting stuck in the velocity limit bound section in dartsim
https://github.com/dartsim/dart/blob/f9ef83beba401fb1a93bd0727ffc98022dc5f916/dart/constraint/JointConstraint.cpp#L292

Hi,
is there a fix for this behavior? In my case only gets stuck in one of the limits (the other works fine), but none of the suggestions above seem to solve it so far. Thanks!


dialogflow: cleanup unused samples

Dialogflow: deletes unused samples + containing files and corresponding rspec tests

@googlebot I signed it!

Dialogflow samples have been moved to googleapis/google-cloud-ruby.


Add UI enhancements

Related Issues

https://acme-climate.atlassian.net/browse/MG-441?atlOrigin=eyJpIjoiZjg0YWIzMGYxN2ExNDlhMWJhMTg5NWQyMWM1M2ZjZTQiLCJwIjoiaiJ9

Overview
This PR updates styling of various components to improve the presentation for users.

This was an accidental merge because of a push to master.


Test cli on pull requests

Addresses #168

thank you!!

thank you @blink1073 !! i'm going to merge this and then create a new release. in the future you can just merge (if you want!) !!


Training is Too Slow

Hi @TreB1eN,
While training on COCO with my RTX 2080 Super GPU, it is taking around 20 hours per epoch which is to slow. I mean to run 30 epochs, I need to wait for around 25 days?
How can the training be accelerated? Thanks

it runs RPN in numpy so speed is slow, you can try using pytorch code to accelerate


training problems

Hello m8, when I try to train a custom model, this error appears and I do not know how to solve it.

loading filelist... done
setup model... done
check forward path... done
starting processes of dataset sampler... done

epoch: 0
inner epoch: 0
Process Process-1:
Traceback (most recent call last):
File "/usr/lib/python3.6/multiprocessing/process.py", line 258, in _bootstrap
self.run()
File "/usr/lib/python3.6/multiprocessing/process.py", line 93, in run
self._target(*self._args, **self._kwargs)
File "/content/waifu2x-chainer/lib/dataset_sampler.py", line 85, in _worker
xc_batch, yc_batch = pairwise_transform(img, cfg)
File "/content/waifu2x-chainer/lib/pairwise_transform.py", line 148, in pairwise_transform
cfg.nr_rate, cfg.chroma_subsampling_rate, cfg.noise_level)
File "/content/waifu2x-chainer/lib/pairwise_transform.py", line 66, in noise_scale
with iproc.array_to_wand(src) as tmp:
File "/content/waifu2x-chainer/lib/iproc.py", line 75, in array_to_wand
dst = wand.image.Image(blob=buf.getvalue())
NameError: name 'wand' is not defined
Process Process-2:
Traceback (most recent call last):
File "/usr/lib/python3.6/multiprocessing/process.py", line 258, in _bootstrap
self.run()
File "/usr/lib/python3.6/multiprocessing/process.py", line 93, in run
self._target(*self._args, **self._kwargs)
File "/content/waifu2x-chainer/lib/dataset_sampler.py", line 85, in _worker
xc_batch, yc_batch = pairwise_transform(img, cfg)
File "/content/waifu2x-chainer/lib/pairwise_transform.py", line 148, in pairwise_transform
cfg.nr_rate, cfg.chroma_subsampling_rate, cfg.noise_level)
File "/content/waifu2x-chainer/lib/pairwise_transform.py", line 66, in noise_scale
with iproc.array_to_wand(src) as tmp:
File "/content/waifu2x-chainer/lib/iproc.py", line 75, in array_to_wand
dst = wand.image.Image(blob=buf.getvalue())
NameError: name 'wand' is not defined

Wand is the cause of the problem.
I use ubuntu 18.04 and cuda 10.0

Please install wand.
http://docs.wand-py.org/en/0.5.4/


chore/rule: add expr test for rule

I found a cast bug

Attempt to divide by zero can be used as a separate test. I refer to calcite to do the same.

I will complete the todo content in the comments and try to fix this bug.
tidb and mysql also seem to have this bug

I have a question that I would like to ask in this PR. I would like to ask if this situation is considered a bug.

I feel that the display results of risinglight's select can be improved, and I am trying this part of the work

Here is the output of DuckDB:
D select cast('8145308033243873281' as double) ;
┌───────────────────────────────────────┐
│ CAST('8145308033243873281' AS DOUBLE) │
│                double                 │
├───────────────────────────────────────┤
│                 8.145308033243873e+18 │
└───────────────────────────────────────┘

So yes, I also think the current output of RisingLight is not correct.

Thank you for your reply and patient answer. As far as I know, tidb and mysql also have similar bugs. I will try to solve this problem in risinglight. Finally, I would like to thank you for the risinglight project. As a beginner, I have learned a lot. Thank you very much for your contribution.

The work of risinglight community is very meaningful


Add open-graph tags support

@jchapuis Can you please rebase to solve conflicts? Thanks!

@mkurz sure! done

@jchapuis You need to format the codebase

@jchapuis You need to format the codebase

yes i saw it was build.sbt i believe, pushed a commit

/cc @mdedetrich maybe you can also take a look here if you have time


Keep serverdata from base URL

When querying multi-page data, the current code completely takes the value from the next-parameter in the json answer.
However if the netbox instance is multi-homed, or accessed unusually, for example by using a tunnel, or using direct IP notation (no working DNS), we may run into an Exception.
With this patch, instead of overriding the complete URL with data from next parameter, we keep the server name/port and scheme with which the api was originally initiated.

Closing, not GitHub issue, no tests, no response


Building on 32bit chroot on 64bit host produces objects in /usr/lib64

While building in a 32bit chroot on a 64bit host the shared object is installed in /usr/lib64 instead of /usr/lib.
I don't see any cmake option to specify libdir.
This line seems to be doing it:
get_property(LIB64 GLOBAL PROPERTY FIND_LIBRARY_USE_LIB64_PATHS)
I can get around it by sedding the CMakeLists.txt for now, but wonder if there's a better way?

I should add that this is also the case when building in a 32 OS where the CPU is 64bit.

So… you are using CMake? I'm asking as there are literally 3 supported build systems.

Yes.. but autotools seems to work better. I noticed the cmake files but not the configure. Thanks for the headsup and sorry for the noise. Closing.

@slackhead Uh, they are not autotools, but a hacked-up version of FFmpeg's configure script. But good to hear it's working.

Oops that's a different project I maintain. sorry about the noise.

I think the issue itself still stands. My gut tells me that it's your CMake setup's problem.


uplink credentials

Hello,
I'm using verdaccio both on raspberrypi on my local network and on a server on the cloud.
The one on the cloud it's restricted to a small collection of users. I want to be able to publish to my local verdaccio registry on my raspberry pi, and defer the publication to the cloud to it.
I have the uplink of my PI configured to point to my verdaccio running on the cloud. I have also generated a .npmrc file with the login credentials of the uplink registry.
However, I'm unable to install packages that are not on my local PI registry. The remote verdaccio is not getting the required credentials, therefore it is returning a 403 response. How can I make verdaccio pick the credentials?
Thanks in advance.

@danielo515  Could you provide any verbose log, configuration?

Sure @juanpicado, what is the correct setting for verdaccio ? Just setting the level to debug is enough ?

@danielo515 for extreme debug you could do the following in each instance
DEBUG=express:* verdaccio

and then npm in verbose mode using npm install --verbose xxxx

@danielo515 When you say "you created a .npmrc file with the login credentials of the uplink registry", the registry your client uses is still your RPi, right? The verdaccio server running there won't pass the credentials provided by npm to the upstream registry. At least I wouldn't know that this is currently supported.
If you want the verdaccio server running on your RPi to authenticate against the verdaccio server running in the cloud you have to configure it that way (i.e. provide the credentials for the upstream server in your verdaccio config file). Here's an example config snippet to do so:
  npmjs:
    SOMEOTHEROPTIONS
    headers:
       Authorization: "Basic dXNlcjpwYXNzd29yZA=="

Dear @jachstet-sea,
Thank you for your answer. The scenario you described is almost the one I have except for one detail : mi RPI has a global .npmrc file that defined the uplink and the credentials. I thought that was enough, but since verdaccio uses plain HTTP requests it is probably not. Can I specify tokens auth instead of basic auth ? Where is the documentation of the option you mentioned ? Can I use for something that is not npmjs?

Yes, the verdaccio server running on the RPi won't read the .npmrc file. With the config snippet I posted, you can send any static headers to the server you like. So if your tokens are valid for a looooong time, I don't see a problem. If they are only valid for a day, you'd have to change/update the config every day which is not a good option.
The option I mentioned is "documented" in conf/full.yaml. The other option is that someone implements a proper username/password auth option in verdaccio that does not use a static header. OR a way to pass on headers provided by the client to the upstream server. But this has to be done carefully since it could leak auth credentials to servers who should not see them.

Thank you @jachstet-sea
I'll try your suggestions.

Any idea about how to use npm tokens instead of basic auth? I want to get socped packages from npm

This will be shipped on v2.4.1. Closing this ticket.

Hello @juanpicado
What is exactly being shipped ? The ability to read a .npmrc ? What is the new feature exactly ?
Thanks in advance

@danielo515 check #332 . I'll ship as soon this milestone is completed https://github.com/verdaccio/verdaccio/milestone/13


Backport runAllTasks changes from Netty5 to Netty 4

Expected behavior
set the ioRatio to 1 will not delay the ScheduledTask running
Actual behavior
Currently, in Netty 4.1, If I change the ioRatio from default 50 to 1,  the current runAllTasks in Netty 4 may delay the ScheduledTask running
Netty version
4.1

@He-Pin PRs welcome


PyCon US 2024

This is a draft of the PyCon US 2024 videos data set. It should be largely complete, but I need to manually clean up a few issues before it can be merged:

Some talks may be missing videos. These should be corrected if the videos are available or removed if they are unavailable / private.
Only one session of lightning talks is present, so the others should be added.
Speaker info may be missing when the official PyCon US API response did not include it (e.g. in the "Welcome" session). These cases should be fixed.


Feature/vue200fix

support vue2.0

Sorry to not merge this PR directly because I had been working on support for Vue 2.0+ some time ago but still thank you for your contribution!

It's ok.
BTW, I'm a JS rookie, and not sure whether the code is properly written.


[Update] Facebook for iOS v4.32.0

Updated to version 4.32.0
Added PlacesKit framework

@dalexsoto Added suggested changes, also, I updated it to v4.33. PlacesKit didn't change but CoreKit and LoginKit had some little changes.


Problem with Bicep Template

Bicep CLI version 0.30.3
I'm trying create a Private Endpoint for a Web App. I'm deploying via Powershell.
All the resources excist, except for Network Interface Card, Private IP address and the Private Endpoint itself (of course) which will be deployed with this template.
I get this error message:
"New-AzResourceGroupDeployment: 14:07:26 - The deployment 'ec6adf02-bb19-49da-8a7c-c4de5c920cab' failed with error(s). Showing 1 out of 1 error(s).
Status Message: Unable to process template language expressions for resource '/subscriptions/subscriptionID/resourceGroups/bicep-rg-test-001/providers/Microsoft.Network/privateEndpoints/privateapptest-pep-test-001' at line '6' and column '9'. 'The language expression property array index '1' is out of bounds.' (Code:InvalidTemplate)
CorrelationId: 5eaf3597-e4e7-4e52-873e-a6e568f00cdd"
Line 6 is a Description, so it makes no sense...
Here is the code:
`
@description('Is used for the CreationDate-tag so you do not have to enter the date manually')
param TodaysDate string = utcNow('dd.MM.yy')
@minLength(3)
param EmailAddress string
@allowed([
'test'
'prd'
'dev'
])
param Environment string
@description('Tags that will be associated with every resource deployed with this template.')
param Tags object = {
Env: Environment
CreatedBy: EmailAddress
CreationDate: TodaysDate
}
@minLength(3)
@description('Name of the App Service.')
param AppServiceName string
@minLength(3)
@description('Name of the App Service.')
param PrivateIPAddress string
@minLength(3)
@description('Custom Domain of the App Service')
param CustomDomainName string
@description('The geographical location where the resources will be deployed.')
param Location string = resourceGroup().location
var PEPSubnetName = 'privateapp-subnet-prd-001'
var PrivateDNSZoneName = 'privatelink.azurewebsites.net'
var PrivateEndpointName = '${AppServiceName}-pep-${Environment}-001'
var NetworkInterfaceName = '${AppServiceName}-nic-${Environment}-001'
var PrivateIPName = '${AppServiceName}-ip-${Environment}-001'
resource PrivateDNSZone 'Microsoft.Network/dnsZones@2018-05-01' existing = {
name: PrivateDNSZoneName
}
resource PEPSubnet 'Microsoft.Network/virtualNetworks/subnets@2024-01-01' existing = {
name: PEPSubnetName
}
resource AppService 'Microsoft.Web/sites@2023-12-01' existing = {
name: AppServiceName
}
resource PrivateEndpoint 'Microsoft.Network/privateEndpoints@2024-01-01' = {
name: PrivateEndpointName
location: Location
tags: Tags
properties: {
privateLinkServiceConnections: [
{
name: PrivateEndpointName
id: AppService.id
properties: {
privateLinkServiceId: AppService.id
groupIds: [
'sites'
]
privateLinkServiceConnectionState: {
status: 'Approved'
actionsRequired: 'None'
}
}
}
]
manualPrivateLinkServiceConnections: []
customNetworkInterfaceName: NetworkInterfaceName
subnet: {
id: PEPSubnet.id
}
ipConfigurations: [
{
name: PrivateIPName
properties: {
memberName: 'sites'
groupId: 'sites'
privateIPAddress: PrivateIPAddress
}
}
]
customDnsConfigs: [
{
fqdn: CustomDomainName
ipAddresses: [
PrivateIPAddress
]
}
]
}
}
resource PrivateEndpointName_default 'Microsoft.Network/privateEndpoints/privateDnsZoneGroups@2024-01-01' = {
parent: PrivateEndpoint
name: 'default'
properties: {
privateDnsZoneConfigs: [
{
name: PrivateDNSZoneName
properties: {
privateDnsZoneId: PrivateDNSZone.id
}
}
]
}
}
`

Found out that i had to reference the Vnet as parent.


adding contribution in readme.md

I have added contribution guidelines in README.md file
#8

@Ayushparikh-code please review this PR

@Ayushparikh-code please review this PR

I approved and merged but can u explain me levels & points section u added ?


DApp does not show current state of smart contract

Description
Type: Bug
Summary: the embark_demo default page returns blank value for state of smart contract
Expected behavior
After /browse to the correct URL and clicking on `Get Value , the DApp page should show the smart contract has an initial value of 100.
Actual behavior
The initial value returns blank.
Reproduction
/debug On
adb reverse tcp:8081 tcp:8081
adb reverse tcp:3449 tcp:3449
adb forward tcp:5561 tcp:5561

embark simulator
embark run
Browse to appropriate URL.
Zoom in. Click "Get Value."
Click the button to get the current value. The initial value is
Additional Information

Operating System: Android

Hi @coyotespike, I believe this issue is now quite outdated. Is this still an issue for you?


COWSwap trades "Not enough asset.chain to cover gas"

Overview
Currently in production, when performing a near max balance trade of a COWSwap eligible trade pair, The app blocks the user providing a "Not enough asset.chain to cover gas"  of the outgoing asset in the trade pair.  Trades leaving ample dust are able to be performed.
References and additional details
https://jam.dev/c/91141bf8-26df-43bb-a264-fd80fd47d08e
Video with console log errors ^
Additionally there is a captured issue of Li.Fi having issues calcualting gas that was captured in the recreation of the trade error message.
 A user should not have to be left with dust if they would like to trade their whole balance of an asset and have the correct gas in thir address to pay for the trade.
Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

Main issue to be addressed here
Trade MAX of an ERC20 fails with cowswap

For reference after checking the jam: Li.Fi errors are 429s - not so much failures but rather rate limiting, trying again would make things happy, though we should obviously fix those 429s with our own Li.Fi proxy eventually. See https://docs.li.fi/rate-limits-and-api-key

Re: CoW, following the new fee model, the fees are added on top of the amount being sold. i.e if selling 200 FOX, you'd be selling 20 FOX + at most  0.03299079992581711 FOX fees bribing the solvers.
See this doc for ref:
https://docs.google.com/document/d/1w1jxRUfSbQU-15FgC-sw85pY2Ep8-VDACEwzvndylvk/edit#heading=h.mk00c6ka1y5k
Now the fee would always be quite low, and at execution time, there may not be a fee at all in the end.
Since close to send maxes seem to be the issue here, I believe the best way to get around this would be to simply assume no fee - and construct a payload with no fees in the case the amount is between max balance and max-balance - fee.
See this PR for reference https://github.com/shapeshift/web/pull/6455

Since close to send maxes seem to be the issue here, I believe the best way to get around this would be to simply assume no fee - and construct a payload with no fees in the case the amount is between max balance and max-balance - fee.

If we construct a payload with no fees, doesn't the order have less of a chance of being filled?  Why can we not instead construct the payload to include the fee, but just less sell amount and if the user ends up with some small dust, thats an acceptable outcome.

(again, assuming the estimated fee is very low)

Since close to send maxes seem to be the issue here, I believe the best way to get around this would be to simply assume no fee - and construct a payload with no fees in the case the amount is between max balance and max-balance - fee.

If we construct a payload with no fees, doesn't the order have less of a chance of being filled? Why can we not instead construct the payload to include the fee, but just less sell amount and if the user ends up with some small dust, thats an acceptable outcome.

We absolutely could - though that's not the spec that we were told by by the CoW team @0xean. But yeah, I also believe this is the best option, removes the guesswork both for us and users (paying additional token fees is quite uncommon to say the least!)

great, thanks!


[UI] Add 'User Access Token Expiry Time' and 'Refresh Token Expiry Time' as mandatory fields in UI when adding OAuth/OpenID Connect Configuration while creating SP

== Environment ==
IS 5.4.0 Update 3 - 2 node cluster
JDK - 1.8
DB - mysql 5.6
OS - ubuntu 16.04
Browser - Chrome - Version 59.0.3071.115
== Steps to Recreate ==

Login to management console, click 'Add' under Service Providers
Register a service provider
Go to 'Inbound Authentication Configuration' and click configure in 'OAuth/OpenID Connect Configuration'
Provide a callback url
Keep empty at least one from 'User Access Token Expiry Time' and 'Refresh Token Expiry Time'

== Actual Outcome ==
There're warning pop up to fill the respective field.

== Expected Outcome ==
Mention above fields as mandatory in UI or keep the optional by allowing to save the oAuth applciation without any if the above 2 fields

these fields should not be mandatory if only the client-credentials grant type is enabled.

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


Cannot find module scroll-into-view-if-needed

After upgrading to v5.0.5, this error popped up:
Cannot find module 'scroll-into-view-if-needed' imported from<EMAIL_ADDRESS>Not sure if this only happens in pnpm. I need to install scroll-into-view-if-needed manually to resolve it.

Oh interesting, I had assumed vite deps inlining would bundle the code from scroll-into-view-if-needed (see fbf9d8e). Should probably have read the docs first. I'll pull v5.0.5 from NPM.


Preparing release v0.8.0

I'm preparing the v0.8.0 release and at the same time trying to switch to use the OpenAstronomy Azure Pipelines template for auto-releasing when a tag is pushed. I tried to keep the release instructions as close to the original as possible. In practice, for other packages I don't bother with the release branch and PR, so the process is simpler (update changelog and versions, push, then once CI passes, tag, push to master).

Awesome, thanks!


improve error and node

NOT READY FOR DEVELOP

fix(rust): ockam_error not present in cargo lock
refactor(rust): remove address, node, queue and worker
refactor(rust): remove worker examples
feat(rust): improve error and move it to ockam core
feat(rust): improve std node and node macro
feat(rust): add node examples

This PR moved a lot of things around but is based heavily on previous work by @jared-s @robinbudd & @SanjoDeundiak
I will add Co-authored-by: tags to commits when its ready for merge.
@jared-s @robinbudd @SanjoDeundiak pushing so you know the state and can work in parallel.

This PR moved a lot of things around but is based heavily on previous work by @jared-s @robinbudd & @SanjoDeundiak
I will add Co-authored-by: tags to commits when its ready for merge.
@jared-s @robinbudd @SanjoDeundiak pushing so you know the state and can work in parallel.

@SanjoDeundiak please review the changes to error ... it has moved to ockam_core::Error

@SanjoDeundiak please review the changes to error ... it has moved to ockam_core::Error

the commit stack uptil 4a3bb01f3f05bbb4a850139f60d2e48aa3d161cb can be a separate PR that can be merged sooner ... break it down

the commit stack uptil 4a3bb01f3f05bbb4a850139f60d2e48aa3d161cb can be a separate PR that can be merged sooner ... break it down

this is stacked on top of #791

this is stacked on top of #791