Fix switching explorer from admin panel

when switching explorer from admin panel, switch the default explorer and the default identity
change explorer instance in zos sal if the default explorer url is changed

https://github.com/threefoldtech/js-sdk/issues/94

in jsng an identity has concept of explorer... I think it makes more sense to keep it this way and allow the portal to switch between identities (which are linked to an explorer) not switching the explorer itself within an identity cause this same identity might not exist  on another explorer...
In all the code above I see we are trying to get the 'default' explroer instead of the explorer of the current identity j.me.explorer...
imho it should be the other way around...


Support simple query strings

Right now, it appears that I have to use:
https://covidcast.cmu.edu/?sensor=doctor-visits-smoothed_adj_cli&level=county&date=20201120&signalType=value&encoding=color&mode=export
if I were to provide somebody a link, and want them to land on the "Export Data" tab.  A lot of this info in the query string seems unnecessary for this tab, and conceptually it seems like all that should be needed is:
https://covidcast.cmu.edu/?mode=export
This brings me to the "Export Data" tab but then it hangs for me (nothing loads).  FWIW, seems like it attempts to bring up this page: https://covidcast.cmu.edu/?mode=export&sensor=doctor-visits-smoothed_adj_cli&level=county&date=20201120&signalType=value&encoding=color
Can we support simple query strings like this one?  It would be great to do something similar for all the tabs: for each one, figure out the minimal amount of info that's needed to load this tab, and allow those query strings (rather than the full ones).

Yeah, this has bothered me a bit also.  We can work on simplifying the query string scheme after this release.


add support for array arguments and return types in UDFs

What kind of change does this PR introduce?
Feature
What is the current behavior?
Arrays are not supported in arguments and as return types of user defined functions.
What is the new behavior?
Arrays are supported in arguments and as return types of user defined functions.
Additional context
Default values for array type arguments are not supported.

@olirice arrays of composite types are filtered out now. Docs and changelog have also been updated.

Filtered out functions with enum and enum arrays as args or return values. Added a task in #410 to add support for them.

@imor Please verify this is not my setup and help to get rid of errors:
begin;

create schema if not exists a1; grant usage on schema a1 to public;
create table a1.foo(id int primary key); grant select on table a1.foo to public;
insert into a1.foo (id) values (1);
create or replace function a1.bar(a1.foo) returns int[] stable return array[1, 2, 3]::int[];
grant execute on function a1.bar(a1.foo) to public;
create or replace function a1._the_foo(a1.foo) returns int[] stable return array[1, 2, 3]::int[];
grant execute on function a1._the_foo(a1.foo) to public;
set local search_path to a1;
comment on schema a1 is e'@graphql({"inflect_names": true})';
select graphql.resolve($$query {fooCollection {edges{node{ id bar }}}}$$); -- invalid return type from function
select graphql.resolve($$query {fooCollection {edges{node{ id the_foo }}}}$$); -- Unknown field 'the_foo' on type 'Foo'
select graphql.resolve($$query {fooCollection {edges{node{ id _the_foo }}}}$$); -- Unknown field '_the_foo' on type 'Foo'

rollback;


Add support for angle bracket matching and colorization in Typescript

This PR fixes #151196
cc @hediet

This looks good, thank you for the PR! Did you test it / can you post a screenshot?

So far this is all I can think of.

Hmm, it should probably also be ignored for jsx elements (return

;).

Hmm, it should probably also be ignored for jsx elements (return

;).

It looks like we just need to filter meta.brace.angle and punctuation.definition.tag again.

But I feel that this implementation does not seem to work well and there is always the possibility of missing something. Are there other better options? Or use the balancedBracketScopes option to implement a similar whitelist pattern?
Another question is that I'm not sure if the use of "balancedBracketScopes": ["punctuation.definition.typeparameters"], overwrites the currently colorization brackets, and if so, that's a bit tricky.

This would need to allowlist all the cases for (...), {...} and [...] too then... Which might be reasonable though.

From the description, it is no different from the current solution, only now I'm doing the exclusion method. Then let's continue with the current solution, if you don't have any objections either. 🥰

I will merge this after the June release, so we have enough time to test this.


fix(behavior_path_planner): revival of skipped lane_ids by smooth goal connection

Description
Smooth goal connection, described as follows, removes lane_ids within a certain-ranged circle, which makes beahvior_velocity_planner not work.
https://autowarefoundation.github.io/autoware.universe/main/planning/behavior_path_planner/#smooth-goal-connection
Fixed the issue by filling removed lane_ids to pre_goal lane points.

This PR includes

small refactoring (I thought this refactoring is related to the fix, but it's actually not, sorry about that).
test for the issue

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

I will check if this works well with planning simulator from now


add option for early convert arguments #391

allows passing in imagemagick convert switches that will precede the
source file name.  useful for flags such as -density that must be set
before the source file in the convert command
Example:
image.process(:convert, “100x100”, early_args: “-density 150”)

I've used this in one of my projects. Everything worked as expected.

hi - thanks for this. I'd prefer "input_args" - what do you reckon?

agreed. better name.

travis is failing because it can't bundle rubysl-openssl. Does seem like anything related to this pull request.

cheers - merged in


Cannot clone dpnn

I encountered this error message:
Cloning into 'dpnn'...
fatal: unable to connect to github.com:
github.com[0: <IP_ADDRESS>]: errno=Connection timed out
Error: Failed installing dependency: https://raw.githubusercontent.com/torch/rocks/master/dpnn-scm-1.rockspec - Failed cloning git repository.

Did you try :
luarocks install dpnn

Or
git clone git@github.com:nicholas-leonard/dpnn.git
cd dpnn
luarocks make rocks/[TAB]


Feature/readme cleanup

Starting to clean up and add interesting stuff to the README. Feedback and ideas obviously welcome 🤭

Gonna merge it for now just to have a cleaner landing page for the game, might add screenshots and stuff later on ;)

Gonna merge it for now just to have a cleaner landing page for the game, might add screenshots and stuff later on ;)


Page description

We should add a page description, currently snapshot appears like this on google

I'll work on this. Also if we have,

set of keywords
Open Graph images
Sitemap

@samuveth

@samuveth @samuveth We kind of blocked by this 🤔
https://github.com/snapshot-labs/snapshot/issues/1210

Sort of new here, but my opinion on these SEO related issues is the following:
Adding page descriptions is more of a SEO workaround for the current project setup.
The project right now is setup as a SPA which means that everything you do around SEO will be by default not optimal for search engines. Although search engines can crawl SPA content it's often not as reliable as Server Side Rendered content.
Therefore a proper solution for this is to add SSR at least for some pages which would benefit from it. Some examples are:

Main page (explore page)
Spaces pages including about page for each space
Proposal pages

Technical implementation notes
We could use vite-ssg
We would have to change the url structure and remove the hash from the url /#/. In order to avoid breaking change we can implement a temporary redirect from all routes with /#/something to /something.
Lastly this could have an impact on certain code and the way the website is currently deployed.
Long term, having SSR would be way more beneficial for the project meaning that proposals & spaces are indexed by search engines and will appear in search results providing more website traffic.

@cristijora Yes we have considered SSR, but we use Fleek to deploy/host our files on IPFS and they do not provide a server for SSR. I have been looking into JAMstack which would allow us to statically generate pages (spaces, proposals, strategies, ...) at build time and then upload them to IPFS. This might sound like we lose a lot of dynamic functionality but this doesn't seem to be the case.
These are some resources that I found:

https://withblue.ink/2020/01/21/watch-javascript-apps-going-inter-planetary.html
https://www.smashingmagazine.com/2020/07/differences-static-generated-sites-server-side-rendered-apps/
https://hasura.io/blog/approaches-to-add-dynamic-content-statically-generated-sites-jamstack/
https://www.smashingmagazine.com/2019/12/dynamic-async-functionality-jamsstack-websites/

Would love to hear your thoughts!
@sasicodes sorry didn't mean to delete your comment.
@bigint as a fix just for our homepage we can use something vue-meta I think.

@samuveth You can statically generate pages but in my opinion that wouldn't really fully solve the issue.
Consider that now there are 4.44k spaces. Only generating pages for those spaces might take a bunch of time at build time and if you include proposals it goes trough the roof.
Indeed you don't lose any dynamic functionality but if you want to generate the pages at build time for all spaces/proposals it will be very hard to maintain as those numbers grow.

@cristijora Hm, then I wonder how something like twitter handles this (I heard they switched to JAMstack). Another solution could be using our own server and as service like https://prerender.io/

Added a new pitch in the Project Roadmap that needs scoping :pray:

To clarify, this issue was marked as closed due to the lack of a viable solution for maintaining SEO-friendly meta tags on the homepage without cluttering the space page.


Can I use T265 to sample the imu data at 200HZ and camera data at 20HZ at the same time?

rs2::pipeline pipe;
rs2::config cfg;
cfg.enable_stream(RS2_STREAM_ACCEL, RS2_FORMAT_MOTION_XYZ32F);
cfg.enable_stream(RS2_STREAM_GYRO, RS2_FORMAT_MOTION_XYZ32F);
cfg.enable_stream(RS2_STREAM_FISHEYE);
rs2::pipeline_profile profile = pipe.start(cfg);
when i sample the imu and camera data, I found the ratio of sample was depend on camera ratio

Hi @zhuoyueljl
I believe that the blocking version of start will reduce FPS to the lowest common denominator.
You could use asynchronous API as shown in rs-callback and rs-pose-predict to get the full framerate

@zhuoyueljl there is now an example showing how to use the callback API to do this here:
https://github.com/IntelRealSense/librealsense/pull/3522
It needs more documentation before it will be merged into librealsense, but it should help you get started.

Pre-Release of version 2.20.0 has been posted on site:  https://github.com/IntelRealSense/librealsense/releases/tag/v2.20.0


etcdHighNumberOfFailedGRPCRequests alarm not correct due NaN return value

