[Bug]: Blocked aria-hidden on a  element

Reproduction
Blocked aria-hidden on a  element
Describe the bug
When I click on the Combobox, something like this will appear in the console
There is a similar situation in the console of https://www.shadcn-vue.com/docs/components/combobox
Blocked aria-hidden on a  element because the element that just received focus must not be hidden from assistive technology users. Avoid using aria-hidden on a focused element or its ancestor. Consider using the inert attribute instead, which will also prevent focus. For more details, see the aria-hidden section of the WAI-ARIA specification at https://w3c.github.io/aria/#aria-hidden.
System Info
window : window 11 23H2
Chrome : 128.0.6613.114

Contributes

[ ] I am willing to submit a PR to fix this issue
[ ] I am willing to submit a PR with failing tests

https://discord.com/channels/1121805548782878813/1121805549621743658/1276045976301473824

Does this issue still happening? I tried to replicate it but not seeing the error 😬

It doesn't seem to happen now.

chrome: 128.0.6613.138
https://github.com/user-attachments/assets/8fc20c80-0c5a-4940-b778-1248599faf4b

When the dialog opens, it uses useHideOthers() to mark sibling elements as aria-hidden="true", including #app, which is the ancestor of #radix-vue-tabs-v-3-content-preview. Clicking on #radix-vue-tabs-v-3-content-preview gives it focus, which conflicts with the rule: aria-hidden elements must not contain focusable elements.
A quick fix is to update useHideOthers() in useHideOthers.ts:
const rootEl = document.querySelector('#app') as HTMLElement // get root element
undo = hideOthers([el, rootEl]) // don't add aria-hidden on #app
rootEl.setAttribute('inert', true) // use inert to prevent interactive, don't forget to remove it when execute undo()

Hope this will help.

The issue is still happening as of radix-vue 1.9.6

Guys, this still hasn’t been fixed. Does anyone have instructions on how I can fix this in my project without modifying the code in node_modules?


A Rust implementation?

I noticed that there is a Rust implementation of the Postgres client, but not for the server. I think that the server would benefit from the speed a Rust implementation (possibly more than the client) could bring.
This should maybe be in the realtime repository instead.

You're right about the client rust libraries. Bobbie has already started some of this work here: https://github.com/supabase/postgrest-rs
This library will get wrapped into a supabase/supabase-rs library which provides the broader functionality (sockets, auth etc).
As for re-writing the servers, Rust is very interesting for us and maybe one day there will be a re-write! Right now we want to build fast using languages which have massive community support. Node/Js has that feature (for better or worse), and we're a small team so we need a language which anyone in the team can jump into the code at any time. The realtime server is written in elixir for its concurrency and fault tolerance, so it's a unique case in our repos.
Also check out PostgREST: https://github.com/PostgREST/postgrest/. We use this heavily in Supabase but it's not one of our own repos, just a tool which we know and love. It's written in Haskell and Supabase wouldn't exist without it


compose table widget?

Is there any table wdigets ? to show grid table datas ?

Have you tried LazyVerticalGrid?

@toplinuxsir do you have any other questions here?

@akurasov , No , thanks!

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Multiple Slider Push Mode issue

Hi
Not sure it is an issue or a misunderstood from myself, but it seems Multiple slider does not work in push mode. On the http://jsfiddle.net/Greggg/57zhz7py/1/  content is hidden by the top and left sliders. Clic on Entre Preview button to see them.
Regards
Greg

This is specified in docs as a limitation, generally the css transition system isn't equipped to handle multiple element animations at same time.

Hi
The documentation advice to use overlay and recommend to set the transition to overlay. Nothing about  a limitation !
Multiple sidebars can be displayed at the same time only when using a supported animation like push or overlay.
Do you expect to allow Push or is there any turn around?
Thanks

Here is a full CSS example on what it would be nice to do with Semantic Slider http://codepen.io/anon/pen/avNMYL

You're right, the documentation is incorrect. However i can confirm its not supported
The 3dtranslate values are determined dynamically based on your sidebar's content size. This matrix does not support automatically generation for multiple sidebars.

Hi Jack,
Do you think you will be able to find a solution in future release, or is there any chance to see multiple sliders working with push?

This is not a high priority given the other roadmapped features for the project.


Restored terminals not working correctly

This has been happening for awhile

Have some terminals open
Reload the window
The terminals are restored
When I switch to the restored terminal, the focused tab doesn't update, and the terminal is not focused. If I click in the terminal, then it gets fixed
And if I run the "Focus Terminal" command, it creates a new terminal instead of focusing the existing terminal

It doesn't happen every time but I see it fairly often.
Also this gif is showing all the terminal tab titles disappearing when the broken tab is revealed. I haven't seen that before

Related: https://github.com/microsoft/vscode/issues/137514

Actually, I think it is a duplicate of that since you're reloading not restarting (the session contents restored message threw me off).


Switch to Jakarta by default

In 2.0 we will switch to Jakarta in the main code base and generate Java EE JPA artifacts instead.

Even though i see this is older, it now becomes at least more relevant since

spring 6/ spring boot 3 is using jakarta
AFAIU hibernate 6+ also uses jakarte (but you know better for sure)

I'am not sure what the effort here is, it seems like basically renaming the imports is about everything that needs to be done - so most probably this is rather a devop effort + of course you would need to duplicate code for that (somehow, right?)
Any plans on this in general?

Jakarta is supported through the jakarta specific artefacts:
<dependency>
    <groupId>com.blazebit</groupId>
    <artifactId>blaze-persistence-core-api-jakarta</artifactId>
    <version>${blaze-persistence.version}</version>
    <scope>compile</scope>
</dependency>
<dependency>
    <groupId>com.blazebit</groupId>
    <artifactId>blaze-persistence-core-impl-jakarta</artifactId>
    <version>${blaze-persistence.version}</version>
    <scope>runtime</scope>
</dependency>

This is done using a bytecode replacement.
The issue is about switching over to the Jakarta API's by default and instead relying on the bytecode replacement / special artefacts for the javax API's.
Pragmatically however, it might also be possible for 2.x to drop support for the javax. API's and older Hibernate versions entirely.

Not sure how this helps, but i'am also not used doing it this way, so i might do it the wrong way here.
Adding those dependencies, which you describe will replace it in the bytecode, will not fix my issue in terms of i cannot even compile due to the missing symbols.
I'am entirely unsure how this is supposed to work, since jakarta.persistence.EntityManager will not become javax.persistence.EntityManager and vice versa right? So if spring boot exposes and EM of type jakarta.persistence.EntityManager  and i feed this into BP at some point, the compiler will complain.
Could you explain how you expect this to work / how it should be used?

We transform our artifacts into jakarta variants which only contain uses of the jakarta namespace, which you can use when you append the -jakarta suffix to the artifact ids of your blaze-persistence dependencies. There is no Spring 6 and Spring Boot 3 integration yet though.

Thank you both for helping out here. IMHO we might have a short doc which describes what -jakarta packages are available, or just, that you need to basically suffix everything beside the hibernate 6.0 integration.
This would help people to migrate until this issue/FR has been moved on (moving to jakarta by default).
If it is just me and obvious to anybody else, just ignore this comment :)


Support drag and drop

@cdrake added support for drag and drop. This is great for testing problematic cases. However, it would be great if the niivue-ui could fully support this. Consider the demo niivue-ui: if you drag and drop a NIfTI image, it is loaded as the background image (good) but the ui decorations list the background image as "mni152.nii". Also, the overlay (hippo.nii) is still loaded, even if it is not registered to the background.
I would suggest the default behavior should be for drag-and-drop to load the image as the background and close overlays. A modifier key might allow the new image to be loaded as a new overlay on top of the currently loaded images.

@neurolabusc, yes I noticed this over the weekend as well. I have some plans to make the UI widgets (names, etc.) more reactive to changes like dropping a new image on the canvas. @cdrake has already included the rxjs library in NiiVue that implements a sort of pub/sub model for reacting to custom events. I'll build on that in order to keep UI state in sync with the NiiVue state.


Support all types of nodes for selinux

By default vps nodes are disabled for selinux, we should enable them it there as well.
Signed-off-by: Vasu Kulkarni<EMAIL_ADDRESS>

I'd like to see some test coverage on this :)

Tests are running here, will take some time for it to run: http://pulpito.ceph.com/vasu-2016-02-11_09:35:28-selinux-v10.0.3---basic-vps/

closing this because somehow the vps nodes seems to have issues when testing with selinux, the osd path in vm's internally comes up as a long uuid instead of /var/lib/ceph/... causing lots of unlabeled selinux issues


adjust order of side bars

Why are these changes needed?
Adjust the order of side bars to make it easier to follow.
Improve documentation.
Prepare for new release v0.2.20.
Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 12.87%. Comparing base (b4fb3c7) to head (d71afec).

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #2040       +/-   ##
===========================================
- Coverage   36.90%   12.87%   -24.04%
===========================================
  Files          66       66
  Lines        7031     7014       -17
  Branches     1538     1502       -36
===========================================
- Hits         2595      903     -1692
- Misses       4210     6098     +1888
+ Partials      226       13      -213

Flag
Coverage Δ

