[FEATURE REQUEST]  match ai voice with srt

is it possible to match an ai voice to the coresponding srt file

That would require manual work, or a script/extension.

https://github.com/ThioJoe/Auto-Synced-Translated-Dubs
Like here


PROD-1074: Upgrade to django 2.2.

Description: This PR updates code-annotations to be compatibile with django 2.2.  It also adds tox environments for testing django 1.11, 2.0, 2.1, 2.2.  Finally it fixes some lint errors that arose while updating dependencies.
JIRA: Link to JIRA ticket
Dependencies: dependencies on other outstanding PRs, issues, etc.
Merge deadline: List merge deadline (if any)
Installation instructions: List any non-trivial installation
instructions.
Testing instructions:

Do thing A
Expect B to happen
If C happened instead - check failed.

Reviewers:

[ ] tag reviewer
[ ] tag reviewer

Merge checklist:

[ ] All reviewers approved
[ ] CI build is green
[ ] Version bumped
[ ] Changelog record added
[ ] Documentation updated (not only docstrings)
[ ] Commits are squashed
[ ] PR author is listed in AUTHORS

Post merge:

[ ] Create a tag
[ ] Check new version is pushed to PyPi after tag-triggered build is
finished.
[ ] Delete working branch (if not needed anymore)

Author concerns: List any concerns about this PR - inelegant
solutions, hacks, quick-and-dirty implementations, concerns about
migrations, etc.

@fysheets @jmbowman I believe I have addressed all of your comments, please have another look when you get a chance.  Thanks!


Add Store.Versions()

This can be used by compacter and other code wanted to get a list of all available state versions

Superseded by #49


docs: remove requirement of template analyzer from DoD

Removes requirement to have TA in templates

@kristenwomack @gkulin FYI


Update arrays-and-collections.md

Proposed fix for https://github.com/dotnet/docs/issues/17384
Summary
Describe your changes here.
Fixes example code
Fixes #Issue_Number (if available)
https://github.com/dotnet/docs/issues/17384

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


Disabling creation for PowerShell 6 Functions

