Support for Python 2.7.x on windows

At the moment, the installation on Windows platform / Python 2.7.x is failing with this error (tested on appveyor) :
c:\projects\pyjks.tox\py\include\site\python2.7\twofish:
running install
running build
running build_py
creating build
creating build\lib.win32-2.7
copying twofish.py -> build\lib.win32-2.7
running build_ext
building '_twofish' extension
creating build\temp.win32-2.7
creating build\temp.win32-2.7\Release
creating build\temp.win32-2.7\Release\twofish-0.3
C:\Users\appveyor\AppData\Local\Programs\Common\Microsoft\Visual C++ for Python\9.0\VC\Bin\cl.exe /c /nologo /Ox /MD /W3 /GS- /DNDEBUG -Itwofish-0.3 -Ic:\python27\include -Ic:\projects\pyjks.tox\py\PC /Tctwofish-0.3/twofish.c /Fobuild\temp.win32-2.7\Release\twofish-0.3/twofish.obj
twofish.c
c:\users\appveyor\appdata\local\temp\1\pip-build-in0ieu\twofish\twofish-0.3\twofish.h(13) : fatal error C1083: Cannot open include file: 'stdint.h': No such file or directory
error: command 'C:\Users\appveyor\AppData\Local\Programs\Common\Microsoft\Visual C++ for Python\9.0\VC\Bin\cl.exe' failed with exit status 2
Any advice to make it work ?
Thx for your help

Thanks for the heads up!
The issue is that visual studios doesn't support C99 standard, which twofish depends on.
We might be able to switch to a pure-python implementation of twofish for windows:
http://www.bjrn.se/code/twofishpy.txt
Gpgme has a twofish implementation available -- that may be another possibility:
https://gnupg.org/documentation/manuals/gcrypt/Available-ciphers.html#Available-ciphers
However, pure-python seems the more realistic fix.  I don't know if anybody has the appetite for new bindings for this old algorithm.

More useful for you, try setting python to use MinGW which does support C99:
http://cython.readthedocs.io/en/latest/src/tutorial/appendix.html
Download the MinGW installer from http://www.mingw.org/wiki/HOWTO_Install_the_MinGW_GCC_Compiler_Suite. (As of this writing, the download link is a bit difficult to find; it’s under “About” in the menu on the left-hand side). You want the file entitled “Automated MinGW Installer” (currently version 5.1.4).

Run it and install MinGW. Only the basic package is strictly needed for Cython, although you might want to grab at least the C++ compiler as well.

You need to set up Windows’ “PATH” environment variable so that includes e.g. “c:\mingw\bin” (if you installed MinGW to “c:\mingw”). The following web-page describes the procedure in Windows XP (the Vista procedure is similar): http://support.microsoft.com/kb/310519

Finally, tell Python to use MinGW as the default compiler (otherwise it will try for Visual C). If Python is installed to “c:\Python27”, create a file named “c:\Python27\Lib\distutils\distutils.cfg” containing:

[build]
compiler = mingw32

@kurtbrose : Thx for your answer ;)
I indeed agree that the pure-python implementation seems really interesting ^^ ... I'll try to have a look at it.
But ATM, MinGW is indeed a good backup solution - I may install it on my windows VM :)
Cheers


Fix bash warts

This should make it pass our shellcheck test on travis.
Also simplifies the helm delete command
Also removes a extra inventory var that was created but never used

@evrardjp done!

It looks good with my eyes, but I haven't tested the code.
Did you have a go at it, to see if it breaks the deployment? :)

It looks good with my eyes, but I haven't tested the code.
Did you have a go at it, to see if it breaks the deployment? :)

currently running it to see if I broke everything!

Updated with a small fix for the extra vars to be passed correctly to ansible-playbook even when they are empty
Still testing the rest

Tested and it seems correct.

Should be better now!

I am still testing this one out :)

This kinda behaves differently than what we have right now, so I will give it another go in my next rebuild.

@Itxaka needs a rebase

