App is using an old version of Google Play developer API.

Godot version:
2.1.6 stable

OS/device including version:
Windows 10 64bit

Issue description:
Google Play Console is showing me this message:
"We’ve detected that your app is using an old version of the Google Play developer API. From 1 December 2019, versions 1 and 2 of this API will no longer be available. Update to version 3 before this date. Learn more"

Steps to reproduce:
Export game for andorid that uses in-app purchases.
I also use admob module from kloder-games
Minimal reproduction project:

I looked for any references to the "androidpublisher" API this refers to, but couldn't find any in either Godot or the AdMob module.
So I'm clueless about this warning. Maybe a thirdparty library downloaded by Gradle uses it?
CC @volzhs

Do you get the issue if you upload a game without the AdMob module?

I have another game exported with godot 2.1.6 that uses the same admob module, iap and google paly services from this module, and I dont get this message on that game. Also user "u/SHlNOBU" on this reddit post commented: "This also appeared on mine. I have no idea what it means tho. I only have admob and share module without iap"

I have another game exported with godot 2.1.6 that uses the same admob module, iap and google paly services from this module, and I dont get this message on that game.

That's pretty weird then :/
Is there any way to get more details about what the Google Play Console "detected"?

Send me your email I will send you an invite to add you to my google play developer account with read only access.

Related issue: #30323

Maybe something related with this file?
https://github.com/godotengine/godot/blob/2.1/platform/android/java/aidl/com/android/vending/billing/IInAppBillingService.aidl

i have no clue either.

Buil-in in-app purchases uses deprecated AIDL billing API in both 2.1 and 3.1

Any idea about how to fix that? My game not used any in-app purchases, but google console show that warning. Godot 2.1.6 without any modules.

I have an application published on the Google Play Store too, but I don't have any warning from google about it (maybe it will come later). I don't use any ad in my game (no admob, no ads at all).
About the deprecation, I think that the IAP implementation of Godot is completely fine because:

(cf: https://android-developers.googleblog.com/2019/03/changes-to-google-play-developer-api.html)
Im also using the Google Play Service API, but I've made my own module, so feel free to use it if needed:
https://github.com/binogure-studio/godotandroid

As of today, Google states that AIDL is deprecated and apps have to migrate to the Google Play Billing API (source: https://developer.android.com/google/play/billing/api)

I'm looking into working on a PR, but it will probably break the GodotPayment API. What do you think @akien-mga, should this still land in 3.2? I'd write an upgrade guide for users of the current payment module.

If this a changing requirement from Google, we have no option but to include the changes in 3.2 so that users can keep using it in production.
But if we can avoid breaking the API in 3.2, that would be better. We can break it in 4.0 to make the plugin better, but for 3.2 the priority should be that users can keep using their existing code.
Note however that 3.2.2 already breaks compat somehow by changing the plugin system, and renaming the singleton from GodotPayments to GodotPayment, so if we have to break compat further, we should probably do it in 3.2.2 and have a clear migration guide for GodotPayment users.
CC @m4gr3d

@akien-mga I worked on it yesterday and I came pretty far. I migrated away from using a callback object to using signals to handle callbacks (which is broken on current master and 3.2, but fixed with https://github.com/godotengine/godot/pull/39029). Should I open a draft PR to discuss issues?

Sounds good yeah :)

https://github.com/godotengine/godot/pull/39034

Fixed by #39034 and #39050.


Embed config is not supported in RemoteEmbeddedPlayer

Error while updating property 'fullscreen' of a view managed by: ReactYouTube
null
This YouTubePlayer has been released

Do you get any solution?


Fix issue - Sliders colors change #102

https://github.com/matyunya/smelte/issues/102

Great job, as always!


Request to be removed from Deny List

Hotspot Name
Mean Tortilla Sardine
Hotspot b58 Address
11naDsWAuKYBiKmMYbWdj6pJx8rVdezyRMwzKtqyqnLNdrUbw6M
Discord Handle
#1776
Hotspot Manufacturer
PantherX 2
Removal Reason
I bought 6 blacklisted PantherXs in the second-hand market of salted fish in China. I didn't want to buy 6 units at a time. The seller said that the price is cheap to buy them all together. In the end, each set was sold for 2000 CNY. I bought a lot of accessories and DIY. , Install 3 units first, then find a suitable place to install others or sell them. These 3 units were installed on the roof of a friend's apartment and a company, using POE power supply and transmission network, and waterproof treatment. 3 The platform has all been installed and requested to be removed from the rejection list. Thanks to the team for their efforts to maintain the healthy development of the helium network, thank you!
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information

We have taken into account this removal request in the next iteration of the denylist. It may take some time before it's processed. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


Adding boot from volume support for OpenStack.

This change adds support for boot from volume to the OpenStack CPI.  To use, add the following property to the Micro BOSH or BOSH deployment manifest:
cloud:
  plugin: openstack
  properties:
    openstack:
      boot_from_volume: true

Changes:

Add a reference to the fog Volume API which supports created volumes with an imageRef
Create a boot volume before creating the VM with the disk size specified by the flavor
Add a block device mapping to the VM on creation to mount the new volume at /dev/vda
Skip setting the "personality" property on VM creation since file injection is not supported with BFV

Does the boot from volume survive restart, If some data exists on boot volume(say some packages), will Bosh boot from the same volume in case of say VM restart or recreation?

vm restart is typically not bosh involved so thats up to openstack. vm recreation  gurantees that root and ephemeral disks are new. packages are reinstalled everytime vm is created.
if you need to store some data i would recommend placing it on /var/vcap/store which is for persistent data.
Sent from my iPhone

On May 26, 2017, at 6:41 AM, shashankmjain<EMAIL_ADDRESS>wrote:
Does the boot from volume survive restart, If some data exists on boot volume(say some packages), will Bosh boot from the same volume in case of say VM restart or recreation?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks.
Yes for persistent data we use store.
Question is about packages and then also the advantage of booting from a
volume. Openstack provides a reserve feature in case we want to get the
same volume for the rootdisk.
Thanks
Shashank
On Fri, May 26, 2017 at 8:03 PM, Dmitriy Kalinin<EMAIL_ADDRESS>wrote:

vm restart is typically not bosh involved so thats up to openstack. vm
recreation gurantees that root and ephemeral disks are new. packages are
reinstalled everytime vm is created.
if you need to store some data i would recommend placing it on
/var/vcap/store which is for persistent data.
Sent from my iPhone

On May 26, 2017, at 6:41 AM, shashankmjain<EMAIL_ADDRESS>wrote:
Does the boot from volume survive restart, If some data exists on boot
volume(say some packages), will Bosh boot from the same volume in case of
say VM restart or recreation?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cloudfoundry/bosh/pull/445#issuecomment-304298777,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADzK1lyjZhT7sJI8psHBaWgN0ust9NPDks5r9uKngaJpZM4BIyqf
.


⚠ Bofrost.at has degraded performance

In 8a62fab, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1795 ms

Resolved: Bofrost.at performance has improved in 81f3f37 after 14 minutes.


use distinguished names for issuers (not just Subject Common Name)

This does make some labels very long - thoughts on that?
This is live at https://people.mozilla.org/~dkeeler/dashboard/

(this should fix #57)

I think the long name issue is OK. We could make it a little more pretty but it's better to have this more accurate and not aggregate when it shouldn't.

LG, thanks!


[Snyk] Security upgrade sanitize-html from 1.7.1 to 2.0.0

Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

package.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Arbitrary Code Execution SNYK-JS-SANITIZEHTML-585892
  233

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Arbitrary Code Execution

test


Update README.md

Added example for inheriting default lookAndFeel. Updated smart pointers.

Sorry, seems I missed it and I cannot have a look at it since the source seems have been deleted.
Thanks anyway for your effort.


validate non-empty String baseDataSource in MaterializedViewSuperviso…

re: https://github.com/apache/incubator-druid/issues/6074
I am working on this.

@jihoonson could you please add WIP here?

@jon-wei could you please remove WIP flag? Thanks


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 960 ms

Resolved: Bofrost.be performance has improved in 73f24e4.


itkThresholdImageFilter result differs when employing streaming

Description
Results of itkThresholdImageFilter differ significantly with and without streaming for same input.
Steps to Reproduce
Tested with https://github.com/romangrothausmann/ITK-CLIs/blob/556423bf4c04c4bcd60a3bd420bb3df0c7d2fe65/thresh-erase.cxx
and
https://github.com/romangrothausmann/ITK-CLIs/blob/556423bf4c04c4bcd60a3bd420bb3df0c7d2fe65/.gitlab-ci.yml#L81-L89
Expected behavior
Independent of streaming, both version thresh-erase and thresh-erase_SDI should lead to the same result if the same input file is used.
Actual behavior
Result of thresh-erase is correct (Min: -0.499998 Max: 0.499997): https://gitlab.com/romangrothausmann/ITK-CLIs/-/jobs/177668140
Result of  thresh-erase_SDI is wrong (Min: 0 Max: 255): https://gitlab.com/romangrothausmann/ITK-CLIs/-/jobs/177668142
Reproducibility
100%
Versions
ITK master (b2884129bf6db53c) (https://github.com/romangrothausmann/ITK-CLIs/blob/556423bf4c04c4bcd60a3bd420bb3df0c7d2fe65/Dockerfile#L23)
Environment
ubuntu:16.04: https://github.com/romangrothausmann/ITK-CLIs/blob/556423bf4c04c4bcd60a3bd420bb3df0c7d2fe65/Dockerfile

In your CMakeLists.txt, you use thresh-glob.cxx as the source for the thresh-erase_SDI executable, which I imagine is a copy-paste mistake. Since it uses a BinaryThresholdImageFilter it explains why the output is binary.

Many thanks @maekclena for taking this close look, it was a copy-paste mistake, output as expected now:
https://gitlab.com/romangrothausmann/ITK-CLIs/-/jobs/180281298


Replicate BabyWalk for CVDN

I noticed your two submissions on the CVDN leaderboard. Could you please point me to the code and instructions to replicate those results?
Also, I'm a bit confused because your paper is on VLN, but I do not see any BabyWalk submissions on the VLN leaderboard itself. Could you please explain?

Dear Karthik,
Thanks for your interest in our work. We are running on experiments to prepare a Journal submission for our work, which includes new evaluation tasks, such as CVDN. The BabyWalk entry on the leaderboard you see is a straightforward application of this codebase, but on the CVDN dataset.
Unfortunately, before the acceptance of the our work, we would not be able to release the code with respect to the CVDN part. Hope you would understand this.
Best,
Bill

@Bill1235813 sure, no worries, although I personally think it should be fine to release the code if it is only a straightforward application of the ACL paper with no novel, CVDN-specific additions.
Can you also please share why there is no entry for BabyWalk on the VLN leaderboard? I got confused because the BabyWalk paper is written for VLN, and it was surprising to see no leaderboard entry for VLN, and instead for CVDN.
Thanks!

Hi @g-karthik ,
Our main idea of the ACL paper is to help VLN agent to improve the generalization over different task horizons. Therefore, our focus is the Room-4-Room (and Room-6-Room, Room-8-Room), which do not have a leaderboard to upload the result.

@Bill1235813 From what I see in your paper, you trained on R2R also, and measured out-of-domain generalization from R4R --> R2R.
So I'm not sure I understand your explanation.
Also, I assume you are training with R4R for your CVDN submission as well. But that didn't stop you from submitting to the CVDN leaderboard :)

Hi @g-karthik,
“@Bill1235813 From what I see in your paper, you trained on R2R also, and measured out-of-domain generalization from R4R --> R2R.”
Transferring learning from R4R to R2R is obviously going to be inferior to R2R --> R2R without adaptation. The result is not going to be as competitive to other models so there is no point of uploading to leaderboard. Please see Table 5 in the paper.
"Also, I assume you are training with R4R for your CVDN submission as well. But that didn't stop you from submitting to the CVDN leaderboard :)"
As already mentioned, the results on CVDN is based on a extension of the original paper. It is not trained on R4R and transferred to CVDN, but directly training on the CVDN dataset.

Transferring learning from R4R to R2R is obviously going to be inferior to R2R --> R2R without adaptation. The result is not going to be as competitive to other models so there is no point of uploading to leaderboard.

In my previous message, when I said "I assume you are training with R4R for your CVDN submission as well", I meant for your new journal paper submission for a similar analysis as Table 5, not the leaderboard submission.
The actual VLN leaderboard submission could still be BabyWalk R2R --> R2R. My question is, why was that model not submitted to the VLN leaderboard (i.e., the 3rd row, with SR=43.8, CLS=54.4, etc.)? Does it mean the BabyWalk model is not SOTA for VLN on the leaderboard metrics?

Hi @g-karthik:

The actual VLN leaderboard submission could still be BabyWalk R2R --> R2R. My question is, why was that model not submitted to the VLN leaderboard (i.e., Table 5 3rd row, with SR=43.8, CLS=54.4, etc.)? Does it mean the BabyWalk model is not SOTA for R2R on the leaderboard metrics?

Thanks a lot for your interest in our work and all your previous comments. If I understand correctly, you would like to see our model trained on R2R and evaluated on the standard VLN benchmark. We did not do this because of the following reason:
As you might already noticed, Babywalk is not the best model to explore some ''shortcuts'' in the data (as what was shown in Jain et. al. ACL 2019) and achieve a superior success rate. Instead, Babywalk is better at following long instructions and produce high-fidelity paths. Therefore it generates a better result in terms of the instruction following metrics such as CLS, or sDTW.
Unfortunately, as far as I know, the current VLN leaderboard focuses on evaluating the model's capability in goal-reaching and does not support the above two evaluation metrics. We believe that our model would not be able to outperform existing leaderboard results in terms of success rate.
Please let me know if you have any other questions.
Best,

@hexiang-hu thanks for the detailed response! When you say "instruction following", that is agnostic to length of the instruction, right? So if I understand correctly, BabyWalk is better at long instruction following (as in CVDN, treating the entire dialog history as an instruction), but poor at short instruction following (as in VLN)?
Also, your CVDN entries have very high Goal Progress. Am I right in inferring that BabyWalk does well on Goal Progress only if the language instruction is quite long?


Caddy stuck on "acme: Registering account"

When starting caddy with SSL, the server get stuck on a loop of:
2015/12/15 02:08:52 [INFO] acme: Registering account for<EMAIL_ADDRESS>This is my Caddyfile:

https://example.org, https://www.example.org{
    root /home/caddy/domains/example.org/html
    log ../logs/access.log {
        size 10
        age 7
        keep 5
    }
    errors {
        log ../logs/error.log {
            size 10
            age 7
            keep 5
        }
    }
    gzip
    ext .html .htm .php
    redir https://www.example.org https://example.org{uri} 301
    fastcgi / /var/run/example.org.php.sock php
}

I have caddy binary at /opt/caddy/caddy owned by root. Web files and web logs are stored at /home/caddy/, process log is stored at /var/log/caddy/caddy.log Caddyfile is stored at /etc/caddy/Caddyfile, pid file is at /home/caddy/caddy.pid, all is owned by caddy user/group. I'm starting caddy using this upstart conf file:

description "Caddy Server startup script"
author "Mathias Beke"

start on runlevel [2345]
stop on runlevel [016]

setuid caddy
setgid caddy

respawn
respawn limit 10 5

script
    cd /home/caddy/
    exec ./caddy -agree -email<EMAIL_ADDRESS>-log /var/log/caddy/caddy.log -pidfile /home/caddy/caddy.pid -conf="/etc/caddy/Caddyfile"
end script

Having the Caddy binary at the caddy user home, owned by the caddy user doesn't work as well.
Thank you very much!

When starting caddy with SSL, the server get stuck on a loop of:

Can you explain more what you mean by this? "stuck on a loop of" -- what's the full output? What do you mean by "stuck" and "loop"?

Sorry for this delay. "stuck on a loop of" means that when I'm using tailf to read the log, caddy will keep putting this line to the output as long as I let it run. No error before this step, but I don't have the logs here to put the entire output of caddy server. As soon as I could get the caddy logs I will put it here.
Thank you.
P.S.: I think I provided enough information for you to try reproduce this error. If not, just ask what you need and I will try to give you the information.

I really need the log output to know how far apart the looping occurs; is it instant or are there several seconds between the loop, etc.

Closing due to inactivity.

I guess this is the same issue that I'm having.
Jul 22 00:03:01 example.org systemd[1]: Stopped Caddy HTTP/2 web server.
Jul 22 00:03:01 example.org systemd[1]: Starting Caddy HTTP/2 web server...
Jul 22 00:03:01 example.org systemd[1]: Started Caddy HTTP/2 web server.
Jul 22 00:03:01 example.org caddy[11403]: Activating privacy features... 2019/07/22 00:03:01 [INFO][FileStorage:/var/lib/caddy] Started certificate maintenance routine
Jul 22 00:03:01 example.org caddy[11403]: 2019/07/22 00:03:01 [INFO] acme: Registering account for<EMAIL_ADDRESS>
It's a fresh Cent OS install with yum install caddy, some directoryies created under /var/www for different websites and then a config file under /etc/caddy/conf.d with with domain { path \ tls email }.

If it's just hanging there, my guess is some sort of network blockage, either on your local net or the wider Internet, some middlebox maybe. If you want to find out for sure, I can suggest some places to add some debug lines in the code and try again, and we can figure that out.


Game backend

Added routes and controller for game. It now supports:

create a game
join in a game
assign roles
start the game
make next move
get current game state

Adjusted some methods in game engine according to needs from game handlers.
More adjustment may be made .

Added end-turn and restart ,methods, redesigned code structure in back end controller.


add INVALIDATE_PAGE_CACHE_ON_STARTUP setting

Description
My use case is one where I really don't want the page cache to be invalidated on the start of a new instance for a variety of reasons:

Google App Engine's built-in memcache implementation doesn't fully work until the app is initialized.
The app is set up to scale instances up frequently, and I don't want to invalidate the cache for every single instances each time a new instance is started.

Checklist

[x] I have opened this pull request against develop
[ ] I have added or modified the tests when changing logic
[x] I have followed the conventional commits guidelines to add meaningful information into the changelog
[x] I have read the contribution guidelines  and I have joined #workgroup-pr-review on Slack to find a “pr review buddy” who is going to review my pull request.

@devyntk Thanks for contributing to DjangoCMS, we really appreciate it. I will approve the run for test suite and we can take it from there.

@devyntk Seems like the docs fail to build. The error seems to be short underline under the title.
sphinx.errors.SphinxWarning: /home/runner/work/django-cms/django-cms/docs/reference/configuration.rst:953:Title underline too short.
[187](https://github.com/django-cms/django-cms/runs/5618947094?check_suite_focus=true#step:6:187)

[188](https://github.com/django-cms/django-cms/runs/5618947094?check_suite_focus=true#step:6:188)
CMS_INVALIDATE_PAGE_CACHE_ON_STARTUP
[189](https://github.com/django-cms/django-cms/runs/5618947094?check_suite_focus=true#step:6:189)
==============

Could you please fix that so the test could run and pass again?

Thank you @devyntk for this.


When using "Run all", is the order of execution of cells guaranteed?

I'm using httpbook to automate API testing, and I saw in one instance that my tests are flaky when run with the "Run All" toolbar button.
Running them one cell after another works perfectly.
I cleared my history and reopened the request file to debug the order of calls, but now it works again.
My question: is the order of calls guaranteed when using Run All? is the runner waiting for the response of one cell before running the next one?

Actually, the call on my part would always be the same. But it may be related to the error from #102 or maybe the cells I receive from the notebook are not sorted. Is the error related to missing variables? Do you have a stack or error message for me?

I could not reproduce the problem deterministically -- the error was in the test of the business logic.
Basically, I'm setting a boolean to true, testing that it is true, setting it false, testing that it is false, and those tests were randomly failing.
I'm going to test extensively this and report back.

Seems to work fine! we can close the issue

THX fornfeedback


Snapshot pop-up is not shown

Describe the bug
Snapshot pop-up is not shown when I used a Hangul-named dataset.
To Reproduce
Steps to reproduce the behavior:

Import & wrangle 아로나민 골드 데이터 수정본.csv.txt
Click "Snapshot"
Nothing happens

It returns an unknown error (Malformed input or input contains unmappable characters: snapshots/아로나민 골드 데이터 수정본.csv.txt_20190711_062129)
Expected behavior
Show the pop-up, at least.
If Hangul path is not acceptable, change it with Hangul-Roman converting library.
Screenshots

Desktop (please complete the following information):

OS: MacOS
Browser: Chrome
Version: 68.0.3440.106

It's been fixed already. (Maybe with another issue)


Refine .teamcity/pom.xml

The pom is simplified by removing unncessary dependencies.
All dependencies are upgraded to latest version.

I've triggered the following builds:

Pull Request Feedback (Trigger) (Check)

Build Scan

The following builds have passed:

Pull Request Feedback (Trigger) (Check)

Build Scan


refactor: remove /hub and /pages subfolder routing

Description
Remove /hub and /pages routing.

What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[x] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings

Steps to QA

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[ ] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Removing the pages/[pageId]/* routes breaks the explore page and associated filter pages
/javascript - topic
/javascript/(dashboard/contributors/activity) - topic tabs
/javascript/dashboard/filter/(recent/top100/etc) - filters
/javascript/dashboard/filter/open-sauced/app - specific repo
The pages/pages/* and hub/* routes can be removed safely

We should revisit whether we want to redirect those and put them under a /topic path or put repo pages under a /repo path

Removing the pages/[pageId]/* routes breaks the explore page and associated filter pages
/javascript - topic /javascript/(dashboard/contributors/activity) - topic tabs /javascript/dashboard/filter/(recent/top100/etc) - filters /javascript/dashboard/filter/open-sauced/app - specific repo
The pages/pages/* and hub/* routes can be removed safely

What @brandonroberts  mentioned. I thought it was a different route being removed. We can also maybe just revise how the explore route works as it's the only affected route? It relates to https://github.com/open-sauced/app/issues/2600

I am all for whatever is decided as long as we setup a 301 to the new place. But sounds like a quick chat around what the new explore page could like @isabensusan

Talked with @isabensusan and @nickytonline, current plan related to #2600 is to remove /hub and /pages as @brandonroberts said while we move the /[pageId] under a new /explore route in a separate PR.
That way we can eventually add a dedicated Explore page  that will have a search and featured repos/workspaces in the future (a la https://hot.opensauced.pizza), and clear the catchall to pave way for the repo pages being under the /[org]/[repo] route.


⚠ Text Filter API Endpoint has degraded performance

In 497419f, Text Filter API Endpoint (http://textfilter.roblox.com/) experienced degraded performance:

HTTP code: 200
Response time: 3517 ms

Closing because glitched


Should use routes from route config?

https://github.com/input-output-hk/project-icarus-chrome/blob/ed79fb45b17be8553f0ae3d6037baa3a561239fb/app/Routes.js#L38

You are correct, should be use routes from route config instead of that string.

I'm taking not of this to add it to our task list.
Thanks a lot for your feedback @mrkidsan


Error "encrypted blobs do not add up to file size"

Hi, I was trying to downloading a folder using the command in Readme and ran into this error:
Error: execute command

Caused by:
    0: download mail: `https://app.tuta.com/mail/LZRFBMQ-17-1/LZjPlka----1`
    1: download file #1
    2: encrypted blobs do not add up to file size, should be 78692 bytes but got 78753 bytes

I think it is trying to download a pdf attachment.
Also something I'm not sure is related or not: if I open the link in browser, I'll get redirected to login page and the inbox. The folder id is actually LZRFBMQ-1--1 in browser, but in the program it's LZRFBMQ-17-1.

What size does the PDF attachment have if you store it as a file?
I'm wondering if I misunderstood that size field 🤔

I just retried it again and I realized I made a typo yesterday. Actually because the program is running multi-threaded, there are two failures happening with the same error: one is a png file, one is a pdf file.
The png file give the error mentioned above:
2: encrypted blobs do not add up to file size, should be 78692 bytes but got 78753 bytes

and the pdf file gives
    0: download mail: `https://app.tuta.com/mail/LZRFBMQ-17-1/LZss_IK----1`
    1: download file #1
    2: encrypted blobs do not add up to file size, should be 37863 bytes but got 37921 bytes

I checked the file sizes (I downloaded the files directly with browser) with ls -l on Linux. The png file is 78692 long and the pdf file is 37863 long, so it seems the field itself is right, but the downloaded blobs have some extra data?
I cannot share the files because they contains private info, but I'll try to see if I can find a filed attachment I can share.

So I thought that the field that we get from the server does not describe the actual data size of the final file, but the size if the encrypted data (which has a few extra bytes). At least that was the case for the tests that I ran. Maybe they changed it at some point? Can you roughly tell me when you received that email (like maybe the year and the month, just to roughly if that's recent data or not).

The two emails are from Feb-Mar of 2019, they are quite old.

That's somewhat in line my hypothesis that they've changed that at some point, and hence cannot replicate that now. I'll downgrade that hard assertion to a warning.

Also something I'm not sure is related or not: if I open the link in browser, I'll get redirected to login page and the inbox. The folder id is actually LZRFBMQ-1--1 in browser, but in the program it's LZRFBMQ-17-1.

I'ts unrelated, but still a bug which will confuse people. So I've extracted this into #279. Thanks for reporting!


add about engine dialog

add engine info dialog
show token and processes for the plugin engine
improve installation dialog

I don't understand everything yet, but now I can read vue and have a feeling what is going on at least.

Netlify is great, there is a preview version ready! click Details and brings to the new version.

We need to change this line and not call showEngineDialog since it's not a function anymore.
https://github.com/oeway/ImJoy/pull/85/files#diff-1ccdf80d75855f7db1810e8779e7d17eR3

With this change it seems to be working.
diff --git a/web/src/components/EngineControlPanel.vue b/web/src/components/EngineControlPanel.vue
index 3461650..c090249 100644
--- a/web/src/components/EngineControlPanel.vue
+++ b/web/src/components/EngineControlPanel.vue
@@ -1,6 +1,6 @@
 <template>
   <div class="engine-control-panel">
-    <md-button class="md-icon-button md-primary" v-if="!engineManager.engines || engineManager.engines.length==0" @click="showEngineDialog()">
+    <md-button class="md-icon-button md-primary" v-if="!engineManager.engines || engineManager.engines.length==0" @click="showEngineSettings()">
       <md-icon>🚀</md-icon>
       <md-tooltip>Connection to the Plugin Engine</md-tooltip>
     </md-button>


No non-deforming transform option for ImageDataGenerator

The image data generator's flow method always perform some random deformations (translation, rotation, etc), if those features were selected. However, there should be a method that only applies the normalization options (mean, stnd dev, whitening) and ignores the deformations. Otherwise there is no reasonable way to obtain a normalized version of images at test time without also deforming them.

Hi,
You can just specify 0 or False in the options available when constructing an instance of the ImageDataGenerator class. Defaults to none for those 'deforming' transformations such as translations, flipping and flipping.
From the docs:
keras.preprocessing.image.ImageDataGenerator(featurewise_center=True,     samplewise_center=False,     featurewise_std_normalization=True,     samplewise_std_normalization=False,     zca_whitening=False,     rotation_range=0.,     width_shift_range=0.,     height_shift_range=0.,     shear_range=0.,     horizontal_flip=False,     vertical_flip=False)


Cannot load local mmlu dataset

I want to load the local mmlu dataset, I have already load it from [hails/mmlu_no_train，

then I change the _default_template_yaml as follows:
dataset_path: parquet
dataset_kwargs:
  data_dir: autodl-tmp/mmlu_no_train/all
test_split: test
...

I run
export HF_DATASETS_OFFLINE=1

and
lm_eval --model vllm \
    --model_args pretrained=${model_name},tensor_parallel_size=${GPUs_per_model},dtype=${dtype},gpu_memory_utilization=0.9,data_parallel_size=0 \
    --tasks mmlu \
    --apply_chat_template \
    --batch_size ${batch_size}

got

You need to make sure you have exactly one dataset_kwargs in your yaml file. Otherwise the last dataset_kwargs will overwrite the first dataset_kwargs so the data_dir will be None.  I spent hours to debug and found out this in a painful way..

@wukaixingxp I'm so so so so sorry that you spent much time on my negligence! (T-T) but I still can't load the local dataset, could you give me some more help? Thank you very much!!!!!!!!

now the yaml file as follows, is there any other problem?
dataset_path: parquet
dataset_kwargs:
  data_dir: autodl-tmp/mmlu_no_train/all
  trust_remote_code: no
test_split: test
fewshot_split: dev
fewshot_config:
  sampler: first_n
output_type: multiple_choice
doc_to_text: "{{question.strip()}}\nA. {{choices[0]}}\nB. {{choices[1]}}\nC. {{choices[2]}}\nD. {{choices[3]}}\nAnswer:"
doc_to_choice: ["A", "B", "C", "D"]
doc_to_target: answer
metric_list:
  - metric: acc
    aggregation: mean
    higher_is_better: true
metadata:
  version: 1.0

You need to make sure you have exactly one dataset_kwargs in your yaml file. Otherwise the last dataset_kwargs will overwrite the first dataset_kwargs so the data_dir will be None. I spent hours to debug this error message Parquet magic bytes not found in footer and found out this in a painful way..

Thank you so much, I have solved this problem, the correst yaml should be:
dataset_path: autodl-tmp/mmlu_no_train
dataset_kwargs:
  data_dir: all
...

Thank you so much really!!!!!!

Hi The above changes for local datasets can't work on my environment, and will trigger the following two issues: a) one issue is can't find dev: I need to remove the line "fewshot_split: dev" from the default template YAML, then the issue could pass.
  File "huggingface_transformers/benckmark/lm-evaluation-harness-main/lm_eval/api/task.py", line 845, in __init__
    if self.fewshot_docs() is not None:
       ^^^^^^^^^^^^^^^^^^^
  File "huggingface_transformers/benckmark/lm-evaluation-harness-main/lm_eval/api/task.py", line 974, in fewshot_docs
    return self.dataset[self.config.fewshot_split]
           ~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "anaconda/anaconda202402_work/envs/transinfer_cuda12/lib/python3.12/site-packages/datasets/dataset_dict.py", line 72, in __getitem__
    return super().__getitem__(k)
           ^^^^^^^^^^^^^^^^^^^^^^
KeyError: 'dev'

b) After change with a, I encounter another issue with the dataset name. Then I tried to apply the below patch to bypass this check, and the mmlu_human_aging task can run successfully.
File "anaconda/anaconda202402_work/envs/transinfer_cuda12/lib/python3.12/site-packages/datasets/builder.py", line 342, in __init__
    self.config, self.config_id = self._create_builder_config(
                                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "anaconda/anaconda202402_work/envs/transinfer_cuda12/lib/python3.12/site-packages/datasets/builder.py", line 569, in _create_builder_config
    raise ValueError(
ValueError: BuilderConfig 'human_aging' not found. Available: ['default']

--- a/lm_eval/api/task.py
+++ b/lm_eval/api/task.py
@@ -265,7 +265,7 @@ class Task(abc.ABC):
         """
         self.dataset = datasets.load_dataset(
             path=self.DATASET_PATH,
-            name=self.DATASET_NAME,
+            # name=self.DATASET_NAME,
             data_dir=data_dir,
             cache_dir=cache_dir,
             download_mode=download_mode,
@@ -922,9 +922,9 @@ class ConfigurableTask(Task):
                     )

     def download(self, dataset_kwargs: Optional[Dict[str, Any]] = None) -> None:
+        print(f"dataset_path: {self.DATASET_PATH}, dataset_kwargs: {dataset_kwargs}")
         self.dataset = datasets.load_dataset(
             path=self.DATASET_PATH,
-            name=self.DATASET_NAME,
             **dataset_kwargs if dataset_kwargs is not None else {},
         )

From the below run log, I found this mmlu_human_aging task will load all 14042 prompts to run the benchmark. That means if I specift the "--tasks mmlu", per task will run those 14042 prompts. Is this expected? Thanks
2024-11-06:21:50:17,047 DEBUG    [cache.py:33] requests-mmlu_human_aging-3shot-rank0-world_size1-tokenizer is not cached, generating...
2024-11-06:21:50:17,047 INFO     [task.py:414] Building contexts for mmlu_human_aging on rank 0...
100%|██████████████████████████████████████████████████████████████████████| 14042/14042 [00:51<00:00, 271.75it/s]
2024-11-06:21:51:09,203 DEBUG    [evaluator.py:460] Task: mmlu_human_aging; number of requests on this rank: 56168
2024-11-06:21:51:09,207 INFO     [evaluator.py:489] Running loglikelihood requests
Running loglikelihood requests:   0%|                                                   | 0/56168 [00:00<?, ?it/s]Starting from v4.46, the `logits` model output will have the same type as the model (except at train time, where it will always be FP32)
Running loglikelihood requests:   1%|▎                                      | 537/56168 [00:40<1:01:28, 15.08it/s]

hello, I meet the same error. Have you solve the problem?


goravel workflow 审批流扩展包

📑 Description
Closes https://github.com/goravel/goravel/issues/?

@coderabbitai summary

✅ Checks

[ ] Added test cases for my code

goravel workflow 扩展包

Please solve the conflict.

solve the conflict already

add installer

Great 👍


v3.3.0 induce breaking changes in some packages of the ecosystem

Hello!
I've noticed that some packages (for example: wireit) have been broken with the release of the v3.3.0 of jsonc-parser.
Related to: https://github.com/google/wireit/issues/1111

I've been seeing the same behavior starting 3.3.0. In my case it's through the use of rollup-plugin-esbuild, but the symptom and results are the same.
file:///{{REDACTED}}/node_modules/rollup-plugin-esbuild/dist/index.mjs:11
import { parse } from "jsonc-parser";
         ^^^^^
SyntaxError: Named export 'parse' not found. The requested module 'jsonc-parser' is a CommonJS module, which may not support all module.exports as named exports.

See also:

https://github.com/microsoft/node-jsonc-parser/pull/78#issuecomment-2186933283
https://github.com/microsoft/node-jsonc-parser/pull/90#issuecomment-2186954613

Fix is on the way, I'll revert the change.

3.3.1 has brought things back to the way they were before:

https://arethetypeswrong.github.io/?p=jsonc-parser%403.3.1
<EMAIL_ADDRESS>

3.3.1 has brought things back to the way they were before:
* https://arethetypeswrong.github.io/?p=jsonc-parser%403.3.1

*<EMAIL_ADDRESS>

Oh my god, this is AWESOME, thank you!
Can I remove the overrides in my package.json (using pnpm) and check for you if all works again as expected in my project?

Yes, just avoid 3.3.0.

Yes, just avoid 3.3.0.

It works for me, using wireit, all works again as expected, thank you!

Hello @jakebailey
Should we close this issue as resolved?

Sure (I'm not on the team that works on this package so I can't do it).

Sure (I'm not on the team that works on this package so I can't do it).

Okay, let's close this this!
Feel free to reopen this issue if you need to.


Why I can return MyObject but not Vec<MyObject>?

The code below works:
#[derive(Debug, Serialize, Deserialize, Default, Clone)]
pub struct MyObject {
    info: Option<String>,
    status: Option<String>
}

#[Object]
impl MyObject {
    async fn name(&self, _ctx: &Context<'_>) -> Option<String> {
       ...
    }

    async fn description(&self, _ctx: &Context<'_>) -> Option<String> {
       ...
    }
}

#[Object]
impl QueryRoot {

    async fn get_thing(&self, ctx: &Context<'_>, thing_id: u64) -> MyObject {
      ...
   }
}

I would expect this to work too:
#[Object]
impl QueryRoot {

    async fn get_things(&self, ctx: &Context<'_>) -> Vec<MyObject> {
      ...
   }
}

But I get that
#[Object]     | ^^^^^^^^^ the trait InputTypeis not implemented forMyObject
Why would I even need that? I should already be able to call name() and description() on each element of Vec<MyObject>.
What am I missing?

Hi @fgadaleta! I just tried your code and it compiled with no problems.
This is the code I tried (I added imports and returned dummy data in the methods):
use async_graphql::{Context, Object};
use serde::{Deserialize, Serialize};

#[derive(Debug, Serialize, Deserialize, Default, Clone)]
pub struct MyObject {
    info: Option<String>,
    status: Option<String>,
}

#[Object]
impl MyObject {
    async fn name(&self, _ctx: &Context<'_>) -> Option<String> {
        Some("name".into())
    }

    async fn description(&self, _ctx: &Context<'_>) -> Option<String> {
        Some("description".into())
    }
}

struct QueryRoot;

#[Object]
impl QueryRoot {
    async fn get_thing(&self, ctx: &Context<'_>, thing_id: u64) -> MyObject {
        MyObject {
            info: Some("info".into()),
            status: Some("status".into()),
        }
    }

    async fn get_things(&self, ctx: &Context<'_>) -> Vec<MyObject> {
        let my_object1 = MyObject {
            info: Some("info".into()),
            status: Some("status".into()),
        };

        let my_object2 = MyObject {
            info: Some("info".into()),
            status: Some("status".into()),
        };

        vec![my_object1, my_object2]
    }
}

fn main() {
    println!("Hello, world!");
}

Cargo.toml:
[package]
name = "async-graphql-vec-test"
version = "0.1.0"
edition = "2021"

[dependencies]
async-graphql = "3.0.6"
serde = { version = "1.0.130", features = ["derive"] }

What version of async-graphql are you using?

I just upgraded Cargo.toml to
actix-rt = "2.2.0"
actix-web = "3.3.2"
async-stream = "0.3.2"
# async-graphql = { version = "2.11.3", features = ["dataloader"] }
# async-graphql-actix-web = "2.10.2"
# async-graphql = "3.0.6"
async-graphql = { version = "3.0.6"} #, features = ["dataloader"] }
async-graphql-actix-web = "3.0.6"

Problem seems to be gone.
But now I have a weird error when I call index from the definition of the HttpServer
.service(web::resource("/").guard(guard::Post()).to(index))
    |                                                                  -- ^^^^^ the trait `Factory<_, _, _>` is not implemented for `fn(actix_web::web::Data<async_graphql::Schema<QueryRoot, MutationRoot, SubscriptionRoot>>, GraphQLRequest) -> impl Future<Output = GraphQLResponse> {index}`

        HttpServer::new(move || {
            App::new()
                .data(schema.clone())
                .service(
                    web::resource("/")
                    .guard(guard::Post())
                    .to(index)
                )
                .service(
                    web::resource("/")
                    .guard(guard::Get())
                    .to(index_playground)
                )
        })
        .bind(format!("{}:{}", self.server_address, self.server_port))
        .unwrap()
        .run()
        .await
    }

and below the regular index and playground
async fn index(schema: web::Data<SwampSchema>, req: GraphQLRequest) -> GraphQLResponse {
    schema.execute(req.into_inner()).await.into()
}

async fn index_playground() -> Result<HttpResponse> {
    Ok(HttpResponse::Ok()
        .content_type("text/html; charset=utf-8")
        .body(playground_source(
            GraphQLPlaygroundConfig::new("/").subscription_endpoint("/"),
        )))
}

async-graphql v3 only supports actix-web v4 beta. In addition, I suggest you try poem, I think it is easier to use.

thanks! I will definitely consider poem.
Any idea why with actix-web = { version = "4.0.0-beta.11", default-features = false }  I do get a
thread 'main' panicked at 'No runtime detected. Start a Tokio (or Actix) runtime.', /Users/frag/.cargo/registry/src/github.com-1ecc6299db9ec823/actix-server-2.0.0-beta.9/src/worker.rs:318:17
stack backtrace:
   0: std::panicking::begin_panic
             at /rustc/65f3f8b220f020e562c5dd848ff7319257a7ba45/library/std/src/panicking.rs:525:12
   1: actix_server::worker::ServerWorker::start
             at /Users/frag/.cargo/registry/src/github.com-1ecc6299db9ec823/actix-server-2.0.0-beta.9/src/worker.rs:318:17
             ```

You should use the actix_web::main macro to wrap your main function, it will create an async runtime.
https://github.com/async-graphql/examples/blob/c4043062a12f17ef80373d84a67986e3a27f3421/actix-web/starwars/src/main.rs#L19

I already do that
#[actix_web::main]
async fn main() -> Result<(), Box<dyn Error>> {
 ....

   futures::join!(
        async move {
            // spawn graphql server
            let _ = graphql_server.run().await;
        }
    );

I do that because I launch several services from main
with graphql_server.run()
    pub async fn run(&self) -> std::io::Result<()> {

             HttpServer::new(move || {
            App::new() ....

Show me your Cargo.toml 🙂

[dependencies]
actix-rt = "2.2.0"
actix-web = { version = "4.0.0-beta.11", default-features = false }
async-stream = "0.3.2"
async-graphql = { version = "3.0.6"} #, features = ["dataloader"] }
async-graphql-actix-web = "3.0.6"
dotenv = "0.15.0"
log = "0.4.0"
log4rs = "1.0.0"
futures = "0.3.17"
futures-timer = "3.0.2"
once_cell = "1.0"
rand = "0.8"
serde = "1.0"
serde_derive = "1.0"
serde_json = "1.0"
slab = "0.4.2"
tokio = { version = "1.12.0", features = ["rt-multi-thread", "macros"] }

It seems okay, but this problem is definitely not caused by async-graphql. If you can provide a repo that can reproduce this problem, I can help you solve it.

From main i am launching 2 services, one is a web socket server and a graphql server.
If in main I comment the websocket for now, and if I replace  #[actix_web::main] with #[tokio::main] the graphql server starts...

    futures::join!(
        // async move {
        //     // spawn websocket server
        //     let _ = wsserver.run().await;
        // },

        async move {
            // spawn graphql server
            let _ = graphql_server.run().await;
        }
    );

If I use  #[actix_web::main], I get a panic that no runtime is found.

The websocket server has these dependencies in its Cargo (I kept only the relevant ones)
[dependencies]
actix-web="3.3.2"
actix-web-actors="3" # actors specific to web
actix = "0.10.0" # actors
async-std = { version = "1.6", features = [ "attributes" ] }
futures-util = { version = "0.3.11", default-features = false, features = ["io"] }
futures-channel = "0.3.11"
tokio = { version = "1.12.0", features = ["rt", "rt-multi-thread", "time", "macros"] }

I think you have encountered a very troublesome issue of incompatibility between actix-web and tokio, which is why I don't use actix-web.

ouch...
wise decision... i was always against having 2 different runtimes :(
i guess i will have to keep that part commented for now and eventually port that service to tokio container.

In the meantime, back to the very first problem that started this thread (returning an object from a graphql endpoint) I can return an object MyThing but I cannot return HashMap<i64, MyThing>; without rustc complaining with
#[Object]
    | ^^^^^^^^^ the trait `InputType` is not implemented for `MyThing`

(which is not true). Or should I wrap that HashMap into another struct and derive #[Object] too?

Input values (parameters to fields, fields of input object) must implement the InputValue trait. Output values (objects, interfaces, unions) must implement the OutputValve trait.
Async-graphql does not implement InputValue for HashMap<K, V> because GraphQL does not support it.
If you want any types that implement Serialize/Deserialize to be input or output, you can wrap them with Json.
https://docs.rs/async-graphql/3.0.7/async_graphql/types/struct.Json.html


Obx throw exception when used in dynamic widgets

when using Obx() in a widget that can be destroyed and recreated like items in a ListView or Tabs it throws a exception after the widget is recreated and have a update on controller.
I looked the memory use in devtools and saw that everytime the widget is destroyed the obxx keep it instance and when recreated obxx create a new instance without dispose the previous.
am i using Obx() somehow wrongly?

[ERROR:flutter/lib/ui/ui_dart_state.cc(166)] Unhandled Exception: 'package:flutter/src/widgets/framework.dart': Failed assertion: line 4180 pos 12: '_debugLifecycleState != _ElementLifecycle.defunct': is not true.
E/flutter (12450): #0      _AssertionError._doThrowNew (dart:core-patch/errors_patch.dart:40:39)
E/flutter (12450): #1      _AssertionError._throwNew (dart:core-patch/errors_patch.dart:36:5)
E/flutter (12450): #2      Element.markNeedsBuild (package:flutter/src/widgets/framework.dart:4180:12)
E/flutter (12450): #3      Obxx.build. (package:get/src/rx/rx_obx.dart:18:63)
E/flutter (12450): #4      _rootRunUnary (dart:async/zone.dart:1198:47)
E/flutter (12450): #5      _CustomZone.runUnary (dart:async/zone.dart:1100:19)
E/flutter (12450): #6      _CustomZone.runUnaryGuarded (dart:async/zone.dart:1005:7)
E/flutter (12450): #7      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:357:11)
E/flutter (12450): #8      _DelayedData.perform (dart:async/stream_impl.dart:611:14)
E/flutter (12450): #9      _StreamImplEvents.handleNext (dart:async/stream_impl.dart:730:11)
E/flutter (12450): #10     _PendingEvents.schedule. (dart:async/stream_impl.dart:687:7)
E/flutter (12450): #11     _rootRun (dart:async/zone.dart:1182:47)
E/flutter (12450): #12     _CustomZone.run (dart:async/zone.dart:1093:19)
E/flutter (12450): #13     _CustomZone.runGuarded (dart:async/zone.dart:997:7)
E/flutter (12450): #14     _CustomZone.bindCallbackGuarded. (dart:async/zone.dart:1037:23)
E/flutter (12450): #15     _rootRun (dart:async/zone.dart:1190:13)
E/flutter (12450): #16     _CustomZone.run (dart:async/zone.dart:1093:19)
E/flutter (12450): #17     _CustomZone.runGuarded (dart:async/zone.dart:997:7)
E/flutter (12450): #18     _CustomZone.bindCallbackGuarded. (dart:async/zone.dart:1037:23)
E/flutter (12450): #19     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
E/flutter (12450): #20     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)

Hi, you are using "obx" (lowercase) or "Obxx". This api is very very very experimental, and has not been documented because of that.
Use Obx() (capital and only one x)

I don't believe that hours searching what was wrong and it was just a lowercase letter. haha

Unfortunately, I get this error with Obxwidget on get-5.0.0-beta.36
ObserverComponent._safeRebuild isn't safe :)


Bump python >= 3.8 and fix constraints generation

Although we attempted to increase Airflow version in #1438,
constraints generation is still failing in main as we try to generate
them on Airflow 2.4.3. We no longer would need to generate constraints
for Airflow 2.4.3, as we have the min Airflow version to 2.6.0 now.
With that we also remove the duplicate job in CI for Python 3.11
constraints generation
Additionally, this PR also bumps the min Python version to 3.8,
we also do not already run tests on Python 3.7 in our CI.

Constraints got generated successfully previously with this PR. and hence I am applying the comment now.


Properly return retval in TransactValue

For a CALL transaction, we should return the retval, because some RPC calls need it.

Thanks, WeiTang! LGTM.
I think I just need standard::invoker mod to be pub...


false negative for emits

Just updated to the latest deps and noticed that I'm not getting a lot of linting errors about:

The "input" event has been triggered but not declared on emits option.eslintvue/require-explicit-emits

But I believe my types are sound, as this screenshot would indicate:

The type information highlighted in green is correct and by that logic the value in the first parameter is also correct. However, instead I get the require-explicit-emits linting error.
OS: macOs Monterey
Volar: 0.29.6
Typescript: 4.5.2

Volar do not provide linting, you should report to ESLint. :S

ahhh; but this same signature wouldn't give a linting error in a .ts file ... at least I'm pretty sure it wouldn't.

Maybe it would but it didn't in the past ... very odd. I'll try and raise an issue there though. Thanks for pointing me in the right direction.


refactor peek_definition more  easy to use and more useful

remove close back next keymaps in peek_definition.  if you want go back prev definition just <C-o> .

@scottming can you test this version peek_definition ?

I didn't find any major bugs for now. I like it.
And this area is no need.

Got it, Leaving the line number doesn't affect me too much. Just leave it. Reliability is always the most important thing.
Thanks for your work.

I found another bug, these update will break the normal ctrl-o
before peek: goto definition -> ctrl-o can back
after peek_definition and q: goto definition -> ctrl-o do nothing.
https://user-images.githubusercontent.com/12830256/220334237-c9ed4f5f-3f93-4242-8aa2-65a15d981dc5.mp4

you opened new window the pos is changed.

and in my test it works  fine ?


Is it possible to add a custom onChangeText function within child component in react native that updates value?

I want to create a reusable input component that handles different input types within my app, for example remove all non-numerical numbers and decimals from the value onChange if the type is integer. The parent passes the observable value into the child which sets up the <Legend.input /> component. Example:
const state = useObservable({ email: '', password: '' })

 <Input
    placeholder="Email"
    keyboardType="email-address"
    value$={state.email}
    inputType={InputTypes.integer}
/>

const Input = ({ ...props }) => (<Legend.TextInput
  placeholderTextColor={placeholderColor}
  style={[inputStyle, { width, height }, style]}
  onChangeText$={I WOULD LIKE TO REMOVE ALL NON-NUMERICAL NUMBERS IN A FUNCTION HERE}
  {...props}
/>)

I'd love to not have to do this from the parent component if possible. Usually with the react state you would just pass in the setValue function but I am not sure how to access the set function or if its possible after passing the obersvable through props. Thanks for any help!

I've run into this a couple of times myself and wondered if we should have an onBeforeSet or something for the bound values, to validate them before they get set?
But for now, what I do is just make the Input component take a value$ Observable prop, useSelector to get the value from it, and render a normal TextInput with it. Then in the onChangeText, you can validate it and set it on the observable with value$.set
const Input = ({ value$, ...props }) => (
  <TextInput
    value={useSelector(value$)}
    onChangeText={(newValue) => value$.set(validate(newValue))}
    {...props}
  />
)

Oh, another way to do it using the new two-way computed that's in the @rc version:
const Input = ({ value$, inputType, ...props }) => {
  const validatedValue = useComputed(value$, (value) => {
    value$.set(validate(value, inputType))
  })
  return (
    <Legend.TextInput
      placeholderTextColor={placeholderColor}
      style={[inputStyle, { width, height }, style]}
      value$={validatedValue}
      {...props}
    />
  )
}

But that would add a bit more performance overhead of creating a new observable and hook. It is cool though 😅

+1 for willSet. I think apple nailed the naming of willSet and didSet


Selection from the autocomplete always returns the first value and not the actual selected value

Is there a way to return the tableview selected index value?

I have this problem too, did you solve it?

@mirzadelic
Modifying the plugin was a bit tedious.
I found a much better and simple way of doing this. Please refer to the link below. Just copy and paste the answer in the comments and it works like a charm.
http://stackoverflow.com/questions/26885438/getting-autocomplete-to-work-in-swift
Hope this helps.

Yes, but i need it as dropdown :(

I am attaching a screenshot of how it appears.
It's very similar to this plugin. Hope it helps.

Is this table that hide and show?
I used this code http://stackoverflow.com/a/26975029/1204527
But tableview doesn't show?

Do i need to add tableview in storyboard?

Yes it's a tableview that shows and hides.
Yes add it in the storyboard and set the datasource and delegate and it should work!

Ohh, i tried without adding tableview to storyboard..
Thanks..

Also link the tableview to an IBOutlet if you want it to dismiss when the view ends editing.

How to hide tableveiw when done editing?

Create an IBOutlet for the tableview and then in the didSelectRowAtIndexPath function make the changes below
func tableView(tableView: UITableView, didSelectRowAtIndexPath indexPath: NSIndexPath) {
    let selectedCell : UITableViewCell = tableView.cellForRowAtIndexPath(indexPath)!
    textField.text = selectedCell.textLabel!.text
    tableview.hidden = true
}

I did that, but i now i have problem when i touch outsite textfield, to hide tableview and close keyboard?

Simple. Add this code in your view controller
override func touchesBegan(touches: Set<UITouch>, withEvent event: UIEvent?) {

        self.view.endEditing(true)
        tableView.hidden = true
}

Thanks, it works great..


Inconsistencies regarding the display of annotations / descriptions for graphs and properties

This is perhaps two separate issues, so apologies if they should be separated. I'm having trouble getting multiple annotation properties to display, either for the ontology itself or for individual properties. I believe this is similar to https://github.com/RDFLib/pyLODE/issues/92, but different enough that it seems to remain an issue.
For the ontology, the property used to generate the (html) output of 'Description' changes depending on the input .ttl. For one, dc:description or dcterms:description is used, but the rdfs:comment is ignore. For another file with a dcterms:description and two rdfs:comment properties, one of the rdfs:comment properties is used instead.
As for the property descriptions, several properties have a skos:definition and an rdfs:comment, but the latter is ignored in all cases. I'm not sure if only showing one annotation property is expected behavior, but I think ideally the output should display any accompanying annotations per property.
Any assistance with these issues would be appreciated.

@nicholascar - Thanks for the detailed response. As for the proposed fix:

So I take the outcome of this issue to be to allow multiple values for dcterms:description. Will that do?

I think @graybeal has it correct. Occasionally, some annotation terms seem to be used interchangeably, but as mentioned, ones such as rdfs:comment and skos:definition are fairly distinct and used for different purposes.
In my use case, the ability for properties to display these separately in the output would be ideal, but failing that, multiples of a single dcterms:description may be helpful.

OK, I'm going to go with multiple dcterms:description values initially and then think about @graybeal's point. I do acknowledged it - I've seen ontologies using dcterms:description alongside rdfs:comment etc. but the reason I haven't implemented them separately is because the semantics are un-clear: what does rdfs:comment mean alongside dcterms:description? I can work out what rdfs:comment means alongside skos:definition  - the latter actually defines the class/prperty, the former is just an additional note - but I'd like a single, cohesive set of annotations to be used in general.
I think the best bet is to suggest use of skos:definition  for the defining text and then other SKOS properties with defined intentions - skos:historyNote, skos:scopeNote being the main ones. Since all the SKOS *Note properties are subPropertyOf rdfs:comment, I'd then strongly discourage people from using rdfs:comment and SKOS properties!
I'm thinking of submitting pyLODE to a semantic web workshop later in the year and, if I do, I'll need to tidy up all this theory. I should tidy up the theory regardless of course... Still, I will eventually write up these thoughts and then put it to users to see what they make of them.

Alright, looking forward to the change, thank you!
I agree that the differentiation between rdfs:comment and dcterms:description is a bit unclear, so no real answer for that. I'll most likely end up cleaning up the use of them for the ontology description since now it's a bit messy. As for skos:definition and rdfs:comment, the distinction is a bit more clear as you say, but maybe there is more reasonable way to go about using multiple annotations for single properties.

I think the best bet is to suggest use of skos:definition for the defining text and then other SKOS properties with defined intentions - skos:historyNote, skos:scopeNote being the main ones. Since all the SKOS *Note properties are subPropertyOf rdfs:comment, I'd then strongly discourage people from using rdfs:comment and SKOS properties!

Since I haven't tried this specifically (as I was using rdfs:comment), will pyLODE currently display multiple skos annotations properties (e.g. a skos:definition along with a skos:scopeNote) for a single property, or is this also a change that will be implemented? If it's currently implement, I could easily change my rdfs:comments to a skos property and solve the issue.

...does pyLODE currently display multiple skos annotations properties (e.g. a skos:definition, along with a skos:scopeNote) for a single property...

Have a look at https://github.com/rdflib/pyLODE/#what-pylode-understands and yes, pyLODE understands skos:historyNote at the ontology/vocabulary level and scos:scopeNote at the elements (Class & Property) level. See this DecProv ontology class for a skos:scopeNote example alongside a description.

There are two reasons people would be using rdfs:comment rather than SKOS properties: there isn't a SKOS property that applies; or, they don't want to entail or even reference SKOS in what is explicitly a logic model and not a vocabulary. So I expect many people do use rdfs:comment with disciplined intent.
Keep in mind that all of these allow multiple uses per subject anyway (partly for language reasons, partly to allow multiple independent annotations of a type), so there should not be a case IMHO where you only show one of several.
I confess using the properties dc: and dcterms:description is something I'm most familiar with at the ontology level, but because they can be applied to any entity, I suppose they need to be taken into account throughout. The practice for describing non-SKOS ontological concepts is likely pretty diverse.

Well at the moment rdfs:comment, dc:description, dcterms:description & skos:definition are all catered for, so people are quite able to keep using rdfs:comment if they wish (as many/most do).
Definitely will enable multiple uses per subject! And I do need to work on multiple language rendering. Probably need a setting to indicate a primary language and then support all others.
I think I can do better regarding the description of what's supported and also allow people to ask for more things to be supported (sdo:description perhaps??) since the main goal for pyLODE's structure is to be easy to update.

skos:note is the general case.
All the other annotations are sub-properties of that.

skos:note isn't catered for (see ontdoc.py#L257) but should be, so I'll add it when making the change to handle multiple values.

One way I've handled these multiple text blobs is to just prefix Description sections with the property name used in the source, e.g.
Description
dc:description:  This owl ontology is designed to duplicate the intention of the CGI SimpleLithology201001 vocabulary. It provides an OWL view of the vocabulary ........

rdfs:comment: To see the hierarchy of categories, open in protege (or similar owl-enable software with some inferencing capabilities). In Protege, run 'Pellet' o......


could not get this data  through the msg_fields_buffer.name

Dear Sir
A customer use driver under unbantu without ros for picoScan150.
They feedback just can get the intensity data through the msg_fields_buffer[n]. name and without others data.
Such as:
fields_offset_rang:-1
fields_offset_azimuth:-1
fields_offset_elevation:-1
fields_offset_intensity:12
fields_offset_t:-1
Please help to check what is the reason and how to deal with it, Thanks.

Thank you for your feedback. I think this is the same problem as in the https://github.com/SICKAG/sick_scan_xd/issues/410#issuecomment-2443725824 issue. Otherwise, please post more information, i.e. which API functions, parameters and callbacks your customer is using.
The customer can get the range,azimuth,elevation,i fields by registering a polar point cloud callback. Function SickScanApiRegisterCartesianPointCloudMsg registers for cartesian point clouds with the fields x,y,z,i; function SickScanApiRegisterPolarPointCloudMsg registers for polar point clouds with the fields azimuth,elevation,range,i. Simply replace SickScanApiRegisterCartesianPointCloudMsg(apiHandle, customizedPointCloudMsgCb) with SickScanApiRegisterPolarPointCloudMsg(apiHandle, customizedPointCloudMsgCb) in the code and the callback should get azimuth,elevation,range,i.
See the Generic Driver API for examples and details.


coco pod not working

When installing pod I get the error "Unable to find a specification for InteractiveSlideMenu"

Same problem here.

You need to update your spec repo: pod repo update


sample to use it

Hi , thanks for creating the package . But I still cannot figure out how to use it . I have set the EnableLogging method in my context class as you mentioned . But in my client when I try to log the SQL , I neither get to see context.Database.Log ... not am I able to do IQueryable.ToString to get the SQL like EF6 .
Could you provide some samples on the client please ? As to how exactly to get the SQL from IQueryable .

The SQL would be in your Log4Net logs.
Are you using Log4Net

Here's a sample project https://github.com/camilin87/debug-ef-core-sample


Support macOS

It would be great if mac was supported, as I plan to release a Mac app quite soon. Thanks.

Hi @Jethro87
I'm one of the Product Managers here at Customer.io. Thanks for submitting this request as a future addition for us to look into! I have made an internal note for our team to keep this as an item of interest as we continue to evolve our product.
I'll be closing this issue since it's been documented. Thanks so much for submitting your idea:)


WIP doc: zip processor idea?

Q
A

Bug fix?
N

New feature?
Y

BC breaks?
N

Deprecations?
N

Fixed tickets
N

Good idea for opening this PR!


Fix unbounded growth of Parliament's memory usage

It has been observed in the Presto project deployments that Parliament's memory usage grows over time to excessive levels.  There has been some evidence that this is related to Parliament's backup facility, though this has not been verified.

This issue appears to have disappeared.


Add bages or even reformat list to table with availability sorting.

It is hard to choose most popular library now. For example there are 9 libraries for local storage. I want to decide which is most popular, still maintaned, etc.
Would be nice add github badges with stars/issues/PR counts to list of available libraries using service like https://github.com/ddavison/github-badges or any other. Availability of this kind of information saves time when you searching for new library.
Not sure, is adding badges for large list is good idea (it can cause too much http requests). Maybe use api for aggregating data from repos and cache it. But it requires more efforts and separate site.

Agree with that,
Indeed, It is really hard to choose in a large text plain list

@Koc Good idea, we will consider it.

@Akryum created site for that https://awesomejs.dev/ , so this issue is not actual anymore


OracleSchedulerJob - Added a run command

Added a run command to execute the job asynchronously and added a panel for viewing the job action which could be a PLSQL block up to 4000 characters long.
I'm fairly new to github and eclipse development, so I apologise in advance for any errors.
I've enjoyed learning through making these changes to DBeaver, and hope to contribute more in future.
Please review and let me know if this is a successful PR.
Is it an issue that I developed the change in the master branch?
Should changes first be done in devel?

Thanks,
I'll review it and write some comments.
It is better to work with develop always. Master is always a bit outdated (and sometimes not a bit).

Generally looks good.
There are a couple of issues with formatting (mixed tabs and spaces) and one strange if/else instead of switch. But that's easy to fix.
The only thing which concerns me is quite big handler class which is very similar to existing CompileHandler. Perhaps we need some base class for them..


Preserve execution context when running code in ThreadPool

Currently contextvars from the ASGI context are not preserved into the WSGI context, which can lead to issues with certain middleware.

Eg. see https://github.com/django/asgiref/pull/73
In Connexion, we want to leverage contextvars to provide context to the application

Sorry for the lateness, I have been off for the last few days due to COVID-19.

No worries, was not expecting a response over Christmas anyway. Hope you feel better!

@abersheeran could this be released? We'd like to remove the monkeypatch we have right now in connexion.

@abersheeran could this be released? We'd like to remove the monkeypatch we have right now in connexion.

https://pypi.org/project/a2wsgi/1.7.0/ Thanks for your contribution, this PR is now released in 1.7.


Subdirectories of archives not being extracted

While scanning I noticed, that not all files from (at least) 7z files are extracted.
The archive looks like this:

Upon scanning it without --debug it just shows:
8,33%: C:\apps\audio\The GodFather\The GodFather v0.7.7z - Checking file
0,00%:  -
0,00%: \lame.ini - Checking file from archive
11,11%: \lame.ini -
11,11%: \lib.ini - Checking file from archive
22,22%: \lib.ini -
22,22%: \oggvorbis.ini - Checking file from archive
33,33%: \oggvorbis.ini -
33,33%: \tgf.chm - Checking file from archive
44,44%: \tgf.chm -
44,44%: \tgf.ini - Checking file from archive
55,56%: \tgf.ini -
55,56%: \tgf.mdb - Checking file from archive
66,67%: \tgf.mdb -
66,67%: \TheGodFather.exe - Checking file from archive
77,78%: \TheGodFather.exe -
77,78%: \Uninstall.exe - Checking file from archive
88,89%: \Uninstall.exe - NSIS v2.46
88,89%: \xaudio.dll - Checking file from archive
100,00%: \xaudio.dll -
9,72%: C:\apps\audio\The GodFather\The GodFather v0.7.7z -

with debug I get lots of messages like this one:
System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\Users\Ezatoka\AppData\Local\Temp\42b3b7ed-76f2-43b3-8583-d2719cfad726\Lng\bulgarian.lng'.
But of course the files are there in the 7z file:

rar archives also have this problem:
11,18%: C:\games\Istaria\resources_override\Tools\REShade Istaria + Starstipreset.rar - Checking file
System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\Users\Ezatoka\AppData\Local\Temp\2272aaa6-9e9e-476b-b612-9f28a0f61a6f\reshade-shaders\README.md'.

Is this still happening? A lot has changed in the underlying archive code since.

The cause of this was finally determined and fixed.


added examples of pairtools stats reports

adding examples of the report files that pairtools generates - needed for https://github.com/ewels/MultiQC/pull/1148

Thanks!


No such file or directory, then panic/backtrace

╭ zeph  ~/code/nixcfg 0s
╰─▶ /nix/store/hhclh187i2pkll9i2cqghbbizk7nd06j-amdgpu_top-0.1.7/bin/amdgpu_top
No such file or directory (os error 2)
render_path = "/dev/dri/renderD128"
card_path = "/dev/dri/card0"
thread 'main' panicked at 'explicit panic', src/main.rs:92:21
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

I'm trying to package this for nixpkgs and use it on my G14 2022 (AMD+AMD), but not having a lot of luck so far.
Any advice? Setting RUST_BACKTRACE to 1 or full didn't really help.

I do not provide nixpkg and I am not familiar with nixpkg.
Try a binary built from source or an AppImage binary.
sandbox setting might be affecting it?
https://nixos.org/manual/nix/stable/command-ref/conf-file.html#conf-sandbox

I am building from source, and the binary builds fine, this is at execution time which does not happen in the sandbox.

Please paste the results of running ls -l /dev/dri.
amdgpu_top requires access to /dev/dri/.
Environments with device files located in locations other than /dev/dri are not supported.

❯ exa -al --tree /dev/dri
drwxr-xr-x        - root  2 May 15:33 /dev/dri
drwxr-xr-x        - root  2 May 15:44 ├── by-path
lrwxrwxrwx@       8 root  2 May 15:44 │  ├── pci-0000:07:00.0-card -> ../card1
lrwxrwxrwx       13 root  2 May 15:44 │  └── pci-0000:07:00.0-render -> ../renderD129
crw-rw----@   226,1 root  2 May 15:44 ├── card1
crw-rw-rw-  226,129 root  2 May 15:44 └── renderD129

This system has two GPUs, but I usually boot Linux with the dGPU disabled.
This might be why my system (currently) only has dri1.
I can take a peek at the code...

Run amdgpu_top -i 1.
The result of amdgpu_top --list is also helpful.

╭ zeph  ~/code/nixcfg 21.12s
╰─▶ /nix/store/hhclh187i2pkll9i2cqghbbizk7nd06j-amdgpu_top-0.1.7/bin/amdgpu_top -i 1

seems to work, shows the TUI

╭ zeph  ~/code/nixcfg 1.85s
╰─▶ /nix/store/hhclh187i2pkll9i2cqghbbizk7nd06j-amdgpu_top-0.1.7/bin/amdgpu_top --list
#1
Marketing Name = "AMD Radeon Graphics"
pci = 0000:07:00.0
render_path = "/dev/dri/renderD129"
card_path = "/dev/dri/card1"

╭ zeph  ~/code/nixcfg 21.12s
╰─▶ /nix/store/hhclh187i2pkll9i2cqghbbizk7nd06j-amdgpu_top-0.1.7/bin/amdgpu_top -i 1

seems to work, shows the TUI

╭ zeph  ~/code/nixcfg 1.85s
╰─▶ /nix/store/hhclh187i2pkll9i2cqghbbizk7nd06j-amdgpu_top-0.1.7/bin/amdgpu_top --list
#1
Marketing Name = "AMD Radeon Graphics"
pci = 0000:07:00.0
render_path = "/dev/dri/renderD129"
card_path = "/dev/dri/card1"

Ok, close this issue.

I don't understand. The app tries to load the wrong card? Or maybe you're addressing it and I just don't understand?

I don't understand. The app tries to load the wrong card? Or maybe you're addressing it and I just don't understand?

The first DRM devices will be placed in /dev/dri/renderD128 and /dev/dri/card0 unless intentionally disabled.
Therefore, amdgpu_top selects /dev/dri/renderD128 by default.
Open "/dev/dri/render{}", 128 + i
if an instance (-i) is specified.

Does it make sense to:

Auto-select the first available instead of assuming index 0?
Support multi-GPU setups through-out the app?

I understand if that's all out of scope. It is very jarring to get a file not found and panic, though.

GUI mode implements GPU selection fallback.
This is because in GUI mode, other GPU devices can be started in a new process.
https://github.com/Umio-Yasuno/amdgpu_top/commit/ac94ed0f8ecfa1a45fd20cc2e9dca5ee72bcf840


"Cannot find bazel" return 0 in configure.py

Hello!
In file configure.py inside a function check_bazel_version when bazel is not exists this function returns 0.
Here is a part of code:
if which('bazel') is None:
print('Cannot find bazel. Please install bazel.')
sys.exit(0)
Should be sys.exit(1)? It's an error.

Sounds good. Can you send a PR please?

Yes


Using the Moveit of ROS to control real Z1 Manipulator

Is there a function package for using ROS Moveit to control the real Z1 robot arm? If not, when will it be released?

Hello,it's already in the final stage, will give as soon as possible.

The Z1_ROS package has been provided now.


index.html

added favicon , favicon file is attached

added favicon , favicon file is attached

It's ok. But there have a conflict issue check it once and recreate a new PR.

sorry sir just missed it


Price field doesn't have enough room when prefix added

I think there's likely a different/better way to address this, which would be in the Currency section of settings. There we should select which currency we're using, and an option for how we should display it (e.g. Not at all, prefix/affix to symbol, affix to end...) and let our Price::format() method add it in appropriately.
This would have several down the road benefits. For example, if we have multiple currencies, we can use this configuration to pass which currency we're using to the payment processor. As well as limit what payment options to show depending on the currency etc.

This has been fixed for now at least


Loading bitmap with too large height

I am facing a issue while loading bitmap in imageview of too large height of images but size of images is less than 2MB. Universal Image loader destroying the quality of image and text on it is not readable and if not scaling the image then it is not showing in ImageView. I have posted the same problem in detail on stackoverflow. please check and suggest the better solution.
https://stackoverflow.com/questions/55510041/loading-bitmap-with-too-large-height-in-android-imageview

Answered on StackOverFlow.


Bug in GET_DAQ_CLOCK_MULTICAST fixed

Fixed the GET_DAQ_CLOCK_MULTICAST problem.
High dword and low dword of timestamp swapped.
Wonder why CANape did not complain about this.
Measurement just started with some undeterministic delay.

Hello @RainerZ
I run the lastest V5 and there is still issue 'The determination of the XCP 1.3 DAQ start time failed' here, should i set the CANAPE, ? But i does not find the path in the CANAPE 17.


Revert "Remove the PodSecurityPolicy sample GRC policy (#833)"

This reverts commit fda831dc5bc48dd56368f5634e99f6e3be4786e1. This stems
from @berenss's decision in:
https://github.com/open-cluster-management/policy-collection/pull/171#issuecomment-953988183
Signed-off-by: mprahl<EMAIL_ADDRESS>

/lgtm


Updated to 0.9, now getting t.choices is not a function

Every time I switch to an edit page, I'm now getting this error:
Error: t.choices is not a function
r/<.compile/<.pre@http://localhost:3001/adm/js/vendor/ng-admin/build/ng-admin.min.js:15:2543

«snipped Angular call stack »

 <ma-choice-field class="ng-isolate-scope" field="::field" entry="entry" value="value">

I've narrowed it down to line 36 of maChoiceField.js.  I don't know exactly what's wrong.  I think it's a mistake, because it calls the function then uses the result.
Looks like the change was made in commit 0f0693647f252783d7cb4b0ecd6f1ef00c333167.
If it matters, these fields are using a custom function for the choices.  It's a bit of a hack to allow me to asynchronously load some meta data from the server, but it shouldn't affect anything.  The way it works is like this:
let someChoices = [];
app.config(function(...) {
    field.choices(() => someChoices);
});

...

app.run(function ($http) {
    // load remote values
    someChoices = remoteValues;
});

I'd make a minimal test case, but I'm honestly trying to figure out how to do that, since you need a RESTful backend.

I really wonder about your error. I've never seen it myself. And it seems that it doesn't set a & directive attribute as a function... Which Angular version are you using?

Angular 1.4.5.  What you said in the PR makes sense, but I definitely got that error consistently.  I'll try upgrading again later today.

BTW: I didn't think foo: '&?' was a valid use for optional expressions.  It seems like a mistake to be overriding the passed in expression in the scope property like that.

Why is it a mistake?

Angular 1.4.5

That's why. ng-admin is tested on angular 1.3. Migration to 1.4 is not yet started (cf #567)

Because it's a function created on the scope to call back the expression, and it's being overridden with a static property, breaking the binding.  It makes the logic harder to follow, too, because it switches from a function to an array at some point.

Really? I thought I just read that v0.9 was going to be the switch to ng 1.4, since 0.8 was the last to support 1.3.x.
That sucks for me. :frowning:

OK, this fix worked for me on Angular 1.4.7:
var choices = (angular.isFunction(scope.choices) && scope.choices()) ? scope.choices : (field.choices ? field.choices() : [])

That still doesn't make sense. Does the following work?
var choices = (typeof scope.choices == 'function' && scope.choices()) ? scope.choices() : (field.choices ? field.choices() : []);

0.8 was supposed to be the latest version built on Angular 1.3, but we packed too many features in 0.9 not to release it before breaking compatibility. You'll have to wait a bit for 1.4 compatibility.

Note: I tracked down the BC Break in Angular 1.4 to this commit: https://github.com/angular/angular.js/commit/6a38dbfd3c34c8f9efff503d17eb3cbeb666d422

Yup, looks like that's the change.
The code from above is almost the same thing as mine, I'm just using Angular's built-in isFunction, rather than typeof.  I assume it would work the same.
The other difference is you are calling scope.choices() for the assignment, which is different than the existing code, which looks more like mine:
var choices = scope.choices() ? scope.choices : (field.choices ? field.choices() : []);

Thank you, sir! :-)

This fix is not yet added to built files and cannot be easily installed via Bower, I presume?..

It is added to built files. To get it from bower, you must reference the master branch (see http://stackoverflow.com/a/16951315 for directions).

I've tried and that didn't work - built files were updated 20 days ago and the fix was added 19 days ago...

It should be fixed by now, I've published a new build a couple days ago.

I'm reading the documentation (Getting-started chapter) and i have the  same error when i tried to display the users filter in the post listview section.
I am using angularjs 1.5.0
Should have UI to wait a bit for  a 1.5 compatibility.?


Some question about DSBN

I noticed that, in your ST3D++ paper, the transformation parameters γ and β are shared between two domains since "the transformation parameters γ and β are domain agnostic". but in paper "Domain-specific batch normalization for unsupervised domain adaptation", γ and β are not shared between source domain and target domain. I want to confirm whether the γ and β are shared to reproduce you work. Look forward to you reply, thank you very much!

No. They are not shared as the explained in this work: https://papers.nips.cc/paper/2019/hash/fd2c5e4680d9a01dba3aada5ece22270-Abstract.html.
BTW, I will support DSNorm and SASD officially soon.

In this paper, it's shared:

I'm very puzzled about this.

Well, it's a good news that DSNorm and SASD will release officially, thank you very much!

Sorry, I should say: they are shared. That's a typo.

Supported. You can have a try on it. However, as DSNorm and SASD are designed more for small objects such as pedestrian and cyclist. It might not show significant improvment on Car.

You are really a hardworking and enthusiastic man. Thank you very much! Hope you will get good results on NeurIPS 2022!


Hide header bar

i need help in hide header bar

Please add more detail to this issue.
What sort of help do you need?

i am using this addSlide(AppIntroFragment.newInstance(title, description, image, background_colo

ur)); i want to remove android header bar

i fix it by style but my api is 11 and i want to use android:windowTranslucentStatus i can't use for uper api

Have you tried any of these:
    // OPTIONAL METHODS
    // Override bar/separator color.
    setBarColor(Color.parseColor("#3F51B5"));
    setSeparatorColor(Color.parseColor("#2196F3"));

@guri073 How did you remove that bar from the top?

For others who are searching for this -
In your styles.xml define a style
<style name="FullscreenTheme" parent="Theme.AppCompat.Light.NoActionBar">
</style>

And in your manifest, add it as the theme for the Intro activity
<activity android:name=".AppIntro"
            android:theme = "@style/FullscreenTheme">
</activity>

Thanks @abhishekpriyam  But i already fixed :+1: . I did same as your steps and i add windowTranslucentStatus to style 21 api for status bar color and every thing is working fine now


Fix build process in build jails

Fixes #22

Thank you very much!
Closes #22

Uh... you rejected the pull request.

I'm very sorry - maybe vacation mode? Thanks again and thanks for the heads up!


clickable dashboard (cpu and memory)

which shows sorted hosts, also refactored spinners and sortOrder

@suomiy it does work but there is one strange behavior: if you go to dashboard, click the cpu or memory and than click "back" it turns the app off instead of going back to the dashboard.

@jelkosz I think this should be intended behaviour. Main activity is top level in our hierarchy. So we return from dashboard here with focus on a fragment. If we allowed this feature, the user could go again to dashboard and even repeat it few times, which would create cycle and backstack like this doesn't sound much intuitive to me.

@suomiy you are right, merging


[ARI] Add Stephan as ARI Execution Lead

As I announced in the most recent TOC meeting, I will be going on parental leave for 3 months, scheduled to begin in early February. To help keep ARI running smoothly while I'm out, it's important to have someone in the lead role, either provisionally or permanently as a co-lead.
ARI is a large and diverse working group, and I believe it would be valuable going forward to have a dedicated execution lead, even after I return from my leave.
To fill this role, I nominate @stephanme. Stephan has been an active participant in the working group for a long time, most notably via his contributions to the CAPI area and the monthly CAPI office hours. You may also know him from his position on the CFF TOC 😄.

I guess I should not vote on this PR but I'm fine with the nomination.


Update json2.js

https://github.com/douglascrockford/JSON-js

The version of json2.js included in this project's test/ directory is used as input for JSHint, not direct execution. Because of that, its outdatedness is not a concern. Changing it correctly would require also changing the expected errors encountered, but that isn't desirable in this case. We lint the json2.js source code as a way to identify regressions between releases, so stability there is important. Thanks anyway!


Mock PA2 iteration difference

Hi just wondering why are these two sets of code giving different results?
Stream.iterate(Num.zero(), x -> !x.equals(other), x -> x.succ())

and
for (Num i = Num.zero(); !i.equals(other); i = i.succ()) {
    result = result.add(this);
}

Does this mean that for stream iterations it does not reach the predicate's final value?

how are you terminating the stream?

how are you terminating the stream?

I used
.reduce(thisNum, (a, b) -> a.succ());
return Num.of(numStream.opt);

Just curious, wouldn't the first one add 1 to the result other-times to the result whereas the second (the for loop) adds the value of this to the result other-times? So unless this is an instance of Num.one, these two solutions shouldn't equal one another.

oh yes my bad, I used them for add and mul respectively. I was more confused about how stream's iteration and for's iteration are handled differently. If I were to use the stream's iteration for mul and add (instead of only using it for add in this case), it would give me wrong outputs.


Add support for having multiple independent (also seeded) RNGs in nopython mode

Feature request

For reproducible tests, I need to randomly generate huge amounts of data (in the form of numpy ndarrays), and I also need to fine-tune which of these arrays are generated deterministically (e.g. from a seeded RNG) or not.
Unfortunately, currently numba doesn't support RNG objects in nopython mode (like stated here), but only calls to the global rng (np.random.[...]), hence, when I need to selectively create reproducible data for unit tests, I have to revert to object mode which is slower.
I think the generalization of numba's random backend for allowing independent (and seedable) rngs is a very important feature that could be very useful for speeding up test and debugging scenarios related to pseudorandom data generation.

@pgp thank you for asking about this.
The documentation at:
https://numba.readthedocs.io/en/stable/reference/pysupported.html#random
is for the random module of the Python library.
Numba also has support for the Numpy random module, which is what I think you are looking for:
https://numba.readthedocs.io/en/stable/reference/numpysupported.html#random
This should allow for independent and seedable RNGs are requested.
Please feel free to close this issue again if the implementation suits your needs and comment here if something else is needed, thank you.

Thank you very much for the clarification, my mistake was declaring the Generator object within the njit scope (and not passing a normally initialized one from outside njit), and this caused strange errors. The following code works fine:
from numba import jit
import numpy as np

def newrng(seed=None):
    return np.random.default_rng() if seed is None else np.random.default_rng(seed)

@jit(nopython=True)
def nextint(r, a, b):
    return r.integers(a,b)

def genSomeNums(r):
    for _ in range(5):
        print(nextint(r,10,20))

if __name__ == '__main__':
    # standard rng
    rng = newrng()
    genSomeNums(rng)
    print('*' * 30)
    rng1 = newrng(5)
    genSomeNums(rng1)
    print('-'*30)
    rng2 = newrng(5)
    genSomeNums(rng2)


remove support for make deploy

Nit: We should update the PR name/commit message to say that we're removing make deploy rather than merely deprecating it, since normally deprecation will keep the functionality around for a while.

/retest


Make webpack a peer dependency so that webpack 2 can be used

Sorry for the wait

Any updates on this?

Hi @domaslasauskas
Can you update npm with the latest version which supports this PR ?
Thanks !

I just ran into a really nasty issue where I was trying to load a .json file.   Running webpack normally worked fine but compiling with serverless webpack failed with an error.   After eventually debugging through the serverless webpack code and forcing it to output the webpack version I discovered the version mismatch.   I had webpack 2.2.1 installed globally but the serverless plugin was using 1.14 and causing my error.
I'm anxiously awaiting a new version on npm with this fix...

@jamesandersen this will get you what you need in the meantim: yarn add --dev https://github.com/elastic-coders/serverless-webpack/tarball/master.


Join: Remove Google Groups. Spruce up a bit.

Removed Google Groups from the website completely, ahead of archiving that forum.
Spruced up the Join page a little whilst I was there:

Clearer alert box about the main Nextflow Gitter channel
New consistent call-to-action button to join each channel section (used twitter and YouTube embeds)
Minor rewording, mention of Slack group for GitHub invitations.

I just deleted the google group, so let's merge this ASAP if that's ok :)

On thing with respect to wording: "Follow this link to get an invite to join the nf-core Slack organisation." when refering to a button seems confusing to me.
Pedantic side note: the twitter logo should be the bird without a box around it, which in my opinion would also fit better in with the other logos.

Yup, I agree with both points. The Slack text was the end result of moving around and fiddling repeatedly, which is why it probably sounds a bit odd. Any suggestions for better alternatives? Better still, want to make a PR? 😉


Adding appveyor.yml

Codecov Report

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

@@          Coverage Diff          @@
##           master   #310   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          59     59
  Lines        1370   1370
  Branches      208    208
=====================================
  Hits         1370   1370

Flag
Coverage Δ

#builder
100% <ø> (ø)
:arrow_up:

#runtime
100% <ø> (ø)
:arrow_up:

#srcapp
100% <ø> (ø)
:arrow_up:

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 790163c...60980aa. Read the comment docs.


Providing loading progress for the Web version API for Wasm and Model card

MediaPipe Solution (you are using)
All
Programming language
Javascript
Are you willing to contribute it
None
Describe the feature and the current behaviour/state
The file sizes of both wasm and model cards can be massive. Around 10MB each. It will be great if the web user can get the current loading progress
Will this change the current API? How?
No response
Who will benefit with this feature?
Any web user
Please specify the use cases for this feature
Anyone who are not one a ultra fast internet can have like 10 seconds load time for the library to be ready to use. Having an API to allow developers to implement the loading bar to show the progress of their application would be a great help.
Any Other info
No response

I got this problem as well. However, we can use assets preload library and track the progress without relying on Mediapipe, then convert the loaded assets to blob urls. Similar to this:
await preload.start()

holistic = new Holistic({
          locateFile: function (file) {
                    const binary = preload.binary(file);
                    const blob = new Blob([binary], { type: "application/wasm" })
                    return URL.createObjectURL(blob)
          },
        });

Some solution loading .wasm file needs the mimetype to be "application/wasm"

Hello @edankwan
Thank you for suggesting this feature! It aligns well with current web development trends and presents a valuable addition to MediaPipe. We'll discuss it with the team and prioritize it in our roadmap.
(As a side note, the API being used by @hohoaisan is legacy API)

Any update?


extendable editor

add new entry: /mould/setup
usage
// /mould/setup.ts
import CustomComponent from 'custom-lib'

export default ({Components, Controls}) => {
    return {
        Components: [...Components, {
            type: 'custom component',
            ...CustomComponent
        }],
        Controls: CustomControls,
        ViewWrapper:  View => () => <I18nProvider><NormalizeCSS><View /></NormalizeCSS></I18nProvider>
    }
}

This setup function will run once the editor starts.
tasks

[x] add singleton Mould instance including fields of components and controls
[x] change all deps of components(controls) to Mould instance
[x] prepare setup entry file on dev
[x] remove Mould\Kit\List from components registry, they are core components that should not be overloaded
[ ] inject ViewWrapper & use sandbox for keeping view panel clean

"inject ViewWrapper & use sandbox for keeping view panel clean"

will be resolved by next PR


not really an issue, but does this one still support github actions?

just wondering

You can use the previous deploy.yaml file from the previous repo and it still works

previous repo?

previous repo? could you please send a link? i am new here

https://github.com/siddrajpoot/MicrosoftRewardsClaimer/blob/main/.github/workflows/deploy.yml
Follow the structure I have in my fork


v1.3 - EmailWindow

Created an EmailWindow instead of relying on .txt files, however I'm still in the process of fixing a glitch:
When using the Mac OS, if you change the tab while the EmailWindow is open, it kills the program and gives an NSException, which is related to Apple's JDK environment variables. Still looking into it...

Pull Request Test Coverage Report for Build 345

2 of 78 (2.56%)  changed or added relevant lines in 7 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.2%) to 54.761%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/address/commons/util/QrUtil.java
0
1
0.0%

src/main/java/seedu/address/logic/commands/EmailAllCommand.java
0
2
0.0%

src/main/java/seedu/address/logic/commands/Email.java
0
9
0.0%

src/main/java/seedu/address/logic/commands/MailCommand.java
0
9
0.0%

src/main/java/seedu/address/ui/EmailWindow.java
0
55
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/seedu/address/logic/commands/Email.java
1
0.0%

src/main/java/seedu/address/logic/commands/EmailAllCommand.java
2
0.0%

Totals

Change from base Build 342:
-0.2%

Covered Lines:
1593

Relevant Lines:
2909

💛  - Coveralls


version 1.7.0 is not compatible with ie8

Expected behaviour
connecting to ie8 browsers
Actual behaviour
won't connect, time outs.
Environment Details

Karma version (output of karma --version):
Karma version: 1.7.0
Relevant part of your karma.config.js file

Steps to reproduce the behaviour

https://github.com/browserstack/karma-browserstack-example.git
update package.json to karma version 1.7.0 & karma-browserstack-launcher to 1.3.0
npm install
add ie8 to the karma config, something like this:

ie8: {
        base: 'BrowserStack',
        browser: 'IE',
        browser_version: '8.0',
        os: 'Windows',
        os_version: '7',
      },

npm test

If we update socket.io dependency in node_modules/karma to 2.0.3 it works.
now it's using socket.io 1.7.3
maybe https://github.com/socketio/engine.io/issues/407 is connected.

+1

+1

This issue is fixed on master, socket.io has been upgraded to 2.0.3.