Feature/auto handle scale factor

Problem
It's confusing for users to have to come up with an appropriate scale factor that'll look good in the simularium viewer. Instead, simulariumio should be able to choose an appropriate scale factor based on the spatial positions.
Link to ticket
Solution
Based on manually trying on different scale factors and seeing what positional ranges looked bad, I decided the viewer's ideal range is 5-70, so I made a new constant for that. In each of the converters, I kept track of the ranges of the XYZ coordinates when spatial data is being processed, and then come up with a scale factor accordingly. I then scale radii, positions, subpoints, and box size accordingly. If a scale factor was provided, that will still be used instead of the calculated one, to maintain previous functionality.
Type of change

New feature (non-breaking change which adds functionality)
This change requires updated or new tests

Codecov Report
Patch coverage: 98.17% and project coverage change: +0.21 :tada:

Comparison is base (b2ab7fe) 92.55% compared to head (50bd3f2) 92.77%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #148      +/-   ##
==========================================
+ Coverage   92.55%   92.77%   +0.21%
==========================================
  Files         101      101
  Lines        4542     4706     +164
==========================================
+ Hits         4204     4366     +162
- Misses        338      340       +2

Impacted Files
Coverage Δ

simulariumio/cellpack/cellpack_converter.py
85.09% <85.71%> (+0.40%)
:arrow_up:

simulariumio/trajectory_converter.py
80.43% <95.00%> (+3.72%)
:arrow_up:

simulariumio/constants.py
100.00% <100.00%> (ø)

simulariumio/cytosim/cytosim_converter.py
97.29% <100.00%> (+0.21%)
:arrow_up:

simulariumio/data_objects/meta_data.py
97.43% <100.00%> (ø)

simulariumio/mcell/mcell_converter.py
99.48% <100.00%> (+0.03%)
:arrow_up:

simulariumio/md/md_converter.py
96.46% <100.00%> (+0.38%)
:arrow_up:

simulariumio/medyan/medyan_converter.py
98.19% <100.00%> (+0.17%)
:arrow_up:

simulariumio/physicell/physicell_converter.py
97.87% <100.00%> (+0.15%)
:arrow_up:

simulariumio/readdy/readdy_converter.py
100.00% <100.00%> (ø)

... and 10 more

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


docs(api): improve openapi documentation for dash, charts and queries

CATEGORY
Choose one

[ ] Bug Fix
[X] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[X] Documentation

SUMMARY
Add descriptions for endpoints and fields to improve API's documentation. Also added an improved check on allowed datasource_type checks.
Chart POST schema:

Dashboard POST schema:

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS

Codecov Report

Merging #9724 into master will decrease coverage by 11.81%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #9724       +/-   ##
===========================================
- Coverage   70.72%   58.90%   -11.82%
===========================================
  Files         584      402      -182
  Lines       30382    12564    -17818
  Branches     3112     3112
===========================================
- Hits        21487     7401    -14086
+ Misses       8781     4979     -3802
- Partials      114      184       +70

Flag
Coverage Δ

#cypress
?

#javascript
58.90% <ø> (ø)

#python
?

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

Thks @villebro!


Tests fail on Unix (Linux and Mac OS X)

Hi,
Tests fail UNIX systems, see https://travis-ci.org/pdesgarets/vsts-rm-extensions/builds/364061762
Thanks

#417

@pdesgarets The issue is fixed now.

Thanks ! Do you know when it will be released on VSTS public website ? It may solve https://developercommunity.visualstudio.com/content/problem/230303/artifacts-selection-for-release-works-only-on-wind.html .

@pdesgarets yes to fix that issue we need some server side change also, which should be deployed in 3-4 days.

Great, thanks


Modified serialise and deserialise funcitons

Hi Hrithik, I think we can close this and open a new one since we already completed the serialization and deserialization functions in another PR


Indicate which settings are required/optional in meltano config <plugin> list

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2395
Originally created by @DouweM on 2020-10-28 23:38:07

We already show asterisks (*) next to required fields in the UI, based on the plugin's settings_group_validation property, like tap-gitlab's:
    settings_group_validation:
      - ['api_url', 'groups', 'start_date']
      - ['api_url', 'projects', 'start_date']

The array can contain one or more lists of setting names, each of which is a combination of settings that constitute a valid/complete configuration. In the case of tap-gitlab, this means that api_url and start_date are always required, and that groups, or projects, or both, but not neither, should also be set.
In the UI, we only show an asterisk next to settings that are always required (so not groups or projects), but don't enable the "Save" button until at least one group matches completely. This means that it's not clear to the user what's wrong when all of the apparently required fields are filled in, while the "Save" button is still disabled, but this hasn't caused an issue yet because it's usually clear to users that they have to pick a specific group or project.
In the output of meltano config <plugin> list, we can easily display asterisks where appropriate when there's only a single group of valid setting names, but I'm not sure yet what to do about groups and projects in the example above.

View 2 previous comments from the original issue on GitLab

Related to https://github.com/meltano/sdk/issues/1519

Related:

https://github.com/meltano/meltano/issues/6724


Doesn't Work

Operating system / операционная система
Windows 11
Running as service / Запуск программы как сервис
I installed it as a service / Установил как сервис Windows
Describe the issue / Опишите проблему
Goodbyedpi v0.2.1 is not working on Windows 11...
Additional information / Дополнительная информация
No response

That's not helpful, I can't understand the issue unless you provide at least some information.
http://www.catb.org/~esr/faqs/smart-questions.html

That's not helpful, I can't understand the issue unless you provide at least some information. http://www.catb.org/~esr/faqs/smart-questions.html

It does not work in Windows 11, What more information do you need? -_-

At least dump, used version, windivert version.

It does not work in Windows 11, What more information do you need? -_-

The program works on Windows 11, I've tested it on two desktops and in VM. There are no known global compatibility issues with Windows 11.
If it doesn't work in your case, please describe what exactly does not work, how do you start the program (in which configuration and with what options), whether did it work on other OS, have you tried to debug the issue.


Try new date formats on strptime valueError exception, Fixes yk4ever/…

Fixes yk4ever/py-unrar2/issues/14
Tested and confirmed by users of SickRage.

This patch has been working great for our users ( > 10k users)


Incorrect Compressed Public Key

For number 99 (private_key = bitcoin.sha256(b'99')) I receive this:
0383dfa2168a7b668641ed42ac5fcc4059a7206bb3f44043d46917a25b649a16
instead of:
030083dfa2168a7b668641ed42ac5fcc4059a7206bb3f44043d46917a25b649a16
that two zero at the beginning is missing

And for number 77:
pybitcointools result:
Compressed Public Key (hex) is: 0245dc1aea709e25d262e1afd00704fa53c9fef46946d3906ebeee75b318786a6
Right answer: 02045dc1aea709e25d262e1afd00704fa53c9fef46946d3906ebeee75b318786a6
another zero at the beginning is missing


#35 List of routes view

Add some routes via form to see routes on this view.

@juliaanholcer You can rebase the PR branch ;)

@jacol12345 working on it :-) There's nothing better than some conflicts early in the morning.

I wanted to make all of this on the computer with windows, and it was a really bad idea...


Command failed when used in VS Code plugin

Error mesage:
Command failed: /Applications/Visual Studio Code.app/Contents/MacOS/Electron XXX/node_modules/sync-fetch/worker.js

Can't figure out why this is happening. Took a look at sync-fetch/worker.js and couldn't make any progress.

Possibly related issue https://github.com/microsoft/vscode/issues/144094

Thank you for the report, I will take a look later. Just to make sure, it runs as expected when using it on MacOS outside of VS Code?

yes that's right it works when run on host machine. Possibly related to exec or execsync not being allowed by vs code node, if that's used under the hood?

I haven't been able to test on a Mac so I'm glad that works. Yes, execFileSync is used here: https://github.com/larsgw/sync-fetch/blob/8ddaaa4c86d62c95cf2f0947f4c927e5b776afd6/index.js#L1. I think most alternatives use it it something similar too, but I might be missing something.

Hey @TSMMark did you find another solution for this?
I noticed you had a similar issue on another sync request package. We have a custom eslint rule that uses sync fetching to fetch some data before parsing strings which is not working for the same error above. Running on mac, as an admin user makes no difference here

Are you also running ESLint from VS Code, or separately?

It fails only from VSCode, our build server works fine. Whilst I am sure this is a vscode issue, the issue referenced above appears to be closed for comments and says about running VS as an admin (typically windows).
Still keen to know if @TSMMark was able to find anything else out.

I was not able to find a workaround unfortunately. None of the sync fetch implementations work in VS code, and I couldn't figure out how to make a eslint fixer async.
I assume most async fetch packages would work in VS code, but I'm not sure if eslint visitor funcs or fixer funcs can be async... I did try it, but it didn't work. I could not find any docs or info on async funcs in eslint so I'm not sure if it's possible or not

@StuartMorris0 if you end up having any success please let me know I'd greatly appreciate it

@TSMMark you're correct the eslint rules can't be async, I too went down that rabbit hole for a little while.
I was unable to find a solution, I've disabled our custom eslint plugin for the time being as the sync-fetch error stops vscode eslint from reporting at all :(
I'll monitor the vscode updates in hope it's resolved there sometime.