prometheus-operator 0.30.1 (it was same on 0.30.0) on clean cluster (without any load) is constantly hit with alarms:
[FIRING:1] Monitoring Event Notification
*Alert:* etcdHighNumberOfFailedGRPCRequests - `critical`
 *Description:* etcd cluster "kube-etcd": 100% of requests for Watch failed on etcd instance <IP_ADDRESS>:2379.
 *Graph:* :chart_with_upwards_trend: *Runbook:* <|:spiral_note_pad:>
 *Details:*
  • *alertname:* `etcdHighNumberOfFailedGRPCRequests`
  • *grpc_method:* `Watch`
  • *grpc_service:* `etcdserverpb.Watch`
  • *instance:* `<IP_ADDRESS>:2379`
  • *job:* `kube-etcd`
  • *prometheus:* `monitoring/monitoring-prometheus-oper-prometheus`
  • *severity:* `critical`

As it seems this is covered in https://github.com/etcd-io/etcd/pull/10629.

I commented in the etcd pull request. Let's first discuss there. I don't think that what you are seeing is the same as that pull request. I think what you are seeing is https://github.com/etcd-io/etcd/issues/10289

Closing here as this is really something to be solved in etcd, not here.

FYI, for those using the etcd mixin, I worked around this by tweaking the etcd selector:
        local etcd = import "etcd-mixin/mixin.libsonnet";

        etcd {
          _config+:: {
            etcd_selector: 'job="etcd",grpc_service!="etcdserverpb.Watch"'
          },
        }


bug/gdrive ModuleNotFoundError: No module named 'unstructured.ingest.pipeline.reformat'

Describe the bug
When running the provided code snippet, the application throws a ModuleNotFoundError stating that it cannot find the module 'unstructured.ingest.pipeline.reformat'.
To Reproduce
The error can be reproduced by running the following code:
from unstructured.ingest.interfaces import PartitionConfig, ReadConfig
from unstructured.ingest.runner.google_drive import gdrive
gdrive(
    verbose=True,
    read_config=ReadConfig(preserve_downloads=True, download_dir=str(
        f"/ingest/files/projects/{project_id}/datasets/{session_id}/gdrives/{gdrive_id}"
    )),
    partition_config=PartitionConfig(
        output_dir=str(
            f"/ingest/data/projects/{project_id}/datasets/{session_id}/gdrives/{gdrive_id}"
        ),
        num_processes=config.ingest_batch_size,
        pdf_infer_table_structure=True
    ),
    drive_id=gdrive_id,
    service_account_key="/app/app/data/keys/dev-service-account-key.json",
    recursive=True,
)

Expected behavior
The expected behavior is that the application should be able to import the necessary modules and successfully execute the gdrive function.
Environment Info
Please run python scripts/collect_env.py and paste the output here. This will help us understand more about the environment in which the bug occurred.
Additional context
It appears that there might be a mistake in the module path, or the specified module does not exist in the project or installed packages. Please check the installed packages and the project structure to ensure that the path 'unstructured.ingest.pipeline.reformat' is correct.

Thanks for the report! Fix in progress here

The fix is released! Please update to 0.10.22.

@awalker4 This is still an issue in v0.12.4.
INFO:     Started reloader process [9800] using StatReload
Process SpawnProcess-1:
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 314, in _bootstrap
    self.run()
  File<EMAIL_ADDRESS>line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/Users/ismailpelaseyed/Projects/super-rag/.venv/lib/python3.11/site-packages/uvicorn/_subprocess.py", line 78, in subprocess_started
    target(sockets=sockets)
  File "/Users/ismailpelaseyed/Projects/super-rag/.venv/lib/python3.11/site-packages/uvicorn/server.py", line 62, in run
    return asyncio.run(self.serve(sockets=sockets))
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 190, in run
    return runner.run(main)
           ^^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 653, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "/Users/ismailpelaseyed/Projects/super-rag/.venv/lib/python3.11/site-packages/uvicorn/server.py", line 69, in serve
    config.load()
  File "/Users/ismailpelaseyed/Projects/super-rag/.venv/lib/python3.11/site-packages/uvicorn/config.py", line 458, in load
    self.loaded_app = import_from_string(self.app)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/ismailpelaseyed/Projects/super-rag/.venv/lib/python3.11/site-packages/uvicorn/importer.py", line 24, in import_from_string
    raise exc from None
  File "/Users/ismailpelaseyed/Projects/super-rag/.venv/lib/python3.11/site-packages/uvicorn/importer.py", line 21, in import_from_string
    module = importlib.import_module(module_str)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/Users/ismailpelaseyed/Projects/super-rag/main.py", line 6, in <module>
    from router import router
  File "/Users/ismailpelaseyed/Projects/super-rag/router.py", line 3, in <module>
    from api import delete, ingest, query
  File "/Users/ismailpelaseyed/Projects/super-rag/api/ingest.py", line 9, in <module>
    from service.ingest import handle_urls, handle_google_drive
  File "/Users/ismailpelaseyed/Projects/super-rag/service/ingest.py", line 4, in <module>
    from unstructured.ingest.connector.google_drive import (
ModuleNotFoundError: No module named 'unstructured.ingest'
sys:1: ResourceWarning: unclosed file <_io.TextIOWrapper name=0 mode='r' encoding='UTF-8'>

hi @homanp, I'm not actually able to reproduce this issue on my end. Updated the original code example and it ran just fine. Can you share the version of Unstructured on which you're working and a code sample? (edited)

hi @homanp, I'm not actually able to reproduce this issue on my end. Updated the original code example and it ran just fine. Can you share the version of Unstructured on which you're working and a code sample? (edited)

Hey, v0.12.4, installed unstructured[google-drive]. Using the code sample in your docs.


ashamed I don't get the original textarea to sync with woofmark

Hola, I just stumbled over woofmark last night, and so far I like it.
But I can't figure out how to sync those. I mean, when submitting the form, nothing get posted..?
This is my js to init woofmark..
var rte = document.querySelector('.rich-text');
    woofmark(rte, {
        parseMarkdown: megamark,
        parseHTML: domador,
        defaultMode: 'wysiwyg'
    });

Use case, I want to provide an easy (wysiwyg) forum input but behind the scenes everything should be markdown. Do I need parseMarkdown and parseHTML for this?
Maybe I should learn more vanilla-js..
Saludos, Can

you need to read from woofmark. Do var foo = woofmark(rte, ...) and then foo.value() will always return markdown

Thank you :)


Browse to Website goes to empty url

v 0.5.1 on insiders build 1.20.0

cc @fiveisprime @chrisdias
UPDATE: so does "open in portal"

seems to be a chrome issue.
closing chrome and reopening fixed it.


The query does not return any data.

I can't get the data from the supabase database, I can't get the data from the tables.
In a first project with supabase everything worked fine, when I started a second project, I couldn't get data from the [tables.[]
Future cargarTareas() async {
final supabase = Supabase.instance.client;
final data = await supabase.from('areas').select('nombre');
print(data);
}
returns an empty list.

I guess you have role level security enabled without or with wrong policies. https://supabase.com/docs/guides/auth/row-level-security

I would close this as it seems like the cause is what Vinzent has suggested, but feel free to reopen it if it isn't working after setting rls policies on the table!


Data value not recognise in words.yml

Hello,
Since the translation files updated to the new format, we can't name the items with metadata anymore. For instance, the different types of wood, wood planks, leaves, stones, etc, are all named "wood", "plank", "leaf", "stone". This means that the display list of the rewards will show something like :
Stone -> 1$ 1xp
Stone -> 2$ 2xp
Stone -> 2.5$ 2xp
Stone -> 0.5$ 0.5xp
Which is completely pointless and obscure.
Jobs version: 4.10.1
Spigot version: 1.12.2

So, if you want to use the stone material as an example for an action, does it characterize all types of stones?
This has already been removed due to the change of material data in version 1.13.
And if you have a problem, for example, if you specify granite and diorite material, then indicate the stone material for the job information instead of the specified?
If so, it has already been reported.

I'm not sure I understand what you are saying.
Writing "1-GRANITE: [translation" doesn't work, it's automatically erased upon reload.
Currently I found absolutely no way (despite many tests) to write the different types of stones (and blocks when there are variations).
If it has already been reported, could you reference the issue please ?

Yea, this already been reported.
#288
#296

Okay then my bad, hope it will be fixed soon. Thank you.

Some news about this ?

@montlikadani Hello, would you mind telling us where are we on this issue ? Thank you.

@Ferocimo Actually, nowhere, because we can't find the source of the error, so it's not yet possible to fix. It can take up to several months.

Words_en.yml will get filled by all possible materials automatically with corresponding ID's for them, you cant add extra ones and you cant remove existing. But you should find all of them in new list. There was issue, which was fixed, where you would get duplicated visualizations like that Stone - Stone - Stone, but simply removing Words files and allowing them to be regenerated will fix it


Update ReactionRole use in get_reaction_smiles

Updates get_reaction_smiles to accept ReactionRole.UNSPECIFIED as a marker for reactants and/or products. The test dataset does not have product reaction roles set, and this was causing reaction SMARTS queries to fail since reaction SMILES were not being generated as part of the dataset creation (and thus the rdkit.reactions table was empty).
Adds examples of additional chemistry-aware search functions.
Makes RDKitMol table fingerprint columns explicit in the ORM mapper for improved readability and IDE inference.
Silence RDKit logging when the ORM is imported.
Set the default log level to DEBUG.

Updates get_reaction_smiles to accept ReactionRole.UNSPECIFIED as a marker for reactants and/or products. The test dataset does not have product reaction roles set, and this was causing reaction SMARTS queries to fail since reaction SMILES were not being generated as part of the dataset creation (and thus the rdkit.reactions table was empty).

Does this only affect the test dataset?

Updates get_reaction_smiles to accept ReactionRole.UNSPECIFIED as a marker for reactants and/or products. The test dataset does not have product reaction roles set, and this was causing reaction SMARTS queries to fail since reaction SMILES were not being generated as part of the dataset creation (and thus the rdkit.reactions table was empty).

Does this only affect the test dataset?

No, there are about 1k inputs and 250k outcomes in the database that have an UNSPECIFIED reaction role. In cases where there was no explicit REACTANT or PRODUCT, these reactions will now be able to generate reaction SMILES.

@qai222 could you please approve?


Automated Package Upgrade

:tada: This PR is included in version 2.2.107 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


error in make

