Run engine-dmn through GPT-4 for an example of generated javadoc

This is for discussion. Not for merge.
@javahippie @Noordsestern Regarding https://forum.operaton.org/t/proposal-improving-javadocs/25/1
I took the first project I found, which helps running a code base through GPT for JavaDoc, to get some idea if LLM is any good. I found https://github.com/fynnfluegge/doc-comments-ai, which can be automated to go through a large code base. As such, it calls GPT for every function without existing JavaDoc. It could probably be patched to also generate something for classes and also replace existing docs for function.
The initial impression is the well expected "meh".

Costs are bearable. Probably less than ten euros for a code base of this size wit GPT-4. And it would take hours in time.
It does touch code. Unfortunately. I will try later, if I can revert all changes to code lines, because doc comment lines should be distinguishable.
As such, it also add noise with trivial comments for trivial function.

I didn't try out yet, if it introduced any syntax errors. At first, I should try to revert changes in code. If any of the generated docs look any good, it would be possible to try to iterate this by tweaking "the prompt" with a good example file for better results.

I think for review purposes it would be nice if there wouldn't be whitespace/formatting changes in the same commit. Can you control this?

I didn’t try very hard, but I could not get rid of the formatting changes with just “prompt engineering”. I assume that I can remove them automatically locally from the commit. I’ll maybe able to try it out later today.

Surprisingly, I have been unable to find a way with git to only stage and commit changes being part of a java doc comment (e.g. with a regex). I tried to automate interactive staging (git add -p) with minimal chunk, but it still leaks some unwanted changes.
Yet, for GPT-4-javadoc evaluation this might be good enough. So, let this be an example of, how GPT-generated docs could look. And with good ideas we could run a few iterations with modified LLM prompt.

Thanks for looking into this and investing your time! I think this can be helpful. Do you think there is a way to prevent the LLM from generating JavaDoc which is too trivial, e.g. for Getters an Setters? I think for most projects there is a sweet spot.

@javahippie Most probably. This has to be done in two phases anyway:

GPT phase, where one can “prompt engineer” = guess the right words to get LLM to produce better results

git phase, where one can drop all unwanted changes.

I am still looking for a way to automatically stage changes line by line with git. Its default chunking cannot always be splitter to narrow changes enough.
Would you think that engine-dmn is a good enough subset to iterate with? Or would you suggest something else?

By the way, I came across this project today: https://github.com/abinthomasonline/repo2txt With really large context windows coming to LLMs in the near future, it might be possible to provide the entire code as context.

@datakurre How to continue here? IMO the approach is promising, but I second @javahippie 's opinion. If you could suppress trivial Javadocs and carefully review the output then we should get a mergeable result.

@datakurre I tend to close the PR in a week when there is no action.

@kthoms I can close this. I used this as a platform for discussion. I'll open a new one, once I am able to apply the feedback.
I had not used LLM in this scale before, so this is also experiment to me. I wanted to see, how well they would work out of the box. As we see, the result was lukewarm.
A couple of open issues besides the quality in general:

Skip updating trivial methods. Thanks to the naming convention, this could probably be applied already when selecting, which code to pass to LLM and which not to.

Avoid changes outside docs. It does not seem to be trivial to instruct LLM to NOT touch indentation or code in general. I've also been trying to cope with this with git by simply not staging changes outside docs. But also this has challenges in that git does not allow line by line staging, but has its own chunking algorithm.


Bug: autoloader: global helpers are not loaded when bootstraped with phpunit

PHP Version
8.1
CodeIgniter4 Version
4.4.3
CodeIgniter4 Installation Method
Composer (using codeigniter4/appstarter)
Which operating systems have you tested for this bug?
Linux
Which server did you use?
apache
Database
MySQL 8.1
What happened?
I upgraded the CI Library to v1.0.0-beta.8 which changed how the helpers are loaded: "Helper autoloading has been changed to be done by CodeIgniter's autoloader instead of Composer." https://github.com/codeigniter4/shield/blob/develop/UPGRADING.md#version-100-beta7-to-100-beta8
Therefore I changed the autoloader.php config file to globaly load the "auth" and "setting" helper. After that many of my tests failed due to the helper not being loaded by the testing framework.
Steps to Reproduce

Create any custom helper file with a method "test_autoloader"
Add the helper to the $helper array in Config\Autoload.php
write a test which extends CIUnitTestCase
run function_exists('test_autoloader') - it will return false

Expected Output
when comparing the "normal" bootstrap.php to the bootstrap.php which is loaded on testing, it becomes obvious, that the Services::autoloader()->loadHelpers(); is missing and therefore the helper specified in Config\Autoload.php are not loaded.
In my case this leads to the shield lib failing because the function from the helper can not be found.
Anything else?
vendor/codeigniter4/framework/system/bootstrap.php:
...
require_once SYSTEMPATH . 'Config/Services.php';
require_once APPPATH . 'Config/Services.php';

// Initialize and register the loader with the SPL autoloader stack.
Services::autoloader()->initialize(new Autoload(), new Modules())->register();
Services::autoloader()->loadHelpers();

vendor/codeigniter4/framework/system/Test/bootstrap.php:
...
require_once SYSTEMPATH . 'Config/Services.php';
require_once APPPATH . 'Config/Services.php';

// Initialize and register the loader with the SPL autoloader stack.
Services::autoloader()->initialize(new Autoload(), new Modules())->register();

// Now load Composer's if it's available
if (is_file(COMPOSER_PATH)) {
    require_once COMPOSER_PATH;
}
...

let me clarify, I meant creating any helper, for example test_autoloader_helper.php which implements the method test_autoloader(). Add test_autoloader to the $helpers array in Config\Autoload.php. Therefore<PHONE_NUMBER>333

I sent PR #8275. Please review and approve if it okay.

Yes, this works, at least for my issues, thanks


Client Crash - Shield Projector Tier 4

Hello Play on a public FTB Inifinity Evolved Expert server, I decided to make a Shield Projector Tier 4 and used the Shape Card to make a dimension of 100 blocks out.  once i used a wrench it crash my client.  Now anybody entering my chunks will client crash.  The Error Message when it kicks me says: " Internal Exception: io.netty.codec,DecoderException: java.ioexception: packet was larger than I expected, found 60172 bytes extra whilst reading packet 53"
This is the Paste in the logs.

ffgjH2DM.txt

This is fixed in 4.20. What version are you on?

its the 2.2.2 ftb version which is rftools 4.13

yes so this is fixed. Upgrading would help but that requires the server to upgrade too. Otherwise they can do mcedit to try to remove the shield projector


Add ToastNotificationManger::Setting API

Implements the ToastNotificationManager::Setting API
Includes unit tests checking for the enabled scenario for unpackaged/packaged applications

:shipit:


feat: structure as cargo workspace

Fixes #328
Undoes #232
Forked from #330

[ ] TODO: Rebase onto main once it's merged
TO REVIEW: just look at the second commit

What
Add top-level Cargo.toml, with all project folders as part of this
workspace.
Why
We now encourage everyone to use Cargo workspaces when building Soroban
projects. Let's make it easy to copy-paste these examples into their
projects, rather than requiring special tooling
(like the old stellar contract init behavior) to update the contract's
Cargo.toml.
Known Limitations
Currently, there is a compilation error when attempting to run tests:
error[E0433]: failed to resolve: use of undeclared crate or module
`__get_allowance`
 --> token/src/contract.rs:67:12
  | 67 |     pub fn get_allowance(e: Env, from: Address, spender: Address)
