Fixes a few bugs associated with admin-bots section

The PR resolves the frontend issue associated with #3413 :

On deactivating test bot, the edit-bot button gets removed.
Bot name and email remains striked out if deactivated.
After reactivating a deactivated bot, the edit-bot button works fine.

Todo :

Reactivating a bot should relist it in "Your Bots" section in settings page without reload.
I am not sure how to fix this. It would be great if anyone could help.

Automated message from Dropbox CLA bot
@jajodiaraghav, it looks like you've already signed the Dropbox CLA. Thanks!

@jajodiaraghav Zulip uses a real-time sync approach of calling an zerver/lib/actions.py function for updating the database related things and sending an event to update open browser windows. Currently the function used for reactivating the bot(do_reactivate_user) sends an event only for users. It should be updated to send an event for bots as well.

Thanks a lot @HarshitOnGitHub 😄

@HarshitOnGitHub Done 👍

@jajodiaraghav Thanks! Some frontend tests are failing probably due to some class related changes. Please have a look and fix them.

The issue is not with the class here. As we click reactivate under bots section, it sends a POST request to<EMAIL_ADDRESS>It turns out that the bot gets reactivated but it returns {"msg":"Internal server error","result":"error"} with a status 500. How can we fix this?

Better debug the 500; there should be a traceback in the server.log.

@jajodiaraghav The error is because we are accessing the bot owner's email address even when it is not set, in notify_created_bot. We need to conditionally check if the bot has an owner. If the bot_owner is None then don't include the owner in the event dict.

Made some changes. Please review. :)

@jajodiaraghav it looks like we already merged a version of the backend part of your branch in f0121973d2ba28b78b542c106f4f92047c76c47c; can you rebase?

There's just 4 lines of frontend changes, and I'm having a hard time figuring out how to reproduce the issue they fix.  @jajodiaraghav maybe after rebasing, you can squash your changes and make the commit message clear about the precise bug this fixes?
With your changes, I can still reproduce deactivating a bot in the /#settings/your-bots page doesn't live-update the /#administration version to show a reactivate button, which is what I thought this was supposed to fix...

@timabbott Updated the commits!

With your changes, I can still reproduce deactivating a bot in the /#settings/your-bots page doesn't live-update the /#administration version to show a reactivate button, which is what I thought this was supposed to fix.

Although this wasn't mentioned as an issue in #3413, I am trying to fix this although having a hard time figuring out how to. Could you help?

@timabbott Updated commits :)

@timabbott Is the travis error due to master branch?

@jajodiaraghav The error is not due to the master branch but rather due to the fact that you have not capitalized the strings correctly. You will need to change "Edit User" to "Edit user". See http://zulip.readthedocs.io/en/latest/translating.html#capitalization for more information.

@timabbott Do you think we should change that message to something like "Strings not properly capitalized in frontend" instead of "Strings not capitalized in frontend"?

Thanks for the help @HarshitOnGitHub
I too figured out the error, but it would be nice to update the statement to something more explicit :)

Yeah let's improve the error -- that check is new in the last day and I think we could make it a lot clearer.

(Just submit a PR!)

@timabbott Done #4017.

@jajodiaraghav for the second commit, I think you need the update to go in server_events.js.  Otherwise, this won't work if there are multiple browser windows open and you do the editing in one of them -- the second won't update.
Also it looks like the first commit has a stray whitespace change; can you fix that?

Done! Although i couldn't get the travis error right. :'(

@jajodiaraghav Okay I did some manual debugging and the error is due to the fact that you are having a call to admin.setup_page() in the remove bot event but that function is not stubbed in node tests and hence the error. To fix it you will need to stub the admin.setup_page() function. However I strongly feel that you should not be calling the admin.setup_page() for updating the users table since that is pretty expensive operation just for changing the state of a button. I think you should be modifying the admin's update_user_data() function for making such kind of changes. You will probably want to get the relevant bot using the user_id and set the state of the button using the bot's is_active flag. Let me know if you get any problem in doing so.
Regarding the irrelevant traceback, I have also been annoyed by it a lot of times but always forgot to open an issue. But this time when I saw it again, the first thing I did was to open an issue for this. It will probably get fixed today. :)

Thanks @HarshitOnGitHub

@HarshitOnGitHub The changes were probably made after i updated my commits.
Anyway, I have updated the code. Thanks for reviewing this. :)
@timabbott

@timabbott Done. Although I'm not sure what's causing the travis error.

Travis problem looks to be some issue with their infrastructure; will probably work if you rebase to have it retry.

Looks good, merged, thanks @jajodiaraghav!


coinmix start stop gui button dont work

as it is now u need to add the line
enablemixTX=1
into diamond.conf to start coin mixing at wallet startup
u can not start coin mixing with the button that should be able in gui
u can stop it with that button but then u can start again
only with restart wallet and have the conf line active

please explain a little bit more, unable to understand, or fetch information

use a gui wallet in testnet and try to start coinmixing from the wallet
it impossible u get the error message u need to have at least 14 DMD balance


[Pricing Summary] Fixing wrong early quit logic

Describe your specific features or fixes
Resolves: MWPW-136869
Test URLs:

Before: https://main--express--adobecom.hlx.live/express/learn/students?lighthouse=on
After: https://mwpw-136869--express--adobecom.hlx.live/express/learn/students?lighthouse=on

Codecov Report

Merging #290 (b9920da) into stage (47ed500) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##            stage     #290      +/-   ##
==========================================
+ Coverage   47.19%   47.30%   +0.10%
==========================================
  Files          31       31
  Lines        9359     9359
==========================================
+ Hits         4417     4427      +10
+ Misses       4942     4932      -10

Files Changed
Coverage Δ

express/blocks/pricing-summary/pricing-summary.js
81.81% <100.00%> (+0.86%)
:arrow_up:

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


issue/5066-fix-format-warnings

Description
https://github.com/inmanta/inmanta-core/pull/4979 removed InmantaWarnings which use another formatter than the default warnings. As we want to keep both formats, this PR adds the InmantaWarnings back.
closes https://github.com/inmanta/inmanta-core/issues/5066
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

Pull request rejected by merge tool. The tests for this branch did not succeed.

Processing this pull request

Failed to merge changes into iso5 due to merge conflict. Please open a pull request for these branches separately by cherry-picking the commit that was made on the branch master (git cherry-pick ec66c4c87b85dddeca9c190101f00afa6f3a2cb8).

Failed to merge changes into iso4 due to merge conflict. Please open a pull request for these branches separately by cherry-picking the commit that was made on the branch master (git cherry-pick ec66c4c87b85dddeca9c190101f00afa6f3a2cb8).

Merged into branches master in ec66c4c87b85dddeca9c190101f00afa6f3a2cb8

Not closing this pull request due to previously commented issues for some of the destination branches. Please open a separate pull request for those branches by cherry-picking the relevant commit. You can safely close this pull request and delete the source branch.

This branch was not deleted as it seems to still be in use.


Fetching data from database

Could someone explain how I fetch data from meteor?
In Details.js data is fetched but presented using the MeteorListView. I'd like to just print the fetched data in the console but I can't figure out how to do that. The handle doesn't contain any objects.
export default createContainer(() => {
  const handle = Meteor.subscribe('details-list');
  return {
    detailsReady: handle.ready(),
  };
}, DetailsContainer);

To publish the data in the console you could run Meteor.collection('details').find() which will return an array of objects once the subscription has completed.
Be sure to check out the react-native-meteor docs for more info!


[glib] update to 2.72.3

Library name: glib
New version number: 2.72.3
Other information that may be useful (release notes, etc...)
GLib 2.72 release makes it possible to integrate GLib into foreign event loops.

@desowin I see the latest version of glib is 2.73.2, I will update version to this.
https://gitlab.gnome.org/GNOME/glib/-/tree/2.73.2

@desowin I see the latest version of glib is 2.73.2, I will update version to this. https://gitlab.gnome.org/GNOME/glib/-/tree/2.73.2

When the second number is odd, it indicates unstable release.

Thanks for your reply, I get it. :)

From GLib SECURITY.md

Under GLib’s versioning scheme, stable release series have an even minor
component (for example, 2.66.0, 2.66.1, 2.68.3), and development release series
have an odd minor component (2.67.1, 2.69.0).


Implement manual update / timer countdown

It would be great to be able to manually update data from the systray context menu or see when the next update will be triggered, e.g. Update data (23:40)

[x] manual update (done in commit c08dff69d9b55b6cebde10259d447fb01017d9c6)
[ ] timer countdown

Fixed in commit 9f626ddf4cd1b8934d0f744152bc3e4dc5a7fa17


Removing the master and slave language

For diversity reasons, it would be nice to try to avoid 'master' and 'slave' terminology in this repository which can be associated to slavery. The master-slave terminology could be problematic for people in several countries which has the history of slavery like Romania, USA and many others. Thank you for considering the proposal. Let me know if any changes in the PR are needed, I would be happy to implement them.

Really appreciate your work, but "master - slave" is a commonly used term in database area, which is exactly the area our project aimed at, we cannot simply change this term to "main - subordinate" as it can cause misunderstandings, at least for now.
If the database itself changed its term completely, we would reconsider it.


mod_fastcgi not building on OSX 10.11

brew install mod_fastcgi is resulting in the following compile error.
Last 15 lines from /Users/mikehancock/Library/Logs/Homebrew/mod_fastcgi/01.apxs:
#define ap_log_error(...) ap_log_error__(__VA_ARGS__)
                          ^