Hi
I have problem in make .so in linux
/app/hrportal/sap/php7-sapnwrfc/sapnwrfc.c:532:67: error: macro "zend_hash_add" requires 6 arguments, but only 3 given /app/hrportal/sap/php7-sapnwrfc/sapnwrfc.c:649:32: error: macro "RETVAL_STRINGL" requires 3 arguments, but only 2 given /app/hrportal/sap/php7-sapnwrfc/sapnwrfc.c:665:32: error: macro "RETVAL_STRINGL" requires 3 arguments, but only 2 given /app/hrportal/sap/php7-sapnwrf1/sapnwrfc.c:871:68: error: macro "zend_hash_update" requires 6 arguments, but only 3 given

Which PHP version are you trying to build for?

php 7.0.2
but my php 7.0.2 was instaleed in  /app/hrportal/opt/php-7.0.27/bin/php   and  /usr/include/php  is phph 5

Make sure you use PHP 7 includes when building. Something like the following:
make distclean
/app/hrportal/opt/php-7.0.27/bin/phpize
./configure --with-sapnrfc=/path/to/nwrfcsdk --with-php-config=/app/hrportal/opt/php-7.0.27/bin/php-config
make


Update tokio to 1.0.1

Rename with_io to try_io and drop with_poll - https://github.com/tokio-rs/tokio/pull/3306

Thanks again, @kiron1, and thanks for the help, @Darksonn!


A better way to detect WSL.

I changed my wsl kernel to https://github.com/Locietta/xanmod-kernel-WSL2, but vim's has("wsl") doesn't work. so I changed the way to match the kernel version info.
utils.is_wsl = function()
    return string.find(vim.loop.os_uname().release, "WSL") ~= nil
end

Agreed. The old detection method was a solution I got on vi.stackexchange which was quite old fashioned. This new way of detection is much better. Thanks @atauzki !
Fix for this is made in 4892f81.


Error when trying to use it with Vue3

I get this error when trying to use it with Vue3 :
external_commonjs_vue_commonjs2_vue_root_Vue_default
any ideas ?

Sadly, currently the package does not support Vue 3. And I also do not have a plan to update it.


Depricated Head Tag

Why are you using the Head tag over exporting metadata? Do you know something I don't, did i miss something?

@JustScriptin  Thank you so much, i will remove it in my next pull.


ERROR: An error occurred during the fetch of repository 'local_execution_config_python':

ERROR: An error occurred during the fetch of repository 'local_execution_config_python':

For anyone already did these 2 steps ,but still get above error:

you have set python path though: bazel build -c opt --define MEDIAPIPE_DISABLE_GPU=1 --action_env PYTHON_BIN_PATH="D:\\anaconda\\envs\\py310\\python.exe" mediapipe/examples/desktop/helloworld.dll
you have installed numpy to your python

You can comment out these 2 methods, _check_python_lib and bin. Then you can pass the error.
I didn't know why it got when check, but obviously google's people have too much time on written so much complicated code while they can do it 1000x easier.
And of course users get 10000x harder using their source codes.

Hi @jinfagang,
Could you please elaborate your query with complete details, error logs and steps to reproduce the issue from our end.


cluster_visualize Error

Hello, I encountered a problem while visualizing the cluster.
Function draw_ Segmentation_ Mask error: ValueError, Pass individual images, not batches
I checked that the dimensions of the mask are [num_heads, H, W], which may indicate that the channel dimension should be 1. Should I just take one of the heads?

I have resolved it. Thank you for the open source code.
It's because my IMG forgot to remove the batch_ Size。


1.20.4 Update?

Howdy, it appears that there is some ground work for a 1.20.4 update but upon building and running, in the IDE it fails to run client and after building then placed into a regular Minecraft fabric instance it fails on world creation. Is there any hope of getting a working 1.20.4 version? Thank you.

The crash is probably caused by Reach Entity Attributes, since it was outdated at the time of the initial port to 1.20.4. Since my PR for updating Reach Entity Attributes has been merged since a while ago, you can use that in your fork (no need to change repositories, it's already published in their maven.)
But in the meantime, there is no ETA as to when the port will be continued, and I can't really work on it right now considering that there is a planned minor release

I came across this while debugging my load order. I thought it was a shot in the dark but low and behold, it worked. I updated Reach Entity Attributes to the latest version and compiled from source. As far as i can tell, everything is working fine now, with the only bug I've encountered so far being an annoying flicker effect that goes away by disabling V-sync


Can't connect instavpn server on Amazon AWS

I installed instavpn on my server at Amazon AWS, opened udp ports : 1701/4500/500 , but I could not connect to vpn with my phone or pc. but i can see the web UI.

You need to forward port 8080 to see the web UI, just went through that myself :)


Inaccurate estimated download times

Description
Currently, Jellyseer displays estimated download times for requested titles that often differ significantly from what the actual downloader states. This discrepancy can lead to confusion and frustration among users who rely on accurate timing information to plan their downloads effectively.
Desired Behavior
We would like Jellyseer to either synchronize its estimated download time calculations with those provided by the downloader itself or implement a more accurate algorithm for estimating wait times. This would ensure that users receive consistent and reliable information regarding when their requested titles will be available for download.
Additional Context
Currently, when a user requests a title for download through Jellyseer, the estimated download time displayed can be significantly different from what the downloader actually indicates. This issue persists across multiple instances and does not seem to improve with different network conditions or download priorities.
Here is an example of the issue:
Jellyseer estimates a download time of 24 hours for a requested title.
The downloader interface indicates a remaining download time of 6 hours

.
Such discrepancies not only confuse users but also hinder their ability to effectively manage their downloads. We suggest exploring options to improve synchronization between Jellyseer's estimated times and those provided by the downloader, or enhancing the algorithm used to calculate these estimates.
Improving this feature would greatly enhance user experience by providing more accurate and reliable information, thereby reducing frustration and improving overall satisfaction with Jellyseer's download management capabilities.
Code of Conduct

[X] I agree to follow Jellyseerr's Code of Conduct

The estimated time doesn't come from the downloaded but from sonarr/radarr. These tools sync every two minutes with the actual client. There's nothing we can do about this. We don't connect to download clients and don't plan to do so.

The estimated time doesn't come from the downloader but from sonarr/radarr. These tools sync every two minutes with the actual client. And we sync to them.
There's nothing we can do about this. We don't connect to download clients.

@Nabstar3 #866 should provide a way to refresh the timings once implemented. You can follow that issue for the development progress c:

Description
Currently, Jellyseer displays estimated download times for requested titles that often differ significantly from what the actual downloader states. This discrepancy can lead to confusion and frustration among users who rely on accurate timing information to plan their downloads effectively.
Desired Behavior
We would like Jellyseer to either synchronize its estimated download time calculations with those provided by the downloader itself or implement a more accurate algorithm for estimating wait times. This would ensure that users receive consistent and reliable information regarding when their requested titles will be available for download.
Additional Context
Currently, when a user requests a title for download through Jellyseer, the estimated download time displayed can be significantly different from what the downloader actually indicates. This issue persists across multiple instances and does not seem to improve with different network conditions or download priorities.
Here is an example of the issue:
Jellyseer estimates a download time of 24 hours for a requested title. The downloader interface indicates a remaining download time of 6 hours

 . Such discrepancies not only confuse users but also hinder their ability to effectively manage their downloads. We suggest exploring options to improve synchronization between Jellyseer's estimated times and those provided by the downloader, or enhancing the algorithm used to calculate these estimates.
Improving this feature would greatly enhance user experience by providing more accurate and reliable information, thereby reducing frustration and improving overall satisfaction with Jellyseer's download management capabilities.
Code of Conduct
* [x]  I agree to follow Jellyseerr's Code of Conduct

Just curious, did you use ChatGPT to write this?

Just curious, did you use ChatGPT to write this? I mean wtf is this:

LOL, yeah I did generate this issue with ChatGPT. The description and behavior were alright in my opinion and I didn't extend my review to the additional context.

Just curious, did you use ChatGPT to write this? I mean wtf is this:

LOL, yeah I did generate this issue with ChatGPT. The description and behavior were alright in my opinion and I didn't extend my review to the additional context.

There is no "algorithm" or so. We don't calculate the estimated values. Which is why it made no sense


fix: OutOfMemory when try to indent pom.xml without default size

If you add many modules and change the indentation size to 4, when newer dependencies are added to pom.xml the current indentation algorithm causes a loop that increase memory exponentially by each dependency, to avoid this we changed to use a XML formatter.
fix #10129

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@cwichoski : thanks for reporting the issue, and for trying to fix it!
I've worked to improve the behavior of maven-model-helper, the library that we use to serialize pom.xml, so we can avoid rewriting afterwards the pom.xml: https://github.com/fabric8io/maven-model-helper/pull/151
Once the new version is released, things should be fixed with #10143


add PinchToZoom event

Hello,
I use your code with a iPad and i can't zoom with my fingers. Do you have a solution ?
Thanks

Jack seems to be very inactive lately regarding his own stuff on github, I would be surprised if he ever add new functions to it. A pinch zoom would be really great with this, even better if it was added to the jquery zoom.

+1 for this

+1

+1

Hey guys, commenting "+1" spams the issue tracker. Instead you might add your thumbs up to the issue itself.
Anyway: This module can be forked by any of you guys and add a pinch gesture. Also currently it cant be imported by modern JS/TS.


Consider adding isPast() and isFuture() methods for asserting dates

Feature summary
The new methods would be syntactic sugar for assertions similar to:
assertThat(LocalDate.now().minusDays(1)).isBefore(LocalDate.now());

Example
assertThat(LocalDate.now().minusDays(1)).isPast();
assertThat(Instant.now().plusSeconds(3)).isFuture();

We have isInThePast() but that's a date assertion so it would make sense to add them for the date/time introduced in java 8.

Hi @joel-costigliola is it possible to work on this issue?

@StefanBratanov thanks! please pick one of the task and raise a PR for it.

Awesome. Thank you @joel-costigliola, and @StefanBratanov for taking this on :)

thanks @StefanBratanov for your contribution.

thanks @StefanBratanov for your contribution.

No worries. Thank you for maintaining the library. :)


Consider adding prompt history to ChatCraft