unittests
12.87% <100.00%> (-24.04%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@sonichi https://github.com/microsoft/autogen/pull/2040/files#diff-c2cfa3b1948c53e5e734bd126473aa55c2d09810cc7075bb488963f9639a0aa1 overwrite the formatting fix from #2041. Was this intentional? Sorry for missing this in my review.


ServiceAddress is empty

[
  {
    "Node": "consul-staging-a-1.org",
    "Address": "<IP_ADDRESS>",
    "ServiceID": "mesos-slave-staging-a-1.org:determined_bartik:5000",
    "ServiceName": "service1",
    "ServiceTags": null,
    "ServiceAddress": "",
    "ServicePort": 4003
  },
  {
    "Node": "consul-staging-a-1.org",
    "Address": "<IP_ADDRESS>",
    "ServiceID": "mesos-slave-staging-a-1.org:angry_hypatia:5000",
    "ServiceName": "service1",
    "ServiceTags": null,
    "ServiceAddress": "",
    "ServicePort": 4007
  }
]

This is what I get from querying the Consul service API (/v1/catalog/service/service1).
Commands that I used to start registrator and services:
docker run -d  -v /var/run/docker.sock:/tmp/docker.sock -h $HOSTNAME gliderlabs/registrator consul://consul-staging-a-1.org:8500

docker run -d -p 4003:5000 -e "SERVICE_NAME=service1" docker-training/hello-world
docker run -d -p 4007:5000 -e "SERVICE_NAME=service1" docker-training/hello-world

Any steps I'm doing wrong? How do you assign the hostname to ServiceAddress field?

I can confirm seeing the same behavior for the latest image using the consul catalog.

Since the v6 release (now as :latest) ServiceAddress is always set. In the future this may become optional: https://github.com/gliderlabs/registrator/issues/240
As a side note, you should be running Consul Agent on all hosts and so you should be able to use the Address field from the Consul API if you've set everything up correctly.


add a button "Deploy to heroku" on the repo

For when we release v0.1 "Quixote

Duplicate of #180 (okay, it's the other way around, but the other issue has more details)


removeInRFC problem in non-text formats

Describe the issue
When you do a <note removeInRFC="true">, it looks like the non-indented initial word/heading "Note" is ONLY present in the .txt output but is missing in the .pdf and the .html (and also in the htmlized text). See, for example, various formats of draft-ietf-nfsv4-delstid-04
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

@kesara, should this be an rfc2html issue rather than one for the datatracker?

@rjsparks yes, you are correct. I'll transfer this to rfc2html.


Change controller-tools from twiest fork to OpenShift Master team's fork.

[x] Change controller-tools vendoring from twiest's fork to OpenShift Master team's fork
[x] Update kube vendor to 1.13.4
[x] Fix up Hive to work with Kubernetes 1.13.4.
[x] Regenerate assets using OpenShift Master Team's controller-tools.
[x] Test change

/hold

@csrwng PTAL

@twiest this looks good to me. In a follow up we can add //+required to any fields that are truly required and also remove the code where we fix up the cluster deployment status to avoid the required validation errors.

/test e2e

/test e2e

@twiest the e2e failure seems legit. For whatever reason the operator doesn't seem to be starting the controller.

@csrwng ok, digging deeper. The last e2e failure didn't seem legit. It was a Terraform error creating an IAM role.

@csrwng I think this is ready now. Would you mind giving it another look please?
/hold cancel

@twiest this looks good to me. But because this is such a big change, I'd like to be in the office in case there's something we need to immediately fix on stage. I'll tag it on Tuesday.

/lgtm


authorWorks and authorBio section

link to original issue https://github.com/suwebdev/earth-covers/issues/5
Added authorBio and authorWorks files.
Do you guys have any suggestions or changes to the presentation and layout?

No suggestions to the presentation and layout yet, I think it looks fine!

The button serves not purpose in the Author Bio section, so I think we can get rid of it.


Adding Minutely forecast for Home Assistant

I have added functionality to homeassistant/core/components/openweathermap (fork: here) to create a new service with 60 sensors for each minute of the hour-long Minutely forecast.
Having that many new sensors wasn't my first choice but it seems to be the least worst option. Adding extra_state_attributes is discouraged for database growth reasons, which rules out having just one new sensor to cover the full hour with lots of attributes. Including a minutely forecast as part of OpenWeatherMapWeather also doesn't appear to be properly supported by HA's WeatherEntityFeature: only hourly and daily (and twice daily?).
On the plus side, having the sensors under a dedicated service does allow all the entities to be disabled quickly if they aren't required, every sensor may be added to a history chart with only a few clicks, and the native values are more easily accessible than attributes - which should aid with automation.
If 60 sensors is too much, it could be reduced to a sensor per 2, 3, or 5-minutes - with either an average or maximum value used.
Note that there is a dependency on the changes made in PR#13 - thank you @freekode for merging it.
Feedback welcome, particularly regarding the number of sensors.

If I understand you correctly, you are creating 60 new sensors for minute forecast.
Home Assistant rather is moving away from creating many sensors. Since current service call doesn't support minute forecast, I would suggest you to create a new one (similar to this) which will be available only for OWM integration. That way you won't have so many sensors by default and can create only required ones using template.

If it's my turn to understand correctly, the result of creating a sensor in the way you suggest would be to have a single sensor with up to 60 attributes?
The sensor documentation suggests that - done badly - it might not be good idea: "Instead of adding extra_state_attributes for a sensor entity, create an additional sensor entity. Attributes that do not change are only saved in the database once. If extra_state_attributes and the sensor value both frequently change, this can quickly increase the size of the database."
However, maybe if the sensor value doesn't change very often, having many attributes would be OK.
I also think it wouldn't be part of the Weather platform and extend homeassistant.components.weather.SingleCoordinatorWeatherEntity, since WeatherEntity seems to define specific data points that you cannot add to - unless I've missed something?
Happy to rework it into a single sensor with multiple attributes.

If it's my turn to understand correctly, the result of creating a sensor in the way you suggest would be to have a single sensor with up to 60 attributes?

Not necessary, you don't need to create a sensor at all. Users will do that by using a template where they can extract as much information as they want. All you need is to provide a service, similar to weather.get_forecasts which returns minutely forecast. Although this service should be implemented in OWM integration not in generic Weather.

Good idea. A service is much simpler and neater: https://github.com/10100011/core/tree/minutely-service/homeassistant/components/openweathermap
Please feedback.

Looks nice.
Somewhere here I think you need to check what current forecast mode selected, because minutely forecast is available only in onecall 3.0 (i guess) mode. So you want to register the service only when onecall selected.

This should do it. The service will only appear if v3.0 is selected in the Configuration.
The service will also be removed if the user selects current or forecast after previously selecting v3.0. It does not seem to remove the service if the user moves from v3.0 to v2.5, but given that v2.5 has now been completely deprecated, if a user chooses that path then they have bigger problems!

LGTM :)
Nice work


fix: consumer name contain "-"

Type of change:

[x] Bugfix
[ ] New feature provided
[ ] Improve performance
[ ] Backport patches

What this PR does / why we need it:

The schema of APISIX consumer doesn't allow the  character -, so we need to replace - to _.
APISIX schema def：https://github.com/apache/apisix/blob/master/apisix/schema_def.lua#L679
fix #752
Pre-submission checklist:

[x] Did you explain what problem does this PR solve? Or what new features have been added?
[ ] Have you added corresponding test cases?
[ ] Have you modified the corresponding document?
[x] Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

cc @gxthrj @tao12345666333


chore(template) grpc_pass now has support for variables

Summary
With latest OpenResty based on Nginx 1.7.x series we can now use variables in scheme part of grpc_pass, thus we can now remove some repetition from our templates.
There is one additional commit in this PR that removes support for 1.15.8.x series of OpenResty, because after this PR, the templates won't work anymore with the earlier version.

Delayed temporarily until after 2.2.


Added Cask for p4 version 2016.1.1411799

Adding a new cask

[x] Checked there aren’t open pull requests for the same cask.
[x] Checked there aren’t closed issues where that cask was already refused.
[x] When naming the cask, followed the token reference.
[x] Commit message includes cask’s name.
[x] brew cask audit --download p4 is error-free.
[x] brew cask style --fix p4 left no offenses.
[x] brew cask install p4 worked successfully.
[x] brew cask uninstall p4 worked successfully.

I am not able to verify 32 bit.

Thank you for the contribution. It needed some fixes, so they were made in d584c1dbd72e6e74d95dadc7c3b6eb6bfcddf232. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


[5.7] Extract fixtures for Session tests.

Extract fixtures for Session tests.

I'm really fine with having these in the same class.


OVH blocking zmap scanning due SYN FLOOD

hi, idk if there's any way to prevent this, but OVH blocks my vps when i start a zmap scan due a SYN FLOOD, there's any way to reduce the requests from my server? thanks

Decrease your scan rate with -r or -B


Ylee/text changes

https://github.com/WSWCWaterDataExchange/WestDAAT/issues/230 & https://github.com/WSWCWaterDataExchange/WestDAAT/issues/222 screenshot:

https://github.com/WSWCWaterDataExchange/WestDAAT/issues/226 Citation text update
https://github.com/WSWCWaterDataExchange/WestDAAT/issues/225 Disclaimer text update
https://github.com/WSWCWaterDataExchange/WestDAAT/issues/224
no url example:

url example:

@yxlee61, whenever you'd like, you're welcome to merge this to Staging. All seems alright to me. Thank you


Player stats in Menu

In the lobby menu, players now have their "games won" and "grubs killed" stats listed in addition to their playtime.
https://github.com/apetavern/sbox-grubs/assets/43252311/32fca608-1870-43b9-981f-6f3e618c3824
To accomplish this, I refactored PlayMenu's player entries into PlayerEntry.razor, which holds player data + cached stats (rather than fetching everytime we swap stats). Currently, the interval between stats swapping is 15 seconds (for sake of demonstration, the video has 5 second intervals).

Refactored how stats are defined and stored in PlayerEntry, as well as the timer/stat rotation functionality.
I think my only gripe with this system now is that we rely on a static index (_statsIndex), which by itself I think is fine, however, this means that PlayerEntry.NextStat() needs to know the length of the (instanced) Stats container to wrap around to index 0. To circumvent this, I have a static int _statsLength that gets populated in OnAfterTreeRender(), but this feels gross to me. The reason the StatInfo container (_stats) is instanced instead of one static definition is that we have unique values per player entry, meaning with a static StatInfo container we'd need an extra container to hold each value + reaching all around to bring the two together.
Any ideas?


Add check to ensure only s2n_pkeys of the same s2n_pkey_type are compared.

Resolved issues:
Description of changes:
Adds a check in s2n_pkey.s2n_pkey_match to first validate that the public and private key being compared are of the same key type.
Call-outs:
This check will not allow RSS PSS keys to be compared to RSS keys.
Testing:
Added a unit test.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #2074 into master will increase coverage by 23.41%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           master    #2074       +/-   ##
===========================================
+ Coverage   57.02%   80.43%   +23.41%
===========================================
  Files         155      233       +78
  Lines       11597    17623     +6026
===========================================
+ Hits         6613    14175     +7562
+ Misses       4984     3448     -1536

LGTM - simple and well tested.

I did think of a slightly simpler way to do it, instead of storing the key type and comparing that, I could just compare the match function, like this:
S2N_ERROR_IF(pub_key->match != priv_key->match, S2N_ERR_KEY_MISMATCH);
What do you think?

LGTM - simple and well tested.

I did think of a slightly simpler way to do it, instead of storing the key type and comparing that, I could just compare the match function, like this:
S2N_ERROR_IF(pub_key->match != priv_key->match, S2N_ERR_KEY_MISMATCH);
What do you think?

That's next level simplicity :)  I like it :)


.MLCodec_rans|.MLCodec_CXX could not be resolved.

