Test: server make book collection

User Story
As a quality assurance engineer,
I want to thoroughly test the functionality of the new collection created to keep track of books in the server,
So that I can ensure proper storage, retrieval, and manipulation of book-related data in the application.
Acceptance Criteria

[ ] Develop test cases to cover all aspects of CRUD operations on the book collection in the server.
[ ] Implement and execute automated tests to verify the proper creation, reading, updating, and deletion of book records.
[ ] Test the functionality of the book collection with various scenarios, including adding, updating, and deleting different types of book records.
[ ] Verify that the book collection allows for efficient storage and retrieval of book data.
[ ] Ensure proper error handling and validation in the book API endpoints.
[ ] Document and report any issues or bugs found during testing.

Related Issues

[Issue number]: Create a new collection to keep track of books in the server

Notes

Thorough testing of the book collection functionality is crucial to ensure proper storage, retrieval, and manipulation of book-related data in the application.
Test cases should cover various scenarios, including edge cases and error conditions.
Collaborate with backend developers to address any issues or bugs found during testing.

Tasks

[ ] Create comprehensive test cases for CRUD operations on the book collection in the server.
[ ] Set up an automated testing environment and framework if not already in place.
[ ] Execute the test cases to verify the proper functionality of the book collection.
[ ] Test the book collection with various scenarios to ensure reliability and efficiency.
[ ] Ensure proper error handling and validation in the book API endpoints.
[ ] Document and report any issues or bugs discovered during testing.

Starting

done


Group 9: Invariant returns refactoring (Second try)

Breaking change

Proposed change

Motivation
Second try. Apologies for the last pull requests, it seems like our fork was more up to date to the Home Assistant repo, we're unsure why this happened. Hopefully everything works as intended this time.
This PR contains fixes for code smells of the type 'Invariant function return'.
Invariant return values can lead to incorrect program behavior. Functions should provide meaningful and context-specific results based on their inputs and the current state of the program. Invariant return values may not accurately represent the outcome of the function's operation. Many returns of the same value also makes the code less readable and more difficult to comprehend and maintain.
Addressed by:
Code has been refactored to maintain the same logic flow. Each modified function only has one instance of the return.
Each change has been tested so the logical flow of the code is still the same.
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New integration (thank you!)
[ ] New feature (which adds functionality to an existing integration)
[ ] Deprecation (breaking change to happen in the future)
[ ] Breaking change (fix/feature causing existing functionality to break)
[x] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[x] The code change is tested and works locally.
[x] Local tests pass. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist
[ ] I have followed the perfect PR recommendations
[x] The code has been formatted using Black (black --fast homeassistant tests)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly.
Updated and included derived files by running: python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt.
Updated by running python3 -m script.gen_requirements_all.
[ ] For the updated dependencies - a link to the changelog, or at minimum a diff between library versions is added to the PR description.
[ ] Untested files have been added to .coveragerc.

To help with the load of incoming pull requests:

[ ] I have reviewed two other open pull requests in this repository.

Tests: azure_event_hub fails now.
Motivation: good
Pull request: The azure_event_hub change is wrong as the second conditional in async_setup is not nested within the first. While all the tests for the other components pass, I did not check all changes manually and I don't trust the tests since I don't know what they check. In principle, the changes are fine though.


⚠ GitLab Server has degraded performance

In 8131bef, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 2552 ms

Resolved: GitLab Server performance has improved in 6847160.


sharing the shares folder to users exits with different status code than in oc10 backend

Describe the bug
Sharing the shares folder to users exits with a different status than in oc10 backend.
Response OCS status code for the different versions are also different.
Steps to reproduce
Steps to reproduce the behavior:

create users uu1, uu2 and uu3
upload a file tmp.txt using user uu1
share file tmp.txt with user uu2 using user uu1
accept incoming share using user uu2
share folder Shares with user uu3

Expected behavior
In Oc10 backend:
Request fails with

OCS status code: 403
OCS status message: Path contains files shared with you

Actual behavior
Request fails with

with ocs-api version: 1

OCS status code: 996

with ocs-api version: 2

OCS status code: 500

Setup
Please describe how you started the server and provide a list of relevant environment variables.

export OCIS_URL=https://localhost:9200
export STORAGE_HOME_DRIVER=$DRIVER
export STORAGE_USERS_DRIVER=$DRIVER
export STORAGE_DRIVER_OCIS_ROOT=/tmp/ocis/storage/users
export STORAGE_DRIVER_LOCAL_ROOT=/tmp/ocis/local/root
export STORAGE_METADATA_ROOT=/tmp/ocis/metadata
export STORAGE_DRIVER_OWNCLOUD_DATADIR=/tmp/ocis/owncloud/data
export STORAGE_HOME_DATA_SERVER_URL=http://localhost:9155/data
export STORAGE_USERS_DATA_SERVER_URL=http://localhost:9158/data
export STORAGE_SHARING_USER_JSON_FILE=/tmp/ocis/shares.json
export PROXY_ENABLE_BASIC_AUTH=True
export WEB_UI_CONFIG=$HOME/www/ocConfigs/ocis-config/ocis-config.json
export IDP_IDENTIFIER_REGISTRATION_CONF=$HOME/www/ocConfigs/ocis-config/idp.yml
export OCIS_LOG_LEVEL="error"
export SETTINGS_DATA_PATH=/tmp/ocis/settings

if [ $DRIVER == "owncloud" ]
then
	export STORAGE_DRIVER_OWNCLOUD_REDIS_ADDR=localhost:6379
else
	export STORAGE_DRIVER_OWNCLOUD_REDIS_ADDR=
fi

Additional context
Add any other context about the problem here.

The behavior has changed a little bit now in OCIS
it gives 404 OCS status code in both versions with message No share permission
And gives 404 HTTP status in ocs-api version 2
in oc10 the status is 403 for all with message Path contains files shared with you

@kiranparajuli589 @SwikritiT can this be closed since the shares jail has been implemented?

This topic is no longer relevant. Closing.

Reopening as the behavior is still the same. The steps are as described in the scenrio here apiShareManagementBasicToShares/createShareToSharesFolder.feature:714
While sharing the Shares folder using new or old DAV version, oCIS server responds with status code 404 and message No share permission.
if this is the expected feature we've to adjust the tests.

We still have the test related to this issue in expected to fail
https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/coreApiShareCreateSpecialToShares2/createShareDefaultFolderForReceivedShares.feature
The test is tagged to wrong issue https://github.com/owncloud/ocis/blob/c8f8bc55e22597bc3d53b63b2f38f345b4814b07/tests/acceptance/expected-failures-API-on-OCIS-storage.md?plain=1#L645
Sharing the Shares folder in OC10 now responds
curl --user swikriti:0123456789 -v \
     "http://host.docker.internal:8080/ocs/v1.php/apps/files_sharing/api/v1/shares" \
     --data 'path=/Shares' \
     --data 'shareType=0'  \
     --data 'permissions=31' \
     --data 'shareWith=admin'

<?xml version="1.0"?>
<ocs>
 <meta>
  <status>failure</status>
  <statuscode>404</statuscode>
  <message>You are not allowed to share /swikriti/files/Shares</message>
  <totalitems></totalitems>
  <itemsperpage></itemsperpage>
 </meta>
 <data/>
</ocs>

While in ocis
curl --user einstein:relativity -vk \
     "https://localhost:9200/ocs/v1.php/apps/files_sharing/api/v1/shares" \
     --data 'path=/Shares' \
     --data 'shareType=0'  \
     --data 'permissions=31' \
     --data 'shareWith=marie'

<?xml version="1.0" encoding="UTF-8"?>
* Connection #0 to host localhost left intact
<ocs><meta><status>error</status><statuscode>404</statuscode><message>No share permission</message></meta></ocs>%

Both return with 404 but the message is different. The text expectation is 400 , what is the expection here? if 404 is okay then tests need adaptaion, if not is 400 the expected status code? Or we do not care about new or old DAV versions.  The error message also is not appropriate in ocis.
cc @ScharfViktor @micbar

The oC10 core test scenario is now here:
https://github.com/owncloud/core/blob/master/tests/acceptance/features/apiShareManagementBasicToShares/createShareToSharesFolder.feature#L728C3-L728C3
That expects OCS status code 403. And that is passing in oC10 core. So I am a bit surprised that the example curl command to oC10 is responding with 404

There's this test here
https://github.com/owncloud/core/blob/684cea2b152e7f18fd8b5e228a65a3197bcdefb7/tests/acceptance/features/apiShareCreateSpecialToShares2/createShareDefaultFolderForReceivedShares.feature#L10
That works according to the curl request
Maybe the status code is different depending on if the Shares folder is empty or has some resources

yes if the Shares has resources then the curl responds with following in OC10
<?xml version="1.0"?>
<ocs>
 <meta>
  <status>failure</status>
  <statuscode>403</statuscode>
  <message>Path contains files shared with you</message>
  <totalitems></totalitems>
  <itemsperpage></itemsperpage>
 </meta>
 <data/>
</ocs>

However, the behaviour is the same for oCIS

OK - then @micbar @ScharfViktor or whoever need to decide what are the expected/required responses in these situations.
After that, we can adjust the tests to match the requirements. An then a developer can adjust code to make the tests pass (if neded)

Re-sharing has been deleted so now user hasn't permission to share folder Shares -> 403 with No share permission message


Updated the detect provider button's background color

Added active pseudoclass, so it indicates when the button is clicked on
Updated the background color of the hover pseudoclass to make it look more visible

Hi, please kindly update your local repo, some updates has been made to the code base. After updating please make your PR.


Implement better output using an actual logging library

Right now everything outputs directly via console.log; this should be handled in a more sane matter with an actual logging library, such as winston.

https://github.com/winstonjs/winston

https://github.com/winstonjs/winston


Keyboard navigation jump back

Expected behaviour
When holding ctrl/shift you can navigate through year/month, when you then release ctrl/shift it should remain on the selected year/month
Actual behaviour
Jumps back to current or previously input date
Datepicker version used
Which ever is used on the demo, also present in 1.5.1
Example code
Just use the demo

Try to set updateViewDate to false, it should solve this issue.
I let this as an issue as updateViewDate should be false by default.

@Azaret how can updateViewDate be the problem? As this was merged in 1.7.0 and it is already a bug in 1.6.4

