Change error message of ced for promote command

Describe your changes
Make promote command throw error message specific to CED message constraints
Edit the corresponding test case to pass the test
Issue ticket number and link
closes #191
Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] If it is a core feature, I have added thorough tests.

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

Files with missing lines
Coverage Δ
Complexity Δ

...edu/address/logic/parser/PromoteCommandParser.java
73.33% <100.00%> (ø)
2.00 <0.00> (ø)


Automatic Session Cleanup Not Executing

PDA version
0.4.1
Python version
3.10
Steps to Reproduce
I have the following setup:
2 PDNS Admin instances, on 2 different servers, sharing the same MariaDB database (PDNS Admin runs in docker, powerdnsadmin/pda-legacy:0.4.0 - I've tested with 0.4.1 and it's the same thing)
2 HAproxy servers, each doing a healtcheck on each PDNS Admin server every 2 seconds (HTTP GET /)
Every time an HTTP GET / request is executed, an entry is created in PDNS DB, table sessions.
This translates into 2 queries * 2 HAProxy * 2 PDNS Admin servers / 2 (every 2 seconds) = 2 entries / second.
In 60 days the table got to about 10 million entries and the table takes up 3GB of disk space.
Expected Behavior
Old sessions should be deleted from database, in order not to fill up the DB
Observed Behavior
Old sessions are not deleted from database

We can confirm the behaviour for 2 independent installations with 2 pods each in k8s.

Same thing here.
Is a workaround there or any perspective for bug fixing?

@joek-office @hardoverflow @technischeHilfskraft @andrei-marinache I have committed a fix for this issue to the dev branch.
Dependency building is jacked up once again, so I'm working on getting that corrected in order for the Docker image to build automatically again.

This issue is a duplicate of issue 1483. Closing.


[BUG]: ELF loader missing ".dynamic" parsing

The ELF loader currently searches for the needed sections in the ELF loader, instead of loading the .dynamic section, and getting it's resources from there. As a result, the ELF loader is incapable to reporting dependencies, but also is a lot slower than it needed to be.

Fixed


Update existing repo:

When a user submits a repository that already exists, a page is returned where he can choose if he wants to go to the existing source or force the update.

Added a new action "Update" (and a View) in the UploadController.
Refactored some of the ProcessSolution logic into the UploadRepository class.
As suggested in the comments (by Amadeus Wieczorek), added support for multi-solution repository and parallel solution processing.
(Also proposed in issue #114)

Thanks for the contribution!
It's been a long time since we've done anything here, but I'll try to jog my memory and take a look this weekend.
/cc @AmadeusW

Yes, I noticed the repo's inactivity. Thank you for your time.
I'm actually using SourceBrowser for a project of mine, and I came across the inability to update the previous version that I uploaded, so this is the main functionality that I'd like to see in production.
There is one problem with the multi-solution repository support, though: references of types defined in another solution are not recognized during the processing, therefore you're not able to "click" on a type that is defined in another solution.
Let me know if this PR requires some refactoring to be ready to be merged.


Do what is necessary to get travis working in master

Merged over the stuff in development branch that made it work properly there.

Confirmed. Not breaking unittest on dynamic version on node and assembled with jsunit.
But I noticed root/testSuiteDyn.html had some failures. Could you check on this?


Update windows-autopatch-device-registration-overview.md

Updated to remove incorrect information and align with other docs

Why

Closes #[Issue Number]

Changes

@tiaraquan FYI on this PR

As discussed, closing PR as I tackled this in a different one :)


Fix scaling functions

Description
This PR corrects two small errors that causes the improper results reported in the referenced issue:

A copy paste typo (most likely) which sets the Dy component of the image rectangle to Dx.
The tmp array in fastBoxSampling was not properly reset because the last argument in memset should be the number of bytes to fill in and not the number of elements.

Reference issue
Fixes https://github.com/pion/mediadevices/issues/312

I found another problem when using  the nearest neighbor scaling function when the same size as the input image:

It is demonstrated by the test added in d56d4fb42127acf4b34b821c81aca55dc453f99b.

The need for the fix introduced in ee67dee4b812bb68e026cfb0b57115d9fb8a1b45 is demonstrated by the test added in 5f2216ace1313f2e521e9fe4b5ee0b82bff0f590.

The need for the fix introduced in ad6b0ad5e1c7878befe0dca92d82e09b3dae345d is demonstrated by the test added in edef5f5dee11be34eb8c12b719064b2764b6fd4d

@at-wat I added some tests and tried to demonstrate they would fail on master by reverting each change and let CI fail.


Create the SBOM_BLOB_URL for v0.2 buildah tasks

Now that the BASE_IMAGE_DIGESTS result has been removed, there should be enough room for us to re-add the SBOM_BLOB_URL. This will enable EC verification of the SBOM based on the digest which is recorded in the provenenance. It will prevent supply-chain attacks which are driven by modifying the floating tag of the uploaded SBOM.
Even with the referrer's API, the digest can be used to identify which SBOM should be the one built from Konflux.
This was added in #645 and then removed in #654 due to the limitation of Tekton results' size.

Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.

Can we also stop injecting the SBOM into the image (maybe bump the version to 0.3 for responsible versioning?)

For open-source-ness: discussed on internal slack, decided to defer this to a separate change because it could have some implications


MSID: 92495 Version: 2 DOI: 10.1101/2<IP_ADDRESS>1724

MSID: 92495
Version: 2
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>1724
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=92495
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>1724

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/92495v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>1724
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/92495/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

@fred-atherden Adam Charles' email address is incorrect

Thanks for flagging @naushinthomson. I have uploaded a corrected package to s3://prod-elife-epp-meca/92495-v2-meca.zip. It should be fixed in a couple of hours


fix(nuxt): tree shake all occurrences of <DevOnly>

🔗 Linked issue