-> Option<AllowanceValue> {
  |            ^^^^^^^^^^^^^ use of undeclared crate or module
`__get_allowance`
  | help: there is a crate or module with a similar name
  | 67 |     pub fn __allowance(e: Env, from: Address, spender: Address) ->
Option<AllowanceValue> {
  |            ~~~~~~~~~~~

As discussed in #328, we have decided not to implement this change.


Commands are not recognized.

I installed it using minpac
I have config.yml for db and the Lsp was working fine.
but the editor can't seem recognized any of the commands provided.
I tried using lighttiger2505/sqls.vim, commands was there but it doesn't seem support nvim-lsp.
OS
Windows 10
Terminal
Powershell v7.2.2
Editor
nvim v0.6.1
Database
postgresql 13.6

Are you calling this plugin's on_attach() function in your config?
require('lspconfig').sqls.setup{
    on_attach = function(client, bufnr)
        require('sqls').on_attach(client, bufnr)
    end
}

@nanotee thanks, you're on spot. close #18


Update apache-arrow script for new dependencies in 2.0.0

Companion to https://github.com/autobrew/homebrew-core/pull/31
See also https://github.com/r-windows/rtools-packages/pull/160

FYI I also had to add https://github.com/autobrew/scripts/commit/659df0305c0ab67151bac300245530c33bde1f82 to make it work.


Unable to declare a Vector-Field for a JsonModel

The Bug
I'm trying to declare a Field as a Vector-Field by setting the vector_options on the Field.
Pydantic is then forcing me to annotate the field with a proper type.
But with any possible type for a vector, I always get errors.
The one that I think should definitly work is list[float], but this results in
AttributeError: type object 'float' has no attribute '__origin__'.
Example
class Group(JsonModel):
    articles: List[Article]
    tender_text: str = Field(index=False)
    tender_embedding: list[float] = Field(
        index=True,
        vector_options=VectorFieldOptions(
            algorithm=VectorFieldOptions.ALGORITHM.FLAT,
            type=VectorFieldOptions.TYPE.FLOAT32,
            dimension=3,
            distance_metric=VectorFieldOptions.DISTANCE_METRIC.COSINE
        ))

No Documentation for Vector-Fields?
There seems to be no documentation about this feature.
I just found a merge-request which describes some features a bit, but there is no example, test or any documentation about this.

I'm running into the same thing. It looks like they want you to type it as tender_embedding: list[list[float]] but even with that, I'm having issues getting any reasonable results when searching.

@Pwuts or @chayim could one of you provide a working example of how to use the vector field? Or potentially just some tests that can be referenced?


[Database] ClickHouse

Details

TCP
Default port 9000
ClickHouse protocol does not have formal specification yet, but it can be reverse-engineered from ClickHouse source code (starting around here) and/or by intercepting and analyzing TCP traffic: https://github.com/ClickHouse/ClickHouse/tree/master/src/Client

Prior Art

https://github.com/suharev7/clickhouse-rs
https://github.com/ClickHouse/clickhouse-go

Here is the formal specification of the ClickHouse protocol:
https://clickhouse.com/docs/en/native-protocol
I also want this feature in sqlx!


Attempting to change the web GUI TCP port results in the following text on a white screen upon clicking "Save"

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[ ] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[ ] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
Attempting to change the web GUI TCP port results in the following text on a white screen upon clicking "Save":
Fatal error: Uncaught TypeError: Cannot access offset of type string on string in /usr/local/www/system_advanced_admin.php:175 Stack trace: #0 {main} thrown in /usr/local/www/system_advanced_admin.php on line 175
Tip: to validate your setup was working with the previous version, use opnsense-revert (https://docs.opnsense.org/manual/opnsense_tools.html#opnsense-revert)
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Wanted to change the GUI TCP port
Describe alternatives you considered
A clear and concise description of any alternative solutions or workaround you considered.
Screenshots
If applicable, add screenshots to help explain your problem.
Relevant log files
If applicable, information from log files supporting your claim.
Additional context
Add any other context about the problem here.
Environment
Software version used and hardware type if relevant, e.g.:
OPNsense 24.7.a_184-amd64
FreeBSD 13.2-RELEASE-p10
OpenSSL 3.0.13
QEMU Virtual CPU version 2.5+ (4 cores, 4 threads)

root@OPNsense:~ # awk 'NR < 170{next}1;NR==180{exit}' /usr/local/www/system_advanced_admin.php
        $config['system']['webgui']['port'] = $pconfig['webguiport'];
        $config['system']['webgui']['ssl-certref'] = $pconfig['ssl-certref'];
        $config['system']['webgui']['ssl-ciphers'] = $newciphers;
        $config['system']['webgui']['interfaces'] = $newinterfaces;
        $config['system']['webgui']['compression'] = $pconfig['compression'];
        $config['syslog']['nologlighttpd'] = empty($pconfig['loglighttpd']);

        if (!empty($pconfig['deployment'])) {
            $config['system']['deployment'] = $pconfig['deployment'];
        } elseif (isset($config['system']['deployment'])) {
            unset($config['system']['deployment']);

It's a wild guess, but can you try?
opnsense-patch 3111a05

Nice guess! That fixed it.
root@OPNsense:~ # opnsense-patch 022ab7a
Fetched 022ab7a via https://github.com/opnsense/core
Hmm...  Looks like a unified diff to me...
The text leading up to this was:
--------------------------
|From 022ab7ad75a994eb92d8e923ba904e033c593392 Mon Sep 17 00:00:00 2001
|From: Ad Schellevis<EMAIL_ADDRESS>|Date: Wed, 13 Mar 2024 08:42:45 +0100
|Subject: [PATCH] System: Settings: Administration - possible issue when syslog
| container doesn't exist in the config. closes
| https://github.com/opnsense/core/issues/7323
|
|---
| src/www/system_advanced_admin.php | 7 ++++---
| 1 file changed, 4 insertions(+), 3 deletions(-)
|
|diff --git a/src/www/system_advanced_admin.php b/src/www/system_advanced_admin.php
|index b12eb8f6c4..ae14a3c286 100644
|--- a/src/www/system_advanced_admin.php
|+++ b/src/www/system_advanced_admin.php
--------------------------
Patching file www/system_advanced_admin.php using Plan A...
Hunk #1 succeeded at 79.
Hunk #2 succeeded at 152.
Hunk #3 succeeded at 165.
Hunk #4 succeeded at 173.
done
All patches have been applied successfully.  Have a nice day.


Go test -run BuildImage_Peer panics and fails when Consensus plugin set to &amp&amp&

Description:
Just ad-hoc testing while executing my "Default to noops when we don't understand the plugin name" test by putting in a malformed XML entity (Ampersand), and the entire build process ground to a halt.
Steps:

Open openchain.yaml
Edit consensus plugin to "&amp&amp&"
Save openchain.yaml
Attempt to build obc-peer docker image

Results:
vagrant@vagrant-ubuntu-trusty-64:/opt/gopath/src/github.com/openblockchain/obc-peer/openchain/container$ go test -run BuildImage_Peer
panic: Fatal error config file: While parsing config: yaml: line 151: did not find expected alphabetic or numeric character

goroutine 1 [running]:
panic(0x98a6e0, 0xc82015eb00)
        /opt/go/src/runtime/panic.go:464 +0x3e6
github.com/openblockchain/obc-peer/openchain/container.SetupTestConfig()
        /opt/gopath/src/github.com/openblockchain/obc-peer/openchain/container/config.go:70 +0x2c1
github.com/openblockchain/obc-peer/openchain/container.TestMain(0xc820125ee8)
        /opt/gopath/src/github.com/openblockchain/obc-peer/openchain/container/vm_test.go:35 +0x18
main.main()
        github.com/openblockchain/obc-peer/openchain/container/_test/_testmain.go:74 +0x114
exit status 2
FAIL    github.com/openblockchain/obc-peer/openchain/container  0.046s

Expected results:
System should build, not understanding the parameter for consensus plugin should result in Noops being used.

I disagree violently.  Incorrect settings should not be ignored.

But that's what we state in the .yaml file, isn't it?  That an unrecognized plugin results in Noops being used?  This exposes a bug in that if there were to be a future public OR private consensus plugin with symbols in the name, we run the risk that such a customer not even be able to build because there's a symbol in the name.  I know this test seems pedantic, just trying to make the system as bullet-proof as possible, IBM's customers deserve that.

isn't this a Viper/YAML issue ? I'll bet we fail if you put that string anywhere in openchain.yaml.

What about localization in other languages that might result in a non alpha-numeric characters?  Kanji for example?  Does viper not work with other languages?
If that's the case it's at least a good thing to be aware of... is there a flag for KB option?

everything in .yaml files is not text but  keywords which we restrict to specific strings which are never translated.
BuildImage_peer should fail a bit more gracefully though.

Just reproduced with https://github.com/hyperledger/fabric/commit/ee024d56f1a42b9eda2902f45099695608ca7634

basically, further investigation reveals this is working as designed... could be more graceful means of exiting, I agree but closing.


Add getMiddleware function to Route class

The getMiddleware function is missing and reported here: https://github.com/flipboxstudio/lumen-generator/issues/25
Also I think you have to be able to query the middlewares.
If the function is not necessary and you can query the middlewares with another method, please write it in the comments.

I noticed that this issue has not been released yet and I would like to submit a suggestion. Within the Laravel own Route class there is a method called gatherMiddleware that collects the middleware of the route. Shouldn't the method at least be called the same, since it is used in the same manner?


feat: supported content parts

Resolves #129

/deploy-review

/deploy-review

/deploy-review

/deploy-review


Build for arm64 too

We can build for ARM64 similarly to how we build for x64, just need to pass the architecture info here and there.
Helps fix https://github.com/orgs/community/discussions/53407 and https://github.com/actions/runner/issues/801. (Doesn't actually fix, because changes will also be required in the Actions Worker, but this is a good start.)
There are two to-dos:

Set up an ARM64 runner for testing the ARM64 binary (comment left in matrix).
Ensure uploading multiple artifacts like this works (not sure, also this is an outdated upload-artifact action).

I have build Node manually on my computer like this (M1 on ARM alpine). It worked!

@actions/actions-runtime, please consider this.  We need arm64 support on Alpine.  Thanks!


File transfer enhancements

Take into account both file size and connection speed to better estimate timeouts when transferring files to and from a target.

This would resolve https://github.com/ARM-software/workload-automation/issues/470

Implemented connection polling to check whether a transfer is still active instead of relying on timeouts with https://github.com/ARM-software/devlib/pull/504


feat: add replacement of _.uniq (#23)

Ping @onebytegone

@onebytegone This is ready for your re-review

LGTM. Thanks!


Plans on implementing Nuxt-fontaine?

Request Description
Nuxt Fontaine is a cool library that'll help with page load speeds by default by doing some magic with font metrics. It'll be cool if this library has it by default.
Alternative Solutions
None.
Additional Details
No response

Hi, thanks for the suggestion.
This module already uses methods for best performance when it comes to font loading.
The module automatically detects the configuration, depending on the strategy whether it is local or external, and adds preconnect and preload link tags in the head section with minified inline styles for the @font-face rules.
This is all done by the minimal footprint so there is no additional code bloat. The module has only one dependency (for styling console logs) which is also under 1kb.
I just did some quick tests and here are the results:

Test environment

Default nuxt-app project
nuxt-font-loader with the local strategy

// nuxt.config.ts

{
  modules: ['nuxt-font-loader'],

  fontLoader: {
    local: [
      {
        src: '/fonts/AspektaVF.woff2',
        family: 'Aspekta Variable',
        weight: '100 900'
      }
    ]
  }
}

Run generate and preview commands

So, in terms of page load speed, that's pretty much it.
As for your suggestion, I don't plan to implement it right now, I want to keep the module simple.
But just out of curiosity, have you had any performance or page load issues with this module?


Enable users notifications on approaching events or new events creation.

Enable users to get notified if they have subscribed to events.

Follows https://github.com/PkFokam-Alumni-Network/CommunityBackendService/issues/25

Should discuss with @keyangueleslie and @atakoutene what is better to use between mail notifications and SMS notifications

Okry. 👍👍
On Sat, Dec 14, 2024 at 1:48 PM Warren Fongang @.***>
wrote:

Enable users to get notified if they have subscribed to events.

Follows #25
https://github.com/PkFokam-Alumni-Network/CommunityBackendService/issues/25

Should discuss with @keyangueleslie https://github.com/keyangueleslie
and @atakoutene https://github.com/atakoutene what is better to use
between mail notifications and SMS notifications
—
Reply to this email directly, view it on GitHub
https://github.com/PkFokam-Alumni-Network/CommunityBackendService/issues/26,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3HNHGTWYVDYN753Q3XRPZT2FR4OFAVCNFSM6AAAAABTTWZCCSVHI2DSMVQWIX3LMV43ASLTON2WKOZSG42DAMBRGAZDCNY
.
You are receiving this because you were mentioned.Message ID:
@.***>


Bug Report: The content inside general message in resources page is displaying as a link

Reproduction steps
While scrolling till the ending of resources page the text inside general message is displayed as a link.
Expected behavior
It should be as a simple text in the form of paragraph.
Actual Behavior
Displaying the general message as paragraph or text
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

I am a GSSOC 22 Participant please assign this issue to me

Thanks, I have started working on it


feat: Make UI follow track colors

Fixes #1405
Changes:

Add track color attribute in database
Add functionality to color track item globally and locally
Lazy load the color of track on UI

Screenshots for the change:

@the-dagger Please review


Fix bouncing animations on nav link hover

Fix navigation button animations and padding

Hovering cursor over the top edge of the button caused a flickering
animation.  This commit tweaks the CSS to fix that, and also adds some
horizontal padding to the button text to make it prettier.

Local testing shows this doesn't seem to fix the states issue, simply shift it. but also strips the text styling from the buttons.
https://file.house/dVs2.mp4


Added backend, database to Docker Compose, now we have 2 separate Compose files for staging & production

A proxy már fent van, működik, de a projekttől teljesen független: https://gitlab.com/webmenedzser/hosting-with-docker
A dokumentációja szerint kell expose-olni: https://github.com/nginx-proxy/nginx-proxy
"The containers being proxied must expose the port to be proxied, either by using the EXPOSE directive in their Dockerfile or by using the --expose flag to docker run or docker create and be in the same network. By default, if you don't pass the --net flag when your nginx-proxy container is created, it will only be attached to the default bridge network. This means that it will not be able to connect to containers on networks other than bridge."


added test cases for get_file_content() func from utils

This fixes #21

@hpatel292-seneca  Very good, Thank Harshil


Update SDK to the 12.0.0 version

@ifaouibadi is taking some time to research on deeplinks with this version, then we will upgrade the dex-ui

Lets postpone this for now. Until @bogdan-manole verifies the call data issue

just for tracking this the calldata topic -> https://github.com/aeternity/aepp-calldata-js/issues/155

Lets wait until the SDK is updated to the new calldata lib

Lets move to 13 instead :)


[New Sheet] Mythic D6

Changes / Comments

Sheet submitted on behalf of publisher, Khepera Publishing
the exact name of the game is "Mythic D6", with a space
Game system is it's own, so should get it's own subcategory
Rules available on DriveTruRPG: https://www.drivethrurpg.com/product/243769/MYTHIC-D6-Revised--Expanded
sheet have been created with then new Character Sheet Enhancement framework.

Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[x] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

I clearly stated the sheet was made on the behalf of the publisher, but it's not mentioned among the publisher-created sheets.
Do you need actual proof or can you take my word on this, having already helped/created 2-3 publisher sheets in the past? (Stargate RPG & Hc Svnt Dracones 2E)


Make embed take up more of iframe height

Now that the list view is shown side by side with the map, the map embed isn't really taking up the full height available in an iframe embed on desktop.
Bumping up the embed body height to 70vh to fill up more of the height on large screens. This translates to a min iframe height of ~650px before scroll bars are shown on the embed. Takes up most of the iframe height now at more usable large iframe heights.

Note that pushing the iframe height past 750px starts making the white space gap noticeable again, but achieving true responsiveness is gonna more a more in depth change.

The issue is that they don't all give us that much vertical real estate and we want the footer to show no matter how small our box is... @linderr can you think of an embed example that has the shortest window for our iframe that we can test this on?

Hey sorry, late to the game --- try: https://northernontarioppe.ca/pages/initiatives

@sunnymui @ngottlieb I don't think I see a huge difference on this collaborator's site? Just tested on Inspector

The height of the iframe script we share now is 800px.
I couldn't see a difference like your screen shots on https://northernontarioppe.ca/pages/initiatives @sunnymui  ?
I think we will just have to ask each collaborator change the iframe height to 1000px to achieve the balanced header to map look??

^1000 px iframe height

Just to make the exact thing I'm talking about more clear:

The blue highlighted area is the area of the actual iframe element i.e. the 800px, but you can tell the actual document content in the iframe only takes up some of that space, leaving extra white space below it.
You'll notice the large whitespace gap to the next element on that page. That's basically where the list view used to be positioned. Technically, I could just put overflow: hidden on the iframe element to hide that space, but I'd rather still have it scroll on overflow, just in case the content somehow gets larger than the frame. That way a user can still access the content within.
At 70vh body height, the minimum height of the iframe element before scrollbars are shown is 650px. On GetUsPPE, I have that iframe element at 700px. If you want to enforce a larger minimum iframe embed height, you can increase vh to match, I just put 70vh to be conservative as probably the smallest you'd want that embedded map and have it still be usable.
On the linked example for northontarioppe, that one actually has the issue exaggerated even more because it's using the mobile media query height, so it's only 35vh. Probably need to toggle a class when the list view is hidden so it can show a greater height for the map.

@ngottlieb @sunnymui I found an even more exaggerated example to test on 😄
https://coronacourier.nyc/#/resources enjoy

Lol a 300 px height on that iframe is just asking for a bad time.
You'd have to have a height media query or window.innerHeight detection that applies a bunch of styles shrinking or hiding elements, significantly changing the layout to make the map halfway usable. Maybe that can be a future upgrade.
Increasing the embed body height to 70vh in this extreme case only makes a slight difference in the visibility of the bottom bar w/ edit map button--both require scrolling down to see the text.

Hahah I don’t even know why this is happening.. checked outbound emails, team definitely linked them 800px, not sure what happened on partner’s end - miraculous part is this page drives traffic but it’s probably reverse UX/UI psychology when embed is basically unusable & ppl just come to the main so.. 👌🏻


Decorators not allowed in JS even with 'experimentalDecorators' set

Create a new folder.
Add a tsconfig.json/jsconfig.json.
Add "experimentalDecorators": true.
Create a new .js file that has a decorated class.

Expected: No error.
Actual: Error that you need to add "experimentalDecorators": true.

Hmmm, isn't this exactly what https://github.com/Microsoft/TypeScript/pull/6881 fixed? Maybe a regression (though hard to say, as you didn't mention what build/branch you are using :-p ).

p.s. Just tried it with a build from a couple days ago in master, and it works fine with the below tsconfig.json. It gives the error if I change the setting to false (as expected). Please provide a repro (and details on what branch/build) if you are sure you are seeing issues.
{
    "compilerOptions": {
        "experimentalDecorators": true,
        "allowJs": true
    }
}

i see this on latest too.

@mhegazy you see the behavior I'm reporting, or are you also saying it works for you?
@jramsay saw the same issue a few days ago as well. I used the nightly, so Version 1.9.0-dev.20160525-1.0.

Cannot repro this either. Talked to @mhegazy, turns out he had an invalid tsconfig when he saw this. Close until a repro can be found.

I'm working on a TS project that's using gulp (+ aurelia-cli). Even with experimentalDecorators: true I still get red squigglies in VSCode.
{
  "compileOnSave": false,
  "compilerOptions": {
    "sourceMap": true,
    "target": "es5",
    "module": "amd",
    "declaration": false,
    "noImplicitAny": false,
    "removeComments": true,
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "moduleResolution": "node",
    "lib": ["es2015", "dom"]
  },
  "exclude": [
    "node_modules"
  ],
  "filesGlob": [
    "./src/**/*.ts",
    "./test/**/*.ts",
    "./typings/index.d.ts",
    "./custom_typings/**/*.d.ts"
  ],
  "atom": {
    "rewriteTsconfig": false
  }
}

I ran into this when I open a root folder that contains TS project for client and TS project for server (File-> Open Folder ..  /root ). The root has no tsconfig file. VSCode does not respect the tsconfig in each respective project folder ( /root/server/tsconfig.json  and  /root/client/tsconfig.json ). It is expecting a tsconfig to be in the root of the folder opened in VSCode ( /root/tsconfig.json ).
When I open another instance of VSCode and open the  /root/client  or  /root/server  folder the decorator warning goes away, as it should.
Is this by design? I'm not sure i'd expect it to find the nearest tsconfig.json in local folder or above.
I hope I'm making sense :-P

@akanieski it shouldn't be the case, the tsconfig file in the same folder should be respected regardless where you open as root in VSCode. Can you provide a small sample of the issue for reproing? I tried on my machine but I didn't see it happen. Also with your tsconfig.json content, I didn't get the error for the decorators. What version of VSCode / TypeScript are you using?

Sure. Checkout the following repo:
https://github.com/akanieski/decorator-issue

TS Version 2.1.0-dev.20160724

That doesn't seem to be the case for me:

Did you do anything else other than open the files, like switch git branches etc.?
I am using the same TS version.

Nope. Let me try from another pc. Perhaps I have something interfering.

Not sure if this helps but.. if I remove this line:
"lib": ["es2015", "dom"]
reopen the file and wait a second the error goes away.

Having this problem right now. VSCode 1.5.3 with Typescript 2.0.3. Running in a Ubuntu 16.04 box. Tried all the suggestions in this thread.
My tsconfig.json
{
  "compilerOptions": {
    "declaration": false,
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "lib": ["es2015", "dom"],
    "mapRoot": "./",
    "module": "es2015",
    "moduleResolution": "node",
    "outDir": "../dist/out-tsc",
    "sourceMap": true,
    "target": "es5",
    "typeRoots": [
      "../node_modules/@types"
    ]
  }
}

I had this problem in one project, but not in another, both using TS 2.0.3 and VS Code 1.5.3. The tsconfig that wasn't working (ignoring filesGlob and exclude):
{   "compilerOptions": {     "target": "es5",     "module": "commonjs",     "emitDecoratorMetadata": true,     "experimentalDecorators": true,     "sourceMap": true,     "moduleResolution": "node",     "allowJs": true   } }
And the one that does:
  "compileOnSave": false,   "compilerOptions": {     "sourceMap": true,     "target": "es5",     "module": "amd",     "declaration": false,     "noImplicitAny": false,     "removeComments": true,     "emitDecoratorMetadata": true,     "experimentalDecorators": true,     "moduleResolution": "node",     "lib": ["es2015", "dom"]   },
I copied over the one that worked into the other project, and they both work. I tried swapping out every single option looking for a hint has to what might stop the experimentalDecorators flag being honoured, but with no luck. So I have it working, but I have no idea why.

You have allowJs to true in the first one. This should be giving you an error when you try to compile that any .js files will overwrite themselves. You'd need to either specify "noEmit": true or "outDir": "../someOtherFolderNotInTheCompilation" for this to be a valid configuration.

@billti Thank. It's work for me.
This is tsconfig.json in my React Native project:
{
    "compilerOptions": {
        "experimentalDecorators": true,
        "allowJs": true
    }
}

In atom I tried @anhtuank7c suggestion and replaced my tsconfig with
''''
{
"compilerOptions": {
"experimentalDecorators": true,
"allowJs": true
}
}
''''
which fixed the issue, then I tried reverting to my original config that I need that was causing the issue and the issue no longer occurs. Great!

Confirmed the two compilerOptions settings resolved the warnings.  After adding settings back one at a time, It appears the conflict was the following:
  "include": [
    "*.ts",
    "src/**/*.ts"
  ],

I ended up replacing the include with exclude instead.
  "exclude": [
    "node_modules"
  ]

Someone already said this but when I added  "lib": ["es2015", "dom"] to the top level of the config file it started working / something stopped working.

jsconfig.json
{
"compilerOptions": {
"experimentalDecorators": true
}
}
When using VS Code in the root directory of your project and the warnings go away.
In my case I'm learning React and there aren't that many TypeScript tutorial versions
around.

Same issue here. I've tried the solutions proposed above, but the warning is still there (VSCode 1.21.1 under Windows).
My jsconfig.json file:
{
	"compilerOptions": {
		"experimentalDecorators": true
	},
	"exclude": [
		"node_modules",
		"**/node_modules/*"
	]
}

Same thing with a tsconfig.json file:
{
	"compilerOptions": {
		"experimentalDecorators": true,
		"allowJs": true
	},
	"exclude": [
		"node_modules",
		"**/node_modules/*"
	]
}

I followed the  solutions but I'm still getting a Declaration expected warning from vscode.
Here is a snippet from the compilerOptions of my tsconfig.json
 "compilerOptions": {     "noImplicitAny": true,     "removeComments": true,     "preserveConstEnums": true,     "experimentalDecorators": true,     "emitDecoratorMetadata": true,     "allowJs": true,


BST Pet Skill Names Incorrect (GetPetAction)

Testing out comparing the pet action, and the Name field is returning invalid values.
For example with Headbreaker Ken (dragonfly pet)
"Cursed Sphere" returns "Earth Breath"
"Venom" returns "Aqua Breath"
"Somersault" returns "Tidal Wave"

Fixed in 1.03.  Thank you!


Reduce memory usage of ILTypeDefs, ILMethodDefs and ILAttributes

This is a companion to #660.
As indicated by the memory analysis http://fsharp.github.io/2015/09/29/fsharp-compiler-guide.html, the lazy-initialization implementation of ILTypeDefs, ILMethodDefs and ILAttributes is a consistent memory hog, especially for F# solutions with many imported assemblies, but also for code generation.  The lazy thunks and their closures just involve an unnecessary number of objects.
This implements these in a more storage-efficient way, nulling the thunks when things are done.

Don,
did you repeat the analysis with these changes and did you see the expected benefit?
Thanks
Kevin

I need to define a stable baseline and set of exact, replicable scenarios. The baseline in the analysis was VS2013 but nothing has changed with this code since the. It would be awesome if we could generate a devenv.exe or fsi.exe managed memory dump+report automatically in appveyor, such a report would likely be stable even on VMs

This is good to go in

merged into vs2015


Fl legislators special election

Added missing FL legislators after the special election in 2019. Related to issue #185.

Pull Request Test Coverage Report for Build 548

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

Totals

Change from base Build 537:
0.0%

Covered Lines:
969

Relevant Lines:
1511

💛  - Coveralls


Multisig. Create multisig. Account selection should show possible value when in focus

Description
Example:
https://github.com/user-attachments/assets/ffc7319a-e5b6-4614-af60-30f050bd5cd1
Expected result:
Show possible values when field in focus
Environment
https://github.com/novasamatech/nova-spektr/commit/6ca515af94bd4d44ea25ada390bcc01455cd4bc4

This is an issue I had identified in the past, and the clean way to fix this is to use a new property in the headlessUI component. See https://github.com/novasamatech/nova-spektr/pull/1931
Now since this is a much larger refactoring we agree not to deal with it now.
This behavior would be much much better in many other places of Spektr, such as the transfer component, where you could see and select the addresses from your address book for the recipient.
My message a while ago:

Status verification: ✅


Called Wallet.Sign() with this message on WebGL build, failed and got this error: invalid hexlify error

The string i want to sign:
string domain = "myDomain";
string chainId = "1";
string message = "Sign in";
string uri = "myUri";

string data = "{\"domain\":\"" + domain + "\",\"chainId\":" + chainId + ",\"message\":\"" + message + "\",\"wallet\":\"" + walletAddress + "\",\"nonce\":\"" + nonce + "\",\"uri\":\"" + uri + "\"}";
return await sdk.Wallet.Sign(data);

You can try copying this code for SIWE https://github.com/thirdweb-dev/unity-sdk/blob/dafafeaaf343d3de6c934212c0fe4bac662bc537/Assets/Thirdweb/Core/Scripts/Wallet.cs#L136-L149
Or use v5 Utils.GenerateSIWE

I'm encountering an invalid hexify error when including the chainId in the JSON data string, but the error disappears when I exclude it. Is there a specific reason for this behavior?
Because i need to sign using that format.


Don't translate function prototype in built-in doc

The string (value) is part of the function prototype for property setters in the built-in doc:
set_position(value)

It's currently being translated, but GDScript doesn't support unicode identifiers yet. So it should not be translated to
set_position(值)

I understand the rationale, but this string is not meant as an actual variable you would use in your code, but as a placeholder for where you put the value you pass to the function.
If we don't translate it, users need to know what "value" means. I'm fine with it but it's a tradeoff and I'm not sure the actual issue with having translated is really a problem. Did you witness situations where users tried to write set_position(值) in their code? (set_position(value) would fail compiling too unless they actually define var value = something beforehand).

Did you witness situations where users tried to write set_position(值) in their code?

No. But quite an amount of users in Chinese Godot community use a third-party build that allows unicode identifiers, the usage is valid & not uncommon there.
I thought value is just like a regular parameter in the doc. But yeah, regular parameters are explained in the function description, while this value is not explained anywhere.


operator/condition: add initial list of condition names

Collect all conditions that are spread across different controller loops and concentrate them in one place with proper godoc.

@sttts thanks! updated godoc.

/approve
/lgtm


Make list more comprehensive

The list is missing a few file extensions, e.g.:

.cur
.webp
.jxr / .hdp / .wdp
...

See also: http://en.wikipedia.org/wiki/Image_file_formats

Didn't knew I missed out that much ones, sorry :) Will fix that.

you are all faggots!!!!

@alrra @sindresorhus @agcolom @arthurvr ARTHUR UR A FAGGOOOTTTTTT

@sindresorhus @arthurvr @alrra Here you faggs:


qa_openstack: Adjust VG setup

Let openstack-loopback-lvm from openstack-quickstart do the
LVM volume group setup.

Depends on  https://github.com/SUSE-Cloud/openstack-quickstart/pull/56

+1

not sure I get that... why not create two volume groups on dev/vdb  and just add vdc as an extra pv to the vg if it exists?

not sure I get that... why not create two volume groups on dev/vdb and just add vdc as an extra pv to the vg if it exists?

@dirkmueller I think you can't create multiple VGs on a single PV. Can you?

accidentally closed. pardon.

+1


error on training APC using custom dataset

The training code is as follows:
from pyabsa.functional import Trainer
from pyabsa.functional import APCConfigManager
from pyabsa.functional import ABSADatasetList
from pyabsa.functional import APCModelList

config = APCConfigManager.get_apc_config_chinese()
config.evaluate_begin = 4
config.dropout = 0.5
config.l2reg = 1e-8
config.model = APCModelList.FAST_LCF_BERT
# config.spacy_model = 'zh_core_web_sm'
# chinese_sets = ABSADatasetList.Chinese
# chinese_sets = ABSADatasetList.Chinese
chinese_sets = '/datasets/apc_datasets/666.hotel'
sent_classifier = Trainer(config=config,  # set config=None to use default model
                          dataset=chinese_sets,  # train set and test set will be automatically detected
                          checkpoint_save_mode=1,
                          auto_device=True  # automatic choose CUDA or CPU
                          ).load_trained_model()

the dataset seems to load ok:
Loading /datasets/apc_datasets/666.hotel dataset from: https://github.com/yangheng95/ABSADatasets
Some weights of the model checkpoint at bert-base-chinese were not used when initializing BertModel: ['cls.seq_relationship.bias', 'cls.predictions.transform.dense.weight', 'cls.predictions.transform.LayerNorm.bias', 'cls.predictions.decoder.weight', 'cls.predictions.bias', 'cls.seq_relationship.weight', 'cls.predictions.transform.dense.bias', 'cls.predictions.transform.LayerNorm.weight']
- This IS expected if you are initializing BertModel from the checkpoint of a model trained on another task or with another architecture (e.g. initializing a BertForSequenceClassification model from a BertForPreTraining model).
- This IS NOT expected if you are initializing BertModel from the checkpoint of a model that you expect to be exactly identical (initializing a BertForSequenceClassification model from a BertForSequenceClassification model).
loading: ABSADatasets/datasets/apc_datasets/666.hotel/hotel.train.txt
  0%|          | 0/8383 [00:00<?, ?it/s, preparing dataloader...]Using bos_token, but it is not set yet.
Using eos_token, but it is not set yet.
100%|██████████| 8383/8383 [00:04<00:00, 1980.14it/s, preparing dataloader...]
Dataset Label Details: {'Positive': 7213, 'Negative': 1124, 'Neutral': 25, 'Sum': 8362}

and then the following error:
Output exceeds the [size limit](command:workbench.action.openSettings?[). Open the full output data [in a text editor](command:workbench.action.openLargeOutput?f79ace7b-3847-4bda-b2ba-30cefbc72f5f)
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
/Users/harrywang/sandbox/hotel-service-bot/apc-train-chinese/train.ipynb Cell 2 in <cell line: 10>()
      [6](vscode-notebook-cell:/Users/harrywang/sandbox/hotel-service-bot/apc-train-chinese/train.ipynb#W1sZmlsZQ%3D%3D?line=5) # config.spacy_model = 'zh_core_web_sm'
      [7](vscode-notebook-cell:/Users/harrywang/sandbox/hotel-service-bot/apc-train-chinese/train.ipynb#W1sZmlsZQ%3D%3D?line=6) # chinese_sets = ABSADatasetList.Chinese
      [8](vscode-notebook-cell:/Users/harrywang/sandbox/hotel-service-bot/apc-train-chinese/train.ipynb#W1sZmlsZQ%3D%3D?line=7) # chinese_sets = ABSADatasetList.Chinese
      [9](vscode-notebook-cell:/Users/harrywang/sandbox/hotel-service-bot/apc-train-chinese/train.ipynb#W1sZmlsZQ%3D%3D?line=8) chinese_sets = '/datasets/apc_datasets/666.hotel'
---> [10](vscode-notebook-cell:/Users/harrywang/sandbox/hotel-service-bot/apc-train-chinese/train.ipynb#W1sZmlsZQ%3D%3D?line=9) sent_classifier = Trainer(config=config,  # set config=None to use default model
     [11](vscode-notebook-cell:/Users/harrywang/sandbox/hotel-service-bot/apc-train-chinese/train.ipynb#W1sZmlsZQ%3D%3D?line=10)                           dataset=chinese_sets,  # train set and test set will be automatically detected
     [12](vscode-notebook-cell:/Users/harrywang/sandbox/hotel-service-bot/apc-train-chinese/train.ipynb#W1sZmlsZQ%3D%3D?line=11)                           checkpoint_save_mode=1,
     [13](vscode-notebook-cell:/Users/harrywang/sandbox/hotel-service-bot/apc-train-chinese/train.ipynb#W1sZmlsZQ%3D%3D?line=12)                           auto_device=True  # automatic choose CUDA or CPU
     [14](vscode-notebook-cell:/Users/harrywang/sandbox/hotel-service-bot/apc-train-chinese/train.ipynb#W1sZmlsZQ%3D%3D?line=13)                           ).load_trained_model()

File ~/sandbox/hotel-service-bot/venv/lib/python3.8/site-packages/pyabsa/functional/trainer/trainer.py:149, in Trainer.__init__(self, config, dataset, from_checkpoint, checkpoint_save_mode, auto_device, path_to_save, load_aug)
    145     config.model_path_to_save = None
    147 self.inference_model = None
--> 149 self._run()

File ~/sandbox/hotel-service-bot/venv/lib/python3.8/site-packages/pyabsa/functional/trainer/trainer.py:164, in Trainer._run(self)
    162 self.config.seed = s
    163 if self.checkpoint_save_mode:
--> 164     model_path.append(self.train_func(self.config, self.from_checkpoint, self.logger))
    165 else:
    166     # always return the last trained model if dont save trained model
    167     model = self.model_class(model_arg=self.train_func(self.config, self.from_checkpoint, self.logger))
...
    318         continue
    319     target[direct + '_' + data_item] = source[data_item]
--> 320 target[direct + '_dist'] = int(abs(np.average(list(source['aspect_position'])) - np.average(list(target['aspect_position']))))

ValueError: cannot convert float NaN to integer

it seems to be about aspect_position being NaN - but not sure what is wrong. Any help?
Thanks,
Harry

You need to report the error example to help locate the problem

Or you can share your dataset to locate the problem

You need to report the error example to help locate the problem

I don't understand what you mean - I posted the error msg - I cannot share the data yet.

I mean you need to find which example cause this problem, as you know there is no similar errors in this repo.

i.e., debug to find which data example raise this error

I found the data example to cause the error:

the error was fixed after I changed it to:


Add double tap to zoom in Camera; update Gesture API

Closes #355 by adding double tap to zoom.
Also updates code to use the newest react-native-gesture-handler API with Gesture & GestureDetector. This makes code cleaner to read & makes it easier to support multiple gestures without needing multiple components.

Added animation on double tap.


Round-trip failure: ser -> de transforms Float(1.0) into Integer(1)

This example program probably ought to exit successfully
fn main() {
    let input = "1.0";
    let v: ron::Value = ron::from_str(input).unwrap();
    println!("{:?}", v);
    let ser = ron::to_string(&v).unwrap();
    println!("{:?}", ser);
    let roundtrip = ron::from_str(&ser).unwrap();
    println!("{:?}", roundtrip);
    assert_eq!(v, roundtrip);
}

But it doesn't:
Number(Float(Float(1.0)))
"1"
Number(Integer(1))
thread 'main' panicked at 'assertion failed: `(left == right)`
  left: `Number(Float(Float(1.0)))`,
 right: `Number(Integer(1))`', src/main.rs:9:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

This seems plausible as a ser/de behavior if RON only has a number type, but if that's the case ron::Value::Number should depend only on the numeric value not the discriminant.

I'm not quite sure how this can be fixed in a nice way. If you just have a specific usecase where the roundtrip should work, I'd suggest using
ron::ser::to_string_pretty(&v, ron::ser::PrettyConfig::new().decimal_floats(true))

Even though the deserialized Value is an Integer in this case, it still deserializes perfectly into an f64 (handled internally by serde's Deserialize impl for f64).
The only thing that is really missing is describe how two structurally different Numbers can deserialize into the same value. This could be done, e.g. like so:
impl PartialEq for Number {
    fn eq(&self, other: &Self) -> bool {
        use serde::de::IntoDeserializer;

        match (self, other) {
            (Self::Integer(a), Self::Integer(b)) => a.eq(b),
            (Self::Integer(a), Self::Float(b)) => {
                let af: Result<f64, serde::de::value::Error> = f64::deserialize(a.into_deserializer());

                af.map_or(false, |a| Float(a).eq(b))
            },
            (Self::Float(a), Self::Integer(b)) => {
                let bf: Result<f64, serde::de::value::Error> = f64::deserialize(b.into_deserializer());

                bf.map_or(false, |b| Float(b).eq(a))
            },
            (Self::Float(a), Self::Float(b)) => a.eq(b),
        }
    }
}

However, this would then also require adjusting the Hash, PartialOrd and Ord impls and get quite confusing.
Another quick and dirty fix could be to simply enable decimal_floats by default.


How to get three channel output from IDWT?

I have a three-channel image (RGB). While performing IDWT using the coefficients obtained from DWT, I got one channel output. Can you please let me know how to obtain three-channel output from IDWT?

Thenk you for openning this issue.
I see the problem, for now I will recommand you to do the dwt for each channel separatly.
I will work on a fix for this issue hopefully I will fix it soon

Thanks. That is what I am doing now. It would be of help if it can be generalized for n-channel inputs. Thanks again.

Any updates on this? I would like to be able to use this for a project I am working on. I am able to get the right numbers for the output, but the shape seems to be wrong.
Here's the code i am using to test it out
if __name__ == '__main__':
    transform = DWT(name='db4', concat=0)
    inverse = IDWT(name='db4', concat=0)
    ones = tf.ones((3, 160, 64, 1))

    output = transform(ones)
    inverted = inverse(output)
    print(ones.shape, output.shape, inverted.shape)
    # expect to be (3, 160, 64, 1) (3, 80, 32, 4) (3, 160, 64, 1)
    # found to be  (3, 160, 64, 1) (3, 80, 32, 4) (3, 64, 160, 1)

Thanks, I added a fix for the shape issue


Fix for MGS1, MG1&2 and BONUS

Please let your fix work with them too, they all lacking resolution selection, all has same aspect ratio problems. And in case of MGS1 its possible to swap some values in memory to change internal rendering resolution of PS1 emulator (and hopefull other settings like bilinear filtering of textures) to actually make less ugly.  I suggest to mimic Duckstation resolution list, indlucing 2x. 4x, 8x, 9x, auto. Not sure which emulator they used but i believe it may also has some values for overscan zoom in crop or smart black bar detection to fix NTSC versions black bars.

And in case of MGS1 its possible to swap some values in memory to change internal rendering resolution of PS1 emulator
What values?

I like hope the OP mentions MGS1 can be fixed, but is too damn lazy to actually provide said values. Great going, OP, no really.

I like hope the OP mentions MGS1 can be fixed, but is too damn lazy to actually provide said values. Great going, OP, no really.

Are you mad? Why should i provide values that i dont have? I know its easy to find them with any debuger, even cheat engine, but i dont have time for that. Man who does this fix probably do have time to mess with debuger and search for values.
Game is PS1 rom inside emulator, pal (EU) and ntsc (US\JP) versions uses internal resolutions of 320x224 and 320x240, so they are easy to find and replace.

Man who does this fix probably do have time to mess with debuger and search for values.

Why do you feel entitled to other people's time? Your post and response just come as condescending. Probably not the best way to go about asking someone to do something for you, for free.

This is forum for reporting issues and posting ideas and suggestions for software creators.
Thats what i did , i reported issue and suggested idea. And its up to author of software what do with it.
And i am not developer of this software for someone to demand some offsets from me and calling me lazy for suggesting ideas to author instead of doing everything myself.
If i had time for that i would not be here reporting and suggesting and would make my own fix, and probably everyone else by logic of such commenters should not ask anyone who already did something - to add some improvements, and instead should go make his own version of same thing - this is insane and stupid!).
 Nobody asked you to comment on that
Its not forums for personal opinions on someone's reports and suggestions. So you both can STFU and GTFO if you have nothing valuable to say on subject and nothing to report and nothing to suggest to author of this software.
@Lyall  can you please delete those nonsense comments from Cassian and roflcopter (and my answers to them as well) to keep issue tracking clean?

And in case of MGS1 its possible to swap some values in memory to change internal rendering resolution of PS1 emulator

The way you phrased this, ie "it's possible." vs "is it possible?", implied that you KNEW that there's memory values that can be swapped to do that, which is why they were asking for you to clarify on which values you were referring to. The way it was written up was presented as a statement, not a question.

And in case of MGS1 its possible to swap some values in memory to change internal rendering resolution of PS1 emulator

The way you phrased this, ie "it's possible." vs "is it possible?", implied that you KNEW that there's memory values that can be swapped to do that, which is why they were asking for you to clarify on which values you were referring to. The way it was written up was presented as a statement, not a question.

It is a statement, i know that such values exist, because they always exist in every game in every emulator! Any value any games uses is stored in process memory, and resolution of screen used in game is also a value that always exist and sits at some offset waiting for someone to find it and change it.
This whole fix based on this principle - that value of screen resolution is always exist in process memory and can be found and swapped to change resolution! And hell lof of other fixes, patches and mods (including a couple i made for other game a while ago) for games did that before!
I know that MGS1 emulator has value for internal resolution, author of this fix knows how to find resolutions and change them, but for some reason he did not do it with MGS1, so probably he did not think of that or forgot about it, so i guided him in this direction, we both know its possible and its a conversation between us, and comments here is not place to attack me or write your personal opinions on who is lazy or not, its only for bug reports and suggestions! Has nothing to report and suggest? Move along!
 I mead dudes, what are we even discussing here? Can you stop this already and not comment anything anymore about that and stop ashaming yourself and crapping into issue tracker?
I already told before its not the place for your personal opinions! Move along if you dont know what you talking about and has nothing to suggest or report!
 Best strategy in case if you dont know something, not comment anything and move along!
I swear - anyone who will continue this discussion and will not post anything that is report related to subject or  suggestion related to subject - will prove he is complete and talal imbecile. So stop now!
Hope that when Lyall will see the mess you guys did in this thread, he will clean all "personal" comments.

I found that MGS1 uses M2's emulation framework. A little history https://arstechnica.com/gaming/2019/05/required-gamer-viewing-the-history-of-m2s-japanese-retro-port-empire/
i found tool to unpack m files and messed up with scripts a little, it seems to be possible to set up resolution this way, but it did not work yet, coud be that extra settings disable in game code and even enabling them via scripts may not have effect. But its a promising start.

https://github.com/nuggslet/MGSM2Fix


( Request ) Hit Chance For Projectiles.

IF its possible.

Why would you even want this

To look more legit when killing ppl

bcz fed>rijin soon..


would saving all jittered train data into X array be too-memory-wasted?

@TianzhongSong
I admire your code, way better than garyli1019 version.
but...
as code shown in this line:
https://github.com/TianzhongSong/PointNet-Keras/blob/master/data_loader.py#L74
When I saw you saved all the pre-calculated (rotate and jitter) data into a np.array called X, I was wondering whether would it use too much memory?
Here I am, providing a solution, can you thing about it:
class RotateJitterPC(keras.callbacks.Callback):
    def on_epoch_begin(self, epoch, logs={}):
        self.model.x = jitter_point_cloud(rotate_point_cloud(train_points_r))

and then use this callback in model.fit() or whatever you use to fit
# Fit model on training data
history = model.fit(
    train_points_r, Y_train,
    batch_size=32, epochs=50,
    validation_split=0.2,
    callbacks=[rotate_jitter_pc, early_stop]
)

In this way, it will be calc-on-run, and memory can be reallocated on time.

大哥我错了，是我py技术太菜，没有意识到fit_generator和yeild关键字的作用，你的代码太6了，我学了好久才懂，我还拽英文真是打脸


CalTRACK Issue: Modifications to the Temperature Variable

Prerequisites

[X] Are you opening this issue in the correct repository (caltrack/grid/seat)?
[X] Did you perform a search of previous Github issues to check if this issue has been previously considered?
(Addressed in issues #124 and #111, which can be rolled into this scan if approved)

Articles #s: 3.1, 3.3, 3.9
Description
For billing, daily, and hourly methods, the current models use temperature or HDD/CDD. While unmodified temperature variables are highly correlated with the energy use of HVAC systems, there are a few modifications to the temperature variables that are readily tested using available data and could potentially lead to further reductions in residual variance. In other words, they may be more appropriate for representing the dynamics of outdoor heat penetrating the building shell. In modeling thermostat data, I have seen average out-of-sample CVRMSE reductions of up to 15-20% at the site level from these modifications, so I am curious whether they will have a similar impact on the daily or hourly methods for CalTRACK.
The two modifications I propose are:

Lagged rolling averages for temperature or HDD/CDD: Heat takes time to penetrate the building shell. Especially for hourly calculations (e.g. HDH/CDH in Issue #120), averaging the previous 2-3 hours can mitigate the impact of spikes and dips on assumed usage, and align the temperature variable with actual consumption. A heat build-up term with a weight to prior temperatures that decays with time difference (e.g. 100% for t, 75% for (t-1), 75%^2 for (t-2), etc.) should also be tried.
Incorporation of relative humidity for a heat index and wind for wind chill: Although much of the work for heat indices (a.k.a. sultriness indices) revolves around their application for humans or animals, HVAC systems do work harder to get rid of heat when it is hot and humid, and a heat index term has consistently beaten dry bulb temperature in my modeling of whole building energy use. I have had some success with using wind chill, although I don't think it would work as well for C&I.

Proposed test methodology

Use a test C&I data set and a test residential data set
Build alternate formulations for temperature in place of dry-bulb temperature
a. 1 to 6 hour lags, inclusive and exclusive of current temperature
b. A couple heat build-up specifications
c. A couple specifications for relative humidity (e.g. from https://journals.ametsoc.org/doi/pdf/10.1175/1520-0450(1979)018<0861%3ATAOSPI>2.0.CO%3B2) and wind chill.
Calculate CDD and HDD off of modified temperature variables
In-sample and out-of-sample testing of CalTRACK methods using the modified variables

Acceptance Criteria
Demonstrated reduction of in-sample and out-of-sample CVRMSE and/or bias (i.e. not harming either while improving at least one)

I'd like to support the first modification (Lagged rolling averages for temperature or HDD/CDD) but would suggest this could be analyzed as part of the more general issue #124.

I believe that this method would be applied to hourly temperatures, but apply to both hourly and daily data, and deals more with short term conduction of heat through the building shell. As far as I understand it, #124 is much longer term. Is that not the case?

@jkoliner: True; but #124 targets the same offset between temperature changes and energy changes. If we were to  confirm a significant impact from daily variations via that issue, perhaps the same setup & approach & testing might be shared with this issue? Also, we're not aware of anyone getting paid yet based on the hourly methods so perhaps this finer detail is not as urgent (?).

At the very least, this seems very related to #124 and maybe should be handled together with the same general approach.

You keep this flurry of commenting up and I'm gonna nominate you for Chair for next year!

Jon

Get Outlook for Androidhttps://aka.ms/ghei36

From: Dan R<EMAIL_ADDRESS>Sent: Tuesday, October 29, 2019 2:58:43 PM
To: energy-market-methods/caltrack<EMAIL_ADDRESS>Cc: Jon Koliner<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [energy-market-methods/caltrack] CalTRACK Issue: Modifications to the Temperature Variable (#134)
At the very least, this seems very related to #124https://github.com/energy-market-methods/caltrack/issues/124 and maybe should be handled together with the same general approach.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/energy-market-methods/caltrack/issues/134?email_source=notifications&email_token=AL43VLDGTUD3ZDMIVRTNX4LQRCIXHA5CNFSM4I2GYKR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECR4DOI#issuecomment-547602873, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AL43VLCVAVSQFXDAJ26MDSLQRCIXHANCNFSM4I2GYKRQ.


Documentation

The project has grown quite big in code and would make sense to have the project properly documented. So it can be easily understood in few hours of poking around

@juicycleff I would like to understand the project and also contribute to the Documentation. Can you help me get started ?

@perceptron007 sure, could you tell me what you don't understand :)

link to docs https://github.com/juicycleff/ultimate-backend-docs

@juicycleff Hey :) I have a question regarding setting up as well. I'll pose it here, hope it's an appropriate place.
I am trying to start api-admin. Created a config.yaml in apps/api-admin based on the example. Started consul with consul agent --dev and put a config with consul kv put ultimatebackend/config/io.ultimatebackend.api.admin config.yaml. I checked the counsul kv store and it seems to contain a config with appropriate key
$ consul kv export
[
        {
                "key": "ultimatebackend/config/io.ultimatebackend.api.admin",
                "flags": 0,
                "value": "Y29uZmlnLnlhbWw="
        }
]

When i run api-admin, an error is returned, regardless of whether the consul store is populated or not.
$ npx nest start api-admin
Starting type checking service...
Hash: e92a9e59e02f02c2c03c
Version: webpack 4.42.1
Time: 33348ms
Built at: 2020-06-07 17:15:10
Entrypoint main = apps\api-admin\main.js
[Nest] 3232   - 2020-06-07 17:15:12   [NestFactory] Starting Nest application...
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] BootModule dependencies initialized +84ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] GlobalClientModule dependencies initialized +0ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] CoreModule dependencies initialized +0ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] PassportModule dependencies initialized +1ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] WebhooksModule dependencies initialized +0ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] NotificationsModule dependencies initialized +0ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] HttpModule dependencies initialized +4ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] PlansModule dependencies initialized +1ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] AppModule dependencies initialized +0ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] ConsulModule dependencies initialized +1ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] RolesModule dependencies initialized +0ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] CqrsModule dependencies initialized +12ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] TenantsModule dependencies initialized +0ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] BillingsModule dependencies initialized +0ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] CardsModule dependencies initialized +1ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] TenantMembersModule dependencies initialized +0ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] UsersModule dependencies initialized +0ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] AccessTokenModule dependencies initialized +0ms
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] GraphQLSchemaBuilderModule dependencies initialized +0ms
[Nest] 3232   - 2020-06-07 17:15:12   [ConfigModule] ConfigModule initialized
[Nest] 3232   - 2020-06-07 17:15:12   [InstanceLoader] ConfigModule dependencies initialized +0ms
[Nest] 3232   - 2020-06-07 17:15:12   [ExceptionHandler] OAuth2Strategy requires a clientID option +3ms
TypeError: OAuth2Strategy requires a clientID option
    at FacebookStrategy.OAuth2Strategy (C:\Users\Dmitrijs\ftw\ultimate-backend\node_modules\passport-oauth2\lib\strategy.js:86:34)
    at new Strategy (C:\Users\Dmitrijs\ftw\ultimate-backend\node_modules\passport-facebook\lib\strategy.js:56:18)
    at new MixinStrategy (C:\Users\Dmitrijs\ftw\ultimate-backend\node_modules\@nestjs\passport\dist\passport\passport.strategy.js:31:13)
    at new FacebookStrategy (C:\Users\Dmitrijs\ftw\ultimate-backend\dist\apps\api-admin\main.js:12436:9)
    at Injector.instantiateClass (C:\Users\Dmitrijs\ftw\ultimate-backend\node_modules\@nestjs\core\injector\injector.js:289:19)
    at callback (C:\Users\Dmitrijs\ftw\ultimate-backend\node_modules\@nestjs\core\injector\injector.js:76:41)
    at processTicksAndRejections (internal/process/task_queues.js:89:5)
    at async Injector.resolveConstructorParams (C:\Users\Dmitrijs\ftw\ultimate-backend\node_modules\@nestjs\core\injector\injector.js:117:24)
    at async Injector.loadInstance (C:\Users\Dmitrijs\ftw\ultimate-backend\node_modules\@nestjs\core\injector\injector.js:80:9)
    at async Injector.loadProvider (C:\Users\Dmitrijs\ftw\ultimate-backend\node_modules\@nestjs\core\injector\injector.js:37:9)
    at async Promise.all (index 7)
    at async InstanceLoader.createInstancesOfProviders (C:\Users\Dmitrijs\ftw\ultimate-backend\node_modules\@nestjs\core\injector\instance-loader.js:42:9)
    at async C:\Users\Dmitrijs\ftw\ultimate-backend\node_modules\@nestjs\core\injector\instance-loader.js:27:13
    at async Promise.all (index 14)
    at async InstanceLoader.createInstances (C:\Users\Dmitrijs\ftw\ultimate-backend\node_modules\@nestjs\core\injector\instance-loader.js:26:9)
    at async InstanceLoader.createInstancesOfDependencies (C:\Users\Dmitrijs\ftw\ultimate-backend\node_modules\@nestjs\core\injector\instance-loader.js:16:9)

