Enforcing scopes when accessing a resource

My issue is with line 146 in ZF\MvcAuth\Authentication\DefaultAuthenticationListner
if ($this->oauth2Server->verifyResourceRequest($oauth2request))
The verifyResourceRequest method has a $scope parameter that isn't getting passed to the oauth server in this line so it is assumed that any scope is vaild for this request as $scope defaults to null.
I'm aware that at this point it would be necessary to identify what scope should be required for this request and pass it through to the verifyResourceRequest method so that the token can be checked for this scope.
Is the DefaultAuthenticationListner designed to be overridden so that I can implement functionality to check which scope is required for this request or am I missing something?

This repository has been closed and moved to laminas-api-tools/api-tools; a new issue has been opened at https://github.com/laminas-api-tools/api-tools/issues/72.


Add kuzureta by borame

Song Name
kuzureta
Artist Name
borame
Source
Youtube
Youtube Link
https://youtu.be/gBdKQVucA_c?si=1qNnL6Gp4wkF0Hny
Direct File Link
No response
Song ID
1024087
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 65692344

accept


adding combat level notifiers to idlenotifier plugin

I found myself forgetting to pot up while in the nightmare zone or while doing slayer tasks. This addition to the idle notifier lets you receive notifications when you should pot up again

dont we already have this in boost indicators

@deathbeam yes but that is pretty inflexible for when you have multiple boosts


start problem

from Search_2D import plotting, env，显示错误import“Search_2D" could not be resolved

I found a solution by comparing Sampling_based_Planning and Search_based_Planning folders. In the former the imports are not good.
So if you switch from  from Sampling_based_Planning.rrt_2D import env, plotting, utils to from rrt_2D import env, plotting, utils
it will work.


I want use casp13 dataset.

I want use casp13 dataset.
And i want use covid-13 protein pdb file.
Please give me solutions.

