log structure is unmaintainable

Currently log files get placed in an unwieldy directory structure
/var/foo/log/bar/live/web/site/django.log
This is really difficult to maintain, rebuild after a failure, and do log rotation. Log files should rather go into
/var/foo/log/bar-live-web-site.log
and
/var/foo/log/bar-live-web-site.err
That way it's easier to do things like multitail *.log in one place.

It does not use nested paths anymore.


Improve possibly confusing optional argument syntax

The syntax for first-position optional parameters in function signatures can confuse newbies and devs from other languages.
The gulp.task API is so important that clarity & completeness should have a higher priority than EBNF notation.

I'm going to close this as I don't think it helps and actually recommends the non-recommended pattern.


[WIP] Terms and Conditions modal

This is addressing #761.  I've added items to the EN localization.  Those would need to get added to the google docs as well.  But looking for feedback on general functionality.
Also, could we get a lawyer to look at the text here?  I took a stab at creating something simple but terms and conditions and privacy policy really should be vetted properly by a lawyer.
As for privacy policy, we currently don't use cookies do we?  I was thinking of saving a cookie once they accepted the terms so this doesn't display every time.  But then I checked sites like breweries that have age-specific requirements and they ask every time.  If we were to save a cookie, we'd want to add that to the privacy policy.

Thanks for taking on this issue!
So usually I trend towards accepting liberally, but this is a sensitive situation, so this may take some time to work out. A few thoughts (sorry about the length):

If we were to get the opinion of a lawyer, would we need to pay them to make it official? Would we need some kind of signature/witness/record of the approval of such a person?
In my mind, that modal would pop up when they went to fill in the form, not on the home page. i.e. no need for a disclaimer if they're just going to the About page to read about the project.
Some of us are used to a lot of big vocabulary words, but this tool may be used by folks who have expertise in other parts of life, not vocabulary. I suspect that if you ran the language of that disclaimer through a readability test, it wouldn't turn out well. If we want some legalese, I think it should be separate from the explanation that first hits the user.
I'd like more interactivity than is currently there if possible. I'd like something that makes sure the user is engaged in this dialogue. That's what I was trying to describe in my comment in #761.
I'm not sure we need a full disclaimer right now. Maybe I'm wrong, though. The goal that I'd understood was a simple dialogue to bring about the understanding that the calculations made by this tool cannot be trusted.
We don't use cookies and I'd personally prefer to continue not using them for now. Do other folks have other thoughts of the pros and cons? Currently the only stuff that's saved is the stuff for the Dev HUD and has nothing to do with any user.
The feedback form is the only place where privacy becomes an issue and we may not need more of a privacy statement than is currently given in that interaction. Try pressing 'Ready' on that bad boy and take a look at the next prompt that comes up. Don't worry, nothing will be submitted.

Latest push simplifies the text and adds requested radios.

Oh, hmm. We might want a 'Cancel' button as well. Thoughts?

Also, it's looking pretty swank :)

Ask and ye shall receive.  Cancel added.  That was annoying me too.  :)

Oh, it also occurs to me - I don't know what color would be better, but I'm not sure the 'Continue' button should be red. Anyone know what the convention for this kind of interaction is?

@knod I suppose another question that should be asked is whether the modal should be red at all.  I made the header red because a red modal was originally what was discussed.  A red button just looked more consistent.  Maybe the whole thing should just be the same color as the rest of the site since it sounds like it's less important that this look like a warning and more to just slow the user down and make them think.

That's definitely something we can discuss. Personally, I'd be partial to our orange - I think it'd be good to queue people that this is important/different/significant with a color cue (and possibly an exclamation mark?). It is a 'danger' situation as I see it. If orange is the main color, though, it doesn't fix the button color issue. A teal button (our green) may work better aesthetically with the orange (our orange), though.

It occurs to me that with the presentation looming, it may be good to have a way to turn this off without the devhud visible. Maybe we can put an 'off' switch for it in DevSwitch temporarily? I don't want it interfering with the demo.

Sorry if this changes the goal posts. If you want, we can leave that feature for the next PR.

@knod, Sure, I can do that.  I'll create a new issue and PR to address that though.  Let's get this initial functionality wrapped up and merged first.

Closes #761


@conda-forge-admin please add user @dwsmith1983

Comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added user @dwsmith1983 in conda-forge/dataconf-feedstock#9.


New GND endpoint -> now named DNB (Deutsche Nationalbibliothek)

Many thanks, @hannahbast and @joka921, for the swift implementation of the GND dataset during the Wikidata Data Modelling days!
So far, all queries at https://qlever.cs.uni-freiburg.de/gnd result in the error
Unexpected token '<', "<!DOCTYPE "... is not valid JSON.
According to #1171 that might just be a generic error message.
Could you please have a look into what's the cause here?

@brunnerpaul Thanks for the reminder! That just means that the backend is down (the supoptimal error message is a temporay bug in the QLever UI, we are currently working on a refactoring PR concerning this). Here is what happened:
During the meeting last week, I set up an instance for a selection of files from https://data.dnb.de/opendata , which worked fine. After the meeting, I tried to set up an instance for all the data I could find on https://data.dnb.de/opendata, namely:
curl -L -C - --remote-name-all https://data.dnb.de/opendata/authorities-gnd_lds.nt.gz https://data.dnb.de/opendata/bib_lds.nt.gz https://data.dnb.de/opendata/dnb-all_lds.nt.gz https://data.dnb.de/opendata/dnb-all_ldsprov.nt.gz https://data.dnb.de/opendata/zdb_lds.nt.gz
Unfortunately, it turned out that some of these files are not formatted correctly, and like most SPARQL engines, QLever refuses to index data that is not formatted correctly. Two questions:

Which files should we index? If only a subset of the above, why only a subset?

Wouldn't the name dnb be more appropriate for the instance than gnd? The many abbreviations used on the site are really confusing (dnb, gnd, lds, zdb, ...)

Which files should we index? If only a subset of the above, why only a subset?

I've added what I know about the specific datasets here:

https://data.dnb.de/opendata/authorities-gnd_lds.nt.gz
1.9G
Stabiler Link auf den aktuellen Gesamtabzug der GND im Format RDF (N-Triples)

This is the most commonly used dataset AFAIK, authority files for persons, institutions, places, thesauri, and should have the highest priority.

https://data.dnb.de/opendata/bib_lds.nt.gz
4.7M
Stabiler Link auf den aktuellen Gesamtabzug der Adressdatei (ISIL- und Sigelverz.) im Format RDF (N-Triples)

This can be left out IMO, it’s kind of an address book of partner libraries.

https://data.dnb.de/opendata/dnb-all_lds.nt.gz
4.5G
Stabiler Link auf den aktuellen Gesamtabzug der DNB-Titeldaten im Format RDF (N-Triples)

This is bibliographic data, all the books of the German National Library, I think. Would be nice if Qlever could offer that because of its size which makes processing the file on smaller machines difficult, but with lower priority.

https://data.dnb.de/opendata/dnb-all_ldsprov.nt.gz
1.2G
Stabiler Link auf den aktuellen Gesamtabzug Metadatenprovenienz DNB-Titeldaten im Format RDF (N-Triples)

If the bibliographic data is offered, this should also be offered. Also lower priority.

https://data.dnb.de/opendata/zdb_lds.nt.gz
549M
Stabiler Link auf den aktuellen Gesamtabzug ZDB-Titeldaten im Format RDF (N-Triples)

More bibliographic data, magazines only. Also lower priority.

Wouldn't the name dnb be more appropriate for the instance than gnd? The many abbreviations used on the site are really confusing (dnb, gnd, lds, zdb, ...)

Yes, good point. "DNB" (Deutsche Nationalbibliothek) as the data provider makes more sense as a name, especially if you want to add other datasets in the future. People working in GLAM mostly use the GND dataset (and call it "GND") but it would be good to keep the instance more general-purpose and have that reflected in the name "DNB".

Thanks, Paul, that was very helpful indeed.
I have now indexed all the files you listed, except bib_lds.nt.gz because that contains malformed IRIs. Good that you say that it's not important and can be left out. The file dnb-all_ldsprov.nt.gz contains several invalid floating point literals, but QLever has an option to ignore those, which I did.
The instance is now live under https://qlever.cs.uni-freiburg.de/dnb . A few interesting example queries would be welcome (you can just post them in reply to this issue if you have any).
I have also added a Qleverfile for whoever wants to host an instance themselves: https://github.com/ad-freiburg/qlever-control/blob/python-qlever/Qleverfiles/Qleverfile.dnb

@brunnerpaul Does it work for you now?

Works great, thanks a lot!
I'll put together some sample queries and post them here. I have a few queries that I could combine into a single more useful query now because Qlever can just process it in one go.


fix: Check csi health step is not idempotent

Signed-off-by: Liucw<EMAIL_ADDRESS>
What type of PR is this?
/kind feature

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes https://github.com/kubeclipper/kubeclipper/issues/368
Special notes for reviewers:

Does this PR introduced a user-facing change?

None

Additional documentation, usage docs, etc.:

@zhuzhenfan @x893675

Codecov Report

Merging #369 (1a030f3) into master (8cd3320) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #369   +/-   ##
=======================================
  Coverage   12.94%   12.94%
=======================================
  Files         102      102
  Lines       15737    15737
=======================================
  Hits         2037     2037
  Misses      13454    13454
  Partials      246      246

/lgtm

/lgtm

/approve

/cherrypick release-1.3


DatabaseClient connection overhaul

Fixes

Part of reconciliation of https://github.com/Police-Data-Accessibility-Project/data-sources-app/issues/303 and https://github.com/Police-Data-Accessibility-Project/data-sources-app/issues/318

Description

In DatabaseClient, when a method is called, cursor_manager is now used to create a cursor. The cursor is closed and discarded once the method concludes its execution. Before, a cursor object would be created and passed to the DatabaseClient object on initialization and used for as long as the object was used.
A new connection object is opened with each DatabaseClient instance and then closed after use of the instance concludes.
These changes come at a significant cost to execution time while testing, and may be rethought later.

Testing

Checkout the branch
Setup testing environment like normal
Run pytest in the terminal

Performance

Running pytest test_database_client.py increases execution by almost double, 23.63 seconds vs 43.17 seconds
Running pytest increases execution time from 94 seconds to 164 seconds