/usr/include/apache2/http_log.h:366:28: note: expanded from macro 'ap_log_error__'
    do { const server_rec *sr = s; if (APLOG_MODULE_IS_LEVEL(sr, mi, level)) \
                           ^
fcgi_config.c:975:9: error: expected expression
/usr/include/apache2/http_log.h:363:27: note: expanded from macro 'ap_log_error'
#define ap_log_error(...) ap_log_error__(__VA_ARGS__)
                          ^
/usr/include/apache2/http_log.h:367:76: note: expanded from macro 'ap_log_error__'
             ap_log_error_(file, line, mi, level, status, sr, VA_ARGS__);    \
                                                                           ^
6 warnings and 8 errors generated.
apxs:Error: Command failed with rc=65536

Turns out --with-homebrew-httpd24 was required to get it to use the correct apxs during build.
Might it be worth adding more specific instructions for each installation path (2.2, 2.4)?

Got the same issue. --with-homebrew-httpd24 solves the build, but the Apache bundled with macOS Sierra will not serve my Sites (Safari says "No connection") but not throw any errors to console.
When I comment out the LoadModule it serves the pages again.
Anyone has a fix for this?

Seems like the same issue I run into: #139

This tap is now deprecated, sorry.


Add OpenTF support

Describe the solution you'd like
As recently announced, the OpenTF fork should drop soon. We should make sure that Terragrunt supports it out of the box.
Here's an idea for an initial implementation: anywhere Terragrunt calls out to the terraform binary, we should update it to check for the terraform binary first; if that's not found, then try the opentf binary; and if that's not found, show an error (as we do today). Whatever we pick, we should mention in the debug log.
Describe alternatives you've considered
In theory, you can already use OpenTF by using the terraform_binary  config or --terragrunt-tfpath CLI flag or TERRAGRUNT_TFPATH env var, but those all require extra steps, whereas I'd like OpenTF to "just work" with Terragrunt.
Additional context
OpenTF should work as a "drop in" replacement for Terraform, so in theory, no other changes should be necessary. However, we should run our tests to make sure once the opentf binary is available for testing. One place where OpenTF will differ is stdout and stderr output will say OpenTF instead of Terraform; if Terragrunt is parsing that log output anywhere, we'll need to update the code accordingly.
This is just the initial implementation in the sense that it should be backward compatible. In the future, we can reverse the lookup order, and perhaps even drop support for terraform, but those would be significant breaking changes (especially the latter), so we'd want to see how things go with OpenTF and what the community wants first.
Note on registry usage
HashiCorp recently did an update (one that was totally unannounced, BTW) to its Registry terms to say:

You may download providers, modules, policy libraries and/or other Services or Content from this website solely for use with, or in support of, HashiCorp Terraform.

Terragrunt uses go-getter to download modules specified in the source URL, and it's possible that URL points to the Terraform Registry. For now, Terragrunt only works with Terraform, so that is still inline with the terms. If we add support for OpenTF, then in theory, anyone using Terragrunt with OpenTF might be violating the registry terms.
Possible solutions:

Cheapest/fastest: If we detect a registry source URL and opentf as the binary, give the user a warning.
Better solution: If we detect a registry source URL and opentf as the binary, automatically use OpenTF's registry (coming soon) instead of HashiCorp's.

From a technical perspective, I expect that it will be a no-op for the time being as OpenTF anticipates maintaining compat/interop for the foreseeable future. (However, OpenTF has stated that some functionality may be added to OpenTF that is a superset of what's in base Terraform. [source])
From a process perspective:

If there is a bug in OpenTF that does not exist in Terraform (or vice-versa), will bug reports (as they pertain to this project) be treated equally on both platforms?

Will testing be performed with code targeting both platforms?

For OpenTF's potential future "superset" functionality, will that be treated as a first-class citizen?

Came across this comment last night.
https://www.linkedin.com/feed/update/urn:li:ugcPost:7102220371797389313?commentUrn=urn%3Ali%3Acomment%3A(ugcPost%3A7102220371797389313%2C7102263805740503040)&replyUrn=urn%3Ali%3Acomment%3A(ugcPost%3A7102220371797389313%2C7102264759957213184)&dashCommentUrn=urn%3Ali%3Afsd_comment%3A(7102263805740503040%2Curn%3Ali%3AugcPost%3A7102220371797389313)&dashReplyUrn=urn%3Ali%3Afsd_comment%3A(7102264759957213184%2Curn%3Ali%3AugcPost%3A7102220371797389313)

I just updated the issue description with:
Note on registry usage
HashiCorp recently did an update (one that was totally unannounced, BTW) to its Registry terms to say:

You may download providers, modules, policy libraries and/or other Services or Content from this website solely for use with, or in support of, HashiCorp Terraform.

Terragrunt uses go-getter to download modules specified in the source URL, and it's possible that URL points to the Terraform Registry. For now, Terragrunt only works with Terraform, so that is still inline with the terms. If we add support for OpenTF, then in theory, anyone using Terragrunt with OpenTF might be violating the registry terms.
Possible solutions:

Cheapest/fastest: If we detect a registry source URL and opentf as the binary, give the user a warning.
Better solution: If we detect a registry source URL and opentf as the binary, automatically use OpenTF's registry (coming soon) instead of HashiCorp's.

From a technical perspective, I expect that it will be a no-op for the time being as OpenTF anticipates maintaining compat/interop for the foreseeable future. (However, OpenTF has stated that some functionality may be added to OpenTF that is a superset of what's in base Terraform. [source])
From a process perspective:

If there is a bug in OpenTF that does not exist in Terraform (or vice-versa), will bug reports (as they pertain to this project) be treated equally on both platforms?

Yup. Until otherwise announced, we should support both platforms equally.

Will testing be performed with code targeting both platforms?

Yes, we should test both platforms.

For OpenTF's potential future "superset" functionality, will that be treated as a first-class citizen?

Probably, but hard to say until we know what that functionality is. I think we'll have to decide on a case by case basis, with input from the community.

Your response is greatly appreciated.
This whole situation has created uncertainty at a time when we could really do without it. My company leverages several open-source and SaaS offerings in the Terraform ecosystem, and I've been reaching out across the ecosystem to understand where everyone is on this topic, and identifying what we need to watch out for.
Hoping for the very best from this whole situation, but planning ahead for the absolute worst.

I've been using the unofficial dev snap OpenTF v1.6.0-dev, but even with --terragrunt-tfpath set to /snap/bin/opentf I get an error: Unable to parse Terraform version output: OpenTF v1.6.0-dev on linux_amd64. I'd say parsing that was bound to be brittle :-P

Released in https://github.com/gruntwork-io/terragrunt/releases/tag/v0.52.0


(react-hooks): exports property in package.json does not correctly specify types

"exports": {
    ".": "./dist/index.js",
    "./use-inbox": "./dist/inbox/use-inbox.js"
  },

above specification will result in ts complaining that Could not find a declaration file for module '@trycourier/react-hooks'. Declaring something like below solves the issue.
"exports": {
    ".": {
      "import": {
        "types": "./typings/index.d.ts",
        "default": "./dist/index.js"
      },
      "require": {
        "types": "./typings/index.d.ts",
        "default": "./dist/index.js"
      }
    },
    "./use-inbox": "./dist/inbox/use-inbox.js"
  },

node - 20.13.1
@trycourier/react-hooks - 5.1.1
typescript - 5.3.3
one more thing I think is relevant is tsconfig, esp. moduleResolution value
"module": "ESNext",
"target": "ESNext",
"moduleResolution": "Bundler",

ok thanks, ill try this out.  ill most likely need to update a bunch of our packages if this is required for a setup like yours

Thanks. And to reproduce it, I am quite sure now, that moduleResolution of bundler or node16/nodenext is necessary, cause only those actually use exports field in package.json

any updates on this? should we create a PR for this thing?

Hi!  Yep I merged the change but we need to do a release and it's bundled in a 6.0 breaking change
It should be released soon :)

released 6.0

Thanks so much


Make setup.py parseable on Python 3 with LANG=C (or unset).

Running pip install kappa in a barebones environment (e.g. AmazonLinux Docker container), or anything where LANG is not UTF-8 (e.g. en_US.UTF-8) fails without this patch. This is because setup.py runs open_file("README.rst"), which in turn opens the file in text mode. On Python 3, this uses the ASCII codec, but README.rst contains UTF-8 characters.
This patch forcibly parses README.rst (and requirements.txt) as UTF-8 regardless of the user's codec.

dacut, im having the same issue.
Do you know how to set LANG on  AmazonLinux Docker container?

Build a new container; in the Dockerfile for it, include this before installing kappa (or whatever depends on kappa):
FROM amazonlinux:latest
ENV LANG=en_US.UTF-8

Or you can prefix your pip install:
LANG=en_US.UTF-8 pip install kappa

Thanks :)


1.21

port to 1.21, fabric? Also can you upload it to modrinth?

It is on modrinth..? Where did you get it?
https://modrinth.com/mod/butter

Will update it soon ^_^

Any updates?

I'm really looking forward to the update


fix: nodes over window in short display

As the discussion in #1003, add a CSS rules to avoid the goal tree over the goal window in shorter display.

Tested it out and this looks great! I'll merge this now -- also have something in progress for the discussion about #987


Roll the Dice

Before jumping into Pr first comment for assign.
Problem statement : Roll the Dice
Probelm Link : https://edabit.com/challenge/6ejJ6e7fLdqzGPa3q
Choose one Cateorgy -  Expert
question category : Math, Numbers

Before submitting a PR please Check some details.

[ ] fileName formet should be in camelcase
example- returnTheNextNumber.py
[ ]  Choose label for Problem , example- "good first issue", "easy"
[ ]  In file there should be problem statement and link to problem

/* Problem-Task : This program will add two numbers
 * Problem Link : https://edabit.com/challenge/SFzHtm63XT6EYNHWY
*/
int main() {
    int bla;
    return 0;
}

This repository is vaild for HacktoberFest2020
Read before making PR - How to contribute on Github
Good Luck,

I would like to contribute to this... So pelase assign me...

[ ] Task assigning to @vivekkj123

Best of Luck

Sorry, i cant do that now due to some personal issues, so please assign anyone....

Could I get this one?


Fix dragging working not correct

Description

This PR fix the bug
What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[✔ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents
Fixes #2009
Mobile & Desktop Screenshots/Recordings

Added tests?

[ ] 👍 yes
[ ✔] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 examples
[ ✔] 🙅 no documentation needed

Thx a lot!)

Does this fix: https://github.com/react-grid-layout/react-grid-layout/issues/1877

Ah better explained in: #2009

Can this be approved / merged? It fixes a significant usability issue with RGL. Thank you!

@artem-malko @ItsNoHax @egucciar
I closed the current PR #2017 because I found that another PR #2059 which fixes a similar issue has been merged.
I will verify the issues related to PR #2017 based on the latest code from the main branch.
If the issues persist, I will submit a new fix PR based on the main branch and provide a new temporary patch for version 1.4.4 to issue #2009.
There’s no need to pay attention to the current PR anymore.


Set attenuation by inputing numbers

Operating system or device - Godot version:
Win 7 64 bit Godot 2.1.3
I would like an ability to set attenuation for lights by inputing numbers (in addition to the existing way of dragging the curve around).
Use case: I have two headlights on my car, and for obvious reasons I want their parameters to be identical when tweaking them. Using presets, when I'm constantly changing values, is an overkill.

Can you further describe this feature request? Are you saying that you want two values to be linked, so that changing one changes the other?

We have a number above the curve. So I'd like to be able to set the attenuation value by inputting the value instead of dragging the curve around.

Hello :) I would like to work on this enhancement

I'm working with a colleague on this, we would like to know if anyone knows where this issue is in the code, we've been looking for it since two weeks ago and we're having troubles finding it.
Thank you!

The proposal refers to this widget:

To find out where its code is, you can start by checking the SpotLight class:
scene/3d/light.cpp
463:    ADD_PROPERTYI(PropertyInfo(Variant::REAL, "spot_attenuation", PROPERTY_HINT_EXP_EASING, "attenuation"), "set_param", "get_param", PARAM_ATTENUATION);

There you see that the property hint is PROPERTY_HINT_EXP_EASING which uses EditorPropertyEasing from editor/editor_properties.cpp.


[Doc-Enhancement] Message action overflow menu is not showing custom app actions by default

