[SPARK-39081][PYTHON][SQL] Implement DataFrame.resample and Series.resample

What changes were proposed in this pull request?
Implement DataFrame.resample and Series.resample
Why are the changes needed?
To Increase pandas API coverage in PySpark
Does this PR introduce any user-facing change?
yes, new methods added
for example:
In [3]:
   ...: dates = [
   ...:     datetime.datetime(2011, 12, 31),
   ...:     datetime.datetime(2012, 1, 2),
   ...:     pd.NaT,
   ...:     datetime.datetime(2013, 5, 3),
   ...:     datetime.datetime(2022, 5, 3),
   ...: ]
   ...: pdf = pd.DataFrame(np.ones(len(dates)), index=pd.DatetimeIndex(dates), columns=['A'])
   ...: psdf = ps.from_pandas(pdf)
   ...: psdf.resample('3Y').sum().sort_index()
   ...:
Out[3]:
              A
2011-12-31  1.0
2014-12-31  2.0
2017-12-31  0.0
2020-12-31  0.0
2023-12-31  1.0

How was this patch tested?
added UT

This is amazing!

cc @itholic @xinrong-databricks mind taking a look when you find some time please 🙏

Merged to master.


Add new unit test coverage

Proposed changes
Tidied up some testing and made the code more readable
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[x] I have read the CONTRIBUTING document
[x] I have run make install-tools and have attached any dependency changes to this pull request
[x] If applicable, I have added tests that prove my fix is effective or that my feature works
[x] If applicable, I have checked that any relevant tests pass after adding my changes
[ ] If applicable, I have updated any relevant documentation (README.md)
[ ] If applicable, I have tested my cross-platform changes on Ubuntu 22, Redhat 8, SUSE 15 and FreeBSD 13

I see that in some places you have renamed dataplane to dataPlane. I think you have missed a few other places in the codebase where the name would need to be changed also to be consistent.

I just did it in some places and not all to make the PR smaller. I tried to do it in the files I touched mostly


WIP: Try to implement a base for future BlankLineAfterStatementFixer

This is an attempt to try to solve the oldest issue in this repo #1122.
The implementation of BlankLineAfterStatementFixer.
As suggested by @SpacePossum:

Maybe best to merge BlankLineAfterNamespaceFixer and BlankLineAfterOpeningTagFixer in a new one BlankLineAfterStatementFixer with config and than the request change here to it as a new config option.

it is a WIP, since I am new using this codebase and I never implement a fixer. so any help or feedback is welcome.

@keradus could you add the label "help wanted" to this PR?

Coverage decreased (-0.4%) to 91.918% when pulling 9c1e92cade0010b8938d41ab7264d4345cd18a8f on shakaran:unify_blank_line_after_fixes into 74397ac3439e1f8d2564f70df6e9be93c809e9fc on FriendsOfPHP:master.

Closing this as it:

is stale for 1.5 year and still in Draft
does not have tests

@shakaran feel free to provide new PR, but it should start with failing test cases to show what issues are going to be fixed and then implementation should cover those cases. Also, it should be branched from latest master.


Supports multi --output-format at the same time

For example, while running in the GitHub Actions, I want to use the github format to see details on the web ui, while also outputting the SARIF format to upload the result to the CodeQL.

Thanks for opening the issue. Yeah, this is a use case that ruff doesn't support well today. I wonder if we could just write a cheap conversion between the two formats. Anyway, this has come up before, see https://github.com/astral-sh/ruff/issues/2388. I'll close this issue in favor of the other rone.


pip3 install pyknowhere.xx.whl, but "import pyknowhere" Error: ModuleNotFoundError: No module named 'pyknowhere'

Hello, Everyone! I want to use the package pyknowhere. I followed github's guide and successfully compiled the package, and after the installation was successful, it still prompted that the python package could not be found, what is going on?
The installed package only has the directory pyknowhere-0.0.0.dist-info and no directory pyknowhere
Here are the details:

(z00xxxxxxx) [root@node57 python]# pip3 install ./dist/pyknowhere-0.0.0-cp310-cp310-linux_x86_64.whl
Looking in indexes: http://mirrors.tools.huawei.com/pypi/simple
Processing ./dist/pyknowhere-0.0.0-cp310-cp310-linux_x86_64.whl
Installing collected packages: pyknowhere
Successfully installed pyknowhere-0.0.0
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
(z00xxxxxxx) [root@node57 python]# python
Python 3.10.12 (main, Jul  5 2023, 18:54:27) [GCC 11.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
import pyknowhere
Traceback (most recent call last):
File "", line 1, in
ModuleNotFoundError: No module named 'pyknowhere'
exit()
(z00xxxxxxx) [root@node57 python]# ls /root/miniconda3/envs/z00878108/lib
/python3.10/site-packages/ | grep knowhere*
knowhere
knowhere-1.0.0.dist-info
pyknowhere-0.0.0.dist-info

OS: CentOS 7

It seems that you should use import knowhere

It seems that you should use import knowhere

But, the project 'ann-benchmarks' has source code pyknowhere.Index(), import knowhere report error 'No Attribute'.

This seems to be due to version inconsistency. I see that 'ann-benchmarks' uses version 1.x, but now knowhere should be version 2.x

However, when I checkout 1.x branch to generate the pyknowhere in the knowhere project, it still report error: Not Found module. Here is the details:

(z00xxxxx) [root@node57 knowhere]# git branch -v
* 1.x c0aa5fb Fix DiskANN's Async cache making (#250)
(z00xxxxx) [root@node57 knowhere]# cd python/
(z00xxxxx) [root@node57 python]# pip3 install ./dist/knowhere-1.0.0-cp310-cp310-linux_x86_64.whl
Looking in indexes: http://mirrors.tools.huawei.com/pypi/simple
Processing ./dist/knowhere-1.0.0-cp310-cp310-linux_x86_64.whl
Installing collected packages: knowhere
Successfully installed knowhere-1.0.0
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
(z00xxxxx) [root@node57 python]# python
Python 3.10.12 (main, Jul  5 2023, 18:54:27) [GCC 11.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
import pyknowhere
Traceback (most recent call last):
File "", line 1, in
ModuleNotFoundError: No module named 'pyknowhere'

How to import pyknowhere? I can't use it.

I think this project 'ann-benchmarks' should use a historical version, but I am not sure. If you want to run it, I suggest you modify the relevant code in 'ann-benchmarks' according to the current knowhere code

Hello, Everyone! I want to use the package pyknowhere. I followed github's guide and successfully compiled the package, and after the installation was successful, it still prompted that the python package could not be found, what is going on? The installed package only has the directory pyknowhere-0.0.0.dist-info and no directory pyknowhere Here are the details:

Sorry, I can't do it. The work so hard. I think there must be a version of the source code that can generate this corresponding package, otherwise the ann-benchmark project cannot run the algorithm. If anyone knows please let me know, thank you very much!

I think this project 'ann-benchmarks' should use a historical version, but I am not sure. If you want to run it, I suggest you modify the relevant code in 'ann-benchmarks' according to the current knowhere code

Sorry, I can't do it. The work so hard. I think there must be a version of the source code that can generate this corresponding package, otherwise the ann-benchmark project cannot run the algorithm. If anyone knows please let me know, thank you very much!


Re-create CLI from scratch. Using Mandarine

This project will be totally re-created using Mandarine's framework as its CLI code. Everything will be done from scratch.
Mandarine CLI will also receive more love, it is kind of abandoned here, I believe Mandarine's CLI has a great potential...
This should land along with version 3.0.0 of MandarineTS. https://github.com/mandarineorg/mandarinets
The infrastructure to re-create it, it's not quiet there yet, but it will be, I will provide more information soon.

hi, i hope using mandarine cli with flag --importmap. because deps.ts is  inhuman


Handle completed futures

If a completion stage supplier returns an exceptionally completed future, the exception in the whenComplete is not wrapped in a CompletionException, and the retry code would handle it as a success. This change would make sure that any exception is handled as an error, and still unwrap the CompletionException when needed.

Thanks @leblonk!

Released 2.0.1 with this fix.


azcli error

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az login
Errors:
No module named 'azure.cli.core.vendored_sdks.subscriptions.v2019_11_01.operations._subscription_client_operations'
Traceback (most recent call last):
Temp\pip-unpacked-wheel-yccaebu4\knack\cli.py, ln 233, in invoke
cli\core\commands\__init__.py, ln 664, in execute
cli\core\commands\__init__.py, ln 727, in _run_jobs_serially
cli\core\commands\__init__.py, ln 720, in _run_job
Local\Temp\pip-unpacked-wheel-rg200gj8\six.py, ln 703, in reraise
cli\core\commands\__init__.py, ln 698, in _run_job
cli\core\commands\__init__.py, ln 331, in __call__
azure\cli\core\__init__.py, ln 816, in default_command_handler
...
core\vendored_sdks\subscriptions\_subscription_client.py, ln 160, in tenants
vendored_sdks\subscriptions\v2019_11_01\__init__.py, ln 9, in <module>
vendored_sdks\subscriptions\v2019_11_01\_subscription_client.py, ln 21, in <module>
subscriptions\v2019_11_01\operations\__init__.py, ln 12, in <module>
ModuleNotFoundError: No module named 'azure.cli.core.vendored_sdks.subscriptions.v2019_11_01.operations._subscription_client_operations'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az login

Expected Behavior
Environment Summary
Windows-10-10.0.19041-SP0
Python 3.6.8
Installer: MSI

azure-cli 2.19.1 *

Additional Context

@jiasli for awareness

This version of Azure CLI is too old, please install the latest version.


Ignore plugin files that start with "off."

This allows to quickly deactivate files that should be executable, but should not be displayed in bitbar (helper scripts, deprecated plugins, ...)

Yes, that sounds good too

I wonder if you could check to see if that actually works? :)

I'll give it a try. I suppose this should work in most cases, but some scripts might not react well to being executed via a symlink.

Alternatively, add a myscript.n.sh file-extension prefix (a la 5m, 2h) that never runs it. This could mitigate luke3butler's problem with #79 at the same time.

Recommend
./plugins
├── available
└── enabled

with plugins being symlinked to enabled.

@manojlds nginx does the same thing and it’s bad. I keep everything under /conf.d/ instead.
The best thing is not to run the script if it doesn't specify a timer.

@lucianmarin but why? bash-it does this, and it is more relevant to this tool.

I agree that ignoring plugins that request an update every 0s or 0m is a good idea.
I do think that we need a better way of managing plugins (install, update, remove) and distinguishing plugins from helper scripts. I've opened Issue #103 to discuss how we could do this.

My current setup is similar to what @manojlds proposes, but I don't think there's a need for available/active subdirectories. I created a subdirectory Enabled in the plugins folder. I symlink plugins into it and put the subdirectory in the .gitignore file. That way it's easy to test and work with the plugins while the repository stays clean.
I can even change the interval without having to modify the original filename.
./Plugins
`─ Enabled
   `─ vox.15s.sh -> ../Music/vox.30s.sh

@jessor that's a great idea. Do you fancy writing that up in the README for others?

Please move discussion to: https://github.com/matryer/bitbar/issues/112


Issue when running npm install: missing history

node --version: v5.5.0
npm --version: 3.5.3
The error:
<EMAIL_ADDRESS>/Users/mathieu/mozilla/kinto-admin
├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

We should add an explicit npm version in our package.json requirements, eg. { "engines" : { "npm" : "^2.14.7" } }.

The issue is caused by npm v3. The solution is to downgrade to npm v2.
Your solution @n1k0 will display a warning (or block the installation if "engineStrict": true is added), but only for the installation of the package itself (not of its dependencies, check http://www.marcusoft.net/2015/03/packagejson-and-engines-and-enginestrict.html).

Displaying a warning sounds good enough to me. At list it's properly informative, and so much better than without it.


Daikin FTXD25DVMA Working with Broadlink 1101

Home Assistant version
2021.12.10
SmartIR version
1.17.4
SmartIR configuration
smartir:
  check_updates: false

climate:
  - platform: smartir
    name: Bedroom Thermostat 2
    unique_id: bedroom_generic_thermostat2
    device_code: 1101
    delay: 0.25
    controller_data: remote.broadlink_remote
    temperature_sensor: sensor.bedroom_temperature
    humidity_sensor: sensor.bedroom_humidity

Describe the bug
No bug, just reporting that climate code 1101 is working with my Daikin FTXD25DVMA Reverse Cycle Split System and Broadlink remote. I have tested all of the modes and they are all working.
Debug log
N/A
Additional context
Thanks very much for this great project, I was very pleasantly surprised to find my system was working. Saves me painstakingly programming in 480 codes! Not to mention the awesome full featured thermostat!
Please let me know if I can provide any more information or testing to assist.

Thanks!


ACL-Files with mysql.

Hi.
I'm sorry to say, that I do not understand how to configure ACLs.
I've read https://github.com/emqtt/emqttd/wiki/ACL-Design for houres now and tried different things, but I can't get easiest thinks working.
I even don't see what possibilities I have. I found that there is $c for clientId and $u for userId. And in the mysql example is $all used. Are there more?
I do not understand the example for the eq-tag and when I do have to use it.
I understand

{deny, all, subscribe, ["$SYS/#", "#"]}.

as there are two rules. One for $SYS/# and one for #
But I do not understand

{deny, all, subscribe, [{eq, "$SYS/#"}, {eq, "#"}]}.
is it for $SYS/#/#?

I need the following:
All Clients can pubsub to public/#
All Clients can pubsub to $u/#
System, Dashboard can #
All Clients can't pubsub #
This is my last config. But when I try to subscribe or publish my client disconnects.
100 1 NULL $all NULL 3 public/#
101 1 NULL $all NULL 3    $u/#
400 1 <IP_ADDRESS> NULL NULL 2 $SYS/#
401 1 <IP_ADDRESS> NULL NULL 2 #
402 1 NULL dashboard NULL 1  $SYS/#
500 0 NULL $all NULL 3   #
Thank you for your help.
Torsten

Have you enabled the mysql plugin via emqttd_ctl plugins load emqttd_plugin_mysql?
What does your plugins/emqttd_plugin_mysql/etc/plugin.config config file look like.

Hi.
Thank you for your reply.
The plugin is enabled and seems to be working with the rules from the wiki or no rules but not with mine.
My plugin.config (Password is replaced with *):

[
{emqttd_plugin_mysql, [
{mysql_pool, [
%% ecpool options
{pool_size, 8},
{pool_type, round_robin},
{auto_reconnect, 3},
   %% mysql options
   {host,     "localhost"},
   {port,     3306},
   {user,     "emqtt"},
   {password, "******************"},
   {database, "emqtt"},
   {encoding, utf8}

]},
%% select password only
{authquery, "select password from mqtt_user where username = '%u' limit 1"},
%% hash algorithm: plain, md5, sha, sha256, pbkdf2?
{password_hash, sha256},
%% select password with salt
%% {authquery, "select password, salt from mqtt_user where username = '%u'"},
%% sha256 with salt prefix
%% {password_hash, {salt, sha256}},
%% sha256 with salt suffix
%% {password_hash, {sha256, salt}},
%% comment this query, the acl will be disabled
{aclquery, "select allow, ipaddr, username, clientid, access, topic from mqtt_acl where ipaddr = >'%a' or username = '%u' or username = '$all' or clientid = '%c'"},
%% If no rules matched, return...
{acl_nomatch, allow}
]}
].

I think i understood what # is.
eq # means that you disallow to use # for the user while # means the topics itself.
Thats ok now.
But there is something strange with acls:
If I use now rule, everything works fine.
If I create a dissalow rule, nothing works. Thats fine.
But when I try to allow only one topic and dissallow the other topics, all topics keep dissallowed.
It looks like this:
id, allow,ipaddr,username,clientid,access,topic
1 , 1 , null, $all, null, 3, public/#
1000, 0, null, $all, null, 3, #521
Isn't it first match?

I don't know how the rules engine handles rules where there are multiple matches. From the example mysql file, it looks like they setup the deny rules first. Eg:
INSERT INTO `mqtt_acl` (`id`, `allow`, `ipaddr`, `username`, `clientid`, `access`, `topic`)
VALUES
(1,0,NULL,'$all',NULL,1,'$SYS/#'),
(2,0,NULL,'$all',NULL,1,'eq #'),
(3,1,'<IP_ADDRESS>',NULL,NULL,2,'$SYS/#'),
(4,1,'<IP_ADDRESS>',NULL,NULL,2,'#'),
(5,1,NULL,'dashboard',NULL,1,'$SYS/#'),

so it may be that your less specific deny rule is overwriting your more-specific allow rule.
What we've done is disallow all topics by default by editing the mysql config:
%% If no rules matched, return...
    {acl_nomatch, deny}

and then explicitly allowed certain topics in mysql.

This is working.
I am not sure if it's the right way and I think I will edit the wiki.
Thank you very much for helping me out of this.
:+1:

Basic ACLs are working. But I have a new problem where I do not understand where it comes from.
I want (working):
$u/#             Everything allowed in user-path
and:
#/public/#     subscripe every possible path that is "public"
for example: if a user creates: username/public/temperaturGermany/garden everyone should be able to read it.
I have two rules:
ID, allow, ipaddr, username, clientid, access, topic
10 1 NULL $all NULL 3 $u/#
20 1 NULL $all NULL 1 #/public/#
Thank you again.

@51systems, Are you able to give me a hint, https://github.com/emqtt/emqttd_plugin_mysql/issues/13


Enable WriteCore feature for managedserviceidentity

We need to enable WriteCore feature to managedserviceidentity. Therefore add use-write-core: true and do a regen.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.ManagedServiceIdentities


Update A7-Insufficient-Attack-Protection.md

Added new content.

Hi @davewichers can you double check these changes and send a PR with your views


Is it possible to add a configurable default volume?

I'd love for there to be an option to have a default volume for videos via this extension. Videos always start muted for me and I haven't been able to figure out a way to change that. If the extension had a settings page to configure that, that'd be incredible.
Absolutely no idea how much work this would require so no worries if it's not worth it to you. Really love the extension as-is, it's the only one that has consistently worked for me and I appreciate you making it!

Hi,
Should be possible to implement. However, there are a few browser restrictions. For example, you are not allowed to auto-play a video with sound. The browser will automatically mute it. You need to manually start the first video on the page in order to play with sound.
This could be an issue on the Reals page, but I have to try it out first.
That's probably why the original page is muted by default.
I'll let you know if my tests were successful.

@tev-tev I was able to implement this, but I'm not happy with the result.
The volume is saved and the volume is applied when reloading the page. However, the default volume control doesn't show the volume level until you un-mute the video, by clicking the speaker icon:

There is no visual indicator that the last volume is stored. And if you click in the volume bar to un-mute it, you simply set a new volume.
For the time being, I'll implement it this way. To make this feature work properly, I either have to write my own custom player bar, where I have full control over the volume bars visuals - or simply disable auto-play for the first video, so the volume is set, but you have to hit play instead (not a fan of this). Browsers don't allow to set both.


Check if data is still valid regarding to model

Algorithms can crash if a model is modified after creation of data struct. Add a method to check if a data is still valid regarding to model.

It seems that model as a const ref is not mandatory in the Data struct. The idea is to allow a default constructor with the max number of joints (useful for pre-allocation) or with a model. We also need a method with model as argument to build the data struct.

Solved by #320


Misspelling under Diaper Drive Participants screen

Hi there! "Business" is spelled wrong in the first heading under the "Diaper Drive Participants" page. See pic below. Thanks!

Working on it!


onUpdate-like onEnter handler for whole Router

I have components with need properties specifying actions required on mount. e.g.
class SomeComponent {
...
}

SomeComponent.need = [
  fetchDataForSomeComponent
];

Fetching required data onUpdate is nice because I can use Promise.all to fetch all required fetches simultaneously, across all components participating in routing. However this can be visually jarring, since the component loads before route swap, and data may change after the fetch.
The solution I am currently implementing is to move my data fetch to the onEnter hook of each of my routes. This is fine - but if I have multiple loaded routes requiring data, the data fetches must be completed one after another. I want to be able to Promise.all all my data at once.
onUpdate runs after component render. What if there were an onEnter hook that applies to the Router component rather than individual Routes? This way we could access this.state.components and get all fetch requirements simultaneously. Or perhaps this.state.components isn't accessible ... I'm really not sure how this would be implemented, but I'd love some ideas.

You should handle this with a router middleware. Refer to how this is handled in e.g. react-router-relay.

@taion OK. If this is going to be the "official" recommendation for React Router, there should be better documentation for it.

I checked out the render prop of Router, and I think that's good enough. Confused by applyRouterMiddleware.. if that's a superior interface, perhaps it should be integrated with React Router?

applyRouterMiddleware is part of this library.

Aha - I found this and so I believed it to be separate.
There's no API documentation I can find referencing applyRouterMiddleware. The one piece of documentation I did find does not explain how to do anything other than pass props down to routes (synchronously, it seems).
I took you to mean, in your initial reply, that I can use applyRouterMiddleware to delay rendering until a Promise has resolved? This is the value I find in onEnter. Since no callback is to provided to the render prop, I don't yet see how async behavior could be possible.

I get a console warning when I try dispatching an action during render (as I should) ... it's inadvisable to change the app state while rendering. I guess what I want is to sometimes be able to cancel sending new route props to the RouterContext until after my Promises have resolved. That sounds like it might be more a job for Redux middleware, but I'm unsure how I should be accessing my component fetch requirements if the location change hasn't been processed by the Router yet.

My current workaround is to use Route-level onEnter, as I was before, but instead of working on the current component I use Promise.all for nextState.routes.map(route => route.component) and get all component fetch requirements from there. Since multiple routes may have fetch requirements, I tell the Redux state that I am fetching so I'm able to only perform the fetch once. I change the state back once the Promise resolves or rejects.


Update on Doc Changes/New Release

Currently, docs are generated on startup, but there's no system in place to update them.
If three.js#21885 is merged, I propose generating on fetch and using page headers/GH API to determine whether an update has been made.

Howdy @CodyJasonBennett. Can I step up on this issue as this would be my first pull request.
:)

@vaish567, sure thing, and I appreciate the help.

Hello @CodyJasonBennett I'm still not able to figure out the problem? Could you please elaborate?

@vaish567, three.js has monthly releases with changes to documentation and examples. The bot needs to detect a new(er) release and regenerate its collections of docs and examples.

I think the script for generating docs is in my three.js fork I guess.


uniqueness with no parameters?

(Forgive me if there is a better avenue for this question.  The README does not indicate any place to ask for help.)
We had an event today where Sidekiq went down, but Clockwork kept pushing worker requests into Redis.  When we restarted Sidekiq, multiple instances of my MxxWorker were queued up, and Sidekiq started running them concurrently, despite my having set sidekiq_options unique: :all in the class.
The worker takes no parameter, so these instances should not be unique.  Why didn't Sidekiq discard the dupes.  Am I misunderstanding how this is supposed to work?
Here is the relevant part of the class definition:
class MxxWorker
  include Sidekiq::Worker

  sidekiq_options unique: :all

  def perform
    MxxWorker.run(Rails.logger)
  end

  def self.run(logger = Logger.new(STDOUT))
    # blah code
  end

  # more code
end

Thanks to anyone who can enlighten me.  I'd be happy to write some documentation about this once I understand it.

I have the same issue, when performing a long job and we deploy, sidekiq will give 8 seconds before killing the worker and push the job back to redis; when sidekiq starts up it will enqueue two jobs despite the uniqueness


Update webhook cert-manager install to work with GKE autopilot clusters

This PR changes the make install-cert-manager command to one that works for both GKE standard and autopilot clusters. The previous kubectl apply command to install cert-manager is incompatible with GKE autopilot since the default global.leaderElection.namespace is a managed namespace on autopilot.
This PR was tested by installing cert-manager using the helm commands on both standard and autopilot clusters and verifying that the webhook was successfully issued a certificate.

/gcbrun


Hang on MacOS when moving window from Mac screen to external monitor

Describe the bug
When moving from mac display to an external display, sometimes the application hangs.
To Reproduce
Steps to reproduce the behavior:

Clone the repository: https://github.com/cangosta/AvaloniaHangSolution and make it run
Click on 'Open Window Example'
Drag the application window into an external display
Drag again the application window into the mac display
Repeat steps 3 and 4 till you get the busy cursor and a hang
Check attached gif

Screenshots
https://user-images.githubusercontent.com/848857/147569367-c646ba41-8cda-4fb9-85e2-11f4f08a7d03.mp4
Desktop (please complete the following information):

OS: MacOS BigSur
Version: happens from versiom v0.10.6 till v0.10.11
Display:
-- MacBook Pro: 16-inch (3072 × 1920)
-- External Display: 1920x1080

Additional context
Add any other context about the problem here.

Unable to reproduce here (macOS Monterey 12.0.1) - @danwalmsley @Takoooooo are you able to repro?

Hi @grokys @danwalmsley @Takoooooo, any news here?

We talked with this user internally and it turned out to be not Avalonia problem.


feat(stepper): add the ability to control the position of the header in a horizontal stepper

Allows users to control where the header of a horizontal stepper is rendered. In some cases it might make more sense to have the buttons be under the content so that the user doesn't have to go back in the page layout in order to continue to the next step.

What is the status of this feature?


✨ Create font-size control

Closes #9.

Build succeeded!
Built with commit d9455f091e5eee108d1685e33a594a7fee5b70c3
https://deploy-preview-22--padpad.netlify.com


Use codecs.open to read files with python

This is the only way to specify the encoding parmeter with Python2.
The encoding parameter is needed else Python uses the locale to guess the encoding. For LANG=C, this means using ascii encoding and thus having "UnicodeDecodeError: 'ascii' codec can't decode byte ..." exceptions.

Unfortunately, we will have to keep Python 2 support for a bit more time, since it is still the default on CentOS 7 (used on a lot of super-computing facilities), which will reach EOL in 2024 ...
Fortunately, keeping Python 2 compatibility is not too hard!


Staff long range tagged as melee

Using the Trident of the Swamp on long range tags melee gear.

Having this same issue with the shadow and sanguinesti staff. Will tag melee gear on long-range like you have nothing equipped or a melee weapon. Happened around colosseum release.

having the same issue

This issue is persisting for me as well, how do I apply the fix?

Just submitted a PR which fixes this issue.


Proposal: Expand to multilingual translation using Crowdin, etc.

I love this modification and would like to make it more user friendly in my country, Japan. I hope to be able to actively help with translations as well ...

make a pull request with the translation


[Server] Credential crud

Notes for Reviewers
This PR adds credential CRUD, same as meshery cloud.
Signed commits

[X] Yes, I signed my commits.

Codecov Report
Patch coverage has no change and project coverage change: -4.29 :warning:

Comparison is base (fc58e64) 16.29% compared to head (33412bf) 12.01%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7513      +/-   ##
==========================================
- Coverage   16.29%   12.01%   -4.29%
==========================================
  Files         119      120       +1
  Lines       16455    16598     +143
==========================================
- Hits         2682     1994     -688
- Misses      13286    14282     +996
+ Partials      487      322     -165

Flag
Coverage Δ

gointegrationtests
?

unittests
12.01% <0.00%> (-0.11%)
:arrow_down:

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

Impacted Files
Coverage Δ

server/handlers/credentials_handlers.go
0.00% <0.00%> (ø)

server/router/server.go
0.00% <ø> (ø)

... and 12 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

Would you please make necessary changes?

Check it out

cha-cha-chickety-check it out, yo.

@Azanul  Let's discuss this on the Meshery Dev call. Please add this as an agenda item in the meeting minutes if you would. :)

@theBeginner86 and/or @MUzairS15 will need to review.


[SUPPORT]Performance degrade for migrating from Hudi 0.7 to Hudi 0.14

Hi Team,
I am upgrading my Spark EMR jobs FROM [Spark 2.4.8, EMR-5.36.1, Hudi 0.7] TO [Spark 3.3.1, EMR 6.10.1, and Hudi 0.14]. This upgrade is leading to a 230% performance degradation. Previously, the jobs were running in 18 minutes, but now they are taking over an hour to complete. I'm sharing screenshots below for reference. There have been no code changes apart from upgrading the dependencies. I am writing to the Hudi table in version 0.14 the same way as I did in version 0.7. For this upgrade, I have created a new copy-on-write table and am using the Simple Indexing approach.
Could you please help me debug this issue or suggest any additional configurations that might be required to improve performance?

hoodie.simple.index.parallelism can be modified to adjust the parallelism of stage 121, but it may cause the parallelism of stage 120 to decrease. You can give it a try.

Hi @KnightChess Thanks for commenting. But my major doubt is  why shuffle write is nearly doubled in hudi 0.14? And that is leading to issues in step 121

@bibhu107 if next parallelism is too bigger, shuffle data will grow. And the mayjor reason is task is too much because high, spark need to scheduler too mush task.

@bibhu107 and why shuffle data grow, I haven't looked at the code in detail; the following is just my guess. you have too much reducer, so the shuffle data may be need more meta. And on the other head, 0.7 -> 0.14, may be the shuffle's java object attr has change, this alse can cause diff. But I think parallelism is the major problem.

@bibhu107 does it can work for you? I misread the stack trace, so this parameter hasn't taken effect in the question stage. The parameter that needs to be set is a different one. If you are insert, try to set hoodie.insert.shuffle.parallelism, if upsert, set hoodie.upsert.shuffle.parallelism

Slack Thread https://apache-hudi.slack.com/archives/C4D716NPQ/p1718122404452279

Hello @KnightChess, thank you for your suggestions.
Initially, the Adaptive Query Execution (AQE) feature was disabled for the jobs because we were explicitly setting spark.sql.shuffle.partitions . Later, we enabled it using the following configuration:
spark.sql.adaptive.coalescePartitions.enabled=true
spark.sql.adaptive.skewJoin.enabled=true

Additionally, we removed the spark.sql.shuffle.partitions configuration. This change resulted in better job performance. However, we have not yet conducted any load/pressure testing.
We will share the results once we perform load testing. For now, we have moved back to Hudi 0.8 and using Spark3.3.1.
Thank you for raising the PR.

I have one query: Why do we need this PR? I expected Hudi to automatically take the deduced parallelism from Hudi 0.13.
As mentioned in the documentation for hoodie.upsert.shuffle.parallelism, it states:
From version 0.13.0 onwards, Hudi by default automatically uses the parallelism deduced by Spark based on the source data.

@bibhu107 hi, this pr is target to imporve the deduced parallelism and more user friendly, in hudi side, AQE can not effort because use rdd directly

I am closing this Issue. Thanks for support @KnightChess


Adding Multi-Line, RTL Text, and Better Caret Support to Keyboard Preview and Non-Native Keyboard

Overview
This adds better multi-line and caret support to the keyboard preview in MRTK3.  Also, the non-native keyboard had no caret/cursor on HL2.  This change adds the caret to the non-native keyboard on HL2.

Changes

Fixes: https://github.com/microsoft/MixedRealityToolkit-Unity/issues/11558

Verification

Tested on HL2 with native a non-native keyboard
Created additional unit test

/azp run


dynamic: Time Travel Debugging (TTD) integration

Summary
Develop a TTD exctractor and add keywords to the rules to use trace files generated by TTD to improve capa dynamic analysis and defeat packers.
Motivation
Because capa is trying to develop some dynamic analysis features, I would like to suggest using Microsoft TTD. Thanks to TTD, you can generate trace files that record the context of the binary at each instruction.
I could develop a TTD extractor that would add new features to capa from the trace file.
In the end, one could scan a binary sample with a TTD trace and use new rules to select a position in a trace where capa should work.
The TTD extractor would need time indicators to know when to scan in the timeline. These indicators can be TTD cursors (time position) or functions that would be hooked in the trace.
Also, the extractor would require a memory range to scan. Hence, several optimizations can be developed like scanning the heap, the module memory, the stack...
Here is a quick look of what a rule could look like:
...
- ttd:
  - time:
    - cursor: ["100:0", "200:a"]                                       # provide hardcoded time position
    - hook: ["ntdll!NtCreateThreadEx", "ntdll!NtCreateUserProcess"]    # provide functions to hook
  - memory: ["heap", "module"]                                         # select memory ranges to scan
...

This rule tries to detect thread and process creation, and scan the heap at these time positions to search some shellcode that could be loaded by a packer, or some useful strings loaded dynamically.
Alternative projects
I'm currently working on https://github.com/airbus-cert/yara-ttd which aims to apply yara rules on TTD trace files thanks to these TTD bindings.
The tool is currently working and has many use cases when dealing with yara rules on a packed binaries.
I also read the dynamic-feature-extraction branch you are working on to integrate CAPE.
The TTD integration could work alongside this project to provide a more precise analysis and an in-depth dynamic memory scan.

I didn't started yet to implement the extractor because I wanted some validation from the capa team before. Of course, I will welcome any kind of advice!

I think capa + TTD could be amazing! We've also discussed this before with @xusheng6, so tagging him here.

Thank you for your feedback! I totally agree with your proposal!
If I summarize a bit this TTD integration:

It should extend capa to analyze memory dumps based on defaults hooks
These hooks/positions should be tweakable thanks to extra command line args in capa

In the end, it should look like:
capa sample.exe sample.run                                       # analyze TTD trace sample.run with the default hooks
capa sample.exe sample.run --ttd-hook ntdll!NtCreateUserProcess  # specify a hook
capa sample.exe sample.run --ttd-cursor 100:1A                   # specify a cursor position

I have few questions though regarding the implementation:

Should I create a new TTD feature extractor like @yelhamer did for CAPE?
If so, should I base my PR on master or on the dynamic-feature-extraction branch? Even if CAPE and TTD aren't linked, I saw you discussed and work a lot on how to integrate these dynamic features in this branch, and I was wondering if there was some code that could be necessary for me in this branch.

I'm still trying to familiarize myself with the project to figure out how I'll integrate TTD, I might come with other questions soon 🙂

If I summarize a bit this TTD integration:

It should extend capa to analyze memory dumps based on defaults hooks
These hooks/positions should be tweakable thanks to extra command line args in capa

Both of these sound great, and so do the proposed command lines.

Should I create a new TTD feature extractor like @yelhamer did for CAPE?

I think it should look more like the Binary Ninja feature extractor that @xusheng6 added in #1343. That's because I'd recommend that you focus on static analysis of memory snapshots, not dynamic analysis API traces. Conceptually, capa static analysis is things like functions/basic blocks/instructions while its (proposed) dynamic analysis is things like API calls found in threads and processes. In this issue, lets focus on static analysis of memory snapshots derived from TTD traces. I'll open another issue to track the use of TTD for dynamic analysis.
Given that the idea is to have capa analyze snapshots of memory at specific points of time in TTD traces, I wonder if we can start by building:

TTD memory snapshot exporter: given a TTD trace and cursor position (or later, hook specification), write a memory snapshot and metadata to a file. Ideally we could use a common format, like minidump or similar, but not required.
a capa memory snapshot feature extractor.

These could be built in parallel as temporarily separately utilities. Then we can wire 1 and 2 within capa and add the CLI arguments, etc. The benefit is that we might also be able to provide memory snapshots from other systems, like sandboxes, which would be neat. I also suspect the TTD memory snapshot exporter might be generally useful for other things like dumping unpacked executables.
I'm just brainstorming here. What do you think?

added #1654 to track static analysis of memory snapshots

added #1655 to track dynamic analysis via TTD traces.

First of all, I think your ideas are really great!

TTD memory snapshot exporter: given a TTD trace and cursor position (or later, hook specification), write a memory snapshot(s) and metadata to a file(s). Ideally we could use a common format, like minidump or similar, but not required.

Just to be sure, should this snapshot exporter be part of capa or should it be a dependency that I could develop in another repo?
For the second point, I'll start looking at BN feature extractor to understand better then.

should this snapshot exporter be part of capa or should it be a dependency that I could develop in another repo?

I think this can be up to you. If you can find other consumers for the library, then maybe it makes sense to be external. Or maybe capa is just a good central place to store and distribute this. shrug. It's also fine to start in your own repo and then merge into capa when you're happy.

For the second point, I'll start looking at BN feature extractor to understand better then.

Great!
I'm also going to do a bit of background research on what we'd need to implement a memory snapshot feature extractor. At least so I can talk intelligently with you about it, and/or to write code alongside you :-)

