High voltage on VCC rail (discussion).

I'm bringing the #1 discussion here so as not to derail it from the original topic.

Thanks for the oscillograms. I assume those are shots from the VCC rails at different moments. Seems like I should program the MCU and test how it behaves upon toggling the different power domain control lines.

Ah thanks for creating a new issue. Yes, indeed. They were from the VCC rail during powerup when VUSB was applied.
If you are going to experiment with this i can recommend you to desolder the flash in order to protect it from high voltage. The MCU is 3.3v capable and it can handle up to 4V without getting damaged (table 17 https://www.st.com/resource/en/datasheet/stm32h7b0ib.pdf ), but higher than that will probably not be very good for it.

I'm already experimenting and very probably putting my unit into risk. Although no damage seems to have occurred despite triggering the high voltage issue so many times.
I'm quite perplexed by the mode of failure. Manually toggling the domain enable lines doesn't seem to trigger the issue.
From what I have gathered pin 85 enables 3.3V on the LCD (active high), while pin 82 toggles power on the exteral flash and another portion of the LCD (active low). Pin 85 seems to drive what seems to be the enable line on a 3.3V regulator (backlight?) and pin 82 drives what seems to be a mosfet that simply passes 1.8V to flash and quite possibly the LCD control logic.
By the way, that colored GIF is really helpful. Thanks to Upcycle-Electronics for such valuable info.

What I have gathered so far...

When triggered, the rail on the output side of the 1.9V stays at about 4V even after the power control domain lines are turned off.
In order to clear the problem, the entire board needs to be power cycled. Turning on and off the device doesn't clear it.
The MOSFET simply passes 4V to flash and LCD. It doesn't really seem to be a problem related with it.

This is a bit scary, but causing a very brief short circuit to ground on the output of the 1.9V regulator, clears the error condition and causes voltage to drop down back to 1.9V. This is hinting a positive loop situation somewhere. Could the MCU be involved in delivering 4V through one of it pins to where it shouldn't?

I had absolutely forgotten I had this little gadget in my desk.
The regulator is as I suspected a switching one. A buck type to be precise. The signal you're seeing is the switch output (on the other side of the inductor, the one that isn't Vcc).

And... the moment of truth. This is what happens when we trigger the failure:

That is, an steady 4V output from the switch.

Oh! That's super interesting. Almost looks like the feedback is broken somehow? I mean the buck should have a feedback line that is used to balance the output voltage.

The regulator is quite simply dying on us. The output switch gets stuck at the ON condition. Now, there is a feedback pin, but that is already giving the 4V back to the regulator.

This could be a failure on the design of the regulator chip itself. Probably triggered by the MCU triggering loads too fast. The regulator is quite possibly entering some kind of internal deadlock condition.

Now, the battery could be somehow dampening the effect of loads being triggered. I'm going to bet that placing a capacitor along the input terminals will fix the issue.

Yup! Absolutely as expected. I just added an aluminum capacitor by some precarious pin bending. Turned the unit on and off over 30 times in a row. No metaestable condition was triggered.

Now, there's already a multi-layer capacitor on the board (In fact I put the aluminum one in parallel by touching their pins together), but seems like it's not enough for dampening the inrush current caused by loads being turned on.
This absolutely expains why, by adding delays, you managed to solve the issue in software, as you give the circuit some time to recover from the voltage drops.
Too bad that the part number that Nintendo chose has this issue. If this were other kind of design, where output voltage is absolutely critical, this could have fried tons of units.
At least, I think we can now put this issue to rest now. Or at least for me, my OCD is already satisfied ;-)

Woah. Thanks for your thorough investigation. So what you're saying is, that the sleep is actually the proper solution here without a hardware mod?

Yes! absolutely!

Cool. Then I can also sleep well after this.

From now on, I'll try not to turn the unit whitout the battery inside. We should probably recommend this to everyone, as the battery being present prevents this from happening.

Of course being careful in software has an added benefit. But as we know, not all programs take this extra precaution, in particular, the stock firmware doesn't.

Thanks again for your investigation. I am closing this issue as the problem has been fully understood now.

Sorry for necromancing a topic that's already closed, but I think I can get a little more insight into this topic. We've been having some talk with discord user tim about this issue, looking for ways to avoid this issue through software only. Here's an oscilloscope capture at the instant of the failure. In yellow we have the PMC output (the input to the 1.8V regulator) and in Magenta we have the output of the 1.8V regulator. Voltage scales are 1V/div for both channels and 1us/div for time. As we can see, the PMC output drops quite dramatically, causing the 1.8V regulator to fail and latch the output transistor permanently on.

If, in the other hand, we leave the battery on, we can still see a drop, although this time the drop is quite minor and doesn't cause the 1.8V regulator to even budge a little, proving with almost certainty that leaving the battery on causes the voltage drop to be dampened. This behavior is for the stock firmware by the way.

The length of the drop for my unit is quite short (about 400uS or 0.4ms), meaning that the safe time marging should be above this value. Though I'm quite certain that there will be variations from device from device, as this is highly dependant on the internal resistance/impedance of the PMC.
I'm leaving this here as reference, so others can quickly check. Have a nice day!

Thanks for the updates!

Hi all,
The battery charger IC has an in-rush current limit feature on VIN (VUSB) designed to meet USB guidelines. Current limit is set to approx. 200mA by pins EN1 (logic low), EN2 (logic high) and ILIM resistor (7.5kOhms) (see datasheet  and schematic).
With the battery connected, any supplementary current over these 200mA drawn by the application circuit (for example in-rush current when turning LCD backlight on) will be drawn from the battery.
In the absence of a battery, the current limit kicks in and the storage caps on bus V3v7 get depleted, causing a voltage drop and the behaviour observed with the 1.8V buck converter. Of course adding bulk storage helps, as experienced by @joksan above.
Changing R17 to 1.1kOhms would set the current limiter to the chip's maximum supported 1.5A and might also help. Of course, that would probably break strict USB compliance (maybe?).
All of this is just a guess from reading the schematic and datasheet, I have not yet opened up my Game and Watch unit. Let me know if it makes sense.


Add VSCode in python gitignore

Reasons for making this change:
IDE related files and folders are not part of a software repository
Links to documentation supporting these rule changes:
Visual Studio Code doc states that it makes a .vscode file on the workspace
https://code.visualstudio.com/docs/getstarted/settings

Thanks, but see Global\VisualStudioCode.gitignore


Building new tables based on newly created schema

Ash's Contributions
Why was this work done?

Fixed husky install inside package to make linting work.
Resolved linting errors to streamline code for every developer.

What work was done?

Added script to package to automatically lint.
Reconfigured husky, prettier, and eslint.

 As a developer, I want to create a better quality back-end, so that we can increase the usability of the game.


Backup VM and VMI object graph

What this PR does / why we need it:
In case of backing up a VM and not a namespace (for example by using backup by label) we want all the related objects of the VM/VMI to also be backed up so the VM will be able to restore properly.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Missing some functional-tests
Release note:

Backup VM and VMI object graph

Great start!

/retest

/lgtm
/approve


Ration customization

I really love the plugin and it would be great if the ratio section could be customizable. We need 1:1 and 2:3 ratios in one of the project where in one place 1:1 ratio must be forced and in different place the 2:3 is a must.
So it would be a great feature if we can access and change the list of ratios according to our needs
Thank you in advance!

Hello, is it possible to customize ratios using ImageEditor?
Using it with ImageCropper class is pretty straightforward, but I don't see a way to implement it using ImageEditor class

you can pass cropAvailableRatios in ImageEditor class


[BUG] Add-PnPPage with Translate  & TranslationLanguageCodes failures after 2nd call

Reporting an Issue or Missing Feature
This is an Issue, not a missing feature.
Expected behavior
No exception should be thrown when calling Add-PnPPage twice with -Translate and -TranslationLanguageCodes parameters.
Actual behavior
Second call to Add-PnPPage fails.
Steps to reproduce behavior
The site is an english communication site.
When executing the following script :

Connect-PnPOnline -Url https://YourTenantHere.sharepoint.com/sites/TestMultiLingual -Credentials WinCredHere

Add-PnPPage -Name "TestMultiA" -Translate -TranslationLanguageCodes 1036,1031 -Publish

Add-PnPPage -Name "TestMultiB" -Translate -TranslationLanguageCodes 1036,1031 -Publish

Disconnect-PnPOnline

