Text Wrapper crashes when calling rollback twice

When i try to call getRollback twice on a text wrapper i get a Symfony\Component\Console\Exception\RuntimeException: Unable to write output. Exception.
Example:
$app = new \Phinx\Console\PhinxApplication();
$wrap = new \Phinx\Wrapper\TextWrapper($app);
$wrap->setOption('environment', 'db');
$wrap->setOption('configuration', 'phinx.php');
$wrap->setOption('parser', 'php');

echo $wrap->getRollback(null, 0);
echo $wrap->getMigrate();
echo $wrap->getRollback(null, 0); // <- exception here

When i setup a new wrapper in between it works:
$app = new \Phinx\Console\PhinxApplication();
$wrap = new \Phinx\Wrapper\TextWrapper($app);
$wrap->setOption('environment', 'db');
$wrap->setOption('configuration', 'phinx.php');
$wrap->setOption('parser', 'php');

echo $wrap->getRollback(null, 0);
echo $wrap->getMigrate();

$app = new \Phinx\Console\PhinxApplication();
$wrap = new \Phinx\Wrapper\TextWrapper($app);
$wrap->setOption('environment', 'db');
$wrap->setOption('configuration', 'phinx.php');
$wrap->setOption('parser', 'php');

echo $wrap->getRollback(null, 0); // <- no exception here

Edit: Version is 0.6.4

hi @moritz-h, what are you putting in your phinx.php file?

@robmorgan I created a minimal example: https://gist.github.com/moritz-h/579f43c8af02f3644115d91185b73f6e
(The<PHONE_NUMBER>0000_init_db.php  file should be in a migrations subdirectory, could not name it with path in the gist.)
I run composer install and start it with php -S localhost:8000 index.php.
The database settings in the phinx.php match to an existing mysql database and my php version is 7.0.8 (default Ubuntu 16.04).

@robmorgan I found out where this error comes from:
The TextWrapper creates a new Output Instance for each call:
https://github.com/robmorgan/phinx/blob/master/src/Phinx/Wrapper/TextWrapper.php#L215
This OutputInterface Object is recieved by execute() of a command (i.e. Status) and then passed to bootstrap()
https://github.com/robmorgan/phinx/blob/master/src/Phinx/Console/Command/Status.php#L68-L70
AbstractCommand::bootstrap() passes this to AbstractCommand::loadManager() where a new Manager object is created, but only if no Manager was created before:
https://github.com/robmorgan/phinx/blob/master/src/Phinx/Console/Command/AbstractCommand.php#L285-L288
The Manager object saves the OutputInterface as property. And that is the source of the error.
If i call the command a second time the TextWrapper creates a new OutputInterface and passes this to the command. But the command then has cached the Manager with the reference to the old OutputInterface from the first call.
To fix this, it would be possible to here create always a new Manager:
https://github.com/robmorgan/phinx/blob/master/src/Phinx/Console/Command/AbstractCommand.php#L285-L288
But i don't know if this would has any other bad side effects.
Otherwise perhaps its possible to just update the InputInterface and OutputInterface of the Manager.
Or perhaps the TextWrapper also should reuse the same OutputInterface for all command calls.
(Then with Symfony\Component\Console\Output\BufferedOutput instead of Symfony\Component\Console\Output\StreamOutput no not mess up with fopen)


[1.0] Make internal macros use macro dispatch pattern

Description
Backport of #72.
Fixes customized macro names to use the same names as dbt-spark for backward compatibility for users who override the equivalent macros in their projects.

The tests "verify packages" will be fixed once dbt-spark==1.0.1rc1 is published to pypi.


Airplay log very chatty

What version of Music Assistant has the issue?
2b1
The problem
The log file airplay_bridge.log is very chatty, with many logs per second.

Home Assistant OS
On what type of hardware are you running?
Raspberry Pi

It is very chatty because its in trouble, looks like you have a malfunctioning airplay device in your house ?


Testing datastore default_credentials are used

#1837 2nd issue
I think it is about mistakenly patching httplib for local testing.
Test script
from gcloud import datastore

client = datastore.Client()
entity = datastore.Entity(client.key('Test1'))
client.put(entity)

Setting up environment
This one needs:

Application default credentials set up

Showcase
I think this manifests with a Connection refused error:
(ve) javier@theoden:~/projects/spinoffs/gae$ python test-script.py
Traceback (most recent call last):
  File "test-script.py", line 5, in <module>
    client.put(entity)
  File "/home/javier/projects/spinoffs/gae/ve/lib/python3.4/site-packages/gcloud/datastore/client.py", line 319, in put
    self.put_multi(entities=[entity])
  File "/home/javier/projects/spinoffs/gae/ve/lib/python3.4/site-packages/gcloud/datastore/client.py", line 345, in put_multi
    current.commit()
  File "/home/javier/projects/spinoffs/gae/ve/lib/python3.4/site-packages/gcloud/datastore/batch.py", line 260, in commit
    self._commit()
  File "/home/javier/projects/spinoffs/gae/ve/lib/python3.4/site-packages/gcloud/datastore/batch.py", line 243, in _commit
    self.project, self._commit_request, self._id)
  File "/home/javier/projects/spinoffs/gae/ve/lib/python3.4/site-packages/gcloud/datastore/connection.py", line 331, in commit
    _datastore_pb2.CommitResponse)
  File "/home/javier/projects/spinoffs/gae/ve/lib/python3.4/site-packages/gcloud/datastore/connection.py", line 121, in _rpc
    data=request_pb.SerializeToString())
  File "/home/javier/projects/spinoffs/gae/ve/lib/python3.4/site-packages/gcloud/datastore/connection.py", line 93, in _request
    method='POST', headers=headers, body=data)
  File "/home/javier/projects/spinoffs/gae/ve/lib/python3.4/site-packages/oauth2client/client.py", line 631, in new_request
    redirections, connection_type)
  File "/home/javier/projects/spinoffs/gae/ve/lib/python3.4/site-packages/httplib2/__init__.py", line 1314, in request
    (response, content) = self._request(conn, authority, uri, request_uri, method, body, headers, redirections, cachekey)
  File "/home/javier/projects/spinoffs/gae/ve/lib/python3.4/site-packages/httplib2/__init__.py", line 1064, in _request
    (response, content) = self._conn_request(conn, request_uri, method, body, headers)
  File "/home/javier/projects/spinoffs/gae/ve/lib/python3.4/site-packages/httplib2/__init__.py", line 987, in _conn_request
    conn.connect()
  File "/usr/lib/python3.4/http/client.py", line 871, in connect
    self.timeout, self.source_address)
  File "/usr/lib/python3.4/socket.py", line 516, in create_connection
    raise err
  File "/usr/lib/python3.4/socket.py", line 507, in create_connection
    sock.connect(sa)
ConnectionRefusedError: [Errno 111] Connection refused

This is caused because it tries to make a auth token refresh against the emulator. There are no relevant logs for the emulator.
Workaround
I thought I had found a workaround but I cannot reproduce the workaround, therefore not a workaround.
The problem is the default credential, so it can be fixed by (ordered by personal preference):

Create a configuration with no default credentials, for the same project, or;
Remove the default credentials gcloud beta auth application-default revoke~

Hopefully you know how to patch the library not to collide!

When no default credentials exist, the following error is triggered:
(ve34) javier@theoden:~/projects/spinoffs/gae$ python test-script.py
Traceback (most recent call last):
  File "test-script.py", line 3, in <module>
    client = datastore.Client()
  File "/home/javier/ve34/lib/python3.4/site-packages/gcloud-0.12.0-py3.4.egg/gcloud/datastore/client.py", line 173, in __init__
    super(Client, self).__init__(credentials, http)
  File "/home/javier/ve34/lib/python3.4/site-packages/gcloud-0.12.0-py3.4.egg/gcloud/client.py", line 122, in __init__
    credentials = get_credentials()
  File "/home/javier/ve34/lib/python3.4/site-packages/gcloud-0.12.0-py3.4.egg/gcloud/credentials.py", line 82, in get_credentials
    return client.GoogleCredentials.get_application_default()
  File "/home/javier/ve34/lib/python3.4/site-packages/oauth2client-2.1.0-py3.4.egg/oauth2client/client.py", line 1398, in get_application_default
  File "/home/javier/ve34/lib/python3.4/site-packages/oauth2client-2.1.0-py3.4.egg/oauth2client/client.py", line 1388, in _get_implicit_credentials
oauth2client.client.ApplicationDefaultCredentialsError: The Application Default Credentials are not available. They are available if running in Google Compute Engine. Otherwise, the environment variable GOOGLE_APPLICATION_CREDENTIALS must be defined pointing to a file defining the credentials. See https://developers.google.com/accounts/docs/application-default-credentials for more information.

Ok, I have found something really strange.
Datastore running:
javier@theoden ~/p/s/gae> bash
sessions should be nested with care, unset $TMUX to force
sessions should be nested with care, unset $TMUX to force
javier@theoden:~/projects/spinoffs/gae$ activate 34
(ve34) javier@theoden:~/projects/spinoffs/gae$ $(gcloud beta emulators datastore env-init)
(ve34) javier@theoden:~/projects/spinoffs/gae$ python test-script.py
Traceback (most recent call last):
  File "test-script.py", line 5, in <module>
    client.put(entity)
  File "/home/javier/ve34/lib/python3.4/site-packages/gcloud-0.12.0-py3.4.egg/gcloud/datastore/client.py", line 319, in put
    self.put_multi(entities=[entity])
  File "/home/javier/ve34/lib/python3.4/site-packages/gcloud-0.12.0-py3.4.egg/gcloud/datastore/client.py", line 345, in put_multi
    current.commit()
  File "/home/javier/ve34/lib/python3.4/site-packages/gcloud-0.12.0-py3.4.egg/gcloud/datastore/batch.py", line 260, in commit
    self._commit()
  File "/home/javier/ve34/lib/python3.4/site-packages/gcloud-0.12.0-py3.4.egg/gcloud/datastore/batch.py", line 243, in _commit
    self.project, self._commit_request, self._id)
  File "/home/javier/ve34/lib/python3.4/site-packages/gcloud-0.12.0-py3.4.egg/gcloud/datastore/connection.py", line 327, in commit
    _datastore_pb2.CommitResponse)
  File "/home/javier/ve34/lib/python3.4/site-packages/gcloud-0.12.0-py3.4.egg/gcloud/datastore/connection.py", line 117, in _rpc
    data=request_pb.SerializeToString())
  File "/home/javier/ve34/lib/python3.4/site-packages/gcloud-0.12.0-py3.4.egg/gcloud/datastore/connection.py", line 93, in _request
    error_status = status_pb2.Status.FromString(content)
  File "/home/javier/ve34/lib/python3.4/site-packages/protobuf-3.0.0b3-py3.4.egg/google/protobuf/internal/python_message.py", line 780, in FromString
    message.MergeFromString(s)
  File "/home/javier/ve34/lib/python3.4/site-packages/protobuf-3.0.0b3-py3.4.egg/google/protobuf/internal/python_message.py", line 1083, in MergeFromString
    raise message_mod.DecodeError('Unexpected end-group tag.')