@Itxaka also, please rather fix upstream scripts of those we are using (and let's focus on getting the downstream ones away)

I believe this should be reworked to just check our downstream files as there is no bash checks upstream and we will get rid of those hopefully soon and just use the upstream ones directly

I believe this should be reworked to just check our downstream files as there is no bash checks upstream and we will get rid of those hopefully soon and just use the upstream ones directly

Agreed

will reopen a new one when time permits it :D


Code enhancement in Teams Page

Path: community_app_boilerplate/lib/ui/screens/Teams.dart
We have stored the Teams Images in assets. We could use free storage services and host it on a server and instead of AssetImage we could use it CacheNetworkImage
It would reduce app size and also since it is a cache network Image so the image will be loaded only when the image is changed, other times the widget would directly take the image from the cache itself.
Also wherever you have used Network Image. I would like to use the CacheNetworkImage widget.
If you allow me, I would like to do these changes.

Sure @elaishane, should I assign it to you?

i can take this. you can upload images on free storage and give link?

Yes

@elaishane You are saying yes to work on this issue?

Yes

@himanshusharma89 I would be needing a storage location, it their image links.

@elaishane Do you have a platform in mind for the deployment of the images?

I think github would do.

Okay @elaishane, so I guess all the assets are already uploaded you can work on it right away check out the assets folder.

Sure. Thank you

Hey @elaishane, any update? and should I assign it to you?

I am working on it. It would be completed by tomorrow evening

Thanks for the update @elaishane. Assigning it to you.

Hey, @himanshusharma89 I will be needing a link for the images. Like Twitter, Facebook, GitHub but separate repo. A link that would not change even with new changes

Hi @elaishane, what about these links:
https://img.icons8.com/fluent/50/000000/github.png
https://img.icons8.com/color/48/000000/twitter.png
https://img.icons8.com/color/48/000000/linkedin.png
You can directly use them in NetworkImage

Ok

Hey, can I work on this issue as no one is on it already?

Hey, can I work on this issue as no one is on it already?

Sure @Saransh-cpp

Sure @Saransh-cpp

Should I create a new branch? Also after adding cached_network_image dependency in the pubspec.yaml file and running the code in my system, it automatically edited .idea/libraries/Dart_Packages.xml file, should I add this in git or should I push changes without adding this edited file, keeping the old file? Please guide

Should I create a new branch? Also after adding cached_network_image dependency in the pubspec.yaml file and running the code in my system, it automatically edited .idea/libraries/Dart_Packages.xml file, should I add this in git or should I push changes without adding this edited file, keeping the old file? Please guide

@Saransh-cpp Yeah, create a new branch. And the changes you mentioned are already ignored using .gitignore. You can start working on the issue.

@Saransh-cpp Yeah, create a new branch. And the changes you mentioned are already ignored using .gitignore. You can start working on the issue.


Segmentations panel is not being updated after selecting another series

Bug Report
If user opens the Segmentations panel and selects another series, the panel is not being updated. The user has to perform a second action to see the updated panel.

Steps to Reproduce:

Go to study: https://viewer-dev.ohif.org/viewer/1.2.8<IP_ADDRESS>80043.2.1125.1.99615878007501689887866317012655114
Open the Segmentations panel and verify that the panel has no segments
Select 1st thumbnail in the Series list (it has 1 segment)
Verify that the Segmentations panel was not updated and still empty
Click on Measurements button
Click again on Segmentations button
Verity that the Segmentations panel was updated

Expected Result:

When user selects a different Series, the Segmentations panel should be updated to display the correct information related to that series.

Fixed here: https://github.com/OHIF/Viewers/pull/1685.


JSON payload helpers don't accept top level arrays

The jsonPayload NSURLRequest extensions have type NSDictionary, but top level arrays are valid JSON too.
I'd love to use Argo's JSON type, but there are two problems:

Can't go from JSON to something accepted by NSJSONSerialization.dataWithJSONObject
Can't provide override var jsonPayload for NSMutableURLRequest, because override var only works in extensions if the return type is Objective C compatible.

We could use a custom JSONPayload, and define it only on NSMutableURLRequest, but this seems sub optimal...

@sidraval what's your current workaround for this?

https://github.com/thoughtbot/Swish/tree/sr-top-level-arrays

Damian and I were working on this, and came up with: https://github.com/thoughtbot/Swish/commit/a56452b52d84d984f5a6b207be1ce2b24ada5b41
While AnyObject? isn't ideal, our thoughts are that once the end user sets .jsonPayload = ..., they probably won't have a reason to fetch/manipulate it afterward.
Another solution we considered was a SwishRequest, that contained both the NSMutableURLRequest and a JSONPayload enum that had cases for arrays and dictionaries. This added a lot of overhead in code in exchange for no gain, as far as we could tell.
Thoughts? cc @thoughtbot/ios @paulyoung

@sidraval can you open a PR so we can discuss there?

:+1: https://github.com/thoughtbot/Swish/pull/59

Closing, as #59 is merged.


kubernetes.client.read-idle-timeout setting does not seem to be taken into account by KubernetesConfigMapWatcher

Steps to Reproduce

Observe that KubernetesConfigMapWatcher installed stream always times out after exactly 5 minutes without event:io.reactivex.exceptions.UndeliverableException: The exception could not be delivered to the consumer because it has already canceled/disposed the flow or the exception has nowhere to go to begin with. Further reading: https://github.com/ReactiveX/RxJava/wiki/What's-different-in-2.0#error-handling | io.micronaut.http.client.exceptions.ReadTimeoutException: Read Timeout

Try setting kubernetes.client.read-idle-timeout to 0s or a negative duration, in either bootstrap.yml or application.yml
Redeploy.

Expected Behaviour
The KubernetesConfigMapWatcher installed stream doesn't time out.
Actual Behaviour
The KubernetesConfigMapWatcher installed stream still times out after 5 minutes without event.
Are Configuration Properties for KubernetesConfiguration applicable to the KubernetesClient used by KubernetesConfigMapWatcher?
Environment Information

Micronaut Version: 2.5.3

@rdesgroppes I'm closing this issue b/c since the v3 we use official k8s client and the configuration of the underlying OkHttpClient is configurable using bean listener:
@Singleton
public class ApiClientListener implements BeanCreatedEventListener<ApiClient> {

    @Override
    public ApiClient onCreated(BeanCreatedEvent<ApiClient> event) {
        ApiClient apiClient = event.getBean();
        OkHttpClient okHttpClient = apiClient.getHttpClient().newBuilder()
                .readTimeout(5345, TimeUnit.MILLISECONDS)
                .build();
        apiClient.setHttpClient(okHttpClient);
        return apiClient;
    }
}


Typo on technical specs

https://getmonero.org/resources/technical-specs/
then --> than

+resolved


Hstring from implementation for OsStr

Can HString have a from implementation that converts OsStr to HString?

Duplicate of #760

Thanks for the feedback - that is something I'd like to support. I'll track it as part of #760


Remove dao code related to letters

User Story - Business Need

[ ] Ticket is understood, and QA has been contacted (if the ticket has a QA label).

User Story(ies)
As a Notify developer,
I want to remove unused code and other technical debt
So that the code base is easier to maintain.
Additional Info and Resources
Recent tickets removed routes and route tests associated with sending letters and PDFs.  This ticket is to remove dao code that is now unused and to remove tests of that code.
Engineering Checklist
The following PRs delete a lot of code.  Audit the deleted code for the removal of import statements for dao functions.  If a dao function isn't used elsewhere, delete the dao function, and delete or change affected dao tests.

[ ] 1457
[ ] 1458
[ ] 1460

Acceptance Criteria

[ ] The code has been deleted, and all unit tests pass.

QA Considerations
For QA to populate. Leave blank if QA is not applicable on this ticket.

[ ] double-check responses for any removed properties and do the needful with the regression suite
[ ] run regression in perf and ensure things work as expected

Out of Scope
This does not fully remove letter-related code.  For example, the Notification model has columns like "postage".

Hey team! Please add your planning poker estimate with Zenhub @babyjunior56 @cris-oddball @EvanParish @k-macmillan @kalbfled @ldraney @nikolai-efimov

We discussed in Refinement that this should include a migration to remove unused letter fields.  That's currently "out of scope."

Closing/no longer relevant.


Modal Not Working in Laravel 10

Hi tried the free modal component with my new laravel 10 project and it didn't work, so i bought the pro license thinking the pro might be up-to-date with new laravel version, but still i can't get the model to work with my project.
<button wire:click="$emit('modal.open', 'hello-world')">Open Modal</button>
Also use this action:

Projects
         <x-buttons.primary class="p-2 h-8" wireClick="$emit('modal.open', 'app.admin.modals.project.create-project')" text="Create" />

I have the modal component already for both the HelloWord and CreateProject.

Hello. Could you share the exact error it is throwing?.

Working now. I had to reinstall the right versions of tailwindcss and alphine.js.
Initially, I installed the latest livewire version which comes with alphine included.
But for the Pro version, still not able to get it to work. Modal shows at the bottom of the page not as a modal

Noted 👍

@ambrosethebuild if the modal shows at the bottom, the CSS is often omitted, ensure you import the required CSS as shown in the demo repository.
Feel free to contact<EMAIL_ADDRESS>if you still have a problem with getting the pro version to work.


Strange behavior when scrolling mouse to view file preview content in rio terminal

What system are you running Yazi on?
macOS
What terminal are you running Yazi in?
Yazi 0.2.5 (Homebrew 2024-04-23) / rioterm 0.1.0
Did you try the latest code to see if this problem got fixed?
Tried, but the problem still
yazi --debug output

Yazi
    Version: 0.2.5 (Homebrew 2024-04-23)
    OS: macos-aarch64 (unix)
    Debug: false

Emulator
    Emulator.via_env: ("rio", "rio")
    Emulator.via_csi: Ok(Unknown([]))
    Emulator.detect: Iterm2

Adaptor
    Adaptor.matches: Iterm2

Desktop
    XDG_SESSION_TYPE: None
    WAYLAND_DISPLAY: None
    DISPLAY: None

SSH
    shared.in_ssh_connection: false

WSL
    /proc/sys/fs/binfmt_misc/WSLInterop: false

Variables
    SHELL: Some("/bin/zsh")
    EDITOR: Some("nvim")
    ZELLIJ_SESSION_NAME: None
    YAZI_FILE_ONE: None
    YAZI_CONFIG_HOME: None

file(1)
    Version: Ok(Output { status: ExitStatus(unix_wait_status(0)), stdout: "file-5.41\nmagic file from /usr/share/file/magic\n", stderr: "" })

Text Opener
    default: Some(Opener { run: "${EDITOR:=vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })
    block: Some(Opener { run: "${EDITOR:=vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })

tmux
    TMUX: false

Ueberzug++
    Version: Err(Os { code: 2, kind: NotFound, message: "No such file or directory" })

--------------------------------------------------
When reporting a bug, please also upload the `yazi.log` log file - only upload the most recent content by time.
You can find it in the "/Users/cavivie/.local/state/yazi" directory.

Describe the bug
https://github.com/sxyazi/yazi/assets/103884097/12dbb149-326d-4edc-9fc0-45973387e88a
This was recorded in Rio Terminal.
Expected Behavior
https://github.com/sxyazi/yazi/assets/103884097/723bc833-7f71-428d-a043-64a21bb4401f
This was recorded in iTerm2 Terminal.
To Reproduce
In rio terminal, the mouse scroll cannot be fixed when viewing the file preview content. It always scrolls the parent directory list where the file is located, rather than the file content itself. However, the behavior in iTerm2 seems to be correct. I didn't find any issues related to this in rio issue.
Configuration
No response
Anything else?
I am so sorry to create this issue If it is a known yazi or rio issue.

I currently install via brew install yazi --HEAD, I think I need to build the latest yazi from source?

Thanks. I verified the latest source code and found that the two behave the same. I will close this issue.


TableDemo height

Hello,
TableDemo windows is growing whenever a mouse is moved. Screen recording is from web demo. Windows desktop app has the same problem.
https://user-images.githubusercontent.com/104624770/165926374-d175e9d6-73b8-47b0-a6ae-2d601614f265.mp4
.

Additional info:
I have 2 monitors and this is an issue only on HD monitor (4096 x 2160)

I can reproduce this. On my notebook without external display attached, the guessed window scale factor is 1.75 and the table / strip demos are growing when the mouse is moved. However, the changes of https://github.com/emilk/egui/pull/2099 solved that issue too.

It also happens for the Strip Demo.

I think it is a issue of ui.avalible_rect(). Seems that it doesn't take into account the margins of the frame and reports wrong size. Then the widget will try to expand larger, and in the next frame the window will try to fit it, then the window will grow a bit and margins are added to this new size.

Plus: a bottom_up layout will also cause this issue.


Add DragonFly support for KEEPINTVL/KEEPCNT

Those are supported on DragonFly, build fails without them.
Not sure what to do with test, mine showing:
---- only_v6 stdout ----
thread 'only_v6' panicked at 'assertion failed: `(left != right)`
  left: `true`,
 right: `true`: initial value and argument are the same', tests/socket.rs:1094:1
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- tcp_keepalive stdout ----
thread 'tcp_keepalive' panicked at 'assertion failed: `(left == right)`
  left: `75s`,
 right: `30s`', tests/socket.rs:727:5

Not sure what to do with test, mine showing:
---- only_v6 stdout ----
thread 'only_v6' panicked at 'assertion failed: `(left != right)`
  left: `true`,
 right: `true`: initial value and argument are the same', tests/socket.rs:1094:1
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- tcp_keepalive stdout ----
thread 'tcp_keepalive' panicked at 'assertion failed: `(left == right)`
  left: `75s`,
 right: `30s`', tests/socket.rs:727:5

For the only_v6 can you add DragonFlyBSD to the cfg statements here: https://github.com/rust-lang/socket2/blob/9d3e39c696e76b26af5faf0dffb7dd5b53dc0ebc/tests/socket.rs#L1093-L1097. It seems that by default DragonFly's IPv6 sockets are IPv6 only already, just like on FreeBSD and Windows. I would do it myself, but I don't have DragonFlyBSD to test it on.
I think the tcp_keepalive test should be fixed if add DragonFly to the cfg list in TcpKeepalive::with_interval.

We v6 still doesn't work for me:
---- only_v6 stdout ----
thread 'only_v6' panicked at 'failed to set option: Os { code: 45, kind: Other, message: "Operation not supported" }', tests/socket.rs:1111:1
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

We v6 still doesn't work for me:
---- only_v6 stdout ----
thread 'only_v6' panicked at 'failed to set option: Os { code: 45, kind: Other, message: "Operation not supported" }', tests/socket.rs:1111:1
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

That odd... does DragonFlyBSD not support setting IPv6 only?
For the sake of merging this pr you can remove DragonFlyBSD from the tests.

Commented out DragonflyBSD in last test leaving comment that it's actually not supported.

Merged as b96fa3bc7273a6b8cd5bdae9cb27fa4fc7458b62. Thanks @kworr.


Bug Report: 500 Internal Server Error with Spring Cloud Gateway

Description
I am experiencing intermittent 500 Internal Server Error responses when making requests to my Spring Cloud Gateway, which routes to a stack of microservices registered with a Spring Eureka server. The issue occurs "almost all the time," but occasionally requests are successful for one or two services.
Environment
Framework: Spring Boot 3.3.3
API Gateway: Spring Cloud Gateway 4.1.5
Service Discovery: Spring Eureka
Deployment: Docker Swarm
Network: All components are on the same Docker overlay network
Steps to Reproduce
Deploy microservices, Spring Cloud Gateway, and Eureka server using Docker Swarm.
Register all microservices with the Eureka server.
Send a request through the Spring Cloud Gateway to one of the registered services.
Observe the response; it often results in a 500 Internal Server Error.
Expected Behavior
The Spring Cloud Gateway should successfully route requests to the registered microservices without consistently returning a 500 Internal Server Error.
Actual Behavior
Most requests result in a 500 Internal Server Error.
Occasionally, a request succeeds for one or two services.
application.yml
cloud:
    loadbalancer:
      cache:
        ttl: 60s
    gateway:
      metrics:
        enabled: true
      discovery:
        locator:
          enabled: true
          lower-case-service-id: true
          predicates:
            - name: Path
              args:
                pattern: "'/'+serviceId+'/**'"
          filters:
            - name: RewritePath
              args:
                regexp: "'/' + serviceId + '/?(?<remaining>.*)'"
                replacement: "'/'+serviceId+'/${remaining}'"

service has context-path who is the id of the service
Stack:
xchange_gateway.1.givw3dpqetj7@kevin    | 2024-09-04T22:48:03.440Z TRACE 1 --- [gateway] [     parallel-8] o.s.c.g.h.p.PathRoutePredicateFactory    : Pattern "/goods/**" matches against value "/goods/graphql"
xchange_gateway.1.givw3dpqetj7@kevin    | 2024-09-04T22:48:03.440Z DEBUG 1 --- [gateway] [     parallel-8] o.s.c.g.h.RoutePredicateHandlerMapping   : Route matched: ReactiveCompositeDiscoveryClient_GOODS
xchange_gateway.1.givw3dpqetj7@kevin    | 2024-09-04T22:48:03.440Z DEBUG 1 --- [gateway] [     parallel-8] o.s.c.g.h.RoutePredicateHandlerMapping   : Mapping [Exchange: POST http://<IP_ADDRESS>:8044/goods/graphql] to Route{id='ReactiveCompositeDiscoveryClient_GOODS', uri=lb://GOODS, order=0, predicate=Paths: [/goods/**], match trailing slash: true, gatewayFilters=[[[RewritePath /goods/?(?<remaining>.*) = '/goods/${remaining}'], order = 1]], metadata={management.port=8083}}
xchange_gateway.1.givw3dpqetj7@kevin    | 2024-09-04T22:48:03.440Z DEBUG 1 --- [gateway] [     parallel-8] o.s.c.g.h.RoutePredicateHandlerMapping   : [90e0f925-4] Mapped to org.springframework.cloud.gateway.handler.FilteringWebHandler@76075815
xchange_gateway.1.givw3dpqetj7@kevin    | 2024-09-04T22:48:03.440Z DEBUG 1 --- [gateway] [     parallel-8] o.s.c.g.handler.FilteringWebHandler      : Sorted gatewayFilterFactories: [[GatewayFilterAdapter{delegate=org.springframework.cloud.gateway.filter.RemoveCachedBodyFilter@bd28c86}, order = -2147483648], [GatewayFilterAdapter{delegate=org.springframework.cloud.gateway.filter.AdaptCachedBodyGlobalFilter@128a188a}, order = -2147482648], [GatewayFilterAdapter{delegate=org.springframework.cloud.gateway.filter.NettyWriteResponseFilter@1786150f}, order = -1], [GatewayFilterAdapter{delegate=org.springframework.cloud.gateway.filter.ForwardPathFilter@14499b52}, order = 0], [GatewayFilterAdapter{delegate=org.springframework.cloud.gateway.filter.GatewayMetricsFilter@3164d5aa}, order = 0], [[RewritePath /goods/?(?<remaining>.*) = '/goods/${remaining}'], order = 1], [GatewayFilterAdapter{delegate=org.springframework.cloud.gateway.filter.RouteToRequestUrlFilter@3ef089ce}, order = 10000], [GatewayFilterAdapter{delegate=org.springframework.cloud.gateway.filter.ReactiveLoadBalancerClientFilter@14d1032a}, order = 10150], [GatewayFilterAdapter{delegate=org.springframework.cloud.gateway.filter.LoadBalancerServiceInstanceCookieFilter@414e8b31}, order = 10151], [GatewayFilterAdapter{delegate=org.springframework.cloud.gateway.filter.WebsocketRoutingFilter@be083e1}, order =<PHONE_NUMBER>], [GatewayFilterAdapter{delegate=org.springframework.cloud.gateway.filter.NettyRoutingFilter@b90c5a5}, order =<PHONE_NUMBER>], [GatewayFilterAdapter{delegate=org.springframework.cloud.gateway.filter.ForwardRoutingFilter@3340ce58}, order =<PHONE_NUMBER>]]
xchange_gateway.1.givw3dpqetj7@kevin    | 2024-09-04T22:48:03.440Z TRACE 1 --- [gateway] [     parallel-8] o.s.c.g.filter.RouteToRequestUrlFilter   : RouteToRequestUrlFilter start
xchange_gateway.1.givw3dpqetj7@kevin    | 2024-09-04T22:48:03.441Z TRACE 1 --- [gateway] [     parallel-8] s.c.g.f.ReactiveLoadBalancerClientFilter : ReactiveLoadBalancerClientFilter url before: lb://GOODS/goods/graphql
xchange_gateway.1.givw3dpqetj7@kevin    | 2024-09-04T22:48:03.444Z TRACE 1 --- [gateway] [     parallel-8] s.c.g.f.ReactiveLoadBalancerClientFilter : LoadBalancerClientFilter url chosen: http://<IP_ADDRESS>:8083/goods/graphql
xchange_gateway.1.givw3dpqetj7@kevin    | 2024-09-04T22:48:03.445Z DEBUG 1 --- [gateway] [     parallel-8] g.f.h.o.ObservedRequestHttpHeadersFilter : Will instrument the HTTP request headers [Authorization:"Bearer authtoken", Content-Type:"application/json", User-Agent:"PostmanRuntime/7.29.4", Accept:"*/*", Cache-Control:"no-cache", Postman-Token:"e3df98e3-794c-4262-a942-244eaa87184a", Host:"<IP_ADDRESS>:8044", Accept-Encoding:"gzip, deflate, br", Content-Length:"1728", Forwarded:"proto=http;host="<IP_ADDRESS>:8044";for="<IP_ADDRESS>:46654"", X-Forwarded-For:"<IP_ADDRESS>", X-Forwarded-Proto:"http", X-Forwarded-Port:"8044", X-Forwarded-Host:"<IP_ADDRESS>:8044"]
xchange_gateway.1.givw3dpqetj7@kevin    | 2024-09-04T22:48:03.445Z DEBUG 1 --- [gateway] [     parallel-8] g.f.h.o.ObservedRequestHttpHeadersFilter : Client observation  {name=http.client.requests(null), error=null, context=name='http.client.requests', contextualName='null', error='null', lowCardinalityKeyValues=[http.method='POST', http.status_code='UNKNOWN', spring.cloud.gateway.route.id='ReactiveCompositeDiscoveryClient_GOODS', spring.cloud.gateway.route.uri='lb://GOODS'], highCardinalityKeyValues=[http.uri='http://<IP_ADDRESS>:8044/goods/graphql'], map=[class io.micrometer.core.instrument.Timer$Sample='io.micrometer.core.instrument.Timer$Sample@35962551', class io.micrometer.core.instrument.LongTaskTimer$Sample='SampleImpl{duration(seconds)=2.25614E-4, duration(nanos)=225614.0, startTimeNanos=32145399770171}'], parentObservation=org.springframework.security.web.server.ObservationWebFilterChainDecorator$PhasedObservation@799125b} created for the request. New headers are [Authorization:"Bearer authtoken", Content-Type:"application/json", User-Agent:"PostmanRuntime/7.29.4", Accept:"*/*", Cache-Control:"no-cache", Postman-Token:"e3df98e3-794c-4262-a942-244eaa87184a", Host:"<IP_ADDRESS>:8044", Accept-Encoding:"gzip, deflate, br", Content-Length:"1728", Forwarded:"proto=http;host="<IP_ADDRESS>:8044";for="<IP_ADDRESS>:46654"", X-Forwarded-For:"<IP_ADDRESS>", X-Forwarded-Proto:"http", X-Forwarded-Port:"8044", X-Forwarded-Host:"<IP_ADDRESS>:8044"]
xchange_gateway.1.givw3dpqetj7@kevin    | 2024-09-04T22:48:33.478Z DEBUG 1 --- [gateway] [or-http-epoll-1] .o.ObservationClosingWebExceptionHandler : An exception occurred and observation was not previously stopped, will stop it. The exception was [io.netty.channel.ConnectTimeoutException: connection timed out after 30000 ms: /<IP_ADDRESS>:8083]
xchange_gateway.1.givw3dpqetj7@kevin    | 2024-09-04T22:48:33.479Z ERROR 1 --- [gateway] [or-http-epoll-1] a.w.r.e.AbstractErrorWebExceptionHandler : [90e0f925-4]  500 Server Error for HTTP POST "/goods/graphql"
xchange_gateway.1.givw3dpqetj7@kevin    |
xchange_gateway.1.givw3dpqetj7@kevin    | io.netty.channel.ConnectTimeoutException: connection timed out after 30000 ms: /<IP_ADDRESS>:8083
xchange_gateway.1.givw3dpqetj7@kevin    | 	at io.netty.channel.epoll.AbstractEpollChannel$AbstractEpollUnsafe$2.run(AbstractEpollChannel.java:615) ~[netty-transport-classes-epoll-4.1.112.Final.jar:4.1.112.Final]

Hi @kev205 ,
I'm facing the same problem on my side and I haven't found any clue how to fix it.
Environment :

Spring Boot 3.3
Spring Cloud Gateway 4.1.5
Docker compose 2.27

Details :
I have 4 microservices, a gateway and a discovery server.
All requests are being forwarded except for one microservice. Their configuration files are pretty much the same.
Everything works fine locally, but as soon as docker containers are deployed using docker compose, I get the ConnectTimeoutException. I also check my network bridges, they are all correct.
I haven't changed my gateway configuration in a long time and before that everything was fine.
Do you have any ideas? How did you solve your problem?

Thanks for your time and your quick answer 🙏
Here the application.yml of my Spring Cloud Gateway server :
server:
  forward-headers-strategy: framework

spring:
  application:
    name: api-gateway-server

  output:
    ansi:
      enabled: ALWAYS

  cloud:
    gateway:
      globalcors:
        cors-configurations:
          '[/**]':
            allowedOriginPatterns: "http://localhost:3000"
            allowedMethods: "*"
            allowedHeaders: "*"
            allowCredentials: true
            exposedHeaders: "Content-Disposition"
      discovery:
        locator:
          enabled: true
      routes:
        - id: authorization-service
          uri: lb://api-authorization-service
          predicates:
            - Path=/api/roles/**, /api/auth/users/**, /api/tenants/**, /api/users/**
          filters:
            - AuthorizationFilter

        - id: session-service
          uri: lb://api-session-service
          predicates:
            - Path=/api/setups/**, /api/engine/**, /api/projects/**, /api/sessions/**, /api/certification/**
          filters:
            - AuthorizationFilter

        - id: report-service
          uri: lb://api-report-service
          predicates:
            - Path=/api/notifications/**, /api/reports/**
          filters:
            - AuthorizationFilter
            - DedupeResponseHeader=Access-Control-Expose-Headers Access-Control-Allow-Credentials Access-Control-Allow-Origin RETAIN_UNIQUE

        - id: suite-service
          uri: lb://api-suite-service
          predicates:
            - Path=/api/engines/**, /api/suites/**
          filters:
            - AuthorizationFilter

        #################
        # Documentation #
        #################

        - id: openapi-authorization
          uri: lb://api-authorization-service
          predicates:
            - Path=/v3/api-docs/authorization
          filters:
            - RewritePath=/v3/api-docs/authorization, /v3/api-docs

        - id: openapi-session
          uri: lb://api-session-service
          predicates:
            - Path=/v3/api-docs/session
          filters:
            - RewritePath=/v3/api-docs/session, /v3/api-docs

        - id: openapi-report
          uri: lb://api-report-service
          predicates:
            - Path=/v3/api-docs/report
          filters:
            - RewritePath=/v3/api-docs/report, /v3/api-docs

        - id: openapi-suite
          uri: lb://api-suite-service
          predicates:
            - Path=/v3/api-docs/suite
          filters:
            - RewritePath=/v3/api-docs/suite, /v3/api-docs

springdoc:
  swagger-ui:
    urls:
      - name: api-authorization-service
        url: /v3/api-docs/authorization
      - name: api-session-service
        url: /v3/api-docs/session
      - name: api-report-service
        url: /v3/api-docs/report
      - name: api-suite-service
        url: /v3/api-docs/suite
    path: /openapi.html
  cache:
    disabled: true

eureka:
  instance:
    prefer-ip-address: true
    # Indicates the interval of heartbeats that the client sends to the server
    # The default value is 30 seconds
    lease-renewal-interval-in-seconds: 10
  client:
    service-url:
      defaultZone: http://localhost:8761/eureka/
    # Indicates how often the client should fetch the registry from the Eureka server
    # It determines how frequently the service discovers other registered services in Eureka.
    # The default value is 30 seconds
    registry-fetch-interval-seconds: 10

logging:
  include-application-name: false
  pattern:
    correlation: "[${spring.application.name:},%X{trace_id:-},%X{span_id:-}] "

######################
#   Profile - Test   #
######################
---
spring:
  config:
    activate:
      on-profile: test
  main:
    banner-mode: off

eureka:
  client:
    enabled: false

logging:
  level:
    root: off

########################
#   Profile - Docker   #
########################
---
spring:
  config:
    activate:
      on-profile: docker

eureka:
  client:
    service-url:
      defaultZone: http://api-discovery-server:8761/eureka/

Hello @RomRom1 , try removing prefer-ip-address: true or setting it to false for your discovery clients. Also ensure that all your services are consistently discovered by the eureka server

i try to enable trace logs on your gateway please. So we can see how it routes incoming requests


Ferrari 458 spyder wheel issue

Hello there. I have an issue with my Ferrari 458 Spyder racing wheel. I have downloaded this file, and whenever I connect it my computer it thinks whenever I turn my wheel I am pressing Left Trigger. Does anybody know how to fix this?
 Leviticus 20:15

Hello there. I have an issue with my Ferrari 458 Spyder racing wheel. I have downloaded this file, and whenever I connect it my computer it thinks whenever I turn my wheel I am pressing Left Trigger. Does anybody know how to fix this?


Moving TS file causes import path formatting to change for unrelated files

Issue Type: Bug
Found while verifying https://github.com/Microsoft/vscode/issues/50707

Add "typescript.updateImportsOnFileMove.enabled": "always" to settings
In VSCode repo, drag a file referenced by other files into a different folder. issueReporterModel.ts is a nice example because only two other files reference it

The path to the file is correctly updated in files referencing it, but paths in other files are also reformatted (relative paths like './list' change to be from workspace root 'vs/base/browser/ui/list/list'), causing a large number of editors to open:

VS Code version: Code - Insiders 1.25.0-insider (f2fa6bea46bde9856e6a593e4d4c5af2b303ac66, 2018-06-27T05:11:02.122Z)
OS version: Darwin x64 17.4.0

Same issue here, it looks like it opens almost every type definition file.  It doesn't seem like there are actually any changes made to the files, but when I try to close one it will prompt me to save the changes or they will be lost.
VS Code Version: Version 1.24.1 (1.24.1)
OS: MacOS High Sierra version 10.13.5

I have exactly the same issue. I didn't even move a .ts file - I just copied one. Now I have 670 open editors :open_mouth: They don't show up in Stable when opening the same project. I even tried re-installing Insiders, but my 670 open editors are still there.
Is there a way to close the editors without having to click "Don't Save" 670 times? :slightly_smiling_face:
VS: 1.25.0-insider
OS: Linux Ubuntu 16.04

Duplicate of https://github.com/Microsoft/TypeScript/issues/24914
Try upgrading your workspace to use typescript@next by following these instructions


GET /supports?{filters for proposal and participant}

Needed for oliverbarnes/participate/issues/86, to find out if a proposal is already supported when clicking on the support button.
Filtering is the way I found to make this work between ember-jsonapi-resources and jsonapi-resources, in the future perhaps a clearer endpoint could be implemented.
Also, currently the related resources links are coming through as uris to relationships, but I'd like to keep the endpoints as flat (top level) as possible to keep things simple. Opening a separate issue to customize the base resource to return top level endpoints.

merged it, but please feel free to add comments and suggestions after the fact


Update vesting wallet contracts and tests

Fixes #13
Changes proposed in this PR:

Updated vesting wallet contracts and tests.

Can you confirm: does the cli still work, under these changes?


Close the milestone on release

As a developer, I want to release the project automatically to reduce errors and speed up the release process
Acceptance criteria:

The milestone should be closed once the release has been created

Should be OK


Add dark theme support and add copy data button

also made buttons prettier

@pajlada Seems the site wasn't updated yet, just a ping so it doesn't get forgotten

@fourtf :scroll: :point_up:


Not working when I put the component inside a div with overflow: scroll !

When the entire component is rendered on the browser screen the Scrollchor with #id works, but when the same component is kept in a div with overflow: scroll property in parent div it doesn't work.
I had the same issue with react-scroll Link.
They mentioned that react-scroll won't work inside a div.
Is this the same issue with react-scrollchor?

Could you share a repo with your issue??

I´m affraid this will not work cause what is scrolled is the top document container not a individual component or div. This is implemented on top of document.documentElement or document.body scrollTop, a browser use case.
See src: https://github.com/bySabi/react-scrollchor/blob/master/src/utils.js#L3

Ohk I get it.
So in my case what would you suggest?
Any other libs, within your knowledge, that would satisfy my purpose.

Sorry! I don´t know any React package that do that out of box. Never need something like this.

closed due inactivity


HSEARCH-2110 Update the Elasticsearch documentation prior to release

NOTE: this takes into account the fact that the artifact and the package has been renamed to remove the "backend" part.

merged, thanks!


Explain why we use trim when reading test file contents

Adding some explanatory comments around the trim in the content of the files when testing, as suggested here

Thank you! 👍  :)


Make syncPurchases function to actually returning a promise

This small change will make capacitor to actually resolving or rejecting the promise of syncPurchases. Without this change, syncPurchases function will always return -1 no matter if it is successful or not.

@tonidero hey can you help to review this PR?

Hi @jarvisluong, many thanks for the contribution!
I was taking a look, this would indeed make it so it waits until the native part is called but we also need to fix the android native code to only call the completion of the call, once sync purchases is complete: https://github.com/RevenueCat/purchases-capacitor/blob/a8bb5ff42ca2fbeb33d439dcca0c379163b4d55f/android/src/main/java/com/revenuecat/purchases/capacitor/PurchasesPlugin.kt#L316. That should be something like: syncPurchasesCommon(getOnResult(call))
In iOS, we should already be holding until the sync purchases call is complete: https://github.com/RevenueCat/purchases-capacitor/blob/a8bb5ff42ca2fbeb33d439dcca0c379163b4d55f/ios/Plugin/PurchasesPlugin.swift#L260
Thanks again for this and let me know if you have questions or I can help!


Make minimum vso node api version required to 0.7

vso-node-api has not been published to npmjs.org yet.  I'm taking other changes to it first.

Merging this as its required for publishing a new preview version


Race in userdata.json has problems being read

userdata.json:
{
  "user" : {
    "name" : "Neo",
    "phone" : "88889999",
    "email"<EMAIL_ADDRESS>    "address" : "Lower Downtown, Capital City, USA",
    "race" : "",
    "major" : "",
...

Log:
Mar 30, 2023 11:37:23 AM seedu.address.storage.user.JsonUserDataStorage readData
INFO: Illegal values found in data/userdata.json: Race should only contain alphabets and spaces
Mar 30, 2023 11:37:23 AM seedu.address.MainApp initModelManager
WARNING: Data file not in the correct format. Will be starting with blank UserData

Replacing "" with valid inputs such as "chinese" doesn't work either.

Closed as deleting the two jsons fixed it


Add latency output for debugging networking

Add a latency output to show the quality of the connection, maybe as a debug display so it can be easily removed for final builds.

Resolved by #75


cleaning and reducing repo

cleaned out repository to limit it to the new scope
Here are questions and things left to do:

is the OSC folder still needed?
need to rerun the script to check that it works

scale and errors removed
renumber sampling.df
make sure heat maps are created

where are the metadata files?
how many images are removed after the scale bar and blobs are checked?
should the repo be renamed to "minnow_segmented_traits"?

@megbalk See below for my thoughts on your questions:

is the OSC folder still needed?

I don't think so. Users can just run this in their home directory. All data needed can be downloaded from our Dataverse instance.

where are the metadata files?

Which metadata files are you asking about? After running the workflow the bgnn-formatted drexel_metadata files are in the subdirectory segmentation/Metadata/ and the original drexel_metadata files are in the subdirectory segmentation/DrexelMetadata/.

how many images are removed after the scale bar and blobs are checked?

Perhaps you could code something into the script to record this info.

should the repo be renamed to "minnow_segmented_traits"?

That sounds good to me. The nice thing about renaming is GitHub will maintain the older repo names.

i meant the metadata files that come from Tulane: Image_Metadata_v1_20211206_151152.csv and Image_Quality_Metadata_v1_20211206_151204.csv. I don't see them in the folder "Files" anymore

i meant the metadata files that come from Tulane: Image_Metadata_v1_20211206_151152.csv and Image_Quality_Metadata_v1_20211206_151204.csv. I don't see them in the folder "Files" anymore

@megbalk These files are now stored in Dataverse:

https://datacommons.tdai.osu.edu/dataset.xhtml?persistentId=doi:10.5072/FK2/QOHJGD&version=DRAFT
https://datacommons.tdai.osu.edu/dataset.xhtml?persistentId=doi:10.5072/FK2/ZIFDTJ&version=DRAFT

These input files are now downloaded as part of the first rules of the workflow.
Here is the config that describes the DOIs from Dataverse and the filenames:
https://github.com/hdr-bgnn/Minnow_Segmented_Traits/blob/78e665fceb1236efbd5491ae94393dcdb5b30d08/config/config.yaml#L4-L7
Here are the two rules that download the input files:
https://github.com/hdr-bgnn/Minnow_Segmented_Traits/blob/78e665fceb1236efbd5491ae94393dcdb5b30d08/workflow/Snakefile#L41-L57

i need to fix the readme to reflect where things are

@johnbradley i fixed the read me! I have place holders for when I rerun the code and will fix then.

The input CSV format and files have dramatically changed. The data is now spread over multiple CSV files and many column names have changed. There are changes here that rename the fields and read data from the new CSV files, but I don't think the changes are complete.
I propose instead of reworking everything to fit the new format we could add a new first step that converts the new CSV input files into the old format. Then nothing else would need to change. Thoughts @megbalk?

@johnbradley yes, let's revert back. We can archive the images that are used in the study and for the segmentation model later.

To get started I'm going to create another branch based on the changes here. That way we can easily compare against these  changes to make sure we have everything we need.
I think commit modifying scripts https://github.com/hdr-bgnn/Minnow_Segmented_Traits/pull/30/commits/0f958035696abf0a7708ee33754095683701693a was the first that begin the transition to the new input format. I am going to create a new branch starting with commit https://github.com/hdr-bgnn/Minnow_Segmented_Traits/pull/30/commits/f9933879e5b504eb8b46fb4f7dfc319be85bad9b. You can browse the code at the point right now via https://github.com/hdr-bgnn/Minnow_Segmented_Traits/tree/f9933879e5b504eb8b46fb4f7dfc319be85bad9b.

I'm closing this PR since the work has been done via PR #33.


Implement video graph browser example

Connect with YouTube or Vimeo API, request related videos, display the network and allow infinite zoomability.

Notes about YouTube API:
Thumbnail images for videos
See https://developers.google.com/youtube/v3/docs/thumbnails
Search videos related to a video or keyword
Possible with relatedToVideoId property. See https://developers.google.com/youtube/v3/docs/search/list

Notes about Vimeo API because YouTube API authentication seemed more complex:

Get access token
Get a video
Get related videos

Other notes:

Ajax Request without jQuery

Vimeo's "get related videos" seems veeryyy poor... almost every video has the same set of related videos. Vimeo's own site does not seem to use it, the site only gives the list of videos from the same author.
Let us try DailyMotion API...
GET video
GET related videos
GET id, title, and thumbnail sprite
And no authentication required! How about CORS? Tested in dev toolbar, no CORS issues. Fk yeah!


cppSlint not working

cppSlint failing, with the following error:
autotests260arm64-1  | slint linuxkms backend: unrecognized renderer skia, falling back default
autotests260arm64-1  | Hello Torizon!
autotests260arm64-1  | Using Skia OpenGL renderer
autotests260arm64-1  | Rendering at 1920x1080
autotests260arm64-1  | thread '<unnamed>' panicked at api/cpp/lib.rs:64:6:
autotests260arm64-1  | called `Result::unwrap()` on an `Err` value: OtherError("Error presenting fb: permission denied")
autotests260arm64-1  | note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

That would be a bug in the docker file. Maybe it changed since my PR. What version/branch are you using?

Sorry, I meant docker compose file

Also, are there any other processes running on the system that may claim the display by chance?

The next branch. Also, this error happened on the release (Dockerfile).

Interestingly, I double checked it outside of the automated tests and it looks like it is working now:
Attaching to testslint-1
testslint-1  | slint linuxkms backend: unrecognized renderer skia, falling back default
testslint-1  | Hello Torizon!
testslint-1  | Using Skia OpenGL renderer
testslint-1  | Rendering at 1920x1080

Maybe it was just some bug on my display

Now it worked even on the automated tests :)

Yay :). Very much looking forward to the next release.


stopped working

When I save a file it should compile the minified version, but there is an error:
Name expected  @ main.js : 143

I'll look into this a bit further but can you verify this is an issue with the plugin and not your code? It could very well be an error from UglifyJS saying there was a problem minifying your js because of something.
Can you create a new js file, write something simple and then try to minify that?

Yep, it was a mistake in the code (,; at the end of a statement), thanks


Categories are missing in some custom apps when using global nav: home-to-next-seach/navigation

Do you have the latest versions of Spicetify, Spotify and Better Bloom?

[X] I'm using the latest versions of Spicetify, Spotify and Better Bloom

Is there already an issue for your problem?

[X] I have checked older issues, open and closed

Is your problem definitely caused by the theme?

[X] I have checked the app behavior without the theme and it is different

Environment & Computer Info
- Spotify version: <IP_ADDRESS>8.g0ea3f38b
- Installed from: Spotify site
- Spicetify version: 2.36.12
- Color scheme: dark
- Extensions:
- Custom apps: marketplace, new-releases, lyrics-plus, stats, library
- Better Bloom installation method: Spicetify Marketplace

Description

categories missing in some custom apps when using global nav: home-to-next-seach/navigation
but when using global nav: control, there are categories, but they are at the top
in stats custom apps, categories does not missing, but it's just invisble
and when I scrolling, there are categories

Screenshots
I'am gonna mark that as yellow
,

sample: stats custom app when using global nav: control
,

lyrics-plus when using  global nav: control
,

lyrics-plus when using global nav: home-to-next-seach/navigation
,

stats when using global nav: home-to-next-seach/navigation
,

and when scrolling

sorry for the inconvenience ! i will check out this custom app and fix it


Why is there no operator[] overloading ?

Hi,
I would like to test your bitset implementation but there is no operator[] overloading.
Is it normal ?
And what is the performance of your implementation over std::bitset ?
Thanks,

Hi, (I had comment forwarding turned off, so didn't notice this before).
bit_set<N> is a packed representation for std::set<int>. Set have bidirectional iterators, no random access iterators and also no operator[].
I don't have systematic performance tests available. That might change in the future, but no promises.


Rehydrate app state

discussion on https://app.asana.com/0/260082342996612/371491806721831/f

Looks good to me seems to work fine 👍🏼


restaurant recommendation feature

Pull Request Test Coverage Report for Build 640511679

17 of 17 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 91.424%

Totals

Change from base Build 640201964:
0.1%

Covered Lines:
1226

Relevant Lines:
1341

💛  - Coveralls


suggestion: "reveal solution" button in side bar before actually giving puzzle answers

I love this plugin for stuff like SOTE where the whole friggin library just SUCKS, but then doing DT2 recently without a guide at all was actually really satisfying. I'd like to suggest a config option that "pauses" the quest helper before giving you the answer to puzzles, just to give you a moment to try to figure it out for yourself.
While it's always possible to manually enable/disable the plugin or even just the quest hint settings, this would streamline the process a good bit and I know there's gotta be at least two other users who would appreciate that lmao

Currently got a PR up to add a filter option for puzzles https://github.com/Zoinkwiz/quest-helper/pull/1444.


[backport] Correct Embed ID documentation

Backport of #7484

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 9729cae08a9e07d17696dfb74a20bcf55b9fa8a8, target branch v6) failed with status FAILURE.

LGTM


Docs: Flux Store snippet is outdated

FluxUtils to the rescue.

That counter example is meant to be as simple and generic as possible. That way you can be coming from another flux implementation and understand what's being discussed. I'd rather keep this library-agnostic.

Current example isn't really library-agnostic since it relies on AppDispatcher which has Facebook's Flux signature.

Also, this quote:

the boilerplate of Flux stores

and a lot of others quotes about Flux implies to Facebook Flux implementation.


Remove embed library option

Removes unused embed library option from TensileCreateLibrary.

@KKyang @jichangjichang please review or recommend someone as reviewer.


Add CLI11 library

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?  In order to motivate your point, it may be valuable to explain why this change is useful or what problem is being solved.
Additional Notes
This is the section to put technical guidance/constraints, call out potential regressions, cite sources, and in general offer some exposition on the how of the PR.  This section doesn't need to be incredibly detailed, but it makes life easier for reviewers!
How to test the change?
Describe here in detail how the change can be validated.  This is a great section to call out specific tests you've added or improved, or to acknowledge code sections which are particularly difficult to test.

Current dependencies on/for this PR:

main

PR #141

PR #142   👈

PR #143

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: Fh1RcgohLIXtvXiDWMuk

Graphite Merge Job
Current status: 🔄 Waiting for checks to pass
This pull request is currently waiting for CI and mergeability checks to pass before being merged.
This comment was auto-generated by Graphite.

Job Reference: RRcBgCgNPigo0atqHNO7


Chart 8.5.0  spec.ambassador_id in body must be of type array: "string"

Describe the bug
The helm chart 8.5.0 is invalid.
❯ helm upgrade test-ambassador datawire/emissary-ingress --version 8.5.0 -f values.yaml -n test
Error: UPGRADE FAILED: failed to create resource: Module.getambassador.io "ambassador" is invalid: spec.ambassador_id: Invalid value: "string": spec.ambassador_id in body must be of type array: "string"
In commit https://github.com/emissary-ingress/emissary/commit/8ca5a6f461a4629c5d4578c15e4816419e22088f
there has been an update from getambassador.io/v2 to getambassador.io/v3alpha1.
But in v3alpha1:
ambassador_id:
    description: "AmbassadorID declares which Ambassador instances should
      pay attention to this resource. If no value is provided, the default
      is: \n \tambassador_id: \t- \"default\" \n TODO(lukeshu): In v3alpha2,
      consider renaming all of the `ambassador_id` (singular) fields to
      `ambassador_ids` (plural)."
    items:
      type: string
    type: array

this field now is an array, not a single field anymore.
Expected behavior
It should not fail.
Versions

Chart 8.5.0
Every kubernetes version supported

I will prepare the MR for temporary fix this


Possible wrong value in the LANG variable

This line sets the LANG variable to en_us.UTF-8, but it should be en_us.UTF8 without the hyphen.
My mother tongue is portuguese and by having the hyphen, causes text wrapping issues when the text has special characters áãéíóú like so:

When I manually set the LANG variable in the .hyper.js config, it fixed those issues. My settings are:
module.exports = {
  config: {
    env: {
      LANG: 'en_us.UTF8',
      TERM: 'msys'
    },
    fontSize: 12,
    fontFamily: 'monaco',
    cursorShape: 'BLOCK',
    padding: '12px 14px',
    cursorColor: '#f06292',
    shell: 'C:\\Program Files\\Git\\git-cmd.exe',
    shellArgs: ['--command=usr/bin/bash.exe', '-l', '-i'],
  },
  plugins: [
    'hyper-blink',
    'hyperterm-tabs',
    'hyperterm-paste',
    'hyperterm-tab-icons',
    'hyperterm-material-theme'
  ],
  // in development, you can create a directory under
  // `~/.hyper_plugins/local/` and include it here
  // to load it and avoid it being `npm install`ed
  localPlugins: []
}

Windows 10 x64
I didn't test it on Linux or Mac.

https://perlgeek.de/en/article/set-up-a-clean-utf8-environment looks like the hyphen is correct?

glibc should normalize both values that  so UTF-8 would have the same effect as utf8, but that is not happening on Windows 10. If I use en_US.UTF-8 it has the same effect of just en_US and the wrapping issue appears.
I am not sure if this is an issue on how environment variables are passed or in a DLL from windows.

Hey there, please try again with the new Hyper v2 release and open a new issue if the problem stills exists. Thank you!


Fix reimpl show language in status

[ ] Added an entry to CHANGELOG.md if this change could be valuable to users

please rebase on master


implicit argument lists

It is fairly standard practice in implicit-heavy libraries to write implicit parameter lists like this
  def canBuildFromFormat[F, E, T]()(implicit
    cbf: CanBuildFrom[F, E, T],
    ef: SexpFormat[E]
  )

but unfortunately I can't set up scalariform to use this trailing-implicit format. Is this something you could support with a new option?
Note that it is possible to do this sort of thing
  def canBuildFromFormat[F, E, T]()(
    implicit
    cbf: CanBuildFrom[F, E, T],
    ef: SexpFormat[E]
  )

I'm looking for any way to use scalariform without condensing curried function signatures, but the issue isn't just with implicit parameters, it's any kind of currying.
e.g., when I see code like:
def aReallyLongMethodName(thatHasSomeParameters:WithLongTypes)(andThenAnother:ParameterList) = foo

I'd like to be able to write it as:
def aReallyLongMethodName(thatHasSomeParameters:WithLongTypes)
                         (andThenAnother:ParameterList) = foo

But it automatically gets reformatted back to the first one by scalariform.  This prevents me from keeping a tight limit on org.scalastyle.file.FileLineLengthChecker in scalaStyle, and I've often seen single lines of code that get excessively long.
From the preview here it looks like characters aren't monospaced, but what I was getting at in my example is I would typically align the open-paren of my curried parameters, implicit or not.
Ideally I think I'd rather just have scalariform not touch any whitespace between a close and then open paren.  I hadn't considered your style for writing implicit parameters.

I'm looking for any way to use scalariform without condensing curried function signatures, but the issue isn't just with implicit parameters, it's any kind of currying.
e.g., when I see code like:
def aReallyLongMethodName(thatHasSomeParameters:WithLongTypes)(andThenAnother:ParameterList) = foo

I'd like to be able to write it as:
def aReallyLongMethodName(thatHasSomeParameters:WithLongTypes)
                         (andThenAnother:ParameterList) = foo

But it automatically gets reformatted back to the first one by scalariform.  This prevents me from keeping a tight limit on org.scalastyle.file.FileLineLengthChecker in scalaStyle, and I've often seen single lines of code that get excessively long.
From the preview here it looks like characters aren't monospaced, but what I was getting at in my example is I would typically align the open-paren of my curried parameters, implicit or not.
Ideally I think I'd rather just have scalariform not touch any whitespace between a close and then open paren.  I hadn't considered your style for writing implicit parameters.

I think I'll create a separate issue for what I'm describing since the option I'm requesting for curried parameters is a bit different.

I think I'll create a separate issue for what I'm describing since the option I'm requesting for curried parameters is a bit different.

This issue was moved to scala-ide/scalariform#180


Uncontrolled inputs.

Currently uncontrolled inputs (One's that aren't linked to state) are re-rendered on every update.
Below when the "Toggle" button is clicked it will also clear the text input. React doesn't act this way and I'm wondering if there is a way around it without explicitly controlling the inputs state?
Home = {
  getInitialState: function() {
    return {};
  },
  handleClick: function(e, component, setState) {
    var state = component.state;
    return setState({
      clicked: !state.clicked
    });
  },
  render: function(arg) {
    var props, state;
    props = arg.props, state = arg.state;

        { state.clicked ?
Clicked
 : undefined }
        <button onClick={ Home.handleClick }>Toggle</button>
        <input type="text"/>

  }
};

Seems to happen when the order of elements gets moved around (Maybe internally the child list is flattened?)

Yeah this was something we were messing around with recently and something we deliberately did different. The only reason we didn't make it similar to React is that we've never run into a case where this didn't work for us.
In this case, the way the diffing works, is that it ignores undefined or null elements. So it's like it's not even there, meaning that when the div exists, the indexes all shift and they're replacing the HTML elements. Because the DOM keeps state you lose the value the user added.
To get around this you can either:

Not use undefined, and instead use a  or similar.
Just hide the div instead of removing it
Make the input a component and use the key attribute so the state is kept around (we don't yet support keys on elements, only on components).

Hope that helps!

That helps, that's what I thought was happening. Works fine without nullish elements. I am curious though, what is the benefit of ignoring null items as children? Seems like it would be convenient to keep them simply for child indexes.

That's a good point. I mainly use it so that I can just easily show and hide elements conditionally:
let sheet = null;
if (thereShouldBeASheet) {
  sheet = <Sheet />
}

return
{sheet}

But that could easily be done another way. What would you expect the falsy values to look like? Just empty strings?

Not 100% on the internals of deku, but I would guess that an nullish value when diffed would simply act as a removeChild operation.

Not 100% on the internals of deku, but I would guess that an nullish value when diffed would simply act as a removeChild operation.

Yeah this is how it works. If a virtual node is seen on the left, but on the right the value is undefined, it's treated as a removal. However, when rendering virtual elements, any null children are removed before the diff. This is creating the weird behaviour you're seeing (and others have brought up).
We could just keep null elements in there and render them as empty strings potentially. We'd need to do this because the diff looks for the real DOM elements via the index. So if the children looked like this:
[
  null,

]
```

But the real children looked like this:

```html

```

Then when using the indexes it would be wrong. We could either be tricky with this and shift the index, or we could just render a `<noscript/>` for `null` values so nothing renders.

@anthonyshort was this resolved?


fix(ui): Sort top users by their query count in datasets stats tab

Summary:
Currently the top users within the stats tab of a Dataset is not sorted by their query count. This change reflects the top users by their counts.
Example:

Sample usage statistics:
    "aspectName" : "datasetUsageStatistics",     "aspect" : {       "value" : "{ \"timestampMillis\":1674749347000,\"partitionSpec\": {\"type\": \"FULL_TABLE\", \"partition\": \"FULL_TABLE_SNAPSHOT\"}, \"totalSqlQueries\": 155, \"uniqueUserCount\": 1, \"userCounts\": [{\"user\": \"urn:li:corpuser:datahub\", \"count\": 21, \"userEmail\": \"datahub@abc.COM\"},{\"user\": \"urn:li:corpuser:jkadambi\", \"count\": 100, \"userEmail\": \"jkadambi@OPTUM.COM\"},{\"user\": \"urn:li:corpuser:eaipeas\", \"count\": 9920, \"userEmail\":\"eaipeas@abc.COM\"},{\"user\": \"urn:li:corpuser:jac_script\", \"count\": 35, \"userEmail\": \"jac_scriptt@abc.COM\"},{\"user\": \"urn:li:corpuser:dbrumba2\", \"count\": 2035, \"userEmail\"<EMAIL_ADDRESS>      "contentType": "application/json"
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Actually it looks like we are failing the frontend build. Can you please make sure the usage of useMemo is NOT conditional before we accept this?

(You should not need to use useMemo at all)

(You should not need to use useMemo at all)

Will do this shortly. I wondered if I should in the first place. Let me remove it.

(You should not need to use useMemo at all)

@jjoyce0510 - removed useMemo. Please review and accept. Thanks!

(Assuming you've tested this locally)
Once we are green we will merge it in. Thanks!

(Assuming you've tested this locally)
Once we are green we will merge it in. Thanks!

Yes. Looks green. Thanks!


hound: unstable-2018-11-02 -> unstable-2021-01-26

Motivation for this change
Things done
Tested search and works like a charm.

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

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

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

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 112650 run on x86_64-darwin 1

1 package built:

hound

Tested it and search works like a charm.


SCM viewlet focus colors need work

#20999
You cannot tell the difference between an opened diff editor and a focused diff editor line in the viewlet:
Diff editor is opened and focused:

Diff editor line is focused:

Clean button focused (needs more contrast):

.travis.yml is focused, .gitignore is open (this looks like the opposite of what it should be to me):

For reference here's the explorer:
Item focused:

Editor focused:

.travis.yml is focused, .gitignore is open (again this looks like the opposite of what it should be to me, cc @bpasero):

This is probably due to using the list widget vs tree widget that has different styles. +1 for same looks.

The current list is now exactly the same as the tree. As for the opposite, we'll let theming solve it. 👍


network_security_groups created through inject_into_existing_vcns have autogenerated names

network_security_groups created through inject_into_existing_vcns have autogenerated names - the attribute display_name seems to be ignored.

Fixed in v1.3.2


Get-AzureADMSDeletedDirectoryObject Requires an Object Id

There is currently no way to retrieve all soft deleted users. The Get-AzureADMSDeletedGroup has an optional Id parameter to return all deleted groups.
This command seems to be very immature, the Id should check all ID fields (immutableId, UPN, email address, etc.) and should be optional.
There should also be a parameter for the object type (User/Group/etc.).
Other parameters such as -Filter, -All, -Top etc. should be added.

Hi @DougChandler
Sorry for the delay in response.  I have opened a work item for this so that engineering can take a look and possibly address this.  At this point, I am going to go ahead and close this issue and will post any update I have here.  Thanks!
Bill


[Feature] Reusable Components

Summary
Reusable components - widgets, APIs or DB Queries. There is often a use case while we code in React we create one component and use it across different pages.
Motivation
Currently, it is possible by copying the component and use it on a different page so that we don't have to create one again and again. But this itself does not solve the purpose of reusablity. As if one component changes we need to go across changing all components on different pages.
Consider a use case:
We have a page with multiple widgets with a particular CSS styling. We copy all the widgets and use them on different pages. Now at someday, we realize the CSS styling has to be changed in all widgets of the page. We have to go across changing in a lot of places.
Usually, while we code, we keep reusable components in some common directory so that we can use it across easily and when the component changes it is reflected on every page required.

@KirtiChug thanks for the details!

++ to this.
retool calls it "modules". Oracle Apex calls it "shared components".
The ability to drag a component from page-scope to app-scope, and then back down to another page is a feature I'll be looking for in appsmith's future.

@segfall thank you for the upvote! We're really excited about this feature as well. Stay tuned for a product note on it. We'd love your input

Closing this as a duplicate of #5229


Falsey values do not trigger subview's waitFor condition

My specific example is a 'number' type where the incoming JSON value is a numeric 0. Doesn't trigger the subview render.
After some testing, this affects ALL falsey values I've tried. So a boolean field with an incoming value of false will not trigger the waitFor. Null/undefined, I can understand the reason not to have those function, but boolean false and numeric 0 are very valid use cases.

Yep... this is the problematic line:
https://github.com/AmpersandJS/ampersand-view/blob/master/ampersand-view.js#L301
Should waitFor only be triggered for non-null or undefined values or do we have more in the list?
P.S - This is a breaking change

Yes, we should probably be just more explicit about those checks.

This hack has worked for me thus far. Any issues with this? If not, I could whip up a PR soon.
waitFor = getPath(this, opts.waitFor);
if (!opts.waitFor || waitFor || (waitFor !== undefined && waitFor !== null)) {

So how would you wait for collections to be populated? If you waitFor: 'model.collection.length that would work in the current code, however, what if the collection truly has no models? Any ideas how to correctly waitFor a collection?

@bryanspears, looks like you had an open PR for this issue.  what was the scoop w/ that?


Add open-source alternatives for translation management systems (transifex, lokalize, etc)

Everything is in the title
I know of weblate for example https://github.com/WeblateOrg/weblate

Hey @Startouf,
That's a good suggestion! Do you feel like making a PR for that?


Release Google.Cloud.ApigeeRegistry.V1 version 1.0.0-beta02

Changes in this release:
New features

Added support for force field for API and API version deletion (commit f4c4c28)

Documentation improvements

Updated proto comments to align with the public documentation (commit f4c4c28)

Created release for Google.Cloud.ApigeeRegistry.V1-1.0.0-beta02


refactor(transformer/class-properties): rename class_binding

#7516
#7532
#7531
#7533  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Provide official Docker Image in a public container registry

Currently several people (including me) are hosting their own builds of the Druid Docker image on docker hub.
It would be very helpful to have an official docker image provided by this project so we aren't reliant on building our own or trusting an unknown docker image.

Apache does not provide official docker images. We can however provide convenience binaries on the apache dockerhub account. I just requested to have automated builds be setup there (see https://jira.apache.org/jira/browse/INFRA-18887)

https://hub.docker.com/r/apache/incubator-druid is set up. A new image will be pushed automatically with the next release.

https://hub.docker.com/r/apache/incubator-druid is set up. A new image will be pushed automatically with the next release.

Awesome, I'll be sure to add it to the release notes for 0.16.0

Well done! I also raised up a new PR https://github.com/apache/incubator-druid/pull/8403 to add a docker shield with this official dockerhub link.


Calendar picker input button not rendering correctly on Firefox with grid < 1200px and Segoe UI

This renders correctly on IE and Chrome but on Firefox, the calendar button is outside the box.
Using "Segoe UI" and Max-Width of 1000px (Problem)
https://codepen.io/anon/pen/EEJXGE
Using "Segoe UI" and Max-Width of 1200px (No Problem)
https://codepen.io/anon/pen/NYmvdp
Although at below 1000px, no font renders correctly on firefox.
https://codepen.io/anon/pen/LdvjjK

This is a firefox bug https://bugzilla.mozilla.org/show_bug.cgi?id=1043520
I fix this behavior in 4.1.14 (maybe today)

fixed


MetastoreScanInputFormat uses a single ThriftClient in many threads

See the code that passes a single ThriftClient to many theads.
I don't think the ThriftClient is thread-safe.  We have many databases, so many threads (16 by default as defined in the source code) will be running on a single ThriftClient, which causes unexpected failures on the server side.
Suggestion fixes:
A. Create a new ThriftClient for each database, or
B. Just get rid of the multi-threaded case here since it won't take too much time to list through tens or even hundreds of databases.

Fix in #41


bug: Plugins signed with ethpkg 0.9.0 does not work

Grid v1.6.2 fail loading plugins signed with ethpkg 0.9.0.
The console show a error similar that "signature invalid or corrupt file"
I resolved this problem signing my plugin with ethpkg 0.3.0, but is really difficult put to work this old version
In other hand a reference in docs about this compatibility issue will be really useful

Hi @gonzalobellino thanks for the report. We are planning to do a new Grid release soon and I might be able to assist your use case to make sure you can either use the new version or fork and use an older version. Will give an update here once the scope of the release is clear which should happen in the next 1-2 weeks.

I can publish a docker image with ethpkg 0.3.0 if you think that will be useful


Layer view, CSS, Panels, Plugins

The CSS and creation of layer views needs to be reviewed. It should be possible for plugins to hook into the layer panels which are created in the layers.view() method.

It should be possible to have additional methods for a drawer object assigned to a drop down menu.


feat: Make Python functions support visible without an env var

Removes the requirement for the env var PYTHON_FUNCTIONS_ALPHA to be set, in order to use the --language python option to sf generate function, sf run function start and sf run function start local, since we are now moving to the beta phase.
The readme changes were generated using yarn build && npx oclif readme.
GUS-W-12110506.

https://github.com/salesforcecli/plugin-functions/releases/tag/1.17.0


java.lang.IllegalStateException: path.home is not configured

Version Information

Search Guard - 5.0.0-17
Elasticsearch - 5.0.0
JVM version - Oracle 64-Bit 1.8.0_111
OS - Ubuntu 16.04
Number of nodes in your cluster - 1

Description of the bug
I am trying to attach a TransportClient to my local ElasticSearch server (thus the number of nodes is 1) with the following code:
Settings settings = Settings.builder()
                .put("cluster.name", "<cluster_name>")
                .put("client.transport.sniff", true)
                .put(SSLConfigConstants.SEARCHGUARD_SSL_TRANSPORT_KEYSTORE_FILEPATH, "<key_store_full_path>")
                .put(SSLConfigConstants.SEARCHGUARD_SSL_TRANSPORT_TRUSTSTORE_FILEPATH, "<truststore_full_path>")
                .put(SSLConfigConstants.SEARCHGUARD_SSL_TRANSPORT_KEYSTORE_PASSWORD, "<keystore_password>")
                .put(SSLConfigConstants.SEARCHGUARD_SSL_TRANSPORT_TRUSTSTORE_PASSWORD, "<truststore_passwrod")
                .build();

this.client = new PreBuiltTransportClient(settings, SearchGuardSSLPlugin.class)
			.addTransportAddress(new InetSocketTransportAddress(new InetSocketAddress("localhost", 9300)));

Whenever I run this code I get the following exception:
java.lang.IllegalStateException: path.home is not configured

	at org.elasticsearch.env.Environment.<init>(Environment.java:115)
	at com.floragunn.searchguard.ssl.SearchGuardKeyStore.initSSLConfig(SearchGuardKeyStore.java:172)
	at com.floragunn.searchguard.ssl.SearchGuardKeyStore.<init>(SearchGuardKeyStore.java:139)
	at com.floragunn.searchguard.ssl.SearchGuardSSLModule.<init>(SearchGuardSSLModule.java:29)
	at com.floragunn.searchguard.ssl.SearchGuardSSLPlugin.createGuiceModules(SearchGuardSSLPlugin.java:125)
	at org.elasticsearch.plugins.PluginsService.createGuiceModules(PluginsService.java:280)
	at org.elasticsearch.client.transport.TransportClient.buildTemplate(TransportClient.java:134)
	at org.elasticsearch.client.transport.TransportClient.<init>(TransportClient.java:228)
	at org.elasticsearch.transport.client.PreBuiltTransportClient.<init>(PreBuiltTransportClient.java:69)
	at org.elasticsearch.transport.client.PreBuiltTransportClient.<init>(PreBuiltTransportClient.java:65)

If I add "path.home" settings the TransportClient gets initialized successfully. Is it necessary to provide "path.home" settings during initialization of a TransportClient? I do not see any reason to init my TransportClient as passing "path.home" setting. Am I missing something?
Thanks,
Radoslav

"path.home" is enforced by elasticsearch core, so this is not Search Guard related

Here is an example how to make transport client work: https://github.com/floragunncom/search-guard/blob/5.0.0/src/test/java/com/floragunn/searchguard/SGTests.java#L2156

@floragunncom thanks you!


Parameter of type Object matches first type of Cell regardless of T's type

write: func(data: Object) {
	(match (data) {
		// This below case matches Cell<T> for any T, which is a problem related to how generic types are implemented
		case c: Cell<String> =>
			"To get here I should be a Cell<String> but apparently I'm a " + c class name + "<" + c T name + ">"
		case cell: Cell<Int> =>
			cell get() toString()
		case =>
			"FAIL"
	}) println()
}
write(Cell new(2))

outputs

To get here I should be a Cell<String> but apparently I'm a Cell<Int>

Beyond that, I still think it's a better idea to nest matches by hand.

Can you give an actual example that allows me to match on the T in Cell<T> after I've matched on Cell?
write: func <T> (data: Object) {
    (match (data) {
    // "case value: Cell =>" won't build because "No type parameters for Cell. It should match Cell",
    // as already mentioned, so doing "Cell<Object>" instead
		case value: Cell<Object> =>
			(match (value T) {
				case valTee: String =>
					value get()
	        	case valTee: Int =>
            		value get() toString()
            })
        case =>
            "FAIL"
    }) println()
}

This won't build because "No such function toString() for Object". Fair enough, but if I change it to
case value: Cell<String> =>
    (match (value T) {
    case valTee: String =>
        value get()
    case valTee: Int =>
        value get() as Int toString()

I instead get the error Missing info for type argument T. Have you forgotten to qualify write<T> (data : Object :Object), e.g. List<Int>? on the call write(Cell new(2)). This may very well be a legit error, but I don't know how to fix it. It sounds to me as if I'm supposed to write write<Cell>(Cell new(2)), which I'm pretty sure was exactly what you said you didn't like about how other languages do generics. It also makes rock segfault... :)
Ok, so let's ignore the function that takes an Object for now, and try something simpler: A function that takes Cell<T>:
write: func <T> (data: Cell<T>) {
    (match (data T) {
        case value: String =>
            data get() as String
        case value: Int =>
       	    data get() as Int toString()
        case =>
            "FAIL"
    }) println()
}
write(Cell new(2)) // prints "FAIL"
write(Cell new("Hello")) // also prints "FAIL"

Well, that didn't work.
What about (Cell<T>, T)?
write: func <T> (data: Cell<T>, t: T) {
    (match (data T) {
        case value: String =>
            data get() as String
       	case value: Int =>
            data get() as Int toString()
        case =>
            "FAIL"
    }) println()
}

Well, C compiler failed on module testtestest from testtestest, bailing out. Obviously, I have no idea what I'm doing anymore. Help?

It's kind of hard to read through all that so it's entirely possible that I missed soemthing, but I noticed something.
You can either do
cell: Cell<T>
match (cell T) {
  case Int =>
    i := cell get() as Int
  case Float =>
    f := cell get() as Float
}

(which is long-form), or you can do the short form:
cell: Cell<T>
match (cell get()) {
  case i: Int =>
    // whatever
  case f: Float =>
    // whatever
}

But above, you're conflating the two, so there's no chance it'll work :) Don't confuse generic values & generic types.

@davidhesselbom here's a full example that works well for me:

Whatever: class { init: func }

write: func ~noCell <T> (t: T) {
    write(Cell new(t))
}

write: func <T> (cell: Cell<T>) {
    match (cell get()) {
        case i: Int =>
            "Got int #{i}" println()
        case f: Float =>
            "Got float #{f}" println()
        case =>
            "Got other type #{cell T name}" println()
    }
}

write(42 as Int)
write(3.14 as Float)
write(Whatever new())

(And indeed, it's a pity that the second write here needs a <T>, it shouldn't. I think Cell<?> syntax should exist. The compiler check for unqualified generic types is probably too severe, but believe me, it did a lot of damage control back in the days...)

Splendid, thank you sir!

No problem. Sorry it took a long-ass issue for this info to get out. It'd be probably right at home somewhere at https://github.com/fasterthanlime/ooc-lang.org/blob/master/content/docs/lang/generics.md, perhaps in a more documentation-y formulation. If someone has time, don't hesitate to send in a pull request.

I'll look into it. Meanwhile, I gave you another, much smaller one :)

So, matching the actual typeArgs should be another issue (if at all), but matching generic types without specifying typeArgs at all is now possible.


chore: Handle clippy warnings

Clippy was being noisy about certain patterns that are now addressed.
Handled some deprecation warnings as well.

Clippy work was done in a separate PR: https://github.com/Corpauration/mdbook-treesitter/pull/3


Add support for multiple gateways

Description:
I want to have several gateways(envoy proxies) under one envoy gateway and use parentRef in the HttpRoute to decide on which gateway the HttpRoute should get deployed. I tried the above scenario with the v0.2.0-rc1 and couldn't make it work.
Steps I followed:

Followed the quick start guide and deployed 1 EG, 1 gateway, httpbin and 1 HttpRoute and verified the functionality. It works as expected.
Deployed a second gateway using the following CRD using kubectl apply -f gateway.yaml.

apiVersion: gateway.networking.k8s.io/v1alpha2
kind: Gateway
metadata:
  name: egtest
spec:
  gatewayClassName: eg
  listeners:
    - name: http
      protocol: HTTP
      port: 8080

I'm getting the following error in envoy gateway container logs.
2022-09-13T07:05:50.939Z	ERROR	runner/runner.go:95	unable to validate infra ir, skipped sending it	{"runner": "gateway-api", "error": "listener ports field required", "errorCauses": [{"error": "listener ports field required", "errorCauses": [{"error": "listener ports field required"}]}]}
github.com/envoyproxy/gateway/internal/gatewayapi/runner.(*Runner).subscribeAndTranslate
	/home/runner/work/gateway/gateway/internal/gatewayapi/runner/runner.go:95

It says listener ports field required but I have provided it as you can see here.

Note, this is similar to or maybe identical to #349 -- multiple Gateways, each with Listeners with identical hosts/ports/protocols.

fixed with https://github.com/envoyproxy/gateway/pull/404


getPicture decode WsDr and search picture twice.

Description

Did you have any problems?  What's the result you expected, what's the result you expected? Please follow the issue template, and briefly describe the problem you are having in a few paragraphs.

Did you got any problems?  What's the result you expected, what's the result you expected? Please follow the issue template, and briefly describe the problem you are having in a few paragraphs.

Maybe what I need is a function to directly get all the pictures in the xlsx file, for example, get a picture map indexed by rows and columns. This will avoid decoding WsDr in the loop.


Adding constraints to CQM

I'm trying to use CQM for an optimization problem and have basically formulated my model using two segments :

Objective
Constraint

I created a BQM to model the objective and basically just set that as the objective of the CQM.
['INTEGER', 53 rows, 53 samples, 40 variables]
there should at least be one x_i equal to 1 so we are using at least one subset to cover the U list, but it completely ignores the constraint and just minimizes it by using no subsets.
Is the issue from me implementing the constraints wrong (is there another form I should use to add constraints? I use this form as my approach to implement constraints : https://github.com/dwavesystems/dimod/issues/1165
how can I resolve this so that the model actually takes the constraint into effect when running?
P.S this is a code question as my mathematics has checked out, and am purely asking about the code, which I assume this issue is under logical error of not adding constraints right(otherwise it should be working).
Please do not close this as I have seen many issues regarding adding constraints to CQM model and none have been answered fully. Thank you.

Hi @ACE07-Sev,
I suggest that you try filtering by feasible solutions:
print(sampleset.filter(lambda d: d.is_feasible))

I am now going to close this issue because it is a debugging question and therefore not on topic. I am going to again direct you to our Leap community for this sort of question in the future.


sorters option

When I enable subtotal.js, the sorters option in pivot doesn't work.
Do this option not exist when using subtotal.js? Or is there an equivalent?

scratch that - looks like an issue my end. Sorry!
Great plugin by the way (Y)


sdk-auth Deprectated ?

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az account show
Errors:
unrecognized arguments: --sdk-auth

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az account show --sdk-auth --debug

Expected Behavior
Environment Summary
macOS-12.5-x86_64-i386-64bit, Darwin 21.6.0
Python 3.10.6
Installer: HOMEBREW

azure-cli 2.39.0

Extensions:
azure-devops 0.23.0
application-insights 0.1.16

Dependencies:
msal 1.18.0b1
azure-mgmt-resource 21.1.0b1

Additional Context

@jiasli for awareness

@revinjchalil Please go through this issue for further details https://github.com/MicrosoftDocs/azure-docs/issues/92827

Well, i wasn't doing az ad sp create-for-rbac but instead was doing a az account show. So, is it deprecated for both of these commands?
Also, what was really weird was - look at the output in the below screen shot - the first example it lists down is with the option of --sdk-auth  - missing documentation update?

az ad sp create-for-rbac still has --sdk-auth, but we don't recommend using it anymore.
--sdk-auth was dropped from az account show in https://github.com/Azure/azure-cli/pull/21219. You may check that PR for more details.
The examples are from Aladdin AI knowledge base. We are working with Aladdin team to remove it.

@jiasli  - Thank you for the clarification. Our use case is indeed to leverage the Azure Login Github Action. If --sdk-auth is not recommended, what is the alternate way to use this Github Action?
Unfortunately, our organization needs elevated privilege(s) to execute the az ad sp create-for-rbac command. Anyways, I think the option for me is to manually construct the json to leverage in Github Actions.

We have removed the outdated example with Aladdin recommendation team:


test: add test for importUnmanagedInstance

Description
Refactored smoke test for unmanaging a  VM and then importing it.
Moved the test class to anew file to prevent Trillian re-running them in case of failures of other tests in test_vm_life_cycle.py.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[x] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

Tested with a VMware env having changes from #6384
[root@pr6385-t4175-vmware-67u3-marvin marvin]# nosetests --with-xunit --xunit-file=results.xml --with-marvin --marvin-config=./pr6385-t4175-vmware-67u3-advanced-cfg -s -a tags=advanced --hypervisor=VMware tests/smoke/test_vm_life_cycle_unmanage_import.py
/usr/local/lib/python3.6/site-packages/paramiko/transport.py:33: CryptographyDeprecationWarning: Python 3.6 is no longer supported by the Python core team. Therefore, support for it is deprecated in cryptography and will be removed in a future release.
  from cryptography.hazmat.backends import default_backend
/usr/local/lib/python3.6/site-packages/paramiko/transport.py:236: CryptographyDeprecationWarning: Blowfish has been deprecated
  "class": algorithms.Blowfish,

==== Marvin Init Started ====

=== Marvin Parse Config Successful ===

=== Marvin Setting TestData Successful===

==== Log Folder Path: /marvin/MarvinLogs/May_13_2022_08_08_47_OI6KDY All logs will be available here ====

=== Marvin Init Logging Successful===

==== Marvin Init Successful ====
=== TestName: test_01_unmanage_vm_cycle | Status : SUCCESS ===

=== Final results are now copied to: /marvin//MarvinLogs/test_vm_life_cycle_unmanage_import_M0XC3L ===

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

This is WIP

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 3401

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 3402

@blueorangutan test centos7 vmware-67u3

@shwstppr a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) has been kicked to run smoke tests

Trillian test result (tid-4180)
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 38610 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6385-t4180-vmware-67u3.zip
Smoke tests completed. 96 look OK, 1 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_01_isolated_persistent_network
Error
62.73
test_persistent_network.py

test_02_L2_persistent_network
Error
1.47
test_persistent_network.py

test_03_deploy_and_destroy_VM_and_verify_network_resources_persist
Failure
51.11
test_persistent_network.py

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 3409

@blueorangutan test centos7 vmware-67u3

@shwstppr a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) has been kicked to run smoke tests

Trillian test result (tid-4189)
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 41151 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6385-t4189-vmware-67u3.zip
Smoke tests completed. 97 look OK, 0 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File


Add missing french translations of plugin

Added missing some french translations (exception Activity Log details).
Please notice that I also changed the label name of first_name field.yaml (model User), from Name to First Name. This gave a wrong translation for some forms (eg Groups).
Also notice that I prefered to translate "Ban" to the "Block" general meaning, more understandable in french as "Bannir" (ban). But we can talk about this if you prefer "Bannir".

Thanks!


Add preWrite to ReplicaConsistency

Internally we are experimenting with a new way of tracking inconsistencies. We
increase an integer in the transaction just after changes committed. We use
ReplicaConsistency#write. This approach has some disadvantages:

it doubles the number of transactions. It also doubles the number of changes to be replicated. It may affect replication lag.
we need two database calls to propagate the information (update + commit).
We increase inconsistency time. We don't track when changes are propagated, but we handle changes after the changes are propagated.

Connection#preWrite allows us to inject our counter bump into the same transaction.

@tciesla-atlassian we can use it to improve sequence-value-consistency-mode. It may reduce the replication lag.