Thank you for your greak work!
When I was working on the released DCVC-HEM code,  I found some errors saying   'Import ".MLCodec_rans"|".MLCodec_CXX"  could not be resolved' in the entropy_models.py file and I didn't found any clue about these two folders/files/libraries. So could you guys help me with this?

Please follow the following link to build the C++ code.
https://github.com/microsoft/DCVC/tree/main/DCVC-HEM#build-the-project

@ustclibin thanks for you helpful reply firstly. However, although I have built the C++ code following the doc,  the editor still report the error.

Did you build the C++ code successfully? Are there any binary output after building C++ code?

@ustclibin can not appreciate your help anymore, I found why the error occurred. Now it works.

@ustclibin can not appreciate your help anymore, I found why the error occurred. Now it works.

I have the same problem as you, although I have built the C++ code, the editor still report the error. Can you tell me how you solved it?

Compiling reports no error on windows, yet on my linux😢. So.. just transfer into windows🤦♂

@MagicS0ng It works, thank you!


Fix tfsec AWS security warnings

This changes fixes the following TFSec warnings.
https://aquasecurity.github.io/tfsec/v1.8.0/checks/aws/autoscaling/enforce-http-token-imds/
https://aquasecurity.github.io/tfsec/v1.8.0/checks/aws/cloudwatch/log-group-customer-key/
The first one has to be verified that it actually works with IRSA.

Won't do. AWS support has been removed.


Crop down API

API endpoints for creating/reading persons, awards, tasks, organizations, and airports are not being used at all. Maybe it's time to remove them.

@fmichonneau can you please describe which parts of API you're using?

At this time, I need the "persons" endpoint (only reading).  I'm working on a way to automate the workshop website creation, and need to validate (through the GitHub username) that the user is in AMY. I don't anticipate needing the other endpoints.

@pbanaszkiewicz are you basing this on logs of access to these endpoints? We've got a plan afoot to use a periodically updated cache of AMY data to drive our instructors page (read-only).  The capabilities I need are mostly contained in the output of api/v1/persons right now.

@jduckles:

@pbanaszkiewicz are you basing this on logs of access to these endpoints?

No, but I know that some endpoints are linked with a feature that will be removed (e.g. user-todos); some other endpoints (like person-list, airport-list, event-list, etc.) were created with JS frontend in mind, but this is not going to happen (soon). If you and @fmichonneau have ideas how you can use these endpoints, then there's no point in removing them.

I'm deferring this for now.


Clear travis

Should revert previous commit about travis. This is the clear one

Which PR will add the travis config?
this PR ?? or this #4  or  this PR #3
Please close the other similar PR please.


The apps array in user profile is not getting populated

It seems like the new authentication doesn’t create entries in apps array in user profile.
Please see below the code I am using for authentication:
const authOptions = {
          userSession: new UserSession({
              appConfig: new AppConfig(['store_write', 'publish_data'], window.location.origin)
          }),
          appDetails: {
            name: 'xordrive',
            icon: iconPath
          },
          sendToSignIn: signIn,
          finished: ({ userSession }) => {
            // set user session
          }
        }
showBlockstackConnect(authOptions)

@muneebm I've pushed a fix to the authenticator that should fix this - there was an issue with how we were handling scopes. We had the logic setup properly to update the apps array, but there was an issue with how we handled scopes in a certain part of the flow. Please try again and let me know if it's still an issue.

That fixed it! Thanks @hstove


Do not send old notifications if proxy sends no revision

Motivation
SM always sends all existing notifications and the proxy applies them even when doing full resync.
Approach
If the proxy connects without last_notification_revision query parameter, SM does not send old notifications, but only new ones.
Pull Request status

[x] Initial implementation
[x] Unit tests

Coverage decreased (-0.3%) to 88.932% when pulling cdbdbf48956a78aaaf338c9fb5a6565cfb23d179 on no-notification-revision into e27bfbe019a1a4ba49c721273e6653147323ffec on master.


Parameterized test for TestHoodieClient 2) Added TestUtils for MOR readClient changes

@n3nash - Looks good. I am going to merge this after cutting a release soon.

@n3nash - Can you resolve conflicts?

Fixed merge conflicts and added fix for https://github.com/uber/hoodie/issues/192 in this PR since it is related to MOR tests. Closed https://github.com/uber/hoodie/pull/194.


Add binary field

This PR:

adds field.Binary - to store binary data in []byte and display it HEX
adds 0 padder to DE3 (Processing Code)

Codecov Report

Merging #98 (c1ab4e9) into master (5ff58a9) will decrease coverage by 0.52%.
The diff coverage is 55.55%.

@@            Coverage Diff             @@
##           master      #98      +/-   ##
==========================================
- Coverage   69.17%   68.64%   -0.53%
==========================================
  Files          24       27       +3
  Lines         824      925     +101
==========================================
+ Hits          570      635      +65
- Misses        201      225      +24
- Partials       53       65      +12

Impacted Files
Coverage Δ

field/binary.go
55.55% <55.55%> (ø)

field/composite.go
84.44% <0.00%> (ø)

encoding/ebcdic.go
100.00% <0.00%> (ø)

prefix/ebcdic.go
66.66% <0.00%> (ø)

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 5ff58a9...c1ab4e9. Read the comment docs.


now invalidating last note data if restart during pause happened

[2023-04-05 22:18:26.437 WRN] RS2SB :: Song restarted! songTimer=8,026 | lastSongTimer=10,298


Wandb local downgrade

Hi, I'm Jerry in Korea.
I'm using wandb/local on my local mini pc.
When I updated wandb/local 0.9.40 to 0.9.49, I got some error like "Error: Could not save view." on page.
So, I tried to downgrade wandb/local 0.9.49 to 0.9.40.
But, I got a error message in docker logs.
*** Running /etc/my_init.d/00_regen_ssh_host_keys.sh...
*** Running /etc/my_init.d/01_enable-services.sh...
*** Copying services to runit
*** Copying jobber template
*** Enabling production mode
*** Running /etc/my_init.d/02_load-settings.sh...
*** Loading settings...
*** Booting runit daemon...
*** Runit started as PID 36
*** Starting wandb servers...
*** Migrating database...
panic: file does not exist

goroutine 1 [running]:
main.main()
        /mnt/ramdisk/core/services/gorilla/cmd/migrate/main.go:115 +0xb05
*** All services started
*** Access W&B at http://localhost:8080
!!! Unable to start backend, see logs at /var/logs.  Trying again in 1 minute
!!! Set the environment variable LOCAL_RESTORE=true if you're unable to login to the admin console

Have you ever encountered this error?

Hey Jerry,
Could to exec into your docker by using docker exec -it wandb-local bash and then get the output for /var/logs/gorilla.log for me? That should have some more info on what is going wrong here.
Thanks,
Ramit

Hi ramit,
I got repeated log.
{"level":"INFO","time":"2022-12-08T23:48:27.850995702Z","info":{"program":"gorilla","source":"gorilla/main.go:268","pid":15032},"data":{"config":"{Port:8081,MetadataStore:[REDACTED],MetadataStoreReplica:,MetadataStoreReplicaRate:0,HistoryStore:mysql://wandb_local:wandb_local@<IP_ADDRESS>:3306/wandb_local,FileStore:s3://MpSOR7Ug2DFo47cS5z337w==:9Ti8EEsTbF7IKB94Ohei9DUww6m5xj2IES0iWk1v81o=@<IP_ADDRESS>:9000/local-files,FileStoreIsProxied:true,FileStreamStore:{Address:mysql://wandb_local:wandb_local@<IP_ADDRESS>:3306/wandb_local MySQL:{DialTimeout:0s ReadTimeout:0s WriteTimeout:0s MaxIdleConns:0 MaxOpenConns:0 ConnMaxLifetime:0s} BucketURL:},EmailSink:sendgrid://null,EmailFromAddress:noreply@wandb.com,SweepProvider:http://<IP_ADDRESS>:8082,SettingsCache:redis://<IP_ADDRESS>:6379,Cache:noop://,Limiter:redis://<IP_ADDRESS>:6379,Locker:redis://<IP_ADDRESS>:6379,Tracer:noop://,BadgeRenderer:chart://,TaskQueue:noop://,NotificationsQueue:,BenchmarkSubmissionNotifier:noop://,DataFrameStore:noop://,TrendingRepoStore:noop://,FrontendHost:http://localhost:8080,FileHost:http://localhost:8080,StaticHost:,Dev:false,Secret:[REDACTED],FlexSecret:[REDACTED],Statsd:{Host:<IP_ADDRESS> Port:0},GCPProject:dev~wandb-local,GoogleApplicationCredentials:,StorageBucket:s3://local-files,ArtifactBucketRegions:,DefaultRegion:minio-local,SentryDSN:,SentryEnvironment:onprem-local,SendgridAPIKey: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,ServicesHost:,ServicesSecret:[REDACTED],SlackClientID:,SlackSecret:[REDACTED],StripeKey:[REDACTED],GitHubOAuthClientID:,GitHubOAuthSecret:[REDACTED],GitHubAppInstallationID:0,GitHubAppIntegrationID:0,AnalyticsSink:dummy://,AuthJWKURL:http://<IP_ADDRESS>:8083/api/jwks.json,Auth0Domain:,Auth0ClientID:,Auth0MgmtClientID:,Auth0MgmtSecret:,InsecureAllowImpersonation:false,InsecureAllowAPIKeyAdminAccess:false,License:,LicenseKeyPath:,LicenseCertPath:/var/app/jwks.json,AzureAccountKey:,Onprem:true,OnpremAPIKeyPrefix:local,MySQL:{DialTimeout:0s ReadTimeout:0s WriteTimeout:0s MaxIdleConns:0 MaxOpenConns:0 ConnMaxLifetime:0s},DefaultRateLimits:{FileStream:0 GraphQL:0},PaidRateLimits:{FileStream:0 GraphQL:0},PProfAddr:,MinCliVersion:0.8.20,MaxCliVersion:0.10.28}"},"message":"Running with config {Port:8081,MetadataStore:[REDACTED],MetadataStoreReplica:,MetadataStoreReplicaRate:0,HistoryStore:mysql://wandb_local:wandb_local@<IP_ADDRESS>:3306/wandb_local,FileStore:s3://MpSOR7Ug2DFo47cS5z337w==:9Ti8EEsTbF7IKB94Ohei9DUww6m5xj2IES0iWk1v81o=@<IP_ADDRESS>:9000/local-files,FileStoreIsProxied:true,FileStreamStore:{Address:mysql://wandb_local:wandb_local@<IP_ADDRESS>:3306/wandb_local MySQL:{DialTimeout:0s ReadTimeout:0s WriteTimeout:0s MaxIdleConns:0 MaxOpenConns:0 ConnMaxLifetime:0s} BucketURL:},EmailSink:sendgrid://null,EmailFromAddress:noreply@wandb.com,SweepProvider:http://<IP_ADDRESS>:8082,SettingsCache:redis://<IP_ADDRESS>:6379,Cache:noop://,Limiter:redis://<IP_ADDRESS>:6379,Locker:redis://<IP_ADDRESS>:6379,Tracer:noop://,BadgeRenderer:chart://,TaskQueue:noop://,NotificationsQueue:,BenchmarkSubmissionNotifier:noop://,DataFrameStore:noop://,TrendingRepoStore:noop://,FrontendHost:http://localhost:8080,FileHost:http://localhost:8080,StaticHost:,Dev:false,Secret:[REDACTED],FlexSecret:[REDACTED],Statsd:{Host:<IP_ADDRESS> Port:0},GCPProject:dev~wandb-local,GoogleApplicationCredentials:,StorageBucket:s3://local-files,ArtifactBucketRegions:,DefaultRegion:minio-local,SentryDSN:,SentryEnvironment:onprem-local,SendgridAPIKey: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,ServicesHost:,ServicesSecret:[REDACTED],SlackClientID:,SlackSecret:[REDACTED],StripeKey:[REDACTED],GitHubOAuthClientID:,GitHubOAuthSecret:[REDACTED],GitHubAppInstallationID:0,GitHubAppIntegrationID:0,AnalyticsSink:dummy://,AuthJWKURL:http://<IP_ADDRESS>:8083/api/jwks.json,Auth0Domain:,Auth0ClientID:,Auth0MgmtClientID:,Auth0MgmtSecret:,InsecureAllowImpersonation:false,InsecureAllowAPIKeyAdminAccess:false,License:,LicenseKeyPath:,LicenseCertPath:/var/app/jwks.json,AzureAccountKey:,Onprem:true,OnpremAPIKeyPrefix:local,MySQL:{DialTimeout:0s ReadTimeout:0s WriteTimeout:0s MaxIdleConns:0 MaxOpenConns:0 ConnMaxLifetime:0s},DefaultRateLimits:{FileStream:0 GraphQL:0},PaidRateLimits:{FileStream:0 GraphQL:0},PProfAddr:,MinCliVersion:0.8.20,MaxCliVersion:0.10.28}","logging.googleapis.com/trace":""}
{"level":"INFO","time":"2022-12-08T23:48:27.851137438Z","info":{"program":"gorilla","source":"gorilla/main.go:279","pid":15032},"data":{},"message":"created tracer","logging.googleapis.com/trace":""}
{"level":"INFO","time":"2022-12-08T23:48:27.851151067Z","info":{"program":"gorilla","source":"gorilla/main.go:288","pid":15032},"data":{},"message":"initialized sentry","logging.googleapis.com/trace":""}
{"level":"INFO","time":"2022-12-08T23:48:27.851286766Z","info":{"program":"gorilla","source":"gorilla/main.go:300","pid":15032},"data":{},"message":"read schema","logging.googleapis.com/trace":""}
panic: github.com/wandb/core/services/gorilla/mysql.(*metadataStore).prepareFetchRunQueueItemWithLock-fm: Error 1054: Unknown column 'r.resulting_run_id' in 'field list'

