[WIP] Simplify status reporting of sync errors

Combine the logic that updates the sync status to report both
the remediator and applier errors at the same time.
Combine the call to SetSyncStatus with the logic that updates
the cached in-memory state.
These changes prevent unnecessary updates and avoid loss of errors.
Fix TestConflictingDefinitions_RootToRoot to have correct
expectations. When the webhook is enabled, the 1st RootSync to
apply a resource won't report any conflict errors, because the
2nd RootSync won't be able to adopt the resource. The webhook will
cause the apply to error, even with the applier configured to
AdoptAll.

/hold
Turns out I unintentionally deleted the cross-RootSync conflict reporting in this PR. While this functionality is of debatable value, it does help compensate for the flakiness of errors from the applier, which are deleted before each apply run.
So after talking to @nan-yu and @janetkuo, we decided to remove cross-RootSync conflict reporting, but only after adding error merging to the applier. Error merging will hold onto apply errors until we see an event like the one that caused it. This will avoid loss of errors (including conflict errors) while the applier is running. Afterwards, we can continue this PR to remove the cross-RootSync conflict reporting.

/retest

/unhold
I reverted the UpdateConflictManagerStatus removal and went with a refactor instead, to make sure it was being called consistently.
We can remove it later, but it doesn't need to block the rest of these status consistency improvements.


Exempt video.SE from black magic rules

Reduce FP rate from 18/37 to 10/29, including all the recent FPs.
Monologue at http://chat.stackexchange.com/transcript/message/35175862#35175862

Somebody fix the typo in the commit, please ...!


převod opera

This is ONLY used for reporting bugs.

Expected Behavior

Current Behavior

Possible Solution

Steps to Reproduce

Context (Environment)

Detailed Description

Possible Implementation

translation metamask opera


Derive "Type" of DataElement

ValuePool
FreeText
DateTime

Fixed since https://github.com/Hochfrequenz/mig_ahb_utility_stack/commit/685cdeaeb57c2084a761bc0b69c89bc1bb6948b4


fix: cleanup grant logic

Test Plan

[ ] acceptance tests

[ ] …

References

Hi there, I do believe snowflake_warehouse_grant has the same problem as when applying the tf I can see it grants privileges on the warehouse to the role over and over... even it didn't cause any problem but I hope we can fix this issue.


Cannot convert TF InceptionResNet to IR

Platform (like ubuntu 16.04/win10):ubuntu16.04
Python version:3.5
Source framework with version (like Tensorflow 1.4.1 with GPU):TF 1.5 with GPU
Destination framework with version (like CNTK 2.3 with GPU):Caffe
Pre-trained model path (webpath or webdisk path): https://yadi.sk/d/OnH0586P3SqoV3
Running scripts:

Cannot convert the provided net to IR.
I tried to create new graph using freeze_graph.py and after that to use optimize_for_inference_lib.py.
I got a new graph, but when I try to convert the net to IR using python -m mmdnn.conversion._script.convertToIR I get the following error:
Info: Trying to parse file [optimizedScript2.pb] with binary format but failed with error [Error parsing message].
Traceback (most recent call last):
File "/usr/lib/python3.5/runpy.py", line 184, in _run_module_as_main
"main", mod_spec)
File "/usr/lib/python3.5/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/ml/.virtualenvs/tensorflow3/lib/python3.5/site-packages/mmdnn/conversion/_script/convertToIR.py", line 140, in
_main()
File "/home/ml/.virtualenvs/tensorflow3/lib/python3.5/site-packages/mmdnn/conversion/_script/convertToIR.py", line 135, in _main
ret = _convert(args)
File "/home/ml/.virtualenvs/tensorflow3/lib/python3.5/site-packages/mmdnn/conversion/_script/convertToIR.py", line 48, in _convert
parser = TensorflowParser(args.network, args.weights, args.frozen_pb, args.dstNodeName)
File "/home/ml/.virtualenvs/tensorflow3/lib/python3.5/site-packages/mmdnn/conversion/tensorflow/tensorflow_parser.py", line 185, in init
model = TensorflowParser._load_meta(meta_file)
File "/home/ml/.virtualenvs/tensorflow3/lib/python3.5/site-packages/mmdnn/conversion/tensorflow/tensorflow_parser.py", line 83, in _load_meta
load_protobuf_from_file(meta_graph, model_network_path)
File "/home/ml/.virtualenvs/tensorflow3/lib/python3.5/site-packages/mmdnn/conversion/common/IR/IR_graph.py", line 28, in load_protobuf_from_file
text_format.Parse(file_content.decode('UTF-8'), container, allow_unknown_extension=True)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xe4 in position 92: invalid continuation byte
though the input file is binary.
what should I do to avoid this error&

Hi @KyriaAnnwyn , frozen graph is not supported now. Welcome to contribute to it (not much different from tf checkpoint format). Thanks.

Hi @kitstar! Thank you for your answer. I also tried other graphs (optimized without freezing and just meta file).
If I take optimized *.pb I get this error message:
Info: Trying to parse file [optimizedScript2.pb] with binary format but failed with error [Error parsing message].
And after that the file is parsed as text file and it leads to an error as it is binary.
If I try to convert *.meta file saved in the checkpoint I also get error:
Traceback (most recent call last):
File "/usr/lib/python3.5/runpy.py", line 184, in _run_module_as_main
"main", mod_spec)
File "/usr/lib/python3.5/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/ml/.virtualenvs/tensorflow3/lib/python3.5/site-packages/mmdnn/conversion/_script/convertToIR.py", line 140, in
_main()
File "/home/ml/.virtualenvs/tensorflow3/lib/python3.5/site-packages/mmdnn/conversion/_script/convertToIR.py", line 135, in _main
ret = _convert(args)
File "/home/ml/.virtualenvs/tensorflow3/lib/python3.5/site-packages/mmdnn/conversion/_script/convertToIR.py", line 72, in _convert
parser.run(args.dstPath)
File "/home/ml/.virtualenvs/tensorflow3/lib/python3.5/site-packages/mmdnn/conversion/common/DataStructure/parser.py", line 22, in run
self.gen_IR()
File "/home/ml/.virtualenvs/tensorflow3/lib/python3.5/site-packages/mmdnn/conversion/tensorflow/tensorflow_parser.py", line 280, in gen_IR
func(current_node)
File "/home/ml/.virtualenvs/tensorflow3/lib/python3.5/site-packages/mmdnn/conversion/tensorflow/tensorflow_parser.py", line 350, in rename_Placeholder
IR_node = self._convert_identity_operation(source_node, new_op='DataInput')
File "/home/ml/.virtualenvs/tensorflow3/lib/python3.5/site-packages/mmdnn/conversion/tensorflow/tensorflow_parser.py", line 387, in _convert_identity_operation
TensorflowParser._copy_and_reop(source_node, IR_node, new_op)
File "/home/ml/.virtualenvs/tensorflow3/lib/python3.5/site-packages/mmdnn/conversion/tensorflow/tensorflow_parser.py", line 297, in _copy_and_reop
assert source_node.layer.attr['dtype'].type in TensorflowParser.dtype_map, 'type [{}] is unknown.'.format(source_node.layer.attr['dtype'].type)
AssertionError: type [10] is unknown.
This refers to Switch operators and also produces error messages:
Tensorflow has not supported operator [NoOp] with name [InceptionResnetV1/Conv2d_1a_3x3/BatchNorm/cond/AssignMovingAvg/sub/Switch:1]
Maybe you could help me with dealing this error.

I tried inception_resnet_v2 from slim library and its conversion works ok. The code for model creation is quite similar:
https://yadi.sk/d/_nhw2ze33SsJ38 - contains 2 py model files

Hi @KyriaAnnwyn,

"type[10] is unknown" fixed.

Could you provide your convertToIR command? Seems you didn't set a proper dstNodeName. You can also provide the meta file for me to reproduce.

Thanks.

Hi, @kitstar
This is my convert cmd line: python -m mmdnn.conversion._script.convertToIR -f tensorflow -d IRV1 -n model-20180113-230848.meta --dstNodeName embeddings
and meta file for it https://yadi.sk/d/euys0ILO3StePQ
I've checked the net has node embeddings.
Maybe I have some old version old mmdnn: I installed this one
pip install https://github.com/Microsoft/MMdnn/releases/download/0.1.3/mmdnn-0.1.3-py2.py3-none-any.whl
Your help is really valuable for me!


Value labeling on charts

Originally suggested by Rasmus in slack
Feature Description
Include option to show value labels directly on a chart.
For example, having a value per series and total label for stacked bar charts:

Goal of Feature

Show full information on chart, so it works if someone screenshots, exports PDF, or prints a report
Quicker display of info without needing to hover for tooltip

Current Solution / Workarounds

Not possible today

Implementation Notes
We are quite close to being able to release this for a few chart types, but there are many edge cases to consider, where labels may overlap/conflict.
Options I think we would need to include to maximize the situations the labels could fit into:

Formatting (should match y format to start, may need to offer another format prop to control the labels independently)
Conflict control (hide overlapping vs. show all vs. move overlapping)
Font size (e.g., for crowded chart, would be nice to be able to tick down the font size to get it to look nice)
Position (should default to the best position for each chart type, with options available - like inside end, outside end, center, etc.)
Font colour (ECharts auto-handles the font colour based on contrast with underlying colour, not sure if we'd want to offer customization there)
Column selection (on a scatterplot or bubble, I could see multiple columns being feasible labels for a point)

Have had a start on some of these to see how good the defaults are after applying our number formatting:

I think there's enough here with the defaults that with some polishing off and adding options to control the outputs, we can cover a good chunk of use cases.
The challenge will be figuring out how to stop situations like this from happening:

One way would be to have labels turned off by default, then include the props I mentioned above so the user can adjust as needed until it looks presentable. I don't like having something that could make something unpresentable by default, but it may be the fastest way to make this feature available.

My 2c

off by default I think is right. turn on with valueLabels=true Unless we have a predictable way of calculating if they will look bad (which i guess we do not).
if possible we should add tiny white borders round the labels when they are black

@archiewood as far as I can tell, we won't have a predictable way of calculating it. There might be some rules of thumb we can use around # of series/x values on the chart (e.g., if you have > 20 bars on a bar chart, labels will likely not do much).

if possible we should add tiny white borders round the labels when they are black

This is possible, but I recall some browsers have an issue rendering it. Will give it a shot though.

We can achieve a stacked bar total by inserting a transparent line series into the ECharts config:

Question about stacked bar total:

Should this label be shown by default when the user sets labels/valueLabels to true? Or should it be an additional label that can be added?

on by default. I also think it should be bold

ECharts seems to have reasonable handling of label overlap - this is using their hide overlap feature:

That overlap handling is very good, +1 on total labels when value labels are on.

That overlap handling is very good,

Agreed - I would also suggest exposing the prop to force it to show overlapping with hideOverlap=false
(though true by default)


College "Hetauda School of Management" added

I added my college affiliated with Tribhuvan University i.e. Hetauda School of Management.
There are thousands of students at our college who are missing out on the chance to truly work with professional IDEs.
Hence, I ask you to consider adding my college name.

@keeran-npn First stage of verification passed, official domain is hsm.edu.np as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@keeran-npn Pull request merged. Thank you.


Seychelles now offer same sex marriages

The British high commission in the Seychelles is now offering same-sex
consular marriages. The advice should mirror same outcomes we have for
Bolivia same-sex marriage.

You've mentioned that the outcome for Seychelles should match that of Bolivia. I can see that you've added a test for Seychelles but the lack of an equivalent test for Bolivia would make it easy for the two to drift apart in future. Is that something we're concerned about?
I'd find it useful if the commit note included the command used to generate seychelles_organisations.json.
Apart from that one question and minor comment, this all looks good to me.

You've mentioned that the outcome for Seychelles should match that of Bolivia.

That's was just the easiest way to define requirements for FCO. Those two countries are not related and are not expected to stay in sync.

I'd find it useful if the commit note included the command used to generate seychelles_organisations.json.

I'll add it


harden $LANG check in 3bot

related to https://github.com/threefoldtech/js-sdk/issues/496

I have missed with my $LANG env. variable to be able to reproduce the issue
echo $LANG
threebot start --local

I got  this

from what I see it is already hardened with a clear error message
so are we okay with that @xmonader? or do you want another thing, if so please specify it to be able to work on it

I have searched on how to solve this and be able to harden this error thrown by Click library but all the solutions, I have found that we have to export this env. variable before the python3 script run as python3 use locale setting
as mention here
https://stackoverflow.com/questions/39206705/python-3-throws-unicodeencodeerror-with-environment-variable-lang-c
https://stackoverflow.com/questions/47572800/setting-up-the-appropriate-character-encoding-for-a-pipenv-project-in-the-pychar
https://stackoverflow.com/questions/62054143/readthedocs-local-install-runtimeerror-click-will-abort-further-execution-beca
https://stackoverflow.com/questions/57652720/runtimeerror-click-will-abort-further-execution-because-python-3-was-configured
https://forum.qiime2.org/t/runtimeerror-click-will-abort-further-execution-because-python-3-was-configured-to-use-ascii-as-encoding-for-the-environment/7460
I have tried to use Locale module in python to solve this
locale.setlocale
https://docs.python.org/3/library/locale.html
but it didn't work either, so from what i see that while initiating python3 it check the locale then it will be too late to change it as click package will crash


Create wwcsd.txt

@ebraptus Pull request merged. Please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!
For the record: the proof of this domain ownership is at http://edtech.wwcsd.net/2017/05/getting-to-know-gmail/


[libheif] update to 1.10.0

Update libheif to the latest version 1.10.0
This fixes an access violation when encoding an image with the dynamically linked version of x265.

Thanks for the PR!

Thanks for the PR!

I tried updating libheif to v1.11.0, but I couldn't figure out from which file the SHA512 hash is generated.

@EwoutH
Change the REF to point to v1.11.0 in the vcpkg_from_github helper, then try to install the port (remove the old version if you have it installed).  The installation will fail due to SHA mismatch but it will tell you what the actual SHA should be.


Why isn't the token taken as an input with a default?

I am curious why the token 1) is taken as an environment variable instead of an input, and 2) not defaulted to ${{ github.token }}, which seems to be the idiom on most other actions, especially the official ones. If it's just for legacy/compatibility reasons, totally understandable. Just checking if I am missing something here and if there are some semantic differences that I am missing.

Some workflows may require custom secrets different from ${{ github.token }}. For examples, releases created with ${{ github.token }} do not trigger any events, and if you want them to do so, you will need to provide a personal access token.

Sure. But taking that as an input that defaults to ${{ github.token }} would still allow for overriding it, and is how most other actions do it these days.


Header error

