Permission management

Proposal for Capacity to manage (edit/remove) only by admin bug.
Granted on Permission.CHANGE_SERVER_SETTINGS, so give the capacity to create a dedicated role.
The best could be to create/register a new specific permission, but not sure possible with TC 8.1

Hi,
Thank you for contributing and please excuse me for long response, I will comment changes inline

@axel3rd I've merged your branch with 2 minor fixes:
Internal ProjectID is expected in AuthUtil
Edit actions should not be visible for upper projects without edit permission (eg a project admin has edit permission for a child project only while server is defined in a parent project)

@Linfar : Thanks for fix (I'm very newbie in TeamCIty plugin dev ...).
Could the build #23 pinned ? The download link on home/readme.txt (.lastPinned) is build #20 => The documentation content about permission is not contained in this build).

@axel3rd done

@Linfar Thanks


Add section for locally measuring coverage

Add a section for measuring coverage locally on Ubuntu.
Signed-off-by: Stephen Brawner<EMAIL_ADDRESS>

Going through and doing some cleanup. I see no reason why this can't be merged.

Thanks Kat!


ERROR in Entry module not found

When starting the development process the following error is thrown:
ERROR in Entry module not found: Error: Can't resolve './src' in '/Users/me/Development/webxr-webpack-boilerplate'
preceeded with the following warning:
WARNING in configuration The 'mode' option has not been set, webpack will fallback to 'production' for this value. Set 'mode' option to 'development' or 'production' to enable defaults for each environment. You can also set it to 'none' to disable any default behavior. Learn more: https://webpack.js.org/concepts/mode/
The project was cloned and setup according to First run
Expected behavior
Default site should show in localhost:9000
Please complete the following information

Device:
OS: MacOS 10.13.6
Browser (if any):
WebXR Webpack Boilerplate Version: 0.9.16

I've followed the first run instructions this time. :)
Ultimately I got things working by updating yarn from v1.9.4 to v1.10.1 with npm install -g yarn. Then I updated all the packages with yarn upgrade. Doubt the latter was necessary though. Along the way I messed up the url omitting  https which is of course necessary for the Service Workers.
The problem I had before was that versions after 0.9.10 didn't work by following the instructions. To fix the first error I stripped the content of src/js/app.js and placed it into a new src/index.js which is webpack's default entry point.
Where as the mode warning can be taken care of by adding the CLI --mode=development into the start command in package.json. These steps aren't necessary any longer though.


feat: use miden-crypto for advice provider

This commit introduces the usage of MerkleTree, MerklePathSet, SimpleSmt, NodeIndex, and MerkleError as replacement to the types implemented for the advice provider.

### Must have

- [x] Replace `MerkleTree`, `MerklePathSet`, `SimpleSmt`, `NodeIndex`, and `MerkleError` from `miden-crypto` into the advice provider

### Nice to have

- [x] Move `RandomCoin` and `RandomCoinError` as import from `miden-crypto`
- [x] Implement `fmt::Display` and `error::Error` for `ExecutionError`
- [x] Move `AdviceSet` from `miden-core` to `miden-processor` as `MerkleSet`
- [x] Fix test example in `miden/examples/mod.rs`
- [x] https://github.com/0xPolygonMiden/crypto/pull/62

### Tech-debts introduced
- [ ] #702

miden-crypto v0.1.2 is now on crates.io.

@bobbinth updated!


Dask dataframe isna

Top level dd.isna method to stay consistent with pandas. From #3239.

This looks great to me.  Thank you for implementing this @cr458 .  A few other administrative things to clean up, hopefully these are easy.

Can you add a small note to the changelog in docs/source/changelog.rst.  Hopefully the pattern there should be clear enough (although this may be the first change in the new version, so you may have to look down a bit for examples to copy).  Don't forget to add your name and a website to the bottom of that document
Should we add this as a method on _Frame as well so that both DataFrame and Series get this method?  The version issue might come up.  I recommend that we check the version within the method and raise an informative error if appropriate.
After that can you add isna to the DataFrame API docs in docs/source/dataframe-api.rst?

The pleasure was all mine @mrocklin. Thanks for the helpful comments, I think it's fair to say the little work that was done here was actually done by you guys. Have updated the changelog now.
Happy to implement the method in _Frame as well. Which based on the other methods for _Frame I'm guessing it would look something like:
    @derived_from(pd.DataFrame)
    def isna(self):
        if hasattr(pd, 'isna'):
            return self.map_partitions(M.isna)
        else:
            raise ImportError

although I'm having a hard time seeing where the variable M is set in the class? It seems to be used throughout various methods.

It's not part of the class.  It is imported from utils.py I think
On Fri, Mar 23, 2018 at 7:20 PM, Christopher Ren<EMAIL_ADDRESS>wrote:

The pleasure was all mine @mrocklin http:///mrocklin. Thanks for the
helpful comments, I think it's fair to say the little work that was done
here was actually done by you guys. Have updated the changelog now.
Happy to implement the method in _Frame as well. Which based on the other
methods for _Frame I'm guessing it would look something like:
@derived_from(pd.DataFrame)
def isna(self):
    if hasattr(pd, 'isna'):
        return self.map_partitions(M.isna)
    else:
        raise ImportError

although I'm having a hard time seeing where the variable M is set in the
class? It seems to be used throughout various methods.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/pull/3294#issuecomment-375823128, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszLmHLkIPrQnzbEUCONPJavpIsJc7ks5thYNBgaJpZM4SvSkf
.

@mrocklin shall I add a test to test_embarrassingly_parallel_operations in test_dataframe.py for isna as well?

It's not necessary, the test you have seems fine to me.  An alternative test would have been to add isna to that test instead, but what you've done here seems good to me.
In general I'm pretty satisfied with this.  I've merged with master and added isna to API docs.  Merging.
Thanks @cr458 for your work on this!  Hopefully it is the first of many :)


[New Feature] Adding a upload hotels image functionality for canteen owners

Objective:
Implement an email verification system for user registration and login processes to enhance security and user experience.
Key Features:
Custom hotel image upload functionality for canteen
After Sign-in as a canteen there should be a functionality so that  an owner can upload his/her hotel image.
How to do it :
You can use multer and cloudinary or any other cloud platform for implementing this feature .

assign  me  this issue

Hey, I want to contribute to this project. I am a MERN stack developer and I have done internship, worked on several projects and won hackathons as well. I can do upload image functionality.
Here's my linkedin profile : https://www.linkedin.com/in/nakshatravyas/
Github profile : https://github.com/nakshatravyas

I have done an image upload on Cloudinary with the help of multer middleware,
I'm interested to contribute, can u assign me this issue under GGSOC'24

I am interested in resolving this issue. I have experience with Google OAuth 2.0, cloudinary and multer. Please assign this to me.

Similar issue has alredy been raised , if he fails to do so , i will continue with this convo.


Added 2fa-support

Added 2fa-support plugin

Obviously ignore my comment left on the issue where you mentioned your plugin :)
Thanks for this plugin !


View all media issue

It works, but im hoping you can make it function on media that is added as you scroll.  it only works on the initially loaded page.

What version are you using (Ver number, extension/userscript)?
Can you tell me if you encounter this issue with NeverEndingVoat or in threads or both?


Support GCC7 on Linux

Our only GCC7 issue appears to that it needs to be linked against
libatomic explicitly. This commit links to libamotic explicitly on any
Linux platform regardless of compiler version.
Closes #1756

This passes CI so, no problem for using with GCC6 on Trusty.
@lisael @russel could you verify this fixes #1756 for you?

I set up a Sid VM and verified both the issue and the fix. I'll have a different PR for instructions for building on Sid as I had to figure that out.

-latomic needs to be added during the linking phase as well

Ok, I have fix for linux. verifying it passes our CI.
Going to check to see if fixes freebsd issue as well.

This doesnt work on FreeBSD. I've updated the PR to be linux only. If this passes CI, its good to merge.

Reported in on #1756 that 499a1dfffa479d54f0069fc21d901b58235496d9 seems to work fine on Debian Sid with GCC 7.


ROS 2 Humble: Pick & Place demo

