Improved template documentation

Please select if your request is either something new or an enhancement

[x] Enhancement of an existing Feature.
[ ] Request of a new feature.

Please select the area your request applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[ ] Workspace - VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup - Dendron's Lookup Command
[ ] Views - Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema - Dendron Schemas
[ ] Pod - Data import from / export to Dendron
[ ] Publish - External Site Publish
[ ] Markdown - Markdown features, such as syntax support and features
[ ] Documentation

Is your feature request related to a problem? Please describe
When trying to understand how templates and schemas work in detail, I often have to look into the code or run repeated experiments.
Describe the solution you'd like
I'd like to be able to answer questions like these from the documentation:

Does a template node need frontmatter?
What is "custom frontmatter" that will be copied from template -> note? As in, which keys.
How does variable substitution work inside frontmatter?
What's the relationship between the in-built journal functionality and templates?

Why does traitIds: [ journalNote ] not get copied from template to note? Maybe related to above.
How is the title of the journal note set? Can this be modified?
Is it possible to have a custom template for the markdown content while still using the inbuilt functionality? If so, how?

What are the options supported by dendron.yml? How can I find documentation that explains each option in detail without reading the code?
Which parts of the wiki / docs relate to which versions of dendron?

Are some features still being worked on? For example, tasks seem to be in some state of flux. There's an open RFC, but there's also some features shipped, while the docs (or wiki?) seem to describe features that are not yet released, or deprecated?

Is there any naming requirement to templates?

Describe alternatives you've considered
It's possible these questions are answered and I've missed them. In the process of writing this ticket I found the answer to some questions and removed / updated them.
Additional context
My entirely subjective experience is that the docs are somewhat messy and not 100% in sync with the code. I could well be wrong, maybe I'm missing things. I know that links between docs & wiki are broken so it's sometimes hard to navigate.
Also, what's on the wiki and what's in the docs? This separation is hard to understand and I haven't found it written down anywhere. I usually open both and search both not knowing where I'll find a given answer.
I hope this doesn't come across as overly critical. I'm a big fan of dendron so far.

Thank you for the issue.
I understand your frustration with the documentation. Keeping our documentations easy to follow is an ongoing effort.
I believe @ScriptAutomate has written some guides recently on templates and schemas, and there were a few community members that have shared their working schemas that could be a starting point.
We also recently initiated an effort to establish a moderation group for Dendron (Dendrologists). Among their roles and focus areas, one is to keep the documentation consistent. You can read more about Dendrologists in the discussion here
Feel free to leave your thoughts on the discussion above.
I'll answer some questions that I can immediately answer and pass this on to @ScriptAutomate so we can figure out how to move forward 🙂

What are the options supported by dendron.yml? How can I find documentation that explains each option in detail without reading the code?

This is where we are documenting all supported configuration options in dendron.yml.
Please note that we are in the middle of a rather aggressive config namespace consolidation so there will be a lot of config keys scheduled for deprecation that have moved elsewhere (these are marked as such and linked to the appropriate note that lists the up to date configs)

Also, what's on the wiki and what's in the docs? This separation is hard to understand and I haven't found it written down anywhere. I usually open both and search both not knowing where I'll find a given answer.

wiki.dendron.so is the user guide for Dendron and docs.dendron.so is primarily a reference for the Dendron Team and contributors who are directly working with the codebase. I understand why you had to look into both. Ideally we want all things related to using Dendron day-to-day be sufficiently provided in wiki.dendron.so, and we are on the process of improving this by means I have mentioned above. I apologize for the inconvenience.

@hikchoi Thanks for the link to the dendron.yml docs, you're absolutely right, it is documented there. I think it could be that my issues are caused by not being able to use the search properly. For example, searching dendron.yml on the wiki don't bring up the Configuration documentation, but searching on DuckDuckGo does right away. Maybe I need to use that more and rely less on the site search.

I think it could be that my issues are caused by not being able to use the search properly.

I understand. If you have suggestions to improve the search function on published Dendron sites, please do give us a feature request.

For example, searching dendron.yml on the wiki don't bring up the Configuration documentation, but searching on DuckDuckGo does right away. Maybe I need to use that more and rely less on the site search.

Just wanted to mention that both wiki.dendron.so and docs.dendron.so are Dendron vaults themselves. (These are both included in our monorepo so that we can read and edit docs while we work on Dendron, for example 😄 ).
This means if you want to have the documentation for Dendron inside VSCode, you can create a workspace (or use an existing one) and add dendron-site and dendron-docs as remote vaults to use VSCode's search capabilities instead of using DuckDuckGo.
We never expect users to pull these two vaults locally and use that as a primary source of information. The search functionality should definitely be up to par so that our users can reliably find what they need within the published website. I just wanted to let you know that this is possible if that's something you are willing to do.

@hikchoi That's a freakin excellent idea, I could pull them both in as vaults and have it locally!
As for search, alas, I don't have any good recommendations. Oh wow 🙈 - I just realised that I need to add ? to make the search do a full text search. Alas, the top results for ? dendron.yml are changelogs and other dated stuff. Maybe there's a simple sort algorithm that would work there, like putting guides & so on first. If my thoughts on that clarify any I'll open a new issue. :-)


"Fixes GetterSetterToPropertyAdvancedPass by visiting the bases and conve...

...rting their properties first. This is neccessary for overrided properties of derived classes to work."
Hi, thanks for the quick fix for my previous crash report. I've got one more.
In the GetterSetterToPropertyAdvancedPass, QtGui generation crashes, when trying to access the base property of overrided property of a derived type. This happens with QWindow.surfaceType, which overrides QSurface.surfaceType.
In AstVisitor, it always visits the base classes of visited class declarations, but the overrided VisitClassDecl of GetterSetterToPropertyAdvancedPass doesn't do that - that's why the properties of the base classes aren't set.
Strange that it has worked this far, though. I wonder what changed?

I changed that at https://github.com/mono/CppSharp/commit/ec0f6a3d63db330fab90967f7603a7d3750d5266 . There was a crash with QtCore, I believe. Could you please generate QtCore with your change and see what happens?

I see, thanks for information. I checked your fix... while it indeed prevents the crash in QtCore, it doesn't seem to fix the problem. After quick debugging (man, I hate these deep recursion walk debugs, but I'm starting to get the knack of it :P)
The actual problem before your fix seems to be:
Walk to QMetaObject, walk to its base, QObject, walk QObject's methods, walk one of the methods' return type, QThread, walk to one of QThread's method's return type, QAbstractEventDispatcher, walk to its base which is QObject, it deems QObject already visited, so start unwinding the recursion stack: return to QAbstractEventDispatcher, generate QAbstractEventDispatcher properties. One of the properties overrides QObjects property, but it isn't generated yet -> CRASH.


fix(bio): allow pipe characters in description #131

closes #131

Oh nice fix, thank you @AllanRegush . Must have missed the notification for this 🤦♂ . Thanks @BOLT04
@AllanRegush could you fix the conflict? If you need any help let me know


Center color circle loose circle shape when defining 'oldColor' params

Hello here,
First of all thanks for your work on this one !
I have an issue: when I set a value for oldColor params the center circle (with old/current color) switch from a perfect circle to something strange between square and circle I would say.
Any idea of the origin of that ? Thanks a lot for infos

I resolved the issue by changing the following the StyleSheet
 selectedPreview: {
        position: "absolute",
        borderColor:'transparent',
        borderWidth:1,
        // borderLeftWidth: 0,

    },
    originalPreview: {
        position: "absolute",
        borderWidth: 1,
        borderColor:'transparent',
        // borderRightWidth: 0,
    },

I think by making the border widths 0 on iOS had some weird effect on the other border radiuses of the view. Idk


Development improvements

Hi!
I was trying to get the tests passing locally and noticed some issues. Some commits you might not be interested or might just be plain wrong, so feel free to cherry pick what you like!
Regards.

Thanks for your comments, I'll update the patch.

@deivid-rodriguez thanks for the patch. There's quite a lot here. Can you remove all the things that we've talked about so that we can look at it again.

You (and Travis) can do a second review pass.

Here you go!

Done, this should now be good to go!

This LGTM. Merging.

Thanks!


The on screen keyboard hiddes the TextFormField .

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Run the shared code snippet.
Tap on the text form field.
When the keyboard is raised, it hides the text form field.

Note: I tried to resolve the issue with the SingleChildScrollView widget, but it caused a conflict with the interactive viewer scrolling and panning.
Expected results
The text form field must be shown above the keyboard.
Actual results
The keyboard is shown over the form field.
Code sample
Code sample
class MyWidget extends StatefulWidget {
  const MyWidget({super.key});

  @override
  State<MyWidget> createState() => _MyWidgetState();
}