Hi Team,
When the user clicks on the three dots, the overflow menu is displayed. However, LOB/3P related message actions are not displayed by default. Once the user clicks on 'More Actions', they can see the message actions and click on them. The respective message action will then be displayed in the overflow menu by default for next time. In summary, the overflow menu displays the three most recent message actions only, and there is no possibility to pin a message action to be displayed by default. Please add this information to the documentation.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: efcdf768-1513-2249-b344-bf38ef64c70d
Version Independent ID: 415b5d57-2d41-9540-55a7-2a33a4315e37
Content: Define message extension action commands - Teams
Content Source: msteams-platform/messaging-extensions/how-to/action-commands/define-action-command.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: anclear

Hi @surbhigupta12 - Could you please help here to add a note in this doc as mentioned above?


Address book for Consensus <-> ParaTime transactions

Please add possibility to add ParaTime addresses to Address book, so when we are transferring from oasis -> emerald and emerald -> oasis we have an option to choose from Address book (to avoid mistakes and re-entering every-time again and again).

Sort of related to #190


Add support for -i --info option

First of all, great tool! I've been wanting something like this for a long time but could never find the motivation to actually do it.
I think it could be helpful to see what the scripts actually do, so this adds that ability using a -i/--info flag.

@bclinkinbeard awesome, very good idea and PR :smile:
just a couple of comments and we can merge it :+1:

Cool, I will address all of these in the next day or so. Thanks!

Updated and squashed.

Awesome :confetti_ball:
:+1: many thanks!


Add support for pathlib Paths

Is your feature request related to a problem? Please describe.
Running
from pathlib import Path
import scmdata

root = pathlib.Path('foo')

scmdata.ScmRun(
    str(
        root
        / "data/rcmip-emissions-annual-means-v5-1-0.csv"
    ),
    lowercase_cols=True,
)

should work without having to convert to a string first.
Describe the solution you'd like
I think here it needs to accept a PathLike[str] as well: https://github.com/openscm/scmdata/blob/master/src/scmdata/run.py#L347

Probably need to open up something like this I think https://github.com/openscm/scmdata/blob/71ce9e1c1af6aa66a61a82b03a7e4ca80138e353/src/scmdata/run.py#L496

@lewisjared ping so this is on your radar


Add LLM improvements

Description

Move summary notes to BE to prevent bypassing prompt
Update request format

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[x] Code quality improvements

Checklist

[x] I have tested this code
[ ] I have updated the documentation

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

3 of 33 (9.09%)  changed or added relevant lines in 3 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.3%) to 31.208%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/features/sicp/chatCompletion/sicpNotes.ts
0
1
0.0%

src/features/sicp/chatCompletion/api.ts
0
11
0.0%

src/pages/sicp/subcomponents/chatbot/ChatBox.tsx
3
21
14.29%

Files with Coverage Reduction
New Missed Lines
%

src/pages/sicp/subcomponents/chatbot/ChatBox.tsx
2
5.97%

src/features/sicp/chatCompletion/chatCompletion.ts
3
0.0%

Totals

Change from base Build<PHONE_NUMBER>2:
-0.3%

Covered Lines:
4818

Relevant Lines:
14585

💛  - Coveralls

Corresponding BE PR: https://github.com/source-academy/backend/pull/1144


Update CryptoSwift to 1.1.3 and Reachability.swift to 4.3.1 to quiet build warnings

Description of the pull request
Update CryptoSwift to 1.1.3 and Reachability.swift to 4.3.1 to quiet build warnings.
Why is the change necessary?
Xcode 11.2.1 show several build warnings with these third party libraries.
See https://github.com/pusher/pusher-websocket-swift/issues/238
While Reachability.swift 4.3.1 is not the latest released version, it is the most recent one that builds without further changes to the codebase.
Let me know if I need to make any adjustments to the CHANGELOG or the Carthage and SwiftPM files, I tried to keep changes to a minimum.

Note, I've only tested this by updating my Podfile reference and it works fine. Please check out the Travis build errors for CryptoSwift.
https://travis-ci.org/pusher/pusher-websocket-swift/builds/609024149?utm_medium=notification&utm_source=github_status

This removes CryptoSwift to keep up to date with master

Hi @marcetcheverry. Thank you very much for the PR. We've updated the Reachability to a more recent version since you made this PR so I'm going to close it. Sorry that it wasn't merged sooner.


Make sure DNS discovery URLs are never present if using a genesis file

PR description
Fix for DNS discovery URLs being added outside of the few named networks.
Fixed Issue(s)

fixes #1634
Changelog

[x] I thought about the changelog and included a changelog update if required.

Unit test?

That's way to subtle and the fact I had to ask indicates it's testing as a side effect not as an intentional test.  We need an intentional test for this.
I'd like to see a test where the CLI (not via TOML) is explicitly testing for (a) no DNS with a --genesis-file file is set, (b) dns for when --network=mainnet is set and (c) dns for when neither --genesis-file nor --network is set.
If we don't deliberately and readably test for these conditions they will come back in the future and bite us.  Someone could very easily refactor overrideDefaultValuesIfKeyIsPresentInConfigFile and remove the side effect test since it's not clear from the test code that is a valuable assertion.


[3.0.0] Import API returns an empty archive

Description:
/export/api GET resource of admin API returns an empty zip file.
sample cURL
curl -X GET "https://localhost:9443/api/am/admin/v0.15/export/api?name=PizzaShackAPI&version=1.0.0&providerName=admin&format=JSON" -o "export.zip" -k -H "Authorization: Bearer $token"

Steps to reproduce:

Create an API in APIM 3.0.0
Export the api using the above cURL command
Try to extract and explore the returned archive content. This will be empty

Affected Product Version:
APIM <IP_ADDRESS>
Environment details (with versions):

OS:
Client:
Env (Docker/K8s):

Optional Fields
Related Issues:

Suggested Labels:

Suggested Assignees:

In latest APIM versions this Admin API's GET /export/api resource is moved to Publisher API's GET /apis/export resource. And this issue cannot be reproduced with the latest version of APIM (tested on carbon-apimgt 9.5.8-SNAPSHOT)


Error Response returns Gzip

Hi,
All my errors come in gzipped, is there a way to handle this without an external package?
My config is using apiVersion:3.

Hey @datdevboi, I'm not sure what issue you're having, but I was receiving the same sort of errors until I figured out that some calls like "store info" have to be v2 and some have to be v3.  I solved the issue and have some sample code here: https://github.com/getconversio/node-bigcommerce/issues/66#issuecomment-490114975
I hope this helps.

Hey @philspitler, I appreciate the response. The only time I am getting a gzip response is when I get an error back, otherwise any successful response is sent in json.

This is a bug. The problem is with these lines: https://github.com/getconversio/node-bigcommerce/blob/master/lib/request.js#L112
I've checked the response headers directly in that file, and they are gzip. With an error, the script simply creates a new error and sets the raw (gzipped) body to the error responseBody key, and returns the rejected error .
The script should prep the data for the response and handle gzipped accordingly, regardless if it's rejecting with an error, or returning the successful response.


tusd as a regular file upload server

I am trying to use tusd as file upload server for nativescript apps. There are some plugins available  but they have different issues and unreliable.
I also tried using the tus js client but due to lack of nodejs features support that does not work either.
Finally, I am trying to use https://github.com/NativeScript/nativescript-background-http which is a regular HTTP file upload client for nativescript. But, uploads fail with connect reset error on client.
JS:   "_session": {
JS:     "_id": "4320a9f1-f7a4-ef21-d555-9f6c952cd715"
JS:   },
JS:   "_id": "4320a9f1-f7a4-ef21-d555-9f6c952cd715{2}",
JS:   "_description": "Uploading VID_202113_225759.mp4",
JS:   "_upload": 4096,
JS:   "_totalUpload": 7384982,
JS:   "_status": "error"
JS: }
JS: error: java.net.SocketException: Connection reset
JS: responseCode: "-1"
JS: response: "null"
JS: type: "error"

My question is: Is it possible to configure tusd so that it behaves as a regular file upload server? If yes, how?
Here are my header values from client:
        headers: {
          "Content-Type": "application/octet-stream",
          "X-Requested-With": "XMLHttpRequest",
          Authorization: "token",
          "Upload-Metadata": "something",
          "Tus-Resumable": "1.0.0",
          "Upload-Defer-Length": 1
        },

Server logs:
tusd             | [tusd] 2021/01/04 03:58:08 event="RequestIncoming" method="POST" path="" requestId=""
tusd             | [tusd] 2021/01/04 03:58:08 event="HookInvocationStart" type="pre-create" id=""
tusd             | [tusd] 2021/01/04 03:58:08 event="HookInvocationFinish" type="pre-create" id=""
tusd             | [tusd] 2021/01/04 03:58:08 event="UploadCreated" id="4dc88cbccb867c080f58d468b1911d9d" size="7384982" url="https://tus.example.com:8888/files/4dc88cbccb867c080f58d468b1911d9d"
tusd             | [tusd] 2021/01/04 03:58:08 event="ResponseOutgoing" status="201" method="POST" path="" requestId=""
tusd             | [tusd] 2021/01/04 03:58:11 event="RequestIncoming" method="POST" path="" requestId=""
tusd             | [tusd] 2021/01/04 03:58:11 event="HookInvocationStart" type="pre-create" id=""
tusd             | [tusd] 2021/01/04 03:58:11 event="HookInvocationFinish" type="pre-create" id=""
tusd             | [tusd] 2021/01/04 03:58:11 event="UploadCreated" id="a612a4ef0eab53633d45a3a8e31d89d1" size="7384982" url="https://tus.example.com:8888/files/a612a4ef0eab53633d45a3a8e31d89d1"
tusd             | [tusd] 2021/01/04 03:58:11 event="ResponseOutgoing" status="201" method="POST" path="" requestId=""

empty (0 byte) files do get created on upload dir with .info files.

I also tried using the tus js client but due to lack of nodejs features support that does not work either.

It is true that tus-js-client does not support NativeScript right now. However, you can use the httpStack and fileReader options from tus-js-client (https://github.com/tus/tus-js-client/blob/master/docs/api.md#httpstack) to implement own versions of the Node.js/browser-dependent parts of tus-js-client. Then it should also be usable for NativeScript applications.

Is it possible to configure tusd so that it behaves as a regular file upload server?

I am not sure what you mean with "regular file upload server"? Can you clarify? tusd only accepts tus uploads and no other upload format.

empty (0 byte) files do get created on upload dir with .info files.]

It looks like you are not sending an PATCH requests. Maybe you are missing these requests with the actual content of the file.

I also tried using the tus js client but due to lack of nodejs features support that does not work either.

It is true that tus-js-client does not support NativeScript right now. However, you can use the httpStack and fileReader options from tus-js-client (https://github.com/tus/tus-js-client/blob/master/docs/api.md#httpstack) to implement own versions of the Node.js/browser-dependent parts of tus-js-client. Then it should also be usable for NativeScript applications.

Is it possible to configure tusd so that it behaves as a regular file upload server?

I am not sure what you mean with "regular file upload server"? Can you clarify? tusd only accepts tus uploads and no other upload format.

empty (0 byte) files do get created on upload dir with .info files.]

