[BUG]AttributeError: module 'torch.nn.functional' has no attribute 'scaled_dot_product_attention'

Dear authors,
I install Deepspeed through pip install deepspeed. And my torch version is 1.12.1 and cuda 11.3
However, when i try to use following code to count the flops, it gives me error: AttributeError: module 'torch.nn.functional' has no attribute 'scaled_dot_product_attention'
if step == profile_step: # if using multi nodes, check global_rank == 0 as well             prof.stop_profile()             flops = prof.get_total_flops()             macs = prof.get_total_macs()             params = prof.get_total_params()             if print_profile:                 prof.print_model_profile(profile_step=profile_step)             prof.end_profile()
I check the README file and it written that the package support torch>=1.9, so it should work with my version. Do you know how to solve this problem? Thanks a lot.
ds_report output
Please run ds_report to give us details about your setup.
Screenshots

I have met the same error.
The environment is torch1.13.1+cu117

Hi @lemyx or @harborsarah - could you test with #6517?  I believe I've made the required changes there.


TR - Data issue with attribute Platform Exit

Check this TR response
https://gist.github.com/vasile/b926da3d3611e6fc1b5acbdd32922907#file-tr_exit_both_platforms-xml-L131-L142
The service has both Aussteigeseite: Links, Aussteigeseite: Rechts . Is it a data issue?
      <Attribute>
          <UserText>
              <Text xml:lang="de">Aussteigeseite: Links</Text>
          </UserText>
          ojp91001DH_InfoCall240_109019_1
      </Attribute>
      <Attribute>
          <UserText>
              <Text xml:lang="de">Aussteigeseite: Rechts</Text>
          </UserText>
          ojp91001DH_InfoCall240_111055_1
      </Attribute>

I checked the data's, there are no Attributes for this service:

Was it a Request on the same day? So again, it's possible the data's are coming from the Realtime service?

@TO-mdv what is the realtime service?
The gist contains a TR response for a request in 2024-05-27
https://gist.github.com/vasile/b926da3d3611e6fc1b5acbdd32922907#file-tr_exit_both_platforms-xml-L131-L142

@u233336 just checked again, the information comes from Realtime. A request for Tomorro don't deliver any "Aussteigsseite"
https://tools.odpch.ch/ojp-demo-v2/search?from=ch:1:sloid:4100:0:54&to=ch:1:sloid:7000:4:8&trip_datetime=2024-12-12 10:15


[discussion] Default root size - 20GB

I run EKS cluster for one week, already get "no space" issue on EKS workers.
Found the code here that the default root size is 20GB.
https://github.com/awslabs/amazon-eks-ami/blob/master/eks-worker-al2.json#L22
      "volume_size": 20,

Should we increase it to 50GB or more?
I can update my codes to start with more space, but 20GB is too small, which I recommend to update in this AMI directly.

This can be overridden when you launch a node group by editing the CloudFormation template,
  NodeLaunchConfig:
    Type: AWS::AutoScaling::LaunchConfiguration
    Properties:
      ...
      BlockDeviceMappings:
      - DeviceName: /dev/xvda
        Ebs:
          DeleteOnTermination: true
          VolumeType: gp2
          VolumeSize: 50

@micahhausler
Thanks.
I know how to custom this option, but my question is, should we increase the default value directly from source here?

I think we’ll keep the default to 20. A more advanced configuration could include mounting a second volume for docker, so the kubelet can still operate even if the primary disk is full

@micahhausler: is there a way to increase the size of 20GB in an already deployed EKS cluster?

For kops, they use a much higher default (120GB). In my opinion, this is way to low to run in with any sort of production traffic.

How does network traffic increase disk usage @njfix6?
I think it matters more what the nature of the workload is. For example we have node pools where the workload uses no (local) disk at all. If the root disk is big enough for a dozen images and 10MB log/container, then it is plenty big enough. In other cases all writeable disk are separate mounts as @micahhausler mentioned. For those workloads 120 GB or even 50 GB would be a waste of many TB of unused storage.


[WIP] to fix #26

I'm going to make sure that the less-symmetric data in "all-types.root" can be read properly. Although I'm testing it in pure Java, it should fix the problems you've been seeing in Spark with fixed-width and variable-width arrays.

