Helm Chart loop monitor sidecar

Added loop inside sidecar cilium-monitor DeamonSet. It's necessary because when cilium agent is starting it takes some time to create socket and sidecar is exit with error
$ kubectl logs cilium-smjv6 cilium-monitor
Press Ctrl-C to quit
time="2022-04-08T07:40:37Z" level=error msg="Cannot open monitor socket" error="Cannot find or open a supported node-monitor socket. /var/run/cilium/monitor1_2.sock: dial unix /var/run/cilium/monitor1_2.sock: connect: no such file or directory"

And the status of cilium-agent is the following:
cilium-smjv6   0/2     CrashLoopBackOff   1 (10s ago)   13s
cilium-smjv6   0/2     Completed          2 (16s ago)   19s
cilium-smjv6   0/2     CrashLoopBackOff   2 (12s ago)   30s
cilium-smjv6   0/2     CrashLoopBackOff   2 (22s ago)   40s
cilium-smjv6   1/2     CrashLoopBackOff   2 (22s ago)   40s
cilium-smjv6   2/2     Running            3             41s

It spent 3th restart sidecar until the socket is up and running

/test-runtime

/test-gke

/test-gke


Added Example property to OpenApi attribute

Added new Example property to the OpenApiPropertyAttribute. Allows settings an example value at an individual property level rather than only at the class/model. This is included in the OpenApi schema for the property and displays in the swagger schema and example.
Follows the same format as the Default property, i.e. allows string, number, boolean.
Example:
[OpenApiProperty(Description = "The name of the pet", Example = "Dog")]
public string Name { get; set; }

Property schema includes the example:

Model/schema example value also includes the property example:

Is this PR likely to be reviewed any time soon?
I have colleagues who would like to submit similar contributions for elements like "format", "minimum", "maximum", etc.
At the moment there are too many things missing compared to what can be manually produced in SwaggerHub, and that feature gap is delaying our adoption of the APIM Developer Portal as a means of publishing APIs.

Very much interested in this as well!


fix: update Heading component built in styles [ALT-154]

enables more built in style options in the design sidebar for the default  Heading component
changes the logic in useComponentProps that splits css styles into componentStyles vs editorWrapperStyles

structure components and components with children enabled should pull the height/width/maxWidth out of the component element styles and move those style props to the editor wrapper (draggable) element
other components will keep the height/width/maxWidth style props on the component element

I think we should think about applying the following props to the wrapper draggable component instead of the underlying component

I pushed updates to split those three data-cf-node props into a new wrapperProps object.
I'm not sure if that was expected to fix the styling problems for nested containers and such, but it doesn't look like it did so I need to keep the other changes for editorWrapperStyles which solve those issues.

I think we should think about applying the following props to the wrapper draggable component instead of the underlying component

@chasepoirier - I pushed updates to split those three data-cf-node props into a new wrapperProps object.
I'm not sure if that was expected to fix the styling problems for nested containers and such, but it doesn't look like it did so I need to keep the other changes for editorWrapperStyles which solve those issues.

