picon enhanced

could be possible to have support for Picons by 'Service name" please

If for display in the program, then I do not see any big problems. Main question is assignment. How to combine the two variants. Here needs to think. One global option or, maybe, some additional menu items.

Could be possible to have support for DTT Italy picons.cz?
With condition If exists "DIgitale Terrestre" in bouquet name....

Did I understand correctly that this combination does not work?

If so, it is desirable to send me your set of channels.

With condition If exists "DIgitale Terrestre" in bouquet name....

Of course it can be made, but it's a bad idea (if I understand correctly). Such a feature should be universal, and not tied to a specific provider or name. Perhaps we need some additional input field or something like that.

It works, I had filter enabled.
Is it possible if vHannibal settings SAT + DTT are installed that works with filter enabled?

Since the filtering occurs by position, and for these packages the author of the picons does not provide such information, it is currently impossible.

Since the filtering occurs by position, and for these packages the author of the picons does not provide such information (API peculiarity), it is currently impossible.

Since the filtering occurs by position, and for these packages the author of the picons does not provide such information (API peculiarity), it is currently impossible.

Hello, what exactly does the DE look for in the package name for DTT?

Hi,
When parsing the mapping file, we have, for example, such results:

'8178' ->  'piconmirrorglass-400x240-DTTitaly_by_chocholousek.7z'
'8166' ->  'piconmirrorglass-400x240-4.8E_by_chocholousek.7z'

DE parses the file name and looks for the position (4.8E in our case). But there is no position for the value 'DTTitaly'!

Also, this value is not in other columns of the main channel list (for example, we could use provider name etc.)
Filtering and searching through bouquets list does not make sense. Since users can use any names and bouquets set. Perhaps there are other options. Ready to discuss.
Anyway, thanks for your picon sets. They are really cool!

Hello, and thanks. ;)
I understand, I thought that DE is looking for some nickname dvb-t dtt or terr for terrestrial as a replacement and non-existent position. But it's better to leave everything as it is.

could be possible to have support for Picons by 'Service name" please

Added basic support. So far, only for display in the program. As for the assignment, it is not yet entirely clear how best to do this.

Possible but not easy. The whole project is adapted only for service reference. I'll try to figure something out.

Of course it's possible. :) The main question is how to make it more user-friendly. Additional menu item, additional function or option for conversion (for example, on the "Picons" tab), etc.

Can you explain that?

If short...
You have channel names:

HTB.
НТВ.

They looking the same. But... These are different names.
1 - written Latin alphabet, and 2 nusing the Cyrillic alphabet!
Here is a function that converts the service name to the necessary name of a picon. Everything seems to be logical and simple.  BUT the ASCII table does not contain all characters used by Cyrillic! We need transliteration!  That is, before we convert the name, we need to find out in what language and in what encoding it is written, and then convert it according to certain rules! Since the lists can be large and contain names in different languages, the performance issue can become quite really.

Yes, I also encounter this problem in the Czech language. It's been a long time since I created my own script that solves this problem.

A problem that can be solved.

A problem that can be solved.

Yes, in your particular case. But keep in mind that the terminal also uses certain language settings for each specific system. Also, when run from Python, the result may be different. In addition, you have an example for Cyrillic, but what to do with Arabic or Chinese characters, for example.

the question is how to solve this in the PLI image itself?

I think that this is decoded in a certain way at the stage of channel scanning and converted (possibly additionally cached). I can't say exactly. It need to look at the source. In addition, Enigma2 uses a single platform and most channel names use the Latin alphabet (which is why it is not relevant for most). BUT we still have IPTV! And anything can be found here.:)
Also, when you run the program on a PC, you do not know what OS you are using and what encodings and language settings you'll have.
P.S. As I already wrote, using an id channel avoids many problems and that is why this approach is used in most cases.

but what to do with Arabic or Chinese characters, for example.
I only support European satellites, sorry.

I only support European satellites, sorry.

Yes, I know. It was more a rhetorical question in your case. :)
At the moment, displaying piсons by name in most cases working (excluding the nuances with encodings, but I think over time it will be solved. It just needs to modify properly this function.) The main, we need to decide, how to assign new picons.

Additional menu item, additional function or option for conversion (for example, on the "Picons" tab), etc.

I'll be glad to suggestions.

https://picon.cz/dnes-pridany-picony-transparent-snp-220x132-take-do-chocholousek-picons-pluginu/
Hi, SNP picons added... ;)

Hi, Coool! :)
Display and download work!

Just filtering function correction is needed.
As for the names in Cyrillic. I found a working way. But... There are questions to Enigma2 itself.  The method that is currently used to convert names is not suitable. At least it doesn't work for me. Therefore, for now I will leave the conversion in the app unchanged (it is almost identical to the one used in enigma2 itself)

looks no issue here and could be close

Fine. If there are suggestions, we can return to this discussion.


Optimization attempt #1

Hi!
This is my proposal from #7 in action. I tried running the existing tests in the way you described. I got mixed results so I am not sure if the results are conclusive, so I'd really appreciate it if somebody else also compared the performance before/after these changes.
This is the output of the tests:
Before:
测试[ast]...
测试[ast]用时[0.112]
测试[grammar]...
测试[grammar]用时[0.009]
测试[grammar_check]...
测试[grammar_check]用时[0.008]
测试[syntax_check]...
测试[syntax_check]用时[0.008]
测试[perform]...
[test\perform\1.txt]测试完成，大小[42.265]kb，速度[1.854]mb/s，平均用时[22.800]毫秒
[test\perform\2.txt]测试完成，大小[6.595]kb，速度[7.328]mb/s，平均用时[0.900]毫秒
[test\perform\3.txt]测试完成，大小[1.257]kb，速度[0.662]mb/s，平均用时[1.900]毫秒
[test\perform\4.txt]测试完成，大小[4923.960]kb，速度[2.219]mb/s，平均用时[2219.000]毫秒
[test\perform\5.txt]测试完成，大小[1451.154]kb，速度[3.643]mb/s，平均用时[398.333]毫秒
[test\perform\6.txt]测试完成，大小[38.577]kb，速度[2.717]mb/s，平均用时[14.200]毫秒
[test\perform\7.txt]测试完成，大小[52.448]kb，速度[1.609]mb/s，平均用时[32.600]毫秒
综合性能测试完成，总大小[456.419]kb，速度[2.056]mb/s，用时[0.222]秒
测试[perform]用时[4.380]
测试完成

After:
测试[ast]...
测试[ast]用时[0.144]
测试[grammar]...
测试[grammar]用时[0.008]
测试[grammar_check]...
测试[grammar_check]用时[0.011]
测试[syntax_check]...
测试[syntax_check]用时[0.040]
测试[perform]...
[test\perform\1.txt]测试完成，大小[42.265]kb，速度[1.049]mb/s，平均用时[40.300]毫秒
[test\perform\2.txt]测试完成，大小[6.595]kb，速度[8.244]mb/s，平均用时[0.800]毫秒
[test\perform\3.txt]测试完成，大小[1.257]kb，速度[0.662]mb/s，平均用时[1.900]毫秒
[test\perform\4.txt]测试完成，大小[4923.960]kb，速度[2.452]mb/s，平均用时[2008.000]毫秒
[test\perform\5.txt]测试完成，大小[1451.154]kb，速度[3.502]mb/s，平均用时[414.333]毫秒
[test\perform\6.txt]测试完成，大小[38.577]kb，速度[2.816]mb/s，平均用时[13.700]毫秒
[test\perform\7.txt]测试完成，大小[52.448]kb，速度[1.619]mb/s，平均用时[32.400]毫秒
综合性能测试完成，总大小[456.003]kb，速度[1.869]mb/s，用时[0.244]秒
测试[perform]用时[4.411]
测试完成

Do you have the scripts needed to execute the benchmark for the Lua Language Server described here? I think that could give better insight if I opened an existing code base before/after the changes than an artificial test environment.

Do you have the scripts needed to execute the benchmark for the Lua Language Server described here? I think that could give better insight if I opened an existing code base before/after the changes than an artificial test environment.

No, I just read the log.

I tried looking in the logs, but I couldn't determine anything useful. Do you have an idea how I could benchmark this?

I will write a script for benchmark these days.

Thanks for the benchmark!
It looks like my try didn't really improve performance at all. I tried for 10 times my branch and master. The difference was negligible or the master even outperformed my changes by a small margin. Because of that, I am closing this.


In the map view the observation names should be cutted properly, check also the other buttons

Already fixed in previous merge. Closing branch.


Limit CPU percent panels in cloud view to 100%

Set the y-max limit to 100 on panels that show a percentage. In some
cases a cloud restart or re-connection to STF after a period of no data
can result in a spike of the graph, resulting in a momentary peak of
several million percent.
This change sets the graphs to always show a range of 0 to 100% to limit
the visual impact of an overscale / high peak value problem.
Related: rhbz#2006970

@csibbitt bug is in libpod-stats

Seems like a bandaid, but headed in the right direction. Looking at the expr, I see this is basically a bare metric, not a calculation, so I guess the real fix is likely in libpodstats (possibly collectd I guess?)

Definitely a stopgap. Paul and I talked about it today and the related BZ we're leaving open as a task to have this fixed in libpodstats. Paul seemed to think he had an idea where the issue might lie, so hopefully we'll fix this properly in a future OSP release.


Fix 404 Error Bug #1461

Hi
I have implemented a piece of code that replaces the unsafe '' with '/' in URLs. Please let me know if you need further improvements to consider to merge.
Thanks
Best Regards
Raghav Jajodia

Thanks for this fix


Use translation file for check column

Use the translated version of Yes/No as default for the check button in list views.

Thank you @jnoordsij ! You're right on this, of course. Thank you @pxpm for reviewing it. Just merged it, it'll be available with a composer update later today, as 4.0.18.
Thanks again. Cheers!


how to save workspace in snap.svg?

Hi, i want to save my workspace ,is there any option to export and import  json format in snap.svg?

There is no way to save anything in Snap specifically....you could somehow dump the html/svg possibly and do something with it, and reimport later


The httpclient5 and httpcore5 libraries can be removed from DITA-OT distribution

Discussed in https://github.com/orgs/dita-ot/discussions/4232

Originally posted by jlacour31 July 11, 2023
In DITA-OT 4.1 distribution there are 3 extra libraries:

httpclient5-5.1.3.jar
httpcore5-5.1.3.jar
httpcore5-h2-5.1.3.jar

These libraries are brought into the distribution by xmlresolver-5.1.2.jar used in Saxon.
I think they can be excluded.

We removed it after integrating DITA-OT 4.1 in Oxygen and tests are still working. They can be considered bloat.


[BUG]: Report not uploaded to workspace

Describe the bug
When I run out whole test suite, the test report is not uploaded to our workspace. The test run appears there, but the results are missing.
In my pipeline it says "The request made to the server is invalid. Please check the request parameters and try again."
When I run only a few tests, this issue does not occur. Only when I execute a larger amount of tests.
To Reproduce
Steps to reproduce the behavior:

Execute test suite (30+ tests) via Azure Devops pipeline
Observe the error

Expected behavior