google.protobuf.message.DecodeError: Unexpected end-group tag.

Datastore outputs:
[datastore] May 31, 2016 10:52:52 PM com.google.appengine.tools.development.LocalResourceFileServlet doGet
[datastore] WARNING: No file found for: /v1beta3/projects/testing:commit

No idea on the reason...

Thanks for filing. We have a workaround for do-nothing credentials in our system test.
I'd imagine your Unexpected end-group tag error comes from an invalid request being sent.

@txomon Try gcloud==0.12.1. Until that release, the path to the emulator needed to be manually specified to include /datastore, as in /datastore/v1beta3/projects/testing:commit (the URL given in the emulator's error really isn't being served).

@bendemaree I assume you mean gcloud>=0.12.1?

Yes. I was assuming @txomon wanted that release for a specific release series vs. newer one and that's the highest for 0.12.x. 🙂

I think this can be specified as resolved maybe? I would create a Feature request ticket to have a way not to do that do-nothing credentials setup however.
The trouble was coming indeed from overlapping several emulator instances (connection refused) / restarts, and old version of gcloud and not having setup do-nothing account credentials from the code.

Gotcha. Thanks.


The calculator icon is incorrect

Microsoft PowerToys version
0.78.0
Installation method
GitHub
Running as admin
No
Area(s) with issue?
PowerToys Run
Steps to reproduce
Open PowerToys Run and enter the number
✔ Expected Behavior
When entering numbers, the correct Calculator icon should appear
❌ Actual Behavior
! [image] (https://github.com/microsoft/PowerToys/assets/125674921/ef598dc2-46e4-4e5b-8e47-18bc005fb7bd)
An incorrect Search icon is displayed
Other Software
No response

I just restarted it, and now it looks like this:

Big in PT Run. The appdata of PT Run needs to be cleared (image cache).

@Aaron-Junker
We closed my old issue a few weeks ago because it is hard to reproduce. Mostly this happens after a crash of PT Run.


[WIP] Step 2 split data tree

Description
This is the second phase of the split data tree. In the previous version, we collected all config paths in each entity and put them in the __config__ property. All those config properties do get inserted into final data tree which we don't need at all.
As part of this change, we will be creating another tree i.e 'configTree'  which will contain all config of each entity.
unEvalTree is split into 2 trees =>

unEvalTree
configTree

Example:
previous unEvalTree Api1 content

After this change
unEvalTree Api1 content

Note- above example doesn't have 'config' property
configTree Api1 content

Type of change

Chore (housekeeping or task changes that don't impact user perception)

How Has This Been Tested?

Manual
Jest
Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=30df990

/ok-to-test sha=12284ff

/ok-to-test sha=b31ae58

/ok-to-test sha=b05da8c

/ok-to-test sha=bf35323

ok-to-test sha=5f09c65

/ok-to-test sha=5f09c65

/ok-to-test sha=1e18ee4

/ok-to-test sha=f45c1c6

/ok-to-test sha=e9e939a

/ok-to-test sha=fe2f829

/ok-to-test sha=b61a564

/ok-to-test sha=810b49d

/ok-to-test sha=9718cb4

/ok-to-test sha=467e5cd

/ok-to-test sha=28e2aa0

/ok-to-test sha=fee73a3

/ok-to-test sha=36e0add

/ok-to-test sha=df2ffa1

/ok-to-test sha=df2ffa1

/ok-to-test sha=7f3d8d2

/ok-to-test sha=bb75a85

/ok-to-test sha=ec34c08

/ok-to-test sha=decc3d4

/ok-to-test sha=172fc9a

/perf-test

/ok-to-test sha=cf6fbb5

/perf-test

/ok-to-test sha=bc563ea

/perf-test

/ok-to-test sha=880e994

/ok-to-test sha=4a2f6c9

/ok-to-test sha=bd8a0ef

/ok-to-test sha=a0558cc

/ok-to-test sha=1be4122

/ci-merge-check

/build-deploy-preview env=release

Qa for this will be taken care by regression as there are not specific functional area change but overall functional change
cc @ramsaptami @Aishwarya-U-R


Add close_stack to stack.consolidate_transforms

This was a minor change @TotteKarlsson and I made a while ago to have a close_stack option when consolidating transformations.  (If this belongs elsewhere, please let me know.)

EM doesn't want to close stacks ever (always streaming processing) but I think its a useful option to have.


Leetcode Question Number 1403

Leetcode Question Number 1403 solution needs to be added in the repo .
The link for the question is this.
Accepted Languages C++ / JAVA .
You Only need to add the  class Solution in the file.
Make Sure you have read the contributions guidelines before creating an PR.

Assign me


Autodiff sylvester

Codecov Report
Patch coverage: 13.46% and project coverage change: -26.38% :warning:

Comparison is base (7cbeeb2) 88.51% compared to head (9ee688c) 62.13%.
 Consider uploading reports for the commit d0744ff to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #45       +/-   ##
===========================================
- Coverage   88.51%   62.13%   -26.38%
===========================================
  Files           8        8
  Lines        3778     3716       -62
===========================================
- Hits         3344     2309     -1035
- Misses        434     1407      +973

Files Changed
Coverage Δ

src/MacroModelling.jl
57.74% <12.00%> (-32.09%)
:arrow_down:

src/get_functions.jl
61.34% <50.00%> (-28.74%)
:arrow_down:

... and 4 files with indirect coverage changes

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


Inconsistent settings for FSDP Precision

Bug description
The FSDPPrecision class controls the dtype of the model parameters, autocasting and grad scaler.
It has two possible inputs: precision="16-mixed" or "bf16-mixed":
https://github.com/Lightning-AI/lightning/blob/6464650a3c91a75d4a7c4009d08a68a84037a9a1/src/lightning/fabric/plugins/precision/fsdp.py#L32-L34
In all other precision plugins, "mixed" refers to mixed precision training with model weights in float32 and autocasting inputs and operations to lower precision. However, the FSDPPrecision plugin sets the dtype of the model parameters to float16/bfloat16 regardless. This means we are actually running "16-true".
https://github.com/Lightning-AI/lightning/blob/6464650a3c91a75d4a7c4009d08a68a84037a9a1/src/lightning/fabric/plugins/precision/fsdp.py#L50-L60
Proposal:
For the mixed precision settings, set param_type=torch.float32. For the current case, introduce the precision settings "16-true" and "bf16-true".
cc @awaelchli @carmocca

This bug causes training to fail when performing certain fp32 operations, such as bicubic interpolation. I will submit a pull request later tonight to address this issue.


Should the error type implement From?

Similar e.g. to: https://doc.rust-lang.org/src/core/num/error.rs.html#32-36
IIUC, this will allow using TryInto with eyre error type where blanket impls (like From->TryFrom) may return Infallible as the error type?

@yaahc Thanks a million for clarifying, that makes sense! (closing this)


🚀 announcements: Set or modify announcements dates

Plugin Name
announcements
🔖 Feature description
Add Announcements date edition
🎤 Context
Since the timeline has been introduced, announcements can be related to an event in the future. On announcement creation or edition users can set a date.
Original feature request: https://github.com/procore-oss/backstage-plugin-announcements/issues/301
✌ Possible Implementation
{
  startTime: Date;
  endTime: Date;
  active: boolean;
}

👀 Have you spent some time to check if this feature request 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

Are you willing to submit PR?
Yes I am willing to submit a PR!
### Tasks

@gaelgoth announcements support active: boolean now 😄 looking forward to a pr 😉


Content offset doesn't work

Hi, I'm creating a spreadsheet view that gets its data periodically refreshed. Whenever I reload data, I want to remember where the user scrolled to. However, the view will always reload to the beginning of the scroll view. Here's my code:
override func willMove(toSuperview newSuperview: UIView?) {
        super.willMove(toSuperview: newSuperview)
        timer = Timer.scheduledTimer(timeInterval: 2.0, target: self, selector: #selector(updateContents), userInfo: nil, repeats: true)
        timer.fire()
    }

func updateContents() {
        let contentOffset = spreadsheetView.contentOffset
        print(contentOffset)
        spreadsheetView.reloadData()
        spreadsheetView.contentOffset = contentOffset
}

I'm printing out the contentOffset values and they seem to be calculated correctly.

Do you know what the issue might be? Thanks!

@zzhao36 Spreadsheet view keeps contentOffset after reloading. It is the same behavior as UITableView or UICollectionView. I'd like to know what happens correctly. Can you provide a reproducible sample code?

Closing due to no response

Hi, sorry for the super late reply! Essentially, I was constantly reloading my spreadsheet view to update the information in the cells. Basically, the content offset would always be off:

I think it's because reloadData() was called before the scrolling stopped, and therefore the content offset was not the actual, intended content offset. I fixed this by adding a new variable 'isScrolling' to the spreadsheet view, and then modified the scrollview delegate:
extension SpreadsheetView: UIScrollViewDelegate {
       public func scrollViewDidScroll(_ scrollView: UIScrollView) {
               isScrolling = true
               // more code
       }

       public func scrollViewDidEndScrollingAnimation(_ scrollView: UIScrollView) {
               isScrolling = false
               // more code
       }
}

And then when I called my reload function, I checked to make sure the user wasn't scrolling:
// fires once a second!
func shouldUpdateSpreadsheetView() {
        if !isScrolling { return }
        // more code
}

In the future, I think there should be two optional functions for SpreadsheetViewDelegate exposed in the public API that does something similar to what I did here - spreadsheetViewDidScroll, and spreadsheetViewDidEndScrollingAnimation. Please let me know if you have any questions, thanks!

I believe this was fixed by https://github.com/kishikawakatsumi/SpreadsheetView/pull/117. If you still in trouble, feel free reopen this issue or create a new issue. Thanks for reporting!


First implementation of compression

Create a draft pr for starting discussion.
https://github.com/Azure/azure-functions-kafka-extension/issues/207
I add three configuration parameter to Kafka output.
CompressionType, LingerMs and CompressionLevel  I've never tested the compression, so that before make this PR available, I'd like to test it.
sovles
https://github.com/Azure/azure-functions-kafka-extension/issues/207

Since the KafkaProducer is shared among the two client, the configuration should be in host.json

If everyone is ok, I'll add documentation and merge it.

Wouldn't mind getting this reviewed:
https://github.com/Azure/azure-functions-kafka-extension/pull/175

move to #175 PR.


Add .flowconfig and instructions to enable Flow

As discussed with @dikaiosune and @brentvatne in https://github.com/react-community/create-react-native-app/issues/79, adds

[x] .flowconfig inherited from underlying react-native version
[x] .flowconfig [ignore] list to deal with badly behaving dev-dependencies
[x] .flowconfig [ignore] to temporarily ignore Expo SDK types until https://github.com/exponent/exponent-sdk/issues/36 is solved
[x] RELEASES.md

Until the ignore for https://github.com/exponent/exponent-sdk/issues/36 is removed, Flow-users will need to generate stubbed types for Expo SDK modules using flow-typed install.

Whoa, that was a surprising amount of missed 💩 from my side! Coming right up.

No worries at all! Thanks so much for working on this.

Up for re-review @dikaiosune

Up for re-re-review @dikaiosune (no new commit, amended the last one to move section to correct README)

Wonderful! Thank you so much.

Always a pleasure to add more types to the world 🤠

we like your type of people around here @jevakallio

Hi, thank you for the documentation about Flow in the README.md
I think for Nuclide it is also important to mention that flow needs to be installed globally with npm install -g<EMAIL_ADDRESS>(and not with "brew install flow", as brew only have very few flow versions and not the current 0.38.0). Maybe it can save some time to some beginner like me :)

@mouneyrac are you sure your Homebrew installation is up to date? The latest version of flow is 0.42.0.

Hi @skevy, yes I upgraded brew. brew only had 0.32.0 and 0.42.0, and a very old one (when I searched all flow versions in brew). From what I read I need to use the flow version mentioned in .flowconfig as react native is implented against it. So the only way I found to install 0.38.0 is to use npm and not brew. That said changing .flowconfig to 0.42.0 did seem to work but I suppose I may have some bugs along the way if react native is not tested on a version not defined in .flowconfig ?

@mouneyrac I think you may want to track this issue: https://github.com/facebook/nuclide/issues/570
I'd really like to avoid getting too much into the weeds in terms of documenting editor configurations that are likely to change over time. If the issue with nuclide and local flow-bin continues to be a problem, might be worth reconsidering.

I also think that you should probably be OK using a slightly newer version of flow, but I may be wrong.

@dikaiosune no worries, it thought I would mention my problem with flow-bin/homebrew/nuclide as create-react-native-app will be used by many beginners. After reading the Nuclide issue, I think you are right about the issue been a Nuclide problem. Like they mention in the issue, it is probably a matter of Nuclide to display a popup saying that flow is not properly running and suggesting solutions (npm i -g flow-bin@...). Thank you :)


Update alert.md

Button alignment updated

Thank you! Now the CLA Sign has left to make this PR green. 🚀
Also, if you want, you can updated the latest versioned docs, so the fix will appear in the default selected docs version:

https://github.com/facebook/react-native-website/blob/master/website/versioned_docs/version-0.64/alert.md

@Vishnukr1101 This PR was valid, why you have closed it and removed the branch?
Unfortunately I'm not able to reopen this PR now.

@Vishnukr1101 This PR was valid, why you have closed it and removed the branch?
Unfortunately I'm not able to reopen this PR now. 😅

facebook-github-bot was not passing CLA Sign check.


patch to work with triton-2.0 on low mem devices

Issue described here https://github.com/openai/triton/pull/1208
This patch will allow benchmark to work on lower memory devices with triton-2.0.
Also described here...
https://github.com/qwopqwop200/GPTQ-for-LLaMa/issues/129

basically ensures that a tuple of len 3 is always returned...

I also opened up https://github.com/qwopqwop200/GPTQ-for-LLaMa/pull/145 as the same issue exists on the triton branch.
credit for the workaround goes to @fmac2000


New .Net Logging Adapters release notes page

Hi @abaranch, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (abaranch). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

Merged by request.


Make docker requirement optional

While working on https://github.com/google/turbinia/pull/537 with version 20190819.7
/usr/bin/turbiniactl -S server
Traceback (most recent call last):
  File "/usr/bin/turbiniactl", line 6, in <module>
    from pkg_resources import load_entry_point
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 3088, in <module>
    @_call_aside
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 3072, in _call_aside
    f(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 3101, in _initialize_master_working_set
    working_set = WorkingSet._build_master()
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 574, in _build_master
    ws.require(__requires__)
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 892, in require
    needed = self.resolve(parse_requirements(requirements))
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 778, in resolve
    raise DistributionNotFound(req, requirers)
pkg_resources.DistributionNotFound: The 'docker' distribution was not found and is required by turbinia

From the configuration:
# This will enable the usage of docker containers for the worker.
DOCKER_ENABLED = False

Is docker a required dependency or does need to be handled this more gracefully?

I see docker was added to requirements.txt in a3508eefb917994b670af8f5620a5d02ac6ac6f8.

@aarontp per our conversation, this is not urgent

@wajihyassine Maybe we can selectively import this if the option is enabled in the config?

Yeah that'd work, can assign this issue to myself to implement.

Releasing to wild for to be picked up given other constraints


Fix key label name from FILES_GMSEG to FILES_LESION

Fix the key label name from FILES_GMSEG to FILES_LESION once https://github.com/spinalcordtoolbox/spinalcordtoolbox/issues/4149 is addressed.
https://github.com/ivadomed/model_seg_sci/blob/1cfbfe992ce615c78aa38232a0bb36afeab56e1a/dataset-conversion/convert_bids_to_nnUNetv2.py#L91

Closing -- convert_bids_to_nnUNetv2.py has been deleted in https://github.com/ivadomed/model_seg_sci/pull/50.


Debounce not bound to state => may yield surprising results

Hi! I really like the simplicity and expressiveness of Svelte-FSM.
While implementing a FSM I tripped over the fact that debounce calls are not bound to the event where they were called from.
<script>
	import fsm from "svelte-fsm";

	const switchWithDelayedAction = fsm("off", {
		off: {
			click: "on",
			doStuff() {
				console.log("Surprise!");
			},
		},
		on: {
			_enter() {
				this.doStuff.debounce(1500);
			},
			click: "off",
			doStuff() {
				console.log("Do stuff in ON state.");
			},
		},
	});
$: console.log("state: ", $switchWithDelayedAction);
</script>

<button on:click={switchWithDelayedAction.click}>Click me</button>

https://svelte.dev/repl/56d96df27fea4d68a179084edc22a3ab?version=3.46.4
If I double click on the button (second click less than 1500 ms after the first 🙂 ), then "doStuff" is called in the "off" state. I expected the "doStuff" method in the "on" state to be called.
A workaround is to call "this.doStuff.debounce(null)" in the "_exit" method of the "on" state. (https://svelte.dev/repl/1b041923ec1e48258f8858c1fd597f46?version=3.46.4)
I see two "simple" solutions (hopefully there are better ones out there 🙂)

Document this behaviour and keep the implementation simple.
Clear all timers of the state when exiting the state.

I tried to fast double click the example you sent and couldn't see the described behaviour.

@kju2 thanks for reporting this, and sorry for not looking at it sooner. I'll look at your example and suggestions and let you my thoughts on this soon.

Hi @kju2,
I gave this some thought – I think the current behavior is what I would expect, and allows for more flexible scenarios.
Consider a scenario where you have a multi-step process with each step represented by a state, and where you need to complete all steps before a timer runs out. The timer is started with debounce when you enter step 1. A fallback timesUp action transitions to a outOfTime state. In this scenario, you would not want debounce to be bound the the state in which it was invoked.
An alternative solution you might consider is to pass an additional from argument when you invoke via debounce, then inspect this arg in your handler. See updated example:
https://svelte.dev/repl/0ff0912819904880b5808f5269858348?version=3.48.0
Per your recommendation, I will document this behavior to reduce confusion. I'll close the issue once I've updated the docs.
Thank you again for using svelte-fsm and for reporting this!
Regards,
Ken


Feature Request: Log Warnings to the Error List window and/or Output window of VisualStudio

Would be great if warnings and errors (missing translations, string format mismatches, etc.) could be logged to the VisualStudio ErrorList Window and/or the Output Window.
I'm aware of the "show only lines with warnings and changes" feature but I would really like to see these warnings in the log.
ErrorListProvider might be the way to go: https://msdn.microsoft.com/en-us/library/microsoft.visualstudio.shell.errorlistprovider(v=vs.140).aspx

Sounds like a cool feature.
When should it log the errors? Always if the window is open (like e.g. XAML designers), or only during build?
Can you provide a PR for this?

I would log the errors when opening the solution (if this is possible), when saving the file(s) and during build.
I cannot provide a PR at the moment.


Handle metadata collection flag on MetadataManager

What does this PR do?

Deprecate AgentCheck.is_metadata_collection_enabled() in favor of filtering submissions directly on MetadataManager.
TODO: actually this doesn't prevent integrations from doing unnecessary processing to retrieve metadata (eg fetching the version), which is probably why we added is_metadata_collection_enabled() in the first place. Hmm…
Motivation

Remove the need to always check for is_metadata_collection_enabled() in integrations, reducing boilerplate and potential for errors.
Immediately add support for honoring enable_metadata_collection Agent flag to all integrations. (Otherwise we're going to have and add an if statement to all our integrations, eg see #6061.)

Additional Notes

is_metadata_collection_enabled originally added in #5748.

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Closed, as integrations typically need to make processing to retrieve the version, and we'd want to prevent that processing from happening too, which this doesn't solve.


make switches and research upgrade sections hideable

Since the upgrade section now has separated areas for switches and research center upgrades, I would welcome an option to hide these (separately, preferably). They aren't needed most of the time but take up valuable space. Not everyone has an HD screen and I can barely fit in all buildings and the folded upgrade area of the current live version on my laptop screen even in fullscreen mode. With the three new buildings space there is even more precious.

Hmm hide how, in settings?  a + next to them?

If you could click the bar for them, that'd be great, Like the area I have put a red box around in this screenshot

Sounds like a good idea

This is really old, but I just noticed this.  Would this still be needed with the game already automatically reduce to one line?

I would like this, or something similar (a "lock these buttons" option) to avoid accidentally buying the chocolate egg or switching seasons during a cookie storm or when clicking a reindeer.

I have looked into this and found it to be pretty difficult to neatly implement.
The problem is that the bar with is not actually an html element, but rather padding of a section.
Therefore, we can't register a click on the bar but only on the whole section. This is problematic as this would trigger the visibility-toggle with every purchase of an update.
We could rewrite the code to generate the store and make it include a new first clickable element or replace the padding with an actual element. However, after spending quite some time understanding this quite obscure and badly documented part of the source-code I am a bit burnt out.
I think it will be better to ask Orteil to implement this, as I believe this is a good addition to the base-game and should not be that hard. I could even code it myself and deliver it to him to implement it, which would take much less time compared to making CookieMonster do this.


Groovy gstrings are not highlighted anymore

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.78.1-1683194560
OS Version:
PRETTY_NAME="Debian GNU/Linux 11 (bullseye)"
NAME="Debian GNU/Linux"
VERSION_ID="11"
VERSION="11 (bullseye)"
VERSION_CODENAME=bullseye
ID=debian

Groovy gstrings are not highlighted anymore
Steps to Reproduce:

Install any groovy extension
open a groovy file with a gstring like:
`def a = "some gstring $var"

I expect the $product to be in blue (highlighted)

@BarelElbaz what theme are you using? Asking since I've seen a couple issues lately where users have switched back to Dark (Visual Studio) instead of Dark+ (the old default theme) after the new default themes were shipped. The syntax highlighting is slightly different between the two even though the rest of the UI looks the same.

@daviddossett I'm using Dark (Visual Studio)
Because Dark+ is too dark for me ;-)

Dark+ is the same as Dark (Visual Studio) other than the slightly different syntax highlighting. I know, it's a bit confusing—we want to simplify this naming confusing.
If you hadn't previously changed themes, you would have been using Dark+ given that it was the default. Dark Modern is the new theme, which is darker.
Could you verify that you see the same issue when switching to Dark+?

Changing theme to Dark+ was fixed the problem
Thank you @daviddossett !

Great to hear. And we'll work on resolving this since you're not the only one to have switched back to what, at a glance, looks like the old default theme.


Fix for prefetching causing CSP violations

Fixes #1577
Since the "prefetch" requests made by replaceState don't understand content in context of CSP they fall-back to use the default-src directive. Setting default-src to be 'self' fixes that.

Well, good catch ;)
r+


MARS or Batch query support

Hi,
is very useful to have mars or batch support.
Seems that semicolon between query isn't enough, and the server return an syntax error.

This is the "limitation" - design choice - of NuoDB's protocol. You cannot send multiple commands in batch. One ExecuteXxx is one statement.

Hi,
Thank you for the quick answer 😊.
I found a workaround; I open a DBConnection, create a DBCommand associated to the DBConnecion, create a DBDataAdapter associated to the DBCommand; then for each query to execute i change the DBCommand.CommandText property (and reset DBCommand.Parameters if needed) and call DBDataAdapter.Fill() method.
Seems that everything is working pretty well.
I did it in this way to not waste memory comsuption creating every time a DBCommand.
What do you think about this solution? Any tips?
-=-=-
Michael Sogos
GuruStudioWeb
Da: Jiri {x2} Cincura
Data invio: lunedì 10 agosto 2015 10:22
A: nuodb/nuodb-dotnet
Cc: Michael Sogos
This is the "limitation" - design choice - of NuoDB's protocol. You cannot send multiple commands in batch. One ExecuteXxx is one statement.
—
Reply to this email directly or view it on GitHub.

Yep. That's viable.


fixes https://github.com/fabric8io/fabric8/issues/4059 so that we auto-default the user/password for gogs so that the jenkins DSL can reuse it. Many thanks to @jimmidyson for the changes to fabric8/gogs docker image: https://github.com/fabric8io/docker-gogs#creating-an-initial-admin-user

fixes https://github.com/fabric8io/fabric8/issues/4059 so that we auto-default the user/password for gogs so that the jenkins DSL can reuse it. Many thanks to @jimmidyson for the changes to fabric8/gogs docker image: https://github.com/fabric8io/docker-gogs#creating-an-initial-admin-user

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/524/
Test PASSed.


Stub test not working for imaging interface

This issue is a reminder to investigate why, with the current implementation, even if the stub test option is True, the imaging data are saved with the total number of frames.

Hmm, I would start debugging with checking what happens if you call frame_slice() on the imaging extractor
like : self.imaging_extractor.frame_slice(start_frame=0, end_frame=stub_frames)

Bug found, detail in https://github.com/catalystneuro/mousev1-to-nwb/pull/11#discussion_r1471677162
Fixed in 4b4259a72e7deb0c8e583f04139feb0c0237e09d

Bug found, detail in https://github.com/catalystneuro/mousev1-to-nwb/pull/11#discussion_r1471677162
Fixed in 4b4259a72e7deb0c8e583f04139feb0c0237e09d


toolset refactor

Most of this is a huge refactor that simplifies the Toolset struct massively. Before it was heavily coupled to the Config struct. Now it just manages runtime versions. There was code strewn throughout the codebase to fetch versions and manually install, this consolidates a ton of that logic.
This made it possible to add several UX improvements:

Parallel plugin and runtime installation (Fixes #72)
Show last line of install output in UI (Fixes #133)
Defaults to installing plugins alongside runtimes (Fixes #134 and Fixes #80)

It also lays some groundwork for fixing #98.
Originally I thought this would require a MV bump, but I think this is actually pretty safe to do even though it's a massive change under the hood. I don't think parallel install should mess up workflows (it can be disabled with --jobs 1 or with equivalent settings).
This also changes the behavior to make rtx install --all the default behavior, so it will automatically install plugins. While this is a significant change, I don't think it would be a breaking change for anyone. The flag is still there it just is hidden and won't do anything.
The downside of this is we can't just output things to stderr since we have a huge progress bar on top of everything, so we have to be careful to suspend it if we want to output things.

looks like I need to add openssl to the build image

this is turning into a big change, but it's got a lot of great UX improvements and should vastly simplify the codebase. Here's a vide of where it is at so far.
https://user-images.githubusercontent.com/216188/219876716-4efba553-2741-47a5-90ff-5cf5f053c170.mov

I think this is finally ready


error message is missleading

if rpm-devel is missing, a misleading error message is issues:
    [ERROR] CMD: [/usr/bin/rpm --query rpm-devel --quiet], Return Code: [1] at [/tmp/pip-build-fnv3a91e/rpm-py-installer]
    Traceback (most recent call last):
      File "install.py", line 581, in <module>
        main()
      File "install.py", line 576, in main
        app.run()
      File "install.py", line 35, in run
        self._verify_system_status()
      File "install.py", line 165, in _verify_system_status
        raise InstallError(message)
    __main__.InstallError: Required RPM not installed: [rpm-devel].
    Install it by "dnf install rpm-devel".

Which part of the message is misleading?
What message do you prefer for that?

Well, the first line is referring to RPM and the last to DNF... ;)

@goern Your opinion makes sense.
"the first line is referring to RPM and the last to DNF... ;)" should be fixed.
Yeah, I wanted to show users how to solve the error concretely in the error message.
Thought the solution might be rpm -i pkg_name, yum install pkg_name or dnf install pkg_name.

@goern as I fixed this issue by PR #77, I would close this.
But feel free to reopen if there is something.


BeanCreationException upgrading from 2.1.32 to 2.1.33 or later

I upgraded s3mock to the latest version and now get an exception when starting. The breakage seems to be from version 2.1.32 to 2.1.33 with all later versions exhibiting the same behaviour.
S3Mock is being started in a test class using the following code (kotlin):
S3MockApplication.start(mapOf(
        S3MockApplication.PROP_HTTPS_PORT to S3MockApplication.RANDOM_PORT,
        S3MockApplication.PROP_HTTP_PORT to S3MockApplication.RANDOM_PORT,
        S3MockApplication.PROP_INITIAL_BUCKETS to "bucket"
    ))

This results in the following exception:
org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'connectionFactory' defined in class path resource [org/springframework/boot/autoconfigure/r2dbc/ConnectionFactoryConfigurations$Generic.class]: Bean instantiation via factory method failed; nested exception is org.springframework.beans.BeanInstantiationException: Failed to instantiate [io.r2dbc.spi.ConnectionFactory]: Factory method 'connectionFactory' threw exception; nested exception is org.springframework.boot.autoconfigure.r2dbc.ConnectionFactoryBuilder$ConnectionFactoryBeanCreationException: Failed to determine a suitable R2DBC Connection URL
	at org.springframework.beans.factory.support.ConstructorResolver.instantiate(ConstructorResolver.java:657)
	at org.springframework.beans.factory.support.ConstructorResolver.instantiateUsingFactoryMethod(ConstructorResolver.java:637)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.instantiateUsingFactoryMethod(AbstractAutowireCapableBeanFactory.java:1341)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBeanInstance(AbstractAutowireCapableBeanFactory.java:1181)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:556)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:516)
	at org.springframework.beans.factory.support.AbstractBeanFactory.lambda$doGetBean$0(AbstractBeanFactory.java:324)
	at org.springframework.beans.factory.support.DefaultSingletonBeanRegistry.getSingleton(DefaultSingletonBeanRegistry.java:234)
	at org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:322)
	at org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:202)
	at org.springframework.beans.factory.support.DefaultListableBeanFactory.preInstantiateSingletons(DefaultListableBeanFactory.java:897)
	at org.springframework.context.support.AbstractApplicationContext.finishBeanFactoryInitialization(AbstractApplicationContext.java:879)
	at org.springframework.context.support.AbstractApplicationContext.refresh(AbstractApplicationContext.java:551)
	at org.springframework.boot.web.servlet.context.ServletWebServerApplicationContext.refresh(ServletWebServerApplicationContext.java:143)
	at org.springframework.boot.SpringApplication.refresh(SpringApplication.java:755)
	at org.springframework.boot.SpringApplication.refresh(SpringApplication.java:747)
	at org.springframework.boot.SpringApplication.refreshContext(SpringApplication.java:402)
	at org.springframework.boot.SpringApplication.run(SpringApplication.java:312)
	at org.springframework.boot.builder.SpringApplicationBuilder.run(SpringApplicationBuilder.java:140)
	at com.adobe.testing.s3mock.S3MockApplication.start(S3MockApplication.java:214)
        [redacted]
Caused by: org.springframework.beans.BeanInstantiationException: Failed to instantiate [io.r2dbc.spi.ConnectionFactory]: Factory method 'connectionFactory' threw exception; nested exception is org.springframework.boot.autoconfigure.r2dbc.ConnectionFactoryBuilder$ConnectionFactoryBeanCreationException: Failed to determine a suitable R2DBC Connection URL
	at org.springframework.beans.factory.support.SimpleInstantiationStrategy.instantiate(SimpleInstantiationStrategy.java:185)
	at org.springframework.beans.factory.support.ConstructorResolver.instantiate(ConstructorResolver.java:652)
	... 50 more
Caused by: org.springframework.boot.autoconfigure.r2dbc.ConnectionFactoryBuilder$ConnectionFactoryBeanCreationException: Failed to determine a suitable R2DBC Connection URL
	at org.springframework.boot.autoconfigure.r2dbc.ConnectionFactoryBuilder$ConnectionFactoryOptionsInitializer.connectionFactoryBeanCreationException(ConnectionFactoryBuilder.java:227)
	at org.springframework.boot.autoconfigure.r2dbc.ConnectionFactoryBuilder$ConnectionFactoryOptionsInitializer.initializeOptions(ConnectionFactoryBuilder.java:156)
	at org.springframework.boot.autoconfigure.r2dbc.ConnectionFactoryBuilder.of(ConnectionFactoryBuilder.java:62)
	at org.springframework.boot.autoconfigure.r2dbc.ConnectionFactoryConfigurations.createConnectionFactory(ConnectionFactoryConfigurations.java:51)
	at org.springframework.boot.autoconfigure.r2dbc.ConnectionFactoryConfigurations$Generic.connectionFactory(ConnectionFactoryConfigurations.java:90)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:566)
	at org.springframework.beans.factory.support.SimpleInstantiationStrategy.instantiate(SimpleInstantiationStrategy.java:154)
	... 51 more

Looking through the release notes and code diff between the two versions did not reveal anything that I would expect could cause an issue like this.

Not reproducible.
See #276 which adds a unit test that does what your code does (only it's in Java).
2.1.32 to 2.1.33 was actually a big change in dependencies. I updated almost every dependency we have, most notably the Spring Boot dependency, which in turn updates the Spring Framework.
Maybe I should have changed the minor or major version instead of the patch version for that release - if you're using the S3MockApplication directly, or one of the testsupport modules in your code from within a Spring application, this may break your project. :(
My guess: your project has some Spring Framework / Spring Boot dependency that includes an auto configuration for R2DBC. The newer Spring Boot/Framework version we provide in S3Mock now detects the autoconfiguration and tries to instantiate the beans as part of the S3Mock.

Can you run Docker in your build?
I just added a testsupport module for TestContainers (will be released later today).
If you start through the Java API, there will always be the risk that some dependency in your project will break the S3Mock, and vice versa.
Using Docker in your tests will insulate both applications dependencies and application life cycles.

Ahhh, I was wondering how much was hiding behind those parent pom changes. Thanks for looking into this. I noticed your testcontainer support when I was investigating and thought that would be a much better solution. I'll go down that path instead.
The project I'm working in doesn't include any spring dependencies other than the ones pulled in by s3mock. The only dependencies which are changed from what s3mock would have by default are log4j (upgraded to 2.14.1) and jackson (upgraded to 2.12.4). My project does use jooq, which depends on io.r2dbc:r2dbc-spi:0.9.0.M1, I wonder if that might have caused the issue.
If you want to spend any more time looking into this issue, I'm happy to help where I can. Otherwise, this can probably be closed. Either way, I'll probably move to the testcontainers option.

My project does use jooq, which depends on io.r2dbc:r2dbc-spi:0.9.0.M1, I wonder if that might have caused the issue.

Spring is always very clever on which parts of its own configuration to use and which beans to instantiate.
There's a  @ConditionalOnClass(ConnectionFactory.class) (on io.r2dbc.spi.ConnectionFactory to be exact) that was triggered because the class is available in your classpath when starting the S3MockApplication:
https://github.com/spring-projects/spring-boot/blob/main/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/r2dbc/R2dbcAutoConfiguration.java#L38
I just released 2.2.0 for TestContainers support. :)


Feature+Refactor: Distribute fields across processes with MPI

Still need to make distributed fields work with even-odd preconditioned fields, and just make even-odd preconditioned Wilson-Clover work in general


Adding backbone to dependencies causes underscore error

When adding "bower": ['backbone'] to a js dependency, I'm getting the following error in my browser console as soon as the libs.js is loaded: "Uncaught TypeError: Cannot read property 'extend' of undefined" where _ is what is undefined.
However, I can properly use _ from the browser console, so underscore.js is indeed loaded.
Here's my manifest.json:
{
  "dependencies": {
    "main.js": {
      "files": [
        "scripts/includes/**/*.js",
        "scripts/main.js"
      ],
      "main": true
    },
    "jquery.js": {
      "bower": ["jquery"]
    },
    "libs.js": {
      "bower": [
        "modernizr",
        "foundation",
        "backbone"
      ]
    },
    "main.css": {
      "files": [
        "styles/main.scss"
      ]
    },
    "editor-style.css": {
      "files": [
        "styles/editor-style.scss"
      ]
    }
  },
  "config": {
    "devUrl": "https://dev.example.com"
  },
  "paths": {
    "source": "assets/",
    "dist": "dist/",
    "bower" : "bower_components/"
  }
}

Any idea why _ would be undefined when simply including/injecting the backbone bower dependency?
Thanks!
gkn

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/27069834-adding-backbone-to-dependencies-causes-underscore-error?utm_campaign=plugin&utm_content=tracker%2F8853550&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F8853550&utm_medium=issues&utm_source=github).

Update:  The following manifest.json does load up Backbone and Underscore correctly, but only works pointing directly to the backbone.min.js file with external: true.  Additionally, even though the backbone script is referenced from the libs.js file I want to export, it is actually included in main.js.
{
  "dependencies": {
    "main.js": {
      "files": [
        "scripts/includes/**/*.js",
        "scripts/main.js"
      ],
      "main": true
    },
    "jquery.js": {
      "bower": ["jquery"]
    },
    "libs.js": {
      "files": [
        "bower_components/backbone/backbone.min.js"
      ],
      "external": true,
      "bower": [
        "modernizr",
        "foundation"
      ]
    },
    "main.css": {
      "files": [
        "styles/main.scss"
      ]
    },
    "editor-style.css": {
      "files": [
        "styles/editor-style.scss"
      ]
    }
  },
  "config": {
    "devUrl": "https://example.com"
  },
  "paths": {
    "source": "assets/",
    "dist": "dist/",
    "bower" : "bower_components/"
  }
}


⚠ Thumbnails API Endpoint has degraded performance

In 3d34421, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 2661 ms

Resolved: Thumbnails API Endpoint performance has improved in 54431f9 after 19 minutes.


There exists a bug about SDL version when I run the hellopbr sample

Describe the bug
There exists a bug about SDL version when I run the hellopbr sample
To Reproduce
Steps to reproduce the behavior:

~/ge/filament$ ./build.sh debug
~/ge/filament$ ./out/cmake-debug/samples/hellopbr

Expected behavior
show a window display the sample
Screenshots

Desktop (please complete the following information):

OS: Linux Ubuntu
GPU: NVIDIA GTX 1060
Backend: OpenGL

Additional context
The output information as follows:
utils::PostconditionPanic
in void *getNativeWindow(SDL_Window *):27
in file filament/libs/filamentapp/src/NativeWindowHelperLinux.cpp
reason: SDL version unsupported!
#0      hellopbr                            0x7f445631776d (null) +<PHONE_NUMBER>86093
#1      hellopbr                            0x7f445632e32b (null) +<PHONE_NUMBER>79211
#2      hellopbr                            0x7f445632e0ef (null) +<PHONE_NUMBER>78639
#3      hellopbr                            0x7f445632dfc1 (null) +<PHONE_NUMBER>78337
#4      hellopbr                            0x7f4455dbc155 (null) +<PHONE_NUMBER>68917
#5      hellopbr                            0x7f4455dac1be (null) +<PHONE_NUMBER>03486
#6      hellopbr                            0x7f4455da8638 (null) +<PHONE_NUMBER>88248
#7      hellopbr                            0x7f4455d765bb (null) +<PHONE_NUMBER>83323
#8      libc.so.6                           0x7f44559e9d90 (null) +<PHONE_NUMBER>62064
#9      libc.so.6                           0x7f44559e9e40 __libc_start_main + 128
libc++abi: terminating due to uncaught exception of type utils::PostconditionPanic
Aborted (core dumped)

    SDL_SysWMinfo wmi;
    SDL_VERSION(&wmi.version);
    ASSERT_POSTCONDITION(SDL_GetWindowWMInfo(sdlWindow, &wmi), "SDL version unsupported!");

Looks like we couldn't even get window info and/or create the window.

Are you running wayland?  (check with echo $XDG_SESSION_TYPE).

Are you running wayland? (check with echo $XDG_SESSION_TYPE).
I am not running in wayland. And the result of the code echo $XDG_SESSION_TYPE is blank.

I see, then are you running WSL (Windows Subsystem for Linux)? I don't think this is a supported path for compilation. You can try the regular windows build: https://github.com/google/filament/blob/main/BUILDING.md#windows

I see, then are you running WSL (Windows Subsystem for Linux)? I don't think this is a supported path for compilation. You can try the regular windows build: https://github.com/google/filament/blob/main/BUILDING.md#windows
The filament in windows running well. Maybe it is the WSL that occurs the problem. I will try run filament in a separate Linux.

Do not run filament in  WSL (Windows Subsystem for Linux). Because WSL lacks a lot of packages.


Add /status endpoint

This adds new /status endpoint which should report current status of relay, i.e. list of endpoints, amount of buffer used and whether the buffering is active or not.
That info is crucial for implementing proper monitoring of relay, predicting acceptable downtime (as mentioned in #49) and scaling.
Side effect of this patch is refactor of jsonError function to a more general jsonResponse. We've been using a variation of this patch for a couple of months in production with no noticeable side-effects.
Example output (using sample_buffered.toml) with local1 instance being available and local2 being unreachable:
$ curl -s localhost:9096/status |python -m json.tool
{
    "status": {
        "local1": {
            "buffering": "0",
            "maxSize": "104857600",
            "size": "0"
        },
        "local2": {
            "buffering": "1",
            "maxSize": "104857600",
            "size": "75"
        }
    }
}

This is great! I was just looking to go about a patch for something similar. Shame this doesn't appear to have been merged yet...

The /status endpoint would be great.
Unfortunately, these changes cause influxdb-relay to randomly crash.
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x66a4b4]
goroutine 39820761 [running]:
github.com/influxdata/influxdb-relay/relay.(*retryBuffer).post(0xc42013ca00, 0xc422eb9980, 0x1886, 0x18cd, 0xc421e90b60, 0x1e, 0xc422a5b0b0, 0x26, 0x0, 0x0, ...)
/root/go/src/github.com/influxdata/influxdb-relay/relay/retry.go:76 +0xd4
github.com/influxdata/influxdb-relay/relay.(*HTTP).ServeHTTP.func1(0xc423ea9120, 0xc4201518e0, 0xc422eb9980, 0x1886, 0x18cd, 0xc421e90b60, 0x1e, 0xc422a5b0b0, 0x26, 0xc420154400, ...)
/root/go/src/github.com/influxdata/influxdb-relay/relay/http.go:232 +0xd3
created by github.com/influxdata/influxdb-relay/relay.(*HTTP).ServeHTTP
/root/go/src/github.com/influxdata/influxdb-relay/relay/http.go:241 +0xc44

That is quite possible. I'd suggest using one of the active forks of this project.

Hey Marko,
Which fork do you suggest?
Regards,
Tim Raphael

On 20 Feb 2018, at 1:58 am, Marko Crnic<EMAIL_ADDRESS>wrote:
That is quite possible. I'd suggest using one of the active forks of this project.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Can't really suggest any of them as I haven't tried them. Current version with this patch works for my use-case.

Hello, we're planning to maintain an active fork here: https://github.com/vp-noc/influxdb-relay.
We've done the job inspired by your PR.

This PR is now merged on https://github.com/vente-privee/influxdb-relay/tree/develop.


ARROW-10859: [Rust] [DataFusion] Made collect not require ExecutionContext

This PR observes that ExecutionContext::collect(&self, plan: Arc<dyn ExecutionPlan>) does not use self on its implementation.
Using this observation, it refactors out collect out of ExecutionContext (into physical_plan::collect), thereby simplifying the execution of the plans, by not requiring creating an ExecutionContext just to execute a physical plan.
From a design's perspective, this makes it obvious that the execution of a physical plan is entirely independent of a context and its state.

Codecov Report

Merging #8874 (562bc1f) into master (7d509dc) will increase coverage by 0.05%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #8874      +/-   ##
==========================================
+ Coverage   52.90%   52.96%   +0.05%
==========================================
  Files         171      172       +1
  Lines       30655    30687      +32
==========================================
+ Hits        16218    16253      +35
+ Misses      14437    14434       -3

Impacted Files
Coverage Δ

rust/datafusion/examples/flight_server.rs
0.00% <0.00%> (ø)

rust/datafusion/src/execution/context.rs
0.00% <ø> (ø)

rust/datafusion/src/execution/dataframe_impl.rs
0.00% <0.00%> (ø)

rust/datafusion/src/physical_plan/filter.rs
0.00% <ø> (ø)

rust/datafusion/src/physical_plan/mod.rs
0.00% <ø> (ø)

rust/datafusion/src/physical_plan/sort.rs
0.00% <ø> (ø)

rust/arrow/src/array/transform/fixed_binary.rs
42.10% <0.00%> (ø)

rust/arrow/src/array/array_binary.rs
90.82% <0.00%> (+0.20%)
:arrow_up:

rust/arrow/src/array/equal/mod.rs
91.50% <0.00%> (+0.37%)
:arrow_up:

rust/arrow/src/array/transform/mod.rs
76.85% <0.00%> (+0.80%)
:arrow_up:

Continue to review full report at Codecov.

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


Failed to connect to WSL over tunnels

VS Code Version:
Version: 1.82.0-insider (user setup)
Commit: 83fc3ad9bd553869e8fea3d370d1d2bd438f9e54
Date: 2023-08-04T05:35:09.023Z
Electron: 22.3.18
ElectronBuildId: 22689846
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.25387

OS Version: Windows 11 Insider

Steps to Reproduce:

Start a tunnel via CLI on Windows machine: code-insiders tunnel
Open VS Code Insiders, open Remote Explorer
In Tunnels/SSH view, select connect to WSL under the tunnel that's running
VS Code reloads, but gets stuck on Starting VS Code in WSL (Ubuntu-20.04)
See in the VS Code terminal: VS Code CLI is running in WSL... [2023-08-04 16:16:46.473] Installing server in Ubuntu-20.04
See in my Windows terminal where I'm running the tunnel: [2023-08-04 09:16:46] warn [rpc.1] failed to authenticate with child process failed to handshake spawned process: Custom { kind: InvalidData, error: "invalid type: integer 70, expected struct PartialIncoming" }

I'm not able to reproduce this. Can you get the output from the WSL log on the window you're connecting from?

String.fromCharCode(70) is "F" so I think rather than the protcol bash logged some "Failed to execute..." process when it tried running the CLI in WSL. But I would need a log to help diagnose things 🙂

Yeah sorry about not grabbing the full log! I'll be sure to share if this happens again. I just tried this flow again in a few scenarios (killed the server and ran again, with and without refreshing the list of tunnels in the Remote Explorer), and I still can't repro.
Also just to double check: Should the remote explorer only say WSL rather than WSL // tunnel  <x> or WSL: Ubuntu // tunnel: <x>? Is there a feature request for showing more info (similar to how we show container and SSH info when using Dev Containers + Remote-SSH together)?

good reminder -- had to do the last approval on the build from yesterday that fixes that


Add Matrix Chat to Community Dropdown

I spread the notion to add the Unofficial Minetest Riot/Matrix Chat to the Minetest.net Website.
Part of issue: #136
Dear Maintainers, feel free to make edits to this pull request.

Matrix is not an official chat channel


Unable to attach a http server with aedes server , like mosca's attachHttpServer . Any workaround ?

This is not in the scope of this project, and it will be integrated with Mosca soon. You can achieve the same thing by using http://npm.im/websocket-stream.

I am sending my client side packet using mows.js and I am trying to pass the incoming stream to aedes using websocket-stream . Here is my code snippet
`var http = require('http')
var websocket = require('websocket-stream')
var RedisPersistence = require('aedes-persistence-redis')()
var aedes = require('aedes')({
concurrency: 1000,
persistence : RedisPersistence
})
var httpServ = http.createServer()
var wss = websocket.createServer({server: httpServ},handle)
function handle(stream){
stream.pipe(server)
}
var server = require('net').createServer(aedes.handle)
var port = 1884
httpServ.listen(port, function (req,res) {
console.log('server listening on port', port)
})`

You have to bind aedes to wss, not a new tcp server.
var http = require('http')
var websocket = require('websocket-stream')
var RedisPersistence = require('aedes-persistence-redis')()
var aedes = require('aedes')({
    concurrency: 1000,
    persistence : RedisPersistence
})

var httpServ = http.createServer()
var port = 1884

var wss = websocket.createServer({server: httpServ, port: port}, aedes.handle)

This should work

@alliv8 explained it perfectly.

Got a clear idea , Thanks @alliv8  .


Is this project mysql only?

Seems don't support postgresql ? Just want to confirm that.

@kuchaguangjie see: https://github.com/casbin/casdoor#config
Casdoor uses XORM to connect to DB, so all DBs supported by XORM can also be used.

XORM supports a lot of DBs including PostgreSQL: https://gitea.com/xorm/xorm#drivers-support

It does support pg, but require some modification to work, since the code include some mysql specific config.
Refer to my fork & branch for postgres: https://github.com/kuchaguangjie/casdoor/tree/postgres
The changes I made to make it work with postgresql:
1. Create db by hand.
2. Edit conf/app.conf

dataSourceName = "user=postgres password=xxxxxx host=<IP_ADDRESS> port=5432 dbname=casdoor sslmode=disable"

3. Edit object/adapter.go

In InitAdapter()
Change:

adapter = NewAdapter("postgres", beego.AppConfig.String("dataSourceName"))

In open()
Remove:

if err := a.createDatabase(); err != nil {
panic(err)
}

Change:

engine, err := xorm.NewEngine(a.driverName, a.dataSourceName)`

4. Edit authz/authz.go

In InitAuthz()
Change:

a, err := xormadapter.NewAdapter("postgres", beego.AppConfig.String("dataSourceName"), true)

Then it will able to start.

BTW, I think the code could be refactored to support various db without changes. As you mentioned, it use XORM, and the dataSourceName config is for github.com/casbin/xorm-adapter/v2.

@Kininaru


Wrong slot texture in induction Furnace

Thank you very much for reporting!
the issue will be fixed in the next release


CoordinateExtensions.ToLinearRing: remove exception

This PR is analogous to #1521 and should fix https://github.com/Mapsui/Mapsui/issues/1520#issuecomment-1564929374.

I tested the generated nupkgs, and after the second commit 'polygon.Positions.Clear()' works as expected.


Last values could be expanded to subsection of the buffer

Instead of only providing statistics on the last n elements of the buffer, these functions could return info on a subset of the buffer.
Like:
float getAverageLast(uint16_t start, uint16_t count) get the average of count elements after start.
Making the functions overflow so that one value gives the last elements and two values gives this subset.
Would be useful to compare various regions in the buffer and also when doing a running average comparison between now and x amount of recordings ago. I now used to store multiple rolling buffers to do this.
I might have a crack at it later and do a pull request. But now have to finish my project and this way i do not forget the idea!

Thanks for the ideas,
A PR would be great of course!

Some points:

the internal buffer is a ring buffer, the function will need modulo math for the start and count
count should be checked again internal count
if  partial is used it should be in the equation somewhere

something like this?
// returns NAN if no elements.
float RunningAverage::getAverageSubset(uint16_t start, uint16_t count)
{
  if (_count == 0)
  {
    return NAN;
  }

  uint16_t cnt = _count;
  if (cnt > count) cnt = count;

  float sum = 0;   //  do not disrupt global _sum
  for (uint16_t i = 0; i < cnt; i++)
  {
    uint16_t idx = _index + start + i;
    while (idx >= _partial) idx -= _partial;
    sum += _array[idx];
  }
  return sum / cnt;
}

I made this for my project back then. But still have to verify if it works. (ended up using a different method to track changes over time).
Using the same methodology as i found in the getAverageLast function before.
float RunningAverage::getAverageLast(uint16_t count){
    getSubsetAverage(_count-count, count);
}

float RunningAverage::getSubsetAverage(uint16_t start, uint16_t count)
{
    uint16_t cnt = count;
    if (cnt > _count) cnt = _count;
    if (cnt == 0) return NAN;

    float sum = 0;
    uint16_t idx = _index + start + count;
    idx = idx % _size;
    for (uint16_t i = 0; i < cnt; i++)
    {
        if (idx == 0) idx = _size;
        idx--;
        sum += _array[idx];
    }
    return sum / cnt;
}

Main reason i did not post it yet and wanted to test was to see if the idx--; had to be before or after the sum to be correct.
And i was trying to make both function the same code. With one calling the other.
But you choose as developer obviously. To me these functions for subset average, subset min and max would be a great addition.

The best way is to write a test sketch that verifies some known values and compare the old functions with the new wrapper based.
Also important is to check border cases in such sketch. Think of count is zero, no elements added, less elements than requested.
I try to have such checks in the unit test in TEST folder. If you create a PullRequest and add these tests too, the new functions are automatically verified with every change/ release.

@jokohoko
Please add also examples to the Pull Request

@jokohoko
Any progress to report?

@jokohoko
As this issue is open for about a month without PR from your side I close it.
Feel free to reopen it if you have your PR ready.


Defining Example objects

Is there any way to generate example objects of operation responses, as foreseen by the specification?

This scenario is currently not supported.

Is this a duplicate of https://github.com/NSwag/NSwag/issues/47?

Yes it is.
The scope is to find a way to create request/responses samples to be used in automated creation of api documentation.

Can you post a sample? i.e. a sample of an expected Swagger spec?

What do you mean by sample?
I would like to create object samples and specify them statically at the method level. One option could be to extend the ResponseType attribute to receive on abjiect which specifies the mime type and and object sample. For example, instead of this
[ResponseType( "200", typeof( BaseServiceResult ), Description = "Success" )]
it can be done like this
[ResponseType( "200", typeof( BaseServiceResult ), Description = "Success",
                          new ExampleObject( "application/json", new {
                             propertyA = "",
                             propertyB = 123
                          } ) )]

However this is not the best solution at all because you need to specify an object for every ResponseType. In my case I can live with this

I think you cannot create a new instance of an own class in an attribute (limitation in C#)...

See

http://swagger.io/specification/#responseObject
http://swagger.io/specification/#exampleObject

I think you cannot create a new instance of an own class in an attribute (limitation in C#)...

Right. What about passing a stringified json to be parsed internally?

Yes, this would work...

1

Maybe this solves your problem?
https://github.com/RSuter/NSwag/blob/master/src/NSwag.SwaggerGeneration.WebApi.Tests/OperationProcessors/SwaggerOperationProcessorAttributeTests.cs

Could you please add a sample usage?

https://blog.rsuter.com/nswag-tutorial-implement-a-custom-operation-processor-to-define-redoc-code-samples/

I had the same requirement and am currently comparing and deciding on whether to use Swashbuckle or NSwag.  There is a way to do this in Swashbuckle described here, don't know if that's any help here.
Contrary to the comment about this not changing anything in the Swagger UI, it actually did also render my example response under the "Example Value" section of Response Class.
The generated swagger specification then looks something like this:
"definitions": {
	"User": {
		"description": "External representation of a User",
		"type": "object",
		"properties": {
			"name": {
				"description": "User name.",
				"type": "string"
			},
			"language": {
				"description": "User's language.",
				"type": "string"
			},
			"age": {
				"description": "user's age.",
				"type": "int"
			}
		},
		"example": {
			"name": "Bob",
			"language": "en-GB",
			"age": 36,
		}
	}
}

You can also implement a custom processor for NSwag: https://blog.rsuter.com/nswag-tutorial-implement-a-custom-operation-processor-to-define-redoc-code-samples/

Does that work for SwaggerUI as well as redoc?

I think x-code-samples is specific to ReDoc, but you can add any additional properties with a custom processor - you have to check the Swagger UI documentation what properties are supported...

Ref: https://stackoverflow.com/questions/50466680/how-can-i-generate-a-custom-response-example-value-in-nswag-asp-net-core

Why not extract the  tag like it is done for the  tag on each of the property ?


[Question]: Count number of nodes in VectorIndex that are similar above a similarity threshold

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
One can do _vector_store_index.as_retriever(similarity_top_k=desired_number_of_results).retrieve(query_text) to get top k nodes that have text similar to query_text. How to count total number of nodes in the Vector store index that have text similar to query_text above a similarity threshold?

Use a huge top k, filter from there?

Use a huge top k, filter from there?
Doesn't work with huge top k; fails to find even a single node then.

Well that doesn't really make sense? index.as_retriever(similarity_top_k=1000) ?
I don't think there's any other option


[BugFix] Avoid calling GetReadStatistics function when accessing juicefs to avoid be crash.

What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

As the GetReadStatistics is not supported in juicefs hadoop sdk, which will cause the be crash when visiting juicefs. So we avoid to call this function for juicefs.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[x] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_admit_test

better to rename it like is_support_get_stats . But let's refactor it next time.

@Mergifyio backport branch-2.5

@Mergifyio backport branch-2.4


Duplicates in array

Description 📜
I had explained the problem statement ,what Floyd's Cycle detection algorithm is, and how it can be applied in solving this problem. I had included the intuition behind it, algorithm and also C++ code which was commented wherever necessary.
Fixes #4685

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[x] DSA
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x]  I have commented my code, particularly in hard-to-understand areas.
[x]  My changes generates no new warnings.
[x]  I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@japneetbhatia Pls review my pr. I had made the suggested changes.

@Eswari-Priya Pls add references and we are good to go.


Add initial Kotlin support

Currently, some of Kotlin's features do not work in NativeScript, most notably the Companion objects. This PR enables their usage. The PR also adds JS tests for Kotlin specific language features which do not have an analog in Java.
The PR ensures Kotlin features could be used in JS the same way Kotlin could be used in Java projects. Calling Kotlin from Java is well described in the official Kotlin docs: https://kotlinlang.org/docs/reference/java-to-kotlin-interop.html
Syntax for calling Kotlin properties in JS as if they have been fields is also added with the restriction that properties whose names start with 'is' should be used by calling their backing method as in if they were used in Java.

test


Add inversion mode to chord quiz

The chord quiz could benefit from having a setting where chord inversions are explicitly tested. Currently, any inversion will work for every chord, but it would be a nice addition to train inversions specifically.

Hello @j-vb
Sorry i let MIDI Jar development aside for some months.
I have this features in my mind already, but i still have issues working on MIDI Jar again.
I'll try to keep in touch with this.

Hello, this is my top 2 priority, I’ll try my best to keep you updated as I’m trying to get back to developing MIDI Jar features.

Hello @ArTiSTiX
Thank you for the reply, and I'm glad to hear that you've found time for MIDI Jar. I'm looking forward to future improvements!


Solidity verifier does not accept optimization mode

🐛 Bug Report
📝 Description
Calling the verifier api with a contract compiled using any optimization mode except "3" will not result in the correct compilation from the verifier.
🔄 Reproduction Steps
Compile a contract with zksolc optimizer mode "z"
Use hardhat verifier, manually or on explorer.
🤔 Expected Behavior
Verifier should pass and verify the contract correctly
😯 Current Behavior
Verifier errors: Deployed bytecode is not equal to generated one from given source
🖥 Environment
Any relevant environment details.
📋 Additional Context
This seems to happen because for building the solc input, the verifier does not pass on optimizerMode like it does for the vyper input:
https://github.com/matter-labs/zksync-era/blob/main/core/bin/contract-verifier/src/verifier.rs#L304
See implemented correctly for vyper: https://github.com/matter-labs/zksync-era/blob/main/core/bin/contract-verifier/src/verifier.rs#L304
📎 Log Output
Error in plugin @matterlabs/hardhat-zksync-verify: Backend verification error: Deployed bytecode is not equal to generated one from given source

ZkSyncVerifyPluginError: Backend verification error: Deployed bytecode is not equal to generated one from given source

let settings = Settings {
libraries: None,
output_selection: Some(default_output_selection),
optimizer: Optimizer::new(request.req.optimization_used),
is_system: request.req.is_system,
metadata: None,
};

Hi @monokh, thanks for raising it! This a bug indeed and I will prepare a fix soon, but it affects only solidity-single-file verification mode, while solidity-standard-json seems to be working correctly with optimizer modes. AFAIK, hardhat plugin uses only solidity-standard-json so you should be able to verify contract with non-default optimizer mode using plugin

Hi @monokh, thanks for raising it! This a bug indeed and I will prepare a fix soon, but it affects only solidity-single-file verification mode, while solidity-standard-json seems to be working correctly with optimizer modes. AFAIK, hardhat plugin uses only solidity-standard-json so you should be able to verify contract with non-default optimizer mode using plugin

Thanks for looking into this
You are right, the hardhat plugin is using standard json input.
The way i found this issue is using single solidity file, especially on the zksync explorer and etherscan.
The zksync explorer doesn't support standard json input and the etherscan explorer doesn't allow me to upload because the size of the input (3.2 MB) is above the allowed size (3MB) :/
Yet I am still unable to verify our contract, I have checked that the compiler versions match through the verification payload and build.json
Reproducable example here if you would like to try it: https://github.com/bebop-dex/bebop-jam-contracts?tab=readme-ov-file#deploying-to-zksync

I should also mention that our contract deploys another contract in the constructor
Does this cause an issue with verification? I believe that i saw an issue somewhere about this

Thanks, it should be enough to investigate, let me come back later after I try it

sorry had the mainnet adress wrong 😅  I've updated it - correct is 0x574d1fcF950eb48b11de5DF22A007703cbD2b129

@monokh we've found a problem in hardhat verification plugin. Problem was triggered because you are using hardhat-preproccesor plugin. For now please verify contracts compiled with this plugin with no-compile flag, i.e.npx hardhat verify --no-compile ....


color problem with tmux git version

I cloned tmux from github then install and run it, i found that there is no color on the foot bar.
like this:

Does it mean it's not support git version of tmux?
OS: 4.8.15-300.fc25.x86_64
.tmux: latest from git

Have you ever install and configure the powerline?
Read the configuration of this repositry may help.
The configuration of fonts is also important. You can also find the way in the link above. By the way, it is a good idea to install the specified font via otf file by this repository.
FYI.

Yes, i have installed and configure the powerline. The fonts also be installed.
If i use the tmux which was installed by dnf command, everything is ok.

What about using xterm-256color. See it.

outside of tmux, $TERM must be set to xterm-256color.

Hope it helps you.

Yes, it's already xterm-256color.
I'm using xshell, and set the terminal type by Properties->Terminal->Terminal Type.
After login by ssh, i confirm the term type by echo $TERM, and it shows xterm-256color.

Do you have colors when not using this configuration?
Make sure you kill all tmux processes then try
$ tmux -f /dev/null
$ tmux set -g status-right '#[fg=#e4e4e4,bg=#d70000,none]TEST'

You should see the word TEST written in white on a red background.

found the problem.
'tmux_conf_theme_24b_colour=true' was the reason.
but for tmux 2.2, it's ok.

tmux_conf_theme_24b_colour=true works for me, running macOS Sierra

@gpakosz can you share the script (24-bit-color.sh)?
By the way, is this
powerline

? how to config it?

Here is 24-bit-color.sh: https://gist.github.com/gpakosz/4f5a87739c9ef45bce2f8be870450c1d
Here is an article describing how I configure my prompt: http://pempek.net/articles/2013/10/27/pretty-elided-shell-prompt/ (sorry for the poor SSL certificate, I'm about to switch to let's encrypt)

@gpakosz Thank you so much. I'll try it.

Any news?

Seems that my terminal can't display 24bit color correctly(by run the script of 24-bit-color.sh). So i set tmux_conf_theme_24b_colour=false to resolve this problem.
Thanks.


Setting deterministic lockstep changes the time step for the physics engine

Check out the deterministic lockstep PG: https://www.babylonjs-playground.com/#DU4FPJ#3.
Changing the value for lockstepMaxSteps  affects the speed of the physics simulation.

Ping @RaananW

That PG gets faster and faster on each run even without changing lockstepMaxSteps.

The PR gets faster due to a problem with creating an engine and deleting it. This addresses the issue - https://github.com/BabylonJS/Babylon.js/pull/3300

Are these two distinct issues? Running locally (without deleting the engine), I'm still seeing a direct correlation between render speed and lockstepMaxSteps.

Yep, those are two distinct issues. I already found an error in the calculation of the deterministic step (will probably commit tomorrow), I just wanted to be sure we can test it correctly.

I have pushed a fix. I am pasting my explanation from the PR here just for reference:
We should notice how deterministic step is working, and how we deal with max steps. If the scene is running at 20 FPS and we set the nax step to 3 and up, the scene will run in "simulated" 60 fps (as the max FPS target is 60 fps). if, however, we will set it to 1 or 2, the simulated fps will be lower (20 or 40 respectively).


Metadata-only update for ipywidgets 8, licenses

Thanks for this package. Here are some updates to:

better support ipywidgets (throws warnings now)
support more recent JupyterLab features like listing bundled licenses
be more discoverable on PyPI with trove classifiers

Changes

[x] adds @jupyter-widgets/base": "... || ^6
[x] adds some JupyterLab trove classifiers
[x] bumps @jupyterlab/builder to get third-party-licenses.json built

Screens

I don't fully understand the test fails, but otherwise looks good on binder, etc.

The tests are a bit unmaintained.
Trying to get the time to work on ipycytoscape, but ti's been hard.
I'll probably apply to get a NumFOCUS grant on this, so I can teach someone what I know and pay them to do some work on it though :) and have another person to take care of it with me.
Thanks for the PR @bollwyvl, this is great and highly appreciated! ✨


Make max_sequence_length config check optional.

This does not seem to get set for most cases since the config constructed by the model creation is never actually passed back out and therefore this value does not get set for llama models.
Was getting while building my llama model
Traceback (most recent call last):
  File "/home/noah/anaconda3/envs/tvm-build-venv/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/home/noah/anaconda3/envs/tvm-build-venv/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/home/noah/workplace/mlc-llm/mlc_llm/build.py", line 13, in <module>
    main()
  File "/home/noah/workplace/mlc-llm/mlc_llm/build.py", line 10, in main
    core.build_model_from_args(parsed_args)
  File "/home/noah/workplace/mlc-llm/mlc_llm/core.py", line 578, in build_model_from_args
    mod = mod_transform_before_build(mod, param_manager, args, config)
  File "/home/noah/workplace/mlc-llm/mlc_llm/core.py", line 361, in mod_transform_before_build
    max_seq_len = config["max_sequence_length"]
KeyError: 'max_sequence_length'

This still results in an error when you don't explicitly pass in the max-seq-len arg:
  File "/home/sudeepag/qa_latest/mlc-llm/mlc_llm/core.py", line 368, in mod_transform_before_build
    max_seq_len)
UnboundLocalError: local variable 'max_seq_len' referenced before assignment

oh right I've only checked vicuna 7B and 13B and thought all models have this config. But what is the expected behavior when both max_sequence_length and the cmd line max-seq-len are not provided?

Updated to handle the case where the config is not set by the model with an error and also have the model pass back a model_config from the builder to be used for this.

This change is causing an error when building Llama models:
Traceback (most recent call last):
  File "/home/sudeepag/.conda/envs/sudeep-mlc-llm/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/home/sudeepag/.conda/envs/sudeep-mlc-llm/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/home/sudeepag/qa_latest/mlc-llm/mlc_llm/build.py", line 13, in <module>
    main()
  File "/home/sudeepag/qa_latest/mlc-llm/mlc_llm/build.py", line 10, in main
    core.build_model_from_args(parsed_args)
  File "/home/sudeepag/qa_latest/mlc-llm/mlc_llm/core.py", line 583, in build_model_from_args
    mod = mod_transform_before_build(mod, param_manager, args, model_config)
  File "/home/sudeepag/qa_latest/mlc-llm/mlc_llm/core.py", line 362, in mod_transform_before_build
    if "num_attention_heads" in config and "hidden_size" in config:
TypeError: argument of type 'LlamaConfig' is not iterable


countdown timer not runing multiple times

I need to re run the timer by getting the time from server.
when time reaches 0.
.on('finish.countdown', function(event){})
I am calling a function which uses ajax to get the data and I am updating the time.
It works perfectly fine for 2 times/ 3times. after that finish.countdown is not triggering.
Any solution for that ?

hi @rjsnh1522  if you need server time to start counting, just passed time from server not all script, put server unixtime in variable or put server time in body attribute and in your countdown get that and add you needed time to this
AJAX called and fetch total script not good and made problems


Set up cascading writes for the article author table

Currently, the authors of articles are stored as one long string in the database. This is not optimal, and it's better to have an authors table where we document authors. (Realized while working on #99)

I'm having second thoughts about setting up a table for article authors, as there are no good ways to determine the uniqueness of the authors. For example, if John Doe wrote an Article A, and John Doe wrote an Article B, how can we determine that these people are the same?
Maybe a better way to approach this is to store the author information as a JSON field in the Articles table.

I'm closing this as wont-fix since this is not a feature that we will implement soon.


fix: define the service account namespace for the annotate commands

these service accounts exist in the jx namespace
fixes the problem mentioned in https://github.com/ankitm123/terraform-aws-eks-jx/pull/47

@hutchic Sorry, just saw the PR now, but you should open it in the main repo. This is just my fork.


RSetCache to support update expiration

Like RMapCache can update tll with updateEntryExpiration method.
There is no similar method in RSetCache

RSetCache.add() method invocation for existing element will update the ttl value.

RSetCache.add() method invocation for existing element will update the ttl value.

I don't add it if it not exist, but want update ttl if exist

Done


Firebase Messaging Game Stuck on iOS

Hi @stewartmiles
I am using Firebase Messaging SDK in unity app. After the pop up comes to allow notifications the game is totally stuck and i select any of the options the game is completely stuck. Infact its happening for anytime the app goes in background. Here are the last logs which i can see.
-> applicationWillResignActive()
2016-12-30 15:37:27.326132 pokersquare[27637:1515918] FCM: Connected to FCM
2016-12-30 15:37:27.327797 pokersquare[27637:1515918] You are using download over http. Currently unity adds NSAllowsArbitraryLoads to Info.plist to simplify transition, but it will be removed soon. Please consider updating to https.
FCM: Connected to FCM
(Filename: /Users/builduser/buildslave/unity/build/artifacts/generated/common/runtime/UnityEngineDebugBindings.gen.cpp Line: 42)
2016-12-30 15:37:28.684: <FIRInstanceID/WARNING> APNS Environment in profile: development
WARNING -> applicationDidReceiveMemoryWarning()
Please Help. I can see same issue here
http://stackoverflow.com/questions/41177624/firebase-unity-sdk-100-hang-on-ios-when-the-application-resumes

We're investigating this issue at the moment.  We'll keep you posted.

@stewartmiles  any update on by when we can have a fix for it.

We're investigating the issue at the moment.  We're hoping very soon.
On Tue, Jan 3, 2017 at 10:43 PM, AIGameStudio<EMAIL_ADDRESS>wrote:

@stewartmiles https://github.com/stewartmiles any update on by when we
can have a fix for it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/quickstart-unity/issues/33#issuecomment-270304929,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFw1RCYILA43petZUiwH3CevFDz_lb9gks5rOz-ogaJpZM4LYGuY
.

Experiencing the same issue - started happening once we integrated FCM, now getting a complete freeze and no frame-advancement any time the app is re-opened after leaving (can be for as little as a few seconds).
We're not creating our own App Controller. The only other major component that might be interfering that I can think of is the Facebook SDK.
Obviously would be great to get a fix.

We're seeing the same. Major issue in that is blocking a release.

Yep, we're still working on it.  At the moment we suspect our method
swizzling is causing problems. Will let you know when we have a fix.
On Wed, Jan 11, 2017 at 2:02 AM, Tom Newton<EMAIL_ADDRESS>wrote:

We're seeing the same. Major issue in that is blocking a release.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/quickstart-unity/issues/33#issuecomment-271827559,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFw1RPZZpCD9I6y6kwL1NePROzIVdiCfks5rRKjLgaJpZM4LYGuY
.

We found the problem, it's an interesting interaction between the Firebase Analytics App Delegate Proxy class (which swizzles the App Delegate's ISA) and the C++ library's caching of swizzled App Delegate methods.  We have a fix which we're testing now, will update the bug when we have cut a release with the fix.

Hey - great that you've found the problem. Wondered if there's an ETA on a fix? Thanks.

We're running through QA right now, worst case a couple of days.
On Mon, Jan 16, 2017 at 5:43 AM, HandCircus<EMAIL_ADDRESS>wrote:

Hey - great that you've found the problem. Wondered if there's an ETA on a
fix? Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/quickstart-unity/issues/33#issuecomment-272864975,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFw1RJcs9LDSKXioVwc9dlLXJXljHCHNks5rS3QEgaJpZM4LYGuY
.

We have released version 1.1.1 of the Firebase Unity SDK, which fixes this issue. You can view the release notes and download the new version from http://firebase.google.com/docs/unity/setup.

I am using the Firebase Unity SDK.
Topic works well for AOS but not for iOS Topic.
In iOS, if send a bundle, you will receive a notification. If send it to Topic, it will not be received.
Can not subscribe to topic: / topics / *** with token: (null)
like this
Firebase-unity version : 3.0.1
Unity version : 5.5.1f

@JangT are you able to reproduce this problem with the sample application?  Also, which iOS version are you targeting?
I just tried this out with Xcode 8, Firebase Unity 3.0.1 and Unity 5.6.0b11 and everything works fine (Unity version in this case really shouldn't matter since the meat of the code is all Obj-C / C++).

@stewartmiles
I used the iOS 9.x version. And this issue happens every time.

@JangT to confirm you used the sample application?
Also, did you run through the sample application setup instructions (see https://github.com/firebase/quickstart-unity/blob/master/messaging/testapp/readme.md)?Specifically you need to enable push notifications and remote notifications.