Pass url parameters from dashboard to charts

CATEGORY
Choose one

[x] Bug Fix
[x] Enhancement (new features, refinement)
[ ] Refactor
[x] Add tests
[ ] Build / Development Environment
[x] Documentation

SUMMARY
Currently URL parameters are correctly handled in explore mode. However, current functionality is limited/broken in the following ways:

Any default values for url_params specified in charts are erased if the chart is opened via the Charts page.
query parameters passed to dashboards are not forwarded to chart requests, making them unusable in dashboards.

This PR does the following:

Default values for query parameters are left unchanged when opening a chart via the Charts page.
URL parameters passed to a dashboard are updated to the chart defaults prior to rendering.
Add note about setting default values for url_params.
Add cypress test to ensure params are passed through from the dashboard to chart requests.
Add python test to ensure that url params are appended to pre-defined params.

SCREENSHOTS
Datasource query:

Chart metadata with default values for url_params:

Dashboard with url parameter (note name2 falling back to default value):

TEST PLAN

Tested locally
CI
Ensured that new tests fail when run on master branch

ADDITIONAL INFORMATION

[x] Has associated issue: closes #6708
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS
@senturkayhan @rsdpyenugula @mistercrunch @graceguo-supercat @etr2460

Codecov Report

Merging #8536 into master will decrease coverage by <.01%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##           master    #8536      +/-   ##
==========================================
- Coverage   66.72%   66.71%   -0.01%
==========================================
  Files         449      449
  Lines       22696    22699       +3
  Branches     2366     2366
==========================================
+ Hits        15143    15144       +1
- Misses       7415     7417       +2
  Partials      138      138

Impacted Files
Coverage Δ

...t/assets/src/dashboard/reducers/getInitialState.js
0% <0%> (ø)
:arrow_up:

superset/views/core.py
71.98% <100%> (+0.01%)
:arrow_up:

superset/utils/core.py
88.65% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

This is ready for review

Nudging testers/reviewers, would be great to get 👀 on this to kill this long standing bug.

Hi, i am trying this and have followed the solution but it will not filter for me. i have done they query in sql lab, made the chart and changed the metadata that makes a table with name_1, name_2, but changing the URL does not seem to filter my dashboard. Any idea why this is not working? Did you test this on a server rather than locally just? thanks!
Version 0.35.1

@ForzaDesignStudio unfortunately this had to be dropped from 0.35.2 as it didn't apply cleanly to the 0.35 branch. As it stands this fix will be available in the 0.36.0 release, probably due out this month.

thank you for the quick reply. i will try it in the next release.


load map and close btn in infobox example over https

Prevent Mixed Content error on basic and label infobox example pages.

Thanks Honza


react-spring.es.js?1586:2126 Uncaught (in promise) TypeError: __WEBPACK_IMPORTED_MODULE_4_react___default.a.createContext is not a function

it showing me following error: react-spring.es.js?1586:2126 Uncaught (in promise) TypeError: __WEBPACK_IMPORTED_MODULE_4_react___default.a.createContext is not a function.
Here is my code:

<Transition
native
config={{ tension: 1, friction: 10 }}
keys={location.pathname.split('/').filter(a => a)[0]}
from={{ transform: 'translateY(500px)', opacity: 0 }}
enter={{ transform: 'translateY(0px)', opacity: 1 }}
leave={{ transform: 'translateY(500px)', opacity: 0 }}>
{style => (

)}

    </ErrorBoundary>
  </Router>

@anilljoshi Do you have any errors in terminal where you run webpack?

@anilljoshi You probably need to update react. spring requires react 16.3

Take a look at npm/yarn warnings please
https://github.com/drcmda/react-spring/blob/master/package.json#L75-L76


:bug: last update date is always 00:00 in users page

This is because the lib fails to parse timestamp passed to from Firestore, and hence moment(timestamp) becomes today's date.


[BUG] capture_RPi should output ERROR message for failed capture

capture_RPi needs to output an ERROR message (debug level 1) with the retcode when unable to take a picture, and if /tmp/capture_RPi_debug.txt exists, add that to allsky.log.
It should also tell RPi_capture() to log the next libcamera-still output to /tmp/capture_RPi_debug.txt.

Closing.  Duplicate of #2972.


Relative imports do not work in the browser namespace

Inside browser/a.sol, an import import "./x.sol"; does not find x.sol, although browser/x.sol works.
It seems like a.sol is not properly passed to the compiler with its fully qualified name, or there are some problems with slashes in the beginning.

I can't reproduce - both import "./Untitled1.sol";  and import "browser/Untitled1.sol"; work . Are you sure it was with the browser namespace ?

I still have it here...

This is still happening to me.


fix CI

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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 82.612%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2442

Relevant Lines:
2956

💛  - Coveralls


[zh] Linear Algebra and Calculus

linear algebra part is finished.

Awesome work @tigerneil! Now let's start the review process (cc: @spin6lock).


fragment: ParseContentPath doesn't account for URL re-writes

If a fragment store is configured to use Find/Replace, then ParseContentPath() of fragments from that store gives the wrong journal name (specifically, it gives the post-replace name, which is incorrect).
I don't think it's fixable. Instead, ParseContentPath should be removed altogether, and the store List() functions should instead use ParseContentName. They already have the correct journal name available to them anyway, and should use it.

fixed.


Update readme, prepare to release yaml_decode v1

Fixes https://github.com/dart-lang/json_serializable/issues/437

@natebosch – you seem the kinda guy who would want to chime in on the package name
Thoughts?

It looks like the reason this exists as a separate package is because we don't want add a dependency from json_annotation to yaml or vice versa.
Can we consider changing the signature of toParsedYamlException such that we pass the individual fields from the CheckedFromJsonException instead of that type? Then this code could live in package:yaml right?

Going to rename the package to checked_yaml – see the last PR. We have a better plan here.


Fix diagnostic SIenph

What changes does this PR introduce?
small bug fix
What is the current behaviour?
diagnostics of SIenph does not include HSNOW
What is the new behaviour
SIenph now includes HSNOW
Does this PR introduce a breaking change?
No
Other information:
Suggested addition to tag-index
Include HSNOW into diagnostics of SIenph

@amc2507 Thanks. Going to merge this PR soon.


Web page constantly requests microphone access in Firefox because it's served 80/http instead of 443/https

Description of issue in object-deviation format
When  attempting to record phrases in Firefox on Linux, the below warning is repeatedly presented when ascribing microphone permissions to the web page. This dialogue is shown every time you press the space bar, which is pretty frustrating. You can't tell it to remember the choice because the mimic-recording-studio is served unencrypted.

I know this isn't any fault of the mimic-recording-studio but it's likely to be a bit of a support headache
Workaround
Use Chrome instead, this behaviour is not observed in Chrome.
Desired solution
Is it possible at all to serve this over https/443 to prevent the error? I'm guessing a lot of the people who would likely take mimic-recording-studio for a spin are likely to use Firefox in preference to other browsers.

I was about to open an issue with this subject. I solved it by using an apache reverse proxy, as I have a wildcard lets encrypt certificate. On my Apache configuration I just had to create a new VirtualHost as follows.
 <VirtualHost mimic.docampo.net:443>
     ServerAdmin<EMAIL_ADDRESS>     ServerName mimic.docampo.net:443
     ServerAlias mimic.docampo.net:443
     SSLEngine on
     SSLProxyEngine on
     SSLCertificateFile "/root/.acme.sh/*.docampo.net/fullchain.cer"
     SSLCertificateKeyFile "/root/.acme.sh/*.docampo.net/*.docampo.net.key"
     ErrorLog "/var/log/httpd/mimic.docampo.net-error_log"
     CustomLog "/var/log/httpd/mimic.docampo.net-access_log" common
     ProxyPass / http://<IP_ADDRESS>:3000/
     ProxyPassReverse / http://<IP_ADDRESS>:3000/
 </VirtualHost>

It just need to be documented while a self-signed certificate is built into the docker image.

By the way, when I tried on Chrome without SSL, the application crashed

So, two problems solved ;)

I have worked on similar code that uses the microphone like this.
I believe that many browsers deny access to the microphone unless the web page is served via HTTPS or from localhost.  Firefox may be an exception.
I believe that the constant re-prompting is due to some specific browser API, the result of which could be re-used.  Re-using that result would make it a single prompt when the page is loaded.

A find solution - just add to start line - ""start":"HTTPS=true react script start" in frontend/package.json. yarn will create certificates, you just need to accept them. As for me - i changed port to usual ssl port 443.

To ignore Chrome’s secure origin policy, follow these steps.
Navigate to `chrome://flags/#unsafely-treat-insecure-origin-as-secure` in Chrome.
Find and enable the `Insecure origins treated as secure` section (see below).
Add any addresses you want to ignore the secure origin policy for.
 Remember to include the port number too (if required).Save and restart Chrome.


D435 Get IR PROBLEM

When I just get Depth stream and Color Stream is working，but when I add IR stream
It show rs2::error
Does anyone can help me Idk how to deal with

@alexex0723 , can you provide some details ?

Camera type / OS / SDK version/ Executable
The error snapshots/logs encountered

Camera Model - D435i
OS - Win10
Platform - PC
SDK Version 2.17.1
My code is like this (I just show some setting part bcz my code is long)
rs2::pipeline pipe;
rs2::config cfg;
cfg.enable_stream(RS2_STREAM_DEPTH, 640, 480, RS2_FORMAT_Z16, 30);
cfg.enable_stream(RS2_STREAM_COLOR, 640, 480, RS2_FORMAT_BGR8, 30);
cfg.enable_stream(RS2_STREAM_INFRARED, 640, 480, RS2_FORMAT_Y8, 30);
rs2::pipeline_profile profile = pipe.start(cfg);
rs2::colorizer c;
auto depth_stream = profile.get_stream(RS2_STREAM_DEPTH).asrs2::video_stream_profile();
auto color_stream = profile.get_stream(RS2_STREAM_COLOR).asrs2::video_stream_profile();
auto infrared_stream = profile.get_stream(RS2_STREAM_INFRARED).asrs2::video_stream_profile();
Error is say 0x00007FFC2B299129 (in Depth_camera.exe) An unhandled exception occurred: Microsoft C++ has an exception: rs2::error (memory location 0x000000B5B435C9B0). occur