Awesome! Then I'll start with an external repo and see next if it makes sense to merge into capa!
I saw your links in #1654 I'll take a look!
Thanks again for your interest in this feature!

FYI I think you could use the programmatic api to instrument and run the capas

TTD live recorder API sample
This is a sample demonstrating how a program can use TTD's live recording API to record portions of itself.

https://github.com/microsoft/WinDbg-Samples/tree/master/TTD/LiveRecorderApiSample


Instructions for using .proto

I've tried generating swagger from the proto file, but the instructions are missing.
What I've done

$GOPATH was set.

dependencies were downloaded:
$ go get -u github.com/grpc-ecosystem/grpc-gateway

built protoc-gen-swagger:
$ go build github.com/grpc-ecosystem/grpc-gateway/protoc-gen-swagger

After that, I get the following error:
$ make
protoc -I ./ -I /home/psafont/dev/ga4gh/task-execution-schemas/src/github.com/grpc-ecosystem/grpc-gateway/third_party/googleapis/  \
        --swagger_out=logtostderr=true:. \
        task_execution.proto
protoc-gen-swagger: program not found or is not executable
--swagger_out: protoc-gen-swagger: Plugin failed with status code 1.
make: *** [Makefile:13: swagger] Error 1

The progra is in pwd, so there's something I'm obviously missing as a go luddite, any directions?

