Update JSON validation test to handle UTF-8 Byte-Order Mark (BOM) error

The JSON validation test fails if Byte-Order Mark (BOM) is present in the file and throws an ambiguous error. The validation test should be updated to warn the user to take corrective action to remove BOM. It is recommended that BOM should not be used in UTF-8.
https://tools.ietf.org/html/rfc3629#page-6
Steps to reproduce:

Use notepad++ or some text editor that prepends a UTF-8 BOM
Save a new workbook as UTF-8
Execute jsonValidationTest.js
Test will fail, throwing an error saying unexpected "blank" token was found

SyntaxError: Unexpected token  in JSON at position 0
      at TryParseJson (test/jsonValidationTest.js:149:16)
      at validateJsonStringAndGetObject (test/jsonValidationTest.js:115:15)
      at results.filter.forEach.file (test/jsonValidationTest.js:55:21)
      at Array.forEach (<anonymous>)
      at browseDirectory (test/jsonValidationTest.js:54:18)
      at next (test/jsonValidationTest.js:96:31)
      at /home/travis/build/Microsoft/Application-Insights-Workbooks/test/jsonValidationTest.js:102:25
      at next (test/jsonValidationTest.js:96:31)
      at /home/travis/build/Microsoft/Application-Insights-Workbooks/test/jsonValidationTest.js:106:21
      at FSReqCallback.oncomplete (fs.js:161:5)

