Add ShardingSphereChaos CRD and convert chaos to Chaos Mesh logic in Reconcile function

Add basic convert type: Selector,Pod Chaos,Network Chaos,WorkFlow.
Add convert functions in Controller.Add convert and create chaos logic in Reconcile function.
Add ChaoKind Enums

I have tested it usage in kubenetes environment.It works well As expected.

lgtm

LGTM


Default module doesn't work with current version.

In WHMCS 7.5.x if I install this module as is and place the tests in the recommended location (updating the tests to extend TestCase), I cannot run the tests successfully. I get the following errors:
1) WHMCSModuleTest::testRequiredConfigOptionsParametersAreDefined
call_user_func() expects parameter 1 to be a valid callback, function 'addonmodule_config' not found or invalid function name

/var/www/clients/tests/WHMCSModuleTest.php:37

1) WHMCSModuleTest::testRequiredConfigOptionsFunctionExists
Failed asserting that false is true.

/var/www/clients/tests/WHMCSModuleTest.php:29

Turns out I just needed to make sure the _bootstrap.php file was being used by phpunit.
For anyone else just trying to run this as-is, you need to update the unit tests to extend TestCase as I mentioned, so the test class should look like this at the top:
use PHPUnit\Framework\TestCase;

class WHMCSModuleTest extends TestCase


--inspect-brk causes e2e tests to hang

It's not easy to debug e2e tests at the moment because you can't step through them. A temporary work around is to run ./scripts/test.js with --inspect-brk --separate-browser-processes --no-headless

Was fixed by #418


Quotes auto-completion

@AndreSayreMSFT @EricMSFT @Microsoft/poshtools

Should fix #137 ?

Looks good to me.

Yes, this will fix #137


Add VAT ID format per country

Updating VATnumber row

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

File
Status
Preview URL
Details

:x:Error

Details

[Error-Forbidden.GitSSOAuthorization] Resource protected by organization SAML enforcement. You must grant your OAuth token access to this organization.

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


refactoring CLI as proposed in #21

Refactoring CLI commands a bit as proposed in #21
% ftw

Usage: ftw [OPTIONS] COMMAND [ARGS]...

  Fields of The World (FTW) - Command Line Interface

Options:
  --help  Show this message and exit.

Commands:
  data       Downloading, unpacking, and preparing the FTW dataset.
  inference  Running inference on satellite images plus data prep.
  model      Training and testing FTW models.

% ftw data
Usage: ftw data [OPTIONS] COMMAND [ARGS]...

  Downloading, unpacking, and preparing the FTW dataset.

Options:
  --help  Show this message and exit.

Commands:
  download  Download the FTW dataset.
  unpack    Unpack the downloaded FTW dataset.

% ftw inference
Usage: ftw inference [OPTIONS] COMMAND [ARGS]...

  Running inference on satellite images plus data prep.

Options:
  --help  Show this message and exit.

Commands:
  download  Download 2 Sentinel-2 scenes & stack them in a single file...
  run       Run inference on the stacked satellite images

@calebrob6 - I merged #19 into here and refactored to be its own command. For my initial commit I aimed to keep it faithful to yours (not trying to add anything new). I think the one thing I lost was the 'warning' about it not being over 1 if it's geographic, as I'm not sure how to get that info from reading the file in.
% ftw inference polygonize --help
Usage: ftw inference polygonize [OPTIONS]

  Polygonize the output from inference

Options:
  --input_fn PATH   Input raster file to polygonize.  [required]
  --output_fn TEXT  Output filename for the polygonized data.  [required]
  --simplify FLOAT  Simplification factor to use when polygonizing.
  --overwrite       Overwrite outputs if they exist.
  --help            Show this message and exit.

% ftw inference polygonize --input_fn ~/Downloads/nl_output.tif --output_fn ~/Downloads/nl_inf_20.gpkg --simplify 20 --overwrite
Polygonizing input file: /Users/cholmes/Downloads/nl_output.tif
Processing mask windows: 36it [00:35,  1.01it/s]
Finished polygonizing output at /Users/cholmes/Downloads/nl_inf_20.gpkg in 38.12s


Removed all hardcoded string in DeleteHelper

Hello, i did small changes for you.
Removed all hardcoded strings in DeleteHelper and refractor they ID fo better navigation in Strings.xml
Tested on my Redmi4x and Nexus One(emul) API 25 and 22. Everything works correctly.

Can you please rebase your branch to fix conflicts. It is a good practice to fetch and merge upstream/master before creating a new branch.

I removed the conflict in the branches.If I do something wrong, please tell me.
I have little experience using git and make mistake

Thanks @PavelAplevich , git part seems fine now. But after your changes some tests fails:
fr.free.nrw.commons.delete.DeleteHelperTest > makeDeletionForNullToken FAILED
    java.lang.NullPointerException at DeleteHelperTest.kt:79
fr.free.nrw.commons.delete.DeleteHelperTest > makeDeletion FAILED
    java.lang.NullPointerException at DeleteHelperTest.kt:63
254 tests completed, 2 failed

Can you please review them?

@neslihanturan I will try, but I still can not understand what the problem is. I need some time

Take your time @PavelAplevich , I see some possible problems on the code now, I will submit a detailed review tomorrow, it should help.

@neslihanturan I think I solved the problem.

Codecov Report

Merging #3052 into master will increase coverage by 0.03%.
The diff coverage is 66.66%.

@@            Coverage Diff            @@
##           master   #3052      +/-   ##
=========================================
+ Coverage    4.41%   4.45%   +0.03%
=========================================
  Files         259     259
  Lines       12309   12313       +4
  Branches     1056    1056
=========================================
+ Hits          544     548       +4
  Misses      11726   11726
  Partials       39      39

Impacted Files
Coverage Δ

.../java/fr/free/nrw/commons/delete/DeleteHelper.java
54.34% <66.66%> (+2.07%)
:arrow_up:

Continue to review full report at Codecov.

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

I hope this time everything is done well)

"decision problem with color of info icon #2940"
I wanted to do it in another pullrequest(

I create copy icon with right color, because drawabletint dont work with API lower 23

I dont undestand, how i can delete my comments and how i can comment what i want. In "Correction DeleteHelper and string.xml" I correcting hardcoded string.  In "decision problem with color of info icon #2940" I tried to fix issue #2940 . But i do it in this pullrequest(

I dont undestand why i dont see problems in this files in my commit. Only added 1 file and change 1 line

Accept my apologies for my carelessness. Think this time my changes in DeleteHelper and string.xml right

Sorry @PavelAplevich but some of the irrelevant changes are still there. I noted it to the file. You can always click on files changed tab and see differences your PR made. Other parts seems correct to me, thanks for your fast responses and cooperation:)

I revert my commit. I think we can do #2940 issue in another pull request.
Everything should be right now.


[Status bar: rich hover] Still hard to mouseover

Same as #126888
When moving the cursor diagonally it closes too eagerly.

I do wonder if we can either set a bit of a delay before the mouse hover goes away or have an extra buffer around the tooltip.
I believe this happens because you moved your cursor around the item:

Definitely not a great experience

Some more details in https://github.com/microsoft/vscode/issues/138291

My 2 ideas:

increase the delay for this particular case
do not hide the hover when the hover is visible and mouse moves into the neighbour status item

Please use: https://github.com/microsoft/vscode/issues/95816
Closing as duplicate.


does not return error code when passed file does not exist

Run the command on a file that does not exist: pyright foo.py
The program will print File or directory "foo.py" does not exist., however no exit code is set (echo $? returns 0).

Sorry for the delayed response, I think this has been fixed. I'm seeing non zero exit codes on the latest version.


[FLINK-31109][yarn] Support Hadoop proxy user when delegation token f…

…etch is disabled

What is the purpose of the change
FLINK-28330 removed old delegation token framework code as part of it removed the existing support for delegation tokens that are managed outside of Flink
Brief change log

As part of YarnClusterDescriptor#setTokensFor load the available delegation tokens in the client machine and then load the tokens obtained through DelegationTokenManager if security.delegation.tokens.enabled is set to true.
HadoopModule should throw exception if the UserGroupInformation.currentUser is a hadoop proxy user and also security.delegation.tokens.enabled is set to true.

Verifying this change

Added tests in the HadoopModuleTest and in KerberosLoginProviderITCase.
Manually tested in our env where delegation token fetch is managed.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Please review. cc @MartijnVisser @gaborgsomogyi @becketqin

Looking into the test failure, somehow it worked locally but failing in the pipeline.

e2e tests are still running so most probably it's going to pass.

@gaborgsomogyi So all there's left are the minor review comments and then it's ready to merge from your perspective?

@MartijnVisser when then mentioned suggestions are fixed then it can go. Planning to track the changes and approve it when I think it's flawless.

@MartijnVisser everything is green so good to go from.my side.

@venkata91 Can you also create a back port towards release-1.17?

I've tried specifying proxyUser using HADOOP_PROXY_USER and maintaining the delegation tokens with an external service and sending them into jobmanager, but I can't seem to update delegation tokens of taskmanager.
Flink seems to use DefaultDelegationTokenManager#listener to update the delegation tokens of taskmanager. Can we provide a custom DelegationTokenManager?
@venkata91 @gaborgsomogyi Could you please take a look?

I've tried specifying proxyUser using HADOOP_PROXY_USER and maintaining the delegation tokens with an external service and sending them into jobmanager, but I can't seem to update delegation tokens of taskmanager.

Not sure what you mean here. If you update the tokens externally then it's the external code responsibility ti do its job properly, right?

Flink seems to use DefaultDelegationTokenManager#listener to update the delegation tokens of taskmanager. Can we provide a custom DelegationTokenManager?

In your own distribution you can do anything you can imagine. There is no configuration infra for this so you need to change the source (namely DefaultDelegationTokenManagerFactory).
In general, what is the use-case what you would like to achieve here? If I understand correctly the goal is to have 2 user in a single cluster (proxy user + the other one who is creating tokens)?

