Enable Club routes in Northstar

What's this PR do?
This pull request enables v3/clubs API routes and /clubs Web routes in Northstar 🎉
All the business logic was already copied over from Rogue, so this pull request just worked on getting the tests for these routes working again! ✅
How should this be reviewed?
👀
Relevant tickets
References Pivotal #176851324.
Checklist

[x] This PR has been added to the relevant Pivotal card.
[ ] Documentation added for new features/changed endpoints.
[ ] Added appropriate feature/unit tests.
[ ] If new attributes were added to users, there is a corresponding PR to surface these in Aurora.
[ ] If new attributes were added to users, then the data team already knows about these changes.

Thanks @aaronschachter 🕺


feat: fix CUDA images and update go-llama to use full GPU offloading

nvcc is not compatible with GCC 12 yet. golang:1.20 images got upgraded to debian bookworm, that uses GCC-12 by default. This PR downgrades images to use bullseye.
Fixes: https://github.com/go-skynet/LocalAI/issues/611

I have tested this PR on Linux(x86_64) with docker build -t localai . and docker build --build-arg BUILD_TYPE=cublas localai . but it is still not working very well.

Don't get that. What's not working well? CI builds fine.

I have tested this PR on Linux(x86_64) with docker build -t localai . and docker build --build-arg BUILD_TYPE=cublas localai . but it is still not working very well.

Don't get that. What's not working well? CI builds fine.

I test it again, it works for me now.

After 6-7 requests i stumbled upon
CUDA error 12 at /build/go-llama/llama.cpp/ggml-cuda.cu:2127: invalid pitch argument
https://github.com/ggerganov/llama.cpp/blob/d411968e990c37f51328849c96a743dd78f3c3dd/ggml-cuda.cu#L2127
Not sure if its a memory issue on my end, since i push it quite a bit right now and it works reliable on the first few ones

Running into this same problem


getFrom()

guys, i'm facing problems using getFrom()...
I want to make a system to check if whoever clicked on the button with callback_data is the same as I use the /start command (returns the button with the callback_data)
I tried to do like this:
`
$callback_query = $this->getCallbackQuery();
$compare_id_01 = $callback_query->getMessage()->getReplyToMessage()->getFrom()->getId();
$callback_data = $callback_query->getData();
$callback_UserID = $callback_query->getFrom()->getId();
if($callback_data == 'start'){ if($callback_UserID == $compare_id_01){ return Request::sendMessage(['text' => 'working']);
}else{
return $callback_query->answer([ 'text' => '🚫 Not Permission', 'show_alert' => 1, // Randomly show (or not) as an alert. 'cache_time' => 10, ]);
}
}
`
this works but after a few minutes it returns in the terminal this message: Fatal error: encaught error: Call to a member function getFrom() on null...
do you know how to help me with this?

Edit: it looks like this occurs when a user doesn't use their personal account to send the command.

before you call getFrom() make sure getMessage() first.
you can read the telegram api docs and you can see the from type is inside message type

antes de ligar getFrom()certifique-se getMessage()primeiro. você pode ler os documentos da API do telegram e pode ver que o fromtipo está dentro do messagetipo

ok, i will try that.

before you call getFrom() make sure getMessage() first. you can read the telegram api docs and you can see the from type is inside message type

still returns error...
$compare_id_01 = $callback_query->getMessage()->getReplyToMessage()->getFrom()->getId();

Do not add getReplyToMessage()
Just $callback_query->getMessage()->getFrom()->getId();

Do not add getReplyToMessage() Just $callback_query->getMessage()->getFrom()->getId();

Ok

Do not add getReplyToMessage() Just $callback_query->getMessage()->getFrom()->getId();

I was wanting to make a comparison between the user_ids of users who click on a button contained in a callback_data if($user_id == $user_id_compare){ ... } knows how to do ?

Do not add getReplyToMessage() Just $callback_query->getMessage()->getFrom()->getId();

I was wanting to make a comparison between the user_ids of users who click on a button contained in a callback_data
if($user_id == $user_id_compare){ ... }
knows how to do ?

You don't need to validate user id

Do not add getReplyToMessage() Just $callback_query->getMessage()->getFrom()->getId();

I was wanting to make a comparison between the user_ids of users who click on a button contained in a callback_data
if($user_id == $user_id_compare){ ... }
knows how to do ?

You don't need to validate user id

I need it, because I don't want another user (who hasn't called the command) to be able to move the buttons, just the one who called the command. I managed to do this, thanks anyway!


Prevent users from creating OIDC providers in the environments repo

User Story
As a mp user
I should not be allowed to create OIDC providers via the member pipeline
Because this would allow access via uncontrolled means
User Type(s)

Value
Improved security
Assumptions / Hypothesis / Questions / Unknowns

Definition of done

[ ] readme has been updated
[ ] user docs have been updated
[ ] another team member has reviewed
[ ] tests are green
[ ] UR test OR added to continual research plan

Reference
How to write good user stories

@davidkelliott can you provide a bit more detail - and whether you know how to do this

Just checked, this has already been done


Tagging issue feature

Closes #276

[x] Start suggesting appropriate issues if the user starts typing a number just after # in the issue description

https://user-images.githubusercontent.com/37345795/229311395-a29eeee8-72f0-4649-b19a-637b86eaa986.mp4

[x] Make "#{a number}" clickable while rendering an existing issue's description

https://user-images.githubusercontent.com/37345795/229311404-11964b60-c02f-4304-815e-6a253ab24b74.mp4

Why is it opening the issue in the browser?We should open the issue in the app itself.

@letsintegreat can you please fix the conflicts?


Flatten Vec<Vec> to Vec

This simplifies code and should be better on memory usage since it is less fragmented.
Also added width and height attributes to Layer.
Closes #54

Could you add any interface changes to CHANGELOG.md?


Correction for raw affiliation CIRAD, UPR AIDA, International Institute of Tropical Agriculture (IITA), Cotonou, Benin

Correction needed for raw affiliation CIRAD, UPR AIDA, International Institute of Tropical Agriculture (IITA), Cotonou, Benin
raw_affiliation_name: CIRAD, UPR AIDA, International Institute of Tropical Agriculture (IITA), Cotonou, Benin
new_rors: 0556kt608;01hv86c27
previous_rors: 0556kt608
works_examples: W4388807448
contact: 213adce09ce9eb10ac8fbed46d488970:ec218d2e4b42b49041e7a9b4a660 @ cirad.fr

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


Add instructions on how to use prombench in readme

During the release of 2.4.0-rc, goutham said he could'nt find instructions on how to start benchmarking.
Signed-off-by: Harsh Agarwal<EMAIL_ADDRESS>

How about:
How to trigger tests on Github.
How to run tests manually.
1.Complete the "Setup Prerequisites."
2. ....
Setup Prerequisites.
Lets make this PR for the:  How to trigger tests on Github. and will do the rest in a separate one as it will take longer to get right.


Remove old standalone Firebase feature impls

These currently live in frontend/firebase, but most or all of them are replaced by GraphQL resolvers. The old ones need to be removed.

Done!


Climate settings unable to be reconfigured

System Health details
System Information

version
core-2024.4.3

installation_type
Home Assistant Supervised

dev
false

hassio
true

docker
true

user
root

virtualenv
false

python_version
3.12.2

os_name
Linux

os_version
5.10.0-28-amd64

arch
x86_64

timezone
Europe/Paris

config_dir
/config

Home Assistant Community Store

GitHub API
ok

GitHub Content
ok

GitHub Web
ok

GitHub API Calls Remaining
5000

Installed Version
1.34.0

Stage
running

Available Repositories
1403

Downloaded Repositories
26

Home Assistant Cloud

logged_in
true

subscription_expiration
19 juin 2024 à 02:00

relayer_connected
true

relayer_region
eu-central-1

remote_enabled
false

remote_connected
false

alexa_enabled
false

google_enabled
true

remote_server
eu-central-1-18.ui.nabu.casa

certificate_status
ready

instance_id
381927b4921b47a797e587e8790e1775

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Home Assistant Supervisor

host_os
Debian GNU/Linux 11 (bullseye)

update_channel
stable

supervisor_version
supervisor-2024.04.0

agent_version
1.5.1

docker_version
26.0.0

disk_total
194.4 GB

disk_used
20.8 GB

healthy
true

supported
true

supervisor_api
ok

version_api
ok

installed_addons
Studio Code Server (5.15.0), Samba Backup (5.2.0), AppDaemon (0.16.4), Mosquitto broker (6.4.0), Zigbee2MQTT (1.36.1-1), Z-Wave JS UI (3.4.2)

Dashboards

dashboards
2

resources
12

views
9

mode
storage

Recorder

oldest_recorder_run
16 février 2024 à 21:39

current_recorder_run
16 avril 2024 à 12:21

estimated_db_size
1671.70 MiB

database_engine
sqlite

database_version
3.44.2

Checklist

