Add support for mDNS registration

In order to interoperate with services like Home Assistant, it would be good if a device running AtomVM could register services it is running via mDNS.
Proposal:
Augment the current network interface to allow configuration of mDNS entries, when the device is configured in STA mode.
This change just requires the addition of an optional field in the STA configuration.  If the mDNS configuration is not present, mDNS is not initialized.  Otherwise, mDNS is initialized, as each entry in configuration is added to the internal mdns server.  Note that this list may be empty, though at present there would be no use case for this.  (In the future, we may want to add an API for mDNS query, but this issue only addresses registration).
Here is the suggested type specification:
-type sta_config_property() ::
    ssid_config()
    | psk_config()
    | dhcp_hostname_config()
    | mdns_config(),   %% <-- new
    | sta_connected_config()
    | sta_disconnected_config()
    | sta_got_ip_config().

-type mdns_config() :: {mdns, [mdns_entry()]}.
-type mdns_entry() ::
    [
        {service_name, string()}, %% optional
        {protocol, tcp | udp}, %% required
        {port, port_number()}, %% required
        {txt_data, [{string(), string()}]} %% optional
    ].

Note that this proposal does not support modification to mDNS entries after initialization.  A future API could be crafted to support this.

This would be a great improvement! We should make it an option in AP mode as well, in case the user wants to start a private network, but be able to reach the host via mdns.
I think eventually we might want to break out a separate module to add more mdns features, in addition to query like adding or removing services and delegate hosts after the network has been started.


Closed Count Report Displaying Incorrect Expected Count Data and Blank 'acceptedByUser' Field

Current behavior
The expected count data is incorrect for some count IDs.
The 'acceptedByUser' field appears blank for the affected count IDs.
Expected behavior
The expected count data in the closed count report should be accurate.
The 'acceptedByUser' field should display the name of the user who accepted the count for each count ID.
Steps to reproduce the issue
Navigate to the closed tab and download the closed count report.
Check the expected count data for specific count IDs.
Observe that the expected count data is incorrect.
Check the 'acceptedByUser' field for the same count IDs.
Observe that the field is blank, even though a user should be listed.
Environment details
App version : v2.1.0
App environment : UAT
Additional information
CycleCounts-19 Sept 2024, 09_20_26.csv

Issue related to expectedQuantity is fixed now. The acceptedByUser (InventoryCountImport.uploadedByUserLogin) is not getting set when uploading counts using csv because counts are created by system.
@dt2patel The acceptedByUser user should be the one who accepted that item, should be add a new field in InventoryCountImportItem for this? Currently we have countedByUserLoginId and createdByUserLoginId fields in InventoryCountImportItem entity.

Depends on Maarg and OMS both - Related PRs are merged on 3rd December 2024.

Verified the change in the Closed Count report: acceptedByUserLogin is now used instead of uploadedByUserLogin.

ISSUE: In the Closed Count report, the lineStatus is not accurately displayed; for rejected items, the status is shown as "Completed."


[Bug]: 创建项目时返回Parameter error

Preflight Checklist

