Safari 12 issue: fields dynamically added at runtime are not positioned correctly

@martinisraelsen commented on Tue Apr 09 2019
When I add a FormLayout/FormItem dynamically, some of the last elements are not always positioned correctly. Resizing the screen will cause the fields to be repositioned to their correct location.
I've seen this happen in Safari on V12 and V13. I have not been able to reproduce the issue on other browsers. Vaadin V10 and V11 doesn't appear to have this bug either. It's possible that it's a Safari bug rather than a Vaadin bug, but then I would expect it to also happen consistently with V10 and V11.
I have attached a Vaadin starter project that has the error. To reproduce the error, run the project with jetty:run and open the app in Safari. Click "Add line" until a row appears without the select field and the button.
Also included a few screenshots etc.

Vaadin-Appstarter.zip

@martinisraelsen commented on Sun Apr 14 2019
Here's an updated app that exhibits both this issue and the layout issue reported in https://github.com/vaadin/flow/issues/4530 (which may or may not be the same).
To reproduce this layout issue, run with jetty, open in Safari. Click "Add User" followed by "Add User". The second time all fields are missing, but will show up if you resize the window or click inside one of the (missing) fields.
Vaadin-Appstarter.zip

@martinisraelsen commented on Wed May 08 2019
@ZheSun88 Perhaps this issue should be logged in https://github.com/vaadin/vaadin-form-layout? Are you able to move it over?

@ZheSun88 commented on Thu May 09 2019
@martinisraelsen Hi, sure I will move it there. :-)

Safari 12 seems to have a bug where the result of getComputedStyle for an element doesn't always accurately reflect values from recently changed CSS variables. Reading the element's offsetWidth causes a style recalc which seems to update them.
https://bugs.webkit.org/show_bug.cgi?id=170708
Source:
https://github.com/Polymer/polymer/blob/164a7cad43c1f4f75b406557f894ac3a4394972a/test/unit/styling-cross-scope-var.html#L940

See also https://github.com/vaadin/vaadin-form-layout/issues/111#issue-447558602

The Safari bug mentioned above (related to getComputedStyle and offsetWidth) doesn't seem to be related to this issue.
This issue seems to be a semi-randomly occurring rendering bug in Safari which seems to be triggered by this animation style in our CSS:
:host {
  animation: 1ms vaadin-form-layout-appear;
}
@keyframes vaadin-form-layout-appear {
  to {
    opacity: 1;
  }
}

This animation serves no visual purpose, but it is used to detect that the element has been rendered (to trigger additional logic) in an animationend event listener. It's basically just a dummy animation that triggers an animationend event when the element is rendered by the browser.
It seems that in Safari this animating of opacity from 1 to 1 causes weird flickering when the components are being added to DOM and sometimes the components get left rendered in wrong place (even though when inspecting in dev tools they seem to be positioned correctly).
Reproducing
Here's a pretty minimal HTML page that reproduces the issue easily:
<!doctype html>

    <link rel="import" href="/bower_components/vaadin-form-layout/vaadin-form-layout.html">
    <link rel="import" href="/bower_components/vaadin-text-field/vaadin-text-field.html">

    <button id="btnAdd">Add</button>

Workaround
As mentioned in #111 you can workaround this issue by adding a style of opacity: 0.99 !important; to the vaadin-form-layout.
Instead of that you can also use e.g. transform: translateX(0); so you don't need to use !important or actually change opacity.
This workaround is best applied to all form layouts by creating a style module, but it can also be added to specific form layouts by adding it just as an inline style (either in an HTML template or in Java via formLayout.getStyle().set("transform", "translateX(0)");)
Fix
It seems that we can fix this in the component by changing the animation CSS. From quick tests it seems that we can just use another CSS property in the animation (e.g. visibility: visible;) instead of opacity: 1; to get this working in Safari without problems and still get the animationend event we need.
I'll do some more testing to check if the visibility: visible; approach for the animation event works consistently in all browsers. If it works fine, I'll commit a fix.

Using visibility: visible; seems to work well otherwise, but it seems that when the developer has set visibility: hidden; on the component, it can flash visible when adding it to the DOM. This doesn't seem to happen with opacity: 1; in the animation even if the developer has set opacity: 0; for the component.
I found that a slightly better way to fix it seems to be just adding !important to the animation rule so it's:
@keyframes vaadin-form-layout-appear {
  to {
    opacity: 1 !important;
  }
}

This actually makes it so that the rule is ignored (by the spec) but it still seems to trigger the animationend even we need (in all browsers).

Declarations in a keyframe qualified with !important are ignored.
– https://developer.mozilla.org/en-US/docs/Web/CSS/@keyframes#!important_in_a_keyframe

"None of the properties interact with the cascade (so using !important on them is invalid and will cause the property to be ignored)."
"properties qualified with !important are invalid and ignored"
– https://drafts.csswg.org/css-animations/#keyframes

Though according to MDN !important rules in keyframes are not ignored in IE11 or Edge, but that doesn't cause an issue for us here since opacity: 1; was good enough for all other browsers atm. except new Safari which has some rendering bug that this seems to solve.
I tested that using this opacity: 1 !important; in the animation triggers the event (even if opacity: 0; is set by some other style) and doesn't cause visual issues in: Chrome 74, Firefox 67, Safari 12.1.1, Edge Canary 76, IE11, Edge 17, iOS 9.3 Safari, iOS 12.2 Safari.

The fix for this has been released in vaadin-form-layout v2.1.3.

The fixed version should be included to next maintenance release of Vaadin 13 and next prerelease of Vaadin 14 but in the meanwhile (if you need this fix right now in a Flow app) you can pin this dependency to your pom.xml to use the latest version with the fix:
<dependency>
    <groupId>org.webjars.bowergithub.vaadin</groupId>
    <artifactId>vaadin-form-layout</artifactId>
    <version>2.1.3</version>
</dependency>


consumer config

@h1alexbel 4 puzzles #55, #56, #58, #61 are still not solved.

@h1alexbel all 4 puzzles are solved here: #55, #56, #58, #61.


Issue 184 integrate benchmark with dask

Resolve #184
Implemented dask for benchmark.
Updated run_benchmark script to accept as input btb.benchmark.challenges as input and btb.benchmark.tuners tunables in order to run targeted benchmarks.

Codecov Report

Merging #185 into master will decrease coverage by 1.06%.
The diff coverage is 73.77%.

@@            Coverage Diff             @@
##           master     #185      +/-   ##
==========================================
- Coverage   94.19%   93.13%   -1.07%
==========================================
  Files          41       42       +1
  Lines        1103     1150      +47
==========================================
+ Hits         1039     1071      +32
- Misses         64       79      +15

Impacted Files
Coverage Δ

btb/benchmark/tuners/__init__.py
83.33% <100%> (+8.33%)
:arrow_up:

btb/benchmark/__init__.py
82.69% <63.63%> (-12.77%)
:arrow_down:

btb/benchmark/tuners/hyperopt.py
78.37% <78.37%> (ø)

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 49e346d...6b4441d. Read the comment docs.


Update SUSE CaaS Platform entry

Update URL and description.

Deploy preview for landscape ready!
Built with commit 4ae7795acd2a8f281b2a8f21c1a5ed9fb286d383
https://deploy-preview-445--landscape.netlify.com

LGTM - tested website.

Looks great, thank you.

Dan, wow, that was ultra-fast! Thanks!


Hover colours on chapter and verse numbers in selector

When you touch over the chapter and verse numbers in the Android app, notice how the colour changes to something slightly darker when your finger is touching each button. It would be nice to reproduce this in the PWA.

@rpmargetts I believe the background color comes from ui.button.book-grid { background-color }.  How are you determining the color for the touch? Are you just adjusting the HLS of the background-color ?


[Foundation] replaceSubrange in the cases of immutable and mutable backing stores should recalculate length per the reference backing store change and not the length of the replacement

Explanation:
This addresses a failure in Data.replaceSubrange that would inappropriately funnel to a replacement method on backing stores that encompass objective-c stores for either mutable or immutable backing where the length would not be re-calculated per the backing store's length but instead be the replacement region length.
Scope:
This only applies to runtime behavior of the Data.replaceSubrange family of functions
Radar (and possibly SR Issue):
rdar://problem/33098202
https://bugs.swift.org/browse/SR-5449
Risk:
Medium to low risk; the swift backing store appropriately handles most cases but in the cases of bridged Data from objective-c the calculation was clearly incorrect. No one should be relying on this behavior.
Testing:
Additional unit tests were added to validate the correctness of this patch.

@swift-ci please test

changes lgtm

@tkremenek gah I clicked merge on the wrong PR; this should have gone through the additional process before merging. Do I need to do anything additional here?

If @parkera is fine with the change this still seems in scope for Swift 4.

Looks good to me.


Support bindings failures

This PR includes support for bindings analysis failures to ensure the build can continue.
This fixes zeit/ncc#334 where the fallback path analysis will cause a failure that isn't a runtime failure.

Codecov Report

Merging #15 into master will decrease coverage by 0.09%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master      #15     +/-   ##
=========================================
- Coverage   71.73%   71.64%   -0.1%
=========================================
  Files           8        8
  Lines         736      737      +1
=========================================
  Hits          528      528
- Misses        208      209      +1

Impacted Files
Coverage Δ

src/asset-relocator.js
77.69% <0%> (-0.2%)
:arrow_down:

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 2c2aa78...faf81a8. Read the comment docs.


add ioBroker.alarm sources-dist.json

Sorry but I dont understood the issues from the adapter checker.

[E502] "admin/img/info-big.png" not found, but selectID.js used in index_m.html

when you use selectID,js you need to also provide the mentioned file and this one is missing.

[E503] "iob_npm.done" found in repo! Please add it to .gitignore

delete the file iob_npm.done from Github or all to gitignore file. This comes because you copied and install folder from iobroker and is not allowed to be in the git or npm package!!

[E603] NO "## Changelog" found in README.md

Add a "## Changelog" headline/section to the readme. This is used in Admin

[E604] No "## License" found in README.md

Add a "## License" headline/section to the readme. This is used in Admin

[W113] Adapter should support compact mode

You implemented comact mode, but io-package.json says "compact: false"

Hi Apollon77,
to E502--> thats not clear for me, it is a script that comes with the admin admin adapter. I thing the problem comes from  the script himself?
to W113--> The compact mode is not so easy. All async things like setState and so on are very critical and im not a professinal coder. You said "You implemented comact mode", how can I prevent this? Comment out the "compact: false" in ther io-package?
The rest is done...

Actually I planned "alarm" word for exactly such functionality and it should be sometimes important, like scenes, javascript, devices....
Adapter must work :)