[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of currently previous issues..

Describe the issue
When i want to reconfigure a existing cover, i'm unable to update Climate settings

Reproduction steps

Create a cover
Try to reconfigure the previous created as show in description

Diagnostics dump
No response

It’s part of the fine tune blind settings menu 😉

this is what i have and i don't have configuration for temperature threshold and sensors

Ok my bad, i've find now that's in a second step of fine tune blind settings.
Thank you

Just click on valider, it will be in the next step


[flutter_tools] revert dart format changes

Description
Revert to invocation of dartfmt snapshot instead of dartdev tool

@pcsosinski @zanderso

we should only land this on 1.20


Add template function on range to return items x through z

Currently there is a first function that returns the first n items. I would like to get a function that returns items n items starting at position x.
i.e. {{ range items 5 5 .Data.Pages }} would return items 5,6,7,8,9 in the list

We now have first, after and last and then some. Plenty of flexibility to solve this use case.


Add option to assign run length values to last day of run period rather than first one

At the moment, for a dataset where we want to find the run lengths of cells with values of 1, for example:
[ 0, 1, 1, 1, 1, 1, 0, 1, 0, 1, 1, 0, 0, 0]
rle is producing an output where the values of the run lengths are saved on the cell of the first occurence:
[ 0., 5., nan, nan, nan, nan, 0., 1., 0., 2., nan,  0., 0.,  0.]
However, it might be needed that the value of the run length needs to be saved on the last day, like this:
[ 0., nan, nan, nan, nan, 5., 0., 1., 0., nan, 2.,  0., 0.,  0.]
In order to do this, we have added a lastday parameter to rle. I'll open a PR in a minute.

Other methods related to rle might need also updating as well as calls to the rle method from elsewhere.


Doesn't work with RN 0.61+

The podspec needs to be moved out of ios folder, and maybe some more changes need to be done

Please try again with version 1.0.10

@FlorianBergmann  I'm using 1.0.10 with RN 0.61.5 and I get error "cannot read property 'getAdvertisingId' of undefined"

i have the same issue with RN 0.61.5

It should now work with RN 0.61 with version 1.0.11

@FlorianBergmann I'm using 1.0.10 with RN 0.61.5 and I get error "cannot read property 'getAdvertisingId' of undefined"

With RN 0.61.x, we still have to add
    <application>
        <meta-data
            android:name="com.google.android.gms.ads.AD_MANAGER_APP"
            android:value="true"/>
    </application>

in AndroidManifest.xml.

@FlorianBergmann I'm using 1.0.10 with RN 0.61.5 and I get error "cannot read property 'getAdvertisingId' of undefined"

With RN 0.61.x, we still have to add
    <application>
        <meta-data
            android:name="com.google.android.gms.ads.AD_MANAGER_APP"
            android:value="true"/>
    </application>

in AndroidManifest.xml.

Release App is still crashing after this.


Update server is down?

I installed arkOS for rgb30 with the image in wiki, since it's rather old (Updated 10/16/2023), I try to update from option menu, but always told me OTA service is down.
And I tried the update script, still cannot connect to the server.
So I check the update script and try to access the patch link from macOS at (http://gitcdn.link/cdn/christianhaitian/arkos/main/11042023/arkosupdate11042023.zip), it shows "Application Error".
Is the update server down?

Where are you located?

China, and I also tried the China location, it's also unable to access.
christianhaitian @.***> 于2023年12月17日周日 16:12写道：

Where are you located?
—
Reply to this email directly, view it on GitHub
https://github.com/christianhaitian/arkos/issues/859#issuecomment-1859068837,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAEQ5R5HLYAJAVSHS4X76TYJ2SPJAVCNFSM6AAAAABAYDEBSSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJZGA3DQOBTG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

I can download the update zip manually, is it possible I can copy it to some folder on SD card 2 and do the update offline?

github is usually not accessible from mainland China.  Unfortunately, the updates are designed to be done via ota.  Currently, there's no solution to do them offline.


[5.7 & v8] Blocks within a stack don't display custom styles

To reproduce, add a stack with a few blocks and block padding, then add it to a page. No padding appears on the stack outputted to the page, while it does within the dashboard.
Caused by the core_stack_display block controllers runAction() not passing calls to it's own on_view_page() method. runAction() is overridden from the base class and passes calls directly to its child blocks. This results in the core_stack_display never outputting custom styles to the page as the collections outputCustomStyleHeaderItems() only processes custom styles for blocks that directly reference the page.
However, it may be an idea at a later date to move the output of the stacks custom block styles into the pages outputCustomStyleHeaderItems() method, it would neaten the custom style handling up as everything else seems to be handled there.

Thanks – I actually fixed this in a slightly different way, but wouldn't have figured it out without your helpful pull request. 1bedb1b


Unable to read replicated hyperdrives with mounts

I ran into a bug with hypercore-8 branch where hyperdrive hangs if you try to read a mount from replicated hyperdrive. I'm not sure whether this is a bug with hyperdrive or mountable-hypertrie.
I created a test case and it's available at https://github.com/mafintosh/hyperdrive/pull/257

Appreciate the test @markopolojarvi. I'll check it out and report back here when I figure out the root cause.

K so looks like the issue here's that when you create drive3, you're passing in a null value for drive1.key because drive1 isn't ready yet.
To fix the test, you can just defer the creation of drive3 until right after the drive1.ready line, or just use drive1.key in the second_drive function (since it doesn't look like you're calling any methods on drive3 directly) :)
Getting the initialization order right is definitely tricky, but most operation's (including the getters) won't work correctly until after ready's been called as a general rule of thumb.
Don't think there are any deeper replication issues here -- at least not surfaced by #257. Very much appreciate including a good test with the issue though!

Actually I'll leave this open in case the create issue was just a small mistake and not related to the core issue.
@markopolojarvi is the test case you made a modified version of some other failing case you're seeing elsewhere.

@andrewosh What a brainfart with they drive 1 key... sorry about that.
All this is related to the problem with the same nested replication structure over hyperswarm, test case at https://pastebin.com/UmVmFRdf. Unless I'm having another brainfart, in that test case initialization is done right but for some reason it still dies.
The reason I did the tape test was that @mafintosh asked if I could replicate the problem with plain old replicate() and it seems I could, if I did the initialization wrong ;)


Add Action to publish docker image on Github registry

Hello, with this now we can publish the docker image on the Github registry the example below:
To being able to publish a new image you need to make a new release tag on Github. Also is expected that the tag has this format vX.X.X and the npm package should exist already. We can also add the npm publish to the same Github action, but that is more complex.
🗒Tasks

We can also publish to the docker hub registry but we need the secret
We can publish the npm package also with the secrets

Thanks so much. I'll likely move the publish step into the circleCI config since that's where everything else is published, but it looks to be an easy port with the code you added.

Great thanks! I already started to use the docker image on docker hub.

Perfect. I realized that I had been a bad maintainer and let two different
folks create PRs to add Docker images. I ended up using the smaller
original Dockerfile but copied all your deploy code into the circleCI
process so it deploys at the right spot.
-Brandon
On Mon, Apr 27, 2020 at 5:45 PM Mijail Rondón<EMAIL_ADDRESS>wrote:

Great thanks! I already started to use the docker image on docker hub.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/bbyars/mountebank/pull/541#issuecomment-620274483,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAARFPZ6C3M5XQYPYEYWCSTROYDIPANCNFSM4MIFPW4A
.


languages-map.json not created following installation steps in readme

I needed to run pnpm build-languages for the file to be generated.
Also FYI, ghloc.ifels.dev is down, so your website doesn't work. Anyone who wants to use this will need to run their own instance of ghloc in server mode.

Hi, sorry for late response. Thanks for reporting issues with development docs! I've updated dev/build scripts. ghloc.ifels.dev had some issues with SSL certificates which were resolved the same day.


⚠ ClamAV has degraded performance

In 27584d5, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 70 ms

Resolved: ClamAV performance has improved in 8b7e3dd.


T-posing after using Dark Discorporation

The char keeps the T posing and makes you unable to do anything else,  like casting something else, even after canceling the invocation

I can't replicate this. It's possible there's an interaction with another mod.
Please attach your player.log and gamelogfull.txt

GameLogFull.txt
Player.log

This appears to only be logging up to the main menu. I need a log from after the issue occurs.
Also this might help and even if it doesn't, it will clean up the logs.


Support for SurrealDB

Describe the feature
Support for SurrealDB
Motivation
SurrealDB is becoming popular in NewSQL paradigm hence favorite SDK like Gorm should have support for this.
Related Issues

Sorry, I currently don’t have the capacity to implement support for SurrealDB. However, I’d be happy if the community could take on this effort. The implementation can either be maintained under the GORM organization or independently. We will add relevant links to recommend it.


[Kernel] Define LogReplay assertLogFilesBelongToTable() helper method

Which Delta project/connector is this regarding?

[ ] Spark
[ ] Standalone
[ ] Flink
[x] Kernel
[ ] Other (fill in here)

Description
Resolves https://github.com/delta-io/delta/issues/2150
Defines LogReplay assertLogFilesBelongToTable() helper method based on the Spark implementation here as specified in issue.
Verifies that a set of delta or checkpoint files to be read actually belong to the transaction log of the delta table. Throws AssertionError if violation detected.

@vkorukanti resolved your comments - thank you for the feedback!

@vkorukanti Feel free to dismiss other reviewers if you are ready to merge. Thank you for the review!

@PatrickLeahey Will merge this PR once https://github.com/delta-io/delta/pull/2212 lands. It is adding lot of more tests and will verify whether anything break.

@vkorukanti Looks like we are still good to go after #2212 merged

@PatrickLeahey Will merge this PR once #2212 lands. It is adding lot of more tests and will verify whether anything break.

✅


Merge rc bugfix into develop

Merge bug fixes from main to develop

Note that CircleCI will have failures when dealing with main until we merge the CircleCI configuration into main. I have tested that this does in fact build, both on my local machine and on discover.


Peer @hapi/hapi needs update

Hello,
I was trying to update to hapi@21 today, and had an issue, noticed that schmervice peer dep range doesn't include 21
{
   "peerDependencies": {
    "@hapi/hapi": ">=19 <21"
  }
}

Will be getting out new versions with official support for hapi v21 soon!  In practice schmervice should not be affected by the breaking changes in v21 👍


php: fix extensions on php-cgi

Description of changes
Before
❯ php-cgi -m
[PHP Modules]
cgi-fcgi
Core
date
hash
json
libxml
pcre
Phar
Reflection
SPL
standard
xml

[Zend Modules]

After
[PHP Modules]
bcmath
blackfire
calendar
cgi-fcgi
Core
ctype
curl
date
dom
exif
fileinfo
filter
ftp
gd
gettext
gmp
hash
iconv
intl
json
ldap
libxml
mbstring
mysqli
mysqlnd
openssl
pcntl
pcre
PDO
pdo_mysql
PDO_ODBC
pdo_pgsql
pdo_sqlite
pgsql
Phar
posix
readline
redis
Reflection
session
SimpleXML
soap
sockets
sodium
SPL
sqlite3
standard
tokenizer
xml
xmlreader
xmlwriter
Zend OPcache
zip
zlib

[Zend Modules]
Zend OPcache
blackfire

Things done

Built on platform(s)

[X] x86_64-linux
[X] aarch64-linux
[X] x86_64-darwin
[X] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg build tests.php


Move unit tests to new jscodeshift test util