goroutine 1 [running]:
main.main()
        /mnt/ramdisk/core/services/gorilla/cmd/gorilla/main.go:307 +0x5c65

Looks like the database was not correctly migrated when you upgraded. Let's start with getting you back down to 0.9.40, could you run the following set of commands?
docker pull wandb/local:0.9.40
docker stop wandb-local
docker run --rm -d -v wandb:/vol -p 8080:8080 --name wandb-local wandb/local:0.9.40

Once we can get you back to a place where your server is running fine, the recommended course of action would be to back up your database and then try to upgrade incrementally to make sure your database upgrades correctly as well.

Thank you for answering.
but, I already run that commands.
So, I got this error on 0.9.40 version.
https://github.com/wandb/local/issues/99#issue-1481201538

We don't officially support downgrading, you'll need to upgrade the instance.  Version 0.9.40 is over a year old.  The current version is 0.25.0.  I would atleast upgrade to 0.11.0 from May of this year, but you should get up to 0.25.0 to ensure you're running an up to date and secure version of the server.

@vanpelt
Thank you, Chris.
So, I have another question.
After closing this issue, Could I open a new issue?

Sure

Please, close this issue


Find way to test e-mails on development server

Right now, we're using letter_opener to hijack e-mails when running Reservations locally in development, which is great. Unforunately, despite the fact that the app runs in the development environment on the dev server, there's no way to see if e-mails are working. This should be fixed so we can test e-mails before deploying to production.

In retrospect, letter_opener is probably as good as we're going to get with this. Figuring out a Yale-specific "live" testing solution for e-mails is definitely beyond scope for us. Closing.


WIP: Documentation

Closes: #8

Codecov Report

Merging #35 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #35   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           6      6
  Lines         455    433   -22
=====================================
- Hits          455    433   -22

Impacted Files
Coverage Δ

ldap2pg/script.py
100% <0%> (ø)
:arrow_up:

ldap2pg/config.py
100% <0%> (ø)
: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 5323219...5dbdf91. Read the comment docs.


[Feature Request]:  Light/Dark Mode Toggle Button

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
This update introduces a new feature that allows users to switch between light and dark modes seamlessly. The toggle button enhances user experience by providing a visually comfortable environment suitable for different lighting conditions.
Use Case
The light/dark mode toggle button enhances user experience by allowing individuals to choose their preferred visual interface, reducing eye strain in low-light conditions and catering to personal preferences. This feature improves accessibility for users with visual impairments and adheres to standards, ensuring a wider audience can use the application. By saving user preferences in local storage, the application maintains consistency across sessions, fostering personalization and increasing user retention. The modern aesthetic of dark mode aligns with current design trends, making the project visually appealing and adaptable to various environments. Overall, this feature promotes user satisfaction and encourages continued engagement with the application.
Benefits
The light/dark mode toggle button brings numerous benefits to the project and its community by enhancing user experience and accessibility. Users can customize their interface to reduce eye strain in varying lighting conditions, promoting longer usage without discomfort. This personalization fosters a sense of ownership, leading to higher user retention and continued engagement. Additionally, the feature accommodates users with visual impairments, making the application more inclusive. By aligning with modern design trends, it keeps the project visually appealing and relevant, while also encouraging community feedback and collaboration for further enhancements. Overall, this feature significantly improves usability and community involvement.
Priority
High
Record

[X] I agree to follow this project's Code of Conduct
[ ] I'm a WOB contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

assigned to you @Priyanshu282003
Let me know if you need any help

Can you assign me this issue?

hey @4darsh-Dev can i work on this issue?

This feature has been already added in the website,

Hey @Priyanshu282003 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


[WIP] Add support for timsTOF data to MetaMorpheus

This PR is downstream of a related mzLib PR. It adds support for searching timsTOF data in MM. Calibration/quant is not currently supported

Currently, everything fails because I reference an mzLib version that doesn't exist outside of my local machine


Update numpy version requirement to make compatible with clarite

Codecov Report

Merging #21 (d9e236d) into master (968af39) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #21   +/-   ##
=======================================
  Coverage   89.78%   89.78%
=======================================
  Files          35       35
  Lines        1616     1616
=======================================
  Hits         1451     1451
  Misses        165      165

Flag
Coverage Δ

unittests
89.78% <ø> (ø)

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

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 968af39...d9e236d. Read the comment docs.


🌐 Language request: Slovenian

🌐 Language name (native)
Slovenščina
🌐 Language code
sl-SI

Added. Thanks for your contribution :)


Update schema_registry.go

