New feature

Do you ever think of considering to add more features to the webpanel like rats have? run file from url for example or something like that?

No I don't intend for the web panel to be like a RAT, it would be much better to just use a normal RAT for that purpose.

No I don't intend for the web panel to be like a RAT, it would be much better to just use a normal RAT for that purpose.

what about a stealer?

If I make a stealer it would be a separate program (I have one that's almost done but haven't finished it), but if I finish it I would most likely integrate it into the web panel as well (a new tab).

If I make a stealer it would be a separate program (I have one that's almost done but haven't finished it), but if I finish it I would most likely integrate it into the web panel as well (a new tab), haven't finished it since making http(s) calls in C isn't very fun.

oh, good job. yeah i believe that, from my point of view coding anything makes my brain really hurt and i can never make anything perfect so i just look for other people to do it

yo estimatetly how long will it take

It's not my priority at the moment.

ah are you making a stealer as a separate program?

Yes a stealer isn't a miner and shouldn't really be connected to a miner so it would be a separate program.


README Formatting

  Please Check The Preview

Add Tags

emulator

diagrams

processor

cpu

16-bit

alu

logism

chip

Add Description
Small 16 Bit Processor

Disable unused Widgets:

Packages

Releases

Disable unused Tabs:

Projects

Actions

General formatting / spacing

Sorted files

Added quicklinks

If you have any questions.

In case you do not agree with some of the changes made in this PR,
please first let me know about them, before disregarding it entirely so
we can possibly find a middle ground and have it not go to waste.

             No, I'm not a robot.
     Yes, you are not the first.
      I just made a PR template.

If you are new to GitHub and aren't sure how to accept commits,
please familiarize yourself with the basics of Git / GitHub first.
Pull Requests
To accept pull requests you simply have to
press the      Merge PullRequest      button.
Never Copy-Paste Someones Contribution,
this robs them off their attribution and is thus
just plain copyright theft.

If you want to know more about Markdown check out:

Looks great to me, I'm a tad new to the customization so I appreciate the contrib :>


style fixes

Codecov Report

Merging #516 into master will decrease coverage by 0.02%.
The diff coverage is 80.48%.

@@            Coverage Diff             @@
##           master     #516      +/-   ##
==========================================
- Coverage   92.35%   92.32%   -0.03%
==========================================
  Files          37       37
  Lines        2877     2879       +2
==========================================
+ Hits         2657     2658       +1
- Misses        220      221       +1

Impacted Files
Coverage Δ

vent/api/plugins.py
92.74% <80.48%> (-0.17%)
:arrow_down:

Continue to review full report at Codecov.

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


Add documentation for Hands MK5 sensor scheme

The frame to document the Hands MK5 connection scheme and other infos useful for the correct configuration of the robot is prepared.
Some additional info is to be added when the FW work is ready.

It's not necessary to close the PR as it automatically updates.
I've seen that you force-pushed to your master, which is correct 👍🏻


Update commons-text to 1.11.0

About this PR
📦 Updates org.apache.commons:commons-text from 1.10.0 to 1.11.0
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (1.10.0).
You might want to review and update them manually.
pom.xml

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.apache.commons", artifactId = "commons-text" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.apache.commons", artifactId = "commons-text" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, old-version-remains, commit-count:1

Superseded by #182.


Missing requirements.txt

Instructions reference requirements.txt but none exists in the repo. Oversight?

Ah yup, thank you. I just added it 🙏
the autoStattic script was originally in the Stattic repo which has one, so when I separated it I didn't add it's own requirements file.


refresh on job detail

When I access job detail and refresh the page it's returning an internal server error.

That's working!


asynchronous storage support, for database integration

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

Project coverage is 95.74%. Comparing base (0f83261) to head (de5f845).

Files
Patch %
Lines

src/fritter/persistent/database.py
0.00%
69 Missing :warning:

src/fritter/drivers/scheduling.py
0.00%
19 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             trunk       #3      +/-   ##
===========================================
- Coverage   100.00%   95.74%   -4.26%
===========================================
  Files           25       27       +2
  Lines         2052     2140      +88
  Branches       168      181      +13
===========================================
- Hits          2052     2049       -3
- Misses           0       90      +90
- Partials         0        1       +1

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


Fix build for Android armv7

Motivation:
Fix compilation error because of a type mismatch between st_mode and mode_t on 32-bit Android.
Modifications:

Zero-extend mode_t constants on Android armv7 to match the types.
Scope an earlier modification in this vein to armv7 only.

Result:
The code builds on Android armv7 again.
This line just added in #327 yesterday broke the armv7 build on my daily Android CI. Adding these conversions fixes it and I went ahead and scoped the earlier change from #45 more narrowly too.

Thanks! :sparkles:


keithley2700: dont crash on unknown card

Fixes
Traceback (most recent call last):
  File "test3.py", line 20, in <module>
    kmeter = Keithley2700(adapter.gpib(5))
  File "/home/mcmaster/doc/ext/pymeasure/pymeasure/instruments/keithley/keithley2700.py", line 150, in __init__
    self.determine_valid_channels()
  File "/home/mcmaster/doc/ext/pymeasure/pymeasure/instruments/keithley/keithley2700.py", line 175, in determine_valid_channels
    channels = [100 * slot + ch for ch in channels]
UnboundLocalError: local variable 'channels' referenced before assignment

LGTM, thanks for the fix! We can merge after the author list update.

Done

Thank you for your contribution!


fabric 1.17.1

May I ask a few things? Why loom 0.9.9 instead of the current 0.9.26? Just put 0.9-SNAPSHOT to be always up-to-date with v0.9.
About Gradle - The latest one is 7.1.1.

sure! the loom version was an oversight (alphabetical ordering on the fabric maven). Since I now have dependabot enabled for this repo it will notify me when a new version comes out (once this is merged). And the gradle version I checked a while ago.


add embedding model by default to distribution templates

What does this PR do?
Adds the sentence transformer provider and the all-MiniLM-L6-v2 embedding model to the default models to register in the run.yaml for all providers.
Test Plan
llama stack build --template together --image-type conda
llama stack run ~/.llama/distributions/llamastack-together/together-run.yaml

Since existing run.yaml does not have an embedding model registered explicitly for all-MiniLM-L6-v2, the distribution does not know which provider to route to for this model. The stack trace is this:
Traceback (most recent call last):
  File "/Users/dineshyv/code/llama-stack/llama_stack/distribution/server/server.py", line 187, in endpoint
    return await maybe_await(value)
  File "/Users/dineshyv/code/llama-stack/llama_stack/distribution/server/server.py", line 151, in maybe_await
    return await value
  File "/Users/dineshyv/code/llama-stack/llama_stack/providers/utils/telemetry/trace_protocol.py", line 101, in async_wrapper
    result = await method(self, *args, **kwargs)
  File "/Users/dineshyv/code/llama-stack/llama_stack/distribution/routers/routing_tables.py", line 314, in register_memory_bank
    raise ValueError(f"Model {params.embedding_model} not found")
ValueError: Model all-MiniLM-L6-v2 not found


Implement Auto trait

Per https://github.com/PyO3/pyo3/issues/2140#issuecomment-1029440783

[x] Make a job for nightly feature in the CI - I'm not sure where this fits best.

(I could just squash on merge however sometimes it's nice to not just have one mega commit either 🤷)

Yeah the commits are pretty messy 😭
Let me see if I can git-fu this...

Thanks, nice work! Because this PR has updated the job names I had to tweak the branch protection rules. Anything else will have to rebase on this on in order to me mergeable.


Implement comprehensive query wildcard re-write

This patch implements comprehensive rewrite of wildcards, for both the "source" of the SELECT statement, and the GROUP BY. The parser prevents statements such as SELECT *,value, so those unit tests are commented out. However, the query parser should be fixed up eventually to allow this, as other SQL-based query engines support this.

This changes uses dedicated code for SELECT query rewrite, and is very close in form to what existed already for SELECT *. It's not clear to me if we should be using some node-walking re-write code.
@pauldix @benbjohnson @corylanou

Addresses issue #1624.

@dgnorton -- could do with your review too.

+1

Are we going to handle wildcard expansion for multiple source measurements? e.g.

Wildcard method added @pauldix @benbjohnson -- there are a couple of different places the check could be placed, of course.

@pauldix is refactoring a chunk of the query engine, so this PR is on hold. He and @benbjohnson have promised to merge (or possibly rework) this when ready.

Rebased on master, and green build now.


Various optimizations

See commit list. Removed a bunch of features that aren't needed, and most importantly make the sanitization lazy so that we only pay for it if the body is actually read, and that it can be delayed in case the request wasn't fully buffered yet.

Closing in favor of https://github.com/Shopify/rack-sanitizer


Images from S3

Referencing this issue: https://github.com/aframevr/aframe/issues/2156#issuecomment-411880032
Using <Entity primitive="a-image" src="whaetever" /> does not work when trying to load image from an S3 bucket. Bucket is set to public.

I'm facing the same issue, ever got it working around?


DatePicker

Hi,
We are using botfront (v0.5.9) as our rasa chat widget. We need to have a datepicker for one of our use case. Can this be achieved with botfront or please let us know if there is any other way around.

We were able to do this by creating a datepicker component similar to carousel and all. We need to send back the selected date to Rasa. So far could not find a way to do that. We are checking on it, meanwhile if you have any pointers do let us know too.

Nevermind we figured it out. For those looking the same, we can pass intent and slots as below:
dispatch(emitUserMessage("/intent{slot1: slot1Val}"));


Keys show type

I wish You add a description of every key, like this:

The problem is:

Solution:
Remove hidden titles ;)

Yes, it's a good feature. For now the visual part is a bit... too minimal.

Doesn't https://github.com/zmkfirmware/zmk-studio/issues/17 already encompass this issue?  Or is this requesting something different?


Enterprises who received more

Creating a list of the enterprises who receive more to help the field investigators.

In a way @pmargreff has already done it. @cabral do you need it in any specific format?

