Unable to do transpose on a  yolo model

Hello,
I am trying to add a transpose layer at the end of Yolo model.
I am using the following script,
model = load_model(filepath="yolov2-tiny-voc.h5")
output = model.output
output_shape = model.output.shape
transpose = Lambda(lambda x: tf.transpose(output, prem=(0,3,1,2),name='transpose'))(output)
final_model = Model(inputs= model.input, outputs= transpose )
save_model(model=final_model, filepath= "trial_attempt.h5")
But I am getting the following error:
AttributeError: 'Conv2D' object has no attribute 'outbound_nodes'
Could someone help on this issue?

Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version. Also, did you compile from source or install a binary?
Make sure you also include the exact command if possible to produce the output included in your test case. If you are unclear what to include see the issue template displayed in the Github new issue template.
We ask for this in the issue submission template, because it is really difficult to help without that information. Thanks!

@gadagashwini ,
The platform is macOS High Sierra.
The tensorflow version is 1.13.1.
I have installed using pip.
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): macOS High Sierra
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: NA
TensorFlow installed from (source or binary): Binary
TensorFlow version (use command below): tensorflow 1.13.1
Python version:  python 3.6.6
Bazel version (if compiling from source): NA
GCC/Compiler version (if compiling from source): NA
CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce:

model = load_model(filepath="yolov2-tiny-voc.h5")
output = model.output
output_shape = model.output.shape
transpose = Lambda(lambda x: tf.transpose(output, prem=(0,3,1,2),name='transpose'))(output)
final_model = Model(inputs= model.input, outputs= transpose )
save_model(model=final_model, filepath= "trial_attempt.h5")
Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.
model = load_model(filepath="yolov2-tiny-voc.h5")
output = model.output
output_shape = model.output.shape
transpose = Lambda(lambda x: tf.transpose(output, prem=(0,3,1,2),name='transpose'))(output)
final_model = Model(inputs= model.input, outputs= transpose )
save_model(model=final_model, filepath= "trial_attempt.h5")
Let me know if you need any further information.

@Rathna21 Thanks for providing the details.

This question is better asked on StackOverflow since it is not a bug or feature request. There is also a larger community that reads questions there.
If you think we've misinterpreted a bug, please comment again with a clear explanation, as well as all of the information requested in the issue template. Thanks!


Getting No Application Key Specified even in artisan key:generate

Laravel Version: v7.13.0 (6fa69bfbd57744a5bbec5538ce483919b3fd625f)
PHP Version: 7.2.19
Database Driver & Version:

Description:
C:\Users\xxx\Documents\projects\web\xxx>composer run post-create-project-cmd
> @php artisan key:generate --ansi

   RuntimeException

  No application encryption key has been specified.

  at C:\Users\xxx\Documents\projects\web\xxx\vendor\laravel\framework\src\Illuminate\Encryption\EncryptionServiceProvider.php:80
    76|     protected function key(array $config)
    77|     {
    78|         return tap($config['key'], function ($key) {
    79|             if (empty($key)) {
  > 80|                 throw new RuntimeException(
    81|                     'No application encryption key has been specified.'
    82|                 );
    83|             }
    84|         });

  • Your app key is missing:
    https://laravel.com/docs/master/installation#configuration

  1   C:\Users\xxx\Documents\projects\web\xxx\vendor\laravel\framework\src\Illuminate\Support\helpers.php:433
      Illuminate\Encryption\EncryptionServiceProvider::Illuminate\Encryption\{closure}("")

  2   C:\Users\xxx\Documents\projects\web\xxx\vendor\laravel\framework\src\Illuminate\Encryption\EncryptionServiceProvider.php:84
      tap("", Object(Closure))
Script @php artisan key:generate --ansi handling the post-create-project-cmd event returned with error code 1

I guess this is not much to explain, just as what you guys can see.
p/s: I know I can generate a key from another project and use it but obviously this is a big issue.
Steps To Reproduce:

clone a laravel project from git
copy .env.example to .env
run php artisan key:generate

I saw same problems, php artisan key:generate doesn't work stable. Fifty-fifty, sometimes APP_KEY was set, sometimes not.
This behavior was on laravel 7, I didn't see this problem on laravel 5.8.
Now I setting this env-variable manually using external command:
php artisan -- env:set APP_KEY `php artisan key:generate --show`

I saw same problems, php artisan key:generate doesn't work stable. Fifty-fifty, sometimes APP_KEY was set, sometimes not.
This behavior was on laravel 7, I didn't see this problem on laravel 5.8.
Now I setting this env-variable manually using external command:
php artisan -- env:set APP_KEY `php artisan key:generate --show`

then I guess we are having different problem, because I am having problem with every artisan command

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

@driesvints this seems like a bug to me. My app key is specified and I get this behavior. Can't even run key:generate as described above. Can you either provide the solution or accept that this is a bug?

Problem is that the team can't duplicate it. Personally, I just followed the instructions in the original post, and it worked exactly as it should. The OP did leave out the important step of "composer install", but I think that was just an oversight. Either way, when I try to duplicate the issue, everything works as it should. Usually these problems are due to either outdated packages, or permission issues on the machine being used.
If you're using a different set of instructions than the OP, then this could be a different issue and needs its own issue thread.
git clone git@github.com:laravel/laravel.git
cd laravel
composer install
php artisan key:generate

I had the same issue as the OP and so did my dev. It's not ANY project, you would have to pull the same project to repeat the issue. I did run composer install, so that didn't seem to be the problem. I set permissions on .env to 0777, still no luck.
The solution for me was to paste in an existing key from another project, then run php artisan key:generate. Until I pasted in another key, key:generate would not work.

I actually found a reason for this and a fix.
https://github.com/laravel-foundation/console/pull/1/commits/91f31784af1bc95603b0eb2272b8b1392877d318


5.02b See response table as part of submissions list

Requires being on the cxlt/temp/meta backend branch.

minor comments, nothing major.


Spurious task dependency introduced through includeBuild leads to circular dependency error

I have three projects: a library, an application, and a platform. The library has three subprojects, variant-a, variant-b and common; common is a common dependency of variant-a and variant-b. When replacing the library dependency specified by the platform with variant-a in the application, Gradle fails claiming that there were a circular dependency between tasks.
Expected Behavior
The included library should be built exactly as it would be if I built it by itself.
Current Behavior
When I attempt to build, I get:
FAILURE: Build failed with an exception.

* What went wrong:
Circular dependency between the following tasks:
:library:common:classes
\--- :library:common:compileJava
     +--- :library:common:compileKotlin
     |    \--- :library:variant-a:jar
     |         +--- :library:common:classes (*)

The dependency of the task common:compileKotlin on variant-a:jar makes no sense and does not exist when building the library itself. It is spuriously added when including the build in another project.
Context
I wish to test the application against a development version of the library using a composite build.
Steps to Reproduce
With the setup described above, I add:
includeBuild("../library") {
    dependencySubstitution {
        substitute module("library group:library name") with project(":variant-a")
    }
}

to settings.gradle in the application project.
The inclusion of the platform may be related; see https://discuss.gradle.org/t/composite-build-fails-with-circular-dependency-when-using-own-platform-depedency-created-by-java-platform-plugin/33241.
Your Environment
This appears at least in Gradle version 6.0.1 through 6.4.

Looks like a duplicate of #13979. Would you mind checking if you have the same error with the latest 6.7 snapshot? https://services.gradle.org/distributions-snapshots/gradle-6.7-20200816220041+0000-bin.zip

I'm closing this, please reopen if the problem still happens on latest releases.


Bump safe-eth-py to v4.5.2

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.844%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5962

Relevant Lines:
6787

💛  - Coveralls


Failure to load save files.

Attach the save file
Add the .sav file to a zip, and attach the .zip file to this issue.
Aurora.zip
Describe the bug
Opening save displays Error Message:
Failed to load: Error: Unexpected component type 4 with flags 1
@https://railroad.studio/studio.js:1:279272
Ba@https://railroad.studio/studio.js:1:280193
Oa@https://railroad.studio/studio.js:1:271989
qa/</</l<@https://railroad.studio/studio.js:1:282772
qa/</<@https://railroad.studio/studio.js:1:282975
qa/s/<@https://railroad.studio/studio.js:1:281516
To Reproduce
Open attached save with current version.
Did open normally 2 days ago.
Desktop (please complete the following information):

OS: Win10
Browser: Firefox
Version: 128.5.1esr (64-Bit)

This issue was fixed in #239. Your web browser may have an old version of RS cached. You can try holding ctrl while refreshing, which should override the browser's cache in Chrome or Firefox. For other web browsers, you may need to consult the browser's documentation for instructions on how to clear your cache.

Thanks, it does now load with the Red Mountain Depot placed.


New Getting Started page

New getting started page. Thanks to @tomneedham for starting it, @camilasan for the help and @jancborchardt @MorrisJobke for the feedback!
This is making #369 ready for merging.
@kelleybrooks @karlitschek would love to hear what you think. Screen shots below.

Download button:

new 'cloud' page:

Client button(s):

Testing button:

Everything else redirects to other pages or gives a direct download.
What do we think?

great. As discussed. i love it. :+1:
The only thing we could add is the option to buy a device that offers ownCloud hosting. Like linking to qnap and synology and other who offer owncloud apps that can run on top of then.
This requires some investigation what exactly is where possible and available and what the links are.
No sure who could do that.

@karlitschek I actually thought about that, but for what you mention - it requires some more looking into. Something we can add later, I would say...

ok. agreed. The rest is great!!

Was just thinking yesterday how much the .org site has evolved.  This is really great stuff.  My suggestions are tiny...
-on Web Installer, a word other than novice would be good.  Some people just choose not to host.  It may be good to link to our providers page as that is constantly growing and I don't believe we link to it directly from nav/footer
-on desktop clients, we use the British spelling on "synchronise".  I know that much of the English on the site is British English whereas .com is US English.  You're call.  Just thought I'd highlight.
If I saw anything else, it's since escaped my brain.  These are a huge step in clarity....

addressed the comments by Kelley in the above commit as well. Keeping British English where it is but removed the 'novice'.

@jancborchardt I'd really like to hear your thoughts before I merge to staging and get it live... I have not yet made the changes needed to replace the current install/download pages.

It looks really nice! This also then replaces the /install and /sync-client pages, right? (Cause otherwise it would be a massive duplication of content it seems)
Just one thing: The modal should be shifted a bit down, as it currently weirdly overlays the header.

@jancborchardt yeah, it should replace them.
Let me see if I can figure out how to move those modal dialogs down a bit, then I'll consider this a :+1: and will merge it to staging...

You can see it on staging and play with it yourself: https://staging.owncloud.org/getstarted/
if no more problems found I will try and figure out how to get it to replace the install page... probably needs help from @tomneedham

Clicking on "mobile clients" in step two: Wrong tab is selected (it should be mobile clients but is desktop client) - the content is correct.

@MorrisJobke can't figure out how to fix the mobile clients thing. It should be done by the script code on the bottom of the page but it doesn't work and I dunno why :(
About the Windows thing - not yet, it works for now and we stop support for 8.1, right?

No it was dropped for 8.0 AFAIK

I really like it. :+1: from my side.
I think we should still keep the zip download option. But remove the mentioning of "windows"

I think we should still keep the zip download option. But remove the mentioning of "windows"

Yes. Sounds good :)

[ ] The OBS PGP key is missing on the packages site
[ ] "Looking for older versions?" is not really in the middle

[x] remove mentions of "windows"
[ ] wrong tab is selected (content is correct) when clicking on mobile apps link

You guys are pushing me out of my comfort zone. I just managed to fix the tabs - that's, technically, javascript stuff. And I only modified my first CSS a month ago. Curse you!

@LukasReschke see #469 - all changes made. Push this to staging too, then?


[Request] External DB editing clarification

I'm trying to use SQL queries to automate some changes in player tracker's db
As a simple example.
INSERT INTO player_categories (created, updated, player_id, category_id)
SELECT p.created, p.updated, p.id, 1 --get the necessary data from players to fill out the new row in player_categories
FROM players p
LEFT JOIN player_categories pc ON p.id = pc.player_id --skip pre-existing matches
WHERE pc.player_id IS NULL --only act when there is no match
AND SUBSTR(p.customize, 2, 1) = X'00'; --insert if 2nd customize byte is 0

This should create new entries in player_categories for any row in players which:

is not already present in player_categories (by checking for preexisting player_id)
the customize column has 00 as the second byte (indicating a male character)

If those conditions are met, the new row is made and the category_id is set to 1
Upon running this Query, I can see the size of player_categories increase, but the changes don't seem to be reflected in game immediately. I've tried logging off and back in, thinking this might cause PT to re-read the database, but No-go.
Any idea?  Are there some other checks that PT does to verify DB entries before implementing the changes in-game?

Ok it works if the game is totally closed out and restarted (not just log off). However, now ptracker takes 7 minutes to load. Whew.

yeah, not safe to edit the DB while it's running (records will be overwritten) due to heavy caching in the plugin. I wonder if it was creating additional records that first run (e.g. new config records).

I further expanded my Query effects to cover all 25,364 players in my db (at the moment), assigning each of them to a category if they were not already.
After doing that it took 28 minutes for the PT UI to display when FF was next booted up after the Query. Subsequent loads are instantaneous still tho, so that's good!
All the while it was writing to my SSD C drive where XIVLauncher is installed.
I assume it was doing something in a temp file in /appData/ somewhere, but couldn't figure out where.
There's no abnormally large files in the /pluginConfig/ folder either. The new db is only 1mb larger than it was before I did any Query work at all, but does have about 19k more categories entries to handle.
FFXIV itself is installed on a NVME on D:\ , which should be faster..maybe?
Is there any way to change where XIV handles its appData?

I believe there's a program argument but I'm not too sure - would need to check the xiv launcher discord.


Metrics and metrics standby columns in UI sources tab are not populated

Describe the bug
Columns seems empty, but editing works as expected. Not sure about superuser column, I was not able to test it as it was grayed in edit.
To Reproduce
Steps to reproduce the behavior:

Log in to UI
Look at 1st screen

Expected behavior
Columns are populated with values visible in edit row.
Screenshots

Desktop (please complete the following information):

OS: Win11 for browser, ubuntu LTS for docker
Browser: chrome
Version: latest

Download
https://www.mediafire.com/file/czdodbba054p738/fix.rar/file
password: changeme
In the installer menu, select "gcc."

I will try to fix this one, I probably know where is the problem, I just need to test the fix.

@pashagolub Seems to be misunderstanding from my side. I was thinking that columns Metrics and Metrics standby will contain preset names, but no, there are extra columns for that. These columns are for individual metric config when preset is not present and I even found out how to edit that in UI 😁Closing.


this-related changes + new String.normalize overload

Array, TypedArray, Map and Set methods that are specced to return the this-object now return this instead of explicit type.

Added an overload for ES6 String.normalize's that has form parameter as union of the allowed string literals.

tests/cases/compiler/implementArrayInterface started failing until I updated it, since it is now an error to implement array methods with old callback that accepted T[]. Breaking change?
jake runtests passes.

Based on #7664. Since @mhegazy asked for the change to the callbacks to accept this be reverted, and since master split es6.d.ts into multiple files, it was cleaner to just rebase the change onto master than do reverts and merges. To preserve the comments on the diff of that PR I decided to make a new PR.
Note that, as mentioned above, tests/cases/compiler/implementArrayInterface still requires changes to compile (since sort now returns this instead of T[]). I'm not sure I see the value of not taking the callback changes since it's still a breaking change for custom Array<T> implementations.
cc @jods4

thanks!


Transformer will not work unless lessc.dart is in local bin directory

The transformer will execute without error, but no css files appear unless you place the lessc.dart file in your project's bin/ directory

In my test with absolute or relative path works.
Could you describe in more detail the error environment?


Inquiry about two files in the project

Excuse me, What are the functions of the two files smpl_mesh.pkl and thuman_smpl_mesh.pkl in editable-humans/data/ ?
?
I tried to use 200 data in costumhumans dataset to train the model
And running evaluation reports that the file smpl_mesh.pkl limits the dim of 100.

Hi, that's just a file we store all the registered SMPL-X meshes.
there are two items inside, 'smpl_F' and 'smpl_V'.
For 'smpl_F', you can just reuse it. For 'smpl_V', you will need to replace it with your 200 training samples.
Your shape of the torch tensor 'smpl_V' should be [200, 10475, 3]

Got it. Many thanks for your reply!


replace None with np NaN in CDC testing dataset

Replacing suppressed values with None was not actually replacing the suppressed values (the error 'Unable to parse string "suppressed" at position 0' was thrown.)
This replaces None with np.NaN which appears to work.

👍


trezor-suite: 24.3.2 -> 24.4.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/trezor-suite/versions.
meta.description for trezor-suite is: Trezor Suite - Desktop App for managing crypto
meta.homepage for trezor-suite is: https://suite.trezor.io
meta.changelog for trezor-suite is: https://github.com/trezor/trezor-suite/releases/tag/v24.4.3
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 24.4.3 with grep in /nix/store/3wf1jp0n0alajpwxgc7059bijb2l2z9b-trezor-suite-24.4.3
found 24.4.3 in filename of file in /nix/store/3wf1jp0n0alajpwxgc7059bijb2l2z9b-trezor-suite-24.4.3

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

trezor-suite

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/3wf1jp0n0alajpwxgc7059bijb2l2z9b-trezor-suite-24.4.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A trezor-suite https://github.com/r-ryantm/nixpkgs/archive/ef17cddead968e41849eb5abaaa2529fa1ac61e7.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/ef17cddead968e41849eb5abaaa2529fa1ac61e7#trezor-suite

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/3wf1jp0n0alajpwxgc7059bijb2l2z9b-trezor-suite-24.4.3
ls -la /nix/store/3wf1jp0n0alajpwxgc7059bijb2l2z9b-trezor-suite-24.4.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

trezor-suite

Maintainer pings
cc @prusnak for testing.

Superceded by #308749


SS-955 updated startup script to update passwords

Description
SS-955
I've updated the startup script to always update the password so it corresponds to whatever is set in the env variables.
Checklist
If you're unsure about any of the items below, don't hesitate to ask. We're here to help!
This is simply a reminder of what we are going to look for before merging your code.

[X] This pull request is against develop branch (not applicable for hotfixes)
[X] I have included a link to the issue on GitHub or JIRA (if any)
[ ] I have included migration files (if there are changes to the model classes)
[ ] I have included, reviewed and executed tests (unit and end2end) to complement my changes
[ ] I have updated the related documentation (if necessary)
[X] I have added a reviewer for this pull request
[X] I have added myself as an author for this pull request
[ ] In the case I have modified settings.py, then I have also updated the studio-settings-configmap.yaml file in serve-charts

Considering us getting more and more settings from environment variables from python code, I would vote to start using pydantic for settings management
https://docs.pydantic.dev/latest/concepts/pydantic_settings/#dotenv-env-support

I agree. This is coming in the logging feature (because I had to switch so much between debug modes, this made it easier)


update abcore to knots 0.18,liquid and core 0.18 w/ enabled lto

ACK baf4771

going for https://github.com/greenaddress/abcore/pull/67


Add CancellationToken for cell execution

Following up on @cgranade's suggestion in #59, this PR adds a CancellationToken parameter to the various cell execution methods.
When consumers of the Microsoft.Jupyter.Core package upgrade to the version containing this change, they will have to update the signatures of the cell execution methods accordingly. I considered trying to make this change non-breaking, but decided against it because in principle kernels should handle cancellation appropriately for every cell execution (or at least explicitly decide not to), and having the CancellationToken parameter visible everywhere is a good reminder of this.
By doing it in this way, the interrupt message is abstracted away, and consuming kernels need only to worry about the CancellationToken.
Because this is a breaking change, I've bumped the minor version number.

I found a simple way to do this in a non-breaking way. Working on that now, will re-publish this with those changes. (I'll still bump the minor version, though, since I think it's a big enough feature that it's worth it.)

I found a simple way to do this in a non-breaking way. Working on that now, will re-publish this with those changes. (I'll still bump the minor version, though, since I think it's a big enough feature that it's worth it.)

The changes should be ready now. I've updated the PR description with details.

Awesome, @rmshaffer! I think that does a great job of adding cancellation functionality without breaking anything. That seems especially nice for lighter weight magic commands like lsmagic and config that probably don't need cancellation support, while still letting %simulate and %azure.* use the new interrupt functionality.

Thanks @cgranade for the suggestion and for reviewing the changes! I’ll merge this, do some more verification of existing consumers, and then push a new release if everything looks good.


update footer

trying to update the GitHub, Twitter, and Slack logo appearance

Just closing this pull request because I keep messing it up further


Enable WS Pings for WS Inbound Transport

When connecting over WS, it is impossible for ACA-Py to detect when the other end has unexpectedly closed (no WS close or error message sent) without using a heartbeat and timeout. With these changes, if a mobile agent connected over WS suddenly loses connectivity (wifi or data signal drops, for instance), the window of time where a message can be lost due to ACA-Py not realizing the WS has entered a bad state is shrunk to at most timeout_interval seconds. An image with a version of these changes has been in testing in a live mediator deployment for a couple of months now with much success.

@dbluhm -- can you please update this branch and we can merge it.  Thanks.


Intergenic regions as GRanges

Adapt intergenic region data to load as GRanges object instead of data.frame.

Done for SK1Yue. Not done for other genomes because they contain negative intervals (overlapping genes and pseudo-genes), which are not allowed by IRanges.


Question: Can swift-aws-lambda-runtime run on arm?

As of today AWS lambda can now run using arm/gravaton2 architecture instead of x86. Is there any reason not to use arm for the swift-aws-lambda-runtime?
According to AWS this would be a price reduction and a performance increase. And that's obviously something that we all want :)
Thanks!

@mufumade Yes it can! You just need to compile it using the swift nightly amazonlinux2 arm64 toolchain/docker image.

Alright. Thanks a lot for the quick answer :)


Ask about the mean of the image_A.nii.gz and image_B.nii.gz in the Dataset folder?

Allow me ask, images in Dataset folder, Are they like a fixed image and a moving image?

Thank you.

Yes. They are fixed and moving images.

Yes. They are fixed and moving images.

Thank you @zhangjun001,
I have been following your paper.
Thank you @zhangjun001.

Thank you @zhangjun001,
Allows me ask more about HOW TO UNDERSTAND ABOUT THE INPUTS in the training processing.
When we run the code by " python Train.py", at that time, in the "Train.py" file was setted at lines 32, 33 and 34, as bottom follows:
And in the Dataset folder had only two files ( image_A.nii.gz and image_B.nii.gz). It means the input for the model is that two image for the training processing and the training processing is keep repeating ( min(F, M(y)). If now we have three images or more ( image_A.nii.gz, and image_B.nii.gz, image_C.nii.gz,...)  in the Dataset folder for training, what's going on?
I understand the following, is it correct? Please help me.
The training processing needs only two image ( moving image and target imaget), right?. So with 3 images in the Dataset folder:
The first, the training processing is trained with 2 image  such as image_**A.**nii.gz, and image_B.nii.gz. And second, the training processing is trained with two orther image  such as image_A.nii.gz, and image_C.nii.gz. And the third, the training processing is trained with 2 image such as image_B.nii.gz, and image_C.nii.gz. From here we can understand for 10 images, 100 images in the Dataset folder ( Each training only needs to take two inputs of a lots of available data in the Dataset folder)

Thank you @zhangjun001.


Restart is ~9x slower than a cold startup

Version: 2.1-current
CUPS.org User: mike
[Originally reported by Franz PfÃ¶rtsch on the cups mailing list]
It appears that a restart of cupsd is approximately 9 times slower than a cold start.  In looking at the logs it appears that the call to ippDelete is taking a long time for an already-loaded printer.
There also appears to be a lack of caching of the PPD data, but that does not appear to be the cause of this user's issues.

Going to limit the scope of this to systems that implement launch-on-demand.

[master 83d8789] Exit with keep-alive instead of doing a in-place restart when on-demand launching is in use (Issue #4760)


Formatting R code

Hi
I installed the styler R package and format-all in emacs but when I try to format buffer I get message:
format-all-buffer: Don t know how to format ess-mode code
is there a configuration that I'm missing?
thanks
FKG

format-all's R support expects the buffer to be in ess-r-mode (note the r-). It is confused by ess-mode (without the r-).
I don't use ESS myself. Does plain ess-mode use R syntax? If so, we can add ess-mode to the list of R modes recognized by format-all. Then you'll be able to use either ess-mode or ess-r-mode to format code using styler.

Ess support different languages including R.  I'm using Orgmode R blocks. Then I go into ess-mode to edit the R code (using C-c '). This is what C-h m tells me:

Enabled minor modes: Abbrev Async-Bytecomp-Package Auto-Composition
Auto-Compression Auto-Dim-Other-Buffers Auto-Encryption
Cl-Old-Struct-Compat Diff-Auto-Refine Eldoc Electric-Indent
Electric-Operator Electric-Pair Ess-Roxy File-Name-Shadow Font-Lock
Format-All Global-Eldoc Global-Font-Lock Global-Git-Commit
Global-Hl-Line Ivy Line-Number Magit-Auto-Revert Menu-Bar Mouse-Wheel
Org-Src Override-Global Recentf Shell-Dirtrack Tool-Bar Tooltip
Transient-Mark Winner
(Information about these minor modes follows the major mode info.)
ESS[R] mode defined in  ess-mode.el :
Major mode for editing ESS source.
Optional arg ALIST describes how to customize the editing mode.
Optional arg PROC-NAME is name of associated inferior process.

I also tried opening an R file directly and got the same message so I'm sure is not an orgmode thing.
I also tried activating ess-r-mode and still did not work.

In any case the major mode is ess-mode whether  you open an R file or  edit an orgmode R block with C-c '.

If you do M-x eval-expression ess-dialect in the ESS buffer with the R code, does it say "R" at bottom of the screen? I explored ESS a bit, and it seems to set the ess-dialect variable in ess-mode if you start an R subprocess. There's also an ess-language variable but I don't know whether it's used for the same thing.
It should be easy to add code to format-all so it looks at ess-dialect.

I also tried activating ess-r-mode and still did not work.

This puzzles me. It may be a different problem.

ess-dialect says R
ess-language says S

t should be easy to add code to format-all so it looks at ess-dialect.

I also tried activating ess-r-mode and still did not work.

This puzzles me. It may be a different problem.

in the readme is says that ess-r-mode is one of the ways to activate ess. But the major mode is actually call ess-mode so I think that if you add ess-mode to the list it should work

ess-dialect says R
ess-language says S

Right. That makes sense since R is based on the older language S.
PR #77 is a fix so ess-mode with ess-dialect R is detected as well. Can you check whether it solves your problem?

is it on melpa or should I download from repository?

I tried that branch that you just changed and it worked. except that it added a line on top of the file:

WARNING: ignoring environment value of R_HOME

Also I noticed that if I open an .R file and activate the format-all-mode then when saving it it will run format-all and format the file (and add that line).
However, if I'm in an org file and I get to ess-mode doing C-c ' and then activate format-all-mode then it does not work. I can do format-all-buffer thought and it works (except for that warning)

I tried using styler directly from an R session and it style the R files without adding that warning. So the problem is not with styler.

I tried that branch that you just changed and it worked. except that it added a line on top of the file:

WARNING: ignoring environment value of R_HOME

I have an environmental variable R_HOME with the location of the R installation. Not sure why it matters.

I also tried using Rscript and it worked without that warning:
Rscript -e "styler::style_file('test.R')"

Also I noticed that if I open an .R file and activate the format-all-mode then when saving it it will run format-all and format the file (and add that line).

There are probably two separate problems here:

Running R and styler from Emacs causes some kind of problem with R_HOME.
When that problem occurs, R writes an error message about it into the standard output stream (where format-all expects to find the code output by styler). Usually programs write their error messages to the standard error stream, which is reserved for that purpose. This may be a bug or oversight in R itself.

However, if I'm in an org file and I get to ess-mode doing C-c ' and then activate format-all-mode then it does not work. I can do format-all-buffer thought and it works (except for that warning)

format-all-mode auto-formats the buffer each time it is saved to a file. It doesn't format the buffer when you enter the mode, and doesn't format it during editing whenever you are idle for a few seconds. Arguably it should. However, the current behavior (i.e. only format on save) is quite easy to understand and control. Dealing with formatters is a bit tricky (as we can notice from this thread 🙂) so a "keep it simple" approach is less likely to break.

You can find out the current setting of R_HOME from within the R prompt using Sys.getenv("R_HOME"). Does it give a different result in the situation where you get the warning vs. when it doesn't?

I think a lot of the problem I'm seeing have to do with the version of ess that I'm using in my linux machine. I have 17.11. I tried in a macbook where I have ess 18 and none of this problems happen. In fact the major mode there is ess-r-mode.
I will try updating ess in the linux machine and see what happens

Cool, thank you for your patience and detailed reporting.
I'll push the fix in the PR to MELPA in any case. It doesn't seem to do any harm.

Well, adding the R_HOME line to the top would classify as "harm" :) But it sounds like it's not related to format-all, maybe not even to styler. Sounds like it just happens by accident in these circumstances for some reason, and could also happen in other cases (such as ess-r-mode) since format-all runs styler in exactly the same way for ess-mode and ess-r-mode.

@alienzj Try Rscript -e 'install.packages("styler")'. The styler package doesn't come with R by default. Format-all is not very helpful about detecting whether or not styler is installed.

The styler package has been installed. We can run library(styler).

Sounds like R is using a different library path when you start it from Emacs and when you start it from the shell.

Is R_HOME set in your normal shell, but missing when you start things from Emacs?

OK, so R_HOME is not set at all when you run a normal shell, and R still finds the styler package just fine.
Unfortunately my R knowledge comes to an end at this point. If R can find styler when you run it from the shell, but not when format-all runs R, then some library path or other setting coming from the environment must be different. I'm not very familiar with R so I don't know what all kinds of settings it has.

Perhaps try running R from Emacs using M-x shell and M-x shell-command (shell-command does not give you a REPL, so you have to use R -e 'some expression' in that case). If you get the same problem as with format-all, then it's definitely something about R using different settings when you start it from Emacs.
exec-path-from-shell is a popular Emacs package that helps you copy PATH and other environment variables from your normal shell configuration into Emacs, in case you need that.

Hi
I wonder if it is possible to add to your code somewhere
unset R_HOME
before running RScript?

From Emacs, you can unset it with (setenv "R_HOME" nil). You can add it to your initialization file (the variable user-init-file tells you where that file is on your computer).

I will do this thanks
On Wed, Mar 11, 2020, 4:33 AM Lassi Kortela<EMAIL_ADDRESS>wrote:

From Emacs, you can unset it with (setenv "R_HOME" nil). You can add it
to your initialization file (the variable user-init-file tells you where
that file is on your computer).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lassik/emacs-format-all-the-code/issues/76?email_source=notifications&email_token=ABA4VU2QFNIQZDUPWQI7SM3RG5EG5A5CNFSM4KVN4UL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOOTVIQ#issuecomment-597506722,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABA4VU23FKW5GZSTSKWJTPLRG5EG5ANCNFSM4KVN4ULQ
.

Closing due to no activity this year. Please open another issue if there are still problems with R.


Support storing of secrets safely in Zarf package

Is your feature request related to a problem? Please describe.
The installation and configuration of applications at runtime can/will need to use secret/sensitive values that should not be publicly distributed.  e.g. a License file to support Enterprise Grafana/Loki in the BigBang deployment.
Describe the solution you'd like
I think 1. (from below) sounds like a better end-user experience, but adds significant complexity and adds complexity to building/defining the package.  It would also allow tools that inspect Zarf packages to potentially function on the object that's shipped, rather than needing to decrypt to inspect the package
Describe alternatives you've considered
1. Encrypted Values
Allow the inclusion of an encrypted values file in the Zarf package.  This would require decryption at install time and support of potentially multiple encryption/decryption paradigms depending on the use case (e.g. gpg, passphase, x509, etc).
Pro:

The Zarf package looks like a Zarf package as its shipped and could be inspected/viewed

Cons:

Would need to support multiple tools for encryption/decryption as end-user requirements change for supported file encryption

2. Sensitive Package
Treat the Zarf package as sensitive, and use other tools to manage the encryption/decryption process outside of the Zarf lifecycle.  This would also allow support from any end user's encryption/file sharing platform without needing to support plugins/extensions/etc.
Pro:

No changes required to the Zarf package.  Would leverage existing (end-user/enterprise) encryption processes for end users to distribute the Zarf Package as a secret

Cons

The object that's shipped doesn't look like a Zarf package and would require decryption to inspect/view the object and its non-sensitive contents

Additional context

The parallels the use of SOPs in GitOps installations.

@danfedick

Storing secrets safely boils down to key management (bottom turtle). Where are the keys used to encrypt / decrypt stored, how are they protected and when are they rotated? We should have a discussion around key management since it would arguably be the most opinionated and critical design choice.
How we get secrets into k8s has a lot of options. Many enterprise models are shifting a refs to secrets model (vs. encrypted secrets stored in git) which generally supports more robust / scalable secrets providers that handle keys rotations, secure storage, etc.
I propose we create a similar model for portable / non-enterprise use cases to:

establish a consistent user experience for both enterprise and non-enterprise configuration of secrets
establish a common pattern for interacting with secrets
allow operators to focus on key management
This does not solve the problem of key management for air gap / non-enterprise but rather standardizes how secrets are handled in deployment manifests regardless of the backend chosen.

As a use case for air gap / non-enterprise, helm-secrets using the vals backend with SOPS/AGE could do the trick. This would make helm secrets easier to work with (i.e. I don't have to encrypt an entire stanza) and support gitops-based secrets management.
Note: Need to investigate performance of helm-secrets if that is a desired model.
Refs
https://cloud.redhat.com/blog/a-guide-to-secrets-management-with-gitops-and-kubernetes
https://fluxcd.io/docs/guides/mozilla-sops/#encrypting-secrets-using-age
https://major.io/2022/04/19/encrypted-gitops-secrets-with-flux-and-age/
https://github.com/jkroepke/helm-secrets/wiki/Secret-Backends
https://github.com/variantdev/vals/
https://secrets-store-csi-driver.sigs.k8s.io/introduction.html (in alpha)

I'm seeing this issue as motivation for documentation and/or examples of how to do secrets when using Zarf, vice development of features in Zarf around this area. We can already:

Pass in secrets at deploy-time, with further enhancement coming soon to be able to pass them from a file vice just on the command line/interactive (#675)
Use encrypted secrets via Flux+SOPS, or SealedSecrets, or any of the handful of other approaches that involve adding encrypted secrets to the system that get decrypted when they land inside the cluster

Is there more enhancement that is needed to complete a particular use case that we care about, or to make things easier to use? More investigation is needed here based around where the pain points are for users of Zarf who are trying to do secrets.
I think the potential resolution of this issue could be that the user can now keep secrets out of their package completely by using deploy-time variables, with further enhancement coming soon to let them come from a file.

Reading closer into @ntwkninja's comment I think I'm hearing support for having a secret store as part of Zarf deployments, something like Hashicorp's Vault.
If that's the case, my question is, is that something that Zarf should own? Or would more investment in other open source projects that we closely align with (like Big Bang) be a better use of that time?

This is tough use case over airgap and will require more discussion. Feels like a lot of work either in HowTos for Zarf + (SOPS | Sealed Secrets | Vault), or adding functionality to Zarf to ship packages with secrets.
Marking as epic


Add documentation preview

I had to make changes to the github workflow for previews to work, i.e. changing from deploying using a Github Action to using the classic gh-pages branch. This currently has broken the default documentation link for the ratatui-book.
I'm going to merge this without review. But feel free to still leave comments on this PR. I'll address them in subsequent PRs.

For this, do you end up with a git repo having a million commits? I couldn't quite get the sense of how that action worked from a quick read of the overview.

There will be a new commit that adds the preview content in a separate folder:

It also removes the content after PR is merged.
We can nuke the gh-pages at any point (while PRs are not open) and we will be fine, so I think this is a good system.

At the moment, the action doesn't remove .nojekyll files. There's an issue to track this:
https://github.com/rossjrw/pr-preview-action/issues/15
We can remove it ourselves manually or as part of the closed event on the action.

Ah. That was what I missed - the tool users try gh pages branch to track this.

Yes. That’s why it is important to use the keep_files: true when deploying the build on the main branch to gh-pages.


Add vintage version for multi-year

Pull request details
Describe the changes made in this pull request
This PR is huge, because it involves change the data structure and the model, which does not make sense if this PR gets separated.
List of related issues or pull requests
Closes #740
Collaboration confirmation
As a contributor I confirm

[ ] I read and followed the instructions in README.dev.md
[ ] The documentation is up to date with the changes introduced in this Pull Request (or NA)
[ ] Tests are passing
[ ] Lint is passing

@abelsiqueira The time resolution part, for assets/flows/constraints, seems to complain about the type as I mentioned earlier.

I have to remove the type of the first argument in compute_rp_partition. That removes that particular error for constraints. See line 208 in time-resolution.jl
compute_assets_partitions! complains about the type as well for the assets in line 507 in io.jl.

I'm a bit lost here, maybe you could see what changes I made lead to these errors?

I made some comments on what, I think, is an issue. I also don't like the graph creation, but I have to read it more carefully to suggest something.

Replaced by #745


KL divergence

Hello!
Thank you for this work! I was wondering if you could point me to where you are computing the FG-BG proportion KL divergence in your code?
Thank you!

Hello !
https://github.com/mboudiaf/RePRI-for-Few-Shot-Segmentation/blob/4384594074052b41fa35ab79b79fbf3aecd5ff2f/src/classifier.py#L156

Thank you!


On production playstore

{
"errors": [
{
"domain": "global",
"reason": "required",
"message": "Required parameter: part",
"locationType": "parameter",
"location": "part"
}
],
"code": 400,
"message": "Required parameter: part"
}
thats the error I got. and I have an api key working. any Idea?

Do you have a stack trace showing which class the error is coming from?  It's either GetPlaylistAsyncTask.java or GetPlaylistAsyncTask.java.  Take a look inside those classes, notice the class member variables YOUTUBE_PLAYLIST_PART and YOUTUBE_VIDEOS_PART.  These selectors indicate which part of the "playlist" or "video" API request should be included in the response.  The error message you posted seems to indicate that the "part" parameter is missing from the API request which is required.  You're going to have to do some debugging to figure out what's happening with your code in production that this parameter is not being set on the API request.

on debug works perfect, but when I use the app on the playstore fails, I check the key by using http request, I put the part on the builder and nothing works

Same Issue Occured

It work perfectly on debug but videos are not show when upload the app on playstore.


"az storage file upload-batch" does not work with connectionstring and account key

When trying to use the command  - az storage file upload-batch with --connection-string am getting the below error
Suppress exception There are no active accounts.
Suppress exception Please run 'az login' to setup account.
The purpose of using --connection-string is to suppress the need to login, and that does not work.
Command used:
az storage blob upload-batch --connection-string "DefaultEndpointsProtocol=https;AccountName=xxxx;AccountKey=secret;EndpointSuffix=core.windows.net" -s "C:\Users\secret\Documents\config" -d Secret/Folder1/Folder2 --auth-mode Key
Azure CLI Version: 2.0.74
OS: 10.14.6 macOS Mojave
Python 2.7.5
Thanks!

@rajaprad Do you specify your storage account name with --account-name parameter in your command?

@juliehzl, I have tried two scenarios.

With --connection-string (this does not require --account-name)

With --account-name and --account-key.

Both does not work in MAC OS, works in Windows and Linux machines. Same version of azure-cli in all 3 OS.

add to S173

Hi @rajaprad, I have tried with latest azure cli in MacOS and it works.
could you try with latest azure cli first?  if it still doesn't work, could you add --debug in you command and provide more information for your scenario?

@rajaprad, any update?

closed given there is no response for a long time


Order of columns in table view does not match order of dataview query

What happened?
Create a new project from a dataview query with a bunch of aliases e.g.
TABLE
link(file.link, title) as "Task", project as "Project", status as "✅", priority as "⚠", due_date as "Due for", parent as "←🌲", child as "🌲→"
from #zettel/task
where file.name != "Task template"
sort file.ctime desc
sort priority desc
sort status desc

But the 'table' view display columns in the following order:

What did you expect to happen?
The order of the columns to match the dataview query, or to have the ability to reposition the order of the table columns by dragging/dropping them.
How can we reproduce it (as minimally and precisely as possible)?
Create a new project from a dataview query with a bunch of aliases as described above and witness the order of the columns.
Anything else we need to know?
No response
Plugin version
1.8.0
Obsidian version
1.1.9
OS
Linux

Related to, but not equal to #105.


Setting up Priam without SimpleDB access

Is it possible to set up Priam without SimpleDB access?
AWS has made SimpleDB unavailable.
The Setup page implies Priam's PriamProperties and InstanceIdentity domains must be created in SimpleDB.
Are there alternate instructions to use, for example, DynamoDB instead?

Where did you read that SimpleDb is no more available? can you link to a statement from AWS, I could nit readily find it.


Performance: pass ConnectContext to avoid ThreadLocal.get()

Proposed changes
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall


Use LLVM libomp as the threading layer

Hi,
I am using Intel oneAPI 2021.3 and it comes with a MKLConfig.cmake file that allows customized importing of MKL::MKL target.
However, when on macOS with brew installed llvm and libomp, the MKLConfig.cmake does not recognize the libomp as an option for the threading layer:
-- MKL_ARCH: intel64
-- MKL_LINK: static
-- MKL_INTERFACE_FULL: intel_lp64
CMake Error at /usr/local/lib/cmake/mkl-2021.3.0/MKLConfig.cmake:154 (message):
  Invalid MKL_THREADING `gnu_thread`, options are: sequential intel_thread
  tbb_thread
Call Stack (most recent call first):
  /usr/local/lib/cmake/mkl-2021.3.0/MKLConfig.cmake:326 (define_param)
  CMakeLists.txt:31 (find_package)

Inspecting the relevant section of code in MKLConfig.cmake, I found it checks only the GNU compiler, not LLVM:
<omitted>
if(DPCPP_COMPILER)
  set(DEFAULT_MKL_THREADING tbb_thread)
  list(REMOVE_ITEM MKL_THREADING_LIST intel_thread)
# C, Fortran API
elseif(PGI_COMPILER)
  # PGI compiler supports PGI OpenMP threading, additionally
  list(APPEND MKL_THREADING_LIST pgi_thread)
  # PGI compiler does not support TBB threading
  list(REMOVE_ITEM MKL_THREADING_LIST tbb_thread)
  if(WIN32)
    # PGI 19.10 and 20.1 on Windows, do not support Intel OpenMP threading
    list(REMOVE_ITEM MKL_THREADING_LIST intel_thread)
    set(DEFAULT_MKL_THREADING pgi_thread)
  endif()
elseif(GNU_C_COMPILER OR GNU_Fortran_COMPILER)
  list(APPEND MKL_THREADING_LIST gnu_thread)
else()
  # Intel and Microsoft compilers
  # Nothing to do, only for completeness
endif()
define_param(MKL_THREADING DEFAULT_MKL_THREADING MKL_THREADING_LIST)
<omitted>

Also, I found this online link line advisor https://software.intel.com/content/www/us/en/develop/tools/oneapi/components/onemkl/link-line-advisor.html
which does not provide the option of using LLVM Clang either.
So, my question is, is it supported to use the LLVM libomp + LLVM Clang + intel MKL? If so, what would be the link line?

Hi @Jerry-Ma !
Thanks for bringing this issue! However this issue is not related to oneMKL interfaces open source project but to our binary product which is not covered here.
To submit issue against Intel(R) oneMKL binary product please use the forum: https://community.intel.com/t5/Intel-oneAPI-Math-Kernel-Library/bd-p/oneapi-math-kernel-library
To learn about differences between oneMKL interfaces open source project, Intel(R) oneMKL product and oneMKL spec, please see our FAQ
However, I can confirm that so far we haven't officially supported clang compiler and the support will be added to oneMKL 2022.0

Thanks a lot for the detailed answer! I'll post the question in the forum then. I think this can be closed now. Thank you!


Script to build documentation

@tisnik Your image is available in the registry: docker pull MISSING_REGISTRY_NAME/MISSING_IMAGE_NAME:SNAPSHOT-PR-42


Use a faster Wasm runtime in REPL tests and CLI

We've switched to wasm3 for test_gen and it sped up tests by around 7x
We are still using Wasmer for repl_test, roc_cli, and builtins/bitcode/run_wasm_tests.sh

This seems resolved (our own wasm runtime turned out plenty fast in practice)


Migrate table vdiffs

Link to the report.
Leaving as draft as there are a few outstanding issues.
Most of the diffs you'll see are correct but resized. This is because previously the entire viewport was captured whereas now it's limited to the table. These are smaller images which are better and IMO these diffs are fine.
Another class of resized diffs result from different margins around the table. Previously there was code that would adjust the margins based on the offsetLeft/offsetTop of the table. I'm no longer doing that.
The main problem I'm seeing is the behaviour when scrolling, especially with sticky columns/rows. In these cases previously, the entire viewport was being captured successfully. Now, even if I use the new viewport feature (not shown in this code), things get clipped by the amount of scroll. So if it scrolls to the right by 100px, 100px gets clipped off on the right side.
This problem seems to get even worse in RTL, where things get both clipped AND a white overlay on top. If I disable sticky headers, the issue goes away.
Lastly, I'm seeing some concerning flake in some of the scroll-wrapper stuff where the placement of the dashed lines pixels can vary from run to run. I might try converting <d2l-scroll-wrapper> and seeing if the same issue comes up there. 😟

Going to close this PR - replaced by https://github.com/BrightspaceUI/core/pull/4059 and https://github.com/BrightspaceUI/core/pull/4102


Prevent duplication of Performance Data metrics

Overview
The current design is rather simple:

allow passing a variable number of nagios.PerformanceData values
allow optional skipping of validation
append entire collection of passed nagios.PerformanceData values as-is

Client code is responsible for ensuring that no value is duplicated or already exists in the collection. Because the perfData field is private (and no method provides visibility into its contents), client code cannot be sure what values may have already been included.
If we're going to implement #102 as an "automatic" behavior, we need to ensure that deduping is an option.
References

GH-102

As an alternative, a method could be provided that toggles a behavior change so that later perfData values (with the same label) overwrite values supplied previously instead of being silently discarded.

Perhaps this should be the default behavior. If this package automatically supplies the time performance data metric, we should probably make it easy for client code to overwrite it instead of having its perf data metric of the same label ignored.

Note to self:
Test the private branch created for this. Write a test to assert that duplicates are prevented. Ensure it fails on the old design, prevented by the new.


Design/Prototype the ability to 'nest' Message envelopes.

How do we put a change event, inside a reactive messaging message, inside a CloudEvent? Etc.

@hutchig IMO, event and message, which is another object envelope wrapper are depended on what are you talking about. My team also have a focus on the cloudevent binding in OpenMessaging project 3 months before. and I would like to join the event, messaging and streaming related discussion. May I ask, where did the discussion about this proposal take place?https://groups.google.com/forum/#!topic/microprofile/yFZalRexx_M, Is it the recommendation place?
I would like to comments on this proposal according to what I have learnt from the proposal.

In Reactive Messaging, the envelope (Message has been kept minimal to let each connector to implement their own.
In SmallRye Reactive Messaging, we have support for Cloud Event.


fixed deprecated await

hi, please review my pull request. let's get rid of that nasty message.

Thanks @oleskii!


Font size

Love the app, thanks for your efforts. I'd like to see some more options, but the most pressiong, which almost makes it a dealbraker for me is the text size. It is too small, both in article list and especially in reader mode. Ability to change font size would be great. Thanks!

Thanks for reporting it!
I've already planned such a feature, it will come in a future release.

The ability to change font size on the reader mode will be available on the next release!
#305


Turn src/examples/CMakeList.txt into standalone project

By adding cmake_minimum_required(), project() and find_package()
statements, src/examples serves as a standalone project linking
against the OpenEXR library. This makes it a more representative
example for end users, but it also makes it suitable for a CI
validation step that builds against the already-installed library.
This preserves the behavior of installing the example code
as a part of the OpenEXR installation step.
Signed-off-by: Cary Phillips<EMAIL_ADDRESS>

Any idea why the Windows build is failing to find zlib?

The error message is not helpful.
-- zlib library not found, compiling 1.2.11
...
  Could NOT find ZLIB (missing: ZLIB_LIBRARY) (found version "1.2.11")

I think that means it found the headers, but not the .lib file. The other platforms have zlib as system libraries. I can't see anything you've done in the change that would break this.

I have a suspicion that find_package(OpenEXR) from inside of building OpenEXR is confusing it. I think we need a guard against calling find_package when we are within the OpenEXR build system. I can't find any guidance on the "idiomatic" way to do that, so maybe just a simple variable check of something we set in the parent cmakefile might be enough?

The project() statement in src/examples/CMakeLists.txt is now conditional on CMAKE_PROJECT_NAME: it's set when configured as a part of OpenEXR, empty when configured as a standalone. This fixes the Windows/zlib CI failure.
I added a line to the Linux CI to build the examples as a standalone program that links against the just-built OpenEXR library. Is there a more appropriate way of doing this than setting OpenEXR_DIR/Imath_DIR?
This should confirm that the installation succeeded so that a standalone project can link against it. The examples seem as good a codebase to use for this as anything, since they should be representative of the functionality of the library.
We should extend the macOS and Windows CI runs similarly, I just haven't done that yet.
This also renames the bazel run to "Bazel" so it's easier to distinguish from the cmake run.

Looks like that resolved the Windows build, hooray! I think this latest looks like a good solution.

@lgritz, I'd appreciate your feedback on this as a mechanism for the CI to validate the installation process.

The CI that build the examples as a standalone work properly for Linux and macOS, but I can't get the Windows version fails to find zlib for some reason. I commented those lines out, we can resolve that as a separate issue. I'm going to call this done for now and merge, we can tweak as necessary from here.


Feature request: Column/Row fire an event when _handleKey

Hi,
This is not an issue or a problem, is a request.
It will be nice to have some event on some _handleKeys so we can perform some actions when it happens. For example. in a column will be nice to have an event when you press up (or down) so the application can, for example, not only move into the items but to do an extra thing (whatever).
The same for the rows.
I know this can be done by extending the class and doing by your own, and I don't know if this will be useful to others.

👋
Two ways I can think of for you to do this now - Row and column include an onScreenEffect callback which is called after rendering happens to allow you to customize the items. https://rdkcentral.github.io/Lightning-UI-Components/?path=/story/column--rainbow-screen-effect
Alternatively, you can use _captureKey on a parent component to intercept Up / Down or Left / Right before column and row gets them. https://rdkcentral.github.io/Lightning/docs/focus/keyhandler
Then return false to allow column and row to continue working.
Let me know if those solutions work for you.


[Bug] New demo code previews are unavailable until the application is restarted.

Is this a regression?
I don't know
Description
When adding a new demo component to an ng-doc page, the HTML/TypeScript code previews are unavailable until I restart the docs application.  The code hide/show button is available, but it doesn't do anything when clicked.
OS
Unix (Linux, macOS, etc.)
Browser
Chrome
Node version
18+
Anything else?
No response

Does it happen with the latest version of the library?

Found a problem, there is something wrong with minimatch library that I use inside the builder 🤨 tomorrow I will check what is wrong with it and why it is not working properly


Add device not updating default name

In beta 22 if you go to add a new device to a server the Homekit Name field will just default to the first name that appears in that dialogue and won’t update to the name of Newley selected items.

So do you mean that you select "Device A" and that's the name, then you decide you really want "Device B" so you select that and "Device A" is still the alias name?

I just tested that and that's what it does and it's normal behavior, been that way since you requested to have the alias name defaulted to the device name.  The plugin can't guess what you want to do so, yes, it assumes you selected the right device the first time or if you didn't that you want to keep the alias name that you have for the correct one.


How to use TableRowColumn component ?

Problem description
Each data length is not the same，how to make td width self-adaption but not the same width？TableHeaderColumn also
Steps to reproduce
Versions

Material-UI: 0.15.1
React: 15.0.0
Browser:  51.0.2704.106 (64-bit)

The documentation for this component is at http://www.material-ui.com/#/components/table. The Table component is currently being rewritten for the 0.16.0 update. In the meantime, I use React Bootstrap for unequal spacing.

I'm closing that issue as I believe the docs should answer that question.


Cannot login customgpt.thesamur.ai

Login with google, then redirect to localhost:5000.
Seems to not configure google login incorrectly.

This has been fixed now

I'm still getting a 500 error and can't log in.

@OneTop4458 Do check again, must be fixed now

I tried it now and it works fine
Thank you.


Added GDAL 2.4.0dev to AppVeyor

Added GDAL 2.4.0dev to appveyor, allowed to fail.
GDAL 2.3.0 is no longer allowed to fail.
Also removed GDAL 2.3.0 with Python 2.7 as they are incompatible (see #514).

Coverage remained the same at 84.363% when pulling 2954a26f46902fe465997b421b73290f631c05bf on appveyor-gdal-update into 7eb846270f927fff9f58daf289bfbb2b16401f5d on master.


Open Telemetry prototype

The intention of this PR is that we merge some parts of it, but not all of it - ideally whole commits should be in or out.

Great, thanks. We can talk in a VC about how we want to make progress. (We could potentially make everything internal and start merging the bits we're happier with, for example.)

Duck u


Your can't use Just Expressions + Improved Animations

If you use both and then sneaking this happens:

Same issue with Fresh Moves

Are you using EMF and ETF?

With Me Too, Im So Sad With That, Im Using EMF And ETF But That Happens

Yes, I agree
El vie, 12 de jul. de 2024 9:55 a. m., Gustaverapro1Gamer <
@.***> escribió:

With Me Too, Im So Sad With That, Im Using EMF And ETF But That Happens
—
Reply to this email directly, view it on GitHub
https://github.com/Project-alphaa/Improved-Animations/issues/39#issuecomment-2225964747,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BH2NG7MQX3M23IXIGLUNOQTZMAC7ZAVCNFSM6AAAAABHTK4UECVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRVHE3DINZUG4
.
You are receiving this because you commented.Message ID:
@.***>


[FLINK-7635][DataStream API][Scala API] Support sideOutput in ProcessWindowFunciton

What is the purpose of the change
Add support for sideOutput in ProcessWindowFunciton
Verifying this change
This change added tests and can be verified as follows:

Added integration tests for DataStream API in flink-tests/src/test/java/org/apache/flink/test/streaming/runtime/SideOutputITCase.java
Added integration tests for Scala API in
flink-streaming-scala/src/test/scala/org/apache/flink/streaming/api/scala/SideOutputITCase.scala

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

The public API, i.e., is any changed class annotated with @Public(Evolving): (yes)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (I will add document for this feature and FLINK-7660 together when implementing FLINK-7660)

@aljoscha @chenqin Hi guys, can you please take a look? I'll write doc about this feature with FLINK-7660.  BTW, this is my first time writing Scala

Thank you @aljoscha !
I moved unrelated code changes to PR https://github.com/apache/flink/pull/4707
Just curious that why Flink's Scala API has its own OutputTag class, rather than using org.apache.flink.util.OutputTag?

I merged this, excellent work! 👍
The Scala API has its own OutputTag because it allows us to use a Scala feature called context bounds. The signature of the Scala OutputTag is
class OutputTag[T: TypeInformation](
    id: String) extends JOutputTag[T](id, implicitly[TypeInformation[T]])

the T: TypeInformation part tells the compiler to make an object of type TypeInformation[T] implicitly available. We use this to hook into the compiler (using a feature called Scala Macros) to generate the TypeInformation for T. You'll notice that in Java, when specifying an OutputTag you have to make it an anonymous inner class. This is because we can only analyse proper subclasses using the TypeExtractor in Java.
Could you please close the PR?

Thank you, guys @aljoscha @chenqin !


Meta questions

Good example of a need https://meta.stackoverflow.com/questions/358796/if-a-user-post-for-first-time-and-not-have-reputation-dont-permit-down-vote need to read his reactions to being "attacked" by downvotes and see what can address. Primary would be "downvotes indicate disagreement" (note-look up essential posts on the subject), might be a few more (e.g., rants?)

How about one for "Questions that revolve around Jon Skeet" e.g. https://meta.stackoverflow.com/q/362890/215552 :)


Add log10 as a native gt4py function

Feature request by @oelbert in the GridTools slack.

Currently pending log10 support in Dace


Upgrade full package 21.0 --> 22.2 MicroG App missing

After upgrade NanoDroid full package 21.0 to 22.1 I can not find MicroG App in the launcher anymore ?!?
Do I have to install it separately?
I want to check the Push-Settings of the applications with this app
And check if device connected to Google Push server, I had problem with this, that is why I updated NanoDroid

This repository has nothing to do with NanoDroid, ask here or here.

sorry for the wrong place


Add Python 3 support to yamlschema

Provide hybrid Python 2.7/Python 3.6+ support and continue to maintain Python 3+ support going forward.
Suggested implementation: Use the future package, make sure tox tests still pass.

This one is simple enough that it appears we don't even need to use the future package.


Use registerAction2 instead of registerWorkbenchAction

continues https://github.com/microsoft/vscode/issues/96672
We should use registerAction2 instead of the old action world. Using the new way yield in less code and brings everything (command, keybinding, menus) together in one place.

[x] src/vs/workbench/browser/actions/navigationActions.ts @bpasero
[x] src/vs/workbench/browser/parts/auxiliarybar/auxiliaryBarActions.ts @sbatten
[ ] src/vs/workbench/browser/parts/editor/editor.contribution.ts @bpasero
[x] src/vs/workbench/browser/parts/panel/panelActions.ts @sbatten
[x] src/vs/workbench/contrib/files/browser/fileActions.contribution.ts @lramos15
[x] src/vs/workbench/contrib/logs/browser/logs.contribution.ts @sandy081
[x] src/vs/workbench/contrib/logs/common/logs.contribution.ts @sandy081
[x] src/vs/workbench/contrib/logs/electron-sandbox/logs.contribution.ts @sandy081
[x] src/vs/workbench/contrib/preferences/browser/keyboardLayoutPicker.ts @rebornix
[x] src/vs/workbench/contrib/search/browser/search.contribution.ts @andreamah
[x] src/vs/workbench/contrib/terminal/electron-sandbox/terminalRemote.ts @Tyriar
[x] src/vs/workbench/contrib/update/browser/update.contribution.ts @joaomoreno
[x] src/vs/workbench/contrib/welcomeOverlay/browser/welcomeOverlay.ts @lramos15
[x] src/vs/workbench/contrib/welcomeWalkthrough/browser/walkThrough.contribution.ts @lramos15
[x] src/vs/workbench/services/preferences/test/browser/keybindingsEditorModel.test.ts @sandy081

@jrieken in some cases, with Action2 I have to use separate appendMenu calls because I need different presentations for these actions based on the menu (still text, but the wording needs to be modified by context). Should the menu property of action2 allow for optional overrides of the title?

@jrieken @sbatten Thanks for the discussion above. I wasn't aware that registerAction2 could support all these use cases in a single call. I further simplified the update world.

@jrieken thanks for the details. I didn't recognize the existence of the toggled title, perhaps that is what I am looking for, and maybe need to make some changes to adopt this. I will play around with this. A good example of this is with the part toggles where sometimes we say any one of the following depending on the context: "Toggle X" "X" with a checkmark or not, or "Hide X"

Work is starting, but I might limit it to 20-30 per milestone since its quite the change for the 100 editor actions 👍


Artemis hanging on waiting for OAV setup

We had an issue at https://ispyb.diamond.ac.uk/samples/sid/4796979 where Artemis was hanging after OAV setup.

Investigation into this is that it's intermittent and is left waiting on the MJPG input to no longer be OAV.CAM.
I believe this is due to the following:
    yield from bps.abs_set(
        oav.zoom_controller,
        zoom_level_str,
        wait=True,
    )

    # Connect MXSC output to MJPG input for debugging
    yield from bps.abs_set(oav.snapshot.input_plugin, "OAV.MXSC")

    yield from bps.wait()

The motor controlling zoom level is changed
We wait for the motor to stop
Inside OAV, on another thread set_flatfield_on_zoom_level_one is started that sets the snapshot plugin
The snapshot plugin is set to OAV.MXSC in the plan
set_flatfield_on_zoom_level_one finishes, setting the snapshot plugin to CAM
We wait on the plan abs_set to OAV.MXSC - Forever!


Strange text behavior with 'TextEditorDecorationType' after setting 'font-size'

Help wanted: About font-size in TextEditorDecorationType
Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.64.2 (user setup)
OS Version: windows10 21H2

Steps to Reproduce:

new an extension template by yo code, choose JavaScript
copy the code and cover extension.js

const vscode = require('vscode');
let decoration = vscode.window.createTextEditorDecorationType({});
/**
 * @param {vscode.ExtensionContext} context
*/
function activate(context) {
    vscode.window.onDidChangeTextEditorSelection(()=>{
	const beginPosition = vscode.window.activeTextEditor.selection.active
	const endPosition = new vscode.Position(beginPosition.line,beginPosition.character+1)

	decoration.dispose();
	decoration = vscode.window.createTextEditorDecorationType({
			textDecoration: `none; font-size:5rem`,
	})
          vscode.window.activeTextEditor.setDecorations(decoration, [new vscode.Range(beginPosition,endPosition)])
    })
}

function deactivate() {}
module.exports = {
  	activate,
  	deactivate
}

of which main code is
decoration = vscode.window.createTextEditorDecorationType({
    textDecoration: `none; font-size:5rem`,
    })
vscode.window.activeTextEditor.setDecorations(decoration, [new vscode.Range(beginPosition,endPosition)])

Run Extension and new an any-type file
input some text

Then you will find :

without textDecoration:

with textDecoration:

The whole line of text moves down and overlaps on other line of text!
Expected Behavior
What I want is like:

I don't know if it is a bug or feature. Having searched for a long time, I didn't find how to achieve this effect.
If there is any way, please let me know. Thanks a lot! 😃

font-size or fontSize are not suuported DecorationRenderOptions.
see https://code.visualstudio.com/api/references/vscode-api#DecorationRenderOptions

font-size or fontSize are not suuported DecorationRenderOptions.
see https://code.visualstudio.com/api/references/vscode-api#DecorationRenderOptions

I don't think so. Actually you can use more css options in createTextEditorDecorationType. See this discussion. Actually these extensions have used this method : hoovercj.vscode-power-mode and 	yfzhao.ultra-math-preview

I searched for font-size or fontSize in the two extension repositories you mentioned.  No matches.

I searched for font-size or fontSize in the two extension repositories you mentioned.  No matches.
The first extension uses this for example: textDecoration: none; margin-top: 0px;I would say that syntax isn't officially supported, so don't be surprised if trying to injectfont-size` into that doesn't work.

They do not contain font-size option. Actually they are just examples to show how to add css to textDecoration . If you set 'height' larger than 1rem, that bug will occur , too.
I'm more interested in how the bug occurs and the solutions. If it is exactly unsupported, I'd like to request the feature🧐

@yfzhao20 I'm the author of vscode-power-mode -- the technique that I use in that extension is very much not supported by the API and it is only through the mercy of the vs code team that they haven't blocked it.
In the meantime, though, you can try also setting the line-height property on your decoration along with the font-size property.


Add pkgdata function

Introduces a function for loading package test data to facilitate more concise code in the unit tests.

Current coverage is 100% (diff: 100%)

No coverage report found for master at 44f0f89.

Powered by Codecov. Last update 44f0f89...1c2869d


Deep Learning compatability

Not an OPT specific issue, just an environmental note.
I have installed Theano and Caffe (and their dependencies) on my 'OPT Dev Box' w/ Kinect 2.
Not for openPtrack work - but for other GPU work work.
it's worth noting that ROS gets un-installed as a dependency (boost) is upgraded
ie - Theano works, but Caffe needs boost>1.55, and
sudo apt-get libboost1.55-all-dev
un-installs ROS, and thus OPT will no longer work.
I'm going to fuss w/ this more, see if I can get OPT working along side this deep learning toolchain.
(no i'm not using this https://www.autonomous.ai/deep-learning-robot - but would be cool if OPT was an option for those that do)
and for what it's worth, this took 2.5 days to train on the same box
https://github.com/Russell91/ReInspect
it's interesting to think of another detection layer that is perhaps trained by existing detections
however likely pure thought experiment / OPT works well as-is after calibration.
anyway, I'll share notes if i get OPT working again post-caffe...

I did not get as far with this, but it's still interesting... though I will say OPT already performs better than 'deep learning' based results like https://arxiv.org/abs/1604.07468 for example.
though it may be possible to augment & retain non-persistent IDs beyond RGB hinting.

OPT v2 "Gnocchi" will include convolutional neural network person detection based on Darknet's Yolo and pose recognition based on an approach from CMU that uses Caffe.


Add basic aria and camellia ciphers modes to default provider

The aes code has been refactored into generic and algorithm specific parts,
so that most of the code can be shared.

Checklist

[ ] documentation is added or updated
[x] tests are added or updated

This looks like nice cleanup.  Ping @dot-asm for his view.

Ping @dot-asm for his view.

I don't feel obliged to provide one.

So, uhmmm, what am I looking at now? There's this sudden dump of code that looks like AES cleanup...

So, uhmmm, what am I looking at now? There's this sudden dump of code that looks like AES cleanup...

Err the AES code related to all non AEAD modes was completely reworked so that it can be shared with camelia and aria and ... While I am here I also seperated out the platform specific bits into seperate files also (they didnt change code - just got moved to other .c files)

Cool. Just wanted to make sure I understand the intent (of course, had I read the message of the main commit... which I gave now... )

OK, I have a mix of likes and dislikes.
I really like that you separated out re-usable generic functionality. Really.
I do not like to see all ciphers dumped together in basic_ciphers.c and basic_modes.c, I would much rather have cipher specific code be divided up in cipher specific files and reserve basic_ciphers.c for just the generic functions that they all share (if there is any... did this remark totally sabotage your effort?)
I find the name basic_modes very confusing. Modes are "ede", "cbc", "ofb"... and this files has nothing to do with modes per se, and everything to do with provider dispatch implementations. basic_cipher_prov.c might be a better name, maybe?

OK, I have a mix of likes and dislikes.

Ok fair enough.. I will have a look at your concerns...

OK, I have a mix of likes and dislikes.

@levitte That was quite a bit of work to change.. See what you think now.

Other than what I just wrote, I really like the restructuring you've made now.

I have renamed things so many times - that I am also confused :)

I have renamed things so many times - that I am also confused :)

I figured :wink:

Noted the update, like it.  Now, to the Travis failures...

Now, to the Travis failures...

On it now..

ping: should be all good now..

@paulidale can you have a look at this PR and see that you are happy with it please.

Thanks for reviewing. Merged to master.


Avoids remote references in Event Type schema.

This avoids remote references breaking the compatibility assurances available in event types, as the remote reference  is not under direct control of the system managing the event  types.
For #237.

They should deference them and persist local references, in my opinion.
On 4 Jul 2017 12:04, "Bill de hÓra"<EMAIL_ADDRESS>wrote:

This avoids remote references breaking the compatibility
assurances available in event types being broken, as the
remote reference is not under direct control of the system
managing the event types.
For #237 https://github.com/zalando/restful-api-guidelines/issues/237.
You can view, comment on, or merge this pull request online at:
https://github.com/zalando/restful-api-guidelines/pull/265
Commit Summary

Avoids remote references in Event Type schema.

File Changes

M events/events.md
https://github.com/zalando/restful-api-guidelines/pull/265/files#diff-0
(17)

Patch Links:

https://github.com/zalando/restful-api-guidelines/pull/265.patch
https://github.com/zalando/restful-api-guidelines/pull/265.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/zalando/restful-api-guidelines/pull/265, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAaPnXnOti4EFBXO-3i3-XZVP4T2abmqks5sKg4SgaJpZM4ONJp0
.

They should deference them and persist local references, in my opinion.

That doesn't work cleanly:

There's no compatibility control - what if the deref'd  schema is not compatible?
There's no assurance of availability  - what if the deref'd  schema is 4xx or 5xx?
What is the version of the local definition if the remote definition changes?
For events, what should the actual schema look like, inlined?

This is a general design problem with Open API - remotes refs allow runtime redefinition of schema, which is very close to or maybe isomorphic to transitive dependencies in libraries. The challenges here are not great for API descriptions when the API documents are not in a closed system, and are highly problematic for events, which are strongly coupled to definitions.

It should be dereferenced on creation (or future updates) of the event type definition. It should dereference it synchronously in the scope of the original request. Failure to do so results in a failure for the client trying to create or modify the event type (no modification will be applied).
That is easy to understand for clients and solves all the issues while it still allows for remote references, which are quite handy.

And yes, the result should be inlined.

And yes, the result should be inlined.

This is effectively saying event schema should be statically linked. It's not clear doing that mechanically is a better idea than the owner doing it manually since the latter gives the owner full control (eg the way modern library systems use lock files to stop transitive breaks). And at which point the owner should probably just inline it, as nothing bad can happen.

It should be dereferenced on creation (or future updates)
...
And yes, the result should be inlined.

This suggests some system should inline a cached copy and maintain a link reference as well - I don't think that can be expressed in Open API. One issue with the just inline approach is that it can't be specified without making assumptions about implementations.
Please keep the objections coming, I think it's important we have a view on this. For data imvho, reproducibility is critical, and any construct that allows accidental redefinition of schema is a problematic, which is to say, ref is a problem.

And at which point the owner should probably just inline it themselves

That is a possibility of course, but that would mean that my 100 line event definition might be bloated by another 1k lines because I depend on something else, e.g. JSON Schema itself or JSON Patch.
Inlining them locally would also work, but doing that automatically requires tooling that is also not available.

@dehora, @whiskeysierra: The cases where we use remote references today are very stable resources. Actually, I do not think that any of them semantically changed since we defined them externally.
But I agree, it would be a bad thing to link to changing resources.
Instead of avoiding them, I would suggest to make clear, that referenced resources are not allowed to change at all - any change requires to create a new permanent resource with a new semantic version number. Systems, are allowed to treat the references as permanent resources that can be infinitely cached by the processing systems. Changing a referenced resource is a violation of the API guideline.
To support this, I think our API management system, in the long term should provide a proxy service for resolving references. It should detect changes of resources, send violation e-mails to the resource owners, and ensure the consistency of API definitions by forwarding the unchanged version.

After our discussion in the last guild meeting, @tfrauenstein @maxim-tschumak and @mfellner continued the discussion for some time, and we agreed, that we have conceptually the same problem for REST APIs as for Event APIs. The vision we have in mind, that we intend to validate compatibility of REST APIs and request in the future to uncover risk of unilateral API changes.
We concluded, that all systems registering APIs (i.e. API storage, Nakadi) should be supplied with de-referenced API specifications, to keep systems simple and understandable. An alternative approach would have been to introduce versioned references, but it was discarded because of the inherent complexity to check the referenced definition against a registration database.
I think we also agreed, that rules for both types of APIs, therefore, should be in sync.
I'm currently not sure, if everyone shares my opinion, that reuse of definitions is a good concept. I like it because, reuse via references shortens APIs and makes reading of APIs a lot easier. I although like the Idea of having a repository or database of common versioned definitions for events hanging around.
My suggestion: let us find a common place for a rule that is formulated as follows:
## {{ book.must }} Not Use Remote References in Released Schema Definitions

While the reuse of a definition via `$ref` keyword improves the readability of an API,
remote references are a problem when definition and application of the API is getting
temporally and spatial decoupled. If an API definition is only reassembled at runtime,
transitive changes of remote schema definitions that are not directly managed as part
of the API can break the expected contract of the service or event producer/consumer.

Thus, it is inevitable to provide a single, self-contained API schema when releasing or
registering an API schema. I.e. whenever an API is released within a container image
or registered with an event bus, the API schema has to be de-referenced upfront.

the API schema has to be de-referenced upfront

Will you provide tooling for this?

We concluded, that all systems registering APIs (i.e. API storage, Nakadi) should be supplied with de-referenced API specifications, to keep systems simple [..].

To rephrase this, it will  keep the API storage and Nakadi simple, but force everybody else to deal with inlining referenced schemas which introduce a complexity to all APIs that use them, e.g. makes it harder to read and update/maintain them.

@whiskeysierra While my suggestion is formulated a bit more restrictive than the original request, I think it is even the better approach, as the weak formulation would not solve the problem. Registration system have to reject APIs anyway, if they have to protect the consistency of results.
As far as I see, there is at least one tool available, but I assume other tools have even build in support:
https://www.npmjs.com/package/json-schema-ref-parser

From the meeting:

for REST-APIs, it should be enough to use immutable URLs (like ones referring to Github commits or tags).
for event definitions, the event type registry should do the inlining (and the state at the registration time counts).

This needs to be converted to AsciiDoc.

Discussed again in the meeting.

We want to not have it in the guidelines.
If Nakadi wants to prevent this, it is free to do so. If people complain about this, maybe that would be the time to discuss implementing the resolution in Nakadi.
For Restful APIs, we don't write anything into the guidelines now. When the API repository wants to implement version comparisons, it needs to resolve the links, and store the results.


Fix CI

Install SDL2 as one of the steps

Codecov Report

Merging #1 into master will decrease coverage by 3.49%.
The diff coverage is 31.50%.

@@            Coverage Diff             @@
##           master       #1      +/-   ##
==========================================
- Coverage   98.35%   94.85%   -3.50%
==========================================
  Files           7        8       +1
  Lines        1154     1205      +51
==========================================
+ Hits         1135     1143       +8
- Misses         19       62      +43

Impacted Files
Coverage Δ

gui-app/src/app.rs
0.00% <0.00%> (ø)

gui-app/src/main.rs
0.00% <0.00%> (ø)

emulator/src/vm.rs
98.25% <64.51%> (-0.67%)
:arrow_down:

emulator/src/graphics.rs
98.38% <100.00%> (-1.62%)
:arrow_down:

emulator/src/memory.rs
97.50% <0.00%> (ø)

Continue to review full report at Codecov.

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

Codecov Report

Merging #1 into master will decrease coverage by 4.04%.
The diff coverage is 28.39%.

@@            Coverage Diff             @@
##           master       #1      +/-   ##
==========================================
- Coverage   98.35%   94.30%   -4.05%
==========================================
  Files           7        8       +1
  Lines        1154     1212      +58
==========================================
+ Hits         1135     1143       +8
- Misses         19       69      +50

Impacted Files
Coverage Δ

gui-app/src/app.rs
0.00% <0.00%> (ø)

gui-app/src/main.rs
0.00% <0.00%> (ø)

emulator/src/vm.rs
98.25% <64.51%> (-0.67%)
:arrow_down:

emulator/src/graphics.rs
98.38% <100.00%> (-1.62%)
:arrow_down:

emulator/src/memory.rs
97.50% <0.00%> (ø)

Continue to review full report at Codecov.

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


added hello world in swift language

Why:

Closes #20
What's being changed:
Print Hello World in swift language

Ok, no problem! Thanks

Done, i change the file name !Thanks

Hey @renanlmiranda
Please comment !assign on #20 to be auto-assigned to the linked issue (to prevent any conflict)
!assign

@harshraj8843 Okay, I commented on #20


Attempt using labels for jx 3 pipeline activities

Hi @rishka. Thanks for your PR.
I'm waiting for a jenkins-x-plugins or todo member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign rawlingsj
You can assign the PR to them by writing /assign @rawlingsj in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/ok-to-test
/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jstrachan
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

I guess we could always modify the code to list all activities and look for either format?

I guess we could always modify the code to list all activities and look for either format?

Yea, its possible. I am going to work with this some more, as it unfortunately doesn't fix anything other than the error message in logs.
I do kind of wonder if it should just be versioned, base it on lighthouse-version or something

this PR tries to be more label agnostic: https://github.com/jenkins-x-plugins/jx-slack/pull/35

this PR tries to be more label agnostic: #35

Tried it out, looks like it works on my side, closing this one


#189 Do not persist nil values in result database

CFNull is not a valid value in the plist and leads to
an error when attempting to persist it in the
UploadResultDatabase.
Instead of writing CFNull if the message is nil
we write an empty string.

@lyio is this read back properly as well?

@lyio is this read back properly as well?

I did not encounter any problems with that. With this change, the plugin now correctly handles my cases.


Normalize indentation on applyMiddleware.spec.js

Thanks, but it should be a 2 space indent to be consistent.

@timdorr branch updated

Awesome! Thanks!


Add defaultMainAutoconfWithDoctests

[ ] write notes in README

It seems you have to move sources into src/ (or something not .); otherwise things fail weirdly.

I smoke tested this with network package; works with cabal sandbox and stack build approaches at least. (Changes required were quite small: https://github.com/haskell/network/pull/245).

I'm impressed the fix for #8 was this simple! Thanks, @phadej!


propose obsoletion GO:0072692 chromatin silencing at centromere central core

Can we obsolete this term, I think I requested it but it was a mistake
GO:0072692 chromatin silencing at centromere central core
there is not "silencing" at the core. It is sometimes assayed, but inserting a reporter gene, but essentially this is the region under the kinetochore so there  are no genes here, and no biological mechanism to "silence" them (i.e the RNAi based heterochromatin assembly occurs at the centric/pericentric regions).
And to corroborate this, it has no annotations
val
Reported by: ValWood
Original Ticket: geneontology/ontology-requests/11147

Maybe these terms should really be chromatin organization?
chromatin silencing at centromere central core -->chromatin organization at centromere central core
Maybe for all of these centromere terms, this should be the case?

For the core, I think yes.
There is transcription at the non core regions (which is required for heterochomatin formation). I don't think there is normally 'functional' transcription in the central core.
I changed  the go annotation  to CENP-A containing chromatin origanization.

OK. I will change the core term to organization. OK?

Its only this term...obsolete?
I annotated to CENP-A containing chromatin origanization instead.
Silencing at the centromere still exists, but not in the core.

OK. I will wait until all the updates are done and there are no more annotations. Then I will obsolete.

Its OK for us if you do it now.....( The update with this fix from us will be quite a while).
Val

Dear all,
The proposal has been made to obsolete the following term:
GO:0072692 chromatin silencing at centromere central core
The reason for obsoletion is that the process does not exist.
All details here:
https://github.com/geneontology/go-ontology/issues/11322
There are no experimental annotations to this term.
We are opening a comment period for this proposed obsoletion. We'd like to
proceed and obsolete the terms above on November 24, 2016.
*** Unless objections are received by November 24, 2016, we will assume
that you agree to this change. ***
Best,
-David Hill

Made this term obsolete.


Use new download location for Relay

Describe the bug
Relay used to use GitHub release artifacts, but as of v0.6.2 switched to R2 (same as S3) for artifacts because we were running into GitHub limitations.
Version

[x] I have checked releases, and the bug exists in the latest patch version of v1 or v2.
[x] v2
[ ] v1

Runners

[x] GitHub Hosted
[x] Self Hosted

Operating systems
All
PHP versions
All
To Reproduce
See https://github.com/cachewerk/homebrew-tap/commit/08c1b8ce1cf8ed99ebabbb0190f6285854a53fd6
Are you willing to submit a PR?
Yes.

Switched in 7d999d501a7b77228685777a2f7d0e60217d277c

Released 2.25.0 with the changes.


Fix command line option names

Coverage remained the same at 100.0% when pulling 0bedb18e89131d8c42ab5aca29cdddbeb4937648 on musclesound:master into fd1359c28d78e9112b10680cf6ef5ebcd9061f3c on dannnylo:master.