When calling to retrieve the schemaInfo the library github.com/riferrei/srclient v0.5.4  calls this code which calls to Rlock, creating a performance problem  in the library, the following code is a workaround to avoid calling to that cache always
// RLock locks rw for reading.
//
// It should not be used for recursive read locking; a blocked Lock
// call excludes new readers from acquiring the lock. See the
// documentation on the RWMutex type
func (client *SchemaRegistryClient) getVersion(subject string, version string) (*Schema, error) {

	if client.getCachingEnabled() {
		cacheKey := cacheKey(subject, version)
		client.subjectSchemaCacheLock.RLock()
		cachedResult := client.subjectSchemaCache[cacheKey]
		client.subjectSchemaCacheLock.RUnlock()
		if cachedResult != nil {
			return cachedResult, nil
		}
	}

reviewed and accepted changes!

@oscar067
After merging #149, I found conflicts here. I fetched the changes to solve the conflicts, but there are too many changes. So, I created #151 to address the issue while considering what @riferrei suggested.

I'll close this in favor of #151.


Support bootstrapping a pixi config from environment.yml

I'd like to use pixi on some projects that already use Conda and have an environment.yml file.
Would be nice to be able to pixi add -f environment.yml or some other way to bootstrap a pixi.toml from an environment.yml.

That is a good idea! Will take a look at that!

My current workaround
yq < environment.yml -r '.dependencies' -o tsv | tr '\t' '\0' | xargs -0 pixi add

pixi import would be a nice alias for this (at least I would have expected it there)


Image size for @1X and @2X considerably larger than the orignal @3x

i have an image that i exported from photoshop as png at 414bytes but the 1 x and 2x are 1k bytes and 2kbytes respectively. i would expect them to be 104 and 207 bytes.

This is due to the compression format of PNG, this will help https://imageoptim.com/


flag_value paramters are cast with the wrong type

Since click 8.0 passing flag_value enum value , it is converting it to string which breaks the CLI.  (7.x works fine)
For example
@unique
class Operation(Enum):
    CREATE = "create"
    UPDATE = "update"

@click.option(
    "--update",
    "operation",
    flag_value=Operation.UPDATE
)
@click.option(
    "--create",
    "operation",
    flag_value=Operation.CREATE
)
def my_cli(operation)
   if operation == Operation.CREATE:
       print("create")

 if operation == Operation.UPDATE:
       print("UPDATE")

if are run it with cli --create
Expected behavior
operation is Operation.CREATE of type Operation and the cli prints "create"
Actual behavior
operation is a string with value "Operation.CREATE"  and the cli prints nothing
Environment:

Python version: 3.7+
Click version: 8.0

The default param type for types that Click does not have special handling for is str. Click does not have special handling for Enum. The processing pipeline was updated to apply type casting to all data sources consistently. Either use type=click.UNPROCESSED or write a custom type to handle enum members.


Breaks with 7.0.0-beta.3

I just past to the last beta and I get the following error:
Module build failed: TypeError: Cannot read property 'imports' of undefined
    at isImportLocalName (node_modules/babel-plugin-react-intl-auto/lib/index.js:58:40)
    at isValidate (node_modules/babel-plugin-react-intl-auto/lib/index.js:109:8)
    at PluginPass.CallExpression (node_modules/babel-plugin-react-intl-auto/lib/index.js:12:14)

Any idea on this ?

Thank you for your issue @hlehmann .
Sorry about that.
I have not made babel7 correspondence yet.
I will work on it as soon as babel-plugin-tester supports babel 7.

@akameco any update on this ?

@hlehmann
I publish new version👍
Support babel7.
Please install the latest version.

Works great thanks


Fix mozilla-download options changed in travis-ci config

This pr tweaks the .travis-ci.yml to fix failed builds related to changes in the mozilla-download command line options syntax.
Build passed: https://travis-ci.org/rpl/firebug.next/builds/57773104

Ah, this was the problem, perfect!
Honza


fix systems filter and sortedby

This PR aims to solve issue #320
Changes to the backend:
The original cursor sort string f"system_info.results.overall.{sort_field}.value" cannot work for cursor.sort() because the structure of system_info.results.overall is array[array[Performance]] instead of a dictionary. Move the sort function to backend/src/impl/db_utils/system_db_utils.py.
Changes to the frontend:

Change Created At to Sorted By for more informative labels, and add tooltip "Sorted by creation time by default".
Reset Sorted By when the filter task changes because the metrics in the sorting dropdown are changed with the task.

Sure! (and thanks @lyuyangh)


Add Endpoint.ipAddr

Motivation:
Currently, Endpoint.host() is used as a hostname or an IP address of
an endpoint. This does not work well for the Endpoints whose IP
addresses are known already. For example, let's say we want to use
DnsServiceEndpointGroup with the following SRV records:
_https._tcp.foo.com. 3600 IN SRV 10 100 443 one.foo.com.
_https._tcp.foo.com. 3600 IN SRV 10 100 443 two.foo.com.
_https._tcp.foo.com. 3600 IN SRV 10 100 443 three.foo.com.

Because the client will have to create a TLS connection with proper SNI
settings, the Endpoint should provide foo.com as the host name and
provide <the IP address of {one,two,three}.foo.com> as the IP address
of foo.com, which is not possible with our current API.
Modifications:

Add a new property ipAddr to Endpoint
Add Endpoint.withIpAddr()
Handle the case where a host name in an Endpoint is an IP address
Add test cases
Compare authority, IP address and port only

Take port and ipAddr into account when comparing.
Do not take weight into account when comparing.

Add host name validation using Guava InternetDomainName
Fix incorrect uses of Endpoint.of() in some test cases
Deprecate Endpoint.of(host, port, weight) in favor of
Endpoint.of(host, port).withWeight(weight)

Result:

Fixes #1087
A user can create an Endpoint with an IP address.
Endpoint.equals() makes more sense.

/cc @anuraaga @imasahiro

Thanks, reviewers!


Optimize Loop in Unify

This PR:

Refactors _merge_dicts and _replace_graph_df_nodes by removing the functions and putting the code inline.
Optimizes the double for loop that merges old_to_new and temp_dict from O(n^2) to O(n). This was an oversight during original implementation.

The performance improvements of this PR are documented in #170. #170 avoids needing to merge dictionaries as it only calls concat_thickets with 2 Thickets, however using the concat_thickets function with 3 or more Thickets will still benefit from this PR.
This PR is better for fewer files with many nodes, while #170 is better for many files since it avoids merging old_to_new and temp_dict.

@pearce8 this PR is ready for your review


fix for the jira issue https://github.com/ballerinalang/ballerina/issues/1800

…ues/1800

fix for the issue #1800

I think we need to revisit how the worker scopes are implemented.
In the current implementation since workers resides in the package scope, even two workers with same name placed inside two different functions cannot co-exists.


Simpler getting started

Deku is a great package but its learning curve for those new to build systems is harsh. I think the project would get better adoption if there was a simple two or three lines that a new user could run and be presented with a working deku sample page containing client-side jsx.

Unfortunately deku doesn't support client-side JSX, as far as I know.

I've updated the documentation for v2. You can find it here. I'm always open to any suggestions for improving it. The docs are just in the repo, so feel free to open a PR also.

good stuff!

404

https://dekujs.github.io/deku


Storage breaks when calling storage.clearStore() on empty storage

Using React Native 0.73.
I noticed that when calling authStorage.clearStore() twice makes it behave weirdly and not accept any values anymore. I do get the following warning when trying to set a value on that storage.
Trying to set a object value to hook for type boolean is not allowed.
The storage was initialized like that.
export const authStorage = new MMKVLoader().withEncryption().withInstanceID('auth').initialize()

After clearing the storage two times, I try to set it's value like this
const authInfoTmp = {
  id: 'abc',
  phone: {
    countryCode: '+1',
    number: '12345',
  },
  role: 'user',
} satisfies AuthInfo
setAuthInfo(authInfoTmp)

Then I do get the warning mentioned above and setting doesn't work. What is really interesting, the storages value is undefined when being queried like that:
const [authInfo] = useMMKVStorage<AuthInfo>('auth', authStorage)

When force closing the App and reopening it, the storage has the value true and still won't accept new values until cleared again (once). Then it works normally again.
I wonder if this is an underlying issue or just a user issue. I'm happy to help investigating this issue further if needed.
Thank you in advance!!

I don't think this is related to react-native-mmkv. Try calling the clearAllKeys() function yourself


1.a - Use in-memory SQLite database for testing

As a developer, I want to use an in-memory database for testing, so we don't have direct dependency for a certain database.
Configure integration test to use in-memory database
Acceptance criteria:

Integration tests now use in-memory database
Tests no longer depend on a live database

We use in-memory for all unit-test, but because we dont have any integration test (for now), we dont meet this acceptance criteria yet.


On agent init, re-join on existing cluster networks

When worker leaves and re-join the cluster, make sure it re-joins the gossip for the existing cluster networks.
Related: https://github.com/docker/docker/issues/24463
Signed-off-by: Alessandro Boch<EMAIL_ADDRESS>

LGTM


[Visualizer] Markdown lists are not rendered

Platform

JavaScript

Author or host
Developer
Version of SDK
master
Issue
The visualizer doesn't render the lists as shown in the image below. All the other renderers are actually able to render the lists (even when the behavior is not 100% consistent between them all)

The problem is that the visualizer uses CSS styles that interfere with bulleted lists. The offending style is declared in conceptual.css and sets the padding for bulleted lists to 0, which has the effect of removing the bullets:

So, should we close this one?

No, it should be fixed. @matthidinger?

Indeed this is fixed on the upcoming build of the site. Feel free to close this if ya like

Fixed in CI. Will show up on next deployment.


Specify new language

I want to add support for a new language but the generators are tangled up.
For example there is code referencing each language generator in the generic generator class.
I don't think this scalable are you going to make it easier to add new languages.

+1 a generic generator with language specific components kept separate would be a great benefit

Even an intermediate format would be good, to allow generators to be written in other languages.
I think a djinni to SWIG meta generator would be excellent.

+1 @skabbes


isReadOnly events can't be styled separately from editable

Version

1.7.0
Development Environment

Chrome/Linux
Current Behavior

Read only events on the same calendar aren't distinguishable from editable events.
// Write example code

Expected Behavior

A css class or color config option is added to readonly events to allow styling differently.

@jgadbois The Schedule.customStyle can add inline-styles to an event. I use it to present not_confirmed, unaccepted events, for example.

Perfect, thanks


chore(rust): start using jemalloc to avoid memory fragmentation issues

It seems like the memory fragmentation only happens on macOS


Hardware requirements

Hi there,
first of all thank you for sharing this amazing work!
I was wondering what is the minimum required setup in terms of GPUs/RAM to train the model?
I saw in your paper you guys are using 4/8 A100, is there any chance to train the model using consumer GPUs?
Thanks

Hi @nicrusso7 I just updated the README.md with sample sample data and training instructions for 3090. Training on consumer grade GPUs are entirely feasible, just a little slower.

Amazing, thanks!
I've a RTX 3070 with just 8GB vRAM and 16GB system RAM, not sure the model will fits but I'll try!


Info about code coverage report

Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[x] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library

Description
Info about code coverage report (documentation update)
Related Tickets & Documents

Related Issue #OLS-186

Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing

N/A

/approve
@onmete can lgtm after his question is resolved

/lgtm


Update Container Sensor configuration

Don't allow custom certs for the injector
Remove runasuser/runasgroup as this breaks on OpenShift
Update configmap with container entries
Disable pull secret creation unless configured
Autogenerate certs for the injector

Ignore the chart version number failure. It shouldn't be 0.10.0. This PR also creates the version it should be at.


[Feat] Add feedback and about to viewer

Add's the feedback button to the viewer as well as an about button + dialog.

The dialog appears automatically on a user's first visit to the viewer.

Then there's a funky commit: b725184c9ba86983ea6191d18e43b4e2e4db0132
We're currently embedding global styles (e.g. tailwind and any other global styles) in a <style global> element in the root component. Unfortunately, it seems that Svelte's preprocessing is pretty fragile when it comes to using global styles. So, that approach has forced me to use various workarounds (a funky style rewrite hack to get some dark-mode styles to work, splitting up style declarations, making more styles global than actually desired) and finally it totally blocked me from using the tailwind typography plugin.
So, this commit tries another approach that's at least equally as funky, BUT it works 🤷. And remember, this is only the viewer 😉, so it's the only thing using this weird approach where as FWL doesn't use anything funky AND now benefits from not having to write styles in a way that makes global happy.
FYI The viewer is currently broken and being fixed in #1105

pushed my own change to modify the version being used when reporting feedback


how to create new user?

I studied full document but there no code available for user create, user info and so.
i want to create new user with admin client with golang

you will have to use https://github.com/minio/madmin-go

Any example code for crate user in python programming language


Opening a node programatically

How would you open a node programatically? I'm using a scrollToItem method and I would like to open the parent nodes if the item is not visible. I would map the parent ids to the item id, so just need to open the nodes based on the ids I have.
Good job with the library!

I got this to work by setting the isOpenByDefault value inside the treeWalker to true if that item is in the array of ids I have in my local state, then I had a useEffect trigger when that array of ids changed to call recomputeTree.
Example:
const [openItems, setOpenItems] = useState([]);
const treeRef = useRef();

const treeWalker = useMemo(() => {
    return function* treeWalkerFn(refresh) {
        ...
       const isOpened = yield refresh
          ? ({
              id: node.id,
              isOpenByDefault: openItems.includes(node.id),
           })
          : node.id;
        ...
    }
}, [openItems]);

useEffect(() => {
    if (treeRef.current){
        treeRef.current.recomputeTree({
           refreshNodes: true,
           useDefaultOpenness: true
        })
    }
}, [openItems]);

return (
    <FixedSizedTree
       ref={treeRef}
       treeWalker={treeWalker}
    />
);

Hi @kmrsfrnc, sorry for the late response.
Unfortunately, there is no easy way to do it. You may use the approach @woollsta suggested, but it will probably close all elements opened by the user unless you track them in the openItems array manually.
I'm going to mark this issue as a feature request and implement a solution when I have time. Thanks for your contribution!


Plotly 'notebook' renderer creates only blank output using preview notebook editor

Environment data

VS Code version: 1.58.0-insider
Jupyter Extension version (available under the Extensions sidebar): v2021.7.949309867
Python Extension version (available under the Extensions sidebar): v2021.7.922878407-dev
OS (Windows | Mac | Linux distro) and version: Windows 10 21H1 (19043.1052)
Python and/or Anaconda version: Python 3.9.0 64-bit
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Jupyter server running: Local

Expected behaviour
Can use Plotly renderer types 'notebook' or 'notebook_connected' when using the preview notebook editor.
Actual behaviour
Only an empty cell output is shown but the cell output is resized to what appears to be the possible dimensions of the Plotly graph.
Interestingly, generating a Plotly graph using 'notebook' renderer with the preview notebook editor, saving and then re-opening the .ipynb file with the old notebook editor will show the graph without re-generating the cell output. So it seems that this is an issue with the cell display itself but not with the generation of the cell output content.

More details

Currently, the default 'vscode' (aka 'plotly_mimetype') doesn't seem to support Plotly animations (see #4364). When using the old notebook editor "Jupyter Notebook (custom)" it is possible to use 'notebook' or 'notebook_connected' as the Plotly renderer which then also supports Plotly animations in VS Code. With the new notebook editor (Jupyter Notebook (preview)" only 'vscode' seems to work as inline renderer type but this renderer option still does not support Plotly animations or other interactive features in VS Code. This applies to insider version (1.58.0-insider) as well as the stable version (1.57.1).
Note 1: Other non-inline renderers like 'browser' are working as expected and would be a possible work-around to some extent. But of course it would not present the same experience as an inline renderer.
Note 2: In older versions (1.55.2 stable) the 'iframe' and 'iframe_connected' renderers were working with the old notebook editor as well but are not working anylonger (at least since 1.56.2 stable).

Steps to reproduce:
import plotly.express as px
df = px.data.gapminder()
fig = px.scatter(df, x="gdpPercap", y="lifeExp", animation_frame="year", animation_group="country",
           size="pop", color="continent", hover_name="country",
           log_x=True, size_max=55, range_x=[100,100000], range_y=[25,90])
fig.show(renderer='notebook')

Logs

Console output

VM369:4 UI PostOffice Received IPyWidgets_mirror_execute
VM369:4 UI PostOffice Received IPyWidgets_msg
VM369:4 UI PostOffice Received IPyWidgets_msg
VM369:4 UI PostOffice Received IPyWidgets_msg
VM369:4 UI PostOffice Received IPyWidgets_msg
VM369:4 UI PostOffice Received IPyWidgets_msg
VM369:4 UI PostOffice Received IPyWidgets_msg
VM384:5 Uncaught TypeError: require.undef is not a function
    at <anonymous>:5:17
    at S (notebookWebviewPreloads.js:3)
    at notebookWebviewPreloads.js:3
(anonymous) @ VM384:5
S @ notebookWebviewPreloads.js:3
(anonymous) @ notebookWebviewPreloads.js:3
async function (async)
(anonymous) @ notebookWebviewPreloads.js:3
$.outputs.set.queue @ notebookWebviewPreloads.js:3
enqueue @ notebookWebviewPreloads.js:3
(anonymous) @ notebookWebviewPreloads.js:3
postMessage (async)
(anonymous) @ main.js:833
(anonymous) @ host.js:27

Please see this issue here on using something other than the default renderer for plotly:
https://github.com/microsoft/vscode-jupyter/issues/4364

@rchiodo this issue is specifically about the differences between the old custom notebook editor and the new preview notebook editor. It is not really clear what the expected behavior should be. Is it supposed that 'vscode' is the only supported renderer type when using Plotly with VS Code? Why does the 'notebook' renderer work with the old notebook editor and not with the new notebook editor. Coming from the stable version of VS Code and the Python / Jupyter plugins (which uses the old notebook editor) this seem like a regression when switching to the new notebook editor.

Why does the 'notebook' renderer work with the old notebook editor and not with the new notebook editor. Coming from the stable version of VS Code and the Python / Jupyter plugins (which uses the old notebook editor) this seem like a regression when switching to the new notebook editor.

I don't know for sure, but my guess is that plotly has code in it to detect some of the things we had in the DOM for our old editor. If it's not working with the new notebook editor with a renderer type of 'notebook', then I'd guess they need to update whatever they were doing before to work with the new editor. AFAIK, it's never worked with anything other than 'vscode'.
Looking at the console output, it looks like a renderer type of 'notebook' assumes it can use require.js (which is not there in the VS code case).

This PR might make it work again:
https://github.com/microsoft/vscode-jupyter/pull/6318

Looks promising. I tried to compare both cell outputs for new vs. old notebook editor but apart from some styling change and positional movements there was no real difference in the generated output. So it looks like the old notebook editor does support the require function somehow (?). I don't have enough insight but I would assume that's independent from Plotly.

@rchiodo and @jackthepanisher Does this look like the expected output? This is running with my PR change.

@IanMatthewHuff Yep that's it. Thanks.
The notebook renderer also makes it possible for the animations to run. At least it does for me.

@jackthepanisher our next insiders build should have this fix in it. Probably in the next day or two.

That's good news. Thanks for the quick response and fix.

@rchiodo and @IanMatthewHuff I have applied the changes to my 1.57.1 production setup of VS Code with Jupyter extension v2021.6.999 and it works like a charm.

It seems like there is an issue with opening a notebook containing cell output in 'notebook' renderer mode. The output stays blank and only shows up again when re-running the cell. Do you guys want me add logs here or create a new issue?

It seems like there is an issue with opening a notebook containing cell output in 'notebook' renderer mode. The output stays blank and only shows up again when re-running the cell. Do you guys want me add logs here or create a new issue?

Please create a new issue. I'm guessing it requires our kernel to run something first as 'require.js' is loaded as part of our kernel loading.


Release v3.5.1

Minified assets for CDN
Switched to TypeScript

@arturovt - It looks like when I merged the CDN stuff in with typescript, I'm getting some errors when trying to build. I think because the CDN stuff is trying to use JS/babel. I get this error:
src/index.js → dist/umd/pushin.min.js...
[!] Error: Could not resolve entry module (src/index.js).
Error: Could not resolve entry module (src/index.js).
    at error (/Users/nathanblair/Sites/pushIn-js/node_modules/rollup/dist/shared/rollup.js:198:30)
    at ModuleLoader.loadEntryModule (/Users/nathanblair/Sites/pushIn-js/node_modules/rollup/dist/shared/rollup.js:22436:20)
    at async Promise.all (index 0)

I tried changing the entry point to src/index.ts, but then it has issues importing helpers.ts. I wanted to run this by you before I went too deep down that rabbit hole.

Yeah, I knew those 2 PRs are incompatible

I’ll fix it

Should I make a pr to this branch?

@arturovt Thanks for getting this working, looks great! I also added minified CSS for the CDN.
Considering all the ideas we have in the works, I think I will release this as a pre-release to v4. We can continue working on some of the other big improvements that we've discussed and release them all in v4 once ready.


Add algolia app id

We recently replaced swift type with algolia (https://github.com/pulumi/pulumi-hugo/pull/2868). This PR adds the relevant keys to the config.

Thanks @zchase! And apologies for the oversight.


Support ESLint 8

When I try to run suppress-eslint-errors, I get this error:

ERR src/app/graphQlClient/index.ts Transformation error (CLIEngine is not a constructor)
TypeError: CLIEngine is not a constructor
at codeMod (/home/gabrielx/projects/oss/oss_ms_teams/node_modules/suppress-eslint-errors/transforms/suppress-eslint-errors.js:12:23)

My project uses ESLint and I see that CLIEngine has been removed in ESLint 8: https://eslint.org/docs/8.0.0/user-guide/migrating-to-8.0.0#the-cliengine-class-has-been-removed

As a workaround, I was able to temporarily just revert from eslint": "^8.8.0, to eslint": "^7.32.0. Then I could run suppress-eslint-errors, and immediately switch back to the latest eslint.

This also caught me up, but was able to use the recommended 7.32.0. @mizdra I would love to try yours out sometime as well. I like the interactive idea a lot!

Fixed in 3.0.1


error when making proofs with fcomm

I have occasionally observed transient errors when making proofs with fcomm. See example below. I also observe a signal 11. The errors I saw were not immediately reproducible.
➜  examples git:(improve-makefile) ✗ time make fibonacci-proof
cargo run --release -- prove --expression fibonacci.lurk --proof fibonacci-proof.json --lurk
    Finished release [optimized] target(s) in 0.13s
     Running `/Users/clwk/fil/lurk-rs/target/release/fcomm prove --expression fibonacci.lurk --proof fibonacci-proof.json --lurk`
libc++abi: terminating with uncaught exception of type std::__1::system_error: mutex lock failed: Invalid argument
make: *** [fibonacci-proof.json] Abort trap: 6
make fibonacci-proof  73.70s user 3.01s system 292% cpu 26.260 total

Confirmed.

This was encountered recently in CI:
https://gist.github.com/huitseeker/5d20091bc7768d6f63f13f70f99fcee2
From https://app.circleci.com/pipelines/github/lurk-lang/lurk-rs/1494/workflows/a7122220-5b12-46fe-b76d-1d8cd50c445c/jobs/8545 (on c480bbc)

I performed some experiments so far, trying to make a reproducible example of the error. fcomm cli supports -r flag that corresponds to Lurk's number of circuit reductions per single step (ReductionCount). By default this value is set to 10. It Supported values are 1, 5, 10 and 100. Executing experiment that runs Fibonacci example (used originally by @porcuquine) 100 times with fixed number of circuit reductions shows that segfaults occur when ReductionCount=1 in 15% (15 times per 100 executions, with rest successful) and are not observed for other values. I'm attaching correspondent log files here: r1.txt.
Also, manual weird debugging with a help of printlns shows that issue used to occur directly at proving step  of Nova:
let res = RecursiveSNARK::prove_step(
                &pp.pp,
                recursive_snark,
                circuit_primary.clone(),
                circuit_secondary.clone(),
                z0_primary.clone(),
                z0_secondary.clone(),
            );

which further goes outside of lurk-rs to nova-snark dependency. Therefore I can't rule out the possibility of problems with input validation on Nova side.

After experimenting a bit more with fcomm with local nova-snark package, I'm more and more leaning to think that issues that cause segfaults are outside of fcomm and lurk-rs. When I tried to remove parallelism, introduced in Nova while computing intermediate values in commit_T function for instance (see this patch for complete changes introduced), I tried performing mentioned above experiment with 100 executions of Fibonacci example once again (when ReductionCount=1) and got no segfaults, which gives me more confidence in the source of the issues.
In this regard, I think it makes sense to narrow the searching to evaluating detected functions (checking characteristics of input data) on Nova side. I guess it may help in 100% reproducing of the segfaults with fixing them subsequently

Just triggered this on a Nova test run, using nextest which gives good indication of provenance (though this is still hard to reproduce):
--- STDERR:              nova-snark tests::test_ivc_nondet_with_compression ---
libc++abi: terminating due to uncaught exception of type std::__1::system_error: mutex lock failed: Invalid argument

This is the T.txt and commitment-key.txt that I used as a source of the data (GitHub doesn't support JSONs in comments, so in order to use them, they should be just renamed to *.json)

So, supranational suggest to use older C++ compiler / OS version on Mac (it is declared that on with MacOS 12.6, clang 14.0.0 and Rust 1.67). I didn't check it as it is a bit painful to downgrade the development machine. We can try to setup additional CI job with runner that has specific software versions and check this actually

FYI, I saw this recently (not via fcomm). However, I'm already using clang 14.0.0.
➜  clang --version
Apple clang version 14.0.0 (clang-14<IP_ADDRESS>)
Target: arm64-apple-darwin21.6.0
Thread model: posix

I'm not sure if this contradicts the suggestion or not, but it's a datum to consider.

This was with Rust 1.68, but I'm doubtful that's the cause.
➜  rustc --version
rustc 1.68.2 (9eb3afe9e 2023-03-27)

I've now observed this when running the recursive-snark benchmark from Nova. In fact it seems to happen pretty often there. I think we may already have reached this conclusion, but it certainly suggests this is not a Lurk issue (though still important for us to have fixed).

This will be resolved by the next release of sppark:
https://github.com/supranational/pasta-msm/issues/2#issuecomment-1562993118


TechDocs: Improve TechDocs homepage which shows list of current documentation sites

I think this has been shipped with TechDocs V1 i.e. we can browse the list of documentation websites.
@spotify/techdocs-core I'm closing this, but we can re-open if this means something else.

@OrkoHunter Can we leave this issue open but just rename it a little? I think we can definitely work more on the TechDocs Homepage which is what I think you refer to as the list of current sites. That page doesn't deliver a good UX for more than 10 sites I would say.

@emmaindal Sure, absolutely!

Hi @katz95 cc @soapraj ! We need some design inspiration on how the techdocs plugin's home page should look like and what features it should have. We can take our internal techdocs homepage as an inspiration as well. Few thoughts to consider from the team -

Pagination: The homepage shows the list of all documentation sites but what should we do for a large number of them (~500). How should we paginate?
Categorization of documentation sites
Have an all/owned section just like the entity page
Favorites: Can users star/fav doc sites.  We can just redirect to the right scaffolder page for this.

We will then split this issue into separate issues after the design decision has been made. Thank you!
(I am happy to jump in on a team sync including @garyniemen if that helps)

@OrkoHunter thanks for tagging me :) spoke to @soapraj yesterday and will be looking into this! will get something to you by next week

Been going over some of these design decisions with @garyniemen and @emmaindal and we were thinking along these lines.
Overview page

Has a prominent search bar that allows users to quick search (pulls up 5 results). If user clicks 'enter' or clicks 'advanced search', they will be directed to the 'All Documents' page, which is a comprehensive page with all the docs.
Also has 'Your Favorite' documents and 'Recently Read' documents sections. For the Favorites section, we will likely need to add a 'View More' button - once we figure out where we can store an archive of all Favorites (perhaps as another tab within TechDocs?)
Bottom section includes 'Recommended for You' section featuring docs represented in cards that have tags on them

All Documents

Filter capabilities allows users to hone in on the docs they're looking for
Users ideally can favorite and share docs

Owned Documents

Empty state if users don't have any documents
Once users have their own docs, they can see a table similar to the table under the 'All Documents' page, where perhaps they can view metrics for their documents (views, interactions, shares, etc.)

Any comments/feedback would be appreciated!!

complete GIF

complete GIF

Hey @katz95, thanks so much for these designs. Its great! As search isn't ready yet I suggest we create some separate issues out of this, based on the following user stories:

As a user, I should be able to favorite any docs page.
As a user, I should be able to see my list of favorited docs pages on the docs landing page (see design above).

As a user, I should be able to create a new component based on the docs template from the TechDocs HomePage, by clicking on a "create component" button (see design above).

As a user, I should be able to view support information on the TechDocs HomePage (see design above)

As a user, I should be able to view what documents I own rom the TechDocs HomePage (see design above - all documents).

And then we wait with, search, "recently viewed" as well as "recommended for you". If needed we can replace "recommended for you" with displaying all documents that exist (no matter if you own them or not) together with some filters but without search.
What do you think @OrkoHunter @garyniemen @katz95?

Hey @katz95, thanks so much for these designs. Its great! As search isn't ready yet I suggest we create some separate issues out of this, based on the following user stories:

As a user, I should be able to favorite any docs page.
As a user, I should be able to see my list of favorited docs pages on the docs landing page (see design above).

As a user, I should be able to create a new component based on the docs template from the TechDocs HomePage, by clicking on a "create component" button (see design above).

As a user, I should be able to view support information on the TechDocs HomePage (see design above)

As a user, I should be able to view what documents I own rom the TechDocs HomePage (see design above - all documents).

And then we wait with, search, "recently viewed" as well as "recommended for you". If needed we can replace "recommended for you" with displaying all documents that exist (no matter if you own them or not) together with some filters but without search.
What do you think @OrkoHunter @garyniemen @katz95?

Agree to splitting up into several issues. I like how you broke this down into several pieces, it's awesome!

Agree to splitting up into several issues. I like how you broke this down into several pieces, it's awesome!

@garyniemen @OrkoHunter @soapraj I've created all the sub issues above as discussed, feel free to add any missing information if you feel its needed.

@garyniemen @OrkoHunter @soapraj I've created all the sub issues above as discussed, feel free to add any missing information if you feel its needed.

Thank you for creating them @emmaindal ! ⭐

Thank you for creating them @emmaindal ! ⭐

Closing this issue now, since we have split it into multiple smaller issues.


integration prometheus metrics

I have a question. There is jaeger server integrated with prometheus and I use the code as follows:
import (
	"github.com/uber/jaeger-client-go"
	jaconfig "github.com/uber/jaeger-client-go/config"
	"github.com/uber/jaeger-lib/metrics/prometheus"
)
...
metricsFactory := prometheus.New()
// Initialize tracer with a logger and a metrics factory
_, err = jcfg.InitGlobalTracer(
	appConfig.Tracing.ServiceName,
	jaconfig.Logger(jaeger.StdLogger),
	jaconfig.Metrics(metricsFactory),
)

But after some check ups it turned out that no have application-level metrics: span count, current load/endpoint (i check jaeger-host/metrics). They should appear automatically or need "manual" activation? Spans in jaeger are created correctly, I create them manually in middleware

You need to run an endpoint that exposes prometheus metrics; the code above simply sends them to the in-process collector.

for example, https://github.com/jaegertracing/jaeger/blob/master/examples/hotrod/services/route/server.go#L64

Stop stop stop.... I have the integration according to the scheme: app -> jaeger -> prometheus
not:
app -> jaeger

Jaeger is not a metrics system. It provides observability into its own operations by exposing Prometheus metrics.


Overflow Bins for Plotting

Suggestion for adding overflow and underflow bins for the plotting of histograms. Ideally, this could be added as an additional parameter to choose whether or not these should be displayed. These would be especially useful for distributions with long tails.

I think the easiest way would be to just move the bin content of the overflow bin into the last bin. If that's sufficient, it should be easy to add


Added silence_level_analysis (sla) for easier deciding of silence level

It is a bit time consuming to test silence levels for videos by repeatedly entering different silence level commands , and some videos I used unsilence on had 0 difference using the default -35 silence level, so I thought of creating a table with multiple silence levels that would make good reference for user to decide what the silence level should be. Can be easily expanded by just changing the list values.
Here’s a short video demo of how it works on my Win10:
https://www.youtube.com/watch?v=raWyQhvf4WI&feature=youtu.be

This is an awesome idea, this will definitely be part of the next big release. I have to more ideas that will probably make this even better:

Instead of exiting the program after selecting no when asked to continue with the original silence, adding the option to select a new silence level via console input will save even more time, since you don't need to re-run the detection.

I think we should add a string parameter after -sla to select the range of silence levels to be tested. So you have e.g. the options low,normal,high,full for low, normal and high silence decibel levels, and full if you want to test every level?

What do you think about that?

Hi, I like your improved ideas very much. Hadn't thought of having a low, normal, high, full version. That would be quite comprehensive and nice.
I'll work on them tomorrow along with one feature I was thinking of - If someone uses unsilence and there is no difference in output video (0.00% delta for all audible, silence and combined), then it would suggest to use sla so that user can choose a correct silence level value.

Great idea! I'm looking forward to your update!
Do you think it is necessary to also add these features to the python library?
I wanted to keep the command line tool and the library in sync, but I think adding sla to the library will bloat the code, since this can be done easily in code, eventhough it is a really useful thing to have in the command line tool.

Hi, I don't think it's necessary in the library. As you said, it should be easy for user to implement this in their code if they use the library. I would not want to unnecessarily bloat the code either.
I've added the features that we discussed in the latest commit.

Console input to enter new silence value after SLA if user chooses to
low, normal, full and high options for decibel ranges
Rerunning run with argument sla on if user chooses to after finding 0 delta/no change in output intervals.

Hey, thanks for your great work 😀 I'm going to look over the changes later this day and merge it into the next release branch. I will also probably add a lru cache to the silence calculation in order to avoid executing the function again with the same parameters (caching would be a big speed up compared to a rerun)


Keyword documentation is unavailable

The link to keyword documentation broke and must be corrected.

The repository of the documentation had to move end of September/2021 . The broken link is now corrected.


[Platform] Dogecoin Transaction Inputs

Blockatlas has a bug when you receive Dogecoin transactions from it.
If you look at the "inputs", you will see the numbers doesn't have the decimal places and is incorrect.
Here is an example:
Dogechain API:
https://dogechain.info/api/v1/transaction/b02977b96e5c65fd807e28230375c1267ded1de7c2c43292bf36552283bc5696
Inputs sum: 10
Blockatlas API:
http://localhost:8420/v1/doge/address/DPoYGk1wGQ3uWs5G3exd9WKvVyu8weKYVA

Inputs sum: 10
But if you look at the other values, they have the decimal places, just this value not.

@PAkhidue

BA does support passing DOGE address in order to get transactions list
If query https://dogechain.info/api/v1/transaction/b02977b96e5c65fd807e28230375c1267ded1de7c2c43292bf36552283bc5696 in Postman response for the value comes back with decimals points

Hmm... I'm not sure if I explained it correctly.
So the first output is "10" from Blockatlas response (second picture in my first post).
But shouldn't it be like the second output or first input (10 * (10 ^ 8)) or how can I see if it's with or without decimal places?
9973378 would be 0,0997
10 would be 0,0000001
That's why I am confused.

@kolya182
Hmm... I'm not sure if I explained it correctly.
So the first output is "10" from Blockatlas response (second picture in my first post).
But shouldn't it be like the second output or first input (10 * (10 ^ 8)) or how can I see if it's with or without decimal places?
9973378 would be 0,0997
10 would be 0,0000001
That's why I am confused.

So outputs[0].value expected to =<PHONE_NUMBER><PHONE_NUMBER>/(10**8)=10) isntead of 10
9973378/(10**8) => 0.09973378 - correct output.
Can i have xpub you used to get this transactions list ?
Trezor Blockbbok response with<PHONE_NUMBER> as expected by we return 10

I used this address for everything: DPoYGk1wGQ3uWs5G3exd9WKvVyu8weKYVA
Trezor: https://doge1.trezor.io/api/v2/address/DPoYGk1wGQ3uWs5G3exd9WKvVyu8weKYVA
Trezor tx: https://doge1.trezor.io/api/v1/tx/b02977b96e5c65fd807e28230375c1267ded1de7c2c43292bf36552283bc5696
In my first post I used the "dogechain.info" API: https://dogechain.info/api/v1/transaction/b02977b96e5c65fd807e28230375c1267ded1de7c2c43292bf36552283bc5696
Thanks for the answers and help :).