class _MyWidgetState extends State<MyWidget> {
  @override
  Widget build(BuildContext context) {
    return InteractiveViewer(
      child: Stack(
        children: <Widget>[
          Positioned(
            left: 0,
            top: 700,
            right: 20,
            bottom: 10,
            child: TextFormField(),
          ),
        ],
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.10.6, on Microsoft Windows [Version 10.0.19044.3208], locale en-US)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Professional 2022 17.3.4)
[√] Android Studio (version 2021.3)
[√] VS Code, 64-bit edition (version 1.80.1)
[√] Connected device (3 available)
[√] Network resources

Hi @deepika134. Thank you for filing this. I am able to reproduce this on the latest master and stable channels. Can you try this out if it works in your case?

Hi @deepika134,
The widgets you are using appear to be working as intended.
Your TextField is inside of a Stack and additionally a Positioned, meaning it is at a fixed position on the screen making it impossible for any other Widget to move the TextField into view when the keyboard opens. This is also prone to failing on screens with different sizes, clipping your TextField out of view.
Your Stack is also wrapped inside of an InteractiveViewer, which is a Widget to pan accross its children;
This does not seem to be fitting to contain a TextField.
I cant quite make out what your intention is with those widgets but if you wish to create a TextField that sticks to the bottom of the screen and reacts to the keyboard showing up you could look into showBottomSheet.
If you need help planning out your layouts I can strongly recommend you check out the community resources:
https://flutter.dev/community

Thanks for your input @clragon. I agree with this, @deepika134 Kindly look out for how you are planning your layouts, you might want to describe what you are trying to achieve if you think its an issue with flutter itself.


Resolve issues with PowerCalc template

Requirements

[x] Resolve issue where sensor.grid_power does not work after startup.

Resolved by #645


Fixes for Puppet 4 strict_variables

Add default values for variables that break compilation when strict_variables is enabled in Puppet 4

👍

This looks good, thanks!
Any suggestion for how we can enforce this in the tests?
The documentation suggests that setting the environment variable STRICT_VARIABLES=yes is sufficient, but I can't generate a test failure with this:
STRICT_VARIABLES=yes LOGSTASH_VERSION=5.2.2 PUPPET_VERSION=4.9.4 BEAKER_set=ubuntu-1604 bundle exec rake beaker

Ah OK. Found a way. It's not very nice, but it works :)
https://github.com/elastic/puppet-logstash/compare/strict-variables?expand=1#diff-174f7dfb508155027cb4c3e9a1840591

Nice. The tests now enforce this rule and your patch looks good.
https://travis-ci.org/elastic/puppet-logstash/builds/215382169
Please consider signing the open-source contributor agreement and I'll get this merged and released.
Thanks again.

@jarpy Any chance of getting this in now that the CLA has been signed?

Of course! This thread was quiet so I didn't realise that the CLA was signed. Thanks again for the patch, @mrbanzai.

Release on Puppet Forge as version 5.2.1
https://forge.puppet.com/elastic/logstash/5.2.1/readme

Oops, sorry for the ninja-sign.  Thanks for the merge!


Remove custom domain issue as it is fixed in PDF Generation v1.1.1

@pratikgoyal9
Hi Praktik,
Luyao is away until Monday. Can this PR wait until then, or does it need to be published immediately?

@MarkvanMents Sure, it can wait until Monday.

@pratikgoyal9 Hi Praktik, Luyao is away until Monday. Can this PR wait until then, or does it need to be published immediately?

I checked with my team. We've released a version on marketplace with the fix. It would be nice to release soon just incase the docs are updated with the known issues as well. If it's possible, it would be nice else ofcourse we can wait until Monday 👍


Support for h.265 codec

Is the h.265 codec supported? As it is currently, the web demo isn't showing the video output for a h.265 encoded video.

https://github.com/media-kit/media-kit?tab=readme-ov-file#supported-formats


docker swarm service update

Is it possible to trigger an update to a docker swarm service ?

At the moment watchtower can only update local running containers. I've briefly taken a look into the api, I think it should be possible to enhance watchtower with this functionality.

That would be of great help for us as well. I hope we see this functionality soon. Thank you for your hard work

this feature is must for future. Im looking forward to see this one day

🙏

:+1:

Would deploying watchtower service to all manager & worker nodes in a swarm cluster achieve the same functionality as is in non-swarmed nodes??

@oguya No, it does not work, because swarm manages the running containers itself.
As it turns out, automatically updating containers in a swarm-mode cluster is a lot simpler, since swarm itself queries the registry and updates the running containers every time you call swarm update --image. That means you just need to trigger a service update.
That is what I have just done with shepherd.

@djmaze looks interesting.. I assume shepherd needs to be constrained to the manager ?

@danieljuhl You are right, of course. Also there needs to be exactly one replica running.
I updated the README accordingly. Thanks!

+1

Is there any update on this issue?

+1 https://github.com/djmaze/shepherd seem to be working fine. Just a concern that I see that logs talks about updating service all the time, regardless if the base image is changed or not...

@dhruvit01 exactly, That's an issue I'm experiencing with shepherd... And would like watchtower to pick up where they slack off.

@Dunky13 : Agree. At this point I am using both but would like to have watch tower to support swarm. Any news/timeline on if/when this should be coming in?

With that said; what is theactual functional requirements? Would someone even like to have a stab at implementing it? If so, feel free to - I'll make sure to review the PR. Thanks 🙏

Don't think this should be closed or stale imho. It's an issue that can/should be addressed in the future. Closing it will put in too much in the background.

If I use local containers and swarm containers will this try to update the swarm containers?
Does this at least support swarm insofar as it will skip them?

@sixcorners I dont know to be totally honest as I don't use swarms myself. Feel free to try and report back! 😅

+1

Closing this issue. As @stffabi stated previously, the swarm support is currently extremely limited (if any). If you have any more specific errors, or would like to have a stab at implementing more thoroughly covering swarm support, please open a new issue and/or PR respectively.
Thank you.


-FILTER error. output not going to stdout

I Just cloned impala-tpcds-kit. When running ./gen-dims.sh and ./gen-facts.sh
I get:
ERROR: option 'FILTER' or its argument unknown.
the usage text that follows shows "_FILTER" should be used instead.
There is no more error, but the data is copied to a local file instead of stdout so no data is piped to hdfs
as the script intends to:
${TPCDS_ROOT}/tools/dsdgen
-TABLE $t
-SCALE ${TPCDS_SCALE_FACTOR}
-DISTRIBUTIONS ${TPCDS_ROOT}/tools/tpcds.idx
-TERMINATE N
-FILTER Y
-QUIET Y | hdfs dfs -put - ${FLATFILE_HDFS_ROOT}/${t}/${t}.dat &

I recently made some changes to the repo.  This should get you back the the version that works.
git clone https://github.com/gregrahn/tpcds-kit.git
cd tpcds-kit
git reset --hard eff5de2c30337b71cc0dc1976147742d2c65d378

Thanks, that fixed it!


Fix gateway container log file mount

Which issue this PR addresses:
None

Fixes
This fixes an incorrect ordering of the gateway container log mounts, preventing it from coming online.
What this PR does / why we need it:
See slack thread

Test plan for issue:
Running deploy against INT with this pull request.

Is there any documentation that needs to be updated for this PR?

lgtm - can you update the description with the issue in a bit more detail.

This fixes an incorrect ordering of the gateway logrotation directory which prevents the gateway service from coming online.

No problem, updated the description with more info.


reduce number of conditions

I tried to reduce the number of conditions (if changed_since = adaptive) so that the handling occurs at (ideally) a single place. What do you think?

cool, thanks for this, what a nice surprise to wake up to 😄 .
yeah I wanted to put the meat of it directly into the Reader method, not sure why I didn't.  Thanks also for fixing the no-tags case


Add drag and drop upload using dropzone.js

Love this PR! Thank you @harimayco

Thank you @harimayco. This is awsome!!

Any documentation how to use this?

@ibnuh I think the video explains everything.
https://postimg.org/image/llfungf6t/ (by @harimayco)

Yes, thank you very much! @youchenlee @harimayco. It works now
Just started using laravel for a week and I found it really awesome! Especially with this laravel-filemanager plugin.


MRPHS-3566 : Shutdown operation after restart, reset is getting failed.

Jira id
MRPHS-3566
Problem
Shutdown operation after restart, reset is getting failed as the tools are not started after the vm is reset/restart.
Solution
wait for the machine status to turn to green after the vm is reset/restarted
Unit Testing
Tried restart and reset on vms where vmware tools are installed and not installed.

Action - Restart, Tools - Installed
[ Result : Machine is restarted and shut down immediately after restart. ]

Action - Reset, Tools - Installed
[ Result : Machine is reset and shut down immediately after reset. ]

Action - Restart, Tools - Not Installed
[ Result : Machine is not restarted and error is returned ]

Action - Restart, Tools - Not Installed
[ Result : Machine is restarted and shut down immediately after reset fails as tools are not installed. Error is returned. ]

Please paste unit test logs here.

Testing logs:
c3ntry.txt


CAMEL-10641: Load Camel TypeConverters when a new TypeConverterLoader…

… OSGI service is detected
https://issues.apache.org/jira/browse/CAMEL-10641
Note https://github.com/apache/camel/pull/1371 is needed to run the tests.

Yeah sorry about that, fixed :)

Thanks for the PR it has been merged. Do you mind closing this?

Thanks for looking into this :)


[router] 4.10 Missing validation on receiverPrepare

Some missing validation:

bidExpiry - If expired, cancel
re-verify against configured allowedSwaps


Create System OS

Reasons for making this change:

TODO
Links to documentation supporting these rule changes:
TODO
If this is a new template:

Link to application or project’s homepage: TODO

To hide OS files


Configure dependabot to only open security-related PRs

Related Issue or Background Info
Dependabot is currently opening PRs for any version bump of all packages, which is way too much. This PR configures dependabot to only open security-related PRs.
Changes Proposed

Sets the open-pull-requests-limit Dependabot option to 0. According to the docs, this only affects non-security related PRs, effectively meaning we should only get security-related PRs now.

Docs: https://docs.github.com/en/code-security/supply-chain-security/configuration-options-for-dependency-updates#open-pull-requests-limit
Checklist for Author and Reviewer
UI

[N/A] Any changes to the UI/UX have been approved by design
[N/A] Any new or edited error messages have been approved by design

Testing

[x] Includes a summary of what a code reviewer should verify

Changes are Backwards Compatible

[N/A] Database changes are submitted as a separate PR
[N/A] GraphQL schema changes are backward compatible with older version of the front-end

Security

[N/A] Changes with security implications have been approved by a security engineer (changes to  authentication, encryption, handling of PII, etc.)
[N/A] Any dependencies introduced have been vetted and discussed

Cloud

[N/A] DevOps team has been notified if PR requires ops support

I would actually rather leave the backend and workflow-action updaters on full verbosity—the dependency trees there are pretty short, and dismissing a few PRs is a comparatively low price to pay for getting informed when some things that desperately need updating are getting updates. 😄


nftables: flatten rulesetFile when check

Description of changes
When we use rulesetFile for networking.nftables, preCheckRuleset will not work because ruleset.conf is just including it. So flatten the file might be helpful.
Use case:
networking = {
    useDHCP = false;
    firewall.enable = false;
    nftables = {
        enable = true;
        rulesetFile = ./firewall.nft;
        preCheckRuleset = ''
            sed -i 's/skuid clash/skgid nogroup/g' ruleset.conf
        '';
    };
    ...
};

Related: #230745
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[x] Fits CONTRIBUTING.md.

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

This commit ("Merge branch 'NixOS:master' into nft-precheck-flatten") can be avoided by rebasing your commit against the master branch and then force-pushing it.

Tested on my router.

Certainly, rebased.


Create README.md

Just an artificial pull-request, which triggers FindBugs on the pull request builder

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

@mambu
Thanks a lot. Now the pull request builder will start printing warnings on https://jenkins.ci.cloudbees.com/job/plugins/job/dashboard-view-plugin/ if FindBugs check fails


Loop through illegal Character Replacement

Hello,
There is an issue with values that contain multiple illegal characters. For example if a string contains multiple ampersand signs it sends an incorrectly constructed url. I think line 14 to 16 of odatavalue.js should perform a while loop to replace the illegal characters like so:
for (var key in illegalChars) {
    while (string.indexOf(key) != -1) {
        string = string.replace(key, illegalChars[key]);
    }
}

instead of
for (var key in illegalChars) {
    string = string.replace(key, illegalChars[key]);
}

Thanks
Patrick

I've made this fix and created a pull request.


feat: Cobra Kai Fira theme

Submit a new Plugin to Marketplace
Plugin Github repo URL:
Github releases checklist

[x] a legal package.json file.
[ ] a valid CI workflow build action for Github releases. (Optional for theme plugin only).
[-] a release which includes a release zip pkg from a successful build.