Moves all the unit tests in this project to the new jscodeshift test util I added recently. This avoids the need to use the global "test" function that's being pulled in from the react-codemod project.
Unfortunately Git doesn't seem to have picked up that the test files were both moved and edited, probably because they changed so much :/
All unit tests pass on my system, with the exception of addImportCJSBasicConfig. That test is failing for me and it looks like some line ending issue (\n vs \r\n - I'm on Windows). It's giving me output where the expected value and actual value look identical:
● addImportCJSBasicConfig › it should add to a non first statement
  - Expected: '/* @flow */

          'use strict';

          var A0 = require('A0');
          var A1 = require('A1');
          var A3 = require('A3');

          aaa;' toEqual: '/* @flow */

          'use strict';

          var A0 = require('A0');
          var A1 = require('A1');
          var A3 = require('A3');

          aaa;'

Thank you so much! Can you make the same change for https://github.com/reactjs/react-codemod as well? :)
Why did you call it defineTest? I think const test = require('jscodeshift').test would have been quite nice.

Can you make the same change for https://github.com/reactjs/react-codemod as well? :)

Yeah, I'm working on that now :+1:

Why did you call it defineTest? I think const test = require('jscodeshift').test would have been quite nice.

I wanted to differentiate between defining a test (which does the define and it) call, and executing the test (the actual test code itself). test is unclear as to which of those two it is. You can rename it if you like, though. I'm not too attached to any particular name.  :stuck_out_tongue:


Updated markdown

Updated markdown relative to includes.

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/enterprise/connect-your-bot.md
:warning:Warning

Details

docs/enterprise/authenticate.md
:white_check_mark:Succeeded

docs/enterprise/connect-your-bot.md

Line 49: [Warning] Invalid file link:(~/enterprise/.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Integerate Tapas in PrimeQA

PrimneQA will support TableQA through TAPAS

PR raised for TAPAS integration with run_tableqa.py entry point. Code review/modifications in progress.


Fix gorethink

Moved from docker fork (based on v3.0.0 + sirupsen cap patch), back to upstream (on v3.0.5).

no code change in notary itself, except dancannon -> gorethink on imports
Vendor changes:
updated from cenk to cenkalti
removed dep on gorethink.v2 (no longer needed)

Signed-off-by: Olivier Gambier<EMAIL_ADDRESS>

CI failure seems unrelated (fails on master as well). @endophage do you now why it's failing?


feat: adds new method for bulk purchasing

References CAR-6428

@masone wasnt sure in what section of README.MD to document this new method. The existing (singular) one wasn't documented.


Add tests for failure cases

What problem does your feature solve?
Right now we are only testing the happy path.
What would you like to see?
We should make sure the failure cases actually fail as expected.
What alternatives are there?
None

Resolved by #38


Fix purge step

Getting the action output was still using the juju2 api method.
Updating it must have been missed in #444 (add the purge step)
after it was updated with main after #451 (port to juju3) was merged.
Fixes: #488

superceded by #490


0098 split data granularities

Heuristics for handling dual parsing modes before/after 2020-03-10. Notebooks are updated but unit tests are now broken. US counties are no longer being generated by CSSE so I am not emitting them to JSON.
Haven't tested this in the full pipeline because it'll have an impact on S3 if I do. @pr3d4t0r would you mind doing some quick validation, and let me know if anything is broken?

Going to put the boats into a separate JSON and resubmit.


fix : job_run_input call from airflow worker

fixes instance name in job_run_input call in airflow worker
fixes airflow restart issue in dev

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 78.541%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7997

Relevant Lines:
10182

💛  - Coveralls


How to determine a profit or a loss?

Hello
I can't seem to find any method that shows you whether you have made a profit or a lose from your last trade, or even if that trade has been concluded
Can anyone point me in the right direction with some working code examples?
Thank you in advance

I was unable to find proper solution for this and that is why I made it in this way:
def make_bid(pair, direction):     threadUpdater = Thread(target=update_reports)     threadUpdater.start()     ... (making bid)
def update_reports():     time.sleep(2 * 60)     for k, item in iqoption.portfolio.items():         if item.win == "equal":             continue         ... (handle "k" as bid result)
This example ignores cases when result is really "equal", but it is fine for me.
It is not a production solution

I was unable to find proper solution for this and that is why I made it in this way:
def make_bid(pair, direction):
    threadUpdater = Thread(target=update_reports)
    threadUpdater.start()
    ... (making bid)

def update_reports():
    time.sleep(2 * 60)
    for k, item in iqoption.portfolio.items():
        if item.win == "equal":
            continue
        ... (handle "k" as bid result)

This example ignores cases when result is really "equal", but it is fine for me.
It is not a production solution

@djumpen @harwee
I've encountered a new problem
What do you do if Portfolio is never populated?
dict_items([])
In such a situation it is impossible to determine a win, lose or break even.
Is there any other way to determine if a bid has ended and the result?


Max Angle Delta not working when using Anti-Aim

How to fix this? @danielkrupinski

89 and -89 max of pitch angles


Update README.md

Signed-off-by: StarNumber<EMAIL_ADDRESS>

yes

fixed in the last commit

I resolved all issues, waiting for mantainers review...

imagine not caring about a PR

Ik, but last review suggested changes, which I did

imagine not caring about a PR

Sorry?


Unsure of Usage of amplifyPush Script

** Please describe which feature you have a question about? **
For normal environments from branches that I connect in the Amplify Console via the web GUI, we have amplifyPush --simple at the beginning of our amplify.yml. That's all good.
Now, we have an environment that we need to programmatically create using the AWS CLI create-branch and start-job commands. In this scenario, amplifyPush --simple fails with this output:
usage: amplify-push <--environment|-e <name>> <--simple|-s>
--environment  The name of the Amplify environment to use
--simple  Optional simple flag auto-includes stack info from env cache

** Provide additional details**
I can see from the source code that this happens when the $ENV variable is not valid. We already have an existing backend that we would like to use with this env called stg.
I tried some other approaches, thinking maybe I need to explicitly state which backend I want this build to use: amplifyPush --environment stg --simple but I am still getting the same output.
Is there anyone on the team that can tell me what this roadblock that I am hitting means? Why is the script failing when we use the AWS CLI to "provision" the branch and job? Is what I am trying to do impossible with the current offering that Amplify provides?
For reference, I am also setting the backend ARN for stg in the create-branch step to make sure I associate this branch with the correct backend.

Hi @jhoffmcd can you please share your amplify.yml file.

Sure, in the below example, you can see I commented out the amplifyPush command that did not work:
amplifyPush -e stg -s
version: 0.1

backend:
  phases:
    build:
      commands:
        - '# Execute Amplify CLI with the helper script'
        # - if [[ ! "${AWS_BRANCH}" =~ ^release\/.* ]]; then amplifyPush -e stg -s; else amplifyPush -s; fi
        - amplifyPush -s;

frontend:
  phases:
    preBuild:
      commands:
        - echo Entered the pre-build phase...
        - nvm use $VERSION_NODE_12
        - yarn install
        - echo Linting code...
        - yarn lint
        - echo Starting test run...
        - yarn test --ci
        - echo pre-build phase complete.
    build:
      commands:
        - echo Entered the build phase...
        - yarn run build
        - echo build phase complete.
    postBuild:
      # Note: If we see a need to add tagging we can do it in post build, however we will need a way to push the tag
      commands:
        - echo Entered the post-build phase...
        # Run post build script for staging env (checks if it's in a release branch context)
        - bash ci-build-scripts/post-build-staging.sh
        - echo post-build phase complete.
  artifacts:
    baseDirectory: public
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*

I did some testing locally, I believe that what is happening is that the -e stg I am passing to the script is getting overridden by USER_BRANCH environment variable. This would definitely result in an error because there would not be an environment to match a branch-like release/1.5.0.
I believe if we pass something to this script via -e then that should override all other ENV checks, otherwise what is the point of making this argument available? I can submit a PR with an idea of what I am talking about.

This is now addressed: https://docs.aws.amazon.com/cli/latest/reference/amplify/create-branch.html. You can point to a specific backend environment to link it to your branch using the [--backend-environment-arn <value>]


Branch for easy rebase to master

Please describe the feature you'd like to see added.
Maybe I'm missing some Git magic — please teach me if so.
I think it would be great to have a clean branch (i.e. without stable-branch-related commits like bitcoin/bitcoin@fccd32efe6e2950b2c74fdec2ade54040ca32a2c) for ease rebase to Bitcoin master (including the disabling of mainnet and testnet).
Is your feature related to a problem, if so please describe it.
No response
Describe the solution you'd like
With inq as this repo's git remote and origin as bitcoin/bitcoin, one can easily do following:
$ git checkout v27.0-inq
$ git switch -c myinq # git checkout -B myinq
$ git rebase origin/27.x

Describe any alternatives you've considered
Rebase to origin/master of bitcoin/bitcoin repo fails.
Please leave any additional context
$ git remote -v show
inq	https://github.com/bitcoin-inquisition/bitcoin (fetch)
inq	https://github.com/bitcoin-inquisition/bitcoin (push)
origin	https://github.com/bitcoin/bitcoin (fetch)
origin	https://github.com/bitcoin/bitcoin (push)

Thanks.


Add onContextMenu

Clear and concise description of the problem
I just wanna have control over right clicks on slots.
Suggested solution
onContextMenu={(e) => onSelect && onSelect(event, e)}
Alternative
Also, there is another option to add a custom event handler onContextMenu at the top of the components
Additional context
No response
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Would you like to open a PR for this feature?

[X] I'm willing to open a PR

i was able to use onContext menu by customising dateCellWrapper and timeSlotWrapper

Can you add an example how?

timeSlotWrapper: (timeSlotWrapperProps) => (

and SchedularCell looks loke this :
 {
                if (
                    scRef.current?.props?.className !==
                    localRef.current?.props?.className
                )
                    scRef.current?.hide();
                scRef.current = localRef.current;
                localRef.current?.show(e);
            }}
            className={
                selected.start <= WrapperProps?.value &&
                WrapperProps?.value < selected.end
                    ? 'schedular-cell selected-event'
                    : 'schedular-cell'
            }
            onClick={(e) => {}}
        >

            {WrapperProps.children}

i used  with dateCellWrapper too ,
its working like a charm,hope this guide you a little


To set clusterwide istio-init and istio-proxy image to be pulled from private repo.

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
As per our project architecture we need to have the istio sidecar image that is istio-proxy container image to be stored in a private ecr repo and the istio-proxy container should pull the image from the private repo. We have already downloaded the relevant docker image and stored in our ECR repo and we are using "pull through cache configuration".
We have achieved this configuration in our istiod helm chart and can see the "discovery" container is pulling from private ecr repo. Sample terraform codebase for istiod
resource "helm_release" "istiod" {
repository       = "https://istio-release.storage.googleapis.com/charts"
chart            = "istiod"
name             = "istiod"
namespace        = "istio-system"
create_namespace = false
depends_on       = [helm_release.istio_base]
set {
name  = "meshConfig.outboundTrafficPolicy.mode"
value = "REGISTRY_ONLY"
}
set {
name  = "meshConfig.accessLogFile"
value = "/dev/stdout"
}
set {
name  = "pilot.nodeSelector.NodeType"
value = "${local.name_prefix}-k8s_static_ng"
}
set {
name  = "pilot.hub"
value = "xxxxx.dkr.ecr.xxxx.amazonaws.com/docker-hub/istio"
}
}
But this setting is not applied clusterwide/globally the istio-egressgateway and the istio-proxy sidecar container is still trying to download the image from docker.io/istio/proxyv2:1.22.0.
1. Please let us know how to do the setting clusterwide so that istio-init and istio-proxy containers will pull images from private repo.
2.  Followed this - https://istio.io/latest/blog/2020/docker-rate-limit/?_ga=2.10153846.252119387.1717407231-889315782.1709888551 but nothing is mentioned clearly on where to apply.
Base chart - https://github.com/istio/istio/blob/release-1.21/manifests/charts/base/Chart.yaml
istiod chart - https://github.com/istio/istio/blob/release-1.21/manifests/charts/istio-control/istio-discovery/Chart.yaml
egress gateway chart - https://github.com/istio/istio/blob/release-1.21/manifests/charts/gateway/Chart.yaml
Version
Kubernetes Version - Server Version: v1.28.9-eks-036c24b
Istio Version - 1.22.0

Additional Information
No response

Solved the issue by setting global.hub instead of pilot.hub
set {
name  = "global.hub"
value = "xxxxx.dkr.ecr.eu-west-1.amazonaws.com/docker-hub/istio"
}

closing the issue


Opening an object

Description
What steps will reproduce the problem?

Using the following code:
import pyvista as pv

# Create source to ray trace
sphere = pv.Sphere(radius=0.85)

# Define line segment
start = [-0.5, -0.5, -0.5]
stop = [0.5, 3, 0.5]

# Perform ray trace
points, ind = sphere.ray_trace(start, stop)

# Create geometry to represent ray trace
ray = pv.Line(start, stop)
intersection = pv.PolyData(points)

# Render the result
p = pv.Plotter()
p.add_mesh(sphere, show_edges=True, opacity=0.5, color="w", lighting=False, label="Test Mesh")
p.add_mesh(ray, color="blue", line_width=5, label="Ray Segment")
p.add_mesh(intersection, color="maroon", point_size=25, label="Intersection Points")
p.add_legend()
p.show()

Everytime that I then open one of the variables such as "ray" or "intersection", I get a error from Spyder. The error says "Spyder has encountered an internal problem". Eventhough the problem is there, the ojects open and I can still work with it.
Traceback
ERROR:root:No data to measure...!

 I was able to reproduce this :+1::

Seems like the error dialog is shown due to an error triggered from vtk:
2024-07-23 16:27:05.819 (   1.395s) [7CF0B703D051AD1E]            vtkMath.cxx:778   WARN| vtkMath::Jacobi: Error extracting eigenfunctions
2024-07-23 16:27:05.821 (   1.396s) [7CF0B703D051AD1E]  vtkMassProperties.cxx:60     ERR| vtkMassProperties (000001AA87B82370): No data to measure...!

Just in case, as a per session a workaround, you could prevent the error dialog from appearing by checking the Hide all future errors during this session checkbox. Also, I think those kind of variables are currently unsupported over Spyder so clicking the Exclude unsupported data types (over the Variable Explorer options menu) should hide them (to prevent trying to interact with them).

Pinging @larsoner since I saw he works in PyVista. Do you have any advice for us on how we could handle this error?

For me locally running the top-comment code in regular Python things are fine -- the plot renders in a window, etc. I can at least get the __repr__s of the objects:
$ python -ui ~/Desktop/rep_pv.py
>>> ray
PolyData (0x732f15706b00)
  N Cells:    1
  N Points:   2
  N Strips:   0
  X Bounds:   -5.000e-01, 5.000e-01
  Y Bounds:   -5.000e-01, 3.000e+00
  Z Bounds:   -5.000e-01, 5.000e-01
  N Arrays:   2
>>> intersection
PolyData (0x732f22f9c400)
  N Cells:    2
  N Points:   2
  N Strips:   0
  X Bounds:   -4.938e-01, -1.202e-01
  Y Bounds:   -4.783e-01, 8.292e-01
  Z Bounds:   -4.938e-01, -1.202e-01
  N Arrays:   0

Then doing
$ conda create -n spyder spyder pyvista
$ conda deactivate && conda activate spyder
$ spyder

I can reproduce the ERROR:root:No data to measure...! error as well as this in the console I used to launch Spyder:
2024-07-24 11:57:20.133 (   0.587s) [    7CB6B25EF740]            vtkMath.cxx:778   WARN| vtkMath::Jacobi: Error extracting eigenfunctions
2024-07-24 11:57:20.134 (   0.588s) [    7CB6B25EF740]  vtkMassProperties.cxx:60     ERR| vtkMassProperties (0x572dc14a8c20): No data to measure...!

So it's something about how Spyder inspects the objects I think. One potential hint/idea comes from when I try to inspect the Plotter by double-clicking on it:

If there is some pickling step involved in the inspection (?), then I'm not too surprised it fails. I'm not sure which if any vtk objects (which wrap underlying C++ objects) can be safely pickled and unpickled.

If there is some pickling step involved in the inspection (?), then I'm not too surprised it fails. I'm not sure which if any vtk objects (which wrap underlying C++ objects) can be safely pickled and unpickled.

Yep, that's exactly what we do: we pickle objects in the kernel and then unpickled them in Spyder so that users can inspect them through the Variable Explorer.
So, I guess we need to catch this error and show an appropriate message to fix it.


[UI] Bifurcation of Actions button (model-click vs single-click)

Current Behavior
@dragon-slayer875 please verify whether this is a duplicate or not.
Desired Behavior

Implementation

Acceptance Tests

Mockups

Contributor Guides and Handbook

🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

I'm interested in working on this issue.

@dragon-slayer875 confirm please is this was completed?

@dragon-slayer875 is this complete?


Extension fails on WebSocket authentication in JupyterHub

A really compelling use for this extension is in a multi-user environment like JupyterHub where users can submit notebook cells for background execution. Additionally, this extension can improve UX in cases of poor network access as computation is not interrupted. Unfortunately, as it currently exists this extension does not work in JupyterHub.
A WSServerHandshakeError is raised on a POST request to the execute endpoint. I have tracked down the problem, which is that in JupyterHub token authentication is broken - self.token is not populated with the token string. Whereas this extension works as expected in JupyterLab. The full error:
[E 2022-12-19 11:34:55.342 SingleUserLabApp web:1798] Uncaught exception POST /user/matt/api/kernels/0c2afc52-b7fb-4a06-b8f2-f1c85167dd41/execute (::ffff:<IP_ADDRESS>)
    HTTPServerRequest(protocol='http', host='localhost:8000', method='POST', uri='/user/matt/api/kernels/0c2afc52-b7fb-4a06-b8f2-f1c85167dd41/execute', version='HTTP/1.1', remote_ip='::ffff:<IP_ADDRESS>')
    Traceback (most recent call last):
      File "/opt/miniconda3/envs/py39/lib/python3.9/site-packages/tornado/web.py", line 1713, in _execute
        result = await result
      File "/opt/miniconda3/envs/py39/lib/python3.9/site-packages/jupyter_kernel_executor/handlers.py", line 138, in post
        await self.execute(client, code, document_id, cell_id)
      File "/opt/miniconda3/envs/py39/lib/python3.9/site-packages/jupyter_kernel_executor/handlers.py", line 153, in execute
        result = await client.execute(code)
      File "/Users/matt/github/jupyter_kernel_client/jupyter_kernel_client/client.py", line 109, in execute
        async with session.ws_connect(
      File "/opt/miniconda3/envs/py39/lib/python3.9/site-packages/aiohttp/client.py", line 1141, in __aenter__
        self._resp = await self._coro
      File "/opt/miniconda3/envs/py39/lib/python3.9/site-packages/aiohttp/client.py", line 795, in _ws_connect
        raise WSServerHandshakeError(
    aiohttp.client_exceptions.WSServerHandshakeError: 403, message='Invalid response status', url=URL('ws://<IP_ADDRESS>:57936/user/matt/api/kernels/0c2afc52-b7fb-4a06-b8f2-f1c85167dd41/channels?session_id=00164db0-f768-498b-a60c-725058e7bf05')

My hacky fix is to extract the token from the Authorization header using Tornado. On line 109 of handlers.py where we instantiate the KernelWebsocketClient, change it to:
auth_header = self.request.headers.get('Authorization')
request_token = auth_header.split(' ')[1]

client = KernelWebsocketClient(
    kernel_id = kernel_id,
    host = self.serverapp.ip,
    port = self.serverapp.port,
    base_url = self.base_url,
    token = request_token,
    encoded = True,
)

Which fixes this bug in JupyterHub without affecting JupyterLab.
I'm still familiarizing myself with Jupyter's backend, and so am not sure what the best practice is. Do you have any suggestions on how to resolve this the Jupyter way?

@matthewwiese Thank you for your feedback
Jupyterhub uses a minxin class to create a compatible Application class with JupyterHub authentication mixins enabled,
called SingleuserApp: https://github.com/jupyterhub/jupyterhub/tree/main/jupyterhub/singleuser
I think we can use some authentication options in the request, such as cookies, Authorization header, and use self.token as a fallback option
This is already provided in the client as auth_header: https://github.com/Wh1isper/jupyter_kernel_client/blob/master/jupyter_kernel_client/client.py#L61
What we need to do is something like
auth_header = self.get_request_auth(self.request) # -> Dict() or dict with auth info
if not auth_header:
  auth_header = self.get_auth_header() # {Authorization:f"token {self.token}"}
client = KernelWebsocketClient(
    kernel_id = kernel_id,
    host = self.serverapp.ip,
    port = self.serverapp.port,
    base_url = self.base_url,
    auth_header = auth_header,
    encoded = True,
)

Could you open a pr for it? Or I may implement it about this week

I open a pr for it https://github.com/Wh1isper/jupyter_kernel_executor/pull/10
Can you help me try to see if this branch works? I don't have a JupyterHub related environment @matthewwiese
https://github.com/Wh1isper/jupyter_kernel_executor/pull/10

fixed in https://github.com/Wh1isper/jupyter_kernel_executor/pull/10


update pyo3, add 3.10 wheels to release, 1.2.1

Closes https://github.com/aviramha/ormsgpack/issues/79


Fix wrong class name for medium warning

The element gets generated with the class warn-med but the style uses warn-medium leading to a white coloring instead of the supposed orange one. This fixes that by using the correct class name in the style.

Is this not on the site yet or is my browser just really wanting to cache that style? :S

i had forgot to deploy it, done now.


window wrong size when opening second window on master layout

Hyprland Version
Hyprland, built from branch main at commit 739598717bbff79aad8599dcd0ca7a423682482a dirty (remove shadow no_gaps_when_only (2956)). Tag: v0.28.0-36-g73959871
Bug or Regression?
Bug
Description
This only happens on master layout and when no_gaps_when_only = 1 and gaps are present.
When an existing window is open, opening a second window causes the first window to be incorrectly sized. The original window seems to have the border width added to its dimensions.
This happens with most applications I've tested with, for example qutebrowser, chromium, firefox and foot all exhibit this. It does not happen if the second window is kitty (regardless of what the other window is).
The problem is corrected if I switch workspaces, open a third window or, as I just discovered, take a screenshot with grim.
I'm attaching a screenshot where I've set gaps and borders big to make the problem more obvious.
I hope this makes sense, it's a little tricky to explain.
I'll gladly provide more info if required, there's nothing exotic in my configs, and I'm using hyprland-git from the AUR.
How to reproduce

use master layout
set no_gaps_when_only = 1
set gaps and borders to something chunky to make the problem obvious
open (for example) foot twice

Crash reports, logs, images, videos

cannot repro with foot, are you by any chance setting a border workspace rule?

Okay, thanks. I just tried with a default config with just the changes I mentioned and it doesn't happen. So it must be something else in my config. I had assumed not, since I'm not doing anything weird.
Thanks for your help, I'll find the culprit then close the issue.

Right, it happens on this config for me.
The only things I've changed from default are:

changed layout to master
increased border size to 10 to make it more noticeable
added no_gaps_when_only = 1 inside master layout
changed default terminal keybinding from kitty to foot

Can you confirm that it doesn't exhibit on this config for you?
hyprland.conf.txt

my bad was setting no_gaps in dwindle and master was using no_gaps = 2

should be fixed https://github.com/MightyPlaza/Hyprland/tree/border-improv
can you test it?

should be fixed https://github.com/MightyPlaza/Hyprland/tree/border-improv
can you test it?

Yup, that fixes it. That's great, thank you!


Inline private modifier of delegate should be ok (instead => Style/AccessModifierDeclarations)

Inline private modifier of delegate is necessary for the resulting method to actually be private. But rubocop
complains with [Correctable] Style/AccessModifierDeclarations: private should not be inlined in method definitions.
Consider the following code:
# person.rb

# frozen_string_literal: true

class Person
  def initialize(firstname:)
    @firstname = firstname
  end

  attr_reader :firstname
end

# greeter.rb

# frozen_string_literal: true

class Greeter
  def initialize(person:)
    @person = person
  end

  def call
    "Hello, #{firstname}!"
  end

  private

  attr_reader :person
  private(*delegate(:firstname, to: :person))
end

we get the following behavior
[1] pry(main)> ann = Person.new(firstname: 'Ann')
=> #<Person:0x00007f5f550e9e60 @firstname="Ann">
[2] pry(main)> greeter = Greeter.new(person: ann)
=> #<Greeter:0x00007f5f564299d0 @person=#<Person:0x00007f5f550e9e60 @firstname="Ann">>
[3] pry(main)> greeter.call
=> "Hello, Ann!"
[4] pry(main)> greeter.firstname
NoMethodError: private method `firstname' called for an instance of Greeter
from (pry):4:in `__pry__'

which is the desired behavior.
If we make the suggested change with
class Greeter
  def initialize(person:)
    @person = person
  end

  def call
    "Hello, #{firstname}!"
  end

  private

  attr_reader :person
  delegate(:firstname, to: :person)
end

the firstname method can be accessed which is not the intention.
Expected behavior
Rubocop should not raise a Style/AccessModifierDeclarations for inline private modifier of delegation using delegate and autocorrection should not alter the inline privacy modifier.
Actual behavior
Rubocop raises a Style/AccessModifierDeclarations violation and -a autocorrection will remove the private modifier in a way that does not preserve the privacy of the resulting method.
See below for the full output
Steps to reproduce the problem
See code above for a functional example, as for my configuration:
require:
  - rubocop-capybara
  - rubocop-minitest
  - rubocop-performance
  - rubocop-rails

AllCops:
  TargetRubyVersion: 3.3
  TargetRailsVersion: 7.1
  NewCops: enable

RuboCop version
$ [bundle exec] rubocop -V
1.65.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.31.3, running on ruby 3.3.4) [x86_64-linux]
  - rubocop-capybara 2.21.0
  - rubocop-minitest 0.35.1
  - rubocop-performance 1.21.1
  - rubocop-rails 2.25.1

Full output
root@ab222e695eb2:/some_app# be rubocop app/poros/greeter.rb --debug
For /some_app: configuration from /some_app/.rubocop.yml
configuration from /usr/local/bundle/gems/rubocop-capybara-2.21.0/config/default.yml
Default configuration from /usr/local/bundle/gems/rubocop-1.65.0/config/default.yml
configuration from /usr/local/bundle/gems/rubocop-capybara-2.21.0/lib/../config/default.yml
configuration from /usr/local/bundle/gems/rubocop-minitest-0.35.1/config/default.yml
configuration from /usr/local/bundle/gems/rubocop-minitest-0.35.1/config/default.yml
configuration from /usr/local/bundle/gems/rubocop-performance-1.21.1/config/default.yml
configuration from /usr/local/bundle/gems/rubocop-performance-1.21.1/config/default.yml
configuration from /usr/local/bundle/gems/rubocop-rails-2.25.1/config/default.yml
configuration from /usr/local/bundle/gems/rubocop-rails-2.25.1/config/default.yml
Use parallel by default.
Skipping parallel inspection: only a single file needs inspection
Inspecting 1 file
Scanning /some_app/app/poros/greeter.rb
Loading cache from /root/.cache/rubocop_cache/bf1b41c785ed7c21e82cef7ae46ec2c3de7e00d7/9741b73e32a55fa25bfd0632cfcd5496c34a0c79/9ac823e6c338faf8d6d0a914f7282a9aed8e102b
C

Offenses:

app/poros/greeter.rb:15:3: C: [Correctable] Style/AccessModifierDeclarations: private should not be inlined in method definitions.
  private(*delegate(:firstname, to: :person))
  ^^^^^^^

1 file inspected, 1 offense detected, 1 offense autocorrectable
Finished in 0.455986370001483 seconds

This is kind of an extension of #13213, I think splats should be accepted on methods as well.
FWIW, if you're using Rails 6+ (see https://www.bigbinary.com/blog/rails-6-adds-private-option-to-delegate-method) you could change your delegate to
delegate(:firstname, to: :person, private: true)


Poseidon2 Custom Matrix (not MDS)

To address #134

I'm traveling soon, could someone else take the review? cc @jtguibas @vivekvpandya @jorgeantonio21

Good idea! Added a test, currently debugging why it fails


The build is breaking markdown links

Noticed some funky stuff here https://github.com/electron/electron.atom.io/commit/ee8e01298be2678cb882461a43521a1ad829bb3c

Similar to #567?

Yup. Dupe of #567.


enable query of wallets getting fees - from fee burner

currently wallets we give fees to are saved with a mapping
 mapping(address=>uint) public walletFeesInBps;
so for keeping track. only events will work.
better add an array of all wallet addresses.
will enable to query which wallets get fees and what is the fee.

leave it as is.
can add wallet address to verification script and use known addresses to query fees.


OPERA-SDS-VnV-11: Verify SDS can recover from disruptions

Split this into multiple tickets depending on what disruption is simulated

Case 1: disrupt system by killing Mozart

Start system
Run 100 jobs to completion
Create a backup
Start another 100 jobs
In the middle of the run, take down the Mozart machine
Restore the state of the system from the backup
Re-enable the timers and verify that they automatically re-run the same jobs

Case 2: disrupt by killing one or more of the SPOT workers

Case 3: Disruptions of the DAAC services for querying metadata, downloading input data, archiving output data

Close the outbound ports on whatever machine is running the query, download or upload
Then open the ports again and verify that the system can recover


pyparsing and Python 3 (in)compatibility

Trying to setup travis with a gemini install for Python 3 and getting an exception for the pyparsing module (v1.5.7) which doesn't seem to be compatible with Python 3 syntax:
    Traceback (most recent call last):
      File "<string>", line 20, in <module>
      File "/tmp/pip-build-76omn6aw/pyparsing/setup.py", line 9, in <module>
        from pyparsing import __version__ as pyparsing_version
      File "/tmp/pip-build-76omn6aw/pyparsing/pyparsing.py", line 855
        except ParseBaseException, err:
                                 ^
    SyntaxError: invalid syntax

I looked briefly in your setup.py and it seems you aim to support Python 2/3 so I thought I'd open this issue :)
Perhaps a later version of pyparsing does include Python 3 support?

we do not support python3. Only python 2.7 will work at this time.

Sorry, my bad! Cool project either way :)

No problem. Let us know if you find any issues. Support for python3 is on the distant horizon, but not impending.


1372 Add indication in User List/View for 2FA vs Password authentication.

Proposed change to #1372
NOTE: Did not see CONTRIBUTING.md but did review code of conduct. Apologize if this PR is missing something expected in your PR process/standards. Standing by to correct any shortcoming.
Added indicators if 2FA was active for users in the UserList and UserView views. In the user-list view '(2FA)' is displayed next to the user name if the user has 2FA active, otherwise it's omitted. In the user view 'Authentication: 2FA' or 'Authentication: Password' is displayed. These UI elements require 'View user permissions' scope to view or are otherwise omitted. Screenshot below:

A modification was also made to onMounted to resolve a 403 error if a user views the UserView view but does not have the 'View user permissions' scope:

This is just proposed and represent some subjective decisions on my part that I'm willing to adjust to the desires of the project maintainers. Go/javascript are not my primary languages so also open to any best-practice corrections to the proposed implementation. Thanks for your consideration and standing by for questions, direction, comment, etc.

Looked at the failures and they seem unrelated to the changes I made. If I have free time I may try to work on them actually. This seems like a good project to contribute to.

Yeah, this is because of changes I was doing related to how a secret was preserved, and it doesn't act well on PRs. I'll see if I can fix it.


Create a mapping for remote sensor id to internal.

This is an up-to-date version of #271 .
I will keep this branch updated to support those with trackers that do not have a contiguous list of sensor id's.
The reason why this is a pull request is for users with this issue to easier find this build.
.jar build as of 2023-01-06:
slimevr.zip

I don't think this should be marked as a draft, because any user finding this merge request will assume that the solution doesn't work.

we should just finish it and merge

It is finished?

Then we should merge it

Is it superceeded by #464?

#464 is a better solution if it tests out okay.

If they solve the same problem, #464 is better. Closing this.


Unable to get docs from database in CouchDB

System information:

Operating system (distribution) and version: Windows 10 Pro 20H2
DBeaver version <IP_ADDRESS>103080721

Connection specification:

Database name and version - CouchDB 1.6.1
Driver name CouchDB
Do you use tunnels or proxies (SSH, SOCKS, etc) - no

Describe the problem you're observing:
I coudn`t get list of docs from CouchDB, but I see list of DBs

Is it possible to send log files completely https://github.com/dbeaver/dbeaver/wiki/Log-files?
Do you try to open a database via DB Navigator to see docs and receive an error?

couch.log
I tried throught main window


Troubleshooting on demo rpc app startup

Adding notes on some additional troubleshooting docs that can possibly be added from trying out steps here
https://github.com/bcgov/traction/tree/main/demo/plugins/rpc
Issue with .venv
On startup of the devcontainer on my machine it didn't seem to be creating venv.
So documented step of source .venv/bin/activate was erroring

had to create a .venv

then it worked
also to start up debugging had to set a python intepreter in vscode

command pallette (ctrl shift v) > Select Interpreter

Enter interpreter path
find
select .venv that was created

Copy Sample env to .env

Documentation note to create an .env file
Check default Traction ports in the sample file

Resolved by #1155


Users can find out what we are working on

What
We want to make it clearer to our users what the team is working on, the progress of items in the community backlog and the roadmap.
Why
Users have told us that they don't know what we are working on. Some users know about our sprint board and roadmap board on Github, but don't frequency visit and many don't know it exist at all.
Users (and the team) have also told us that the community backlog is particularly difficult to see progress and it looks very stagnant. The board view isn't easy to find.
Who needs to know about this
Team and the community
Further detail
Done when

[ ] ...

Previous body of issue:
What
We want to make it clearer to our users what the team is working on, the purpose of the community backlog, progress of items in the community backlog and the roadmap.
Why
Users have told us that they don't know what we are working on. Some users know about our sprint board and roadmap board on Github, but don't frequency visit and many don't know it exist at all.
Users (and the team) have also told us that the community backlog is particularly difficult to see progress and it looks very stagnant. The board view isn't easy to find.
Who needs to know about this
Team and the community


uncounted people stacking up

Hi i have managed to set up the system and it counts but i'm using it to know how many people are there in my apartment at any given time. I have counted every people at my apartment at the begining and started my database from 52. But there are some issues system doesn't count perfectly there are some people uncounted. This won't be a problem for a mall or a bussines place since you can reset the people in it at night when there is nobody at the building. But at an apartment you can't know the people in at any moment. I want to know if these errors will add up in time and affect the result.
If it affects the results how can i prevent it is there any suggestions ? I don't want to see some ridiculus numbers when i check my database 1 year later and i don't want to constantly count people manually once a month or something.

Hi,
Since computer vision isn’t 100% accurate, the counter will make mistakes. We found it accurate to about 7% (i.e. if it reports 100 people coming in, the true figure varies between 93-107).
Unfortunately we’ve found that it does tend to bias its mistakes in a particular direction, since lighting/shadows are the thing that most confuses it. Eg if a person walking in has a shadow ahead of them, sometimes the counter will think the shadow is a person going out. Similarly, the shadow of a person walking out can also be counted as a second person going out. This may be because we have our camera (and hence shadows) on the stairs - on flat ground results may be different.
If you pay extremely careful attention to placement, and experiment to see what position/lighting gives the best results, you can probably get it more accurate. It’ll never be perfect though - this software is meant to give a rough comparison of numbers and peak hours, not be accurate to the individual level.
All that said, we’re working on improving the image detection whenever we have a moment spare, so if you spot a new version do try it out.
Hope that explains things.
Oliver.


Fix Swagger schema API endpoint & add a test for it

Description

Checklist:

[ ] I've made sure that tests are updated accordingly (especially if adding or updating a template option)
[ ] I've updated the documentation or confirm that my change doesn't require any updates

Rationale

Currently, after a new project generation with use_drf: y, the API call to the schema endpoint fails with the error
AssertionError: Incompatible AutoSchema used on View <class 'drf_spectacular.views.SpectacularSwaggerView'>. Is DRF's DEFAULT_SCHEMA_CLASS pointing to "drf_spectacular.openapi.AutoSchema" or any other drf-spectacular compatible AutoSchema?

So, the change sets the default schema class as drf_spectacular.openapi.AutoSchema as per drf-cpectacular docs and adds a test to make sure sure that schema generation works properly

Changes done, thanks for the feedback @browniebroke

Thanks again for the fix 🎉


Fix links to packages documentation

Codecov Report

Merging #19315 (7ac2a0f) into main (d242511) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main   #19315      +/-   ##
==========================================
- Coverage   47.51%   47.49%   -0.02%
==========================================
  Files         944      944
  Lines      131549   131549
==========================================
- Hits        62500    62479      -21
- Misses      61541    61558      +17
- Partials     7508     7512       +4

Impacted Files
Coverage Δ

modules/process/manager_exec.go
85.36% <0.00%> (-7.32%)
:arrow_down:

modules/git/repo_base_nogogit.go
70.58% <0.00%> (-3.93%)
:arrow_down:

modules/git/utils.go
66.29% <0.00%> (-3.38%)
:arrow_down:

modules/queue/queue_channel.go
78.70% <0.00%> (-2.78%)
:arrow_down:

models/issue_comment.go
51.35% <0.00%> (-0.68%)
:arrow_down:

routers/web/repo/view.go
39.00% <0.00%> (-0.50%)
:arrow_down:

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 d242511...7ac2a0f. Read the comment docs.


Typescript workspace problems not being detected

VsCode will only report typescript problems when files with the problems are open even though there is a tsconfig.json file.

VSCode Version: 1.18.1
OS Version: Arch Linux 4.13.12-1 (Kernel version)

Steps to Reproduce:

Create typescript project with two typescript files: index.ts and test.ts
export class from test.ts
import class from test into index.ts and call non-existant method on the imported class. Error appears
close index.ts error disappears

tsconfig.json contents
{
	"compilerOptions": {
			"target": "es6",
			"module": "commonjs",
			"sourceMap": true,
			"rootDir": "./"
	},
	"include": [
		"**/*.ts"
	]
}

test1.ts contents
export class test {
    public static test123(): number {
        console.log('Hello World');
        return 0;
    }
}

index.ts contents
import { test } from "./test";

test.main();

Reproduces without extensions: Yes

In the gif, the tsconfig.json has the 'files' option, but I can confirm that it is still happening when using the 'include' option

duplicate of #13953


something wrong with  "answer_vectors = default_collate(_answer_vectors)" in ansemb/dataset/data_utils.py

train E000:   0% 0/3467 [00:02<?, ?it/s]
Traceback (most recent call last):
File "train_vqa_embedding.py", line 266, in
main(args)
File "train_vqa_embedding.py", line 239, in main
train(context_net, answer_net, train_loader, optimizer, tracker, args, prefix='train', epoch=i)
File "train_vqa_embedding.py", line 108, in train
for v, q, avocab, a, labels, idx, q_len in tq:
File "/home/anaconda3/lib/python3.6/site-packages/tqdm/std.py", line 1130, in iter
for obj in iterable:
File "/home/anaconda3/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 345, in next
data = self._next_data()
File "/home/anaconda3/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 856, in _next_data
return self._process_data(data)
File "/home/anaconda3/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 881, in _process_data
data.reraise()
File "/home/anaconda3/lib/python3.6/site-packages/torch/_utils.py", line 394, in reraise
raise self.exc_type(msg)
TypeError: Caught TypeError in DataLoader worker process 0.
Original Traceback (most recent call last):
File "/home/anaconda3/lib/python3.6/site-packages/torch/utils/data/_utils/worker.py", line 178, in _worker_loop
data = fetcher.fetch(index)
File "/home/anaconda3/lib/python3.6/site-packages/torch/utils/data/_utils/fetch.py", line 47, in fetch
return self.collate_fn(data)
File "/home/answer_embedding-master/ansemb/dataset/data_utils.py", line 95, in collate_fn
answer_vectors = default_collate(_answer_vectors)
File "/home/anaconda3/lib/python3.6/site-packages/torch/utils/data/_utils/collate.py", line 55, in default_collate
return torch.stack(batch, 0, out=out)
TypeError: stack(): argument 'tensors' (position 1) must be tuple of Tensors, not Tensor

Hi,
It seems that the variable ``batch'' here is not a tuple (or a list) of tensors.
I think the easiest way to debug this is to set the number of workers to be 0 and then trace the line#95 of collate_fn  in data_uils.py to see what kind of data structure this ``_anwer_vectors'' is.
Best,

Thank you so much.
After seting the number of workers to be 0, some errors really disappear but some remain the same.
like:
_train E000:   0% 0/3467 [00:02<?, ?it/s]
Traceback (most recent call last):
File "train_vqa_embedding.py", line 266, in
main(args)
File "train_vqa_embedding.py", line 239, in main
train(context_net, answer_net, train_loader, optimizer, tracker, args, prefix='train', epoch=i)
File "train_vqa_embedding.py", line 108, in train
for v, q, avocab, a, labels, idx, q_len in tq:
File "/home/chenzhuo/anaconda3/lib/python3.6/site-packages/tqdm/std.py", line 1130, in iter
for obj in iterable:
File "/home/chenzhuo/anaconda3/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 345, in next
data = self._next_data()
File "/home/chenzhuo/anaconda3/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 385, in _next_data
data = self._dataset_fetcher.fetch(index)  # may raise StopIteration
File "/home/chenzhuo/anaconda3/lib/python3.6/site-packages/torch/utils/data/_utils/fetch.py", line 47, in fetch
return self.collate_fn(data)
File "/home/chenzhuo/answer_embedding-master/ansemb/dataset/data_utils.py", line 95, in collate_fn
answer_vectors = default_collate(_answer_vectors)
File "/home/chenzhuo/anaconda3/lib/python3.6/site-packages/torch/utils/data/utils/collate.py", line 55, in default_collate
return torch.stack(batch, 0, out=out)
TypeError: stack(): argument 'tensors' (position 1) must be tuple of Tensors, not Tensor
And I use
print(" data structure of _answer_vectors:", type(_answer_vectors))
to  see the data structure of this ``_anwer_vectors''  before
answer_vectors = default_collate(_answer_vectors)
it print that : _data structure of answer_vectors: <class 'torch.Tensor'>

this is the printed text before:
@XXX:~/answer_embedding-master$ python train_vqa_embedding.py --gpu_id 1
{'gpu_id': 1, 'finetune': False, 'batch_size': 128, 'max_negative_answer': 12000, 'answer_batch_size': 3000, 'loss_temperature': 0.01, 'pretrained_model': None, 'context_embedding': 'SAN', 'answer_embedding': 'BoW', 'name': None}
{'cache_path': '/home/answer_embedding-master/.cache', 'output_path': '/home/answer_embedding-master/outputs', 'embedding_size': 1024, 'seed': 1618, 'question_vocab_path': '/home/answer_embedding-master/data/question.vocab.json', 'image_size': 448, 'output_size': 14, 'preprocess_batch_size': 100, 'output_features': 2048, 'central_fraction': 0.875, 'TRAIN': {'epochs': 50, 'batch_size': 128, 'base_lr': 0.001, 'lr_decay': 15, 'data_workers': 0, 'answer_batch_size': 3000, 'max_negative_answer': 8000}, 'TEST': {'max_answer_index': 3000}, 'VQA2': {'qa_path': '/home/answer_embedding-master/data/vqa2', 'feature_path': '/home/answer_embedding-master/features/vqa-resnet-14x14.h5', 'answer_vocab_path': '/home/answer_embedding-master/data/answer.vocab.vqa.json', 'train_img_path': '/home/answer_embedding-master/data/vqa2/images/train2014', 'val_img_path': '/home/answer_embedding-master/data/vqa2/images/val2014', 'test_img_path': '/home/answer_embedding-master/data/vqa2/images/test-dev2015', 'train_qa': 'train2014', 'val_qa': 'val2014', 'test_qa': 'test-dev2015', 'task': 'OpenEnded', 'dataset': 'mscoco'}, 'VG': {'qa_path': '/home/answer_embedding-master/data/vg', 'feature_path': '/home/answer_embedding-master/features/vg-resnet-14x14.h5', 'answer_vocab_path': '/home/answer_embedding-master/data/answer.vocab.vg.json', 'train_qa': 'VG_train_decoys.json', 'val_qa': 'VG_val_decoys.json', 'test_qa': 'VG_test_decoys.json', 'img_path': '/home/answer_embedding-master/data/vg/images'}, 'Visual7W': {'qa_path': '/home/answer_embedding-master/data/v7w', 'feature_path': '/home/answer_embedding-master/features/vg-resnet-14x14.h5', 'answer_vocab_path': '/home/answer_embedding-master/data/answer.vocab.v7w.json', 'train_qa': 'v7w_train_questions.json', 'val_qa': 'v7w_val_questions.json', 'test_qa': 'v7w_test_questions.json', 'train_v7w_decoys': 'v7w_train_decoys.json', 'val_v7w_decoys': 'v7w_val_decoys.json', 'test_v7w_decoys': 'v7w_test_decoys.json', 'img_path': '/home/answer_embedding-master/data/v7w/images'}}
Output data would be saved to /home/answer_embedding-master/outputs/SAN_BoW_vqa_batch_softmax_embedding_2020-08-25_12:45:17.pth

Loading vectors to .vector_cache/glove.840B.300d.txt.pt
import answer vocabulary from: /home/answer_embedding-master/data/answer.vocab.vqa.json
extracting answers...
loading cache from: /home/answer_embedding-master/.cache/v2_OpenEnded_mscoco_train2014_questions.json.v2_mscoco_train2014_annotations.json.pt
import answer vocabulary from: /home/answer_embedding-master/data/answer.vocab.vqa.json
extracting answers...
loading cache from: /home/answer_embedding-master/.cache/v2_OpenEnded_mscoco_val2014_questions.json.v2_mscoco_val2014_annotations.json.pt
/home/answer_embedding-master/ansemb/models/layers.py:101: UserWarning: nn.init.xavier_uniform is now deprecated in favor of nn.init.xavier_uniform_.
init.xavier_uniform(w)
/home/answer_embedding-master/ansemb/models/embedding.py:48: UserWarning: nn.init.xavier_uniform is now deprecated in favor of nn.init.xavier_uniform_.
init.xavier_uniform(m.weight)
Context Model:
StackedAttentionEmbedding(
(embedding): Embedding(15419, 300, padding_idx=0)
(drop): Dropout(p=0.5, inplace=False)
(text): Seq2SeqRNN(
(rnn): LSTM(300, 512, batch_first=True, bidirectional=True)
)
(attention): Attention(
(v_conv): Conv2d(2048, 512, kernel_size=(1, 1), stride=(1, 1), bias=False)
(q_lin): Linear(in_features=1024, out_features=512, bias=True)
(x_conv): Conv2d(512, 2, kernel_size=(1, 1), stride=(1, 1))
(drop): Dropout(p=0.5, inplace=False)
(relu): LeakyReLU(negative_slope=0.01, inplace=True)
)
(mlp): GroupMLP(
(conv1): Conv1d(5120, 4096, kernel_size=(1,), stride=(1,))
(drop): Dropout(p=0.5, inplace=False)
(relu): LeakyReLU(negative_slope=0.01)
(conv2): Conv1d(4096, 1024, kernel_size=(1,), stride=(1,), groups=64)
)
)
Answer Model:
MLPEmbedding(
(mlp): GroupMLP(
(conv1): Conv1d(300, 4096, kernel_size=(1,), stride=(1,))
(drop): Dropout(p=0.5, inplace=False)
(relu): LeakyReLU(negative_slope=0.01)
(conv2): Conv1d(4096, 1024, kernel_size=(1,), stride=(1,), groups=64)
)
)
train E000:   0% 0/3467 [00:00<?, ?it/s] data structure of _answer_vectors: <class 'torch.Tensor'>
train E000:   0% 0/3467 [00:00<?, ?it/s]
Traceback (most recent call last):
....error text ...

And the way I used the image future file refer to vqa-resnet-14x14.h5  from How to generate "vqa-resnet-14x14.h5"?
and I modied  the preprocess-images.py from pytorch-vqa repo, I dont know whether the error is raised by this?

Hi,
I think this error is very likely due to the change of pytorch version. Unfortunately that I do not have the machine & data to re-run this experiment at this moment.
Can you see if it would work for you to change the L#95 ``answer_vectors = default_collate(_answer_vectors)'' to
answer_vectors = default_collate((_answer_vectors,))

If not, can you check the shape of ``_answer_vectors''?

data structure of _answer_vectors: **<class 'torch.Tensor'>**
shape of _answer_vectors: **torch.Size([128, 3000])**

I change the L#95 ``answer_vectors = default_collate(_answer_vectors)'' to
answer_vectors = default_collate((_answer_vectors,))
which raised new error:(But the original error is gone)
Traceback (most recent call last):
  File "train_vqa_embedding.py", line 266, in <module>
    main(args)
  File "train_vqa_embedding.py", line 239, in main
    train(context_net, answer_net, train_loader, optimizer, tracker, args, prefix='train', epoch=i)
  File "train_vqa_embedding.py", line 115, in train
    answer_var, answer_len = loader.dataset._get_answer_vectors(avocab)
  File "/home/answer_embedding-master/ansemb/dataset/base.py", line 93, in _get_answer_vectors
    vector[idx, :] = self._encode_answer_vector(self.index_to_answer[answer_id])
**KeyError: tensor(0)**

This is because the answer_id is a tensor. I think you can change it to:
vector[idx, :] = self._encode_answer_vector(self.index_to_answer[answer_id.item()])

train E000:   0% 0/3467 [00:07<?, ?it/s]
Traceback (most recent call last):
  File "train_vqa_embedding.py", line 266, in <module>
    main(args)
  File "train_vqa_embedding.py", line 239, in main
    train(context_net, answer_net, train_loader, optimizer, tracker, args, prefix='train', epoch=i)
  File "train_vqa_embedding.py", line 133, in train
    acc = utils.batch_accuracy(predicts.data, a.data).cpu()
  File "/home/answer_embedding-master/ansemb/utils.py", line 27, in batch_accuracy
    agreeing = true.gather(dim=1, index=predicted_index)
RuntimeError: invalid argument 4: Index tensor must have same dimensions as input tensor at /pytorch/aten/src/THC/generic/THCTensorScatterGather.cu:16

it seems that some error raised during the computing of acc

This one is also due to the API change of Pytorch.
You can check the shape of true'' and predicted_index'', to make sure they have the same dimensionality.
Please refer to this https://pytorch.org/docs/stable/generated/torch.gather.html.
Check some examples about how to use ``torch.gather'' on stackoverflow would be helpful for you to debug.

yes they dont  have the same dimensionality
shape of true: torch.Size([1, 128, 3000])
type of true: <class 'torch.Tensor'>
shape of predicted_index: torch.Size([128, 1])
type of predicted_index: <class 'torch.Tensor'>

It work!
after i change the
agreeing = true.gather(dim=1, index=predicted_index)
into
agreeing = true[0].gather(dim=1, index=predicted_index)
And fix up some small problem.
Thank you so much again! :)

Hi，
With the SAN context_embedding and BoW answer_embedding in vqa2 setting with default config.py,
after 5 epoch ,the result: (in fact, The accuracy  has been maintained at about 24% since the first epoch)
val E004: 100% 1675/1675 [19:46<00:00,  1.41it/s, acc=0.2413]
train E005:  30% 1027/3467 [35:23<1:24:04,  2.07s/it, acc=0.240499, loss=nan, lr=0.000794, t_data=1.44, t_encode=0.0946, t_model=0.0706]

Is this make sence? and it seems that the experiment setting is open-ended, and the size of result pool is 3000.
I have this question because it is different from the result in paper table 3. Or is it because I understood it wrong？


haddock archives

Is there a place where one can download an archive of the haddocks for stackage snapshots? If not, would there be any interest in making such a thing available? It's quite nice to browse these locally.

Great question. It's not an advertised feature, and we may elect to change the way we do things in the future, but to date, every Stackage snapshot has uploaded a bundle including all of the Haddocks. For example:
https://s3.amazonaws.com/haddock.stackage.org/lts-9.0/bundle.tar.xz

This is fantastic. Perhaps it should be an advertised feature? I imagine there might be other people who would take advantage of it.

My only hesitation is the potential for needing to change it in the future. For example, now that Stackage Server is moving over to PostgreSQL, it may be much more logical to store hashes to the contents in the database so we don't end up storing the exact same content in S3 at a dozen different keys. But even so... yeah, we can probably continue uploading these bundles.
Would you be interested in advertising it a bit? Options I can see are:

Add a link to the Stackage page for a snapshot
Put out a blog post/mailing list post
Add it to a README (not sure which one)

Maybe a link at the top level of a snapshot's docs, like here: https://www.stackage.org/lts-9.1
"this documentation is available as a compressed archive: link"

Sounds reasonable, interested in taking a stab at it?
On Tue, Aug 22, 2017 at 11:44 AM, Matthew Leon<EMAIL_ADDRESS>wrote:

Maybe a link at the top level of a snapshot's docs, like here:
https://www.stackage.org/lts-9.1
"this documentation is available as a compressed archive: link"
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/fpco/stackage/issues/2788#issuecomment-323960464, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AADBB7SFU9vXjRS0g7O8wFIQ9XSijldZks5sapUAgaJpZM4O-R81
.

interested in taking a stab at it?

sure

Closing this one off then since https://github.com/fpco/stackage-server/pull/235 got in 🔥


CLI improvements

Print nanoseconds.
cargo cli is release by default (because it often is used for timing). Use cargo debugcli to get a debug version.
Time query runtime, not including printing results!
Make our release builds really release: no debug symbols!
Expose prepared queries in the CLI. We can't hold on to them, but we can separate the execution time and preparation time, and verify that behavior is the same as non-prepared.

Before these changes, a release build:
mentat_cli: 11626392
With these changes:
mentat_cli: 9646304
Saving 2MB in debug symbols!


Updates for 2023-04-01

ARM (Control Plane) API Specification Update Pull Request
Purpose of this PR
What's the purpose of this PR? Check all that apply. This is mandatory!

[x] New API version. (Such PR should have been generated with OpenAPI Hub, per this wiki doc.)
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix swagger quality issues in S360.
[ ] Other, please clarify:

This is a new API version with a breaking change. Breaking change being: on a non-existing resource of type Microsoft.Management/managementGroup, ARM will be returning 404 instead of the current 403. Logic for change in status code is happening in ARM and not in RP. Attaching

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I have reviewed the general guidance on the spec PR review process: https://aka.ms/specprreview.
[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data-plane related specifications.
[x] I commit to follow the Breaking Change Policy.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can request review from an ARM API Review board. This has been approved by the breaking change review team as well.

ARM API changes review

If you want for the ARM team to review this PR, you must add the ARMReview label.
The automation may automatically add the ARMReview label, if appropriate.
If this happens, proceed according to guidance given in GitHub comments also added by the automation.

Breaking change review
If you have any breaking changes as defined in the Breaking Change Policy,
follow the process outlined in the High-level Breaking Change Process doc.
Getting help

For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For additional help, see https://aka.ms/azsdk/support/spectools.

https://msazure.visualstudio.com/One/_workitems/edit/23007234 is the workitem related to this breaking change. An email conversation is also attached in this - which suggests to link the discussion during Swagger PR.

https://msazure.visualstudio.com/One/_workitems/edit/23007234 is the workitem related to this breaking change. An email conversation is also attached in this - which suggests to link the discussion during Swagger PR.

Hi @ramoka178! For review efficiency consideration, when creating a new API version, it is required to place API specs of the base version in the first commit, and push new version updates into successive commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

Can you please add the previous version as the base commit. Also this PR does not seem to be using Openapi hub as per the first checked comment ?


Modifying the map settings

Hi,
is it possible to modify the map settings and then run the server with your own settings? My server does not seem to start after I modify the map settings files and then try to create the docker.

I did modify my settings, and the server started just fine.
The other option is to create a single player save with whatever options you'd like, and load that for the server.
I still haven't managed to get past my port issue though, so haven't actually connected to try...


Explore the topic of coverage %

One of the holy grails of OSM is to figure out how much of the places in the city you have mapped.
Check out the exploratory research Minh has done.  Can you think of a different way to estimate how much of the South Bay places have been mapped?  Something scalable?
Remember to consider Terms of Use.  Scraping Google APIs or Yelp for these purposes is not okay.
/cc @1ec5

To clarify, the approach I took was to count entries in the phone book and query OSM for features; these are hard numbers rather than estimates. I've largely wrapped up my counts for the 408 area code, but I welcome feedback about the approach and conclusions.
I'd also welcome a more scalable approach to facilitate an expansion into other area codes such as 650. My stunt of manually paging through the phone book could be streamlined using OCR. (The corresponding OSM queries are effortless to repeat, so no problem there.)
Beyond that, there's a question of whether the phone book is comprehensive enough to treat as a gold standard, especially given that the yellow pages demand payment for inclusion. I think we can test its comprehensiveness by comparing the coverage of certain categories or chains to official sources. For example, compare the list of dentists to a government database of licensed dentists.
An alternative to taking a census would be to sample an area's coverage, but we'd need to control for a different set of factors, such as development patterns that affect POI density or the focus areas of individual mappers.

Not sure if this is related, but in terms of coverage one of the things I have done for SCC is develop a tag availability index for key transportation related variables that are used to derive an LTS score. In theory something similar could be done for a broad spectrum of tags that are determined to be critical. This would only apply to a network however.
Alongside Yellow Pages, there are sales leads sites  that very often overlooked as source of geocoded data.  While I don't know if it is against their Terms of Service to use it directly for contributions, it might be scalable method to compare POI densities such as what @1ec5  is suggesting. The difference in densities between the two sources would be a potential indicator of an area in need of attention at the very least.


DefaultLanguage does not work in GetTrailers(tmdbID) and GetCollection(tmdbID)

HI
i always create a new API object in this way:
_TMDBApi = New TMDbLib.Client.TMDbClient(Settings.ApiKey)
_TMDBApi.GetConfig()
_TMDBApi.DefaultLanguage = Settings.PrefLanguage
It works with:
_TMDBApi.GetMovie(tmdbID)
_TMDBApi.GetCollection(tmdbID, _MySettings.PrefLanguage)
_TMDBApi.GetMovie(tmdbID, TMDbLib.Objects.Movies.MovieMethods.Trailers).Trailers
But it does not work (always english results):
_TMDBApi.GetCollection(tmdbID)
_TMDBApi.GetTrailers(tmdbID)
Is this intended or a bug?

Bug: quick look into the overloads leads me to this => return GetCollection(collectionId, null, extraMethods);

see fix https://github.com/LordMike/TMDbLib/commit/5f14cc7d895060f7e5d3388aad7e34ee03522cff
Couldn't find the GetTrailers thing you speak off, just submit a pull request for it if you locate it

Thank you for fixing.
it's the "public Trailers GetMovieTrailers(int movieId)" thing.

That impacts all other sub methods as well. I think we can just assume that if it's null we need to set the default language and if that is not set it will just be ignored. Won't do that fix from my browser, need to be able to run the tests. I'll get to it when I get home later today.

I see. So in theory, we should always set DefaultLanguage at the lowest levels, if the provided language is null?
One such place: https://github.com/LordMike/TMDbLib/blob/master/TMDbLib/Client/TMDbClientMovies.cs#L111

I would say YES.

This should fix it: https://github.com/LordMike/TMDbLib/commit/255b7847b8d1408cdcac1d56e2e51e5217b0f269

Was just working on that, spotted some issues in the tests though so went out to fix that.. Will check up on your changes @Naliath.

Seems good :+1:


Pre-trained concept embedding seems barely improve the model performance?

Hi Edward,
As I am reading some nlp papers about using pre-trained embedding can largely improve the downstream nlp tasks, like NMT. I am wondering is it still true that the pre-trained concept embedding can improve the healthcare tasks, like disease prediction in your paper.
It seems that in your experiment, the pre-trained concept embedding barely improve the model performance (only 1% increase of the recall rate), while the pre-trained RNN on scutter dataset clearly largely improve the model performance on MIMIC dataset.
So, why the pre-trained concept embedding seems barely improve the model performance
Is it because:

the quality of the pre-trained embedding is not good enough as data source is relatively small, skip-gram will suffer from the zipfian's law.
there is an aggregate operation on the visit-level, which sums all the medical code embedding and hence downplay the influence of the medical code embedding.
the characteristic of the task itself. There are two many new diseases that are unpredictable, with 70% recall rate is already reaching the max performance.

thank you so much!
Xianlong

Hi Xianlong,
You raise an important question, and your answers are all relevant to some extent.
Based on my experience (mostly from experiments I ran for several papers), the general rule of thumb for using the pre-trained embedding is as follows:

Pre-training your medical concept embeddings and fine-tuning them during the downstream prediction task should demonstrate at least the same level of performance as when you don't pre-train your embeddings.
The benefit of pre-training the embeddings varies depending on your task, dataset size, and model architecture.
-- The more data you have, the smaller the benefit of using pre-trained embeddings will be.
-- In my experience, you will benefit more from using pre-trained embeddings when your task requires correlation information between medical concepts, because you typically pre-train your embeddings using cooccurrence-based algorithms For example, DoctorAI tries to predict all codes occurring in the next visit, hence can benefit from code correlation information. On the other hand, when you have an external label (e.g. mortality, length of stay), then the benefit of using pre-trained embeddings diminishes. But this is coming from very limited experience so this may not generalize.
-- As you pointed out, summing the embeddings at the visit-level might cancel out the embedding values. This is one of the reasons I used ReLU at the code-level embedding in Med2Vec. But it seems neural network can learn whatever you tell it to learn, regardless of the summation at the visit-level, because after Med2Vec, I didn't particularly stick to ReLU at the code-level. Of course, a rigorous experiment to answer this question can be an interesting topic. (But I doubt there will be significant difference, because I've seen NLP studies where they sum the word embeddings to derive sentence representation and they seemed to work fine).
Many unsupervised representation learning algorithms use co-occurrence information of concepts (medical concept or words), and I suspect using co-occurrence information can only go so far. Maybe you can come up with a novel representation learning approach and help many researchers out there :)

Hope this helps
Ed

Hi Edward,
Really appreciate your response! It is very helpful!
I ran some disease prediction experiments with mimic-3 dataset using the same model (RNN) and hyper-parameters (2000 RNN hidden units, 200 embedding size and ect.) as you did in the paper.

Without pre-trained embedding, the recall@30 is around 52.0%. (similar to the results in your paper)
With skip-gram pre-trained (on mimic-3), the recall raise to 56.0%.
It is interesting that when I use your med2vec pre-trained embedding (1/3 code initialized randomly as not appear in the med2vec vocabulary), the recall rate is only 53.5%.

Since med2vec is using a much larger dataset, I would expect it has a greater influence on the disease prediction task.
I think it might because of heterogeneity of medical codes? (i.e. codes in children dataset might have different "meaning" to codes in ICU, and their co-occurrence information will then be different). What you you think?
Thanks!

Hi Xianlong,
First of all I have to ask:
When you pre-trained your embeddings on mimic-3, did you pre-train the embeddings on the entire mimic-3, or just the training set you used to train the disease prediction model?
If it's the former, then you are spilling information from the test set indirectly into the model, so comparing that to med2vec embeddings wouldn't be fair.
That being covered, I agree with your point; Med2vec embeddings on my webpage were trained on records from children's hospital only, so the diagnosis/medication/procedure distribution would be very different from MIMIC-3, which consists of ICU records. But still, it should be better than nothing (as demonstrated by your experiments), because MIMIC-3 is such a small dataset (only 7~8k patients who made at least two visits).
Best,
Ed

Hi Ed,
Sorry, I somehow missed your comment.
 I did pre-train on the entire dataset instead of the training dataset.
Pre-train only on training gives me a 53% recall rate.
Thanks for your valuable comment!


The "Free Shipping" badge renders differently between the filter panel and listing card

@jjeffryes I'm having a difficult time tracking this one down. It appears both instances have the same classes applied, but they render differently.
The instance on the listingCard is incorrect. It seems to have some extra vertical padding.

I'm not able to reproduce this, they both look the same to me:

Maybe it was fixed? If I'm wrong, I will promptly re-open.

Must have been fixed. Thanks for checking on it 👍

I fixed it a while ago and forgot to close the issue.