Added a header image background

Please, go through these steps before you submit a PR.

[x] My Pod Leader knows I'm working on this Pull Request
[x] I've explained what the Pull Request is adding.
[x] I've explained why this is important.

Added a background image which can be used for multiple screen sizes

Hey @sgoyal5! I can't see the change you made in the deploy preview, could you please maybe add a screenshot or something?

Dear Ms. Basu,
Attached are the changes that were made. Let me know if you need anything additional.
Sincerely,
Sana Goyal
The University of North Carolina at Chapel Hill
B.S. Computer Science & Statistics, Data Science Minor | May 2025
@.*** |<PHONE_NUMBER>

From: Sumana Basu @.>
Sent: Thursday, March 9, 2023 3:22 AM
To: MLH-Fellowship/prep-portfolio-23.MAR.PREP.1 @.>
Cc: Goyal, Sana @.>; Mention @.>
Subject: Re: [MLH-Fellowship/prep-portfolio-23.MAR.PREP.1] Added a header image background (PR #20)
Hey @sgoyal5https://github.com/sgoyal5! I can't see the change you made in the deploy preview, could you please maybe add a screenshot or something?
—
Reply to this email directly, view it on GitHubhttps://github.com/MLH-Fellowship/prep-portfolio-23.MAR.PREP.1/pull/20#issuecomment-1461545034, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXJOF5AHRTYFC2YUOOVJUXDW3GHODANCNFSM6AAAAAAVUSOFTE.
You are receiving this because you were mentioned.Message ID: @.***>

Dear Ms. Basu,
I just wanted to reach out to say that I wasn't able to talk during the meeting today since I woke up with a big fever this morning. My seasonal allergies began flaring up yesterday and I woke up this morning feeling extremely congested and feverish. My partner said that she would be able to show you our changes after the meeting, but I just wanted to reach out to apologize for not responding on the call today. Thank you so much, and please let me know if you need anything additional from me.
Sincerely,
Sana Goyal
The University of North Carolina at Chapel Hill
B.S. Computer Science & Statistics, Data Science Minor | May 2025
@.*** |<PHONE_NUMBER>

From: Goyal, Sana @.>
Sent: Thursday, March 9, 2023 12:32 PM
To: MLH-Fellowship/prep-portfolio-23.MAR.PREP.1 @.>
Subject: Re: [MLH-Fellowship/prep-portfolio-23.MAR.PREP.1] Added a header image background (PR #20)
Dear Ms. Basu,
Attached are the changes that were made. Let me know if you need anything additional.
Sincerely,
Sana Goyal
The University of North Carolina at Chapel Hill
B.S. Computer Science & Statistics, Data Science Minor | May 2025
@.*** |<PHONE_NUMBER>

From: Sumana Basu @.>
Sent: Thursday, March 9, 2023 3:22 AM
To: MLH-Fellowship/prep-portfolio-23.MAR.PREP.1 @.>
Cc: Goyal, Sana @.>; Mention @.>
Subject: Re: [MLH-Fellowship/prep-portfolio-23.MAR.PREP.1] Added a header image background (PR #20)
Hey @sgoyal5https://github.com/sgoyal5! I can't see the change you made in the deploy preview, could you please maybe add a screenshot or something?
—
Reply to this email directly, view it on GitHubhttps://github.com/MLH-Fellowship/prep-portfolio-23.MAR.PREP.1/pull/20#issuecomment-1461545034, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXJOF5AHRTYFC2YUOOVJUXDW3GHODANCNFSM6AAAAAAVUSOFTE.
You are receiving this because you were mentioned.Message ID: @.***>

Hey @sgoyal5 I am sorry to hear that, I hope you feel better soon. Don't worry, take rest. Yes, Jenna was able to clear everything up😊

This photo has a watermark in it. Maybe use a one which doesn't have it?


Update Gradle to 4.5.1 / Adopt common Gradle scripts / Update dependencies

Armeria 0.58.0
Guava 24.0
Jackson 2.9.4
Spring Boot 1.5.10
gogradle 0.8.1
gradle-docket-plugin 3.2.4
client-dependencies 1.4.1
jGit 4.10.0
TestNG 6.14.2

Codecov Report

Merging #134 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #134      +/-   ##
==========================================
+ Coverage   61.36%   61.37%   +0.01%
==========================================
  Files         260      260
  Lines        8770     8770
  Branches     1016     1016
==========================================
+ Hits         5382     5383       +1
+ Misses       2783     2781       -2
- Partials      605      606       +1

Impacted Files
Coverage Δ

...internal/replication/ZooKeeperCommandExecutor.java
72.27% <0%> (-0.34%)
:arrow_down:

...ernal/storage/repository/git/CommitIdDatabase.java
73.27% <0%> (+1.72%)
:arrow_up:

Continue to review full report at Codecov.

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

Thank you, @hyangtack !


do i need to setup key/value store to be able to add multi-host network?

I'm trying to understand if on Docker 1.12 with swarm - do i need to configure a key/value store like etc/consul/zookeeper or is that no longer needed with swarm on 1.12? i was under the impression that it does all of it on it's own after docker swarm init and docker join commands ?
so on a brand new 3-node cluster I followed all steps but the overlay network gets created in a single node (on manager node) only and when i scale service tasks - it doesn't seem like other hosts are getting picked up by the haproxy that gets deployed on manager node.
could you elaborate on what i'm missing, whether it is a key-value store or something else that i need to do for things to work properly on swarm 1.12 ?
thank you

upgrade docker to rc4 and network started to work but now ports don't get exposed anymore

No kv is needed. The network only shows on a node when a task is scheduled
on that node that needs the network.
On Jul 13, 2016 9:41 PM, "Alexei Smirnov"<EMAIL_ADDRESS>wrote:

I'm trying to understand if on Docker 1.12 with swarm - do i need to
configure a key/value store like etc/consul/zookeeper or is that no longer
needed with swarm on 1.12? i was under the impression that it does all of
it on it's own after docker swarm init and docker join commands ?
so on a brand new 3-node cluster I followed all steps but the overlay
network gets created in a single node (on manager node) only and when i
scale service tasks - it doesn't seem like other hosts are getting picked
up by the haproxy that gets deployed on manager node.
could you elaborate on what i'm missing, whether it is a key-value store
or something else that i need to do for things to work properly on swarm
1.12 ?
thank you
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ehazlett/interlock/issues/188, or mute the thread
https://github.com/notifications/unsubscribe/AAP6IqCSmtNKk9c05fGOQ9JRoR2hJXYcks5qVa_xgaJpZM4JMDgX
.


Version number out of date

I am using Kahlan 1.1.8 but it reports as 1.1.3
The cause is this line, which is hard coded:
https://github.com/crysalead/kahlan/blob/master/src/cli/Kahlan.php#L35
Thanks,

Gavin

@jails, i think we must do some composer magic for not hardcoding version into config file.
@gavD, good catch :+1:

I'm not sure composer can be of any help here.
I think this is going to pissing me off for a while !
Btw, thanks for the head up @gavD !

Fixed in https://github.com/crysalead/kahlan/commit/8fad06d63a5d7083822041a35a45b98e9b9ce29c


msal-browser throws error when running unit tests in Jest

Please follow the issue template below. Failure to do so will result in a delay in answering your question.

Library

[ ]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Important: Please fill in your exact version number above, e.g<EMAIL_ADDRESS>Framework
React 16.13.0
create-react-app
jest
Description
new msal-browser package throws error when running unit tests in jes.
Error Message
 Test suite failed to run

    BrowserAuthError: crypto_nonexistent: The crypto object or function is not available. Detail:Browser crypto or msCrypto object not available.

      42 | };
      43 |
    > 44 | export const msalApp = new PublicClientApplication({
         |                        ^
      45 |     auth: {
      46 |         clientId: "58a18e2e-f3cb-4c6d-b07f-77703e6a215b",
      47 |         authority: "https://login.microsoftonline.com/b36c7ed3-4a9c-4ba9-bdf9-9db03fcbd392",

      at BrowserAuthError.AuthError [as constructor] (node_modules/@azure/msal-common/dist/index.es.js:109:24)

Security

[ ] Is this issue security related?

Regression

[x] Did this behavior work before?
Version:
Yes<EMAIL_ADDRESS>
MSAL Configuration
// Provide configuration values here.
// For Azure B2C issues, please include your policies.
{
    auth: {
        clientId: "<>",
        authority: "<>",
        validateAuthority: true,
        redirectUri: window.location.origin
    },
    cache: {
        cacheLocation: "localStorage",
        storeAuthStateInCookie: false
    },
    system: {
        navigateFrameWait: 0
    }
}

Reproduction steps
export const msalApp = new PublicClientApplication({
    auth: {
        clientId: "<>",
        authority: "<>",
        validateAuthority: true,
        redirectUri: window.location.origin
    },
    cache: {
        cacheLocation: "localStorage",
        storeAuthStateInCookie: false
    },
    system: {
        navigateFrameWait: 0
    }
});
...

Expected behavior
Browsers/Environment

[ ] Chrome
[ ] Firefox
[ ] Edge
[ ] Safari
[ ] IE
[x] Other (Node / jest)

If you're upgrading from 1.3.0, we never used any of the crypto objects/APIs in that version and previous. It's possible that you need to mock the instantiation of the crypto objects in your tests now after not having to do so before. This can be seen in PublicClientApplication.

@nick-romano-kci Specifically, you will need to mock the window.crypto object. See here

Closing this as we're assuming the problem is solved due to inactivity. Please open a new issue if you have further questions. Thanks!

To solve this problem the following jest config settings helped me
module.exports = {
  globals: {
    crypto: require('crypto')
  }
};


PGlite Next - Based on Postgres 16.x

This PR changes for support building based on 16.x postgresql releases directly by patching in wasm support, emscripten dynamic loading for postgres extensions and ready for electric sql support via PGlite Typescript interface.
It also introduces CI with development builds on each PR, and support for extensions with pgvector being the first.
You can try this new build and the new REPL here: https://668ce81c27123c506c6a53a6--pglite-dev-demos.netlify.app/pglite/examples/repl
or, download the "PGlite" package from CI, unzip it, and manually install it in a project.
Each build now has a set of "interim build files", these are the output of the Emscripten build step. If you want to work on just the TypeScript side of PGlite, you can copy these files into /packages/pglite/release/ and run pnpm build:js to run the second step of the build.
Some of the issues that will be resolved by this PR include:

Refresh materialised views #63
Issues with TOAST #83
Missing timezone data #62
EMULATE_FUNCTION_POINTER_CASTS #56
PGlite crashing after errors #94

if this MR merged, if i want to using some other extension, for example pg-tde(encrypt the whole database),
how can i do this? should i write some script and compile it to wasm? but i'am not good at cpp or c language.  can you provide some docs on this ?

Hey @jueinin,
Yes, we will provide docs and examples showing how to build custom extensions.

Just a heads up, I'm hitting some issues running this on Next.js (webpack) with the error:
Module not found: Can't resolve './postgres.so'

I'm still digging into this, but I believe this is related to the new URL("...", import.meta.url) syntax (here) which Next.js tries to resolve at build time and can't since postgres.so doesn't exist for web builds. I'm fighting through some issues trying to get PGlite building on my local dev machine with the new infrastructure in this branch (running on a mac) which I'll need to resolve before I can debug this further.

Hey @gregnr
Thanks for the report. I think @pmp-p may have been working on something in this area?
We should add e2e tests for various frameworks and bundlers to catch these issues.

I managed to get past the above error by temporarily commenting out the postgres.so URL and switch logic. Interested to hear @pmp-p's thoughts!
Now running into a new unreachable WASM error that seems to happen in _pg_initdb:
Uncaught (in promise) RuntimeError: unreachable
    at postgres.05b6d054.wasm:0x33ae7a
    at postgres.05b6d054.wasm:0x253d96
    at postgres.05b6d054.wasm:0x325aef
    at postgres.05b6d054.wasm:0x2b883a
    at a._pg_initdb (chunk-6ZCTMRJP.js:15:55901)
    at Ha.Ds (chunk-6ZCTMRJP.js:15:82830)
    at async run (db.ts:24:5)

Let me know if you've seen something like this or if a repro repo would be helpful.

@gregnr postgres.so ( which was in fact libecpg.so ) is gone, only some symbol where needed in there so i re implemented them in static pglite main, it makes the bundle 520KiB smaller and extensions building easier so it was really important to do it before merge.
freezing now, unless some bug is found ofc !

@pmp-p Hi I'am wondering, can we add a password feature for pglite?
just like postgres server side?
given the right password, so that this pglite can be connected?

md5 passwords can be handled and currently defaults to "password" but anyone could find it looking at the wasm or pglite source code


update libraries

@lucabello seems like the integration tests might need some updating to accommodate for the changes in the lib.

@lucabello seems like the integration tests might need some updating to accommodate for the changes in the lib.

I don't think this is changes in the lib. The integration test looks here. And there are two dashboards:
Traceback (most recent call last):
  File "/home/runner/work/cos-configuration-k8s-operator/cos-configuration-k8s-operator/tests/integration/test_grafana_dashboards.py", line 86, in test_dashboard_files_ingested_by_grafana
    assert "up with dropdowns" in dashboard_titles
AssertionError: assert 'up with dropdowns' in ['up (with datasource variables)', 'up (without datasource variables)']

Those match up to the titles in the dashboards it passes. There just doesn't seem to be a dashboard titled up with dropdowns in the repo at all. I don't see a commit where it would have been dropped, either.
@sed-i am I missing something?

It's an integration test that has not been updated; you can see the dashboard the test is trying to find here.
It also means this is probably not the first time this test should have failed (and in fact the last merge has failing CI). In any case, I'm updating the dashboard names to the ones we currently have!


Rate limiting support for web socket connections

Rate limiting support for web socket connections
Is there a recommended way to rate limit the web socket connections ? I went through the docs and some other resources and seems like there's no straight forward way to have the rate limiting for web socket connections.(Maybe use the network rate limit filter? ) So basically my requirement is to capture web socket frames and possibly decode them to extract the information I need for rate limiting. Is it possible to achieve my requirement by writing a network filter to capture, extract and send information to an external rate limiting service ?

cc @alyssawilk I think using the HTTP ratelimit filter in the upgrade filter chain would work here? cc @phlax also for potential doc opportunities.

So your first request was to limit WebSocket connections, which as Matt says can be done with the existing rate limit filter.  Your follow-up comment makes it look like you actually want to rate limit not just WebSocket connections but actual WebSocket work, which can't be done with Envoy today as Envoy doesn't frame WebSocket.  Which are you hoping for?  You could arguably write a filter which does the second, if that's the granularity you're looking for.

@mattklein123 @alyssawilk Thanks a lot for the quick response. So yeah I want to rate limit web socket work not just the number of web socket connections. So can I achieve that by writing a network filter to capture the web socket frames ? I mean will the web socket frames hit the network filters before proxying to upstream clusters ?

Again I think you want to write an HTTP filter, not a network filter, since you want to take the HTTP payload and parse WebSocket frames from there.
There may also be some work to get RateLimit code to do what you want - I'm less familiar with that part of the stack.

@alyssawilk okay but will the web socket frames go that far to the http filters ? As I read only the initial upgrade request go to http filters and then after the connection establishment it behaves more like a raw TCP proxy.

You can use an upgrade filter chain to use normal HTTP filters before the data hits the tcp_proxy layer.
I think your best bet would be to write a custom filter that sets dynamic metadata with some business logic, and then uses the normal HTTP rate limit filter to rate limit on dynamic metadata.

@mattklein123 thanks a lot. will try it out and see.

@mattklein123 @alyssawilk I implemented a http decoder filter to log data when decodeData() is called. Web socket frames are getting captured with that filter as you said.
[2020-11-08 13:19:52.464][49714][trace][filter] [http-filter-example/http_filter.cc:44] Filter sample-filter decoded DATA : ����Am�
Since web socket frames don't contain any meta data or header , is there any other connection related meta data i can use ?
Maybe connection_id /socket_id ?
i.e - Count the no of frames or payload in bytes that is related to a particular connection_id/ socket_id.

@mattklein123 @alyssawilk I created a custom http filter and trying to send the web socket frames through that after the upgrade. The connection gets upgraded successfully but the custom filters are not applied. Only the usual http filter chain is applied. Anything wrong with the configuration ? Thanks
static_resources:
  listeners:
    - address:
        socket_address:
          address: <IP_ADDRESS>
          port_value: 10000
      filter_chains:
        - filters:
          - name: envoy.http_connection_manager
            config:
              upgrade_configs:
              - upgrade_type: websocket
              - filters:
                - name: sample
                  config:
                    key: via
                    val: xxx
                - name: envoy.router
              stat_prefix: http_proxy
              codec_type: AUTO
              route_config:
                name: all
                virtual_hosts:
                  - name: allbackend_cluster
                    domains:
                      - '*'
                    routes:
                      - match: { prefix: "/admin"}
                        direct_response: {status: 403, body: { inline_string: "Forbidden, yo"} }
                      - match: { prefix: "/"}
                        route: { host_rewrite: echo.websocket.org, cluster: ws_cluster, upgrade_configs: {upgrade_type: websocket, enabled: true}}
              http_filters:
                  - name: sample
                    config:
                      key: via
                      val: sample-filter
                  - name: envoy.router
  clusters:
    - name: ws_cluster
      type: logical_dns
      dns_lookup_family: V4_ONLY
      connect_timeout: 10s
      lb_policy: ROUND_ROBIN
      hosts: [{ socket_address: { address: echo.websocket.org, port_value: 80 }}]

That looks correct to me at a glance, would have to defer to @alyssawilk for any missing details.

I was confused too - we have regression tests for this!
After a whole bunch of frustrating debugging, it turns out the difference is between
          - upgrade_type: websocket
            filters:
            - name: sample
              config:
                key: via
                val: sample-filter-upgrade

Which configures a custom filter chain for "websocket"
and
- upgrade_type: websocket
- filters:
- name: sample
config:
key: via
val: sample-filter-upgrade
Which configures a websocket upgrade with no custom filter chain, and a custom filter chain for upgrade type "" due to the extra "-".  Removing the "-" in front of filters should fix things for you.
For Envoy, cc @htuch,  I know tweaking the config after the fact is disallowed, so how can we disallow empty upgrade types in Envoy?  Can we just enforce in Envoy and comment in the API it should be non-empty?

Actually cc @envoyproxy/api-shepherds since there's an @ for that.

Would adding a min_len PGV constraint work?

Yeah IMO we should just add the constraint as it is implied and broken without it.

+1, but I wasn't sure if adding constraints after the fact was "legal"
Agree it's broken if you're doing any zero length fields, just not sure if failing to accept on invalid-but-previously-accepted-config is OK.

I think in this case it is ok to just add the constraint. I would not expect anyone to use empty upgrade type.

Hi , I have just one more question regarding the upgrade mode for web sockets. (Sorry if i'm spamming the thread with tons of questions 😅)
So the access logging (envoy.access_loggers.http_grpc) is applied for every http connection when i configure it in the http connection manager config. But for web sockets since the connection stays open , the access logging filter is not applied per web socket frame. It gets applied when the connection is closed which is the intended way for usual http connections.
So I just wanna know is there anything I could do to apply access logging per web socket frame without having to wait for the connection termination ? Thanks.

@mattklein123 @alyssawilk As matt said I am trying to set up a custom header and use that header to send the web socket frames to an external rate limit service using the global rate limit filter. (See the upgrade filter chain).
But only the initial upgrade request hits the global rate limit filter. The web socket frames contain the custom header "via: sample-filter-upgrade" and I have configured to catch the request header name "via" but it does not hit the global rate limit filter. Do I have to configure anything additionally ?
I tested for normal http requests and it works fine. I can provide any additional info and trace. Thanks
static_resources:
  listeners:
    - address:
        socket_address:
          address: <IP_ADDRESS>
          port_value: 10000
      filter_chains:
        - filters:
          - name: envoy.http_connection_manager
            config:
              upgrade_configs:
              - upgrade_type: websocket
                filters:
                - name: sample
                  config:
                    key: via
                    val: sample-filter-upgrade
                - name: envoy.rate_limit
                  config:
                    domain: rl
                    request_type: external
                    stage: 0
                    rate_limited_as_resource_exhausted: true
                    failure_mode_deny: false
                    rate_limit_service:
                      grpc_service:
                        envoy_grpc:
                          cluster_name: ws_cluster
                - name: envoy.router
              stat_prefix: http_proxy
              codec_type: AUTO
              route_config:
                name: all
                virtual_hosts:
                  - name: allbackend_cluster
                    domains:
                      - '*'
                    routes:
                      - match: { prefix: "/service1"}
                        route:
                          cluster: app1_cluster
                          rate_limits:
                            - actions:
                              - request_headers:
                                  header_name: "via"
                                  descriptor_key: "foo"
                      - match: { prefix: "/"}
                        route:
                          host_rewrite: echo.websocket.org
                          cluster: ws_cluster
                          rate_limits:
                            - actions:
                              - request_headers:
                                  header_name: "via"
                                  descriptor_key: "foo"
              http_filters:
                  - name: sample
                    config:
                      key: via
                      val: sample-filter
                  - name: envoy.rate_limit
                    config:
                      domain: rl
                      request_type: external
                      stage: 0
                      rate_limited_as_resource_exhausted: true
                      failure_mode_deny: false
                      rate_limit_service:
                        grpc_service:
                          envoy_grpc:
                            cluster_name: ws_cluster
                  - name: envoy.router
  clusters:

    - name: app1_cluster
      connect_timeout: 1s
      type: strict_dns
      lb_policy: round_robin
      load_assignment:
            cluster_name: app1_cluster
            endpoints:
              - lb_endpoints:
                - endpoint:
                    address:
                      socket_address:
                        address: <IP_ADDRESS>
                        port_value: 8000

    - name: ws_cluster
      type: logical_dns
      dns_lookup_family: V4_ONLY
      connect_timeout: 10s
      lb_policy: ROUND_ROBIN
      hosts: [{ socket_address: { address: echo.websocket.org, port_value: 80 }}]

Regarding access logging, I think if you want per-WebSocket logging you're going to want to include that in  your filter.  Both HTTP and TCP proxy sessions log relevant information per-stream, and there's nothing saying your filter (which is doing the framing) couldn't determine the relevant loggers and logger->log(per-WebSocket info) on each frame.

@alyssawilk Thanks a lot. I went through the code and still couldn't figure it out how to include access logger in my filter and call that logger_->log() method.

@alyssawilk Hi, is it possible to get metadata related to web socket frames when it hit the stream filter ? I just want to differentiate web socket data frames from ping/pong and open/close frames. According to web socket specs, we can use the opcode for that differentiation but from my decodedata() method I'm getting a raw byte stream for web sockets.

@alyssawilk Hi, is it possible to get metadata related to web socket frames when it hit the stream filter ? I just want to differentiate web socket data frames from ping/pong and open/close frames. According to web socket specs, we can use the opcode for that differentiation but from my decodedata() method I'm getting a raw byte stream for web sockets.

yeah, I would expect decodeData to get a raw byte stream, and you'd want to hand that byte stream to a WebSocket framer, and have it do the parsing.  I'd be game for landing that framer upstream even if the WS ratelimiter doesn't land there, since so many folks using Envoy use WS.  Or maybe this is a case for non-core extensions the @envoyproxy/maintainers have been discussing.  But in-house or upstream, I think you'd want a WS framer, with an API like the HTTP codecs, which took in a byte stream and called back on important framing events (for HTTP/2 headers, bodies, pings, rst, etc, for WS I guess the frames you mention)

yeah, I would expect decodeData to get a raw byte stream, and you'd want to hand that byte stream to a WebSocket framer, and have it do the parsing.  I'd be game for landing that framer upstream even if the WS ratelimiter doesn't land there, since so many folks using Envoy use WS.  Or maybe this is a case for non-core extensions the @envoyproxy/maintainers have been discussing.  But in-house or upstream, I think you'd want a WS framer, with an API like the HTTP codecs, which took in a byte stream and called back on important framing events (for HTTP/2 headers, bodies, pings, rst, etc, for WS I guess the frames you mention)

I think it's reasonable to start with a framer for WebSocket per @alyssawilk. We have something similar for gRPC here: https://github.com/envoyproxy/envoy/blob/master/source/common/grpc/codec.h. Also per @alyssawilk given how many people use WS and Envoy I think it's reasonable to also think about more first class codec/filter support for it.

I think it's reasonable to start with a framer for WebSocket per @alyssawilk. We have something similar for gRPC here: https://github.com/envoyproxy/envoy/blob/master/source/common/grpc/codec.h. Also per @alyssawilk given how many people use WS and Envoy I think it's reasonable to also think about more first class codec/filter support for it.

@alyssawilk @mattklein123 Right now I'm kinda tight on my timeline of the current project I'm doing. So I'm gonna just create a function to decode the byte stream and read the op code to decide the frame type. But I would love to contribute to that when I get some free time and if you give some tips to start with. Thanks

@alyssawilk @mattklein123 Right now I'm kinda tight on my timeline of the current project I'm doing. So I'm gonna just create a function to decode the byte stream and read the op code to decide the frame type. But I would love to contribute to that when I get some free time and if you give some tips to start with. Thanks

@alyssawilk Hi ,I have few questions and clarifications about the way decodeData works when a web socket frame passes through a filter.
1 . So far what I've seen is for each web socket frame, decodeData gets called once and I can access all the frame data in one go. But I'm not sure what will happen when a very large payload comes, whether decodeData() gets called multiple times or not ?

What happens when web socket frames are continuously passing through the filter ? They get buffered and I get like a slice of that buffer via onDecode callback ?

Simply my concern is that whether we always get the whole web socket frame from decodeData or we get a slice of the frame data depending on a max buffer size ? I want to read the 1st byte of each web socket frame to determine the frame type but it's only possible if decodeData gives a whole web socket frame.

Yeah, there's no guarantee that a decodeData call will contain a single complete websocket frame.  It's possible to have multiple sent at once, and it's possible to have a large frame spanning several decodeData calls.  You'll basically want to handle streaming data, either by buffering until the full frame is present, or draining the buffer and stashing it internally.  You can test this by just chunking your data into small bits and sending them one at a time, to make sure it works as expected.

@alyssawilk I already implemented my filter assuming that it will get called for every web socket frame although i had this confusion. And as you mentioned in the previous comment, I couldn't find a proper websocket framer written in C++ and it's not feasible to implement a framer myself at the moment. Ideally I'm looking for something that I can do framer::getframes(decodeDataBody) or any library that do callbacks like onFrame(). Any suggestions ?

yeah sorry, I've not actually had to frame websocket, I just pass the bytes back and forth to people who do :-P

Haha okay :-D . Thank you for all the info.

@NomadXD  were you successful  in rate limiting your web socket work >]? can you share your results ?

I implemented the filter both as a native filter and also as a WASM plugin using proxy-wasm-cpp-sdk.

Would you be kind enough to share your methodology for rate-limiting websockets? What was the impact on latency and error-rates for all web-socket connections?


Add support for stub files, deleter props, google-style class docstrings, bug fixes

I should be better at breaking up branches+PRs...
Queuing up for switching to calendar versioning with features/fixes that addresses #128 #114 #126 and #122 🎉

closing since this has been broken up into other PRs


Fix input handler

That, and add the option for the Makefile to generate an executable with
debugging symbols, good for running it with gdb or other debuggers

Thanks for fixing the debug compilation mode!
I'd purposefully not given much effort into the testing block, by the way. If you hadn't updated it, it probably would've never been modified :P

Merging...

But now we have the pieces for our final input handler. I'd still call it a win. :)


Upgrading Golang version from 1.17 to 1.18 in jx-admin

Golang supports only 2 major releases: https://endoflife.date/go
Go 1.19 is expected to be out in August 2022.
Wrapping with the flow of this https://github.com/jenkins-x/jx/issues/8283

Seems like this https://github.com/jenkins-x-plugins/jx-admin/pull/264 updates the go version to 1.18 for jx-admin, so can close this issue.


docs: Use flexbox (for Hugo documentation)

Fixes #2373 #2938 #2955 #2956 #2957 #2958 #2959 #2960 #2961 #2962 #2963 #2964 #2965 #2966 #2967 #2968 #2969.
(See also #1725 #1523 #1362.)
Provides a flexbox layout for Hugo's documentation (based on this model) which is visually identical to the current one, except for the following features:

Space efficiency: more document content immediately visible (on desktop):

Header height lessened
Next/previous navigation arrows:

Extra horizontal padding removed
Stacked (vertically, conserving horizontal space)
Use no width if absent

Multi-document menu:

Adjusts to widest entry
Sub-entry groups fully visible (vertically)
No extra horizontal padding when toggled to hidden

Responsive on smartphones:

Multi-document menu:

No longer disappears on resizing
Arrows:

Better positioning (vertically)
Corrects "staircase" bug (seen in Chrome & Opera)

Tested with developer tools

Printing
Functional with or without JavaScript — now, these fall back gracefully:

Hamburger (for smartphones)
Menu expansion

Multi-document menu:

Top level, single entries (which lack sub-entries) now highlight, when selected
Sub-entries:

Right-justified (on desktop and smartphones)
Centered (on wide smartphones)

Scrollbar position remembered while toggling
No longer partially remains, when toggled to hidden (sometimes)

Table of Contents:

Scrolls (on overflow)
Text wraps
On narrow desktops:

No longer disappears
Partially transparent overlay

GitHub button styling unified (visually), since all are (equally) clickable
Tooltips added, for:

Hamburger
Logo (and version)
Multi-document menu entries (including sub-entries)
Each GitHub button

Accessibility:

Keyboard users:

Focus indications added:

Hamburger
Version (and logo: both link to the main page)

Screen readers (and keyboard users):

Tab order improved (except Firefox, due to this bug):

Forward:

Document content is placed first in the DOM (after the search box)

Backward, from the end:

Header is placed nearest:

Hamburger is reached first

Next/previous navigation arrows are placed next (-nearest)
Multi-document menu is third (-nearest)
Table of Contents is fourth (-nearest)

Site Showcase widened to 4 columns, except:

3 columns, on narrow desktops
1 column, on smartphones
2 columns, on wide smartphones

Includes Autoprefixer output for:

Last 2 versions (of all major browsers)
Browsers > 1% (selected by global usage statistics)

Readable "view page source" (to help new users)

Tested on:

Firefox
Google Chrome
Opera (latest for desktop, plus version 36 for Windows XP)
IE 11 (plus 10 by emulation)

I appreciate the effort, but we need to land the discussion on the docs before we can take PRs.


[otbn] Always run OTBN smoke test in CI

The OTBN smoke test tests OTBN (as the name implies), but also depends
on other IP blocks/primitives in OpenTitan. Changes to those blocks can
hence fail the test, even if no OTBN changes were made.
Currently, we only run the OTBN smoke test if OTBN changes have been
detected. #7694 is a good example that this heuristic isn't reliable:
a PR went in making changes to prim_lfsr. The PR CI passed because the
OTBN test wasn't executed, and then master CI failed (because there we
always run the OTBN tests), leading to a broken master tree.
This commit removes the condition on the test and always runs it.

Could also remove the change detection from master/ci/scripts/get-build-type.sh ?

I left it in there to have the variable around if we need it again, but I could be convinced otherwise :-) What do you think?

No strong feelings really happy to leave it in.


Update release, testing checklist

Fixes | Closes | Resolves #1384

Please remove anything marked as optional that you don't need to fill in.
Choose one of the keywords preceding to refer to the issue this PR solves, followed by the issue number (e.g Fixes # 666).
If there is no issue, remove the line. Remove this note after reading.

Changes introduced in this PR:

Types of changes
What types of changes does your PR introduce?
Put an x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features to not work as expected)
[x] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing
Requires testing

[ ] Yes
[ ] No

In case you checked yes, did you write tests?

[ ] Yes
[ ] No

Documentation
Does your contribution include breaking changes or deprecations?
If so have you updated the documentation?

[ ] Yes, docstrings
[ ] Yes, main documentation
[ ] Yes, deprecation notices

Further comments (optional)
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered and more.

@kcpevey @trallard I took your suggestions and added links to the items on the testing checklist (the keycloak url will work once this PR is merged).


Resolver Expand functionality

Expand to obtain a list of wanted identifiers (of different types)

Implemented in 9f129a2e293aee57c36cdba33e72a66b3ecc994d


Move SQLite ref integrity pragma to keylime_db

This will make sure it's consistent across verifier, registrar and tenant
Signed-off-by: Michael Peters<EMAIL_ADDRESS>

Codecov Report

Merging #1101 (4af9a3b) into master (3ff90dc) will increase coverage by 0.08%.
The diff coverage is 82.70%.

:exclamation: Current head 4af9a3b differs from pull request most recent head e90a0ce. Consider uploading reports for the commit e90a0ce to get more accurate results

Additional details and impacted files

Flag
Coverage Δ

packit-e2e
73.32% <82.70%> (+0.07%)
:arrow_up:

testsuite
51.57% <67.21%> (+0.69%)
:arrow_up:

unittests
13.61% <10.21%> (+0.14%)
:arrow_up:

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

Impacted Files
Coverage Δ

keylime/cmd/registrar.py
66.66% <ø> (ø)

keylime/ima/ast.py
77.57% <0.00%> (ø)

keylime/revocation_actions/update_crl.py
0.00% <0.00%> (ø)

keylime/elchecking/policies.py
68.00% <40.00%> (-3.43%)
:arrow_down:

keylime/revocation_notifier.py
61.63% <50.00%> (+1.02%)
:arrow_up:

keylime/tpm/tpm_main.py
74.31% <70.37%> (-0.14%)
:arrow_down:

keylime/ca_util.py
53.70% <75.00%> (-0.03%)
:arrow_down:

keylime/web_util.py
82.11% <78.48%> (-5.65%)
:arrow_down:

keylime/config.py
75.94% <79.34%> (+2.38%)
:arrow_up:

keylime/cloud_verifier_tornado.py
67.18% <83.63%> (-0.27%)
:arrow_down:

... and 21 more


Data Engineering with Snowpark Pandas API QuickStart

New Quickstart for doing Data Engineering with Snowpark Pandas API.

Looks good overall! Some minor comments in the review above.

It looks like the draft of another quickstart was merged in. Is this intentional?
In addition, could you rename the folder from site/sfguides/src/data_engineering_piipelines_with_snowpark_pandas to site/sfguides/src/data_engineering_pipelines_with_snowpark_pandas`?
Thanks!

@sfc-gh-dolee it should all be fixed now.
@iamontheinet Would it be possible for you to merge this?
It is a new qucikstart for Snowpark Pandas.
Many thanks!


Question of RAdam's dependence on the number of examples

I'd like to confirm what:

Specifically, we identify that, due to the limited amount of samples in the early stage of model training, the adaptive learning rate has an undesirably large variance and can cause the model to converge to suspicious/bad local optima.

means. I read the the paper and didn't see RAdam depend on the number of training examples. So I don't really undestand what you mean by " limited amount of samples in the early stage of model training". If you are training via epochs the model "sees" the entire training set at every "epoch step". So I am unsure how to interpret that.
Thanks for sharing your work with us!

Hi, thanks for the question.
The amount of samples here refer to the number of batches (e.g., if one epoch has 1000 batches,  the optimizer sees 1000 samples after its training on this epoch). I believe the controlled experiments (Adam-2k v.s. Adam wo warmup) is the best justification for this. Note that simply making the batch to be 2000x larger would still leads to convergence failure, if not using warmup.
In RAdam, we try to update less when the variance is large (w. small number of batches).


Opening Hours?

There's a way to get the Opening hours object using this lib?

The suggest.gmaps object contains the complete response from Google. As this library uses the  googleMaps.places.AutocompleteService, there are no opening hours. You can use the place_id returned from Google and query for the opening hours at the Places API.


Local document does not work

When I try to use the RAG feature, the Local documents is gray and can only use web URL.

HI xiongyu0523,
Currently, you need to first use a terminal (CUI) to create a directory where you store your own files.
Then on the GUI, the directory shows up and you can select it.
For the details, please follow this direction.
https://github.com/NVIDIA-AI-IOT/jetson-copilot?tab=readme-ov-file#2-build-your-own-index-based-on-your-documents
Essentially, you would need to execute the following, either inside the container or even outside of the container (native):
cd jetson-copilot
mkdir Documents/Jetson-Orin-Nano
cd Documents/Jetson-Orin-Nano
wget https://developer.nvidia.com/downloads/assets/embedded/secure/jetson/orin_nano/docs/jetson_orin_nano_devkit_carrier_board_specification_sp.pdf


fix: Plugin initializers onActivate onDeactivate

fixes https://github.com/gravitee-io/issues/issues/6942

Still need to discuss this while this is in draft.
@tcompiegne @leleueri


No support for multiple control replicates

There is currently a TODO to add support for control replicates (R/parseHelpers.R, removeControls function).

No longer relevant with v2 of metagene.


Issue #952: Rename use: "value" to use: "constant"

Renames "value" to "constant" when defining the use property of models.

thanks!


Uncaught Error: ENOENT: no such file or directory, uv_chdir

[Enter steps to reproduce below:]

Create a new file and switch the syntax to SuperCollider
cmd-shift-K

Atom Version: 1.0.0
System: Mac OS X 10.10.3
Thrown From: supercollider package, v0.5.5
Stack Trace
Uncaught Error: ENOENT: no such file or directory, uv_chdir
At /Applications/Atom.app/Contents/Resources/app.asar/node_modules/babel-core/node_modules/regenerator/node_modules/commoner/node_modules/graceful-fs/polyfills.js:14

Error: ENOENT: no such file or directory, uv_chdir
  at Error (native)
  at process.chdir (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/babel-core/node_modules/regenerator/node_modules/commoner/node_modules/graceful-fs/polyfills.js:14:9)
  at Repl.module.exports.Repl.startSCLang (/Users/mac/.atom/packages/supercollider/lib/repl.coffee:53:15)
  at /Users/mac/.atom/packages/supercollider/lib/controller.coffee:47:12
  at Workspace.module.exports.Workspace.openURIInPane (/Applications/Atom.app/Contents/Resources/app.asar/src/workspace.js:442:22)
  at Workspace.module.exports.Workspace.open (/Applications/Atom.app/Contents/Resources/app.asar/src/workspace.js:375:19)
  at Controller.module.exports.Controller.openPostWindow (/Users/mac/.atom/packages/supercollider/lib/controller.coffee:118:22)
  at Controller.module.exports.Controller.recompile (/Users/mac/.atom/packages/supercollider/lib/controller.coffee:131:8)
  at atom-workspace.<anonymous> (/Users/mac/.atom/packages/supercollider/lib/controller.coffee:18:38)
  at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:241:29)
  at CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:61)
  at CommandRegistry.module.exports.CommandRegistry.dispatch (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:156:19)
  at EventEmitter.<anonymous> (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:70:30)
  at emitOne (events.js:77:13)
  at EventEmitter.emit (events.js:166:7)

Commands
     -1:45.8.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -0:33.4.0 grammar-selector:show (atom-text-editor.editor)
     -0:28.8.0 core:move-down (atom-text-editor.editor.mini.is-focused)
     -0:28.3.0 core:move-up (atom-text-editor.editor.mini.is-focused)
     -0:12.9.0 grammar-selector:show (atom-text-editor.editor)
     -0:09.4.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -0:02.5.0 supercollider:recompile (a.inline-block)

Config
{
  "core": {
    "projectHome": "/Users/mac/Desktop/",
    "disabledPackages": [
      "language-supercollider"
    ]
  }
}

Installed Packages
# User
supercollider, v0.5.5

# Dev
No dev packages

it seems its trying to chdir to a non-existent directory
what is your project directory ?  by which I mean: did you open Atom on a directory of files, or did you just open it with an empty window ? (though I just tried this and it worked fine for me)
does /Users/mac/Desktop/ exist ?

new version: It no longer changes directory

config
sclang:/Applications/SuperCollider/SuperCollider.app/Contents/MacOS/sclang
sclang:/Applications/SuperCollider/SuperCollider.app/Contents/MacOS/sclang
Executable not
found:/Applications/SuperCollider/SuperCollider.app/Contents/MacOS/sclang
booting
Error: spawn
/Applications/SuperCollider/SuperCollider.app/Contents/MacOS/sclang ENOENT
sclang exited
On Fri, Jul 3, 2015 at 12:20 AM, Chris Sattinger<EMAIL_ADDRESS>wrote:

Closed #46 https://github.com/crucialfelix/atom-supercollider/issues/46.
—
Reply to this email directly or view it on GitHub
https://github.com/crucialfelix/atom-supercollider/issues/46#event-347154443
.

That is what its supposed to do: check the path and report to you that there is no 'sclang' at that path.
Look for /Applications/SuperCollider/SuperCollider.app/Contents/MacOS/sclang
does it exist ?


Maximum recursive updates exceeded in component in 3.4.15

Vue version
3.4.15
Link to minimal reproduction
https://play.vuejs.org/#eNqNU01v2kAQ/SsjXwA12I1oLxRQmiiV2kOLmh59MfZATL0f2p0lRMj/vbO7QI3bRLl5Z+bNvPfGc0g+a53uHCbTZGZLU2sCi+T0Ipe10MoQHKBUQjvCClpYGyVgwPWDTl6jsUp+KUpS5vlclGbLEB98ymUuSyUtgTZKW5hDheta4tK/hodcAshC4BTCJwA9a348kKnl5iqGGFC4hqYwuFUrng3Q5rIddVuHYdz7xHY4HMF8cUluGAikftrIg2dZ1Mxq+UEodFMQ8gtgVtW7xeEkLkDSXdE4hLadZT7p4R1IcpVER8ai0OmWQexpUJQfEzZPzhrzhE307zx5JNJ2mmVlJRlWYVPvTCqRMqlFdsNlmXGSaoHjSombSfohvf7IBCx14ylaMV4Z9WTRcJc8ORoX5mQc3KEZG5QVGjRvnduDXczu5f6Zf9oR20KWV7SuNz1T/KLqBs0PTTWv8MKcomnU07cQI+PwLKZ8xPL3f+Jbu4+ilgYDs44BVJgNUkzfP3zHPX+fk0JVrjku4oXkT7SqcZ5jLLt1smLanbrA9mvYMf+yv+z9nlDakyhPNLgR6sNC7l6R/pfuJJ10XIzXxB6yg+fTM7juXeXlPdwp/kX4KLhu+D6eC+4Ddu1k6Yf3DiScRiQT+6jVlvFHdvFMfbNQFwS1oSt0Bx4P5d0crjkJDCBnpG/FT5aTtH8AoE2GOg==
Steps to reproduce
Open the link and see the maximum recursion error
What is expected?
No maximum recursion error
What is actually happening?
Maximum recursion error
System Info
System:
    OS: macOS 14.1.1
    CPU: (12) arm64 Apple M2 Max
    Memory: 65.34 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.13.0 - ~/.nvm/versions/node/v18.13.0/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 8.19.3 - ~/.nvm/versions/node/v18.13.0/bin/npm
    pnpm: 8.8.0 - ~/Library/pnpm/pnpm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 120.0.6099.234
    Safari: 17.1

Any additional comments?
I understand that computed properties should not have side effects, and this example does have a side effect, however there are some cases where classes/factories might modify some global reactive state for tracking purposes. In 3.4.14 and before this example works, but this breaks in 3.4.15.
 I don't know if this is necessarily a realistic requirement in the real world, especially if all previous versions of Vue 3 (whether intentionally or unintentionally) supported this.
I believe other people are possibly running into this issue:
https://github.com/vuejs/core/issues/10210
https://github.com/vuejs/core/issues/10172
https://github.com/vuejs/core/issues/10158
I'm pretty confident that this PR introduced this behavior: https://github.com/vuejs/core/pull/10123

Got this issue earlier today when updating to 3.4.15. Checking at where this is happening, I don't think I have any side effects from computed values but I do have a reference to an external class which might be causing this. Checking if I can build a repro for this.

Got this issue earlier today when updating to 3.4.15. Checking at where this is happening, I don't think I have any side effects from computed values but I do have a reference to an external class which might be causing this. Checking if I can build a repro for this.

So, please ignore my previous comment. I fixed the issue on my side. Was actually a simple fix once I understood what was happening.
The only thing I'd say is that even though this is the correct behaviour, this breaks previous behaviour so it should be marked as a breaking change or pushed back to a major release.

@johnsoncodehk I think this is also a question about the computed trigger their deps while getting the effect's value. However, computed will not cause a maximum recursion issue since computed values are lazily calculated, and they are only marked as MaybeDirty. It might be more reasonable if computed, render, and watch all have the same behavior regarding maximum recursion (either all cause it or all do not cause it).

We have Maximum recursive updates eceeded issue SINCE 3.4.16.

I just tried 3.4.16 and I'm getting the same "Maximum recursive updates eceeded".

you can commit a new issue and provide a minimal reproduction since this issue is closed

Confirmed that the fix worked fine for my case.


Critical bug fix and speed improvements

0.3.1

Fixes critical bug that allowed access when ips is empty and mode == 'allow'.
Adds minor speed improvements for middleware.
Minor spelling and documentation fixes in README

@armiiller are you ready to merge this?

@ryanbillingsley - yep, go for it

@ryanbillingsley - thanks for the quick response and merge! 👍


RFC: Mixed mutable variables

This RFC proposes Mixed mutable variables, including Mixed mutable references.
Rendered
struct Point2 {x : f32, was_x : f32}

let mut p1 : Point2 = Point2 {x:1.0, was_x:2.0};

fn pntm_store (&^type p : & ^{^const x, ^mut was_x} Point2)  {
   *p.was_x = *p.x;
}

pntm_store(&^arg p1); // effective mutable reference

let ^type p2 : ^{^mut x, ^const was_x} Point2 = Point {x:1.0, was_x: 4.0};

let ref_p2 = &^max p2; // mixed mutable reference

This is an extension to  Partial Types Proposal #3426

As I see, Partial Mutability is a better choice Mixed mutable variables, so I close this draft proposal.
Thanks to everyone read it!


Problem with prod environment

Hi @yceruto,
I've just upgraded a multi-kernel application from Symfony 4.4 to 5.3. I use multi-kernel in order to use 2 separate easy_admin configurations.
Everything works as expected in dev environment, but prod environment looks weird. I log into admin (APP_NAME=main), then log to extranet (APP_NAME=extranet), then request the main kernel, but it has extranet configuration loaded (routing, easy_admin config).
The only difference with your skeleton is that I do not use env var in web context :
<?php

declare(strict_types=1);

use App\Kernel;

require_once \dirname(__DIR__).'/vendor/autoload_runtime.php';

return static function (array $context) {
    // Virtal Kernel
    $appName = strpos($context['REQUEST_URI'], '/extranet') === 0 ? 'extranet' : 'main';

    return new Kernel($context['APP_ENV'], (bool) $context['APP_DEBUG'], $appName);
};

Any idea of what could go wrong on my side ?

My problem seems resolved by differenciating containerClass :
    protected function getContainerClass()
    {
        return ucfirst($this->name).parent::getContainerClass();
    }

I don't know why yet...

Hi @alterphp
I will update the template to take this into account. Thanks!


refactor: Example application with image processing workflow

Includes:

removal of libp2p record checks on scheduler (leaves it to worker)
update across the board toward async channels
resource map now includes content over the network as part of fetching
new fetch module
better use of channels to spread out work between internal and swarm events
websocket-relay app documentation and more

Closes https://github.com/ipvm-wg/homestar/issues/246.

@bgins is looking at a warning and a fix to activeworkflowstate for ordering of workflow plays.


SPIKE: Add Python nose unit testing support

This card is to be elaborated

Getting type mismatch error. See my stackoverflow question for details.

I FIX IT!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! See commit 06a4855 and I'll post an article to talk about that.

add test to verify module load and test function.


Add Information to each Node

Make each node component store the following information: Two strategies (They can be empty), The Prisoner's dilemma points, Current points each strategy has, Name

Make sure it's connected to the store as well


Play URI to local file

Hi,
I've been trying to get play_url. to work with a local .mp3 file.
I've tried sonos.play_uri('path/to/file.mp3') and sonos.play_url('file://path/to/file.mp3') however I get a UPnP error 714 thrown.
The question is can the module play from local files or do songs have to come from .mp3 http streams? All examples i've seen show remote http streams however i'd ideally like to simply play local mp3 files.
Thanks!

Sonos doesn't have access to files hosted locally on your computer by default. You could run a simple HTTP server to change this, however, and then play_url('http://yourcomputer/file.mp3')

You can play files from any file system via Sonos, but the permissions must be set so Sonos can read from the file system. I use SoCo to play tracks from my NAS which has read access to all local devices.
import soco
device = soco.SoCo('192.168.1.xx')
device.play_uri('x-file-cifs://My_NAS/Music/track_1.mp3')

You have to use the prefix 'x-file-cifs://'
You can play tracks form your PC if you set the permissions correctly and use the PC host name.
Hope this helps,
Cheers David

Hallo @RobSmith2
I can see that there are a couple of suggestions on how to do what you want, so I'll consider this closed. Please feel free to re-open if your question was not answered.


Add sample Sprint Boot vulnerable application

Hello!
I am proposing this link as it was deemed useful by the community to reproduce the issue without spending time building a vulnerable Java application.
Thanks for the resources!
Christophe

Thanks for your contribution !


I'm not sure what this is, turning annotations off isn't making it go away.

The video at https://www.youtube.com/watch?v=NB4-I5M9YXA has three things appear at 11:12, which I thought were annotations, that link to other videos. Turning annotations off is not disabling these things. What are these things, and how do I disable them? Is this a flaw in YouTubeCenter, or do I have some kind of mental malfunction?

New Youtube feature which is still rolling out, endscreen cards. They are not the same as annotations and because of that they cannot be disabled like them.

Auto play is now enabled I have noticed the last few days. Something else that needs fixing. Wish YT would stop "improving" things.

OK, it is a feature. How do I disable the feature?

@SteegHobland You can disable it by this script.
I hope I'm not breaking rules by posting this.

@SteegHobland as this addon is no longer in active development, I find it unlikely that the author will implement a feature for this. However, I've changed the title and added a feature request label just in case.
@Pezor As long as it is nothing malicious it is fine.


warn the user that the Excel spreadsheet must contain  a Sheet named MediaCompounds

Fix for JIRA 1274; but I haven't been able to test it because I can't download the test data (see JIRA 2813).

Can you submit this against the develop branch instead of master?

I think you also want to do exit(1) so that the script terminates with an error, otherwise it appears that the script ran successfully.


eth_blockNumber returns L1 block for Arbitrum fork

Component
Anvil
Have you ensured that all of these are up to date?

[x] Foundry
[x] Foundryup

What version of Foundry are you on?
forge 0.2.0 (6defdce 2023-05-17T00:04:50.286076000Z)
What command(s) is the bug in?
anvil
Operating System
macOS (Apple Silicon)
Describe the bug
$ anvil --chain-id 42161 --fork-url https://arb1.arbitrum.io/rpc &>/dev/null &
$ cast block-number --rpc-url https://arb1.arbitrum.io/rpc
168785678
$ cast block-number --rpc-url http://<IP_ADDRESS>:8545
18971606

An Anvil fork of Arbitrum returns the L1 block number over JSON-RPC. Based on the Arbitrum docs and the behavior of other providers, I expected it to return the L2 block number.

I see, this makes me think calling this when we instantiate the fork is wrong:
https://github.com/foundry-rs/foundry/blob/177a1f41658db813fc4d7f50f40ab7804fcb427c/crates/evm/core/src/utils.rs#L163-L169
but in solidity block.number is still the L1 block, right?
so this is a bit horrible. We want to fork off the L2, what should happen with the block.number variable over time?

Partly related to #748, you can see the arbitrum block.number case is called out there. @mattsse you are right that in solidity block.number returns L1 block values as described in https://docs.arbitrum.io/for-devs/concepts/differences-between-arbitrum-ethereum/block-numbers-and-time

hmm, okay so the right fix for abitrum would be to not use the l1blocknumber for the fork target block number (L2), but unclear what we should do to evm.blocknumber once new blocks are mined, simply increase by one as well or with some multiplier (L2 blocktime vs mainnet blocktime?)

hmm, okay so the right fix for abitrum would be to not use the l1blocknumber for the fork target block number (L2)

I believe this is correct, since the RPC returns the L2 block number, we want to use L2 block numbers as the fork blocks

but unclear what we should do to evm.blocknumber once new blocks are mined, simply increase by one as well or with some multiplier (L2 blocktime vs mainnet blocktime?)

Paging @gzeoneth to help confirm and decide the best course of action. Both of these options seem ok but I'm not sure what's best for users, but I think I'd avoid a 1:1 relation between and L1 and L2 block numbers since that will likely never occur in practice

hmm, okay so the right fix for abitrum would be to not use the l1blocknumber for the fork target block number (L2)

I believe this is correct, since the RPC returns the L2 block number, we want to use L2 block numbers as the fork blocks

but unclear what we should do to evm.blocknumber once new blocks are mined, simply increase by one as well or with some multiplier (L2 blocktime vs mainnet blocktime?)

Paging @gzeoneth to help confirm and decide the best course of action. Both of these options seem ok but I'm not sure what's best for users, but I think I'd avoid a 1:1 relation between and L1 and L2 block numbers since that will likely never occur in practice

Yes, the ArbBlockNumber (L2 block number) will need to be tracked separately. I think having evm.blocknumber being static (last known value) might make the most sense here, since it technically is unrelated to L2 blocks. It should still be able to advance evm.blocknumber with vm.roll but no need to make it correlate with other activity.

Ok I see my issue was closed as a dupe, but does add some important context - this issue goes away if you do not specify ----chain-id 42161

okay, I thought about this for a bit and I think we need to track the best block separately in a dedicated variable and not using the evm.blocknumber since this is different for arb

isn't simpler to reuse storage best number e.g.
     /// Returns the current best number of the chain
     pub fn best_number(&self) -> u64 {
-        self.env.read().block.number.try_into().unwrap_or(u64::MAX)
+        self.blockchain.storage.read().best_number.try_into().unwrap_or(u64::MAX)
     }

@grandizzy hmm I think you're right,
I'm not sure what we should do with the env block number once we mined new blocks, also increase by 1?

@mattsse IMO that would be perfectly fine, the only thing to make sure is we don't get best block overwritten when mining new blocks, so line here
https://github.com/foundry-rs/foundry/blob/master/crates/anvil/src/eth/backend/mem/mod.rs#L926
should probably be
let block_number: U64 = storage.best_number.saturating_add(U64::from(1));

I will do a PR with proposed changes

Ok I see my issue was closed as a dupe, but does add some important context - this issue goes away if you do not specify ----chain-id 42161

looks like a different issue and will address in same PR - specifics of chain (like the fact that on Arbitrum block.number is the L1 block) are set based on the value of --chain-id, so when not passed chain id defaults to 31337 and Arbitrum specifics are not applied


weird random italicized words not reflected in the markdown

these are coming out of nowhere and showing like this in my docs.

Please provide reproduction or link to your repo/file

i found the solution, the $ sign is formatting to math formula therefore needs to be escaped like this $
reference: https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/writing-mathematical-expressions


Allow image specific limits, requests and annotations

Finally it works. Just need to update tag "stable".


Update http4s-core, http4s-dsl to 0.21.23

Updates

org.http4s:http4s-core
org.http4s:http4s-dsl

from 0.21.14 to 0.21.23.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch

Superseded by #266.


docs: Improve JSDoc

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #
Test URLs:

Before: https://main--aem-boilerplate--pzrq.hlx.page/
After: https://develop--aem-boilerplate--pzrq.hlx.page/

👍🏼  https://google.github.io/eng-practices/review/developer/small-cls.html


fix fixed size array deserialization

fixes #391
@erickt @dtolnay can we make this breaking change assuming that anyone relying on this behaviour already wrote buggy code?

I would prefer to fix this in bincode and schedule this PR for 0.8.0 with a big mention in the release notes.


[1.16] Infinity tools report a capacity value of -1 FE, creating Artifact-level exploit paths.

This can be most easily seen if you have Draconic Evolution installed and have access to its Energy Transfuser or Generator.
To reproduce,

Give yourself an Energy Transfuser and Generator:
give @s draconicevolution:energy_transfuser
give @s draconicevolution:generator
Give yourself an Infinity Tool without any energy.
Place the Infinity Tool in the Energy Transfuser.
Notice that its capacity is reported as -1.
Place the Infinity Tool in the Generator.
Notice that it loses 1 FE/t.
Notice that when the energy level in the Infinity tool hits 0, it decrements once more to -1 while simultaneously powering up to Artifact tier.

This is important because that exploit allows players to get Artifact-level Infinity Tools without having to invest the 9.223 quintillion FE required to reach that level.
One fix would be to have the tool report a capacity of 2**63-1 (the aforementioned 9.223 quintillion) instead of -1.
Industrial Foregoing Version: <IP_ADDRESS>
Titanium Version: <IP_ADDRESS>
Draconic Evolution Version: <IP_ADDRESS>8
Mod Pack: FTB Oceanblock 1.11.0

I made an infinity drill on my server, got it to 5x5 size before I ran out of power, went mining and then the power amount instead of stopping at 0, went into minus and now I ended up with the 13x13x13 drill. Tested twice more and it just switches as soon as it reaches minus which it shouldn't even do.


There is no specific role for Giving App access to client and Sending notification from project.

Contact Page Roles

Project Page Roles

Contact Page - App Access
User who has only edit contact role can give and revoke the access of his clients.
Project Page- Send Notification
And to send notification from project page user require either to be project manager or else need to have anyone of following roles.

Change Status
Change project members
Change Tags
Change Budget
Change Unique Id
Change Year

1. Need new role for Contact Page
2. Leave project role as it is for now.  -On behalf of vinod sir.


Ci improvements

Redoing this, was getting all kinds of weird errors because of sub-modules.
This PR implements an updated launcher script that supports multiple MCUs. Currently MAX32655 and MAX32665.
MAX32690 will be added later once I get those examples upto date, would like to get this merged before then, to fix failing DATC/S connection tests. Its been troubling for a while.

Are the Examples modifications merged from MAX32655?

Tested it , OOB works fine between ME17 and ME14

I figured it was a merge, just wanted to make sure. Not sure if we've missing anything but I'll take a closer look.

There are some pretty minor differences. We should also update the READMEs to mostly match.
three-way-merge.txt


Feature/gritas proc

Initial commits supporting processing of ods files through gritas. Markdown included, though it may need to be migrated elsewhere in time.

After you make the minor changes I suggest, I will take it in and we can talk about the to-do. This is very nice.

Colin: one more thing. earlier today I committed an update to the components.yaml file sitting on the top directory. You might want to take that update - it's in the branch you created.


could not find ditaa.jar...

When I write the ditaa example with org-babel, C-c C-c reports:
org-babel-execute:ditaa: Could not find ditaa.jar at /Users/mls/.emacs.d/.local/packages/elpa/contrib/scripts/ ditaa.jar.
It seems that there is no such problem before, and I don’t understand what it is.
After brew install ditaa, although there is /usr/local/bin/ditaa, it doesn't seem to be the ditaa.jar it wants, it doesn't work.

You need to set org-ditaa-jar-path to the location of ditaa.jar. The default location appears to be "~/.emacs.d/local/packages/elpa/contrib/scripts/ditaa.jar", which will never exist, because the contrib folder will never exist.
It hasn't been changed on Doom's end, so I don't know why it would've stopped working for you.

Yes, this is my mistake and did not attract my attention.
I really apologize.

No problem! I'm glad we could figure it out.


Add Immutable Fedora install

adding Immutable Fedora linux install command for curl and, libcurl development package

I changed the chapter title from docker to container because, it's a better title considering that other containerization tools like, Podman are being referenced

Please rebase this branch onto master and force-push


Update user-properties.md

Added line 71.

Learn Build status updates of commit a592cc4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/active-directory/external-identities/user-properties.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@msmimart
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @husousa, thank you for this suggestion. Some Power BI content was published/updated recently that addresses how to share content to external users with B2B collaboration. We might be able to address Power BI and B2B more thoroughly by adding an entry to the FAQ and pointing to the relevant articles, for example:
Can B2B collaboration users access Power BI content?
Yes, you can distribute Power BI content to external guest users using B2B collaboration. To share Power BI content across Microsoft clouds, you can use Microsoft cloud settings to establish mutual B2B collaboration between your cloud and an external cloud.
Let me know your thoughts, thanks!

Hi @husousa - I added the entry to the B2B collaboration FAQ as indicated earlier, hopefully this helps clarify the use of B2B collaboration for Power BI content, but if not, let me know. Once the change publishes, this issue will close automatically. Thanks so much for your feedback!


Fix Chapter 7 Code

[x] Fix 07.02 OpenCV NormalBayesClassifier function implementation.

[x] Remove notebook checkpoints.

Notebook checkpoints removed in commit.

Normal Bayes Classifier fixed in commit.


New user struggling with how to mock winston logger instance encapsulated in service class

Hi,
I am a new user struggling with understanding how to mock a winston.Logger instance that is encapsulated within a service class. I have included my code below.
I cannot get the mocked logger instance to be triggered from within the service class. Can anyone explain where I am going wrong?
Kind regards
dcs3spp
import * as winston from 'winston';

import { loggerOptions } from '../logger/logger.config';
import { LoggingService } from '../logger/logger.service';

const logger: winston.Logger = winston.createLogger(loggerOptions);

// trying to mock createLogger to return a specific logger instance
const winstonMock = jest.mock('winston', () => (
    {
        format: {
            colorize: jest.fn(),
            combine: jest.fn(),
            label: jest.fn(),
            timestamp: jest.fn(),
            printf: jest.fn()
        },
        createLogger: jest.fn().mockReturnValue(logger),
        transports: {
            Console: jest.fn()
        }
    })
);

describe("-- Logging Service --", () => {
    let loggerMock: winston.Logger;

    test('testing logger log function called...', () => {
        const mockCreateLogger = jest.spyOn(winston, 'createLogger');
        const loggingService: LoggingService = LoggingService.Instance;
        loggerMock = mockCreateLogger.mock.instances[0];
        expect(loggingService).toBeInstanceOf(LoggingService)
        expect(loggingService).toBeDefined();
        expect(mockCreateLogger).toHaveBeenCalled()

        // spy on the winston.Logger instance within this test and check
        // that it is called - this is working from within the test method
        const logDebugMock = jest.spyOn(loggerMock, 'log');
        loggerMock.log('debug','test log debug');
        expect(logDebugMock).toHaveBeenCalled();

        // now try and invoke the logger instance indirectly through the service class
        // check that loggerMock is called a second time - this fails, only called once
        // from the preceding lines in this test
        loggingService.debug('debug message');
        expect(logDebugMock).toHaveBeenCalledTimes(2);
    });

   ...

LoggingService debug method code
public debug(message: string) {
		this.logger.log(
			{
				level: types.LogLevel.DEBUG,
				message: message,
				meta: {
					context: this.contextName
				}
			}
		);
	}

Update: 3/09/2019
Refactored my LoggingService to dependency inject winston logger instance in constructor to facilitate unit testing. This enables me to use jest.spyOn on the winston logger's log method and check that it has been called within the service instance:
// create winstonLoggerInstance here, e.g. in beforeEach()....
const winstonLoggerMock = jest.spyOn(winstonLoggerInstance, 'log');
serviceInstance.debug('debug sent from test');
expect(winstonLoggerMock).toHaveBeenCalled();


Add more constellations

Codecov Report

Merging #467 (501c43c) into main (e5ec9e6) will decrease coverage by 0.09%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #467      +/-   ##
============================================
- Coverage     93.03%   92.94%   -0.10%
+ Complexity     2101     2099       -2
============================================
  Files           233      233
  Lines          4381     4381
  Branches        450      450
============================================
- Hits           4076     4072       -4
- Misses          186      188       +2
- Partials        119      121       +2

Impacted Files
Coverage Δ

src/main/java/net/datafaker/service/FakerIDN.java
63.63% <0.00%> (-9.10%)
:arrow_down:

...ker/idnumbers/pt/br/IdNumberGeneratorPtBrUtil.java
92.59% <0.00%> (-3.71%)
:arrow_down:

.../java/net/datafaker/service/FakeValuesService.java
87.09% <0.00%> (-0.22%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


compute the diagonal of the true Hessian

I added a field h to SensitiveFloat (defined in CelesteTypes.jl). It has the same dimension as the first derivative d. We can store the 2nd derivatives (i.e. the diagonal of the Hessian) there.
For testing, I suggest modifying test/test_derivs.jl to also test these new 2nd derivatives. We can either extend test_by_finite_differences to also test second derivatives, or make a new function, e.g. test_second_deriv_by_finite_differences. In either case, we'd want to use GSL.deriv_central again (as we've done in test_by_finite_differences), but this time on the derivative values (the SensitiveFloat.d values, that is, which we've already verified) rather than the objective.

I'm assigning myself this for the task of hard-coding the actual Hessian, not just the diagonal.

Just curious, Jeff, do you have thoughts about what kind of data structure would be best for the Hessian object inside the SensitiveFloat?  You know much more about the performance of different julia data types than I do.
I'm planning to treat the Hessian of the brightness (G in the derivations) differently from the Hessian of the ELBO, because the Hessian of G is only nonzero within a source.  Also, we'll pretty much only write to the full Hessian once, whereas the G Hessians will need to be read again and again as we perform transforms on SensitiveFloats.
I'm currently just treating the G Hessians as length(ids) x length(ids)matrices.  Most of the entries in these matrices are zeros most of the time.  But I'm not sure if the lookup costs (and complexity) that would be associated with sparse matrices or some other custom data structure would be worth it.
Better to get your thoughts sooner rather than later!

How about H::Vector{Matrix{Float64}}? The outer vector has one entry per
source. Each inner matrix is roughly 30x30, a dense matrix that stores the
block of the Hessian corresponding to a particular source.
An alternative is Array{Float64, 3}, but this is worse because until Julia
0.5 H[:,:,s] copies rather than returns a view without copying.
Another alternative is a sparse matrix. That's no good either since blocks
of it are dense, and we only want to compute the block diagonal anyway, I
think.
On Mon, Nov 16, 2015 at 3:35 PM Ryan<EMAIL_ADDRESS>wrote:

Just curious, Jeff, do you have thoughts about what kind of data structure
would be best for the Hessian object inside the SensitiveFloat? You know
much more about the performance of different julia data types than I do.
I'm planning to treat the Hessian of the brightness (G in the derivations)
differently from the Hessian of the ELBO, because the Hessian of G is only
nonzero within a source. Also, we'll pretty much only write to the full
Hessian once, whereas the G Hessians will need to be read again and again
as we perform transforms on SensitiveFloats.
I'm currently just treating the G Hessians as length(ids) x length(ids)matrices.
Most of the entries in these matrices are zeros most of the time. But I'm
not sure if the lookup costs (and complexity) that would be associated with
sparse matrices or some other custom data structure would be worth it.
Better to get your thoughts sooner rather than later!
—
Reply to this email directly or view it on GitHub
https://github.com/jeff-regier/Celeste.jl/issues/15#issuecomment-157209423
.

Cool, good to know.  I had been using Array{Float64, 3} but I didn't realize that problem with copying.  I'll do Vector{Matrix{Float64}}.
Could I also ask you to comment up the parameter stuff starting around line 400 of CelesteTypes.jl?  I'm considering writing generic tools for taking functions of SensitiveFloats.    (Sorry, I think it'll be quick for you to add a few comments and slow for me to try to figure out on my own what's going on in there.  My julia meta programming chops are not so good.)

Hey Ryan, I'll add some comments but I can't get to it today. That block of
metaprogramming didn't turn out to be a slick as I'd hoped: you should feel
free to modify/delete it as you see fit. My aim was to use the type system
in Julia to control how different types of derivatives related to each
other, without any run time penalty, but now I think there must have been a
better way
On Mon, Nov 16, 2015 at 3:54 PM Ryan<EMAIL_ADDRESS>wrote:

Cool, good to know. I had been using Array{Float64, 3} but I didn't
realize that problem with copying. I'll do Vector{Matrix{Float64}}.
Could I also ask you to comment up the parameter stuff starting around
line 400 of CelesteTypes.jl? I'm considering writing generic tools for
taking functions of SensitiveFloats.
(Sorry, I think it'll be quick for you to add a few comments and slow for
me to try to figure out on my own what's going on in there. My julia meta
programming chops are not so good.)
—
Reply to this email directly or view it on GitHub
https://github.com/jeff-regier/Celeste.jl/issues/15#issuecomment-157212465
.

Done in 33ae77af052d9cf7cc6134a1340e4dd53000e453.
whew


How to use WaitForElement instead of delay?

how to use waitForElement present instead of delay or sleep?

If you want to ensure that a UI element is present and visible on the page, you can use the AssertElementVisible keyword. However, most of the times you don't even need to use that. For example, if you want to click an element which is dynamically created on the page and takes a couple of seconds to arrive, you don't need to do anything other than just calling the Click keyword, since the wait logic is already embedded into it (as well as all other Selenium keywords) and this keyword will automatically wait for up to 10 seconds for UI elements to arrive on the page. So 99% of the times, you will not need any delays or asserts, and you can simply perform the action that you need (Click, SendKeys, etc.)


Ux testing build

Describe the Issue
Would you like to work on the issue?

@shelaghm
Prefix is missing from given questionnaire form, I am adding number prefix to every question item. e.g 1. 2. 3. and so on.

Prefix is missing from given questionnaire form.
I am adding number prefix to every question item.
e.g 1.  2.  3.  ...

Sdc module is not able to render the instructions from shared questionnaire form.
It is required to edit the shared questionnaire form to update the instructions as per instructions code.
Attaching screenshots for reference

For Q.2 and Q.3 app get crashed, as these question has type quantity and uses min and max extension, where it throws exception java.lang.IllegalArgumentException: Cannot compare different data types: Quantity and integer         at com.google.android.fhir.MoreTypesKt.compareTo(MoreTypes.kt:49)
Integer type is updated for Q.2, Q.3

Q5(What city does the client live in?) to use the "Autocomplete" widget
Working on it.

Q11 to be the "slider" widget.
Working on it.

@shelaghm
Would you like to add instructions to instruct the user to input the value in the range of 1900 -2022 for question "What year were they born?"

@shelaghm
Would you like to modify below instructions

Type the name “Orange Flower”  => Type the name e.g “Orange Flower”
Type the phone number “98 87 76 65 => Type the phone number e.g “98 87 76 65"
Select 4 May 2020 => e.g 4 May 2020

Q. 9 "Does the client have any of the following symptoms?" is not rendered as open choice view.
=> modifying extension in questionnaire json.

@shelaghm Would you like to modify below instructions

Type the name “Orange Flower”  => Type the name e.g “Orange Flower”
Type the phone number “98 87 76 65 => Type the phone number e.g “98 87 76 65"
Select 4 May 2020 => e.g 4 May 2020

@santosh-pingle leave these instructions as is. For this specific usability test we want participants to enter these specific names and numbers, so they don't accidentally enter anyone's real names or phone number

@santosh-pingle can you also make it a paginated questionnaire? Thank you!

@santosh-pingle , is this issue only for testing or does it include any development as well?

@Tarun-Bhardwaj
Questionnaire json changes are only for ux testing build, these changes will not be part of sdc code repository.

Enter a year between 1900 and 2022

@shelaghm
I will make changes to support the paginated layout as well.

@shelaghm
shared ux testing build which supports questionnaire in default and pagination layout.

amazing, thanks so much @santosh-pingle

Ux testing build is delivered, so closing the ticket.


Question about the details of codebook

Hi~ Thanks for sharing the source code.
I noticed you sample from a Bernoulli distribution to generate the codebook, but seems the codebook is not updated along the training.
Would it be better to update it? Or is it unnecessary to do so? Do you have any theoretically proof on this part?
And also I noticed you emphasize orthogonal several times in your paper, does that mean as long as the orthogonality is ensured the updating of codebook is not needed?

Hi @speedcell4 ,
the codebook is a fixed codebook and will not be updated during training. We have done a version where we update the codebook before the training, that is running with --codebook-method O, this will maximize the Hamming distance between every class, we noticed a consistent improvement over all datasets and number of bits. See "D.3 Orthogonal Targets Generation" in the supplementary material for the experiment results.
At the moment we think it is already good by fixing the codebook, it is because when the training loss is minimized to 0, the learned codes have a maximum average hamming distance between every class (i.e., distance = #bits / 2), hence should be optimal. And this can be achieved because the codes in the codebook between each other are orthogonal (cosine similarity = 0 = #bits / 2).

Thank~


General Maintenance / JDK 22

Summary
General cleanup and maintenance, including version bumps of build-time stuff like GHA actions; also includes support for the latest JDK release, version 22.
Included PRs:

#343
#342
#341
#340
#339
#335

Changelog

chore: update bzlmod lock after apple_support update
chore: update all npm deps
chore: cleanup older python-based version-gen script
chore: update ci actions
chore: apply ci hardening
chore: support for latest gvm release

@fmeum This PR looks large, but I promise most of that is just removal of the Python stuff. Very basic changes ultimately, with the JDK 22 upgrade also baked in (adding the new download links and SHA256 hashes).
I really do want to stop merging stuff on this repo if it isn't reviewed by at least one other stakeholder. If you are busy no worries, I can try and pull a friend who is familiar with Starlark.

@sgammon Just noticed that the release archives are now missing the external/integration_tests/bzlmod directory, which means that the BCR checks can't run: https://github.com/bazelbuild/bazel-central-registry/pull/1909


Add build info to pytest_report_header

Motivation and Context

This adds some additional debug info the pytest output (related to how habitat-sim was built). This is done by adding a special function called pytest_report_header to the conf_test file which then prints out a list of settings on the habitat_sim object and whether they are enabled or not.

After this change, the following info is prepended to the pytest report output:
C++ Build Info:
--audio_enabled: False
--built_with_bullet: True
--cuda_enabled: False
--vhacd_enabled: False

How Has This Been Tested

Types of changes

[X] Docs change / refactoring / dependency upgrade
[X] New feature (non-breaking change which adds functionality)

Checklist

[X] My code follows the code style of this project.
[X] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[X] I have completed my CLA (see CONTRIBUTING)
[X] I have added tests to cover my changes.
[X] All new and existing tests passed.

The python_lint flakes seem to related to a flake8 upgrade.


[HBASE-24470]Add store file info when log meessages in HStore.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 31s
Docker mode activated.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 12s
the patch passed

+1 :green_heart:
compile
1m  6s
the patch passed

+1 :green_heart:
javac
1m  6s
the patch passed

+1 :green_heart:
shadedjars
6m 26s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 51s
hbase-server in the patch failed.

_ Other Tests _

-1 :x:
unit
143m 33s
hbase-server in the patch failed.

171m  5s

Subsystem
Report/Notes

Docker
Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1804/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1804

JIRA Issue
HBASE-24470

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 1ecc4b8faa52 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / fe1fc25fba

Default Java
2020-01-14

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1804/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1804/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1804/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1804/1/testReport/

Max. process+thread count
4114 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1804/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 34s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m 51s
master passed

+1 :green_heart:
compile
0m 55s
master passed

+1 :green_heart:
shadedjars
5m 31s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 38s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 28s
the patch passed

+1 :green_heart:
compile
0m 56s
the patch passed

+1 :green_heart:
javac
0m 56s
the patch passed

+1 :green_heart:
shadedjars
5m 30s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 35s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
152m 51s
hbase-server in the patch passed.

177m  0s

Subsystem
Report/Notes

Docker
Client=19.03.10 Server=19.03.10 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1804/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1804

JIRA Issue
HBASE-24470

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 804d9515cfd2 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / fe1fc25fba

Default Java
1.8.0_232

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1804/1/testReport/

Max. process+thread count
3764 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1804/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.


Prakiti smart patch 1

Added a fingerprint for "_TZ3000_zwaadvus/TS011F"

Hi PrakitiSmart
Add a fingerprint for "_TZ3000_zwaadvus/TS011F", but what profile?
the file is incomplete

I thoguht I did add two-outlet as device profile name.

Added a fingerprint for "_TZ3000_zwaadvus/TS011F" as two-outlet profile

Added to main branch


AndroidManifest.xml

Expected Behavior

Current Behavior

Context

https://firebase.google.com/docs/hosting?hl=es-419

Expected Behavior

Current Behavior

Context

project.sdk-enterprises-gracida-veracruz/overview

Expected Behavior

Current Behavior

Context

https://github.com/gradle/gradle/issues/15355


Don't read gguf header with mmap

This seems redundant.

@booxter why is this on hold?

@mergify.io refresh


[Oracle] Rowid is not used when table is opened in SQL Editor

System information:

Operating system (distribution) and version Win10
DBeaver version 21.3.4
Additional extensions

Connection specification:

Database name and version Oracle

Describe the problem you're observing:
Persisting changes to a table without primary key is possible in data editor with the usage of Oracle rowid column, but it won't be used if the table is opened in SQL Editor, the user will be prompted to create virtual keys.
Steps to reproduce, if exist:

Create a table without a PK
Opene it with a basic SELECT in SQL Editor
Update or delete a row using UI
Click Save

Expected: Changes are saved
Actual: Prompt to create virtual key appers.

Works as expected


Expression Generator: better way to random floating-point numbers

Currently, we are using a range [-10, 10] for the first summand of the addition expression, it would be better if we can revise how the number is picked.
Our concern is we have to be careful about the floating-point number as it's sensitive so we don't want to change its order of magnitude.

Closing as we are retiring the known-value shader generator.


Autosubmit bot will not merge, claiming false negatives

From https://github.com/flutter/flutter/pull/131556#issuecomment-1714756218

auto label is removed for flutter/flutter/131556, due to This PR has not met approval requirements for merging. Changes were requested by {TahaTesser}, please make the needed changes and resubmit this PR.
You are not a member of flutter-hackers and need 1 more review(s) in order to merge this PR.
Merge guidelines: You need at least one approved review if you are already part of flutter-hackers or two member reviews if you are not a flutter-hacker before re-applying the autosubmit label. Reviewers: If you left a comment approving, please use the "approve" review action instead.

The autosubmit bot was mistaken on two points here:

The bot claimed it did not have approval from 2 flutter-hackers members - but it did.
The bot claimed there was a review requesting changes in place, that review had already been dismissed, after which the PR received the necessary approvals

/cc @ricardoamador

Okay I see the issue here. This an odd corner case based on how Github reports the reviews. Looking at all members who have submitted a review; 'TahaTesser' requested changes and it was dismissed by Piinks. However github only reports that in the UI and TahaTesser's review persists as requesting changes as he did not change his review to approve.
Autosubmit is honoring all reviews and Github does not remove them from the list of reviews in the api.
Are we allowing any developer to simply dismiss someones request for changes? In this case 'TahaTesser' is a flutter-hacker so his review was not dismissed by the bot since he did not change his review to approved.

I think that flutter-hackers are only able to dismiss a review.

Correct.

In this case Autosubmit did the right thing. I don't think it matters that everyone here was a flutter-hacker. Dismissing reviews can become a slippery slope.


[integration/sql] Adding additional clarification in sql integration README

Signed-off-by: howardyoo<EMAIL_ADDRESS>Problem
The instruction mentioned in integration/sql/README.md for python language does not specify the directory the user should be in when running test and build command.
Solution
Added additional instruction to make sure user is in the proper directory.

[ ] Your change modifies the core OpenLineage model
[ ] Your change modifies one or more OpenLineage facets

If you're contributing a new integration, please specify the scope of the integration and how/where it has been tested (e.g., Apache Spark integration supports S3 and GCS filesystem operations, tested with AWS EMR).
Checklist

[x] You've signed-off your work
[x] Your pull request title follows our guidelines
[ ] Your changes are accompanied by tests (if relevant)
[x] Your change contains a small diff and is self-contained
[ ] You've updated any relevant documentation (if relevant)
[ ] You've updated the CHANGELOG.md with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary)
[ ] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)
[ ] You've added a header to source files (if relevant)

SPDX-License-Identifier: Apache-2.0
Copyright 2018-2022 contributors to the OpenLineage project

Ok, will fix it. Thanks!
On Fri, Dec 2, 2022 at 8:28 AM Maciej Obuchowski @.***>
wrote:

@.**** requested changes on this pull request.
In integration/sql/README.md
https://github.com/OpenLineage/OpenLineage/pull/1360#discussion_r1038197897
:

+Make sure you are in the correct directory.
+bash +cd iface-py +

It's right, but those lines should be after cargo test. You should run
tests from root directory.
—
Reply to this email directly, view it on GitHub
https://github.com/OpenLineage/OpenLineage/pull/1360#pullrequestreview-1202582139,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHZNLLT5G5TOADUU6ZMFKUDWLIBRZANCNFSM6AAAAAASRLLRJM
.
You are receiving this because you authored the thread.Message ID:
@.***>


fix #1229

Speaking of dangerous refactors with unclear benefits --- here's #1229!

Actually feeling pretty good about this!


gateway/runservice: add api to delete step logs

users should be able to delete step logs (i.e. if they noticed that the logs leak some private information)

Implemented in #157

@camandel. Thanks. Closing.
The cli related issue is #183, while the web related issued is https://github.com/agola-io/agola-web/issues/5


#84 fix the object name replacement issue

issue refs: https://github.com/aliyun/aliyun-oss-php-sdk/issues/84

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