Problems adding integration

System Health details
System Information

version
core-2023.12.1

installation_type
Home Assistant OS

dev
false

hassio
true

docker
true

user
root

virtualenv
false

python_version
3.11.6

os_name
Linux

os_version
6.1.63-haos

arch
x86_64

timezone
Europe/Berlin

config_dir
/config

Home Assistant Community Store

GitHub API
ok

GitHub Content
ok

GitHub Web
ok

GitHub API Calls Remaining
3298

Installed Version
1.33.0

Stage
running

Available Repositories
1426

Downloaded Repositories
87

Home Assistant Cloud

logged_in
false

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Home Assistant Supervisor

host_os
Home Assistant OS 11.2

update_channel
stable

supervisor_version
supervisor-2023.11.6

agent_version
1.6.0

docker_version
24.0.7

disk_total
121.3 GB

disk_used
45.9 GB

healthy
true

supported
true

board
ova

supervisor_api
ok

version_api
ok

installed_addons
Samba share (12.2.0), FTP (4.7.3), Terminal & SSH (9.8.1), Glances (0.20.0), AppDaemon (0.16.0), File editor (5.7.0), Home Assistant Google Drive Backup (0.112.1), Log Viewer (0.16.0), MariaDB (2.6.1), ZeroTier One (0.17.3), phpMyAdmin (0.9.0), Traccar (0.23.1), SQLite Web (4.0.0), Studio Code Server (5.14.2), Zigbee2MQTT (1.34.0-1), Advanced SSH & Web Terminal (17.0.0), Portainer (2.19.4), room-assistant (2.20.0), ESPHome (2023.11.6), Mosquitto broker (6.4.0), InfluxDB (4.8.0), EnOcean MQTT (dev) (0.1.28), Prometheus Node Exporter (0.7.0), Grocy (0.20.1)

Dashboards

dashboards
7

resources
53

views
54

mode
storage

Recorder

oldest_recorder_run
3. Dezember 2023 um 20:14

current_recorder_run
13. Dezember 2023 um 13:01

estimated_db_size
4587.04 MiB

database_engine
sqlite

database_version
3.41.2

Checklist