Hi guys,
Hope you're well. I've been digging into your TIAGo robot simulation – awesome work!
I saw the Pick & Place tutorial for ROS 1 (http://wiki.ros.org/Robots/TIAGo/Tutorials/MoveIt/Pick_place), but I'm using ROS 2 Humble. Any plans to drop a similar tutorial for ROS 2 Humble with the demo too ?
Cheers,
Yassine

Hey again,
Thanks for getting back to me. I'm really interested in diving into the TIAGo simulation and learning more about Pick and Place on ROS2 Humble. However, I have to admit that I'm a beginner in this area and currently lack the skills to put together the demo files and a full tutorial.
But I think it would be amazing if someone, either from your team or the community, could jump in and create this tutorial. Having a step-by-step guide for Pick and Place using MoveIt in ROS2 Humble would not only help me but also others who are looking to grasp MoveIt's functionality in a practical context.
Thanks once again for considering this. Looking forward to exploring and learning more about TIAGo and ROS2.
Cheers,
Yassine

Hello @yassineyakercap,
We will try to have it as soon as possible, but we cannot make any promises yet. Thank you for your interest. We are happy to contribute back to the community. Once, we have some resources available, we will work on this part.
Thank you,
Best Regards,
PAL Robotics Team.

Hello @cmower,
Unfortunately, we didn't have time to work on this part yet. Feel free to open a PR, we are happy to review and merge them.
Best Regards,
PAL Robotics Team

Thank you for understanding! We will be happy to review them and get them upstream.
Best Regards,
PAL Robotics Team


Don't scroll if element is already visible (optional behaviour)

Actually this is a new feature request.
Would be nice to have an extra option to specify if scrolling of the element is necessary, if it is already fully visible in container. In other words if element completely fits the view, then do nothing (analogue of scrollIntoViewIfNeeded()).
I think in some scenarios this is very ergonomic, as will reduce the number of scrolls and thus visual pressure on the user.

Hi @dmak, I think, given what you ask happens "before" calling the plugin and the fact that this plugin is parallel to $().animate(), I think this check should be done outside the plugin.
You can include a plugin that helps you make the check and avoid calling this plugin the element is already on screen.
I'll close this because I think it's out of the scope of the plugin
Thanks for taking the time to propose the feature

That is right: that can be done outside the plugin. However it requires the calculation of element height (including borders and margin) and calculating basically the scroll offset. After the computation is completed, the calling the plugin is not needed, as value for scrollTop() is already calculated.

Actually don't bother: I have decided to implement this function myself (see result here). Actually I first tried to improve jquery.scrollTo() but I found it too difficult as I need only vertical scrolling (x-axis).

What I COULD do is (maybe) move the code that calculates the final position, based on all the settings (offset, margin, etc), to a utility method that is exposed. This was the amount of code doesn't change but you can precalculate the final position, see if scroll or not and pass it directly to the plugin (or scrollTop())

Well, that's up to you. I have decided to stick to more light-weight solution which I can manage. In total it's about 50 lines, and I am not sure that connecting to some API will save much coding.


android: cannot find symbol mInstabug.setUserName(username);

I saw in a closed issue and in a commit that you fixed the setUsername to be setUserName for consistency with iOS.  However, we are now seeing this gradle build error when building for Android:
cannot find symbol mInstabug.setUserName(username);
As seen here:
https://stackoverflow.com/questions/43932286/gradle-build-error-cannot-find-symbol-for-minstabug-setusernameusername
So I changed line 186 to the previous casing to get a successful build:
    /**
     * Sets the user name that is used in the dashboard's contacts.
     *
     * @param username
     */
    @ReactMethod
    public void setUserName(String username) {
        try {
186 >            mInstabug.setUsername(username);
        } catch (Exception e) {
            e.printStackTrace();
        }
    }

Same thing happening to me

Hello @jplanders,
Thank you for reporting this, It will be released in minutes since we are not expecting to merge Dev branch soon, So we are releasing the fix right now.


feat: add sort for results page, add 'not found' block

Can you please also implement mobile sorting?


feat: added height width props to LayoutSelector

PR Checklist
Adds height/width props to the Layout Selector. Defaults to 3x3
@sedghi

Thanks for your PR @Ouwen , does it makes sense to say rows, and columns instead of height and width?

sure thing, I can change the variable to rows/columns

@sedghi I've updated to rows/columns and put an example in the longitudinal mode.


Feat/delete and edit question on note (less in action 08 2024)

Great! Have you tried e2e test with Cypress + Cucumber?

Not yet, I made the pr by mistake before it was done 😅


Do not change rendering order to 'geometry' for opaque objects.

Fixes gearvrf/GearVRf-Tests#126

The test does pass but I don't think the test is correct anymore. The texture has no alpha and rightly so the rendering order gets reverted. It seems like a useful feature to me that if possible we should keep.
I updated the test here: https://github.com/gearvrf/GearVRf-Tests/pull/136. A framework change is necessary since there is a path through the asset loader where the bitmap will not be tested for alpha otherwise. Here: https://github.com/Samsung/GearVRf/pull/1437.

Will merge on Friday.


Istiod doesn't respect affinity or nodeSelector attributes

(NOTE: This is used to report product bugs:
To report a security vulnerability, please visit https://istio.io/about/security-vulnerabilities
To ask questions about how to use Istio, please visit https://discuss.istio.io)
Bug description
Unlike Istio addons, the core istiod component doesn't provide a way to set nodeSelector and affinity or anti-affinity attributes.
[ ] Docs
[x ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[x ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
Expected behavior
Istiod deployment should have nodeSelector and/or affinity attributes based on pilot/global values, same as the istio addons (ingress, egress, etc...), but the istio-discovery chart doesn't support this.
Version (include the output of istioctl version --remote and kubectl version --short and helm version --short if you used Helm)
1.8.2
1.8.1
How was Istio installed?
helm charts
This bug is referenced in #29642 but the relevant PR was closed.
Simply copying the _affinity.tpl from the addons should solve this

It is recommend to set resources/affinity/annotation... with k8s settings, see: https://github.com/istio/istio/tree/master/operator#new-api-customization
We don't want to add helm parameters for each option in the values.yaml any more, it's hard to maintain so many options in the helm values.

It is recommend to set resources/affinity/annotation... with k8s settings, see: https://github.com/istio/istio/tree/master/operator#new-api-customization
We don't want to add helm parameters for each option in the values.yaml any more, it's hard to maintain so many options in the helm values.

Hi, @morvencao,
I'm installing Istio via helm-charts. How can I use this new API customization

Also, if I want these settings to apply to all components (for example - hpa and anti-affinity), do I have to redefine it explicitly for each component?

@yfried Yes, this is the api doc for k8s settings: https://github.com/istio/istio/tree/master/operator#new-api-customization
Also some examples:
https://github.com/istio/istio/blob/master/operator/cmd/mesh/testdata/manifest-generate/input/pilot_k8s_settings.yaml
https://github.com/istio/istio/blob/master/operator/cmd/mesh/testdata/manifest-generate/input/ingressgateway_k8s_settings.yaml

@yfried Yes, this is the api doc for k8s settings: https://github.com/istio/istio/tree/master/operator#new-api-customization
Also some examples:
https://github.com/istio/istio/blob/master/operator/cmd/mesh/testdata/manifest-generate/input/pilot_k8s_settings.yaml
https://github.com/istio/istio/blob/master/operator/cmd/mesh/testdata/manifest-generate/input/ingressgateway_k8s_settings.yaml

We are also using the helm charts to install istio into our cluster which does not contain the istio operator. So if i'm reading the documentation right these settings won't work without the istio-operator.
Is there any other way to add node affinity?

Maybe kustomize is another option if you're using helm chart.


Publish package to npm

I use npm for developing client side. Do you think you could publish this package on there?
Also, you should probably move your package.json dependencies to devDependencies

One temporary solution is using cdnJS:
<EMAIL_ADDRESS>

yep, it will mostly be used with some bundler as webpack. I found on npm ics-browser-gen, which seems to be this library, but importing it as ES module fails with message it is not a function.
Any idea how to import this lib inside front end framework if ES Module imports are not working?

@PerpetualWar, any news related to this?


#181151230 Configure Authors Haven to use PostgreSQL as DB Engine

What does this PR do?

Configures Authors Haven to use PostgreSQL as DB Engine

Description of Task to be completed?

Create PostgreSQL database
Configure Authors Haven to use PostgreSQL as DB Engine

How should this be manually tested?

A database with a database name is created; In this case, the database name used is "Invictus".
By running "psql -U postgres", provide a password if required, run "postgres=# \c invictus;", (invictus is the name of the database). In order to see the tables that have been created, run "invictus=# \dt;".

Any background context you want to provide?

No

What are the relevant pivotal tracker stories?

https://www.pivotaltracker.com/story/show/181151230

Screenshots (if appropriate)

Questions:

None

[Start #181151230]

Looks okay


Request: Keep features selected between searches

We have so many features that it's tricky to search for exactly the ones you're looking for. Editing related features often involves multiple searches.
Current behavior:
Any new feature search removes all selections you've made.
Requested behavior:
A new feature search keeps your current selections, moving them to a new "Selected Features" section above the search results.

Unfortunately this can't be implemented by configuration of the django admin framework, and would require a custom view with custom HTML, CSS and maybe JS. In other words, too much time for what's worth.
I can, on the other hand, allow to visualize more than just 100 feature at a time, in order to probably bypass the need to change page.

I'm adding the ability to search multiple values with | separator


!fossilsearch doesn't tell you if you made a typo

If you make a typo or enter a fossil name that turbot doesn't recognize, it doesn't tell you that it didn't recognize it - it just ignores it.
It should tell you the names that it doesn't recognize so that you can correct your typo or whatever was going through your head

Fixed by #25


Allow multiple widgets on the same page

This version of click2voxjs would be used in both portal and click2vox at the same time.
Don't merge until fully tested and approved by QA
@san983 We probably should consider releasing a new minor version for this new feature in click2vox. From the CDN we should be serving this one as click2vox-2.3.0

@agonza1 There where some features that you mentioned that needed sync between click2vox and the portal, they all should be addressed here. Can you assure that please?

It seems to work OK. Despite I found a bug, it was not introduced in this PR (https://github.com/voxbone-workshop/portal-widget/issues/173)


st terminal is seriously broken

st terminal is seriously broken. If you like CTRL+z + fg workflow forget about st as you can expect this all the time (stdout gets mixed with vim buffers after CTRL+z):

I can't replicate this problem.

I also do not know how to replicate this, it looks like it is non deterministic, it just sometimes happens on my machine.


This add default icon if the image is not avaliable

Fixes: https://github.com/zulip/zulip-electron/issues/356

I think it would be great if we can handle this in our domain-util since all the logic for domain/server/icon handling lies there.

Yeah, i will do so, i did not know where to put so i put it here.

@akashnimare rather leave it where it right now because domain-util.js solely handles downloading icon and add it to db. So we should leave it where it is now since this is where all the event listeners are handled.

FYI, the tests are failing because, PR #353 is not merged yet.

@cPhost can you confirm, if it fixes the issue? I still see the broken image icon.

I should mention that the problem occurs sometimes when we reload the app. In the image below you can see the broken server image but if we do insect and check the src the image loads up successfully. Also, the same image is being used in the server section which surprisingly works correctly -

Merged, thanks @cPhost.


Event Questions

What is your question?
Ok so a few questions...

Do events count as notes?
Can they be iterated over?
If they can, how do you grab the name, value 1, and value 2?

Events aren't notes
No, I've tried

Events aren't notes
No, I've tried

Thanks for the info...


How clickhouse-operator we can use?

Hi,
I would like to understand clickhouse-operator installation or how to integrate this with clickhouse backup , Need guidance to use clickhouse-operator

Did you look
https://github.com/Altinity/clickhouse-backup/blob/master/Examples.md#how-to-use-clickhouse-backup-in-kubernetes ?

Hi @Slach, actually I had a similar question.
I had an already existing deployment of clickhouse for posthog (our analytics tool). And we wanted to add backup functionality to it. So I created a new yaml file with the values you have linked and used kubectl apply -f to apply it to the cluster
It created two new pods prefixed with chi-bakup-
If I connect to the container clickhouse-backup inside it I am not able to list the tables of my clickhouse table. What I suspect is that I it is connecting to the other container called clickhouse-pod in the same chi-backup pod.
How do I make the clickhouse-backup container use my correct clickhouse-operator for backup?

@utsavll0 you should not just blind apply new manifests, which provided on https://github.com/Altinity/clickhouse-backup/blob/master/Examples.md#how-to-use-clickhouse-backup-in-kubernetes
instead of it, you should modify your exists manifest to modify potTemplates in posthost kind: ClickHouseInstallation to add clickhouse-backup container
if you install your clickhouse via posthog helm chart, could ask in posthost github, how to modify podTemplate in their helm chart via values


Let exceptions inherit standard exception classes with strings

The exception class inherits directly from std::exception. But - your exceptions seem to be either runtime errors (file not found), or invalid arguments (e.g. passing a null extension) etc. So, inherit from the appropriate standard exception class. That would also remove the need for the base class with the string member - don't reinvent the wheel unless you have to.

Actually, consider #32 as well - it's not clear you even want custom exceptions at all.


C++ Manager Plugin for Testing

What
We want to build a simple C++ manager plugin to be used for testing the C++ manager system.
Acceptance Criteria

A manager plugin inside the OpenAssetIO examples directory.
It should only do the required capabilities and resolve.
Entity references it resolves are hard coded, we have BAL as a mock.
Independently buildable, not linked to OpenAssetIO's build system, but there should be CI/CTest tests to make sure it builds and runs.
Link from README "quick start" section.

Out of Scope
Not packaged and published, to build it, you'll need to pull the OpenAssetIO repository.

Whilst developing this we discovered that the PyPI build of OpenAssetIO cannot load C++ manager plugins.
This is because the PyPI build uses a single shared library (the Python extension module) with the core C++ library baked into it as a static library.
This causes two problems:

Our build system setup is designed to block all symbols from being exported except for those explicitly made visible. This means the Python extension module hides the symbols from the "foreign" C++ core library, so there are no symbols for a C++ manager plugin to link to.
Python extension modules are loaded by the Python interpreter using RTLD_LOCAL (on POSIX), which prevents symbols from "leaking" between binaries, so even if our Python extension module exposes the necessary symbols, they will be "blocked" from other shared libraries.

The first problem could be solved by refactoring our build system to allow the libopenassetio.a library's symbols to be exported from the _openassetio.so Python extension module. This means

No longer having CMake generate an export.h header (generate_export_header()), since the generated export header for a static library is just a stub without any visibility flags. Instead, we would have to roll our own that works for both static and shared libraries and across platforms (lots of other projects do this).
Ensure we use $<LINK_LIBRARY:WHOLE_ARCHIVE,openassetio-core>, so that all symbols, not just those used by the Python extension module, are exported from the static library.
Somehow exclude the core C++ library from the --exclude-libs,ALL linker argument, probably requiring us to eschew ALL and instead provide an explicit list of what libraries to exclude.

The second problem is more serious, though easier to "fix".

We would have to (temporarily) override the flags using sys.setdlopenflags(os.RTLD_GLOBAL | os.RTLD_LAZY). Though of course that's POSIX only.
However, the Python docs say

Portability therefore requires not to make any assumptions about symbol visibility. This means that all symbols in extension modules should be declared static, except for the module’s initialization function, in order to avoid name clashes with other extension modules [...] And it means that symbols that should be accessible from other extension modules must be exported in a different way.

That last point especially means that the correct solution would be to split the Python extension module and the core C++ (shared) library, so that the extension module can depend on the shared library, as can plugins.
Note that all the above applies to POSIX. Windows is another story and would need alternative fixes, if any are even possible (not explored in my investigations).
In the past, separating the core C++ library and the Python extension module in the PyPI package caused problems with linker search paths.  On Linux we have RPATH and on macOS we have @loader_path, but on Windows we have nothing. Fixing this will take further experimentation/consultation.


Multi language

We are thinking of starting to use this project.
Do you line up to make it multi-linguage?

@jlusarreta I can do the support for Arabic and French languages. I know a little of Korean but I am not a native speaker.

I can do it in spanish.
My question is about multilanguage for products pages or menu.... and make
multisites web or sites with different languages ( ..../es .../en,.....)

Multilanguage feature is supported. @thiennn Would you please confirm that?

For dynamic content like products, at the moment we don't support multiple languages. The only option is to deploy multiple instances and use a reverse proxy to make the url look like a single site. But the drawback is the data of each instances is separated


npm 包名能更换吗？携带了@开头，导致和本地 冲突

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

很报歉，不方便修改包名，一般来说scoped的包不会与本地产生冲突。您得自行想办法解决本地冲突，或者直接下载源文件进行使用

感谢回复，我将源码包下载下来，手动改了包名了。


Add topic name to debug output

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

[x] ♻ Refactor
[ ] ✨ Feature
[x] 🐛 Bug Fix
[ ] 👷 Optimization
[ ] 📝 Documentation Update
[ ] 🚩 Other

Description
Closes #5
Related Tickets & Documents
Added to documentation?

[ ] 📜 readme
[x] 🙅 no documentation needed

[optional] What gif best describes this PR or how it makes you feel?

I'm seeing the topic name, which means this PR is probably good to go, but I think it uncovers why I was confused in the first place. When using a sql schema file, eg:
CREATE TABLE "ecommerce"."products" ...

I would expect the topic name to be ecommerce.products based on the fully qualified table name, but it looks like it's just products.
I think the offending line is here in src/schemas/parseSqlSchema.js:
        let schema = {
            tableName: table.table[0].table,
            columns: []
        };

After some experimenting, I believe this would work to get the fully qualified table name and use that as the topic name:
tableName: Object.values(table.table[0]).filter(x => x).join(".")

The filter is to filter out null values if that array contains any.

Sweet! I like that idea! Just updated the topic name to use the full table name!


[dev-preview2] pin opm to v1.30.0

(cherry picked from commit aea99f2b0e02707e733dc4a56216db3728d4d09c)

+1 thanks :)

@stuggi maybe I'm getting it wrong, but shouldn't this PR be against openstack-k8s-operators:dev-preview2 instead of openstack-k8s-operators:main (at least that what it seems from the message at the top of the page):
stuggi wants to merge 1 commit into openstack-k8s-operators:main stuggi:pin_opm_dev-review2

@stuggi maybe I'm getting it wrong, but shouldn't this PR be against openstack-k8s-operators:dev-preview2 instead of openstack-k8s-operators:main (at least that what it seems from the message at the top of the page):
stuggi wants to merge 1 commit into openstack-k8s-operators:main stuggi:pin_opm_dev-review2

yes, created the PR wrong. thanks for the pointer

https://github.com/openstack-k8s-operators/install_yamls/pull/613


Server shutdown problem

What's the issue?
After saving chunks, the server process cannot be finished properly due to this mod. Fabric API version: 0.104.0+1.21.1
Screenshots
No response
Mod and Minecraft version
Mod: 1.6.2+mc1.21 Minecraft: 1.21.1 (Fabric)

Thank you for the report! Looks like a small oversight that should be easy to fix!


wg_engine: external webgpu handles (web integration)

move instance, adapter and device creation from renderer to application
its necessary for web integration, because browser have its own mechanics to create hardware handles
this changes makes webgpu canvas more universal to use in case of system and web applications
See https://github.com/thorvg/thorvg/issues/2410

@hermet

Please merge createHandles(); into WgWindow().
Its already in WgWindow class, what do you mean "merge"

@hermet

Please merge createHandles(); into WgWindow().
Its already in WgWindow class, what do you mean "merge"

@SergeyLebedkin Merge the code from createHandle() into WgWindow(). Rather than adding function call.


N2V convenience config function channel axis constraints [BUG]

Describe the bug
create_n2v_configuration does not allow specification of axes with C dim of size 1. It is totally possible to have data with dimensions (S, 1, Y, X) and it means users have to do an extra step of removing the C channel from their data.
To Reproduce
Code snippet allowing reproducing the behaviour:
from careamics import CAREamist
from careamics.config import create_n2v_configuration

config = create_n2v_configuration(
    experiment_name="Demo",
    data_type="array",
    axes="SCYX",
    patch_size=[8, 8],
    batch_size=1,
    num_epochs=1,
    n_channels=1,
)

results in error:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/melisande.croft/Documents/Repos/careamics/src/careamics/config/configuration_factory.py", line 484, in create_n2v_configuration
    raise ValueError(
ValueError: Number of channels must be specified when using channels (got 1 channel).

The error is raised from:
https://github.com/CAREamics/careamics/blob/0a29ea2da44b3745a33bedb93143f4ca6b98a9da/src/careamics/config/configuration_factory.py#L482-L486
Expected behavior
Should be allowed to specify 1 channel. The convenience functions could have n_channels: Optional[int]=None instead.
Additional context
This might happen in all the convenience functions not just create_n2v_configuration.

Yes, that's an interesting point.
The reason for writing the convenience function like this was that this is very unlikely in "real" data. By real data, I mean data that users acquired on a microscope. Singleton dimensions are not really a thing in bioimages, I've only seen it when people are converting their data for DL pipelines (which is not needed in CAREamics)... I prefer covering the very likely case of people having channels and not specifying the number, rather than an edge case.
But if we can come up with an elegant way to cover that case and still have a meaningful way to warn users that they should specify the number of channels, then we should indeed do it!

I think I encountered this when I was writing a test that parametrised the number of channels, and this error made implementing it slightly more annoying. I can see scenarios where preprocessing of data might add a singleton channel axis and also cases where a custom file type and read function has a singleton channel axis.
I think we can implement it by having the default value of n_channels be None. So we can do:
if "C" in axes and n_channels is None:
    raise ValueError(
        f"Number of channels must be specified when using channels."
else:
    n_channels = 1


wip -Reduce code duplication in ProjectsController

https://codeclimate.com/repos/54d4fa64695680762a0024c5/ProjectsController

 Code Climate has analyzed this pull request.

@calebthompson I can't auto-merge, can you please take a look and address?

That's fine - this isn't ready yet.

@calebthompson if this isn't ready, can you delete your PR?

I need it for the Code Climate reviews, but I have WIP in the title if that's any help.

You can use brakeman locally.
On Mon, Feb 23, 2015 at 4:02 PM, Caleb Thompson<EMAIL_ADDRESS>wrote:

I need it for the Code Climate reviews, but I have WIP in the title if
that's any help.
--
Reply to this email directly or view it on GitHub
https://github.com/projectjellyfish/api/pull/266#issuecomment-75646448.


toColorspace('p3') outputs an sRGB image

Possible bug
Is this a possible bug in a feature of sharp, unrelated to installation?

[x] Running npm install sharp completes without error.
[x] Running node -e "require('sharp')" completes without error.

If you cannot confirm both of these, please open an installation issue instead.
Are you using the latest version of sharp?

[x] I am using the latest version of sharp as reported by npm view sharp dist-tags.latest.

If you cannot confirm this, please upgrade to the latest version and try again before opening an issue.
If you are using another package which depends on a version of sharp that is not the latest, please open an issue against that package instead.
What is the output of running npx envinfo --binaries --system --npmPackages=sharp --npmGlobalPackages=sharp?
System:
    OS: Linux 5.15 Debian GNU/Linux 12 (bookworm) 12 (bookworm)
    CPU: (16) x64 AMD Ryzen 7 5800X3D 8-Core Processor
    Memory: 7.55 GB / 15.62 GB
    Container: Yes
    Shell: 5.2.15 - /bin/bash
  Binaries:
    Node: 18.17.0 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 9.6.7 - /usr/local/bin/npm

What are the steps to reproduce?

Convert a 16-bit image to a Display P3 JPEG/PNG/TIFF
Convert the same 16-bit image to the same format but to sRGB
Note that these are bit-exact images

The same appears to be the case when passing any random string to toColorspace, but some choices do work, e.g. b-w.
Notably, calling withMetadata({icc: 'p3'}) does work in adding a P3 ICC profile (but doesn't convert the actual image).
What is the expected behaviour?
The output image should be converted to Display P3. This is supported by libvips, though an error for invalid arguments would be expected as well.
Please provide a minimal, standalone code sample, without other dependencies, that demonstrates this problem
const sharp = require('sharp');

sharp('dog.png')
  .toColorspace('p3')
  .toFile('dog_p3.jpeg');

sharp('dog.png')
  .toColorspace('srgb')
  .toFile('dog_srgb.jpeg');

Please provide sample image(s) that help explain this problem
16-bit PNG image (converted from a .CR3 using Imagemagick):

Hi, did you see https://sharp.pixelplumbing.com/api-colour#tocolourspace and its link to https://github.com/libvips/libvips/blob/3c0bfdf74ce1dc37a6429bed47fa76f16e2cd70a/libvips/iofuncs/enumtypes.c#L777-L794 ?
P3 is an RGB profile rather than a colourspace, so either RGB or RGB16 would be appropriate.

If I try 'rgb', I get the error: Uncaught Error: vips_colourspace: no known route from 'rgb16' to 'rgb'.

I did some testing and it seems calling pipelineColorspace is sometimes needed for the final image to have the right colors.
Given this original image in Display P3:

Here are the outputs of the following two commands:
await sharp('Italy-P3.jpg')
  .withMetadata({icc: 'p3'})
  .rotate()
  .toFormat('jpeg', {quality: 100, chromaSubsampling: '4:4:4'})
  .toFile('Italy-P3-withMetadataP3.jpeg')

await sharp('Italy-P3.jpg')
  .pipelineColorspace('rgb16')
  .withMetadata({icc: 'p3'})
  .rotate()
  .toFormat('jpeg', {quality: 100, chromaSubsampling: '4:4:4'})
  .toFile('Italy-P3-withMetadataP3.jpeg')

The version without pipelineColorspace has somewhat degraded colors compared to the version with it when viewed on a P3 display (but I think they look the same when color managed into sRGB). Since the original is 8-bit, shouldn't rgb16 be unnecessary here?

If you're using wide-gamut input and would like wide-gamut output with a P3 profile then using 16-bit rgb16 as the pipeline colourspace would be a good choice to avoid any narrowing of colour range.
sharp(input)
  .pipelineColorspace('rgb16')
  .withMetadata({ icc: 'p3' })
  ...

@lovell I forgot to mention, but I wasn't ever able to get toColorspace('rgb') to work regardless of the source image. While VIPS_INTERPRETATION_RGB is a member of VipsInterpretation, it's not mentioned in the docs describing each option. Not sure if this is a sharp issue or a libvips issue. Possibly related, toColorspace('scrgb') always produced a completely black image for me.

The colourspace conversions supported by libvips can been seen at https://github.com/libvips/libvips/blob/5fb9f6182ca9d396536f87bbd4ca08a993791c30/libvips/colour/colourspace.c#L221

AFAIK, one should use VIPS_INTERPRETATION_sRGB instead of VIPS_INTERPRETATION_RGB after commit https://github.com/libvips/libvips/commit/e48f45187b350344ca90add63b4602f1d1f5a0a0. Perhaps the libvips docs should be clearer about this.

Possibly related, toColorspace('scrgb') always produced a completely black image for me. Should I make a new issue for this or re-open this one?

This is being tracked in issue https://github.com/libvips/libvips/issues/2858.


Changeover to NetDotNet

Mantikor is supposed to build the network packages with the library NetDotNet in the next version. For this purpose, the hardware address is also to be resolved via IPv6.

NetDotNet is no longer supported for the time being


Speed up GitHub CI build by limiting ABI and building release and debug separately

CI gfxreconstruct build queued with queue ID 275389.

CI gfxreconstruct build # 5019 running.

CI gfxreconstruct build queued with queue ID 275405.

CI gfxreconstruct build queued with queue ID 275422.

CI gfxreconstruct build # 5020 running.

CI gfxreconstruct build queued with queue ID 275441.

CI gfxreconstruct build # 5021 running.

CI gfxreconstruct build # 5021 passed.


Update to latest NuGet, which includes support for Icon

If my package contains a <icon> node, when I click Publish it will casue an exception.
Maybe a bug ?

Please create a new issue. That's easier to track and discuss, thanks!

#869


Fix apt key

DataDog seem to have switched their apt key, update accordingly - https://docs.datadoghq.com/agent/basic_agent_usage/ubuntu/#manual-install

Closed by #13


Add blog post about patching Backstage lint command to output a summary file

Motivation
I worked with a client that had an insane amount of lint errors amassed in their Backstage project so I had to come up with a solution of being able to fix those errors without putting a stop to development while trying to prevent the list of errors from growing.

@minkimcello what do you think about this image? I generated it with Midjourney.

It's the only one with a black background.

yeah, it's true. let me play around with it some more.

@minkimcello how about these?

How is this?

It looks too dark, I think we need a lighter version

@minkimcello what do you think about this

@minkimcello here is another one, that I made to show progression from red -> yellow -> green.


Enable lower and upper percentile to filter bins and re calculate color

Pass in percentile as upperPercentile and lowerPercentile prop
Sort counts of bins on first pass when data is passed in and save to state. No need to recalculate data when percentile changed.
User count domain calculate by lower and upper percentile as color domain to calculate color. If colorDomain is passed in as a prop, use it instead.
Set alpha of bins outside percentile to 0. In vertex shader, if alpha = 0, set hexagon radius to 0. This will prevent hexagon from being rendered and pickable.
Add upperPercentile and lowerPercentile to getColor updateTrigger, so that only colors are recalculated when percentile changed.

@heshan0131 Since we have now moved development to master, to minimize rebase problems, I recommend opening up a new PR from latest master.
Since things are little easier if this a single commit, you can still land this PR on dev (to squash it), or manually do a rebase -i to squash your commits here into a single commit.
Then create a new branch off latest master, and cherry-pick this commit into that branch, and open a new PR.

Yeah. Shan, please rebase this PR to dev, squash and merge it to dev, and then cherry-pick to apply it on our master and open a new PR against master

Create https://github.com/uber/deck.gl/pull/403 to fix conflict


Masked Multi Head Attention

c = torch.zeros_like(src)
x1 = self.fcn(self.transformer_decoder1(*self.encode(src, c, tgt)))   # x1-->O1
# Phase 2 - With anomaly scores
c = (x1 - src) ** 2
x2 = self.fcn(self.transformer_decoder2(*self.encode(src, c, tgt)))

Excuse me, in the TranAD model, which code is Masked Multi Head Attention reflected in? From the above code, the encoders of stages 1 and 2 are the same, and Masked Multi Head Attention is not involved in stage 2.

Please refer to the source code here https://pytorch.org/docs/stable/_modules/torch/nn/modules/transformer.html#TransformerDecoder

Sorry, the code I have doubts is as follows:
https://github.com/imperial-qore/TranAD/blob/main/src/models.py
line 490--524
# Proposed Model + Self Conditioning + Adversarial + MAML (TKDE 21)
class TranAD(nn.Module):
	def __init__(self, feats):
		super(TranAD, self).__init__()
		self.name = 'TranAD'
		self.lr = lr
		self.batch = 128
		self.n_feats = feats
		self.n_window = 10
		self.n = self.n_feats * self.n_window
		self.pos_encoder = PositionalEncoding(2 * feats, 0.1, self.n_window)
		encoder_layers = TransformerEncoderLayer(d_model=2 * feats, nhead=feats, dim_feedforward=16, dropout=0.1)
		self.transformer_encoder = TransformerEncoder(encoder_layers, 1)
		decoder_layers1 = TransformerDecoderLayer(d_model=2 * feats, nhead=feats, dim_feedforward=16, dropout=0.1)
		self.transformer_decoder1 = TransformerDecoder(decoder_layers1, 1)
		decoder_layers2 = TransformerDecoderLayer(d_model=2 * feats, nhead=feats, dim_feedforward=16, dropout=0.1)
		self.transformer_decoder2 = TransformerDecoder(decoder_layers2, 1)
		self.fcn = nn.Sequential(nn.Linear(2 * feats, feats), nn.Sigmoid())

	def encode(self, src, c, tgt):
		src = torch.cat((src, c), dim=2)
		src = src * math.sqrt(self.n_feats)
		src = self.pos_encoder(src)
		memory = self.transformer_encoder(src)
		tgt = tgt.repeat(1, 1, 2)
		return tgt, memory

	def forward(self, src, tgt):
		# Phase 1 - Without anomaly scores
		c = torch.zeros_like(src)
		x1 = self.fcn(self.transformer_decoder1(*self.encode(src, c, tgt)))
		# Phase 2 - With anomaly scores
		c = (x1 - src) ** 2
		x2 = self.fcn(self.transformer_decoder2(*self.encode(src, c, tgt)))
		return x1, x2

x2 = self.fcn(self.transformer_decoder2(*self.encode(src, c, tgt)))
for self.encode(src, c, tgt)
What I understand is that Masked Multi Head Attention needs to be used in Phase 2.

Thanks for your reply!
I combined the transformer code to sort out the network structure of TranAD.https://github.com/imperial-qore/TranAD/blob/main/src/models.py [line490-524],
which is exactly as you said: "Masked Multi-Head Attention is ued in both phases as it is part of the decoder. It is implemented as part of the TransformerDecoder model in pytorch".
I have several new doubts in the process of understanding the code, if you could help me to solve it, I would be very grateful!

During the training process of the TranAD model https://github.com/imperial-qore/TranAD/blob/main/main.py[line 251-273],
I found that when calculating the loss, the weights of the reconstruction loss and the adversarial loss are (1 / n)  and  (1 - 1/n), respectively. Will such changes affect the training results?

It is still the Masked Multi-Head Attention problem. In the z = model(window, elem)[line 264], I debugged and still did not find that there is masked data in the decoder process. This confuses me a lot.

I would appreciate your suggestions,
Cheers

Thank you very much for your reply！
I will continue to study the code and look forward to your reply.
Thank you again！


Azure Pipelines | Unknown argument --include-category

I'm running a job (series of tasks including nuget restore, build & signing application) on Azure DevOps to build android application and Test (Xamarin.UITest) application through Appcenter.
Test runs fine, if I don't specify any arguments on "Additional options for run" on "Test with Visual Studio App Center" task. When I do that (--include-category LandingPage) task throws error saying:
Error: Command 'test run manifest --manifest-path d:\a\12\a\AppCenterTest\manifest.json --app-path d:/a/12/b/Release/PUBLISH_PACKAGE_NAME_PHONE_ANDROID.PUBLISH_PACKAGE_NAME_PHONE_ANDROID.apk --app SampleApp/Android --devices 12345abcd --test-series DevOps --locale en_US --include-category LandingPage --quiet --token ***' failed with exception "Unknown argument --include-category"
Same result for --exclude-category as well
What I'm missing?
[command works perfectly when I run appcenter test run from my local machine.]

I'm having the same issue.
failed with exception "Unknown argument --exclude-category
Running visual studio app center task version 1.*

I'm having the same issue.
failed with exception "Unknown argument --exclude-category
Running visual studio app center task version 1.*

make sure you're providing your --include-category AppCenterTest under Prepare Tests area not under Run tests. Additional Options under Run Tests have different purpose it seems.
Discussion on stackoverflow: [https://stackoverflow.com/questions/58099354/azure-pipelines-app-center-test-error-when-including-category]

Thanks  @karthiksubramaniam, that seemed to have solved the issue.

Looks like this issue has been resolved, so I'll close this issue - feel free to re-open if you are still facing this issue.

Can you please share a working example? I tried providing the below option but still getting same error.
prepareOptions: '--include-category Test1'.


Remove parade from code

:(

Codecov Report

Merging #1389 (2a523f7) into master (300802f) will decrease coverage by 0.98%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1389      +/-   ##
==========================================
- Coverage   41.92%   40.94%   -0.99%
==========================================
  Files         155      152       -3
  Lines       12209    11859     -350
==========================================
- Hits         5119     4856     -263
+ Misses       6379     6323      -56
+ Partials      711      680      -31

Impacted Files
Coverage Δ

api/controller.go
38.88% <ø> (-0.04%)
:arrow_down:

stats/collector.go
59.09% <0.00%> (-2.28%)
:arrow_down:

pyramid/tier_fs.go
58.77% <0.00%> (+1.75%)
: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 300802f...2a523f7. Read the comment docs.

Nooo......
@guy-har wanted to do this one!  We had a figh^Wdiscussion and all...


Remove propertyNames

See https://json-schema.org/understanding-json-schema/reference/object#propertyNames
Those are not supported in version 3.0

See https://json-schema.org/understanding-json-schema/reference/object#propertyNames
Those are not supported in version 3.0

Thank you for your contribution

completed the PR merge


Got "Invalid escape character in string" in settings.json?!

VSCode Version: 1.11.0-insider
OS Version: win 10 64bit

Steps to Reproduce:

Open your user settings
Add

// Place your settings in this file to overwrite the default settings
{
    "flowide.pathToFlow": "D:\dev\myproject\node_modules\flow-bin"
}

Complains about "\d" in "D:\dev\myproject\node_modules\flow-bin" because is not a valid escape character in string.

I would expect to treat it like a normal string. Is doing more than that.. If I fix to have "D:\dev\myproject\node_modules\flow-bin", flow is not found...

This is following the JSON specification. Backspashed is the escape character and can only be followed by a limited set of characters.
See http://www.json.org/


SAK-28975: preferences missing active primary button at bottom of form

The main preferences page has buttons at the top and bottom of the form. The ones in the bottom of the form are not wrapped in a div with a class of "act".
This poses a problem for institutions that styles the primary action buttons based on this structure (.act input.active[type="submit|button|reset"]).
The attached patch simply changes the parent div of the bottom buttons to apply the "act" class (and remove some unnecessary formatting).

+1


Broken logic for pull request statuses in PR resource

Expected Behavior
When a check is updated in the PR pipeline resource, the corresponding check should be updated on github side.
Actual Behavior
The update may happen or not depending on the case.
Additional Info
The code in https://github.com/tektoncd/pipeline/blob/8f427a84c7a975d3e5d9dea5b046dc583d8a5a33/pkg/pullrequest/api.go#L275-L278 makes the assumption that the label, or context in case of GitHiub, is a unique key to identify a "check".  That is not the case for GitHub - the API model maintains a list of checks, which may have the same context, but different state and id.
The reason why this issue happens is that in some cases the code in the pullrequest/api packages makes a map of the checks based on the label, so the stated stored in the map may be that of any of the status available.
Even if it was considered responsibility of the CI system to manage these multiple states, the pull request resource does not make that possible by hiding the check id away.
When a process in the CI system sends an update to a status, it should get back the ID from github. That ID can then be used by the CI system to update the correct status once the CI job is complete.
Unfortunately the GitHub API docs don't really explain more in which cases a new status is created even if one exists already with the same name. It might be that that happens every time an update is sent without the ID. Or it may be that the API allows for multiple statues against the same job to allow for parallel checks. I would say the former is more likely, but I struggle to understand how we could end up then with 7 different IDs for the same context like in the example from the gist.

/kind bug

https://github.community/t5/GitHub-API-Development-and/Is-there-a-way-to-delete-a-GitHub-commit-Status-via-the-API/td-p/28778


Fix Fix load string dict issue for transactional hive tables.

Proposed changes
Fix load string dict issue for transactional hive tables. The column name need to pass 'row.column_name'.
https://github.com/apache/doris-thirdparty/pull/112
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 48.87 seconds
stream load tsv:          554 seconds loaded<PHONE_NUMBER>9 Bytes, about 128 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          32 seconds loaded 861443392 Bytes, about 25 MB/s
insert into select:          33.8 seconds inserted 10000000 Rows, about 295K ops/s
storage size:<PHONE_NUMBER>5 Bytes

run buildall

run buildall

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.95 seconds
stream load tsv:          542 seconds loaded<PHONE_NUMBER>9 Bytes, about 131 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.4 seconds inserted 10000000 Rows, about 340K ops/s
storage size:<PHONE_NUMBER>1 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.19 seconds
stream load tsv:          536 seconds loaded<PHONE_NUMBER>9 Bytes, about 133 MB/s
stream load json:         19 seconds loaded<PHONE_NUMBER> Bytes, about 118 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
insert into select:          29.7 seconds inserted 10000000 Rows, about 336K ops/s
storage size:<PHONE_NUMBER>3 Bytes


Beam checking: skip PA

As per the commmit message:
skip position angle checks: they can be horribly wrong, especially around
PA=0, and often do not affect how the beam should be averaged. Of course, in
some cases, they can have dramatic effects, but it has been causing more harm
than good so far

Coverage remained the same at 73.227% when pulling 48bbe81c6ab7985be1cf48bee0f1f17bd062a0a2 on keflavich:beam_check_no_pa into 926595672bea9b330db5b10afe7ae400f4cff4c3 on radio-astro-tools:master.


Write to .xlsx

We are currently using openxlsx in rio. It offers read/write to .xlsx. We'd like to incorporate reading from .xls (and thus import readxl), but we don't want to lose write to .xlsx support. Any chance you can add write to .xlsx (ala what is available in openxlsx)? I don't think it make sense for us to import two different Excel-related packages.

Write is a pretty big job, so I'm not planning on it at the moment.


caughtUp: Fix bug that overwrote CaughtUp state.

When we made a request for a narrow, we would overwrite the CaughtUp
state for that narrow in the other direction - i.e. if we were fetching
old messages, then we would forget that we've seen the most recent
message.
This commit fixes that by only updating each bit of the CaughtUp state
from false to true (on a fetch-messages result), and never the other
way around.

Looks good, thanks, merged!
For cross-reference in the future when this is no longer obvious: a previous version of this change was in drafts of #4678 and discussed there.


Add/swagger

adding the first test endpoint, which should help to set up continous integration (with an encrypted developer key) and the first go at using pyswagger to query the SOM api with dummy data.

tests aren't running, I think we need to trigger master branch. This is bad practice, but it's not so terrible, since it's just me, so I'll merge and then debug master.


TFRS - Revise error message when organization does not have enough compliance units to initiate a transfers

Describe the task
Revise the error message that the user receives when an organization does not have enough compliance units in their balance to initiate the transfer. This error message occurs when the user enters in a quantity of units that is greater than their organization's balance (minus any units in reserve) and then tries to sign and send the transfer (clicks on the sign and send button).
Current message: Private Zenhub Image
Revised message: Error! Unable to initiate transfer. Your organization does not have enough compliance units for this transfer.
Purpose
Update the language to align with the new Act
Acceptance Criteria

[ ] The revised error message should clearly indicate the reason for the transfer failure (insufficient compliance units).
[ ] The language should be concise, user-friendly, and in line with the new Act's terminology.
[ ] The message should be displayed promptly and clearly when the transfer initiation fails due to a lack of units.

Hey team! Please add your planning poker estimate with Zenhub @jig-patel @prv-proton


Create an action to upgrade the version when merge on master

We need to create a GitHub action to upgrade the version of package.json when a merge on master occurs before build.

semantic-release can help.

We could use something like standard-version and bump the version and generate the CHANGELOG.


Background geolocation is used all the time

Your Environment

Plugin version: 4.16.5
Platform: iOS
Device OS version: 17.5.1
Device manufacturer / model:  Apple iPhone 14 Pro
XCode version: 15.4
Capacitor info (npx cap doctor):
@capacitor/cli: 6.1.2
@capacitor/core: 6.1.2
@capacitor/android: 6.1.2
@capacitor/ios: 6.1.2
Plugin config provided to #ready:

{
  backgroundPermissionRationale: { title, message, positiveAction, negativeAction },
  disableLocationAuthorizationAlert: true,
  startOnBoot: true,
  stopOnTerminate: false,
}

Expected Behaviour
App uses background location only when needed.
Actual Behaviour
App uses background location all the time.
Steps to Reproduce

https://github.com/suyundukov/capacitor-background-geolocation-issue-reproduction
Install app
Add some point
In 3-4 days I'm receiving this notification:

Context
I'm working with geofences and I want to show a notification whenever user is in geofence point. This is how I use it:
BackgroundGeolocation.onGeofence((_, taskId) => {
  LocalNotifications.schedule(...);
  BackgroundGeolocation.finish(taskId);
});

await BackgroundGeolocation.ready({
  backgroundPermissionRationale: { title, message, positiveAction, negativeAction },
  disableLocationAuthorizationAlert: true,
  startOnBoot: true,
  stopOnTerminate: false,
});

BackgroundGeolocation.startGeofences();

It works most of the time, but my biggest problem is that app is using background location all the time. I thought it will use it only when user is near geofence point.

I thought it will use it only when user is near geofence point.

Your assumption was incorrect.  Geofences are managed purely by the OS.  How do you think the OS knows when the device is near a geofence:  by periodically sampling the location.
the user will get that dialog once for the life of the app install.  There’s no way to avoid it.

I thought that geofences are managed by the OS after being set up by the app. The app defines the geofence's parameters and actions, while the OS handles the location monitoring and triggers the app when the geofence is crossed, even if the app isn't running. But by the way it looks, app is constantly monitoring location in background or am I wrong on this one?

But by the way it looks, app is constantly monitoring location in background or am I wrong on this one?

You are wrong.  Your app is constantly monitoring location only if a solid location arrow is shown in the OS status bar.
see your device Settings Privacy and Security for more info:

If I’ve only set up a single geofence point, why is it showing that the app has accessed my location at multiple places?

How do you think the OS knows where you’re at so it can evaluate a geofence to ask “am I in this geofence?”

Isn't it the OS, rather than the app itself, that utilises location services in that case?

What does your privacy settings show for your app?

What does your privacy settings show for your app?

Allow Location Access: "Always".

Isn't it the OS, rather than the app itself, that utilises location services in that case?

According to Apple’s documentation, "the system monitors regions and wakes up your app as needed when conditions change between satisfied and unsatisfied states". Given this, why does it indicate that my app is using geolocation even when user is in a different country from the original geofence location?

After further investigation, I realized that the issue was actually on my side, and not related to the project. I apologize for the oversight and any confusion this may have caused. I'll be closing this issue now. Thank you for your understanding.


Add parameter for max circuit-proving-parameters size.

[ ] add this parameter
[ ] add a check ensuring it is (or is projected to be) satisfied by a candidate construction claimed to meet all requirements
[ ] acquire buy-in from relevant members of project team for concrete values (which may be informed by work in #17). (cc: @pooja for tracking of this line)

Ubercalc now belongs in Filecoin spec.


Redraw based on only changed data

Hello!
I read your whole tutorial for mithril, awesome reading!
For example in chapter 12:
If I add console.log(label + ' redraw'); into view() function (~67th line) inside createTemperature() I see
Air redraw
Water redraw

every time I click any button!
Is it possible to rewrite meiosis pattern to rebuild vdom only for components dependent of changed data?
For example if I increase water temperature only Water redraw line must appear at console.

@d9k Greetings! Thank you for your feedback, much appreciated!
You ask a good question. This is addressed in Chapter 3 - Update Model:

Note: you might wonder about rebuilding the whole view and re-rendering it when something changes. Generally speaking, producing a vnode is not a performance concern. Further, re-rendering the view is what virtual-DOM libraries are good at: figuring out what minimal changes are needed to make the real DOM reflect the view.

That being said, nevertheless I will show how to prevent re-construction of sections of the virtual-DOM tree that have not changed.
I will add a link here when that section is ready. Stay tuned!

@foxdonut, cool! :sunglasses: :+1:

@d9k I still need to do a writeup, but in the meantime I thought you might be interested in seeing the code example.
Essentially to prevent re-renders, this example:

uses immutable objects, to easily determine a change with !==
uses Mithril components to have access to lifecycle methods
uses Mithril's onbeforeupdate lifecycle method to determine whether to re-render

You can see the result in the console. For example, when you change the Air temperature, you only see render Temperature Air, the other components are not re-rendered.
Hope that helps!

@d9k reference material:

Medium article regarding performance concerns
Preventing Re-renders on the Meiosis Wiki


Center prisma icon in seti-ui repo

Bug description

Expected behavior
Icon should be properly centered.
Information
The initial PR was done here, so it needs to be updated there as well with a new PR.

I can confirm this for VS Code ☺


IAP audience autodetermination does not work on Compute Engine

For AppEngine, IAP autoconfiguration works great.
For Compute Engine and Kubernetes Engine (which runs on Compute Engine), it will fail with message "Service ID expected not to be null". This is because metadata is the wrong place to be getting the Backend Service ID. The right place is...complicated. I've asked IAP team for design guidance.
In the meantime, IAP starter/autoconfig users should provide the correct audience through spring.cloud.gcp.security.iap.audience property, overriding automatic determination (IAP docs have guidance on how aud claim is composed).

In the meantime, can we update the refdoc to indicate that the audience determination doesn't work on GCE (App Engine Flex and GKE)?

Removed autoconfig and updated docs in#1293.

Wont-fix; see #1468 for explanation.


Apm 334309 update api token requirements

@sq2gxo why? this is required

@sq2gxo why? this is required https://www.dynatrace.com/support/help/extend-dynatrace/extensions20/extension-lifecycle/#permissions

You are right, I've tried install withot Read/WriteConfig permission and it worked (probably due to fact that we are providing assets only) but it is safer to keep the requirement.


feat(specs): deleteByParams

close #1015

Ah yes you still need to update the test file with deleteByParams now anyway

@shortcuts Done ✅


P004 swift submission

Wondering if there was a standard implementation of seeded gaussian distributions so we could all get comparable outputs

Wouldn't it be better to use the built in dot(_:_:) method rather than remaking it?

it certainly sounds like it would be... You're not talking about this dot function are you? That is very explicitly for vectors of 4 numbers. (Actually, what it is about is SIMD: check it out and the original swift proposal)... So you might ask, could we leverage SIMD, using typical padding approaches so it works when our vectors are not %4==0 ? Sure, there's a standard approach to that sort of thing, and it does stand to give a performance benefit. But, no, that is even less legible.

@DelSquared Are you going to continue implementing it, or can we accept robbiemu's submissions?

You can accept it :)

Thank you!


fix pilot  ENABLE_WASM_TELEMETRY flag

Please provide a description of this PR:
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.
ConstructVMConfig should respect ENABLE_WASM_TELEMETRY
https://github.com/istio/istio/blob/0f92d7f2048ce09954becb897cd849d6c196e147/pilot/pkg/model/wasm.go#L28

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[x] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@zirain: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
34fb99dae8eb86ba9fc62e2fe92a1ae1d44cb1fb
link
true
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


[0.5/vk] skip unknown memory properties

Should address https://github.com/gfx-rs/wgpu/issues/718
It's a bit risky, but that's the recommendation from Vulkan so far.

r? @Ralith

@Ralith please take another look, this is now rewritten

bors r=Ralith


types(vite): replace any with Matcher for chokidar WatchOptions#ignored

Description
Make type def more accurate.

Additional context

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

wow. Not expected PR(paulmillr/chokidar#1140) being merged immediately.

Warning: types/chokidar.d.ts:124:3 - (ae-forgotten-export) The symbol "AnymatchMatcher" needs to be exported by the entry point index.d.ts

API Extractor completed with warnings

The warning is killing me. I couldn't solve it by reading the warning and doc. Simply re-export it in ' types/chokidar.d.ts' doesn't solve the problem and I fail to find location of entry point index.d.ts.


Support for host_rewrite_literal in gRPC xDS Clients

Feature Overview
👋 Currently the host_rewrite_literal route configuration is ignored by gRPC xDS clients. We would like change it so the authority is set based on the host_rewrite_literal. This would only apply when trusted_xds_server is set, just as is the case for the auto_host_rewrite feature introduced in A81.
Motivation
We target a reverse proxy using a single cluster from the gRPC xDS client. The internal routing within the proxy will be determined by the authority header .
            +------------------+
            |  gRPC xDS Client |
            +------------------+
                     |
                     |
                     |
                 +---------+
                 |  Proxy  |
                 +---------+
                      | (Proxy routing logic based on Authority header)
      .---------------+--------------.
     /                |               \
+--------+       +--------+       +--------+
| Svc A  |       | Svc B  |       | Svc C  |
+--------+       +--------+       +--------+

The motivation is similar to the one described in  A81 for the auto_host_rewrite field, but in our case we target the proxy using a single cluster, and so cannot use the auto_host_rewrite header.
Would it be possible to extend gRFC A81 to also include the host_rewrite_literal? Alternatively we could create a separate gRFC.

This is not in scope for A81, but you could write a separate gRFC for this if you want.
Can you explain why you can't use auto_host_rewrite here?  It seems like the only difference is where the authority is encoded -- with auto_host_rewrite, it would be in EDS, but what you're proposing here would put it in CDS instead.  Why is it possible for your control plane to do this in CDS but not EDS?
@ejona86 @dfawley

but what you're proposing here would put it in CDS instead

No, not in CDS, RDS instead.
The addresses the client sees are the proxy. And it will use that same proxy for Svc 1, 2, and 3. So a single Cluster/DNS or EDS will be configured for the proxy. But based on the route they want to change the authority so the proxy will route to the correct service. From the client's perspective, I think only the route is unique per Svc.

No, not in CDS, RDS instead.

Sorry, yes, you're right.

The addresses the client sees are the proxy. And it will use that same proxy for Svc 1, 2, and 3. So a single Cluster/DNS or EDS will be configured for the proxy. But based on the route they want to change the authority so the proxy will route to the correct service. From the client's perspective, I think only the route is unique per Svc.

Ah, okay.  It wasn't clear to me that the endpoints were the proxy addresses, but I do now see that they said it was a single cluster.
An alternative to consider would be to use the HTTP CONNECT functionality described in pending gRFC A86 (https://github.com/grpc/proposal/pull/455).  That would allow using "real" endpoint addresses but configuring use of the proxy to reach them.  At that point, you could actually set the hostname in EDS instead of RDS.  But I do recognize that this would be a significant change from what you're doing now.
If you do want to proceed with host_rewrite_literal support, feel free to submit a new gRFC.  You would probably also need to contribute the implementation to whichever gRPC language(s) you need.

Thank you both for the quick responses! I've started a draft of the gRFC and hope to share it soon. I've included rationales for both why auto_host_rewrite and HTTP CONNECT won't be ideal for our use case, mostly centered around connection pooling, but also because it would be a significant deviation from how we have things set up.
Our existing setup was designed for usage with envoy, where this is supported and it's worked well enough.


Add a lifetime application track to service connection for aspnetcore

TODO: add Context.Abort() test hubs
It's to solve the issue that the outgoing pipeline closes too early that context.Abort() might not working, we now wait for 5 seconds before shutdown the outgoing task

Logistically it is all about 3 things:

communicating the right message to the service
maintaining the list of client connections
completing transport pipes

Rather than "making it work" in each method that touches either 3 of the areas above, can we build a few abstractions around these 3 to make them thread safe and unified? So we won't have to worry about keeping all code paths dealing with these 3 in sync?

Hi @KKhurin I think the issue the PR wants to resolve is one thing: to make the lifetime for a client connection context clean and the pipe completion is controlled by the single consumer. The 3 things you mentioned is out of the scope of this PR. I think we can discuss offline and make other PRs to resolve your concern if there is a need.


Update project-service-core-solution-version-history.md

Updated release notes.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.


anyone have successfully made bots join bg?

I have tried to queue for bg, but bots seems not queuing, might have to increase the density of certain level band like made all bots level 80 to increase the chance to act queuing bg.
Does anyone tried yet? cos I have difficult to support too many bots online at same time.

I noticed that with a larger amount of bots, at least 100 to 200 bots are even more active, such as creating queues of BGs. But I'm not really sure about that.

After enter bg, playerbots are just standing still and wont go out for bg activities, looks like there are no bg stategy implemented for bots yet. however, they do fight back when they are provoked.


Switch acs-engine to use CoreDNS instead of kube-DNS

Is this a request for help?:
No

Is this an ISSUE or FEATURE REQUEST? (choose one):
Feature request

What version of acs-engine?:
All current versions

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm)
Kubernetes
What happened:
Kubernetes conformance tests:
[Fail] [sig-network] DNS [It] should provide DNS for services  [Conformance]
[Fail] [sig-network] DNS [It] should provide DNS for the cluster  [Conformance]
are failing because we are receiving authoritative does not exist upon record lookup via test invoking kube-DNS. Since Kubernetes 1.11, Kubernetes uses coreDNS by default and we need to switch over.

Here's the KEP recommending CoreDNS as default for v1.11
https://github.com/kubernetes/community/blob/master/keps/sig-network/0012-20180518-coredns-default-proposal.md

https://github.com/Azure/acs-engine/issues/3385

I know that CoreDNS is the default now as of 0.23 - When does this become officially "Stable" / "GA"? :) Whats the criteria for that? Seems pretty solid so far, are we tracking any known issues? I'm mostly concerned about Windows nodes at this point. @CecileRobertMichon  @PatrickLang

coredns is now default for 1.12+


Question on formula

Hi,
Can you elaborate on where the first set of formula's come from: \delta a_1 = \delta b_1 * a1 / (b_1 - \delta b_1) ?
Thanks!
Ygor

This comes from the invariant formula of uniswap V2 AMM, which is a*b = k.
So when you calculate the output amount of a swap, that is (a + \delta a) * (b - \delta b) = a * b. You can calculate the delta a by given a delta b or do a reverse calculation.
For more, you can see the Uniswap source code:  https://github.com/Uniswap/uniswap-v2-periphery/blob/master/contracts/libraries/UniswapV2Library.sol#L42-L59
The formula \delta a_1 = \delta b_1 * a1 / (b_1 - \delta b_1) is in the getAmountIn function. And for simplicity, we do not consider the fee calculation in this project.(In uniswap, there's a swap fee of 0.3%).


use_cookie example not reproducible

The use_cookie references a trait FromToStringCodec this is not hyperlinked in the docs: https://docs.rs/leptos-use/latest/leptos_use/fn.use_cookie.html
below the example code it references "the codec chapter".
From there, it looks like there is an issue rendering the docs book

the link for FromToStringCodec is broken: https://github.com/Synphonyte/leptos-use/blob/main/src/utils/codecs/string/from_to_string.rs
It would be great if the documentation for this handy function provided a reproducible example. As it stands, there is no clear way to use this function—which is a bummer because it looks so useful!

We recently moved all the codecs into its own crate called codee. Leptos 0.7 resources now use that crate as well.
I guess we forgot to update the chapter in the book. Sorry about that!

Gocha! So including codee to Cargo.toml is the way forward, ya?

You can find the corrected chapter now here: https://github.com/Synphonyte/leptos-use/blob/main/docs/book/src/codecs.md


Sign in form#516

Related Issue
Fixes #516
Description
[Please include a brief description of the changes or features added]
Improved CSS in form page
Type of PR

[x] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]
Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

@SUGAM-ARORA pls review this PR

done!!!!

Why this is happening i got merge conflicts then resolved them now deployment failure

Now Check this


feat(opendataset): add dataloader for RarePlanes_real dataset

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7632

Relevant Lines:
8828

💛  - Coveralls


usb: udc_nrf: change init priority to PRE_KERNEL1

Change the priority of udc_nrf so that it gets initialized before cdc_acm_uart.
The current setup is triggering a priority error:
$ west build -p -b nrf52840dk_nrf52840 samples/subsys/usb/cdc_acm \
  -DCONFIG_CHECK_INIT_PRIORITIES=y -DCONF_FILE="usbd_next_prj.conf"
...
ERROR: /soc/usbd@40027000/cdc_acm_uart0 PRE_KERNEL_1 55 < \
  /soc/usbd@40027000 POST_KERNEL 50

There is direct dependency between UDC driver an a CDC ACM instance, I habe to look at it probable find another fix if any.

Cool, we can tweak the script for special cases if necessary though I'd rather avoid it if possible, feel free to suggest a better fix.

Cool, we can tweak the script for special cases if necessary though I'd rather avoid it if possible, feel free to suggest a better fix.

What I want to do is to remove cdc_acm_uart nodes from the bus, because virtual UART interface is actually other side of the implementation.
diff --git a/dts/bindings/serial/zephyr,cdc-acm-uart.yaml b/dts/bindings/serial/zephyr,cdc-acm-uart.yaml
index 8e0a003097..18926e8643 100644
--- a/dts/bindings/serial/zephyr,cdc-acm-uart.yaml
+++ b/dts/bindings/serial/zephyr,cdc-acm-uart.yaml
@@ -7,8 +7,6 @@ compatible: "zephyr,cdc-acm-uart"

 include: uart-controller.yaml

-on-bus: usb
-
 properties:
   tx-fifo-size:
     type: int
diff --git a/samples/subsys/usb/cdc_acm/app.overlay b/samples/subsys/usb/cdc_acm/app.overlay
index c50d0fd9d6..367c6dc721 100644
--- a/samples/subsys/usb/cdc_acm/app.overlay
+++ b/samples/subsys/usb/cdc_acm/app.overlay
@@ -4,7 +4,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */

-&zephyr_udc0 {
+/ {
        cdc_acm_uart0 {
                compatible = "zephyr,cdc-acm-uart";
        };

But it would require a lot of changes in samples/applications. PRE_KERNEL_1 is too close to the clock drivers and should probably be avoided. Not sure what to do.

But it would require a lot of changes in samples/applications. PRE_KERNEL_1 is too close to the clock drivers and should probably be avoided. Not sure what to do.

How about moving cdc_acm_uart to POST_KERNEL instead?

How about moving usbd_cdc_acm to POST_KERNEL instead?

From the experience so far, all serial drivers should be PER_KERNEL. Deviations cause problems.

From the experience so far, all serial drivers should be PER_KERNEL. Deviations cause problems.

Ok then I'm back proposing to anticipate the level of the usb driver. About clock, if there's a dependency with the clock controller, then that should be checked as well for priority by the very same script I'm using to detect this. That's not the case for udc_nrf since there's no clock device, but it is should be the case for other controllers.


Improve Lighthouse scores

Here’s a Lighthouse report on a sample page of our docs.

The report offers several suggestions. My suggestions for starting places:

Remove JS from the <head> and put it at the bottom of the <body>.
The sidebar JS executes inline and therefore blocks page draw. I think the JS part of it can be deferred (moved) to later in the page -- again, bottom of <body>.

@anagio, you are working on docs site performance. Are you working on this already?


[iOS] - app crashing when trying to change ResolutionPreset.low.

Hello,
I'm trying to use the app to do text recognition and reading barcode, when I set the ResolutionPreset.low to ResolutionPreset.veryHigh / high /medium the app is crashing.
It's running fine when I set ResolutionPreset.low but the issue is it's not able to detect barcode and text very well.
here is a little of code snippet:
Future _startLiveFeed() async {
    final camera = cameras[_cameraIndex];
    _controller = CameraController(
      camera,
      ResolutionPreset.low,
      enableAudio: false,
    );
    _controller?.initialize().then((_) {
      if (!mounted) {
        return;
      }
      _controller?.startImageStream(_processCameraImage);
      setState(() {});
    });
  }

any recommendations?

Closing the issue because I haven't received any feedback.

having the same problem !

actually it works but very slow with ResolutionPreset.medium and crashes with ResolutionPreset.high and max (android)


Release cargo-apk-0.9.7

Fixes #375

Like I said I don't have much time for all this shuffling around. It's okay with me as long as it's not disruptive at this moment.

you're saying it's too inconvenient to open a PR in the correct repo?

It would be a bit inconvenient to transplant the commits to the other repo (but that has to happen regardless).
@rib mentioned in emails to owners for these crates that I won't be allowed to make releases, but I can since I am part of @rust-windowing/publishers, and cargo doesn't care which git repositories a crate is hosted at for that to work. Regardless, it'd be great if I could have direct control over the crates (and in turn grant publishing rights to a new group over at @rust-mobile).

Mmh you won't be allowed to make releases? I'm missing some context here. Are you and @rib still not getting along?

I would, unless I missed some context when @rib emailed the current owners of these crates (on crates.io) where he mentioned that. Which I thank him for sending that by the way, saves me some time and effort!

@rib mentioned in emails to owners for these crates that I won't be allowed to make releases, but I can since I am part of @rust-windowing/publishers

aah, right, good point, for some reason I was thinking that once the repo is split then you'd end up loosing permission to publish because I was thinking that you were reliant on credentials that were associated with the android-ndk-rs repo.
If you're also a member of @rust-windowing/publishers then there's hopefully nothing to worry about in the short term, and we can follow up on any ownership changes for crates asynchronously

Mmh you won't be allowed to make releases? I'm missing some context here. Are you and @rib still not getting along?

:) no worries here; @MarijnS95 isn't saying that I was asserting that they would be disallowed. I sent some emails to tomaka and Osspial asking them to give @MarijnS95 co-ownership of the cargo-apk and ndk-build crates on crates.io because I was worried that @MarijnS95 wouldn't be able to make releases from the standalone repo.

It would be a bit inconvenient to transplant the commits to the other repo (but that has to happen regardless).

btw @MarijnS95 I was thinking we may be able to directly transfer any pending PRs/Issues for cargo-apk directly from this repo - though that requires that you have write permission to this repo to do that: https://docs.github.com/en/issues/tracking-your-work-with-issues/transferring-an-issue-to-another-repository
It would be rather awkward to land anything here because it means I have to re-run the filtering etc for the standalone repo.
If someone gives me write access to this repo I could look at transferring cargo-apk issues/PRs (probably best to wait until we merge PR #374 ideally)
Alternatively for smalls PRs like this then I can cherry pick the patches and create an equivalent PR

I've created an equivalent PR against the standalone repo here: https://github.com/rust-mobile/cargo-apk/pull/1

Disclaimer: What I ended up replying here should probably all have gone into #372...

aah, right, good point, for some reason I was thinking that once the repo is split then you'd end up loosing permission to publish because I was thinking that you were reliant on credentials that were associated with the android-ndk-rs repo.
If you're also a member of @rust-windowing/publishers then there's hopefully nothing to worry about in the short term, and we can follow up on any ownership changes for crates asynchronously

Yes I think it's fine for now, but we have to make sure that @rust-windowing/publishers doesn't kick me out after effectively all repos I need publish rights for have moved out. Until I receive the appropriate rights of course.
I will also see about updating the GitHub secret to use a new token from my account, instead of publishing every release (performed by me for the past year-or-so) via Francesca's account :grimacing:

btw @MarijnS95 I was thinking we may be able to directly transfer any pending PRs/Issues for cargo-apk directly from this repo - though that requires that you have write permission to this repo to do that: https://docs.github.com/en/issues/tracking-your-work-with-issues/transferring-an-issue-to-another-repository

I definitely wanted to transfer relevant issues to the other repo before opening PRs against it. That doesn't seem possible for PRs though, since they're heavily dependent on the available git history (I don't even think you can do it between direct forks).
However, it seems I'm only allowed to transfer to other repos under the @rust-windowing organization:

It would be rather awkward to land anything here because it means I have to re-run the filtering etc for the standalone repo.

That is true, and mentioned above by me; I just wasn't sure how far along your filtering was in terms of demonstrating "this is how it will end up looking like" versus "everything is moved now, please use the new system". Looks like the latter, I'll try and review your PRs soon to make sure the CI publishing keeps working when we land (the "transferred" version of) this PR.

If someone gives me write access to this repo I could look at transferring cargo-apk issues/PRs (probably best to wait until we merge PR #374 ideally)

I don't have enough rights for that, unfortunately. @rust-windowing/publishers, any help here? Can you also give me enough rights to access the settings menu and initiate the transfer to @rust-mobile? @madsmtm?

Alternatively for smalls PRs like this then I can cherry pick the patches and create an equivalent PR

It's weekend now, I can easily take care of transplanting my own PRs on Monday...

I've created an equivalent PR against the standalone repo here: rust-mobile/cargo-apk#1

But you already took care of that :no_mouth:


services: add balance of satoshis

Related: https://github.com/rootzoll/raspiblitz/issues/1142

kept under separate user to simplify uninstall and avoid running with sudo access

TODO:

[x] pin version to latest tested

sweet :)


#99: Remove merge button safe-guards (closes #99)

Issue #99
Test Plan
tests performed
I've built the extension and installed locally. All functionalities are still there, except for the gatekeeper.

@gabro I personally agree with removing it, but did you discuss it with others? Is anyone using it or willing to use it?

right, I just assumed nobody's using it since it's not working and nobody complained :-)
I'll ask


modelist123 recent commit

thank you for checking my contribution

The English version of these lessons lives in the primary folder structure, so not necessary to have an English translation file. Thank you for offering the contribution!


Release 1.1.2

@rultor release it, tag is 1.1.2.

@rultor release it, tag is 1.1.2.

@rafamizes OK, I will release it now. Please check the progress here

@rultor release it, tag is 1.1.2.

@rafamizes Done! FYI, the full log is here (took me 3min)


Remove specific class references when tree shaking happens

I always used proguard on android to remove any debug method when compiling for release. This help so much since I didn't have to remove them myself and there were still there in the debug version. For example for logs I would use:
-assumenosideeffects class android.util.Log {
    public static boolean isLoggable(java.lang.String, int);
    public static int v(...);
    public static int i(...);
    public static int w(...);
    public static int d(...);
    public static int e(...);
}

Is there a way to achieve this with Dart?

Dart does tree-shaking, no need to do anything manually.

You can use assert() to make code used only in debug mode.
https://stackoverflow.com/questions/24208213/how-to-exclude-debug-code/24208259#24208259
There is currently no better way, but there is an (one or more) issue in dart-lang/sdk to improve that.

looks good enough for now, can you share a link for the dart issue?

I would have to search myself but I'm sure I saw some discussions. Also discussed in related to requests for "macros" support.

https://github.com/dart-lang/sdk/issues/27192

Can we close this issue?
It's rather a Dart than Flutter issue.

sure


Enable URLSession tests in TestFoundation

URLSessions tests had been disabled because of a libkqueue failure seen in CI. See: https://bugs.swift.org/browse/SR-3464
Given that we've done away with libkqueue in Dispatch and SR-3464 is no longer relevant, can we re-enable the URLSession tests in TestFoundation?
cc @gparker42

@swift-ci please test

Yah, let's make sure this passes at least once then re-enable it for good. If it starts failing intermittently again we can re-disable it.

The only annoying thing about this test is that when run on Darwin the triggering of running the HTTP server (that is fired up to test the client) invokes the macOS firewall asking if you want to allow/deny connections. Can we make it conditional so that it doesn't run when kicked off from a desktop build in some way?

Unrelated failure. Re-triggering test.
@swift-ci please test

@alblue For that, we'd need to figure out how to disable the firewall in code. Btw, I came across a command that can enable/disable the firewall here: http://krypted.com/mac-os-x/command-line-alf-on-mac-os-x/

Or we could just not run the test inside Xcode?
Sent from my iPhone 📱

On 28 Feb 2017, at 06:28, Pushkar N Kulkarni<EMAIL_ADDRESS>wrote:
@alblue For that, we'd need to figure out how to disable the firewall in code. Btw, I came across a command that can enable/disable the firewall here: http://krypted.com/mac-os-x/command-line-alf-on-mac-os-x/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@alblue I am OK with that. What do you think @parkera ?

I think that if tests will be disabled for XCode, there should be at least some notification that they are disabled, since URLSession is not fully implemented.
Also, using commands from the link didn't give any results on disabling/enabling firewall, but maybe it's just me.

I think we could continue the discussion around the proposal for disabling URLSession tests in Xcode over the mailing list.

@swift-ci please test and merge