Today, I was using ChatCraft to generate a cover photo for my blog post. Since I didn't like one of the responses, I started clicking "up" arrow out of instinct, trying to get previous command and making changes to it.
However, I realized this functionality doesn't exist in ChatCraft but could be kinda useful, especially for people using ChatCraft commands a lot.
I was wondering if we could implement a Prompt History in user's local storage, and pressing up and down arrows would allow navigating through previous prompts just like we can do in any command line interface.
We could probably limit the number of prompts that are saved (maybe 10), so we do not eat up too much memory on user's computer.
And we could also provide an option in user pref to set a customer prompt history.

We actually do support up-arrow, but in a very limited way.  I like your idea.  We don't even need to do any changes to localStorage or the database, since we already have all the messages in indexeddb, and would need to get the "previous" human message.  You could support infinite history with this.

We could do that, but the problem is many human messages do not exactly hold the text originally entered in the prompt.
This especially happens when we are using ChatCraft command. The entered prompt gets lost and only the response is stored as a Human Message.
One example is the /image command.

Even with Web Handlers, only the response is saved that is practically no use for the user to have in the prompt.

Also, while looking into this, I've noticed that we are using Human Messages for Image generation and Web Handlers. Should they be System Messages instead?

@humphd If we don't like this approach, we could also have a separate collection for our history stack, and only push prompts that are a ChatCraft command to it.
Although I think it would be better to go with the first approach for wider support.
Please let me know what you would prefer.

I'm not against trying the localStorage approach, but it becomes one more thing you have to keep in sync.
Here's another thought: what if up-arrow goes through the previous messages in the chat, no matter what type they are?  I ask because I'm not sure if "previous prompts" are useful outside of the current chat.  I often want to edit the AI responses too.
NOTE: we can't use System Messages for these, as this has a specific meaning for LLMs.


Added MuiButtonBaseProps to DrawerRailItemProps

Fixes BLUI-3122

Changes proposed in this Pull Request:

Added MuiButtonBaseProps in DrawerRailItemProps

To Test:

yarn start:showcase

do you have an example of how to use this for rail?

do you have an example of how to use this for rail?

Please checkout feature/rail-item-style-example branch in react-showcase-demo


GEODE-6861: Generify ClusterManagementService

the key change here is the signature of ClusterManagementService.list from
<T extends CacheElement> ClusterManagementResult list(T config);
to
<T extends CacheElement & RespondsWith<R>, R extends CacheElement> ClusterManagementResult<R> list(T config);
this resolved many warnings and allowed removal of clazz parameters

I understand the need to separate concerns into different interfaces, but if they are always used together, It's probably a good idea to lump them into one concept for better understanding and readability. Everywhere in the code I see T extends CacheElement & RespondsWith<R>, which indicates that we can bake RespondWith<R> into CacheElement.  If you declare CacheElement as
public abstract class CacheElement<R extends CacheElement> implements Identifiable<String>, Serializable
we can totally get rid of RespondWith and have the code a lot more succinct.

Everywhere in the code I see T extends CacheElement & RespondsWith<R>, which indicates that we can bake RespondWith<R> into CacheElement

@jinmeiliao they were initially combined in this PR; this separation was made in response to your earlier feedback "you are trying to use [basetype of T] to do a multitask, one is [rest-specific], another is to create a link between the static config and the runtime config. I would prefer you separate these two tasks."  The result of separating out what "creates a link between the static config and the runtime config" is RespondsWith.
While it is slightly more verbose in terms of character count, perhaps it's worth it for readability because it more clearly expresses the purpose of the generic type parameter?  Looking at the declaration of a request type, which is more intuitive:
public class RegionConfig extends CacheElement<RuntimeRegionConfig>
or
public class RegionConfig extends CacheElement implements RespondsWith<RuntimeRegionConfig>
?

@onichols-pivotal sounds good. My original comment is about separating the RestfulEndPoint with RespondWith. Essentially every CacheElement needs to tell what to respondWith, so it "could" be nice to combine them. but  I don't feel very strongly either way.  Let's see what @jmelchio @Petahhh @jdeppe-pivotal think about this PR.

I've read through the changes and it makes sense to me. Joris and I have been brewing another alternative to our infrastructure that we are still ironing out. For now the decisions in this PR work for me.


hackerpulp.com website is down

I would like to see this tutorial serie but the website is down. Does anyone has a backup or something?

I was able to find the articles archived on an older laptop and they have been added to the Wiki now.
I never completed the series and so there's only two articles unfortunately.


Get-AzAutomationCredential is not supported, this documentation should be updated.

the link from "New-AzureAutomationCredential" near the top of the page points to
according to https://docs.microsoft.com/en-us/powershell/module/servicemanagement/azure.service/new-azureautomationcredential?view=azuresmps-4.0.0
"This PowerShell command for Azure Automation is longer supported as of 01/23/2020. The Azure Service Management model is deprecated for Azure Automation and was disabled on that date. Please use the commands which support the Azure Resource Management model in Az.Automation."

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

ID: 2f830218-a56b-6a7e-d6e7-7736896d3f91
Version Independent ID: ae33d704-d0b7-d478-8602-cc787f706d72
Content: Manage credentials in Azure Automation
Content Source: articles/automation/shared-resources/credentials.md
Service: automation
Sub-service: shared-capabilities
GitHub Login: @SGSneha
Microsoft Alias: v-ssudhir

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

Assigning to content author @SGSneha for doc-enhancement

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary based on resourcing, so we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary. #please-close


Remove duplicate messages about CI.

MetaSmoke seems to be outputting duplicate messages relating to CI.
For example:
[ metasmoke ] ci/circleci success on cae3898: Your tests passed on CircleCI!
[ metasmoke ] ci/circleci success on cae3898: Your tests passed on CircleCI!

Permalink
This is takes up premium real estate on the screen, preventing users from acting on spam. However, this does not occur 100% of the time. This is a minor and (probably) easy to fix bug.

Wrong repo lol.

@gsquaredxc The messages come from Metasmoke AFAIK, so I think the issue is more appropriate here.


Test coverage fix

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
no

Merged. Thanks!


Fix misleading error TS2410 from issue #10601

Fixes #10601

[x] There is an associated issue that is labelled
'Bug' or 'Accepting PRs' or is in the Community milestone: see #10601
[x] Code is up-to-date with the master branch
[x] You've signed the CLA
[x] There are new or updated unit tests validating the change
[ ] You've successfully run jake runtests locally: No, tslint did not run when I checked out the project. The tests run though.

Hi @lukaselmer, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


Problem matcher can't support absolute and relative filenames at the same time

I have a problem matcher that works on the output of the Delphi compller. Sometimes it gives relative and absolute file names at the same time. When I set "fileLocation": ["absolute"]  the messages having relative paths can't be opened. When I set "fileLocation": ["relative", "${workspaceRoot}"] the messages having absolute paths can't be opened anymore.
Is it possible to define a a problem matcher that uses relative paths as long as the mentioned file exists and uses absolute paths as a fallback mode?

The idea was to file path autodetection support. Something like:
"fileLocation": ["autoDetect"]
It would check whether a path appears to be relative or absolute (e.g. starts with / or a drive letter). Would that help in your case.

That would solve my problem, yes.