Appreciate any help.

@dayrim Facing same issue here. I also got another problem when try to build api service:
TS2740: Type 'Tenant' is missing the following properties from type 'TenantEntity': currentPlan, autoCommit, publish, commit, and 8 more.

@dayrim make sure you change consul kv store type to yaml, that solves the issue in most cases.

@dayrim this is a typescript issue perhaps due to an upgrade in the typescript lib, you can just use ts-ignore to remove it

@juicycleff Thanks, changing kv store type to yaml via Consul web UI solved it.

@juicycleff Hello is there any material to read about what // @inject_tag: bson:"id,omitempty" type of comments mean in proto files ?

@dayrim please ignore those flags its important in a golang replica I have of the Ultimate backend that is closed source. Its important for codegenration and biding to mongo bson schema. I just shared the proto files between I am planning to clean them


New line?

How do i start a new line? For example i want 2 values on one line and another 2 in their own line.
Thanks for the great work btw, keep it up.

You can get a new line by using \n


Python version check in start_server.sh fails for 3.10

I suppose this is not correct:
~/automatic/extensions/Auto-Photoshop-StableDiffusion-Plugin$ ./start_server.sh
Already up to date.
Your version (Python 3.10.6) not valid, should be >3.7

@ArtemKo7v What OS are you running?
@evvfebruary Can you take a look at this issue