Ahmm, well I guess the issue was solved in the 1.7 but updateViewDate broke it back, if you do set it to false the issue is gone. I didn't make an expensive research on the changes in 1.7 that has solve this issue but I can look at it this weekend.

This is still happening in 1.8, there are plans to solve it?

is there any other workaround with less side-effects than 'updateViewDate'?


show "this is deleted" instead of null values

Front-end implementation of #319.

see #342

Oops


Wrong type in LayoutComponent exported from @remix-run/react

Reproduction
root.tsx
import type { LayoutComponent } from '@remix-run/react/dist/routeModules'

export const Layout: LayoutComponent = function Layout({ children }) { return <div /> }
// Property 'children' does not exist on type '{}'.ts(2339)

System Info
System:
    OS: Windows 10 10.0.19045
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 5.06 GB / 15.71 GB
  Binaries:
    Node: 20.11.1 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - C:\Program Files\nodejs\yarn.CMD
    npm: 10.2.4 - C:\Program Files\nodejs\npm.CMD
    pnpm: 8.15.3 - C:\Program Files\nodejs\pnpm.CMD
  Browsers:
    Edge: Chromium (121.0.2277.128), ChromiumDev (123.0.2400.1)
    Internet Explorer: 11.0.19041.3636
  npmPackages:
    @remix-run/dev: ^2.7.2 => 2.7.2
    @remix-run/express: ^2.7.2 => 2.7.2
    @remix-run/node: ^2.7.2 => 2.7.2
    @remix-run/react: ^2.7.2 => 2.7.2
    @remix-run/server-runtime: ^2.7.2 => 2.7.2
    vite: ^5.1.4 => 5.1.4

Used Package Manager
pnpm
Expected Behavior
type LayoutComponent = ComponentType<{
  children: ReactElement<
    unknown,
    ErrorBoundaryComponent | HydrateFallbackComponent | RouteComponent
  >;
}>;

to follow the docs on the root layout export.
Actual Behavior
Type is ComponentType, aka ComponentType<{}>; a user-defined component that takes no props.

We can get this updated internally but FWIW route module component types are not exported through the top-level @remix-run/react because they're not currently public API.  I think there's some hope eventually TS will be able to type at the module level at which point Remix would be able to provide better type safety on module exports, but until then we haven't made the decision to export those types for RouteComponent, ErrorBoundaryComponent, HydrateFallbackComponent, or LayoutComponent.


Add note on how support branches work

We’re adding docs on using support branches to our team docs as part of alphagov/design-system-team-docs#21.
Update our release docs for v3.x to reflect the fact that this branch is a support branch and to link to the new docs.

I'm also going to use this branch as a bit of a test and to configure the required status checks for the support/3.x branch.


Is it possible to implement/use a custom Viewport other than Viewport provided by flutter

Would be nice if I don't have to constrain the height/width if I'm using either Axis.horizontal or Axis.vertical. Is it possible to have a behaviour like SingleChildScrollView combined with Row?

Hi @natintosh, this may be achieved with SliverList instead of SliverFixedExtentList that is currently used in the package code. I've explained about this in more details in this issue (https://github.com/GeekyAnts/infinite-carousel-flutter/issues/17#issuecomment-1756484873). Please check it out once if this helps. Thanks.

Yes @manuindersekhon I agree using SliverList fixed the issue with Fixed Extent on the main scroll axis and allow dynamic size on the main scroll axis but soon the cross axis we still need to specify a fixed size (height/widget).


Remove TraceKit source fetching (now handled by Sentry server)

:tada:

Re-opening against master, which'll effectively be "3.x"


Precomputed RR

Can you merge the precomputed RR functionality of tpcf into the master branch?

@mjvakili - this feature was brought into the master branch by #401 .


translation: translate 'node' as the software product 'node.js' and not as 'knot'

Example: in "Request '{_request}' was cancelled because node is unresponsive." the word 'node' was translated to 'Knoten' (knot) in German. In the context of VS Code every occurrence of 'node' refers to the product 'Node.js'. I have changed the string in node-debug to use 'Node.js'.

Ported this issue to loc. Once the string change gets into loc file, translation will be updated accordingly.

@weinand source changes reflected to loc, instance of "node" in vscode-node-debug are modified to "Node.js"


Documentation for how to use Filters

Need some documentation and example use-cases of how and why you might want to use filters.
Possibly even provide some common/example filters in the library itself (e.g. MethodParameterFilter to automatically add ChannelItemParameters).
Examples:

#114
#101
#62

Another example use case is automatically adding bindings
e.g.
public class DefaultChannelFilter : IChannelItemFilter
{
    public void Apply(ChannelItem channelItem, ChannelItemFilterContext context)
    {
        channelItem.Bindings = new ChannelBindingsReference("default");
    }
}


command-line parser errors out on unknown arguments

the old way was just masking errors
fixes #39

👍


Additional alias documentation

Does this work?

Hi @lebogan , I think if you remove your rubocop-related commits and rebase your branch on master the build should pass. Or if you like, I can just cherry-pick in your documentation updates. Let me know. Thanks!

Open for too long.


Custom images for open graph etc

maybe a bit pie-in-the-sky but it'd be nice to replace this sort of thing...

...with this sort of thing...

...or this sort of thing:

For this, you would need to have set the value for <meta content="https://svelte.technology/images/twitter-card.png" name="twitter:image"> dynamically.
Maybe you can ask how CodePen.io do it on their end ;-)

@Rich-Harris do you have any ideas on how to proceed with this?
I was thinking maybe running a carbon service, similar to how we have bundle.run currently.
And then pointing the open graph meta tag to the service
Example repl gets rendered like this


From private repo

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


 Smh

We can wait till the new installer. Even though I'm not 100% against this change 👍

No comment.


Windows:  The specified procedure could not be found

There seems to be some regression with node-mapnik in conjunction with Mapbox Studio on Windows.
I tried on W7, W8.1 and Server 2012, all the same.
Steps:

clone mapbox-studio
copied node.exe from Studio installer into clone dir
npm install

some errors that dedupe of node-pre-gyp failed

node index.js errors with

module.js:356
  Module._extensions[extension](this, filename);
                               ^
Error: The specified procedure could not be found.
c:\mb\mapbox-studio-dummy\node_modules\tilelive-bridge\node_modules\mapnik\lib\binding\node-v11-win32-x64\mapnik.node
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (c:\mb\mapbox-studio-dummy\node_modules\tilelive-bridge\node_modules\mapnik\lib\mapnik.js:26:15)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)

I already checked with

DUMPBIN
ctypes.CDLL('mapnik.node')
ProcessMonitor

But found nothing.
@springmeyer @flippmoke any ideas?

Forgot: I changed the node-pre-gyp version in all modules to 0.6.4 (that's what node-mapnik uses) and run dedupe manually.
Didn't change anything.

Ahhh, node-mapnik-3.2.0 branch works out of the box.
However, I solved it by also changing Studio's package.json to mapnik:3.2.0 and deduping mapnik.

@BergWerkGIS aha, yes, see https://github.com/mapbox/mapbox-studio/pull/1287. That is just waiting on https://github.com/mapnik/mapnik/commit/51b65534f5b6f11a6403e27de52cbfea66b86958 to trickle through and then all tests should be passing and it can get merged.


package intstallation timeout

package intstallation timeout - sometimes it needs to be adjusted as the default 900 seconds  might not be enouhg

Looks like foodcritic doesn't like the timeout.  Looks like a FC bug that may be fixed in a newer version.
Are you deploying a large number of cassandra nodes at the same time?  I've been doing 20-30 at a time in AWS and haven't hit a timeout.  I did refactor at one point to try to install each package individually prior to installing dse-full but it was only for ubuntu/debian packages.
Are you using a rhel based distro?  Doublecheck your chef run and see which package/packages it's failing on.  You may be able to do the same thing for rhel that I did for debian packages.

Hi,
I am trying to build a docker container of centos 6.5 and dse-full inside. On my vagrant everything works fine but when I try building it via our Jenkins the chef run exits with "ERROR: yum -d0 -e0 -y install dse-full-4.8.0-1 exceeded timeout 900".
I will check the chef run.

My guess is that it is due to all the dependencies of dse-full.  you may be able to copy the debian package installation loop.  The package names should be the same between debian and rhel.
I recommend something similar to this as it just moves the loop up to encompass the rhel distros:
node['cassandra']['packages'].each do |install|
  case node['platform']
  # make sure not to overwrite any conf files on upgrade
  when 'ubuntu', 'debian'
    package install do
      version node['cassandra']['dse_version']
      timeout node['cassandra']['pkg-timeout']
      action :install
      options '-o Dpkg::Options::="--force-confold"'
    end
  when 'redhat', 'centos', 'fedora', 'scientific', 'amazon'
    package install do
      version node['cassandra']['dse_version']
      timeout node['cassandra']['pkg-timeout']
      action :install
    end
  end
end

Give that a try and see if it fixes it up for you.

Yep, I took the dependency packages and looped their installation through.
Thanks fo rthe suggestion.


Cookies to remember states/countries?

I'm living in Kentucky and refreshing this page every day - it helps me handle going to work.
I was wondering if you could leave a cookie to remember what a user sets it to be, and then a reset button to clear it?

As a stopgap, you can bookmark the page, since the URL changes according to the displayed data. For (only) Kentucky, here's the link.
https://aatishb.com/covidtrends/?region=US&location=Kentucky
Adding more states or changing parameters also changes the URL, which means you can bookmark several preferred views.

I thought about the issue for some time aswell. The solution of MrSpiffyClean works fine but is not as simple for the average user as cookies would be.

I thought about the issue for some time aswell. The solution of MrSpiffyClean works fine but is not as simple for the average user as cookies would be.


matterbridge: 1.23.2 -> 1.24.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/matterbridge/versions.
meta.description for matterbridge is: "Simple bridge between Mattermost, IRC, XMPP, Gitter, Slack, Discord, Telegram, Rocket.Chat, Hipchat(via xmpp), Matrix and Steam"
meta.homepage for matterbridge is: "https://github.com/42wim/matterbridge"
meta.changelog for matterbridge is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/xywbq75bgxkjn5yvw2myqqhw1b8lwwzy-matterbridge-1.24.0/bin/matterbridge had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 1.24.0 with grep in /nix/store/xywbq75bgxkjn5yvw2myqqhw1b8lwwzy-matterbridge-1.24.0
found 1.24.0 in filename of file in /nix/store/xywbq75bgxkjn5yvw2myqqhw1b8lwwzy-matterbridge-1.24.0

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

2 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
matterbridge

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

matterbridge

Maintainer pings
cc @ryantm @kalbasit for testing.

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

1 package built:

matterbridge

FYI: Matterbridge 1.25.0 was just released.


Fix implicit API layers being loaded as explicit on Android

This replaces the hardcoded API layer type with an actual type parameter that's supplied to the function, otherwise implicit API layers are treated as explicit.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Good catch, thank you!


Add size-limit

Will fail the CI if the library grows too big.
Run npm run size -- --why to run Webpack Bundle Analyzer.

:+1: every lib should have that and there should be a badge for the size as well :)