No zip pkg.

[x] a clear README file, ideally with an image or gif showcase. (For more friendly to users, it is recommended to have English version description).
[x] a license in the LICENSE file.

hmm, this is just Cobra theme with a different font?

Yes, it is. In addition, the original Cobra theme fails to be installed from the marketplace, I think that because it has not a release (see https://github.com/santiyounger/Cobra/issues/6).


Will you release the codes especially the calculation of CHAIR on  NoCaps Dataset?

Hi, I 've read your paper multiple times and found it very inspiring. I'm interested in learning more about how to compute the CHAIR metric on the NoCaps dataset since it has not been used on this dataset before. Additionally, I was wondering if you have any plans to release your codes for this purpose.
Thank you very much.

also interested in

also interested in


added 'last updated' value on the dashboard

Added last updated value on both Imaqal and Worldbank to their correct values and also included an alert in case the updated time exceeds 30 minutes.

I have added alert class to make the alert noticeable as mentioned on the issue by Luke

I can't see any correction left on my side can you comment on that part you showed me its pending


Check for previous/existing GitHub issues/module proposals

[X] I have checked for previous/existing GitHub issues/module proposals.

Check this module doesn't already exist in the module indexes

[X] I have checked for that this module doesn't already exist in the module indexes.

Bicep or Terraform?
Bicep
Module Classification?
Pattern Module
Module Name
avm/ptn/azd/aks
Module Details
Please add a Pattern Module named avm/ptn/container-service/aks, this module will create the following three resources:

Azure Kubernetes Service (AKS) Managed Cluster.
Azure Container Registry (ACR).
Key Vault.

For details, please refer to azd:
https://github.com/Azure/azure-dev/blob/main/templates/common/infra/bicep/core/host/aks.bicep
CC: @jongio
Do you want to be the owner of this module?
No
Module Owner's GitHub Username (handle)
jongio
(Optional) Secondary Module Owner's GitHub Username (handle)
No response

@Menghua1, thank you for proposing this module and thank you for your patience on triaging it! Many of us on the AVM core team have been OOF in the last couple of weeks, but we're back now! 😊
As this is categorized as a pattern module, its approval will require a more thorough assessment. We'll reach out to @jongio internally for more context and details as this proposal may require some adjustments to fit AVM's pattern definition and intent. The AVM core team will get back to you soon. Thank you for your patience!

Hi @jongio,
Thanks for requesting/proposing to be an AVM module owner!
We just want to confirm you agree to the below pages that define what module ownership means:

Team Definitions & RACI
Shared Specification (Bicep & Terraform)
Module Support

Any questions or clarifications needed, let us know!
If you agree, please just reply to this issue with the exact sentence below (as this helps with our automation 👍):
"I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"
Thanks,
The AVM Core Team
#RR

I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER

Hi @jongio,
Thanks for requesting/proposing to be an AVM module owner!
We just want to confirm you agree to the below pages that define what module ownership means:

Team Definitions & RACI
Shared Specification (Bicep & Terraform)
Module Support

Any questions or clarifications needed, let us know!
If you agree, please just reply to this issue with the exact sentence below (as this helps with our automation 👍):
"I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"
Thanks,
The AVM Core Team
#RR

Module published. Index to be updated soon.


LibWeb: Make Selection APIs throw on DocumentType node inputs

This matches the behavior of all major engines, and is covered by hundreds of subtests in WPT.
Spec PR: https://github.com/w3c/selection-api/pull/342

Not importing WPT tests here since the /selection/ tests is verrrrryyy slow in our test runner. Need to investigate that separately.


Use a buffered channel for singal

See https://pkg.go.dev/os/signal#Notify .

Thanks for the PR. I see. Just a little confused as to whether there's a downside to the current approach which works. Did you face any issues running the program?


[sw, dif_rstmgr] Add DIF Reset Manager unittests

[sw, dif_rstmgr] Add DIF Reset Manager unittests
Related to #3454 .

Ping. If there are no major issues with this, can we merge it?


Reload features from Parquet metadata

The idea would be to allow this :
ds.to_parquet("my_dataset/ds.parquet")
reloaded = load_dataset("my_dataset")
assert ds.features == reloaded.features

And it should also work with Image and Audio types (right now they're reloaded as a dict type)
This can be implemented by storing and reading the feature types in the parquet metadata, as we do for arrow files.

I'd be happy to have a look, if nobody else has started working on this yet @lhoestq.
It seems to me that for arrow features are currently attached as metadata here  and retrieved from the metadata at load_dataset time here.
This will need to be replicated for parquet via calls to this api from io.parquet.ParquetWriter and io.parquet.ParquetReader respectively.
Any other important considerations?

Thanks @MFreidank ! That's correct :)
Reading the metadata to infer the features can be ideally done in the parquet.py file in packaged_builder when a parquet file is read. You can cast the arrow table to the schema you get from the features.arrow_schema

#self-assign


[BUG] Not able to get all apps that use a Hybrid Connection in an App Service Plan

Library name and version
Azure.ResourceManager.AppService
Describe the bug
Get all apps that use a Hybrid Connection in an App Service Plan.
Request:
GET /subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Web/serverfarms/{name}/hybridConnectionNamespaces/{namespaceName}/relays/{relayName}/sites<>?api-version=2024-04-01
Response:
The requested operation requires an element of type 'String', but the target element has type 'Object'.
Expected behavior
Correctly get all apps that use a Hybrid Connection in an App Service Plan
Actual behavior
Not able to parse request response to expected type (string)
Reproduction Steps
var appServicePlan = await _azureResourceGroup.GetAppServicePlanAsync("someAppServicePlanName");
var appServicePlanResource = await appServicePlan.Value.GetAsync();
var relays = appServicePlanResource.Value.GetHybridConnectionRelaysAsync();
await foreach (var relayOverview in relays)
{
var relay = await appServicePlanResource.Value.GetAppServicePlanHybridConnectionNamespaceRelayAsync(relayOverview.ServiceBusNamespace,
relayOverview.Name);
var listOfAppServicesWhichAreUsingHybridConnection = relay.Value.GetWebAppsByHybridConnectionAsync();
}
Environment
No response

The same problem for sync method GetWebAppsByHybridConnection()

Thank you @JagodaMika for your feedback, @HarveyLink please look into this issue.

@JagodaMika Thank you for your feedback. I tried to reproduce the issue using the code you provided, but I did not encounter the problem you mentioned. Here is the code I used for reproduction:
AppServicePlanResource plan = client.GetAppServicePlanResource(new ResourceIdentifier("myResourceId"));
var relays = plan.GetHybridConnectionRelaysAsync();
await foreach (var relayOverview in relays)
{
    var relay = await plan.GetAppServicePlanHybridConnectionNamespaceRelayAsync(relayOverview.ServiceBusNamespace, relayOverview.Name);
    var listOfAppServicesWhichAreUsingHybridConnection = relay.Value.GetWebAppsByHybridConnectionAsync();
}

(I used an AppService plan resource that contains a web app, and the web app is configured with two HybridConnections). Based on the tests conducted with version 1.3.0 of AppService, I did not find this issue. If possible, please provide more information so that we can better assist you in resolving the problem.

Hi @mcgallan The code you wrote works, but when you try to iterate through 'listOfAppServicesWhichAreUsingHybridConnection' to get actual values from it, then you will receive an exception which I mentioned:
System.InvalidOperationException: 'The requested operation requires an element of type 'String', but the target element has type 'Object'.'
Sorry I didn't clear that out.


[Learn] Ops pathway

See https://github.com/pulumi/pulumi-hugo/tree/devrel/learn-operations
Ref: #828

Assigning this to @interurban; this may be an issue you'll want to close as not planned once you and Torian sort out the plan for tutorials.

Closing as not planned per team consensus


Setup Instructions for Windows not accurate

The setup instructions say the model needs to be extracted extracted into ~/.talon/vosk/, but I think it needs to be %APPDATA%\talon\vosk\ on windows, and the actual model path is something like C:\Users\<USER>\AppData\Roaming\talon\vosk\vosk-model-de-0.6.
I think it would be great to update the README accordingly =)

Oh god I saw this just now, going to try to get to it


Add support for vw/vh/vmin/vmax units

The CSS Polyfill does not support Viewport Units. This PR adds support for vw/vh/vmin/vmax units. It doesn’t cover vi/vb units.
One possible improvement could be to add a resize observer onto the window, which would calculage and cache the px values for the vw/vh units.
Demo which uses this version: https://codepen.io/bramus/pen/ExgOPRw

Anything that can be done to help get this PR through? I just ran into this not being available so I'm managing window dimensions with a resize event and then passing them to CSSUnitValue for now.

So I think with the new API (@kevers-google PR) the only parsed length is currently view-timeline-inset. It may be worth having support for viewport units there (when the polyfill supports this) and in possible future length additions, but I think we'll want to be careful to properly track invalidations. I.e. when something is based on viewport units it should trigger an invalidation on window resize events.
The TLDR, we could add this, with TODO's for missing pieces (e.g. invalidations on viewport size changes). It currently isn't used by anything, but will likely be a benefit in the future to support these units.

I tried to merge this with the changes on the main branch, but the modified calculateScrollOffset function no longer exists after being removed in https://github.com/flackr/scroll-timeline/pull/60/files#diff-34c57dd70dd577f57be1349b4bf48ccbd5f91db5abaae114048da4d5d57b3412L135
I'm not quite sure if we have anything equivalent to this function afterwards, @kevers-google ?

I’ll try to see where exactly this would fit in nowadays.

@bramus
We have implemented the simplify a calculation tree procedure to resolve values for animation range start and end, in order to support math functions such as calc(), min() and max(). Currently we pass in the information that is available for resolving values as a parameter.
https://github.com/flackr/scroll-timeline/blob/5a2639adcd5691004177b10bd9970f4cea4b4fc0/src/scroll-timeline-base.js#L676-L680
We could resolve the viewport units there:
https://github.com/flackr/scroll-timeline/blob/5a2639adcd5691004177b10bd9970f4cea4b4fc0/src/simplify-calculation.js#L75-L83
I just pushed a draft PR where I use the procedure to support CSSUnitValues for ViewTimeline inset:
(That PR depends on #176)
https://github.com/flackr/scroll-timeline/blob/ab45fa780aa1da0023dfcf3a36be8b11a676e973/src/scroll-timeline-base.js#L753-L756


[BUG]: StripeElements defaultValues aren't working

What happened?
I am trying to set default country value for the Stripe payment element.
loadStripe(this.config.stripeApiKey)
.then((stripe) => {

    // Create a Stripe client
    this.stripePayment = stripe;

    this.stripeElements = this.stripePayment.elements({
        clientSecret: setupIntent.clientSecret,
        locale: 'en'
    } as never);

    this.paymentElement = this.stripeElements.create('payment', {
        defaultValues: {
            billingDetails: {
                address: {
                    country: 'United States'
                }
            }
        }
    });
    this.paymentElement.mount('#stripe-payment-element');

});

But the country is still the one got from my IP address - Poland.
Environment
Chrome 120.0.6099.234 and MacOs Ventura 13.4
Reproduction
No response

Hi, this looks like an integration question, and we aren’t able to answer those here on GitHub. Please reach out to Stripe Support for help.


Add OS signal sending from process

The idea is that we can have messages like sigterm, sigkill so that we can send such signals to a particular child process.  We can base the effort in what OSProcess has. See class ExternalUnixOSProcess protocol 'OS signal sending' and class UnixProcess protocol 'OS signal sending'

Max,
I added the support for signals and you can see the 2 tests I added:  testSigTermInsideLoop and testSigTerm. I think such support (sigterm) is enough as for what matters to the framework.
What I originally thought about killIfTakesLongerThan: is too much dependent of a particular scenario. Others may want another signal or another condition. So I think the framework should just allow signaling and then it's up to the user to decide what signal and when to send it.
Agree?

I can't find those tests. Where are they?

Sorry Max, those are in the master, not in the stable release.
You must execute something like:
Metacello new
    baseline: 'OSSubprocess';
    repository: 'github://marianopeck/OSSubprocess:master/repository';
    load.

BTW, I was planning to add methods like terminate and kill which will internally do sigterm and sigkill.

What I originally thought about killIfTakesLongerThan: is too much dependent of a particular scenario. Others may want another signal or another condition. So I think the framework should just allow signaling and then it's up to the user to decide what signal and when to send it.

Yes, I agree.
I noticed that #sigTerm uses a primitive for exactly that signal. I'm used to the kill command and to the way signals are specified there (e.g. by number or name). Naming a signal explicitly makes it more flexible I think, especially since you would only need a single primitive to handle all signals. You should still offer convenience methods for accessing the signals of course (I don't think you really want users specifying signal names as strings :))

OK, we already have all singals sending (a method per signal) and we also included 2 faclities methods terminate and kill.


What should be in "My sources"?

How are sources supposed to show up in "My sources"? I am an editor for some sources but they don't seem to be in "my sources", so what determines this?

It looks like the logic is that it includes sources that were either (a) created by you or (b) for which you are listed as a "current editor".
Could you give me an example of such a source, and then I can look at which "role" you have with the source that isn't one of these two?

wait, I found the sources I thought I was lacking after all--weirdly, buried in different places in my (very long) list of sources, even though I thought I was added to both as an editor at the same time pretty recently (so I was expecting them together and near the surface.)  So the logic is sound, it's actually a question of the order things are presented to the user (probably the most recently modified should be at the top?)

There is also the "Sources created by user" sidebar, but to me that makes more sense to keep it ordered by date created instead of date updated.

There is also the "Sources created by user" sidebar. To me it makes more sense to keep it ordered by date created instead of date updated, but we can easily change that as well.

Ok, in testing on staging it looks like #1472 changed the ordering of the "Sources created by user" sidebar, but not the main source list on the "My Source" page.


[WIP]Add jdk http plugin

Please answer these questions before submitting pull request

Why submit this pull request?

[ ] Bug fix

[x] New feature provided

[ ] Improve performance

Related issues
#2709

Bug fix

Bug description.

How to fix?

New feature or improvement

Describe the details and related test reports.

OK

@candyleer Let me do some JDK9 test tomorrow, and how it works based on @raphw suggestions.

JDK 9-11 tests done in #3194

I test this plugin from jdk1.6 to jdk11 ,all works well.

I think we should add a test for this. @ascrutae How to copy a new optional folder? Which line should change?

/run e2e

Let's follow the document of bootstrap instrumentation, move this to bootstrap-plugins folder, and make agent test works.

Look forward this PR included in 6.3

/run e2e

/run e2e

/run e2e

A new update for JDK12 #3197

/run e2e

Testcase PR:https://github.com/SkyAPMTest/agent-auto-integration-testcases/pull/81
Report: https://github.com/SkyAPMTest/agent-integration-test-report/blob/master/2019/8/candyleer/testReport-feature-jdk-http-plugin-2019-08-02-19-32.md
Tested on jdk8
Test Report - 2019-08-02 19:32

tests  : 1 passed. 0 failed
branch name : add-http-plugin
commit id : 19169d2e1ac8c897b7976534f3bd0a45cf55bd60
cases branch: feature-jdk-http-plugin
cases commit id: 89e03c0779a816ac0f0d1c8786c371b184cfcdbe

Cases List

Case description
Status
Cases

JdkHttp
1 passed. 0 failed
click me

JdkHttp

Version
Passed
Failed

all
:heavy_check_mark:

/run e2e

/run e2e

@kezhenxu94 reboot case is failing. Please verify, is it related to this PR?

@candyleer You could run e2e reboot locally, to recheck too.

/run e2e


Added to support BYOD (simple entities) label translation

I think this line needs an if statement so it doesn't run if @translateBYOD_enums parameter is false? Otherwise it always tries to run and fails if the srsanlysisenums table isn't deployed.
execute sp_executesql @srsanalysisenumsTemplate;

Thanks Joel,
I'll take a look at that as soon as possible.
Regards,
Steve
Steven Koppens
Mobile:   +64 21 2426332
<EMAIL_ADDRESS>Advanced notification of vacation:
N/A

From: Joel Leichty @.>
Sent: Wednesday, 3 July 2024 5:54 am
To: microsoft/Dynamics-365-FastTrack-Implementation-Assets @.>
Cc: Steven Koppens @.>; State change @.>
Subject: Re: [microsoft/Dynamics-365-FastTrack-Implementation-Assets] Added to support BYOD (simple entities) label translation (PR #332)
I think this line needs an if statement so it doesn't run if @translateBYOD_enums parameter is false? Otherwise it always tries to run and fails if the srsanlysisenums table isn't deployed.
execute sp_executesql @srsanalysisenumsTemplate;
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/Dynamics-365-FastTrack-Implementation-Assets/pull/332#issuecomment-2203969948, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APKCUPE3CPBQJQFQAFLYP7TZKLSOJAVCNFSM6AAAAABJRHGL72VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBTHE3DSOJUHA.
You are receiving this because you modified the open/close state.Message ID: @.***>


#234 check port before starting

The exception is falsely raised when use_reloader is True and the reloader kicks in :-(
Any idea @dinhlongviolin1 ?


Newly created .mdx files give a 404 when reloading the page

Bug Report
Describe the bug
If you have docz running the newly created .mdx file appears nicely in the sidebar and navigating to it works fine but if you reload the page after that it gives you a 404 and you have to restart docz to have it appear again.
To Reproduce

Run docz:dev
Create a new .mdx file
Navigate to that newly created .mdx file
Reload the page
404

Expected behavior
After reloading the page it should correctly display all created .mdx files that were created when docz is running.
Environment

OS: OS X 10.13.6
Node/npm version: Node 10.0.0

Try it now with v0.10.2 @rovansteen, was a bug inside our cache system 🙏

I'm getting this issue in v2.1.1, how do I fix it?

I'm getting this issue in v2.1.1, how do I fix it?

Same here, v2.2.0.


Company Communicator - "An error occured" and "You don't have permission"

I'm facing 2 errors with Company Communitor on Teams.
Teams Desktop App - "Sorry, you don't have permission to access this page. Contact the administrator to receive permission."
Teams Web App - "An error occured while trying to access this service."
I'm attaching the screenshots.
I was looking at the others Issues posted here, I understood that I must configure via Azure to Sync to another URL.
I've done that but the error is still occuring. I attached the screenshot so you guys can see if I've done anything wrong.

Hi @silva1512 ,
Thanks for raising the query. Please confirm on the below point.
-" I've done that but the error is still occurring". as you said, please mention the steps performed.

here it is
https://github.com/OfficeDev/microsoft-teams-apps-company-communicator/issues/1244#issuecomment-1835498617

Hi @silva1512 ,
Thank you for raising the query. The app service sync failure issue is now resolved, and the changes have been pushed to Company Communicator official GitHub repository.
If current deployed version of the Company Communicator application is pointing to v5.5 (latest version), please follow the below procedure to resolve the issue -

Go to Azure Portal --> Go to Resource Group (where the CC deployment has been done) --> Select Web App (App Service) --> Go to Deployment Center --> Click on SYNC

Once the status shows successful within logs section of Deployment Center.

Note: Please make sure the Repo URL and Branch name is pointing to below values in Deployment Center Screen of app service

Try accessing the CC application on Teams web client to avoid any cache related issue.

If current deployed version of the Company Communicator application is pointing to v5.4.1 or previous versions, please upgrade your app to latest version by following the migration guide as the fix is pushed to the main branch in Company Communicator's official GitHub repo.
If it’s a fresh deployment then proceed with the remaining steps as per the deployment guide.
Please let us know if you are still facing the issue.

"If current deployed version of the Company Communicator application is pointing to v5.5 (latest version)"
Where can I check this?

Hello gsv022,
Thank you for the information provided.
I was able to check and we're currently running the version 5.1.0

Well, I followed the procedure you sent me but had no success.
I'm attaching the logs. I believe there's something that is preventing the app to be updated.
As you can see, it shows that a log from June 14, 2022 is still trying to deploy something...
Also, I'm attaching another error that is appearing for us, I believe it's related to the version being 5.1.0

Hi @silva1512 ,
Since you are on v5.1.0, please perform migration steps as per the guide. If performed already, please share the screenshot as shown in the migration guide for function apps and app service

Alright,
I already followed the guide, you sent me in the other reply.
I understood that this v5.1.0 is the one being "redeployed" and when I try to fetch the v5.5 it shows an error.
So, it seems like it is "stuck" at v5.1.0...
Do you know where I can look at this "Diagnostic Logs" ?

As you can see, whenever I try to sync it starts to run the deployment from v5.1.0, i don't know why...

Hi @silva1512 ,
Currently you are in the logs section of app service, switch to settings in the same page. Then disconnect and connect the repository pointing to main branch. After that check the deployment status in the logs section of deployment center. Do the same for all resources(data-function, send function, prep-function)

Hi @gsv022 ,
Thank you for your help.
I was able to update to version 5.5 and this error has been solved.
Unfortunately, another error appeared, but since the main error was solved, i'll create a separate issue for the other one.
I solved by disconnecting and connecting the external source - Git.
I appreciate your help! Thanks again.


LAMMPS tutorial not working

The LAMMPS tutorial is currently not working.
The issue starts with the line atoms = frame_data_to_ase(client.first_frame, topology=True, positions=False), which raises a NoneType error.
This raises the question of whether we are able to maintain the LAMMPS integration.
We have been discussing using the state-utils repo as a place to put unmaintained utilities scripts/code (see the issue on the repo here). We could move these tutorials to that repo so that they are still available for use but people are aware that they are not being maintained. Otherwise, we should either remove this tutorial or update it so that it's working.

Agree that it's unclear whether we are able to maintain it. I'm not sure if anyone actually uses it--possibly @jc14269 or @mcsloy ? If we do want to continue supporting it then it'd be ideal to update it to fit in with OmniRunner to make it a little less work to maintain.

The problem with LAMMPS data files is that they don't inherently specify atom types but instead atom masses. We've got around when visualising LAMMPS trajectories in VR by using dictionaries to convert the masses to atom elements.
But I haven't used it really at all apart from to visualise a trajectory, so I'm not really sure if its worth it at this point.

Closed since we aren't supporting LAMMPS for now


Gpio/add simulated devices

Summary of the PR
Add simulated devices, for review only. I ran aground a little in converting the start_backend from a templified call to one that can deal with multiple device types. Any advice welcome.
Requirements
Before submitting your PR, please make sure you addressed the following
requirements:

[x] All commits in this PR are signed (with git commit -s), and the commit
message has max 60 characters for the summary and max 75 characters for each
description line.
[x] All added/changed functionality has a corresponding unit/integration
test.
[x] All added/changed public-facing functionality has entries in the "Upcoming
Release" section of CHANGELOG.md (if no such section exists, please create one).
[x] Any newly added unsafe code is properly documented.

@vireshk I think I've addressed everything aside from the annoying musl bug which I'm not sure the right cfg fragement

@stefano-garzarella please approve this one too.

I must admit that I do not really like how the #[cfg] blocks need to be distributed through the code... I am not seeing a good reason for why we need this enum to differ the types at all.
After all, the idea is that simulated devices can be dropped in to mock real ones, right? So why don't we just refactor the code so that - other than actually creating the devices - there is no distinguishment. Instead of introducing a GpioDeviceType I would just use &GpioDevice (or maybe Box<dyn GpioDevice> if needed).
I will play around and see whether I can come up with something...

The only thing I cared about was to avoid increasing size of the end binary for normal use case. There should be a way for us to drop all tests and the mock device from that. That's all.

I commented per-commit, which renders some comments as "outdated" since the code got moved in later commits. Hopefully it still makes sense. Unfortunately, github does not display on which commit I commented :/

That said. I personally do not see a big issue with at least enabling this feature by default. It feels like this would make development a lot simpler compared to needing to test different feature sets all the time.

During development we can always use --all-features, isn't it ? I always thought that default features are the one that are used in installations, everyone else knows the internal details and can select a few extra options ?

That said. I personally do not see a big issue with at least enabling this feature by default. It feels like this would make development a lot simpler compared to needing to test different feature sets all the time.

During development we can always use --all-features, isn't it ? I always thought that default features are the one that are used in installations, everyone else knows the internal details and can select a few extra options ?

Yes. But passing --all-features to LSP, tests, ... is similary cumbersome as needing to set PKG_CONFIG_PATHs today :). With the exception that PKG_CONFIG_PATHs are going away once newer libgpiod versions are more widely available. Personally, I think any feature acts as a multiplicator on development, documentation and testing overhead. There are of course good reasons to do it in order to limit exposure to unneeded dependencies and binary size. But here it is just a little bit of code size. In fact I just tested it and measure 7.2M vs 7.1M. A ~1.5% increase in release build size.
Overall, I am not convinced that it is worth the maintenance overhead. But that is just my 2 cents. I am happy with getting it merged they way it is currently designed. I just suggested the default feature as a balance between our standpoints :). But if you prefer it off by default that's fine too. With the other suggestions that I made, I think we can lower the complexity regardless of whether we disable the feature by default or not.

Yes. But passing --all-features to LSP, tests, ... is similary cumbersome as needing to set PKG_CONFIG_PATHs today :). With the exception that PKG_CONFIG_PATHs are going away once newer libgpiod versions are more widely available. Personally, I think any feature acts as a multiplicator on development, documentation and testing overhead. There are of course good reasons to do it in order to limit exposure to unneeded dependencies and binary size. But here it is just a little bit of code size. In fact I just tested it and measure 7.2M vs 7.1M. A ~1.5% increase in release build size.

Overall, I am not convinced that it is worth the maintenance overhead. But that is just my 2 cents. I am happy with getting it merged they way it is currently designed. I just suggested the default feature as a balance between our standpoints :). But if you prefer it off by default that's fine too. With the other suggestions that I made, I think we can lower the complexity regardless of whether we disable the feature by default or not.