@AbdullahAlfaraj
Yep, of course, checking it right now!

@ArtemKo7v
Thank's for spotlight this issue!
Fixed this in https://github.com/AbdullahAlfaraj/Auto-Photoshop-StableDiffusion-Plugin/pull/152


Add powershell support

On this PR I am adding Powershell support to the SSH support.
This allows to run Powershell commands in the same scope of the scripts, allowing to use other scripts to help teachers on the management of the attendees activities repos.
Sample GitHubDevelopersTrainingTeacherScripts
cc: @parkerbxyz @amyschoen

cc: @parkerbxyz @amyschoen

Hi @amyschoen please merge the PR as the branch restrictions avoids not authorized users to do so. Thanks.
cc: @parkerbxyz

Merged @rulasg


feat(clickhouse): common/clickhouse-srv -> opensrv-clickhouse

I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/
Summary
This is a fairly large commit, but in fact just migrating common/clickhouse-srv to opensrv-clickhouse

common-tarcing -> tracing
common-io -> opensrv-clickhouse/io (contains only marshal and unmarshal)
some corresponding minor modifications to ensure that the tests all pass

Since we don't need a top-level crate to manage the server for now, remove srv.

Will we publish them to crates.io?

Will we publish them to crates.io?

Yes.

Love this repo ❤


