Implement context header mappings - Feature Request Issue #939

Description
Adds a setting that allows mapping an API Gateway context variable to an HTTP header. Document the feature in the readme and add unit tests.
GitHub Issues
https://github.com/Miserlou/Zappa/issues/939

Coverage increased (+0.1%) to 73.097% when pulling efeabc1a2570f31becdb3e4a76592e2e3a007fe1 on brianrower:feature/context-to-http-header-mapping into a07ddbaee7a2a8c8bf5bb45e536865f9a361509e on Miserlou:master.

Coverage increased (+0.1%) to 73.097% when pulling efeabc1a2570f31becdb3e4a76592e2e3a007fe1 on brianrower:feature/context-to-http-header-mapping into a07ddbaee7a2a8c8bf5bb45e536865f9a361509e on Miserlou:master.

Coverage increased (+0.1%) to 73.097% when pulling efeabc1a2570f31becdb3e4a76592e2e3a007fe1 on brianrower:feature/context-to-http-header-mapping into a07ddbaee7a2a8c8bf5bb45e536865f9a361509e on Miserlou:master.

Coverage increased (+0.1%) to 73.097% when pulling efeabc1a2570f31becdb3e4a76592e2e3a007fe1 on brianrower:feature/context-to-http-header-mapping into a07ddbaee7a2a8c8bf5bb45e536865f9a361509e on Miserlou:master.

Coverage increased (+0.1%) to 73.097% when pulling efeabc1a2570f31becdb3e4a76592e2e3a007fe1 on brianrower:feature/context-to-http-header-mapping into a07ddbaee7a2a8c8bf5bb45e536865f9a361509e on Miserlou:master.

Coverage increased (+0.1%) to 73.097% when pulling efeabc1a2570f31becdb3e4a76592e2e3a007fe1 on brianrower:feature/context-to-http-header-mapping into a07ddbaee7a2a8c8bf5bb45e536865f9a361509e on Miserlou:master.

Coverage increased (+0.1%) to 73.131% when pulling f9b71f72079e00da6fb6bc4fb45486be226d2119 on brianrower:feature/context-to-http-header-mapping into 7085972a50836abe893e0c8deff8a51deb1b005e on Miserlou:master.


add pasteasy functionality

Done: add pasteasy functionality
To do: 1) allow photo gallery to work for pictures of different sizes
2) rotation

actually pasteasy part is just a thin layer of fs.watch, low level though
it's nice to have it as a isolated part and configure it in the top level of code, but i prefer to implement it as a Class or function for example:
var dir_watcher = new DirWatcher(opt);

for better reuse of code, also DirWatcher is a essential part of our program if you consider data persistent on disk.
That's the reason why i comment that part out, cause i don't want it to be that way.


Midterm Blog Post

I have added midterm blog post report in content/report/osre23/uga/GenomicsWFModels/20230803-charishulu/

I cherry-picked commit 61aea5d into pull request #246 and merged it. Closing this one.


utf-8 errors in Python 2 (ok in Python 3)

when importing nfe's with non ascii chars in Python we have this kind of errors:
>>> len(str(u'blé'))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
UnicodeEncodeError: 'ascii' codec can't encode character u'\xe9' in position 2: ordinal not in range(128)

the generated code would need to be rewritten like:
>>> len(str((u'blé').encode('utf-8')))
4

(And this works with Python 2 also)
As the error occurs only with Python 2, do we care about it?

fixed


New update

Black screen apon starting teardown with the new grav gun mod

Can't reproduce after a clean install. Neither the current release (0.5.2) nor the current experimental version (0.5.3) lead to a black screen.
Please attempt a clean install making sure your game version is not 0.4.6


[scatter] the legend does not make sense.

Instead of one of the StatVars, should it be ""? Or, just not have a legend?

To clarify: https://screenshot.googleplex.com/8fzbYv3q3yE22qD

Thanks.


[hotfix] [EOSF-514] Use providers list from APIv2 instead of hardcoded list

Ticket
https://openscience.atlassian.net/browse/EOSF-514
Purpose
Updates the discover route so that Preprint Providers in from the API are used in the whitelist (not hardcoded)
Changes
/preprints/discover (OSF Preprints only) should now have all of the providers available. Providers in the API, but not in SHARE will be displayed with with a document count of 0.

Side effects
Providers should now load automatically when added to the API

Thought I was going to use loadAll, but realized I needed findAll instead, so this should be g2g once CI finishes


Topics comparison with inconsistent number of topics / date selection sliders

Here I selected 5 topics.
I would expect to see 5 lines in the plot with one unique date slider. Instead there are 4, it is not clear how they interact with each other nor why they are 4 instead of 5 (or... of 1!)

I changed the time resolution to Yearly and the number of sliders has grown

Now I removed three of the 5 topics. Correctly, the corresponding lines in the plot are also removed, but the sliders remain.

Hi, Thanks for reporting the issue. It's resolved and new changes have been pushed.


Q22Z

Use the preview tab to see the example given below. Edit the relevant information and preview the changes before submitting. You may delete these instructions.

Key
Value

ID
Q22Z

Source
Vaaran Bhai Gurdas Ji

Page
22

Line
gur isKhu gur isKu, hoie hYrwixAw ]12]

Correction
THIS ≠ THAT

PROOF (EDITION)
Image:

Correction:
gur isKhu ≠ gurisKhu
Proof:
Vaaran Bhai Gurdaas Ji | November 2011 | Shamsher Singh Ashok | SGPC, Sri Amritsar
Image:


Docs: MP gRPC Client states 'class' when should be 'interface'

Source: https://helidon.io/docs/latest/#/mp/grpc/02_mp_clients (via /docs/mp/grpc/02_mp_clients.adoc)
Example:
@ApplicationScoped
@io.helidon.microprofile.grpc.core.Grpc
public class StringService {  // <-- that should be interface

    @io.helidon.microprofile.grpc.core.Unary
    public void upper(String s, StreamObserver<String> response);  // <-- public not needed
}

There's 3 examples in the ASCIIDoc that need altering.
Also, I question that it's under the section gRPC servers which I think is due to sitegen.yaml#L115 and sitegen.yaml#L272.

Fixed by PR #2457


John Olutubo: Headless-CMS,  3 weeks

my fork of module repository

Learning Objectives

headless cms
Getting Started
How to study the code in this repo

Installing node

You will need [NPM](https://docs.npmjs.com/downloading-and-installing-node-js-and-npm) and [nvm](https://github.com/nvm-sh/nvm#installing-and-updating) on your computer to study this material

You will also need [Postman](https://www.postman.com), please install it on your local machine.

All strapi installations will use
<EMAIL_ADDRESS>as username and
Testuser1 as password

Learning goals

[x] - 🥚 Understanding what a content management system is, and why people use it
[ ] - 🥚 Given an existing CMS installation fill in and publish a new piece of content
[ ] - 🥚 Perform a simple http call via postman to retrieve a piece of information.
[ ] - 🐣 Extend and adapt an existing CMS configuration
[ ] - 🐣 Perform a http call with filtering via Postman
[ ] - 🐣 Perform a http call with "include" via Postman
[ ] - 🐣 Perform a http POST call to create new data
[ ] - 🐣 Perform a http PUT call to update existing data
[ ] - 🐣 Configuring endpoint permissions (Authentication vs Authorization)
[ ] - 🐥 Design a data model from scratch based on user requirements

Week 1

[ ] I have pushed my progress to my fork of exercises repo

I Need Help With:
Running strapi on the forked repo with my VScode
What went well?
Understanding CMS and the use of strappi
What went less well?
same as mentioned in what I need help with
Lessons Learned:
Designing data models
Sunday Prep Work
...ongoing

Week 2
I Need Help With:

Not yet

What went well?

Using postman

What went less well?

Nothing for now

Lessons Learned

request methods

Sunday Prep Work

ongoing

Week 3
I Need Help With:

No needed help observed so far

What went well?

Using postman

What went less well?

Nothing for now

Lessons Learned

Authentication, posting, deleting, adding content through APIs.

Sunday Prep Work

ongoing


example field is not working using a $ref

Describe the bug
example field is not "populated" on studio.
payload output generated on the studio:
{
  "id": "497f6eca-6276-4993-bfeb-53cbbbba6f08",
  "type": "string"
}

when it should be:
{
  "id": "497f6eca-6276-4993-bfeb-53cbbbba6f08",
  "type": "product_stock"
}

How to Reproduce
asyncapi: 2.5.0
info:
  title: Warehouse Domain
  version: 1.0.4
  description: This domain contains warehouse's product stock information
channels:
  product_stock:
    subscribe:
      message:
        $ref: '#/components/messages/product_stock'
components:
  schemas:
    id:
      type: string
      format: uuid
      description: id in UUIDv4 format
    type:
      type: string
      description: type of the event
  messages:
    product_stock:
      summary: lorem ipsum
      description: >-
        lorem ipsum
      payload:
        type: object
        additionalProperties: false
        required:
          - id
          - type
        properties:
          id:
            $ref: '#/components/schemas/id'
          type:
            $ref: '#/components/schemas/type'
            example: product_stock

Expected behavior
The payload output generated on the studio must show:
{
  "id": "497f6eca-6276-4993-bfeb-53cbbbba6f08",
  "type": "product_stock"
}

@dcorto You try to "extend" the object by $ref with other keywords in this same level. AsyncAPI doesn't support such a thing and parser "remove" other keywords on this same level as $ref keyword.
Please check with this:
          type:
            example: product_stock

And you will see:

Why do you try to do something like that? Are you have a OpenAPI background? If yes, then AsyncAPI is different in some parts, so please don't treat AsyncAPI structure/possibilities == OpenAPI structure/possibilities.
Any questions?

Hi @magicmatatjahu thanks for the feedback. I get the point :)
But why if i go to Generate code/docs and i choose "HTML website" the generated html, works as I expect on the issue That is why i'm confused.
Also if I generate the HTML with the docker image asyncapi/generator it's generate as i expect.
Thanks in advance :)

@dcorto Ahh yes, we have some differences between Studio and other tools. Out core tool - ParserJS - will use another reference library underneath in next major version and now, only Studio uses that new version, another tools uses the old one. In older version of ParserJS, the reference library support such a references, while in the AsyncAPI specification we do not support and this is a "bug" on the side of the tool. If I were you, I would not use such references because they will be "deprecated" in the future :)