I am okay with keeping it in default if others want it that way.


JavaScript editor using Chrom browser on Mac OS has issue for editing.

One of the user of Node-RED reported that JavaScript editor ( function node ) is not usable. There is a problem in cursor position. For example, put the mouse cursor among the code, and then try to edit goes to different place.
Env: Mac OS
Browser: Chrom
If you need additional details, I can ask the developer to report the detail.

Function node editor seems to work fine for me on latest OSX / Chrome / Node-Red.

Please provide lots more information, such as steps to recreate, versions of node-red, chrome etc.
I develop using chrome on osx and have never seen anything like this.

@ibmamnt,
any more info on this ? or can we close ?

Oh, sorry. I asked the user to input. But looks like he did not respond to write error. OK to close for now.

My friend and I meet the same problem on Ubuntu system, I also try with firefox, but seems useless.
Chrom Version: 51.0.2704.79 Ubuntu 15.10 (64-bit)
node-red version: 0.17.3

I have the same problem. Chrome and Firefox on Arch-Linux. I suspect something with the ACE Editor not using a monospaced font like here: https://stackoverflow.com/questions/15183031/ace-editor-cursor-behaves-incorrectly.

I found the culprit: It is a problem with monospaced fonts. However if I change the css to just font-family: monospace it does not work. The original line is: Menlo, Consolas, 'DejaVu Sans Mono', Courier, monospace !important and if I remove Menlo,  it works. Maybe there is a non-monospaced font named Menlo on Linux? If I use monospace in Inkscape it does not look monospace, so maybe my font configuration is messed up.

