Implementation of the ShEx-language features

Our goal is to support all ShEx language features. In this issue we will track the features that we support.
ShEx features based on: ShEx semantics document
Validation process

Feature
Supported
Pending
Comments

Node-shape
:+1:

Node-Schema (start declaration)

Work in progress

Shape map

Work in progress

Shape Expressions

Feature
Supported
Pending
Comments

ShapeAnd

ShapeOr

ShapeNot

NodeConstraint

Shape

ShapeRef

ShapeExternal

Closed shapes

Extra IRIs in shapes

Extends/Inheritance

Triple expressions

Feature
Supported
Pending

EachOf

SomeOf

Inclusion

Triple constraints

Feature
Supported
Pending

Inverse qualifier

Negated qualifier

Min cardinality

Max cardinality

Unbounded cardinality

Node constraints

Feature
Supported
Pending

IRI kind

BNode kind

Literal kind

Non literal kind

Datatype

Value set value

Int value

Decimal value

Double value

Length facet

MinLength facet

MaxLength facet

Pattern facet

MinInclusive facet

MaxInclusive facet

MinExclusive facet

MaxExclusive facet

Totaldigits facet

Fractiondigits facet

Stem

Stem range with exclusions

Stem range with wildcard

Current status of test_suite:
Passed: 295, Failed: 292, Skipped: 22, Not implemented: 557


Replace flag doesn't work on RBAC-enabled clusters

When a cluster is RBAC-enabled, such that service accounts don't (by default) have privileges to do anything with the Kubernetes API, the replace flag doesn't work properly because the kdo-replacer container that is injected into the pod tries to use kubectl to scale replicas or update service selectors and this fails.
It seems like replacement will need to be done by the client, and reverting will need to be handled by the cluster-wide server components (which will also need to be installed with an appropriate server admin service account).

The workaround for the getting started tutorial is to run this:
kubectl create clusterrolebinding kdo-todo-app --clusterrole=cluster-admin --serviceaccount=kdo-todo-app:default

Resolved by #39.


Hurt notes breaking on arrow rotation

Describe your bug here.
So, i was playing around with a script that made the arrows spin around, and i got in a song with hurt notes (the ones that come with the engine) and when they weren't on the default rotation, they get offset really, REALLY FAR from where they should be. Pls fix this.
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
I charted a song with hurt notes and i made the script that made the arrows rotate.

i don't think custom notes are supposed to rotate so they just break built in ones don't change that

Here's what i mean, look closely
https://user-images.githubusercontent.com/95985177/158398623-0fbed8f9-5e51-4760-9c0f-02c3a98da70e.mp4
It's kinda hard to notice it in that really short video but i had to make it that short because of the 10MB file size limit of GitHub.

its actully kinda hard not to notice
but psych engine is for modding so that means bugs like this may not be fixed.
because the script either wasn't coded for special notes or it was the engine


Dashboard Configuration

@satishc1

New tab added to the web pages for Dashboard Configuration
Documentation for Dashboard Configuration
Changes to config.yml

@tabladrum LGTM


Fixes 3491: bootstrap candlepin connection

Summary
Testing steps
Checklist

[ ] Tested with snapshotting feature disabled and pulp server URL not configured if appropriate

https://issues.redhat.com/browse/HMS-3491


feat: add tags and description to train dialogs

Currently not working correct due to the matrix of edits allowed but I wanted to show what I have to get feedback.

now saving on detected changes

Can you explain this a bit more? Like which line? Might be related to one of the bugs I saw.
I had not intended to save on change. I think the user should always have to click the button and only then does it save.

change two product elements

Also confused by this. Which elements? And why only two?
It changes the server, models, ui, and tests. Yes it's a large change but you're suppose to keep the PR's as atomic things like entire features are entire bugs. That way people can revert if needed.  If you commit  half a feature it gets harder to manage and I think it's harder to review since you can't see all the code changes in one place and have to reference back and forth.

Since this is hard to capture in pictures took a video:
https://microsoft.sharepoint.com/teams/convsysresearch/Shared Documents/Forms/AllItems.aspx?slrid=41efb69e-e0cf-0000-b690-368c22799cc7&viewid=a70ab912-6a5d-4369-bda7-749835f301d4&id=%2Fteams%2Fconvsysresearch%2FShared Documents%2FGeneral%2FtagsAndDescriptionDemo.mp4&parent=%2Fteams%2Fconvsysresearch%2FShared Documents%2FGeneral

Previously:

There was suggestion and discussion to change "Description" to "Scenario" and move it to the top.

I think description is the most open term and makes the user think they can put whatever they want in there. The other suggestion was title but I thought that was to restrictive and not the type of information we wanted to capture.  Scenario seems like a middle ground.
In my mind tags would be the higher level concept or scenario that they want to categorize on and the description is more specific of what actually happened in the dialog.  I'm sure we'll hear from customers either way.

There is a bug in this PR. Repro Steps:

Create a new Train Dialog and save it.
Edit the Train Dialog and modify either "Scenario" or "Tags"

Expected: The Abandon button to show up.
Actual: Save Edit or Delete buttons are the only way out of the dialog.

Merging bit early to unblock Lars.  I had mistakenly merged the corresponding change in models before this was also ready to be merged.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Use rfc6902 in server

Follow-up to https://github.com/medplum/medplum/pull/1493
That PR replaced fast-json-patch with rfc6902 in @medplum/mock.
This PR replaces fast-json-patch with rfc6902 in @medplum/server.

Coverage: 93.764% (+0.006%) from 93.758% when pulling 7e876707e4fca110d4469831e5268267d243892c on cody-use-rfc6902-in-server into 466af609b4c395072c299293e41be85cb86bb9b2 on main.


Vendor UI should open to max required size by text contained.

Description

To explain the title, currently if you access a vending machine in game which has an item with a really long name, it cuts off the end of the items name and the remaining amount until you resize the window.
Obviously resizing the window isn't the end of the world but for some that might not know this is possible and just from a general polish standpoint, the UI should open to the max required size governed by the lengths of text within.
The alternative would be to not make vendor windows resizable and just make them open to their maximum with a table of two columns, one column for name and one for amoint that that it is always consistent.
Screenshots

Additional context

In this specific case I would maybe suggest even changing the name of the "Small high-capacity power cell" to "High capacity power cell (S)". Just a thought to cut down the bulk. The player will get a warning that it doesn't fit in what they want anyway and it's pretty self explainatory by the sprite etc.

See above issue for more details.


Inline docBlock is replaced with function docBlock

/** @var object */
$var = $this->getObject();

becomes:
/**
 * @param int $count
 *
 * @return bool
 */
$var = $this->getObject();