@PAkhidue We want to close this issue, could you send your xpub if yo uhave to my telegram t.me/mykola ?


Seemingly no tick function?

https://github.com/monkeyhue/btl-smp-demo/assets/131806616/aaff8d95-7cea-4d99-ad1a-a36fd14916e2
The datapack, when downloaded from this very github page, does not work.
Advencements and basic functions work, but nothing after that.
Just thought I'd mention/ask why.

Because this is the raw github download, it does not come bundled with any of the smithed libraries required to run. Please download and use the following libraries in order for the pack to function properly:

Smithed Actionbar (0.4.1+)
Smithed Block (0.2.0+)
Smithed Crafter (latest)

Once these are downloaded (make sure to merge with mito or something), do a reload and things should all work fine

Yeah, I knew I just misunderstood the readme, just couldnt figure out how.


travis: Automatically use the latest Ansible patch releases

When a new patch version of Ansible is released, currently the
travis configuration needs to be updated to consume it for tests.
This patch implements a change in the configuration to ensure that
the latest patch release for each minor series tested is always used.

Haha, the failure to download the dashboard is fixed in https://github.com/cloudalchemy/ansible-grafana/pull/40 :)

Fantastic, thanks. Do you guys hang out on IRC/Slack somewhere? My next updates are more feature orientated so it might need some discussion.

