Carthage firebase, minimal used framework to use in device mobile text recognition and face detection

Hi,
I download the firebase iOS framework fo mobile text recognition and face face detection using carthage.
it download a lot of others frameworks more than 40. unfortunately this makes my app size grow. can you help me with the minimal configuration to run in device text recognition and face detection.

@soufianee You can try the following set:
binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseAnalyticsBinary.json"
binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseMLVisionBinary.json"
binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseMLVisionTextModelBinary.json"
binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseMLVisionFaceModelBinary.json"

binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseAnalyticsBinary.json" contains core Firebase components required for Firebase to work. If you are not going to use Analytics, you may skip copying frameworks like GoogleAppMeasurement.framework, FirebaseAnalytics.framework, FirebaseCoreDiagnostics.framework to you project to save some space.
Please let us know if it is helpful.

thank you for your response,
I try your solution, still get this errors

here is the list of imported frameworks

any idea please.

also
i have i delete the FIRAnalyticsConnector.framework. and still get this error and warnings

@soufianee what version of the Firebase frameworks are you using? FIRAEvent is a very old class that was renamed a while ago, so the symbols won't exist in the latest Firebase Analytics Carthage distribution.

hi,
fine build working now with the 6.27.0 version.
but I get this erreur in text extraction.
-[GMVTextDetector initWith:size:angle:]: unrecognized selector sent to instance 0x60000112e040
any idea.

Can you share the full stack trace?

hi,
here is the console trace

It looks like you may have duplicate class definitions in your app's runtime, based on the multiple Clearcut warning log. Can you confirm if this is the case? If so, you should remove those as they can cause undefined behavior at runtime.


Publicise the launch of the new PostHog Open Source

According to @tiina303 it sounds like we are ready to start letting people know about the availability of free Clickhouse to our users more broadly (relevant issue here).
2 questions:

How ready do we want to be before we start publicising? Do we want to make any further website and/or docs updates? (@kpthatsme @leggetter)
If we think we are ready now, how do we want to publicise? We could:

Include in the next PostHog Array as the headline item - the most recent one only went out yesterday though, and historically we've been sending these out every ~4 weeks
Send a one-off email announcing the change
Not send an email, and just publish on our socials and in the user Slack

I think the second option makes the most sense (once we're happy we're ready) given it's a big change that's good for our users, but keen to hear other perspectives on this. For context, we have 553 newsletter subscribers currently.
cc @shehuphd

We can do #2 and #3 - wouldn't worry about overloading our audience with emails at the moment since they're not that frequent.