Oops… sorry. I comment as it was an Issue, not a PR. My bad.

WIP
Now have a very basic function to call every subquota and receive basic info

Closed by inactivity


Stage and Scene classes were imported

Stage and Scene classes were not imported.

@patriarh12  thank you very much! My mistake.

@fletcherist Thank you for SDK ;)


This app currently uses flask, we need to initialize and set up a Django server that is connected to the database.

Database lives on AWS RDS. Engine: MySQL

I successfully fixed this issue, closing it now


History Tab and Latest history left panel doesn't show info

I just deployed version 23.0, I'm able to submit queries succesfully, but when I try to see queries history I get nothing on history tab and latest history left panel.
Here is the evidence

I just have one datasource which is trino/presto version 349, but this behavior also is present with trino 351, therefore I suppose this is not about trino.
I tried switching my yanagishima version to 22.0 and 21.0 but the result is the same.
I'm using jdk 11.0.23 and Node.js v14.21.3
Is there some workaround ? I'm a novice using Vue

I'm gonna close the issue since I found the reason why history tab doesn't show queries info, It's necessary to set "Server" option in Storage config in order to get queries history from beckend mysql db.
After doing this I was able to see my queries


Document various concepts in the code-base

There are various concepts from the "terminal world" in the codebase that are not very trivial for developers to immediately understand. It would be good to have a sort of "cheat sheet" document where one could look them up and understand what they mean, as well as where they fit in with the Mosaic project.
@alekspickle - since you rightfully pointed out this issue, I hope you'd be willing to briefly review this document once I write it (hopefully today or tomorrow) and let me know if it makes sense to you.

@imsnif Please ping me for a review of that PR, I want to make sure I understand as well :)


Is this feature Generally Available on all tenants now?

Please include the status of this feature to better understand the availability. I am not able to see the "Edit conditional formula" section for columns on my lists and libraries. Seems that this feature is not yet GA.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 536f88e0-51aa-9a3a-20da-64b4a8b155a1
Version Independent ID: efa34015-6402-a024-ae62-3b0c6c0865ea
Content: Show or hide columns in a list form
Content Source: docs/declarative-customization/list-form-conditional-show-hide.md
Product: sharepoint
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

We also do not have conditional access available for "Show/hide" columns in our lSharePoint Online lists and libraries


Adaptative domain between a relational and non-relational domain

Create a combined domain between e.g., (zones x terms) and intervals such that  the domain is zones x terms if the number of variables does not exceed a threshold, otherwise it degrades to intervals.

This issue was moved to seahorn/crab#7


KTOR-7748 Generate a server project from OpenAPI specification

https://youtrack.jetbrains.com/issue/KTOR-7748

@antonarhipov, could you also check?

@vnikolova, please check the docs

@Stexxe, thanks for the PR.
Could you tell me if you thought how we would generate openAPI clients for ktor clients?

I tried this:
anton@macbook-pro tmp % ./ktor-cli/ktor openapi -o ktor-project openapi.yml
Error downloading OpenAPI utility. Please try again later.
For more details, see the log: /Users/anton/.ktor/run.log.
anton@macbook-pro tmp % less /Users/anton/.ktor/run.log:
2024/11/22 18:25:34 16: fetch OpenAPI JAR: unexpected response status 404 from the server:

404 - Path /org/openapitools/openapi-generator-cli/7.10.0-SNAPSHOT/openapi-generator-cli-7.10.0-20241106.082455-66.jar not found in local storage of repository "Snapshots" [id=snapshots]
...

I tried this:
anton@macbook-pro tmp % ./ktor-cli/ktor openapi -o ktor-project openapi.yml Error downloading OpenAPI utility. Please try again later. For more details, see the log: /Users/anton/.ktor/run.log.
anton@macbook-pro tmp % less /Users/anton/.ktor/run.log:
2024/11/22 18:25:34 16: fetch OpenAPI JAR: unexpected response status 404 from the server:
404 - Path /org/openapitools/openapi-generator-cli/7.10.0-SNAPSHOT/openapi-generator-cli-7.10.0-20241106.082455-66.jar not found in local storage of repository "Snapshots" [id=snapshots] ...

That's strange. I cannot reproduce it at the moment. Can you please try again?


All Notifications of the App getting cleared when I click one notification (Android)

This issue happens only on android, when I have more than one notification for the App and if I click any one, then all other notifications of the App is getting cleared automatically. What could be the problem or should I add any flags etc ?
Checked on
Android 4.4 and 5.0
React-Native 0.43.0-rc.4
React-Navigation  1.0.0-beta.11

In your custom notification class extending PushNotification, returned by getPushNotification(...), override function onOpened, so it looks like this:
@Override
public void onOpened() {
    digestNotification();
}

@Lastin thanks for the reply I will try your solution and respond back.

@ravirajn22 You will have to make your notifications work however.
In com.wix.reactnativenotifications.core.NotificationIntentAdapter function createPendingNotificationIntent uses flag PendingIntent.FLAG_ONE_SHOT when calling PendingIntent.getService which causes the pending intent with same hash (see) to not be able to run activity more than once.
Solution:
Use PendingIntent.FLAG_UPDATE_CURRENT flag instead

This caused an issue for me. When I received multiple notifications, tapping on any notification would always bring up most recently received notification. Instead, I followed the advice here: https://stackoverflow.com/questions/28091784/multiple-pending-intents
So now my createPendingNotifications method in core/NotificationIntentAdapter looks like this:
public static PendingIntent createPendingNotificationIntent(Context appContext, Intent intent, PushNotificationProps notification) {
    intent.putExtra(PUSH_NOTIFICATION_EXTRA_NAME, notification.asBundle());
    intent.setAction(Long.toString(System.currentTimeMillis()));
    return PendingIntent.getService(appContext, PENDING_INTENT_CODE, intent, PendingIntent.FLAG_ONE_SHOT);
}

And it seems to work as intended.

@kaloncheung124 and @mfekim I ended up overriding the getCTAPendingIntent() and onOpened(). Thanks for your help.


chore(docs): set stability badge to "stable"

Summary
This PR removes the use of any beta versions and updates the stability badge on the repo's README to "Stable"

@ryparker add the back port labels?

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1330

Questions ?
Please refer to the Backport tool documentation


24/7 hosting! help

i wanna runn the server 24./7 any help? im using a digitalocean vps btw

instructions is provided in readme and on main project page.
Also you can use tmux to run server in background console with ability to attach at any time you wish


Converted to using omnisharp-node-client to control the server connectio...

...n.  Refactored to not wrap the client up too much and reduce events that are getting emitted.
Note: I fully expect this PR to around for a while :dash:

Will require this to be completed at somepoint. (or move it to the npm binaries repo and complete it there).
https://github.com/OmniSharp/omnisharp-roslyn/pull/186

Alright, this should be good finally... rebased with the latest master.

@nosami @ctolkien @willl @sgwill anyone have a chance to try this one out yet?

Sorry, I'm at a wedding. I'll try tomorrow or Monday.
On May 2, 2015 17:05, "David Driscoll"<EMAIL_ADDRESS>wrote:

@nosami https://github.com/nosami @ctolkien
https://github.com/ctolkien @willl https://github.com/willl @sgwill
https://github.com/sgwill anyone have a chance to try this one out yet?
—
Reply to this email directly or view it on GitHub
https://github.com/OmniSharp/omnisharp-atom/pull/157#issuecomment-98401435
.

Pfft Weddings :-1:  :smiling_imp:

OK.... Just tested this. Seems to be working well.

cc @Mpdreamz

@ctolkien when you get back up tomorrow, try it again, I've made some changes that should move the server automatically inline with the client, and hopefully avoid that long path issue.

Progress!


Some inssues about ompv4.py(openvas_lib file)