good debugging. thanks


Implement New Item "Mystic Amulet" and Add Interaction in the Game

Description :
What was changed?
I implemented a new item, Mystic Amulet , which can be found in the game's **"Enchanted Forest .
The Mystic Amulet allows the player to gain a temporary shield, protecting them from one attack from enemies.
I added an interaction in the game where players can pick up the Mystic Amulet when they encounter it, and use it during their journey for extra protection.
Why was this change made?
The Mystic Amulet was added to introduce an exciting new gameplay mechanic. This item gives players an additional tool for strategic decision-making, especially when facing powerful enemies.
It also adds variety to the types of items available in the game, enhancing the overall player experience by providing them with a way to survive tougher battles.
What was tested?
I tested the item pickup mechanic to make sure the player can properly collect the Mystic Amulet when they enter the Enchanted Forest.
I also tested the shield effect to ensure the player receives the temporary protection after using the item and that the effect ends correctly after blocking an attack.
The item’s visual representation was checked to ensure it displays correctly in the inventory.
Additional Notes :

The Mystic Amulet’s power duration is currently set to 5 seconds. We might adjust this in the future depending on feedback and testing.
This item is still being tested in a few other levels, and we will monitor its impact on gameplay balance.

Checklist :

[x] New item "Mystic Amulet" implemented.
[x] Interaction for item pickup added in the "Enchanted Forest."
[x] Shield mechanic tested and functional.
[x] Item added to player inventory with correct display.
[ ] Balance adjustments for item duration may be required after additional testing.

Screenshots (Optional):

If relevant, you can add screenshots here, such as:

A screenshot showing the Mystic Amulet in the inventory.
A screenshot or gif showing the shield effect when the player uses the item.

Summary :
This pull request introduces the Mystic Amulet , a new item that grants players a temporary shield to protect them from enemy attacks. The item has been tested for functionality, and its mechanics are working as expected. Further balance testing is planned to ensure the item is well integrated into the overall game.

The described mechanics sound like an interesting addition to "Enchanted Forest" room.
I like that you clearly stated some aspects of the functionality (like the current length of the items effect) and which parts already worked in your tests.
However it seems like you might have forgotten to add the changes to the main branch before commiting, since the changes you made only affect the code for the table tennis room, which was already present. You can also see that this pull request has no conflicts with the base branch, which should not be the case when adding a new class.
After you change this, I can check if your code works for me as well and if I can approve it.


Playlist is getting recored as a single file

Hi,
after I got this project running I tried with a playlist and I was surprised that all the songs in this playlist are stored in one single mp3 file. Is this a proper behavior? Or did I miss some settings some where?
I user the application without docker.
Thx and regards :)

ok this project is definitely dead!


feat(share/discovery): Add callback on peers set updates.

Overview
Notify Discovery clients about peers set changes in async manner by calling provided callback.
(ps *limitedSet)Peers method splitted into two funcs: ListPeers and WaitPeer, so callback is not locked in case there are no peers.
Closes https://github.com/celestiaorg/celestia-node/issues/1610
Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

Updated with @distractedm1nd suggestion to only emit diff, not the full peers list. This removes some changes in set.go and in tests. As a note: callback provider will needs to be careful with keeping state up to date so it don't diverge from discovery one.


Add gradle wrapper checksum verification

Add steps/peoperty values to verify gradlewrapper checksum
the repo uses gradle-4.5.1
$ sha256sum gradle-wrapper.jar
3bf04e39ed259ff0a1217a875199a11775855d2a29207b98318ac79178249de8  gradle-wrapper.jar

$ cd gradle/wrapper
$ curl -s --location --output gradle-wrapper.jar.sha256 \
   https://services.gradle.org/distributions/gradle-4.5.1-wrapper.jar.sha256
$ echo "  gradle-wrapper.jar" >> gradle-wrapper.jar.sha256

$ sha256sum --check gradle-wrapper.jar.sha256

I've tried to set this up using GitHub Actions: https://github.com/marketplace/actions/gradle-wrapper-validation
Are you saying that action wasn't set up properly?

Its setup correctly...but to me the validation an arms length away:
if i were to git clone this repo, i'd like to "see" all the components that went into it (eg, source, bash scripts, etc).  The gradle wrapper is the opaque one that i woudn't be able to readily verify on my own (since its done implictly via github).   it'd help with transparency if there was a line in the readme like "If you would like to verify the integrity of the gradle wrapper manually, see Verifying the integrity of the Gradle Wrapper JAR"

I already had a comment in the readme about validation of the jar using the github action. It's easy enough to add a link to perform local validation. Thanks for the suggestion.


Add tags for V&V status of each model

closes #479

Job VTB Documentation, step Sync Docs on f9bbded wanted to post the following:
View the site here
This comment will be updated on new commits.

Test failure is unrelated, this PR only modifies the doco


Links to source code are not working.