@alexex0723 Did you have such IR issue with latest RealSenseViewer(https://github.com/IntelRealSense/librealsense/releases/download/v2.24.0/Intel.RealSense.Viewer.exe)?

@alexex0723  Any update? Looking forward to your reply. Thanks!

@alexex0723 Could you please update if you still need support on this? Thanks!

@alexex0723  As we didn't get update from your side, will close the issue now. Please feel free to create another new one if you still have issue. Thanks!


Can I parallelize just one large layer?

Can I parallelize just one large layer across gpus and keep all other layers the same and work with distributed data parallel?
For example, a regular resnet18 model, which is small enough but the number of classes is 1 million, how do I parallelize just the classification layer?

Hey,
Did you try this? did it work?


SUPPORT - Send authenticated email

Hello guys,
Which parameter I will use send authenticated email ?
I tried
DD_EMAIL_URL='smtp://USER@:PASSWORD@SERVER:25'
But doesn´t work.
I see another parameter :
EMAIL_HOST = 'smtpout.your_domain.com'
EMAIL_PORT = '25'
EMAIL_USE_TLS = True
Deployment method
[ X ] Kubernetes

DD_EMAIL_URL works, but  just for special characters you need to use url encoding:
smtp://username:passwrd@smtp:25
in if you have username@domain:password@smtp_server then encode first @ (IIRC)

@dsever
For example:
username:
<EMAIL_ADDRESS>pwd:
Work_2021!
smtp:
smtp.office365.com
The request will be:
<EMAIL_ADDRESS> ?


Update to v0.10.3

The 0.10.3 release includes a fix for a severe startup performance regression introduced in v0.10.1 (https://code.visualstudio.com/updates)

Thank you for the contribution. It needed some fixes, so they were made in 54d0c5cb7e91d2324fedee61f3082efd1fa77b6c. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


[ISSUE #4606] Build trace msgs for DefaultLitePullConsumer when poll is called

Make sure set the target branch to develop
What is the purpose of the change
Fix #4606
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.04%) to 47.721% when pulling 5c3a3ddfc9074ffd9631b4c98f75f88e45e2a89f on cserwen:lite_pull_trace into 74e1a60b37829e3730ad165ff6ea73b6c52b3dfe on apache:develop.


DRYed GeoFeatureModelSerializer

This PR is an attempt to DRY out the GeoFeatureModelSerializer: main field loop is done in the DRF base class ModelSerializer, GeoFeatureModelSerializer only re-shuffles and adds geo specific fields. Also provides hooks to modify properties field.

hi @pglotov, won't this conflict with #71?

It will, but i think it is a better approach since we don't duplicate field loop, just repackage base class result and add geo specific fields which base class is not aware how to deal with. I believe it's a cleaner approach. @sh4wn what do you think, are the hooks in this PR enough for customizing properties field?

I think #71 is a good approach, the naming of the methods are clear, there's a bit of usage documentation and I don't see a duplication of the field loop. If you see any way to improve #71 leave comments there please.

Compare
https://github.com/tomchristie/django-rest-framework/blob/master/rest_framework/serializers.py#L450
and
https://github.com/djangonauts/django-rest-framework-gis/blob/master/rest_framework_gis/serializers.py#L144

Thank you @pglotov, we can keep track of this in a new issue, could you open it?


Add a default isSmallChange method that returns false

The default action for isSmallChange should be false so it is not required to implement this method when the user doesn't want this feature.

It's false by default already, right?

You're right, It would return undefined from the execute helper. For some reason I thought if it wasn't defined it would throw an error, but that's not the case.

Actually, I'm even more wrong. You did define it as false. I completely missed it. :disappointed:

hahaha :stuck_out_tongue:


fix containerservice parent pom

The parent pom (containerservice/resource-manager/pom.xml) should be a aggregate pom, which would not publish jar.
But still I could not find any artifact related to these poms, either on maven or sonatype.


Optimize master log, add workflow instance id and task instance id in log

Purpose of the pull request
close #10496
Add log prefix help for troubleshuting.

Codecov Report

Merging #10516 (fd614aa) into dev (79ce5d7) will decrease coverage by 0.09%.
The diff coverage is 7.28%.

@@             Coverage Diff              @@
##                dev   #10516      +/-   ##
============================================
- Coverage     40.87%   40.78%   -0.10%
+ Complexity     4852     4847       -5
============================================
  Files           886      888       +2
  Lines         36032    36101      +69
  Branches       3998     3993       -5
============================================
- Hits          14729    14723       -6
- Misses        19847    19919      +72
- Partials       1456     1459       +3

Impacted Files
Coverage Δ

...apache/dolphinscheduler/common/thread/Stopper.java
0.00% <0.00%> (ø)

...he/dolphinscheduler/common/thread/ThreadUtils.java
0.00% <0.00%> (ø)

...e/dolphinscheduler/server/master/MasterServer.java
0.00% <0.00%> (ø)

...r/server/master/processor/StateEventProcessor.java
0.00% <0.00%> (ø)

...er/server/master/processor/TaskEventProcessor.java
0.00% <0.00%> (ø)

...master/processor/TaskExecuteResponseProcessor.java
0.00% <0.00%> (ø)

...r/server/master/processor/TaskRecallProcessor.java
0.00% <0.00%> (ø)

...ter/processor/queue/StateEventResponseService.java
0.00% <0.00%> (ø)

...uler/server/master/runner/EventExecuteService.java
0.00% <0.00%> (ø)

...er/server/master/runner/FailoverExecuteThread.java
0.00% <0.00%> (ø)

... and 30 more

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 79ce5d7...fd614aa. Read the comment docs.

Just take a quick look, and did you check whether MDC applies in this case? I think we can leverage MDC to uniformly add workflow id / instance id / task id / task instance id , etc. in our logs without too many efforts, also we won't miss these when adding new logs in the future.
MDC: https://logback.qos.ch/manual/mdc.html

Thanks for your wonderful suggestion, MDC can work well.


Assert if we're looking at captures before they're computed, and fix isolation checking bugs

This PR adds some stronger assertions and fixes the fallout. A follow-up PR will actually introduce the lazy requests for captures, because this change is already getting too large.

@swift-ci Please smoke test

@swift-ci Please test source compatibility


Flashing caustics

Hi,
As I'm testing LuxCore for Blender, I've encountered an issue where I get flickering caustics even with persistent photon mapping caching turned on. The object is static with a moving camera that has DOF. Lighting is done by HDRI. I've tested without caching the caustics photons and the flickering happens then as well. The render is only 128 samples but many neighbouring frames does not exhibit the flashing caustics which leads me to believe it is not purely a sampling issue?
I've uploaded the scene and render here: https://drive.google.com/drive/folders/1fEXR8i7L5-berCS0ZeDIOQLd3v8egyF1?usp=sharing
Blender 2.83.5
LuxCore 2.4
Thanks,
Ole

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

I'm somewhat aware of the problem, it should be tracked back some Metropolis numerical precision problem and env. light (distant light sources like sun, sky, HDR, etc.).
It shouldn't happen whit local lights like point, spot, area, etc. light sources.

I can not download your test scene.
Anyway, I should have fixed the problem. The fix will be included in next automatic v2.5 build (available in a couple of hours): https://forums.luxcorerender.org/viewtopic.php?f=9&t=736


Pan & Zoom: Enable Feature Flag

When pan and zoom is ready for release, we need to enable the feature flag.
Let's not remove the feature flag, or set it's default to true because we may want to use it as a testing ground for other animation effects in the future. For this ticket, we should just remove the conditional rendering (based on the feature flag) for the pan + zoom animation effect in the effect chooser

This feature looked good in bug bash so we can merge as soon as Claudio is done with QA.
@csossi we need this one for the 1.6 release 🙃

Verified in QA


node2vec

node2vec example and model

Hi @b4rthm and @rusty1s . I'm interested in the structural similarity between nodes (for example, capturing roles that nodes play in a social network graph). Is there some plan to integrate other node embeddings methods to PyG, such as struc2vec (https://arxiv.org/pdf/1704.03165.pdf)?

Not yet, but please feel free to request that feature in a new issue so we can keep track of it there.


Control when checks are triggered

For example, I don't need to run my test suite if the only staged file is a Markdown file.
Possible API:
precommit.check(checks.Whatever(trigger_exclude=r".*\.md"))

Actually this already works with the exclude parameter: by default if neither include nor exclude is specified (as it is for the Command check), the check will run whenever any files are staged. If either is specified, then it works the same way regardless of the pass_files parameter.


Release v0.10

Prepare release v0.10.
Depends on

https://github.com/trussed-dev/trussed-auth/pull/21
https://github.com/Nitrokey/trussed-secrets-app/issues/58

Using direct commit hash for this release, as trussed-auth is not yet released

Done:  https://github.com/Nitrokey/trussed-secrets-app/releases/tag/0.10.0


COSMIC App Store is built with PackageKit support

COSMIC App Store is built with PackageKit support, but rpm-ostree doesn't support PackageKit, which means a lot of apps will show a "System" option, and when you try to install using it, it will fail.
Maybe there is some way to rebuild the store without the PackageKit feature on from this repo?

Not sure if there's anything we can do to explicitly disable PackageKit support in the store, however you can get around the errors by adding Flathub as a user Flatpak repo.  The App Store prefers Flatpaks over PackageKit for all packages except Steam and Lutris.
For Steam and Lutris, the App Store quietly falls back to Flathub.


acctz-5.1 & 6.1 test plans require an update

Tests acctz-5.1
https://github.com/openconfig/featureprofiles/blob/a889483110c5e6e7371b57c8313c4a2d2cbd7db6/feature/security/gnsi/acctz/RecordSubscribeIdleTimeout/README.md?plain=1#L1
and acctz-6.1
https://github.com/openconfig/featureprofiles/blob/a889483110c5e6e7371b57c8313c4a2d2cbd7db6/feature/security/gnsi/acctz/RecordSubscribeIdleTimeoutDoA/README.md?plain=1#L1
ask to validate the legacy timeout behavior that was removed in later versions of the acctz.proto (client streaming RPC with 60s keepalive / 120s timeout)
We believe the test plans should be updated or removed. @morrowc @haussli @xw-g
cc @cm-nokia

Thanks. I will update these next week.

Reopening since the pr (#2708) is still open


Man/disable directive types

Summary of the changes (Less than 80 chars)

Added support to remove built in directive types in schema

Closes #4795

This is not the right way to solve this issue. The issue stated is that there are some directives that should be hidden from the Apollo schema. The way we want to solve this is in the federation schema printer.  The change we have in mind is probably a 10 line code change.

I am closing this PR since we want to solve this approach with a completely different approach.

That said, you can still have a go at it. Ping me on slack and we can discuss how to implement the change.
https://slack.chillicream.com


Metatype key paths

This is a branch of #23358 including work on a feature I've removed from the accompanying evolution proposal. At some point I'll sort through this and merge the good parts of it behind a feature flag so it doesn't rot.
The feature is metatype key paths:

You can form a key path to a static or class property using \T.Type.foo
You can apply a metatype key path to a metatype using static subscript(keyPath:)
The feature should backwards deploy with some minor issues

When I stopped, I was adding a resilience test that revealed some of what I'd done did not work with static let properties; I was also having issues with the identity values chosen for static key paths. I probably need to go back, figure out what values I actually want to have in the sil_propertys in various cases, and test that I'm getting them.

Can one of the admins verify this patch?

Please update the base branch to main by Oct 5th otherwise the pull request will be closed automatically.

How to change the base branch: (Link)
More detail about the branch update: (Link)


582 enhancement pull request template

Merge Request - GuideLine Checklist
Guideline to check code before resolve WIP and approval, respectively.
As many checkboxes as possible should be ticked.
Checks by code author:
Always to be checked:

[x] There is at least one issue associated with the pull request.
[x] The branch follows the naming conventions as defined in the git workflow.
[x] New code adheres with the coding guidelines
[x] Make sure that the pre-commit linting/style checks pass.
[x] No large data files have been added to the repository. Maximum size for files should be of the order of KB not MB. In particular avoid adding of pdf, word, or other files that cannot be change-tracked correctly by git.

If functions were changed or functionality was added:

[x] Tests for new functionality has been added
[x] A local test was succesful

If new functionality was added:

[ ] There is appropriate documentation of your work. (use doxygen style comments)

If new third party software is used:

[ ] Did you pay attention to its license? Please remember to add it to the wiki after successful merging.

If new mathematical methods or epidemiological terms are used:

[ ] Are new methods referenced? Did you provide further documentation? Has the glossary been updated?

The following questions are addressed in the documentation if need be:

[ ] Developers (what did you do?, how can it be maintained?)

[ ] For users (how to use your work?)

[ ] For admins (how to install and configure your work?)

For documentation: Please write or update the Readme in the current working directory!

Checks by code reviewer(s):

[ ] Is the code clean of development artifacts e.g., unnecessary comments, prints, ...
[ ] The ticket goals for each associated issue are reached or problems are clearly addressed (i.e., a new issue was introduced).
[ ] There are appropriate unit tests and they pass.
[ ] The git history is clean and linearized for the merge request. All reviewers should squash commits and write a simple and meaningful commit message.
[ ] Coverage report for new code is acceptable.
[ ] No large data files have been added to the repository. Maximum size for files should be of the order of KB not MB. In particular avoid adding of pdf, word, or other files that cannot be change-tracked correctly by git.

@mknaranja @DavidKerkmann @jubicker @annawendler @HenrZu
Please have a brief look at a first version, and feel free to provide suggestions.
You can easily view this file in Markdown in VSC with a markdown view (on Windows it is STR+shift+v).

Looks good and way more easy to read. 👍
I think a little more detail wouldn't hurt in some places,  since we're primarily targeting developers with the template who are new to the project. Therefore, I would also add the restriction to small files to the review instructions again. Just to avoid adding large data which is hard to resolve if merged once.
Also in my opinion it is important to mention that the documentation should be with doxygen.

Looks good and way more easy to read. 👍
I think a little more detail wouldn't hurt in some places, since we're primarily targeting developers with the template who are new to the project. Therefore, I would also add the restriction to small files to the review instructions again. Just to avoid adding large data which is hard to resolve if merged once.
Also in my opinion it is important to mention that the documentation should be with doxygen.

Thanks, that's helpful, if you could make code suggestions for that, that would be awesome. :-)

I also think that it looks good! The only remark that I have is that the links to the coding guidelines etc. need to be updated.

@mknaranja @DavidKerkmann
Could you have a short look at this, so i can get this out of my mind? :-)
I attached a simple side by side for a quick overview for you.

Thanks very much @mknaranja, i added almost all your suggestions, one or two I abbreviated so that that line is not too long.


SP-1040 - Workaround for versioning in actions #patch

Codecov Report

Merging #29 (3b2a84c) into main (d35017d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #29   +/-   ##
=======================================
  Coverage   80.52%   80.52%
=======================================
  Files          17       17
  Lines         380      380
=======================================
  Hits          306      306
  Misses         48       48
  Partials       26       26

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 d35017d...3b2a84c. Read the comment docs.


Improvements to developer onboarding documentation

Summary
In a recent conversation with @damianavila, he noted that it was a bit confusing to learn a few important things about the pilot hubs:

The overall architecture and design of the pilot hub infrastructure
Information about cloud accounts and projects that exist, and how these relate to our deployment infrastructure
Getting started information to quickly get up-to-speed with development
Information about hub infrastructure that is not in pilot-hubs and how to access it.

Acceptance criteria

We've got updated Pilot Hubs documentation that @damianavila thinks would improve upon these issues. (I think Damian's perspective is particularly useful as he is still relatively new to this repository).

Tasks to complete

[ ] Agree upon a subset of additions that we should make to the documentation
[ ] Make it clear where the documentation is for "hub administrators" vs. where it is for 2i2c engineers who might work on the base infrastructure
[ ] ...

@damianavila perhaps you could share a few other items (or just update the comment above) with some specific ways that the pilot hubs docs could be improved?

I look forward to this, @damianavila! The current architecture is a bit slapdash, and this will give us an opportunity to keep this more focused.

I've started https://github.com/2i2c-org/pilot-hubs/pull/352 to refactor some of the things in this repo. I'm hoping that a combination of code review and refactoring will make this repository easier to understand :)