The test results should get uploaded to the workspace
Any error message such as "The request made to the server is invalid. Please check the request parameters and try again." should be more descriptive of the root cause

Screenshots

Setup information (please complete the following information):

Azure Subscription Id: 1ecbf7b1-d5aa-4de7-971e-7c806693e766
Workspace Name: greenventoryworkspace
Service trace Id (if available):
Log snippet (Azure Pipeline Output):

2024-10-02T12:06:35.0570830Z   3 failed
2024-10-02T12:06:35.0571256Z     [chromium] › <SOME TEST>
2024-10-02T12:06:35.0571911Z     [firefox headed] ›<SOME TEST>
2024-10-02T12:06:35.0572472Z     [firefox headed] › t<SOME TEST>
2024-10-02T12:06:35.0572809Z   1 flaky
2024-10-02T12:06:35.0573300Z     [firefox headed] ›<SOME TEST>
2024-10-02T12:06:35.0573639Z   1 skipped
2024-10-02T12:06:35.0573856Z   29 passed (14.3m)
2024-10-02T12:06:36.3522649Z The request made to the server is invalid.  The request made to the server is invalid. Please check the request parameters and try again.
2024-10-02T12:06:36.9980560Z
2024-10-02T12:06:37.0040994Z ##[error]Script failed with exit code: 1

Browser on which tests are executed (if applicable): firefox, chrome
Time range (in PST) when the issue was experienced: Tested at various times today and yester (e.g. today at ~4 am)
Playwright Version: 1.47.2
Client machine OS: linux
OS requested on Service: linux

Additional context

Yes, the same behavior has been observed while running 14 tests on non-CloudHostedBrowsers and sending the test report to our workspace, but it looks okay when running fewer tests.
Behavior while running fewer test
1 passed (13.7s)
Uploading test results.
[========================================] 100%
Test report: https://playwright.microsoft.com/workspaces/westeurope_b87e9689-b6a0-480e-a941-ed7a982d5833/runs/11134673564-1-af88b00ef61e9f587ba0c57b00046fe8c5dbb63b-Integration
Behavior while running more tests:
12 passed and 2 failed
The request made to the server is invalid. Please check the request parameters and try again.
Uploading test results.
[===                                     ] 6%
[=====                                   ] 13%
[========                                ] 19%
[==========                              ] 25%
[=============                           ] 31%
[===============                         ] 38%
[==================                      ] 44%
[====================                    ] 50%
[=======================                 ] 56%
[=========================               ] 63%
[============================            ] 69%
[==============================          ] 75%
[=================================       ] 81%
[===================================     ] 88%
[======================================  ] 94%
[========================================] 100%
Test report: https://playwright.microsoft.com/workspaces/westeurope_b87e9689-b6a0-480e-a941-ed7a982d5833/runs/11134808876-1-4d2e489913fad943933bfc8d3059f7ee3257ac4e-Integration

The request made to the server is invalid. Please check the request parameters and try again.

Thank you for reporting this issue.
We are actively looking into it.

Sorry for the inconvenience caused.
We have just put out a fix for the above. Can you please try again.
In case you face similar issue again, please feel free to comment.
Thanks

@mkgreenventory , @muthamizh1504  We have resolved this. Could you retry and verify if it is working for you

@mkgreenventory , @muthamizh1504 We have resolved this. Could you retry and verify if it is working for you

It is now working for us. Thanks for the quick fix!

@vvs11 It is also working for me now. Thanks!


Symlinks for samtools are not created for a fresh installation

Version info

bcbio version (bcbio_nextgen.py --version): 1.2.9 (development)
OS name and version (lsb_release -ds): Ubuntu 18.04.6 LTS

To Reproduce
Exact bcbio command you have used:
bcbio_nextgen_install.py /usr/local/share/bcbio-nextgen --isolate --minimize-disk --nodata -u development --revision master
/usr/local/share/bcbio-nextgen/anaconda/bin/bcbio_nextgen.py upgrade --isolate --tooldir=/usr/local --tools --toolconf packages-conda.yaml

packages-conda.yaml is the current file from Cloudbiolinux.
Installation completes without error. However, there are no symlinks in /usr/local/bin for tools included in samtools like samtools itself or bgzip. Symlinks for all other tools exist, as far as I can see. /usr/local/share/bcbio-nextgen/anaconda/bin/samtools exists.
Executing bcbio_nextgen.py upgrade again does not help.
Any hints what could be wrong?

Hi @gabeng !
Thanks for reporting!
I suspect you might have another samtools in the system. What is your:
which -a samtools

Also, why do you install it in two steps instead of
python3 bcbio_nextgen_install.py [bcbio_path] --tooldir=[bcbio_path]/tools --nodata --mamba

Sergey


fix(Makefile): do not ask for confirmation for deleting vms

Description
When doing make clean, do not ask for confirmation for deleting VM's

Already merged with UT branch

:tada: This PR is included in version 1.7.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


replace audio in video

How can I replace the audio of a video?

I've rewritten the package and it has support for multiple inputs (complex filters and mapping). Could you test the v7 branch? Almost no breaking changes except for some filter stuff in HLS exports.
https://github.com/pascalbaljetmedia/laravel-ffmpeg/tree/v7
composer require "pbmedia/laravel-ffmpeg:v7.x-dev"
<?php

FFMpeg::fromDisk('local')
    ->open(['video.mp4', 'audio.m4a'])
    ->export()
    ->addFormatOutputMapping(new X264, Media::make('local', 'new_video.mp4'), ['0:v', '1:a'])
    ->save();

Fixed in v7.0


Variable for Project Name in Security Updates

Is there ability to use a variable, such as $$PROJECTNAME$$ when making security updates?  I haven't figured out how to accomplish this when I'm updating a project team's permission settings.  I also could use this to copy the settings from another team, or in adding groups.

I'm not sure I understand what you mean. On the security page, you can manage groups across all selected Team Projects and the project is implied when applying security permissions. So when you add/update a group, it's always a group inside of the Team Project(s) being processed.
To copy settings from another group, you can retrieve them first (in the view/export/delete page), export to file, then load the settings (on the add/update page) and apply. Makes sense?