It looks like you are not sending an PATCH requests. Maybe you are missing these requests with the actual content of the file.

Closing this issue due to inactivity. Feel free to leave a comment if you want to continue the discussion :)


Refactor StreamerNode class, add tests

Changes

Refactor the StreamerNode into its own class without the main()
function so that it can have its own test suite.
Split Initialize function into two to be able to unit tests the initialization
without mocking out the remote kinesis calls in stream setup.
Add more tests to improve code coverage.

Notes

I didn't like splitting the StreamerNode Initialize function into two sections, however I couldn't figure out a way of testing KinesisVideoStreamerSetup without mocking the KinesisStreamManager which would require creating a new KinesisStreamManager Factory that would produce a mock object which i could then use to mock the functions KinesisVideoStreamerSetup, InitializeVideoProducer etc. If anyone knows of an easier way to mock this without some major refactoring please let me know.

Code Coverage Results:
Overall coverage rate:
lines......: 85.8% (497 of 579 lines)
functions..: 89.4% (126 of 141 functions)
branches...: 32.5% (596 of 1832 branches)
Issue #, if available: ROS-952
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@timrobotson Have you had a chance to build this with the latest kinesis_manager?

@timrobotson Have you had a chance to build this with the latest kinesis_manager?

I think it breaks due to some recent in changes in kinesis_manager, could you please update the streamer tests to be compatible?

Included in https://github.com/aws-robotics/kinesisvideo-ros1/pull/20


Test Python 3.10

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
19 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-0.3%) to 90.569%

Files with Coverage Reduction
New Missed Lines
%

gnosis/eth/oracles/oracles.py
2
89.27%

gnosis/eth/typing.py
2
83.33%

gnosis/safe/safe_tx.py
2
88.72%

gnosis/safe/safe.py
3
88.2%

gnosis/eth/ethereum_client.py
5
86.46%

gnosis/protocol/gnosis_protocol_api.py
5
87.36%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
2708

Relevant Lines:
2990

💛  - Coveralls


Pom.xml changes not reflected.

I've changed the pom.xml file in intellij and start running but nothing changes.
I don't think the changes has been reflected in the repo.
How to make sure that intellij pulls in the latest from the maven pom.xml?

Solution at https://github.com/CertifaiAI/CertifAI-Knowledge-Base/wiki/Update-maven-pom.xml


Docker Windows - Replication Lab - can't run db-init.sh

When using Docker for Windows on the Replication Lab, I get the following error after issuing a "docker-compose up" command.
Step 3/4 : RUN chmod +x /db-init.sh
---> Running in 3606a883e723
chmod: changing permissions of '/db-init.sh': Operation not permitted
ERROR: Service 'db1' failed to build: The command '/bin/sh -c chmod +x /db-init.sh' returned a non-zero code: 1
Tried tweaking this to the ubuntu builds but get the same error. Is there something that I need to do within the containers to enable this or a different command to run?
Edit: Changed the "Dockerfile" for these to comment out the "RUN chmod" line and it starts now. I may have missed that in the documentation when running on Docker for Windows.

We will investigate this

this problem occurs due to sql2019 GA now being non-root containers. I have a fix committed and will show up in our next PR


APP-159: Description reviewed for C++ SDK.

APP-159: Description reviewed for C++ SDK.

Looks good for me


[Search] Allow custom display fields

This change allow the developter to choose the field to display in the results. So we don't have to change our json's object field names to the fixed ones (title, price and description).
usage:
$('div[name="searchDiv"]').search({
source: [{id:0, myfield: 'some text', extrafield: 'ST'}, {id:1, myfield: 'another text', extrafield: 'AT'}],
searchFields: [
myfield
],
displayField: extrafield
});

Thanks I'll review this with 2.1

I've decided to implement this in a slightly different way, but the gist is the same.

All server fields are now a setting and can be modified at will.
I've added examples to the docs as well on how to do this.
Here's a quick fiddle to demo.
http://jsfiddle.net/npyfwkLa/

Nice work man... Congrats.

Added to dropdown with remoteData as well


submit button positioned wrong in horizontal form

Submit button is missing form-control class and column definition in horizontal form.
Generated HTML (taken from dummy app):
<form id="ember541" role="form" class="ember-view form-horizontal">
    <button id="ember602" type="submit" class="ember-view btn btn-primary">Submit </button>
</form>

Should be (bootstrap documentation):
<form class="form-horizontal">

      <button type="submit" class="btn btn-default">Sign in</button>

</form>

