clamav: 0.105.0 -> 0.105.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/clamav/versions.
meta.description for clamav is: "Antivirus engine designed for detecting Trojans, viruses, malware and other malicious threats"
meta.homepage for clamav is: "https://www.clamav.net"
meta.changelog for clamav is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.105.1 with grep in /nix/store/if9pczdqi570nzvbvqwijj6sm41zwp4j-clamav-0.105.1
found 0.105.1 in filename of file in /nix/store/if9pczdqi570nzvbvqwijj6sm41zwp4j-clamav-0.105.1

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
clamav

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/if9pczdqi570nzvbvqwijj6sm41zwp4j-clamav-0.105.1
ls -la /nix/store/if9pczdqi570nzvbvqwijj6sm41zwp4j-clamav-0.105.1/bin

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

1 package built:

clamav

Maintainer pings
cc @robberer @qknight @fpletz @globin for testing.

Result of nixpkgs-review pr 183206 run on x86_64-linux 1

1 package built:

clamav


docs: examples generator scripts

Usage:
cd tools/example-generator
npm intall
pip install -r requirements.txt
./run.sh

Sparse clones/checks out the vizzu-lib repo to the current folder.
Generates ipynb files for each example file found in the vizzu-lib repo
test/integration/test_cases/web_content/sample_static/*.mjs,
test/integration/test_cases/web_content/templates/*.mjs
into docs/examples/static, docs/examples/animated folders

Known problems:

Should checkout specific commit corresponding the vizzu version used by ipyvizzu instead of HEAD

@nyirog @veghdev
The examples are dependent on ~450KB CSV data files, should I commit them into this repo or what to do?
Alternatively, I could create a new data repo and put them there.

CSV generator added too.

On Sat, Mar 26, 2022, 11:55 PM Laszlo Simon @.***>
wrote:

@nyirog https://github.com/nyirog @veghdev https://github.com/veghdev
The examples are dependent on ~450KB CSV data files, should I commit them
into this repo or what to do?

I think git can handle ~450kB csv data easily. As far it is not part of the
pipy package it should not bother the end user.
If someone checkout the git repo for play a bit with the source code.
He/she will have a complex dataset, too. So I think the large csv file can
add extra value to the development experience.
Regards,
Gergo

Alternatively, I could create a new data repo and put them there.
—
Reply to this email directly, view it on GitHub
https://github.com/vizzuhq/ipyvizzu/pull/78#issuecomment-1079789800, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJDTSD7CWY2QPYKECV5AVDVB6IURANCNFSM5RXYODEQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

well, i have already made a converter which generates the csv from the js data files.


Your super loud ads compared to the video damage my ears

Can I sue you? I will. You caused me hearing loss and Tinnitus.
It has been a known problem for years.
I WILL FIX THIS PERSONALLY IF YOU LET ME!
It is not that hard, is it?

Great, again, YouTube physically damaged me.

PLEASE!

PLEASE! I WILL DO IT! JUST ASK FOR IT!
 There are better support channels for the product YouTube than to leave comments in a channel meant for developers pertaining to a specific library.
Pretty sure you can leave feedback (respectfully) for Google and YouTube within the apps themselves.


Update the-escapers-flux to 7.1.4

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if updating a cask:

[ ] sha256 changed but version stayed the same (what is this?).
I’m providing public confirmation below.


Include "id" to data for UserTypes actions - RECEIVED_PROFILE_IN_CHANNEL

Summary
Include "id" to data for UserTypes actions - RECEIVED_PROFILE_IN_CHANNEL
Note: Will update commit of mattermost-redux once corresponding change is merged - https://github.com/mattermost/mattermost-redux/pull/499.
Ticket Link
Jira ticket: MM-10455
Device Information
This PR was tested on: [iOS 11.2 iPhone 8 simulator]

Will merge this now.


Fix #191 Updating highlight.js to 9.1.0. Adding various languages.

I added all the available languages from the website.

When I cloned the latest version and compiled I have
Untracked files:
  (use "git add <file>..." to include in what will be committed)

        Bonobo.Git.Server/Content/components/highlight/.bower.json
        Bonobo.Git.Server/Content/components/highlight/.gitignore
        Bonobo.Git.Server/Content/components/highlight/AUTHORS.en.txt
        Bonobo.Git.Server/Content/components/highlight/AUTHORS.ru.txt
        Bonobo.Git.Server/Content/components/highlight/docs/
        Bonobo.Git.Server/Content/components/highlight/src/
        Bonobo.Git.Server/Content/components/highlight/tools/

Is there something missing in the .gitignore or should those files be added to the project?

These are leftovers from you building highlight the first time. Now we have the precompiled version from the website.

@RedX2501 How did you add the new highlight? bower?
I recently pulled master and I get an error with a reference to the github.css in highlight/styles - it seems that when I launch VS I restores the bower packages, which states that we are using highlight 8.0.0 here and hence overwrite this commit.
Also there is a path here which needs to corrected too - if this is the right way to update to highlight 9
What do you think @jakubgarfield? You didn't get this problem?

@larshg I pretty much downloaded what was available on the website after selecting all packages. In the csproj I fixed the paths to use styles/github.css instead of src/styles/github.css.
I was not aware of any other automated steps done to highlight.js. If it exists i totally overlooked it. Maybe we can add it to the wiki.

@larshg @jakubgarfield What advantages do we have from using bower to get the highlight.js source instead of just getting the pre-compiled version from the website? Are we using any special options? How are we building highlight.js?

These build artefacts are really annoying.
Could we update .gitignore to hide these files?

@latop2604 As I can figure its not build artifacts. I don't think highlight was ever builded. I'm trying to figure out how this bower thing works and it seems that it only downloads the source files, but its not compiled by bower. For that I think you need to use some grunt or the like to make this compile.
However the github.css file is referenced from the package downloaded by bower instead of the one from the prebuild library.
You can properly delete the files, but we need to update the bower.json file, so it doesn't download the old 8.0 version.
It seems cumbersome for now to get it working using bower - so I think the best solution is to remove it from bower and use the prebuild packages from their website like @RedX2501  did.
We just need to fix a few paths, that I also mentioned earlier - any thoughts @latop2604 @RedX2501 @jakubgarfield ?

I have created a pull request #427 to remove the old highlight version from bower.

@larshg Yes, bower will only download the packages, most of them will need building. At least that's what highlight.js states.
@jakubgarfield How do you compile (if needed) font-awesome and pure?

I'm happy for you guys to add just precompiled versions of those packages - whatever is easier to maintain and build.

@RedX2501 I don't think I was compiling them, maybe just using them just as downloaded and put them in the normal pipeline. (but it's a quite a long time ago)


created about page for missing persons

This PR contains some conflicts, would you please resolve them first.

@wkambale while the conflicts are being resolved by @Nyayic , maybe you could review this and see if the copy works as well.
Nice work @Nyayic

Is anyone reviewing this?

Is anyone reviewing this?

It has some conflicts but would also need a copy review.


chore(migration): generic store for migrations

Description
This is a take on a generic store for migrations.  The tooling from the make bootstrap_migration target was updated to take in a list of storage protos.  For each storage proto it will create a gen.go that will create a store gogen -run pg-table-bindings is executed.  The store being generic will allow for the previous data to be retrieved as those functions simply return the protos (or IDs) generically.  The generated store will contain the insert logic for the latest proto and as such will insert data in the form the current release understands.  See the testing section below to look at an example.
I made a copy for the migration to provide a level of isolation.  If we decide we can simply use the generic store in pkg/search/postgres I would recommend we do that under additional tech debt tickets.  Since currently we cannot use a copy of a store for a migration easily due to the move to the generic store.  Similarly if we decide we want to handle sac in migrations that would be another issue to handle as tech debt.  I simply removed all the SAC checks in the copy of the store for the migrator.
This also includes any necessary changes for ROX-17423, ROX-18778.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
~- [ ] Evaluated and added CHANGELOG entry if required~
~- [ ] Determined and documented upgrade steps~
~- [ ] Documented user facing changes (create PR based on [openshift/openshift-docs]~(https://github.com/openshift/openshift-docs) and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Created #7387 where I created m_188_to_m_189_test_generic_example to demonstrate the use of the generic store and executed the migration test.  (Ignore the m_187_to_m_188_add_nftables_to_policy as that was my first example attempt)

At first glance, the changes seem to introduce a copy of the generic store with the following changes:

calls to metrics monitoring are removed
Scoped access control checks are removed

Technically, the generated metrics functions could be replaced by an empty pass-through function. On the Scoped access control side, I'd rather be in favour of using sac.WithAllAccess(ctx) in the migrations. That would leave room for some foreseeable changes to the way stores do work (the SAC checks could be moved from the generic stores further down to the SQL query generator).

The idea with the copy was to strip out stuff the migrations didn't need while also isolating the migrations from changes to the generic store so that we would be more intentional about what was included.  Regarding SAC, the migrations do run with sac.WithAllAccess(ctx) passed through the caller to the functions, so that should be fine.  I'm going to spend some time cleaning this up next couple days.  Any additional thoughts you may have would be appreciated.

That's great info.  Thanks.  I'm HOPING I can get back to this and finish it in the near future.  Especially since the SAC changes were moved.  Should simplify things.


[fix](s3 client)add default ca cert list for s3 client to avoid problem:'curlCode:77'

Proposed changes
Issue Number: #23163

add a be config field: ca_cert_file_paths , this is a default ca cart file list for different OS
And selects a valid file for initialization of the s3 client
if All ca cert file path is invalid ，s3 client will no set this config and use s3‘s default config
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

@ryanzryu I have picked this pr to branch-2.0 #32485


safari not rendering svg

Currently on some svg's I have repeating issue with svg not rendering in safari. I found that using object tag instead of svg or img allows safari to render svg correct.
Example of svg that not rendered being inlined:
7bit_logo.svg.zip
And it correctly wrks when referred in object tag

yep, that is a known bug

https://stackoverflow.com/questions/27245673/svg-image-element-not-displaying-in-safari

https://github.com/arkon/ng-inline-svg/blob/2720eb209e5d113f78cd667dc02cb90b7d79101a/src/inline-svg.directive.ts#L119
Probably we can make a switch here and add option for rendering object

PRs are welcome!


Does the example app work on android?

I just tried to run the example app on a physical lollipop device and it crashed immediately with this:
/com.airmapsexplorer E/ReactNativeJS: undefined is not a function (evaluating 'EventPluginHub.injection.injectInstanceHandle(ReactInstanceHandles)')

Any ideas?

I am getting exactly same error on Galaxy S6
Using commit#1004b03

Same problem in Redmi Note 3 (5.1.1)

I'm having this exact same message too, but I'm not using react-native-maps: stumbled on this page via Google. I'd just upgraded my RN project from 0.20 to 0.24 - some issue there?

Which version of RN are you using and which version of React is it?

RN 0.24, React 0.14.8.
I eventually got it working by clearing the npm cache and deleting the
packager's cache.
K
On Tue, 3 May 2016 at 12:42 AM, Jean-Richard Lai<EMAIL_ADDRESS>wrote:

Which version of RN are you using and which version of React is it?
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/lelandrichardson/react-native-maps/issues/185#issuecomment-216253875

The example app works fine on android now

Thanks! 😀
On Sat, 10 Sep 2016, 5:32 AM Felipe Lima<EMAIL_ADDRESS>wrote:

The example app works fine on android now
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/lelandrichardson/react-native-maps/issues/185#issuecomment-246015865,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB7b5wMV_vRsmgJKhbaqnmYsgwc5Meeqks5qobRPgaJpZM4IK0Cg
.


HTML response for dataset with weird CVs

I tried sumbmitting this:
{"@context": "https://downloads.dbpedia.org/databus/context.jsonld", "@graph": [{"@id": "https://dev.databus.dbpedia.org/denis/lodcloud-crawl/http://rdf.naturallexicon.org/zh/ont", "@type": "Group", "title": "LODCLOUD CRAWL", "abstract": "URIs crawled from the LOD CLOUD", "description": "Data from https://lod-cloud.net/, grouped by their title into artifacts."}, {"@type": "Dataset", "@id": "https://dev.databus.dbpedia.org/denis/lodcloud-crawl/http://rdf.naturallexicon.org/zh/ont/Cedict/2022-10-28#Dataset", "hasVersion": "2022-10-28", "title": "http://rdf.naturallexicon.org/zh/ont/Cedict", "abstract": "An RDF translation of the original CEDICT , which is written in\n its own tabular format.\nOriginal source: https://www.mdbg.net/chinese/export/cedict/cedict_1_0_ts_utf-8_mdbg.txt.gz, from https://www.mdbg.net/chinese/dictionary?page=cedict", "description": "\n## Description\n\nAn RDF translation of the original CEDICT , which is written in\n its own tabular format.\nOriginal source: https://www.mdbg.net/chinese/export/cedict/cedict_1_0_ts_utf-8_mdbg.txt.gz, from https://www.mdbg.net/chinese/dictionary?page=cedict\n\n## Other metadata\n\n| Category         | Value  |\n|--------------|-----------|\n| Timestamp (process start)     |   2022-11-02 07:55:24.096292        |\n| Timestamp (process end)     |   2022-11-02 07:55:24.875363        |\n| Keywords     |   Chinese,Chinese-English,bilingual dictionary        |\n| Domain     |   linguistics        |\n| Content Type Header     |   text/html; charset=utf-8        |\n| Contact:      | {'name': 'Eric Scott', 'email'<EMAIL_ADDRESS>|", "license": "https://creativecommons.org/licenses/by-sa/3.0/", "distribution": [{"@id": "https://dev.databus.dbpedia.org/denis/lodcloud-crawl/http://rdf.naturallexicon.org/zh/ont/Cedict/2022-10-28#Cedict_distinctid=id0_status=200_type=full-download.ttl.gz", "@type": "Part", "file": "https://dev.databus.dbpedia.org/denis/lodcloud-crawl/http://rdf.naturallexicon.org/zh/ont/Cedict/2022-10-28/Cedict_distinctid=id0_status=200_type=full-download.ttl.gz", "formatExtension": "ttl", "compression": "gz", "downloadURL": "https://github.com/ont-app/cedict/blob/master/data/cedict_1_0_ts_utf-8_mdbg.ttl.gz", "byteSize": 12345, "sha256sum": "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa", "dcv:distinctid": "id0", "dcv:status": "200", "dcv:type": "full-download"}, {"@id": "https://dev.databus.dbpedia.org/denis/lodcloud-crawl/http://rdf.naturallexicon.org/zh/ont/Cedict/2022-10-28#Cedict_distinctid=id1_status=200_type=full-download.ttl", "@type": "Part", "file": "https://dev.databus.dbpedia.org/denis/lodcloud-crawl/http://rdf.naturallexicon.org/zh/ont/Cedict/2022-10-28/Cedict_distinctid=id1_status=200_type=full-download.ttl", "formatExtension": "ttl", "compression": "none", "downloadURL": "https://github.com/ont-app/cedict/blob/master/data/cedict-schema.ttl", "byteSize": 12345, "sha256sum": "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa", "dcv:distinctid": "id1", "dcv:status": "200", "dcv:type": "full-download"}]}]}

The error is here probably the URI as  the value for the artifact http://rdf.naturallexicon.org/zh/ont/Cedict, but the response is not really helpful at all. Also for automated tasks a response as HTML is pretty bad, how can I ensure the response is in a proper format?

Something went wrong deploying bioportal-aero to the databus: Could not deploy dataset to databus. Reason: '<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">

<title>502 Proxy Error</title>

Proxy Error

The proxy server received an invalid
response from an upstream server.

The proxy server could not handle the request
Reason: Error reading from remote server

<address>Apache/2.4.41 (Ubuntu) Server at dev.databus.dbpedia.org Port 443</address>

'
{"@context": "https://downloads.dbpedia.org/databus/context.jsonld", "@graph": [{"@id": "https://dev.databus.dbpedia.org/denis/lodcloud-crawl", "@type": "Group", "title": "LODCLOUD CRAWL", "abstract": "URIs crawled from the LOD CLOUD", "description": "Data from https://lod-cloud.net/, grouped by their title into artifacts."}, {"@type": "Dataset", "@id": "https://dev.databus.dbpedia.org/denis/lodcloud-crawl/eagle-i-meharry/2022-10-28#Dataset", "hasVersion": "2022-10-28", "title": "eagle-i-meharry", "abstract": "Groundbreaking biomedical research requires access to cutting edge scientific resources; however such resources are often invisible beyond the laboratories or universities where they were developed. eagle-i is a national research resource discovery platform to help biomedical scientists search for and find previously invisible, but highly valuable, resources. \nResource descriptions collected at each participating institution are freely available as linked open data.", "description": "\n## Description\n\nGroundbreaking biomedical research requires access to cutting edge scientific resources; however such resources are often invisible beyond the laboratories or universities where they were developed. eagle-i is a national research resource discovery platform to help biomedical scientists search for and find previously invisible, but highly valuable, resources. \nResource descriptions collected at each participating institution are freely available as linked open data.\n\n## Other metadata\n\n| Category         | Value  |\n|--------------|-----------|\n| Timestamp (process start)     |   2022-11-02 08:00:14.883436        |\n| Timestamp (process end)     |   2022-11-02 08:01:45.036827        |\n| Keywords     |   deref-vocab,format-bibo,format-dc,format-ero,format-foaf,format-ncbitaxon,format-oboso,format-pobo,format-ro,format-vivo,lifesciences,provenance-metadata,published-by-producer,vocab-mappings        |\n| Domain     |   life_sciences        |\n| Content Type Header     |   ERROR        |\n| Contact:      | {'email'<EMAIL_ADDRESS>'name': 'The eagle-i Network'}  |", "license": "https://no.license.available/", "distribution": [{"@id": "https://dev.databus.dbpedia.org/denis/lodcloud-crawl/eagle-i-meharry/2022-10-28#eagle-i-meharry_distinctid=id0_status=ERROR_type=sparql", "@type": "Part", "file": "https://dev.databus.dbpedia.org/denis/lodcloud-crawl/eagle-i-meharry/2022-10-28/eagle-i-meharry_distinctid=id0_status=ERROR_type=sparql", "formatExtension": "file", "compression": "none", "downloadURL": "http://meharry.eagle-i.net/sparqler/sparql", "byteSize": 12345, "sha256sum": "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa", "dcv:distinctid": "id0", "dcv:status": "ERROR", "dcv:type": "sparql"}, {"@id": "https://dev.databus.dbpedia.org/denis/lodcloud-crawl/eagle-i-meharry/2022-10-28#eagle-i-meharry_distinctid=id1_status=ERROR_type=other-download.+optional%7B%3Fp+rdfs%3Alabel+%3Fp_label+%7D+.+optional%7B+%3Fo+rdfs%3Alabel+%3Fo_label%7D%7D+", "@type": "Part", "file": "https://dev.databus.dbpedia.org/denis/lodcloud-crawl/eagle-i-meharry/2022-10-28/eagle-i-meharry_distinctid=id1_status=ERROR_type=other-download.+optional%7B%3Fp+rdfs%3Alabel+%3Fp_label+%7D+.+optional%7B+%3Fo+rdfs%3Alabel+%3Fo_label%7D%7D+", "formatExtension": "+optional%7B%3Fp+rdfs%3Alabel+%3Fp_label+%7D+", "compression": "+optional%7B+%3Fo+rdfs%3Alabel+%3Fo_label%7D%7D+", "downloadURL": "http://meharry.eagle-i.net/sparqler/sparql?view=published-resources&format=text/turtle&query=PREFIX+rdfs%3A+%3Chttp%3A%2F%2Fwww.w3.org%2F2000%2F01%2Frdf-schema%23%3E+PREFIX+%3A+%3Chttp%3A%2F%2Feagle-i.org%2Font%2Frepo%2F1.0%2F%3E+construct+%7B%3Fs+%3Fp+%3Fo+.+%3Fp+rdfs%3Alabel+%3Fp_label+.+%3Fo+rdfs%3Alabel+%3Fo_label%7D+where+%7Bgraph+%3ANG_Published%7B%3Fs+%3Fp+%3Fo%7D+.+optional%7B%3Fp+rdfs%3Alabel+%3Fp_label+%7D+.+optional%7B+%3Fo+rdfs%3Alabel+%3Fo_label%7D%7D+", "byteSize": 12345, "sha256sum": "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa", "dcv:distinctid": "id1", "dcv:status": "ERROR", "dcv:type": "other-download"}, {"@id": "https://dev.databus.dbpedia.org/denis/lodcloud-crawl/eagle-i-meharry/2022-10-28#eagle-i-meharry_distinctid=id2_status=ERROR_type=other-download.+optional%7B%3Fp+rdfs%3Alabel+%3Fp_label+%7D+.+optional%7B+%3Fo+rdfs%3Alabel+%3Fo_label%7D%7D+", "@type": "Part", "file": "https://dev.databus.dbpedia.org/denis/lodcloud-crawl/eagle-i-meharry/2022-10-28/eagle-i-meharry_distinctid=id2_status=ERROR_type=other-download.+optional%7B%3Fp+rdfs%3Alabel+%3Fp_label+%7D+.+optional%7B+%3Fo+rdfs%3Alabel+%3Fo_label%7D%7D+", "formatExtension": "+optional%7B%3Fp+rdfs%3Alabel+%3Fp_label+%7D+", "compression": "+optional%7B+%3Fo+rdfs%3Alabel+%3Fo_label%7D%7D+", "downloadURL": "http://meharry.eagle-i.net/sparqler/sparql?view=published-resources&format=text/turtle&query=PREFIX+rdfs%3A+<http%3A%2F%2Fwww.w3.org%2F2000%2F01%2Frdf-schema%23>+PREFIX+%3A+<http%3A%2F%2Feagle-i.org%2Font%2Frepo%2F1.0%2F>+construct+%7B%3Fs+%3Fp+%3Fo+.+%3Fs+a+%3Ftype+.+%3Ftype+rdfs%3Alabel+%3Ftype_label+.+%3Fp+rdfs%3Alabel+%3Fp_label+.+%3Fo+rdfs%3Alabel+%3Fo_label%7D+where+%7Bgraph+%3ANG_Published%7B%3Fs+%3Fp+%3Fo%7D+.+optional%7B%3Fs+a+%3Ftype%7D+.+optional%7B%3Ftype+rdfs%3Alabel+%3Ftype_label%7D+.+optional%7B%3Fp+rdfs%3Alabel+%3Fp_label+%7D+.+optional%7B+%3Fo+rdfs%3Alabel+%3Fo_label%7D%7D+", "byteSize": 12345, "sha256sum": "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa", "dcv:distinctid": "id2", "dcv:status": "ERROR", "dcv:type": "other-download"}]}]}

Error logging and input format has been reworked, will try to reproduce with similar input

Converted to new input, interface and output now fails with json format and log entry:
Group uri <http://localhost:3000/janni/lodcloud-crawl/http://rdf.naturallexicon.org/zh/ont> must have exactly 2 path segments relative to the Databus base url <http://localhost:3000> (found 7)


Remove ParcelConverter

As stated here

Parcel is not a general-purpose serialization mechanism. This class (and the corresponding Parcelable API for placing arbitrary objects into a Parcel) is designed as a high-performance IPC transport. As such, it is not appropriate to place any Parcel data in to persistent storage: changes in the underlying implementation of any of the data in the Parcel can render older data unreadable.

Therefore it is a bad idea at all to do it like suggested here

ParcelConverter is a builtin Converter that allows you to easily store Android's Parcelable objects into a SQLite BLOB column. Here's an example of storing an Android Parcelable Location object in an Entity:

So I think the best and least confusing would be to remove the ParcelConverter at all.

Yes this true sure, however if you defining your own Parcel class then it lies in your area of control. Will update the documentation about this and if it nots really used then will deprecate it.


196 scanning cannot be completed correctly

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
work-around for db timeouts
What is the current behavior? (You can also link to an open issue here)
problems with long scans
What is the new behavior (if this is a feature change)? If possible add a screenshot.
fixed?
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
no

Something not ok on my side. I downloaded this:
$ git clone --branch 196-scanning-cannot-be-completed-correctly https://github.com/BenediktMKuehne/embark.git
Uploaded the iso from the bug:
android-x86_64-9.0-r2.iso

when I hit Analyze I get 500:

Only the android-x86_64-9.0-r2.iso dropped 500, I tried another binary and EMBArk has started the scan:

I started a new test with a binary that was never available under EMBArk.
I hope this will show that the web site can handle long runs.

fixed problems with APK extractor log output


Smaller docker image

Reduce calico/node from ~500MB to ~250MB

A couple suggestions for additional comments, but otherwise, LGTM.


ibazel doesn't watch changes in BUILD files that affect the build

If I run ibazel on some targets, and then totally mess up the BUILD file defining those targets (removing needed dependencies), they are not rebuilt/retested.

@endobson, thanks for filing an issue! I'm excited to have people trying things out.
I think this issue is the same as issue #3. I have a PR out ( #7 ) that adds the functionality you're looking for. Would you be willing to see if it works for you?

That doesn't even to listen for changes just constantly runs.
Testing //libraries/...
State: QUERY
Querying for BUILD files...
Error watching file tools/defaults/BUILD
Error: lstat tools/defaults/BUILD: no such file or directoryQuerying for source files...
State: RUN
INFO: Reading 'startup' options from /Users/endobson/.bazelrc: --watchfs
WARNING: Streamed test output requested. All tests will be run locally, without sharding, one at a time.
____Loading complete.  Analyzing...
____Found 265 targets and 26 test targets...
____Building...
____Building complete.
____Elapsed time: 0.246s, Critical Path: 0.00s
//libraries/algorithms:depth-first-search-test                  (cached) PASSED in 0.1s
//libraries:byte-escapes-test                                   (cached) PASSED in 0.1s
//libraries/data:bytes-test                                     (cached) PASSED in 0.1s
//libraries/data:dict-test                                      (cached) PASSED in 0.1s
//libraries/data:list-test                                      (cached) PASSED in 0.1s
//libraries/data:maybe-test                                     (cached) PASSED in 0.1s
//libraries/data:regexp-test                                    (cached) PASSED in 0.1s
//libraries/data:set-test                                       (cached) PASSED in 0.1s
//libraries/data:tuples-test                                    (cached) PASSED in 0.1s
//libraries:large-function-test                                 (cached) PASSED in 0.1s
//libraries:lexer-test                                          (cached) PASSED in 0.1s
//libraries:module-signature-serialization-test                 (cached) PASSED in 0.0s
//libraries/prim-language:cast1_test                            (cached) PASSED in 0.1s
//libraries/prim-language:color-graph-test                      (cached) PASSED in 0.0s
//libraries/prim-language:exit_test                             (cached) PASSED in 0.1s
//libraries/prim-language:factorial_test                        (cached) PASSED in 0.1s
//libraries/prim-language:hello-world_test                      (cached) PASSED in 0.1s
//libraries/prim-language:pointers1_test                        (cached) PASSED in 0.0s
//libraries/prim-language:pointers2_test                        (cached) PASSED in 0.1s
//libraries/prim-language:spill_test                            (cached) PASSED in 0.1s
//libraries:sexp-parser-test                                    (cached) PASSED in 0.1s
//libraries:sexp-printer-test                                   (cached) PASSED in 0.1s
//libraries:tail-call-test                                      (cached) PASSED in 0.4s
//libraries:type-checker-test                                   (cached) PASSED in 0.0s
//libraries:type-unification-test                               (cached) PASSED in 0.1s
//libraries:types-test                                          (cached) PASSED in 0.1s

Executed 0 out of 26 tests: 26 tests pass.
There were tests whose specified size is too big. Use the --test_verbose_timeout_warnings command line option to see which ones these are.
State: WAIT
State: DEBOUNCE_RUN
State: DEBOUNCE_RUN
State: DEBOUNCE_RUN
State: DEBOUNCE_RUN
State: DEBOUNCE_RUN
State: DEBOUNCE_RUN
State: RUN
INFO: Reading 'startup' options from /Users/endobson/.bazelrc: --watchfs
WARNING: Streamed test output requested. All tests will be run locally, without sharding, one at a time.
____Loading complete.  Analyzing...
____Found 265 targets and 26 test targets...
____Building...
____Building complete.
____Elapsed time: 0.249s, Critical Path: 0.00s
//libraries/algorithms:depth-first-search-test                  (cached) PASSED in 0.1s
//libraries:byte-escapes-test                                   (cached) PASSED in 0.1s
//libraries/data:bytes-test                                     (cached) PASSED in 0.1s
//libraries/data:dict-test                                      (cached) PASSED in 0.1s
//libraries/data:list-test                                      (cached) PASSED in 0.1s
//libraries/data:maybe-test                                     (cached) PASSED in 0.1s
//libraries/data:regexp-test                                    (cached) PASSED in 0.1s
//libraries/data:set-test                                       (cached) PASSED in 0.1s
//libraries/data:tuples-test                                    (cached) PASSED in 0.1s
//libraries:large-function-test                                 (cached) PASSED in 0.1s
//libraries:lexer-test                                          (cached) PASSED in 0.1s
//libraries:module-signature-serialization-test                 (cached) PASSED in 0.0s
//libraries/prim-language:cast1_test                            (cached) PASSED in 0.1s
//libraries/prim-language:color-graph-test                      (cached) PASSED in 0.0s
//libraries/prim-language:exit_test                             (cached) PASSED in 0.1s
//libraries/prim-language:factorial_test                        (cached) PASSED in 0.1s
//libraries/prim-language:hello-world_test                      (cached) PASSED in 0.1s
//libraries/prim-language:pointers1_test                        (cached) PASSED in 0.0s
//libraries/prim-language:pointers2_test                        (cached) PASSED in 0.1s
//libraries/prim-language:spill_test                            (cached) PASSED in 0.1s
//libraries:sexp-parser-test                                    (cached) PASSED in 0.1s
//libraries:sexp-printer-test                                   (cached) PASSED in 0.1s
//libraries:tail-call-test                                      (cached) PASSED in 0.4s
//libraries:type-checker-test                                   (cached) PASSED in 0.0s
//libraries:type-unification-test                               (cached) PASSED in 0.1s
//libraries:types-test                                          (cached) PASSED in 0.1s

Executed 0 out of 26 tests: 26 tests pass.
There were tests whose specified size is too big. Use the --test_verbose_timeout_warnings command line option to see which ones these are.
State: WAIT
State: DEBOUNCE_QUERY
State: QUERY
Querying for BUILD files...
Error watching file tools/defaults/BUILD
Error: lstat tools/defaults/BUILD: no such file or directoryQuerying for source files...
State: RUN
INFO: Reading 'startup' options from /Users/endobson/.bazelrc: --watchfs
WARNING: Streamed test output requested. All tests will be run locally, without sharding, one at a time.
____Loading complete.  Analyzing...
____Found 265 targets and 26 test targets...
____Building...
____Building complete.
____Elapsed time: 0.240s, Critical Path: 0.00s
//libraries/algorithms:depth-first-search-test                  (cached) PASSED in 0.1s
//libraries:byte-escapes-test                                   (cached) PASSED in 0.1s
//libraries/data:bytes-test                                     (cached) PASSED in 0.1s
//libraries/data:dict-test                                      (cached) PASSED in 0.1s
//libraries/data:list-test                                      (cached) PASSED in 0.1s
//libraries/data:maybe-test                                     (cached) PASSED in 0.1s
//libraries/data:regexp-test                                    (cached) PASSED in 0.1s
//libraries/data:set-test                                       (cached) PASSED in 0.1s
//libraries/data:tuples-test                                    (cached) PASSED in 0.1s
//libraries:large-function-test                                 (cached) PASSED in 0.1s
//libraries:lexer-test                                          (cached) PASSED in 0.1s
//libraries:module-signature-serialization-test                 (cached) PASSED in 0.0s
//libraries/prim-language:cast1_test                            (cached) PASSED in 0.1s
//libraries/prim-language:color-graph-test                      (cached) PASSED in 0.0s
//libraries/prim-language:exit_test                             (cached) PASSED in 0.1s
//libraries/prim-language:factorial_test                        (cached) PASSED in 0.1s
//libraries/prim-language:hello-world_test                      (cached) PASSED in 0.1s
//libraries/prim-language:pointers1_test                        (cached) PASSED in 0.0s
//libraries/prim-language:pointers2_test                        (cached) PASSED in 0.1s
//libraries/prim-language:spill_test                            (cached) PASSED in 0.1s
//libraries:sexp-parser-test                                    (cached) PASSED in 0.1s
//libraries:sexp-printer-test                                   (cached) PASSED in 0.1s
//libraries:tail-call-test                                      (cached) PASSED in 0.4s
//libraries:type-checker-test                                   (cached) PASSED in 0.0s
//libraries:type-unification-test                               (cached) PASSED in 0.1s
//libraries:types-test                                          (cached) PASSED in 0.1s

Executed 0 out of 26 tests: 26 tests pass.
There were tests whose specified size is too big. Use the --test_verbose_timeout_warnings command line option to see which ones these are.
State: WAIT
State: DEBOUNCE_QUERY
State: DEBOUNCE_QUERY
State: DEBOUNCE_QUERY
State: QUERY
Querying for BUILD files...
Error watching file tools/defaults/BUILD
Error: lstat tools/defaults/BUILD: no such file or directoryQuerying for source files...
State: RUN
INFO: Reading 'startup' options from /Users/endobson/.bazelrc: --watchfs
WARNING: Streamed test output requested. All tests will be run locally, without sharding, one at a time.
____Loading complete.  Analyzing...
____Found 265 targets and 26 test targets...
____Building...
____Building complete.
____Elapsed time: 0.252s, Critical Path: 0.00s
//libraries/algorithms:depth-first-search-test                  (cached) PASSED in 0.1s
//libraries:byte-escapes-test                                   (cached) PASSED in 0.1s
//libraries/data:bytes-test                                     (cached) PASSED in 0.1s
//libraries/data:dict-test                                      (cached) PASSED in 0.1s
//libraries/data:list-test                                      (cached) PASSED in 0.1s
//libraries/data:maybe-test                                     (cached) PASSED in 0.1s
//libraries/data:regexp-test                                    (cached) PASSED in 0.1s
//libraries/data:set-test                                       (cached) PASSED in 0.1s
//libraries/data:tuples-test                                    (cached) PASSED in 0.1s
//libraries:large-function-test                                 (cached) PASSED in 0.1s
//libraries:lexer-test                                          (cached) PASSED in 0.1s
//libraries:module-signature-serialization-test                 (cached) PASSED in 0.0s
//libraries/prim-language:cast1_test                            (cached) PASSED in 0.1s
//libraries/prim-language:color-graph-test                      (cached) PASSED in 0.0s
//libraries/prim-language:exit_test                             (cached) PASSED in 0.1s
//libraries/prim-language:factorial_test                        (cached) PASSED in 0.1s
//libraries/prim-language:hello-world_test                      (cached) PASSED in 0.1s
//libraries/prim-language:pointers1_test                        (cached) PASSED in 0.0s
//libraries/prim-language:pointers2_test                        (cached) PASSED in 0.1s
//libraries/prim-language:spill_test                            (cached) PASSED in 0.1s
//libraries:sexp-parser-test                                    (cached) PASSED in 0.1s
//libraries:sexp-printer-test                                   (cached) PASSED in 0.1s
//libraries:tail-call-test                                      (cached) PASSED in 0.4s
//libraries:type-checker-test                                   (cached) PASSED in 0.0s
//libraries:type-unification-test                               (cached) PASSED in 0.1s
//libraries:types-test                                          (cached) PASSED in 0.1s

Executed 0 out of 26 tests: 26 tests pass.
There were tests whose specified size is too big. Use the --test_verbose_timeout_warnings command line option to see which ones these are.
State: WAIT
State: DEBOUNCE_QUERY
State: QUERY
Querying for BUILD files...
Error watching file tools/defaults/BUILD
Error: lstat tools/defaults/BUILD: no such file or directoryQuerying for source files...

@endobson, sorry about that. I'll write on the chalkboard "I will not push my untested code into a pending PR no matter how confident I am that it will work". I've been using it in test/run mode (I rarely use build but did test it once by hand) for the past hour.
The most annoying thing I've seen thus far has been that there doesn't seem to be a way to query without fetching all the dependencies for a project so when I wrote a BUILD file recently it decided to go download 150mb of golang. That's unfortunate. Also there appears to be some strangeness with my vim where I will write a file once, it will trigger, then I will write it again and it won't trigger. If you have theories on that I'd be interested.

My guess for the vim issue is that vim is not actually making changes to the file system the second time as it can see that the file is up to date. (This is assuming you aren't changing the contents between the two writes.)

I filed an issue for the file move #9 and this is the same as #3 which is closed so I'm going to close this. Thanks for filing the bug!


Installation confusion

The readme says "Once installed, the Community Store API is accessed...", but I can't see which files and/or directory structure I need from the github repository to be able to ftp to my site and install using the Concrete5 "Add functionality" page.
(It may be there but I haven't been able to find a corresponding package on the C5 marketplace.)
Guidance would be greatly appreciated.
Thanks for a giving us such great tool in Community Store.
Ray

Download the release zip from here:
https://github.com/concretecms-community-store/community_store_api/releases/tag/v1.0.4
Unzip it to your packages directory, making sure the folder name is just community_store_api
Install it via the Extend Concrete section of the dashboard
Then you'd just go from here: https://github.com/concretecms-community-store/community_store_api?tab=readme-ov-file#setup

Thanks Ryan, both for the answer and prompt replay


The dev server is running all the time, and it still requires me to log in almost every time I refresh the docs.

@djpugh: The dev server is running all the time, and it still requires me to log in almost every time I refresh the docs.
Originally posted by @soderluk in https://github.com/djpugh/fastapi_aad_auth/issues/16#issuecomment-734814213

cc @tricosmo

Did you try: in terms of needing to login again, I'm assuming you mean when you are restarting the development server? You can set (as env variables or in a .env file):
SESSION_AUTH_SECRET="****"
SESSION_AUTH_SALT="****"
SESSION_SECRET="****"

Which will ensure restarting the server doesn't reset the cookie sessions

Is the server restarting at all, or is this just on a live running server - I will check the session length defaults that they aren't too short

No, for me, the server is running all the time. no reload, or restarting. but when I refresh page, I need to authorize myself again.
Even if I open a new tab in the same browser after authorized, it becomes unauthorized again.
This is know I run the server.
    uvicorn.run(app, host='<IP_ADDRESS>', port=8000, debug=True, reload=True)

Again thank you very much for your time.

@tricosmo - the default behaviour will re-init the session secret each reload - so with reload=True, it may well ask you to login, unless you set the session secrets as above - with those set in my .env file, it does work on reload/new tab etc.

My .env file is:
AAD_CLIENT_ID=<my-client-id>
AAD_TENANT_ID=<my-tenant-id>
LOGLEVEL="DEBUG"
AAD_CLIENT_SECRET=<my-client-secret>
SESSION_AUTH_SECRET="8e1cb550-4721-4319-a9a1-2d66d10a4a2f"
SESSION_AUTH_SALT="5d56f34a-f30f-4650-8483-6420d7fcc65b"
SESSION_SECRET="5aa36ab7-3e09-4f61-a300-cb1513d88e05"

Currently the UI based auth is via a session cookie, hence the comment above - if it is still occuring for you (this now easily persists for me across multiple restarts etc.)

Hi @djpugh,
My problem is not even not persists after server restarts. It loses authentication just simply by refreshing the page. So you authorize, refresh page, then you need to authorize again.
I got similar .env as you
COMPONENT_NAME=LCR
ENV=development
DEBUG=true
LOGLEVEL="DEBUG"
AAD_CLIENT_ID = "<my_client_id>"
AAD_TENANT_ID = "<my_tenant_id>"
SESSION_AUTH_SECRET = "43cca74e-ac3f-4c42-8450-f0bcf8609b9d"
SESSION_AUTH_SALT = "14641397-9c17-439a-9b9e-e4378976c0d2"
SESSION_SECRET = "ff05f998-4799-4e45-ff3a-4e0f4fe12e35"

Code that uses .env
auth_provider = AADAuth()
init_oauth = auth_provider.api_auth_scheme.init_oauth
app = FastAPI(title='Fincad API', docs_url='/api/docs', swagger_ui_init_oauth=init_oauth)

thanks again.

Hmm, are you just using this for the token validation? If you mean that you need to re-auth on the api/docs page, I think that is a function of the swagger ui - as that is all a Single Page Application, it is refreshed each time you refresh the page.
I'm not sure (but happy to look at) a way to extend that from the session auth, but if you aren't adding session auth (as in your sample above), then that won't help

but if you aren't adding session auth (as in your sample above),

do you mean this line

auth_provider = AADAuth()

I thought this will automatically config the auth session as long as I have those variables in the .env
It would be nice for the Single Page Application to retain authorized even after refreshing. Thanks

Hi @djpugh thank you so much. I will try it once I have time.


Devel michael

This PR contains corrections for different reactions of the ascaroside biosynthesis. It is also a test for the integration pipeline.

@JakeHattwell can we merge? Or ar the " a problem? Then I will try to remove them with the next commit


Verify the case where k < windowsize

I understand in the current code the variable k can be specified independently of the window size of the re-ranker.
In particular, it must be k <= windowsize (if k > windowsize => exception).
Now, when k < windowsize, the M-Table should be applied up to position k. After that position, we do not verify if the number of protected elements is greater or equal than anything, because there is no M-Table. In other words, the M-Table applies to the first k elements, the remaining elements are returned in the order in which they come, independently on whether they are protected or not.
This issue requires:

@purbon to certify that indeed k can be < windowsize

@tsuehr to check that the re-ranker works correctly when it receives more elements than the size of the M-Table

Fair point @chatox , this relates to our last conversation about k and window_size. Certainly k can be either smaller or bigger than the windows_size  value used in the query,  as of the way the code is written right now.
But this does not limit us to fix this situation and make it better and more consistent with the experience.
Your solution works fine for me, we only need to add a new exception for this case in the reranker function.

Now, when k < windowsize, the M-Table should be applied up to position k.

This will be always like this.

Now that I think more on this problem, this situation will only occur when using the fixed lookup for proportion strategy.


Requested Preset Changes - Rogue Gearset and Talents

### Gearsets
Attached are json export files for new proposed gearsets for Rogues. The goal with these gearsets are to show the 2 most common architypes, and show two variants of each architype to showcase the gearing goals.
I think these presets should replace the current gearsets.
They are:

P3 Mutilate HaT - Standard Mutilate build. HaT is called out specifically here because HaT prefers strong offhands, regardless of speed. This is opposite the next gearset, and also opposite most gearing suggestions up to this point, so I feel it is important to call out.
P3 Mutilate FA/CP - Standard Mutilate build when not using HaT, which wants a fast offhand, so Julie's dagger is used instead.
P3 Saber Fists - Saber Slash build when using the Fist weapons from ST. This is generally the best option for weapons
P3 Saber Swords - Saber Slash build when using swords. This is generally the best option when playing Human, where this will outperform the Fists layout.
Additionally, while my main testing used the max damage I could get with Darkmoon trinket x2, I am recommending these gearsets use more commonly accessible options, so I have selected Breadth / Gyro trinket instead.
An argument could be made that I should have also selected Hookfang rather than Julie's for an offhand, and I would be open to that changing.
Talents
Below are talent export strings I am proposing. There are 5 builds here:
I think these presets should replace the current "Cold Blood" talent presets

P3 Mutilate Carnage:<PHONE_NUMBER>51041-0053-02
P3 Mutilate CttC:<PHONE_NUMBER>51041-0053-02
P3 Saber Carnage:<PHONE_NUMBER>51051-320004
P3 Saber CttC:<PHONE_NUMBER>51051-320004
P3 Mutilate CttC Opportunity:<PHONE_NUMBER>5102-0053-05
They are self explanatory, I feel. The Opportunity option at the end is for use with FA/CttC, where it specifically outperforms the other options. I wasn't sure if it should be named "Opportunity" (the talent it takes) or if it should be named "FA/CttC" (the situation in which you take this build). Open to others opinions on the name.
I believe the "CB/DWS" talent builds could remain or go, I do not think it is necessary to keep them myself. If, however, we do keep them, I think they should be renamed to "Blade Flurry" and the point in CB should be moved into "Blade Flurry". The entire point of those talents were to support a 2-target cleave situation, and to do so you'd move CB -> BF.

Forgot to attach the gearset files:
Gearset - P3 Mutilate FA-CP.json
Gearset - P3 Mutilate HaT.json
Gearset - P3 Saber Fists.json
Gearset - P3 Saber Swords.json


[Merge] 'immunodeficiency without anhidrotic ectodermal dysplasia'

Mondo term (ID and Label)
MONDO:0010368 'immunodeficiency without anhidrotic ectodermal dysplasia'
Reason for deprecation
merged in OMIM
Merge with:
MONDO:0010386 'immunodeficiency 33'

@nicolevasilevsky looking at the x-ref, I realized that 'immunodeficiency 33' (MONDO:0010386) is both a "disease or disorder" and a "disease susceptibility"
The problem is that the reasoner makes this disease as subclass of "mycobacterial infectious disease", which is wrong.
Can you take a look and help me figure out what is going on? My specific questions are:

is it correct to have 'immunodeficiency 33' (MONDO:0010386) be both a disease and a susceptibility? (it makes sense intellectually, but not ontologically)

(maybe @matentzn and/or @cmungall can help with this) I don't understand how the reasoner makes 'immunodeficiency 33' a subclass of 'mycobacterial infectious disease' (see the red note in the following image):

is it that the reasoner "thinks" that the term has subclass annotations
"disease and disorder"
AND
"disease has 'infectious agent" some Mycobacteria"
EVEN THOUGH these subclasses are coming from different branches?

id33 is a disease or disorder (row 12-15)
id33 has infectious agent some M (row 3)
infectious disease = 1 AND 2 (row 16)

The questionable axiom is row 12 IMO.

I don't think a disease should be a proper disease and a susceptibility. I think the easiest thing would be to exclude the superclass 'X-linked Mendelian susceptibility to mycobacterial diseases' from 'immunodeficiency 33'. Looking at the OMIM record, I interpret this as an immunodeficiency disease that causes increased susceptibility to infections, but it seems like it is considered a disease itself.

I think this can be closed, yes?

No, this issue should not be closed yet. We still need to do the main work for this issue, which is term merge. I added to the merge list for a later release.


How do I do multiple graphs training?

❓ Questions & Help
Is it possible to do multiple graphs training with outputs of all nodes classification? If so, is there a sample code or any instruction?
Thanks very much.

Sure, that‘s possible, just input your graphs into the DataLoader and train against your node targets. There is no example, but it is quite similar to gcn.py except that you need to iterate over batches.

So I just stack all my feature vectors up into one array?

Yes, this is exactly what the DataLodaer is doing internally.

How does this work when you only want to train using a subset of the nodes (i.e. a semi-supervised fashion)?
If I combine several graphs into a batch using the DataLoader, and say each graph has a subset of node with labels that I want to use to train, how do I index these nodes in the batch?

I suggest that you add a mask tensor of size [num_nodes] to each Data object. You can then filter out the correct nodes by running x[mask] even in a mini-batch scenario.

Hi, I have similar question.
I want to train the model with multiple graphs and output all nodes classes for each graph and then use this model for all node classification in unseen graphs. I stored all my training graphs in list of dataset = [Data(...), Data(...)] and then input this list to DataLoader(dataset, batch_size=8). I wanted to start with your gcn.py but (if I correctly understand) the pooling operation changes shape of graph, so should I remove it from forward() function? Because if I input graph with shape [num_nodes, num_features] = [100, 20] to model my BCELoss function expects [num_nodes, 1] = [100, 1] shape. This type of classification doesn't support the pooling operation or am I wrong?

Hi, I have similar question.
I want to train the model with multiple graphs and output all nodes classes for each graph and then use this model for all node classification in unseen graphs. I stored all my training graphs in list of dataset = [Data(...), Data(...)] and then input this list to DataLoader(dataset, batch_size=8). I wanted to start with your gcn.py but (if I correctly understand) the pooling operation changes shape of graph, so should I remove it from forward() function? Because if I input graph with shape [num_nodes, num_features] = [100, 20] to model my BCELoss function expects [num_nodes, 1] = [100, 1] shape. This type of classification doesn't support the pooling operation or am I wrong?

Yes, you do not need a pooling operation for node classification (there's no pooling operation in gcn.py). For an example of doing node classification on multiple graphs, see examples/ppi.py.

Yes, you do not need a pooling operation for node classification (there's no pooling operation in gcn.py). For an example of doing node classification on multiple graphs, see examples/ppi.py.

Actually I was looking at benchmark/kernel/gcn.py. Thanks for fast reply!

Actually I was looking at benchmark/kernel/gcn.py. Thanks for fast reply!


feat(expr): support #> and #>> operator for extracting jsonb at a path

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
This PR adds support for jsonb path extraction:
jsonb #> text[] → jsonb
jsonb #>> text[] → text

Limitations:
SELECT '{"a":"b","c":[1,2,3]}'::jsonb #> '{0}';

such queries will output different results than postgres, because the literal {0} is interpreted as text but not text[] by default. I don't know how pg handles this case.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[x] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[x] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

support the following operators:
jsonb #> text[] → jsonb
Extracts JSON sub-object at the specified path, where path elements can be either field keys or array indexes.
'{"a": {"b": ["foo","bar"]}}'::json #> '{a,b,1}' → "bar"

jsonb #>> text[] → text
Extracts JSON sub-object at the specified path as text.
'{"a": {"b": ["foo","bar"]}}'::json #>> '{a,b,1}' → bar

Limitations:
SELECT '{"a":"b","c":[1,2,3]}'::jsonb #> '{0}';

such queries will output different results than postgres, because the literal {0} is interpreted as text but not text[] by default. I don't know how pg handles this case.

My bad ... If we do not reuse JsonbAccessInner / JsonbAccessStr but introduce new ExprType in proto, it would work "automatically"

Limitations:
SELECT '{"a":"b","c":[1,2,3]}'::jsonb #> '{0}';

such queries will output different results than postgres, because the literal {0} is interpreted as text but not text[] by default. I don't know how pg handles this case.

My bad ... If we do not reuse JsonbAccessInner / JsonbAccessStr but introduce new ExprType in proto, it would work "automatically"

soga. I'll add new expr types.


Add stability information to streaming results.

Towards #2522.
This changes the surface, since is_final and stability are per-result and not a per-alternative.

I'm totally open to ideas(regarding StreamingSpeechResult class). I had this class in a previous branch, and from discussions before, it sounded like you were for bringing protobuf data into our own classes as an abstraction.

@dhermes CircleCI is winning my love.
Did you have anything else for this?

You also forgot to check in a new RST file for result.py

LGTM once you fix your b0rken docs situation

Everything went green. Squashed and I'll just make sure it goes green again and then I'll merge.


Run the tests on Node.js version 10

Node 8 is now a maintenance LTS, and I think that from now on we should test Node 10, which is the active LTS, in the main.
For the time being we try to run the tests in parallel.
https://github.com/nodejs/Release#release-schedule

Codecov Report

Merging #2601 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2601   +/-   ##
=======================================
  Coverage   80.24%   80.24%
=======================================
  Files          57       57
  Lines        4510     4510
=======================================
  Hits         3619     3619
  Misses        891      891

Continue to review full report at Codecov.

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

Codecov Report

Merging #2601 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2601   +/-   ##
=======================================
  Coverage   80.24%   80.24%
=======================================
  Files          57       57
  Lines        4510     4510
=======================================
  Hits         3619     3619
  Misses        891      891

Continue to review full report at Codecov.

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

Codecov Report

Merging #2601 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2601   +/-   ##
=======================================
  Coverage   80.24%   80.24%
=======================================
  Files          57       57
  Lines        4510     4510
=======================================
  Hits         3619     3619
  Misses        891      891

Continue to review full report at Codecov.

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

Codecov Report

Merging #2601 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2601   +/-   ##
=======================================
  Coverage   80.24%   80.24%
=======================================
  Files          57       57
  Lines        4510     4510
=======================================
  Hits         3619     3619
  Misses        891      891

Continue to review full report at Codecov.

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

Thank you very much for quick reviews and merging!


Scaling LLC for high ingestion rate from Kafka

Use of per-partition consumption in Pinot has been able to scale kafka consumption from one host to multiple hosts while keeping segments movable between hosts.
For use cases where we want to minimize hardware, and use more memory to consume a large number of partitions per host, it is useful to optimize LLC further by doing the following:
Limit the number of partitions that can create segments on the server: Segment creation puts a lot of pressure on GC since a large amount of memory is released at the time. If there are N partitions of the same topic in a server, it usually happens that they reach a segment boundary more or less at the same time (especially when consuming from earliest offset available).
If we limit the number of partitions that can go in parallel, then it is possible that the SegmentCompletionManager times out and hands over the segment creation to another replica because the first one assigned with creating the segment did not complete in time.  One way to overcome this is to have a larger time-out on a per-table basis. Better yet, take the initial time-out from the table config and adapt it as needed, by using local (per-table) value of allowed segment completion time on the leader controller, re-computed every time a segment completes for a table.
Lastly, since the server's segment completion protocol is to go back to the holding state on any error during completion, it is possible that if the segment completion time is small the completion task will bounce back between replicas. So, it is useful to keep around a copy of the last segment constructed in case a server is asked to build the same segment again. The copy can be deleted when the CONSUMING to ONLINE transition happens for the segment.
It is important that these three optimizations above be done together in order to achieve maximum benefit across multiple tables.
We need to investigate whether sending in a count of number of segments being built at the time the SegmentConsumed request is sent can allow SegmentCompletionManager to pick a winner with least number of segment builds. However, this may delay the picking of winner.

When the server gets to "KEEP" a segment, it can decide not to build the segment, but wait until it gets the ONLINE transition from helix. At that point, it can download the segment. This change should reduce the number of segment builds on the server (since building a segment is more expensive than downloading it).
On the controller (SegmentCompletionManager) we remove the FSM if it times out. Not sure if keeping it around in a new state (TIMED_OUT) may help. It may be same as increasing the commit timeout.

The segment build time for the real-time path can be reduced bu removing the first pass over the data to get min/max values, statsCollector, etc. These should already be there in the realtime in-memory segment.

We could have a thread in the server that updates the controller periodically that the segment build is going on. Then the state machine in the controller side can be retained for a longer time. If we couple that with limiting the number of simultaneous segment builds in the server then we can stagger the segment builds enough. Over time, hopefully they drift apart that they don't have to wait for building a segment.

Will be good not to stop consumption at all.
One early idea we considered was to let the controller add a new segment once the winner is picked. Not sure if we want the old segment still in CONSUMING state, or change it to be ONLINE If former, then we double the writes to zk for creating a new segment (because we need to update it to ONLINE later on). If latter, then servers will look to downloading the segment, and we will need to introduce special code to see if they are consuming the segment then just catch up to the target offset and respond OK to helix. After that, it has to keep checking to see if the segment is availalble, and then download and replace it at some point.
a server may go down and come back up, and if the state is ONLINE and the segment is not availble, it has to keep retrying? Or, may be start consuming while still retrying.
If the committer goes down, then this logic will not work, so someone needs to commit the segment to the controller.

Another thought is to change the Helix state model to include a BUILDING_SEGMENT state. Not sure how this will work for multiple cases and failures. Need to be ironed out.

We will be working on the following three things in the coming weeks:

Optimize realtime segment generation by folding in the stats collector object into the RecordReader. The realtime record reader does not have to do a full pass through the data to get the things needed for segment creation, in that case.

We will keep the old tar file generated and remove it only when when the segment goes to ONLINE state. This is for a back-up that we may not use ultimately, but good to have so that we can prevent infinite loop.

We will enhance the completion protocol to include a lease time in the COMMIT message from the SegmentCompletionManager. This is the max time that is given to the servers to build a segment. If the server does not complete building the segment during this time, it will send a new message to ask for more time. The SegmentCompletionManager maintains info on time taken to build the segment and adjusts the initial lease value accordingly.

We will table the other options for now until we have these and take fresh measurements.

8efbe511c - [PINOT-4603] Added server side code for extending lease for build time (#1174) (1 year, 8 months ago)
d88c2a4bc - Included more parameters in realtime server http requests to the controller. (#2722) (7 months ago)


Pull Requests - Size Guidance - List should be re-worded

https://github.com/microsoft/code-with-engineering-playbook/blob/master/code-reviews/pull-requests.md
Besides the size, remember that every PR should:

Be consistent,
not break the build, and
include related tests as part of the PR.

It looks a bit weird, as if it is a sentence that is just split in 3

@TessFerrandez I just saw this issue and actually addressed this in my PR about commit message: https://github.com/microsoft/code-with-engineering-playbook/pull/525
Could you please take a look! I am very keen to hear you view and suggestions. Could we attach this issue to that PR?

Fixed by #525


Define dynamic object handlers for @nestjs/microservices

These functions are required for this module to work correctly when using @nestjs/microservices.
For context -- we are adding a gRPC microservices to an existing nest application. We do not want to try to use a separate microservice and modules for this implementation.

Thanks for moving on this so fast.
Do you think you'll have time to push a release this week?

I'm going to push a release this evening CET time.
Thank you for your contribution!
wt., 8... lis 2022, 17:08 użytkownik jkalberer @.***>
napisał:

Thanks for moving on this so fast.
Do you think you'll have time to push a release this week?
—
Reply to this email directly, view it on GitHub
https://github.com/kugacz/nj-request-scope/pull/6#issuecomment-1307459815,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARR5U5KR7KK5LGY6JD5DFADWHJ3IFANCNFSM6AAAAAARZW7L2E
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Avoid pandas deprecation warning.

This also simplifies code by being explicit about usage of "base" and "values"
for numpy and pandas objects.
More info in issue 3018

Looks like the test failure here is already present on master: https://travis-ci.org/SciTools/iris/jobs/397436716

@pelson the tests are all passing now and I think you're closer to this than I - want to do the honours here if you're happy with this proposal?

Thanks for this @hdyson!

Thanks both, that's great.


Support custom input object mapper

Pull Request type

[ ] Bugfix
[x] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Other (please describe):

Changes in this PR
Issue #775
With this PR it's possible to provide your own InputObjectMapper. This can be useful for example to filter out fields from an input object, which is mostly to integrate with other frameworks/libraries, and is not intended to be used by most users.
Note that because DefaultInputObjectMapper calls itself recursively, for each field it goes a level deeper into an object structure it will call the custom mapper if available. The custom mapper in turn, might still call the DefaultInputObjectMapper after modifying the input map. It's easy to create a never ending recursion here, so be careful implementing this!
This extension does not make it possible to drop in a Jackson Object Mapper, since the input isn't JSON.

@paulbakker any plans to implement custom OUTPUT mapper?


Change multicluster "make build" to "make bin"

Signed-off-by: Jianjun Shen<EMAIL_ADDRESS>

/test-multicluster-e2e

@jianjuns the MC e2e failed due to a flaky issue which will be fixed by PR #4547.

/test-multicluster-e2e

/skip-all
The commit passed all tests earlier.


Screenshot plugin: Tainted canvas

Hi there,
I am finishing up building the openseadragon-screenshot plugin, however I repeatedly get the javascript security error "openseadragonScreenshot.js:329 -> Uncaught SecurityError: Failed to execute 'toBlob' on 'HTMLCanvasElement': Tainted canvases may not be exported."
I have never seen this error before, and before I was always using the latest, stable version of OSD (2.2.1). Now, I switched to the latest version in the master branch as this one includes the #837 fix which I will need to implement this. Is this a problem introduced somewhere along the line in the master branch? You can see the problem directly if you go to https://ktgleiden.github.io/test.html. Hope you can help!
Edit: If you want to reproduce the error, please make a screenshot ;) this is done by clicking the button with the square on it.
~Koen

Hello,
Good work on the screenshot plugin!
Your issue happens because the tiles are hosted on a different host than the one you are hosting the example.
You will need to enable CORS to be able to do screenshots.
Github already has the CORS headers, so all you need to do is open the DZI with the crossOriginPolicy set:
	    var viewer = OpenSeadragon({
	        id: "openseadragon1",
	        prefixUrl: "images/",
	        tileSources: "https://openseadragon.github.io/example-images/duomo/duomo.dzi",
	        crossOriginPolicy: "Anonymous"
	    });

Thx! :)
Awesome! That is something I wasn't aware of. You fixed it right away, which means that it works properly now. I will update the readme.md with this info and add the plugin to the list ;)
~Koen

I have worked on this today, including finding a user friendly version for IOS (that doesnt really implement user download, user have to click on the image to get the image save dialog (not supported directly by canvas)).
I have a blocking issue with Explorer and it seem to be related to this thread (I was also using toblob / filesaverjs as base solution). The crossOrigin Policy is not sufficient and toBlob (6.https://www.npmjs.com/package/canvas-toBlob) and toDataUrl are blocked.
Is it possible to get it working without using liberal CORS? (* / anonymous). We have control on the serveur CORS but it's currently set to grant right to the serveur on the domain and our local dev machine.
In the other hand, what I have read so far suggest that crossOrigin on IMG is not supported on Explorer.
Someone was able to implement file save on Explorer?
So far, it looks like we will have to implement the solution using the backend with content-disposition and more hoops to make sure the angular app wont be removed on a failed load (I have used jquery file download for this in another project, so I know it's a viable solution).


Edits to punctuation to conform to style guide

Fixing punctuation around code examples to conform to the style guide. Only punctuation has changed.

I rather like that code examples are introduced by colons; this seems the more intuitive usage for me as a reader. Can we conform the style guide to the actual usage instead?

Can we conform the style guide to the actual usage instead?

That's exactly why I posted over at the Swift Forums. It doesn't matter to me either way. I did notice, though, that newer additions to sections tended to conform more closely to the present style guide. So, either way, there's some cleanup to be done. I'm happy to go in either direction. I just need to know which one. :)

The following comments are from Susan (one our editors), who reviewed the PR today:
The punctuation/wording, etc is fine, except for these few instances where colons are needed below:
The following example reports a compile-time error.
let i = 1

Should be:
The following example reports a compile-time error: (use a colon because the sentence both directly references and precedes the code snippet.)
let i = 1

And the others (changed here):
However, the alternative example below is valid:
let i = 1

The example below uses the initializer to try to convert a String into an Int:
let possibleNumber = "123"

Write an optional binding for an if statement as follows:
if let <#constantName#> = <#someOptional#> {

The following if statements are equivalent:
if let firstNumber = Int("4"), let secondNumber = Int("42"), firstNumber < secondNumber && secondNumber < 100 {

The rubric is: ONLY use colons when the sentence preceding the code snippet directly references that code;  otherwise use a period.


Get the multi_queue length problem in P4

I recently used P4 to do a load balancing experiment with multiple paths. If there are four equivalent paths, when a packet arrives, I want to get the queue length of each path and find out the path of the shortest queue , and send packets to that shortest queue path.(the topo is shown as below,find the shortest depart queue from s1->s2,s1->s3,s1->s4,s1->s5,and send packet to the shortest link.)
h1-------------|                     |--------s2--------|                       |----- h7
|                     |                           |                       |
h2-------------|                     |--------s3--------|                       |---- -h8
|--------s1--- |                           | ------- s6-----|
h3-------------|                     |--------s4--------|                       |----- h9
|                     |                           |                       |
h4-------------|                     |--------s5--------|                       |------- h10
My idea is to define four registers, get the length of each queue and store them in registers, and then get the minimum value by comparing them one by one.
I found that there is a metadata about the queue in p4c.
header queueing_metadata_t {
bit<48> enq_timestamp;
bit<24> enq_qdepth;
bit<32> deq_timedelta;
bit<24> deq_qdepth;
}
register_write(qlength_reg,queueing_metadata.deq_qdepth)
I want to store the queue length in a register like this, but there are four links so that I don't know how to get the queue length of each link.
Therefore,I have two main questions：
1.Bmv2 is whether to use one queue for all ports or one queue for each port.
2.How to get the queue length of each different link?
Thanks! :)

The topology is like this, the above format is not correct.

Others can comment with more details and knowledge, but I just wanted to comment so that you are aware that accessing not merely 1 queue length in a P4 program, but multiple queue lengths, might be a very demanding thing that will work for some P4 targets, but not others.  It is certainly possible to implement access to such things in a software switch like bmv2 (whether or not it is currently implemented, I do not know, but probably not).  It becomes very challenging to implement such access in targets that have much lower $ cost per terabit/second.

@jafingerhut   Thanks for your reply.
Is there a way to implement this method in bmv2? How to obtain the length of each link queue in the bmv2 environment?
Thanks:)

@antoninbas  thank you so much!
I will have a try :)


Fix: iOS rate is reset to 1.0 after play/pause #1930

Update the documentation
No changes to documentation needed.
Update the changelog
Will do.
Provide an example of how to test the change
If the rate prop is set it will apply as expected. However, when the video is paused & played the rate will be set to 1.0 - the custom rate is not re-applied. This bug is present on any fresh build.
Describe the changes
When playback resumes we check the new playbackRate against the props rate. If they're different we set the playbackRate to match the props rate.

Thanks @redspear for this fix! I'm using it in a production app and it works great. Do you know when this is going to be merged?

I'm glad to hear it helped you. I don't know when/if it'll be merged.


Crash when suspending FBI (pressing home button) while installing CIA

I believe this can be prevented with respective AM Stop/Resume calls in APT callback, though I haven't tested yet myself.

CDN installations shouldn't encounter this problem anymore. However, AM does not provide commands to stop/resume CIA installation. Using the stop/resume commands for titles provided by am:net wouldn't work either, as I would also need access to the content installation handle(s) in use during the CIA installation to stop/resume them as well.


Error when queue retried: exponentialBackOff is not a function

my environment running ArangoDB
I'm using the latest ArangoDB of the respective release series:

[x] 3.0.3

On this operating system:

[x] MacOS, version: 10.11.6

[x] docker - official docker library

Foxx
How to reproduce:
create service for queue
'use strict';
const request = require('@arangodb/request');
const util = require('util');
const body = module.context.argv[0];
const config = module.context.configuration;

const opts = {
  url: config.functionUrl,
  method: 'POST',
  body: body,
  json: true
};

const res = request(opts);

if (res.body) {
  if (res.statusCode !== 200) {
    throw new Error(util.format(
      'Server returned HTTP status %s with message: %s',
      res.statusCode,
      res.body
    ));
  }
} else {
  throw new Error('Server sent empty response with status ' + res.statusCode);
}
if (res.statusCode != 200) {
  throw new Error(res.message);
}

console.log(res.body)

module.exports = res.body;

push job:
queue.push(
    {mount: '/queue', name: 'queue'},
    {asd:"asdasd"},
    {
      success: function () {
        console.log('Yay!');
      },
      failure: function () {
        console.log('Fail');
      },
      maxFailures: 2
    }
  );

and then, try to make response code from 3rd party is not 200.
It will return:
TypeError: exponentialBackOff is not a function
at getBackOffDelay (/usr/share/arangodb3/js/server/modules/@arangodb/foxx/queues/worker.js:75:10)
at db._executeTransaction.action (/usr/share/arangodb3/js/server/modules/@arangodb/foxx/queues/worker.js:156:33)
at ArangoDatabase._executeTransaction
(/usr/share/arangodb3/js/server/modules/@arangodb/arango-database.js:119:10)
at Object.exports.work (/usr/share/arangodb3/js/server/modules/@arangodb/foxx/queues/worker.js:131:6)
at eval (eval at undefined, <anonymous>:8:59)
at eval (eval at undefined, <anonymous>:11:17)
at eval (eval at undefined, <anonymous>:11:29)

and when I check internal module, there are no exponentialBackOff function.

the function was removed with a80d6635ab14aef031ba4d3739ac84eec3aedfc9 - still investigating.

Should be fixed in 2.8, 3.0 and devel branches now via the above commits.

@de3 since we just released 3.0.4, you can obtain the fix by downloading this file:
https://raw.githubusercontent.com/arangodb/arangodb/3.0/js/server/modules/%40arangodb/foxx/queues/worker.js
and replace<EMAIL_ADDRESS>on your installation with it.
The next release containing this fix will be 3.0.5.
Thanks for reporting.


Issues with WebP images

Attempting to save https://www.phoronix.com/news/Google-Linux-Binder-In-Rust results in a:
Error: Error! Unable to extract the image type! img-126424042635.TODO-EXTRACT https://www.phoronix.com/assets/categories/google.webp
Maybe this kind of error could be made non-fatal (e.g. ignore the image, but still extract the rest)?

The fix is quite easy (as a personal remainder, webp header is 52 49 46 46), however I've no idea if ebook softwares (kindle, kobo, koreader...) support webp images.
Yes, I agree...it's not a good idea to have a fatal errore when an image format is unsupported. I've opened a dedicated issue #19


Use go:embed for embedding files instead of vfsgen

Feature Description
Now that we have a minimum of go1.16 set, we can use go's embed functionality to embed templates/web assets (css/js), and translations.
Screenshots
No response

example:
https://github.com/woodpecker-ci/woodpecker/blob/master/web/web.go#L10-L11

only we will have to do some restucture of code as you can not embed files that are not in the same directory or in child directory where golang code is where you are embedding it.

Hey @techknowlogick , I would love to contribute to this issue under Hacktoberfest. How can I get started?

the main concern is that if we ose std embeded the compressed files would be decompressed and compressed on each request that indicated gzip support ...
... @techknowlogick should we close this issue?

There could be a comment in code for why the vfsgen is chosen, to help future developers know the background. Otherwise months later the knowledge may be lost.

As well, this is still a valid issue. silverwind posted above a potential https://github.com/vearutop/statigz that allows serving embedded files via gzip

precompress

Doesn't vfsgen have done that?

Yes, but only gzip. We want Brotli as well for another ~30% size reduction over gzip.

Are we fine with dropping support for non-bindata builds (e.g. always embed, don't support non-embedded assets). I fail to see a real use case for non-bindata builds and it just complicates the implementation a lot.

Assuming make watch will still work: I'm okay with it.
I noticed that when I compile with TAGS="bindata …" make watch,
the frontend doesn't work anymore and as I only get the raw HTML and some browser alerts, but no CSS or JS…

Indeed, watch mode depends on file in the file system. One solution is to have go run a reverse proxy on the assets path in dev mode, which would proxy asset requests to webpack-dev-server or similar.

Thought generally I'd like to avoid webpack-dev-server if possible and use vite which also has one built in, but that of course requires migrating to vite first.

Done in #22887


already in use for ssh port

Hey, trying out sish, selfhosted, my server already runs ssh - obviously at port 22, I can't disable it otherwise it would cut me off, what is the correct approach to this to be able to use sish?
Log repeatedly doing this:
sish           | 2020/06/13 - 22:15:06 | [INFO][cache:0xc000074410] Started certificate maintenance routine
sish           | 2020/06/13 - 22:15:06 | listen tcp :22: bind: address already in use

@etheaven howdy!
I recommend running it either on a port other than 22 or running ssh on a port other than 22.

Guess there was no other way. Solved by making ssh default to 2222 as is in config example.


Quote argument of view.flows.resolve

Description
Fix #4899.
Quote the argument of internal view.flows.resolve calls.
Not sure what is an effective way to test it. The internal view.flows.resolve happens after the user invokes :view.flows.remove ... Should the test be written in test_view.py, test_types.py or test_integration.py?
Checklist

[ ] I have updated tests where applicable.

Not sure what is an effective way to test it. The internal view.flows.resolve happens after the user invokes :view.flows.remove ... Should the test be written in test_view.py, test_types.py or test_integration.py?

test_types.py seems most appropriate, I think we already have some tests there that just need additional cases for flowspecs with spaces. 👍


Menutools doesn't work on startup on Windows 11

Just tested this tool and it is working fine.

Same here for me, even reinstalled so it readded the scheduled task. Still no go.

I don't have access to Windows 11, but this worked for me on Windows 10.

Install MT v1.0.2.
Copy and save C:\Program Files\MenuTools\helper.exe.
Export the MenuToools Task Schedule.
Uninstall MT v1.0.2 and install v1.0.1.
Copy the missing helper.exe to C:\Program Files\MenuTools.
Import the MenuToools Task Schedule.


Clarify the usage of a service connection for Azure RG deployment task

Clarify the usage of a service connection instead of Azure subscription.

@brianveltman : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@steved0x: I'm sorry - only the author of this article, @RoopeshNair, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks, all.