We need this feature for vshaxe (https://github.com/vshaxe/vshaxe) as well - the Haxe compiler reports errors with absolute paths if the error is in a haxelib somewhere outside of the current project, and relative otherwise.

It is also required by Dlang compilers.

I am hitting the same issue using C++ with clang. Sometimes error paths are reported as relative and sometimes are absolute.
The fileLocation could be a list of paths, with absolute and relative specification, just like a C++ include path list.

Same issue with mbed compiler, it generally just prints the filename that errors are in, it would be great if it could file the file in the browse path or let me supply a list of paths to search in.

I would like to up this one more, this is quite a show stopper as there is no real workaround once you have an absolute path, for instance on a different drive.

Would anyone like to work on a PR for it.

For now, we found a workaround for this in the Haxe extension: provide two problem matchers, one of them being listed first to have priority and having a pattern that only matches against absolute paths ((?=[a-zA-Z]:|\\/)). This seems to work nicely.
The only annoying part is that "Configure Default Build Task" copy-pastes the problem matchers of contributed tasks into tasks.json, so users who have already done that in the past will have to add the additional problem matcher manually.

Somewhat related, I am building my code with cmake. I do it like this, which I believe is conventional:
mkdir build
cd build
cmake ..
make

Now the relative paths are wrong for the editor (off by the '..').
Setting the cwd in the tasks.json doesn't improve things.
         "options": {
            "cwd": "${workspaceRoot}/build"
         },

It would be nice, if one could specify a "builddir", that is used as the base for interpretation of relative paths.

MSVC sometimes outputs paths in relative paths, and sometimes in absolute path. I would like the problem matchers to try harder to find files.  If the problemMatcher is upgraded I prefer there to be no "relative" or "absolute" option and "fileLocation" to always be a path and "autoDetect" is the only option. Autodetection I can't imagine being too heavy just test if the file exists if not try as absolute, if not then try relative to project, if that fails try relative to build and if that fails then nothing much can be done. That will cover majority of the cases and will allow having it much simpler and not requiring to specify a fileLocation.

I have the same issue with problem matcher, I uninstalled all extensions except Platformio.io and C/C++ (Microsoft), but the links in the problem matcher are still broken because the path is duplicated.
Unable to open 'Datalogger.ino': File not found (file:///home/felipe/Documents/PlatformIO/Projects/Bluepill_Datalogger/home/felipe/Documents/PlatformIO/Projects/Bluepill_Datalogger/src/Datalogger.ino).
I've tied to clean compile with these three settings and the wrong path managing persists :
            "problemMatcher": {                 "base":"$platformio",                 "fileLocation": ["relative", "/"]                 // "fileLocation": ["relative", "${workspaceRoot}"]                 // "fileLocation": "absolute"
I hope you could fix this bug. Thanks for listening.
Related threads:
https://github.com/Microsoft/vscode-cpptools/issues/1915

@alexr00 thanks for addressing this.

Nice. Looking at the commit https://github.com/microsoft/vscode/commit/c5410560cf49af4fffb255b444552dcf7b74f5ce it looks like  fileLocation documentation hasn't been updated  "AutoDetect" though. Will this value be documented and available?

We usually don't write documentation until we've release a feature. I'll add documentation around the time when autoDetect is part of a release. You can try it the insiders build soon.


why is there a utilities in the root directory and the example notebooks directory?

there should only be one and it should be referenced correctly.

Because I don't know how to import utilities from the directory above example notebooks... So like the notebooks are in the cartopy directory, but I need to import utilities from two directories up? How do to that?! I guess that needs to be put in the path? I was really just trying to keep things simple in the examples.

Since none of the notebooks depend on the utilities folder, I removed it from this directory.


fix: datasource title not editable and ctas disabled on create

Description
This fixes even more issues on create datasource flow which are missed out when fixing #18600 . This fixes cases listed in the below issue.

TL;DR Create datasource flow permission driven cta fixes.

Fixes #18644
Media
https://www.loom.com/share/5555fd58ffbd44f08971acc82def770c
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manual

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] PR is being merged under a feature flag

/ok-to-test sha=933e0e5

@berzerkeer just checked the loom video attached in the description, I hope the buttons do not read as Testsssss and Savesssss like it does in the video.

@berzerkeer just checked the loom video attached in the description, I hope the buttons do not read as Testsssss and Savesssss like it does in the video.

Haha no if it was it would've been in the file changes right . Also those changes were made in EE repo anyways. So no need to worry.


[feature request] add error-awared functions like: makei_error, map_error

It's often the case when I want to construct an Array[T] or FixedArray[T] from a function that can raise error. For example, I want to construct a Tensor from a Json value:
fn main {
  match json {
    Json::Number(x) => Tensor::new(x)
    Json::Array(xs) => {
      let value : FixedArray[Tensor] = FixedArray::makei(xs.length(), fn(i) {
        Tensor::from_json!(xs[i], @json.add_index(path, i))
      })
      Tensor::stack(value)
    }
    _ => raise @json.JsonDecodeError((path, "Expected an array"))
  }
}

For now it is not possible to achieve this, since FixedArray::makei require the callback to be error-free.

@tonyfettes  Maybe the functions of all core module have to add the error feature. But i wish you do not use error feature in you code. Because this feature influences all related code if you use it. Just like now scene you face.

let value : FixedArray[Tensor] = FixedArray::makei(xs.length(), fn(i) {
        Tensor::from_json!(xs[i], @json.add_index(path, i))
      })
Tensor::stack(value)

Note you work around this using for  .. in


Catch Exception when attempting lock

When Hazelcast attempts to acquire a lock, it will block until the lock is available. If another cluster member has the lock and that member times out of the cluster, an exception is thrown by tryLock.
This fix will catch any exception (not just InterruptedException) to force a retry of lock acquisition.
See https://github.com/spring-cloud/spring-cloud-cluster/issues/24

:+1:

Merged in dffa9cd


Publish 7.5-20220504102347+0000

Publish 7.5-20220504102347+0000

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

Error while scheduling pre-tested commit, please contact the administrator.

@bot-gradle test and merge

OK, I've already triggered a build for you.


React componentDidUpdate method won't fire on inherited props change if connected to a store that didn't change

I want my component know if some library is already loaded. To know that from any context i connect it to the "library" reducer of my store. The problem is that i pass a configuration object (this.props.dataObject) to my component from the parent where the component has been called. This prop changes and "componentWillUpdate" method won't fire. If i remove the connect from the component it works as expected. I'm missing something here?
class GoogleButton extends Component {
    render() {
        if (this.props.libraries.google) {
            return
        } else {
            return null
        }
    }

    componentDidUpdate() {
        console.log('component did update')
        gapi.interactivepost.render('sharePost', this.props.dataObject)
    }
}

function mapStateToProps(state) {
    return { libraries: state.libraries }
}

export default connect(mapStateToProps)(GoogleButton)

This qualifies as a "usage" question, which should be asked on Stack Overflow.  I see that you did already post the question there, so I'll link that and close this.
http://stackoverflow.com/questions/37623628/react-componentdidupdate-method-wont-fire-on-inherted-props-change-if-connected


[DEVOPS-2522] increase default node_max_old_space_size to 6G

Just curious are we getting heap related error?

I believe that this was an issue when we updated node to 16, the build would run into heapspace errors here and there. If there are smarter solutions, happy to try, I believe @ppratikcr7 and I tried a few things to resolve that and found that this common solution was the simplest fix.

should we delete the line ENV NODE_OPTIONS=--max_old_space_size=4096 from the Dockerfile so there is not a conflict?
Also, after this goes in, what will we need to do in our AWS environments to process these terraform change? Is this functionally equivalent to an elastic beanstalk configuration getting updated by hand?

@danoswaltCL  , no need to change it , its acting as default value and overwritten by the beanstalk env vars
I did the change manually , so this PR to catch the drift only

ah ok i see that update made in prod config, ty @TamerElfeky


CM bugfix

Bug found by @dangershony
Bug setup: we start the node at height 100 and advance 1-2 blocks to height 102. After that we find a better chain with fork at 80 and length of 120. We rewind but we don't have blocks that are being rewinded in memory and therefore disconnectedBlocks list will contain nulls for blocks that are missing.
Then we find out that that better chain is invalid and we try to reconnect old blocks. But we can't- we have nulls in the list.
Fixed by loading from DB in case it's not found in cache.

looks ok

Reverted last change.
@Aprogiena tx


Too many redirects?

Hi,
Don't know can anybody reproduce this error or is this due some bad configuration on my laravel, but, I have routes like this
Route::group(['middleware' => 'role:parent'], function()
{
Route::get('parent', 'ParentController@index');
Route::get('parent/invoices', 'ParnetController@invoices');
});
Everything works fine expect when I'm on url for example http://mydomain.com/parent/invoices and leave it open for few hours (until session expires). Then I refresh page, i got browser error that there were too many redirect and that browser cannot display page.
If I got to root of site and login as "parent", everything works fine again.
Is this due some redirects in role middleware or something else?

Have a look at documentation. https://github.com/romanbican/roles#middleware


chore: sort package.json

Summary
This uses sort-package-json to order the contents in a consistent order, which I find helps when looking over the file.
Pull Request checklist

[x] ~Add/update test to cover these changes~
[x] ~Update documentation~
[x] ~Update CHANGELOG file~

Makes the blame a lot less useful.
Really worth it?

Yes I think it is - until it's done, then you'll never get the benefit and then afterwards the blame will recover as further changes are made; you can also --ignore-rev to exclude the sorting revision (you can also create a .git-blame-ignore-revs to have the GitHub viewer also ignore it, though I typically don't).
For me this falls into the same camp as tools like prettier: they'll always add noise to the git blame when they're first introduced but that should mean getting them in ASAP rather than not using them all together 🤷

@G-Rath add in:

.git-blame-ignore-revs to have the GitHub viewer also ignore it, though I typically don't).

and I'll merge it!

@justin808 that's not possible since the commit will change when this gets merged - it has to be in a follow up PR

@G-Rath OK! go at it with the follow up!


can i use vs team services whitin visual studio

VSCode Version:1.8.2
OS Version:ubuntu 15.10

Steps to Reproduce:

sign up for team services
enjoy integrations in vscode

An extension was published recently for this https://marketplace.visualstudio.com/items?itemName=ms-vsts.team


Client sided block support

It would be nice to have client sided blocks support.
Use cases:

Restricted area access; bridge for people with access, air for people without.
Filters; would allow to change all blocks for a player (i.e. custom status effect which turns the whole world into a rainbow variant)

You can do client sided blocks via the block change or multi block change packets.

I am 100% for this, we need an API for this, per-player block modification is really useful
It should be considered whether or not to make these changes persistent, as persistent changes would be more difficult since the server would have to have a copy of the modified chunk in memory.

I mean the api would just be like player.setBlock or sendBlock, that's about it lol

Block change packets won't work since right clicking will make it dissapear

This will have to wait for the instance abstraction rework.
#1488


feat: supporting android and ios rp origins

While using the lib, I noticed that the rpOrigin, as returned by the android native fido2 client attestation response, is considered invalid.
Made this change for the lib to support android FacetIDs as origins.
In depth explanation: https://fidoalliance.org/specs/uaf-v1.0-id-20141122/fido-appid-and-facets-v1.0-id-20141122.html#the-appid-and-facetid-assertions
Also added support for the iOS FacetID prefix, as specified in the fido2 docs above.
Empirically, our experiments with native iOS clients resulted with https rp origins - so the docs might be outdated, but I added the FacetID support just in case.

Codecov Report
Patch coverage: 80.00% and project coverage change: -0.03 :warning:

Comparison is base (aef3754) 92.95% compared to head (72d84d1) 92.93%.

:exclamation: Current head 72d84d1 differs from pull request most recent head 8a21561. Consider uploading reports for the commit 8a21561 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #121      +/-   ##
==========================================
- Coverage   92.95%   92.93%   -0.03%
==========================================
  Files          16       16
  Lines        5992     6002      +10
==========================================
+ Hits         5570     5578       +8
- Misses        422      424       +2

Impacted Files
Coverage Δ

lib/toolbox.js
89.77% <80.00%> (-0.28%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

Hey @TomerDavid-Transmit thanks for the contribution! Are you trying to achieve the same thing as in #122 ?

@JamesCullum yes, except for the prefix I think we should look for. #122 searches android: and ios: as prefixes, and I think we should search for android:apk-key-hash: and ios:bundle-id: for specificity.
was just working on adding the iOS support + tests to this PR

Very much needed addition!

Finally!

looks like an encompassing code.
good one :)

LGTM 👍  This one is slightly more complete than #121

Nice change!

@Hexagon I think 3.4.0 would make more sense as new feature, as previously we didn't support it.
Will merge and do it - thanks!


[storage-credits] Support locking tokens

Add a new method lock(did: string, purpose: string, amount: uint). This locks the specified number of tokens for a specific purpose for a specific did.
For example, a did may lock 1,000 VDA tokens for the purpose of early-access-verida-ai
Requirements:

The lock() request must by signed by the did. Throws invalid request signature
The method may accept VDA tokens sent in the requests, otherwise it will allocate tokens previously deposit()ed into the did’s credit.
The method may return insufficient funds error if there are not enough tokens included in the request or there are not enough token credits in the smart contract for the requesting did.
A did can call lock multiple times to add more tokens.

Add a new method locked(did: string, purpose: string). This returns the number of tokens locked by a did for a specific purpose.
Requirements:
If there are no tokens locked for the specified did and purpose, throws did has no locked tokens for this purpose.
Add a new method unlock(did: string, purpose: string, withdrawWallet: string | undefined). This unlocks all the tokens locked for the specified purpose.
Requirements:

The unlock() request must be signed by the did. Throws invalid request signature
The method may return lock does not exist error.
If withdrawWallet is specified, the amount of tokens should be withdrawn to the specified wallet, otherwise the tokens will be deposited as credit for the did.

@tahpot Designed the lock() function as following:
  function lock(
    address didAddress,
    string calldata purpose,
    uint amount,
    bool withDeposit,
    bytes calldata requestSignature,
    bytes calldata requestProof
  ) external;

locked(did: string, purpose: string)

How about to return 0 for unknown purposes instead of throwing error?

** unlock() **
I designed the function as following:
  function unlock(
    address didAddress,
    string calldata purpose,
    address to,
    bytes calldata requestSignature,
    bytes calldata requestProof
  ) external;

The to address can't be optional. If we don't specified the recipient wallet address, the transaction can be attacked by the front-running attackers.
In your design, the withrdawAddress is optional, in case, we should unlock to the msg.sender correct?
The attackers monitors the mempool and clone a user's unlock transaction and send it with more gases. The contract will unlock the tokens to the attacker.

Also, I think we need a function get the list of registered purpose for a given DID.
function getPurposeList(address didAddress, uint pageSize, uint pageNumber) returns (string[] memory)

locked(did: string, purpose: string)

How about to return 0 for unknown purposes instead of throwing error?

That’s fine

locked(did: string, purpose: string)

How about to return 0 for unknown purposes instead of throwing error?

That’s fine

** unlock() ** I designed the function as following:
  function unlock(
    address didAddress,
    string calldata purpose,
    address to,
    bytes calldata requestSignature,
    bytes calldata requestProof
  ) external;

The to address can't be optional. If we don't specified the recipient wallet address, the transaction can be attacked by the front-running attackers. In your design, the withrdawAddress is optional, in case, we should unlock to the msg.sender correct? The attackers monitors the mempool and clone a user's unlock transaction and send it with more gases. The contract will unlock the tokens to the attacker.

Aren’t we protected from this because the request signature includes the address?

Also, I think we need a function get the list of registered purpose for a given DID.
function getPurposeList(address didAddress, uint pageSize, uint pageNumber) returns (string[] memory)

Agree

unlock()
If we don't specify the recipient address, the transaction'd be as following:

unlock(didAddress, purpose, requestSignature, requestProof)

If there is no recipient address, we'd unlock the token to the transaction caller.
By the way, the attacker hook this transaction information from the mempool before the transaction completed.
The attacker will submit the same transaction with higher gas.
The contract only verify the didAddress and purpose with requestSignature and requestProof. There is no logic that checks the transaction caller.
So, the transaction of the attacker is completed first and the original transaction will be reverted.

So, we should include the recipient address in the parameter.
unlock(didAddress, purpose, recipient, requestSignature, requestProof)

In case, though any attackers clone this transaction, the tokens are unlocked to the recipient address.

unlock()
If we don't specify the recipient address, the transaction'd be as following:

unlock(didAddress, purpose, requestSignature, requestProof)

If there is no recipient address, we'd unlock the token to the transaction caller. By the way, the attacker hook this transaction information from the mempool before the transaction completed. The attacker will submit the same transaction with higher gas. The contract only verify the didAddress and purpose with requestSignature and requestProof. There is no logic that checks the transaction caller. So, the transaction of the attacker is completed first and the original transaction will be reverted.

The tokens should never be released to the blockchain wallet caller. If. i wallet address is specified, the tokens are added as credit to the DID.


Problem installing tileboard

Hi
I created the www/tileboard folder, altered the config.exemple to config.js, tried to go to the default page.
I get connection refused, with this page, http://prntscr.com/kxtco4
As far as i get it should bring up a login screen, yes?
Config is as follows, http://prntscr.com/kxtdsx
Can you help please?
Best regards
Nuno Reis

For some reason Chrome is redirecting you to localhost. Try clearing browser's cache.

Hi
Done that, ctrl+F5 after,  not working, i am using Hass.io in a VM, so the localhost in the base config.js, should work.
Altered to ip as in picture, same error.
Tought it could be some config missing in yaml file for HA, setup http: manually, with IP:port and API password, same error.
:\

404 for custom.css is not an error. I really don’t know how you have configured VM but it should not work with localhost.

What if you open the page in Firefox?

Well in Edge browser works.
Chrome issue, then.
Tileboard is stored in the vm in local folder so local shouldwork, but i get that you connect to the HA so in fact wont work with localhost, either way i have it setup with IP.
Like i said, with Edge works fine.
Browser related bug? Any ideias?

No clue why Chrome redirects to localhost honestly. I have mine running in docker and everything works fine.

I have same problem in safari, Firefox and chrome(

@ngreis @tsvdonsk what about using predefined api tokens?
https://github.com/resoai/TileBoard/blob/master/config.example.js#L18
At least it should work without redirections

Hmmm, I get redirected to localhost even on firefox.
Can't even try Tileboard... :(

On the other hand, changing localhost to hassio.local (or however else you access your server) on config.js:
...
   serverUrl: "http://hassio.local:8123",
   wsUrl: "ws://hassio.local:8123/api/websocket",
...

Seems to work for both Chrome and Firefox (plus, cleaning your file cache (CTRL+SHIFT+DELETE))

That's a weird issue. No matter what you put in the config, chrome arbitrarily decides to redirect to http://localhost:8123/local/index.html?oauth=1&code=<...> when it should redirect to http://hassio.local:8123/local/index.html?oauth=1&code=<...> :/

Update: i found the error. The it is not a redirect at all. It's just that the default config entry pointing to
localhost doesn't get deleted when you refresh or(!) delete the cache of Chrome. In Chrome under "Clear browsing data" you have to select "Content settings" and likely "Hosted app data" to get this to update and then it works with either the mdns or the ip in serverUrl and wsUrl.


Cannot read properties of undefined (reading '__nuxt_plugin')

branch: unocss
https://github.com/productdevbook/productdevbook.com/tree/unocss

hi antfu, iam update fixed, but dont change color and dont hot reload

hmm, is fixed :D , thats true ?

iam remove windicss but, dont color change. Does windicss have to be installed ?

This is a bug of Nuxt SSR, I am working on it.

You can now use @unocss/nuxt for that. (the FOUC is still in progress)


Regaining Focus

When I load up the demo level (I had to create a default C++ FPS project and then copied over the HTML and Content folders after setting up the plugin) I can click on the UI windows only by using shift+F1 and then afterwards I'm unable to regain focus and pan around without exiting the play. Anyone know what i'm doing wrong? Is it a problem with how I set it up or with running in the editor?

@thechiclayano You need to set ShowMouseCursor to true.

@buddytex, that did not help. As soon as you click once, anywhere, inputs act weird and look like registering once every 20 clicks. Any other idea?


Ctrl+click to withdraw from mats pane

Is there a particular reason I can't withdraw from the materials pane with Ctrl+click? It seems like that's how it should work.

its easy to create hooks for things that the game is already using for player interaction, which is why item clicking in the normal panes works - we just hook into the function that allows you to use cons or kits straight from the chest and its all good. The material pane doesn't have the same behaviour, so its trickier to get it done

Next release, https://github.com/HasKha/GWToolboxpp/commit/2a9bb4a8cab7787b220cbf45567a706462362499

Next release, https://github.com/HasKha/GWToolboxpp/commit/2a9bb4a8cab7787b220cbf45567a706462362499


Sign request on server, direct upload on browser, without client library

Hi,
I'd like to use cloudinary without your client side library. With nodejs cloudinary.utils.sign_request I can get a signature, but now what? How do I need to design my post request? What url do I need to target my post request to?

Would be happy to know how as well.

@Haocen,
This was also discussed on a different thread:
https://github.com/cloudinary/cloudinary_js/issues/81
If you run into some specific issues, please feel free to open a support ticket, share some context of the code you're using, and we'll be happy to assist.

@taragano That should be in your official documentation, much more comprehensive.

Yeah, I managed to do it and failed to update this question accordingly. So again, for signed uploads:
First I calculate the signature on server side using
cloudinary.utils.sign_request({ timestamp: Math.round(new Date().getTime()/1000) });
// returns { signature, api_key, timestamp }

On the browser, I call this method through my API and attach the properties as fields like here (with superagent)
const request = superagent
        .post("https://api.cloudinary.com/v1_1/<cloud name>/auto/upload")
        .on('progress', ({ percent }) => console.log(`${percent}%`))
        .field('api_key', data.api_key)
        .field('signature', data.signature)
        .field('timestamp', data.timestamp);

And done!

Btw. I agree this could be made more clear in the docs.

@Haocen @bkniffler,
Thank you both for your feedback. We'll definitely do our best to make this clearer in our docs.


Add lxd-init-network and lxd-init-storage config keys

Closes #105

@simondeziel test failures look legitimate

@stgraber should be good now.


reference implementation of #1016

sshagent plugin:

manage setup of sshagent in bash session:

force opinionated agent management based on ~/.ssh/ file

support for shared file systems on different host (file name contains hostname)

check env for any live/dead agents
if alive - use it
otherwise - spawn our own

Signed-off-by: Max Kovgan<EMAIL_ADDRESS>

Thanks!


Upgrade to dotnet core 3.1

References

Migrate from ASP.NET Core 3.0 to 3.1 | Microsoft Docs

@adamhathcock Could you confirm the following?

Workflows Waiting for Status in GitHub - CircleCI


Add code for creating Generic ML devices to NPU starter sample

Also renamed variables/modified codeflow to improve readability.

@microsoft-github-policy-service agree company="Microsoft"


Kubernetes  pod scaling

Recently we tried container monitoring with Prometheus.
Is there any approach where we can scale  container/pod using cpu or memory metric in prometheus and alertmanager?

You could write an alert at which you want to scale (ex: CPU > 80% FOR 10mins).
Run Alertmanager and use a webhook receiver to write your scaling logic.
Usage related questions are best limited to: https://groups.google.com/forum/#!forum/prometheus-users

Will it work when it recover it should do scale down.
Can you please refer any blog or mail thread on this subject.

On Tue, Mar 28, 2017 at 4:22 PM, prasenforu<EMAIL_ADDRESS>wrote:

Will it work when it recover it should do scale down.

Yes, you can also configure the webhook notifier to send notifications on
resolved alerts. See https://prometheus.io/docs/alerting/configuration/#
<webhook_config>

Can you please refer any blog or mail thread on this subject.

For example, https://www.robustperception.io/audio-alerting-with-prometheus/

Thanks for response.
I am more interested as follows,
When cpu high it should trigger one webhook.
Which I am able to do by setting receiver webhook.
But when it resolved it should trigger another webhook.
Which I am not able to configure.
Plesae advice.

We don't support different receivers for firing/resolved for the same alert, but you could emulate it with two alerting rules, one that fires when something is bad and one that fires on the opposite condition (everything is good), and then notify different webhooks for each when they are firing. For example:
ALERT InstanceDown up{job="myservice"} == 0
[...]

ALERT AllInstancesUp avg(up{job="myservice"}) == 1
[...]

Not sure how great that pattern is though :)

Closing, as questions are for the users mailing list, not GitHub issues. See https://prometheus.io/community/.

If I create another alert for scale down.
Like cpu < 70%
But in that scenario it is allways true if I do then it will scale down to zero than mean no pod running.
My question any alternative approach or does it support alert as range.
CPU > 40% but CPU < 70%
in a single alert.

@prasenforu If you ask your question on the mailing list I may be able to help further.


use Travis!

Prepare proper configuration for Travis.
Main problem - gt.m installation.

Done. Travis checks the build result.


Add range validators

coverage: 93.194% (+0.08%) from 93.11% when pulling 186a979a194892d694b4a8a5480f1dda1aec2441 on feature/add-range-validators into cd37964f5436f65c1028ef494561c73bc066b397 on main.


Suport for interactive Python shell code?

It is quite often markdown contains interactive Python shell code like:
>>> print("Hello World!")
Hello World!
>>> def add(a, b):
...     return a + b
...
>>> add(2,3)
5

Currently, I got following exception for these blocks in my README.md.
invalid syntax (<string>, line 1)

Can you add support to test it? I expect the behaviour to be same as  pytest --doctest-modules, as if the code block is a function's doc-string.

Interesting idea! I don't have any spare time to devote to this right now, but you can either submit a PR or sponsor me so I can move this into my working hours.

I came here looking for this too, I just thought it worth mentioning that Github supports syntax highlighting of interactive codeblocks with a different code than normal Python, pycon, e.g.:
```pycon
 >>> "Hello " + "stranger"
 'Hello stranger'
```

Renders as:
>>> "Hello " + "stranger"
'Hello stranger'


Autorest az 1.8.0 failing with TypeError: extension.updatePythonPath

Hello,
I am trying to generate client libraries using the following command
autorest --az --version=3.0.6370 --azure-cli-extension='' --sdk-flatten <path-to-readme.md>
This fails with the following error.
AutoRest code generation utility [cli version: 3.6.2; node: v14.17.3]
(C) 2018 Microsoft Corporation.
https://aka.ms/autorest
info    | AutoRest core version selected from configuration: 3.0.6370.
   Loading AutoRest core      'C:\Users\djaiganesh\.autorest\@autorestcore@3.0.6370\nodemodules\@autorest\core\dist' (3.0.6370)
   Loading AutoRest extension '@autorest/az' (1.7.2->1.7.2)
   Loading AutoRest extension '@autorest/python' (5.4.0->5.4.0)
   Loading AutoRest extension '@autorest/clicommon' (0.6.1->0.6.1)
   Loading AutoRest extension '@autorest/modelerfour' (4.15.421->4.15.421)
TypeError: extension.updatePythonPath is not a function
Process() cancelled due to failure

I noticed this fails during the dependency installation of autorest/python 5.4.0.
I tried all the workarounds mentioned in some other issues like installing @autorest/python 5.4.0's dependencies in its virtual environment. But none of this seems to fix this issue.
I also tried older versions of az (1.7.0, 1.6.2), none of which worked.
Please let me know if there is some workaround that actually works. Or if there are any patch available?
Thanks

@jjdharini22 Did you solve the issue? I encountered the same problem

+1 looking for response

@edyoung


Update index.js

Hello @Caloy9999, welcome to GitHub! I see that you are uploading your client secrets in a public pull request 😱, this is dangerous if someone uses your credentials without your consent.
Anyway, if you want to make my bot working, you only need to duplicate the file .env-template   with the name .env and fill the variables.
If you need more help, you can open an issue , good luck and happy coding 🥳


Add modify_passenger_position power type

This PR adds a modify_passenger_position power type, which as its name suggests, modifies the position of the passenger(s) of the entity that has the power. I'm not totally sure whether the offset field being a double vector would suffice or if there should be offset_AXIS_modifier(s) fields instead (or both) :thinking:
Fields

Field
Type
Default
Description

bientity_condition
Bi-entity Condition Type
optional
If specified, the position of the passenger will only be offset if either or both the passenger(s) (actor) and the entity that has the power (target) fulfill this bi-entity condition.

offset
Vector

Determines the offset to apply to the position of the passenger(s).

Making this PR a draft for now. I might also include a modify_riding_position power type (like in eggolib)


updated the button component to better render aria-label

Fixes how the aria label was being rendered by the button component
https://github.com/Capgemini/dcx-react-library/issues/618

The aria-label should default to the label if not specified as voice activation users won't know what type of button it is or that it has "-button" added afterwards.
The aria-label attribute should always be rendered if a value is given, it doesn't make sense to hide it if the label property is being used.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (be71118) to head (9f1b1e8).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@                     Coverage Diff                     @@
##           epic/accessibility-issues      #619   +/-   ##
===========================================================
  Coverage                     100.00%   100.00%
===========================================================
  Files                             85        85
  Lines                           1512      1511    -1
  Branches                         527       526    -1
===========================================================
- Hits                            1512      1511    -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


build(anoncreds): remove node package from deps

Codecov Report

Merging #1339 (c2586e1) into main (e14d853) will increase coverage by 9.19%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1339      +/-   ##
==========================================
+ Coverage   77.28%   86.47%   +9.19%
==========================================
  Files         719      775      +56
  Lines       16899    18375    +1476
  Branches     2841     3120     +279
==========================================
+ Hits        13060    15890    +2830
+ Misses       3836     2478    -1358
- Partials        3        7       +4

Impacted Files
Coverage Δ

packages/askar/src/utils/askarKeyTypes.ts
100.00% <0.00%> (ø)

...ages/openid4vc-client/src/OpenId4VcClientModule.ts
100.00% <0.00%> (ø)

...es/tenants/src/context/TenantSessionCoordinator.ts
100.00% <0.00%> (ø)

...ges/openid4vc-client/src/OpenId4VcClientService.ts
82.19% <0.00%> (ø)

...ckages/tenants/src/services/TenantRecordService.ts
100.00% <0.00%> (ø)

packages/bbs-signatures/src/index.ts
100.00% <0.00%> (ø)

packages/core/src/utils/mixins.ts
100.00% <0.00%> (ø)

samples/extension-module/dummy/handlers/index.ts
100.00% <0.00%> (ø)

packages/askar/src/utils/index.ts
100.00% <0.00%> (ø)

...s/extension-module/dummy/repository/DummyRecord.ts
95.00% <0.00%> (ø)

... and 208 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Does support app dir?

No, Unfortunately NextUI does not support app dir, only using use client

@Siumauricio  looks like there is a feature for nextui with app router -> https://github.com/nextui-org/next-app-template.
Any plan to upgrade to the latest App Router in this awesome template?


Install gcc 7.3.0 on all Linux build machines

Oracle is starting to build with gcc 7.3.0 so we should install this on our Linux build machines.
Ref: https://bugs.openjdk.java.net/browse/JDK-8200375
Source: ftp://ftp.mirrorservice.org/sites/sourceware.org/pub/gcc/releases/gcc-7.3.0/gcc-7.3.0.tar.xz
ccache update source: https://www.samba.org/ftp/ccache/ccache-3.4.2.tar.xz

My current plan is to build from source on each of the three platforms (Linux on x64, ppc64le and s390x) with --prefix=/usr/loccal/gcc --program-suffix=-7.3 then create a tarball of the /usr/local/gcc directory that we can use to deploy via ansible in order to save the large amount of time to compile from source on each ansible run.

Build machines for the compiler:
x86: build-joyent-centos69-x64-1
ppc64le: build-osuosl-centos74-ppc64le-2
s390x: build-marist-rhel74-s390x-1

How are these tarballs going to be more convenient than pulling from a repo?  What is the plan for storage/access for users of the playbooks?
Related PRs #441 (ubuntu) and #442 (centos)

@jdekonin Are you aware of any official (trustworthy) repo for gcc 7.3.0?

Not one source for all platforms I don't no.  There might be architectures and operating systems that won't have pre-built packages, that we will need to compile from sources, sles11 for example.  But from what I have seen there are pre-built packages for x,p,z,arm for ubuntu, redhat and centos available.  I don't see why we wouldn't take advantage of installing the gcc7 (and support packages) just as we would wget.

The gcc 7.3 requires a newer linker - ld - version (at least on CentOS 6 and RHEL 6) which is included in the devtoolser-7 packages, see devtoolset-7-binutils. devtoolset-7 would resolve and install all required dependencies.
It also makes it easier to switch between the gcc 4.8 and 7.3 toolchain environments as they will coexist for awhile.

@jdekonin Where are the RHEL/CentOS packages you've seen for p/z?
@vsebe I'm aware of the devtoolset-2 packages but I'm not sure if they're available for non-x86 architectures and I'm nervous about having different setups on the machine across architectures

I'm not sure if our RHEL7/s390x machines are configured in a way that will allow that.
But we are using CentOS on ppc64le, so my preference would likely be to do it consistently unstalling under /usr/local/gcc from source across platforms, since unless someone doing it for themselves on anything other than x86-64 (where they could get it from a CentOS RHEL or similar) has a RHEL subscription it will prevent them from accessing the devtoolset repos.

(If - as mentioned by @vsebe earlier - we need an additional later ld that will impact the solution for x86-64 where we build on v6, so on there it may make sense to do it via devtoolset-2 even though the installation is could be more fiddly for the end user)

@sxa555 @jdekonin - For RHEL7/ppcle the IBM Advanced toolchain for Linux on Power provides a self-contained toolchain, similar to devtoolset-7 for x86. It can be installed via yum, rpm or manually (using  Advance Toolchain (AT) downloader tool). See detailed instructions on: https://developer.ibm.com/linuxonpower/advance-toolchain/advtool-installation/.
Advance toolchain 11.0-3 comes with:

gcc 7.3.1 20180207 (Advance-Toolchain-at11.0) [ibm/gcc-7-branch revision 257393](gcc, c++/g++, libstdc++-v3)
GNU Binutils - 2.29
performance and debugging tools
The Release notes provides further details.
It has a modulefile that enables the environment to gcc 7.3. This would allow easily to switch between gcc 4.8 and gcc 7.3 environments.

It installs in  /opt like devtoolset-7 on x86.
To enable the gcc 7.3 environment just run: module load at11.0 (devtoolset-7: scl enable devtoolset-7 bash). To disable and return to the system environment run:  module unload at11.0 (devtoolset-7: exit the bash to return to original environment).

Looks like the advanced toolchain is across operating systems too, with the exception of ubuntu18.

Hmmm wasn't aware of that toolchain before, although that would mean we ended up with three different solutions - one for each of x86, ppc64le and s390x

I'll add my 2c on my philosophy of maintaining packages for build/test machines:
Build and Test hosts should have a known, good set of packages at set versions that we can produce repeatable builds from.  That typically means we have to protect ourselves against O/S package updates due to security and stability patch cycles.
Philosophy 1: Never have the build / test scripts rely on default packages which can get upgraded underneath them!
Proposed Solution(s):  Install versioned packages in a consistent (within each platform) non-default location.  Ensure that the build/test scripts reference those versioned libs directly or create symlinks so that if the regular package is called it calls the versioned one we want instead (this assumes package managers don't override the symlink when upgrading the default package).
Philosophies 2&3: Prefer the official (or defacto) package repository for packages on a per platform as they're likely to be better tested and more trustworthy.  Only consider a cross platform solution if it is an official or defacto std.
Proposed Solution(s):  If ansible can support the official / defacto repository for a package for a particular platform, then use that. Else use a cross platform solution if there's strong defacto support for it. Else build the package from source (making sure that we checksum / security check the source) and store the resulting binary (with checksum) on our own repository (which we need to decide whether it is CI, GitHub or Ansible Tower).
Thoughts on that as a policy?

While ideal, I think (1) May be difficult since that would imply not allowing things like make to be upgraded.
Otherwise it seems reasonable, but that means not using the IBM advance toolchain as its' not an "official/defacto" repository, so we're down to source builds on pLinux/zLinux.  At the moment the adoptopenjdk RedHat build machines have been tested using gcc 7.3 built from source on x/p/zLinux although we should be able to switch the x86-64 ones to using the devtoolset-7 repository if we go down that route.

@sxa555 I think using IBM's toolchain is OK.  I've updated Philosophies 2&3.

All the main architectures are done, so I'll close this and leave #475 which is still awaiting an arm32 version


fix:Make None return all intent results

closes: microsoft/BotFramework-Composer#8121
#minor
Description
If you triggered None intents, they were not logging all the results from the recognizer.

Pull Request Test Coverage Report for Build 959415032

5 of 6 (83.33%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.008%) to 85.426%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

libraries/botbuilder-ai-orchestrator/src/orchestratorRecognizer.ts
5
6
83.33%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

Totals

Change from base Build 952706112:
0.008%

Covered Lines:
19308

Relevant Lines:
21503

💛  - Coveralls

@daveta, is this still in progress?


Sentinel-2 phoebus files

Still seeing these files causing issues:
Error while reading: /vsis3/EODATA/Sentinel-2/MSI/L2A/2018/08/20/S2B_MSIL2A_20180820T093029_N0208_R136_T34TES_20180820T120528.SAFE//dpc/app/facilities/PHOEBUS-core/PHOEBUS-core-3.4.317/DPC/exec/data/share/wp_in_progress/fasttrack_20180820_4/app_data/step_step_EUP_FastTrack/launch_8/.33099_1534783898789/.S2B_OPER_MSI_L2A_TL_MPS__20180820T120528_A007598_T34TES_N02.08/GRANULE/L2A_T34TES_A007598_20180820T093530/IMG_DATA/R20m/T34TES_20180820T093029_SCL_20m.jp2
j-6e637a64b9fa41f9b92f3de8d0c1c1db
https://finder.creodias.eu/oldresto/api/collections/Sentinel2/search.json?box=21.657597756412194%2C46.02854700799339%2C21.95285234099209%2C46.23461502351761&sortParam=startDate&sortOrder=ascending&page=1&maxRecords=100&status=0|34|37&dataset=ESA-DATASET&productType=L2A&cloudCover=[0%2C95]&startDate=2018-03-01T00%3A00%3A00Z&completionDate=2018-10-31T23%3A59%3A59.999999999Z returned

Another one:
Lost task 20.3 in stage 32.0 (TID 1123) (<IP_ADDRESS> executor 10): java.io.IOException: Error while reading: /vsis3/EODATA/Sentinel-2/MSI/L2A/2018/06/03/S2B_MSIL2A_20180603T101019_N0208_R022_T32UQB_20180603T130307.SAFE//dpc/app/facilities/PHOEBUS-core/PHOEBUS-core-3.4.317/DPC/exec/data/share/wp_in_progress/fasttrack_20180603_5/app_data/step_step_EUP_FastTrack/launch_7/.14103_1528036131394/.S2B_OPER_MSI_L2A_TL_SGS__20180603T130307_A006483_T32UQB_N02.08/GRANULE/L2A_T32UQB_A006483_20180603T101329/IMG_DATA/R20m/T32UQB_20180603T101019_SCL_20m.jp2

The first example seems like corrupt data. The following features where captured on the same second, and have the same area code, however, they have a different geometry.
S2B_MSIL2A_20180820T093029_N0208_R136_T34TES_20180820T120528.SAFE
S2B_MSIL2A_20180820T093029_N9999_R136_T34TER_20221025T001026
Probably, the processing baseline 2.08 can be ignored, as this is the one that contains invalid URLs.
This could be the same issue as in: https://github.com/Open-EO/openeo-opensearch-client/issues/15


Unable to ingest metrics from prometheus

Hello!
I am trying to follow the README. I have created a service account with organizationadmin and metricsViewer and populated the .my_env with the api keys and secret.
By looking into the logs of the prometheus feeder i can see that its failing the readiness check:
This is parts of the .my_env i think will be relevant.
 METRICS_API_SERVER_URL=api.telemetry.confluent.cloud/v2/metrics/cloud/export
#  METRICS_API_SERVER_URL=http://localhost:9091

METRICS_API_SERVER_ENABLE_AUTH=True
# METRICS_API_SERVER_ENABLE_AUTH=True

#HTTPBasic is the only supported auth type as of now. Do NOT change this
METRICS_API_SERVER_AUTH_TYPE=HTTPBasicAuth

METRICS_API_SERVER_AUTH_USERNAME=xxxxx
# METRICS_API_SERVER_AUTH_USERNAME=admin

METRICS_API_SERVER_AUTH_PASSWORD=xxxx
# METRICS_API_SERVER_AUTH_PASSWORD=password

#Do NOT change this if you will be using the provided docker-compose file
CHARGEBACK_SERVER_URL=http://prometheus_for_chargeback:9090

I also looked into the logs of the chargeback-handler application:
Traceback (most recent call last):
  File "/app/main.py", line 23, in <module>
    execute_workflow(arg_flags)
  File "/app/helpers.py", line 35, in add_entry_exit_logs
    ret = func(*args, **kwargs)
          ^^^^^^^^^^^^^^^^^^^^^
  File "/app/workflow_runner.py", line 134, in execute_workflow
    ccloud_orgs = CCloudOrgList(
                  ^^^^^^^^^^^^^^
  File "<string>", line 4, in __init__
  File "/app/ccloud/org.py", line 225, in __post_init__
    temp = CCloudOrg(
           ^^^^^^^^^^
  File "<string>", line 4, in __init__
  File "/app/ccloud/org.py", line 100, in __post_init__
    self.billing_handler = CCloudBillingHandler(
                           ^^^^^^^^^^^^^^^^^^^^^
  File "<string>", line 8, in __init__
  File "/app/data_processing/data_handlers/billing_api_handler.py", line 73, in __post_init__
    self.read_all(start_date=self.start_date, end_date=end_date)
  File "/app/helpers.py", line 35, in add_entry_exit_logs
    ret = func(*args, **kwargs)
          ^^^^^^^^^^^^^^^^^^^^^
  File "/app/data_processing/data_handlers/billing_api_handler.py", line 158, in read_all
    temp_data = [
                ^
  File "/app/data_processing/data_handlers/billing_api_handler.py", line 161, in <listcomp>
    BILLING_API_COLUMNS.env_id: item["resource"]["environment"]["id"],
                                ~~~~^^^^^^^^^^^^
KeyError: 'resource'

Have i misconfigured something?

Issue 1
Looks like there was some newer billing RESOURCE TYPE added to the Billing API response. I can definitely take a look at it. Will it be possible for you to enable DEBUG log and post the log file somewhere? Make sure you redact your keys and secrets from the log file before you post it.
If you are running it via docker compose, you can update the LOG value to DEBUG here and the next line to True and restart the docker compose.
P.S: I am working on making the Log value configurable via the env file as well, so that its value can be changed as needed.

Issue 2

METRICS_API_SERVER_URL=api.telemetry.confluent.cloud/v2/metrics/cloud/export

Also, its mentioned in the docs that the code does not connect Metrics API directly. You will need to provide it with URL of the Metrics API Prom server or use the chargeback prom server url if you dont have scraping for Metrics API setup.

@vSivarajah93 : I have fixed an edge case for Billing API fetch use case where some of the required parameters were missing. Please try out v1.0.4 and let me know if you still see the error.