What's the serial number format?

Serial number does not match any of the numbers from the device...

can you give me an example ?


Item config should handle vendor bought items (vials)

There are some areas vials are still hardcoded; need longer term solution. i.e. get_bb_data, line 73:75

Fixed in #115


Problems with GPS in CCES Data

Recent processing of CCES data using PAMPal (by @shannonrankin)  led to a large number of warnings when the GPS data was added from the Pamguard Databases (provided by Anne Simonis) to the Pampal Acoustic Study.
Apparently the previous processing that yielded no GPS errors was done using a previous version of PAMpal with a bug in the GPS processing function. Now that this bug has been fixed-- it is identifying a LOT of errors.  Previously processed data or databases with GPS in them will have these errors.
In preliminary scanning of the data, Taiki found that there were at least a few chunks of GPS data that were missing, but seem to exist in the saved SPOT gps files. These SPOT GPS files are saved in various formats/combinations.
Around Nov 3, 2022, Marina Bozinovic (student) found a gap in her analysis of one of the drifts (from 9/12/2018 to 10/3/2018), and then Taiki found others. From those messages:
From @TaikiSan21 :
I took a look at the GPS data I have for each drift, and it looks like there are two distinct gaps. First the one from 9/12/2018 to 10/3/2018 that Marina identified, and a second one from 11/10/2018 to 11/17/2018. Nothing else seems obviously wrong or missing, there are just the occasional odd times where there are between 2 and 24 hours between GPS pings, but that seems more like a SPOT issue than a missing data issue. Looks like this affects drifts CCES 4, 10, 12 &13 (September gap) and CCES 17, 19, 20 & 22 (November gap). Haven't yet taken a look through the collection of CSV files I have to see if I can find any of this data.
Response from @asimonis :
We've uncovered yet another annoying bug resulting from datetime formatting.
The datetimes were saved in a different format (without seconds) in the CSV file for Sept 12-Oct 3, and Oct 29 - Nov 9. The way the code was written, datetimes from those files turned into NA values. At the time, I must have just copied the code that Jeff gave me, and didn't catch that error.  I modified the code to account for the different formatting and am attaching an updated dataframe with the 2018 CCES drift positions here.  Now we need to figure out how much this actually impacted our downstream processes.....
Another message from @asimonis :
I'm a little confused, because the maps in the final report (attached) all look like they were made with complete data.
I'm wondering if at some point AFTER the report was published, I opened the CSV files in Excel, and the datetime formats were automatically (and accidentally) altered, and then later I used the old routines to create the data frames, which then had the missing data.  Can we blame Excel? I really hope so...
Another message from @TaikiSan21 :
Updated GPS is now in the DriftWatch database, checked the new plots and looks like all is well. CCES_020 has a bit of a gap from 11/17 to 11/19, but I think that's probably just a SPOT issue (gap is present in the updated data you sent and the original data I had. Hard to tell from the plot of drift 20 in your report if that GPS data is any different than what we have, that plot is fairly small and the straight section isn't super long/obvious.
WHAT NEXT?

We need to make sure that all data on the synology (and that we each retain on our hard drives) is accurate
Taiki is reviewing the GPS data now....
Please use this thread to document any other information, and please tag people you want/need to review

I checked the deployment data in Tethys and found gaps. The CCES deployment worksheets need to be updated with the complete drift tracks as well

I loaded all these various CSVs and compared them to what I have in the DriftWatch database for CCES. I was able to identify a number of gaps in the DriftWatch data that can be filled in with some of the SPOT data. I'm guessing that there were just some mistakes when the individual CSV files for each drift were being compiled from the various SPOT downloads. There are still some gaps that can't be filled in, those could easily be explained by SPOT malfunctions. Example of big gap in CCES_020 shown here

Okay, here's some results of the GPS data I'm going to add for CCES. The plotted drifts are the only ones where I could see differences between the CSV files I linked above and the existing GPS data I had. Blue is what I already had in the database, red is missing data that will be added. Some were very small chunks, but others were quite large. Not sure why CCES_014 had only a tiny snippet of GPS data before, maybe I made a mistake when originally adding it? I'm going to try adding these updated GPS to the CCES data and see how many warnings I get

@TaikiSan21 Once the Driftwatch data has been updated, let me know and I can update the Tethys records. Thanks

I tested adding the updated GPS data to the CCES AcousticStudy, and there are still some gaps around our detections, affecting 27 events. 21 of these events are during the large gap (3 days) in CCES_020 I showed above. Then there's 1 event in CCES_022 that happens 16 hours after the last available GPS point. The rest of the events are only 1-2 hours away from the nearest GPS point, just from slight hiccups in the SPOT tracker. I think for those closer ones just interpolating between points is probably "good enough", but the larger gaps are a bit more questionable. Unless there's somewhere else some SPOT downloads might be hiding, I think this is probably the best we can do. All updated drift tracks are now in the DriftWatch database, so CSV files here are now updated.
@shannonrankin I also updated PAMpal to give less aggressive warning messages about GPS mismatch. It used to give a warning for every detector that couldn't match GPS properly, now it just tells you how many events were affected so that you don't have an overflow of warning messages. Update is on GitHub.
@asimonis If we don't have any other places to look for GPS data to add, what do you think is the best way to update the Pamguard databases with the new GPS data? I can go ahead and add the GPS data to the databases I have, but do you think we need to make new copies of the databases to store on the Synology or anything like that? And is this folder pictured on \pam2 the final resting place for these files?