A work-a-round is to wrap bs-button in bs-form-element if it's a horizontal form:
{{#bs-form formLayout="horizontal"}}
  {{#bs-form-element}}
    {{bs-button defaultText="Submit" type="primary" buttonType="submit"}}
  {{/bs-form-element}}
{{/bs-form}}

At least this should be documented.

It is neither the scope of bs-form nor bs-button to add that markup, so yes, using bs-form-element is probably the way to go. So it is indeed more of a documentation thing...

Wrong examples in documentation are at:

https://github.com/kaliber5/ember-bootstrap/blob/master/addon/components/bs-form.js#L40-L46
https://github.com/kaliber5/ember-bootstrap/blob/master/addon/components/bs-form-element.js#L21-L28
https://github.com/kaliber5/ember-bootstrap/blob/master/addon/components/bs-form-element.js#L35-L45

But I'm not quite sure if it should be treated as documentation issue. To fix dummy app this code would be needed:
{{#bs-form formLayout=formLayout model=this action="submit"}}
  {{#if (eq formLayout 'horizontal')}}
    {{#bs-form-element}}
      {{bs-button defaultText="Submit" type="primary" buttonType="submit"}}
    {{/bs-form-element}}
  {{else}}
    {{bs-button defaultText="Submit" type="primary" buttonType="submit"}}
  {{/if}}
{{/bs-form}}

And I guess choosing between horizontal or vertical form layout depending on window width is a common use case. Perhaps a bs-form-submit component is needed?

The documentation defaults to Bootstrap 4. So this isn't a documentation issue anymore. I would also not expect to see improved support for positioning the button in Bootstrap 3 as that one has reached end of life. Closing the issue therefore.


Update readme

Setting PYTHON_PATH is not a good way to "install" Python packages.
Updated setup.py to avoid using deprecated ditsutils.

Would you consider switching to a pyproject.toml based installation? That way the pandas, scipy and cartopy dependencies can be added as optional dependencies quite easily (I forget how to do it with setuptools).

Are pandas, scipy and cartopy actually optional dependencies? There don't seem to be any guards around the imports!

I'm closing this pull request as it has been addressed by #32


scalapblib resolves scalapbc_2.12 in 2.13 builds of mill

In the post-0.6.2 builds of mill that are built on 2.13, compileScalaPB fails because the ScalaPBModule uses 2.12.4 as target version when it resolves the scalapbc dependency:
def scalaPBClasspath: T[Loose.Agg[PathRef]] = T {
  resolveDependencies(
    Seq(
      coursier.LocalRepositories.ivy2Local,
      MavenRepository("https://repo1.maven.org/maven2")
    ),
    Lib.depToDependency(_, "2.12.4"),
    Seq(ivy"com.thesamet.scalapb::scalapbc:${scalaPBVersion()}")
  )
}

This should be fixable by replacing the hardcoded "2.12.4" with scalaVersion(), I think.

What's really needed here is the Scala version with which mill is running. I have no idea, how we can detect that at runtime? Maybe, we should provide some API in Mill to get access to these info.

Then there are two approaches I can see:

Add the scala version used to compile as a target (or something like the $MILL_VERSION) in the assembly.

Detect "features" in the Scala API by the use of reflection and use those to determine the version of Scala that is currently loaded in the classpath.

Until that is in place, we should just bump the version here so that it at least runs.


predict() missing 1 required positional argument: 'model_or_filelike'

This previously was working for me, but it is still possible i am doing something wrong.
I am using: annotation,pos_memb=tmhmm.predict(fasta,'TMHMM.model'), as I iterate through a list of FASTA sequences.
I am (now) getting the following error:
"TypeError: predict() missing 1 required positional argument: 'model_or_filelike'"
Any tips would be appreciated.

@dgershlick this should work now. But just pass the protein sequence, not fasta.
annotation,pos_memb=tmhmm.predict(seq)


chore: merge fork's mainline back in

this was too hard to reason out how to update, so ended up doing #15 which didn't fix the issue of having the "Sync with fork" tooltip still hanging around


UI/UX, why C# with Blazor is not mensioned?

I found very interesting the information and suggestions in this document,
but why in the section UI/UX - Recommended Technologies, C# and Blazor are not mensioned?
Thanks

... so are not recommended in the  UI/UX, Front End Development, or Web Development?


Preserve Tag

Add option to tags which will prevent field clearing.

Codecov Report

Merging #1 into master will increase coverage by 7.67%.
The diff coverage is 81.48%.

@@            Coverage Diff             @@
##           master       #1      +/-   ##
==========================================
+ Coverage   59.82%   67.50%   +7.67%
==========================================
  Files           4        5       +1
  Lines         112      120       +8
==========================================
+ Hits           67       81      +14
+ Misses         33       27       -6
  Partials       12       12

Impacted Files
Coverage Δ

encrypt.go
70.21% <75.00%> (+1.32%)
:arrow_up:

decrypt.go
65.95% <81.81%> (+14.84%)
:arrow_up:

tags.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Fixed possible color collisions in the generated colormap

Motivation and context

How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[x] I have added description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

@dvkruchinin Hi, could you prepare a test for this case?

@dvkruchinin Hi, could you prepare a test for this case?

Hi, sure. A test will be prepared.


condition count is not working for arg=1

Pipeline snippet:
    - conditional:
        condition:
          type: count
          count:
            arg: ${LOG_FREQUENCY}
        processors:
          - log:
              level: ERROR
              message: "Gotcha!"

In this example I expect LOG_FREQUENCY environment variable to control how often a line is logged.
With LOG_FREQUENCY=100 every 100th message triggers a log line.
With LOG_FREQUENCY=0 nothing is logged, as expected.
However,
with LOG_FREQUENCY=1 I expect every message to trigger a log line. While in reality nothing is logged again.

Hey @voronaam, with that config and LOG_FREQUENCY=100 I would expect every message to trigger a log line except for the 100th message. In order to perform the behaviour you're specifying you would need to place your count condition within not:
    - conditional:
        condition:
          not:
            count:
              arg: ${LOG_FREQUENCY}
        processors:
          - log:
              level: ERROR
              message: "Gotcha!"

In which case I would expect both LOG_FREQUENCY=0 and LOG_FREQUENCY=1 to trigger logs for all messages, which is correct according to the documentation:

Counts messages starting from one, returning true until the counter reaches its target, at which point it will return false and reset the counter.

Can you double check your config and the behaviour you're seeing?

I see, I got the way this condition works in reverse. Thank you for the fast response.
With not there is slight problem though. Well, problem with my expectation. It works as expected for 1, 100, etc, but for 0 it sill logs everything.
I could add another condition with an and and check for the variable to not be 0.
Is it how you would do it as well or there is a better way?

Yeah, it's a little long-winded but assuming performance isn't absolute highest priority I would probably solve that with:
    - conditional:
        condition:
          and:
          - not:
              count:
                arg: ${LOG_FREQUENCY}
          - check_interpolation:
              value: ${LOG_FREQUENCY}
              condition:
                number:
                  operator: greater_than
                  arg: 0
        processors:
          - log:
              level: ERROR
              message: "Gotcha!"

If performance is a huge concern then you'll need to use two variables:
    - conditional:
        condition:
          and:
          - not:
              count:
                arg: ${LOG_FREQUENCY}
          - static: ${ENABLE_LOG:true}
        processors:
          - log:
              level: ERROR
              message: "Gotcha!"

Thank you! This answers all of my questions. Awesome.


CosmosDB output binding C# not working

Hi,
In the document, you write that the following block should be added to the run function:
[CosmosDB(databaseName: "my-database", collectionName: "my-container",     ConnectionStringSetting = "CosmosDbConnectionString"     )]IAsyncCollector<dynamic> documentsOut,
However, even with v4 of the core function tools and adding the v4 dotnet package: dotnet add package Microsoft.Azure.WebJobs.Extensions.CosmosDB
I still get errors when trying to create the CosmosDB object. It keeps saying the following:
The best overload for 'CosmosDBAttribute' does not have a parameter named 'collectionName' same for ConnectionStringSetting
I tried adding using Microsoft.Azure.WebJobs.Extensions.CosmosDB; but it says it is unnecessary
So how do I get my Visual Studio Code to figure out the appropriate CosmosDB extension/package?
Thanks

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 62965f3a-d852-cbe8-3d01-a5126a16c660
Version Independent ID: 2af2bfc6-bf21-08c6-a874-467781a4af90
Content: Connect Azure Functions to Azure Cosmos DB using Visual Studio Code
Content Source: articles/azure-functions/functions-add-output-binding-cosmos-db-vs-code.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Also, changing collectionName to containerName does not work as ConnectionStringSettings is still unknown

Here is a nice link to have:
https://learn.microsoft.com/en-us/azure/azure-functions/functions-bindings-cosmosdb-v2-output?tabs=in-process%2Cextensionv4&pivots=programming-language-csharp
which provides the whole list of attributes and their names for different versions.

@oragain
Thanks for your feedback! We will investigate and update as appropriate.

@oragain Thanks again for sharing this feedback! The samples (C#, Javascript, and Python) in this doc target the versions prior to the v4 of the CosmosDB extension. Glad to hear that you were able to locate the doc that helped you get unblocked, the following is stated specifically around the difference in attributes between v4 and prior versions:

For use with extension version 4.x, you need to replace the string collection in property and attribute names with container

As the next step, we're proceeding to assign this issue to the content author to review further and take the next course of action.
related: MicrosoftDocs/azure-docs/issues/105123


Fix partial validation sequence when a fork occurs

Review but DONT MERGE until BSQ issue with persisting has been resolved.
Fixes #2153
When a fork occurs we don't want to continue full validation of each block on the forked chain until the proof of work is higher of a block on the fork.
This will stop until the fork is 500 blocks from the fork point and the MaxReorgViolation is thrown.
There are some integration test changes in this PR which was solely put there to help write the integration tests.

PS this PR will only apply once the BSQ issue has been resolved that @zeptin is working on.

OK @Aprogiena and @dangershony please review again.
I had to refactor OnPartialValidationSucceededAsync so that we work with a PartialValidationSucceededResult to determine what should happen next. Also, the logic is now a lot tighter so that it reads and flows a lot better (which plays a big role in why these bugs keep on creeping in). Once you have approved I will fix the unit tests (which will be broker at the moment).
I synced the node past block 488095 whilst connected to the un-upgraded stratis-x core and still successfully threw the MaxReorgViolationExpception which caused the Stratis-X node to be disconnected correctly. The node then proceeded to sync further.
There is however an issue in CHT with reconnecting the old chain after the fork got disconnected where block 448096 does not yet have the block loaded (which I think is correct). It was throwing a null reference when it tries to calculate the UnconsumedBlocks.

pls only address 1 issue at the time, i can't see why not just do those 3 lines of code mentioned previously

Could you please convince me that the fix I suggest of changing just three lines of code would not work?

Could you please convince me that the fix I suggest of changing just three lines of code would not work?

Can you convince me that the logic in that method now is not better?

Won't just
if ((chainedHeaderBlocksToValidate != null) && ((connectBlocksResult == null) || connectBlocksResult.Succeeded))
instead of
if (chainedHeaderBlocksToValidate != null)
on line 467 fix the bug?

Can you convince me that the logic in that method now is not better?

That is a subjective metric.


Ignore invalid env for SSH Server (OSX fix)

I was trying to use built-in ssh server from an OSX client and I couldn't push, it didn't even log an error. I added some debug code and found that it was unexpected env pushed.
2015/12/13 06:04:33 [I] [env]: payload LC_CTYPEUTF-8' replaced LC_CTYPEUTF-8' args: '[]string{"\bLC_CTYPE\x05UTF-8"}'
2015/12/13 06:04:33 [...s/modules/ssh/ssh.go:62 func·001()] [E] len(args) 1 != 2

I found the reason:
SSH: SSH-2.0-OpenSSH_6.9
OS: OSX El Capitan
/etc/ssh/ssh_config file from OSX (I guess, I didn't put that there myself):
 Host *
   SendEnv LANG LC_*

This pull request just log invalid env arguments instead of silently return.

Thanks your PR!
But please read https://github.com/gogits/gogs/wiki/Contributing-Code

closed in favor of https://github.com/gogits/gogs/pull/2168 which also use log.Warn instead of Info

Also on develop branch


Plugin statics are ignored

Hi,
thank you for this awesome package! :+1:
I think, I found a bug. I wrote a custome plugin, and define a static function, but this function is in the Model undefined.
Plugin
const plugin = async (schema: Schema, options: IPluginOptions) => {
          (schema.static as any).search = async function () {
                   ....
            }
        }

Schema
@plugin(plugin, { ...options })
export class StoreDbModel extends Typegoose implements IStore {

}

Test
const StoreModel = new StoreDbModel().getModelForClass(StoreDbModel);
// search is undefined
StoreModel.search()

Thank you for your help

my fault, I use the static function definition after an await.
Now it is working


Add a classmethod to convert odict to dict

Lately I'm using libmproxy with requests to handle HTTP flow.
I find that many method in libmproxy.protocol.http will return a netlib.odict.ODict object, but  built-in dict type is used more frequently when interacting with other packages, such as requests.
So, a classmethod that return the built-in dict type in netlib.odict.ODict will be very handy.
I would like to add this feature if you think it is a good idea!

The key problem here is that an ODict can have multiple entries with the same key - how would you transform that to a dict? For HTTP headers we have RFC7230, which allows us to concat with ,, but that doesn't work for the other cases.
If you want to export to a "pure" datastructure, you'd have a [ [key, value], ...] list, i.e. odict.lst.
Thoughts?

Firstly, sorry for my inconsiderable idea. I only focus on the get_form() and get_query()method but never consider about the multiple entries with same key. You see, I' m working with requests, which send a POST request like:
requests.post("http://httpbin.org/post", data = {"key":"value"})

It requires a dict type object as input , at the beginning I just simply use dict(ODict) to attain that. And it will return something like {"key1":['value1'], "key2":['value2']}. Basically, I want a function can return the built-in dict type when I dealing with these things.
Maybe I make this question too general, but is it possible to add these function somewhere else?

Basically, I want a function can return the built-in dict type when I dealing with these things.

Not sure if I understand you correctly. :smiley:
Given the following ODict.lst as input:
[
  ["a", 42],
  ["a", 43],
  ["b", 44]
]

What does your function return?

As I said before, I never consider about the multiple entries with same key. :pensive:
I know that parameters with same name are allowed in HTTP, but I surprisingly find that requests does not support this feature. It seems that there is no way to send correct requests with data like you said above with requests. So I will try to use urllib2 to implement the HTTP request, I think use dict(ODict) will be OK. That is {"a":[42,43],"b":[44]}. Thanks for your help , can close this issue now!

Ok, great. Thanks for the suggestion anyway! :smiley:


Release v0.4.4

Greetings,
Would it be possible to release a new version with the latest MSSQL fix so that the Chocolatey package can be updated and then deployed :smile: ?
Thanks !
Adrien

Good point :) Tagged it now, binaries should be available in a couple of minutes!


Add more introductory quantum algorithms

Any algorithms at all will work. They belong in the examples folder :)

#46


spam report

It was found that the sauroter.de website was used to send phishing emails. Hugo was used on the website. Is there a vulnerability? Please help to investigate.
Sender<EMAIL_ADDRESS>Subject:<PHONE_NUMBER>
Nickname:<PHONE_NUMBER>
Time: Wed, 14 Aug 2024 01:48:54 +0000 (UTC)

Hugo was used on the website. Is there a vulnerability?

Hugo produces static files (HTML) and cannot not send mail.


Inversion of words in the description of "grille" mechanism

Oups, of course, sorry.

I just correct the original yaml too, no time to check if gitignoring the .json doesn't break the docausorus dev experience.

no time to check if gitignoring the .json doesn't break the docausorus dev experience.

Ok I'll do it


feat: add song editor and validator

add validator in editor page
i18n wording

Seems this should fix #28


Syntax error reported in Array library 'module Array exposing'

When I try to elm-make a file I get this error.
-- SYNTAX PROBLEM --------- elm-stuff\packages\elm-lang\core\4.0.0\src\Array.elm

I ran into something unexpected when parsing your code!

1│ module Array exposing
                ^
I am looking for one of the following things:

    a listing of values and types to expose, like (..)
    reserved word `where`
    whitespace

I had upgraded elm, but apparently had to open a new instance of vim.

I had Same problem in VSC. Had to reboot my editor aS well.


Line should be strokePickable: true by default

This is from https://github.com/phetsims/geometric-optics/issues/351, it seems like this case will come up much more often than just that one case.
From discussion with @jonathanolson, we like the idea @pixelzoom had in https://github.com/phetsims/geometric-optics/issues/351#issuecomment-1055826651 to have strokePickable:true be the default for all Line instances.
For testing, @jonathanolson and I are a bit worried about regressions, since if any ancestor to the Line as an input listener, it will cause this hit to be absorbed. In addition to pixel comparison, we will want to have QA take a good pass on a variety of sims.

I found that HSeparator and VSeparato were already set as strokePickable:true by default. This is nice because it limits down the number of cases that may have regressed. I'm running snapshot comparison against this patch right now and will report back:

Index: scenery/js/nodes/Line.ts
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/scenery/js/nodes/Line.ts b/scenery/js/nodes/Line.ts
--- a/scenery/js/nodes/Line.ts	(revision 39a6f77c6eb678487d059accdaf371780f16cfe0)
+++ b/scenery/js/nodes/Line.ts	(date<PHONE_NUMBER>540)
@@ -52,6 +52,12 @@
   constructor( x1?: number | Vector2 | LineOptions, y1?: number | Vector2, x2?: number | LineOptions, y2?: number, options?: LineOptions ) {
     super( null );

+
+    // TODO: DELETE ME
+    const strokePickableFalse = _.some( [ ...arguments ], arg => arg && arg.hasOwnProperty && arg.hasOwnProperty( 'strokePickable' ) && !arg.strokePickable );
+    if ( strokePickableFalse ) {
+      debugger;
+    }
     this._x1 = 0;
     this._y1 = 0;
     this._x2 = 0;
@@ -72,18 +78,22 @@
           y1: x1.y,
           // Second Vector2 is under the y1 name
           x2: ( y1 as Vector2 ).x,
-          y2: ( y1 as Vector2 ).y
+          y2: ( y1 as Vector2 ).y,
+
+          strokePickable: true
         }, x2 ); // Options object (if available) is under the x2 name
       }
       else {
         // assumes Line( { ... } ), init to zero for now
         assert && assert( y1 === undefined );

-        // Options object is under the x1 name
-        options = x1;
-
-        assert && assert( options === undefined || Object.getPrototypeOf( options ) === Object.prototype,
+        assert && assert( x1 === undefined || Object.getPrototypeOf( x1 ) === Object.prototype,
           'Extra prototype on Node options object is a code smell' );
+
+        // Options object is under the x1 name
+        options = extendDefined( {
+          strokePickable: true
+        }, x1 ); // Options object (if available) is under the x2 name
       }
     }
     else {
@@ -99,11 +109,16 @@
         x1: x1,
         y1: y1,
         x2: x2,
-        y2: y2
+        y2: y2,
+        strokePickable: true
       }, options );
     }

     this.mutate( options );
+    if ( !strokePickableFalse && !this.strokePickable ) {
+      debugger;
+    }
+    assert && !strokePickableFalse && assert( this.strokePickable, 'should be stroke pickable unless passed through' );
   }

Index: energy-skate-park/js/common/view/ReferenceHeightLine.js
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/energy-skate-park/js/common/view/ReferenceHeightLine.js b/energy-skate-park/js/common/view/ReferenceHeightLine.js
--- a/energy-skate-park/js/common/view/ReferenceHeightLine.js	(revision 3fba31fae1d5107c4a3e2ece29fdd21010d2c37e)
+++ b/energy-skate-park/js/common/view/ReferenceHeightLine.js	(date<PHONE_NUMBER>818)
@@ -42,9 +42,6 @@
     const lineStart = new Vector2( 0, 0 );
     const lineEnd = new Vector2( lineLength, 0 );
     const frontLine = new Line( lineStart, lineEnd, {
-
-      // strokes are not generally pickable, this allows the line itself to be dragged
-      strokePickable: true,
       lineWidth: 4,
       lineDash: lineDash,
       stroke: 'rgb(74,133,208)'
Index: geometric-optics/js/common/view/OpticalAxisNode.ts
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/geometric-optics/js/common/view/OpticalAxisNode.ts b/geometric-optics/js/common/view/OpticalAxisNode.ts
--- a/geometric-optics/js/common/view/OpticalAxisNode.ts	(revision be961c159683622627913a23cd2aa79639b31a01)
+++ b/geometric-optics/js/common/view/OpticalAxisNode.ts	(date<PHONE_NUMBER>825)
@@ -41,7 +41,6 @@

       // LineOptions
       stroke: GOColors.opticalAxisStrokeProperty,
-      strokePickable: true,
       lineWidth: 2,
       lineDash: [ 8, 5 ]
     }, providedOptions ) );
Index: sun/js/HSeparator.js
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/sun/js/HSeparator.js b/sun/js/HSeparator.js
--- a/sun/js/HSeparator.js	(revision 51d6a7b8d7522a31a819dfd54b926a5a0e7d56ec)
+++ b/sun/js/HSeparator.js	(date<PHONE_NUMBER>821)
@@ -19,8 +19,7 @@
   constructor( width, options ) {

     options = merge( {
-      stroke: 'rgb( 100, 100, 100 )',
-      strokePickable: true
+      stroke: 'rgb( 100, 100, 100 )'
     }, options );

     super( 0, 0, width, 0, options );
Index: molecule-polarity/js/common/view/BondNode.js
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/molecule-polarity/js/common/view/BondNode.js b/molecule-polarity/js/common/view/BondNode.js
--- a/molecule-polarity/js/common/view/BondNode.js	(revision 3652cc38bcd2cf915f224a1716e04a22da2ebd95)
+++ b/molecule-polarity/js/common/view/BondNode.js	(date<PHONE_NUMBER>805)
@@ -28,7 +28,6 @@
     options = merge( {
       stroke: MPColors.BOND,
       lineWidth: 12,
-      strokePickable: true, // include stroke in hit-testing
       tandem: Tandem.REQUIRED,
       visiblePropertyOptions: { phetioReadOnly: true }
     }, options );
Index: sun/js/VSeparator.js
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/sun/js/VSeparator.js b/sun/js/VSeparator.js
--- a/sun/js/VSeparator.js	(revision 51d6a7b8d7522a31a819dfd54b926a5a0e7d56ec)
+++ b/sun/js/VSeparator.js	(date<PHONE_NUMBER>810)
@@ -19,8 +19,7 @@
   constructor( height, options ) {

     options = merge( {
-      stroke: 'rgb( 100, 100, 100 )',
-      strokePickable: true
+      stroke: 'rgb( 100, 100, 100 )'
     }, options );

     super( 0, 0, 0, height, options );

For some general record keeping, there are 386 usages of new Line( and 13 usages of extends Line . I am 1/3 the way through the pixel comparison, and nothing has shown up yet. Wahoo!

Snapshot comparison flagged the following repos, but upon looking, it was only minor PDOM changes that I can't explain, but don't seem to have anything to do with Line, furthermore, I saw them occur sometimes when testing against the same shas, so I think it is unrelated.
I'm going to commit!

Committed, we should have QA poke around some simulations to see if anything is weird.


Node bootstrap hangs waiting for cluster upgrade

What happened:
Node was terminated and needed to be replaced while the control plane was being upgraded.
What you expected to happen:
Node could be bootstrapped and replace the previous node with the same AMI.
How to reproduce it (as minimally and precisely as possible):

Create a cluster on an older kubernets cluster version.
Create a managed node group with an Amazon EKS AMI
Start a cluster upgrade to a newer kubernetes cluster version
While upgrading, increase the nodegroup size to spin up new nodes
SSH into the new instance and observe that the bootstrap is hanging on bootstrap.sh -> aws eks wait cluster-active

Anything else we need to know?:
Seems to be hanging on aws eks wait cluster-active - which waits for cluster status == ACTIVE.
In this scenario, either checking for the actual values or allowing cluster status == UPDATING should also be allowed to continue to boot.

Thanks for reporting this. This looks like something we should fix. I'll work with the team to get this prioritized but we welcome contributions too. :)


Unnecessary null check on local variable after definitely assigning a non-null value

In the collections package, in lib/src/algorithms.dart, there is the following method:
int binarySearch<T>(List<T> sortedList, T value, {int compare(T a, T b)}) {
  compare ??= defaultCompare<T>();
  int min = 0;
  int max = sortedList.length;
  while (min < max) {
    int mid = min + ((max - min) >> 1);
    var element = sortedList[mid];
    int comp = compare!(element, value);
    if (comp == 0) return mid;
    if (comp < 0) {
      min = mid + 1;
    } else {
      max = mid;
    }
  }
  return -1;
}

The parameter compare is nullable, but is definitely assigned a non-null value on the first line of the method. The migration engine adds a ! on the line int comp = compare!(element, value);, which seems to be unnecessary.

The same thing happens in a slightly different form later in the same file:
void reverse(List list, [int start = 0, int? end]) {
  if (end == null) end = list.length;
  _reverse(list, start, end!);
}


Fixes #466 Add package-lock.json

Fixes #466
Checklist

[x] I have read the Contribution & Best practices Guide and my PR follows them.
[x] My branch is up-to-date with the Upstream master branch.
[x] I have added necessary documentation (if appropriate)

Changes proposed in this pull request:

Added package-lock.json

@mariobehling @vaibhavsingh97 @ParthS007 @gabru-md Please review. Thanks !!

@bhaveshAn Update?

@vaibhavsingh97 This package-lock.json was generated automatically. No idea of the lines.
See this https://github.com/npm/npm/blob/latest/doc/files/package-lock.json.md


WE-707 Integrate PWA and Service Worker

Fixes
This pull request fixes WE-707
Description
Added the possibility to download the application as a PWA
Type of change

New feature (non-breaking change which adds functionality)
Enhancement of existing functionality

Impacted Areas in Application

Frontend

Checklist:
Please tick all the boxes or remove the ones that aren't needed and explain why
Communication

[X] PR is related to an issue
[ ] I have made corresponding changes to the documentation

Code quality

[X] Code follows the style guidelines
[X] I have self-reviewed my code
[X] No new warnings are generated

Test coverage

[X] Existing tests pass
[ ] New code is covered by passing tests

Further comments
Missing logo/icon for manifest file

The cache in service worker needs to be looked at before implementing/merging.
Closing PR since PWA is not necessary in WITSML-Explorer yet.


Algorithm Development

In React (Javascript):

[ ] unit conversion: mm to in, in to ft

mm2in()
in2ft()

[ ] assuming the value in 7.125 inch rounded to nearest divisor of an in

analytically: floor(7.125) +  ceil(0.125*divsor)/divisor "in"
interpolation: x=0:divisor:1; i=interp1(x,0.125,'ceil'); y=x(i);

[ ]  assuming the value in 7.125 ft rounded to nearest ft + divisor of an in

analytically: floor(7.125) "ft" + ceil(ft2in(0.125)*divsor)/divisor "in"

In Python:

[ ] Two routes, one for each method to learn differences between dynamic ON-EVENT and PUT/GET static

Hey @brio50 , may I review the changes/enhancements to this.
Please assign this to me.

I enabled interaction limits through Moderator Options for 6 months. I do not want external influences as I'd like to build my skills first.

React Familiarization

https://react.dev/learn/add-react-to-an-existing-project#using-react-for-a-part-of-your-existing-page

createRoot()

Task 1 Completed [v0.1.0]

https://github.com/brio50/divisor/commit/cf302f034b6cbf97d1f24278f339e08631ac31d2#diff-3d74dddefb6e35fbffe3c76ec0712d5c416352d9449e2fcc8210a9dee57dff67


Issue with version 1.86 with ssh connection

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: January 2024 (version 1.86)
OS Version: Windows 11

Steps to Reproduce:

Use ssh extension to connect to a sever
You get the following error: "The connection is failed with "[Could not establish connection to "hostname": The VS Code Server failed to start]"

(Downgrading the VS code version + prevent it to update automatically solved the issue)

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.86.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

/duplicate

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


[Mizar-Arktos-Integration]Tenant-vpc-support: code changes for mizar pod controller and mizar arktos network controller

What type of PR is this?
/kind feature
What this PR does / why we need it:
In terms of the design document at https://github.com/CentaurusInfra/arktos/blob/poc-2022-01-30/docs/design-proposals/multi-tenancy/multi-tenancy-network-mizar-integration.md, this PR implements the following two functions:

Mizar-arktos-network-controller (existing controller) automatically creates Mizar CRDs - VPC and subnet, based on vpcID coming from corresponding default arktos network object, which is in Ready state, using Dynamic Client technology including Discovery REST mapper and Dynamic REST interface.

The json template files for Mizar CRDs - VPC and Subnet are located as default at the directory ./hack/runtime/.
./hack/runtime/default_mizar_network_subnet_template.json (the below is an example)
{
   "apiVersion": "mizar.com/v1",
   "kind": "Vpc",
   "metadata": {
      "name": "{{.Tenant}}-default-network"
   },
   "spec": {
      "ip": "<IP_ADDRESS>",
      "prefix": "16",
      "dividers": 1,
      "status": "Init"
   }
}

./hack/runtime/default_mizar_network_vpc_template.json (the below is an example)
{
   "apiVersion": "mizar.com/v1",
   "kind": "Subnet",
   "metadata": {
      "name": "{{.Tenant}}-default-network-subnet"
   },
   "spec": {
      "ip": "<IP_ADDRESS>",
      "prefix": "24",
      "bouncers": 1,
      "vpc": "{{.Tenant}}-default-network",
      "status": "Init"
   }
}

Mizar-pod-controller (existing controller) gets vpcID information from arktos network object in Ready state, automatically creates the following annotations, then patches the objects(pods) with correct annotations and updates to API server, finally notifies the annotations to Mizar via grpcAdaptor.

    mizar.com/subnet: <tenant name>-default-network-subnet
    mizar.com/vpc: <tenant name>-default-network

Which issue(s) this PR fixes:.
The new feature is part of Mizar Arktos integration.
Fixes # N/A
Special notes for your reviewer:
Test steps:
Note: These steps have been tested with system tenant in Carl's local dev environment which runs the codes in poc-2022-01-30 branch q131172019:CarlXie-poc-2022-01-30-tenant-vpc-support,

Prerequisite step
1)Configure two template files of vpc and subnet

./hack/runtime/default_mizar_network_subnet_template.json

change the parameters - ip, prefix, dividers based on above example

./hack/runtime/default_mizar_network_vpc_template.json

change the parameters - ip, prefix, bouncers based on above example

1.2) Run script "hack/update-generated-mizar.sh'" to regenerate gRPC codes including the file pkg/controller/mizar/builtins.pb.go if you make the changes in the mizar grpc proto file pkg/controller/mizar/builtins.proto
1.3) Run command "CNIPLUGIN=mizar ./hack/arktos-up.sh" to start Arktos scale-up cluster with Mizar. Or follow up the steps 0.6) through step 1 at https://github.com/q131172019/arktos/blob/CarlXie_singleNodeArktosCluster/docs/setup-guide/single-node-dev-scale-up-cluster-with-Mizar.md to ensure Mizar CRDs and pods are in Running state, especially check whether CRDs - vpc (system-default-network) and subnet (system-default-network-subnet) as well as dividers and bouncers of tenant 'system' are created correctly based on above two template files of VPC and Subnet .
./cluster/kubectl.sh get pods
./cluster/kubectl.sh get crds
./cluster/kubectl.sh get vpcs
./cluster/kubectl.sh get dividers
./cluster/kubectl.sh get subnets
./cluster/kubectl.sh get bouncers
./cluster/kubectl.sh get droplets
./cluster/kubectl.sh get endpoints.mizar.com
./cluster/kubectl.sh get network -AT

Test whether system tenant's two nginx pods can talk with each other via 'curl'
2.1) Use the following yaml file to create two nginx pods of tenant 'system'
~/TMP/mizar/pod-vpc-1-without-annotation.yaml

apiVersion: v1
kind: Pod
metadata:
  name: ying-nginx-system-1
  tenant: system
spec:
  containers:
  - name: nginx
    image: nginx
    ports:
      - containerPort: 443
---
apiVersion: v1
kind: Pod
metadata:
  name: ying-nginx-system-2
  tenant: system
spec:
  containers:
  - name: nginx
    image: nginx
    ports:
      - containerPort: 443

./cluster/kubectl.sh apply -f ~/TMP/mizar/pod-vpc-1-without-annotation.yaml

2.2) Check whether the annotations including two mizar CRDs - vpc and subnet are correctly added into two nginx pods
./cluster/kubectl.sh get pods/ying-nginx-system-1 --tenant system -o yaml |head -15
./cluster/kubectl.sh get pods/ying-nginx-system-2 --tenant system -o yaml |head -15

  annotations:
    <snip>
    mizar.com/vpc: system-default-network
    mizar.com/subnet: system-default-network-subnet

2.3) Check whether two nginx pods can communicate each other via curl
POD1="ying-nginx-system-1"
POD2="ying-nginx-system-2"
IP_POD1=`./cluster/kubectl.sh get pods --tenant system -o wide |grep $POD1 |awk '{print $7}'`
IP_POD2=`./cluster/kubectl.sh get pods --tenant system -o wide |grep $POD2 |awk '{print $7}'`
echo "$POD1: $IP_POD1"
echo "$POD2: $IP_POD2"
./cluster/kubectl.sh exec -ti $POD1 --tenant system -- curl $IP_POD2
./cluster/kubectl.sh exec -ti $POD2 --tenant system -- curl $IP_POD1