In test / sync mode I get the following error using my CloudFlare account as a source:
resp = self._try_request(
File "/home/russ/src/dns/venv-octodns/lib/python3.10/site-packages/octodns_cloudflare/init.py", line 113, in _try_request
return self._request(*args, **kwargs)
File "/home/russ/src/dns/venv-octodns/lib/python3.10/site-packages/octodns_cloudflare/init.py", line 136, in _request
raise CloudflareError(resp.json())
octodns_cloudflare.CloudflareError: Invalid request headers
I've set the following environment variables:
export CLOUDFLARE_TOKEN="l***********U"
export<EMAIL_ADDRESS>

I believe Cloudflare switched it's auth setup at some point in the past, but I don't recall the details. If what you're using is something CF is calling a token you might try removing the email part, my CF provider config looks like:
  cloudflare:
    class: octodns_cloudflare.CloudflareProvider
    token: env/CLOUDFLARE_TOKEN

That's what I have locally and it's working for me.
When both email & token are set it does the auth-key method, when there's just a token it uses standard http auth mechanisms.
https://github.com/octodns/octodns-cloudflare/blob/f8929f49ddb71ffeb0b43ec18673183b40009b5e/octodns_cloudflare/__init__.py#L96-L101

Facing a similar issue as of OctoDNS  version 1.0.0. Version 0.9.2 is working fine:
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.11.5/x64/bin/octodns-sync", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/opt/hostedtoolcache/Python/3.11.5/x64/lib/python3.11/site-packages/octodns/cmds/sync.py", line 57, in main
    manager.sync(
  File "/opt/hostedtoolcache/Python/3.11.5/x64/lib/python3.11/site-packages/octodns/manager.py", line 694, in sync
    ps, d = future.result()
            ^^^^^^^^^^^^^^^
  File "/opt/hostedtoolcache/Python/3.11.5/x64/lib/python3.11/concurrent/futures/_base.py", line 456, in result
    return self.__get_result()
           ^^^^^^^^^^^^^^^^^^^
  File "/opt/hostedtoolcache/Python/3.11.5/x64/lib/python3.11/concurrent/futures/_base.py", line 401, in __get_result
    raise self._exception
  File "/opt/hostedtoolcache/Python/3.11.5/x64/lib/python3.11/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/hostedtoolcache/Python/3.11.5/x64/lib/python3.11/site-packages/octodns/manager.py", line 457, in _populate_and_plan
    plan = target.plan(zone, processors=processors)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/hostedtoolcache/Python/3.11.5/x64/lib/python3.11/site-packages/octodns/provider/base.py", line 226, in plan
    exists = self.populate(existing, target=True, lenient=True)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/runner/work/dns/dns/src/octodns-cloudflare/octodns_cloudflare/__init__.py", line 336, in populate
    records = self.zone_records(zone)
              ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/runner/work/dns/dns/src/octodns-cloudflare/octodns_cloudflare/__init__.py", line 280, in zone_records
    zone_id = self.zones.get(zone.name, False)
              ^^^^^^^^^^
  File "/home/runner/work/dns/dns/src/octodns-cloudflare/octodns_cloudflare/__init__.py", line 130, in zones
    resp = self._try_request('GET', '/zones',
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/runner/work/dns/dns/src/octodns-cloudflare/octodns_cloudflare/__init__.py", line 91, in _try_request
    return self._request(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/runner/work/dns/dns/src/octodns-cloudflare/octodns_cloudflare/__init__.py", line 110, in _request
    raise CloudflareError(resp.json())
octodns_cloudflare.CloudflareError: Invalid request headers

Provider is defined as:
  cloudflare:
    class: octodns_cloudflare.CloudflareProvider
    token: env/CLOUDFLARE_TOKEN
    pagerules: false

Email is not defined in env vars:
      - name: Plan sync with DNS providers
        if: github.event_name == 'pull_request'
        run: octodns-sync --config-file=./config/production.yaml
        env:
          CLOUDFLARE_TOKEN: ${{ secrets.CLOUDFLARE_TOKEN }}

Facing a similar issue as of OctoDNS version 1.0.0. Version 0.9.2 is working fine:

Shouldn't be any differences in octodns core that impact the HTTP requests of octodns-cloudflare. Did the versions of anything else change as well?
What version of octodns-cloudflare are you running? The line numbers in your stacktrace above don't align with 0.0.1, 0.0.2, or what's currently in HEAD.

My bad, I had indeed pinned the version to:
-e git+https://git@github.com/octodns/octodns-cloudflare.git@0a6d3e82ab1904aca549196a5a477f8bd74482c7#egg=octodns_cloudflare
# octodns_cloudflare==0.0.1 - Awaiting support for disabling page rules

Bumping to following solved the issue:
octodns_cloudflare==0.0.2

Thanks for the quick support and apologies for wasting your time on such issue.

@quistian Do you still have that issue? It seems that there might be a confusion/documentation issue with the key type that is usable. See for example:

https://github.com/traefik/traefik/issues/2219
https://github.com/traefik/traefik/pull/2558/files

Could you double-check if you are using the correct type of key?

I am still having issues!

2023-10-29T18:23:59  [8179962624] INFO  Manager __init__: config_file=config.yaml, (octoDNS 1.2.1)
2023-10-29T18:23:59  [8179962624] INFO  Manager _config_executor: max_workers=1
2023-10-29T18:23:59  [8179962624] INFO  Manager _config_include_meta: include_meta=False
2023-10-29T18:23:59  [8179962624] INFO  Manager _config_auto_arpa: auto_arpa=False
2023-10-29T18:23:59  [8179962624] INFO  Manager __init__: global_processors=[]
2023-10-29T18:23:59  [8179962624] INFO  Manager __init__: global_post_processors=[]
2023-10-29T18:23:59  [8179962624] INFO  Manager __init__: provider=zones (octodns.provider.yaml 1.2.1)
2023-10-29T18:23:59  [8179962624] ERROR Manager _get_{}_class: Unable to import module octodns_cloudflare.CloudflareProvider
Traceback (most recent call last):
  File "/opt/homebrew/lib/python3.11/site-packages/octodns/manager.py", line 330, in _get_named_class
    module, version = self._import_module(module_name)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/octodns/manager.py", line 316, in _import_module
    module = import_module(current)
             ^^^^^^^^^^^^^^^^^^^^^^
  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 1140, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'octodns_cloudflare'
Traceback (most recent call last):
  File "/opt/homebrew/lib/python3.11/site-packages/octodns/manager.py", line 330, in _get_named_class
    module, version = self._import_module(module_name)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/octodns/manager.py", line 316, in _import_module
    module = import_module(current)
             ^^^^^^^^^^^^^^^^^^^^^^
  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 1140, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'octodns_cloudflare'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/homebrew/bin/octodns-dump", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/octodns/cmds/dump.py", line 50, in main
    manager = Manager(args.config_file)
              ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/octodns/manager.py", line 123, in __init__
    self.providers = self._config_providers(providers_config)
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/octodns/manager.py", line 215, in _config_providers
    _class, module, version = self._get_named_class(
                              ^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/octodns/manager.py", line 335, in _get_named_class
    raise ManagerException(
octodns.manager.ManagerException: Unknown provider class: octodns_cloudflare.CloudflareProvider, config.yaml, line 9, column 5

@jdogcoder it seems that the last two comments were related to #66 ?
@quistian I will close this one as outdated. Feel free to re-open, if you still encounter issues.


Update proof value test title

Adds 2 new tests that assertion on the signature in proof.proofValue
new test title:
the signature value (proofValue) MUST be expressed according ' +
            'to section 7 of [RFC4754] (sometimes referred to as the IEEE ' +
            'P1363 format) and encoded according to the specific cryptosuite ' +
            'proof generation algorithm.

When generating ECDSA signatures, the signature value MUST be expressed according to section 7 of [[RFC4754](https://w3c.github.io/vc-di-ecdsa/#bib-rfc4754)](sometimes referred to as the IEEE P1363 format) and encoded according to the specific cryptosuite proof generation algorithm.

this only covers the issuer side not the verifier side of this statement.


add refresh tasks API to client

What changed?
Expose Admin Task Refresh API into workflow service

Why?
We need to provide ability to refresh tasks on stuck workflows from SDK

How did you test it?
locally

Pull Request Test Coverage Report for Build 3ae6fb94-86e6-427a-9068-3576035d6f14

0 of 137 (0.0%)  changed or added relevant lines in 12 files are covered.
22 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-0.02%) to 56.918%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

client/frontend/grpcClient.go
0
4
0.0%

client/frontend/thriftClient.go
0
4
0.0%

service/frontend/grpcHandler.go
0
4
0.0%

service/frontend/thriftHandler.go
0
4
0.0%

client/frontend/client.go
0
6
0.0%

client/frontend/retryableClient.go
0
9
0.0%

client/frontend/metricClient.go
0
12
0.0%

common/types/mapper/proto/api.go
0
16
0.0%

service/frontend/accessControlledHandler.go
0
17
0.0%

client/frontend/errorInjectionClient.go
0
18
0.0%

Files with Coverage Reduction
New Missed Lines
%

common/task/weightedRoundRobinTaskScheduler.go
2
89.64%

service/matching/taskListManager.go
2
74.19%

common/cache/lru.go
3
90.73%

common/task/fifoTaskScheduler.go
3
84.54%

service/history/task/fetcher.go
3
86.67%

common/persistence/nosql/nosqlExecutionStore.go
9
60.46%

Totals

Change from base Build a1c814c1-329a-4c78-813e-2df1bdfad32c:
-0.02%

Covered Lines:
83208

Relevant Lines:
146189

💛  - Coveralls


Pipeline repair: Add function graspy.embed.MultipleASE.transform & add get_score to get score matrices

Reference Issues/PRs

What does this implement/fix? Explain your changes.
Any other comments?

Codecov Report

Merging #237 into master will decrease coverage by 3.06%.
The diff coverage is 12.5%.

@@            Coverage Diff             @@
##           master     #237      +/-   ##
==========================================
- Coverage   86.14%   83.08%   -3.07%
==========================================
  Files          33       33
  Lines        1935     2252     +317
==========================================
+ Hits         1667     1871     +204
- Misses        268      381     +113

Impacted Files
Coverage Δ

graspy/embed/mase.py
86.53% <12.5%> (-13.47%)
:arrow_down:

graspy/__init__.py
100% <0%> (ø)
:arrow_up:

graspy/embed/mds.py
100% <0%> (ø)
:arrow_up:

graspy/plot/plot.py
52.41% <0%> (+0.61%)
:arrow_up:

graspy/utils/utils.py
93.46% <0%> (+2.29%)
:arrow_up:

Continue to review full report at Codecov.

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

@QuasiLegendre @tathey1 Can someone tell me what is happening in this PR?

I am going to close this pull request because I don't think this code is ready to be merged. Some of what you are changing here are major design decisions for the package that we would need to talk about more. You are more than welcome to keep working on your branch, however. A few comments though:

Do not change the base classes extensively without talking to me and Jaewon (if you have the intention of getting things merged), you will likely break a lot of things unintentionally. You are more than welcome to override methods in the base class if you need to, however.
I wouldn't call this a classifier, it's just one step in your pipeline
There is no need for a get_latent method, you can just to estimator.latent_left_
Make sure you follow all of the instructions in the contributing guidelines before PRing code

Fine, thanks.

I will be careful next time.


allow swift to connect to proper memcached host

the default memcached ip address of <IP_ADDRESS> resulted in errors of the swift-proxy service. the respective line is now commented out in the /etc/memcached.conf file and the appropriate bond0 ip address is used. after these file edits, memcached will be restarted to ensure that the changes actually take place.

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

Please don't link to private bug trackers in public repos.

ok to test

retest

This passed local CI, merging.


Add a metadata generator

Add a tool to generate metadata with different formats (ttl, xml, json...) according to the SHACL shapes. This tool should be simple and easy to use, better to be an UI app or web app than a  command-line tool. The using experience should feel like filling a customised form.
Maybe we should create another repo for this tool.
@c-martinez Do you know any similar tools?

I don't know of any similar tools. But I think such a tool should indeed be in a separate repo.


added support for byteCount in plugin Parser

Description
Added support forv byteCount in plugin parser
Issues Resolved
Resolves #[Issue number to be closed when this PR is merged]
Check List

[ ] New functionality includes testing.
[ ] New functionality has a documentation issue. Please link to it in this PR.

[ ] New functionality has javadoc added

[ ] Commits are signed with a real name per the DCO

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

@Durdush , Thank you for this contribution. We have this functionality now, so we can close this PR.


Question about Thread Model

hi,
For multi-threads, seems Ardb employs multi-epoll, each epoll functions within one thread.
Why not one-epoll with a thread pool ?
Correct me if i'm wrong. Thanks.

it's much more easy to implement a pipeline protocol(redis/http) which have no sessionId/sequeceId.

Thanks. Got it.

One more thing, will performance be affected if multi-epoll is employed compared with one epoll + multi threads ?

multi-epoll:    would block all connections accepted by the same epoll  for ONE slow request.


Gh 4899 allow for storage specialization in path iteration

GitHub issue resolved: #4899
Briefly describe the changes proposed in this PR:
The PathIterartion has  a data structure for a pair of values. I made this extend BindingSet so that we can use the capability for implementing specialized collections also for this value pairs.
While doing so I noticed a bug due to a path name not being the same in places. Avoided this in the future by using methods to generate the same string for a variable name by reducing code duplication.

PR Author Checklist (see the contributor guidelines for more details):

[x] my pull request is self-contained
[ ] I've added tests for the changes I made
[x] I've applied code formatting (you can use mvn process-resources to format from the command line)
[x] I've squashed my commits where necessary
[x] every commit message starts with the issue number (GH-xxxx) followed by a meaningful description of the change

I turned on auto-merge if passes.


[Model] Add tie_word_embedding option for Minicpm model

This PR updates the Minicpm model with the tie_word_embedding option, as most models in the Minicpm family, like MiniCPM-2B-sft-bf16, require tie_word_embedding to be turned on.
The preset of minicpm_2b_sft_bf16 is also added in this PR.
Here is the demonstration after the update:
tlopex@tlopex-OMEN-by-HP-Laptop-17-ck1xxx:~/mlc-llm$ mlc_llm chat dist/MiniCPM-2B-sft-bf16-q4f16_1-MLC \
>              --device "cuda:0" --overrides context_window_size=2048 \
>              --model-lib ./dist/libs/MiniCPM-2B-sft-bf16-q4f16_1-cuda.so
[2024-08-15 00:02:59] INFO auto_device.py:79: Found device: cuda:0
[2024-08-15 00:02:59] INFO engine_base.py:143: Using library model: ./dist/libs/MiniCPM-2B-sft-bf16-q4f16_1-cuda.so
[00:02:59] /home/tlopex/mlc-llm/cpp/serve/config.cc:668: Under mode "local", max batch size will be set to 4, max KV cache token capacity will be set to 4096, prefill chunk size will be set to 2048.
[00:02:59] /home/tlopex/mlc-llm/cpp/serve/config.cc:668: Under mode "interactive", max batch size will be set to 1, max KV cache token capacity will be set to 4096, prefill chunk size will be set to 2048.
[00:02:59] /home/tlopex/mlc-llm/cpp/serve/config.cc:668: Under mode "server", max batch size will be set to 80, max KV cache token capacity will be set to 28092, prefill chunk size will be set to 2048.
[00:02:59] /home/tlopex/mlc-llm/cpp/serve/config.cc:748: The actual engine mode is "interactive". So max batch size is 1, max KV cache token capacity is 4096, prefill chunk size is 2048.
[00:02:59] /home/tlopex/mlc-llm/cpp/serve/config.cc:753: Estimated total single GPU memory usage: 5113.495 MB (Parameters: 1461.996 MB. KVCache: 1524.583 MB. Temporary buffer: 2126.916 MB). The actual usage might be slightly larger than the estimated number.
You can use the following special commands:
  /help               print the special commands
  /exit               quit the cli
  /stats              print out stats of last request (token/sec)
  /metrics            print out full engine metrics
  /reset              restart a fresh chat
  /set [overrides]    override settings in the generation config. For example,
                      `/set temperature=0.5;top_p=0.8;seed=23;max_tokens=100;stop=str1,str2`
                      Note: Separate stop words in the `stop` option with commas (,).
  Multi-line input: Use escape+enter to start a new line.

>>> Who are you?
I am an AI assistant. How can I help you today?
>>> Can you tell me something about Chile?
Chile is a country located in South America. It has a population of around 19 million people. It has a high standard of living and is known for its beautiful landscapes. What else would you like to know about Chile?

By the way, this PR deletes some unused lines of previous model in preset. And MOE version of Minicpm will be followed up in the next PR.

cc @tqchen @MasterJH5574


Standalone ingestion jobs do not clean up output files after completing loading

When running an ingestion job using the 'standalone' execution framework, the files written to 'outputDirURI' persist after the job completes. A couple issues arise from this:

This causes subsequent ingestion runs to add the left over files from previous runs in addition to the files for the current run
If concurrent jobs are running with the same storage location they would attempt to load each others files

I haven't dug into the code but it seems like the job should:

clean up after itself
only load segments from the outPutDirURI that it created in the job

If there are reasons why it shouldn't / can't do this, additional documentation on the behavior / purpose of the outputDirURI with standalone jobs would be helpful to callout the cleanup / URI uniqueness requirements.

@kishoreg @plaisted Please assign this task to me, I would like to work on this issue.

Here is my analysis, Looked at LaunchDataIngestionJobCommand.java and it's all job runner (Segment Generator and Push job). Did not see implementation to clean up outputDirURI after the push but do see tempDir cleanup after copying the generated segments to outputDirURI.
@kishoreg @plaisted We have 2 option, let me know which one would be appropriate:

Backward compatible - Introduce new property in SegmentGenerationJobSpec._cleanupOutputAfterPush to remove segments from outputDirURI once after push call finishes successfully. CleanupOutputAfterPush can be configurable using yaml and default it to false
Cleanup by default - Enhance the push job to remove the segments from the outputDirURI once after push call finishes successfully.

Note:
Both Spark and Hadoop push jobs does not support cleanup today.

For the ingestion job, it's by design to keep the segments in output directory. The reason is that for URI and METADATA push job, the output dir is treated at the source of truth of the segment. E.g. users will use this job to generate segments and directly write into s3, then push metadata to Pinot for loading segments from the same s3 directory.

I think it's ok to add a config like cleanUpOutputDir to delete the output directory if the push mode is TAR and the default value should be false.

We usually expect the ingestion job output directory to be empty, but you are right, if there are segments already there or building in progress, then it will push them all.

To solve this I feel we can:

Merge segment generation and push into one task;
Let segment generation job return an array of generated tar file URIs
Push task will take the array and do the work.

@fx19880617 Thanks for the clarification and suggestion.
Shouldn't we just delete the files instead of deleting the output directory for both Tar and Uri push? Also, instead of merging the segment generation and push task, maybe add the cleanUp to remove the tar files from the output directory once push finishes similar to SegmentMetadataPush, this will eliminate removing incorrect segment file(s) generated by another job (s).

@plaisted Could you please mark this issue closed? I do not see an option to close.
CC: @fx19880617

@plaisted Could you please mark this issue closed? I do not see an option to close.
CC: @fx19880617

Ty @amarnathkarthik will check the changes out

Ty @amarnathkarthik will check the changes out

Thanks, @amarnathkarthik !!!
This is awesome!

Thanks, @amarnathkarthik !!!
This is awesome!


python setup.py install requires copying files to /usr/local/lib/python* dir

Hi,
I was following the instructions [1]
but on the last step,
python setup.py install
... fails, because it tries to copy the basemap files to /usr/local/lib/python2*/dist-packages/ dir
Is this a known issue?
so, I used sudo. But then, I had 2 mpl_toolkits in the classpath 1. /usr/local/lib/python2*/dist-packages/  2. ~/.local/lib/python2*/site-packages/
then, it is unable to find the basemap. so, I copied all the contents from the /usr/local/lib/python2*/dist-packages/mpl_toolkits to ~/.local/lib/python2*/site-packages/mpl_toolkits dir & now it works properly now.
[1] https://matplotlib.org/basemap/users/installing.html

I think you can pass --user when you do setup.py install and it will install to your user directory. I don't think this is basemap specific. All packages will normally install to an os wide dir but you can modify that with command line arguments

This is definitely not basemap specific but a generic Python installation/environments misconception.
+1 for closing


'teamviewer-np' (15.3.2682)  hash check failed.

I'm not sure why you closed my previous issue so fast without solving it...
But it does exist...
Would you please wait at least to when the issue is solved and confirmed by the issue submitter?
Installing 'teamviewer-np' (15.3.2682) [64bit]
TeamViewer_Setup.exe (25.7 MB) [==========================================================================================================] 100%
Checking hash of TeamViewer_Setup.exe ... ERROR Hash check failed!
App:         nonportable/teamviewer-np
URL:         https://dl.teamviewer.com/download/TeamViewer_Setup.exe#/setup.exe
First bytes: 4D 5A 90 00 03 00 00 00
Expected:    df578d88e38a3f7ff81e27f11db25664d56fb082a5c9588395ec8d06eada7646
Actual:      a8bb637ca9d4995de4e105f49fa162a92126197fe40de6c6c12fb08367b23c0c
Please try again or create a new issue by using the following link and paste your console output:
https://github.com/oltolm/scoop-nonportable/issues/new?title=teamviewer-np%4015.3.2682%3A+hash+check+failed

BTW, I saw a PR which seems to be a fix...

It wasn't me that closed it; it was GitHub Actions.


Update IdentityModel submodule to preview2

@jennyf19 - Looks like this causes a prebuilt for Newtonsoft.Json.13.0.3. This version of that package needs to be added to https://github.com/dotnet/source-build-reference-packages. You can follow the instructions in the README there for how to do that. Once the PR for that gets merged, it will produce a new version of the SBRP package that will get merged into this repo. Once that happens, we can incorporate that change into this PR and we should be good.

@jennyf19 - Looks like this causes a prebuilt for Newtonsoft.Json.13.0.3. This version of that package needs to be added to https://github.com/dotnet/source-build-reference-packages. You can follow the instructions in the README there for how to do that. Once the PR for that gets merged, it will produce a new version of the SBRP package that will get merged into this repo. Once that happens, we can incorporate that change into this PR and we should be good.

I think a better solution is to upgrade the version we are source-building to be 13.0.3.  There appear to be several repo on this version already (razor, fsharp, roslyn, etc).  We are source-build 13.0.1.  This repo resides in https://github.com/dotnet/source-build-externals but the upgrade should be done in a stand-alone PR.

closing as we cannot ship w/this dependency. We will open another one once we have newtonsoft completely removed. Thanks for your help today @mthalman

closing as we cannot ship w/this dependency. We will open another one once we have newtonsoft completely removed.

@jennyf19 - what is the backstory here?  I'm curious why this dependency can't be shipped.


Getting remark to leave this TOC Markdown untouched

Description
Background:
I am using the Jekyll Documentation Theme for a project. It uses a piece of Markdown for setting up the table of contents on a page.
* Table of Contents
{:toc}

Problem:
Need to get Remark (in use as an atom-beautify plugin)  to leave it untouched. The beautified code doesn't play well with Jekyll. Is there a way to tell Remark not to reformat this?
I have played with the settings but cannot get it to work. I have posted some debug information. If more information would help, I am happy to try to obtain it, but might need some guidance on obtaining it.
Is this a bug? If not, any workaround?
PS: I am new to Markdown
Input Before Beautification
This is what the code looked like before:
* Table of Contents
{:toc}

Expected Output
The beautified code should have remained like this:
* Table of Contents
{:toc}

Actual Output
The beautified code actually looked like this:
Using Remark:
-   Table of Contents
    {:toc}

Hi @JetStreamVelocity!
Jekyll stuff isn’t part of markdown. It’s often used as a template, and when compiled with data, results in HTML or markdown. Remark expects markdown, so it treats the list ('* ') as a list, and the contents ('Table of Contents\n{:toc}') as a value in that.
However, I’m wondering, why do you even use a list? Why * Table of Contents\n{:toc} instead of Table of Contents\n{:toc}? Or even ## Table of Contents\n\n{:toc}?

Hi @wooorm, thanks for looking into this.
I tried both your suggestions above. And fiddled some more. None worked. Like I mentioned, I'm a newbie in this area. It appeared for some reason the syntax expected this to be a list. So I went exploring further and came across the kramdown documentation.
I found that this TOC syntax is part of the kramdown feature set. Automatic “Table of Contents” Generation has a more detailed explanation - and indeed, it expects a list.
Would it be right to say the kramdown flavour is not supported?
Worth considering this as a feature request then, please?

Hi @JetStreamVelocity, sorry for the delay!
Have you tried this code?
# Contents

{:toc}

# H1 header

## H2 header

It may just work. If not, could you provide more info why not?

Would it be right to say the kramdown flavour is not supported?

Correct, kramdown’s extensions are not supported out of the box.

Worth considering this as a feature request then, please?

I’m not interested in adding support for that. However, everything is possible through plugins. Plugins do still need someone to champion and create them though.
There’s also a a pretty good ToC plugin out there already!
Finally, I’m closing this as this is not something that should go in this project. Plugins could do it, but someone needs to create them first.


require image load wrong image in android

Description
Whenever I add a new image to the project, the next update presents this problem in production, both in apk and aab. The problem is that some of the images in the app point to other images and not to those in source. for example instead of a wave or icon, I get something else. This does not happen in the android emulator, it only happens if I update the app from a previous version, if I install it from scratch it does not access. I uploaded two photos from two different phones, one before the update, one with the update.  Can you give me a hand?
Version
0.66.3
Output of react-native info
OS: macOS 10.15.7
CPU: (4) x64 Intel(R) Core(TM) i5-3330S CPU @ 2.70GHz
Memory: 52.42 MB / 8.00 GB
Shell: 5.7.1 - /bin/zsh

Binaries:
Node: 12.22.1 - /usr/local/bin/node
Yarn: Not Found
npm: 6.14.12 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.10.1 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 14.4, DriverKit 20.2, macOS 11.1, tvOS 14.3, watchOS 7.2
Android SDK:
API Levels: 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30, 31
Build Tools: 28.0.3, 29.0.2, 29.0.3, 30.0.2
System Images: android-30 | Google APIs Intel x86 Atom, android-30 | Google Play Intel x86 Atom
Android NDK: Not Found
IDEs:
Android Studio: 2020.3 AI-203.7717.56.2031.7784292
Xcode: 12.4/12D4e - /usr/bin/xcodebuild
Languages:
Java: 11.0.13 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.2 => 17.0.2
react-native: 0.66.3 => 0.66.3
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
Whenever I add a new image to the project
Snack, code example, screenshot, or link to a repository

Solved, problem was affecting Fresco, in particular version 2.6.0, downgrade to 2.5.0 I solved it

For those who don't know how to do it. You can go to the android dir and run
./gradlew app:dependencies

it will show your android dependecies.
In the android/app/build.gradle add following
configurations.all {
    resolutionStrategy {
        force "com.facebook.fresco:fresco:2.5.0"
    }
}

I placed it at the same level as the dependencies section.

@pierroo

I have no mention of fresco in my app/build.gradle

It is most likely 3d party dependency.  Try ./gradlew app:dependencies in the android directory to see the android dependencies tree.

where I should place it in the file?

check my previous comment

I placed it at the same level as the dependencies section.

...
configurations.all {
    resolutionStrategy {
        force "com.facebook.fresco:fresco:2.5.0"
    }
}

dependencies {
    implementation fileTree(dir: "libs", include: ["*.jar"])
    //noinspection GradleDynamicVersion
    implementation "com.facebook.react:react-native:+"  // From node_modules
...

Thank you @dimaportenko
So according to your comment, the configurations.all should come BEFORE dependencies, so I just did then ran gradlew app:dependencies , which gave me:

(I extracted the part that mentioned fresco)
So here we can see fresco is at 2.5.0, but then for some reason below it goes 2.5.0 -> 2.6.0
=> is this something I should worry about?
Should I add lines in resolutionStrategy to force 2.5.0 on all those lines that mention 2.6.0?
Again, thank you so much for your help, this is one mean bug right here that is completely messing my app in production. :/

@pierroo tbh, I don't know. My case was slightly different and force "com.facebook.fresco:fresco:2.5.0" was enough.


[#168833488] io-utils integration

This PR aims to use the models automatically generated from the OpenAPI specifications file by the gen-api-models tool of the io-utils library. In order to achieve this result, this library has been replaced with a forked version supporting the openapi format.
Moreover, some schemas in the OpenAPI specs file have been refactored in order to make optional their current nullable properties, and the code has been changed accordingly.

Affected stories

🌟 #168833488: Integrare tool io-utils

  Generated by :no_entry_sign: dangerJS against 383f1231fb0fdbb9cad1c961646920b2a300d586


Add Rails/ActiveRecordAliases cop.

This finds usage of update_attributes/!, which is an alias for update/!. The unaliased names should be preferred for clarity, and the aliases will be deprecated in Rails 6.0 (https://github.com/rails/rails/commit/5645149d3a27054450bd1130ff5715504638a5f5).
#bad
Book.update_attributes!(author: 'Alice')

#good
Book.update!(author: 'Alice')

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

@koic Thank you for the helpful comments. Changes have been made.

Thanks for working on this! I really appreciate it! 🙇


messages are not received consistently

messages are not received consistently when communicating from iOS client to Android client.

iOS client sends a message

nothing is received on Android client -

Android client sends a message
Android client displays both messages

Fixed


Matmul result overflowing despite casting to higher precision data type

Description
When I call np.matmul with two int8 inputs and output set to dtype=np.int16, the output can hold numbers that would overflow with int8.
But when I try the same with cupy and dtype=cp.int16, I get overflows if my output has values outside the range of int8.
Here is a google colab notebook demonstrating the issue:
https://colab.research.google.com/drive/1eH89v3UFZ0j3fCM7VxpSnFnXDVstYIIa?usp=sharing
Is this difference in casting behavior an intended design?
To Reproduce
import numpy as np
import cupy as cp

print('NumPy')
h_a = np.array([[30.,0.],[0.,4.]])
h_b = np.array([[5.,0.],[0.,4.]])

h_c = np.matmul(h_a,h_b)
print('full precion result')
print(h_c)

h_c = np.matmul(h_a.astype(np.int8),h_b.astype(np.int8))
print('\nint8 result (overflow)')
print(h_c)

h_c = np.matmul(h_a.astype(np.int8),h_b.astype(np.int8), dtype=np.int16)
print('\nint8->int16 result')
print(h_c)

print('\nCuPy')
d_a = cp.array([[30.,0.],[0.,4.]])
d_b = cp.array([[5.,0.],[0.,4.]])

d_c = cp.matmul(d_a,d_b)
print('full precion result')
print(d_c)

d_c = cp.matmul(d_a.astype(cp.int8),d_b.astype(cp.int8), dtype=cp.int16)
print('\nint8->int16 result')
print(d_c)

Installation
Wheel (pip install cupy-***)
Environment
OS                           : Linux-5.4.144+-x86_64-with-Ubuntu-18.04-bionic
Python Version               : 3.7.12
CuPy Version                 : 10.2.0
CuPy Platform                : NVIDIA CUDA
NumPy Version                : 1.21.5
SciPy Version                : 1.4.1
Cython Build Version         : 0.29.28
Cython Runtime Version       : 0.29.28
CUDA Root                    : /usr/local/cuda
nvcc PATH                    : /usr/local/cuda/bin/nvcc
CUDA Build Version           : 11010
CUDA Driver Version          : 11020
CUDA Runtime Version         : 11010
cuBLAS Version               : (available)
cuFFT Version                : 10300
cuRAND Version               : 10202
cuSOLVER Version             : (11, 0, 1)
cuSPARSE Version             : (available)
NVRTC Version                : (11, 1)
Thrust Version               : 100910
CUB Build Version            : 100910
Jitify Build Version         : <unknown>
cuDNN Build Version          : 7605
cuDNN Version                : 7605
NCCL Build Version           : 2708
NCCL Runtime Version         : 2708
cuTENSOR Version             : None
cuSPARSELt Build Version     : None
Device 0 Name                : Tesla P100-PCIE-16GB
Device 0 Compute Capability  : 60
Device 0 PCI Bus ID          : 0000:00:04.0

Additional Information
No response

Does this have the same cause as in https://github.com/cupy/cupy/issues/6715 @takagi?

This is another issue as non-batched matmul takes another path.


fix: regex to support Gitlab subgroups

I think this fixes #4106

Updated regex a bit since a couple of tests failed.

Rabased a couple of hundred commits from main as well. Previously failing tests runs fine locally at least.

Hey there! It would be really cool if someone could approve the workflows to run. Thank you!

Hmm, the python test-error doesn't seem related to my change, right?

Hmm, the python test-error doesn't seem related to my change, right?

No, it shouldn't be, I think it'll be fixed by a rebase.
As for the change, have you been able to test this change end-to-end? We don't really have access to a GitLab setup, so testing this is a bit tricky from our end.

Ok, rebased another 13 commits. I haven't even tried testing this end to end, not even sure how to go about doing that using the dependabot-gitlab service. 🤷🏻♂

@argoyle @jurre I'm currently facing this issue so if there's a docker image tag I can update my helm chart and test.

@willianpaixao I assume you mean a docker image och dependabot-gitlab? The only way to get that would be to first build dependabot-core and include that version somehow in dependabot-gitlab I think.

Ok, rebased another 13 commits. I haven't even tried testing this end to end, not even sure how to go about doing that using the dependabot-gitlab service. 🤷🏻♂

Sorry, I also don't know how to do that, we don't maintain that service

@argoyle @willianpaixao You should be able to test this fix using this image: registry.gitlab.com/dependabot-gitlab/dependabot/dev:chore-test-regex-fix-d3528a54. I just built it using github repo and branch directly rather than gem from rubygems.

Thanks @andrcuns, trying it out now

And now all checks passed as well. 😊

Thanks!

Thank you from us Gitlab-users. ❤


Add StopBANG! survey for at-risk patients

@ekontos @remomueller Moving this to release 8, as that's when we'll be doing the next batch of surveys and testing new survey launch

Need to revisit if this is useful enough to include for researchers, or if there is a better way to capture data for at-risk patients. Also, current focus is on diagnosed patients, moving out of 8.0 for time being. Can revisit.

8.0 introduced survey builder - research team working on creating relevant surveys.


Smart Backspace feature

There is a good example of how PHPStorm handles it, https://blog.jetbrains.com/phpstorm/2014/09/smart-backspace-in-phpstorm-8/ VSCode really lacks feature like this as it requires to delete all the spaces / tabs before the cursor when you want to go back to previous line end using backspace.

This VS Code extension seems to implement this exact feature, although it may have slightly different default keybinds than what all the JetBrains IDEs have.
https://marketplace.visualstudio.com/items?itemName=jasonlhy.hungry-delete

Let's wait another year or two, it's not scary 🤭
Fortunately, this plugin is available.
https://marketplace.visualstudio.com/items?itemName=jasonlhy.hungry-delete

I'm looking for this to be built into vscode since https://marketplace.visualstudio.com/items?itemName=jasonlhy.hungry-delete is erroring out for me multiple times a day causing backspace to not work at all in vscode. My only solution is to reload vscode.

Hungry Delete plugin delays/freezes on backspace on regular lines if auto-suggest box opens as I type. Therefore, I had to uninstall the plugin.

Any updates on this?

Would love to see implementation of this!

Please implement this. Coming from PHPStorm, it drives me insane that this feature is not available in Visual Studio Code.

This feature is truly required, when I use intellij to write java code it feels so pleasant backspace will shift it to previous line when it's just whitespace

+1

@kodmanyagha use Hungry Delete plugin for this :)

@kodmanyagha use Hungry Delete plugin for this :)

Thank you!!! I'm a jetbrain user who is trying to use vscode again and I was missing this feature.
Please vscode team, this feature must be native.

jesus christ! you provide great in container development working experience, but you can't fix this issue for 4 YEARS? Really shocked
Still remove every space...

any updates?

Related on Stack Overflow: Move cursor up after deleting a line in VSCode
How is this not a duplicate of #42785?


Rails 5 compatibility

Would it be possible to remove the rails 4 limitation so it'll run with rails 5?

Is this gem no longer maintained?

This was addressed in #99 which was just merged. Maybe this issue can be closed too, @aserafin?

@jeremywadsack you're right, closing

Can this be added to the Changelog then? Thanks for maintaining the gem :)


Option to use an organization instead.

Currently the tool uses my own repositories. This has the disatvantage of cluttering the public repository list with 'shortened' urls. This problem could be migitated if it were possible to use an organization to store the shortened urls in instead.

I agree that this is a problem, I believe #1 also helps to solve this.
I like the idea, especially for organizations. Going to have to do some quick research about organization push/pull access from individual accounts through the API, but I don't think that'll be a problem.
Interested in hearing more about how an individual account would use this though, would you use a dedicated GitHub organization to store shortened URLs?
This might also call for some sync feature, since presumably multiple people will be pushing to the same organization.
Would love to hear any thoughts you have about this.

@kshvmdn Yep. Basically I'd put shortened repositories into a serparate organization (let's call it foo) that the program then uses instead of my own user-profile, creating repositories and urls like 'http://foo.github.io/bar' instead of 'http://ev1l0rd.github.io/kFG2g8rr'.
That would have the advantage of allowing the shortened URLs not be the users name and of course the aforementioned clutter.
A sync feature could be implemented by creating some sort of meta repository in which a config file is stored that contains the current shortened url list. When adding/deleting a repository, the program could then pull that file (no real need to keep histories here), add/delete the relevant entry and then remove the meta repo and recreate it with the meta file inside it. Not sure on how hard that would be to implement, but that could be a concept.


ARTEMIS-4785 Add log4j2-default.properties for non-run CLI commands

The run command uses log4j2.properties and all other commands use log4j2-default.properties.

Replaced by #5136


WIP: first pass at re-working suspenders

This still needs work, but broad strokes are correct.

@tacaswell I think most of this was addressed by #579 #601 and #625 but that 60e4590 might need to be cherry-picked. Does that look right to you?

#625 took over #521 but nothing from this one

The title of this PR suggests it would be good to revisit it soon.

Soon it will be time to celebrate the 1-year anniversary of this PR.

5+ years

I still think this is worth considering.  The problem that this was trying to fix was suspenders firing while in the pre/post plan of a suspender which at SRX was causing the shutter to flicker so running the pre/post plans in a more isolated task would make those "un suspendable".
This is also the first step to making a "run this (sub) plan and tell me what it is done" message which is something I think we do want (but has not been a high priority).
If there is a reason why this is a bad idea or we see a better way to solve incorrectly re-entrant pre/post plans on suspenders then we should close this.  Just being "old" is not a reason to close something.


dockerfile

Thank for this project.
Can you provide a dockerfile to easily test lighttrack ?

Feel the same. Attach the docker file will greatly reduce the efforts to reproduce the results!


Bug on status pages when live polling enabled.

When live polling is enabled, and pagination links are clicked, the status pages displays css.
E.g. at .../statuses.poll?start=0

@quirkey Looks like this can be closed... :)


Reproduce in Antmaze?

Thank you for your inspirational novel work and open sourcing of the code.
I reproduced the Diffusion Q-leanring, and I want to know why my reproduce is difficult to converge in the antmaze environment? i didn't change any params in the code. the seeds?

The performance curve for antmaze tasks exhibits notable fluctuations, which might be attributed to the aggressive Q-learning. We proposed an offline selection method based on diffusion loss to let it stop at a good point with a good performance.


Border-width not respected below 1px on PDF export

Steps to reproduce
Tell us about your environment:

Puppeteer version:
<EMAIL_ADDRESS>Chrome version:
Version 73.0.3683.103 (Official Build) (64-bit)
Platform / OS version:
CentOS Linux 7
URLs (if applicable):
Node.js version:
v8.11.4

What steps will reproduce the problem?
Create an HTML page with div or table having border width that are under 1px width (I want here to specify in point) and export as PDF.
My HTML page:
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
		"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

	<meta name="Description" content=""/>
	<meta name="Keywords" lang="en" content=""/>
	<meta name="Identifier-URL" content=""/>
	<meta name="Reply-to" content=""/>
	<meta name="Copyright" content="copyright"/>
	<meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/>

			cell1
			cell2
			cell3
			cell4

1pt

2pt

3pt

0.5px

1px

2px

3px

My puppeteer code to export as PDF:
const puppeteer = require('puppeteer-core');

(async () => {

  let fileoutput = process.argv[2];
  let uri = process.argv[3];
  const browser = await puppeteer.launch({ignoreHTTPSErrors: true,executablePath: '/opt/google/chrome/google-chrome',args: ['--no-sandbox','--disable-dev-shm-usage']});
  const page = await browser.newPage();
  await page.goto(uri,{ waitUntil: ['domcontentloaded', 'networkidle2'] });
  await page.pdf({path: fileoutput, format: 'A4',scale: 1,preferCSSPageSize:true, 'printBackground' : true});

  browser.close();
})();

What is the expected result?
We expect to have border width respected as specified in the CSS rules on the PDF export.
I want a border width of 0.25pt for my client document print.
What happens instead?
The border width specified in "pt" and below 1px are not respected in the PDF.
Editing/inspecting the PDF with a vectorial editor (Inkscape for example), we can see that border width of 1px or higher are approximatively respected but under that, there are not.
For example, the div width a border-width of 0.5px is approximatively 1px on the resulting PDF.
Pdf_result.pdf

We're also encountering this behavior

whould ne "nice" ! :-)

Erff I had the same issue here 😫
Any fix in sight ?

Anyone found a workaround for this issue? Like scaling the page or some DPI setting?

The same issue here.
<EMAIL_ADDRESS>

I've got a similar issue where borders between td's are rendered in the printed version even though they are set to none and are not showing up in fireforx / chrome.

That's also happen on element. For example I want to give a line:
h2:after {
  content: '';
  display: inline-block;
  height: 0.5px; // ❌ not working
}

The solution of pseodo line would be like these
h2:after {
  content: '';
  display: inline-block;
  height: 1px;
  transform: scaleY(0.5); // ✅
}

Same issue here.

This seems a chrome's issue, not puppeteer's.


⚠ Samsung has degraded performance

In b0b335c, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2655 ms

Resolved: Samsung performance has improved in 8eae077.


[byon] Implement backend

Since we're hurrying with this, I've implemented it all at once (commits have slight dependency, so it goes as a single PR)
Resolves https://github.com/opendatahub-io/odh-dashboard/issues/153
Resolves https://github.com/opendatahub-io/odh-dashboard/issues/142
Resolves https://github.com/opendatahub-io/odh-dashboard/issues/152
Resolves https://github.com/opendatahub-io/odh-dashboard/issues/151
Resolves https://github.com/opendatahub-io/odh-dashboard/issues/143

/cc @dlabaj

This PR also updates the API - adds fields for id and software.


Gbif React components

The data page in the hosted portal uses a React component (https://react-components.gbif-staging.org/lib/gbif-react-components.js) to display occurences. We assume that the data set page uses such a component as well (https://hp-living-norway.gbif-staging.org/data?view=DATASETS).
Is it possible that you could make this code available?  We are investigating different approaches for our portal and building upon the existing components could be an option. For example to extend the component to accep keywords as a filter when querying the api for datasets.

The code is available here https://github.com/gbif/gbif-web but should be considered unstable and largely undocumented. But  you are of course welcome to use what you can for any purpose.

For example to extend the component to accep keywords as a filter when querying the api for datasets.

That specific example isn't an option since it isn't a dataset search, but an aggregation of distinct datasets within an occurrence search. And the occurrence index do not contain dataset keywords.

@siwelisabeth I feel like I'm not giving you the best help, but I'm not sure what you are aiming for. My contact info is https://www.gbif.org/contact-us/directory?personId=3785 if you would prefer to arrange a meeting.

@MortenHofft We are very happy with the help you are giving :-) For the first version of the portal, we probably will use plain javascript and a solution inspired by your example https://hp-theme.gbif-staging.org/example/dataset?key=492d63a8-4978-4bc7-acd8-7d0e3ac0e744 . In parallell we are exploring other possibilities for future versions, and it would be interesting to see how you have developed the data component in React.  We'll keep in touch.

Be aware that template literals have limited support so it will not work for 5% of your users (assuming global visitor distribution)
https://caniuse.com/template-literals


Change how bitfield masks are calculated

This should lead to correct BIT_MASKs for big endian targets

Indeed, works fine for big endian PowerPC


fix: snap package validation

Fixes https://github.com/microsoft/vscode/issues/127081

To verify SNAP_ENFORCE_RESQUASHFS=1 snap-review <path-to>/code-insiders.snap

[ ] Validated

Since resquash test doesn't allow setuid bit for chrome-sandbox, if we need to pass the validation step then we cannot chmod 4755 in any of the lifecycle steps. Two options for snap package,

Always disable sandbox but eventually we should change our confinement from classic for this to be safer.

Disable setuid sandbox, so that app always default to user namespace sandbox which is available on all kernels >=3.10. For users unable to use namespace sandbox suggest workaround https://github.com/electron/electron/issues/17972#issuecomment-486927073 if they can configure.

Another option, reach out to the snapcraft team and see if we can get an exception in the resquash test.

my 2 cents: if we cannot get sandbox to work with snap, we should stop releasing VSCode on snap.
Until we are there to enable sandbox, we can go back to no sandbox if that helps for snap only.

Agree, will disable sandbox in snap for now, opened https://github.com/microsoft/vscode/issues/127140 to track next steps.


TestHost vs Kestrel differences

I have some ASP.NET Core 1.1 code that I'm upgrading to 2.0. During this process I encountered a problem with my middleware. It was using IOptionsSnapshot from the constructor. This ends up with a
'Cannot consume scoped service 'Microsoft.Extensions.Options.IOptionsSnapshot' based error.
I got this error when running from Visual Studio.
I also have tests for my code, so I updated them to cover both 1.1 and 2.0. In this code I'm using TestHost. I was hoping the tests for 2.0 would crash the same way as Kestrel, but all my tests are green.
Is this constraint verified in the TestHost? Should it be?

This ends up with a 'Cannot consume scoped service 'Microsoft.Extensions.Options.IOptionsSnapshot' based error.

That's because you have (at least) a bug in your code and now you're getting a warning about it :wink:
Just move the IOptionsSnapshot<T> down into your Invoke method (services injected in ctor is from application scope, while Invoke is request scope):
public Task Invoke(HttpContext context, IOptionsSnapshot<MyOptions> options) { /* blah */ }

I'm assuming you're using WebHost.CreateDefaultBuilder in your production application? In 2.0, that method does some magic to enable scope validation in a development environment. This is to prevent you from accidentally shooting yourself in the foot by creating memory leaks.

Is this constraint verified in the TestHost?

Looks like it isn't.
When you wire up your TestServer, you're responsible for setting up the IWebHostBuilder, so you have to do (more or less) the same as WebHost.CreateDefaultBuilder.
If I were you, I'd call the same method as your production app (probably in Program) to set up the WebHost for your tests, so you get the exact same behavior in your integration tests.

Should it be?

That's a good question. I'd say yes, but I wonder how you could enable that check inside TestServer before building the WebHost.

I guess there could be introduced a static method like this:
public class TestServer
{
    public static TestServer Create(Action<IWebHostBuilder> configure)
    {
        var builder = new WebHostBuilder()
         // .UseEnvironment(EnvironmentName.Testing) <-- Would be a nice addition ;)
            .UseDefaultServiceProvider(o => o.ValidateScopes = true);

        configure(builder);

        return new TestServer(builder);
    }
}

That would allow you to "pre-configure" the web host builder before it's passed to the user for modification, in case the user wants to swap the default service provider.

I'm down with why the error occurs and it's fantastic that we now get an error for this. It would be nice if this validation was default on for the TestServer implementation in 2.0; then we could figure out the errors before running them on Kestrel :)
I tried the UseDefaultServiceProvider tip and it works. You just need to know that you have to write the code. If you have tests that target multiple versions, the code gets uglier since the method doesn't exist before 2.0.
#if NETCOREAPP2_0
                .UseDefaultServiceProvider(p => p.ValidateScopes = true)
#endif

Adding scope validation to TestServer sounds like a good idea. It can be added here:
https://github.com/aspnet/Hosting/blob/076e5fe6afdc8a3a683efd3d8c65434d2746353e/src/Microsoft.AspNetCore.TestHost/TestServer.cs#L41

@Tratcher Won't that override an existing, user-specified service provider?

Hmm, it works in CreateDefaultBuilder because the user code runs after that and can replace it, but in TestServer the ordering is reversed. Fair point.

but in TestServer the ordering is reversed. Fair point.

Yeah, that's why I suggested a static Create method taking a configure-delegate. That way, you could "seed" the builder before passing it to the user for overrides.

What about a public static IWebHostBuilder CreateTestBuilder()?

We periodically close 'discussion' issues that have not been updated in a long period of time.
We apologize if this causes any inconvenience. We ask that if you are still encountering an issue, please log a new issue with updated information and we will investigate.


refactor: remove ACK status type of fail

Signed-off-by: Jim Ezesinachi<EMAIL_ADDRESS>

Codecov Report

Merging #1107 (922b41b) into main (1667aa2) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1107      +/-   ##
==========================================
- Coverage   88.28%   88.28%   -0.01%
==========================================
  Files         705      705
  Lines       16394    16393       -1
  Branches     2651     2651
==========================================
- Hits        14473    14472       -1
  Misses       1914     1914
  Partials        7        7

Impacted Files
Coverage Δ

...ges/core/src/modules/common/messages/AckMessage.ts
100.00% <ø> (ø)

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


[BUG] Issues not being created when merging dev branch to master

Describe the bug
Issues are not being created when I merge the dev branch to the master.  Originally I thought it was due to my dev branch merge being pretty sizable.  I created a test TODO and then merged that separately.

Hi @jestarr - can you share your .github/config.yml file if you have one? And any more information about your repository, and possibly a link to the commit would be helpful!

I don't have a .github/config.yml file.  It's a private repo for an R package that I built.

I use the dev branch to do most of my work, and then merge to master for major changes.

Thanks for that info @jestarr. Are you merging via a pull request, or just manually doing something like git merge? It matters because todo treats those very differently.

I’m almost 100% sure I used a git merge To just merge the dev Into master

Gotcha. Ok, so this is expected, but not good - todo will ignore merge commits because the handling of merge-related things is typically done through PRs:
https://github.com/JasonEtco/todo/blob/e71c4905f5df8ef5120051798cae1010303c3b21/lib/push-handler.js#L16-L18
That said, I don't think that's the correct behavior. Ignoring merge commits from a PR is correct, but merge commits outside of a PR should be treating like a regular commit. If you're interested in contributing, that'd look something like:
  const commit = await context.github.git.getCommit(context.repo({commit_sha: context.payload.head_commit.id}))
  const isMergeCommit = commit.data.parents.length > 1
  if (isMergeCommit) {
    // Figure out if the commit is part of a pull request
    const isFromPr = true
    if (isFromPr) return
  }

Could likely leverage the GraphQL API's Commit.associatedPullRequests connection, and maybe even save the first request.

Sorry.  Not sure I follow what I should be doing to fix this?

Relevant to me, too :)


Build istio/proxy failed in linux

Hi, when I run make build under the tag 1.16.2 of istio/proxy, I encounter the following problems in different situation:

Run make-build directly
ubuntu@ip-10-1-13-15:~/runlin/proxy$ sudo make build
/bin/bash: line 1: clang++: command not found
echo clang
clang
echo clang++
clang++
export PATH=/usr/lib/llvm-10/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/snap/bin CC=clang CXX=clang++ &&
bazel build --config=libc++ //...
INFO: Repository local_config_cc instantiated at:
/DEFAULT.WORKSPACE.SUFFIX:509:13: in
/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/bazel_tools/tools/cpp/cc_configure.bzl:183:16: in cc_configure
Repository rule cc_autoconf defined at:
/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/bazel_tools/tools/cpp/cc_configure.bzl:143:30: in
ERROR: An error occurred during the fetch of repository 'local_config_cc':
Traceback (most recent call last):
File "/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/bazel_tools/tools/cpp/cc_configure.bzl", line 125, column 33, in cc_autoconf_impl
configure_unix_toolchain(repository_ctx, cpu_value, overriden_tools)
File "/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/bazel_tools/tools/cpp/unix_cc_configure.bzl", line 349, column 17, in configure_unix_toolchain
cc = find_cc(repository_ctx, overriden_tools)
File "/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/bazel_tools/tools/cpp/unix_cc_configure.bzl", line 314, column 23, in find_cc
cc = _find_generic(repository_ctx, "gcc", "CC", overriden_tools)
File "/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/bazel_tools/tools/cpp/unix_cc_configure.bzl", line 310, column 32, in _find_generic
auto_configure_fail(msg)
File "/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/bazel_tools/tools/cpp/lib_cc_configure.bzl", line 112, column 9, in auto_configure_fail
fail("\n%sAuto-Configuration Error:%s %s\n" % (red, no_color, msg))
Error in fail:
Auto-Configuration Error: Cannot find gcc or CC (clang); either correct your path or set the CC environment variable
ERROR: /DEFAULT.WORKSPACE.SUFFIX:509:13: fetching cc_autoconf rule //external:local_config_cc: Traceback (most recent call last):
File "/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/bazel_tools/tools/cpp/cc_configure.bzl", line 125, column 33, in cc_autoconf_impl
configure_unix_toolchain(repository_ctx, cpu_value, overriden_tools)
File "/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/bazel_tools/tools/cpp/unix_cc_configure.bzl", line 349, column 17, in configure_unix_toolchain
cc = find_cc(repository_ctx, overriden_tools)
File "/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/bazel_tools/tools/cpp/unix_cc_configure.bzl", line 314, column 23, in find_cc
cc = _find_generic(repository_ctx, "gcc", "CC", overriden_tools)
File "/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/bazel_tools/tools/cpp/unix_cc_configure.bzl", line 310, column 32, in _find_generic
auto_configure_fail(msg)
File "/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/bazel_tools/tools/cpp/lib_cc_configure.bzl", line 112, column 9, in auto_configure_fail
fail("\n%sAuto-Configuration Error:%s %s\n" % (red, no_color, msg))
Error in fail:
Auto-Configuration Error: Cannot find gcc or CC (clang); either correct your path or set the CC environment variable
INFO: Repository com_google_absl instantiated at:
/home/ubuntu/runlin/proxy/WORKSPACE:71:19: in
/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/envoy/bazel/repositories.bzl:192:21: in envoy_dependencies
/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/envoy/bazel/repositories.bzl:648:26: in _com_google_absl
/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/envoy/bazel/repositories.bzl:30:23: in external_http_archive
/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/envoy_api/bazel/envoy_http_archive.bzl:16:17: in envoy_http_archive
Repository rule http_archive defined at:
/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/bazel_tools/tools/build_defs/repo/http.bzl:372:31: in
INFO: Repository com_google_protobuf_protoc_linux_x86_64 instantiated at:
/home/ubuntu/runlin/proxy/WORKSPACE:71:19: in
/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/envoy/bazel/repositories.bzl:194:25: in envoy_dependencies
/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/envoy/bazel/repositories.bzl:768:30: in _com_google_protobuf
/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/envoy/bazel/repositories.bzl:30:23: in external_http_archive
/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/envoy_api/bazel/envoy_http_archive.bzl:16:17: in envoy_http_archive
Repository rule http_archive defined at:
/root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/bazel_tools/tools/build_defs/repo/http.bzl:372:31: in
ERROR: /root/.cache/bazel/_bazel_root/be4dc5ab53cdb7a33977de3d680ca317/external/com_github_google_flatbuffers/BUILD.bazel:82:10: @com_github_google_flatbuffers//:flatc depends on @local_config_cc//:cc-compiler-k8 in repository @local_config_cc which failed to fetch. no such package '@local_config_cc//':
Auto-Configuration Error: Cannot find gcc or CC (clang); either correct your path or set the CC environment variable
ERROR: Analysis of target '//extensions/common:node_info_fbs' failed; build aborted:
INFO: Elapsed time: 0.257s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (5 packages loaded, 7 targets configured)
Fetching repository @go_sdk; starting
make: *** [Makefile.core.mk:84: build] Error 1
^^ echo $CC and echo $CXX is both empty

when using Clang/LLVM 15.0.5 in Makefile (tags/1.16.2) (echo $CC is empty, echo $CXX is empty) will give same error ^^^ (either correct your path or set the CC environment variable)

when using Clang/LLVM 15.0.5 (echo $CC is /usr/bin/gcc-11, echo $CXX is /usr/bin/g++-11)

echo /usr/bin/gcc-11
/usr/bin/gcc-11
echo /usr/bin/g++-11
/usr/bin/g++-11
export PATH=clang+llvm-15.0.5-x86_64-linux-gnu-ubuntu-18.04:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/snap/bin CC=/usr/bin/gcc-11 CXX=/usr/bin/g++-11 &&
bazel build //...
INFO: Build options --action_env, --define, and --linkopt have changed, discarding analysis cache.
INFO: Analyzed 180 targets (1012 packages loaded, 52935 targets configured).
INFO: Found 180 targets...
WARNING: cleared convenience symlink(s) bazel-bin, bazel-testlogs because their destinations would be ambiguous
ERROR: /home/ubuntu/runlin/proxy/extensions/access_log_policy/config/v1alpha1/BUILD:26:14: Generating proto_library //extensions/access_log_policy/config/v1alpha1:access_log_policy_config_proto failed: (Exit 1): protoc.exe failed: error executing command (from target //extensions/access_log_policy/config/v1alpha1:access_log_policy_config_proto) bazel-out/k8-opt-exec-2B5CBBC6/bin/external/com_google_protobuf_protoc_linux_x86_64/protoc.exe '--cpp_out=bazel-out/k8-fastbuild/bin' ... (remaining 4 arguments skipped)
Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
extensions/access_log_policy/config/v1alpha1/access_log_policy_config.proto: warning: directory does not exist.
Could not make proto path relative: extensions/access_log_policy/config/v1alpha1/access_log_policy_config.proto: No such file or directory
INFO: Elapsed time: 16.165s, Critical Path: 0.09s
INFO: 11 processes: 9 internal, 2 linux-sandbox.
FAILED: Build did NOT complete successfully
make: *** [Makefile.core.mk:84: build] Error 1
Would appreciate any help!

How does it works? Could you explain why the version is the problem?

I think the key point is the version of Clang/LLVM, I can build proxy Ubuntu 22.04.


Both ant_ars and halfcheetah_ars broken

Julia 1.7.2 running on macOS 12.3, Apple M1 architecture. After the training is complete, the examples throw an identical error:
    episode 98 reward_evaluation -0.7817167591651243. Took 36.599132333 seconds
    episode 99 reward_evaluation -13.705568624874973. Took 36.127634708 seconds
    episode 100 reward_evaluation -10.772545859258324. Took 38.790820875 seconds
    rewards = [44.0188651955438, 84.07397185902886, 58.151621089522905, 74.00135312248605, 61.31391504999342]
    mean(train_time_best) = 2776.0249286366666
    std(train_time_best) =<PHONE_NUMBER>3333417
    mean(rewards) = 64.31194526331501
    std(rewards) = 15.355529928613961
    WARNING: both Dojo and Base export "close"; uses of it in module Main must be qualified
    ERROR: LoadError: UndefVarError: close not defined
    Stacktrace:
     [1] display_policy(env::Environment{Dojo.Ant, Float64, Mechanism{Float64, 35, 13, 13, 9}, BoxSpace{Float64, 8}, BoxSpace{Float64, 37}, Nothing}, policy::Policy{Float64}, normalizer::Normalizer{Float64}, hp::HyperParameters{Float64}; rendering::Bool)
       @ Main ~/.julia/packages/Dojo/frGnC/examples/reinforcement_learning/algorithms/ars.jl:189
     [2] display_policy(env::Environment{Dojo.Ant, Float64, Mechanism{Float64, 35, 13, 13, 9}, BoxSpace{Float64, 8}, BoxSpace{Float64, 37}, Nothing}, policy::Policy{Float64}, normalizer::Normalizer{Float64}, hp::HyperParameters{Float64})
       @ Main ~/.julia/packages/Dojo/frGnC/examples/reinforcement_learning/algorithms/ars.jl:173
     [3] top-level scope
       @ ~/.julia/packages/Dojo/frGnC/examples/reinforcement_learning/ant_ars.jl:124
     [4] include(fname::String)
       @ Base.MainInclude ./client.jl:451
     [5] top-level scope
       @ REPL[3]:1
    in expression starting at /Users/glenhenshaw/.julia/packages/Dojo/frGnC/examples/reinforcement_learning/ant_ars.jl:124

I made a change to use Base.close. This should fix the issue you are seeing.

I just pulled Dojo#main and the problem is still there. Running halfcheetah_ars.jl:
    episode 28 reward_evaluation 5.2598850727605795. Took 32.763520375 seconds
    episode 29 reward_evaluation 6.304627245502632. Took 31.461139625 seconds
    episode 30 reward_evaluation 4.928108782533084. Took 31.937667875 seconds
    rewards = [8.6721774823914, 37.47027757269155, 70.57463206704156, 48.87879789080837, 63.227295557744455]
    mean(train_time_best) =<PHONE_NUMBER>53
    std(train_time_best) = 7.095665020032407
    mean(rewards) = 45.76463611413546std(rewards) = 24.366089389557583
    WARNING: both Dojo and Base export "close"; uses of it in module Main must be qualified
    ERROR: LoadError: UndefVarError: close not defined
    Stacktrace:
     [1] display_policy(env::Environment{Dojo.HalfCheetah, Float64, Mechanism{Float64, 23, 7, 7, 9}, BoxSpace{Float64, 6}, BoxSpace{Float64, 18}, Nothing}, policy::Policy{Float64}, normalizer::Normalizer{Float64}, hp::HyperParameters{Float64}; rendering::Bool)
       @ Main ~/.julia/packages/Dojo/6iIp0/examples/reinforcement_learning/algorithms/ars.jl:189
     [2] display_policy(env::Environment{Dojo.HalfCheetah, Float64, Mechanism{Float64, 23, 7, 7, 9}, BoxSpace{Float64, 6}, BoxSpace{Float64, 18}, Nothing}, policy::Policy{Float64}, normalizer::Normalizer{Float64}, hp::HyperParameters{Float64})
       @ Main ~/.julia/packages/Dojo/6iIp0/examples/reinforcement_learning/algorithms/ars.jl:173
     [3] top-level scope
       @ ~/.julia/packages/Dojo/6iIp0/examples/reinforcement_learning/halfcheetah_ars.jl:84
     [4] include(fname::String)
       @ Base.MainInclude ./client.jl:451
     [5] top-level scope
       @ REPL[4]:1
    in expression starting at /Users/glenhenshaw/.julia/packages/Dojo/6iIp0/examples/reinforcement_learning/halfcheetah_ars.jl:84

Ant ARS should work again, although the hyperparameters might need some tuning for it to walk properly. The halfcheetah example has been removed, but the mechanism still exists, so people could create the example themselves.


Revert "Update Rust dependencies: (#825)"

Overview
A problem with application was discovered in our system tests:
ejb_1_1d90542c7589 | ./exonum-java: symbol lookup error: ./exonum-java: undefined symbol: _ZN134_$LT$java_bindings..runtime..service_factory_adapter..JavaServiceFactoryAdapter$u20$as$u20$exonum..helpers..fabric..ServiceFactory$GT$7command17h55cdb2ff9ab423f7E

This reverts commit 32007a56

Definition of Done

[ ] There are no TODOs left in the code
[ ] Change is covered by automated tests
[ ] The coding guidelines are followed
[ ] Public API has Javadoc
[ ] Method preconditions are checked and documented in the Javadoc of the method
[ ] Changelog is updated if needed (in case of notable or breaking changes)
[ ] The continuous integration build passes

Coverage remained the same at 86.054% when pulling 1c5bd8016eeb612dfd8b3e9dec10cd210c8cd979 on dmitry-timofeev:backout-dep-updates into 54d0bb9a718d7e52cc7fcbdfa41c52603ab94698 on exonum:master.


fix: update install script for alpine

This PR fixes the install script to work on Alpine 64-bit. Our arch function doesn't recognize 32-bit Alpine, which fallsback to npm/yarn install (which is why it works on 32-bit Alpine), but tries to use the standalone install for 64-bit, which doesn't work.
This fixes that.
Fixes #3421

I don't have access to Alpine 64-bit. @code-asher can you test? (Or is there a way for me to get access to some VM running Alpine 64-bit?


Build error: too many arguments in call to yaml.NewDecoder

With Golang 1.12.6:
BUILDSTDERR: # github.com/docker/compose-on-kubernetes/internal/parsing
BUILDSTDERR: _build/src/github.com/docker/compose-on-kubernetes/internal/parsing/loader.go:14:22: too many arguments in call to yaml.NewDecoder
BUILDSTDERR: _build/src/github.com/docker/compose-on-kubernetes/internal/parsing/loader.go:14:45: undefined: yaml.WithLimitDecodedValuesCount

NewDecoder seems to only take one argument:
https://github.com/docker/compose-on-kubernetes/blob/7d98bfa0e73dbcf774624f1fb0ab2d3644309f94/vendor/gopkg.in/yaml.v2/decode.go#L245-L249
I can't find yaml.WithLimitDecodedValuesCount anywhere.

Seems to depend on https://github.com/go-yaml/yaml/pull/375

You are right about the PR on go-yaml, but since it was never merged, the project depends on a fork of this library : https://github.com/docker/compose-on-kubernetes/blob/master/Gopkg.toml#L67
It adds a fix to avoid the infamous YAML bomb https://github.com/docker/compose-on-kubernetes/blob/master/internal/parsing/loader_test.go#L17
How are you building the project? I wonder how you got this error as the dependency is vendored and versioned... I'll close this issue but feel free to re-open it if you think this answer doesn't help you fix the issue 👍

I am building the project unbundled, not using go modules yet (don't have the tooling distro side yet). So I need to build each dependency separately. I've used simonferquel fork and replaced the import path accordingly. I have yet to test it but it should work.

The project does not use go modules yet but dep instead. As the vendoring is embedded and versioned I still don't understand why do you need those manual steps?
Are you wiping the vendor directory?


AVRO-2623: Enable all available linters when running tests for the whole codebase

Make sure you have checked all steps below.
Jira

[x] My PR addresses the following Avro Jira issues and references them in the PR title. For example, "AVRO-1234: My Avro PR"

https://issues.apache.org/jira/browse/AVRO-2623
In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

No additional test, since it's a fix for a test script itself.
Commits

[ ] My commits all reference Jira issues in their subject lines. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain Javadoc that explain what it does

This PR should be merged after #651, which contains a fix for a JavaScript linting error.

Thanks for checking @RyanSkraba! I've just kicked Travis again to confirm the linter succeeds this time.

Hmm, the JS interop test failed. Will try to repro on my local environment.

It seems that we should listen to an 'end' event here instead of 'finish', just like the 'createFileDecoder' test in test_files.js. Will update the PR.


Incorporate better xcode support

Right now, it's a little bit clunky to get support for xcode like here. There should be an easier way to send up this information

this was done in 4.0.0


hello, how to hide defx buffer?

environment

vim: neovim 4
vim-airline: ????
OS: linux
Have you reproduced with a minimal vimrc: ???
What is your airline configuration:

if you are using terminal:

terminal: yes
$TERM variable: xterm-256color
color configuration (:set t_Co?):
if you are using Neovim:
does it happen in Vim: i do not know

actual behavior

expected behavior
do not show defx buffer in buffer line

You could use :bdelete <number_of_buffer> which would give, in your case :bdelete 3
But it's a vim command and not a vim-airline command

You could use :bdelete <number_of_buffer> which would give, in your case :bdelete 3
But it's a vim command and not a vim-airline command.
See :h buffer in vim

oh, i am soryy, what i want is that do not show defx in airline-buffer rather thant delete it

Oh hum I don't know if it's possible. But what you're asking for is very similar to the temporary buffers that some plugins open and disappear when they're closed

You are talking about the tabline, right?
Is it only visible when the defx window is shown and you act with it or is it always shown? Have you checked the documentation regarding excludes:
https://github.com/vim-airline/vim-airline/blob/master/doc/airline.txt#L389-L399
Does this patch fix it?
diff --git a/autoload/airline/util.vim b/autoload/airline/util.vim
index 145246a..6cc1809 100644
--- a/autoload/airline/util.vim
+++ b/autoload/airline/util.vim
@@ -124,7 +124,7 @@ endfunction
 function! airline#util#ignore_buf(name)
   let pat = '\c\v'. get(g:, 'airline#ignore_bufadd_pat', '').
         \ get(g:, 'airline#extensions#tabline#ignore_bufadd_pat',
-        \ 'gundo|undotree|vimfiler|tagbar|nerd_tree|startify|!')
+        \ 'gundo|undotree|vimfiler|tagbar|nerd_tree|startify|!|defx')
   return match(a:name, pat) > -1
 endfunction

yes, this patch fix it, thank you very much.
but here is an another problem which about "!", it does not prevent terminal buffers to appear in the tabline, look at the pic below

that is neovim right? It doesn't use the standard !sh convention.

hello, there is an another question, when i use neovim edit a file which called defx.vim or startify.vim or any filename matching to airline#extensions#tabline#ignore_bufadd_pat , it does not show in the bufline

yes, that is so.