There might be an alternative solution for Node v16 and later. I can try to make a development version if you would be interested in testing (otherwise I'll likely wait until Node v14 is EOL)

Good news and bad news: it works, takes just as long, and even Node.js v12 supports it. However, I think the only way to implement it is with something of the form while (!isFinished); which takes a lot more CPU than the execSync solution (1103 ms vs 7 ms of CPU time). You can try it at https://github.com/larsgw/sync-fetch/tree/worker_threads but I don't think it's very feasible.

It works perfectly! It's unfortunate that it uses more CPU but I'm personally fine with this tradeoff since the alternative is it just not working at all.
Maybe instead of fully replacing the internals with worker threads, it can be can mode which you can opt into via an options object for example. That way users would default to the more performant mode, but have the option if needed?
Thanks for your work on this!

I am not really sure of the effects of using while (!isFinished);. What you could do, depending on where you want to use this package, is install it directly from GitHub like so:
npm install git://github.com/larsgw/sync-fetch#worker_threads


Sum with axis full support

Directly support multiple axes + keepdims in C++. sum_axis, max_axis and min_axis can be depreciated. Refer to https://github.com/dmlc/mxnet/issues/2197 .
Also, the mx.nd.sum(a) will now return a NDArray with shape (1,). The previous sum operator implemented using Python will cast it to a float (See https://github.com/dmlc/mxnet/blob/master/python/mxnet/ndarray.py#L875-L878). I'm not sure whether this change will affect other packages like Minpy.

cc @jermainewang

Let us remove the deprecated ops after one week it get merged. In the future, let us always make the op naming convention be as compatible as possible as numpy

Why ndarray instead of float?

ndarray is async, so it won't force a sync point unless asscalar is called

thanks this is great! Can we do similar renaming for broadcast_axis, maybe numpy.broadcast_to?

@tqchen broadcast_axis is useful when we want to broadcast along a single axis, e.g, b = broadcast_axis(a, axis=1, size=100). I can add another broadcast_to operator that is similar to numpy.broadcast_to. We can also allow zeros in the shape argument, which means to keep the original dimension (like http://mxnet.readthedocs.io/en/latest/packages/python/symbol.html#mxnet.symbol.Reshape).


[AutoPR monitor/resource-manager] Support logicAppReceivers in actionGroup apis

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2511

Codecov Report

Merging #2006 into restapi_auto_monitor/resource-manager will increase coverage by 0.1%.
The diff coverage is 59%.

@@                           Coverage Diff                            @@
##           restapi_auto_monitor/resource-manager    #2006     +/-   ##
========================================================================
+ Coverage                                  53.04%   53.14%   +0.1%
========================================================================
  Files                                       4746     4736     -10
  Lines                                     118237   117496    -741
========================================================================
- Hits                                       62716    62448    -268
+ Misses                                     55521    55048    -473

Impacted Files
Coverage Δ

...onitor/azure/mgmt/monitor/models/voice_receiver.py
55.55% <ø> (ø)
:arrow_up:

...onitor/operations/metric_definitions_operations.py
93.93% <100%> (+0.18%)
:arrow_up:

.../monitor/operations/event_categories_operations.py
93.54% <100%> (+0.21%)
:arrow_up:

...zure/mgmt/monitor/operations/metrics_operations.py
91.3% <100%> (+0.19%)
:arrow_up:

...gmt/monitor/operations/activity_logs_operations.py
94.59% <100%> (+0.15%)
:arrow_up:

...mgmt-monitor/azure/mgmt/monitor/models/__init__.py
100% <100%> (ø)
:arrow_up:

...itor/operations/alert_rule_incidents_operations.py
30.18% <50%> (+2.73%)
:arrow_up:

...t/monitor/operations/metric_baseline_operations.py
25% <50%> (+2.58%)
:arrow_up:

...itor/operations/tenant_activity_logs_operations.py
37.14% <50%> (+1.84%)
:arrow_up:

...gmt/monitor/operations/action_groups_operations.py
16.77% <50%> (+4.1%)
:arrow_up:

... and 59 more

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 37b2aa7...12f460e. Read the comment docs.

Will rebuild Monitor directly


Add domain specification

Allow users to specify domains in addition to IPs.  See: https://github.com/charliesome/better_errors/issues/306

@manafire recently I run into issue that my web server binds to Unix domain socket and code fails when executing allow_ip?(env).
What do you think about changing order of the execution
if allow_domain?(env) || allow_ip?(env)

?
In that case I could solve my problem with:
BetterErrors::Middleware.allow_domain!('socket')

@szemek I have changed the order as you've requested.  Hope that works for you.

its possible merge this pull request ? =) i'm working with vagrant env and i'm using custom domain for that instead ips .

@Rubemlrm using a custom domain is not secure, as was discussed in #306. Please see the wiki for instructions on setting up vagrant correctly.

@RobinDaugherty  sorry for the duplicated error :/ i will check that later , and i will test with docker too , since i've discard the vagrant usage to use docker instead. But anyway thanks for the reply and support =)


Timesheet - blank 'My Timesheet' page

Sample
Timesheet
Author(s)
@aprildunnam
What happened?
Your Weekly Timesheet is a great tool. I am able to see the data is getting logged into 'TimeEntries' list. However, 'MY TIMEHSEETS' and 'APPROVALS' pages are completely blank.
I have downloaded your *.msapp file. I think this is the latest one from 4 months ago.
Could you please help to make above mentioned pages work?
Thanks in advance!
Sandeep
Steps to reproduce

Expected behavior
above mentioned pages are blank.

Hello! Did you solve the problem? i have the same issue

I'm unable to re-produce this problem. The Item filter on the My Timesheets screen is simple and filtering on Employee.Email = currentUser.Email. I would do some troubleshooting to see what the currentUser.Email property is outputting in your app and compare that to entires in the SharePoint list to make sure they are the same.

I'm unable to re-produce this problem. The Item filter on the My Timesheets screen is simple and filtering on Employee.Email = currentUser.Email. I would do some troubleshooting to see what the currentUser.Email property is outputting in your app and compare that to entires in the SharePoint list to make sure they are the same.


Add nerdctl support

Hey! We started using containerd as the runtime on our nodes instead of dockerd and would like to check whether it's possible to ship nerdctl as part of the EKS AMI so it's easier to install and manage container stuff on the node for debugging or any other purposes.
NerdCTL: https://github.com/containerd/nerdctl
Environment:

AWS Region: us-east-2
Instance Type(s): Graviton based
EKS Platform version (use aws eks describe-cluster --name <name> --query cluster.platformVersion): v1.20
Kubernetes version (use aws eks describe-cluster --name <name> --query cluster.version): v1.20
AMI Version: ami-09aac4079b301ef20
Kernel (e.g. uname -a):
Release information (run cat /etc/eks/release on a node):

I'm not in favor of adding tools like this to the EKS-optimized AMI, because it:

Enlarges our attack surface and support matrix
Isn't well-aligned with our charter, which is to support Kubernetes; not a general-purpose container environment. The upcoming removal of the docker command-line is related to this focus.

We'd be more likely to add crictl (though its UX admittedly needs improvement, and is beta software).
@mmerkes what's your opinion on this?

I'm going to close this; I recommend you take the container-based approach that @stevehipwell suggested.


[Win 7] Can't open SS Console

I installed on Windows 7 Ru. (See details: https://github.com/subutai-io/installers/issues/47). But failed to open the SS Console.

After some delay (~20 sec) I am getting this:

I restarted subutai vm instance twice and checked again. Same result.
Network settings:

Both of ping and telnet 8443 to <IP_ADDRESS> don't work.
Please let me know should I attach any logs or config files. Also we can organize TeamViewer session for remote debug.

yes, need karaf logs, ifconfig from RH

2016-06-12_install_log.txt
I checked RH: there is no management. I will reinstall and check again.

Yes, after reinstalling I could open the SS console successfully and register a peer to Hub.
We have to make the tray check if management is installed. For example, using some REST endpoint.
Otherwise, some new users will get same situation and be confused.


feat: enable GH actions to report reusing of deprecated reaction and metabolite identifiers

Main improvements in this PR:

Adjust sanityCheck script to detect the cases of reusing deprecated reaction and metabolite identifiers, as discussed in #265.

I hereby confirm that I have:

[X] Tested my code on my own computer for running the model
[X] Selected develop as a target branch

accepted


Remove warnings

Remove warnings.

Only merge after 134

Only merge after 134

@saadrahim , the tets passed. I think we can merge this. What do you think?

#138 contains this changes.


#61 - Migrate to Graphology

This PR:

Migrates from ngraph to graphology
Fixes node drag bug
Adds no overlap layout
Adds forceatlas2 layout
Removed lodash
Doc Cleanup

I get this error with the new version in nextjs 13:
- error Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/path_to_project/node_modules/graphology-layout/circular' imported from /path_to_project/node_modules/reagraph/dist/index.js
Did you mean to import graphology-layout/circular.js?
    at new NodeError (node:internal/errors:400:5)
    at finalizeResolution (node:internal/modules/esm/resolve:260:11)
    at moduleResolve (node:internal/modules/esm/resolve:879:10)
    at defaultResolve (node:internal/modules/esm/resolve:1087:11)
    at nextResolve (node:internal/modules/esm/loader:161:28)
    at ESMLoader.resolve (node:internal/modules/esm/loader:834:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:415:18)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:77:40)
    at link (node:internal/modules/esm/module_job:76:36) {
  digest: undefined
}

Did the previous version work?

Yes, it works

Thank you - was just trying to isolate the issue.

Any workaround?

Any workaround?

It should be fixed - let me know if not

@amcdnl I just generated a new NextJS project using t3 (https://t3.gg/) and installed the package: "reagraph": "^4.13.0",
Then I just followed the basics example, from here: https://reagraph.dev/?path=/docs/docs-getting-started-basics--docs
Giving me the following error:
Error: Cannot find module '/Users/max/dev/docker-vis/node_modules/graphology-layout/circular' imported from /Users/max/dev/docker-vis/node_modules/reagraph/dist/index.js
Did you mean to import graphology-layout/circular.js?


Fix exec magic number

I missed updating the exec magic number in the 5.2.0minus-1 merge (#106).

Yes, I think that's a good idea. I've repurposed this PR to do that.


[DO NOT MERGE] chore: upgrades to Zipkin Opentracing 2.x.

This PR adds Zipkin Opentracing 2.0 as a next step to support PHP 8.0.

Zipkin Opentracing 2.0 is released, @limingxinleo could you please update this PR accordingly?

I will merge it today.


[api-docs] Mark description as optional when creating commands (#5532)

A new commit was made into the api-docs repo: https://github.com/discord/discord-api-docs/commit/2b86bfe9e34dc7a8a0c5cdba81028b987d6ce76e
No details given.
This is a bot created issue.

as far as I could see is this already implemented?


update config and add new tests

update config to fix issue #1
add new tests to run all simplified scripts

any update ?


Deletion of pre-existing route folders in urara.clean()

https://github.com/importantimport/urara/blob/b5520f925fea031fc04d672dbb2f8a40fcddf48d/urara.js#L86-L96
I noticed that pre-existing folders in the routes directory are blown away by the urara.clean function, as you can see in the section of code highlighted above. Urara apparently just deletes all routes folders arbitrarily, without checking if it was a route that it created or not.
This means if I had a subfolder for photos, for example, after running the dev server urara would destroy them and not even realize it made a mistake. This has huge implications down the line. Unfortunately I don't have the bandwidth to draft a PR right now, but I'll still try to squeeze it in.
That being said: wonderful project you have here so far, I'm quite eager to try it in production as soon as possible 😉

🤔 hmm... my suggestion is to put all the folders in the /urara/ directory, but this really needs improvement.  I may update in the next few days, or welcome PR 👌

🤔 hmm... my suggestion is to put all the folders in the /urara/ directory, but this really needs improvement.  I may update in the next few days, or welcome PR 👌

I'll try to put something together this weekend 😅

I'll try to put something together this weekend 😅

progress: It is currently possible to delete all files copied from /urara/, but it is difficult to delete all empty directories.
const cleanFile = src =>
    fs.readdir(src, { withFileTypes: true }).then(files => {
      files.forEach(file => {
        const dest = path.join(src, file.name)
        file.isDirectory()
          ? cleanFile(dest)
          : file.name.startsWith('.')
            ? log('cyan', 'ignore file', dest)
            : rmFile(dest)
      })
  })
cleanFile('urara')

What if it kept track of all the files/dirs copied in the build step? Maybe in some sort of context or an array that can keep state? That would remove all the guesswork and be quite precise, all it would need to do is iterate through the array and delete items that match.

What if it kept track of all the files/dirs copied in the build step? Maybe in some sort of context or an array that can keep state? That would remove all the guesswork and be quite precise, all it would need to do is iterate through the array and delete items that match.

💀 It would be too complicated in this case.
the temporary solution is to delete all files and folders copied from /urara/, The disadvantage is that cannot use /src/routes/xxx/yyy.svelte and /urara/xxx/zzz.svelte.md at the same time (if both are placed in the urara directory, then they can)

Update:
I tried to implement the scanDir() function to create a list of files and so far it works fine for copying but removing is still hard to do. files are in kwaa/blog/urara.js

What's the problem that you're running into with file removal? I might have a chance to look at it this week.
Just after looking briefly at the code I had a few ideas jump out:

One thing you could do is get the flat list of everything from urara, then get a set of all of the dest files from it, and finally you'd need to scan the src/routes directory, and add any files that aren't in the dest set to your map from urara.
Another option would be to scan src/routes before a build, and create a routes.lock (or something) file with the current file structure. Then you could just load that during a clean and not delete anything in it. The main disadvantage of that route is you'd still delete anything that was created since the last clean
Is it possible to change the directory of the files that are copied from src/routes to src/routes/generated or something like that? In that case you could just avoid the whole scanning mess altogether.

What's the problem that you're running into with file removal? I might have a chance to look at it this week.
Just after looking briefly at the code I had a few ideas jump out:

One option would be to scan src/routes before a build, and create a routes.lock (or something) file with the current file structure. Then you could just load that during a clean and not delete anything in it. The main disadvantage of that route is you'd still delete anything that was created since the last clean
Is it possible to change the directory of the files that are copied from src/routes to src/routes/generated or something like that? In that case you could just avoid the whole scanning mess altogether.
Add a prefix/suffix to all copied files/dirs (maybe the current epoch time?) and write that out to a lockfile. Then you'd have an easy regex filter on your clean method.

edited: after re-reading, the first and second options I originally suggested were basically the same thing, just the first was more complicated.

Hello, my idea is to keep it simple while solving the problem, so the newest solution is to scan the files under the /urara/folder to generate a temporary index while copying/cleaning, and copy or remove them in depth order.
Copying goes well, the problem is with cleanup - when it starts to determine if the folder is empty, the files that need to be removed have not yet been removed.

Ah, interesting. Things have been crazy with work, but I think I'll have some time Thursday to look at it. My thought was to see if it'd be viable to create two arrays (one for dirs and one for files within the scan method. That way it should be possible to iterate through all of the files schedule those to be deleted async first, wait for them to all be deleted, then remove the directories.

So, I was playing around with the javascript file a bit, but decided to just start over from scratch using typescript, at least for the file management part just because it's a bit more familiar.  It's not the prettiest solution but it works and I generally just prefer the clarity of functions over assigning lambdas. I wrote the method from scratch, then I found node-dir and added a little wrapper around it to give depth info.
Anyways, here's a gist with the output (it's similar for both). There's some logging to clean up or move over to .debug in there.
https://gist.github.com/NullFragment/ccb1cfb5fda14d904f82e2f0ce09136d
The output of either method is similar, except that the node-dir one uses os-specific path separators (/ vs ).

So, I was playing around with the javascript file a bit, but decided to just start over from scratch using typescript, at least for the file management part just because it's a bit more familiar. It's not the prettiest solution but it works and I generally just prefer the clarity of functions over assigning lambdas. I wrote the method from scratch, then I found node-dir and added a little wrapper around it to give depth info, so there are options.
Anyways, here's a gist with the output (it's similar for both). There's some logging to clean up or move over to .debug in there. https://gist.github.com/NullFragment/ccb1cfb5fda14d904f82e2f0ce09136d
The output of either method is similar, except that the node-dir one uses os-specific path separators (/ vs ).
Filling out the rest of the copy and removal should be pretty easy, I just wanted to make sure you'd be comfortable with the solution before I finish implementing it.

Great work!
just don't really want to add node-dir to the dependencies, I'll try it in a few hours.

Yeah, I figured I’d leave both options there. One thing I thought of is adding a dry-run option for clean that would print out what it would delete before you go ahead and run it.

😥 Ok, now the problem comes back to deleting files...
The current remove function is still problematic, and I can't think of a better way to do this.

Sorry for the delayed response. I missed the message about removal. I have a few questions:

Are you trying to remove the dirs in parallel? If you are, you need to make sure you get a sorted list of the keys and go in reverse order since the dirs and files structures are organized by depth. So you'd have a serial loop that spawns parallel deletes at each depth, starting from the deepest.
Is it even worth parallelizing this? I'm not sure that there would be much benefit to parallelization until you hit a large number of files.
Is copying even necessary or could symlinks just be used for the directories?

Sorry for the delayed response. I missed the message about removal. I have a few questions:

Are you trying to remove the dirs in parallel? If you are, you need to make sure you get a sorted list of the keys and go in reverse order since the dirs and files structures are organized by depth. So you'd have a serial loop that spawns parallel deletes at each depth, starting from the deepest. The files shouldn't matter because they're not dependent on the hierarchy, but the dirs are.
Is it even worth parallelizing this? I'm not sure that there would be much benefit to parallelization until you hit a large number of files.
Is copying even necessary or could symlinks just be used for the directories?

Ah, no problem.

I reverse the Map and use forEach to remove. Since the remove function is not modified, the same problem does occur... I'm not very good with files.

hmm... It's just that I wanted to use fs/promises, but some blogs can indeed have very much files.

I haven't tried it. Maybe it's worth a try?


tests: drivers: pwm: api: run PWM tests in user space

Run the PWM API tests in user space.
Signed-off-by: Henrik Brix Andersen<EMAIL_ADDRESS>

@anangl, @nashif Any objections to this? This change provides us coverage of the PWM API from user space.


How not to have printing callbacks?

Is there a way of constructing a Learner without certain callbacks?
julia> Learner(predict, lossfn; callbacks = [Metrics(accuracy)]).callbacks
FluxTraining.Callbacks(FluxTraining.SafeCallback[Metrics(Loss(), Metric(Accuracy)), ProgressPrinter(), MetricsPrinter(), StopOnNaNLoss(), Recorder()], FluxTraining.LinearRunner(), {5, 5} directed simple Int64 graph, false)

julia> Learner(predict, lossfn).callbacks
FluxTraining.Callbacks(FluxTraining.SafeCallback[ProgressPrinter(), MetricsPrinter(), StopOnNaNLoss(), Recorder(), Metrics(Loss())], FluxTraining.LinearRunner(), {5, 5} directed simple Int64 graph, false)

Or at least, to remove callbacks after construction?

You can either create the learner with Learner(...; usedefaultcallbacks=false) to turn off all the included callbacks, or remove them with FluxTraining.removecallback!(learner, CallbackType) (see removecallback!).
In your case, using FluxTraining.removecallback!(learner, ProgressPrinter) should do the trick.

Thanks. Looking into where this would go into the docs now.


No way to create Page Objects

What are you trying to achieve? (Expected behavior)
Create instances of Page Objects similar to how Behat does it
What do you get instead? (Actual behavior)
No way to create Page Objects
Details

Php-webdriver version: 1.4.1
PHP version: 7.0
Selenium server version: 3.4
Operating system: Any
Browser used + version: Any

@josh-palan Hi, what exactly do you mean?
Creating page object is plain and simple - it could be a simple PHP class with passed webDriver instance. For example:
class MyPageObject
{
    /** @var RemoteWebDriver */
    private $webDriver;

    public function __construct(RemoteWebDriver $webDriver)
    {
        $this->webDriver = $webDriver;
    }

    public function getPageHeader()
    {
        $this->webDriver->findElement(WebDriverBy::cssSelector('h1.title'));
    }

    public function login($userName)
    {
        $this->webDriver->findElement(WebDriverBy::name(['login']))
            ->sendKeys($userName)
            ->submit();
    }

    ...
}

If you want some convenience layer (eg. with prepared abstract page objects, so you don't have to implement this yourselves), you can try some library providing this on top of php-webdriver, like https://github.com/lmc-eu/steward (see page object example here) or http://codeception.com/docs/06-ReusingTestCode#PageObjects.
Is this what you've been looking for?

Yes and no. Unfortunately, I've tried quite a few libraries that attempt to provide page objects on top of php-webdriver, but they aren't compatible with the facebook/php-webdriver.
What I was hoping for is a php implementation of the Selenium page object java class, including their implementation of a page factory, so that I could use things like annotations as a way to instantiate the page's properties.

Are there any updates on this? Any plans on implementing native page objects?

@josh-palan IMO this is out of scope of php-webdriver. Its aim is to provide low layer OOP abstraction over the Selenium protocol(s) and access to features of Selenium.
Extensions like PHPUnit integration, Page Objects, syntax sugar etc. should be done in other libraries. Checkout some of them (all built on top of php-webdriver):

https://github.com/lmc-eu/steward
https://github.com/Codeception/Codeception
https://github.com/laravel/dusk

to see if some fits your needs or if they provide some built-in Page Objects abstraction (or maybe fill issue there - it may be interesting extension for them).
Thanks and happy testing :)

@OndraM I understand wanting to stick to only things that are included in Selenium by default, and just creating abstractions over existing features, however, page objects are a default feature in Selenium


USDR Superuser has ability to update ARPA output template from the web application

Context
Updating the arpa output template is a common task that may need to happen at the end of a quarter. This is something that needs to be accomplished and deployed within a short period of time in order to unblock partners.
This is the output template equivalent of this issue
Current State
Each time we need to update an output template we'll need to make a PR similar to this #1274, following instructions here . This is ends up making the CI/CD pipeline and engineering resources a dependency to getting partners an updated input template.
Future State
When a new ARPA output template is available. An Admin/super-user can go into the ARPA tool and update the template version by uploading the new file.
Design
@caitlinwinner 's team to confirm this-
There will likely be two new pages added.

Template form upload page

User visits /arpa_reporter/update-output-template-version
Page displays a form to upload a file
When a file is uploaded, there is a view to confirm the new file version as well as the updated outputTemplates.json
If the user is satisfied with the changes they can apply the new version for all partners.
(Note we may want to combine this with the UI for the input template from https://github.com/usdigitalresponse/usdr-gost/issues/1267)

Template form view page

User can also visit a new page /arpa_reporter/current-output-template-version
This page will display the current version name as well as the outputTemplates.json file for the admin to inspect any issues.

Engineering
To Add

arpa-output-template-files S3 bucket to hold the output template files
arpa-output-template-rules S3 bucket to hold the outputTemplates.json files. Note: we will continue to hold all previous versions of this file.
arpa_templates Database table to manage template records

DB Schema
arpa_templates

id : pk int
external_id : uuid that can be used while saving file to s3
input_filename : varchar - points to a S3 key in arpa-output-template-files bucket
rules_filename : varchar - points to a S3 Key arpa-output-template-rules bucket
effective_at: datetime - the time at which point the file was made to be the latest arpa input template

/* to retrieve the latest arpa template */
SELECT      *
FROM         arpa_templates
ORDER BY effective_at DESC
LIMIT          1;

To Update

Code that refers to outputTemplates.json will now need to retrieve this data from S3
Code that refers to the output templates will now need to retrieve it from S3.

New Endpoints

POST /upload-new-output-template-version

Validates that user is bumping the version up and checks that the name follows known format.
Adds file to arpa-output-template-files S3 bucket
Adds a record in arpa_templates with the current_time as the effective_at
Runs the same script as yarn workspace server run generate-rules to update outputTemplates.json and upload the file to S3.
Redirects to/arpa-output-template-version/latest upon success.

GET /arpa-output-template-version/:versionId

If a :versionId is not supplied default to latest or most effective version of the template
Returns { inputTemplate: <signed S3 URL to download template>, templateRules: <JSON from S3> }

@as1729 we need to think through how we would combine this with the input template paths/routes /db/ui


Skip debase extconf.rb on non-CRuby

Related: https://github.com/ruby-debug/ruby-debug-ide/pull/240

Ping, I'd like to merge this.

@hurricup Could you merge this?
See for instance https://github.com/oracle/truffleruby/issues/3359, that needs both ruby-debug-ide and debase to no-op on TruffleRuby and other non-CRuby.

I'm sorry, I still don't understand what are you trying to achieve? This gem is not intended to be used with truffleruby as well as ruby-debug-ide.

what are you trying to achieve?

To let the gem install fine on truffleruby, so there is no need to tweak the Gemfile.
In some cases it is (AFAIK) not possible to tweak the Gemfile, we had such a case internally at Oracle:
Suppose there is a gem my-dev-tools that depend on ruby-debug-ide (but not only, also other gems which do work on truffleruby) and that's part of the Gemfile.
There is no solution there, because removing my-dev-tools would remove too much and break the app/tests.
Using something like gem 'debase', platforms: :mri does not work.
The only solution is to let ruby-debug-ide (and its dependencies as known by RubyGems, so that's debase) install fine on TruffleRuby and noop at runtime.
It's a fairly common practice to let gems install on TruffleRuby/JRuby/etc even when they are not supported, so at least they don't break bundle install. Some examples are stackprof, bootsnap, etc.


How to define array of models in @Returns?

Information

Version: 5.x
Type: Issue/Story

Description
@Returns(200, { description: "ok", type: UserInfo})

I expect array of UserInfo to be sent. How to define it?
Currently my api returns an array but in swagger it shows object as an response.

Maybe ReturnsArray :)
https://tsed.io/tutorials/swagger.html (section decorators)
I agree isn’t evident :)
See you

@Romakita  It worked. Thanks a lot. May be we should add this in the example.

@Romakita  It worked. Thanks a lot. May be we should add this in the example.

Yes :)


Fix an issue preventing OAuth2 authentication on ArcGIS Enterprise servers

@Mergifyio backport release/4.4.x


Installation on windows requires VS2015 redist

Steps to Reproduce
I am working on a project to bootstrap development on fresh windows installs.   I ran into a problem which might be helped by better documentation:

open a new windows sandbox  (https://learn.microsoft.com/en-us/windows/security/application-security/application-isolation/windows-sandbox/windows-sandbox-overview)
run rye installer
Watch installer fail, with a missing .dll

I realize this is not something which can be prevented (unless rye is statically linked), but the installation instructions might want to mention this and where to find the required prerequisite (https://www.microsoft.com/en-us/download/details.aspx?id=48145)
After installation, if I uninstall the redistributable, rye refuses to run.  So, it is a requirement for both the rye installer and rye itself. However, the python installed by rye<EMAIL_ADDRESS>runs fine, so this is a requiement for rye only.
edit:  Was a bit late to realize that the rye installer and rye is the same executable so, there :)
Expected Result
installer runs
Actual Result
installer complains about a missing VCRUNTIME140.dll
Version Info
0.21.0, windows, x64
Stacktrace
No response

It surprises me that the Python version does not require it as it's sort of documented that it does in some cases at least: https://github.com/indygreg/python-build-standalone/blob/94de081efb139cc48a0cdb9c891374ad9b2cdfe2/docs/running.rst#windows

Interesting.  Firing up Python 3.11 here, and using sysinternals procexplorer, I see that it does indeed run
vcruntime140.dll, as well as over 40 other dlls, just in the interactive prompt.
However, the python 12 installed by rye seems to be differently linked, probably statically.  This is the dlls that it uses:

You are right that at the very least the documentation needs to provide this. I'm not entirely sure what the correct solution here is. Ideally one would not need to document it but the installer actually also ensures the redist is installed. Static linking does not seem ideal.

Well, Rust is known for creating standalone binaries, would be interesting to see how much the .exe grows when static linking.  Rust probably isn't using much of the c runtime...
I see that you made a PR, will be watching it.

Just checked running Rye in my sandbox, sysinternals reports that the only non-system dll it is running is inded vcruntime140.dll:

In that case I am not entirely opposed to statically linking vcruntime. Unfortunately I don't have my windows PC until next week again so I can't test that the PR actually fixes that.


ir: Skip function template earlier.

This fixes #364.
r? @emilio

:pushpin: Commit 912abe5 has been approved by emilio

I think this change is fine, but may not be the only case of that assertion triggering.
Will take a deeper look at this, meanwhile this is fine to merge.
@bors-servo r+

:zap: Test exempted - status


Add indexing documentation

Adds documentation to the core website around indexing:

Sync/async indexing
Bulk
Bulk helpers
Ingest nodes
Pipelines
Timeouts

Ported to master and 7.x


Add code execute button

Berkeley, I'm getting a strange assertion error. Are you seeing this on your end? It happens whenever I try to load a bonfire, and is preventing any other effort on my side as it just blocks any event handlers, properties, state, the whole shebang from being rendered
https://www.evernote.com/shard/s550/sh/c2740c2c-9766-435f-9cf4-e4647cc9992e/c0a439b3415d5454f318eae78706fe88

Nope. Geez, that is the most helpful error message in the world.


Chapter images fail to load sometimes

It just gets stuck on the "Loading from " screen. Maybe due to a timeout?

Fixed, in a partial way with #13. There's still more work to do around not ever reaching a timeout, but #13 is a good first step towards a better experience there.


fix backticks in inline code (fixes #312)

Fixes #312.
Some simple test cases are included in linked issue.

Closing as having merge conflicts, stale, tagging with #983


"minecraft:icon" component format has changed (1.20.60+)

minecraft:icon has a different format in format versions 1.20.60+
It is now:
"minecraft:icon": {
      "default": "custom_item"
}

Sorry, this is the wrong syntax
Correct one below:
"minecraft:icon": {
"textures": {
"default": "custom_item"
}
}


platoon_launcher vs platoon-launcher

Can I suggest platoon-launcher as the name of the executable (dash rather than underscore)? The underscore is necessary for modules because of the operator. For command-line stuff, the extra shift is annoying to type. This also keeps consistency with blocks-continue, fuel-convert, etc.

Sure, I have no objection.
Is there a linux/POSIX standard for that?

No, but http://unix.stackexchange.com/a/7371

I don't have a preference, but I agree, as it will be written frequently,
not having to press shift and for consistency, I think it would be good to
change it.
On Thu, Jan 28, 2016 at 8:49 PM, Bart van Merriënboer <
<EMAIL_ADDRESS>wrote:

No, but http://unix.stackexchange.com/a/7371
—
Reply to this email directly or view it on GitHub
https://github.com/mila-udem/platoon/issues/40#issuecomment-176516725.

Ok, changing it!


Add a protector test that demonstrates the base tag diagnostic

Per https://github.com/rust-lang/miri/pull/2519#issuecomment-1232736295, this demonstrates this case for protector diagnostics:
help: <3131> was created here, as a base tag for alloc1623
  --> tests/fail/stacked_borrows/invalidate_against_protector3.rs:10:19
   |
10 |         let ptr = std::alloc::alloc(std::alloc::Layout::for_value(&0i32)) as *mut i32;
   |                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

This diagnostic is inspired by what Miri used to do with https://github.com/rust-lang/rust/issues/60076#issuecomment-1214169179

The diagnostic cannot just happen for protectors, right? At least that function that computes it seems to be more generic than just protectors.

The diagnostic cannot just happen for protectors, right?

I think actually it can only happen for protectors. When the problem is that the used pointer has been invalidated, we print the creation span of the invalidated pointer, not the invalidating pointer. And since you can't invalidate the base tag, I don't think we'll ever print its span.
We also can't print the base tag's creation span for a use-outside-provenance issue, because the base tag is always valid for the whole allocation. And the base tag always has write permission.

I think actually it can only happen for protectors.

Ah, makes sense, thanks.

@bors r+

:pushpin: Commit 3cfb9915fcf97ac3f312d2277193d2c2cf934057 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 3cfb9915fcf97ac3f312d2277193d2c2cf934057 with merge 2d50443401d04becfa13ad350d2fba1b9819a686...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 2d50443401d04becfa13ad350d2fba1b9819a686 to master...


Add linux aarch64 support

Hi!
I've been really struggling to install the package in an EC2 instance that happens to have an aarch64 processor and is running ubuntu. Are there any plans to add aarch64 support for linux machines in the future?
Regards

Hi @leonardodiegues , currently we use github action and github hosted runners to manage our release process. So we cannot support aarch64 linux since it is not in the list here. For linux aarch64, you may need to build your own wheel file like mentioned in this doc

Thank you for your answer. I'll follow the docs build instructions!

The build instructions do not work out of the box, especially as it points to ever changing Rust nightly versions.
Lacking official Linux aarch64 support hinders the adoption of this library, as it is basically the version that every developer using a mac M1 and docker will end up needing for local testing. Could we have better instructions/support for that?

Hi @Sirsirious , thanks for the comment. For mac M1 we do have precompiled wheel file support. You can just run pip install connectorx to install it. So only linux users with aarch64 processor need to build connectorx from the source. We also tried to cross compile it in our release action (like what we did for mac m1) but still has some issues. A good news is that we have switched to rust stable since the GAT is stabilized !

Hi @wangxiaoying,
I am a big fan of this library, use it a lot for EDA in Jupyter Notebooks with Polars with big datasets >= 200GB. I want to now use it in a production env on Airflow! Unfortunately my whole team uses mac M1 and every team member builds a local a local dev airflow environment from a docker-compose file on a shared github repo!
I currently use polars but read in the data using pd.read_sql_query and then transform the data to a polars dataframe with  pl.from_pandas! I would prefer to use connectorX with Polars!
Below is the error log trying to install ConnectorX. Currently building from source for the whole team is currently not an option.
Is there any update on the support for aarch64 support?
 Downloading polars-0.16.5-cp37-abi3-manylinux_2_17_aarch64.manylinux2014_aarch64.whl (13.3 MB)
#0 11.85      ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 13.3/13.3 MB 14.2 MB/s eta 0:00:00
#0 11.89 ERROR: Ignored the following versions that require a different python version: 1.0.0 Requires-Python >=3.6, <3.9; 1.0.1 Requires-Python >=3.6, <3.9; 1.0.2 Requires-Python >=3.6, <3.9; 1.0.3 Requires-Python >=3.6, <3.9; 1.0.4 Requires-Python >=3.6, <3.9; 1.1.0 Requires-Python >=3.6, <3.9; 1.1.1 Requires-Python >=3.6, <3.9; 1.1.2 Requires-Python >=3.6, <3.9; 1.10.0 Requires-Python >=3.6, <3.9; 1.10.1 Requires-Python >=3.6, <3.9; 1.2.0 Requires-Python >=3.6, <3.9; 1.3.0 Requires-Python >=3.6, <3.9; 1.4.0 Requires-Python >=3.6, <3.9; 1.5.0 Requires-Python >=3.6, <3.9; 1.6.0 Requires-Python >=3.6, <3.9; 1.6.1 Requires-Python >=3.6, <3.9; 1.6.2 Requires-Python >=3.6, <3.9; 1.6.3 Requires-Python >=3.6, <3.9; 1.7.0 Requires-Python >=3.6, <3.9; 1.8.0 Requires-Python >=3.6, <3.9; 1.8.1 Requires-Python >=3.6, <3.9; 1.9.0 Requires-Python >=3.6, <3.9; 1.9.1 Requires-Python >=3.6, <3.9; 1.9.2 Requires-Python >=3.6, <3.9; 1.9.3 Requires-Python >=3.6, <3.9; 1.9.4 Requires-Python >=3.6, <3.9; 1.9.5 Requires-Python >=3.6, <3.9; 1.9.6 Requires-Python >=3.6, <3.9; 2.0.0 Requires-Python >=3.6, <3.9; 2.0.1 Requires-Python >=3.6, <3.9; 2.1.0 Requires-Python >=3.6, <3.9; 2.2.0 Requires-Python >=3.6, <3.9; 2.3.0 Requires-Python >=3.6, <3.9
#0 11.89 ERROR: Could not find a version that satisfies the requirement connectorx==0.3.1 (from versions: 0.2.3)
#0 11.89 ERROR: No matching distribution found for connectorx==0.3.1
#0 11.89
#0 11.89 [notice] A new release of pip available: 22.3.1 -> 23.0
#0 11.89 [notice] To update, run: python -m pip install --upgrade pip
------
failed to solve: executor failed running [/bin/bash -o pipefail -o errexit -o nounset -o nolog -c pip install -r ./requirements.txt]: exit code: 1

I am also interested if this is on the roadmap, linux ARM64 support is very important for some of the work we are doing and it would be very nice if this feature was officially supported.

For mac M1 we do have precompiled wheel file support. You can just run pip install connectorx to install it.

This works great for local mac M1 development purposes, but when using Docker on M1 it does not work well since Docker on Mac is (ideally) Linux aarch64/arm64. (You can have Docker on M1 emulate x86_64 / amd64, but it's painful...)

Hi, my team has been attempting to retrieve a connector-x python wheel on aarch64 for an EC2 instance, and we've succeeded in building a v0.3.1 for the same by following the build instructions. I'll be pasting the Dockerfile used to retrieve the wheel file here so that it reduces the effort of having to set everything up from scratch.
# Retrieving the base image from manylinux 2014 because that's what connectorx
# 0.3.1 was built on.
FROM quay.io/pypa/manylinux2014_aarch64

# Installing devel dependencies
RUN yum install -y epel-release
RUN yum install -y mysql-devel freetds-devel postgresql-devel

# Creating and changing to a new directory
RUN mkdir /wheeler
WORKDIR /wheeler

# Installing and setting up rust
RUN curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y
ENV PATH="$PATH:/root/.cargo/bin"

# Installing just through cargo
#RUN "$HOME/.cargo/bin/cargo" install just
RUN cargo install just

# Installing python3.9.6 from source
RUN yum install -y wget
RUN wget https://www.python.org/ftp/python/3.9.6/Python-3.9.6.tgz
RUN tar -xvf Python-3.9.6.tgz
RUN cd Python-3.9.6 && ./configure --enable-optimizations
RUN cd Python-3.9.6 && make install
RUN pip3.9 install poetry

# Cloning the connectorx repo and switching to the 0.3.1 tag
RUN git clone https://github.com/sfu-db/connector-x.git
WORKDIR /wheeler/connector-x
RUN git checkout tags/v0.3.1

# Installing the nightly version of rust upon which connectorx was built
RUN cargo install nightly-2022-09-15
RUN cargo override set nightly-2022-09-15

# Installing maturin
RUN pip3.9 install maturin==0.12.1

# Building the python wheel through maturin
RUN maturin build -m connectorx-python/Cargo.toml -i python3.9 --no-sdist --release --manylinux 2014

# Copying the wheel into the host system
COPY /wheeler/connector-x/connectorx-python/target/wheels/connectorx-0.3.1-*.whl .

# The wheel should be on your system at this point.

Hope this helps someone out with the build.
Please let me know if I can help out with sending a PR or something :D

Lovely stuff @dat-adi, thanks for sharing!
Note that if you're looking for "pip install connectorx" in an arm-based Dockerfile, you can use adi's Dockerfile as a build stage in a multi-stage build to accomplish this.
When naming the stage connectorx-builder as such:
FROM quay.io/pypa/manylinux2014_aarch64 as connectorx-builder

You will be able to copy and pip install connectorx in another stage of your Dockerfile:
COPY --from=connectorx-builder /wheeler/connector-x/connectorx-python/target/wheels/connectorx-0.3.1-*.whl ./
RUN pip install connectorx-0.3.1-*.whl

(Small side note: I had to install and configure the correct nightly version of rust the following way, as opposed to the cargo commands mentioned above)
RUN rustup install nightly-2022-09-15
RUN rustup override set nightly-2022-09-15

When referring to these docs here, I'm trying to find the right rust version as stated to "search for rust". Everything above references night builds but is it now just the "stable" version of rust?

Yes, just recently built from source using the above method described. Only now you don't need to specify the nightly anymore! Stable works.
So you can remove these lines:
RUN rustup install nightly-2022-09-15
RUN rustup override set nightly-2022-09-15

@duvenagep and @dat-adi Thanks for putting out this information. I tried building wheels using the docker specks you provided but I'm running into the issue below. Any suggestions? Do you perhaps have an updated script for version 0.3.2?
> [18/19] RUN maturin build -m connectorx-python/Cargo.toml -i python3.9 --no-sdist --release --manylinux 2014:
365.8
366.5 error: aborting due to previous error
366.5
366.5
366.5 For more information about this error, try `rustc --explain E0554`.
366.5
366.5 error: could not compile `connectorx-python` (lib) due to 2 previous errors
366.5 💥 maturin failed
366.5   Caused by: Failed to build a native library through cargo
366.5   Caused by: Cargo build finished with "exit status: 101": `cargo rustc --message-format json --manifest-path connectorx-python/Cargo.toml --release --lib --`
------
Dockerfile:42
--------------------
  40 |
  41 |     # Building the python wheel through maturin
  42 | >>> RUN maturin build -m connectorx-python/Cargo.toml -i python3.9 --no-sdist --release --manylinux 2014
  43 |
  44 |     # Copying the wheel into the host system
--------------------
ERROR: failed to solve: process "/bin/sh -c maturin build -m connectorx-python/Cargo.toml -i python3.9 --no-sdist --release --manylinux 2014" did not complete successfully: exit code: 1

Hi @vnijs,
When I tried to compile the binary a few weeks ago, it was working fine. Could you check to see whether adding the nightly flags fixes the issue?
It seemed to be necessary on my end, and did not work as expected without it. Also, check whether you can retrieve a logfile or a trace.

When I add nightly stuff I get.
#21 [17/21] RUN cargo install nightly-2022-09-15
#21 0.073     Updating crates.io index
#21 0.305 error: could not find `nightly-2022-09-15` in registry `crates-io` with version `*`
#21 ERROR: process "/bin/sh -c cargo install nightly-2022-09-15" did not complete successfully: exit code: 101
------
 > [17/21] RUN cargo install nightly-2022-09-15:
0.073     Updating crates.io index
0.305 error: could not find `nightly-2022-09-15` in registry `crates-io` with version `*`
------

So fully the script you initially provided
# Installing the nightly version of rust upon which connectorx was built
RUN cargo install nightly-2022-09-15
RUN cargo override set nightly-2022-09-15

Update: Replace the cargo commands with rustup gets me a bit further (see below what I added and what I commented out).
Unfortunately I'm not sure where the wheel is now because I get the below. Any suggestions, please let me know. Thanks
#25 [21/21] COPY /wheeler/connector-x/connectorx-python/target/wheels/connectorx-0.3.1-*.whl .
#25 ERROR: lstat /tmp/buildkit-mount562135541/wheeler/connector-x/connectorx-python/target/wheels: no such file or directory
------
 > [21/21] COPY /wheeler/connector-x/connectorx-python/target/wheels/connectorx-0.3.1-*.whl .:
------
Dockerfile:48
--------------------
  46 |
  47 |     # Copying the wheel into the host system
  48 | >>> COPY /wheeler/connector-x/connectorx-python/target/wheels/connectorx-0.3.1-*.whl .
  49 |
  50 |     # The wheel should be on your system at this point.
--------------------
ERROR: failed to solve: lstat /tmp/buildkit-mount562135541/wheeler/connector-x/connectorx-python/target/wheels: no such file or directory
-----------------------------------------------------------------------
Docker build for connectorx was not successful
-----------------------------------------------------------------------
usage: sleep seconds

RUN rustup install nightly-2022-09-15
RUN rustup default nightly-2022-09-15
# RUN cargo install nightly-2022-09-15
# RUN cargo override set nightly-2022-09-15

I was able to extract the wheel but when I tried to install it I got the below. I had hoped this would work with Ubuntu 22.04 (aarch64). Any help on next steps would be much appreciated.
❯ pip install connectorx-0.3.1-cp39-cp39-manylinux_2_17_aarch64.manylinux2014_aarch64.whl
ERROR: connectorx-0.3.1-cp39-cp39-manylinux_2_17_aarch64.manylinux2014_aarch64.whl is not a supported wheel on this platform.

Below the platform I was hoping to install this wheel into:
❯ cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04.3 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.3 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy
❯ uname -m
aarch64

Got this working with connectorx 0.3.2, ubuntu 22.04, and python 3.11.7. See the code for the docker image below.
FROM arm64v8/ubuntu:22.04

# Installing devel dependencies
RUN apt-get update
RUN apt-get install -y \
    libmysqlclient-dev \
    freetds-dev \
    libpq-dev \
    wget \
    curl \
    build-essential \
    libkrb5-dev \
    clang \
    git

# Creating and changing to a new directory
RUN mkdir /wheeler
WORKDIR /wheeler

# Installing and setting up rust
RUN curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y
ENV PATH="$PATH:/root/.cargo/bin"

# Installing just through cargo
RUN cargo install just

# Installing python3.11.7 from source
RUN wget https://www.python.org/ftp/python/3.11.7/Python-3.11.7.tgz
RUN tar -xvf Python-3.11.7.tgz
RUN cd Python-3.11.7 && ./configure --enable-optimizations
RUN cd Python-3.11.7 && make install
RUN pip3.11 install poetry

# Cloning the connectorx repo and switching to the 0.3.1 tag
RUN git clone https://github.com/sfu-db/connector-x.git
WORKDIR /wheeler/connector-x
RUN git checkout tags/v0.3.2

# Installing maturin
RUN pip3.11 install maturin[patchelf]==0.14.15

# Building the python wheel through maturin
RUN maturin build -m connectorx-python/Cargo.toml -i python3.11 --release

# Copying the wheel into the host system
# the below didn't work for me
# COPY /wheeler/connector-x/connectorx-python/target/wheels/connectorx-* .

# use the below to access the wheel in /wheeler/connector-x/connectorx-python/target/wheels/
# docker run -it -v ./:/root your_user_name/connectorx /bin/bash

# then navigate to the directory below and copy the wheel to the home directory
# which is mounted to your current directory on your host OS
# /wheeler/connector-x/connectorx-python/target/wheel


implement function let Koalas DataFrame constuctor take Koalas Series

Koalas DataFrame constuctor should take Koalas Series https://github.com/databricks/koalas/issues/466

Codecov Report

Merging #470 into master will decrease coverage by 0.09%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #470     +/-   ##
=========================================
- Coverage   93.32%   93.22%   -0.1%
=========================================
  Files          29       29
  Lines        3773     3780      +7
=========================================
+ Hits         3521     3524      +3
- Misses        252      256      +4

Impacted Files
Coverage Δ

databricks/koalas/frame.py
94.87% <100%> (-0.09%)
:arrow_down:

databricks/conftest.py
92.85% <0%> (-3.58%)
:arrow_down:

databricks/koalas/utils.py
97.29% <0%> (-2.71%)
:arrow_down:

databricks/koalas/namespace.py
88.46% <0%> (-1.29%)
:arrow_down:

databricks/koalas/series.py
92.53% <0%> (+0.53%)
: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 685176c...3dde609. Read the comment docs.

Thanks! merging.


Change to use Builder instead of deprecated create()

I changed to use SqlBrite.Builder instead of SqlBrite.create() which is deprecated.

Thanks!


Implement schema index for dynamic imports

Todo

[ ] Tests
[ ] Implement code generator for the actual index

Codecov Report
Attention: Patch coverage is 78.84615% with 11 lines in your changes are missing coverage. Please review.

Project coverage is 95.81%. Comparing base (a22673a) to head (048bef9).

Files
Patch %
Lines

src/kio/static/index.py
78.84%
10 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #150      +/-   ##
==========================================
- Coverage   96.67%   95.81%   -0.87%
==========================================
  Files          14       15       +1
  Lines        1022     1074      +52
  Branches      139      140       +1
==========================================
+ Hits          988     1029      +41
- Misses         26       36      +10
- Partials        8        9       +1

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


Help with M-Bus Slave

Hello!
I used a diffrent M-Bus Slave board and wanted to know if anyone has any experience with the "replica board".

I have set up everything like the Hardware Installation in the read me recommends, but as a value in HA i just get "unknown". I already let it run for 1 hour, just to be sure. Now my question is: Might it be the board or something different I've done wrong. It seems like I don't receive any data.
Best regards Erik!

Hi,
I'm working on updating the component and eventual native integration into esphome.
You could try my version and see if it helps: https://github.com/SimonFischer04/esphome/issues/1

So as you got it working, this issue can be closed now?


(Bug) withdraw card should be always green

Description
Fix old withdraw feeds which doesn't have otplStatus field
so all feeds with type=withdraw should be green
Checklist:

[ ] PR title matches follow: (Feature|Bug|Chore) Task Name
[ ] My code follows the style guidelines of this project
[ ] I have followed all the instructions described in the initial task (check Definitions of Done)
[ ] I have performed a self-review of my own code
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I have added reference to a related issue in the repository
[ ] I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
[ ] I have added screenshots related to my pull request ( for frontend tasks)
[ ] I have pasted a gif showing the feature.
[ ] @mentions of the person or team responsible for reviewing proposed changes

test with feed without otplStatus


Incorrectly applied to CSS Selectors & comments

In css mode, the color picker is being applied to named css selectors:

and inside of comments:

hi @Mottie
This is a bug. I will fix the bug as soon as possible.

@Mottie
I released v1.0.9 for this issue.
please test for color.


Misc fixes/improvements and ability to retrieve certs from acm-pca

This improves the config file parsing a bit, fixes a crashing bug when hitting the "no config file" fallback, and adds the ability to retrieve certificates directly from acm-pca by ARN

Can you update the slots section of the README with the new ca_arn and certificate_arn options?

We'll probably also need an update on this line to have CI build the acm-pca SDK library.

Ok thanks, I'll update later today or tomorrow

Updated. Additionally, you can now ommit ca_arn as it's usually a substring of the certificate_arn so I added code to just extract it if unspecified

Thanks, changes look good to me! Merging!

@ozbenh Hmm, openssl doesn't seem to care for the *X509 parameters being marked as const. Did you hit this issue at all? Wondering if I should bump the version of openssl or if we need to drop those const qualifiers.
#!/bin/bash -eo pipefail
AWS_SDK_PATH=$HOME/aws-sdk-cpp make
Using C++ SDK static libraries
Using C SDK static libraries
g++ -shared -fPIC -Wall -I/home/circleci/aws-sdk-cpp/include -I/usr/include/p11-kit-1/p11-kit -I/usr/include/json-c -fno-exceptions -std=c++17 attributes.cpp aws_kms_pkcs11.cpp certificates.cpp unsupported.cpp debug.cpp aws_kms_slot.cpp \
    -o aws_kms_pkcs11.so  /home/circleci/aws-sdk-cpp/lib/libaws-cpp-sdk-kms.a /home/circleci/aws-sdk-cpp/lib/libaws-cpp-sdk-acm-pca.a /home/circleci/aws-sdk-cpp/lib/libaws-cpp-sdk-core.a /home/circleci/aws-sdk-cpp/lib/libaws-crt-cpp.a -Wl,--start-group /home/circleci/aws-sdk-cpp/lib/libaws-checksums.a /home/circleci/aws-sdk-cpp/lib/libaws-c-common.a /home/circleci/aws-sdk-cpp/lib/libaws-c-event-stream.a /home/circleci/aws-sdk-cpp/lib/libaws-c-auth.a /home/circleci/aws-sdk-cpp/lib/libaws-c-http.a /home/circleci/aws-sdk-cpp/lib/libaws-c-io.a /home/circleci/aws-sdk-cpp/lib/libaws-c-mqtt.a /home/circleci/aws-sdk-cpp/lib/libaws-c-cal.a /home/circleci/aws-sdk-cpp/lib/libaws-c-compression.a /home/circleci/aws-sdk-cpp/lib/libaws-c-s3.a /home/circleci/aws-sdk-cpp/lib/libaws-c-sdkutils.a /home/circleci/aws-sdk-cpp/lib/libs2n.a -Wl,--end-group  -lcrypto -ljson-c -lcurl
attributes.cpp: In function ‘CK_RV do_get_raw_cert(const X509*, CK_VOID_PTR, CK_ULONG_PTR)’:
attributes.cpp:221:29: error: invalid conversion from ‘const X509*’ {aka ‘const x509_st*’} to ‘X509*’ {aka ‘x509_st*’} [-fpermissive]
  221 |     CK_ULONG len = i2d_X509(cert, &buffer);
      |                             ^~~~
      |                             |
      |                             const X509* {aka const x509_st*}
In file included from /usr/include/openssl/ec.h:17,
                 from attributes.cpp:1:
/usr/include/openssl/x509.h:551:1: note:   initializing argument 1 of ‘int i2d_X509(X509*, unsigned char**)’
  551 | DECLARE_ASN1_FUNCTIONS(X509)
      | ^
attributes.cpp: In function ‘CK_RV do_get_raw_name(const X509_NAME*, CK_VOID_PTR, CK_ULONG_PTR)’:
attributes.cpp:231:34: error: invalid conversion from ‘const X509_NAME*’ {aka ‘const X509_name_st*’} to ‘X509_NAME*’ {aka ‘X509_name_st*’} [-fpermissive]
  231 |     CK_ULONG len = i2d_X509_NAME(name, &buffer);
      |                                  ^~~~
      |                                  |
      |                                  const X509_NAME* {aka const X509_name_st*}
In file included from /usr/include/openssl/ec.h:17,
                 from attributes.cpp:1:
/usr/include/openssl/x509.h:545:1: note:   initializing argument 1 of ‘int i2d_X509_NAME(X509_NAME*, unsigned char**)’
  545 | DECLARE_ASN1_FUNCTIONS(X509_NAME)
      | ^
attributes.cpp: In function ‘CK_RV do_get_raw_integer(const ASN1_INTEGER*, CK_VOID_PTR, CK_ULONG_PTR)’:
attributes.cpp:241:37: error: invalid conversion from ‘const ASN1_INTEGER*’ {aka ‘const asn1_string_st*’} to ‘ASN1_INTEGER*’ {aka ‘asn1_string_st*’} [-fpermissive]
  241 |     CK_ULONG len = i2d_ASN1_INTEGER(serial, &buffer);
      |                                     ^~~~~~
      |                                     |
      |                                     const ASN1_INTEGER* {aka const asn1_string_st*}
In file included from /usr/include/openssl/ec.h:17,
                 from attributes.cpp:1:
/usr/include/openssl/asn1.h:570:1: note:   initializing argument 1 of ‘int i2d_ASN1_INTEGER(ASN1_INTEGER*, unsigned char**)’
  570 | DECLARE_ASN1_FUNCTIONS(ASN1_INTEGER)
      | ^
make: *** [Makefile:172: aws_kms_pkcs11.so] Error 1

Exited with code exit status 2
CircleCI received exit code 2

On Wed, 2023-02-01 at 18:07 -0800, Ian Haken wrote:

@ozbenh Hmm, openssl doesn't seem to care for the *X509 parameters
being marked as const. Did you hit this issue at all? Wondering if I
should bump the version of openssl or if we need to drop those const
qualifiers.
Ah, I have worked around this for OpenSSL 1.0.2 in my amzn2 branch, we
can move that workaround to master and 1.1.1, I'll look into this when
I'm back home tonight or tomorrow. It's fine with OpenSSL 3.0

Cheers,
Ben.

Fixes in https://github.com/JackOfMostTrades/aws-kms-pkcs11/pull/24

Tagged a release v0.0.10 with these changes.

Thanks. Note that it gets quite slow with a few keys in there. In my case, it looks like the plugin is opened/closed often and getting to load all the certs from acm-pca is taking a while. I'll look at adding lazy loading of them. I'm also thinking of caching the SDK client objects when the region isn't changed from one call to the next.


yii\grid\CheckboxColumn with multiple=false doesn't work as expected.

yii\grid\CheckboxColumn with multiple=false doesn't work as expected.
I created a GridView with CheckboxColumn setting multiple=false as below:
echo GridView::widget([
    'dataProvider' => $dataProvider,
    'tableOptions' => ['class' => 'table'],
    'summary' => 'Exibindo {begin}-{end} de {totalCount} item(ns).',
    'id' => 'grid',
    'columns' => [
        ['class' => 'yii\grid\CheckboxColumn',
            'multiple'=> false,
        ],
        'id',
        'nome',
        ['class' => 'yii\grid\ActionColumn'],
    ],
]);

The CheckboxColumn still allows multiple select, but without the select in the head.
The result can be viewed attached.

I found this Issue https://github.com/yiisoft/yii2/issues/6639 that already treated the same question.
I'll analyze the solutions proposed.
Thank you everybody.


Consume from VCPKG

Add Sample which shows building an app with azure-sdk-for-cpp vcpkg dependency
Ideally sample is "template" starting point for users

It might make sense to add the ability to consume this from a "nightly" drop to a vcpkg fork so we can test against releases from master on any given day

It might make sense to add the ability to consume this from a "nightly" drop to a vcpkg fork so we can test against releases from master on any given day

@vhvb1989 if you plan on working on this, feel free to use the samples from here as a starting point: https://github.com/ahsonkhan/azure-sdk-ux-study-answers


Local Manifest

Is it possible to use Local Mainfest instead of Public?

That's not something that is supported right now, however, we are working on adding private repositories where you have full control over the manifests.

I second this. At a minimum it would be good to have the ability to point appget to a user defined git repository containing the application manifests.

as I explained in https://github.com/appget/appget/issues/24#issuecomment-464832669 there is quite a bit of work that goes into making the package repo work and it's not as simple as pointing appget to a different git repo (current app doesn't directly point to the appget.packages repo).
As I mentioned before, we are working on adding support for hosted private repositories, if you guys could share a bit about your use case we can make sure that they will be covered in our implementation.

I think you have three basic scenarios to consider.

Most organizations will likely expect to be able to keep local repositories either on a SMB/CIFS share. File shares might be Linux/Samba servers or actual Windows servers.
In some cases, such as automated deployment testing, it might be advantageous for AppGet to treat a local directory as a repository.
A public or private git server hosting specific YAML manifests.

Ideally, AppGet will have a simple configuration file that contains repositories of interest to the administrator. Also, the ability to filter out certain packages would be useful. For example, you have a product with a limited number of licenses described in an AppGet YAML manifest. It would be handy to ensure only one machine can see that package on an appget search and install it. Other machines either don't see that package or see one unique to them. Unfortunately, not everyone has site licenses.

appget looks interesting, but until private repos are available for production server workloads I'll be sticking with DSC+OneGet+ChocolateyGet+Chocolatey.


[Calling] Do not disable button on failed action

Purpose

Do not disable button on failed action. If camera turn on/off or switch is failed or mute on/off failed then do not disable button, so user may re-try

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

Pull request checklist
This PR has considered:

[ ] Color Theming
[ ] iOS supported versions (all major versions greater than or equal current target deployment version)
[ ] VoiceOver and Keyboard Accessibility
[ ] Internationalization and Right to Left layouts
[ ] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)
[ ] View Data Injection
[ ] Demo App UI/UX update

How to Test

Open...
Click on...

What to Check

Before
After

Screenshot or description before this change
Screenshot or description with this change

Other Information

how come your PR contains changes for hide lobby participants?

how come your PR contains changes for hide lobby participants?

There was one additional fix merged to develop (bugfix/skip_mute_merging_to_alpha is based on develop) to filter our also disconnected users, which changes lines around inLobby filtering.


Mask-rcnn torchscript doesn't load in c++: Unknown builtin op: torchvision::_new_empty_tensor_op.

I was trying to load a torchscript version of mask-rcnn into c++ but unfortunately this failed. Here is the error message / stacktrace I got:
./run_model ../torchscript.pt
terminate called after throwing an instance of 'torch::jit::script::ErrorReport'
  what():
Unknown builtin op: torchvision::_new_empty_tensor_op.
Could not find any similar ops to torchvision::_new_empty_tensor_op. This op may not exist or may not be currently supported in TorchScript.
:
  File "/home/user/.local/lib/python3.6/site-packages/torchvision/ops/new_empty_tensor.py", line 16
        output (Tensor)
    """
    return torch.ops.torchvision._new_empty_tensor_op(x, shape)
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
Serialized   File "code/__torch__/torchvision/ops/new_empty_tensor.py", line 3
def _new_empty_tensor(x: Tensor,
    shape: List[int]) -> Tensor:
  _0 = ops.torchvision._new_empty_tensor_op(x, shape)
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
  return _0
'_new_empty_tensor' is being compiled since it was called from 'interpolate'
Serialized   File "code/__torch__/torchvision/ops/misc.py", line 123
    align_corners: Optional[bool]=None) -> Tensor:
  _31 = __torch__.torchvision.ops.misc._output_size
  _32 = __torch__.torchvision.ops.new_empty_tensor._new_empty_tensor
  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
  _33 = uninitialized(Tensor)
  if torch.gt(torch.numel(input), 0):
'interpolate' is being compiled since it was called from 'GeneralizedRCNNTransform.resize'
Serialized   File "code/__torch__/torchvision/models/detection/transform.py", line 78
    target: Optional[Dict[str, Tensor]]) -> Tuple[Tensor, Optional[Dict[str, Tensor]]]:
    _18 = __torch__.torchvision.models.detection.transform.resize_boxes
    _19 = __torch__.torchvision.ops.misc.interpolate
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
    _20 = __torch__.torchvision.models.detection.transform.resize_keypoints
    _21 = uninitialized(Tuple[Tensor, Optional[Dict[str, Tensor]]])
'GeneralizedRCNNTransform.resize' is being compiled since it was called from 'GeneralizedRCNNTransform.forward'
  File "/home/user/.local/lib/python3.6/site-packages/torchvision/models/detection/transform.py", line 47
                                 "of shape [C, H, W], got {}".format(image.shape))
            image = self.normalize(image)
            image, target_index = self.resize(image, target_index)
                                  ~~~~~~~~~~~ <--- HERE
            images[i] = image
            if targets is not None and target_index is not None:
Serialized   File "code/__torch__/torchvision/models/detection/transform.py", line 28
        pass
      image0 = (self).normalize(image, )
      _2 = (self).resize(image0, target_index, )
                                 ~~~~~~~~~~~~ <--- HERE
      image1, target_index0, = _2
      _3 = torch._set_item(images0, i, image1)

Aborted (core dumped)

All the details:
I tried it with the latest versions of pytorch/libtorch
import torch, torchvision
torch.__version__
'1.4.0.dev20191205+cpu'
torchvision.__version__
'0.5.0.dev20191204+cpu'

The export (and import) in python works:
model = torchvision.models.detection.maskrcnn_resnet50_fpn(pretrained=True)
model.eval()
script_model = torch.jit.script(model)
script_model.save("torchscript.pt")

After which I followed the basic steps of importing a torchscript module into c++ from  https://pytorch.org/tutorials/advanced/cpp_export.html .
Also here I've downloaded the latest version of libtorch:
/pytorch /pytorch ~/project
3fd4c696d9b869b03c47dae35ba908c07e6fa1b5

1.4.0.dev20191205+cpu

The c++ code is a just a very basic loading operation:
#include <torch/script.h> // One-stop header.

#include <iostream>
#include <memory>
#include <chrono>

int main(int argc, const char* argv[]) {
  if (argc != 2) {
    std::cerr << "usage: example-app <path-to-exported-script-module> \n";
    return -1;
  }
  torch::jit::script::Module module;
  module = torch::jit::load(argv[1]);

  std::cout << "ok\n";
}

For completion, the cmakelist (since the c++ version got changed to 14 https://github.com/pytorch/vision/pull/1635 ):
cmake_minimum_required(VERSION 3.0 FATAL_ERROR)

project(custom_ops)

find_package(Torch REQUIRED)

add_executable(run_model run_model.cpp)
target_link_libraries(run_model "${TORCH_LIBRARIES}")
set_property(TARGET run_model PROPERTY CXX_STANDARD 14)

Hi,
For this particular error, I think you can follow the comments in https://github.com/pytorch/vision/pull/1407#issuecomment-562096191 to fix this issue

Hi,
For this particular error, I think you can follow the comments in #1407 (comment) to fix this issue

Thanks! I missed that one.

Let me know if this is enough to fix your issue, and if you manage to fix the List[Tensor] issue, can you comment in there as well?

Thanks! I after some (cmake) issues I managed to get the loading of the networks running with the code in https://github.com/pytorch/vision/pull/1407#issuecomment-562096191 . Not yet tried forward passing or anything
I had to fix some headers though: https://github.com/pytorch/vision/pull/1644

Thanks for the fix in https://github.com/pytorch/vision/pull/1644!
I'm assuming that this issue can be closed now. Let me know if you still face issues

Hi, I am also facing the similar issue by using the latest libtorch. Is there any plan to make the missing torch vision operators available in C++?


prevent ranked-switch text to wrap, media query and padding adjustment

Relevant issue number

Resolves #859

Please check the following
Please describe additional details for testing this change

modified the media query so the text doesn't wrap between 966 and 980px
added a media query to handle the same problem on  xxs screens ( < 350px)
adjusted the padding on small screen so the element are displayed more evenly in the ranked-switch component.

I feel he whole component could be styled in a much simpler way. Some simple flex-box would be enough to prevent the use of media queries.  But I'm not sure how customizable vuetify components are.

@Haviles04 Probably.  I'm not used to styling vuetify components and how you guys handle it for cuttle.

@Haviles04 Probably.  I'm not used to styling vuetify components and how you guys handle it for cuttle.

Just put text-wrap: nowrap in the .rank-switch main style


Switch to less obnoxious stakepool explorer

In the iOS app, when going to Staking Center and tapping the title of any pool, it opens adapools.org, which is full of animated ads, pop-ups, and flashy/pulsating buttons. It’s obnoxious and may scare users away because it looks scammy.
Do you know of any other Cardano stakepool explorers that could be used instead? Something calm and uncluttered like the Cardano transaction explorer: https://explorer.cardano.org/en

Maybe pooltool, or pool.pm could be parsed and used in the delegation screen?


Remove dependency on Date.parse from YAML specs

Writing it as Date.new is just as clear, has one less dependency and matches the Date statements in library/yaml/shared/load.rb

Thank you!


Node js deprecation warning coming along with sf org login command

Summary
This was previously reported in https://github.com/forcedotcom/cli/issues/2570 and closed due to version of NodeJS.
I'm recreating this issue as I've seen this warning upon multiple deployment pipelines with the latest Salesforce CLI and my node js version is node-v20.18.0. I've seen this warning with both CLI version 2.63.7 and 2.65.8. We recently made a major CLI upgrade (from 2.10.2 which didn't have this warning).
When running the echo {url} | sf org login sfdx-url --set-default --alias {alias} --sfdx-url-stdin command to authenticate to our org before starting a deployment in our CI/CD container, we consistently get this NodeJS warning.
I'm just running the  echo {url} | sf org login sfdx-url --set-default --alias {alias} --sfdx-url-stdin command  directly in the CI/CD shell which uses the environment variables defined at the job-level for authentication.
(node:37) Warning: Closing file descriptor 23 on garbage collection
(Use `node --trace-warnings ...` to show where the warning was created)
(node:37) [DEP0137] DeprecationWarning: Closing a FileHandle object on garbage collection is deprecated. Please close FileHandle objects explicitly using FileHandle.prototype.close(). In the future, an error will be thrown if a file descriptor is closed during garbage collection.

Authentication to the org is still successful and the deployment starts per normal, but I've consistently received this warning the 1st time I run the org login command in each new container. If I run a 2nd org login command in the same session, the warning doesn't appear, so it appears to be isolated to running the org login command the first time in a fresh container instance.
Steps To Reproduce

Run echo {url} | sf org login sfdx-url --set-default --alias {alias} --sfdx-url-stdin command in a newly created Docker container
Confirm authentication is successful.

Expected result
authentication is successful without any Node warnings
Actual result
authentication is successful with the Node warning consistently in multiple containers when running the command the 1st time.
Warning doesn't happen when running org login command multiple times in the same session.
System Information
Ubuntu latest Docker
GitLab CI/CD
Here's our Dockerfile. Open to any suggestions but this is a pretty straight forward container and we need to publish custom container images to our work's container registry to avoid hitting a Docker Hub pull rate limit.
This Dockerfile is the same as we used with 2.10.2, except for the CLI version declared and the addition of the new  SF_CI_HEARTBEAT_FREQUENCY_MS environment variable.

I'm able to repro by setting these env vars:

the warning only happens once after running sf org list for the first time after auth, likely related with the generic unix keychain implementation.

Even sf -v comes with a warning

Yea saw that warning too. At this point, I added this environment variable to ignore all of these warnings until a new CLI addresses these.
NODE_NO_WARNINGS=1


Catch checksum validation failure from download, delete corrupt file

This change depends on https://github.com/GoogleCloudPlatform/google-resumable-media-python/pull/31

Needs a new unit test for the case where the download object raises resumable_media.DataCorruption.

@tseaver - there's an integration test for this case

@mfschwartz We require 100% unit test coverage:  see the current CI failure.
Note the linter failures in that output as well.

@tseaver @mfschwartz I'm happy to take up the testing mantle. Before this can go in I need to get an LGTM on
https://github.com/GoogleCloudPlatform/google-resumable-media-python/pull/33
and cut a release there.

@mfschwartz RE:

there's an integration test for this case

He (Tres) was referring to tests in this library, not in google-resumable-media (which is in some sense an opaque dependency).

@jonparrott @lukesneeringer @tseaver Can I get an LGTM?

Since @tseaver has made a lot of user_project changes since 1.4.0 I am toying with doing a release for this with a cherry-pick branch (though I'm not that excited about it).
I made one branch just by starting from the storage-1.4.0 tag, cherry-picking #4133 and #4180 and then "fixing" it up (build).
I made another branch from master and then just did a git show storage-1.4.0:${NAME} > ${NAME} for all the changed files, then made sure the changes from #4133 and #4180 stayed in while git add-ing (build).
Both approaches will make figuring out what has changed for the next release difficult. It will at least be difficult via git log storage-1.4.1..HEAD since they will no longer share a merge base.
So the question I put to @lukesneeringer and @jonparrott, is it worth it to cherry-pick this?
The question I put to @tseaver, could we just release the user_project stuff as-is? If yes, please take the lead on writing the release notes.


Unable to retrieve server certificates

This is probably an issue on their end but I couldn't get dnscrypt-proxy to work with the OpenNIC servers.

If you are using drill to manually check the cert, you need to ask the right server. Other servers will not return a certificate that is not theirs.
For example <IP_ADDRESS> is the IP for 2.dnscrypt-cert.fvz-rec-us-wa-01.dnsrec.meo.ws.

Ugh, good point.

Oh, that was it indeed. Looks like it was a copy from July somewhere.

I met this error too, and it solved after the libopenssl and zlib installed.


Ask for finetune/LORA code

Thanks for the nice work!
Do the authors plan to release codes for finetune or lora?

For fine-tuning from a pretrained weight, all you need to do is add --pretrained_checkpoint <your_checkpoint_path> to your script.
We currently have no plans to incorporate LORA, but it should be relatively straightforward to add LORA training to the code using Hugging Face's PEFT library.


Prefix not in the table?

Hi, I know this project is very niche and not sure if you're still using it, so feel free to ignore this issue :D
My router is currently dying, so I was checking if I could connect through hikari with my raspberry, but I get an error when running the calculator:
$ cargo run calculate "2001:f70:3ac0:2800:X:X:X:X"
Error: unknown prefix: 2001:f70

Did they perhaps add new prefixes?
I triple checked my public IPv6 on my PC, router and http://check.xpass.jp/, which all indicate that it's 2001:f70:3ac0:2800:X:X:X:X, any ideas what could be going on?

Hello. I'm happy to say I'm still using this for my router (as described here) with no issues.
My guess at what's going on here is that you've got a DS-Lite ISP, not JPNE MAP-E. My setup is JPNE, and I think the DS-Lite setup is different enough this repo probably won't be of any help :sweat_smile:.
One of the blogs I referenced when setting up my router was this one
It looks like that same person also switched to a DS-Lite setup and wrote a newer post about that, so their info might be useful for figuring this setup out!
Good luck!


Use hamcrest in unit tests

Another unit test PR, this replaces assertTrue() calls with assertThat() using Hamcrest matchers. This provides more informative results when tests fail due to greater than/less than conditions (among others). For example, instead of seeing only:
HttpFilterTest.testFiltering:365 null

We now see:
Failed tests:
HttpFilterTest.testFiltering:365
Expected: a value less than <-1L>
but: <11897708424077L> was greater than <-1L>

I also included a fix to the failing HttpFilterTest.testFiltering test I mentioned above. After some debugging, I discovered that the serverToProxyResponseReceived() method was actually being called after the response entity was consumed. I'm not familiar enough with the internal LP/Netty architecture to know why the server-to-proxy filter methods can be called after a response has already been consumed by the client, but adding a small delay after consuming the response does at least prevent intermittent test failures.

Since this is another all-unit-test-related PR, I'll go ahead and merge this, unless anyone would like me to hold off.

Hello @jekh, I've noticed usage of hamcrest <IP_ADDRESS> fails with Java 6. See hamcrest/JavaHamcrest#105 and ganskef/LittleProxy-mitm@41a4bb51a859b160a9f9434d274d45a97a11ae2a
I will try to create a branch to create a PR to fix it using hamcrest <IP_ADDRESS>. Let me get some practice.

Oh, Hamcrest <IP_ADDRESS> breaks BaseChainedProxyTest, since it lacks in(Collection). Please have a look at that. It's not an easy one for me.

Unit tests also fail under Java 6 because of the pginstrument jar. Basically you can't run our unit tests under Java 6. Considering that Java 6 has been EOL'd for over 2 years, and even Java 7 is now EOL'd, I would not consider this to be a serious problem.


Add a scroll bar to the slate sample interaction

Describe the problem
The 2D slate panel tool described here is very usefull however the scrolling can be incoveniant and it does not work on Hololens 1.
Describe the solution you'd like
Add a scroll bar to the right.
Describe alternatives you've considered
Add top and down buttons.

Same question, do you get any feedback? Thanks!

No I didn't, there was a change on this prefab in the RC 1 Refresh release, I don't know if it is related. I am working on something else for the moment, I only tried to migrate from RC 1 to RC1 Refresh and it totally broke my testing project.

Thanks for you reply.  I added theHandInteractionPan.cs  and change the finger index to the head ray, but  it still can't work on hololens 1. My unity version is 2018.3.14f1.
HandInteractionPan.cs: This script handles articulated hand input for touching and moving the content on the slate's ContentQuad.

I close this issue because it has many duplicates : #4555 #4732 #4727


Add unixodbc dependency to CONTRIBUTING.md

On second thought, let's not add this and remove the dependency for ODBC be implementing: https://github.com/sodadata/soda-sql/issues/567


⚠ Bofrost.at has degraded performance

In 2138c6d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1552 ms

Resolved: Bofrost.at performance has improved in d08d330 after 13 minutes.


Checkout fields

Can I change checkout fields?

Address 1
Address 2
Post code

Can I hide them or set as optional?

@sonufrienko
Now we have a new step in Checkout process - Shipping.
This form will change by Shipping Service. We can fill address or local pickup.


[Doc Request]: add guide for change data file location

Is there an existing issue about this doc request?

[X] I have checked the existing issues.

Describe the doc request.
Please add a guide for changing data file location for 0.7 in both English & Chinese doc for developer support.

@domingozhang
If I want to redirect the data store location, can I change the log.toml file parameters in the mo folder?
log.toml
|data-dir = "./mo-data"|Default data directory|

This action can modify the data file path for file system.


MN 2023 taxable income from gssi off

See
http://taxsim.nber.org/out2psl/mn2
MN taxable income is off in emulator, hence tax is off

my bad

No, I  think this issue is valid.


Again notifications and SDK UPGRADE

Starting with SDK 33 they introduced runtime permissions for notifications.
From docs:

So all our apps (cordova not yet support sdk 33) installed first time on Android 13 will use this logic.
I have a problem with my app, that it does not ask for the permission until I literally send first push notification and when the app is in foreground then it asks me for the permission. So  1 notification is always gone. After allowing, it works fine.
So the qustion is - what does it mean to open notification channel?
Are you experiencing the same?
Regards,
Michal

Having the same problem.

Just an idea but can you do a double opt-in, where upon startup if it's never opted in, it sends the first notification just to get the permission?

@dchase-sir will it then automatically display Allow notifications prompt?

Hi @michal-kroliczek , you can check out this fork: https://github.com/4sh/cordova-plugin-fcm-with-dependecy-updated
We've just added the support for Android 13+ and updated the README page on our fork.
Installation can be done by targetting the git tag:
npm way
npm install cordova-plugin-fcm-with-dependecy-updated@git+https://github.com/4sh/cordova-plugin-fcm-with-dependecy-updated.git#7.10.0

old cordova fashion way (not tested)
cordova plugin add cordova-plugin-fcm-with-dependecy-updated@git+https://github.com/4sh/cordova-plugin-fcm-with-dependecy-updated.git#7.10.0

Afterward, in your app, calling FCM.requestPushPermission() will trigger the permission dialog on Android 13+.

@mcastets , did not help, I tried integrating and tried sending 3-4 push, as per new Android guidelines.
any suggestions?

Hi @shilpidaswani, with our fork I've managed to send some push notifications on Android 12+ and it works but there's a new issue: when tapping on the notification app is not opening...
The logcat shows:
Indirect notification activity start (trampoline) from com.your.app blocked

We have tried some workarounds with no luck, I'd switch to another maintened plugin (https://github.com/dpa99c/cordova-plugin-firebasex).