All links at https://www.boost.org/users/download/ lead to page that says "forbidden".

same issue

When I wrote to jfrog about it they replied with:

Hi,
Thank you for contacting JFrog Support. With regards to query on unable to download files from boostorg, I verified the account and found that the OSS quota limit has reached the maximum and the account was blocked. I further checked internally and unblocked the account, now you should be able to download the files without any issues. Please let me know if you have any further queries, I will be glad to assist you.
Looking forward to your response.

I've managed to download sources here:
http://capsel.org/tmp/boost_1_72_0.7z
http://capsel.org/tmp/boost_1_71_0.tar.bz2

This appears to have broken the boost Conan packages as well: https://stackoverflow.com/questions/60918097/why-do-my-conan-packages-appear-to-be-getting-corrupted

Dup of #384. The issue is due to overuse of the monthly bandwidth allowance, wait until 1st of April.

I am observing this problem again today.  Waiting until the 1st of July is not a good fix IMO.
Is there something users of Boost can do to help fix this?  Donate or a paid version for instance?


Create jekyll-gh-pages.yml

Can you mention your issue here!

Deploying website
On Sat, 1 Jun 2024 at 1:48 AM, Sukhbir Singh @.***>
wrote:

Mention your issue here!
—
Reply to this email directly, view it on GitHub
https://github.com/khushi-joshi-05/Food-ordering-website/pull/432#issuecomment-2142924147,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQW4QR7CSGZ33A7RLKJEDKTZFDLKPAVCNFSM6AAAAABIDFBQJSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBSHEZDIMJUG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

You can't create direct PR this will not count as a GSSoC contribution.

Sorry for inconvenience, I’ll raise another issues and fix them
On Sat, 1 Jun 2024 at 3:14 PM, Sukhbir Singh @.***>
wrote:

You can't create direct PR this will not count as a GSSoC contribution.
—
Reply to this email directly, view it on GitHub
https://github.com/khushi-joshi-05/Food-ordering-website/pull/432#issuecomment-2143383126,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQW4QR6PX22XOUQTP5VUAD3ZFGJYPAVCNFSM6AAAAABIDFBQJSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGM4DGMJSGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add HOWTO: BatchNorm in a multi-device setup

Every device updates its normalizing parameters (ra_mean and ra_var in the code) based on its own batch statistics (unless we specify axis_name which makes it global). However, these parameters are only stored in batch_stats and not params so they are never synced across devices unless specifically done so.
If they aren't synced, they can theoretically diverge, but if your data is fairly uniform across shards they're likely to trend towards similar values.
Syncing before eval is definitely a good idea though, since otherwise your eval results will depend on which devices process which examples.
Note: this doesn’t affect training at all, because training doesn’t use the running batch statistics (mean and var of the current batch are used during training).

Closing since I've added this content to the discussion post of #921

Closing since I've added this content to the discussion post of #921


Issues creating GIF

Alright so I just got this setup.  I am taking a video and feeding it using:
Regift *test = [[Regift alloc]init];
NSURL *newGif = [test createGIFFromURL:self.gifURL withFrameCount:16 delayTime:0.2 loopCount:0];
The frame count and delayTime are hardcoded just for testing.
  What am I doing wrong?  Thanks so much for your time and this great library!

Not sure if this is related to your issue, but all of the methods on Regift are class funcs, so you probably shouldn't need to alloc/init anything. (Not a huge fan of having everything being a class func though, so I'd be happy to accept a PR fix if you feel like making it :))
Let me know how that goes!

I have the same issue, can you tell me how do you have solved this problem?

Closing this. Feel free to reopen if the issue persists in master (for non-git submodule installation).
Thanks for your help 👍


only attempt to print existing attack graphs

Describe your changes
Screenshots - If Any (Optional)
This PR fixes:

Resolved #1032

Checklist before requesting a review

[ ] My code follows the style guidelines of this project
[ ] I have commented on my code, particularly in hard-to-understand areas
[ ] I have performed a self-review of my code
[ ] If it is a core feature, I have added thorough tests.
[ ] New and existing unit tests pass locally with my changes

Please open the PR against the dev branch (Unless the PR contains only documentation changes)

Are you sure this is the root cause of #1032 ?

The panic definitely comes from this, and it's a good practice to check for map element presence, but I haven't gone deeper to see why in verbose we try to get non existing graphs.

Are you sure this is the root cause of #1032 ?

The panic definitely comes from this, and it's a good practice to check for map element presence, but I haven't gone deeper to see why in verbose we try to get non existing graphs.

Nevermind, we only print attack graphs in verbose. I think the fix is valid, but I still would like to hear from @amirmalka


Use DRIVER CLASS NAME in JDBC sink for as its name represents

Unlike its description, DRIVER CLASS NAME in JDBC sink is in fact used for data source class name, not driver class name. Moreover, JDBC sink doesn't allow providing driver class name instead of data source class name.
Since some storage drivers don't support data source class (javax.sql.DataSource) we just would be better to change the usage of DRIVER CLASS NAME to driver class name, as the name is representing.
This change might be breaking change for someone since putting com.mysql.jdbc.jdbc2.optional.MysqlDataSource or org.postgresql.ds.PGSimpleDataSource will not work afterwards.

Merged.


HSEARCH-3419 + HSEARCH-4031 + HSEARCH-4036 + HSEARCH-4037 Move to Mockito + build improvements

HSEARCH-4037: Stop adding -Dcom.sun.xml.bind.v2.bytecode.ClassTailor.noOptimize=true to JVM system properties when testing
HSEARCH-4036: Upgrade to ForbiddenApis 3.1
HSEARCH-4031: Move from Easymock to Mockito
HSEARCH-3419: Remove --add-opens java.base/java.lang=ALL-UNNAMED from the JVM arguments used in our JDK11 integration tests

@fax4ever This is mostly refactoring, so unless you have something against it, I'll merge as soon as the CI build passes.

Full build: https://ci.hibernate.org/job/hibernate-search/job/PR-2388/2/

We have few code: +2,772 −3,318. That is nice


copr fedora-messaging

Just to mock webhooks and builds.

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

/packit copr-build

100th comment and we are good to close this 💯


Create Demonstration BlawxBot Interview Showing Object-Specific Relevance

It would be nice, for the sake of completeness to have a demonstration that works in BlawxBot and shows that the reasoner can distinguish between an attribute being relevant with regard to one object, and irrelevant with regard to another of the same type.

This is being closed as duplicate to #261 .


Update array.fs

copied from https://github.com/fsharp/fsharp/pull/533
There may be a missing length check on exists, as without hte length check the cost of a comparison and a call is incurred.
Adding the simple len > 0 check before the loop appears (in the basic cases I have tested) to near triple performance.
let f arr =
    Array.exists (fun e -> e = true) arr

for i in 0..60000000 do f ([||]:bool []) ;; // 1.8 seconds

let f2 arr =
    if Array.length arr > 0 then
        Array.exists (fun e -> e = true) arr
    else
        false

for i in 0..60000000 do f2 ([||]:bool []) ;; // 0.430 seconds

Note: this is something I have found an isolated instance of, it may be appropriate to ensure that such checks (if they are required, I can only speak for my use case) are consistant accross other modules, such as Seq\List.

Hi @xenocons, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

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

TTYL, MSBOT;

:+1:

Are you sure your changes give the same perf advantage? Looks like the first call to the inner loop function has the same check. So you are optimizing one function call, right?

@forki according to my quick test on my machine with the inner implementation:
let exists (f: 'T -> bool) (array:'T[]) =
    let len = array.Length
    let rec loop i = i < len && (f array.[i] || loop (i+1))
    loop 0

let exists2 (f: 'T -> bool) (array:'T[]) =
    let len = array.Length
    if len = 0 then false else
    let rec loop i = i < len && (f array.[i] || loop (i+1))
    loop 0

#time
for i in 0..60000000 do exists (fun e -> e = true) ([||]:bool []) ;;
// Real: 00:00:00.506, CPU: 00:00:00.514, GC gen0: 343, gen1: 0, gen2: 0

for i in 0..60000000 do exists2 (fun e -> e = true) ([||]:bool []) ;;
Real: 00:00:00.418, CPU: 00:00:00.421, GC gen0: 344, gen1: 0, gen2: 0

about 100ms for 60 billion calls

So it's indeed faster, but only a little bit. I wonder if a loop would be
better...
On Jan 7, 2016 12:23, "Max Malook"<EMAIL_ADDRESS>wrote:

@forki https://github.com/forki according to my quick test on my
machine with the inner implementation:
let exists (f: 'T -> bool) (array:'T[]) =
let len = array.Length
let rec loop i = i < len && (f array.[i] || loop (i+1))
loop 0
let exists2 (f: 'T -> bool) (array:'T[]) =
let len = array.Length
if len = 0 then false else
let rec loop i = i < len && (f array.[i] || loop (i+1))
loop 0
#time
for i in 0..60000000 do exists (fun e -> e = true) ([||]:bool []) ;;
// Real: 00:00:00.506, CPU: 00:00:00.514, GC gen0: 343, gen1: 0, gen2: 0
for i in 0..60000000 do exists2 (fun e -> e = true) ([||]:bool []) ;;
Real: 00:00:00.418, CPU: 00:00:00.421, GC gen0: 344, gen1: 0, gen2: 0
about 100ms for 60 billion calls
—
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/visualfsharp/pull/854#issuecomment-169634655
.

@forki I'm not good at loops, it's so imperative :)

yeah, but computers are really good in such things.

@mexx your same code on my PC, win10,16GB RAM,i7-4600U @ 2.10GHz, 64bit, 32bit FSI with --optimize:
Real: 00:00:00.927, CPU: 00:00:00.906, GC gen0: 686, gen1: 2, gen2: 1
val it : unit = ()
>
Real: 00:00:00.507, CPU: 00:00:00.500, GC gen0: 687, gen1: 0, gen2: 0
val it : unit = ()
>

Mhm. What about 1 element arrays? IIRC we have lots of functions that do:
match a.Length with
| 0 ->...
| 1 ->...
| n ->...

On Jan 8, 2016 00:46, "David Klein"<EMAIL_ADDRESS>wrote:

@mexx https://github.com/mexx your same code on my PC, win10,16GB
RAM,i7-4600U @ 2.10GHz, 64bit, 32bit FSI with --optimize:
Real: 00:00:00.927, CPU: 00:00:00.906, GC gen0: 686, gen1: 2, gen2: 1
val it : unit = ()

Real: 00:00:00.507, CPU: 00:00:00.500, GC gen0: 687, gen1: 0, gen2: 0
val it : unit = ()

—
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/visualfsharp/pull/854#issuecomment-169842638
.