Fixed typos and missing variables

General fixes of mistakes and typos

Removed a leftover encoder declaration due to a bad merge
Added missing imports
Fixed typos in variables
seq_length is in the end needed for the ConvolutionGluBlock, so I added it
max_seq_length added as a config param and argument
reordered new imports to be alphabetical

sorry, saw this too late. This should also be fixed in the convseq2seq branch now.


Print revisited

I actually meant this print statement:
diff --git a/iex/stocks.py b/iex/stocks.py
index 8d031e0..03c9d8e 100644
--- a/iex/stocks.py
+++ b/iex/stocks.py
@@ -26,7 +26,8 @@ class Stock:
     def _get(self, url, params={}):
         request_url =f"{BASE_URL}/stock/{self.symbol}/{url}"
         response = requests.get(request_url, params=params)
-        print(response.url)
+        # Removed print for every URL we request
+        #print(response.url)
         if response.status_code != 200:
             raise Exception(f"{response.status_code}: {response.content.decode('utf-8')}")
         result = response.json()

my bad.

i believe this has been resolved.


Rolling back 'java: optimize partition key (#1125)'

https://github.com/googleapis/gax-java/pull/241 hasn't been submitted yet, so this PR shouldn't have been submitted.

Codecov Report

Merging #1131 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #1131   +/-   ##
=========================================
  Coverage     85.71%   85.71%
  Complexity     3608     3608