@gaborgsomogyi Thank you for your reply.
I want to implement impersonation mode for flink engine in Apahce Kyuubi. Flink proxy user mode relies on external services to maintain delegation tokens, and kyuubi server can do this well.
Kyuubi server uses super user proxy to sessionUser to obtain delegation tokens, and brings them to flink jobmanger when starting the flink engine. It will regularly update delegation tokens and send them to jobmanager, and at the same time I want to update taskmanager tokens (but I can't find a suitable way).
Related issues:
https://github.com/apache/kyuubi/issues/6367
https://github.com/apache/kyuubi/issues/6368

In your own distribution you can do anything you can imagine. There is no configuration infra for this so you need to change the source (namely DefaultDelegationTokenManagerFactory).

Is this improvement reasonable? Can we introduce it upstream?

Just to give some historical background Flink (just like Spark) can't really guarantee that an externally given proxy user TGT will be renewed. This may end-up in stopped streaming workloads which will be bad UX for users. As a conclusion we decided to say plz manage it externally. If some internal Flink constructs can be re-used to do that then it's fine but that was not an explicit aim.

Is this improvement reasonable? Can we introduce it upstream?

Adding a config which manager implementation should be used is no brainer but the default manager has some requirements in its constructor: https://github.com/apache/flink/blob/5f5722fe916bafc00ec05539d6c19ac28ac1c5c4/flink-runtime/src/main/java/org/apache/flink/runtime/security/token/DefaultDelegationTokenManager.java#L101-L105
When we would say that the manager can be changed that would mean this constructor param set must be a user facing API which I wouldn't do. These however are subject to change.
As a conclusion I can imagine the following:

update the UserGroupInformation directly (maybe with side car) and turn off Flink delegation token manager
create a custom Flink distro where custom manager is included (I think this can't be really an option for you)

Flink and Spark session cluster scalable multi-user handling is something which can be enhanced but it's pending.

update the UserGroupInformation directly (maybe with side car) and turn off Flink delegation token manager

Yes, I want to update UserGroupInformation directly, but this update is difficult to pass to taskmanagers since we turned off delegation token manager.  If there are any exposed interfaces, that would be great.
https://github.com/apache/flink/blob/5f5722fe916bafc00ec05539d6c19ac28ac1c5c4/flink-runtime/src/main/java/org/apache/flink/runtime/resourcemanager/ResourceManager.java#L1578-L1597

The framework is so flexible that one can do anything with it 🙂
One thing which just came into my mind:

Remove HadoopModuleFactory from security.module.factory.classes
Turn off all existing delegation token providers security.delegation.token.provider.X.enabled=false
Create your own DelegationTokenProvider + DelegationTokenReceiver implementation(s)
Implement any custom logic what you can imagine there
Add them to flink/libs as external jar
Enjoy

The framework is so flexible that one can do anything with it 🙂 One thing which just came into my mind:

Remove HadoopModuleFactory from security.module.factory.classes
Turn off all existing delegation token providers security.delegation.token.provider.X.enabled=false
Create your own DelegationTokenProvider + DelegationTokenReceiver implementation(s)
Implement any custom logic what you can imagine there
Add them to flink/libs as external jar
Enjoy

Thanks @gaborgsomogyi , this is an effective way, I've tried it in kyuubi and it mostly works well. Just one point, since we disabled security.delegation.token.provider.hadoopfs.enabled, hdfs token will not be fetched and sent to yarn am container.
https://github.com/apache/flink/blob/4b342da6d149113dde821b370a136beef3430fff/flink-yarn/src/main/java/org/apache/flink/yarn/YarnClusterDescriptor.java#L1374-L1378
Shall we use DelegationTokenReceiverRepository here to receive tokens obtained by DelegationTokenManager?


Do you support multiple GPUs to run pipeline speculative decoding?

Hi, I am very interested in your work on PipeInfer!
However, the current implementation does not seem to support multiple GPUs. Are there any upcoming plans or suggestions for integrating support for GPUs with pipeline speculative decoding?
I have experimented with various approaches, but so far, none of them can work for me.
Thanks a lot!

In order to use GPUs, the code needs to be updated to the backend v2 system in upstream llama.cpp. That porting effort was started and is available here:
https://github.com/AutonomicPerfectionist/llama.cpp/tree/mpi-gpu
On that branch, pipeinfer is implemented in its own pipeinfer example rather than replacing speculative. I have managed to perform tests with Nvidia GPUs using it, however I would not consider it finished, and it still lags behind the current upstream llama.cpp.

Can you support tensor parallelism within a gpu node, or just leveraging pipeline parallelism?

Yes, tensor parallelism within a GPU node is supported. Essentially, the MPI backend wraps whatever other backends are being used, providing communication between nodes but otherwise leaving computation to the wrapped backends. This allows for PipeInfer to transparently support GPUs of multiple vendors, as well as any additional features the wrapped backends support. For example, the CUDA backend supports tensor parallelism and its own pipeline parallelism, so both are automatically supported by the MPI backend and, by extension, PipeInfer. Bare in mind, however, that the CUDA backend's pipeline parallelism is not visible to PipeInfer because of this transparent, backend-agnostic design. If one wants to use PipeInfer with multiple GPUs in a pipeline-parallel configuration, multiple MPI processes would need to be ran, one for each GPU, and each GPU would need to be dedicated to exactly one of these processes. This limitation could probably be fixed in the future, with additional updates to the MPI backend.

Thanks! But how can I run Pipeinfer on multiple GPUs within a node with tensor parallelism? It seems that the readme in the repo is just for CPU run.

The GPU support is present on a different repo for now, due to reproducibility requirements and to maintain the git history of this repo. I should be able to merge the support into this repo in a few weeks. For now, you can find the updated implementation here:
https://github.com/AutonomicPerfectionist/llama.cpp/tree/mpi-gpu/examples/pipeinfer

Thanks! But I wonder which parameter is for tp degree configuration?

Passing the --help option to either PipeInfer or the regular main binary will output a list of available options. I believe the tensor parallelism is on by default, and how tensors are split between GPUs is controlled by the --tensor-split option.


Add example of metadata.json file that is referenced by metadata2gha

Generating Github Actions outputs shows output of parsing a metadata.json file but the puppet_metadata projects doesn't include a metatadata.json file that was used to generate said output.
It'd be nice to have a metadata.json or metadata.json.EXAMPLE included in the project to be able to reproduce the output provided in the README.md.

When I wrote the documentation I thought the structure would be way more important than the actual data. So I showed how it's generated and then made the output readable.
Since then the output has changed because of GitHub internals. Yesterday I was thinking about writing a description of what it would do. So describe the operating systems it found and whether it will (or won't run tests on them). Perhaps that's more useful than this.

When I wrote the documentation I thought the structure would be way more important than the actual data. So I showed how it's generated and then made the output readable.
Since then the output has changed because of GitHub internals. Yesterday I was thinking about writing a description of what it would do. So describe the operating systems it found and whether it will (or won't run tests on them). Perhaps that's more useful than this.

It would be most useful if the documentation described what OSes will and won't run.  I was surprised to find out that
bundle exec metadata2gha --use-fqdn
did not output anything for Fedora when I added the following to my metadata.json
    {
      "operatingsystem": "Fedora",
      "operatingsystemrelease": [
        "38"
      ]
    },

Based on #106, I'm learning that some OSes don't run but again, as an amateur, it's been a difficult experience trying to learn the 'whys' and 'why nots' of how this entire process behaves.

I was surprised to find out that
bundle exec metadata2gha --use-fqdn
did not output anything for Fedora when I added the following to my metadata.json

That surprises me, because the whole idea is that you pass in a metadata and it parses that. That's the whole concept of the gem: abstractions on top of your metadata.json. The metadata2gha command extracts useful information to generate variables to run GitHub Actions.
What was your expectation that it was operating on?

Here's the metadata.json that I used.  It's highly likely that I'm doing something wrong...hence the/my need for an example metadata.json file to be added.

What was your expectation that it was operating on?

My completely amateur/noob expectation was that any value for $operatingsystem would be considered valid and that value be parsed and output for each $operatingsystemrelease value as well.  Obviously now, I was incorrect in that assumption.  Somewhere (and possibly still related to #106) there is code that filters out Fedora and other 'non-supported' OSes.  I've not found documentation yet that gives  a list of supported OSes.  Again, probably this is in the code but the info should be documented somewhere.


[🐛 BUG]: !BADKEY error on service name with dot (.)

No duplicates 🥲.

[X] I have searched for a similar issue in our bug tracker and didn't find any solutions.

What happened?
if service name contains dot (.) then rr fail.
I didn't find any mention about this in docs
Version (rr --version)
version: 2023.3.12, buildtime: 2024-02-29T18:24:06+0000
How to reproduce the issue?
# rr.yaml
service:
  "boom.":
    command: "echo 'hello'"
    exec_timeout: "10s"
    remain_after_exit: true
    process_num: 1

Relevant log output
[INFO] RoadRunner server started; version: 2023.3.12, buildtime: 2024-02-29T18:24:06+0000
[INFO] sdnotify: not notified
{"time":"2024-03-28T16:01:28.179604595Z","level":"ERROR","msg":"plugin returned an error from the Serve method","!BADKEY":"exec: no command","plugin":"*service.Plugin"}
error: exec: no command
plugin: *service.Plugin
  my-app exited with code 1

Hey @gam6itko 👋
You're using binary build for different architecture. E.g.: musl-linux binary with glibc env.

As for the dots: they're not allowed in yaml, not in the RR. boom. parsed by yaml like boom. or boom.command.<echo>.
BTW: docs nowadays are here: https://docs.roadrunner.dev

Docs are here for that: https://docs.roadrunner.dev/general/config

same as here https://github.com/roadrunner-server/roadrunner/issues/1529

Yep 😃


⚠ Bofrost.fr has degraded performance

In c25bd80, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1211 ms

Resolved: Bofrost.fr performance has improved in 34a007e after 8 minutes.


Awkward description / naming of Tasks

The setup-connections-wizards role has some misleading names of tasks, so
https://github.com/HCL-TECH-SOFTWARE/connections-automation/blob/497ef1d1c9dfebbf3c96b280d42d00118846e756/roles/hcl/connections-wizards/tasks/setup_connections_wizards.yml#L91 and https://github.com/HCL-TECH-SOFTWARE/connections-automation/blob/497ef1d1c9dfebbf3c96b280d42d00118846e756/roles/hcl/connections-wizards/tasks/setup_connections_wizards.yml#L61
are named "cleanup" -> these tasks drop the database, yes that's cleanup too, but dropping means deleting and not cleaning.
Run migrations for everything but ICEC https://github.com/HCL-TECH-SOFTWARE/connections-automation/blob/497ef1d1c9dfebbf3c96b280d42d00118846e756/roles/hcl/connections-wizards/tasks/setup_connections_wizards.yml#L146
There are several tasks mentioning migration, but they just create the database. Migration would be 6.5 -> 7.0 but not createDB or appGrants.
As I haven't dig into every role, I didn't create a PR for this, but consider a renaming in future versions please.
There are more and more inexperienced users of Ansible and Connections setup, so this can cause major problems when databases are dropped. Mainly because the example variables which are provided have all variables set for database "cleanup".
Regards
Christoph

Noted. Thanks for the observations. It will be fixed in the next push.

Mostly fixed


Can't exit permanently

Hi.
First: thanks for building the plugin, it is very useful.
I am compiling the demo (unchanged) using Phonegap Build. I add <plugin name="cordova-plugin-kiosk" spec="https://github.com/honza889/cordova-plugin-kiosk.git" source="git" /> in the config.xml.
Everything is working well, except the KioskPlugin.exitKiosk() part. It show the "Select destination.." popup, when I tap another launcher, it switch to that launcher, BUT when I tap the android home button, it show the kiosk plugin demo again instead of the launcher that I just select.
Is there any solution for this?
Thanks a lot for your help. I really appreciate it...

Sorry, it would require to change the launcher from app and it is impossible to do from non-system app.
(and system app would not be possible to install to non-rooted device + it would be much more complicated to install)

OK. Thanks a lot!


从4.7.2使用sync更新到4.7.3报错

描述遇到的问题
从4.7.2使用sync更新到4.7.3报错，不能merge
是不是都要手动调啊？
相应配置
NotionNext
截图

环境

操作系统: [windows]
浏览器 [例如. chrome, firefox]
版本 []

e，我怎么没有这个问题？
你就备份以下你的config之类的个人设置，然后Discard changes再重新调整吧。
我不是开发者。

When you are working with Git and trying to merge changes from an upstream repository into your own, you may encounter conflicts. These conflicts happen when both repositories have made changes to the same lines of code or files. Here’s a step-by-step guide to merging changes from the upstream repo and resolving conflicts:
Steps to Merge and Resolve Conflicts
1. Add the Upstream Repository
If you haven’t already added the upstream repository, you need to do this. The upstream repository is the original repository you are syncing your work with.
git remote add upstream https://github.com/user/repo.git

Replace https://github.com/user/repo.git with the actual URL of the upstream repository.
Check your remotes with:
git remote -v

2. Fetch the Latest Changes from the Upstream
Fetch the latest updates from the upstream repository. This will not modify your working files, it just brings in the latest commits from the upstream repo.
git fetch upstream

3. Merge the Upstream Branch into Your Branch
After fetching the upstream changes, you need to merge them into your local branch (typically main or master, but could be another branch if you're working on one).
First, ensure you are on the branch you want to merge into:
git checkout main

Then merge the upstream branch into your current branch:
git merge upstream/main

If there are no conflicts, the merge will complete automatically. However, if there are conflicts, Git will stop and notify you which files have conflicts.
4. Resolve the Conflicts
Now, you will need to manually resolve the conflicts. Git will mark the conflicting parts in the affected files. It will look like this:
<<<<<<< HEAD
Your changes
=======
Upstream changes
>>>>>>> upstream/main

<<<<<<< HEAD contains the code in your current branch (your changes).
======= separates your changes from the upstream changes.
>>>>>>> upstream/main contains the changes from the upstream repository.

You need to manually decide which parts of the code to keep. You can:

Keep your changes.
Keep the upstream changes.
Combine both sets of changes.

Once you’ve edited the file to resolve the conflicts, save the file.
5. Stage the Resolved Files
After resolving the conflicts in the files, you need to mark them as resolved by staging them:
git add <filename>

Or you can add all files:
git add .

6. Complete the Merge
After staging the resolved files, complete the merge by committing the changes:
git commit

Git will use a default merge commit message, but you can modify it if necessary.
7. Push Your Changes
Now that the conflicts are resolved and merged, you can push your changes back to your GitHub repository:
git push origin main

Summary of Commands

Add the upstream remote (if not added already):git remote add upstream https://github.com/user/repo.git

Fetch the upstream changes:git fetch upstream

Merge upstream changes into your branch:git merge upstream/main

Resolve conflicts, if any, by editing files.
Stage resolved files:git add <filename>

Commit the merge:git commit

Push changes to your repository:git push origin main

描述遇到的问题 从4.7.2使用sync更新到4.7.3报错，不能merge 是不是都要手动调啊？
相应配置 NotionNext
截图
环境

操作系统: [windows]
浏览器 [例如. chrome, firefox]
版本 []

请参考我这篇文章，可以快速进行修复，不懂就在评论下面问
https://www.linyunlink.top/article/post-240629


Live share disconnect on manual address navigation

Product and Version [VS/VSCode]: VS and VSCode
OS Version [macOS/Windows]: Windows 10 x64
Live Share Extension Version:  0.3.262.16695
Target Platform or Language : ASP.NET Core
Steps to Reproduce / Scenario:
1.Sharing live share connection
2.Connection works fine
3. navigating to page using browser address bar manually
4. i get an error message

Live Share
You were disconnected from the collaborating session due to a connectivity problem or the owner going offline. Join again to retry.

joining again and same issue happening

Hi @AhmedVolcano,
This sounds like #496, which was fixed by an update yesterday. Please update to the latest release and try again.


Query about Persistent Use of Masking during Inference and Finetuning in MAE

Hello,
I have been reviewing the MAE implementation, and I noticed that during both inference and finetuning, the model continues to use the same mask ratio (0.75) as was used during pretraining. Could you clarify why the model does not encode the entire image instead of using a masking approach? I am curious about the advantages or the rationale behind continuing with this masking strategy post-pretraining.
Thank you for your insights!

Hi, in finetuning, the encoder is reloaded to ViT_Classifier where the masking is discarded.

Thank you for your reply!
Is this way same with the original MAE?

Yes, that is the same with the original MAE, although their implementation may be slightly different.

Thanks a lot for your kindness.
However, I can not find where the masking function off.
Could you indicate where is it?

Here we use the encoder from MAE to build the ViT_Classifier. In the __init__ and forward here you can see anything except the masking is in use.

I really appreciate it !


luci-mod-network: add acs_exclude_dfs option

This option acs_exclude_dfs was added in the openwrt core package mac80211
with commit a5ec41b0e598f73cd1f878a107c480f472dfb15a.
This should also be configurable via LuCI.

This option is only valid when we are in the 5Ghz band. (done)
Also, the option should only be displayed if the channel selection is set to auto on the 5Ghz band. Unfortunately, I was not able to add a depends here. @jow I have already tried everything I could think of. But I couldn't do it because it's its own CBI form.Value. I don't understand what the _freq option looks like. (failed)

The frequency list in that rpc call might fail if you don't have the rpcd-mod-iwinfo & iwinfo & rpc installed.
You can obtain the data returned in that call using:
ubus call iwinfo freqlist '{ "device": "radio0" }'
get the list of devices from:
ubus call network.wireless status
A single channel from my results:
ubus call iwinfo freqlist '{ "device": "radio0" }'
{
	"results": [
		{
			"channel": 36,
			"mhz": 5180,
			"restricted": false,
			"active": false
		}
	]
}

@BashfulBladder Thank you for your advice.
But that is not what I am looking for. Or I do not understand your suggestion.
My problem is that the option acs_exclude_dfs is only valid when we are in the 5GHz range and the frequency selection option is set to auto.
The problem now is that the frequency and band selection is a separate cbi form where I can't access the selection directly what the user has done.
I can only use the virtual option _freq.

Ah.
Maybe something like o.depends(o.formvalue(radioNet.getWifiDeviceName(), "auto"));

o = ss.taboption('general', form.Flag, 'acs_exclude_dfs', _('Disable DFS'), _("Do not use DFS channels on auto selection"));
o.depends({'_freq': '11a', '!contains': true}); //this works for me; it was changed later (to what works for you apparently)
o.depends({'_freq': 'channel', '!contains': 'auto'});
o.default = o.disabled;

uci show wireless
wireless.radio0=wifi-device
wireless.radio0.type='mac80211'
wireless.radio0.hwmode='11a'
wireless.radio0.path='soc/1b500000.pci/pci0000:00/0000:00:00.0/0000:01:00.0'
wireless.radio0.country='US'
wireless.radio0.cell_density='0'
wireless.radio0.htmode='VHT80'
wireless.radio0.log_level='1'
wireless.radio0.channel='auto'
wireless.radio0.acs_exclude_dfs='1'

@BashfulBladder After a bit of back and forth, I think I've found a solution that works.
Can you verify whether it works for you too?
o = ss.taboption('general', form.Flag, 'acs_exclude_dfs', _('Disable DFS'), _("Do not use DFS channels on auto selection"));
o.depends({'_freq': '5g', '_freq': 'auto', '!contains': true});

If you know how to do it, it's not that difficult :joy:
I would update the P/R

While searching luci.js (trying to figure out why I can't disable a button after rendering), I stumbled upon  the dom (DOM) class with these 2 methods: findClassInstance & callClassMethod. It might be able to get the selection via dom.findClassInstance('channel') & then if dom.callClassMethod('channel','getValue') == 'auto'.
I'm just guessing - this could be useless info.
Also, I imagine you could set a breakpoint in your browser somewhere & inspect the values in the function(s).


Redirect to Success Page not work on Magento <IP_ADDRESS>

Am using the Paystack Inline Module <IP_ADDRESS> for my Magento which i installed via the Magento Connect but i am having problem redirecting to a success page after a successful transaction.
Am using
Magento <IP_ADDRESS>
PHP Version 5.6.30
Paystack_Inline | <IP_ADDRESS> (stable)

Any technical information of the error? @alan-qbasic

I guess none because i did the installation as requested. This is the screen it always redirect to after the transaction

Please open your error logs and Magento admin error messages to check

I got this from one of my logs
/magento1/index.php/admin/system_config/edit/section/payment/key/16abfb31df44681d96c5ca39104b33bf/ HTTP/1.1" 200 447569
::1 - - [14/Nov/2017:13:48:51 +0100] "POST /magento1/index.php/admin/system_config/save/section/payment/key/6c5f03e9069529f786482d09d25302a7/ HTTP/1.1" 302 -
::1 - - [14/Nov/2017:13:48:56 +0100] "POST /magento1/app/etc/local.xml HTTP/1.1" 403 235
::1 - - [14/Nov/2017:13:48:53 +0100] "GET /magento1/index.php/admin/system_config/edit/section/payment/key/16abfb31df44681d96c5ca39104b33bf/ HTTP/1.1" 200 447688
::1 - - [14/Nov/2017:13:49:04 +0100] "GET /magento1/paystack/payment/pop/ HTTP/1.1" 200 60221
::1 - - [14/Nov/2017:13:50:42 +0100] "GET /magento1/paystack/payment/response/?orderId=100000036&reference=100000036-2207-4acc-b258-71f279b1f264 HTTP/1.1" 302 -
::1 - - [14/Nov/2017:13:50:43 +0100] "GET /magento1/checkout/cart/ HTTP/1.1" 200 51118

Any error on Magento Admin?

None

Hey @alan-qbasic,
Is there a link we can test this out to see?
Can you elaborate on the steps you took to install the plugin?

You can check it here http://<IP_ADDRESS>/havilah/

I downloaded Paystack_Inline-<IP_ADDRESS>.tgz
I installed the plugin via Magento Connecto Manager and it was successful
I did my configuration with the Test public and secret key provided then i test run it

weird, without the actual error message, it's hard understanding what went wrong. Does your server have any error logs?

Hey @ibrahimlawal
I was going through the PaymentController.php in the plugin and i realize the page was redirecting to
Mage_Core_Controller_Varien_Action::_redirect('checkout/cart');
I then commented it out and redirect it to
Mage_Core_Controller_Varien_Action::_redirect('checkout/onepage/success');
Why is it executing if(!$success) even after the interface shows successful payment.

Awesome getting this far. Love that you can edit the code. Please try dumping the raw response from Paystack after we call the verify. That will tell us something about the error

ok boss

object(stdClass)#17 (1) { ["error"]=> string(73) "cURL said:SSL certificate problem: unable to get local issuer certificate" }
I think it has to do with the SSL

This may be of help: https://github.com/yabacon/paystack-php/wiki/cURL-error-60:-SSL-certificate-problem:-unable-to-get-local-issuer-certificate-(see-http:--curl.haxx.se-libcurl-c-libcurl-errors.html)

Thank you Boss!!

It's working now.
Thank you so much @ibrahimlawal and @steveamaza .

Awesome! Glad that's sorted. Thank you for choosing Paystack.


Filtered characters in fakedir refers to Windows systems only

When parsing a text file containing file and folder names, a number of characters get stripped from names which refers to forbidden characters in Windows systems:
https://github.com/earnestt1234/seedir/blob/300a22ddae5114e19335e2be99f9faecc7e7616d/seedir/fakedir.py#L1370
It would be nice to import platform and defines the filtered chars based on the current system (or through a parameter), i.e.:
import platform

if platform.system()=='Windows' bad_chars='/:?"*<>|'
if platform.system()=='Linux' bad_chars='/'
filtered = "".join([c for c in keyboard_chars if c not in bad_chars])

EDIT: The list of filtered characters can indeed be bypassed by providing start_chars, so this is not really an issue, sorry!

Thanks for raising this.  Even if there is a workaround with start_chars and/or name_chars, that is not entirely obvious.  I like your approach for making parsing more automatic across systems, though I wonder if having the algorithm dependent on the operating system could lead to some unexpected results for reproducing things.
I would like to rewrite/refactor this whole function TBH - it's very old code and I've refrained from touching it since it works for most typical cases I've tried.  But it's not very readable.
Next time I work on this, I will have a look at this function and keep this issue in mind.  Let me know if using start_chars/name_chars has ended up not being a good approach for you.


[STT] downloading of  deepspeech2offline_librispeech is too slow

I am on Ubuntu 20.04 platform. I live in Türkiye. I installed PaddleSpeech via pip.
When I want to test offline model with:
    paddlespeech asr --model deepspeech2offline_librispeech --lang en --input ./en.wav -v

it starts to download but it's extremely slow around 300KB.
Is it possible for me to download the speech model manually and extract it to PaddleSpeech's directory (I don't know where)?
Best regards.

You can find the pretrained models' url in https://github.com/PaddlePaddle/PaddleSpeech/blob/develop/paddlespeech/resource/pretrained_models.py
the model will be downloaded to ${HOME}/.paddlespeech/models, as for deepspeech2offline_librispeech, here is the file tree:

by the way, you must keep the *.tar.gz file for md5 checking

@yt605155624 Thank you for responding and the info you gave in detail. I downloaded /asr0_deepspeech2_offline_librispeech_ckpt_1.0.1.model.tar.gz
file and extracted it to the directory as you instructed. But as my surprize, it needs another file, around 9GB, common_crawl_00.prune01111.trie.klm
Assuming that those files will be loaded into RAM, it's too much for my 8GB personal computer.
Thank you for all your help. I'm grateful to you.
All the best.


[SPARK-45417][PYTHON] Make InheritableThread inherit the active session

What changes were proposed in this pull request?
Make InheritableThread and inheritable_thread_target inherit the active session.
Why are the changes needed?
Sometimes the active session can be null in Python threads, even with InheritableThread or inheritable_thread_target.
Example:
# repro.py
# run as: bin/spark-submit repro.py
from multiprocessing.pool import ThreadPool
from pyspark import inheritable_thread_target
from pyspark.sql import SparkSession

spark = SparkSession.builder.appName("Test").getOrCreate()
spark.sparkContext.setLogLevel("ERROR")

def f(i, spark):
    print(f"{i} spark = {spark}")
    print(f"{i} active session = {SparkSession.getActiveSession()}")
    print(f"{i} local property foo = {spark.sparkContext.getLocalProperty('foo')}")
    spark = SparkSession.builder.appName("Test").getOrCreate()
    print(f"{i} spark = {spark}")
    print(f"{i} active session = {SparkSession.getActiveSession()}")

pool = ThreadPool(4)
spark.sparkContext.setLocalProperty("foo", "bar")
pool.starmap(inheritable_thread_target(f), [(i, spark) for i in range(4)])

Does this PR introduce any user-facing change?
Yes. When using InheritableThread or inheritable_thread_target, the active session will be copied from the parent thread.
How was this patch tested?
python/run-tests --testnames pyspark.tests.test_pin_thread
Was this patch authored or co-authored using generative AI tooling?
No

Thanks for taking a look. Mind opening a PR against master branch? so we can backport the change to branch-3.5.

Opened a PR against master here: https://github.com/apache/spark/pull/43231


Fix newlines not being properly escaped

Closes #28 https://github.com/ruby-formatter/rufo-vim/issues/28

Pretty sure this fixes it. I tested it on a file and it seems to work fine.


Twitter automation

It will check your internet speed and if it's lower than what they requested. it will tweet the internet provider about that

@viralRupani Go ahead


Use jq's -r switch instead of sed

Use jq's -r switch instead of sed to remove double-quotes.

@zokradonh btw. when I click on the "dockercloud" CI I only get an error 500. Not quite sure what it adds over Travis as well.

dockercloud CI sets the hub.docker.com image description, thats the reason I have this automated build still on. Normal docker push does not set it.
Though, since some month it's possible to set it externally by this:
https://stackoverflow.com/a/55082686/1431178
Maybe we should implement this and disable dockercloud CI completely.

Ah, that is neat. The last time I looked for this the internet still said that this is not possible. Do you want to work on this or shall I give it a try?
The linked github repo also has some other interesting script for deleting old tags, etc. (kopano_base is quite polluted with tags).


Update cogs-examples.ipynb COG Note

Apply @kylebarron suggestion to clarify a note around how "cloud-optimized" enables partial file reads.

Something went wrong with editing the notebook raw text. Going to start over in fresh branch.


How do I change auto-assiged class vp-center to vp-top?

Be sure to search for your issue before opening a new one.
Current Behavior

As I scale down my browser's screen size, a huge amount of padding is added to the top of the video player area. Upon inspection, this can be resolved favorably-enough by changing the  to . If that padding can be reduced, that would be more ideal, otherwise I simply want to set that class!
Expected Behavior
I don't want so much vertical on smaller screens.
Steps to Reproduce
This is a NextJS project.

npm install react-player
import ReactPlayer from 'react-player';
Within a React Functional Component, return( whatYouSeeInTheJSfiddle )

Environment

URL attempting to play: https://ecosystem-omega.vercel.app
Browser: Chrome
Operating system: MacOS Big Sur 11.6.4
jsFiddle example: https://jsfiddle.net/5xgvz2Lc/

Other Information

Going through the same issue, really need to assign vp-top. Any way we can attach css through ReactPlayer

sorry this is out of the hands of ReactPlayer because vp-center is a part of the Vimeo player embed and it's in an iframe which can't be manipulated by outside scripts.


Update fr-CA.json

"Size"

We've decided not to add a size field to contracts, as that would only be useful for some listings. In the future, please submit all changes to a repo in a single PR, instead of one per file.


Run_me.ps1 is missing in the latest Collector zip folder

Hi Guys,
Run_me.ps1 file is missing in the latest collector zip folder file.

run_me.ps1 is a simple script used when you need to pass some attributes to the MPARR_Collector.ps1 script, to use and replace at the task in the task scheduler. I'll added shortly.

Updated, I added the run_me.ps1 script with the detailed explanation.

Really appreciate it.
Thank you very much.

From: Sebastián Zamorano A. @.>
Sent: Saturday, November 4, 2023 3:39 AM
To: microsoft/Microsoft-Purview-Advanced-Rich-Reports-MPARR-Collector @.>
Cc: Mohamed Hasan Adnan @.>; Author @.>
Subject: Re: [microsoft/Microsoft-Purview-Advanced-Rich-Reports-MPARR-Collector] Run_me.ps1 is missing in the latest Collector zip folder (Issue #17)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
Updated, I added the run_me.ps1 script with the detailed explanation.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/Microsoft-Purview-Advanced-Rich-Reports-MPARR-Collector/issues/17#issuecomment-1793067260, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BDVXUCMZJQFFAKQESUF2O7DYCVJBHAVCNFSM6AAAAAA632MRHCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJTGA3DOMRWGA.
You are receiving this because you authored the thread.

You can contact me through LinkedIn, to share your email address.
https://www.linkedin.com/in/profesorkaz/


Twitter Bridge 403 error . okay after deleting cache

I am in debug mode. I get a 403 error on calling a username on twitterbridge. After deleting everything in cache folder it works again.

Currently at commit 22a01f1093
Yup, this issue https://github.com/RSS-Bridge/rss-bridge/issues/1584 was closed but I'm still seeing 403 errors as well.
I had a cron job that deleted ../cache/TwitterBridge on the hour but had disabled it because I thought this issue had been resolved.  It has not.  Cron job is now re-enabled.
.

as an alternative to the cron I put
if(strpos ($this->getCacheFile(),"TwitterBridge") !== false) 			return $this;
at the very beginning of public function saveData($data) in caches/FileCache.php

It's because the bridge only expires the guest token based on number of uses and not by time.

Checked access.log on my https://feed.eugenemolotov.ru/
$ sudo cat /var/log/nginx/feed_access.log | grep Twitter | awk '{print $9}' | sort | uniq -c
  12973 200
    168 301
      4 304
   3832 400
      3 403
    291 404
  19046 429
      4 499

Using memcached as cache. I get mostly 429 errors, instead of 403.

@em92 how many requests per hour are you making to Twitter? The more requests more you make, the more often the guest token will be renewed.

$ sudo cat /var/log/nginx/feed_access.log | grep Twitter | grep "12/Jun/2020:15:"| awk '{print $9}' | sort | uniq -c
    706 200
     12 301
    365 400
     16 404
   1525 429
$ sudo cat /var/log/nginx/feed_access.log | grep Twitter | grep "12/Jun/2020:14:"| awk '{print $9}' | sort | uniq -c
    599 200
      9 301
    483 400
     12 404
   1562 429
$ sudo cat /var/log/nginx/feed_access.log | grep Twitter | grep "12/Jun/2020:13:"| awk '{print $9}' | sort | uniq -c
    710 200
     12 301
    145 400
     18 404
   1316 429

First result is between 15:00 and 15:59, second - 14:00 - 14:59, third - 13:00 - 13:59

Has anyone tried to add call to purgeCache in TwitterBridge? Just an idea, haven't tested it myself

I think 100 queries every two hours hour will be enough to expire the token in time. You are well over that.

Can someone test #1606 ?

Can someone test #1606 ?

I'm hitting a lot of 403 errors, I'm testing this now.

Can someone test #1606 ?

have been running this branch since this morning and am still seeing 403 returned intermittently

Deleting the cache for TwitterBridge did not resolve it in my case:
Twitter @Linuzifer was unable to receive or process the remote website's content!
Error message: `The requested resource cannot be found!
Please make sure your input parameters are correct!
cUrl error: (0)
PHP error: `
Query string: `action=display&bridge=Twitter&context=By+username&u=Linuzifer&format=Html`
Version: `git.master.22a01f1`

Does anyone have an idea what the problem is?

Does anyone have an idea what the problem is?

Works for me with #1606

Does anyone have an idea what the problem is?

Works for me with #1606

Thansk for the hint! It works now, but I also deleted the complete cache content. Only deleting the TwitterBridge cache did not work.

have been running this branch since this morning and am still seeing 403 returned intermittently
After making sure to remove all cache sub-dirs, this is no longer a problem for me with #1606.
Can this PR and #1607 get accepted?

Having this problem on both my new hosts at Scalingo and Heroku. Scalingo did work first time then broke, Heroku hasn't worked since I first set it up today.

Having this problem on both my new hosts at Scalingo and Heroku. Scalingo did work first time then broke, Heroku hasn't worked since I first set it up today.

Have you solved the problem?
There's still the same problem in Heroku.

I just use nitter to open twitter profiles and grab an rss link

I just use nitter to open twitter profiles and grab an rss link

Thank you. What should I do to not use nitter url?

Thank you. What should I do to not use nitter url?

If the problem is with the cache, maybe destroy and re-create the Heroku project?

Thank you. What should I do to not use nitter url?

If the problem is with the cache, maybe destroy and re-create the Heroku project?

The same problem occurs.


Consistent query version check

This diff does the following

Refactors query state machine to make terminal state more generic
Adds terminal state of failed to query
Combine client version checker and feature checker into a single checker interface
Add consistent query feature check
Fail queries which are responded from worker which does not support consistent query

Coverage decreased (-0.03%) to 66.679% when pulling 5e60f4d1806ab72c05094d505a60388fd531ce9c on andrewjdawson2016:ConQueryVersionCheck into 589424187ec374f736ef3ac0b2d8997575211c05 on uber:master.


LoadIQM: attempt to load texture from IQM at loadtime.

tries to load the texture with the base path of the original IQM file, relative paths should work.
LoadIQM currently does not load textures. this change attempts to resolve this.

admittedly this could be seen as a breaking change, as it SHOULD load textures but currently doesn't.
so folks have worked around this feature.


partially address https://github.com/basho/riak-nodejs-client/issues/78

fail with callback on parse response instead crash

Hi @mogadanez
I have taken your commits and made some minor formatting and other code changes, and have opened PR #81


Use napari logo as taskbar icon on windows when run as python script

Description
On windows use napari logo as taskbar icon when running from script. Fixes  #1168
Type of change

[x] Bug-fix (non-breaking change which fixes an issue) #1168

References
https://stackoverflow.com/questions/1551605/how-to-set-applications-taskbar-icon-in-windows-7/1552105#1552105
How has this been tested?
Run on windows 10 and icon is changed from default python logo
Final checklist:

[x] My PR is the minimum possible work for the desired functionality

@Czaki, by putting this in gui_qt it will not work if someone has started the qt loop outside of napari (such as by using ipython --gui qt or elsewhere) right? Was that intentional?

My intention is to not conflict with persons who would like to embed napari in their application. From issues which I read You suggest person to dock their widget instead of dock Viewer as widget in their window. So overriding this value on every Viewer start may break some usages.

This works for me when launching from command line!
But does not work when launching from ipython, if that's the intent.
(not work=python logo instead of napari)

Maybe this version?

@Czaki Now launching from the commandline nor from ipython show the napari logo.

Ok. This ctypes change need to be done before windows creation.
Last commit should work (I again login on windows machine for test)

Awesome @Czaki! Icon shows up when launching from commandline and from ipython now!

Test fails because of scikit-image update. See #1249

rebased after skimge fix

Thanks all for working to resolve my issue!


Increase in Columns

After running raw_to_Xy to take in multi indexed data, how do I amend the code in the getting started Python notebook such that it now takes into account the change in matrix size? Sorry, I am still trying to figure it out and is quite new to this

Hey @isaactyj
Do you think you could share a minimal reproducible example of your issue? Without leaking any private information - feel free to to create a minimal dataset and upload it if necessary.

The data I use is just from Yahoo Finance, so i can just upload the code to get the multi-index csv
stocks = ['AAPL', 'GOOGL', 'MSFT', 'AMZN', 'TSLA']

start_date = '2013-01-01'

end_date = '2019-01-01'

data = yf.download("AAPL", start=start_date, end=end_date)

data = data.reset_index()

columns_multi_index = pd.MultiIndex.from_product([stocks, ['Close', "Volume"]], names=['Stock', 'Metric'])

all_data = pd.DataFrame(columns=columns_multi_index, index=dates)

for i in stocks:
    print(i)
    data = yf.download(i, start=start_date, end=end_date)
    close = data['Close']
    volume = data['Volume']
    all_data[(i, 'Close')] = close.tolist()
    all_data[(i, 'Volume')] = volume.tolist()

n_timesteps = len(all_data)  # 20
n_channels = len(all_data.columns.levels[0])  # 2
n_assets = len(all_data.columns.levels[1])  # 2
lookback, gap, horizon = 5, 2, 4
n_samples =  n_timesteps - lookback - horizon - gap + 1  # 10
X, timestamps, y, asset_names, indicators = raw_to_Xy(all_data,
                                                      lookback=lookback,
                                                      gap=gap,
                                                      freq="B",
                                                      horizon=horizon,
                                                      use_log=True)
 assert X.shape == (n_samples, n_channels, lookback, n_assets)
 assert timestamps[0] == all_data.index[lookback]

I instantly get an assertion error for both assertions. I am actually using 18 stocks but for simplicity ill just change it to 5 stocks


Suppress the compiler warnings for unused parameters.

Summary
This PR removes the -Wno-unused-parameter compiler flag and adds a new UNUSED macro. This will aid in development as it will force the developer to explicitly set a parameter to UNUSED when it is not being used.

Nice addition, thanks for this!


Copy command documentation from TSTool

A number of TSTool commands have been added to StateDMI to provide useful functionality.  The code for these commands implement Help buttons in the command editors.  The following needs to be done:

Implement underlying code to view the Help documentation, similar to TSTool.  A separate issue needs to be added to the StateDMI main repo to address this.
Copy TSTool command documentation to StateDMI user documentation, similar to TSTool.  This involves adding to the command reference section in alphabetical order.  I will need to work with Justin to deploy to the web so that we can test the Help button links.
The StateDMI configuration file needs to contain the URLs for the location of the documentation, similar to TSTool.  TSTool has a system/TSTool.cfg file but StateDMI has not used such a file.  We need to add the file.  This can be done in the installation system folder or a .statedmi folder in user files.  Start with the former and we can discuss whether to take the latter approach similar to newer TSTool.

Justin pushed his changes to the master.  I uploaded to Amazon so that the documentation is visible as the online StateDMI User Documentation.  However, the software is now complaining when using the Help buttons on dialogs:
Warning[1]:  Unable to display documentation for group "command" and item "WriteTableToDelimitedFile" - no URL formatter defined.

Justin needs to add some additional code similar to TSTool to make the Help buttons work.  In TSTool, I have code that the Help button events call and formats a URL for display.  This is managed through an interface so it can be updated later, for example if the URLs are put into an input file rather than being formed in code.
The online documentation can now be used to take this to the next step.

Added code to update functionality of help button.
See the following commit in cdss-app-statedmi-main


Forward compatibility with Stream v1.0 and v0.7

Builds on top of #183

Updated to simplify buffering tests by using new react/promise-stream release :shipit:

@clue changes look good, but any idea why the lowest builds are failing?

@WyriHaximus Updated the first commit to target at least react/stream v0.4.6 which contains a bugfix for the ThroughStream that is now used in the test suite :shipit:

Cheers 👍


fix: add left rounded to first selected cell

Problem: First selected cell background is not rounded. Only in default style, new york is not affected
before:

after:

@shadcn please taka a look :)


Scripts in /usr/local/etc/rc.d/*.sh are no longer started on boot

As of 1.15.7, these scripts no longer appear to be executed on boot, though they still appear to be killed on shutdown.
I really wish there was a way to start/stop the standard rc scripts, but that's a whole other issue.

I think it has always been this way? Are you referring to a particular script?

There used to be a mechanism that would run scripts that end in .sh that are placed there, specifically because the rc system is intentionally broken in *sense, so that it's possible for users to run custom code/daemons.

I wouldn't call it broken. It just doesn't fit a dual use case as a contained product merged with a plain FreeBSD type server. We've worked hard on getting rid of custom hooks and execute-this-custom-blob type scripting that makes for tomorrows security nightmares. The new package system will support this in a cleaner way, you won't even have to deal with GUI if you don't want to and all the tools to build the package will be freely available so you won't have to push your code upstream (I reckon it is internal code). Does that sound acceptable to you?

We're discussing a couple of different things here.  The subject of this issue is that prior to 1.15.7, the existing mechanism inherited from pfSense of running addtional code on boot - placing a .sh file in /usr/local/etc/rc.d/ - worked, but from 1.15.7 of OPNsense, this does not work, and now there is no mechanism other than modifying files that will be overwritten by upgrades.
On the topic of whether 'breaking' (I don't think the term is incorrect - the standard boot process was hacked to disable it) the standard FreeBSD RC system was a good design decision, that's a more complex question.  If I was designing something like this from scratch, I'd try really hard to integrate with the existing system before I discarded it, because you get the benefit of having edge-cases solved once rather than multiple times, easy transferal of skills, and lower cost to add new features, at the expense of some maintenance overhead and a slight reduction in flexibility.  The way the FreeBSD rc system works via variables, it would have been pretty easy to integrate using some sourced config files generated from templates.  None of that is related to this particular issue though.

Yes, please help us design such a system. You will most likely end up with something that you'll be happy with. :)

So, I'm still not clear - was this functionality removed on purpose?

The location /usr/local/etc/rc.d/ is used by all the standard scripts included in all the packages, so enabling that might kick to much in at the moment. I agree it could be practical to include your own rc scripts (for more advanced users/system integrators). I will mark this one as feature request for the moment, as soon as we find the time to implement something we will pick it up. For our new components (using the new framework) we probably have to change this area on some degree anyhow.

I think there's still confusion here - this was a feature inherited from pfSense, and worked until 1.5.17, where it was broken.

Your probably right, there might be confusion over here. I was under the impression that the rc.d scripts weren't used by the custom rc scripting.  Where all the scripts started before? or only some? In my install there are quite some scripts in there placed by the standard packages which (probably) shouldn't be started by default.

Only scripts placed there, ending in .sh would be started.  None of the standard rc.d scripts end in .sh, so you're correct in thinking that they aren't started by the custom rc stuff.  Didn't seem like a particularly sensible way of doing things, but it was the only way of starting non-core stuff that I could find for pfSense.

I haven't found the old feature yet in the sources, but I do think we should replace it for something more sensible. Let's keep this issue open for now, I will look into it further.

Found it, the old option to start the scripts was placed inside the package code we removed (rc.start_packages).  I will keep this issue open and think of an alternative.

Thanks for following up @AdSchellevis

The next version will include a new rc.d hook into the base system, see https://github.com/opnsense/core/commit/f290b78c5b240df2e1a2d504a7b433ea83243d3b and some additional info in the forum https://forum.opnsense.org/index.php?topic=274.msg1020#msg1020
Is it ok to close this issue?

Looks like this ought to work for my use-case.  Will this also result stop being issued on shutdown?  Just thinking that some daemons may have custom safe shutdown handling in their rc scripts.

I kind of forgot and was just looking at it :-) I will make sure the stop is also being issued (or stays in place if it already is decent)

The normal shutdown sequence should kind of shutdown the daemons, but I rather make the behaviour explicit and make sure the normal stop operation is used. https://github.com/opnsense/core/commit/d6ae716445c1ef6d66a0297709035e586dad8106 contains the fix.
I'll close the issue.

It looks like this functionality has evaporated in the intervening years. Is there a new preferred way to start a script at boot? In my case I want to launch iperf3 daemons on several ports at boot.

Just open a proper bug report and avoid unfounded generalisation on tickets from years ago that fixed a particular bug. It is of no use to anyone.


Malformed .web_data.json resulting in: JSON.parse: expected property name or '}' at line 2 column 1 of the JSON data

Hi,
I have the following Dockerfile:
# Pull base image.
FROM jlesage/baseimage-gui:ubuntu-18.04-v4.0.3

RUN apt-get update && apt-get -y install libreoffice

# Copy the start script.
COPY startapp.sh /startapp.sh

# Set the name of the application.
RUN set-cont-env APP_NAME "libreoffice"

and the following startapp.sh:
#!/bin/sh
export HOME=/config

exec /usr/bin/libreoffice --calc

and when I open the URL to see the app I see:

I have docker exec'd into the container and see this:

I think the -ne strings should not be there.
Any idea what could the issue be?
Really liking the idea and docker image. Outstanding work.

Thanks for reporting, I'm fixing this.

Fixed in version 4.0.4.


Resolve use-after-free compiler warnings

This should resolve the following warnings, which I'm seeing on Fedora 37's gcc 12.2.1:
src/ros2/performance_test_fixture/test/benchmark/benchmark_malloc_realloc_no_memory_tools.cpp: In function ‘void benchmark_on_realloc(benchmark::State&)’:
src/ros2/performance_test_fixture/test/benchmark/benchmark_malloc_realloc_no_memory_tools.cpp:74:16: warning: pointer ‘ptr’ may be used after ‘void* realloc(void*, size_t)’ [-Wuse-after-free]
   74 |       std::free(ptr);
      |       ~~~~~~~~~^~~~~
src/ros2/performance_test_fixture/test/benchmark/benchmark_malloc_realloc_no_memory_tools.cpp:72:38: note: call to ‘void* realloc(void*, size_t)’ here
   72 |     void * realloc_ptr = std::realloc(ptr, realloc_size);
      |                          ~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~
src/ros2/performance_test_fixture/test/benchmark/benchmark_malloc_realloc.cpp: In member function ‘virtual void PerformanceTestFixture_benchmark_on_realloc_Benchmark::BenchmarkCase(benchmark::State&)’:
src/ros2/performance_test_fixture/test/benchmark/benchmark_malloc_realloc.cpp:111:16: warning: pointer ‘ptr’ may be used after ‘void* realloc(void*, size_t)’ [-Wuse-after-free]
  111 |       std::free(ptr);
      |       ~~~~~~~~~^~~~~
src/ros2/performance_test_fixture/test/benchmark/benchmark_malloc_realloc.cpp:109:38: note: call to ‘void* realloc(void*, size_t)’ here
  109 |     void * realloc_ptr = std::realloc(ptr, realloc_size);
      |                          ~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~

Linux
Linux-aarch64
Windows

That code snippet does not resolve the warning:
src/ros2/performance_test_fixture/test/benchmark/benchmark_malloc_realloc.cpp: In member function ‘virtual void PerformanceTestFixture_benchmark_on_realloc_Benchmark::BenchmarkCase(benchmark::State&)’:
src/ros2/performance_test_fixture/test/benchmark/benchmark_malloc_realloc.cpp:112:16: warning: pointer ‘ptr’ may be used after ‘void* realloc(void*, size_t)’ [-Wuse-after-free]
  112 |       std::free(ptr);
      |       ~~~~~~~~~^~~~~
src/ros2/performance_test_fixture/test/benchmark/benchmark_malloc_realloc.cpp:110:38: note: call to ‘void* realloc(void*, size_t)’ here
  110 |     void * realloc_ptr = std::realloc(ptr, realloc_size);
      |                          ~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~

I really don't care about stopping the test early - it's up to you.

I really don't care about stopping the test early - it's up to you.

I probably should have been more specific about my concern.  I also don't care that much about exiting the loop early, but I think the code both before and after this change is buggy.  Here is the current code:
  for (auto _ : state) {
    void * ptr = std::malloc(malloc_size);
    if (PERFORMANCE_TEST_FIXTURE_UNLIKELY(nullptr == ptr)) {
      state.SkipWithError("Malloc failed to malloc");
    }
    void * realloc_ptr = std::realloc(ptr, realloc_size);
    if (PERFORMANCE_TEST_FIXTURE_UNLIKELY(nullptr == realloc_ptr)) {
      std::free(ptr);
      state.SkipWithError("Malloc failed to realloc");
    }
    std::free(realloc_ptr);
    benchmark::DoNotOptimize(realloc_ptr);
    benchmark::ClobberMemory();
  }

If std::malloc here returns a nullptr, then we mark it with SkipWithError, but we don't exit the loop.  We then go on to call realloc on it, but that essentially maps to a malloc at that point.  This sequence is the same in the updated code.
I don't have a particularly strong feeling on how we solve it, but I do think we should fix this bug while we are in here.

I reviewed the code, and there are some important snips that we should address. You're right to call it out.
First, we're not supposed to call SkipWithError multiple times, and right now, that's totally possible.

REQUIRES: 'SkipWithMessage(...)' or 'SkipWithError(...)' has not been
called previously by the current thread.
Report the benchmark as resulting in an error with the specified 'msg'.
After this call the user may explicitly 'return' from the benchmark.

Second, I was under the impression that using the iterator-based solution largely acted the same as calling KeepRunning, but that's not the case. We do need to explicitly prevent future iterations:

If the ranged-for style of benchmark loop is used, the user must explicitly
break from the loop, otherwise all future iterations will be run.
If the 'KeepRunning()' loop is used the current thread will automatically
exit the loop at the end of the current iteration.

These have implications far beyond the goals of this PR, and we may even need to update benchmarking code outside of this package. With that, I'm rather opposed to trying to remedy the situation in this PR.

Alright, this isn't quite as bad as I'd thought. Calling SkipWithError a second time doesn't crash anything, it just ignores subsequent calls.
As for aborting the loop, I tested that more iterations do indeed progress after a call to SkipWithError, but calling it before the iterator is created seems to work okay and the loop doesn't start. So our calls to SkipWithError in setup and fixtures should be okay as long as they don't result in crashes.
Any time that SkipWithError is run, the currently running benchmark results are completely discarded, but other benchmark functions still execute normally.
So I'm back to being okay with the current code. We might waste a little time running unnecessary iterations, but that's pretty much the only downside I can see.

Going back to the original issue:

src/ros2/performance_test_fixture/test/benchmark/benchmark_malloc_realloc.cpp: In member function ‘virtual void PerformanceTestFixture_benchmark_on_realloc_Benchmark::BenchmarkCase(benchmark::State&)’:
src/ros2/performance_test_fixture/test/benchmark/benchmark_malloc_realloc.cpp:112:16: warning: pointer ‘ptr’ may be used after ‘void* realloc(void*, size_t)’ [-Wuse-after-free]
112 |       std::free(ptr);

I guess GCC is upset here because technically realloc could have invalidated the contents of ptr, even though we know that this won't happen if realloc returns NULL.  In that case, your fix here is the only one that will satisfy the newer version of GCC.
I'm still not super comfortable with essentially testing something that we don't intend in some cases, but I guess since we won't crash and the result is discarded, it can't cause any harm.  I'll go ahead and approve this, as it does seem like an improvement.


"attr_reader" which implements an abstract method but lacks "override" does not cause error

Input
→ View on sorbet.run
# typed: true
extend T::Sig

module Test
  extend T::Sig
  extend T::Helpers

  interface!

  sig { abstract.returns(String) }
  def foo; end
end

class Implementation
  extend T::Sig
  include Test

  def initialize
    @foo = T.let('Foo', String)
  end

  sig { returns(String) }
  attr_reader :foo
end

Observed output
No errors! Great job.

Expected behavior
I expected a error indicating that the reader sig was not declared with override. Similar to this:
editor.rb:23: Method Implementation#foo implements an abstract method Test#foo but is not declared with override. https://srb.help/5051
    23 |  def foo
          ^^^^^^^
    editor.rb:11: defined here
    11 |  def foo; end
          ^^^^^^^
Errors: 1

Duplicate of #1841


Examples showing #cloud-config files with stages.boot sections don't work

Kairos version:

PRETTY_NAME="Ubuntu 24.04 LTS"
NAME="Ubuntu"
VERSION_ID="24.04"
VERSION="24.04 LTS (Noble Numbat)"
VERSION_CODENAME=noble
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=noble
LOGO=ubuntu-logo
KAIROS_VERSION="v3.1.3-v1.31.0-k3s1"
KAIROS_PRETTY_NAME="kairos-standard-ubuntu-24.04 v3.1.3-v1.31.0-k3s1"
KAIROS_FLAVOR="ubuntu"
KAIROS_RELEASE="v3.1.3"
KAIROS_REGISTRY_AND_ORG="quay.io/kairos"
KAIROS_BUG_REPORT_URL="https://github.com/kairos-io/kairos/issues"
KAIROS_GITHUB_REPO="kairos-io/kairos"
KAIROS_NAME="kairos-standard-ubuntu-24.04"
KAIROS_VERSION_ID="v3.1.3-v1.31.0-k3s1"
KAIROS_IMAGE_LABEL="24.04-standard-amd64-generic-v3.1.3-k3sv1.31.0-k3s1"
KAIROS_ARTIFACT="kairos-ubuntu-24.04-standard-amd64-generic-v3.1.3-k3sv1.31.0+k3s1"
KAIROS_FLAVOR_RELEASE="24.04"
KAIROS_SOFTWARE_VERSION_PREFIX="k3s"
KAIROS_ID="kairos"
KAIROS_IMAGE_REPO="quay.io/kairos/ubuntu:24.04-standard-amd64-generic-v3.1.3-k3sv1.31.0-k3s1"
KAIROS_VARIANT="standard"
KAIROS_MODEL="generic"
KAIROS_TARGETARCH="amd64"
KAIROS_HOME_URL="https://github.com/kairos-io/kairos"
KAIROS_SOFTWARE_VERSION="v1.31.0+k3s1"
KAIROS_ID_LIKE="kairos-standard-ubuntu-24.04"
KAIROS_FAMILY="ubuntu"

CPU architecture, OS, and Version:

Linux beta 6.8.0-44-generic #44-Ubuntu SMP PREEMPT_DYNAMIC Tue Aug 13 13:35:26 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Describe the bug

After trying to use the examples that use PXE and cloud-init to provision a system, none of the steps in stages: boot: were being applied. If I rename the step to boot.after: it will work.
After a lot of digging through the code, it appears that Yip detects the "#cloud-config" line at the top of the file and transpiles from cloudInit style to yipYaml which tries to merge the stages: sections for boot. I'm not sure if this is a problem when the PXE install happens or after the installation occurs.
To Reproduce

Create cloud-config file with "#cloud-config" at the top, auto install indicated, and "stages: boot:" section that does something that can be observed (e.g., touch a file in an ephemeral, per-boot directory)
PXE boot client with config_url= set to that cloud-config

Expected behavior

Installation should occur, reboot, and then apply steps from "stages: boot:"
Logs

Additional context

When the steps are renamed to "boot.after" this appears to work.

umm, I tested this with the following cc:
#cloud-config

install:
    grub_options:
        extra_cmdline: rd.immucore.debug
    poweroff: false
    reboot: false
stages:
    boot:
        - files:
            - content: "test\n"
              group: 1000
              owner: 1000
              path: /run/bar
              permissions: 511
          name: BOOT STAGE TEST
    initramfs:
        - name: Set user and password
          users:
            kairos:
                passwd: kairos
        - hostname: kairos-{{ trunc 4 .Random }}

Installed, rebooted and I can see the file there:
kairos@kairos-xhwk:~$ stat /run/bar
  File: /run/bar
  Size: 25        	Blocks: 8          IO Block: 4096   regular file
Device: 0,26	Inode: 2335        Links: 1
Access: (0777/-rwxrwxrwx)  Uid: ( 1000/  ubuntu)   Gid: ( 1000/  ubuntu)
Access: 2024-10-02 10:11:38.036511722 +0000
Modify: 2024-10-02 10:11:38.036511722 +0000
Change: 2024-10-02 10:11:38.036511722 +0000
 Birth: 2024-10-02 10:11:38.036511722 +0000
kairos@kairos-xhwk:~$ cat /run/bar
test

And I can see the stage being run in the service:
Oct 02 10:11:38 kairos-xhwk kairos-agent[1695]: 2024-10-02T10:11:38Z INF Processing stage step ''. ( commands: 0, files: 0, ... )
Oct 02 10:11:38 kairos-xhwk kairos-agent[1695]: 2024-10-02T10:11:38Z INF Processing stage step 'BOOT STAGE TEST'. ( commands: 0, files: 1, ... )
Oct 02 10:11:38 kairos-xhwk kairos-agent[1695]: 2024-10-02T10:11:38Z INF Done executing stage 'boot'

And this is with an example from the examples docs you linked. The only things changed were to add a name to the stage so I could see it on the logs and setting the group/owner to 1000 so it picks up the default ubuntu user on my iso.
If you want us to solve this we will need:

full cc you used
logs from the boot stage service (journalctl -u cos-setup-boot)

Also can you confirm that this happens AFTER install, i.e. from an installed system? During installation (livecd/pxe) im not sure its possible for the boot service to read from the config_url as that is ingested after it has run, so it would be good to clear that the reproduction is as follow:

Cloud Config has some stages.boot actions (create file, set hostname, whatever)
Boot from IPXE with config_url
Install
Reboot to installed system
Boot service runs but the boot stages are not executed
Moving those to boot.after makes them run

Thanks!

I'm closing this since because it's stalled. Feel free to re-open if you have more information to help us reproduce.


Error: GetId operation: Invalid login token. Incorrect token audience.

Following this blog post I set up Cognito Identity Pool and role for Basic AuthFlow:
https://awsteele.com/blog/2023/10/25/aws-role-session-tags-for-github-actions.html
I also tried another workflow from catnekaise/cognito-idpool-basic-auth@alpha
The error I see is:

An error occurred (NotAuthorizedException) when calling the GetId operation: Invalid login token. Incorrect token audience.

Any idea what may be wrong. How can I troubleshoot it?

@pgasiorowski this action and the other has audience as an input parameter with default value cognito-identity.amazonaws.com.
If using this default audience the GitHub Actions identity provider in AWS IAM must be configured with this value as one of its audiences.
The screenshot below shows the GitHub Actions identity provider configured to enable using his action with its default audience value.

I will give it a try. Thank you sir!


Deploy info updating

Deploy info occasionally fails and gets into a 'stuck' state. There are two issues with this...
firstly, the fact that the deployinfo is stale is not surfaced anywhere in the application
secondly, this stuck state should be detected and dealt with by killing or restarting the spawned process appropriately

Closing as so old that if it was important, it would be fixed.


Sep 8 - Get to Know Rust: Fun, Fast, and Efficient Programming

Event Information

Category
Details

Reactor Topic
Coding, Languages, Frameworks

Monthly Topic (if applicable)
Skill Up September

Weekly Topic (if applicable)
Rust

Stream Topic

Stream Date
Sep 8, 2020

Stream Time
4PM PT

Stream Duration
60 minutes

Title
Get to Know Rust: Fun, Fast, and Efficient Programming

Description
Rust is a programming language that empowers developers to build reliable and efficient software. It is fun to write, easy to read, and high-performing. In this session, we’ll explore the fundamentals of the Rust programming language and uncover common programming concepts and syntax. We will then put Rust into action as we build a simple command-line program.

Speaker
@meaghanlewis

Moderator

SharePoint ID
7960

Additional Resources:

Event Checklist
10 Weeks Prior

[X] Reactor Content Team (RCT) or Speaker opens a new issue for an upcoming stream filling in the Event Information table. Both RCT and Speaker are marked as assignees.
[X] RCT and Speaker discuss details of the event in the comments of this issue. RCT adds a confirmed-event label and global-twitch-event tag to the issue and adds event to the Global Live Streams project. Issue will appear in the Upcoming Streams column
[X] Speaker adds a ready-for-marketing-edit label so the RCT can review title and description

8 Weeks Prior

[ ] RCT does a review of title and description and removes the ready-for-marketing-edit label
[ ] RCT moves the issue to the Ready for SharePoint column
[ ] RCT informs the Reactor Marketing Team of the new event information
[ ] RCT provides speaker with assets (e.g. slides, overlays) and the survey data for the event
[ ] RCT moves the issue to the MONTH Streams Prep column (according to the month it will be streamed in)

2 Weeks Prior (Done in /microsoft/Reactors)

[ ] Speaker opens a PR with the new content for the stream on the customer-facing Reactor repository and adds the content-development and global-twitch-event tag to the PR.
[ ] Speaker adds to the same PR a modification to the Streaming README to include their stream information.
[ ] Speaker adds the RCT assigned to the issue as the reviewer when it is ready for review/merge
[ ] RCT reviews, approves, and merges the changes

1 Week Prior

[ ] Speaker and Moderator test setup for streaming (instructions coming soon)
[ ] RCT checks in with the speaker and moves the issue to the Ready to Stream column

During and After Stream

[ ] Speaker and Moderator run the stream
[ ] RCT submits recording to be uploaded to the Reactor YouTube and to be made available on LearnTV
[ ] RCT moves issue to the Stream Complete column

@meaghanlewis
Please review the slightly edited abstract below:
Get to Know Rust: Fun, Fast, Efficient Programming
Rust is a programming language that empowers developers to build reliable and efficient software. It is fun to write, easy to read, and high performing. In this session, we’ll explore Rust programming language fundamentals and uncover common programming concepts and syntax. We will then put Rust into action as we build a simple command-line program.

Thank you!

@meaghanlewis
Approved then? :)

Yep.

@mabaumb
Abstract final and move to Ready for Sharepoint.

Uploaded into SharePoint (6/23)

@KimberlyBolton1 here is the revised abstract for this stream:
Rust is a programming language that empowers developers to build reliable and efficient software. It is fun to write, easy to read, and high performing. If you have been curious about the Rust programming language, or if you are simply looking for a memory-safe, fast, general-purpose way to program your next project, Rust may be a great option for you.
Join Program Manager, Meaghan Lewis, and Software Engineer, Arlie Davis, to discuss the benefits of using Rust and how it is being used at Microsoft today. We’ll then explore some fundamentals of Rust concepts and syntax through a simple command-line application.
Please feel free to edit and update the description up top. 😄

@meaghanlewis @mebaumb
Looks good. Just did some minor grammatical edits. Moved to Ready for Sharepoint.

quick fix to paragraph issue in description.

This change has been communicated to the team.

Event has been published on youtube! Closing Issue!


Investigate performance

In release mode performance is really good and we can maintain 60 fps. We still want to ensure if we could improve it by avoiding too many object creations.

With #31 performance are fine, let reopen this issue if we face performance issues


diff & score calculator

copy from https://github.com/simpspector/simpspector/pull/98
fixed #12

Is this ready for review and merging or still WIP?

This is still WP, sorry^^ but a review is welcome. :wink:

btw i wanted add a simple scoring system but it escalated quickly :D

This is quite a big PR. For the future I would strongly suggest keeping these smaller. But this is important stuff that we need to get on the road, so ...
I would really like to use aliases on the use statements for the two distinct Result objects, that would have saved me a lot of confusion while reviewing this code
use SimpSpector\Analyser\Result as AnalyserResult;
use SimpSpector\Gadget\Result as GadgetResult;

Still I would like to have a quick walk through today before evaluating the changes any further.

will be moved to inspector

only score calculator will be moved to inspector.

rebase & remove scoring

update description

ready to merge

:+1:
no time to review - we can refactor everything on the hackday anyway :smile:


GSoC2024 DL Starter Problem Solution

Add my solution and writeup to DL starter problem.

This was a starter problem for GSoC2024 and not meant to be merged.


[Conference] PyCon PL (Poland)

Conference Name
PyCon PL
URL
https://pl.pycon.org/2022/en/agenda/
Conference Dates
3-6 November 2022
Conference Type
in-person
Conference Location
Gliwice, Poland
CFP URL
https://pl.pycon.org/2022/en/agenda/
CFP Dates
2022-09-02
Conference Description
PyCon PL is a Polish Python conference - one of many PyCons happening around the world. It aims to integrate developers, designers and managers interested in using Python. Such meeting makes a great opportunity to meet new people and exchange experiences and ideas. The conference program should suit as well Python professionals as people just beginning their Python adventure. Everyone should find something of their own interest. The program consists of both official talks, semi-official panel discussions, as well as social meetings and recreation elements. The organization of PyCon PL is lead by members of Polish Python community.

Thank you for adding this event! \n We'll happily consider it for the next episode. \n\n The Issue autocloses but we'll be reviewing it at https://github.com/users/kjaymiller/projects/4/views/3


feat: make images to build sources on start

Fixes: #102

I guess I'll wait with using this branch?

I guess I'll wait with using this branch?

Now should be good to test ! 👍

I've now tried this ... still unfortunately no response is returned from the model .. "just" hammering the threads allowed on the CPU's on the VM on where the Local-AI Pod is running.
Dang!

ok cool thanks @LarsBingBong ! - let's see if this fixes other issues related to missing instructions cpu set and collect more feedback. However I think your issue might be either related to the environment - can you open an issue with logs/info on the environment?


CVE 5.0 converter

We should implement a CVE 5.0 converter to/from OSV.

One question: there are some things missing from OSV that the CVE JSON format has, and vice versa, what are the plans for that?
One possible answer: you can embed the entire CVE JSON in the database_specific field so data is not lost and then add support later since the data is there.

One question: there are some things missing from OSV that the CVE JSON format has, and vice versa, what are the plans for that?
One possible answer: you can embed the entire CVE JSON in the database_specific field so data is not lost and then add support later since the data is there.

Yep indeed! database_specific is what this is for. CVE JSON also has a lot of stuff that I'm not sure we'd want to include in OSV -- we want OSV to remain simple and easily understandable for open source. A lot of the extra things in CVE come from the fact that they need to describe vulnerabilities in closed source as well.

Stashing https://github.com/ossf/osv-schema/issues/109#issuecomment-1486078258 here to keep things together.

Related: https://github.com/CVEProject/cve-schema/issues/241

Drive by comment:

One possible answer: you can embed the entire CVE JSON in the database_specific field so data is not lost and then add support later since the data is there.

This seems like an unnecessary duplication:

the source of truth record in the CVEList should remain around ~forever
the source of truth record could evolve independently, and should be consulted for any future repeat conversion attempts


Pivot table implemented

implemented pivot table using library. Note to @mahtaia  pls fix the css, its working right now but I just included it in the index.html which is so horrible lol

addressed


Cant find incubator-mxnet/example/ssd/

I was curious if there is a new place to find https://github.com/apache/incubator-mxnet/blob/master/example/ssd?

Hi @haylie-murray , the ssd example has been removed in the PR #18525, since it uses mx.module.*, which is deprecated in MXNet 2.0.
It can be found in the v1.x branch: https://github.com/apache/incubator-mxnet/tree/v1.x/example/ssd


Release upgraded guava dependencies because of vulnerabilities

Hi Google-APIs-Team,
I'm not sure if this project is still actively maintained but it would be great to release your latest commits esp. regarding the guava-upgrade from 20.0 to 26.0-android (see commit: https://github.com/googleapis/api-common-java/commit/5fab976c83592ed57b768991a772d61394d50ff)
I couldn't find a newer release than 1.7.0 on maven-central: https://search.maven.org/search?q=g:com.google.api AND a:api-common&core=gav
which is in fact already quite old (August 2018).
Background: Our vulnerability scanners don't like guava <= 24.0 because of https://nvd.nist.gov/vuln/detail/CVE-2018-10237
Cheers
Thomas

Hello, we've just released v1.8.0 to maven, which points to guava's 26-android. Thanks for filing!


HOW TO USE AUTO AUG POLICES？

Hi @rhythm-A,
You may add AutoAugment in the train pipeline,
train_pipeline = [
    dict(type='LoadImageFromFile'),
    dict(type='AutoAugment', policies=policies),
    dict(type='RandomResizedCrop', size=224),
    dict(type='RandomFlip', flip_prob=0.5, direction='horizontal'),
    dict(type='Normalize', **img_norm_cfg),
    dict(type='ImageToTensor', keys=['img']),
    dict(type='ToTensor', keys=['gt_label']),
    dict(type='Collect', keys=['img', 'gt_label'])
]

and specify the policies in the format of list[list[dict]], such as:
policies = [
    [
        dict(type='Posterize', bits=4, prob=0.4),
        dict(type='Rotate', angle=30., prob=0.6)
    ],
    [
        dict(type='Solarize', thr=256 / 9 * 4, prob=0.6),
        dict(type='AutoContrast', prob=0.5)
    ]]

Unfortunately we currently do not support searching the policy yet.

@LXXXXR Could you please provide the config of AutoAug applied to ImageNet? Thanks.

Hi @d-li14 ,
We will release the config with ViT training config, which shall also be in this month release version.

Hi @LXXXXR
I notice most recent ViT variants are trained with RandAug, following the training recipe of DeiT. So which AutoAug policy will you use? Is it exactly the one provided in the original paper https://arxiv.org/abs/1805.09501? Thanks.

Hi @d-li14,
Sorry for the late reply.
We are still experimenting on the training recipe. We currently use the policy from original paper. Unfortunately the code was not released with the paper, so we follow the widely adopted third party implementation: https://github.com/DeepVoltaire/AutoAugment and it seems to give the performance an increase of 0.98%. The only difference is that we use cv2 as backend instead of pillow.

Hi @LXXXXR
Thanks, I see it. It is also interesting to see that the ViT's accuracy is clearly improved with AutoAug instead of the officially suggested RandAug.

The question looks solved.


Analyzer Feedback from IntelliJ

Version information

IDEA AI-201.87<IP_ADDRESS>99119
2.12.2
AI-201.87<IP_ADDRESS>99119, JRE 1.8.0_242-release-1644-b01x64 JetBrains s.r.o, OS Windows 10(amd64) v10.0 , screens 1536x864

@zain098 , closing this issue there is not enough information for us to take action. Next time please make sure that the logs are attached.


Setup a search-only "lite" version of the JS client

I'm working on a JS single-page app where we're going to be embedding many of our vendor files.  I'd prefer to not embed the entire REST client, as we'll be using search-only keys in a read-only context.  Would there be any way we could subscribe to a JS "lite" version of the client that you guys support to keep our bundle size down?

Hi @acorncom, that's feasible. It depends on the size reduction you are looking at.
Do you know how much size you would want to shrink?
We have ways to reduce the filesize to smg like 8kb for a search only client

@vvs 8kb seems quite reasonable. Just would rather not be including code to handle creating indexes when I know I will never use it on the client :-)

@vvs the reduction in size might also help with the resistance you guys seem to be getting from the Bootstrap folks ... :-)

Yes but we are a bit relunctant to build, document and maintain an 8kb js client.
Is 12kb filesize reduction on a 1,5mb website worth it?
Will it make a difference on the page load time? On a low connection network yes, but not if your webpage weights 1.5mb because people will left the wenaite before we start downloading 8kb client.
Do you see what I mean? Can you provide us some numbers (current webpage size) or maybe your website url?
I am still ok for doing the light version, only want to really understand the use case

Just would rather not be including code to handle creating indexes when I know I will never use it on the client :-)

Depending on the other libraries you are using (only algolia?) there is a good chance you have a lot of unused code in your webpage. Not just algolia.

Depending on the other libraries you are using (only algolia?) there is a good chance you have a lot of unused code in your webpage. Not just algolia.

Agreed.

Yes but we are a bit relunctant to build, document and maintain an 8kb js client.

Understood on that.  What I had envisioned when looking at your code was in essence a trimmed down "search-only" parent class with the main library inheriting from it.  But understood about the reluctance to take on the long-term maintenance :-)

Is 12kb filesize reduction on a 1,5mb website worth it?

For you to make something specifically for me wouldn't make sense, I'd agree (and because I'm using an Ember app, the payload does indeed get quite large).  I'd wondered if there would be a desire / demand for it from other folks (where I might be the first to voice the thought, but not be the primary target audience).
That make sense?  Happy to have you choose either way.

Understood on that. What I had envisioned when looking at your code was in essence a trimmed down "search-only" parent class with the main library inheriting from it

We may be able to do so yes and provide the library users a search only requireable file (and build).
This is a good idea but not top priority.
If you wanna help us: GO!
It would be great, the codebase should be straightforward.
I am available for any question regarding the code base and organisation of the future build.

This is a good idea but not top priority.

Understood on it not being top priority.  Re: helping on it, I may have time, but don't plan on my doing it :-)

As we are using lodash and sharing it amongst our projects I must say that we will maybe never do this unless we drop lodash at some point.
You could always shim the necessary lodash requires with your own utilities, that would work if you really need light size


🐞 Renaming a file/folder uses the wrong background color (blue)

Description
When a file/folder is renamed in the navigator and a character is deleted or inserted the background color becomes blue.
To Reproduce

Open a workspace
Click on a file
Rename it using the Enter key or click
insert or delete a character
the background color should become blue.

Expected Behavior
The background should not become blue until the file has stopped renaming.
Version Information
CodeEdit: [e.g. 0.0.3-alpha.21]
macOS: [e.g. 13.2.1]
Screenshots
https://user-images.githubusercontent.com/128280019/231071434-eed682be-6263-40f4-8191-fa1e7bc6d46a.mov

Fixed in #1241


Install and configure strict TypeScript pre-commit checking

Currently, no strict TypeScript checker is applied to code in this repository. Incorporate a strict TypeScript checker to pick up things like let variables that could be upgraded to const, missing JSDoc comments on exported symbols, etc.

The addition of ESLint has helped, but we still need to configure a git pre-submit hook.


Flesh out self-wait-subscribe APIs

What was wrong?
I got in a hurry and didn't take into account that this needed to be sorted out.
The various APIs for waiting on subscriptions, checking subscriptions, waiting on connections and checking connections don't always behave in intuitive ways when dealing with a singular endpoint as if it was two endpoints which is a feature that we want to support.
How was it fixed?
I don't think there was one correct behavior here, at least not by default.  Before the revers connections change, you did have to manually connect an endpoint to itself for this to work.  This is now replaced by a call to EndpointAPI.enable_local_broadcast()
Once this has been enabled, the endpoint will treat itself exactly as if it was another connected remote.  This includes triggering all of the correct wait_ apis.
Comparing this to the previous state of affairs prior to reverse connections, I believe it is functionally equivalent. In previous code, if you simply replaced endpoint.connect_to_endpoint(config_for_self) with endpoint.enable_local_broadcast() I believe that the functionality would be identical.
Cute Animal Picture

This also means I'll be releasing lahja 0.14.0 with great shame.

@cburgdorf and @lithp I sent this to both of you but I'm fine with just one review, though I woudn't mind confirmation that this is an API you like or suggestions on something alternate.

@pipermerriam Taking your comment from discord here:

I do believe the API should be opt-in (it was beforehand as well)  Without this, endpoint.are_all_endpoints_subscribed_to(...) has really un-ergonomic and surprising behavior (that it won't return true until the local endpoint also subscribes which is rarely what the caller wants in a situation that spans processes and is truly a broadcast to consumers)

I think I would rather resolve that with an extra parameter exclude_own_endpoint that defaults to True.
So, while I'm all for updating namof the APIs and ergonomics, I'd like to hear an argument for why this functionality is wrong but the previous functionality was right (requring that you establish a self connection over an actual socket).
Well, requiring a connection to self was more like a band aid after the topology switched from centra-router-based to mesh-based. When it was central-router-based, subscribing to events on the same endpoint that created these events just worked out-of-the-box without any special API calls.

Closed in favor of #122