I pushed additional updates to the useComponentProps hook to fix the Heading block width settings to behave consistently in Editor mode vs Preview/Delivery mode.
I think the changes landed in a better spot with that update (less of the ternary logic for structure component vs not).
Now all components will split the editor styles vs component styles in the same way (which is basically to move the margin, width, maxWidth and height to the editor wrapper (draggable) element, and normalizing these styles on the component element to fill the wrapper space (margin: 0; width: 100%; height: 100%; maxWidth: none)

I pushed additional updates to the useComponentProps hook to fix the Heading block width settings to behave consistently in Editor mode vs Preview/Delivery mode.
I think the changes landed in a better spot with that update (less of the ternary logic for structure component vs not).
Now all components will split the editor styles vs component styles in the same way -- basically after moving the margin, width, maxWidth and height to the editor wrapper (draggable) element, the styles are adjusted on the component element to fill the wrapper space (margin: 0; width: 100%; height: 100%; maxWidth: none)

Nice I like this solution a lot, good work!


Error: Invalid count argument

I am trying to deploy a Big-IP VM using v1.1.2 of this module and Terraform 1.0.2 but I'm getting invalid count argument errors when I try to plan or apply:
│ Error: Invalid count argument
│
│   on .terraform/modules/bigip/main.tf line 149, in resource "google_project_iam_member" "gcp_role_member_assignment":
│  149:   count   = var.gcp_secret_manager_authentication && var.service_account == "" ? 1 : 0
│
│ The "count" value depends on resource attributes that cannot be determined
│ until apply, so Terraform cannot predict how many instances will be
│ created. To work around this, use the -target argument to first apply only
│ the resources that the count depends on.
╵
╷
│ Error: Invalid count argument
│
│   on .terraform/modules/bigip/main.tf line 156, in resource "google_project_iam_custom_role" "gcp_custom_roles":
│  156:   count       = var.gcp_secret_manager_authentication && var.service_account == "" ? 1 : 0
│
│ The "count" value depends on resource attributes that cannot be determined
│ until apply, so Terraform cannot predict how many instances will be
│ created. To work around this, use the -target argument to first apply only
│ the resources that the count depends on.

Unless I use GCP secret manager it doesn't seem that I'm able to work around this problem (I would rather not use it just for Big-IP as we don't use it for anything else). You can find my Terraform config here: https://gist.github.com/scrossan/42b108778adfdec3eba99a91b9c65d9f

Created [INFRAANO-727] for internal tracking.

@scrossan is this issue reproducible, i used module like below, not noticed any issues. please let me know if i miss anything.
module "bigip" {
  source          = "F5Networks/bigip-module/gcp"
  version         = "v1.1.2"
  prefix          = format("%s-1nic", var.prefix)
  project_id      = var.project_id
  zone            = var.zone
  image           = var.image
  service_account = var.service_account
  mgmt_subnet_ids = [{ "subnet_id" = google_compute_subnetwork.mgmt_subnetwork.id, "public_ip" = true, "private_ip_primary" = "" }]
}

I am facing the same error. Below is the module I am using.
module bigip {
count           = var.bigip.count
source          = "F5Networks/bigip-module/gcp"
prefix          = var.bigip.prefix
project_id      = var.project.id
zone            = var.bigip.bigip_zone
image           = var.bigip.image
service_account = "${google_service_account.bigip.email}"
mgmt_subnet_ids = [{ "subnet_id" = lookup(var.standalone_vpc_subnets_cidr, "public_resources"), "public_ip" = true, "private_ip_primary" = "" }]
}
Can you please provide the details for the subnet that you have used. I am using a CIDR  range in the "subnet_id" attribute in "mgmt_subnet_ids".

@scrossan is this issue reproducible, i used module like below, not noticed any issues. please let me know if i miss anything.

I notice that you're using a variable to set service_account but @smitha-ap and I are trying to interpolate the email from a google_service_account resource. I did find a workaround - target the service account and then you can apply the module since the service account exists in state at that point.
Please look at my gist for a sample of code that can reproduce this error. In the module currently you can't apply the service account and the module at the same time, the service account has to exist first.
Ideally Terraform would be able to work out that the module depends on the service account and ensure that the service account is created before creating the module, but due to Error: Invalid count argument Terraform just exits instead.


How to capture 403 pull replication errors

Hi there! Sometimes I get differents pull replication errors, like Sync: {error=forbidden, id=027a5148-73e5-4893-9d29-80f1b216489b, reason=missing channel access, status=403}: _bulk_docs got an error: com.couchbase.lite.replicator.PusherInternal. I need to capture theese events to retry the save after x seconds. Pull replicator has addChangeListener method, but I don't receive these error in that listener. I'm only able to see these errors in the logcat. Is there any way to capture these errors?

Hi @vbenso,
403 is a permanent error.  The error should be notified through ChangeListener. Can you check the return value from ChangeEvent.getError() method?
https://github.com/couchbase/couchbase-lite-java-core/blob/master/src/main/java/com/couchbase/lite/replicator/Replication.java#L676

Hi @hideki, I have log every changeevent that arrives to changeEventListener, but I dont receive any 403 nor 401 change event. For example, when my session is expired, I can see in the logcat a lot of 401 errors:
ChangeTracker{http://couch.hablemos.com:4999/hablemos_test_dev_02, OneShot, @18eba06}: Change tracker got error 401
ChangeTracker: ChangeTracker{http://couch.hablemos.com:4999/hablemos_test_dev_02, OneShot, @d1738c7}: Change tracker got error 401
ChangeTracker: ChangeTracker{http://couch.hablemos.com:4999/hablemos_test_dev_02, OneShot, @783fb08}: Change tracker got error 401
ChangeTracker: ChangeTracker{http://couch.hablemos.com:4999/hablemos_test_dev_02, OneShot, @9de82a1}: Change tracker got error 401
But I don't receive any changeEvent with that error. I do receive other change events with different replication status (IDLE, STOP..).
I'm using 1.4.0 version.

Hi @vicpinm,
We need to reproduce the problem. To do it, we need some information.

What server do you use? Sync Gateway or CouchDB?  What version?
Scenario how to reproduce the problem. Middle of oneshot (non-continuous) pull replication, logged in account lose the access to channel?

Thanks!

Hi @hideki
We use Sync Gateway version 1.3. Is there any difference if we use CouchDB? I don't have clear the difference.
The problem appears with continuous replication, in a logged account which tries to replicante an object throw a channel in which I don't have permissions, so I get a 403 error. That is right, the problem is that I need to be notified of that event. I can see a locat line with that error:
Sync: {error=forbidden, id=027a5148-73e5-4893-9d29-80f1b216489b, reason=missing channel access, status=403}: _bulk_docs got an error: com.couchbase.lite.replicator.PusherInternal.
But I am not notified with a change event in my change listener. The same problem appears when my session is expired. I can see several 401 errors in logcat, but my listener is not invoked with that error.

Any news on this?

Hi @hideki
I have managed to solve the issue with 401 errors by cloning the project and adding
if (tracker.getLastError() != null) {                         setError(tracker.getLastError());                     }                     else {                         setError(new Throwable(msg));                     }
before this line: https://github.com/couchbase/couchbase-lite-java-core/blob/e276993f4fdd4beced1aea942b6a266eb848c648/src/main/java/com/couchbase/lite/replicator/PullerInternal.java#L939

Any updates with this?

Notes:
CBL iOS implementation: Pull replicator obtains ChangeTracker's error variable from ChangeTracker when ChangeTracker stopped. But CBL Android implementation does not.
https://github.com/couchbase/couchbase-lite-ios/blob/86c63659b56d87406eac3b1e4db631fe67098139/Source/CBLRestPuller.m#L270-L281
// The change tracker reached EOF or an error.
- (void) changeTrackerStopped:(CBLChangeTracker *)tracker {
    if (tracker != _changeTracker)
        return;
    NSError* error = tracker.error;
    LogTo(Sync, @"%@: ChangeTracker stopped; error=%@", self, error.my_compactDescription);

    BOOL continous = _changeTracker.continuous;
    _changeTracker = nil;

    if (error) {
        if (CBLIsOfflineError(error))
            [self goOffline];
        else if (!self.error)
            self.error = error;
    }

    [_batcher flushAll];
    if (!continous)
        [self asyncTasksFinished: 1]; // balances -asyncTaskStarted in -startChangeTracker
    if (!_caughtUp)
        [self asyncTasksFinished: 1]; // balances -asyncTaskStarted in -beginReplicating
}

https://github.com/couchbase/couchbase-lite-java-core/blob/master/src/main/java/com/couchbase/lite/replicator/PullerInternal.java#L922-L967
Solution: In case ChangeTracker is stopped with an error, pull replicator obtains error from ChangeTracker, and handles it accordingly.

Ref: CBSE-4462

Hi @vicpinm and @sebastianmacarescu,
We build the developer build with the fix for this issue. The solution is almost same with @sebastianmacarescu's suggestion. If you have time, could you please try the developer build?
The version number is 1.4.2-80. Please refer the following link for installation.
Instruction:
https://github.com/couchbase/couchbase-lite-android#adding-couchbase-lite-to-your-gradle-project
Couchbase custom maven repo:
http://files.couchbase.com/maven2/com/couchbase/lite/couchbase-lite-android/1.4.2-80/

Hi @hideki, I just realized that I made a mistake when I wrote the issue. The problem is in the PUSH replicator, not pull replicator, as I happens when I save a new document, and I get a 403 error after that. I thought that error was notified through the pull replicator, but I think I was wrong. Sorry about that :(

NOTES: How to reproduce this error.

Create ACL doc (which gives access to user to certain channel)
Write doc to that channel

In this scenario, if 2 is processed by SG before 1, then a 403 error is thrown because ACL access rule has not yet been created.

@vicpinm,
The fix, to remove the lines to ignore 401 or 403 error for push replication, causes the behavior change. It could break the applications that rely on current behavior. So we can not fix in short period of time. We will fix this for 1.5.0 release.


fix: Use prompt by task_type

Previously, prompts were selected for each task by task_name, but now they can be selected by both task_name and task_type. I tested ai-forever/ru-en-RoSBERTa, and the results matched the leaderboard. I think that model_encode can now take prompts and pass them to the model.
Checklist

[X] Run tests locally to make sure nothing is broken using make test.
[X] Run the formatter to format the code using make lint.

Any reason to not extract it from the task_name?
task = mteb.get_task(task_name)
task_type = task.metadata.type

I also think the current six-step flow is a bit complicated, might be worth documenting it in the docstring not only in model_encode function but also in the interface

That's great idea

I created a separate class for wrapping models and removed the encode_corpus and encode_queries methods #1284

@Samoed is this ready for review?

Yes, PR ready for review, but I runned bge-ase-en-v1.5, multilingual-e5-large, mxbai-embed-large-v1, but their results, don't match with leaderboard. I'll rerun them with mteb from main to check them.

I have evaluated models and their results are the same with main branch, except for AskUbuntuQuestions.
mteb_res.zip
I tried to reevalute locally AskUbuntuQuestions on multilingual-e5-small model and if I remove prompts I get same results for this task as in main, but if I add prompts I get a bit lower results.
Mteb from main branch:
INFO:mteb.evaluation.evaluators.model_encode:No combination of task name and prompt type was found in model prompts. Removing prompt_name argument.
INFO:mteb.evaluation.evaluators.model_encode:Encoding 6743 sentences.
INFO:mteb.evaluation.evaluators.RerankingEvaluator:Evaluating...
INFO:mteb.evaluation.MTEB:Evaluation for AskUbuntuDupQuestions on test took 4.56 seconds
INFO:mteb.evaluation.MTEB:Scores: {'default': {'map': 0.5680031628804745, 'mrr': 0.7094974277799763, 'nAUC_map_max': 0.2685051042082004, 'nAUC_map_std': 0.2013157249456535, 'nAUC_map_diff1': 0.04353361902146947, 'nAUC_mrr_max': 0.38623389097835026, 'nAUC_mrr_std': 0.23623505561915978, 'nAUC_mrr_diff1': 0.1194833073518608, 'main_score': 0.5680031628804745}}

This branch with prompts:
INFO:mteb.models.sentence_transformer_wrapper:Using prompt name=`passage`  for task=AskUbuntuDupQuestions prompt_type=passage
INFO:mteb.models.sentence_transformer_wrapper:Encoding 6743 sentences.
INFO:mteb.evaluation.evaluators.RerankingEvaluator:Evaluating...
INFO:mteb.evaluation.MTEB:Evaluation for AskUbuntuDupQuestions on test took 4.85 seconds
INFO:mteb.evaluation.MTEB:Scores: {'default': {'map': np.float64(0.5642404534571142), 'mrr': np.float64(0.6937211449676823), 'nAUC_map_max': np.float64(0.27381102594997414), 'nAUC_map_std': np.float64(0.1693578941478997), 'nAUC_map_diff1': np.float64(0.08137714630662059), 'nAUC_mrr_max': np.float64(0.38416684118951216), 'nAUC_mrr_std': np.float64(0.22024098705009196), 'nAUC_mrr_diff1': np.float64(0.21548592467525338), 'main_score': np.float64(0.5642404534571142)}}

This branch without prompts:
INFO:mteb.evaluation.MTEB:Scores: {'default': {'map': np.float64(0.5680031628804745), 'mrr': np.float64(0.7094974277799763), 'nAUC_map_max': np.float64(0.2685051042082004), 'nAUC_map_std': np.float64(0.2013157249456535), 'nAUC_map_diff1': np.float64(0.04353361902146947), 'nAUC_mrr_max': np.float64(0.38623389097835026), 'nAUC_mrr_std': np.float64(0.23623505561915978), 'nAUC_mrr_diff1': np.float64(0.1194833073518608), 'main_score': np.float64(0.5680031628804745)}}

I tested AskUbuntuQuestions locally using the multilingual-e5-small model. When removing prompts, I obtained the same results as the main branch. However, when I added prompts, the results were slightly lower.

So, should I remove the query and passage prompts for Reranking?

I have highlighted a section where the might be a bug - that might resolve the discrepancy

@KennethEnevoldsen I tested the commit just before merging #1221 (commit 11518edd03b916a63e84505592f0d1a32a058d49) and obtained the following results:
mteb run -m intfloat/multilingual-e5-small -t SciDocsRR --lang eng --overwrite

INFO:mteb.evaluation.MTEB:Scores: {'default': {'map': 0.7825588176371663, 'mrr': 0.9386869847654161, 'nAUC_map_max': 0.5018386842952304, 'nAUC_map_std': 0.6401228473171651, 'nAUC_map_diff1': 0.08046436666347959, 'nAUC_mrr_max': 0.7836770455942673, 'nAUC_mrr_std': 0.7179299106806627, 'nAUC_mrr_diff1': 0.5521448612994969, 'main_score': 0.7825588176371663}}

These results match my implementation, suggesting that #1221 did not cover everything, and some parts might have been missed.

I believe this is all ready to merge! Just checked it to see if it introduced any breaking changes, and the only one is the prompt, which I believe is fine. I have changed it to a minor bump though


examples/sprites: Low FPS on macOS (Metal) with 10K sprites

The FPS is dropped to 30. This happens on v1.11.0-alpha, then this is not a regression. I think updates in Metal affects this, but I am not sure.
This is not reproduced with OpenGL (ebitengl) version.

That's pretty odd but this doesn't happen on spriteshd.

The threshold seems whether the graphics command is split due to too many indices (about 10K, 65536 indices).

Rebooting my MacBook Pro fixed this issue...


Fixed the flow throughout | Added audio control UI

🛠 Fixes Issue
#105
Related Issue/Addition to code

Optimized/Fixed loop over a song
Optimized/Fixed shuffle playlist

👨💻 Changes proposed

UI changes in audio control options
Added pause and volume control
Fixed a lot of logical issues with the flow
Refractored code to optimize for performance

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

Thoroughly test the code
Try to recreate different scenarios while testing
Also test if changes for shuffle and loop buttons are persisted after refresh

📷 Screenshots

Original
Updated

** original screenshot
** updated screenshot **

Thanks for your contribution @sharmavivek223.


Add vendored dependencies

remove vendor from .gitignore file
generate vendor folder with go mod vendor command

Have you tested that this compiles? I see they have a note about goautoneg, want to make sure that isn't going to cause further issues?

@JoelSpeed Yup, all compiles.
In regards of autoneg - this comment seems outdated to me. Found only github.com/munnerz/goautoneg in go.sum
$ go mod why github.com/munnerz/goautoneg
# github.com/munnerz/goautoneg
k8s.io/cloud-provider-vsphere/cmd/vsphere-cloud-controller-manager
k8s.io/cloud-provider/app
k8s.io/apiserver/pkg/server
k8s.io/kube-openapi/pkg/handler
github.com/munnerz/goautoneg


Group multiple parameter definitions for better maintainability

Idea
Sometimes you have some parameters you want to provide on every path. At the moment it is possible to reference them with the $ref tag. This works great if you have a lot of different parameter combination for each route.
But we now have a complex api with a lot of optional get parameters for every path. Now the problem we encounter is, that if we have to add new parameters to this get parameters, it is very cumbersome and inconvenient to set the reference to them on each path. Often it happens that you forget one.
It would be great if we could group parameter definitions for a better maintainability and usability.
Example
As already presented on stackoverflow here an example of the definition (how it could look like):
parameters:
  MetaDataParameters:
    # Meta Data Properties
    - name: id
      in: query
      description: Entry identification number
      required: false
      type: integer

    - name: time_start
      in: query
      description: Start time of flare
      required: false
      type: string

    - name: nar
      in: query
      description: Active region number
      required: false
      type: string

And here is how I would like to reference it:
/test/:
  get:
    tags:
      - TEST
    operationId: routes.test
    parameters:
      - $ref: "#/parameters/MetaDataParameters"
    responses:
        200:
          description: OK

To get more on the user experience, this definition is much more readable than 20 parameters for each route.
Other Ideas?

Regards
Florian

There's a valid example further up the thread, and here's my working example in YAML that adds the parameters to all get requests:
overlay: 1.0.0
info:
  title: Add parameters to every endpoint
  version: 0.1.0
extends: file:///home/lorna/demo/overlays-traits-example/museum.yaml
actions:
  - target: $.paths.*.get
    update:
      parameters:
        - name: id
          in: query
          description: Entry identification number
          required: false
          type: integer

        - name: time_start
          in: query
          description: Start time of flare
          required: false
          type: string

        - name: nar
          in: query
          description: Active region number
          required: false
          type: string

You can do this today using any of the published overlay tools such as the CLIs from Speakeasy and Bump.sh.
I'm going to close this issue since I do feel that the question is answered, but do follow up with new issues or discussions around this feature or similar use cases - we're happy to hear feedback at this early stage.

But this is not the specific function that is asked for in the OP, or by many of us. This is only a solution if you want to put this combination of parameters on ALL endpoints of a certain type. The option to bundle parameters and reference them on most, but not all of my endpoints, or have somewhat different bundles of parameters I can reference is not fixed with this solution. It's been open for close to 10 years and now we're closing the issue because there's a somewhat related option to do something similar but not actually adress this specific problem? If we just decide that this will never be relevant or useful that's one thing, but the solution of just defining things on ALL gets or endpoints is not even what the very first example by OP showcases. It's about referencing a bundle of parameters that makes it easy to maintain several re-useable bundles of parameters and a function like that would still be of great use to us.
The best example is a set of standard parameters we have for 70% of our endpoints, but not all of them, and the logic on which do or don't is not as simple as "all gets". Currently this means referencing 5 different parameters instead of doing it like in the OP and there's still no fix for it afaik.

@antonMetodika
I also have the same opinion. However, this issue is now part of Overlay specification and not the original OpenAPI specs. So, in case of overlay specs, it is a valid solution.
Probably, we need to open a new issue on OpenAPI specs. BTW, when this issue was created, there was no overlay specs.

@antonMetodika @sharmasourabh this got moved to Overlays because the discussions became mostly about "traits".
For OAS 3.x, there are several issues already tracking grouping and re-use:

OAI/OpenAPI-Specification#1364 (specifically about parameters, which is the other reason this issue got sent to Overlays- the non-overlay part was redundant)
All issues tagged re-use: ref-group-combine
You might also find those tagged re-use: globals/defaults relevant (although there is onlyone at the moment)

These sorts of topics are also being discussed for the next major version at OAI/sig-moonwalk.

@antonMetodika I think Overlays can still help in the use case that you mention, but the mechanics are a bit different. For example, you could use a particular tag or even an extension on the endpoints in OpenAPI that should get the group of parameters applied (or tag the ones that should not, if that's easier!) - and then Overlays can add the group of parameters to those endpoints. I did post a simplified example, but we can expand it for something more specific to your use case if that would make it clearer how it can meet your needs?

@lornajane Thank you for clarifying, for some reason I did not make the connection that I can use that with custom tags rather than just on all get/post etc. endpoints. I think the example in the closing comment could benefit from that clarification so slow people like me can make sense of it faster! :D If it works as described I think this covers the use cases I can think of pretty well, thanks again!


python demo.py --input image data/demo.png is not working ...

Your 1. example in Tutorial -> Getting started does not work in my environment but gives:
2018-02-08 20:43:57,633 INFO Starting inference using data/demo.png as input
Traceback (most recent call last):
File "demo.py", line 219, in
tf.app.run()
File "/Users/erkki/tensorflow/lib/python3.6/site-packages/tensorflow/python/platform/app.py", line 124, in run
_sys.exit(main(argv))
File "demo.py", line 175, in main
min_conf=0.50, tau=hypes['tau'], color_acc=(0, 255, 0))
File "incl/utils/train_utils.py", line 103, in add_rectangles
from utils.stitch_wrapper import stitch_rects
ImportError: dynamic module does not define module export function (PyInit_stitch_wrapper)
I'm running your code on macOS High Sierra version 10.13.3 and installed tensorflow and all SW you mentioned according your instructions and instructions by tensorflow. How could this situation be fixed ?

Happened to me using a conda environment.
Apparently, during setup make is compiling a cython module against Python2 and not Python3, as your environment expects (pip linked to python3?).
Try doing manually step 3 of setup, like:
cd submodules/utils
pip install runcython3
makecython3++ stitch_wrapper.pyx "" "stitch_rects.cpp ./hungarian/hungarian.cpp"

@apgiorgi manually doing step 3 as you have shown gives a segfault when running:
from utils.stitch_wrapper import stitch_rects
in KittiBox/submodules/utils/train_utils.py


learn module URLs

Link to article:
https://docs.microsoft.com/en-us/learn/
Problem:
The module tiles on https://docs.microsoft.com/en-us/learn/ link to an URL with "https://docs.microsoft.com/en-us/modules/..." which ends in 404, where the correct path would be "https://docs.microsoft.com/en-us/learn/..."

@rwarnke Which modules, for example?


Gulp task vscode-linux-x64-build-debian creates dummy DEB installer

"Dummy" meaning only includes the meta data for the Application Menu, Icon, File Association, etc. None of the actual executables for the program.

VSCode Version: 3864be5
OS Version: Debian 9.1.0 (Stretch)

Steps to Reproduce:

Install dependencies
Clone
./scripts/npm.sh install --arch=x64
gulp vscode-linux-x64-build-deb
Check ./.build/linux/deb/amd64/deb/<deb installer here>'s file size (8kb)

At some point the dependency for that task was changed I think in order to lessen resources on the build agent. See 277dc8bdb99ac2ae820d5ee35d9fa2fedd65d0b1

Then the GitHub Wiki page needs to be updated with the correct steps.

@khionu good call, fixed 😃

Sorry to point this out this much later, but this section also could be updated.


godot aapt2 compile and aapt2 link

Godot version
3.3 STABLE
System information
LINUX
Issue description
godot aapt2 compile and aapt2 link
I'm trying to generate the res folder from godotgame.aab using aapt2 compile and aapt2 link but at the time I run error, so please tell me how godot generates the res folder using aapt2 compile and aapt2 link, how can I use the lib godot godot-lib.release.aar to generate the aab correctly, with all the res/ that org.godotengine.godot asks when run for example my ta failing in org.godotengine.godot.R$layout ??
Steps to reproduce
Generate aab without gradle using android cmdline, using:
aapt2 compile
aapt2 link
javac
d8
bundle-all.jar
Minimal reproduction project
No response

Duplicate of https://github.com/godotengine/godot/issues/55738. Please do not create additional issues if you don't get an answer quickly – it's likely that nobody knows the answer. Also, exporting to Android from a 32-bit OS is no longer officially supported by Google.


Remove ServiceEvents() call from Android and Python controllers

Problem

#7804
In the Android controller there is no device layer, so ServiceEvents() does nothing

Change overview
Remove ServiceEvents() from both controller apps
Fixes #7804
Testing
Commissioned m5stack running all-clusters-app and sent zcl commands in chip-device-ctrl and Android CHIPTool

Approving pending build failures


[Chrome] Not working if refreshing after opening devtools

I can't reproduce this issue. Please give exact details about what browser you are using. check the about:version page of your browser.

Oh! I'm really sorry, actually, it was an issue in my code and not the library.
Still, thanks for the fast reply.
We can close this issue.


feat: add command to restart Gleam language server

close: #44

Let me know if you'd like me to merge or wait for any more

Thanks for your review, I believe this pull request is ready to be merged😊.


Please support zoom in/out on pictures

https://github.com/cloudflare/cloudflare-docs/blob/production/content/firewall/cf-firewall-rules/order-priority.md

I have no idea how to clearly see the picture without opening it in a new tab. Please support clicking on the image and then displaying the original one.

Hi there - I was able to zoom in (Firefox and Chrome) by using keyboard shortcuts. Just hold control and the plus button at the end of your number keys and you can zoom in on the image.


Visual cue for when compile is finished

This is how it currently looks (in development)

i.e. note the "Success" label in the middle top. When it fails, the label is red, and reads "Failed"
Comments about changes are welcome

Abandoned, since Delphi 10.4.2 now has an improved compiler progress dialog


Need public API to seek to another APNG frame

Disclaimer
This issue is based on my current, evolving understanding of the requirements that Chromium and Skia architectures impose upon an underlying codec implementation.  I am opening this issue early to facilitate a transparent discussion, but before accepting the new APIs we should definitely require that I first finish prototyping the APNG-focused part of the Chromium/Blink/Skia integration (as a proof-of-concept that the new APIs actually work and solve my use case).
Problem 1a: No way to get to next image data
When Reader.next_interlaced_row reaches the end of a frame, then it gets "stuck" - subsequent calls will continue returning None (which correctly indicates that no more rows in the current frame) and there is no way to proceed to the next frame.
Problem 1b: No way to read the next fcTL chunk
To successfully decode a frame, one needs to have its FrameControl data, but Info.frame_control won't be updated until the fcTL chunk is encountered and parsed.  There is no way to ask Reader to get the next FrameControl metadata.
Problem 2: No way to go back and decode an earlier frame
Skia and Blink abstractions allow asking a codec to (again) decode an earlier frame:

Skia allows that via Options.fFrameIndex parameter of SkCodec::onStartIncrementalDecode (and onGetPixels) which can specify an earlier frame.

Example implementation can be found in the SkWuffsCodec here
Note that there is no requirement to go to a later frame - the codec may choose to not report frames for which no metadata has been seen yet.  For example, SkWuffsCodec populates its fFrames field incrementally - see here.

Chromium allows that via index parameter of blink::ImageDecoder::DecodeFrameBufferAtIndex and Decode

Example implementation can be found in the libpng-backed PNGImageReader here

Early discussion
Initially I tried solving problem 1 by publicly exposing Reader.read_until_image_data.
One solution to problem 2 would be to (brainstorming quality, please bear with me):

Add Reader.seek_to_frame(&mut self, frame_index: usize) -> Result<&FrameControl, DecodingError> method that is hidden behind where R: Seek

As proposed above, this would be only for APNG frames.  If IDAT is not part of the animation, then there is no way to seek back to IDAT.
Implementation would seek to the start of compressed data stream in an IDAT or fdAT chunk and would need to restore the related state in StreamingDecoder.

Option 1: restoring to State::ImageData(...) or State::ApngSequenceNumber (this requires remembering and restoring  current_chunk - type, crc, remaining
Option 2: restoring to State::U32 { kind: U32ValueKind::Length } (no need to remember chunk contents;  requires seeking to a slightly earlier offset)

Modify Reader.read_until_image_data to store frame offsets if possible

I think that we can find a way to store offset = None when R: !Seek (maybe by introducing a private trait that is implemented for Read + !Seek and for Read + Seek)
In addition to offsets, for each frame we would have to store the data needed to restore decoding state (as described earlier):

FrameControl
fdAT / fcTL sequence number (can add a crate-internal accessor to StreamingDecoder?  or maybe we Decoded::ChunkBegin should be replaced with Decoded::DataChunkBegin(opaque_state_to_restore)?)

I think that if we have seek_to_frame API, then we would still need to make read_until_image_data public (to support moving to the next fcTL when the input stream doesn't implement Seek).
I am not sure if there are other, better ways to solve problem 2.

I note that libpng and StreamingDecoder use a push model (slices with additional data is pushed into their APIs) and Reader uses a pull model (calling Reader APIs triggers Read.read from the underlying data stream).  In a push model the client is responsible for managing seeking/offsets.  But a push model may still have a state that needs to be restored (I am not sure how exactly that works in libpng).

Let me also be transparent that I am not yet 100% sure about the motivation here.
The immediate motivation is that the AnimatedPNGTests.ParseAndDecodeByteByByte test requires seeking to an earlier frame (at least as currently written), because it calls DecodeFrameBufferAtIndex in two separate loops: here and here.  But fixing this test doesn't necessarily require seeking to an arbitrary earlier frame:

Restarting from the beginning of the PNG file is sufficient to fix the test.  For example, one way forward may be to have SkPngRustCodec::startDecoding handle 1) decoding the current/latest frame, or 2) restarting for options.fFrameIndex == 0, (rather than supporting arbitrary frame indices).
Alternatively, the test itself could be rewritten to avoid the 2nd loop.  (During the first loop the image is decoded byte-by-byte, testing the codec's ability to resume after UnexpectedEof - this seems what this test helper wants to focus on.  Re-decoding and checking image hashes in the 2nd loop tests a slightly different thing.)

I am working with the Skia team to better understand the motivation for supporting seeking to an earlier frame.  One hypothesis is that this is needed when decoding the current frame depends on first restoring one of previous frames (e.g. because one or more earlier frames use DisposeOp.Previous).  Discarding previous frames seems to be handled by SkiaImageDecoderBase::CanReusePreviousFrameBuffer, but maybe previous frames can still be discarded in some scenarios (just guessing: maybe when working under memory pressure?).

Seeking back to the first frame makes sense to me, but I'd be curious what use seeking to a different frame would be? I'd expect that you wouldn't be able to reconstruct it without potentially needing to know all previous frames.
Another question I have looking at the current API is that we don't seem to distinguish between the static image and the first frame of the animation, which may or may not be the same. Might be worth separating them like image-webp does.
As far as API changes, it might be worth investigating whether to unconditionally require Seek in the next major release. That would also enable the decoder to save the locations of metadata chunks and decode them lazily when requested, rather than always saving them without knowing whether they'll be needed

The comparison to image-webp and tiff is interesting, there's lots of overlap in the required seek functionality. In particular, I can well see the use-case in building a list of all fcTL chunks which includes the necessary information to compute any presentation dependency (i.e. walk backwards until a full dispose, or opaque blending with the full area) . With that graph an efficient lookup and rebuild for individual frames is also possible, similar  to keyframes. I think all of this can algorithmically and in types live separate from the encoder itself. That introduces a slight imprecision where the depencency graph of an image might be used with a mismatched decoder, but I think that's more than acceptable instead of introducing a lot of complexity within the decoder.

As an aside: with regards to Seek as bound, these exact issues are good reasons to investigate Seek as a runtime property instead a compile time. You're now aware of the problem space, please consider this solution prototype. Significant unsafe and an optional unstable attribute, so escalate these internally to wherever appropriate.

For now I've implemented seeking to an earlier frames by 1) rewinding the input stream to the beginning and recreating png::Reader and 2) reading/moving frames one-by-one until the desired frame.  This lets me make further progress on passing additional Chromium/Blink tests, and set the efficient-seeking problem as something that we may reconsider later - for now I've opened https://crbug.com/371060427 to track this on my side.

we don't seem to distinguish between the static image and the first frame of the animation, which may or may not be the same.

Yes.  FWIW this is not blocking - the client can recognize the situation (animation_control present, but frame_control missing in the image Info) and handle it as needed (skipping the first frame somehow - either via next_frame, or by a new API that wraps read_until_image_data)

Seeking back to the first frame makes sense to me, but I'd be curious what use seeking to a different frame would be? I'd expect that you wouldn't be able to reconstruct it without potentially needing to know all previous frames.

I can well see the use-case in building a list of all fcTL chunks which includes the necessary information to compute any presentation dependency (i.e. walk backwards until a full dispose, or opaque blending with the full area) . With that graph an efficient lookup and rebuild for individual frames is also possible, similar to keyframes. I think all of this can algorithmically and in types live separate from the encoder itself.

Right - see how:

SkiaImageDecoderBase::Decode calculates which previous frames need to be decoded
SkFrameHolder::setAlphaAndRequiredFrame calculates whether some previous frame is needed (based on dispose and blend modes) to decode a given frame (the "requires" relationship is for a full image buffer, not just for the previous frame's subject).

(Note that the code above is codec-independent.)
But, I am still not sure in what scenario a previous/required animation frame would be unavailable.   If required frames may be discarded (out-of-viewport?  low-on-memory?) then maybe it should be okay to restart the animation from the first frame?  I dunno...

I think implementing seek-to-frame-N as rewind-and-skip-N-1-frames can be made a bit faster if we won't try to decompress the image data as we skip over the frames.  FWW I've prototyped this at https://github.com/image-rs/image-png/compare/a268b031276c0df43452a852ad97e6b63e4dfbc1...anforowicz:image-png:image-data-sink-fn.

@anforowicz I've implemented a similar compression skipping for GIF. Maybe use similar approach in PNG if possible?
https://github.com/image-rs/image-gif/blob/51ea9b0bf334b918bd24751812ea49c3bf1152f8/src/reader/decoder.rs#L755

Thanks for the pointer @kornelski.  It's interesting how gif follows a similar StreamingDecoder / Decoder pattern - I didn't realize this before.  IIUC gif::StreamingDecoder doesn't provide a way to mutate its skip_frame_decoding field, but in the png case we could support crate-internal mutation (via, say StreamingDecoder.set_skip_frame_decoding(new_value: bool)), and then ReadDecoder::finish_decoding_image_data could call self.decoder.set_skip_frame_decoding(true) (and ReadDecoder::read_until_image_data could set skipping to false;  I think other methods wouldn't touch this flag/option, including ReadDecoder::decode_image_data).  This is indeed a bit simpler: https://github.com/image-rs/image-png/compare/6016c9bee0fbd6244ebe49ed19a25094898c8001...anforowicz:image-png:skip-frame-decoding-flag-in-streaming-decoder.  I'll see if I can polish it up and submit as a PR next week.


Linode Networking tab copy

Can we have click-2-copy button on the linode networking tab for IP addresses. It exists on summary lists, just not within the node itself.

@tsmgeek this will be implemented in our next release :)


error: Cannot read properties of undefined (reading 'retweeted_status_result')

When I tried to export bookmarks, I encountered the following error:
error: Cannot read properties of undefined (reading 'retweeted_status_result').
I am using the Arc browser on macOS, version: latest version.
I'm not sure if this is related to me adding too many bookmarks (548).

I've checked all issues but cannot get the solution of this problem.
thanks a lot! wish you have a good day：)

Hi,
Thanks you for your feedback. Please open your browser's DevTools (by pressing ⌘⌥I) and check the console for more detailed error messages.
Would you mind sharing the API response which caused the error? You can do this by following these steps:
(The response data may contain sensitive information, so please review it before sharing it here. Or you can send it to my personal email: hi at prin dot studio)

hi! thanks for your timely help!
I've sent a mail includes the response contents you need, my mail is i at atago dot moe
and this a picture that my console prints

@YiniRuohong Thank you. I have received your email.
However, the response seems to be not very helpful, since it does not contain any tweet. This could happen if you reached the end of your bookmark list.
Could you please reload the webpage and try it again? Once the error occurs, please send me the latest API response at that moment. The correct response should be much longer than the provided one.

I apologize for my mistake, I have found the contents you need and resent a mail to you

I have identified the cause of the error with the help of data you provided. It seems that Twitter does not return some fields when there is an error about a tweet.
This issue should be fixed in the next release. Thank you for your feedback!

Thanks for your timely help! have a good day!

The latest nightly build should have fixed this issue. Please give it a try and let me know if it works for you.
https://github.com/prinsss/twitter-web-exporter/releases/download/nightly/twitter-web-exporter.user.js

Everything is working fine now! I can now export bookmark content and media files normally. Thank you for your support!


Preferred way to change env in the secondary frontend project?

Which Category is your question related to?
env
Amplify CLI Version
10.0.0
According to the docs, we should avoid modifying backend in the secondary project. So recommended way to start using the existing backend in the secondary project is to use amplify pull.
But how can we change env in the secondary project? If my understanding is right, the only way to do this is to rm -rf amplify/ rm src/aws-exports.js and amplify pull again, which I think it's kinda cumbersome operation.
Is there any easier way to do this? or any plan to make it easier?

Maybe #9457 could make it easier

Hey @taneba :wave: thanks for raising this! For these types of projects we will need to pull the additional environments with amplify pull --appId <app-id> --envName <env-name. And while we work to improve the multi-env workflow, we can change environments by rerunning the pull command with the specified app ID and environment name

@josefaidt
Thank you for your super rapid reply as always!
That command should help!
And it'd be great if it is on the document (can I help with improving documentation?)
I'm ok with closing this, but leave it you ;)


Bug | Avatar | Color contrast

Issue:
When user hovers over an Avatar icon, the bg-text colour contrast is janky.
Solution:
Change the hover colors:
The new combination of bg & text color should pass this test.
https://webaim.org/resources/contrastchecker/

I'd like to work on this issue


?

Not sure if this article reflects article's title.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 62325703-30d4-29d7-88ca-02064648dc73
Version Independent ID: c293f825-c66d-58d4-ed55-fdfe992c832a
Content: Tutorial - Integrate an existing forest and a new forest with a single Azure AD tenant using Azure AD Connect cloud sync. - Microsoft Entra
Content Source: articles/active-directory/hybrid/cloud-sync/tutorial-existing-forest.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@dusansk
Thanks for your feedback! We will investigate and update as appropriate.

@dusansk
Could you please elaborate the query for better understanding. So, we can assist you further.

Hi,Yes... looking at the article it should show how you can use two domains and create AD Connect sync with one Azure AD tenant. Not sure that is the case with the article. From the article... at least I got lost, as only contoso.com domain is used and when you read it you are confused as to what goes where... The article says: "You'll set up cloud sync for the new forest." but then contoso.com screenshot is used as a reference... Could be just me but I find it confusing as reference to fabrikam.com is never used.Regards,Dusan On Sep 9, 2023, at 7:33 AM, Saibaba Balapur Hireka @.> @. you please elaborate the query for better understanding. So, we can assist you further.—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

@dusansk
I'm going to assign this to the document author so they can take a look at it accordingly.


Proppy subscribers are non removed properly

Hey Fahad! Great job on Proppy, I've been quite amazed by the project so far but ran into a small bug:
The implementation of removeCallback will not remove all handlers when .destroy is called, you can see an example here:
https://codesandbox.io/s/o499mj9nxy?expanddevtools=1&module=%2Fsrc%2Findex.js

@reaktivo: Marcel!!! :D
Thanks for the kind words, mate! I will take a look at your PR when home and take action soon.
Cheers!


Ractify missing from dependencies

First of all, thanks for your work on the component.
When using the component with browserify it complains about reactify not being installed. Adding it to the package.json dependencies should fix that?

Reactify does not work in dev-dependencies. It needs to be installed in dependencies, because dev-dependencies are only installed at the top layer. Adding it to dev-dependencies does nothing, in my project.
This is not a pressing issue for me, as I have simply included reactify in my dev-dependencies. I think this is really more browserify's fault, although the author of browserify, @substack, maintains that one should always build packages before putting them on npm.


[FEATURE] integrating envpool

Please describe the purpose of the feature. Is it related to a problem?
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
The current implementation using envpool on clearnrl is buggy and doesn't use pmap, having envpool integrating with Stoix would be useful.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Implementt envpool
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
How do we know when implementation of this feature is complete?
Checklist:

[ ] envpool
[ ]

Additional context
Add any other context about the feature request here.

Hey, I was planning on doing this at some point. I know envpool can be jitted but I'm not fully aware of its pmap capabilities but I will get around to this at some point soon.

Hello, so I was playing around with envpool and unfortunately it seems to have some limitations with regards to its xla compatibility. For example, although it can be jitted, it seems unable to vmap due to not having xla batching primitives and consequently I imagine that means there are issues for pmapping (at least in the current form of how stoix does it). There are obviously solutions to this, for example running the environment loop separately and collecting data then splitting it into the correct shapes to then pmap only the SGD steps however that semi-defeats the purpose and general design paradigm of stoix where both environment data collection and SGD steps happen on all devices. If you give it a go then I'd be more than happy to review it and then subsequently adapt it for Stoix's use case.

@EdanToledo I suppose that makes sense. My use case would be for for training atari models, but I think end-to-end jax rl should be enough considering the limiting factor seems to be cpu-gpu communication anyways since envpool still runs the environment itself on cpu.


fix error when logging to progress bar with reserved name

What does this PR do?
Fixes #5299
The statement
self.log("loss", loss, prog_bar=True)

currently results in an error, because the loss is already logged to the progress bar by default.
Instead of an error message, we overwrite the progress bar dict entry by what the user logged and print a warning.
The warning was suggested by @edenlightning. Alternatively, we could also just overwrite the value without the warning because the user explicitly calls self.log anyway.

@awaelchli can you move this test to logging_tests directory?. For the reasons described in https://github.com/PyTorchLightning/pytorch-lightning/pull/4451 (and https://github.com/PyTorchLightning/pytorch-lightning/pull/5452#pullrequestreview-565300740)
We should be more mindful about where we place our tests

@awaelchli can you move this test to logging_tests directory?. For the reasons described in https://github.com/PyTorchLightning/pytorch-lightning/pull/4451 (and https://github.com/PyTorchLightning/pytorch-lightning/pull/5452#pullrequestreview-565300740)
We should be more mindful about where we place our tests

We should be more mindful about where we place our tests

I asked but was not fast enough to react before it was merged. I am mindful and I have suggested several times before to define a consistent testing file/folder structure. Here #5667

We should be more mindful about where we place our tests

I asked but was not fast enough to react before it was merged. I am mindful and I have suggested several times before to define a consistent testing file/folder structure. Here #5667


Does not compile because of missing context

~/import/kubeless16:11 >make
CGO_ENABLED=1 ./script/make.sh binary
---> Making bundle: binary (in .)
../gopath/src/k8s.io/kubernetes/vendor/k8s.io/client-go/rest/request.go:21:2: cannot find package "context" in any of:
/home/thomas/import/gopath/src/k8s.io/kubernetes/vendor/context (vendor tree)
/home/thomas/import/go/src/context (from $GOROOT)
/home/thomas/import/gopath/src/context (from $GOPATH)
Makefile:15: recipe for target 'binary' failed
make: *** [binary] Error 1

hi @thomasfricke Thanks for your report. I just did a quick check by running make with the master branch. It worked as below:
Tunas-MacBook-Pro:kubeless tuna$ make
CGO_ENABLED=1 ./script/make.sh binary
---> Making bundle: binary (in .)
Build successful. Program saved as ./kubeless

May I ask you to double check your $GOPATH in order to make sure you have a separate location for kubeless ? As I checked in kuberless/vendor folder that kubeless has no dependency to rest/request.go

@thomasfricke May I also ask your golang version currently in use ?

So @thomasfricke I just committed my vendor folder containing all dependencies of kubeless. If you could do a git pull and run make again to see if it works or not.

The context package was added to the standard library in go 1.7.  Because of this transitive dependency, kubeless (and much of the rest of the golang ecosystem fwiw) effectively requires go >= 1.7


Documentation for Simple Example

Hi there,
This library looks great, but I'm wondering if there is documentation in addition to some of the examples that are hosted in the repo. It's a little intimidating for someone that doesn't have any Lib UI experience, so maybe there is some documentation for beginners somewhere? I'm basically trying to make something that looks like the attached image. I can also close this and post it to StackOverflow if that'd be a better place, but I thought it would maybe be appropriate as an issue as there might be other beginners like me looking for this type of documentation.

This code seems to accomlish your goal from testing on the Mac only at least:
require 'glimmer-dsl-libui'

# This is a LoginRequest model
class LoginRequest
  ALLOWED_USERNAMES = %[john joe bob nancy gomez]

  attr_accessor :username

  def login
    ALLOWED_USERNAMES.include?(username.to_s.downcase)
  end
end

# By declaring this view as a Glimmer::LibUI::CustomWindow, you can later use `login_screen`
# as Glimmer GUI  keyword to instantiate LoginScreen underneath a parent
class LoginScreen
  include Glimmer::LibUI::CustomWindow

  attr_reader :login_result

  before_body do
    @login_request = LoginRequest.new
  end

  body {
    window('Login') {
      margined true

      vertical_box {
        form {
          entry {
            label 'Username:'
            text <=> [@login_request, :username]
          }
        }

        button('Login') {
          on_clicked do
            @login_result = @login_request.login
            # the following 2 lines close window and LibUI runtime for that window
            body_root.destroy
            ::LibUI.quit
          end
        }
      }
    }
  }
end

# By declaring this view as a Glimmer::LibUI::CustomWindow, you can later use `copy_screen`
# as Glimmer GUI  keyword to instantiate CopyScreen underneath a parent
class CopyScreen
  include Glimmer::LibUI::CustomWindow

  option :item

  body {
    window('Copy') {
      margined true

      label("Copying #{item} in progress...")
    }
  }
end

# By declaring this view as a Glimmer::LibUI::CustomWindow, you can later use `show_screen`
# as Glimmer GUI  keyword to instantiate ShowScreen underneath a parent
class ShowScreen
  include Glimmer::LibUI::CustomWindow

  option :item

  body {
    window('Copy') {
      margined true

      label("Showing #{item} content...")
    }
  }
end

class App
  include Glimmer::LibUI::Application

  ITEMS = 4.times.map {|n| "Item #{n+1}" }

  before_body do
    # login_screen builds an instance of LoginScreen
    login_window = login_screen
    # the show method will block until the window is closed
    login_window.show
    # if login result is false, then exit app completely
    if !login_window.login_result
      exit(0)
    end
  end

  body {
    window('App') {
      margined true

      vertical_box {
        ITEMS.each do |item|
          horizontal_box {
            label(item)

            button('Copy') {
              on_clicked do
                copy_screen(item: item).show
              end
            }

            button('Show') {
              on_clicked do
                show_screen(item: item).show
              end
            }
          }
        end
      }
    }
  }
end

App.launch

At the bottom of the README Usage section, it says the following for Beginners (I probably should move this elsewhere to make it more prominent):
"If you are new to Glimmer DSL for LibUI, check out the Glimmer GUI DSL Concepts, Glimmer Command, Girb and Examples to quickly learn through copy/paste. You may refer to the API later on once you have gotten your feet wet with Glimmer DSL for LibUI and need more detailed reference information."
I think what matters the most are:

Glimmer GUI DSL Concepts
Examples

Learn by going through every example (run locally) while searching the README for anything that matches what you see in the examples. You need to graduate gradually in knowledge on how to build desktop applications.
Otherwise, I highly watching the RubyConf 2022 presentation on Building Native GUI Apps in Ruby:
https://www.youtube.com/watch?v=1Bh4CnJqHyY&feature=youtu.be
It breaks down the Glimmer DSL for LibUI concepts into small bits. But, you still need to apply the concepts by going through the Examples.
You are also welcome to ask me any questions here in Issues or on the Glimmer Gitter chat. I don't mind answering questions assuming an educated and serious Software Engineer.
In the meantime, if you could think of any improvements to documentation for beginners, please share ideas in Issues or doc submissions in Pull Requests.

Wow, this is incredibly helpful. I will try and put together a working copy today based off of this and let you know how it goes. Thank you!

Alright, I got something working locally and pushed an initial version to https://github.com/jakebruemmer/adamantite.
I made a couple of tweaks to the code that you provided, but it's pretty much the same. Thank you for helping me understand the Glimmer UI lib a bit better. The creation and usage of classes makes a lot more sense now.

This is great news!! I am closing this issue as a result.
If you think of any more questions in the future, you are welcome to ask me in the Glimmer Gitter chat or here if it’s related to the code mentioned above.
Cheers.


fix: correct the positional encoding of Transformer in pytorch examples

What does this PR do?

Fixes #19138

Before submitting

[x] Was this discussed/agreed via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
Did you make sure to update the documentation with your changes? (if necessary)
Did you write any new necessary tests? (not for typos and docs)
Did you verify new and existing tests pass locally with your changes?
Did you list all the breaking changes introduced by this pull request?
Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

Reviewer checklist

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

📚 Documentation preview 📚: https://pytorch-lightning--20203.org.readthedocs.build/en/20203/

The difference between this example and the original one pytorch example is that the default value of argument  batch_first is set to False in PyTorch, whereas here it is set to True.
In order to use the positional encoding correctly, I made the above changes. But I don't know how to write a test case.

@Borda Hi, could you tell me how to deal with the two failed workflows?


Temperature sensor

Should be easy, just calling sd_temp_get.
Maybe check if there's a HAL trait out there and implement that.

There isnt a eh trait that Im aware of.
The nrf hals do this
https://github.com/nrf-rs/nrf-hal/blob/4692919084613fb5f697021897de697b3f8fe43f/nrf-hal-common/src/temp.rs
I presume its wrapped because of the blocking nature
https://infocenter.nordicsemi.com/index.jsp?topic=%2Fcom.nordic.infocenter.s132.api.v3.0.0%2Fgroup___n_r_f___s_o_c___f_u_n_c_t_i_o_n_s.html&anchor=gade0ea69f513ff1feab2c4f6e1c393313
"It takes around 50us from call to return"


[infra] add sub git module hertzbeat-helm-chart

What's changed?

Checklist

[ ]  I have read the Contributing Guide
[ ]  I have written the necessary doc or comment.
[ ]  I have added the necessary unit tests and all cases have passed.

Add or update API

[ ] I have added the necessary e2e tests and all cases have passed.

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

Project coverage is 29.02%. Comparing base (3a2350b) to head (9b63894).
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #2522   +/-   ##
=========================================
  Coverage     29.02%   29.02%
  Complexity     1624     1624
=========================================
  Files           429      429
  Lines         17704    17704
  Branches       2307     2307
=========================================
+ Hits           5138     5139    +1
+ Misses        11983    11982    -1
  Partials        583      583

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


Improve Chinese translation.

@erickzanardo @renancaraujo @yang-lile I am thinking about if we should mix non-English content with Chinese content. Like the descriptions are Chinese, but the content it links to is not. It might be confusing/frustrating.

@erickzanardo @renancaraujo @yang-lile I am thinking about if we should mix non-English content with Chinese content. Like the descriptions are Chinese, but the content it links to is not. It might be confusing/frustrating.

Good point, I Agree!

@erickzanardo @renancaraujo @yang-lile I am thinking about if we should mix non-English content with Chinese content. Like the descriptions are Chinese, but the content it links to is not. It might be confusing/frustrating.

This is a good idea. But I don't have Chinese tutorials or documentation. so I can only use Chinese descriptions and English content for the time being. I will learn flame and synchronize translations these English content.

@erickzanardo agree on against or for mixing?
@yang-lile I personally would prefer to only add entries that link to Chinese content. Probably the most troublesome point would be to keep both lists in sync for each new PR.

@erickzanardo agree on against or for mixing?
@yang-lile I personally would prefer to only add entries that link to Chinese content. Probably the most troublesome point would be to keep both lists in sync for each new PR.

hahaha, sorry, I agree with you, we shouldn't mix the languages.

@impulse Something likes non-English add entries that link to English content, and English content add entries that link to non-English content?
Sorry,  I do translation for the first time, if there is a problem, please give more advice.

I'm sorry that the previous translation wasn't perfect, but now I've added a directory and a link to the original text. Synchronizing two pull requests is a tedious thing, but I don't know how to optimize it.

@impulse Something likes non-English add entries that link to English content, and English content add entries that link to non-English content?

如果我们加英语内容的话，每个PR都需要更新这个文件。Flame覆盖的中国市场很小，而且我不认为我们可以找到很多可以坚持这样做的中国人。
Your idea is perfectly fine considering we have a lot of entries that are just code-based projects. But given the reasons above I think it's just not feasible to do.
I am happy to update the file with table of contents and the corrections you have made for the existing entries. But I can't expect other contributors to add entries in one (or maybe more one day) language that they don't speak.

Maybe you are right, Flutter Chinese user is less, Flame user is less. I will work on my fork, until it's all translated. 感谢您的建议和帮助。

Maybe you are right, Flutter Chinese user is less, Flame user is less. I will work on my fork, until it's all translated. 感谢您的建议和帮助。

没事，谢谢你的工作


added my goals

new-branch pull request

Kenny, got the file (the PR worked). But the file is empty. No contents. Not sure what's going on there, but you need to add and commit after you make any changes...

I’ve made a new branch and tried again…fingers crossed.
Sent from my iPhone

On Nov 12, 2021, at 7:40 AM, C-STYR @.***> wrote:

Kenny, got the file (the PR worked). But the file is empty. No contents. Not sure what's going on there, but you need to add and commit after you make any changes...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Still nothing in this file. Not sure what's going wrong here; can you add some screenshots showing file contents and your terminal commands?

When I try the process again it says tree empty nothing to commit.  However here is a screenshot of my last session where I believe it says files have been committed.
From: C-STYR @.>
Sent: Monday, November 15, 2021 2:59 AM
To: C-STYR/GitHubTrainer @.>
Cc: KWFelts @.>; Mention @.>
Subject: Re: [C-STYR/GitHubTrainer] added my goals (PR #16)
Still nothing in this file. Not sure what's going wrong here; can you add some screenshots showing file contents and your terminal commands?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/C-STYR/GitHubTrainer/pull/16#issuecomment-968670346 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AWLQK2RQILNSD4GMR6REFPDUMDDV3ANCNFSM5HZNFA3A .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .   https://github.com/notifications/beacon/AWLQK2XNUMDCBWNHB3A4I4DUMDDV3A5CNFSM5HZNFA3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHG6LZCQ.gif

OK, I've approved, see if you can merge it.

I think I was getting hung up on the first screen after pull request.  I never changed the branch.  I merged, but after I merged I see that I merged with eefb1ae?
Thanks
From: C-STYR @.>
Sent: Monday, November 15, 2021 3:00 PM
To: C-STYR/GitHubTrainer @.>
Cc: KWFelts @.>; Mention @.>
Subject: Re: [C-STYR/GitHubTrainer] added my goals (PR #16)
OK, I've approved, see if you can merge it.
—
You are receiving this because you were mentioned.
Reply to this email directly,  https://github.com/C-STYR/GitHubTrainer/pull/16#issuecomment-969315277 view it on GitHub, or  https://github.com/notifications/unsubscribe-auth/AWLQK2TFFVKIJUM45Q46ZXTUMFYEJANCNFSM5HZNFA3A unsubscribe.
Triage notifications on the go with GitHub Mobile for  https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 iOS or  https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub Android.   https://github.com/notifications/beacon/AWLQK2QTOLRQXO6OYWAKBOTUMFYEJA5CNFSM5HZNFA3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHHDJHTI.gif


Optionally disable number line

Personally I find that I almost never look at the line containing the number of items fzf is searching and the number currently matched (and occasionally the spinner), especially when I'm using fzf with vim/nvim. It would be nice to be able to optionally disable printing this and save a line, which on smaller screens can make a difference. Could an option like --no-info be implemented?

Thanks for the suggestion. But I don't think saving a line would make a big difference and would rather not add yet another option which I don't think will be widely popular. You can easily increase the height of :FZF split with g:fzf_height (default: 40%) or you can just use :FZF! on smaller screens. Also, since the info line shows the number of selected items when --multi option is set or if sorting is enabled when --toggle-sort option is used, it can be confusing to completely hide it. Instead I tried placing the numbers after the query on the last line, but I didn't like that how it moves along as I type in more characters.

Added --inline-info option, will be released with 0.9.11.

if has('nvim')
  let $FZF_DEFAULT_OPTS = $FZF_DEFAULT_OPTS.' --inline-info'
endif

@junegunn
Just chiming in to say I like the new --inline-info UI a lot.

agreed, @junegunn has an impressive history of listening to suggestions and improving them. great work as always

Thanks, I wasn't perfectly sure if it looks okay (colors, separator, ...), but glad to hear that you like it.


fix(test/{ MLList + finset_repr + trace + vec_notation }): silence tests

More silencing of tests, mostly trading
#eval (show MetaM Unit from) do

for
run_cmd Lean.Elab.Command.liftTermElabM do

Added import Mathlib.Tactic.RunCmd to test/MLList.lean.
Added #guard_msgs in test/trace.lean
Affected files:
test/MLList
test/finset_repr
test/trace
test/vec_notation

bors merge


Multi-GPU inference issue - "Expected all tensors to be on the same device"

System Info
Tried multiple use cases on hugging face with V100-32G node - 8 GPUs, 40 CPU cores on the node.

I could load the model to 8 GPUs but I could not run the inference code due to "RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:1 and cuda:3! (when checking argument for argument mat2 in method wrapper_mm)"

Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[ ] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[ ] My own task or dataset (give details below)

Reproduction
Example code (https://huggingface.co/google/flan-ul2):
from transformers import T5ForConditionalGeneration, AutoTokenizer
import torch
model = T5ForConditionalGeneration.from_pretrained("google/flan-ul2", torch_dtype=torch.bfloat16, device_map="auto")
tokenizer = AutoTokenizer.from_pretrained("google/flan-ul2")

input_string = "Answer the following question by reasoning step by step. The cafeteria had 23 apples. If they used 20 for lunch, and bought 6 more, how many apple do they have?"

inputs = tokenizer(input_string, return_tensors="pt").input_ids.to("cuda")
outputs = model.generate(inputs, max_length=200)

It throws:
File /anaconda/envs/azureml_py310_sdkv2/lib/python3.10/site-packages/torch/nn/modules/linear.py:114, in Linear.forward(self, input)
    113 def forward(self, input: Tensor) -> Tensor:
--> 114     return F.linear(input, self.weight, self.bias)

RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:1 and cuda:3! (when checking argument for argument mat2 in method wrapper_mm)

Expected behavior
the model could generate the outputs with the auto device map

The following code could work well:
from transformers import T5ForConditionalGeneration, AutoTokenizer
import torch
model = T5ForConditionalGeneration.from_pretrained("azureml-models/flan-ul2/1/flan-ul2", torch_dtype=torch.bfloat16, device_map="balanced_low_0")
tokenizer = AutoTokenizer.from_pretrained("azureml-models/flan-ul2/1/flan-ul2")
######### the following code is added to make it work################
from accelerate import init_empty_weights, dispatch_model, infer_auto_device_map, load_checkpoint_and_dispatch, load_checkpoint_in_model, dispatch_model
load_checkpoint_and_dispatch(
model,
<your_model_path>,
device_map=model.hf_device_map,
offload_folder=None,
dtype='float16',
offload_state_dict=True
)
################### end #####################
input_string = "Answer the following question by reasoning step by step. The cafeteria had 23 apples. If they used 20 for lunch, and bought 6 more, how many apple do they have?"
inputs = tokenizer(input_string, return_tensors="pt").input_ids.to("cuda")
outputs = model.generate(inputs, max_length=200)
print(tokenizer.decode(outputs[0]))

Executing your reproducer works perfectly on my side, and I get the following answer after adding this line:
tokenizer.decode(outputs[0].tolist())

' They have 23 - 20 = 3 apples left. They have 3 + 6 = 9 apples. Therefore, the answer is 9.'
Are you sure you are using the last version of Accelerate? Can you try with a source install as well?

@sgugger Thank you for your fast response.
The original code(the first comment) does not work with the V100-32G node (with 8 GPUs)
My second run does work(the second comment) after I added the load_checkpoint_and_dispatch() function.
Do you mean that you can run the code even with the code in the first comment on a multi-GPU node?

Yes, I can run the code at the top without any problem. Could you try with a source install?

I get exactly the same error with exactly the same example script. Have you found the solution?

@sgugger Thank you for your fast response.
The original code(my first comment) does not work with the V100-32G node (with 8 GPUs) My second run does work(my second comment) after I added the load_checkpoint_and_dispatch() function. However, it is not reflected on the model card (and I would suggest adding the load_checkpoint_and_dispatch function to the model card so that other people with multi-GPU could run it through in one shot)
Do you mean that you can run the code(without load_checkpoint_and_dispatch) in my first comment on a multi-GPU node?
I have been using the latest accelerate, with version as Version: 0.16.0 https://pypi.org/project/accelerate/

Not sure if this is a coincidence, I run into the same error with 4 V100. I'm going to see if running on 4 A100 will get to the same error.

@serenalotreck Your code sample contains a folder I don't have access to, so I can't execute it, but
checkpoint = 'hiepnh/vicuna-13B-1.1-HF-sharded'
model = AutoModelForCausalLM.from_pretrained(checkpoint, torch_dtype=torch.float16, device_map='auto')
tokenizer = AutoTokenizer.from_pretrained(checkpoint)
inputs = tokenizer("Hello my name is", return_tensors='pt')
inputs = inputs.to(0)
output = model.generate(inputs['input_ids'])
print(tokenizer.decode(output[0].tolist()))

works without any issue on my side.

@sgugger sorry about that! It's just my relative path to the repo obtained by running git clone https://huggingface.co/hiepnh/vicuna-13B-1.1-HF-sharded with git lfs`.
The model is too large to load into memory all at once on my machine, so I wanted to use load_checkpoint_and_dispatch specifically. If you replace the relative path I have there with the path to the repo I linked above, are you able to run my code as-is?

@serenalotreck The command I pasted will load the model shard by shard and is the best way to load big models for inference.

@sgugger thanks!
I'm currently running the code you posted, and it's been downloading shards for at least 5 minutes. When I run the code that I posted, which is directly basedon this tutorial, load_checkpoint_and_dispatch runs in a few seconds. I fully recognize I don't have a complete understanding of what's happening under the hood, and that potentially what's taking up time using AutoModelForCausalLM.from_pretrained() doesn't happen until I call generate in my own code, but I was wondering if you had thoughts on why what's suggested in the tutorial is faster and/or what the benefit of using the code you posted over what's in the tutorial would be?

@sgugger Thanks for the clarification, that was helpful! I ran it a second time and it was indeed faster.

By the way in your original sample of code, what is missing is the name of the modules to keep on the same device (which Transformers fetches for you):
model = load_checkpoint_and_dispatch(
    model,
    '../models/vicuna-13B-1.1-HF-sharded/',
    device_map='auto',
    no_split_module_classes=["LlamaDecoderLayer"],
)


Prawn error when using autowidth ("~") column

Using: Asciidoctor PDF 1.5.0.alpha.17 using Asciidoctor 2.0.9 [https://asciidoctor.org]
A Prawn::Errors::CannotFit (Prawn::Errors::CannotFit) is returned when a column is set to autowidth using ~:
[cols="1,~", options="header",]
|===
|Col1|Col2
|`0x00`|`UNSPECIFIED`
|`0x01`|`OK`
|===

If the ~ is removed (see below), the pdf generation works without problem:
[cols="1,5", options="header",]
|===
|Col1|Col2
|`0x00`|`UNSPECIFIED`
|`0x01`|`OK`
|===

Are you using the version of prawn-table from master? The released version
still has layout problems if memory serves me correctly.
On Fri, May 10, 2019, 11:36 Julien Pfefferkorn<EMAIL_ADDRESS>wrote:

Using: Asciidoctor PDF 1.5.0.alpha.17 using Asciidoctor 2.0.9 [
https://asciidoctor.org]
A Prawn::Errors::CannotFit (Prawn::Errors::CannotFit) is returned when a
column is set to autowidth using ~:
[cols="1,~", options="header",]
|===
|Col1|Col2
|0x00|UNSPECIFIED
|0x01|OK
|===
If the ~ is removed (see below), the pdf generation works without problem:
[cols="1,5", options="header",]
|===
|Col1|Col2
|0x00|UNSPECIFIED
|0x01|OK
|===
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/asciidoctor/asciidoctor-pdf/issues/1060, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAATL53XJ6I4L3D3ZSKYWXLPUVF2NANCNFSM4HMB5D7Q
.

I am using:
prawn (2.2.2)
prawn-icon (2.3.0)
prawn-svg (0.29.1)
prawn-table (0.2.2)
prawn-templates (0.1.1)

What should I use instead?

See https://github.com/asciidoctor/asciidoctor-pdf/blob/master/Gemfile#L8

Thanks!

I just tested with prawn-table master and it seems like this is still a problem.

Oh, I see. The autowidth column is simply not implemented in Asciidoctor PDF.

Actually, the problem is that you're trying to set the column width to a value which is too small. Effectively, you're using cols="1,99". Prawn does not resize a fixed-width column based on content. Thus, the result is a cannot fit error. You need to reserve more room for the first column. For example, cols="10,~". This is a fundamental limitation in Prawn.

Thanks for the clarification. I did not realize that the col width becomes a percentage when at least 1 column width is set to ~ (but it is indeed very clearly documented in the user guide)
If I remember well, in the antora output the size of the 1st column wasn't constrained to 1% (I have to check though), but seems to use the as much as needed to fit the content. Is it intended to be like that?

OK. I just checked, the HTML output from antora.
It contains width: 1%, the browser (chrome and firefox) nevertheless not constrain the width to 1%, but gives that column more space.
That's what mislead me...
Below an example with 3 cols:


Add api token action to handle creation of api token index

Description
Adds an API action to handle creation of API tokens index during PUT calls
Issues Resolved
[List any issues this PR will resolve]
Is this a backport? If so, please add backport PR # and/or commits #, and remove backport-failed label from the original PR.
Do these changes introduce new permission(s) to be displayed in the static dropdown on the front-end? If so, please open a draft PR in the security dashboards plugin and link the draft PR here
Testing
[Please provide details of testing done: unit testing, integration testing and manual testing]
Check List

[ ] New functionality includes testing
[ ] New functionality has been documented
[ ] New Roles/Permissions have a corresponding security dashboards plugin PR
[ ] API changes companion pull request created
[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Closing in favor of: https://github.com/opensearch-project/security/pull/4921


运行 generate.py的时候报错，cudnn 已经安装了啊？

(vicuna) hope@hope-08:~/work/Chinese-Vicuna$ python generate.py
Traceback (most recent call last):
File "generate.py", line 2, in
import torch
File "/home/hope/miniconda3/envs/vicuna/lib/python3.8/site-packages/torch/init.py", line 217, in
_load_global_deps()
File "/home/hope/miniconda3/envs/vicuna/lib/python3.8/site-packages/torch/init.py", line 177, in _load_global_deps
raise err
File "/home/hope/miniconda3/envs/vicuna/lib/python3.8/site-packages/torch/init.py", line 172, in _load_global_deps
ctypes.CDLL(lib_path, mode=ctypes.RTLD_GLOBAL)
File "/home/hope/miniconda3/envs/vicuna/lib/python3.8/ctypes/init.py", line 373, in init
self._handle = _dlopen(self._name, mode)
OSError: libcudnn.so.8: cannot open shared object file: No such file or directory

谢谢，我重新安装了 cuda 12，cudnn 前面不报错了，新问题来了，在找原因，呵呵
(vicuna) hope@hope-08:~/work/Chinese-Vicuna/scripts$ bash generate.sh
Traceback (most recent call last):
File "/home/hope/work/Chinese-Vicuna/generate.py", line 3, in
from peft import PeftModel, PeftModelForCausalLM, LoraConfig
File "/home/hope/miniconda3/envs/vicuna/lib/python3.8/site-packages/peft/init.py", line 22, in
from .mapping import MODEL_TYPE_TO_PEFT_MODEL_MAPPING, PEFT_TYPE_TO_CONFIG_MAPPING, get_peft_config, get_peft_model
File "/home/hope/miniconda3/envs/vicuna/lib/python3.8/site-packages/peft/mapping.py", line 16, in
from .peft_model import (
File "/home/hope/miniconda3/envs/vicuna/lib/python3.8/site-packages/peft/peft_model.py", line 22, in
from accelerate import dispatch_model, infer_auto_device_map
File "/home/hope/miniconda3/envs/vicuna/lib/python3.8/site-packages/accelerate/init.py", line 3, in
from .accelerator import Accelerator
File "/home/hope/miniconda3/envs/vicuna/lib/python3.8/site-packages/accelerate/accelerator.py", line 36, in
from .tracking import LOGGER_TYPE_TO_CLASS, GeneralTracker, filter_trackers
File "/home/hope/miniconda3/envs/vicuna/lib/python3.8/site-packages/accelerate/tracking.py", line 49, in
import wandb
File "/home/hope/.local/lib/python3.8/site-packages/wandb/init.py", line 26, in
from wandb import sdk as wandb_sdk
File "/home/hope/.local/lib/python3.8/site-packages/wandb/sdk/init.py", line 5, in
from .wandb_artifacts import Artifact  # noqa: F401
File "/home/hope/.local/lib/python3.8/site-packages/wandb/sdk/wandb_artifacts.py", line 33, in
from wandb.apis import InternalApi, PublicApi
File "/home/hope/.local/lib/python3.8/site-packages/wandb/apis/init.py", line 42, in
from .internal import Api as InternalApi  # noqa
File "/home/hope/.local/lib/python3.8/site-packages/wandb/apis/internal.py", line 1, in
from wandb.sdk.internal.internal_api import Api as InternalApi
File "/home/hope/.local/lib/python3.8/site-packages/wandb/sdk/internal/internal_api.py", line 45, in
from ..lib import retry
File "/home/hope/.local/lib/python3.8/site-packages/wandb/sdk/lib/retry.py", line 17, in
from .mailbox import ContextCancelledError
File "/home/hope/.local/lib/python3.8/site-packages/wandb/sdk/lib/mailbox.py", line 10, in
from wandb.proto import wandb_internal_pb2 as pb
File "/home/hope/.local/lib/python3.8/site-packages/wandb/proto/wandb_internal_pb2.py", line 1, in
import google.protobuf
ModuleNotFoundError: No module named 'google.protobuf'

没有的库用pip或者conda装不就行了？


Fix use of ocaml re package

The ocaml re package is used for error messages. But it seems to be crashing when using ocaml 4 on some machines.

[x] Assess if the packages is really needed or if hax could do without.
[ ] If it is really needed, assess the issues with ocaml 4 and either fix them or require ocaml 5 for hax.

re is used to link our printed Rust to pretty printed rustfmt Rust, this pretty printing can be disabled with HAX_RUSTFMT=no.
We need to use the new generic printer to write a Rust printer, and then we can stop using rustfmt within the engine.

Quoting @franziskuskiefer: "running it on some of the Rust crypto crates like sha2 or hmac will run into that issue"
Let's investigate when we have time, we should have a reproducer here. It makes no sense that re behavior changes between OCaml 4 and OCaml 5.


Publish specs on github pages

This PR will create a static html file listing the pdf files generated in the build.
A current limitation is that the pdfs need to have unique names, otherwise only one of them is kept.
At the moment,  in master, this is what is happening, because both ledger and midnight pdfs are called PDF.pdf , however this PR  addresses this, so, after it is merged,  the page will show to different pdfs called "cardano-ledger.pdf" and "midnight-example.pdf".
An example of how this page looks like: https://bienpulenta.github.io/formal-ledger-specifications
In order for the github page to be created, we need to change the "Pages" Settings in this repo, and set the deploy-source to branch "gh-pages", like in this example screenshot:

Indeed, force-pushed to rebase


Generalize alloc_box test for different platforms

Different platforms may number their labels a bit differently. For example, on s390x, the check match label:10: instead of lable:9:. By removing the exact number, I am trying to generalize this test for more platforms.

Why are we IRGen’ing a different label on s390x?

Why are we IRGen’ing a different label on s390x?
There is an extra lshr instruction due to the endiness issue on s390x, happening before label:9:. There on s390x, it is label:10: instead. Btw, why does it check 9 specifically?

Alright.  I think the label's in there to get FileCheck to move directly to the line before the call to swift_deallocateUnusedObject.  But if you delete this and match the ret at the end of the function it does the same thing.
// CHECK-LABEL: @"$s9alloc_boxyyXEfU_"
// CHECK: call void @swift_deallocUninitializedObject
// CHECK-NOT: call void @swift_setDeallocating
// CHECK: ret void

Thanks, and i have modified the commit.

@swift-ci please smoke test and merge

@swift-ci please smoke test and merge

@swift-ci please smoke test

⛵

@CodaFi, thanks.


BUG: Added Picture in landing page is not responsive.

Is there an existing issue for this?

[X] I have searched the existing issues

What went down?

I will make it responsive.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[ ] I'm a VSoC'24 contributor
[X] I want to work on this issue

@mdazfar2 please assign me !

@zalabhavy we are currently working to the landing page and will assign it to you once it is finalized.


Swagger Key

- [x ] bug report -> please search issues before submitting
- [ x] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

I have tried multiple approaches to specify a key on swagger page in order to "Try It"

using just generated Bearer Token
Malirezai posts  here https://github.com/malirezai/WorkbenchAzureFunctionAPI  and here https://medium.com/@malirezaie/how-to-enable-programmatic-interaction-with-azure-blockchain-workbench-apis-56c0d95c79c0

Nothing helps.
Please, help.

Even I am facing same issue. Can you please suggest which key to use?

Hello @ygizhitsa  and @Dpkkmr , can you post images of the errors that you are getting when using bearer token and following Malirezai's post?
Thanks,
Ramya

Hello @ygizhitsa  and @Dpkkmr, if you have tried Postman for this since Postman lets you specify a bearer token, can you let me know if that helped?
Thanks,
Ramya


Add multiply and sum methods to Bytes class.

Hi! Thanks for such cool lib!
Is it possible to implement Bytes.multiply and Bytes.sum methods?

Thanks I'll look into it :)

So the problem with this feature is that it would just replicate the features of BigInteger.
I would be open to add it as a BytesTransformer (or so could you in your project), so if you're interested I'll encourage you to create a PR.
This would add a new inner class BytesTransformer and should probably support inPlace altering and should have signed/usinged mode.


Cannot retrieve token 64_11_0

Returns Not Found but it should exist as it shows on Litho https://lithoverse.xyz/nft/64/11/0

 Resolve issues with NFT API

might be cause right now we are scanning data from ~ 1st Feb 2022 onwards

We can rescan to check this... as we did not take into account batch utility earlier... not sure of the cause here though

https://nft-api.centrality.me/nft/token/64_11_0 this is now available... we can close this one.


don't set frame background-color

This was causing the current editor to show an ugly white background over the dark theme.

This is the cause of the current white background for web samples on the playground UI, right?

yeah, it was a regression


new C# file icons

I made (ripped off rider) 2 new icons for C# and unity MonoBehaviour. since the current one is just from IntelliJ idea and looks kinda out of date.
Here they are:

This is the base, normal C# file. Here is one for unity Monobehaviour:

You can use these exact one if you want! but here are the photoshop files in case you wanna modify it your self. I think I don't know enough about java or software to commit to this repo... Maybe one day!
Here are the photoshop files.

Hello. Thanks for you suggestion, but i will refuse current suggestion. Why.
First of all i wanna update all icons from IDEA but with my fixes. File icons not updated for now, due i have ,any icons which is need make by self.
But - there one thing. I have new functional named "Icon Theme", plugins can provide own icons.
Later i want add Visual Studio icon theme from public Visual Studio Image Library (https://www.microsoft.com/en-us/download/details.aspx?id=35825).  But there one thing. Each icon need repack to inline svg attributes (svg is prefer over png, due it have better scaler options).
If you want - i can create plugin, with VS icons (only few, due i don't want lose time for it) and you can update it, and then use plugin.

If you want - i can create plugin, with VS icons (only few, due i don't want lose time for it) and you can update it, and then use plugin.

That would be awesome! btw have you considered making a public trello
 ? It is a site where you can organise tasks and lots of cool stuff! If you already have one, can I get the link 😄 ? If you do make the plugin, I'll be happy to update it with many of my vs code favourites and others! although ill have to relearn java cuz it's been 2 years since I've written java (assuming that the majority of that plugin might be written in java). Thanks!

Github have own small trello implementation.
https://github.com/orgs/consulo/projects
But it's hard for me control this list, due when i don't have ideas for some issues - i'm moved to another plugin.
Too many plugins, which i want. Too many issues for one person :D
Thanks, i will create new icon plugin now.

Also i have project like this https://github.com/consulo/long-term-todo/issues

Yea, I feel you! This is such a big project for 1 person, I am trying to help by implementing few features ( or trying to ) like this one :D also I know that GitHub has its own implementation but it is quite limited.... I recommend trello more as you can organise stuff way faster and divide them up, making it very organised and easier to keep track of ! You can make lists for all the repos in one place instead of GitHub projects in every repo, ( I don't know if you actually do make projects in all repos ) so just a friendly suggestion, give trello a shot ( it has plugins too ! I use custom fields plugin the most ) it will make the development faster, I think!
Anyway, you do you !
Thanks.

Short list
https://github.com/consulo/EntryPoint/tree/master/repositories

Initial version.
https://github.com/consulo/incubating-consulo-visual-studio-theme
I am writing readme for it

Initial version.
https://github.com/consulo/incubating-consulo-visual-studio-theme
I am writing readme for it

thanks a lot! But currently, consulo is just crashing, #427 issue


Results browser tweaks

See #195
Added some tests, cleaned up some HTML and moved the template file location.

@jmancewicz, can you take a look at this?

Rebased to master after #249 changed the autodoc files. See #257 for how to avoid this type of thing in the future.

@jmancewicz, updated to include your suggestions from https://github.com/lukeyeager/DIGITS/commit/c7aa5d1418033d484272595d747ed7f04ed64fb3#commitcomment-13007232. Does this look better?
Next time, add comments to the PR diff instead of to the comment itself so that the comment stays attached to this PR. See https://github.com/NVIDIA/DIGITS/pull/249#discussion-diff-38360470 as an example.


[server][fc] Integrate server side handlers with gRPC service + mTLS

[server][fc] Integrate server side handlers with gRPC service + mTLS

Adapts Netty Channel pipeline and corresponding handler classes to be compatible with gRPC requests
Can collect server side statistics and enforce read quota for gRPC requests
Introduce mutual TLS support on gRPC enabled servers and fast client, when an SSLFactory is present
Update unit tests and integration tests

How was this PR tested?

CI/Unit Tests
Does this PR introduce any user-facing changes?

[ ] No. You can skip the rest of this section.
[x] Yes. Make sure to explain your proposed changes and call out the behavior change.
If SSL is enabled for a gRPC enabled Venice Server, we start the gRPC server with SSL active and Client Auth required. If this is the case, we have to pass in an SSLFactory instance in the Client Configs when creating a gRPC enabled FC. If this isn't the case, then we would not be able to connect to a Venice Server instance.

🚀