Chrome link preview covers last row

The link preview on Chrome (showing the link in the bottom left corner) covers up the last row of text. This is particularly noticeable on the Node page, where it covers up the name of the Serial Server.

@jwenner, I've noticed this, but if you actually mouse over the link preview, it moves out of the way, so it doesn't cover a link you are actually trying to click. Does that make this less of a problem for you?

Maybe less of a problem, but still not negated. Would it be possible to add one blank line at the bottom of the page (likely outside the table) to allow for anything like this bar?

I agree that this is annoying and has confused me several times. Definitely not a priority IMHO though.

There is no way to disable the status bar in Google Chrome.
The only "fix" for this issue would be to add a fixed footer at the bottom of the page, but that's essentially just permanently covering whatever is at the bottom of the screen, instead of temporarily.
You can't just add padding to the bottom of the page, because that doesn't stop the issue occurring when the page is longer than the window, as the padding wont even be on the page.
I'm marking this wont-fix, but perhaps it is an argument to increase browser support to at least Firefox so users have some flexibility to work around browser features (such as this status bar) that they don't like.

@adamcw, what I was imagining was just adding a blank line at the bottom. While I agree that this wouldn't fix if the page is longer than the window, I (and probably 99.999%+ of users) am used to scrolling down in such a case. The real issue is that, for the last line, I can't scroll down to make the last line always visible.

@jwenner I'm not sure I like implementing a footer for the sake of a browser feature, but I see your point. I'll make a PR with the change and see what the others think.


Changing some documents requires application restart

I have difficulties keeping my table and the database in sync. After modifying some documents, my GUI does not display the latest changes.
Also a
database.reload();
database.close();
database.open();
does not help.
Only after restarting the whole application, I can see the changes.
Is there anything going on behind the scenes when shutting down the JVM? How can I ensure I query for the latest version of my documents?