I was also reminded today (https://posthog.slack.com/archives/C0113360FFV/p1627389342215900?thread_ts=1627373899.206200&cid=C0113360FFV) that we don't have a good story for migrations yet, but we likely want to have that FAQ at least beforehand (fast to add)

We'll potentially want to wait until https://github.com/PostHog/product-internal/issues/114 is resolved and chart default tag/sha updated

We'll potentially want to wait until PostHog/product-internal#114 is resolved and chart default tag/sha updated

Ok yep makes sense to wait then.

We can still prep everything ready to go now.

Action plan for next week based on conversation just now in Marketing sprint planning with @shehuphd and @leggetter:

[ ] Confirm that we have everything we need ready on the website and Docs (e.g. FAQ) - @leggetter
[ ] Create a layered blog post - @shehuphd

[ ] Intro announcement that we have just launched PostHog Free, which now let's you scale PostHog open source to mm+ of users, no longer limited. It's the same as the tech we use for PostHog Cloud, isn't that cool etc.
[ ] Then an interview with an engineer who worked on this to go into the technical depth.
[ ] Push out on usual channels and send as a one-off email to our list as well - @shehuphd
[ ] Contact Clickhouse and see if they want to share the story, mention us on their socials etc. - @piemets
[ ] Run some paid ads to support - @piemets

Just to mention that as with any new release we might discover some things we want to fix a bit, so maybe it's good to do the paid ads part a bit latter (thinking to target starting 2 weeks after the email).

Doing an interview with James G next week on this; will take it through the usual workflow.

@shehuphd as referred to here, we should just call this PostHog Open Source (new and improved, better, shinier etc)

Should we also pin the charts-clickhouse repo in the Posthog organization in GitHub?

We currently pin posthog, posthog.com and posthog-js. We can pin up to 6, so that seems to make sense! Anyone else we need to ask or shall I just do it?

Just noticed https://posthog.com/blog/the-posthog-array-1-27-0#clickhouse-is-now-free-to-use we already mentioned ClickHouse being free for everyone

@piemets @tiina303 can you update this so we know whether or not we need to carry this forward to the next sprint! Basically just looking for a what has been done and what is still outstanding that still needs to be done. Thanks!

In terms of this checklist, I believe nothing is ready to be checked off yet - @shehuphd is just back from holiday now and I think the next step is to have Sam edit the interview vide (@shehuphd please add any relevant updates here?)

Yes - first draft is done, finalizing video edits today for Sam. Will push a PR after our interview.

Excited to see it 🥳

First draft here - needs additional/more accurate details in the first part of the announcement.

@piemets - I think we should ensure we have the user guide docs in place. See https://github.com/PostHog/posthog.com/issues/1744

@piemets - I think we should ensure we have the user guide docs in place. See PostHog/posthog.com#1744

I think we can actually close this particular issue - we publicised the launch of Clickhouse Free back in August.
Oh actually, are you referring to this issue instead around funnels @leggetter?

@piemets ah, yes, sorry. I had it in my head that we were publicising the new amazing funnels experience. I don't know if we have an issue for that or plan to do it? Anyway, for this "new PostHog Open Source" issue it's not related so I wouldn't see the docs as a blocker.

Ok cool, we can close this issue then 👍


Web -  Refreshing the editing photo section will make the process to go back

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to settings > Profile
Click the avatar and then the "upload photo" Option
Choose the photo you want to upload and refresh

Expected Result:
It should still be in the "Edit Photo" section.
Actual Result:
Refreshing the section will make it go back to the profile section
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.40-5
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/b97112f5-2aff-42a8-9c4c-5074c76d9bc2
https://github.com/Expensify/App/assets/93399543/984c07e3-a7e9-4c6f-8671-ce1aaa974f95
Expensify/Expensify Issue URL:
Issue reported by: @Yokabdk
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1689344116454879
View all open jobs on GitHub

I agree. I think auto-saving is a feature, and not doing that would be a regression actually.

We decided to close this for now.


fix: linting is broken if dependant project doesn't have postcss-scss package in deps

@ogonkov @ValeraS @amje Could someone take a look?


wallet.sol can't be compiled on latest ethereum wallet

When I try to compile wallet.sol in latest ethereum wallet I get the following error:

Return value of low-level calls not used.
_to.call.value(_value)(_data);
^---------------------------^
,

That's a good thing! The limit functionality is totally broken. Some people have suggested better wallets. Let's not use this anymore.

commenting that lines allows to compile this contract, but even if you manage to compile it, it's totally broken, you can't even receive any deposit.

This wallet seems much cleaner, perhaps you can use that: https://github.com/ConsenSys/MultiSigWallet/


add Beacon definition.

Add explanation of Beacon chain to the glossary of terms
Any helpful background information around the changes?
no dependencies added
No relevant screenshot/gifs
closes/addresses issue: [Definition Request]: beacon chain ](https://github.com/narbs91/glosseta/issues/87)
Reviewer: @narbs91

Transaction link: https://viewblock.io/arweave/tx/0lfxZ6v3WrdNNUXqBursT348jrENo-6kXKpVrlANQpU

@all-contributors please add @naftalimurgor for data


Move DataBuilder code into a shared class

It seems that the DataBuilders have quite a bit of code shared with their corresponding DataProcessors. This PR moves the code into a shared class, SpongeDataBuilder, which uses reflection for create() and delegates createFrom(DataHolder) and build(DataView) to the DataProcessor. This PR also provides implementations for ImmutableDataBuilder via SpongeImmutableDataBuilder.
@gabizou Thoughts?

So you're replacing a direct implementation with one based on reflection, you should probably consider the performance impact of all of those reflection calls, in particular I was thinking about the frequency that create() may be called.

@Deamon5550 I think that depends on whether the DataManipulatorBuilders are used from Sponge itself or only from plugins using the API. In the latter case I figure the calls are very few and the reflection not that pricy. Far worse things happen underneath the Data API implementation (e.g. synchronous GameProfile lookups on the main thread when setting a Skulls owner).

Just something to keep in mind.

DataManipulatorBuilders are used for EVERY DataManipulator, regardless whether it's Sponge provided or plugin provided. At the very least, I was considering trying do this very same idea, however, again, the issue is reflection.
Another thing is: Unnecessary creation of various DataManipulators without validating first that the DataContainer has the proper data is an issue by itself. I was considering however, segmenting out create() and createFrom(DAtaHolder) such that method references could either be sent out as Functions or something thereof.

You've got some interesting changes that I don't quite see as to why they're resulting in entire files being deleted and rewritten. I think I have an alternative solution to reflection though that will make DataManipulatorBuilders work in a factory sense.

So, if it is line endings issues: https://help.github.com/articles/dealing-with-line-endings/#platform-windows is a good way to fix them. Unfortunately, it means having to redo any changes of the files that git already marked as re-created.

@gabizou Thanks.

@gabizou I just thought of a way to avoid the reflection performance costs. When the DataManipulator is registered, I create one and only one instance of the ImmutableDataManipulator using reflection. DataBuilder.create() will convert it to a mutable DataManipulator and return that, while ImmutableDataBuilder.createImmutable() will return the immutable single instance.

@JBYoshi I was just fiddling with DataConstants, I realized that there would've been a big issue with the way you approached this is that serialization setup occurs during the preinit phase of the SpongeGameRegistry whereas the CatalogTypes are all registered during the init phase, of which any references to the actual static values would've equated to null at runtime during the preinit phase of which you were hoping to use an ImmutableDataManipulator variant for the DataManipulatorBuilders. The issue with this is that since you now have this ImmutableDataManipualtor that would've still failed to initialize due to the various null checks, it would've been impossible to have this work when in the context of actually working with the CatalogTypes that haven't been initialized yet.
Granted, with the DataConstants I have to be extremely careful with the static initialization of the DataConstants class, so there's some delicacy to using various CatalogType constants throughout the Data implementation that have to be carefully used only and only when the SpongeGameRegistry has completed all registrations.


Remove ? from middleware done arg to avoid checking it's presence

If I understand correctly, the question mark was there with intention to use it in both variants:
// sync
router => (toState, fromState) => true
// or async
router => (toState, fromState, done) => {
  setTimeout(() => done(), 500)
}

But in reality it's not helping in any way, because it's treated not like "we can define function with this argument or not", but instead "when we're called it can be undefined".
And AFAIK if we got that arg defined in middleware, then it will always be defined and act asynchronously. Is It correct?

Hi @faergeek, you are correct: middleware functions are always called with done (https://github.com/router5/router5/blob/master/packages/router5/modules/transition/resolve.js#L32).
👍


Add S3/GCS env vars for unit tests

The distribution storage unit tests are basically no-ops unless actual AWS or other provider credentials are set. We need to set the env vars at https://travis-ci.org/deis/distribution and ensure we're actually running all the storage tests so we have confidence our modifications aren't breaking things.

This is done, see https://travis-ci.org/deis/distribution/builds/113528041.


RAINCATCH-652 - Integration with file-storage module

Motivation
Integrate storage module into cloud app.

Providing PR in order to verify changes.
I do not think that we should merge this changes to demo.
Demo app should have file module that would use local file storage (See flag in file module)
Having storage module for gridfs or s3 would complicate application setup.
Developers who wants to use storage module can import it to their app etc.
@nialldonnellyfh WDYT ?


Home Button Needed

Currently no way to get back to homepage once you click the wiki buttons.

resolved, put under developer menu for now as full app wont need it.


netdata: Access to file is not permitted

Issue description
After upgrade to 18.09 (from 18.03), netdata shows only this error in the browser:

Access to file is not permitted: /nix/store/lj1ifxwl3cw8xg9l9a3cgmxip59im26a-netdata-1.11.0/share/netdata/web//index.html

The strange thing is that i can access it without problems on console:
[root@nas:~]# sudo -u netdata less /nix/store/lj1ifxwl3cw8xg9l9a3cgmxip59im26a-netdata-1.11.0/share/netdata/web//index.html

[root@nas:~]# echo $?
0

A similar patch was removed here and a test was added, that might not have detected this problem.
https://github.com/NixOS/nixpkgs/commit/0254ae4e80f9bf659430d2ec3e452ccb049ce2fb#diff-c080a3ec04b35f95254741800e1a3697
This test should also have failed:
curl -s http://localhost:19999/api/v1/data\?chart=users.pwrites | \
...

Response for http://nas.lan:19999/api/v1/data/?chart=users.pwrites on my system is:
Chart is not found: users.pwrites (HTTP Status Code 404)
I can reproduce this on 2 separate systems.
cc @lethalman
Steps to reproduce

have netdata enabled
try to access it with browser

Technical details

system: "x86_64-linux"
host os: Linux 4.14.83, NixOS, 18.09.1446.db561c90ef3 (Jellyfish)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.1.3
channels(root): "nixos-18.09.1446.db561c90ef3, nixos-unstable-19.03pre159472.80738ed9dc0"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

I am having the same problem. I've had netdata installed for a while and now I'm getting
Access to file is not permitted: /nix/store/lj1ifxwl3cw8xg9l9a3cgmxip59im26a-netdata-1.11.0/share/netdata/web//index.html
Stat gives me
  File: /nix/store/lj1ifxwl3cw8xg9l9a3cgmxip59im26a-netdata-1.11.0/share/netdata/web//index.html
  Size: 328676    	Blocks: 648        IO Block: 4096   regular file
Device: 1ah/26d	Inode: 78177322    Links: 1
Access: (0444/-r--r--r--)  Uid: (    0/    root)   Gid: (    0/    root)
Access: 2018-11-19 04:42:08.000000000 +0000
Modify: 1970-01-01 00:00:01.000000000 +0000
Change: 2018-11-19 04:42:08.381484497 +0000
 Birth: -


URLs not working correctly due to HTML 5 mode!

Problem
With HTML5 Mode when going to a page like http://localhost:8080/home/student works within the application because it's basically one html page.
However, when I try to refresh, it says 404 not found. I did some research and realized it's because the server is searching for home/student.html which is not there obviously since everything is served from index.html, and angular just routes that one page internally.
Solution

Solution could be to not use html5 mode, which will make the page look like this http://localhost:8080/#/home/student
Or have a catch all on the server that right away redirects to index.htm

another solution is to make all frontend urls contain a . (period)
for example; /home/student**.htm**
marking as deferred and low priority


Display The Schedule

Once we have a schedule, we need to display it.

We now have display pages for the schedule; will create more specific issues for it as they arise


How to separate unlisted extensions from the listed ones?

Today under the extensions directory, we have some listed extensions, such as app-configuration or storage-blob as well as unlisted ones http-client-vertx or core. So, looking at the repo and the structure of the directory, it's not clear of which are extensions that can be used by external projects or not.
Wondering if we could have an extensions directory and an internal directory ? Or maybe just prefix the listed extensions with azure, this way we will have azure-app-configuration or azure-storage-blob
@ppalaga any thoughts on that ?

BTW @ppalaga @radcortez @maxandersen @zbendhiba any suggestion on how to refactor the repo so listed extensions are easier to find?

We don't have a strict structure to follow. In Quarkus, we have everything in extensions and core / common stuff in a separate core root folder. So maybe we could do something like:

bom
core

common (moved from extensions - old core)
http-client-vertx (moved from extensions)
jackson-dataformat-xml (moved from extensions)

docs
extensions (public extensions for azure)

azure-app-configuration (rename from app-configuration)
azure-storage-blob (rename from storage-blob)

integration-tests

I wouldnt tie physical repo structure to what is made availble by tooling or not. That most likely evolve.

@maxandersen no but I like to have extensions under the extension directory. Today there are also internal content under extensions, which is not very clear (even to me)

Sorry for coming so late. Yes, I agree having the ancillary extensions in a separate folder might be practical for several reasons:

Contributors can see what is intended for public use
Various checks can be bound to those directories: quarkus metadata, existence of a documentation page, existence of a test, etc.

Or maybe just prefix the listed extensions with azure, this way we will have azure-app-configuration or azure-storage-blob

quarkus-azure- prefix is better IMO. The azure- prefix might awake the impression that they are some sort of official Azure artifacts, which they are not.

Good point @ppalaga. I created https://github.com/quarkiverse/quarkus-azure-services/issues/64


Ignore Eclipse ".project" file

Eclipse automatically creates a file named ".project" for every single project, just like Visual Studio Code creates a ".vscode" file for workspace settings.

I am more than happy to merge this - but we've had a few of these now and I wonder if we can improve the situation.
Is there any reason why we cannot simply add .* to the gitignore file to cover all such files? We do have the .travis.yml file which may need updates - can we handle this still?

Just to chime in – I don't really appreciate extra content related to particular IDEs that I don't have maintained in repos I clone.
I'd point people to Explicit repository excludes.

I think there is a balance to be struck. For example, so many people use vim that it makes sense to have this handled in the gitignore file.
Personally I don't find it too problematic to merge in one of these PRs every now and then :)

I think the .gitignore approach is harder to maintain but feels more practical. Anyway, I am cool with any convention that is going to be established.

I'm merging this for now. If this arises again then I will add a note to the contributing docs and advise others to use the repository excludes as suggested by @andrewcsmith .
Thank you both.


Handle/flag hard-coded references

Describe the feature
I would love for this package to be able to handle (and flag?) hard-coded references (ie raw.stripe.payment vs. {{ source ('stripe', 'payment') }). I specifically noticed that 'fct_unused_sourcescurrently returns sources that are accessed only via hard-coded references instead of thesourceorref` macros. There may be other applications for this functionality within the package as well.
Describe alternatives you've considered
The minimum response to this issue would be to add a note to the readme regarding the results of the fct_unused_sources model to include this possibility. A value-add in that documentation could be a recommendation about how to discern whether the sources are truly unused vs. accessed through hard-coding. For instance, in my case I used the list generated by fct_root_models as a starting place when looking for hard-coded references.
Additional context
This should apply to multiple databases.
Who will this benefit?
This would benefit anyone with hard-coded references in their project, allowing them to systematically transition to appropriate alternatives (source and ref functions).
Are you interested in contributing this feature?
Sure! And I'd be glad for some discussion/input around how to approach solving this.

Hi! Didn't realize we already have an old issue about this - let's move discussion over there. I'm going to try to tackle this today.


Deprecations in 0.9.0 are a little severe

We just pulled in 0.9.0 with its after_hooks change. Nicer API, good work.
Except the checkers methods are spamming deprecation warnings every time they are called (which as you'd expect can be multiple times per test run). I like pulling in the latest versions of things, but it'd be a nicer experience for the user to be a bit gentler with the deprecations. Right now my choice is between changing my code to 0.9.0 (which means if there is a bug forcing a rollback to 0.8, I need to revert this commit as well), or pinning to 0.8.0 until I'm more confident in 0.9.0 (webdriver stuff is so finicky, in fact the recent 2.48.0 selenium webdriver update just hosed us for some reason). FWIW I pinned back to 0.8.0.
I think it'd be nice idea to either 1) warn about deprecations only the first time a method is called, or 2) release 0.9.1 or 2 in a few weeks, once 0.9.x has baked a little, and give it the current aggressive deprecation warnings.
Thanks and hope that didn't come off as entitled- just a (self-interested) suggestion that I and probably other users would really appreciate.

First, what issue are you having with 2.48?
Do I understand you correctly that your concern is with the cluttering of stdout with deprecation warnings?

Not sure what the 2.48 problem is- it only fails on headless on our Ubuntu CI machine (FireFox, not sure about others). It's too difficult to debug remote headless webdriver specs so I don't even have enough info to file a bug report (beyond "telling something to click didn't actually click it" but I doubt that's universally true).
Yes, my stdout is cluttered with deprecation warnings.

You should only be seeing deprecation warnings when you are adding or removing checkers, which shouldn't be all that often. Regardless, warnings should always happen on every call that is made with a deprecated method until support for the deprecated method is removed. The goal is to get people to update their code (and this one in particular should be relatively simple).
Definitely ping me on the Selenium issue tracker if you figure out your Firefox problem.


Inliner v1 not inlining

Inky Inliner v1 is not inlining css into html. When the inliner is run it just hangs u saying that it is loading.

Just found that this was already opened by someone else


update goimports URL

The source still says "https://github.com/bradfitz/goimports" for the goimports URL.
Please update it to https://golang.org/x/tools/cmd/goimports
Thanks!

Patches welcome... I kid, fixed in 737611ee2f8ffd6ae573e37cc04c4ad4abe7697d.


feat(form-field): Initial Implementation

Add mdl-form-field package
Refactor existing input controls to use mdl-form-field

Resolves #4475
[Delivers #130834979]

False negatives from TravisCI for node v4. Flaky SauceLabs. Merging.


Update to Python 3.4 or newer?

Are there any plans for updating to a newer version of Python 3.x? Many libraries on PyPI, for instance, only support Python 3.4 or newer.

https://github.com/qpython-android/qpython3/releases says Python 3.6.1 is in the pipeline, in the testing phase, so I'm closing this issue.


Add support for Serbia holidays

Hi, this is my first contribution. Trying to add support for Serbia holidays.

Pull Request Test Coverage Report for Build 663

55 of 62 (88.71%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 99.902%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

holidays.py
32
39
82.05%

Totals

Change from base Build 662:
-0.1%

Covered Lines:
28564

Relevant Lines:
28592

💛  - Coveralls

Hi,
very good work, thank you very much for your contribution!
I've merged it to beta.
Cheers
dr-p


MINIFI-153 - Adding proxy settings to remote process group

Thank you for submitting a contribution to Apache NiFi - MiNiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with MINIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi-minifi folder?
[x] Have you written or updated unit tests to verify your changes?
[x] - N/A - If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[x] - N/A - If applicable, have you updated the LICENSE file, including the main LICENSE file under minifi-assembly?
[x] - N/A - If applicable, have you updated the NOTICE file, including the main NOTICE file found under minifi-assembly?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Tested with MINIFI-153 branch of my testkit.

Reviewing

Hey @brosander, I just verified with secure and in-secure proxies hitting multiple different logical networks. All worked as expected.
One more thing that should probably be added is a validation such that if RAW is set then the proxy config should not be. (Since they are mutually exclusive)
Aside from that, I am a +1

@JPercivall added validation that if proxy host is set, transport protocol should be http

+1
Visually verified code and did a contrib-check build. Ran multiple different flows which touched a regular NiFi instance (RAW and HTTP) and utilized a secure and unsecure proxy to S2S to instances that were on a different logical network. Everything worked as expected.
Thanks @brosander, I will merge it in.


[#5384] Cleanup code related to cluster

Remove all use of PinpointServer within the ZooferProfilerClusterManager class

Codecov Report

Merging #5438 into master will increase coverage by <.01%.
The diff coverage is 69.23%.

@@            Coverage Diff             @@
##           master    #5438      +/-   ##
==========================================
+ Coverage   40.98%   40.98%   +<.01%
==========================================
  Files        2762     2763       +1
  Lines       83483    83480       -3
  Branches    11123    11124       +1
==========================================
+ Hits        34214    34218       +4
+ Misses      46262    46250      -12
- Partials     3007     3012       +5

Impacted Files
Coverage Δ

...tor/cluster/zookeeper/ZookeeperClusterService.java
29.41% <ø> (+0.67%)
:arrow_up:

...nt/rpc/ExecutionThreadStateChangeEventHandler.java
0% <0%> (ø)
:arrow_up:

...r/cluster/connection/CollectorClusterAcceptor.java
0% <0%> (ø)
:arrow_up:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
79.92% <100%> (+0.7%)
:arrow_up:

.../pinpoint/rpc/LoggingStateChangeEventListener.java
100% <100%> (+50%)
:arrow_up:

.../server/handler/ServerStateChangeEventHandler.java
100% <100%> (ø)

...tor/receiver/AgentLifeCycleChangeEventHandler.java
100% <100%> (+15.38%)
:arrow_up:

...inpoint/rpc/server/DefaultPinpointServerState.java
94.91% <100%> (ø)
:arrow_up:

...tor/receiver/thrift/tcp/AgentBaseDataReceiver.java
66.66% <100%> (+0.87%)
:arrow_up:

...inpoint/rpc/client/PinpointClientHandlerState.java
75.9% <50%> (ø)
:arrow_up:

... and 11 more

Continue to review full report at Codecov.

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


Gateway added to next element in process graph, not to the selected task

Describe the Bug
When adding a gateway to a task, it looks as if it is added to the next element in process graph, not to the selected task.
Steps to Reproduce

create a new BPMN diagram with three sequentially connected tasks (A,B,C)
use fly-out menu of first task A to add a gateway connected to task A
it looks the gateway is connected to task B instead of A

Expected Behavior
Connection alignment for gateways should be consistent with alignment for other elements.

Environment
Camunda Modeler version 3.3.5
Windows 10

When you move the created Gateway, you will discover, that it is added to the selected task. What is kind of broken her is the connection alignment behavior, which should work as for all other elements.

You are right. Then the issue is obviously less significant. I update the ticket description accordingly. Thannks!

The gateway is added to the right task, the connection might be partially obscured by another element. That's not considered a bug. I'll close this issue.


Syntax error near 'order' while executing moor generated sql

Moor generated sql causes error near "order": syntax error
The sql itself is:
CREATE TABLE IF NOT EXISTS audio_stream_db (event_id INTEGER NOT NULL, url VARCHAR NULL, name VARCHAR NULL, `order` INTEGER NOT NULL, PRIMARY KEY (event_id, order));
Maybe it has something to do with the fact that 'order' is a keyword in sqlite?
My local solution was just to call a field "position" instead of "order", but maybe it is something you'd want to address. Thanks for the great library.

Thanks for the report. As you said, the problem is that order is not escaped in the primary key clause. I think that issue should be fixed in the latest moor version (see also #661). Can you try to run flutter pub upgrade and check if that fixes the problem?

Thanks for the report. As you said, the problem is that order is not escaped in the primary key clause. I think that issue should be fixed in the latest moor version (see also #661). Can you try to run flutter pub upgrade and check if that fixes the problem?

New version after flutter pub upgrade resolves the issue indeed.


Bug: Spark JOIN returns cartesian product instead of normal join

Failing test: sameFieldName.test
Not that PlannerSpec covers joins and those tests are passing.
Needs further investigation

I don't this this is a Spark issue (or at least not only a Spark issue) as it looks like the QScript plan is currently nonsensical for the query, see https://github.com/quasar-analytics/quasar/issues/1581#issuecomment-263738179 for more detail.

I need to go to @wemrysi & @sellout academy to improve my qscript reading skills :)
Thx for the info!

Fixed in https://github.com/quasar-analytics/quasar/pull/1922


[MRG] Quick fix of failed tests due to new scikit-learn version (0.20.0)

Fixes #126 #127
This is a quick fix that fixes #126 and #127, by setting ensure_min_samples=2 in checks of the inputs to pass scikit-learn's check_estimator test (see #127), and by updating the scores with the new values for the iris dataset.
But I opened two issues that tackle solving these problems in the long term (cf #127)

issue #128 to decide what is the minimal number of samples that we want to allow the "fit" method on
issue #129 to deal with tests that test some hard values

Some iris tests are still failing ;-)

Oh yes indeed, I used the wrong scikit-learn version for testing locally :p


www/caddy: Refactor general.volt to base_tabs, Update DNS Providers

This one was long overdue.
It refactors the general.volt completely to use base_tabs instead of multiple base_forms as its layout_partials.
The code is a lot cleaner now and many weird custom constructs could be removed:

the weird validation workarounds because it was not appended to all fields
the custom apply and save buttons
the custom tabs that mapped the base_forms into seperate tabs (yikes)

All the prior functionality is retained and improved:

message area still exists
error messages will trigger a danger dialog instead of info
Caddyfile validation is still included inside the SafeFormToEndpoint

In the same PR:

the DNS Providers are updated and some smaller options to the model are added to prevent merge conflicts.
reverse_proxy.volt style has been adjusted to fit with the new general.volt style
Removed useless help text in diagnostics.volt

Fixes: https://github.com/opnsense/plugins/issues/4190

LGTM
FWIW, we could keep this on devel for 24.7.4 if you are unsure about overall impact.

I think this one is pretty safe. I will test this a bit more in my freetime on my production setup and if everything works there its good to ship normally.


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
14 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
17 seconds
deprecated-base-image-check

✅ Succeeded
11 seconds
sanity-inspect-image

✅ Succeeded
55 seconds
clamav-scan

✅ Succeeded
16 seconds
clair-scan

✅ Succeeded
12 seconds
sbom-json-check

✅ Succeeded
8 seconds
sanity-optional-label-check

✅ Succeeded
8 seconds
sanity-label-check

✅ Succeeded
4 seconds
show-summary

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
14 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
11 seconds
sanity-inspect-image

✅ Succeeded
17 seconds
deprecated-base-image-check

✅ Succeeded
55 seconds
clamav-scan

✅ Succeeded
16 seconds
clair-scan

✅ Succeeded
12 seconds
sbom-json-check

✅ Succeeded
8 seconds
sanity-label-check

✅ Succeeded
8 seconds
sanity-optional-label-check

✅ Succeeded
4 seconds
show-summary


Missing link(s) to video/doc

I noticed the following text on the main page (Security > Advanced Network Policy item) without any links:
Get started here or watch a video to learn more.

I can make a fix to add the links, but I am unsure what the intended link targets were here.

Thanks for pointing this out. However, we will be updating this section with #226 so I wouldn't want you to have to do some work that is going to be removed soon :)


Added spanish translation

I was using the edit_skin mod on a server I am creating. The server was in Spanish so I have added the Spanish translation.

Thanks!


Android 10 start activity from background not working, foreground() and unlock() not working

Android 10 start activity from background not working, foreground() and unlock() not working.

Yes, i confirm ANDROID 10 is restrictive on starting activity from background, not allowed on API 29 / ANDROID 10, you must go to SETTINGS → Apps → Your App → Advanced → “Show app on top”, and NOW will work.
Just test it on CORDOVA and emulate with an ANDROID 10 version. Send it to background and then after a few seconds try to bring it back with unlock() or foreground();
Thanks.

I worked out a temporary fix, is working perfect.
Change files:
→ BackgroundModeExt.java
Add this function at bottom
//look for execute function that starts with `public boolean execute(etc, etc)`
//right under in switch(action){ etc }, add this:
    case "permissionontop":
        RequestPermission();
    break;

//send user to settings panel where he must allow "app on top" option
//it makes sure only for ANDROID 10+ versions
private void RequestPermission() {

    if (SDK_INT < 29) {
        return;
    }

    Activity activity = cordova.getActivity();
    Intent intent = new Intent(ACTION_MANAGE_OVERLAY_PERMISSION);

    cordova.getActivity().startActivity(intent);
}

→ background-mode.js
Add this function
/**

Set permission to run on top > API 29.

@return [ Void ]
*/
exports.permissionOnTop = function()
{
if (this._isAndroid)
{
cordova.exec(null, null, 'BackgroundModeExt', 'permissionontop', []);
}
};
→ In your cordova project

//it will send the user to app settings to allow app on top
cordova.plugins.backgroundMode.permissionOnTop();

Hi
I get these error:
D/SystemWebChromeClient: file:///android_asset/www/index.html: Line 125 : Uncaught TypeError: cordova.plugins.backgroundMode.permissionOnTop is not a function
What do i make wrong ?

Got to make ich working.
But at least, in Android 10 the App: ACTION_MANAGE_OVERLAY_PERMISSION  can not add new apps :-(
What is todo, that my app ist listed there also?


Debug Run from VS Code fails

Hello,
When I launch the app it works fine, but when running it in debug from VS Code, it fails (loading robot from urdf file):
Tha page (html) is correctly displayed, but no DOFs are loaded, and I get an error from threejs :
this.world.add(robot);

It seems that the 'traverse' method is never called (so the 'robot' object is not defined):

Any idea?
Thank you

Problem is now solved:
Need to specify this kind of url in the launch.json config file


fix(circuit relay): Auto-discover IP4 addr when CR enabled

:tada: This PR is included in version 1.4.29 :tada:
Your semantic-release bot :package::rocket:


Ripple effects

Found this pretty awesome codepen with ripples in CSS only! can we "borrow"?

Do you have a link to it?

Oops forgot to link: http://codepen.io/sebj54/pen/oxluI

Hmm. I'm not sure how I feel about this. I think it's kind of confusing to see a circle emanate from the center of the button instead of your mouse.
Maybe it could be an optional style.


Create empty state screen

It should display information that User is doesn't have any active subscription yet. For inspiration, you can check https://emptystat.es website.
For CTA we can think about it later, but I think, this:
You doesn't have any subscription yet

               [Add Now]

Is enough. Zeplin URL is TBA.

Mobile pak

Ini udah ada atau belum?

blm. wanna grab it?

Reference: https://xd.adobe.com/view/635e7ac7-86f2-4141-5bb6-607baaf8cf77-156f/


Trying to run Docker

Hi
I think that it is a basic step, I was googling about it but i can't find what is

the correct path to this github repo

to be replace in docker command.
Could anyone help me with this?
Thank you in advance
M

Hey @miguelp77, Its going to be the path to the directory where you have cloned/downloaded this github repo. For example, I downloaded the repo as a folder introtodeeplearning_labs in my Downloads folder. Since I am using my mac, my path to the repo is now ~/Users/avi/Downloads.

Sorry for the late reply! Yes like @avidiotic mentioned just replace that path to where ever you download this repo to on your local computer.


replace favicon

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 76.161%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
246

Relevant Lines:
323

💛  - Coveralls

fix again
https://realfavicongenerator.net/favicon_checker?protocol=https&site=codeigniter.com%2F#.Yb-9s2hByMo

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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 76.161%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
246

Relevant Lines:
323

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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 76.161%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
246

Relevant Lines:
323

💛  - Coveralls


util.prompt

hi there,
I've opened this on the master branch, but according to this branch, the master branch is kinda dead.
So here goes:
id='my id'
secret='my secret'
url='https://mywebsite.mydomain/callback/'
username='my username'
scope='some scopes'
then I do:
token=util.prompt_for_user_token(username ,scope, client_id=id, client_secret=secret, redirect_uri=url)
paste the url, and get:
spotipy.oauth2.SpotifyOauthError: Bad Request
Any thoughts

Is there a specific reason you want to use the prompt_for_user_token method of creating a spotipy client?  I was able to successfully create a spotipy client by creating a SpotipyClientCredentials object and passing it to the static Spotify method on spotipy.  See this thread for more info https://github.com/plamere/spotipy/issues/194#issuecomment-315458391

Hi there,
Yes, part of my application's purpose is to be able to give the user the ability to modify their own stuff.
Wondering if there is a back alley way of doing it? I don't like prompting in terminal anyway, so wonder if there is a way to get around it and make it work

OAuth specifies that the user is redirected to a predefined address after login. So some way of retrieving the redirect address (or more specifically the code parameter) is required to get the tokens. It can be the user copy-pasting to a terminal prompt or, for a smoother user experience, a server listening to that redirect address. The server doesn't even have to be accessible from outside your computer. Without knowing more about your application, perhaps that could be worth a try.


Implement Hotkey for Pasting as Plain Text Without Opening Clipboard Menu

Before Submitting Your Feature Request

[X] Check that there isn't already a similar feature request to avoid creating a duplicate.
[X] I have seen the FAQ.

Problem
The current process of pasting as plain text in Maccy requires opening the clipboard menu, which can be time-consuming and disrupt workflow.
Solution
Introduce a hotkey (suggested: Ctrl + Shift + V / ⌘ + ⇧ + V) that pastes as plain text directly, bypassing the need to open the clipboard menu. This would enhance Maccy's usability and efficiency, particularly for users frequently needing to paste non-formatted text.

You can add your own Shortcut assigned to a specific key that would do the same thing for you. Here is an example:

You can install it from https://www.icloud.com/shortcuts/72e2ec90a9d94086af6cdeacb94dc74f, then use with Control+Command+V.

macOS does have a native shortcut for pasting as plain text.
⇧ Shift + ⌘ Command + ⌥ Option + V
Ref: Hacker News thread


adds Github actions for CI build

Description of changes:
This PR works on adding Github Actions for CI build.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Closing this PR to try https://github.com/amzn/ion-schema-kotlin/actions/new for creating the workflow here in order to test if it works properly.


Configuration Button is not working

Panel or Daemon: Panel (Probably HTML error)
Version of Panel/Daemon: Panel : 0.6.4 Daemon : 0.4.5
Server's OS: Ubuntu Server 16.04 [Linux(x64) 4.4.0-104-generic]
Your Computer's OS & Browser: Windows 10 Pro; Google Chrome | 63.0.3239.108 (Official Build) (64-bit) (cohort: Stable)

Add Details Below:
Clicking on Configuration should open up dropdown menu as I remember, but it only redirects to /#

I need that to get database password. I reinstalled the panel at least three times cause of errors. Of course I reinstalled OS too. (By formatting drive so nothing should be wrong) I don´t know if I only made some installation error, but terminal does not thrown any errors on me. So I think error is somewhere else. Please fix.

This should work. Make sure there are no errors related to the javascript not getting loaded.

I've never heard of anyone else having this issue on the stable build and the information provided leads me to believe it is an issue with your browser caching something wrongly, or you're using a plugin that is blocking JS from being executed.


chore: update scan and add param

Details
Testing new scan package in pipelines
Motivation

Context

Pull request checklist

[ ] Addresses an existing issue: Fixes #0000
[ ] Added relevant unit test for your changes. (yarn test)
[ ] Verified code coverage for the changes made. Check coverage report at: <rootDir>/test-results/unit/coverage
[ ] Ran precheckin (yarn precheckin)

Codecov Report

Merging #1826 (0329216) into main (2199493) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Current head 0329216 differs from pull request most recent head 4e494b2. Consider uploading reports for the commit 4e494b2 to get more accurate results

@@           Coverage Diff           @@
##             main    #1826   +/-   ##
=======================================
  Coverage   94.52%   94.52%
=======================================
  Files          38       38
  Lines        1059     1060    +1
  Branches      145      146    +1
=======================================
+ Hits         1001     1002    +1
  Misses         58       58

Files Changed
Coverage Δ

...kages/shared/src/scanner/crawl-argument-handler.ts
100.00% <100.00%> (ø)


Making target value and max value the same as a variable

So I'm using bar-card for a pH meter and with static values works great but I have a different sensor that returns the value of neutral pH from some calculations based on temperature and I would like for the value in target to be this sensor (entity) value.
What I have:
- entity: sensor.green_house_1_intake_water_ph
    name: Intake Water PH
    max: 14
    min: 0
    entity_row: true
    target: 7
    type: 'custom:bar-card'

What I want:
Entity sensor.green_house_1_intake_neutral_ph value is 8 so max should be 16 (8*2) and target 8
What I tried:
- entity: sensor.green_house_1_intake_water_ph
   name: Intake Water PH
   max: sensor.green_house_1_intake_neutral_ph * 2
   min: 0
   entity_row: true
   target: sensor.green_house_1_intake_neutral_ph
   type: 'custom:bar-card'

Is this possible if yes how can I achieve this?

I agree with @DEADSEC-SECURITY, it would be a nice feature to be able to set dynamic values for max and target (maybe even min if there's a use case for that).
I found a possible solution here:
https://community.home-assistant.io/t/100-templatable-lovelace-configurations/105241


How to get contents of file in JavaScript?

I've tried both ng-model and the output of the function tied to ngf-drop, but all I get is an object with metadata about the file, which is useful, but is not the actual file's data. How can I get the binary or textual contents of the file without sending it to a server?

Ask it on Stackoverflow please since it is a question.
You can read the file content with FileReader api.


Outdated maintainers file

@DagAgren: please update when you have a moment.

@DagAgren I removed you because I don't have an email address. @wojciechczerski is now the listed maintainer. Feel free to add yourself again with the current address.


Feature: Add full message content support

Hey, just found this library and I'm ridiculously happy it exists as I haven't found anything else other than the original library this is based off of.
Just wondering if you'd consider adding support for the full avatar, name, message content as well as just the embed?
Alternatively can you just expose the Markdown parsing so it can be used to parse any message?

Hey, glad this could help you ;)
Yes, I will expose the Markdown parsing in a few hours, i'll see later for implement complete message parsing.
(you can also make a pull request if you want)

Pulled the latest release, works perfectly for me now.
Thanks for responding so quick, if I ever finish the thing I'm trying to do I'll add a PR for the full message support


[Unbelievaboat] Fix depositing

Before this change, this wasn't changing wallet balance properly.

oops, thanks lol


Using Steam.sendMessages causes segmentation fault

Describe the bug
Any attempt to use the Steam::sendMessages function leads to a segmentation fault and the application crashes. Looking more into the SteamNetworkingSockets SendMessages, the GodotSteam usage on Line 4014 seems to be providing an incorrect first parameter. According to the documentation, SteamNetworkingSocket's sendMessages expects the first parameter to be the number of SteamNetworkingMessage_t messages that will be provided in the second parameter.
Bug 1: The segmentation fault occurs partially due to the first parameter being the number of bytes in the message. For the current implementation, I believe a temporary fix would be to set the first parameter to 1 as there is only ever 1 SteamNetworkingMessage_t message allocated and passed.
Bug 2: When I tested with a PackedByteArray data size of 1 byte, the server application still crashed when calling this method. I suspect this crash is due to the SteamNetworkingMessage_t not having its m_pfnFreeData member value set to a callback function that would free the data payload's memory. Refer to SteamNetworkingUtils' AllocateMessage documentation for more details on its expectation when it is called with a parameter of 0. This is a bit more tricky to fix I believe as PackedByteArrays are not refcounted to my knowledge and you need to ensure the PackedByteArray is not freed before the m_pfnFreeData callback is called. A simple solution would be to change to using AllocateMessage(data.size()) and copy the data payload into it, but that would kill the performance optimized purpose of SendMessages.
Separately, I suspect the method signature for Steam::sendMessages that is exposed to the gdscript side likely will need to change since this current signature only supports sending one message at a time, despite the method name of sendMessages (one or more). Something like the following may be needed to properly support the full capabilities of what SteamNetworkingSocket's SendMessages is capable of:
Array Steam::sendMessages(Array messages);

Where each message in the array is a tuple of destination_connection_handle: int, send_flags: int, data_payload: PackedByteArray.
To Reproduce
Steps to reproduce the behavior:

You will need two applications running, one as a server and another as a client.
First start the server application

Create a lobby (Steam.createLobby(Steam.LOBBY_TYPE_PUBLIC, 4))
Create a P2P listen socket (Steam.createListenSocketP2P(0, {}))
Set the lobby's game server to the listen socket details Steam.setLobbyGameServer(lobby_id, "", 0, Steam.getSteamID()).

Then start the client application

Join the server's lobby (Steam.joinLobby(server_lobby_id))
Get the lobby game server details (Steam.getLobbyGameServer(server_lobby_id))
Connect to the server's listen socket (Steam.ConnectP2P(server_id, virtual_port, {}))

Server app receives a signal callback (Steam.network_connection_status_changed) and accepts the client's connection (Steam.acceptConnection).
Server sends a PackedByteArray data payload larger than 1 byte (Steam.sendMessages(data, client_handle, 0))
Server application crashes due to segmentation fault

Expected behavior
Expected behavior is for the server's data payload to be successfully received by the client and the server application continues to run.
Screenshots
N/A
Desktop (please complete the following information):

OS: Windows
Version: 10

Version of Godot:
Godot 4.3
Version of GodotSteam:
GodotSteam 4.11 (non-gdextension version) - https://github.com/GodotSteam/GodotSteam/releases/tag/v4.11
Additional context
N/A

Hey there!  Yeah, you are 100% right.  That function is quite broken but shouldn't be too hard to fix.  I may dig back to see what or if there was an older version that was set up right.  I generally try to avoid temporary fixes as they have a tendency to never be updated.
Thanks for bringing this up!  I'll add it to the patch list for 4.12!

I believe m_pfnFreeData has to be used if we allocate our own message buffer according to the documentation.
Note that if you use your own buffer, it MUST remain valid until the callback is executed.
And also note that your callback can be invoked at any time from any thread (perhaps even before SendMessages returns!), so it MUST be fast and threadsafe.

My understanding of the Steamworks documentation is that SendMessages is responsible for calling Release() as it is the only one that knows when it has finished sending the messages and they can be safely released. If we call Release() after SendMessages finish, it is possible that we may be either double freeing the memory, or freeing the memory before SendMessages has run to completion on its separate thread.

As I often say, take the Steamworks docs with a grain of salt as they are never updated after the first time they are written; always refer to the actual SDK itself.
Riley managed to fix this in Steamworks.NET and I'm not sure if Facepunch has or not.  They are creating their own structs though; which isn't a horrible idea.  However, their crashes were related to Release() more than anything, if I understand their issue threads.
Oddly, Valve never uses this function in their examples.

I think it might make sense to try letting AllocateMessage do the heavy lifting for us in the first test and if that fails, try to wrangle the freeing part second.

I think it might make sense to try letting AllocateMessage do the heavy lifting for us in the first test and if that fails, try to wrangle the freeing part second.

Agreed, that is probably the safest approach and smoother experience despite the performance penalty. Performance could be tuned later if someone actually finds it too limiting.

Yeah, there are zero examples of this function the wild so that's fun.  This may take a few tries to solve, sadly.  I will report back when I have some tests ready!


⚠ Bofrost.fr has degraded performance

In 77ab497, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 2643 ms

Resolved: Bofrost.fr performance has improved in 5e962f1 after 9 minutes.


next-nonstarter-with-examples

https://github.com/framesjs/frames.js/tree/main/templates/next-starter-with-examples

node --version → v20.8.0
git clone https://github.com/framesjs/frames.js.git
cd frames.js
yarn
cd templates/next-starter-with-examples/
yarn install
yarn dev

[Error: ENOENT: no such file or directory, open '/data/code/frames.js/packages/debugger/.next/BUILD_ID']

These steps also don't work:
https://framesjs.org/#start-with-framesjs-in-nextjs-in-three-steps
yarn run dev clearly fails as it was never setup
I guess the missing step is: ...in your Next.js app

npm init frames@latest doesn't work:
Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y)
┌  Welcome to frames.js
│
◇  Enter the name of your project
│  my-app
│
◇  Choose a template for the project
│  next-starter-with-examples
│
◆  Project successfully scaffolded in /data/code/my-app!
│
◇  Do you want to install the dependencies using yarn?
│  Yes
│
│  Installing the dependencies...
│
■  Failed to install the dependencies, please install them manually.
npm ERR! code 1
npm ERR! path /data/code/
npm ERR! command failed
npm ERR! command sh -c create-frames

@adraffy in case of cloned repository you have to run yarn dev in the root of repository this will run debugger and examples.
In case of npm init frames@latest, do you have yarn installed on your system? It seems you don't have yarn therefore you have to go to the project dir and run npm install instead.


Backend needs to handle file upload

Should this be done in pass-core? There was an earlier proposal for having an OCFL implementation to handle file data
pass-ui has some upload related logic that sets up requests that will need to be removed and mocked for now until we have a real implementation

Yes. This would be part of pass-core. The service should be able to persist to S3. Note that https://github.com/UW-Madison-Library/ocfl-java has S3 support.

A dump of notes from meeting with @markpatton
Extend integration tests
Write tests do not blow up
Can be run in any order, and can be run with shared storage <- Can be configured with environment variables

Persist to local file system
Persist to S3
Docker image will be configurable with environment variables
Everything needed to run service can be configured through environment variables
Does it take a byte stream and return an opaque URL?
Call URL get the byte stream back?
Does it perform delete?
Does it interact with the file object in the data model? -> Who is responsible for the File Objects?
Take a look at the pass data model
This is a URI on that model to get a byte stream
This file service upload does not have to worry
How does the pass-ui currently works? <- work on this first
Think about replicating this? Or should we do things differently?
Follow-up with a discussion with the UI team
What are the limits of this service

File limits?
Error Handling
Authorization issues
Security concerns?

File types

Setting headers appropriately

Content type
Do not need to perform system testing at the moment

Notes from meeting with @jabrah and @jaredgalanis

Endpoints:

Upload
Download (Maybe? - depending on implementation) <- might be a good idea to have a URL that they can access

pass on ID as URL parameter, and the download service returns byte stream

Delete

Proposed endpoints:
/File/Upload
-returns identifier to the file entity
/File/UploadMeta
- POST contains meta info about the file
/File/UploadAll

Composed of Upload+UploadMeta
/File/Download
/File/Delete

validations in the backend?

File size
File type <- for now, no implementation, but add a TODO
Add a env var blacklist of file types -> e.g. .exe should not be allowed

What is missing now is the authorization layer. User service will be reimplementated.


⚠ Spryr has degraded performance

In a89d492, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 3551 ms

Resolved: Spryr performance has improved in 2a1016c after 13 minutes.


fix issue #69

changing all "lxml" to "html.parser"

You do know you can install lxml with pip install lxml


Ruby 2.7 deprecated URI.unescape

URI.unescape (alias of URI.decode), has been deprecated. We need to update the implementation of that method.

cheers 👍


fetchplan support in sql and js functions

Ive tried everything I can think of and cant get fetchplans to work within user defined sql functions or when used in db.query calls from js functions. Is this a bug?

@lvca

the fetchplan is not to include it in the result set
How would we include a whole document instead of just its RID. In my case I want to return it as a sub-property of objects in a list

Hi @bilalhaider,
From 3.0 will be available nested projection that solve this exact use case https://orientdb.com/docs/3.0.x/sql/SQL-Projections.html#nested-projections


Plant hydraulics model cleanup

We are moving towards running the plant hydraulics model only via the canopy model interface. In this case,

we dont need to supply the domain as part of the plant hydraulics model - we can supply it just in the canopy model.
ideally, the LAI is just supplied once
we really need to clean up transpiration because it is quite confusing right now.

awesome, thank you for following up on my comments

the LAI, RAI, and SAI will eventually be spatially varying and stored in the aux state, not in a parameter struct. so for now, we won't address that with other changes.


DROOLS-733 - Added default sorting for Process Definitions retrieval …

…without filtering

this is already done on this commit:
https://github.com/droolsjbpm/droolsjbpm-integration/commit/605847a14e6fc3634c341020d89ac5b23fbaeb47#diff-af6acf848ee58d3419c6285d305c1602R690
2015-06-04 9:49 GMT+02:00 Karel Suta<EMAIL_ADDRESS>
…without filtering
You can view, comment on, or merge this pull request online at:
https://github.com/droolsjbpm/droolsjbpm-integration/pull/131
Commit Summary

DROOLS-733 - Added default sorting for Process Definitions retrieval
without filtering

File Changes

M
kie-server-parent/kie-server-remote/kie-server-rest/kie-server-rest-jbpm/src/main/java/org/kie/server/remote/rest/jbpm/RuntimeDataResource.java
https://github.com/droolsjbpm/droolsjbpm-integration/pull/131/files#diff-0
(2)
M
kie-server-parent/kie-server-tests/kie-server-integ-tests-jbpm/src/test/java/org/kie/server/integrationtests/jbpm/RuntimeDataServiceIntegrationTest.java
https://github.com/droolsjbpm/droolsjbpm-integration/pull/131/files#diff-1
(4)

Patch Links:

https://github.com/droolsjbpm/droolsjbpm-integration/pull/131.patch
https://github.com/droolsjbpm/droolsjbpm-integration/pull/131.diff

—
Reply to this email directly or view it on GitHub
https://github.com/droolsjbpm/droolsjbpm-integration/pull/131.

Hi Maciej,
in that commit there was added sorting just for Process Definitions retrieval with filter parameter. See:
https://github.com/droolsjbpm/droolsjbpm-integration/blob/605847a14e6fc3634c341020d89ac5b23fbaeb47/kie-server-parent/kie-server-remote/kie-server-rest/kie-server-rest-jbpm/src/main/java/org/kie/server/remote/rest/jbpm/RuntimeDataResource.java#L293
https://github.com/droolsjbpm/droolsjbpm-integration/blob/605847a14e6fc3634c341020d89ac5b23fbaeb47/kie-server-parent/kie-server-remote/kie-server-rest/kie-server-rest-jbpm/src/main/java/org/kie/server/remote/rest/jbpm/RuntimeDataResource.java#L298

ah, now I got it, I missed one method in the change set. thanks will merge
it today
2015-06-04 12:08 GMT+02:00 Karel Suta<EMAIL_ADDRESS>
Hi Maciej,
in that commit there was added sorting just for Process Definitions
retrieval with filter parameter. See:
https://github.com/droolsjbpm/droolsjbpm-integration/blob/605847a14e6fc3634c341020d89ac5b23fbaeb47/kie-server-parent/kie-server-remote/kie-server-rest/kie-server-rest-jbpm/src/main/java/org/kie/server/remote/rest/jbpm/RuntimeDataResource.java#L293
https://github.com/droolsjbpm/droolsjbpm-integration/blob/605847a14e6fc3634c341020d89ac5b23fbaeb47/kie-server-parent/kie-server-remote/kie-server-rest/kie-server-rest-jbpm/src/main/java/org/kie/server/remote/rest/jbpm/RuntimeDataResource.java#L298
—
Reply to this email directly or view it on GitHub
https://github.com/droolsjbpm/droolsjbpm-integration/pull/131#issuecomment-108822126
.

merged into execution server unification2 branch, thanks @sutaakar


About global_scale parameter values

Hi Phuong,
Could you please explain what is global_scale parameter is and how did you choose the particular values i.e, 2.0 and 5 ? in global_scale = 2.0 * 5/cf_net['n_exits']

Hi,
it's a scalar multiplier for the entire loss. If you set it to 3.0, the loss will be three times as large as when you set it to 1.0.
The value 2.0 was chosen because it worked well for my ImageNet network with five exits.
For a different number of exits, I modify the multiplier proportionally to 1/n_exits. This is simply because the loss is a sum of n_exits terms, and it seems natural to take the average per-exit loss rather than the sum.

Thank you! Makes sense.

Another question just to clarify the total loss (dist_loss and cross_ent). As you mentioned taking average of per-exit loss. Does it mean that if there are 5 exits then we will have 5 (dist_loss and cross_ent) losses per exit averaged over a batch let's say dist_loss = [1,2,3,4,5] and cross_ent = [6,7,8,9,10]. So the average in this case dist_loss = 3 and cross_ent = 8, right? and total_loss will be global_scale * (3+8)?

There is also the constant C (between 0 and 1) that is the relative importance of cross_ent vs. dist_loss, and the distillation temperature T. In the example you give:
total_loss = global_scale * ( (C*T^2) * 3 + (1-C) * 8).
(Here's the code.)

Thank you Mary.


So You Wanna Be a Pandas Expert? (Tutorial) - James Powell | PyData Global 2021

So You Wanna Be a Pandas Expert? (Tutorial) - James Powell | PyData Global 2021
Claim on Pandas Expertise video, by James Powell
Will be uploading Timestamps on few moments

Timestamps (Not finished):
1:50  - Why Pandas? Introduction - Pandas vs. Python Builtins
17:14  - Pandas - Why so much complexity?
18:41  - Numpy - Introduction
26:44  - Numpy Conclusions
27:15  - Xarray - Introduction
31:44  - Xarray - Conclusions
31:59  - Pandas - Series & Arrays
32:50  - Pandas - Index introduction
33:56  - Pandas - Conclusions and Problems
34:47 - Series - Accessor ambiguity & Methods
36:19 Why this Pandas ambiguity?
39:10 Possible Errors & Indexes Types
40:25 Index is a Mechanism - Demo
41:46 Index translates: Human to Physical Memory
45:11 Demo: Index Mechanics on Arithmetics
47:07 Indexes are Operative metadata
50:04 Back to Numpy - Rule comparisson

Hey @torres-federico-e
Can you help me out to understand this repo
I mean there are lots of issues open
but still, no PRs to resolve them
I am just kind of confused.
Thanks.

Of course. The PyData YouTube Channel has opened for collaboration all the Time Stamping efforts for indexing videos of their conferences
You basically claim video from YouTube that you want to watch,  watch it, make the timestamps, and create an issue on this repo and collaborate.
They will add them to YouTube, make a PR, and credit for your help to the community afterwards.

Got it.
Thanks a lot for help @torres-federico-e

1:50 - Why Pandas? Introduction - Pandas vs. Python Builtins
2:26 - Loop+Sets Vs Pandas
3:45 - List and Dictionary Introduction
4:26 - Pandas Intersection of two Collections
6:13 - List Intersection of two Collections
6:48 - Dict Intersection of two Collections
7:41 - Counter Intersection of two Collections
17:14 - Pandas - Why so much complexity?
18:41 - Numpy - Introduction
26:44 - Numpy Conclusions
27:15 - Xarray - Introduction
31:44 - Xarray - Conclusions
31:59 - Pandas - Series & Arrays
32:50 - Pandas - Index introduction
33:56 - Pandas - Conclusions and Problems
34:47 - Series - Accessor ambiguity & Methods
36:19 Why this Pandas ambiguity?
39:10 Possible Errors & Indexes Types
40:25 Index is a Mechanism - Demo
41:46 Index translates: Human to Physical Memory
45:11 Demo: Index Mechanics on Arithmetics
47:07 Indexes are Operative metadata
50:04 Back to Numpy - Rule comparisson


MediaQuery.viewInsets does not update when keyboard is opened

Steps to reproduce

Create a new Flutter application with a single screen that includes a TextField.
Implement MediaQuery to obtain size, padding, viewPadding, and viewInsets values.
Run the application.
Tap on the TextField to bring up the keyboard.
Observe the MediaQuery.viewInsets value.

Expected results

When the keyboard is opened, the MediaQuery.viewInsets value should update to reflect the size of the keyboard.
This update should accurately represent the area of the screen not covered by the keyboard.

Actual results

Even after the keyboard is opened, the MediaQuery.viewInsets value remains zero.
This indicates that MediaQuery is not updating its values in response to the keyboard opening, leading to inaccurate measurements of usable screen space.

Code sample
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        appBar: AppBar(title: const Text('Test')),
        body: const Body(),
      ),
    );
  }
}

class Body extends StatelessWidget {
  const Body({super.key});

  @override
  Widget build(BuildContext context) {
    final size = MediaQuery.sizeOf(context);
    final padding = MediaQuery.paddingOf(context);
    final viewPadding = MediaQuery.viewPaddingOf(context);
    final viewInsets = MediaQuery.viewInsetsOf(context);

    final view = View.of(context);
    final vSize = view.physicalSize;
    final vPadding = view.padding;
    final vViewPadding = view.viewPadding;
    final vViewInsets = view.viewInsets;

    return SingleChildScrollView(
      child: Padding(
        padding: const EdgeInsets.all(20),
        child: Column(
          crossAxisAlignment: CrossAxisAlignment.start,
          children: <Widget>[
            Text('MediaQuery size: $size'),
            Text('MediaQuery padding: $padding'),
            Text('MediaQuery viewPadding: $viewPadding'),
            Text('MediaQuery viewInsets: $viewInsets'),
            const SizedBox(height: 8),
            Text('View physicalSize: $vSize'),
            Text('View padding: $vPadding'),
            Text('View viewPadding: $vViewPadding'),
            Text('View viewInsets: $vViewInsets'),
            const SizedBox(height: 20),
            const TextField(
              decoration: InputDecoration(
                border: OutlineInputBorder(),
                labelText: 'Tap me!',
              ),
            ),
          ],
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
https://github.com/flutter/flutter/assets/8436039/0986f3c0-3d83-4aba-9b2d-cfd84f48a310

Logs
No response
Flutter Doctor output
Doctor output
[!] Flutter (Channel master, 3.22.0-10.0.pre.13, on Microsoft Windows [Version 10.0.19045.4291], locale hu-HU)
    • Flutter version 3.22.0-10.0.pre.13 on channel master at C:\flutter_fork
    ! Upstream repository https://github.com/maRci002/flutter.git is not a standard remote.
      Set environment variable "FLUTTER_GIT_URL" to https://github.com/maRci002/flutter.git to dismiss this error.
    • Framework revision 46fbb73440 (19 hours ago), 2024-04-12 18:41:57 -0400
    • Engine revision 1a13c7d1f4
    • Dart version 3.5.0 (build 3.5.0-47.0.dev)
    • DevTools version 2.34.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\Users\maRci002\AppData\Local\Android\Sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = C:\Users\maRci002\AppData\Local\Android\Sdk
    • ANDROID_SDK_ROOT = C:\Users\maRci002\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.9+0--11185874)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.7.0)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.7.34003.232
    • Windows 10 SDK version 10.0.22621.0

[√] Android Studio (version 2023.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.9+0--11185874)

[√] VS Code (version 1.88.1)
    • VS Code at C:\Users\maRci002\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension can be installed from:
       https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[√] Connected device (4 available)
    • sdk gphone x86 64 (mobile) • emulator-5554 • android-x64    • Android 13 (API 33) (emulator)
    • Windows (desktop)          • windows       • windows-x64    • Microsoft Windows [Version 10.0.19045.4291]
    • Chrome (web)               • chrome        • web-javascript • Google Chrome 123.0.6312.122
    • Edge (web)                 • edge          • web-javascript • Microsoft Edge 123.0.2420.81

[√] Network resources
    • All expected network resources are available.

Your scaffold is consuming the viewInsets, the default value for Scaffold.resizeToAvoidBottomInset is true. Set that to false if you want to handle the viewInsets yourself.

Your scaffold is consuming the viewInsets, the default value for Scaffold.resizeToAvoidBottomInset is true. Set that to false if you want to handle the viewInsets yourself.

You are right, thank you very much.


CSP 'script-src' directive is violated

Hello,
There are some usages of Function in the lodash package, which requires unsafe-eval to be used. In the built version https://github.com/jshint/jshint/blob/master/dist/jshint.js you can find Function('return this')(); and Function(importsKeys, sourceURL + 'return ' + source). The second one is not used at all.
This makes usage of JSHint impossible in restricted environments, where only script-src: self is set. Any chance to remove them?
More information about Unsafe eval expessions.

Thanks for taking the time to file an issue! Lodash is expected to load "just fine" in a CSP-restricted environment. Judging from that library's source code, it seems as though an error will only be encountered if the binding named self is not defined globally. Is that the case for you?

Hello! The problem is that scan tools report such usages, even if not used. The issue you have referenced is from 2014, and back then this code wasn't part of root.js. I've found that this is already addressed with https://github.com/lodash/lodash/issues/4985 and globalThis is suggested.
For the other occurence of eval, which is in lodash.template - since you are not using this function, is it possible to prune it from your bundle? It will also decrease the total size.

Got it. I've updated the title of this issue since there aren't any known Content-Security Policy violations.
A sufficiently advanced build process would likely infer that JSHint doesn't use the offending code and omit it from the final product. Unfortunately, this project's build system is not up to the task right now. I'll gladly review patches to modernize it!


[Tracker] CI, Observability and Meta

[x] #54
This is required for easy deployments
[x] #15
[x] #40
[x] #42
[x] #53
[x] #56

All child issues are complete, hence closing parent issue


Add write support for PDS3Image

There may be parts we will choose to extract up into the PlanetaryImage class, but you can start with PDS3Image.

The syntax for this should be
filename = ...
image = PDS3Image.open(filename)
image.data = image.data + 1
image.write(overwrite=True)

or
...
image.write('newfilename.IMG')

Thoughts on an api more similar to PIL/Pillow or are you basing it on something else?

Good suggestion @wtolson, basing it on something rather than making it up.

We will also want to consider how to take a simple numpy array and write it out as a PDS3Image:
i = PDS3Image(data=np.create_array())
i.write()

or
PDS3Image.write(np.create_array())

@wtolson's suggestion of using pillow as the model is a good one.  I keep forgetting that he started out with that as his model.

I like what your thinking @godber. Maybe something like pillow:
image = PDS3Image.fromarray(data, extra_label_data={'foo': 'bar'})
image.save('foo.IMG')

This has me thinking, we might want to separate out image encoding/decoding from the Image object which would allow us to do something like:
# Detect and use proper encoding to read in the data
image = Image.open('foo.cub')

# Write out the data in a different format
image.save('foo.IMG')

# Create my own image
image = Image.fromarray(data)

# Save it in multiple formats
image.save('test.IMG')
image.save('test.cub')

Not sure how easy this would be for us to separate out image representation from the encoding/decoding formats but it seems to be the architecture used by PIL and scikit image.

Definitely its own issue and a future project though.

Separating the encoding/decoding is really great idea @wtolson.


Removing executable bit from non-executable files.

@tseaver I promise I'm not crazy, I originally noticed that the .proto files were executable when moving things around for v1beta3 datastore changes.

@stephenplusplus @callmehiphop Any idea why the docs resources (which were likely just copied from gcloud-node) have the executable bits set? Should I leave them in place?

Can verify which files had the bit set with:
ls -alFG $(git ls-files) | grep -e '*$'

After this change it leaves
scripts/custom_pip_install.sh*
scripts/merge.sh*
scripts/pep8_on_repo.sh*
scripts/run_system_tests.sh*
scripts/update_docs.sh*
scripts/update_wheels_project.sh*

which all should be executable. We could probably also make our system test runner executable if we felt we needed it.

I'm not sure how they ended up being 755, but 644 is the right way to go.

:+1: Thanks

LGTM


Can we run jitsi web pod on port  TCP 8080 by updating its service value ?

Can we run jitsi web pod on port  TCP 8080 by updating its service value in values.yaml file ?
Link - https://github.com/jitsi-contrib/jitsi-helm/blob/main/values.yaml#L79
Will this port 8080 support to access the jitsi web page ? Is this the right way ?
I don't want port 80 as I need to do a privileged level escalation on my server to access privileged port 80.

Hi @spijet - Do you have idea on this ?

Hello @Govind10g!
You can change the Service port freely, but there should be little difference when doing so. You don't usually expose this Service to the outside world directly. Instead you either use a NodePort Service (which usually gives you a random port anyway) or you expose the Service using an Ingress Controller (that's what I do and recommend personally).


Added comments to a couple things

I think having JS Comments in all of the stores etc will make it much more usable for new people starting out. It will help them understand what each store does as well as how it is formatted

Added

A favicon in the style of the Lego logo
A few JS Comments
A search page
An error page