@GermanBluefox Can you provide infos for E502?
@misanorot Normally compact mode works as when you start the adapter normally. Yiu just need to make sure that all timouts, intervals and servers and other resources are cleared in "unload" correctly.
In fact simply enable compact mode for your iobroker machine and enable it for your instance and test if anything wors as usual

@bluefox
For me it is okay when I rename my little project? When you wish. I think your idea is much much bigger than my. For me it is also okay when the adapter stay out of iobroker sources. You have to decide...

@GermanBluefox Can you provide infos for E502?
@misanorot Normally compact mode works as when you start the adapter normally. Yiu just need to make sure that all timouts, intervals and servers and other resources are cleared in "unload" correctly.
In fact simply enable compact mode for your iobroker machine and enable it for your instance and test if anything wors as usual

Is it possible thats a little problem with v3 admin?
I think in V2 the image is here https://github.com/ioBroker/ioBroker.admin/blob/a937e0388244d18f11eaa156a0240d7c4210c2a9/www/img/info-big.png
In the selectID.js I found this ...
https://github.com/ioBroker/ioBroker.admin/blob/9d4d9303f087c8a5e40cb2263d7415828625c3bc/src/lib/js/selectID.js#L157
and this...
https://github.com/ioBroker/ioBroker.admin/blob/9d4d9303f087c8a5e40cb2263d7415828625c3bc/src/lib/js/selectID.js#L1147
the picture is in V3 here:
https://github.com/ioBroker/ioBroker.admin/blob/master/src/img/info-big.png
I am not sure if the src is correct?

The stuff from arc will be available in the www dir after building. So path should work. Check your local installed version ... there you see where Iris in the final paths

Automated adapter checker
ioBroker.alarm

[E502] "admin/img/info-big.png" not found, but selectID.js used in index_m.html

@GermanBluefox Can you please tell more exact what the adapter dev needs to do here?

@GermanBluefox Can you please tell more exact what the adapter dev needs to do here?

This file https://github.com/ioBroker/ioBroker.admin/blob/master/src/img/info-big.png must be checked in in admin/img/info-big.png, because it used in selectID.js

RE-CHECK!

Automated adapter checker
ioBroker.alarm

[ ] :heavy_exclamation_mark: [E502] "admin/img/info-big.png" not found, but selectID.js used in index_m.html
[ ] :eyes: [W113] Adapter should support compact mode
[ ] :eyes: [W400] Cannot find "alarm" in latest repository

Add comment "RE-CHECK!" to start check anew

I don't know what you mean, sorry. Please help

Add created a Pull Request

Add created a Pull Request

oh my god, so simple......

How is the status? Would Bluefox like the name "Alarm" now? Or why can't the adapter be included in the source list?

RE-CHECK!

Automated adapter checker
ioBroker.alarm

:thumbsup: No errors found

[ ] :eyes: [W113] Adapter should support compact mode
[ ] :eyes: [W400] Cannot find "alarm" in latest repository

Add comment "RE-CHECK!" to start check anew


Error occurs when i try to do az ssh.... 'Profile' object has no attribute 'get_msal_token'

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az ssh vm Extension Name: ssh. Version: 0.1.5.
Errors:
The command failed with an unexpected error. Here is the traceback:
'Profile' object has no attribute 'get_msal_token'
Traceback (most recent call last):
  File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 657, in execute
    raise ex
  File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 691, in _run_job
    result = cmd_copy(params)
  File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/usr/local/Cellar/azure-cli/2.30.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/Users/shipmnts/.azure/cliextensions/ssh/azext_ssh/custom.py", line 21, in ssh_vm
    _do_ssh_op(cmd, resource_group_name, vm_name, ssh_ip,
  File "/Users/shipmnts/.azure/cliextensions/ssh/azext_ssh/custom.py", line 48, in _do_ssh_op
    cert_file, username = _get_and_write_certificate(cmd, public_key_file, None)
  File "/Users/shipmnts/.azure/cliextensions/ssh/azext_ssh/custom.py", line 58, in _get_and_write_certificate
    _, certificate = profile.get_msal_token(scopes, data)
AttributeError: 'Profile' object has no attribute 'get_msal_token'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az ssh vm --resource-group {} --vm-name {}

Expected Behavior
Environment Summary
macOS-11.5.1-x86_64-i386-64bit, Darwin 20.6.0
Python 3.10.0
Installer: HOMEBREW

azure-cli 2.30.0

Extensions:
ssh 0.1.5
interactive 0.4.4

Additional Context

Please update to the latest version of ssh.


KPS4 Does not take into account the initial reel speed

The set initial reel speed from the settings is not taken into account by the find_steady_state!() function of the KPS4. Adding this would allow for better tuning tests.

On this line should this not then be
res1, res2  = vcat(reduce(vcat, res1_), [s.l_tether, s.set.v_reel_out]), vcat(reduce(vcat, res2_),[0,0])

Why do you think this would be useful?

I want to compare the simulation with some real measurements. These might be approximately stable, but I have not yet found cases where the reel speed is 0. Allowing to initialise with a value will allow for better force comparisons.


Add gentree config

Codecov Report

Merging #2 (f6745fd) into master (bdbc5f5) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master        #2   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           95        95
=========================================
  Hits            95        95

Impacted Files
Coverage Δ

tree.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Add profile for Wildfly remote

-Use new version: version.wildfly: 8.2.1

Add profile:  test-wildfly-remote

Thanks for PR..
Merged after fixing travis build..


Update README to note that migrate:fresh is in 5.5

Notes in Requirements section that the migrate:fresh command is included by default in Laravel 5.5, with a link to the Laravel News post announcing this.

Thanks!


[#1] Add deploy folder with instructions about its usage

Hey, is this PR missing something? It's just showing changes in the README. Could you take a look pls? Thx!

Hey @cirocosta, no, this pull does not contain any plugin inside the deploy folder.
We'd need a demo/sample plugin to include inside, like a hello-world or something

Got it, thxx!


[NEW]: Wyre, once worth $1.5 Billion, is Now Shutting Down

Date
2023-01-03
Description
On Friday, employees received worse news: They were told via email the company was basically shutting down, as nearly all its employees were being let go.  Wyre did not respond to emails requesting comment, but CEO Ioannis Giannaros told Axios via email that the company is “still operating but will be scaling back to plan our next steps.”
Links
https://sfstandard.com/business/this-crypto-startup-was-worth-1-5-billion-its-now-said-to-be-shutting-down/
Agreement

[X] I irrevocably agree to release my contribution under the CC-BY 3.0 License

Attribution name
Alan
Personal link
No response

Thanks! https://web3isgoinggreat.com/?id=crypto-payments-platform-wyre-to-shut-down-or-scale-back


Various recompilation errors

I wrote this test program to recompile all tables of a font: https://gist.github.com/madig/221321d2343bfc3f4531511067d6be04.
It does not try all the tables because they don't seem to have to_owned_table() and I haven't yet figured out what to do about it.
Run like:
$ for f in /usr/share/fonts/**/*.ttf
       target/release/mess-with-ot $f
   end

It finds various errors like:
Reading font from: "/usr/share/fonts/google-noto-vf/NotoSans-Italic[wght].ttf"
thread 'main' panicked at src/main.rs:162:14:
Cannot compile table: BuilderError { tag: Tag(GPOS), inner: PackingFailed(Table packing failed with 21 overflows) }

Reading font from: "/usr/share/fonts/google-noto-vf/NotoSans[wght].ttf"
thread 'main' panicked at src/main.rs:162:14:
Cannot compile table: BuilderError { tag: Tag(GPOS), inner: PackingFailed(Table packing failed with 54 overflows) }

Reading font from: "/usr/share/fonts/google-noto-vf/NotoSansEthiopic[wght].ttf"
thread 'main' panicked at src/main.rs:162:14:
Cannot compile table: BuilderError { tag: Tag(GPOS), inner: PackingFailed(Table packing failed with 16592 overflows) }

Reading font from: "/usr/share/fonts/google-noto-vf/NotoSerif-Italic[wght].ttf"
thread 'main' panicked at src/main.rs:162:14:
Cannot compile table: BuilderError { tag: Tag(GPOS), inner: PackingFailed(Table packing failed with 61 overflows) }

Reading font from: "/usr/share/fonts/google-noto-vf/NotoSerif[wght].ttf"
thread 'main' panicked at src/main.rs:162:14:
Cannot compile table: BuilderError { tag: Tag(GPOS), inner: PackingFailed(Table packing failed with 68 overflows) }

Reading font from: "/usr/share/fonts/khmer-os-system-fonts/KhmerOS_sys.ttf"
thread 'main' panicked at src/main.rs:42:14:
Cannot compile table: BuilderError { tag: Tag(name), inner: ValidationFailed(Validation error:
"Unhandled platform/encoding id pair: (1, 20)"
in: Name.name_record[16]
  NameRecord.string

) }

The last one is a good reminder, there are a number of encodings we do not yet support for name records. That should have an issue.
The others are mildly surprising, and I'll investigate when I have a bit of time. :)


Not compatible with build --nolegacy_external_runfiles

Thanks for the work on this project! It's so great but I'm encountering an issue blocking me to continue with it.
Issue:

When enabling build --nolegacy_external_runfiles, buildifier-prebuilt still looks for external/buildifier_darwin_arm64/file/buildifier

Reproduction:

Have a simple Bazel WorkSpace, the following steps will take this one from rules_python as the example
Change WORKSPACE lines 64-72 as instructions in line 63
Integrate buildifier-prebuilt with the legacy-workspace-file method
Create a rule for running buildifier
Run "bazel run //:buildifier.check". it works.
Choose one of the below two steps to operate:
6-1. Add build --nolegacy_external_runfiles into the file .bazelrc; I added it in the first line. Then run bazel run //:buildifier.check agan.
or
6-2. Run "bazel run //:buildifier.check --nolegacy_external_runfiles"
It does not work (no fixes found) now.
(Optional) Add arguments --run_under="bash -x" and --sandbox_debug and run again for more details.

Expectation:

buildifier-prebuilt do compatible with build --nolegacy_external_runfiles

Is the solution here to be using the shell snippet that you copy paste from bazel? should be fine to update if so

No solution was provided from me 😅
But if you run the buildifier in the way mentioned as step 8 above, you can see that the line does readlink fails because the first dir is external whereas it maybe should be ../ because with the --nolegacy_external_runfiles the /external folder will not be created anymore.
++ readlink external/buildifier_darwin_arm64/file/buildifier

can you add a repro case here? I can't repro this in our project. what version of bazel + buildifier prebuilt are you on

or better yet a PR with a failing test

The version of bazel and buildifier-prebuilt used in the reproduction steps in the start of this thread is 6.3.1 and 6.1.2

I found that it can be fixed on my laptop after changing the first .path to .short_path in this line, either specifying nolegacy_external_runfiles or not.
Should I create a PR of that, or any suggestion?

worth a try with a PR but I think that was just changed to support windows

https://github.com/keith/buildifier-prebuilt/pull/77


[stable/graylog] Fix services nodePorts, initContainer env, kubectl

What this PR does / why we need it:
Fix nodePort for TCP and UDP services. Add support for environment variables in init Containers. Update kubectl version.
Special notes for your reviewer:
None.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test

/lgtm


nanopolish_call_methylation.cpp missing include zlib.h

Hello,
FYI
while trying to compile the last tarball release (0.12.0) and I found that nanopolish_call_methylation.cpp is missing zlib.h header
see:
In file included from src/nanopolish_call_methylation.cpp:26:
src/nanopolish_call_methylation.cpp:459:11: error: 'gzFile' does not name a type
  459 | KSEQ_INIT(gzFile, gzread)
      |           ^~~~~~
src/nanopolish_call_methylation.cpp:459:11: error: 'gzFile' was not declared in this scope
  459 | KSEQ_INIT(gzFile, gzread)
      |           ^~~~~~
src/nanopolish_call_methylation.cpp: In function 'int ks_getuntil2(kstream_t*, int, kstring_t*, int*, int)':
src/nanopolish_call_methylation.cpp:459:1: error: 'kstream_t' {aka 'struct __kstream_t'} has no member named 'f'
  459 | KSEQ_INIT(gzFile, gzread)
      | ^~~~~~~~~
src/nanopolish_call_methylation.cpp:459:19: error: 'gzread' was not declared in this scope; did you mean 'pread'?
  459 | KSEQ_INIT(gzFile, gzread)
      |                   ^~~~~~
src/nanopolish_call_methylation.cpp: In function 'int ks_getc(kstream_t*)':
src/nanopolish_call_methylation.cpp:459:1: error: 'kstream_t' {aka 'struct __kstream_t'} has no member named 'f'
  459 | KSEQ_INIT(gzFile, gzread)
      | ^~~~~~~~~
src/nanopolish_call_methylation.cpp:459:19: error: 'gzread' was not declared in this scope; did you mean 'pread'?
  459 | KSEQ_INIT(gzFile, gzread)
      |                   ^~~~~~
src/nanopolish_call_methylation.cpp: At global scope:
src/nanopolish_call_methylation.cpp:459:11: error: 'gzFile' was not declared in this scope
  459 | KSEQ_INIT(gzFile, gzread)

regards
Eric

That was fast :) thanks for the report.

I've added the include, can you see if it fixes the problem in your build?
I have to also fix an issue with the ARM build, once I do that I'll tag v0.12.1.

already fixed on my build.
but not in the same location line 8 instead of line 25 ;-)
and it fix the build