The first Add-PnPPage suceeds, the second fails.
I'm getting the exception :
Add-PnPPage : Feature 'MultilingualPages' (ID:
24611c05-ee19-45da-955f-6602264abaf8) is already activated at scope
'https://YourTenantHere.sharepoint.com/sites/TestMultiLingual'.
The Multilingual feature was activated before script execution and languages (French - 1036 & German - 1031) were also activated.
If I "reset" the connection (Disconnect / reconnect), it works as expected (but it's ugly/ slow/ not acceptable with -Interactive):
Connect-PnPOnline -Url https://YourTenantHere.sharepoint.com/sites/TestMultiLingual -Credentials WinCredHere
Add-PnPPage -Name "TestMultiA" -Translate -TranslationLanguageCodes 1036,1031 -Publish
Disconnect-PnPOnline

Connect-PnPOnline -Url https://YourTenantHere.sharepoint.com/sites/TestMultiLingual -Credentials WinCredHere
Add-PnPPage -Name "TestMultiB" -Translate -TranslationLanguageCodes 1036,1031 -Publish
Disconnect-PnPOnline

So I assume there is some kind of intialization problems related to the check with feature activation, it seems to try to activate a feature already activated.
Note : I noticed the same behavior with Set-PnPPage
What is the version of the Cmdlet module you are running?
1.12.0
Which operating system/environment are you running PnP PowerShell on?

[X ] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

hi @ChristopheAngot , have fixed the issue. Will be available in tomorrow's nightly build.
Thanks for reporting, much appreciated 😊🙏


with auto hide off, after selection, picker selector returns to default category/emoji

with autohide off, after a user selects an emoji it returns back to the initial category & emoji, so if you want to select the same emoji multiple times you have to keep re-selecting the proper category/emoji
screen capture of current behavior  -
https://drive.google.com/file/d/1ZtEOqFPHJte1JbNIlND78aE_KZCgGvGq/view?usp=sharing

This is fixed in the upcoming 5.0.0 release. Will close the issue upon release.

thank you sir, love the picker

This issue is resolved in v5.0.0-beta.6.


Release Testing: celsius_connector python example doesn't make progress on trusty

Ref: #2422
Context: testing examples on a trusty machine, with Wallaroo installed via wallaroo-up.
Running the example app examples/python/celsius_connector stalls at machida starting up, but no progress is made and no numbers are printed on the screen.
I've sourced bin/activate from the wallaroo installation and modified run_sample to run machida without a specific path.
modifications to run_sample
wallaroo = subprocess.Popen([
    # "sleep", "10"])
    #"../../../machida/build/machida",
    "machida",
    "--application-module", "celsius",
    "--metrics", "<IP_ADDRESS>:5001",
    "--control", "<IP_ADDRESS>:6000",
    "--data", "<IP_ADDRESS>:6001",
    "--name", "worker-name",

output from run_sample
vagrant@vagrant:~/wallaroo-tutorial/wallaroo-release-0.5.3/examples/python/celsius_connectors$ ./run_sample
recieving on host: <IP_ADDRESS> port: 8901
****CLUSTERING MODE is active****
****This is an enterprise feature. You may need to obtain a paid usage agreement to use this feature in production. See the Wallaroo Community License at https://github.com/WallarooLabs/wallaroo/blob/master/LICENSE.md for details, and also please visit the page at http://www.wallaroolabs.com/pricing****
****AUTOSCALE MODE is active****
****This is an enterprise feature. You may need to obtain a paid usage agreement to use this feature in production. See the Wallaroo Community License at https://github.com/WallarooLabs/wallaroo/blob/master/LICENSE.md for details, and also please visit the page at http://www.wallaroolabs.com/pricing****
||| Resilience directory: /tmp|||
Set up external channel listener on <IP_ADDRESS>:5050
Running as Initializer...
SLF: control_host = <IP_ADDRESS>, control_service = 6000
!@ Transition to _WaitingForCheckpointInitiationEventLogPhase: checkpoint_id 1
!@ CheckpointInitiator: is_recovering: false
!@ Saving CheckpointId 0 and RollbackId 0
AUTOSCALE: Waiting for new autoscale event.
initializer external: listening on <IP_ADDRESS>:5050
metrics outgoing connected
initializer control: listening on <IP_ADDRESS>:6000
Saving connection addresses!
Initializing application
---------------------------------------------------------
vvvvvv|Initializing Topologies for Workers|vvvvvv

Found 1 pipelines in application
The Celsius Conversion pipeline has 2 uncoalesced runner builders

Preparing to spin up |multiply by 1.8|add 32| on source on initializer
Each worker gets roughly 1 steps

^^^^^^|Finished Initializing Topologies for Workers|^^^^^^^
---------------------------------------------------------
---------------------------------------------------------
|v|v|v|Initializing Local Topology|v|v|v|

Saving topology!
Saving worker names to file: /tmp/celsius-initializer.workers
LocalTopology._save_worker_names: initializer
Creating graph:
GRAPH:
"| Celsius Conversion source | |multiply by 1.8|add 32||" --> "Celsius Conversion sink"

"Celsius Conversion sink" --> <>

Initializing Celsius to Fahrenheit application locally:

Adding sink nodes to frontier
Adding Celsius Conversion sink node to frontier
Adding | Celsius Conversion source | |multiply by 1.8|add 32|| node to end of frontier
Handling Celsius Conversion sink node
Connecting to sink at localhost:7200
Finished handling Celsius Conversion sink node
Handling | Celsius Conversion source | |multiply by 1.8|add 32|| node
----Creating source for Celsius Conversion pipeline with | Celsius Conversion source | |multiply by 1.8|add 32||----
Finished handling | Celsius Conversion source | |multiply by 1.8|add 32|| node
We've already handled | Celsius Conversion source | |multiply by 1.8|add 32|| with id<PHONE_NUMBER>25051482536870295822466419518 so we're not handling  it again
Local topology initialized

|^|^|^|Finished Initializing Local Topology|^|^|^|
---------------------------------------------------------
initializer data channel: listening on <IP_ADDRESS>:6001
!@ BarrierInitiator: add_worker initializer
!@ application_begin_reporting BarrierInitiator
!@ CheckpointInitiator: add_worker initializer
!@ application_begin_reporting CheckpointInitiator
|~~ INIT PHASE I: Application is created! ~~|
Celsius Conversion source attempting to listen on localhost:7100
Celsius Conversion source is listening
|~~ INIT PHASE II: Application is initialized! ~~|
ConnectorSink initializing connection to localhost:7200
ConnectorSink connected
|~~ INIT PHASE III: Application is ready to work! ~~|
initializer reported topology ready!
All 1 workers reporting Topology ready!
Application has successfully initialized.

The process stops here and makes no progress.

The same thing happens on trusty with wallaroo built from source.

The same things happens on windows docker.

Same on Xenial.

PR #2489 should address the possibility of processes remaining in the background causing the run script to fail to start new connectors or spin up its own UDP server in a thread.

Unable to reproduce on trusty wallaroo up, trusty from source, xenial wallaroo up, and windows docker. Assuming #2489 resolved this issue and closing accordingly.


BUG: Fix daily history for minute panel data backtest

Fixes #1356, #1629.
New function zipline.data.resample.minute_panel_to_session_panel.

Coverage decreased (-0.06%) to 87.184% when pulling e159c142b22b0dcdabc3889372cff4de04b769c6 on nathanwolfe:minute-panel-to-daily into 3d528322afdb547c03b509295f2d1fd8b1aa285f on quantopian:master.

Coverage increased (+0.01%) to 87.256% when pulling 85ea1f3f524646aa4afd746e22275ab624697a1b on nathanwolfe:minute-panel-to-daily into 3d528322afdb547c03b509295f2d1fd8b1aa285f on quantopian:master.

Coverage increased (+0.01%) to 87.256% when pulling 85ea1f3f524646aa4afd746e22275ab624697a1b on nathanwolfe:minute-panel-to-daily into 3d528322afdb547c03b509295f2d1fd8b1aa285f on quantopian:master.


HarfangLab EDR connector

Hi,
Could you please merge our HarfangLab EDR connector into the master branch ?
Thanks.

Sure thing!
Looks good based on the API. From what I can see, we could all get more out of this if it was OpenAPI instead as well :)


Can't archive app on Xcode 11 beta 4

The app can run successfully, but can't archive.
RealmSwift 3.17.1.
Xcode 11 beta 4.
macOS 10.15 beta 4.

Do not archive apps on Xcode beta.

Do not archive apps on Xcode beta.

No. Everything was fine in beta 3.

This is a known issue of Xcode 11 beta 4. https://stackoverflow.com/questions/57116047/xcode-11-beta-4-archive-swift-errors
Workaround:

More, the above setting must set on the target of RealmSwift, not your own app.

I have a similar build problem to owenzhao, at least it looks like the same error.
In my case, I'm simply trying to build RealmSwift from command line, using today's master branch,
SHA1: 026a03308360ab301e647754c44881a2ee3b15e1
trying to build using Xcode 11 beta 4 (and yes, Command Line Tools is set to Xcode 11's)
(ios-static and ios-dynamic build fine, btw).
On command line I did:  DEVELOPER_DIR=/Applications/Xcode-beta.app/Contents/Developer ./build.sh ios-swift
Undefined symbols for architecture arm64:
"Swift.String.UTF16View._nativeGetOffset(for: Swift.String.Index) -> Swift.Int", referenced from:
function signature specialization <Arg[0] = Exploded, Arg[1] = Exploded, Arg[2] = Exploded, Arg[3] = Dead> of RealmSwift.gsub(pattern: Swift.String, template: Swift.String, string: Swift.String, error: Swift.AutoreleasingUnsafeMutablePointer<__C.NSError?>?) -> Swift.String? in Util.o
"Swift.String.UTF16View._foreignCount() -> Swift.Int", referenced from: ...
Anyone know how to build RealmSwift from command line using Xcode 11 beta 4?

I think you just need the command line way to modify the build settings like I did in Xcode.
However, I didn't know how to that with command line.
 The changes could easily watched by git. So you can use vim to change the project accordingly.

@owenzhao I have tried the workaround, but it is helpless, same error result...

@owenzhao I have tried the workaround, but it is helpless, same error result...

Please provide your sample Xcode project and I will try it.

My issue is solved by Xcode beta 5. If you can confirm yours @ajmeerkhan , I will close this issue.

@owenzhao its working fine now.

Hello, what was the working solution to this problem?

Hello, what was the working solution to this problem?

You should not encounter this issue as it is fixed in Xcode 11 beta 5.

Was is release version, problem with SwiftSoup. Here solution for that:
https://stackoverflow.com/questions/58001433/xcode-11-stuck-on-archiving


Meta group

Hey @jarektkaczyk,
this is my first draft for #131. There are no tests yet. I would like to get your feedback first, if this is a way you would agree with.
What does it do?

It adds a new column to the table meta_group which is a nullable varchar
when adding a meta item using the setMeta function the user can provide a third argument $group
If none is provided, it is just set to NULL
To get all attributes associated with a group simply use getMetaByGroup('group-name') on your model, which will return a collection of Attributes.

No idea why its failing, when I run phpunit locally it passes. Somehow I can't see the logs on travis.

Hey, pls make sure it's 200% BC and optional, so it doesn't create overhead
for the usual way of handling meta attributes. Back this with tests, and
we're gtg.
Unless my review is what you're after here, rather than the functionality,
then I can spend time on it? Otherwise I just trust you... And the tests ;)
Jarek Tkaczyk
softonsofa :: development with pleasure
On Nov 2, 2016 02:31, "Lukas Oppermann"<EMAIL_ADDRESS>wrote:

Hey @jarektkaczyk https://github.com/jarektkaczyk,
this is my first draft for #131
https://github.com/jarektkaczyk/eloquence/issues/131. There are no
tests yet. I would like to get your feedback first, if this is a way you
would agree with.
What does it do?

It adds a new column to the table meta_group which is a nullable
varchar
when adding a meta item using the setMeta function the user can
provide a third argument $group
If none is provided, it is just set to NULL
To get all attributes associated with a group simply use
getMetaByGroup('group-name') on your model.

You can view, comment on, or merge this pull request online at:
https://github.com/jarektkaczyk/eloquence/pull/134
Commit Summary

in progress
first implementation

File Changes

M src/Contracts/Attribute.php
https://github.com/jarektkaczyk/eloquence/pull/134/files#diff-0 (7)
M src/Contracts/AttributeBag.php
https://github.com/jarektkaczyk/eloquence/pull/134/files#diff-1 (3)
M src/Contracts/Metable.php
https://github.com/jarektkaczyk/eloquence/pull/134/files#diff-2 (2)
M src/Metable.php
https://github.com/jarektkaczyk/eloquence/pull/134/files#diff-3 (15)
M src/Metable/Attribute.php
https://github.com/jarektkaczyk/eloquence/pull/134/files#diff-4 (34)
M src/Metable/AttributeBag.php
https://github.com/jarektkaczyk/eloquence/pull/134/files#diff-5 (28)
M src/Metable/CreateMetaAttributesTable.php
https://github.com/jarektkaczyk/eloquence/pull/134/files#diff-6 (1)
M tests/MetableTest.php
https://github.com/jarektkaczyk/eloquence/pull/134/files#diff-7 (2)

Patch Links:

https://github.com/jarektkaczyk/eloquence/pull/134.patch
https://github.com/jarektkaczyk/eloquence/pull/134.diff

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jarektkaczyk/eloquence/pull/134, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGm5sgtAR-v5heDv6zHL86fX9N0T5PLLks5q54V7gaJpZM4Kma_k
.

Coverage decreased (-0.2%) to 93.36% when pulling b05b0504847330818aacc3baf5f2b7fb26bf49e3 on lukasoppermann:meta-group into 9f2f32c9d36ffbd1a0d0ee88d6bf9e81c207ec44 on jarektkaczyk:5.3.

Okay, it was a styling bug. As you can see the tests all still pass, I only changed one expectation, but for this to work, but this is nothing that changes for a user.
I will add some tests to test the new feature and let you know.

Coverage decreased (-0.04%) to 93.558% when pulling cc685bef4bdae4fe3fecd871db2532b3bc0c34be on lukasoppermann:meta-group into 9f2f32c9d36ffbd1a0d0ee88d6bf9e81c207ec44 on jarektkaczyk:5.3.

Coverage decreased (-0.04%) to 93.558% when pulling cc685bef4bdae4fe3fecd871db2532b3bc0c34be on lukasoppermann:meta-group into 9f2f32c9d36ffbd1a0d0ee88d6bf9e81c207ec44 on jarektkaczyk:5.3.

Coverage decreased (-0.04%) to 93.558% when pulling c5d33dd8e52afeeec5dae03b69ec18806ba9078a on lukasoppermann:meta-group into 9f2f32c9d36ffbd1a0d0ee88d6bf9e81c207ec44 on jarektkaczyk:5.3.

Coverage decreased (-0.04%) to 93.558% when pulling c5d33dd8e52afeeec5dae03b69ec18806ba9078a on lukasoppermann:meta-group into 9f2f32c9d36ffbd1a0d0ee88d6bf9e81c207ec44 on jarektkaczyk:5.3.

Hey @jarektkaczyk I added tests now, however it fails on hhvm and I have no clue how to fix it. Can you have a look and let me know why and how to fix this?

Coverage decreased (-0.04%) to 93.558% when pulling 740bfa35d98d371d0e62e00f594fa1d0d73ed176 on lukasoppermann:meta-group into 9f2f32c9d36ffbd1a0d0ee88d6bf9e81c207ec44 on jarektkaczyk:5.3.

Coverage increased (+0.2%) to 93.756% when pulling 74bf9f23c43a3d79ff9afc4d60c01f107b028b63 on lukasoppermann:meta-group into 9f2f32c9d36ffbd1a0d0ee88d6bf9e81c207ec44 on jarektkaczyk:5.3.

Coverage increased (+0.2%) to 93.756% when pulling 74bf9f23c43a3d79ff9afc4d60c01f107b028b63 on lukasoppermann:meta-group into 9f2f32c9d36ffbd1a0d0ee88d6bf9e81c207ec44 on jarektkaczyk:5.3.

Coverage increased (+0.2%) to 93.756% when pulling 74bf9f23c43a3d79ff9afc4d60c01f107b028b63 on lukasoppermann:meta-group into 9f2f32c9d36ffbd1a0d0ee88d6bf9e81c207ec44 on jarektkaczyk:5.3.

Hey @jarektkaczyk what do you think about this? I am not quite sure why hhvm fails. Are you planning on removing it, since Larave does not support hhvm beginning from 5.3?
Otherwise, are you still interested in this functionality? I am currently not using this. While I still thing it would be very handy for people, I am also fine with just closing it, as I currently don't need it personally.

@jarektkaczyk any comment here?

Hey @jarektkaczyk please can you help here? I would like to make this mergeable.

Hey @jarektkaczyk, are you still interested in this PR? Does it matter that it fails on hhvm? If you are willing to merge it I can fix the conflict that arose from some changes after the PR was send.

hey @lukasoppermann

here's the travis log
```
1) Sofa\Eloquence\Tests\AttributeBagTest::it_gets_attributes_by_group
Use of undefined constant ARRAY_FILTER_USE_BOTH - assumed 'ARRAY_FILTER_USE_BOTH'
/home/travis/build/jarektkaczyk/eloquence/vendor/illuminate/support/Arr.php:517
/home/travis/build/jarektkaczyk/eloquence/vendor/illuminate/support/Collection.php:282
/home/travis/build/jarektkaczyk/eloquence/vendor/illuminate/support/Collection.php:304
/home/travis/build/jarektkaczyk/eloquence/src/Metable/AttributeBag.php:134
/home/travis/build/jarektkaczyk/eloquence/tests/AttributeBagTest.php:107
2) Sofa\Eloquence\Tests\MetableTest::it_gets_attribute_with_group_on_the_bag
Use of undefined constant ARRAY_FILTER_USE_BOTH - assumed 'ARRAY_FILTER_USE_BOTH'
/home/travis/build/jarektkaczyk/eloquence/vendor/illuminate/support/Arr.php:517
/home/travis/build/jarektkaczyk/eloquence/vendor/illuminate/support/Collection.php:282
/home/travis/build/jarektkaczyk/eloquence/vendor/illuminate/support/Collection.php:304
/home/travis/build/jarektkaczyk/eloquence/src/Metable/AttributeBag.php:134
/home/travis/build/jarektkaczyk/eloquence/src/Metable.php:415
/home/travis/build/jarektkaczyk/eloquence/tests/MetableTest.php:572
```

If you can pls update .travis and get rid of hhvm, which we don't care about anyway, then I will merge.
Also, if you can send a PR to 5.4 too would be great.

Coverage decreased (-1.6%) to 92.012% when pulling ae3e3964e508d1fce66e005d19ad4ffb27af6852 on lukasoppermann:meta-group into d058c6963204f42aef1830e60349268c3af77135 on jarektkaczyk:5.3.

Coverage decreased (-1.6%) to 92.012% when pulling c52b0fc4983f2122f28b154652bc12233e826b35 on lukasoppermann:meta-group into d058c6963204f42aef1830e60349268c3af77135 on jarektkaczyk:5.3.

Since the other one is merged, this one can be closed, right?


docs: rephrase explanation of which node-gyp is used by npm

Checklist

[x] npm install && npm test passes
[ ] tests are included
[x] documentation is changed or added
[x] commit message follows commit guidelines

Description of change

Previous explanation went too deep regarding details about files hierarchy of npm. This info may mislead windows users since in windows npm is located in other place than global node_modules.
Also some minor changes like highlighting keywords and using direct names instead of pronouns

@rvagg thank you for review. Your changes are applyed


How to reject non-encrypted connections when I use the TLS function?

Hi,
I set the FTPS server as the example:
var cert = new X509Certificate2("my.pfx", "my-super-strong-password-that-nobody-knows");
services.Configure(cfg => cfg.ServerCertificate = cert);
The FTP server allows to create TLS connections via FTP clients. But it still allows to create non-encrupted connections.
How could I set to reject non-encrypted connections, and only allow TLS connections?
Thanks.

I tried to use implicit TLS and it works fine. Thanks for you great help!


Closed summarizer may log fluid:telemetry:Summarizer:Running:SummarizeTimeout & fluid:telemetry:Summarizer:Running:SummaryAckWaitTimeout events!

Here is an example of query.
Logs from ODSP stress test.
customEvents
| where timestamp > ago(5d)
| where customDimensions.containerId == "9ce04c0a-1dbd-4309-b5af-ba903a83f8d8"
| project timestamp, name, customDimensions.reason, customDimensions.error, customDimensions.category
| order by timestamp asc
| 8/5/2021, 9:16:47.081 AM | fluid:telemetry:Container:Creating Container noninteractive/summarizer |   |   | generic |
  | 8/5/2021, 9:16:47.082 AM | fluid:telemetry:Container:Load_start |   |   | performance |
  | 8/5/2021, 9:16:47.587 AM | fluid:telemetry:OdspDriver:GetWebsocketToken_GetToken_end |   |   | performance |
  | 8/5/2021, 9:16:47.640 AM | fluid:telemetry:OdspDriver:EpochLearnedFirstTime |   |   | generic |
  | 8/5/2021, 9:16:47.643 AM | fluid:telemetry:Container:ConnectionStateChange_Connecting |   |   | performance |
  | 8/5/2021, 9:16:48.551 AM | fluid:telemetry:OdspDriver:TreesLatest_GetToken_end |   |   | performance |
  | 8/5/2021, 9:16:49.016 AM | fluid:telemetry:OdspDriver:ObtainSnapshot_end |   |   | performance |
  | 8/5/2021, 9:16:49.016 AM | fluid:telemetry:OdspDriver:TreesLatest_end |   |   | performance |
  | 8/5/2021, 9:16:49.018 AM | fluid:telemetry:DeltaManager:enqueueMessages | DocumentOpen_pending |   | performance |
  | 8/5/2021, 9:16:49.021 AM | fluid:telemetry:OdspDriver:GetDeltas_end |   |   | performance |
  | 8/5/2021, 9:16:49.022 AM | fluid:telemetry:CodeLoad_end |   |   | performance |
  | 8/5/2021, 9:16:49.022 AM | fluid:telemetry:OdspDriver:CacheOpsRetrieved |   |   | performance |
  | 8/5/2021, 9:16:49.026 AM | fluid:telemetry:ExternalBlobsInSnapshot |   |   | generic |
  | 8/5/2021, 9:16:49.026 AM | fluid:telemetry:ContainerRuntime:ContainerLoadStats |   |   | generic |
  | 8/5/2021, 9:16:49.044 AM | fluid:telemetry:Container:ConnectionStateChange_Connected |   |   | performance |
  | 8/5/2021, 9:16:49.045 AM | fluid:telemetry:ScheduleManager:DeltaScheduler:InboundOpsProcessingTime |   |   | generic |
  | 8/5/2021, 9:16:49.045 AM | fluid:telemetry:OpPerf:GetDeltas_OpProcessing |   |   | performance |
  | 8/5/2021, 9:16:49.046 AM | fluid:telemetry:Container:Load_end |   |   | performance |
  | 8/5/2021, 9:16:49.046 AM | fluid:telemetry:Container:Request_end |   |   | performance |
  | 8/5/2021, 9:16:49.047 AM | fluid:telemetry:Summarizer:RunningSummarizer |   |   | generic |
  | 8/5/2021, 9:16:50.033 AM | fluid:telemetry:Summarizer:StoppingSummarizer | parentNotConnected |   | generic |
  | 8/5/2021, 9:16:50.035 AM | fluid:telemetry:Container:ConnectionStateChange_Disconnected | socket.io: disconnect: socket.io: server_disconnect: TokenExpired |   | performance |
  | 8/5/2021, 9:16:50.035 AM | fluid:telemetry:Container:noWaitOnDisconnected |   |   | generic |
  | 8/5/2021, 9:16:50.036 AM | fluid:telemetry:ContainerRuntimeDisposed |   |   | generic |
  | 8/5/2021, 9:16:50.036 AM | fluid:telemetry:Container:Container Closed |   |   | generic |
  | 8/5/2021, 9:16:50.037 AM | fluid:telemetry:Container:ContainerClose |   |   | generic |
  | 8/5/2021, 9:26:49.049 AM | fluid:telemetry:Summarizer:Running:SummaryAckWaitTimeout |   |   | error |
  | 8/5/2021, 9:26:49.053 AM | fluid:telemetry:Summarizer:Running:GenerateSummary |   |   | generic |
  | 8/5/2021, 9:26:49.053 AM | fluid:telemetry:Summarizer:Running:Summarize_cancel | maxTime | disconnected | generic |
  | 8/5/2021, 9:26:49.059 AM | fluid:telemetry:Summarizer:Running:RefreshLatestSummaryGetSnapshot_end |   |   | performance |
  | 8/5/2021, 9:26:49.394 AM | fluid:telemetry:OdspDriver:getVersions_end |   |   | performance |
  | 8/5/2021, 9:26:49.824 AM | fluid:telemetry:OdspDriver:fetchSnapshot_end |   |   | performance |
  | 8/5/2021, 9:26:49.827 AM | fluid:telemetry:Summarizer:Running:RefreshLatestSummaryGetSnapshot_end |   |   | performance |
  | 8/5/2021, 9:26:49.829 AM | fluid:telemetry:Summarizer:Running:WaitingForSeq_start |   |   | performance |
  | 8/5/2021, 9:27:09.055 AM | fluid:telemetry:Summarizer:Running:SummarizeTimeout |   |   | performance |
  | 8/5/2021, 9:27:49.056 AM | fluid:telemetry:Summarizer:Running:SummarizeTimeout |   |   | performance |
  | 8/5/2021, 9:29:09.056 AM | fluid:telemetry:Summarizer:Running:SummarizeTimeout |   |   | performance |
  | 8/5/2021, 9:31:49.057 AM | fluid:telemetry:Summarizer:Running:SummarizeTimeout |   |   | performance |
  | 8/5/2021, 9:37:09.059 AM | fluid:telemetry:Summarizer:Running:SummarizeTimeout |   |   | performance
As you can see, summarizing container was closed, yet it keep going and did a bunch of network requests (looks really weird, they should have failed with "storage is disposed" kind of failure - we definitely see that in other cases, for example in https://github.com/microsoft/FluidFramework/issues/6896).

@GaryWilber - FYI, this is the case you brought to my attention.

Ok, after latest set of fixes, all the other "real" reasons for summary problems are gone, and this case represents one of the biggest sources of noise in stress tests. I.e. it creates false alarms.
We should prioritize addressing this.

I think we can do 2 things to address this.
(1) Add more checks for disposed before doing things.
(2) For this specifically, it seems like it could be disposing the Summarizer before it finishes starting which is a race condition.

That's same as #6896, so closing


Just added a ref to this.request.body

Straight-forward + non-breaking. Please let me know if enough.

I check the ci.


Correct wording in intro.js

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #45373

Sorry, I have done it by mistake.


Repository name matching should be case insensitive

https://discordapp.com/channels/684898665143206084/684898665151594506/740556533770879048

Fixed in #52


Fix error object when a service throws

The error object needs to be in an "error" field. The code above does this correctly (Method not found) but this code doesn't. The result is that if an unhandled exception occurred the client would get a ServiceUnregistered event and then "ServiceDisappeared" responses. With this, it gets the expected error and does not unregister the service.
@devoncarew I can't figure out how to put automated tests around this (though I did test it manually with my scripts from https://github.com/DanTup/dart_vm_service_register_service_repro). There are some service registration tests, but they seem to go through _delegateRequest whereas this requires _routeRequest. Using the mock doesn't help because it doesn't rest the real implementation. Is there an easy/good way to do this?

If the error in the java bot (https://travis-ci.org/dart-lang/vm_service_drivers/jobs/527248198#L366) is consistent, please file an issue to look at it (the failure is likely unrelated to this PR, so doesn't need to block landing it).

@devoncarew fixed the tweaks, but I don't entirely understand the java failure to be confident I didn't break it. I presume there's no sharing of these files so it's not, but your opinion here would be best :)
(also, I can't land FWIW, so if you're happy with this, please land it - I'll open an issue about the java error if you're sure it's not related)


Table stuff.

This is a pretty big deal.

Table guidelines (as discussed today) are added to the styleguide.
Wrapping text in code elements inside of tables are now optionally enabled via a .table-word-break class applied to the table itself.
Tables adopt the $body-small font size to conserve space.
All tables in Jekyll generated content pick up the .table base class. This is done via the styleguide's bundled JavaScript.
Additionally, all .tables are wrapped in .table-wrapper div to allow horizontally scrolling of the table if it is too wide to fit in the viewport.
This allows us to remove the duplicate table styles in the blog post CSS file, with the exception of styling the wrapper to allow it to be wider than content column on larger screns.
@IndyHurt's table, as generated by Markdown, is added to samples/blog-post.html as a demonstration of how this works in practice. Once this is live, the temporary workarounds added to her blog post should be removed.

cc @souperneon @hanbyul-here @migurski

Could we add some additional sample tables to the guide, to demonstrate the possibilities?

Where should this specific table then? It requires mapzen javascript but it doesn't look like js components at all!

that's actually a good idea. So are there two or three options now @louh?

I can add a table to documentation that shows the code breaking on purpose, and maybe another table to the blog post that has a ton of columns on purpose?

yeah that's how I understood that. Maybe not the 'wrong' ways but atleast the one with many many columns?

Please take a look now @migurski @souperneon


coverage: Evaluate LLVM coverage test runs

Signed-off-by: Lizan Zhou<EMAIL_ADDRESS>Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:

Coverage is slow on the last run here, 2h 21m. Will mark as waiting, LMK when ready for review. I'm excited by the approach being taken.

@htuch I didn't change existing coverage flow, the new llvm_coverage is in its own https://circleci.com/gh/envoyproxy/envoy/208713, so I could compare the coverage report.


Config info is not working

Sorry maybe i am just being a noob, but I cant get the config to save when I add what you suggested.  I also assume one needs to get the oauth ID and secret from somewhere but i am not sure how to find that.  mind helping me out?

ok I sorted out the syntax in my config file.  but I assume I need help to get the oauth info as this is the error I'm getting:
`[1/6/2021, 8:07:30 AM] [EightSleepPod] Get RotationSpeed Error -> InvalidResponseError: invalid response
at Function.wrap (/usr/lib/node_modules/homebridge-eightsleep-pod/node_modules/baseerr/src/index.ts:51:12)
at EightSleepClientApi. (/usr/lib/node_modules/homebridge-eightsleep-pod/node_modules/simple-api-client/src/index.ts:244:34)
at Generator.throw ()
at rejected (/usr/lib/node_modules/homebridge-eightsleep-pod/node_modules/simple-api-client/dist/cjs/index.js:6:65)
at processTicksAndRejections (internal/process/task_queues.js:93:5)
{
"status": 401,
"headers": {},
"path": "users/oauth-token",
"init": {
"method": "POST",
"headers": {
"accept": "application/json",
"content-type": "application/json",
"user-agent": "Eight/786 CFNetwork/1125.2 Darwin/19.4.0",
"accept-language": "en-us",
"accept-encoding": "gzip, deflate, br",
"user-id": "f2886ebe4e854c178541a919851cf905",
"session-token": "f0272abf9cc34486a247c2f90990c515-ce4c7be2275438a8908d96f46d57fdcf"
},
"body": "{"client_id":"clientid....","client_secret":"clientsecret...."}"
}
}
FetchError: invalid json response body at https://client-api.8slp.net/v1/users/oauth-token reason: Unexpected end of JSON input
at /usr/lib/node_modules/homebridge-eightsleep-pod/node_modules/node-fetch/lib/index.js:272:32
at processTicksAndRejections (internal/process/task_queues.js:93:5)
{
status: 401,
headers: Headers {
[Symbol(map)]: [Object: null prototype] {
date: [Array],
'content-length': [Array],
connection: [Array],
'x-powered-by': [Array]
}
},
path: 'users/oauth-token',
init: {
method: 'POST',
headers: {
accept: 'application/json',
'content-type': 'application/json',
'user-agent': 'Eight/786 CFNetwork/1125.2 Darwin/19.4.0',
'accept-language': 'en-us',
'accept-encoding': 'gzip, deflate, br',
'user-id': 'f2886ebe4e854c178541a919851cf905',
'session-token': 'f0272abf9cc34486a247c2f90990c515-ce4c7be2275438a8908d96f46d57fdcf'
},
body: '{"client_id":"clientid....","client_secret":"clientsecret...."}'
}
}`

ok I sorted out the syntax in my config file.  but I assume I need help to get the oauth info as this is the error I'm getting:
`[1/6/2021, 8:07:30 AM] [EightSleepPod] Get RotationSpeed Error -> InvalidResponseError: invalid response
at Function.wrap (/usr/lib/node_modules/homebridge-eightsleep-pod/node_modules/baseerr/src/index.ts:51:12)
at EightSleepClientApi. (/usr/lib/node_modules/homebridge-eightsleep-pod/node_modules/simple-api-client/src/index.ts:244:34)
at Generator.throw ()
at rejected (/usr/lib/node_modules/homebridge-eightsleep-pod/node_modules/simple-api-client/dist/cjs/index.js:6:65)
at processTicksAndRejections (internal/process/task_queues.js:93:5)
{
"status": 401,
"headers": {},
"path": "users/oauth-token",
"init": {
"method": "POST",
"headers": {
"accept": "application/json",
"content-type": "application/json",
"user-agent": "Eight/786 CFNetwork/1125.2 Darwin/19.4.0",
"accept-language": "en-us",
"accept-encoding": "gzip, deflate, br",
"user-id": "f2886ebe4e854c178541a919851cf905",
"session-token": "f0272abf9cc34486a247c2f90990c515-ce4c7be2275438a8908d96f46d57fdcf"
},
"body": "{"client_id":"clientid....","client_secret":"clientsecret...."}"
}
}
FetchError: invalid json response body at https://client-api.8slp.net/v1/users/oauth-token reason: Unexpected end of JSON input
at /usr/lib/node_modules/homebridge-eightsleep-pod/node_modules/node-fetch/lib/index.js:272:32
at processTicksAndRejections (internal/process/task_queues.js:93:5)
{
status: 401,
headers: Headers {
[Symbol(map)]: [Object: null prototype] {
date: [Array],
'content-length': [Array],
connection: [Array],
'x-powered-by': [Array]
}
},
path: 'users/oauth-token',
init: {
method: 'POST',
headers: {
accept: 'application/json',
'content-type': 'application/json',
'user-agent': 'Eight/786 CFNetwork/1125.2 Darwin/19.4.0',
'accept-language': 'en-us',
'accept-encoding': 'gzip, deflate, br',
'user-id': 'f2886ebe4e854c178541a919851cf905',
'session-token': 'f0272abf9cc34486a247c2f90990c515-ce4c7be2275438a8908d96f46d57fdcf'
},
body: '{"client_id":"clientid....","client_secret":"clientsecret...."}'
}
}`

hey! I haven't looked at this in a bit you can just provide email and password in the config it should login for you after that. lmk if that works out for you and feel free to update the docs marking oauthClient as optional

Hi - having the same issue as above. If I remove the oAuth line as suggested, I get an error that it is required.
What do you recommend I try? Here is my current config:
{             "name": "Eightsleep",             "platform": "EightSleepPod",             "email"<EMAIL_ADDRESS>            "password": "mypassword",             "oauthClient": {                 "id": "clientid...."                 "secret": "clientsecret...."             }         }

Hi - having the same issue as the two above....
fyi: when the oauthClient line in the config is omitted, for example:
        {
          "name": "EighSleep",
          "platform": "EightSleepPod",
          "email"<EMAIL_ADDRESS>          "password": "mypassword"
        }

I get the several instances of following oath error in the homebridge logs:
RotationSpeed Error -> BaseError: appApiClient is missing (client should be instantiated with oauthClient)^[[39m
^[[31m    at EightSleepClientApi.getAppApiClient (/usr/local/lib/node_modules/homebridge-eightsleep-pod/node_modules/eightsleep/src/index.ts:144:13

Looking through your very nice Eightsleep api client for Node.js repo, doesn't the client take as input the oath id?
Lmk if I can provide more information to help.

same issue here - the api client wants oauth token (even with oauth attribute omitted from config):
[Eight Sleep Pod] Get Active Error -> BaseError: appApiClient is missing (client should be instantiated with oauthClient)
    at EightSleepClientApi.getAppApiClient (/usr/local/lib/node_modules/homebridge-eightsleep-pod/node_modules/eightsleep/src/index.ts:144:13)
    at /usr/local/lib/node_modules/homebridge-eightsleep-pod/src/EightSleepPod.ts:44:20
    at Object.<anonymous> (/usr/local/lib/node_modules/homebridge-eightsleep-pod/node_modules/memoize-concurrent/src/index.ts:31:22)
    at Generator.next (<anonymous>)
    at /usr/local/lib/node_modules/homebridge-eightsleep-pod/node_modules/memoize-concurrent/dist/cjs/index.js:8:71
    at new Promise (<anonymous>)
    at __awaiter (/usr/local/lib/node_modules/homebridge-eightsleep-pod/node_modules/memoize-concurrent/dist/cjs/index.js:4:12)
    at EightSleepPod.<anonymous> (/usr/local/lib/node_modules/homebridge-eightsleep-pod/node_modules/memoize-concurrent/src/index.ts:29:66)
    at EightSleepPod.getDeviceStatus (/usr/local/lib/node_modules/homebridge-eightsleep-pod/node_modules/mem/index.js:24:21)
    at EightSleepPod.getStatus (/usr/local/lib/node_modules/homebridge-eightsleep-pod/src/EightSleepPod.ts:48:37)

alright this took entirely way too long but I think I resolved it. the reason why I never updated this is bc I was providing oauth id and secret directly, so I never saw the issue when providing user/pass only. anyways at somepoint my auth expired and I ran into the problem myself so I think I fixed it. going to test it out in the next few hours.

hmm okay nvm I am not able to get it to work using only username/password. the oauthClient id and secret are required. the way that I was able to get this information was using the Charles iOS app to spy on all the requests the Eightsleep app was making on my iPhone. look for a POST request to /v1/users/oauth-token and view the request body.
you will see something like this:
{
	"client_id": "<hex-string>",
	"client_secret": "<hex-string>"
}

plug those values into your config:
{
            "platform": "EightSleepPod",
            "email": "<email address>",
            "password": "<password>",
            "oauthClient": {
                "id": "<hex-string-client-id-from-above>",
                "secret": "<hex-string-client-secret-from-above>"
            }
}

Hi tjmehta,
I was able to get my if and secret, and will input that into my config when I get a chance (this week is pretty busy unfortunately). Many thanks for the info!
By the way, I’ve been looking into since I last posted on this issue, and the home-assistant  implementation for 8sleep uses pyeight repo in the background. There are some trade offs compared to your repo but their implementation just required username and password of the eight sleep account: https://github.com/mezz64/pyEight/blob/master/pyeight/user.py
I’ll take a closer look soon, thanks again the the response!

Just tried it, works great!

Here's what I have in homebridge and it's not working. Any ideas?
{
"platform": "ExampleHomebridgePlugin",
"email": "email",
"password": "pass",
"oauthClient": {
"id": "08xxxxxxx",
"secret": "f09xxxxxxx"
}
}

Hm, make sure the platform is "EightSleepPod", and if you're using the home bridge UI to edit the config file, it should point out to you any other syntax errors. Otherwise, you'll have to post the errors your getting to debug.

Getting an error after the new firmware upgrade:
[ExampleHomebridgePlugin] Get TargetHeaterCoolerState Error -> InvalidSideStatusError: invalid currentTargetLevel
at validateSideStatus (/usr/local/lib/node_modules/homebridge-eightsleep-pod/node_modules/eightsleep/src/validateDeviceStatus.ts:76:11)
at Object.validateDeviceStatus [as default] (/usr/local/lib/node_modules/homebridge-eightsleep-pod/node_modules/eightsleep/src/validateDeviceStatus.ts:51:11)
at EightSleepAppApi. (/usr/local/lib/node_modules/homebridge-eightsleep-pod/node_modules/eightsleep/src/EightSleepAppApi.ts:79:32)
at Generator.next ()
at fulfilled (/usr/local/lib/node_modules/homebridge-eightsleep-pod/node_modules/eightsleep/dist/cjs/EightSleepAppApi.js:5:58)
at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:95:5)
{

@carbonappsx same it worked for a few mins before this. any chance you found a fix?

It seems the problem occurs when the unit is set to 0 +- on the app. It has an invalid current target level then,

if someone has the charles app on ios you should be able to snoop on the eightsleep app and figure out what they do for setting the temp to 0, ill get to it eventually.. feel free to PR :-D

I've downloaded Charles but am having trouble getting the client id and secret from the request body. Any help would be appreciated!

I've downloaded Charles but am having trouble getting the client id and secret from the request body. Any help would be appreciated! @tjmehta are you still able to collect them?

Did you ever sort this out? I do notice the ClientID/Secret has changed since the last time I was actively looking at this back in November. Since then, they actually enforce the ClientID/Secret, which they didn't previously.


http:client: Avoid segfault

It gives to user the chance of steal the content buffer without happens
a double free. It's useful to avoid duplicate the buffer's content.
Signed-off-by: Flavio Ceolin<EMAIL_ADDRESS>

@otaviobp could you take a look on it please ?

merged, thanks !


Add kwargs to backend.run

Summary
Details and comments
Fixes #494

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-3 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 64.248%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_ibm_runtime/ibm_backend.py
0
3
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
3143

Relevant Lines:
4892

💛  - Coveralls


Bug Description
Attributes not supported by vue, such as v-model="asdasd" and @click="event"
Relevant Screenshot
No response
To Reproduce
No response
neovim version
NVIM v0.8.3
Build type: Release
LuaJIT 2.1.0-beta3
编译者<EMAIL_ADDRESS>
Features: +acl +iconv +tui
See ":help feature-compile"

     系统 vimrc 文件: "$VIM/sysinit.vim"
         $VIM 预设值: "/opt/homebrew/Cellar/neovim/0.8.3/share/nvim"

Run :checkhealth for more info

Make sure you have done the following

[X] I have updated to the latest version of the plugin.

thanks, fixed

htmlAttribute has several variants in vue, jsx
例如:
vue <template>   <button :type="primary" @click="handleClick" v-if="bol">Hello World!</button> </template>
jsx
The current inserts do not support these types of writing

please open a new bug report
I don't use vue/jsx myself, so I lack the knowledge on the specifics of them. Therefore, someone else will have to do a PR for a proper fix


Added UI Links

A bunch of them

You added some sketch stuff, but didn't add a link to it in the top. Do that and do another pull request


Best Way To Include Multiple Rewards?

Suppose I wanted to have two classifiers dictate the reward signal. Whats the best way to do this outside of just adding their rewards together?

Hi @JosephGatto
Good question. As you suggested I would maybe first try to combine both rewards with some sort of aggregation strategy (sum, mean, weighted sum, etc)
Here I am just brainstorming, but another direction that could make sense would be to train 2 adapters trained each on a different reward classifier then combine those adapters later on. This has been proved to work really well on images recently: https://github.com/huggingface/peft/blob/main/examples/lora_dreambooth/lora_dreambooth_inference.ipynb

So maybe it would also work for text models


log-serialize - patch udp-log

Hi,
Since we've modified the nginx.conf a bit (in order to do some get caching on responses) every time response is served from cache instead of upstream log-serializer/basic.lua fails at
proxy = ngx.var.upstream_response_time * 1000,

when using the udp-log, since the ngx var for upstream is not populated.
Can we do something like in this patch to remedy this ?
diff --git a/kong/plugins/log-serializers/basic.lua b/kong/plugins/log-serializers/basic.lua
index 6571c9d..20d4c0a 100644
--- a/kong/plugins/log-serializers/basic.lua
+++ b/kong/plugins/log-serializers/basic.lua
@@ -8,6 +8,10 @@ function _M.serialize(ngx)
       consumer_id = ngx.ctx.authenticated_credential.consumer_id
     }
   end
+  local local_upstream_response_time=0
+  if ngx.var.upstream_response_time ~= nil then
+         local_upstream_response_time = ngx.var.upstream_response_time
+   end

   return {
     request = {
@@ -27,7 +31,7 @@ function _M.serialize(ngx)
       kong = (ngx.ctx.kong_processing_access or 0) +
              (ngx.ctx.kong_processing_header_filter or 0) +
              (ngx.ctx.kong_processing_body_filter or 0),
-      proxy = ngx.var.upstream_response_time * 1000,
+      proxy = local_upstream_response_time * 1000,
       request = ngx.var.request_time * 1000
     },
     authenticated_entity = authenticated_entity,

Yes, actually this is already done in next since #708, which was necessary because of #594. The log serializer is now like this, and will be part of the next 0.6 release.

Nice to know. Thanks.


Adding grpc-status header value

[Fixes #2679 #2562 #2860]

I can't port this one to 3.1.x because of the test requiring apache httpclient 5


Xaml files outside the current csproj's directory do not appear in the VS solution explorer

In a .NET Core 3 (3.0.100-alpha1-009689) WPF project, including a XAML file from outside the csproj's directory will include the file in the project but not display it in Visual Studio's solution explorer, making it difficult to navigate to during development time.
Sample csproj demonstrating the issue:
<Project Sdk="Microsoft.NET.Sdk.Wpf">
  <PropertyGroup>
    <OutputType>WinExe</OutputType>
    <TargetFramework>netcoreapp3.0</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <!-- These files will appear in VS's solution explorer -->
    <ApplicationDefinition Include="App.xaml" />
    <Page Include="MainWindow.xaml" />

    <!-- This file will *not* appear since it is in a parent directory -->
    <Page Include="../XamlResources.xaml">

    </Page>
  </ItemGroup>

  <ItemGroup>
    <FrameworkReference Include="Microsoft.DesktopUI" />
  </ItemGroup>
</Project>

The recommended approach to porting WPF apps to .NET Core is to create another (netcoreapp3.0) head project in a directory next to the one containing the original NetFx csproj, so I expect this issue to come up a lot while porting WPF apps.

@Pilchie - can you take a look?

This happens because the solution explorer doesn't know what name/path to display the file under the project folder with the Link metadata.  Adding Link="XamlResources.xaml" to the <Page> element makes it show up.
@lifengl I wonder if we could consider a default for Link metadata if it doesn't appear in the file?  Also @dsplaisted and @nguerrera and @davkean.

Yes, we should consider adding Page to the item types that get automatic Link metadata, either in the WPF SDK or just by default.  This issue can be moved to dotnet/cli or dotnet/sdk.
(I would move it myself but the issue mover I normally use seems to be broken).

Same here (with https://github-issue-mover.appspot.com/). 😦
Error creating new issue: NullError: Cannot call "ez" on null

Anyone know who owns that issue mover?


Thread created by SummaryWriter not killed

Hi,
I noticed that the _EventLoggerThread created by summary writer does not get killed by the close method, which will make the number of threads keep increasing until it exceeds the system capacity. Is there any particular reason to not kill these threads?
Best

@wyx-cornell Thanks for filing this issue!
Looking at the code, I believe you're right, and this is a bug.  Have you run into a real-world problem because of this, or did you just notice via code inspection as well?
Would you like to file a pull request to fix this?  Contributions are welcome!

Hi, I am working on this, since haosdent is not working on it anymore.

I'll try to reproduce this, and I'll work on this if I can reproduce this.

@wyx-cornell , according to the latest code and I have tested, number of threads will increase only when you create a new writer every time you want to add_summary. Also, if you kill the thread when close the writer, there will be no thread serving you when you reopen this writer.
how about kill the thread when close the writer and create a new thread when reopen the writer?

I was directed to this particular issue from #2788. I had been using tf.trainLoggingTensorHook which I am assuming uses the lower level summary writers under the hood. I am on tensorflow version 1.2 and upon seeing this I removed the logging and I went from intermittent hanging on start of run to now consistent successful training runs upon every start. I was seeing similar gdb traces on all my threads to the above linked issue and it seems all my threads were in lock. If this isn't the same issue I'm more than willing to open a new one, but it at least seemed related.

@GrandathePanda , can you provide a minimal test case so we can reproduce?

@suiyuan2009
Below is a test that can reproduce:
We want to make a call from C++ to python and TF. Thus, an embedded python interpreter is used.
However, the Py_EndInterpreter call fails with error message:

Fatal Python error: Py_EndInterpreter: not the last thread

This is because there is a daemon thread _EventLoggerThread is still running after the python script finishes.
The output of this test is:
[33, 67.0] [<_MainThread(MainThread, started<PHONE_NUMBER>37984)>, <_EventLoggerThread(Thread-1, started daemon<PHONE_NUMBER>90496)>] Fatal Python error: Py_EndInterpreter: not the last thread
test.py file:
`# import logging as logger
import sys
import time
import tensorflow as tf
from tensorflow.python.platform import tf_logging as logging
from tensorflow.python.client import timeline
import threading
class Train(object):
def run(self):
    v = tf.Variable(dtype=tf.float32, initial_value=tf.constant(1.0))
    a = tf.placeholder(tf.float32, shape=None, name="a")
    b = tf.reduce_mean(a, name="b")
    c = tf.add(b, v, name="c")
    add = tf.assign(v, c, name="assign")
    global_step = tf.contrib.framework.get_or_create_global_step()
    sum = tf.summary.scalar(name="sum", tensor=c)
    global_step_inc = tf.assign_add(global_step, 1)
    hooks = [tf.train.StopAtStepHook(last_step=20)]
    ckpt_dir = './tmp/s1/'
    logging.info('begin run')
    with tf.train.MonitoredTrainingSession(checkpoint_dir=ckpt_dir, hooks=hooks) as mon_sess:
        while not mon_sess.should_stop():
            print (mon_sess.run([global_step_inc, add], feed_dict={a: [1.0, 2.0, 3.0]}))
    logging.info('[finish run]')

if name == "main":
print(sys.version)
Train().run()
print threading.enumerate()
`
And the test cpp code:
`
#include "Python.h"
#include
int main(int argc, char* argv[]) {
// initialize Python
Py_Initialize() ;
PyEval_InitThreads() ; // nb: creates and locks the GIL
PyThreadState* state = PyThreadState_Get();
PyEval_ReleaseThread(state);
PyEval_AcquireThread(state);

const char* pythonFileName = argv[1];
FILE* to_run_script= fopen(pythonFileName, "r");
std::cout << "Run file " << pythonFileName << std::endl;
PyObject * main_module = PyImport_ImportModule("__main__");
PyObject* global_symbol_table = PyModule_GetDict(main_module);;
if (to_run_script != NULL) {
    PyRun_File(to_run_script, pythonFileName, Py_file_input, global_symbol_table, global_symbol_table);

    // if python side has daemon thread. Py_EndInterpreter will throw 'not the last thread' exception.
    Py_EndInterpreter(state);

    if (!PyErr_Occurred()) {
        return 0;
    }else {
        PyErr_PrintEx(1);
    }
}
PyEval_ReleaseThread(state);
Py_Finalize() ;

}
`

I think this is still up-to-date.
See here and here. This test case still runs with latest TF, e.g. here.

@whjiang it's a long time, I'll take a look in this week.


Add “Sign in” button to Live demo tool bar

Open modal and let the user signs in to an existing account

Closing this issue as it is no longer relevant


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #388 (9b7cfbc) into main (2974dfe) will increase coverage by 12.36%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##             main     #388       +/-   ##
===========================================
+ Coverage   83.33%   95.69%   +12.36%
===========================================
  Files           2        1        -1
  Lines         126      395      +269
  Branches       31       53       +22
===========================================
+ Hits          105      378      +273
+ Misses         21       17        -4

Impacted Files
Coverage Δ

src/index.ts
95.69% <100.00%> (+13.69%)
:arrow_up:

example/src/main.ts

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 f9eb50f...9b7cfbc. Read the comment docs.


How to train the model for supporting Arabic Language

Hello,
How I can train the model to support Arabic language
what i understand from  this issue https://github.com/clovaai/donut/issues/77
arabic may not supported ,
but we have team they can create a custom Arabic tokenizer , but we need more info on how to do it and how to integrate it
to donut
thanks in advance

https://github.com/clovaai/donut/issues/135#issuecomment-1426910455


feat: add String.splitOn_of_valid

Add lemmas for Nat.le and List.append.
Add functions on List:

List.rdropSublist drops a sublist from the tail end of a list.
List.splitOnSublist splits a list at every occurrence of a separator sublist. The separators are not in the result.

Add simp lemmas for List.rdropSublist.
State and prove two theorems about String: splitOnAux_of_valid and splitOn_of_valid.

Before this pull request gets merged, I need to replace String.splitOnAux and String.splitOn in Lean core with the modified versions I made:
def splitOnAux' (s sep : String) (b : Pos) (i : Pos) (j : Pos) (r : List String) : List String :=
  if sep == "" then
    let r := (s.extract b s.endPos)::r
    r.reverse
  else if h : s.atEnd i then
    let r := (s.extract b i)::r
    r.reverse
  else
    have := Nat.sub_lt_sub_left (Nat.gt_of_not_le (mt decide_eq_true h)) (lt_next s _)
    if sep.atEnd j then
      if s.get i == sep.get 0
        splitOnAux' s sep i (s.next i) (sep.next 0) (s.extract b (i - j)::r)
      else
        splitOnAux' s sep i (s.next i) 0 (s.extract b (i - j)::r)
    else if s.get i == sep.get j then
      splitOnAux' s sep b (s.next i) (sep.next j) r
    else
      splitOnAux' s sep b (s.next i) 0 r
termination_by _ => s.endPos.1 - i.1

def splitOn' (s : String) (sep : String := " ") : List String :=
  splitOnAux' s sep 0 0 0 []

The function splitOnAux' differs from its original version in two ways. First, it checks whether the separator (sep) is empty. Second, it checks whether sep.atEnd j before it checks whether s.get i == sep.get j. For these reasons, the outputs of the original splitOnAux and its modified version are different for some inputs, as shown in the #eval commands below:
import Std.Data.String.Lemmas

namespace Test

open String
set_option linter.missingDocs false

private def l := "A12a".1
private def m := "B12b".1
private def r := "C12c".1

private def sep₁ := "1".1
private def sep₂ := "2".1
private def acc := ["X", "x"]

#eval splitOnAux  ⟨l++m++sep₁++r⟩ ⟨sep₁++sep₂⟩ ⟨utf8Len l⟩ ⟨utf8Len (l++m++sep₁)⟩
  ⟨utf8Len (sep₁++sep₂)⟩ acc -- output: ["x", "X", "B12b1C", "c"]
#eval splitOnAux' ⟨l++m++sep₁++r⟩ ⟨sep₁++sep₂⟩ ⟨utf8Len l⟩ ⟨utf8Len (l++m++sep₁)⟩
  ⟨utf8Len (sep₁++sep₂)⟩ acc -- ["x", "X", "B12", "C", "c"]

#eval splitOnAux  ⟨l++m++"A".1++r⟩ ⟨"A".1++sep₂⟩ ⟨utf8Len l⟩ ⟨utf8Len (l++m++"A".1)⟩
  ⟨utf8Len ("A".1++sep₂)⟩ acc -- ["x", "X", "B12bAC12c"]
#eval splitOnAux' ⟨l++m++"A".1++r⟩ ⟨"A".1++sep₂⟩ ⟨utf8Len l⟩ ⟨utf8Len (l++m++"A".1)⟩
  ⟨utf8Len ("A".1++sep₂)⟩ acc -- ["x", "X", "B12", "C12c"]

end Test

I'll open an issue on Lean 4 GitHub issues after I get feedback about splitOnAux' and splitOn'.

But why? You say that splitOnAux has different behavior, but this is an auxiliary function, its behavior doesn't matter. Does splitOn have different behavior?
Checking the separator for emptiness in splitOnAux is a bad idea, this is the inner loop of splitOn and sep is a constant. Just check it outside the loop and pass in an appropriate hypothesis.

Looking at the definitions of splitOn and splitOnAux, I think they are correct. splitOnAux has the invariant that sep != "", and j < sep.endPos, which justifies the two aspects that you felt the need to change.

@digama0 There are some errors in splitOnAux', which make splitOn' behave differently from splitOn. I'll reply to your questions after I fix those.
#eval "".splitOn "_" == "".splitOn' "_" -- output: true
#eval "My<>life<>for<>Aiur!".splitOn "<>" == "My<>life<>for<>Aiur!".splitOn' "<>" -- true
#eval "BABABA".splitOn  "A" -- ["B", "B", "B", ""]
#eval "BABABA".splitOn' "A" -- ["B", "B", "BA"]

Why not use the KMP matcher at Std.Data.Array.Match?

@chabulhwi Based on your example, it seems like splitOn is correct and splitOn' is not, no? If you split "BABABA" by "A" the correct answer is ["B", "B", "B", ""], and if you split "ABABAB" by "A" the correct answer is ["", "B", "B", "B"]. splitOn gets both right, splitOn' only gets the second one right.

Right. But I think I can fix this error.

Why not just use and prove properties of the existing function?

(I'll write comments within a few hours...)

But why? You say that splitOnAux has different behavior, but this is an auxiliary function, its behavior doesn't matter. Does splitOn have different behavior?
Checking the separator for emptiness in splitOnAux is a bad idea, this is the inner loop of splitOn and sep is a constant. Just check it outside the loop and pass in an appropriate hypothesis.

I think the current definition of splitOnAux makes it more difficult to prove the following two theorems:
theorem splitOnAux_of_valid (sep₁ sep₂ l m r acc) :
    splitOnAux ⟨l++m++sep₁++r⟩ ⟨sep₁++sep₂⟩ ⟨utf8Len l⟩ ⟨utf8Len (l++m++sep₁)⟩ ⟨utf8Len sep₁⟩ acc =
      acc.reverse++(List.splitOnList.go r (sep₁++sep₂) sep₂ (m++sep₁).reverse).map mk := by
  sorry

theorem splitOn_of_valid (s sep) : splitOn s sep = (List.splitOnList s.1 sep.1).map mk := by
  simpa [splitOn] using splitOnAux_of_valid [] sep.1 [] [] s.1 []

When proving splitOnAux_of_valid, we need to consider the case where the separator is empty. Having splitOnAux check the separator for emptiness helps simplify both sides of the equation we need to prove.

Looking at the definitions of splitOn and splitOnAux, I think they are correct. splitOnAux has the invariant that sep != "", and j < sep.endPos, which justifies the two aspects that you felt the need to change. (Doing the s.get i = sep.get pos check first is a better design, because most characters are not going to be the same as the first character in sep, so this saves one test per character.)

In order to prove splitOnAux_of_valid, we also have to deal with the case where it's true that sep.atEnd j; this is where the problem with the current splitOnAux lies.
When sep.atEnd j is true, the value of sep.get j is default, which is 'A'. In that event, checking whether s.get i == sep.get j is equivalent to checking whether s.get i == 'A'. This leads to the inconsistent behavior of splitOnAux shown below:
#eval "AB".splitOnAux "C" ⟨0⟩ ⟨0⟩ ⟨1⟩ []
#eval "AB".splitOnAux "C" ⟨1⟩ ⟨1⟩ ⟨0⟩ [""]
#eval "AB".splitOnAux "C" ⟨1⟩ ⟨2⟩ ⟨0⟩ [""]
#eval ("B"::[""]).reverse
#eval ["", "B"]

#eval "DB".splitOnAux "C" ⟨0⟩ ⟨0⟩ ⟨1⟩ []
#eval "DB".splitOnAux "C" ⟨0⟩ ⟨1⟩ ⟨0⟩ []
#eval "DB".splitOnAux "C" ⟨0⟩ ⟨2⟩ ⟨0⟩ []
#eval ("DB"::[]).reverse
#eval ["DB"]

In order to prove splitOnAux_of_valid, we also have to deal with the case where it's true that sep.atEnd j; this is where the problem with the current splitOnAux lies.

Like I said earlier, splitOnAux has a precondition that j < sep.endPos, which eliminates this case.

Like I said earlier, splitOnAux has a precondition that j < sep.endPos, which eliminates this case. splitOnAux_of_valid should likewise have this as a hypothesis.

Adding j < sep.endPos to splitOnAux_of_valid as a hypothesis will solve the problems I had. I'll close this pull request and make another one. But I need to take a break first. 🫠


fix snapshot restore and recovery endpoint

Description
Fixed snapshot restore endpoint when wait_for_completion is false.
Also fixed _recovery endpoint needed for testing when wait_for_completion is false.
Issues Resolved
None
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dblock
Thank you for your review.
I think the work is complete.
If there is any work that needs to be done, please let me know.


Adds a post on what is TypeScript

I've been explaining what typescript is quite a lot in the last year, and wanted to centralize the non-technical overview to a single URL.

          2 Warnings

:warning:
Please use TypeScript, capital ‘S’

:warning:
Please use TypeScript, capital ‘S’

  Generated by :no_entry_sign: Danger

OK, I'm happy :shipit:


replaceTracksInPlaylist is missing a range_start parameter

When the replaceTracksInPlaylist method is invoked, Spotify returns an error that the request is missing a range_start parameter. Looking at the Spotify API documentation on replace a playlist's tracks, it would not appear that this field is required, except that it's actually using the same endpoint (https://api.spotify.com/v1/users/{user_id}/playlists/{playlist_id}/tracks) as reorder tracks, for which range_start is a required param.
The fix would likely be to add a range_start parameter (hard-coded to 0?) to the body of the post.
spotifyApi.replaceTracksInPlaylist(userId, playlistId, trackIds)
{ [WebapiError: Missing range_start parameter]
name: 'WebapiError',
message: 'Missing range_start parameter',
statusCode: 400 }

#65

@amarcher The endpoint for replacing all tracks in a playlist doesn't require range_start. It seems to work for me:
var spotifyApi = new SpotifyWebApi();
spotifyApi.setAccessToken('my_access_token');
spotifyApi.replaceTracksInPlaylist('jmperezperez', '01kNoYcRkCcEKaqEW7hXCR', [
  'spotify:track:4kJP8Z888wREJ8bRMWNMuk',
  'spotify:track:16taQAqrrfqCwbkbKFr2OH'
]).then(function(data) {
    console.log(data);
  }, function(err) {
    console.log(err);
  });

outputs
{ body: '',
  headers:
   { server: 'nginx',
     date: 'Mon, 27 Jun 2016 17:24:08 GMT',
     'content-length': '0',
     connection: 'close',
     'cache-control': 'private, max-age=0',
     'access-control-allow-origin': '*',
     'access-control-allow-methods': 'GET, POST, OPTIONS, PUT, DELETE',
     'access-control-allow-credentials': 'true',
     'access-control-max-age': '604800',
     'access-control-allow-headers': 'Accept, Authorization, Origin, Content-Type',
     'x-content-type-options': 'nosniff',
     'strict-transport-security': 'max-age=31536000;' },
  statusCode: 201 }

What is the value of your trackIds variable?
Since there doesn't seem to be an issue with this function, I will close the issue for now. But feel free to reopen it if you want to add any extra information.


Scaffoding Bool values results in incorrect syntax on the Create and Edit models

I have found that if scaffolding a class that contains a Bool value then incorrect syntax is generated on the Create and Edit models (this issue also occurred in .Net Core 2.0):
Steps to reproduce:
public class Customer
    {
        public int CustomerID { get; set; }

        ...

        [Display(Name = "Can Contact?")]
        public bool CanBeContacted { get; set; }

        ...
    }

Then scaffold the class and if you try to build the project it will fail with the following error:
'EditModel' does not contain a definition for 'CanBeContacted' and no accessible extension method 'CanBeContacted' accepting a first argument of type 'EditModel' could be found (are you missing a using directive or an assembly reference?)

The workaround is to change the line in the Create/Edit model from:
<input asp-for="Customer.CanBeContacted" /> @Html.DisplayNameFor(model => model.CanBeContacted)

To:
<input asp-for="Customer.CanBeContacted" /> @Html.DisplayNameFor(model => model.Customer.CanBeContacted)

After this the project builds successfully.

@robinwilson16 - This has been fixed by #709.

It's not fixed though in the latest version from what I can tell.
Looking at #709 it looks like it may have only been fixed in 2.0 with the fix not ported forward to 2.1?
Are there plans to fix in 2.1 as well?
It also adds an extra closing bracket which just ends up appearing as text on the page.
I do hit it every time I add a bool value and can replicate every time.

I installed .Net Core 2.2 last night and see the same issue with bool properties in the model.

I know, not sure why this one is so hard to fix. Trying to scaffold a model will always mess up on bool values generating an incorrectly scaffolded model which you must correct by hand otherwise your project doesn't even compile after scaffoding. It looks like they only fixed the issue in 2.0 but not in 2.1 and the bug has carried over into 2.2. I think the template file probably just needs correcting. Maybe I should have been more specific it was 2.1 in my initial post.

This is fixed in the latest version. Thanks!

Have just logged a new issue as this is still not fully fixed as still generates the additional closing bracket in the create model that shouldn't be there - aspnet/Scaffolding#1212
It doesn't cause syntax errors though like it used to so things have improved.


release: highlights of v0.0.9

/skip-e2e
Signed-off-by: Aslak Knutsen<EMAIL_ADDRESS>

/changelog v0.0.9

/release v0.0.9

/shipit

🎇 🎉 🎆 New version has been shipped! 🚀🚀🚀


URL aliases do not fetch IP addresses

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

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

To Reproduce
Steps to reproduce the behavior:

Go to Firewall > Aliases
Create a new URL (IPs) alias
In the Content field enter: https://raw.githubusercontent.com/crazy-max/WindowsSpyBlocker/master/data/firewall/spy.txt
Click Save
Click Apply

Expected behavior
On the Firewall: Aliases page I can see a new rule with non-zero value in the Loaded# column.
Actual behavior
No IP addresses loaded

Go to Firewall: Diagnostics: Aliases
Select the rule created on step 4

Expected behavior
The list shows fetched IP addresses
Actual behavior
The list is empty

Go to Firewall: Rules: LAN
Create a new rule
Action = Block; Destination = the alias we created on step 4
Click Save
Click Apply

Actual behavior
It doesn't fetch the IP addresses even if I have a rule that utilize the alias we created on step 4

Create a new rule with type URL Table (IPs)
Content: https://raw.githubusercontent.com/crazy-max/WindowsSpyBlocker/master/data/firewall/spy.txt
Refresh Frequency: 0 days, 0.5 hours

Actual behavior
No IP addresses fetched
Screenshots

Environment
OPNsense 23.7.6

Did you apply the aliases?

Did you apply the aliases?

Yes, I did

screenshot states 30 days..... In most cases the log "Firewall: Log Files: General" helps find local configuration issues (usual dns in these cases)

screenshot states 30 days....

I've experimented with different intervals and with different IP lists. It can't fetch the list even when I'm using alias with type "URL (IPs)" (not URL Table (IPs)).

In most cases the log "Firewall: Log Files: General" helps find local configuration issues (usual dns in these cases)

Thanks, I'll check my DNS settings


Best way to specify directory path to output the request

I don't see a way to specify the output directory from command line or the Python module? Can the download_post or other functions be modified to accept a directory path? Or is there a good workaround for this?

For the command line, you can use this: https://instaloader.github.io/cli-options.html#cmdoption-dirname-pattern

For the module, specify dirname_pattern when initializing your Instaloader instance. E.g:
import instaloader as il
l = il.Instaloader(dirname_pattern="...")
...
l.download_post(post, filename)

See the filename specification for possible tokens.

I understand how to pass in a dirname pattern but if you pass in a full directory it doesn't work. It only accepts patterns not for example...
/Users/user/thispath/savehere/

I don't know how you're doing it, but it works perfectly fine here ("dirpath" is not an accepted keyword argument):
$ python3
>>> import instaloader as il
>>> l = il.Instaloader(dirname_pattern="/tmp/pure/path/without/tokens", download_comments=False)
>>> p = il.Post.from_shortcode(l.context, "B2hiXYpBKwm")
>>> l.download_post(p, "test")
/tmp/pure/path/without/tokens/2019-09-17_19-26-45_UTC.jpg [#DoYouFollow footballer broth…] json
True
>>> exit()
$ ls /tmp/pure/path/without/tokens
2019-09-17_19-26-45_UTC.jpg  2019-09-17_19-26-45_UTC.json.xz  2019-09-17_19-26-45_UTC.txt

I'll try again. I tried exactly that and it would put a folder int he current directory named "/tmp/pure/path/without/tokens" and not put the actual file at that path.

Then I suggest you file a proper bug report. It's hard to figure out what's going wrong when you don't show what commands and/or code you're running, and the corresponding output.


Does Docker Swarm support usage of Docker Registry with self-signed certificate?

I've created my cluster based on step from official Docker documentation. It works well, but as soon as I try to login to my Docker Registry I'm getting error message:
$ docker -H :4000 login https://...:443
...
Error response from daemon: Get https://.../v1/users/: x509: certificate signed by unknown authority

Why it does not work?

Apparently I've needed to install registry's self-signed certificate on every cluster node.


Pico Tools - Part 1 of N

@barchetta FYI this is adding a new dep on Handlebars for production use w/in Helidon.


Support for .Net4.6 in LightGBM nuget.

Hi @guolinke,
We need a structure like below to also make .net 4.6 work.

lightgbm

<IP_ADDRESS>

lib

net46

lib_lightgbm.dll
lightgbm.exe

netstandard2.0

runtimes

linux-x86

native

lib_lightgbm.so

osx-x64

native

lib_lightgbm.dylib

win-x64

native

lib_lightgbm.dll
lightgbm.exe

Let me know once you have the nuget ready and I can verify it on ML.NET side before you push it.
@https://github.com/dotnet/machinelearning/issues/494
CC: @eerhardt @shauheen @galoshri @tomfinley @sharpwood

How is progress, can this be released with the ML 0.4 release? @codemzs

For anyone looking for a work around until this issue gets fixed, see https://stackoverflow.com/a/51728825/3680432.
@guolinke - a possible fix for this issue is to follow the same pattern we use in the Microsoft.ML package, which is to add a .props file to the package that adds the native assembly when packages.config is being used.  (Note we also have a .targets file that checks the project is targeting x64.)
Feel free to copy this code into your NuGet package:
https://github.com/dotnet/machinelearning/blob/94a3d90cba7787973d1ee300e0a88a30653b2d9a/pkg/Microsoft.ML/build/netstandard2.0/Microsoft.ML.props
https://github.com/dotnet/machinelearning/blob/94a3d90cba7787973d1ee300e0a88a30653b2d9a/pkg/Microsoft.ML/build/netstandard2.0/Microsoft.ML.targets
Put those 2 files in a build folder in the nupkg and edit any references to Microsoft.ML -> LightGBM.  Note the name of the files need to match the name of your nuget package.

@sharpwood Unfortunately this won't go in for 0.4 release.

@guolinke - MSBuild files go into the build folder.
Check out this document, which has a table showing what each folder in a NuGet package is used for:
https://docs.microsoft.com/en-us/nuget/create-packages/creating-a-package#from-a-convention-based-working-directory
For example, in our Microsoft.ML package, we have MSBuild files in the build folder, .NET assemblies in the lib folder, and native binaries in the runtimes folder:

In Microsoft.ML, we only support netstandard2.0, so that's why we have the MSBuild files in build\netstandard2.0 and our .NET assemblies in lib\netstandard2.0.
However, for LightGBM, since there are no .NET assemblies, you don't need to put anything in the lib folder.  And the MSBuild files can go directly in the build folder (not a subfolder).

Can this issue and original one be closed, since https://github.com/dotnet/machinelearning/pull/1234 has been merged?
@codemzs @eerhardt @guolinke


Javascript client unable to parse DeletePresentationModelCommand

…use javascript client having "DeletePresentationModel" id for DeletePresentationModelCommand
Hi!
Thanks for adding a pull request for Dolphin Platform :)
Please add a short description that explains the changes of your pull request.
We try to review your pull request as fast as possible.

This change is

I did changes for "DeletePresentationModelCommand" to "DeletePresentationModel" in CommandConstant.java because in javascript client  in commandConstants.js  DELETE_PRESENTATION_MODEL_COMMAND_ID is "DeletePresentationModel"

Pull Request Test Coverage Report for Build 3123

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.008%) to 40.189%

Files with Coverage Reduction
New Missed Lines
%

platform/dolphin-platform-remoting-client/src/main/java/com/canoo/dp/impl/client/legacy/communication/AbstractClientConnector.java
1
90.91%

Totals

Change from base Build 3121:
-0.008%

Covered Lines:
4846

Relevant Lines:
12058

💛  - Coveralls

@manish5196 THX

fix for https://github.com/canoo/dolphin-platform/issues/906


Fix incorrect MI_Flags

I noticed while writing code against the MI API that the documentation had values that do not match the values provided by the mi.h header. In all but one case, this appears to be because a 0 was left off the end of a hexadecimal value.
Here are the correct values as specified in the mi.h header:

Flag field
mi.h header value
Decimal representation
Hexadecimal representation

MI_FLAG_NULL
(1 << 29)
536870912
0x20000000

MI_FLAG_BORROW
(1 << 30)
1073741824
0x40000000

MI_FLAG_ADOPT
((MI_Uint32)(1 << 31))
2147483648
0x80000000

You can quickly verify these values using PowerShell:
1 -shl 29

I actually only noticed this typo on one document with one flag, but as it turns out, it existed on three separate docs/flags. I used the following PowerShell to identify all the documents in the doc set that had MI_Flag references, and verified that they were correct.
Get-ChildItem -Recurse | `
    Where-Object { $_.Extension -eq ".md" } | `
    Select-String -Pattern "MI_FLAG_\w+\s" | `
    Group-Object Path -NoElement | `
    Select-Object -ExpandProperty Name

Most used the header notation, although some used the hexadecimal notation. The issues only appeared in articles that used the hexadecimal notation, so I just fixed in place and left in the hexadecimal notation.
Please let me know if you have any questions. Thanks!

@wmgries : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thank you very much!


Exception if you try to open the same study twice

I opened lower cache creek and then opened lower cache creek again.

so the problem we thought was fixed was not actually fixed?
I'd like to get this fixed before we send out the beta.

I think this is a new issue. It is maybe related to the other file locked issue, but we should not allow the same study to open twice. I will check for this. Should be as easy fix.

ohhhhhhhhhhhhhhhh I misunderstood. you have one LCC instance open, and attempt to open another LCC instance (same sqlite file) right?

correct.

This is now fixed in the stage_damage_edited branch. I added logic to check if the selected path is the same as the current study path. I added this logic in two places. One is for the "recents" list and the other is for the "open" menu item.

I am on the stage_damage_edited branch, opened the muncie example study, launched another instance of fda, attempted to open the same study from the recents list, and hit the uncaught exception.

When I attempt to open the same study twice from the open menu item, nothing happens. I think a message would be helpful.
https://user-images.githubusercontent.com/25163165/182042302-1bea3d64-ecea-442f-b103-8e6924a81c44.mp4

I thought about popping up a message but i figured if they are trying to open a study that is already open then we have already succeeded in doing what they wanted. We have opened the study that they selected. It is easy to pop up a message though. Something like "This study is already open." ?

Not totally sure what the best way to handle multiple instances of fda being open. That is not something i have thought about. I don't see anyway of tracking that other than putting a try-catch around the open study and throwing a nicer message if it throws an exception.

I added a try catch around the open study logic. I display the following message:

perfect thank you


[Feature Request] UDP over TCP support

There is actually a standard RFC for proxying UDP packet in HTTP connection. Combining with Unreliable frame in QUIC protocol, it's possible to have UDP problem solved without those "UDP-over-TCP" hack.
However, this requires implementation in both client (either Browser or naiveproxy client), and the QUIC library hprox is using.
Also, it's easy to adopt to UDP-over-TCP hacks other clients are using. But this kind of solution is not future proof and could be broken anytime client updated to a new version. So it's not the solution I'm looking for at this moment.


VM modules requests dcim/interfaces instead of virtualization/interface for getting the interface name

Hi,
the virtual machine module is making a request to the dcim/interfaces api to get the interface name:
DEBUG:urllib3.connectionpool:https://netbox.lab.xxx:443 "GET /api/virtualization/interfaces/?mac_address=92%3Axx%3Axx%3A8a%3A1a%3Ae5&name=ens18&virtual_machine_id=140&limit=0 HTTP/1.1" 200 688
DEBUG:urllib3.connectionpool:https://netbox.lab.xxx:443 "GET /api /dcim/ interfaces/?id=241&limit=0 HTTP/1.1" 200 1408
DEBUG:urllib3.connectionpool:https://netbox.lab.xxx:443 "GET /api/ipam/ip-addresses/?address=213.xxx.xx.x%2F27&limit=0 HTTP/1.1" 200 1045
DEBUG:urllib3.connectionpool:https://netbox.lab.xxx:443 "GET /api/ipam/ip-addresses/595/ HTTP/1.1" 200 993
INFO:root:Assigning existing IP 213.xxx.xx.x/27 to GigabitEthernet1/0/16
The correct call would be:
GET /api /virtualization/ interfaces/?id=241&limit=0
I didn't notice any other impact of this, instead of that the logging is incorrect.

Its this not a duplicate of #226 ?
( is having a pr also)

@KivraChristoffer
I'm not sure if it is the same problem, but the PR is fixing my problem.
Hopefully it can be mereged to the main-branch soon.


Menu to deactivate 2-complexes for specific relators

Upgrade of #35 . Useful for cluttered groups.

since the introduction of the smart relator menu #34, all equations (relators, but multiple relators if there are multiple = signs) get their own line and input field. We could add a small checkbox next to that


Update stable version in repo from 1.9.19 to 1.9.25

Think about update stable version to 1.9.25
Version: stable=1.9.19 (330 days old) => latest=1.9.25 (15 days old)
Installs: stable=22882 (58.56%), latest=2353 (6.02%), total=39077
Click to edit

Update stable to 1.9.26 requested


Another one little enhancement

In merged results mode(and also in single source mode) I suggest to make results info bar fixed, and not move it while use is scrolling results. It will be even more useful in endless page mode.

@Bionus maybe add to 5.6.0 milestone too?

Unfortunately, that's harder than it looks, especially in non-merged mode, where the label should update following scroll, in the same way position: sticky works in CSS. So it's kind of a high cost low return feature. 😞
That's why I prefer to work on the standardized loading and gallery system for the moment.


Change or reassign object typing dynamically for better intellisense

Hello,
I asked a question to stackoverflow without success:
https://stackoverflow.com/questions/44203804/change-or-reassign-object-typing-dynamically-in-typescript
Is it possible or do you plan to implement this feature ?
Thanks in advance for your help.

This is a bug tracker, not a support forum.

Ok, no problem. I will try to find a workaround for this limitation in typescript.


#24 verify email address

Hi Andres, good morning.
Can you please first fix the merge conflicts in this branch before I review it.
This is usually done by merging from master -> this branch
cheers
Rainer

OK i will try


Support reserved_concurrent_executions

Depends on
N/A
What
Support reserved_concurrent_executions. When set to a value other than default, will not lifecycle ignore this value.
Why
Would like to specify a custom value of Support reserved_concurrent_executions via terraform.

Doesn't look like this approach will work because of this
Any thoughts on removing this lifecycle block altogether in a new release?

Doesn't look like this approach will work because of this
Any thoughts on removing this lifecycle block altogether in a new release?

@anaoum yep, this makes it a little tricky.  Had a long convo with DC about exactly this earlier to determine the blast radius.   I think we can; just need to check in with one team as its a breaking change.

I can't provide context and why this is necessary as I don't have access to the link put in this repo and given this is public we probably shouldn't link it. Based on a brief discussion sounds like someones trying to get only 1 running which can run into issues in itself. There's many ways to work around this in this module but if it's a blocker just don't use the module. There's really not a lot going on in this module to rack brains and cause issues over. My opinion...

I would assume consumers would be tagging a version of this module and we can make breaking changes and just increment the version. Though I agree I’d probably be better off just creating the lambda directly.
I’m curious about your comment that “which can run into issues in itself”

I really am missing a lot of context, so take this with a grain of salt, but I've seen some oddball behaviour with concurrency 1 if you have another queue feeding into the Lambda internal queues it's still going to bounce back and potentially trigger dead letter. It's one of a few scenarios I've seen with attempts at using Lambda for only once concurrency. It's certainly a valid solution, just need to make sure that it actually makes sense and to watch out for gotchas. I imagine the ignore after the initial push was done because people needed to control outside the bounds of terraform or had something else dynamically changing it down to 0.


Add interval reset hotkey

Right now this is defined in hotkeys but doesn't actually do anything, so it should either be removed or do something

done & tested. also changed "" to NONE as it's more screen-reader friendly and also makes more sense as we don't actually use quotes


Fixed PendingFileRenameOperations Query

When using your improved version of the PendingReboot script with the CIM-methods I realised that the PendingFileRenameOperations is only returning correct values if the reg key is of type 'REG_SZ'. But if there is more than one rename operation pending this reg key is of type 'REG_MULTI_SZ' and the detection fails.

added an if statement to query the reg key with CIM Method 'GetMultiStringValue' when 'GetStringValue' returns nothing.
added code to improve the PendingFileRenameOperations list by removing empty lines from the list and removing leading characters from the path strings
added / updated the comments to reflect that change
changed parameter 'Name' to 'MethodName'
fixed minor code formatting issues

@realslacker

@3vent-Horiz0n thanks for contributing


Broken link

The link appears broken for me.

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

ID: 0e7fc930-905a-0c66-447a-208e09a09d85
Version Independent ID: 6d952b9e-2e16-0087-9f9f-d72c20069d77
Content: Windows Event Tracing in Microsoft Dynamics AX (White paper)
Content Source: dynamicsax2012-technet/windows-event-tracing-in-microsoft-dynamics-ax-white-paper.md
Product: dynamicsax-2012
GitHub Login: @Khairunj

It should work now.


Without  the class giscus, it shows globally.

Describe the bug A clear and concise description of what the bug is.
I just added script tag in the __layout.svelte.
I didn't add any giscus class or div. But it shows everywhere on my site.
https://svelte-flow.vercel.app/

This is expected behavior. Per the documentation,

Add the following <script> tag to your website's template where you want the comments to appear. If an element with the class giscus exists, the comments will be placed there instead.

By default, it creates a <div class="giscus"> where the <script> tag is placed unless it already exists. You shouldn't put it in __layout.svelte. Only put it in a layout where you want the comments to appear.

Svelte allows to add only on script tag in a file. If I add giscus code in the script tag with other code, it omits an error.
What is the best solution?

Use the dedicated Svelte component (npm install @giscus/svelte) instead of the client script.
or...
I'm also building a native web component but that's still very early in development and I haven't published the repo. You should be able to just use it, e.g.
<script lang="ts">
  import "giscus";
</script>

<giscus-widget
  repo="giscus/giscus"
  repoid="MDEwOlJlcG9zaXRvcnkzNTE5NTgwNTM="
  category="General"
  categoryid="MDE4OkRpc2N1c3Npb25DYXRlZ29yeTMyNzk2NTc1"
  mapping="specific"
  term="Welcome to giscus!"
  reactionsenabled="1"
  emitmetadata="0"
  inputposition="top"
  theme="light"
  lang="en"
/>


explorer.exe is extremely slow after debloating

Describe the bug
my explorer crashes and takes a long time to open after debloating
Expected behavior
for it not to lag?
is there any way to reverse it?

Heya,
Have you restarted your PC since running the script?

Heya,
Have you restarted your PC since running the script?

this has been an issue for weeks

I see. Could you please answer the following to help narrow down the issue:

What exactly is slow? Is it only when opening a new explorer window?
What happens when explorer crashes? Does the window just close out of nowhere or does it show a 'Not responding' error window?
Did this issue start right after running the script?
Did you run the script with the default settings or did you select your own. If you selected your own, what settings did you select for the script?
Are you running Windows 10 or 11?
Do you have any network drives connected to your pc?

What exactly is slow? Is it only when opening a new explorer window?
opening a new explorer window takes about 20 seconds to load
What happens when explorer crashes? Does the window just close out of nowhere or does it show a 'Not responding' error window?
no buttons work then it just stays there and eventually does not respond
Did this issue start right after running the script?
only after restarting the pc after running the script
Did you run the script with the default settings or did you select your own. If you selected your own, what settings did you select for the script?
default settings and re-enabled dark mode
Are you running Windows 10 or 11?
11
Do you have any network drives connected to your pc?
at this very moment no as i’ve been trying to diagnose the issue, but previously yes but they were all local to my second pc
Extra: i’ve resigned and decided to reset the pc in the meantime

Thanks for getting back so quickly.
I'm unsure what part of the script's defaults could cause this, as not many of the default changes affect Explorer directly. If I had to hazard a guess I'd say it's related to disabling bing web search, as that does affect Explorer's search too.
Please let me know if the issue is resolved after resetting your PC.

@Jozkah Do you have any updates on this issue?

@Jozkah Do you have any updates on this issue?

well it's now fixed after reinstalling windows so there's not much to add unfortunately, i still have the same network drives and the same settings, just didn't debloat windows

Thanks for the update. I'm unfortunately unable to pinpoint what may've caused your issue. For that reason I'll leave this issue open for now in case anybody else runs into this issue aswell.

been running into the same problem for a while now, thought it was just a windows 11 bug, but now that you mention it after reinstalling and not debloating, it seems to work fine

The issue does indeed seem to lie with Bing Web Search. I went through the steps described in https://github.com/Raphire/Win11Debloat/discussions/114

Bing Web Search in Windows Search can be restored by running the Enable_Bing_Cortana_In_Search.reg file that is provided by the script, You can find it by navigating to Win11Debloat > Regfiles > Undo.

to reenable it and the immense slowdown went away instantly.