Possibly you need to add export PATH=$PATH:$GOPATH/bin ? In other words, add Go's bin directory to your path?

I'll try, thanks!

Yeah, just did make install-tools and make.
Should have read the Makefile more carefully.


Fix issue#110: Communications go offline and are not restored

First described in https://github.com/goToMain/libosdp/issues/110, there is case where CP detects a timeout after 200 ms so it resets communication. However,  before that PD still send a reply to CP after CP already timeout, this created a problem with sequence mismatch errors.
This PR solves the reported issue by making PD check if it has been more than OSDP_RESP_TOUT_MS before sending a reply. Also, added a unit test for this case.

Thank you for the PR and the testing efforts involved. But this change is only fixing one of the symptoms and not the underlying issue itself.
The issue is this: one side is mistreating a response while the other side thinks it responded within the time frame it's allowed to respond. To fix this, we need to make packet handling more robust on either side and then expect them to play nice when they come together.
I'm aware of this issue (and it's been on my TODO list for a while) and currently not looking at it because it's not causing any immediate problem. For the time being, I'm spending all the time I have assigned to this project in the rust language support; but if you think issue is blocking, I can take a look at it sooner.

Closing this PR for now.


⚠ ClamAV has degraded performance

In 3d40de9, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 39 ms

Resolved: ClamAV performance has improved in 126e974.


🐛 Fix namespace override which was missing on ServiceAccount

What does this PR do?
It adds namespace override on ServiceAccount. It was missing on this object.
Motivation
See #690.
More

[x] Yes, I updated the tests accordingly
[x] Yes, I ran make test and all the tests passed

I deployed traefik using heml charts with the custom namespace option
helm install --namespace=mynamespace-traefik-v2 traefik traefik/traefik
CHART           APP VERSION
traefik-26.0.0	v2.10.6

However I still do not see the service account being created. What am I missing ?


📝 (README) Fix typo

👋🏻

👋

This project seems dead. Close it to avoid notifications on my side.


lte & gte w/ month and next month

Fix for #1363 that builds off https://github.com/amazeeio/lagoon/pull/1380
Bugfix - Define the Date Format for loading of Hits
Closing issues
closes #1363

This PR is no longer needed


Controller information missing