=========================================
  Files           366      366
  Lines         14212    14212
  Branches       1614     1614
=========================================
  Hits          12182    12182
  Misses         1586     1586
  Partials        444      444

Impacted Files
Coverage Δ
Complexity Δ

...pi/codegen/viewmodel/BatchingPartitionKeyView.java
100% <ø> (ø)
2 <0> (ø)
:arrow_down:

.../transformer/java/JavaGapicSurfaceTransformer.java
98.79% <ø> (-0.01%)
43 <0> (ø)

...e/api/codegen/transformer/BatchingTransformer.java
100% <100%> (ø)
16 <0> (ø)
: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 4d79dcc...3a2fe54. Read the comment docs.

@pongad FYI


Dev/rpc calls inputs and outputs

Hey there, i completed rpc list can you check it?
Thanks

Remove similar calls folder. Move all to rpc-calls folder


Multiple stacked bars

Hi,
Is it possible to create multiple stacked bars in one chart?
Example: http://www.highcharts.com/demo/column-stacked-and-grouped
I only found a sample with one stacked bar.
Seb.

@Seb33300 this is not currently possible with Chart.js.
Closing as duplicate of #1151


FeatureLayerPopup - Layer selection doesn't work

Selecting layers from the sidebar works perfectly but:
When I click on the map on an object and I choose a layer from the popup NOTHING happens. It's not highlighting any layer on the sidebar nor activating details panel...
Tool not usable at the moment. Tested on Safari/Chrome