Well, at first, I installed this project, but I get an error .
This is because I set tab as four space, and I got an error at :

 **File "/usr/local/lib/python2.7/dist-packages/openvas_lib/ompv4.py", line 154
    request = """<create_target>**

Then I used, here is four space :
**sed 's/\t/    /g' openvas_lib/ompv4.py  >/usr/local/lib/python2.7/dist-packages/openvas_lib/ompv4.py**
But I also got an error:
class OMPv4(OMP): NameError: name 'OMP' is not defined
So I  was very confused about it.
Here is my code:
    **try:
        scanner = VulnscanManager('<IP_ADDRESS>', 'admin', '123')
        scan_id, target_id = scanner.launch_scan(target = myurl, profile = "Full and fast")
        openvas_results = scanner.get_results(scan_id)
        scanner.delete_scan(scan_id)
        scanner.delete_target(target_id)

        print openvas_results
    except VulnscanException as e:
        print "[x]Openvas Error:"+str(e)
    pass**

Maybe no error at here, right?
I hope author or someone can help me with this issue , thanks !!!

Thanks for the report!
I don't know why spaces/tabs was mixed. It should be fixed in the commit 01434ade20d83ed52702e26ea429a9a560166045
Also was fixed the second error. I missed to import a dependency


Remove storage type from filter bar

SCRD-2193  Updated template names since the input model no longer has references to SES. Therefore having the storage type filter box with one available value 'Swift' does not make sense. Commented out one line of code for now, in case they want to resurrect SES in the future.
Depends on this backend change: https://gerrit.suse.provo.cloud/#/c/3335 which is going through the gate at the moment

The backend change was merged into master. Please feel free to test it with this UI change.


Boot mode from hardware on 12-pin microcontroler

Hello,
does anyone know/tried how to get to boot mode from hardware on Patriot Supersonic Express, but on 2307 microcontroler? The difference is, that this one has only 12 pins
Thanks

I have the same problem :/


Japanese script not supported

https://twitter.com/BuildMeAVideo/status/1557878504347160576?t=0Dty2OpZYmiIP63ih5DU5w&s=19

Supported by https://github.com/shotstack-samples/build-me-a-video-twitter-bot/pull/17


[PY] fix: Do not throw error if there is no text content (but attachments)

Linked issues
closes: #1777
Details
If there is a message that has no text content, but it does have some non-textual attachments, we currently throw. However, the message could have attachments that could be used for further processing by the pipeline. As such, it doesn't make sense to throw an error in this case.
Change details
Before the error:

After the error:
https://github.com/user-attachments/assets/01a1f260-8bde-4d01-bd55-a660c1a905fa
Note:
I didn't see any existing tests that tested this particular path (esp using options.ai). Any pointers on building that out? I started adding to test_app.py but wanted to know if there were easy ways of mocking out the AI layer out. Hence no tests yet, but happy to add it if it's not that hard.
Attestation Checklist

[x] My code follows the style guidelines of this project

I have checked for/fixed spelling, linting, and other errors

I have commented my code for clarity

I have made corresponding changes to the documentation (updating the doc strings in the code is sufficient)

My changes generate no new warnings

I have added tests that validates my changes, and provides sufficient test coverage. I have tested with:

Local testing
E2E testing in Teams

New and existing unit tests pass locally with my changes

Additional information

Feel free to add other relevant information below

This would need to be added to JS and C# as well


Plugin: Bookmarks for maps and portals (bookmarks-by-zaso)

Any possibility of adding the ability to import KML files? Similar to the overlay KML plug-in however they are stored as bookmarks instead of an overlay

You should probably contact Zaso directly (he's a friendly type) at https://github.com/ZasoGD

Thanks
Get Outlook for Androidhttps://aka.ms/ghei36
On Fri, Jan 6, 2017 at 6:34 PM +1100, "nhamer" <notifications<EMAIL_ADDRESS>wrote:
You should probably contact Zaso directly (he's a friendly type) at https://github.com/ZasoGD
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/iitc-project/ingress-intel-total-conversion/issues/1170#issuecomment-270848446, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AXylv-oKGrOcttkWWcuL-QKMhsnfCgHwks5rPe57gaJpZM4LcMZv.

@zasogd may correct me, but thinking about it - imported bookmarks need portal GUIDs, which you aren't going to find in KML.

I can import KML now and have it put location markers on portals therefore all the required information must already exist. Attached is an example KML file
Get Outlook for Androidhttps://aka.ms/ghei36
On Sat, Jan 21, 2017 at 3:47 PM +1100, "nhamer" <notifications<EMAIL_ADDRESS>wrote:
@zasogdhttps://github.com/zasogd may correct me, but thinking about it - imported bookmarks need portal GUIDs, which you aren't going to find in KML.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/iitc-project/ingress-intel-total-conversion/issues/1170#issuecomment-274235417, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AXylvyA7JqmBsokzvACjJgl4L8vyx3FKks5rUY3XgaJpZM4LcMZv.

Can you upload the kml either directly here to github or to pastebin or
something? It didn't come through in mail.
On Fri, Jan 20, 2017 at 10:02 PM, whartyENL<EMAIL_ADDRESS>wrote:

I can import KML now and have it put location markers on portals therefore
all the required information must already exist. Attached is an example KML
file
Get Outlook for Androidhttps://aka.ms/ghei36
On Sat, Jan 21, 2017 at 3:47 PM +1100, "nhamer"<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
@zasogdhttps://github.com/zasogd may correct me, but thinking about it

imported bookmarks need portal GUIDs, which you aren't going to find in
KML.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<https://github.com/
iitc-project/ingress-intel-total-conversion/issues/1170#
issuecomment-274235417>, or mute the thread<https://github.com/
notifications/unsubscribe-auth/AXylvyA7JqmBsokzvACjJgl4L8vyx3
FKks5rUY3XgaJpZM4LcMZv>.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/iitc-project/ingress-intel-total-conversion/issues/1170#issuecomment-274239652,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXzED324ANTdxA0vCL8f-i3TKGuDlfLEks5rUZ9ZgaJpZM4LcMZv
.

Silly question, how do I add attachments :-)

Oh. I suppose the answer is actually "upload to pastebin, add a link". Sorry, just getting used to the toolchain.

In IITC A portal can be found without using its GUID window.selectPortalByLatLng(lat, lng).
If you have not the GUID is not a problem (you can create a fake GUID when you add/create one o more portal(s) in bookmarks), but bookmarks when you click a portal bookmark from the list uses the window.zoomToAndShowPortal(guid, latlng) to move the map and select the portal.
Furthermore, without the GUIDs the star not works properly.
It's not possible import correctly KML data in bookmarks if in the KML are not the GUID of portals.
Why don't edit the plugin? I'll open an issue about this (I hope that @nhamer and others will want to partecipate).

Here's an example KML file created by exporting from google maps. It contains ingress portals for overlay in IITC http://pastebin.com/rPiEAaGk

Yeah, no GUIDs. So essentially, highlight can't work, star can't work. I suppose you could create a fake GUID on import, but the limitation is going to be confusing.
What's your use case here? It would seem more natural to import KML to e.g. drawtools.

These kml files overlay portals in IITC (using the Overlay KML plugin) making it easy to locate and navigate to them. The problem is that when iitc is closed, you need to load the file again. Saving as bookmarks means they would stay loaded and be synced between devices

Ref. #1154 possibly - possibly overlay-kml should persist a kml and handle network-links? I wasn't aware it didn't even save the KML between sessions (that's dum), but network links would also keep it current (not even using sync plugins)?

If you're sharing files between people I think you may need a shared network location to link too. Not sure as I've never tried it


Remove UIBlocker

my requirement is when loading UI should not block, I want allow to click back button for customers.How to do that.

@kavin06: You might want to refer to #174 and the solution there.


Defunct process issue under latest version

[Using puppeteer-sharp under asp.net .net core 2.2 keeps defunct chrome processes when Puppeteer object is disposed]
Steps to reproduce:

Install asp.net core 2.2 api
With simple request call puppeteer, lunch browser, get content close browser
Observe a lot of defunct processes
Sooner or later observer Docker: "No space left on device"

 using (var browser = await Puppeteer.LaunchAsync(options).ConfigureAwait(false))
  {
                var allPages = await browser.PagesAsync().ConfigureAwait(false);
                var page = allPages.First();

                try
                {
                    await page.SetBypassCSPAsync(true);
                    await page.SetRequestInterceptionAsync(true).ConfigureAwait(false);

                    DisableMediaContent(page);

                   // navigation to page omitted here
                }
}

What I am doing wrong? Any recommendations are much appreciated. Thank you in advance.

Hey @mesanja. It seems Puppeteer also have this issue. Could you check this issue and see if any of the comments there help?
https://github.com/GoogleChrome/puppeteer/issues/1825

Hey @mesanja. It seems Puppeteer also have this issue. Could you check this issue and see if any of the comments there help?
GoogleChrome/puppeteer#1825

Hi @kblok thanks for response, they propose to use dumb-init, but we should have separate runner for chromium. I was thinking about console app that will encapsulate all work with puppeteer-sharp in order for chromium to have that console app as parent, so we could send kill -s 9 console_app_pid.

I ran into some issues when running Puppeteer-Sharp on a machine with low resources. I did this:
_chromiumProcessID = browser.Process.Id;

public static void EnforceChromiumClose()
{
    var process = Process.GetProcessById(_chromiumProcessID);
    if (_chromiumProcessID > 0 && process?.HasExited == false)
    {
        process.Kill();
    }
}

Not clean at all. But if Chromium can't gracefully close itself. I would force kill it.

Which Puppeteersharp version does the problem occur with? We recently fixed a chromium close timeout issue.

It's turned out that issue is not related to puppeteer it self but for docker nature. In order to wipe zombies I used this solution https://github.com/Yelp/dumb-init.
For anyone who faced with same issue, please use dumb-init.
Thanks all.

@mesanja thank you for sharing that back here!


Discussion: Animation fine control -- chain, cancel, reverse, pause

Once you have any animations, you typically run into the following use cases, in decreasing order of importance (my opinion):

Run several animations in parallel
I assume this is the base case and will be supported by default

Chaining animations, i.e. start another as soon as the first one finishes (maybe with a delay)
Could be supported with an event, but with the potential for network delay, perfect animations could probably not rely on that.

Canceling an animation, e.g. when user clicks a button
Python API would need a reference to the running animation.

Running the same animation back and forth

Pausing an animation with an option to continue (needs a reference, again)

Yes, this is called in Flutter terms "explicit" animations: https://www.youtube.com/watch?v=GXIJJkq_H8g
What is currently implemented in Flet (and should be documented) are "implicit" animations.
Cancelling, pausing, back/forth are all responsibilities of AnimationController which I was planning to expose in Flet API.

Thanks, good to hear. Looks like AnimationController has a addStatusListener that can be used to create events to Python, e.g. when the animation ends.


nuraft param with_log_sync_stopping_gap causes new follower to stop resync

https://github.com/eBay/HomeStore/pull/546/files
Uploading not-working.tar.gz…

@sanebay logs are not downloadable

For some reason, its stuck at uploading. I will paste in the internal channel.

This issue is mostly due to nuraft will use log pack to sync the new follower till the new follower catch up to log_sync_stopping_gap, however ,  in HomeStore we localize the journal entries , there is no callback available in log pack path,  the log from leader will be added as-is to the HomeStore which is obviously wrong.
The #546 make log_sync_stopping_gap to default 99999 which is larger than any of the written IO counts in our UTs that avoid the log pack sync.
#587  set it to max_int32 as well as enable the learner feature, with the learner feature new member will not go through log pack in any case.


[FEATURE_REQUEST] allow toggle fullscreen for iframes in workspace view

Is your feature request related to a problem? If so, please describe.
Toggle fullscreen buttons currently don't work in workspace view because iframes need to have
<iframe allow="fullscreen"></iframe> set.
Describe the solution you'd like
Set allow="fullscreen" globally for dashy, or if there are any drawbacks that im not aware of providing it as an optional setting would be great
Priority
Medium (Would be very useful)
Is this something you would be keen to implement
No

Sure, good idea - will look into this and hopefully get a PR submitted this week :)
And thank you so much for the sponsorship 💖

Apologies again for the delay, have had a lot on. This has now been implemented in #557 :)


⚠ Bofrost.at has degraded performance

In 74b23b0, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2174 ms

Resolved: Bofrost.at performance has improved in d5b5caa.


Multi language books

Taken from a Spigot post

A book will be give with a command to a player base on his minecraft language.
So I can create ONE book for french players and another book for english players.
If the minecraft client language has no book that was link to this language, the player will receive an english books with the command you already provide to give a book.

This can now be done by inserting the locale placeholder in the give command, since the locale placeholder has been added to the PlaceholderAPI Player-Expansion


Adds VercelPostgres vector store

Use PGVector from Edge environments!
Opted to use the pool syntax instead of the preconfigured sql one to allow for different connection strings and keep one execution path, but could optimize in the future.
Docs to follow!

Most internals are similar to @bracesproul's PGVector implementation, may be worth combining if we can figure out an elegant way to do so, but I have a feeling this will naturally diverge a bit over time.


Increase slippage error

Hi guys, when I increase the slippage to > 1. I get the following error in my Python console:
Could not identify the intended function with name swapExactTokensForETH, positional argument(s) of type (<class 'int'>, <class 'int'>, <class 'list'>, <class 'bytes'>, <class 'int'>) and keyword argument(s) of type {}.
Found 1 function(s) with the name swapExactTokensForETH: ['swapExactTokensForETH(uint256,uint256,address[],address,uint256)']
Function invocation failed due to no matching argument types.
`from decimal import *
from web3 import Web3
from pancakeswap import PancakeSwap
w3 = Web3(Web3.HTTPProvider("https://bsc-dataseed1.binance.org:443"))
address = "private"
private_key = "private"
bnb = w3.toChecksumAddress("0xbb4cdb9cbd36b01bd1cbaebf2de08d9173bc095c")
token = w3.toChecksumAddress("0x4e6415a5727ea08aae4580057187923aec331227")
buy_price = w3.toWei(Decimal('20'), 'ether')                            #1 BNB
pcs_wrapper = PancakeSwap(address, private_key, version=2, max_slippage=2)
pcs_wrapper.make_trade(token, bnb, buy_price)`

Oops just found out that max_slippage=0.1 equals to 10% slippage...


Host Config fontFamily as array

Problem
Host Config is a shared cross-platform configuration, but doesn't currently address the reality that different platforms have different fonts.
Our current Host Config lists fontfamily as a string.
"fontFamily": "Segoe UI",

Spec
fontFamily should become an array of strings (similar to CSS), and the renderer should apply the first one that exists.
"fontFamily": [ "Segoe UI", "Roboto", "serif" ],

This was already discussed before, and we had agreed that fontFamily could remain a comma-separated list of font names as that is how both HTML and XAML do it. Any particular reason why we now believe that isn't good enough?

Sorry for the delay here. This seems like a pretty strange design that we would have to carefully document in each renderer. Since it's really an array of strings it seems like we should model that properly in JSON. It's a lot less error prone to string.Join(arr, ", ")  over string.split(', ') given we would need to carefully check for ',' or ', ' etc. Feels like this is our best chance to clean this up before release and not break folks.

No strong opinion, but again this was discussed and agreed upon before. IMO XAML and HTML both using just a string is indication enough that the simple string model is good enough.

Indeed CSS and XAML allow comma separated string in FontFamily, unfortunately Android and iOS do not as far as I can tell. So having to crack open the string based on a comma seems a bit error prone, especially when considering new renderers come online they may also have to split the string.

Just looking at Android:

The names of the supported font families are not formatted the same as in HTML
Only one font family can be specified

So in the end, a HostConfig has to be specially crafted for Android, and a single string is enough. I haven't looked, but I'm going to assume the situation is similar on iOS. Why would we need an array?

We are trying to specifically avoid having to have separate host configs. The plan for Android would be loop through the fontFamily array and find the first font that exists on the system. Same as what CSS and XAML do, but we have to do it manually, since the native UI platform doesn't have it for us

I don't know how you do that given that Android font family names are not the same are those used in HTML (and likely iOS.) To solve that problem, Adaptive would have to define its own font family names and each renderer would have to map them to their target platform. IMO that makes very little sense given that one HostConfig file is supposed to be specific that that one host anyway.

Hmm, I don't think I'm following.
The proposal is that the Renderer would enumerate all the font names in the array and apply the first one that exists. In CSS and XAML this works automatically since their UI platforms already work this way  (as long as we join the arrray into a single string with string.join(arr, ", "):)
Example
This array would work for iOS, since SF Pro Display would match. It would work for Windows because of Segoe UI, and Android would work since it would find Roboto, lastly as a catch-all any other renderer that didn't match anything would match serif.
"fontFamily": [ "SF Pro Display", "Segoe UI", "Roboto", "serif" ]

But what is the scenario where a single HostConfig is used across multiple hosts?


Upgrading kubeadm clusters

This page link says
on first control plane node, run "sudo kubeadm upgrade apply v1.21.x"
For other control plane nodes, run "sudo kubeadm upgrade node"
When applying a script for upgrade via some automation service, which will run this script one node at a time, how could the node distinguish itself as first node or not, so it can call "...upgrade apply" or "...upgrade node" ?
Thank you,
Renuka

hi, the same page explains:

Pick a control plane node that you wish to upgrade first. It must have the /etc/kubernetes/admin.conf file

this is considered the "first" CP node to run "plan / apply" on.
/close
/kind support

An automated service, picks any one control plane node at a time and run the script and move on to next node.
We would not know ahead of time, which node is first picked.
So the script should be able to programmatically identify, if this is the first node or other.
Every control plane node has "/etc/kubernetes/admin.conf".
Question: Programmatically how the script can identify first or other ?
per doc: "Your cluster was upgraded to "v1.21.x" upon apply in first node. How could this be detected programmatically ?


The PdfRenderer cannot set the API level to 19...

Is it posible to set the API level to 19?

This API has been introduced in Android Lollipop which is corresponding with API level 21.
So it's not possible to change the API level and compile this sample with API levels lower than 21.


Improved exception handling

This PR improves exception handling by providing clearer exception messages when an APIException occurs. To revert to the previous behavior, set the environment variable INDUCTIVA_DEBUG_API_TRACEBACK to True.

QA
To test this PR we need to create an API Exception.
We can do that by doing inductiva tasks list --task-id 5dm4qty5srrd9rtje07fer2vq that will give Error: Task ID not present in the list of tasks owned by the user.
The same should also be testes using a notebook by using:
t=inductiva.tasks.Task("5dm4qty5srrd9rtje07fer2vq")
t.get_info()

@bgregoinductiva @Franreno @luispcunha

I observed the expected behavior while testing. I didn't test it in a notebook because I was not able to set the API key.
(env_tests) brunogrego@oxygen:~/inductiva/test_scripts$ inductiva tasks list --task-id 5dm4qty5srrd9rtje07fer2vq
Error: Task ID not present in the list of tasks owned by the user.

(env_tests) brunogrego@oxygen:~/inductiva/test_scripts$ export INDUCTIVA_DEBUG_API_TRACEBACK=True

(env_tests) brunogrego@oxygen:~/inductiva/test_scripts$ inductiva tasks list --task-id 5dm4qty5srrd9rtje07fer2vq
Traceback (most recent call last):
  File "/home/brunogrego/inductiva/env_tests/bin/inductiva", line 8, in <module>
    sys.exit(main())
  File "/home/brunogrego/inductiva/env_tests/lib/python3.10/site-packages/inductiva/_cli/main.py", line 96, in main
    exit_code = args.func(args)
  File "/home/brunogrego/inductiva/env_tests/lib/python3.10/site-packages/inductiva/_cli/cmd_tasks/list.py", line 32, in list_tasks
    table_dict = tasks.to_dict(task_list)
  File "/home/brunogrego/inductiva/env_tests/lib/python3.10/site-packages/inductiva/tasks/methods.py", line 33, in to_dict
    execution_time = task.get_computation_time(cached=True)
  File "/home/brunogrego/inductiva/env_tests/lib/python3.10/site-packages/inductiva/tasks/task.py", line 828, in get_computation_time
    return self._get_duration(
  File "/home/brunogrego/inductiva/env_tests/lib/python3.10/site-packages/inductiva/tasks/task.py", line 797, in _get_duration
    info: TaskInfo = self.get_info() if not cached else self.info
  File "/home/brunogrego/inductiva/env_tests/lib/python3.10/site-packages/inductiva/tasks/task.py", line 382, in info
    return self.get_info()
  File "/home/brunogrego/inductiva/env_tests/lib/python3.10/site-packages/inductiva/tasks/task.py", line 395, in get_info
    resp = self._api.get_task(params, skip_deserialization=True).response
  File "/home/brunogrego/inductiva/env_tests/lib/python3.10/site-packages/inductiva/client/paths/tasks_task_id/get.py", line 258, in get_task
    return self._get_task_oapg(path_params=path_params,
  File "/home/brunogrego/inductiva/env_tests/lib/python3.10/site-packages/inductiva/client/paths/tasks_task_id/get.py", line 202, in _get_task_oapg
    raise exceptions.ApiException(status=response.status,
inductiva.client.exceptions.ApiException: (404)
Reason: Not Found
HTTP response headers: HTTPHeaderDict({'content-type': 'application/json', 'X-Cloud-Trace-Context': 'a79e531111a73fa5e47152a3a3fcae7c', 'Date': 'Tue, 18 Jun 2024 17:13:54 GMT', 'Server': 'Google Frontend', 'Content-Length': '72'})
HTTP response body: b'{"detail":"Task ID not present in the list of tasks owned by the user."}'

@sergiomsantos can you test this ?

INDUCTIVA_DEBUG_API_TRACEBACK=true

After some investigation:
This is expected. If the environment variable isn't set initially, we replace the Ipython custom_exc method. If you later set the environment variable and reload the module, the code to replace the custom_exc method won't run again because it was already executed the first time, meaning the exception handler was already replaced.
To "fix" this, we would need to save the default handler and update custom_exc to use the default handler when debug is on, and our custom handler when debug is off.
@sergiomsantos do you think we should do this ?


Initial commit to Hyperledger Labs

Signed-off-by: YazzyYaz<EMAIL_ADDRESS>

@ryjones if it's good with you, I'll merge it as soon as you give me a thumbs up

Also, would it be possible to give @bobsummerwill rights to this repo too? He's also part of the ETC Coop who helped push for Ethercluster to join Hyperledger Labs


Set the header's size to fit to be CGSizeZero

Fixes #965
When there was no size context for the ShowFeeds's header stackview's height, it would default to the one provided by systemLayoutSizeFittingSize which is a full screen. Defaulting to 0,0 makes it fit to the smallest possible value.

Awesome :+1:

Please?

legit

:+1:  done


feat: allow passing pkg manager version

Proof it works:

https://github.com/graphprotocol/graph-tooling/pull/1528
closes #311

@saihaj can we test it on one of the repos that already use pnpm v8? just to make sure this new input and pnpm version picking is not breaking anything

https://github.com/dotansimha/graphql-yoga/pull/3130

This has broken the release action of yoga, I don't know why it worked in your test, but it seems the type attribute is detected as required in this action: https://github.com/n1ru4l/envelop/actions/runs/7168575591


Tick system

For those of you who find it very frustrating to work with ticks, I have come up with a solution.
I don't know how many of you who would find this useful, but I do at least.
You just implement this abstract class, and you'll be ready to go.
An example on how you could use IsActive, is when wrapping around peds.
This can also be found in my mod, which you can see on my Github page.
You could make it tick only when the ped is alive, and so on.
IsActive = Ped != null && Ped.IsAlive;
Basically, it organizes the ticks, and makes the whole thing a lot easier.
You can also give any instance a lifespan.
public abstract class Updater {
    private bool _active = false;

    private int _activeTick;
    private int _inactiveTick;

    private readonly int _startTick;
    private int _tick;

    private Dictionary<int, Action> _actionQueue = new Dictionary<int, Action>();

    protected Updater(int startTick = 0) {
        _tick = startTick;
        _startTick = startTick;
    }

    public void Tick() {

        OnUpdate(_tick);

        if (_actionQueue.ContainsKey(_tick)) {
            _actionQueue[_tick].Invoke();
        }

        if (_tick == 0 || _tick == _startTick) {

            OnFirstUpdate();

        }

        if (_active) {

            _inactiveTick = 0;

            OnActiveUpdate(_activeTick, _tick);

            if (_activeTick == 0) {
                OnFirstActiveUpdate(_tick);
            }

            _activeTick++;

        } else {

            _activeTick = 0;

            OnInactiveUpdate(_inactiveTick, _tick);

            if (_inactiveTick == 0) {
                OnFirstInactiveUpdate(_tick);
            }

            _inactiveTick++;

        }

        _tick++;

    }

    /// <summary>
    /// Fired everytime a tick has been made.
    /// </summary>
    /// <param name="tick">The total amount of ticks.</param>
    protected abstract void OnUpdate(int tick);

    /// <summary>
    /// Fired on the absolute first update.
    /// </summary>
    protected abstract void OnFirstUpdate();

    /// <summary>
    /// Fired everytime a tick has been made while active.
    /// </summary>
    /// <param name="activeTick">The current tick on this activity state.</param>
    /// <param name="tick">The total amount of ticks.</param>
    protected abstract void OnActiveUpdate(int activeTick, int tick);

    /// <summary>
    /// Fired everytime a tick has been made while inactive.
    /// </summary>
    /// <param name="inactiveTick">The current tick on this activity state.</param>
    /// <param name="tick">The total amount of ticks.</param>
    protected abstract void OnInactiveUpdate(int inactiveTick, int tick);

    /// <summary>
    /// Fired the first time an active update has been made.
    /// </summary>
    /// <param name="tick">The total amount of ticks.</param>
    protected abstract void OnFirstActiveUpdate(int tick);

    /// <summary>
    /// Fired the first time an inactive update has been made.
    /// </summary>
    /// <param name="tick">The total amount of ticks.</param>
    protected abstract void OnFirstInactiveUpdate(int tick);

    public Dictionary<int, Action> ActionQueue { get { return _actionQueue; } set { _actionQueue = value; }}

    /// <summary>
    /// Returns the tick of which the ticking started.
    /// This isn't always 0, as the StartTick can be set.
    /// </summary>
    public int StartTick { get { return _startTick; }}

    /// <summary>
    /// Returns how many times this Updating instance has ticked.
    /// </summary>
    public int Tick { get { return _tick; } }

    /// <summary>
    /// If active, returns how many times it has been ticked during its session.
    /// </summary>
    public int ActiveTicks { get { return _activeTick; }}

    /// <summary>
    /// If inactive, returns how many times it has been ticked during its session.
    /// </summary>
    public int InactiveTicks { get { return _inactiveTick; }}

    /// <summary>
    /// Returns whether this Updating instance is active.
    /// </summary>
    public bool IsActive {
        get { return _active; }
        protected set { _active = value; }
    }

}

Looks like a kind of event broker. Could become interesting.

Yeah, pretty much.

I think this would be perfect for a new wiki page :+1:
Would you mind if we move this to the wiki? @Nacorpio

You know what I find as good as the code itself? You added XML comments.  I always say the best framework is worth nothing without a decent documentation. So chapeau.

I don't mind, @JohnnyCrazy. =D

Done :)
Also added a list to the Get Started page

Good job.


New interface traits

Wow, I didn't expect this to impact performance much on my project (8 bit unbuffered parallel interface using fill_contiguous), but its over twice as fast as the master branch!
Please try it out, I think it might be faster than #146

I don't think it's a good idea to remove support for display-interface entirely at this point in time. We should at least have a fallback option for users that use custom display-interface impls, for example the FSMC support for STM32F4 (https://github.com/stm32-rs/stm32f4xx-hal/tree/master/src/fsmc_lcd).

I don't think it's a good idea to remove support for display-interface entirely at this point in time. We should at least have a fallback option for users that use custom display-interface impls, for example the FSMC support for STM32F4 (https://github.com/stm32-rs/stm32f4xx-hal/tree/master/src/fsmc_lcd).

How would you want to address this though? Deprecate DI with 0.9 and give some time? Use features to switch between DI and "PI" versions? I don't want to overcomplitcate things or delay them unless necessary.
I think if we release this as 0.9, FSMC can keep supporting 0.8 until they reimplement for the new version. I find it interesting they decided to implement DI inside the HAL directly.

I think it's technically possible to create an adapter to convert an old display_interface into the new trait:

expand me!
pub struct LegacyAdapter<DI: WriteOnlyDataCommand>(pub DI);

impl<DI: WriteOnlyDataCommand> Interface for LegacyAdapter<DI> {
    type PixelWord = u16;
    type Error = display_interface::DisplayError;

    fn send_command(&mut self, command: u8, args: &[u8]) -> Result<(), Self::Error> {
        self.0.send_commands(DataFormat::U8(&[command]))?;
        self.0.send_data(DataFormat::U8(args))
    }

    fn send_pixels<const N: usize>(
        &mut self,
        pixels: impl IntoIterator<Item = [Self::PixelWord; N]>,
    ) -> Result<(), Self::Error> {
        self.0
            .send_data(DataFormat::U16BEIter(&mut pixels.into_iter().flatten()))
    }

    fn send_repeated_pixel<const N: usize>(
        &mut self,
        pixel: [Self::PixelWord; N],
        count: u32,
    ) -> Result<(), Self::Error> {
        self.send_pixels((0..count).map(|_| pixel))
    }

    fn flush(&mut self) -> Result<(), Self::Error> {
        Ok(())
    }
}

I don't think it is a good idea for us to have this adapter though, because it doesn't work with Rgb666 and it doesn't solve any of the performance problems. You're better off just sticking with 0.8.0 instead.
I merged the CommandInterface and PixelInterface traits into a single Interface trait. The split was necessary in a previous version of this PR, but it isn't anymore.
almindor suggested putting the interface trait in it's own crate. I think this is probably the right thing to do eventually, but I would like to wait to see if the trait needs any more changes. It should be backwards compatible to move the trait to it's own crate as long as we publicly export it.

I think we should make a call. My take atm is that we should do this:

Separate the new trait into Write and Read version (read can be completely unimplemented at the moment). I was close to adding a DI read version ~1 year back but my MCU fluffed out on me
Separate the interface into its own crate, either in this repo or its own. @GrantM11235 I'll let you decide which way since its your code
Bump mipidsi version to 0.9 after and release

There's also the option of using the existing DI repo for this since I have write access, but I don't have admin so it'd be a cointoss on how that'd go longer term.
When it comes to dependants of DI I see only the two hal crates outside of direct drivers or implementations. I don't think it's worth doing some sort of deprecation delay. Looking at the code, it should be super trivial for someone to re-do the existing hal implementations for the new traits. And the nice thing is they can technically implement both alongside as well.

There is nothing to split, this trait is already effectively the Write version. The Read version will almost certainly look something like pub trait ReadInterface: Interface { /* ... */ }, but that is outside the scope of this PR.
The main reason to put the trait in it's own crate is to provide external implementers with stability even when there are breaking changes to the main mipidsi crate. I think it is premature to do this now because we may or may not need to make changes to the trait. Once we are reasonably confident in the trait we can release mipidsi-interface (or whatever) as instant 1.0.0.
The trait is specifically made to be used by mipidsi, not any other types of displays, so I think it should stay in this repo even after it is put in it's own crate.
I agree that we should publish version 0.9 once this is merged :rocket:

I don't think it is a good idea for us to have this adapter though, because it doesn't work with Rgb666 and it doesn't solve any of the performance problems. You're better off just sticking with 0.8.0 instead.

We should at least mention this in the migration guide.

There's also the option of using the existing DI repo for this since I have write access, but I don't have admin so it'd be a cointoss on how that'd go longer term.

We could also reuse the display-driver-hal repo, which was supposed to add higher level traits, like backlight control, but I don't see a reason not to also include the display-interface replacement.

When it comes to dependants of DI I see only the two hal crates outside of direct drivers or implementations. I don't think it's worth doing some sort of deprecation delay. Looking at the code, it should be super trivial for someone to re-do the existing hal implementations for the new traits. And the nice thing is they can technically implement both alongside as well.

I don't think that expecting HALs to implement special traits just for our crate is a scalable solution. If the new traits turn out to be superior to display-interface we might as well try to convince other display driver crates to use them instead.

Once we are reasonably confident in the trait we can release mipidsi-interface (or whatever) as instant 1.0.0.

The "or whatever" is definitely the way to go, because mipidsi-interface is very misleading, because this will still not have anything to do with the actual MIPI DSI.

I didn't have time to look into the new traits in detail, but set_pixels being generic over N and still using an iterator doesn't seem to be the perfect solution. Why do we need to use iterators if we can just pass slices to the pixel data around? It might monomorphize into the same code in some cases, but with IntoIterator we need to unnecessarily rebuffer the pixel data in the Interface impls when the pixel data is available as a buffer of bytes in the first place.

On the topic of separate crate. If we do not attempt any kind of unification outside of this driver (mipidsi), then I suggest we make this interface private (as in not really needed to be exposed to the user explicitly) and hide it behind the builder.
Users would then just do either Builder::spi(...)... or Builder::new().with_spi()... and that's it. Everything else would be sort of hidden.
Unless I'm forgetting something, users don't really need to care about the DI. We'd have to make monomorphised builder due to release but that's not such a big problem.

I didn't have time to look into the new traits in detail, but set_pixels being generic over N and still using an iterator doesn't seem to be the perfect solution.

[Word; N] is supposed to represents one pixel[^1], so N will be between 1 and 3. When sending rgb565 pixels over an 8 bit interface, we will always send data as pairs of bytes, so we will use an iterator of [u8; 2]. I suspect (but haven't tested) that this is more efficient compared to just an iterator of bytes because the compiler doesn't need to worry about running out of buffer space mid-pixel.

Why do we need to use iterators if we can just pass slices to the pixel data around?

99% of the time we can't just use slices of pixels. For starters, embedded-graphics only uses iterators. Even if we did have a slice of pixels, we would still need to convert rgb565 to big-endian or convert rgb666 to left-aligned. Even if the pixels were already in the right format, they might still need to be copied to a dma capable buffer

but with IntoIterator we need to unnecessarily rebuffer the pixel data in the Interface impls

Using an iterator at the interface boundary instead of a slice means that we don't need to unnecessarily buffer the data when using an unbuffered interface like ParallelInterface
[^1]: If we add support for rgb444, it will probably involve sending pairs of pixels packed into [u8; 3], but I'm not sure exactly how or if that would work at this point.

One of my design decisions when making this trait was to only support mipi dcs style interfaces, not all "displays" that have some concept of "commands" and "data". The reason is that other display interfaces work differently and have different usage patterns. For example, SSD1306 oled command parameters are sent as "commands" but in mipi dcs they are "data".
That means the only other crates that can use this trait are the other crates that support the same displays as us, such as ili9341. They are welcome to use the trait, but they are also welcome to deprecate themselves and just use our crate instead :shrug:

We can't make the trait private/sealed because we want to allow high performance, hardware specific impls like stm32's fsmc, rpi's pio, or esp32's i80. These impls don't actually need to be in the HALs, they can be in third party crates as long as the HALs support the basic operations.
We could still add convenience constructors like Builder::new_spi(...) but I'll leave that for another day.

But one thing I would like to bring up is the mutable slice that is used as a buffer in SpiInterface. There might be use cases where this would be harder to work with, because a Display that uses SpiInterface couldn't be stored in the same struct as the SPI buffer. What is the reason you decided to use this approach instead of making the buffer a const generic inside the SpiInterface?

Some chips can only dma from certain memory regions. If the user is able to create the buffer in one of those regions, then their hal's spi impl won't need to copy the data into another buffer.
I changed it to be generic over AsMut<[u8]>, so now you can use &mut [u8], [u8; N], or even Box<[u8]>, but I'm not sure if it is worth the extra confusion and I'm worried it could encourage people to use a large array and blow up their stack.
Initializing a display with an inline array buffer involves creating the array, moving it into the interface, then moving that interface into the display driver. Each of those moves involves allocating space for the whole array on the stack and copying the whole array. LLVM is notoriously bad at optimizing these copies away. Moving the display driver after that will have the same problem.
For practical purposes it's probably easier to just use static-cell to get a &'static mut [u8]. Then the interface/display is also 'static and can be moved around freely

But one thing I would like to bring up is the mutable slice that is used as a buffer in SpiInterface. There might be use cases where this would be harder to work with, because a Display that uses SpiInterface couldn't be stored in the same struct as the SPI buffer. What is the reason you decided to use this approach instead of making the buffer a const generic inside the SpiInterface?

Some chips can only dma from certain memory regions. If the user is able to create the buffer in one of those regions, then their hal's spi impl won't need to copy the data into another buffer.
I changed it to be generic over AsMut<[u8]>, so now you can use &mut [u8], [u8; N], or even Box<[u8]>, but I'm not sure if it is worth the extra confusion and I'm worried it could encourage people to use a large array and blow up their stack.
Initializing a display with an inline array buffer involves creating the array, moving it into the interface, then moving that interface into the display driver. Each of those moves involves allocating space for the whole array on the stack and copying the whole array. LLVM is notoriously bad at optimizing these copies away. Moving the display driver after that will have the same problem.
For practical purposes it's probably easier to just use static-cell to get a &'static mut [u8]. Then the interface/display is also 'static and can be moved around freely

Correct, when it comes to CS there are essentially two scenarios. Either the CS is handled by the SPI/HAL directly or by us. In general we should favour the SPI/HAL to handle it because at least some MCUs have HW control of CS built-in to the "fast SPI" implementation. Note that many HALs have this implemented "wrong" and do not automatically switch CS and other SPI settings (such as mode, speed etc.) properly when switching between SpiDevice instances, but that's a problem out of scope for this driver.
If we control CS we need to ensure it's reset properly at the end of a transaction of course. If we do not, the best we can do is just document that it's the upstream's responsibility to not let CS hang.
DMA buffers are a must, the speed increase when using them is quite large.

I'm confused, the message you are replying to isn't about CS

I'm confused, the message you are replying to isn't about CS

Up, misclicked. I read parts in email and then came here and clicked the wrong thing. It was meant mostly as a reaction to the CS discussion, but also l inked to the DMA buffer requirement.

@rfuest I'll let you review one more time since your changes were done, but I'm inclined to go ahead with this and merge soon to get the ball rolling.

The PR is in already in a very good state, but there are still 2 unresolved conversations that should be addressed before this can be merged. The one about the duplicate name isn't too important, but I think that the one about the flush method needs to be discussed.


MINOR: Flaky Optimization test due repartition topic creation failures

The topology optimization test was getting intermittent failures because of failures to create repartition topics on startup.  This PR Increased admin client retries
I kicked off the system test with 25 repeats, all passed http://confluent-kafka-branch-builder-system-test-results.s3-us-west-2.amazonaws.com/2018-12-21--001.1545436859--bbejeck--MINOR_flaky_optimization_test_create_repartition_fails--6cd55e2/report.html
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

ping @guozhangwang , @mjsax  and @vvcephei  for reviews


IOS build error

Hi, my IOS build is error, after searching the issue this caused by target architecture which should be arm64 and i have update it in xcode.

But i still can't build it. Below is the error log
ld: building for iOS Simulator, but linking in dylib built for iOS, file '~/app/ios/Pods/ScanKitFrameWork/scansdk-ios-tool-<IP_ADDRESS>0/ScanKitFrameWork.framework/ScanKitFrameWork' for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

@ariona
Generally Simulator is x86 architecture, not ARM64. You can only use a real machine to debug this library.

Aah. thats why is there any workaround to make it work in simulator?
and maybe you need to add about turning off bitcode to the readme file.


[#157510099] bump paas-admin to v0.4.0

What
Bump paas-admin to v0.4.0

paas-admin v0.3.0 implements the first version of the pricing calculator
v0.4.0 handles not allow remove the latest org manager
How to review

Code review
Who can review
Not @chrisfarms

Wait, there is more changes I am going to approve now in paas-admin

Sorry. Take it away.

@paroxp updated and merged :+1:


Tooltip repositioning strategy/context

Currently, even if tooltip is configured to be displayed below the original element, but there is not enough space on the screen (uses window object for space calculation, e.g. $(window).height()), tooltip gets repositioned to the top.
It would be great to have an option to change context from screen to page (to use document object for space calculation, e.g. $(document).height())
@louisameline, what do you think about this improvement?

I'm keen to see this enhancement as well, especially since it will change to top position even if there is even less space above.

Tooltipster v4 is out and offers smart positioning in the viewport, or optionaly relatively to the document only, and offers the possibility to disable repositioning. Thank you.
What's new in v4: #565
Migration guide from v3 to v4: #566


Fw 3477 v2 story image stretch fix

Definition of Done

[x] Acceptance criteria has been met

song images should be displaying properly


SDK should log to logging facility, not to console.

There a few places in the code where this needs to be cleaned up.

Fixed in 0.10.0


Fixing topomaps.

Fix #129

These modifs seems to fix most of the problem... but there appear to still have some issues. Comparing with MNE topomaps using:
import mne
fname = "/Users/christian/Code/pylossless/pylossless/assets/test_data/sub-s02/eeg/sub-s02_task-faceO_ica1_ica.fif"
ica =  mne.preprocessing.read_ica(fname)
ica.plot_components()

we obtain:

Compared to what we get with TopoViz:

The first 5 seem to be the same, with an inversion in the camp (red becomes blue)... then, some seem to correspond but not all, and not for the same index...

I pushed a couple commits to get this PR over the line, our topoplots now match those produced by MNE. Using one of our pylossless test files:
fpath = Path(".") / "pylossless" / "pylossless" / "assets" / "test_data" / "sub-s01" / "eeg"
ica_fpath = fpath / "sub-s01_task-faceO_ica2_ica.fif"
ica = mne.preprocessing.read_ica(ica_fpath)

MNE:

and then selecting sub-s01_task-faceO_eeg.edf from the dropdown menu of the dashboard launched by pylossless_qc:

@christian-oreilly just give me a thumbs up if you agree that this looks like it solves the problem. FYI see commit 3dd5988 for the code I added.
Then there's just a failure in test_GridTopoPlot that needs to get worked out, and maybe we should try to add a specific test that assures that a topo of one component looks like one produced by MNE.

Note: that there is also a typo here (color should be "black" not "back") that we should fix before merging.

Nice job. Yes, I agree, these topomaps now seem to be identical. I suspect that the CI issue is the same one discussed in the context of the "rejection policy" PR, which seems to have nothing to do with either PR.

Nice job. Yes, I agree, these topomaps now seem to be identical. I suspect that the CI issue is the same one discussed in the context of the "rejection policy" PR, which seems to have nothing to do with either PR.

The test failure is related to this PR: It was caused by the change committed to pylossless/dash/tests/test_topo_viz.py, here

The test failure is related to this PR: It was caused by the change committed to pylossless/dash/tests/test_topo_viz.py, here

Oh... sorry about that. I just assumed. I should not have assumed without taking some time to check.
I must say that I don't remember why I changed it that way. I think our tests should guide us there... I'd go with your reverting of this change if it allows all tests to pass. We can always change this back later (adding corresponding tests) if we come up with a use case that requires this change...

The test failure is related to this PR: It was caused by the change committed to pylossless/dash/tests/test_topo_viz.py, here

Oh... sorry about that. I just assumed. I should not have assumed without taking some time to check.
I must say that I don't remember why I changed it that way. I think our tests should guide us there... I'd go with your reverting of this change if it allows all tests to pass. We can always change this back later (adding corresponding tests) if we come up with a use case that requires this change...

No worries, it has been quite some time since we worked on this! I already checked, and reverting the specific change to test_topo_viz fixes the failure. Per my comment above, it makes since to me that the input to GridTopoPlot should be a list of dict, so the code as it was before makes since to me. I'll revert the change.

I'm trying to get our code coverage to be ever increasing and there are missed lines in the added code. Will have to loop back to this when we have time.

@scott-huberty Can we merge this? I need it to be merged before forking this repo to start splitting the dash components into CONIE.

@christian-oreilly sure, but can you commit my suggestions before?
Note that dash/tests/test_topoviz/test_TopoViz is failing, IIRC it was a chrome driver issue. I marked it for skip in #144


ci: set main entrypoint for goreleaser

Codecov Report

Merging #40 (ea5d407) into main (b1ecf6c) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main      #40   +/-   ##
=======================================
  Coverage   84.86%   84.86%
=======================================
  Files           7        7
  Lines         304      304
=======================================
  Hits          258      258
  Misses         31       31
  Partials       15       15

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Support new FlexibleUNet pre_conv to none by default

Update tooltracking bundle to support new FlexibleUNet of MONAI 1.2

Thanks for the updates, I changed the base branch into dev-1.2 as we may have multiple changes on multiple bundles before 1.2 is released.

/build

Thanks for the updates, I changed the base branch into dev-1.2 as we may have multiple changes on multiple bundles before 1.2 is released.

this PR impacts MONAI label side, thus it should be merged into dev, I will change the base back.
cc @Nic-Ma

No problem, it's not in a hurry, monailabel 0.7 release candidate has passed with some changes there, we can make this stable and then push monailabel part back.


oidc login does not respect base uri

If you run headlamp on a server, and configure both the oidc login and base uri, then when you go to login the authentication popup is not respecting the configured base uri.

Hi @mhaddon , we (assumed we) have fixed this issue in #238 which went out in version 0.4.0. Can you confirm you're running that version?
Once it's confirmed we'll look into the issue.
Thanks in advance.

It still seems to be an issue.
I am using 0.4.0.

Thanks for confirming. We'll look into it, @mhaddon .

Fixed in:

#1158


Markdown is processed incorrectly

Actual behavior
Markdown is not processed as it should be.

How to reproduce

See https://busy.org/@espoem/growing-the-potential-audience-of-video-content or https://busy.org/@lemouth/particle-physics-utopian-implementing-an-lhc-analysis-on-a-computer-the-physics-objects

Browser: Chrome 66
Operating system: Windows 10

Hi, It's fine on my end on chrome Version 67.0.3396.62 (Official Build) (64-bit)

Also in my firefox 60.0.1

Does it persist?

I've revert the change on busy.org but the bug should still be visible on staging.busy.org @royachiron

Perhaps we can use this https://github.com/commonmark/commonmark.js instead of markdown-it, it's recommended by @hernandev

Actually commonmark does not handle table, so best option is still markdown-it for now. We can also take a look at marked.

We can migrate to markdown-it, but it still has this issue #1647.
I will use workaround from @hernandev and try to fix the issue in markdown-it directly and submit PR there.


Parameterized suite, parameterized test + cmd commands

Hi,
I spent some hours on my free time and at my work to develop two functionalities :

Parameterized tests (more exactly -> parameters for tests)
Cmd commands (--test, --suite, --ignoreparam, feel free to add a --help if you want to)

Actually I made cmd for the needs of the parameterized tests.
Important : Please note that I did not testes it on other platforms than Windows, in *.exe mode because I think you have more knowledges than me for this kind of testing :) Anyway the code is fully tested (see unittest-cpp\tests\TestParameterizedTest.cpp). Can I let you update the code for it works as a *.dll, *.lib, etc and for other platforms ?
For your test program you have to set the main this way :
int main(int argc, char**argv)
{
    return UnitTest::RunTestsCmd(argc, argv); // <- new entry point, replaces UnitTest::RunAllTests
}

And here is a basic example, the only macro you need is SET_TEST_PARAMETER :
// File: MyParameters.h
// Note: I recommand prefix your parameter name with "pz"
SET_TEST_PARAMETER(string, pzOneTwo, {
	values.push_back("one");
	values.push_back("two");
});

// File: MyTests.cpp
string result;
TEST(OneTwoIsFollowing)
{
	result += pzOneTwo();
	result += " ";
}
TEST(OneTwoIsFollowing_Verify)
{
	CHECK_EQUAL("one two ", result);
}

See file unittest-cpp\tests\TestParameterizedTest.cpp for more complex examples.
Best regards

Damned I forgot to push the two last commits !!!

Apparently it does not compile with older version than Visual Studio 2010, I will fix it soon

Thanks for the PR.
I'm a bit concerned about the size and scope of this. I realize that I don't have a contribution guide posted (bad maintainer, no Twinkie) -- but there are three (possibly more?) things going on here that are not all related:

Changing int to size_t in several places, including interfaces
The new runner
The new parameterized method of running a test with multiple inputs

At the least, they need to be broken up into separate PRs so the changes can be considered individually. That said, I don't want to waste your time on doing all that without offering some feedback first.
Changing int to size_t is likely the right thing to do, but I have to check my notes -- I seem to recall there's an issue with using size_t here for certain, shall we say, odd compilers that I try to maintain support for. Given that a 2.0 release is forthcoming, now would be a great time to make that change if we can.
The new runner is something many people have done from outside of the framework, and I would not be opposed to adding an "official" one to the project. I don't think I grasp, though, why this one is as complicated as it is, or why the RunTestsCmd function has a signature that takes more than argc and argv. Can you explain it to me? I don't see any unit tests for it that might help me understand.
Finally, parameterized tests. I have a lot of issues with this feature as implemented, and frankly I feel that it's just... out of place. I think, if we worked together, we could come up with an implementation myself and the users would be happy with, but this is the kind of new feature that would really lend itself to a discussion ticket before being implemented. Of course, having a proposed implementation to start is helpful for the discussion.
I don't want to discourage you from contributing, but I am closing this PR. Please feel free to continue the discussion here or on the issue board, and by all means open up separate PRs as you see fit.

Hi John,
Thank you for your review, I am a bit disappointed but not discouraged so I will improve until you can validate the PR ;)
I will shortly tell you the story of the tests parameters :
I began to wrote the first version at my work, locally in my project. I needed the functionality quickly and I was not thinking "open source" but when I realized I did a cool stuff I ask to the boss if I can publish the functionality and surprisingly he said yes :) Then I forked the repo and integrated the functionality in the project. I added some functionalities and here we are.
About the size_t, I must admit that I do not remember why I had to replace int with size_t. I think it was not that much necessary and for simplification of my next PR I will revert that.
About the runner, I wrote it because of test parameters needs (for the parameter --ignoreparam, which is helpful). It is that much complicated because I tried to respect the way of the original RunAllTests was written, but I did really wanted to make it simpler, in a dedicated class and using strings etc. If you want I can make a simpler version in a dedicated PR ?
I opened an issue, for discuss about the modifications that you would like.
Best regards

If you'd like to put the int->size_t changes into a pull request I'll take a look, although now that you point it out I may go after it if I don't see that PR in a few days.
As for the runner, I'd be more interested in a simpler version that the user can just pass argc and argv straight through. It'd probably be good to put only one or two options on it at first so that it can be built up incrementally. How about just building one that accepts one --suite SuiteName argument for starters?


uhi

Describe the bug
jhb
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-tjmkpp bb
Steps to reproduce
hb
Expected behavior
vv
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 131
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 78151765,
  "usedJSHeapSize": 76377833,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = Mü
Chat ID = 6b6ca1dc5995

Additional context
No response

nn

v


Ability to view full account history

currently the account history is limited to 200 transactions. this is due to the fact that when you load more transactions, you aren't loading the next page but the WHOLE list is being reloaded, e.g. 1-25, 1-50, 1-75, ... 1-175, 1-200
so the suggestion is to use offset in the RPC request and ask for 1-25, 26-50, 51-75 etc.
things to check: if 1 or more transactions are added to the beginning of the transactions list, loading more transactions should not result in duplicates (e.g. entry 25 is shifted to the place 26 which is first item on the next "page")
downsides of the suggestion: new transactions will no longer appear in the beginning of the list when loading next "page". this is completely acceptable but could be solved by #307

Sounds like it should be possible


feat: add tf-controller CRDs

CRDs from https://github.com/weaveworks/tf-controller

thank you @ishioni
can you also please provide a link to the CRD in the repo and just the project?


Cert error for https://download.tensorflow.org/

Not sure if this domain is still used. I sent a PR for #34460 , but this domain also exists in various places.
For example: tensorflow/lite/micro/examples/magic_wand/train/README.md

Hi @lr1d! I could not find any similar issue in documentation. Please create a new issue if you find any in current versions.
Thank you!

@mohantym I'm not aware of this HTTPS url in the doc. There are HTTP URLs and they work fine. However, note that the cert for https://download.tensorflow.org is still misconfigured.

@lr1d! I checked for leftover urls starting with "https://download.tensorflow.org" in website documentation. Mostly, they seem to have been replaced by  "https://storage.googleapis.com/download.tensorflow.org". Could point out any left out documentation that contains the above URL . Thank you!


Problems after updates in last days

I decided to make updates and I hit on some problems:
1. awesome-typescript-loader 3.0.0-beta.17
Although it proceed to markable speed up during build (during developing with dev server, time space between Invalid detection and rebuild process disappeared), when it's used with universal, this problem appears - https://github.com/s-panferov/awesome-typescript-loader/issues/323
2. t.map is not a function during starting
This error appears in universal console when I open browser with universal. I didn't find reason of this problem yet.
EDIT:

Fix should be released soon

I found it that this message is exactly - ctorParameters.map is not a function.. - and it's because some plugins updates (like angular2-modal...) now needs new version of angular 2 (at least 2.3) but angular2 universal does not support this new versions yet. So solution was set up fixed versions of those plugins in package.json in angular2 until angular2 universal will not have support.

FIXED

Hi,  @inzerceubytovani
I got the same error : "ctorParameters.map is not a function". But I don't understand your solution.
I'm trying to use the AngularFire module.
Which plugins must have fixed versions in package.json ?
Thanks

You have to find older version of modules which generate this error and use them.


[Bug] MongoDB crashes on startup post-install on OSX.

Description
On setting up Appsmith with docker on Mac OSX, MongoDB fails shuts down a few seconds after starting. This happens if you are running docker in a mounted drive like Desktop.
Steps to reproduce the behaviour:
Add steps to reproduce this behaviour, including console/network logs & screenshots

Download the install.sh file to OSX desktop
Run it
You'll see the mongo service running for a few seconds before it crashes

Important Details

Version: Self-Hosted v1.2.6
OS: MacOSX

This is a known issue with docker mongo pertaining to mounted directories.

WARNING (Windows & OS X): The default Docker setup on Windows and OS X uses a VirtualBox VM to host the Docker daemon. Unfortunately, the mechanism VirtualBox uses to share folders between the host system and the Docker container is not compatible with the memory mapped files used by MongoDB (see vbox bug, docs.mongodb.org and related jira.mongodb.org bug). This means that it is not possible to run a MongoDB container with the data directory mapped to the host.

The fix is to keep the /data/db mounted directory out of mounted volumes (like downloads, user etc.) or to create a volume with docker volume create.
Would be helpful if this was added as a warning to docs.
reference: https://iainhunter.wordpress.com/2016/01/12/avoiding-pitfalls-running-mongo-3-2-in-docker-on-osx/


Like parallax

Hi,
Thank You for your great library. can you explain how can i do some thing like these pictures with your library?
Thanks

I suggest to use stackoverflow for this kind of question.


feat: use lcov file if it already exists [PLAT-1985]

Description
Uses lcov file if it already exists - this is to handle the case of NextJS which we always rebuild so we don't need to download a coverage artifact from a previous build PLAT-1985

Falls back to other most recent artifact for the workflow run

Screenshots (if appropriate)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 33 (18.18%)  changed or added relevant lines in 2 files are covered.
9 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-15.9%) to 17.391%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main.ts
4
7
57.14%

src/actions.ts
2
26
7.69%

Files with Coverage Reduction
New Missed Lines
%

src/main.ts
2
41.38%

src/actions.ts
7
6.35%

Totals

Change from base Build<PHONE_NUMBER>:
-15.9%

Covered Lines:
15

Relevant Lines:
78

💛  - Coveralls


Stability tests refactor and support run on VM

Why do you use delays? I supposed that we can delayed generates by scheduler.

@ifilippov Start timeout is used to let generator stabilize speed. Packets sent and received
during timeout do not affect test result (because initial speed is high and a lot are dropped).
Description is corrected.

Set schedSpeed to 1ms instead of making delays. This will reduce speed ~immediately. Can you try this?


Add docstring for scanvi unlabeled category

[x] Closes #2091
[x] Tests added and passed if fixing a bug or adding a new feature
[x] All code checks passed.
[x] Added type annotations to new arguments/methods/functions.
[x] Added an entry in the latest docs/release_notes/index.md file if fixing a bug or adding a new feature.
[x] If the changes are patches for a version, I have added the on-merge: backport to x.x.x label.

meeseeksdev backport to 1.0.x


make -j problems building from scripts

Here:
https://github.com/openthread/borderrouter/blob/master/script/_otbr#L50
In the scripts directory, make is invoked with the number of processors for the "-j" parameter.
On a reduced target such as a Beagle Bone runs out of memory when compiling things.

RaspberryPi => 1GIG ram
BeagleBone => 512MEG ram.

for example:
  CXX      wpan-controller/libotbr_web_la-dbus_scan.lo
  CXX      wpan-controller/libotbr_web_la-dbus_ifname.lo
  CXX      wpan-controller/libotbr_web_la-wpan_controller.lo
  CXX      mdns-publisher/libotbr_web_la-mdns_publisher.lo
  CXX      pskc-generator/libotbr_web_la-pskc.lo
  CXX      web-service/libotbr_web_la-web_service.lo
[59433.551382] Out of memory: Kill process 21993 (cc1plus) score 707 or sacrifice child
[59433.559409] Killed process 21993 (cc1plus) total-vm:369304kB, anon-rss:352000kB, file-rss:712kB
g++: internal compiler error: Killed (program cc1plus)
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-4.9/README.Bugs> for instructions.
Makefile:839: recipe for target 'web-service/libotbr_web_la-web_service.lo' failed
make[3]: *** [web-service/libotbr_web_la-web_service.lo] Error 1
Makefile:425: recipe for target 'all-recursive' failed
make[2]: *** [all-recursive] Error 1
Makefile:556: recipe for target 'all-recursive' failed
make[1]: *** [all-recursive] Error 1
Makefile:481: recipe for target 'all' failed
make: *** [all] Error 2
debian@beaglebone:~/borderrouter$

Wow - I have not hit this type of problem in a very long time.
It's not really the "-j" option, the BBB does not have enough ram to compile the web_service.cpp file in its present form. End of story, full stop.
The issue really comes down to the BBB having 512K of RAM vrs 1GIG for the Raspberry PI.
Enabling virtual swap space is not really viable here.
At this point, I've split the file up into two parts (A) and (B) ... so the total "translation unit" is smaller.
That said, I think splitting the file into 3 equal sized parts is perhaps better.
I'll know more, as I get further into this.

More detail, these three functions seem to be the beasts from a compiler memory point of view.
void WebServer::HandleHttpRequest()
void WebServer::WebServer() <- The constructor
void WebServer::~WebServer() <- The destructor

I can provide a solution to this when I get some other things lined up.
Basically all that is required is to enable a SWAPFILE ...
I'm adding a script to the "script" folder to do that.

@DuaneEllis-TI Thanks for sharing the solution. Would you like to submit a PR for this issue?

This can be closed, as it is addressed as part of: https://github.com/openthread/borderrouter/pull/115

Closing.


[php] missing user code auto-complete on 0.10.1

When writing PHP code VSCode worked well completing user variables beginning with a $ sign or completing user functions. That worked in 0.9.1 but is missing in 0.10.1 - now only PHP built-in code completions are shown.

Fix delivered


Update SCDF core to 1.3.0.BUILD-SNAPSHOT

Fixes #361
Fixed #359

@sobychacko: It might be better to switch the CF-deployer to 1.3 BS as well. That way, we can run the CI plan against the OOM fixes included in the CF-deployer.

LGTM

Squashed, merged.


TextureEncoding inconsistency

Upstream thread: https://github.com/KhronosGroup/glTF-Sample-Models/pull/296
easiest repro for me is to load this, while debug console is opened:
https://cx20.github.io/gltf-test/examples/playcanvas/index.html?category=tutorialModels&model=TextureEncodingTest&scale=0.2&type=glTF-Binary
The spheres in left column are brighter than in the other columns, while they should be the same.
It does not happen each time this is reloaded.

managed to reproduce in the viewer too:
https://playcanvas.com/viewer?load=https://cx20.github.io/gltf-test/tutorialModels/TextureEncodingTest/glTF-Binary/TextureEncodingTest.glb
(Reload tab with browser cache disabled)

@cx20 We don't seem to be able to repro this any more. I suspect it has been fixed. Can you confirm with the gltf-test texture encoding test?

I tested it again with PlayCanvas v1.55.0.
This problem is no longer reproducible, so I think it's probably fixed.

Closing this as fixed.


Remove mass list ims optimisations

Note: as i described in https://github.com/robinschmid/mzmine3/pull/3, the mzxml import is broken by this. However, Robin volunteers to fix it! :D

@robinschmid , you have been volunteered! :)