All clear! :100:
Thanks for your quickly responses.
I close the issue


Upgrade workflows to actions/checkout@v3

Upgrade workflows to actions/checkout@v3

Spec failure fixed by https://github.com/ManageIQ/manageiq-providers-ibm_cic/pull/23


Remove travis

@kytrinyx after we have GitHub Actions available on the organisation and it will be publicly available in November, I'm in a process of removing travis-ci from the erlang track and associated repositories and replacing it with equivalent work flows.
I now have added a stub .travis-ci.yml which does nothing but printing a message.
Could you disable travis-ci completely or do we want to keep it, in case we need it again?

I can disable it completely, thanks!


Fix Version Aliases

Description
There exists a scenario where when we resolve properties, someone may have jackson.version and another may have version.jackson. This won't pass our duplicate checks up front, but will conflict later down the road when we apply the version alias generator.
For now, we are removing version aliases from everything except for the root BOM. For any transitive BOMs, replace all properties with the resolved value even if they are unique.
We may re-add transitive properties back in at a later time, possibly by prefixing with the artifact (or something along those lines).

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (72bf953) 82.89% compared to head (ae1a1d6) 86.07%.

Files
Patch %
Lines

...kotlin/dev/aga/gradle/versioncatalogs/Generator.kt
97.29%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #90      +/-   ##
============================================
+ Coverage     82.89%   86.07%   +3.18%
  Complexity       77       77
============================================
  Files            10       10
  Lines           339      316      -23
  Branches         56       57       +1
============================================
- Hits            281      272       -9
+ Misses           28       15      -13
+ Partials         30       29       -1

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


Clean up Dockerfile

Current Dockerfile is convoluted. We need to move to multi-stage builds to keep the image size under control.

Closed by #54


Examples panic in glyphon code

Is there an existing issue for this?

[X] I have searched the existing issues.

Is this issue related to iced?

[X] My hardware is compatible and my graphics drivers are up-to-date.

What happened?
On the master branch, the tour example crashes with the following error:
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', /home/kai/.cargo/git/checkouts/glyphon-70ff9ac92aaa9d8a/f145067/src/text_render.rs:105:30
What is the expected behavior?
The example should run.
Version
master
Operative System
Linux
Do you have any log output?
wgpu examples run fine.

Git bisect tells me that this is the first bad commit: 9b4bcd287a7f4822314e158990d1dc023d5aab51

I am also attaching a full log file.

out.log

I believe #1883 should fix the issue.

Thanks! It did fix the problem for me.
(Seems like cosmic_text does not find any fonts on my system, which might be why some/all glyphs are missing)


Fixed the entropy calculation of the von Mises distribution.

(scipy.stats.vonmises)

check_private_entropy says:

compare a generic _entropy with the distribution-specific implementation

but the generic entropy is broken.  I have no idea how to fix this.

You'll need to skip the test if distname =='vonmises'

@@            master   #6013   diff @@
======================================
  Files          238     238
  Stmts        43822   43824     +2
  Branches      8215    8215
  Methods          0       0
======================================
+ Hit          34251   34253     +2
  Partial       2603    2603
  Missed        6968    6968

Review entire Coverage Diff as of 8f6bc2a

Powered by Codecov. Updated on successful CI builds.