Test whether non-system tenant aaa's two nginx pods can talk with each other via 'curl'
3.1) Create non-system tenant 'aaa'

cat  ~/TMP/mizar/tenant-aaa.yaml

apiVersion: v1
kind: Tenant
metadata:
  name: aaa
spec:
  storageClusterId: "1"

./cluster/kubectl.sh apply -f ~/TMP/mizar/tenant-aaa.yaml

3.2) Check whether non-system tenant aaa's mizar CRDs are created successfully
./cluster/kubectl.sh get tenants
./cluster/kubectl.sh get networks -AT
./cluster/kubectl.sh get vpcs
./cluster/kubectl.sh get subnets

3.3) Use the following yaml file to create two nginx pods of non-system tenant 'aaa'
~/TMP/mizar/pod-vpc-1-without-annotation.aaa.yaml
apiVersion: v1
kind: Pod
metadata:
  name: ying-nginx-aaa-1
  tenant: aaa
spec:
  containers:
  - name: nginx
    image: nginx
    ports:
      - containerPort: 443
---
apiVersion: v1
kind: Pod
metadata:
  name: ying-nginx-aaa-2
  tenant: aaa
spec:
  containers:
  - name: nginx
    image: nginx
    ports:
      - containerPort: 443

./cluster/kubectl.sh apply -f ~/TMP/mizar/pod-vpc-1-without-annotation.aaa.yaml