Fix diff for the workbook to remove the BOM:
@@ -1,4 +1,4 @@
-<EF><BB><BF>{
+{
   "version": "Notebook/1.0",

I'm tryng to add @ericc1103 to who it is assigned to and somehow it autocompleted to 04diiguyi?  wth github?

That is not me... Now sure why..
Thanks,
Eric

From: John Gardner<EMAIL_ADDRESS>Sent: Thursday, December 13, 2018 10:36 AM
To: Microsoft/Application-Insights-Workbooks
Cc: Eric Chong; Mention
Subject: Re: [Microsoft/Application-Insights-Workbooks] Update JSON validation test to handle UTF-8 Byte-Order Mark (BOM) error (#89)
I'm tryng to add @ericc1103https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fericc1103&data=02|01|Eric.Chong%40microsoft.com|6121cdcda95c4f334dff08d66129e11a|72f988bf86f141af91ab2d7cd011db47|1|0|636803229833587234&sdata=VVyfDsskf7OIemb4WWmSnKXVzHhWXWou4SNOdqG8D6E%3D&reserved=0 to who it is assigned to and somehow it autocompleted to 04diiguyi? wth github?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2FApplication-Insights-Workbooks%2Fissues%2F89%23issuecomment-447073820&data=02|01|Eric.Chong%40microsoft.com|6121cdcda95c4f334dff08d66129e11a|72f988bf86f141af91ab2d7cd011db47|1|0|636803229833587234&sdata=VuwNNCVxKGhWuNrI4OUclRjrwV%2BSHIOeAihztpb7EpI%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAE6stp0M4V9KfrKud8h7GWWUA48-jPTsks5u4p4lgaJpZM4ZQ8zZ&data=02|01|Eric.Chong%40microsoft.com|6121cdcda95c4f334dff08d66129e11a|72f988bf86f141af91ab2d7cd011db47|1|0|636803229833597238&sdata=W2mkJ9JkTjstW6CFIwIa%2Bkmtpn1pLXR6NbcPOaOs1cE%3D&reserved=0.


Set vmid from profile, fix gitignore for local

What does this PR do?
Allow vm profile to set vmid number instead of using next available
What issues does this PR fix or reference?
Fixes: https://github.com/salt-extensions/saltext-proxmox/issues/10
Previous Behavior
Only settable automatically

Testing was already in place

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@3a12d12). Click here to learn what that means.
Patch has no changes to coverable lines.

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

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #17   +/-   ##
=======================================
  Coverage        ?   49.90%
=======================================
  Files           ?        7
  Lines           ?     1016
  Branches        ?      185
=======================================
  Hits            ?      507
  Misses          ?      466
  Partials        ?       43

Flag
Coverage Δ

Linux
49.90% <0.00%> (?)

project
43.02% <0.00%> (?)

py310
49.90% <0.00%> (?)

py37
49.10% <0.00%> (?)

py38
50.00% <0.00%> (?)

salt_3006_1
49.90% <0.00%> (?)

tests
59.90% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Idle Session Timeout value above 5 minutes is not being honored in v7.0

Describe the issue:

The Idle Session Timeout value that is configured via the Console app in WSO2 Identity Server v7.0 is not honored when the value is above 5 minutes. When the modal with "It looks like you have been inactive for a long time" appears and the user clicks on the "Go back" button after the idle timeout period, the user is not logged out, contrary to the behavior in the carbon management console that the new Console replaces.

Upon further analysis, it was observed that the "console.session.params.sessionRefreshTimeOut" property was set to 5 minutes by default, which meant that the console app executes /authorize API calls with the prompt=none parameter in 5-minute intervals to refresh the session. Since the default Idle Session timeout value is 15 minutes, this means that the user will not get logged out after 15 minutes and the user will be able to continue using the session after clicking on the "Go back" button.
This needs to be fixed so that the Idle Session Timeout value configured in the UI is being honored and has precedence over recovering the session. A workaround is to set the sessionRefreshTimeOut value to one that is larger than that of the configured Idle Session Timeout in the deployment.toml file.
[console.session.params]
sessionRefreshTimeOut = 28800.0

Additionally, when performing the above analysis, it was observed that the Console app also does not perform checksession calls according to the defined time interval. This should also be fixed along with the above.
[1] - https://is.docs.wso2.com/en/next/guides/account-configurations/login-security/session-management/
Environment information :

Product Version: v7.0

Fixed with

https://github.com/wso2/identity-apps/pull/6135


Enable GitHub actions access to packages

Currently, GH actions fails to push image and the following settings would need to be enabled.
https://github.com/docker/build-push-action/issues/463#issuecomment-924995909

I think I just changed this setting. Let me know if it is resolved. Also, I noticed the package visibility is private for now. Let me know if/when I should change that to public.

@praveingk Is this resolved? If so, can you close it?

Yes, This seems to be fixed, with release action being successfully executed.


Wrong MinecraftCapes cape

Describe the bug
Cosmetica claims that this is my MinecraftCapes cape, but it is not.
To Reproduce
Steps to reproduce the behavior:

Cosmetica settings
Customize Cosmetics
Cape
Remove Cape

Expected behavior
Expected the Cosmetica cape to be removed, but somehow it was replaced by this instead. It'd be understandable if that was actually the one I have on MinecraftCapes, but I don't know, what cape that is.
Screenshots

Setup (please complete the following information):

OS: Manjaro Linux 22.0.0
Minecraft Version 1.19.2
Version 1.2.1

Additional context
Add any other context about the problem here.

I guess this is the replace feature.


Does this work on Android?

Hi, I have not tested it myself yet, but is it possible to deploy the Unity App on an Android device with functional ROS2 communication?

FYI: It does not

Thanks for checking :+1:
We currently support Ubuntu and Windows, but there is a chance for android in the future.


Can't receive generated code

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe.
After an update, LinkingMethod.js never worked again
the code I'm using is:
npm install https://github.com/MobCode100/whatsapp-web.js.git
const { Client, LinkingMethod, LocalAuth} = require('whatsapp-web.js');
const client = new Client(
    {
        linkingMethod:  new LinkingMethod({
            phone: {
                number: "<+556292286543>"
            }
        }),
        puppeteer: {
            args: ['--no-sandbox'],
        },
    }
);
client.on('code', (code) => {
    console.log('CODE RECEIVED', code);
});
client.on('ready', () => {
    console.log('Client is ready!');
});
client.initialize();

Describe the solution you'd like.
I would like you to solve the error while maintaining the same code structure
Describe an alternate solution.
No response
Additional context
No response

Hi, I see that the phone number has the '<' '>'. You should remove the characters, it's meant to be a placeholder when I'm giving an example code.

Hi, I see that the phone number has the '<' '>'. You should remove these characters, they're meant to be a placeholder when I'm giving an example code.

Yes, I already removed '<' '>'
but the code is still not generated even for the number +556292286543
nor the number +5562992286543 (including the addition of the extra number 9 that exists in my country)
I updated the library, but even so the code doesn't even give an error, and it doesn't generate the code numbers. Is there a log that I should put to locate the error?

Yes, I already removed '<' '>'
but the code is still not generated even for the number +556292286543
nor the number +5562992286543 (including the addition of the extra number 9 that exists in my country)
I updated the library, but even so the code doesn't even give an error, and it doesn't generate the code numbers. Is there a log that I should put to locate the error?

I'm away from my pc now, but could you add configuration headless: false under puppeteer ? And see which part it stuck.

I'm away from my pc now, but could you add configuration headless: false under puppeteer ? And see which part it stuck.

this error is reported, and actually in the puppeteer window it just opens whatsapp-web but does not select anything
`C:\Users\Phelipe\node_modules\whatsapp-web.js\node_modules\puppeteer-core\lib\cjs\puppeteer\common\Connection.js:329
error: new Errors_js_1.ProtocolError(),
^
ProtocolError: Waiting for selector .//span[text() = "Link with phone number"] failed: Protocol error (Runtime.callFunctionOn): Destination closed.
in C:\Users\Phelipe\node_modules\whatsapp-web.js\node_modules\puppeteer-core\lib\cjs\puppeteer\common\Connection.js:329:24
in the new promise ()
in CDPSessionImpl.send (C:\Users\Phelipe\node_modules\whatsapp-web.js\node_modules\puppeteer-core\lib\cjs\puppeteer\common\Connection.js:325:16)
in ExecutionContext._ExecutionContext_evaluate (C:\Users\Phelipe\node_modules\whatsapp-web.js\node_modules\puppeteer-core\lib\cjs\puppeteer\common\ExecutionContext.js:211:46)
in process.processTicksAndRejections(node:internal/process/task_queues:95:5)
in async JSHandle.evaluateHandle (C:\Users\Phelipe\node_modules\whatsapp-web.js\node_modules\puppeteer-core\lib\cjs\puppeteer\common\JSHandle.js:97:16)
in async WaitTask.rerun (C:\Users\Phelipe\node_modules\whatsapp-web.js\node_modules\puppeteer-core\lib\cjs\puppeteer\common\WaitTask.js:114:28)
in async Promise.all (index 1)
in async TaskManager.rerunAll (C:\Users\Phelipe\node_modules\whatsapp-web.js\node_modules\puppeteer-core\lib\cjs\puppeteer\common\WaitTask.js:197:9) {
original message: ''
}`

All work very well:
npm uninstall whatsapp-web.js
npm install https://github.com/MobCode100/whatsapp-web.js.git

const { Client, LinkingMethod, LocalAuth} = require('whatsapp-web.js');
const client = new Client(
{
linkingMethod: new LinkingMethod({
phone: {
    number: "+447721734626"
}
}),
puppeteer: {
args: ['--no-sandbox'],
headless: false
},
}
);
client.on('code', (code) => {
console.log('CODE RECEIVED', code);
});
client.on('ready', () => {
console.log('Client is ready!');
});
client.initialize();

Works well even with headless: true. But sometimes get this error (no matter of headless value):

            throw new Error('Node is either not clickable or not an HTMLElement');
                  ^

Error: Node is either not clickable or not an HTMLElemen

I think it cause of too fast page loading.

I think it cause of too fast page loading

I think so too, but it's hard for me to reproduce the error consistently. I'll look for it later, thanks for the logs & the error messages 👍🏻.

I must have something wrong with my PC, I tried countless times and the code is not passed, and in the puppeteer window, it just stays on the QRcode authentication screen

I think your problem is related to the error above. I'll try to take a look, must be a problem with the selector.

All work very well:
npm uninstall whatsapp-web.js
npm install https://github.com/MobCode100/whatsapp-web.js.git

const { Client, LinkingMethod, LocalAuth} = require('whatsapp-web.js');
const client = new Client(
{
linkingMethod: new LinkingMethod({
phone: {
    number: "+447721734626"
}
}),
puppeteer: {
args: ['--no-sandbox'],
headless: false
},
}
);
client.on('code', (code) => {
console.log('CODE RECEIVED', code);
});
client.on('ready', () => {
console.log('Client is ready!');
});
client.initialize();

I think it cause of too fast page loading.

I must have something wrong with my PC, I tried countless times and the code is not passed, and in the puppeteer window, it just stays on the QRcode authentication screen

Additional info about env:
node: v21.6.1
npm: 10.2.4
And this might seem strange, but you might try creating it in a new folder, because sometimes it crashes (wwebjs) even if I don't change anything, but if I copy it to a new folder, everything works fine.

Hi @Rasket, I think the problem 'Node is either not clickable or not an HTMLElement' should have been fixed with the new update. The problem is because the link with phone button was clicked too fast, so I added a delay to prevent that.
@pfnh26 Hopefully your problem was fixed with the new update. Just npm update or reinstall, and can you please retry that again?

Hi @Rasket, I think the problem 'Node is either not clickable or not an HTMLElement' should have been fixed with the new update. The problem is because the link with phone button was clicked too fast, so I added a delay to prevent that.
@pfnh26 Hopefully your problem was fixed with the new update.
Just npm update or reinstall, and can you guys please retry that again?

Still nothing happens in the puppeteer window, not generating the code, in this case the option to generate the code by number is not clicked.
I updated via npm, reinstalled but it didn't work

I updated via npm, reinstalled but it didn't work

Is the WhatsApp web not in English? Can you inspect element the button and show me the html code?

I updated via npm, reinstalled but it didn't work

Is the WhatsApp web not in English? Can you inspect element the button and show me the html code?

Sorry for the delay, I believe it's because of this, would there be any way to force the page to open in English

Conectar com número de telefone

Sorry late, I've done a little programming on my phone and have made some changes to support multiple languages but I can't test them now as I'm away from my PC.
Can you try and see if it'll work now?

Sorry late, I didn't expect another language other than English. I even used VPN to test that but the page stays in English. Turns out it's because of cookies.
I've done a little programming on my phone and have made some changes to support multiple languages but I can't test them now as I'm away from my PC.
Can you try and see if it'll work now?

the button was clicked, but the code was not generated due to this error:
`C:\Users\Phelipe\node_modules\whatsapp-web.js\node_modules\puppeteer-core\lib\cjs\puppeteer\common\ElementHandle.js:781
throw new error (error);
^
Error: node is not of type HTMLElement
in ElementHandle._ElementHandle_scrollIntoViewIfNeeded (C:\Users\Phelipe\node_modules\whatsapp-web.js\node_modules\puppeteer-core\lib\cjs\puppeteer\common\ElementHandle.js:781:15)
in process.processTicksAndRejections(node:internal/process/task_queues:95:5)
in async ElementHandle.click (C:\Users\Phelipe\node_modules\whatsapp-web.js\node_modules\puppeteer-core\lib\cjs\puppeteer\common\ElementHandle.js:444:9)
in async handleLinkWithPhoneNumber (C:\Users\Phelipe\node_modules\whatsapp-web.js\src\Client.js:338:17)
in async Client.initialize (C:\Users\Phelipe\node_modules\whatsapp-web.js\src\Client.js:423:17)
Node.js v18.17.0`

but the code was not generated due to this error:

Fixed, try again. Sorry for the trouble I've caused.

Thank you very much, it worked perfectly, I hope to complete my project as quickly as possible so that I can learn and help update your repository.
But for now, thank you very much!!

Thank you very much, it worked perfectly, I hope to complete my project as quickly as possible so that I can learn and help update your repository.
But for now, thank you very much!!

Thank you so much too! For reporting the problem. Now, this feature can be used by non-English countries which greatly improve this library. I wish you luck for your endeavors.
I will close this issue, feel free to open it back if the problem recurs. Who knows when will WhatsApp Web change its code.


add link prefix: string option

/fixes #259
Builds on previous commits / PRs (#268 and #271) which were needed to fix implicit some assumptions in the glob patterns and construction of the basename to which to prepend the prefix.

This looks good, but I'll wait until #271 is resolved/merged before merging this. Do you think it would be helpful to expand a bit on the documentation in the README for this? Or working this into the full example? I think the main use case for this will be to prepend: '.' along with a glob, right?

Thank you! One question, just to confirm, was the last * in *files** a stray asterisk (076939da8ee683d3cba791914c227f29c21fe042)?

Yup - extra - thanks for the catch.


UI.Window show(): crash after compilation with SDK 3.1 and latest pebble.js

I just compiled my Watchapp with the latest SDK 3.1 and latest pebble JS. Before it worked without problems but now it crash every third or fourth time execution this code:
var setSceneSuccessWindows = new UI.Window({fullscreen: true});
var image = new UI.Image({
  position: new Vector2(0, 0),
  size: new Vector2(144, 168),
  image: successImage,
  backgroundColor: 'black'
});
setSceneSuccessWindows.add(image);
setTimeout(function () {
  setSceneSuccessWindows.hide();
}, 1000);
setSceneSuccessWindows.show();
}

Crash log:
[INFO    ] E ault_handling.c:77 App fault! {000d9985-39b4-42b2-8d92-e2fa6e329a07} PC: 0x1b22 LR: 0x1aed
[WARNING ]    Program Counter (PC):     0x1b22 /Users/mactron/dev/pebble-dev/projects/simbble_hue/src/util/list1.h:42
[WARNING ]      Link Register (LR):     0x1aed /Users/mactron/dev/pebble-dev/projects/simbble_hue/build/../src/simply/simply_res.c:80
[INFO    ] JS: stopping app: 000D9985-39B4-42B2-8D92-E2FA6E329A07 simbble hue

Thanks for reporting the issue, and the logs! Hmm, simply_res.c:80 is not a call to a list1 function. It seems you are on an older commit, such as 020e55b3ae78857686fbe648a199921d72e7e2a4.
Can you try again with master fa651024703fd8a2000766cdb2ade0232d02e352 merged into your branch?

I'm on fce2f79.

Ah, sorry, I should have clarified that I tried your example but it didn't result in a crash for me, so I started investigated beyond that. I definitely want to fix this, but I'll need a way to reproduce this. There are one of four things you can do:

Check the md5sum of your simply_res.c. md5sum src/simply/simply_res.c results in 993edab77811d0abde70410f8a71c11c  src/simply/simply_res.c for me. If it's different, there may have been a bad merge down the line. If it's the same, try the other options.

Is your project private? At the very least, you can email me your build directory (it'll have debug symbols); plus I'll need the source files simply_res.c and list1.h. Although, more helpful is the entire directory. The git history is a plus, but I understand if you want to leave that out. My email is in the commit history.

If that's not an option, I would try making a minimal example that crashes. Try to see if your minimal example you have here crashes (without all of your other logic). If it does, maybe something is missing that causes it not to crash on my end. Try with a public image you're willing to share.

If you're unable to make a minimal example that crashes it, I would try cloning the pebble.js repo and copying (not cherry-picking) your JavaScript app over. If there are modifications to the base pebble.js however, 2 or 3 are better options.

Hey @mactr0n, I eventually found crash issues with windows on basalt that may be what you have been running into! Could you give it another try? If it still crashes, let me know what firmware version you encounter the crash on. I'll load it on a Pebble Time and try it again. For memory issues, that may be the one thing I'm not replicating correctly.

Hi @Meiguro, I'm experiencing the same bug. You can replicate it with this app.js:
var UI = require('ui');

badCard = new UI.Card({
  title: 'bad card',
  body: 'some long long long long long long long long long text'
});
var menu = new UI.Menu({sections:[]});

badCard.show();
menu.show();
badCard.show();

getting the following log:
Installing app...
App install succeeded.
[13:31:34] ault_handling.c:78> App fault! {133215f0-cf20-4c05-997b-3c9be5a64e5b} PC: 0x676e6f6c LR: 0x805de3f
Program Counter (PC)    : 0x676e6f6c ???
Link Register (LR)      : 0x805de3f  ???

Note that if you replace some long long long long long long long long long text with some short text the app will still crash, but on the terminal you won't see the App fault message.
I found out that it started on commit 3367afcdcfeeeacd06b8ca3868db445344dac232 (git bisect <3). I'm using Mac OS X 10.11 Public Beta (15A243d), Pebble SDK 3.2.1.

Thanks @youtux, I've reverted that particular commit. That seemed to just create more of an edge case. Does that solve the issue?

That fixed the crash, but it intruduces a new bug: take this app:
var UI = require('ui');

badCard = new UI.Card({
  title: 'bad card',
  body: 'some long long long long long long long long long text'
});
var menu = new UI.Menu({sections:[]});

// badCard.show();
menu.show();
badCard.show();

It correctly push the menu and the badCard, but when you click back twice, you don't quit the application, but you are stuck on the badCard, which strangely is also in fullscreen, and you have to longClick back to quit.

Ah, that was a hasty modification. I've disabled animations on basalt for the time being since I want master to at least work reasonably. There are no animations, and windows are now only destroyed on deinit which I was hoping to avoid.

👍


Unable to get any error massage, just a cross with 0 that is X 0

When I submit my solution it passes one of the test and gives one test failed with no error massage just a cross and zero.
Error snap shot is provided below:
(https://cloud.githubusercontent.com/assets/21168511/18250016/659dac3c-734f-11e6-8d37-31761af865bd.png)
Program text is provided too:
Program.txt

Which kata is this for?

Link for the Kata is given below:
https://www.codewars.com/kata/common-denominators
Can you please also tell the meaning of the error "cross zero" as shown in snap shot.
Thank you

I believe this is not Codewars issue.
Your solution is simply failing the first test. I think that kata also tests empty input and expects zero.


Night Battle Indicator on Practice

I dunno if this is intended but from what i tried night battle indicator on Practice is always cross marked while its actually need to night battle if want to secure better score

It sounds like a bug or the API actually doesn't provide that.

ah @bamboo3250 have you checked the response? I haven't looked at it and still left PvPNight as an empty function

@bamboo3250 after re-checking the image, seems like the fourth eq counts collided with fuel and ammo gauge on single fleet,

@dragonjet I'm going to check it now in case you haven't fixed it yet xD
@McDohl oh that's true. I didn't notice it before. Will fix it now :open_mouth:


Working with Null Values and non-mandatory fields

How do I specify non-mandatory fields or fields that can allow NULL values?

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

ID: 70f6104a-a223-4bb3-4101-f553002b9b26
Version Independent ID: 15e79e80-8d46-7cd5-c883-6cc0f5940a1b
Content: Scaffolded Razor Pages in ASP.NET Core
Content Source: aspnetcore/tutorials/razor-pages/page.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

That's the default.


[PE-D][Tester A] Edit meetup date accepted an invalid date and parse it to a valid date silently

steps to reproduce:

Start with default data
Switch to meetup mode
edit 1 f/2000-02-31 00:00

expected: "Invalid date" error
actual: The app parse the date to "2000-02-29 00:00" and edit proceed

Perhaps this could lead to unexpected behavior, for example, I wanted a meeting from 13 Feb, but I made a type as 31 Feb, and the resulting date become 29 Feb.

Labels: severity.Low type.FeatureFlaw
original: KengHian/ped#10

Mention in UG as a warning for 31 Feb.


Add support for Python 3.

Does it build against both 2.7 and 3.6?

You did a lousy job - it was broken on both Python 2.7 and 3.4. I re-implemented Python 3 support, cleaned up the mess and fixed the breakage you introduced. It's Passover and I feel nice, so I credited you by merging your PR and mentioning you in the contributors list, although I don't think you deserve it.

Indeed I had a problem with my PR code due to a fluke with git. Like you said, it was Passover, I was abroad and couldn't respond to your questions until I saw you already merged it.
Sorry for the trouble, hope the base code helped your fix (I guess it did if you decided on the contribution thing).


feat: automatically prompt user for updated credentials if they become invalid during use

Resolves #102.
This PR adds logic to detect 401 (Un-authorized) responses from Sonatype IQ Server and will open the extensions Options Page with a notice prompting the user to update their credentials (currently in English only).

@maurycupitt has confirmed locally also.

:tada: This PR is included in version 2.13.0 :tada:
The release is available on:

Chrome Web Store
v2.13.0
GitHub release

Your semantic-release bot :package::rocket:


Make ansi-term-color-vector modification compatible with Emacs 24.3

:+1:
This fixed my ansi-term issue on a combo of Mac OSX Yosemite, iTerm2, GNU Emacs <IP_ADDRESS> and fish shell.

Shouldn't this have been a PR against chriskempson/base16-builder? Regenerating the themes for this repo from the builder will wipe out these fixes; meanwhile anyone using base16-builder directly to generate their emacs themes doesn't pick up this fix. // @chriskempson

@toolbear yes, it should. I just checked upstream and there is no issue or pull request. @fgallina would you do that, please? It's your code, after all.


OrangePi5 FP16_VA = 0, while been activated for llama.cpp

Prerequisites
Please answer the following questions for yourself before submitting an issue.

[+] I am running the latest code. Development is very rapid so there are no tagged versions as of now.
[+] I carefully followed the README.md.
[+] I searched using keywords relevant to my issue to make sure that I am creating a new issue that is not already open (or closed).
[+] I reviewed the Discussions, and have a new bug or useful enhancement to share.

Current Behavior
I have runned pip install llama-cpp-python and llama.cpp make on OrangePi5
And saw that with python it's 20% slower
I found that it's may be relevant to the difference in compilation parameters
llama-cpp-python
AVX = 0 | AVX_VNNI = 0 | AVX2 = 0 | AVX512 = 0 | AVX512_VBMI = 0 | AVX512_VNNI = 0 | FMA = 0 | NEON = 1 | ARM_FMA = 1 | F16C = 0 | FP16_VA = 0 | WASM_SIMD = 0 | BLAS = 0 | SSE3 = 0 | SSSE3 = 0 | VSX = 0 | MATMUL_INT8 = 0 |

llama.cpp
AVX = 0 | AVX_VNNI = 0 | AVX2 = 0 | AVX512 = 0 | AVX512_VBMI = 0 | AVX512_VNNI = 0 | FMA = 0 | NEON = 1 | ARM_FMA = 1 | F16C = 0 | FP16_VA = 1 | WASM_SIMD = 0 | BLAS = 0 | SSE3 = 0 | SSSE3 = 0 | VSX = 0 | MATMUL_INT8 = 0 |

Expected Behavior
You can see that in llama-cpp-python FP16_VA = 0 while it's different for the default installation of llama.cpp
If it not uses vector arithmetic on ARM sure it will be slower, I don't see the way to activate it, it's auto thing

Observed same FP16_VA=0 issue on AWS c6gd instance type (Graviton2 CPU)

Prerequisites
Please answer the following questions for yourself before submitting an issue.

[+] I am running the latest code. Development is very rapid so there are no tagged versions as of now.
[+] I carefully followed the README.md.
[+] I searched using keywords relevant to my issue to make sure that I am creating a new issue that is not already open (or closed).
[+] I reviewed the Discussions, and have a new bug or useful enhancement to share.

Current Behavior
I have runned pip install llama-cpp-python and llama.cpp make on OrangePi5 And saw that with python it's 20% slower I found that it's may be relevant to the difference in compilation parameters
llama-cpp-python
AVX = 0 | AVX_VNNI = 0 | AVX2 = 0 | AVX512 = 0 | AVX512_VBMI = 0 | AVX512_VNNI = 0 | FMA = 0 | NEON = 1 | ARM_FMA = 1 | F16C = 0 | FP16_VA = 0 | WASM_SIMD = 0 | BLAS = 0 | SSE3 = 0 | SSSE3 = 0 | VSX = 0 | MATMUL_INT8 = 0 |

llama.cpp
AVX = 0 | AVX_VNNI = 0 | AVX2 = 0 | AVX512 = 0 | AVX512_VBMI = 0 | AVX512_VNNI = 0 | FMA = 0 | NEON = 1 | ARM_FMA = 1 | F16C = 0 | FP16_VA = 1 | WASM_SIMD = 0 | BLAS = 0 | SSE3 = 0 | SSSE3 = 0 | VSX = 0 | MATMUL_INT8 = 0 |

Expected Behavior
You can see that in llama-cpp-python FP16_VA = 0 while it's different for the default installation of llama.cpp If it not uses vector arithmetic on ARM sure it will be slower, I don't see the way to activate it, it's auto thing

At first glance I thought we met the same problem, but I doublechecked and found out that in my situation even llama.cpp had been built with FP16_VA=0.
I am quite sure my device supports FP16_VA since it supports armv8.2-a.
So could you please kindly teach me how to eable FA16_VA option during make in llama.cpp itself? Did you barely use make and by default it turned on FP16_VA?

@Yuchen-Cao just run make
So can't help you with your problem

same question

Hello,
For me when building with cmake I've got the FA16_VA enabled when passing march armv8.2-a+fp16 or native as CFLAGS.
mkdir build
cd build/
cmake -DCMAKE_C_FLAGS="-march=armv8.2-a+fp16" ..
cmake --build . --config Release

So I think you can get it enabled by installing like:
CFLAGS=-march=armv8.2-a+fp16 pip install llama-cpp-python

I didn't try it, hope it helps.

@GiviMAD it works for me, great thanks
After CFLAGS=-march=armv8.2-a+fp16 pip install llama-cpp-python
Got FP16_VA = 1


Doesn't work in Edge

Seems the status-bar won't show in Edge (87.0.664.66, Chromium-based). Could be a CORS issue.

@marek-dudas thanks for the report on this, I'll try and take a look, see if I can spot what's going on. 👍

@marek-dudas thanks for the report on this, I'll try and take a look, see if I can spot what's going on. 👍

Actually, now I am not sure if it wasn't just my mistake, as it started working. But I got some CORS error at some point in Edge, although being it from localhost.

Actually, now I am not sure if it wasn't just my mistake, as it started working. But I got some CORS error at some point in Edge, although being it from localhost.

O.k. that's good to know
I'll do some testing anyway, just to be sure. Thanks again for letting us
know.
On Fri, 8 Jan 2021, 14:06 Marek Dudas<EMAIL_ADDRESS>wrote:

Actually, now I am not sure if it wasn't just my mistake, as it started
working. But I got some CORS error at some point in Edge, although being it
from localhost.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sorry-app/status-bar/issues/64#issuecomment-756772670,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFWEKZPTIUPHDIIPBHLMDDSY4GP3ANCNFSM4V2OVZ6Q
.

O.k. that's good to know
I'll do some testing anyway, just to be sure. Thanks again for letting us
know.
On Fri, 8 Jan 2021, 14:06 Marek Dudas<EMAIL_ADDRESS>wrote:

Actually, now I am not sure if it wasn't just my mistake, as it started
working. But I got some CORS error at some point in Edge, although being it
from localhost.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sorry-app/status-bar/issues/64#issuecomment-756772670,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFWEKZPTIUPHDIIPBHLMDDSY4GP3ANCNFSM4V2OVZ6Q
.


inputrc test fails intermittently

e.g. https://travis-ci.org/thoughtbot/gitsh/jobs/504373680
I thought I had tracked down the cause of the problem in 08398ed39ee3bf98425ea292d8fe95e080ebe075, but it's still failing sometimes. The supposed fix did do something, because prior to that commit I could reproduce the failure locally by running RSpec with the same seed, but after that commit I can't reproduce the issue reliably.

Breaking up the travis.yml script into multiple lines in 92b06f143dc2668208ad7d4dd1db4b8dad628a04 seems to have helped, although I have no idea why!

Spoke too soon: it was just a coincidence that we had a few consecutive green runs after that change.


UF-4484: Adjusted circuit breaker settings

Removed circuitbreaker annotation on feign client
Updated Dept44 to version 1.30

3 issues exist in sonatype. Boyscout?


Can I get the position of a character to align an annotation?

I have an editor where the user can highlight a region of text and create an annotation that is aligned with the highlight.  It is implemented using CodeMirror, which exposes methods charCoords and cursorCoords that give the coordinates of a bounding box around, respectively, the character at a given position in the editor, and an imaginary cursor at a given position in the editor. When the editor contents change, CodeMirror emits an event, and I recheck all the character positions I am tracking and adjust the positions of the annotations as necessary.  I am investigating switching to draft.js, but I couldn't find any API to implement this functionality in the docs.  Do I need to fish spans corresponding to the highlights out of the DOM and call getBoundingClientRect on them?

Same puzzle to me. I only find that there is a method to identify block level position, not character.

@flarnie Could you help us with this question?

Do I need to fish spans corresponding to the highlights out of the DOM and call getBoundingClientRect on them?

yes -- (if i understand the question correctly) as far as i know this is the best solution here. we do something along these lines in our application using draftjs.


Add drop-down navigation menu?

Shall we add drop-down navigation menu? The current menu are basically tabs.

no response for a long time. close now.


Deprecate Autotelic Abstraction

Oleksii called me out hard 😄 for having an interface that will lead to unnecessary breaking changes. While I doubt many, if any, will create their own implementation of the IPasswordlessClient it is technically a breaking change to add a method to a public interface while it is not to add a new method to PasswordlessClient.
While I do like having interfaces for the purposes of mocking, the better more resilient thing to do is to make your own interface that you own in your application that wraps the usage you have of any said library. Bitwarden server follows this practice for our uses of Stripe with StripeAdapter. We should recommend users of our library, who want to mock our service, do the same.
What we can do today:

Change examples and documentation that use IPasswordlessClient to PasswordlessClient
Stop adding new methods to IPasswordlessClient and only add the implementation in PasswordlessClient.

When we do other and only when we do other breaking changes we can but don't have to:

Make IPasswordlessClient have the [EditorBrowsable(Never)] attribute.

We could do this before a breaking change without actually breaking anyone but users who had previous uses of IPasswordlessClient and want to use it elsewhere could be thrown off.

Change the AddHttpClient method use PasswordlessClient as service instead of IPasswordlessClient

This changes logs from looking like: System.Net.Http.HttpClient.IPasswordlessClient.ClientHandler to System.Net.Http.HttpClient.PasswordlessClient.ClientHandler this is breaking because some users could use that category name to tune logs. Rare, but could cause some annoyance to consumers.
This doesn't need to be done other than people being slightly confused why the interface name is showing up in the logs when they don't use the interface at all.
This also changes the name of the client for if people are manually configuring/injecting the HttpClient themselves, this is not a situation we support or document anywhere but it's something that can be done before that would no longer work.

Write a recommended migration path of just removing the I from the type.
(NOT RECOMMENDED) we could also take a more aggressive approach and add [Obsolete] to the type but I don't think this is warranted.

Sorry, I didn't mean to call anyone out specifically 😅
As an interim solution, we can also add a remark to IPasswordlessClient to discourage the end-users using it for mocking/faking in tests and recommend creating a domain-local abstraction instead.


When blocks viewed in backend, possible to change block titles dynamically?

Hi,
Is it possible to change the block titles that appear in the backend- eg in the example below, "Project preview" would be replaced with the title of each project?

Hi Seb,
it is not possible with the current implementation but we could provide a hook in the form of a helper function you have to redefine. Here's the code that's currently pulling the title of each block from configuration:
https://github.com/area17/twill/blob/209e63f0d21e909cba6370d9d853c66a1b9fabf0/src/Repositories/Behaviors/HandleBlocks.php#L112-L125
I'm thinking we could replace line 123 by something like:
'title' => getBlockCustomTitle($block, $blockTypeConfig['title']),

where the first parameter is the block object itself, so you can access browsers data inside and use that information as the title, and the second is the default value, which is the current value being used from configuration. The default implementation of this function coming with Twill would only return the default value.

Thanks for going into so much detail. Where/how would I redefine this function? (Sorry if that's a bit of a beginner question!)

You could override the existing function in your Module Repository.
Example:
<?php

namespace App\Repositories;

use A17\Twill\Repositories\Behaviors\HandleBlocks;
use A17\Twill\Repositories\ModuleRepository;
use App\Models\Article;

class ArticleRepository extends ModuleRepository
{
    use HandleBlocks;

    //...

    public function getFormFieldsHandleBlocks($object, $fields)
    {
        $fields['blocks'] = null;
        if ($object->has('blocks')) {
            $blocksConfig = config('twill.block_editor');
            foreach ($object->blocks as $block) {
                $isInRepeater = isset($block->parent_id);
                $configKey = $isInRepeater ? 'repeaters' : 'blocks';
                $blockTypeConfig = $blocksConfig[$configKey][$block->type] ?? null;
                if (is_null($blockTypeConfig)) {
                    continue;
                }
                $blockItem = [
                    'id' => $block->id,
                    'type' => $blockTypeConfig['component'],
                    'title' => getBlockCustomTitle($block, $blockTypeConfig['title']),
                    'attributes' => $blockTypeConfig['attributes'] ?? [],
                ];
                if ($isInRepeater) {
                    $fields['blocksRepeaters']["blocks-{$block->parent_id}_{$block->child_key}"][] = $blockItem + [
                        'max' => $blockTypeConfig['max'],
                        'trigger' => $blockTypeConfig['trigger'],
                    ];
                } else {
                    $fields['blocks'][] = $blockItem + [
                        'icon' => $blockTypeConfig['icon'],
                    ];
                }
                $fields['blocksFields'][] = collect($block['content'])->filter(function ($value, $key) {
                    return $key !== "browsers";
                })->map(function ($value, $key) use ($block) {
                    return [
                        'name' => "blocks[$block->id][$key]",
                        'value' => $value,
                    ];
                })->filter()->values()->toArray();
                $blockFormFields = app(BlockRepository::class)->getFormFields($block);
                $medias = $blockFormFields['medias'];
                if ($medias) {
                    $fields['blocksMedias'][] = collect($medias)->mapWithKeys(function ($value, $key) use ($block) {
                        return [
                            "blocks[$block->id][$key]" => $value,
                        ];
                    })->filter()->toArray();
                }
                $files = $blockFormFields['files'];
                if ($files) {
                    collect($files)->each(function ($rolesWithFiles, $locale) use (&$fields, $block) {
                        $fields['blocksFiles'][] = collect($rolesWithFiles)->mapWithKeys(function ($files, $role) use ($locale, $block) {
                            return [
                                "blocks[$block->id][$role][$locale]" => $files,
                            ];
                        })->toArray();
                    });
                }
                if (isset($block['content']['browsers'])) {
                    $fields['blocksBrowsers'][] = $this->getBlockBrowsers($block);
                }
            }
            if ($fields['blocksFields'] ?? false) {
                $fields['blocksFields'] = call_user_func_array('array_merge', $fields['blocksFields'] ?? []);
            }
            if ($fields['blocksMedias'] ?? false) {
                $fields['blocksMedias'] = call_user_func_array('array_merge', $fields['blocksMedias'] ?? []);
            }
            if ($fields['blocksFiles'] ?? false) {
                $fields['blocksFiles'] = call_user_func_array('array_merge', $fields['blocksFiles'] ?? []);
            }
            if ($fields['blocksBrowsers'] ?? false) {
                $fields['blocksBrowsers'] = call_user_func_array('array_merge', $fields['blocksBrowsers'] ?? []);
            }
        }
        return $fields;
    }
}

@sebastianlenton &  @ifox - The idea is good, but which title should be shown if you have more then one entry in the Browser block?

Thanks @noxify for taking the time to explain that to me- much appreciated, looks like I'm on the right track now.
In my case I'm only allowing one item to be selected in the browser field, but aside from that you raise a very good point!

If you have some minutes, I can give you a running version ;)

Here my running version.
https://gist.github.com/noxify/237b7ee4b5cecffa1feaf5fb5be033fc#file-articlerepository-php-L34

Nice @noxify! However I would not recommend overriding the entire getFormFieldsHandleBlocks function as it is pretty complex and critical (which tells me we should probably split it for better extensibility). This is the only solution for now though, so thanks for sharing, but I think we should update Twill to call that function if it is defined instead.

@ifox - absolutely!
As a temporary solution the code snippet it should be good enough to continue, but it's not maintainable :) looking forward how this could be solved in the future.
Idea: maybe it's possible to create a "flat browser list" or "simple relation list" where the selected relations are shown directly without the "accordion".

Closing for inactivity.


how to create or add custom source and header file in AWS freeRTOS

Describe the bug
i have customised json file, now how add my (json) custom file in AWS freeRTOS?
how to create or add custom source and header file in AWS freeRTOS.
System information
Hardware - ESP32-WROOM-32 D
Operating System Linux-Ubuntu
Version of FreeRTOS (run git describe --tags to find it)
Project AWS FreeRTOS shadow demo
Thank you!

Hi @SGOSAI602,
Could you elaborate little further on what your problem is ? Also could link to shadow demo you are using?

Hi ravibhagavandas,
I have a json file optimized and this file is platform independent. We can use any platform, how to use it in free RTOS, how to use this file and send data to the cloud.
Now how to add my (Json) custom file to AWS FreeRTOS?
Thank you!

Hi ravibhagavandas,
I have a json file optimized and this file is platform independent. We can use any platform, how to use it in free RTOS, how to use this file and send data to the cloud.
Now how to add my (Json) custom file to AWS FreeRTOS?
Thank you!
On Saturday, October 31, 2020, 1:39:38 AM GMT+5:30, Ravishankar Bhagavandas<EMAIL_ADDRESS>wrote:

Hi @SGOSAI602,
Could you elaborate little further on what your problem is ? Also could you link to shadow demo you are using?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi @SGOSAI602
Its not clear from the post what you are trying to do with custom JSON document, but from the first post it seems you are trying to include the JSON within a shadow document.
If so, you can take a look at the shadow demo here. You can make an update to the json document from any client by including the changes in "state" : {  "desired" : { .... }  } section of the shadow. The delta can be received on the device and device can publish back the applied changes in "state" : {  "reported" : { .... }  } section. This will complete the shadow update process for that change.
More details can be found in the document here.

Hi ravibhagavandas,
We are trying to have the same code for all platforms, so I have common codes for data convert into JSON format, now how to use it in AWS freertos code.
i hope now you are clear what we saying.
Thank you!

Thanks for the clarification.
The JSON parser used in the shadow demos is a light weight parser which uses only two APIS: JSON_Search and JSON_Validate (more details in README  here] . You can replace this with your custom code for JSON parsing if needed. To convert data into JSON format, demo uses snprintf to insert data into a predefined template. This could be replaced with your custom code as well.

hi ravibhagavandas,
thank you for understand,
And now my question is how to add my custom file to AWS freeRTOS, which I have to use API from define in this file.
Thank you!

ESP32 project uses CMake as the build system. So you can add your custom source file under afr_module_sources() section within the shadow demo CMakeFile. Header file can be placed in a directory and the directory path can be added to the above CMakeFile as follows:
afr_module_include_dirs(
    ${AFR_CURRENT_MODULE}
    INTERFACE
        "<header file directory path>"
)

hi ravibhagavandas,
as per you suggested add my custom source file under afr_module_sources() section within the shadow demo CMakeFile. but in my freertos demo directory not show this CMakeFile file.

@SGOSAI602,
You are unable to find the CMakeLists.txt, since it is pointing to the master branch. As you have correctly suspected, this is a new file added to the master branch recently. You will be able to make the edits proposed by @ravibhagavandas in the CMakeLists.txt file shown in your screen shot. You can find the file from 202007.00 release here.
Hope this answers your question. Please let us know if you have any more questions.

thank you ravibhagavandas and leegeth, for support and valuable reply.

thank you ravibhagavandas and leegeth, for support and valuable reply.

Hi, @SGOSAI602 .  I'm afraid I might be misunderstanding your question, especially what you mean by "merge".  Let me point to portions of the demo, and see if that helps.  The shadow demo uses a custom JSON document, defining it as a macro with a format string for use with snprintf.
This is where it is defined:
https://github.com/aws/amazon-freertos/blob/145dfbe7c9a9de16ba3c86d996e1b8edf3e41dff/demos/device_shadow_for_aws/shadow_demo_main.c#L89-L97
This is where it is used:
https://github.com/aws/amazon-freertos/blob/145dfbe7c9a9de16ba3c86d996e1b8edf3e41dff/demos/device_shadow_for_aws/shadow_demo_main.c#L911-L921
This link gives further details on the JSON format understood by the shadow service:
https://docs.aws.amazon.com/iot/latest/developerguide/device-shadow-document.html
I hope this shows how you can use JSON with the shadow service.  If I may be of further assistance, please try to be as specific as possible, perhaps showing any code you might already have written.  Thank you.

hi dan4thewin,
saw Json file but not understand the how to replace AWS JSon file with my custom file and then send data on AWS server.
below attached my custom file and aws_iot_demo_shadow code .
aws_iot_demo_shadow.zip
cJSON.zip
I hope you can understand what I am trying to do.
Thank you!

Hi, @SGOSAI602 .   I hope I understand better now.  You have written a JSON library.  You need only use your functions to create your desired JSON document in a buffer.  This buffer is used as the body of an MQTT publish call (see pciUpdateDocument in the call to PublishToTopic() above).  The topic for that MQTT publish comes from the shadow library, also visible in the PublishToTopic() call above.  The shadow library does not call any JSON-related functions, nor does it call MQTT functions.  These steps happen in the application.

I am closing this issue due to inactivity. Please reopen this issue if you have any further questions.


Sublime functions list

How I can add es6 method to function list?

Hey @max-mykhailenko, this is a known issue. The thing is that short method definitions are not scoped as methods so sublime doesn't show them in the symbols list. I haven't figured out a good way to match them yet, but I welcome PRs :)

@max-mykhailenko, this is fixed in https://github.com/6to5/6to5-sublime/pull/25. Package Control should pick up the update in like an hour.


Replace with | replace/find next | replace in selection | replace all

Hello!
The four "replace" in the replace box take too much space and force the editor to resize over 80 columns.
Replacing them with: "Replace w/", "/find next", "in selection" and "all" would save some space.
That would also enable to increase the size of the "replace string" QLineEdit which currently allows to display only the first 3 characters of the replacement string.
?
Thank you

i would like if u assign me on this issue

Hi @mvillion this is a good point. Thanks for reporting this. I think we should not change the "Replace with" one but the other 3, we could change them for "Find next", "In selection", "All". What is your thought on this @ccordoba12 ?
@TediaN97 thanks for offering to solve this. Once @ccordoba12 approves these changes, I'll assign you to this issue!

FYI here is a screenshot of that box.

Here is a screenshot of the current replace box:

What is your thought on this @ccordoba12 ?

I agree, that's a very good suggestion @juanis2112! Let's implement those changes!

@TediaN97 thanks for offering to solve this

Yeah, thanks a lot for that! Please don't forget to read our Contributing guide before getting started.


Add a link to the Ember guide in the Readme.

Awesome!  Thank you!

No problem, can't wait for the bit about Ember.on to get published too.


createnetworkform: Displaying api errors

Shows errors while fetching network offerings. Especially noticeable when there are multiple guest physical networks in a zone but are untagged.
Makes life easier for the user so he knows what's wrong rather than just shown empty dropdowns

@blueorangutan package

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/885 (JID-3713)

Could this break previous behaviour @davidjumani, does old UI show any exception? Also is this shown only to admins?

@rhtyd It copies the behaviour of the old UI even for regular users

@davidjumani actually the error is not very clear, I'll let @DaanHoogland @PaulAngus advise further on this.

@rhtyd Changing the error response will require a backend fix. This just copies the old UI functionality so the user knows that an error has occurred trying to create a network (while fetching network offerings)

+1, reverting back to the old user experience is definitely better than the new UX.
Ideally, we'd find a way such that the admin can't get CloudStack networking in that state, but that's not one for today.


Settings getting overlapped

Issue

Type: Bug
Under the workspace settings tab, when you scroll down multiple settings are getting overlapped. Not able to identify what settings are present where.
VS Code version: Code 1.36.1 (2213894ea0415ee8c85c5eea0d0ff81ecc191529, 2019-07-08T22:55:08.091Z)
OS version: Linux x64 5.0.0-29-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-5300U CPU @ 2.30GHz (4 x 2618)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: unavailable_offviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
0, 0, 0

Memory (System)
7.68GB (0.25GB free)

Process Argv
--unity-launch

Screen Reader
no

VM
0%

That looks like an old version of vscode, I think this was already fixed.


Refactor presto/prestoadmin installation out of base_product_case.

In order to support writing product tests for the YARN integration with
slider, we need to refactor the product tests so that we can reuse the
existing docker/configurable cluster logic with slider-based presto
installation and configuration. This change extracts the installation of
presto-admin, presto, and setting the default topology from base_product_case
and moves it to individual modules.
A quick guide to the changes:
In the beginning base_product_case contained the logic for installing presto and presto-admin on a cluster, and for setting the topology. This worked, and code in base_product_case handled setting up clusters with and without presto for various tests.
With the upcoming changes to support YARN integration, we need to use base_product_test as a base class for tests that install presto using slider. Those tests will share the need to set up a cluster and install presto-admin, but the configuration and presto installation will be different. To make it a little easier to do that work I've done the following:
Pulled most of the installation logic out of base_product_case and separated it into modules that install the following:

prestoadmin_installer: install presto-admin
topology_installer: install the topology
presto_installer: install presto

A cluster type is now defined by the set of installers that have to be run against a bare cluster to bring the cluster to the requested state. Each of docker_cluster and configurable_cluster can supply a bare cluster to base_product_case, which is then responsible for running the installers before returning the cluster to the caller.
Additionally, docker_cluster knows how to commit a cluster as an image to use to create clusters again in the future without going through the expensive installation process again. Some of this capability existed before, but we were only using it for a fully-installed presto cluster. In particular, we were recreating the base image every time we created a base cluster. The new behavior is to save an image for reuse for every cluster type, including for the bare cluster.
The tests currently run against a docker cluster. I'll update the PR with any changes needed to run against a configurable_cluster later.
Update:
I have the smoke tests running against a vagrant cluster and so far things appear to be OK. I'm cautiously optimistic that there won't be a big change to get configurable cluster fully working.
Also, can I please get a look from @cawallin, @rschlussel.

lgtm mod comments. Are you planning to update PR #69 after this?

looks good % small comments and running lint

looks good, please squash the feedback commits together


[Bug]: Invalid Player Data when removing startinghouse

Minecraft Version
1.18.2
Fabric Version
0.51.1
Mod Version
3.1.5
Details
When you remove the value for StartingItem in the config, "StartingHouse" and you just make it a quote, when you start the game, this line gets removed from the config. When you join a single player world, or even a server, you get invalid player config.
If this is not how you remove the starting house, what is the proper way to do this? There is no "true/false" entry".
[19:15:04] [Server thread/ERROR]: Couldn't place player in world
java.lang.NullPointerException: Cannot invoke "com.wuest.prefab.config.ModConfiguration$StartingItemOptions.toString()" because "this.startingItem" is null
	at Not Enough Crashes deobfuscated stack trace.(1.18.2+build.3) ~[?:?]
	at com.wuest.prefab.config.ModConfiguration.writeCompoundTag(ModConfiguration.java:59) ~[prefab-fabric-3.1.5.jar:?]
	at com.wuest.prefab.events.ServerEvents.lambda$playerJoinedServer$1(ServerEvents.java:52) ~[prefab-fabric-3.1.5.jar:?]
	at net.fabricmc.fabric.api.event.lifecycle.v1.ServerEntityEvents.lambda$static$0(ServerEntityEvents.java:42) ~[fabric-lifecycle-events-v1-2.0.3+2d83e92c60-9b014c802997cc2d.jar:?]
	at net.minecraft.server.world.ServerWorld$ServerEntityHandler.handler$fgh000$invokeEntityLoadEvent(ServerWorld:1635) ~[?:?]
	at net.minecraft.server.world.ServerWorld$ServerEntityHandler.startTracking(ServerWorld:1519) ~[?:?]
	at net.minecraft.server.world.ServerWorld$ServerEntityHandler.startTracking(ServerWorld:1480) ~[?:?]
	at net.minecraft.server.world.ServerEntityManager.startTracking(ServerEntityManager:210) ~[?:?]
	at net.minecraft.server.world.ServerEntityManager.addEntity(ServerEntityManager:180) ~[?:?]
	at net.minecraft.server.world.ServerEntityManager.addEntity(ServerEntityManager:161) ~[?:?]
	at net.minecraft.server.world.ServerWorld.addPlayer(ServerWorld:846) ~[?:?]
	at net.minecraft.server.world.ServerWorld.onPlayerConnected(ServerWorld:832) ~[?:?]
	at net.minecraft.server.PlayerManager.onPlayerConnect(PlayerManager:233) ~[?:?]
	at net.minecraft.server.network.ServerLoginNetworkHandler.addToServer(ServerLoginNetworkHandler:130) ~[?:?]
	at net.minecraft.server.network.ServerLoginNetworkHandler.acceptPlayer(ServerLoginNetworkHandler:117) ~[?:?]
	at net.minecraft.server.network.ServerLoginNetworkHandler.redirect$fhk000$handlePlayerJoin(ServerLoginNetworkHandler:563) ~[?:?]
	at net.minecraft.server.network.ServerLoginNetworkHandler.tick(ServerLoginNetworkHandler:67) ~[?:?]
	at net.minecraft.network.ClientConnection.tick(ClientConnection:243) ~[?:?]
	at net.minecraft.server.ServerNetworkIo.tick(ServerNetworkIo:183) ~[?:?]
	at net.minecraft.server.MinecraftServer.tickWorlds(MinecraftServer:918) ~[client-intermediary.jar:?]
	at net.minecraft.server.MinecraftServer.tick(MinecraftServer:851) ~[client-intermediary.jar:?]
	at net.minecraft.server.integrated.IntegratedServer.tick(IntegratedServer:97) ~[?:?]
	at net.minecraft.server.MinecraftServer.handler$chk000$modifiedRunLoop(MinecraftServer:5396) ~[client-intermediary.jar:?]
	at net.minecraft.server.MinecraftServer.runServer(MinecraftServer:676) ~[client-intermediary.jar:?]
	at net.minecraft.server.MinecraftServer.method_29739(MinecraftServer:273) ~[client-intermediary.jar:?]
	at java.lang.Thread.run(Thread.java:833) [?:?]

The comment says it right there for you. It's the word nothing.....

The comment says it right there for you. It's the word nothing.....

What comment????

There is no comment?

Oh...well color me embarrassed...I thought there was!
Sorry for the snarky response! I must be thinking of the forge configuration.
In the field write the word "Nothing" instead of what is currently there ("StartingHouse").
Capitalization is important.
I will have the wiki updated to make sure this is known.

It's alright, we all have our moments. Thanks, I'll swap the value
On Sat, May 7, 2022, 5:21 PM Brian Wuest @.***> wrote:

Oh...well color me embarrassed...I thought there was!
Sorry for the snarky response! I must be thinking of the forge
configuration.
In the field write the word "Nothing" instead of what is currently there
("StartingHouse").
Capitalization is important.
I will have the wiki updated to make sure this is known.
—
Reply to this email directly, view it on GitHub
https://github.com/Brian-Wuest/MC-Prefab-Fabric/issues/64#issuecomment-1120322014,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIL5CKAY4B7S7WFBAMUVJHLVI4CHNANCNFSM5VJWAP7A
.
You are receiving this because you authored the thread.Message ID:
@.***>

wanted to give a heads up on this that there are still no comments in the config just in case other people come across this issue


[WIP] Story activity

This is a new activity for Sugarizer.

When I'm trying to test it, the browser is stuck and I've got a continuous list of error messages.
Any idea?

Is this error coming on every browser every time? Or on any specific browser (Like Safari)?

I've tested only on Chrome.

When I'm trying to test it, the browser is stuck and I've got a continuous list of error messages.

@llaske Thanks for the testing. Fixed.

Nice @saurabhhere it works now. Thanks.
I've got two remarks:

I think that next/previous icons are too small. It will be better to increase size. To easily be usable on touch screens, icons should have the size of a finger. See for example icons size when reading in eBook activity.

It could be nice to navigate directly to an image by clicking on it from the 9 images mode.

@llaske Thanks for your feedback.
I have implemented both suggestions and you can test them now.

@saurabhhere I think there is an issue when switching context. See video capture below.

@llaske Thanks. Fixed.

Nice.
I suggest few improvements:

Will be nice to ensure that the cursor is always at the end of text when switching from an image to another or from a view to another
Will be nice to disable Play/Speak button when there is nothing to Play/Speak so the user will have a visual indication that he doesn't need to click on it
I don't see any way to restart or change the image set. It was possible on the original activity via a Restart button but I think we should be better to do a palette for this. May be it could be a way to decide the number of image at start: 2x2 or 3x3 (take inspiration from Memorize activity)

@llaske Thanks for your suggestions. I have implemented all these features along with Shared mode and you can test them now.

During PDF generation it will be nice to hide items and show a spinner during the process instead of letting images change.
What kind of spinner are you expecting here? We have to render images here because html2canvas is taking screenshots of them and adding them to pdf.

Could you hide the HTML elements then add a spinner GIF like this one on the top.

@saurabhhere Nice job on the PDF export :+1:

@llaske When we log off and then re-login again the onJournalDataLoaded function is returning null as data in Story Activity, it's working fine if we don't log off. Do you have any idea about this? Anything else we have to add to our activity to handle this. Maybe the synchronization with the server is not working fine.

@saurabhhere you're right it come from a synchronization issue in Sugarizer core so the problem is not related to the activity. I'm working on it. Could you fix other issues and clean your code (commented code)?

@llaske Both issues which you mentioned related to storage are due to synchronization issue in Sugarizer core?
I have fixed the issue with export to ODT and added spinner in export to PDF and will update the clean code.

@saurabhhere No, the error reopening instance when the window is closed (instead of activity stopped) should be related to the fact that you probably save the instance only when button stopped is clicked. A better practice is to save it when the instance is initialized (to be sure that it's not null) then to save it regularly, for example when changing view/picture.

@llaske Thanks.
I have fixed that issue which is coming while closing activity without the stop button and also updated the clean code.

Thanks @saurabhhere, it works now.
I've got another issue. There is an error when trying to export sound:

Look like you've copy/paste a code that use an undeclared that variable.  Please double check that the same issue is not present elsewhere in your code.

@llaske My bad, Thanks.
I have updated the PR and checked if the same issue is present somewhere else also.

@saurabhhere I've found the issue on Android - the problem will be the same on iOS.
Because the app size is limited on Google/Apple store the Abecedarium activity is built without the database (images and sounds). See the option minsize in Sugarizer APK Builder. When this option is active, all images/sounds are removed from the package (see here).
The Abecedarium activity and the Journal chooser popup are both able to dynamically detect if the database is stored locally or if the database should be load from internet. It's done by trying to load the _ping.png dummy image during init. If this image can't be loaded locally it mean that the database should be load from internet. See the source code here in Abecedarium and here in Journal chooser popup.
Currently, the Story activity doesn't detect if the database is stored locally or not so all images loading failed when images are not stored locally.
Could you add the same detection process for Story?

@saurabhhere I think I understand the issue. The problem comes from your checkFileExists function here.
Your implementation consider that XmlHttpRequest is run synchronously but:

It's not the default behavior
Synchronous XmlHttpRequest are deprecated
Due to the need of cordova-plugin-wkwebview-file-xhr plugin, XmlHttpRequests in iOS are necessarily asynchronous.

So the current implementation will not work on iOS and has risks of failure in future browsers versions.
Could you refactor the implementation of the checkFileExists function? It will require to pass a parameter to the function for callback and handle asynchronous calls of this method in the activity.

@saurabhhere thanks for your work. Good idea to rely on async and promises.
BTW it seems that it raise a new issue::when a new board is created it's added to the existing one instead of replacing it.
See below:

@llaske, Thanks for pointing that.
Just forgot to clear the image array as per our new asynchronous function.

Thanks @saurabhhere. Good job 👍
Both issues are now fixed: the one on iOS and the one on all platforms.
BTW I find another issue related to generation this time. The generation no longer works as expected. Some images are not generated. Here's for PDF but it's the same issue for other formats and it's not depending of the platform.

@llaske It's working fine on my side in all formats.

Is this issue appear in your system after switching to asynchronous function?

No, I don't think recent changes should affect the export feature.
Maybe you should try once by clearing the browser cache.

I've reproduced the issue on my second PC too.
Just by generating several times PDF I get randomly a PDF with between 3 and 6 images inside.
@saurabhhere could you try the same on your side?


List environment variables in alphabetical order

It's kind of trivial, but it'd be really nice if the environment variables were listed in alphabetical order by default by 'deis config'.

Are they not sorted already? https://github.com/deis/deis/blob/master/client/deis.py#L1005 seems to indicate that it should be. :)

Hah, you are right.  Not sure what I was thinking sorry.  Closing.


Remove the workaround for DEFAULT_HASHING_ALGORITHM ImproperlyConfigured warnings

Depends on the fix of https://github.com/jazzband/django-configurations/issues/263


Connect to UNIFORM bucket

Is there any way to use the adapter to connect to an uniform bucker or it works only with fine-grained ?

Works ok for me just use noPredefinedVisibility


Where can I find simulator for livox HAP?

Hi, I want to get information about livox HAP simulator.
Is there any plan to provide scan mode data of HAP for simulators?
Or if you have a official simulator, please let me know.

Hi, the HAP setup was added, and I think you can simulate it now using the latest repo.


Deployed built-in docker images are not the same as the ones referenced in GitHub/Docker Hub links

Just created an App Service on Linux instance with a PHP 7.3 runtime stack and choosing the option to publish code. Opening an SSH console and inspecting the system I find it's not the same runtime as the ones referenced in the links to GitHub/Docker Hub above (first Q&A in Built-in images section). The actual deployed image seems to be outdated (PHP version is older, no MS-ODBC Driver for SQL Server on Linux installed, etc). Is it the documentation that's imprecise or maybe I'm doing something wrong while creating my instance?
Regards

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

ID: b421fa01-1f39-28a5-78c2-63dd5700c8c5
Version Independent ID: 10ed9c91-16e0-d371-fcfc-d6d5a7f984b2
Content: App Service on Linux FAQ - Azure
Content Source: articles/app-service/containers/app-service-linux-faq.md
Service: app-service
GitHub Login: @msangapu-msft
Microsoft Alias: msangapu

Hi @juanii, thank you for the detailed question. We are currently investigating and will update you when we have accurate information to provide.

Just in case this is relevant, looking at the App Service logs it seems like an older image (appsvc/php:7.3.2-apache_1903071319) is specifically selected when the App Service instance is configured to use the PHP 7.3 runtime:
docker run -d -p 46693:8080 --name xxxxx_0 -e WEBSITE_SITE_NAME=xxxxx -e WEBSITE_AUTH_ENABLED=False -e WEBSITE_ROLE_INSTANCE_ID=0 -e WEBSITE_HOSTNAME=xxxxx.azurewebsites.net -e WEBSITE_INSTANCE_ID=xxxxx -e HTTP_LOGGING_ENABLED=1 appsvc/php:7.3.2-apache_1903071319 /bin/init_container.sh

thanks for the additional information @juanii!  We were able to confirm what you're seeing. We are  reaching out to the engineering team to see if this expected behavior or not.

Hi @juanii, thanks for your patience!  The engineering team is aware of this issue and will be deploying a release in a few weeks which will have the latest image.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Feature/codemagic tutorial

The only remaining thing is to move screenshots from Google Photos into the repository and we're set 🙂


Add a tool to generate IDs for our entities

We should integrate this in the first draft from the data download on #34.
To help in the development, a tool to compute and fix the IDs of already downloaded data will be benefitial too for several reasons:

In case that we change the algorithm to do so on the pre-released data (as we might see some issues on the first round of analysis for the card data)
Will be used to fix issues if some data processing fails and the ID should be fixed afterwards
Might help into the workflow if we decide to remove/split the generation from the initialization step
Might help on development process and or for testing purposes

Some UUID generation first draft is already implemented in YgojsonIDGenerator, although I am thinking to change the implementation to use JUG instead of uuid-creator. Should evaluate.

Usign JUG already done (see #131). Reducing priority to medium cause the tool requires that we already have the data on a persisted way (thus, requires #34).


C API

wonderful project , would you consider to offer C API for this framework, so people could port it to other languages with minimal effort, Python/go/... make community bigger and stronger .
like Qt and FFMPEG , accessible via various of bindings , PyQt/GoQt/...

I provided the C interface before, but I didn't maintain it later.
Now I have provided  the Android JNI interface,you can refer to this to implement the C interface.
I think I have time to get a C interface library later, but it depends on my schedule.

I have now provided the C interface：
https://github.com/xiongziliang/ZLMediaKit/tree/master/api

thanks , i will give a try

took a quick scan , nice job .
i am guessing there will be more methods/interfaces exported later ?

a get started ReadME/documentation would be nice
a sample project/snippet would even be better
consider change common.h/common.cpp naming to something like base.h/base.cpp or init.h/init.cpp, make it self-explanatory .

i am still studying your framework , since i am not experienced in media streaming/processing , i will try to make a golang binding as soon as i get a solid grasp of your work .

Agree, would be appreciated to have a readme or some working examples which demonstrate how to use the C API.

In fact, there are two c sdk test examples ：https://github.com/xiongziliang/ZLMediaKit/tree/master/api/tests.
Since the header file comments are very detailed, I didn't write a separate document。
The header notes are in Chinese may not be friendly to non Chinese users, so I suggest you use Google translation.
As a matter of fact, it's very troublesome for us Chinese to read English annotation codes. Some annotations also need to use translation software. In fact, I reply you in English with using translation software now.  I am sorry for my poor English。


jsonfile has string values that should be ints

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

in api.Files it has 3 (2 new and one old) that should be ints,
and the Dataset version has some that should also be converted


Remove hasValue property and clear button workaround

Description
We only have hasValue as a property because of IE and legacy Edge. See https://github.com/vaadin/vaadin-text-field/pull/277/commits/d504cc9774e7bdeb95782accd41e0c0dc1113edd
It makes sense to remove this in the next major. IMO it's better to do it in a separate PR after Polymer 3 conversion.

This was done in the same PR as Polymer 3 conversion.


Alphabetic order of country dojo's not correct

Hello, just noticed my own dojo is the last on the Belgian list ;-)
http://zen.coderdojo.com/dashboard/dojo-list > List > Belgium
Ieper is last and not in correct alphabetic order.
Mistake by me while editing or in the code?

@FDelporte I'm willing to bet this is something to do with "leper" being in lowercase. We can update the sorting to be case insensitive, but in the meantime you could update it to be uppercase and it would likely be in order. Thanks for the report!

nope, is uppercase in the editor...
the other one I can edit "Roeselare" is correct in the list

In the database it is lowercase, and on the listing. I updated it to be "Leper" uppercase and it appeared correctly alphabetically in the list. We still need to fix the case sensitive issue so this can stay open :)


missing [DOCS] about CSRF deactivation [skip ci]

the docs lacked the part where it showed how to disable the CSRF validation.

Merged. Thank you!


Does not serialize nested properties

Description
Does not serialize nested reference properties
How to reproduce?
I tryed to serialize any model with nested reference properties
Expected behavior
nested reference properties has description
Actual behavior
nested referenece type has None description

Additional info
I think this is trouble with MaxDepth setting of JsonSerializer

Hi @Danika1808 and sorry for the delay, this year has been crazy for me.
I am currently working on a NET8 refactor, this should not occur anymore.
Will keep you posted


Does the handle returned by OpenRequestorToken need to be closed?

Does the handle returned by OpenRequestorToken need to be closed?  If so, there's a resource leak in DokanFileInfo.GetRequestor(), which obtains the handle, then uses it to construct a System.Security.Principal.WindowsIdentity.  Per the reference source, WindowsIdentity calls Win32Native.DuplicateHandle on it after it verifies that it's valid.  The original goes out of scope without being closed.

(Since handle values of both 0 and -1 seem to be invalid to WindowsIdentity, I'd suggest a subclass of SafeHandleZeroOrMinusOneIsInvalid would be appropriate.  And since it doesn't need to be used outside of that function, might I suggest SafeFileHandle?  Otherwise, I can whip up a quick SafeTokenHandle class... but that seems a bit overkill for just one spot.)

Hi @kyanha ,
Thank you for the report, you are right ! The handle needs to be closed and I agree on the solution you suggest. Do you think you could make a PR with the changes ? 🚀

As an aside, do you know anyone who has the proper P/Invoke declarations for the functions in authz.dll?  That's Microsoft's user-mode component that provides access to a vetted and correct ACL evaluation implementation.

(I just looked at the Appveyor test results.  This patch doesn't touch any code that it complains about?)

Pull request #176 rebases the 3 prior commits I made into one unified patch.  (I have closed pull request #175.)


translate edit translations button

Haha! Sorry for not replying sooner, @gotrecillo .
This is a good PR, thank you. It's on the board for the next things to include but unfortunately, because of a bug in the way Backpack loads its lang files, this is considered a breaking change, so I'll have to postpone it a bit, until the next minor update (v3.3.0 - I hope next week).
Why it's a breaking change (maybe you find a better solution than me):

backpack uses {{ trans('backpack::crud.toggle_filters') }} to show translated line, which loads the line from resources/lang/vendor/backpack/crud.php; it won't find that line, since most people have old language files published; so it'll try to go to the fallback file (in the package) and try see if it's there;
the fallback file from the package folder, however, might not be picked up correctly, because only the last loaded backpack package has the fallback file properly loaded; that's because $this->loadTranslationsFrom(realpath(__DIR__.'/resources/lang'), 'backpack'); in the service provider overwrites the namespace, instead of adding files to it;

So the result would be that people that have the crud.php langfile published (most people) will see backpack::crud.toggle_filters instead of the actual texts. Take a look:

I've been searching for a solution to ADD to the namespace instead of OVERWRITE it for ages, but haven't found one yet. If you have any ideas on how to do that, please share. Otherwise, we'll have to bundle this breaking change with many others, unfortunately.
Thanks again for the PR. Great work, and looking forward to seeing your next ones :-)
Cheers!

Maybe what we need is a command thad updates our published assets, filling the keys that are missing in the already published files ?
We could hook it it the postUpdate in composer too so it is automatic.
Would it be nice ?

Hi @gotrecillo ,
That would definitely be nice, but as far as I know, postUpdate in composer only works on the first layer (the main project). You can't do that for dependencies (like Backpack\CRUD, etc). I mean you can, but they will not be run...

Hi @gotrecillo - we found a solution to the translation problem, so this is no longer a breaking change. FINALLY :-)
I've merged your PR into dev, thanks a lot for it. Cheers!


Simplify Splicing

Since splicing will potentially become yet more complex with https://github.com/googleprojectzero/fuzzilli/pull/256, I think it'd make sense to try and simplify the logic a bit, since it's an important part of the fuzzing logic, and so should be easy to understand/verify.
My suggestion would be to remove the code that supports rewiring inputs (so that the slice is directly connected to the host program through data-flow), and instead replace that with a simple variant of the InputMutator, the TypeAwareInputMutator, which takes type information into account. The idea here being that rewired splicing is roughly equivalent to a "normal" splice followed by an input mutation, the latter being much simpler to implement though.
Any thoughts?

The idea here being that rewired splicing is roughly equivalent to a "normal" splice followed by an input mutation, the latter being much simpler to implement though.

Presumably the inputs for mutation would be retrieved for the host program rather than the target? I think this would be the simpler approach but I fear we would be losing out on the aspects of the existing inputs. For example if we have the following target program:
v1 = CreateObejct( with properties)
... instructions that modify v1 (e.g. adding/deleting properties, store/load props, etc)
CallFunction(x, v1)

So if we splice CallFunction and replace v1 with an input from the host program that has the same type (i.e. object with same properties) then we'd lose out on the modification to v1. Not sure if it's worth the effort to preserve these aspects.
Also, what would we do if there are no available inputs of the desired type in the host program?

So I think what you are describing are issues with the current approach, where variables are rewired directly during splicing.
What I'm proposing is to simplify the splice algorithm to never rewire inputs. So in your example, we would copy the entire CreateObject block (well, at least we should probably copy the entire block including its content if we need v1, I think?) and the CallFunction instruction. This would be one mutation (a CodeGenMutation) and we'd then execute the resulting program etc. Then, we'd also add a variant of the InputMutator that is aware of types. This could, during a subsequent mutation, modify the CallFunction instruction to use a different 2nd input, but one that has a "compatible" type. The result would then be roughly equivalent to a splice with rewired inputs. Except that it is simpler and happens over multiple mutations.

Yep fair, that makes sense now. I'm in favour of the approach you've described above.

https://github.com/googleprojectzero/fuzzilli/pull/263


[ERROR] The batch file cannot be found.

Error shown after each scenario when run by maven.

Probably caused by extentreport misconfiguration - error is gone fter #26 and #8


bash: spotdl: command not found

What is the purpose of your issue?

[x] Bug
[ ] Feature Request
[ ] Question
[ ] Other

Description
i get bash: spotdl: command not found
Log
$ pip3 install -U spotdl

Collecting spotdl
Using cached https://files.pythonhosted.org/packages/b8/bc/ef3b1aed5edea24c8176cf460bfbc45fa388cf99bcad58f94540f1dd9059/spotdl-1.2.1-py3-none-any.whl
Collecting lyricwikia>=0.1.8 (from spotdl)
Collecting titlecase>=0.10.0 (from spotdl)
Using cached https://files.pythonhosted.org/packages/1c/6b/1af08c2d82cf3bad9caf764bca8c398c78f811d0638e5cb268952f45c917/titlecase-0.12.0-py3-none-any.whl
Collecting youtube-dl>=2017.9.26 (from spotdl)
Using cached https://files.pythonhosted.org/packages/45/60/3fc3026b82a3a0e37e6b598260c68abb446e7e85cba04318d2da17a95fac/youtube_dl-2019.5.20-py2.py3-none-any.whl
Collecting pathlib>=1.0.1 (from spotdl)
Collecting appdirs>=1.4.3 (from spotdl)
Using cached https://files.pythonhosted.org/packages/56/eb/810e700ed1349edde4cbdc1b2a21e28cdf115f9faf263f6bbf8447c1abf3/appdirs-1.4.3-py2.py3-none-any.whl
Collecting pafy>=<IP_ADDRESS> (from spotdl)
Using cached https://files.pythonhosted.org/packages/b0/e8/3516f761558525b00d3eaf73744eed5c267db20650b7b660674547e3e506/pafy-0.5.4-py2.py3-none-any.whl
Collecting logzero>=1.3.1 (from spotdl)
Using cached https://files.pythonhosted.org/packages/97/24/27295d318ea8976b12cf9cc51d82e7c7129220f6a3cc9e3443df3be8afdb/logzero-1.5.0-py2.py3-none-any.whl
Collecting mutagen>=1.41.1 (from spotdl)
Collecting beautifulsoup4>=4.6.3 (from spotdl)
Using cached https://files.pythonhosted.org/packages/1d/5d/3260694a59df0ec52f8b4883f5d23b130bc237602a1411fa670eae12351e/beautifulsoup4-4.7.1-py3-none-any.whl
Collecting PyYAML>=3.13 (from spotdl)
Collecting unicode-slugify>=0.1.3 (from spotdl)
Collecting spotipy>=2.4.4 (from spotdl)
Collecting six (from lyricwikia>=0.1.8->spotdl)
Using cached https://files.pythonhosted.org/packages/73/fb/00a976f728d0d1fecfe898238ce23f502a721c0ac0ecfedb80e0d88c64e9/six-1.12.0-py2.py3-none-any.whl
Collecting requests (from lyricwikia>=0.1.8->spotdl)
Using cached https://files.pythonhosted.org/packages/51/bd/23c926cd341ea6b7dd0b2a00aba99ae0f828be89d72b2190f27c11d4b7fb/requests-2.22.0-py2.py3-none-any.whl
Collecting soupsieve>=1.2 (from beautifulsoup4>=4.6.3->spotdl)
Using cached https://files.pythonhosted.org/packages/b9/a5/7ea40d0f8676bde6e464a6435a48bc5db09b1a8f4f06d41dd997b8f3c616/soupsieve-1.9.1-py2.py3-none-any.whl
Collecting unidecode (from unicode-slugify>=0.1.3->spotdl)
Using cached https://files.pythonhosted.org/packages/31/39/53096f9217b057cb049fe872b7fc7ce799a1a89b76cf917d9639e7a558b5/Unidecode-1.0.23-py2.py3-none-any.whl
Collecting certifi>=2017.4.17 (from requests->lyricwikia>=0.1.8->spotdl)
Using cached https://files.pythonhosted.org/packages/60/75/f692a584e85b7eaba0e03827b3d51f45f571c2e793dd731e598828d380aa/certifi-2019.3.9-py2.py3-none-any.whl
Collecting urllib3!=1.25.0,!=1.25.1,<1.26,>=1.21.1 (from requests->lyricwikia>=0.1.8->spotdl)
Using cached https://files.pythonhosted.org/packages/e6/60/247f23a7121ae632d62811ba7f273d0e58972d75e58a94d329d51550a47d/urllib3-1.25.3-py2.py3-none-any.whl
Collecting idna<2.9,>=2.5 (from requests->lyricwikia>=0.1.8->spotdl)
Using cached https://files.pythonhosted.org/packages/14/2c/cd551d81dbe15200be1cf41cd03869a46fe7226e7450af7a6545bfc474c9/idna-2.8-py2.py3-none-any.whl
Collecting chardet<3.1.0,>=3.0.2 (from requests->lyricwikia>=0.1.8->spotdl)
Using cached https://files.pythonhosted.org/packages/bc/a9/01ffebfb562e4274b6487b4bb1ddec7ca55ec7510b22e4c51f14098443b8/chardet-3.0.4-py2.py3-none-any.whl
Installing collected packages: six, soupsieve, beautifulsoup4, certifi, urllib3, idna, chardet, requests, lyricwikia, titlecase, youtube-dl, pathlib, appdirs, pafy, logzero, mutagen, PyYAML, unidecode, unicode-slugify, spotipy, spotdl
Successfully installed PyYAML-5.1 appdirs-1.4.3 beautifulsoup4-4.7.1 certifi-2019.3.9 chardet-3.0.4 idna-2.8 logzero-1.5.0 lyricwikia-0.1.9 mutagen-1.42.0 pafy-0.5.4 pathlib-1.0.1 requests-2.22.0 six-1.12.0 soupsieve-1.9.1 spotdl-1.2.1 spotipy-2.4.4 titlecase-0.12.0 unicode-slugify-0.1.3 unidecode-1.0.23 urllib3-1.25.3 youtube-dl-2019.5.20

$ spotdl

bash: spotdl: command not found

What is the output of:
$ pip3 show spotdl

?

@ritiek well, i tried sude pip3 install spotdl & its worked so yeah

@EsmailELBoBDev2 It would've worked without sudo as well but I think ~/.local/bin isn't under your PATH variable so that's probably why it couldn't find it.

You can add ~/.local/bin to your PATH variable with this command:
export PATH=$PATH:~/.local/bin

You can add ~/.local/bin to your PATH variable with this command:
export PATH=$PATH:~/.local/bin

Thank you so much!
This solved the issue!


closeOnSelect doesn't work on version 0.9.0

I installed version 0.9.0 the closeOnSelect doesn't work. I set it to true but when I select a date, the popup doesn't close and no error message.
$scope.datepickerObject = {
    closeOnSelect: true,
    templateType: 'popup',
    to: new Date(),
    callback: function (date) {
          console.log('callback');
    }
};

closeOnSelect was added by me onyl to days ago. So it's not in 0.9.0 release. Update you version to use master branch version and it will work.

I've installed the master branch but the issue still happen. Please see https://github.com/rajeshwarpatlolla/ionic-datepicker/issues/111

This feature will make the user feel strange. Let's say when a user clicks on a date but don't want to select that date, even in that case the popup will close, again the user have to open it again. So i don't think, this feature will be used by so many. I suggest this feature to be implemented those who wish, but not as part of this component. Suggestions are welcome.

After I sent app to my users, they prefer the close on select function because it does like date picker in many web sites are using and it doesn't need many clicks which is suitable for mobile device. For example, the app use for planing many tasks in a day.

So, it's a kind of on demand feature. I was discussing with few people about this feature, most of them are not preferring this way, as it makes most of the users confuse. The user may wish to select day first and then month and then year, in this case the user should be abel to do all in one go. If we have this feature, then the user has to open the datepicker for 3 times.

But the version 0.9.0 doesn't support the way to select date first and then month and then year. When I select date first and then month, the picker doesn't highlight date on the selected month.
For example, I select the date 3nd on November then change month to April and click on Set button. The output date still be November 3 not April 3. This is why my user confuse about.
So, I think the close on select should be an option for developer to customize it behavior.

I can confirm what @chompunut said. In ver 0.9.0. If i check date and then change month, then the date wont be that months. Chaning month dosen't change selected date.

Hi can Any one plz help me with this datepicker I have installed everything but I could not get modal .can any one plz suggest

First of all, great plugin, really handy :) however I am in agreement that closeOnSelect should be an option for the developer (doesn't work for me currently), most of my users prefer that option as well, considering how in my case a user can only select a date within a period of 7 days after tomorrow. In which case there should also be options to hide all the buttons.  Also can you please tell me if there is a way to generally change the theme color of the plugin from one central setting. Thank you in advance. :)

I can confirm....
closeOnSelect
doesn't work in the current released bower version as it is not implemented and
does not work with #master version because of errors - it is implemented.
Hope that this will be fixed  - I like this datepicker - and yeah I would need this option too.

Hi,
This issue also occurs in the master branch, could you give an estimate on when the fix will be made?
Thanks in advance!

I am trying to make few changes and once they are done, i will release a new version with this feature.

+1

Please check the new release v1.0.0. This issue has been addressed.


Mutlilabel metrics

Add multilabel metrics to factory_model_performance
The function should be able to take a multilabel model and x_test, y_test as params. it should produce for the model a dict containing:

per-class recall, precision, f1
macro and micro recall, precision, f1 scores
Hamming loss
Jaccard score
Exact accuracy (although this might be depressingly low, maybe ignore)

Currently just prints to terminal but can modify to write txt file to folder once rest of pipeline is established


QA needs to be able to see a unique build number

Add version number and build number to the lower left of the screen on the blue bar.

@birchamp add screenshot from 1.0

@cckozie @elsylambert Latest build is here (https://create-app.netlify.app/) with latest commit hash 66d96f2

On the Footer added two text spans. One with larger text id='build_version', and the smaller text id='build_hash'

Looking good in v0.2.0 build 66d96f2, even with id attributes!


Environment variables in cmd different than in integrated terminal

VSCode Version: 1.5.3
OS Version: Windows 10 64bit

Steps to Reproduce:

type echo %ProgramFiles% in cmd -> C:\Program Files
type echo %ProgramFiles% in integrated terminal -> C:\Program Files (x86)

Why do the two commands not yield the same result?

This is because your %COMSPEC% environment variable is set to the 32-bit version of cmd.exe. If you want it to run the same shell you will want to use the sysnative version of cmd.exe as per the config docs https://code.visualstudio.com/docs/editor/integrated-terminal#_configuration

Thanks for the clarification