Hi @kerner1000
Could you please share some additional info about your application? How are you using OrientDB? Embedded or remote?
A little hint (but it's definitely a guess), try a db.getLocalCache().clear()
Thanks
Luigi

Hi Luigi
db.getLocalCache().clear() worked! Thanks so much! I already wasted days on this topic.
But I am wondering what is 'best practices' do deal with this caching? I doubt I am the only one running into unexpected behavior here.
About our setup:
We establish a db connection basically at application startup:
ODatabaseDocumentTx db = OPartitionedDatabasePool.acquire()
and shut it down only when the application shuts down.
db.close()
For all reading and writing we use the same instance of db and do everything single threaded (multi thread access was causing us some other headaches)
after each document alteration, we call document.save().
We do not use transactions.
We could not figure out a decent way to perform save() in a more intelligent way.
I would be great if you could share some 'best practices' on saving documents, caching and maybe multi thread access!
Many thanks for helping!
Best,
Alex

Hi Alex
OrientDB multi-threading can be tricky, because ODatabaseDocumentTx is not thread safe. What we recommend is that every thread opens and uses its own db connection, this should be safe in general.
About the save operations, if you are working in remote I suggest you to do small transactions instead, this way you will have network communication only at commit() and not at every save(), with obvious performance improvement. Just pay attention to the transaction isolation when you mix java and SQL (http://orientdb.com/docs/2.1/Transactions.html)
Thanks
Luigi

Hi,
Can i add something on top of this, do an 'OPartitionedDatabasePool.acquire()' at the start of the application is not really the way orientdb was thought, the correct way to use it creating the pool at startup, and acquiring/releasing the ODatabaseDocument on group of operations/methods, with the standard pattern:
ODatabaseDocument db = null
try {
    db = pool.acquire();
    //do your operation here
} finally {
   if(db != null)
     db.close();
}

or more concise java7 syntax.
try(ODatabaseDocument db = pool.acquire()){
   //Do your operation here
}

this follow the same pattern of many ORMs or of JDBC with the a connection pool.

Hi all,
thanks for your explanations. Can I find somewhere documentation about database caching? (´db.getLocalCache()`)
When should I generally clear the cache and when not? ;)
Many thanks!

Hi guys, please never call db.getLocalCache() directly. It may lead to serious data consistency problems. If you have to clear cache it means that you incorrectly use DB API.


Replace Rerubi with FiOne

I kept forgetting to merge this. Fix the conflicts and I'll merge it in later... or I'll just deal with it later this week.

File Hierarchy was changed since this pull request was made. Redoing the pull request will likely correct this problem

https://github.com/Sceleratis/Adonis/pull/140


Added Section Component

ids for NavBar:
info
faq
event

The pull request needs to be between hackCWRU:dev and yourbranchname:dev, not your branch:dev and master. I can't merge it in the current state.

I believe I have fixed it. Let me know if I need to make any further changes
Sent from Mail for Windows 10
From: Daniel Grigsby
Sent: Monday, November 13, 2017 7:15 PM
To: HackCWRU/HackCWRU-Website
Cc: cgilker; Author
Subject: Re: [HackCWRU/HackCWRU-Website] Added Section Component (#24)
The pull request needs to be between hackCWRU:dev and yourbranchname:dev, not your branch:dev and master. I can't merge it in the current state.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


[TASK] Sub-event selection from drop-down list.

Description
List all the current events for users to choose as sub-events in the super event.
Acceptance Criteria
user can pick the event from the list as a sub-event.

Hi, @bingzhang, I think this can be closed. Please confirm and close. Thanks.


WIP: Temp(fe)/tx debugger

Do not merge
TX history debugger enabled.

Reworked in v3. Not applicable to v3.


sql: add the ability to access a column from the results of an SRF

PR: https://github.com/cockroachdb/cockroach/pull/24832
From release notes:

Set Returning Functions (SRF) can now be accessed using (SRF).x, where x is the name of a column returned from the SRF or a *. For example, SELECT (information_schema._pg_expandarray(ARRAY['c', 'b', 'a'])).x and SELECT (information_schema._pg_expandarray(ARRAY['c', 'b', 'a'])).* are now both valid. Also, the naming of the resulting columns from SRFs now provide more information about the resulting tuple. [#24832][#24832] {% comment %}doc{% endcomment %}

Relates to #3142


Doesn't work in Cordova

Looking into more of the cordova web view settings and seeing if there is anything I can find that would stop it, but it just returns the unsupported callback. I'd love to utilize this library in my hybrid app for a few things. Yours is the first library to include a callback like this so I wasn't sure if the other libs weren't working or if I was just crazy haha.
I'll do some digging and report back what I find, if anyone else has an idea of why the cordova web view (iOS 9, uses UIWebView same as safari) wouldn't work definitely chime in.

Please provide information about the device you are seeing the issue on
Device is a iPhone 6S+
Operating system is iOS 9.1
Browser is UIWebView (not sure the versioning...)

Hey @NorthMcCormick cool idea! Unfortunately from what I have noticed is that the UIWebView actually lags in support behind the actual Safari app. From my knowledge the web view does not support force values on touch events. That is why any other apps the use the web view like the chrome app do not support 3D touch.
I am going to double check on this but I am pretty sure that is why. Hopefully Apple will add support soon because that would be a big deal!

I was afraid of that. I do know that the iOS 4.0+ platform version with cordova is giving us the option for and updated UIWebView as well as WKWebKit. So if we can leave this open, once I am able to get my project updated, I can test this with the WKWebKit. The newest chrome I think is using the new WKWebView as well. I'll double check that and maybe use it as a test bed too.

I just confirmed, as of Chrome 48 (49 is the current production version) on iOS 9 and up pressureJS does work in the browser. So that's good news! Once I get the WKWebView working I can further test to see if it works without any further options.

Oh that is great news! Thanks for doing that research, that is really good to know. So it sounds like it is supported in the WKWebView but not the UIWebView. I would love to know if you switch to the WKWebView if it will work. To be honest I did not know there were two different types of embedable web views on iOS.

Yes, there are two, however the WKWebView has been bug-city for the last year or so. Cordova FINALLY has it supported with that 4.0 platform version.
In fact, it's been so buggy that (I'm pretty positive) iOS Safari didn't use WKWebView until iOS9, but it's been in the system since like iOS7, publicly available in iOS8.
Yeah cross my fingers it will work, it should open up some cool doors for hybrid apps.

Apple introduced new JavaScript engines with iOS 4.3 . Since then, they were Safari only, 3rd party apps had to use the old classes, known as UIWebKit. Since iOS 8, WKWebKit is available which now offers the same classes Safari uses.

Thanks everyone for the input on this. I made two small iOS apps to test in the simulator, one running the UIWebView and one running the WKWebView. As we suspected pressure and 3D touch WORK in WKWebView but NOT in UIWebView. So if anyone is looking to use pressure (or any 3D touch in general) in an iOS web view use WKWebView and not UIWebView.
:white_check_mark: WKWebView
:x: UIWebView


(maint) Update beaker to version 4

Test FAILed

CLA signed by all contributors.

Test PASSed

Test FAILed


Request: a separate mini-forecast weather display

A small mini-window without decoration that can be set to clickthrough, showing the time until the weather changes and the next weather for the current zone, would be a nice little QoL addition. For fishing, you'd be able to tell how long you have until the weather you need (or how long you have for the weather you need), and if it can work in the Diadem then miners and botanists can get an idea of how long until the timed nodes come up or go down.
Optionally, being able to show more than just the next weather could be neat, but I figured the most useful would be "what weather is coming up and how long do I have until then" for the reasons above.

Maybe as part of the gather window, I'll think about it.


Make stat var explorer dcid browser link more prominent; Change place link from timeline tool to place/stat-var page

thanks!


update

Codecov Report

Merging #23 into master will increase coverage by 26.02%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master      #23       +/-   ##
===========================================
+ Coverage   60.92%   86.95%   +26.02%
===========================================
  Files          19        4       -15
  Lines         151       69       -82
  Branches       15        3       -12
===========================================
- Hits           92       60       -32
+ Misses         58        8       -50
  Partials        1        1

Impacted Files
Coverage Δ

angular/src/app/navpages/page0/page0.component.ts

angular/src/app/navpages/auth/auth.component.ts

angular/src/environments/environment.ts

angular/src/app/navpages/info/info.component.ts

...pp/navpages/notification/notification.component.ts

angular/src/app/firebase/data/data.component.ts

angular/src/app/navpages/page1/page1.component.ts

angular/src/app/navpages/svg/svg.component.ts

angular/src/app/navbars/navbar/navbar.component.ts

...gular/src/app/service/notification-post.service.ts

... and 5 more

Continue to review full report at Codecov.

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


ci: add Codecov

Closes #572

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@b1287e2). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #754   +/-   ##
=======================================
  Coverage        ?   69.29%
=======================================
  Files           ?      227
  Lines           ?     9756
  Branches        ?        0
=======================================
  Hits            ?     6760
  Misses          ?     2996
  Partials        ?        0


Replace use of deprecated function Base.download with something that's not deprecated

It was deprecated in v1.6 https://docs.julialang.org/en/v1.6/NEWS/

I think I add Downloads into dependency.
(@v1.10) pkg> dev ReverseGeocode

(@v1.10) pkg> activate ReverseGeocode
  Activating project at `~\.julia\dev\ReverseGeocode`

(ReverseGeocode) pkg> st
Project ReverseGeocode v0.5.0
Status `~\.julia\dev\ReverseGeocode\Project.toml`
  [336ed68f] CSV v0.10.14
  [a93c6f00] DataFrames v1.6.1
  [864edb3b] DataStructures v0.18.20
  [b8a86587] NearestNeighbors v0.4.16
  [a5390f91] ZipFile v0.10.1
  [f43a241f] Downloads v1.6.0
  [56ddb016] Logging

(ReverseGeocode) pkg> add Downloads
   Resolving package versions...
    Updating `~\.julia\dev\ReverseGeocode\Project.toml`
  [f43a241f] + Downloads v1.6.0
    Updating `~\.julia\dev\ReverseGeocode\Manifest.toml`
  [0dad84c5] + ArgTools v1.1.1
  [f43a241f] + Downloads v1.6.0
  [7b1f6079] + FileWatching
  [b27032c2] + LibCURL v0.6.4
  [ca575930] + NetworkOptions v1.2.0
  [deac9b47] + LibCURL_jll v8.4.0+0
  [29816b5a] + LibSSH2_jll v1.11.0+1
  [c8ffd9c3] + MbedTLS_jll v2.28.2+1
  [14a3606d] + MozillaCACerts_jll v2023.1.10
  [8e850ede] + nghttp2_jll v1.52.0+1

(ReverseGeocode) pkg> instantiate
(ReverseGeocode) pkg> update
    Updating registry at `~\.julia\registries\General`
    Updating git-repo `https://github.com/JuliaRegistries/General.git`
    Updating registry at `~\.julia\registries\HolyLabRegistry`
    Updating git-repo `git@github.com:HolyLab/HolyLabRegistry.git`
  No Changes to `~\.julia\dev\ReverseGeocode\Project.toml`
  No Changes to `~\.julia\dev\ReverseGeocode\Manifest.toml`

the project.toml have Downloads in [deps] but not in [compat]
name = "ReverseGeocode"
uuid = "96b87e3f-b46e-456c-8178-0e438baab6fa"
authors = ["Jan Mrkos<EMAIL_ADDRESS>version = "0.5.0"

[deps]
CSV = "336ed68f-0bac-5ca0-87d4-7b16caf5d00b"
DataFrames = "a93c6f00-e57d-5684-b7b6-d8193f3e46c0"
DataStructures = "864edb3b-99cc-5e75-8d2d-829cb0a9cfe8"
Downloads = "f43a241f-c20a-4ad4-852c-f6b1247861c6"
Logging = "56ddb016-857b-54e1-b83d-db4d58db5568"
NearestNeighbors = "b8a86587-4115-5ab1-83bc-aa920d37bbce"
ZipFile = "a5390f91-8eb1-5f08-bee0-b1d1ffed6cea"

[compat]
CSV = "0.10"
DataFrames = "1"
DataStructures = "0.18"
NearestNeighbors = "0.4"
ZipFile = "0.10"
julia = "1"

julia> using ReverseGeocode
julia> Geocoder();
[ Info: Downloading 'countryInfo.txt' from http://download.geonames.org/export/dump/countryInfo.txt.
[ Info: cities1000.csv not found in ~\.julia\dev\ReverseGeocode\data.
[ Info: Downloading cities1000 zip from http://download.geonames.org/export/dump/cities1000.zip.
[ Info: Processing cities1000.zip.
[ Info: Reference dataset sucessfuly extracted and saved in ~\.julia\dev\ReverseGeocode\data\cities1000.csv.

But I don't know how to create a new pull request without my previous pull request.
(This is my first time to contribute existing package with other on Github.)

I think I add Downloads into dependency.

Ok, so I just looked at the download function, and Downloads package is part of Julia standard library, so you don't need to install it. Just use
using Downloads

Downloads.download(......

That way, project.toml does not need to be updated.

I don't know how to create a new pull request without my previous pull request.

No worries, you can open a PR from any branch of your forked repository. So, to make a new PR, start a new branch from the master (that corresponds to the latest release here) and put your changes there. Once you push that branch into your fork, you can open a PR with just the changes to the download.

I tried directly add Downloads in ReverseGeocode.jl
but it failed
using ZipFile,
    CSV,
    NearestNeighbors,
    Logging,
    DataFrames,
    DataStructures,
    Downloads

julia> using ReverseGeocode

Precompiling ReverseGeocode
        Info Given ReverseGeocode was explicitly requested, output will be shown live
ERROR: LoadError: ArgumentError: Package ReverseGeocode does not have Downloads in its dependencies:
- You may have a partially installed environment. Try `Pkg.instantiate()`
  to ensure all packages in the environment are installed.
- Or, if you have ReverseGeocode checked out for development and have
  added Downloads as a dependency but haven't updated your primary
  environment's manifest file, try `Pkg.resolve()`.
- Otherwise you may need to report an issue with ReverseGeocode
Stacktrace:
 [1] macro expansion
   @ .\loading.jl:1776 [inlined]
 [2] macro expansion
   @ .\lock.jl:267 [inlined]
 [3] __require(into::Module, mod::Symbol)
   @ Base .\loading.jl:1753
 [4] #invoke_in_world#3
   @ .\essentials.jl:926 [inlined]
 [5] invoke_in_world
   @ .\essentials.jl:923 [inlined]
 [6] require(into::Module, mod::Symbol)
   @ Base .\loading.jl:1746
 [7] include
   @ .\Base.jl:495 [inlined]
 [8] include_package_for_output(pkg::Base.PkgId, input::String, depot_path::Vector{String}, dl_load_path::Vector{String},
load_path::Vector{String}, concrete_deps::Vector{Pair{Base.PkgId, UInt128}}, source::Nothing)
   @ Base .\loading.jl:2222
 [9] top-level scope
   @ stdin:3
in expression starting at ~\.julia\packages\ReverseGeocode\kue0J\src\ReverseGeocode.jl:1
in expression starting at stdin:3
  ✗ ReverseGeocode
  0 dependencies successfully precompiled in 3 seconds. 48 already precompiled.

ERROR: The following 1 direct dependency failed to precompile:

ReverseGeocode [96b87e3f-b46e-456c-8178-0e438baab6fa]

Failed to precompile ReverseGeocode [96b87e3f-b46e-456c-8178-0e438baab6fa] to "~\\.julia\\compiled\\v1.10\\ReverseGeocode\\jl_AC6B.tmp".
ERROR: LoadError: ArgumentError: Package ReverseGeocode does not have Downloads in its dependencies:
- You may have a partially installed environment. Try `Pkg.instantiate()`
  to ensure all packages in the environment are installed.
- Or, if you have ReverseGeocode checked out for development and have
  added Downloads as a dependency but haven't updated your primary
  environment's manifest file, try `Pkg.resolve()`.
- Otherwise you may need to report an issue with ReverseGeocode
Stacktrace:
 [1] macro expansion
   @ .\loading.jl:1776 [inlined]
 [2] macro expansion
   @ .\lock.jl:267 [inlined]
 [3] __require(into::Module, mod::Symbol)
   @ Base .\loading.jl:1753
 [4] #invoke_in_world#3
   @ .\essentials.jl:926 [inlined]
 [5] invoke_in_world
   @ .\essentials.jl:923 [inlined]
 [6] require(into::Module, mod::Symbol)
   @ Base .\loading.jl:1746
 [7] include
   @ .\Base.jl:495 [inlined]
 [8] include_package_for_output(pkg::Base.PkgId, input::String, depot_path::Vector{String}, dl_load_path::Vector{String},
load_path::Vector{String}, concrete_deps::Vector{Pair{Base.PkgId, UInt128}}, source::Nothing)
   @ Base .\loading.jl:2222
 [9] top-level scope
   @ stdin:3
in expression starting at ~\.julia\packages\ReverseGeocode\kue0J\src\ReverseGeocode.jl:1
in expression starting at stdin:
Stacktrace:
  [1] pkgerror(msg::String)
    @ Pkg.Types ~\AppData\Local\Programs\Julia-1.10.2\share\julia\stdlib\v1.10\Pkg\src\Types.jl:70
  [2] precompile(ctx::Pkg.Types.Context, pkgs::Vector{…}; internal_call::Bool, strict::Bool, warn_loaded::Bool, already_instantiated::Bool, timing::Bool, _from_loading::Bool, kwargs::@Kwargs{…})
     Use `show(err)` to see complete types.

You are right, I just tested this, and it only works with installed Downloads. The changes work nicely, though, with no warnings :-). So, all good. Anyway, I think we need the compact entry for the new package version to be registered without issues. Can you put 1.5.3 as the Downloads version there? There is something weird going on with the versions in that package and 1.5.3. is the latest relaease.

no matter how I specify the version of Downloads, it always gives add Downloads 1.6,
as same as in issue#238
and it is indeed 1.6. in the commit
somehow the version number on github page isn't updated.

Addressed by #22
@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/107944
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.5.1 -m "<description of version>" 57a5c4499b6fc13bfedac9e74e620f575f1ffecd
git push origin v0.5.1


Azure SQL Serverless - current supported methods of provisioning?

Given the response from #41350 which states there is no ARM support currently, is there currently support for provisioning Serverless SQL via Azure CLI? or is PowerShell the ONLY programatic way to provision this type of database currently?

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

ID: 46602618-c925-75b6-bf3e-1039f168f141
Version Independent ID: aa73eb53-55b6-6aa2-2542-e26fd32764fe
Content: Azure SQL Database serverless (preview)
Content Source: articles/sql-database/sql-database-serverless.md
Service: sql-database
Sub-service: service
GitHub Login: @moslake
Microsoft Alias: moslake

@wahyuen Thank you for reaching out. We are actively investigating and will get back to you soon.

@wahyuen There is currently no roadmap for getting this done. But you can definitely upvote the already created link for this issue on UserVoice
The PG team refers to user voice on regular basis and try to incorporate the features in high demand. They provide the feedback as well when they start a feature work.
I know this is not much to go on, so please be patient as once the service will be out of preview and generally available, it will have many additional features.

@wahyuen We will proceed to close this issue now.
If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Is this project abandoned?

The latest update is about 3 months ago.

I'm curious too ... would love to pick it up at some point


Remove bad generation

Changes

w.r.wait(update_function) is wrong and shouldn't be part of generated code. Not sure why it was generated, using the latest SDK (v0.31.0) deco tool is failing generation.
Tests

[ ] make test run locally
[ ] make fmt applied
[ ] relevant integration tests applied

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (a1055d1) 57.89% compared to head (1296980) 57.89%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #539   +/-   ##
=======================================
  Coverage   57.89%   57.89%
=======================================
  Files          44       44
  Lines       27956    27956
=======================================
  Hits        16185    16185
  Misses      11771    11771

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


ci: merge opae-libs

Automated changes by create-pull-request GitHub action.
Must be use a normal merge (not squash-merge and not rebase-merge)

Coverage increased (+0.02%) to 64.356% when pulling 08cadf70085398630dcd2dc62888092a58d465ac on auto/update-libs into 2b1cc0fdfdcd95d78163e93c2245b124a73d14cc on master.


Use 1.16

Signed-off-by: Matthias Wessendorf<EMAIL_ADDRESS>As per title.
Main is used as the base for release-next, and this change was tested here:
https://github.com/openshift-knative/eventing-kafka/pull/261
/assign @aliok

/lgtm
/approve


What is the context size of TigerBot?

I found that in both config.json and this file the model_max_length is 1024:https://github.com/TigerResearch/TigerBot/blob/509a50e429af3f2a2f391f8308ffc3c00349faa5/infer.py#L54
But in this file the seqlen is 2048: https://github.com/TigerResearch/TigerBot/blob/509a50e429af3f2a2f391f8308ffc3c00349faa5/gptq/tigerbot_infer.py#L62
So I'm confused what is the context size (i.e. maximum tokens limit per inference) of TigerBot's context. Is it 1024, 2048 or longer?
And will TigerBot-7B and TigerBot-180B have different context size?
Any help will be appreciated.

Thank you for your kindness🥰


Add code related to registration, key policies, instance policies and allowed IP port

Added SDK support related to registration, key policies, instance policies and allowed IP port.
Also added more examples to README.

Created new branch and opened new PR after restructuring the project into 1 module.


Adding ReadUtils and updating VariantUtils for upcoming VerifyBamId p…

…ipeline

LGTM

LGTM
note to self - there are a few preexisting methods in src/main/java/com/google/cloud/genomics/dataflow/utils/VariantUtils.java that could be refactored into utils-java since those have no dataflow dependencies


Cleanly close app

Is there a clean way of stopping the bottle server from running when the app window has been closed? Python process is left open when the window is closed. I'm assuming this is the bottle server that is still running.

Discovered more information, which may be known already but wanted to provide my findings.
I appears that gevent is looping to keep the window open.

I had this problem (closing the window not closing the app) when I first started, but it turned out that I had not added the eel.js to my HTML page. Once I added that, the closing of the window seemed to properly send the sys.exit (which you can let happen or use a try/except on)

As mentioned by patter001, an option is to do a try/catch in my case i manually close the python.exe process on except:
try:
      #Start the application and pass all initial params below
      eel.start(....)
except (SystemExit, MemoryError, KeyboardInterrupt):
      #Handle errors and the potential hanging python.exe process
      os.system('taskkill /F /IM python.exe /T')

This may be overkill or not necessary in some cases, but works for me.

I'm going to close this based on @patter001's comment. Eel should definitely be shutting down when the last websocket connection closes - if this isn't happening, please submit a new issue as a bug report with code (ideally a link to an open github repo) so that we can reproduce and investigate. Thanks!

https://github.com/deepakrathore2k1/Instant-Bot

I'm going to close this based on @patter001's comment. Eel should definitely be shutting down when the last websocket connection closes - if this isn't happening, please submit a new issue as a bug report with code (ideally a link to an open github repo) so that we can reproduce and investigate. Thanks!

https://github.com/deepakrathore2k1/Instant-Bot


ui: DPID autocomplete should remove local ports

As noticed by viniarck in PR #120
@rmotitsuki I wonder if we should exclude the local port<PHONE_NUMBER> for instance, on a dpid 1, 00:00:00:00:00:00:00:01:4294967294.

Results are coming from REST services. A new service should remove the local ports.


Live

Docs Build status updates of commit 7bbb7ca:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


add support for docker registry token auth server

Fix https://github.com/go-gitea/gitea/issues/2316
similar to https://github.com/go-gitea/gitea/pull/14919 but only token server

do not store registry repo to database
fix auth scope type registry:catalog:*
fix Unauthorized response message

ref:

https://docs.docker.com/registry/spec/auth/token/
https://github.com/distribution/distribution

Codecov Report

Merging #16048 (0542f63) into main (8947422) will decrease coverage by 0.04%.
The diff coverage is 35.17%.

@@            Coverage Diff             @@
##             main   #16048      +/-   ##
==========================================
- Coverage   44.07%   44.03%   -0.05%
==========================================
  Files         682      686       +4
  Lines       82433    82631     +198
==========================================
+ Hits        36336    36388      +52
- Misses      40181    40309     +128
- Partials     5916     5934      +18

Impacted Files
Coverage Δ

modules/packages/docker/auth.go
0.00% <0.00%> (ø)

routers/repo/docker.go
0.00% <0.00%> (ø)

routers/repo/setting.go
13.40% <0.00%> (-0.14%)
:arrow_down:

services/forms/repo_form.go
40.71% <ø> (ø)

modules/setting/docker.go
25.00% <25.00%> (ø)

modules/packages/docker/token.go
75.64% <75.64%> (ø)

models/repo_unit.go
88.76% <100.00%> (ø)

models/unit.go
60.00% <100.00%> (+13.42%)
:arrow_up:

modules/context/repo.go
53.36% <100.00%> (+0.08%)
:arrow_up:

modules/setting/setting.go
51.34% <100.00%> (+0.09%)
:arrow_up:

... and 16 more

Continue to review full report at Codecov.

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

Is it possible to use the existing JWT functionality? This would prevent another dependency:
https://github.com/go-gitea/gitea/blob/main/models/oauth2_application.go

Is it possible to use the existing JWT functionality? This would prevent another dependency:
https://github.com/go-gitea/gitea/blob/main/models/oauth2_application.go

Yes it would be

@dragonflylee It's better there is a repository level option like issue when global container is enabled.

You may not need a different private key, see #16010

You may not need a different private key, see #16010

registry only support rsa/ecdsa sign
not all jwt auth method

This PR seems broken, many unrelated changes.
Either close this one and start a new clean one, or reset a clean branch from main and force push.


Automatic container vulnerability scan

Implement a vulnerability scan, using trivy, when building the container.
Scan results should be available es artifact inside the pipeline or alternatively as pipeline report.

Feature implemented with PR #15
Scan report output example: https://github.com/cybcon/modbus-server/actions/runs/5230715354


Build doesn't show up in the build list anymore, waiting for it to appear again (check your email for processing issues if this continues)

Hi,
I am getting below error in Azure pipeline
Build doesn't show up in the build list anymore, waiting for it to appear again (check your email for processing issues if this continues)
When I checked into iTunes, the build is already processed without any error.

Can you please create a new issue and fill the issue template so we have some context? (Most important is the commadn that is being executed and the full output you are getting) Thanks.


⚠ Bofrost.de has degraded performance

In 16b1d6a, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1592 ms

Resolved: Bofrost.de performance has improved in 2757bee.


Adding separate timeout for hooks

closes #11983
What this PR does / why we need it:
This PR adds another timeout specifically for hooks, it allows for long-running hooks to complete with an installation/upgrade/deletion to succeed without having to set a large general timeout when the normal K8s objects in the chart do not need the large timeout and there would lose the ability to find issues quickly if they do timeout.
Special notes for your reviewer:
I have added the timeout option to every operation that had hooks involved, the only thing I wasn't sure on was the release testing operation, the timeout seemed to only be used for hooks and therefore I can possibly change that timeout to be a hook time out to be more consistent or I can leave that operation with the standard time out or I can leave it with both incase the standard timeout is needed in future for something else.
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

@gjenkins8 Thank you for your time, I'll get on with the suggestions

@gjenkins8 I have changed the default value to be 0 seconds and then with each operation before using the --hook-timeout value checking to see if it set and if not then set it to the same value as the --timeout value. This should cover already working instances where only --timeout is set and the behaviour should remain the same but gives the option to set the other kind

@gjenkins8 Thank you again for looking at my PR. Is there anything I'm missing? Just wondering why a maintainer hasn't commented anything


CMS_decrypt_set1_*(): fix NULL deref on unsuitable content type

Fixes #19975 in the obvious way.
Also a related potential crash in CMS_decrypt_set1_password().
The point is that that the input might have an unexpected content type, so a guard is needed at both places where ec is obtained using ossl_cms_get0_env_enc_content(cms).
Note that in CMS_decrypt_set1_pkey_and_peer()  there was no such ec != NULL guard in
    if (ris != NULL)
        debug = ec->debug;

maybe because it is implied here by ris != NULL.
Still I added it there, too, to be on the safe side.

The mistake of dereferencing what could be a NULL pointer return occurs in more than this place in the CMS code.
Every call to CMS_get0_RecipientInfos or ossl_cms_get0_env_enc_content needs to be able to handle the return being NULL and trap that and report errors rather than blindly dereference the returned pointer.

I added tests: exempted but maybe it would be easy enough to add a test.

The mistake of dereferencing what could be a NULL pointer return occurs in more than this place in the CMS code. Every call to CMS_get0_RecipientInfos or ossl_cms_get0_env_enc_content needs to be able to handle the return being NULL and trap that and report errors rather than blindly dereference the returned pointer.

Agreed - except for cases where the context already prevents an unsuitable content type, which may be non-trivial to determine at places.
And as I mentioned in #18919, more effort should be spent on cleaning up the CMS implementation, such as removing the strong overlap with PKCS#7.

Agreed - except for cases where the context already prevents an unsuitable content type, which may be non-trivial to determine at places.

Which is why checking in all places is the right sort of defensive programming style to use.
It does no harm to check these sorts of things - especially in a part of the code base which has seen less focus that other parts.

Ping for second review

This pull request is ready to merge

Should this also go to 3.1, 3.0, 1.1.1?

This is not applicable to 3.1, 3.0 as https://github.com/openssl/openssl/issues/19975 is master-only regression.

This is not applicable to 3.1, 3.0 as #19975 is master-only regression.

Right.
Yet this reminded me of an exchange we had in https://github.com/openssl/openssl/pull/19981#issuecomment-1414230784 about backporting that PR, which I just did in #20209, also including the fixup done here.

Merged to master - thanks @t8m and @beldmit

Merged to master branch. Thank you for your contribution.


Fix the off-by-one on TLS1.3 record sizing

Please note that while we are transitioning from travis-ci to AWS CodeBuld, some tests are run on each platform. Non-AWS contributors will temporarily be unable to see CodeBuild results. We apologize for the inconvenience.
Issue # (if available):
N/A
Description of changes:
When using TLS1.3 with s2n_connection_prefer_throughput set and sending large payloads, s2n_record_encrypt called from s2n_record_writev will fail as the preceding s2n_stuffer_raw_write would fail to resize a tainted buffer.
tainted = 1; is only set either in s2n_stuffer_raw_write or s2n_stuffer_raw_read.
the conn->out stuffer is initialized here:
  GUARD(s2n_stuffer_resize_if_empty(&conn->out, S2N_LARGE_RECORD_LENGTH));

however S2N_LARGE_RECORD_LENGTH was based off 2^14 which is the max fragment size for a TLS1.2 record.  When using TLS1.3, this becomes 2^14+1.
We then taint the stuffer here:
  uint8_t *digest = s2n_stuffer_raw_write(&conn->out, mac_digest_size);

If we are trying to write the tls13 max fragment size into the record, we will have insufficient space to write the encrypted payload, we will fail the tainted check in s2n_stuffer_resize which is called when we write the data to be encrypted here:
  /* Do the encryption */
  struct s2n_blob en = { .size = encrypted_length, .data = s2n_stuffer_raw_write(&conn->out, encrypted_length) };
  GUARD(s2n_record_encrypt(conn, cipher_suite, session_key, &iv, &aad, &en, implicit_iv, block_size));

As we only use this constant for allocating stuffers, the simplest thing is to allocate them to the maximum value across all specs to avoid the switching at runtime.  TLS version specific code should be validating the appropriate maximum fragment sizes are not being exceeded when computing the records.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1780 into master will decrease coverage by 0.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1780      +/-   ##
==========================================
- Coverage   81.79%   81.68%   -0.12%
==========================================
  Files         224      224
  Lines       16579    16579
==========================================
- Hits        13561    13542      -19
- Misses       3018     3037      +19

I think fixing maximum record length should bring us in the right direction. Not being able to resize the stuffer may still be a little bummer (it'd be nice to say use 4k memory per connection, then grown them as the records expand) but this solution here is simpler. Using a sufficient stuffer size probably also prevent issues on the receiving end, although I would like to verify that too.


Show channel & administrator id of community for new user's search

Hello.  I have a suggestion.  When a new user types the command
/findcomms XXX
to find a community, the bot will only show the community name and group ID.  I think it is better to display the channel ID and administrator ID of community.
Some users don't have the patience to join the group and follow the chat.  But it is necessary to follow the channel and know who is the creator of the community.

Hello, Can you assign me this issue?


Feat/1582 add science lessons to search

Description

Deletes filter to add science lessons back into the search

Issue(s)
Fixes #1582
How to test

Go to {cloud link}
Click on _______
You should see _______

Screenshots
How it used to look (delete if n/a):

How it should now look:

Checklist

[ ] Added or updated tests where appropriate
[ ] Manually tested across browsers / devices
[ ] Considered impact on accessibility
[ ] Design sign-off
[ ] Approved by product owner

:tada: This PR is included in version 1.185.7 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


State tree / app state garbage collection

Duplicate of https://github.com/consensus-shipyard/ipc/issues/154.


[chromedriver] Update chromedriver download URL

Google changed the Chromedriver download domain to "storage.googleapis.com".

Landing directly as the patch is trivial and to unblock the CI.


virt-handler, VMI iface status info source: Add pod info source

The new 'pod' info-source indicates whether an interface is weird between the guest and the external interface (add by the CNI), i.e.: in case of bridge binding interface, the existence of the tap and bridge exist in virt-launcher pod.
The info source depends on checking in the network status volatile cache, if the interface exist it means the network plumbing for it finished and its ready to attach to the domain.
The indication whether an interface is weird to the guest in virt-launcher (e.g.: tap and bridge exist) is the missing part that required for virt-controller to clear interfaces with 'absent' state from the VMI spec.

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

/uncc @oshoval @rhrazdil
/cc @EdDev @AlonaKaplan

/test pull-kubevirt-unit-test
/test pull-kubevirt-e2e-k8s-1.27-sig-network
/test pull-kubevirt-build
/test pull-kubevirt-generate
/test pull-kubevirt-manifests

/close


Can this be used for anomaly detection?

Hi,
Yes, of course. The purpose of autoencoders is to compress the data, then rebuild it. Along this process, the main properties of the data are kept (in order to reduce the loss on average), however  unexpected variations, that are more than noise, and happen according to random patterns (if this is an acceptable definition of an anomaly), are removed in the reconstruction phase.
Really, think of it as a process summarizing the main idea of your data.
So, if your network if good enought to rebuild the main shapes of your data, while removing these unexpected variations, then you could expect it to remove anomalies (by focusing on where the reconstruction error is 'big', i.e > a certain threshold carefully chosen).

I have tried to replicate this  https://arxiv.org/pdf/1910.03818.pdf  paper
using your code,however I could only get a F1 score of about 0.45 .I tried
using various threshold.I tested it on Yahoo S5 dataset .It would be great
if you could mention what are the changes I could make to implement this
paper
On Mon, Mar 16, 2020 at 3:54 PM Dany Yatim<EMAIL_ADDRESS>wrote:

Hi,
Yes, of course. The purpose of autoencoders is to compress the data, then
rebuild it. Along this process, the main properties of the data are kept
(in order to reduce the loss on average), however unexpected variations,
that are more than noise, and happen according to random patterns (if this
is an acceptable definition of an anomaly), are removed in the
reconstruction phase.
Really, think of it as a process summarizing the main idea of your data.
So, if your network if good enought to rebuild the main shapes of your
data, while removing these unexpected variations, then you could expect it
to remove anomalies (by focusing on where the reconstruction error is
'big', i.e > a certain threshold carefully chosen).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Danyleb/Variational-Lstm-Autoencoder/issues/5#issuecomment-599456807,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJF4BUHFSGSBQW3724IQA2LRHX45TANCNFSM4LKPAZUA
.

Hi,
I'm sorry, but I won't read this paper for you.  Read it, understand it, doubt about it, implement it. This is how it works, and how you learn :-)
Have a nice day


improve ui

Ok here are my thoughts for improving the UI:

For starters, whatever UI library Alexis/Bhavik have in mind I'm sure it looks sick so for colors and fonts let's just go with that.

Clean this up a bit obvi

Ok instead of breaking down the user inputs into TO, CC, BODY, etc. Lets just have them all be under 1 section: User input fields. Then, instead of having the titles be TO: or Reciever: lets have the code be something more human-readable. Instead of just TO: maybe Who is this email being sent to? etc. Have this be coded by the user like {@email: Who is this email being sent to?} or something like that.

Notice the delivery date and receiver on that last screenshot. It's the same value. To reduce redundancy, if a user puts the same tag twice, only generate 1 input for it to save time.

lmk what others think but these are my initial ideas