[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the issue tracker for an issue that matches the one I want to file, without success.
[X] I am not looking for support or already pursued the available support channels without success.

Version
1.15.0
Installation Type
Official Docker Compose
Service Name
DongTai-Web
Describe the details of the bug and the steps to reproduce it
新安装的1.15.0，使用./dtctl install -v 1.15.0。安装完毕后仅改了个密码。
创建项目返回错误

Additional Information
No response
Logs
No response

reinstall latest docker and docker-compose v1 can fix this problem.
https://docs.docker.com/compose/install/standalone/
./dtctl install -v 1.15.0


Update past Envoy 1.10

[ ] Pick an appropriate commit
[ ] Rebase the Ambassador code onto it (all, y'know, six commits outstanding, I think?)
[ ] Make sure tests pass

This is the first of a series. We should do these every few weeks. Since @LukeShu is doing the update to 1.10, either @gsagula or @containscafeine should take this one.

I can take this one if no one is taking it.

I'd like to mention that the following bug is fixed in envoy after 1.10 https://github.com/envoyproxy/envoy/issues/6190
I'd love to see this included in Ambassador's build.

@containscafeine, I'd like you to take this one so that we continue spreading the Envoy knowledge around. @gsagula, you can take the one after that (or I will, we'll sort it out :smile:).

@gsagula I have the PR for forwarding to v1.10.0 at https://github.com/datawire/envoy/pull/10 but there are some failures there as I've mentioned there. Can you check?

Actually wait, things have changed ...

(Assigning to @gsagula because it's required for the fix for #1595...)


fix wps process properties (#522)

This PR fixes issue #522.
Fixes parser for WPS properties (statusSupported, storeSupported).

test failures not related to this PR.


JMX exporter: quoted key property values captured incorrectly

Object names with quoted key property values are being incorrectly captured by regular expression defined in the JMX exporter configuration.
Current behavior
The quotes and the value enclosed by the quotes are being captured.
Expected behavior
Only the value enclosed by quotes should be captured.
Example of Prometheus metric after name object name property has been converted to processor label:
org_apache_camel_ExternalRedeliveries{context="MyCamel",processor="\"log1\"",type="processors",} 0.0

See file with more examples: metrics.txt

PR https://github.com/fabric8io-images/s2i/pull/194 fixes this issue


Remove promxy from CAPA

Towards https://github.com/giantswarm/roadmap/issues/3218 this PR removes Promxy from CAPA.
⚠ Do not merge until all CAPA has been migrated to Mimir
Checklist

[x] Update changelog in CHANGELOG.md.

Did we also remove Promxy as a datasource from Grafana and set Mimir as default ?

No that's why it's a do not merge :D

Ok let's wait then


Do not create IAM roles if no instances are created

For the moment, it is not possible to add a count to a terraform module.
When we do not want it to be created under a certain environment, we can now specify instance_count to 0 and nothing will be created.

Why would you want to use this module and set it to 0?


Not able to recover the error message originated from failwith.

Hello everyone,
I am trying to invoke my contract with conseiljs.TezosNodeWriter.sendContractInvocationOperation().
Everything works fine with a positive scenario but when I purposely fail the script with some error message like "not enough balance".
All I get in try-catch is  :
Error: failed: (temporary: proto.006-PsCARTHA.michelson_v1.runtime_error), (temporary: proto.006-PsCARTHA.michelson_v1.script_rejected)

It just says that your script failed because it hit failwith somewhere but I want the message.
Is there any way to do it ?
I was not able to find anything in the documentation.
Thanks in advance.

Sounds like the contract rejected your transaction for one reason or another. I would expect that it's a FAIL or FAILWITH in the code of the contract itself.

Hi @anonymoussprocket
Yes, I know it is failing, that's what the logic is supposed to do.
The problem is I am not able to recover the error code returned by the FAIL or FAILWITH  statement.
for eg: its returns "31".
I am not able to find any way in conseilJS to do so.
It will be great If you know if you can point me to the answer.
Thanks.


Precision recall curve problems

Hi,
I use the metric standard like you said but i still can not get the curve like you did.
I don't know what's wrong with it and maybe i misunderstood something.
Here is my metrics code
iou_list means each proposal generated from each action from one image, gt_masks is the same one in your code, and confidence is the qval of each action from one image.  precision_recall_per_img is the list of p-r value of size 50.
`def precision_recall_curve_threhold(iou_list,gt_masks,confidence):
threhold_list = np.arange(0,5,0.1)
precision_recall_per_img = []
TP_FN = float(gt_masks.shape[2])
a = zip(confidence, range(len(confidence)))
a.sort(key=lambda x: x[0])
index = [x[1] for x in a[::-1]]
for i_th in range(len(threhold_list)):
    proposal_num = 0
    flag = [0] * int(TP_FN)
    cls = [-1] * len(index)
    # caculate proposal_num
    for i in index:
        if confidence[i] > threhold_list[i_th]:
            proposal_num += 1
        else:
            break
    # caculate per threhold
    # object
    if proposal_num!= 0:
        for i_ob in range(int(TP_FN)):
            max=0
            max_id = -1
            # proposal
            for j in index[:proposal_num]:
                if cls[j] != 1:
                    cls[j] = 0
                if iou_list[j][i_ob]> 0.5 and iou_list[j]>max:
                    max = iou_list[j][i_ob]
                    max_id = j
                if max_id != -1:
                    cls[max_id] = 1
                    flag[i_ob] = 1
        TP = float(cls.count(1))
        FP = float(cls.count(0))
        TP_r = float(flag.count(1))
        precision = TP / (TP + FP)
        recall = TP_r / TP_FN
        precision_recall_per_img.append((precision, recall))
    else:
        precision_recall_per_img.append((-1,-1))

return precision_recall_per_img`

When each time i get the whole test precision_recall value on the one test image. I sum the precision and recall of all test images on each threshold.
for i in range(50):
               if precision_recall_c_new[i][0]!=-1:
                   sum_p[i] += precision_recall_c_new[i][0]
                   sum_r[i] += precision_recall_c_new[i][1]
                   count[i] += 1

And then do mean like this:
 for i in range(50):
        if count[i]!=0:
            f1.write(str([float(sum_p[i])/count[i],float(sum_r[i])/count[i]]))
            f1.write('\n')

Am i wrong in calculating curve ?

Hi!
The confidence of each bounding box that I considered is not the qvalue for the the action that generated that bb, but the qvalue of the trigger action in that moment. I don't know if you are doing this.
Miriam

Hi !
Sorry for replying late. I tried many times. At first, I use wrong confidence, then i trans to qval of trigger action in that moment. It improved. But still not ideal. I wonder to know if you take all iou>0.5 as TP or one ground truth only has one proposal and take that as tp. Here is the precision-recall text i got. The first line is precision, and the second is recall. The threshold is increasing from top to bottom.
[0.22499416433239963, 0.2314542483660131]
[0.22682072829131653, 0.2265522875816994]
[0.221260458206271, 0.22151067323481122]
[0.22318398623817343, 0.22151067323481122]
[0.2238075297120523, 0.2209380234505863]
[0.26063829787234044, 0.22056737588652484]
[0.25732600732600736, 0.21547619047619052]
[0.2740963855421687, 0.21696787148594376]
[0.2901678657074341, 0.2267386091127098]
[0.3089080459770115, 0.23836206896551726]
[0.3157894736842105, 0.25771929824561407]
[0.3227848101265822, 0.2550632911392404]
[0.32575757575757575, 0.26111111111111107]
[0.3157894736842105, 0.2570175438596491]
[0.30851063829787234, 0.2457446808510638]
[0.3125, 0.23874999999999996]
[0.3088235294117647, 0.23676470588235296]
[0.35714285714285715, 0.26964285714285713]
[0.375, 0.27291666666666664]
[0.36363636363636365, 0.28863636363636364]
[0.3333333333333333, 0.24166666666666664]
[0.25, 0.146875]
[0.2857142857142857, 0.16785714285714287]
[0.4444444444444444, 0.2611111111111111]
[0.5, 0.29375]
[0.5, 0.29375]
[0.42857142857142855, 0.19285714285714287]
[0.25, 0.0625]
[0.25, 0.0625]
[0.3333333333333333, 0.08333333333333333]
[0.0, 0.0]
[0.0, 0.0]

From all the regions selected during one sequence, I analyzed if any of them had more than 0.5 intersection over union with any object of the ground truth, and that is a TP, so one gt object only has one TP.


Use a new hooks dir struct

This is a broken change. It will change the hooks directory structure.

LGTM

go-gitea/gitea#1006 has a migration for this broken.

Please don't merge until go-gitea/gitea#1006 is reviewed and ready for merge :)


Fix dataloader usage in keyvadapter package

The DataLoader constructor expects a function which returns (effectively) a Promise<Array<V>>. Right now, as we have it, our function returns a Promise<V>[] (array of Promises) instead.
Not sure why TS didn't catch this (something to do with dataloader's use of PromiseLike and ArrayLike that means our existing function somehow conforms to the signature?).
Fixes #166
TODO: add changeset and test reproduction

Huh, my changes triggered the same exact bug in the batching test.

Resolved upstream via https://github.com/jaredwray/keyv/pull/432


Generator using code_builder

A work in progress, generating code is safer with less boilerplate!

@Ali1Ammar Generators are now implemented more elegantly, with less boilerplate code and more maintainable = 🎉


Getting missing AccZ GyY values from calibration

Getting this from Calibration mode
13:58:48.156 -> 88 loops / 4 axis calibrated. Missing:  AccZ GyY
13:58:48.361 -> 89 loops / 4 axis calibrated. Missing:  AccZ GyY
...
Nope I am not on a roller coaster.
[Edit]
14:09:33.081 -> 298 loops / 4 axis calibrated. Missing:  AccZ GyY
14:09:33.286 -> 299 loops / 4 axis calibrated. Missing:  AccZ GyY
14:09:33.487 -> 300 loops / 4 axis calibrated. Missing:  AccZ GyY
14:09:54.087 -> 1 loops / 1 axis calibrated. Missing: AccX AccY AccZ GyX GyY
14:09:54.289 -> 2 loops / 4 axis calibrated. Missing:  AccZ GyY
14:09:54.462 -> 3 loops / 4 axis calibrated. Missing:  AccZ GyY
just loops after 300, it seems to want to tell me what is flat and whats not rather than using what it's got as flat and working from there.

Hi @foxabilo, thanks for posting!
As I've realized, in some architectures calibration may not converge (in my ESP32, for example) with the default deadzones. As a workaround, you can set new deadzone values and check if it'll converge.
For my ESP32 I'll be using 0.01 m/s² for accelerometer and 0.06 degrees/second for the gyro (defaults are 0.002 m/s² and 0.015 degrees/second, respectively).
Let me know if this helps!

Ahh! Yes I am running on a WiFi KIT 32 ESP32 with the little oled from HELTECH, I'll try the new values and see how it does, I kind of need calibration to be as good as it can get and then just run with what its got as having the calibration cycle potentially stop a remote unit from booting and doings its initialisation functions would be undesirable.

Did you mean to half the values? that would make it more sensitive I think, I have changed the values to 0.04m/ss and 0.03d/s and it now goes up to 5 axis calibrated, but the Z remains stubborn, I will see at what value it passes.

#define DEFAULT_ACCEL_DEADZONE          0.011 // m/s² #define DEFAULT_GYRO_DEADZONE           0.021 // Degrees/second
Those are the absolute minimum values I can use for it to pass calibration

Those are the absolute minimum values I can use for it to pass calibration

I'm glad you found some acceptable values for your application!
Yet, if you need to change these values later, you don't have to actually look for it on the header file. There are methods for that, as below:
void SetAccelDeadzone (float deadzone)
void SetGyroDeadzone (float deadzone)

Results will be the same, but it makes your code more intuitive.

Most of the issue is on the Z axis

Yeah, I realize that. The reason for this is that calibration for the Z axis is quite different from the others because of the offset (gravity). I had a hard time while trying to make Z axis to work and probably there's a better way of doing this but it works!

Indeed, perhaps a completely separate Z deadband?

perhaps a completely separate Z deadband?

Yeah idk, maybe as something optional. Great idea, I'll consider implementing this on the next releases. Thank you for the feedback!
If you need help with anything else, let me know!


Could this player play video only using youtube url ?

[ ] I have verified there are no duplicate active or recent bugs, questions, or requests
[ ] I have verified that I am using the latest version of Easy Video Player.
[ ] I have given my issue a non-generic title.
[ ] I have read over the documentation (before asking questions on how to do something).

Include the following:

Easy Video Player version: x.x.x.x
Device OS version: 7.0
Device Manufacturer: Huawei
Device Name: Nexus 6P

Also, please wrap Java with correct syntax highlighting.
System.out.println("Hello, world!");

Reproduction Steps

Expected Result
Actual Result

If you get the URL to the actual file.

This is one of those things that isn't really an issue that is quickly solved by the Library Dev @vsingh-techracers
When you have a youtube URL, IE: https://www.youtube.com/watch?v=7tzaWOdvGMw
That isn't the direct video url. What you need for this to work is the direct video URL of the actual file to view. his can be obtained or converted through conversion programs
(IE, http://www.clipconverter.cc/)
Which uses regular expressions to reverse engineer the URL:
https://r5---sn-25ge7ney.googlevideo.com/videoplayback?mn=sn-25ge7ney&mm=31&key=yt6&ip=<IP_ADDRESS>&pl=24&signature=9ED66FE84101E037F0B2B0EB62862E1F410E9401.16B9E1285535F9DFE16A2F51D242C7A1CB479E9F&initcwndbps=2082500&mv=m&mt=1482261751&ms=au&ipbits=0&requiressl=yes&sparams=dur,ei,id,initcwndbps,ip,ipbits,itag,lmt,mime,mm,mn,ms,mv,nh,pl,ratebypass,requiressl,source,upn,expire&dur=518.989&id=o-AGFZ88_MQjEXyA1Yx2PXJr5vCff6pHRSewQkptvmi4-H&ei=QIZZWLzYIsy7cIjwn5gK&itag=22&nh=IgpwcjAxLnBhcjEwKgkxMjcuMC4wLjE&mime=video/mp4&expire=1482283680&lmt=1471447373855172&upn=u6BaqyZe434&source=youtube&ratebypass=yes&&title=Why+We+Haven't+Cured+Cancer
(Which is significantly longer). The short answer (TL;TR) is, if you want it to work with youtube videos, you will need a way to convert the youtube URL string to the actual video String and use that. Keep in mind though that Google changes this every once in a while and your code may or may not work all the time.


ChartQA link is no longer accessible

Hello, thanks for your great work!
There seems to be a minor problem that the google drive link you provided for ChartQA images (and maybe eval_chartqa.py and annotations as well) is no longer accessible. Would you please check it out and renew the link when you are available, thanks!

Maybe you can try to download the ChartQA dataset on huggingface and extract the images.


WASM doesn't render correctly on Mac OS X

Testing on OS X revealed that the WASM runs and correctly sends logs to the console, but the OpenGL render box itself is a black screen with nothing rendered! It's very odd, there were no shader compilation errors or panics - the GL layer thinks that it is running.
This may take some time to fix; I only have sporadic access to a Mac to test on.
See also: https://github.com/thebracket/rustrogueliketutorial/issues/33

I did a bunch of reading, and found that lots of people have problems with WebGL2 on OS X - including quite a few Unity users hitting the exact same bug (black screen). See the branch, webgl2-osx for work on this.
Based on my reading, the patch ( c3d1481ca98199e4c75351e3eb4d4cb50eae2402 ):

Downgrades shaders to version 100 from version 300 es.
Removes the need for all extensions.
Tweaks the framebuffer creation (WASM only to avoid problems with AMD on Glutin) to use a GL_UNSIGNED_BYTE backing buffer.
Tweaks the framebuffer mode to use GL_NEAREST and not the various GL_LINEAR modes.

Downgrading to version 100 and tweaking the shaders to use the old syntax still ran on Windows, but the console gave a "GL invalid format" error, which looked like a big clue. So I performed the other steps, and now it runs well on my phone (Android, both Firefox and Chrome) and laptop (Windows 10, Chrome and Firefox).
I'll test as soon as I get access to a Mac. If anyone reading this has a Mac available, could you give these a go, please?
http://bfnightly.bracketproductions.com/wasmtest-old2/ex10/
http://bfnightly.bracketproductions.com/wasmtest-old2/ex01/

Hearing from friends with Macs that this didn't help.

Well, I've fixed a bunch of OTHER little bugs in the WebGL pipeline, and got it down to 1 pass with no framebuffer. Finally got a VM Mac working (slowly), so I can now see/replicate the error. Will keep hacking.

It's not committed or working for all the examples yet, but I've made some pretty significant progress. This ended up expanding into "this really could be better code".
So the new pipeline:

Uses a single shader to render consoles.
Each console is one draw call of a single quad covering the whole screen.
Each console also updates a couple of textures containing character number, and colors (tiny dynamic textures, 80x50)
The shader now calculates the console position, samples the console buffer and font buffers and renders the console in a single pass.

I've got to get background colors, offsets and the post-processing in there - but so far, so good.
The following should give you example 1, including on a Mac (it works on my VirtualBox version):
http://bfnightly.bracketproductions.com/wasmtest-old2/ex01/

Woot! I still have to figure out the mipmap warning, but all the demos are now working on my virtual Mac.
http://bfnightly.bracketproductions.com/wasmtest-old2/ex01/
(replace the 01 with 01 - 17 to see them all)

Merged branch to master. Going to pull it into the roguelike tutorial and rebuild to test there also, so far so good.

I believe it is working! :-)


Update graphql-config to version 4

Description
This PR picks up from @dahukish previous PR to upgrade graphql-config to version 4.
Fixes (issue #)

Type of change

Could be major, however the main breaking change in graphql-config 4 was dropping support for node 10, the previous major release of these packages dropped support for now 12. There is a minor loader type change.

[x] graphql-typescript-definitions Minor: New feature (non-breaking change which adds functionality)
[x] graphql-config-utilities Minor: New feature (non-breaking change which adds functionality)
[x] graphql-validate-fixtures Minor: New feature (non-breaking change which adds functionality)

Checklist

[ ] I have added a changelog entry, prefixed by the type of change noted above (Documentation fix and Test update does not need a changelog as we do not publish new version)

Let us know if you have any feedback on the new version or the migration process that we could help with!
cc @B2o5T @kamilkisiela

/snapit


Add me as a contributor?

Hi Chris,
there might come a moment in my copious amounts of spare time where I'd like to try out perl6 implementations of phylogenetic trees, nodes and taxa. Might as well do this as part of bioperl6, so if you trust me enough not to make too much of a mess, may I have push access?
Thanks,
Rutger

Why certainly :).  Will add you to the team

You should have push now.  I may move this over to the main bioperl org at some point but will transfer everyone there if that happens.

Thanks, pushing works.


Bump to 3.40.0

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


20151208...md: reduce screenshot width

Desktop version of page has 800pix width, mobile version has 500pix width.
Set screenshot width=250pix to use < 1/2 page width.
ref #17

half-width is 320 pix


Stop() Method is missing.

I have installed TopShelf 4.0.3 via Nuget packages.
Seems Stop() method has been renamed as  Dtop() by mistake.
 x.Service<WebServer>(sc =>
                {
                    sc.ConstructUsing(name => new WebServer());
                    sc.WhenStarted(c => c.Start());
                    sc.WhenStopped(c => c.Dtop());
                });

That Stop() method is in your WebServer class, not Topshelf.


Install go-plus instead of go-config and go-get

As part of https://github.com/joefitzgerald/go-plus/pull/503, go-config and go-get will be unpublished. go-plus now provides the go-config and go-get services.
This PR should be merged and released when v5.0.0 of go-plus is published, which should occur on the 19th December at 4:00 pm UTC (https://www.timeanddate.com/worldclock/meetingdetails.html?year=2016&month=12&day=19&hour=16&min=0&sec=0&p1=75).

As of go-plus v5.0.7 only the 2.0.0 service for go-get exists. Can you update the PR please?


Export CTDirect functions

Do not forget for the next release to export useful functions from the CTDirect.jl package.

@PierreMartinon In OptimalControl.jl, do we export all the useful functions from CTDirect.jl?

@PierreMartinon In OptimalControl.jl, do we export all the useful functions from CTDirect.jl?

The new tutorials will help to check this. The ones for initial guess and continuation cover the most common uses, I guess we only have the NLP/DOCP handling left. I can start a basic tutorial with just function calls, upon which @jbcaillau can expand.

New exports can be seen in the draft PR https://github.com/control-toolbox/OptimalControl.jl/pull/203


Remove sharepointids from delta query $select params

We are not making use of GetSharepointIds() anymore. It was removed a month back, see PR. So we don't need to fetch this per item-property from graph during delta queries.
This will reduce overall mem usage since we don't need to store references to the sharepoint id map. It can also make delta queries go slightly faster since we are transferring & processing fewer bytes over network.

Does this PR need a docs update or release note?

[ ] :white_check_mark: Yes, it's included
[ ] :clock1: Yes, but in a later PR
[x] :no_entry: No

Type of change

[ ] :sunflower: Feature
[ ] :bug: Bugfix
[ ] :world_map: Documentation
[ ] :robot: Supportability/Tests
[ ] :computer: CI/Deployment
[x] :broom: Tech Debt/Cleanup

Issue(s)

#

Test Plan

[x] :muscle: Manual
[x] :zap: Unit test
[ ] :green_heart: E2E

This PR sounds like it's alright to merge because we're not using this field, but we need to be really careful about changing delta query parameters because delta tokens encode them. Just merging this doesn't mean that all new backups we do will use the new set of parameters. If we do an incremental backup with an older token it may still fetch this parameter


Fix import statement in selection_example.py

BTB version: 0.2.2-dev
Python version: Python 2.7.10
Operating System: Mac

Description
Import error occurred while running selection_example.py.
Problem: wrong file to import from
What I Did
python selection_example.py
Traceback (most recent call last):
File "selection_example.py", line 8, in
from btb.selection import Selector
ImportError: cannot import name Selector

Closed by #91


Disable CIS benchmark report reconciliation by default

Towards https://github.com/giantswarm/giantswarm/issues/22754
Checklist

[x] Update changelog in CHANGELOG.md.
[ ] Make sure values.yaml and values.schema.json are valid.
[ ] (Giant Swarm) If creating a release, bump the version and appVersion in Chart.yaml.

Tested in WC


cargo-make: 0.35.16 -> 0.36.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/sagiegurari/cargo-make/releases.
meta.description for cargo-make is: "A Rust task runner and build tool"
meta.homepage for cargo-make is: "https://github.com/sagiegurari/cargo-make"
meta.changelog for cargo-make is: ""
Updates performed

Rust version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.36.0 with grep in /nix/store/r6yq4ab19f6iy26pk2sxmfwz9fgc60p7-cargo-make-0.36.0
found 0.36.0 in filename of file in /nix/store/r6yq4ab19f6iy26pk2sxmfwz9fgc60p7-cargo-make-0.36.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cargo-make

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/r6yq4ab19f6iy26pk2sxmfwz9fgc60p7-cargo-make-0.36.0 \
  --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 cargo-make https://github.com/r-ryantm/nixpkgs/archive/d7dade46c61caf018b85731c817c5577b73acc01.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/r6yq4ab19f6iy26pk2sxmfwz9fgc60p7-cargo-make-0.36.0
ls -la /nix/store/r6yq4ab19f6iy26pk2sxmfwz9fgc60p7-cargo-make-0.36.0/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.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cargo-make

Maintainer pings
cc @xrelkd for testing.

Result of nixpkgs-review pr 188931 run on x86_64-linux 1

1 package built:

cargo-make


Homemade Masks shows up twice in filter list?

For whoever picks this up, something is wrong with formEnumLookups.js. Something here I think...


Docker improvements

Hi guys,
I changed the Dockerfile to first copy and install all dependency related code, so when rebuilding the Dockerfile it doesn’t need to install them again. I also added the copying of the lockfile to prevent some other versions getting installed in the Dockerfile.

Ohhhhhh, good optimization @fabianriewe!
Thank you!


Issue with generating multiconformer_model2.pdb with example 3K0N

Hi,
I setup the github repository in my location machine (Ubuntu 22) by cloning the repository and following the instructions in the README file.
After which, I rand the code with the example files provided (qfit_protein example/qfit_protein_example/3k0n_map.mtz -l 2FOFCWT,PH2FOFCWT example/qfit_protein_example/3k0n_refine.pdb).
The code generated each residue in a separate directory A_i. However, at A_7, the code gives a warning [A/PHE7] Too many conformers generated (17100). Splitting QP scoring and the tqdm bar does not progress from there. Similar problems happen with multiple cycles.
Is there a reason why the code does not work with the examples provided on my end?
Please let me know if you need any additional details.

Hi @sai-advaith,
We apologize. There is an error with oversampling aromatic residues in the main branch.  We are hoping to update the main this week, but I would suggest using the dev branch in the meantime, as that error is fixed there.

I see. Has the error been fixed in the main branch?
Also, I am currently working with the dev branch and it works fine. So, thanks for the suggestion!
I wanted to run the code with my own data. In the paper, your team mentioned that the input density files are converted to an absolute scale, i.e., to electrons per Angstroms cube units.
In my experiment, I am using a 2fofc ccp4 file along with the protein structure from rcsb (with resolution provided). Where in the codbase is the density grid of 2fofc file converted to an absolute scale? I noticed that this only happens in FFTransformer, which is used for MTZ inputs. Could you please let me know where the conversion to absolute scale take place for ccp4 inputs.
Thanks!

Hi,
Thank you for your reply.
I tried setting up both, dev and main branch on my local machine a few days ago. After that, I tried to generate 5 alternate conformers in residue 113 of 3K0N's chain A using the command from README: qfit_protein qfit_protein_example/3k0n_map.mtz -l 2FOFCWT,PH2FOFCWT qfit_protein_example/3k0n_refine.pdb --residue A,113
I get similar outputs from both branches: [A/PHE113] Too many conformers generated (17100). Splitting QP scoring.
Do you know a branch and hash commit I can use to effectively generate appropriate alternate conformers using the files in the example/ directory?
Thanks!
Additional information regarding the packages:

python version: 3.10
Numpy version: 1.22
cvxpy version: 1.5.2
Pandas version: 2.2.2
pyscipopt version: 5.1.1
Scipy version: 1.14.0


Missing 2.8.0 release

The 2.8.0 release is missing on GitHub: https://github.com/icl-utk-edu/magma/tags

Thank you for reporting this.
The 2.8.0 tag is added. Note that tags differ from the official releases posted on MAGMA's home page here (https://icl.utk.edu/magma/). A little background below:
With very few exceptions, MAGMA is primarily developed in double-complex precision, and relies on a 'precision generator' to generate the other three precisions (single, double, and single-complex). The official releases on MAGMA's homepage are tarballs that contains all the source files for all four standard precisions. If you download directly from GitHub, the MAGMA build system will run the precision generator first before compilation. Both are basically the same thing, but with an extra step if you use the tags.

I understand the difference, we would just like to move from https://bitbucket.org/icl/magma.git to github for the source in the Arch Linux package. How is the migration going? I hope future tags will appear on github early :smile:

The migration is complete. We no longer use bitbucket, which will be deleted soon. I don't know how the 2.8.0 tag was dropped during migration, though.


Add walrus operator

Change Summary
Use walrus operator that removes extra lines.
Checklist

[ ] The pull request title is a good summary of the changes - it will be used in the changelog
[x] Unit tests for the changes exist
[x] Tests pass on CI
[ ] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

We still support 3.7, so can't use walrus operators.

Ok. but I fixed the typo too. :(

Ok. but I fixed the typo too. :(

Please update your PR to contain only the typo fix.

Imho, walrus operator decreases code readability by making separate statements one-liner. It makes you stuck more on if statements (I mean not in this case, but in general)

@mo1ein Do you want to update your PR and keep the typo fix? Otherwise, I will close the PR

Time waster or bot.


jira comment return success, but comment isn't posted

Running
jira comment STH-100 "Some comment" yields a Comment to issue [STH-100] was posted!. but the comment doesn't show on Jira, and the comment count on jira show [STH-100] doesn't doesn't change

I am experiencing the same thing, is it something to do with permissions.  I'm the admin so I should be able to do this?

Same issue here

If you are still facing the issue please feel free to reopen the issue, and we'll solve it together : ) .


Update EndpointRouter.cs

Fixed spelling mistake in log message

Hi @ErazerBrecht, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

thanks!


Update _fluent.scss

Recommending some changes to the fluent type scale based on what we have in our Indigo.Design UI kit for Fluent, which closer to what MSFT has. Andrea did not have access to make a branch.

I believe @sdimchevski must approve this first. Before we can merge and release any changes introduces in this PR, a new version of the UI kits with the changes made here must be released, or at least released at the same time.

@ig-georgeA great suggestion for these changes. We are working on Indigo.Design UI kit for Fluent in Figma now and we'll update these font-weights and letter-spacings. @simeonoff we should sync on the release as we have a way to go with the Figma kit and no resources for Sketch at the moment

@sdimchevski What's the status of the changes introduced in this PR with regards to the UI kits? Is it safe to merge it?

Closing as the typography was revised by Svilen's Team and is inline with what we have in the UI kits.


eslint errors

Link #1461 (fix eslint warnings)

max-len: most of the sensible max-len errors are fixed
no-unused-vars: most of these types of errors are fixed by deleting not needed declarations
@typescript-eslint/no-unused-vars: same as above

Tests pass locally.

🤔 why are the tests failing in CI environment? My local env was

node v14.11.0
pnpm 5.10.1

🤔 why are the tests failing in CI environment? My local env was

node v14.11.0
pnpm 5.10.1

hard to know, you might go one rule at a time.

Tests seems to pass now but Code scanning/CodeQL has errors. May be we should look at fixing them in other issues.

Tests seems to pass now but Code scanning/CodeQL has errors. May be we should look at fixing them in other issues.

It is fine, we have enabled them only for master, the idea is use that info to improve code in 5.x eventually, but for now still to early. Thanks for pointing this out.

Maybe. Before merge you want to use the changeset feature I am testing, check the contributions the contributions guideline.

Hi @juanpicado
Just to confirm, you want me to run pnpm changeset on the branch?

Hi @juanpicado
Just to confirm, you want me to run pnpm changeset on the branch?

yes, you run pnpm changeset , follow the instructions, add this changes as a patch, add your fancy comment, all that info will be visible in the changelog, you can tag yourself if you like, then commit and push, then I will merge.

From 198 warnings to 78 :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap:


Fix incorrect .gitignore contents, fixes #2180

The Problem/Issue/Bug:
#2180 points out that the generated .ddev/.gitignore has the wrong files gitignored in .ddev (.ddev-docker-compose-*)
How this PR Solves The Problem:
Make it right.
Manual Testing Instructions:
Automated Testing Overview:

Related Issue Link(s):
Release/Deployment notes:

I'll add a test to this:

git init in a fresh directory
ddev config in a fresh directory
git add . - the only new "new file" should be .ddev/config.yaml
ddev start
Check for untracked files; there should be none.

Sounds like a good way to go. I'll add that it is essential, that the ddev config used, contains all options that result in the generation of additional files.


feat: mad-core now adds username to tracking after authenticating

@HavardNot We do the same in mad-expo-core
setUsername adds real name to main log, and obfuscated name to long term log


[LANG-763] Reintroduce DateUtils.UTC_TIME_ZONE constant:

add the reusable DelegatingTimeZone and ImmutableTimeZone classes
add back the DateUtils.UTC_TIME_ZONE constant with some unit tests

Wie yoda-time and the java.time package introduced in Java 8, do we still need this?

I raised this issue originally, but I don't need it anymore.


Dispose method is not called on objects registered for the TestThread container

Reqnroll Version
1.0.1
Which test runner are you using?
MSTest
Test Runner Version Number
n/a
.NET Implementation
.NET 8.0
Test Execution Method
Visual Studio Test Explorer
Content of reqnroll.json configuration file
No response
Issue Description
If an object/class is registered to the test thread container (via TestThreadContext.TestThreadContainer), that implements IDisposable, it is expected that the Dispose method will be called latest before the test execution is finished.
But this will not happen, because Reqnroll does not dispose the test runners at the test-run end (TestRunnerManager.FireTestRunEndAsync).
Steps to Reproduce
See repo/workaround at https://github.com/reqnroll/Sample-ReqOverflow/blob/main/ReqOverflow.Specs.WebUI/Support/BrowserFactory.cs#L61
Link to Repro Project
https://github.com/reqnroll/Sample-ReqOverflow/blob/main/ReqOverflow.Specs.WebUI

Is it expected? The behaviour with IDisposable has always been a bit fuzzy, but the rules I understand is that if you create the object, you are responsible for its lifecycle and calling Dispose() when done. If the container creates the object, it should be responsible for its lifecycle. If the container was given an instance of the object, it should not dispose the object unless it was given an explicit instruction to take ownership of the lifecycle.

@Code-Grump Yes, thx for the clarification. I was not clear enough. The problem is that if the container creates the object (so you just have a type registration) the Dispose will not be called. I'll change the description to make this clear.


Zero values for day low and open

When using the below code, I am getting zero values for day low and open.
entry_date = datetime.datetime(2024, 4, 19)
end_date = entry_date + timedelta(days = 1)
entry_date_str = entry_date.strftime("%Y-%m-%d %H:%M:%S")
end_date_str = end_date.strftime("%Y-%m-%d %H:%M:%S")
index = "CNXBAN"

hist_cash = breeze.get_historical_data_v2(interval="1day",
                                        from_date= entry_date_str,
                                        to_date= end_date_str,
                                        stock_code= index,
                                        exchange_code= "NSE",
                                        product_type="cash")

if isinstance(hist_cash, dict) and hist_cash['Success']:
    hist_cashdf = pd.DataFrame(hist_cash['Success'])

Dear Client,
We are looking into it and will update you shortly.
[E29CF836]
Thanks & Regards,
Hiten Solanki,
Team Breeze API,
ICICI Securities

From: thelearner04 @.>
Sent: Saturday, August 24, 2024 8:06 PM
To: Idirect-Tech/Breeze-Python-SDK @.>
Cc: Subscribed @.***>
Subject: [Idirect-Tech/Breeze-Python-SDK] Zero values for day low and open (Issue #131)
CAUTION: This email originated from outside the organization! Do not click links, open attachments or reply, unless you recognize the sender's email address! Use 'Report suspicious Email' button in the toolbar to report Phishing mails. Beware!
When using the below code, I am getting zero values for day low and open.
entry_date = datetime.datetime(2024, 4, 19)
end_date = entry_date + timedelta(days = 1)
entry_date_str = entry_date.strftime("%Y-%m-%d %H:%M:%S")
end_date_str = end_date.strftime("%Y-%m-%d %H:%M:%S")
index = "CNXBAN"
hist_cash = breeze.get_historical_data_v2(interval="1day",
from_date= entry_date_str,
to_date= end_date_str,
stock_code= index,
exchange_code= "NSE",
product_type="cash")
if isinstance(hist_cash, dict) and hist_cash['Success']:
hist_cashdf = pd.DataFrame(hist_cash['Success'])
—
Reply to this email directly, view it on GitHubhttps://github.com/Idirect-Tech/Breeze-Python-SDK/issues/131, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3TQZ464TMWHCKH62YVW7KLZTCK5PAVCNFSM6AAAAABNBUUWY2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ4DINRTHE4TSOI.
You are receiving this because you are subscribed to this thread.
Print this mail only if absolutely necessary. Save Paper. Save Trees." "The information contained in this e-mail and any attachments to this message are intended for the exclusive use of the intended recipient and may contain proprietary, confidential or legally privileged information. If you are not the intended recipient, please note that you are not authorised to disseminate, distribute or copy this e-mail or any parts of it or act upon/rely on the contents of this e-mail in any manner. Please notify the sender immediately by e-mail and destroy all copies of this e-mail and any attachments. Please also note that ICICI Bank or its subsidiaries and associated companies, (collectively "ICICI Group"), are unable to exercise control or ensure the integrity of/over the contents of the information contained in e-mail transmissions and that any views expressed in this e-mail are not endorsed by/binding on the ICICI Group unless the sender does so expressly with due authority of ICICI Group. Before opening any attachments please check them for viruses and defects and please note that ICICI Group accepts no liability or responsibility for any damage caused by any virus that may be transmitted by this email. Any unauthorized use of this email or any attachment(s) may be treated as infringement of right(s) including right of privacy under the applicable laws of India including Information and Technology Act, 2000. Thank you for your cooperation. Regulatory Disclosures are available on https://www.icicisecurities.com (for institutional business) and www.icicidirect.com (for retail business).

Dear Client,
Please check now, as low and open value are now coming in the API response.
[E29CF836]
Thanks & Regards,
Hiten Solanki,
Team Breeze API,
ICICI Securities

From: Hiten Solanki /EPG/ISEC/NAVI MUMBAI @.>
Sent: Monday, August 26, 2024 2:33 PM
To: Idirect-Tech/Breeze-Python-SDK @.>; Idirect-Tech/Breeze-Python-SDK @.>
Cc: Subscribed @.>
Subject: Re: [Idirect-Tech/Breeze-Python-SDK] Zero values for day low and open (Issue #131)
Dear Client,
We are looking into it and will update you shortly.
[E29CF836]
Thanks & Regards,
Hiten Solanki,
Team Breeze API,
ICICI Securities

From: thelearner04 @.>
Sent: Saturday, August 24, 2024 8:06 PM
To: Idirect-Tech/Breeze-Python-SDK @.>
Cc: Subscribed @.***>
Subject: [Idirect-Tech/Breeze-Python-SDK] Zero values for day low and open (Issue #131)
CAUTION: This email originated from outside the organization! Do not click links, open attachments or reply, unless you recognize the sender's email address! Use 'Report suspicious Email' button in the toolbar to report Phishing mails. Beware!
When using the below code, I am getting zero values for day low and open.
entry_date = datetime.datetime(2024, 4, 19)
end_date = entry_date + timedelta(days = 1)
entry_date_str = entry_date.strftime("%Y-%m-%d %H:%M:%S")
end_date_str = end_date.strftime("%Y-%m-%d %H:%M:%S")
index = "CNXBAN"
hist_cash = breeze.get_historical_data_v2(interval="1day",
from_date= entry_date_str,
to_date= end_date_str,
stock_code= index,
exchange_code= "NSE",
product_type="cash")
if isinstance(hist_cash, dict) and hist_cash['Success']:
hist_cashdf = pd.DataFrame(hist_cash['Success'])
—
Reply to this email directly, view it on GitHubhttps://github.com/Idirect-Tech/Breeze-Python-SDK/issues/131, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3TQZ464TMWHCKH62YVW7KLZTCK5PAVCNFSM6AAAAABNBUUWY2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ4DINRTHE4TSOI.
You are receiving this because you are subscribed to this thread.
Print this mail only if absolutely necessary. Save Paper. Save Trees." "The information contained in this e-mail and any attachments to this message are intended for the exclusive use of the intended recipient and may contain proprietary, confidential or legally privileged information. If you are not the intended recipient, please note that you are not authorised to disseminate, distribute or copy this e-mail or any parts of it or act upon/rely on the contents of this e-mail in any manner. Please notify the sender immediately by e-mail and destroy all copies of this e-mail and any attachments. Please also note that ICICI Bank or its subsidiaries and associated companies, (collectively "ICICI Group"), are unable to exercise control or ensure the integrity of/over the contents of the information contained in e-mail transmissions and that any views expressed in this e-mail are not endorsed by/binding on the ICICI Group unless the sender does so expressly with due authority of ICICI Group. Before opening any attachments please check them for viruses and defects and please note that ICICI Group accepts no liability or responsibility for any damage caused by any virus that may be transmitted by this email. Any unauthorized use of this email or any attachment(s) may be treated as infringement of right(s) including right of privacy under the applicable laws of India including Information and Technology Act, 2000. Thank you for your cooperation. Regulatory Disclosures are available on https://www.icicisecurities.com (for institutional business) and www.icicidirect.com (for retail business).


Add simple query to work with common counter metrics

In conjunction with https://github.com/MicrosoftDocs/azure-docs/pull/39703 this sample query will help those who are new to KQL to quickly analyze custom counter data to view requests per minute

@vyta : Thanks for your contribution! The author(s) have been notified to review your proposed change.


[question]how to disable serial debug logging on orin nano

I would like to use the UART port /dev/ttyTHS0 as a normal serial output to other UART devices.
Those devices are currently acting strangely, which is making me think that serial logging is enabled. Serial logging on Jetson should be enabled with the kernel flag console=ttyS0.
I'm not sure what would be enabling console=ttyS0, or how I could disable it.
Any advice for how to disable serial logging?

Possibly related discussion at https://github.com/OE4T/meta-tegra/issues/84

There are some threads on the NVIDIA dev forum about this here and here.
Kernel console is normally routed to that UART via /dev/ttyTCU0 on the Orins. If I'm understanding correctly from the forum threads, you tweak the boot configuration to switch that serial interface be "normal" instead of "debug", and that will route the UART through /dev/ttyTHS0 instead, so you can use it for other things.

Thank you for the advice!! I am kinda in too much of a rush rn to be messing around with devicetree, but my question is resolved as there it will clearly work eventual with enough fiddling.
For future reference to others running into the issue, I used the adafruit USB to TTL Serial Cable to get a clean UART from /dev/ttyUSB0, which worked flawlessly in yocto by adding the kernel-module-cp210x package in IMAGE_INSTALL:append. I don't know if this is the best solution: it takes up a USB port, and I am having some issues with signal stability when running a 1k resistor from TX to RX (for TMC2209), but it worked first try and I haven't run into any issues yet.


amend load print

Add a final print to log at the end of init.lua to indicate the mod was loaded successfully. This idea was derived from other mods which are specifically aimed at MT servers.
This could be useful not only for player support (e.g. in the MT forum) but would certainly be useful for MT server admins using the CLI and reading their server logs.

Closing for the following reasons:

print does not log to debug.txt (what headless server owners likely use)
The "info" log level already logs all mods.
The chat command /mods lists them (also works with ncurses admin prompt, what server owners also could use)
Recent Minetest versions no longer start up when mod dependencies are not fulfilled. All specified mods must load.


Pass transaction ABIs to SignatureProvider when signing

Custom signature providers may want to use ABIs. For example, a signature provider may want to display a Ricardian contract to a user without going to the chain and getting it itself. In this PR, we pass ABIs to the signature provider during signing.
Prior to this PR, the API's transact() method (during action serialization) was already calling out to get ABIs for each action in the transaction. Now, we do that step before the actions are serialized and pass those ABIs along when signing. There's no added network overhead. In fact, in some cases, there is less overhead.
For example, prior to this PR, if a transaction had multiples of the same action and the given contract was not previously cached, the contract for the given action (and, as a result, the abi) would be fetched multiple times. Now, it's guaranteed to only be fetched once and is then cached.

Thanks, @chris-allnutt.
This PR simply provides a custom, offline signature provider with ABIs so that it can reference them in the transaction for on-chain verification purposes before packing and signing. Allowing service providers to BYO-ABI is a good idea for a future improvement, though. 👍

Cool i remember that being part of a conversation just wasn't sure if this was meant to solve it.  Changes look clean!
:+1:


Bulk load method silently fails

Code similar to the below never creates a batch. I've tried some variation of function signatures to no avail. Using the more verbose examples from here does create a batch. Hopefully this saves someone else time.
const accounts = [...];

const batch = conn.bulk.load('Account', 'update', null, accounts);
setTimeout(() => {
  console.info(batch.id); // undefined
}, 5000);

batch.on('queue', console.info); // never emitted
batch.on('error', console.error); // never emitted
batch.on('response', console.info); // never emitted

I'm going to close this, we have an example here showing how you can await it to get the records:
https://jsforce.github.io/document/#bulk-api
also tried the code above and got it working on jsforce v3. if someone finds a similar problem please log a new issue.


Implement Clap-based CLI argument parsingfeat

Hello,
I've submitted a pull request that implements Clap-based CLI argument parsing, enhancing the codebase by providing structured and user-friendly command-line options. This improvement makes it easier to configure and run the EKS token generation tool.
I'd appreciate your review and feedback on these changes.
Thank you!

Hey @wmBolles ,
Thanks for the PRs. I will review them once I finish work. Glad to see someone else is finding this codebase useful.
I will also look at getting it updated now that it isn't just for my own use.
Thanks,
Ali

So this PR is unfortunately not required. What you have done just replaced the clap derive based argument parsing with that of the builder variant which results in more code for the same result.
Please check the docs here: https://docs.rs/clap/latest/clap/_derive/index.html and the tutorial here: https://docs.rs/clap/latest/clap/_derive/_tutorial/index.html then you will see what I have done with the struct and the:
[derive(Parser)]
#[clap(author, version, about, long_about = None)]
```.

For reference this is what I get on the current code base:

❯ cargo run -- -h                                                                                region:eu-west-2
Finished dev [unoptimized + debuginfo] target(s) in 0.05s
Running target/debug/kashay -h
Usage: kashay [OPTIONS] --cluster-name <CLUSTER_NAME>
Options:
-c, --cluster-name <CLUSTER_NAME>  Name of the EKS Kubernetes cluster to get a token for
-R, --role-arn <ROLE_ARN>          ARN of a role to assume to be able to authenticate with the Kubernetes cluster
-s, --skip-cache                   Skip the cache and always get a new token
-r, --region               Name of the AWS region that the cluster is in [default: eu-west-2]
-h, --help                         Print help information
-V, --version                      Print version information

I do not see any other improvements hidden in the PR over the current code but if I have missed any please point them out and we can get them added in but for now I will close this PR as it seems to be a step down from the current derive based argument parsing.

Thanks for the PR though and please do keep them coming.

Hello @alisonjenkins .
Thank you for your feedback and for taking the time to review the pull request. I'd like to provide some additional context regarding the Clap-based CLI argument parsing changes.
While I understand your concerns about the increase in code size, the decision to use Clap was driven by the desire to provide a more standardized and user-friendly interface for configuring and running the EKS token generation tool. Clap is a widely adopted and well-documented library that simplifies command-line argument handling, making it easier for users to understand and use the tool.
By adopting Clap, we aim to improve the overall user experience and maintainability of the project, especially as it may be used by a broader audience. This change also aligns with best practices in Rust development.
I would appreciate your consideration of these factors before closing the pull request. If there are specific concerns or if there are ways to address the code size issue, I'm open to discussing and making further adjustments.
Thank you for your attention to this matter.
Best regards,
@wmBolles

Hello @alisonjenkins .
Thank you for your feedback and for taking the time to review the pull request. I'd like to provide some additional context regarding the Clap-based CLI argument parsing changes.
While I understand your concerns about the increase in code size, the decision to use Clap was driven by the desire to provide a more standardized and user-friendly interface for configuring and running the EKS token generation tool. Clap is a widely adopted and well-documented library that simplifies command-line argument handling, making it easier for users to understand and use the tool.
By adopting Clap, we aim to improve the overall user experience and maintainability of the project, especially as it may be used by a broader audience. This change also aligns with best practices in Rust development.
I would appreciate your consideration of these factors before closing the pull request. If there are specific concerns or if there are ways to address the code size issue, I'm open to discussing and making further adjustments.
Thank you for your attention to this matter.
Best regards,
@wmBolles

Hmm I don't think you understand. What we have there is already using Clap but the struct derive variant so this is already using the widely adopted solution.
It also provides equivalent functionality to what you provided in your PR using a fraction of the code as the code you wrote gets generated for us by Clap's derive macros from the comments on the struct and contents of the Cargo.toml.
Please re-read the comment I made and look at the documentation I linked which as you can see in the URL is a link to Clap's crate but instead of using the builder it is using derive.
I once did use this builder variant in my codebases but have been actively swapping to using derive as it results in less code which is easier to maintain.


"UnboundLocalError: local variable 'dshapes' referenced before assignment" when using quantize_net_v2 with calib_data=None

https://github.com/apache/incubator-mxnet/blob/2e33e961b6339f32680e38aa72ac28bc94215460/python/mxnet/contrib/quantization.py#L897
Pretty straightforward bug.  If calib_data is None then dshapes is never assigned and this line will throw an error.

Hi @jmracek thanks for reporting this bug. I've submitted a quick fix: https://github.com/apache/incubator-mxnet/pull/19501

@jmracek Fix for this Issue has been merged to v1.x branch. If that's fine for you, you can close this issue.


Unable to start the connector from a remote server

Hello,
I am trying to connect to a Kafka broker on a remote server over TCP. The Kafka server/zookeeper is running on a Server A and I am trying to start the adapter on Server B. The attempt is getting timed out with below error (For this testing we have opened all the traffic between the 2 servers A and B) :
[2019-07-09 14:22:03,168] ERROR Stopping due to error (org.apache.kafka.connect.cli.ConnectStandalone:122)
org.apache.kafka.connect.errors.ConnectException: Failed to connect to and describe Kafka cluster. Check worker's broker connection and security properties.
at org.apache.kafka.connect.util.ConnectUtils.lookupKafkaClusterId(ConnectUtils.java:64)
at org.apache.kafka.connect.util.ConnectUtils.lookupKafkaClusterId(ConnectUtils.java:45)
at org.apache.kafka.connect.cli.ConnectStandalone.main(ConnectStandalone.java:81)
Caused by: java.util.concurrent.ExecutionException: org.apache.kafka.common.errors.TimeoutException: Timed out waiting to send the call.
at org.apache.kafka.common.internals.KafkaFutureImpl.wrapAndThrow(KafkaFutureImpl.java:45)
at org.apache.kafka.common.internals.KafkaFutureImpl.access$000(KafkaFutureImpl.java:32)
at org.apache.kafka.common.internals.KafkaFutureImpl$SingleWaiter.await(KafkaFutureImpl.java:89)
at org.apache.kafka.common.internals.KafkaFutureImpl.get(KafkaFutureImpl.java:262)
at org.apache.kafka.connect.util.ConnectUtils.lookupKafkaClusterId(ConnectUtils.java:58)
... 2 more
Caused by: org.apache.kafka.common.errors.TimeoutException: Timed out waiting to send the call.
This works if I start the adapter on the server running the Kafka Server. Not sure why the remote connection is giving error.Could you please help here.
Regards

Hi,
When you started Kafka Connect using the connect-standalone.sh script you need to pass it a connect-standalone.properties file that points to the correct location of Kafka. Can you ping Server A from Server B? And is the address you use to ping Kafka from Server B the one you provided in the connect-standalone.sh script as the bootstrap.servers address?

Hi Katherine,
Regarding the first point : When you started Kafka Connect using the connect-standalone.sh script you need to pass it a connect-standalone.properties file that points to the correct location of Kafka.
I verified the connect-standalone.properties file on Server B and we have the correct IP address mentioned there for Kafka Server Installation(Server A) :
bootstrap.servers=<Public IP of the server running Kafka Server(Server A)>:9092
I am able to telnet the Kafka Server (Server A) IP and Port 9092 from the Server B (where I am trying to start the adapter) :
telnet  9092
Trying ...
Connected to .
Escape character is '^]'.
Regarding the 2-nd Point : And is the address you use to ping Kafka from Server B the one you provided in the connect-standalone.sh script as the bootstrap.servers address?
I don't see the attribute -bootstrap.servers in the connect-standalone.sh file. It's actually in the connect-standalone.properties file and is correctly set with the IP Address of Server A and port details(9092) asmentioned in the point 1.
In addition I noticed that the connector waits exactly for 2 mins before throwing the timeout error. The Adapter execution gets stuck at the below point :
:287)
[2019-07-10 12:09:54,130] WARN The configuration 'value.converter.schemas.enable' was supplied but isn't a known config. (org.apache.kafka.clients.admin.AdminClientConfig:287)
[2019-07-10 12:09:54,130] WARN The configuration 'value.converter' was supplied but isn't a known config. (org.apache.kafka.clients.admin.AdminClientConfig:287)
[2019-07-10 12:09:54,131] WARN The configuration 'key.converter' was supplied but isn't a known config. (org.apache.kafka.clients.admin.AdminClientConfig:287)
[2019-07-10 12:09:54,131] INFO Kafka version : 2.0.0 (org.apache.kafka.common.utils.AppInfoParser:109)
[2019-07-10 12:09:54,131] INFO Kafka commitId : 3402a8361b734732 (org.apache.kafka.common.utils.AppInfoParser:110)
Finally after 2 mins of time : we get the below exception :
ERROR Stopping due to error (org.apache.kafka.connect.cli.ConnectStandalone:122)
org.apache.kafka.connect.errors.ConnectException: Failed to connect to and describe Kafka cluster. Check worker's broker connection and security properties.
at org.apache.kafka.connect.util.ConnectUtils.lookupKafkaClusterId(ConnectUtils.java:64)
at org.apache.kafka.connect.util.ConnectUtils.lookupKafkaClusterId(ConnectUtils.java:45)
at org.apache.kafka.connect.cli.ConnectStandalone.main(ConnectStandalone.java:81)
Caused by: java.util.concurrent.ExecutionException: org.apache.kafka.common.errors.TimeoutException: Timed out waiting to send the call.
at org.apache.kafka.common.internals.KafkaFutureImpl.wrapAndThrow(KafkaFutureImpl.java:45)
at org.apache.kafka.common.internals.KafkaFutureImpl.access$000(KafkaFutureImpl.java:32)
at org.apache.kafka.common.internals.KafkaFutureImpl$SingleWaiter.await(KafkaFutureImpl.java:89)
at org.apache.kafka.common.internals.KafkaFutureImpl.get(KafkaFutureImpl.java:262)
at org.apache.kafka.connect.util.ConnectUtils.lookupKafkaClusterId(ConnectUtils.java:58)
... 2 more
Caused by: org.apache.kafka.common.errors.TimeoutException: Timed out waiting to send the call.
And there is one attribute which has a timeout set for 2 mins : request.timeout.ms = 120000.
On another note : Is this solution tested for any bulk load as well (say 1.5million-2Million transactions/day for a single queue) ? Just wondering if there are any numbers available.
Regards

I was finally able to make the connections from server B(by executing the adapter on Server B) to kafka installation on server A.  I was doing this on AWS EC2-Instances. For communication to go through, I put both of the servers in same security-group and allowed all TCP traffic on all ports in between them. Does this mean that there is some firewall requirement apart from 9092 port as well between Server A and Server B ?

Hi, I'm glad you got it to work. There isn't a firewall requirement built into Kafka for them to communicate, but it is possible that AWS was preventing traffic between them. Have you been able to run multiple brokers on different servers in the past and allow them to communicate, or even run an application talking to Kafka on a different server? Either of these scenarios would work the same as Kafka Connect trying to talk to Kafka.

Are you ok for us to close this issue as it sounds like it was a problem with the underlying infrastructure, not the connector?

Sure, before closing could you please update if this adapter has been load tested for any bulk volume.

We have indeed load tested the connector with a high volume of messages, however the exact performance you will get out of it depends on your setup. For example the underlying infrastructure, networking, where the connector is running etc.
I'll go ahead and close this issue now.


Pick-blocking Plutonium Ore Does Not Work

When I Pick-Block Plutonium Ore, I get a block without a texture called "tile.blockOre.name" with data value 33

Strangely enough, I just found the source of this issue - it's been around for some time...
Apparently, the pickBlock method looks for the metadata of the dropped item, rather than the block itself, so it ended up being a rather simple fix :)


frontend contrib

Pull Request Description
Changes:
Updated arbitrum.ts: Adjusted configuration for improved functionality.
Added .gitpod.yml: Introduced Gitpod configuration for easier onboarding and development setup.
Frontend Enhancement Contribution
Benefits:
Enhances Arbitrum performance.
Streamlines development environment setup for contributors.

The PR is not useful and the milestone is not relevant to Arbitrum.


chore: autoupdate tokens

What I did

add rhds tokens stylelint plugin
update linting rules and scripts

Testing Instructions

do some linting!

Notes to Reviewers
Closes #tokens

Thanks, @brianferry
I adjusted the lint rules to align with the token values. we can always reevaluate those preferences later.

we can always reevaluate those preferences later.

I think we can definitely iron out the details on the preferences.
Otherwise, I think this PR is great, I ran it on the whole project and targeted a specific component.  It found lots of spacing mistakes that wouldn't catch my eye otherwise.  This will definitely be handy to clean up formatting issues with the -fix flag, as well as more complex issues that would need more review like no-descending-specificity.

Great. If we merge today I can put together a PR that lints the files tomorrow, and we'll go from there


03/frontend no framework

Framework-less build example

can you please rename your folder to 3b_... thanks!


torrent magnet link?

Any plans on having a torrent magnet link as an option to download the gui and cli?

+feature

This issue is old and was discussed and closed on gitlab. Please reopen if you feel it's still relevant.


Support Radio Group widget

This is a variation of the Dropdown, so it ideally use the same model/controller/widgetState. The difference is each Radio widget can be put inside other widgets but still link to its siblings (meaning all the radio buttons are not necessarily wrapped inside a RadioGroup parent.
Column:
  children:
    - RadioButton
         groupId: myGroup
         ....

Column:
  children:
     - RadioButton
          groupId: myGroup

@vusters please also advise how to configure options for a radiogroup such as -
name
value
required

@vusters please see my comment above and describe the yaml for specifying options for radio groups especially validations such as required and how to get a value from a radioGroup

how is the selected radio value submitted with form submit or how does any code access it? is it by groupId? so is groupId available in js to call properties on? does each RadioButton have a name or id, how do I know if it is selected or not?

@hemish11 good one for you to take on. Feel free to look at the existing PR - https://github.com/EnsembleUI/ensemble/pull/842 or start over


First version of percent gauge

{{percent-gauge value=50 title="Pods" subtitle="210 of 440 used" ticks=ticks}}

Looks good, do you want to work on the cluster dashboard page?
3 gauges for cpu, memory, and pods (not storage, we don't have data):

Read from /v3-machines (filtered by clusterId)
requested on each machine is how much of that resource is "used", allocatable is the total allowed.  So if requested.cpu = 500m and allocatable.cpu = 940m, show 53%.
Add up all the requested and allocatables for each machine and divide to get the average for the cluster.  Separately save the smallest and largest percentage of any individual machine to show the tick marks.
While you're looping through machines, keep track of how many there are and how many are in active state for the next section.

On the right side, read from componentStatuses on the cluster and show 4 lines:

etcd (all componentStatuses with an id starting with etcd- have a condition of type=Healthy and status=True)
Controller (the componentStatus with id=controller-manager has healthy/true)
Scheduler (id="scheduler" ...)
Nodes (all the nodes you saw in the graph part are active)

At the bottom, if any of the conditions are not healthy, show a banner saying which thing is bad.  For nodes, show a banner for each node what is not active.

Sure. I will do it on Tuesday once I get a customer project done.

@vincent99 Do you mean /v3/clusters/${cluserId}/schemas/node ? There is an allocatable field in node schema. But there is no requested field there. So I am using some random value for the gauges.

/v3/machines entries should have allocable and requested:

@vincent99 done.

@vincent99 updated.


mesheryctl: Fix system start --reset to reset the meshconfig before s…

Signed-off-by: Amit Kumar<EMAIL_ADDRESS>Description
This PR fixes #2957
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Codecov Report

Merging #4166 (feaf0cb) into master (12e4d79) will decrease coverage by 7.73%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #4166      +/-   ##
==========================================
- Coverage   26.91%   19.18%   -7.74%
==========================================
  Files          59       59
  Lines        5031     5031
==========================================
- Hits         1354      965     -389
- Misses       3262     3771     +509
+ Partials      415      295     -120

Flag
Coverage Δ

e2etests
?

gointegrationtests
?

unittests
19.18% <0.00%> (ø)

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

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/start.go
0.99% <0.00%> (-53.98%)
:arrow_down:

mesheryctl/internal/cli/root/system/status.go
0.00% <0.00%> (-62.83%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
0.85% <0.00%> (-36.76%)
:arrow_down:

mesheryctl/internal/cli/root/system/update.go
1.05% <0.00%> (-34.74%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
1.78% <0.00%> (-32.15%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.39% <0.00%> (-22.38%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
0.00% <0.00%> (-21.60%)
:arrow_down:

Continue to review full report at Codecov.

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

yay, @amitech! 🎈

Thanks all for reviewing. Happy to see first PR approved. 😇

Same here. 😄


Broken URL CDN

Good evening, I have a project and use RECORDRTC, today in the morning I noticed that it showed an error when recording, in console it showed me that I could not find the following resource:
https://cdn.WebRTC-Experiment.com/RecordRTC.js
Apparently the problem arises when solving the dns.
I used another CDN service, but I report this problem because if one of you is using it, you will see that the problem persists.

Change the "cdn" to "www" and it should work. See the latest commits for more info.

I tried and now it works. At some point announced the change? It would be useful to know about the notifications and thus prevent an incident.
Thank you very much for the contribution


camerad: move camera exposure logic to CameraExposure Class

Reopened PR: https://github.com/commaai/openpilot/pull/33018
This PR refactors the camera exposure logic by encapsulating related functions and variables into a new CameraExposure class.
Unlike the previous PR (#33018), this refactor is done in place of the original code, except for adding a new header file camera_exposure.h for the CameraExposure class. This approach ensures that the logic remains unchanged, making it easier to review and verify that the functionality of camerad is not affected.
Next step:
Move the source code of CameraExposure's member functions into camera_exposure.cc.

trigger-jenkins


Support macOS

I think everything should already work on macOS. What we need to do:

Check it works (ideally on multiple systems)
Add wheel building to github actions
Check wheels work across OS versions
Add to documentation, setup.py etc.

Add dev instructions for zsh.

Add dev instructions for zsh.

https://www.gitmemory.com/issue/mu-editor/mu/852/498769312

Also install python from conda-forge on a mac seems to both speed up cellfinder and also allow napari to run on big sur.

On older os (e.g. <10.13) may need to install some packages from conda-forge. e.g. conda install -c conda-forge scikit-learn

Appears to work ok, but some issues:

Installation on older OSs (<10.13)
On one machine, issue with napari plugin & multiprocessing

Added in 463ffd7c94edd54de3dd011e147d0709a31dea3e


Update auth, dynamodb, s3, sqs to 2.17.114

Updates

software.amazon.awssdk:auth
software.amazon.awssdk:dynamodb
software.amazon.awssdk:s3
software.amazon.awssdk:sqs

from 2.17.66 to 2.17.114.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #810.


Monitor  CPU  and  Memory

I want to monitor CPU and Memory about zookeeper.How should i do?

You should be using something that collects OS level information, for example the node_exporter


Resolve Lint/ConstantDefinitionInBlock warning from Rubocop

When running tests this was causing output like:
/vendor/bundle/ruby/2.7.0/gems/snfoil-rails-0.8.1/lib/sn_foil/searcher.rb:21: warning: previous definition of ASC was here
/vendor/bundle/ruby/2.7.0/gems/snfoil-rails-0.8.1/lib/sn_foil/searcher.rb:22: warning: already initialized constant SnFoil::Searcher::DESC
/vendor/bundle/ruby/2.7.0/gems/snfoil-rails-0.8.1/lib/sn_foil/searcher.rb:22: warning: previous definition of DESC was here
/vendor/bundle/ruby/2.7.0/gems/snfoil-rails-0.8.1/lib/sn_foil/searcher.rb:21: warning: already initialized constant SnFoil::Searcher::ASC

https://www.rubydoc.info/gems/rubocop/RuboCop/Cop/Lint/ConstantDefinitionInBlock

Rubocop
Inspecting 80 files
................................................................................

80 files inspected, no offenses detected


Better integration with numpy

The functionalities of mpmath matrix class has overlap with the numpy ndarray class. But matrix class and numpy array class are not compatible. Currently, linalg functions such as qr, lu, norm can only work with mpmath matrix. It seems not difficult to change the code base to make these functions support numpy arrays.
It would be convenient if mpmath matrix and the relevant functions are compatible with numpy arrays. This allows users to use numpy array to manage mpmath.mpf objects, as numpy array has so many functionalities to manipulate array and matrix objects that mpmath matrix does not have.
Will the integration with numpy be considered in the future?

see also https://github.com/mpmath/mpmath/issues/217, https://github.com/mpmath/mpmath/issues/618

Technically, it's not that difficult to make mpmath aligned better with numpy, IMO. The question is the goal and the perspective of the package: whether numpy compatibility will be considered in the future releases. If so, I'm happy to make contributions. At beginning, it is not necessary to integrate every feature of numpy. Upgrading the existing functionalities does not look like too much work. It just needs to make the matrix class a subclass of numpy array, and refactor the relevant linalg and calculas functions following the numpy array APIs.

mpmath does not depend on numpy so it is not possible to subclass numpy's ndarray.
Adding an array API compatible module as an interface within mpmath could be reasonable though.

It may not be necessary to sublcass numpy array or install numpy. At some point, this NEP https://numpy.org/neps/nep-0041-improved-dtype-support.html#fixed-high-precision-math will be available in numpy. Then the numpy array can provide the functionalities of mp.matrix .
In mpmath, functions in the linalg module just need to follow the numpy API conventions. This will allow mpmath linalg functions supporting numpy arrays automatically. For example, one of the simplest change would be to replace the rows and cols in mp.matrix with an attribute shape. functions like eig, schur will automatically support numpy arrays.

In mpmath, functions in the linalg module just need to follow the numpy API conventions.

That does make sense.  Just keep in mind, that linalg functions and the matrix class are part of the mpmath's public API.  We should support old attributes for some time, add deprecation warnings, etc.


[Bug]: operator towhee/load-image not found!

Is there an existing issue for this?

[ ] I have searched the existing issues

Current Behavior
I followed the process of offical document of towhee DataCollection occured "operator towhee/load-image not found!" error when I run below codes.
dataset = DataCollection.from_glob('towhee.jpeg') \         .load_image() \         .random_resize_crop(224) \         .random_horizontal_flip() \         .cvt_color('RGB2BGR') \         .normalize() \         .towhee.image_embedding(model_name='resnet101') # redirect function call to towhee operators
Expected Behavior
No response
Steps To Reproduce
1. In the towhee env followed by offical document procedure
2. from towhee import DataCollection
3. run the previous code and occur the errors shown above

Environment
- Towhee version(e.g. v0.1.3 or 8b23a93):
- OS(Ubuntu or CentOS):ubuntu
- CPU/Memory:cpu
- GPU:
- Others:

Anything else?
No response

Recently, we changed some API name, some example has expired, we will fix it, thanks very much.


Disabling x11-ssh-askpass while still using Xserver

It appears that the SSH program module in nix, forces the use of an askpass program if xserver is enabled.
https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/programs/ssh.nix#L220
https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/programs/ssh.nix#L231
https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/programs/ssh.nix#L37-L41
If you wanted to not use a graphical askpass program while still using xserver, what would be right solution to this? Just set programs.ssh.askPassword to an empty string? In such a case, the environment variable is still defined, but just set to an empty variable, some non-ssh programs may rely on this variable and behave strangely if its set to an empty string.
I got caught out on this when trying to use git push on an XMonad workspace, and I couldn't see the graphical prompt, expecting just a terminal prompt. Eventually figured out that it had something to do with askpass when I noticed that sudo git push worked and didn't hang.

Absence of $SSH_ASKPASS may result in ssh launching inexisting but hardcoded ${pkgs.openssh}/libexec/ssh-askpass (#23500)

Doesn't SSH come with a standard terminal based password asking (how would it work in Cygwin if not)? I'm not aware if this is somehow meant to be ssh-askpass program, or it's part of SSH itself.

It is fallback when there is no terminal: https://github.com/openssh/openssh-portable/blob/V_7_5/readpass.c#L119
Actually, I too would prefer terminal input if there is a terminal available.
This means not only rejection to use $SSH_ASKPASS but also x11_ssh_askpass must be somehow passed to openssh as compile-time constant _PATH_SSH_ASKPASS_DEFAULT to ensure the fallback works in case of no terminal. I guess there will be opposition to make openssh dependent on xorg libs.

Perhaps unsetting $DISPLAY before running git would help: https://github.com/openssh/openssh-portable/blob/V_7_5/readpass.c#L119

You wrote this before:

It is fallback when there is no terminal: https://github.com/openssh/openssh-portable/blob/V_7_5/readpass.c#L119
Actually, I too would prefer terminal input if there is a terminal available.
This means not only rejection to use $SSH_ASKPASS but also x11_ssh_askpass must be somehow passed to openssh as compile-time constant _PATH_SSH_ASKPASS_DEFAULT to ensure the fallback works in case of no terminal. I guess there will be opposition to make openssh dependent on xorg libs.

In what situation would SSH be launched without a terminal?
If SSH were to be used in an automated manner, then I suspect that the automater should decide how to setup the SSH ASK PASS, and set it explicitly, or pass a password in using standard methods of interactive automation.

SSH could be launched without terminal, for example, when it is launched from GUI programs.
No, usually they do not take care of setting SSH_ASKPASS to their own password prompt tool nor even have such a tool.

This seems related to #24311

Is there some workaround for this problem? x11-ssh-askpass is really really awful.


Does stdin work with anyone?

Environment
Windows. C#.

Tesseract Version: v5.0.0-alpha.20200328
Platform: Windows 10 64bit.

Current Behavior:
stdin doesn't work for me.
process.StartInfo.Arguments = "stdin" + " stdout -l " + language + " --psm " + PSM_setting + " -c preserve_interword_spaces=1";
process.StartInfo.RedirectStandardInput = true;
process.StartInfo.RedirectStandardOutput = true;
process.Start();
//To write bmp to stdin
BinaryWriter writer = new BinaryWriter(process.StandardInput.BaseStream);
bmp.Save(writer.BaseStream, System.Drawing.Imaging.ImageFormat.Bmp);
//To read the result
StreamReader reader = process.StandardOutput;
string output = reader.ReadToEnd();//<-Hangs right here.
Expected Behavior:
...work?
Suggested Fix:
Does it work for others? How?

We do not support 3rd program. Replicate problem with tesseract executable to make sure problem is not in your code.

How do you replicate stdin problem by only using tesseract executable? I am using tesseract executable.

in command line ;-)
And already got reply at SO that problem is in your code.

Yes, thank you. process.StandardInput.BaseStream.close(); was needed. 👍


golang: expose add{Decoded,Encoded}Data so that people can add data when processing headers/data/trailers

Title: golang: expose add{Decoded,Encoded}Data so that people can add data when processing headers/data/trailers
Description:

Describe the desired behavior, what scenario it enables and how it
would be used.

We would like to provide a way to add extra data when processing headers/data/trailers. For example, turn a headers only request into a request with a body, add more data when processing trailers, and so on.
The add{Decoded,Encoded}Data serves our purpose well so we want to expose this method in Go, like:
f.callbacks.DecoderFilterCallbacks().AddData([]byte("blah"), true)
# equal to `addDecodedData(Instance "blah", true)
f.callbacks.EncoderFilterCallbacks().AddData([]byte("blah"), true)
# equal to `addEncodedData(Instance "blah", true)

Here I expose all the arguments of add{Decoded,Encoded}Data as only the callers know what arguments should be passed inside.
CC @doujiang24
[optional Relevant Links:]

Any extra documentation required to understand the issue.

sounds good, let's do it.

By the way, can I modify request headers according to request body?( for later routes use)

@lwabish
Yes. You could take a look at how we provide DecodeRequest to modify request headers with body in the Envoy Go wrapper: https://github.com/mosn/htnn/blob/9f234b90e7cea687cd828003edf115e295c97432/api/pkg/filtermanager/filtermanager.go#L644


feat improve backlog resiliency

Reduces Kafka poll() records to 30 and sets http connection timeout.
This should improve resiliency when there is a higher kafka backlog, to
prevent the consumer being kicked out due to inactivity (timeouted poll).
EVNT-789

/retest

@gwenneg @jpramos123 please have a look, whether the configured values make sense. Thanks.

@gwenneg @jpramos123 ping

/retest

/retest

/retest

/retest

/retest


error while installing testrpc

Trying to install testrpc and keep getting this error
Expected Behavior

Current Behavior

.h
Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
C:\Program Files\MSBuild\Microsoft.Cpp\v4.0\V140\Platforms\Win32\PlatformToolsets\v140\Toolset.targets(34,5): error MSB
8036: The Windows SDK version 8.1 was not found. Install the required version of Windows SDK or change the SDK version
in the project property pages or by right-clicking the solution and selecting "Retarget solution". [C:\Users\user\AppDa
ta\Roaming\npm\node_modules\ethereumjs-testrpc\node_modules\scrypt\build\copied_files.vcxproj]
C:\Program Files\MSBuild\Microsoft.Cpp\v4.0\V140\Platforms\Win32\PlatformToolsets\v140\Toolset.targets(34,5): error MSB
8036: The Windows SDK version 8.1 was not found. Install the required version of Windows SDK or change the SDK version
in the project property pages or by right-clicking the solution and selecting "Retarget solution".  System Windows_NT 6.1.7601
gyp ERR! command "C:\Program Files\nodejs\node.exe" "C:\Users\user\AppData\Roaming\npm\node_modules\npm\node
modules\node-gyp\bin\node-gyp.js" "rebuild"
gyp ERR! cwd C:\Users\user\AppData\Roaming\npm\node_modules\ethereumjs-testrpc\node_modules\scrypt
gyp ERR! node -v v6.3.0
gyp ERR! node-gyp -v v3.3.1
gyp ERR! not ok
C:\Users\user\AppData\Roaming\npm
`-- (empty)
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Users\user\AppData\Roaming\npm\node_modules\npm\bin\npm
-cli.js" "install" "-g" "ethereumjs-testrpc"
npm ERR! node v6.3.0
npm ERR! npm  v3.8.7
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the scrypt package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs scrypt
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls scrypt
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\user\npm-debug.log
npm ERR! code 1
Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Version used:
Environment name and version (e.g. PHP 5.4 on nginx 1.9.1):
Server type and version:
Operating System and version:
Link to your project:

The error output clearly shows:
The Windows SDK version 8.1 was not found. Install the required version of Windows SDK or change the SDK version
Do you have Windows SDK version 8.1 installed?

Can you give the TestRPC beta a try? First uninstall the normal one, if installed (or partially installed):
$ npm uninstall -g ethereumjs-testrpc

And then install the beta version:
$ npm install -g ethereumjs-testrpc

The beta version significantly improves installation and should bypass your installation issues.
Since this is a beta version, I'd very much appreciate feedback on how it works for you and if you run into any errors during use.

$ npm install -g ethereumjs-testrpc worked. Can we close this now?

Please reopen if this is an issue in the latest version.


Updated go.mod

Description:

update go version to 1.16
update ginkgo and gomega versions

/retest

/lgtm


writeflux doesn't work

When I try to writeflux, it just crashes:
terminate called after throwing an instance of 'std::runtime_error'
  what():  Attempt to access uninitialised flag
Abort

So I can see that the input flux file (goonies.flux?) contains tracks 0-39, which is only the first half of the disk, but luckily you're using a 40-track drive and (presumably) not using the --40-track option so that should sort itself out. (FluxEngine uses 80-track numbering throughout so the tracks on a 40-track disk should be in 0, 2, 4, 6 etc.) Also, each track contains 199ms of data. Your 360kB 5.25" drive spins at 300rpm and not 360rpm, right?
I don't have a 360kB 5.25" drive so I tried writing it to a 3.5" drive, which spins at 300rpm, and it mostly worked when reading back with FluxEngine --- I got increasing numbers of bad sectors on the higher tracks, but the lower ones worked fine. What happens when you try to read the written 5.25" disk with FluxEngine?

Yes, the 360k drive is 300 RPM. The original scp file contains only 35 tracks, with the other 5 empty and all tracks for side B/1 empty. While the scp format has a flag to mark a disk as single sided, it doesn't work and putting in empty tracks is the way the Windows scp program does it. The command I am using to write the converted file is:
fluxengine writeflux -s goonies.flux:t=0-34

If I try to include :s=0 in then it complains and tells me that side 1 is missing. Same if I try to use --40-track.
The command I just tried to read back in the goonies.flux as created above is:
fluxengine read c64 --40-track -o goonies-flux-test.d64

and I end up with a 0 byte goonies-flux-test.d64 file. Without the --40-track flag it tries to read 80 tracks from a 40 track drive.
I'm unable to create a new flux image from the physical floppy with fluxengine because I cannot find the right combination of arguments that will make it stop reading at track 40 (or only read tracks 0-34) and I have to interrupt it and I get no output.
Commands I've tried to read the disk back in as a flux file:
fluxengine writeflux -i :s=0:t=0-34 -o goonies-flux-test.flux
fluxengine writeflux -o goonies-flux-test.flux:s=0:t=0-34
fluxengine writeflux -o goonies-flux-test.flux
fluxengine writeflux --40-track -o goonies-flux-test.flux

writeflux was hacked in and is a bit of a mess, and really needs to be thrown away and replaced completely (I'm actually working on a major rearchitecture right now). What it does is that it reads all of the source, and then writes it to all of the destination. The reason why you're getting the error is that the destination defaults to :t=0-79:s=0-1 and it's trying to find data for side 1. So, you'll want to use fluxengine writeflux -s goonies.flux -d :t=0-34:s=0.
Re --40-track: this tells the client to halve the track numbers when accessing the drive. So, from FluxEngine's perspective, tracks 0, 2, 4 end up on 0, 1, 2 on the drive. This means that FluxEngine's internal track numbering will end up consistent regardless of what kind of drive you have. scptoflux is generating the wrong track numbers so this can't be used for writing, but it should work for reading.
To read from the disk and also generate a flux file, this command should work:
fluxengine read c64 --40-track --write-flux=out.flux -o goonies-flux-test.d64

I'm surprised the one you use above isn't working --- can you paste a few lines of the logging as it runs?

It doesn't make any sense to me to halve/double track numbers when reading from a 48tpi drive. I should be able to tell it to just use the first 35-40 tracks. The drive has less tracks, it's not skipping tracks like a 96tpi drive. I would also expect to see track data in the file in the first 40 tracks of a side, not track data skipping every other track. The duplication of flags is also very confusing. It's hard to know when -d is the output or -o is the output. Same for -i or -s and input.
fluxengine writeflux -s goonies.flux:t=0-34:s=0
Writing to: :d=0:s=0-1:t=0-81
  0.0: Using FluxEngine with serial number 7403020B1B0A3374
199 ms in 49949 bytes
  0.1: Error: missing in source

fluxengine writeflux --40-track -s goonies.flux:t=0-34
Writing to: :d=0:s=0-1:t=0-81
  0.0: Using FluxEngine with serial number 7403020B1B0A3374
199 ms in 49949 bytes
  0.1: 0 ms in 0 bytes
  1.0: Error: cannot write to odd physical tracks in 40-track mode

I've attached the out.flux generated from the read c64 command you gave me above. The d64 itself is still empty.
out.zip

The problem with using drive numbering is that you can put the same disk in an 80-track drive, and now the tracks are in a different place. So, FluxEngine works in terms of the position of the track instead.
BTW, -s and -d and source and destination disks. -i and -o are for input and output images. See "Source and destination" and "Input and output" here: http://cowlark.com/fluxengine/doc/using.html
Re the two writeflux commands: you have to tell it where to write with the -d option, otherwise it'll try to write to all tracks and both sides --- you can see the 'Writing to' line. Try this:
fluxengine writeflux -s goonies.flux -d :t=0-34:s=0

Re the two writeflux commands: you have to tell it where to write with the -d option, otherwise it'll try to write to all tracks and both sides

Since I didn't specify the destination, shouldn't the destination default to matching the source?

No, it's the other way around --- the process is driven from the destination with data being read from the source lazily. (This is only a problem with writeflux, because it was rapidly hacked together. The new version will have an improved architecture for this.)

The problem with using drive numbering is that you can put the same disk in an 80-track drive, and now the tracks are in a different place

Yes, but no. If you're reading/writing a format that isn't matched to the hardware then shuffling the tracks around should be handled during reading/writing. The track arrangement in the file shouldn't be munged to pretend to be a 96tpi source when it really did come from a 48tpi disk.

the process is driven from the destination with data being read from the source lazily

But I'm a lazy user and that trumps all laziness! ;-)

Yes, but no. If you're reading/writing a format that isn't matched to the hardware then shuffling the tracks around should be handled during reading/writing. The track arrangement in the file shouldn't be munged to pretend to be a 96tpi source when it really did come from a 48tpi disk.

I'll admit to not being particularly wed to the idea --- I don't have a 40-track drive to test with, so I don't actually get to try it in real life. But it's not a simple as you're making out: for example, right now I'm reading 48tpi disks by doing a 96tpi read on them, with two reads per 48tpi track... and sometimes the two reads are different. I'll agree it's surprising, and surprising things are bad in computing. It needs work. But that's where it is right now.


robots.txt not supporting $

According to robots.txt specifications found on Google the $ command is:

$ designates the end of the URL

OSS does not process this rule. It just takes $ as an standard character from URL.
Here's an example:
In my robots.txt I have a rule:

Allow: /opac7/bib/search$

This means that a robot should process:
DOMAIN/opac7/bib/search
but not any domain with suparameters:
DOMAIN/opac7/bib/search?paramters=3&......
If check this in manual Crawl within OSS I get:
Dissalow:
DOMAIN/opac7/bib/search
DOMAIN/opac7/bib/search?paramters=3&......
Allow:
DOMAIN/opac7/bib/search$
I don't know if this is a bug or just a missing functionality, but it would really helped us, if this gets implemented.

Thanks for your ticket.
The current (and initial) implementation followed those minimal "specs":
http://www.robotstxt.org/robotstxt.html
I will refresh that by having a look at:
https://developers.google.com/webmasters/control-crawl-index/docs/robots_txt
So, the support of * and $ is described in the chapter "URL matching based on path values".
Implementation in progress...


First launch can miss initial output

On Linux (?), initial output from the first main method launch after startup is lost.
For that first launch, the config.launch(...) method appears to take a long time to return; when it does, the launch has already terminated. Probably related.
(Imported from Bitbucket #15 reported by maxg on 2010-06-11)

Also observed once on OS X. I think.
(maxg on 2010-06-24)


New events related to permission groups changes

I want to merge this change because it adds new subscription base webhooks for permission group create, update, delete events.
New webhook events:

PAGE_TYPE_CREATED
PAGE_TYPE_UPDATED
PAGE_TYPE_DELTED

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-7476 Add permission group webhook events