3.4) Check whether the annotations including two mizar CRDs - vpc and subnet are correctly added into non-system tenant aaa's two nginx pods
./cluster/kubectl.sh get pods/ying-nginx-aaa-1 --tenant aaa -o yaml |head -15
./cluster/kubectl.sh get pods/ying-nginx-aaa-2 --tenant aaa -o yaml |head -15

  annotations:
    <snip>
    mizar.com/vpc: aaa-default-network
    mizar.com/subnet: aaa-default-network-subnet

3.5) Check whether non-system tenant aaa's two nginx pods can communicate each other via curl
POD1="ying-nginx-aaa-1"
POD2="ying-nginx-aaa-2"
IP_POD1=`./cluster/kubectl.sh get pods --tenant aaa -o wide |grep $POD1 |awk '{print $7}'`
IP_POD2=`./cluster/kubectl.sh get pods --tenant aaa -o wide |grep $POD2 |awk '{print $7}'`
echo "$POD1: $IP_POD1"
echo "$POD2: $IP_POD2"
./cluster/kubectl.sh exec -ti $POD1 --tenant aaa -- curl $IP_POD2
./cluster/kubectl.sh exec -ti $POD2 --tenant aaa -- curl $IP_POD1

Test to ensure system tenant's two nginx pods SHOULD not talk with non-system tenant aaa's two nginx pods
4.1)  Get IPs of all pods including system tenant's nginx pods and non-system tenant's nginx pods

./cluster/kubectl.sh get pods -AT -o wide

4.2) Test whether system tenant's two nginx pods SHOULD not talk with non-system tenant aaa's two nginx pods
./cluster/kubectl.sh exec -ti ying-nginx-system-1 -n default --tenant system -- curl <IP of ying-nginx-aaa-1 pod>
./cluster/kubectl.sh exec -ti ying-nginx-system-1 -n default --tenant system -- curl <IP of ying-nginx-aaa-2 pod>
./cluster/kubectl.sh exec -ti ying-nginx-system-2 -n default --tenant system -- curl <IP of ying-nginx-aaa-1 pod>
./cluster/kubectl.sh exec -ti ying-nginx-system-2 -n default --tenant system -- curl <IP of ying-nginx-aaa-2 pod>

4.3) Test whether non-system tenant aaa's two nginx pods SHOULD not talk with system tenant's two nginx pods
./cluster/kubectl.sh exec -ti ying-nginx-aaa-1 -n default --tenant aaa -- curl <IP of ying-nginx-system-1 pod>
./cluster/kubectl.sh exec -ti ying-nginx-aaa-1 -n default --tenant aaa -- curl <IP of ying-nginx-system-2 pod>
./cluster/kubectl.sh exec -ti ying-nginx-aaa-2 -n default --tenant aaa -- curl <IP of ying-nginx-system-1 pod>
./cluster/kubectl.sh exec -ti ying-nginx-aaa-2 -n default --tenant aaa -- curl <IP of ying-nginx-system-2 pod>

Clean-up after verification
5.1) Delete non-system tenant 'aaa'

./cluster/kubectl.sh delete -f ~/TMP/mizar/tenant-aaa.yaml

5.2) Delete system tenant's two nginx pods
./cluster/kubectl.sh delete -f ~/TMP/mizar/pod-vpc-1-without-annotation.yaml

Does this PR introduce a user-facing change?:
Yes.

A clean PR 1281 for final code review is created, which is used to merge into poc-2022-01-30 and includes the code changes based on HongWei's feedback and Ying’s feedback in PR1262 for master branch.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/568 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target.git

Docs Build status updates of commit 94ce85a:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-BackendEngineExceptionError] An exception occurred during the build process. Please check the raw log and retry building your content again. If the issue persists, please open a ticket in https://SiteHelp and include the URL of publishing report as well as below information: One or more errors occurred..

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Play integration example