@PerilousApricot The generated file is a little weird (probable mistake with signed/unsigned), but the scalars at least agree with uproot:
[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running edu.vanderbilt.accre.root_proxy.TTreeTest
[TRACE] 11:39:58.561 e.v.a.l.s.S.SlimTBasket - getbasket len: 36
[0, 1, 2, -2147483648, -2147483647, -2147483646,<PHONE_NUMBER>,<PHONE_NUMBER>,<PHONE_NUMBER>]

>>> import uproot
>>> uproot.open("testdata/all-types.root")["Events"].array("ScalarI32")
array([          0,           1,           2, -2147483648, -2147483647,
       -2147483646,<PHONE_NUMBER>,<PHONE_NUMBER>,<PHONE_NUMBER>], dtype=int32)

@jpivarski -- I "think" that should be right if you're doing the unsigned. I generate them in testdata/gen/make.py -- but the idea is to do [0,1,2,MIN_INT, MIN_INT -1, MIN_INT - 2, MAX_INT, MAX_INT - 1, MAX_INT - 2] ... the idea being that I wanna make sure I'm properly hitting the ends of the ranges

Merge this whenever you feel comfortable doing so: I'm done working on it. It ensures that all the {int1, int2, int4, int8, float4, float8} × {scalar, array, slice} cases work. The new tests are at the end of TTreeDataSourceUnitTest.java, but I verified that these tests call the functions with the same arguments as Spark (by running TTreeDataSourceIntegrationTest.java.
Oh, and you don't really have any examples of array data with different values in each element of the array—that symmetry has yet to be broken. However, I did find a bug related to that and I'm 95% sure that I fixed it properly.
Types that haven't really been covered: booleans, strings, unsigned types, ROOT's Float16 and Double32.

Thanks for all the help! I'll look at the patch -- I'll work on the more complicated types as I come across them -- trying to figure out the best way to handle the unsigned integers currently, then I'll move on to the other types.
And I like doing as much as possible in the pure Java testing since the integration testing is so slow. It'd be nice to have some sort of "inverse mock" functionality to trace and replay the calls that spark makes downwards

merged, thanks!


Instaloader stops after a random short amount of images from hashtag, no error

Hi, not sure what happened since last time I used but I haven't used the script for about a month and half and it does not seem to work anymore. I have not changed anything in my parameters. Made sure to update to 4.1 (I think I was already on 4.1), but still doing it.
Describe the bug
Instaloader stops after 2-3 images, or 9, or 17, no error. I tried different hashtags, always the same pattern. It just stops without giving any error
To Reproduce
Steps to reproduce the behavior:
instaloader #themandalorian --fast-update --filename-pattern={profile}_{date_utc}_UTC --no-videos --no-video-thumbnails --no-captions --no-metadata-json
or
instaloader #kenobi --fast-update --filename-pattern={profile}_{date_utc}_UTC --no-videos --no-video-thumbnails --no-captions --no-metadata-json
Error messages and tracebacks
No error, it just stops
Instaloader version
4.1
Thanks

Thanks for reporting this! Thanks for also showing us the exact invocation where this happened, which really helped us to easily reproduce the bug and find its cause.
The reason was, that --fast-update aborts the download as soon as a post is encountered where nothing is downloaded. Hence, with --no-videos --no-video-thumbnails it aborted the download after reaching the first video. We just fixed this error in the logic with 41e734e.


New Google Cloud Region europe-west8 (Milan)

Today the Compute Engine API has returned machine types in europe-west8. The region will be the new location in Milan. Region and Zone as well as Billing API do not know the region yet. I have therefore removed them manually. When the region is 100% ready it must be removed from instances/clean_up.sql.

Done: https://github.com/Cyclenerd/google-cloud-compute-machine-types/commit/36736c9fb9c2816a56aaa7033548340435f5b907

New region: https://gcloud-compute.com/europe-west8.html


[WIP] Admin timeline API

This implements an endpoint for the Admin API which returns time series data in the form of JSON which contains a record for each week. Each record contains the week in YYYY-MM-DD format, the total number of VASPs, and counts of the VerificationState changes broken down by status. The value of this endpoint is that it can be used to render time series graphs on the frontend.
TODO

[x] Make start/end date configurable in the request
[x] Figure out what the route should be called, right now it's /v2/timeline
[x] The VASP count is not always going to be correct (e.g., if the state of a VASP doesn't change for a week then it won't be counted)...should we just remove it from the output?

Codecov Report

Merging #128 (d771f56) into main (f089caf) will decrease coverage by 0.23%.
The diff coverage is 33.67%.

:exclamation: Current head d771f56 differs from pull request most recent head 9414f79. Consider uploading reports for the commit 9414f79 to get more accurate results

@@            Coverage Diff             @@
##             main     #128      +/-   ##
==========================================
- Coverage   39.44%   39.20%   -0.24%
==========================================
  Files          29       29
  Lines        2870     2956      +86
==========================================
+ Hits         1132     1159      +27
- Misses       1557     1613      +56
- Partials      181      184       +3

Impacted Files
Coverage Δ

pkg/gds/config/config.go
66.15% <ø> (ø)

pkg/gds/models/v1/models.pb.go
22.03% <9.23%> (-4.66%)
:arrow_down:

pkg/gds/admin/v2/client.go
56.55% <50.00%> (-0.39%)
:arrow_down:

pkg/gds/models/v1/models.go
76.99% <92.00%> (+4.26%)
: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 f089caf...9414f79. Read the comment docs.

@bbengfort Mind taking a second look before I merge? I implemented the ISOWeek alignment scheme following your suggestion.


Add Cloudflare cache purge action

Made a simple action that calls the Cloudflare API to purge the cache of a website, which can be a helpful last step after deploying a new version.
Repo link: https://github.com/jakejarvis/cloudflare-purge-action
Marketplace link: https://github.com/marketplace/actions/purge-cloudflare-cache
Thanks for the great list! 😊

@jakejarvis your changes have been merged, thanks for your contribution 👍


Resolve: loading editorial package as timeline

Changelog Description
Loading Editorial package product (OTIO with resources video files) as timeline.
Additional info
Currently updating method is not developed and we are not yet planning to add it any time soon. There is not certain design when it comes to updating loaded timelines.
Testing notes:

you need to have available editorial_pckg product before testing!

open loader and load any of published editorial_pckg product published versions.

Where can I find editorial_pckg???

Where can I find editorial_pckg???

I suspect in Tray Publisher since this PR: https://github.com/ynput/ayon-frontend/issues/410

Side note: the c in that abbreviation really trips me up. Isn't package usually abbreviated to pkg?

you are right, I will change it. Thanks


[Feature Request]: custom speaker_selection_method should get extra param `last agents'

Is your feature request related to a problem? Please describe.
It is very hard to control speaker selection when adding multiple nested chats that use the same agent.
Describe the solution you'd like
custom state_transition:
def state_transition(last_speaker, groupchat, last_agents):
...

This way we can ask which was the previous agent and have better control over which speaker to select next.
I think this is an easy mod.
Additional context
No response

closing as this can be derived from group chat.messages


add basic nydusify conversion case

The newly added case can measure conversion time and trim unnecessary cases with private testing image reference

@changweige , a new test job has been submitted. Please wait in patience.

@changweige , your test job is running now. Please wait in patience.

@changweige , your test job is running now. Please wait in patience.

@changweige , The CI test is completed, please check result:

Test CaseTest Result

merge-target-branch:white_check_mark:SUCCESS

build-docker-image:white_check_mark:SUCCESS

compile-nydus:white_check_mark:SUCCESS

compile-ctr-remote:white_check_mark:SUCCESS

compile-nydus-snapshotter:white_check_mark:SUCCESS

start-nydus-snapshotter-config-containerd:white_check_mark:SUCCESS

run-container-with-nydus-image:white_check_mark:SUCCESS

Sorry, the current test job result is not valid, because your pull request has been updated, and a new test job will be submitted.

/retest


Improve the Readme

 Issue by alex-hse-repository
Friday Oct 21, 2022 at 09:33 GMT
Originally opened as https://github.com/tinkoff-ai/etna/issues/994

🚀 Feature Request
It seems we have not appropriate readme structure, it should be slightly improved
Proposal
TODO
Test cases
No response
Additional context
No response

This issue is empty.


Fix #251: Correct return type for next_version()

@tomschr if you could take a look that would be appreciated.

PS: you might want to squash these commits since I merged newer changes from the repo into my old fork

@tlaferriere  haha, we had the same idea. :smile:  I've created #255.
There is one thing you could improve: Could you copy the CHANGELOG from my PR? You should also adapt the PR number to this. Other than that, it looks ok.

I was just about to do that


[8.x] Re-add update from support for PostgreSQL

This PR re-introduces "update ... from ..." support for the PostgreSQL grammar. Since there wasn't a good option or straight way to re-implement the behavior after the refactor in #29393 I opted to add a new dedicated updateFrom method specifically for the PostgreSQL grammar.
This implementation is a bit cumbersome and crude as people will need to specifically use updateFrom instead of update if they want the correct behavior. Ideally this would be done automatically.
I've tested this with the queries from #33265 and got them back working with this. I also re-added the original tests that were removed.
Before merging this I'd like to:

Get a review from @staudenmeir
Get feedback from people who reported #33265
Maybe clean up the code in the PostgreSQL grammar

If we get feedback from someone who can think of a better way to solve this then gladly. Otherwise I think this is the only solution for the problem at hand.
Fixes #33265

Thanks for this! I wonder if updateWith might be better suited?
updateFrom works though, and another option I'd thought of, updateUsing, would also make sense, but I thing the "using" terminology is better for callables.

@mikerockett I personally think updateFrom is probably the most straightforward name since that indicates the fix its making.
One other thing that might be considered is changing the API of the method slightly so the name makes more sense or its usage is more straightforward.


Fixes issue #10

Seems @jmuhbagir already fixed this problem 2months ago

Closing PR until @ClusterHQ chimes in on issue #10


Update Detector IDs

Remove duplicate Detector IDs
Correct existing Detector ID URLs

Other PR made this one have conflicts, I've solved those and other things in the readme in #16.
Thank you!

OK, thanks!


Handling ports disconnecting

As it is, RMR does not track ports disconnecting; I have to add some utility code for that.
I am not sure what indicators of disconnection are available.
Are connections and disconnections described by SND_SEQ_EVENT_PORT_SUBSCRIBED and SND_SEQ_EVENT_PORT_UNSUBSCRIBED events?

I did a tiny test with VMPK.
When I pick a port in a dialog and enter it, I catch a SND_SEQ_EVENT_PORT_SUBSCRIBED event in alsa_MIDI_handler.
When I exit VMPK, I catch a SND_SEQ_EVENT_PORT_UNSUBSCRIBED event in alsa_MIDI_handler.
I also catch ``SND_SEQ_EVENT_PORT_SUBSCRIBED` when I reopen VMPK.
Then I repeated it with RMR (virtual input and normal output) and the result is the same. It subscribes, unsibscribes, no "SND_SEQ_EVENT_PORT_EXIT".
Trying to catch exit, I tried RMR yet again, now with virtual output and normal input modes.
When I close a virtual output port, it "unsubscribes", no "SND_SEQ_EVENT_PORT_EXIT" firing. Now that is interesting!

I think all these events are related to the port only, not to a client. Next thing to do is to try and intercept SND_SEQ_EVENT_CLIENT_START and SND_SEQ_EVENT_CLIENT_EXIT.

I tried to catch a SND_SEQ_EVENT_CLIENT_START and SND_SEQ_EVENT_CLIENT_EXIT events with a "virtual output to input" mode. It failed.
Then I tried "output to virtual input" configuration.
It also throws port subscription / unsibscription events.
My current conclusion: I'll have to redesign the output to run in a duplex mode later so I can know if I can detect SND_SEQ_EVENT_CLIENT_START and SND_SEQ_EVENT_CLIENT_EXIT events, but I am currently not sure that has any worth.


feat: Adds :all option for metadata to logger backend

Closes #522

@cleptric the CI seems broken, outside of my change.

I've tried fixing the CI, but it looks like it needs more help than I'm able to give right now.

hey @isaacsanders finally looking at this, sorry for the delay.
If we include all metadata, should we maybe exclude the things listed here which are added by default? Especially crash_reason which cannot be serialized as mentioned in #533 .

When is the soonest we know we can’t serialize something? If you want to start a list of things we exclude, we can start it here, but we could also move this to the request serialization part of the code.

This is obsolete now that we merged #605. Thanks @isaacsanders 💟


All annotations are populating in first page and not considering if any offsets are applied to the page

Attach (recommended) or Link to PDF file here: https://web.archive.org/web/20110918100215/http://www.irs.gov/pub/irs-pdf/f1040.pdf
Configuration:

Web browser and its version: Chrome (84.0.4147.89)
Operating system and its version: Windows 10
PDF.js version: 2.6.0
Is a browser extension: No

Steps to reproduce the problem:

Implementing the acroforms example in angular by npm install pdfjs-dist.
Able to achieve everything as per the acroforms example, except annotations layer positioning.

What is the expected behavior? (add screenshot)

What went wrong? (add screenshot)

Could you please explain, what must be causing this issue? What files in pdfjs-dist are responsible for annotations population aligned to the pdfviewer? A similar issue is already reported #10333, but resolution info is not there.
Thanks

Please see https://github.com/mozilla/pdf.js/blob/master/.github/CONTRIBUTING.md (emphasis mine):

If you are developing a custom solution, first check the examples at https://github.com/mozilla/pdf.js#learning and search existing issues. If this does not help, please prepare a short well-documented example that demonstrates the problem and make it accessible online on your website, JS Bin, GitHub, etc. before opening a new issue or contacting us in the Matrix room -- keep in mind that just code snippets won't help us troubleshoot the problem.

Thank for the response. I have gone through most of the existing issues, but could able to get any solution for the issue. As requested I have prepared and hosted a basic application and the codebase can be accessed at https://github.com/VineelG/ng-pdfjs-acroforms.
Can you please explain, how this can be resolved or where I can check for the issue.
Thanks

You're missing crucial CSS rules, which is pretty obvious if you compare your code with the one in https://github.com/mozilla/pdf.js/blob/master/examples/acroforms/acroforms.html; note in particular https://github.com/mozilla/pdf.js/blob/7b0c52fdfed4642366796021093833df63236550/examples/acroforms/acroforms.html#L28-L36

Thank you very much, this helped to solve the issue. Added style to page class in promise after page is drawn, as applying style to .page class in html was not getting applied.


commit referenced in changelog missing from 2.3 branch

On the master branch commit 0b943b80d36ab1fc16aaf4415e16e2a2c222f324 references "Fixes worker hanging when hard time limit exceeded"   This commit is mentioned in the 2.3 changelog, however when inspecting the code on the 2.3 branch it does not appear to exist there.  It looks like it was never merged.

Thanks!  It's in the 2.3 branch now.


[FEATURE] Support Multiple Data Source in Observability-integration

Is your feature request related to a problem?
With multiple data source enabled, users of OpenSearch Dashboards are able to create data source connections and query local and remote OpenSearch clusters. Enabling multiple data source at dashboards plugin would ensure users that have been using the multiple data source feature to have a consistent experience in the dashboard, and allows users to use the same dashboard for different clusters.
What solution would you like?
Support multiple data source in the Observability integration component and allow user to use one dashboard for different backend clusters
What alternatives have you considered?
A clear and concise description of any alternative solutions or features you've considered.
Do you have any additional context?
Add any other context or screenshots about the feature request here.

Catch All Triage - 1, 2, 3
@ps48 Is there a triage meeting for this repo?


Support Symbol

Would a PR that adds support for Symbols be accepted?
In the current state they are dropped since they are not a enumerable property which are the only ones that are managed since Object.keys is used.

@DLMR I think I'll implement support Symbol in next major release.

Do you have a rough estimate on when that might happen?

@DLMR sorry, no I haven't.


std::bad_alloc for large scale data

Code:
import numpy as np
import KMeansRex
X = np.random.rand(50000000, 9)
Mu, Z = KMeansRex.RunKMeans(X, 1000)

Result:
terminate called after throwing an instance of 'std::bad_alloc'
  what():  std::bad_alloc
[1]    26712 abort (core dumped)  python test.py

The same thing also happens in MATLAB:
[idx, C]=kmeansrex(allLambdas,100);
Unexpected Standard exception from MEX file.
What() is:bad allocation
..
allLambdas is a vector of 3965199 elements (1-dimensional).


bug(engine): slow processing of many accumulated events

Sending 6k events with a worker being offline seems to be slow (e.g. once 6k events are added and pending and then the worker is started, even though jobs finish immediately, queueing seems to take a long time)

fixed via #310


Nest Hello Camera excluded by default

Plugin is excluding Nest Hello device by default even though no exclusion rules are configured.
[9/24/2024, 5:07:51 PM] [NestAccfactory] Device "Brampton - ON" ignored due to it being marked as excluded
Same behavior exhibited in Homebridge & Docker variants.

Is the related to alpha or latest version?

Latest
On Tue, Sep 24, 2024 at 4:05 PM mark @.***> wrote:

Is the related to alpha or latest version?
—
Reply to this email directly, view it on GitHub
https://github.com/n0rt0nthec4t/homebridge-nest-accfactory/issues/4#issuecomment-2372267432,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPSCTMDA725BFT3MDMR5B3ZYHAZTAVCNFSM6AAAAABOYYRZI6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZSGI3DONBTGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

testing in 0.2.0-alpha.1 for fix. Please understand the 'readme' as this version includes major breaking changes moving forward

Fixed included in 14d1a067cfa4a8f643cd65f37a2fa323d8da7db9

Fix include in 14d1a067cfa4a8f643cd65f37a2fa323d8da7db9


Double notification

Hi,
My kresus is running full time on my cozy, and is fetching new operations every night.
Every day (or almost) two notifications with same number of fetched operations is raised.
I said almost, because I have doubts about the cause :

I  have 4 accounts, and one is my main account, so the double notification could be due to the fact that operations were fetched for two accounts (i don't think this is the case), and then, there would be a problem in the number of fetched operations in the notification
It is a systematic situation (two notifications are always raised in this case)

I'will check more preciselly the next days, to validate one or the oter option

Interesting, thanks for reporting. When you say "several accounts", do you mean in several banks, or are they all in the same bank? I could imagine that having several connected banks would lead to several notifications.

3 accounts for a bank
1 for the other (the main account)

Should be fixed in incoming. The WeboobManager is actually a single state that was shared among all banks. Rather than rewriting the entire class to not do that, I've chosen to have one instance of WeboobManager per registered bank, which means you'll get one notification per registered bank. A more proper fix is to come, hopefully...


Recognise IAsyncEnumerable as a channel type

Fixes #38

Needed to go to .net standard 2.1 to get IAsyncEnumerable, which also required updating HelloSignalr to .net core 3.1 to support it.


Inject FLEX in assistivetouchd

Hello !
Do you have any clue how I could use FLEX to debug Assistive Touch UI elements ?
Thanks

Hey! I was actually looking into this myself the other day…
I have no idea how to make it appear, I'm not sure what's wrong. I modified FLEXing so that it showed the toolbar on process launch and it still wouldn't appear for assistivetouchd. I guess I need to debug it and find out why FLEX doesn't appear.

I'm going to move this to the actual FLEXing repo though, this is the wrong place for this issue.


Use ring buffer made by ATS2 for serial console

AVR libc has interrupt disable function.
  uint8_t oldSREG = SREG;
  cli();
  /* Some atomic code. */
  SREG = oldSREG;

I know that Will Blair did this a while ago. I will inform him of what you are doing.

Thank's!

Sometimes following is useful for it:
Effective ATS: Producer-Consumer
http://ats-lang.sourceforge.net/EXAMPLE/EFFECTIVATS/Producer-Consumer/main.html


Weird shifting of list elements at startup

Weird thing is it only happens sometimes. From what i've noticed it works correctly first time i open the app, but it doesn't when the fragment containing the RecyclerView is getting recreated. Any clues as to what might be causing this?
I'm using 5.0.0-b8.
note: animations in the attached screen recording were slowed down to 20%

@shymmq, that should a normal effect if the header items are added after the initialization. In beta8 the headers depend by scrolling animation, if disabled, headers will be insert with notifyItemInserted instead. You can read the comments of the methods.
I worked on that behaviour and removed the scroll animation dependency for the rc1.
So, I suggest to adapt your code to the current Snapshot, because by end of the year I will publish the rc1 that comes from this Snapshot.
To see the snapshot changes content read #39 last post, I also made a migration guide, that is better if you read it :-)

Thanks, it seems to work now. Keep up the good work!


Add Ruby 3.1 to the CI matrix

Same change as in gollum/gollum#1812. 😄

This PR adds Ruby 3.1 to the CI matrix.
Using 3.0 without quotes, ruby/setup-ruby will use 3.1 instead of 3.0.
To avoid similar parsing issues, all version numbers are enclosed in quotes.

Great thanks @mishina2228!


CloudFormation template has only C4 instances for ECS job queue instance types

Problem
Deployed Metaflow using the CFn template provided. Tried to test GPU with @batch(cpu=1, memory=500) but Metaflow execution stuck with no progress.
The Cfn template does not have P instances but only C4 instances specified.
  ComputeEnvironment:
    Type: AWS::Batch::ComputeEnvironment
    Properties:
      Type: MANAGED
      ServiceRole: !GetAtt 'BatchExecutionRole.Arn'
      ComputeEnvironmentName: !Join [ "-", [ 'batch-compute-env', !Ref 'AWS::StackName' ] ]
      ComputeResources:
        MaxvCpus: !Ref MaxVCPUBatch
        SecurityGroupIds:
          - !Ref SagemakerSecurityGroup
        Type: EC2
        Subnets:
          - !Ref Subnet1
          - !Ref Subnet2
        MinvCpus: !Ref MinVCPUBatch
        InstanceRole: ecsInstanceRole
        InstanceTypes:
          - c4.large
          - c4.xlarge
          - c4.2xlarge
          - c4.4xlarge
          - c4.8xlarge

Question
Please clarify to use GPU, P instance types need to be added in the CFn template.

Hi @savingoyal , it appears to me you are the only person responding to github issues and topics in the gitter. Kindly advise what responses and actions the Metaflow users can expect for issues or assistances from the Netflix team if we users get into issues. Without responses from the development team, I may be afraid choosing Metaflow could be a risk and the community may not grow. Appreciate your comment.

@oonisim You can actually add more instance types under the c4 family list (for example p2 types). We should have made it clearer in the doc that the InstanceTypes field should be customized as needed. Btw, this repo https://github.com/Netflix/metaflow-tools is a better place to discuss issues related to CF template.


Mark an Address as Undeliverable

https://help.salesforce.com/s/articleView?id=sfdo.NPSP_Mark_Undeliverable.htm&type=5
Script: https://docs.google.com/document/d/15wHZblCdCYs5PM_AQnAH3DX2OOiKKsrXUz3yXCkkB9o/edit?usp=sharing

working on this one

I'm in to the script doc finally! Happy for your assistance if you're still around.

NOTE: the Trailhead documentation also covers filtering reports for Undeliverable Addresses, which I did not include here. Suggest this as a separate video.


Added name and description info for PFAM domains

Fix #61
Colors are still missing. We should fix the color issue in this PR before merging it.

Codecov Report

Merging #65 into master will decrease coverage by 0.45%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##             master     #65      +/-   ##
===========================================
- Coverage      8.81%   8.36%   -0.46%
  Complexity       29      29
===========================================
  Files            29      30       +1
  Lines           771     813      +42
  Branches         74      77       +3
===========================================
  Hits             68      68
- Misses          692     734      +42
  Partials         11      11

Impacted Files
Coverage Δ
Complexity Δ

...ioportal/genome_nexus/annotation/domain/PfamA.java
0% <0%> (ø)
0 <0> (?)

...tion/domain/internal/PfamDomainRepositoryImpl.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...tal/genome_nexus/annotation/domain/PfamDomain.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Quests plugin not working at all.

Build Used: Tried 3.1.2. Just tried 3.1.5 which was uploaded about 20 minutes ago, also doesn't work.
Problem: Quests plugin is not showing up at all. When I type /quests it just pops up with Minecraft's response to unknown commands: "unknown command.", it isn't in the /plugins list either. It's in the server's plugins folder and uploaded properly. Vault is installed as well.
We also had an issue with version 3.1.2 and while that does not work at all now, it worked a week ago. During that time it also refused to save quests.

Please post your startup log for analysis.

2017-12-16-9.log.gz
Sorry for the delay, was very ill the past week.


the node does not show gpumem allocatable resources

What happened:
After installation, when I use kubectl describe node nodegpu1, it does not show the gpumem related resource.
What you expected to happen:
the '  nvidia.com/gpumem: ' show be shown under 'Allocatable' part of the node.
How to reproduce it (as minimally and precisely as possible):
according to the installation steps, and then run kubectl describe node nodegpu1.
My current info is:
Capacity:
  cpu:                   160
  ephemeral-storage:<PHONE_NUMBER>Ki
  hugepages-1Gi:         0
  hugepages-2Mi:         0
  memory:<PHONE_NUMBER>Ki
  nvidia.com/gpu:        80
  pods:                  110
Allocatable:
  cpu:                   160
  ephemeral-storage:<PHONE_NUMBER>715
  hugepages-1Gi:         0
  hugepages-2Mi:         0
  memory:<PHONE_NUMBER>Ki
  nvidia.com/gpu:        80
  pods:                  110

Anything else we need to know?:

The output of nvidia-smi -a on your host
Your docker or containerd configuration file (e.g: /etc/docker/daemon.json)
The hami-device-plugin container logs
The hami-scheduler container logs
The kubelet logs on the node (e.g: sudo journalctl -r -u kubelet)
Any relevant kernel output lines from dmesg

Environment:

HAMi version:

root@node7vm-1:~/test# helm ls -A | grep hami
hami            kube-system     2               2024-11-14 15:18:36.886955318 +0800 CST deployed        hami-2.4.0                      2.4.0
my-hami-webui   kube-system     4               2024-11-14 17:18:24.678439025 +0800 CST deployed        hami-webui-1.0.3                1.0.3

nvidia driver or other AI device driver version:
Docker version from docker version
Docker command, image and tag used
Kernel version from uname -a
Others:
node's annotation:

                    kubernetes.io/os=linux
Annotations:        csi.volume.kubernetes.io/nodeid: {"csi.tigera.io":"node7bm-2"}
                    hami.io/node-handshake: Requesting_2024.11.14 11:25:56
                    hami.io/node-handshake-dcu: Deleted_2024.11.14 07:22:47
                    hami.io/node-nvidia-register:
                      GPU-fc28df76-54d2-c387-e52e-5f0a9495968c,10,49140,100,NVIDIA-NVIDIA L40S,0,true:GPU-b97db201-0442-8531-56d4-367e0c7d6edd,10,49140,100,NVID...
                    kubeadm.alpha.kubernetes.io/cri-socket: unix:///run/containerd/containerd.sock
                    nfd.node.kubernetes.io/feature-labels:
                      cpu-cpuid.ADX,cpu-cpuid.AESNI,cpu-cpuid.AMXBF16,cpu-cpuid.AMXINT8,cpu-cpuid.AMXTILE,cpu-cpuid.AVX,cpu-cpuid.AVX2,cpu-cpuid.AVX512BF16,cpu-...
                    node.alpha.kubernetes.io/ttl: 0
                    projectcalico.org/IPv4Address: <IP_ADDRESS>/24

yes, that resources aren't registered in kubelet, you can see it by visiting {scheduler node ip}:31993/metrics

The design of device plugins is to simplify resource management, ensuring that each instance focuses on managing a single type of device resource. The related interfaces, like Registration and ListAndWatch, enforce this by limiting each device plugin instance to reporting and managing only one resource type.
In our case, to handle GPU resources like cores and memory for scheduling, we register the total allocatable amounts as annotations on the nodes. You can see an annotation like this:
hami.io/node-nvidia-register: GPU-fc28df76-54d2-c387-e52e-5f0a9495968c,10,49140,100,NVIDIA-NVIDIA L40S,0,true:GPU-b97db201-0442-8531-56d4-367e0c7d6edd,10,49140,100,NVID....

And because we can only report nvidia.com/gpu to the kubelet, this results in the 2 Insufficient nvidia.com/gpumem warning message from the default scheduler in the previous issue #611 .


Fixing horizontal scroll issue in SQL Migration Dashboard

This PR fixes #15736 and also fixes #15737
When zoomed (horizontal scroll bar on the bottom)
Before:

After:

Portrait Mode fixed:
Before:

After:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
26783

Relevant Lines:
57117

💛  - Coveralls

I have tested this change on other tabs, and it everything seems fine.


Impelement collect image / video and upload to google drive.

[Issues #7]
Use oauth2 protocols and google-api-nodejs-client to upload files to drive.
Only enables upload image function now, because video file size may always over 5MB(problem describe below).
[To fix]
Google-api-nodejs-client now only provides uploadType=multipart  which cannot upload file over then 5MB .
To upload file more then 5MB, we should write a new post function as uploadType=resumable.
large file problem :

https://github.com/google/google-api-nodejs-client/issues/276
https://github.com/google/google-api-nodejs-client/issues/530

possible solution :

https://gist.github.com/dam1/f77fcbea52ca9124ff50f3598d280710

[Steps to verify]
First, follow the step1 in this url to get client_secret.json and save it to project root.
Second, run:
$ node authGoogleDrive.js

Visit the given url provided above. Get the auth code and paste it to console.
Then the program will save your google drive access_token locally at ./.gdrive_access_token.
Make sure you've also set GOOGLE_DRIVE_IMAGE_FOLDER = folderID in .env file.
Note : Program will skip uploading file, if you haven't set client_secret.json, .gdrive_access_token and GOOGLE_DRIVE_IMAGE_FOLDER.
generate client_secret.json

Sorry Github did not inform me of this pull requests QQ
Will start reviewing this in the following days

Will push to staging bot tonight (in Vienna time :P)

When deploying to Heroku, we cannot use credential files like .gdrive_access_token.
It seems that others will put such credential directly in environment variable (heroku config) directly as a JSON string.
I will modify the PR to do the same.

The change that removes the need for .gdrive_access_token:
https://github.com/cofacts/rumors-line-bot/commit/0ab7863f1edae9fde6cc5bba054c8691c8948da8

Deployed to staging.

@MrOrz
I really appreciate your help and will try to do better next time. m(_ _)m


Colorbar auto handling LabelMargin, introduced LabelIsVisible propery

Hi !
Please have a look, for sure there are things to fix and improve ;)
thanks very much! :)

Hi @bukkideme, thanks for this PR!
To be honest it took me a long time to figure out what it does 😅
Here's what I came up with to demonstrate what this PR does:
var cb = formsPlot1.Plot.AddColorbar();
cb.Label = "My Colorbar label\nis really\nreally big";

double[] fractions = { 0, .5, 1 };
string[] labels = { "0000000000", "1111111111", "2222222222" };
cb.SetTicks(fractions, labels);

formsPlot1.Plot.Layout(right: 200);
formsPlot1.Render();

Before

After


Doc Feedback: Topic links are not working

Location : https://apim.docs.wso2.com/en/3.0.0/Learn/APISecurity/APIAuthentication/secure-apis-using-basic-authentication/

Fixed in https://github.com/wso2/docs-apim/pull/735
Removed the topic links because they are redundant due to the TOC on the right side of the page.


Is it possible to filter the records for use in a multi-tenant environment?

We are looking at using this great sync tool, but one of our design requirements is to have the back-end be multi-tenanted.  Is there a way we can easily apply filters to ensure we do not update, add, get records in the wrong tenant?

Hi Mark!
I think you could either create a seperate database for every tenant and then create a datacontext pointing to the right database for every user
or
add a field "TenantId" to all the tables and filter by the TenantId of the user when pulling records in the controller.
When the user syncs the records also have to contain the field TenantId.
Let me know if you need any further infos!
kind regards


Try renovate automerge for minor/patch revisions

Given it passes checks

[x] Make sure to have UI test first (with CircleCI?) cause some of these compose dependencies breaks our UI but not unit tests

https://github.com/lydavid/MusicSearch/commit/87da4fb35748b1b3b570a7df74d08031fc5829a0


Searched term specific address opening

Is it possible now or could such a feature be added to Kiwix, so as, after configuring once the zim file, when any searchTerm specific address of the extension is visited (like moz-extension://4bdb0fa0-b097-4e0d-838d-c0bb566f3a71/www/term.html or moz-extension://4bdb0fa0-b097-4e0d-838d-c0bb566f3a71/www/index.html?title=A/term.html), the proper offline Wiktionary page to open (as happens by visiting https://en.wiktionary.org/wiki/term on the online Wiktionary)?
This is possible with some online Wiktionary extensions e.g. by visiting moz-extension://67f64f05-efc0-4366-8a29-de881b3aa9ad/main.html#input=term#English (Quick Dictionary extension).

The project is stalled since many years and has not reached the v1.0. Archiving the project. Please contact @julianharty if you are interested to work on the project.

The project is stalled since many years and has not reached the v1.0. Archiving the project. Please contact @julianharty if you are interested to work on the project.


Face Recognition

Python programs to capture a user's face and generate dataset. Then detect and recognize the user's face by using opencv.

I'm the Participant of GSSOC'21. Please assign me this issue under GSSOC'21.I would love to work on this.

@Dhanush2612  this has been already implemented, face recognition done by Microsoft Azure's Face API here


bug: Hello World CLI pipeline example does not work

Description
The provided example in the 4th query on the Hello World CLI public documentation does not work. This example:
# How many big purchases happen each hour and where?
let cadence = hourly()

# Anything can be named and re-used
let hourly_big_purchases = purchase
| when(Purchase.amount > 10)

# Filter anywhere
| count(window=since(cadence))

# Aggregate anything
| when(cadence)

# Shift timelines relative to each other
let purchases_now = count(Purchase)
let purchases_yesterday =
   purchases_now | shift_by(days(1))

# Records are just another type
in { hourly_big_purchases, purchases_in_last_day: purchases_now - purchases_yesterday }
| extend({
  # …modify them sequentially
  last_visit_region: last(Pageview.region)
})

To Reproduce
Steps to reproduce the behavior:

Follow the steps in the onboarding CLI hello world doc: https://kaskada.io/docs-site/kaskada/main/getting-started/hello-world-cli.html#querying-data

Actual Behavior
This error is returned:
Executing query...

state: STATE_FAILURE
config:
  dataTokenId: 6c2ac48f-6490-41cb-a146-23800d901562
analysis: {}
fenlDiagnostics:
  fenlDiagnostics:
  - severity: SEVERITY_ERROR
    code: E0006
    message: Unbound reference
    formatted: |+
      error[E0006]: Unbound reference
        --> Query:5:28
        |
      5 | let hourly_big_purchases = purchase
        |                            ^^^^^^^^ No reference named 'purchase'
        |
        = Nearest matches: 'Purchase', 'cadence'

  - severity: SEVERITY_ERROR
    code: E0006
    message: Unbound reference
    formatted: |+
      error[E0006]: Unbound reference
         --> Query:23:27
         |
      23 |   last_visit_region: last(Pageview.region)
         |                           ^^^^^^^^ No reference named 'Pageview'
         |
         = Nearest matches: 'cadence', 'Purchase', '$input', 'purchases_now', 'purchases_yesterday'

  numErrors: "2"
metrics: {}
requestDetails:
  requestId: 1dce10248b88c9b3a04bf6fe7fbec104

Expected Behavior
The query runs and I understand what's happening.
Additional context
N/A
Relevant Logs / Links
N/A

Note that the first error will be fixed by #328, but the 2nd error still needs to be looked into

Note that this issue is not specific to the CLI getting-started. It is also present in the python/jupyter getting started: https://kaskada.io/docs-site/kaskada/main/getting-started/hello-world-jupyter.html


fix spaces in quickorder form

## PR Type[x] Bugfix[ ] Feature[ ] Code style update (formatting, local variables)[ ] Refactoring (no functional changes, no API changes)[ ] Build-related changes[ ] CI-related changes[ ] Documentation content changes[ ] Application / infrastructure changes[ ] Other: ## What Is the Current Behavior?Heading of quickorder form table isn't in line to the quantity fields. Between form elements is to much space.## What Is the New Behavior?The form elements stand closer together, so the quantity field is in line with its heading.## Does this PR Introduce a Breaking Change?[ ] Yes[x] No## Other Information
AB#66041

@cbratsch I cannot review your pull request. The demo server is not working (504 Gateway Time-out) :(

@M-Behr  Review server is working again

I've got a link for you: http://pwa-gh-review-832-universal-b2b.azurewebsites.net/home;theme=blue


Update JDK versions to 9-ea+142

Bumped the MacOSX and Linux Java SE downloads to latest version available on https://jdk9.java.net/jigsaw/ - version 9-ez+142

Sorry, I missed a change required in the path.  I will close this pull request and add the download change to my other pull request


Fix tests

Some docker tests fail with invalid tag format
https://github.com/exasol/script-languages-container-tool/actions/runs/7963849164/job/21740242571

Related to

https://github.com/exasol/integration-test-docker-environment/pull/395
https://github.com/docker/docker-py/issues/3223


Make a Navbar.

Make a responsive navbar , make sure you make it look good and attractive.

Preffered Background Color : #ddf7e4
Navbar Brand : Milan
Elements : Home, clubs, events, About Us, Profile Icon
Link the elements to respective pages

Could i try this?

Hello. If you assign me this issue, I will create a responsive navbar. Thanks.

hey, I can work on this issue

hey, I can work on this issue

Please take a look at the Readme.


category gets descendants in filters

I want to merge this change because...

Screenshots

Pull Request Checklist

[ ] Privileged views and APIs are guarded by proper permission checks.
[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Database queries are optimized and the number of queries is constant.
[ ] The changes are tested.
[ ] The code is documented (docstrings, project documentation).
[ ] GraphQL schema and type definitions are up to date.

@pitero92 Thanks for opening the PR! We care not only about the quality of our code but also about the quality of all the written part on Github 😉. It would be great if you could:

Edit the issue description with an explanation of the problem it fixes and concerning the issue it resolves (if there is one). You should use Github syntax for closing issues.

In case of any question feel free to ask. Thanks!

@maarcingebala Thank You for your feedback.
I've already corrected what you asked for.
It was my first PR, I hope that with next one will go better.

Closing due to inactivity.

Continued in #3535


Added conditional rendering for admin search

Short Description
Added conditional rendering for admin search so only admins have access (Temporary work around for demo)
Rationale/purpose of the feature
Prevent non admins from accessing admin dashboard search
Link to relevant User Story
Flag admin user in local storage to conditionally render Admin Search
Optional: Screenshot(s) of intended state of feature, wire-frame(s), flowchart
Optional attachments here
Checklist for author and reviewers

[ ] Commits are well named and descriptive
[ ] PR is focused on a single problem, is small enough and easy to review
[ ] PR requests were sent to multiple reviewers
[ ] PR is reviewed by more than one team member
[ ] Code is consistently formatted
[ ] Code is suitably commented on
[ ] There is no duplicate code
[ ] Code tries to follow best practices
[ ] Decide with reviewers if this PR should have tests before merging

I thoroughly enjoyed working on this together. What a jungle that was, but we came up with a solution and delivered! I won't approve because we know this is just for demo purposes, but this was a great experience working with a codebase that we didn't write and providing solutions for bugs that we can't access. Great job, thank you so much for all of your input and feedback while we worked through the various challenges.

I enjoyed the process of working on this as well. It was great experience for dealing with complex code bases and working out how to get the functionality that we were looking for. It's challenges like this that take the monotony of day to day work away and make what we do for a living fun! You are very welcome! It was a pleasure being able to have this experience and contribute to solving a major issue in the code base. Thank you so much for your patience and your input while solving this issue. It was a real pleasure working with you!


Add option to API to update PullRequest base branch (#11666)

Backport: #11666
EditPull: add option to change base
Close #11552

Make lgtm work


add 3 books

Thanks for the rebase.
These are superb additions!
Here are the issues caught by the linter (from travis output)
662:3-662:90  warning  Missing PDF indication                       books-links        remark-lint
799:1-808:97  warning  Alphabetical ordering: swap l.808 and l.807  alphabetize-lists  remark-lint
808:3-808:97  warning  Missing PDF indication                       books-links        remark-lint
While you're at it, consider adding author names. see #2383


Remove rapids_cpm_thrust and rapids_cpm_libcudacxx.

Description
This PR removes the deprecated functions rapids_cpm_thrust and rapids_cpm_libcudacxx.
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[x] The cmake-format.json is up to date with these changes.
[ ] I have added new files under rapids-cmake/

[ ] I have added include guards (include_guard(GLOBAL))
[ ] I have added the associated docs/ rst file and update the api.rst

This PR isn't complete yet -- I need to come back to it another time. Too much on my plate at the moment. I'd be happy if someone wants to adopt this PR and fix the remainder of the tests.

This PR isn't complete yet -- I need to come back to it another time. Too much on my plate at the moment. I'd be happy if someone wants to adopt this PR and fix the remainder of the tests.

I will adopt this PR

Closing as https://github.com/rapidsai/rapids-cmake/pull/665 replaces this


how to use my translation files for mat-datepicker

I want to translate my calendar into a language that is not in LOCALE
I would like to use my files with translation
Could you help me please

Closing this issue as it does not follow the issue template. Please file a new issue that follows the template, which gives the team the information needed to investigate.


Infer tract names from nodes.csv

This pull request resolves #49 by adding another queue await call to read in nodes.csv and populate var tracts using the data therein. This change is located in tract-details. It also moves the color definitions into a separate file tract-colors.js so that we can easily change color palettes in the future. Lastly, it removes the canvas-loader library from index.html. This library was left over from a previous attempt to create a loading page and is no longer needed.

LGTM. I'll wait for our meeting tomorrow to merge.

Just pushed another commit to resolve #16

Pushed another commit to remove obsolete debug printing of 3d panel width and height.


coprocessor: make the error format the same as tidb

What is changed and how it works?

Issue Number: Close #16407

What's Changed:
When the result of pow is out of range, the error message in TiDB looks like this
DOUBLE value is out of range in 'pow(-36452.389, 382218415)'

But in TiKV, the error message is
DOUBLE value is out of range in '-36452.389.pow(382218415)'

It may make some tests fail, this pr change the error format in TiKV
Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

make the error format the same as tidb when the result of `pow` is out of range

/check-issue-triage-complete

/merge


Nav drawer icon problem in fragment replace

I was using replace the container with fragment.
I want when replace the fragment, the nav drawer icon from hamburger change to back arrow, when user on back press, the nav icon turn back to hamburger. Below is my code.
public void setActionBarArrowDependingOnFragmentsBackStack() {
    int backStackEntryCount = getActivity().getSupportFragmentManager().getBackStackEntryCount();
 if(backStackEntryCount >= 1) {
       ((ActionBarActivity)getActivity()).getSupportActionBar().setDisplayHomeAsUpEnabled(true);
	}
   }

It should turn back to hamburger icon when the backstackentrycount less than 1. But it not able to turn back. Is that i miss out any code? Thanks a lot if you can help me.

Try this answer http://stackoverflow.com/a/26755373/3410697
You will need to call syncState as far as I know

i have write the syncState inside the function. it was like
if (backStackEntryCount == 0) {
 mActionBarDrawerToggle.syncState();
}

It able to function when user click the 1st back button. But if user open onbackStack again, the setDisplayHomeAsUpEnabled was unable to work. It still display hamburger icon. Although the backStackEntryCount is more or equal to 1. It feel weird with the code just able work on 1st time.

Please check if your backstack is really getting popped.
Also make sure you're calling ((ActionBarActivity)getActivity()).getSupportActionBar().setDisplayHomeAsUpEnabled(false); before the syncstate because if you're setting it to true and never turn it back to false you will never get it right the second time you click.
I will create 2 methods to show back button and show drawer button so it can be simpler to other folks.
In the meanwhile check this answer too for a better explanation:
http://stackoverflow.com/a/26671641/3410697

thanks a million kanytu! i was not calling false on setDisplayHomeAsUpEnabled, was searching for about an hour and a half for the solution. Thanks man


[DO NOT MERGE] 0.10.0 release patch for flink

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

will merge to master (release-0.11) or release-0.10.1 ?

will merge to master (release-0.11) or release-0.10.1 ?

release 0.10.0 would include these fix.

closing this out.


attr('propName', function(d){}.bind(this)) error

xxx.attr('x', d => (d))
or
xxx.attr('x', function(d){
return d;
}.bind(this));
both is error, so how can I bind this;

How is xxx defined here? What is the error exactly?

If the first is an error, it’s probably because the browser you are using doesn’t support ES2015 fat arrow functions.
The second is a syntax error. You need to wrap the function in parentheses to use function.bind in that way:
xxx.attr('x', (function(d) { return d; }).bind(this));

Of course, if you’re not planning on using the bound this, then there’s no reason to use function.bind.
If you have further questions, please use Stack Overflow tag d3.js to ask for help. Although I make an effort to assist everyone that asks, I am not always available to provide help promptly or directly. Stack Overflow provides a better collaborative forum for self-help: tens of thousands of D3-related questions have already been asked there, and some answered questions may be relevant to you.
When asking for help, please include a link to a live example that demonstrates the issue, preferably on bl.ocks.org. It is often impossible to debug from code snippets alone. Isolate the issue and reduce your code as much as possible before asking for help. The less code you post, the easier it is for someone to debug, and the more likely you are to get a helpful response.
If you have a question about D3’s behavior and want to discuss it with other users, also consider the d3-js Google Group or joining the d3-js Slack.
Thank you! 🤗


Configure multipass as a remote LXD host to simplify access through LXD client on Mac and windows

What about the case when there's no LXD client installed? Original idea was not to be dependant on local client. If this is needed it should be implemented as a separate use case.


Usage of internal?

There are a lot of core runtime packages for openfga under the internal path. Why are those packages there and not part of pkg? It seems to be the opposite of "open"

Hi @mxplusb
In general we try to follow the Go guidelines towards internal, and try not to put too many things outside internal b/c we don't want people to assume that the interfaces are stable.
However if there are particular functionalities you'd like exposed, let us know and we can either move them out as is or work on an interface that we believe is more stable for external use. PRs are also always appreciated, but let us know what you are planning to do before spending the effort, just in case we have other options in mind.

I’ve worked with Go for a long time and understand internal, thanks for the refreshers since it had been awhile 🙂
For my use case, I’m evaluating embedding OpenFGA in my database so I can provide native FGA functionality at the lowest levels of data operations. The embedding would essentially be a complete rewrite of the OpenFGA runtime architecture to fit into my database’s runtime architecture, and I would also be providing my own storage implementation for the graph. I know I would be taking on a ton of ownership if I went that route, so I was looking through the OpenFGA code base and wanted to ask about the internal usages because it felt like stuff was inconsistently placed there.
Ory’s Zanzibar impl, Keto, puts almost everything in internal and that really took their implementation off the table for me. I like OpenFGA because it’s less of that, and it feels less “corporate” (even though I know there’s corporate backing for the project). Is y’all’s usage of internal focused on safe early iterations of unstable APIs, or is there a long-term plan for putting more stuff in it? If I decide to embed OpenFGA, would I risk losing access to the existing libraries in pkg in later releases? Should I create a one-time fork and then port updates to the internal libraries as needed?
Personally, partially because of Keto’s egregiously offensive usage of internal, I’m finding the landscape for FGA solutions is very limited, and embedding what exists is very risky. The only path forward for my use case is embedding because I can’t take an external dependency on another database cuz I’m building a database 🤷🏼♀
Whatever y’all end up using internal for is, of course, up to y’all. If my use case is something y’all are interested in supporting, keeping very limited things in internal so I can consume OpenFGA exclusively as a library that would be super helpful. I’ll hard fork and port updates if my use case is out of scope, it’ll just be more effort for me. That being said, if there’s interest and alignment in supporting my use case, I’m happy to donate and maintain any authz schemas for my database as reference templates (once I get that far) to OpenFGA‘s code base.
Happy to answer questions y’all might have.

For example, we have a public contract with the server and storage and typesystem packages (among a few others), and this is because we need those imports in other projects. If another team were to want to embed OpenFGA as a library, then they would need the same imports from pkg but not from internal.

I missed this nuance before I replied, it’s pretty much exactly what I want to do! If you can keep everything but the CLI/server runtime code in pkg, that would be amazing. Personally, I’ll need the storage interface and the graph SDK as a starting point.

If you can keep everything but the CLI/server runtime code in pkg, that would be amazing. Personally, I’ll need the storage interface and the graph SDK as a starting point.

@mxplusb Our intent is for the Server to be imported if you need the RPCs to serve queries and write/read things. From the sounds of your use case, that should suffice, because it allows you to provide any implementation of the storage interface and it will allow you to embed it wherever you need in your Go app. We don't intend to have a public contract with the interna/graph package symbols at this time, and so if you are referring to that package and have a need for the deeply internal graph resolution code, then it's unlikely we'll make that public unless we have a compelling reason to do so.
Could you elaborate more about your use case and why you need access to the internal graph package?

Could you elaborate more about your use case and why you need access to the internal graph package?

Sure. I'm providing high-level details, but I'm not sure how much detail you want, so let me know if you have more questions.
My use case is to provide complex authz for a distributed, monolithic, multi-trillion key value store. The kv store itself has a relatively simplistic interface, but it's the platform for complex database storage structures, such as graphs with Gremlin, distributed SQL, RDF triple stores, and other complex data storage structures. To ensure the kv store platform can model complex authz relationships to support a complex set of data structures, I've identified FGA as a foundational technology to provide complex authz support. The requirements for embedding are fairly low: a storage interface which can be implemented, and the underlying graph library.
The storage interface is a requirement because the backend is not compatible with off-the-shelf solutions, and I can't take an external system dependency. The storage implementation is a sharded kv store so it can store the graph across multiple nodes in my kv store cluster. This allows the storage implementation to have centralized writes while having distributed yet linearizable reads, meeting the scalability requirements.
Direct access to the underlying graph library is needed because the app framework needs access to manage the lifecycle, manage the storage implementation, and operate the graph without exposing it to end users. By exposing the graph directly to users through RPC, it creates risks for data integrity/security/compliance, and I don't want to introduce that kind of risk.
Nearly all calls to the graph will be internal to each node as it will primarily be used to check kv store access on reads and writes - with trillions of keys the kv store can't make network hops or serializations on every read or write. Any RPCs for graph management that would be needed will be specific to the kv store, and the platform uses a mixture of HTTP/3 + QUIC.

I did dig deeper into the server structure to see what the opportunities are there. I did notice y'all separated the graph operations into pkg/server/commands. From what I can tell, that library is closest to being able to meet my requirements, but it relies heavily on checks, validations, and other bits and pieces from pkg/server, which takes a hard dependency on internal/{gateway,graph,validation}.

We don't intend to have a public contract with the interna/graph package symbols at this time, and so if you are referring to that package and have a need for the deeply internal graph resolution code, then it's unlikely we'll make that public unless we have a compelling reason to do so.

That code is already public, it's just not available through go get. To say, "it's unlikely that we'll make that public unless we have a compelling reason to do so" really sounds like it's not actually OpenFGA, but PublicFGA. Is this decision because there's actually too much instability in the API, or because it creates the risk other companies might create their own FGA offerings to compete with Auth0 & Okta using work from said companies' engineers & PMs?
If the real concern is there is real instability in the internal packages, then I would argue the instability be documented and versioned so devs can make their own choices. I'm happy to submit a PR to make those APIs public, and document any instabilities they might have for downstream consumers.

@mxplusb thanks for the detailed overview. I've taken a couple of the excerpts and responded inline.

The storage interface is a requirement because the backend is not compatible with off-the-shelf solutions, and I can't take an external system dependency. The storage implementation is a sharded kv store so it can store the graph across multiple nodes in my kv store cluster.

Awesome, that shouldn't be an issue. You can implement the OpenFGADatastore interface in pkg/storage for your kv store in a way that meets your requirements for consistency, latency, and availability.

Direct access to the underlying graph library is needed because the app framework needs access to manage the lifecycle, manage the storage implementation, and operate the graph without exposing it to end users. By exposing the graph directly to users through RPC, it creates risks for data integrity/security/compliance, and I don't want to introduce that kind of risk.

Gotcha. If you embed the OpenFGA Server (e.g. pkg/server) in your application, you don't have to expose the RPCs of the OpenFGA server structure to users. You can delegate calls to the RPC stubs of the server without exposing them publicly in any way. Let's pretend you're implementing a server for your application and you have a public HTTP handler.. The following code demonstrates how you can use the OpenFGA server embedded without exposing any of the underlying OpenFGA RPC queries to the end user:
type MXPlusBServer struct {
    openfga *openfga.Server
}

func (s *MXPlusBServer) SomeHTTPHandler(w http.ResponseWriter, r *http.Request) {

    checkResponse, err := s.openfga.Check(r.Context(), &openfgapb.CheckRequest{...})
    if err != nil {
        // handle error
        w.Write([]byte("internal error"))
        w.WriteHeader(http.StatusInternalServerError)
        return
    }

    if checkResponse.GetAllowed() {
        // request is authorized, do something
       return
    }

    w.Write([]byte("unauthorized"))
    w.WriteHeader(http.StatusUnauthorized)
}

and you can construct the OpenFGA server with your custom storage implementation. For example:
import (
    openfgastore "github.com/openfga/openfga/pkg/storage"
    openfga "github.com/openfga/openfga/server"
)

var mxplusbKvStore openfgastore.OpenFGADatastore

openfgaServer := openfga.New(
    &openfga.Dependencies{Datastore: mxplusbKvStore, ...},
    &openfga.Config{...},
)

Nearly all calls to the graph will be internal to each node as it will primarily be used to check kv store access on reads and writes - with trillions of keys the kv store can't make network hops or serializations on every read or write. Any RPCs for graph management that would be needed will be specific to the kv store, and the platform uses a mixture of HTTP/3 + QUIC.

If I understand this correctly you're saying that you cannot afford to use the OpenFGA server RPCs because of the overhead of serialization to/from the storage layer as you resolve the graph query and the network hops to/from the storage layer is also not acceptable on every read/write? In which case, what is the plan for evaluating the graph with persistent data?

That code is already public, it's just not available through go get. To say, "it's unlikely that we'll make that public unless we have a compelling reason to do so" really sounds like it's not actually OpenFGA, but PublicFGA. Is this decision because there's actually too much instability in the API, or because it creates the risk other companies might create their own FGA offerings to compete with Auth0 & Okta using work from said companies' engineers & PMs?

I think this is a matter of personal perception.. If code is publicly available on the internet and carries with it a permissive license (e.g. Apache 2.0), that to me it is highly open (from a legal perspective). You can create derivative works off it without almost any hassle given the permissiveness of Apache 2.0. To say that source code inside of an internal package is not "open" is a misrepresentation of the permissive nature of Apache 2.0 licensed software. The code may not be importable, but that is by design, and this is for the same reason that other developers make use of features of other languages for package scoping (e.g. in Java we have private/protected/public). If we don't intend to have symbols publicly consumable, that is not because we are trying to protect ourselves from competition in the market (otherwise we wouldn't make it publicly available on the internet 😄 ), but that is because we don't see a strong use case for making it public and we don't want the internet to build contracts on it if we do not intend to maintain said contracts.  Hyrum's law says

With a sufficient number of users of an API, it does not matter what you promise in the contract: all observable behaviors of your system will be depended on by somebody.

We're trying to reduce that blast radius by using package scoping mechanisms like so many other languages do. That's all.
There are handfuls of open-source projects that no one would argue are not "open" and they make use of the internal package pattern. Here are a few examples:

https://github.com/goreleaser/goreleaser
https://github.com/traefik/traefik
https://github.com/redis/go-redis
etc..

Again, we're open to the idea of moving any code out of internal into pkg, but we just want to see more community reasons to do so before we go making something a public part of our API contract. And before we move anything from internal to pkg we'd probably want to do our due diligence to cleanup the interfaces etc..
Could we schedule a quick call to discuss further? It may help to get in person.

If you embed the OpenFGA Server (e.g. pkg/server) in your application, you don't have to expose the RPCs of the OpenFGA server structure to users. You can delegate calls to the RPC stubs of the server without exposing them publicly in any way.

Is your implementation of pkg/server.Server not bundled with a network implementation? It looked like it was but maybe I misunderstood the architecture. If it's not bundled with a network implementation, it very well might meet my needs. I want to bring my own storage and skip the networking, so as long as your server implementation lets me do that, then it might be sufficient.

If I understand this correctly you're saying that you cannot afford to use the OpenFGA server RPCs because of the overhead of serialization to/from the storage layer as you resolve the graph query and the network hops to/from the storage layer is also not acceptable on every read/write?

Not quite. The generic flow is {Get,Put}KeyHandler -> GlobalKeyStore -> ReplicationEngine -> LocalNodeKeyStore -> {Get,Put}KeyFromStorage. I'm wanting to introduce OpenFGA between {Get,Put}KeyHandler and GlobalKeyStore so authz can be verified before there's network roundtripping or data storage operations.
Because this is a hot path for data operations, I'm not able to do is make an external network request, even an RPC call, to an external system between GetKeyHandler and GlobalKeyStore. An external network request introduces unnecessary overhead and it creates an external dependency. But, if your pkg/server.Server implementation doesn't require networking, then I'll definitely take a look!

In which case, what is the plan for evaluating the graph with persistent data?

Can you clarify the question? I'm not following, sorry. Are you asking about performance impacts for trying to resolve the relationships of {m,b,tr}illions of keys?

Hyrum's law says

Fair point.

Again, we're open to the idea of moving any code out of internal into pkg, but we just want to see more community reasons to do so before we go making something a public part of our API contract. And before we move anything from internal to pkg we'd probably want to do our due diligence to cleanup the interfaces etc..

Another fair point.

Could we schedule a quick call to discuss further?

If you'd like. My email is in my bio (not gonna paste it cuz bots), I'm generally available most days after 3p MT.

@mxplusb

Is your implementation of pkg/server.Server not bundled with a network implementation? It looked like it was but maybe I misunderstood the architecture. If it's not bundled with a network implementation, it very well might meet my needs. I want to bring my own storage and skip the networking, so as long as your server implementation lets me do that, then it might be sufficient.

The only network involved in the RPC stubs is if the storage layer involves network. If you pass in a storage implementation to the Server struct that does not do any network hops, then no network will be involved. A good example of this is our memory storage adapter in pkg/storage/memory. For example
import (
  openfga "github.com/openfga/openfga/pkg/server"
  openfgamem "github.com/openfga/openfga/pkg/storage/memory"
)

openfgaServer := openfga.New(
  &openfga.Dependencies{Datastore: openfgamem.New(...), ...},
  &openfga.Config{...},
)

result, err := openfgaServer.Check(...) // no network is used since we're using the memory storage adapter

Ahhhh that was the part I didn't know, thank you for helping me get there!
I will experiment with it and provide some feedback.

Note that in the next release, the syntax will be simplified to
openfgaServer := openfga.NewServerWithOpts(
	openfga.WithDatastore(datastore),
)

datastore is the only required parameter, everything else will be optional.

@mxplusb I'm going to close for now as it seems we have a path forward. If you find otherwise, please feel free to reach back out and we can re-open and setup a call etc.. 👍

Ahhhh that was the part I didn't know, thank you for helping me get there!

We should document in the readme or somewhere in the docs that this project can also be used as a library without the networking baggage

Actually, right now, you shouldn't use the project as a library because you will lose the validation layer that is currently implemented as a server middleware 😬

Thank you for the head's up!!


fix: remove blinking in SwipeToConfirm

Description
Prevent icon to be shown in SwipeToConfirm until loading is complete
Now everything is smooth
Linked Issues/Tasks
closing #2162
Type of change
Bug fix
Tests
No test
Screenshot / Video
https://github.com/user-attachments/assets/134296b3-f1dc-44d3-8516-c45597a0bbbf
QA Notes

Verified in v1.0.4 https://github.com/synonymdev/bitkit/commit/6404b079a1dd23f079515b3d7a14865baf959914
Boost swipe is working just fine.


Add delete and display bookmark functionality

Aim
This PR encompasses

Adding a "Save filter" button to the case filter panel which will save the filter with a default name "New unnamed filter"
Adding a Saved filter expandable panel which will have the filters displayed to click on.
By each saved filter, an Edit button which will overlay an input field to change the name of the filter and Save and Cancel buttons.
By each saved filter, a delete button for deleting the filter.

LTD-3665

We could use some JS tests since we have been trying to add them whenever JS is involved but yea looking good.