I have been trying to do something similar to compare different methods.
Here is a nice package (from the readme) that can get the casp data for you.
https://github.com/jonathanking/sidechainnet
import sidechainnet as scn
d = scn.load(casp_version=12, thinning=30)
d['train']['seq'] #sequence
d['train']['crd] #coordinates

You would need to format the sequence and generate a msa for each sequence you want to test. Try hhblits for the msa.


Example using nginx-proxy for mailservers

First I like to thank you for one of the greatest docker apps. Everything is fine but I am struggling a little bit how to get it working as a Mail-Proxy-Server (Imap,smtp,postfix, http).
What is the best way to do it? Create a new template? Per vHost-Configuration?
I hope someone could help. For sure I would share my experience with the nginx-proxy community.
Regards

Since Nginx itself can't proxy SMTP, it's not going to be possible for this image to support it.
What are you trying to achieve? Are you looking for a container that can accept SMTP connections for a bunch of different domains and proxy incoming messages for different domains to different backend containers?

I should say Nginx in its default configuration can't do SMTP. It does support it with ngx_mail_core_module, but that's not compiled into the nginx executable used by this image.

@shl-berlin You could probably run a custom nginx image and the separate container install instructions here: https://github.com/jwilder/nginx-proxy#separate-containers  Probably would need a custom template as well: https://github.com/jwilder/nginx-proxy#custom-nginx-configuration

Hey!
Thanks for your answers. In my case I have two Mailservers. One is in the production state and the other one I want to prepare to take over the first one.
The first one is installed on the host system directly, without docker. The second one should be in a docker container. At the moment I am planing what is the best solution and the beay way to switch over. For testing proposes I thought it would be nice to do a kind of subdomain splitting.
This means:
first mailserver = masterdomain.com
second mailserver = testing.maildomain.com
Thanks a lot for you help guys.


Add setting to italicize with asterisks

Resolve #26 by adding a property to italicize text with asterisks instead of underscores (markdownShortcuts.italicizeAsterisks).

Closing this as it was added through a separate channel


Feat/cli dependency updates

Updated CSA deps and removed mdsvex help text.

Meh, I'm gonna nuke got from the package

@niktek reminder this is a thing that exists :D

Just let me know if this needs to go in for tomorrow's release!

Alright, closing for now. Reopen if you need to


How to call object tracking  by C++ Interface ?

Object tracking support in the demo is provided by libtensorflow_demo.so. I successfully had built it with the cmake options. But I'm not sure how to call the dynamic library correctly, using the C++ interface. The way the JAVA is called in the demo, but I don't really understand the specific process of its work. I hope I can call it in the way of C++ to implement my application, and do not know how to do it?
@andrewharp ,
Please help, Thanks

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

It has been 15 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 30 days with no activity and the awaiting response label was assigned. Is this still an issue?

We are closing this issue for now due to lack of activity. Please comment if this is still an issue for you. Thanks!


Update configure-synapse-link.md

Update instruction to enable synapse link

Learn Build status updates of commit a85a2a3:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/cosmos-db/configure-synapse-link.md
:warning:Warning

Details

articles/cosmos-db/configure-synapse-link.md

Line 48, Column 4: [Warning: file-not-found - See documentation] Invalid file link: './media/configure-synapse-link/enable-synapse-link.png'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 317abd6:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/cosmos-db/configure-synapse-link.md
:warning:Warning

Details

articles/cosmos-db/media/configure-synapse-link/enable_synapse_link.png
:white_check_mark:Succeeded

articles/cosmos-db/configure-synapse-link.md

Line 48, Column 4: [Warning: file-not-found - See documentation] Invalid file link: './media/configure-synapse-link/enable-synapse-link.png'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 590d333:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/cosmos-db/configure-synapse-link.md
:warning:Warning

Details

articles/cosmos-db/media/configure-synapse-link/enable_synapse_link.png
:white_check_mark:Succeeded

articles/cosmos-db/configure-synapse-link.md

Line 48, Column 4: [Warning: file-not-found - See documentation] Invalid file link: './media/configure-synapse-link/enable-synapse-link.png'.
Line 129, Column 4: [Warning: file-not-found - See documentation] Invalid file link: './media/configure-synapse-link/enable_synapse_link_existing_container.png'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 6c8f347:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/cosmos-db/configure-synapse-link.md
:warning:Warning

Details

articles/cosmos-db/media/configure-synapse-link/create-container-analytical-store.png
:white_check_mark:Succeeded
n/a (file deleted or renamed)

articles/cosmos-db/media/configure-synapse-link/enable_synapse_link.png
:white_check_mark:Succeeded

articles/cosmos-db/configure-synapse-link.md

Line 48, Column 4: [Warning: file-not-found - See documentation] Invalid file link: './media/configure-synapse-link/enable-synapse-link.png'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

#sign-off

@soferreira, please replace underscores with hyphens in the new file names. Underscores are not allowed in Microsoft Learn file names. Thanks.

Learn Build status updates of commit 9f8a206:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/cosmos-db/configure-synapse-link.md
:warning:Warning

Details

articles/cosmos-db/media/configure-synapse-link/create-container-analytical-store.png
:white_check_mark:Succeeded

articles/cosmos-db/media/configure-synapse-link/enable_synapse_link.png
:white_check_mark:Succeeded

articles/cosmos-db/media/configure-synapse-link/enable_synapse_link_existing_container.png
:white_check_mark:Succeeded

articles/cosmos-db/media/configure-synapse-link/enable-synapse-link-feature.png
:white_check_mark:Succeeded
n/a (file deleted or renamed)

articles/cosmos-db/configure-synapse-link.md

Line 48, Column 4: [Warning: file-not-found - See documentation] Invalid file link: './media/configure-synapse-link/enable-synapse-link.png'.
Line 129, Column 4: [Warning: file-not-found - See documentation] Invalid file link: './media/configure-synapse-link/enable-synapse-link-existing-container.png'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit eae2308:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/cosmos-db/configure-synapse-link.md
:white_check_mark:Succeeded

articles/cosmos-db/media/configure-synapse-link/create-container-analytical-store.png
:white_check_mark:Succeeded

articles/cosmos-db/media/configure-synapse-link/enable-synapse-link-feature.png
:white_check_mark:Succeeded
n/a (file deleted or renamed)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@soferreira, can you submit these changed through the private repo? Things that affect page look and formatting need to be submitted through the private repo, so we can see them on staging. Also, Microsoft staff are generally expected to work in the private repo.
For details and instructions, see the Contributor Guide.

New PR, closing this one #please-close


Terraform: Use backend configuration from tf-file

Hi,
is it possible to use the configuration from the tf-file's terraform.backend section?
Currently I have to set it in the pipeline too - so I have to configure it twice.
Or do I miss some best practice?
Thanks.

Similar problem here. I would like to use remote state variables and subsequently multiple backends/state files configured across different modules in my build artifact.
This tool doesn't seem to let me do that as it mandates I have to configure the backend with the terraform init command and then overrides the local config.

@LukasVice Right now it is not supported to set terraform.backend section from tf-file's . It will be enhancement but due to other priority work we are not picking it now


Snap to street grid for unroutable addresses

certain addresses are geolocated to a point that's not close enough to the road network to route to, so no route is returned. We should look at graphhopper's snapping code and make adjustments necessary to get routes in this case.
Example: Rosamunde Sausage Grill

so after increasing max node counts (#40), address like Rosamunde and other known previously unrouteable addresses are now routable. but only with public transit. fully bike routes are not returned. hypothesis is that it takes a lot of nodes visited to get to these places cause of weirdness in OSM, and routing only on the road network is still exceeding some sort of internal limit. will need some investigation as to why the bike only route isn't being returned. or better snapping

After some investigation, it looks like the stationRouter is never making it all the way back to the startNode, even with the maxVisitedNodes bumped up.  the node counts don't seem to be the problem. it's interesting that using the PT graph as a bridge a path can be found, but not while staying on the road network. maybe there are disconnected parts of the network? or maybe GraphExplorer is not letting certain edges through or MultiCriteriaLabelSetting is falsely discarding the label? it could also be something with snapping, or a weird OSM data thing

After some investigation, the problem is likely something in the way the public transit endpoint explores edges around a node. This problem does not exist in the standard endpoint, and exists regardless of the connecting profile. It seems to be some issues in certain regions of the map where the the routing cannot complete without a public transit leg. This leg often has some gap between it and the starting leg.


Update xterm.js

Mainly brings in the texture atlas page/merging changes
Fixes #157102

@meganrogge did you fix that private API conflict you mentioned yesterday?

Ah I guess you mean the failure I'm seeing now


Database update

Hi,
What is the frequency of your SBT database update?
Is there an option to execute this update locally on user side?
Thanks
Eric

Great,
thank you @Alan-Collins


Database pylint patches

All tests are passing, Pylint issues that I can code out, for now, have been removed.

As far as pylint goes, not sure what these are about:
************* Module profile_page.tests
profile_page/tests.py:1:0: C0114: Missing module docstring (missing-module-docstring)
profile_page/tests.py:1:0: W0611: Unused TestCase imported from django.test (unused-import)
I have a docstring for the file, and the entire file uses the tests


daphne_worker: Process reports in order of arrival

Based on #58 (merge that first).
Partially addresses #25.
Change report selector for DaphneWorkerConfig so that instead of
scheduling work per task and batch interval, reports are processed in
order of arrival.
A new DO is implemented, LeaderAggregationJobQueue, which maintaines
a queue of ReportStore instances. Once a report arrrives to an empty
ReportStore instance, an "aggregation job" is added to the queue; once
the instance becomes empty, the job is removed from the queue. Jobs are
ordered by the first report's time of arrival so that the oldest data is
processed first.
Other changes:

daphne: Remove task_id from DapLeader::get_pending_collect_jobs()
so that scheduling work on collect jobs doesn't have to depend on a
particular task.

Remove a TODO that was addressed by a previous change.

Rebased on top of @nakatsuka-y's last PR.


Update module to support latest react-native-camera

Will need to check on Android as I don't have access to a device.

0.1.0 or even 1.0.0. Usually breaking changes warrant a new major version, as it’s not backwards compatible.


Proposal: Horizontal service Autoscaling

I have been working with docker SWARM lately and noticed there is a lack of features Horizontal service Autoscaling.
Currently we have shell script that scale UP/DOWN containers as new servers get added/removed to cluster but it would be amazing feature if swarm manager can automatically scale up/down containers to match average CPU utilization specified by user.
Swarm manager should periodically adjusts the number of replica in a service to match average CPU utilization to the target specified by user.

While I agree that this should be built into Swarm, there's an open source solution here:
https://github.com/gianarb/orbiter
Maybe Moby/Docker can endorse the project and bake it into Docker itself?


Can not open conversation with three people

I have created conversation with two other users and now non of us can close it nor open it. It's showing in the left pane

and while I'm clicking on it another entry pops up in the pane

I'm using desktop app - version 4.4.2 on Windows 10
I was trying to reload app with clearing cache without success.
In the browser it looks exactly the same.
It's not possible to close entry from pic. 1

@pulkownik1803 What Mattermost server version are you on?

Hi @pulkownik1803. I wanted to follow up on this and confirm whether it's been resolved.

Closing.


Type multiple_images throws exception on saving

Laravel Version: 5.4.28
Voyager Version: 0.11.0
PHP Version: 7.x
Database Driver & Version:

Description:
If I try to save a datatype with a field multiple_images following exception occures:
at HandleExceptions->handleError(8, 'Undefined variable: resize_width', '/var/www/vendor/tcg/voyager/src/Http/Controllers/Controller.php', 178, array('request' => object(Request), 'slug' => 'assets', 'row' => object(DataRow), 'content' => null, 'file' => object(UploadedFile), 'filename' => 'qIryr0CUMsiEiBZpQbCP', 'path' => 'assets/June2017/', 'files' => array(object(UploadedFile), object(UploadedFile), object(UploadedFile), object(UploadedFile)), 'filesPath' => array('assets/June2017/qIryr0CUMsiEiBZpQbCP.JPG'), 'key' => 0, 'filePath' => 'assets/June2017/qIryr0CUMsiEiBZpQbCP.JPG'))in Controller.php (line 178)
The variable $resize_with is used, but not defined,
Steps To Reproduce:

create a table
add a textfield
add bread from type multipe_images
create a row
save

Please upgrade to the latest version. Problem is in vendor.

I've updated to v0.11.14 and still getting the error above in Controller.php (line 178)

next package ,hope fix it

This is either fixed in master or in an open pull request, so it should definitely be in the next release


Make bench.record support avg

Usually, to ensure the accuracy of the performance comparison test, we often run the test several times and then average it for comparison.
However, the current ticat can only perform a single comparison, such as ticat write-key : tpcc1w : bench.record : write-query : tpcc1w : bench.record.
So, I would like to add an operator, bench.res.select.by-record-id.avg [1,2], which can average the specified tests by passing in a list of ids, and similarly we can also support other operators.
Further, it would be more convenient to have a systematic analysis by tagging.

An alternal approach is:

select records as the exists method: using bench.result.select.*
select (by an arg provided by end user) record values by names, as columns
aggregate records having the same(not less, not more) tags, as rows
draw the table

The result will be similar to:

This issue should be in another repo ticat-mods/tidb.bench.
The requement was addressed at support aggregating display of bench.result

Here is how it looks.

Without aggregation:

With avg aggregation:

Futher works:

Support custom display order (the first is baseline), we already generate virtual record id in each aggregated result for this
Support more aggregating algorithm, currently we only have avg, the next should be natural, it will use the extra info(write by recorder) to do the aggregating


macOS: zsh features not found

zsh features (zsh-syntax-highlighting & sh-syntax-highlighting) not found when installed in /usr/local/share/
/Users/harry/.zshrc:source:120: no such file or directory: /opt/homebrew/share/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh
/Users/harry/.zshrc:source:121: no such file or directory: /opt/homebrew/share/zsh-autosuggestions/zsh-autosuggestions.zsh

Fixed


Open editor un a new undocked window by default

From<EMAIL_ADDRESS>on 2010-11-17T14:47:12Z
What steps will reproduce the problem?

Open a new .py file What is the expected output? What do you see instead? Upon opening a new python file (or creating one from scratch) if I have an undocked editor I would expect to see this new opened file only in the editor. Instead it gets opened in the undocked editor and in a (new) docked version of the same editor. I can even edit the same files in the docked editor at the same time as the undocked one (there can be multiple instances of the editor).
There should only be one instance of the editor at any time opened. What version of the product are you using? On what operating system? 2.0 on windows and linux (ubuntu 10.4).

Please provide any additional information below
.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=455

From pierre.raybaut on 2010-11-17T12:57:00Z
Actually this is the intended behavior: this document mirroring feature allow the user to edit two different area of the same file in two different editors - this is very convenient to compare files.
What's wrong with it to your opinion?
Status: WontFix

From<EMAIL_ADDRESS>on 2010-11-17T13:19:37Z
Ok. I see it's a feature and not a bug. Haven't looked at it that way.
Maybe this behaviour should be toggle-able so the guys like me, who come
from Matlab environment and have found spyder to be the greatest editor, can
turn it off and don't have to struggle finding where the editor is or why
there are two instances of and editor open.
Great program by the way. Thanks to you guys we've detatched from Matlab and
are now teaching python for numerical and scientific courses at my
university. This gives us a ton of freedom.
If you're ever in Chile, please contact us. We'd be happy to recieve you
here.

From pierre.raybaut on 2010-11-28T02:50:31Z
I understand and you're right, it may be an unexpected behavior for ex-Matlab users. But unfortunately, this is a very low-level feature that can be disabled without rethink/reimplement the whole editor widget.
I'm thinking of rewriting in the future because I'm not satisfied of some of its features/behaviors. But it's not my top priority right now.
Thanks for your interest in Spyder and thank you for your kind invitation!


Remove ->forwardAgent(), plus other notes

I found this page very helpful in figuring out agent forwarding with deployer, thanks.
I offer this PR with some suggested edits:

Removes ->forwardAgent(), which is not needed (at least in current versions of deployer, I tested on v4.3) when ssh config has ForwardAgent yes — and https://deployer.org/docs/hosts.html specifically recommends not calling ->forwardAgent() and instead having it in the ssh config
Adds notes about also specifying the user in ssh config instead of ->user('username') (similar recommendation in the docs), in this case just as a note since perhaps some teams use a specific account for deploying and therefore are fine sharing it.

Thank you for PR.
I didn't have much time to update this document for the version of Deployer.


Plugin Size Reduction

First, I want to say I love this tool and find it extremely useful.
In tiddlywiki reducing plugin size is quite important, and when dealing with large libraries like my tiddly2pdf lib, a plugin can grow in size very quickly. My plugin currently sits at 1.9 MB, and that can be quite the turn-off from general use (It's bigger than some wikis.) I'm wondering if something can be done to reduce the impact large libraries have on plugin size, possible through a process like tree shaking?

The framework internally uses esbuild to compile the plugin , esbuild has enabled the ES6 Module Tree Shaking , and will generate the plugin js and css to minimize compression . Please check the Modern.TiddlyDev#Minify field.
See doc

That's unfortunate then. I think that's enabled. Maybe the lib I'm using just doesn't minimize well.

Esbuild in this is upgraded, have a try on latest version.


Instagram stopped working

Maybe related to https://github.com/spicywebau/craft-embedded-assets/issues/150 ?

The same here, still get the need to Login Informations even with tokens in use.

I wonder if it's this: https://github.com/spicywebau/craft-embedded-assets/issues/150#issuecomment-733025585

Nope not working in our case because we need the texts from Social Media not the embedded link. All texts show the Login hint.

What happens when you manually do a curl get on your server, with the access_token, like this:
curl -X GET "https://graph.facebook.com/v8.0/instagram_oembed?url=https://www.instagram.com/p/fA9uwTtkSN/&access_token=xxxxxxxxxxx366bcadb8472eced"

Can you check the output? Should return some data, including the embed code if successful.

Yes with a new access token i will get data back. Token will be invalid after 30 days.

Just released Embedded Assets 2.4.5, which increased the Embed v3 minimum version requirement to 3.4.13, from the previous 3.4.9; Embed 3.4.11 fixed some issues with Instagram/Facebook, so it's worth retrying with the newer version.
If it still doesn't work after updating, though, I would recommend following the instructions from https://github.com/spicywebau/craft-embedded-assets/issues/150#issuecomment-723916346.


Custom config parameters for tofn malicious behaviours

Tofnd allows the end-user to specify a malicious behaviour in a config file or CLI flag.  We currently enforce that all malicious behaviours include a victim arg, even those malicious variants that do not have a victim arg.  For those variants, the victim arg is simply ignored:

https://github.com/axelarnetwork/tofnd/blob/192d95441ebba70024826939b9187da0ff179642/src/config/mod.rs#L71
https://github.com/axelarnetwork/tofnd/blob/192d95441ebba70024826939b9187da0ff179642/src/config/mod.rs#L91

We do this only because it's easy.  Instead, we should demand a victim arg only for those malicious variants that actually need it.
In general, each malicious variant might have ts own custom args list.  For each such variant, we'll need to write custom code to parse config data for that variant. That could become tedious in the future but we should do it anyway.  See discussion in axelarnetwork/tofn#38.

n/a


libzip: 1.3.0 -> 1.5.2

Motivation for this change
While fixing viking I noticed libzip is out of date. I haven't ran a full nix-review on this, but mysql-workbench did compile. More to come.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Output of nix-review:
https://github.com/NixOS/nixpkgs/pull/68597
23 package are marked as broken and were skipped:
gimpPlugins.focusblur gimpPlugins.texturize hhvm libsForQt511.khotkeys natron php72Packages-unit.php_excel php72Packages-unit.v8 php72Packages-unit.v8js php72Packages.php_excel php72Packages.v8 php72Packages.v8js php73Packages-unit.php_excel php73Packages-unit.v8 php73Packages.php_excel php73Packages.v8 phpPackages-unit.php_excel phpPackages-unit.v8 phpPackages-unit.v8js phpPackages.php_excel phpPackages.v8 python37Packages.cntk qgis qgis-unwrapped

4 package failed to build:
cntk mysql-workbench python27Packages.cntk vcv-rack

216 package were build:
ahoviewer arcanist ark collectd collectd-data deadbeef deadbeef-with-plugins digikam dolphin dragon drush dsview ebook_tools elisa fileshelter freedink gImageReader gimp gimp-with-plugins gimpPlugins.exposureBlend gimpPlugins.fourier gimpPlugins.gap gimpPlugins.gimplensfun gimpPlugins.gmic gimpPlugins.lightning gimpPlugins.lqrPlugin gimpPlugins.resynthesizer gimpPlugins.ufraw gimpPlugins.waveletSharpen gwenview haskellPackages.LibZip haskellPackages.bindings-libzip icingaweb2 ideviceinstaller idevicerestore k3b kcachegrind kde-cli-tools kdeApplications.baloo-widgets kdeApplications.dolphin-plugins kdenlive kfind okular libsForQt5.baloo libsForQt5.kfilemetadata kdeplasma-addons kdev-php kdev-python kdevelop kdevelop-unwrapped kile kmenuedit kwin-tiling plasma5.khotkeys libsForQt511.baloo libsForQt511.kfilemetadata libsigrok libsigrok-0-3-0 libsigrok4dsl libsigrokdecode libykneomgr libzip lsp-plugins lutris lutris-free matomo mgba mudlet nagios nextcloud-news-updater openlierox openrct2 openscad osrm-backend peruse pgloader phoronix-test-suite php php-embed php-unit php72 php72-embed php72-unit phpPackages-unit.apcu phpPackages-unit.apcu_bc phpPackages-unit.ast phpPackages-unit.box phpPackages-unit.composer phpPackages-unit.couchbase phpPackages-unit.event phpPackages-unit.igbinary phpPackages-unit.imagick phpPackages-unit.mailparse phpPackages-unit.memcached phpPackages-unit.mongodb phpPackages-unit.oci8 phpPackages-unit.pcs phpPackages-unit.pdo_sqlsrv phpPackages-unit.php-cs-fixer phpPackages-unit.php-parallel-lint phpPackages-unit.phpcbf phpPackages-unit.phpcs phpPackages-unit.phpstan phpPackages-unit.pinba phpPackages-unit.protobuf phpPackages-unit.redis phpPackages-unit.sqlsrv phpPackages-unit.xdebug phpPackages-unit.yaml phpPackages-unit.zmq php72Packages.apcu php72Packages.apcu_bc php72Packages.ast php72Packages.box php72Packages.composer php72Packages.couchbase php72Packages.event php72Packages.igbinary php72Packages.imagick php72Packages.mailparse php72Packages.memcached php72Packages.mongodb php72Packages.oci8 php72Packages.pcs php72Packages.pdo_sqlsrv php72Packages.php-cs-fixer php72Packages.php-parallel-lint php72Packages.phpcbf php72Packages.phpcs php72Packages.phpstan php72Packages.pinba php72Packages.protobuf php72Packages.redis php72Packages.sqlsrv php72Packages.xdebug php72Packages.yaml php72Packages.zmq php73Packages-unit.apcu php73Packages-unit.apcu_bc php73Packages-unit.ast php73Packages-unit.box php73Packages-unit.composer php73Packages-unit.couchbase php73Packages-unit.event php73Packages-unit.igbinary php73Packages-unit.imagick php73Packages-unit.mailparse php73Packages-unit.memcached php73Packages-unit.mongodb php73Packages-unit.oci8 php73Packages-unit.pcs php73Packages-unit.pdo_sqlsrv php73Packages-unit.php-cs-fixer php73Packages-unit.php-parallel-lint php73Packages-unit.phpcbf php73Packages-unit.phpcs php73Packages-unit.phpstan php73Packages-unit.pinba php73Packages-unit.protobuf php73Packages-unit.redis php73Packages-unit.sqlsrv php73Packages-unit.xdebug php73Packages-unit.yaml phpPackages.apcu phpPackages.apcu_bc phpPackages.ast phpPackages.box phpPackages.composer phpPackages.couchbase phpPackages.event phpPackages.igbinary phpPackages.imagick phpPackages.mailparse phpPackages.memcached phpPackages.mongodb phpPackages.oci8 phpPackages.pcs phpPackages.pdo_sqlsrv phpPackages.php-cs-fixer phpPackages.php-parallel-lint phpPackages.phpcbf phpPackages.phpcs phpPackages.phpstan phpPackages.pinba phpPackages.protobuf phpPackages.redis phpPackages.sqlsrv phpPackages.xdebug phpPackages.yaml plasma-desktop plasma-workspace powerdevil systemsettings powerline-rs ppsspp pulseeffects pulseview python27Packages.r2pipe python37Packages.r2pipe qcachegrind r2-for-cutter radare2 radare2-cutter runzip sc-im sigrok-cli sit subsurface tiscamera tldr ufraw unit wp-cli xournalpp yle-dl yubikey-neo-manager

It seems as if something has changed in mysql-workbench since I tried this lately...? I'll look into this hopefully soon.

Done in #68044

Thanks! :tada:


Lock and LockRevokerRights classes.

I'm including the dao here so that everything compiles.  Additional dao tests will be included in followup pr

@adamkorynta  @MikeNeilson  What other changes do you need to see in here?   The Dao issues will get re-litigated when Controller/Handler IT are further along.

I'll let Peter review the JSON/DTO, I don't have anymore comments on the rest.


Add Instructions for UnifiOS

Created these based on my installation. My ISP is AU Japan and they use the same EAP authentication for their ONTs. I've had it running for a few days now and haven't had any issues.

Thanks! I actually moved from my ER-4 to a UDM Pro about a year ago and during that transition, I also pulled the certs from an AT&T RG and was able to switch to wpa_supplicant, so I'm no longer using eap_proxy. I've got a backlog of PRs to review.

I did look into that route, but for smaller providers where certs are much harder to get, this package still works great!


TLB: Respect valid bit in EntryLo* when doing TLB lookup

Can have multiple TLB entries that map to the same address, only one being valid. Right now, we return the first one we find (valid or not) during lookup.

Fixed.


nixos/display-managers: don't start pulseaudio

Hey, we have sockets.

Motivation for this change
@jtojnar and I believe this shouldn't be needed anymore.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Have used this with GNOME and Pantheon for the past two days without issue.

LGTM then.


Question about marginal_likelihood in VAE

Thank you for your working and sharing. I learned a lot from them.
However, I have a question about the VAE implementation.
In VAE.py, you calculate the marginal_likelihood as an cross entropy:
marginal_likelihood = tf.reduce_sum(self.inputs * tf.log(self.out) + (1 - self.inputs) * tf.log(1 - self.out),[1, 2])
However, I am confusing as the formular is :

where the first item on the right side should correspond to the marginal_likelihood. I think the latent variable z should be included to calculate the likelihood but you use the initial input, self.inputs.
So I am a little confusing, can you explain it?
Thank you very much!

Hi.
There two tricks on implementation of marignal likelihood.
1st is to use Monte-carlo technique instead of integral.
2nd is to use only one sample for Monte-carlo technique.
With these tricks, marginal likelihood is approximated to log(p(x_i|z)).
Since p(x_i|z) is assumed to follow bernoulli distribution, log(p(x_i|z)) is cross entropy between network input and output.
Please check following.

@hwalsuklee Hi, thanks for the explanation. I understand it.
I have another question. There is a KL gap between the ELBO and p(x), so how can I compute the p(x)?
Thank you!

Dear @hwalsuklee:
do you know how negative log-likelihood for f.ex Cifar test set is reported in VAE-like models? Paper examples etc are included in my question here:
https://www.reddit.com/r/MLQuestions/comments/9sp6d3/how_to_calculate_log_likelihood_for_vaevqvae/


showing up in test results section

Describe your problem and how to reproduce it:
It was reported in a recent forum topic that the test text was mentioning something about <wbr> in the test results section (fake console) on the production site.  I did some searching throughout the master branch code and found three challenges (see below) which have such text in them.
Users with incorrect solutions are seeing these <wbr>s in the test results and are not sure if they need to add them to their solution.  With a correct solution, the user never sees these, but this should be resolved so as not to confuse users who are trying to create a passing solution.
Add a Link to the page with the problem:
I have only linked to the English version of the three challenges, but this issue affects all the languages.
Intermediate Algorithm Scripting: Spinal Tap Case
  - text: spinalCase("thisIsSpinal<wbr>Tap") should return "this-is-spinal-tap".
    testString: assert.strictEqual(spinalCase('thisIsSpinalTap'), "this-is-spinal-tap", 'spinalCase("thisIsSpinal<wbr>Tap") should return "this-is-spinal-tap".');
  - text: spinalCase("The_Andy_<wbr>Griffith_Show") should return "the-andy-griffith-show".
    testString: assert.strictEqual(spinalCase("The_Andy_Griffith_Show"), "the-andy-griffith-show", 'spinalCase("The_Andy_<wbr>Griffith_Show") should return "the-andy-griffith-show".');

jQuery: Learn How Script Tags and Document Ready Work
  - text: 'You should add $(document).ready<wbr>(function() { to the beginning of your script element.'
    testString: 'assert(code.match(/\$\s*?\(\s*?document\s*?\)\.ready\s*?\(\s*?function\s*?\(\s*?\)\s*?\{/g), "You should add $(document).ready<wbr>(function() { to the beginning of your script element.");'
  - text: 'Close your $(document).ready<wbr>(function() { function with });'
    testString: 'assert(code.match(/\n*?\s*?\}\s*?\);/g), "Close your $(document).ready<wbr>(function() { function with });");'

Basic HTML and HTML5: Link to External Pages with Anchor Elements
  - text: You need an a element that links to http://freecatphotoapp<wbr>.com
    testString: assert(/http:\/\/(www\.)?freecatphotoapp\.com/gi.test($("a").attr("href")), 'You need an a element that links to http://freecatphotoapp<wbr>.com');

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

Closing this issue, because it has been fixed in beta. Viewable at https://www.freecodecamp.rocks/

I've been getting <wbr> in the  - Intermediate Algorithm Scripting: Spinal Tap Case

@jacknehme Did you post your code on the forum for someone to review?  The <wbr> has nothing to do with the reason your code is failing.  That was just a bug in how the message displayed.

@RandellDawson Thanks for replying. No, I didn't post to the forum.
I see that this issue is closed yet I was getting <wbr> inserted in the string log box.
I just thought I could bring it to someone's attention.
I did eventually get my code to work.


plain javascript

I like the idea of code generation, actually your issue on Prisma's "graphqlgen" is what brought me here.
However, neither tool gives the possibility of generating JavaScript, I'm stuck with having to choose between TypeScript and Flow
Would it be possible to add an option for generating resolvers in JavaScript ?
If not, then could you explain why ?

Hi @Thebigbignooby
Can you please elobarate on your needs?
We are generating code for TypeScript and Flow because those are JavaScript supersets that supports types, so we are using the strongly-type principle of GraphQL and transforming it into language types.
JavaScript by itself does not support types, so that's why we don't have plugins for it.
Note that we do have plugins that generates other type of outputs (JSON, Java and more).
If you have an idea for a new plugin, we can definitely consider to add it (or, you can write it yourself if you wish).

Hi @@Thebigbignooby, I'm closing for now. Feel free to reopen if you have more questions.


Wk3pt1 - Modify java script file with name

Didn't realize I had to do a pull request till I was near the end of part 2, sorry about that!


Fine-tuning from last layer

Hi,
I'm using wav2vec2 model, and want to finetune on my own data using Libripeech transformer based recipe by changing the last layers only.
How can I finetune the weights & labels of last n layers?
Thanks

Hi,
I think here you would like to freeze the wav2vec2 model and only train the parameters of some additional layers placed on top of that, right? In this is the case, you can use the freeze parameter that you find in https://github.com/speechbrain/speechbrain/blob/develop/speechbrain/lobes/models/fairseq_wav2vec.py#L78.

I just need to set freeze=True, and the rest will be taken care of by it? It will be like fine-tuning the wav2vec2 model, on my custom data.

Yes, this is exactly what will happen.


Bitstamp error 403

Note: for support questions, please join our Discord server

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Action taken (what you did)
I set up gekko with all my config, running the paper trader works fine, but when I run it with the actual trader, I see an error from the bitstamp api.

Expected result (what you hoped would happen)
Gekko would start trading.

Actual result (unexpected outcome)

2017-12-18 06:37:09 (ERROR):    BITSTAMP API ERROR: BitstampError {
  name: 'BitstampError',
  message: 'Bitstamp error 403',
  meta: { status: 'error', reason: 'Invalid signature', code: 'API0005' } }
2017-12-18 06:37:09 (DEBUG):    Bitstamp returned an error, retrying..

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)

The bitstamp key I'm using is activated. I'm not very familiar with node, but if you can point me to where this might be raised, I can put some more log statements if that helps.

I figured out the issue.
If you're using sub accounts with bitstamp, for the username, you must use the id of the main account, and not the sub account.


Added scroll effect for navbar

The navbar should only appear after the user scrolls past the cover page and reaches "Where we plant" section. The dark background of the nav bar should also be added only then. Otherwise the Navbar should be where it is without the dark background.
Hello @Virtual4087 I have fixed the above issue ,you may review it and merge to main branch.

Functionality Demo : https://drive.google.com/file/d/1hcBLLnUFoQMMGfoameRUt9RRSG7i1w6A/view?usp=drive_link
UI Demo : https://drive.google.com/file/d/13hfokaUe-CAVMlxZgao5bKtvhTM0nFKX/view?usp=drive_link

@smitesh-pednekar that was a very detailed explanation. The UI demo would've been enough but thanks anyways cause now I don't need to run it in my own env. Thanks for your contribution and feel free to raise any issues if you want, I'll gladly assign them to you.


dubbo tracing support

After #16049 is completed, tracers can be used by various proxies, not just limited to HTTP. In envoy, now it is possible to trace dubbo/thrift streams as well as HTTP requests.
Dubbo Proxy would be a good start.

cc @zyfjeff @lizan

cc @zyfjeff @lizan

Hello, has Dubbo trace supported it? I'm looking forward to it @wbpcode

cc @zyfjeff @lizan

Hello, has Dubbo trace supported it? I'm looking forward to it @wbpcode

sorry, it's not supported for now.

Hi,
I'm trying to add a filter of dubbo, like the Router do, for learning purpose. Current only want to read from config and log something.
So, I created a filter, I want to read config using the config field of DubboFilter which is an Any. I give my filter a name, and register it with config field type envoy::extensions::filters::network::http_connection_manager::v3::HttpConnectionManager::Tracing, it built.
However when running, envoy complains about it don't know how to unmarsal yaml config to object.
DubboFilters::FilterFactoryCb LalalaFilterConfig::createFilterFactoryFromProtoTyped(
    [[maybe_unused]] const envoy::extensions::filters::network::http_connection_manager::v3::HttpConnectionManager::Tracing& proto,
    [[maybe_unused]] const std::string& stat_prefix,
    Server::Configuration::FactoryContext& context) {
    // log proto...
  return [&context](DubboFilters::FilterChainFactoryCallbacks& callbacks) -> void {
    callbacks.addFilter(std::make_shared<Lalala>(context.clusterManager()));
  };
}

[2022-07-26 17:01:19.835][9004][critical][main] [source/server/server.cc:116] error initializing configuration '/path/envoy.yaml': Unable to parse JSON as proto (INVALID_ARGUMENT:(dubbo_filters[1].config): invalid value Invalid type URL, unknown type: extensions.filters.network.http_connection_manager.v3.HttpConnectionManager.Tracing for type Any)

Is there some docs about this? Or there already some other filters, codes that did things like this, that I can imitate？

Hi @wbpcode ,
If I want to achieve tracing for dubbo_proxy, for the filter's config,  I can just reuse the Tracing proto defined in http connection manager, rather than introduce a new proto.
Did I get it wrong?

Hi @wbpcode ,
If I want to achieve tracing for dubbo_proxy, for the filter's config, I can just reuse the Tracing proto defined in http connection manager, rather than introduce a new proto.
Did I get it wrong?

I think it'ok to reuse the exist Tracing proto for now. May be we will have protocol-independent proto for tracing in the future, but it will be something just like current Tracing proto.

Hi @wbpcode ,
https://github.com/envoyproxy/envoy/blob/5313fa4206217e831367932a21c3e3bc6bf6efb4/source/extensions/filters/network/http_connection_manager/config.cc#L166
https://github.com/envoyproxy/envoy/blob/5313fa4206217e831367932a21c3e3bc6bf6efb4/source/extensions/filters/network/http_connection_manager/config.cc#L190-L195
I want to use the factory to create tracer implementation object like the http connection manager do. It uses singleton mechanism.
Should I import this in dubbo proxy, to reuse the singleton http_tracer_manager instance, or to imitate it, give a dubbo_tracer_manager like this:
DubboFilters::FilterFactoryCb LalalaFilterConfig::createFilterFactoryFromProtoTyped(
    const envoy::extensions::filters::network::http_connection_manager::v3::HttpConnectionManager::
        Tracing& proto,
    const std::string&, Server::Configuration::FactoryContext& context) {

  Tracing::HttpTracerSharedPtr tracer = nullptr;

  if (proto.has_provider()) {
    Utility::Singletons singletons = Utility::createSingletons(context);
    tracer = singletons.dubbo_tracer_manager_->getOrCreateHttpTracer(&proto.provider());
  }

  return [&context, tracer](DubboFilters::FilterChainFactoryCallbacks& callbacks) -> void {
    callbacks.addFilter(std::make_shared<Lalala>(context.clusterManager(), tracer));
  };
}

SINGLETON_MANAGER_REGISTRATION(dubbo_tracer_manager);

Utility::Singletons Utility::createSingletons(Server::Configuration::FactoryContext& context) {
  auto dubbo_tracer_manager = context.singletonManager().getTyped<Tracing::HttpTracerManagerImpl>(
      SINGLETON_MANAGER_REGISTERED_NAME(dubbo_tracer_manager), [&context] {
        return std::make_shared<Tracing::HttpTracerManagerImpl>(
            std::make_unique<Tracing::TracerFactoryContextImpl>(
                context.getServerFactoryContext(), context.messageValidationVisitor()));
      });
  return {dubbo_tracer_manager};
}

If import and reuse, the trade off is initialized some other object like the route_config_provider_manager, which is useless in dubbo_proxy. (the side effect in the http ver createSingletons()). Or if create a dubbo_tracer_manager, it go against the 'singleton' purpose.
Need some suggestion. Thanks.

Basically, this could be achieved by the new generic proxy-based dubbo proxy. So, I will close it. If someone still want this, ping me to reopen it or create a new issue.

has Dubbo trace supported now?

cc @cwww3 sorry, but I won't add new feature to dubbo_proxy for now because the limited free bandwidth.
IMO, it's recommend to use the new generic_proxy with dubbo codec to proxy dubbo traffic. And it provide full featured tracing, logging, and stats support.
See https://www.envoyproxy.io/docs/envoy/latest/configuration/listeners/network_filters/generic_proxy_filter


Some questions about publish and send

Hello,
I have just used MassTransit in my project, .Net core2.0. It is great but there are some concerns:
That is different between Publish vs Send. In my scenario, I have one email service to send email to out side. Other services will pass request to email service via RabbitMQ. So, in this case we should use "Publish" or "Send".
With Send, we need to pass the full URL of endpoint. There is any best practice to manage endpoint? Because if we have 10 commands, we need to manage 10 endpoints. Is it right?
Relate to event(Publish), if one service is deployed on multiple instances, when one event is published to queue. It will be processed one time or will be processed many times on each instance.
Could you please share me one unit test for consumer? Because with harness test, it seems we just ensure message was queued.
Masstransit is ready for .Net Core 2.1?
Many thanks,

Did your questions get answered on the chat room? If so, please close this issue.

But about unittest for consumer. I am not still clear it. Could you please explain more detail?


Feature Request: 1:1 NAT using a WAN with DHCP (Not Static)

Forum user requesting this, he as two dhcp WAN addresses and wants to use one of them for 1:1 NAT. He can only enter a static address.
Would it be possible to use an alias here so as to track the WAN address and adjust accordingly?

two addresses sounds a bit odd, but that maybe besides the point.
What's the use case for this? and how should it work?

Erm.. I believe it's two WAN interfaces, not two addresses on one interface. :)

ok, and want to forward all traffic to the current public ip to a single host in the local network?

Yup.. the same as a static address.

I never realised you couldn't do it till I tried.

you should be able to use a portforward, just select any and the wan address as destination

But wont you need the same on the outbound too?.

Sorry I meant inbound

Just be nice to do it with 1:1 was my thinking,

it doesn't do anything else then a port forward in this case, the 1:1 option is mainly useful if you need to map ranges.

fixed one minor glitch on the port forward page https://github.com/opnsense/core/commit/ee98ebb29e89686fea75ac689ba2f0e6c75ce89e while looking at this one.

True, but you only need to make one entry rather than two.

I'm not sure what you mean, as far as I know the user can already reach his goal with the functionality integrated.

Yes he can, It's just a case of making it stupidly simple. :)


miscellaneous build / warning fixes:

cli/aiff.c: eliminate declaration-after-statement for c89 compilers
cli/aiff.c: lose c99 vla and use a fixed [256] array for compressionName
silence incompatible pointer warning for GetFileSize().
cli/utils.c: define _WIN32_WINNT as 0x0500 for GetConsoleWindow().
win32: don't define fileno as _fileno to avoid possible errors/warnings

If this goes in, watcom support is possible with only one FIXME (see: https://github.com/sezero/WavPack/commit/289938707a4914066bc5b329fff4755282d0e0b3 - should I create a PR for that after this?)

Sorry, I really should have marked this as WIP. It's not at all tested and I haven't even built on MSVC yet (which is the intended toolchain for Windows). But okay, I'll work around this. Thanks!

If this goes in, watcom support is possible with only one FIXME (see: sezero@2899387 - should I create a PR for that after this?)

I'm very curious about the Watcom support. Could you help me understand the purpose of this? I'm all for improving compatibility, but I'm also hesitant to promise that WavPack code will never utilize any improvements to the C language that are newer than 1989. And of course since I only use newer tools (having recently upgraded to MSVC 2019) it's always possible that I (or someone else) will break this accidentally and unless you're continuing to build nobody will ever know.
Finally, I assume that this does not target architectures with 16bit integers, right?

I'm very curious about the Watcom support. Could you help me understand the purpose of this?

My actual purpose was creating a usable os/2 version. I almost always work on and cross-compile from linux, and since the os/2 emx environment has never been ported to be used as a cross-toolchain, open watcom is my option.

Finally, I assume that this does not target architectures with 16bit integers, right?

The compiler can do that, but I never intended that, nor have I ever tested in such a way. (And I very much doubt the library would be buildable in that way.)
I created PR #136, but I can close it or mark it as WIP. Your call.


Suscoin listing for Pegasys DEX

name: Token Request
about: Request a token addition
title: 'Add {TOKEN_SYMBOL}: {TOKEN_NAME}'
labels: token request
assignees: ''

[*] I understand that token listing is not required to use the Pegasys Interface with a token.
[*] I understand that filing an issue or adding liquidity does not guarantee addition to the Pegasys token list.

Please provide the following information for your token.
Token Address:
0xB3d06b7d05969f806603702b084786e2ABe63460
Token Name (from contract):
Suscoin
Token Decimals (from contract):
18
Token Symbol (from contract):
SUS
Pegasys Pair Address of Token:
0x9a7787351c6eB126f66AE1eD8A982721090FD627
That is the SYS/SUS LP
Link to the official homepage of token:
https://suscointoken.com
Link to CoinMarketCap or CoinGecko page of token:
We are waiting for a response to our coinmarketcap application and wanted to ask about the API as far as getting listed on coingecko goes. In the meantime we are listed on Nomics.com and DexScreener at these links:
https://nomics.com/assets/sus4-suscoin
https://dexscreener.com/syscoin/0x9a7787351c6eb126f66ae1ed8a982721090fd627

I have followed all steps asked of in the listing requirements. Flames, the Discord server mod has been most helpful in getting this done. I hope to hear back soon.

I talked with Flames and they let me know it's probably a no for getting listed. I totally get your rugpull concerns and would like to see if we can work out a way to get you comfortable with listing.
I have ~30 million tokens that I'm holding on to for future DEX LPs and I get that it poses a big risk. What if we came up with something like putting it in a multisig wallet with you guys or even letting you take custody until the L2 rollout.
I had also considered we could do some sort of OTC transaction trading SUS for PSYS at a steeply discounted rate so I may relinquish control and give you guys incentive to list the project since you'll be big holders.
 I'm disabled and run my projects as a way to pass time and learn more plus I want to support Syscoin as it's the only chain other than Bitcoin I care for at all.
Just wanted to open a dialog with you about this so we can hopefully get on the same page. I can be contacted on discord at uttke.eth#1404 twitter @transbitcoiner email<EMAIL_ADDRESS>

The reason liquidity is so low also is that I'm on a fixed income and can only provide small amounts of SYS monthly to the LP. This is why also I'm not applying now for an $UTTKE listing since I don't have the money to invest a bunch in both so I'm letting it grow organically.


Job object is not accessible through the use of SubprocessExecutor

.sdk.execution.SubprocessExecutor creates a .sdk.job.Job, and writes the code to a file to be ran, but doesn't allow access to the object information itself. This was noticed while trying to use the Job.uuid as the identifier for tracking the current processes being ran by qiime-studio, and realizing that only the Future object was being returned.
Found while working on PR#32 in qiime-studio

I'll take this one, it depends on the workflow inversion/refactoring I'm starting soon. Job should probably know how to execute its code and SubprocessExecutor can be removed.

@jakereps, SubprocessExecutor.__call__ now returns a tuple (future, job), let me know if you run into any issues using it in the studio!


DSM airdrop calculation page

DSM airdrop calculation page should be able to:

Allow user to insert the address
Able to calculate the airdrop amount after user click the "calculate" button

Calculation should consider the following:
a.  Users who belong to the criteria below should receive more amount of DSM airdrop:

Staked more with non-zero commission validators
Staked more with Forbole validator
Have not staked with centralized exchanges
Have not staked with validators operated by foundation/official team of the projects

@kwunyeung Please review


Improving Apps Structure: Part 2 - Consolidate graph density based functions into dense module

This PR carries out the second part of the plan to improve the import structure of the applications layer.
Remaining steps are:

Consolidate sampling functions into a single module (e.g., currently in sample and g_sample)
Consolidate some top-level functions into another module and also update the __init__.py in each case

I'd like to suggest that we rename the module from dense to subgraph.
This matches better with the focus of the module. For example:
from strawberryfields import gbs

gbs.subgraph.search(...)

It also means we can move is_subgraph and subgraph_adjacency utility functions into this module.
What do people think?

I'd like to suggest that we rename the module from dense to subgraph.
This matches better with the focus of the module. For example:
from strawberryfields import gbs

gbs.subgraph.search(...)

It also means we can move is_subgraph and subgraph_adjacency utility functions into this module.
What do people think?

Ok, let's leave this out of this PR for now and revisit in https://github.com/XanaduAI/strawberryfields/pull/190 😁


Training can be potentially speed up by 2x?

On this branch training a Mip-NeRF 360 scene (garden) for 20k iterations took 11mins to finish. On wandb it shows 150k training rays/sec and 706k eval rays/sec.
On the other hand I implemented the nerfacto model in the nerfacc repo and using this script to train the garden scene for 20k iterations with the same batch size took only 5mins to finish (see numbers here).  That leads 266k training rays/sec and 760k eval rays/sec.  So there should be a ~2x speedup opportunity in the current pipeline.
The command I used to reproduce the training on the ethan/benchmarks branch:
command = " ".join(
    (
        f"ns-train {method}",
        "--vis wandb",
        f"--data { data_path / capture_name}",
        "--output-dir outputs/nerfacto-ablations",
        "--steps-per-eval-batch 0 --steps-per-eval-image 0 --steps-per-save 20000",
        "--steps-per-eval-all-images 20000 --max-num-iterations 20001",
        f"--wandb-name {capture_name}_{table_row_name}",
        f"--experiment-name {capture_name}_{table_row_name}",
        # extra_string,
        table_row_command,
    )
)

Moreover, I observe the GPU utilization is pretty low during the training (GPU 5 is training with nerfstudio and GPU 6 is training with nerfacc). I suspect it might be the dataloader that is bottlenecking the pipeline.

Not sure if it worth to look into this or it is already a known situation to tradeoff flexibility and GPU memory for speed. Apparently nerfacc preload all images while nerfstudio is loading on the fly.
Happy to look into it once I get my hands freed but think would be worthy to post it here and see if anyone wants to take a bite on it.

(Maybe another possibility is viewer contention?)

I think this overhead still exists when the viewer isn't enabled. My guess is the slowdown is in the data loading. My hope is that preprocessing and caching the data as individual pixels will result in a speed up.

Hi, I came from the above issue - did someone find a solution for the slow data loading? the bottleneck seems highly innegligible.
Or, (a question for the contributers) is there a possibility to release a version with the same dataloaders as in instant-ngp (or something based upon it)?
Thanks again!

Is there any solution to the problem?

I think this overhead still exists when the viewer isn't enabled. My guess is the slowdown is in the data loading. My hope is that preprocessing and caching the data as individual pixels will result in a speed up.

By the way, how to disable the viewer?


cross-origin errors

Background-check fails not-so-gracefully if image elements don't share the same origin domain.
Modifying line 43:
attrs.images        = getElements(a.images || 'img', true);

to:
attrs.images        = getElements(a.images || 'img[src^="'+window.location.protocol+'//'+window.location.hostname+'"]', true);

prevents this issue when no image selector is provided, but it's still possible for a user to provide a selector that includes cross origin images.

Just download background-check and in slider example I get:
Uncaught SecurityError: Failed to execute 'getImageData' on 'CanvasRenderingContext2D': The canvas has been tainted by cross-origin data.
The above doesn't fix it.

#60 might help you @webmatrixxxl


Provider flag for init

Problem
When I'm using Prisma and run the init command, it always defaults to using PostgreSQL. It would be nice if you could choose your provider through the init command.
Suggested solution
Perhaps on the prisma init command there could be a flag such as --postgresql or --sqlite, or --provider-postgresql. This would just configure the Prisma project for that provider on initialization.
Additional context
This might already exist, and I'm not aware of it, or it might have been implemented some other way. Thanks!

Related: https://github.com/prisma/prisma/issues/1854

Thanks, should I close this now?

I closed the other ones, as you actually had a concrete suggestion.
From the other issue:
Small note: you can use this cli for reproductions: https://github.com/timsuchanek/auto
It basically does the same thing described but also install deps and setup a reproduction script.
Originally posted by @pantharshit00 in https://github.com/prisma/prisma/issues/1854#issuecomment-824942541

Thanks

In the last 2 years this actually got implemened: prisma init --datasource-provider sqlite changes the provider and the default value of DATABASE_URL.


Add filesystem ops

@mihaimaruseac
This PR adds filesystem ops. Regarding the problems with directory, I copy the current approach. But as far as I know, the curent approach can not guarantee the same behavior with posix filesystem. I think it maybe the good time to change the gcs filesystem design and drop all the fake directory support. In particular, CreateDir, DeleteDir, ... will always return ok. What do you think ? I do not have access to the internal code so could you check it for me. Thank you

So I have done some testing. Turn out the current implementation is more complex than I thought and it has the same behavior with local filesystem. But it has one trade-off, it sometimes shows the wrong information compared to the server.
I have an object 'test/foo' on the server.
tensorflow 2.2.0 returns true if I call tf.io.gfile.exists("test") or tf.io.gfile.exists("test/") but google-cloud-cpp and google-cloud-storage (python) return not found.
I would like to ask if there is any particular reason that we have to keep this behavior ? ( I think the reason is that we want to have the same behavior between filesystems )

tf.io.gfile.exists(path) should return true whenever path exists.
I think the other 3 only return true if the path exists and is a file.
Note that the plugin interface checks for a path, not a file https://github.com/tensorflow/tensorflow/blob/e25d3e084b330de37284f54853295ce28ea93d12/tensorflow/c/experimental/filesystem/filesystem_interface.h#L506-L517

If we do like this, we will check that if there is a object whose key contains that path ( like object test/foo contains test). I think it is not so useful in general ?

Users might ask if a path exists (file/directory) to determine what operations to run in the future. This is much better than the alternative where if they wanted to see if a file exists they can also use IsDirectory and look at the return value (the Status tells them if the path exists but it is a file).

Ok I got it. So first I will copy the current implementation and we could return to this topic after everything is done

I would still like to split this PR into components:

will just create/delete files/directories
will just add support for moving/renaming
will add support for checking if paths exists, if they are directories, for the stats, etc.
will finally allow matching glob patterns and getting all children of a path

Close for smaller PRs


Add checks to env['xyz'], env.Append, env.Replace to either issue warnings or make it harder to do the wrong thing.

This is a WIP issue to gather all the  ideas of what can reasonable be checked when setting, appending, replacing and Environment() variable.
@mwichmann
Started as dicussion on discord
bdbaddogToday at 10:02 AM
I had an idea which probably belongs in #loony-channel ... It should be possible to write a SConstruct Lint mode..
so if you do thinks like env['LIBS']='a' instead of env['LIBS']=['a'], it would warn you etc..
The things an experienced user has done wrong and fixed.
mwichmannToday at 10:05 AM
probably
bdbaddogToday at 10:05 AM
I'm going to file an issue for that so it doesn't get lost..  and we can add ideas to it to flesh out what could be done?
mwichmannToday at 10:07 AM
any kind of checker/linter would be great, been on my wishlist
loonycyborgToday at 10:19 AM
scons scripts are python. Maybe it could be plugin of existing python checker?
bdbaddogToday at 10:22 AM
@loonycyborg - nope..  This would have to hook in to the logic which implements env['XYZ'] = value
mwichmannToday at 10:27 AM
I don't know that that particular one is a problem... env is much like a UserDict, where there's a backing data element (_dict), and pylint doesn't have any trouble with userdicts
but there's other goodies
(or _data, or whatever it's called)
mwichmannToday at 10:39 AM
Yet Another Nitpicky Question:  if a consvar is set to a tuple, should you be able to append to it?
env['FOO'] = ('one', 'two')
env.Append(FOO="three")
what should happen?
bdbaddogToday at 10:43 AM
UserError.. ?
mwichmannToday at 10:45 AM
the current error message is unenlightening and indicates another unexpected path through a rather complex algorithm... AttributeError: 'str' object has no attribute 'insert':
the algorithm here takes new and inserts old in front of it, thus...
bdbaddogToday at 10:46 AM
that's not very helpful error message for sure.
mwichmannToday at 10:46 AM
it does that when they types differ and other conditions are not met, but it guessed wrong here :slight_smile:
bdbaddogToday at 10:50 AM
gotcha
loonycyborgToday at 11:01 AM
would be cool if env variables had some inherent type. Like if variable type is known as list then Appending either list or single item always has well defined semantic
bdbaddogToday at 11:02 AM
Moving to #loony-channel

Followed by
bdbaddogToday at 11:02 AM
@loonycyborg - yes that would make sense for some variables..
All of which (I think) falls under the umbrella of "make it harder to do bad/silly/un-sconsian things"
or make it easier to figure out how to do the "best" thing?
loonycyborgToday at 11:03 AM
Just to make things less tricky
if Append(x = "x") and Append (x = ["x"]) always does the same thing then it becomes less error prone
actually this can be formulated not as strict typing but rather having a strict "kind"
where "kind" is either scalar or list or mapping.
probably it's a good idea to make some env variables to always have some particular "kind"
and scons adjusting all operations like Append to act according to it
so far I encountered many issues due to particular variable having unexpected kind
while issue of wrong type is uncommon since everything tends to be string, or something stringifiable

Each variable can be one of following: unset, None, scalar, list, mapping. Also value that is passed to Append() etc can be anything of the above. That creates quite a combinatorial explosion. I think it's impossible to always do the right thing without knowing what is the variable expected to be. Like LIBS is supposed to be a list, SHLIBPREFIX is supposed to be a scalar etc. Currently SCons sets initial values to some variables that make things less unpredictable but still if you assign something directly to env["something"] you just overwrite it. I think safest would be to have some auto-coercion feature but it would need to hook directly into operations like []/__getitem__()/__setitem__() and also there would have to be a way to tag a variable as particular kind, i.e. whether it's scalar or list or mapping.

I've gotten deeper into this than I ever wanted to so plenty of thoughts here, will probably dribble them out over time.
There is some auto-coercion - e.g. if you Append to CPPDEFINES, it immediately forces the existing value into a list. Well, if it was a string. LIBS needs to do the same bue doesn't. The special-casing extends only to Append and AppendUnique (with the latter containing some logic errors), and misses Prepend and PrependUnique.  The idea of having four separate implementations of almost-the-same-thing was bound to lead to unintented divergence, and we have it. See #3876 for more on that.

I managed to implement all combinations of {Append|Prepend}{Non-unique|Unique} in a single C++ template function: https://github.com/loonycyborg/scons-plusplus/blob/master/src/python_interface/environment_wrappers.hpp#L165
But direct assignment is still not covered, so it's possible to unexpectedly assign a string to env["LIBS"] and a list/dict to env["SHLIBPREFIX"]. If there are custom __getitem__/_setitem__ those cases would be covered too.

The relevant class is actually SubstitutionEnvironment, and of course there's already some "magic" hiding there.

@loonycyborg - test

Just note that in the past this was considered performance sensitive stuff... the __setarttr__ may not want to grow lots of checks.

So could some variant of Typing help?  Right now SCons has a place where it's somewhat aware of types of things, that's in ParseFlags, where there's a dictionary of knows consvars the method might distribute values to. A subset:
    'CPPFLAGS'      : CLVar(''),
    'CPPPATH'       : [],

That doesn't seem too wildly different from:
CPPFLAGS: CLVar = CLVar('')
CPPPATH: list = []

If that information is somewhere, the "checker mode" (or a separate sconslint) could perhaps flag a warning that you just assigned a str to env['CPPPATH']... just thinking out loud, not sure how viable that is applied to dictionary key/values.

Just dropping this here to track an issue that isn't decided yet - modifications to env's in Action functions seem like a bad idea. Is this something a checker could detect and maybe warn about?


Update docs location

Context
*.api.publish-teacher-training-courses.service.gov.uk needs to redirect to /docs as the root domain (*.publish-teacher-training-courses.service.gov.uk) will now serve the Rails app.
In order to do this, we need to move the location of the middleman docs to a different directory and use a URL constraint to handle the redirect behaviour.
This PR introduces some changes to accommodate the above.
Changes proposed in this pull request

Move location of the api docs
Fix broken links (corresponding PR https://github.com/DFE-Digital/tech-docs-gem/pull/1)

There is an on going bug with the rendering of the actual endpoint labels in the docs which is being addressed in a separate ticket. This PR is to just get the new location working and serving the docs under the new path.
Guidance to review
The docs should function the same as before excluding the bug mentioned above
https://teacher-training-api-pr-2433.london.cloudapps.digital/docs/
The route constraint has to be tested in QA.
Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

https://teacher-training-api-pr-2433.london.cloudapps.digital/docs

Discussed IRL, this is fine as it for now


udp: add support for multiple listener filters

Signed-off-by: Xie Zhihao<EMAIL_ADDRESS>Commit Message: udp: add support for multiple listener filters
Additional Description:
Risk Level: Medium
Testing: WIP
Docs Changes: N/A
Release Notes: WIP
Platform Specific Features: N/A
Fixes #15887

Cool. Maybe You can change its status to get reviewed. 👍

I found the patch surprisingly easy, so maybe I missed something I did not notice. But I am going to mark it as ready for review.

It is impossible to create an integration test, since either UDP proxy filter or DNS filter stops iteration at the end of data handling. But I am willing to create an unit test for active UDP listener instead to test its correctness on multiple listener filters.


Reduce duplication by merging worker_groups and worker_groups_launch_template?

Would be great now that we've been on TF 0.12 for a while now.
We could just add a type option to the worker group map to set whether it's LT or LC based?
And then remove workers_launch_template.tf completely?

When I started using this module (in Terraform 0.12), this was my first thought :)
I think this will be done with for_each support and will probably solve https://github.com/terraform-aws-modules/terraform-aws-eks/issues/476

I did a little testing:
with for example this structure:
  worker_groups = [
    {
      type = "LC"
      instance_type = "m4.large"
      asg_max_size  = 5
      tags = [{
        key                 = "foo"
        value               = "bar"
        propagate_at_launch = true
      }
    },
    {
      type = "LT"
      instance_type = "m4.large"
      asg_max_size  = 5
      tags = [{
        key                 = "foo"
        value               = "bar"
        propagate_at_launch = true
      },
      {
      type = "LT"
      instance_type = "m4.large"
      asg_max_size  = 5
      tags = [{
        key                 = "foo"
        value               = "bar"
        propagate_at_launch = true
      }
   ]

Doing this lookup({for r in var.worker_groups: r.type => r...}, "LT")
returns:
[
    {
      type = "LT"
      instance_type = "m4.large"
      asg_max_size  = 5
      tags = [{
        key                 = "foo"
        value               = "bar"
        propagate_at_launch = true
      },
      {
      type = "LT"
      instance_type = "m4.large"
      asg_max_size  = 5
      tags = [{
        key                 = "foo"
        value               = "bar"
        propagate_at_launch = true
      }
   ]

@barryib @max-rocket-internet - I've been considering moving workers / workers_launch_templates to a sub-module like node_groups.
A question I have is: What is the benefit of supporting both Launch Config and Launch Template type worker groups? Can we do everything with Launch templates or are there limitations?
I also think that we could support usage of the node / worker group submodules directly, this would solve the issues that people face when trying to do something after the EKS control plane has been created but before the nodes or workers have been created. Specifically custom CNI configuration.


Gettting could not determine kind of name ZMQ related errors

Hi @dwhitena, I ran into an issue, haven't been able to load the Go kernel yet, this might be the reason:
☻  go get github.com/gophergala2016/gophernotes
# github.com/gophergala2016/gophernotes/Godeps/_workspace/src/github.com/alecthomas/gozmq
could not determine kind of name for C.ZMQ_HWM
could not determine kind of name for C.ZMQ_MCAST_LOOP
could not determine kind of name for C.ZMQ_RECOVERY_IVL_MSEC
could not determine kind of name for C.ZMQ_SWAP
I believe I have successfully followed all of the instructions with the exception of the messages above. gophernotes is listed under my bin directory. But when I open the gophernotes notebook, there is no Go kernel.
Could this be the reason and, if so, how to resolve it?

Hi @carlisia!  Thanks for trying out the notebook.  What version of ZeroMQ do you have?  Also, did you modify the Jupyter kernel.json file?  If so, could you copy the contents here?

Check this out:
☻  brew install zeromq22
Warning: homebrew/versions/zeromq22-2.2.0 already installed

{
    "argv": [
    	"/Users/carlisiacampos/.gvm/pkgsets/go1.5.1/global/go/bin/gophernotes",
    	"{connection_file}"
    	],
    "display_name": "Golang",
    "language": "go",
    "name": "go"
}

Oops, the path is wrong!

Ok, fixed the path (there was an extra go in there) but still the same.

Not the kernel.json file looks like this:
{
    "argv": [
    	"/Users/carlisiacampos/.gvm/pkgsets/go1.5.1/global/bin/gophernotes",
    	"{connection_file}"
    	],
    "display_name": "Golang",
    "language": "go",
    "name": "go"
}

Can you run the gophernotes binary in a terminal, just as gophernotes?  Any output?  Also, what version of Jupyter?

☻  gophernotes --help
Usage of gophernotes:
  -autoimport
    	formats and adjusts imports automatically
  -context string
    	import packages, functions, variables and constants from external golang source files
  -debug
    	Log extra info to stderr
  -pkg string
    	specify a package where the session will be run inside

~GOPATH/src/github.com/gophergala2016/gophernotes
☻  jupyter --version
4.1.0

If you run gophernotes without the --help do you get any output?  Also, you could try putting the kernel file under $HOME/.ipython/kernels.

☻  gophernotes
2016/05/01 15:29:30 Need a command line argument for the connection file.

I'll try placing the kernel file in that dir.

Yeah try ~/.ipython/kernels/gophernotes instead of ~/.local/share/jupyter/kernels/gophernotes

Sorry, got side-tracked.
OMG it works! So awesome. Thank you.


Ethan: Watch<PHONE_NUMBER>

Ethan requests the watch of the watch_number<PHONE_NUMBER>. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Spevacus in Charcoal HQ


Check double type support before using it in benchmarks

Check fp64 aspect at runtime before launching benchmarks using that type. This will allow runtime errors on devices lacking that aspect.

This slightly differs from the upstream PR as we introduced the option to not run hierarchical kernels


unable to send push notifications using FCM/GCM?

Go through this link
https://forums.xamarin.com/discussion/104452/unable-to-send-push-notifications-using-fcm-gcm/p1?new=1

Hi this seems related to send push notifications, not to the plugin. Did you try sending with postman?

Thanks for the reply.
No, i didn't tried with postman.

Now i tried with postman.
Here is the response
{
"message_id":<PHONE_NUMBER>899751000,
"error": null
}

Are you subscribing to the topic you are sending?

No, I want to send the notification to all devices.
To = "/topics/all",  // to send notifications for all devices, Is this Right?

You need to subscribe to "all", there's not such topic by default

//Subscribing to single topic
CrossFirebasePushNotification.Current.Subscribe("all");

Where I have to paste this in ( MainApplication )?
And
To = "/topics/all", // to send notifications to all devices, Is this Right then?

Is this a Xamarin Forms app or an Xamarin Android app?

Xamarin Forms App(PCL)

I would do it typically just were the sample is doing it. On the App.cs

you have added the topic as general in the OnStart method.
To = "/topics/general"  is it fine now?

Yeah,  it's working now.
Thanks for the quick answers.
And How can i send to particular user using FCMClient?

should be just as you were doing, but not sure since haven't used much.


Add requirements checks for official docs

https://trello.com/c/3E2vqBkS/1507-support-official-documents-for-featured-file-attachments
Please see the commits for more details.
In doing the work for this PR, I noticed that the EditionChecker has no spec, despite not being (fully) covered at a higher level. It's difficult to add one due to the lack of a common interface for the checkers it delegates to. For the purpose of this PR, I've avoided making any changes to the structure of checkers, and will look at how to write a test for this in a subsequent PR.

[ ] Work out how to test the EditionChecker

Only thing I'm a bit concerned about is the loops of items to create tests (I have déjà vu from the early days of this project when this was a hot debate).

@kevindew that's a fair comment, as we don't do it elsewhere. I think the main concern with this technique is the complexity of the test being repeated (low in this case, higher in previous discussions) and the data used for the repetition (single strings). I also agree that other ways (I can think) of doing this are less good:

Using a shared example (more abstraction, it's not really shared)
Putting all the checks in one it (similarly unconventional, harder to structure)
Writing out all the tests (tired fingers)


It would be nice to have some documentation, README.md, docs

It could be my limited intelligence.
But I actually think there would be a lot of people who would prefer to have a little documentation, like some samples in the README.md to get them going more quickly using this library.
Now you have to get it all from the code itself, which just takes more time.

Ah wait, I already started to do something but I had missed the included example completely

Some formal documentation would be very useful indeed

I actually already wrote some. I'll do a pull request when I changed my code examples in that with those in the index.html which I discovered after my weiting. As I won't have to test/improve those like I'd still need todo with mine. Weekend first...

I did a pull request a while ago...

For linkage the PR is #57
I really wish more open source projects focused on documentation


Access all calendar in a much more concise way

Thanks for stopping by to let us know something could be better!
Is your feature request related to a problem? Please describe.
as you can see in the github issue, I had to make a super nested loop to just find the current event of the user. Not only this made the end result not work all the time but heavily prone to a break due to the heavily nested structure.
Describe the solution you'd like
events_result = service.events().list(calendarId= "Al"l , timeMin=now,
maxResults=10, singleEvents=True,
orderBy='startTime').execute()
could replace something like All to be able to retrieve all of the users calendar instead of just one.
Describe alternatives you've considered
It would be great if we can also have a feature where with one if not a few lines we can know the google event of the user at the point in time.
Additional context
Add any other context or screenshots about the feature request here.

Unfortunately, this feature request is out of the scope of this library as it is related to the underlying API and not the google-api-python-client library itself. Please follow the steps outlined on the Google Calendar API Help Site if you'd like to submit a feature request to show events from all calendars in a single call to service.events().list(). Follow the steps under the missing features section. Once you create the feature request, please add the link here so that others can 'star' it which may raise visibility of the feature request.

Unfortunately, this feature request is out of the scope of this library as it is related to the underlying API and not the google-api-python-client library itself. Please follow the steps outlined on the Google Calendar API Help Site if you'd like to submit a feature request to show events from all calendars in a single call to service.events().list(). Follow the steps under the missing features section. Once you create the feature request, please add the link here so that others can 'star' it which may raise visibility of the feature request.

I've closed this issue but please feel free to re-open with more information if you have additional questions.

I've closed this issue but please feel free to re-open with more information if you have additional questions.


Compilation bug

Recently I started running into this compiler error:
Build script running, compiling Proj
⏳ Compiling...   > 👀 Enabling debug
   > Proj: tact compiler
   > Proj: func compiler
[ 1][t 0][2024-04-11 07:50:28.469000][Fift.cpp:67]	top: <continuation 0x1b07f20>
level 1: <continuation 0x1b07fa0>
level 2: <text interpreter continuation>

main.fif:5929:	$Proj$_internal_binar:-?
💥 Compilation failed. Skipping packaging
Error: Could not compile tact

This error is not very helpful, and I couldn't isolate the bug.

Sometimes the error is:
Build script running, compiling Proj
⏳ Compiling...   > 👀 Enabling debug
   > Proj: tact compiler
   > Proj: func compiler
File not found
💥 Compilation failed. Skipping packaging
Error: Could not compile tact

I've ran into those errors when I:

modify the layout of large structs
have complex logic branches involving options(?) and !! operator

This looks a lot like #75 and #171

This should be fixed with #297


Refactor FsRepository's renders

The goal is to make it so it is possible to open repo without creating the renders directory for the current user, in situations where that renders directory will not be used. Examples include when using fuse or when running spfs clean.
This introduces different flavors of a "render store" besides RenderStore that either represent a render store that hasn't been created yet (but can be) or the lack of a render store. Operations that don't require a render store can access the local repository without the renders directory being created.
WIP: The test suite passes[^1] with this code but it is unfinished.
[^1]: On my machine... A limitation of the test suite is that it uses the installed spfs binaries instead of the spfs code in the current branch.

Tests are now passing legitimately (at least locally, with an spfs installed from this code). What's left besides general cleanup is creating some new tests that specifically exercise the desired outcome: not creating the renders directory if using fuse. I'm not sure if that is true yet with these changes but the framework for making it possible should be there.

Making progress but the behavior of spfs clean is not right yet (it currently skipping cleaning renders entirely). The existing tests via the fixture create a repo with RenderStore (pre-created renders) but spfs renders uses MaybeRenderStore, then the clean code treats this flavor as not having renders.
I'm trying to get it so spfs clean does clean any existing renders that are eligible without creating any new renders directories.


add parse amount for increase-inbound-liquidity command

Added parsing of amount for increase-inbound-liquidity command.
Files changed:

bos

https://github.com/alexbosworth/balanceofsatoshis/issues/185

Oops. There is a mistake in this. I'll have to change it.

what's the mistake? i can fix it

Fixed now.

ah double tokens, I changed that in the merge


Draft state change diagram related to versions/drafts

Not sure I captured this correctly, @jcoyne, but here's a stub anyway.

@jcoyne can you add the images to the app wiki or README? then we can close this issue.

@mjgiarlo this ticket was for the inception deck, where they are now, so I'm just going to close.


19 update xdr files to latest protocol

Added modifications to allow type building with the latest Stellar .x files.
Upon changing the Rakefile to point to the latest types, two main issues were faced.
First, the generated .ts file is missing the constant declarations. A  post-process script will append them to the file.
Second, some generated structs and enum definitions made references to non primitive types. This created a cyclical reference error upon compilation. For this reason those types must be wrapped in Box pointer.
For instance, the type:
pub struct ScSpecTypeOption {
    pub value_type: Box<ScSpecTypeDef>
}

In this fix these types are flagged by name prefix (ScSpecType..), which is not a future-proof solution. Ideally, the types that require this Boxed reference should be detected automatically and not by name so the code is resilient to future Stellar types changes.

When running npm run build I get the following error:

/Users/marcel/Documents/substrate-stellar-sdk/autogenerator/node_modules/ts-node/src/index.ts:587
    return new TSError(diagnosticText, diagnosticCodes);
           ^
TSError: ⨯ Unable to compile TypeScript:
x2JavaScript/generated/stellar-xdr_generated.ts:1021:22 - error TS2304: Cannot find name 'SC_SPEC_DOC_LIMIT'.

1021   ["doc", xdr.string(SC_SPEC_DOC_LIMIT)],
                          ~~~~~~~~~~~~~~~~~
x2JavaScript/generated/stellar-xdr_generated.ts:1038:22 - error TS2304: Cannot find name 'SC_SPEC_DOC_LIMIT'.

1038   ["doc", xdr.string(SC_SPEC_DOC_LIMIT)],
                          ~~~~~~~~~~~~~~~~~
x2JavaScript/generated/stellar-xdr_generated.ts:1054:22 - error TS2304: Cannot find name 'SC_SPEC_DOC_LIMIT'.

1054   ["doc", xdr.string(SC_SPEC_DOC_LIMIT)],
                          ~~~~~~~~~~~~~~~~~
x2JavaScript/generated/stellar-xdr_generated.ts:1069:22 - error TS2304: Cannot find name 'SC_SPEC_DOC_LIMIT'.

1069   ["doc", xdr.string(SC_SPEC_DOC_LIMIT)],
                          ~~~~~~~~~~~~~~~~~
x2JavaScript/generated/stellar-xdr_generated.ts:1124:22 - error TS2304: Cannot find name 'SC_SPEC_DOC_LIMIT'.

1124   ["doc", xdr.string(SC_SPEC_DOC_LIMIT)],
                          ~~~~~~~~~~~~~~~~~
x2JavaScript/generated/stellar-xdr_generated.ts:1141:22 - error TS2304: Cannot find name 'SC_SPEC_DOC_LIMIT'.

1141   ["doc", xdr.string(SC_SPEC_DOC_LIMIT)],
                          ~~~~~~~~~~~~~~~~~
x2JavaScript/generated/stellar-xdr_generated.ts:1158:22 - error TS2304: Cannot find name 'SC_SPEC_DOC_LIMIT'.

1158   ["doc", xdr.string(SC_SPEC_DOC_LIMIT)],
                          ~~~~~~~~~~~~~~~~~
x2JavaScript/generated/stellar-xdr_generated.ts:1175:22 - error TS2304: Cannot find name 'SC_SPEC_DOC_LIMIT'.

1175   ["doc", xdr.string(SC_SPEC_DOC_LIMIT)],
                          ~~~~~~~~~~~~~~~~~
x2JavaScript/generated/stellar-xdr_generated.ts:1192:22 - error TS2304: Cannot find name 'SC_SPEC_DOC_LIMIT'.

1192   ["doc", xdr.string(SC_SPEC_DOC_LIMIT)],
                          ~~~~~~~~~~~~~~~~~
x2JavaScript/generated/stellar-xdr_generated.ts:1209:22 - error TS2304: Cannot find name 'SC_SPEC_DOC_LIMIT'.

1209   ["doc", xdr.string(SC_SPEC_DOC_LIMIT)],
                          ~~~~~~~~~~~~~~~~~
x2JavaScript/generated/stellar-xdr_generated.ts:1226:22 - error TS2304: Cannot find name 'SC_SPEC_DOC_LIMIT'.

1226   ["doc", xdr.string(SC_SPEC_DOC_LIMIT)],
                          ~~~~~~~~~~~~~~~~~
x2JavaScript/generated/stellar-xdr_generated.ts:1646:36 - error TS2304: Cannot find name 'SCSYMBOL_LIMIT'.

1646 xdr.typedef("ScSymbol", xdr.string(SCSYMBOL_LIMIT));
                                        ~~~~~~~~~~~~~~

    at createTSError (/Users/marcel/Documents/substrate-stellar-sdk/autogenerator/node_modules/ts-node/src/index.ts:587:12)
    at reportTSError (/Users/marcel/Documents/substrate-stellar-sdk/autogenerator/node_modules/ts-node/src/index.ts:591:19)
    at getOutput (/Users/marcel/Documents/substrate-stellar-sdk/autogenerator/node_modules/ts-node/src/index.ts:921:36)
    at Object.compile (/Users/marcel/Documents/substrate-stellar-sdk/autogenerator/node_modules/ts-node/src/index.ts:1189:32)
    at Module.m._compile (/Users/marcel/Documents/substrate-stellar-sdk/autogenerator/node_modules/ts-node/src/index.ts:1295:42)
    at Module._extensions..js (node:internal/modules/cjs/loader:1300:10)
    at Object.require.extensions.<computed> [as .ts] (/Users/marcel/Documents/substrate-stellar-sdk/autogenerator/node_modules/ts-node/src/index.ts:1298:12)
    at Module.load (node:internal/modules/cjs/loader:1103:32)
    at Function.Module._load (node:internal/modules/cjs/loader:942:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:83:12)

I guess the code in post-process.js should actually be executed at the beginning of the generate script and not at the end to avoid this error @gianfra-t ?

Yes that's correct! Sorry, I will change the order.

Thanks, that fixed the error about constant. But now I get the following error for npm run build:

><EMAIL_ADDRESS>generate
> node post-process.js && npm run generate-all-types && npm run format

><EMAIL_ADDRESS>generate-all-types
> DESTINATION=../src/xdr MAIN_FILE_NAME=types.rs npx ts-node x2JavaScript/generated/stellar-xdr_generated.ts

/Users/marcel/Documents/substrate-stellar-sdk/autogenerator/node_modules/js-xdr/src/index.ts:1
import { DefinitionFactory, XdrType } from "../types/types";
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at internalCompileFunction (node:internal/vm:73:18)
    at wrapSafe (node:internal/modules/cjs/loader:1166:20)
    at Module._compile (node:internal/modules/cjs/loader:1210:27)
    at Module._extensions..js (node:internal/modules/cjs/loader:1300:10)
    at Object.require.extensions.<computed> [as .ts] (/Users/marcel/Documents/substrate-stellar-sdk/autogenerator/node_modules/ts-node/src/index.ts:1286:43)
    at Module.load (node:internal/modules/cjs/loader:1103:32)
    at Function.Module._load (node:internal/modules/cjs/loader:942:12)
    at Module.require (node:internal/modules/cjs/loader:1127:19)
    at require (node:internal/modules/helpers:112:18)
    at Object.<anonymous> (/Users/marcel/Documents/substrate-stellar-sdk/autogenerator/x2JavaScript/generated/stellar-xdr_generated.ts:21:1)

Do you have any other local changes that are not pushed? I am on node -v v19.6.0

That one is strange since I was able to run it properly in my local. Let me try to clone it fresh and see if I can replicate that error.

Okay 🤔 are you running it on macOS or a Linux OS?

I am on Linux, but I can confirm now I get that error. Strange. I will look into it.

Okay so I changed the script to a .ts one, nevertheless what solve the error in my case is to install typescript with npm install typescript ts-node --save in the  autogenerate folder. Which is strange since it should be handled by package.json. Please let me know if it works for you, and if you know of a change that wont require to manually run that command!

So you are saying that npm install typescript ts-node --save in the /autogenerator folder fixed the issue for you? I tried and I still get the same error :(
npm install typescript ts-node --save would save those two into the dependencies though instead of devDependencies. Maybe that changes something for you? But as I said, for me it's still the same. Which node and npm version are you using?

Yes that solved it for me! So, npm version 9.8.1 and node v20.6.1, could that be the issue?

Yes, I think is unlikely this will break in the short term. We can create a new issue and wait for @TorstenStueber input on this.
I am updating the README and testing one last time.


Maintain doc README up to date using a soft link in the repo

Apparently this should work, and its better as the onnx.ai/onnx-mlir uses the docs/README.md file directly.
Signed-off-by: Alexandre Eichenberger<EMAIL_ADDRESS>

Jenkins Linux s390x Build #1550 [push] try soft links (#789)  S... started at 18:33

Jenkins Linux ppc64le Build #690 [push] try soft links (#789)  S... started at 18:38

Jenkins Linux amd64 Build #1552 [push] try soft links (#789)  S... started at 17:33

Jenkins Linux s390x Build #1550 [push] try soft links (#789)  S... passed after 48 min

Jenkins Linux amd64 Build #1552 [push] try soft links (#789)  S... passed after 52 min

Jenkins Linux ppc64le Build #690 [push] try soft links (#789)  S... passed after 58 min