The instructions for configuring error handling do not say which controller the error handler should be in (Section https://docs.microsoft.com/en-us/aspnet/core/fundamentals/error-handling?view=aspnetcore-2.0#configuring-a-custom-exception-handling-page).

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

ID: 0e03fdf4-b19b-17c3-04f4-336ab6d2ee06
Version Independent ID: 38515dfb-91a5-b395-db9d-084bbaf095c8
Content: Handle errors in ASP.NET Core
Content Source: aspnetcore/fundamentals/error-handling.md
Service: unspecified
Product: asp.net-core
GitHub Login: @ardalis
Microsoft Alias: tdykstra

Great idea @jwdavidson! For MVC, we should call out the HomeController. For Razor Pages, we should call out the Error page.


Data Science with Python : Multilayer Perceptron's

Description
Name - Shraddha Pattanshetti
Batch - Data Science with Python
Sr no - 649
Batch - 17
Approach:
Introduction, What is Neuron, Hidden layers, Universal Approximators, Activation function, Conclusion
Domain
Datascience with Python
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

Issue is assigned to @ShraddhaP123 for documentation

You don't have any mentor assigned to your issue. Your assigned mentor in Aishani Singh | Kareem Negm @ShraddhaP123

Hello, I am Deepthi M with serial number:172, Batch-5. I would like to do audio on this issue.
My approach: a detailed explanation

Hello, I am Deepthi M with serial number:172, Batch-5. I would like to do audio on this issue.
My approach: a detailed explanation

Issue assigned to @deepthi1107 for audio contribution

@prathimacode-hub , Mam
I am M. Ashish Reddy. I would like to do video on this issue.
Batch number: 5
serial number:174.
Approach: A detailed explanation with examples and real life uses

Issue assigned to @ashish-reddy-20-08 for video contribution


Refactoring to support new CSI 0.2.0 spec

WIP
Please ignore for now the presense of csi.pb.go.... it is a part of PR ONLY to be able to run sanity tests sucessfully, once the last breaking change PR merges to CSI spec master branch, dep will be used to pick up right version of csi spec.
This PR refactors GOCSI to support breaking changes introduced by CSI Spec 0.2.0.
GetNodeID  ---- > NodeGetId
VolumeINfo ---- > Volume
UserCredentials ---- > (ControllerCreateCredentials, NodePublishCredentials etc)
PublishVolumeInfo ---- > PublishInfo

@akutz Appreciate a bit of help, I see a failure in make csi-sp, everything else build and test ok.
building csi-sp:
# csi-sp/service
service/node.go:27:7: undefined: csi.NodeGetIdRequest
service/node.go:28:3: undefined: csi.NodeGetIdResponse
service/service.go:29:18: cannot use service literal (type *service) as type Service in return argument:
        *service does not implement Service (missing GetNodeID method)
Makefile:89: recipe for target '/home/sbezverk/developing/go/src/csi-sp/csi-sp' failed

vendor has updated version of csi.pb.go, is there anything specific in building csi-sp which makes it to fail?

@akutz csi-sp test is failing because you do some funky stuff with building it and until 0.2.0 changes becomes official in gocsi repo, I doubt csi-sp building will succeed.  Appreciate your review of the proposed changes. thanks a lot.

Hi @sbezverk,
This is a known issue and a giant PITA. Please see golang/dep#1583 for more information.
The tl;dr is that the csi-sp target will not build if the master branch of GoCSI is incompatible with the changes being built because gocsi.sh script always creates a Gopkg.toml file that points to GoCSI master so that a new SP pulls in the latest changes. I tried editing the script to create a dependency manifest that points to the branch being built, but due to several reasons outlined in golang/dep#1581 and golang/dep#1583, this does not work.
I will push a change to GoCSI master that marks the csi-sp build job as an allowed failure for now.
One last thing, I asked that you don't make changes to the vendored copy of csi.pb.go. That will not work as dep is used during the build.

@akutz thank you for clarification.  As per csi.pb.go, do you want me to completely drop vendor/* from the commit? I have no problem doing it as long as CI pulls master for CSI spec and uses the latest changes.

I just tested it by manually failing the csi-sp job. The change works. I will merge and rebase your support branch on master.

Hi @sbezverk,
Regarding vendor, you shouldn't drop vendor, we commit vendor. What you should never, ever, do is modify files in vendor manually.
What you should do is follow the proper procedure using dep to update a dependency. So you'd change the line in the Gopkg.toml file that indicates the version of the spec to branch = "master" and then run dep ensure on the CLI.
However, before you do anything I suggest you read up on dep and become comfortable with it before trying to use it and possibly altering files that will cause your PR to be rejected.

Hi @sbezverk,
I just rebased support/sbezverk onto master so your target should have the allowed failure scenario for csi-sp.

Hi @akutz,
ok, thank you, let me push again to trigger retest.

Hello @akutz
As you can see, as soon as I deleted the commit with updated csi.pb.go, all tests started failing. Godep.toml points to the master of CSI spec, so it seems when pipeline starts the gate job, dependencies do not get updated and it uses whatever you have in vendor. Could you please check? thanks a lot. Serguei.

Hi Serguei,
When I said dep is run, I do not mean for GoCSI. I mean dep is executed for
the csi-sp generated code. You're responsible for updating vendor locally
via dep and then comitting those changes. That's why i said to make sure
you're comfortable with dep before modifying GoCSI's vendor with it.
On Thu, Feb 8, 2018 at 1:22 PM, Serguei Bezverkhi<EMAIL_ADDRESS>wrote:

Hello @akutz https://github.com/akutz
As you can see, as soon as I deleted the commit with updated csi.pb.go,
all tests started failing. Godep.toml points to the master of CSI spec, so
it seems when pipeline starts the gate job, dependencies do not get updated
and it uses whatever you have in vendor. Could you please check? thanks a
lot. Serguei.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/thecodeteam/gocsi/pull/80#issuecomment-364219602, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAGK3UbcSnJ9lEMEKHaRsOksdDJaqrvSks5tS0j2gaJpZM4R6Q3a
.

--
-a
"Ideally, a code library must be immediately
usable by naive developers, easily customized
by more sophisticated developers, and readily
extensible by experts."
                                                 -- L. Stein

@akutz ok I am familiar witjh dep ;)  it is not a big mystery. I am fighting with travis test results.  Locally on my workstation everything works fine.

Hi @sbezverk,
Great! Then it shouldn't be an issue to use it as I requested in the original issue instead of modifying things in vendor directly.

@akutz as you requested, no changes directly in vendor and all, well almost, tests are green. Appreciate if you could review this change.

Hi @sbezverk,
Thank you. I'm looking now.

Hi @sbezverk,
Please revert .travis.yml to its original version:
$ git checkout master .travis.yml

Additionally, it looks like far more than just the CSI spec was updated by dep. Can you please do the following?
$ rm -fr vendor
$ git checkout master vendor
$ dep ensure -update github.com/container-storage-interface/spec

I forgot that dep ensure updates everything allowed by the lock settings unless explicitly told otherwise.
I'm continuing to review it. I think this is it, but these two were noticeable so I went ahead and commented so hopefully they're complete by the time I finish reviewing.

Hi @sbezverk,
Also, please go ahead and squash the commits. In this instance there really is zero point in separate commits as the first commit cannot build without the updated CSI spec in the vendor directory in the second.

@akutz by some reason running dep ensure -update github.com/container-storage-interface/spec does not update the csi spec in vendor folder.
Any thoughts why would that be?


Mobs not dropping experience

I'm sure the mod originally did allow the spawned mobs to drop exp but it no longer seems to be the case, is this intentional? or is a bug? if so I would love to see it getting fixed

It's a config option and is off by default


Compiling glTF-IBL-Sampler fails

Platform: macOS, arm
I followed the instructions in the repo, and manually cloned the submodule, however I am now hitting build errors with the C++ project. It is very probable this is a me problem. 🙂
Apple clang version 15.0.0 (clang-15<IP_ADDRESS>)
Target: arm64-apple-darwin23.2.0
Thread model: posix

Errors:
message.txt

I was able to figure this out, despite my ignorance: https://github.com/pcwalton/gltf-ibl-sampler-egui/pull/3


95454: remove link in intro

The docs link to this Node.js sample app. Intro of Readme in this sample app links back to docs. Creates confusion. There's already a link to the docs at the end of this readme.

I think it might make more sense to keep the link there but point it to the landing page.

Thanks for the merge.
About the link, I did see that other projects point to the landing page. I didn't make the link go there bacause of a coupld of reasons. One is the landing page itself, which doesn't provide any value to folks just starting to learn about the project. And folks are probably coming to this project from a link either the WDC docs or from the demo (which links to WDC). Creating link to WDC has caused confusion.  We've gotten that feedback from other places where we link back and forth extensively.
I think we should minimize the links and include them if they're useful. That's why I put it in a troubleshooting section for those people who are either done reading the code or need more help. Another solution might be a "related information." section.


Constructors and ToString

Reference
https://github.com/csci-1301/csci-1301.github.io/tree/main/labs/PreciseRectangle
Describe the issue
This lab should include more emphasis on constructors and ToString. This was an oversight from summer. There is a Circle class task and I edited it to include constructor and ToString as they are included in the solution but were not in the instructions until now. But this seems very light. There should be more practice of these two topics.
Suggested solution
There is a Room class which helps practice these concepts. We should edit this lab more to include Room or edit the instructions for Circle. I am concerned implementing PreciseRectangle, Circle, and Room all at once is too ambitious for a ~50 min lab.
I am making a note of this since I might not come back to edit this lab any time soon, but it should be improved.

There is now a lab for ConstructorToString, closing this issue


Bad data output on RS485 port

I have a problem with the T-RSC3 board the output on the RS485 is not ok. I wrote a simple test program sending 8 byte of data and catch this with unusual output. Sending : 01,03,00,00,07,04,08 HEX gives an output of 7F,7E,FF,FF,F1,F7,EF,FE
Any thoughts?

Glad to hear I'm not the only one having issues trying to write data using this board

I have a problem with the T-RSC3 board the output on the RS485 is not ok. I wrote a simple test program sending 8 byte of data and catch this with unusual output. Sending : 01,03,00,00,07,04,08 HEX gives an output of 7F,7E,FF,FF,F1,F7,EF,FE
Any thoughts?

I had similar issue. please specify last additional argument true when you open Serial1
Serial1.begin(9600, SERIAL_8N1, RS485_RX_PIN, RS485_TX_PIN, true);

Thanks for the tip @sayzard
I had bought a logic analyser to try and test, but weather hasn't been suitable for me to go access this board

for RX so invert would just break it?

My board is  T-RSS3  (ESP32 S3).
In the example, utilities.h has a pin definition.
#define RS485_RX_PIN            2
#define RS485_TX_PIN            3
#define RS485_CON_PIN           4
However, there is a problem with the signal level, so I need to specify an additional true argument for invert.
Serial1.begin(9600, SERIAL_8N1, RS485_RX_PIN, RS485_TX_PIN, true);
I was able to use the ModbusMaster library on Arduino to control the inverter.

I'm using the C3 version of the board rather than the newer S3

I also use T-RSC3. When using modbus, I also had to invert the pins on the RS485 interface to successfully read/write. Are there any major differences in the pins between T-RSC3 and T-RSS3?


Update: remove parser demo

Refs https://github.com/eslint/website/issues/652
This PR removes the parser demo.
Two follow-up tasks that I'll take care once we land this:

Remove the bundling of Espree in eslint/eslint.
Replace instances of referring users to the parser with links to AST Explorer instead.

eslint/eslint PR here.


[FEAT] Implement Backend Support for Analytics Charts on Dashboard

Issue-link: https://github.com/hngprojects/hng_boilerplate_java_web/issues/39
Description
Implement backend support for analytics charts to visualize data insights for users on the dashboard. This involves creating API endpoints, processing data, ensuring security, and providing documentation.
Acceptance Criteria

API endpoints to fetch chart data are implemented and tested.
Data is processed and aggregated to provide meaningful insights.
Endpoints are secured and accessible only to authorized users.
Real-time data updates (if applicable) are implemented using WebSocket or similar technologies.
Clear documentation is provided for the API endpoints.

Purpose
To enable the frontend to integrate and display analytics charts by providing the necessary data and ensuring seamless data flow and security.
Requirements

Data API Endpoints:

Create and document API endpoints for fetching data required by various types of charts (e.g., line charts, bar charts, pie charts).

Data Aggregation and Processing:

Aggregate and process raw data to provide the necessary insights and trends for the charts.

Authentication and Authorization:

Implement authentication and authorization checks to secure the data endpoints.

Real-Time Data Updates (if applicable):

Implement WebSocket or similar technologies to push real-time data updates to the frontend.

Documentation:

Provide detailed documentation on how to access and use the API endpoints, including request and response structures.

Expected Outcome

Users can view and interact with analytics charts on the dashboard, gaining valuable insights from visualized data trends.
API endpoints are performant and secure, handling large datasets efficiently.
Clear and comprehensive documentation is available, enabling easy integration with the frontend.

Steps to Complete

Design API Schema:

Define the structure of the data that the backend will provide to the frontend.

Implement Endpoints:

Develop the API endpoints in the backend to fetch and process the data required for the charts.

Test Endpoints:

Test the API endpoints to ensure they return the correct data and handle errors gracefully.

Secure Endpoints:

Implement authentication and authorization to secure the endpoints.

Real-Time Data Updates:

If applicable, implement WebSocket or similar technologies for real-time data updates.

Documentation:

Write clear documentation on how to use the API endpoints, including examples of requests and responses.

Endpoints and Request/Response Formats

Summary Endpoint:

Endpoint: GET /api/v1/analytics/summary
Request Body: None
Response Body: (Status Code: 200){
    "status": true,
    "status_code": 200,
    "total_users": 1000,
    "active_users": 800,
    "new_users": 50,
    "total_revenue": 10000
}

Error Status Code: 404
Error Response:{
    "status": false,
    "status_code": 404,
    "error": "ResourceNotFound",
    "message": "The summary data could not be found.",
    "details": {
        "resource": "Summary"
    }
}

Line Chart Data:

Endpoint: GET /api/v1/analytics/line-chart-data
Request Body: None
Response Body: (Status Code: 200){
    "status": true,
    "status_code": 200,
    "month": ["Jan", "Feb", "Mar", "Apr", "May"],
    "totalSale": [10.0, 20.0, 30.0, 25.0, 15.0]
}

Error Status Code: 404
Error Response:{
    "status": false,
    "status_code": 404,
    "error": "DataFetchError",
    "message": "There was an error fetching the line chart data.",
    "details": {
        "chart_type": "line"
    }
}

Bar Chart Data:

Endpoint: GET /api/v1/analytics/bar-chart-data
Request Body: None
Response Body: (Status Code: 200){
    "status": true,
    "status_code": 200,
    "month": ["Jan", "Feb", "Mar", "Apr", "May"],
   "totalSale": [10.0, 20.0, 30.0, 25.0, 15.0]
}

Error Status Code: 404
Error Response:{
    "status": false,
    "status_code": 404,
    "error": "DataFetchError",
    "message": "There was an error fetching the bar chart data.",
    "details": {
        "chart_type": "bar"
    }
}

Pie Chart Data:

Endpoint: GET /api/v1/analytics/pie-chart-data
Request Body: None
Response Body: (Status Code: 200){
    "status": true,
    "status_code": 200,
    "month": ["Jan", "Feb", "Mar", "Apr", "May"],
    "totalSale": [10.0, 20.0, 30.0, 25.0, 15.0],
    "totalSaleInDegree": [36, 72, 108, 90, 54]

}

Error Status Code: 404
Error Response:{
    "status": false,
    "status_code": 404,
    "error": "DataFetchError",
    "message": "There was an error fetching the pie chart data.",
    "details": {
        "chart_type": "pie"
    }
}

Available Endpoints

GET /api/v1/analytics/summary
GET /api/v1/analytics/line-chart-data
GET /api/v1/analytics/bar-chart-data
GET /api/v1/analytics/pie-chart-data

@billmal071  Could you please review this PR?

Please follow buka guidelines


Review current probe available and used in PluginMaintenanceScoring

The current implementation of PluginMaintenanceScoring was made before a lot of new probes were added.
Maybe some of the new probe could be used there.

Here I have to look for all the probes that are not part of this map?
https://github.com/Jagrutiti/plugin-health-scoring/blob/9f0401c255f5042eb6f3a13e4355847512167634/core/src/main/java/io/jenkins/pluginhealth/scoring/scores/PluginMaintenanceScoring.java#L53-L61
Then add the test cases for this?


⚠ Bofrost.be has degraded performance

In f1c6720, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1457 ms

Resolved: Bofrost.be performance has improved in 6559ba6 after 6 minutes.


[AutoPR machinelearningservices/resource-manager] [READY TO MERGE] Add missing usages api optional parameter

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6656

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/5286


Remove the disableCanvasToImageConversion functionality from PDFThumbnailView (PR 7029 follow-up)

This functionality was originally implemented in PR #7029; however it's not, nor has it ever been, used as far as I can tell.[1]
Note in particular that the default viewer does not expose either a preference or even an option with which disableCanvasToImageConversion can be toggled, and source-code modification is thus required.
Furthermore, note also that we have multiple other instances of canvas-data accesses in both the src/display/canvas.js and src/display/text_layer.js files. If any of those are blocked, by e.g. browser settings, there will be outright rendering bugs and non-working thumbnails thus seem like a very small issue in the grand scheme of things; hence why I'm suggesting that we remove the unused disableCanvasToImageConversion functionality.

[1] For the Tor use-case mentioned in issue #7026, I believe that the solution was to white-list canvas-data accesses for its built-in PDF Viewer.

/botio-linux preview

Looks like a good plan to me. Thanks!


Plugin suggestion should allow suppressing suggesting the plugin

Type: Feature Request
The notification suggesting a plugin has a cog with an option "Don't show again". It's however not clear whether this suppresses suggesting the specific plugin, or any plugin. Either this message should be clarified ("Don't show this plugin again") or a second option should be added.
VS Code version: Code 1.84.2 (1a5daa3a0231a0fbba4f14db7ec463cf99d7768e, 2023-11-09T10:51:52.184Z)
OS version: Windows_NT x64 10.0.22621
Modes:

Thank you.


Python error upon running "az ad sp create-for-rbac"

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az ad sp create-for-rbac
Errors:
400 Client Error: Bad Request for url: http://localhost:50342/oauth2/token
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 206, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 603, in execute
    raise ex
cli/core/commands/__init__.py, ln 661, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
...
python3.6/site-packages/msrestazure/azure_active_directory.py, ln 486, in get_msi_token
    result.raise_for_status()
python3.6/site-packages/requests/models.py, ln 940, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: http://localhost:50342/oauth2/token

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az ad sp create-for-rbac --sdk-auth

Expected Behavior
Environment Summary
Linux-4.15.0-1052-azure-x86_64-with-debian-stretch-sid
Python 3.6.5
Shell: bash

azure-cli 2.0.70

Extensions:
interactive 0.4.3

Additional Context

This is known issue in Azure Cloud Shell. If you run CLI from your desktop (instead of Cloud Shell), does it work?

It does. Could Cloud Shell be updated to supply a better error message?

This is actually expected error. We are working on a fix internally.

Could you run again to see if it is working? If not, please run with --debug and share the output.

It's working in Cloud Shell now. Thanks!

add to S172


Fatal signal 11 (SIGSEGV)

[x] Review the documentation: https://virocore.viromedia.com/
[x] Search for existing issues: https://github.com/viromedia/virocore/issues
[x] Use the latest ViroCore release: https://virocore.viromedia.com/docs/releases
[x] Turn off Instant Run in Android Studio

1.Windows
2. Last version of ViroCore
3. Galaxy A6 (2018)
I received this error at different time intervals
2018-09-01 10:57:19.817 15244-15244/com.example.virosample A/libc: Fatal signal 11 (SIGSEGV), code 1, fault addr 0x0 in tid 15244 (mple.virosample)

Hi @artur0503, are you getting this error with the standard HelloWorld example? Or is it on another example project or your own project? Let us know, so we can help debug. Currently, I'm unable to repro this on the devices we have that include a Mali GPU chip.

My project is base on ARPlacingObjects project.

Currently ViroCore samples with placing objects in AR works.
Closing due to inactivity.


[rmz] Symbolic Links

I'm getting a pretty destructive behavior, while trying to remove symbolic link.
rmz fails to remove symbolic link and responds with Not a directory (os error 20), in reality, it nukes contents of the directory, that symbolic link points to.
I know that rmz is not advertised to be able to remove symbolic links (no idea if they fall under file category), but removing content that lies under it - is a very different story. Not sure if it's a bug or a feature request, but it would be better if this behavior would be ether documented or fixed.
Version: 1.1.1

Shit, I'm really sorry about that, definitely a bug. I goofed using metadata instead of symlink_metadata. Added some regression tests and released v1.1.2.


Added unique prime factorisation (python), #881 [GSSoC'  21]

Created a python program to find the unique prime factors of a given number using a modified sieve of eratosthenes.
The code is well commented and I've provided  two test cases at the end
Please review this @tarun26091999 @OjusWiZard @rudrakshi99 @plazzy99

#881

Readme was not updated


Can't merge user_target_xcconfig for pod targets: ["OSMFlutterFramework", "metamask-ios-sdk"]. Singular build setting EXCLUDED_ARCHS[sdk=iphonesimulator*] has different values.

I have to use flutter osm plugin and metamask-ios-sdk  but I can't . How can I fix this? Someone help me please?
`
platform :ios, '15.0'
CocoaPods analytics sends network stats synchronously affecting flutter build latency.
ENV['COCOAPODS_DISABLE_STATS'] = 'true'
project 'Runner', {
'Debug' => :debug,
'Profile' => :release,
'Release' => :release,
}
def flutter_root
generated_xcode_build_settings_path = File.expand_path(File.join('..', 'Flutter', 'Generated.xcconfig'), FILE)
unless File.exist?(generated_xcode_build_settings_path)
raise "#{generated_xcode_build_settings_path} must exist. If you're running pod install manually, make sure flutter pub get is executed first"
end
File.foreach(generated_xcode_build_settings_path) do |line|
matches = line.match(/FLUTTER_ROOT=(.*)/)
return matches[1].strip if matches
end
raise "FLUTTER_ROOT not found in #{generated_xcode_build_settings_path}. Try deleting Generated.xcconfig, then run flutter pub get"
end
require File.expand_path(File.join('packages', 'flutter_tools', 'bin', 'podhelper'), flutter_root)
flutter_ios_podfile_setup
target 'Runner' do
use_frameworks!
use_modular_headers!
flutter_install_all_ios_pods File.dirname(File.realpath(FILE))
pod 'PhoneNumberKit/PhoneNumberKitCore', :git => 'https://github.com/marmelroy/PhoneNumberKit', :tag => '3.6.6'
pod 'metamask-ios-sdk', :git => 'https://github.com/MetaMask/metamask-ios-sdk'
pod 'OSMFlutterFramework' , '0.4.9'
end
post_install do |installer|
installer.pods_project.targets.each do |target|
flutter_additional_ios_build_settings(target)
target.build_configurations.each do |config|
# remove min deployment target
config.build_settings.delete 'IPHONEOS_DEPLOYMENT_TARGET'
  if ['OSMFlutterFramework', 'metamask-ios-sdk'].include? target.name
    config.build_settings['EXCLUDED_ARCHS[sdk=iphonesimulator*]'] = 'i386'
  end

  # code from https://stackoverflow.com/a/59206937/3506115
  config.build_settings['EXPANDED_CODE_SIGN_IDENTITY'] = ""
  config.build_settings['CODE_SIGNING_REQUIRED'] = "NO"
  config.build_settings['CODE_SIGNING_ALLOWED'] = "NO"
  config.build_settings['ENABLE_BITCODE'] = 'NO'

  # You can enable the permissions needed here. For example to enable camera
  # permission, just remove the `#` character in front so it looks like this:
  #
  # ## dart: PermissionGroup.camera
  # 'PERMISSION_CAMERA=1'
  #
  #  Preprocessor definitions can be found in: https://github.com/Baseflow/flutter-permission-handler/blob/master/permission_handler_apple/ios/Classes/PermissionHandlerEnums.h
  config.build_settings['GCC_PREPROCESSOR_DEFINITIONS'] ||= [
    '$(inherited)',

    ## dart: PermissionGroup.calendar
    # 'PERMISSION_EVENTS=1',

    ## dart: PermissionGroup.reminders
    # 'PERMISSION_REMINDERS=1',

    ## dart: PermissionGroup.contacts
    # 'PERMISSION_CONTACTS=1',

    ## dart: PermissionGroup.camera
    # 'PERMISSION_CAMERA=1',

    ## dart: PermissionGroup.microphone
    # 'PERMISSION_MICROPHONE=1',

    ## dart: PermissionGroup.speech
    # 'PERMISSION_SPEECH_RECOGNIZER=1',

    ## dart: PermissionGroup.photos
    # 'PERMISSION_PHOTOS=1',

    ## dart: [PermissionGroup.location, PermissionGroup.locationAlways, PermissionGroup.locationWhenInUse]
    # 'PERMISSION_LOCATION=1',

    ## dart: PermissionGroup.notification
    'PERMISSION_NOTIFICATIONS=1',

    ## dart: PermissionGroup.mediaLibrary
    # 'PERMISSION_MEDIA_LIBRARY=1',

    ## dart: PermissionGroup.sensors
    # 'PERMISSION_SENSORS=1',

    ## dart: PermissionGroup.bluetooth
    # 'PERMISSION_BLUETOOTH=1',

    ## dart: PermissionGroup.appTrackingTransparency
    # 'PERMISSION_APP_TRACKING_TRANSPARENCY=1',

    ## dart: PermissionGroup.criticalAlerts
    # 'PERMISSION_CRITICAL_ALERTS=1'
  ]

end

end
end
`

i will check the issue and i will update ios sdk and let you know

thank you @liodali . I am here

i will check the issue and i will update ios sdk and let you know

Hi again. Is there anything new? I am stuck on there. I can not solve it. Need your response

try in exclude archs set this : 'i386' to 'i386 arm64'

try in exclude archs set this : 'i386' to 'i386 arm64'

I already did all of them. Unfortunately there is no solution .
https://stackoverflow.com/questions/63607158/xcode-building-for-ios-simulator-but-linking-in-an-object-file-built-for-ios-f?rq=2
I tried all of them

you're using arm mac machine m1 or m2 or intel mac ?

you're using arm mac machine m1 or m2 or intel mac ?

İntel mac

try to do pod update in ios folder of you app and see if the OSMFlutterFramework
I removed EXCLUDED_ARCHS[sdk=iphonesimulator*]

try to do pod update in ios folder of you app and see if the OSMFlutterFramework
I removed EXCLUDED_ARCHS[sdk=iphonesimulator*] from our sdk

Thank you very much. It worked. Really thanks


WASI support

Since marlowe-rs now supports targetting wasi, we should do the same here. With ongoing work for vscode to support wasi in web extensions, this might also end up being a low effort way to allow the extension to be installed in vscode.dev in the future.

completed in https://github.com/OlofBlomqvist/marlowe-lsp/commit/2f1d8cd14ad22b5a26b1d7dea78b22794366f64c


Include nslookup info in sunstone

Author Name: Alvaro Simon (Alvaro Simon)
Original Redmine Issue: 4772, https://dev.opennebula.org/issues/4772
Original Date: 2016-09-12

Hi
This is just a cosmetic request, right now Sunstone shows the IP of the running VMs. It would be possible to include the hostname instead of the IPs?
Sunstone could use the nslookup command to check if the IP is included in the DNS, if not it will show the IP as usual.
This change will help to the users to connect and handle their VMs if some IP pools are already available in the DNS.
Cheers and thanks
Alvaro

Original Redmine Comment
Author Name: Ruben S. Montero (@rsmontero)
Original Date: 2016-09-13T08:32:22Z

THANKS for the feedback!


Resolve references (eval)

I'm looking into having references like in terraform.
resource aws_something my_something {
  name = "something"
}

output foo {
  value = resource.aws_something.my_something
}

Is this something you already thought about and can share your thoughts? Is this something that you think is in scope of this library or should be external (extra crate inside or outside of this repo)?

Hi,
you can already implement something like this yourself, by using parts of the body as evaluation context, here's a quick example:
use hcl::eval::{Context, Evaluate};
use indoc::indoc;
use serde_json::value::Value;

fn main() -> Result<(), Box<dyn std::error::Error>> {
    let input = indoc! {r#"
        resource "aws_s3_bucket" "bucket" {
          bucket = "my-bucket"
        }

        resource "aws_s3_bucket_acl" "bucket_acl" {
          bucket = aws_s3_bucket.bucket.bucket
          acl    = "private"
        }
    "#};

    let body = hcl::parse(input)?;

    let value: Value = hcl::from_body(body.clone())?;

    let mut ctx = Context::new();

    for (key, value) in value["resource"].as_object().unwrap() {
        ctx.declare_var(key.clone(), hcl::to_value(value)?);
    }

    let evaluated = body.evaluate(&ctx)?;

    let formatted = hcl::format::to_string(&evaluated)?;

    let expected = indoc! {r#"
        resource "aws_s3_bucket" "bucket" {
          bucket = "my-bucket"
        }

        resource "aws_s3_bucket_acl" "bucket_acl" {
          bucket = "my-bucket"
          acl = "private"
        }
    "#};

    assert_eq!(formatted, expected);

    Ok(())
}

However, this approach only works if the thing that is referenced already has a known value (that is: it's not an expression that itself needs to be resolved first). For unknown values you would need to do multiple resolution passes.
I'm not sure if this is something that should be part of this crate, since the self-referential natural of terraform and the treatment of dynamic values is something rather special and not part of the original HCL spec. So maybe this would make sense in separate crate that extends the HCL functionality.
Terraform also has computed attributes (e.g. aws_s3_bucket.bucket.id is computed), and i'm not sure how to model these. It probably requires a configuration schema that's specific to terraform.
Also have a look at https://github.com/martinohmann/hcl-rs/issues/150 which provides examples for a similar issue.

First of all, thank you for getting back with an example - pretty cool!
I have done that approach where I made a tree of dependencies (dive into all structures to find the Expression::Variable and Expression::Traversal leaf nodes) and then figure out in what order I have to resolve them - but as you said: self-referential blocks are a problem (because I was resolving full blocks, later tried to dynamically split up evaluation into finer chunks...but that got messy)
For my next attempt I'm thinking about extending eval::Context or replace it with a trait for additional flexibility (lazy eval)

This gave me an idea. It might be possible to support partial evaluation via an additional evaluate_partial method on the Evaluate trait which would just skip evaluating expressions that contain variable references that are not known yet, instead of failing the evaluation. That way you could evaluate an HCL document in multiple passes, by adding variable values to the context that were resolved in a previous pass.
I'll try to explore if this idea would work out as I imagine when I have some time.

Closing in favor of https://github.com/martinohmann/hcl-rs/issues/184


Feature Request: First emoji reaction should trigger a notification

Current problem:
When I have a private chat / stream chat and ask something and the other part adds an emoji reaction like 👍 for yes, I don't get a notification and have to check manually if someone responds.
Wanted Solution:
So I'd like to have a feature that the first emoji reaction should trigger a notification (limit to first, cause in streams there could be really many emoji reactions).

Closing as a duplicate of #3285.


Get CI/CD to run browser test

with #1585 in mind we should be able to cook something up that automatically runs all universal test + browser specifics in a real browser.
I don't know how yet. puppeteer, airtap, karma is just a few examples of things that can execute stuff in a browser.

This is a replica of #1583

i learned about webdriver yesterday. it the new hot thing in regards to testing browsers with a rest api / protocol and controlling one or more browsers in a spec'ed way.
things like nightwatch support webdriver and all browser has it.

Let me know if having access to browserstack can help. We got (had?) an opensource-project license we never really started using.

I have prepared the CICD to run browser tests as part the regular cicd tests
https://github.com/AlaSQL/alasql/blob/develop/.github/workflows/build-test-node-browser.yml#L63


Needs Updating due to Pain Day III

Needs an update because of Pain Day III, if I knew what to do I would have done so myself
Object reference not set to an instance of an object.

All tools are broken right now due to a general custom tools issue, see here: https://discord.com/channels/833983068468936704/977177359344082994/977286123225239652

Since we have it, I get a new Error:
  A Task's exception(s) were not observed either by Waiting on the Task or accessing its Exception property. As a result, the unobserved exception was rethrown by the finalizer thread. (Value cannot be null. (Parameter 'relativeTo'))   Value cannot be null. (Parameter 'relativeTo')

This is fixed in 67406fe54fb2eac96b16e46161e167f089f2f289 (will do a release in a second) but you should really set up the tool properly and select the addon you want to export to:

I did that beforehand, but yeah, thanks for fixing it.


Another bug with moat

See also: #312
This hasn't been fixed yet but it currently annoys me a lot as I started working on my modded wolf this week.
Short: So the issue is that the AI's (wolf) defensive digging units don't reliably dig his own moat.
Long: I only ever noticed it due to the current problem being an extreme example but the digging units often stop working for a period before going back to work. However there is a high chance they will stop working completely at one point or another. Now his current castle in testing include moat that is surrounded by walls but is still accessable by stairs.

In this picture you see the pikemen just sitting around with only one of them going back and forth a few tiles without really doing anything.
Aspect 1:
As their ally I did a bunch of testing and an obvious problem is, that units can't reach the remaining moat near the isolated towers. Even when they are inside of moat (that makes them somewhat transparent to the map and they can move through blocks; going on towers from moat is just one example of that) they can target the moat but cannot reach it. It is possible to fix that problem by manually removing moat and clearing a path, then sending units in to dig up the moat from behind. Easiest fix is to adjust the aiv to prevent enclosing moat entirely.
Aspect 2:
I also noticed, that some of my own units outside of the castle would stop digging moat. My guess is that these units tried to target the unaccessable tiles within the castle which caused them to stop working because it was an invalid action to make.
Aspect 3:
Even though all of the moat is placed exactly in one build step in the AIV, the AI digs up the moat in a very specific order which includes e.g. that they won't start digging up the accessable moat outside the castle as long as the prioritized parts are not dug up. Even allied digging units that are ususally being send to help will not come digging up any part - unless you go for the fix I explained in A1.
Aspect 4:
The blue marked area in the picture is supposed to be moat but it is not being marked by the AI. You can mark and dig it up yourself, deleting the moat, deleting other moat, placing any other tile and deleting that, but the AI will not refresh the moat whatsoever. Even when the AI luckily dug up all moat around the isolated towers, it will then prioritize the small line of moat and get stuck their. Even though there is no digging plan for that tile, the AI will still get stuck trying to dig it. Placing your own moat over it also does not fix that problem.
So even if I adjust my AIV with the isolated towers' moat, there is still that bug in the back that causes the digging units to get stuck and not dig up the frontal moat.

I think the diggers generally only automatically move to another tile of marked moat if its within a certain distance (at least, from memory, for the player, you usually have to manually tell them to move to further away moat...) not sure why the AI wont give that instruction though...

The ai digs up the moat roughly circular around the keep if there are areas of the moat that are unaccessible the units will stop digging the moat and if you have too large sections of moat placed at the same time it will also eventually stop digging the moat.
The way to avoid those issues is to have all areas of moat be accessible for the digging units by doing some clever bridging sections or  https://i.imgur.com/2BFYHMC.png like here for example and breaking up the steps of moat painted at a time into more steps.
Another thing I think can also contribute is the number of moat tiles in the aiv through I cannot confirm, but I encountered it multiple times where the ai only finished the moat after I removed a couple of moat tiles form the aiv making the overall amount of moat slightly smaller but digging up 8 times the removed amount ingame

The ai digs up the moat roughly circular around the keep if there are areas of the moat that are unaccessible the units will stop digging the moat and if you have too large sections of moat placed at the same time it will also eventually stop digging the moat.
The way to avoid those issues is to have all areas of moat be accessible for the digging units by doing some clever bridging sections or https://i.imgur.com/2BFYHMC.png like here for example and breaking up the steps of moat painted at a time into more steps.
Another thing I think can also contribute is the number of moat tiles in the aiv through I cannot confirm, but I encountered it multiple times where the ai only finished the moat after I removed a couple of moat tiles form the aiv making the overall amount of moat slightly smaller but digging up 8 times the removed amount ingame

My Nizar has X times the amount of moat that my wolf has and it is all placed at the same time and I never ran into any problems with him so far, so I think that isn't an issue.
When I started editing AIV in 2018 I noticed, that the AI would always layer all of its moat at the same time even if I had several steps of moat being delayed to the end of the building order.
Can you explain a bit more detailed what you are marking on your picture and what is happening there?

ah yeah  the red arrows show the path units can take to dig up the moat

essentially using diagonal connections between stair and wall you can create bridges through which units can cross paths on different heights without interacting with one another.
as for units stopping to dig up moat i had it happen multiple times with my aivs and removing a certain amount of moat etc did help with it but yeah maybe it was a different thing that did that would def be good to know what it is as that would open up a couple more buildingsteps for some of my castles that have alot of them xP like this one for example

i mean its not even that much moat so you are probably right that its something else causing it

I see you're playing Minesweeper xD

hehe haven't blown up yet ;)


Warning: The set-output command is deprecated

This action triggers a deprecation:
Warning: The `set-output` command is deprecated and will be disabled soon.
Please upgrade to using Environment Files. For more information see:
https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

@enumag Thanks for the warning.
It is fixed in v0.5.1.
https://github.com/hankei6km/gdrive-act-send/releases/tag/v0.5.1


Enforce semver format for version

What would you like to be added:
The version of a workflow definition can be any string. Enforcing semver should be considerated.
Why is this needed:
In my understanding, the version is used to distinguish different iterations of a said workflow. For instance, a user creates a workflow named Beautiful Workflow. After some time, something needs to be changed for whatever reason. The user updates the Beautiful Workflow. There are two versions of the same workflow living side by side, for instance 1.0.0 and 1.1.0.
Currently, a version could be anything, a user could deploy workflows such as { "name": "pokemon", "version": "red" } and { "name": "pokemon", "version": "blue" }. I'm not sure that it's an intended use of this field.
WDYT ?

Hey, remember we had these discussions before and ended up with not enforcing a particular versioning style.
Worth discussing again so lets get everyones opinions here.

Sorry if I'm bringing back a dead thread, feel free to close the issue. Your take on it @tsurdilo is to have "free" version?

Let's keep it open to gather peoples opinions as it might be relevant.
At the time we discussed it some users mentioned they have very customized versioning needs and said enforcing would not fit their use case, thats why I remember we did not enforce (I was fan of semver too by the way :) )

Here is a comment I made on an old issue from last year:
"The idea in the text is that if you use a standard such as SemVer rather than some custom versioning scheme, there is a higher probability chance it will be supported by more runtimes"

It's also mentioned in spec documentation in this section: https://github.com/serverlessworkflow/specification/blob/main/specification.md#workflow-versioning

Agreed to keep an open string. IMHO, there's no strong reason for the spec to enforce such a pattern.

Not sure. On Kubernetes, for instance, they use v1, v1alpha1, and so on. Each user might have a way of setting up versions. I've seen things like 20220922 or even branch naming. I'd let the conventions be for users.

Bad example: K8s uses semver ;) (like github, gitlab, knative, argo, etc.)And versions you supplied are valid semver suffixes.

Bah, so bite me. :D
I never used these versions and have always relied on v.1.2.3 my whole life (IIRC semver recommend this approach). If semver is that permissive, I don't see why not.

@cdavernas not sure if they are compatible, at least from the regexp pov: https://regex101.com/r/vkijKf/1/
Test the examples I had. From a SDK perspective, I would apply this regexp, which will fail if someone out there decide to use one of them.

@ricardozanini yeah, alpha alone is indeed invalid, but it's a valid suffix: 1.0.0-alpha for example.
And yeah, I feel you man, I likewise used single then four digits versions a lot until I ran into semver.

As discussed in today's meeting, +1 from my side will potentially include this in the next release. It will not only help implementations to have a straightforward way to control the workflow instances version as business to follow a clear standard when defining their workflows.
@tsurdilo do you mind sharing your thoughts?

I'm all for it. small change and would bring imo good value for portability

@JBBianchi, are you opening a PR?

@ricardozanini sure, I'll try to open a draft asap.


Bug located in Maxwell2d.variable_manager

Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

Description of the bug
The evaluated value retrieved from the variable_manager[variable].evaluated_value is not correct. For variable = 'electric_period' shown in the image below, I'm expecting the evaluated value to be '0.0029411765' as displayed in EDT.

However, when I use the script below, the evaluated value returned eval_value = "0.00294117647058824rad"

Steps To Reproduce

Create a .aedt project with 3 properties:

Run the following code

Which Operating System are you using?
Windows
Which Python version are you using?
3.7
Installed packages
aiohttp==3.8.3
aiosignal==1.3.1
appdirs==1.4.4
async-timeout==4.0.2
asynctest==0.13.0
attrs==22.1.0
certifi==2022.9.24
cffi==1.15.1
charset-normalizer==2.1.1
clr-loader==0.2.4
cycler==0.11.0
fonttools==4.38.0
frozenlist==1.3.3
idna==3.4
imageio==2.22.4
kiwisolver==1.4.4
matplotlib==3.5.3
multidict==6.0.2
numpy==1.21.6
packaging==21.3
pandas==1.3.5
Pillow==9.3.0
plumbum==1.8.0
pooch==1.6.0
psutil==5.9.4
pyaedt==0.6.28
pycparser==2.21
pyparsing==3.0.9
python-dateutil==2.8.2
pythonnet==3.0.1
pytz==2022.6
pyvista==0.37.0
pywin32==305
requests==2.28.1
rpyc==5.0.1
scooby==0.7.0
six==1.16.0
typing-extensions==4.4.0
urllib3==1.26.12
vtk==9.2.2
wslink==1.9.1
yarl==1.8.1

Hi @ansRMorin ,
Have you tried to use numeric_value or value instead of evaluated_value?
These two should give you back the expected number as a float.

Then you can use the round Math function to round the number.

Hi @gmalinve,
Thank you for the response. In fact, the issue shows up when I retrieve the units, which I'm expecting a unit of time (if solved explicitly) or an empty string to be returned (if read from the units column in Maxwell). The way the variable is defined we divide an angle (rad or deg) by rpm which is a rotation (=360deg or 2pi rad) by a time so the outcome should be a time.

#2126


fix(dal): : synchronize graph access in dependent values updates

The concurrency types used by the WorkspaceSnapshot to provide interior mutability in the tokio run time are not sufficient to prevent data races when operating on the same graph on different threads, since our graph operations are not "atomic" and the graph WILL end up being read from different threads while a write operation is still in progress if it is shared between threads for modification. For example after a node is added but before the edges necessary to place that node in the right spot in the graph.
A more general solution is needed to represent the concept of a "transaction" when mutating the graph, to prevent other readers from seeing incomplete graphs from in-progress write operations.
For now, we have added a lock to DependentValuesUpdate that ensures the portion of the update that gathers the inputs to a function acquires a read lock, (which is released as soon as the function is sent to the executor, so that we don't hold on to it while executing in veritech), while the portion of the update that updates the graph with the execution result acquires a write lock.

bors r+


Use user for rbac from workflows

this ensures group is passed across correctly
/cc @gmcculloug

@gtanzillo Looks good to me.  Please review

Looks good to me too :+1:


Question: List items on event dnd-dragstart

I am trying to make a deep cloning of the list just before the dragging event starts. This is for the case of UNDOING the move in case something fails in the backend processing etc.
Are the list items still in original state and not modified on this event dnd-dragstart and can I safely make a backup copy of the list on this event and ensure the original items are present and no additional item is added to the list by this library at this point? I can see that on dnd-moved event there are extra items in the list so just wanted to be sure.
Thanks!

The directive doesn't modify any of your models, except for inserts on drop. Any other modifications are up to you by using callbacks, e.g. dnd-moved


Save/Load buttons in settings menu are not working properly.

Empty window pops up and that's it.

Use the save in the viewport if you want to download the model


Fastnlight: Watch zacharydanielconcrete.com

Fastnlight requests the watch of the watch_keyword zacharydanielconcrete\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Jeff Schaller in Charcoal HQ


Does it work for EU domains?

I am struggling to make it work.
I think the problem is that I am an EU user and there is no option to choose wether to use a the global (https://api.mailgun.net) or the EU server (https://api.eu.mailgun.net).
Trying to access an EU account from the global server doesn't work

Hello, thanks for your feedback. You are correct, currently there is no support for EU region. This would make a good addition. An update for this will come in the next coming weeks. Most likely in the form of a region select option in the Send node configuration.

An update with support was published last month. We'll close this issue. Let us know if you face any other issues.


Fix: Calendar fix for DST

Brief Information
This pull request is in the type of:

[x] bug fixing
[ ] new feature
[ ] others

What does this PR do?

Fixes calendar chart layout when user is in a timezone with DST
Fixed issues

https://github.com/apache/incubator-echarts/issues/12143
https://github.com/apache/incubator-echarts/issues/10430
Details
Before: What was the problem?

When user is in a timezone with DST, the last several months of the year on the calendar chart begin to overlap eachother.

After: How is it fixed in this PR?

The fix is based on the solution from https://github.com/apache/incubator-echarts/issues/10430

There is currently another PR attempting to fix this problem here: https://github.com/apache/incubator-echarts/pull/12081 - however, from testing, it did not appear to correct the problem.
Usage
Are there any API changes?

[ ] The API has been changed.

Related test cases or examples to use the new APIs
NA.
Others
Merging options

[ ] Please squash the commits into a single one when merge.

Other information

This fix works great for us, would be nice to get this merged

@damiangreen This PR is schedule for 4.8.0 so it will be reviewed soon. Thanks!

Thanks @Ovilia , we're really looking forward to it

Great! When will it hit npm?

@damiangreen We need a few more weeks to go through Apache release progress, which usually takes about 2-4 weeks.

@Ovilia I have checked the master branch recently and noticed that my changes have been reverted - do you mind confirm what happened please?
https://github.com/apache/incubator-echarts/commits/master/src/coord/calendar/Calendar.js

@mikeyshing88 Thanks.