editorconfig and omnisharp.json guidance

Hi, I've got formatting rules in onmisharp.json and in editorconfig. I don't know what the preferred path is, where there is overlap or if one should be removed.
Can you please provide some guidance on how to use these two configuration files?

omnisharp.json predated the editorconfig support. However, editorconfig is natively supported by the Roslyn C# compiler and is the preferred source of configuration for formatting and code style. Here are a few links for more information.

editorconfig documentation
example editorconfig
formatting options

I have similar issue with C# dev kit. I have my editorconfig setup as:
# EditorConfig is awesome: https://EditorConfig.org

# top-most EditorConfig file
root = true

# Don't use tabs for indentation.
[*]
indent_style = space
# (Please don't specify an indent_size here; that has too many unintended consequences.)

# Code files
[*.{cs,csx,vb,vbx}]
indent_size = 4
insert_final_newline = true
charset = utf-8-bom

# XML project files
[*.{csproj,vbproj,vcxproj,vcxproj.filters,proj,projitems,shproj}]
indent_size = 2

# XML config files
[*.{props,targets,ruleset,config,nuspec,resx,vsixmanifest,vsct}]
indent_size = 2

# JSON files
[*.json]
indent_size = 2

# Powershell files
[*.ps1]
indent_size = 2

# Shell script files
[*.sh]
end_of_line = lf
indent_size = 2

# Dotnet code style settings:
[*.{cs,vb}]

# IDE0055: Fix formatting
dotnet_diagnostic.IDE0055.severity = warning

# Sort using and Import directives with System.* appearing first
dotnet_sort_system_directives_first = true
dotnet_separate_import_directive_groups = false
# Avoid "this." and "Me." if not necessary
dotnet_style_qualification_for_field = false:refactoring
dotnet_style_qualification_for_property = false:refactoring
dotnet_style_qualification_for_method = false:refactoring
dotnet_style_qualification_for_event = false:refactoring

# Use language keywords instead of framework type names for type references
dotnet_style_predefined_type_for_locals_parameters_members = true:suggestion
dotnet_style_predefined_type_for_member_access = true:suggestion

# Suggest more modern language features when available
dotnet_style_object_initializer = true:suggestion
dotnet_style_collection_initializer = true:suggestion
dotnet_style_coalesce_expression = true:suggestion
dotnet_style_null_propagation = true:suggestion
dotnet_style_explicit_tuple_names = true:suggestion

# Non-private static fields are PascalCase
dotnet_naming_rule.non_private_static_fields_should_be_pascal_case.severity = suggestion
dotnet_naming_rule.non_private_static_fields_should_be_pascal_case.symbols = non_private_static_fields
dotnet_naming_rule.non_private_static_fields_should_be_pascal_case.style = non_private_static_field_style

dotnet_naming_symbols.non_private_static_fields.applicable_kinds = field
dotnet_naming_symbols.non_private_static_fields.applicable_accessibilities = public, protected, internal, protected_internal, private_protected
dotnet_naming_symbols.non_private_static_fields.required_modifiers = static

dotnet_naming_style.non_private_static_field_style.capitalization = pascal_case

# Non-private readonly fields are PascalCase
dotnet_naming_rule.non_private_readonly_fields_should_be_pascal_case.severity = suggestion
dotnet_naming_rule.non_private_readonly_fields_should_be_pascal_case.symbols = non_private_readonly_fields
dotnet_naming_rule.non_private_readonly_fields_should_be_pascal_case.style = non_private_readonly_field_style

dotnet_naming_symbols.non_private_readonly_fields.applicable_kinds = field
dotnet_naming_symbols.non_private_readonly_fields.applicable_accessibilities = public, protected, internal, protected_internal, private_protected
dotnet_naming_symbols.non_private_readonly_fields.required_modifiers = readonly

dotnet_naming_style.non_private_readonly_field_style.capitalization = pascal_case

# Constants are PascalCase
dotnet_naming_rule.constants_should_be_pascal_case.severity = suggestion
dotnet_naming_rule.constants_should_be_pascal_case.symbols = constants
dotnet_naming_rule.constants_should_be_pascal_case.style = constant_style

dotnet_naming_symbols.constants.applicable_kinds = field, local
dotnet_naming_symbols.constants.required_modifiers = const

dotnet_naming_style.constant_style.capitalization = pascal_case

# Static fields are camelCase and start with s_
dotnet_naming_rule.static_fields_should_be_camel_case.severity = suggestion
dotnet_naming_rule.static_fields_should_be_camel_case.symbols = static_fields
dotnet_naming_rule.static_fields_should_be_camel_case.style = static_field_style

dotnet_naming_symbols.static_fields.applicable_kinds = field
dotnet_naming_symbols.static_fields.required_modifiers = static

dotnet_naming_style.static_field_style.capitalization = camel_case
dotnet_naming_style.static_field_style.required_prefix = s_

# Instance fields are camelCase and start with _
dotnet_naming_rule.instance_fields_should_be_camel_case.severity = suggestion
dotnet_naming_rule.instance_fields_should_be_camel_case.symbols = instance_fields
dotnet_naming_rule.instance_fields_should_be_camel_case.style = instance_field_style

dotnet_naming_symbols.instance_fields.applicable_kinds = field

dotnet_naming_style.instance_field_style.capitalization = camel_case
dotnet_naming_style.instance_field_style.required_prefix = _

# Locals and parameters are camelCase
dotnet_naming_rule.locals_should_be_camel_case.severity = suggestion
dotnet_naming_rule.locals_should_be_camel_case.symbols = locals_and_parameters
dotnet_naming_rule.locals_should_be_camel_case.style = camel_case_style

dotnet_naming_symbols.locals_and_parameters.applicable_kinds = parameter, local

dotnet_naming_style.camel_case_style.capitalization = camel_case

# Local functions are PascalCase
dotnet_naming_rule.local_functions_should_be_pascal_case.severity = suggestion
dotnet_naming_rule.local_functions_should_be_pascal_case.symbols = local_functions
dotnet_naming_rule.local_functions_should_be_pascal_case.style = local_function_style

dotnet_naming_symbols.local_functions.applicable_kinds = local_function

dotnet_naming_style.local_function_style.capitalization = pascal_case

# By default, name items with PascalCase
dotnet_naming_rule.members_should_be_pascal_case.severity = suggestion
dotnet_naming_rule.members_should_be_pascal_case.symbols = all_members
dotnet_naming_rule.members_should_be_pascal_case.style = pascal_case_style

dotnet_naming_symbols.all_members.applicable_kinds = *

dotnet_naming_style.pascal_case_style.capitalization = pascal_case

# error RS2008: Enable analyzer release tracking for the analyzer project containing rule '{0}'
dotnet_diagnostic.RS2008.severity = none

# IDE0073: File header
dotnet_diagnostic.IDE0073.severity = warning
file_header_template = Licensed to the .NET Foundation under one or more agreements.\nThe .NET Foundation licenses this file to you under the MIT license.\nSee the LICENSE file in the project root for more information.

# IDE0035: Remove unreachable code
dotnet_diagnostic.IDE0035.severity = warning

# IDE0036: Order modifiers
dotnet_diagnostic.IDE0036.severity = warning

# IDE0043: Format string contains invalid placeholder
dotnet_diagnostic.IDE0043.severity = warning

# IDE0044: Make field readonly
dotnet_diagnostic.IDE0044.severity = warning

# CSharp code style settings:
[*.cs]
# Newline settings
csharp_new_line_before_open_brace = none
csharp_new_line_before_else = false
csharp_new_line_before_catch = false
csharp_new_line_before_finally = false
csharp_new_line_before_members_in_object_initializers = false
csharp_new_line_before_members_in_anonymous_types = false
csharp_new_line_between_query_expression_clauses = false

# Indentation preferences
csharp_indent_block_contents = true
csharp_indent_braces = false
csharp_indent_case_contents = true
csharp_indent_case_contents_when_block = true
csharp_indent_switch_labels = true
csharp_indent_labels = flush_left