RIght now we don't have any, but I can set up IRC channel. I'll be back with more info.

I'm already on freenode, if that helps.

That's exactly where I wanted to start a channel :smile:

@rdemachkovych
@odyssey4me
I've started a channel #cloudalchemy on freenode.net: http://webchat.freenode.net/?channels=cloudalchemy


Upload from clipboard

Is it possible to tweak the code to upload from clipboard? I am new to bash and wondering if that's something that may be easy tweaked or not

I think implementing this would be out of the scope of this project, however you can write the clipboard to a temporary file and upload that file using imgur-screenshot!


Plugin not unload in the server restart

Describe the bug
The plugin crashes when deactivated and does not shut down the server
To Reproduce
Steps to reproduce the behavior:

Restart a server

Expected behavior
The plugin shuts down without crashing and the server restarts
Screenshots

Software (please complete the following information):

Server Implementation: Purpur
Minecraft Version 1.19.3
Java Version 17
Plugin Version 3.0.5

Additional context
Add any other context about the problem here.

[15:55:17 WARN]: [nChat] The PlaceholderAPI placeholder translator took more than 50ms to process the input "%hexnicks_nick%" for Y2k_vinix (took 135.84ms).

I assume you're using MySQL? I've seen this issue with others using it and have a fix I believe. People should be testing it now.