[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of any previous issues..

Describe the issue
I have added you integration using HACS
When I add
battery_notes:   enable_autodiscovery: true
in my configuration.yaml and reboot HA I get an error that the integration does nocht support yaml.
Reproduction steps

Add integration using HACS
add configuration.yaml
Reboot HA
...

Debug logs
No log needed

Diagnostics dump
No response

Are you using the beta version of Battery Notes (version 1.1.1?)
If not you do not need to add that to the configuration.yaml
Also, enable_autodiscovery: true should be on a line below battery_notes and indented.

I am using 1.0.1 and the I have enable_autodiscovery: true in a second line. the text is in wrong format.
How do I achieve the autodiscovery?
Thank you!

autodiscovery was only released in beta yesterday, I've just added a comment to the readme to say this.
If you wish to try this version please go into HACS, select Battery Notes and in the top right menu choose Redownload, switch on Show Beta Versions and select the latest version (1.1.1 currently or the latest)
Then restart HA
Otherwise remove the lines from the configuration.yaml for now until 1.1 is released fully.

Thank you. I have switched to 1.1.1. Now it is working


DataObject::getChangedFields() / isChanged() don't work in onAfterPublish

Affected Version
4.4
Description
Wasn’t sure which repo to open this against, as I’m not really sure whether this is a bug in versioned or framework...
This is occurring against pages, I’ve not tested versioned DataObjects:
public function onAfterPublish()
{
    parent::onAfterPublish();

    if ($changed = $this->getChangedFields(false, DataObject::CHANGE_VALUE)) {
        var_dump($changed);
        die;
    }
}

Change whatever fields you like, the above var_dump() will never be triggered as the changed fields array is always empty. I guess something must be being reset before the hook is called...
onAfterWrite() works as expected, so a workaround for now is to use that to set flags that can then be called upon in onAfterPublish()

AFAIK getChangedFields is not versioned API. And also, I'm not sure this is a bug - I always expected getChangedFields is used with onBeforeWriteoronAfterWrite. I think it sort of "resets" the changed state at the very end of onAfterWrite`. For instance this means:
public function onAfterWrite()
{
    $changed = $this->getChangedFields(); // Shows changed fields
	parent::onAfterWrite();
	$changed = $this->getChangedFields(); // Shows no changed fields
}

I haven't tested this, just how I thought it was supposed to work.

The changed fields are still present in onAfterWrite(), even after parent::onAfterWrite() has been called.
I guess this happens because when using the “publish” workflow, ChangeSet or ChangeSetItem calls DataObject::write() and then calls onAfterPublish() once that has completed, so what happens is:

DataObject::write()

onBeforeWrite() - changes are present
onAfterWrite() - changes are present
Changes are reset

onAfterPublish() - no changes present

This might be a deceptively difficult one to solve...

onAfterPublish takes the $original instead. You should compare the field values on this object and the current object instead. You could do a toMap() and manually diff the array of all fields if you need the list of changes.
It's not really a bug; You simply need to use another approach . :)

Ah yeah that’s a good alternative, let’s close this then 😄


Adding Facebook ID to manifest meta-data causes compile error

This seems to be related to Firebase UI module.

This is an old issue.


PE Data Value Issue When Converting From PC Worlds

When importing a schematic from PC to PE, the data values of some blocks are lost, these include but are not limited to, Slabs, Double Slabs, Command Blocks, Hoppers, Concrete, Etc. This has already been reported in issue #800, but since then, now I know it occurs to blocks with multiple data values. It mainly occurs with data values 1, and 10-15.  The weird thing is that this doesn't occur when transferring PE schematics to PE, only from PC to PE. This happens on all OSes and has been happening even before the new PE support. I know porting isn't a native feature yet, but this seems like a side effect of a bug as @neonerz said. Thanks! 😄
Edit: If you want me to close this issue and put it in #800 instead, just tell me.

I think it's a good thing to reopen #800, and use it to track PE/PC DV differences.
This will avoid data/discussion spreading.
You can also rename #800 :smile:


Switch to service account

Switch the pre-release registry away from @GregBrimble's personal account to use a proper service account (cc @workers-devprod)

On hold pending internal domain changes


it will crash when  given a wrong image uri

it will crash when  given a wrong image uri .

I find this bad practice as well. I read my uri's from a remote db, they could be invalid. Now I will have to validate each string before.


Allow user to retrieve raw PointData buffer

I separated the loadPointData function into two functions, so that the user can either get a View or a Binary object.

OBE via #5.


Process undefined children for Translate component

🐛 Bug Report (Proposal)
Prerequisites

[x] I'm using the latest version of Docusaurus. (latest stable, 2.0.0-beta.5)
[x] I have tried the npm run clear or yarn clear command.
[x] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[x] I have tried creating a repro with https://new.docusaurus.io
[x] I have read the console error message carefully (if applicable)

Description
TLDR: On my opinion, Translate component should be able to process undefined children, if we're sure that we have enough i18n dictionaries for it
But after 2.0.0-beta.0 this behavior has changed - there is appeared strange invariant check. that children always should be string

Yeah, it shouldn't be ReactNode, but we haven't to specify this children always!

Have you read the Contributing Guidelines on issues?
Yes
Steps to reproduce
Use https://new.docusaurus.io to create a CodeSandbox reproducible demo of the bug.

Sorry, too much for my proposal

Not pass children in <Translate /> component from @docusaurus/Translate (another words - children === undefined

Try to launch page with this

Expected behavior

Render without crashing
There no need to specify default value for translation, if you have only two locales with two i18n-dictionaries

Actual behavior

Render with crashing

If I try to define as <Translate id="...">FOO_BAR</Translate> or <>{translate(...)}</> - this will work, but its crazy approach in this case :/

Your environment

Public source code: REPO | Described code fragment | i18n dictionaries for locales (only ru, en)

Public site URL: http://feature-sliced.design/

Docusaurus version used: 2.0.0-beta.5

Environment name and version Firefox 91.0.2 (64-bit), Node.js v12.18.0:

Operating system and version Windows 10:

Reproducible demo
Use https://new.docusaurus.io to create a CodeSandbox reproducible demo of the bug.

Sorry, too much for my proposal, u can see at point public source code

So, what I understand is that you want this to work despite not using default translation message in your code?
Then, what should be the behavior of the docusaurus write CLI command, that is supposed to extract these messages from your code? Should we extract the message ids instead, as the code does not contain any message?

Thanks for quick response,
from @feature-sliced core-team

(1) you want this to work despite not using default translation message in your code?

Yeap, at the moment our team maintain only two locales (ru, en), with one default (ru)
And the main problem was that docs for default locale should be located out from i18n (usually in /docs/**)
But other locales placed in a completely different place ("purely scattered at project")
Because of it - our idea was to place both locales closely with each other - for explicitness, discoverability and more easier maintaining
(another point is that we should specify default value in JSX, bot other values we should specify at json-dictionaries - "purely decoupled too")

(2) Should we extract the message ids instead, as the code does not contain any message

Yeah, at least for us, it would be preffered behaviour
But the final solution is yours, because this is question yet about compatibility and TMTOWTDI
Our team's opinion, it's not very good practice to scatter i18n-logic between JSX-code and json-dictionaries
(because of it - we don't specify translations on code)

And the main problem was that docs for default locale should be located out from i18n (usually in /docs/**)

That's not totally true.
The default translations you have in your code can be "examples" so that it's easier to understand what a good translation message can look like. You can still provide the real translations in i18n/ru and it would override the Russian translations found in code.
But I understand it may also not be ideal to have duplicate labels in 2 places and may be confusing for contributors.

Yeah, at least for us, it would be preffered behaviour

Your use-case makes sense to me, and it shouldn't be too hard to add support for this.

The default translations you have in your code can be "examples" so that it's easier to understand what a good translation message can look like. You can still provide the real translations in i18n/ru and it would override the Russian translations found in code.

Yeah, we are aware of this, but the same purpose "to help translators with additional context" could be reached by dictionaries[item-index].description field, couldn't it?))

But I understand it may also not be ideal to have duplicate labels in 2 places and may be confusing for contributors.

Exactly 👍

Your use-case makes sense to me, and it shouldn't be too hard to add support for this.

Thank you for understanding  🚀


fixchange lag/lead function signature (#40060)

pick from master #40060
remove CharType from the signature
What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Release note
None
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[ ] No.
[ ] Yes.

Does this need documentation?

[ ] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall


created md file in new branch my-bio

Checklists
General Checks

[x] the branch is up to date with main/master
[x] the code works when pulled and run locally
[x] All CI checks pass
[x] all conflicts are resolved (if any)
[x] PR has a descriptive title
[x] PR has appropriate labels and milestones for easy identification
[x] PR it is assigned to the owner
[x] reviewers are assigned
[x] the PR contributes only one focused change
[x] It is in the appropriate column in the project board (if necessary)
[x] has short and clear description
[x] is linked to an issue (if it is related)
[x] feedback is addressed (if any and if it is appropriate feedback.)

Markdown

[x] the markdown source is formatted
[x] spelling and grammar is correct in all text
[x] The markdown looks correct when you preview the file
[x] all links and images work

Do the npm run sctipts, that mentioned in package.json. Maybe then the lint won't argue

Do the npm run sctipts, that mentioned in package.json. Maybe then the lint won't argue

no you have to edit the README file first. because  i am adding some formats to it by this pr. so resolve the conflict and merge this pr. we cant edit on it because of the restriction.

I have solved the conflict, but lint still argue. What could you do?

leave it for now. After every one pushed their commit we will edit it with the master branch. for now i see many conflicts


Drag and drop support

 Issue by SirCmpwn
Friday Dec 23, 2016 at 17:07 GMT
Originally opened as https://github.com/Cloudef/wlc/issues/213

 Comment by Timidger
Friday Dec 23, 2016 at 18:47 GMT

Will patches be needed for both wayland -> Xwayland and Xwayland -> wayland (eg, similar to this patch and this other patch?) or is only one required?
From my testing it seems like wayland apps sometimes can access my copy buffer, so I'm wondering if one of the directions has already been implemented.

 Comment by SirCmpwn
Friday Dec 23, 2016 at 19:38 GMT

Clipboard is unrelated to this.

 Comment by SirCmpwn
Friday Dec 23, 2016 at 19:38 GMT

But yes.

 Comment by Timidger
Friday Dec 23, 2016 at 19:48 GMT

Ah my mistake, mis-read the feature. At least I linked the right patches.


fix(tools): update github actions and workflows

This PR aims to:

[x] Remove the merge-conflict action. Will need a different approach for that action.
[x] Adds a Unit Tests action (to transition away from Travis).
[x] Have some uniformity on the Lighthouse CI and the Cypress Tests actions.

Note: This PR was intentionally not made from my fork to avoid the pitfalls of GitHub and be closer to the real setup.

Tested that a job can prevent further runs, bringing parity with Travis.

I wanted to add on a wait on checks action in there but it seemed a bit unreliable. This should be good to once the run is green.
I will remove the Travis integration after a few days, if the actions work well.


Test Windows output of python reference library

Need to verify the output of the python reference library looks correct in Windows, because of funny interactions between the universal newline modifications that open() and csv.DictWriter() do independently. If there's a blank line between rows, then just adding lineterminator='\n' to the csv.DictWriter() init should be sufficient. I think.

 U is deprecated and default.
It has to do with the csv writer adding it's own line endings based of os.linesep. On Windows, csv it'll append \r\n to the end of line, but then open will try to be clever and replace \n with \r\n as well, making the EOL \r\r\n (Mac EOL + Windows EOL).

Seems fine on Win10 with the lineterminator='\n' change.


Latest version doesn't compile on arm-thumbv6m targets

This is due to the dependency on bytes. The workaround (using nightly compiler and patching their code) isn't really acceptable IMO.
See https://github.com/tokio-rs/bytes/issues/461

Thanks for opening this issue, @henrikssn!
This is the second serious issue related to bytes (see #33), and it seems like this one isn't a straight-forward fix. I'm open to suggestions, but maybe the solution is to revert #28. For now, I recommend using version 0.3 (which doesn't depend on bytes) as a workaround.
cc @xoac

Thanks for opening this issue, @henrikssn!
This is the second serious issue related to bytes (see #33), and it seems like this one isn't a straight-forward fix. I'm open to suggestions, but maybe the solution is to revert #28. For now, I recommend using version 0.3 (which doesn't depend on bytes) as a workaround.
cc @xoac


[SPARK-34666][SQL][TESTS] Test DayTimeIntervalType and YearMonthIntervalType as ordered and atomic types

What changes were proposed in this pull request?
Add DayTimeIntervalType and YearMonthIntervalType to DataTypeTestUtils.ordered/atomicTypes, and implement values generation of those types in LiteralGenerator/RandomDataGenerator. In this way, the types will be tested automatically in:

ArithmeticExpressionSuite:

"function least"
"function greatest"

PredicateSuite

"BinaryComparison consistency check"
"AND, OR, EqualTo, EqualNullSafe consistency check"

ConditionalExpressionSuite

"if"

RandomDataGeneratorSuite

"Basic types"

CastSuite

"null cast"
"up-cast"
"SPARK-27671: cast from nested null type in struct"

OrderingSuite

"GenerateOrdering with DayTimeIntervalType"
"GenerateOrdering with YearMonthIntervalType"

PredicateSuite

"IN with different types"

UnsafeRowSuite

"calling get(ordinal, datatype) on null columns"

SortSuite

"sorting on YearMonthIntervalType ..."
"sorting on DayTimeIntervalType ..."

Why are the changes needed?
To improve test coverage.
Does this PR introduce any user-facing change?
No
How was this patch tested?
By running the affected test suites.

Test build #135880 has started for PR 31782 at commit 7703910.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/40463/

Test build #135880 has finished for PR 31782 at commit 7703910.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/135880/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/135882/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40465/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40465/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/40465/

Test build #135883 has started for PR 31782 at commit 049edf6.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/40466/

@cloud-fan @yaooqinn @HyukjinKwon Could you review this PR, please.

thanks, merging to master!

+1


Cropped Image is looking blur?

When I crop image and use it's base 64 generated text in image src, result image is blur. How can we resolve it?

+1. I am also facing the same issue. Also, the image I crop is square always even though I have set parameters as 4:3 ratio. Below are my parameters.
this.cropperSettings = new CropperSettings();
this.cropperSettings.width = 150;
this.cropperSettings.height = 113;
this.cropperSettings.canvasWidth = 500;
this.cropperSettings.canvasHeight = 500;
this.cropperSettings.minWidth = 10;
this.cropperSettings.minHeight = 10;
this.cropperSettings.rounded = false;
this.cropperSettings.keepAspect = true;
this.cropperSettings.cropperClass = 'hideImageCanvas';
this.cropperSettings.croppingClass = 'showImageCanvas';
this.cropperSettings.cropperDrawSettings.strokeColor = 'rgba(255,255,255,1)';
this.cropperSettings.cropperDrawSettings.strokeWidth = 2;
this.cropperSettings.compressRatio = 10.0;
this.cropperSettings.preserveSize = false;

Am I doing anything wrong?


[TelegramClass.connect] Invalid token - typo bug in code

with last commit i have an issue with TelegramTask
telegram_token is in self.config, not in self.bot.config.telegram_token
I modified
self._tbot = telegram.Bot(self.bot.config.telegram_token)
in
self._tbot = telegram.Bot(self.config.get('telegram_token'))
anyway, please add an exception message in function like this:
try:
   self.update_id = self._tbot.getUpdates()[0].update_id
except IndexError:
   self.update_id = None
   logger.error("can't create a telegram bot")

and a "seat belt"
if self.tbot:
   self.tbot.sendMessage(chat_id=master, parse_mode='Markdown', text=msg)
else:
   print "telegram in null?!!?"

I just downloaded Telegram for my android phone, How can i find the token?
Thank you! first time using this.

Here for create bot and get telegram_token
https://core.telegram.org/bots#6-botfather

Add new bot in telegram chat

Ask a telegram your chat_id (aka master in config.json)
https://api.telegram.org/bot/getUpdates

Set master and token_id in config.json

if you wont add a custom methods for bot, add these in telegram_handler.py like this:

...
elif event == 'bot_start':
   msg = "ok...stay sitting. I'm going to play"
...

good luck friend! :D

Just got my token. But I cannot find my chat_id. I tried /getUpdates command.

you need to set "master" to either a numeric value (-> I don't know how to find out the correct value...), or to "@username" (which starts to send notifications after you sent the initial "/info" to the bot).
If this does not work still, post your config (use pastebin or similar, please) and I will look into this.

To get the numeric value, just send a message to the bot and it will write to the log output something like "message received from username (userid): messagetext".
You can also send a message to @get_id_bot in Telegram and it will tell you your id.

para çekme işlemi yapacağım ama üyelığımı onaylamam gerekıyormus  canlı destek cevap vermıyor nasıl yapabılırım

Token please


Add support of coboldoc

COBOLDoc is an auto-documentation markup and command-line tool used to annotate COBOL source code, a successor of JSDoc, PHPDoc and Javadoc industrial standards.
Please let programmers quickly generate the comment-blocks in COBOLDoc format by using keyboard shortcuts (like /**ENTER in JetBrains IDEs). Of course, implementation of the comments must be considered in each dialect.
Example:

sources: https://github.com/OlegKunitsyn/core-string/blob/master/src/string.cbl
auto-documentation: https://github.com/OlegKunitsyn/core-string/blob/master/coboldoc/string.cbl.md

Micro Focus uses the Microsoft standard xmldoc document syntax, so this will need to considered as well.
I think veryant also use a different format.

From veryant/isCOBOL docs:

Next, ROBOTDOC has support for COBOL and it looks like it was added by "Brian Tiffin" which I believe worked on GnuCOBOL.
https://github.com/gumpu/ROBODoc
https://rosettacode.org/wiki/Documentation#COBOL
So far we have four different mechanisms... two for GnuCOBOL, one for Micro Focus, one for isCOBOL

Actually we have 2 that may be used in any dialect (the free ones, used for example in GnuCOBOL, that are created outside of the compiler), one (MF) that may only work there (@spgennard can you please post a sample like you did for isCOBOL?) and one that only works with a specific dialect with a specific output format.

This is a complete example but it gives you an idea.
       *>  for more examples see https://docs.microsoft.com/en-us/dotnet/csharp/codedoc
       *>> ... what ever
       *>>
       *>> <example>
       *>>   Insert example on how to use the class here
       *>>
       *>>    declare a as ilexample
       *>>    display a.MyProperty
       *>>
       *>> </example>
       class-id ilexample public.

       *>> <value>Get MyProperty</value>
       01 MyProperty       public string.
       *>> A constructor for this class
       *>> <remark>This is an example</remark>
       method-id new public.
       procedure division.

           exit method.
       end method.

       *>> method sample
       *>> <remarks>
       *>>   <para>Some information</para>
       *>>   <para>... and again </para>
       *>> </remarks>
       *>> <returns>something...</returns>
       *>> <exception cref="member">description</exception>
       *>> <param name="a-name">a name of something</param>
       *>> <see cref="other" />
       *>> <seealso cref="elsewherfe" />
       method-id sample.
       linkage section.
       01 a-name   pic x(10).
       procedure division using by reference a-name.

       end method.

       end class.

Fyi.. ground work in progress

From veryant/isCOBOL docs:

Can you link to the document, please? isCOBOL documentation and samples in the trial distribution does not anything about.

see https://support.veryant.com/resources/isCOBOL2009_releaseoverview.pdf

Okay, got to a reasonable place to release it.. if you have coboldoc installed.. try using "right mouse"/show doc when editing.. for example:

(must be using 6.7+)

Nice! Is it possible to add comment-blocks by using keyboard shortcuts? It was a primary goal.

What keystroke are you thinking of?
And what would the contents be?

And what would the contents be?

That's too "on-site-specific", and likely also "per-project". Therefore I'd suggest using a template ".tmpl"  that contain whatever the local template it - which also takes care of different users using different COBOL documentation tools.
I'd suggest to have the template names fixed "per use" (see below) and search if there's a template next to the file (and if not then all the way up to the workspace root), do an exchange of some predefined variables (like program/function/(section/paragraph name, author, USING items) and then insert it to the appropriate place.
In any case there would be the need of at least two templates:

one for the program/function - containing at least the PROCEDURE DIVISION USING items
one for sections/paragraphs
possibly one for variables

You can create per-workspace snippets and assign a keybinding to the snippet.

https://code.visualstudio.com/docs/editor/userdefinedsnippets
https://stackoverflow.com/questions/44312494/how-to-create-per-workspace-snippets-in-vscode

Sorry for misunderstanding.... Shortcut to coboldoc generate src/* -o doc in the terminal is not a problem.
Developers hate writing documentation, make typos and forget to update. The benefit is to save time on creating and updating of the doc-blocks, delivering a minimal documentation for COBOL software:

file-descriptor, on the top of the file
program-descriptor, on the top of the program, by taking into account Linkage section
function-descriptor, on the top of the function, by taking into account Linkage section

There is nothing new.
Keystrokes
Does not matter which one to choose.
JetBrains
https://www.jetbrains.com/help/idea/working-with-code-documentation.html#add-new-comment
Eclipse
http://www.eclipseonetips.com/2010/05/05/add-comments-and-javadocs-in-eclipse-with-a-single-keystroke/
Visual Studio
https://docs.microsoft.com/en-us/visualstudio/ide/reference/generate-xml-documentation-comments?view=vs-2019
Formats
Formatting standards of the doc-blocks are dialect-dependent and partially described in https://github.com/brunopacheco1/coboldoc i.e. MSDN-like for Visual COBOL / Raincode COBOL, Javadoc-like for isCOBOL
Actual format of a file could be chosen by https://github.com/spgennard/vscode_cobol#shortcuts
For dialects without documented format i.e. GnuCOBOL, ICOBOL, Enterprise COBOL, I suggest default https://github.com/brunopacheco1/coboldoc#comment-blocks which is Javadoc-like.
Freedom of mixing several standards in a project leads to additional entropy and considered counterproductive from the Quality Assurance perspective. Ordnung muss sein :)

It looks like "COBOLDoc" project has stalled with a lot of outstanding issues.
I will leave the existing support in place but if the COBOLDoc project does not progress in the next couple of weeks, I will look at the pulling the support as it is not functional enough to be good enough for the "real" world applications (sadly).

Let me comment given bullet points one by one:

lot of outstanding issues

All those issues are enhancements.

does not progress

There is no deadlines in a community-driven development.

not functional enough

No bugs open. The tool is up and running in my projects (at least).

COBOLDoc state

Not archived. What is your SLA? Version incrementing on weekly basis is not a problem.

Do you intend to work on its issues?

I'm a contributor, too. I've contacted alive COBOL vendors collecting doc-blocks standards and code samples and put the finding in the topic. I have an idea to rewrite cli to yarn, because I personally like it :) but there is no real reason for that.

leave the existing support

Feel free. The goal of this topic was to add auto-documentation to COBOL sources but not an integration with an external tool.

I consider the issues such as the lack of support of classes, methods, entry points and other language structures as a part of the normal use of COBOL, therefore I would say they are bugs.
Support for the doc comments and the viewer go hand in hand, if one goes there is no point in promoting something that is unfortunately incomplete.
I think I understand your view point and I wish you well.

For the record, the auto doc was functional but not as refined as I would like it.. here is the screenshots of it in action.


Allow users to switch between runtime, avg. and peak memory for all tests at once

Gaining a general overview of the historical performance of all tests on any metric that is not the default (i.e., anything but wall time) is virtually impossible because you have to change the view for each test individually. It would be great if we could do this for all the tests with just one click.

I generally support this feature request but I'm hesitant to invest a lot of time in the existing dashboard. The implementation feels a little brittle and I'm not entirely convinced that the technology choice was the best all things considering.
If somebody knows their way around altair, I'd be happy to see this change but I wouldn't want us to invest a lot of time in this.


FYI: wasm-compiled tengo playground running in a browser

In case you may find it useful, here's a version of the Tengo playground running entirely in a browser and deployed as a static website:
https://tengo.mkuznets.com
https://github.com/mkuznets/tengo-wasm
The WASM blob is ~5.5 MB (1.4 MB when compressed).

Oh this is so cool! Just curious, did you have any particular use cases? Regardlessly this is really exciting to see.

I initially just wanted to run a version of the playground with the latest Tengo (the currently deployed one is missing some new stdlib methods). I wondered if I could make it work without running an API :-)


[qa] Find out why parallel tests fail

This fails:
./runtests.py --parallel --keepdb

We should find out why and how to fix it.
We can take advantage of this chance to also suggest to run tests this way in the README to help contributors to run tests faster.

Fixed by 8d3de2f696700d9a907dff70526c7a68a48fd01a


Accept Keystore password from file or environment variable

Hello There,
Our Infosec team is not happy about passing keystore password as a CLI argument --storepass=PASS  as its visible to everybody on  executing command ps, so I wonder if --storepass  can accept  a password in a file --storepass=PATH  or environment variable--storepass=$PATH .
  --keystore=PATH               Path to keystore (combined PEM with cert/key, or PKCS12 keystore).
  --cert=PATH                   Path to certificate (PEM with certificate chain).
  --key=PATH                    Path to certificate private key (PEM with private key).
  --storepass=PASS              Password for keystore (if using PKCS keystore, optional).

That's a reasonable feature request, we can add that. Should be easy enough to accept an env var for this same as we do for the PKCS#11 flags.


Only log informations for commands of interest

Hey there ! Thx for this plugin
Would it be possible to silence the messages ?
It's a bit annoying when using poetry commands unrelated to the versioning:
> poetry env list
poetry-version-plugin: Using __init__.py file at pylsp_mypy_rnx/__init__.py for dynamic version
poetry-version-plugin: Setting package dynamic version to __version__ variable from __init__.py: 0.1.0
pylsp-mypy-rnx-Thao6M6B-py3.9 (Activated)

Indeed it would be very nice to either use a verbosity level, or send output to stderr.
Especially for commands like poetry version -s

Thanks for the patience with my reply! :sweat_smile:
I just marked this project as deprecated, I'm currently not using it and I think these ideas can be achieved in better ways: https://github.com/tiangolo/poetry-version-plugin#-warning-deprecated-
Given that I'll close this one, but thanks for the interest! :coffee:


Editing/revision: Developer documentation

This pull request suggests edits to all sections under 'Developer documentation'.
@ch1bo, just one request, please: can you rename the 'Architecture Decision Records' title to 'Architectural decision records' for consistency with other titles?  We won't be changing anything in the actual ADR section.

@olgahryniuk ADRs are capitalized because its an acronym. Shall we add "(ADR)"?

Why are we putting entire paragraphs on a single line? This is more difficult to edit.

@olgahryniuk Could you fix the conflicts here and on https://github.com/cardano-scaling/hydra/pull/1498 please so I can merge them?

@locallycompact, I think I managed to address the merge conflicts. Can you please double-check if everything is okay?


gtk::TextView: thread 'main' panicked at 'str::ToGlibPtr<*const c_char>: unexpected '' character: NulError

Hi.
I am using a TextView to show a string which has non-ascii information.
The string was generated through String::from_utf8_lossy() and afterwards put into a textview. This is the code:
let a = std::string::String::from_utf8_lossy(&saz[computed_index].file_request_contents.as_bytes());
let b = std::string::String::from_utf8_lossy(&saz[computed_index].file_response_contents.as_bytes());

dbg!(&b);

requestTextView.get_buffer().unwrap().set_text(&a);
responseTextView.get_buffer().unwrap().set_text(&b);

But, unfortunately, I get a panic:
thread 'main' panicked at 'str::ToGlibPtr<*const c_char>: unexpected '' character: NulError(604, [72, 84,...])', /home/romeu/.cargo/registry/src/github.com-1ecc6299db9ec823/glib-0.10.3/src/translate.rs:379:32

I did cut the bytes from NulError, there is a lot of information.
For more information, please check this Gist I created: https://gist.github.com/RomeuG/e3778d8ab203ea7c12affcbb8a89fb97
Maybe it is something that I am missing, but all I want to do is to show a string that includes many non-ascii bytes in a readable format inside a TextView.
Am I in the correct path, or am I donig something wrong?
Thanks.

Your string contains \0 characters, which is not allowed for C strings and would cause the string to be cut off at that character when being displayed in a text view (or anywhere else).

I see. The problem is obvious and it makes total sense.
But then again, how would I go about showing that text in a textview? As an example, Fiddler captures bytes and shows them like this:

What could I do to get the same results?

You would replace the \0 characters with some replacement character before passing it to GTK.

I see, thank you!


ZVDB®: Data Mining

Data Mining for ZVDB®:

Mining the Capsid Protein and their respective gene data - from heterogeneous data sources,
Processing the acquired data and knowledge extraction,
Storing the information in to Flat/JSON file - to use in further research activity,
Writing PERL/Python/R programs for the above cases.

Note: "pub", folder should be in root directory.

Zika Virus - NC_012532.

Zika Virus Genome Neighbours [82]:-

DQ859059
EU545988


mlv: Remove unecessary function names.

@otester but the name is already part of the MessageListView.prototype?

@lonerz you are correct if you inspect MessageView on its own you will see the name of the function and an anonymous function which works. But the time its not that easy is when you use debugger for tracking something down, the debugger will most of the time show an anonymous function in chrome's case f () {} in the debugger.
So in the worst case scenario if you are debugging MessageView you will have a hard time/

For real-world example look at this issue in node core it also has a screenshot (which i can't get right now which makes what i am saying more clear!) https://github.com/nodejs/node/issues/8913#issue-180731660.

Merged, thanks @lonerz!


[Bug] Cant get actual segments list for segmentation

Describe the Bug
In your example you have one segmentation with two segments with different colors.
But in cornerstoneTools.segmentation.state.getSegmentations()[0].segments i see only one segment. some in other in my cases with many segments

Steps to Reproduce

open example https://www.cornerstonejs.org/live-examples/labelmapsegmentspecificconfiguration
see in console cornerstoneTools.segmentation.state.getSegmentations()[0].segments

The current behavior
cornerstoneTools.segmentation.state.getSegmentations()[0].segments take only one segments
The expected behavior
expected - cornerstoneTools.segmentation.state.getSegmentations()[0].segments take two (many) segments
OS
macos 14.6.1
Node version
18.20.2
Browser
Chrome 131.0.6778.69

@sedghi @jlopes90
hello! can u help us?

some at https://www.cornerstonejs.org/live-examples/segmentationvolume example after click "load seg"

+1 for this issue
while in OHIF the same study and file is imported fine:
https://viewer.ohif.org/segmentation?StudyInstanceUIDs=<IP_ADDRESS>.4.1.145<IP_ADDRESS>.256467663913010332776401703474716742458

I think we should have them synced yes

I think we should have them synced yes

It would be a life-saver for us, thank you!


What is the safest walking speed m/s

What is the safest walking speed m/s whilst still being able to hatch eggs? Is it 4.16?

I guess you just have to search it. I usually set mine around 10km/h.


collate

Coverage remained the same at 80.348% when pulling 909cbb54af966246cf56a3241d701b17a547d4b1 on Pengyuz:master into 874d091256290c6782810d04c299fde676c524a4 on CS2103AUG2017-W09-B3:master.


Windows 10 20H2 Connection endpoints

Would it be OK to ask for a copy review for this PR by JohanFreelancer9, who regularly does that job on most PRs?


use wails for account creation

Codecov Report
Patch coverage: 41.17% and project coverage change: +1.79 :tada:

Comparison is base (bc4c74d) 30.59% compared to head (663dde8) 32.38%.

:exclamation: Current head 663dde8 differs from pull request most recent head d663f7f. Consider uploading reports for the commit d663f7f to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #117      +/-   ##
==========================================
+ Coverage   30.59%   32.38%   +1.79%
==========================================
  Files          10       10
  Lines         657      704      +47
==========================================
+ Hits          201      228      +27
- Misses        431      453      +22
+ Partials       25       23       -2

Impacted Files
Coverage Δ

pkg/wallet/prompt.go
0.00% <0.00%> (ø)

pkg/wallet/wallet.go
0.00% <0.00%> (ø)

internal/handler/wallet/export.go
48.57% <53.33%> (+3.11%)
:arrow_up:

internal/handler/wallet/get.go
68.65% <53.33%> (+0.96%)
:arrow_up:

internal/handler/wallet/create.go
84.44% <94.73%> (+42.77%)
:arrow_up:

internal/handler/wallet/api.go
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


usbreset: init at unstable-2018-09-21

Description of changes
This package allows to reset usb devices. This helps in some cases if other tools fail to reset e.g. internal wifi or bluetooth adapters connected via usb.
https://github.com/jkulesza/usbreset
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

We already have https://github.com/ralight/usb-reset so do we need a another one?

I used the tool to reset a broken bluetooth adapter in my old laptop. Since i have a new machine, i don't need the tool anymore and sadly cannot test the alternative you mentioned. Therefore closing this.


Screen is all black

Here is an example animation that fails to record:
https://p5.javascript.org.pl/frantic-average?template=none
I was testing simpler code with the recorder and using script tag this way works fine. Except it fails to record any video, even though it's a simple rotating square.
This is a simplified version where I have a more complex function instead of a square. I've tried to reproduce the issue.

I'm not sure what the problem is I was changing random things it something work sometimes doesn't.
I use Google Chrome on Fedora GNU/Linux.

Hi, @jcubic. Thank you for reporting this one!
I found that you are using the old p5.capture version in your example. Is this intentional?
Do you have the same problem when you try with the latest version?

I used the version that was provided in your examples. I think that you should update your example and don't hardcode the version. You don't need to do that on jsDelivr.
Will try the latest version and let you know.

I don't understand maybe you will be able to figure out if will have time.
I have my playground:
https://p5.javascript.org.pl/panicky-interest?template=none
Where I have a lot of comments (sorry they are in Polish) and redefined functions, the code is for my programming lessons where I have multiple functions as notes on how I should show the code step by step.
And the animation doesn't work, the video is all black. But when I copy-pasted only relevant code into that previously linked sketch it did not work.

I think that you should update your example and don't hardcode the version. You don't need to do that on jsDelivr.

Oh, that was my mistake and I fixed my examples.
Thanks!

I confirmed that the video is all black.
I think the cause is that the background is transparent. On the web it looks normal because there is a white background behind the canvas. But if you don't specify a background, the canvas background seems to be transparent and is recorded as black pixels in the video.
Can you test if it works using the background function?

There is white background, there is rect(0, 0, width, height); at the beginning. If you take a screenshot it's not transparent.
PS: You can edit the sketch, this is a prototype of the application that everyone can edit. I yet need to add some kind of user auth and permissions. I created a copy of my working sketch just in case if someone edit or delete the code.

Yes, you're right. I was testing screenshots in the first frame only, if I start the animation, and grab the screen I got the transparent background.
Thanks for your help to debug this.
BTW: Your project is great, I love it.


Please consider fixing errors reported by adapter checker

Adapter CHecker reported the following erros / warning for 2.6.0.
Please consider fixing them for the next release:
[E518] "@alcalzone/release-script" is used, but ".releaseconfig.json" not found
[W513] "gulpfile.js" found in repo! Think about migrating to @iobroker/adapter-dev package
[W505] setTimeout found in "main.js", but no clearTimeout detected

Error corrected in 2.6.1


Add changelog

Solves #313
I thought to directly use the CHANGELOG.md file from Github, but sadly....CORS!
So the changelog.md file is stored under kofta/CHANGELOG.md. If merged, this will be available at dogehouse.tv/changelog.md
@benawad if you want to fetch directly from Github on the server side and then provide it to the frontend, you just need to change the changeLogUrl in the constants/.env file.
Anybody is welcome to help to improve the markdown CSS. kofta/src/css/markdown.css

Yeah, any CSS changes are welcome.

I want to just link to the changelog on github for now instead of having it be a page on dogehouse

@benawad I have now linked it to Github and remove extra components.

Maybe have a master changelog md document that links to all the other changelogs?

Yeah, any CSS changes are welcome. You are very good at CSS @ajmnz

Appreciate it. I really really think every UI change must be first designed and approved. Improvising the design on the go is just gonna introduce more and more inconsistencies.

@ajmnz yeah we scraped it anyway.

Maybe have a master changelog md document that links to all the other changelogs?

Versioning would be another reasonable approach too.

See netlify logs, you need to remove unused import and checkout the  component I added. If you like it, we don't need
I'll setup semantic releases later to auto create changelogs

Okay I was not sure why my checks keep failing.

@benawad Which component did you add?

oh my bad, github stripped out my thing <Spinner />

@benawad The checks have passed now 🥱

Did you just add a new footer link? I see 5 now. @benawad

Did you just add a new footer link? I see 5 now. @benawad

Yep, he added the Origin Story (link to the yt video)


pip install -e research error.

Same problem
cpl/research does not appear to be a Python project: neither 'setup.py' nor 'pyproject.toml' found.

Marking as resolved -- the correct solution should be pip install -e ., apologies for the mistake in documentation.


ES filtering bugfix

Please merge the commits of Volker first

Replaced by https://github.com/openlibraryenvironment/gokb/pull/416.


feat: add student pulse

Your checklist for this pull request
🚨Please review the guidelines for contributing to this repository.

[x] Make sure you are requesting to pull a topic/feature/bugfix branch (right side). Don't request your default branch!
[x] Make sure you are making a pull request against the default branch (left side). Also you should start your branch off default branch.
[x] Check the commit's or even all commits' message styles matches our requested structure.
[x] Check your code additions will fail neither code linting checks nor unit test.
[x] I have added necessary documentation (if appropriate)

Linked Issues
Fixes
Description
Please describe your pull request.
❤Thank you!
Post merge checklist

[ ] Follow steps from the guidelines for contributing to this repository.
[ ] If you are a new contributor, ping in the thread and one of the maintainers will add you to all-contributors list.

@priyadarshini-silpa @120EE0692 are the sizing and formatting issues that were occurring with calendar side by side been all fixed?

@rutajdash yes.


Create check-for-integration-result.yml

Closes #

Changelog

New

Changed

Removed

Rollout strategy

[ ] Patch release
[ ] Minor release
[ ] Major release; if selected, include a written rollout or migration plan
[ ] None; if selected, include a brief description as to why

Testing & Reviewing

Merge checklist

[ ] Added/updated tests
[ ] Added/updated documentation
[ ] Added/updated previews (Storybook)
[ ] Changes are SSR compatible
[ ] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge
[ ] (GitHub staff only) Integration tests pass at github/github (Learn more about how to run integration tests)

this is a comment!


Add fallback if we fail to convert github list to a curl string

Enhancement/Feature

What does this PR do?
Adds a fallback to use the local list if we fail to parse the content in the github list after successfully fetching it.
Relates to OSD-22628

Codecov Report
Attention: Patch coverage is 0% with 14 lines in your changes missing coverage. Please review.

Project coverage is 24.85%. Comparing base (0069dfe) to head (87a128b).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #257      +/-   ##
==========================================
+ Coverage   24.82%   24.85%   +0.02%
==========================================
  Files          23       23
  Lines        1740     1738       -2
==========================================
  Hits          432      432
+ Misses       1286     1284       -2
  Partials       22       22

Files
Coverage Δ

pkg/verifier/aws/entry_point.go
0.00% <0.00%> (ø)


[BUG] Hauler cannot load copy images to registry from RHEL8 system when $HOME has noexec flag set

Environmental Info:

Linux  4.18.0-425.3.1.el8.x86_64 #1 SMP Fri Sep 30 11:45:06 EDT 2022 x86_64 x86_64 x86_64 GNU/Linux

Hauler Version:

GitVersion: 1.0.3
GitCommit: 7fc24f6
GitTreeState: clean

Describe the Bug:

When attempting to run hauler store copy <registry> on a RHEL8 STIG'd system, hauler throws an error ERR fork/exec /home/<user>/.hauler/cosign: operation not permitted"

Steps to Reproduce:

Copy Hauler to any authorized path (without noexec flag set) (e.g., /usr/local/bin)
Load images into local airgapped store: hauler store load <archive>
Run hauler store copy registry://<registry>
Observe the error above

Expected Behavior:

Command should run to completion.

Actual Behavior:

cosign attempts to run from inside ${HOME}/.hauler/cosign. This is unauthorized behavior according to DISA and blocked.

Additional Context:

Need to be able to specify another path or, preferably, an external cosign binary that is pre-positioned

hey @NullOranje, thank you for submitting this issue! we will begin testing and validating it and let you know our way forward. please let us know if there is any additional information.

Dunno if such is STIG compliant but a workaround might be to setup a $HOME/.config/systemd/hauler.mount unit that sets up a bind-mount for $HOME/.hauler to something like /var${HOME}/.hauler or something similar?

I did something similar.  I moved the bin to an allowed path and set a symlink ${HOME}/.hauler/cosign to the binary.  It seems to be workable solution for now.

I have attempted the workaround for this by trying to do a symbolic link from a lvm that does not have noexec set but I am still getting this error.
I also attempted to make the serviced with the following code:
[Unit]
Description=Data mount

[Mount]
What=/var/root/.hauler
Where=/root/.hauler
Type=none
Options=bind

[Install]
WantedBy=multi-user.target

I have turned off selinux and fips to test if that was getting in the way as well and it was still not working. I don't mind looking into resolving if this is not being currently worked.

Hey @NullOranje, that is what most customers are doing when running into this issue. Thanks for your patience on this.

I did something similar. I moved the bin to an allowed path and set a symlink ${HOME}/.hauler/cosign to the binary. It seems to be workable solution for now.

Hey @jaredpricedev! If you have the time, then it would be appreciated!
We were discussing implementing a similar function to the tempdir directory override that was implemented a few versions ago so users can use the flag --tempdir/-t and change the default temporary directory so creating something similar for the configuration directory.
https://github.com/hauler-dev/hauler/blob/main/cmd/hauler/cli/store/load.go

[!NOTE]
If you are attempting a work-around until a fix is in place, any alternate location you have setup will need to be:

on a filesystem sans noexec
on a location your user can write to

[!NOTE]
If SELinux is enabled and enforced you will want the .hauler directory to have the bin_t type label or similar.

If the filesystem that /home/<user>/.hauler exists on is not mounted noexec and the location is writable by you but you are still seeing permission denied on an SELinux-enforcing system, the root user should be able to inspect the audit log to determine if that is indeed the case, e.g.:
ausearch -m avc -ts recent | grep -E 'hauler|cosign'

Hey @NullOranje @jaredpricedev... please check out the recent PR and let us know if it looks good. If you have time to test it, please pass along any feedback!
git clone https://github.com/zackbradys/hauler.git && cd hauler && make build

@zackbradys Finally got to test.  Looks like it addresses my issue.  Thanks!

Glad to hear it! Thanks for the patience and testing it


Adopt Tendermint style validator priority logic for leader selection

Context:

https://docs.tendermint.com/master/spec/consensus/proposer-selection.html
https://github.com/tendermint/tendermint/blob/3c8955e4b824b1dcec7176832b2d5d96dad9fcad/types/validator_set.go#L116

Covered by https://github.com/rollkit/rollkit/issues/815


Create inference service for Together.AI

I've been hearing good things!
@rbyh Can you create us an EP account at together.ai

Done
Adding to pricing sheet

service = "together"


Added code-compilation-benchmark

@googlebot I signed it!

@sbmaruf We are now requesting that task authors please include an explicit section in the README.md file called "data source", listing exactly where you got your data from (or explicitly state that you made it up yourself). Many thanks!

@shyamupa are one of the assigned reviewers for this task.  Please provide feedback and be sure to notify me with a formal accept or does not meet criteria by Tuesday June 22 (if you have not already done so).

@dptam are one of the assigned reviewers for this task.  Please provide feedback and be sure to notify me with a formal accept or does not meet criteria by Tuesday June 22 (if you have not already done so).

Dear reviewers, please let us know if you find any issue with the dataset so that we can cooperate.
Just to add a small note, we were not able to provide a larger version of the dataset because of the size limitation of 50k samples per dataset. Since we are providing samples for 32 different languages (unique upto versions), we tried to cover as much as possible.
We are always available for any suggestion and modification.
Waiting for your reviews.

Hey @sbmaruf
I am having some issues with finding the task in the colab notebook. I found out from someone else to use this command:
#!pip install git+$bigbench_branch
!pip install git+https://github.com/google/BIG-bench.git@refs/pull/429/head

and it worked for several tasks, but I couldn't seem to find your task.

Since the task is still in the process of review, it's not merged with this Github repo. Not sure why you are able to see other un-merged task.
To get this task you can do something like this (works for me locally),
git clone https://github.com/google/BIG-bench
cd BIG-bench
git fetch origin pull/429/head:code-compilation-benchmark
git checkout code-compilation-benchmark
cd bigbench/benchmark_tasks/code-compilation-benchmark

If it is still not getting the pull-branch then you can just clone our source branch,
git clone https://github.com/sbmaruf/BIG-bench.git
git checkout code-compilation-benchmark

Please let us know if this solves the problem.

The amount of tasks we received this round is tremendous.  With so many tasks to review we have decided to extend the review period to Tuesday, June 29.
Reviewers: if the submitter has made revisions please be sure to notify me with a formal ”accept” or ”does not meet criteria” by Tuesday, June 29  (if you haven't already done so).
this is an automated message

I am one of the reviewers assigned to your task.
Correctness:
The format of the json file is correct, and all tests passed.
Formatting:
I ran the colab notebook, and it looks fine. One question, why do you use exact string match where the model has to generate "true". Wouldn't it be a bit easier for the model if it was formulated as a multiple choice problem where the options are "true" and "false"? Also, adding some more context to the prompt like "Does the following code compile?" might help?
Specificity:
The task captures the ability of LMs to check if source code can successfully compile or not by feeding in source code and checking if the model correctly recognizes it compiles or not. The task setup is well justified since other examples will be fed in.
Thoroughness:
The number of correct examples seems like a good way to measure the capability of the LM to evaluate if source code compiles or not.
Difficulty:
The task is not easily solved by current LMs (I think) or humans.
Not solvable by memorizing the Internet:
The sourcode is from CodeForces, and it is possible the model can memorize the answer if CodeForces includes whether it succesfully compiled, but it is obvious the model will.
Novelty:
I am not aware of any datasets that measure this.
Justification:
This can be used to automatically correct compilation error for source code in smart editors.
Size: There are are 32 subtasks (langauges), with the shortest subtask having 16 examples, the longest having 6k examples, and a total of 50k examples. All but the shortest subtask have 32 or more examples.
@chiafullo Accept!

@dptam
Thank you for your review.
Changelog

According to the reviewer's suggestion, we have updated the metric as multiple choice and also updated the keywords accordingly.
The prompt format we used is, "\nDoes the following {lang} code compile?\nCode:\n". Here the {lang} means the language of the Code that is given to the model.
Recently we got access to davinci API from OpenAI. We found that,

stop=["\n"] is a critical argument for the api. So we have updated stop_string argument in our task as \n.
We have changed the few-shot separator from <sep> to \n since it works with the API.
We have updated the "example_output_prefix"="\nA:". Note that the default "example_output_prefix"="\nA: " doesn't work with the davinci API.

The prompts are given in the following manner.  A sample python code that we used for testing.
import openai

openai.api_key = "{{hidden}}"

prompt = """Does the following GNU C++ code compile?
Code:
#include<iostream>\r\n#include<cstdio>\r\n#include<cmath>\r\nusing namespace std;\r\nint a,b,c,d,x,y;\r\nint main(){\r\n\tscanf(\"%d%d%d%d\",&a,&b,&c,&d);\r\n\tscanf(\"%d%d\",&x,&y);\r\n\tint cha=c-a,chb=d-b; \r\n\t\r\n\t \r\n\tif(x==0&&cha!=0)printf(\"NO\\n\");\r\n\t else if(y==0&&chb!=0)printf(\"NO\\n\");\r\n\t    else { c=abs(cha);d=abs(chb);\r\n\t\t       if (c % x!=0||d % y!=0)printf(\"NO\\n\");\r\n\t\t\t    else {\r\n\t\t  \t             a=cha/x;b=chb/y;\r\n\t\t  \t             if(abs(a-b)%2!=0)printf(\"NO\\n\");\r\n\t\t  \t                 else printf(\"YES\\n\");\r\n\t\t       }\r\n\t\t  }\r\n\t\t\r\n\t    \treturn 0;\r\n} \r\n
 choice: true
 choice: false
A:false
Does the following GNU C++ code compile?
Code:
#include<bits/stdc++.h>\r\n#define rep(i,a,n) for(int _tmp=n,i=a;i<=_tmp;++i)\r\n#define dep(i,a,n) for(int _tmp=n,i=a;i>=_tmp;--i)\r\n#define fec(i,a) for(__typeof a.begin() i=a.begin();i!=a.end();++i)\r\n#define For(i,u) for(int i=H[u];i;i=nxt[i])\r\n#define re(i,a,b) for (int i=a;i<b;++i)\r\n#define si(x) (int)x.size()\r\n#define pb push_back\r\n#define mp make_pair\r\n#define X first\r\n#define Y second\r\nusing namespace std;\r\ntemplate<class T> inline void read(T&x){char c;bool fu=0;for(c=getchar();c<=32;c=getchar());if(c=='-')fu=1,c=getchar();for(x=0;c>32;c=getchar())x=x*10+c-'0';if(fu)x*=-1;}\r\ntemplate<class T> inline void read(T&x,T&y){read(x);read(y);}\r\ntemplate<class T> inline void read(T&x,T&y,T&z) {read(x,y);read(z);}\r\ntypedef long long ll;\r\ntypedef pair<ll,int> pii;\r\nconst int N=111111;\r\nll K,k,n;\r\nint main(){\r\n\tread(K);n=2;\r\n\tfor(ll i=1;i<=K;++i){\r\n\t\tcout<<((i+1)*(i+1)*i*i-n)/i<<endl;\r\n\t\tn=i*(i+1);\r\n\t}\r\n\treturn 0;\r\n}
 choice: true
 choice: false
A:true
Does the following GNU C++ code compile?
Code:
#include <iostream>\r\n\r\nusing namespace std;\r\n\r\nint main()\r\n{\r\n    long long n=0,m,z;\r\n    cin>>z>>m;\r\n    long long x[z],y[z];\r\n    for(int i=0;i<z;i++)\r\n    {\r\n        x[i]=i+1;\r\n    }\r\n    for(int i=0 ;i<z;i++)\r\n    {\r\n        if(x[i]%2!=0)\r\n        {\r\n            y[n]=x[i];\r\n            n++;\r\n\r\n        }\r\n    }\r\n    for(int i=0;i<z;i++)\r\n    {\r\n        if(x[i]%2==0)\r\n        {\r\n            y[n]=x[i];\r\n            n++;\r\n        }\r\n    }\r\n    cout<<y[m-1]<<endl;\r\n\r\n    return 0;\r\n}\r\n
 choice: true
 choice: false
A:"""

completion = openai.Completion.create(
  engine="davinci",
  n=1,
  max_tokens=100,
  stop=["\n"],
  prompt=prompt,
  temperature=1,
  top_p=1,
  presence_penalty=0,
  frequency_penalty=0,
)
choice = completion["choices"][0]
print("[choice object]", choice)
print("[choice text]", choice["text"])

big-bench pytest passed

Looks good!

Your task has been accepted in the initial review stage.  The next stage of this process is for us to assign your task a meta-reviewer for final review and merge.  An assigned meta-reviewer will follow-up by commenting on the PR should it need further revisions.

Hi @sbmaruf, thank you for your contribution! I'm the meta-reviewer for this task as well. Could you also adjust the name of this task to use underscores instead of hyphens? Regarding data source - we just need to confirm whether it's all right to use CodeForces data, as in your other PR. Besides that this should be good to merge.

Hello @sbmaruf, we are trying to finalize tasks for the benchmark; please address the above issues by Wednesday, Aug 18, to have your task included in BIG-bench.  Thank you.

Hi! we are extremely sorry for the delay.
After having a month long disconnected communication, codeforces replied that, they don't like the idea of creating a dataset using their web site data and they don't have time (though we have mailed them a year ago) to take a deep look into the issue.. Nothing more or less.
I guess this is also a learning curve for us to familier with data permission issues.
I feel really bad at this point for the reviewers who spend their valuable time for this pull request. We apologies for the inconvenience caused by this.
I am closing the pull request since we haven't got any concrete reply from the codeforces.

@sbmaruf That is very unfortunate! Thank you for letting us know and for all the work you put into this task. I'm sorry it didn't work out.


Event priority ?

Hello, I have a problem :
https://github.com/MinelinkNetwork/CombatTagPlus/issues/65
Can you fix this issue please ?

@NathanWolf Is it a Magic issue ?

Don't think so, but would have to try it myself, not sure when I'll get the time for that.
The event priority is already set in a very specific way to avoid conflicts with at least two other plugins, so if that is the problem it may not be something I can fix.

So can you create a specific version that fix conflicts between CombatTagPlus and Magic ? Even if that specific version doesn't avoid conflits with the two other plugins you mentionned...

Not unless you never want updates.

At any rate, best to not jump the gun. I will look at this when I can, setting up a test will take time, especially if it involves another player.
If the author is really only killing the player, then I don't see why that plugin would be doing anything with the death event.

Okay, please let me know if you discover anything. By the way, about CTP, we believe -but we're not sure- that it's a special kill when the player disconnect : something that manually drops everything on the floor, and erases the entire player inventory. That would explain the wand loss.

Update on this: I tried it out for myself, and the basic issue is that Magic is not set up to handle players after they have logged out. Their data has already been forgotten and queued for asynchronous saving.
So when CombatTagPlus kills the player on logout, Magic sees the logout event and then the death event, and basically doesn't handle the death event at all.
This will be tricky to resolve in a clean way (like, very tricky). I can't just put the items back in the player's inventory, I have to save them in Magic's player data to return them when they relog and respawn. But I can't do this if that data is already saved and out of memory.
I'll keep this open and give it some more though, but I'm not currently seeing a lot of hope for these two plugins working well together.

Yes it can be a solution, but I think there is another solution.
If CTP can use the Magic API to check each item in the inventory to disable drop with them, and drop other items only, this could be fine.

That would work as well, but I don't expect Byteflux would be real open to that option. He's not currently dealing with the player's inventory at all on death, it'd be kind of a pain to support this just for the sake of my obscure little plugin.
That said, I've been thinking about this- maybe I could delay forgetting/saving player data on logout for one tick. I'll have to think about this to see if there are going to be any negative side-effects, particularly with other plugins that handle Magic data.

Ok I think I have this fixed. I'll need to keep an eye out for side-effects, but delaying player removal and saving on logout by one tick fixes the incompatibility issues.
Please try the latest dev build: http://jenkins.elmakers.com/job/MagicPlugin/1652/

Oh thanks !
But I'm still in 1.8.8, is it compatible ?

Ha, I should've guessed as much.
No, it's not, but I just backported it for you.
http://jenkins.elmakers.com/job/MagicPlugin-1.8/37/com.elmakers.mine.bukkit.plugins$Magic/

Thank you so much :)

@NathanWolf
The plugin didn't work at all when we put the version : no spell working, wand loss, etc. I don't know why :

[21:53:32] [Server thread/WARN]: [Magic] Error loading mobs.yml
java.lang.IllegalArgumentException: The embedded resource 'defaults/mobs.defaults.yml' cannot be found in plugins/Magic.jar

[21:53:32] [Server thread/WARN]: [Magic] Error loading items.yml
java.lang.IllegalArgumentException: The embedded resource 'defaults/items.defaults.yml' cannot be found in plugins/Magic.jar

That is pretty odd- I just tried the jar myself, as well as uncompressed it and looked inside. Those files are definitely in there, and the plugin is working for me. (Tried the one straight off my Jenkins link)
You sure you don't have multiple versions in your plugin folder or... something?

What do you mean by multiple versions ?

More than one copy of the plugin in your plugins folder- kind of a stretch, but only thing I can really think of there. I'll double-check to make sure I didn't have a weird/bad merge...

No more than one copy in my plugin folder :/

Hrm- um maybe try re-downloading then? Here's a direct link:
http://jenkins.elmakers.com/job/MagicPlugin-1.8/37/com.elmakers.mine.bukkit.plugins$Magic/artifact/com.elmakers.mine.bukkit.plugins/Magic/5.5.6-SNAPSHOT/Magic-5.5.6-SNAPSHOT.jar
I really don't know what could be going on there, I ran that exact jar myself. That error is really weird- you could also try decompressing the jar and just seeing if those files are in there, in the "defaults" folder. They should be.

Sounds like the jar file may be corrupted.
On Mon, May 2, 2016, 16:58 Nathan Wolf<EMAIL_ADDRESS>wrote:

Hrm- um maybe try re-downloading then? Here's a direct link:
http://jenkins.elmakers.com/job/MagicPlugin-1.8/37/com.elmakers.mine.bukkit.plugins$Magic/artifact/com.elmakers.mine.bukkit.plugins/Magic/5.5.6-SNAPSHOT/Magic-5.5.6-SNAPSHOT.jar
I really don't know what could be going on there, I ran that exact jar
myself. That error is really weird- you could also try decompressing the
jar and just seeing if those files are in there, in the "defaults" folder.
They should be.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/elBukkit/MagicPlugin/issues/94#issuecomment-216361228

Thanks, this is working now, sorry it was my fault, I was trying to unload / and load the plugins without restarting.

@dumptruckman ?

@Blamo27 You may want to update to the latest dev build- there was a pretty bad bug in the change I made to support CombatTagPlus, players could lose their inventories when logging out with their spell inventory active.


AWS credentials ending in "=" don't work

Output of restic version
restic 0.5.0
How did you start restic exactly? (Include the complete command line)
restic init
The problem is when the AWS credentials (in this case from env vars), end in the character "="
Minio will automatically generate such credentials...
What backend/server/service did you use?
S3 / minio
Expected behavior
Restic will work with an S3 backend that requires credentials ending in "="
Actual behavior
restic init

create backend at s3:http://<IP_ADDRESS>:9002/backups failed: client.BucketExists: Missing fields in request.

Steps to reproduce the behavior
Use minio to bring up a server setting either the access key or secret key to a string ending in "=".
Attempt to use restic to init a repo using these credentials.  Note, mc has no difficulty using such credentials to access the same server, and minio will randomly generate such credentials.
Do you have any idea what may have caused this?
My guess is the = isn't being properly escaped in some URI being used to access the S3 backend.

Huh, interesting! I'll look into it, thanks for the report!

I'm unable to reproduce this issue for the AWS_SECRET_ACCESS_KEY (works fine with an equals sign at the end), but setting a AWS_ACCESS_KEY_ID with an equals sign at the end provokes the error.

Something is strange here.  I've verified that restic just passes the values in the environment variables to the minio-go s3 library, so it's not a bug in restic.
Where did you get the credentials? As far as I know, AWS_ACCESS_KEY_ID should be an alphanumeric string, so it should not contain equals at all. The AWS_SECRET_ACCESS_KEY is base64 encoded, so for certain lengths there will be one or two equals signs at the end of the string. My minio binary here will only generate secrets with a length that does not result in an equals sign when base64 encoded. And I've verified (by manually configuring a minio server with a secret of DiU8zVwhiXowkApWRvdQhd7bQeUk+CxBgngnATM=, that works just fine.
So I'm a bit at loss what's happening here, but I suspect that it is not a bug (neither in restic nor in minio-go).
@harshavardhana Could you comment on the format of the AWS_ACCESS_KEY_ID maybe? Thanks!

I may have been mistaken about AWS_SECRET_ACCESS_KEY, thought I checked that one too.
I lost about 2 hours today to this... Could not figure out what had changed that broke restic. The ransom credentials generated by minio each had an = at the end.

I'm sorry that you lost so much time. I'm not sure how you ended up with credentials containing = at all:
For the AWS_ACCESS_KEY_ID, minio will generate a string of 20 random characters from the set<PHONE_NUMBER>ABCDEFGHIJKLMNOPQRSTUVWXYZ, so that should never contain =: https://github.com/minio/minio/blob/master/cmd/credential.go#L32 https://github.com/minio/minio/blob/master/cmd/credential.go#L105-L110
For the AWS_SECRET_ACCESS_KEY, the minio server generates 40 random bytes, encodes them with base64, and takes the first 40 bytes. Since = is only valid at the end of a base64 encoded string, there cannot be a = in that string either: https://github.com/minio/minio/blob/master/cmd/credential.go#L117
Since 0.5.0 and master we've upgraded the minio-go library, so it may well be that there was a bug in the library. Using 0.5.0 with a minio server is very slow, so there may be something strange going on.
Could you test the master branch?

I'm no longer sure how I ended up with those credentials either.  I plucked them off the screen that minio dumps to stdout when it starts. But that was during a test that I discarded soon after I figured out what was going on and submitted this issue.
I'm using the latest minio.  I'm working off a fork of restic, partially because of the slowness issue, partially because my project requires the S3 / filesystem path layout parity that we've discussed (which I have implemented), and partially because I require a build of restic with cgo enabled, since the go runtime without cgo does funny things with DNS resolution that make using local DNS lookups unusable (say by registering redundant minio servers with Consul, and using an address like minio.service.consul to load balance). See here if you are curious about that... https://golang.org/pkg/net/#hdr-Name_Resolution

Just to add, I did test with official restic 0.5.0 (and not just my fork) before submitting this issue. Unfortunately I'm on deadline and don't have much more time to help with this.

Ok, thanks for taking the time to report this. We'll wait and see what @harshavardhana thinks.
By the way: I've already implemented the default repository layout for s3 in the master branch, you can use it by setting -o s3.layout=default when creating a new repository. Afterwards you don't need that parameter any more, restic will auto detect the layout used for the s3 repository and do the right thing.
For cgo, you just need to change this one line in build.go: https://github.com/restic/restic/blob/master/build.go#L198

There's the new release: https://github.com/restic/restic/releases/tag/v0.6.0-rc.1

@vsivsi I am pretty you must have interchanged the keys.
AccessKey can never be alphanumeric unless you manually edited it.
Will test by manually setting it and see what happens.

I have no idea how I end up with such keys.  I'm working fast and furious right now, so it's possible there was human error in the loop (as alway).  At a minimum, restic need pa to give a more helpful error message in such cases than: missing fields in request

Oops, didn't mean to close the issue.

@fd0  thanks for the link, I'll try to check out the new RC, but my ship literally sails on Saturday, so time is short!

I'll close this issue for now, we can always reopen it later. Thanks!


feat(hmac-auth) add 2 endpoints to interact with hmac-auth credentials

/hmac-auths/ to paginate through all hmac-auth credentials
/hmac-auths/:credential_key_or_id/consumer to retrieve the Consumer
associated with a credential

Thanks again! Marking this one for review as well :)

@thibaultcha I've updated the naming in the file overall.
I've assumed we will squash and merge.
If we decide to put the renaming of other variables in a separate commit, then I can put a better commit message.

Thank you!

@hbagdi  yay! 🎉


b575f42a seems to have broken split view

The recent commit b575f42a seems to have broken split view.  Now the split mode shows the notes on both displays.

Oh, ok. I might must have overlooked this during testing. :/
I will take a look in the evening.
Thanks for reporting this.

Should be fixed by afe13ad

Works for me.  Thanks!


Error while running Pydot in Python 3 on  Mac

Using Jupyter Notebook 6.0.3 on Macbook.
pydot==1.4.1
pydotplus==2.0.2
When I run the code below:
import pydot
G = nx.complete_graph(4)
pos = nx.nx_pydot.graphviz_layout(G)
pos = nx.nx_pydot.graphviz_layout(G, prog='dot')
I am getting the following error:

AttributeError                            Traceback (most recent call last)
 in
----> 1 pos = nx.nx_pydot.pydot_layout(G)
~/opt/anaconda3/lib/python3.7/site-packages/networkx/drawing/nx_pydot.py in pydot_layout(G, prog, root)
307
308     """
--> 309     import pydot
310     P = to_pydot(G)
311     if root is not None:
~/Google Drive/BITS-DSE/3-Graphs-Algorithms-Mining/Assignment-5/pydot.py in
2 G = nx.complete_graph(4)
3
----> 4 pos = nx.nx_pydot.pydot_layout(G)
5 pos = nx.nx_pydot.pydot_layout(G, prog='neato')
~/opt/anaconda3/lib/python3.7/site-packages/networkx/drawing/nx_pydot.py in pydot_layout(G, prog, root)
308     """
309     import pydot
--> 310     P = to_pydot(G)
311     if root is not None:
312         P.set("root", make_str(root))
~/opt/anaconda3/lib/python3.7/site-packages/networkx/drawing/nx_pydot.py in to_pydot(N)
201     graph_defaults = N.graph.get('graph', {})
202     if name == '':
--> 203         P = pydot.Dot('', graph_type=graph_type, strict=strict,
204                       **graph_defaults)
205     else:
AttributeError: module 'pydot' has no attribute 'Dot'

Not sure if I am seeing this right, but could it be that import pydot in line 309 accidentally does not import the real pydot library, but instead imports your own script again that also happens to be named pydot.py?
Maybe try renaming ~/Google Drive/BITS-DSE/3-Graphs-Algorithms-Mining/Assignment-5/pydot.py to something like myscript.py.
And please format program output as code, btw. See https://guides.github.com/features/mastering-markdown/

Since there was no further reply, I assume this issue has been resolved and I will close it now. Feel free to reopen it if necessary.

Not sure if I am seeing this right, but could it be that import pydot in line 309 accidentally does not import the real pydot library, but instead imports your own script again that also happens to be named pydot.py?
Maybe try renaming ~/Google Drive/BITS-DSE/3-Graphs-Algorithms-Mining/Assignment-5/pydot.py to something like myscript.py.
And please format program output as code, btw. See https://guides.github.com/features/mastering-markdown/

In a similar way, I had unintentionally named my Python script as "dot_parser.py" and it seemed to overwrite one of the pydot imports. Just changing the name solved my issue!


Added a solution for LeetCode's 'Remove Duplicates from Sorted Array'…

Added a solution for LeetCode's 'Remove Duplicates from Sorted Array' problem

Implemented a Python solution to remove duplicates from a sorted array.
Went through the sorted array and keeps count of the unique elements.
Returned the count of unique elements as the result.

LGTM


doas support?

Would you please support doas if you like?

I agree with this.
suda] sudo: invalid option -- 'p'
[suda] usage: doas [-Lns] [-C config] [-u user] command [args]

This is what I receive after inputting my password when running :SudaWrite, I don't think it'd take much to add it in.

please make this hapeend

Maybe make it use su instead, as that's installed on most if not all systems as far as I know. For example, makepkg uses su if sudo is not found...

Oops. I'm sorry I didn't notice this request. Well, I have no idea what "doas" and I have no idea what to do to support it.

doas is alternative to sudo
https://wiki.archlinux.org/title/Doas
sudo has the -p option, but doas does not, so symbolic links will not work

I've already cloned this project and tried to work on it, but I've hit a "roadblock,", doas doesn't have the flag that sudo does to pass the password to, so you're probably going to use expect or socat or something like that.

Probably, I'm not going to support doas or su in suda.vim while suda.vim has technical limitation (#34). Instead, maybe I'll create a new fresh plugin for handle #34 and then I'll consider to support doas and/or su.

This issue is fixed by #58

@aarondill it's not, -p flag is still used by the plugin

@lambdalisue
Line 18 of autoload/suda.vim needs to be changed to fully support doas (when a password is needed)

@acid-bong I've looked at the source for doas, and it seems there's no way to pass a password to stdin, instead of requiring a terminal. Changing line 18 (see above) would allow it to work with any programs which read from stdin, but if it attempts to open a tty (like doas), we can only support the no password version.
I'm not certain of any tools for this purpose that do except through stdin, but the above mentioned change would allow them to work by default (after setting the executable variable)

@aarondill a possible solution could be found somewhere in Emacs' tramp.el (for current stable branch, emacs-28), doas is one of the preconfigured privilege elevation tools in it. I know it does work flawlessly* (except when your default shell is zsh, but that's a different issue), but i'm yet to find out how it passes the password, because it doesn't use -S for sudo

I still don't use doas so it's a bit hard for me to support it while I've no idea what I need to do so PRs are very welcome to support it.

@aarondill a possible solution could be found somewhere in Emacs' tramp.el (for current stable branch, emacs-28), doas is one of the preconfigured privilege elevation tools in it. I know it does work flawlessly* (except when your default shell is zsh, but that's a different issue), but i'm yet to find out how it passes the password, because it doesn't use -S for sudo

Sadly, I can't for the life of me read lisp, especially when it's that abstracted. If you can figure it out, please let me know. My guess would be that emacs allows commands to open a tty when executing (like vim did), which nvim doesn't (I'm likely misunderstanding the root issue, but still), which is the original reason this plugin was necessary.

With the merging of #60, you can now use doas only with nopass configured!


fix(album): now duplicated media will be hidden in album modal

this PR resolves #78

:tada: This PR is included in version 1.0.0-dev.8 :tada:
The release is available on:

v1.0.0-dev.8
GitHub release
v1.0.0-dev.8

Your semantic-release bot :package::rocket:


r2.15 cherry-pick: 66681ea34be "Add missing headers to the Windows Libtensorflow archive"

Refer to the original commit: https://github.com/tensorflow/tensorflow/commit/66681ea34bee93fa359486458f92ceea864cc6de

@learning-to-play The Windows libtensorflow archive is missing few header files. Due to this, Windows users are running into errors when trying to use Libtensorflow (see https://github.com/tensorflow/tensorflow/issues/59762 and https://github.com/tensorflow/tensorflow/issues/61964). This PR amends the Windows libtensorflow build script to add the missing headers to the archive. It will be great if we can have this PR cherrypicked before we do the TF 2.15 final release.


It seems that GEOS-Chem 12.1.0 do not save boundary conditions?

Model version: 12.1.0
AMI id: ami-0ee8892ae47c31be1
Instance type: c5.4xlarge
All the steps were copied from https://cloud-gc.readthedocs.io/en/latest/chapter02_beginner-tutorial/research-workflow.html except for several configurations:
In input.geos:
%%% SIMULATION MENU %%% :
Start YYYYMMDD, hhmmss  : 20160701 000000
End   YYYYMMDD, hhmmss  : 20160703 000000

%%% NESTED GRID MENU %%%:
Save TPCORE BC's        : T
Input BCs at 2x2.5?     : T
Over North America?     : F
TPCORE NA BC directory  : BC_2x25_NA/
Over Europe?            : F
TPCORE EU BC directory  : BC_2x25_EU/
Over China?             : T
TPCORE CH BC directory  : BC_2x25_CH/
Over Asia?              : F
TPCORE AS BC directory  : BC_2x25_AS/
Over Custom Region?     : F
TPCORE BC directory     : BC_2x25/
BC timestep [sec]       : 10800
LL box of BC region     :   9  26
UR box of BC region     :  29  41
I0_W, J0_W, I0_E, J0_E  :   3   3   3   3

In HISTORY.rc
#============================================================================
EXPID:  ./OutputDir/GEOSChem

COLLECTIONS: 'Restart',
             #'SpeciesConc',
             #'Budget',
             'AerosolMass',
             'Aerosols',

# %%%%% THE AerosolMass COLLECTION %%%%%
#
# Aerosol and PM2.5 mass
#
# Available for full-chemistry and aerosol-only simulations
#==============================================================================
  AerosolMass.template:       '%y4%m2%d2_%h2%n2z.nc4',
  AerosolMass.format:         'CFIO',
  AerosolMass.frequency:      00000001 000000
  AerosolMass.duration:       00000001 000000
  AerosolMass.mode:           'time-averaged'
  AerosolMass.fields:         'AerMassBC                    ', 'GIGCchem',
                              'AerMassNH4                   ', 'GIGCchem',
                              'AerMassNIT                   ', 'GIGCchem',
                              'AerMassPOA                   ', 'GIGCchem',
                              'AerMassSAL                   ', 'GIGCchem',
                              'AerMassSO4                   ', 'GIGCchem',
                              'AerMassSOAGX                 ', 'GIGCchem',
                              'AerMassSOAMG                 ', 'GIGCchem',
                              'PM25                         ', 'GIGCchem',
                              'TotalOA                      ', 'GIGCchem',
                              'TotalOC                      ', 'GIGCchem',
::
#==============================================================================
# %%%%% The Aerosols COLLECTION %%%%%
#
# Aerosol optical depth, surface area, number density, and hygroscopic growth
#
# Available for full-chemistry or aerosol-only simulations
#==============================================================================
  Aerosols.template:          '%y4%m2%d2_%h2%n2z.nc4',
  Aerosols.format:            'CFIO',
  Aerosols.frequency:         00000000 010000
  Aerosols.duration:          00000001 000000
  Aerosols.mode:              'instantaneous'
  Aerosols.fields:            'AODDust                       ', 'GIGCchem',
                              'AODDustWL1_?DUSTBIN?          ', 'GIGCchem',

Also made BC_2x25_CH and OutputDir directories prior to running the model. The model run smoothly but no BC files generated with BC_2x25_CH directory.

Thanks for the report. I think that's because the boundary condition files are still in bpch format and you need to compile with BPCH_DIAG=y.
I also put your log message into "```" blocks so it renders better. For long messages you can just upload the log file -- just drag into GitHub.

Thanks for the report. I think that's because the boundary condition files are still in bpch format and you need to compile with BPCH_DIAG=y.
I also put your log message into "```" blocks so it renders better. For long messages you can just upload the log file -- just drag into GitHub.

Thanks for your reply. Following your clue, I did the following things.

make realclean
make -j4 mpbuild # I guess BPCH_DIAG=y is the default and hence omitted it in command
./geos.mp

Then I got the following error.
GEOS-Chem ERROR: No output scheduled on last day of run!                                                                                -> at Is_Last_Day_Good (in module GeosCore/input_mod.F)                                                                                ===============================================================================                                                                                ===============================================================================                                                                                GEOS-Chem ERROR: Error reading the "input.geos" file!                                                                                -> at Read_Output_Menu (in module GeosCore/input_mod.F)                                                                                ===============================================================================                                                                                ===============================================================================                                                                                GEOS-Chem ERROR: Error in "Read_Output_Menu"!                                                                                -> at Read_Input_File (in module GeosCore/input_mod.F)                                                                                ===============================================================================                                                                                ===============================================================================                                                                                GEOS-CHEM ERROR: Error encountered in "Read_Input_File"!                                                                                STOP at GEOS-Chem (in GeosCore/main.F)                                                                                ===============================================================================                                                                                Program received signal SIGSEGV: Segmentation fault - invalid memory reference.                                                                                Backtrace for this error:                                                                                #0  0x14a8679842da in ???                                                                                #1  0x14a867983503 in ???                                                                                #2  0x14a866dd1f1f in ???                                                                                #3  0x565468655d7f in cleanup_                                                                                at /home/ubuntu/GC/Code.GC/GeosCore/cleanup.F:266                                                                                #4  0x565468a05659 in __error_mod_MOD_geos_chem_stop                                                                                at /home/ubuntu/GC/Code.GC/GeosUtil/error_mod.F:725                                                                                #5  0x565468a05975 in __error_mod_MOD_error_stop                                                                                at /home/ubuntu/GC/Code.GC/GeosUtil/error_mod.F:634                                                                                #6  0x5654685ba525 in geos_chem                                                                                at /home/ubuntu/GC/Code.GC/GeosCore/main.F:493                                                                                #7  0x5654684c75b8 in main                                                                                at /home/ubuntu/GC/Code.GC/GeosCore/main.F:32                                                                                Segmentation fault (core dumped)
I guess it may be associated with input.geos and HISTORY.rc?
Anyway, GEOS-Chem 12.2.0 can output boundary conditions within BC_2x25_CH.

GEOS-Chem ERROR: No output scheduled on last day of run!

Ah this is a common error for early versions of GEOS-Chem with BPCH diag: http://wiki.seas.harvard.edu/geos-chem/index.php/Common_GEOS-Chem_error_messages#No_output_scheduled_on_last_day_of_run
Typically I would set
08: Schedule output for JUL :<PHONE_NUMBER>333333333333333333333

in input.geos

GEOS-Chem ERROR: No output scheduled on last day of run!

Ah this is a common error for early versions of GEOS-Chem with BPCH diag: http://wiki.seas.harvard.edu/geos-chem/index.php/Common_GEOS-Chem_error_messages#No_output_scheduled_on_last_day_of_run
Typically I would set
08: Schedule output for JUL :<PHONE_NUMBER>333333333333333333333

in input.geos

Personally I prefer netCDF more and like to specify output in HISTORY.rc, so may try this in a later time. BTW, may I confirm that if I compile the model with NC_DIAG=y and specify HISTORY.rc for collections, there is no need to do anything on Output Menu in input.geos?

there is no need to do anything on Output Menu in input.geos?

Right, I think most diagnostics sections in input.geos will become irrelevant

Closing if no further questions


[Task]: create basic layout components and associated utils

Description
Objective - Create scaffolding for layout components (LC).

Created LayoutComponentHOC that will wrap the FlexLayout component and provide all the necessary functionality that is expected of an LC.
Created LayoutFactory to initial available LCs.
Created base LCs that will be used to construct presets - AlignedColumn, AlignedRow, Column and Row.
Created various utils to manage functionality of LCs.
Added unit tests for utils.

PR fixes following issue(s)
Fixes #27000
Fixes #27001
Fixes #27002
Type of change

Please delete options that are not relevant.

New feature (non-breaking change which adds functionality)

Testing
How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[x] Manual
[ ] JUnit
[x] Jest
[ ] Cypress

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test

Some points of discussion wrt to code design:
To enable layout components inside the existing canvas flow i.e, the renderAppsmithCanvas we would need a Parent Component per each renderMode that would kick start rendering of layout components with the layout prop.
AnvilLayoutEditorCanvas and AnvilLayoutViewerCanvas.
in both components an entry point is needed.
I see that renderLayouts is the kick stater which iterates through layout data of a canvas and gets the layout component based on type and renders it.
each layout component is wrapped with a hoc that adds a DnD layer and takes the decision whether the component should render layouts or render its own implementation of renderChildWidgets
Registration of layout components like widgets has hurdles coz we wont have access to app state during the registration cycle so doing it on-demand would not be possible with todays arch. I would recommend we skip the same widget registration flow and instead just have a constant mapping layout type and the component.
for the entry point mentioned above for AnvilLayoutEditorCanvas, Would it make sense to use a parent Component LayoutComponetsMaker, to pick the layout component from a constant which would be a map of all layout components which is mentioned in the layout property and render it instead of picking it from a layout registry that adds a HOC for every widget.
regarding the static functions per layout component, if they are to DRY the components,
I would suggest we use an Abstract base class for Layout components(BaseLayoutComponent) just like BaseWidget and extend them per component with their own definitions.
The LayoutComponentHOC currently adds a DnD layer and the decision to renderLayouts or widgets, we can move this as a common logic inside the abstract class(BaseLayoutComponent) for layout components.
A bit skeptical about the effect of Object.assign in a HOC.
one more concern was that I feel a LayoutComponent is trying to do a lot by being a renderer of layout skeleton as well as being a renderer of children in a particular layout manner. there by the prop layout has multiple type definitions, I think we should isolate them, like have layout skeleton renderers and layout with widget renderers. and have appropriate type definitions and components for each. and whenever we see an opportunity to dry them we have tools like hooks, hocs and utils to do that.
@prsidhu @riodeuno let me know ur thoughts.

@riodeuno @marks0351 I have added the canvas implementation to this PR, along with minor review suggestions.
I will include the larger suggestions in the anvil sagas PR as it will allow me to make the desired changes across the board in a single attempt.
Additionally, I've added three tasks to reconsider epic based on the review suggestions.

/ok-to-test

/ok-to-test

Closing this. The changes have been merged to release via #27980


Create github actions workflow for release

Situation
We have often problems with tagging the client and server tags. When a master build is already running, pushing a tag remotely afterwards does often not result in github action workflow execution (on push). But we need the git tags for documentation.
Wanted

We want a possibility to trigger manually the release build
The release build shall tag the versions automatically
Shall only be on master

Solution
We define a new workflow release and use worfklow_dispatch with manual input - see
https://docs.github.com/en/actions/reference/events-that-trigger-workflows#workflow_dispatch
Because this is not a git event we must checkout master branch dedicated, see https://github.com/actions/checkout
Additional links:

https://github.com/actions/create-release
https://github.com/actions/upload-release-asset

Albert Tregnaghi<EMAIL_ADDRESS>Daimler TSS GmbH, imprint

Done, see https://github.com/Daimler/sechub/actions?query=workflow%3ARelease