# Prefer "var" everywhere
csharp_style_var_for_built_in_types = true:suggestion
csharp_style_var_when_type_is_apparent = true:suggestion
csharp_style_var_elsewhere = true:suggestion

# Prefer method-like constructs to have a block body
csharp_style_expression_bodied_methods = false:none
csharp_style_expression_bodied_constructors = false:none
csharp_style_expression_bodied_operators = false:none

# Prefer property-like constructs to have an expression-body
csharp_style_expression_bodied_properties = true:none
csharp_style_expression_bodied_indexers = true:none
csharp_style_expression_bodied_accessors = true:none

# Suggest more modern language features when available
csharp_style_pattern_matching_over_is_with_cast_check = true:suggestion
csharp_style_pattern_matching_over_as_with_null_check = true:suggestion
csharp_style_inlined_variable_declaration = true:suggestion
csharp_style_throw_expression = true:suggestion
csharp_style_conditional_delegate_call = true:suggestion

# Space preferences
csharp_space_after_cast = false
csharp_space_after_colon_in_inheritance_clause = true
csharp_space_after_comma = true
csharp_space_after_dot = false
csharp_space_after_keywords_in_control_flow_statements = true
csharp_space_after_semicolon_in_for_statement = true
csharp_space_around_binary_operators = before_and_after
csharp_space_around_declaration_statements = do_not_ignore
csharp_space_before_colon_in_inheritance_clause = true
csharp_space_before_comma = false
csharp_space_before_dot = false
csharp_space_before_open_square_brackets = false
csharp_space_before_semicolon_in_for_statement = false
csharp_space_between_empty_square_brackets = false
csharp_space_between_method_call_empty_parameter_list_parentheses = false
csharp_space_between_method_call_name_and_opening_parenthesis = false
csharp_space_between_method_call_parameter_list_parentheses = false
csharp_space_between_method_declaration_empty_parameter_list_parentheses = false
csharp_space_between_method_declaration_name_and_open_parenthesis = false
csharp_space_between_method_declaration_parameter_list_parentheses = false
csharp_space_between_parentheses = false
csharp_space_between_square_brackets = false

# Blocks are allowed
csharp_prefer_braces = true:silent
csharp_preserve_single_line_blocks = true
csharp_preserve_single_line_statements = true

# warning RS0037: PublicAPI.txt is missing '#nullable enable'
dotnet_diagnostic.RS0037.severity = none

[src/CodeStyle/**.{cs,vb}]
# warning RS0005: Do not use generic CodeAction.Create to create CodeAction
dotnet_diagnostic.RS0005.severity = none

[src/{Analyzers,CodeStyle,Features,Workspaces,EditorFeatures, VisualStudio}/**/*.{cs,vb}]

# IDE0011: Add braces
csharp_prefer_braces = when_multiline:warning
# NOTE: We need the below severity entry for Add Braces due to https://github.com/dotnet/roslyn/issues/44201
dotnet_diagnostic.IDE0011.severity = warning

# IDE0040: Add accessibility modifiers
dotnet_diagnostic.IDE0040.severity = warning

# CONSIDER: Are IDE0051 and IDE0052 too noisy to be warnings for IDE editing scenarios? Should they be made build-only warnings?
# IDE0051: Remove unused private member
dotnet_diagnostic.IDE0051.severity = warning

# IDE0052: Remove unread private member
dotnet_diagnostic.IDE0052.severity = warning

# IDE0059: Unnecessary assignment to a value
dotnet_diagnostic.IDE0059.severity = warning

# IDE0060: Remove unused parameter
dotnet_diagnostic.IDE0060.severity = warning

# CA1822: Make member static
dotnet_diagnostic.CA1822.severity = warning

# Prefer "var" everywhere
dotnet_diagnostic.IDE0007.severity = warning
csharp_style_var_for_built_in_types = true:warning
csharp_style_var_when_type_is_apparent = true:warning
csharp_style_var_elsewhere = true:warning

[src/{VisualStudio}/**/*.{cs,vb}]
# CA1822: Make member static
# Not enforced as a build 'warning' for 'VisualStudio' layer due to large number of false positives from https://github.com/dotnet/roslyn-analyzers/issues/3857 and https://github.com/dotnet/roslyn-analyzers/issues/3858
# Additionally, there is a risk of accidentally breaking an internal API that partners rely on though IVT.
dotnet_diagnostic.CA1822.severity = suggestion

note the new line rulesets

but it doesn't use it


Update Storage libraries for accurate released packages

To resolve https://github.com/Azure/azure-sdk-for-net/issues/22704

@tamram @CamSoper @jsquire @scottaddie Could I get someone to review this? I don't believe I have permissions to add reviewers.

Can someone also merge this if this PR is good enough? I don't have merge permissions on this repo. Thanks!


(59) feature: file explorer for "Open Project" dialog

This pull-request adds a file explorer to the "Open Project" dialog
which allows the users to navigate through their filesystem just like a
file manager, instead of being required to enter the absolute path.
The explorer's path is synced with the input's path, so they both work
at the same time.
📹  Quick demo

Holy sh**!!! @mdibaiee this is sooo coool!!! I'll test it tomorrow! 😄

When opening the open file dialog nothing happens until I write a "/":

Also, syntax highlighting is broken:

Haven't tested it more thoroughly, could you fix those things?
Great job!!!! It is awesome!! 😄  Thank you!

@NickSeagull about the issues, you are right I had noticed the first issue, but I couldn't figure out why it was happening, the problem is the first time the function fsTree is run, doesDirectoryExists "/home/mahdi" returns False, while if you re-trigger the function by pressing backspace in the input, it works just fine. I have to investigate why that is the case, I guess it might be related to the second issue, too, will take a look :D :+1:

7d74167c57150de215a07f57660efffe12cc3d78 fixes the first issue (having to type something to start working, and loading last project correctly)

d7d6f94 fixes code highlighting (JS hack required)


Is it possible to do a write only SCP user?

I have need of a shell that will allow a user to connect and upload a file, but I don't want that user to be able to download (or list) any other files. I don't see anything in the documentation about this.

scponly doesnt do anything to manage read/write permissions.  The
ssh applications already do that just fine.  If you use scponly, be
aware that good old unix file permissions are still doing the work
of protecting your files.


Not ask confirmation after running "Kubernetes: Delete"

It is better to ask end user double confirm whether they want to execute "delete" command.

This is only an issue if we are using the active document as indicating what to delete.  In previous discussions we decided we didn't want to do this.  So we should fix this by ignoring the active document during delete.


Must use 8.3 format with cmd /c when concatenating args

VSCode Version: 1.3.0
OS Version: Win 10, 14367

Steps to Reproduce:

Setup a tasks.json with "cmd" and tasks that it calls to invoke an exe that requires additional params passed in args. Examples of one that works using 8.3 naming and one that doesn't using long paths below.

{
    // See https://go.microsoft.com/fwlink/?LinkId=733558
    // for the documentation about the tasks.json format
    "version": "0.1.0",
    "command": "cmd",
    "args": ["/C"],
    "isShellCommand": true,
    "isBuildCommand": false,
    "suppressTaskName": true,
    "showOutput": "always",
    "tasks": [
        {
         "taskName": "Task (one lame)",
         "args":[
             "C:\\PROGRA~2\\MICROS~1.0\\Common7\\IDE\\devenv.exe",
             "/?"
         ],
         "taskName": "Task (two better but broken)",
         "args":[
             "C:/Program Files (x86)/Microsoft Visual Studio 14.0/Common7/IDE/devenv.exe",
             "/?"
         ]
        }
    ]
}

Here are some additional details based on what I'm actually trying to do. These are not easily run as they are using local paths on my machine, you need to setup Arduino IDE and have an Arduino Zero. So below is the output for each of the linked examples "Verify (Build)" task.
While one of these examples does work in Code and the cmd prompt it doesn't unblock me as I have another task with a different command that also passes a lot of info as params, "Upload (Deploy to board)". It is failing as it isn't getting the paths in some of those args right even though they visibly appear so. Per the first example below, at least on Windows, something is different in how paths are interpreted when run by Code vs Windows. I've tried a lot of permutations of long and short paths with different escaping but not found one that works yet for this command.
tasks-buildtasklongwshortworks (does not work in command prompt, works in VS Code)
running command> cmd /C C:/Program Files (x86)/Arduino/arduino-builder.exe -compile -logger=machine -hardware C:\PROGRA~2\Arduino\hardware -hardware C:\Users\mgoodner\AppData\Local\ARDUIN~1\packages -tools C:\PROGRA~2\Arduino\TOOLS-~1 -tools C:\PROGRA~2\Arduino\hardware\tools\avr -tools C:\Users\mgoodner\AppData\Local\ARDUIN~1\packages -built-in-libraries C:\PROGRA~2\Arduino\LIBRAR~1 -libraries D:\Source\Sketchbook\libraries -fqbn=arduino:samd:arduino_zero_edbg -ide-version=10608 -build-path d:\Source\Sketchbook\Blink.vscode -warnings=none -prefs=build.warn_data_percentage=75 -verbose d:\Source\Sketchbook\Blink\Blink.ino
tasks-buildtaskshortwshortworks (Works in both command prompt and VS Code)
running command> cmd /C C:\PROGRA~2\Arduino\ARDUIN~1.EXE -compile -logger=machine -hardware C:\PROGRA~2\Arduino\hardware -hardware C:\Users\mgoodner\AppData\Local\ARDUIN~1\packages -tools C:\PROGRA~2\Arduino\TOOLS-~1 -tools C:\PROGRA~2\Arduino\hardware\tools\avr -tools C:\Users\mgoodner\AppData\Local\ARDUIN~1\packages -built-in-libraries C:\PROGRA~2\Arduino\LIBRAR~1 -libraries D:\Source\Sketchbook\libraries -fqbn=arduino:samd:arduino_zero_edbg -ide-version=10608 -build-path d:\Source\Sketchbook\Blink.vscode -warnings=none -prefs=build.warn_data_percentage=75 -verbose d:\Source\Sketchbook\Blink\Blink.ino

Could you try the following. You actually don't need to specify cmd and the command if isShellCommand is true. You could do:
{
   "command": "C:/Program Files (x86)/Microsoft Visual Studio 14.0/Common7/IDE/devenv.exe"
   "isShellCommand": true,
   ...
}

which then knows better about escaping and how to pass that correctly to the OS.

[Fixed in new terminal runner]


Requirements do not contain version numbers

The list of requirements in the README does not contain version numbers as noted in https://github.com/openjournals/joss-reviews/issues/464
I'll list the version numbers from the current docker container as versions which are known to work. We should probably note that other versions might work as well but are not guaranteed to. We should also discuss some way to keep this listing up to date.

Required Software:

[ ] Jellyfish
[ ] Spades
[ ] bowtie2
[ ] NCBI-Blast+
[ ] Samtools
[ ] Bedtools
[ ] GNU R
[ ] Ghostscript
[ ] Python
[ ] Perl

Required Perl modules:

[ ] Moose
[ ] Log::Log4Perl
[ ] Term::ProgressBar
[ ] Graph
[ ] IPC::Run
[ ] File::Which


Added German translations

This pull request is in relation to #618 and adds some German translations.

Fixed missing exports


[dev.icinga.com #13865] PHP 5.5 dependency issues on Ubuntu 16.04, we should use 5.6.

This issue has been migrated from Redmine: https://dev.icinga.com/issues/13865
Created by fretb on 2017-01-03 22:31:59 +00:00
Assignee: (none)
Status: New
Target Version: (none)
Last Update: 2017-01-03 22:31:59 +00:00 (in Redmine)

Php 5.5 has dependency issues on Ubuntu 16.04 in combination with the ondrej ppa. The dependency issues are concerning php-apcu and php-yac. As icingaweb2 is php 5.6 compatible, and 5.6 is present in the ondrej ppa, I've made a PR.
https://github.com/Icinga/chef-icinga2/pull/69

Hello,
I tried to find php5.X in xenial, but can't. Do you sure that it's right version? I can find only php7 in xenial.
http://packages.ubuntu.com/search?suite=xenial&arch=any&searchon=names&keywords=php5

My pull request (https://github.com/Icinga/chef-icinga2/pull/246) should resolve this ticket

@cyberflow Closing in favor of PR #246
👍
Will publish a new cookbook version soon.


Add alpine-based build

Image size is ~175MB instead of ~660MB.

Nice, @cap10morgan! Do you see any downside switching over to this distro?

The biggest downside is that it's different so downstream users will need to adjust. I would propose something along the lines of what the upstream java image did:
Build clojure:lein-[version]-alpine images and push those to the Docker Hub alongside the ubuntu-based ones for now.
That will give people the ability to opt-in to alpine if they want to. At some point in the future a switch could be announced ahead of time where the ubuntu-based image either gets dropped or moved to a clojure:lein-[version]-ubuntu tag and the alpine-based one takes over the clojure:lein-[version] tag.
Seem reasonable?

@cap10morgan sounds good to me. Do you want to be the maintainer for the alpine variant?

Sure, I can do that.

On Mar 30, 2016, at 2:16 PM, Paul Lam<EMAIL_ADDRESS>wrote:
@cap10morgan sounds good to me. Do you want to be the maintainer for the alpine variant?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

Could you update the info in the Dockerfile, and I've merged your other PR (thanks!), could you rebase that here too? Thanks!

@Quantisan How's that?

@cap10morgan perfect! thank you!

Cool, thanks! I issued this PR against the official-images repo: https://github.com/docker-library/official-images/pull/1587

:+1:
I was going to refactor out some commands into a shell file so that the 2 versions can share some code

OK. Would you like me to close that PR until you do?

nah, you're right to get it out there ASAP

On 3 Apr 2016, at 00:20, Wes Morgan<EMAIL_ADDRESS>wrote:
OK. Would you like me to close that PR until you do?
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub https://github.com/Quantisan/docker-clojure/pull/20#issuecomment-204811791


[Bug]Accident count by severity - missing TEXT field from API

Currently Hard coded
Screenshots

Currently all data returned as on long string (29/4)
Data should be broken into 4 frgaments (main content text, secondary text 1, secondary text 2, secondary text 3)
Should be implemented after API is updated

export interface ILocationMeta {
  location_info: {
    resolution: string;
    road1: number;
    road_segment_name: string;
  };
  location_text: string;
};

check out accident_count_by_severity in mock widgets data
data structure is a following:
data: {
        items: {
          severity_light_count: 81,
          severity_fatal_count: 1,
          severity_severe_count: 7,
          start_year: 2015,
          end_year: 2019,
          total_accidents_count: 89,
        },
      },

All data comes from API out of text field
Should be implemented after API is updated

partiality solved by #242


fix mysql truncate error

can not create auth.User fixtures because the username lenght filed it is too long!

That absolutely makes sense -- a similar change was already included in a previous version, so the issue is already fixed by now. Sorry for the long wait!

Thank you for your efforts! :) Keep coding.


UriData.uri.data returns not the same object

main() {
  UriData uriData = new UriData.fromString("Some data");
  print(uriData); // data:,Some%20data
  print(uriData.uri.data); // data:,Some%20data
  print(uriData.uri.data == uriData); // false
}

Souldn't uriData.uri.data return the same uriData object?
For UriData == operator see #28683

The UriData class doesn't override operator==, so you are comparing the objects for identity.
The Uri class doesn't cache the UriData object, so it returns a new one every time. It could, but that would mean having an extra field on all Uri objects that only gets used in a minuscule number of cases.
Not impossible, though - we could add a _CachingDataUri class extending _Uri that has a UriData field storing the value, and use that when creating a Uri from a UriData.
It's not something we promise in general, but it might be worthy of an optimization for special cases.


It should be possible to call find_package(HPX) multiple times

https://github.com/STEllAR-GROUP/tutorials uses a project structure with the main project calling find_package(HPX) and subprojects also calling find_package(HPX), which leads to errors because of e.g. hpx::boost being created twice (https://circleci.com/gh/STEllAR-GROUP/tutorials/121).
I think this is a reasonable use-case to support. @aurianer what do you think? This should be as easy as
if(TARGET hpx::boost)
 ...
  add_library(hpx:boost INTERFACE IMPORTED)
  ...
endif()

yep totally !


[ci skip] adding user @claytonparnell

Hi! This is the friendly automated conda-forge-webservice.
I've added user @claytonparnell as instructed in #24.
Merge this PR to add the user. Please do not rerender this PR or change it in any way. It has [ci skip] in the commit message to avoid pushing a new build and so the build configuration in the feedstock should not be changed.
Please contact conda-forge/core to have this PR merged, if the maintainer is unresponsive.
Fixes #24

@conda-forge-admin, please ping conda-forge/core

the owner of this feedstock has been unresponsive

You mean on a 2 day old PR? That's an unreasonably short period to expect someone (especially a single maintainer) to see something. Basically everyone is doing maintenance here as a volunteer.
That said, it's great that you're interested in helping out! Let's give this a few more days for @raybellwaves to have a say, otherwise I'll happily merge it for you.


Rate observations for inflation bond

Created rate observations (and corresponding rate observation functions) which store start index value.
This work is required by #784

Renamed the classes to avoid the name "Bond" as they are not really bond specific.


configure prettier & eslint

I've added configs for prettier & eslint with a ts integration, along with husky and lint-staged.
In addition to extending 'eslint:recommended, react/recommendedand @typescript-eslint/recommended I've included most rules available in the eslint-typescript as warnings. I'm fairly sure we don't need all of them. My thought is that we can disable them by process of elimination or set ones we find important to error level.
I've also added devDependencies for eslint-filtered-fix and eslint-nibbl which allows you to fix eslint rules by rule, instead of fixing everything fixable at once.
I haven't run prettier over the src directory in this MR since I don't want the entire git history being overwritten due to formatting changes. I think there's a way to format it which lets us keep the line history somehow.

These seems to be an error when installing the dependencies.
librephotos-docker-frontend-1  | There might be a problem with the project dependency tree.
librephotos-docker-frontend-1  | It is likely not a bug in Create React App, but something you need to fix locally.
librephotos-docker-frontend-1  |
librephotos-docker-frontend-1  | The react-scripts package provided by Create React App requires a dependency:
librephotos-docker-frontend-1  |
librephotos-docker-frontend-1  |   "babel-eslint": "9.0.0"
librephotos-docker-frontend-1  |
librephotos-docker-frontend-1  | Don't try to install it manually: your package manager does it automatically.
librephotos-docker-frontend-1  | However, a different version of babel-eslint was detected higher up in the tree:
librephotos-docker-frontend-1  |
librephotos-docker-frontend-1  |   /usr/src/app/node_modules/babel-eslint (version: 10.1.0)
librephotos-docker-frontend-1  |
librephotos-docker-frontend-1  | Manually installing incompatible versions is known to cause hard-to-debug issues.
librephotos-docker-frontend-1  |
librephotos-docker-frontend-1  | If you would prefer to ignore this check, add SKIP_PREFLIGHT_CHECK=true to an .env file in your project.
librephotos-docker-frontend-1  | That will permanently disable this message but you might encounter other issues.
librephotos-docker-frontend-1  |
librephotos-docker-frontend-1  | To fix the dependency tree, try following the steps below in the exact order:
librephotos-docker-frontend-1  |
librephotos-docker-frontend-1  |   1. Delete package-lock.json (not package.json!) and/or yarn.lock in your project folder.
librephotos-docker-frontend-1  |   2. Delete node_modules in your project folder.
librephotos-docker-frontend-1  |   3. Remove "babel-eslint" from dependencies and/or devDependencies in the package.json file in your project folder.
librephotos-docker-frontend-1  |   4. Run npm install or yarn, depending on the package manager you use.
librephotos-docker-frontend-1  |
librephotos-docker-frontend-1  | In most cases, this should be enough to fix the problem.
librephotos-docker-frontend-1  | If this has not helped, there are a few other things you can try:
librephotos-docker-frontend-1  |
librephotos-docker-frontend-1  |   5. If you used npm, install yarn (http://yarnpkg.com/) and repeat the above steps with it instead.
librephotos-docker-frontend-1  |      This may help because npm has known issues with package hoisting which may get resolved in future versions.
librephotos-docker-frontend-1  |
librephotos-docker-frontend-1  |   6. Check if /usr/src/app/node_modules/babel-eslint is outside your project directory.
librephotos-docker-frontend-1  |      For example, you might have accidentally installed something in your home folder.
librephotos-docker-frontend-1  |
librephotos-docker-frontend-1  |   7. Try running npm ls babel-eslint in your project folder.
librephotos-docker-frontend-1  |      This will tell you which other package (apart from the expected react-scripts) installed babel-eslint.
librephotos-docker-frontend-1  |
librephotos-docker-frontend-1  | If nothing else helps, add SKIP_PREFLIGHT_CHECK=true to an .env file in your project.
librephotos-docker-frontend-1  | That would permanently disable this preflight check in case you want to proceed anyway.
librephotos-docker-frontend-1  |
librephotos-docker-frontend-1  | P.S. We know this message is long but please read the steps above :-) We hope you find them helpful!
librephotos-docker-frontend-1  |
librephotos-docker-frontend-1  | npm ERR! code ELIFECYCLE
librephotos-docker-frontend-1  | npm ERR! errno 1
librephotos-docker-frontend-1  | npm ERR<EMAIL_ADDRESS>start: `react-scripts start`
librephotos-docker-frontend-1  | npm ERR! Exit status 1
librephotos-docker-frontend-1  | npm ERR!
librephotos-docker-frontend-1  | npm ERR! Failed at the<EMAIL_ADDRESS>start script.
librephotos-docker-frontend-1  | npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
librephotos-docker-frontend-1  |
librephotos-docker-frontend-1  | npm ERR! A complete log of this run can be found in:
librephotos-docker-frontend-1  | npm ERR!     /root/.npm/_logs/2022-03-05T20_50_01_590Z-debug.log
librephotos-docker-frontend-1 exited with code 1

Deleting the node_modules folder and the package-lock.json did not work. I used the containerized dev environment, where "npm install --legacy-peer-deps" is called to install the dependencies. Could that be the reason or did I miss a step?

i forgot to remove babel-eslint since it's not used in the final .eslintrc.js config. I removed it from the dependencies. I also added a .nvmrc file to freeze the node version (v12.22.5 is my current default and that seems to work)

just updated the dangling comma rule since it was complaining about them in imports as well.

Got a similar issue for eslint.
Got over it by bumping the react-script to 4.0.3 and downgrading eslint to 7.11.0. Now it's complaining about the config. I would guess that I have to bump react-script to version 5.0.0 in order for it to work with eslint 8.10.0. Will try that tomorrow :)

Hmm, I'm going to test the stuff in the container and get back to you. Doesn't seem fair that you have to debug my PR. Should I close this PR and open a new one when it all works properly?

It's fine :D The problem is mostly caused by old dependencies. You can just continue to work in this pr :) Upgrading the version to react-scripts 5.0.0 worked. But now we have this issue in fomantic-ui: https://github.com/fomantic/Fomantic-UI/issues/2027

Somebody opened up a pr, but it never got merged. But we could link to the pr git repo instead of the npm package.

I'm fiddling about with the dependencies, splitting deps from devDeps and getting the lint and builds to work properly. Once I've got that up and running I'll have to test if everything still works as expected.

I managed to get it going by removing the "semantic-ui-css"<EMAIL_ADDRESS>dependency and including the css via a CDN as described in their docs: https://react.semantic-ui.com/usage/#option-2-cdn-no-bundler
I wasn't sure that the users would appreciate a CDN dependency, but I noticed that there's also a unpkg dependency to leaflet.css.
I'm looking at patching it it via patch-package to remove the CDN dependency.

I fixed the fomantic-ui-css issue using patch-package.

I updated the prettier config to match the current code style as good as I could, but as far as I can tell the style isn't 100% in line everywhere. With the current configuration the new style will be applied whenever a file is changed with lint-staged. In my experience this isn't ideal. When someone submits a new PR, the changes will be mixed with those of the prettier config, making reviews difficult. I couldn't find a way to apply the new prettier config without loosing all git-blame information. One option would be to rebase this commit further into the history, but I'm not sure that that's a good idea either.

Installation of the dependencies and the changes that were needed in the dockerfiles works :+1:
I have now a new issue: When I navigate to localhost:3000, I only get a blank page.
I had this issue here https://github.com/facebook/create-react-app/issues/11779
Forwarding /ws in the proxy and upgrading the connection and setting "WDS_SOCKET_PORT=0" in the environment variables like here, made the error go away. but I still have only a blank page :sweat_smile: Does this only happen in the containerized environment?
I think formatting the codebase in one commit, would be a good enough solution. git blame does not work directly, but I still see in the history, who made the actual change.

I only get a blank page.

I'm seeing it too now. There's some funky stuff going on with the routing. Once logged in it's directing from root to login and back again on every hard refresh. I'm going to continue to chip away at it until I can get the frontend/Dockerfile to build and run locally.

I think formatting the codebase in one commit, would be a good enough solution. git blame does not work directly, but I still see in the history, who made the actual change.

I guess that's an option, but the inline-git-blame will show the formatting commit by default. I'm not sure how important it is to keep this information at this point though. I'm with you on formatting it in one go though.

Looks good :+1:


Updated to the new file format (pt-BR)

Hello!
Here it is! The new version of the lang file.
I hope it don't have too much errors. I will be fixing it from time to time.
Use it! =)