@TaikiSan21 Yes - The 'pam2' Synology path you identified is where the final version of the beaked whale databases are saved. However - I don't think any of those files have had the GPS data added to them, so there's nothing that must be updated
Can you also upload the databases with updated GPS data to this shared folder?
I also have a local copy of these on Shannon's desktop (for RoboJay) that will need to be updated. Do you have local RoboJay copies that need to be updated too?
There are also 3 .rdata files here which will need the drifter and event GPS data to be updated.
We might want to update a few other folks who work with this data, including:
Marina Bozinovic (SFSU grad student)
Jennifer McCullough (she's about to publish a paper with the BWC detections)
Jenny Trickey
Jeff Moore
Jay Barlow

If/When anyone updates any specific file or set of files-- please identify those updates in this thread (so we know).
Specifically, @TaikiSan21 did you update the database files in the shared folder identified by Anne?

Unless anyone else has done so, I can update my desktop (for RoboJay) if I can have access to them (sometimes different logins cannot access data stored by another login).

I am slowly downloading all of the data from the Synology (Finalized BW Analysis) and I can double-check that these databases do not have the GPS (and if they DO, then I can overwrite them with the ones from Anne's shared folder)

@asimonis Question for you: Do you think there are other updates to the databases in the shared drive such that I should overwrite all the databases on the Synology?

Also, I will add the final GPS csv files to the Synology

@Shannon Rankin - NOAA Federal @.***> The 'final'
databases on the Synology drive are the most recent. All CCES events and
species labels should match what is available on the shared Google Drive.
On Wed, Feb 22, 2023 at 9:38 AM Shannon Rankin @.***>
wrote:

If/When anyone updates any specific file or set of files-- please identify
those updates in this thread (so we know).
Specifically, @TaikiSan21 https://github.com/TaikiSan21 did you update
the database files in the shared folder identified by Anne?

Unless anyone else has done so, I can update my desktop (for RoboJay)
if I can have access to them (sometimes different logins cannot
access data stored by another login).
I am slowly downloading all of the data from the Synology (Finalized
BW Analysis) and I can double-check that these databases do not have the
GPS (and if they DO, then I can overwrite them with the ones from Anne's
shared folder)
@asimonis https://github.com/asimonis Question for you: Do you think
there are other updates to the databases in the shared drive such that I
should overwrite all the databases on the Synology?
Also, I will add the final GPS csv files to the Synology
—
Reply to this email directly, view it on GitHub
https://github.com/SAEL-SWFSC/adrift-analysis-methods/issues/7#issuecomment-1440486982,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADBM562TF4PNJG5O2NZOKVLWYZFJLANCNFSM6AAAAAAU4HIBPY
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Anne Simonis (she/her https://www.mypronouns.org/she-her)
Acoustic Ecologist
Contractor with Ocean Associates in support of NOAA Fisheries
We are ADRIFT in the California Current. Check out our Sound Bytes blog
https://www.fisheries.noaa.gov/taxonomy/term/1000356091 here.
(619) 786-3578
www.fisheries.noaa.gov

Added updated GPS (and removed existing old GPS data) from the databases in that folder. I uploaded them to a separate folder (here)[https://drive.google.com/drive/folders/1yA7zWAUSjmjGz9nhfznxYKqhvfR3pWy5] instead of overwriting the existing files.
@asimonis I will take a look at the .rdata files and get them updated

I updated the databases on Shannon's desktop for RoboJay.  I also added a shortcut to the updated databases in our shared RoboJay google folder


cli: when should the cache be invalidated?

I wasted 30 minutes trying to debug a issue where the search and show paths were mismatched. It turned to that I fixed a bug but my fix was not seen since the entries file was outdated and I was reusing the same search term test and therefore a new search was never triggered... I need to change this. Below is the offending lines
  if (cache.DefaultConfig.term !== term || cache.DefaultConfig.repository !== repo) {
    cache.reset();
    cache.save(term, repo);
  }

https://github.com/scanf/ggrep/blob/master/packages/cli/ggrep.js#L56-L59

Code path has changed since last time, https://github.com/scanf/ggrep/blob/eb3382a865459ff24fc677fa33153dc374e628f0/packages/cli/lib/search.js#L36-L43


feat(SME): expose available connectors for the platform

gravitee-io/issues#6132

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
Congrats, all the CI steps were 🟢 . The Bot was able to deploy the build of this PR, so you can now try it directly here! 🚀
Notes: The deployed app is linked to the management API of the Element Zero team's environment.

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
Congrats, all the CI steps were 🟢 . The Bot was able to deploy the build of this PR, so you can now try it directly here! 🚀
Notes: The deployed app is linked to the management API of the Element Zero team's environment.


Pin Homebrew version

Unlike Linux package managers like apt-get, Homebrew likes to update itself when updating the package list (cough git repo cough). I'm not sure if this is possible, but we should pin the Homebrew version to avoid having surprises like #483.

@aneeshusa I think this article describes a solution to pinning the version of brew http://stackoverflow.com/questions/10093918/ignore-formula-on-brew-upgrade

@Coder206, I should have been more clear. Homebrew has two versioned parts, the brew binary itself and the homebrew-core package formulas. I'd like to be able to pin both of those versions, both:

installing pinned versions of both for fresh installs
updating our pinned version of both for existing installs

I'm not sure ,but I'm not aware if Homebrew actually currently has this functionality.


feat: add current price

@hieronx current price is not available in portfolio runtime V1, we need to handle the different cases for V1 and V2.
  ActiveLoanInfoV2: {
    activeLoan: 'PalletLoansEntitiesLoansActiveLoan',
    presentValue: 'Balance',
    outstandingPrincipal: 'Balance',
    outstandingInterest: 'Balance',
    currentPrice: 'Option<Balance>',
  },
  ActiveLoanInfoV1: {
    activeLoan: 'PalletLoansEntitiesLoansActiveLoan',
    presentValue: 'Balance',
    outstandingPrincipal: 'Balance',
    outstandingInterest: 'Balance',
}

--> Planned for next sprint


The 'maximize = True' option in  xgboost.cv don't work

I  want to  use an user-defined metric by  'feval'  parameter in xgboost.cv, but it seems that the 'maximize = True ' option don't work at all.  It seems that it's a bug.
from __future__ import print_function
import numpy as np
import pandas as pd
import xgboost
from sklearn import datasets
from sklearn.model_selection import train_test_split

data,label = datasets.make_classification(n_samples= 10000, n_features=20, n_informative= 6 ,
             n_classes=2, n_clusters_per_class=10,random_state=0)
X_train,X_test,y_train,y_test = train_test_split(data,label)

# custom metric: ks, it is a score, larger is better
from scipy import stats
def ks(preds,xgbtrain):
    label = xgbtrain.get_label()
    df = pd.DataFrame(data = np.array([preds,label]).T,columns = ['preds','label'])
    df_0,df_1 = df[df['label']<0.5],df[df['label']>=0.5]
    ks,ks_pvalue = stats.ks_2samp(df_0['preds'].values,df_1['preds'].values)
    return 'ks',ks

dtrain = xgboost.DMatrix(X_train,label = y_train)
dtest = xgboost.DMatrix(X_test,label = y_test)

param = {'max_depth': 2, 'eta': 0.1, 'silent':1, 'objective':'binary:logistic'}

# use xgboost.cv to use the ks as the custom evaluation but  'maximize = True' don't work
cvresult = xgboost.cv(param, dtrain, num_boost_round = 1000,
                  nfold=5, seed=10, metrics = 'auc' ,feval = ks, maximize = True,
                  callbacks=[ xgboost.callback.print_evaluation(show_stdv=False),
                             xgboost.callback.early_stop(10) ])

num_round_best = cvresult.shape[0] - 1

the results as below:
[0]	train-auc:0.674289	train-ks:0.238805	test-auc:0.665216	test-ks:0.234206
Multiple eval metrics have been passed: 'test-ks' will be used for early stopping.

Will train until test-ks hasn't improved in 10 rounds.
[1]	train-auc:0.680778	train-ks:0.248258	test-auc:0.669496	test-ks:0.235578
[2]	train-auc:0.692075	train-ks:0.273699	test-auc:0.679234	test-ks:0.260029
[3]	train-auc:0.697084	train-ks:0.281107	test-auc:0.686079	test-ks:0.267151
[4]	train-auc:0.700154	train-ks:0.281611	test-auc:0.687756	test-ks:0.267845
[5]	train-auc:0.707491	train-ks:0.291344	test-auc:0.69528	test-ks:0.284074
[6]	train-auc:0.710668	train-ks:0.293191	test-auc:0.696747	test-ks:0.285947
[7]	train-auc:0.716494	train-ks:0.309238	test-auc:0.702744	test-ks:0.294814
[8]	train-auc:0.719062	train-ks:0.318497	test-auc:0.704835	test-ks:0.299245
[9]	train-auc:0.72203	train-ks:0.324148	test-auc:0.707431	test-ks:0.310519
[10]	train-auc:0.72729	train-ks:0.33985	test-auc:0.712009	test-ks:0.320517
Stopping. Best iteration:
[0]	train-auc:0.674289+0.00230273	train-ks:0.238805+0.00439313	test-auc:0.665216+0.00744289	test-ks:0.234206+0.0127197

You should pass maximize to xgboost.callback.early_stop, since you are manually adding it to callbacks:
cvresult = xgboost.cv(param, dtrain, num_boost_round = 1000,
                  nfold=5, seed=10, metrics = 'auc' ,feval = ks, maximize = True,
                  callbacks=[xgboost.callback.print_evaluation(show_stdv=False),
                             xgboost.callback.early_stop(10, maximize=True) ])

See the documentation at https://xgboost.readthedocs.io/en/latest/python/python_api.html#xgboost.callback.early_stop

Thanks, I got it.


Moving to python module

Changes so far

Client access token is set in code instead of in config file
Created setup.py with install instructions
Removed credentials.ini.enc

TODO

[ ] Bugs when running from command line
[ ] Redo tests

I decided to remove command line functionality from the package for now. Being able to call the package from the command line doesn't add a whole lot to the intended usage, so I'm not too bothered leaving it out. I changed the tests to rely on an environment variable, GENIUS_CLIENT_ACCESS_TOKEN.
I'm hoping the Travis tests will pass once we merge this PR to master; Travis removes secure environment variables on PRs from forked repositories.
Thank you for your help! Next step is to set the package up on PyPI.


overriding deserilization setting for AzureStack backup admin

Adding customization layer for overriding serialization/deserialization settings.

@deathly809  It seems you do not need this behavior. This PR will be closed EOD today.

@shahabhijeet We are just going to change the spec.  Thanks anyways.


fix: max crate size limit

fix #184 by add upload body limit to axum

Thank you for the fix. It will be in the next release in a short time.


Support w3c elements

Elements can be returned via ELEMENT or element-6066-11e4-a52e-4f735466cecf. This updates the code to support either.
You can see an example in the python driver here: https://github.com/SeleniumHQ/selenium/blob/eb4a3c475c10ef0c7d65dce45fa179ce2c109e5d/py/selenium/webdriver/remote/webdriver.py#L282
I noticed this issue attempting to run tests against macos/safari on sauce labs.

Thanks!


UIKit v3

[ ] React 18
[ ] Single CSS bundle

@amje  We can also remove Toaster singleton (and it legacy methods), i just merged context for working with Toaster via hooks/hoc

Pending breaking changes to consider:

#241
#238
#222
#123

I have approval for #241 and #222 from @korvin89 and #238 from @amje


Added upgrades and upgradesOf

These combinators support validating versioning data and transforming it to the latest version in the process.
Closes #15

Codecov Report

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

@@          Coverage Diff          @@
##           master    #16   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines         276    291   +15
=====================================
+ Hits          276    291   +15

Impacted Files
Coverage Δ

dist/partial.lenses.validation.cjs.js
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Prettier update file with settings and serviceWorker file add for Netlify deploy.

Description
Added serviceWorker in order to deploy on Netlify, it would not compile without it. I also added the prettier configuration file to match the backend configurations and taking into account jsx for the front-end.
Fixes # (issue)
Netlify serviceWorker build error
Type of change
Added serviceWorker file and prettier file
Please delete options that are not relevant.

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Change Status

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

How Has This Been Tested?

[X] Test A - Tested branch on Netlify and it deployed, I switched it to master again for deployment after merge.
[ ] Test B

Checklist

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

 Deploy frontend

 Configure continuous integration (CI)

 Set up test suite


ui-ninezone documentation

Is there any documentation available on ui-ninezone? I see the sample app here (https://github.com/imodeljs/imodeljs/tree/master/ui/ninezone), but is there any actual documentation?

⚠ Do not edit this section. It is required for imodeljs.github.io ➟ GitHub issue linking

LINK: iModel.js-API-Reference-Documentation---iModel.js-link-to-docs
TAG: imodeljs-docs-question
SOURCE: https://imodeljs.github.io/iModelJs-docs-output/reference/

Andrew,
There are 4 iModel.js UI packages: ui-core, ui-components, ui-ninezone and ui-framework. ui-core & ui-components will be part of the 1.0 release. However, ui-ninezone and ui-framework are still in active development and will be available in a later release, which will include a ninezone sample application.
If Hatch would like to work directly with Bentley as an early adopter, you will receive guidance on all UI packages and developing ninezone applications.
Dan

@AndrewStraussHatch with the latest update of the iModel.js documentation, a section on [UI 9-Zone}(https://imodeljs.github.io/iModelJs-docs-output//learning/ui/ninezone/) has been added.
Let us know what you think.  Thanks!


Refactor usages of global Homebrew.args.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

Pass args down the call chain instead of using global state everywhere. This should make refactoring CLI::Parser possible in order to use it for cask subcommands.

Is it worth removing the global Homebrew.args entirely to avoid usage?

The goal is to have multiple levels of args:

brew level: e.g. --debug, --verbose (which is now the global Homebrew.args)
brew <subcommand> level
brew cask <subcommand> level

brew level: e.g. --debug, --verbose (which is now the global Homebrew.args)
brew <subcommand> level

This sounds good. Still think I it may be worth having a slightly different API at this time to avoid Homebrew.args being called in new code and getting unexpected results.

3. brew cask <subcommand> level

Why would this need to be separate from the brew level, out of interest?

Why would this need to be separate from the brew level, out of interest?

There are some flags which all cask commands support, e.g. --appdir, and there are  more specific flags for cask subcommands. Maybe this can also be refactored then.


feat: add gradient modern dark

:tada: This PR is included in version 1.10.0 :tada:
The release is available on:

GitHub release
Visual Studio Marketplace

Your semantic-release bot :package::rocket:


bug: delete work cannot finish executing

Ran clusteradm delete work addon-work-manager-deploy  --cluster itdove2-4-10 and it never finished, maybe because the cluster is already removed.
 oc get managedcluster
NAME           HUB ACCEPTED   MANAGED CLUSTER URLS                                                      JOINED   AVAILABLE   AGE
itdove-4-10    true           https://api.aws-4-10-rc4-sno-2x-vxxxxx:6443/   True     True        16d
itdove2-4-10   true           https://api.aws-4-10-rc4-sno-2x-xxxxxx:6443/   True     Unknown     13d

When the cluster is "available" it works.
reported by @itdove

/assign @ycyaoxdu


We should remove the sync folder settings for rsync from the default Vagrantfile

Because the default settings does not make any sense for a Vagrant guest running on a Windows machine. Windows does not have rsync installed so even if vagrant up tells that the sync happened on Windows, but it did not do any sync
Vagrant.configure('2') do |config|
        config.vm.synced_folder ".", "/vagrant", type: "rsync"

Removing that line completely would results in vagrant up waiting until the timeout elapses (5 minutes by default), and the process ends with an error code. We don't include the VirtualBox Guest Additions in the official CentOS Vagrant images - this would be a breaking change for us (at least as long as we can't customize the embedded Vagrantfile).
https://github.com/CentOS/sig-cloud-instance-build/issues/58#issuecomment-242964669


fix fix failed to specify the number of buckets when bucket auto

Proposed changes
Issue Number: close #xxx

fix failed to specify the number of buckets when bucket auto
delete unused SessionVariable
if mtmv used external table ,check isMaterializedViewRewriteEnableContainForeignTable

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run feut

run feut


understand and document typing/typing_extensions.deprecated

Is your feature request related to a problem? Please describe.
it would be great if mkdocstrings could add a warning to docs when the typing/typing_extensions.deprecated decorator is used, see PEP 702.
I know this pep is in draft but it's already in typing_extensions and supported by vscode.
Docs example: https://docs.pydantic.dev/dev-v2/api/main/#pydantic.main.BaseModel.copy
Describe the solution you'd like
It would be great if mkdocstrings added a markdown warning "Deprecated: "
Describe alternatives you've considered
I think we'll add the warning manually ourselves for now.
Additional context
cc @Kludex @tpdorsey

How would you want the warning to look like? A "danger" (red) admonition named "Deprecated" with the message as body?

Ah! @pawamoy, I hadn't told you (sorry), I was talking with Jelle Zijlstra, the maintaner of typeshed (typing), etc. He's also the author of @deprecated.
Because deprecated overlaps with my doc() thing, and because it would be easier reach consensus/acceptance if the scope of my doc thing is smaller, my plan (agreeing with Jelle) was to reduce doc() to only documentation, and then deprecation would be done by deprecated().
Currently, the deprecated PEP only talks about usage as a decorator, but it doesn't forbid using it in Annotated. It's something that could be done, and even specified in a future PEP (but could be used right away).
I haven't been able to update my doc thing (and talk to you) yet because I've been deep down the rabbit hole of the Pydantic v2 migration in FastAPI, but that was the next thing I wanted to work on.
...I'm so sorry I soft-encourage you to implement everything in my current/old version that tried to cover everything. :grimacing:
Now the idea is to try and make my doc() thing a draft PEP. It could end up in typing_extensions. If that doesn't happen I think maybe it could be accepted in https://github.com/annotated-types/annotated-types (from @adriangb, @samuelcolvin...), that could simplify adoption and maybe your implementation (not sure).

Just to clarify, what Samuel is asking is only the standard usage of @deprecated() as a decorator.
What I'm saying is that my doc thingy will no longer have the parameter/functionality to specify that something is "deprecated", or the exceptions it raises. So, for usage as a decorator, this would replace my doc thingy (that you already implemented :pray: ).
And the other thing I'm saying is that the intention is to extend the usage of deprecated() a bit, and also use it in Annotated for parameters. That will replace that feature of my doc thingy when used in Annotated. I'll use that in FastAPI.

Not sure about others, but the way I would like to see is the yellow box (warning) with the "Deprecated" title. The position I guess would be after the description and before the attribute/parameters table.

I actually haven't thought much about the final rendering in the docs :thinking:
In editors I like the strikethrough, but I don't have a strong opinion in the docs.

A "danger" (red) admonition named "Deprecated" with the message as body?

Sounds perfect.

Adding a CSS class on the deprecated object might be a nice touch, enabling (optional) front end customization. Something like doc-object-deprecated maybe. It doesn't have to do anything by default, but nice to be able to pick up via the CSS.

First version available to sponsors :slightly_smiling_face:
https://mkdocstrings.github.io/griffe-typing-deprecated/


Read-only view UI issues:

[x] I think it's still hard to see the text in dark view

[ ] Drop-downs look like they are active and have no carrot

[x] Radio buttons are focusable & the text is too dark to see

[x] Add buttons just look like floating text?  maybe the outline is too dark for the fields?

-[ ] Resource Fields need styling?

[x] Scaling and Upgrade policy fields need styling

Light View

[x] Can't see any of the field label or text inside the fields

@vincent99 can you look at "Drop-downs look like they are active and have no carrot"

https://github.com/rancher/dashboard/pull/1966
https://github.com/rancher/dashboard/pull/1963
https://github.com/rancher/dashboard/pull/1946

https://github.com/rancher/dashboard/pull/1966
https://github.com/rancher/dashboard/pull/1963
https://github.com/rancher/dashboard/pull/1946


[Waiting for ionide-vscode-helpers] Upgrade to VS Code's webview API and fix show project status

This fix #958 and fix #1003
Implementation has been adapted from webview-sample
This PR needs https://github.com/ionide/ionide-vscode-helpers/pull/27 to merged before. I will update the dependencies when ionide-vscode-helpers will be updated.

It probably also fix #1002

I’ve merged the helpers PR. Awesome work, thanks a lot!

About #1002 I don't know.
The fix included in this PR, is only for the Show project status feature. Because the following lines is called only when generating the HTML code.
I think the function which generates the files list is adding Folder and File into it. And because the files is of type Model list it's a valid representation. I think we have an issue in the code that generates the "Workspace" tree.


add example on how to update deps db

I was trying to figure this out a while back. IMO, it would be very useful if the readme at least contained a short mention of how to do it.
Now, looking through the Nix expressions in this repo again, it seems that, if not using Nix flakes, the simplest way would be to regenerate the flake.lock file locally and somehow patch it over this repository's content after fetching it with builtins.fetchGit. Is that right?


$fetch is broken on post FormData

Environment

Operating System: Darwin
Node Version:     v16.14.2
Nuxt Version:     3.0.0-rc.1
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      ssr, css, typescript, runtimeConfig, buildModules, components, intlify, unocss, vite
Runtime Modules:  -
Build Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>

Reproduction

Open a clean project
Try post form data
body is stringified

Describe the bug
There is a known bug in ohmyfetch link:https://github.com/unjs/ohmyfetch/pull/36#discussion_r767707178
Body is getting stringified on FormData posts. Since Nuxt3 using ohmyfetch by default client, It is now impossible to use native client for formdata requests.
Additional context
No response
Logs
No response

Issue can be closed, no longer relevant


updated README.md Authors section

Done.
Thank you very much for letting me work on this and learn more about creating pull requests.

Thanks for contribution but there is already other #29 PR raised for same issue.


Add use cases to DG

Added use cases related to the 7 basic commands.

Codecov Report

Merging #21 (1e1a3f5) into master (2580eb3) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #21   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 2580eb3...1e1a3f5. Read the comment docs.


Add support for being able to get KeyStatus

As part of Un-Prefixed EME we should be able to query for KeyStatus. I request this support to be added to Shaka Player.

https://w3c.github.io/encrypted-media/#widl-MediaKeySession-keyStatuses

The correct link is: https://w3c.github.io/encrypted-media/#idl-def-MediaKeyStatus

Looks like this is already implemented in shaka player: https://github.com/google/shaka-player/blob/master/lib/media/eme_manager.js#L539

Okay, then if you have what you need, I'll close this issue.  Thanks!

I'm not sure if this is what you are looking for. This checks for error key statuses and dispatches an error event if one is found. The ability to query the current keyStatuses does not exist.


[Q] duktape and test frameworks

Has anyone ever used one of the well known test frameworks (Mocha, Jasmine, QUnit) with duktape? I can't get any of them to work here. At least I can successfully import Jasmine, but using it fails with undefined identifier errors I can't explain.
Is there perhaps a different test framework that works with duktape?

I don't have first hand experience - probably those undefined identifier errors are just plain missing bindings such as expecting setTimeout, window, etc.

OK, just was wondering if anyone ever made it to use a test framework with duktape. I came quite far with Jasmine, but the Jasmine npm requires quite some NodeJS machinery to work (and the browser version relies too much on things only available in a browser). Thanks.

Maybe someone else has done that, I've only needed to run some test sets like test262, underscore tests, bluebird tests etc, and most of those need a small shim to provide missing bindings. But if the test suite expects a lot of helper modules, it's probably not going to be a trivial task.

@mike-lischke Just as a concrete example, here's an example of what Bluebird test set needed: https://github.com/svaarala/duktape/blob/master/util/bluebird-test-shim.js. Fully fledged test frameworks will probably require many more bindings, but just to give an idea what kind of stuff is usually needed.

The thing is I have to implement quite a number of NodeJS modules anyway (file system, process, path, util etc.) so it's not so much extra work for Jasmine (or other test frameworks). Problems arise when such a library makes use of the more complex stuff like events (EventEmitter etc.).


[proposal] Adding the keyword 'interface'

Hello!
I have only recently started learning Dart, so I apologise if my suggestion comes across as silly.
I would like the Dart language to implement the interface keyword, which would be understood by the compiler identically to the abstract class. For example, the following two code fragments should be perceived identically by the compiler:
abstract class Person {
  void display();
}

interface Person {
  void display();
}

I realise this is syntactic sugar. But it would be great! :+1:

That is a valid idea. It's already suggested in the language issue tracker as https://github.com/dart-lang/language/issues/3736


Add median and interquartile range as alternative uncertainty measure

Mentioned in #84.

This brings up an interesting question: Currently the fields of the uncertainty data structure are named lodetection and stddev1. Should we use a better name for the latter that does imply it being a standard deviation? @kathapand

Good point - a suitable, more general name would be spread, which can be expressed by both standard deviation or interquartile range

variation might also be possible, I guess

yes, equally fine by me


Connects but n

lancelot@slipstream:~> hpsdrconnector -s 384000
2021/09/09 01:58:39 [INFO] Listening on IQ port 4590
2021/09/09 01:58:39 [INFO] Starting radio on <IP_ADDRESS>:1024
TX light on HL2 goes off
In firefox <IP_ADDRESS>:4590 times out after about a minute --- no firewall enabled.
The connection has timed out
The server at <IP_ADDRESS> is taking too long to respond.
The site could be temporarily unavailable or too busy. Try again in a few moments.
If you are unable to load any pages, check your computer’s network connection.
If your computer or network is protected by a firewall or proxy, make sure that Firefox is permitted to access the Web.

Could you explain what you tried in a bit more detail? hpsdrconnector is not meant to run standalone and you can't connect to it with a browser. It is run by OpenWebRX in order to connect to an HPSDR radio. Have you installed OpenWebRX?

Thanks, I need to install OpenWebRX

Feel free to re-open if you run into issues.

Thanks, will do.
73 ... Sid.
On 09/09/2021 13:31, Jim Ancona wrote:

Feel free to re-open if you run into issues.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jancona/hpsdrconnector/issues/4#issuecomment-916049153,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA5I2K4ZE7B55SW3B6THMMDUBCSLTANCNFSM5DWAHRWA.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Sid Boyce ... Hamradio License G3VBV, Licensed Private Pilot
Emeritus IBM/Amdahl Mainframes and Sun/Fujitsu Servers Tech Support
Senior Staff Specialist, Cricket Coach
Microsoft Windows Free Zone - Linux used for all Computing Tasks


game use?

honestly, probably the main reason someone would want to use lua on the web is either on the server back-end, or for game scripting...
any pointers how i might implement it in a game or game engine? i'd ideally not want to be interpreting the scripts every single time i call them, because the use cases for this would probably be giving objects customizable behavior, which means executing that code probably once per frame/update...

I tend to think you would probably not use Fengari for a web-based game. You can, but if you need a whole game engine you'll probably go the WSAM route now.
Maybe look at frameworks that already support this (e.g. https://defold.com)?

well, the goal is to have web and stand-alone desktop versions. want to support custom content and would much rather not have random scripts running with potentially no control over what they can do.
web assembly right now doesn't appeal to me as much as it first did...

Fengari is not really a good fit for gamedev unless you're really light on performance requirements.

One of the best use cases for Lua is scripting in general, while embedded in a larger application, and running it inside the browser isn't actually that far off conceptually. Mainly it's slower due due to the added overhead and occasionally cumbersome to use due to lack of APIs/documentation.

any pointers how i might implement it in a game or game engine? i'd ideally not want to be interpreting the scripts every single time i call them, because the use cases for this would probably be giving objects customizable behavior, which means executing that code probably once per frame/update...

This statement confuses me. Would you execute the same code in JavaScript every frame and if so, how would that be different? Doesn't the real overhead come in when you must interface between the JavaScript VM and the Lua VM? At least that's the only scenario where I've had performance issues, and that was real-time parsing of (large amounts of) binary data which I simply do in JS now.
From what I understand, the scripts are loaded once just like .js scripts would be. But maybe I'm misunderstanding the issue? At least in my use case, 90-95% of the time is taken up by the rendering engine, which runs in pure JavaScript anyway. But of course I'm not making the next AAA game, so YMMV.

something like that
yeah, the interfacing part is what concerns me. definitely won't be parsing files with it. mainly just altering the behavior of some pre-defined objects
that would be ideal. i just worry even if the scripts are permanently loaded that the overhead of doing
object > script > return
object > script > return
countless times would cause noticeable slowdown

Have you done any benchmarking? The way I understand it, the overhead of merely running the Lua VM (inside the JS VM) won't really be a problem. Everything else depends on your choice of technology, i.e. the game engine you're planning to interface with and the target platform's capabilities (mobile vs desktop etc).
Of course running Lua natively, especially offline and with LuaJIT would be faster, but if you manipulate Lua tables there shouldn't be an issue. If you need to access JS objects it really depends on the number of times you have to interface.
You could always try spinning up a prototype and see how it does.

right now, it's purely in the theoretical/planning stage. engine will be fully custom, specific for re-making a certain flash game.
and native is part of the problem as it has to be integrated somehow, too. in theory, most objects won't be using scripts, so that could help. but i'd rather plan for and assume they ALL are.
i probably will eventually. certainly won't build a full implementation if a benchmark performs poorly. irl lately has been a nightmare, it happens literally EVERY time i think i might be able to work on a project like this.


Update specs2-core, specs2-scalacheck to 4.12.11

Updates

org.specs2:specs2-core
org.specs2:specs2-scalacheck

from 4.12.0 to 4.12.11.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.specs2" } ]

labels: test-library-update, semver-patch

Superseded by #179.


[REQUEST & ISSUES]

Checklist

[X] I have read the FAQ
[X] I have checked if this issue already exists (both open and closed issues)
[X] I am using the most recent version of pogom from the master branch
[ ] If this is an installation issue, I am posting my OS, OS version, python version, pip version
[ ] I'm posting the complete log
[ ] I'm using a descriptive title for this issue

My Issues & Requests

Multiple notifications of same pokemon with wrong timer
Notifications on mobile doesn't work yet (iOS 9.3.4, tested browser: safari, chrome, opera)
need a list of scanned locations to make navigation easy
need data of every single scanned pokemon and show on map like on "PokemonGo-Map" (date, time, number of spawned pokemon of same species) [Improvement of Pokestats]

I really would like to implement this on my own for us, but unfortunately i'm not familiar with Python :-(
I don't want you to implement the improvements above immediately, but i want to know if there is any chance the devs are working on something like that.
Thank you for reading and not flaming ;-) !!!

I'm using a descriptive title for this issue
I have checked if this issue already exists (both open and closed issues)


feat: support RawSlice as argument

Not sure if Sway supports it though.
From @segfault-magnet :
I tried implementing RawSlice support, turns out it isn't difficult.
A RawSlice is encoded as two u64 ptr and len. So encoding it would look like this:
fn encode_raw_slice(data: &[u64]) -> Result<Vec<Data>> {
    let tokens = data
        .iter()
        .map(|&word| word.into_token())
        .collect::<Vec<_>>();
    let encoded_data = Self::encode_tokens(&tokens)?;
    let len = data.len() as u64 * 8;
    Ok(vec![Data::Dynamic(encoded_data), Self::encode_u64(len)])
}

the function selector is rawslice.
ParamType::try_from_type_application needs to be updated to detect raw slices. This helper is used by the index team to generate fn selectors in runtime from only the json abi file.
Its encoding width is 2 WORDS, so compute_encoding_width needs to be updated.
Also, you need to add it to should_compute_custom_input_offset.
That's all you need to nest raw slices however deep in other types when used as function arguments.

Not sure if Sway supports it though.

I figured out the encoding by doing examples in Sway. So passing in raw slices definately works if you're referring to that.
Didn't try nesting it though in inputs, but I nested it in a tuple and returned it and the compiler didn't complain.
Have a feeling if that passed so will nesting when used in fn args

Closed by #904.


[Feature] Reorganise sounds

Ability to drag and drop sounds to reorganise them.

note to self: use
CanReorderItems="True" AllowDrop="True"

Since the catalog is growing too, it could be handy to have a 'favorite' feature where you can star your favorite sounds and have them in a separate bucket at the top?
Maybe a 'most recently' listened bucket too?
Make it feel like Netflix 😋

Yes, this would be excellent!

Since the catalog is growing too, it could be handy to have a 'favorite' feature where you can star your favorite sounds and have them in a separate bucket at the top?

I started a separate issue for that in case anybody else wants to drop ideas/related requests there: #245

already fixed a long time ago in #304


error in the reflection oracle in Grover's

Description of Issue (if Applicable)
There is an error in the reflection oracle of Grover's algorithm. Let |s> denote the equal superposition state, then the reflection oracle can be expressed as $U_s=2|s><s|-1$ or $U_s=H^n U_0 H^n$ where $U_0=2|0><0|-1$.
Fixes #446
What Changes Were Made?
The oracle in the equation should be U_0, not U_s, that flips every state other than |00>. The boxed gates in the image below should also change to U_0. If you want U_s, you must include the Hadamards before and after.
How Was This Tested?
It's a theoretical error, so no tests are required.
Anything Else We Should Know

Thanks for this, I have made some more changes (fixed the images / added extra explanation) and made a PR to your branch.

@frankharkins great! While I was reviewing this PR, I noticed that the image needs to be corrected  as well and was going to make an issue about it after the approval. :)

@frankharkins I've merged the pr.


Boost profile image fetching significantly

Bause fetching a profile requres the current profile-service to also talk with the user-service we get a significant hit in percentile values for this service.
Currently we are just interested in the profile image, and it is already available as a special methiod getProfileImages, this method will be much faster, becuase the profile service is not required to talk with other services for this call.
Can we add a special resolver for the profile-image, so that we can skip resolving the whole client when only the image is requested?

Why does unleash care about profile images at all? Especially finn internal thrift services

Wrong repo ;)
On 3 Nov 2016 9:27 p.m., "Simen Bekkhus"<EMAIL_ADDRESS>wrote:

Why does unleash care about profile images at all? Especially finn
internal thrift services
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/finn-no/unleash/issues/161#issuecomment-258263844,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJs5JJhUgIqVcZClOqTBww7iAVbkFzoks5q6kOggaJpZM4KoaVR
.

Ah, finnlet. Gotcha!


Update CI Settings

From #10
Either fix the settings on CircleCI or transition to TravisCI if CircleCI is no longer an option.

@ashfurrow I found this on their website; under «pricing plan» for this org.

Builds will run in a single container.
1 container is free, forever.
Additionally, projects that are public on GitHub will build with 3 extra containers -- our gift to free and open source software

Super! That's a relief :sweat_smile:

@esttorhe that's enabled now (go chat-ops!) I can totally admin circle on my phone! We have a per org setting that I just enabled.— Marc O'Morain (@atmarc) November 19, 2015

This is fixed now; closing it.


pns_idx referenced before assignment, raw.plot_psd()

Hi,
I'm reading in EGI mff data with an ECG lead.  When I try to raw.plot_psd() I get the following error
/mne-python/mne/io/egi/egimff.py in _read_segment_file(self, data, idx, fi, start, stop, cals, mult)
571                     s_end = s_start + samples_read
572
--> 573                     data_view = data[n_data1_channels:, s_start:s_end]
574                     _mult_cal_one(data_view, block_data[:n_pns_channels],
575                                   pns_idx,
UnboundLocalError: local variable 'pns_idx' referenced before assignment
I can make the plot if after I:
del raw._raw_extras[0]['pns_names']
because that's the variable whose length is checked in that case, however this is likely not the best solution.

Is it easy for you share a small file that we can use to reproduce locally?

Attached is a smallish file.
raw = mne.io.read_raw_egi('_20170725_094007_mrr_qrs_obs.mff')
raw.plot_psd()
should reproduce the error (in '0.15.dev0').  The actual cap only has 128 channels so 129 is 'dead', still tracking down why the EGI file comes out that way...  There are 4 'stim' channels that show up too, which were originally events...
(what is the best way to debug these modules? I have a few other things I'm interested in adding but my print statement additions and module reload as a quick check to see what's happening don't seem to work... will try pdb next..)
Thanks.
test_data.zip

will try pdb next.

This is how I usually do it. You can use an IDE to set debug points, or if/when you hit an error do post-mortem debugging with import pdb; pdb.pm()

So I think the bug is on ~line 518 in egimff.py.  The pns data is read regardless of whether it is required ( or in chs_to_read => pns_chans is empty and pns_idx doesn't get assigned. )  Adding the check for len(pns_idx) (or just using that as the switch) on this line should fix things - at least it does for me in this instance.
if 'pns_names' in egi_info and len(pns_chans) > 0:

That fix sounds reasonable to me, can you make a PR to fix it?


Casting to str() required by widgets.Select()

The options argument of widgets.Select() only accepts a list of strings. If a list of integers is used, it returns:

TraitError: The 'selected_label' trait of a Select instance must be a unicode string, but a value of 0 <type 'int'> was specified.

I think it is fair to not do too much magic for Select. Logically the right container to use for Select is a dictionary with keys being strings, and corresponding value to use, so that you can pass functions, or any other object. It's not that much code much of the time {str(v):v for v in container} and prevent many mistakes.

@arisolt do you have a use case where the solution provided by @Carreau does not work?  Or a reason to argue for the change?

I was asked to add this ticket by one of the dev's in the Gitter room, actually

On 22 Feb 2016, at 21:57, Jonathan Frederic<EMAIL_ADDRESS>wrote:
@arisolt do you have a use case where the solution provided by @Carreau does not work? Or a reason to argue for the change?
—
Reply to this email directly or view it on GitHub.

I was asked to add this ticket by one of the dev's in the Gitter room, actually

Yes that was me, I wanted to be able to discuss it here

I've been given some thought to @Carreau 's suggestion, and I think it makes sense.
I think we are all on the same page. Basically my concern is that there is no straight-forward way in the current version to select from a custom list of integers (only from an automatically generated list with fixed increment). This is necessary for functions that can only take something like [1, 7, 92].
If SelectionSlider will be included in the next version, there probably isn't a reason to also alter the logic of Select.


Internal Downgrade should be to FREE_NO_PLAN

Description
Task: https://github.com/dust-tt/dust/issues/4286
When we downgrade a workspace, it needs to be downgraded to FREE_NO_PLAN and not FREE_TEST_PLAN.
Two ways to be downgraded:
1/ The user ends their subscription to the PRO plan (during of after trial period) -> Nothing to do for me, the Stripe webhook ends the Subscription on db and does not create a new one so we're on default in FREE_NO_PLAN
2/ Someone from Dust wants to downgrade a workspace that was in Pilot or Friends Plan -> Edit Poké so that the poké downgrade route puts the workspace on FREE_NO_PLAN.
So, done on this PR:

Edit Poké downgrade route front/pages/api/poke/workspaces/[wId]/downgrade.ts to call internalSubscribeWorkspaceToFreeNoPlan over internalSubscribeWorkspaceToFreeTestPlan
UX simplification: Rename the Poké button "Invite or Upgrade to plan" to "Manage Plan" and put alls plan edition button inside it. This allowed me to remove the "Legacy workspace action" dropdown.
Remove DELETE from front/pages/api/w/[wId]/subscriptions/index.ts along with downgradeWorkspaceToFreePlan that does not seem used anywhere.

Risk
/
Deploy Plan
Nothing special.

re Deploy plan:
This can't really be merged until we have rolled out the subscribe page and hence the free trial right?


refactor: make NOMT values raw Vecs, not Rc

We previously wanted to avoid cloning these values while updating RocksDB. This is no longer necessary, and it incurs no performance overhead to take these Vecs by-value, since we don't clone them at all. In fact, this PR removes a deep clone.
The impetus for this change is to alter NOMT's API to support threaded access to Sessions.

#435
#434  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rphmeier and the rest of your teammates on  Graphite

#435
#434  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rphmeier and the rest of your teammates on  Graphite

Merge activity

Oct 9, 10:43 AM EDT: A user started a stack merge that includes this pull request via Graphite.


Make ServiceBusReceivedMessage immutable

Fixes #11782

Can one of the admins verify this patch?

/azp run net - servicebus - tests


Add answers in theory.md

Added answers in topic neural networks and neural networks for computer vision.

Thank you for your contribution!


Warn user about incompatible Java / Scala versions

Questions about compile errors which are the result of a combination of a too new Java version and a too old Scala version come regularily in the chat and issues. We could maintain a simple matrix for minimal required Scala versions per Java version and log a warning or add an additional message if the compilation fails, to guide the user.
The comfigured Scala version {version} is known to be not compatible to JVM {version}.
Please update to at least Scala {version} by overriding:

  def scalaVersion = "{version}"

Hi @lefou, This is my first attempt at contributing, so please excuse me if my questions seem trivial.

For defining the compatible versions matrix, can i rely on Scala JDK compatibility docs or is there any other resource that we are expected to refer to?

Can this problem be addressed by introducing a check task in the build.sc file that runs before the compile task?

Thanks in advance !

Hi @saitejaswinisamala, welcome to Mill.

For defining the compatible versions matrix, can i rely on Scala JDK compatibility docs or is there any other resource that we are expected to refer to?

I think it's a good start. We can deviate if needed. Ideally you add a link to that page into a comment, so later maintainer have the origin source at hand.

Can this problem be addressed by introducing a check task in the build.sc file that runs before the compile task?

A check task would be nice. To be helpful, the warning is show close to a potential compilation error, which we can't guarantee if it's a separate task though. You could let this task return some message, which you also append to any potentially error message from the compile task.


Wallet import- change the error message when wrong password entered

User flow
As a user, when I load an existing wallet to thyra and that I entered the wrong password, I want to see an error message that allows me to know that my password is
How to
Change current error message "Error while creating your wallet. Try again" to "Wrong password. Try again"

The confirm dialog on delete account is missing, the associated PR adds it and display proper message on cancel


Thank you very much for sharing your work. I am asking for the weights.thank you

Thanks for your interest in our work. I am uploading the weights for our best network (OF-Horizontal), but it is taking more than expected. As soon as I have the link, I'll let you know here.

You can find a temporal link to the weights here: http://<IP_ADDRESS>/weights/AUNets


build tags in Azure Pipelines

Hi Cameron. You can create a build/release pipeline with a branch filter that looks like this (or customized to match your release tag format): refs/tags/*
https://twitter.com/dstaheli/status/1043635525918380033

The branch, tags, and pull requests are not building with:
trigger:
  - refs/heads/*
  - refs/tags/*

I know that in Teamcity, the branch spec is this:
+:refs/heads/*
+:refs/pull/*/merge
+:refs/tags/*

This made this branch and pull request build again, but the tag is not creating a build.
trigger:
  - +:refs/heads/*
  - +:refs/pull/*/merge
  - +:refs/tags/*

@lkillgore it is set to override! I don't remember setting that. May that is one by default. I'll unset that and try again.

It is still not building off of tags in this build-tags branch. I'll merge to master anyway and see if that makes a difference. I've tried by doing a release on github.com named 0.0.7. I also tried by simply pushing a git tag:
$v="0.0.6"
git tag $v
git push origin $v

After testing, I clean up with:
git tag -d $v
git push --delete origin $v


getInputFluid/getOutputFluid on Create Basin won't change when the content in the basin changed.

Describe
getInputFluid/getOutputFluid on Create Basin won't change when the content in the basin changed.
Steps to reproduce

Place a computer
Place a basin next to the computer
Put a bucket of water into the basin
open the computer, and typebasin = peripheral.find('basin')
basin.getInputFluid()

It will print two minecraft:empty

Multiplayer?
No
Version
Own compiled build.
Minecraft, Forge and maybe other related mods versions
Forge 43.2.8 Minecraft 1.19.2
Screenshots or Videos
No response
Crashlog/log
No response

The reason is getInputFluid used getRenderedFluid(), but for some reason it seems won't update during the game.


Have router.start() return promise

It would be nice for terse testing if router.start() returned a promise that completed upon first-route resolve.
window.location.hash = '/tos'
await router.start()

I think that might be nice.

Currently start() returns this, so a change would be a major version change. It also only addresses the convenience of the first time routing.
What if instead there was a router.routing promise that refers to the current route completion promise. While it isn't routing, it would be null. What do you think?

I don't see value in start() returning this. Chaining would happening earlier (.get, .post, etc).
Terse testing win:
await router.start({url})
// check stuff
router.stop()


Unhandled exception: AnalysisException

I'm getting an unhandled exception while trying to make the core libraries strong-mode clean.
The way I work:
mv out/ReleaseX64/dart-sdk/lib/math out/ReleaseX64/dart-sdk/lib/math_copy
ln -s $PWD/sdk/lib/math out/ReleaseX64/dart-sdk/lib/math

And then run the analyzer on the sdk sources.
Until today this made it possible to work on the core libraries without needing to rebuild. It seems this workflow has been broken recently.
$ out/ReleaseX64/dart-sdk/bin/dart --package-root=$PWD/out/ReleaseX64/packages /mnt/ssd/floitsch/code/dart/sdk/pkg/analyzer_cli/bin/analyzer.dart --no-hints --strong --warnings out/ReleaseX64/dart-sdk/lib/math/math.dart
Analyzing [out/ReleaseX64/dart-sdk/lib/math/math.dart]...
Unhandled exception:
AnalysisException: Cannot compute DART_ERRORS for /mnt/ssd/floitsch/code/dart/sdk/out/ReleaseX64/dart-sdk/lib/math/math.dart
Caused by Mismatch in DeclarationResolver while resolving /mnt/ssd/floitsch/code/dart/sdk/out/ReleaseX64/dart-sdk/lib/math/point.dart
Found element with name "factor" at 1765, but expected offset of 1774
Path to root: DeclaredSimpleIdentifier, SimpleFormalParameterImpl, FormalParameterListImpl, MethodDeclarationImpl, ClassDeclarationImpl, CompilationUnitImpl
#0      RecursiveAstVisitor&ExistingElementResolver._mismatch (package:analyzer/src/generated/resolver.dart:3433:5)
#1      DeclarationResolver._findWithNameAndOffset (package:analyzer/src/generated/resolver.dart:2697:9)
#2      DeclarationResolver._findIdentifier (package:analyzer/src/generated/resolver.dart:2670:23)
#3      DeclarationResolver._getElementForParameter (package:analyzer/src/generated/resolver.dart:2741:12)
#4      DeclarationResolver.visitSimpleFormalParameter (package:analyzer/src/generated/resolver.dart:2543:34)
#5      SimpleFormalParameterImpl.accept (package:analyzer/src/dart/ast/ast.dart:8862:15)
#6      NodeListImpl.accept (package:analyzer/src/dart/ast/ast.dart:7663:20)
#7      FormalParameterListImpl.visitChildren (package:analyzer/src/dart/ast/ast.dart:4706:17)
#8      RecursiveAstVisitor.visitFormalParameterList (package:analyzer/dart/ast/visitor.dart:1587:10)
#9      FormalParameterListImpl.accept (package:analyzer/src/dart/ast/ast.dart:4702:15)
#10     MethodDeclarationImpl.visitChildren (package:analyzer/src/dart/ast/ast.dart:7170:18)
#11     RecursiveAstVisitor.visitMethodDeclaration (package:analyzer/dart/ast/visitor.dart:1737:10)
#12     DeclarationResolver.visitMethodDeclaration (package:analyzer/src/generated/resolver.dart:2517:35)
#13     MethodDeclarationImpl.accept (package:analyzer/src/dart/ast/ast.dart:7162:15)
#14     NodeListImpl.accept (package:analyzer/src/dart/ast/ast.dart:7663:20)
#15     ClassDeclarationImpl.visitChildren (package:analyzer/src/dart/ast/ast.dart:1896:13)
#16     RecursiveAstVisitor.visitClassDeclaration (package:analyzer/dart/ast/visitor.dart:1425:10)
#17     DeclarationResolver.visitClassDeclaration (package:analyzer/src/generated/resolver.dart:2234:34)
#18     ClassDeclarationImpl.accept (package:analyzer/src/dart/ast/ast.dart:1836:15)
#19     NodeListImpl.accept (package:analyzer/src/dart/ast/ast.dart:7663:20)
#20     CompilationUnitImpl.visitChildren (package:analyzer/src/dart/ast/ast.dart:2427:21)
#21     RecursiveAstVisitor.visitCompilationUnit (package:analyzer/dart/ast/visitor.dart:1449:10)
#22     CompilationUnitImpl.accept (package:analyzer/src/dart/ast/ast.dart:2420:15)
#23     DeclarationResolver.resolve (package:analyzer/src/generated/resolver.dart:2209:10)
#24     BuildCompilationUnitElementTask.internalPerform (package:analyzer/src/task/dart.dart:998:33)
#25     AnalysisTask._safelyPerform (package:analyzer/task/model.dart:329:9)
#26     AnalysisTask.perform (package:analyzer/task/model.dart:229:7)
#27     AnalysisDriver.performWorkItem (package:analyzer/src/task/driver.dart:276:10)
#28     AnalysisDriver.computeResult (package:analyzer/src/task/driver.dart:109:18)
#29     AnalysisContextImpl.computeResult (package:analyzer/src/context/context.dart:667:14)
#30     AnalysisContextImpl.computeErrors (package:analyzer/src/context/context.dart:615:12)
#31     AnalyzerImpl.prepareErrors.<anonymous closure> (package:analyzer_cli/src/analyzer_impl.dart:130:17)
#32     _PerformanceTagImpl.makeCurrentWhile (package:analyzer/src/generated/utilities_general.dart:212:15)
#33     AnalyzerImpl.prepareErrors (package:analyzer_cli/src/analyzer_impl.dart:128:30)
#34     AnalyzerImpl._analyzeSync (package:analyzer_cli/src/analyzer_impl.dart:164:5)
#35     AnalyzerImpl.analyzeSync (package:analyzer_cli/src/analyzer_impl.dart:123:12)
#36     Driver._runAnalyzer (package:analyzer_cli/src/driver.dart:575:34)
#37     Driver._analyzeAllImpl (package:analyzer_cli/src/driver.dart:216:30)
#38     Driver._analyzeAll.<anonymous closure> (package:analyzer_cli/src/driver.dart:159:14)
#39     _PerformanceTagImpl.makeCurrentWhile (package:analyzer/src/generated/utilities_general.dart:212:15)
#40     Driver._analyzeAll (package:analyzer_cli/src/driver.dart:158:27)
#41     Driver.start (package:analyzer_cli/src/driver.dart:139:32)
#42     main (file:///mnt/ssd/floitsch/code/dart/sdk/pkg/analyzer_cli/bin/analyzer.dart:11:11)
#43     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:259)
#44     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:148)

#0      AnalysisContextImpl.computeResult (package:analyzer/src/context/context.dart:671:7)
#1      AnalysisContextImpl.computeErrors (package:analyzer/src/context/context.dart:615:12)
#2      AnalyzerImpl.prepareErrors.<anonymous closure> (package:analyzer_cli/src/analyzer_impl.dart:130:17)
#3      _PerformanceTagImpl.makeCurrentWhile (package:analyzer/src/generated/utilities_general.dart:212:15)
#4      AnalyzerImpl.prepareErrors (package:analyzer_cli/src/analyzer_impl.dart:128:30)
#5      AnalyzerImpl._analyzeSync (package:analyzer_cli/src/analyzer_impl.dart:164:5)
#6      AnalyzerImpl.analyzeSync (package:analyzer_cli/src/analyzer_impl.dart:123:12)
#7      Driver._runAnalyzer (package:analyzer_cli/src/driver.dart:575:34)
#8      Driver._analyzeAllImpl (package:analyzer_cli/src/driver.dart:216:30)
#9      Driver._analyzeAll.<anonymous closure> (package:analyzer_cli/src/driver.dart:159:14)
#10     _PerformanceTagImpl.makeCurrentWhile (package:analyzer/src/generated/utilities_general.dart:212:15)
#11     Driver._analyzeAll (package:analyzer_cli/src/driver.dart:158:27)
#12     Driver.start (package:analyzer_cli/src/driver.dart:139:32)
#13     main (file:///mnt/ssd/floitsch/code/dart/sdk/pkg/analyzer_cli/bin/analyzer.dart:11:11)
#14     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:259)
#15     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:148)

I believe that what's happening here is that analyzer has failed to flush an element model that was invalidated by some change, and that it subsequently attempted to use that element model when re-creating a resolved AST. We've seen similar exceptions before, but have not yet been able to track down (a) whether that conjecture is accurate or (b) where it's happening.
You should be able to continue working if you force server to re-analyze everything.

How does one force the command-line tool to reanalyze everything?

You can't because it always reanalyzes everything; I just didn't notice you were using the command-line tool. I'm guessing that summaries have broken this.

Yes, I think this happens because we started to use SDK summaries in command-line analyzer.
But it seems that we should not do this when we are analyzing SDK itself. Actually it works, but only if SDK source correspond to summaries, which is not the case when you change SDK source and want to see it they can be analyzed. So, we should disable it for SDK sources.

I'm not sure if it's related, but containing Caused by Mismatch in DirectiveResolver and the like are now happening a lot. They also on user code as well as the SDK.
We've saw a few thousand instances of this problem via instrumentation for 1.16.0-dev.5.0.

Well, in case of modifying SDK and using summaries this exception is expected.
In other cases - it is not.
So, I guess it happens at the same position, but by a different reason.

https://codereview.chromium.org/1963023002

https://github.com/dart-lang/sdk/commit/e7b51f7bd127757c031b79435cbf15a5009cf923


How to redirect screen when in AR view.

I would like to know if this module supports react-native-router-flux code. because I want to redirect the user after tapping a button or object in the AR world to another screen(not webview).

Hi, unfortunatley this has not been developed yet, you are welcome to submit a pull request with your implementation.


feat(unstorage): add ttl support

Ttl support is being added to unstorage through https://github.com/unjs/unstorage/issues/262
For now the vercel-kv and redis drivers are supported, but as more support is added, this will start working for all of them.

Do you know if the adapter breaks if you use this with unstorage <1.9? Wondering if this is a breaking change or if it can be a minor change.

I guess this doesn't matter since since you can pass arbitrary arguments in Javascript without any errors. Can you add a new changeset with type set to "minor"?

Sorry I'm a little confused, this project isn't using changeset, what should I do ?

Thanks!