Now the last thing would require a bit of wrangling with git, I'm afraid. First, please don't merge master into feature branches. Workflows differ; we typically prefer rebasing on master if necessary. Second, six commits for ten LOC is a bit much (there's no hard rule to limit the number of commits per PR, but here it seems fairly clear). Thus --- would you please squash the commits. While you're at it, it could be a good time to also rewrite the commit message to start with a standard acronym (http://docs.scipy.org/doc/numpy-1.10.1/dev/gitwash/development_workflow.html). Something like "BUG" would be appropriate, I'd think. Thanks!

Squashed and merged in 58b4f54ea77e47b754bcf237f0149e95a5511770.
Thanks @NeilGirdhar and congratulations with the first scipy commit! (6c7ec1d2c7be60a2a3b482eb7f4e889288707ecb)

@ev-br  Thanks for taking care of the squash.


Find a way to have the react-native app convert to a react app when used on the phone.

I would like either the react-native application to be running the same way on the computer as well or a way to interchange the screen when rendered on different devices.

adding expo to the repo would be the right way to go about the issue


Eventhub

Description
New module migrated from CARML to AVM.
Related to module proposal https://github.com/Azure/Azure-Verified-Modules/issues/611
Status badge:

Note: the original CARML module didn't have the idempotency test loop.
I've addedd it to most of the test, but with the ecryption one the iteration gives conflict issues, so on the encryption test i had to remove it.


Fix istioctl pc secret cert validity not accurate

Please provide a description of this PR:
This is found when I try to access my ambient service, the waypoint cert is expired which lead to service not being accessiable, but istioctl pc secret gives me the valid result:
istioctl pc secret namespace-istio-waypoint-868957c498-8n469.ambient
RESOURCE NAME     TYPE           STATUS     VALID CERT     SERIAL NUMBER                        NOT AFTER                NOT BEFORE
default           Cert Chain     ACTIVE     true           6fbee254c22900615cb1f74e3d2f1713     2023-05-16T01:32:52Z     2023-05-15T01:30:52Z
ROOTCA            CA             ACTIVE     true           193a543fe2b0d9cd4847675394dfc54      2033-05-02T03:41:33Z     2023-05-05T03:41:33Z

After the fix it should be:
RESOURCE NAME     TYPE           STATUS     VALID CERT     SERIAL NUMBER                        NOT AFTER                NOT BEFORE
default           Cert Chain     ACTIVE     false          6fbee254c22900615cb1f74e3d2f1713     2023-05-16T01:32:52Z     2023-05-15T01:30:52Z
ROOTCA            CA             ACTIVE     true           193a543fe2b0d9cd4847675394dfc54      2033-05-02T03:41:33Z     2023-05-05T03:41:33Z

To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[x] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[ ] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

@hanxiaop: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
f1c4ed05376c3ea52ee6f66fb48b169311477f38
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
f1c4ed05376c3ea52ee6f66fb48b169311477f38
link
true
/test release-notes

lint_istio
f1c4ed05376c3ea52ee6f66fb48b169311477f38
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/cherry-pick release-1.18

@hanxiaop: new pull request created: #45343

In response to this:

/cherry-pick release-1.18

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/cherry-pick release-1.17

/cherry-pick release-1.16

@hanxiaop: new pull request created: #45344

In response to this:

/cherry-pick release-1.17

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@hanxiaop: new pull request created: #45345

In response to this:

/cherry-pick release-1.16

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


how to train

Thank you for your wonderful work!!!
I have two questions about how to train self-annotated dataset:
1、Could you please show me yourself annotated dataset format screenshot, should there be all categories of targets in the annotation .txt file of each frame, like if i want to train bike,car and person, but the first frame just has car, this will be ok for training? And if i want to train other classes not  the 5 classes detection & tracking: car, bicycle, person, cyclist, tricycle you list, could you please tell me where should i change in the code?
It would be better if you could share the dataset you annotated.
2、I'm not sure this code can use the DCNV2 to train in this repository, if not,could you please tell me if this is feasible?
Thank you again for this MCMOT contribution,Look forward to your reply!!!Thanks a lot.

The annotated dataset  is just no difference from FairMOT except the first column in each row, which is the actual class index of object type, like:
0 1 0.650781 0.725463 0.193750 0.225926
0 2 0.523177 0.587037 0.164583 0.178704
0 3 0.458333 0.498148 0.133854 0.152778
0 4 0.080469 0.385648 0.141667 0.305556
0 5 0.017188 0.157407 0.034896 0.063889
1 1 0.778385 0.723148 0.077083 0.119444
1 2 0.763281 0.672685 0.088542 0.124074
1 3 0.719010 0.595370 0.111458 0.130556
1 4 0.669010 0.580093 0.091667 0.105556
1 5 0.381510 0.331481 0.046875 0.075000
1 6 0.373698 0.324537 0.063542 0.074074
2 1 0.948177 0.683796 0.043750 0.183333
3 1 0.262500 0.395833 0.034896 0.122222
3 2 0.311719 0.370370 0.034375 0.110185
3 3 0.141146 0.156481 0.014063 0.054630
2 2 0.830469 0.493519 0.042708 0.123148
0 means car for my pre-trained model, 1 for bicycle, 2 for person, 3 for cyclist, 4 for tricycle.
The actual meaning is: class_id, track_id, bbox_center_x, bbox_center_y, bbox_width, bbox_height
As for the second question, the answer is yes. DCNV2 is compiled under my python3 site-packages, so it is a little different from the way FairMOT import dcnv2. Besides, other backbones like Resnet series is also applicable.

Thanks for your reply!!!
So my first question is
0 1 0.650781 0.725463 0.193750 0.225926
0 2 0.523177 0.587037 0.164583 0.178704
1 1 0.778385 0.723148 0.077083 0.119444
1 2 0.763281 0.672685 0.088542 0.124074
2 1 0.948177 0.683796 0.043750 0.183333
3 1 0.262500 0.395833 0.034896 0.122222
3 2 0.311719 0.370370 0.034375 0.110185
2 2 0.830469 0.493519 0.042708 0.123148
for example,this annotated .txt is just the first frame(I think one .txt of annotated picture is just for one frame, am i right?),but there is not the class_id 4 for tricycle, is that be ok for training? Once again thank you for your reply!!!

Yeah, one txt label file for one image, but no txt label for pure backgound image is ok too.
The 5 classes objects are not all necessary for each image, the training will be ok.

For custom dataset training, just modify the configuration in opts.py, my is:
self.parser.add_argument('--reid_cls_ids',
default='0,1,2,3,4',  # car, bicycle, person, cyclist, tricycle
help='')  # the object classes need to do reid
If you have 6 object classes and only need 3 object classes to do tracking(detection+reid) and the other 3 classes only do detections, it's ok to set reid_cls_ids less than 6.
If you only want to do detection, just make track_id to 0 will be ok.

Thank you very much for your detailed reply,it really helped me a lot.

Thank you for your wonderful work!!!
I have two questions about how to train self-annotated dataset:
1、Could you please show me yourself annotated dataset format screenshot, should there be all categories of targets in the annotation .txt file of each frame, like if i want to train bike,car and person, but the first frame just has car, this will be ok for training? And if i want to train other classes not the 5 classes detection & tracking: car, bicycle, person, cyclist, tricycle you list, could you please tell me where should i change in the code?
It would be better if you could share the dataset you annotated.
2、I'm not sure this code can use the DCNV2 to train in this repository, if not,could you please tell me if this is feasible?
Thank you again for this MCMOT contribution,Look forward to your reply!!!Thanks a lot.

@LrobotD Hi, How did you manage to self-annotate data/videos for tracking and generate Ground truth because I am working on a dataset which has no annotations. Although, I Was able to generate BB annotations for all the frames but not sure how to link them up for the test of tracking and generate a txt file for each image. If you used some tool for the same, I would love to hear from you.

@CaptainEven It would be great if you can explain properly, steps to take while training on custom dataset such as parameters to change in the opts file and the model to be used and where that model is available for download.
Thanks a lot for providing this code but it will be great if you can add the above mentioned things in your repository to make it more understandable for people trying this code on their custom datasets.

Sorry for this primitive question.
When I want to change backbone for training, is it collect to add like '--arch resdcn_34'?
Or do I need to replace the first consumed pretrained model, which is '../exp/mot/default/mcmot_last_track_resdcn_18_visdrone.pth' by default?


Example fallback

@artyom-beilis if you want to improve the error message when an op is not implemented for ocl (first commit) or make it fallback to using the CPU for ops that are not implemented (second commit).
You don't have to merge this, but I felt  it was the simplest way to share the idea with you!
If you want some other variation of this, feel free to share it and we can see what can be done!

Thanks a lot.
I think I got the idea. I merged it so I can see how to use it. Actually I had several cases when automatic CPU fallback have saved me lots of troubles!

Looks like I'll need to add aten::_copy_from_and_resize to make it work properly
Thanks again :-)

No worries! Happy this helps!


Labeling of coronary arteries

Hello, thank you for the great package!
I successfully segmented the coronary arteries in one of my CTA cases, but I was wondering if there was a way to obtain different segmentations for the different arteries (e.g., RCA, LCX, etc).
If not, what should be a starting point to tackle this problem ?
Thank you,
Francesco

So far there is no way to get the different arteries. You can try to split it up by some postprocessing logic. Maybe looking at the proximity to the different heart chambers. But this could be a bit tricky.


self.debug does not exist

https://github.com/htshinichi/caffe-onnx/blob/master/src/caffe2onnx.py#L481
run into an issue here that self.debug does not exist in the class.

@LiyouZhou @htshinichi same issue here.

@LiyouZhou I also tried removing the if else statement,
But that gave me this
添加参数upsample_16_W输入信息和tensor数据
添加参数upsample_16_b输入信息和tensor数据
upsample_16 节点构建完成
Failed at layer concat, layer's bottom not detected ...
Traceback (most recent call last):
File "convert2onnx.py", line 34, in
main(args)
File "convert2onnx.py", line 27, in main
c2o = Caffe2Onnx(graph, params, onnx_name)
File "/content/caffe-onnx/src/caffe2onnx.py", line 28, in init
self.__getNodeList(LayerList)
File "/content/caffe-onnx/src/caffe2onnx.py", line 395, in __getNodeList
Concat_node = op.createConcat(Layers[i], nodename, inname, outname, input_shape)
File "/content/caffe-onnx/src/OPs/Concat.py", line 21, in createConcat
output_shape = getConcatOutShape(input_shape)
File "/content/caffe-onnx/src/OPs/Concat.py", line 12, in getConcatOutShape
n,c,w,h = input_shape[0][0],0,input_shape[0][2],input_shape[0][3]
IndexError: list index out of range

workaround, delete these two lines. https://github.com/htshinichi/caffe-onnx/blob/master/src/caffe2onnx.py#L481-L482

@LiyouZhou did, some other issue crept up.


More robust solution to getting shell by piping getent passwd to cut?

https://github.com/OSC/bc_desktop/blob/62376c8b1dff1659c68d27c654c6635541d1b92d/template/script.sh.erb#L10 assumes that the login shell is always the 7th field of the passwd struct. Is there ever a case where this is not true?
If Etc is already required and available to be used in erb rendering in batch connect apps, it would seem this might be preferable:
export SHELL="<%= Etc.getpwuid.shell %>"

That assumes that the passwd struct for the user on the web node is the same as on the compute nodes. Is this a safe assumption? Except if you have the case where there are multiple clusters with separate home directories, and a newer cluster has that zshell you want to use, and the older cluster has bash... then perhaps you do want the batch job to determine this, not the web app.
Another option would be to add a get_preferred_shell helper function to the list of helper functions available for all batch connect apps (see ood_core). We could then keep the implementation as above, but that would provide the ability to override this helper function in the rare case this code was problematic.

The output and order of fields in getent passwd isn't going to change.  See man 5 passwd.  This is the format from man page: name:password:UID:GID:GECOS:directory:shell.  Pulling it from the compute environment is probably the most reliable thing to do. I think the current solution is about as robust as this should get.  If a user isn't available to getent passwd then they are more than likely not available on the system.


Fix contract field access

Closes: #93

For contributor use:

[ ] Targeted PR against main branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the standards mentioned here.
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

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

Project coverage is 86.33%.

Additional details and impacted files
@@                   Coverage Diff                   @@
##           add-gas-setting-txn      #94      +/-   ##
=======================================================
+ Coverage                86.31%   86.33%   +0.01%
=======================================================
  Files                       17       17
  Lines                      921      922       +1
=======================================================
+ Hits                       795      796       +1
  Misses                     126      126

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


Importing esp-now in ESP8266_RTOS_SDK (AEGHB-555)

Hi ,esp now support ESP8266 but I couldn't find any example on the net. Is it possible to import esp now in esp8266_rtos_sdk. If yes how can I do that.
Thanks.

There is an example for espnow in wifi folder of ESP8266_RTOS_SDK.


toml support

I think toml support is useful for this project

Hi @eliastor thanks a lot for your contribution.  We're happy to merge it.
I've noticed that this only covers simple key/value type TOML and not, for example, tables, but we'll add some issues to the project to keep track of that.

Partial fix for: #14

@eliastor issue: #44 captures what remains to be done regarding TOML.
Thanks again!


Empty elements cause crash

For example, <description/> causes a crash.

This isn't so much a bug as a missing feature. We should be able to specify that, for example, a description can be empty. So, I've restyled this as an enhancement.


python3Packages.paver/nose: remove nose and modernize

Description of changes

Addresses #326513. Disables new tests that require distutils.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

What about adding https://github.com/paver/paver/commit/e71a7355bbf7482eb5f785790c73c3c2bb218985 to remove six dependency?

Not sure if it's worth it. The commit version could be buggy for all we know.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 330858 run on x86_64-linux 1

4 packages built:

python311Packages.paver
python311Packages.paver.dist
python312Packages.paver
python312Packages.paver.dist


Disable 2FA for MFTF

These changes are added to take care of the MFTF tests.
When 2FA is disabled, checking for the configuration in the CLI or in the tests will throw an exception.
This happens when the value is set to 0. When 2FA is enabled, running bin/magento config:show twofactorauth/general/enable  will return 1, where as when its disabled it will return an exception which we use as an indicator to know that 2FA is disabled and that we should bypass the filling of the OTP key.
In this case, rewriting the action group and its helper classes is necessary because all helper classes must extend Magento\FunctionalTestingFramework\Helper\Helper

@markshust What do you think about the proposed changes?

Hi @BorisovskiP, thanks so much for the PR. Can you provide a couple scenarios that I can run to test this out?

Hi @markshust , you can easly test this with any MFTF that has the action group AdminLoginActionGroup.
If you have 2FA enabled and configured correctly, then all tests should use the predefined credentials and log in to the admin panel without issue.
If you have it disabled, then the tests will fail because in the action group 'AdminLoginActionGroup' (vendor/magento/module-two-factor-auth/Test/Mftf/ActionGroup/AdminLoginActionGroup.xml) there are two helper classes that fill in the credentials in any case.
This means they do not check whether or not 2FA is enabled or disabled. When you run these tests with 2FA disabled, an error with be thrown and the test will fail. In the changes proposed in the PR, we check if that error is thrown, and if it is we know that 2FA is disabled and just return false.
We tested in Magento 2.4.0

Hi @markshust, maybe some more concrete steps might help you test this better:

We used Magento 2.4 with PHP version 7.4 on ubuntu 20.04

Configure MFTF  (https://devdocs.magento.com/mftf/docs/getting-started.html)

Configure 2FA for MFTF  (https://devdocs.magento.com/mftf/docs/configure-2fa.html)

Follow the installation steps for https://github.com/markshust/magento2-module-disabletwofactorauth, but keep 2FA enabled

Run  AdminDeleteConfigurableProductTest via vendor/bin/mftf run:test AdminDeleteConfigurableProductTest  located under vendor/magento/module-configurable-product/Test/Mftf/Test/AdminDeleteConfigurableProductTest.xml

You should expect the test to be executed successfully if all other previous steps have been correctly configured.

Now you should disable 2FA via bin/magento config:set twofactorauth/general/enable 0

Without applying the changes from the PR, run  AdminDeleteConfigurableProductTest via  vendor/bin/mftf run:test AdminDeleteConfigurableProductTest located under vendor/magento/module-configurable-product/Test/Mftf/Test/AdminDeleteConfigurableProductTest.xml

In this case the test should fail because it cannot find the form to be filled with the login credentials

Apply the changes from the PR, keep 2FA disabled and the again run vendor/bin/mftf run:test AdminDeleteConfigurableProductTest

Now the test should be executed successfully

@BorisovskiP thank you very much! I apologize in my delay of responses, I mute GitHub notifications now and only check it once or twice a week. The testing criteria you just posted helps a ton. I’ll look into this shortly.

@markshust Did you find time to test this?

Not yet, will try to over this weekend. Sorry extremely busy time of year.

@markshust did you have time to test the PR? Could you give me an update on the current status of the PR?

@markshust did you have time to test the PR? Could you give me an update on the current status of the PR?

Hey @markshust, were you able to test the PR?

Hey @markshust, were you able to test the PR?

Hi @BorisovskiP - I apologize for the delay. I haven't had time to test this out, but I don't believe merging any of this code in will cause any issues. The code looks good 👍  I'll trust it's all OK and merge this in.

Hi @BorisovskiP - I apologize for the delay. I haven't had time to test this out, but I don't believe merging any of this code in will cause any issues. The code looks good 👍  I'll trust it's all OK and merge this in.

Released with 1.1.2

Released with 1.1.2


.Net: Rename 'Skills' -> 'Plugins' - Part 8

Motivation and Context
Description

Contribution Checklist

[ ] The code builds clean without any errors or warnings
[ ] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[ ] All unit tests pass, and I have added new tests where possible
[ ] I didn't break anyone :smile:

/sk generate-pr-description


The Apple connection Domain name validation is not applied correctly.

Describe the issue:
The domain name validation is not applied correctly inside the custom connector Apple authenticator and also when trying to Add an authentication within the Sign-in Method tab.

How to reproduce:

Create an application
Navigate to the Sign-in Method tab
Add Authentication -> Configure new Authenticator
Try to select Apple from the list

Expected behavior:
The Apple authenticator should be disabled.

Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [e.g., IS 5.10.0, IS 5.9.0]
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

Apple authenticator should remove from here. As it's duplicated since there is a dedicated template for it.


Update spackage in preparation for release

PR Summary
We have updated some of our dependencies since the last version. This change simply updates the package.py to reflect that the new version will depend on these new packages.
Specifically, the two main updates are

ports-of-call to 1.6.0
Catch2 to version 3

PR Checklist

N/A Adds a test for any bugs fixed. Adds tests for new features.
N/A Format your changes by using the make format command after configuring with cmake.
N/A Document any new features, update documentation for changes made.
[x] Make sure the copyright notice on any files you modified is up to date.
N/A After creating a pull request, note it in the CHANGELOG.md file.
[ ] LANL employees: make sure tests pass both on the github CI and on the Darwin CI

If preparing for a new release, in addition please check the following:

[ ] Update the version in cmake.
[ ] Move the changes in the CHANGELOG.md file under a new header for the new release, and reset the categories.

@rbberger can you review this?
Can you also take a glance at the package.py and make sure I didn't miss anything that needs to be updated for the new version?

@jhp-lanl I think all the plugins logic should be changed from @main to @1.9.0:.

@jhp-lanl I think all the plugins logic should be changed from @main to @1.9.0:.

I believe this should be good now.
@Yurlungur I think this MR is ready


Setting to modify default browser for all links

Would be great to have a setting to modify default browser for VS Code only.
Use case:
I have different browsers for personal use and for dev work.
My personal browser is set as system default in the OS, but I want all links from VS Code (links to PRs, issues, etc) to open with my dev browser.

Merging to https://github.com/microsoft/vscode/issues/96132


[d3d9] Black ground textures in some zones in Everquest

The main game, some areas (newly texture starting zones and so on) have black textures with green lines at the seams for the textures
Software information
Everquest Live from Daybreak Games
System information
-GPU: Nvidia Geforce 1650 Super
-Driver: 455.26.01
-Wine version: wine-5.19-88-gf4a66ad257d (Staging)
-DXVK version: dxvk-mingw-git 1.7.2.r7.g6a10c81d-1
-Kernel version 5.9.0-1-tkg-cfs### Apitrace file(s)
ApiTrace
https://drive.google.com/file/d/1We7EJfDzm3tXxCbCtSxJLFL3CunaudDi/view?usp=sharing
Log files
https://drive.google.com/file/d/1L0oUhRWoHGlOpIoxSb7o-I2AvmkWMH49/view?usp=sharing

It's caused by not supporting R8G8B8, this fixes it for me anyways, no idea if it's right or not:
diff --git a/src/d3d9/d3d9_format.cpp b/src/d3d9/d3d9_format.cpp
index a51faba7..965953b1 100644
--- a/src/d3d9/d3d9_format.cpp
+++ b/src/d3d9/d3d9_format.cpp
@@ -7,7 +7,11 @@ namespace dxvk {
     switch (Format) {
       case D3D9Format::Unknown: return {};

-      case D3D9Format::R8G8B8: return {}; // Unsupported
+      case D3D9Format::R8G8B8: return {
+        VK_FORMAT_R8G8B8_UNORM,
+        VK_FORMAT_R8G8B8_SRGB,
+        VK_IMAGE_ASPECT_COLOR_BIT
+      };

       case D3D9Format::A8R8G8B8: return {
         VK_FORMAT_B8G8R8A8_UNORM,
@@ -554,4 +558,4 @@ namespace dxvk {
         || (supported.optimalTilingFeatures & Features) == Features;
   }

-}
\ No newline at end of file
+}

@Joshua-Ashton
Is there a reason it's marked "unsupported" in the code?

A lot of GPUs don't support that format properly. As far as I remember, even Nvidia and AMD Windows D3D9 drivers don't support it either.

I wonder if it uses it as a scratch buffer or systemmem. I'll take a look later.

I found some problems, first I thank you for your work and second, there are some problems with this game, when I try to fight the game just crashes.

medieval2_d3d9.log

That is not related to this problem, open a new issue, since it's not even the same game, let alone the same issue.


Could you please add a LICENSE?

Could you please add a license to this repo? I want to use the code but have copyright concerns. Thanks!

No problem, what would be the most relevant license for you that ensures maximum freedom? If you don't mind suggest a LICENSE.md file that I can add and I'll add it

Thank you! MIT license is preferred, and I won't use it for commercial purposes, so any one of the GPL licenses will also be fine.


Restart Router streams when gateway token changes

Refs #397
Streams from the router to the monitor will be addressed in a separate PR

Coverage remained the same at 75.217% when pulling 2a2564681150c034f8d463cba42d9c7db94087cd on issue/397-gateway-stream-tokens into 5aa334fc7dcbd91c6552a6ac54a83f5a46098e89 on develop.


Feature Request: Database management and support for multiple projects

Feature Description:
Since BloodHound CE supports multiple users, one would expect support for multiple projects as well. It would be useful to have a button to clear the database like there was before with the original BloodHound as well.
Current Behavior:
Currently, Bloodhound CE imports all data into a single Neo4J database.
Desired Behavior:
BloodHound CE would manage multiple collections of ingestion imports separately or together.
Use Case:
It would be useful to have one BloodHound instance available for multiple testers and teams with separate projects.
Implementation Suggestions:
This probably means BloodHound would manage the connections for multiple Neo4J databases that would separately hold data for multiple projects/clients.
Additional Information:
/

how can i remove the old database i upload , after some searching i found this
 but i don't where is my uploaded .json file in this

We would suggest deploying multiple compose stacks utilizing the existing toolsets and examples, which will also let you segment access to the data for each project.


Add generation of interface tuple default values

Needed for blackBox module instantiation to become an overloaded operation, with the combinational and sequential instantiations yielding default values for the behaviour of an unspecified external block (as currently used in the pinmux).

Self-merging minor change.


Korean Version

Hello, I would like to translate this page in Korean.
I would like to ask if I can translate in Korean.
I created a repository that named webpack-korea.
I connected the repository to Travis and completed it for my local environment.
I'm waiting for your answer.

Awesome! We already have a chinese version maintained by @lcxfs1991 and recently added a dropdown for languages in the main site. Once you get something solid going we could add a link in the languages dropdown:

Also, just so you are aware, we are considering some tools to make internationalization/translation easier, see #295.
@bebraw feel free to elaborate or add anything. From my standpoint, this would be fine as long as it's maintained in a separate repo.

It's alright.

@Lutece can you share a link to your repo? If anyone creates issues over here re korean translation, I'll send them over to your repository. Also, you may want to get some ideas from @lcxfs1991 as their chinese repository is already pretty far along. He can discuss our translation branch a bit, which may be useful to you as well.

this is korean repo link :) - https://github.com/webpack-korea/webpack.js.org
Should i use Locki when i translate webpack.org?
Could you please explain how to proceed with the translation?

Hi Lutece, I am willing to help.
Locki is needed more work to be integrated into the main repo.
You can take a look at branch translate. You can arrange friends to help translate first.  Any questions you can raise issue here or directly contact me via Twitter Message.
https://twitter.com/lcxfs1991

Should I use Locki when I translate webpack.org?

Not yet, as @lcxfs1991 said, the Locki setup probably won't be for a bit. Just wanted you to be aware, and might be worth subscribing to the issue (#295).

Could you please explain how to proceed with the translation?

What you proposed, as in maintaining a separate repo, would be fine for now. Past that, I would discuss with @lcxfs1991. I'll leave this issue open for a few more days in case you'd like to discuss more here. Also, I may add a translation section to the readme with links to each of your repositories.

Closing this, but I will add a translation/internationalization section to the readme with links to both repos. And, once the korean version is significantly built out, we'll add a link to wherever you decide to host it.

@lcxfs1991 @skipjack Thank you for answering me.
I started to translate to Korean, so when I finished translate, I'll inform you.
I'll be back to here soon!

@Lutece  Good luck man. It'd better ask more friends to join you.

@Lutece Hi, Lutece. I want to contribute korean translation project! Could I join the project?

@mojosoeun Sure! I sent you an email for the invitation and i made a slack channel for webpack-korea. if you want to talk to me any time, please join the slack channel

@Lutece OO Thank you :)

I have a few question about Webpack version.
Translation branch is still version 2.3.3 and my repository is watching translation branch. so my version is 2.3.3 too, how can I change version 2.3.3 to version 3.0.0?
Do I need to update to 3.0.0 after I finish 2.3.3 version?

Hmmm so what do you mean by:

Translation branch is still version 2.3.3

Do you mean the version number in the sidebar:

That's dynamic now so if you update your fork it should basically always be on the latest version. One of you could also PR an update to the translation branch if it's behind. @lcxfs1991 @bebraw could probably give you slightly better advice since I'm still not exactly sure what the translation workflow is.
You could also keep your fork a little behind and wait for things to stabilize a bit before updating -- meaning things are still changing quite rapidly (for example all the guides cleanup in #1258) so you might end up having to update the translations a lot.


ability to rollback service to previously used images

# update case: `update_docker_service.sh gcorelu`
# for every container will do:
#   1. Check image in registry, if there's an update, then
#   3. Mark currently used image as "previous"
#   3. Kills container
#     _Assumed_ that Systemd re-creates container using new image
#
# rollback case: `update_docker_service.sh gcorelu rollback`
# for every container will do:
#   1. Check if image with "previous" tag is locally available
#   2. Tag "previous" image as current
#   3. Kills container
#
# In case at least 1 container has been rolled back,
# further service update is disabled.
# one have to manually resolve an issue and remove $LOCKFILE

There's a fix for "update_docker_image.sh" script, coz it used wrong variable to compare image after pulling from a repo, and also didn't use puppet's cmd value for all commands. Now it's fixed
"update_docker_service.sh" script now can be used to rollback the service back to the previously used images

Since we decided to continue development Internally, please remove script creation from the repo to avoid conflicts.
Thanks


Is this project dying itself alone ?

Hello,
From an external point of view this project has had no commits since 3 years and none of the 3 proposed pull requests has been discussed (one is nearly one year old). So, sadly, it looks like it is dying.
I am wondering if

arcnmx the owner of this project is open to enable one or two people to give any help to manage it
there is anyone that is willing to help the project

This is a call to move forward on this project so as not to let it die.

Since what happen with XZ Utils lately I think this issue might be wrongly understood (moreover this issue does not help in anyway) so I prefer to close it.


[Backport to v1.0] Upgrade PostgreSQL to v13

Is your feature request related to a problem? Please describe.
We need to back port task #1861 to v1.0 branch.
In this task we should keep all known PRs related to PostgreSQL upgrade until the feature is back ported.
Known related PRs

#2500
#2501
#2519
#2522
#2539
#2545
#2546
#2566
#2598

DoD checklist

[ ] Changelog updated
[ ] COMPONENTS.md updated / doesn't need to be updated
[ ] Schema updated / doesn't need to be updated
[ ] Feature has automated tests
[ ] Automated tests passed (QA pipelines)

[ ] apply
[ ] upgrade

[ ] Idempotency tested
[ ] Documentation added / updated / doesn't need to be updated
[ ] All conversations in PR resolved
[ ] Solution meets requirements and is done according to design doc

This is new functionality so we are not back porting it.


[DM-29923] Initial version

These are great comments. Thank you!
I added a new section explicitly addressing unauthenticated routes and optionally authenticated routes. We may need to add some new features to Gafaelfawr to support them, but it should be possible to do. A possible implementation approach is now spelled out in the docs.
Good point that cross-origin requests may not be as rare as I was hoping. I rewrote the section on Notebook to Portal cross-origin requests to be more generic.


Update Spec for block-independent PRTCL changes

The formal spec needs to be update with the changes made in #1450

closed by #1457


[Feature]: More precise slider movements for touch input

Requirements

[X] I have updated Mushroom to the latest available version
[X] I did a search to see if there is a similar issue or if a pull request is open.

Is your feature request related to a problem?
It is hard to do fine-grained changes to sliders by touch on a small device like a smartphone.
Hitting exactly 1% (instead of 0%) or other small values requires many tries.
While during day-time a difference of few percentages points aren't noticeable, at night in a low-light setting, the difference between 1% or 4% brightness of a main light source makes a strong perceivable difference.
Even more so, choosing a color from the color band is difficult to do precisely.
Describe the solution you'd like
Instead of having to hit the absolute value by finger (like the left-most pixel of the slider for 0%), the slider could be moved relatively at lower speed.
Example: the slider is at 10%. The finger touches down anywhere on the right half of the slider and moves to towards the left. The value is then decreased by half the way the finger travelled.
This would allow for precise placement of the slider. Similar to how scrubbing works in many smartphone apps like the audible player.
Describe alternatives you've considered
Instead of the slide speed being fixed at eg. 50% it could be adjusted by dragging the finger downwards like many other apps do.
Additional context
I hope the description of the problem and the solution are understandable.
I already implemented this solution and was about to create a PR when I saw it requires creating an issue first.
It can be found here (https://github.com/piitaya/lovelace-mushroom/pull/935)
Notice: The changes only affect the panning (pointer down, move, pointer up). Clicking/Tapping an absolute value still works like before.

I don't think we should add this type of control. If you move your finger from 30px, the slider should move by 30px.
The proper way to have more precise control is to use a full screen slider like this one :

I don't think we should add this type of control. If you move your finger from 30px, the slider should move by 30px.
The proper way to have more precise control is to use a full screen slider like this one :
When is release date for new HA cards and other dashboards things?

hey @piitaya, thank you for your answer and for looking into it so quickly.
I see that you're heading for a solution like implemented by Apple's HomeKit, so maybe let's discuss my approach in two parts:

Relative Movement:
I think it's easy to see that even when moving the slider at the same speed as the finger, there is much better control of it, if it's possible to drag and move the slider from any position instead of moving the finger at the exact position needed to set a certain value. Apple even implemented it in the same way for the vertical slider you reference above.

Reduced Slide Speed vs Overlay:
"If you move your finger from 30px, the slider should move by 30px. The proper way to have more precise control is to use a full screen slider like this one"
I would disagree with that statement in general. Both are different approaches for solving the issue. They are being used by other apps and have their own drawbacks and advantages.
Personally, I always liked the mushroom cards because of their convenience and efficiency: The values are editable in place, on the dashboard without the need for additional overlays and more clicks. It's so quick to open the app, slide down the brightness of a light and close it again. (Unlike with Apple's HomeKit)
I've been using my approach of 50% pan speed for a few days and have to say it made things a lot easier on phone and tablet and feels like a natural improvement to the mushroom cards without taking anything away.

Yes, some apps like Apple Home use relative instead of absolute. It's a bit disconcerting when you use to use absolute before. All Home Assistant slider are absolute for now. That's why I preferred to keep the same usage.
I like to use quick shortcuts to edit brightness or something else too. But IMO, it should be quick actions, not precise brightness adjustment. A fullscreen experience is preferable for that.

I agree that 1% brightness is hard to reach. There are already issues for this. We can set the minimal to 1% instead of 0% to fix that issue so it's easy to set the minimal value for a light.

My lamp is very noticeably dimmer at 1% than any other setting. 2% and 4, even 5% - not much difference. 1% and 2% - like night and day(pun not intended).
I usually like my light at 1% so a simple toggle in the card settings like "Turn off with brightness slider?" that's setting the min slider value to either 0% and 1% would be great

The option to set a min and max range for sliders would be great

As someone who frequently adjusts sliders on a small device, I know how frustrating it can be to hit the exact value I want. This new solution of being able to move the slider relatively with more precision will be a game-changer.
Not only will it make adjusting sliders easier, but it will also allow for more control and accuracy when making important adjustments, like setting the brightness or color in a low-light setting. The ability to adjust the speed by dragging the finger downwards is also a great touch and makes the feature even more customizable. I would love to try this feature out for myself and see the improved user experience firsthand.
However, I also understand that the vision for the Mushroom extension is important to maintain consistency in its user experience. That's why I suggest making the new feature of relative slider control optional in the settings. This way, users who prefer the traditional way of adjusting sliders can continue to do so, while those who would benefit from the added precision can easily turn the feature on.
By making this feature optional, you can still maintain the original vision for the extension while providing additional value to users who need it. I believe this is a win-win solution that will allow the original developer to stay true to their vision while still incorporating useful and requested features.

I decided to fork the project and bring it to HACS as "Mushroom Better Sliders"

I’m using the fork now (which is bound to get out of sync.)
Wouldn’t it be possible to make this control method configurable?
in this way, we avoid splitting off this package.
@piitaya, @phischdev


Rive animation view absorbs all touch events

Rive Android should allow touch events to continue propagating to parent views after Rive has processed it.
This can be an optional flag.

Related https://github.com/rive-app/rive-android/issues/276


Added option to enable nana config

I've added option to enable nana config. That was the only way to actually use config.hpp in order to benefit from nana precompiled extrlib AND keep building the whole thing via cmake.
Additional points:

Usage of libpng from OS almost always implies usage of the release build of libpng.  Which can lead to unexpected crashes as this one
https://stackoverflow.com/questions/22774265/libpng-crashes-on-png-read-info
I have experienced this one myself.

I've added extrlib folders as cmake projects which allows me to

move them outside of the nana folder and use without any further changes in neither nana's cmake nor config and even the resulting project
use the debug version of libpng in the debug build of my project

I'll put my code sample here. It can be placed in the wiki as example for self-implementing or actually included in the nana/extrlib packages.
cmake_minimum_required(VERSION 3.5)

project(png)

add_library(png INTERFACE)

get_filename_component(debug_lib libpng.MDd.x64.lib ABSOLUTE)
get_filename_component(release_lib libpng.MD.x64.lib ABSOLUTE)
target_link_libraries(png INTERFACE
    debug ${debug_lib} zlib
    optimized ${release_lib} zlib)
target_include_directories(png INTERFACE .)
target_link_options(png INTERFACE "/ignore:4099")

I want to discuss the necessity of the difference between
enable_png.cmake  with target_link_libraries(nana PUBLIC png)
and enable_jpeg.cmake with target_compile_definitions(nana PUBLIC -ljpeg)
and possibly change latter to target_link_libraries(nana PUBLIC jpeg)

Thanks for your PR. It has been checked into the hotfix-1.7.3 branch.


Add tone() and note() functions

I think a tone() function would be nice to have in the next update (4.2.0)
Initial thoughts on how this could work ...

use the existing pin management features of this library
can auto-attach or attach to specified channel
this function to be non-blocking
the frequency and resolution will affect a secondary channel. Therefore, ensure that this secondary channel is still useable for LED fade control (variable duty).
Add an ESP32 example showing use tone, servo and led fade control.

Tone function added to version 4.2.0


added myself as contributor 0035

[ ] 🐙 Follow the Pull Request Workflow
[ ] 📝 Update the CHANGELOG.md
[ ] 📙 Add #'s to term tracker item

Reviewer

[ ] 🐙 Follow the Reviewer Guide
[ ] 🐙 Provided feedback and show sufficient appreciation for the work done

Since you add yourself as 0036, I changed the title of the PR.
If you add the line "closes #1594" in the PR description, your issue #1594 will be closed automatically when this PR is merged. That's a nice feature that we use to keep the issue list up to date.


Add instructions how to load translations from command line

Hi,
would like to see how to build a script (with the appropriate command line) that loads a translation file (the JSON) and deploys it to the model... I guess that's something a lot of people might use for deploying translations until there's a functionality for that in the Power BI UI :-)
Thanks...

This is an interesting scenario that I hadn't thought about. Unfortunately it's not documented and there's no intellisense in the script editor to help you, but it can be done using a C# script:
// Save all translations to JSON file:
SaveFile(@"c:\Test\translations-all.json", Scripter.ScriptTranslations(Model, Model.Cultures));

// Save specific translation to JSON file:
SaveFile(@"c:\Test\translations-DK.json", Scripter.ScriptTranslations(Model, Model.Cultures.Where(c => c.Name == "da-DK")));

// Import translations:
var json = ReadFile(@"c:\Test\translations-DK.json");
TabularCultureHelper.ImportTranslations(json, Model, overwriteExisting: true, haltOnError: false);

To execute this from the command line you have to save this script to a file, which you can then execute from the command line with the -S switch. Now unfortunately, the script hard codes the names of the JSON files, so one option is to generate the script on the fly on the command line:
echo TabularCultureHelper.ImportTranslations(ReadFile(@"c:\Test\Translations-DK.json"), Model, true, false); >> ImportTranslations.cs
start /wait TabularEditor "localhost" "AdventureWorks" -S ImportTranslations.cs -D "localhost" "AdventureWorks" -O

The first line generates the script that imports a translation from the specified file. The second line calls Tabular Editor, loads an existing model from a server/database name, applies the scripts and performs a deployment.
If you're executing this from a DevOps pipeline you can substitute the name of the json file as well as the "localhost" and "AdventureWorks" strings with variables.
Hope this helps!

Sounds great! I'll give it a try, soon!. One further question might be how to create an initial translation file... As far as I understand in your tool you can only export an existing translation, modify it and upload it again. I didn't find any way how to create a new one besides modifying the JSON manually... Did I miss something? This is a special situation for Power BI users, in Visual Studio it's just easy...

Sure - that's easy to script as well:
// Add a danish translation:
Model.AddTranslation("da-DK");

// Save to file:
SaveFile(@"c:\Test\translations-DK.json", Scripter.ScriptTranslations(Model, Model.Cultures.Where(c => c.Name == "da-DK")));

Again, if you need to run this from the command line and would like to avoid hard-coding the language (da-DK), you have to generate the script on the command line. See here for instructions on how to output multiple lines of text to a file from the command line.

You're great! Very much appreciated! Will try it later!

I already asked Christian Wade about his point of view about that but probably you have your own opinion about that...
Using Tabular Editor I can now easily add a translation to a Power BI model. If I would like to deploy this model to different workspaces I can now use the typical methods to do so and add translations after that via Tabular Editor. So far so good.
What would make things even easier would be if the model itself can save the translations. So I thought about is loading the model into PBI desktop, then adding the translation via Tabular Editor and save the file to deploy it later. I saw the warnings about TOM and so I was hesitant to proceed. What do you think?

For now, I wouldn't do that. You may encounter weird errors and crashes when you add translations to a model loaded in Desktop, which is the reason we're showing this warning. Later this year, however, Desktop will fully support translations that were added externally, at which point you can safely use the approach you described.

Hello TePeBoxer,
We probably have the same issues than you.
I would like to know how you will manage the translations. Will you modify the JSON file directly ? Will you do it by yourself or will you send it to different translators ? Do you have any plans for the translations maintenance after creation of new tables/columns/measures/display folders/ ... and after their modifications ?
Didier

Didier,
very relevant questions and we're just starting to think about that...
I guess we're just early in the process, tools are not fully aligned here (i.e. PBI Desktop not supporting translations, PBI Service not offering Object Level Security, VS not supporting PBI roles, ...), as soon as that happens (I hope in the timeframe of XML/A R/W GA) at least some of your questions would be solved. I think a good practice is to have just a single language per JSON so you can send it to different developers.
As soon as PBI Desktop will support translations that will be easier (no import/export of JSONs anymore) but also more complicated (only a single user can work with it). So still Visual Studio might play a role (or out much beloved Tabular Editor).
Translation management needs to be an integrated part of the model development, as it was before. Well, an organizational challenge... I'm not aware of a tool which makes you aware of not-fitting or missing translations...
Thomas

Thomas,
Yes we are early but we waited for such a long time for translations. If we want to increase the adoption of our reports, we reach a population who doesn't understand correctly english. Some confusions may happen so for me it's an important topic and I don't want to wait more. I didn't hear about any possibility to translate models or visuals within the desktop in the near future. I think Power BI team relies on external tools to progress on this topic.
To be precise, Power BI desktop already uses translations we input in tabular model through Tabular editor. It even works without the new preview option about XMLA endpoint. But, for the moment, it is not recommended at all to deploy it in production. We just need to wait a few months so I try to get prepared making a lot of tests.
Translations management is an important process. I have a quite a precise view how I will manage it. It will involve IT teams and business teams who will not accept to use a JSON file. I suggest you have a look at this short video. Your comments/critics/ideas will be very welcome.
https://thebipower.fr/index.php/2020/04/12/automatic-translation-of-tabular-model/
I'm available for further discussions around translations management but I don't know if it is the right place for that.
Daniel's opinion would be precious also.
Didier

Didier,
cool stuff... Question would be if it's possible to integrate this into a DevOps pipeline as well, so can we use command lines to apply custom scripts? But nice stuff you presented! Anyway modifying the internal .pbix file wouldn't be something I would do right now due to what Christian Wade gave me as feedback... However it might be a great option in the future...
Thomas

@didierterrien and @TePeBoxer - feel free to use this thread for discussion.

I'm not aware of a tool which makes you aware of not-fitting or missing translations...

You can use Tabular Editor's Best Practice Analyzer to detect if any objects are missing translations! These rules are already part of the standard rules collection found here.
To load these rules into your model, go to Tools > Manage BPA Rules. Here is the direct URL to the standard rules JSON file on GitHub:
https://raw.githubusercontent.com/TabularEditor/BestPracticeRules/master/BPARules-standard.json

To link this file directly in your model, click "Add..." and then "Include Rule File from URL". Then paste the URL above into the textbox and click OK. Alternatively, you can download the file and merge it into your %localappdata%\TabularEditor\BPARules.json%-file, to ensure that the rules are always applied to any model you load on your machine. More information here.
Once the rules are imported, you should see them appear in the UI:

When you click OK, Tabular Editor will show the count of objects that are in violation of best practice rules at the bottom of the screen. Clicking the link will bring up the Best Practice Analyzer that shows you the list of objects in violation. You can double-click an object to navigate directly to it.

Note, there are many other rules in the JSON file provided above. If you don't need the Best Practice Analyzer for anything other than ensuring translations are applied, feel free to disable these other rules as you see fit.

if it's possible to integrate this into a DevOps pipeline as well, so can we use command lines to apply custom scripts?

Well, absolutely! Use the -S option for executing a script through the command line (provide the path of a text file containing the script after the -S switch).

Sorry, being far from a DevOps expert... Is my understanding correct that when I would use something like Azure DevOps we would run into an issue because there's no way to install Tabular Editor on a build server, so I can't deploy my languages? The only solution I would see then is to import the languages, then create some XMLA out of that using Tabular Editor and then deploy it from Azure DevOps?

It's easy to use Tabular Editor in an Azure DevOps pipeline. Simply include a PowerShell step that downloads it before the step where you actually want to use Tabular Editor. This technique is described here (if you're using a self-hosted build agent, you can just pre-install Tabular Editor on that agent). Not sure how this would work with CI/CD offerings other than Azure DevOps, but I guess most of them have similar options for executing a script that would download and install software.

Closing this thread to clean up the backlog of issues, but feel free to open new issues if you have any further questions!


Add option to set restore max delay

Hi,
I've been having problems to automatically restore my previous sessions automatically. I've finally had some time to look into why this has been happening and in my specific case (.tmux.conf) the culprit is a slow loading plugin (tmux-remote-sessions), but this can also happen with a lot of enabled plugins or on a really slow machine.
This may be the reason for other people having the same experience: #19 & #90
This PR makes the 10 seconds introduced in 4e2279688d8a577ed14f42c4e8af1acb58100a57 configurable with the option @continuum-restore-max-delay.

Maybe add it to this doc https://github.com/tmux-plugins/tmux-continuum/blob/master/docs/faq.md ?


Updated broken links and Contributing section

Description
Updated broken links.
Updated Contributing section to reflect latest README.md

Thank you for your pull request and welcome to our community! We require contributors to sign a Contributor License Agreement and we don't seem to have CLAs on file for these contributors to the Pull Request: (@godds). In order for your PR to be reviewed and merged, please follow the directions at the link above.
Project team: please do not merge this Pull Request until Foundation staff have confirmed that a CLA is in place for the new contributor(s) listed above.
If you still have any questions or just don't know what to do next, please email<EMAIL_ADDRESS>/CC @finos-admin

@maoo can you please add @godds to the list of users who are covered by the Scott Logic CLA? Thanks

@finos-cla-bot[bot] check

The cla-bot has been summoned, and re-checked this pull request!


fix: errors occuring during application boot fail silently

This LGTM, just curious as to reasoning behind removing some return types in d05c3777403b136fcfe9b68b3d45bf603e5dc57a? Otherwise :+1:

@javamonn Awesome! I just removed the return types that could be inferred.


WEBPACK_EXTERNAL_MODULE is not defined with dynamic amd require

We trying to migrate from old requirejs build pipeline to webpack. In the legacy code, we imported the i18n related data with something like
import i18n from '@webui/i18n-amd-loader!@webui-legacy/upgraded-url';

Requirejs can recognize it and use the i18n-amd-loader to get the resource of @webui-legacy/upgraded-url.
When moving to webpack, in case this requirejs-specific code can't be understood, I put it into externals:
function(context, request, callback) {
    if (/@webui\/i18n-amd-loader!/.test(request)){
        return callback(null, 'amd ' + request);
    }
    callback();
}

We expect webpack to just skip this line, not do anything, just wait for require to run it and get the resource.
Build is fine, and the file is compiled into amd mode. However, when running, it shows error before it gets to the code:
ReferenceError: __WEBPACK_EXTERNAL_MODULE__900y__ is not defined

WEBPACK_EXTERNAL_MODULE__900y is the variable representing @webui/i18n-amd-loader!@webui-legacy/upgraded-url.
It seems webpack tried to find the resource of '@webui/i18n-amd-loader!@webui-legacy/upgraded-url' but failed.
Is there work around to fix this issue? I tried ignorePlugin but it doesn't work.

Make sure to make your output file a AMD library with output.libraryTarget: "amd". This way it's wrapped in define which is needed for AMD to work.

@sokra Thanks for the quick response. The output.libraryTarget is amd. The file's name is campaign.js. If I build it separately, I can see @webui/i18n-amd-loader!@webui-legacy/upgraded-url in the dependency list of the define. However, when I built the entire large app, the campaign.js get chunked and then this error shows.

If I build the entire app with libraryTarget: 'amd', it will fail without any error.

I think I understand the reason of this issue. The entire app is not built into amd mode.


Release build configuration

I dont have the VSCode Release build configuration. Where can I get it??

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

ID: 40d7fd04-e358-9a83-aef2-bfcba1169699
Version Independent ID: f5df932b-b092-2832-9c9a-61a9c43dcdb3
Content: Debug a .NET console application using Visual Studio Code - .NET
Content Source: docs/core/tutorials/debugging-with-visual-studio-code.md
Product: dotnet-fundamentals
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

Release build configuration is available by default in your .NET console app -- do you get an error when you run the command shown in the tutorial -- dotnet run --configuration Release ?


'Issue template' support issue

Some repos use lowercase filename 'issue_template.md', you can check this repo https://github.com/shadowsocks/shadowsocks-android. It seems ForkHub only load the uppercase ones.
Here is another issue. Some repos have issue template at other place, MS VS Code, for example, have it in root folder. Meanwhile the filename is lowercase, so I am not sure if it is the place or the lower/upper case that causes the issue.

#158?

@TheMolkaPL For some reasons, some repo's issue templat is not loaded on my phone, made me thought there was no support for that. Forgive me for forgetting to search the existing issues first.
I have figured out the cause and revised the scription of this issue. I move the original post here[F12].

What when it's 1Issue_Template.mdfor example? The filenames should not be hardcoded would be checked by theequalsIgnoreCase` method.

I have created a repo for testing. You can check https://github.com/ayopmail/test.
Here is what I found. The website will load the 'first' issue_template. The ForkHub app only load the ISSUE_TEMPLATE.
Considering that no one will use multiple template files, ignoring case maybe the easiest way.

Thanks for all your testing, I'll take a look at it today.


How to set the command help

bin/console my:command --help
bin/console make:crud --help

Symfony's make:crud shows some help lines at the bottom, how is that set in an invokable command?    I tried adding this to __invoke, but it didn't seem to work (although again, it might be related to lazy-loading).
        $this
            // the command help shown when running the command with the "--help" option
            ->setHelp('This is the help, not the description, and can be multiple lines.');

Ah, I had put it in the wrong spot, it needs to be set in configure()
    public function configure(): void
    {
        $this->setHelp("This is the help for my:command");
    }

What do you think about a class attribute for help, so as to avoid calling this,
#[AsCommand('my:command', 'From the extra-console documentation')]
#[CommandHelp(help: 'This is the help for my:command']
final class MyCommand extends InvokableServiceCommand

Of course, looking at this I have to wonder if this shouldn't just be added to the AsCommand attribute.
With an attribute in this class, what about having the option of passing in a filename?  Symfony puts the help in separate files:

Yeah, that could indeed be useful!

What do you think about proposing for AsCommand itself in Symfony?
#[AsCommand('my:command', help: __DIR__.'/..')]


Renamed destination to 'topic' in C++ code

Motivation
Similarly to #1280 renamed usages of "destination" in C++ code.

LGTM 👍


updated for godot 4.1 added a C# version of the GDScripts

updated for godot v4.1
added a CSharp version of the GDScripts
added paragraph for using the C# version in README

For using the C# version in the example project, you need to set the Main Scene to res://CSharp/World_CSharp.tscn

@JCAguilera Little reminder

Thank you so much for the PR (and the reminder!) I'll check it on my free time 😁👍


Using useNavigate() threw an error

When call the useNavigate function, useRouter() cannot find a router context.
Uncaught Error: Make sure your app is wrapped in a <Router /> arises, but the app is certainly wrapped in a Router component.
Here is some of the code:
// index.tsx
import { render } from 'solid-js/web'
import { Router, useRoutes } from 'solid-app-router'
import { Component, lazy } from 'solid-js'

const routes = [
 {
   path: '/',
   component: lazy(() => import('./pages/Home')),
   children: [
     {
       path: '/',
       component: lazy(() => import('./pages/scripts/ScriptList')),
     },
     {
       path: '/script/add',
       component: lazy(() => import('./pages/scripts/ScriptAdd')),
     },
   ],
 },
]

const App: Component = () => {
 const Routes = useRoutes(routes)
 return <Routes />
}

render(
 () => (
   <Router>
     <App />
   </Router>
 ),
 document.getElementById('root')
)

// other.tsx
import { useNavigate } from 'solid-app-router'

const fn = () => {
 const navigate = useNavigate()
 navigate('/script/add')
}

Is anything wrong in this?

Context only works under component render/primitive(hook) execution. If you call use from like an event handler context it won't be there. I suspect that is what is happening here.

@ryansolid Thank you so much. It works well after taking the function out of the event handler scope.

I am having this problem and don't understand the solution. In my component there is a button, and when you click on it, I need to navigate to a specific page after taking various actions. Those actions happen inside the click handler callback of the button. How can I move the navigate call outside of the click handler?

Try grabbing a reference to navigate at the top of your function, then use it in the event handler...
function Counter() {
  const navigate = useNavigate();

  return (
    <button
      onClick={() => {
        // do other stuff
        navigate("somewhere");
      }}
    >
      Click Me
    </button>
  );
}


text inside TextBox is moving right/left when enter/delete it

I think issue could be in layout rounding optiona enabled.

@QuantumDeveloper can you more specify what is going wrong? i don't really know what do you mean?

I mean that when you enter text inside textbox, text moves to one/two pixels from rigth to left and back. And it seems that it happens because in template enabled layout rounding option.

@QuantumDeveloper i don't see this in latest source

Please, try this inside datagrid column

@QuantumDeveloper works fine in the demo, so create a short sample and/or make a creencast...


Extract tile encoding to encode_tile()

Isolate tile-specific code from frame-wise code.

Coverage increased (+0.005%) to 82.996% when pulling 83927f91a5cb0ca397af88eaf14ce019d1d109fd on rom1v:encode_tile into 265889fae2d3250b53e824ee868863f7455823a3 on xiph:master.


[OpenSearch Playground] Add Alert for the playground

Is your feature request related to a problem?
We didn't have alerts for playground site, we need to leverage the OpenSearch Dashboard's existing feature of alert or Anomaly detection to trigger alert when playground site down.
What solution would you like?
OpenSearch Dashboard Alert or Anomaly detection plugin.
What alternatives have you considered?
No
Do you have any additional context?
No

Resolved by #68


check_external_links doesn't work with email addresses

When using syntax:
Email<EMAIL_ADDRESS>
or
Email: [foo<EMAIL_ADDRESS>
Gutenburg spits out this error:
Failed to build the site
Error: Failed to render content of path/to.md
Reason: Link<EMAIL_ADDRESS>is not valid: invalid URI for Client Request

Didn't think of email addresses!
Should be an easy fix if someone wants to gets started with Gutenberg or I'll fix it later otherwise


VoodooSDHC possible replacement

Hi, what's your exact SD card reader PCI address? You can retrieve it from any PCI utility such as Hackintool, in PCIe section.

Hi. It's a Ricoh SD Card reader.
PciRoot(0x0)/Pci(0x1C,0x0)/Pci(0x0,0x0)
I think the kext is based on this one:
https://www.reddit.com/r/thinkpad/comments/5wbv0s/i_fixed_the_unusable_thinkpad_x230s_ricoh_sd_card/

Oh alright, since it's a PCI reader, can you tell me vendor-id and device-id?

vendor-id: 0x1180
device-id: 0xE823
IOService:/AppleACPIPlatformExpert/PCI0@0/AppleACPIPCI/EXP1@1C/IOPP/SLOT@0

Ouch, seems like VoodooSDHC is the only known way to get the SD Card reader working. Closing this issue...


firefox-bin DRM module failure

Issue description
Trying to play music from https://music.amazon.de/ through firefox-bin with DRM module enabled in config. No music is playing.
On stderr the following message can be observed around the time that the website os opened:
(/nix/store/964rifgrzk5p6xbn9z0mg92qfjsimfrq-firefox-release-bin-unwrapped-63.0/usr/lib/firefox-bin-63.0/firefox:4011): dconf-WARNING **: 16:23:05.284: Unable to open /run/opengl-driver/share/dconf/profile/user: Permission denied
Steps to reproduce

Open ${pkgs.firefox-bin}/bin/firefox in Terminal
Enable DRM in the settings
Open website https://music.amazon.de/
Login with a valid user that has music available
Try to play a song and listen to silence.

Technical details

system: "x86_64-linux"
host os: Linux 4.18.17, NixOS, 18.09.1262.f99bab3e612 (Jellyfish)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.1.3
channels(root): "nixos-18.09.1262.f99bab3e612"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Seems to be no longer an issue.


Fix #292 -- Change admin widgets' JS media ordering

Select2Mixin now loads admin/js/jquery.init.js, just like the AutocompleteMixin of Django Admin. Fixes #292.

@mardukbp I made an amendment to your patch and moved the behavior to the admin mixin only. Can you double-check if that works for you?

@codingjoe It does not work. Why did not you wait for my reply? Why did you modify my PR, when I had already tested that it solves the problem? Please use my original code.

Hi there,
Don't take this wrong, but I maintain dozens of packages. I don't remember why, I did something 3 weeks ago. Upon reviewing your code again, it wasn't something I would have merged. You modified the base widget that shouldn't implement anything specific to Django's admin. Which you may have noticed because all tests related to that were failing. Thus, your code
Please check if you are using the admin mixin, that is required to make select2 work in Django's admin. If the error persists, please open a new ticket.
Kindly,
Joe

Thank you for your prompt and careful reply.  I appreciate that you take the time to respond quickly. I agree that my modification was not done at the right place and the right place is the AdminMixin.
I think that as a maintainer you should merge PRs only when there's convincing evidence that they solve a problem and not rush to get rid of open PRs. Now we have to start all over again with another issue and another PR, which costs both of us time. In other words, I appreciate your dilligence, but don't be too efficient :)


With Karaf 3.0 the db-examplejpa shows error of EntityManager

I've tried these samples with Karaf 2.3.0 , Karaf 2.3.3, Karaf 3.0 and Fuse 6.1 ( with Fabric ) and they only work ( for me ) in Karaf 2.3.3
In the other versions it always give an error when trying to start the bundle net.lr.tutorial.karaf.db:db-examplejpa:bundle:1.0-SNAPSHOT :
org.apache.aries.blueprint.core - 1.3.0 | Bundle db-examplejpa is waiting for
dependencies [(&(&(org.apache.aries.jpa.proxy.factory=true)(osgi.unit.name=person))
(objectClass=javax.persistence.EntityManagerFactory))]

cschneider, I am getting this same error using Hibernate 4.2.15.Final and Karaf 4.0.4.
Do you have any suggestion?

karaf@root()> info
Karaf
Karaf version               4.0.5
Still getting same issue. Any suggestions?


Per item caching

Add support to cache item size in addition to the existing row / column caching.
See #562 for details
The PR is not ready:

Expose the interface to clearItemHeights, clearItemHeight, clearItemWidth, clearItemWidths.
Replace item by cell, I found it confusing since you called the Row/Column cache, cellCache, when actually it doesn't cache anything about the cell. So I used item in the meantime but it is probably a poor choice.
Write tests

Nice work @Mathieuu! :tada::beer:

Hey @Mathieuu. Thanks again for sharing this proposal!
I'm happy to share that per-item-size-caching will be enabled in the upcoming, fiber-friendly version 9 release (currently tracked as PR #577). Unfortunately CellMeasurer has dramatically changed in this release and so this PR is not really relevant anymore. I'm going to close it for that reason, but, the general idea will make its way into the next major. I hope to release an RC for that tonight or tomorrow, and a major release sometime later this week.
Cheers!


Fix reparent() losing owner

Fixes #81480, but I am open to suggestions on any changes that need to be made - I just took the most obvious path suggested in the issue for a fix.
Thanks!

I think this makes sense, but let me bring a couple of points up:

Setting the owner requires additional steps, like checking the new owner is valid (an ancestor). Let me suggest moving the code of Node::set_owner() to a private function _set_owner() that also takes a bool p_show_warning. The Node::set_owner() would just call through it with true. reparent() would instead call it with false to try to set the original owner, but not printing an error in case that one is no longer valid. Additionally, in cases like this we likely don't want owner_changed_notify() to be called (see _set_owner_nocheck()) when the original owner is successfully restores. Maybe this warrants a more comprehensive rewrite so all this is taken into account at a more elemental level instead of being that "patchy."
Documentation would need to be updated (haven't checked, but it likely says that reparent() is exactly like removing plus re-adding somewhere else; the owner-preserving behavior should be mentioned).

this doesn't feel much less patchy, but maybe we can include a bool p_pending_reparent in the Node class that will skip clearing owner on Node::_propagate_after_exit_tree() if true? that way there doesn't have to be a private Node::_set_owner() implementation. just set it to true before reparenting and then false immediately after

this doesn't feel much less patchy, but maybe we can include a bool p_pending_reparent in the Node class that will skip clearing owner on Node::_propagate_after_exit_tree() if true? that way there doesn't have to be a private Node::_set_owner() implementation. just set it to true before reparenting and then false immediately after

That would be very hacky and prone to issues, like leaving pointers to the old owner when the nodes are no longer in a relationship. What Pedro suggests is much more robust and in line with how we often do stuff. And also much simpler.

i guess the issue i'm seeing (and realizing my original solution is definitely inadequate for) is that children of the reparented node will lose their owner, which would mean that if the unique node is a child of the reparented node, the reference is lost. i can use the private _set_owner() call to set the owner of the reparented node, but any children will find their owner lost.
i also may have misunderstood the original intention here! i was under the impression that you would want to keep the owner during reparent, even if the new parent doesn't share a branch with the original parent.

i was under the impression that you would want to keep the owner during reparent, even if the new parent doesn't share a branch with the original parent.

Owner must be an ancestor of the node. If after reparenting this is no longer the case, then losing the owner is the correct behavior.

my mistake! so i think the assessment of this is the nodes being reparented need to be checked for who their owner is and if they're being reparented to a descendant of that owner. then the owner can be reset on those nodes (and not any nodes that did not originally have that owner).
i apologize, i'm sure it's obvious that i'm also familiarizing myself with the idiomatic way of doing things as i go and want to make sure this is done correctly, but it definitely ended up being a bit more involved than i anticipated :sweat_smile:

No need to apologize! You're trying, learning and addressing feedback. You can't be asked for more.

btw turns out the label "good first issue" was wrongly assigned on this one. It's more complex than I anticipated, sorry 🙃

all good! i appreciate the patience everybody. once i'm back in town middle of next week i'll sit down and get this worked out.

here's a first pass at a way to resolve all of the needs of this fix. it's, admittedly, an extremely naive take - my c++ experience is about ten years out of use at this point and was already rusty as i was using it, haha. i'm welcome to any recommendations as a result, and if this needs to be a more fundamental change, then so be it! as it stands i'm a little uncertain how this might be taken as it involves three iterations in order to solve the original issue, which is a bit undesired, i feel like.

... once i fix all of the style issues, of course.

I will likely add new comments in the future (as soon as I can reason carefully about this). That said, your C++ is fine. (And if it's "rusty," well, it's trendy. 😃)

hi! i just wanted to see if there was any chance to check this out a little more. if not, no problem, but i would love to be able to contribute a fix if possible.

This is still under my radar, but I have been too busy. I will come back very soon!

no problem at all! i appreciate the update, i also know it's probably gotten quite a bit busier with all of the new people recently :)

I think the general implementation is ok, there are a few details to iron out.

squashed to one commit. thanks again for the assistance and patience!

Last git nitpick, could you tidy the commit's description too?

See https://github.com/godotengine/godot/blob/master/CONTRIBUTING.md#format-your-commit-messages-with-readability-in-mind for general advice on commit messages for Godot.

not a nitpick at all! i, uh, didn't realize i had been taking everybody else on the same journey i had been going on while updating this PR

Sorry for stepping in when everything looked settled. Just a few minor things I think can be improved.

no problem! i'm definitely also learning as i go with this experience so it's useful for me as well

For the record, we discussed this together with other PR with Yuri and Clay from the production team and decided that this wasn't critical for 4.2 (which is nearing release candidate), and might need some testing to ensure everyone agrees with the behavior change, hence the milestone bump to 4.3.

sounds good - i think that's completely reasonable given this is a fairly common action

hi! just wanted to check in and see if this is still a good candidate for 4.3, and if there's anything else required of me to assess/change

The review comments you marked as resolved but didn't apply maybe?

hmm, i'm not really sure... i applied all of the requested changes as far as i know, so i'm not certain why it's saying that there are more changes. maybe i missed one? reviewing the four requested changes i do see, i did enact them all before listing them as resolved (at the very least, the current version of the code reflects those changes)

Unresolved the ones you didn't apply 🙂

ahh okay - the ones from KoBeWi were previously applied, but i had changed them out based on later requested changes by RandomShaper. they were resolved at the time though they do conflict now, is there a consensus on which direction i should go with those particular lines?

Then resolved, my bad, the change is correct 🙂

not a problem at all, and i appreciate the check! i just wanted to be sure i wasn't missing anything, haha

CC @RandomShaper

Thanks!

This doesn't seem to be included in changelog

correct me if i'm wrong, but i think this would only be in there if it is a part of a release. this missed the dev 2 window so it won't be in engine until dev 3 at earliest

The interactive changelog isn't updated automatically, it was last synced when I released 4.2-dev2. So yeah changes merged after dev2 aren't reflected yet.


[Microsoft.Maintenance] Added support for subscription and resource group assignment. 2023-03-01 API version release.

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Microsoft.Maintenance


Crash SIGSEGV in libjsc.so on Android

Description
Crash report from firebase crashlytics in the production mode on Android 10 with Huawei and Xiaomi devices

Crashed: Thread :  SIGSEGV  0x0000000000000010
#00 pc 0x73121a9748 libjsc.so
#01 pc 0x73121f5fe8 libjsc.so
#02 pc 0x7407046ba8 libc.so

React Native version:
react-native 0.63.2
Steps To Reproduce
I have just a crash report from firebase crashlytics in the production mode. In the develop mode it is not possible to catch it. No steps!
Expected Results
The app does not crash.

The JSC is the javascript engine. I think you would need the actual javascript that caused the crash reproducibly, posted as a minimal reproducible example, on the JSC project's bug tracker?

This is the most frequent crash on my apps, but haven't been able to reproduce it locally. Is there any way to log the crash better?

Same issue, any update?

same crash in my app and i m not able to reproduce.

not exactly a fix, but we switched to use Hermes instead of JSC which prevents this.

I think you would need the actual javascript that caused the crash reproducibly, posted as a minimal reproducible example, on the JSC project's bug tracker?

Thanks for pointing this out @mikehardy That's the correct solution.
Given that using Hermes here fixes the problem, I'm going to close this issue. Please move to the JSC issue tracker or reopen an issue here if it's affecting both engines.

Looks lite the issue is back
have a crash on Galaxy S20 5G/Note10 Lite/A22 5G/Galaxy A53 5G/Galaxy S10+/Galaxy S22/Galaxy A52s 5G/Galaxy S21 Ultra 5G/Galaxy S21 Ultra 5G/Pretty much all samsungs, android 12 and react 0.63.4
Crashed: Thread :  SIGSEGV  0x0000000000000040
#00 pc 0x646bbc libjsc.so
#01 pc 0x71ef7c85cc
#02 pc 0xf540 libart.so
#03 pc 0xf540 libart.so

0.63.4

I don't believe this version is supportable at all - please see if you can reproduce on current https://github.com/facebook/react-native/releases