Cheers :)


Make library more easily used by other

From babel-polyfill:

This will emulate a full ES2015+ environment and is intended to be used in an application rather than a library/tool.

If you are looking for something that won't modify globals to be used in a tool/library, checkout the transform-runtime plugin.

When I was using roar-mongo I was getting errors about babel-polyfill only being allowed once.
This change uses the transform-runtime plugin instead of babel-polyfill

Can confirm 0.3.0-alpha is working in a client project. It throws warnings while being webpack'd, but that's a different issue.


Context route in manifest is not honoured

This has been observed by code inspection.

This is not an issue due until this enhancement request has been fulfilled: https://github.com/cloudfoundry/cli/issues/1159


build_token_enforcer_tokenizer_data takes too long on some tokenizer

When using the function build_token_enforcer_tokenizer_data on the bloom tokenizer (https://huggingface.co/bigscience/bloom), it takes a really long time to finish.
The problem comes from this loop : https://github.com/noamgat/lm-format-enforcer/blob/7cfb693495e9ba3305e230cc62e05b383b6c717a/lmformatenforcer/tokenizerprefixtree.py#L72
Indeed one decoded token in the bloom tokenizer have length 600 and several have a length superior to 100 (compared to 16 maximum for llama for example). So the double loop is really long to complete

A PR that will soon be merged allows limiting the max string length in this
section, will improve performance
On Fri, Feb 16, 2024, 13:16 Gautier Solard @.***> wrote:

When using the function build_token_enforcer_tokenizer_data on the bloom
tokenizer, it takes a really long time to finish.
The problem comes from this loop :
https://github.com/noamgat/lm-format-enforcer/blob/7cfb693495e9ba3305e230cc62e05b383b6c717a/lmformatenforcer/tokenizerprefixtree.py#L72
Indeed one decoded token in the bloom tokenizer have length 600 and
several have a length superior to 100 (compared to 16 maximum for llama for
example). So the double loop is really long to complete
—
Reply to this email directly, view it on GitHub
https://github.com/noamgat/lm-format-enforcer/issues/73, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKFA2AQ6KXB6I5JD42GPXTYT45XBAVCNFSM6AAAAABDL3M4LOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGEZTQMZZGEYDQMQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Preference file implementation using viper

now user can provider a prefernce file where s(he) can mention
what controllers to generate.

Coverage remained the same at 46.678% when pulling ab1b83e67afb8634f9046331364d79c04017e811 on surajssd:preferencefile_viper into 4b3094d8acb5af5c676b92cdcd7530c925372841 on kubernetes-incubator:master.

This will need an update to docs for this to be merged in. No idea what's happening here.
A better commit message too please detailing the new command-line function being added.

In regards to 'GlobalProfile' I haven't seen this feature other than https://github.com/kubernetes-incubator/kompose/pull/155
Can an issue be opened giving an outline as to why we need this feature as well as a tutorial, documentation, examples on how to use it?
As well as a [WIP] tag being added to this PR? I'm not too sure that we've ironed out all the features here for a global profile.

I feel as though this feature is over-engineering something simple. Why would we need both a yaml file for converting docker-compose as well as a separate viper-specific file for conversion?
Seems like a hack to fix something that needs to be refactored within kompose. Specifically, something like adding labels as to what @kadel did. Or introducing better conversion methods.

Coverage remained the same at 47.007% when pulling cc944f42b9578e20b558638038587aa167cca427 on surajssd:preferencefile_viper into 2ba88312e169634f71d4aedf910c8dba888ea003 on kubernetes-incubator:master.

I feel as though this feature is over-engineering something simple. Why would we need both a yaml file for converting docker-compose as well as a separate viper-specific file for conversion?

this is basically to do somthing similar to .kube/config. See full discussion at https://github.com/kubernetes-incubator/kompose/issues/39

Seems like a hack to fix something that needs to be refactored within kompose. Specifically, something like adding labels as to what @kadel did. Or introducing better conversion methods.

Labels in kompose is to overcome what docker-compose lacks but this is more of influence functionality of kompose. The things that can happen from command line can be influenced from another file.
So you can see this is only under kompose convert. kompose up and down are still done using defaults.

I'd suggest @ngtuna having a look at my comments for this PR. Personally I'm against such a feature as we want to keep Kompose nice and simple / straight-forward. But that's just me. In regards to the changes I've updated my comments ^^.

This preference file still confuses me a lot :-(
I think that we should outline hot it should work before jumping more into implementation.

The way this behaves is also strange :-(
If i set KOMPOSE_CONFIG where I have 'default' section I would expect that it will be use as default if I don't set any profile.

@kadel I totally agree, docs + spec for a good foundation before any sort of implementation.

Lets go back to discussion in #39, so we have everything in one place and figure out what we wan't to do.
Than we can go back to this PR with clear goal.

Let's close this PR for now until it's a WIP again :)


fix: Fix test signal server port collisions

copilot:all

Current dependencies on/for this PR:

main

PR #3295   👈

This comment was auto-generated by Graphite.


attribute: Object type changes

A specification can be now created without 'rest_name'
'subtype' is now applicable for attribute type 'object'
When 'type' is 'object' user can optionally select another entity as 'subtype'. 'subtype' will be set with value of 'entity_name' of entity being referred to.

Sample generated code:
timers.spec
{
    "attributes": [],
    "children": [],
    "model": {
        "allowed_job_commands": null,
        "create": null,
        "delete": true,
        "description": "This is a dummy description",
        "entity_name": "Timer",
        "extends": [],
        "get": true,
        "package": null,
        "resource_name": "timers",
        "rest_name": null,
        "root": null,
        "template": null,
        "update": true,
        "userlabel": "Timer"
    }
}

Attribute using the above entity as subtype:
        {
            "allowed_chars": null,
            "allowed_choices": null,
            "autogenerated": false,
            "channel": null,
            "creation_only": false,
            "default_order": false,
            "default_value": null,
            "deprecated": false,
            "description": null,
            "exposed": true,
            "filterable": true,
            "format": null,
            "max_length": null,
            "max_value": null,
            "min_length": null,
            "min_value": null,
            "name": "timer1",
            "orderable": true,
            "read_only": false,
            "required": false,
            "subtype": "Timer",
            "transient": false,
            "type": "object",
            "unique": false,
            "uniqueScope": null,
            "userlabel": "Timer1"
        }

Spec-director UI screenshots:

@natabal can you pls provide your feedback on this proposal

@killanch If the user choose the type as object then subtype must be available as one of entities!

@natabal can you pls review? added validation for subtype, not mandatory if type is either list or object.


next step after FreeACS installation

Hi,
I was able to install FreeACS with all components on Centos 7.5 + MySQL 8 (it is a Linode VM).
So far so good! :)
We have an small installation with 20- ONTs ZTE-F660 that I want to start configuring using TR069.
Do I have to define any king of Unit Type at GUI ?
Which is the default user/pass to configure on the devices ?
Regards,
Jose

Jarlah is right. Set TR069 in discovery mode
discovery.mode = true
then new units will be available in web panel.
In discovery mode you not need user and pass for devices. Devices pass it at first request

There has been made a great deal of improvements to the install process. We now have deb and rpm packages. And docker images.
I suggest download the rpms from the latest release and install them in centos. But disable the old services with systemctl disable freeacs-[module]. And you might need to Port some custom config to /etx/default/freeacs-[module]

Aaaaand now I have added install script for CentOS. See wiki.


Migrate from CircleCI to Github Actions

┆Issue is synchronized with this Jira Task by Unito

Unit tests and linting merged, but not required for merging -- we're evaluating it versus Circle CI first.
Integration tests have performance issues that I believe are caused by a smaller runner being used in GHA (this is not configurable; you use their static-sized runners or provide your own). There's a discussion here about how to move forward: https://gr-oss.slack.com/archives/C03F1DS85AQ/p1654807364006189 -- it boils down to three options:

Investigate hosting our GHA runners
Revert all test migrations to GHA
Continue to use a split setup, with unit tests and linting running alongside CodeQL in github actions, with integration tests running in circle ci. This is the option I think gives us the best mix of functionality and quick runtime.

I have a PoC running tests using paid-for github runners in ~15-17 minutes. https://github.com/jayofdoom/armada/runs/7733643008
There are a handful of things needed before this is ready for landing:

Do some matrix testing across multiple different instance sizes; be sure we're at the best size
Ensure in the case where we cannot spawn a runner, we still run integration tests on a free runner (if possible)
Ensure all items supported by circleci (including releases / uploads of docker images) are supported in our GHA

@jayofdoom thanks for doing this.  I've never checked out the paid-for runners -- do we just pay more and maybe get that time down under 10mins?

I'm not convinced that vertical scaling is possible without actual work to parallelize the tests explicitly. I'm trying to explore those possibilities before doing the cleanup work around this to make it mergeable. (Which will include making sure we're at the point where we have all the performance we can pay for without excess)
For instance, I started something testing before I stepped away this afternoon that is using docker's new build system which might be faster outright, but if it isn't I'm convinced it could be made to be faster with a few changes to our dockerfiles.

https://github.com/jayofdoom/armada/runs/7736074257 looks like I just gained quite a bit of time just by setting an environment variable 👍

One #1156 is merged, we'll have the following remaining steps:

Implement releases in Github Actions
Disable run of jobs in CircleCI

Hey team! Please add your planning poker estimate with Zenhub @ClifHouck @dejanzele @kannon92 @richscott @Sharpz7 @suprjinx

Migration completed. Wow that was quick!


Remove whitespace around two bash examples in LLMOps tutorial

I observed some redundant whitespaces around two bash script examples in the LLMOps tutorial.
I have tried to fix this in this PR.
See below for current state in Edge:

@ritesh-modi
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@ritesh-modi
Can you review this old PR and determine whether it needs to be closed or merged?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge. If the changes no longer apply, add an explanation and a #please-close comment to close the PR.
@MicrosoftDocs/public-repo-pr-review-team

@lgayhardt
Can you review this old PR and determine whether it needs to be closed or merged?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge. If the changes no longer apply, add an explanation and a #please-close comment to close the PR.
@MicrosoftDocs/public-repo-pr-review-team

#sign-off
Thanks!


Update content on /sign-in

Update the list of services on www.gov.uk/sign-in:

Add 'Report a COVID-19 rapid lateral flow test result'
Move 'Passenger locator form' up a position & update the description
Replace 'Manage your student loan balance' with 'Manage your GOV.UK emails'

After these changes are deployed to production we'll need to run the publishing_api:publish_help_page[sign_in] rake task to update the page.
Trello
Figma
The new page:

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠

I have a 👍🏻  from the UCD team for the changes.


Override markdown-it/linkify normalizer

Override the markdown-it/linkify normalizer to insert HTML word break elements (<wbr>) into link URL strings according to The Chicago Manual of Style, specifying opportunities where an HTML renderer (screen or print) should wrap long URLs onto multiple lines.

@mphstudios I'm trying to test this but not seeing any results either visually or in the code. To test, I took the below block with many URLs and dropped it into the cover.md file. When running npm run dev the preview didn't look any different than it did in the main feature/11ty branch, and opening up web inspector, I'm not seeing any <wbr> tags inserted. Screenshots from Chrome below, but I also looked in Safari and Firefox.
http://collections.vam.ac.uk/item/O91709/perfume-flask-cabarin-genevieve/, https://collection.mobiliernational.culture.gouv.fr/objet/GMTT-108-001, http://collections.vam.ac.uk/item/O7767/cistern-leake-ralph/, https://www.pan3sixty.co.uk/virtual_tours/goldsmiths-company/#exhibition-room, http://collections.soane.org/THES92585, http://collections.vam.ac.uk/item/O88524/cupid-on-a-dolphin-statuette-fanelli-francesco/, http://collections.vam.ac.uk/item/O60649/clock-gregory-jeremie/, http://collections.vam.ac.uk/item/O1257366/fontaines-et-iets-deau-dessines-print-fanelli-francesco/, http://emuseum.toledomuseum.org/objects/55455/table-centerpiece, http://collections.soane.org/THES89145, https://www.nationaltrustcollections.org.uk/object/109448, https://www.metmuseum.org/art/collection/search/231564 and https://www.metmuseum.org/art/collection/search/231584, https://mba.caen.fr/oeuvre/portrait-de-lorfevre-nicolas-de-launay-et-de-sa-famille, http://collections.vam.ac.uk/item/O1043356/nouveau-livre-dorfeverie-title-page-marot-daniel/ and http://collections.vam.ac.uk/item/O156506/charger-paul-crespin/, http://parismuseescollections.paris.fr/fr/petit-palais/oeuvres/ecuelle-couverte-0, https://collections.mfa.org/objects/52615/ecuelle, https://www.musee-lorrain.nancy.fr/en/collections/les--uvres-majeures/medaillier-de-lorraine-65, https://collections.louvre.fr/en/ark:/53355/cl010249246.


rename project to sparv-sbx-ocr-correction

[x] rename repo
[ ] rename pyproject
[ ] rename github project
[ ] update makefile
[ ] update ci
[ ] updade pypi

Also see spraakbanken/sparv-pipeline#145


add-migration error in "release/2.0.0"

I have received
"The current CSharpHelper cannot scaffold literals of type 'Microsoft.EntityFrameworkCore.CassandraClusteringOrderByOption'. Configure your services to use one that can".
when set
            modelBuilder.Entity<Applicant>().ForCassandraSetClusteringOrderBy(new[]                 {                     new CassandraClusteringOrderByOption("Id", CassandraClusteringOrderByOptions.ASC)                 });
in OnModelCreating.

The migration error is fixed : https://github.com/simpleidserver/medikit/commit/cf338406afdb03c410fbc5e8edc096cf2d72f93f
In the Applicant table, the id column is defined as a partition key.
You should use Order column because it's defined as a cluster column.


Support attachments

This should include all of the various ways if interacting with attachments:

Inline base64 (may already work)
HEAD/GET/PUT/DELETE /db/docid/attachment
GET multipart/related
Dealing with compression appropriately (hopefully just a case of piping data through and ensuring accept-encoding/content-encoding headers are relayed properly).

Have you got any plans about attachment support? This feature becomes very important for us.

@burzakovskiy I'm working on this right now.

Attachment support is in 0.3.0 of Envoy.


Get an OpenSSF Best Practices Badge

https://bestpractices.coreinfrastructure.org/en

Started the porcess here:
https://bestpractices.coreinfrastructure.org/en/projects/7533/edit
Though bumping it to the next iteration, for now.


Function

Hi, I would like to add one function next to the buttons (screenshot), but I can't find where the buttons are located. The question is where are the buttons located?
Thanks
https://cdn.discordapp.com/attachments/707651653935038594/817849442714648586/unknown.png

You can find those buttons in the Menu.vue
It is located at "[...]/filegator/frontend/views/partials/Menu.vue", Lines 15 to 32.
The Menu.vue then gets imported into the Browser.vue, which is the "main page".

But I don't have the fronted folder, nor is it in the release https://github.com/filegator/filegator/releases/tag/v7.5.0

Do you have a discord?  @morph3us-net

It is not recommended to modify the release itself.
If you want to edit filegator's code, you should have a look here.
It shows you how to set up the development environment. It also works on windows (Just a little bit differently.)
You can also modify the release files, if you really wish to do so, but it's tedious and I wouldnt know how to specifically do that in your case.
I do have a discord account, however just started with development this year myself, so will not be of much help.
Good luck to your endeavors, I cant tell you much more than I did, I'm afraid!

When I want to use the version with Menu.vue and I go to the web, I have a white screen


Fix new clippy lints

I'll need to send a PR to gir too to fix the missing #[must_use] in the auto code.

Looks good otherwise, thanks!

Looks good otherwise, thanks!

Once the gir PR is in, I'll send PRs to all repositories (including gstreamer ;) ).

Once the gir PR is in, I'll send PRs to all repositories (including gstreamer ;) ).


Fixed #30324 -- Technical_500.html Internal Unicode Error Resolving

Ellipsis caused a Unicode error.
So the technical_500.html is generally not rendered.
In versions prior to Django 2.2, elipsis was 3 dots.
Therefore, modify elipsis to 3 dots.
Corrected the technical_500.html to render normally.

See #11170 for an alternative solution and my comment on the ticket.

@chihun-jang Thanks for this patch :+1: , however I agree that NIck's alternative solution suits better.

Thank you so much for fixing the error.👏👏👏


Minor updates to fields based

Building on this https://github.com/acteng/atip/commit/8533cefc0759574374487f26ddfaadc76118fd81 there are a couple more tweaks to make. https://acteng.github.io/schema/v1 isn't quite complete but has everything for the 'base fields' that do not change.

This difference: no Description field.

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/textarea

To my surprise, it seems to have worked #18 without re-running the render local shell script, this is cool!

Textarea. Description could be longer than one line.
And also on other two pages please
On Sat, Oct 29, 2022, 3:52 PM Robin Lovelace @.***>
wrote:

To my surprise, it seems to have worked #18
https://github.com/acteng/atip/pull/18 without re-running the render
local shell script, this is cool!
[image: image]
https://user-images.githubusercontent.com/1825120/198838133-3109a592-f6bd-4a54-9240-86261010b3b0.png
—
Reply to this email directly, view it on GitHub
https://github.com/acteng/atip/issues/16#issuecomment-1295859937, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMWLF6GJ3RAGQSSDSYTBQLWFU2ZFANCNFSM6AAAAAARRZ3GDI
.
You are receiving this because you commented.Message ID:
@.***>

Will give it a bash :+1:


[7.0.x] reconfiguration and custom planet package

Description

Use the correct node profile and runtime package locator for the reconfigure operation.
Updates https://github.com/gravitational/gravity/issues/2026.
Type of change

Bug fix (non-breaking change which fixes an issue)
This change has a user-facing impact

Linked tickets and other PRs

Refs https://github.com/gravitational/gravity/issues/2026

TODOs

[x] Self-review the change
[ ] Write tests
[x] Perform manual testing
[ ] Address review feedback

Testing done

So far, only tested the peripheral function of gravity stop / gravity start without the actual reconfiguration.

Hi @a-palchikov, do you plan to merge this pr on your next release?

Let me bring it up-to-date and I'll merge it once it passes the tests.


Only check OS/platform on MacOS

Temporary fix for Linux not working
References: http://trac.macports.org/ticket/58497

Seems like an os_platform mismatch should always be fatal. I'd suggest this instead:
index 63439d85..b1172076 100644
--- a/src/macports1.0/macports.tcl
+++ b/src/macports1.0/macports.tcl
@@ -706,7 +706,7 @@ proc mportinit {{up_ui_options {}} {up_options {}} {up_variations {}}} {
     }

     # Check that the current platform is the one we were configured for, otherwise need to do migration
-    if {($os_platform ne $macports::autoconf::os_platform) || ($os_major != $macports::autoconf::os_major)} {
+    if {($os_platform ne $macports::autoconf::os_platform) || ($os_platform eq "darwin" && $os_major != $macports::autoconf::os_major)} {
         ui_error "Current platform \"$os_platform $os_major\" does not match expected platform \"$macports::autoconf::os_platform $macports::autoconf::os_major\""
         ui_error "If you upgraded your OS, please follow the migration instructions: https://trac.macports.org/wiki/Migration"
         return -code error "OS platform mismatch"

@raimue additionally noted:

Changing just this one line will not help much, as the installed ports also store the platform information in the registry and will all be considered outdated. I guess the real fix would be not to store the Linux kernel version in os_major, but I am also not sure what else to use.

However I'm not as much concerned if we end up with "outdated" ports and just rebuild them. At the moment we are not even able to run anything.


can't get paste to work

I added an option to DomTerm to use a trivial Webview front-end for the GUI.  (I basically just load a URL specified on the command-line, which is invoked by the domterm driver/server.)  It mostly works, though there are some glitches.  Most critically: paste doesn't work, whether I type the keyboard shortcuts, use my custom menu entry, or the default context menu.  The build instruction discuss the experimental --with-webview configure option and --webview run-time option.
What OS are you using (uname -a, or Windows version)?
I tried both Fedora GNU/Linux 32 and MacOS Catalina.
What programming language are you using (C/C++/Go/Rust)?
C++ (g++) for the Webview "browser", C for the core server, JavaScript for the GUI.
What did you expect to see and what you saw instead?
Clicking Ctrl-Shift-V (on Fedora) or Cmd-V (on MacOS) should paste from the clipboard, but nothing happens.  Likewise using the Paste entry from the Edit or popup menu.
The JavaScript normally uses document.execCommand("paste", false) (along with a "paste" event handler).  I also tried using navigator.clipboard.readText().

I don't think execCommand will works properly in cross platforms, In Guark (webview based) I'm trying a workaround and the results are promising...
You need to handle keyboard shortcuts (ctl+v, ctl+c) in Javascript and prevent default you can use this js lib: https://github.com/jaywcjlove/hotkeys
then on copy you need to write to clipboard and on paste you need to read from clipboard and change the value of focused field (field or input... it depend's on your app), in Go for example:
https://github.com/guark/plugins/tree/master/clipboard
It works fine in my fedora 31, I'm not sure yet it's gonna to work fine or not.
see #397 a related issue

Sorry if my question was unclear.  I will try to clarify my problem.
"I don't think execCommand will works properly in cross platforms"
I'm not necessarily asking for execCommand to work - I'm asking for something that works.  navigator.clipboard.readText() doesn't work either.  There are portability problems with Copy/paste when using custom menus, but at least Ctrl-Shift-V works on most browser engines I've tried (Firefox, Chrome, Electron, Qt) - but not Webview.  I'm fine with changing my code - but I can't find any Webview documentation that mentions the clipboard.
"You need to handle keyboard shortcuts (ctl+v, ctl+c) in Javascript"
Of course.  I do that.  The question is what action do I do in the event handler to access the clipboard.
"then on copy you need to write to clipboard and on paste you need to read from clipboard"
Yes, that is my question - how to do that.  However, Webview does not make the clipboard available, as far as I can tell, either from JavaScript, or from C++.  I could call out to something like xclip or pbpaste but I would really like to avoid that, mainly because of code cleanliness and portability issues.  Also, I don't want to add any dependency on Go.

clipboard support not supported in webview yet (and I think webview org has plans to add it in the future), for C++ you can use:
https://github.com/dacap/clip
or implement something similar to it.

Thanks.  I'm looking into clip.

@PerBothner I have the same issue so if you find a workaround would you be so kind to share it?

I'm using libclipboard. It has a similar purpose as the clip library mentioned above, but libclipboard is smaller, simpler, and in C (rather C++), which make it easier for my environment.
The JavaScript event handler traps the Paste key-board or menu events, and just sends a special "REQUEST-CLIPBOARD-TEXT" escape sequence over WebSockets to my server, which detects this escape sequence, calls
the clipboard_text function from libclipboard, and sends the result is a special OSC escape sequence.  The browser detects the escape sequence and does the appropriate thing.
If you're using Webview exclusively it might make more sense to use the JavaScript bridge (and perhaps use the clip library).  But this approach also fixes problems with Paste on Firefox and other browsers, and was easier for me.
Currently, I've only worked on Paste - I'll look into Copy later. It works on Linux and almost works on MacOS. (Some rough edges I'm working on.)

Thanks a lot! Will look into libclipboard.
All I'm currently interested in is actually pasting. In both Windows & Linux CTRL+V works out of the box but navigator.clipboard and document.execCommand('paste') don't. On MacOS none of the above work.
I'm currently looking into whether I can add permissions to allow access to navigator.clipboard or document.execCommand('paste') so I can get pasting to work from my JS context menu but maybe I'll shelve that idea for now and try your approach to at least get pasting via Cmd+V to work in MacOS in the meantime.
Thanks again!

@PerBothner did you just use custom paste functions or did you try to hack the browsers' API to work?
I'm using libraries which access the clipboard and therefore tried shadowing the native functions as follows but it didn't work:
document.__proto__._queryCommandSupported = document.__proto__.queryCommandSupported;
document.__proto__.queryCommandSupported = (command) => {
  if (command === 'paste') {
    return true;
  }
  return document._queryCommandSupported(command);
};

document.__proto__._execCommand = document.__proto__.execCommand;
document.__proto__.execCommand = (command) => {
  if (command === 'paste') {
    return document.execCommand('insertText', false, getTextFromBackend());
  }
  return document._execCommand(command);
};

Seems to be overriding the command no problem but doesn't actually paste anything...

Once the browser gets the paste text from the server, it handles the paste using custom JavaScript code.  In "line-edit [CANON] mode" the text is inserted at the caret, using direct DOM changes,  In the default "char" [RAW] mode the text  is sent back to the server, which passes it to the process's pty (remember domterm is a terminal emulator)  - which may then sends echo back to the browser.  So some extra back-and-forth of text between processes, but requesting the clipboard is at least processes on teh same computer.

I see so you capture the keypress, right?
I was trying to get both the keyboard shortcut and the native JS paste to work. For the latter I looked into enabling it in the respective webview libraries with limited success so far: https://github.com/webview/webview/pull/405
For the keyboard shortcut I solved it similar to you, capturing the keypress in JS and calling a function which gets the clipboard content from the server via WebSockets.

"I see so you capture the keypress, right?"
Yes.
In DomTerm I have an event listener for "paste".  It gets ev.clipboardData.getData("text") and uses the result as appropriate (either insertion into DOM or sending it to the user/shell process).  This works on desktop browser such as Firefox.  It would probably also work on Webview if issue #405 is fixed.

@PerBothner #405 has landed. On all platforms except EdgeHTML you can use navigator.clipboard now. You can package [Edge Webview runtime] in your installer to make it work on Windows machines without Edge Chromium installed.
I set this up in a WiX installer like so:
<!-- ... -->
<UI>
    <!-- ... -->
    <ProgressText Action="InstallWebView2Runtime">Installing runtime libraries. This may take a few minutes.</ProgressText>
</UI>
<Binary Id="WebView2Runtime" SourceFile="MicrosoftEdgeWebView2RuntimeInstallerX64.exe"/>
<CustomAction Id="InstallWebView2Runtime" BinaryKey="WebView2Runtime" ExeCommand="/silent /install" Return="check" Execute="deferred" Impersonate="no" />

With the latest check-in, Paste (with from keyboard short-cut and mouse/menu actions) seems to work as expected on Fedora Linux. Copy using keyboard short-cuts works, though mouse/menu action doesn't (but that could be relating to mouse-handling issues).
Both Paste and Copy seem to be still broken on MacOS, but it's quite possible I'm doing something wrong.  (I'm getting a strange "beep" instead.)

To make it work on Mac listen to the keypress event in JS and then call document.execCommand("paste").

I added these two listeners and copy paste keyboard shortcuts works on MacOS (Big Sur Version 11.2.1)
And no "beep"!
const webview = new Webview(
  { url: 'https://www.duckduckgo.com' },
);

const copyPasteShortcut = `
window.addEventListener("keypress", (event) => {
  if (event.metaKey && event.key === 'c') {
    document.execCommand("copy")
    event.preventDefault();
  }
  if (event.metaKey && event.key === 'v') {
    document.execCommand("paste")
    event.preventDefault();
  }
})
`

webview.eval(copyPasteShortcut);

await webview.run();


upgrade apache commons-text dependency 1.7->1.8

to be in sync with zebrunner agents (zafira client) we have to bump up apache commons-text dependency to .8

done: https://github.com/qaprosoft/carina/commit/1e543c2708008981284df36bb773b928bcf8a04e
6.4.37-SNAPSHOT+

@vdelendik could you please add details for testing this ticket?

Opened pom.xml and see that  <java.version>1.8</java.version>. Is it expected for this fix?

Tested via https://github.com/qaprosoft/carina/blob/196a41f07e34fda74adbaf48934e9ecad7e90ddc/pom.xml#L52