Thanks for the issue report. You can use a previous version while you wait for a fix https://maputnik.github.io/releases/v1.5.0

Layer selection from Popup isn't working since https://github.com/maputnik/editor/pull/483/commits/8f391d7d523c5f8df6d0d048d59ef09889269d0f .
@sk1p would you mind taking a look?

@pathmapper Thanks for letting me know, I'll try to find time this weekend

From https://github.com/maputnik/editor/issues/594#issuecomment-573537487:

When I have about 300 layers [...] I cannot choose proper layers – and it’s impossible to do it manually scrolling that big list.


Documentation: Add lib comparisons

It would be fruitful for Sugar's users (and its Authors) to create some comparisons to other similar libraries. I'm starting a list of libs that Sugar could compare use-cases with:
For dates
https://github.com/arshaw/xdate
For Arrays/Objects
http://documentcloud.github.com/underscore/

I'm always open to new ideas, so I like this idea. That being said I did just go through xdate pretty thoroughly (thanks for this one) and I had a very thorough look at Underscore just before Sugar was first released. In fact, many methods, particularly those of the function class owe their inspiration to Underscore directly.
Anyway, if you (or anyone else) comes across other libraries that can be compared, list them there... thanks @jasonkuhrt

This issue is about creating code-comparisons on the sugarjs.com website. I know you already looked at these two libs extensively, so they should be easy starting places to create the same use-case in both those libs and sugarjs's version.
The point here is you make lib-decision-making easier by showing them comparisons that they likely want to see.