Various issues exist that may or may not be related to this bugfix.
The main problem solved is that the first occurence of <DevOnly> is not properly stripped from module sources, likely leading to every component within not to be tree-shaken. At runtime the behaviour is not noticable because of the fallback feature / rendering using import.meta.dev.
There does exist an issue which seems related to it: https://github.com/nuxt/nuxt/issues/21012.
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Because RegExp is stateful, the .test call to bail early causes the lastIndex to be set after the first occurence of the DevOnly. This causes the following matchAll to exclude the first match of DevOnly tags.
Setting lastIndex to zero resets the state of the RegExp.
I came about this issue while trying to clone the devonly implementation for my own (a variation controllable with an env var, so that we can have some extra tools on an internal version of the system.
This while production will not expose the tools.
As I was debugging my variation on the compiler plugin I noticed that the code sometimes still.

📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Nice spot. I think we should just use a non-global regexp here though, as we do elsewhere:

Ah yes, then you avoid the whole stateful ness alltogether.

Seems to be ok for me locally through a unit test. Do you want me to merge it within this PR ?
I'll take a look at what's going on with the fixtures tests

Thank you ❤

strange... can't reproduce the failing tests locally 🥲 ...

@huang-julien To reproduce, run pnpm nuxi build test/fixtures/basic && node test/fixtures/basic/.output/server/index.mjs. Then open http://localhost:3000. You should see there's been a hydration error and the content of the page (from the DevOnly component onward) is reproduced twice.


core: prohibit reentry to Notary withdraw

If we're withdrawing funds to contract that has onNEP17Payment method, then it may call Notary's withdraw one more time, but the account's state is not yet updated by this moment.
The problem is similar to https://github.com/neo-project/neo/pull/2734.
@roman-khimov, if I'm not mistaken and the problem exists, then you'll be glad to see this fix. I'll create test for it.

We don't filter accounts that deposit to notary, so even though usually we assume them to be simple single-signature accounts technically contracts can deposit some GAS and play this game. Yeah, this must be fixed.


create unit tests for UI

The UI needs unit tests to verify the rendering and behaviour of the UI components
related issue(s)

#445

This framework has been suggested for creating unit tests<EMAIL_ADDRESS>


models/release: Add note() method

Change-type: minor
See: https://www.flowdock.com/app/rulemotion/resin-devices/threads/Qm7IbwavwgvLoJ5THN0Lfoo86Mk
Signed-off-by: Thodoris Greasidis<EMAIL_ADDRESS>
Contributor checklist

[ ] Includes tests
[ ] Includes typings
[ ] Includes updated documentation
[ ] Includes updated build output

@balena-ci I self-certify!

@balena-ci rebase


"Unable to initialize 3D output" after installing anything with Addon Manager

Please attach a copy of config.yaml or config.ini to the issue.
I don't know where to find this. God forbid you include instructions on where it should be located. The only file named config I could find is the config.xml file in the GW2 Addon Manager's directory, which I can't attach.
Describe the bug
I reported this issue here with multiple other people responding with the same issue, and it was closed without explanation. Terrible user support.
From that issue, because nothing has changed:

Like the title says, no matter what I install, not just the DirectX 12 proxy, I get this:

My GPU supports DirectX 12, my drivers are up to date, and I reinstalled the GPU drivers and DirectX 12 just to be safe. Nothing changes.
The only way I've found to fix the issue once I get that error is to delete my bin64 folder. None of the actions available in the manager fix it.

To Reproduce
List steps to reproduce the bug, if possible.

Launch GW2 Addon Manager
Install literally any addon on the list
Try to run game
Above error is returned

Additional context
Add any other context about the problem here.
Don't close issues that are not resolved without even giving a reason. That is not how you support a project
don't forget to attach the config file to the issue

Terribly sorry you feel the "user support" experience for this free project ran by volunteers in their personal time is not up to your standards. Until a few days ago, nobody was maintaining this project, maybe you preferred that?
Unfortunately for you, this issue does not appear to be related to the addon manager. This, plus its age, is why it was closed. Feel free to open an issue on the addon loader repository or the d912pxy one (though good luck there seeing as it's been deprecated).

The OP should consider what might possibly be wrong with his GW2 installation, because that is a GW2 error. I wouldn't blame the mods, because if you have to delete BIN64, then you are deleting the GW2 system files, which are likely rebuilt by running the program again.
Also, you might want to verify that you have DirectX9 (not DirectX12).  I understand that your GPU supports DX12, but GW2 uses DX9, and the game cannot be run without it.  In fact, I believe even the DX11beta still requires DX9 at the moment.
So my recommendation is to cleanly uninstall all mods.  Delete the Addons folder and the d3d9.dll in bin64, and d3d11/dxgi.dll in the root if they exist, and addonloader.dll in the root. Remove the mods and the AddonManger from the equation and see if the issue persists.  If it does, then you are barking up the wrong tree and should do a Guild Wars repair.
If it works, then you should attach the AddonManager log, gw2al_log.txt in the GW2 root. Here is an example:

You can always ask for assistance as well on the Discord for the GW2 Development Community.  I have found not only the developers of this project (and its new maintainer), but also the developers for other associated projects very helpful and very quick to respond.

@Friendly0Fire I appreciate that you seem to have taken up this abandoned project and are doing this in your free time. However, that's not super useful in terms of issue reporting if you give zero support or feedback. I don't think it's too much to ask that you give a reason when you close an issue that multiple users are experiencing.
The issue was less than two months old, was also being experienced by at least two other users, and only occurs when installing addons with the addon manager, so I don't know how you've come to the conclusion that it has nothing to do with the addon manager.
The issue is definitely NOT d912pxy, because it works fine when I install it manually, and I explicitly stated that this happens regardless of which addon I install. ArcDPS and your own GW2Radial also kick out this same exact error when installed with the Addon Manager.
If you could possibly give some context as to why this is supposedly the addon loader's fault, or where I could even find that repo, that would be helpful.
@xivor I appreciate that unlike Friendly0Fire you're actually considering the issue critically and expanding upon your thought process, rather than dismissing it out of hand for no discernable reason.
Technically, I have figured out since posting that I only have to delete the d3d9.dll added by the manager. Also, as I thought I made clear in the description but perhaps I could have been more clear, the game runs perfectly without addons, and fixing the issue by deleting bin64 restores the game to working order. This would be with d9. I have not opted into the d11 beta for more than a few minutes before turning it back off.
The issue does not persist even just deleting the bin64, so I don't know why you think it might persist if I remove everything. I don't say this to be glib. I'm honestly curious what that would change.
I have tried from clean GW2 installations, and the game runs fine normally, but the second I install any single addon with GW2 Addon Manager, the game breaks with the above error. I will attach my gw2al_log.txt from my most recent attempt installing addons to a clean GW2 install, but there's very little in it.
To be clear, I can manually install GW2Radial, ArcDPS, and d912pxy together and everything works great. None of these addons are breaking anything in and of themselves.
It seems to me that there's an issue with the d3d9.dll file being created by this program or how it's being read, which is throwing an error in DirectX9 and causing GW2 to crash.
gw2al_log.txt
l.

THis is what my log file looks like in a DX9 configuration:

Does your GW2 root installation have this:

Does your addons folder have this:

Which contains this:

And of course, do you have this if you are using arcDPS:

Which contains this:

If missing any of these, I would do the following, which has helped me in the past. Heck, I would do this ANYWAY, because it ensures a complete installation of the entire AddonManager and mods (from as far as I can tell):

When trying to test things and uninstalling and reinstalling mods, I have made mistakes where files were deleted or still there when I didnt want them, and this always seemed to fix the problem for me.


listen on <IP_ADDRESS> instead of <IP_ADDRESS>

Hello,
I tried to make ketchup listen on <IP_ADDRESS>, so other computer can access it.
I made following configuration file based on https://github.com/octavore/nagax/blob/master/router/module.go
but still it's listening on localhost.
austin@cti:~/work$ cat config.json
{
  "bindext": true,
  "port": 9999,
  "data_dir": "data",
  "bolt": {
    "path": ""
  },
  "backup": {
    "enabled": false
  },
  "themes": {
    "path": "themes",
    "registry_url": "http://themes.ketchuphq.com/registry.json"
  }
}
austin@cti:~/work$ ./ketchup start
2017/12/17 13:42:50 listening on <IP_ADDRESS>:9999
2017/12/17 13:42:50 [INFO] done rebuilding router
2017/12/17 13:42:50 [INFO] loading certs from data/tls/*.key

Looks like it's because the version of nagax in this repo was out of date. I've updated it on master now so bindext should work; note that you'll need to run make prepare-vendor again after pulling to sync the new dependencies.

Thanks. Confirm fixed.
austin@cti:~/golang/src/github.com/ketchuphq/ketchup$ ./ketchup start
2017/12/17 15:00:30 [INFO] listening on <IP_ADDRESS>:9999
2017/12/17 15:00:30 [INFO] done rebuilding router
2017/12/17 15:00:30 [INFO] loading certs from data/tls/*.key


Add support for streaming results

As per the recommendation in Neotest, it would be great to add streaming capabilities to this adapter. Since Jest supports a multitude of configuration variations out of the box, I hope this is not too complicated to do.
Ideally I'd like to see Neotest results auto update on save so that I can easily iterate changes. As it stands today I have to trigger that myself on each change however and it gets tiring quite fast. It ends up being easier to get another terminal with jest --watch running instead but I'd love to change that and get the feedback instantly, inside Neovim.

Hey!
I've just added a PR that allows streaming results. You can take a look and see if it works for you. Open to any comments and opinions

Unfortunately I am having trouble getting this working. It seems to run the tests once but I don't get any updates. How can I troubleshoot/debug this?
neotest config
local neotest = require('neotest')

neotest.setup({
  icons = {
    running = "🏃",
    failed = "✖",
    passed = "✔",
    skipped = "⤵",
    unknown = ""
  },
  adapters = {
    require('neotest-jest')({
      jestCommand = "yarn test",
      cwd = function()
        return vim.fn.getcwd()
      end,
    }),
  },
})

vim.keymap.set('n', '<Space>tt', neotest.run.run)
vim.keymap.set('n', '<Space>ta', neotest.run.attach)
vim.keymap.set('n', '<Space>tf', function() neotest.run.run(vim.fn.expand('%')) end)
vim.keymap.set('n', '<Space>ts', neotest.summary.toggle)
vim.keymap.set("n", "<Space>tw", function() neotest.run.run({ jestCommand = 'yarn test --watch ' }) end)
vim.keymap.set('n', '[n', function() neotest.jump.prev() end)
vim.keymap.set('n', ']n', function() neotest.jump.next() end)

Likewise, from my quick attempt running tests in watch the tests do seem to be rerun, but neotest does not update the summary or panel.


v1.27.0[docker-healthcheck.service] is broken

Hi when we run flatcar linux with mc v1.27.0  the container-runtime "docker" get constantly killed as the health-monitor think docker is not functional.
bash -x ./health-monitor.sh.bak container-runtime
+ set -o nounset
+ set -o pipefail
+ [[ 1 -ne 1 ]]
+ KUBE_HOME=/home/kubernetes
+ SLEEP_SECONDS=10
+ component=container-runtime
+ echo 'Start kubernetes health monitoring for container-runtime'
Start kubernetes health monitoring for container-runtime
+ [[ container-runtime == \c\o\n\t\a\i\n\e\r\-\r\u\n\t\i\m\e ]]
+ container_runtime_monitoring
+ local -r max_attempts=5
+ local attempt=1
+ local -r crictl=/home/kubernetes/bin/crictl
+ local -r container_runtime_name=docker
+ local 'healthcheck_command=docker ps'
+ [[ docker != \d\o\c\k\e\r ]]
+ timeout 60 'docker ps'
timeout: failed to run command 'docker ps': No such file or directory
+ (( attempt == max_attempts ))
+ echo '1 initial attempt "docker ps"! Trying again in 1 seconds...'
1 initial attempt "docker ps"! Trying again in 1 seconds...
+ sleep 2

https://github.com/kubermatic/machine-controller/commit/88b46cccdaa3352d657c1b7a0d85a5ae029459da#diff-882f730a6b1860cdad806bdb544592fa90ef5e91f1e50e1f45cf61bd6b5bc727
i reverted the changes to the container-runtime function
--- health-monitor.sh.bak	2021-03-15 12:15:39.000000000 +0000
+++ health-monitor.sh	2021-03-15 12:16:03.000000000 +0000
@@ -30,7 +30,7 @@

 # We simply kill the process when there is a failure. Another systemd service will
 # automatically restart the process.
-function container_runtime_monitoring() {
+function container_runtime_monitoring {
   local -r max_attempts=5
   local attempt=1
   local -r crictl="${KUBE_HOME}/bin/crictl"
@@ -45,7 +45,7 @@
   fi
   # Container runtime startup takes time. Make initial attempts before starting
   # killing the container runtime.
-  until timeout 60 "${healthcheck_command}" >/dev/null; do
+  until timeout 60 ${healthcheck_command} >/dev/null; do
     if ((attempt == max_attempts)); then
       echo "Max attempt ${max_attempts} reached! Proceeding to monitor container runtime healthiness."
       break
@@ -54,7 +54,7 @@
     sleep "$((2 ** attempt++))"
   done
   while true; do
-    if ! timeout 60 "${healthcheck_command}" >/dev/null; then
+    if ! timeout 60 ${healthcheck_command} >/dev/null; then
       echo "Container runtime ${container_runtime_name} failed!"
       if [[ "$container_runtime_name" == "docker" ]]; then
         # Dump stack of docker daemon for investigation.

Now it is working again
bash -x ./health-monitor.sh container-runtime
+ set -o nounset
+ set -o pipefail
+ [[ 1 -ne 1 ]]
+ KUBE_HOME=/home/kubernetes
+ SLEEP_SECONDS=10
+ component=container-runtime
+ echo 'Start kubernetes health monitoring for container-runtime'
Start kubernetes health monitoring for container-runtime
+ [[ container-runtime == \c\o\n\t\a\i\n\e\r\-\r\u\n\t\i\m\e ]]
+ container_runtime_monitoring
+ local -r max_attempts=5
+ local attempt=1
+ local -r crictl=/home/kubernetes/bin/crictl
+ local -r container_runtime_name=docker
+ local 'healthcheck_command=docker ps'
+ [[ docker != \d\o\c\k\e\r ]]
+ timeout 60 docker ps
+ true
+ timeout 60 docker ps
+ sleep 10
+ true
+ timeout 60 docker ps
+ sleep 10

also the braces are not needed in as the function keyword is given:
https://github.com/kubermatic/machine-controller/commit/88b46cccdaa3352d657c1b7a0d85a5ae029459da#diff-882f730a6b1860cdad806bdb544592fa90ef5e91f1e50e1f45cf61bd6b5bc727R33
Please have a look. Thanks in advance

@HaveFun83 seen we will have new patch release to include the fix.

@kron4eg thanks a lot for the quick response.


Compilation error for ios or mac

Godot version:
3.2
Issue description:
I try running
scons platform=osx arch=x86_64 --jobs=$(sysctl -n hw.logicalcpu)
and
scons p=iphone tools=no target=release arch=arm
I keep getting the same issue
can_build() takes 1 positional argument but 2 were given
I looked into this and I know can_build was updated to receive 2 parameters, but for some reason I dont have that. I am pointing to the master branch and did a pull before doing the command.
Is there something im missing what do you recommend I do, not sure where to look to solve this

The master branch is Godot 4.0, not 3.2.
The error comes from a thirdparty module that you must be using and whose config.py is not compatible with the version of Godot you're building.
The error is weird though as can_build takes 2 arguments both in 3.2 and in master, and not 1. Are you sure the error is not the other way around? Otherwise you might be building a very old Godot branch instead.

My bad, thought the latest was 3.2, I'll try switching to 3.2. but yeah i am sure that the message from the console. It asks for 1 argument and 2 is given.
This is the full console message
scons platform=osx arch=x86_64 --jobs=$(sysctl -n hw.logicalcpu)
scons: Reading SConscript files ...
Building for macOS 10.12+, platform x86-64.
TypeError: can_build() takes 1 positional argument but 2 were given:
  File "/Users/cesar.amaro/Documents/godot-3.2/SConstruct", line 544:
    if config.can_build(env, selected_platform):

Ok, so yes, it means you have a thirdparty module whose can_build() definition only takes one argument and not two. So it's not suitable for Godot 4.0.

If you want to build Godot 3.2, you should use the 3.2 branch (3.2.3-beta) or a tag specific to a stable version (e.g. 3.2.2-stable).

Got it, also how can you tell is a third party I could not pick that up from that :/

Well I know that all modules part of the Godot repository compile fine, so it must be a module that you added yourself (thus it's a third party module).

Noted, apologies

No problem :)
I'll close as it's not an engine issue then. If this module comes from a thirdparty with a public issue tracker, you should open an issue there to ask them to update their config.py for Godot 3.1+.


Fix race condition between Authorizer and BotResource

Describe your changes :
BotResource depended on DefaultAuthorizer for adding the roles to the bot's users, but DefaultAuthorizer initialization happened after BotResource ones. Due to this bug, the EntityUpdater class instantiation failed with an NPE when the updating user of the updated entity was not an admin since the SubjectCache instance was initialized by the DefaultAuthorizer.
Type of change :

[x] Bug fix

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers
Backend: @open-metadata/backend

@nahuelverdugo isn't DefaultAuthorizer starting ahead of all the resources? during the server start

@harshach no, it is initialized after the resources. I think we should take out from the resources the initialization of the system objects. For example, we initialize the Organization in the TeamResource class and we have a dependency on it on the UserRepository here.

@nahuelverdugo please look at the line 130 that's where we are creating the authorizer https://github.com/open-metadata/OpenMetadata/blob/main/openmetadata-service/src/main/java/org/openmetadata/service/OpenMetadataApplication.java#L130 ( register authorizer)
and on line 150 registerResources https://github.com/open-metadata/OpenMetadata/blob/main/openmetadata-service/src/main/java/org/openmetadata/service/OpenMetadataApplication.java#L153

@mohityadav766 @nahuelverdugo if we initialize the cache instances after the resources initialize that would resolve it ?


Document decision to use Next.js App Router

Ticket
Resolves https://github.com/navapbc/template-application-nextjs/issues/233

@lorenyu Good question. Updated the decision outcome section to mention that: https://github.com/navapbc/template-application-nextjs/pull/244/commits/85c945c9c4dc34b000c8c5201c23e1d60136f606


Neptune Logger Proxies

🚀 Feature
It would be nice to use the NeptuneLogger with proxies as Neptune Client does support this.
Motivation
Many compute clusters have proxies.
Pitch
It would be nice to initialise NeptuneLoggerlike this:
logger = NeptuneLogger(proxies={'http':'a', 'https':'b'})

The class has a method with first lines:
    def _create_or_get_experiment(self):
        if self.offline_mode:
            project = neptune.Session(backend=neptune.OfflineBackend()).get_project('dry-run/project')
        else:
            session = neptune.Session.with_default_backend(api_token=self.api_key)

One could easily augment
            session = neptune.Session.with_default_backend(api_token=self.api_key, proxies=self.proxies)

after copying proxies into self.proxies.
See: https://docs-legacy.neptune.ai/api-reference/neptune/index.html#neptune.Session
I would be happy to give it a try.
Alternatives
N/A
Additional context
@kamil-kaczmarek Is this impacted by the #6867 issue?

Hi @cemde ,
I can confirm that we work on the improved NeptuneLogger -> you will have an option to specify proxies for NeptuneLogger.

Thanks for the reply and the upcoming improvement. I assume it will be available from v1.5. Until then I will just use my fork.


Issue | Remove Dark mode TW classes

Dark mode is currently not configured and out of project scope at the moment.
However, there are places where dark: Tailwind class is used in certain places.
In Tailwind, the dark:[class] is applied automatically if user's default theme in OS is set to dark.
Therefore, Remove all tailwind classes with prefix dark: so that all type of users see uniform styling.
Screenshot:

hey, can I work on this issue?

Hey @aadeshkulkarni , here's the PR - #12


How to get trip data?

Hi, I was wondering how you guys get the data for an individual trip? From what I can see, there is no dedicated parameter (as opposed to many other car manufacturers apis), all which is there for me is "mileage_of_the_day", which aggregates all trips of a day. And there is mileage_since_last_charge, which seems to aggregating as well and resets whenever the car bets charged e.g. recharged.
So are you calculating trip data from any of these? If so, how?

I've had a play with the API data....looks like we have trip info available:

We can expose the "last trip mileage" counter to HA, it seems more reliable than "since last charged" and "since today"

I thought nobody would comment on this, so thank you @nanomad for getting back to me. Initially I played with "last_trip_mileage" in HA, but it gave me wrong readings. I eventually realized that it gets resetted at around 4pm each day.  But I am unaware where to set it. Even in the app I could not see anything with regards to this.
Do you have any idea?

Does the trip id change as well? I've just started investigating this yesterday night ☺
I can definitely confirm that the other mileage values are useless as they get reset randomly as well (but we can compute at least the since last charged via automation so that's not a big deal)

Kudos for investigating! With rgds to Trip ID - not sure, I need to check. Need to do some driving beforehand, the car is sitting in the garage for 2 weeks now. Will update ...

@bjoernkowarsch I'm releasing the "current trip distance" sensor (disabled by default) with 0.6.X.


Development: Add support for Apollon 3.0.0

This PR updates the version of Apollon used in Artemis to 3.0.0. This Apollon update adapts the format how diagrams are stored, adds support BPMN diagrams in a beta version and allows user to zoom in and out on the canvas.
Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Motivation and Context
This PR is intended to update Artemis with the latest version of Apollon.
Steps for Testing
When testing this change, the following 4 areas are critical.

[ ] 1. Modeling by student in course and exam exercises
[ ] 2. Creation of example solution by instructors
[ ] 3. Assessment of a model by tutors
[ ] 4. Creation of a quiz drag and drop exercises by instructors

For all these, it should be ensured if Artemis still behaves as expected in regards to the Apollon integration and if the zoom-in and zoom-out functionalities are available and operating as expected on the Apollon canvas. Additionally, BPMN diagrams should be available for creation but marked as still in beta.

Somehow the deployment through github doesnt work. Maybe you have to update your branch?
There are some Merge conflicts which have to be resolved beforehand.

Blocked by another issue found during the testing session, the assessment does not seem to work for modeling exercises

The issue #7484 mentioned in #7419 (comment) seems to arise from this PR, I could not reproduce it on develop but when using this branch.
The exercise stays in a broken state, which is why we thought it would also be broken on develop.

This issue should be fixed now. It was caused by the UML parser not being able to parse diagrams created with Apollon v3, so I restructured the parser into separate parsers based on version, meaning everything should work fine with both V2 models and V3 models.


Add my site to blogs list

Add cboxes.icu

Thank you for considering my pull request! I have added my blog to the sites.json file as requested. Please let me know if there are any issues or further changes needed. Have a great day!

Thanks for the submission! It seems like the value ought to be www.cboxes.icu? Can you update to that value? Thanks!

Hi, I have updated the blog URL to www.cboxes.icu as requested. Please let me know if there are any further changes needed. Thanks!

Thanks @Aduhelm! Have a great week!

Sorry, I missed updating a part. Thank you very much.


feat(devimint): start from a previous state (skip setup)

I need a way to verify certain behaviors manually between versions, etc. Similiar to upgrade tests etc.
It occurred to me that it should be possible to just start devimint skipping all the DKGs and other setups, and it should be relatively easy. So I implemeted it.
One blocker that I've hit was inability to tell fedimintd to use certain URL to connect to bitcoind, instead of whatever was added to the config. I made new env variables that can be used to force bitcoind kind&url. The way existing env vars are working is confusing (they are only used as a default for DKG, so I renamed them while at it (with backward-compat.  of course).
Another blocker was that on killing devimint, we send kill -9 right away, which makes bitcoind not get any chance to flush database to disk, and thus lose block history. All other daemons get really upset about it.  A bit slower (fedimimtd takes some time to shutdown), but works, and seem better. We can fix & improve with time.
The implementation is a bit meh, but can be improved with time. I would also like to modularize the setup so e.g. test that doesn't need LND, or electrs, etc. does not need to start them, and can configure it via builder.
Testing
Start devimint e.g. like this:
> just devimint-env
...
2024-04-13T04:29:23.013121Z  INFO fm::devimint: Devimint data dir path=/tmp/devimint-3321804-158

Stop it, and then start it again, but skip the setup.
env FM_TEST_DIR=/tmp/devimint-3321804-158 FM_SKIP_SETUP=true RUST_LOG=fm::devimint=debug just devimint-env

It takes it sweet time as LND Gateway blocks on LND starting to return route hints, but works.

Would be great for @maan2003 to ack it at some point.

next week, a bit busy

@maan2003 Thx. I think we can just remove the label to mark the missing approval as taken care of.


Obscure NSFW content title

If NSFW content is not enabled in settings, thumbnails for NSFW content are obscured, but the file titles are not. This change hides the title information, so no (likely) NSFW text (in the title or username) is shown either. The implementation before redesign actually blurred both the image and the text.
It looks like this now:

Personally, I think all NSFW content should be omitted from the home screen is disabled, rather than having a home screen full of pointless blurred spaces.

@Smurphonomics I agree. I'm pretty sure there is an issue to add an additonal setting which would remove NSFW content from the app.
There will be two settings (worded better than what I have):

Obscure NSFW files
Hide all NSFW content

Maybe instead of hiding the title all together, we can show a message? Saying this is hidden because you have a setting checked.
Also should we hide the channel name too?
@liamcardenas @tzarebczan @kauffj

I am very much in favor of allowing you to navigate to the file page and showing the full title there (still obscuring the thumbnail). I do not like how you have to turn off the setting to view NSFW content.

@kauffj Yes, the point 4. was why I made this PR. If only to prevent this happening while a more permanent solution is implemented.

@miikkatu Ah, I'm a bit slow sometimes (actually I was racing through email and should have taken longer to get the full context). Good and thoughtful change!

@seanyesmunt I like how it was done in the previous version of the app - I believe the channel name + entire card was blurred which alerted the user and gave them an option to go into settings.
What do we want to go forward with in the redesign release?

Lets hide the channel name and title, and show a message:
"This content is obscured because it is NSFW. You can change this in your settings"
There is probably a better way to word that.

@seanyesmunt Your wording sounds good to me. I updated the code, and it looks like this now:

@miikkatu can we make "settings" a link to the settings area? Make it a different color so it stands out a bit.
Previous version for reference:

How about having a separate NSFW tab?
When clicking the NSFW tab everything is blurred with a message saying that it can be enabled in the settings, maybe the settings can even have an option to hide the NSFW tab altogether with an option to put a password on re-enabling it.
Now the homepage is completely family friendly, in fact, the whole app is family friendly. The NSFW tab can have separate discover, trending, top bids etc.
I think this solution is beneficial for both those who are interested in NSFW and those who are not.

@m160 not the right place to discuss, but we'll enable something similar to that down the road where users will need to opt into certain types of content. For now, this behavior is staying as such.

@tzarebczan Should we go with the original wording then?

@miikkatu I think the original wording is fine for now. There is a better description in the settings page.


Version change to 2.0 in README

I am little confused but looks like a typo.
In mix.exs, the version is @version "2.0.0" so should we not use ~> 2.0 in Installation section now.
Installation section has ~> 1.2
# mix.exs

defp deps do
  [{:guardian, "~> 1.2"}]
end

Please ignore if I missed some hidden context here

Codecov Report

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

@@           Coverage Diff           @@
##           master     #621   +/-   ##
=======================================
  Coverage   86.52%   86.52%
=======================================
  Files          22       22
  Lines         438      438
=======================================
  Hits          379      379
  Misses         59       59

Continue to review full report at Codecov.

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

Thank you!


docs: rename README.md to README_CN.md,add new README.md

re #40

Hey~ no need to add chinese readme.


Unsupported APIs

Add list of APIs that throw PlatformNotSupportedException on .NET Core.

I'm not convinced this should be positioned under the "Port from the .NET Framework" section of the TOC, but also not convinced it belongs in the .NET Core breaking changes TOC. Open to suggestions.
@PriyaPurkayastha @marklio Could you review please?

Looks good. It would be great if the last column for Platform can be aligned.

@mairaw Are you ready to approve this?


Apps 193 error handling

Will aim to review the Java by end of day next Wednesday, the fifth. Cheers.

Had some issues getting my comments to show up due to the new Reviews system, sorry about that @parikkk. Everything should be alright now, thanks for your patience.

I'll take a look at the Java by tomorrow


vi editor operation error in Windows

in Windows Env, when I connect vmware's virtual machine( ubuntu ) with SSH, everything's OK,but when i use vi/vim editor, it can's use directionkeys.

I have same problem


Boxplot readme description + properties

This adds documentation to vx/boxplot.  It was brought up (by me) in this issue.
I tried to follow the style in the rest of the documentation, so I added

A short description of what a boxplot is
A table documenting the properties of the <BoxPlot /> component

Questions I have:

Is this written in the correctly?
Is more information necessary?
Is a picture necessary?

P.S. This is my first PR (ever!), so please let me know if there's anything I missed.  I'm still learning to etiquette of OSS 👶

@trainorpj thanks for your interest in vx and for making the docs better for everyone! 🎉
The description looks good to me and with the PropTable I think it's complete enough to get someone up and running 👍  Congrats on your first PR 👏


 That includes:

2FA via OTP or WebAuthn
main credential via WebAuthn only
passwordless login (send email, click link)

webauthn-rs could be used to implement an experimental WebAuthN support. I was planning in using that to build my own auth server for a matrix based application that should have passwordless single factor authentication. Glad to see the plans for Authentication Services and better that the implementation is in Rust :)


Download browser in npm test step, chrome stable if BROWSER and/or BVER are not specified

Description
Download browser in npm test step
Chrome stable if BROWSER and/or BVER are not specified
Purpose
Download browser automatically locally and on travis.
@fippo PTAL

Interesting, even the pestering Chrome Beta installation now works :)


Errors are never displayed

DiscordPHP silences all the errors, making it impossible to know what is failing in the program.
Code to reproduce:
$discord = new Discord(...);
$discord->on("message", function($message){
    throw new \Exception("Test exception");
});

This is a known issue, see https://github.com/teamreflex/DiscordPHP/issues/358

This is a known issue, see #358

It was closed. Is there any fix?

Did you update to the recent version and tried?

Did you update to the recent version and tried? (See 8410c57)

Yes, I did.

I think you need to look at vendor/evenement/evenement/src/Evenement/EventEmitterTrait.php after install and wrap the call_user_func's in function emit around a try/catch to see if that does what you want.

I think you need to look at vendor/evenement/evenement/src/Evenement/EventEmitterTrait.php after install and wrap the call_user_func's in function emit around a try/catch to see if that does what you want.
Consider this a workaround for testing. I have an idea...

Thanks, but your PR is still not working.

well, I won't mess in external libraries itself, this is as close as I can get to it

well, I won't mess in external libraries itself, this is as close as I can get to it

Does anybody know why this happens?

It's an 'issue' with ReactPHP promises (issue in quotations because technically it's not really an issue) where any exceptions that are not handled are swallowed. In promises v2.0 any unhandled exceptions will be thrown like normal exceptions, but we will have to wait a while for that.
For now, I've fixed @key2peace's PR so it should emit the exception and log it in the console: 1ad7758c050a733511cb023a4780647b7c255ee1
Could you try update with composer update?

It's an 'issue' with ReactPHP promises (issue in quotations because technically it's not really an issue) where any exceptions that are not handled are swallowed. In promises v2.0 any unhandled exceptions will be thrown like normal exceptions, but we will have to wait a while for that.
For now, I've fixed @key2peace's PR so it should emit the exception and log it in the console: 1ad7758
Could you try update with composer update?

@davidcole1340 I did update, and it still doesn't work.

After b063fe9
call_user_func_array() expects parameter 1 to be a valid callback
, class 'Discord\Parts\User\Client' does not have a method 'parentEmit' in
 /home/bot/vendor/team-reflex/discord-php/src/Discord/Discord.php on line
1459

Should be changed on the master branch. I will push a new version soon. Exceptions are now thrown like normal exceptions and will cause the script to exit, so make sure you handle them.

b063fe9 fixes it, thank you.

Reopening because the errors are now displaying in events and timers but not in ExtendedPromiseInterface->done(), etc.


about demo results

First thank you for sharing the nice work. I have some questions.
Is the middle one depth image? How can it be separated with the colorful background in the code?

First thank you for sharing the nice work. I have some questions.
Is the middle one depth image? How can it be separated with the colorful background in the code?

Sorry for bothering you. Have you met the error that "LSQLIN requires an initial point for the 'active-set' algorithm."? I downloaded the repo but when I ran the demo.m, I met this error. Since the issues in this repo have not been answered from 2017, the only thing I can do is finding someone who ran this repo successfully. I would appreciate it if you have met this error before and willing to tell me the way you fix it. Thanks. Sorry for bothering you again.

Hello, I didn't meet the error. Perhaps it is the problem of software version.


How to close the autocomplete dropdown?

I have a scenario, where the user can input something in the textbox and he can:

choose something from the dropdown (works ok)
not choose anything and type his own custom text (not sure how to close the dropdown with suggestions without clicking on any of the suggestions)

I can easily write a PR to solve this, by adding a onEditingEnd prop on TextInput and setting the state.listViewDisplayed to false. Does this sound okay, should I make a PR? I can also do that all this is enabled when setting a prop to the GooglePlacesAutocomplete, to keep backwards compatibility.

Hello, as you wish since there is still backward compatibility

Here is a PR: https://github.com/FaridSafi/react-native-google-places-autocomplete/pull/19
I have tested it and works nice, you can try it too before merging.

Looks like here is an issue with selecting a  value, sorry about that. Will try to fix it.

Seems it is not so easy to fix. With my PR, when clicking on any suggestion, the onEndEditing is executed, which first hides the suggestions, and then it is not selected. Do you have any suggestions how to fix this? I tried a few approaches, but no luck yet.

Hi @tomazahlin !
Using onEndEditing event handler doesn't quite fit for this purpose. It'll fire in both cases: when you leave text input and select an item from dropdown list.
Currently if you type random text without selecting any suggested places, it'll remain unchanged.
If you need to hide the dropdown, you'll need to do it manually (button press, etc.).

@myaskevich How can we do it with a button press? Note that I have installed the package. I went through the source code and I see that _onBlur is the only way through which we can hide the listView. Can you let me know whether there is any option available for us to hide the list by maybe passing a state to the component?

Great news everyone!
I found some time to look at the problem and has eventually put together a simple solution: listViewDisplayed prop.
Available in v1.2.4

@myaskevich @FaridSafi I have created a variable called shouldDisplayListView and set listViewDisplayed to that as shown in code below:
let shouldDisplayListView = false;
<GooglePlacesAutocomplete listViewDisplayed={shouldDisplayListView}
    textInputProps={{
        onFocus: () => shouldDisplayListView = true;
        onBlur: () => shouldDisplayListView = false;
    }} ............ />

But still I cannot see the listView. Instead if I set listViewDisplayed to "auto" then I can see the listView. But then onBlur of textInput I am unable to hide it.

https://github.com/FaridSafi/react-native-google-places-autocomplete/issues/293

listViewDisplayed={shouldDisplayListView} this solution is not working for me.
Can anyone suggest any solution?
In my emulator when I tap on location from the suggestion list then the list is closed. But in my phone after clicking on location from suggestion list the list is opened as well not closed. What is the problem I can't understand, please suggest anyone any solution.
I am using :
 react-native : 0.54.4    "react-native-google-places-autocomplete": "^1.3.6",

listViewDisplayed={shouldDisplayListView} this solution is not working for me.

same here in my case also
any other solution..??

I also found that listViewDisplay={false} does not cause the list view to hide.  But you can call one of the methods if you set a ref
ref.current.blur()
It seems to hide the list by blurring, as expected but it seems to be very hard to actually blur the input field without clicking a result from the list.  So manually blurring worked for me


Programs metadata

At the moment of writing this, each program contains a set of metadata in the "Readme" tab.

As a user, I would like to see the following additional metadata

Contact information if available (Discord, Twitter or Email)
Github repository link
Open PRs in Github (similar to NPM)
Open Issues in Github (similar to NPM)

Is the API part for this done?


component becomes uneditable after making a css change

STR

New project
Click into <App> and select the div
Change its opacity via the inspector
Change its opacity to an equation via the code editor (e.g. opacity: .5 + .2 )

Observed: The div becomes deselected, you can no longer enter / edit the component itself
Expected: remains selected, remains editable
Video:
https://user-images.githubusercontent.com/2945037/119543337-7620e000-bd88-11eb-8f00-189c419e8887.mp4
Redux devtools json export: https://www.dropbox.com/s/ha1311jbjqizj9p/element-deselected-uneditable-after-css-change.json?dl=1

@maltenuhn I think this bug has the video for #1299.

@seanparsons Oops, so it does. Here's the one I'd meant to upload!
https://user-images.githubusercontent.com/2945037/119812205-d0808480-bedf-11eb-819d-31a0b5dae54e.mp4

(However, I'm unable to reproduce this right now on pizza - not sure if it's fixed by something merged yesterday, or I hit a specific failure mode not in the repro steps)


[Question]: How to use ports with alarm isolates?

What is your question?
I want to communicate with the isolates fired by the alarm manager plugin from my main isolate. The only way to do so in dart is to used SendPort and ReceivePort. How do I use ports with this plugin, or is it even possible?
Checklist before submitting a question

[X] I Google'd a solution and I couldn't find it
[X] I searched on StackOverflow for a solution and I couldn't find it
[X] I read the README.md file of the plugin
[X] I am using the latest version of the plugin
[X] All dependencies are up to date with flutter pub upgrade
[X] I did a flutter clean
[X] I tried running the example project

So I finally found it.
In your alarm callback isolate, do the following to receive messages:
String portName = "portName";

@pragma('vm:entry-point')
void alarmCallback(int id, Map<String, dynamic> params) async {
  ReceivePort receivePort = ReceivePort();
  IsolateNameServer.removePortNameMapping(portName );
  IsolateNameServer.registerPortWithName(receivePort.sendPort, portName );
  receivePort.listen((message) {
     // Do stuff
  });
}

And then in your main isolate:
SendPort? sendPort = IsolateNameServer.lookupPortByName(portName );
sendPort?.send("data");

And vice versa if you want to send from alarm isolate to main isolate.


toStars(addr) validate coin type

coin type 118 is compatible with keplr
https://medium.com/chainapsis/keplr-explained-coin-type-118-9781d26b2c4e

make a strict whitelist of coin type 118. test and add new coins manually


(HTTP code 500) server error - {"message":"Get https://registry-1.docker.io/v2/: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)"}

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

Please make sure that you're on a stable connection and use a solid DNS such as Google DNS.


Added tokenization, serialization methods for AlchemicalNetwork, Transformation

This includes some changes already proposed in #42, but just limits scope to this functionality.

Yeah, sorry for delay, this is fine. I've been working on tomorrow's (today's) talk for a while. I'm ignoring the codecov complaints on the basis that they should be fixed in #42.

No worries, thanks @dwhswenson!


Update reactor-grpc-stub to 1.2.0

Updates com.salesforce.servicelibs:reactor-grpc-stub from 1.0.1 to 1.2.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (1.0.1).
You might want to review and update them manually.
code-gen/src/main/scala/scalapb/reactor/ReactorCodeGenerator.scala
examples/build.gradle

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.salesforce.servicelibs", artifactId = "reactor-grpc-stub" } ]

labels: library-update, semver-minor, old-version-remains

Superseded by #54.


Update AVDecVideoAcceleration_H264 to call out MF behavior

Add a clarifying statement to AVDecVideoAcceleration_H264 CodecAPI to describe MF behavior.  The current document is misleading in that it only applies when using DShow transforms.  But CodecAPIs are also available for MF transforms.

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

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


add pause to post so it's not quite so fast

at the moment the post is instantaneous - should we slow it down?

could it even be that we wait 5 or 10 minutes and don't post if a human has posted before us doing the welcome?


refactor(reactivity): rename activeEffectScope to currentEffectScope

A readability improvement.
get active() can today cause confusion in that one would think it means this === activeEffectScope.
This change aligns it with the naming of getCurrentScope.

Thank you for your contribution. However, this refactor may not be worth it. While the renaming might improve readability slightly, the existing naming convention is already clear and well-understood. Additionally, the changes do not provide significant improvements in functionality or performance.


Remove the extra option max steps

The added option of max_steps was making the script fail.

This has already been resolved.


customerRecover, customerReset and invalid type ID!

I'm currently implementing a password reset flow, and ran into an issue with the customerReset mutation. Firing customerRecover sends an email notification, as expected. I modified the email notification template to point to my server instead, for example http://localhost:3000/reset-password/6655363669/7c4a5966d0901b57ee96af96eddb0eb5-1503254727.
The problem is that customerReset takes a customer ID, which should be<PHONE_NUMBER> here. But from what I can tell, that's a Shopify Customer ID, but not necessarily a Shopify Storefront API Customer ID? The exact error I get is "GraphQL error: Variable id of type ID! was provided invalid value". The docs show an ID that looks like Z2lkOi8vU2hvcGlmeS9FeGFtcGxlLzE=, and not<PHONE_NUMBER>. Do I need to modify the email notification template? If so, what Liquid template variable(s) do I use to create a correct password reset URL?
An example that includes password recovery would be great, but it also just looks like that mutation is expecting the wrong arguments.

Yeah the Storefront API expects any field/input with the type of ID to be in our "Storefront" format (an encoded ID). So an integer based ID direct from Shopify admin/links don't work.
This is an interesting use case that we hadn't thought about yet or ran into. Unfortunately there's no Liquid variable/filter exposed which can do this.
We'll need to think about this 🤔

@swalkinshaw, thank you for the quick reply.
Is there any workaround for this? The only thing I can think of is setting a temporary password for the user via the Admin API, and then forcing them to set a new password on authentication. Is there any way to derive the Storefront ID from the integer based one?
We're relying on the Shopify Storefront API extensively, and would love to figure this one out as soon as possible.

The quick workaround is to build the encoded ID yourself:
gid://shopify/Customer/6655363669 is the format. Then you Base64 encode it.
Warning: do not rely on this long term. These IDs are encoded for a reason because they can change. This is just a quick fix for this.
We'll have to figure out a proper solution to this.

@nickdandakis can you provide more details on your custom template?
How are you generating that reset-password URL? Is that directly from a Liquid variable? Or are you using a combination of others to build it?

So, I looked for a Liquid variable for just the reset password token and couldn't find one. customer.reset_password_url is used in the 'Customer account password reset' email notification template, however that's undocumented as well.
I ended up with this hack:
{% assign reset_password_url = customer.reset_password_url | replace: "https://token-local.myshopify.com/account/reset", "http://localhost:3000/reset-password" %}

Which is also fine for right now, but obviously don't want to rely on this either. Here's a Shopify forum post I found that's related to this.
What I'd like to have is access to Liquid variables for the Customer Storefront API ID, and the Customer Reset Password Token.
I'm assuming the customerActivate mutation will run into the same issue too. So might be best to also have a Liquid variable for the Customer Activation Token!

Well that's terrible 😄 .
Yeah our first idea is to add a liquid variable for the token and then update our mutation to directly take that instead of the customer ID. We likely don't need it since the token can identify the customer anyway.
We'll either need to create a brand new mutation like customerResetByToken or somehow migrate the existing in a non-breaking way.

The liquid variable would tremendously help, for sure.
Would making the ID an optional parameter in the existing customerReset mutation be the fix non-breaking fix? I'm honestly surprised that all this happened, and wonder if anyone is actually using this mutation.

I'm honestly surprised that all this happened, and wonder if anyone is actually using this mutation.

We've seen about 6 queries with it in the past week and 4 were yours. There's only one way to use it properly right now and that's via a native app which can register URL handlers to open the reset link in a native app.
We can make it optional but we need to verify usage has migrated over before removing it completely.

i'm doing something similar using the Storefront API to create our own reset password page; however, when i go to submit, i'm receiving an "Invalid reset token" error. i'm using the workaround mentioned above to properly encode the customer ID for the request, but maybe i'm missing something with the reset token? the reset token i'm using is just taken from the end of the original reset password url (.../account/reset/<user-id>/<reset-token>).

@yekimmi are you referencing the  customerReset mutation?
It takes an id argument which is the "encoded" customer ID, and then a resetToken within input. The resetToken is just the string from that URL though. It doesn't need to be encoded (only fields with the ID type do).

yup, that’s exactly what i’m doing and getting the invalid reset token error. the user-id from the path i posted i’m encoding and using as the id and the reset-token from the path i’m using as-is as the resetToken in the input.

cc @Shopify/storefront-api we'll have to verify behaviour of this mutation

Hi @swalkinshaw, just wondering if there's been any development on this - is building the encoded ID manually is still the required workaround?

The workaround is still required but it's been added to our roadmap recently.

Thanks for the update, @swalkinshaw!

Hey @swalkinshaw
Has there been any progress on this? I'm running into the same issue in my app as well.

We added a new customerResetByUrl mutation about a year ago to fix this issue. It takes the reset password url to make it easier on clients and get around this ID issue.
I guess this issue should have been closed so I'll do that now 😄

@swalkinshaw Thank you! Quick question if you have time.
Where do I find the resetUrl for the customerResetByUrl mutation? Am I simply passing in the URL of my custom storefront page? Then this gets set to the "reset password link" emailed to the user?
I guess I'm confused because the description for the above mutation says the URL comes from CustomerRecover but it looks like the latter only returns customerUserErrors?
I'm probably missing something obvious here :)

Ok I finally figured out that customerResetByUrl  takes the default Shopify password reset URL. This is the same URL set within the password reset email template.
Is customerResetByUrl  intended to be used programmatically? I'm trying to build the password reset flow on my own site without the user going to Shopify. So ideally, I'm trying to force the password reset link to point to my own server without modifying the email template.
But it seems I have no way of fetching the resetUrl via the API. The only place I see resetUrl is within the password reset email.
Any direction would be much appreciated!

Unfortunately, it can't be used programmatically at this time. We already have this concern on our radar and will update on this ticket once we've addressed it.

Thanks for the update!

So if I am following correctly, the only way to do a full password reset flow with storefront api is to:

Call customerRecover mutation

And either:

Pass the default Shopify password reset url generated in the email, in to the link to your app (via query params?)
Use the default Shopify url in customerResetByUrl

Or

build the encoded ID and base64 Encode it and then use that in customerReset

Is that correct? And customerResetByUrl is the preferred method?

Is that correct? And customerResetByUrl is the preferred method?

Yes it is. There's a similar question on our forums with more details: https://community.shopify.com/c/Shopify-APIs-and-SDKs/Storefront-API-Help-using-customerResetByUrl-mutations/td-p/556812


Error with already changed handle

Hi,
thanks for the great tool, set it up to run automatically.
First run had just one issue: https://twitter.com/Jaykul
He has<EMAIL_ADDRESS>in his bio. Visiting this handle in mastodon manually the mastodon bio says "My New Account is<EMAIL_ADDRESS>

ive had a couple of people have that too and ill have to look at the data to see if i can get forwards. if not, i will reach out to him, there must be a way to get this in the API, ill check

Thanks. I think the problem here is that the user has moved between mastodon instances, just as by design, but kept his first account in his twitter bio. This may become a larger problem over time when a user does not use twitter any more and has fully set to mastodon.


It looks like this branch to .got_elias could result in returning without cleaning up the saved registers:
...
.got_bit:      bcs.s .got_elias     ; done if control bit is 1
.elias_bt:     add.b d1,d1          ; read data bit
               addx.l d0,d0         ; shift data bit into value in d0
               bra.s .elias_loop    ; keep reading

.done:         movem.l (sp)+,a2/d2  ; restore preserved registers
.got_elias:    rts

Code comprehension error on my part. 8(


wired postgres through to oauth, set cookies for sessions with cookie…

…-session, and working smoothely
Description
Wired postgres to Oauth to reg and auth.
Fixes # (issue)
Added database functionality to Oauth to register auth, and set cookies for users.
Type of change
passport.js in the passport.use new GoogleStrategy callback, the db functionality was added.   Also passport.serializeUser and passport.deserializeUser.  Also cookie session was installed and utilized.  In server.js I set middleware of passport.initialize, and passport.session, as well as the cookie-session.
Please delete options that are not relevant.

[x ] Bug fix (non-breaking change which fixes an issue)
bugs arose with the oauth flow, and were troubleshooted.  They may arise again as code merges, so we will need to keep version control, and watch closely as we integrate.
[x ] New feature (non-breaking change which adds functionality)
Postgresql utilized to register and log in users.  Log out route added as well.

Change Status

[x ] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

How Has This Been Tested?

[x ] Test A
First make sure the keys are all properly configured.  Then go to '/auth/google', then click to log in , then go to '/api/users' which is a protected route.  Then go to '/api/logout'.  Check your cookie in your devtools.  If you are having trouble setting up locally, let me know, there may be a configuration piece missing because we gitignored the keys needed to run this.
[ ] Test B

Checklist

[x ] My code follows the style guidelines of this project
[ x] I have performed a self-review of my own code
[ x] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] There are no merge conflicts

Need to fix merge conflicts. No way to update PR without closing.


Ensure all GWAS Catalog accession IDs in identifiers.org

All GWAS Catalog accession IDs should be resolvable in identifiers.org
Currently published GCSTs are e.g. https://identifiers.org/resolve?query=GCST:GCST005179 but "unpublished" are not e.g. GCST90000032.

done


Chatdown: ConversationUpdate activity does not provide the recipient information

Describe the bug
The conversation activity does not provide the recipient information. This seems due to a bug in https://github.com/Microsoft/botbuilder-tools/blob/master/Chatdown/lib/index.js#L176 where the recipient information is tried to be retrieved using botId as key.
To Reproduce
Steps to reproduce the behavior:

Generate a transcript out of https://github.com/Microsoft/botbuilder-tools/blob/master/Chatdown/Examples/ConversationUpdate.chat
Check that the `conversationUpdate  activity in the transcript doesn't have the recipient information

Expected behavior
The conversationUpdate activity should have the recipient information
Screenshots
The screenshot below shows that the conversationUpdate activity in the transcript does not have the recipient information.

BTW, the from property is also missing

fixed in chatdown 1.0.10


chore: remove unfunctional property 'oauthID'

This removes an apparent unused schema property from both cloud and oss.

We're removing instead of deprecating?

Whichever you prefer, it was a 5 minute change that felt like we might as well remove it since it's incorrect (rather than do roughly the same amount of work now to mark it as deprecated, just to come along later to remove it :shrug: ).


0.5.0 crash on startup

latest.log
just updated bclib from bclib-0.4.4.jar till 0.5.0 and got a crash on startup

Issue is on BetterNether side

Will there be a fix for it? @paulevsGitch

its already fixed. Update BetterNether and Bclib


Workbox Broadcast Update in Safari

Is it possible to use Broadcast Cache Update plugin in Safari? If not, is there anyway that safari be able to detect an server update and get newest content?

Yes, workbox-broadcast-update can be used in Safari. It will use postMessage(), which Safari supports.


Create Signable interface

Closes #20

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@bf76c43). Click here to learn what that means.
The diff coverage is 68.88%.

@@            Coverage Diff            @@
##             master      #32   +/-   ##
=========================================
  Coverage          ?   74.28%
=========================================
  Files             ?       23
  Lines             ?     1042
  Branches          ?        0
=========================================
  Hits              ?      774
  Misses            ?      226
  Partials          ?       42

Impacted Files
Coverage Δ

api/apihttp/protocol.go
62.5% <ø> (ø)

tests/e2e/setup.go
92.3% <100%> (ø)

client/client.go
61.9% <58.82%> (ø)

sign/sign.go
67.27% <67.27%> (ø)

api/apihttp/apihttp.go
40.65% <70%> (ø)

Continue to review full report at Codecov.

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

We didn't rebase, so we deteleted the merge commit and pushed manually e9bad4c


Eu registrations feature

Thank you for submitting a pull request on this HMRC repository. A member of the HMRC team will review your changes and action accordingly.
HMRC reviewer: please reference this runbook for guidance on how to proceed.

test this please

Squashed, see https://github.com/hmrc/one-stop-shop-registration-frontend/pull/201


PLEASE add move in direction Camera is facing!

Hi all! Can someone add a quick hack to this to allow character to move in the direction the camera is facing using a cinemachine camera?
Much MUCH thanks, Love & appreciation!

Someone?


Subject id not inserting while update

Subject Id not inserting while updating data
activity()->causedBy(Auth::user()->id)
        ->performedOn(New User)
        ->withProperties(['attributes' => $request->all()')])
        ->log('User Edited');

Hey,
a new User does not have any ID - so there is no way to log it as subject. If you want to log model changes you should also use the package provided log model events trait: https://docs.spatie.be/laravel-activitylog/v3/advanced-usage/logging-model-events/
Your code snippet could be changed to:
activity()
    ->causedBy($request->user())
    ->performedOn($user)
    ->withProperties(['attributes' => $request->all()])
    ->log('User Edited');

Hey,
a new User does not have any ID - so there is no way to log it as subject. If you want to log model changes you should also use the package provided log model events trait: https://docs.spatie.be/laravel-activitylog/v3/advanced-usage/logging-model-events/
Your code snippet could be changed to:
activity()
    ->causedBy($request->user())
    ->performedOn($user)
    ->withProperties(['attributes' => $request->all()])
    ->log('User Edited');

->performedOn($user) what is $user here

It's the user or any other model you've edited, created or done any other action on.


Removed unused variable raising warnings

Fixes #????

PR Checklist

[x] Tests
[ ] Documentation
[ ] Changelog entry

I have tested and recompiled with solidity 0.8.0 using Hardhat
Compiled Without warning as expected.

Hello @Madeindreams.
The contracts in this repository are not hand-written. They are automatically transpiled from the @openzeppelin/contracts package. This "unused variable" issue is a (small) side effect from the transpiler.
A commit that changes that, such as yours, will be automatically overwritten next time the transpilling workflow is executed. This should be solved at a higher level, directly in the transpiler ... which really is not that easy to do.


CUDA-Team: reduce source build time and binary size significantly

Reduce source build time and binary size significantly
This issue tracks the feature-level work done to allow users to more quickly rebuild or download cached binaries.

[ ] https://github.com/NixOS/nixpkgs/issues/232501 to reduce closure sizes.
[ ] https://github.com/NixOS/nixpkgs/issues/224533 to reduce closure sizes.
[ ] #237761

Additional supporting infrastructure which would be nice:

track statistics about ELFs (dt_needed, run paths, etc.)
track closure size
track build time

@SomeoneSerge I believe you had mentioned possibly getting some of this information from a Hercules CI instance?

@SomeoneSerge you mentioned that there might already be an issue to track that discussion -- how does it fit in with the broader "which capabilities do we provide by default, and in what way"? As in, multi-capability binaries vs. single-capability binaries.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/cuda-team-roadmap-and-call-for-sponsors/29495/1

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/cuda-team-roadmap-update-2023-08-29/32379/1

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/community-calendar/18589/103


Add tabbed dialog

Add a tabbed dialog widget in order to create a dialog with multiple tabs. This is a first go around, needs optimization for sure, but it's there.

Codecov Report
Attention: Patch coverage is 97.75281% with 2 lines in your changes missing coverage. Please review.

Project coverage is 97.59%. Comparing base (e357184) to head (090191a).

Files with missing lines
Patch %
Lines

src/qtantic/tabbed_dialog_tools.py
97.67%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #9      +/-   ##
==========================================
+ Coverage   97.54%   97.59%   +0.04%
==========================================
  Files           3        4       +1
  Lines         204      291      +87
==========================================
+ Hits          199      284      +85
- Misses          5        7       +2

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


Changes packaging of mrfutils

Following the official packaging guide, I made these changes to make mrfutils packaging a bit smoother.
Previously, installs using pip install -e worked "correctly," but they had the negative side effect of adding things to the global package space other than mrfutils. Specifically, if you pip install -e . with the old package structure, you can now import helpers, which definitely should be a side effect of installing mrfutils. Additionally, pip installs without the --editable flag simply didn't work at all (feel free to try, I recorded a whole shell session to share but decided it was long and boring).
With this new package structure, both editable and non-editable installations work correctly and there are no additional packages other than mrfutils added at the global level. As a result, imports needed to change, e.g., from helpers import X changes to from mrfutils.helpers import X.
I went ahead and kept the things exposed by mrfutils itself the same, but adding from converters import * to __init__.py, where converters.py is the new name for what was previously called mrfutils.py (renamed simply because converters is a better name for the module than mrfutils, which is the name of the package as a whole).

Reviewing, thanks.

Edited the initial commend, it should definitely not be a side effect of installing mrfutils that a new global module helpers exists, previously I had left out the key word "not" there. Oops.


Azure.Identity simplify exception messaging

issue#7598
This update simplifies error messages of azure-identity. Below are some examples of how error messages are changed.
DefaultAzureCredential when no credentials are available
Without this update
// error message
(node:26364) UnhandledPromiseRejectionWarning: AggregateAuthenticationError: Authentication failed to complete due to the following errors:

AuthenticationError: An error was returned while authenticating to Azure Active Directory (status code 400).

More details:
{
  "error": "missing_environment_variables",
  "errorDescription": "EnvironmentCredential cannot return a token because one or more of the following environment variables is missing:\n\nAZURE_CLIENT_ID\n\nTo authenticate with a service principal AZURE_TENANT_ID, AZURE_CLIENT_ID, and either AZURE_CLIENT_SECRET or AZURE_CLIENT_CERTIFICATE_PATH must be set.  To authenticate with a user account AZURE_TENANT_ID, AZURE_USERNAME, and AZURE_PASSWORD must be set.
  ......
}

With this update
// error message
(node:10960) UnhandledPromiseRejectionWarning: AggregateAuthenticationError: DefaultAzureCredential failed to retrieve a token from the included credentials:

AuthenticationError: EnvironmentCredential is unavailable. Environment variables are not fully configured.(status code 400).
......

AuthenticationError: ManagedIdentityCredential is unavailable. No managed identity endpoint found.(status code 400).
......

DefaultAzureCredential when authentication fails
Without this update
// error message
(node:12952) UnhandledPromiseRejectionWarning: AggregateAuthenticationError: Authentication failed to complete due to the following errors:

AuthencationError: An error was returned while authenticating to Azure Active Directory (status code 400).

More details:
{
   "error": "unauthorized_client",
   "errorDescription": "AADSTS700016: Application with identifier 'c0c7e041-4e82-4920-8bde-54977f103f57' was not found in the directory '72f988bf-86f1-41af-91ab-2d7cd011db47'. This can happen if the application has not been installed by the administrator of the tenant or consented to by any user in the tenant. You may have sent your authentication request to the wrong tenant.\r\nTrace ID: 8a190ce8-3fb5-412b-aa28-05437e8a0c00\r\nCorrelation ID: 9635bff1-86c5-409b-a3a4-1f58a7a87845\r\nTimestamp: 2020-04-01 09:35:25Z",
   "Correlation ID": "9635bff1-86c5-409b-a3a4-1f58a7a87845",
   "errorCodes": [
      700016
   ],
   "Timestamp": "2020-04-01 09:35:25Z",
   "traceId": "8a190ce8-3fb5-412b-aa28-05437e8a0c00"
}

With this update
(node:8852) UnhandledPromiseRejectionWarning: AggregateAuthenticationError: DefaultAzureCredential authentication failed.:

AuthenticationError: EnvironmentCredential authentication failed.(status code 400).
More details:
unauthorized_client

AADSTS700016: Application with identifier 'c0c7e041-4e82-4920-8bde-54977f103f57' was not found in the directory '72f988bf-86f1-41af-91ab-2d7cd011db47'. This can happen if the application has not been installed by the administrator of the tenant or consented to by any user in the tenant. You may have sent your authentication request to the wrong tenant.
Trace ID: 8a190ce8-3fb5-412b-aa28-05437e8a0c00
Correlation ID: 9635bff1-86c5-409b-a3a4-1f58a7a87845
Timestamp: 2020-04-01 09:35:25Z
......

I think removing what environment variables are missing makes this error message worse. My goal when working on error messages is trying to understand how quickly the user can fix the issue once they read the error message. Sometimes this means that the message may be a little longer, but if the user can solve the issue faster, then that's the preferred error message.

/azp run js - identity - ci


Terms of use bolding in English

Discrepancy on https://staging.covid-hcportal.cdssandbox.xyz/en/terms/
Not the same as Figma screens: https://www.figma.com/file/znPqLTQQDcUkaNjdDcShiz/Exposure-Notification-Health-Portal?node-id=7814%3A2

In Figma:
and is bolded
In staging:
and is not bolded

Pull Request for this issue has been created - https://github.com/cds-snc/covid-alert-portal/pull/733

Merged in stage and production.


raven.utils.init.get_versions extremely slow (on aws)

we ran into a situation that this call takes 12-15 seconds in a Django app on first access of a log function after startup.
this leads to the situation that first hits to log.error or log.warn or anything that has the sentry handler configured are very very slow even if the default http handler is async.
I didnt dive any deeper, and adding  'include_versions': False  in the RAVEN_CONFIG, disabled it, but it might be wise not to have this disabled by default or add a warning somewhere or initilaize this in the async part of the request logging if possible, might make it a bit better.

I believe I'm experiencing the same issue. I have a large Django app with 10s of apps. I ran line profile on get_versions  with my INSTALLED_APPS as the argument and I found this (get_version_from_app was:
where all the time was being taken):
Total time: 32.966 s
File: /home/pete/.venvs/project/local/lib/python2.7/site-packages/raven/utils/__init__.py
Function: get_version_from_app at line 61

Line #      Hits         Time  Per Hit   % Time  Line Contents
==============================================================
    61                                           def get_version_from_app(module_name, app):
    62       142          137      1.0      0.0      version = None
    63
    64                                               # Try to pull version from pkg_resource first
    65                                               # as it is able to detect version tagged with egg_info -b
    66       142          111      0.8      0.0      if pkg_resources is not None:
    67                                                   # pull version from pkg_resources if distro exists
    68       142           63      0.4      0.0          try:
    69<PHONE_NUMBER>4 232140.5    100.0              return pkg_resources.get_distribution(module_name).version
    70       136          284      2.1      0.0          except Exception:
    71       136           72      0.5      0.0              pass
    72
    73       136          310      2.3      0.0      if hasattr(app, 'get_version'):
    74         1            0      0.0      0.0          version = app.get_version
    75       135          146      1.1      0.0      elif hasattr(app, '__version__'):
    76         5            4      0.8      0.0          version = app.__version__
    77       130          154      1.2      0.0      elif hasattr(app, 'VERSION'):
    78         1            0      0.0      0.0          version = app.VERSION
    79       129          115      0.9      0.0      elif hasattr(app, 'version'):
    80                                                   version = app.version
    81
    82       136          129      0.9      0.0      if callable(version):
    83         1           10     10.0      0.0          version = version()
    84
    85       136          278      2.0      0.0      if not isinstance(version, (string_types, list, tuple)):
    86       129           65      0.5      0.0          version = None
    87
    88       136           68      0.5      0.0      if version is None:
    89       129           48      0.4      0.0          return None
    90
    91         7           10      1.4      0.0      if isinstance(version, (list, tuple)):
    92                                                   version = '.'.join(map(str, version))
    93
    94         7            8      1.1      0.0      return str(version)

I haven't confirmed this is the cause of my problem in production but I believe it is at the moment. From a quick Google it seems that pkg_resources.get_distribution is known to be slow.

Not really an solution to the problem, but if you are okay with dropping versions from your sentry logs, adding 'include_versions': False to RAVEN_CONFIG seems to do the trick for me on Django.

Affects us as well, 'include_versions': False makes startup time go from 28 to 5 seconds.

I haven't profiled this myself, but reading through the comments and code in get_version_from_app, it seems like pkg_resources.get_distributions is the slow part and it's getting called on everything in INSTALLED_APPS. get_versions_from_app seems to have a lot of shortcuts to avoid the pkg_resources.get_distributions but they get called only if the get_distributions can't be imported.
It seems like https://github.com/getsentry/raven-python/compare/master...AlexRiina:master might help.


Releases freestyle-core 0.4.0

freestyle-docs is failing until frees-integrations are released too. https://github.com/frees-io/freestyle/issues/380 should fix the issue next time.


Fix 357

make sure omitempty is applied for id field.
Tested already, and it works for adding user role during controller initialization.

LGTM


Pointer delay; Explorer crash; reversion to small taskbar icons

Upon first run (of SmartTaskBar 1.1.8 - not of previous versions) the following happened: my mouse pointer became very slow; then explorer.exe crashed and restarted.
Also: since I installed the new version of SmartTaskbar, the size of my taskbar icons keeps reverting from normal to small, even though I do not have the 'small icons' option ticked in the options of SmartTaskbar. (Or at least I believe that I do not. Since the unticked state is represented with no icon at all, it is somewhat hard to tell.)
Version 1.1.8 running on Windows 8.1 Pro x64.

My God, I may have written a bug here.
Please try to download and install it again once to see the effect.
If it still doesn't work, use 1.1.7 first. I will check it out.

Can you specify whether it is in "Adaptive " or "Auto" mode?

I am currently using a Windows 8.1 x64 computer, so far I can't reproduce the bug you said. Could you give me more Information, such as when these exceptions are triggered?

About the first run crash. The first installation package I uploaded yesterday did have this problem. If you download and install again, it should not crash anymore.
As for the second question, if the currently running mode is Adaptive Mode, then it is normal.

The problem of the taskbar icon disappearing when changing the button size. This is the taskbar itself refreshing, the speed of the refresh depends on the hardware of the computer.I tried to solve this problem, but no progress = =

Hello! The program showed both 'adaptive' and 'auto' as unchecked when I first ran it. I've set it to 'auto' a few hours ago and have had no problems since then (though I have not yet rebooted).

As for the second question, if the currently running mode is Adaptive Mode, then it is normal.

Aha. Perhaps the program should by default start in 'auto' mode then?

About the first run crash. The first installation package I uploaded yesterday did have this problem. If you download and install again, it should not crash anymore.

I've just tried the new installer. It seemed to work perfectly - but I'll see whether it continues to function. If it does, I'll close this issue. Thank you for your work.

(●'◡'●)


1.1.10 into 1.1.x

You did it @rakeshnpatel!
Thank you for signing the Anthem Opensource Software CLA.


Script skips everything (too old)

Not sure why this keeps happening, but every time I start the feed2discord.py, it loads up fine and connects to the bot account, and every time it reads the RSS URLs (just Reddit so far), it keeps throwing back similar line to this:

INFO:main:kpopreddit:item t3_43oas1 unseen, processing:
INFO:main:kpopreddit:too old; skipping

I have no idea why it keeps saying that all the posts are old, because one was made only minutes before on /r/kpop. The bot has all the required permissions to connect and post messages in the Discord channel too. It doesn't post anything from the feed in any of the channels and eventually it closes with:

/usr/lib/python3.5/asyncio/selector_events.py:577: ResourceWarning: unclosed transport <_SelectorSocketTransport fd=10>

If it's of anything, this is how I've got the feed in feed2discord.ini:

[kpopreddit]
channels = rkpop
feed_url = https://www.reddit.com/r/kpop/.rss
fields = link
max_age = 7200

As well as the Channels section:

[CHANNELS]
rkpop =<PHONE_NUMBER>01890816

[DEFAULT]
rss_refresh_time = 500
max_age = 86400
send_typing = 1

It does the same thing for all 3 entries I have in the CHANNELS section, which are all formatted similarly (and I figure it's correct, hopefully, with the last number in the channel's URL).
This is all running on a Ubuntu 14.04 server with Python 3.5.0 installed, for reference.
Sorry if this is long, but I'm sorta new to Github and needed some help with this. Thanks!

The "too old; skipping" means that the item (post) in the feed is older than the "max_age" for that feed. Doesn't look like any kind of permissions problem for the channel (or if there is, that's a separate issue)
When I examine that RSS feed by hand, I see that the first item in the feed has a date of March 25...  Try this feed_url instead: https://www.reddit.com/r/kpop/new/.rss

With the new feed URL, I'm still getting the same "too old; skipping" error. Really have no idea why...
With debug set to 3, these are the last lines. All previous ones say "too old" as well, even though I'm using the URL for the new feed rather than default.
DEBUG:__main__:kpopreddit:item:checking database history for this item
INFO:__main__:kpopreddit:item t3_4cqjh6 unseen, processing:
INFO:__main__:kpopreddit:too old; skipping
DEBUG:__main__:kpopreddit:sleeping for 500 seconds
WARNING:asyncio:Executing <Task pending coro=<background_check_feed() running at feed2discord.py:320> wait_for=<Future pending cb=[Task._wakeup()] created at /usr/lib/python3.5/asyncio/tasks.py:503> created at feed2discord.py:333> took 0.247 seconds

What date/time does your system think it is? (run date -u to see)

Server's time/date is Fri Apr  1 01:52:57 UTC 2016 should be fine, right?

Yes, that date/time looks about right...
Please do a "git pull", set debug to 4 and try again with the reddit feed. I've added some additional debugging that will output all the relevant dates...
That TypeError on the instagram feed is definitely a bug, but a different bug. Can you put that into a new ticket?

I think I have an error with accidentally treating an item's date as if it's in the local timezone (but without converting it)...  I'll have to look at that more carefully.

Because I can't figure out how to post the log here and because it's so long, I've uploaded the text file with log here in it's entirety. Debug was set to 4 for this. Will make a ticket for the TypeError too.

This is the bit I needed:
DEBUG:__main__:kpopreddit:item:processing this entry DEBUG:__main__:kpopreddit:item:id:t3_4czg2d DEBUG:__main__:kpopreddit:item:checking database history for this item INFO:__main__:kpopreddit:item t3_4czg2d unseen, processing: INFO:__main__:kpopreddit:too old; skipping DEBUG:__main__:kpopreddit:now:1459561548.9034908 DEBUG:__main__:kpopreddit:now:gmtime:time.struct_time(tm_year=2016, tm_mon=4, tm_mday=2, tm_hour=1, tm_min=45, tm_sec=48, tm_wday=5, tm_yday=93, tm_isdst=0) DEBUG:__main__:kpopreddit:now:localtime:time.struct_time(tm_year=2016, tm_mon=4, tm_mday=2, tm_hour=10, tm_min=45, tm_sec=48, tm_wday=5, tm_yday=93, tm_isdst=0) DEBUG:__main__:kpopreddit:pubDate:2016-04-02T01:45:32+00:00 DEBUG:__main__:kpopreddit:pubDate_parsed:time.struct_time(tm_year=2016, tm_mon=4, tm_mday=2, tm_hour=1, tm_min=45, tm_sec=32, tm_wday=5, tm_yday=93, tm_isdst=0) DEBUG:__main__:{'id': 't3_4czg2d', 'link': 'https://www.reddit.com/r/kpop/comments/4czg2d/what_song_do_you_have_on_repeat_right_now/', 'guidislink': True, 'author': '/u/YeBeAWitch', 'updated': '2016-04-02T01:45:32+00:00', 'content': [{'type': 'text/html', 'base': '', 'value': '

I'm normally into more upbeat stuff but for me it's Sunggyu's kontrol. For the last week I've been playing it first no matter what I'm listening to.

   submitted by    /u/YeBeAWitch
 [link]   [comments]', 'language': None}], 'href': 'https://www.reddit.com/user/YeBeAWitch', 'authors': [{'name': '/u/YeBeAWitch', 'href': 'https://www.reddit.com/user/YeBeAWitch'}], 'updated_parsed': time.struct_time(tm_year=2016, tm_mon=4, tm_mday=2, tm_hour=1, tm_min=45, tm_sec=32, tm_wday=5, tm_yday=93, tm_isdst=0), 'tags': [{'scheme': None, 'term': 'kpop', 'label': '/r/kpop'}], 'links': [{'rel': 'alternate', 'type': 'text/html', 'href': 'https://www.reddit.com/r/kpop/comments/4czg2d/what_song_do_you_have_on_repeat_right_now/'}], 'title': 'What song do you have on repeat right now?', 'summary': '

I'm normally into more upbeat stuff but for me it's Sunggyu's kontrol. For the last week I've been playing it first no matter what I'm listening to.

   submitted by    /u/YeBeAWitch
 [link]   [comments]', 'title_detail': {'type': 'text/plain', 'base': '', 'value': 'What song do you have on repeat right now?', 'language': None}, 'author_detail': {'name': '/u/YeBeAWitch', 'href': 'https://www.reddit.com/user/YeBeAWitch'}}

Yeah, I think this is likely a timezone bug...  Probably have to change how I do those date comparisons.
In the meantime, I think you can work around this by setting the "TZ" environment variable to "UTC" when you run the script. Like: TZ=UTC python feed2discord.py

TZ=UTC python feed2discord.py did the trick, posts fine now and I've set it up to posts 3 subreddits so far, works beautifully. Thanks!

I just pushed a change that should force it to do the equivalent on its own. If you git pull you should be able to run without manually doing the "TZ=UTC" thing anymore.

Runs fine with just python feed2discord.py now, much better.

FYI, there's some major improvements to the timezone handling. Should work on Windows or Linux, and does things even smarter than that workaround. A "git pull" to get the latest should work, or you can look at #24 to see the details.


Test all resources in  Get-ResourceByType unit tests

Closes #302
Note: This is in draft mode since I'm not sure if this is the approach we want to take.

Iterate through the ResourceType enum and use it to determine the Powershell function name
Dynamically determine test cases based on the ResourceType enum
Add new Get-ResourceByType parameters to test

@quigg-caroline can you look into a way to check to be sure the correct parameters are being called? Unlike a compiled language if you write code in PS that calls a function with the wrong parameters there is no compile time, or IDE error. I'm wondering if at the same time you're generating the test cases if another parameter to the test cases could be the parameters of the cmdlet that is being called and that it can verify the correct parameters are being called in Get-ResourceByType for each of the different resource types.
This is why several of the example tests failed at one point or another was because the wrong parameters were being called. Something like (Get-Command Confirm-AzBPAppInsights).Parameters could get you started thinking about it.
I hope that makes sense, if not hit me up and we can discuss.

And based on the failing unit tests... it looks like it works! (or something is really wrong LOL)

And based on the failing unit tests... it looks like it works! (or something is really wrong LOL)

That would be because my branch is not up to date with main :P