As we are quickly approaching the end of like support for PowerShell 6 (reference), we would want to disable the functionality of creating new PowerShell 6 functions.
I believe we are tracking to support PowerShell 7 (#1866) in the language dropdown,  it would be great to disable PowerShell 6 after a user could explicitly pick PowerShell 7 from the language drop down to avoid any confusion.
Please let me know if I can provide more information.

This issue has become stale and is at risk of being closed. The community has 60 days to upvote the issue. If it receives 5 upvotes we will keep it open and take another look. If not, we will close it. To learn more about how we handle issues, please see our documentation.\n\nHappy Coding!

:slightly_frowning_face: In the last 60 days, this issue has received less than 5 community upvotes and we closed it. Still a big Thank You to you for taking the time to create it! To learn more about how we handle issues, please see our documentation.
Happy Coding!


Linking requests to multiple specimens.

Hi,
How should we link multiple specimens to a single observation? For example, a creatinine clearance observation can be configured in our system as a calculated result with two specimens. However, the international FHIR profile limits the observation.specimen link to max. 1 specimen reference.
I mailed the HL7 Belgian workgroup about this a few months ago and I got the following recommendations :

One recommendation is that we use the values from the different measurements into different Observations and group all of them using derivedFrom into a single Observation. So, blood tests = Observation1, Urine = Observation2, Total result= Observation3, which is derivedFrom Observation1 and Observation2.

Alternatively, we could use an extension like “additionalSpecimens” to capture the other specimens. That is actually one of the options on the table. But I think that this may be a bit more risky approach, because then we may lose some of the details about the individual results (datetime of the urine test vs blood test, etc.).

I prefer the second approach since it is more closely resembles the situation in our application. Moreover, it does not exclude the first approach. What are your thoughts on this?
Best,
Thibault

An example with glucuse and two specimens:
MSH|^~&|||||20220201163652||ORU^R01^ORU_R01|6730233880634243423|P|2.5|||||||||
ORC|SC|OP_00001^ORBIS|OF_00001^Glims|OPGN0001^ORBIS|A||||20201010050505
OBR|1|OP_00001^ORBIS|OF_00001^Glims|GLUD^GLUD^L^L_GLUD^GLUD^LOINC|||""||||||||||||||||||P|||||||JVA&&&&&&&&L^20100228183010+0000
TQ1|||||||||A
OBX|1|NM|GLU^GLU^L^L_GLU^GLU^LOINC||82.7|^^L|20 - 90|""|||P|||20100228080010+0000|^^L||||20100228183010+0000
NTE|1|L|Result External Comment|P
SPM|1|SPM_OF_00001&Glims^SPM_OF_00001&Glims||SERUM^Serum^L^L_SERUM^Serum^LOINC||||||||||External Specimen Comment|||20100228080010+0000|20100228083010+0000||Y
SPM|2|SPM_OF_00002&Glims^SPM_OF_00002&Glims||URINE^Urine^L^L_URINE^Urine^LOINC||||||||||External Specimen Comment Urine|||20100228080010+0000|20100228083010+0000||Y

The WG postpones the issue until there is a question from the field.


2754 partial instead of error

fix #2754

I don't increment the version, let's wait for https://github.com/huggingface/dataset-viewer/issues/2898, to save some resources.

Once deployed, I'll relaunch the datasets with error


New import/export interface

Dataset and databook import_ and export methods
Overloaded import_set and import_book functions to handle defined format
Passing kwargs to formatters to allow some customization

:sparkles: :cake: :sparkles:


Bugfixes

Wrong chain in new referenda title (#869435jfk)
Blank screen on push notification settings (#86942wtmf)
Update status for Passing referenda (#869435k5c)
Update staking rewards screen title (#869435kab)
Supports sound

Task linked: CU-869435kab iOS. Update staking rewards screen title

Task linked: CU-869435k5c iOS. Update status for Passing referenda

Task linked: CU-86942wtmf iOS. Blank screen on push notification settings

Task linked: CU-869435jfk iOS. Wrong chain in new referenda title


v2.4.5 Binding UI wont open

So i couldnt open binding UI window yesterday (17th of October). Every time i try it would show me "This site can't be reached". I decided to upgrade to the newest version, but it didnt solve the issue. I still cant open binding UI. Steam doesnt write any errors, steamVR is on and running, headset is connected, controllers are on, internet connection is good. I couldnt find ppl with same issue on forums, and i have no idea why it could occur, since everything worked few days ago.

Unfortunately the port has changed. You can either manually change the port in the url to 27062, or open the binding ui through the steamvr menu (SteamVR -> Menu -> Controller Settings) You may need to be running your app to change bindings.
This will be fixed soon

or... if you open the script SteamVR.cs on line 372 there is a hard coded string called bindingurl
Changing the port number in this string fixed the issue for me

I am trying to connect to http://localhost:27062/dashboard/controllerbinding.html?app=application.generated.unity.testproj.exe as well as to the default 8998 port  but at no avail. Also the controller settings from the steamvr menu just shows a darker overlay when I am trying to edit controller settings. I am using unity 2019.1.5f1 on win10.

Changing the port in the script as @ataylor09 is probably the best solution for right now, a beta is coming soon to address this.

Thanks again for the report! This has been fixed with a proper api in the new 2.5.0 release on the asset store and here: https://github.com/ValveSoftware/steamvr_unity_plugin/releases/tag/2.5.0

still having this problem on 2.5.0 (sdk 1.8.19)


what would it take to use this from Java code?

the project looks very interesting !
I am not familiar with AWS Serverless Application Repo, but how complicated would it be to use this from a Lambda developed in Java with the Selenium WebDriver library ?

Hi @gotson,
It would be slightly complicated, but not impossible (maybe several days to a week or two to get a prototype working?). You'd need to:

Port the code in index.js to the Java equivalent. It basically is just a wrapper around a bundled binary that it calls (the webdriver and chrome browser instance). It's not complicated JavaScript but it'd help to be familiar with node.js conventions.
Compile your existing Java selenium tests to work in the AWS Lambda environment and bundle with the function.

Depending on how many Java selenium tests you have or how familiar you are with AWS Lambda/node, it might be easier to rewrite your existing tests in Javascript.
The benefit of using a scripting language (python or node.js) to run tests in lambdium is that they don't have to be compiled beforehand. This allows people to specify the selenium script at runtime vs deploying them with the function.


Fix support Django 1.10

Fixes #6
I add support for Django 1.10
Please waiting for my comment to fully ready this PR for merge. Thanks!

@mishbahr all works with: https://github.com/mishbahr/django-connected
Please merge this PR with:

https://github.com/mishbahr/django-connected/pull/4
https://github.com/mishbahr/djangocms-instagram/pull/8


feat: Support TypeScript (reverted)

close: https://github.com/baseballyama/svelte-preprocess-delegate-events/issues/24

see: https://github.com/baseballyama/svelte-preprocess-delegate-events/pull/27


Add CI for markdown link check and linting.

Changes:

markdown link check to identify dead links.

Demo

markdown linter for properly formatted markdown docs.

Demo

thanks @yankeexe, decided the repo doesn't deserve a CI 😅


Add commit_time to the checkpoint in case of consistent streaming

Need to make change to subtract 1 in all cases.


LF Can't Be Treated In Source

Program Scanner assumes only CR+LF for new line in program source file (LF is bad case).

Fixed in Program Scanner 0.5b.


How do I use html placeholder text in filter, insert and edit textboxes

This is minor, I'm having difficulty putting placeholders in the filter row textboxes, insert and edit textboxes.

Good point, there should be a config option for this.
For now you can redefine _createTextBoxmethod of the field like following:
_createTextBox: function() {
    var $result = jsGrid.fields.text.prototype._createTextBox.call(this);
    $result.attr("placeholder", "Field" Placeholder);
    return $result;
}

Thanks a lot man!
I used:
editTemplate: function() {
var $result = jsGrid.fields.text.prototype._createTextBox.call(this);
$result.attr("placeholder", "Field Placeholder");
$result.attr("value", value);
return $result;
}

When I clicked to edit, the values kept disappearing, so I added
$result.attr("value", value);
But I now get a jsgrid.min.js:7 Uncaught TypeError: Cannot read property 'val' of undefined error, and the grid doesn't display.

The issue with editTemplate is that you should also pass the params like that:
editTemplate: function() {
    var $result = jsGrid.fields.text.prototype._createTextBox.apply(this, arguments);
    ...

Hey tabalinas,
dsGrid is a very good control. I don't think the user can be expected to figure out the above. I'm doing Ajax postback with some read only fields in the table, so I'm using editTemplate. Can you make editTemplate somehow easier to use? Or at least put the above in the docs? Thanks!

jsGrid was so easy to setup, I never dug into any code - I set it up and it just worked.  So this took me a little time to figure out how to do this, and I have some note that may help others.
You can pass custom functions to the fields, that is what is above.  I created a function before of the jsgrid init::
var placeholderFilterTemplate = function () {
    var $result = jsGrid.fields.text.prototype.filterTemplate.apply(this, arguments);
    $result.attr("placeholder", "Search " + this.title + "...");
    return $result;
}

$("#jsGrid").jsGrid({

Then I made all the field configs that have filtering enabled point to this filter template function:
           fields: [{
                name: "first_name",
                title: "First Name",
                type: "text",
                filterTemplate: placeholderFilterTemplate,
            },
            {
                name: "last_name",
                title: "Last Name",
                type: "text",
                filterTemplate: placeholderFilterTemplate,
            },
           ...


Too complicated to use

Why not provide a built-in database?
docker run --rm -it opensuse/portus:2.2 ok ?

Portus follows the common practice to run a single application per  container. That what the main purpose of Docker is.
You can find many sources on the net
Eg. https://developers.redhat.com/blog/2016/02/24/10-things-to-avoid-in-docker-containers/

@Vad1mo
你好，我不是菜鸟，我就是觉得 Portus 不能一条命令运行起来，如： nexus 3 提供的 docker ，单纯使用一条命令即运行起来，明白我的意思没有？
Hello, I'm not a rookie, I just don't think Portus can run a single command, such as the docker provided by nexus 3, and just use a single command to run it, and understand what I mean?

典型的** Container Registry **应用程序由许多不同的进程/应用程序组成。 Portus WebUI，Docker Registry，PortusCtl，Clair，最后是DB，它通常是一个共享实例。
将数据库放入与Portus WebUI相同的容器中并不会产生太大的变化，但会使维护变得更加困难。 你不这么认为吗？
The typical Container Registry application is made up of many different processes/applications. The Portus WebUI, Docker Registry, PortusCtl, Clair and finally the DB, which is often a shared instance anyway.
So Putting the DB into the same container as Portus WebUI would not make much of a difference, but would make it more difficult to maintain. Don't you think so?

I have to admit that I'm not acquainted with the architecture of Nexus, but last time I looked into it, it looks like they provide their own implementation of the registry. We decided to follow another route: simply using the provided and stable API from the upstream project. For us it is better to do this because:

People already using a Docker registry can then add Portus as an additional layer.
There is a clear separation of concerns: the Docker registry (docker-distribution) stores the contents, and Portus is a layer on top of it providing authentication and authorization (and a bunch of other management utilities).

Separating concerns like that was a better use case in our understanding, since then people can deploy components in a more microservices-friendly way. You could have everything into a single container, but then you'd be effectively re-creating a monolith (in a non-intuitive way if I may add). For example, if you discover that the DB is your bottleneck, and not portus, you can simply scale up the DB service and leave portus alone (which is common practice if you are using container orchestrators like Docker Swarm or Kubernetes). The same for the different services orbiting around a typical Portus installation.
That being said, the downside is maybe that running a single docker run command is of course easier (even though I'd then debate whether doing that is desirable at all in a production environment...). We prefer the freedom that separating concerns properly leaves to system administrators to arrange a typical Portus application as they see fit. On the other hand, we are working on making things easier (see our examples directory, or this Helm chart for Kubernetes), so all feedback is welcomed.
Finally, I'll close this issue because issues are meant for discussing bugs, features, etc. For general discussions we can rely on the mailing list.


Can't use comma in filters

Because , is used to separate filters it's impossible to use it as a filter value.

Fixed im https://github.com/prymitive/karma


p2sh psbt convenience methods

UPDATE: Finally ready to merge!
Things missing (see TODOs in code):

[x] Change detection (https://github.com/buidl-bitcoin/buidl-python/pull/81/commits/7c91e98b83c5cfc671d7d3d46076819e4d59a3d7 and earlier)
[x] Add support for non-lexicographical ordering of child pubkeys
[x] Better variable names. UPDATE: done in https://github.com/buidl-bitcoin/buidl-python/pull/81/commits/531bb1ab8e1791cbbbd482bc4fc613acc55ae7cb
[x] Add optional target_addr argument to create_p2sh_multisig method for safety (done in https://github.com/buidl-bitcoin/buidl-python/pull/81/commits/3b939190d1113c7d494609608407ea7cf85548f8)

No

Have option to only require the prev_hash/idx/amount and not the full tx hex for all UTXOs? This isn't compatible with coldcard, so not going to do this for now.

Would input_dicts be better than inputs_dict_list? Similarly for output_dicts vs outputs_dict_list?

@dhruvbansal thanks for the code review!
It makes sense if you want to to do your own custom thing, and buidl allows you to do it however you like.
I got rid of the complicated unsorted RedeemScript helper method, as that's pretty niche. But, if you want to create a RedeemScript from out of order pubkeys the existing method allows it natively (and has a safety check where you can specify the expected_addr).
I left in the general create_p2sh_multisig_psbt() helper method because I think the functionality is important for building on top of (and as a bonus I like adding test coverage to buidl wherever possible). That said, it's in a separate psbt_helper.py1 file so it should keep the psbt.pyfile as easy to read as possible. I'd like to one day add similar methods forp2wsh/p2sh-p2wsh`, so that users can even run a multisig coordinator (and signer) without even needing a bitcoind node.
Does that work/make sense for you? Changes are easiest to see in https://github.com/buidl-bitcoin/buidl-python/pull/81/commits/5cae54f7c870926876fc7b274e9737ae65396d38 .

Now live on pypi:
https://pypi.org/project/buidl/0.2.21/


docs(Dialog) added missing props and fixed story events

Changed

Readme index with default values for Dialog
Dialog event names
let prettier redo some formatting

:tada: This PR is included in version 3.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Blue screen using Firefox

Hello,
sometime tha system crashes using Firefox (Xserver on Windows side).
Verified on two different machines, both Core i7, 16 GB RAM and Build 14367.
Website (and page opened) is random.

I am fairly certain this is a bugcheck in inet sockets that we fixed recently that is making its way to the Windows Insider builds.  Do you have the c:\Windows\MEMORY.DMP file present on your machine?
If so, would you be able to email a link to the dump to<EMAIL_ADDRESS>Include in the subject / message to forward to the WSL team. In the body of the email mention it is for benhill.

No Ben, sorry.
I think deleted with "Disk Clean" utility... I'll ask to friend reported the same crash on his machine.

No problem at all! As I mentioned I'm virtually certain it's a bug that has been fixed, just doing my due diligence.

OK, my friend tells that he turned off the machine after the crash, then the file should be on the disk.
later he will check and if there is he'll send it.

Hi Ben,
mi friend tells the file is too big for email (abt 780 MB).
If you need it we can park it on a FTP server.

That would be great. Thanks!

Hi Ben,
The file is on my FTP, I sent your account info via email.

@iz0eyj Thank you very much, I downloaded your dump and was able to confirm this is the same socket related bugcheck that we fixed recently.  Same issue as #356 and #523.  That fix will be in a Windows Insider build soon.
I have deleted the dump from your FTP share.

To serve & To protect

The fix for this issue is in Build 14376, please let us know if you see any bugchecks in that build.

Hi Ben, I'm testing Firefox and ita seems thar runs fine... some warning and I cannot install Adblock (installation fails with this error: (process:7946): GLib-CRITICAL **: g_path_get_basename: assertion 'file_name != NULL' failed), but works whitout crash.
Warnings at startup:
Federico@DESKTOP-409R74A:~$ firefox
Sandbox: unexpected multithreading found; this prevents using namespace sandboxing.
(firefox:7849): Gdk-WARNING **: XSetErrorHandler() called with a GDK error trap pushed. Don't do that.
(firefox:7849): Gdk-WARNING **: XSetErrorHandler() called with a GDK error trap pushed. Don't do that.
.
.
.
(firefox:7849): Gdk-WARNING **: XSetErrorHandler() called with a GDK error trap pushed. Don't do that.
Tell me Ben, I dont see any changelog for Linux, is complete?

@iz0eyj There is a blog post released recently by @russalex that is probably the best answer for your question: https://blogs.msdn.microsoft.com/wsl/2016/07/08/bash-on-ubuntu-on-windows-10-anniversary-update/

Thanks for the link Ben.
Now is possible to spend a bit of time to create some software that runs on both the side, using the best of Win and the best of Linux.


Use StreamResponse API instead of ResponseBase API in Chunked Download

Description
Prototypes using StreamResponse API instead of ResponseBase API when downloading in parallel to file. The ResponseBase API has additional overhead of converting the raw HttpHeaders into the ResponseBase's headers type when the headers will never be used.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


Add link to Chassis on Computer System

As oneview-redfish-toolkit, I access "/redfish/v1/System/<server_hardware_uuid>" and the info returned contains a link to this ComputerSystem's Chassis (possibly in the form: /redfish/v1/Chassis/<server_hardware_uuid> using the same Server Hardware UUID)

The correct form o the link would be a link to the chassis containing it. In the case, it can be
/Chassis, to list all and the server hardware will be there too or
/Chassis/EnclosureUUID which is the chassis for the given computer system.
I´m implementing the later.


Changing backbone to ViT-B/16

Thanks for your great work!
Have you tried using the ViT-b/16 as a backbone? I was replacing the backbone to align with other work and found that compared to the ResNet50, the ViT-b/16 outputs an extra global token, causing that the features can not be reshaped to  [B, C, H, W].

Hi, thanks for your question.
Indeed, we didn't test ViT architectures, I am not sure if the use of  AdaIN and PIN is straightforward for architectures different from CNNs.

Thanks for your reply. Actually, I have tested ViT architectures, but it seems it doesn't work.

Firstly, I replace the ResNet50 with ViT-B/16 and maintain the decoder DeeplabV3+, but it only get about 50 mIoU in the source training, which is much lower than the performance of ResNet50.

I replace the DeepLabV3+ with the VisionTransformerUpHead in SETR to match the ViT-B/16, but it still get about 50 mIoU.

Do you think PIN and AdaIN have any limitations when using pre-trained ViT of CLIP? Or is there any important part I'm missing that is causing such low performance?

I don't know about using AdaIN with vision transformers (I haven't seen it yet at least).
For your source training are you sure the backbone is frozen? If not it might require some hyperparameter tuning for training, as training CLIP-initialized networks doesn't necessarily work with the same hyperparameters used for ImageNet-initialized ones (as reported in DenseCLIP paper https://arxiv.org/abs/2112.01518).

Yes，I freeze the backbone with the arg --freeze_BB in the source training. In my opinion，you only train the segmentation head（Deeplabv3+） in the source training. Am I right？Is there something I ignore？

Yes in the source training and adaptation we only train the segmentation head. I don't know the problem then, please check your implementation and try tuning the hyperparameters for learning

I'll close this issue now. please feel free to re-open it if you have any other questions.


python310Packages.owslib: 0.26.0 -> 0.27.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.owslib is: "client for Open Geospatial Consortium web service interface standards"
meta.homepage for python310Packages.owslib is: "https://www.osgeo.org/projects/owslib/"
meta.changelog for python310Packages.owslib is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.27.2 with grep in /nix/store/6j1vyw8ilvw336glzjwrp7fkrms4z2q8-python3.10-OWSLib-0.27.2
found 0.27.2 in filename of file in /nix/store/6j1vyw8ilvw336glzjwrp7fkrms4z2q8-python3.10-OWSLib-0.27.2

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.cartopy
python310Packages.owslib
python39Packages.cartopy
python39Packages.owslib
qgis
qgis-ltr

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6j1vyw8ilvw336glzjwrp7fkrms4z2q8-python3.10-OWSLib-0.27.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.owslib https://github.com/r-ryantm/nixpkgs/archive/b149fa192c85d093bbb70befeafa73065438ee03.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6j1vyw8ilvw336glzjwrp7fkrms4z2q8-python3.10-OWSLib-0.27.2
ls -la /nix/store/6j1vyw8ilvw336glzjwrp7fkrms4z2q8-python3.10-OWSLib-0.27.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 45m and timed out

Maintainer pings

Test failures for me on macos 10.15:
__________________________ test_ogcapi_records_pycsw ___________________________

    @pytest.mark.online
    @pytest.mark.skipif(not service_ok(SERVICE_URL),
                        reason='service is unreachable')
    def test_ogcapi_records_pycsw():
        w = Records(SERVICE_URL)

        assert w.url == 'https://demo.pycsw.org/cite/'
        assert w.url_query_string is None

        api = w.api()
        assert api['components']['parameters'] is not None
        paths = api['paths']
        assert paths is not None
        assert paths['/collections/{collectionId}'] is not None

        conformance = w.conformance()
        assert len(conformance['conformsTo']) == 12

        collections = w.collections()
        assert len(collections) > 0

        record_collections = w.records()
        assert record_collections == ['metadata:main']

        pycsw_cite_demo = w.collection('metadata:main')
        assert pycsw_cite_demo['id'] == 'metadata:main'
        assert pycsw_cite_demo['title'] == 'pycsw OGC CITE demo and Reference Implementation'  # noqa
        assert pycsw_cite_demo['itemType'] == 'record'
        assert w.request == 'https://demo.pycsw.org/cite/collections/metadata:main'  # noqa
        assert w.response is not None
        assert isinstance(w.response, dict)

        pycsw_cite_demo_queryables = w.collection_queryables('metadata:main')
>       assert len(pycsw_cite_demo_queryables['properties'].keys()) == 11
E       AssertionError: assert 12 == 11
E        +  where 12 = len(dict_keys(['geometry', 'type', 'title', 'keywords', 'parentidentifier', 'time_begin', 'time_end', 'date', 'platform', 'instrument', 'sensortype', 'edition']))
E        +    where dict_keys(['geometry', 'type', 'title', 'keywords', 'parentidentifier', 'time_begin', 'time_end', 'date', 'platform', 'instrument', 'sensortype', 'edition']) = <built-in method keys of dict object at 0x113dafe00>()
E        +      where <built-in method keys of dict object at 0x113dafe00> = {'date': {'title': 'date', 'type': 'string'}, 'edition': {'title': 'edition', 'type': 'string'}, 'geometry': {'$ref': 'https://geojson.org/schema/Polygon.json'}, 'instrument': {'title': 'instrument', 'type': 'string'}, ...}.keys

similar for test test_ogcapi_records_pygeoapi

Though upstream don't appear to run ci on macos, if I hack it to do so it does appear to pass there: https://github.com/risicle/OWSLib/runs/8073774498?check_suite_focus=true

#191333


[System tests] Adding if-exists option to delete topics

Type of change

Enhancement / new feature

Description
Once if-exists option has been added to the strimzi test clients to delete topics, we should use it to avoid making more calls to the admin client cli.
Additional Context
Fixes #1169 .
Checklist

[ ] Write tests
[ ] Make sure all tests pass
[ ] Review performance test results. Ensure that any degradations to performance numbers are understood and justified.
[ ] Make sure all Sonarcloud warnings are addressed or are justifiably ignored.
[ ] Update documentation
[ ] Reference relevant issue(s) and close them after merging
[ ] For user facing changes, update CHANGELOG.md (remember to include changes affecting the API of the test artefacts too).

@strimzi-ci run system tests

:arrow_forward: Build started - check Jenkins for more info. :arrow_forward:

:heavy_check_mark: Test Summary :heavy_check_mark:
BUILD NUMBER: 56
KAFKA VERSION: 3.7.0
TOTAL: 17
PASS: 16
FAIL: 0
SKIP: 1


Termanal and lazygit.nvim have different background color

After update I have different backgrounds in the terminal lazygit and in lazygit.nvim(1st screen). I want to set to black, the same as in terminal.(2nd screen)

How can I change lazygit background color?

I am facing the same issue

Same here

Same here

Isn’t there a way to change the background of LazyGit in their config file?

( sorry, I’ve been away from open source for personal reasons and I still haven’t fully gotten back into it yet, and I haven’t had the chance to check what is going on here )

I did look for it but didn't find any option for the background color. It's just the colors for things like staged/unstaged files, pushed/unpushed commits etc.


switch to the develop branch as default

Currently it is very confusing that the master uses C++14 and develop uses C99. Maybe it's a good idea to switch to the develop as the default branch.

v2.0 (what you see in develop) is still in development and currently not suitable for production use. People tend to checkout and use whatever is defined as default branch and right now, that wouldn't be a good idea at all. Some major changes in operand decoding (and thus the public interface) and especially encoding are still ahead, also we're even considering to rename the whole project in order to reflect that it's no longer just a disassembler. Right now, we prefer people to still use the C++ (v1.0) version.

Okay, could you upload the instruction editor as a binary somewhere though?
I want to try this version but I'm missing the generated opcode files.
On Sat, 8 Apr 2017 at 18:13, Joel Höner<EMAIL_ADDRESS>wrote:

v2.0 (what you see in develop) is still in development and currently not
suitable for production use. People tend to checkout and use whatever is
defined as default branch and right now, that wouldn't be a good idea at
all. Some major changes in operand decoding (and thus the public interface)
and especially encoding are still ahead, also we're even considering to
rename the whole project in order to reflect that it's no longer just a
disassembler. Right now, we prefer people to still use the C++ (v1.0)
version.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zyantific/zyan-disassembler-engine/issues/11#issuecomment-292727844,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWCmSxSMCl7u0-NBs2UFNVwAocG64u2ks5rt7I2gaJpZM4M3vnR
.

Mhh, I don't have an up-to-date binary at hand right now either, but compilation probably failed because of the missing encoder table anyways, which currently isn't generated by the instruction editor, but by a temporary Python script.
I pushed a change to develop that allows compilation by updating the CMake defaults and disabling the encoder. If you're interested in that part of the lib, I pasted you the encoder table here:
https://gist.github.com/athre0z/b55273c2106904981c77265ef8ea626d

After installing all the components/packages/whatever in Delphi XE8 I get the following errors compiling:
[dcc32 Error] formMain.pas(934): E2003 Undeclared identifier: 'AccessMode'
[dcc32 Error] formMain.pas(962): E2003 Undeclared identifier: 'AccessMode'
[dcc32 Error] formMain.pas(962): E2003 Undeclared identifier: 'TOperandAccessMode'

But thanks a lot! I'm mainly interested in the decoder and I plan to run the capstone tests to see how this stacks up. My main interest in the InstructionEditor was just to play around and see if I can figure out how it works.

Okay I found the problem, AccessMode was renamed to Action. I'll send a PR later.

Nice, thanks! Please keep in mind that everything you see isn't final, yet. I guess it'll take us another month or two to get to beta state. Besides the operand rework, we're currently developing tests to further optimize the tables and to diff against the "tables" from XED in order to eliminate errors and further tweak our own ones.


Fix flaky test test_random:test_randint_generator

Description
Fixes #13446 flaky test test_random:test_randint_generator
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[X] Changes are complete (i.e. I finished coding on this PR)
[X] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[X] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Comments

Created the alpha variable to ensure consistency
Scipy uses 0.01 for discrete distribution here - https://github.com/scipy/scipy/blob/9f12af697763fb5f9767d5cb1280ce62456a3974/scipy/stats/tests/test_discrete_basic.py#L45
Hence for testing randint (discrete uniform distribution) specifically passed alpha=0.01
Due to random factor lacking robustness had to set fixed seed to prevent CI from failing

@mxnet-label-bot add [Flaky, pr-awaiting-review]

Can you run this test repeatedly 1k times with different seeds to check flakiness.

@anirudh2290 Have tried it 1000 times
MXNET_TEST_COUNT=1000 python -m nose -v --nocapture tests/python/unittest/test_random.py:test_randint_generator

Python3 CPU Linux


Fix the head navigation bar and the "Sign up for newsletter" section

Description
The top navigation bar is very cluttered, and various elements are overlapping themselves. Also, the "Sign up for newsletter" section appears static. I would suggest you add a minimal hover effect on the "Enter your email" and a highlighted hover effect on the Subscribe button.
If you would like, I'll be more than happy to work on this issue.
Steps to Reproduce
N/A
Expected Behavior
No response
Actual Behavior
No response
Screenshots

Additional Information
No response
What browser are you seeing the problem on?

[ ] Firefox
[X] Chrome
[ ] Safari
[ ] Microsoft Edge

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

I have seen many navbar issues, and i have mentioned it before for contributors to not create issues regarding navbar and there should be no more navbar issues, so i have closed this issue, if you have any queries, you can join discord server for further updates and communication.
we are not looking for navbar updates for now, as it was done and changes many times before.
So I am closing this issue.
please understand.
from my side navbar is looking good

and i don't want any improvements further on the "Sign up for newsletter" section


fixed base64 links

here is a small fix to correct the links in devops that are already in base 64 encoding

thanks for the PR @gcerik !


keep updating an autoscaling policy based on an attached HPA

After https://github.com/mercari/tortoise/issues/209, we want to keep changing autoscaling policy based on an attached HPA if tortoise is Off.
Currently, when HPA managed by Off Tortoise gets changed by users, let's say users add a new metric for another container, Off Tortoise cannot generate the recommendation for that Off Tortoise.

But, I don't want to change the bahavior of the current Off Tortoise, because it's much simpler. This proposed behavior looks confusing.
So, let's make it as an opt-in feature (at least for now), maybe special annotation or something.

What about Auto Tortoise?
First, for Auto Tortoise, people shouldn’t change HPA on their own. They change Tortoise’s autoscalingPolicy and Tortoise will modify HPA based on changed autoscalingPolicy.
If they modify HPA directly, Tortoise will just overwrite the HPA and the original HPA will be back.


future leakage?

In demo NeurIPS2018, close/low/high value are used as states, it seems future leakage exist?

This issue is discussed at FinRL: https://github.com/AI4Finance-Foundation/FinRL/issues/938


WIP - Del vsb final

Thank you for contributing to Velero!
Please add a summary of your change
Does your change fix a particular issue?
Fixes #(issue)
Please indicate you've done the following:

[ ] Accepted the DCO. Commits without the DCO will delay acceptance.
[ ] Created a changelog file or added /kind changelog-not-required as a comment on this pull request.
[ ] Updated the corresponding documentation in site/content/docs/main.

time="2023-05-16T21:54:16Z" level=error msg="fail to get resource customresourcedefinitions.apiextensions.k8s.io. no matches for apiextensions.k8s.io/, Resource=customresourcedefinitions"


seeding data: update agreement base image

@jjeroch should i update the value on our environments already or do we wait until the next release?

have to close this pr since i rebased dev into it - sorry. new PR is created


Resolve Merge Conflicts

I accepted your 3 PRs

https://github.com/philippabele/pythonic-api-design/pull/22
https://github.com/philippabele/pythonic-api-design/pull/24
https://github.com/philippabele/pythonic-api-design/pull/23

There are now some merge conflicts. Please take care of them and resolve the conflicts.

I resolved all merge conflicts 👍🏻


Conditionally rendering a Router's <Routes> with if, <Show>, <Suspense> or other causes components to render twice in 0.5.0-beta

Describe the bug
See title. The same does not happen on version 0.4.8. This bug happens on version 0.5.0-alpha, alpha2 and beta.
Leptos Dependencies
leptos = { version = "0.5.0-beta", features = ["csr", "nightly"] }
leptos_router = { version = "0.5.0-beta", features = ["csr", "nightly"] }

To Reproduce
Try this minimal reproduction
use leptos::*;
use leptos_router::*;

fn main() {
    console_log::init_with_level(log::Level::Debug).expect("setup logging");
    console_error_panic_hook::set_once();

    mount_to_body(App)
}

#[component]
fn App() -> impl IntoView {
    provide_context(create_rw_signal(0_usize));

    view! {
      "Settings"

      "Other"

      <Router>

        // Imagine we're waiting for an async task to complete instead of 1 < 2
        <Show when=|| 1 < 2 fallback=|| view! {
"Loading"
 }>
          <Routes>
            <Route path="/settings" view=Settings/>
            <Route path="/other" view=Other/>
          </Routes>
        </Show>

      </Router>
    }
}

#[component]
fn Settings() -> impl IntoView {
    log::debug!("settings");
    let counter = expect_context::<RwSignal<usize>>();
    counter.update(|c| *c += 1);

    view! {
"Hi from settings " {counter}
 }
}

#[component]
fn Other() -> impl IntoView {
    log::debug!("other");
    let counter = expect_context::<RwSignal<usize>>();
    counter.update(|c| *c += 1);

    view! {
"Hi from other " {counter}
 }
}

When you start the web page and click on the Settings or Other hrefs, they log twice to the console and the counter is incremented twice instead of doing both things once. However if you visit each page for the first time, e.g. by manually writing the URL <IP_ADDRESS>:1420/settings or <IP_ADDRESS>:1420/other and pressing enter, they log only once and the counter shows 1 instead of 2. If you remove <Show> and render the <Routes> directly this bug does not occur. Switching back to version 0.4.8 also fixes this issue.
Expected behavior
The components above should only render once when visiting their pages from the href

Not saying that this isn't a bug or that it shouldn't be fixed. However, personally I'd say conditional route definition like this is kind of an "off-label use" of <Routes>. (See also #1488 for another example of conditional route definition causing problems). This is not the intended way to use the router, and for a very good reason: conditional route definition like this, as opposed to conditional route display, will break SSR (I'm pretty sure) because the server generates a route list on startup, so any conditional route definition may not make it to the server (Actix or Axum) router.
Instead, the pattern I'd suggest would be conditional route display. You can do this in the example given pretty easily with one level of nested routing. In the example below, the routes are always defined, but the two child routes are only shown when the condition is true. Conveniently, context works through Outlets (see example), so if you're doing something like loading a resource you can provide it as a context inside the Suspense, and expect that it's there in the child routes.
#[component]
pub fn App() -> impl IntoView {
    provide_context(create_rw_signal(0_usize));

    view! {
      "Settings"

      "Other"

      <Router>

        // Imagine we're waiting for an async task to complete instead of 1 < 2
          <Routes>
            <Route path="/" view=|| view! {
                <Show when=|| 1 < 2 fallback=|| view! {
"Loading"
 }>
                    {
                        provide_context(String::from("blueberry"));
                        view! { <Outlet/>}
                    }
                </Show>
            }>
                <Route path="/settings" view=Settings/>
                <Route path="/other" view=Other/>
            </Route>
          </Routes>

      </Router>
    }
}

#[component]
fn Settings() -> impl IntoView {
    log!("settings");
    let counter = expect_context::<RwSignal<usize>>();
    let ctx = expect_context::<String>();
    counter.update(|c| *c += 1);

    view! {
"Hi from settings " {ctx} {counter}
 }
}

#[component]
fn Other() -> impl IntoView {
    log!("other");
    let counter = expect_context::<RwSignal<usize>>();
    let ctx = expect_context::<String>();
    counter.update(|c| *c += 1);

    view! {
"Hi from other " {ctx} {counter}
 }
}


More facilities for inter-mod dependencies

Based on my experience with modding other games, there are a couple of additional features I suspect would be wise to include in regard to dependencies.  Though, I note that the spec doesn't document the dependencies field at all right now, so I'm not sure what you currently intend to support.
Conflicts
When a mod is known to conflict with another mod and players  should not be able to load them simultaneously (at least, not without bypassing a major disclaimer).
This is also useful for automated testing.  Currently the in-tree mods have a problem that they can't all be tested together but it's too slow to test them separately.  If we knew which mods conflicted then we could automatically partition them into maximal non-conflicting subsets for the purpose of efficient testing.
Ordering dependencies
A mod might be related to another mod in such a way that it should be loaded after that mod, but does not require it to be loaded.
This is a key feature in Factorio.  I'm not sure if this feature is crucial in CDDA because mod load order doesn't affect much, but I suspect it might start to have influence as more features are added.  For example, I saw a feature request for the ability to un-blacklist an item which another mod had blacklisted.  For these kinds of things it is important to load in the correct order.
For reference, the way Factorio mods write their three types of dependencies are as string entries in a json list, each as follows:
"bar >= 0.1" // Depends on the mod bar, version at least 0.1
"? bar" // Should load after bar if both are present
"! bar" // Conflicts with bar

(the latter two cases can also specify versions or ranges of versions).

Hey, thanks for the feedback!
A couple quick responses:

Conflicts were  definitely one of the things I had planned, but have not yet implemented. I'm still forming ideas on what that'd look like in this spec.
Regarding dependency ordering: This sounds something we should do that had not occurred to me, so great callout! I suspect the hard bits will be details of how CDDA resolves dependencies, and for that we'll need involvement from the game devs.

As a general comment, I'm not a fan of encoding stuff like dependency ordering or indication a mod conflicts with another mod in a version string. As a design goal, I'd like the registry spec to be as declarative as possible with as little room for confusion or surprise as possible. (I still need to write up a DESIGN_GOALS doc 🤦♂ ).
If we're going to be adding support for specifying relationships between mods that indicate things other than "we need this", we'll need to work with CDDA devs to enhance the dependencies field of modinfo.json, or come up with a new field that can be used instead. Drawing heavily from CKAN's relationship's spec and CPAN's requirements spec, I think something that'd allow us to declare richer mod relationships could look something like this:
Example modinfo.json (note the relationships field)
{
    "spec_version"   : "0.1",
    "ident"          : "jury_rigged_robots",
    "name"           : "Jury-Rigged Robots",
    "description"    : "Options for salvaging, jury rigging, and reprogramming broken robots.",
    "download"       : "https://github.com/TheSalamanderDJ/Jury-Rigged-Robots/releases/download/1.1/Jury_Rigged_Robots.zip",
    "license"        : "gpl-3.0",
    "version"        : "1.1",
    "release_status" : "stable",
    "cdda_version"   : "0.D",
    // currently
    "dependencies" : [ "dda" ],
    // proposed
    "relationships" : [
      // "dda" as first loaded dependency implied? Not sure how CDDA handles that.
      { "ident": "foo",  "type": "required", "version": "0.1" },
      { "ident": "bar",  "type": "conflict", "version": "0.3" },
      { "ident": "fizz", "type": "optional", "version": "0.5" },
      // The registry spec has defined simple version strings, but there hasn't been much
      // discussion yet on dependency resolution using version strings. This will probs
      // end up being it's own topic of discussion, so consider this example theoretical.
      { "ident": "buzz", "type": "conflict", "version": { "min": "0.7", "max": "0.9" } },
    ],
}

I suspect CDDA game devs may also have some thoughts here, will see if I can get a few of them in on this thread to share their perspectives on this topic.

Here's a few thoughts to kick off a discussion on mod dependency ordering:

Can we infer the optimal ordering of a group of mods based entirely off of a list of required and optional dependencies? If a mod marks something as a dependency (optional or otherwise), it should probably be loaded after all of a that mod's dependencies have been loaded. This effectively defines load order as being mods with no deps first, with mods with larger dependency lists coming last.
In cases we cannot infer correct mod load order, what are approaches that let us allow mods to indicate prefferred load order safely? I don't think we want to allow mods to say "load this before anything else", but it may be acceptable to allow something along the lines of "load me before this particular mod".


[mgmt] network release

https://github.com/Azure/sdk-release-request/issues/5740

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure/arm-network


Logitech C920

Bus 001 Device 007: ID 046d:082d Logitech, Inc. HD Pro Webcam C920
also works well~

does logitech c310 work fine? any idea?


Execution failed for task ':address_search_field:compileDebugKotlin'.

Hi!
when I launch the app get this error:
e: /flutter/.pub-cache/hosted/pub.dartlang.org/address_search_field-1.4.1/android/src/main/kotlin/dev/jostech/address_search_field/AddressSearchFieldPlugin.kt: (13, 14): Redeclaration: AddressSearchFieldPlugin
e: /flutter/.pub-cache/hosted/pub.dartlang.org/address_search_field-1.4.2/android/src/main/kotlin/dev/jostech/address_search_field/AddressSearchFieldPlugin.kt: (13, 14): Redeclaration: AddressSearchFieldPlugin
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':address_search_field:compileDebugKotlin'.

I have solved the problem with:
flutter clean

Hello!
Thanks for commenting, maybe it happened with the last update so you needed to refresh your build.


Added EnableExpress property

Can you please review the changes I made for adding new EnableExpress boolean property
Fixes https://github.com/Azure/azure-sdk-for-js/issues/10893

/azp run js - servicebus - tests

/azp run js - servicebus - tests

/azp run js - servicebus - tests

One test failed.. Unrelated to this PR (being fixed at https://github.com/Azure/azure-sdk-for-js/pull/11144)

Thanks @mohsin-mehmood!


Added tar.gz to packaging

@abonas , @pilhuhn : we had .tar.gz for the main Hawkular distribution, but somehow it was not added to services. If that was not a deliberate decision, this PR re-introduces this format (which is nicer on Linux environments)

@ppalaga : do you miss Hawkular? :) If so, you are also free to review this!

For server optimized Fedora/RHEL/CentOS (ie: minimal installations), unzip is not available. So, for automated deployments, it's useful (and easier) to have .tar.gz as well. Not to mention the --strip parameter for tar, which is not available for zip, meaning that we have to do some bash magic to move from hawkular-services-cd2/hawkular-services-dist-0.0.3-SNAPSHOT to hawkular-services-cd2.

If we had to choose between tar.gz and zip, I'd keep tar.gz, as I'm not even sure -services work on Windows :)

You persuaded me :)

Feel free to merge in which ever way you like


[Issue 2330] Extension - Add T&C for extension and webapp

Describe the task
Add T&C for extension and webapp
Link doc: https://www.notion.so/subwallet/SubWallet-Th-m-T-C-cho-ng-d-ng-243dabfe04f74347bfc400277655485d?pvs=4

🚀 Deployed on https://pr-2379--sw-web-runner.netlify.app


Add support for Short Decimal Type in Native Parquet Reader

Will land it.
From: Jimmy Lu @.>
Sent: Tuesday, October 11, 2022 1:32 PM
To: facebookincubator/velox @.>
Cc: oerling @.>; Mention @.>
Subject: Re: [facebookincubator/velox] Add support for Short Decimal Type in Native Parquet Reader (PR #2714)
@Yuhta approved this pull request.
—
Reply to this email directly, view it on GitHub https://github.com/facebookincubator/velox/pull/2714#pullrequestreview-1138118731 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKPPPT3RYSHUMRM46DB7Z73WCXFFTANCNFSM6AAAAAAQ3HIIQA .
You are receiving this because you were mentioned.Message ID: @.***>


Add a Check for machine status

Before checking other attributes

As discussed with @itzikb-redhat, the testcase fails because the machine does not have the SG defined for the worker that is not on 'Running' state. This check will help to identify the root cause of the failure in the test.
/lgtm

/approve


Readme.md should be enhanced regarding adjustments in Makefile

The Readme says in section Readme.md#makefile to add the following line(s):

EXTRA_COMPONENT_DIRS := externals/lv_port_esp32/components

If only this directory is added, the Kconfig files in the following files are not reflected in menuconfig:

./externals/lv_port_esp32/components/lvgl_esp32_drivers/lvgl_tft/Kconfig
./externals/lv_port_esp32/components/lvgl_esp32_drivers/lvgl_touch/Kconfig

and thus compilation does not work.
Adding the following line works for me:

EXTRA_COMPONENT_DIRS := externals/lv_port_esp32/components externals/lv_port_esp32/components/lvgl_esp32_drivers externals/lv_port_esp32/components/lvgl_esp32_drivers/lvgl_touch externals/lv_port_esp32/components/lvgl_esp32_drivers/lvgl_tft

Hi @an-erd,
You're right, the README haven't been totally updated as should be. As user what do you find the most easy way to use this project as a template for your personal project:

By doing what you did based on the instructions of https://github.com/littlevgl/lv_port_esp32#install-this-project-as-a-library-submodule-in-your-own-project
Deleting the demo_create function call and add your application like any other esp32 idf based project?

I ask because #77 have the same problem

Hi, sorry I missed the point in #77. Please see my comment in #81,

I'm using ESP-IDF with LittlevGL as a submodule, so your first option. I was asking why you put this in externals and not components? In components I've all other submodules, so if there's not a reason for that I'd propose to move to submodules.
So far with the above adjustment in the  makefile it's working pretty good, integration was very easy and straightforward. I currently only have a M5Stack with an ILI9341 without a touchpad. I'm using this code called from main.c I took from the examples, and doing the work in  lv_testdisplay_create(); :

void initialize_lv()
{
lv_init();
lvgl_driver_init();
static lv_color_t buf1[DISP_BUF_SIZE];
static lv_color_t buf2[DISP_BUF_SIZE];
static lv_disp_buf_t disp_buf;
lv_disp_buf_init(&disp_buf, buf1, buf2, DISP_BUF_SIZE);
lv_disp_drv_t disp_drv;
lv_disp_drv_init(&disp_drv);
disp_drv.flush_cb = disp_driver_flush;
disp_drv.buffer = &disp_buf;
lv_disp_drv_register(&disp_drv);
esp_register_freertos_tick_hook(lv_tick_task);
lv_testdisplay_create();
while (1) {
vTaskDelay(1);
lv_task_handler();
}
}

I'm using ESP-IDF with LittlevGL as a submodule, so your first option. I was asking why you put this in externals and not components? In components I've all other submodules, so if there's not a reason for that I'd propose to move to submodules.

I haven't updated the README as I should, should it be?

EXTRA_COMPONENT_DIRS := components/lvgl_esp32_drivers\ components/lvgl_esp32_drivers/lvgl_touch
components/lvgl_esp32_drivers/lvgl_tft\

If you see any other misleading thing on the README please comment it on the README, let get it up to date :)

@an-erd Fixed in my side, let me know if it works for you.


Slider on:input event

Could there be a on:input event implemented in the Slider component. I'm running into problems where on:change gets fired when the user moves the slider, and when the slider value changes. I only want the event to fire when the user moves the slider.
I recreated a demo here:
https://svelte.dev/repl/10aa153589364fddbccb7ec765bde60e?version=3.55.1
Hope it's clear enough, thank you!

Thanks for pointing this out. Slider, like a few other components, are dispatching change reactively when it shouldn't. The problematic code is here:
https://github.com/carbon-design-system/carbon-components-svelte/blob/57b6ea68b53368240b07faff80a8a8157fae0786/src/Slider/Slider.svelte#L119-L121
We're looking to standardize events (see #1621) and it probably makes sense to forward the native change event if one exists, rather than dispatch it manually. Let us know if you're willing to contribute a fix, otherwise we'll have this ready for v1.

Fixed in v0.82.10


Table 6A and Immunizations

In the Data Elements and FHIR Mappings page of the IG, you list the Diagnosis and Procedure resources as the FHIR resource used by Table 6A. Table 6A includes several immunizations that are better represented by the Immunization resource. We request that you also look at Immunizations given when calculating whether a patient meets Table 6A lines (for example, line 24: selected immunizations).
Background:
The UDS manual lists mostly CPT codes for immunizations on Table 6A, however we have heard feedback from our customers that they may not always document the procedure of giving the immunization and drop the specific CPT code for that (especially if it's a simple and easy immunization they can give in-clinic). Instead they may just document that the patient has a specific immunization on the encounter, which is instead mapped to the CVX code. Currently, in Epic we will count that for Table 6A, assuming it meets all of the other 6A requirements, even if it doesn't necessarily use one of the CPT codes suggested in the manual.
We want to make sure our customers can continue to meet Table 6A, without requiring them to add extra documentation.
Consider looking to Immunization resources for Table 6A. Also, consider listing CVX codes in the UDS manual for Table 6A. If UDS+ is going to require FHIR resources which require codes, the UDS manual will need to start specifying as many relevant codes as possible to ensure we cover all of the ways to capture this data.

Will discuss internally with UDS BPHC team and get back.


Sampling strategy

Hi Anders!
There is a mention of a sampling strategy within the paper waldeland2020.pdf
)
![image](https://user-images.githubusercontent.com/26381053/144034634-c129f28c-8f2d-403c-9758-ede7eba5fafc.png
How did you proceed? Did you apply a weighted mse?
Best, Virginie

Hi Virginie.
I think, the sampling strategy mentioned in that paper was based on making a histogram of all tree heights in a training batch, and randomly removing pixels (= replacing them with unknown/ignore) from the over-represented tree heights. Until the histogram-bins had equal size. However, we did not use that strategy in 'this' paper (which is undergoing revision these days).


Max batch size for Dataset

Describe the enhancement requested
I'm using pyarrow.Dataset.to_table(batch_size) and the maximum batch size seems to be hard-coded to 1048576. It'd be good to make it more flexible (in my case I'm interested in numbers like 100 million), however, if it's there for a reason, mentioning it in documentation would already be helpful
Component(s)
Documentation, Python

Can you link to where you're seeing that it's hard-coded and to that number? I see pyarrow.Dataset.to_table. That said, the batches here are RecordBatches which have an upper limit on row count based on your column types due to size limits on Arrow Arrays.
Can you give a bit more context as to why you originally became interested in increasing batch size?

@amoeba I've noticed this magic number appearing in multiple places within the source code, but I'm not sure which occurrence is the actual cause of the issue. When I set the batch size to less than 1048576, it is adhered to; however, any number above that is capped at 1048576.
In my use case, I'm dealing (locally, on a single big-ass machine) with a large number of huge parquet files as input, each containing over 1 billion records, and these files need to be re-partitioned based on the values in certain columns. The number of partitions can be as many as 5,000. Currently, if the data is processed in batches of 1 million (due to the cap), this results in a very large number of files per partition, while each file remains tiny. Consequently, the overall performance drops to below acceptable levels.

Can you share some code to show how you're approaching this? The ultimate thing you're trying to do is possible and part of the design of the Datasets module, see https://arrow.apache.org/docs/python/dataset.html#writing-large-amounts-of-data. Also be aware of the considerations in https://arrow.apache.org/docs/python/dataset.html#partitioning-performance-considerations.
What you most likely don't want to be doing here is converting one or more Parquet files into an Arrow Table, then converting that Arrow Table to one or more Parquet files.

Here's how I do it:
import pyarrow.dataset as ds

def yield_batches(dir_path: str, cols: list[str], batch_size: int = 50_000_000) -> Generator[pd.DataFrame, None, None]:
    dataset = ds.dataset(dir_path, format="parquet")
    for b in dataset.to_batches(columns=cols, batch_size=batch_size):
                yield b.to_pandas()

Experimenting with different batch sizes, you can see that the maximum attainable value is 1048576, even if a larger number is specified.


Ammonite eagerly evaluates lazy list

Hello,
Just noticed all my Scala Stream evaluated eagerly in Ammonite, right after I build them:
Welcome to the Ammonite Repl 1.0.0-RC6
(Scala 2.12.2 Java 1.8.0_60)
If you like Ammonite, please support our development at www.patreon.com/lihaoyi
@ def unfold[A, B](start: B)(f: B => Option[Tuple2[A,B]]): Stream[A] = f(start) match {
    case Some((elem, next)) => elem #:: unfold(next)(f)
    case None => Stream.empty[A]
  }
defined function unfold
@ unfold(0) { (x: Int) => if (x < 10) Some((x + 1, x + 1)) else None  }
res1: Stream[Int] = Stream(1, 2, 3, 4, 5, 6, 7, 8, 9, 10)

Expected output:
Welcome to Scala 2.11.11 (Java HotSpot(TM) 64-Bit Server VM, Java 1.8.0_60).
Type in expressions for evaluation. Or try :help.

scala> unfold(0) { b => if (b > 10) None else Some((b, b + 1)) }
res3: Stream[Int] = Stream(0, ?)

Bug also appears in Ammonite Repl 0.9.3 (Scala 2.11.11 Java 1.8.0_60)

This is expected behaviour. Ammonite will evaluate enough elements to either print a single line, or to lay things out vertically and have it exceed the truncation height. If you want to customize this, you can override the repl.pprinter() with a custom additionalHandlers to special-case Stream

Ah, thanks. I searched for "lazy" in documentation and couldn't find any clues if this is known behavior.

Yeah it's bitten people before. It's a bit unusual, but I don't really think it's a common enough problem to have special casing in master, especially given that it's easy to special-case yourself if desired. For "many" cases where your Streams are non-side-effecting and non-exception-throwing, it also shouldn't cause any change in behavior, and it's nice to nicely peek at them


File does not exist% on all of every eval

ARC
Average: Error: File does not exist%
HellaSwag

Can you send me your config?

what you mean config ,i runned on runpod (with a 3090,and tried nous,openllm,and eq-bench) and all with Average: Error: File does not exist%

Can you send me your config?

Got the same error as OP; my gist link - https://gist.github.com/Haleshot/5042ebdd0da5b1487e7e78e4a1c6dd92
Additional Context: Ran it through the recommended - https://github.com/CultriX-Github/llm-autoeval as referenced in your issue - https://github.com/mlabonne/llm-autoeval/issues/16; I'm running it on lightning.ai and don't have/need runpod resources.

Can you run it in debug mode to see where it fails?

Can you run it in debug mode to see where it fails?

It works now! I switched my studio session to GPU -> 1 L4 instance and used the DEBUG=TRUE in the suggestions/commands mentioned in #16
I'm guessing the GPU switch resolved the issue.

Excellent, yeah that might have been an issue related to the GPU or the VRAM.


First and only VBA for Outlook, PowerPoint and Word FizzBuzz solutions

This pull request adds three, first and only (i.e. do not seem to have ever been created before), original VBA solutions of the FizzBuzz problem for Outlook, PowerPoint and Word in response to Issue https://github.com/NLDev/Hacktoberfest-2020-FizzBuzz/issues/3:

FizzBuzzExcel.vba: a rename from FizzBuzz.vba.
FizzBuzzOutlook: an original, first and only, solution in Outlook.
FizzBuzzPowerPoint: an original, first and only, solution in PowerPoint.
FizzBuzzWord: an original, first and only, solution in Word.

Awesome! Thanks :D


Adjust timeouts for the salt command and api (bsc1130242)

By default, SUSE Manager globally sets timeout and gather_job_timeout to 120 seconds.
So, in the worst case, a Salt call targeting unreachable minions will end up with
240 seconds of waiting until getting a response.

More about these timeouts
https://www.suse.com/documentation/suse-manager-3/book_suma_best_practices_3/data/bp_chap_salt_minion_scaleability_timeouts.html


improve README.md

Make the text keep flowing into the list that follows. A tiny bit jarring otherwise.

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


boesp <ls>

There are many literary source references in the footnotes and corrections of boesp.
These are coded in boesp.all_ansi.txt using all capital latin letters.   The author uses a version of IAST that
is coded in a unique way in the digitization:

Letters followed by numbers represent diacritics  (e.g. N5 represents Ñ).
Some of the author's representation differs from current IAST, and we probably want to change to modern IAST.  e.g. JOGAVA1SISHT2HASA1RA -> YOGAVA1SISHT2HASA1RA
600+ ls instances are currently preceded by the <ls> tag  (without closing </ls> tag).

these should be given a closing </ls> tag

ALL the ls references should be tagged as <ls>X</ls>.

where an ls reference text spans 2 lines of digitization, the digitization should be altered so the full text occurs on 1 line. Example S4A1RN5G. PADDH. SUBHA1SH. (in F105)

In the UTF-8 version of boesp.all_ansi.txt, the number-letter coding needs to be converted to UNICODE.
A listing of full titles for the abbreviation LS references needs to be developed.  Does the author
provide such a list in boesp print?  If not, the PWK list can be used to help develop such a list for boesp.

I prefer  to drop the  tag and tag the literaray references after correcting  and
Boehtlingk does not provide a list of LS because, using the LS in  of boesp presupposes a knowledge of Sanskrit
and (therefore) access to pwk.

to drop the tag and tag the literaray references after correcting

Sounds fine to me.

pushed corrected boesp.all_ansi.txt

@thomasincambodia  -- I find that latest commit (commit hash starts with ffa74d729, per git log)
has removed all the <ls>, as expected.

@funderburkjim,
There is one odd-man ls word RA120JANi1TI at <F>3737, which should've been RA1JANI1TI.

Boehtlingk does not provide a list of LS

I see that all the three Indische Spruche tomes, as also the two Melange supplements, do mention the referred works (in the beginning pages).
[Of course, as usual everywhere else, there could be some additional sources not explicitly mentioned in the books, but cited inside.]

The file boesp_auth_v0.txt contains the abbreviations of referred works as listed in
volumes 1-3.
I don't see similar lists in 4.1 and 4.2 pdfs -- What  source names from these two Melanges Asiatiques supplements should be added to the boesp_auth_v0 file?


OLS-440: New config option: Postgres cache max-entries

Description
New config option: Postgres cache max-entries
Type of change

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

Related Tickets & Documents

Related Issue #OLS-440
Closes #

Checklist before requesting a review

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

/retest

/lgtm

/approve

/retest


Add the host tag to RDS instances' parsed tags

What does this PR do?
This is a follow-up to: https://github.com/DataDog/integrations-core/pull/7353. A previous conversation on that PR (now removed due to a force push) suggested there may be a billing impact to adding the host tag to metrics. At the time, it was removed for investigation. I confirmed that host tags on metrics are not used for metering and will not impact host counts, so it is safe to add this tag back.
Motivation
This tag will allow customers monitoring RDS instances to filter by host:{rds_instance} and group by host. Currently the tag set sent with metrics are:
["host:agent_hostname", ...other tags]

With this change, the tag set will be:
["host:agent_hostname", "host:rds_hostname", ...other tags]

Additional Notes
If approved/merged, I will need help to cut a new release so postgres/mysql integrations can leverage this new functionality.
Review checklist (to be filled by reviewers)

[x] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[x] PR title must be written as a CHANGELOG entry (see why)
[x] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[x] PR must have changelog/ and integration/ labels attached

@yzhan289 any other blockers to merge + release? The CI failures look unrelated to my changes.

@yzhan289 any other blockers to merge + release? The CI failures look unrelated to my changes.

@justiniso nope it looks good to me otherwise.

@justiniso nope it looks good to me otherwise.


Move office hours to engineering handbook

Handing office hours over to engineering to manage going forward.
This makes Office Hours explicitly owned and managed by Engineering. Likewise, Demo Day will be explicitly owned by Marketing/Community going forward. I'll update process and procedures for Demo Day as I refactor them.
cc @aaronsteers @tayloramurphy @DouweM @emilylucie

I've also deleted the existing Office Hours meeting so that a new one can be created. The Zoom link needs to be updated so that someone else owns it and in the past we've had trouble updating that link. Folks on outlook don't get the updated URL so it's best to re-create the invite.

I've updated this to remove conflicts and fix a few links. I'm going to merge this as it gets us closer to the current state. Engineering has been owning Office Hours for a few weeks now 👍


Adds a setting for default lane size

This PR adds a setting for default lane size. It can be found by going to Edit -> Preferences.... We need to find a way to persist this. Note this setting does not change per-lane widths as those should be handled differently.

Will fix these however in another conversation with @mxgrey he mentioned the following crate: https://github.com/ForesightMiningSoftwareCorporation/bevy_polyline
This removes the need for this feature. Will probably use this instead.

Closing in favor of #152


RegisterEndpointNotificationCallback not working

I am trying to create a simple command line program that will detect whenever there is an event of a microphone plug/unplug.
You can see the code here:
https://github.com/hershkoy/headset_detector/tree/naudio
For some reason, this code only works on 1 out of 4 computers that I tried.
On the computer where it doesn't work, it does detect the endpoints, but there are no events:

What could be the issue?

Eventually it kinda worked for me after playing with "realtek HD manager" application. It only detected the back panel jacks, and not the front panel.


Can you please solved the following issues on android and IOS also?

Error are for android:
WARNING: The specified Android SDK Build Tools version (23.0.1) is ignored, as it is below the minimum supported version (28.0.3) for Android Gradle Plugin 3.5.1.
Android SDK Build Tools 28.0.3 will be used.
To suppress this warning, remove "buildToolsVersion '23.0.1'" from your build.gradle file, as each version of the Android Gradle Plugin now has a default version of the build tools.

> Task :react-native-braintree-xplat:compileDebugJavaWithJavac

Attribute application@allowBackup value=(false) from AndroidManifest.xml:11:7-34
	is also present at [:react-native-braintree-xplat] AndroidManifest.xml:14:9-35 value=(true).
	Suggestion: add 'tools:replace="android:allowBackup"' to <application> element at AndroidManifest.xml:7:5-117 to override.

I have set the validation false on android and IOS both, for get the response.
Changes are:
Android: .validate(false);
IOS: card.shouldValidate = No;
I have solved this issue is manually but its not a better way. Validation is not working.

@harleenarora why do you make PR from https://github.com/wkoutre/react-native-braintree-xplat. I don't really understand what you trying to archive.
You can make fork of my repo and make your changes. Then do PR to this repo I'll check it.

Can you please update the library?

what do you mean?

I mean, this library is very old. Can you update this.

No, I'm not going to do it in the near future. Try to use https://github.com/wkoutre/react-native-braintree-xplat I think it more up to date.


Please add to release notes CLI version like "50.1.1 (October 17, 2020) - CLI 7.76.1"

Is your feature request related to a problem? Please describe.
When updating sfdx I see version number like 7.76.1, when I am reading changelog I see version number 50.1.1 (October 17, 2020). I am confused because I don't know if it is the same version.
Additionally there are minor updates of cli that are not mentioned in changelog at all.
What are you trying to do
Clearly identify which changes relate to which CLI version.
Describe the solution you'd like
I would like to see in changelog description like 50.1.1 (October 17, 2020) - CLI 7.76.1.
Describe alternatives you've considered
Alternative is match version number of CLI to 50.x.x schema. As VS Code plugins are already versioned in this way.
Additional context
N/A

Great idea, thanks!  Let me huddle with the team and pick the best option.

We now include the sfdx-cli version with each week's release notes (see here).  Thx for the suggestion!

Thank you :)


Shell checker: Show error and read more button

Changes
Handles #1337.

Adds error messages (stderr) to shell error modal
Show 'Read more' button to open docs page with more info regarding this error

Checklist

[x] have tested my change
[x] updated relevant documentation
[ ] Remove any/all console.logs I added
[ ] eslint is not complaining
[ ] have added myself to the contributors' list in CONTRIBUTING.md
[ ] for feature PRs: PR only includes one feature enhancement.

@chrjorgensen
Your suggestion is more fitting and I've made a change so the shell check only looks at the the stdout (as it should). We already have logic to warn the user if their home directory is bad!

another possibility could be used an SQL query to get user profile info, something like this:
SELECT USER_NAME, HOME_DIRECTORY, PASE_SHELL_PATH  FROM QSYS2.USER_INFO WHERE USER_NAME = 'USER_NAME'
if the HOME_DIRECTORY field is not empty so check the directory path value exist

@angelorpa That's a little bit out of context for this PR, but thanks for the idea nonetheless!


Fixes divide by zero panic

Closes #268

Codecov Report

Merging #272 into master will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #272   +/-   ##
=======================================
  Coverage   61.35%   61.35%
=======================================
  Files          68       68
  Lines        6345     6345
=======================================
  Hits         3893     3893
- Misses       1936     1937    +1
+ Partials      516      515    -1

Impacted Files
Coverage Δ

document/value.go
72.99% <100.00%> (ø)

engine/memoryengine/store.go
91.96% <0.00%> (-1.79%)
:arrow_down:

sql/parser/select.go
65.57% <0.00%> (+1.63%)
:arrow_up:

Continue to review full report at Codecov.

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

@gfelixc we need to update TestValueMod/double(>_maxint)%int64(100) test case in document/value_test.go.

@gfelixc we need to update TestValueMod/double(>_maxint)%int64(100) test case in document/value_test.go.

@tie done


fix: log to stdout should not be rotated

If ssdb config is set to write logs to stdout and the rotate option is set, it won't write logs after a certain amount of the logs are written (due to the rotation).
So, when stdout is set as log output, it should not rotate the log file.

Hi, thanks for your report. But your code is not performance best, so this pull request will not be merged. I will make an update soon.

Update has been made on dev branch.


How to use prototype to classify close-set?

Thanks for your intersting job. In the paper,i find "We find that the close-set classification performance of the latent space is not as well as the traditional softmax classifier. So after identifying known and unknown objects using the latent space constructed by PLN, we apply a softmax classifier on all known objects to classify them into each known category." I want to use the former for closed set classification, what modifications should I make?

You can compute the distances between the proposal feature and prototypes of all known classes. Then classify the proposal into the class with the minimum distance.

I think base on your question, you should change the eval_type to "Closeset" instead of "openset" .


Support user specific CMAKE_MODULE_PATH

What problem does this PR solve?
Issue Number: resolve #2805
Problem Summary:
What is changed and the side effects?
Changed:
Side effects:

Performance effects(性能影响):

Breaking backward compatibility(向后兼容性):

Check List:

Please make sure your changes are compilable(请确保你的更改可以通过编译).
When providing us with a new feature, it is best to add related tests(如果你向我们增加一个新的功能, 请添加相关测试).
Please follow Contributor Covenant Code of Conduct.(请遵循贡献者准则).

I would say to keep the quotation marks as it  also covers the case of PROJECT_SOURCE_DIR containing white spaces.


Remove example clients on the project that are not working properly.

We have an examples folder with a bunch of server projects that doesn't work properly, we should remove that code from the core base.

Done: https://github.com/stateful/runme/pull/307


Cannot load Kubernetes extension; error specifies that it failed to run kubctl

I am unable to load Kubernetes v1.3.15 extension in VS Code v1.85.1 on Windows 10.
I am able to access the kubectl / kubectx / kubens  commands in terminal or command prompt but extension keeps reporting that it is uanble to run kubectl:

I have reviewed the location of the configuration and can see all the neccessary files and able to execute them from command line:
"vs-kubernetes": {
        "vscode-kubernetes.helm-path.windows": "C:\\Users\\rrozinov\\.vs-kubernetes\\tools\\helm\\windows-amd64\\helm.exe",
        "vscode-kubernetes.minikube-path.windows": "C:\\Users\\rrozinov\\.vs-kubernetes\\tools\\minikube\\windows-amd64\\minikube.exe",
        "vscode-kubernetes.kubectl-path.windows": "C:\\Users\\rrozinov\\.vs-kubernetes\\tools\\kubectl\\kubectl.exe",
        "vs-kubernetes.kubeconfig": {
            "nodeType": "context",
            "nodeCategory": "kubernetes-explorer-node",
            "contextName": "docker-desktop",
            "kubectlContext": {
                "clusterName": "docker-desktop",
                "contextName": "docker-desktop",
                "userName": "docker-desktop",
                "active": true
            }
        },
        "vscode-kubernetes.helm-path-windows": "C:\\Users\\rrozinov\\.vs-kubernetes\\tools\\helm\\windows-amd64\\helm.exe",
        "vscode-kubernetes.minikube-path-windows": "C:\\Users\\rrozinov\\.vs-kubernetes\\tools\\minikube\\windows-amd64\\minikube.exe"

In the output log of the Extension, I am not seeing any extra logging errors or even information level log entries.
When I try to add an existing cluster (it does not even see docker desktop one), I get an error saying that I am missing azure cli (which does exist and works well with aks).

Digging a bit deeper in Developer console in the VS Code:
I see the following:
[Extension Host] *** failed to invoke kubectl (from C:/ProgramData/chocolatey/bin/kubectl.exe

I am able to execute kubectl / helm / minikube in the configured locations in the terminal.  How would I troubleshoot the extension further to understand the underlying error message.
Also during the startup, VS Code also reports the issue I linked #1149  where minikube update does not succeed.

After much back and forth with configuring the path, I ended up blowing away the AppData/Roaming/Visual Studio Code/** location and reinstalling code from scratch.  That seems to fix the issue.


'% test' slf4j-simple

I'm still seeing slf4j-simple pulled in (https://github.com/mwunsch/handlebars.scala/issues/48). I think this should fix it.

Good catch; thanks!


Add range option to getGalleryData

Let me think again, we already have getGalleryIdList function.
If we use those together, we don't need that.


feature: add streaming response in production env.

I buid  app, The chat box cannot dynamically display one word at a time.   but in run dev  ,it work
When there is too much content,  the chat box is black and waiting for a long time

Thanks for the feedback.
I am aware that in production the response is not being streamed as of right now. I am working on this.
The other issue of the chatbubble being black, can you elaborate on that issue? Is it only when the content is too much?

I mean is that this is the same question.  The process of AI generating articles word by word is not present, and users can only wait for all the content to be generated before seeing the results.

This, is something that was bugging me as well for https://github.com/jakobhoeg/nextjs-ollama-llm-ui/issues/50; no showstopper though.
It probably has to do with the post-processing of the LangchaingJS stream in React.

@jakobhoeg Any idea when this will be worked on? This would probably bring the most benefit to users from the current state. If you host ollama on a slow machine and worse without graphics card, the speed can be very slow. With streaming at least you see that something is happening.

@jakobhoeg Any idea when this will be worked on? This would probably bring the most benefit to users from the current state. If you host ollama on a slow machine and worse without graphics card, the speed can be very slow. With streaming at least you see that something is happening.

I've been very busy with other stuff these past few weeks, but I'll try to make it a priority and take a look at it this weekend


[FR] Add possible to translate description of the apps in easy way.

In may opinion good idea will be make a script which be copied manifest of the apps in the flathub to weblate of the server of a flathub (or something else). It makes possible to translate the apps in the flathub in easy way.

Most of the app descriptions are already translated (in the appdata files present in the source code o of the apps) but the database is not ready (yet) to store them.
Localization of flathub.org is something I would like to implement this year but I don't have an estimation about when it will be done

Isn't this covered by the Translations button? I don't think anything else is feasible.


Add create2 helper methods

computeCreate2Address(bytes32 salt, bytes32 initCodeHash) returns an address using the default create2 deployer
hashInitCode(bytes, bytes) and hashInitCodeNoConstructorArgs(bytes) make it more convenient to figure out the init code hash in deploy scripts

upstreaming these little helper methods I found useful in our internal deploy scripts (https://github.com/showtime-xyz/showtime-contracts/blob/main/script/utils/Create2Helper.sol)
cc @mds1

Thanks for this! Just a few nits on naming and stuff, curious to get your thoughts

The suggestions make sense, thanks for the fresh pair of eyes!

suggestions implemented in d5255d0

Side note: note sure what's up with the tests taking 46m in one of the CI runs? I was able to repro locally, seems like tests were running normally and then forge would hang out at 0% CPU with all threads waiting.
I can't repro anymore and neither does CI, it suggests something related to the network?

Yea seems related to fork tests, since subsequent runs are a lot faster by pulling from the cache instead. We have a lot of public RPCs in the CI to test the default RPCs in StdChains, so guessing they get throttled and it's waiting to retry requests. Not certain though, will need to look into it


Blacklisting Fake Dependencies

There are many basic system files that will trivially differ on all sorts of machines that are probably not relevant for most projects. We should have a blacklist of such common system files so they don't pollute configs (and it could be user-configurable in case there are various project-specific ones, or maybe one of these "fake dependencies" is actually relevant for that particular application)

done (default blacklist could be expanded of course)


Add hideOnMove option to allow map to move without hiding context menu

Allows a user to provide an option to the constructor to disable hiding the context menu when the map is moved, either through code or dragged by the user.
A default value of true is provided to preserve current behaviour.

@GastonZalba are you willing/able to review this PR? I'm not sure about how this should be handled.

Hello @jonataswalker. First, this Pull Request has nothing to do with the problem/feature-request initially reported in issue #263, it is not going to fix what was described there so it should not be related (the issue should be remain open even if this is merged).
Either way, it adds new functionality with only a small code change. But i think that this addition could be used to accommodate some small things in the code that this PR interacts with (the this.mapMoveListener attribute is declared and not used, and the movestart event is not cleared in the removeListeners function).
On the other hand, if setting this new parameter is intended from the constructor only, probably checking how it is set should be before setting the mapMove listener at the setMap method, and not every time the event is fired (and doing nothing if disabled). If it is intended so that the configuration can be changed even after the instance is created, perhaps it would be nice to specify this in the README: the ability to modify the option with instance.options.hideOnMove = true;, for example, or even better, create a setHideOnMenu method.
Sorry for my english :(


Is this action used to extract image metadata from the DockerHub registry, or from Git?

I'm confused by this action's documentation. It says that it's possible to extract 'tags' and 'labels', but is unclear whether the tags are Git tags (from the Git repository, or tags of remote Docker image stored in the registry.
Does this action allow you to extract metadata from the DockerHub registry?

Seems unlikely. Based on my observations, tags are taken from the tags of the git repo. Eg - if v1.0 is the tag in git repo, docker image will be pushed to Docker Hub using v1.0. If  you don't pass any tag, the tag used in Docker Hub will the name of the branch you use in the Github Actions.
Hope this helps. Please let me know if you have any more questions.

@theo-brown Yes we are using Git ref and GitHub events to generate tags and labels.

Please could this be made explicit in the README?


fix condition for writing enum values to elasticsearch enums index

WHAT:  fix bug with EnumValidator attempting to write to elasticsearch with non-enum metrics if ElasticSearch cluster is down/unreachable.
WHY:  the result from cassandra retrieving enum values return in a non-null object with size 0, and current code only tested for NULL result.
HOW:  modify the condition to check for both non-null and size > 0 for enum values query of a metric from cassandra, if neither is true then it's a no-op for write to ElasticSearch.

@tilogaat @goru97 please review when you have a chance

+1


Fix: incorrect subscribing event name to navigate in cluster view

Originated from #2330
Signed-off-by: Roman<EMAIL_ADDRESS>

@ixrock Are you able to also change src/renderer/components/cluster-manager/cluster-actions.tsx?


KAFKA-12947: Replace EasyMock and PowerMock with Mockito for StreamsMetricsImplTest

Based on the Replace EasyMock and PowerMock with Mockito Jira ticket:

Development of EasyMock and PowerMock has stagnated while Mockito continues to be actively developed. With the new Java cadence, it's a problem to depend on libraries that do bytecode generation and are not actively maintained. In addition, Mockito is also easier to use.

Since the original PR #10881 (KAFKA-12947 Replace EasyMock and PowerMock with Mockito for Streams…) author is no longer accessible, this PR is created to fix what was started more than a year ago.
Each test change has a separate commit  (which can make it a little bit easier to review).
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@ableegoldman, @cadonna could you please help review this changes in the streams test domain?

@cadonna, @clolov can you please help me get this PR merged (if/when you have time, ofc)?

Hello @dplavcic and @cadonna, how is this pull request fairing? Is there something outstanding as a review or comments to be addressed?

@clolov , apologies for a delayed response. I unfortunately don't have time to finish this anytime soon  😞

Hello @dplavcic and @cadonna, how is this pull request fairing? Is there something outstanding as a review or comments to be addressed?

@clolov , I will work on this during the next two weeks. Apologies for a delayed response.

@dplavcic It would be nice to get this over the line. Are you still willing to work on this?

It's also worth mentioning that this is the last remaining Streams test that uses PowerMock.

I will reopen this PR once changes are implemented. I'm currently working on this.


*Partyline should use &#channels instead of ~#channels

Many clients are having problems with partyline using ~#channels (for example: IRSSI, WeeChat, Yaaic). Messages send to those channels appear as private messages to those clients. This is a problem with mobile IRC clients which vibrate on private messages.
We tested changing partyline to use &#channels and all messages sent to partyline appeared normally to channel. This is because most (if not all) clients hardcode & to CHANTYPES (or something what the server sends).

Its more common for networks to use & themselves, so this might cause an issue where you can't connect to channels on a server. The bitlbee server uses & channels if I remember correctly. Perhaps the prefix should be configurable instead of changed, so that users can change the prefix if they wish to.

BitlBee server uses &.
I agree with you that it should be user configurable.
I can try to test does this break & channels.

Tested.
IRCd: charybdis-3.3.0
How I tested:

I connected to network which supports &channels with two connections from which other was direct and another with ZNC which used &#channels with partyline.
I joined both connections to &test and both clients saw each other.

Result; it doesn't break & channels.

Tested with &#Hi and this is where problems came in.

Joined with ZNC to &#hi with server which supports &channels.
Joined with ZNC to &#hi with server which doesn't support &channels.
Joined client to &#hi with direct connection on the server on step 1.

Result:
Step 1 joins to &#hi on server, partyline is not accssible.
Step 2 joins to &#hi on partyline. Client on step 1 is not visible.
Step 3 joins to &#hi on server, and clients on step 1 and 3 see each other.
Result: this is a problem.

No, this is more weird. Client in step 2 appears to partyline to client on step 1 and they can discuss with each other, but client in step 2 can't see client 1 in names list. The direct connection can't see any messages which are sent from ZNCs which are on partyline, but client on step 2 can see what client with direct connection says.

I was thinking that partyline doesn't make as much sense on the newer ZNC with networks, since you have multiple networks to connect with. I think that it would make more sense to have a dedicated connection to the ZNC server for partyline, where it is completely isolated from any IRC server, then we can use #chans.
Since on the git version of ZNC, you can connect to the user and not a network, we could use partyline as user-only.

That sounds like good idea. If it was different connection, would bufferplayback work? (Issue #21).

I brought this issue up a looong time ago on sourceforge, and in response they made it possible to change the prefix character for the channels. I personally use +#channel, which is considered valid by my irc client, while ~# is not (which I also reported as a bug for the client, I believe). To get this, just set lines 14-16. As far as I know, you don't need the hash (#) mark for a valid channel if it has another prefix, but there's no real reason to take it out or change it. Keep in mind you'll have to change this every time you update the partyline module.
It would also be much better if the partyline channels weren't stored with the prefix at all, and it was only shown to the client--this way you could change it at will, and not have to worry about being unable to join the correct partyline channel because the prefix is set incorrectly; currently if you join +#efreak and then change the prefix to ~#, everyone in +#efreak stays there seeing +#efreak, and users will be unable to join/part the old channel.

Whoops, I mixed the issue numbers up