FWIW, debug builds of originals, exists orig takes 20 instructions if empty array, exists2 takes 6 IL instructions if empty, this becomes clearer when you look at the assembler, exists2 exists the function reasonably efficiently, I guess the question is, how often is the input to exists an empty array? In my case it happens a lot, but it turns out to be more efficient to do the Length = 0 check at the site of the my function that calls exists, rather than allow it to go into FSharp.Core first (as each function call incurs a cost), but from my personal expectations, I would assume something like exists to return immediately on empty, but once again this is my own personal tilt:
let exists (f: 'T -> bool) (array:'T[]) =
    let len = array.Length
    let rec loop i = i < len && (f array.[i] || loop (i+1))
    loop 0

IL:
	IL_0000: nop
	IL_0001: ldarg.1
	IL_0002: ldlen
	IL_0003: conv.i4
	IL_0004: stloc.0
	IL_0005: ldarg.0
	IL_0006: stloc.1
	IL_0007: ldarg.1
	IL_0008: stloc.2
	IL_0009: ldloc.0
	IL_000a: stloc.3
	IL_000b: ldarg.0
	IL_000c: ldarg.1
	IL_000d: ldloc.0
	IL_000e: ldc.i4.0
	IL_000f: call bool Program::loop@5<!!T>(class [FSharp.Core]Microsoft.FSharp.Core.FSharpFunc`2<!!0, bool>, !!0[], int32, int32)
	IL_0014: ret

ASM (trimmed to remove getting .Length):
    let rec loop i = i < len && (f array.[i] || loop (i+1))
    loop 0
04B005BB  mov         ecx,dword ptr [ebp-14h]
04B005BE  xor         edx,edx
04B005C0  mov         eax,dword ptr [ecx]
04B005C2  mov         eax,dword ptr [eax+28h]
04B005C5  call        dword ptr [eax+10h]
04B005C8  mov         dword ptr [ebp-8],eax
04B005CB  movzx       eax,byte ptr [ebp-8]
04B005CF  mov         esp,ebp
04B005D1  pop         ebp
04B005D2  ret

let exists2 (f: 'T -> bool) (array:'T[]) =
    let len = array.Length
    if len = 0 then false else
    let rec loop i = i < len && (f array.[i] || loop (i+1))
    loop 0

IL:
	IL_0000: nop
	IL_0001: ldarg.1
	IL_0002: ldlen
	IL_0003: conv.i4
	IL_0004: stloc.0
	IL_0005: ldloc.0
	IL_0006: brtrue.s IL_000a

	IL_0008: ldc.i4.0
	IL_0009: ret

	IL_000a: ldarg.0
	IL_000b: stloc.1
	IL_000c: ldarg.1
	IL_000d: stloc.2
	IL_000e: ldloc.0
	IL_000f: stloc.3
	IL_0010: ldarg.0
	IL_0011: ldarg.1
	IL_0012: ldloc.0
	IL_0013: ldc.i4.0
	IL_0014: call bool Program::'loop@11-1'<!!T>(class [FSharp.Core]Microsoft.FSharp.Core.FSharpFunc`2<!!0, bool>, !!0[], int32, int32)
	IL_0019: ret

ASM(trimmed to remove getting .Length):
    if len = 0 then false else
02320766  cmp         dword ptr [ebp-4],0
0232076A  jne         0232076F
0232076C  nop
0232076D  jmp         02320772
0232076F  nop
02320770  jmp         02320778
02320772  xor         eax,eax
02320774  mov         esp,ebp
02320776  pop         ebp
02320777  ret
02320778  mov         ecx,0C169A0h
0232077D  call        008B30F4
02320782  mov         dword ptr [ebp-18h],eax
02320785  push        dword ptr [ebp-10h]
02320788  push        dword ptr [ebp-4]
0232078B  mov         ecx,dword ptr [ebp-18h]
0232078E  mov         edx,dword ptr [ebp-0Ch]
02320791  call        dword ptr ds:[0C169C4h]
02320797  mov         eax,dword ptr [ebp-18h]
0232079A  mov         dword ptr [ebp-14h],eax
    let rec loop i = i < len && (f array.[i] || loop (i+1))
    loop 0
0232079D  mov         ecx,dword ptr [ebp-14h]
023207A0  xor         edx,edx
023207A2  mov         eax,dword ptr [ecx]
023207A4  mov         eax,dword ptr [eax+28h]
023207A7  call        dword ptr [eax+10h]
023207AA  mov         dword ptr [ebp-8],eax
023207AD  movzx       eax,byte ptr [ebp-8]
023207B1  mov         esp,ebp
023207B3  pop         ebp
023207B4  ret

@xenocons Do we have any measurements of memory usage, allocations and wall clock improvements associated with this change?
Thanks
Kevin

@KevinRansom Is there a suggested methodology for producing such information (apart from counting the number of x86 instructions and the cycle count)?

@KevinRansom - The perf results are here: https://github.com/Microsoft/visualfsharp/pull/854#issuecomment-169842638.  I think those results are sufficient.
I'm happy with this change, it is reasonable to check once for emptiness early to avoid the call.  So we can pull this.


UI enhancements