I'm quite happy with these changes! Excellent work as usual, Kylie.
I did make three changes, and elected to do them myself just to keep things moving quickly and to enable me to utilize this updated database client in some of the new code.
Create Singleton Connection Object
To avoid the overhead of continually creating and closing connections, I updated initialize_psycopg2_connection() to call a Singleton Connection object which always returns the same connection whenever it's called, and which creates a new connection if an existing connection does not exist or is closed.
Consequently, I removed all references to closing the connection object. Now, the connection logic more mirrors how the database client originally was (but with less of the other annoying overhead of passing the connection object.
Modify execute_raw_sql() to always return a list if it has results
The previous design of execute_raw_sql() returned a single result if the results were one, and a list of results if the results were more than one. To test that only one result was in place, type checking was done to ensure the result was a DictRow rather than a list.
This made things slightly more iffy in terms of testing, because it required someone to know that, if only one result was returned, the type would be different. That might pose problems if the type of result returned ever changed.
To simplify things, I elected to have it always return a list of its results, even if there's only one result. I modified tests accordingly so that now, instead of checking the type of the result, they instead check that only one result is returned.
Remove rollback logic
The simplification of connection logic was causing conflicts with tests that involved inserting test data and rolling back afterwards. I eliminated the rollback logic to make tests pass properly. There is now some dead code that needs refactoring, but for now that's a minor issue.

@EvilDrPurple @maxachis nice work!
-does this merit changes in CONTRIBUTING or README about using the DatabaseClient? Something to consider for further SQLAlchemy work, since this is pretty foundational.

does the test data still need to be rolled back? At one point we were using the production database, but that's no longer the case for these tests, right?

* does the test data still need to be rolled back? At one point we were using the production database, but that's no longer the case for these tests, right?

@josh-chamberlain I can at least provide an answer to this. If we use static values in the tests that are inserted into the database, then it would pose a problem for anything with a Unique Constraint on subsequent test runs. However, I think we've changed all the values to be randomized so it may not be an issue anymore!

Heck yeah, tag teaming with @EvilDrPurple 🙏

does this merit changes in CONTRIBUTING or README about using the DatabaseClient? Something to consider for further SQLAlchemy work, since this is pretty foundational.

@josh-chamberlain
Depends on how granular we want to get in the Markdown files. My thinking is that the documentation in the modules themselves should provide the information necessary. While the Markdown files don't exclusively focus on information that can't be easily determined from the app alone, they do generally focus on such topics -- discussing the relation of the project to databases, our general goals, and providing information on where to start in the app.
Additionally, DatabaseClient implementation was not previously mentioned within the markdown files. I will ultimately defer to your judgment on the matter.
Either way, it may be worth making an issue to go through the documentation and add comments and docstrings where needed (might be a good first issue, since a fresh pair of eyes would be most primed to be confused by a lack of documentation).

Nice, I think rolling back after tests is less of an issue, then. Plus, we have means and ways of resetting the test db.
We can hold off on the DatabaseClient documentation, as the name does give people a clue as to where to start. I like the idea of the markdown files commenting on the overall structure, but we can hold off all the same.


New-AzureBatchPool example 2 not working

Hi,
I'm trying to make work the example number 2 of the New-AzureBatchPool powershell command reference page:
https://docs.microsoft.com/en-us/powershell/module/azurerm.batch/new-azurebatchpool?view=azurermps-6.8.1#examples
$imageReference = New-Object -TypeName "Microsoft.Azure.Commands.Batch.Models.PSImageReference" -ArgumentList @("WindowsServer", "MicrosoftWindowsServer", "2016-Datacenter", "*")
$configuration = New-Object -TypeName "Microsoft.Azure.Commands.Batch.Models.VirtualMachineConfiguration" -ArgumentList @($imageReference, "batch.node.windows amd64")
New-AzureBatchPool -Id "MyPool" -VirtualMachineSize "Small" -VirtualMachineConfiguration $configuration -TargetDedicatedComputeNodes 3 -BatchContext $Context
And I'm always receiving the error message: New-AzureBatchPool : Operation returned an invalid status code 'BadRequest'
As example 1 works like a charm, I'm starting to believe that the problem could reside in the object PSVirtualMachineConfiguration . . . anyone knows how to make it work?
Thanks

This is a duplicate issue that was answered on the forums (https://social.msdn.microsoft.com/Forums/azure/en-US/d6960627-4a79-4ba5-a08e-b161824b4f74/newazurebatchpool-reference-page-example-2-not-working?forum=azurebatch).
Let us know if you are running into any other issues
Thanks,
Brian

The linked issue has a workaround.


Text edit tootip arrow is misplaced

Can be seen in the shrine app:

I'd expect the tooltip's arrow to point at the selected text as it does in a native app.

Please add the output of flutter doctor -v.

[✓] Flutter (Channel beta, v0.8.2, on Mac OS X 10.13.6 17G65, locale en-US)
    • Flutter version 0.8.2 at /Users/asd/Developer/flutter
    • Framework revision 5ab9e70727 (4 weeks ago), 2018-09-07 12:33:05 -0700
    • Engine revision 58a1894a1c
    • Dart version 2.1.0-dev.3.1.flutter-760a9690c2

[✓] Android toolchain - develop for Android devices (Android SDK 28.0.3)
    • Android SDK at /Users/asd/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.0, Build version 10A255
    • ios-deploy 2.0.0
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 28.0.2
    • Dart plugin version 181.5616
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[✓] VS Code (version 1.27.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 2.18.0

[✓] Connected devices (1 available)
    • iPhone 5s • A989159F-933B-4CF7-90BE-C530C36DE9DE • ios • iOS 12.0 (simulator)

• No issues found!

#11274 looks related

I seem to have fixed that. ；）
https://github.com/flutter/flutter/pull/31676

Should be fixed now.


Let errors occur when copying the DPU CNI

It is most likely that we will in the future include a command line argument for the CNI path since different cluster types will have different CNI locations.
For now remove the check for "writeable()" and simplify the code to just attempt to copy and make executable. Errors are logged and not act upon.
The writeable() code also had a bug where it would check for the file and not the directory.

/lgtm

2024-04-29T13:54:25Z    INFO    Daemon Init     Daemon init
2024-04-29T13:54:25Z    ERROR   Daemon Init     Failed to prepare CNI binary in /opt    {"error": "open /opt/cni/bin/dpu-cni: read-only file system"}
main.main
        /workspace/daemon/main.go:118
runtime.main
        /usr/lib/golang/src/runtime/proc.go:267
2024-04-29T13:54:25Z    INFO    HostDaemon      starting HostDaemon     {"devflag": false, "cniServerPath": "/var/run/dpu-daemon/dpu-cni/dpu-cni-server.sock"}
I0429 13:54:25.990135  788336 cniserver.go:320] DPU CNI Server creating new router.
I0429 13:54:25.990169  788336 cniserver.go:40] Removing /var/run/dpu-daemon/dpu-cni
I0429 13:54:25.990224  788336 cniserver.go:65] Creating /var/run/dpu-daemon/dpu-cni
I0429 13:54:25.990326  788336 cniserver.go:82] Listen on socket path: /var/run/dpu-daemon/dpu-cni/dpu-cni-server.sock

Logs look like this on a CoreOS based deployment.


next does not preserve the POST params for other routes

I had the following code to check, whether the user has access, until I found out that currently it's impossible to use forms with such an approach
    matchAny (regex "^/admin") $ do
      id <- fmap fst $ sessionState State.get
      case id of
        Nothing -> redirect "/login"
        Just id -> do
          adminAccess <- tx Nothing $ DB.adminAccess id
          unless adminAccess $ raise Unauthorised
          next

Ugh, thanks for reporting. I've got a rewrite of the way request bodies are handled that is about 80% done and should solve this. Unfortunately I'm on another deadline, so not likely to get it finished until next week.

The same here. Just wanted to add some logging for debug purposes.

Hello, is this bug still outstanding? I have a mysterious issue that would be explained by this...