Ah ok... that will take more time but I could see it being useful. Do you have an example of what you would like to see here and/or how it should be presented?

OK this one has taken awhile, but I've finally gotten something out for this. It's a bit preliminary and there will be a lot more data to add (more for Ruby coming soon), but have a look:
http://sugarjs.com/libs

This is a very helpful start. Great work. Thanks Andrew. I'll take a closer look later.

Cool... I will keep working on the libs...  these are all public for people to update... hoping some good collaboration will happen here...

Nice comparison to Ruby and Python!
Found a couple of typos/issues:
For http://sugarjs.com/libs in the Python::Any description
Returns true if all elements in the iterable are true
It should be:
Returns true if any elements in the iterable are true
Also, for Ruby::Clear
The Javascript example could be:
arr = [1,2,3,4];
arr.length = 0;

For Ruby::reverse_each, you forgot the () after reverse. Examples should be:
ES5 Code:
arr.reverse().forEach(fn);

Sugar:
arr.reverse().each(fn);

In Ruby::Squeeze for the Javascript example, try this:
str.replace(/(\s)+/g, '$1')

rather than this:
str.replace(/(S)(s)*/, '$1$2')

Let me know if you'd like me to make a Fork or Patch.
Cheers,
->> Josh W. <<-

Thanks for this! Ya there should be a disclaimer but these are not intended to be perfectly robust solutions. I realized very quickly that if I started down this path I would never get this thing out...
I'll definitely make fixes but feel free to fork and patch as needed! Anything that goes into docs/libs/ruby.js or docs/libs/python.js will make it back out to the site....

I'm going to close this issue for now, but there is now stuff in the README on how to contribute more for the libs page... I will also feature this in a more prominent area soon.
Thanks for all the contributions!


feat: Refactor Logging to use Brillig foreign calls

Description
Reimplement logging in a more generic way uses Brillig foreign calls and AbiType for de/serialization
Problem*
Resolves #1615, #1910, #1925 and partial work towards #1911
Summary*
There is now a new logging function
unconstrained fn println<T>(input: T)

As this matches the println used by the old SSA, some logic had to be adding to filter for the correct println function in the stdlib depending on whether the experimental ssa is activated.
During monomorphization of a function call we check whether we have an oracle for println. We then serialize the AbiType to a string literal and attach it as the latest argument of the function. nargo who is executing the println oracles then know how to interpret these compiler attached arguments.
The example program under crates/nargo_cli/tests/test_data_ssa_refactor/debug_logs/src/main.nr will give this output:
"*** println ***"
"0x05"
["0x05","0x0a"]
{"foo":"0x0f","my_struct":{"x":"0x0a","y":"0x05"}}

The enable_slices flag was renamed to experimental_ssa as it is needed in this PR.
In order to avoid the struct serialization set by aos_to_soa in monomorphization I also added checks of the experimental_ssa flag to monomorphization to avoid using aos_to_soa when we are compiling down to the new SSA pass.
Documentation

[X] This PR requires documentation updates when merged.

[X] I will submit a noir-lang/docs PR.

[X] I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

[X] I have tested the changes locally.
[X] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Fixes #1925

I was thinking that we'd have a Display implementation for InputValue as that's ultimately what we're wanting to display.

I also need the AbiType though in order to convert an InputValue to the expected JsonType

If this is blocking for another issue, but the Display issue with this PR is blocking this one, we can split out the relevant change and merge only that. I'll leave the call to @TomAFrench whether Display is blocking this PR or not but it doesn't sound too major to me.

As far as updating the docs that result from this change, is it just that we are now able to log structs in addition to Fields, integers and arrays?

Seems like so, @vezenovm could you confirm?

@critesjosh yes you can now log structs, but also expressions now (such as x + y). You can also perform string interpolation as of this PR: https://github.com/noir-lang/noir/pull/1952, so you can construct more complex logs using nargo.


Update the events html5 app

In Brussels there are just too many events. Let's store the first 100 events into serviceData and load the other events using rest calls while scrolling down the html5 app.

Maybe an idea to build a cache in local storage, and add a param a query on modified_since in the API call?

Can always improve in the future, for now we need to get rid of events in service data

https://github.com/our-city-app/oca-backend/pull/908
For now i only implemented the branding, but the events are still saved in service data (i commented where we need to change some code)


Porting sample React Todo app to Svelte.

I am exploring best practice implementation of Dexie in a Svelte PWA.
Has anyone ported Dexie/samples/dexie-cloud-todo-app to Svelte?

I also posted an invitation to collaborate on this in Svelte Discord Channel.

I have neither ported the todo app over to Svelte nor am I sure that what I'm about to provide is best practices. However, I've been working on an app that uses both Svelte and Dexie. The code is freely available at https://github.com/Dilden/helth.
Code relevant to Dexie is located in src/stores/db.js and src/stores/stores.js is where the various DB manipulations are abstracted into Svelte stores. These values can be retrieved from these stores throughout the application by prepending the store names with a $.


Support shuffle for bool values.

Resolves #827

@whitneywhtsang @etiotto
Here is a version that should work for other int/fp types and with casts moved to GEN lowering.

As a side comment, I noticed that we produce two shuffles for 64-bit values in our TargetInfo but our GEN shuffle lowering actually accepts 64-bit values. Should we keep 2 shuffles for fp64 and int64?

Let's change to a single shuffle for 64-bit values, and check if that works as expected.


Began Rewrite for v1.0.0

In This Update:

Migrated to OOP Structure for Better Variable Management
Updated Progress Bars (New Look / More Exact Performance / Cross-Platform)
Implemented Song Skipping

Plans and Possibilities (Not Yet Implemented; Take a Stab!):

Skip Song OR Return to Menu (on CTRL+C)
Pausing / Resuming Songs (AHK Bindings or other Hotkey module?)

Bugs:

Fix the Duplicate Progress Bars on Stream Corruption of 403 Error

Closing to Re-Submit with Fixed 'requirements.txt'!


Update apollo_3.0_development_kits_user_manual_cn.md

Can't print velodyne_16 topic information.

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

Please raise an issue instead of creating this pull request. Closing it now.


Fix #306 and restore original color on profile popup buttons

Ideally the theme should take care of this, but for now grey on white is better than white on white.

Closing in favor of a PR into https://github.com/gmusic-utils/gmusic-theme.js