That does make sense within a project.
But is there a way I can reference the ProjectX Team TFS group without the specific project name?    My hope is to update this group for multiple projects without having to update the group name to ProjectY Team, ProjectZ Team and so on.
--
Tracy Dixon, PMP, PMI-ACP
Centric Consulting
In Balance
(904) 813-9661 (tel:(904)%20813-9661)  office/mobile
(904) 638-3671 (tel:(904)%20638-3671)   fax
LinkedIn: linkedIn.com/in/tracyldixon
Twitter: twitter.com/tracyldixon
www.centricconsulting.com (http://www.centricconsulting.com)   |   @Centric (https://twitter.com/centric)

On Mar 4, 2018 at 6:46 AM,  <Jelle Druyts (mailto:notifications@github.com)>  wrote:
I'm not sure I understand what you mean. On the security page, you can manage groups across all selected Team Projects and the project is implied when applying security permissions. So when you add/update a group, it's always a group inside of the Team Project(s) being processed.
To copy settings from another group, you can retrieve them first (in the view/export/delete page), export to file, then load the settings (on the add/update page) and apply. Makes sense?
—
You are receiving this because you authored the thread.
Reply to this email directly,   view it on GitHub (https://github.com/jelledruyts/TfsTeamProjectManager/issues/28#issuecomment-370222659), or   mute the thread (https://github.com/notifications/unsubscribe-auth/AXT52RDZr2Mlw2avvKEDAflqTcCtr8iLks5ta9QugaJpZM4Sa7h3).

Ah so as the group name in the tool, you'd want to enter something like "$$$PROJECTNAME$$$ Team" which is then replaced by the actual Team Project name when applying changes? That's not currently possible in the tool, but just to understand if that's what you mean.
Note that you seem to be referring to Teams a lot, which is not the same as a security group, so if you want to manage Team security that's actually not possible today at all.

Yes, exactly what I’m hoping for on the project variable.
In my current situations, my clients don’t have security variances between teams. Although full team support would be great in the future, I’m only looking for ability now to easily modify the default team created for every project without the use of the explicit name for each.
While I have you here, the ability to specify tfvc security by branch and build security by build definition would also be wonderful. All of my clients have differing security between dev, qa, and prod on those.
--
Tracy Dixon, PMP, PMI-ACP
Centric Consulting
In Balance
(904) 813-9661 (tel:(904)%20813-9661)  office/mobile
(904) 638-3671 (tel:(904)%20638-3671)   fax
LinkedIn: linkedIn.com/in/tracyldixon
Twitter: twitter.com/tracyldixon
www.centricconsulting.com (http://www.centricconsulting.com)   |   @Centric (https://twitter.com/centric)

On Mar 4, 2018 at 7:42 AM,  <Jelle Druyts (mailto:notifications@github.com)>  wrote:
Ah so as the group name in the tool, you'd want to enter something like "$$$PROJECTNAME$$$ Team" which is then replaced by the actual Team Project name when applying changes? That's not currently possible in the tool, but just to understand if that's what you mean.
Note that you seem to be referring to Teams a lot, which is not the same as a security group, so if you want to manage Team security that's actually not possible today at all.
—
You are receiving this because you authored the thread.
Reply to this email directly,   view it on GitHub (https://github.com/jelledruyts/TfsTeamProjectManager/issues/28#issuecomment-370226298), or   mute the thread (https://github.com/notifications/unsubscribe-auth/AXT52ZtIsVWafrP_BlAcOMyT53fES9VYks5ta-FOgaJpZM4Sa7h3).


[Feature Request] Humidifer

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
I would like to ask if you could please adding the support for dehumidifier, it could read the humidity and choose the operation mode.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Cool, you can use the new humidifier integration for this! https://www.home-assistant.io/integrations/humidifier/

Cool, you can use the new humidifier integration for this! https://www.home-assistant.io/integrations/humidifier/

sorry, i dont understannd, after i install this lg intergration, there is no entity added. How can i use humidifier intergration?

I was suggesting if @ollo69 implemented it, instead of a switch they could make a humidifier.name.


Update README.md

Motivation
The command to run the create_data.py script, does not run in shell.
Modification
Proper arguments should be passed to the script.

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


refactor: add empty data structures for storage CT

First step to implement battery storage CT for use with HA Energy dashboard.
This PR creates the empty data structures needed to collect the storage CT data from /ivp/meters and ivp/meters/readings. The data itself is not yet included. The structures are the same as for the production and consumption CT. Some refactoring done to abandon the assumption no more as 2 CT types can report.
Test snapshot updated for the empty data structures added.
A next PR will add data collection and documentation update.

Thanks.


add APOLLO_APP_ID env.

Codecov Report

Merging #2067 into master will decrease coverage by 0.05%.
The diff coverage is 20%.

@@             Coverage Diff             @@
##             master   #2067      +/-   ##
===========================================
- Coverage     50.25%   50.2%   -0.06%
+ Complexity     1981    1979       -2
===========================================
  Files           400     400
  Lines         12349   12354       +5
  Branches       1275    1276       +1
===========================================
- Hits           6206    6202       -4
- Misses         5686    5694       +8
- Partials        457     458       +1

Impacted Files
Coverage Δ
Complexity Δ

...internals/provider/DefaultApplicationProvider.java
65.95% <20%> (-5.48%)
11 <0> (ø)

...work/apollo/biz/message/DatabaseMessageSender.java
57.14% <0%> (-10.21%)
6% <0%> (-2%)

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 21e76b3...5fb7eb7. Read the comment docs.

Coverage decreased (-0.05%) to 53.91% when pulling 5fb7eb7da195f9737c9ac82a1616ba1dc4eb17d4 on JaredTan95:apollo-id-env into 21e76b3d43f455c1e28a9d6626601362cf4997ae on ctripcorp:master.


Destructuring in init defn is not right.

The map destructuring is mapped one layer to deep, so instead of
 (redis/init :url (or (System/getenv "REDISTOGO_URL") local-redis-url)

you must do
(redis/init [:url (or (System/getenv "REDISTOGO_URL") local-redis-url])

I'm pretty sure this was unintentional.

Turns out it actually is intentional; oops.


Max Length with sum k added

Have you read the Contributing Guidelines on Pull Requests?
YEs
Description
Cpp Solution for max length subarray with sum k added
Checklist

[ Yes] I've read the contribution guidelines.
[ Yes] I've checked the issue list before deciding what to submit.
[ Yes] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#1813

#1979

#1979


[falco] target alert in stdout

Signed-off-by: pablopez<EMAIL_ADDRESS>What type of PR is this?
/kind content
Any specific area of the project related to this PR?
/area lab:falco
/area text_content
What this PR does / why we need it:
Makes alert output clearer, filtering the specific alert the student should be aware of.

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: developer-guy, pabloopez
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [developer-guy,pabloopez]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 41d1ff39c29a473f73c209f37f982f9bd465eeee


For Maps key order is important for Rust but not for JS

Take this Map for example
This works
let indexes = new Map<number, number[]>();
    indexes.set(0, [1, 0]);
    indexes.set(1, [3, 2]);

This throws an error during preflight
let indexes = new Map<number, number[]>();
    indexes.set(1, [3, 2]);
    indexes.set(0, [1, 0]);

Either A) we should use objects vs maps as those are alphabetically sorted (I think? maybe?) or better yet you include some util on the bindings side that takes maps and ensures they're ordered however Rust expects them to before delivery.

Here's my specific error when the Map is out of order
{
    "jsonrpc": "2.0",
    "id": 1,
    "result": {
        "error": "HostError: Error(Value, InternalError)\n\nEvent log (newest first):\n   0: [Diagnostic Event] topics:[error, Error(Value, InternalError)], data:\"failed to convert ScVal to host value\"\n   1: [Diagnostic Event] topics:[error, Error(Value, InternalError)], data:\"failed to convert ScVal to host value\"\n\nBacktrace (newest first):\n   0: soroban_env_host::host::conversion::\u003cimpl soroban_env_host::host::Host\u003e::to_host_val\n   1: \u003calloc::vec::Vec\u003cT\u003e as alloc::vec::spec_from_iter::SpecFromIter\u003cT,I\u003e\u003e::from_iter\n   2: core::iter::adapters::try_process\n   3: soroban_env_host::host::frame::\u003cimpl soroban_env_host::host::Host\u003e::invoke_function\n   4: preflight::preflight_invoke_hf_op::{{closure}}\n   5: preflight::catch_preflight_panic\n   6: _cgo_a3255893d7fd_Cfunc_preflight_invoke_hf_op\n             at tmp/go-build/cgo-gcc-prolog:99:11\n   7: runtime.asmcgocall\n             at ./runtime/asm_amd64.s:848\n\n",
        "transactionData": "",
        "events": null,
        "minResourceFee": "0",
        "cost": {
            "cpuInsns": "0",
            "memBytes": "0"
        },
        "latestLedger": "69233"
    }
}


[BUG] attributes function receive modified/formatted href

The doc says that attributes option accepts a function that takes the unformatted href (https://linkify.js.org/docs/options.html#attributes) however, if there is formatHref option which has a function modifying href, the attributes function receive modified/formatted href.
The problem exists on the version 4.0.2 and 4.1.0

Hi @ryaa thanks for reporting. This should be fixed in the latest release v4.1.1

I have installed and tested v4.1.1 and the above problem is still there. attributes function in options still receive formatted/modified href.


Helm Task printing all pods in cluster for each chart deployed in 'upgrade' or 'install' command

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: HelmDeploy
list here (V# not needed):
V0
Direct link: https://github.com/microsoft/azure-pipelines-tasks/blob/bb48cfe4c7ca7ffd3d4e1f36efd7bf334cec0273/Tasks/HelmDeployV0/src/helm.ts#L176
Environment

Server - Azure Pipelines
Agent - Hosted

Issue Description
The Helm task, on successful upgrade, appears to be executing a set of commands to present the status of the cluster to the user.  But the way this is done is causing the logs to be flooded on large deployments.
Starting at this line: https://github.com/microsoft/azure-pipelines-tasks/blob/bb48cfe4c7ca7ffd3d4e1f36efd7bf334cec0273/Tasks/HelmDeployV0/src/helm.ts#L165
The task appears to call helm to get the charts in a deployment.  Then it loops through the charts and calls pushDeploymentDataToEvidenceStore.  The first thing this does is list every pod in the cluster and send it to stdout.
We have a deployment that is pushing out 91 charts.  The end result is 91 calls of the following:
[command]/vsts/agent/_work/_tool/kubectl/1.18.4/x64/kubectl cluster-info
[command]/vsts/agent/_work/_tool/kubectl/1.18.4/x64/kubectl get pods -o json
This leads to us logging 1.3 million lines during a helm upgrade.
It appears a PR was merged in 12 days ago that is the direct result of this: https://github.com/microsoft/azure-pipelines-tasks/pull/12995
If I understand the change, before this would have only happened if --debug was attached as an optional argument.
Task logs
Cannot attach logs as they are private for the company (internal Microsoft), but I hope the above makes sense enough.

@SirAlvarex We're sorry for the inconvenience. The issue is indeed because of the PR that you pointed out. But earlier, it would have never happened, whether --debug was used or not (because of a bug, this code was never executed). It seems that the commands kubectl cluster-info and kubectl get pods -o json are getting executed for every chart which is not required. We'll work on fixing this as soon as possible.

At the same time it seems the task prints out the manifest, which includes secrets. Since we are using helm to base64 encode the values of secrets, Azure DevOps does not recognize them as secrets and does not censor them in the output. That's a pretty major security breach as it prints out passwords out in the open.

Closing the issue. The fix should start rolling out from next week.

This does not seem fixed. The kubectl install/upgrade command is still attempting to get my cluster data into the "EvidenceStore". The task doesn't fail, but it does throw an error:
Release "my-app" does not exist. Installing it now.
NAME: my-app
LAST DEPLOYED: Fri Nov 12 15:34:40 2021
NAMESPACE: default
STATUS: deployed
REVISION: 1
TEST SUITE: None
/usr/bin/kubectl cluster-info
Error from server (Forbidden): services is forbidden: User "system:serviceaccount:default:azure-devops-agent" cannot list resource "services" in API group "" in the namespace "kube-system"

To further debug and diagnose cluster problems, use 'kubectl cluster-info dump'.


fix: correct maven documentation

Resolves #5392

You ever look back on documentation you wrote and can't believe you wrote it as it was clearly wrong?  ugh...

You ever look back on documentation you wrote and can't believe you wrote it as it was clearly wrong? ugh...

Yes, unfortunately that sounds a bit too familiar to me :)


To display entire multibyte character like Japanese.

A character does not display entirely like below.

Are there a any solution?
Version is

Version: 1.0.2

thanks.

I have a feeling this is related to the measurement of the height of the characters. @kntmrkm can you put this in a fiddle?

Actually seeing this in v2.1.4 as well in the x-axis label: https://jsfiddle.net/1zzbv13m/2/. Haven't looked much further into it yet though, possibly the scaleLabel.labelString positioning?

@sassomedia interesting, thanks for the fiddle. I think the multibyte character problem is due to the fact that you can't actually measure the height of the text on the canvas. The return of measureText only has the width property set because no browsers implement it. https://developer.mozilla.org/en-US/docs/Web/API/TextMetrics

@etimberg hmm, wouldn't the text height be equal to the font size? I think the issue is not so much related to multibyte character, just in general that the text is clipped. In @kntmrkm 's example, the "KakaoTalk" is cut as well. It just seems that it's more pronounced in v1 than in v2.
Added couple Japanese and Kanji characters to my existing fiddle, and it's actually fine, compared to the  roman letters: https://jsfiddle.net/1zzbv13m/3/. I guess the type of font used might also play a role, as some go further below the baseline. Given that the font size spans the whole range of above and below the baseline, I think factoring in font size could work as a measurement.

@sassomedia right now the height is simply the font size. Some investigation is definitely required

Hello, today I am closing all issues that are only affecting version 1 of Chart.js as WONTFIX.
If this issue does affect version 2 as well, I apologize for the error. Please create a test case against Chart.js 2 using on of the below websites and we will be happy to reopen the issue and update its classification:

http://jsbin.com/
http://jsfiddle.net/
http://codepen.io/pen/
Premade template: http://codepen.io/pen?template=JXVYzq

@fulldecent this has been closed in error, see my comment above as it also affects v2. I guess the label was just not updated.

@fulldecent thanks!

This still reproduces in v2.5.0 but if the solution described in #4006 is implemented, it could also be used to fix this.

@etimberg did your PR https://github.com/chartjs/Chart.js/pull/4382 fix this one?

@benmccann I don't think either addressed this directly. We'd need to add a similar lineHeight for the tick labels to full address this I think

I think the original issue (tick labels) was fixed before v2.0.0, and the additional issue (scale label) was fixed via #4531 by introducing the default lineHeight value: 1.2. So, I'm closing this.


[add] option exceptMethods to no-es6-static-methods

Options for rule no-es6-static-methods which allow use specific method. Some methods can be shimmed with polyfills or replace function which used from es6 to not used from es6 set (bn.js case: https://github.com/indutny/bn.js/blob/f953de2a4040852c8948f6ac36e501b092379292/lib/bn.js#L1635-L1638).

@nkt can you please take a look? Should I change something for quick review from you?

If problem in using Set from es6, I can rewrite code for working without it.

Any plans to get this merged in @nkt? I'd really like this, as we polyfill some es6 methods and don't want to replace them all just to get linter without errors

Thank you! :tada:
@nkt can you publish new version please?


How to contribute win32 and keyhandling updates to bullet

I have a set of updates written and tested to add correct win32 keyboard input processing and keyhandling additions to the Bullet, Check and Scrollbar classes that I think would benefit other users of bullet.  I have added HOME_KEY and END_KEY keyhandling to all three bullet classes and PG_UP_KEY and PG_DOWN_KEY keyhandling to Scrollbar.
Edit2: I subsequently added a prompt_color variable to all the bullet classes with a prompt and fixed handling of x'08' as the backspace character for the input method of class myInput (and not just x'7f').  I also added "self.result = []" to the launch functions of VerticalPrompt and SlidePrompt so that they work the same if called a second time (i.e., only return the results of the last launch).
Edit 4: One more set of changes - I forgot to put in colors.RESET at the end prompt writes so the next line(s) don't get the prompt_color value.
How can I contribute these changes to bullet?  I am totally ignorant of how github works, so I don't know how to give you my code changes.
Any baby-step instructions for contributing code to this github repository that you can provide would be appreciated.
Peter
Edit4: Diff -u changes attached and updated with latest changes.
bullet-diff.log

I have done some self education in github and have created pull #79 with all of my enhancements.
Please let em know if this set of updates can be merged into the master branch.  I would like my own python projects to require bullet but I  cannot do that unless at minimum the Windows support is merged.
Regards,
Peter


plugin: feature: Add some features!

Add some features!
This issue is for discussion and help needed comments while adding new Features to DFFML.
See the new feature tutorial to get started.

Features as a bit obsolete at this point.


chore: reduce import in NumberTheory.Divisors

I'm not entirely sure this is useful, but I noticed it in review and thought I'd try it. I wouldn't mind if we just said no either.

bors merge


Fix Window positioning bug when bad pivot is stored in app data

This PR fixes an issue where a bad pivot stored in app state would override the pivot actually set from code. The root code of this issue if the Window's State containing pivot in the first place, as pivot is not part of a state to be tracked. It makes the State structure more ergonomic though, so this PR leaves the pivot field but always overrides it with the value provided by user code.
Repro of the original issue
This issue can be reproduced using the re_ui_example app from the Rerun repo, at this commit: https://github.com/rerun-io/rerun/pull/4577/commits/fb5add0047f108769c413393d65d52e362cbcc8e. By using this app.ron file, the bug will appear:
{
    "egui": "(
      areas:{
            ((0)):(
                areas:{
<PHONE_NUMBER>3718688666):(pivot_pos:(x:565.5,y:328.0),pivot:((Min,Min)),size:(x:364.0,y:75.5),interactable:true),
                },
            )
        }
    )",
}

The modal is entered based on it's top-left corner even though the code actually specifies a center-center pivot:

With this PR, the centring is correct even with the "poisoned" app.ron:

The fact that persisted app state can corrupt State makes me think that this "TODO" comment might be incorrect:
https://github.com/emilk/egui/blob/83f5d5c972a39ef509eb3f508ca675e832103121/crates/egui/src/containers/area.rs#L7-L23

The fact that persisted app state can corrupt State makes me think that this "TODO" comment might be incorrect:

The comment is correct. The area state is not stored in Memory::data, but in another part of Memory (so it is still persisted). Functionally this matters little, it is just inconsistent with other widget state.


Add a button to save content and create a new one.

Laravel Version: 5.4.28
Voyager Version: 0.11.14
PHP Version: 7.1
Database Driver & Version: MySQL 5.5

Description:
First of all, I would like to thank you all for this great work you did !
Here is my proposition: when creating/editing new BREAD content, it would be great to have 3 buttons to save :

Save : save and go back to the browse list
Apply : save and stay on the content
Save and Add new : save and create another content

That would be a great time saver.
Steps To Reproduce:
Add a new BREAD content, there is only a "Save" button which will show you the create content.
Have a nice day !

Hi @alanpilloud, we will certainly work on that, it has been requested several times.
I suggest having a config for defining the default button, and instead of 3 buttons, have a drop down next to the submit button (so it doesn't take much space at the UI).

Hi @akazorg, I would be more than happy to contribute on this, could do you provide me any guidance on how you want me to do that ?

+100k for this feature :)


minor fixes ,batch file added

the usage of windll crashed the program for me, returned a negative number for some reason so I used pyautogui(does the same thing)
the selection x,y range was inaccurate for me so I reduced it
moved the program to a standard main setup to allow further scale
added batch file for easy execution of the program

Thank you so much for contributing to this project!! 💚 💙 💜 💛 ❤
These are all very thoughtful and needed changes.

the usage of windll crashed the program for me, returned a negative number for some reason so I used pyautogui(does the same thing)

I somehow missed the pyautogui.pixel(x, y) function, it's certainly better suited for the job. I've also experienced some windll crashes before, get no crashes with your solution.

added batch file for easy execution of the program

Good idea!


FAQ section

This PR contains code for the FAQ section and to make it responsive
Note: svgs will be made responsive tmr after I get designs

also, it looks like the astronaut is appearing in front of the FAQ. could you make it so the astronaut is either below the FAQ section (preferred option), or behind the text (easier option, prob z index)?


whole words

Can you modify this system so that it would transfer whole worlds to the second line?

Maybe in the future.


fleetctl: "askToTrustHost" doesn't work correctly with cluster and empty "known_hosts"

Steps to reproduce:

spawn 3 coreos nodes cluster
create a template<EMAIL_ADDRESS>
[Service]
ExecStart=/bin/bash -c "while true; do echo Hello, World %i!; sleep 1; done"

clear ~/.fleetctl/known_hosts

start units fleetctl start<EMAIL_ADDRESS>

fleetctl status<EMAIL_ADDRESS>then type "yes" for each request. second request will not work until you type it twice.

Yet another example. When you have four nodes and you have units on all of them, try to approve ssh fingerprints for three nodes only and skip last one. Then try to retrieve status once again with the fleetctl status<EMAIL_ADDRESS>you'll notice that you have to type enter three times to get the result. Looks like each ssh connection adds something to stdin so it requires extra "enter" to start work.

@jonboulle I have not a clue what is wrong. But when I removed session.Stdin = os.Stdin string from Execute function, fleetctl started to work fine. I don't know whether this fix is correct, because I still didn't find the root problem.
I also played a lot with these constants inside gossh.TerminalModes and it didn't help at all.

UPD:
Nope, this fix is not correct. fleetctl ssh hello@5 "bash" doesn't work, because stdin is not forwarded.

A few things to check:

check whether we can redirect stdin/out/err after we detect that host fingerprints were changed
check whether sending \n to stdin after close the session will help
check what will happen when we run fleetctl ssh unit1 unit2 unit3

Answer to 1st one:

askToTrustHost in ssh/known_hosts.go is referenced in NewHostKeyChecker in ssh/known_hosts.go

NewHostKeyChecker is referenced in getChecker in fleetctl/fleetctl.go

getChecker is passed into NewSSHClient or NewTunnelledSSHClient in ssh/ssh.go

NewSSHClient or NewTunnelledSSHClient are pass checker into sshClientConfig in cfg.HostKeyCallback config in ssh/ssh.go

HostKeyCallback is used in newClientTransport in golang.org/x/crypto/ssh/handshake.go and assigned to t.hostKeyCallback

t.hostKeyCallback (which is askToTrustHost in ssh/known_hosts.go) is called inside client function in golang.org/x/crypto/ssh/handshake.go

client is called by enterKeyExchange in golang.org/x/crypto/ssh/handshake.go

enterKeyExchange is called by readOnePacket in golang.org/x/crypto/ssh/handshake.go

readOnePacket is called in readLoop in golang.org/x/crypto/ssh/handshake.go

readLoop is called as go routine by newClientTransport (and newServerTransport, but we don't need it) in golang.org/x/crypto/ssh/handshake.go

newClientTransport is called by clientHandshake in golang.org/x/crypto/ssh/client.go

clientHandshake is called by NewClientConn in golang.org/x/crypto/ssh/client.go

NewClientConn is called by Dial in golang.org/x/crypto/ssh/client.go and NewTunnelledSSHClient in ssh/ssh.go

Dial is called by NewSSHClient and NewTunnelledSSHClient in ssh/ssh.go - is better to figure out why we call both Dial and NewClientConn inside NewTunnelledSSHClient. Probably we call NewClientConn twice (directly and by Dial)

NewClientConn is called by NewTunnelledSSHClient in ssh/ssh.go

NewTunnelledSSHClient is called by runRemoteCommand and runSSH in fleetctl/ssh.go

We need runRemoteCommand which is called by runCommand in fleetctl/ssh.go

runCommand is called by runStatusUnits in fleetctl/status.go

runRemoteCommand creates sshClient (*ssh.SSHForwardingClient) and passes it to the ssh.Execute in fleetctl/ssh.go

ssh.Execute is defined in ssh/ssh.go and it calls makeSession which actually sets stdin/out/err:

Answer to 3rd one:
bash expands fleetctl ssh<EMAIL_ADDRESS>into fleetctl ssh<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>so it tries to execute<EMAIL_ADDRESS><EMAIL_ADDRESS>on the server which runs<EMAIL_ADDRESS>

When you use pure bash without any promt string (ssh coreos1 bash), fleetctl status<EMAIL_ADDRESS>works fine.

Added debug of the stdin:
diff --git a/ssh/ssh.go b/ssh/ssh.go
index ca02dd9..32ae5df 100644
--- a/ssh/ssh.go
+++ b/ssh/ssh.go
@@ -18,6 +18,8 @@ import (
        "errors"
        "net"
        "os"
+       "io"
+       "fmt"
        "strconv"
        "strings"
        "time"
@@ -68,7 +70,12 @@ func makeSession(client *SSHForwardingClient) (session *gossh.Session, finalize

        session.Stdout = os.Stdout
        session.Stderr = os.Stderr
-       session.Stdin = os.Stdin
+       //session.Stdin = os.Stdin
+       stdin_file, err := os.OpenFile("/home/core/stdin.log", os.O_WRONLY|os.O_APPEND|os.O_CREATE, 0644)
+       if err != nil {
+               fmt.Fprintf(os.Stderr,"%v", err)
+       }
+       session.Stdin = io.TeeReader(os.Stdin, stdin_file)

        modes := gossh.TerminalModes{
                gossh.ECHO:          1,     // enable echoing
@@ -89,6 +96,8 @@ func makeSession(client *SSHForwardingClient) (session *gossh.Session, finalize

                finalize = func() {
                        session.Close()
+                       stdin_file.Close()
+                       fmt.Fprintf(os.Stderr,"state: %#v\n",oldState)
                        terminal.Restore(fd, oldState)
                }

when you use regular ssh stdin.log contains CR (0xOD) after ssh session is closed. when you use ssh coreos1 bash (which works fine) - stdin.log contains LF (0xOA).

Here is more simple solution fleetctl status<EMAIL_ADDRESS>> output_term1 2>&1:
Which provides this output with ssh coreos1
^[[1;32m●^[[0m<EMAIL_ADDRESS>   Loaded: loaded<EMAIL_ADDRESS>linked-runtime; vendor preset: disabled)^M
   Active: ^[[1;32mactive (running)^[[0m since Tue 2016-03-15 09:07:05 UTC; 16min ago^M
 Main PID: 12496 (bash)^M
   CGroup<EMAIL_ADDRESS>           ├─12496 /bin/bash -c while true; do echo Hello, World 1!; sleep 1; done^M
           └─16661 sleep 1^M
^M
Mar 15 09:23:20 coreos2 bash[12496]: Hello, World 1!^M
Mar 15 09:23:21 coreos2 bash[12496]: Hello, World 1!^M
Mar 15 09:23:22 coreos2 bash[12496]: Hello, World 1!^M
Mar 15 09:23:23 coreos2 bash[12496]: Hello, World 1!^M
Mar 15 09:23:24 coreos2 bash[12496]: Hello, World 1!^M
Mar 15 09:23:25 coreos2 bash[12496]: Hello, World 1!^M
Mar 15 09:23:26 coreos2 bash[12496]: Hello, World 1!^M
Mar 15 09:23:27 coreos2 bash[12496]: Hello, World 1!^M
Mar 15 09:23:28 coreos2 bash[12496]: Hello, World 1!^M
Mar 15 09:23:29 coreos2 bash[12496]: Hello, World 1!^M

ssh coreos1 bash provides this result:
●<EMAIL_ADDRESS>   Loaded: loaded<EMAIL_ADDRESS>linked-runtime; vendor preset: disabled)
   Active: active (running) since Tue 2016-03-15 09:07:05 UTC; 17min ago
 Main PID: 12496 (bash)
   CGroup<EMAIL_ADDRESS>           ├─12496 /bin/bash -c while true; do echo Hello, World 1!; sleep 1; done
           └─16970 sleep 1

Mar 15 09:24:23 coreos2 bash[12496]: Hello, World 1!
Mar 15 09:24:24 coreos2 bash[12496]: Hello, World 1!
Mar 15 09:24:25 coreos2 bash[12496]: Hello, World 1!
Mar 15 09:24:26 coreos2 bash[12496]: Hello, World 1!
Mar 15 09:24:27 coreos2 bash[12496]: Hello, World 1!
Mar 15 09:24:28 coreos2 bash[12496]: Hello, World 1!
Mar 15 09:24:29 coreos2 bash[12496]: Hello, World 1!
Mar 15 09:24:30 coreos2 bash[12496]: Hello, World 1!
Mar 15 09:24:31 coreos2 bash[12496]: Hello, World 1!
Mar 15 09:24:32 coreos2 bash[12496]: Hello, World 1!

@kayrus I'm not sure I follow all the notes, but you should also try to see if a PTY was allocated or not and the interactive mess here.. ?! not sure how the ssh Go implementation works either...

This code removes ^M, but terminal is still broken and requires extra key to start working.
gossh.ONLCR:         0,

https://github.com/madebymany/moltar/blob/master/sshclient.go - yet another ssh terminal implementation

The issue caused by unavailability to terminate io.Copy goroutine when you close session. It's being terminated only when user presses Enter key.

@jonboulle do you know how is it possible to interrupt io.Copy on demand?

I was able to interrupt io.Reader.Read call by closing the fd/0 stdin descriptor but only in small golang test. This solution doesn't work inside fleetctl code. Trying to figure out why.

I guess it's worth a try to test this solution


create associations

In this pull request, I set up the association of school library by following these steps:

[ ] Create a class Classroom with the following:
[ ] @label instance variable, should be initialized in the constructor.
[ ] Setter and getter for @label (remember about attr_accessor).
[ ] Create the has-many/belongs-to relationship between Classroom and Student. The following should be implemented:
[ ] Create the has-many side (a classroom has many students).
[ ] Create the belongs-to side (a student belongs to a classroom).
[ ] Make sure that when adding a student to a classroom it also sets the classroom for the student.
[ ] Make sure that when setting the classroom for a student it also adds it to the classrooms' students.
[ ] Create a class Book with the following:
[ ] @title and @author instance variables, should be initialized in the constructor.
[ ] Setters and getters for instance variables (remember about attr_accessor).
[ ] Create a class Rental with the following:
[ ] @date instance variable, should be initialized in the constructor.
[ ] Setter and getter for @date (remember about attr_accessor).
[ ] Create the many-to-many (also has-many-through) relationship between Person and Book using the intermediate class Rental. The following should be implemented:
[ ] Create the has-many side of Book and Rental (a book has many rentals).
[ ] Create the belongs-to side of Rental and Book (a rental belongs to a book).
[ ] Create the has-many side of Person and Rental (a person has many rentals).
[ ] Create the belongs-to side of Rental and Person (a rental belongs to a person).
[ ] Modify the constructor of Rental so Book and Person are set in it.

Thanks, @franclobo for approving my project. 👍👍👍


Add unit tests for move2kube-api

Use the go's unit test framework similar to https://github.com/konveyor/move2kube/blob/master/types/info/versioninfo_test.go and write unit-tests for move2kube-api.

I'd like to try on this, but I'm new to Golang, could you please give me some hint like which functions need a test function? I checked your sample and noticed that not all functions have a test one. Thanks.

@HarikrishnanBalagopal Can you give some high level guidance on how we could test this package?

I would like to work on this @ashokponkumar

I would like to work on this @ashokponkumar

Sure @nasirhm .

@ashokponkumar / @HarikrishnanBalagopal Looking at the types in move2kube-api, They're all available in a single types.go: https://github.com/konveyor/move2kube-api/blob/main/internal/types/types.go . Should i begin by writing unit test cases for this one ?

@ashokponkumar / @HarikrishnanBalagopal Looking at the types in move2kube-api, They're all available in a single types.go: https://github.com/konveyor/move2kube-api/blob/main/internal/types/types.go . Should i begin by writing unit test cases for this one ?

@HarikrishnanBalagopal Can you please look into @nasirhm 's question.

@ashokponkumar / @HarikrishnanBalagopal Looking at the types in move2kube-api, They're all available in a single types.go: https://github.com/konveyor/move2kube-api/blob/main/internal/types/types.go . Should i begin by writing unit test cases for this one ?

Hi @nasirhm , Thanks for looking into this issue.
https://github.com/konveyor/move2kube/issues/881
You can look at the above for some tutorials and steps on how to get started writing tests.
The types package you mentioned is a good place to start since it has small functions which you can write unit tests.
Please let us know if you get stuck.

@ashokponkumar  @HarikrishnanBalagopal can I work on this issue?

@HarikrishnanBalagopal @ashokponkumar i have raised a PR. Please have a look


Gateway specific extensions require adopting namespacing strategy?

We discussed having at every level an optional "extensions" ability to add arbitrary information that was not yet codified in the schema. These extensions may be used to specify additional information required by a particular Gateway. As a result, it would seem that the extensions should adopt a common namespacing strategy to avoid conflicts.
Perhaps when documenting how these extensions work you could give guidance as to a best practice for namespacing those extensions (a naming convention, or other).

Revisit if becomes problem.


[gulp] Run server in development mode

livereload
compile scss
watch changed files
compile all to app/public
server should use app only. All outside app should be considered as a development area.

Done:

compile all to app. Right now it's only html+css+js app, so no app/public required right now
server should use app only
compile scss

The only left feature is livereload.

The livereload separated to a dedicated issue: #17
Closing.


Button dosent seem to appear when OpenXR App is open

I wanted to try and install this addon, after i executed the Install-Layer.ps1 nothing seems to be changing
I tried:

Switching from SteamVR BETA to Stable
Dropping the folder into the driver folder of steam vr

Am i missing something?

Make sure you leave the folder where it is after you run Install-Layer.ps1, since it adds the current .dll file location to the Windows registry.
You can use OpenXR Explorer to check if the layer has been successfully installed. It should appear below the extension list.
https://github.com/maluoi/openxr-explorer

This is the list of layers at the moment

It's not there, so wither the script hasn't managed to install it, or it is pointing to the wrong folder. The script should openUAC and ask for admin privileges when you launch it, or it won't be able to write to the registry. To check if there is an entry to the wrong folder, you'll need to open regedit and got to \HKEY_LOCAL_MACHINE\SOFTWARE\Khronos\OpenXR\1\ApiLayers\Implicit, and check if there are any keys for it with the wrong location.
I'm planning on making an app to make it easier to install, I just wanted to get this out of the door first.

Ok so good news! I deleted some duplicate paths in the registry and now it succesfuly shows up in OpenXR Explorer. It still does not show up in the dashboard tough

I'm not sure how well this works with OpenVR to OpenXR translation layers, since it needs to use OpenVR to make the menu and get the camera data. If it's starting, it should write a log to %localappdata%\XR_APILAYER_NOVENDOR_steamvr_passthrough.log

Alright, i checked for that path and i do have a log
Here is the file:
XR_APILAYER_NOVENDOR_steamvr_passthrough.log

I think that log is from when OpenXR Explorer ran it. If it hasn't updated after that, the application likely isn't running OpenXR. DCS doesn't use OpenXR natively at least, so it needs something like the OpenXR branch of OpenComposite or the API layer won't work.

Oop... im stupid, I ran Blender in VRMode and now it all works. Im sorry TwT.
Thanks a lot for the help!

Thanks for trying it out though!

No problem! Also do you know if there is a way to use the chroma key override with apps that run OpenVR?

I'm not sure. It might be possible but OpenVR doesn't have an API layer interface, so it would be a ton of work at the very least.

I just looked at OpenComposite and (Since im very unlucky) it dosent support the app that i need to use it (VRChat).  There really dosent seem an app out there that can Chromakey OpenVR stuff and overlay it on top of the camera passthru... It seems weird though as i definitely saw someone do it on VRChat with a Quest Pro...

Thinking about it, it might be possible to do with a SteamVR overlay, since OpenVR allows you to access the compositor output images. I'll have to look into it at some point.

Alr! Let me know how it goes! contact me on my twitter (@LabeVROfficial) when you find something!

There aren't any applications that explicitly support it yet to my knowledge, but Open Brush and Blender should work fine with the override modes.
The layer should write a log to %localappdata%\XR_APILAYER_NOVENDOR_steamvr_passthrough.log. Could to post the contents of the log here?

There aren't any applications that explicitly support it yet to my knowledge, but Open Brush and Blender should work fine with the override modes.
The layer should write a log to %localappdata%\XR_APILAYER_NOVENDOR_steamvr_passthrough.log. Could you post the contents of the log here?

2023-04-07 18:46:52 +0200: XR_APILAYER_NOVENDOR_steamvr_passthrough layer (0.2.3) is active
2023-04-07 18:46:52 +0200: Using layer: XR_APILAYER_NOVENDOR_steamvr_passthrough
2023-04-07 18:46:52 +0200: Requested extension: XR_EXT_active_action_set_priority
2023-04-07 18:46:52 +0200: Requested extension: XR_EXT_dpad_binding
2023-04-07 18:46:52 +0200: Requested extension: XR_EXT_frame_composition_report
2023-04-07 18:46:52 +0200: Requested extension: XR_EXT_hand_joints_motion_range
2023-04-07 18:46:52 +0200: Requested extension: XR_EXT_hand_tracking
2023-04-07 18:46:52 +0200: Requested extension: XR_EXT_hp_mixed_reality_controller
2023-04-07 18:46:52 +0200: Requested extension: XR_EXT_palm_pose
2023-04-07 18:46:52 +0200: Requested extension: XR_EXT_win32_appcontainer_compatible
2023-04-07 18:46:52 +0200: Requested extension: XR_FB_display_refresh_rate
2023-04-07 18:46:52 +0200: Requested extension: XR_HTCX_vive_tracker_interaction
2023-04-07 18:46:52 +0200: Requested extension: XR_HTC_vive_cosmos_controller_interaction
2023-04-07 18:46:52 +0200: Requested extension: XR_HTC_vive_focus3_controller_interaction
2023-04-07 18:46:52 +0200: Requested extension: XR_KHR_D3D11_enable
2023-04-07 18:46:52 +0200: Requested extension: XR_KHR_D3D12_enable
2023-04-07 18:46:52 +0200: Requested extension: XR_KHR_binding_modification
2023-04-07 18:46:52 +0200: Requested extension: XR_KHR_composition_layer_depth
2023-04-07 18:46:52 +0200: Requested extension: XR_KHR_opengl_enable
2023-04-07 18:46:52 +0200: Requested extension: XR_KHR_visibility_mask
2023-04-07 18:46:52 +0200: Requested extension: XR_KHR_vulkan_enable
2023-04-07 18:46:52 +0200: Requested extension: XR_KHR_vulkan_enable2
2023-04-07 18:46:52 +0200: Requested extension: XR_KHR_win32_convert_performance_counter_time
2023-04-07 18:46:52 +0200: Requested extension: XR_MND_headless
2023-04-07 18:46:52 +0200: Requested extension: XR_VALVE_analog_threshold
2023-04-07 18:46:52 +0200: Requesting extension: XR_KHR_win32_convert_performance_counter_time
2023-04-07 18:46:52 +0200: Application OpenXR Explorer creating OpenXR instance...
2023-04-07 18:46:52 +0200: OpenXR instance successfully created
2023-04-07 18:46:52 +0200: Using OpenXR system: SteamVR/OpenXR : lighthouse

I'm not really sure where/what the button is supposed to look like - but if it's in the dashboard that Labe provided a screenshot of I can't see it there at least.

The button should have this icon:

The log will only show the last session, which in this case was OpenXR Explorer (having it open should also make the button appear). Try running the application you want to use and see if the log changes.
It is also possible for OpenXR applications to disable API layers, although I haven't seen any that do it.

Another issue is that some applications might require the code for the API layer to be signed. This seems to currently be an issue with games that use Easy Anti Cheat, like Ghosts of Tabor.
If anyone encounters an OpenXR application that doesn't run the API layer, I would appreciate it a lot if you post the name of the application.


release-21.1: changefeedccl: disallow changfeeds on regional by row tables

PR: https://github.com/cockroachdb/cockroach/pull/63542
From release notes:

Changefeeds will now fail on any regional by row table with an error, CHANGEFEED cannot target REGIONAL BY ROW tables: <table_name>. This is to prevent unexpected behavior in changefeeds until they offer full support for this type of table. [#63542][#63542] {% comment %}doc{% endcomment %}

cc @rmloveland and @kathancox, as I'm not sure how we're documenting this limitation.

This was fixed in https://github.com/cockroachdb/docs/pull/10310


How to add proxy and ssl certs while using the Search API? I am currently trying to access it via my corporate network.

in the below code snippet, I can't find any option to add proxy settings or ssl certs like how you do with requests library.
from serpapi import GoogleSearch
search = GoogleSearch({
    "q": "coffee",
    "location": "Austin,Texas",
    "api_key": "<your secret api key>"
  })
result = search.get_dict()

You don't need to add proxies and/or hack around SSL certifications. Everything is handled on https://serpapi.com/search end.


Update copy in website header

Currently, when you visit Women and Color, we have a big header under the navigation that says:

Find talented women and people of color available for speaking opportunities at tech-related events.

Please update the copy to:

Find talented women, people of color and LGBTQA+ professionals in tech to speak at your next event.

I can pick this up.

Please go ahead! 🙂

Hello, not sure if this issue has been resolved or not, if it has already been resolved then please ignore this post. I was able to change the banner to "Find talented women, people of color, and LGBTQA+ professionals in tech to speak at your next event" per your request. Here's a link to my GitHub repo. Please let me know if there is anything else I can assist you with. :)
https://github.com/famouswolfe/womenandcolor


Add local http_server tests to top level pytest.

Current dependencies on/for this PR:

main

PR #181

PR #182

PR #183   👈

This stack of pull requests is managed by Graphite.

General q - do the tests with only the http_client fixture need to be local, or can they be unit?

All of these run for all the clusters, even if it isn't used, the client is pinging a different type of server.


Horizontal scrolling not working in flutter web

Hi,
I try to display a logfile on my flutter web page. It is a few hundret lines long, and some lines itself are very long.
I try to make it scrollable in both directions. Here is my code:
if (snapshot.hasData) {
      String logFile = snapshot.data as String;
      String fileContentFormatted = preprocess(logFile);

      //make a scrollable singe child ListView
      var listView = ListView.builder(
        scrollDirection: Axis.horizontal,
        itemCount: 1,
        itemBuilder: (context, index) {
          return SelectableText(fileContentFormatted, style: GoogleFonts.sourceCodePro());
          //return Markdown(data: "some Text", selectable: true);
        },
      );

      var scrollbar = Scrollbar(child: listView, isAlwaysShown: true);

      return scrollbar;

If I leave out "scrollDirection: Axis.horizontal,":
The text scrolls vertical and the scrollbar is shown always.
I can perfectly scroll with the scroll whele
But, if I try to klick and drag the scroll thumb, I get an error:
"ScrollController attached to multiple scroll views."
If a line is to long and cannot be displayed in one line on the screen, it is continued at the next line.
But, here the strange behaviour:
If I add scrollDirection: Axis.horizontal like in the example above:
What does work:
The long lines in the text are not continued in the next line. As if I could scroll.
What does not work:
But scrolling does only work in vertical direction. Not in horizontal.
No scrollbars were displayed at all.
I can't drag to make horizontal scrolling.

Hi @Wissperwind
Can you please provide your flutter doctor -v and a minimal complete reproducible code sample?
Thank you

I thin I have forgotton to add a ScrollController. I will rethink my problem.


error message for missing main function in example is non-specific

$ cargo new missing-main
$ cd missing-main
$ mkdir examples
$ touch examples/foo.rs
$ cargo build
   Compiling missing-main v0.0.1 (file:///home/huon/tmp/missing-main)
error: main function not found
error: aborting due to previous error
Could not compile `missing-main`.

Running with --verbose does at least print the last command run (pointing the finger at examples/foo,rs), but it seems bad to have a raw error: printed like this.

Hm perhaps this is more of a compiler issue than a Cargo issue? Cargo currently does not interpret errors at all, it just ships them back to the front. Although one possible step Cargo could take is:
Could not compile `missing-main` at `examples/foo.rs`

(which seems plausible?)

Triage: this is still a problem, but I had to either cargo run --example foo or cargo test; looks like cargo build no longer builds examples by default.
The error messages still don't mention examples/foo.rs though, which is the problem.
$ cargo new missing-main
     Created library `missing-main` project
$ cd missing-main
$ mkdir examples
$ touch examples/foo.rs
$ cargo build
   Compiling missing-main v0.1.0 (file:///Users/carolnichols/rust/missing-main)
    Finished dev [unoptimized + debuginfo] target(s) in 0.19 secs

$ cargo build --verbose
       Fresh missing-main v0.1.0 (file:///Users/carolnichols/rust/missing-main)
    Finished dev [unoptimized + debuginfo] target(s) in 0.0 secs

$ cargo run --example foo
   Compiling missing-main v0.1.0 (file:///Users/carolnichols/rust/missing-main)
error: main function not found

error: aborting due to previous error

error: Could not compile `missing-main`.

To learn more, run the command again with --verbose.

$ cargo run --example foo --verbose
   Compiling missing-main v0.1.0 (file:///Users/carolnichols/rust/missing-main)
     Running `rustc --crate-name foo examples/foo.rs --crate-type bin --emit=dep-info,link -C debuginfo=2 -C metadata=799ee0edb02a8799 -C extra-filename=-799ee0edb02a8799 --out-dir /Users/carolnichols/rust/missing-main/target/debug/examples -L dependency=/Users/carolnichols/rust/missing-main/target/debug/deps --extern missing_main=/Users/carolnichols/rust/missing-main/target/debug/deps/libmissing_main-4976ac668ccc2721.rlib`
error: main function not found

error: aborting due to previous error

error: Could not compile `missing-main`.

Caused by:
  process didn't exit successfully: `rustc --crate-name foo examples/foo.rs --crate-type bin --emit=dep-info,link -C debuginfo=2 -C metadata=799ee0edb02a8799 -C extra-filename=-799ee0edb02a8799 --out-dir /Users/carolnichols/rust/missing-main/target/debug/examples -L dependency=/Users/carolnichols/rust/missing-main/target/debug/deps --extern missing_main=/Users/carolnichols/rust/missing-main/target/debug/deps/libmissing_main-4976ac668ccc2721.rlib` (exit code: 101)

$ cargo test
   Compiling missing-main v0.1.0 (file:///Users/carolnichols/rust/missing-main)
error: main function not found

error: aborting due to previous error

error: Could not compile `missing-main`.
Build failed, waiting for other jobs to finish...
error: build failed

I think this is resolved by rust-lang/rust#48706.


Test Recording files getting deleted after recording tests for ContainerInstance module

Description
Hi,
For ContainerInstance module, I created my branch on top of 'generation' branch, generated code from new swagger, added new tests and ran all tests with the command: pwsh test-module.ps1 -record. All tests ran fine but all previous recording files under test/ folder were deleted, and no new recording files were added. Can you please let me know how to resolve this issue?
    deleted:    Add-AzContainerInstanceOutput.Recording.json
    deleted:    Get-AzContainerGroup.Recording.json
    deleted:    Get-AzContainerInstanceCachedImage.Recording.json
    deleted:    Get-AzContainerInstanceCapability.Recording.json
    deleted:    Get-AzContainerInstanceLog.Recording.json
    deleted:    Get-AzContainerInstanceUsage.Recording.json
    deleted:    Invoke-AzContainerInstanceCommand.Recording.json
    deleted:    New-AzContainerGroup.Recording.json
    deleted:    Remove-AzContainerGroup.Recording.json
    deleted:    Restart-AzContainerGroup.Recording.json
    deleted:    Start-AzContainerGroup.Recording.json
    deleted:    Stop-AzContainerGroup.Recording.json
    deleted:    Update-AzContainerGroup.Recording.json

Hi @diyellap can you push your updates and share your branch?

Yes, all tests ran fine but no recording file was generated:
Tests completed in 1284.34s
Tests Passed: 58, Failed: 0, Skipped: 7,
Here's my branch: https://github.com/diyellap/azure-powershell/tree/spot-confidential-containers . I pushed my updates here along with the newly added test files. It was generated on top of generation branch not main branch.

<EMAIL_ADDRESS>@@.> @FNU @.>
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows
From: @.>
Sent: Friday, February 10, 2023 2:00 AM
To: @.>
Cc: Divya Bharadwaj @.>; @.>
Subject: Re: [Azure/autorest.powershell] Test Recording files getting deleted after recording tests for ContainerInstance module (Issue #1111)
@diyellaphttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdiyellap&data=05|01|diyellap%40microsoft.com|e24517f06de148a2462108db0b4d9f56|72f988bf86f141af91ab2d7cd011db47|1|0|638116200239278463|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=FzKnBsJYGOEtd53kWbXx3Gl5BmG3Uj%2BP5dk2nXW4Cuk%3D&reserved=0 Clean the recording files are expected when you re-record tests. But not sure why there are no new recording files generated. Can you be sure that you run the record test successfully.
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fautorest.powershell%2Fissues%2F1111%23issuecomment-1425544720&data=05|01|diyellap%40microsoft.com|e24517f06de148a2462108db0b4d9f56|72f988bf86f141af91ab2d7cd011db47|1|0|638116200239278463|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=uex4brLlGo%2BjtGQ4VdZAJqVgjJ9obmqvHq2u5PwuvmM%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAF3U6ZJLVR6B33PZQIW7323WWYGTLANCNFSM6AAAAAAUXM5U5A&data=05|01|diyellap%40microsoft.com|e24517f06de148a2462108db0b4d9f56|72f988bf86f141af91ab2d7cd011db47|1|0|638116200239278463|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=yi4FUfC%2B2Aqoqv%2Fd1m6g2Otfgxi%2FZabcT9wNkXDBlno%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

Closed it since the issue can not be reproduced after the developer created a new branch from the generation branch.


Reload component every time when update async component code in  development mode

When using code splitting, additional hot(module) are needed because the updated components are not reloaded. Every time we need to add hot(module) to track the changes of asynchronous components. This makes me feel very troublesome, so every time I call the loadable method in development mode, I'm going to reload the asynchronous components once by default.
This eliminates the need for additional hot(module) inside each asynchronous component.
Only one hot(module)(App)!

Oh this look very nice, I would like to try it before but if it works, I will definitely do it!
I was looking for a solution to avoid marking each sub component with hot, if your solution is working this is great! 🎉

Ping @theKashey

Brilliantly elegant solution. Just wondering why I never think about such easy way to handle things.
But it shall not work - according to React-Hot-Loader tests (https://github.com/gaearon/react-hot-loader/blob/master/test/proxy/static-property.test.js#L141) it will preserve the OLD value of loadingPromise.
It was made to keep the cache, people can store in static props. Actually - exactly this case.
The operation is legit only after RHL swap classes. So - just before .load - create an instance of loader.
 // development mode load every time
  if(process.env.NODE_ENV === 'development'){
    React.createElement(LoadableComponent);
    LoadableComponent.load()
  }

Then - should work. I hope.

Or even better React.createElement(LoadableComponent).type.constructor.load()

Maybe I didn't use static props before, so I didn't notice this.

That’s not about static props, but about RHL behaviour.

@theKashey you are wrong.
A load is sufficient because the loadable component is recreated each time the file is reloaded.

Now added thanks!

Look like it's a more thicky and far more easier that I thought.

.load is work cos you are just calling a function
it does nothing except import
by importing a new version of file, you are re-registering top level variables.
thus, on next render, the old rendered component will be replaced by a new one, leading all further render to a new path.
Profit.

No conditions on componentWillRecieveProps, no updating for the internal promise, nothing.
Just execute the file!
It's brilliant! 🥇 🥇 🥇 🥇

But now one count not”develop” a better code spiriting expirence.
It is literally disabled.


fix(constants.py): set timeout from env

🆙

:tada: This PR is included in version 1.12.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Database] Implement strategy for handling the scheme definition of db artifacts

For many development artifacts, like stored procedures, we need to remove the Schema references.
In the example below, the artifact definition is coming with the "MYSCHEMA".
This schema need to be removed.
For example:
PROCEDURE "MYSCHEMA"."hdb_view::OrderProcedure" ()
   LANGUAGE SQLSCRIPT
   SQL SECURITY INVOKER
   --DEFAULT SCHEMA <default_schema_name>
   READS SQL DATA AS
BEGIN
   /*************************************
       Write your procedure logic
   *************************************/

      SELECT * FROM "hdb_view::Item";
END

It does not matter if the definition is of HANA v1 or Hanav2, the schema need to be removed.

For that reason we need to introduce a XSK strategy environment variable, that will cover two case:

Automatically remove/ignore existing schema definition when processing the artifact
Manual. In this case an exception will be throw nad the user will have to handle manually the removal of the schemas

This strategy need to be applied for all DB artifacts that XSK handle

Environment variable to be introduced and the schema name to be skipped, when enabled.

This issue is closed as the project is no longer maintained


[721] CC can bulk update users with TechSource accounts

Context

https://trello.com/c/1ZdxZJaL/721-form-for-allowing-cc-to-confirm-the-creation-of-techsource-accounts

Changes proposed in this pull request

Adds pages to Computacenter area so they can bulk update users who have TechSource accounts
Add migration to add column on user to track if user has a TechSource accounts, defaults to false

Guidance to review

Login as a computacenter user
Should see section at bottom to bulk update users with TechSource accounts
On form entering blank or not one email per row raise validation error
Submitting the form should update found users to have a TechSource account

This error message is a little odd when what's entered is invalid:


exit command does not work

the command exit should stop Amy but that does not work.
This is Amy. Copyright (c) 2018 the Amy project authors. For help run with flag -h.
22:50:45.586 [main] INFO  d.u.iaas.amyassist.amy.core.Core - run
22:50:47.051 [main] INFO  d.u.i.a.a.c.p.PluginLoader - loaded plugin Amy plugin alarmclock with 9 classes
22:50:47.095 [main] INFO  d.u.i.a.a.c.p.PluginLoader - loaded plugin Amy plugin calendar with 5 classes
22:50:47.105 [main] INFO  d.u.i.a.a.c.p.PluginLoader - loaded plugin Amy plugin email with 4 classes
22:50:47.113 [main] INFO  d.u.i.a.a.c.p.PluginLoader - loaded plugin Amy plugin example with 8 classes
22:50:47.132 [main] INFO  d.u.i.a.a.c.p.PluginLoader - loaded plugin Amy plugin spotify with 12 classes
22:50:47.138 [main] INFO  d.u.i.a.a.c.p.PluginLoader - loaded plugin Amy plugin systemtime with 3 classes
22:50:47.143 [main] INFO  d.u.i.a.a.c.p.PluginLoader - loaded plugin Amy plugin weather with 5 classes
22:50:47.158 [main] INFO  d.u.i.a.a.c.p.PluginLoader - loaded plugin Amy plugin navigation with 9 classes
22:50:48.528 [main] INFO  d.u.i.a.amy.httpserver.Server - start the server

amy> 22:50:50.890 [main] INFO  d.u.iaas.amyassist.amy.core.Core - running
22:50:51.639 [Thread-5] INFO  d.u.i.a.a.c.s.r.SpeechRecognizer - [INFORMATION] :: Speech Recognition activated
exit
22:51:04.435 [Thread-1] INFO  d.u.iaas.amyassist.amy.core.Core - stop
22:51:04.435 [Thread-1] INFO  d.u.i.a.amy.httpserver.Server - shutdown the server
22:51:04.457 [Thread-1] INFO  d.u.iaas.amyassist.amy.core.Core - stopped

after the exit command Amy is still running

this bug must be fixed before #185 can be done

https://github.com/AmyAssist/Amy/blob/dev/core/src/main/java/de/unistuttgart/iaas/amyassist/amy/core/speech/tts/TextToSpeech.java#L204 is never called


Cannot Import Worlds in FTB Infinity Evolved

When trying to import a Spawn world:

I get an error:

My folder looks like this:

I'm playing FTB Infinity Evolved 1.7.10.
I'm not sure if its because it needs to be an FTB world or not. Help please :D

Have you tried importing in spawn in full lowercase?  Some aspects of the plugin are case sensitive.

As MrRiptide suggested, try using all lowercase for the world name, since your folder (world name) is all lowercase. Currently, it's best practice to have all world names to be lowercase as well. Closing this ticket; if this indeed an issue on the latest Multiverse version, please open another ticket.


Definitions for SemEval-2018 Task 9 Dataset

Hello, @VityaVitalich !
I'm having some troubles trying to reproduce your model's performance on SemEval-2018 Task 9 "1A: English" dataset after completing the fine-tuning procedure. Could you please help me clarify two points?
In your paper you mention that you achieved the best results by incorporating into prompts definitions for both train set (extracted from WordNet) and test sets. For the latter the definitions are said to be extracted from either WordNet or Wikidata or generated by Chat-GPT. You specify that you relied on external definitions in Taxonomy Enrichment and Lexical Entailment tasks, but you do not reveal what was your strategy for SemEval-2018. However, this dataset has no predetermined definitions and only two-thirds of its items are present in WordNet 3.1. Here arise two questions.

Did you actually use any externally acquired definitions in your Hypernym Extraction task?
If so, are these definitions available anywhere? In your Data folder you provide SemEval-2018 test subsets with no "child_def" field, and in the repo I found no clue how you could have filled it.

I'd be grateful if you could share your definitions or correct me if I misunderstood something
Sincerely,
Feudor

Hello, Feudor!
For SemEval-2018 Task 9, definitions were not used. The training setup and data are already structured in a way that allows you to infer hypernyms directly. However, I believe incorporating definitions could provide additional benefits.
We didn’t extract definitions for this task because the words are often situated within a specific domain, which LLMs can quickly understand. This is further supported by the observed quality improvements when few-shot learning or slight fine-tuning is applied.
Hope this helps!


Add rustc-serialize/serde Encoder/Decoder implementation

It would be nice to have a suitable Encoder/Decoder (or Serializer/Deserializer) implementation for Bson type in order to automatically serialize serializable types to BSON.

Sounds promising. But I don't have time to implement it recently, so I should mark it as "enhancement".
BTW, I am happy to receive a PR. :P


Feature/toml

This PR indroduces installation via pyproject.toml and version information via setuptools_scm.
Addresses issue https://github.com/experimental-design/bofire/issues/179.

Thanks Johannes. Is the file bofire/version.py  missing? Where do you define the current version?

Thanks Johannes. Where do you define the current version?

Ah, it reads the tag. Ok. Perfect.


is it like a builder where u bind it with the rat or what is it?

ngl i dont know how i should use it

figure it out yourself, if oyu wanna rat im not helping. just check main.py how it can be used.

figure it out yourself, if oyu wanna rat im not helping. just check main.py how it can be used.

alright


upstream 0.15.2

Adds support for:

private fields
static initializer blocks
non-decimal integers with an n suffix

I know @nebrius was recently working on this, too, but I think ran into an issue with private properties while testing it locally – curious if you were able to sort that out, Bryan? https://github.com/observablehq/lezer/pull/9

I know @nebrius was recently working on this, too, but I think ran into an issue with private properties while testing it locally – curious if you were able to sort that out, Bryan? #9

I never did figure it out before the holiday break started. If this works for you Mike, then let's go with this one.

Oh, sorry Bryan, I missed that you already started this. Here’s the diff in this PR compared to yours:
diff --git a/src/javascript.grammar b/src/javascript.grammar
index c9adf2a..c41dd48 100644
--- a/src/javascript.grammar
+++ b/src/javascript.grammar
@@ -1,4 +1,6 @@
 @precedence {
+  intersection @left,
+  union @left,
   else @right,
   member,
   newArgs,
@@ -109,7 +111,7 @@ MethodDeclaration[group=ClassItem] {
   pkwMod<"async">?
   (pkwMod<"get"> | pkwMod<"set"> | Star)?
   (PropertyDefinition | PrivatePropertyDefinition)
-  ParamList
+  functionSignature
   Block
 }

@@ -119,9 +121,7 @@ StaticBlock[group=ClassItem] {

 PropertyDeclaration[group=ClassItem] {
   pkwMod<"static">?
-  pkwMod<"readonly">?
   (PropertyDefinition | PrivatePropertyDefinition)
-  LogicOp<"!">?
   ("=" expressionNoComma)?
   semi
 }
@@ -201,7 +201,7 @@ ArrayExpression {
 propName { PropertyDefinition | "[" expression "]" | Number | String }

 Property {
-  pkwMod<"async">? (pkwMod<"get"> | pkwMod<"set"> | Star)? propName ParamList Block |
+  pkwMod<"async">? (pkwMod<"get"> | pkwMod<"set"> | Star)? propName functionSignature Block |
   propName ~destructure (":" expressionNoComma)? |
   "..." expressionNoComma
 }
@@ -215,8 +215,10 @@ ClassExpression {
   kw<"class"> VariableDefinition? (kw<"extends"> expression)? ClassBody
 }

+functionSignature { ParamList }
+
 FunctionExpression {
-  async? kw<"function"> Star? VariableDefinition? ParamList Block
+  async? kw<"function"> Star? VariableDefinition? functionSignature Block
 }

 UnaryExpression {

To summarize:

The readonly and ! are TypeScript features that should have been removed previously.
The introduction of functionSignature was added in https://github.com/lezer-parser/javascript/commit/2c69ee2ab55850b01f68021d62a3328a2aec82da; I could remove that since it was only needed for TypeScript, but keeping it makes this slightly easier to maintain.
The precedence changes were added in https://github.com/lezer-parser/javascript/commit/0740b29bd2d79036c9ff2679c9363564f0a1d17b. This also appears to be TypeScript related but not totally sure…

In what way did the other branch not work locally? I haven’t tested this too thoroughly.

To summarize:
* The `readonly` and `!` are TypeScript features that should have been removed previously since we don’t (yet) support TypeScript in Observable.

* The introduction of `functionSignature` was added in [lezer-parser@2c69ee2](https://github.com/lezer-parser/javascript/commit/2c69ee2ab55850b01f68021d62a3328a2aec82da); I could remove that since it was only needed for TypeScript, but keeping it makes this slightly easier to maintain.

* The precedence changes were added in [lezer-parser@0740b29](https://github.com/lezer-parser/javascript/commit/0740b29bd2d79036c9ff2679c9363564f0a1d17b). This also appears to be TypeScript related but not totally sure…

Ah yeah, these are good catches and good points.

In what way did the other branch not work locally? I haven’t tested this too thoroughly.

I was getting syntax errors on #private properties, saying # is invalid. I didn't get far in my testing before the day was up, so it could have been a mistake in how I wired in the dev version of lezer, or it could have been something related to the changes in your diff above.


ts-proto: add support for vendor/ type imports

See aperturerobotics/starpc#2
The PROJECT environment variable, set here to github.com/aperturerobotics/starpc for example, is based on the go.mod module.
The patch checks this variable in ts-poet, and remaps any package which has that prefix to "./{package}" and any without to "./vendor/{package}"
For example:

Makefile: https://github.com/aperturerobotics/protobuf-project/blob/starpc/Makefile#L76
Generated same-project import: https://github.com/aperturerobotics/protobuf-project/blob/starpc/example/example.ts#L5
Generated cross-project import: https://github.com/aperturerobotics/protobuf-project/blob/starpc/example/other/other.ts#L4

This should be implemented with a proper ts-proto feature which allows defining the prefix for the local project, and a prefix to append to any module path /not/ located in the current project.

Done with protobuf-go-lite.


Add decorator to tests which require internet connection.

Hello.
I found out that some tests newly require internet and therefore they have to be skipped in an environment without the internet connection.
The latest virtualenv bundles pip 10.0.1. Pip from version 10.0.0 has support for pyproject.toml files (PEP 518) which means that pip now installs build dependencies specified in pyproject.toml file automatically. And it doesn't work without internet obviously.
It would be nice if you can do a release so it would be possible for me to drop downstream patch. Currently, I had to create this patch twice - once for this PR and once for currently released version 0.20.0.
Have a nice day.
Lumír

Please squash the fixup commit during a merge.

Exactly the same here. RPM build in Fedora doesn't have access to the internet.

@frenzymadness , you may have noticed https://bugzilla.redhat.com/show_bug.cgi?id=1761091 .  Good to see you've already been working on trying to solve these problems.  I've added a few more PRs to userpath.  My userpath spec is still using master.tgz because of  https://github.com/ofek/userpath/issues/11 https://build.opensuse.org/package/show/home:jayvdb:py-new/python-userpath
I have hatch 0.20.0 packaged up reasonably well now, using userpath(master) instead of adduserpath, by adding some patches.
https://build.opensuse.org/package/show/home:jayvdb:py-new/python-hatch


chore(deps): update common-dev-assets digest to 4836ea0

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
cdaf79d -> 4836ea0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 2.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[dunfell][gatesgarth][hardknott][honister][kirkstone] {foxy} Upgrade to 2021-11-14 release

https://discourse.ros.org/t/new-packages-for-foxy-fitzroy-2021-11-04/22980

Merged to other branches as well:
7ba3392d5..445886090  dunfell -> dunfell-next
238082088..5f7a73878  gatesgarth -> gatesgarth
238082088..5f7a73878  gatesgarth -> gatesgarth-next
df92acc93..96bd501f6  hardknott -> hardknott
df92acc93..96bd501f6  hardknott -> hardknott-next
dc379cccc..b49e02524  honister -> honister
dc379cccc..b49e02524  honister -> honister-next
dc379cccc..b49e02524  master -> master
dc379cccc..b49e02524  master -> master-next


How to deal with Schwarz theorem?

In most cases $\partial_i\partial_j = \partial_j\partial_i$ this leads to multiple options:

we assume this is always the case (reasonable but might cause issues in the future) and deduplicate
we use an option parameter to decide whether to deduplicate or not (ugly)
we let the GP simulator down the line do the deduplication (which has access to the covariance function - with the information whether or not Schwarz theorem is applicable). This is probably the cleanest design, but it requires more work on the GP side

To some degree the GP side will have to do this deduplication anyway (especially if doing incremental evaluation - check whether a point was already simulated).

I would go for option 1, and document the assumption clearly. Once there are issues, look at the concrete example and use it to decide between 2 and 3.


'Show All' results leads to error

In the Explorer, if I go to e.g. Gestures and Allegories, select e.g. Bleaching and then click on 'Show All' at the bottom, I consistently get an error message: "An error occurred on the client." Tested in current Safari and Chrome on OSX. Try e.g. https://explorer.odeuropa.eu/smells?filter_type=text&filter_gesture=http%3A%2F%2Fdata.odeuropa.eu%2Fvocabulary%2Folfactory-gestures%2F7

Fix deployed


support k-quants for q3

Hi can it be assigned to me? The q2k e2e test seems need q3k quants.

Hi can it be assigned to me? The q2k e2e test seems need q3k quants.

sure, have fun!

IMHO this issue could be resolved, thank you @AuruTus !


Transactions Don't Work with MySQL

The project looks promising, and I was trying to use it for a production web application, but it appears that transactions aren't working with MySQL. I'm using PyMySQL.
From documentation, the migrations should always use transactions unless transactional is set to False. However, the following migration will generate an SQL error, but the test table is still created. This completely breaks the migration capability and manual intervention is required to repair the tables.
from orator.migrations import Migration
class CreateTables(Migration):
def up(self):
    with self.schema.create('test') as table:
        table.increments('id')
        table.string('name', 255)

    # Generate SQL error. Transaction should not commit.
    # Table 'test' should NOT be created.
    self.schema.drop('no_such_table')

def down(self):
    pass

I don't think migration will use transaction. And it is suggested to separate to different migration files for different table

AFAIK, MySQL transaction not support the DDL (data description language), but PostgreSQL supported.

Doesn't seem to work with sqlite either


Input type date issue

input type date looks ugly since it uses flex-direction column. My suggestion to put:
.form-control[type="date"] {
  flex-direction: row;
}

Will be fixed when #21716 lands.

Duplicate of #21651.


client-error event for transfert (user name undefined)

Hi Stewart,
It's really great that you added the concept of client error, I was just about to contact you about it 2 weeks ago. Without this possibility of capturing errors, the server was crashing when there were firewall issue or anormal connection breaks. Now it seems to works well. Whatever I saw that you modified a lot of code that may helped for crash issue....
While simulating connection failures I noticed that the client-error event is triggered twice. So one for the channel command and a second time for the data channel. However, the connection.userName property is valid in the channel command event, but for the data channel event it is undefined. Is this normal? I think the user name of a command channel connection should be preserved while opening the data channel connection.
Best regards,

Great catch!
Triggering twice may not be the best behaviour, I'll take a look.
As for the event, I was setting the connection value incorrectly, it should be:
this.dataSocket.on('error', err => this.server.emit('client-error', {connection: this.connection, context: 'dataSocket', error: err}));

This should ensure connection is set correctly and username will be available.

https://github.com/stewarttylerr/ftp-srv/pull/14/commits/37c3da3a624b82e768fbb2ea7cec98f559f2d897

Hey @purell, this should be fixed now!
Thanks


qa_crowbarsetup: adding format to timestamps

this makes the timestamp format similar to timestamps for start/end of
steps

#2248 looks better


Expose env-var properties accessible by stream and task applications

As a developer, I'd like to investigate how to expose environment variables, so it is accessible by Replication Controller and Boot, bi-directionally.
other items to consider:
The launched application would need to be provided a hint as to the service name in order to resolve the env-var key that could be of pattern: <servicename>_<namespace>_<host>
Acceptance:

Design review
I'm able to successfully discover and bind binders and analytic store by env-var's

It is the other way around.  The replication controller can expose variables which are well known boot properties or can mount a secrets file system with key-Value pairs (in turn exposed as env-vars) that are also well known boot properties.  Can investigate customizing the replication controller to expose specific variables first and prob a separate story , look to use a different strategy for the connector library that does env-var translation from service discovery env-vars that k8s automatically provides.

This relates to spring-cloud/spring-cloud-dataflow-admin-kubernetes#31

No longer relevant AFAICT, we have a way to pass env vars via deployer properties


Minor: fix the unused arguments used by formatting string

Both arguments are string type so the specifier should be %s.
Committer Checklist (excluded from commit message)

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

@chia7712 does this look OK to you?

does this look OK to you?

LGTM :)

retest this please

retest this please

Merged to trunk, 2.5 and 2.4