Similar problem with the static keyword:

   static $var = null;`

   /**

* @return array

*/

static $var = null;

#4 might fix this


Adding a new module to Nuxt breaks component types (shows unknown)

Environment
Stackblitz
------------------------------
- Operating System: Linux
- Node Version:     v18.18.0
- Nuxt Version:     3.10.3
- CLI Version:      3.10.1
- Nitro Version:    -
- Package Manager<EMAIL_ADDRESS>- Builder:          -
- User Config:      extends
- Runtime Modules:  -
- Build Modules:    -
------------------------------

Reproduction
https://stackblitz.com/github/martinszeltins/found-nuxt-components-issue?file=app.vue
Describe the bug
When I add this module to my modules array in nuxt.config.ts then I lose all typings for my components (for both local and 3rd party components). It shows that components have type of unknown. Before I add this module, everything works.
nuxt.config.ts 👇
modules: [
+  '@nuxtjs/i18n', // This breaks all auto-import component typings (components not recognized anymore)
   '@vueuse/nuxt',
   'nuxt-primevue'
],

If I remove it from my modules array then everything works again.
Screenshot before adding the module (everything works): 👍  👇

Screenshot after adding the module (type unknown): ❌  👇

Additional context
No response
Logs
No response

I know @danielroe has been inspecting and debugging this on livestream and reported some upstream bugs. Had similar issues but not sure what the status is. 🤞

I know @danielroe has been inspecting and debugging this on livestream and reported some upstream bugs. Had similar issues but not sure what the status is. 🤞

@frasza I didn't that, could you please link to his livestreams where he talked about this issue, I would be curious to see.

Would you raise an issue upstream on @nuxtjs/i18n? We do need to help resolve this but I think it's not a Nuxt issue so much as a Vue typing issue.

Unfortunately, Nuxt isn't in control of what a module or library may do to types with a module augmentation - module augmentations are global and outside of our scope.

Wasn't aware an issue was open here as well (linking for clarity https://github.com/nuxt-modules/i18n/issues/2839), I looked into this issue and it turns out it's not caused by Nuxt I18n but the combination of Primevue's types and those of Nuxt I18n.
@danielroe
You may recall this issue https://github.com/nuxt-modules/i18n/issues/2483, the same is happening here. These type issues arise when modules augment the GlobalComponents interface in @vue/runtime-core instead of vue, unfortunately the type inference breaks only after there is a mix of modules augmenting this interface in both @vue/runtime-core and vue at the same time.
This issue should be resolved after https://github.com/primefaces/primevue/pull/5419 has been merged and released.

Thank you for your investigation ❤

Closing this, the fix is included in<EMAIL_ADDRESS>


Use AS part of the query from platform

This change was part of the "Add SAP Adaptive Server Enterprise vendor support" PR doctrine/doctrine2#5752. It has been seperated because it is not ASE specific and would increase platform independence in general.
It depends on the  "Add selectAliasColumn to AbstractPlatform" PR doctrine/dbal#2359.
It is required in the platform if the "AS" keyword may differ in platforms (which is probably not very common) and for creating workarounds when using deferred tables to simulate ROWNUM() by identity columns.

Since this depends on a PR that was not merged, I'm closing this PR.


Memory leak in example programs pk_encrypt and pk_decrypt

Summary: This PR fixes the #1128 on memory leaks and free-without-initialization in the example program pk_encrypt. The same issue was found to exist pk_decrypt and is also fixed. Also, some minor style improvements have been made along the way.

@andresag01 I don't like making a PR for a few style changes, or for two completely analogous changes, as I think it introduces overhead for no benefit. I should have put them in a separate commit, though - I'm trying to get better at this, but sometimes I still forget.

@andresag01 Yes, this needs backports.

@hanno-arm: Thanks for confirming.

retest

CI test failures look harmless:

one instance of compat.sh slow start
one use of newer-than-this-PR config.pl baremetal in the windows CI script
two instances of known-flaky timing test (#1517)

@RonEld - Your approval has been outdated by a subsequent commit. Could you please re-approve? Thanks.

retest

retest

retest

retest

retest

This PR needs re-basing since it is missing config option baremetal in config.pl script. CI invokes this option when building baremetal with IAR compiler.

@hanno-arm - Can you please rebase this to remove the conflicts? Thanks.

@sbutcher-arm Done.
@RonEld @andresag01 Please re-review.

@RonEld Addressed your point. Please re-review.


Add auto-completion for available encoders

Hi, this might be useful in case of autocompletion for Bash (including the difference between Linux and macOS) and Zsh.

Oh, this is great, thanks! Could you add a short sentence or two to the README.md to instruct people to install the completion files to appropriate locations (or at least the most generic ones).

On it.


Fix Failed to activate the atom-ctags package

Fixes this

Thanks for the pull request. I use new branch hardcode_1.10.0


Migrate to Ciao volume API

Migrate from the OpenStack block API to a ciao volume API. In this PR the HTTP endpoints are moved to the ciao style using Content-Type for versioning, and the number of endpoints is slightly reduced to remove duplication. The datastructure for use in reporting volume information is standardised on as types.BlockData (but then renamed to types.Volume) for clarity with the endpoint.
See #1451

Coverage decreased (-0.01%) to 65.785% when pulling c190fffbbed9775a28b7c38744eb06084bc09c31 on rbradford:de-openstackify-volumes into 92723a7d9c2d0c0477d1aa45bd02f2682c5d990b on ciao-project:master.

Hey @markdryan. Almost there...just "compute" left to sort out.

Coverage decreased (-0.05%) to 65.753% when pulling 9cab6304d0a9e3a3a8605aa3b38b14f4837e7095 on rbradford:de-openstackify-volumes into 92723a7d9c2d0c0477d1aa45bd02f2682c5d990b on ciao-project:master.

Coverage decreased (-0.05%) to 65.753% when pulling 9cab6304d0a9e3a3a8605aa3b38b14f4837e7095 on rbradford:de-openstackify-volumes into 92723a7d9c2d0c0477d1aa45bd02f2682c5d990b on ciao-project:master.


Emulator crashes on load()

After following the instructions for set up, I run the android emulator along with the following code:
env = android_env.load(
    avd_name='pixel2_30',
    android_avd_home='/path/.android/avd',
    android_sdk_root='/path/Android/Sdk',
    emulator_path='/path/Android/Sdk/emulator/emulator',
    adb_path='/path/Android/Sdk/platform-tools/adb',
    task_path='/path/android_env/tasks/accessibility_forwarder_clock_set_timer.textproto',
)

And the emulator crashes:
Traceback (most recent call last):
  File "/path/android_env/android_env/components/emulator_launcher.py", line 137, in launch
    self._emulator.expect('emulator: INFO: boot completed', timeout=wait_time)
  File "/path/anaconda3/lib/python3.6/site-packages/pexpect/spawnbase.py", line 344, in expect
    timeout, searchwindowsize, async_)
  File "/path/anaconda3/lib/python3.6/site-packages/pexpect/spawnbase.py", line 372, in expect_list
    return exp.expect_loop(timeout)
  File "/path/anaconda3/lib/python3.6/site-packages/pexpect/expect.py", line 179, in expect_loop
    return self.eof(e)
  File "/path/anaconda3/lib/python3.6/site-packages/pexpect/expect.py", line 122, in eof
    raise exc
pexpect.exceptions.EOF: End Of File (EOF).
<pexpect.popen_spawn.PopenSpawn object at 0x7f5a4e4416a0>
searcher: searcher_re:
    0: re.compile(b'emulator: INFO: boot completed')

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "test.py", line 11, in <module>
    env = android_env.load(
  File "/path/android_env/android_env/loader.py", line 73, in load
    coordinator = coordinator_lib.Coordinator(simulator, task_manager)
  File "/path/android_env/android_env/components/coordinator.py", line 88, in __init__
    self._restart_simulator()
  File "/path/android_env/android_env/components/coordinator.py", line 181, in _restart_simulator
    self._simulator.launch()
  File "/path/android_env/android_env/components/base_simulator.py", line 135, in launch
    self._launch_impl()
  File "/path/android_env/android_env/components/emulator_simulator.py", line 114, in _launch_impl
    self.restart()
  File "/path/android_env/android_env/components/base_simulator.py", line 144, in restart
    self._restart_impl()
  File "/path/android_env/android_env/components/emulator_simulator.py", line 102, in _restart_impl
    self._launcher.restart()
  File "/path/android_env/android_env/components/emulator_launcher.py", line 151, in restart
    self.launch()
  File "/path/android_env/android_env/components/emulator_launcher.py", line 143, in launch
    raise errors.SimulatorCrashError('The emulator has crashed: %r' % e)
android_env.components.errors.SimulatorCrashError: The emulator has crashed: EOF("End Of File (EOF).\n<pexpect.popen_spawn.PopenSpawn object at 0x7f5a4e4416a0>\nsearcher: searcher_re:\n    0: re.compile(b'emulator: INFO: boot completed')",)

I'm not sure why this happens, any idea? Thanks!

@AdamIshay could you solve this issue on your system?

Hello @AdamIshay, could you check the locations of the Android Emulator, SDK and its associated tools (adb etc)? They're usually on your home directory, but it looks like you're passing /path/Android/Sdk/emulator/emulator which normally doesn't seem like a valid path. Please see https://github.com/deepmind/android_env/blob/main/docs/emulator_guide.md#ready-to-use and https://github.com/deepmind/android_env/blob/main/docs/instructions.md#create-the-environment.

@kenjitoyama thanks for your input.  so "../Android/Sdk/emulator/emulator"  is not the correct path? Got this same path from https://github.com/deepmind/android_env/blob/main/docs/instructions.md#create-the-environment

Hello @ouya99, normally the emulator is installed in the home directory, so using a relative path is probably not right as the working directory of the binary may change depending on the script location. It's better to use an absolute path.

python3 run_random_agent.py --avd_name='Pixel_XL_API_30' --android_avd_home='/home/username/.android/avd' --android_sdk_root=/home/username/Android/Sdk/ --emulator_path=/home/username/Android/Sdk/emulator/emulator --adb_path=/home/username/Android/Sdk/platform-tools/adb --num_steps=1000 --task_path=/home/username/Dev/android_env/examples/pong_default.textproto
@kenjitoyama this code solves the issue mentioned here. I think @AdamIshay had missed the error message stating that a directory (avd or sdk) had not been found. Yet now I am getting another error
E0622 00:49:36.111660<PHONE_NUMBER>04608 adb_controller.py:202] self._adb_shell.before: b"adb server version (39) doesn't match this client (41); killing...\r\n* daemon started successfully\r\nadb: device 'emulator-15414' not found\r\n" E0622 00:49:36.111877<PHONE_NUMBER>04608 adb_controller.py:203] Could not start ADB shell. Try 1 of 20.

@kenjitoyama ok, got it working after fixing adb version mismatch (hint: just remove the global adb installation, one is enough)
Thank you. You might close the issue

@ouya99 cool, thanks for confirming. I'll close the issue for now given that the OP was probably using the wrong path to the required files.


Unable to create belongsTo/readOnly relationship

Hello,
I'm trying to create a belongsTo/readOnly relationship between two models.
For example I have a registration and variant models.
The registration model contains
variant: belongsTo('variant', { async: true})
While the variant model just has:
registration: attr()
When I try to save the registration only the properties without the relationship get saved.  The variant property is not saved.
I think this should be possible but I'm not sure if I'm doing something wrong or if this is a bug?
Any help would be appreciated.
Version numbers below.
Thank you
DEBUG: -------------------------------
DEBUG: Ember      : 2.5.1
DEBUG: Ember Data : 2.7.0
DEBUG: Firebase   : 3.2.1
DEBUG: EmberFire  : 0.0.0
DEBUG: jQuery     : 2.2.4
DEBUG: -------------------------------

Hey, are you saving the variant first, and then saving the registration model?
Try also setting {async: true, inverse: null} just to be sure there isnt some Ember Data relationship magic getting in the way.

I did get this worked out by using the inverse: null.
Thanks for the response.
Brandon
Brandon Feltman
LinkedIn https://www.linkedin.com/profile/view?id=109506817
This information is confidential and intended solely for the use
of the individual or entities to whom it is addressed. If you have
received this email in error please notify the sender.
On Wed, Sep 28, 2016 at 1:56 PM, Tim Stirrat<EMAIL_ADDRESS>wrote:

Hey, are you saving the variant first, and then saving the registration
model?
Try also setting {async: true, inverse: null} just to be sure there isnt
some Ember Data relationship magic getting in the way.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/firebase/emberfire/issues/422#issuecomment-250281206,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAt87haTyEIHnxsxoZd1OnHpkd-mNDjIks5qusZRgaJpZM4J0Zdw
.


Support per queue metrics for rabbitmq plugins

Hello,
This would be awesome if telegraf could support per queues metrics.
Collectd has a pretty exhaustive plugin here : https://github.com/NYTimes/collectd-rabbitmq
Thoughts ?

thanks for the suggestion, that'll be a good enhancement

I'm working on it.

Thanks @ekini !

Hello @ekini, is there anything to test ? How can I help you ?

Sorry to make you waiting for it so long. I've created the pull request.


udf: support async python functions

Is your feature request related to a problem? Please describe.
Support defining UDFs over async functions.
@udf(input_types=['...'], result_type='...')
async def external_service(input):
    p = preprocess(input)
    return await call_my_external_service(p)

Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

blocking functions are supported by #9819


Fix item size calculation inside layoutGroup

close #125
I think itemSize should not ignore effectiveSizeForContainer.
https://github.com/kishikawakatsumi/IBPCollectionViewCompositionalLayout/blob/master/Sources/IBPCollectionViewCompositionalLayout/IBPCollectionCompositionalLayoutSolver.m#L108
CGSize itemSize = [item.layoutSize effectiveSizeForContainer:container];

At least this changes work well reproduction code https://github.com/ichiko/IBPCVCLLayoutGroupContentInsetsBug

@ichiko This change breaks existing behavior (Run example app and see AdaptiveSectionsViewController).
This PR cannot be merged as-is. I will investigate the correct solution.

Sorry I missed checking all examples.Thank you for your investigating.


Model deployment

Extend the docs to include info about deploying to SageMaker, to reflect the latest status of https://github.com/iterative/example-repos-dev/tree/master/example-get-started-experiments .
Sections to extend:

[ ] https://dvc.org/doc/start/experiments
Maybe a new page about deployment.
[ ] https://dvc.org/doc/studio/user-guide/model-registry
Maybe extend each corresponding page with info/links about the implementation of example-get-started-experiments

Edited the description a little bit but the scope is still not quite clear

Added a checkbox for the new sagemaker integrations page.
@daavoo What would help clarify the scope?

Added a checkbox for the new sagemaker integrations page.
@daavoo What would help clarify the scope?

I talked with @tibor-mach and will be handing this issue over to him, as he will also make some potential changes to the examples.
To me, what is unclear from the scope is how deep to get in each of these sections and how much "sagemake-specifc" we want the content to be.
Hopefully we can get a quick draft with minimal changes to get a feeling and try to cut the scope in parts

@tibor-mach @tapadipti Thoughts on the unchecked items in this list?


Improve HOW.md. Add crate2nix to dev shell.

Added crate2nix to dev shell as so that crate2nix generate can be ran when Cargo.lock changes.

mb haven't seen it

No worries. Don't merge this yet though, as I'm working on this branch right now.

Ok I think I'm done. Please feel free to review.


Kraken updateproducts.sh does not work

Hi,
The Kraken products list is dated with many pairs that no longer exist (all the ones with the old names that do not begin with x or z).
The updateproducts.sh script does not work properly. Here's the error message I get.
Running Ubuntu 16.04 LTS.
I am otherwise able to trade on Kraken (and bittrex's updateproducts works. Incidentally the permissions on bittrex's updateproducts.sh is incorrect and lacks execute (chmod +x) - or should I open a separate ticket for that)
Thanks and keep up the good work.
> don@naked:~/zenbot/extensions/exchanges/kraken$ ./update-products.sh
Error: Could not understand response from server: <!DOCTYPE html>

<meta http-equiv="set-cookie" content="cf_use_ob=0; expires=Tue, 01-Aug-17 20:38:56 GMT; path=/">
<title>api.kraken.com | 520: Web server is returning an unknown error</title>
<meta charset="UTF-8"/>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/>
<meta http-equiv="X-UA-Compatible" content="IE=Edge,chrome=1"/>
<meta name="robots" content="noindex, nofollow"/>
<meta name="viewport" content="width=device-width,initial-scale=1,maximum-scale=1"/>
<link rel="stylesheet" id="cf_styles-css" href="/cdn-cgi/styles/cf.errors.css" type="text/css" media="screen,projection"/>

Error
520
Ray ID: 387b99c0cac73542 • 2017-08-01 20:38:26 UTC

Web server is returning an unknown error

You

Browser

Working

London

Cloudflare

Working

api.kraken.com

Host

Error

What happened?

There is an unknown connection issue between Cloudflare and the origin web server. As a result, the web page can not be displayed.

What can I do?

If you are a visitor of this website:

Please try again in a few minutes.

If you are the owner of this website:

There is an issue between Cloudflare's cache and your origin web server. Cloudflare monitors for these errors and automatically investigates the cause. To help support the investigation, you can pull the corresponding error log from your web server and submit it our support team. Please include the Ray ID (which is at the bottom of this error page). Additional troubleshooting resources.

Cloudflare Ray ID: 387b99c0cac73542
•
Your IP: <IP_ADDRESS>
•
Performance & security by Cloudflare

    at Request._callback (/home/don/zenbot/node_modules/kraken-api/kraken.js:142:33)
    at Request.self.callback (/home/don/zenbot/node_modules/request/request.js:188:22)
    at emitTwo (events.js:125:13)
    at Request.emit (events.js:213:7)
    at Request.<anonymous> (/home/don/zenbot/node_modules/request/request.js:1171:10)
    at emitOne (events.js:115:13)
    at Request.emit (events.js:210:7)
    at IncomingMessage.<anonymous> (/home/don/zenbot/node_modules/request/request.js:1091:12)
    at Object.onceWrapper (events.js:314:30)
    at emitNone (events.js:110:20)
don@naked:~/zenbot/extensions/exchanges/kraken$

It works for me... You seem to be getting the CloudFlare page, Kraken has had some connection issues.
That said, I still seem unable to get the kraken.BCH-ZEUR to work, it keeps telling me Error: Kraken API returned error: Query:Unknown asset pair. I checked the products.js and it is in there...

@timstoop I've had some success after changing it to BCH-EUR in products.js. Though I'm puzzled why this is needed.

That seems to work indeed, but I'm not getting pretty regular:
kraken.BCH-EUR saved 1000 trades 1 days left
....
Trades error:
Error: Kraken API returned error: Service:Unavailable

Wish the script would just retry on this error :(

Sorry, seems to be an intermittent problem with Kraken yes. Tried again today after trying a few times in succession that day and it works now.
Ditto on BCH pairs, can't get them to work either.

So, manually changing the trading pair in products.js (removing the Z from ZEUR, for instance) makes Zenbot see the asset, however, when you try to get the balance, it does not recognise the currency (and displays it as 0). I'm not sure what's going on here :/

I ran updateproduct.sh again and now the result is a bit different (ignore the ./zen.sh invocation, it's the way I run zenbot.sh in a docker container):
tim@idefisk ~/code/upstream/zenbot $ ./zen.sh list-selectors | grep kraken | grep BCH
  kraken.BCH-XXBT   (BCH/XBT)
  kraken.BCH-ZEUR   (BCH/EUR)
  kraken.BCH-ZUSD   (BCH/USD)
tim@idefisk ~/code/upstream/zenbot $ ./zen.sh balance kraken.BCH-ZEUR

getQuote error:
Error: Kraken API returned error: Query:Unknown asset pair
    at Request._callback (/app/node_modules/kraken-api/kraken.js:155:34)
    at Request.self.callback (/app/node_modules/request/request.js:188:22)
    at emitTwo (events.js:106:13)
    at Request.emit (events.js:191:7)
    at Request.<anonymous> (/app/node_modules/request/request.js:1171:10)
    at emitOne (events.js:96:13)
    at Request.emit (events.js:188:7)
    at IncomingMessage.<anonymous> (/app/node_modules/request/request.js:1091:12)
    at IncomingMessage.g (events.js:292:16)
    at emitNone (events.js:91:20)
    at IncomingMessage.emit (events.js:185:7)
    at endReadableNT (_stream_readable.js:974:12)
    at _combinedTickCallback (internal/process/next_tick.js:80:11)
    at process._tickCallback (internal/process/next_tick.js:104:9)
/app/commands/balance.js:38
              if (err) throw err
                       ^

Error: Kraken API returned error: Query:Unknown asset pair
    at Request._callback (/app/node_modules/kraken-api/kraken.js:155:34)
    at Request.self.callback (/app/node_modules/request/request.js:188:22)
    at emitTwo (events.js:106:13)
    at Request.emit (events.js:191:7)
    at Request.<anonymous> (/app/node_modules/request/request.js:1171:10)
    at emitOne (events.js:96:13)
    at Request.emit (events.js:188:7)
    at IncomingMessage.<anonymous> (/app/node_modules/request/request.js:1091:12)
    at IncomingMessage.g (events.js:292:16)
    at emitNone (events.js:91:20)
    at IncomingMessage.emit (events.js:185:7)
    at endReadableNT (_stream_readable.js:974:12)
    at _combinedTickCallback (internal/process/next_tick.js:80:11)
    at process._tickCallback (internal/process/next_tick.js:104:9)

So Kraken seemed to have fixed the actual pairs, but Zenbot is using something else to determine what the pairs are? I'm confused.

I had the same confusion. It took a minute to figure it out for USD...
kraken.BCH-USD
I'm guessing it'd be BCH-EUR for you? Looks like the X and Y was dropped from BCH to USD.

Nope, that gives me:
$ ./zen.sh balance kraken.BCH-EUR
error: could not find product "BCH-EUR"

@timstoop did you change it to BCH-EUR manually in the products.json file before trying? Because that should work. I don't know if updateproduct.sh does this automatically already. It didn't when I last tried.

https://github.com/carlos8f/zenbot/pull/469

FYI, it only seems to impact DASH, EOS, GNO and BCH. We could hard code a check for those tokens and drop the X/Z naming convention. As far as I can tell there is no way to procedurally distinguish this change from the API output for their currency pairs.

A fix for the update script is in https://github.com/carlos8f/zenbot/pull/483

Closing as this should stay fixed now thanks to @cmroche


400 Bad Request: The plain HTTP request was sent to HTTPS port

Hi I'm trying to setup the connection, but getting the below error .
execution error: 400 The plain HTTP request was sent to HTTPS port  400 Bad Request The plain HTTP request was sent to HTTPS port AppleHttpServer/3cf2bca163d83e1139a9fb7d706140b63b127f62
Do you have some idea why this is happening? Enforce "https://" in the beginning of the host doesnt work

Hey thank you for the prompt release! I upgrade to 0.2.5 and seeing different an error now.
ERROR: Error opening connection self signed certificate in certificate chain, {"code":-1,"data":{"driver":"Trino Driver","driverOptions":{}}}


Parametrize the optimization value in the heuristic algorithms

In both simulated annealing and greedy we use the o information as optimization score. But we could want to use tc, dtc or s and we can set this as parameter

This was dployed on version v0.1.15


Easier Inter-cloud configuration and included DB updates in scripts

As part of the installation it should now do the correct IPs for the gateway/-keeper and add the default brokers. The user will be asked.
I also made a couple of new scripts:
sudo ah_gen_system SYSTEM_NAME HOST [SERVICE]
Generate a new application system (under /etc/arrowhead/systems). It will do the certificates, database entries (in arrowhead_system and arrowhead_service), and generate an app.properties template for the system.
sudo ah_gen_cloud CLOUD_NAME HOST
This will generate a new cloud on a detached installation (under /etc/arrowhead/clouds). It generates the certificate, updates the local database tables (using ah_add_neigbor below), and outputs the full ah_add_neighbor command you need to execute on the new cloud when it has been installed.
ah_add_neighbor OPERATOR CLOUD_NAME HOST AUTH_INFO
Updates the tables arrowhead_cloud and neighbor_cloud with a new neighbouring cloud.
You still have to do the orchestration and authorisation tables yourself though.
Until I get the SQL query fixed for dynamic IDs (started to use the hibernate_sequence table when generating new entries), the ah_gen_quickstart script will no longer do orchestration and authorisation neither.

I only looked at the changed files so far, so I did not try out all the new scripts yet in VM, (and my bash script knowledge is inferior to yours), but I have the following question:
At the end of the ah_gen_cloud.sh script, you print out a statement, saying how the ah_add_neighbor script should be called with the own cloud info. Why do you do that? The neighbor script inserts a new row into the arrowhead_cloud and neighbor_cloud tables, but we don't want the own cloud information in the neighbor_cloud tables, that is in the own_cloud table. Does this mean, the common package install now doesn't insert own cloud information into the database?


DDC: Static warnings are reported as errors

Static warnings exist in Dart again, cf. issue #34349, and they should be reported as such and recognized in tests via the outcome expectation static type warning. However, using a fresh SDK checkout (commit fa0d99156f720df84ed48469615d482e81126be5), the following program gives rise to test failures as shown below:
class A {
  int //# 0: static type warning
  operator []=(x, y) {}
  void f([int i = 0]) {}
}

int //# 1: static type warning
set setter(value) {}

int get foo => 42;
set foo(
  String //# 2: static type warning
  s) {}

class B extends A {
  void f([int i = 1]) {} //# 3: static type warning
}

enum E { one, two, three }

void useE() {
  var v = E.one;
  switch (v) {
    case E.one: break;
    case E.two: break;
    /* //# 4: static type warning
    case E.three: break;
    */ //# 4: continued
  }
}

int bar() { return; } //# 5: static type warning

Future<int> barAsync() async { return; } //# 6: static type warning

main() {
  A()[0] = 0;
  setter = null;
  foo;
  foo = null;
  B().f();
  useE();
  bar(); //# 5: continued
  barAsync(); //# 6: continued
}

Here is the output:
tools/testing/dart/main.dart:1: Warning: Interpreting this as package URI, 'package:test_dart/main.dart'.
Test configuration:
    custom configuration(architecture: x64, compiler: dartdevc, mode: release, runtime: chrome, system: linux, vm-options: [], timeout: 960, host-checked, preview-dart-2)
Suites tested: language_2

FAILED: dartdevc-chrome release_x64 language_2/scratch_test/4
Expected: Pass
Actual: CompileTimeError
Unexpected compile error.

--- Command "dartdevc" (took 09.000711s):
DART_CONFIGURATION=ReleaseX64 sdk/bin/dartdevc --dart-sdk-summary /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/ddc_sdk.sum --library-root /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2 --ignore-unrecognized-flags --no-summarize --no-source-map -o /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_compilations/dartdevc/multitest_language_2_scratch_test_4/scratch_test_4.js /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_4.dart -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/async_helper.sum=async_helper -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/collection.sum=collection -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/expect.sum=expect -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/js.sum=js -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/matcher.sum=matcher -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/meta.sum=meta -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/path.sum=path -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/stack_trace.sum=stack_trace -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/unittest.sum=unittest (working directory: /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2)

exit code:
1

stdout:
[error] Missing case clause for 'three'. (out/ReleaseX64/generated_tests/language_2/scratch_test_4.dart, line 25, col 3)

Please fix all errors before compiling (warnings are okay).

--- Re-run this test:
python tools/test.py -m release -c dartdevc --strong --host-checked -t 960 language_2/scratch_test/4

[00:10 |  12% | +    0 | -    1]
FAILED: dartdevc-chrome release_x64 language_2/scratch_test/5
Expected: Pass
Actual: CompileTimeError
Unexpected compile error.

--- Command "dartdevc" (took 09.000783s):
DART_CONFIGURATION=ReleaseX64 sdk/bin/dartdevc --dart-sdk-summary /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/ddc_sdk.sum --library-root /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2 --ignore-unrecognized-flags --no-summarize --no-source-map -o /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_compilations/dartdevc/multitest_language_2_scratch_test_5/scratch_test_5.js /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_5.dart -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/async_helper.sum=async_helper -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/collection.sum=collection -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/expect.sum=expect -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/js.sum=js -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/matcher.sum=matcher -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/meta.sum=meta -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/path.sum=path -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/stack_trace.sum=stack_trace -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/unittest.sum=unittest (working directory: /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2)

exit code:
1

stdout:
[error] Missing return value after 'return'. (out/ReleaseX64/generated_tests/language_2/scratch_test_5.dart, line 34, col 13)

Please fix all errors before compiling (warnings are okay).

--- Re-run this test:
python tools/test.py -m release -c dartdevc --strong --host-checked -t 960 language_2/scratch_test/5

[00:10 |  25% | +    0 | -    2]
FAILED: dartdevc-chrome release_x64 language_2/scratch_test/0
Expected: Pass
Actual: CompileTimeError
Unexpected compile error.

--- Command "dartdevc" (took 09.000897s):
DART_CONFIGURATION=ReleaseX64 sdk/bin/dartdevc --dart-sdk-summary /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/ddc_sdk.sum --library-root /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2 --ignore-unrecognized-flags --no-summarize --no-source-map -o /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_compilations/dartdevc/multitest_language_2_scratch_test_0/scratch_test_0.js /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_0.dart -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/async_helper.sum=async_helper -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/collection.sum=collection -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/expect.sum=expect -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/js.sum=js -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/matcher.sum=matcher -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/meta.sum=meta -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/path.sum=path -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/stack_trace.sum=stack_trace -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/unittest.sum=unittest (working directory: /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2)

exit code:
1

stdout:
[error] The return type of the operator []= must be 'void'. (out/ReleaseX64/generated_tests/language_2/scratch_test_0.dart, line 4, col 3)

Please fix all errors before compiling (warnings are okay).

--- Re-run this test:
python tools/test.py -m release -c dartdevc --strong --host-checked -t 960 language_2/scratch_test/0

[00:10 |  37% | +    0 | -    3]
FAILED: dartdevc-chrome release_x64 language_2/scratch_test/1
Expected: Pass
Actual: CompileTimeError
Unexpected compile error.

--- Command "dartdevc" (took 10.000027s):
DART_CONFIGURATION=ReleaseX64 sdk/bin/dartdevc --dart-sdk-summary /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/ddc_sdk.sum --library-root /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2 --ignore-unrecognized-flags --no-summarize --no-source-map -o /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_compilations/dartdevc/multitest_language_2_scratch_test_1/scratch_test_1.js /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_1.dart -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/async_helper.sum=async_helper -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/collection.sum=collection -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/expect.sum=expect -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/js.sum=js -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/matcher.sum=matcher -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/meta.sum=meta -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/path.sum=path -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/stack_trace.sum=stack_trace -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/unittest.sum=unittest (working directory: /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2)

exit code:
1

stdout:
[error] The return type of the setter must be 'void' or absent. (out/ReleaseX64/generated_tests/language_2/scratch_test_1.dart, line 9, col 1)

Please fix all errors before compiling (warnings are okay).

--- Re-run this test:
python tools/test.py -m release -c dartdevc --strong --host-checked -t 960 language_2/scratch_test/1

[00:10 |  50% | +    0 | -    4]
FAILED: dartdevc-chrome release_x64 language_2/scratch_test/6
Expected: Pass
Actual: CompileTimeError
Unexpected compile error.

--- Command "dartdevc" (took 10.000152s):
DART_CONFIGURATION=ReleaseX64 sdk/bin/dartdevc --dart-sdk-summary /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/ddc_sdk.sum --library-root /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2 --ignore-unrecognized-flags --no-summarize --no-source-map -o /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_compilations/dartdevc/multitest_language_2_scratch_test_6/scratch_test_6.js /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_6.dart -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/async_helper.sum=async_helper -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/collection.sum=collection -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/expect.sum=expect -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/js.sum=js -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/matcher.sum=matcher -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/meta.sum=meta -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/path.sum=path -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/stack_trace.sum=stack_trace -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/unittest.sum=unittest (working directory: /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2)

exit code:
1

stdout:
[error] Missing return value after 'return'. (out/ReleaseX64/generated_tests/language_2/scratch_test_6.dart, line 36, col 32)

Please fix all errors before compiling (warnings are okay).

--- Re-run this test:
python tools/test.py -m release -c dartdevc --strong --host-checked -t 960 language_2/scratch_test/6

[00:10 |  62% | +    0 | -    5]
FAILED: dartdevc-chrome release_x64 language_2/scratch_test/2
Expected: Pass
Actual: CompileTimeError
Unexpected compile error.

--- Command "dartdevc" (took 10.000596s):
DART_CONFIGURATION=ReleaseX64 sdk/bin/dartdevc --dart-sdk-summary /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/ddc_sdk.sum --library-root /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2 --ignore-unrecognized-flags --no-summarize --no-source-map -o /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_compilations/dartdevc/multitest_language_2_scratch_test_2/scratch_test_2.js /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_2.dart -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/async_helper.sum=async_helper -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/collection.sum=collection -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/expect.sum=expect -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/js.sum=js -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/matcher.sum=matcher -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/meta.sum=meta -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/path.sum=path -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/stack_trace.sum=stack_trace -s /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/gen/utils/dartdevc/pkg/unittest.sum=unittest (working directory: /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2)

exit code:
1

stdout:
[error] The return type of getter 'foo' is 'int' which isn't assignable to the type 'String' of its setter 'foo'. (out/ReleaseX64/generated_tests/language_2/scratch_test_2.dart, line 12, col 1)

Please fix all errors before compiling (warnings are okay).

--- Re-run this test:
python tools/test.py -m release -c dartdevc --strong --host-checked -t 960 language_2/scratch_test/2

[00:14 | 100% | +    2 | -    6]

Note that 'test.dart' transforms the expectation to Pass even though the test says static type warning; this is being handled in #36123, and in this case it does not change the situation (which is that the test fails because the outcome is CompileTimeError, and the expectation is different from that).
The subtest 3 (about default values) "succeeds" (which means that the warning is not emitted, even though it should be, but the expectation is Pass so 'test.dart' is happy). In this case the failure is masked until #36123 is resolved.
All other subtests fail because the outcome is CompileTimeError, but the expectation is static type warning.
Note that it's being debated whether the invalid return; statements should be an error or a warning (https://github.com/dart-lang/language/pull/255), which is significant for subtest 5 and 6.

Current status: This topic needs to be clarified, so we'll make it 'area-language' until a decision has been made on the use of warnings in Dart in general.

Closing, the language change decision is handled via dart-lang/language#1012.


Draft: virtio-net implementation using passt

Missing things:

[x] disable TSI hijacking in passt mode
[ ] tests
[x] conditional compilation
[ ] automatically starting passt
[ ] port mapping support  (requires passing command line arguments when starting passt)
[ ] handling passt process quitting
[ ] automatically setting interface IP in init.c - passt supports DHCP
[ ] performace optimizations

[ ]  TCP seems to sometimes slow down, I suspect it may be better to drop packets sometimes instead of blocking and being slow
[ ] use vectorized IO - e.g. sendmsg

Slightly more complex, forward looking API could look something like this: https://gist.github.com/mtjhrc/76148ab3de9ca0c37e109f4f668d523e

More complex, forward looking API could look something like this: https://gist.github.com/mtjhrc/76148ab3de9ca0c37e109f4f668d523e

It's an interesting approach, but I'd prefer to have transparent structs that are both understood by C and Rust sides. This allows us to easily identify API breaking changes that should be dealt by increasing the major version of the package.
Furthermore, I think we should avoid implementing starting passt from libkrun, and always require to be passed a file descriptor for the opened socket, leaving that work to the library consumers (crun, krun, chroot_vm...).
This means we can get away by extending the API with a single new function call like this:
krun_set_net_passt(uint32_t ctx_id, int fd, uint32_t flags);

I'm not really sure if we need the flags argument if the binary is executed by our consumers. If it isn't, it can be dropped. If that function is not called, libkrun will default to the original behavior, using TSI.

Note that I cherry-picked the commits from virtio-gpu PR, to resolve clippy warnings, so they should not be part of this PR


Empty multidimensional arrays are incorrectly serialized

I noticed that empty multidimensional arrays are incorrectly serialized. They are serialized as just one pair of square brackets regardless of the array dimension (tried 2D and 3D). This results in an error when deserializing the JSON string.

I have JKI JSON <IP_ADDRESS> installed

Fixed in 2.0.1


Poster image for youtube-videos

Should it be possible to use poster-images with youtube-videos? I tried it and the poster-image shows, but when I click play, the youtube-video does not load. With a local video, the poster-image works fine.
Thanks
Michael

Really this is managed on YouTube but I might be able to work something out in a future version.

Dear Sam
I forgot to include the use cases:
In our rather large governmental organisation most of the employees cannot
access youtube - management decision :-(
The plattform ist completey blocked on the firewall, so they see errors on
our own website, when a youtube-video ist embedded. A poster-image would be
nicer than an error-message, after the player I would additionally offer a
link to a local mp4-version of the youtube-video.
My other thougts were some sort of easy branding - for very short videos we
avoid special lead and credits with logo and so on.
And a better control over the preview-image, than on youtube would be nice.
Thanks
Michael
2016-07-05 7:20 GMT+02:00 Sam Potts<EMAIL_ADDRESS>
Really this is managed on YouTube but I might be able to work something
out in a future version.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Selz/plyr/issues/294#issuecomment-230389582, or mute
the thread
https://github.com/notifications/unsubscribe/ABVlN7gwY78L7lpfVljcRsihLLWCrk20ks5qSemxgaJpZM4JA-Bv
.

+1
I totally agree that it would be amazing if it's possible to set a poster image. Maybe through "data-poster" attribute. This also solves the issue that the YouTube-Play-Button is still visible behind the Plyr-Play-Button.
 Is there any way I can help?

Some time ago, I made a player for Neos, where I load the iframe on demand: https://github.com/jonnitto/Jonnitto.PrettyEmbedYoutube/blob/master/Resources/Private/Javascripts/Embed.js If I can help @SamPotts , let me know

+1 for this feature.  Agreed it can be handled in YouTube but the power to override that would be very useful.  To give an example scenario, my client has videos he wants to show on his site but the videos are stuck on the account of the company that produced the videos for him and are being very slow about updating the thumbnails to his preference.

Plyr now has a poster element and automatically sets it for youtube. You can set it after creating the instance with instance.poster = .... Should be pretty easy to support setting a custom youtube poster though.

@friday
can this be set for vimeo as well?

@LittleHamster Yes. The method is there and work regardless as long as it's video.

Working with data-poster/poster in
Example :


Use sudo -v so user only needs to enter credentials at the beginning

EDIT: In this issue I initially referred to sudo -k accidentally instead of sudo -v.

Before filing a bug, check the following:

Do you use the latest release of aurutils and its dependencies? If so,
is the issue reproducible from the master branch?

No, but neither of the commits since 0df4848 seem relevant

Does the package conform to PKGBUILD(5) and the AUR package guidelines?
Does the package provide the correct metadata on the
AUR RPC interface?
Does the package build with makepkg -s or extra-x86_64-build ?

N/A

Is the problem reproducible, and not due to a misconfiguration of
pacman, makepkg, sudoers, gpg or others?

Yes, but with some difficulty

If yes to all, create a debug log:
AUR_DEBUG=1 aur <name> <arguments> >aurutils.log

and attach aurutils.log to this issue.

As stated above, this would be annoying to generate and I donʼt think itʼs useful; I can try to re-test on the latest version, with logs, if requested.

Sometimes when a build is particularly long, sudoʼs cached credentials expire and the user needs to re-enter them, in the middle of a build.  This makes it impractical to run aur sync -c (I havenʼt, and wonʼt, check if the -c makes a difference because I donʼt want to build anything out of a chroot) and leave it running, but itʼs also impractical to watch it the whole time when most of the time itʼs giving output on various packagesʼ build progress which isnʼt interesting unless youʼre the packager or developer.
This should probably be fixed, such as by calling sudo -v in a loop in parallel with the main script (and killed when the main script exits, of course), unless thereʼs a better solution I donʼt know.

Running sudo -k in a loop assumes that users have password caching enabled, which isn't necessarily the case.
Have you tried using the proposed solution from the man page?

I didnʼt notice that solution when I looked through the manpage; I think I only looked at the aur-sync one instead because I canʼt always keep track of the whole architecture and remember to look at lower-level ones.
I donʼt like adding programs that can be run without passwords to the sudoers file, but it might be fine in this case.  I still feel like itʼs something that aurutils sholud be able to handle by itself ideally, but Iʼm not sure thatʼs possible without something more complicated.

I donʼt like adding programs that can be run without passwords to the sudoers file, but it might be fine in this case.

It's a better approach than a blanket sudo -v loop since if you use password caching, it's effectively the same as adding NOPASSWD for any command running sudo. This includes malicious or negligent sudo calls in the PKGBUILD.

I still feel like itʼs something that aurutils sholud be able to handle by itself ideally, but Iʼm not sure thatʼs possible without something more complicated.

I've tried a few times to have aur-build drop privileges so you can run sudo aur-build, but an issue is the reliance on makepkg -s to install dependencies. Since -s is a fixed part of makepkg, this part would have to be reimplemented by aur-build. See also #578.

Generally, you wonʼt be doing anything else with the terminal while using it for aur commands, so I think it only exposes you to malicious sudo calls in PKGBUILDs and not really elsewhere (unless you encounter the malicious command within a few minutes of the end of the aur script).  And youʼre already exposed to them with the default configuration.
I think Iʼll look into the build user option; that does seem like the best combined option as long as it isnʼt too hard to set up.  And yeah, going on a permissions roller coaster with lots of automatic elevating and dropping seems like it would cause its own problems.

Exposing malicious PKGBUILDs is probably the worst case senario while also being the one case that is trivial to encounter.  This isn't something aurutils is responsible for because it is an interface issue of both makepkg and sudo. Fortunately sudo does provide you with a remedy even if you "don't like" its suggestions.

I do look over PKGBUILDs and would probably notice one that wanted to use sudo.  This isnʼt an ideal solution (anything that requires diligence every time and doesnʼt let you make a mistake is prone to issues), but itʼs the best thatʼs available for the AUR.
I think making a build user completely addresses my worries about making something NOPASSWD; I was worried about those having issues that could be exploited in other ways, but if nothing can run as the build user except through manual intervention that makes exploiting any issues harder.

Yeah, using a specific user for this is a good idea

#743 gives some more flexibility when using a separate user.

Closing, feel free to re-open if there are more questions.


Need conio.h Alternative

conio.h library provide getch(); and other similar function, LibAstro need to a function similar to getch that works by reading input charecter by charecter but it does not store it in buffer so it returns immediately without waitting for the enter key "return key".
The most common solution is to use ncurses lib but that will make LibAstro useless, or by implementing a function that is similar to getch().

Created new function set that resolved the issue in file input.cpp


afloat made finder permanently "not responding" (even after reboot)

https://github.com/norio-nomura/EasySIMBL/issues/27

michael, do you still use afloat? i'm having the same issue and still have not found a solution, despite intensive googling.

I am not. I'm sorry @lynnex


TCOBSR

Analoguos COBS/R, the last sigil byte could be dropped, when it would be the only one and the then last byte interpreted as sigil byte would cause a sigil byte chain error. When decoding, such situation tells, that the whole message is without any sigil byte.

Not needed


Fixed error when potodo is ran outside of git repositoory

Hello @Seluj78! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file potodo/github.py:

Line 32:89: E501 line too long (99 > 88 characters)


Outdated Azure Portal Screenshots + missing steps for linking storage accounts

Screenshots on https://docs.microsoft.com/en-us/azure/remote-rendering/how-tos/create-an-account#link-storage-accounts do not reflect the current portal
Missing step to enable identity on ARR Resource

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

ID: d6c3c184-b242-c147-d7b6-609d237aa573
Version Independent ID: ff5bf80a-0ed3-ada0-2b36-c7ce69d24ce5
Content: Create an Azure Remote Rendering account - Azure Remote Rendering
Content Source: articles/remote-rendering/how-tos/create-an-account.md
Service: azure-remote-rendering
GitHub Login: @FlorianBorn71
Microsoft Alias: flborn

Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

Hi,
thanks also from the remote rendering team for your feedback. The issue should be fixed now.
Greetings


[BIOS] text: cursor implementation

The text cursor API allows blinking a specific rectangular area between the text color and the cursor color every N VBlanks.
While the IRQs are currently implemented as a stub, the blinking itself is currently not implemented.

First guess: https://github.com/asiekierka/AthenaOS/commit/00e7291e600d639f71752d88b0162dd406a2fe3b


Change MakeReadAsyncBlocking default to false

Now that the underlying open result set counter issue is fixed, it should be safe to change this to false and regain the performance we lost when we had to change it to true.

azp run "CI-BuildNPack"

/azp run CI-BuildNPack


Question about write with response semantics

Today I was investigating a few issue in our app and had to turn on verbose logs. I noticed on a write that the log contained a strange value.
func peripheral(
  _ peripheral: CBPeripheral,
  didWriteValueFor characteristic: CBCharacteristic,
  error: Error?
) {
  RxBluetoothKitLog.d("""
    \(peripheral.logDescription) didWriteValueFor(for:\(characteristic.logDescription),
    value: \(String(describing: characteristic.value?.logDescription)),
    error: \(String(describing: error)))
    """)
  peripheralDidWriteValueForCharacteristic.onNext((characteristic, error))
}

My questions are:

Why are you logging characteristic.value after the write? (Read / Write values do not have to match.)
Is the value supposed to be updated after the write?
If yes, is it by default?
If it's a default, then would it make sense also to emit a value update notification?

We haven't thought about logging of the value before. Probably someone that have added that logging though that it will have updated value.
I cannot say if value supposed to be updated after the write - it is not written in Apple's documentation so we would need to test it to check what behaviour there is.
Have you maybe checked this? Also is that a problem that library is logging that value?

If it's a default, then would it make sense also to emit a value update notification?

Not sure if I understand what you mean but logging and emitting new value is done in peripheral(_ peripheral: CBPeripheral, didUpdateValueFor characteristic: CBCharacteristic, error: Error?)

No issues with library, I just was wondering. Some of our characteristics were adopted to follow a reactive like behavior to emit a new notification when they were updated.
While investigating a few things I was confused by the verbose logs since I myself do log the value I will write to the characteristic but the log from your library reported a different value.
It was more like: Will write 03 and then did write ff000003 (made up numbers). I drilled down the implementation found that you're just writing the current value.
Then I thought maybe I don't know all semantics on how writes work asked you if maybe it's by default that the value of a characteristic is re-read after a write happens.
I think I can now answer that question by myself: A characteristic can be write-only or notifying-write-only which makes a updating of the current value impossible.
Thank you for taking the time for the short discussion. I think you can close the issue now. ;)

I felt like it is more like one way discussion, but yeah, thanks for it ;) Good to know that things.
Closing ;)


EventSubChannelRedemptionUpdateEvent broadcaster/userName returns an empty string

Bug Report
EventSubChannelRedemptionUpdateEvent broadcasterName and userName properties returns an empty string
Expected behavior
It should return the login name or username of the broadcaster/user

Actual Behavior
It returns an empty string

Environment

Version: 4.4.2
Node version: v15.6.0
Operating system: Windows 10

This is a Twitch issue, please track https://github.com/twitchdev/issues/issues/299 instead.


Remove Depot registry pull token auth

We have implemented automatic authentication for the Depot registry inside Depot Actions runners - this means that because the test and check_db jobs run on Depot runners, they now automatically have permission to pull the build container from the Depot registry without needing to create a new depot pull-token as part of the workflow.
This simplifies the workflow and also fixes an issue where if those jobs were retried greater than 60 minutes after the pull token was created, they would fail to pull the build container with the stale credentials.
Eventually this new automatic auth feature will be enabled via a toggle in the Depot UI, for now we've manually enabled it for your pypi org on the backend.
cc @ewdurbin @miketheman

Thank you @jacobwgillespie!


Out of bounds inflate access. Close not found.

Qwik Version
0.0.107
Operating System (or Browser)
Windows / Chrome
Node Version (if applicable)
16.13.1
Which component is affected?
Qwik Runtime
Expected Behaviour
I created a qwik city basic project and added a basic component for testing called user menu.
//usermenu.tsx
import { component$, useStore } from<EMAIL_ADDRESS>
export const UserMenu = component$(() => {
  const store = useStore({isOpen: false});
  return (

      <button onClick$={() => (store.isOpen = !store.isOpen)}>
        User User
      </button>
      {store.isOpen &&
        User Menu is open!
      }

  );
});

I then modified the layout.tsx to include that component.
I expect the component to render, and be interactive
Actual Behaviour
Component renders fine, but the moment I click the button, nothing happens and the javascript console has the following errors
Uncaught (in promise) Error: close not found
    at findClose (virtual-element.js:303:11)
    at getVirtualElement (virtual-element.js:281:23)
    at processVirtualNodes (virtual-element.js:268:25)
    at hasQId (store.js:766:18)
    at Object.acceptNode (store.js:494:20)
    at getNodesInScope (store.js:499:34)
    at resumeContainer (store.js:77:5)
    at resumeIfNeeded (props.js:18:9)
    at useLexicalScope (use-lexical-scope.public.js:31:9)
    at UserMenu_component_div_button_onClick_tlPB8QUzzQU (usermenu_component_div_button_onclick_tlpb8quzzqu.js:3:21)
    at dispatch ((index):101:33)

Additional Information
This may be related to #1093
I would be willing to try work arounds, and spent some time trying to trace into it a bit, but not familiar enough with the framework to know how to go about troubleshooting further.

As requested, here is an example of the layout
// layout.tsx in stock location for qwik city app
import { component$, Slot } from<EMAIL_ADDRESS>import { UserMenu } from "../components/layout/header/usermenu/usermenu";

export default component$(() => {
  return (

     <UserMenu />
     <Slot/>

  );
});

The user menu is located at /src/components/layout/header/usermenu/usermenu.tsx. I could make usermenu.tsx just index.tsx if that is expected to help at all.

Another update...
Directly out of the box it works as expected. I chose tailwindcss in the little cli tool and after adding
<link rel="stylesheet" href="http://https://rsms.me/inter/inter.css"/>

Between head tags in root.tsx
and
@tailwind base;
@tailwind components;
@tailwind utilities;

In global.css, the problem immediately starts occuring. I can get it to work with the tailwind imports, but to get it working after linking the inter.css font, I have to remove the link tag, and restart dev.
Is there a more appropriate way to import a 3rd party css from a cdn or something?

ok, thats what i meant… this problem usually happens with a lot of things happens at the same time. I will look at it

Followed the steps, and i cant reproduce the issue

Do we currently have a place like stackblitz where I can setup a reproduction and send a link to a place here? All i had to do was create qwikcity app with tailwind, make those changes to root.tsx (adding a css link to inter.css between head tags) and the tailwind init stuff in global.css.
It may require a restart of dev server, but thats literally all I did... and then clicking my button which changes state caused that error.

@ronnyek for stackblitz reproduction, it can be setup like this https://github.com/youngboy/yb-sb-starters , or you can just use mine 😅 and update deps

https://stackblitz.com/edit/github-k7xvbn?file=src/root.tsx
I simply added my component, added the link to inter.css in the head tags in root.tsx. If you click the button, then look at errors in the js console, you should be able to see the same error

@manucorporat can you confirm this reproduction is providing enough?

yes, the problem is that  does not have a Slot, so the link is projected in a weird spot inside the  element.
Put the  inside the head component in the head.tsx file, and it should work.
In the meantime, we have updated the starters to make it less error prone.
https://github.com/BuilderIO/qwik/blob/main/starters/apps/blank/src/root.tsx#L16

Hahaha my bad... I totally didn't realize that head tag was an actual component and not just plain html head tags.
I'm sorry I lead everyone on a goose chase here.


fuzz_blockers.py misses some bugs

Some bugs such as https://bugzilla.mozilla.org/show_bug.cgi?id=768756 set the whiteboard like  [fuzzblocker:editor-assertions]. Those are getting missed.
Originally posted by @mathewhodson in https://github.com/mozilla/relman-auto-nag/pull/1564#discussion_r933602912

Good catch, thanks for reporting!

Thank you for reporting this. We reached out to the fuzzing team, who confirmed that this format is no longer supported. The fuzzing team is cleaning open bugs with such a tag format.

Tyson:
We have only been using [fuzzblocker] for at least 3(?) years now


Adapting build scripts for Visual Studio 2015

A start for fixing #432.
I do get the error reporter in the issue thread when building, so these will have to be fixed.

Looks good so far. Related issue: @ploeh should we have a CI build connected to GitHub so we can see the build results here? I believe you have teamcity setup already, is that right? We just need to configure it to report back to GitHub?

Looks good.
@adamchester, from the discussion in #432, I deduce that you have an installation of Visual Studio 2015 without Visual Studio 2013. Have you tried running the build scripts in this pull request on that system? If so, is all good?
I haven't gone around to installing Visual Studio 2015 yet, so my system only has Visual Studio 2013. Both build-release.sh and BuildRelease.ps1 work on my system with this pull request.
If I can get confirmation that it also works on a system with Visual Studio 2015, I'll merge this pull request.

When it comes to connecting a CI build to GitHub, I was actually contemplating decommissioning the centralised build for AutoFixture, because I consider a build server brittle, unreliable, and a central point of failure.
Yet another thing I need to blog about...

@ploeh I look forward to that blog post, and I look forward to watching the arguments it generates :smile:
I can confirm that powershell .\BuildRelease.ps1 correctly builds on a fresh Windows 10 with only VS 2015 installed (no prior versions ever on this machine). There are zero warnings.
From VS 2015 developer command prompt, typing msbuild BuildRelease.msbuild (note: this means MSBuild v14 is used which is a different, newer MSBuild than the normal MSBuild AutoFixture uses) still generates the warnings I mentioned in #432:
       "C:\repo\autofixture\BuildRelease.msbuild" (CompleteBuild target) (1) ->
       "C:\repo\autofixture\Src\AutoFixture.NUnit2.sln" (default target) (8) ->
       "C:\repo\autofixture\Src\AutoFixture.NUnit2\AutoFixture.NUnit2.csproj" (default target) (9) ->
       (ResolveCodeAnalysisRuleSet target) ->
         C:\Program Files (x86)\MSBuild\14.0\bin\Microsoft.CSharp.CurrentVersion.targets(133,9): warning MSB3884: Could
        not find rule set file "AutoFixture.NUnit2.ruleset". [C:\repo\autofixture\Src\AutoFixture.NUnit2\AutoFixture.NU
       nit2.csproj]

       "C:\repo\autofixture\BuildRelease.msbuild" (CompleteBuild target) (1) ->
       "C:\repo\autofixture\Src\AutoFixture.NUnit2.sln" (default target) (8:2) ->
       "C:\repo\autofixture\Src\AutoFixture.NUnit2\AutoFixture.NUnit2.csproj" (default target) (9:8) ->
         C:\Program Files (x86)\MSBuild\14.0\bin\Microsoft.CSharp.CurrentVersion.targets(133,9): warning MSB3884: Could
        not find rule set file "AutoFixture.NUnit2.ruleset". [C:\repo\autofixture\Src\AutoFixture.NUnit2\AutoFixture.NU
       nit2.csproj]

       "C:\repo\autofixture\BuildRelease.msbuild" (CompleteBuild target) (1) ->
       "C:\repo\autofixture\Src\AutoFixture.xUnit.net.sln" (default target) (17:2) ->
       "C:\repo\autofixture\Src\AutoFixture.xUnit.net\AutoFixture.xUnit.net.csproj" (default target) (18:5) ->
         C:\Program Files (x86)\MSBuild\14.0\bin\Microsoft.CSharp.CurrentVersion.targets(133,9): warning MSB3884: Could
        not find rule set file "AutoFixture.xUnit.net.ruleset". [C:\repo\autofixture\Src\AutoFixture.xUnit.net\AutoFixt
       ure.xUnit.net.csproj]

       "C:\repo\autofixture\BuildRelease.msbuild" (CompleteBuild target) (1) ->
       "C:\repo\autofixture\Src\AutoRhinoMock.sln" (default target) (32:2) ->
       "C:\repo\autofixture\Src\AutoRhinoMock\AutoRhinoMock.csproj" (default target) (34:5) ->
         C:\Program Files (x86)\MSBuild\14.0\bin\Microsoft.CSharp.CurrentVersion.targets(133,9): warning MSB3884: Could
        not find rule set file "Migrated rules for AutoRhinoMock.ruleset". [C:\repo\autofixture\Src\AutoRhinoMock\AutoR
       hinoMock.csproj]

    4 Warning(s)
    0 Error(s)

I believe it's safe to call this a separate issue, just wanted to note it.

I think @moodmosaic has a system where only VS 2015 is installed.
On my side, having VS 2015 on my system and 2013 uninstalled, I get the warnings when running the scripts.

@Pvlerick which warnings do you get?

@adamchester the ones you mentioned here.

@Pvlerick OK that is due to MSBuild v14.0 being used, right?

@ploeh just on the topic of builds, it's somewhat easy for us to setup our own CI builds... The problem is, it would be nice to see the big warning against a pull request (eg. Warning when build could not be completed, or big red error for a build failure). It allows discussion/diagnosis/training/etc.
In case anyone is interested here is mine for AutoFixture https://ci.appveyor.com/project/adamchester/autofixture
Here are some settings for appveyor (relevant here because you can test the CS 2015 build along with other configurations):
version: 1.0.{build}
os:
- Default Azure
- Visual Studio 2015
clone_depth: 1
build:
  project: BuildRelease.msbuild
  parallel: true
  verbosity: minimal
artifacts:
- path: NuGetPackages/*

@adamchester Yes I think so.

Running build-release.sh on a clean Windows 8.1 environment with only Visual Studio 2015 installed yields 2 warnings:
Build succeeded.

"h:\Projects\AutoFixture\BuildRelease.msbuild" (default target) (1) ->
(Test target) ->
  MSBUILD : warning MSB6006: "nunit-console.exe" exited with code -2146232576.
[h:\Projects\AutoFixture\BuildRelease.msbuild]
  MSBUILD : warning MSB6006: "nunit-console.exe" exited with code -2146232576.
[h:\Projects\AutoFixture\BuildRelease.msbuild]

    2 Warning(s)
    0 Error(s)

Then, I get prompted to download and install .NET Framework 3.5:

That sounds a bit inconclusive to me. Should I pull, or shouldn't I?
This pull request doesn't have to make everything perfect. The important point is that it makes the situation better for someone, without making it worse for someone else (that's a Pareto improvement, for anyone who cares).
It's not causing me any problems, so I'd be happy to pull, but I also don't want to cause trouble for other people.

@ploeh I think it's better to wait that we figure out what is wrong for the "VS2013 and VS2015 on the same machine" setup, since it's likely to be common.

I run a couple of builds, and the scripts are now executing successfully on each environment:
Visual Studio 2015
After installing .NET Framework 3.5 (which was required also by TestDriven.Net) the build-release.sh and BuildRelease.ps1 scripts run without warnings and without errors.
Visual Studio 2013 and Visual Studio 2015
The build-release.sh and BuildRelease.ps1 scripts run without warnings and without errors.

At this point, based on the results on my machine, we could be good to go.

:+1: I think it makes things better, the other issues already existed.

After updating my system to Windows 10 yesterday, it builds with no warnings, go figure...
@ploeh  I think we are goot to go :-)

Thank you for your contribution!
Since this is a build/compile concern only, there's no release associated with this pull request.


Updating Doc to Follow new changes

Removed the shame from https://github.com/erikng/installapplications/commit/e5b8f04cefa8537a713335c4f5d22609e26068fc.

Heh thanks man!


Can't access some identifiers on a Dictionary

Godot version
803d25a
System information
Windows 10 x64
Issue description
Doing e.g.
var dict = {}
dict.class = "Node"

results in Expected identifier after "." for attribute access. and Expected end of statement after expression, found "class" instead., spammed tens of times. The identifier is incorrectly recognized as a keyword.
Works correctly in 3.x
Steps to reproduce

Use code above

Minimal reproduction project
No response

Just stumbled upon this too. To add a bit of information it happens with all reserved keywords (tested with a dozens of the randomly) except for built-in type name.

So the bug seems to be linked to the list given here in the doc

I followed the link given in the docs that i gave just before and it seems there is no difference marked between keywords and other tokens (such as punctuation, parentheses, etc). I dug a bit more but i'm not acustomed to the source code of Godot. I found that the function parsing is advance in gdscript_parser.cpp. It calls the scan function from gdscript_tokenizer.cpp which will automatically consider the reserved keywords as specific tokens.
The solution might be in the function advance to add a test before scanning to verify if the previous token is not a period and if it is to call a function that would first verify it can be tokenized as an identifier and if it is tokenize it accordingly. If it can't be tokenized as an identifier we can tokenize as normal because we just wan't it to tokenize normally and raise the exception as it is already the case.
As i just realised there is no problem with the $ syntax which means that something similar as already been implemented and can be reused effectively as both cases are quite similar

This is troublesome as Object.get_signal_connection_list() returns a dictionary array where one of the keys is "signal", which triggers this error.

You can use connection["signal"]


Add GitPoap & Degenscore Data Provider (#1555)

Hey @MarkuSchick @mme022 @3Vis I fixed the 2 Data Providers and it works now :)
We will see them in the factory.sismo.io in a few minutes


AzureAD Functional Testing

💡 Summary
As part of Iceberg release, to ensure that the tool's update is accurate, detailed functional testing has been planned. Each policy will be functionally tested to check that all pass/fail scenarios have been correctly processed by the tool using existing automated functional test plans.
This issue functionally tests each policy of Azure AD product of ScubaGear tool. The main objective is to test Azure AD policies and ensure that ScubaGear tool is providing the correct result for each possible configuration option.
Motivation and context
This functional testing ensures that ScubaGear results of pass/fail are accurate based on the latest version of the code.
Implementation notes
For each Azure AD policy do the following:

[ ] Configure the automated testing and run against each tenant type for which a variant testplan is present.  Run the automated test plans (e.g. regular, g5, etc.) and note the results.
[ ] For each test failure, investigate and note the cause of the failure and log any new issues needed to resolve

For details on how to use and execute the functional tests, please see the Testing documentation in Testing/Readme.md or reach out to the dev team via other channels.
Acceptance criteria

[ ] All existing test plan variants for the product have been successfully run
[ ] Test plan results have been reviewed and any issues resolved or reported as new issues (Link created issues below)

E5

##E5 Variant

G5

##G5 Variant

G3

##G3 Variant


possible bug in function sample_true_false_edges_partial

Hi Xie!
I came across some code lines that i think might have a bug? could you give me your opinion on it?
its in file TOLP.py on lines 1782-1784
I also added a screenshot:

considering the other places with similar code, shouldnt line 1784 be as such:
pos_edges = sparse.find(sparse.triu(A_diff,1)) # true candidates
or am I mistaken?
otherwise A_diff is never used inside this loop.
if it is as it should, can you tell me why?
Thanks,
Geut.

Hi Geut,
This is indeed a feature implemented because partial observation should be allowed to see the features from the training target layer as well, which is why the A_tr is directly used instead of using the A_diff, which is the one that should be used if we are considering the completely missing cases. If there's still confusion please let me know.
Best,
Xie

Ok! thank you for the explanation!
so lines 1782-1783 are unnecessary? are fine if deleted?
Best,
Geut.

Yes, that would be fine, I will see to it to be updated as well!
Thank you for finding the issue! Please feel free to submit more if you
find anything or email me directly.
Best,
Xie
On Mon, Dec 16, 2024 at 4:12 AM Geut Galai @.***> wrote:

Ok! thank you for the explanation!
so lines 1782-1783 are unnecessary? are fine if deleted?
Best,
Geut.
—
Reply to this email directly, view it on GitHub
https://github.com/hexie1995/Sequential-Link-Prediction/issues/1#issuecomment-2545466320,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJL7OJHBM2BD237JSPJFGTD2F27TJAVCNFSM6AAAAABTURNDM6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBVGQ3DMMZSGA
.
You are receiving this because you commented.Message ID:
@.***>


AttributeError: 'module' object has no attribute 'SummaryWriter

AttributeError: 'module' object has no attribute 'SummaryWriter

@shede1988  how to solve it?


How to load Dark Mode by default?

I only need dark mode, how can i load dark mode by default?

As of now the first load is based on the system preference, then if the user changes the theme, the new preference is stored in the local storage.

As of now the first load is based on the system preference, then if the user changes the theme, the new preference is stored in the local storage.

can we have this as a parameter in config.toml?

@shawsa-artc You can easily do this using custom js file.
Add a custom.js file with this content under the public directory:
eval(console.log('Setting the default theme as dark.'));
eval(localStorage.setItem('themeVariant', 'dark'));

Now refer this file in config.toml as shown below:
custom_js = "/public/custom.js"

After doing this change when you restart the chainlit app, it will load the dark theme by default :-)


feature-bug/created-date-and-utils-star

PR Overview
This PR will address the following Issue/Feature: Issue #117 and Feature #116
This PR will result in the following new package version: v0.13.0

Since we are changing the created_at field to now be created_date, this will result in a breaking change that customers will need to take into account.
Please detail what change(s) this PR introduces and any additional information that should be known during the review of this PR:
🚨 Breaking Changes 🚨

The created_at field within the stg_hubspot__contact model has been renamed to created_date to be consistent with the source data. Additionally, this ensure there are no duplicate column errors when passing through all property_* columns which could potentially conflict with property_created_at.

Feature Updates

All stg_hubspot__*_tmp models have been updated to leverage the dbt_utils.star() macro. This ensures if the source dimension changes there is no potential for a mismatch in columns error that is commonly seen in Snowflake destinations.

PR Checklist
Basic Validation
Please acknowledge that you have successfully performed the following commands locally:

[X] dbt compile
[X] dbt run –full-refresh
[X] dbt run
[X] dbt test
[X] dbt run –vars (vars used in the integration tests)

Before marking this PR as "ready for review" the following have been applied:

[X] The appropriate issue has been linked and tagged
[X] You are assigned to the corresponding issue and this PR
[x] BuildKite integration tests are passing

Detailed Validation
Please acknowledge that the following validation checks have been performed prior to marking this PR as "ready for review":

[X] You have validated these changes and assure this PR will address the respective Issue/Feature.
[X] You are reasonably confident these changes will not impact any other components of this package or any dependent packages.
[X] You have provided details below around the validation steps performed to gain confidence in these changes.

dbt_utils.star() validaton

I was able to confirm that the inclusion of the dbt_utils.star() macro in each of the *_tmp models did not cause unforeseen issues. They still operated as the normal * had worked previously. However, now we see the full list of columns specified in the compiled code which will ensure the columns cannot mismatch when the source changes.
Original compiled code

select *
from "postgres"."hubspot_source_integration_tests_6"."company_property_history_data"

New compiled code

select "company_id",
  "timestamp",
  "_fivetran_synced",
  "source",
  "source_id",
  "name",
  "value"
from "postgres"."hubspot_source_integration_tests_6"."company_property_history_data"

I also want to add that I tried to leverage the var() in the star macro. However, this caused the following error:

18:58:41  Running with dbt=1.6.3
18:58:41  Registered adapter: postgres=1.6.3
18:58:42  Encountered an error:
Compilation Error in model stg_hubspot__company_property_history_tmp (models/tmp/stg_hubspot__company_property_history_tmp.sql)
  Macro star expected a Relation but received the value: "postgres"."hubspot_source_integration_tests_6"."stg_hubspot__company_property_history_tmp"

  > in macro _is_relation (macros/jinja_helpers/_is_relation.sql)
  > called by macro default__star (macros/sql/star.sql)
  > called by macro star (macros/sql/star.sql)
  > called by model stg_hubspot__company_property_history_tmp (models/tmp/stg_hubspot__company_property_history_tmp.sql)

Therefore, we needed to leverage the source() call. I am comfortable with this is the use of our identifier variables allows the customer to still switch this around if they wish. I was considering adjusting the *_tmp models to not use the var(). However, I didn't think this was entirely necessary and wanted to discuss with the team before applying those changes at scale in this update.

Created At -> Created Date Validation

I was able to recreate the initial error the customer was seeing. Then once using the version of code that is applied in this PR I saw the issue be resolved. The propery_created_at field was added to the seed file so we may ensure this is being tested going forward.

Standard Updates
Please acknowledge that your PR contains the following standard updates:

Package versioning has been appropriately indexed in the following locations:

[X] indexed within dbt_project.yml
[X] indexed within integration_tests/dbt_project.yml

[X] CHANGELOG has individual entries for each respective change in this PR

[X] README updates have been applied (if applicable)

[n/a] DECISIONLOG updates have been updated (if applicable)
[X] Appropriate yml documentation has been added (if applicable)

dbt Docs
Please acknowledge that after the above were all completed the below were applied to your branch:

[X] docs were regenerated (unless this PR does not include any code or yml updates)

If you had to summarize this PR in an emoji, which would it be?

🌟

also for the created_at -> created_date part, is that something we should apply to other tables as well?

@fivetran-jamie great call out here! I hadn't thought about the need to update this in other places since the contact model was the only one the original issue was created within. However, when thinking about the issue as a whole I could see this happening within the company, deal, and ticket models in addition to the contact since we do a similar renaming. After reviewing them I felt it was best to take your advice and make the updates in this PR to account for those future possible issues. Please see my updates for the changes.
Additionally, when reviewing this slack thread I felt it would be an impactful and easy addition to the HubSpot package to introduce variables to disable the property history models. I introduced this variable for all but the ticket model since your PR is working heavily with this model. I would prefer we highlight that history model. Especially since tickets are disabled by default for now.


Memory leak(?) in Version 1.0.24

It seems to be a memory leak in the latest version when calling .exec() within a promise.
I tested two last versions (1.0.23 and 1.0.24) with node 10.14.2 using the test code below. The standard callback approach works just fine in both versions. But when using promises I observed significant heap usage growth in version 1.0.24.
Thank you very much in advance,
Viktor
Test results:

v.1.0.23:
Callbacks:
Start : heapTotal = 6.23 MB, heapUsed = 3.73 MB
Finish: heapTotal = 40.23 MB, heapUsed = 9.94 MB
Promises:
Start : heapTotal = 6.23 MB, heapUsed = 3.73 MB
Finish: heapTotal = 43.23 MB, heapUsed = 19.33 MB

v.1.0.24:
Callbacks:
Start : heapTotal = 6.23 MB, heapUsed = 3.73 MB
Finish: heapTotal = 43.23 MB, heapUsed = 9.95 MB
Promises:
Start : heapTotal = 6.23 MB, heapUsed = 3.73 MB
Finish: heapTotal = 617.23 MB, heapUsed = 578.01 MB

Test :
const conn = require('sqlanywhere').createConnection();

const dbParams = {
    // Server  : 'xxxxx',
    UserId  : 'DBA',
    Password: 'sql',
  };
const select = "SELECT row_num, 'test long long string' || row_num as str FROM sa_rowgenerator( 1, 50000 );"

//----------------------------------------------------------------------
// execute the select 'count' times using the standard callback approach
function testCallback(count) {
    const exec = function() {
        conn.exec(select, (err, result) => {
            if(err) {
                console.log(err);
            }
            else {
                console.log(`Iteration: ${count} Rows: ${result.length}`);
                if(--count > 0) {
                    setTimeout(exec, 0)
                }
                else {
                    conn.disconnect(err => logMemoryUsage(usedStart, process.memoryUsage()));
                }
            }
        })
    }
    conn.connect(dbParams, err => err ? console.log(err) :  exec());
 }

//----------------------------------------------------------------------
// execute the select 'count' times using Promises
async function testPromise(count) {
    const connect = async function() {
        return new Promise( (resolve, reject) => {
            conn.connect(dbParams, err => err ? reject(err) : resolve());
        });
    }
    const disconnect = async function() {
        return new Promise( (resolve, reject) => {
            conn.disconnect(err => err ? reject(err) : resolve());
        });
    }
    const exec = async function() {
        return new Promise( (resolve, reject) => {
            conn.exec(select, (err, result) => err ? reject(err) : resolve(result));
        });
    }
    try {
        await connect();
        for(let i = count; i > 0; --i) {
            const result = await exec();
            console.log(`Iteration: ${i} Rows: ${result.length}`);
        }
        await disconnect();
        logMemoryUsage(usedStart, process.memoryUsage());
    }
    catch(err) {
        console.log(err);
    }
 }

//----------------------------------------------------------------------
const usedStart = process.memoryUsage();
function logMemoryUsage(startMemory, finishMemory) {
    const toMB = mem => Math.round(mem / 1024 / 1024 * 100) / 100;
    console.log(`Start : heapTotal = ${toMB(startMemory.heapTotal) } MB, heapUsed = ${toMB(startMemory.heapUsed) } MB`);
    console.log(`Finish: heapTotal = ${toMB(finishMemory.heapTotal)} MB, heapUsed = ${toMB(finishMemory.heapUsed)} MB`);
}

//----------------------------------------------------------------------
// enable one of the following functions:
//----------------------------------------------------------------------
// testCallback(100);
testPromise(100);

I can confirm having memory leak issues as well using the most recent version.  I have a simple ExpressJS API setup to retrieve data for an internal webapp.
Using async/await the app quickly grows in memory usage crashes.
Just now refactored to not use async/await and the app's heap usage never gets close in comparison.

Has this been fixed?


Fail more cleanly on script errors

If we fail to get info from docker/ip/etc, fail immediately rather than failing with a syntax error due to substituting in an empty value later.
inspired by https://bugzilla.redhat.com/show_bug.cgi?id=1333393, though as noted there, we probably want a better solution eventually to ensure we don't leave cruft in OVS
@openshift/networking PTAL

LGTM

LGTM


Always keep session member view up-to-date

Closes https://github.com/TypeFox/open-collaboration-tools/issues/36
This fixes the issue of missing users in the session view. The main issue was that we didn't call onDidUsersChange in our collaboration instance on a few of the peer related requests. Especially important is the peer/init notification: Upon joining a room their view was empty - only after another user joins would their view get populated. Now the view is populated instantly.

Weirdly enough it worked fine yesterday but today i had the same problem as Miro. Seems like in this case when you are the host you are not in the list of peers in the collaboration instance. But when you are joining a session you are. Also on host side i only see the joined user and not myself as if in that case the host is not adding itself to the list.
No Idea why both sides worked fine yesterday. Maybe some build problem

@jonah-iden I've updated the PR some time ago. Please have another look if you some time.


Dining Card: Mounting Too Frequently

Issue Description
The dining card component is mounting much more often than other card components, indicating there is likely a problem causing the card to re-mount/re-render too frequently.

Hi @bethanybin, sending this your way as it will coincide with #358
This bug is wreaking havoc with our current analytics reporting. The issue is stemming from logic in watchLocation (locationSaga -> watchLocation -> UPDATE_DINING)