@damianavila perhaps you could share a few other items (or just update the comment above) with some specific ways that the pilot hubs docs could be improved?

Sure... let me paste here some of my notes about the pilot and pilot-hubs docs.
pilot.2i2c.org (hub admin consumers)

https://pilot.2i2c.org/en/latest/about/infrastructure.html

Each 2i2c Hub has two main interactive interfaces: Jupyter interfaces (Notebook and Lab), and RStudio. Each of them is accessible from your session via /tree, /lab, and /rstudio endpoints in your URL.

Should mention VSCode if there is an icon in the image...

To learn about how the pilot-hubs/ repository works, we recommend checking out the pilot-hubs documentation.

Missing link

https://pilot.2i2c.org/en/latest/about/projects.html
Pangeo pilot should be mentioned first as in the previous section (https://pilot.2i2c.org/en/latest/about/infrastructure.html)

https://pilot.2i2c.org/en/latest/admin/configuration/login.html#

Currently, there are only two supported methods for authorizing regular users

I was expecting also to add users using the helm config file...
If the authz users are manually added, how are you backing up that info?

https://pilot.2i2c.org/en/latest/admin/configuration/culling.html
For the server culling, I know there is some way in binder to detect inactivity, should not be using that to prevent killing a long computation?

There is currently no maximum time limit for a user’s notebook.

This is confusing...

https://pilot.2i2c.org/en/latest/admin/configuration/default-interface.html
The legacy notebook should have a picture as well...
Do not we supppot VSCode?
Theia?
Why not? License, I guess...

https://pilot.2i2c.org/en/latest/admin/howto/support.html
Who is receiving the emails sent to<EMAIL_ADDRESS>
https://pilot.2i2c.org/en/latest/admin/howto/configurator.html

Warning

Is there any validation before submitting the change?

Once you’re at the control panel, you can access the configurator under Services in the top navigation bar. It will ask you to authenticate to the configurator so it knows you are an admin.

So you need to auth twice?

You must click the submit button at the bottom of the form for your changes to take effect!

A green message should help here...

If you are using a custom docker image, you should make sure these interfaces are available in your docker image!

How to prevent that failure or fail graciously...

There’s a ‘interface selector’ in the hub home page, but unfortunately that does not reflect this option. So if your user selects an interface there, they will be put in that interface but just for the first time after they log in.

How do we start with the selected interface?

https://pilot.2i2c.org/en/latest/admin/howto/environment.html

Many popular data science user interfaces installed

VScode, Theia? Dupe from previous comments...

Ask for changes to the base image¶

The default image should be as small as possible

If the new image you use breaks user server starts, or is broken in some other way, you can revert back to the old image by specifying that.

that? Really?

While tempting, do not use !pip install --user  to install packages.

With conda, you can "save" your temp environment as well... not sure about conflicts

https://pilot.2i2c.org/en/latest/admin/howto/nbgitpuller.html
automatically merged sound nice, wondering about issue with the process and if there is any thought about a conflict resolution UI in the future...

https://pilot.2i2c.org/en/latest/admin/howto/create-content.html
This section probably needs more love (development), right?

https://pilot.2i2c.org/en/latest/admin/howto/manage-users.html
Persistence of manually added users? Dupe question IIRC

https://pilot.2i2c.org/en/latest/admin/howto/control-user-server.html
Switch the warnings. Btw, the RTC does not apply here because you are actually impersonating the user...

https://pilot.2i2c.org/en/latest/admin/howto/share-datasets.html
Wondering how the shared-readwrite is implemented (there are some docs ahead)

https://pilot.2i2c.org/en/latest/admin/howto/replicate.html

Your hub-specific configuratior
Your hub template configuration

Any export tools to get the config you may use in your own deployment?

https://pilot.2i2c.org/en/latest/admin/howto/create-billing-account.html
Fix AWS link

Enter the following user emails under ‘New members’<EMAIL_ADDRESS><EMAIL_ADDRESS>These are two of the folks at 2i2c who will help manage your project.

We should use some generic user... or can others be invited by the two of you? (in any case, it is a bottleneck).

https://pilot.2i2c.org/en/latest/user/download-data.html
Interesting tool, jupyter-tree-download (is there a JLab counterpart?)

https://pilot.2i2c.org/en/latest/user/interface.html
VSCode?

I will post my pilot-hub docs notes in the next message on this thread.

pilot-hubs (2i2c eng docs)

https://pilot-hubs.2i2c.org/en/latest/
hubs.yaml should be always links to the repo, IMHO.

You can add a new hub by adding an entry there, and pushing that to github.

Is the CI deploying the new hub somehow??? (I noticed that is the case after looking at the repo...)

https://pilot-hubs.2i2c.org/en/latest/structure.html

People visit our website to learn about us

Our site is not linking the Hub users/admin site?

https://pilot-hubs.2i2c.org/en/latest/howto/configure/hub-config.html

Each JupyterHub can use its own configuration via the hubs.yaml file.

No link

This should be a valid Zero to JupyterHub configuration.

Any validation?

https://pilot-hubs.2i2c.org/en/latest/howto/configure/auth-management.html#
Double "Manage authentication" in ToC, the same for other sections...

https://pilot-hubs.2i2c.org/en/latest/howto/configure/add-hub-domains.html

For this to happen we need to add the new domain as an ingress domain for this hub using the hubs.yaml configuration file.

Ingress domain, interested to know more about this one...

https://pilot-hubs.2i2c.org/en/latest/howto/configure/update-env.html

Currently there is no automated image building - so you will have to manually build & push it after making a change.

This should be automated!

gcloud tool

CLI? Link missing...

https://pilot-hubs.2i2c.org/en/latest/howto/customize/custom-image.html

Only on merge will the hub get the new image.

How does the automatic deployment work? Looked at the repo, I think I have already got it...

https://pilot-hubs.2i2c.org/en/latest/howto/customize/docs-service.html
Why jupyterbook is not listed here?
I should try Nikola with this one!

https://pilot-hubs.2i2c.org/en/latest/howto/customize/configure-login-page.html

The log-in pages are built with the base template at this repository.

Interesting tool!

https://pilot-hubs.2i2c.org/en/latest/howto/operate/index.html#team-process
Endless loop with Add a new hub link ;)