Was this fixed in v3.1.0?

Closing this as completed given a lack of response and the fact that it should be fixed in v3.1.0.
If this issue still persists please reopen this issue.


Remote configuration is not updated after a while

Since a few weeks we observe a strange behavior with the remote configuration. It seems to be lost after several hours (24?), reverting back to the embedded configuration instead of the last fetched one. Killing and restarting the app fetches the remote configuration again, but this is a bit sad.
We updated Firebase SDK, was there a regression or did we miss something?

Tested today with latest betas on which we experienced this issue, opened after more than one day. Might be related to a temporary Firebase backend issue, now the problem seems gone.

Some test results are in https://jira.srg.beecollaboration.com/browse/PLAYRTS-2000.

Firebase SDK seems to work as expected, so the problem is likely on our side.
Fix proposal on the feature/remote-config-fix branch:

Removal of the activate call at startup, which probably interleaves with the fetch-activate triggered right afterwards. According to some tests this was unnecessary.
Use same settings for nightlies, betas and production versions (during 15 minutes after the last successful remote configuration retrieval the SDK inhibits other attempts). This way it is easier to test the production behavior with nightlies.

This old commit was introduced during the tvOS epic, at the migration of iOS project from Carthage to SPM.
Two commit later, ff81c440eae5923979a77aa2e5ea72dfabe1bd21, 4 hours and a half later, the FirebaseRemoteConfigStorageFix.m workaround was introduced for FireBase on tvOS.
We were not aware of the Firebase storage issue on tvOS. The first commit to call the activate method at startup was a first try to fix remote configuration on tvOS. It wasn't a success.
The second commit to move FireBase database to caches directory worked.
But we kept the first commit.
From your findings and tests, it sounds a good idea to remote it.
Let's test in a nighty build.

The "revert" commit works as expected on the nightlies:

After a remote configuration update on FireBase: 15 minutes later in the background, application reopened displays the previous configuration, and fetches the new configuration in background. Kill and reopen the app displays the new configurations. Application configuration is updated and displays the new informations.
With a remote configuration not changed (but different from the local configuration): 15 minutes later with the application killed, application launches and displays the cached remote configuration. No restauration to the local configuration. Application still has the correct configuration and displays the informations.

Tested on:

iOS 14.2
iPadOS 14.2
tvOS 14.3 beta.

Just updated the application won't update missing update. In my test, "last chance" swimlane is missing.

Updated the remote configuration key without changing the string does not update the local cached value. "last chance" swimlane is still missing.
Updated the remote configuration key with an unknown section that changing the string does update the local cached value: "last chance" swimlane is displayed.

As discussed with @defagos , feature/reset-remote-config proposition is also needed.
Now built and deployed in beta build iOS 351 and tvOS 7.

Not sure the problem has been fixed. Play RTS installed on my iPhone still shows view counts, even after they have been disabled remotely.
I am starting to ask myself if there may be an issue in Firebase itself.

Not sure the problem has been fixed. Play RTS installed on my iPhone still shows view counts, even after they have been disabled remotely.
I am starting to ask myself if there may be an issue in Firebase itself.


Slim down deps on Plutus

Avoid a few needless deps on plutus-tx, keep all libraries in the main path depending only on plutus-ledger-api.
Only import Plutus.Ledger.V1.Api and Plutus.Ledger.V1.Examples.
Bump plutus to make the second change possible.

I'll probably follow this up with a change to make those two modules the only exposed modules in plutus-ledger-api, but this is a prerequisite for that.

Seems to be confused: the evaluation did pass, and the required job succeeded. I could force-push a new commit, but maybe one of you is an admin and can just merge it?

Eh, I just force-pushed.

sorry Michael, I was confused by what I was seeing in CI as well. I tried re-running the tests twice, but it didn't seem to help. glad the force push worked.


[tests] fixed bug when propel trying to execute database migration

Rename table1 and table2 into table1_ts and table2_ts respectively, as table1 already defined in another schema (tests/Fixtures/migration-command/schema.xml).
Executing statement "CREATE TABLE "table1"
(
    "id" serial NOT NULL,
    "title" VARCHAR NOT NULL,
    "table2_id" INTEGER,
    PRIMARY KEY ("id")
)"

  [Propel\Runtime\Exception\RuntimeException]
  Failed to execute SQL "CREATE TABLE "table1"
  (
      "id" serial NOT NULL,
      "title" VARCHAR NOT NULL,
      "table2_id" INTEGER,
      PRIMARY KEY ("id")
  )". Aborting migration.

PS: This happen only when i'm running tests using postgres

Isn't there then a deeper big in schema name usage inside Propel core? We have obviously not issues in our test suite. Maybe you have a newer/older PostgreSQL where some stuff with our schema naming isn't working correctly?

Here are My environment: Ubuntu 12.04, php 5.5.30, postgres 9.1.19. I don't know, may be there are something wrong with local environment, also i'm experienced issue as described here #1082.

closing this.. i'll investigate this issue later