Build AI Agent

Features
Identifying boss status requests via Slack tool
The #Core Engine will pass any messages from the boss to the AI Agent.   The AI agent should determine if these are project status updates.    If they are project status updates then it should proceed to process them.   Otherwise it should ignore them.
Pulling current project statuses via Jira tool
The AI Agent should have access to the Jira tool  (#14) and be able to retrieve the current project status using that tool.
##Retrieving sample project status updates from RAG pipeline
The AI Agent should have access to the RAG tool  (#15 ) and be able to retrieve examples of previous project status updates using it.   The AI Agent will use these to customize its project status updates.
##Sending status updates to the boss via Slack tool
When the AI agent has created a customized message, it should then use the Slack Tool (#13) to send its final message back to the boss.

NIce work @znas-io !!   I'll share the presentation I almost have finalized :)


Where is IReadOnlySession

My apologies for posting a question here.  I didn't find a way to ask this kind of question.
I cloned the source yesterday and I cannot find any of the non-async session code.
Where should I be looking?
Thanks

You're probably looking at the develop branch which is where I'm working on the async only code and the move to .NET Core, have a look at the 6.3.x branch where the currently released code lives

That was it.
Thanks.
And thanks for a great library.


No Syntax Highlighting in nvim_treesitter

Hello, I installed this parser using this config in my init.lua file:
local configs = require "nvim-treesitter.configs"
local parsers = require "nvim-treesitter.parsers"

local parser_config = parsers.get_parser_configs()

parser_config.haxe = {
    install_info = {
        url = "https://github.com/vantreeseba/tree-sitter-haxe",
        files = {"src/parser.c"},
        branch = "main"
    },
    filetype = "haxe"
}

configs.setup {
    ensure_installed = { "haxe" },
    highlight = {
        enable = true
    }
}

and it installs just fine when running :TSInstallFromGrammar haxe. However, I don't get any syntax highlighted in .hx files (Their filetype is set to haxe), and running :checkhealth nvim_treesitter gives me this output:
health#nvim_treesitter#check
========================================================================
## Installation
  - OK: `tree-sitter` found  0.19.4 (parser generator, only needed for :TSInstallFromGrammar)
  - OK: `node` found v15.6.0 (only needed for :TSInstallFromGrammar)
  - OK: `git` executable found.
  - OK: `cc` executable found. Selected from { vim.NIL, "cc", "gcc", "clang", "cl" }
  - OK: Neovim was compiled with tree-sitter runtime ABI version 13 (required >=13). Parsers must be compatible with runtime ABI.

## Parser/Features H L F I
  - haxe           . . . .

 Legend: H[ighlight], L[ocals], F[olds], I[ndents]
         +) multiple parsers found, only one will be used
         x) errors found in the query, try to run :TSUpdate {lang}

and that looks fine to me. However the whole haxe file has no syntax highlighting.

Closing for now in favor of #6


Rebiuild Framework Samsung

Information

Apktool Version (2.1.1) -
Operating System (Windows 10 Pro 64bit) -
APK From? (ROM VZW-G900VVRU2DPD1-20160603114654) -
JRE/JDK Version (jre1.8.0_92/jdk1.8.0_92) -

Stacktrace/Logcat
http://pastebin.com/raw/CTV02T0C (too long)
Steps to Reproduce

apktool
The error says that the resource is not found, but if you search on RES folder is 4 matches.

http://prntscr.com/birpil (screenshot)
Frameworks
http://www.mediafire.com/download/ui0346z12ix8atf/framework-res.apk
http://www.mediafire.com/download/thrjuqbp175zmuv/twframework-res.apk
APK
http://www.mediafire.com/download/thrjuqbp175zmuv/twframework-res.apk
Questions to ask before submission

Have you tried apktool d, apktool b without changing anything? | YES
If you are trying to install a modified apk, did you resign it?
Are you using the latest apktool version? | YES

I duplicate the problem.

Confirmed still on 2.2.2, I think this is related to #1361

➜  Bug1278 apktool if framework-res.apk -t 1278
I: Framework installed to: /home/ibotpeaches/.local/share/apktool/framework/1-1278.apk
➜  Bug1278 apktool d twframework-res.apk -t 1278 -f
I: Using Apktool 2.2.2-b60db0-SNAPSHOT on twframework-res.apk
I: Loading resource table...
I: Decoding AndroidManifest.xml with resources...
I: Loading resource table from file: /home/ibotpeaches/.local/share/apktool/framework/1-1278.apk
I: Regular manifest package...
I: Decoding file-resources...
I: Decoding values */* XMLs...
I: Copying assets and libs...
I: Copying unknown files...
I: Copying original files...
➜  Bug1278 apktool b twframework-res
I: Using Apktool 2.2.2-b60db0-SNAPSHOT
W: Could not find sources
I: Checking whether resources has changed...
I: Building resources...
I: Building apk file...
I: Copying unknown files/dir...
➜  Bug1278

Confirmed fixed with changes from https://github.com/iBotPeaches/platform_frameworks_base/pull/2
Will be in 2.2.2


Faile to run amundsenmetadata service from docker-compose

Today, I have cloned main 31e7784
Expected Behavior

The service amundsenmetadata is stopped
Current Behavior

The log  amundsenmetadata show:
2021-05-20 09:48:15 +0000] [1] [INFO] Starting gunicorn 20.1.0
[2021-05-20 09:48:15 +0000] [1] [INFO] Listening at: http://<IP_ADDRESS>:5000 (1)
[2021-05-20 09:48:15 +0000] [1] [INFO] Using worker: sync
[2021-05-20 09:48:15 +0000] [9] [INFO] Booting worker with pid: 9
[2021-05-20 09:48:15 +0000] [10] [INFO] Booting worker with pid: 10
Using requested Flask module flaskoidc and class FlaskOIDC
[2021-05-20 09:48:19 +0000] [9] [ERROR] Exception in worker process
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/gunicorn/arbiter.py", line 589, in spawn_worker
    worker.init_process()
  File "/usr/local/lib/python3.7/site-packages/gunicorn/workers/base.py", line 134, in init_process
    self.load_wsgi()
  File "/usr/local/lib/python3.7/site-packages/gunicorn/workers/base.py", line 146, in load_wsgi
    self.wsgi = self.app.wsgi()
  File "/usr/local/lib/python3.7/site-packages/gunicorn/app/base.py", line 67, in wsgi
    self.callable = self.load()
  File "/usr/local/lib/python3.7/site-packages/gunicorn/app/wsgiapp.py", line 58, in load
    return self.load_wsgiapp()
  File "/usr/local/lib/python3.7/site-packages/gunicorn/app/wsgiapp.py", line 48, in load_wsgiapp
    return util.import_app(self.app_uri)
  File "/usr/local/lib/python3.7/site-packages/gunicorn/util.py", line 359, in import_app
    mod = importlib.import_module(module)
  File "/usr/local/lib/python3.7/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1006, in _gcd_import
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 967, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 677, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 728, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/usr/local/lib/python3.7/site-packages/metadata_service/metadata_wsgi.py", line 14, in <module>
    or 'metadata_service.config.LocalConfig')
  File "/usr/local/lib/python3.7/site-packages/metadata_service/__init__.py", line 79, in create_app
    app = class_obj(__name__, **flask_kwargs_dict)
  File "/usr/local/lib/python3.7/site-packages/flaskoidc/__init__.py", line 107, in __init__
    _session.app.session_interface.db.create_all()
  File "/usr/local/lib/python3.7/site-packages/flask_sqlalchemy/__init__.py", line 1039, in create_all
    self._execute_for_all_tables(app, bind, 'create_all')
  File "/usr/local/lib/python3.7/site-packages/flask_sqlalchemy/__init__.py", line 1031, in _execute_for_all_tables
    op(bind=self.get_engine(app, bind), **extra)
  File "/usr/local/lib/python3.7/site-packages/flask_sqlalchemy/__init__.py", line 962, in get_engine
    return connector.get_engine()
  File "/usr/local/lib/python3.7/site-packages/flask_sqlalchemy/__init__.py", line 555, in get_engine
    options = self.get_options(sa_url, echo)
  File "/usr/local/lib/python3.7/site-packages/flask_sqlalchemy/__init__.py", line 570, in get_options
    self._sa.apply_driver_hacks(self._app, sa_url, options)
  File "/usr/local/lib/python3.7/site-packages/flask_sqlalchemy/__init__.py", line 914, in apply_driver_hacks
    sa_url.database = os.path.join(app.root_path, sa_url.database)
AttributeError: can't set attribute

Possible Solution

Steps to Reproduce

make oidc-image for frontend and metadata. make image for search
docker-compose -f docker-amundsen.yml up

Screenshots (if appropriate)
Context
Proof of concept to adopt Amundsen like ours principal tool metadata discovery

Your Environment

Amunsen version used: frontend  3.7.0  with OIDC, metadadata 3.5.0 with OIDC and search 2.5.1

Data warehouse stores:
Deployment (k8s or native):  native
Link to your fork or repository:

Hi,
I got this issue as well. Please add SQLAlchemy==1.3.23 in requirements.txt file of metadata service and then run docker-compose -f docker-amundsen.yml up --build. . Should resolve the issue.

@Milanmangar thanks for suggesting the change, do you know if it is due to SQLAlchemy broken change? maybe we should pin the dependency :) Would be great if you upstream the fix.

@feng-tao sure, will do it. :-)


Trigger to build new image

ref: https://github.com/open-cluster-management/backlog/issues/13081
Signed-off-by: clyang82<EMAIL_ADDRESS>

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Verification

/assign @songleo

/retest

/hold

/retest

/retest

/retest

/unhold


Fixing default title coming from configuration of aggregations

See itemsapi/itemsjs#78
A simple fix to take default title from the configuration if it exists, using humanize otherwise.

fixed title defaulting from configuration of aggregations
changed the conjunctive search test

to redefine the test aggregation object with different cases of title (unset, different from id. equal to id)
to include asserts on the computed title of each bucket

Thanks Nicolas. I've published new version 2.1.2 into https://www.npmjs.com/package/itemsjs


Must NBIRTH sequence number always be '0'?

First I'd like to thank all contributors for this helpful library. Great job.
For me it is not quite clear, if the NBIRTH sequence number must always be '0' as it is currently implemented. The SparkplugB 3.0.0 documentation is ambiguous. In chapter 6.4.19 the sequence number "MUST have a value between 0 and 255". But in chapter 12.13. it is written "The NBIRTH MUST include a sequence number in the payload and it MUST have a value of 0."
The following theoretical example shows wrong data if the NBIRTH sequence number is always '0':
Let's assume, the 'bdseq' remains the same. The following messages are sent:
-> NBIRTH (seq=0)
-> NDATA (seq=1)
<- receiving NCMD "Node Control/Rebirth"
-> NBIRTH (seq=0)
Now let's assume the host application receives the 2nd NBIRTH message before the NDATA message. As a result, the host application would process the NDATA message as the sequence number is higher than the sequence number of the 2nd NBIRTH. This may result in wrong data. Unless the timestamp is taken into account as well.
Hence may question: what is the correct behavior of NBIRTH sequence number from your point of view? Thank you very much for any thoughts.

Also seems to contradict section 5.4 - NBIRTH must have a seq number that is between 0 and 255 (inclusive). The next paragraph says - This will become the starting sequence number which all following messages will include a sequence number that is one or more than the previous up to 255 where it wraps back to zero.

Yes. When looking at the sequence number only, the (old) NDATA would be correct. And if a new NDATA will be sent, it would have the sequence number '1'. So the application will ignore it as the 'old' NDATA with sequence '1' was processed.

I'm not sure either. Please follow https://github.com/eclipse/tahu/issues/373 as the team (hopefully) might be able to answer this finally...


Add pre-commit hook fix-byte-order-marker

https://github.com/pre-commit/pre-commit-hooks?tab=readme-ov-file#fix-byte-order-marker
Description
This PR...

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[X] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI
[ ] test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

How did you try to break this feature and the system with this change?

@blueorangutan package

@jbampton a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 11932

@blueorangutan test

@DaanHoogland a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests


LSTPerformanceWeb needs update to incorporate r_vertex plot

this PR added r_vertex plot: https://github.com/SegmentLinking/TrackLooper/pull/276
However the html has not been updated yet.
Some ... scripting needs to be done to add those in and deploy the copies of LSTPerformanceWeb on phi3/lnx7188/hpg.

Closed by https://github.com/SegmentLinking/LSTPerformanceWeb/pull/2


'Key' is not a member type of 'NSAttributedString'

XCode 9.4.1
iOS9 target
Swift 4.1

Closing due to inactivity and fix provided


Unable to mount shared folders

I ran:
dsh self-update
dsh update prerequisites
dsh update boot2docker

which, most notably, updated my VBox to 5.1.2, which I've now learned is an issue with Vagrant, as per this ticket #155
So I've downgraded my VBox to 5.0.20 (which it was at prior to the upgrade) however, I am seeing this error:

Vagrant was unable to mount VirtualBox shared folders. This is usually
because the filesystem "vboxsf" is not available. This filesystem is
made available via the VirtualBox Guest Additions and kernel module.
Please verify that these guest additions are properly installed in the
guest. This is not a bug in Vagrant and is usually caused by a faulty
Vagrant box. For context, the command attemped was:
set -e
mount -t vboxsf -o uid=id -u docker,gid=getent group docker | cut -d: -f3 .ssh /.ssh
mount -t vboxsf -o uid=id -u docker,gid=id -g docker .ssh /.ssh
The error output from the command was:
gid= requires an argument (i.e. gid==)
mount: mounting .ssh on /.ssh failed: Protocol error

shelling into the b2d box, I run
grep -i vbox

and see:

vboxsf                 40960  0
vboxguest             180224  2 vboxsf

Only a partial list of what should be running.
I'm unclear of what I can do next to get my drude environment operational again?

Hi @kyleskrinak
You are not the first reporting this issue. Looks like vagrant did something wrong in their last version.
Try this fix
https://github.com/mitchellh/vagrant/issues/3341#issuecomment-39015570
We are working to get rid of vagrant in version 2

FYI there is also another fix which does the same but with destroying existing VM altogether.
I would try it as a last resort http://www.venumurthy.com/2014/02/vagrant-error-failed-to-mount-folders.html

@achekulaev Thanks for the suggestions. I did try the vagrant vbguest plugin before I posted. However, I get this message:

The guest's platform ("tinycore") is currently not supported, will try generic Linux method...

which leads to

An error occurred during installation of VirtualBox Guest Additions 5.1.2. Some functionality may not work as intended.

I also previously tried vagrant ssh into the b2d machine to run this command:
sudo ln -s /opt/VBoxGuestAdditions-4.3.10/lib/VBoxGuestAdditions /usr/lib/VBoxGuestAdditions
which I presume should be
sudo ln -s /opt/VBoxGuestAdditions-5.1.2/lib/VBoxGuestAdditions /usr/lib/VBoxGuestAdditions
Or whichever version of VBox I'm running. However, the b2d machine only lists
bootlocal.sh
bootscript.sh
bootsync.sh
shutdown.sh
tcemirror

There's no such dir in /opt.
I'm using Drupal VM until I resolve this, but really miss drude.

Have you tried downgrading to VirtualBox 5.0.20 and vagrant 1.7.4?

I had only downgraded VB previously. Downgrading both did the trick. Thanks! Drupal VM is nice, but way overkill, storage-wise.

Great!
I will keep this issue open and update it once it's safe to update to the latest versions.
Going forward we'll get rid of Vagrant.

This is a duplicate of https://github.com/blinkreaction/drude/issues/155
Closing in favor of the issue in the upstream project:
https://github.com/blinkreaction/boot2docker-vagrant/issues/83
Let's move further discussions there.

This post me helped Link solve this problem.


Add label of input node instead of node id in tooltip

Fixes elyra-ai/elyra#2183.
Before changes:

After changes:

Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

Codecov Report

Merging #180 (5a082ed) into master (b248a01) will decrease coverage by 0.84%.
The diff coverage is 12.00%.

@@            Coverage Diff             @@
##           master     #180      +/-   ##
==========================================
- Coverage   73.42%   72.57%   -0.85%
==========================================
  Files          50       50
  Lines        1731     1754      +23
  Branches      410      425      +15
==========================================
+ Hits         1271     1273       +2
- Misses        361      376      +15
- Partials       99      105       +6

Impacted Files
Coverage Δ

...es/pipeline-editor/src/PipelineController/index.ts
71.96% <12.00%> (-6.74%)
:arrow_down:

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 b248a01...5a082ed. Read the comment docs.

As mentioned in a Webex with @marthacryan Friday this LGTM code-wise, but I still need to test locally once an elyra pr is opened that points to this PRs codesandbox

Is there perhaps a way to render this in a way that makes it clear that the displayed string is a reference to another node's output? Maybe some sort of static prefix text or an icon?

For reference:
In the properties view there's a bit more context because the user sees "input source", which helps the user to make the connection.


error detection for 'ok, fine'

'ok, fine' and 'Ok ok ok I ok' is detected as "TURKISH"

If you had read the documentation thoroughly, you would understand that there are not enough distinct ngrams in these phrases to calculate a reliable language estimate. Besides, the word ok is used in many languages so it is not a clear indicator for English.


Re-add accidentally removed profiler marker

This change restores a profiler marker that was accidentally removed during the merge of the development branch into the event_propagation branch.

/azp run


Support for MonetDB

I've extended Flyway to support migrations in MonetDB database (https://www.monetdb.org/).
Steps for creating test MonetDB cluster are described in its documentation.
I've provided bash script for creation test database: create-monetdb.sh.
Please, review my changes and incorporate them into official code. If there are any problems with my changes, please contact me.

Is this going to be added anytime soon?

Hi. I'd also like MonetDB support in Flyway. I have merged @rostrzycki's fork with the latest (as of today) flyway/flyway:master and have it in a monetdb branch. I'm unable to compile and test it because of an S3/Microsoft related error maven throws that I do not understand. I sent a pull request to @rostrzycki but could also send a new one to flyway.
If @rostrzycki cannot, I'd be happy to support this feature moving forward.

Closing old PR as this has zero chance of being merged at this point.


Expention raised when trying to update controls that are not displayed in the current url

I'm using Flet 7.4 and I can update controls that are not displayed on the current url/route. Once I updated to the latest version, an execption is raised when trying to update these controls.

That's right, page was not clearing when removing control from a page/view, thus causing confusion and unpredictable behaviours. What is your use case? Can you provide a code to repro?


[Meta Schedule][M3b] Database

This PR is part of the meta schedule project (#8473) that adds a workload registry, as well as
a generic database of tuning records. This feature is future-compatible with dynamic shape
auto-tuning work by @ZihengJiang and @ArmageddonKnight.
Depend on PR #9059.  The PR is ready for review now!
Co-authored-by: Xiyou Zhou<EMAIL_ADDRESS>Co-authored-by: Bohan Hou<EMAIL_ADDRESS>Co-authored-by: Ruihang Lai<EMAIL_ADDRESS>Co-authored-by: Hongyi Jin<EMAIL_ADDRESS>Co-authored-by: Wuwei Lin<EMAIL_ADDRESS>Co-authored-by: Siyuan Feng<EMAIL_ADDRESS>

CC @comaniac @mbrookhart @jroesch @ZihengJiang

Took a fast pass and had an offline discussion with @junrushao1994.  Two major points:

The APIs are bit confusing (e.g.,lookup_or_add and add). It might be better to use something like commit_workload and commit_record. Meanwhile, since we may add more database implementations in the future (e.g., sqlite), this general API can also make sure we won't need to refine it again in AutoTIR.
Better to remove workload registry accordingly to make the design simple.

Per discussion with @comaniac.
The naming change:

lookup_or_add seems pretty random, and we should rename it as commit_workload which is more consistent with database terminologies
add seems okay but commit_tuning_record sounds better and more consistent with commit_workload

Minor change in the code structure:

Consolidate WorkloadRegistry into JSONFile database, so that there is only one unified database API, and this could mimic the two-table database behavior

It seems the direct use of the Database is all wrapped in a JSON-related file object, which kind of confuses me, would you mind providing a brief explanation of the class structure somewhere?

Hey thanks @mbrookhart for asking! Let's focus on the database API and ignore the workload registry part which will be removed according to previous discussion with Cody in this thread.

The base abstract class is Database, which has 4 pure virtual methods: commit_workload, commit_tuning_record, get_top_k, size
The system interacts with any subclass of Database by calling these 4 methods
To implement a concrete subclass of Database: in pure C++, we can just inherit Database and do so; in pure python, we provide PyDatabase which plays some tricks so that everything can be implemented in pure python
JSONFile is a simple default implementation of Database for open source

The first three points and the last one make sense, but I think I'm a little confused by JSONFile being the default user-facing database. JSON means a very particular thing, you convert to/from it, but it doesn't have class methods or a user interface, per say. I would say all of the JSON functionality is important for conversion and serialization, but I'm not sure it makes sense for it to be the primary interface point?

@mbrookhart Yeah indeed JSONFile can be very confusing given JSON means very particular thing. Would love to hear about your proposals

@mbrookhart @comaniac I updated the patch according to your comments. Major changes:

Remove WorkloadRegistry, WorkloadToken and the annoying token_id_
Rename lookup_or_add => commit_workload
Rename add => commit_tuning_record
Various refactors to consolidate more stuff

Let me know what you guys think :-)

BTW, here is the convention we are using in TensorIR and meta schedule project:

An abstract base class that describe how the interface looks like, e.g. Schedule, Database, Builder, Runner
Concrete implementation as subclasses are hidden in src/ folder, and mostly hidden in cc files, which are not directly visible to developers. For example, PyDatabase, ConcreteSchedule
The abstract class provides factory methods to create these concrete implementations, whose name is consistent with the names of the subclasses. For example:

class Database : ... {
 public:
  static Database JSONFile(...); // TBD: Rename
  static Database SQLiteDB(...);
};

And then another problem here is what name should we change JSONFile for

Hey thanks guys for the discussion! Let's go with the name JSONDatabase then 😄

PR updated according to our discussion 👍  @mbrookhart @comaniac please take another look. Thanks guys for the discussion!

Thanks everybody for the review :-)


Cannot import databricks.koalas

I'm using python3.5, after installing koalas using pip i can't import it.
I get the error  ImportError: cannot import name 'pandas_udf'
Do I need to install Koalas in all worker nodes?

Seems like your PySpsrk version is too low. What's your PySpark version?

Closing this due to no feedback from the reporter.


latest releases missing tags

It would be ideal if the tags were kept up-to-date with the current release to ease examining the differences between releases and viewing old versions. As a note, npm version commits to package.json and pushes the correct tag automatically so this should be a rather trivial task when creating a new release.

My bad, forgot to git push --tags. Should be fixed now.


Add E2E tests to GitHub Actions

[ ] E2E tests
[ ] Unit tests
[ ] Sonar run https://github.com/SonarSource/sonarcloud-github-action

This logic will be merged with issue #263


users should be able to drag-n-drop files into the editor/file browser

[ ] dragging text or files into code editor should insert into editor
[ ] dragging text or files into file browser should create new file

this would also be solved by using a vscode distro

closed by 15b6a4649558cb940d36c2ff1004794ebddc839d


🐛 [Bug Report]: CMS V1.2.1 SMTP not work,config['sess_save_path'] error

BlizzCMS Version
1.2.1
Emulator
AzerothCore
PHP Version
PHP 8.0
Operating system
Linux
Web server
Apache
Database
MySQL 5.7
What happened?
1,SMTP 465  SSL   doesn't work ,can't send mail,no logs in application/log
2,config['sess_save_path'] error
How can the issue be reproduced?
1,Installing v1.2.1, an error of config['sess_save_path'] error occurred during the installation:   message:mkdir() :invalid path fliename:drives/session_flies_drive.php  line number:137
2, It must be changed to: FCPATH.'cache/sess_save_path'; to open the website
3,After setting up smtp and set the password recovery function on, SMTP 465  SSL   doesn't work ,can't send mail,no logs in application/log, and the email cannot be sent, no email be received
Screenshots
https://tc.evewow.com/i/2023/06/26/nx2smw.png

We are aware of this bug, we are working on fixing it. Sometimes this happens because the CMS is distributed for session by file and there are not enough permissions on that path.


Cas30ResponseView throws NPE if attribute value is NULL

Cas 5.0.4 Cas30ResponseView throws an NPE on line 125 if an attribute value is NULL.
Maybe this is still the case for the master branch, haven't checked, yet.
Have not found any documentation stating that attribute values must be non-null in any documentation.
In my AttributeDao implementation, I filter out null-valued attributes for now.

Thanks for the logging the issue report. Please file a pull request to address this case. Thanks very much!

Closed the unresolved issue? Seriously?

For me, this is a bug.


Context Menu: Doesn't work for shortcuts

Hello!
It seems that it doesn't work completely for shortcuts, it works perfectly with files and directories, but not with shortcuts.
It adds new items to context menu well, but when I click on some new added item it doesn't invoke command that should be performed.
I've made small investigation and noticed one thing (I hope, it will help you a little bit) :
when I click on new added item in the context menu in the function 'IContextMenu.InvokeCommand(IntPtr pici)'(SharpContextMenu.cs file) variable 'iciex' in fields 'verb' and 'verbW' has too big numbers like 236255072 or 286797496 and then it can not match this click to the appropriate command.
Thanks in advance.

I just wanted to comment that this problem still persists in V2.2: The new entry is visible in the contextmenu when right clicking a shortcut, but the Click event is not executed... My workaround to interacting with shortcuts is adding something to the send to menu.

Thank @Traubenfuchs I'll look into this

Hi there. After fiddling with the code it seems that the problem lies with SelectedItemPaths from ShellExtInitServer. When selecting multiple files, including a shortcut (.lnk file) and right clicking the shortcut file, SelectedItemPaths will contain ONLY the path referenced by the shortcut (e.g if you have a Foo.lnk on desktop and will reference c:\program files\Foo\Foo.exe the path retrieved will be c:\program files ...\Foo.exe). However, if any of the other files will be right clicked, SelectedItemPaths  will have all the files including the shortcut (.lnk) file.

Are there any improvements or workarounds found for this issue? Or are you guys planning for a solution on this problem?

All,
A full reproduction of the issue is:

Checkout branch d/ContextMenusForShortcuts.
Build in debug mode and run.
Register the 'DebugContextMenu' extension from the samples.
Open the explorer window from the Server Manager.
Navigate to the sharpshell\Tests\TestData\ContextMenus\Shortcuts folder.
Select both files.
Right click on the first file (not a shortcut) and see expected results (attached):
Do the same but right click on the second file (a shortcut) and an exception is thrown.

I'm looking into the fix now, but looking at:
http://microsoft.public.platformsdk.shell.narkive.com/K3aVjyHw/icontextmenu-with-a-shortcut-and-another-file-folder-selected

Hi All,
b6a0440 fixes the first issue (the exception) which gets us somewhere. However there is a separate issue which is more complex. Select two files, one a shortcut and one a normal file. Right click on the normal file and everything works as expected. Right click on the shortcut and you will only get the shortcut target in the selected item paths. This is the issue mentioned above.
However, for those who desperately need the exception resolved, the fix above works. Check it out and build manually if you need it urgently, otherwise I'll be releasing once I fix the related issue.

Hey, thanks for the framework,
This fix is still not in the nuget package, is it?

Hi, I'd like to refresh this issue a little. The problem actually is in the list provided by WinApi in IShellExtInit::Initialize method. There are workaround described at StackOverflow and other forums.
Although this is not the best solution it may be good enough fix if controlled by some kind of option/switch. Important thing is to register context menu in a proper way. I couldn't find description of "*" and "AllSystemObjectFiles" groups but it seems Windows treats it differently.

Closing this issue now, please re-open if needed.


Add --vm-uuid option to add command

Add --vm-uuid option to add command to specify uuid for specific vm without modifying config file directly. This makes #14 handy.

Add new option
Describe how to gather uuid for vm
Update README.md

Closed via #17


Missing CMAKE defines

Is it possible CMAKE_INSTALL_LIBDIR and CMAKE_INSTALL_INCLUDEDIR are not defined?

They are not currently defined anywhere within vcpkg that I'm aware of; if you're authoring a CMakeLists.txt I think you need to include(GNUInstallDirs) which will provide appropriate defaults. We explicitly designed the final layout to be maximally compatible with those defaults, so portfiles should never need to override them.
https://cmake.org/cmake/help/v3.0/module/GNUInstallDirs.html

Hmm. Bizarre.
I'm looking at PTex (http://github.com/wdas/ptex). The output I get is:

CMake Error at CMakeLists.txt:27 (install):
install FILES given no DESTINATION!

The CMakeLists.txt refers  to /vcpkg/buildtrees/ptex/src/ptex-2.1.28/src/ptex.
Even if I include(GNUInstallDirs) in the portfile.cmake. Any ideas?
CONTROL.txt
portfile.cmake.txt


find broken links to PDFs

TBD how we want those reported.

Broken PDF links appear to (indirectly) redirect to the homepage.
$ curl -I -L https://esd.ny.gov/broken.pdf
HTTP/1.1 301 Moved Permanently
Date: Thu, 17 Nov 2022 03:25:23 GMT
Content-Type: text/html; charset=iso-8859-1
Connection: keep-alive
X-Content-Type-Options: nosniff
Location: https://cdn.esd.ny.gov/broken.pdf
Cache-Control: max-age=1209600
Expires: Wed, 30 Nov 2022 21:56:05 GMT
X-Request-ID: v-77d9358a-65f9-11ed-826c-bb790a478789
Via: varnish
X-Cache: HIT
X-Cache-Hits: 3
CF-Cache-Status: HIT
Server: cloudflare
CF-RAY: 76b55bdfcada8c3b-EWR

HTTP/2 302
content-length: 141
content-type: text/html; charset=UTF-8
location: https://esd.ny.gov
server: Microsoft-IIS/10.0
x-powered-by: ASP.NET
x-frame-options: SAMEORIGIN
date: Thu, 17 Nov 2022 03:25:23 GMT

HTTP/1.1 200 OK
Date: Thu, 17 Nov 2022 03:25:24 GMT
Content-Type: text/html; charset=utf-8
Connection: keep-alive
X-Drupal-Cache: MISS
X-Frame-Options: SAMEORIGIN
Content-Language: en
X-Content-Type-Options: nosniff
Permissions-Policy: interest-cohort=()
X-Generator: Drupal 7 (http://drupal.org)
Link: <https://esd.ny.gov/>; rel="canonical",<https://esd.ny.gov/>; rel="shortlink"
Cache-Control: public, max-age=86400
Last-Modified: Wed, 16 Nov 2022 22:45:06 GMT
Expires: Sun, 19 Nov 1978 05:00:00 GMT
Vary: Cookie,Accept-Encoding
X-Request-ID: v-51288b6e-6600-11ed-b4a3-f7404e49308f
X-AH-Environment: prod
Via: varnish
X-Cache: MISS
CF-Cache-Status: HIT
Server: cloudflare
CF-RAY: 76b55be10df08c3b-EWR


plant organ development stage should not be part of shoot system development stage

Plant organ development stage (PO:0025339) currently has a part of relation to shoot system development stage (PO:0025527), but this is not correct, because root development stage is a type of plant organ development stage that is not part of a shoot system development stage.
You could say that phyllome development stage and and floral organ development stage are part of a shoot system development stage.
Reported by: rlwalls2008
Original Ticket: obo/plant-ontology-po-term-requests/582

Thai has been resolved.  Added a new relation:  phyllome development stage (PO:0025579) part_of shoot system development stage (PO:0025527). For more information on the phyllome development stage (PO:0025579), please see #548 .


Inheriting from ThreeJS passes

Hey, very nice library thanks for your work. Is there a way to inherit and use the passes that are built into the examples of ThreeJS - so for instance the UnrealBloomPass - I'm looking for a way to integrate it with your library? Thanks, T

This library is incompatible with the post processing examples from three due to API differences.
However, it's still possible to use the UnrealBloomPass mostly as-is.
See this comment for details. Updated example: https://jsfiddle.net/mabx57q1/
Note that UnrealBloomPass imports a bunch of other files behind the scenes<EMAIL_ADDRESS>

Nice one thanks - this was the key bit:
`
UnrealBloomPass.prototype = Object.assign(UnrealBloomPass.prototype, {
initialize() {},
originalRender: UnrealBloomPass.prototype.render,

render(renderer, inputBuffer, outputBuffer, delta, maskActive) {

	this.originalRender(renderer, outputBuffer, inputBuffer, delta, maskActive);

}

});`
This makes it possible to chain three passes with yours - perfect!
Thanks again.


Devise - current_user

Hi,
Trying to access the current_user as described in the docs in the the finder section:
 def self.records(options = {})
    context = options[:context]
    context[:current_user].posts
  end

For some reason though, context is empty of both the controller and the current_user.
Tried the solutions from #458 but I am still unable to access current_user.
Any ideas?

you can create custom ApiController
class ApiController < ApplicationController
  include JSONAPI::ActsAsResourceController
  def context
    { current_user: current_user }
  end
end

after extend your Api controller
class Api::PostsController < ApiController
end

now you can get current_user from context
it work nice for me.

Thanks @alexeyShumakov, I will try right away. For some reason I thought the context object was auto created by JR. Makes sense that it wouldn't be now that you shared this.
Thank you

Worked!


Need Help Browser support

Hello,
I need to implement streaming feature from client side ( angular ) to upload large video file ( = 200Mo ).
Without streaming, all video content must to be loaded before request was post to back end.
Otherwise I try to implement Streaming with FormData client side and Busboy ( NodeJS ) server side.
I Use Browserify to generate bundle of form_data.js biu when I import it, the Native FormData is already available .. and not its from form-data
   <script src="js/core/custom/form-data/lib/form_data.bundle.js"></script>....

The bundle generated from browserify, below :
form_data.bundle.js.zip
I notice, native FormData does not support Streaming from client side ( arraybuffer, readablestream ) , Do yo have alternative if I can't achieve my goal ?
Thanks in advance.

how do you get the video? a camera recording, a file from the disc?

Without streaming, the entire video content needs to be loaded before it can be sent to the backend, which can cause performance issues for large files. You mentioned that you are trying to implement streaming using FormData on the client-side and Busboy on the server-side.
You also mentioned that you are using Browserify to generate a bundle of form_data.js, but you are encountering issues because the native FormData is already available and not the one from form-data. It seems like the native FormData does not support streaming from the client-side, such as arraybuffer or readablestream.
There are alternative libraries that you can use to achieve your goal of streaming large video files from the client-side to the server-side. One popular library is called axios, which supports streaming using axios.post() with a stream option. Another library is fetch, which also supports streaming using fetch(). You can use these libraries in conjunction with a streaming solution on the server-side, such as Busboy.
Here's an example of how you can use axios to stream a large video file from the client-side to the server-side:
`
// Client-side code
const axios = require('axios');
const file = document.getElementById('file-input').files[0];
axios.post('/upload', file, {
headers: {
'Content-Type': file.type,
'Content-Length': file.size
},
maxContentLength: Infinity,
maxBodyLength: Infinity,
stream: true // enable streaming
}).then(response => {
console.log(response.data);
}).catch(error => {
console.error(error);
});
// Server-side code
const Busboy = require('busboy');
app.post('/upload', (req, res) => {
const busboy = new Busboy({ headers: req.headers });
busboy.on('file', (fieldname, file, filename, encoding, mimetype) => {
console.log(File ${fieldname} started streaming);
file.on('data', (data) => {
// Handle file data
});
file.on('end', () => {
console.log(File ${fieldname} finished streaming);
});
});
busboy.on('finish', () => {
console.log('Busboy finished parsing');
res.sendStatus(200);
});
req.pipe(busboy);
});
`
I using the axios.post() with the stream option to enable streaming, and Busboy on the server-side to handle the streaming. You can modify this code to suit your specific needs, such as adding error handling and progress tracking

😳 @AlexMaisiura
On the client side you are uploading a file with raw bytes.
you have not encoded the file in any way such as it's in a multipart/form-data (aka formdata)
So busboy will not be able to detect any such payload.
axios don't have any streaming functionality on the client side.
so setting stream: true is pointless? so is setting a content-length and a content-type. why may you ask? b/c when you send a file with either xhr or fetch then it will automatically append content-length + content-type for you when you upload a file or a blob. it dose this for everything that you send, URLSearchParams, FormData, Blob, File, and even strings, TypedArrays and ArrayBuffer unless you explicity need to set custom content-type yourself, which you don't do.
So all you really need is:
const file = document.getElementById('file-input').files[0];
axios.post('/upload', file).then(sucfn, errfn)

but i would not even use axios...
it's better to use fetch instead.
const [file] = document.getElementById('file-input').files
const result = await fetch('/upload', { method: 'POST', body: file }).then(res => res.arrayBuffer())

No dependencies needed...
then the hole req will be the file's stream. so no need for serializer such as busboy is even needed.
app.post('/upload', (req, res) => {
  req.pipe(dest)
})

oh... and fetch have streaming upload support now: https://docs.google.com/document/d/e/2PACX-1vQ26giyHPPhxrgVorNtkVSsJPOWecc6c6wHFLFbbl27r0BIBEgRwahh2b37Uk7HfXvZoW1Cu_ed-bRm/pub


Fix error in WIT group creation

Due to a copy'n'paste error I checked for the wrong entity to have a space template associated with it.
Thanks to @michaelkleinhenz for finding it.

Ike Plugins (test-keeper)
Thank you @kwk for this contribution!

It appears that no tests have been added or updated in this PR.
Automated tests give us confidence in shipping reliable software. Please add some as part of this change.
If you are an admin or the reviewer of this PR and you are sure that no test is needed then you can use the command /ok-without-tests as a comment to make the status green.
For more information please head over to official documentation. You can find there how to configure the plugin.

/ok-without-tests

Codecov Report

Merging #2117 into master will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2117      +/-   ##
==========================================
- Coverage   69.04%   68.99%   -0.06%
==========================================
  Files         162      162
  Lines       15113    15113
==========================================
- Hits        10435    10427       -8
- Misses       3730     3738       +8
  Partials      948      948

Impacted Files
Coverage Δ

remoteworkitem/jira.go
75% <0%> (-25%)
:arrow_down:

remoteworkitem/scheduler.go
53.65% <0%> (-7.32%)
:arrow_down:

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 c323d53...b0ea73c. Read the comment docs.


Pull request: feature enhancement

I've made some improvement on this project to add features:
Common:

Improve map UI and fix issues
When mouse hover on table cell, highlight corresponding version of object on map and pan to center
In table, show version which is invisible in red (removed)
Base map layer: OpenStreetMap and Google Satellite
Export full data of specific version in Level0L format

Node:

Show lat, lon in table
Show way(s) which the node belongs to

Way:

Show history versions of way on map

@iandees
I would like to contribute my change back to this project. But the changes are big, I don't know if all these are align with your plan on the project. The source code has been rewritten a lot with ES6, Promise and functional style. (It's supported by most of modern browsers.) So I hesitate to raise a PR. Would you like to have a look first?
Code and readme: https://github.com/aleung/osm-deep-history
Example:

Node
Way (The map needs a few seconds to load)

Your code looks really interesting. I think it'd be great to have it merged in, but I also want to be thoughtful of how much load this sort of thing applies on the OSM API server. I wonder if it would make sense to only do the intensive work of loading history after clicking a button or something.

I have concern on the number of API calls as well. I investigated OSM API v0.6 and Overpass API, not found an approach to retrieve full history with few API calls.
I've updated to make way history only loaded on click a button "Show More on Map".


Ionic2 not supporting the usage of multiple platform-href attributes in Index.html

Short description of the problem:
Ionic not allowing the multiple usage of the below tags in Index.html:

ios-href
md-href
wp-href

What behavior are you expecting?
I would like Ionic to allow the inclusion of multiple instances of stylesheets using the aforementioned tags. One could argue towards the scope of these tags as being a platform stylesheet however, if they're used for importing one common, global stylesheet. However, I want these stylesheets to be cached and therefore I require their direct inclusion in the Index.html. E.g. of required situation:
<link ios-href="./dist/page1.ios.css" rel="stylesheet">
<link md-href="./dist/page1.md.css" rel="stylesheet">
<link wp-href="./dist/page1.wp.css" rel="stylesheet">

<link ios-href="./dist/page2.ios.css" rel="stylesheet">
<link md-href="./dist/page2.md.css" rel="stylesheet">
<link wp-href="./dist/page2.wp.css" rel="stylesheet">

Steps to reproduce:
Not Relevant.
Code extracted from ionic-angular/config/bootstrap.js (lines 101 to 107):
    var modeLinkAttr = mode + '-href';
    var linkEle = document.head.querySelector('link[' + modeLinkAttr + ']');
    if (linkEle) {
        var href = linkEle.getAttribute(modeLinkAttr);
        linkEle.removeAttribute(modeLinkAttr);
        linkEle.href = href;
    }

Other information:
Suggestion: Ideally the code above would loop over the platform specific href attributes and would be able to handle more than one of these stylesheets in the Index.html.
Which Ionic Version?
Ionic2 - Beta 8

Hey @matmica, I don't think this is something that should be added since we have Sass to handle it. For example, if there are specific files that should only be added per mode, you can create them like this:
app/
  pages/
    page1/
      page1.ios.scss
      page1.md.scss
    ...

and then in the app.ios.scss file you would import the ios one:
// App iOS Only Sass
// --------------------------------------------------
// CSS that should only apply to the iOS app

@import "../pages/page1/page1.ios";

And in the app.md.scss file you can import the md one. We are doing this in the preview app:
https://github.com/driftyco/ionic-preview-app/blob/master/app/pages/cards/advanced-weather/styles.ios.scss
https://github.com/driftyco/ionic-preview-app/blob/master/app/theme/app.ios.scss#L36
Then the final app.ios.css stylesheet will include those pages. Does that solve your problem?

Hi Brandy,
Thanks for your suggestions, I do fact the above issues as well. As i'd like to have a more mobilized application. As I would have several "lobbies" which I don't want to combine and concatenate with your stylesheet. Reason being, is I want to do some sort of lazy loading and not request the "global/common" stuff for each lobby.

Hello @alan-agius4 ! Sorry if im misunderstanding your issue, but we only load our stylesheets once (when the app first loads). If you lazy loaded css for each part of your application it would actually be slower than just loading all the css up front. Whenever you load css the browser/webview has to block so if your lazyloading css as you need it, your app will constantly be blocking each time you navigate to that new part of your app. If you want to have separate styles for each page you can just do as brandy showed above. Hope this explains everything for you!

Hello everyone! After discussing this with my teammates we have decided that, while we really appreciate your idea, this is not something we are going to add to the framework. We are more in favor of  using sass as brandy described above. Thanks for the issue though, and have fun using Ionic!


Update ouiTextSubduedColor in next dark theme

Description
Shade the theme's $ouiTextColor instead of using one of the grayscale shades. Only affects next dark theme.
Issues Resolved

fixes #938

fixes #955
demo shows the various pseudo states
https://github.com/opensearch-project/oui/assets/1679762/4aa53e2a-946d-4bf2-878a-90925e465ae1
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.
[ ] All tests pass

[ ] yarn lint
[ ] yarn test-unit

[ ] Update CHANGELOG.md
[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@KrooshalUX Let me know if you want to move forward with this approach, tweak the calculation, or just provide an explicit value.

@joshuarrrr  do you have other non-collapsable nav examples?


Prow tests are failing because unrecognized import path istio.io/test-infra/boskos/cmd/mason_client

Snippet of error logs
+ go get istio.io/test-infra/boskos/cmd/mason_client
package istio.io/test-infra/boskos/cmd/mason_client: unrecognized import path "istio.io/test-infra/boskos/cmd/mason_client" (https fetch: Get https://istio.io/test-infra/boskos/cmd/mason_client?go-get=1: net/http: TLS handshake timeout)
+ cleanup
+ [[ True == \T\r\u\e ]]
+ unsetup_clusters
++ kubectl config current-context
error: current-context is not set
+ PILOT_CLUSTER=

Example PR: https://github.com/istio/istio/pull/13627

Happens on Circle too if that helps https://circleci.com/gh/istio/istio/406594?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

Seems like it has been fixed.


Add provider id for PV for GCP

What does this PR change?
Update to support GCP reconciliation. Adds provider Id's for GCP PVs and makes the general upgrade that all disks will have a provider id equal to their name if they do not already have one assigned. This second change will have an effect for all users that they will see disks that previously did not have a provider id as separate items at the time of the upgrade. This ID is determined at build time, so repairing or rebuilding the ETL will fix the issue.
Does this PR relate to any other PRs?

https://github.com/kubecost/kubecost-cost-model/pull/787

How will this PR impact users?

Users will have provider ids on all disks. This will cause some disks to appear as separate items at the time of upgrade. Repairing/rebuilding older days will remove this discontinuity, applying the new provider id retroactively.

Does this PR address any GitHub or Zendesk issues?

Closes ...

How was this PR tested?

see https://github.com/kubecost/kubecost-cost-model/pull/787

Does this PR require changes to documentation?

https://github.com/kubecost/kubecost-cost-model/pull/787

Have you labeled this PR and its corresponding Issue as "next release" if it should be part of the next Kubecost release? If not, why not?

@AjayTripathy When you have time, would read through this PR to help address possible customer questions.  CC:@nikovacevic

@Sean-Holcomb Should this PR have an additional document update next week?

This is the discontinuity in action. Notice that their minutes total to 1440, which is the number of minutes in a day. The second item has one third the minutes and one third the cost. The only difference is that the second item has the new provider ID. It is worth noting that when reconciliation hits this day the new item will get the full price from the billing export for that day, making the previous item duplicate cost. Using Asset repair on this day will erase this discontinuity.


When the element corresponding to the to selector does not exist, it enters a dead loop.

When the element corresponding to the to selector does not exist, it enters a dead loop and the page is unresponsive.

This should be an easy fix. I'll pick this up soon. Could you in the future please translate your messages using Deepl or a similar service because I don't speak Chinese 😅
这应该很容易解决。我会尽快解决的。今后能否使用 Deepl 或类似服务翻译您的信息，因为我不会说中文 😅


Jedis ShardedJedisPipeline hincrBy takes int instead of long

The regular incrBy in ShardedJedisPipeline also takes an int.

This has been fixed already.


Presign return invalid url

Hello.
Expectation:
/some-bucket/subdirectory-path1/subdirectory-path2/test.jpg
Reality:
/some-bucket//subdirectory-path1/subdirectory-path2/test.jpg
d, err := client.PreSign(context.Background(), "some-bucket", "/subdirectory-path1/subdirectory-path2/test.jpg") fmt.Println(d.Path) // returning /some-bucket//subdirectory-path1/subdirectory-path2/test.jpg
this fix trim objectName path, to correct urlPath

Yes. In S3 object keys are "just text", so even if MinIO cannot handle objects with a / as the first character they are technically allowed.

but if you work with url, its not correct behaviour

@ihatiko it is correct, you should normalize the object names as necessary outside of this SDK.


V2 API Fork functionality

For complete functionality, we will need to be able to fork projects and get a list of forks. This will involve
GET /v2/nodes/{node_id}/forks/
POST /v2/nodes/{node_id}/forks/
as well as adding a relation link for the forks inside the node.

Closed via https://github.com/CenterForOpenScience/osf.io/pull/5534


Clipboard support for pyglet and cocos2d

Thanks to #144 glfw and pysdl2 integrations got their clipboard support. Now it's time to do that for pyglet and cocos2d.

I'm marking this as "shelved" because there is no native stable API that we could use at this point.


[Bug] Android Geofence NullReferenceException

Platforms

[x] Android

Feature Library

[x] Locations

Version of Library
<IP_ADDRESS>7-beta
Version of OS(s) listed above with issue
Android 6.0.1
Steps to Reproduce
Not sure honestly, just got a crash report. Presumably "have a geofence"?
Expected Behavior
No crash.
Actual Behavior
System.AggregateException: One or more errors occurred. (Object reference not set to an instance of an object)

System.NullReferenceException: Object reference not set to an instance of an object
  at Shiny.Locations.GeofenceManagerImpl.Process (Android.Content.Intent intent) [0x0003c] in <45153172f6dc477f9ba4a2b27bd23b41>:0
  at Shiny.Locations.GeofenceBroadcastReceiver+<>c__DisplayClass1_0.<OnReceive>b__0 () [0x0009b] in <45153172f6dc477f9ba4a2b27bd23b41>:0

287 isn’t in public nuget yet.  Please wait for public releases before filing defects on it.  I only work on master so I don’t always have things in order yet

I only work on dev I meant.  So things keep pushing to myget

I couldn't reproduce this.  By crash, are you meaning it is showing up in the logging in the sample?

I haven't seen it yet in the Sample (not certain what Shiny build it's using - that would be a neat addition to the Environment screen), but it has now happened twice in the real app I'm close to releasing.

I do want to correct myself: it's being tracked as an error, not a crash, presumably here: https://github.com/shinyorg/shiny/blob/cfdb474a64d7e85b21bb7206bb7e6fbcaa4f9ffd/src/Shiny.Locations/Platforms/Android/GeofenceManagerImpl.cs#L47-L56
A few comments on that log call in particular:

What if each region.Create() had its own try/catch to include region details with the exception?
AggregateException being a generally unhelpful wrapper around the actual error(s), I wonder about having the Shiny Logging infrastructure check for AggregateException and log the InnerExceptions directly.
Thoughts on having the infrastructure that calls IShinyStartupTask.Start() handle the general try/catch, rather than individual tasks needing to do it?

I need to check with the guy testing the app, but this error may be due to attempting to recreate the geofence after revoking permission for Locations.

What if each region.Create() had its own try/catch to include region details with the exception?

Took a shot at this: https://github.com/shinyorg/shiny/pull/40. I believe this will avoid this particular AggregateException as well.

Your NRE is happening during the geofence process.  Is it coming from your delegate code perhaps?

I'm actually going to move this to an individual call instead of an array.
based on 1, this won't matter
This is by design.  StartupTasks must all pass (unless the author of the task doesn't need them to) as it was designed for wiring up infrastructure.  If they fail, they should crash otherwise you can start seeing missing behavior in your app potentially.

I need to check with the guy testing the app, but this error may be due to attempting to recreate the geofence after revoking permission for Locations.

I can't rule out permissions, but I thought I'd mention I have a separate crash report specifically related to permissions:

android.os.Parcel.createException

Parcel.java, line 1950
java.lang.SecurityException: Geofence usage requires ACCESS_FINE_LOCATION permission
android.os.Parcel.createException Parcel.java:1950
android.os.Parcel.readException Parcel.java:1918
android.os.Parcel.readException Parcel.java:1868
com.google.android.gms.internal.zzeb.zzb
com.google.android.gms.internal.zzbzl.zza
com.google.android.gms.internal.zzbzu.zza
com.google.android.gms.internal.zzbzc.zza
com.google.android.gms.common.api.internal.zzm.zzb
com.google.android.gms.common.api.internal.zzc.zza
com.google.android.gms.common.api.internal.zzbr.zzb
com.google.android.gms.common.api.internal.zzbr.zza
com.google.android.gms.common.api.internal.zzbp.handleMessage
android.os.Handler.dispatchMessage Handler.java:102
android.os.Looper.loop Looper.java:193
android.os.HandlerThread.run HandlerThread.java:65

I'd be curious to read your thoughts on the "Shiny way" to handle this, pre-create and if permissions are lost after the geofence is originally added to Shiny's repository.

Your NRE is happening during the geofence process. Is it coming from your delegate code perhaps?

I'd expect my code to appear in a stack trace if it were, but I doubt it:
        private class GeofenceDelegate : IGeofenceDelegate
        {
            public async Task OnStatusChanged(GeofenceState newStatus, GeofenceRegion region)
            {
                Analytics.TrackEvent("Geofence Change", new Dictionary<string, string>
                {
                    { "id", region?.Identifier },
                    { "center", region?.Center?.ToString() },
                    { "radius", region?.Radius?.ToString() },
                    { "status", newStatus.ToString() },
                });

                switch (newStatus)
                {
                    case GeofenceState.Entered:
                        await Api.Post.GeofenceTrigger(region.Identifier, true);
                        break;
                    case GeofenceState.Exited:
                        await Api.Post.GeofenceTrigger(region.Identifier, false);
                        break;
                }
            }
        }

Your NRE is happening during the geofence process.

I was looking for where it might have been logged. You're thinking it's in here:
https://github.com/shinyorg/shiny/blob/cfd1ed4ab320fc27008243b79379f10ab256b18f/src/Shiny.Locations/Platforms/Android/GeofenceProcessor.cs#L27
via
https://github.com/shinyorg/shiny/blob/cfd1ed4ab320fc27008243b79379f10ab256b18f/src/Shiny.Locations/Platforms/Android/GeofenceBroadcastReceiver.cs#L23-L29
https://github.com/shinyorg/shiny/blob/cfd1ed4ab320fc27008243b79379f10ab256b18f/src/Shiny.Core/Platforms/Android/Dispatcher.cs#L11-L21
Odd that the error was reported as GeofenceManagerImpl.Process(Intent).

Odd that the error was reported as GeofenceManagerImpl.Process(Intent).

The code of where Process was happening has moved in dev which is why you aren't seeing it relating to your error.

I'd be curious to read your thoughts on the "Shiny way" to handle this, pre-create and if permissions are lost after the geofence is originally added to Shiny's repository.

The permissions error makes sense and there isn't much I can do about that.  This is the reason I try to blindly restore the geofences on boot.  If permission has been pulled, you'll likely want to do check when your app is in the foreground.  I could downgrade the log to an event instead of an error.  May even add a category specifically around this type of issue

I'd expect my code to appear in a stack trace if it were, but I doubt it:

I'm still not seeing where the NRE could be.  Is there more to the stack trace in your appcenter log?

I'd expect my code to appear in a stack trace if it were, but I doubt it:

I'm still not seeing where the NRE could be. Is there more to the stack trace in your appcenter log?

There is not. Not sure if that's due to AggregateException or how Dispatcher Task handling works or something else. I can bump to the latest beta to see if this persists.
Beyond #40, which now seems unlikely to be related to what I am seeing, I am not sure how to get a better trace on my actual error. 🤔
I did try revoking Location permission for the sample app with geofences configured…no errors logged so far. 🤷♂

Note: I formatted the original exception stack trace as code, as some of the generics (specifically <OnReceive>) were being interpreted as HTML. I'm looking closer at the Process() code that moved in 1efa4eb1765f367ae48e56a5679d90f13b8c349b, and there are a couple NRE opportunities:
https://github.com/shinyorg/shiny/blob/5cff99d334adb5a3c20633b95eb6dd01409df3cd/src/Shiny.Locations/Platforms/Android/GeofenceManagerImpl.cs#L60-L68
I took a closer look at the GeofencingEvent docs and I'm reasonably certain it's an unhandled geofence error. PR incoming (separate from #40).

I'm going to consider this closed case for now.  Anything related can be a separate issue at this point.


[TOPI] [CUDA] Not in feed graph consumer = extern(argsort_gpu, 0xd3c1490)

I have called the function argsort_gpu in topi/cuda, and I got the log :
/tvm/src/schedule/bound.cc:129: not in feed graph consumer = extern(argsort_gpu, 0xd3c1490)
The input tensor is Tensor(shape=[1, 80, 15130], op.name=argsort_gpu)
I used by call argsort(scores,axis=2, is_ascend=False,dtype="int32")
I got the argsort result but it is very slowly,  the argsort cast about 7000ms.
Can someone give me some solutions or suggestion?

Hi @blacklong617
Please open a thread in tvm forum and provide more details of what you're doing.

Yeah, as @cchung100m suggested, please open a thread in the discussion forum.


expect,Added double speed api or control.

expect,Added double speed api or control.

Documentation:
https://github.com/media-kit/media-kit#set-volume-rate-or-pitch


feat(power-management): add extension

Description

redo of #9873 by @sasivarnan
An extension to toggle the Power Management settings of the macOS without fiddling around the ever-changing macOS settings.
Commands available

Toggle Low Power Mode

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

I don't have such thing in as lowpowermode in Sonoma

It still errors? The code should be grep ing for powermode to first determine the correct. I've removed some more unnecessary logic that should make it work better.

Could you look into the CI errors so I can give it another spin 🙂

CI errors fixed. Turns out I didn't sparse checkout properly, leaving an old file present 😅

Hey! Great job! i though I was the only one struggling with this!
Is there a way to not have to always approve the change with user password?

I won't add this to the extension, but if you were to add
{username} ALL=(root) NOPASSWD: /usr/bin/pmset -a powermode 0
{username} ALL=(root) NOPASSWD: /usr/bin/pmset -a powermode 1
{username} ALL=(root) NOPASSWD: /usr/bin/pmset -a powermode 2

where {username} is your username, to /etc/sudoers.d/pmset
And change line 27 of powerManagement.tsx from
await runAppleScript(`do shell script "pmset -a powermode ${value}" with administrator privileges`);

to
execSync(`sudo /usr/bin/pmset -a powermode ${value}`);

You can toggle low power mode without needing your password.

And change line 27 of powerManagement.tsx from

Where can I find this file powerManagement.tsx ?
The extra step of entering the password is too inconvenient for me.


Filter Fields Are partially hidden

The filter fields are partially hidden.

Expected Behavior
The fields should totally be visible.

Current Behavior
Extra paddings or margin with the fixed height caused the filed to be partially hidden

Your Environment

Tech
Version

Material-UI
^3.9.0

MUI-datatables
^2.4.0  -> 2.5.1

React
16.6.0

browser
Chrome

You're using currently unsupported functionality, as you are modifying the type of input field in the filters. Without official support (which may be coming in the next release), you are on your own here. You have the ability to override any styles provided to the table, so you should be able to fix it by doing that.

You can keep an eye on this https://github.com/gregnb/mui-datatables/pull/708, as I believe it will fix your issue and provide you with a supported API to add custom rendered filers.

Thanks a lot @gabrielliwerant


Core: Rename Throttler into RpsLimiter

Most people don't get the Throttler name right. RpsLimiter seems more explicit.

Good idea, but not worth breaking things.


sbt core/test does not work on OS X

With commit 445e45ca9340bf60c3aa45bd464cbf29387e84c0
$ sbt core/test

causes exceptions and gets stuck. I'm running on OS X 10.11.3, with Kafka <IP_ADDRESS> via Homebrew unofficial package: https://github.com/Homebrew/homebrew/pull/48245
I haven't looked in detail to what could be wrong. I'm expecting master should always pass the tests, right?

Are others able to reproduce this
Is there something I can do about it?

logs-core-test.txt

@akauppi tests in master should work. Could you tell which tests are broken? (away form my laptop now)

I checked tests and they  works for me. Note, that it takes about 20 minutes to complete them on my laptop. Also not that you should allow to auto create new topics in kafka. I updated readme with this small detail.

The log of running the tests is included in the issue.
On 28 February 2016 at 18:29, Alexey Romanchuk<EMAIL_ADDRESS>wrote:

I checked tests and they works for me. Note, that it takes about 20
minutes to complete them on my laptop. Also not that you should allow to
auto create new topics in kafka. I updated readme with this small detail.
—
Reply to this email directly or view it on GitHub
https://github.com/softwaremill/reactive-kafka/issues/94#issuecomment-189899426
.

There are a lot of errors in logs because a spec tests exceptional situations and a reaction to them.
As for me your log looks incomplete, because it should ends with summary. Like this:
===============================================
Command line suite
Total tests run: 36, Failures: 0, Skips: 20
===============================================

[info] ReactiveKafkaPublisherSpec:
[info] - optional_spec104_mustSignalOnErrorWhenFails
[info] - optional_spec105_emptyStreamMustTerminateBySignallingOnComplete !!! IGNORED !!!
[info] - optional_spec111_maySupportMultiSubscribe !!! IGNORED !!!
[info] - optional_spec111_multicast_mustProduceTheSameElementsInTheSameSequenceToAllOfItsSubscribersWhenRequestingManyUpfront !!! IGNORED !!!
[info] - optional_spec111_multicast_mustProduceTheSameElementsInTheSameSequenceToAllOfItsSubscribersWhenRequestingManyUpfrontAndCompleteAsExpected !!! IGNORED !!!
[info] - optional_spec111_multicast_mustProduceTheSameElementsInTheSameSequenceToAllOfItsSubscribersWhenRequestingOneByOne !!! IGNORED !!!
[info] - required_createPublisher1MustProduceAStreamOfExactly1Element !!! IGNORED !!!
[info] - required_createPublisher3MustProduceAStreamOfExactly3Elements !!! IGNORED !!!
[info] - required_spec101_subscriptionRequestMustResultInTheCorrectNumberOfProducedElements
[info] - required_spec102_maySignalLessThanRequestedAndTerminateSubscription !!! IGNORED !!!
[info] - required_spec105_mustSignalOnCompleteWhenFiniteStreamTerminates !!! IGNORED !!!
[info] - required_spec107_mustNotEmitFurtherSignalsOnceOnCompleteHasBeenSignalled !!! IGNORED !!!
[info] - required_spec109_mayRejectCallsToSubscribeIfPublisherIsUnableOrUnwillingToServeThemRejectionMustTriggerOnErrorAfterOnSubscribe
[info] - required_spec109_mustIssueOnSubscribeForNonNullSubscriber
[info] - required_spec109_subscribeThrowNPEOnNullSubscriber
[info] - required_spec302_mustAllowSynchronousRequestCallsFromOnNextAndOnSubscribe
[info] - required_spec303_mustNotAllowUnboundedRecursion
[info] - required_spec306_afterSubscriptionIsCancelledRequestMustBeNops
[info] - required_spec307_afterSubscriptionIsCancelledAdditionalCancelationsMustBeNops
[info] - required_spec309_requestNegativeNumberMustSignalIllegalArgumentException
[info] - required_spec309_requestZeroMustSignalIllegalArgumentException
[info] - required_spec312_cancelMustMakeThePublisherToEventuallyStopSignaling
[info] - required_spec313_cancelMustMakeThePublisherEventuallyDropAllReferencesToTheSubscriber
[info] - required_spec317_mustNotSignalOnErrorWhenPendingAboveLongMaxValue
[info] - required_spec317_mustSupportACumulativePendingElementCountUpToLongMaxValue !!! IGNORED !!!
[info] - required_spec317_mustSupportAPendingElementCountUpToLongMaxValue !!! IGNORED !!!
[info] - required_validate_boundedDepthOfOnNextAndRequestRecursion
[info] - required_validate_maxElementsFromPublisher
[info] - stochastic_spec103_mustSignalOnMethodsSequentially !!! IGNORED !!!
[info] - untested_spec106_mustConsiderSubscriptionCancelledAfterOnErrorOrOnCompleteHasBeenCalled !!! IGNORED !!!
[info] - untested_spec107_mustNotEmitFurtherSignalsOnceOnErrorHasBeenSignalled !!! IGNORED !!!
[info] - untested_spec108_possiblyCanceledSubscriptionShouldNotReceiveOnErrorOrOnCompleteSignals !!! IGNORED !!!
[info] - untested_spec109_subscribeShouldNotThrowNonFatalThrowable !!! IGNORED !!!
[info] - untested_spec110_rejectASubscriptionRequestIfTheSameSubscriberSubscribesTwice !!! IGNORED !!!
[info] - untested_spec304_requestShouldNotPerformHeavyComputations !!! IGNORED !!!
[info] - untested_spec305_cancelMustNotSynchronouslyPerformHeavyCompuatation !!! IGNORED !!!
[info] ScalaTest
[info] Run completed in 13 minutes, 28 seconds.
[info] Total number of tests run: 16
[info] Suites: completed 1, aborted 0
[info] Tests: succeeded 16, failed 0, canceled 0, ignored 20, pending 0
[info] All tests passed.
[info] Passed: Total 16, Failed 0, Errors 0, Passed 16, Ignored 20
[success] Total time: 822 s, completed 28.02.2016 23:27:12

Also I did not find any failures in log.

Ok, I was more patient this time and let the tests run.
You are right, eventually the summary comes up. Only one of the tests failed:
- required_spec313_cancelMustMakeThePublisherEventuallyDropAllReferencesToTheSubscriber *** FAILED ***
[info]   java.lang.AssertionError: Did not receive expected element within 30000 ms
[info]   at org.testng.Assert.fail(Assert.java:78)
[info]   at org.reactivestreams.tck.TestEnvironment.flopAndFail(TestEnvironment.java:180)
[info]   at org.reactivestreams.tck.TestEnvironment$Receptacle.next(TestEnvironment.java:878)
[info]   at org.reactivestreams.tck.TestEnvironment$ManualSubscriber.nextElement(TestEnvironment.java:400)
[info]   at org.reactivestreams.tck.TestEnvironment$ManualSubscriber.nextElement(TestEnvironment.java:392)
[info]   at org.reactivestreams.tck.TestEnvironment$ManualSubscriber.nextElement(TestEnvironment.java:388)
[info]   at org.reactivestreams.tck.PublisherVerification$24.apply(PublisherVerification.java:956)
[info]   at org.reactivestreams.tck.PublisherVerification$24.apply(PublisherVerification.java:949)
[info]   at org.reactivestreams.tck.PublisherVerification$25.run(PublisherVerification.java:966)
[info]   at org.reactivestreams.tck.PublisherVerification.activePublisherTest(PublisherVerification.java:1092)
[info]   ...
[info]   Cause: java.lang.AssertionError: Did not receive expected element within 30000 ms
[info]   at org.testng.Assert.fail(Assert.java:89)
[info]   at org.reactivestreams.tck.TestEnvironment.flopAndFail(TestEnvironment.java:177)
[info]   at org.reactivestreams.tck.TestEnvironment$Receptacle.next(TestEnvironment.java:878)
[info]   at org.reactivestreams.tck.TestEnvironment$ManualSubscriber.nextElement(TestEnvironment.java:400)
[info]   at org.reactivestreams.tck.TestEnvironment$ManualSubscriber.nextElement(TestEnvironment.java:392)
[info]   at org.reactivestreams.tck.TestEnvironment$ManualSubscriber.nextElement(TestEnvironment.java:388)
[info]   at org.reactivestreams.tck.PublisherVerification$24.apply(PublisherVerification.java:956)
[info]   at org.reactivestreams.tck.PublisherVerification$24.apply(PublisherVerification.java:949)
[info]   at org.reactivestreams.tck.PublisherVerification$25.run(PublisherVerification.java:966)
[info]   at org.reactivestreams.tck.PublisherVerification.activePublisherTest(PublisherVerification.java:1092)
[info]   ...

So I think the real problem was I'm just not used to tests showing stack traces on the log. Please check the above fail log and if it seems fine, I don't mind if you close the issue. The machine is Core i7 Macbook Air (mid 2011).

What about rerun only this test?

There are a few tests which take a very long time. They may need bumping timeout in settings. As @13h3r mentioned, some exceptions are expected on the output :)

Ok. Then the instructions on how to bump those timeouts could be clearly stated in the README.

There are a few tests which take a very long time. They may need bumping timeout in settings. As @13h3r mentioned, some exceptions are expected on the output :)

It seems I'm ending my scouting on reactive-kafka, for now. Good luck with the project.

@13h3r Wasn't able to figure out which command reruns a single test in a multi-project setup.

@akauppi Thanks for all the input. Good point with adding information on test timeouts to the README. Looks like this "30000" reported in your output is not the timeout I was thinking about. I'm adding an issue to investigate this further.


Add consistent ordering for external citations

When generating the FDSYS URL, always order the parameters
When generating the layer data, order the matches within a single node

Part of a series to make Python3 results match Python2. Split out into different PRs so that comparisons with existing data are easier to spot.

Good idea, this.


Update constructor to use __construct

The __construct method has been available since PHP5 and makes the code more portable. Class-named constructors are no longer treated as a constructor in namespaced classes since PHP 5.3.3.

https://github.com/google/ReCAPTCHA/pull/7

Fixed in v1.1.


[Bug]: Select Widget - On entering a filter value unrelated values are also displayed

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When a user enters a value in the filter option, other options which are not related to the filter are also displayed.

Steps To Reproduce

Drag the Select widget onto the canvas.
Click on the drop down option.
Enter a value in the Filter field.

Expected: Only entries related to the value entered should show up.
Actual: Unrelated entries also show up
Public Sample App
No response
Version
Release

Hey @shadabbuchh is server side filtering turned on?

@dilippitchika no.

Ok, let me check. If this is not working then it should be high or critical.

@Tooluloope do you know why the search shows results from items which don't have the same characters? It's first doing an exact match then it shows weird results.

THis is weird, Wasn't like this before now.
looking into this

@Tooluloope please clarify the following:
When the user types something in the filter section, options that contain those typed in characters somewhere in them also show up in the suggestions e.g. below I typed in "re" and the suggestions that came up were "Red"(as expected) & "Green"(maybe because it also contains "re" in it). Is this something that is built deliberately or should this also be fixed?

@shadabbuchh the filtering should only shows items that have the filter text,
I.e if "re" is inputed, only options with "re" in either label or value should show up.

Thank you @Tooluloope. Will close this bug as now it is working as expected.


Symlink Client Location for Overviewer

Have the make file create a symlink so we don't have to specify the texture path in the overviewer-settings.py file
If we symlink /opt/mscs/.minecraft/versions to /opt/mscs/client/ then Overviewer will be able to find the default Minecraft textures to render maps.

This also has the added benefit of not having to manually update the texture path in overviewer-settings.py when the server is updated to a new version.

I'll accept a patch that you have tested if you have one.  Otherwise, I'll play around with Overviewer sometime this week and get a patch together.

That's right, I forgot that the client jars were only meant for Overviewer. Since they are only used for mapping, it would make sense to put them where Overviewer expects them.
Overviewer specifically looks for the client jar in the current user's home directory, so more specifically the correct path is ~/.minecraft/versions/
I do need to do some testing with snapshots, Overviewer might not use this "default" texture path and may require this setting to have a value.
I may also have a better default overviewer-settings file, that includes the Nether and The End.

I just sent commit 56950b7e12fcb145ae3812b61e4f4460e08d3c1a that moves the client location to the normal location and removes the setting from the overviewer-settings.py file.  If you have a better default overviewer-settings.py, I'll take a patch or create one if you let me know what it is.

Oops, I didn't mean to close the issue with that comment.

Will do

I think we can call this one closed and open another issue for a different overviewer default file.


The bug that fflush closed file descriptors

[The following exception is coverted to a warning because it is thrown in the DESTROY method]
[System Error]fflush failed:Bad file descriptor
    SPVM__Sys__IO__fflush at Sys/IO.c line 605
    IO::File->flush at /home/kimoto/perl5/perlbrew/perls/perl-5.32.1/lib/site_perl/5.32.1/SPVM/IO/File.spvm line 203
    IO::File->DESTROY at /home/kimoto/perl5/perlbrew/perls/perl-5.32.1/lib/site_perl/5.32.1/SPVM/IO/File.spvm line 122

This is IO::File bug.


Remove unused model exporter

Description
Remove unused code. it does has tests
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

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

Reviewer Checklist:

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

Processing this pull request

Merged into branches master in 05ede19a9a347ba9038ed766de9b7e5c54daed90

Processing #4840.


Documentation Additions and Changes

Here are my contributions to the documentation. I started by adding a page for the Platform and Plugins preferences page.
Done:

External Services
Platform and Plugins

Experimental Features
Update Settings

SDKs/Bundles/Tools

There's more to come!
(P.S. I highly recommend squashing this PR. It has a very messy Git history.)

Hello. Thanks for contribution, but for now mkdocs can't serve this docs due YAML error, due section url not supported (https://www.mkdocs.org/user-guide/writing-your-docs/#configure-pages-and-navigation)
      - Platform And Plugins: platform/preferences/platform.and.plugins.md  !!!Here problem. Need child node for it
        - Experimental Features: platform/preferences/platform.and.plugins/experimental.features.md
        - Update Settings: platform/preferences/platform.and.plugins/update.settings.md

Also due strict mode - moving docs is bad idea (because all links checked).  And moving docs without redirects also bad idea, because it broke old links .
We can use https://github.com/datarobot/mkdocs-redirects plugin for it.
Also new nightly builds(3102+), now have different logic for settings url generate, and need add redirect anyway.
Thanks:)

Also, root path changed from prefereces -> settings (and need make redirect)

It's changed back already...?
On Tue, Jan 4, 2022, 7:49 AM Valery Semenchuk @.***>
wrote:

Also, root path changed from prefereces -> settings (and need make
redirect)
—
Reply to this email directly, view it on GitHub
https://github.com/consulo/consulo-help.github.io/pull/1#issuecomment-1004923136,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHHUNMT3F54BLTYUQZMTWCTUUMJHBANCNFSM5LGG5Z2Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

back? I decided after our dialog, change it. As you suggested.

I changed most references to "Preferences" in documentation to "Settings" leaving alone parts that specifically mention both Preferences and Settings for clarity.


update mulsticluster subscription operator to V0.1.8

Signed-off-by: Xiangjing Li<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.

Flat operator directory structure is obsolete from 23-rd of October 2019, only nested directory structure will be accepted.

Updates to existing Operators

[x] Is your new CSV pointing to the previous version with the replaces property?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
[x] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

@mvalahtv  @J0zi  Could you help merge and publish our operator with higher priority? Our canary test has been blocked due to it since yesterday.  Thanks so much for your help!


Metadata before Data when fragmented

Require implementations to send Metadata before Data when fragmented.
Proposed for https://github.com/ReactiveSocket/reactivesocket/issues/170

@tmontgomery you okay with this wording and example?

I am good.


Allow to have multiple pined station

Add possibilty to add multiple station

done ( but we can't delete them)


chore(operator): apply max_batch_limit for async operator

Patch for #1615

Let's waif for CI setup.


Reland --omit-type-checks for benchmarks.

Because the cost of type checks dominate our dart2wasm benchmarks, we've decided to pass --omit-type-checks for now.
This was previously reverted because the skwasm benchmarks were broken in general for a separate reason, and my getting rid of bringup: true broke the tree. I ended up fixing the benchmarks and getting rid of bringup: true in a separate commit, so this just adds the flag only.

SkiaPerf is reporting both improvements and regressions on this change:
https://flutter-flutter-perf.skia.org/e/?begin=1690818891&end=1690857627&keys=X3ab386d6ec803476646eeb55e187f612&num_commits=50&request_type=1&xbaroffset=36059
https://flutter-flutter-perf.skia.org/e/?begin=1690818891&end=1690857627&keys=X756c4ff65f3199869e13126f6fcffa93&num_commits=50&request_type=1&xbaroffset=36059

The "regressions" are just changes in the outlierRatio, which I think is just a reflection of the fact that the average went down so much for those benchmarks.


Is this Supported on Windows 10 22H2

I ran into this issue recently and have used this guide to fix it. On Windows 11 this seems to do the trick. But now I am hearing from Windows 10 machines that they can't install the app package and get an error "Cannot open app package" reason: "Error in parsing the app package".
The only thing I changed in the appinstaller file is the version number, and the msixbundle url. When I go back to a version from before the publisherbridge change it also works!
Can someone confirm this is supportong on windows 10 22H2, I'm testing on a VM:
Edition	Windows 10 Enterprise
Version	22H2
Installed on	24/05/2022
OS build	19045.2846
Experience	Windows Feature Experience Pack 120.2212.4190.0

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

ID: fdec14ed-d6ca-9bd6-f0f7-de3b30915aa3
Version Independent ID: 7ff21924-0311-5f70-7087-4a1669ee8afe
Content: MSIX Persistent Identity - MSIX
Content Source: msix-src/package/persistent-identity.md
Product: windows
Technology: msix
GitHub Login: @dianmsft
Microsoft Alias: diahar

Been getting this with a few users recently too. Tends to be that a redownload of the .appinstaller and a PC restart that fixes it, though I've had some users need to go the extra mile and use DISM to restore their machine's health. Seems like some sort of weird caching issue but not sure

@jbennink As of now, this is only available for Win 11

Are there any plans to backport persistent identities for Win 10?

@jbennink @benaclejames Please see this thread for the cause of the random version-change-related install failures (with workaround):
https://github.com/MicrosoftDocs/msix-docs/issues/188


BENCH: Add missing pythran dependency

What does this implement/fix?
Running a comparison benchmark, e.g. python runtests.py --bench-compare master fails because pythran isn't listed as a benchmark dependency.

We are actually missing reference of Pythran in the doc as well. The tooling section could be improved for instance.

Thanks for the review @tupui. I only found this because it was blocking my current workflow. If you know of other places that need updated, then definitely open an issue. I'll just merge this as-is.


Cleaned up NuGet Package Duplicates

Removed copies of License/Version from folders packages together, and also updates the nuspec to exclude the yoc and the StandardAssets.meta.

:shipit:


Support for Next.js?

This page claims that this project supports Next.js (and other bundlers). I'm trying to integrate this into my (closed source) project, and my gazelle command isn't able to import styles. It's possible I've missed something, or that Next.js support isn't implemented yet.
If Next.js support is planned, I suppose this can be a tracking issue for it. If I can get an idea of what would be required, I might implement it myself.
If this is already supported, then we should add some docs to the main Readme.
If it's never planned to be supported, then we should update the bazel-gazelle docs accordingly.

@shelbyd The linked PR is some WIP towards rule generation for a Next.JS app
In the meantime it's possible to get Next.JS or other bundlers to work using a macro like this:
https://github.com/aspect-build/bazel-examples/blob/main/next.js/bazel/next.bzl

The latest release has an end-to-end example on a NextJS app built with Bazel in the examples folder


Is i18n(internationalization) applicable in this package

Is it possible to have the returned message in different languages?

In it's current form it is not. I personally haven't been doing validation this way since I started using aldeed:simple-schema. Its a much more meteoric way of doing validation and allows you to secure your writes in such a way to allow you to do all database operations from the client.

You are of course more than welcome to for and modify the repo if you prefer this type of validation.


Staging Test - Added formatting to first note (#4438)

Staging to master workflow test - Added formatting to first note
Staging test
Updated:

User's Guide > Messaging > Mentioning Teammates > @ Mentions

Update source/help/messaging/mentioning-teammates.rst

Co-authored-by: Justine Geffen<EMAIL_ADDRESS>

@amyblais - this docs update is being applied through the new Staging branch for product docs. Because this Staging branch is our new "master", we need at least 1 approving review with write access to proceed. This branch is expected to be out of date with master; I can address any merge conflicts. Thanks!

@cwarnermm, should we close this given that we've changed the process?

@justinegeffen - Yes. This was a good way for us to learn how things work, but now I'll close it to avoid going against our new process.


Invalid JSON when selecting only weapons and characters

Deselect Artifacts
JSON is formatted:
"characters": [ ... ]"weapons": [ ... ]
JSON should be:
"characters": [ ... ], "weapons": [ ... ]

My apologies. Updated to latest version, selected only first and last scan items, and did not encounter the same error.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 958

Questions ?
Please refer to the Backport tool documentation


ID#11129 Quaterly review

ID#11129 Quaterly review

@surajmr please resolve your conflicts

@carmenberdant1 Changes done


Adding retry option if weasyprint process gets timed out

Related to #48 we have problems to get it stable running in your CI System on windows. This affects all up to the newest version of weasyprint. Workaround by limiting the number of process to one for sphinx_simplepdf builds limits us in CI speed and also leads to failed builds sometimes.
So i decided to add a retry configuration option. This could help us in case of timeouts in the pdf generation to retry. I currently use 3 times and builds are not broken any more.
This only works when using weasyprint as executable, not as module. But only in this case the timeout option is also usable.

Severe Error in thePR: Using the default configuration value for retries 0, simplepdf never outputs an pdf without any hint. I think setting retry default value to 1 should fix the problem setting the range value to retries + 1