Aha, I just confirmed that using next makes the body of a POST request disappear for subsequent routes :(
This is a big problem for me, did your rewrite ever get landed @xich ?

candidate fix by @thomasjm : https://github.com/scotty-web/scotty/pull/206/files

part of the fix could be to move ActionEnv inside a state-like variable : https://github.com/scotty-web/scotty/issues/131#issuecomment-63485982

Fixed by #308


Update Available

Issue Type: Bug
When execute Update sugerency data studio it was broke. The message error is: A JavaScript Error Occurred in the Main Process.
Azure Data Studio version: azuredatastudio 1.37.0 (d904740d93d7df76a0ba361f20e4351813b57645, 2022-06-14T01:05:12.352Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-10210U CPU @ 1.60GHz (8 x 2112)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.85GB (8.23GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

azuredatastudio-postgresql
Mic
0.2.7

I did to re install the azure data studio and it worked but i have not accepted the update.

@RodrigoPLpz we haven't seen widespread reports of the error message in your original message, so you may try to update again and see if that was a one-off error that occurred.  Alternatively you could also try uninstalling 1.37 and directly install 1.38 to see if that works.  Please let us know if there's anything else we can assist with for this issue.  Thanks!

I somehow came across this error too today when following below steps, would like to note this down:

Click "Restart to Update" when update is available in ADS on Windows.
Let installation start, but terminate the installer instantly before it completes.
Notice "Rolling back changes..." in installer.
Start Azure Data Studio again (fails to start).

Expected: Previous version restored when update cancelled in between.
Actual: ADS gets corrupted and continues to throw the error in screenshot. Re-installing Azure Data Studio doesn't succeed as well as current installation gets corrupted.

Mitigation: Uninstall Azure Data Studio and start installation again.
P.S. I don't think this is a critical issue but it would be nice to see ADS able to rollback changes successfully is installation fails due to any reason.

@cheenamalhotra the error message is saying that the main ADS entry point file is missing.  This is an indication of a corrupted installation.  When doing an update the first step is to uninstall the existing installation.  There could be a bug in the installer where canceling at certain points doesn't properly undo the uninstall of the previous version leaving the system in the state you've described.  We use InstallShield to build the installer and that library handles the rollback operation so it is likely an issue that is upstream.  In this case, the recommended workaround would be to try to do a clean uninstall & reinstall of the desired version.  I'm not sure about the running ADS process but they could be holding file locks that could be interfering with the installer.


Update C# gRPC documentation

This PR updates the C# documentation as per the latest standards described in the Microsoft gRPC guide: https://docs.microsoft.com/en-us/aspnet/core/grpc/?view=aspnetcore-3.1
The steps to generate the gRPC C# classes are much simpler than what was in the old documentation and do not require you to manually compile the proto files.

I think you may have a cleaner PR if you'd create a branch on your fork from master, then pull request from there, but it's ok as is too.

@bhandras can't you squash all the commits in 1 when merging? Haven't used GitHub this much - I have this future in GitLab :D

@bhandras can't you squash all the commits in 1 when merging? Haven't used GitHub this much - I have this future in GitLab :D

Yeah, we could do that too, but we usually don't in order to retain a clean commit structure. It's not super important in this case as it's only docs. Thanks again for the PR :+1:


Hide "CSP directives" section, if no CSP headers is presented

In addition to https://github.com/fs/security-headers/pull/27

:diamond_shape_with_a_dot_inside:


Split the coverage report

Is there any chance we could split the coverage report between the files changed and the whole test suite?

It's not implemented right now, but I'll happily accept a PR that does this.
My first Idea was to have all the changed files come first in the table of files (maybe with a multi-column-span headline), e.g.:

File
Stmts
% Branch
% Funcs
% Lines
Uncovered Lines

Changed Files
changedfile.js100%100%100%100%
changedfile2.js100%100%100%100%
Unchanged Files
unchangedfile.js100%100%100%100%
unchangedfile2.js100%100%100%100%

What do you think? Or would you also require the summary only for the changed files?

I would do both, adding the summary for the changed files and a section for these files in the details as suggested.
In my case at least, we are working on a project which has a huge lack of tests and we want to improve it - so ideally every new PR should have a high level of code coverage in the changed files but mainly not drop the overall coverage of the application!
The goal would be to display how well covered is the upcoming code and still give the dev an overview on how is he improving the overall application test coverage.
Coverage Summary

  Status Category Percentage Covered / Total

 Changed files (this could be enabled by a configuration flag)
 🟢 Lines 91.2% / 90% 450 / 500
 🟢 Statements 91.2% / 90% 450 / 500
 🟢 Functions 91.2% / 90% 450 / 500
 🟢 Branches 91.2% / 90% 471 / 500
 Overall application
 🟢 Lines 84.66% / 81% 1176 / 1389
 🟢 Statements 84.2% / 80% 1194 / 1418   🟢 Functions 83.62% / 80% 286 / 342   🟢 Branches 72.46% / 70% 471 / 650

File Coverage
    The table you suggested here

What do you think?
If you agree I am happy to help! If you have any guide on how to test it, it will be the first time I would be contributing to GH action repo. Should I fork, open a PR in my fork and test the updated action through there?

Hi @bissolli - I still don't have Contribution Guidelines, so I will add these. 😅
For now it's the usual process:

Fork the repository
Create a new branch onto where you develop the feature
Open up a pull request from the branch of your fork

There are some unit-tests included that you can run with npm run test (don't forget to npm install before). They are not super sophisticated yet unfortunately.

If you have any questions feel free to just ask, happy to assist you here! 👍🏻


Version numbers from certs of the cert chain

Hi,
is it possible to extract the Versions of the certs of the cert chain out of a P7C-file?
I must add the certs with the right version who are in the chain of the owners cert to X509StoreFactory.Create("CERTIFICATE/COLLECTION", xxx) if I sign Content.

Ha... solved it. :-)


straw man for cancelling the active draw

Just wanted to start some discussion here. This is an action and command that cancel the active draw.  I've done some testing and noticed a couple of things:

Deleting all the active data doesn't stop the draw as mentioned in slack - this doesn't seem to cause any exceptions, just seems to swallow the next mouse click and then start drawing again
This also deletes the data from the tool under the mouse on hover - apparently those data points are marked as active as well

That said, it seems like both clearAnnotations and cancelActiveDraw deal heavily in the details of cornerstoneTools. I'd like to suggest that we push both of those in to cornerstoneTools.
It'll be really helpful for us to have support for cancelling the active draw so it is probable something we'll be able to do and contribute back if there's interest.
Perhaps something like the clearToolState here: https://github.com/cornerstonejs/cornerstoneTools/blob/7d67e2a944c8fcd29eae58fdb6935eeaff8516f2/src/stateManagement/toolState.js#L116
https://github.com/trustsitka/sitka/issues/3417

Turns out this works alright for most of the tools. The freehand drawing tool really shits the bed when we cancel this way. I'm going to look into adding something to do this to cornerstoneTools today.

I figured out how to cleanly cancel with FreehandMouseTool - call its cancelDraw function after the changes in this PR. You'd also have to go through and make many of its event handlers check for the presence of data before using it.
Going to close this - even with those changes this is a little messy and the FreehandMouseTool has a few "interesting" behaviors like marking all the measurement data on the image active when a draw activity starts.
We may come at this from another angle later.


Update .env

Fixes #126
Use the default nginx by default

Reviewing the issue linked here, I wasn't able to replicate in my local env (macOS Sonoma with M1 chip). I do however experience two other containers exiting after a couple of seconds:
juan@Juans-MBP compas-deployment % docker logs compas-scl-auto-alignment-1
The current machine does not support all of the following CPU features that are required by the image: [CX8, CMOV, FXSR, MMX, SSE, SSE2, SSE3, SSSE3, SSE4_1, SSE4_2, POPCNT, LZCNT, AVX, AVX2, BMI1, BMI2, FMA].
Please rebuild the executable with an appropriate setting of the -march option.The current machine does not support all of the following CPU features that are required by the image: [CX8, CMOV, FXSR, MMX, SSE, SSE2, SSE3, SSSE3, SSE4_1, SSE4_2, POPCNT, LZCNT, AVX, AVX2, BMI1, BMI2, FMA].
Please rebuild the executable with an appropriate setting of the -march option.The current machine does not support all of the following CPU features that are required by the image: [CX8, CMOV, FXSR, MMX, SSE, SSE2, SSE3, SSSE3, SSE4_1, SSE4_2, POPCNT, LZCNT, AVX, AVX2, BMI1, BMI2, FMA].
Please rebuild the executable with an appropriate setting of the -march option.The current machine does not support all of the following CPU features that are required by the image: [CX8, CMOV, FXSR, MMX, SSE, SSE2, SSE3, SSSE3, SSE4_1, SSE4_2, POPCNT, LZCNT, AVX, AVX2, BMI1, BMI2, FMA].
Please rebuild the executable with an appropriate setting of the -march option.%

juan@Juans-MBP compas-deployment % docker logs compas-keycloak-1
Added 'admin' to '/opt/jboss/keycloak/standalone/configuration/keycloak-add-user.json', restart server to load user
-b <IP_ADDRESS>
=========================================================================

  Using Embedded H2 database

=========================================================================

=========================================================================

  JBoss Bootstrap Environment

  JBOSS_HOME: /opt/jboss/keycloak

  JAVA: java

  JAVA_OPTS:  -server -Xms64m -Xmx512m -XX:MetaspaceSize=96M -XX:MaxMetaspaceSize=256m -Djava.net.preferIPv4Stack=true -Djboss.modules.system.pkgs=org.jboss.byteman -Djava.awt.headless=true   --add-exports=java.desktop/sun.awt=ALL-UNNAMED --add-exports=java.naming/com.sun.jndi.ldap=ALL-UNNAMED --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.base/java.lang.reflect=ALL-UNNAMED --add-opens=java.base/java.io=ALL-UNNAMED --add-opens=java.base/java.security=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.base/java.util.concurrent=ALL-UNNAMED --add-opens=java.management/javax.management=ALL-UNNAMED --add-opens=java.naming/javax.naming=ALL-UNNAMED

=========================================================================

08:26:23,538 INFO  [org.jboss.modules] (main) JBoss Modules version 2.0.0.Final
08:26:24,375 INFO  [org.jboss.msc] (main) JBoss MSC version 1.4.13.Final
08:26:24,385 INFO  [org.jboss.threads] (main) JBoss Threads version 2.4.0.Final
08:26:24,527 INFO  [org.jboss.as] (MSC service thread 1-2) WFLYSRV0049: Keycloak 16.1.1 (WildFly Core 18.0.4.Final) starting
08:26:24,706 INFO  [org.jboss.vfs] (MSC service thread 1-4) VFS000002: Failed to clean existing content for temp file provider of type temp. Enable DEBUG level log to find what caused this
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.wildfly.extension.elytron.SSLDefinitions (jar:file:/opt/jboss/keycloak/modules/system/layers/base/org/wildfly/extension/elytron/main/wildfly-elytron-integration-18.0.4.Final.jar!/) to method com.sun.net.ssl.internal.ssl.Provider.isFIPS()
WARNING: Please consider reporting this to the maintainers of org.wildfly.extension.elytron.SSLDefinitions
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
08:26:26,647 INFO  [org.wildfly.security] (ServerService Thread Pool -- 21) ELY00001: WildFly Elytron version 1.18.3.Final
08:26:29,643 INFO  [org.jboss.as.server] (Controller Boot Thread) WFLYSRV0039: Creating http management service using socket-binding (management-http)
08:26:29,670 INFO  [org.xnio] (MSC service thread 1-2) XNIO version 3.8.5.Final
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00007ffffe330d6e, pid=193, tid=221
#
# JRE version: OpenJDK Runtime Environment 18.9 (11.0.14+9) (build 11.0.14+9-LTS)
# Java VM: OpenJDK 64-Bit Server VM 18.9 (11.0.14+9-LTS, mixed mode, sharing, tiered, compressed oops, g1 gc, linux-amd64)
# Problematic frame:
# V  [libjvm.so+0x670d6e]  DirectivesStack::getMatchingDirective(methodHandle const&, AbstractCompiler*)+0xfe
#
# No core dump will be written. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# An error report file with more information is saved as:
# /tmp/hs_err_pid193.log
#
# If you would like to submit a bug report, please visit:
#   https://bugzilla.redhat.com/enter_bug.cgi?product=Red%20Hat%20Enterprise%20Linux%208&component=java-11-openjdk
#
*** JBossAS process (193) received ABRT signal ***


Options not working in athenapdf-service v3

I tried to pass options to the athenapdf-service:3 but it seems that either my query is wrong or it doesn't work properly.
My request looks like this:
curl -F<EMAIL_ADDRESS>-F process='{"conversion": {"mime_type": "text/html", "options": {"scale": {"double_value": 2}}}}' http://localhost:8080/process > output.pdf

The cmd version options work without issues.

Thanks for reporting this @almini. Looks like the service does not handle jsonpb. I have a fix which I will push up shortly. You should also alter your process JSON string to look like:
{"conversion": {"mime_type": "text/html", "options": {"options":{"scale":{"doubleValue":2}}}}

If you pull the latest image, it should work as expected now (bearing in mind the example I gave):
docker pull arachnysdocker/athenapdf-service:3


Swift 5 warnings fixes

Handle Switch Swift 5 Warnings  (Added @unknown default)
Fixed Example CustomAnimationViewController  (added default values in translation and rotationAngle)

should we close this PL or merge to master?


Update Flyway to 5.0.7

This should be trivial as the package structure hasn't changed.

Latest version is https://github.com/flyway/flyway/releases/tag/flyway-5.0.7
Will you upgrade please ?

I updated to the latest version finally.

Thanks.


Replaces MemorySizer and CompileConfig with RuntimeConfig

We formerly introduced MemorySizer as a way to control capacity independently of size. This was the first and only feature in CompileConfig. While possibly used privately, MemorySizer has never been used in public GitHub code.
These APIs interfere with how we do caching of compiled modules. Notably, they can change the min or max defined in wasm, which invalidates some constants. This has also had a bad experience, forcing everyone to boilerplatewazero.NewCompileConfig() despite that API never being used in open source.
This addresses the use cases in a different way, by moving configuration to RuntimeConfig instead. This allows us to remove MemorySizer and CompileConfig, and the problems with them, yet still retaining functionality in case someone uses it.

RuntimeConfig.WithMemoryLimitPages(uint32): Prevents memory from growing to 4GB (spec limit) per instance.

This works regardless of whether the wasm encodes max or not. If there is no max, it becomes effectively this value.

RuntimeConfig.WithMemoryCapacityFromMax(bool): Prevents reallocations (when growing).

Wasm that never sets max will grow from min to the limit above.

Note: Those who want to change their wasm (ex insert a max where there was none), have to do that externally, ex via compiler settings or post-build transformations such as wabin

rewrote the description based on feedback from @inkeliz which clarified the original intents.

thanks @inkeliz for helping clarify and @anuraaga for the review


TRADING STRATEGIES

https://grainphd.com/wp-content/uploads/2017/07/Options.pdf ARTICLE1

Need more resources


tooltip-popup-delay causes weird placement behaviour in heavy documents

I have noticed that when using a tooltip-popup-delay different than 0 in a heavy document (for instance >30k divs), the tooltip is displaced in the right/bottom compared to its normal position, but for a very short moment.
If I remove the delay, there is not this problem anymore (even if the document is still slow).
Here's an example (it takes a few seconds to display due to heavy div rendering): http://plnkr.co/edit/MOBT1jduhCR7Cyvw1aW7?p=preview

It occures on Chrome and IE but not on Firefox.

It is actually fixed with the 1.3.12 of Angular. Basically, the rendering lags a lot less so it might explains...


Add animation for de-whitelisting a site

Reviewer: @andrey-p

Description:
Following up from previous work in #158, also add an animation for when a site is removed from the whitelist via the main popup toggle.
The animation behaves exactly like the "Added to Whitelist" one, but the text is "Removed from Whitelist"
Steps to test this PR:

Build and reload extension
Try whitelisting a site - a brief transition showing the text "Added to Whitelist" should appear and then tab should reload
Try de-whitelisting a site - the same transition should appear, but with the "Removed to Whitelist" text instead, and a shield icon

Automated tests:

[ ] Unit tests
[ ] Integration tests

Reviewer Checklist:

[ ] Ensure the PR solves the problem
[ ] Review every line of code
[ ] Ensure the PR does no harm by testing the changes thoroughly
[ ] Get help if you're uncomfortable with any of the above!
[ ] Determine if there are any quick wins that improve the implementation

PR Author Checklist:

[ ] Get advice or leverage existing code
[ ] Agree on technical approach with reviewer (if the changes are nuanced)
[ ] Ensure that there is a testing strategy (and documented non-automated tests)
[ ] Ensure there is a documented monitoring strategy (if necessary)
[ ] Consider systems implications

Something was still preventing the opacity animation from running. Took me far too long to spot it, a comma had gone missing.
Hope you don't mind me just jumping in and fixing it!


Adds paginagtion, creates more spacing variables and introdces better namens

@srambach @cshinn @jgiardino @bleathem Can you guys please review this one?
I've change the names of some files to better structure what we are showing.
I've also created the pagination components to test out the spacing variables.
If you like this PR I recommend merging it before start working on the repo and follow the same naming conventions for the rest of the components.
Thanks a lot! 😄

You have button.mustache and 01-button-styles but 02-buttons-sizes. Should it be 02-button-sizes.mustache instead?

RE: button vs buttons, I think it makes more sense for us to refer to our basics in the singular

@cshinn @srambach thanks for the review, I've change the names

LGTM

Looks good

Merge it! @cshinn do you have merge rights to this repo?

Nope!

@cshinn congrats, you are now a core member of patternfly


Support docker 18.09+ for user docker

Docker 18.09 has some changes:
- containerd
- containerd-shim
- ctr
- docker
- dockerd
- docker-init
- docker-proxy
- runc

For 18.06:
- docker
- docker-containerd
- docker-containerd-ctr
- docker-containerd-shim
- dockerd
- docker-init
- docker-proxy
- docker-runc

The binary naming convention has been modified, we need to be compatible with it.

RancherOS v1.5.1-rc1 already supports docker-18.09.0 and docker-18.09.1
It works fine in my environment


Add retries to all commissioning paths

Not all commissioning paths had some retries applied. Now we retry both commisisoning and first interview 3 times on both the code and on network commissisioning.

The main goal is to resolve commissioning issues such as observed with Meross MSS315 plugs. It seems that they fail at certain commissioning step, e.g. in this case 18:
2024-01-04 14:55:51 core-matter-server matter_server.server.device_controller[212] INFO Starting Matter commissioning with code using Node ID 16.
2024-01-04 14:56:27 core-matter-server chip.CTL[212] ERROR Failed to perform commissioning step 18
2024-01-04 14:56:27 core-matter-server matter_server.server.client_handler[212] ERROR [549496677840] Error handling message: CommandMessage(message_id='85292457bd0746e09707e81ae14e8709', command='commission_with_code', args={'code': '10613221685', 'network_only': True})
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/matter_server/server/client_handler.py", line 188, in _run_handler
    result = await result
             ^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/matter_server/server/device_controller.py", line 196, in commission_with_code
    raise NodeCommissionFailed(
matter_server.common.errors.NodeCommissionFailed: Commission with code failed for node 16
2024-01-04 14:56:27 core-matter-server chip.EM[212] ERROR Failed to send Solitary ack for MessageCounter:7068048 on exchange 59541i:src/messaging/ExchangeContext.cpp:103: CHIP Error 0x00000002: Connection aborted

According to the SDK this step is the step where the NOC is added to the device (see https://github.com/project-chip/connectedhomeip/blob/v<IP_ADDRESS>/src/controller/CommissioningDelegate.h#L33-L72, kSendNOC).
There isn't much we can do if a device just disappears during commissioning other then retrying :cry:

Yeah this is a workaround for what appears to be a device (or maybe other device have this as well?) that is just a bit unstable. By adding some retries we increase the chances of a successful commissioning. May also help with the famous Nanoleaf devices to get them onboarded.


spy_messages_to_l1 tests implementation

Stack:

#2284
#2283 ⬅
#2281
#2280

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

✓ Commit merged in pull request #2284


[DOC] update channel and output part

Hi
I used ulog in a AMD environment with an high order channel which override the output channel if config highorder_log level get reached.
ulog is a really nice project for such extending stuff. :muscle:
Would be nice to have some hints in the documentation. So i rewrote the channel an output part cause the channel configuration is part of channel and not the output. Output doesn't get configured at all :wink:
Hope this helps.
Channels
In ulog, logging is always "sent" to exactly one channel. The channel get selected by the configuration. It can also be selected by extending ulog with a mod. (see Custom channel selection )
A channel has one or more configurable outputs that can optionally apply formatting to the message.
By default, two channels exist:

output, for messages of loggers that are in debug mode, or that are at an enabled level
drain, for those messages that are filtered away.

Channel selection
By having a separate channel for the drain, there is the opportunity to use another output than noop(link). So you can configure where discarded messages are going.
For example: send all logging records to a database and filter it later, when it get display for example (see output)
By default all log methods on a logger are associated with one of two channels, output and drain. When the logger is created, each log method is sent either to the output channel, or to the drain channel, based on the current log level for that logger and whether that logger is in debug mode.
To configure the drain / output channel selection the following log config can be used:
log := log_setting [; log_setting] ...
log_setting:= [ logger_name [, logger_name] ... = ] log_level
log_level:= [all | error | warn | info | debug | trace | none]
logger_name:= [ * | [-]{regular expression} ]

Custom channel selection
If the default behavior is not enough the channel selection in ulog can easily be extended like followed:
ulog.use({
    after: function (logger) {
        for (var level in this.levels) {
            var channel = logger.enabledFor(level) ? 'output': 'drain';
            // calculate new channel
            logger[level] = logger.channels[channel].fns[level];
        }
    },
});

Output configuration
The channel output, where messages are going, is completely configurable at runtime.
To configure the outputs for a channel, use the log_ config option. The default channels config options are:

log_output, defaults to console
log_drain, defaults to noop

To configure the output for a channel, we assign the name of the output to use to the relevant loggers:
log_output=console

This setting can include expressions to target individual loggers, just like the debug and log settings:
log_output=console;noisy-lib:*=noop

The value part is actually a kurly format string. The same syntax can be used here as for configuring formatting.
If more than one output is specified, a multiplex function will be inserted that dispatches the logging to all specified outputs.
log_output=console custom_output

Syntax:
log_channel := channel_setting [; channel_setting] ...
channel_setting:= [ logger_name [, logger_name] ... = ] output_name [ output_name] ...
logger_name:= [ * | [-]{regular expression} ]
output_name:= defined outputs in ulog

Hint: The first match get used. Except the default logger_name (*)
Outputs
Outputs are used in channels to peform the actual task and "log" the data.
By default, the following outputs are included:
Output console
...
Output noop
...

@Messj1 Wow another great issue! Thanks! You really invested time in this. That is so great. Thank you.
May I ask have you ever contributed to an Open Source project before? Would you like to give it a shot?
I can learn you how to make a pull request and how to contribute your changes to this project. Let me know if this is something you would like to try.

@Download What do you think about a mini tutorial with the high_order channel mods i wrote as an example:
https://github.com/FriendsOfTYPO3/frontend_editing/blob/master/Resources/Public/JavaScript/Utils/LoggerPersist.js

Create highorder ulog channel.
Creates 2 ulog outputs to persist

persist: with indexedDb in use
server: calling the sendCallback function

@Messj1 Yes that's also an option.
But any changes to the code should happen via Pull Requests. This is why I asked you to create one :)


Table 38 "Purchase Header" - Procedure LookupBuyFromContact from internal to global

Describe the request
Can the procedure please be set from internal to global.
table 38 "Purchase Header"
...
procedure LookupBuyFromContact()

Additional context
We would like to use this functionality in our document pages.
Internal work item: AB#556660

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 26910.


Provide a setter for layers in Sequential and Functional.

The Model class has a setter for layers to provide a more descriptive message on why this property cannot be changed. But it only applies to sub-class models as classes that override layers don't inherit it.
This adds a similar message for functional and sequential models.

Codecov Report
Attention: Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.

Project coverage is 57.43%. Comparing base (9dbbc72) to head (15a80ca).

Files with missing lines
Patch %
Lines

keras/src/models/sequential.py
66.66%
1 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (9dbbc72) and HEAD (15a80ca). Click for more details.
HEAD has 6 uploads less than BASE

Flag
BASE (9dbbc72)
HEAD (15a80ca)

keras
4
1

keras-torch
1
0

keras-tensorflow
1
0

keras-jax
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #20275       +/-   ##
===========================================
- Coverage   78.91%   57.43%   -21.49%
===========================================
  Files         509      509
  Lines       48277    48283        +6
  Branches     8883     8883
===========================================
- Hits        38099    27731    -10368
- Misses       8345    18775    +10430
+ Partials     1833     1777       -56

Flag
Coverage Δ

keras
57.43% <83.33%> (-21.35%)
:arrow_down:

keras-jax
?

keras-numpy
57.43% <83.33%> (+<0.01%)
:arrow_up:

keras-tensorflow
?

keras-torch
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Pip errors fix

I get some errors like subliminal failed to install and found the issue, its the Pip.
Error:
ImportError: cannot import name IncompleteRead
Solution:
sudo apt-get remove python-pip
sudo easy_install pip
Now subliminal was successfully installed

My install log from osmc 2016
https://docs.google.com/document/d/1q6HrPu42xw63_nRhxQGm4oL4bXkSIHJQb_b0gqSXxpA/edit?usp=drive_web


unable to install Eigen and Protobuf in step 2

Thank you so much for the tensorflow-cmake project you create.
I download the Eigen source code eigen-eigen-f562a193118d.tar.gz, decompression it and put it under the directory  /home/minditar/tensorflow.
Then use the command
sudo bash eigen.sh install /home/minditar/tensorflow
then error occurs like:
Finding Eigen version in /home/minditar/tensorflow using method 0...
Finding Eigen version in /home/minditar/tensorflow using method 1...
Finding Eigen version in /home/minditar/tensorflow using method 2...
Failure: could not find Eigen version in /home/minditar/tensorflow
I follow the pattern sudo eigen.sh install <tensorflow-root> [<install-dir> <download-dir>] to change the command into the followings:
sudo bash eigen.sh install '/home/minditar/tensorflow' '/home/minditar/tensorflow/eigen-eigen-f562a193118d'
sudo bash eigen.sh install '/home/minditar/tensorflow' '/usr/local' '/home/minditar/tensorflow/eigen-eigen-f562a193118d'
sudo bash eigen.sh install '/home/minditar/tensorflow' '/home/minditar/tensorflow/eigen-eigen-f562a193118d/Eigen'
sudo bash eigen.sh install '/home/minditar/tensorflow' '/usr/local'  '/home/minditar/tensorflow/eigen-eigen-f562a193118d/Eigen'
They all throw the same error like the above and can not find the Eigen. While I actually put it under the directory.
While I also don't know how to figure out the version of  Eigen and Protobuf  the current tensorflow code needs.

Hello, and thank you for your question.  If all things function as expected, you should never need to manually download Eigen; this is the goal of eigen.sh.  The <tensorflow-root> directory  required by the script is the root directory of the TensorFlow repository itself.  Therefore, all you need to do is clone the TensorFlow repository into some directory and pass that directory into your eigen.sh.
For example, let's say I clone Tensorflow inside /home/cjweeks/git; this will create a /home/cjweeks/git/tensorflow directory which contains the new repository.  To call the install script, simply provided it with this location: sudo ./eigen.sh install /home/cjweeks/git/tensorflow [other-options-here].  The same process can be followed for Protobuf.
You never have to determine which versions of Eigen and Protobuf TensorFlow uses; the scripts do that for you.  All you need is the TensorFlow repository for the scripts to reference.
In review, the arguments for eigen.sh in the install configuration are as follows:

The first arguemnt is install, and this signals the script that you wish to install Eigen.
The second is the root of the TensorFlow directory, as discussed above; this is required.
The third is an optional install path, which defaults to /usr/local.
The fourth is an optional path in which Eigen is downloaded and extracted; this defaults to the current directory.

If you have any more questions, do not hesitate to post here!

I try the method you provide again used the command
./protobuf.sh install '/home/minditar/tensorflow'
and still get the error,
Failure: Could not find all required strings in /home/minditar/tensorflow
When I use old version of tensorflow like tensorflow-0.9.0.zip, the error changes to
Command failed - script terminated

The first problem is strange; when I pull the latest version of TensorFlow and execute the script with similar parameters, it proceeds with no errors:

What is contained in your /home/minditar/tensorflow directory?  This should be the TensorFlow root directory, and should have contents similar to the following:

Is this what the directory structure looks like?  Additionally, these scripts will not work correctly with older versions of TensorFlow, as TensorFlow has changed the way it formats some dependencies.

I downloaded the tensorflow-0.12.0.zip file from https://github.com/tensorflow/tensorflow/releases rather than use files in https://github.com/tensorflow/tensorflow page. And I compile the source code again. Using the command you provide, it works now. The difference that can be easily known is that the site  https://github.com/tensorflow/tensorflow doesn't include jpeg.BUILD file. Maybe there are more differences between the files in the two sites.
And the new version of tensorflow doesn't have avro  in its third_party packages. So you may need small change.
Thanks so much for your help. At last, the problem is not caused by your project or command while due to unknown reasons related to tensorflow.

Thank you for the information; I will look into the possible release differences.  I have also slightly altered the instructions for removing the specified third-party directories as you indicated.  I am closing this issue for now, but if you have any more questions, you may re-open it or open a new issue.

I also ran into this problem.
The solution I found was to replace the line in eigen.sh
ARCHIVE_HEADER="native.new_http_archive\(\s*"
with
ARCHIVE_HEADER="tf_http_archive\(\s*"


camera_models.txt doesn't have proper directory matching structure

Describe the bug
The lines in camera_models.txt don't match the directory matching structure. For example:
__camera_models_acer__
__camera_models_agfaphoto__
__camera_models_benq__
__camera_models_canon__
__camera_models_casio__
__camera_models_concord__

Shouldn't these be:
__camera_models/acer__
__camera_models/agfaphoto__
__camera_models/benq__
__camera_models/canon__
__camera_models/casio__
__camera_models/concord__

Version used
c7434e7
To Reproduce
Steps to reproduce the behaviour, include the prompt you used if applicable:

When you use camera_models it doesn't work correctly, and shows errors in the terminal that the models can't be found.

Also mention which checkboxes have been set.
Expected behaviour
Using the forward slash after camera_models in the txt file allows the script to find the models in the subdirectory.
Additional context
Unless I'm not using it right...

Thanks for that - just uploaded a fix

Same thing with photographers.txt. Doesn't it need to have by_country/ before each country name?


Data Science with Python : Digit Recognizer

Description
Welcome to the 'DSWP' Team, good to see you here.
With this issue, the readers will know to use computer vision with famous MNIST data to recognize the digits
Here is the dataset to follow : https://www.kaggle.com/c/digit-recognizer/data
To get assigned to this issue, add your serial numbers mentioned in the spreadsheet of "Data Science with Python", the approach one would follow and the choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
If you had referred any resources, add them up in "DS Resources". Similarly, if you had used datasets, include them in "DS Datasets".
Domain: Kaggle Competitions
Mentors Assigned : Peehu Saxena | Ankit Kumar
Points to Note :
The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title" should be the same. Include issue number along with it.
Changes should be made inside the Datascience_With_Python/ directory & Datascience_With_Python branch.
Follow Contributing Guidelines & Code of Conduct before start Contributing.
This issue is only for 'GWOC' contributors of 'DSWP' domain.
All the best. Enjoy your journey ahead. 😎
Domain
Data Science with Python
Type of Contribution
Audio, Video, Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

Vishva Rana
Batch -15
Serial - 495
Type - Documentation
Approach : Building a CNN using mnist dataset that predicts the probability of digits in a list and finding the maximum of them to predict which digit it resonates the most

Already an issue is assigned to you. @vishvarana

/assign
Divya Chhoriya
Sr no. 444
Type: documentation
Batch 12
Build CNN to recognize MNIST digits

@prathimacode-hub Kindly assign me this issue

Issue assigned to @divya-chhoriya for documentation

@prathimacode-hub , Mam
I am M. Ashish Reddy. I would like to do video on this issue.
Batch number: 5
serial number:174.
Approach: A detailed explanation with examples and real life uses

Issue assigned to @ashish-reddy-20-08 for video contribution


Document IRC-specific commands.

Is your feature request related to a problem? Please describe.

IRC-specific commands are missing help/documentation or even a way for the user to know of their existance.
Describe the solution you'd like

I would like commands like /raw, /join, /part, /msg, /away, /knock, /kick, /wallops to be documented.
Describe alternatives you've considered

There's no real alternative to consider here apart from removing the commands which is not a good one.
Additional context

Some commands like /who are provided by the IRC server and the client does not know if the server supports them.
After further research I found that RPL_ISUPPORT (005) returns a list of commands supported by the server and thus could be used to show the users that the commands exist.

Twitch uses a very stripped down interpretation of the IRC protocol, so I don't see the point of it. Most of the commands you listed don't exist here - not on Twitch IRC and not on chatterino.
Also I have yet to see an IRC server that sends you available commands with the 005 numeric (or with any other)- Twitch certainly doesn't. I don't know where you got that from.

IRC support was removed
F


bonescript is broken

Bone101 and the BoneScript documentation try to load http://HOST/bonescript.js (where HOST is whatever you fill into the form, or <IP_ADDRESS> by default).  However, on the latest IoT image this results in a 404 error, thus the ability to run examples does not function.
The getting started guide's networking section likewise tries (and fails) to load http://HOST/bonescript.js.  It also tries to load http://HOST:8000/bonescript.js, which causes bonescript.service to be socket-activated on my beaglebone.  However, it seems to fail to adopt the socket provided by systemd and instead tried to open its own socket, which fails (EADDRINUSE).
I haven't tried to debug this issue, but I did notice that the "systemd" module bonescript uses for socket activation does not support nodejs 0.11.0 or later (according to its package.json), presumably because of the hideous hacks it does (uses a nodejs-internal binding and globally patches net.Server.prototype.listen).  You may want to consider replacing it by my node-sd-daemon library, which supports socket activation and a lot more, works on all recent nodejs versions (afaik) and is actually maintained.  I haven't gotten around to pushing it to npm yet, but you can just use the git version directly (npm install github:dutchanddutch/node-sd-daemon).

@jadonk what do we really want to do with Bonescript?  I know you've been transiting many demos to 'fs' over 'bonescript'...
https://github.com/beagleboard/cloud9-examples/commit/b135461b903423c8d0279de68698a4a5398b8d03
https://github.com/beagleboard/cloud9-examples/commit/950e02c0d0be04c355e763cbbdb320349f3aad08
I know you mentioned one the big problems with bonescript, i that it is just so slow on startup..
Do we kill it, is there anything useful, that hasn't been converted to cloud-examples repo?
Regards,

I fixed the loading time, mostly. I just want to get more useful bug reports and community fixes.  Vaishnav fixed a lot of the on usage front. If the server security can be enabled, I’d be okay to leave and support it...if people actually say they want it.

I hope you're doing an origin-check to ensure only beagleboard.org and the beaglebone itself can connect a websocket?

I hope you're doing an origin-check to ensure only beagleboard.org and the beaglebone itself can connect a websocket?

Not at all.

Ah, so any website can control any beaglebone on the visitor's network?  Great feature :-)

@mvduin it's worse then that..  Give me an ip address of a on world accessible Beagle..
http://beagleboard.org/updater
Regards,

who on earth would have a beagle with a world accessible IP address?  If you do that without properly securing your beaglebone you kinda have it coming.  The fact that any website can access beaglebones on any visitor's local network is far, FAR worse.


hosts.toml file resolution does not seem to account for default port being ommitted

Description
nerdctl login https://foo:443
vs.
nerdctl login https://foo
If the hosts.toml path contains the port, the second one will fail.
On the other hand, if it does not, the first one will fail.
Responsibility for walking the host dir is in dockerconfigresolver.go (NewHostOptions > HostDir()).
This does not try to be smart wrt what the host contains (port or no port).
We should patch this so:

if scheme is https, passing :443 or not passing it should work
all of this should work with hosts.toml files path containing the port or not

Steps to reproduce the issue
See description
Describe the results you received and expected
Fail / not fail.
What version of nerdctl are you using?
1.7.6
Are you using a variant of nerdctl? (e.g., Rancher Desktop)
None
Host information
No response

Seems we need rework the whole login and auth part = =

Seems we need rework the whole login and auth part = =

Yes, that is what I am thinking.
I am on it.
Will start with revamping testing.

nerdctl login https://foo:443
vs.
nerdctl login https://foo

What's the behavior in docker? e.g., if you docker login https://foo:443, can you docker pull an image from https://foo (and vice versa)?

nerdctl login https://foo:443
vs.
nerdctl login https://foo

What's the behavior in docker? e.g., if you docker login https://foo:443, can you docker pull an image from https://foo (and vice versa)?

Docker does not support hosts.toml, so, it does not seem like the above would matter here?
Anyhow, here is a tentative proposal to clarify behavior wrt --insecure-registry, hosts.toml, and credentials:
https://github.com/containerd/nerdctl/pull/3417


Methods to speed up overall throughput and Lower Latency w This + WG

Hello - I am still testing ref to IPv6 but i do have a few links up with IPv4 of this for testing right now, its 13% as fast as native WG - so as such i am wondering how you would suggest should i wish to mod this myself to speed it up?
Multithreading on HTTPS side ?
Offloader Cards (Which ones - AES ?)
What else would speed this up & lower latency? I am assuming if i was to try a solid video stream it may become problematic / buffer often ? LEt's say i had a prod use for this requiring this as a larger volume, how would this scale ?
Also would this pass DPI / Deep Packet Inspection and if not, why?

Hello,
Dont think you can. You are running wireguard over udp over websocket over tls over tcp, so at some point you going to take a hit in througput.
you encrypt twice every packet. Once with wireguard and an other one with tls.
Every udp packet data is going to be masked because of websocket requirement (and after unmasked server side), and every udp packet is going to map to a websocket frame which is a lot more heavy weight than sending raw data over udp.
Wgile this being said, you should profile client and server with perf in order to check where is the bottleneck (gc, syscall, cpu) to check what can be done.
Last time i checked for tcp (not udp) it was mostly the gc which was the culprit.
But may not be the same for wireguard and udp

Thanks for the detailed info - is there a way that I could take encapsulation prod that's lighter and would pass DPI ? Also, would this pass DPI ? How Many connections would it handle before toppling over / becoming too much overhead ? Does it not scale whatsoever ? At current notably I route this via an NGINX ingress as i have a lot on port 80, i doubt highly nginx is the bottleneck... by GC, i assume you mean garbage collection? It is slow to kick off stale connections?

Also, in ref to prod, i mean being able to offer a few VPN connections across this for friends and family in places with difficult internet connections censorship wise, hence asking what the breaking point would be. I want to wrap WSTUNNEL in a mobile app with WG as well. Wireguard is easy to detect FW wise / DPI so i am simply trying to mask it. Is there a better way to mask this / more lightweight that will pass firewalls in odd areas of the world :) ?

If it is just a few connections for friends, well you will not have any issue with wstunnel.
Most likely from within a censored country your outbound traffic will be very much limited, so wstunnel/your remote server will not be the limiting factor.
The most lightweight alternative to wirehuard, would just be to use wstunnel at client side directly. Just use the socks5 -D option and ask your friend to configure their browser to use it.
It will way better than trying to encapsulate wireguard

I am also using this tutorial  for wireguard over wstunnel. But the performance is not stable. I know my bandwidth is not the issue because at the same time that I have low download speed through wstunnel I have a decent one through a v2ray proxy connection.
Is there any things that I could do for better performance?
My setup is like this:
WG server on localhost:51822 <-> Wstunnel on server from localhost:51822 to <IP_ADDRESS>:8443 <-> Wstunnel on Client from Server_IP:8443 to localhost:51822 <-> WG client with Endpoint=localhost:51822

I understand, my server has only 1 CPU core so I decided to use OpenVPN (TCP) on wstunnel. So far Performance is stable and haven't experienced any latency spikes.


Fixing saucelabs merge issue

What is the context of this PR?
There was a merge problem with #334 and these went missing. This change fixes them.
See here: https://github.com/ONSdigital/eq-survey-runner/pull/334/files
How to review

Check the tests pass

[ ] Do all commits have sensible commit messages?
[ ] Is all new code unit tested?
[ ] Are there integration tests if needed?
[ ] Is there sufficient logging?
[ ] Is there documentation or is the code self-describing?

Who worked on the PR
Describe who worked on the changes, so that other people can review.

LGTM :+1: Will merge once status checks pass


Format "Creating Feign Clients Manually" snippet

@rafaelrenanpacheco Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@rafaelrenanpacheco Thank you for signing the Contributor License Agreement!

Codecov Report

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

@@             Coverage Diff              @@
##             master      #29      +/-   ##
============================================
- Coverage     76.24%   76.07%   -0.17%
  Complexity      283      283
============================================
  Files            36       36
  Lines          1183     1183
  Branches        183      183
============================================
- Hits            902      900       -2
- Misses          204      206       +2
  Partials         77       77

Impacted Files
Coverage Δ
Complexity Δ

...bbon/HttpClientFeignLoadBalancedConfiguration.java
90.32% <0%> (-6.46%)
2% <0%> (ø)


".profiles/default/auth.json" (OnlyFans Downloader made updates)

The OnlyFans downloader as changed how the settings are organized and what is grabbed. Is it possible to get an update to account for the latest version of the downloader?

I suppose I should, but the extension isn't "missing" any information necessary so I've been kinda lazy about.
The downloader software will fill in anything extra "filler values" (since they're not being used).
Here's a default auth.json created on a first-time start of the software itself:
{
  "auth": {
    "username": "",
    "auth_id": "",
    "auth_hash": "",
    "auth_uniq_": "",
    "sess": "",
    "app_token": "33d57ade8c02dbc5a333db99ff9ae26a",
    "user_agent": "",
    "email": "",
    "password": "",
    "support_2fa": true,
    "active": true
  }
}

Here's an example JSON that the cookie helper extension gives (with censored information):
{
  "auth_hash": "",
  "auth_id": "<REDACTED>",
  "auth_uniq_": "",
  "sess": "<REDACTED>",
  "app_token": "33d57ade8c02dbc5a333db99ff9ae26a",
  "user_agent": "Mozilla/5.0 (X11; Linux x86_64; rv:88.0) Gecko/20100101 Firefox/88.0",
  "support_2fa": true,
  "username": "<REDACTED>",
  "active": true
}

All you really have to do, is replace the object that comes after "auth":  with the one from the cookie helper. Should look like this:
{
  "auth": {
    "auth_hash": "",
    "auth_id": "<REDACTED>",
    "auth_uniq_": "",
    "sess": "<REDACTED>",
    "app_token": "33d57ade8c02dbc5a333db99ff9ae26a",
    "user_agent": "Mozilla/5.0 (X11; Linux x86_64; rv:88.0) Gecko/20100101 Firefox/88.0",
    "support_2fa": true,
    "username": "<REDACTED>",
    "active": true
  }
}

Once you run the downloader software, it will update the auth.json to contain the missing filler values:
{
  "auth": {
    "username": "<REDACTED>",
    "auth_id": "<REDACTED>",
    "auth_hash": "",
    "auth_uniq_": "",
    "sess": "<REDACTED>",
    "app_token": "33d57ade8c02dbc5a333db99ff9ae26a",
    "user_agent": "Mozilla\/5.0 (X11; Linux x86_64; rv:88.0) Gecko\/20100101 Firefox\/88.0",
    "email": "",
    "password": "",
    "support_2fa": true,
    "active": true
  }
}

I'll leave this issue open though, since ideally the cookie helper extension should wrap it inside the "auth" object by default.

Version 1.0.3 has been released that addresses this: https://github.com/M-rcus/OnlyFans-Cookie-Helper/releases/tag/1.0.3


Max header size

Title: Why the 60K cap?
Description:
I'm curious about this comment

Check for maximum incoming header size. Both codecs have some amount of checking for maximum header size. For HTTP/1.1 the entire headers data has be less than ~80K (hard coded in http_parser). For HTTP/2 the default allowed header block length is 64k. In order to have generally uniform behavior we also check total header size here and keep it under 60K. Ultimately it would be nice to have a configuration option ranging from the largest header size http_parser and nghttp2 will allow, down to 16k or 8k for envoy users who do not wish to proxy large headers.

The min() of 80K and 64K is 64K. Can you elaborate on what "generally uniform behavior" is? I'm not sure where this reduction of 4K is coming from. Any reason not to use 64K?

cc @auni53 for comment who recently worked in this area.

Significant changes have occurred in this area. If you look at my latest PRs, the limit is now configurable up to a max of 96 KiB. I can't speak for when that change will be put out, but it should make it into the next release whenever that is.
As for why the conservative approach for limiting? It's an attack vector, so it makes sense to only support what we're 100% won't break.

As for why the conservative approach for limiting? It's an attack vector, so it makes sense to only support what we're 100% won't break

And subtracting 4K gave that 100% guarantee? Not trying to be pedantic, btw. I have a need to raise it in a previous version of envoy so I'm trying understand the rationale and if I'm going to break anything.

Ah, I have no idea or justification for the 60K limit. If you want to raise it on this release you should probably just cherrypick my PRs; I don't remember exactly what breaks when and what needs to be configured.

Thanks for the help @auni53!

Is Max header size is configurable is any recent release?

Yes it is . Please check max_request_headers_kb

@ramaraochavali  any example how to set it?

unfortunately I could not find an example. But you have to set on http_connection_manager config


⚠ Bofrost.be has degraded performance

In 123f5ae, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 5448 ms

Resolved: Bofrost.be performance has improved in 1494abe after 4 minutes.


rename ggdsl to kandy

upgrade kandy v0.3.2 -> v0.4.0-dev-16

Sorry, I mentioned you on review when it already merged. Can't delete(


Change the incorrect Course Offering related list label to "Course Offerings"

Critical Changes
Changes
The Course Offerings related list on the Time Block record is now correctly labeled "Course Offerings."
Issues Closed
Fixes #922
New Metadata
Deleted Metadata
Testing Notes

Create a new Course Offering and specify a Time Block
Navigate to the Time Block
Click on the Related List tab
You will see a list of "Time Blocks" that are actually Course Offerings (you may need to add it to the Time Block page layout)

Tested / Passes QE. @juliesnow these Release Notes look good to me!

@juliesnow FYI, this looks like it still needs your review, when you get a chance this week to review. Thanks!


I am getting this in Safari: "Failed to load resource"

I am using Safari 5.1.7 on Windows 8.1 and I am getting:
Failed to load resource: The requested URL was not found on this server. pointing to file:///soundmanager2_debug.swf
How do I resolve this?

Does it work in other browsers on Windows 8.1?

Works pretty good in Chrome and Firefox.

I don't use Safari much. So am curious, does it support flash (SWF)?
Otherwise the issue might be Safari requiring some extra confignto be
provided in order to load the file.
On 12 January 2015 12:28:15 Rahul Desai<EMAIL_ADDRESS>wrote:

Works pretty good in Chrome and Firefox.

Reply to this email directly or view it on GitHub:
https://github.com/hiddentao/google-tts/issues/17#issuecomment-69529068

Safari support flash, i have some test with jPlayer and it work for my audio project


Correction for raw affiliation French Agricultural Research Centre for International Development (CIRAD), UMR AGAP

Correction needed for raw affiliation French Agricultural Research Centre for International Development (CIRAD), UMR AGAP
raw_affiliation_name: French Agricultural Research Centre for International Development (CIRAD), UMR AGAP
new_rors: 05kpkpg04;02w4exq36
previous_rors: 05kpkpg04
works_examples: W2811403696
contact: a7241cc0a02ab72d884e0aff883eaec5:28f76c71b12bcaa077b63a771c68c0a6f4bb @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-18. The new affiliations should be visible within the next 7 days.


Multiple real valued labels

Hi,
I'm working on a regression problem with multiple real valued targets. An exception is thrown by UMAP (attached below). I assume that it happens because I'm passing a multidimensional array as labels. Am I doing something wrong or is this mode not supported by the algorithm/implementation?
Thanks for everything!
Edit: After digging into the parameters for umap I found target_metric which I set to 'l2', but I still get an error when my target has shape (n_samples, n_targets)
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<timed exec> in <module>()

/usr/local/lib/python3.5/dist-packages/umap/umap_.py in fit_transform(self, X, y)
   1521             Embedding of the training data in low-dimensional space.
   1522         """
-> 1523         self.fit(X, y)
   1524         return self.embedding_
   1525

/usr/local/lib/python3.5/dist-packages/umap/umap_.py in fit(self, X, y)
   1440                     far_dist = 1.0e12
   1441                 self.graph_ = categorical_simplicial_set_intersection(
-> 1442                     self.graph_, y, far_dist=far_dist
   1443                 )
   1444             else:

ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()

You are correct that multi-dimensional arrays are not supported for labels in the current implementation. Hopefully a future version could cope with this, although ultimately this would violate the sklearn API, so may need to be handled in a different way. In the meantime I can offer you a workaround. If you check issue #58 you can find some discussion of merging datasets that have different metrics. For your particular case I would suggest that this comment provides an outline for what you want to do -- just substitute the metric you want to use for the data, and the metric you want to use for the labels ('l2' in this case) for the 'bray-curtis' and 'jaccard' that was used in the example. You will likely want to play with the mix_ratio to get a good balance between the data and the labels.

Thanks for the suggestion and the quick reply. I will try it at as soon as I can and come back with the result.
Cheers.

I've been trying to do a fit for each of my targets and then intersect all of the results without much success. I just get a big blob that actually looks worst than the results I get when I just do a fit without passing the targets.
I was thinking on going into the code and modify umap_.fit to change the call to sklearn.metrics.pairwise_distance so that distances are computed as l2norm(t1-t2) were t1 and t2 are rows of my targets matrix. Would this make sense?
fit1 = umap.UMAP(metric='l2').fit(X_umap ,y=np.squeeze(Y_umap[:, 0]))
fit2 = umap.UMAP(metric='l2').fit(X_umap ,y=np.squeeze(Y_umap[:, 1]))
fit3 = umap.UMAP(metric='l2').fit(X_umap ,y=np.squeeze(Y_umap[:, 2]))
# Intersect all graphs
intersection = umap.umap_.general_simplicial_set_intersection(fit1.graph_, fit2.graph_, weight=0.5)
intersection = umap.umap_.general_simplicial_set_intersection(intersection, fit3.graph_, weight=1/3.)
intersection = umap.umap_.reset_local_connectivity(intersection)

embedding = umap.umap_.simplicial_set_embedding(fit1._raw_data, intersection, fit1.n_components,
                                                fit1.initial_alpha, fit1._a, fit1._b,
                                                fit1.repulsion_strength, fit1.negative_sample_rate,
                                                200, 'random', np.random, fit1.metric,
                                                fit1._metric_kwds, False)

Ah, I see. I think you want something more like:
fit1 = umap.UMAP(metric='l2').fit(X_umap)
fit2 = umap.UMAP(metric='l2').fit(Y_umap)
intersection = umap.umap_.general_simplicial_set_intersection(fit1.graph_, fit2.graph_, weight=0.25)
intersection = umap.umap_.reset_local_connectivity(intersection)
embedding = umap.umap_.simplicial_set_embedding(fit1._raw_data, intersection, fit1.n_components,
                                                fit1.initial_alpha, fit1._a, fit1._b,
                                                fit1.repulsion_strength, fit1.negative_sample_rate,
                                                200, 'random', np.random, fit1.metric,
                                                fit1._metric_kwds, False)

where the weight is a little arbitrary (you may have to play with it a little).

That worked beautifully! Thanks!
One question remains. How can I test new unseen data points. I tried using fit1.transform(test_features) because it was the only obvious thing to do  but that didn't work. Any ideas?
Thanks again for the awesome code!

I think, unfortunately, that transforming new points through this custom
pipeline is going to be non-trivial. It can be done, but I will have to
work out exactly what incantations one would need to do so.
On Fri, Sep 28, 2018 at 3:39 AM GCBallesteros<EMAIL_ADDRESS>wrote:

That worked beautifully! Thanks!
One question remains. How can I test new unseen data points. I tried using
fit1.transform(test_features) because it was the only obvious thing to do
but that didn't work. Any ideas?
Thanks again for the awesome code!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/lmcinnes/umap/issues/145#issuecomment-425350160, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ALaKBXRZPG0h1cCbEeTKXD75ohqBqworks5ufdIxgaJpZM4W4Zo7
.

Ah, I see. I think you want something more like:
fit1 = umap.UMAP(metric='l2').fit(X_umap)
fit2 = umap.UMAP(metric='l2').fit(Y_umap)
intersection = umap.umap_.general_simplicial_set_intersection(fit1.graph_, fit2.graph_, weight=0.25)
intersection = umap.umap_.reset_local_connectivity(intersection)
embedding = umap.umap_.simplicial_set_embedding(fit1._raw_data, intersection, fit1.n_components,
                                                fit1.initial_alpha, fit1._a, fit1._b,
                                                fit1.repulsion_strength, fit1.negative_sample_rate,
                                                200, 'random', np.random, fit1.metric,
                                                fit1._metric_kwds, False)

where the weight is a little arbitrary (you may have to play with it a little). That may well be essentially what you were describing doing above.

This seems to no longer work as is and instead throws AttributeError: 'UMAP' object has no attribute 'initial_alpha'.
Is there are way to get inital_alpha from somewhere or should I set it arbitrarily? I couldn't find anything about the parameter in the documentation.
Additionally, simplicial_set_embedding() seems to now require the parameters densmap, densmap_kwds and output_dens even if densMAP is not used?

I think the best bet right now is to intersect with the label data via the model composition (i.e. build a model on data, a different model on labels and use the * operator on the models) -- see https://umap-learn.readthedocs.io/en/latest/composing_models.html

Thanks for you suggestions.


LemMinX [0.27.0] - OSX (Intel): Bad CPU

I'm getting a bad CPU type in executable: ./lemminx when executing 0.27.0 installed from Mason.
0.26.1 works fine, downloaded from the release page:
https://github.com/redhat-developer/vscode-xml/releases/

I noticed this is already an issue with the official binary.
Opened a new issue here:
https://github.com/redhat-developer/vscode-xml/issues/1001


Flow verify

a qucik fix of flow errors to get travis into green.
revert is more work ATM

patches https://github.com/facebook/react-native/pull/5971

ehm, while testing the patch revert worked landed...


Render slide when visible

Hi,
Thanks for this module, really helpful.  Right now, I'm using this module to display a carousel of articles, which can really slow down global perf.
Any ideas?

Hi @enguerranws,
Well, that's exactly what the underlying FlatList component is supposed to do. Take a look at this note in the documentation.
Typically, you will want to play with props initialNumToRender, maxToRenderPerBatch, windowSize, and updateCellsBatchingPeriod inherited from VirtualizedList.
Hope that helps!

@bd-arc Sure it does. I miss that point, thanks!

If I've got a carousel with 10 items and starts on item 5, is there a way to prevent it from rendering the first 4 items? Flatlist seems to render items from the start of the array up to the current array index.


Web Search should be case insensitive

To elaborate a little; The keyword table for each page is case-sensitive (to offer the possibility of the search being case-sensitive or case-insensitive). It would be fairly easy to force the table generation and/or the search term to a consistent case, but it is fairly important to keep a case-insensitive option:
Object* myobj = object[0];
The current search system was just a basic starting point, but if sticking with it I'd probably try to put a checkbox by the input field that toggled case-sensitivity on and off.

While I understand your reasoning, I still think by default the search should be case insensitive.

I wasn't suggestion the search shouldn't case sensitive, just that achieving this by making use of the existing search function and modifying the underlying data would lose some context about the result. But what you are suggesting is the opposite of that, so that's fine.

I think you misread me, I understood the case for ambiguities of names in AGS, you suggested a check box of case sensitivity, I agree it's a good idea. I was only arguing it be unchecked by default and offered a different take without check box, just to reflect on how the interface should look.

I think #57 is the bigger issue to be discussed, so I am closing this.


Fix dependency resolution issues

See https://github.com/apache/incubator-pekko-projection/pull/19#issuecomment-1484819778

https://github.com/apache/incubator-pekko-projection/pull/19#issuecomment-1484991058 is also relevant


0.4: ArgumentError: function serve does not accept keyword arguments

ERROR: LoadError: ArgumentError: function serve does not accept keyword arguments
while loading /Users/solver/Projects/julia-client/script/boot.jl, in expression starting on line 35
Julia has stopped: 1, null

I just pushed a fix for this to the julia-0.4 branch, if you pull you should be ok.

Thanks!

Did this fix get applied to 0.5.0?

Yes, if you're up to date then you shouldn't see this error on either julia 0.4 or 0.5.


[Bug]: 404 Resources not found Error When Accessing GitHub Branch

Is there an existing issue for this?

[X] I have searched the existing issues

Description
A user created a new GitHub branch, made some additions there, then could not access the branch, getting a 404 Resources not found error. They tried to create a new branch and re-do the changes, after which they received an 404 error again. However, a new branch with no modifications is accessible and works fine.
For more info, check the screen recording of the bug from the Front thread.
Steps To Reproduce

In an app that is connected to Git and has 3 pages (page #1 is the home page), create a new GitHub branch from master.
In the new branch, set page #2 as home page, then delete page #1.
Commit the changes.
Leave the workspace.
Re-access the app in edit mode, then try to access the new branch created in step #1.
You will get the error 404 Resources not found.

Public Sample App
No response
Issue video log
No response
Version
Appsmith self-hosted
 Front conversations

Mom with @nayan-rafiq

Behaviour of the product should not change.  Example, 404 should be thrown when the page is not accessible

When a user is switching the branch the user is interested in accessing the resources inside the branch and not is a specific page or any other resources per se

The existing API has support for accepting the applicationId and branchName and we can use this to fix the behaviour reported in the bug

We should have a discussion with someone from FE to make this change happen on the switch branch CTA.

cc - @infinitetrooper

@AnaghHegde  - Is the way forward to redirect to the homepage when the branch is switched if the active page is not presented in the checked-out branch?
As you mentioned that user is looking for resources in a branch and is not necessarily on the same page across branches.

@infinitetrooper We have to redirect the user to the default page of the chosen branch. It can be the same page or a different page.

@infinitetrooper We have to redirect the user to the default page of the chosen branch. It can be the same page or a different page.

If the same page exists in both branches, the user will still be on the same page when switching between branches right? We redirect the user to the default/homepage only if the page the user is on is deleted in the chosen branch.

@infinitetrooper Yes, that is correct. I believe it should be possible. This has a FE dependency and I can confirm after talking to someone from the team.


Chemistry server updates

The icon was updated recently.
The icons for Chemistry and Biological Sciences were swapped.

Looks good. Thanks @berquist


Jv ebpf deprecation cleanup

Description
Sets the default collection method to CORE_BPF for collector and the integration tests. Fixed unit tests that broke as a result of changing the default collection method to CORE_BPF. Also removed the hardfail config option for CORE_BPF. The code now behaves as though it is always true.
Checklist

[x] Investigated and inspected CI test results
[x] Ran integration tests locally
- [ ] Updated documentation accordingly

Automated testing
- [ ] Added unit tests
- [ ] Added integration tests
- [ ] Added regression tests
This changes the default collection method so testing it is not needed.
If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why you did not do so. (Valid reasons include "CI is sufficient" or "No testable changes")
In addition to reviewing your code, reviewers must also review your testing instructions, and make sure they are sufficient.
For more details, ref the Confluence page about this section.

We probably want to default to core_bpf in the case where the collectionMethod provided by the user is not valid (or no more valid...).
https://github.com/stackrox/collector/pull/1509/files#diff-481ca574af93ac9fb58516ac8df6705b55eff51252c30680b711d4f453ab6f44L132-L135

Done


Aggregate commit label for GCM metrics

This should reduce the number of unique metrics recorded and the cost of using Config Sync with Google Cloud Monitoring. Prometheus metrics still include the commit, to facilitate metrics validation.
Extracted:

https://github.com/GoogleContainerTools/kpt-config-sync/pull/827

/retest

/retest (flakey unit test unrelated to this change)

/retest

/lgtm


Measure perf impact of not being in the platform

Now that EF Core isn't included as part of the platform, look into the perf impact of not having pregenerated ngen/R2R images.

We discussed punting this in the last triage and we decided to keep it in 3.0 because we still have partner teams asking for it to be evaluated. If the performance hit is significant, this can be turned into a docs issue to describe how the hit can be mitigated.


panTo API method

Similar to the one that comes with OpenSeadragon, but taking an annotation (id) as an argument. Also take an immediate argument (and forward to OSD).

A panTo/fitToBounds would be a really handy feature!


Add ReloadAllScripts & AbortAllScripts

I hope the console separate its arguments with spaces as most terminals, not commas, and I hope the console can execute its commands without parenthesis or semicolons as them.

The console is a C# script input window. Commands are C# function calls. You don't need the semicolon, but you sure do need commas and parenthesis. And that's a good thing because it makes the console very powerful.

I thought I needed the semicolon, but actually doesn't. It is true that the C# script input window makes the console very powerful, but we could add Execute command because most average users will usually call only console commands, I think, and we could add commands like Kill(ped) or Spawn(vehicle).
Btw, How about this PR? Is there something wrong with it?

Could pop something into the Wiki showing the basics of the console.

Let me give you an example.
Reload testscript.dll

I'd like to call ordinary console commands like this.
If I'd call some C# functions, I'd like to call like this
Execute Game.Player.Character.Kill()

Now we can reload all scripts with one command, just like v2, yes.


ingestNodes for OpenSearch in API v2

Found a new field in the Instaclustr API v2 doc: https://instaclustr.redoc.ly/Current/tag/OpenSearch-Provisioning-V2#paths/~1cluster-management~1v2~1resources~1applications~1opensearch~1clusters~1v2~1/post!path=ingestNodes&t=request. Need to update the k8s operator OpenSearch CRD.

Implemented


[BUG] - Overlay crash after quest

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Whenever I finish a quest, I get a fatal error which force closes the overlay. The error occurs right as I get to the reward screen, and it happens in every quest I've tried - HR, HR5, G-Rank (the provided quest ID is a GR quest) and Zenith. From what I can tell from the error message, as well as when the crash happens, it has something to do with the quest logging, but idk what.
The same thing happens both with and without high-grade edition.
non_high-grade.log
high-grade.log
How often does the bug occur?
Always
What operating system are you using?
Windows 10 22H2
What version of the program are you using?
Latest version
What version of the game are you using?
Non High-Grade Edition
Quest ID
23255

I identified the cause of the issue which seems to be related to how the user system handles decimals (e.g. 0,0 instead of 0.0). I'll see if I can release a fix soon.

I released an update. I can re-open the issue if its still present.

The update fixed it, yea. Thanks :)


Ablout the gps/imu info.

I have some questions about the GPS/IMU info.
1. About the GPS info, I see in some timestamps, the values of course in GPS are equal, so I thinks that the GPS signals update very slowly(about 5 seconds), so I can't use it every second.
2.About the IMU info, it contains gyro info and accelerometer info, both info contains three values.
I want to know the x,y,z means which direction respectively, for example, the z means perpendicular to the ground, the x means heading direction, the y means left-to-right direction...
I want to know the six values means which direction respectively.
3. After I know the IMU info, how can I calculate the dirft and the direction using these IMU/GPS info in one second roughly.
By any chance, could you help me ? Thanks for your early reply.

Same question.


Questions about departmentID and majorID

When adding majors or departments into the database, the departmentID or majorID are not given as parameters. However, when we do other operations like remove/ get, we should decide to remove/ get which department/ major according to the ID? What's the relationship between the ID and department/ major? I am not clear about this part. Can we just set the ID in our database to serial and ignore this problem?

I think I understand now. Does it mean that the Integer returned by every add function should be the ID of the inserted row?


Add an option to limit how many elements to render for collections and arrays

As mentioned in the readme:
You can ensure that arrays, dictionaries and collections don't output too much by allowing results to be truncated. Do this by setting the MaxCollectionCount property in the tableConfig.
int[] arr = [1, 2, 3, 4];

// Outputs only the first two elements and a message that says: ... truncated 2 items
arr.Dump(tableConfig: new () { MaxCollectionCount = 2 });

I saw that you merged the other PR, but I'm guessing this one needs some more thorough review :-P. Let me know if you need any additional information. Happy to help.
Btw, if you want to see what the truncated output looks like: run the unit tests. All variations should be in the test output.

Hi @hermanussen, yes, I need some more time to review this one and think about it :)
I hope to be able to go over it this weekend.
Thanks BTW for your contributions :)

Hi @MoaidHathot any update on this? Thanks!

@hermanussen, sorry for not updating the progress before. I'm applying some changes and soon will  merge this feature.

@hermanussen, sorry for the delay in reviewing and thank you for your contribution, awesome work!
I'm working on the next release, and I might change a bit the behavior of this PR to align it with future features, but this is great, thanks!


Backport "Bring back ambiguity filter when we report an implicit not found error" to LTS

Backports #20368 to the LTS branch.
PR submitted by the release tooling.
[skip ci]

No regressions detected in the community build up to lts-20279.
Reference