Consider when a task has a long description
100s of task in the system
Sort tasks in meaningful order
Optimize display format for space (e.g. don't show year)
Overdue tasks stand out
More feedback to the user - Jim should see the details of the task added so that he can verify if it is indeed the task that he meant to add. In addition to giving feedback as a text message, also consider adding other visual clues. E.g. Highlighting the item added/edited in the list.

to be refactored


update-properties ignores includes parameter

./mvnw versions:update-properties -Dincludes=a.b.*:* completely ignores the artifacts listed in includes

Actually it also appears to ignore excludes


Out of memory error when starting snark worker

I am running on a 16 GB Memory / 100 GB Disk / NYC3 - Ubuntu 18.04 (LTS) x64 on digitalocean
After I start the snark worker using
coda client set-snark-work-fee 0.025
coda client set-snark-worker -address $MINA_PUBLIC_KEY

I check the status
coda client status

and see
SNARK worker:                                  B62qnBUcZxj2Nq76nCGv1aDrK5htLpJBAYURfEdqcHiDyxrdoZubTLT
SNARK work fee:                                25000000
Sync status:                                   Catchup

Shortly after in the logs I get these errors
Jan 11 05:30:05 mina-testnet coda[22393]: 2021-01-11 05:30:05 UTC [Warn] RPC call error for "get_transition_chain"
Jan 11 05:30:18 mina-testnet coda[22393]: 2021-01-11 05:30:18 UTC [Warn] failed to reset stream (this means it was probably closed successfully): $error
Jan 11 05:30:18 mina-testnet coda[22393]:         error: {
Jan 11 05:30:18 mina-testnet coda[22393]:   "string":
Jan 11 05:30:18 mina-testnet coda[22393]:     "RPC #1985 failed: \"internal RPC error error: unknown stream_idx\""
Jan 11 05:30:18 mina-testnet coda[22393]: }
Jan 11 05:30:20 mina-testnet coda[22393]: 2021-01-11 05:30:20 UTC [Error] error sending message on stream 1080: $error
Jan 11 05:30:20 mina-testnet coda[22393]:         error: {
Jan 11 05:30:20 mina-testnet coda[22393]:   "string":
Jan 11 05:30:20 mina-testnet coda[22393]:     "RPC #2018 failed: \"only wrote 0 out of 36 bytes error: libp2p error error: closed stream\""
Jan 11 05:30:20 mina-testnet coda[22393]: }
Jan 11 05:30:20 mina-testnet coda[22393]: 2021-01-11 05:30:20 UTC [Warn] RPC call error for "get_transition_chain"
Jan 11 05:30:23 mina-testnet coda[22393]: 2021-01-11 05:30:23 UTC [Error] error sending message on stream 1080: $error
Jan 11 05:30:23 mina-testnet coda[22393]:         error: {
Jan 11 05:30:23 mina-testnet coda[22393]:   "string":
Jan 11 05:30:23 mina-testnet coda[22393]:     "RPC #2023 failed: \"only wrote 0 out of 9 bytes error: libp2p error error: closed stream\""
Jan 11 05:30:23 mina-testnet coda[22393]: }
Jan 11 05:30:40 mina-testnet coda[22393]: 2021-01-11 05:30:40 UTC [Warn] RPC call error for "get_transition_chain"
Jan 11 05:30:40 mina-testnet coda[22393]: 2021-01-11 05:30:40 UTC [Warn] RPC call error for "get_transition_chain"
Jan 11 05:30:42 mina-testnet coda[22393]: 2021-01-11 05:30:42 UTC [Fatal] Snark worker died with signal sigkill. Aborting daemon
Jan 11 05:30:42 mina-testnet coda[22393]: 2021-01-11 05:30:42.347534Z Error ("Exception raised to [Monitor.try_with] that already returned.""This error was captured by a default handler in [Async.Log]."(exn(monitor.ml.Error("Mina_lib.Snark_worker_signal_interrupt(-7)")("Raised at file \"src/lib/mina_lib/mina_lib.ml\", line 188, characters 12-56""Called from file \"src/deferred0.ml\", line 56, characters 64-69""Called from file \"src/job_queue.ml\" (inlined), line 131, characters 2-5""Called from file \"src/job_queue.ml\", line 171, characters 6-47""Caught by monitor Cohttp_async.Server.collect_errors"))))
Jan 11 05:31:13 mina-testnet coda[22393]: 2021-01-11 05:31:13 UTC [Warn] RPC call error for "get_transition_chain"
Jan 11 05:31:18 mina-testnet coda[22393]: 2021-01-11 05:31:18 UTC [Warn] RPC call error for "get_transition_chain"
Jan 11 05:31:18 mina-testnet coda[22393]: 2021-01-11 05:31:18 UTC [Error] error sending message on stream 1228: $error
Jan 11 05:31:18 mina-testnet coda[22393]:         error: {
Jan 11 05:31:18 mina-testnet coda[22393]:   "string":
Jan 11 05:31:18 mina-testnet coda[22393]:     "RPC #2229 failed: \"only wrote 0 out of 698 bytes error: libp2p error error: closed stream\""
Jan 11 05:31:18 mina-testnet coda[22393]: }
Jan 11 05:31:18 mina-testnet coda[22393]: 2021-01-11 05:31:18 UTC [Warn] RPC call error for "get_transition_chain"
Jan 11 05:31:21 mina-testnet coda[22393]: 2021-01-11 05:31:21 UTC [Warn] RPC call error for "get_transition_chain"
Jan 11 05:31:24 mina-testnet coda[22393]: 2021-01-11 05:31:24 UTC [Info] verifier successfully got sigkill
Jan 11 05:31:24 mina-testnet coda[22393]: 2021-01-11 05:31:24 UTC [Info] prover successfully stopped
Jan 11 05:31:25 mina-testnet coda[22393]: 2021-01-11 05:31:25 UTC [Info] Daemon started process of kind "Verifier" with pid 23039
Jan 11 05:31:48 mina-testnet coda[22393]: 2021-01-11 05:31:48 UTC [Warn] RPC call error for "get_transition_chain"
Jan 11 05:31:48 mina-testnet coda[22393]: 2021-01-11 05:31:48 UTC [Warn] RPC call error for "get_transition_chain"
Jan 11 05:31:48 mina-testnet coda[22393]: 2021-01-11 05:31:48 UTC [Info] Received a block from $sender
Jan 11 05:31:48 mina-testnet coda[22393]:         sender: {
Jan 11 05:31:48 mina-testnet coda[22393]:   "Remote": {
Jan 11 05:31:48 mina-testnet coda[22393]:     "host": "<IP_ADDRESS>",
Jan 11 05:31:48 mina-testnet coda[22393]:     "peer_id": "12D3KooWRuqMr5DNnanCRvnKvYTAB1NVyKwFkhdP3oJnkP4YkrTM",
Jan 11 05:31:48 mina-testnet coda[22393]:     "libp2p_port": 8302
Jan 11 05:31:48 mina-testnet coda[22393]:   }
Jan 11 05:31:48 mina-testnet coda[22393]: }
Jan 11 05:31:48 mina-testnet coda[22393]: 2021-01-11 05:31:48 UTC [Fatal] Unhandled top-level exception: $exn
Jan 11 05:31:48 mina-testnet coda[22393]: Generating crash report
Jan 11 05:31:48 mina-testnet coda[22393]:         exn: {
Jan 11 05:31:48 mina-testnet coda[22393]:   "sexp": [ "Out of memory" ],
Jan 11 05:31:48 mina-testnet coda[22393]:   "backtrace": [
Jan 11 05:31:48 mina-testnet coda[22393]:     "Raised by primitive operation at file \"src/bytes0.ml\", line 45, characters 22-39",
Jan 11 05:31:48 mina-testnet coda[22393]:     "Called from file \"src/make_substring.ml\", line 233, characters 14-28",
Jan 11 05:31:48 mina-testnet coda[22393]:     "Called from file \"src/make_substring.ml\" (inlined), line 245, characters 42-85",
Jan 11 05:31:48 mina-testnet coda[22393]:     "Called from file \"src/reader0.ml\", line 678, characters 12-67",
Jan 11 05:31:48 mina-testnet coda[22393]:     "Called from file \"src/reader0.ml\", line 700, characters 23-42",
Jan 11 05:31:48 mina-testnet coda[22393]:     "Called from file \"src/deferred0.ml\", line 49, characters 2-10",
Jan 11 05:31:48 mina-testnet coda[22393]:     "Called from file \"src/reader0.ml\", line 945, characters 19-29",
Jan 11 05:31:48 mina-testnet coda[22393]:     "Called from file \"src/deferred1.ml\", line 174, characters 6-13",
Jan 11 05:31:48 mina-testnet coda[22393]:     "Called from file \"src/job_queue.ml\" (inlined), line 131, characters 2-5",
Jan 11 05:31:48 mina-testnet coda[22393]:     "Called from file \"src/job_queue.ml\", line 171, characters 6-47",
Jan 11 05:31:48 mina-testnet coda[22393]:     "Called from file \"bytes.ml\", line 57, characters 10-20",
Jan 11 05:31:48 mina-testnet coda[22393]:     "Called from file \"bytes.ml\" (inlined), line 62, characters 18-43",
Jan 11 05:31:48 mina-testnet coda[22393]:     "Called from file \"lexing.ml\", line 168, characters 17-34",
Jan 11 05:31:48 mina-testnet coda[22393]:     "Called from file \"lib/read.mll\", line 1120, characters 19-39",
Jan 11 05:31:48 mina-testnet coda[22393]:     "Called from file \"src/option.ml\", line 174, characters 11-17"
Jan 11 05:31:48 mina-testnet coda[22393]:   ]
Jan 11 05:31:48 mina-testnet coda[22393]: }
Jan 11 05:31:53 mina-testnet coda[22393]: 2021-01-11 05:31:53 UTC [Info] Daemon child process 22437 terminated after receiving signal "sigkill"
Jan 11 05:31:53 mina-testnet coda[22393]: 2021-01-11 05:31:53 UTC [Error] Child process of kind "Prover" with pid 22437 has terminated
Jan 11 05:31:53 mina-testnet systemd[1480]: mina.service: Main process exited, code=exited, status=99/n/a
Jan 11 05:31:53 mina-testnet systemd[1480]: mina.service: Failed with result 'exit-code'.
Jan 11 05:32:09 mina-testnet systemd[1480]: mina.service: Service hold-off time over, scheduling restart.
Jan 11 05:32:09 mina-testnet systemd[1480]: mina.service: Scheduled restart job, restart counter is at 105.
Jan 11 05:32:09 mina-testnet systemd[1480]: Stopped Mina Daemon Service.
Jan 11 05:32:09 mina-testnet systemd[1480]: Started Mina Daemon Service.
Jan 11 05:32:09 mina-testnet coda[23057]: 2021-01-11 05:32:09 UTC [Info] Coda daemon is booting up; built with commit "[DIRTY]b7eff8e5c966b12bdc298160297c716eae7a35af" on branch "HEAD"
Jan 11 05:32:09 mina-testnet coda[23057]: 2021-01-11 05:32:09 UTC [Info] Removing lockfile for terminated process
Jan 11 05:32:09 mina-testnet coda[23057]: 2021-01-11 05:32:09 UTC [Info] Daemon will expire at "2024-12-10 14:00:00-07:00"
Jan 11 05:32:09 mina-testnet coda[23057]: 2021-01-11 05:32:09 UTC [Info] Booting may take several seconds, please wait
Jan 11 05:32:09 mina-testnet coda[23057]: 2021-01-11 05:32:09 UTC [Info] Reading configuration files $config_files
Jan 11 05:32:09 mina-testnet coda[23057]:         config_files: [ "/var/lib/coda/config_b7eff8e5.json", "/root/.coda-config/daemon.json" ]
Jan 11 05:32:09 mina-testnet coda[23057]: 2021-01-11 05:32:09 UTC [Warn] Could not read configuration from "/root/.coda-config/daemon.json": $error
Jan 11 05:32:09 mina-testnet coda[23057]:         error: {
Jan 11 05:32:09 mina-testnet coda[23057]:   "exn_name": "Async_kernel__Monitor.Error_",
Jan 11 05:32:09 mina-testnet coda[23057]:   "exn":
Jan 11 05:32:09 mina-testnet coda[23057]:     "(monitor.ml.Error\n  (Unix.Unix_error \"No such file or directory\" open\n    \"((filename /root/.coda-config/daemon.json) (mode (O_RDONLY O_CLOEXEC)) (perm 0o0))\")\n  (\"Raised at file \\\"src/result.ml\\\", line 158, characters 17-26\"\n    \"Called from file \\\"src/deferred1.ml\\\", line 17, characters 40-45\"\n    \"Called from file \\\"src/job_queue.ml\\\" (inlined), line 131, characters 2-5\"\n    \"Called from file \\\"src/job_queue.ml\\\", line 171, characters 6-47\"\n    \"Caught by monitor try_with_or_error\"))"
Jan 11 05:32:09 mina-testnet coda[23057]: }
Jan 11 05:32:09 mina-testnet coda[23057]: 2021-01-11 05:32:09 UTC [Info] Initializing with runtime configuration. Ledger name: "testworld"
Jan 11 05:32:09 mina-testnet coda[23057]: 2021-01-11 05:32:09 UTC [Info] Using the compiled constraint constants
Jan 11 05:32:09 mina-testnet coda[23057]: 2021-01-11 05:32:09 UTC [Info] Found genesis proof file at $path
Jan 11 05:32:09 mina-testnet coda[23057]:         path: "/root/.coda-config/genesis/genesis_proof_2eba002d6bc68b79ef1986f747c235f74393f60d4a77cd1383f125cfb5f5bf6d"
Jan 11 05:32:09 mina-testnet coda[23057]: 2021-01-11 05:32:09 UTC [Info] Loaded ledger from $ledger_file and genesis proof from $proof_file
Jan 11 05:32:09 mina-testnet coda[23057]:         ledger_file: "/root/.coda-config/genesis/genesis_ledger_accounts_a0a52362cff41dde1644de9ce44aabd673c5e1e6eed9caaff7ea906b5fe3e988_74bedf9d2d01e70156df8f82da3ba62449a392d67b3de4de3523aca975e0d467.tar.gz"
Jan 11 05:32:09 mina-testnet coda[23057]:         proof_file: "/root/.coda-config/genesis/genesis_proof_2eba002d6bc68b79ef1986f747c235f74393f60d4a77cd1383f125cfb5f5bf6d"
Jan 11 05:32:10 mina-testnet coda[23057]: 2021-01-11 05:32:10 UTC [Info] Setting current protocol version to "1.1.0" from compile-time config

Then the coda daemon becomes unresponsive
root@mina-testnet:~# coda client status
Error: Unable to connect to Coda daemon.
- The daemon might not be running. See logs (in `~/.coda-config/coda.log`) for details under the host:<IP_ADDRESS>.
  Run `coda daemon -help` to see how to start daemon.
- If you just started the daemon, wait a minute for the RPC server to start.
- Alternatively, the daemon may not be running the RPC server on (<IP_ADDRESS> 8301).
  If so, add flag `-daemon-port` with correct port when running this command.

After it comes back then it changes back to
SNARK worker:                                  None
SNARK work fee:                                25000000
Sync status:                                   Catchup

Closing this issue as it is stale for a long time.


Can't refresh dashboard with css in Win10

Hello,
I hope some of you could help me to understand why, when I'm loading a dashboard for the first time page for the first, it will be load correctly, but when page is refreshed, I get an error and page don't load css.
This issue occours on Windows10 and with last version of superset.
I installed Virtual Box Machine with Ubuntu, to replicate this issue and it works just fine.
How to reproduce the bug

Get last version of superset
Run in Windows 10
Create Dashboard for test (background blue on css)
Open Dashboard and Refresh Page

Expected results
When dashboard page is refreshed, css load is required
Actual results
After refresh page the css won't be loaded.
Screenshots
Loading the first time

After refresh:

Error

Environment
(please complete the following information):

Windows 10
superset version: latest
Docker Desktop
Chrome - localhost:8080

if any one can help me, I would appreciate it
Best regards

@andrey-zayats took a look at this and was unable to reproduce the issue. I'm going to go ahead and remove the validation:required label and close the issue. If anyone sees this happening in Superset 2.0.x or later, or on master, we're open to revisiting this conversation, re-opening the ticket, or triaging a new Issue with updated context. Thank you!


Send error for seif version mismatch and add message id to blob details

Currently, we disconnect on every error. We should be sending an error for seif version mismatch during the handshake process.

Added functionality with #7