https://pilot-hubs.2i2c.org/en/latest/howto/operate/cmd-access.html
Get access with local machine

https://pilot-hubs.2i2c.org/en/latest/howto/operate/delete-hub.html

Backup the hub database

How? To where?

Delete our Authentication entry. If the hub hasn’t been recreated anywhere, remove the entry from auth0.com

I guess I need access to auth0.com?

https://pilot-hubs.2i2c.org/en/latest/howto/operate/manual-deploy.html

deploy.py deploy

Mmm... name duplication... confusing, IMHO.

Deploys all the hubs in the cluster passed as argument (the hubs listed in hubs.yaml file unde )

Typo

you first need to make sure you are authenticated in all the right places and have the right credentials exported as environmental variables.

How do you know if this is happening?

https://pilot-hubs.2i2c.org/en/latest/howto/operate/test-hub.html

If the deploy.py script is used to deploy all pilot hubs at once, then a failed hub health check will halt all further deployments!

Mmm... that worrisome, how do you handle a problematic automated deployment?

https://pilot-hubs.2i2c.org/en/latest/howto/operate/grafana.html

For example, the pilot hubs are accessible at grafana.pilot.2i2c.cloud.

Ask user and pass

https://pilot-hubs.2i2c.org/en/latest/howto/operate/setup-new-project.html
How does this work with already existing Hubs/orgs?

https://pilot-hubs.2i2c.org/en/latest/topic/cluster-config.html
Are all the clusters deployments inside the same Kube cluster? I think that is the case, but I need to go deeper to check it out...

https://pilot-hubs.2i2c.org/en/latest/topic/defaults.html
hub/values.yaml is https://github.com/2i2c-org/pilot-hubs/blob/master/hub-templates/base-hub/values.yaml

https://pilot-hubs.2i2c.org/en/latest/topic/hub-templates.html

Desn’t let servers run for more than 8h

Typo

Enables outgoing SSH

More info...

https://pilot-hubs.2i2c.org/en/latest/topic/storage-layer.html

The NFS share creator job will be created pre-deploy, run, and cleaned up before deployment proceeds. Ideally, this would only happen once per hub setup - but we don’t have a clear way to do that yet.

Where is this happening in the codebase?

This feature comes from the custom KubeSpawner that the pilot hubs use, that allows providing extra configuration for admin users only.

That link is probably wrong...

Is there any backup on the disk

@choldgraf and @yuvipanda those are notes I have taken last week almost without checking the repos contain. Now that I have started to look at the repos, some things are clearer than before. I still think this is probably good feedback to fix some bugs and improve some sections in the docs.
You may also find some notes not as useful as they should be (because I wrote them for me first)... feel free to disregard those :wink:
@yuvipanda, I will look into your PR, and try to provide useful feedback!!

This is very helpful, @damianavila! I'm slowly responding to these one by one, either here or by fixing things :)

automatically merged sound nice, wondering about issue with the process and if there is any thought about a conflict resolution UI in the future...

See https://jupyterhub.github.io/nbgitpuller/topic/automatic-merging.html for how this is handled.

@damianavila I think I've addressed your questions around manual deployment with this PR: https://github.com/2i2c-org/pilot-hubs/pull/360

@damianavila as part of clearing up the 'hub templates' documentation, I've just removed the 'ephemeral-hubs' hub template in https://github.com/2i2c-org/pilot-hubs/pull/361.

This is very helpful, @damianavila! I'm slowly responding to these one by one, either here or by fixing things :)

Thank you!! I will check your PRs ASAP :wink:

@damianavila what do you think that we should do with this one? I feel like we should either turn this into a short and actionable list, or just close it because it's quite broad and will be hard to act upon

I think that we've made a lot of improvements to the documentation since it was opened, but am not sure what exactly has been addressed here

I agree, most likely the stuff here was already addressed or it is no longer relevant, so closing here.


The at_hash in id_tokens does not follow the spec

The spec for generating an at_hash can be seen on http://openid.net/specs/openid-connect-core-1_0.html#rfc.section.<IP_ADDRESS>
The existing code does not base64 encode the hash and therefore all existing id_token token flows should fail. Any clients that use this flow with anvil currently are breaking the standard.

Coverage increased (+0.007%) to 79.75% when pulling bdde505352e1f10537f96e60d40beb4b29d72ab1 on john-banks:master into a21dd1ff17d4893bb97eb7beb455888afb10eea0 on anvilresearch:master.

Thanks @john-banks! Good catch. The only reason I'm not merging this just yet is to have a chance to look at the client libs that verify at_hash (I think just anvilresearch/connect-js) and make sure we have that updated as well.


Prepare for new release of Jira

Workflow links: https://github.com/atlassian-labs/atlassian-slack-integration-server/actions/runs/6032977025, https://github.com/atlassian-labs/atlassian-slack-integration-server/actions/runs/6032979517, https://github.com/atlassian-labs/atlassian-slack-integration-server/actions/runs/6032981090.

This one will be addressed along with #315.


Stopped working on IOS without any error

It was working properly on both platform till today, I tested it on IOS when click the button to trigger the login process, nothing happens, I started investigating where the bug is, my code still the same since months, I putted some console.log statements before all calls into this func, and the whole block is wrapped with try/catch block.
The problem is after "await RNTwitterSignIn.init(credentials)" nothing get excuted, without any error, or anything in the debug of Xcode.
I have tried everything, Updated the latest patches, rebuilt like 10 times in two iphones and all simulators, still can't know why this is happening, on android working like charm
can anyone help me with that please?

Guys, I will leave it as closed, the bug was I was putting init method inside async/await, on android works fine but on IOS never get resolved, so just delete await keyword before RNTwitterSignIn.init(YOUR credentials), and done.


Add min, max, and average age upon death statistics to deathlogger and frontend

Self explanatory, extended existing death logger to include this. Willing to take on additional statistics additions if anyone has anything they'd like to include.

What are you using to write code? Are you using VSCode?

yeah using vscode, idk why i keep having linting issues

yeah using vscode, idk why i keep having linting issues

Do you have the Go extension installed properly? gofmt should run automatically when you save files.

yeah I think I had a different linter as my default, changed it and works now


[Move-prover] Allow dataflow analysis to compute error output for a block

Motivation
For one of the phases in the bytecode transformation pipeline, it is possible that an analysis computes an error for a block. This PR generalizes the dataflow analysis to account for error output.  Also, this capability is generally useful for analyses we may write in the future.
Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
Existing tests.
Related PRs

@bors-libra r=junkil-park

:pushpin: Commit 37d4d33 has been approved by junkil-park

:hourglass: Testing commit 37d4d33ee79544c0a50664f3303759944d7c8509 with merge a1f9b8211f113b5939bb6e3bc162246c67497d6e...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: junkil-park
Pushing a1f9b8211f113b5939bb6e3bc162246c67497d6e to master...


Add Ruby 2.7.2 and update Ruby versions in travis.yml

Added Ruby 2.7.0 to build matrix.

This looks good to me. Can you squash your commits into one?

Thank you for reviewing! I've pushed the squashed commit.

Great thanks!


SearchInputView is disappearing when rotating the device

I found this issue and I need your help.
When I open the app (does not matter if it is in portrait or landscape mode) everthing is fine

But when I rotate the device, the SearchInputView in the searchview get lost.

I even can type a word and the search is working, but it displays nothing I typed

Any comment?

It never backs to normal once the issue arises

I made further investigations and I discovered this is happening just when using activities. When using fragments the app behaves correctly.
This is a simple project hightighting the problem.
https://drive.google.com/open?id=0BwHzuXXGXF2BRENYRWZWRE1SbXM

Ok, this is getting weird.
Using the demo code
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main2);

        showFragment(new DemoFragment()); //<- HERE I CREATE THE FloatingSearchView
    }

This work as expected, please notice the onCreateView, when changing orientation, is called twice in this scenario
LOADING FRAGMENT THE FIRST TIME

04-22 09:06:26.353 666-666/ni.com.spiralni.myfloatingsearchview D/DemoFragment: onCreateView: demo
04-22 09:06:26.385 666-666/ni.com.spiralni.myfloatingsearchview D/DemoFragment: onResume: demo

CHANGING ORIENTATION

04-22 09:07:20.119 666-666/ni.com.spiralni.myfloatingsearchview D/DemoFragment: onPause: demo
04-22 09:07:20.127 666-666/ni.com.spiralni.myfloatingsearchview D/DemoFragment: onStop: demo
04-22 09:07:20.222 666-666/ni.com.spiralni.myfloatingsearchview D/DemoFragment: onCreateView: demo
04-22 09:07:20.251 666-666/ni.com.spiralni.myfloatingsearchview D/DemoFragment: onCreateView: demo
04-22 09:07:20.270 666-666/ni.com.spiralni.myfloatingsearchview D/DemoFragment: onResume: demo

But If I ensure the fragment does not get replaced if it already exists, using this code
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main2);

        DemoFragment fragment = (DemoFragment) getSupportFragmentManager().findFragmentById(R.id.fragment_container);

        if (fragment == null) {
            showFragment(new DemoFragment()); //<- HERE I CREATE THE FloatingSearchView
        }
    }

This does not work, please notice the onCreateView,  when changing orientation, is called just once in this scenario.
LOADING FRAGMENT THE FIRST TIME

04-22 09:00:48.898 26962-26962/ni.com.spiralni.myfloatingsearchview D/DemoFragment: onCreateView: demo
04-22 09:00:48.919 26962-26962/ni.com.spiralni.myfloatingsearchview D/DemoFragment: onResume: demo

CHANGING ORIENTATION

04-22 09:01:11.983 26962-26962/ni.com.spiralni.myfloatingsearchview D/DemoFragment: onPause: demo
04-22 09:01:12.076 26962-26962/ni.com.spiralni.myfloatingsearchview D/DemoFragment: onCreateView: demo
04-22 09:01:12.100 26962-26962/ni.com.spiralni.myfloatingsearchview D/DemoFragment: onResume: demo

So using an activity, there is no chance the FloatingSearchView is created twice, resulting in the bug.
Any idea?

@spiralni there appears to be some issues to look at on orientation change. Will have a look at it. Thanks for including the project, it speeds up things.
Thanks fro contributing!
Ari

@arimorty any updates on this?

Yeah. Any update?

Any fork?

This issue is presented also when having two fragments (the first one is with search view). Opining the first one, then the second one and returning to the first one leads to the same behavior.

I have exactly the same bug. Can I help with this task?

Also version 2.0.3 works fine without this bug. So the quick workaround is to roll back to it.