Hi,
First, your project is very interesting ! Thank you for your work ! :)
Next, do you have an example of how to integrate an OutWatch projet in a Play one ?
I saw this kind of ScalaJS integration: https://github.com/vmunier/play-with-scalajs-example (which I already use) with a "frontend" project and a "backend" one, but it uses twirl.
Is it possible to not use twirl and just serve the output of OutWatch "compilation" ?
Thanks,
Jules

It's most definitely possible! If you check out the docs, there's a pretty easy way to get started with just an index.html. You could just serve that index.html from your Play app. :)

To clarify, OutWatch does not support server side rendering (yet). For something that has JVM support and will feel very much the same, without the whole Rx functionality though, have a look at ScalaTags


Shell extensions?

This is my first time working directly with the Windows API so please bear with me.
How do I make a shell extension w/ Rust and preferably windows? I'd rather not use windows-sys if I don't have to. I want to create a shell extension that can register in Windows and allow the Details page to be modified with my own values.
I am thinking I need to use this interface and function: IPropertyStore::SetValue
So I guess my question is how can I make, with Rust, a shell ext that will check for a certain file extension, and if it's there, use the data in that file, do something with it, and output details into the details pane?
There doesn't seem to be an example for a shell extension, so I have to start from scratch and I am kinda lost. I'm assuming I'd make a lib.rs and put something there?

This is more of a general programming question. I'd suggest https://learn.microsoft.com/ or https://stackoverflow.com/ to get started.


UploadFiles bug with FilePaths

If source file is not located which is given in FilePaths parameter an exception is thrown even if Source Action ignore/info is selected.
The reason is that the FileItem class checks that the file exists and that error is not being caught. The error should be caught or the check needs to be done inside SingleFileTransfer class.

Fixed in PR https://github.com/FrendsPlatform/Frends.SFTP/pull/175


add some parameters to the options to enable esbuild

Generally bundlers do not enjoy the use of __dirname this allows us to provide these paths as parameters.
I understand that this may not be taken as-is but you can do with it what you will, (if you want to use it at all).
Feel free to close this PR if you want to

let me try to give some more context here

i needed to add these params for what i was doing
i don't really have time to make a proper PR with unit tests but wanted to share this with others hopefully someone googling around in the future with similar needs may find this useful
i thought perhaps someone could take the code that i have here and make a proper PR if they're interested

as mentioned in the original summary

I understand that this may not be taken as-is but you can do with it what you will, (if you want to use it at all).

I should probably have said I understand this will not be taken as is esp with the tests being commented out etc

closing to clean up my PRs


Add typeclass instances for PersistentSetX

functor, pure, monad, monadZero, monadPlus, traverse, foldable, unfoldable

Released in 2.1.0


Comment count in Badges includes deleted comments

The count of the number of comments in an evidence badge includes comments that have been marked for deletion.

Should be implemented.


XamlReader.Load failed with Win32Exception 'The specified module could not be found'

I have a 32bit console app that loads a XAML-based configuration file at startup. The error below implies that the DLL cannot be found, however it is present at the path below.
Any idea what I might be doing wrong?
Dockerfile
FROM microsoft/dotnet-framework
COPY .\app .\app
CMD ["powershell", "app\app.exe"]

Exception
Exception has been thrown by the target of an invocation. ---> System.TypeInitializationException: The type initializer for '' threw an exception. ---> .ModuleLoadException: The C++ module failed to load during appdomain initialization.
---> System.DllNotFoundException: C:\Windows\Microsoft.NET\Framework\v4.0.30319\WPF\wpfgfx_v0400.dll ---> System.ComponentModel.Win32Exception: The specified module could not be found
---> System.DllNotFoundException: C:\Windows\Microsoft.NET\Framework\v4.0.30319\WPF\wpfgfx_v0400.dll ---> System.ComponentModel.Win32Exception: The specified module could not be found
--- End of inner exception stack trace ---
at MS.Internal.NativeWPFDLLLoader.LoadNativeWPFDLL(UInt16* relDllPath, UInt16* baseDllPath)
at MS.Internal.NativeWPFDLLLoader.LoadCommonDLLsAndDwrite()
at CModuleInitialize.{ctor}(CModuleInitialize* , IntPtr cleaningUpFunc)
at ?A0x4676cc04.CreateCModuleInitialize()
at ?A0x4676cc04.??__E?A0x4676cc04@cmiStartupRunner@@YMXXZ()
at _initterm_m((fnptr)* pfbegin, (fnptr)* pfend)
at .LanguageSupport.InitializePerAppDomain(LanguageSupport* )
at .LanguageSupport._Initialize(LanguageSupport* )
at .LanguageSupport.Initialize(LanguageSupport* )
--- End of inner exception stack trace ---
at .LanguageSupport.Initialize(LanguageSupport* )
at .cctor()
--- End of inner exception stack trace ---
at System.Windows.Markup.XamlReader.Load(XmlReader reader, ParserContext parserContext, XamlParseMode parseMode)
at System.Windows.Markup.XamlReader.Load(Stream stream, ParserContext parserContext)
at System.Windows.Markup.XamlReader.Load(Stream stream)

I fixed this issue by taking the latest version of the dotnet-framework image
Nothing to see here..


CI: Fix CI failures on master

(WIP for now). Starting to track them down.

One is related to https://github.com/dask/dask/pull/2310/
If a user passes Path("s3://bucket/key.csv"), but doesn't have s3fs installed, we raise an ImportError.  So they install s3fs, re-run it, and then we raise an OSError. Not ideal, but I think that's OK. Then the fix is to skip that test when s3fs isn't installed.

https://travis-ci.org/dask/dask/jobs/243250337 should be fixed by adding --doctest-ignore-import-errors to the run doing doctests. Not sure why it didn't fail before though.

Though we probably want all the optional dependencies installed for the doctest run.

Seems like s3fs wasn't being installed anywhere on Travis. My last commit installs it always, though perhaps we want a build without it, to ensure we never accidentally require it to import dask.

I'm able to reproduce the hang (like in https://travis-ci.org/dask/dask/jobs/243394188) locally.

The failures in #2466 are the same issue?

I'm still trying to pin down which tests are causing the hang with pytest-xdist. We could simply disable it for now, which will cause everything to take longer (but at least it'll finish). Let me try that quick.

though perhaps we want a build without it, to ensure we never accidentally require it to import dask.

We have import tests to check the minimal dependencies are actually minimal: https://github.com/dask/dask/blob/master/continuous_integration/travis/test_imports.sh. I'd say we should install everything needed to run all the tests.

All green. I'll spend a bit more time to figure out why those tests are hanging. But I think we should merge this today so that our CI is running again, and open follow up issues for those tests.

Fine by me, thanks for taking care of this.

https://github.com/dask/dask/issues/2471 for the two new skips.


Sites improvements

Junk drawer of relatively easy enhancements for Sites

[x] Favicon customization
[ ] Site theme preview
[x] Custom CSS support
[x] Theme with no CSS
[x] iTunes banner option
[x] Disqus comments option for blog posts

Preview isn't going to happen for now


Grainflow external compatibility  issues

This is an issue to keep track of compatibility issues between the Max and Cpp versions of grainflow in Max.

[ ]  Trav message does not register at all
[ ] Per grain looping (startPoint, endPoint, loopMode) does not function
[ ] Rate quantize semi does not work
[ ] Envelope shape working does not work


'feedback' missing several fields

In v5.03
When using 'feedback' as described in
https://flows.nodered.org/node/node-red-contrib-web-worldmap "Events From the Map".
The documentation states that I should expect to get something like:
{"payload": {
   "action": "feedback",
   "name": "some name",
   "value": "some value",
    "lat":51, "lon":0,
     "layer":"unknown"
     }
}

The value I am returning is a json object.
{"payload": {
     "name":"my_spot_report",
     "action":"feedback",
     "value":{
       "attitude":"friend",
       "geoObject":"Gnd Equip Vehicle",
       "timeout":"600",
       "name":"jan",
       "bearing":0,
       "distance":1,
       "how":"nonCoT"}
     },
   },
  "topic":"tak-map",
  "_sessionid":"4d8ff0dd-c1d7-41f2-8c13-c70c07f63017",
  "_sessionip":"<IP_ADDRESS>",
  "_msgid":"2155e06325ebcb78"
}

The 'lat', 'lon', and 'layer' are all missing.
Is  this is the same problem reported in https://github.com/dceejay/RedMap/issues/278 creeping back in?
https://github.com/dceejay/RedMap/blob/7d1782cc8b89537740921eb2c81123356c0d3a6d/worldmap/worldmap.js#L924-L941

It appears that in order to get the lat,lon the 'name' must be 'map'.

The feedback has more than one option, documentation states under :Event from the map": { "action": "feedback", "name": "some name", "value": "some value", "lat":51, "lon":0, "layer":"unknown" } // when a user calls the feedback function - see below
feedback as utility functions, states other ways to use this function.
if you are using it as utility function and wish to report back the lat lon position, you can use the rclk.lat, rclk.lng variables from your custom feedback code, to create your own custom feedback content.


[REF] merge AWS topic, merge github topic (deleted bad url)

Merge AWS and Github topic

already accepted your one pr , please contribute in another repo
thanks


Added coverage flags and test for readNumber

Added the coverage flags to measure coverage as well as a test to print it for function readNumber in IterImplForStreaming.java
This should resolve #44

Depending on outcome of #42 this will either be approved or changed

Codecov Report

Merging #48 into coverage will increase coverage by 0.04%.
The diff coverage is 90.47%.

@@             Coverage Diff              @@
##           coverage      #48      +/-   ##
============================================
+ Coverage     68.32%   68.37%   +0.04%
============================================
  Files           107      107
  Lines          7337     7358      +21
  Branches       1388     1402      +14
============================================
+ Hits           5013     5031      +18
- Misses         1868     1869       +1
- Partials        456      458       +2

Impacted Files
Coverage Δ

...c/main/java/com/jsoniter/IterImplForStreaming.java
71.09% <90.47%> (+0.82%)
:arrow_up:

Continue to review full report at Codecov.

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


waitForResize before resizing

I'm hoping this fixes the build failures

Looks like the build is still failing with this


Write a user-friendly introductory setup guide.

Feature request
As part of the docs strategy to provide better educational content, create a straightforward reproducible local environment setup to use in the introductory tutorials.
Use case
This setup guide can be referenced everywhere else in the documentation as a first step to test a feature locally. The most likely runtime at this moment would be minikube , because it's multi-platform and easy to install.

/kind documentation

Some content on minikube already at https://github.com/tektoncd/pipeline/blob/main/docs/developers/local-setup.md#using-minikube

@Yongxuanzhang adding docs for using kind: https://github.com/tektoncd/pipeline/pull/4606

The new Introductory docs are available in the "Getting Started" section of the docs page: https://tekton.dev/docs/getting-started/


2024-10-17_01-34-56_master-next_aws-cli

Automatically created.

Successfully created backport PR for kirkstone-next:

#9994

Successfully created backport PR for scarthgap-next:

#9995

Successfully created backport PR for styhead-next:

#9996