I got the same bug :( any luck ?

Hi, I wanted to use this library in a project of mine, but couldn't really use it with this bug. I debugged it and found that the issue was caused by:
File : FloatingSearchView.java
Line : 839
mSearchInput.setTextSize(mQueryTextSize);
If you comment that line out it works again.
Not sure how to fix it so the functionality works again.
I hope this helps.

@spiralni 👍 for providing the sample problem highlighting the issue. The issue is fixed on dev here and will be part of next release (2.1.0)

@spiralni 👍 for providing the sample problem highlighting the issue.


Creating a file association for a code file type in windows explorer assigns the default code icon over available specific icon

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: Version: 1.79.2 (system setup)
OS Version: Windows_NT x64 10.0.19045

Steps to Reproduce:

Assign a file type to VSCode in Windows Explorer (such as .py) that wasn't assigned during install.
File icon assigned to that file type uses the generic code icon at Microsoft VS Code\resources\app\resources\win32\default.ico, eventhough a specific icon for that type is available Microsoft VS Code\resources\app\resources\win32\python.ico .

I think that is the standard Windows behaviour when you tell Windows Explorer to open all .py files with a specific app (e.g. code.exe) every time. But this article (untested by me) looks like it shows how to reassign a type-specific icon.
https://learn.microsoft.com/en-gb/windows/win32/shell/how-to-assign-a-custom-icon-to-a-file-type

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


[YYYY-MM-DD]: [BUG TITLE] cannot read phylip or maf files

Bug Summary
I have a large alignment file of CDS (248 Mb in maf format and 238 Mb in phylip format). I need to remove gaps for using it in codeml, but I get an error with both.
If I don't specify the format, the error is the following:
(venv) (base) antonella@obelix:~/proteins_decont/CDS/fixedCDS$ clipkit output_alignmentwnnA.phy
Format type could not be read.
Please check acceptable input file formats: fasta, clustal, maf, mauve, phylip, phylip_sequential, phylip_relaxed, stockholm
Traceback (most recent call last):
File "/home/antonella/venv/bin/clipkit", line 8, in
sys.exit(main())
File "/home/antonella/venv/lib/python3.9/site-packages/clipkit/clipkit.py", line 220, in main
execute(**process_args(args))
File "/home/antonella/venv/lib/python3.9/site-packages/clipkit/clipkit.py", line 166, in execute
trim_run, stats = run(
TypeError: cannot unpack non-iterable NoneType object
(venv) (base) antonella@obelix:~/proteins_decont/CDS/fixedCDS$ clipkit output_alignmentwnnA.phy -if phylip -co -s nt -l
Traceback (most recent call last):
File "/home/antonella/venv/bin/clipkit", line 8, in
sys.exit(main())
File "/home/antonella/venv/lib/python3.9/site-packages/clipkit/clipkit.py", line 220, in main
execute(**process_args(args))
File "/home/antonella/venv/lib/python3.9/site-packages/clipkit/clipkit.py", line 166, in execute
trim_run, stats = run(
File "/home/antonella/venv/lib/python3.9/site-packages/clipkit/clipkit.py", line 82, in run
alignment, input_file_format = get_alignment_and_format(
File "/home/antonella/venv/lib/python3.9/site-packages/clipkit/files.py", line 32, in get_alignment_and_format
alignment = AlignIO.read(open(input_file_name), file_format.value)
File "/home/antonella/venv/lib/python3.9/site-packages/Bio/AlignIO/init.py", line 389, in read
raise ValueError("More than one record found in handle")
ValueError: More than one record found in handle
(venv) (base) antonella@obelix:~/proteins_decont/CDS/fixedCDS$ clipkit CDSsfinaleswnamenA.maf -if maf -co -s nt -l
Traceback (most recent call last):
File "/home/antonella/venv/bin/clipkit", line 8, in
sys.exit(main())
File "/home/antonella/venv/lib/python3.9/site-packages/clipkit/clipkit.py", line 220, in main
execute(**process_args(args))
File "/home/antonella/venv/lib/python3.9/site-packages/clipkit/clipkit.py", line 166, in execute
trim_run, stats = run(
File "/home/antonella/venv/lib/python3.9/site-packages/clipkit/clipkit.py", line 82, in run
alignment, input_file_format = get_alignment_and_format(
File "/home/antonella/venv/lib/python3.9/site-packages/clipkit/files.py", line 32, in get_alignment_and_format
alignment = AlignIO.read(open(input_file_name), file_format.value)
File "/home/antonella/venv/lib/python3.9/site-packages/Bio/AlignIO/init.py", line 389, in read
raise ValueError("More than one record found in handle")
ValueError: More than one record found in handle
Technical Details

Python version: Run python --version.
Python 3.9.19
ClipKIT version: Run clipkit -v
clipkit 2.3.0

Hi @aebaci,
Thank you for bringing this issue to our attention. We apologize for the inconvenience.
To facilitate resolving this issue, would you be willing to share your input files?
Please feel free to send them an email if you'd like https://jlsteenwyk.com/contact.html.
Best,
Jacob

Hi @aebaci,
Thank you for sending the file via email.
I looked at the input file and noticed that multiple alignment files have been concatenated together (specifically, it looks like the input was generated using something like this command: cat single_gene_alignments* > msa.file).
The input of ClipKIT is a single multiple sequence alignment file at a time; e.g., clipkit single_gene_alignment.
If there is any other confusion, please let me know and I would be happy to help you further.
Cheers,
Jacob


Please add OSX binary

A binary for OSX would be great. Thanks!

Do you have a GOOS/GOARCH combination which you'd like me to add? I suspect GOOS=darwin, but I haven't got an OSX to check the GOARCH. There's at least two, no?

Added darwin with amd64 and 386. Most likely you'll use the amd one. Apparently there's also arm GOOS'es which I'm not supporting at this point for this architecture, but I might add an arm version for Raspberry PIs...
I'm closing the issue, since I think this should resolve it for you. Grab an executable from the releases page :) BR


[SOLVED] ERROR: No value has been specified for property 'manifestOutputDirectory'.

Hi all. I've installed react-native config, successfully linked it, added apply from: project(':react-native-config').projectDir.getPath() + "/dotenv.gradle" line to the app/build.gradle and after that my Android Studio sync fails with ERROR: No value has been specified for property 'manifestOutputDirectory'.. I noticed that this happens only after updagrading gradle to gradle:3.4.0 and gradle-service to distributionUrl=https\://services.gradle.org/distributions/gradle-5.1.1-all.zip. Seems like react-native-config doesn't work with the latest gradle

Seeing this in our own project too (we dont use react-native though). Our errors is:
* What went wrong:
A problem was found with the configuration of task ':*******:processDebugAndroidTestManifest'.
> No value has been specified for property 'manifestOutputDirectory'.

when upgrading to plugin 3.4 /gradle 5.1.1 ... wondering if its an issue with the android plugin. Tried Andoird studio 3.5 and the issue went away so maybe just avoid plugin version 3.4 till they patch it.

I came across this problem in unrelated project, but I found a solution that works for me: disabling Gradle feature "Only sync the active variant". In Android Studio: File > Settings > Experimental > Gradle -> uncheck "Only sync the active variant" checkbox.

@piotrdz Thanks, it works or me too!

@piotrdz, sorry, but Android Studio: File > Settings? I only have File > Others Settings and File > Settings Repository and none of them have Experimental > Gradle.

@rochapablo Are you using macOS? On Mac the settings are under Android Studio -> Preferences.

@piotrdz, yes, I'm using macOS. Founded! thank you.
You know, it did work, but it seems that the error occurs only when you click in Sync (after some change on build.gradle). Even having the error, cleaning the project and rebuilding the error does not happen.
I don't know why this, here I'm having a similar problem. When I Sync I got the error but cleaning and rebuilding the error goes way. But since it's another project, in this case I still have to try the Only sync the active variant option.
Anyway, thank you again for your time.

I came across this problem in unrelated project, but I found a solution that works for me: disabling Gradle feature "Only sync the active variant". In Android Studio: File > Settings > Experimental > Gradle -> uncheck "Only sync the active variant" checkbox.

As long as it seems to be the solution (checked in a new project, it helps) I close the issue

@piotrdz, sorry, but Android Studio: File > Settings? I only have File > Others Settings and File > Settings Repository and none of them have Experimental > Gradle.

After doing that my bundle is not opening automatically from android studio.
can you please help me with that ?

I came across this problem in unrelated project, but I found a solution that works for me: disabling Gradle feature "Only sync the active variant". In Android Studio: File > Settings > Experimental > Gradle -> uncheck "Only sync the active variant" checkbox.

Thanks Dude, you saved my day


class YAMLObject(metaclass=YAMLObjectMetaclass):                               ^ SyntaxError: invalid syntax

Hi,
when I met the OpenCV problem as the author mentioned at the end of readme.md file, I did as the author told us what to do. However, after exporting PYTHONPATH, I have a new problem like this.
Traceback (most recent call last):
  File "/opt/ros/kinetic/bin/rosparam", line 34, in <module>
    import rosparam
  File "/opt/ros/kinetic/lib/python2.7/dist-packages/rosparam/__init__.py", line 63, in <module>
    import yaml
  File "/home/jio/anaconda2/envs/badgr/lib/python3.6/site-packages/yaml/__init__.py", line 374
    class YAMLObject(metaclass=YAMLObjectMetaclass):
                              ^
SyntaxError: invalid syntax

I guess there are some incompatible problems between ROS and Python3.  I am stuck now. Can anyone help me?
Thanks!

Hi,
you can change the name of cv2.so
cd /opt/ros/kinetic/lib/python2.7/dist-packages/
sudo mv cv2.so cv2_ros.so
it works on my computer


Staff Availability Tool

We need a tool to make the management of staff and their availability easier. I'm trying out Float which is good for:

Finding people, looking at their availability
Assigning someone to a project for a full or part day
Assigning someone to multiple projects
Searching for a particular skill
Viewing everyone assigned to a particular project
Searching by availability
Searching by multiple fields (availability, country(ies), skill)
Reports on the number of billable, non-billable, holiday hours and utilisation in a period
Moving someone to another project or splitting a person across multiple projects

See https://wunderuk.floatschedule.com
Feedback welcome.

Interesting, can you share the login?
Reinhard

Am 29.07.2015 um 10:19 schrieb Steve Hunton<EMAIL_ADDRESS>We need a tool to make the management of staff and their availability easier. I'm trying out Float which is good for:
Finding people, looking at their availability
Assigning someone to a project for a full or part day
Assigning someone to multiple projects
Searching for a particular skill
Viewing everyone assigned to a particular project
Searching by availability
Searching by multiple fields (availability, country(ies), skill)
Reports on the number of billable, non-billable, holiday hours and utilisation in a period
Moving someone to another project or splitting a person across multiple projects
See https://wunderuk.floatschedule.com https://wunderuk.floatschedule.com/
Feedback welcome.
—
Reply to this email directly or view it on GitHub https://github.com/wunderkraut/WunderWay/issues/92.

@reinhardweber a login is on its way

We can also add specific skills (D7, D8, analytics, public sector procurement etc) as tags in each user profile and these become searchable. That will help find very specific skills around the group


KAFKA-7287: Set open ACL for old consumer znode path

@junrao PR for 1.1 branch. Please take a look.

@junrao  incase you missed my previous ping.


add permit command

This pr is for permit command.
To permit some user(s) to some dataset(s) access as READER or WRITER or OWNER.

@hirosassa
Hi, this PR is draft.
I'll fix lint and implement "Ask and Confirm" soon.
Please review.

@snowhork Thanks!! I think this functionality is GREATLY needed for BQ admin(s).
Please check CI comments for some formatting issue.

Please add description for this functionality on README

@hirosassa I've fixed!

@snowhork Thanks! Is this PR ready to merge?


vendor: bump runtime-spec version

Fixes #373
Bump runtime-spec version to "5806c35637336642129d03657419829569abc5aa"
Change logs:
5d9aa69 config-linux: Add Intel RDT/MBA Linux support
6f5fcd4 Support for network namespace in windows
06cf899 config-linux: add Intel RDT CLOS name sharing support
f3be7d2 config: clarify source mount
65fac2b Fix camelCasing on idType to align with other Windows spec conventions
da8adc9 incorporating edits from JTerry's feedback
c182ebc meeting: Bump July meeting from the 4th to the 11th
e01b694 config: Add Windows Devices to Schema
81d81f3 docs: Added kata-runtime to implementations
b0700ad Add gVisor to the implementations list
9e459a6 .travis.yml: Get schema dependencies in before_install
692abcb .travis: Bump minimum Go version to 1.9
fd39559 config: Clarify execution environment for hooks
cd9892d config-linux: Drop console(4) reference
e662e5c Linux devices: uid/gid relative to container
74b670e config: Add VM-based container configuration section
cd39042 uidMappings: change order of fields for clarity
2e241f7 specs-go/config: Define RDMA cgroup
9df387e schema/Makefile: fix test
de688f2 config: Fix Linux mount options links
ef008dd glossary: Bump JSON spec to RFC 8259
4e5a137 schema: Completely drop our JSON Schema 'id' properties
70ba4e6 meeting: Bump January meeting from the 3rd to the 10th
8558116 schema: add allowed values for defaultAction
5d9bbad config: Dedent root paragraphs, since they aren't a list entry
e566cf6 version: put master back to -dev
966a58d fix the link to hook

Signed-off-by: Wei Zhang<EMAIL_ADDRESS>

Hi @WeiZhang555 - please could you add in the list of changes this will introduce using the git shortlog as shown here:

https://github.com/kata-containers/community/blob/master/CONTRIBUTING.md#re-vendor-prs

@jodh-intel Oh, thank you for your reminder!

Thanks @WeiZhang555!
lgtm

lgtm


jasmine is not defined when used with jest@27

  ● Test suite failed to run
    ReferenceError: jasmine is not defined
      at Object.<anonymous> (node_modules/jest-mock-console/dist/setupTestFramework.js:9:24)

When using it with<EMAIL_ADDRESS>problem goes away downgrading back to<EMAIL_ADDRESS>

Thank you for bringing that to my attention. Apologies for the delay on responding. I will work on a fix for that now.

This issue for jest@27 appears to be that they have moved away from jasmine for the default test-runner https://jestjs.io/blog/2021/05/25/jest-27#flipping-defaults.
This should only impact the Advanced use case of this library. The work around for now is to also include "testRunner": "jest-jasmine2" in your package.json.
I will leave this issue open for the time being while I look at a longer term resolution that will work with jest-circus

I added that jest-jasmine2 line and my coverage changed dramatically. An hour ago it was 89%, now it's dramatically lower. I can't figure out why. I'm just going to have to remove this tool and see if that's the issue.

Sorry to bother you @bpedersen , but did you ever release a version with this fix on it?

Just pushed version 1.2 which will now check for jest version and modify the describe block as needed.


Add BranchProtectionConfigurationEvent and SecretScanningAlertLocationEvent

--- FAIL: TestProjectsService_DeleteProjectCard (0.00s)
    projects_test.go:611: Projects.DeleteProjectCard returned error: Delete "http://<IP_ADDRESS>:39961/api-v3/projects/columns/cards/1": net/http: HTTP/1.x transport connection broken: http: CloseIdleConnections called

@gmlewis this seems like an intermittent failure unrelated to the changes in this PR. Unable to reproduce it locally. Can we re-run the workflow?

Thank you, @tomfeigin !
Merging.


python36Packages.python-jsonrpc-server: 0.0.1 -> 0.0.2

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/python3.6-python-jsonrpc-server/versions.
meta.description for python36Packages.python-jsonrpc-server is: '"A Python 2 and 3 asynchronous JSON RPC server"'.
Release on GitHub
Compare changes on GitHub

Checks done (click to expand)

built on NixOS

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.0.2 with grep in /nix/store/01vri54s267jly960rgghdvfcy4vwn3r-python3.6-python-jsonrpc-server-0.0.2

directory tree listing: https://gist.github.com/e2ebb8c722481a3c2322e644996ab004

du listing: https://gist.github.com/65c10732b1e43f64c156ddc11fb67b2d

Outpath report (click to expand)

Outpath difference report
32 total rebuild paths
8 package rebuilds
8 x86_64-linux rebuilds
8 i686-linux rebuilds
8 x86_64-darwin rebuilds
8 aarch64-linux rebuilds
First ten rebuilds by attrpath
python27Packages.pyls-isort
python27Packages.python-jsonrpc-server
python27Packages.python-language-server
python36Packages.pyls-black
python36Packages.pyls-isort
python36Packages.pyls-mypy
python36Packages.python-jsonrpc-server
python36Packages.python-language-server

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/01vri54s267jly960rgghdvfcy4vwn3r-python3.6-python-jsonrpc-server-0.0.2 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A python36Packages.python-jsonrpc-server https://github.com/r-ryantm/nixpkgs/archive/cb254b005b0204a0078b853755bf822c57416446.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/01vri54s267jly960rgghdvfcy4vwn3r-python3.6-python-jsonrpc-server-0.0.2
ls -la /nix/store/01vri54s267jly960rgghdvfcy4vwn3r-python3.6-python-jsonrpc-server-0.0.2/bin

cc @mic92 for testing.

Success on x86_64-darwin (full log)
Attempted: python36Packages.python-jsonrpc-server
Partial log (click to expand)
test/__init__.py             0      0   100%
test/test_endpoint.py      120      4    97%
test/test_streams.py        37      0   100%
--------------------------------------------
TOTAL                      423     39    91%
Coverage HTML written to dir htmlcov

========================== 26 passed in 0.32 seconds ===========================
pytestcachePhase
/nix/store/agmmahnm4pyyd4v5rvcn3islzx80xn8a-python3.6-python-jsonrpc-server-0.0.2

Success on aarch64-linux (full log)
Attempted: python36Packages.python-jsonrpc-server
Partial log (click to expand)
test/__init__.py             0      0   100%
test/test_endpoint.py      120      4    97%
test/test_streams.py        37      0   100%
--------------------------------------------
TOTAL                      423     39    91%
Coverage HTML written to dir htmlcov

========================== 26 passed in 0.62 seconds ===========================
pytestcachePhase
/nix/store/bkczd8yxgx2a3yjmn7lgvravzif49w67-python3.6-python-jsonrpc-server-0.0.2

Success on x86_64-linux (full log)
Attempted: python36Packages.python-jsonrpc-server
Partial log (click to expand)
test/__init__.py             0      0   100%
test/test_endpoint.py      120      4    97%
test/test_streams.py        37      0   100%
--------------------------------------------
TOTAL                      423     39    91%
Coverage HTML written to dir htmlcov

========================== 26 passed in 0.31 seconds ===========================
pytestcachePhase
/nix/store/01vri54s267jly960rgghdvfcy4vwn3r-python3.6-python-jsonrpc-server-0.0.2


Move tf broadcasting from randomizer into task, while making it camera/robot agnostic

This is not really that necessary. Will add tf for real robot and camera inside launch files anyways... Closing


Update and convert channel settings to rst

One thing that still needs to be updated in this doc is "dropdown" or "drop-down", wasn't 100% sure what was correct.

As a compound adjective, it would be drop-down. If used as a noun, then dropdown.
I'd be tempted to to leave it out altogether because it looks more like a pop-up than a drop-down; drop-downs typically have a down-arrow that you click. So in this case, you could use channel menu instead of channel drop-down menu.

Thanks @JeffSchering, updated it to drop-down - I left it in there for now because technically there is an arrow beside the channel name, but I'm open to taking it out if other people think it's clearer without "drop-down" there.

@esethna - made the changes except the System Admin one, I don't think there's a guideline for that? And to me System Admin and System Administrator are interchangeable. But if it's something we actually want to do we should add it to guidelines and make a separate ticket since there are many places in the docs and product help text that use Administrator.

Thanks LB. We decided to refer to them as "Admins" in the actual product. But yes, we should add this to the guidelines for docs and then check other places

+1 :)


moving email CSS styles to inline

closes #128

I'm not sure if we can really test this until it's merged in... hopefully this fixes our style woes?

Eh, let’s find out.

…actually, we can test it. I’m going to add an OAuth info to this PR app so you can test.

yasssss

Still need to have this branch be testable with real extracts by email?

@migurski I think we're good, I got that email delivered to me so I think it's good to merge in

K, just waiting for a test email to me as well and then I’ll merge.
How did you get that email delivered from the code on this branch?

I'm honestly not sure? I submitted an extract request on my local build and it sent me an email with updated CSS

Oh, local, of course.


Unify log messages about deleting agent from CV

Unify log messages to make log parsing a bit easier.
Signed-off-by: Karel Srot<EMAIL_ADDRESS>

I want to merge https://github.com/keylime/keylime/pull/982 first, but this change is fine.


Weekend Sprint Review

I'm making this pull request to allow myself to add a comment on it. Next time, please make a pull request and add your project manager as a collaborator and reviewer.

MVP

[X] Answer self-study questions.
[ ] Create a new branch, push changes to that branch, and make a PR from this branch to master.
[ ] Build the HTML and CSS to create the missing navigation and header.
[X] Link the About navigation item to the about.html page.
[X] Color each box.
[X] Build the about page header with respect to home page.
[X] Link the Home navigation item back to the index.html page.
[X] Build the rest of the about page layout to match the design.

Feedback

Good work on multiple commits, Jared, as well as organized CSS classes.
Next time, make a pull request and add your project manager as a reviewer.
Your header needs to be styled like the design file.
I suggest breaking your commits even further into small, descriptive units of change. This might be a little more work at first, but it's important for proper organization in Git!
Also, try not to leave empty styles in your CSS. This is part of general readability/maintainability of code.

Rating (1-3)
1

Hi, Stella. I believe I got everything fixed that needed attention. I also went ahead and completed a few of the stretch problems. Thank you for the feedback - much appreciate!


test(angular.json): update path to fontawesome in test architecture

fixes #5773
Please read and mark the following check list before creating a pull request (check one with "x"):

[x] I read and followed the CONTRIBUTING.md guide.
[x] I read and followed the New Feature Checklist guide.

Short description of what this resolves:
Angular unit tests were not running as the path to fontawesome module was wrong in angular.json

Do I need to do anything else to merge this PR? @ehsanmunna @megalexandre


MSTEST0008: (non-abstract, non-generic) [TestInitialize] not allowed in abstract generic base class (or use case not explicitly forbidden)

Describe the bug

[TestInitialize] is allowed on abstract non-generic types
test inheritance allows multiple [TestInitialize] with one per class, which all run correctly

Steps To Reproduce
public abstract class MyTestBase<T>
{
  protected int a;

  [TestInitialize]
  public void Init() => a = 1; // MSTEST0008
}

[TestClass]
public sealed class MyTest : MyTestBase<int>
{
  private int b;

  [TestInitialize]
  public void Init2() => b = 2;

  [TestMethod]
  public void MyTestMethod()
  {
    Assert.AreEqual(1, a); // ok
    Assert.AreEqual(2, b); // ok
  }
}

Expected behavior

either runtime Exception a la UTA018: Example.Test.MyTest: Cannot define more than one method with the TestInitialize attribute.
or make analyzer value abstract more than generic-ness

Actual behavior
MSTEST0008 gets diagnosed, but unit tests run both Init-Methods; and Init-Methods from abstract generic base classes correctly
Additional context
I guess this would also be the same for TestCleanup.
But not sure how this relates to ClassInitialize/ClassCleanup, since it's also defined in 'this specific test-class' (contrary to AssemblyInitialize/ AssemblyCleanup).

either runtime Exception a la UTA018: Example.Test.MyTest: Cannot define more than one method with the TestInitialize attribute.

This would be problem for Playwright which relies on multiple test setups and teardowns.

MSTEST0008 was buggy (see #3276), we got it fixed.


improve the completion cmd help for generated cli ref md

What this PR does / why we need it:
The help format of the docforge completion is not markdown friendly and the product of docforge gen-cmd-ref for it is not very readable. This PR improves that.

why f and d keys are so close when you git push ...!?


Add 'deploy' in the 'help' list of commands

'deploy' is simply missing from the help

Should probably mention that this PR also adds Kotlin support. ;)

PR title fixed!
That's what happens when you submit a PR during a conf.call :expressionless:


[Enhancement suggestion] Scene muffle percentage

Hi! I love your module, one of my "must-have"! Would it be possible to implement a functionality that would allow to decrease/increase the muffle percentage globally in a scene? Thus, it would be possible to determine scenes with thick walls and scenes with thin walls, as a whole, without having to work wall to wall. Thank you for your work!

This may be done in a future version. The impact of "all sounds being muffled at least some %" is yet to be tested. Albeit this should not affect the cases of:

Direct line to a sound (no muffling) - here maybe a toggle to allow underwater scenes
Ethereal walls that should not muffle or add to muffling


0.5.0 : Model{InvertedHarmonicOscillator}

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/57810
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.5.0 -m "<description of version>" a351ed9ad729a9c013d264f3895f1101a81740f3
git push origin v0.5.0


WIP: Experimental/atomic ref

[ ] Have you provided a meaningful PR description?
[ ] Have you added a test, reproducer or referred to an issue with a reproducer?
[ ] Have you tested your changes locally for CPU and GPU devices?
[ ] Have you made sure that new changes do not introduce compiler warnings?
[ ] If this PR is a work in progress, are you filing the PR as a draft?

I will open a new PR where I am not the reviewer.


include package data in setup.py

this should fix the CI issue in #22

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 83.965%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1241

Relevant Lines:
1478

💛  - Coveralls


Microsoft.Build.Tasks.Copy winerror 1005 with unformatted/raw drive

error MSB4061: The "Copy" task could not be instantiated from "C:\Users\xxx\.nuget\packages\microsoft.build.copyonwrite\1.0.263\build\netstandard2.0\Microsoft.Build.CopyOnWrite.dll".
1>    C:\Program Files\Microsoft Visual Studio\2022\Professional\MSBuild\Current\Bin\amd64\Microsoft.Common.CurrentVersion.targets(4862,5): error MSB4061: System.TypeInitializationException: The type initializer for 'Microsoft.Build.Tasks.Copy' threw an exception. ---> System.ComponentModel.Win32Exception: Failed retrieving volume information for S:\ with winerror 1005

My S:\ drive is going to be a ReFS drive and hold my source, but I am currently using Win11 Canary and don't have DevDrv yet, so the partition is there, but unformatted.
(I had got as far as format S: /devdrv but was told The requested feature is not yet available, stay tuned. ;) )
However, when including the CoW package, I now crash out with winerror 1005.
Diskmanager shows the partition as RAW.
Removing the Drive Letter allowed me to continue building the source (which was on my C: drive)

/cc @AndyGerlicher

Can you create a DevDrive from the UI? I think it's something like Settings App -> System -> Storage -> Disks & Volumes. Not sure if what I'm looking at is current though. I had thought DevDrive was released for testing so I'm surprised by that.
Also, for the error message, do you know why it's trying to check the S: drive? That seems like a somewhat reasonable result if you are trying to copy files from a valid drive to an unformatted drive. But if your build has no reference to S: (and no overridden env variables pointing to S:) and it fails because S:\ happens to exist, that would need to get fixed. Can you check your binlog to see if anything in the build points to that drive?

@AndyGerlicher DevDrive is not available on Canary unfortunately. I missed the window to switch tiers a while back and haven't had the time or energy to rebuild my entire workstation. I do have Dev Home which is interesting and I hope gets some more love.
As it is, it looks like this was a false correlation on my part with regard to removing the drive letter.
The latest version, 1.0.263, appears to work even with the RAW partition having a drive letter.
The issue was actually that I had copied and pasted the line from the readme
<GlobalPackageReference Include="Microsoft.Build.CopyOnWrite" Version="1.0.0" />

, although I had changed GlobalPackageReference to PackageReference to install it normally, but via my Directory.Build.props file.
(It should be noted that the readme doesn't really suggest that you can install it as a normal package at all!)
There isn't a 1.0.0 published so it had resolved 1.0.240 unbeknownst to me. I then changed the version to 1.0.263 that I could see from the nuget package badge

However, it had already loaded the DLL with the older task into memory and that has a problem.
When I removed the drive letter, I had closed and re-opened VS2022 at the same time, to ensure that any static caches in the code cleared (which I had seen from having a quick look at the code). This meant the newer DLL loaded and didn't have the issue anyway.
I think the resolution to this is to change the readme to not have explicit version numbers e.g. use something like 1.0.x to make it clear not to actually copy that in, and perhaps to make it clearer that you can actually use it without CPM/CPV or as an SDK import.
It might also be an idea to un-list the older versions that have bugs/issues? I can see people using this with e.g. SD Cards with linux partitions on them, (like for a PI), which would probably also fail on the older version.
I'm also not sure that checking the root drive is the best mechanism anyway. I would have preferred to mount my ReFS under an existing path and moved my code over to it, so all my existing paths still matched. I'd probably still have to adjust my nuget packages path, but I'm going to have to do that anyway, when I eventually get this to work locally.
I have been using it on my build server successfully though.

For reference the full errors were:
warning NU1603: xxx depends on Microsoft.Build.CopyOnWrite (>= 1.0.0) but Microsoft.Build.CopyOnWrite 1.0.0 was not found. An approximate best match of Microsoft.Build.CopyOnWrite 1.0.240 was resolved.

and
error MSB4061: The "Copy" task could not be instantiated from "C:\Users\xxx\.nuget\packages\microsoft.build.copyonwrite\1.0.240\build\..\lib\netstandard2.0\Microsoft.Build.CopyOnWrite.dll".
error MSB4061: System.TypeInitializationException: The type initializer for 'Microsoft.Build.Tasks.Copy' threw an exception. ---> System.ComponentModel.Win32Exception: Failed retrieving volume information for S:\ with winerror 1005
error MSB4061:    at Microsoft.CopyOnWrite.Windows.NativeMethods.ThrowSpecificIoException(Int32 lastErr, String message) in C:\CoW\lib\Windows\NativeMethods.cs:line 30
error MSB4061:    at Microsoft.CopyOnWrite.Windows.VolumeInfoCache.GetVolumeInfo(VolumePaths volumePaths) in C:\CoW\lib\Windows\VolumeInfoCache.cs:line 148
error MSB4061:    at Microsoft.CopyOnWrite.Windows.VolumeInfoCache.BuildFromCurrentFilesystem() in C:\CoW\lib\Windows\VolumeInfoCache.cs:line 36
error MSB4061:    at Microsoft.CopyOnWrite.CopyOnWriteFilesystemFactory.Create() in C:\CoW\lib\CopyOnWriteFilesystemFactory.cs:line 48
error MSB4061:    at System.Lazy`1.CreateValue()
error MSB4061:    at System.Lazy`1.LazyInitValue()
error MSB4061:    at Microsoft.Build.Tasks.Copy..cctor()
error MSB4061:    --- End of inner exception stack trace ---
error MSB4061:    at Microsoft.Build.Tasks.Copy..ctor()
error MSB4060: The "Copy" task has been declared or used incorrectly, or failed during construction. Check the spelling of the task name and the assembly name.

This clearly shows the error to be in package version 1.0.240, however if you change the version and don't unload and reload, then the older Task is still loaded in memory and will be called from the newer .target file. I think the original error message is actually wrong here, as that is not the location of the DLL in use/in memory, but the updated location it should be using.
It does point to the fact that the VolumeInfoCache should probably have some sort of error handling in it for BuildFromCurrentFilesystem or perhaps VolumeEnumerator should check for a valid filesystem... although both of these are in https://github.com/microsoft/CopyOnWrite/ rather than here...

winerror 1005 is ERROR_UNRECOGNIZED_VOLUME which was added to the CoW package as an ignored error condition in Feb 2023 (CoW package version 0.3.2) and included in the Copy task in Microsoft.Build.CopyOnWrite versions 1.0.242 and later.


UI: Reduce granularity of slider when selecting a timelock in blocks

Currently, the granularity is 1 block, which makes it hard to get the number yout want (e.g.: 6 months) by dragging the slider.
Probably a granularity of 144 blocks would be more enjoyable (ideally, you could still leave the freedom of choosing a different number in the textbox).

Hi @darosior, I'll be happy to work on this issue. Thanks.

Well that was quick. Thanks @jp1ac4!


Reorganize resource labels

We currently repeat the labels section for each resource which is a waste and error prone. We should move them to the _helpers.tpl file as a partial and consume them in the resources themselves from it.

@hagaibarel Can you assign this to me


Regarding additional configuration in creating networks in portainer io

Question:
Hello,
I am trying to create network using portainer.io UI, apart from subnet and gateway, could not able to supply additional configuration such as ip-range, aux-address, scope etc.
Is it possible to pass additional network configurations while creating network?
Thanks,
Mohan

Deploy portainer:develop for our soon the be released version that adds this missing functionality.
Rgds,
Neil Cresswell
On 6/09/2018, at 9:28 PM, MohanrajBabu <notifications<EMAIL_ADDRESS>wrote:
Question:
Hello,
I am trying to create network using portainer.iohttp://portainer.io UI, apart from subnet and gateway, could not able to supply additional configuration such as ip-range, aux-address, scope etc.
Is it possible to pass additional network configurations while creating network?
Thanks,
Mohan
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer_issues_2256&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=0NmrQSChJ-vFPJGvBOhJHtzb_zEmRJueGhVRiqR6GGA&s=Ka69t41t5YOw76NpFc8CGXr8ZA3cgNEM5Q2zXN6HjRA&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AWGrlV1jNl09ZZPOKhXj0MwO7SPFmD5Cks5uYTD0gaJpZM4WdDYL&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=0NmrQSChJ-vFPJGvBOhJHtzb_zEmRJueGhVRiqR6GGA&s=5RzQBvD2-q0tIUXcpkIioQko8_YRO-HDmOqJO8cquBo&e=.

This was added in the implementation of https://github.com/portainer/portainer/issues/2082

@deviantony #2082 explains adding macvlan driver, not what I queried.

@ncresswell Thanks, that would be helpful, in our setup we need to set the iprange.


feat(tutorial): add step 4 content

Related Ticket(s)
#7173
Description
Content for Step 4: How to deploy the files to GitHub Pages
Currently viewable here:
https://carbon-for-ibm-tutorial.s3-web.us-east.cloud-object-storage.appdomain.cloud/deploy-previews/16/index.html
Changelog
New

added content for step 4

Thanks for the review @janchild! I added all your suggestions in the latest commit :)


Happy eyeballs support for Dynamic Forward Proxy (and possibly others)

For this, we need to have

multiple IP addresses per host (right now we roll down to 1 DNS result)
"async" DNS resolution (be able to return v4 and v6 results independantly)
grid style failover logic (possibly pluggable, if we want more than "try first v6, fail over over to v6)

(Notifying @RyanTheOptimist since we'll be collaborating on parts of this.)

After digging into this with @alyssawilk I think we have a plan for how to do this. PR #16913 added an addressList() method to Upstream::HostDescription. This list will be populated by the DNS resolution code and will allow connection logic to potentially use any of the IP addresses to connect.
Figuring where to do the failover logic has been a bit tricky. Simplest would be to do the failover/racing in the Grid but that would prevent the basic connection pools from being able to use this feature. Another thought was to do this racing in the ActiveClient but that seems tricky given the various different ActiveClient implementations. Instead, what we think is the most reasonable path forward is to change DispatcherImpl::createClientConnection(). Instead of returning a Network::ClientConnectionImpl it will instead return something like Network::HappyEyebalssClientConnectionImpl. This will implement the Happy Eyeballs logic and will internally create a Network::ClientConnectionImpl on-demands for each of the addresses in the list that it tries to connect to. Hopefully this proves reasonable!

Instead of returning a Network::ClientConnectionImpl it will instead return something like Network::HappyEyebalssClientConnectionImpl. This will implement the Happy Eyeballs logic and will internally create a Network::ClientConnectionImpl on-demands for each of the addresses in the list that it tries to connect to. Hopefully this proves reasonable!

Do we think we will want this behavior to be configurable? Do we want all callers of this low level API to get this functionality? That would be my main concern.

I think we'll want it on by default, and we can add a knob to configure it off if they want to. If we want to flag it off, I think we'd do so in the dns resolver, since that's where we currently scope down from N hosts to 1 host.

Ah we forgot to tag https://github.com/envoyproxy/envoy/pull/17371 there

Per offline discussion I think we are going to try to get this wired up and some testable form:

@junr03 will add a resolver mode that will return both v4 and v6.
@alyssawilk will do the wiring on the host/connection itself. @alyssawilk on this topic, wdyt in terms of runtime or config? I'm mostly concerned that there may be cases in which people don't want this and they want the old behavior and thus we need config? I could go either way. I guess we could start with runtime (maybe default off at first) and see if anyone complains?

Hm, it's a good question.  I'm inclined to do it as reloadable (default off, as discused) and see if anyone doesn't want it.  I think it makes sense as the default behavior and we can always add a config knob after the fact and use the runtime guard to allow prior behavior while we add the option.


Rename HashOfStringToArrayOfInt type alias to NameToCaptureLocations

The new name actually names the type by its purpose, which makes the code easier
to read.

This type alias lives in the Regexp backend trait.


Modify error handling for timed out signals when connecting.

This commit puts responsibility on anything with a .connect method to raise its own ConnectionTimeoutError, to be then picked up by the DeviceCollector, or not. This simplifies the complex error handling requiring timeouts to be handled at the asyncio level and passed up the called via (now redefined) NotConnected exceptions.
fixes https://github.com/bluesky/ophyd-async/issues/48

New plan:


Add sourcelink support to SDK projects

Fix Issue #1760 .
Changes
Adding common properties to _nupkg.props;
Adding package reference to Microsoft.SourceLink.GitHub
Checklist

[ ] I ran Unit Tests locally.
[X] CHANGELOG.md updated with one line description of the fix, and a link to the original issue if available.

For significant contributions please make sure you have completed the following items:

[ ] Design discussion issue #
[ ] Changes in public surface reviewed

The PR will trigger build, unit tests, and functional tests automatically. Please follow these instructions to build and test locally.
Notes for authors:

FxCop and other analyzers will fail the build. To see these errors yourself, compile localy using the Release configuration.

Notes for reviewers:

We support comment build triggers

/AzurePipelines run will queue all builds
/AzurePipelines run <pipeline-name> will queue a specific build

/AzurePipelines run

/AzurePipelines run

Ok! I will add those ASAP! Give me some minutes!
Thanks for showing that

From: Timothy Mothra<EMAIL_ADDRESS>Sent: Tuesday, April 21, 2020 7:01:10 PM
To: microsoft/ApplicationInsights-dotnet<EMAIL_ADDRESS>Cc: Eddy Nakamura<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [microsoft/ApplicationInsights-dotnet] Add sourcelink support to SDK projects (#1800)
@TimothyMothra requested changes on this pull request.
Hey @eddynakahttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Feddynaka&data=02|01|Eddy.Nakamura%40microsoft.com|f7e34fa84eee457de99008d7e63f80cf|72f988bf86f141af91ab2d7cd011db47|1|0|637231032737387649&sdata=%2Fm8EEGSiMZ5eJC639Igt8472oIVzyY6kJZVVPsrpeE8%3D&reserved=0,
All of this is good but I need you to make one additional change.
The Sourcelink package isn't compatible with our Linux builds.
I need you ta change the  to .
This will unblock Linux builds but will allow our signing to incorporate the SourceLink.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FApplicationInsights-dotnet%2Fpull%2F1800%23pullrequestreview-397707390&data=02|01|Eddy.Nakamura%40microsoft.com|f7e34fa84eee457de99008d7e63f80cf|72f988bf86f141af91ab2d7cd011db47|1|0|637231032737397639&sdata=8lHpeHB2OiW0rxR%2BbBe9oT0hO%2Ff%2BAxv0sve7qBtxHaI%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD3S2ZE6IKXV6DF27CK4QCTRNYJSNANCNFSM4MNS3WTA&data=02|01|Eddy.Nakamura%40microsoft.com|f7e34fa84eee457de99008d7e63f80cf|72f988bf86f141af91ab2d7cd011db47|1|0|637231032737397639&sdata=NV%2F8kxp%2FUjavoKCBUybZ3TspCQxFV5%2BKK9l43gAKSX8%3D&reserved=0.

/AzurePipelines run

/AzurePipelines run

/AzurePipelines run

I would not recommend adding SourceLink to each project individually. It's a pain to then update. Put the reference into a Directory.Build.props file

A few things should be added to a Directory.Build.targets as well:
https://github.com/clairernovotny/DeterministicBuilds


Figure out windows webview2 distribution

For windows users, they must have webview2. This is not so insurmountable.
https://docs.microsoft.com/en-us/microsoft-edge/webview2/concepts/distribution
We can either package webview with the app or send along a bootstrapper.
Other apps have done this before, so maybe we can steal some thoughts from them.
What are people's general thoughts regarding webview2 access on Windows? Should we just fallback to MSHTML if the user doesn't have edge installed?

It looks like the tauri bundler exposes options for embedding webview2 in the installer: https://docs.rs/tauri-utils/1.4.0/tauri_utils/config/enum.WebviewInstallMode.html. We now expose this as part of the bundler config

One remaining issue is that the configuration in Dioxus.toml is not on par with that documented in tauri bundler.
As dioxus uses toml for configuration, the Option<PathBuf> makes no sense in wix.language as we cannot provide a null value in toml. What do you think about reuse tauri_utils::config::WixLanguage, or even the whole tauri_utils::config::BundleConfig directly in our configuration?


Add onCancel handler

@ltk Any idea why coveralls is failing here?

Locally I didn't get errors with npm test:
djmccormick at forge in ~/Projects/microcosm on feature-oncancel
± npm test

><EMAIL_ADDRESS>test /Users/djmccormick/Projects/microcosm
> ava

   127 passed

Looks like a coverage problem in CircleCI. I can take a stab at testing.

Nice, :+1:. Ping me if you're still having issues with Circle/Coveralls and I can take a look.

@djmccormick All good. I'll merge this.

@djmccormick I'll publish a new beta after we sort out >>>>>>> Stashed changes

@ltk I think it was my fault. Let's see what happens next time.


Fix permission denied error upon attempting to init pizzaboxer/wpfui

Previously, running git submodule update --init --recursive would result in this:
Submodule 'wpfui' (git@github.com:pizzaboxer/wpfui.git) registered for path 'wpfui'
Cloning into 'D:/Development/Side Projects/bloxstrap/wpfui'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:pizzaboxer/wpfui.git' into submodule path 'D:/Development/Side Projects/bloxstrap/wpfui' failed
Failed to clone 'wpfui'. Retry scheduled
Cloning into 'D:/Development/Side Projects/bloxstrap/wpfui'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: clone of 'git@github.com:pizzaboxer/wpfui.git' into submodule path 'D:/Development/Side Projects/bloxstrap/wpfui' failed
Failed to clone 'wpfui' a second time, aborting

This was a result of an incorrect submodule url in .gitmodules:
[submodule "wpfui"]
	path = wpfui
	url = git@github.com:pizzaboxer/wpfui.git **bad!**

That's not an incorrect URL, GitHub requires your account to be configured with an SSH private key if you're fetching over SSH. Didn't realize that I accidentally used SSH instead of HTTPS. Sorry.