Added WIFI widget & fixed weather

Pull Request - #105 & #103

Added a WIFI widget, which shows signal strength, signal strength + wifi network name (on label_alt), and opens network settings (on right click)
Implemented @zombie-bear's weather widget fix from #103, as the current API doesn't seem to be working.

Description

I updated the config.yaml "last updated" information to reflect my updates.
Wifi Widget:
I added a new core "wifi' widget to widgets and validation. The widget uses os to get the user's signal strength, then displays said strength in the status bar using nerdfont icons (see image for icons)

On left click, the widget also displays the network name. It should be trivial to configure this to display other info, such as current upload/download rate. On right click, the widget opens Windows' network settings.
Weather Widget:
I implemented @zombie-bear's fix, using wttr.in instead of openweathermap.org. I, like zombie-bear, could not get openweathermap to function (possible due to API changes?), but after changing the API request to wttr.in had it working in seconds. I updated the default weather JSON paths to match the old content (temp in C, current condition), as well as the commented explanation in config.yaml.
Related Issues

#105
#103

Testing

OS: Windows 10 Pro
Komorebi: Yes
Python: 3.11.4
Terminal: Windows PowerShell in Windows Terminal

I ran the updated status bar on two computers in the default configuration (note that the missing icons are due to the default configuation using a nerdfont I don't have installed. This is already the case by default).

I linted the code using pylama (note that all errors seen in the image exist on the current yasb main branch, none of the added code is flagged).

I ran my custom theme with the new widget

I changed the config.yaml default options for wifi (eg: icons, update interval)

I tried multiple cities using wttr.in and verified the weather against local conditions

All of these performed as expected.

I haven't really been maintaining the python version of yasb as of recent, but this contribution looks great. The changes look pretty good so I'm happy to merge this in.
Thank you for your contribution


codimd: build with nodejs-12_x instead of nodejs-10_x

Motivation for this change
#97377
Things done
The codimd test is now also exposed via passthru.tests

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

fixes #97377

@Ma27 yes the test fails for sqlite, you should be able to verify it on the current master or release branch :)

Ported to 20.09 as<PHONE_NUMBER>b541129185ccd671105b078e14a79e.


Angular 4.3.x HttpClient compatibility

Are there any plans to upgrade this service to support Angular 4.3.x's HttpClient?
Commit describing the change
Official docs

Hey @xozzo, sure. Working on it, but as far as I can tell, it will be quite some work.

@neroniaky Is there an active branch? I'm interested in contributing. :)

@xozzo just pushed the branch httpclient. Enjoy :).

@neroniaky I think I've got a rough idea of the changes that need to be made - do we want to keep the service backwards compatible for those who are using <4.3?

As far as I can tell now, being backward compatible will add a lot of complexity. What do you think?

Yeah, I agree. The new @angular/common/http module introduces HttpInterceptor which eliminates the need for a2token to implement http method wrappers. If you really wanted to, maybe we can determine which Http module to use based on version obtained at runtime?

This looks good. The typings still point to lib instead of src in:
https://github.com/neroniaky/angular2-token/blob/httpclient/angular2-token.d.ts

Hi,
Is there a quick workaround to implement an HTTP interceptor around the angular 2 token service http wrapper ? I was thinking about extending the service class and redefining the handleResponse method, but I was wondering if there would be a simpler way while waiting for this branch to be available.

It would be awesome if this branch could be merged, interceptors are awesome! :)

Any advance on this? Will be really good to have this merged.

Just for the sake of people commenting asking for the branch to be merged, there is no actual new code in the branch! All it has is a tweak to the Readme saying the branch is experimenting with HttpClient :(
With HttpModule now deprecated, this service really needs to move to HttpClient at some point. It gives a huge advantage in that we could use interceptors to stop wrapping the http methods and so 3rd party libraries that hide http access (like Restangular) will work. This is how the AngularJS devise token lib works (https://github.com/lynndylanhurley/ng-token-auth).

Hi,
I'm thinking about working on this, at least trying :)
However, after cloning, I wasn't able the run test : https://github.com/neroniaky/angular2-token/issues/445

@neroniaky and I are currently working on a version with HttpClient support and HttpInterceptors. Give us a few more days before we can release an alpha.

I have a working version on my fork if you want to have a look.
On Mon, 18 Jun 2018, 20:49 Arjen Brandenburgh<EMAIL_ADDRESS>wrote:

@neroniaky https://github.com/neroniaky and I are currently working on
a version with HttpClient support and HttpInterceptors. Give us a few more
days before we can release an alpha.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/neroniaky/angular2-token/issues/309#issuecomment-398173785,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADw0sw3X64V_ulcEVuQB14-Y-yHYkxYFks5t-ARjgaJpZM4Otjz7
.

Branch feature/httpClient

Initial version is now in branch 6.x


Adapt to latest breaking changes

Fixes #12

Also maybe @ejmr can give this a shot, the distribution on which I fixed this had python symlinked to python2

Thanks occivink. A future challenge would be to get ride of python and build the same features in plain kak+shell.

Thanks occivink!


Remove Polymer.GestureEventListeners

It is not used in this element.

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


Pinecone Search Component not returning records

Describe the bug
The pinecone search component connects and makes requests to Pinecone successfully, but does not return records through the record connector.
Browser and Version

Browser chrome
Version 1.0 preview

To Reproduce

Add pinecone search component and add API key
connect embedding connector to openai embedding with 003-small model.
connect record connector to a valid component such as Record to Text or Record Output
Connect that component to a valid output
No text or record will display, all outputs will be blank in playground

Screenshots

Additional context
I've used a LLM model in the middle, but was confused why it didn't seem to respond using context. So I simplified the workflow to identify that Pinecone was not returning any context.

Hey @adampatarino
How are you?
The problem is the namespace field. It is set to advanced so you probably have not set it.
I'll update the component to make that visible.
Thanks for the heads up!


@mzikherman => Save intent for most contexts

This adds intent tracking to the auth modal creation points listed below. There are a few more tricky cases, but most of the work here was finding and checking each case was called correctly. The possible signupIntents are:
follow artist
follow profile
follow gene
save artwork
bid
register to bid
sign up

These have been added to:

/artists --> Follow artist
Artist list view - Follow artist
Artist view - Follow artist
Artist view - Related artists - Follow artist
Artwork view - Follow artist
Artwork view - Related artists - Follow artist
Artwork view - Related works - Save work
Artwork view - Save work
Auction Sale - Register to bid
Category view - Follow artist
Category view - follow gene
Fair booth view - Follow artist
Fair booth view - follow profile
Fair booth view - Save work
Fair list view - follow profile
Fair view - follow profile
Fair view - Save work
Gallery list view - follow profile
Gallery view - Artists - Follow artist
Gallery view - Artists - Save work
Gallery view - follow profile
Gallery view - Works - Save work
Home banner - Sign up
Home view - Follow artist
Home view - follow gene
Log in - Sign up
Museum list view - follow profile
Recently viewed works - Save
Show list view - follow profile
Show list view - Save work
Show view - follow profile
Show view - Save work

Looks great! Love the thoroughness.


Add new note about GA integration abilities

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@salarson - thank you for committing my suggestions but I notice you have also closed the PR. Was that intentional?

Complete accident! Sorry about that!
From: JohanFreelancer9 @.>
Sent: Monday, May 3, 2021 8:56 AM
To: MicrosoftDocs/OfficeDocs-SharePoint @.>
Cc: Sam Larson @.>; Mention @.>
Subject: Re: [MicrosoftDocs/OfficeDocs-SharePoint] Add new note about GA integration abilities (#2710)
@salarsonhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsalarson&data=04|01|salarson%40microsoft.com|105f52d69b4044d7d18a08d90e3b33f4|72f988bf86f141af91ab2d7cd011db47|1|0|637556469703497600|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=18e0Y9atJMCJSp6%2F7kh4%2ByC%2FhsVSD4feW2EtAVZP7Mc%3D&reserved=0 - thank you for committing my suggestions but I notice you have also closed the PR. Was that intentional?

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%2FMicrosoftDocs%2FOfficeDocs-SharePoint%2Fpull%2F2710%23issuecomment-831275831&data=04|01|salarson%40microsoft.com|105f52d69b4044d7d18a08d90e3b33f4|72f988bf86f141af91ab2d7cd011db47|1|0|637556469703502578|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=%2BWxliXqk%2FpP0NnF%2FHNQD5XJAqBALc1Eyky3ta8sTVxc%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAFBR4UPOR2VMZYL2MWT5J5TTL2TPRANCNFSM432A4LDA&data=04|01|salarson%40microsoft.com|105f52d69b4044d7d18a08d90e3b33f4|72f988bf86f141af91ab2d7cd011db47|1|0|637556469703507556|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=di09JQTCT5rr4t2de4Dfy5LwBI6Y%2BTcUDUoOtlpbMTo%3D&reserved=0.

No problem. Please reopen and I will approve the copy.

@salarson I made some terminology tweaks. Is this set to be merged? I know there was some discussion in mail about what to include in the note...


APM spec

@vkarpov15 I added the duration measurement and a simple test for it, if you want to take a look

Assuming travis passes (which hopefully it should, since the tests passed locally), I think this branch is ready to merge, assuming there are no more code issues.


Row matching works only partially

Rows matching functionality does not work in all cases. The following code should select all the rows, that have the same value as the current cell holds, but it fails on columns site, race, disease. Only part of the categories in study column are being selected.
let table = grok.data.demo.demog();
let strCol = table.columns.addNewString('test');
let categories = ['f', 's', 't'];
strCol.init(i => categories[i % categories.length]);
let view = grok.shell.addTableView(table);
let grid = view.grid;

grid.onCurrentCellChanged.subscribe(gc => {
  const val = gc.cell.value;
  const colName = gc.tableColumn.name;
  table.rows.match(`${colName} = ${val}`).select();
});

The issue is fixed.


show "SeLockMemoryPrivilege" on process token tab

I noticed that the "token" tab in the "process" property page in systeminformer is missing the display of the "SeLockMemoryPrivilege" privilege property.

One use case for this privilege attribute is the following link:
Address Windowing Extensions
My attempt to enable the privilege property using Systeminformer also failed, but it worked using the windows api, as shown here:
      	bool SetSeLockMemoryPrivilege()
        {
            bool v_ret_val{false};
#if 0

#else
            HANDLE hToken;
            LUID luid;
            TOKEN_PRIVILEGES tkp = {};
#endif

            for(;;)
            {
#if 1
                const auto status = PhAdjustPrivilege(const_cast<PWSTR>(L"SeLockMemoryPrivilege"),0,TRUE);

                if (!NT_SUCCESS(status))
                {
                    break;
                }
#else
                if (!OpenProcessToken(GetCurrentProcess(), TOKEN_ADJUST_PRIVILEGES | TOKEN_QUERY, &hToken))
                {
		            break;
	            }
                	LookupPrivilegeValue(nullptr, SE_LOCK_MEMORY_NAME, &tkp.Privileges[0].Luid);
	            	tkp.PrivilegeCount = 1;  // one privilege to set
	            	tkp.Privileges[0].Attributes = SE_PRIVILEGE_ENABLED;

                if (FALSE == AdjustTokenPrivileges(hToken, FALSE, &tkp, 0,nullptr, nullptr))
                {
                     break;
                }
#endif
                v_ret_val = true;
                break;
            }
#if 0
            CloseHandle(hToken);
#endif
            return v_ret_val;
        }

I noticed that the "token" tab in the "process" property page in systeminformer is missing the display of the "SeLockMemoryPrivilege" privilege property

System Informer will only show privileges currently present in the token you're viewing. The SeLockMemoryPrivilege privilege is missing because the token you're viewing doesn't have the privilege.
If the privilege has been removed from the token then it won't be visible. Windows doesn't record deleted privileges and showing a non-existent privilege would be misleading since the token would appear to have privileges that don't exist and cannot be deleted.

My attempt to enable the privilege property using Systeminformer also failed

You won't be able to enable it because:

Windows prohibits applications from enabling a privilege unless they're present in the token.
Windows prohibits applications from adding a privileges after the program has started executing.
Windows doesn't allow creation of tokens with privileges unless the current user has rights to the privilege (generally configured via group policy).

You need to edit group policy, add your username to the policy and restart:
https://learn.microsoft.com/en-us/windows/security/threat-protection/security-policy-settings/lock-pages-in-memory#location

it worked using the windows api, as shown here:

You code returns an unassigned error code? PrivilegeCheck also failed:
BOOLEAN privilegeEnabled = FALSE;
PRIVILEGE_SET requiredPrivileges;

requiredPrivileges.PrivilegeCount = 1;
requiredPrivileges.Control = PRIVILEGE_SET_ALL_NECESSARY;
requiredPrivileges.Privilege[0].Luid.LowPart = SE_LOCK_MEMORY_PRIVILEGE;
requiredPrivileges.Privilege[0].Attributes = SE_PRIVILEGE_ENABLED;
PrivilegeCheck(GetCurrentProcessToken(), &requiredPrivileges, &privilegeEnabled);

if (privilegeEnabled)
{
    printf("PRIVILEGE_ENABLED\n");
}

I noticed that the "token" tab in the "process" property page in systeminformer is missing the display of the "SeLockMemoryPrivilege" privilege property

System Informer will only show privileges currently present in the token you're viewing. The SeLockMemoryPrivilege privilege is missing because the token you're viewing doesn't have the privilege.
If the privilege has been removed from the token then it won't be visible. Windows doesn't record deleted privileges and showing a non-existent privilege would be misleading since the token would appear to have privileges that don't exist and cannot be deleted.

My attempt to enable the privilege property using Systeminformer also failed

You won't be able to enable it because:

Windows prohibits applications from enabling a privilege unless they're present in the token.
Windows prohibits applications from adding a privileges after the program has started executing.
Windows doesn't allow creation of tokens with privileges unless the current user has rights to the privilege (generally configured via group policy).

You need to edit group policy, add your username to the policy and restart: https://learn.microsoft.com/en-us/windows/security/threat-protection/security-policy-settings/lock-pages-in-memory#location

it worked using the windows api, as shown here:

You code returns an unassigned error code? PrivilegeCheck also failed:
BOOLEAN privilegeEnabled = FALSE;
PRIVILEGE_SET requiredPrivileges;

requiredPrivileges.PrivilegeCount = 1;
requiredPrivileges.Control = PRIVILEGE_SET_ALL_NECESSARY;
requiredPrivileges.Privilege[0].Luid.LowPart = SE_LOCK_MEMORY_PRIVILEGE;
requiredPrivileges.Privilege[0].Attributes = SE_PRIVILEGE_ENABLED;
PrivilegeCheck(GetCurrentProcessToken(), &requiredPrivileges, &privilegeEnabled);

if (privilegeEnabled)
{
    printf("PRIVILEGE_ENABLED\n");
}

Thank you.


modifying installation requirements of tox and setup #204

Codecov Report

Merging #207 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #207   +/-   ##
=======================================
  Coverage   92.66%   92.66%
=======================================
  Files          30       30
  Lines        2249     2249
=======================================
  Hits         2084     2084
  Misses        165      165

Continue to review full report at Codecov.

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


No typing activity if bot is running as a skill

Fixes #2783
Description
Ports functionality from dotnet SDK to not send typing indicators if a bot is running as a skill.
Specific Changes

Adds a TestSkillAdapter helper class that automatically includes
claims to mimic a Skills bot
Adds logic to ShowTypingMiddleware to skip indicators if bot is
running as a skill
Adds a test for this behavior

Testing
Added a test case.

Pull Request Test Coverage Report for Build 169132

13 of 13 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.08%) to 83.35%

Totals

Change from base Build 168962:
0.08%

Covered Lines:
15528

Relevant Lines:
17720

💛  - Coveralls

@gabog I imagine we may extend our test suite with more "bot running as a skill" scenarios which was the main reason for exporting the class. I'm happy to internalize that class within the typing middleware test if you think that's the wrong approach.

not a big deal @joshgummersall. we can merge it as is, if we find ourselves using that helper class over and over, then let's think how we can factor this into the product.

not a big deal @joshgummersall. we can merge it as is, if we find ourselves using that helper class over and over, then let's think how we can factor this into the product.

I'll internalize the class before merging just to reduce our surface area until we decide on it's reusability. I also have to address an issue with the botframework-connector as well so we can't merge yet anyways.


kSecAttrLabel & kSecAttrDescription undefined

Missing API constants kSecAttrLabel & kSecAttrDescription both undefined

Fix is available in 0.1 Preview (December 3, 2015)


Adding a footer in the website. [GSSoC'23]

Please describe the problem
There is no footer to the website where we can have Terms and conditions, Contact us , etc.
Describe the solution you'd like
I can create a footer for the website as well as create terms and conditions , contact us, etc etc pages .
Screenshot

Please assign me this issue under GSSoC'23

please assign this task to me, I'm eager to make a valuable contribution to this issue as part of the GSSOC initiative.

Please assign me this issue under GSSoC'23

@1Shubham7 assigned to you

@AshutoshDash1999  Thank  you for assigning it to me I will surely not disappoint you.

@AshutoshDash1999 Please review this PR.

Thanks a lot @AshutoshDash1999 Bhaiya


Reflect-Metadata doesn't work with WebPack 3 Scope Hoisting

I tried reporting this to the WebPack team and they said it is a problem with Reflect-Metadata.
I'm using Reflect-Metadata in a TypeScript project that's being built and bundled with WebPack 3. When I add the scope hoisting plugin to my WebPack config, the project no longer runs properly.
I get the following error:
Uncaught TypeError: Reflect.hasOwnMetadata is not a function
at injectable.js:7
at Object.__decorate [as a] (tslib.es6.js:48)
at Object../client/app/app.bootstrap.ts (data.service.ts:4)
at webpack_require (bootstrap 1856459e76a82a034076:54)
at webpackJsonpCallback (bootstrap 1856459e76a82a034076:25)
at app.js:1
(anonymous) @ injectable.js:7
__decorate @ tslib.es6.js:48
./client/app/app.bootstrap.ts @ data.service.ts:4
webpack_require @ bootstrap 1856459e76a82a034076:54
webpackJsonpCallback @ bootstrap 1856459e76a82a034076:25
(anonymous) @ app.js:1
Is there a way to make it work, or can this problem be fixed?

Seems totally busted in webpack 4.
In my limited debugging it looks like reflect-metadata is not getting included in the resulting bundle at all.


RFP - Busybox

WIN32 native port of BusyBox. BusyBox is a single binary that contains many common Unix tools. It's often found in embedded Linux systems like routers, in Android smartphones, in Linux containers and anywhere else it would be handy to have a compact set of Unix command line tools.
https://frippery.org/busybox/
https://github.com/rmyorston/busybox-w32

Status: Review
https://chocolatey.org/packages/busybox

Status: Published
https://chocolatey.org/packages/busybox
Can be closed.


decision task

Hi @v1r3n ,
I was trying to make a decision making task in workflow. workflow takes input as number: . Based on this number whether its odd or even, decision should be taken. As input is an user input it could be anything . where to have a logic to find out whether the input is even or odd?
in kitchen example, decisoin task input is like  "oddEven": "${task_2.output.oddEven}". i could not make out how task_2 is generating "0" or "1".

@amoolya19 this logic will be implemented in a worker that takes the input from workflow and outputs the odd/even as true/false, 1/0 or any other value.  Conductor will not any math out of the box.

if i implement worker how to use it ? i am using only json based workflow definitions?

you will implement a new task type (look at the HTTP task implementation) and use it in your json definition with task type as the one you created.

@v1r3n,
yes, but what i asked is if i implement a new task type say "TEST" how can i import this jar /class into conductor? or do i have to do some configuration for this to make conductor understand this.
com.netflix.conductor.contribs.http.HttpTask is the one you are referring i assume. i dint see worker class impl or WorkflowTaskCoordinator class usage. Is my understanding wrong?

can we use ecpression in the case ? like this ${workflow.input.number}%2=0

@amoolya19 we are planning to add expression support for decision task sometime in Q2 of 2017.


stateio: run exports / imports as bg task

Once GUI and DB models are in place, will want to run these export/imports as background (celery) tasks.

While importing to Mongo and ElasticSearch via Spark, update status of Jobs in GUI to importing.  This might be a special status that will rely on bg task to update, not the normal Job.update_status(), as that cannot be aware of background spark processes.

Done.


Cloud Code: Function Code

Hi,
I have a 3 parser-server running (dev/test/prod), and I try use the same file cloud code (main.js).
Can you confirm it's impossible use single Cloud Code file, for many servers ?
I obtain this error (via curl or PFCloud):
Error Domain=Parse Code=141 "Invalid function." UserInfo={code=141, temporary=0, error=Invalid function., NSLocalizedDescription=Invalid function.}
Works only, If I have duplicate cloud code file with other name.
Thanks

Works for me, not sure if I understood the question correctly however.
$ cat >> cloud.js
Parse.Cloud.define('test', function(req, res) { res.success('yay'); });
^D

$ parse-server --appId test1 --masterKey test1 --port 1337 --cloud cloud.js &>/dev/null &
$ parse-server --appId test1 --masterKey test1 --port 1338 --cloud cloud.js &>/dev/null &

$ curl -X POST -H "X-Parse-Application-Id: test1" -H "Content-Type: application/json" -d "{}" http://localhost:1337/parse/function/test
{"result":"yay"}

$ curl -X POST -H "X-Parse-Application-Id: test1" -H "Content-Type: application/json" -d "{}" http://localhost:1338/parse/function/test
{"result":"yay"}

Ok, because you have 2 servers on 2 ports (1337/1338).
Don't work if you have many servers on one port (1337)

Well, ofcourse you can not run two servers on the same port - there have been work to mount multiple ParseServers in one node app however. (Not sure if that will ever be a case)
I simply run a nginx that speaks to multiple parse-server instances (that run on different ports)

No, it's possible.
You can have many servers on one port, but on different path
example:
`
var apiProd = new ParseServer({
serverURL: 'https://xxxx:1337/FGD',
databaseURI: 'mongodb://localhost:27017/xxxx',
cloud: '/....../cloud/main.js',
appId: '',
masterKey: '',
}
var apiR7 = new ParseServer({
serverURL: 'https://yyyy:1337/yyyy',
databaseURI: 'mongodb://localhost:27017/yyyyy',
cloud: '/....../cloud/main.js',
appId: '',
masterKey: ''
});
app.use('/xxxx', apiProd);
app.use('/yyyyy', apiR7);`
The only thing that does not work, that can not have the same Cloud Code.

This is exactly what is not supported, it was not clear in your initial question you mounted them like that.
See https://github.com/ParsePlatform/parse-server/issues/15
It makes no sense running dev/test/prod in the very same server in any case.

You're quite right, it does not make sense, but it was for the purpose of study.
Ok, thanks

You may use Server Fault for questions about managing Parse Server.
For code-level and/or implementation-related questions or technical support, please refer to Stack Overflow.


Failures setting frequency scaling on Android devices.

We are seeing failures setting the frequency scaling, and this is consistent across the 3 Nexus 9's we are running. A quick spot check and I found this happening on other devices also, such as the Nexus 7:
https://chromium-swarm.appspot.com/newui/bot?id=skia-rpi-057&selected=1&sort_stats=total%3Adesc
https://chromium-swarm.appspot.com/newui/bot?id=skia-rpi-002&selected=1&sort_stats=total%3Adesc
Nexus 9 event log:
Message	Type	Timestamp	Task ID	Version
Periodic reboot: running longer than 44976s	bot_rebooting	10/5/2016, 12:23:46 PM (EDT)		e4166244
1/19.PullContent(/system/build.prop): Could not send data to (timeout 10000ms): USBErrorIO()	bot_error	10/5/2016, 8:28:21 AM (EDT)		e4166244
<Device 1/14 HT496JT00270>: Failed to set CPU frequency	bot_log	10/5/2016, 6:55:33 AM (EDT)	31ad4c82d2c49f11	e4166244
Periodic reboot: running longer than 39776s	bot_rebooting	10/4/2016, 10:40:38 PM (EDT)		e4166244
<Device 1/13 HT496JT00270>: Failed to set CPU frequency	bot_log	10/4/2016, 6:27:04 PM (EDT)	31aa9f28f250da11	e4166244
<Device 1/5 HT496JT00270>: Failed to set CPU frequency	bot_log	10/4/2016, 10:36:22 AM (EDT)	31a8f0f29b30ca11	e4166244
Periodic reboot: running longer than 49023s	bot_rebooting	10/4/2016, 10:31:59 AM (EDT)		e4166244
<Device 1/26 HT496JT00270>: Failed to set CPU frequency	bot_log	10/4/2016, 4:50:29 AM (EDT)	31a7b47af83a4f11	e4166244
<Device 1/18 HT496JT00270>: Failed to set CPU frequency	bot_log	10/3/2016, 8:44:50 PM (EDT)	31a5f7957581db11	e4166244
Bot self-quarantined	request_update	10/3/2016, 8:36:04 PM (EDT)		290a16d3
<Device 1/14 HT496JT00270>: Failed to set CPU frequency	bot_log	10/3/2016, 4:10:32 PM (EDT)	31a4fbf909d06411	290a16d3
Periodic reboot: running longer than 44851s	bot_rebooting	10/3/2016, 7:34:41 AM (EDT)		02aa9441
Periodic reboot: running longer than 40289s	bot_rebooting	10/2/2016, 7:05:30 PM (EDT)		02aa9441
Periodic reboot: running longer than 46965s	bot_rebooting	10/2/2016, 7:52:57 AM (EDT)		02aa9441
Periodic reboot: running longer than 35116s	bot_rebooting	10/1/2016, 6:48:39 PM (EDT)		02aa9441
Nexus 7 event log:
Message	Type	Timestamp	Task ID	Version
Bot self-quarantined	request_update	10/5/2016, 4:40:21 PM (EDT)		e4166244
Periodic reboot: running longer than 35336s	bot_rebooting	10/5/2016, 6:22:24 AM (EDT)		e4166244
Periodic reboot: running longer than 39934s	bot_rebooting	10/4/2016, 7:48:24 PM (EDT)		e4166244
Periodic reboot: running longer than 39144s	bot_rebooting	10/4/2016, 8:35:10 AM (EDT)		e4166244
Bot self-quarantined	request_update	10/3/2016, 9:24:25 PM (EDT)		290a16d3
<Device 1/10 AG86044202A04GC>: Failed to set CPU frequency	bot_log	10/3/2016, 4:32:29 PM (EDT)	31a510914794f811	290a16d3
Execution failed: HTTPSConnectionPool(host='isolateserver.storage.googleapis.com', port=443): Read timed out.	task_error	10/3/2016, 4:29:37 PM (EDT)	31a4f1c7aced8811	02aa9441
Periodic reboot: running longer than 40250s	bot_rebooting	10/3/2016, 2:03:28 PM (EDT)		02aa9441

@maruel

@erock2112 can you take a look? @bpastene is it something still often observed?

There are two problems here: failing to set the scaling mode, and the fact that the device is allowed to run tasks even though setting the CPU scale failed.  As for the first, here's a snippet from adb.log:
733 2016-10-06 04:04:39.794 D: 1/27.Write(WRTE, 24, 520, 'RECV5\x00\x00\x00/sys/devices/system/cpu/cpu0/cpufreq/scaling_governor')
733 2016-10-06 04:04:39.802 D: 1/27.Read(OKAY, 520, 24)
733 2016-10-06 04:04:39.811 D: 1/27.Read(WRTE, 520, 24, 'DATA\x0c\x00\x00\x00interactive\nDONE\x00\x00\x00\x00')
733 2016-10-06 04:04:39.820 D: 1/27.Write(OKAY, 24, 520)
733 2016-10-06 04:04:39.828 D: 1/27.Write(CLSE, 24, 520)
733 2016-10-06 04:04:39.835 D: 1/27.Read(CLSE, 0, 24)
733 2016-10-06 04:04:39.843 D: 1/27.Write(OPEN, 25, 0, 'sync:\x00')
733 2016-10-06 04:04:39.850 D: 1/27.Read(OKAY, 521, 25)
733 2016-10-06 04:04:39.859 D: 1/27.Write(WRTE, 25, 521, 'SEND;\x00\x00\x00/sys/devices/system/cpu/cpu0/cpufreq/scaling_governor,33272DATA\n\x00\x00\x00userspace\nDONEW\xcd\xf5W')
733 2016-10-06 04:04:39.868 D: 1/27.Read(OKAY, 521, 25)
733 2016-10-06 04:04:39.876 D: 1/27.Read(WRTE, 521, 25, 'FAIL\x11\x00\x00\x00Permission denied')
733 2016-10-06 04:04:39.886 D: 1/27.Write(OKAY, 25, 521)
733 2016-10-06 04:04:39.886 I: 1/27.SetCPUScalingGovernor(): Failed to push userspace in /sys/devices/system/cpu/cpu0/cpufreq/scaling_governor
733 2016-10-06 04:04:39.895 D: 1/27.Write(OPEN, 26, 0, 'shell:echo "userspace" > /sys/devices/system/cpu/cpu0/cpufreq/scaling_governor ;echo -e "\n$?"\x00')
733 2016-10-06 04:04:39.903 D: 1/27.Read(OKAY, 522, 26)
733 2016-10-06 04:04:39.911 D: 1/27.Read(WRTE, 522, 26, "/system/bin/sh: can't create /sys/devices/system/cpu/cpu0/cpufreq/scaling_governor: Permission denied\r\n")
Are we sure we have root access on this device?  @kjlubick

We're getting to the point where we have enough devices that it's hard to notice when this fails. IMO the device should be quarantined in this situation.

All but one of our devices had root access.  I am fixing the one that does not right now.

The Nexus 7 on skia-rpi-002 is still failing to set CPU frequency.  This fails for me:
$ adb shell "echo "userspace" > /sys/devices/system/cpu/cpu0/cpufreq/scaling_governor"
But the same command succeeds after "adb root".  From what I can tell in adb.log, we never even attempt to restart adbd as root, despite multiple attempts to restart adbd as non-root.  Note that this is an old device running K.  Is it possible to upgrade it?

Okay, I think I found the issue.  We explicitly reboot our Android devices at the end of our recipes.  The Swarming code does not expect this; it does the "adb root" whenever it reboots the device and otherwise expects that adbd is already running as root.  When we reboot the device during the swarming task, we don't run "adb root", so we're in non-root mode.  I think the right solution is to change Swarming to always reboot Skia Android devices after every task and stop issuing the reboot during the task.

Okay, removing the reboot from our recipe and having Swarming do it after every task seems to have fixed the CPU scaling on the N7 on skia-rpi-002.  I'm still not satisfied with the level of visibility of these errors.  IMO any failure in on_before_task should prevent the task from running on that bot, if not quarantine the bot outright.  Maybe we should start querying for bot events and alerting for some failures?

Okay, the N7s are still failing but with a different error:
733 2016-10-17 12:28:58.049 D: 1/6.Write(OPEN, 24, 0, 'sync:\x00')
733 2016-10-17 12:28:58.056 D: 1/6.Read(OKAY, 9, 24)
733 2016-10-17 12:28:58.065 D: 1/6.Write(WRTE, 24, 9, 'SEND;\x00\x00\x00/sys/devices/system/cpu/cpu0/cpufreq/scaling_setspeed,33272DATA\x07\x00\x00\x00640000\nDONE\n\xc4\x04X')
733 2016-10-17 12:28:58.073 D: 1/6.Read(OKAY, 9, 24)
733 2016-10-17 12:28:58.081 D: 1/6.Read(WRTE, 9, 24, 'OKAY\x00\x00\x00\x00')
733 2016-10-17 12:28:58.090 D: 1/6.Write(OKAY, 24, 9)
733 2016-10-17 12:28:58.097 D: 1/6.Write(CLSE, 24, 9)
733 2016-10-17 12:28:58.104 D: 1/6.Read(CLSE, 9, 24)
733 2016-10-17 12:28:58.112 D: 1/6.Write(OPEN, 25, 0, 'sync:\x00')
733 2016-10-17 12:28:58.119 D: 1/6.Read(OKAY, 10, 25)
733 2016-10-17 12:28:58.127 D: 1/6.Write(WRTE, 25, 10, 'RECV5\x00\x00\x00/sys/devices/system/cpu/cpu0/cpufreq/scaling_setspeed')
733 2016-10-17 12:28:58.136 D: 1/6.Read(OKAY, 10, 25)
733 2016-10-17 12:28:58.144 D: 1/6.Read(WRTE, 10, 25, 'DATA\x08\x00\x00\x001200000\nDONE\x00\x00\x00\x00')
733 2016-10-17 12:28:58.153 D: 1/6.Write(OKAY, 25, 10)
733 2016-10-17 12:28:58.161 D: 1/6.Write(CLSE, 25, 10)
733 2016-10-17 12:28:58.168 D: 1/6.Read(CLSE, 10, 25)
733 2016-10-17 12:28:58.169 I: 1/6.SetCPUSpeed(): Failed to push 640000 in /sys/devices/system/cpu/cpu0/cpufreq/scaling_setspeed
We successfully set the scaling governor, and writing the frequency file apparently succeeds, but when we read it back it's still set to the previous value.  This seems to be the case for all 3 of our N7s, and I have no idea how to fix it.


Typo: "Yarn package" looks like "Yarnpackage"

Steps to reproduce:

Go to https://rushjs.io/
Scroll down to the No phantom dependencies! section
Notice the word Yarnpackage

Actual Results:

Expected Results:

Fix:
One character change needed at https://github.com/microsoft/rushjs.io-website/blob/eb03edf46cf9d3239d5ce2a9e8bfa18e5308df34/permalinks/home.html#L108

@sshongru thanks for reporting this!


Create d3 example that shows an alternative orbat view

Make an alternative orbat visualisation based on http://bl.ocks.org/mbostock/1093025
This can be very helpful when building a web application and you want a list of your orbat.

This is now under examples/d3/list_view.html

Nice example! I wonder if it is difficult to add drag and drop functionality. Either to rearrange the hierarchy, or to drag a unit onto a map for changing the position.

To add drag and drop is quite easy. It's just a modified d3 tree view.
http://bl.ocks.org/robschmuecker/7880033
I have been thinking about beeing able to drag units out on the map, and that is a bit more tricky, but can be done i several ways.

I have a good plan for how to make it possible to drag out units to the map, should not be so complicated to write, but I have to see if I find the time...

Hello Måns,
double checked and adapted package.js for a meteor package.
Directory structure:
packages

milsymbol

bin
milsymbol.js
package.js (content see below)

As meteor eaxample I derived your Leaflet example and brought it to
Outside this example I had not much time in the last months to spend on MilSymbol visualization.
Hope to have the opportunity for a new POC application in the company environment soon.
Please let me know if you need further information for your packaging.
/Manfred
—
// package.js for milsymbol.js - package name 'milsymbol'
Package.describe({
summary: 'milsymbol.js; Military Symbology according to US MilStd 2525x / NATO APP6x',
version: '0.5.2'
});
Package.on_use(function (api, where) {
api.export("MS");
api.add_files('bin/milsymbol.js', 'client');
});
—

Am 12.10.2015 um 17:21 schrieb Måns Beckman<EMAIL_ADDRESS><EMAIL_ADDRESS>I have a good plan for how to make it possible to drag out units to the map, should not be so complicated to write, but I have to see if I find the time...
—
Reply to this email directly or view it on GitHub https://github.com/spatialillusions/milsymbol/issues/29#issuecomment-147433541.

—
Manfred Reinart
<EMAIL_ADDRESS><EMAIL_ADDRESS>


Implement bearer authentication with personal access tokens

The current implementation uses basic authentication only.
But our server enables only bearer authentication :(
See https://confluence.atlassian.com/enterprise/using-personal-access-tokens-1026032365.html
Describe the solution you'd like
Implement the bearer flag into the config file. The juice if the JIRA_AUTH_TOKEN contains a password or a personal access tokens will be answered during the init process.
The code for auth will be changed to:
	if c.bearer {
		req.Header.Add("Authorization", "Bearer "+c.token)
	} else {
		req.SetBasicAuth(c.login, c.token)
	}

Describe alternatives you've considered
The suggested auth code is simple. The solution how to create the bearer flag may differ from the description above.
Additional context

@ankitpokhrel - many thanks for finishing this issue.


Update ReadMe with casing correction of installation command

Not functionally required but makes it easier to read.

@bergmeister thanks for contributing


running golangci-lint on pdcs/images

Signed-off-by: Navid Yaghoobi<EMAIL_ADDRESS>

LGTM


Better way to test for non-existing function arguments

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Description
In JS, checking if an argument is passed to a function call, is usually done by verifying against undefined. null is a specific value in JS.

I would like @kvz to have a voice here, because current implementation of this - and many other functions - reflects PHP's behaviour as close as possible.
In PHP a non-existing argument becomes in the end NULL + additional warnings and notices.

@kukawski sure, no problem.
Also, if you support ES6 features, this could be easily be refactored to use "default arguments", because all that code does is making sure that you have a default value, if none is passed as an argument.

Regarding the value + '' -> String(value) change. If we want to improve PHP compatibility, then we should use https://github.com/kvz/locutus/blob/master/src/php/_helpers/_phpCastString.js instead, which casts values to string very close to PHP's algorithm.

ES6 Is fine and I'd be in favor of using default arguments to settle it. You @kukawski ?
As for doing String() vs + ''. Isn't this exactly the same only longer notation? What's the benefit?

@kvz @andreipfeiffer agree, using default params would be an improvement for us.
Regarding String() vs + '', technically both give same results. The only visible benefit would be slightly better code readability (because conversion is explicit).
Don't know about JS engine implementation, but using String() would eventually cause just one memory allocation for the converted string, where + '' probably needs at least 2 allocations - one for casting to string and second for concatenating. But that's minor.
Anyway, there is still the topic of using _phpCastString.js for casting in this case.

Hey @kvz @kukawski in regard to using _phpCastString.js I would not recommend this approach.
If you look at the implementation, it performs a lot of checks, so it's definitely slower than any other approach discussed above.
Also, on the case of Numbers, all it does is perform a '' + value, which you are doing already.
So, just let me know if you need me to prepare another PR, in case you agree upon the explicit String(value) proposal.
Thank you guys!

And in regard to "null checking", I am aware of the "hack" value == null.
It's just that this obscures the intent: you don't know if the developer actually wanted to check for null values or not. As I can tell from the code, you actually want to set a default value, when certain arguments are not supplied to the function.
If this is the case, then those arguments will always be undefined, never null (unless you pass the null value explicitly).
If also you want to avoid null values being passed, and still make sure that the argument is a number with a default value of 1, then another approach would be:
costIns = Number(costIns) || 1

This way, if the type coercion results in a NaN (for undefined/null/NaN) it will be initialized with 1.
To keep in mind that ES6 default params are only checked against undefined, so they work only when you don't pass the argument, or when you pass explicit undefined. When you pass null, it will remain null.
Just let me know what do you decide or if you want to discuss this in more detail.
I appreciate your interest.

Tested in Firefox 57 (String(n)).replace(/[^0-9+\-Ee.]/g, '') is 32% slower than (n+'').replace(/[^0-9+\-Ee.]/g, ''); and Chrome the String object is 9% slower. So if we're talking about performance, casting it to a primitive string and performing the replace is faster.

@Knight-Yoshi I've done some testing myself and I got consistent:

similar perf in Chrome
similar perf in Safari
slightly better perf for '' + foo in Firefox

But the fact is that this perf gain/loss is calculated from a very small value, so the actual impact is minimal.
And also, any perf test should be taken with a large dose of skepticism, because it's not trivial to create relevant perf tests.
Most devs prefer the String(foo) variant, because it's more explicit.
Apparently, both work similarly behind the scenes:
http://2ality.com/2012/03/converting-to-string.html
But this being a library, I totally get it if you prefer to have a less explicit code base in favor of a maybe a slightly better perf.

Sorry for the long delay, while this may or may not be true:

Most devs prefer the String(foo) variant, because it's more explicit.

I personally don't favor the String variant, and since it also comes at a performance cost, this is a no go for me. Sorry!

I've seen a lot of JS, but I've never seen anyone use the String object explicitly.
https://jsperf.com/string-object-vs-string-primative/1
Tested again using Firefox 63, Chrome 70 on Mac and in Windows using Edge (which was listed as Chrome 64 in the results O.o)
Consistently the primitive type was faster, in Firefox it's still considerably faster. Which makes sense. Since the primitive type isn't calling a object constructor.


Inference time, and implementation of NeuS/Instant-NeuS

Thanks for the great work! I have 2 questions:

What is the inference time of the method compared to NeuS and Instant-NeuS? Since your paper mentions that NeuDA converges rapidly and yields the lowest value compared to NeuS and Instant-NeuS, which means NeuDA can produce more clear and sharper surfaces with less time cost.

You compared to NeuS and Instant-NeuS in the paper. Do you mind sharing your implementation of those code?

Hope to hear from you soon, thank you!

Sorry for the late reply.
NeuDA is faster than NeuS since it employs shallow MLPs (4 vs. 8 for NeuS), while its inference time is similar to Instant-NeuS. In addition, our experiments found that NeuDA achieves a better surface reconstruction than NeuS and Instant-NeuS with the same training iterations.
We use the original NeuS code in our experiments. For the Instant-NeuS, we replace the position encoding with hash encoding and leverage shallow (4-layers) MLPs to construct an SDF network that is easy to reproduce. Moreover, there are currently many excellent open-source repos you can refer to, e.g. Instant-nsr-pl, torch-ngp, instant-ngp.


Allows tpm2-abrmd restart for test run

This change introduces a slight change to the package manager
selection and in turn installs python-dbus
Further changes are made to test_restful.py whereby if the
system is running within systemd mode with an active dbus, restart
tpm2-abrmd.service. If not, do not try to restart the tpm2-abrmnd.
The logic being, a non systemd based service will be running a
tpm1.2 system.
The service restart is required as when using mssim, the resource
manager cannot reconnect.
This allows continued use of one set of test scripts for both
TPM versions.

This looks really good, @lukehinds!  Just a few small changes so we can run this from the command line and on non-dnf systems and then I think it is ready to merge.

Thanks @jetwhiz for reviewing , well spotted about the hacked off shebang. I have seen this before and I think it might be a wayward editor extension I have. Change incoming.

@jetwhiz , we should be good now, give me a quick review and I will merge this and we have CI coverage.
p.s.. I was thinking last night, I will refine this over time. It might be more tidy to move the tpm2_wrapper into run_tests.sh. Let's see though, if upstream see the rmd disconnect as an issue (btw, I put a comment in above the dbus python code).

Hi @lukehinds -- this looks better, I think we just need to put in spaces around the "=" sign for the if-else statements (starting at line 142) and then this is ready to merge.
As-is, the run_tests.sh script doesn't run properly on Ubuntu (and non-dnf systems) since if [ $PACKAGE_MGR="dnf" ]; then is always true (the equals is not evaluated when there are no spaces around the =).

Side note. Will look at running a CI job in Ubuntu (which will capture what you found above 👍)


What's the difference between procdump and gcore?

Apart from conditional core dump, e.g monitoring the CPU usage of target process
What's the difference between this and the traditional gcore tool?

gcore is an on demand dump generation tool which works well in the low hanging fruit scenarios where issues are fully reproducible. In production however, that is rarely the case and you need a tool that can actively monitor and trigger dumps. We have a future work item to remove the dependency on gcore to allow more flexibility in the types of dumps we create.

Thanks!


gdb-remote debugging crash

OS: AOSC OS (Linux 5.17) and Arch Linux (5.16.14) x86-64
VSCode version:  1.65.1
CodeLLDB version: 1.7.0
Compiler: gcc
Debuggee: gdbserver
I tried to use codelldb to connect to the gdbserver debugger and my configuration file is as follows:
        {
            "type": "lldb",
            "request": "custom",
            "name": "launcher debug",
            "targetCreateCommands": [
                "target create ${fileDirname}/${fileBasenameNoExtension}"
            ],
            "processCreateCommands": [
                "gdb-remote <IP_ADDRESS>:2333",
            ],
        }
,

After that, I tried to debug the program in VSCode with codelldb:

#include <stdio.h>

int main()
{
    int num = 0;
    while (num < 10000) {
        num += 1;
        printf("%d\n", num);
    }
    return 0;
}

This step will be followed by a crash:
555555555052: 49 89 D1                   movq   %rdx, %r9

VSCode output:
Stop reason: signal SIGTRAP
Stop reason: signal SIGSEGV
Process exited with code 0.

I noticed that journalctl outputs the following information:
systemd-coredump[35258]: [🡕] Process 35241 (hello) of user 1000 dumped core.

                                             Stack trace of thread 35241:
                                             #0  0x0000555555555051 n/a (/home/saki/test/hello + 0x1051)

And use objdump:
    1050:       31 ed                   xor    %ebp,%ebp
    1052:       49 89 d1              mov    %rdx,%r9

It seems to be because the wrong address was accessed? I noticed that the instruction was at 0x1052 but it tried to access 0x1051 and then raised this error.
But using lldb on the terminal to connect to gdb-remote debugging, everything works fine

Just FYI that this has been fixed in https://github.com/llvm/llvm-project/commit/5dbec8c6ce473352cac2d89d6a5b81f65182df59


Enable Mesa workaround also on Mesa 21

Description
This enables the Mesa workaround for Mesa 21.x, too (and all future Mesa 2?.x versions).
My system reports:
GL_RENDERER   = AMD RENOIR (DRM 3.40.0, 5.10.24-051024-generic, LLVM 11.0.1)
GL_VERSION    = 4.6 (Compatibility Profile) Mesa 21.0.0 - kisak-mesa PPA
GL_VENDOR     = AMD

The fix merged in #1588 is limited to Mesa 20.x only, but the problems it works around are present in 21.x, too.
Checklist

[ ] If you are addressing rendering issues, please provide:

[ ] Images of both, broken and fixed renderings.
[ ] Source code to reproduce the issue, e.g. a YAML or rosbag file with a MarkerArray msg.

[ ] If you are changing GUI, please include screenshots showing how things looked before and after.
[ ] Choose the proper target branch: latest release branch, for non-ABI-breaking changes, future release branch otherwise.
Due to the lack of active maintainers, we cannot provide support for older release branches anymore.
[ ] Did you change how RViz works? Added new functionality? Do not forget to update the tutorials and/or documentation on the ROS wiki
[ ] While waiting for someone to review your request, please consider reviewing another open pull request to support the maintainers of RViz. Refer to the RViz Wiki for reviewing guidelines.

Thanks.


Support providing an optional id to limits/counters

add a key_for_counters_v2 function that uses the id as the key if set, otherwise uses the previous key encoding strategy.
updated the distributed store to use key_for_counters_v2.  Since we can’t decode a partial counter from id based keys, we now also keep in memory the Counter in a counter field of the limits map.

moving the id out of the constuctor drastically reduced the size of the change set.

after running the rate_limited_id_counter_with_sync_redis the keys in redis look like:
root@255340cd3402:/data# redis-cli --scan --pattern '*'
"\x02\x1ctest-rate_limited_id_counter"
"\x02\x1ctest-rate_limited_id_counter\x01\x06app_id\x0btest_app_id"

Since I'm new to rust, I'm not sure about the "Should we push the version in our cargo.toml to 2.0.0-dev (and 1.0.0-dev for the crate)?" bit.


Replace html with frontend tookit pattern in agreements section.

Replace html with frontend pattern list in agreements section.  Remove list scss file specific to frontend supplier app as lists are common to digitalmarketplace apps and this should be pulled in from the frontend toolkit.
Relies on - https://github.com/alphagov/digitalmarketplace-frontend-toolkit/pull/225
Screenshot 1:

Screenshot 2:

There seems to be a lot of extra space at the bottom of the page before the green button. I'm not sure if this is an issue? I think the old version looks a bit better maybe?
OLD:

NEW:

The list itself has now had padding removed from the bottom and the margin above the form button is being kept in for the time being.
This now relies on:
https://github.com/alphagov/digitalmarketplace-frontend-toolkit/pull/229

Bit funny how the list style does numbers on the demo page, but doesn't really seem like an issue for our real-life code.
Looks better than before :+1:


ci(gha): fix submodule check

Ensures we fetch git history and submodules.

weird ssl error, don't think anything to do with this PR... let's try again
bors merge

bors merge

bors cancel


Shortcuts not working. Mainly the ones with ctrl in them

VSCode Version: 1.43.
OS Version: Parrot home

Steps to Reproduce:

Get Parrot home
Install vscode

Does this issue occur when all extensions are disabled?: Yes
This is probably an issue with the os, maybe it uses ctrl for something else, even tho' I checked and it isn't, only combinations with alt are used. Furthermore, I have to type ctrl+shift+و to enter preferences. Is this induced by the keyboard app? I do have the options to change the keyboard model current one is generic 105-key Pc (intl.)

They work now. Maybe keyboard isn't good?

@charbelsako Can you please try other software on the machine? How about Chrome, does Chrome receive keybindings with Ctrl?

Ctrl works everywhere else. I think this is because i had arabic as a higher priority than english. @alexdima

:+1: Great that it worked. If you have other issues, you can also try "keyboard.dispatch": "keyCode".


peg2pegtl example

This change implements a pegtl generator from peg grammars, as defined in the original peg paper.  Included is the original peg grammar (peg.peg).  I've regenerated the peg pegtl grammar components in peg.hpp with the output of peg.peg, and it is able to self-host.

Codecov Report

Merging #254 (43e94b4) into master (d2aec05) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #254   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          253       253
  Lines         5786      5786
=========================================
  Hits          5786      5786

Continue to review full report at Codecov.

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

Codecov Report

Merging #254 (43e94b4) into master (d2aec05) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #254   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          253       253
  Lines         5786      5786
=========================================
  Hits          5786      5786

Continue to review full report at Codecov.

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

I fixed the namespace issue (tao::pegtl:: could be redefined to something else via TAO_PEGTL_NAMESPACE and you don't need it anyways)
I used some more idiomatic rules like list<> and until<> and I removed some superfluous seq<>s. It's not exactly the direct mapping from the paper now, but we shouldn't show such inefficient grammars in the examples. I hope you agree :)

I fixed the namespace issue (tao::pegtl:: could be redefined to something else via TAO_PEGTL_NAMESPACE and you don't need it anyways)
I used some more idiomatic rules like list<> and until<> and I removed some superfluous seq<>s. It's not exactly the direct mapping from the paper now, but we shouldn't show such inefficient grammars in the examples. I hope you agree :)


Fix releasing script; [WIP]

Description
The bundle exec fastlane ios release is going to do the followings in order

Ensure there is no uncommitted changes.
Fetch the latest release version from private SDK.
Download the .framework files from the release of private SDK.
Update the changelog according to the changelog of private SDK.
Bump the podspec version number.
Create a new tag and push it to remote.
Push the master branch to remote.
Create the sample project.
Create a new release on GitHub.
Push podspec files to remote.

          1 Warning

:warning:
fastlane/Fastfile modified

  Generated by :no_entry_sign: Danger

I mark this PR as [WIP], because I haven't tested yet.
I am going to test it after there is a new version of private SDK.

@David-Henner @vincentjacquesson
I'm going to merge this, in order to let @jotshin test the releasing flow.
If there is anything needs changing, please inform me and I will create another ticket. 😄


Grid cell area calculation breaks with CF-Python version 3.14.0 and above

The refactor of the CF-Python regridding code for dask implementation in v3.14.0 (https://github.com/NCAS-CMS/cf-python/pull/438) causes the calculation of grid cell area to fail for unifhy.space grids with spherical coordinates.
The _compute_cell_area function in unifhy.space uses the ESMF.Regrid operator calculated as part of a regridding operation initiated through CF-Python.
In CF-Python v3.13.1 (the version immediately prior to v3.14.0) the return_operator kwarg to the cf-python.field.regrids function returns a cf.RegridOperator object via the regrid_create_operator function in cf-python/regrid/utils. The cf.RegridOperator object has a 'regrid' attribute which is the ESMF.Regrid operator object associated with the regrid requested.
This behaviour changes in v3.14.0. cf-python.regrids and regridc now call the regrid worker function in a new regrid/regrid file and the cf.RegridOperator object returned by it no longer has a regrid attribute from which to access the ESMF.Regrid operator object. Instead, this functionality is separated out from the cf.RegridOperator object and incorporated into the regrid worker function/subfunctions, which will return the ESMF.Regrid operator if a return_esmpy_regrid_operator=True kwarg is passed to it. However, neither regrids or regridc define this kwarg, so the only way to access the ESMF.Regrid operator is by calling the regrid worker function directly, which is probably not good practice, but might be a necessary workaround for now to get UnifHy working again.

I'm currently unsure why this only occurs for spherical grids and not cartesian grids... Will investigate.


Delete manifests/m/Metriusz/BedwarsifyOverlay/1.12.1 directory

Reason: The repository is archived.

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


CI: fix more flakes, move itests to GitHub (except ARM itest)

Depends on https://github.com/btcsuite/btcd/pull/1752.
Fixes two problems in the itest:

Some instances of premature channel announcements caused by not all subsystems being in sync --> We fix this by always waiting 50 (or maybe 20 would be enough?) milliseconds after each mined block.
Some instances of the mining btcd and chain backend btcd node losing their connection because of the peer stall detection in btcd --> We fix this by disabling stall detection

Wow, all GitHub itests green on the first run :open_mouth:

Wow, all GitHub itests green on the first run

Is this good or bad?

Wow, all GitHub itests green on the first run

Is this good or bad?

I would say this is very good. Not sure why you would think it wasn't?

I would say this is very good. Not sure why you would think it wasn't?

If the tests were previously flakey because of them running on slow test machines, it could be that they uncovered issues that only show on slow production machines. So perhaps these are missed now with github actions. I have to admit that I don't even know for sure that tests being green is caused by faster test machines.
Sorry about the lack of threading, should have put my initial comment on some line.

If the tests were previously flakey because of them running on slow test machines, it could be that they uncovered issues that only show on slow production machines.

I think it's mainly the lack of consistent timing w/ the series of timeouts we have. When we run w/ Travis (and their potato cluster) we end up with several processes (replicated db, 2x full node, up to 6 lnd nodes in some tests), so it's understandable that we run into some CPU scheduling weirdness that causes these flakes at times. At the same time, we've also eliminated a ton of flakes over the past 2 months due to flake hunting szn.
Travis as a service has really consistently degraded over the past year or so, then they have that massive security failure on top of that. We've given them enough chances to get their services together after being acquired by that PE firm IMO.

I think the bigger gain here is also the restoration of all the lost developer time (sitting there baby sitting the test to restart it, odd failures w/ the machine (?) itself) due to Travis.

Also worth nothing this brings in the btcd fix to disable the stall handler on simnet, which caused disconnections from the main miner node, which caused a ton of issues w.r.t transactions not properly propagating.

Rebased. But still blocked by btcsuite/btcd#1752.

Interceptor tests need a wait.Predicate somewhere:
    lnd_rpc_middleware_interceptor_test.go:417:
        	Error Trace:	lnd_rpc_middleware_interceptor_test.go:417
        	            				lnd_rpc_middleware_interceptor_test.go:125
        	Error:      	"rpc error: code = Unknown desc = the RPC server is in the process of starting up, but not yet ready to accept calls" does not contain "middleware 'itest-interceptor' is currently not registered"
        	Test:       	TestLightningNetworkDaemon/tranche01/84-of-85/btcd/rpc_middleware_interceptor/mandatory_middleware


Allow commands whose output is not logged unless they fail

There is a class of commands whose output is interesting only when they fail. Allow running them, and do not log their output in real-time, but log it if the command exits with non-zero exit code.
Pull Request Checklist

[x] implement the feature
[x] extend the test coverage

Failing test is an unrelated issue #2744.


Is there any method to record SVO file without CUDA?

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues.
[X] This issue is not a question, bug report, or anything other than a feature request directly related to this project.

Proposal
I want to record the SVO file using a mini PC without NVIDIA graphics, is there any method to use this SDK without CUDA?
And then I will use a Linux server with CUDA to export the SVO file in any format.
Use-Case
No response
Anything else?
No response

Hi @memoryunreal
this is the ZED Open Capture repository and it is not related to the ZED SDK.
We cannot add SVO features here because this is an Open Source project and SVO handling is closed.
Please write an email to<EMAIL_ADDRESS>or write a post on the forum for your request.


[MLIR][Brainstorm] Make MLIR become part of release MIOpen package

Existing State
MLIR is an experimental compilation option for MIOpen. Assuming that the MLIR library and header has been shipped at ROCm release folder, a developer can use -DMIOPEN_USE_MLIR=On to turn on MLIR binary backend. This ticket is the umbrella item that documents what needs to be done to make MIOpen release package to also take advantage of MLIR capabilities.
June Demo
All MLIR effort has been geared towards June demo, such that one can run resnet50 NHWC model in TensorFlow using MLIR supported solvers in MIOpen. This means that by June demo we support:

MLIR as an experimental option of MIOpen: -DMIOPEN_USE_MLIR
MLIR as an optional test option of MIOpen: -DMIOPEN_TEST_MLIR
Hip backend, since framework supports miopen-hip only
Package delivery use cmake build
Find API. MLIR solvers won't work in immediate mode because GetWti() not implemented
No tuning for MLIR solvers. In general we do not have tuning capability for NHWC
Perfdb dependency. We consider MLIR to be part of MIOpen, not as an independent release package. Also to achieve the best performance, MLIR need to have access to perfdb.
~GEMM explicitly disabled for WrW. Gemm by default run in NCHW, which give wrong result for NHWC.~
Is not compatible with COMGR, since both ship with llvm and symbol collision may exist.
MLIR shipped as static library
MLIR capable of static shape only. No dynamic kernel supported but they can/will be supported
MLIR exposed binary interface to MIOpen
Revisit existing MIOpen assumptions.

Work items to make MLIR part of released MIOpen
We have to revisit each of the above item one by one to decide the complete work needed.
MLIR as an experimental option of MIOpen
For this we need to flip the switch. We either get rid of -DMIOPEN_USE_MLIR, or keep the option, but make release build with the option. The further implication of this is such that MLIR become part of ROCm release.
MLIR as an optional test option of MIOpen
We need to do the same thing as above section for -DMIOPEN_TEST_MLIR. Besides, instead of making only smoke stages test MLIR, we need to make full test stages to also pick up MLIR for the guarantee of MLIR-based solver correctness.
Hip backend
MIOpen supports both opencl and hip backend. We need to also make sure opencl backend works, and get tracked in CI.
Package delivery
MLIR package delivered to MIOpen through building libMLIRMIOpen.a static library. From MIOpen perspective, this needs to be amended to using cget.
An orthogonal topic, from MLIR perspective, there needs to be a way to deliver the static library and its header together with ROCm release.
Find API
We need to decide how MLIR solvers behave in non-find mode.
No tuning for MLIR solvers
There are two orthogonal topics here.

MIOpen doesn't support NHWC tuning in general. Since part of MLIR solvers capability is the support of NHWC, MIOpen doesn't support the tuning for that partial of solvers.
Tuning support feature in MLIR solvers. By far MLIR is just a kernel generator, we need dedicated APIs for tuning. This means additional work in MLIR, MIOpen and the interface between them. However, we may want to first decide if we want tuning at all for MLIR solvers. If the performance impact between non-tuned and tuned solvers are not as much, we may not need this feature.

Perfdb dependency
Once tuning functionality is implemented, both MIOpen and MLIR will be dependent on perfdb. Right now if MIOpen release package come with MLIR, this won't be a problem. In the future when MLIR is released as an independent package (and no MIOpen is installed), MLIR will be able to work in only heuristic mode.
GEMM explicitly disabled for WrW
This blocks MIOpen to populate correct result for NHWC format. We want to make GEMM to be solvers, for all 3 directions, WrW included.
Is not compatible with COMGR
We have two options:

Make MLIR based on the same commit of LLVM as COMGR does. This eliminate the possibility of symbol conflict.
Decide whether it make sense to build the two features together. It may be okay to leave COMGR and MLIR as two independent features.

MLIR shipped as static library
We may choose MLIR to be shipped as dynamic library. This will go against embedded builds. However, again, we want to decide on this point and this might be okay.
MLIR capable of static shape only
What impact can MLIR has to MIOpen when MLIR also support dynamic shape? Does it change any MIOpen existing assumptions?
MLIR exposed binary interface to MIOpen
Decide on the best and most robust kernel binary interface. Being able to hide the StridedMemRef5D make it a more robust interface. We want to revisit the alternatives and use cases to make this decision.
Revist existing MIOpen assumptions

If 3x3 kernels, in NCHW mode, MIOpen by default fall back to Winograd. We may want to double check if Winograd is better than MLIR kernels.
Immediate Mode by default fall back to GEMM. However GEMM supports NCHW only. MLIR kernels may be the option to fall-back to if it support wide enough configs, NCHW and NHWC.
Pre-compiled kernels. It doesn't make sense for MLIR kernels to be pre-compiled. This is because MLIR kernels compilation time is trivial and it doesn't worth it to trade space with runtime for MLIR kernels. How does it impact MIOpen behavior if MLIR kernels does not get pre-compiled? Nothing needs to be done on pre-compiled kernels end.

Summary of features:

Features
Effort

MLIR as a build option of released MIOpen
Small

MLIR as a test option of released MIOpen
Small

MLIR solvers support opencl backend
Medium

MLIR solvers support immediate mode
Small

Tunning support for MLIR solvers
Large

MLIR package deliver through cget
Small

MLIR multi-threaded library compilation tests
Medium

Misc features (env var to control MLIR solver, 3x3 winograd NCHW fallback, Deprecate cpp backend)
Medium

One comment on tuning costs.
I expect MLIR to be used in multiple contexts, e.g. (1) generating code for MIOpen kernels (2) supporting TF/XLA or similar framework and (3) integration with MIGraphX.  Not all these are fully online or fully offline but potentially a mix of both.  In the context of some of the more offline scenarios e.g. MIGraphX "model optimizer" followed later by deployment - we may want some tuning and be willing to pay some of the costs since it is separated from the execution stage later.
So I think some form of tuning will still be useful for these sorts of scenarios.

I synced with @atamazov offline and below are summary of our conversations:
Tuning:
Key concerns comes from the following area:

Perfdb access time for the initial round can be as high as 200 ms. However, this is not a problem for frameworks because that is done only once.
New kernels will focus more on dynamic kernels. Dynamic kernels does not take advantage of perfdb good enough. - We need to have independent entries for each config that dynamic kernels tuned against.
Tuning (maintenance) overhead is quite high.

The key question therefore becomes a way to evaluate igemm kernels fairly. We ended up doing this in #939.
As it turns out in #939, we can still achieve 5x speedup in tuning in best case scenario, and on average 3x speedup in WrW tuning. Considering this, tuning feature is still necessary. It is possible that disable solvers' whose tuning is not as effective after the tuning feature is implemented.
COMGR:
COMGR and MLIR continue to be independent features. Unless required by customers, we will continue to keep them independent.
Static library
MLIR can continue to ship as a static library and get statically linked to MIOpen. The only caveat is that customer with LLVM symbols will not be able to statically link MIOpen, but use MIOpen as a dynamic library. MLIR can ship as a dynamic library should we have the requirement, we just don't have the incentive to ship MLIR as one dynamic library now.
Existing binary interface
From MIOpen's perspective, the existing binary interface is simple and performant enough.
From MLIR's perspective, the interface is still quickly evolving, we should not optimize this field unless it is completely stablized.

MLIR can continue to ship as a static library and get statically linked to MIOpen.

So we would disable MLIR when building MIOpen statically?

The only caveat is that customer with LLVM symbols will not be able to statically link MIOpen, but use MIOpen as a dynamic library.

Statically linking HIP will bring in LLVM symbols.

MLIR can ship as a dynamic library should we have the requirement, we just don't have the incentive to ship MLIR as one dynamic library now.

But MLIR still needs to be a static library as users wanting to build MIOpen as a static library want to build everything statically. More importantly, some frameworks want to statically link rocm in their pip wheel, so this type of deployment may become more common.

COMGR and MLIR continue to be independent features. Unless required by customers, we will continue to keep them independent.

We should be enabling COMGR by default, as this was a requirement for pytorch's pip wheel. So I dont see how we can treat it as independent. They both need to work together.

@pfultz2 Thanks for bring these up.

So we would disable MLIR when building MIOpen statically?

If COMGR has to come with MIOpen's static release package, then yes. I don't see any requirement that MLIR must be available in that scenario.
My understand is that this milestone can target MLIR to be built as part of the dynamic release package of MIOpen.
@atamazov Please correct me if I'm wrong.

We should be enabling COMGR by default, as this was a requirement for pytorch's pip wheel.

I might need more clarification on it. Who is the stakeholder on this front? I will ping Jeff Daily on it. I need to clarify whether MLIR is a required feature under that scenario.

We should be enabling COMGR by default, as this was a requirement for pytorch's pip wheel.
I might need more clarification on it.

Well, better said, MIOpen needs to be able to compile programs without calling to clang or using headers on the filesystem of pytorch's pip wheel. This also needs to be done more widely as we move to "no path" instead of using absolute or relative path.

However, I seriously doubt it that's achievable considering everything is in so fast development stage.

Ideally, we would move MLIR into our rocm fork. This way the team that merges with LLVM upstream can take care of breaking changes(so we aren't duplicating efforts).
Alternatively, can't we rename symbols(-Dllvm=rocm_lllvm_mlir) to avoid conflicts?

Ideally, we would move MLIR into our rocm fork.

This we are on the same page. I just don't know if MLIR is at that maturity level yet.

Alternatively, can't we rename symbols(-Dllvm=rocm_lllvm_mlir) to avoid conflicts?

Refer to here for everything I have tried the previous time. That -D approach doesn't work because dumped symbols does not change at all before and after applying this. Also BTW, llvm namespace is just one of the many namespaces... Internally we also enabled mlir, lld and so forth. I don't see how this redefinition of namespace can work or scale.
I'm also reading through a number of materials, including this stackoverflow answer. My conclusion has been basically that to hiding or renaming or internal linking those symbols are just not possible. Unless we are on ROCm release and using the same commit of LLVM as the ROCm release.

This is to document that in the meeting @atamazov pointed out that there are duplication between:

Cpp igemm solvers
MLIR backend igemm solvers

Since Cpp igemm solvers compilation speed of slower than that from the MLIR backend, we want to disable / (de-prioritize) cpp igemm solvers.
We will postpone this feature because:

The MLIR backend will be only available in shared library, therefore limited scenarios. This will create a complication that we need to invent the on/off mechanism of those two groups of solvers, and make sure things like perfdb are in sync.

This will no longer be a problem once MLIR backend becomes available in all use cases. If we postpone this feature to that time, the on/off mechanism will no longer be needed

At this milestone, we still use the cpp igemm solvers as the benchmarking baseline (with perfdb on).

@jerryyin should I convert this issue into "discussion"?

@junliume 90% of features in this ticket are done in the past year. I'd treat this as a specification in the past. We can close this one now.

@jerryyin Specifications should not be closed unless outdated/invalid. But we can assume this ticket is an enhancement. Can you please change labels? Thanks.

@atamazov Sounds reasonable either way, we can:

Reopen this ticket as specification as this documents the level of support between rocMLIR/MIOpen integration
Or, keep it closed but add the enhancement label treating this as outdated as we start to phase out the integration

Are you indicating the 2nd alternative?

@jerryyin Yes, the 2nd one.


Seemingly random crashing in audio_file_read

I am getting a number of EXC_BAD_ACCESS crashes in DOUAudioPlaybackItem.m line 71.
That is in the audio_file_read function on this line:
memcpy(buffer, (uint8_t *)[[item mappedData] bytes] + inPosition, requestCount);
I can't say for certain, but I think it's an issue with buffer getting deallocated before this function gets called.

Closing for now, might re-open. I saw some commits changing logic around this.


RD2 [Elevate]: Support Cardholder Name in the Credit Card widget entry form

Add support for capturing a single field for "Cardholder Name" in the RD credit card widget and passing that value to the Tokenization process

Critical Changes
Changes
Issues Closed
Community Ideas Delivered
New Metadata
Custom Label:

gePaymentRequestCardholderName

Deleted Metadata

Work Item: W-7863547

Thanks for the review @andrewyu-salesforce. Changes made.

Thanks for the code review @RadSFDeveloper. All changes made


[WIP] search bar bugfix

Remove the static configuration for the search bar. It's now activated for any release with ..Release.Config.As equal to Stable (releasecontroller.ReleaseConfigModeStable)

/cc @bradmwilliams

/unhold


API doesn't send correct json

All of the web API sends single quotes instead of double quotes
https://github.com/gmag11/EnigmaIOT/blob/master/docs/api.md

Yes, this is intended to simplify string management in C++ code. single quotes are valid in JSON, aren't they? At least Insomnia shows correct decoded JSON data.

Yes I saw that in your code, unfortunately no JSON spec is double quotes only, some references here: https://stackoverflow.com/questions/14355655/jquery-parsejson-single-quote-vs-double-quote
I can edit it, if you don't have time, but I would just switch all of the ' to \" instead

Thank you. I'll change this in next release.

It should be solved in a5b2a76. Thanks again


Consume embedded SEQUENCE in a clean way

I have a BER schema which looks like the following :
SEQUENCE {
    typeA SEQUENCE {...}
    typeB SEQUENCE {...}
}

I try to retrieve fields that are located in typeA SEQUENCE, how can I inspect it in a way to deal with errors, i.e. is it possible to do something like the following :
let (rem, result) = Sequence::from_ber_and_then(input, |i| {
     let (rem2, a) = Sequence::from_ber_and_then(input, |j| {
          let (rem2, a) = u32::from_der(input)?;
          ....
         Ok(...)
})?;

Is it the right to do that ? I'm not able to find the correct code to write in order to run that.
Thanks in advance

You have multiple options
If you know the definition of your types, the recommended solution is to create a struct for each sequence and derive parsers to be able to call from_der. See for example this unit test:

T1 is the container
T2 would be something like typeA in your example
If you want BER (and not DER), use the BerSequence derive and from_ber in all the examples below

Resulting code would look like:
#[derive(Debug, PartialEq, DerSequence)]
pub struct MyType {
    a: TypeA,
    b: TypeB,
}

#[derive(Debug, PartialEq, DerSequence)]
pub struct TypeA {
    field1: u32,
    ...
}

let (rem, obj) = MyType::from_der(input)?;

If you can't derive or prefer to use manual parsers, the solution you describe will work, but it will not be idiomatic.
You have multiple solutions (again, deriving is preferred):

reading the first sequence only, then parsing the content:

let (rem, seq) = Sequence::from_der(input)?;
let (inner_rem, type_a) = Sequence::from_der(seq.content)?;
let (inner_rem, type_b) = Sequence::from_der(inner_rem)?;

there are multiple variants of the above code, for ex:

let (rem, seq) = Sequence::from_der(input)?;
let (inner_rem, type_a) = Sequence::from_der_and_then(seq.content, |i| {
    let (rem2, a) = u32::from_der(input)?;
    ...
   Ok(...)
})?;

let (inner_rem, type_b) = Sequence::from_der(inner_rem)?;

you can also mix both solutions (derive and parsing):

let (rem, seq) = Sequence::from_der(input)?;
let (inner_rem, type_a) = TypeA::from_der(seq.content)?;
})?;

you can also not use the automatic derive, and implement FromDer. See for example x509-parser-0.16.0/src/revocation_list.rs#L128-L148 (and all other parsing code from the x509-parser crate) for example using custom errors
Note: parse_der_sequence_defined_g is the old way to parse a sequence, equivalent to Sequence::from_der_and_then

Thanks for your answer !
I will check for the idiomatic way if my use case grows. For now, I only need to retrieve a few fields in the BER data


fix: fix httpStatusToGrpcStatus always return 13

What is the purpose of the change
fix httpStatusToGrpcStatus always return 13
issus https://github.com/apache/dubbo/issues/14498
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

I will fix it in PR: https://github.com/apache/dubbo/pull/14513


Generic zkvm Witness

This PR contains a refactor of the WitnessColumns of MIPS and the KeccakWitness of Keccak, to follow the same format unified as Witness, and then it is instantiated using a constant generic for the number of columns each of them requires.
I suggest reviewing this PR per-commit. The diff is not as large as it seems, because there's a lot of code moving around and simplifications. But other than the refactors, it is not introducing any extra logic.
Addresses https://github.com/MinaProtocol/mina/issues/15162 and https://github.com/o1-labs/proof-systems/issues/1718
A follow-up PR will use this to make the proof.rs file generic as well, so it can be reused for both keccak and mips (https://github.com/o1-labs/proof-systems/issues/1705)

I'm reviewing now. I'm running the code locally on a FILENAME=2023-12-13-15-02-00-op-program-data-log.sh

Perf:
[2024-02-27T11:17:39Z INFO  kimchi_optimism::mips::witness] Snapshot state in snapshot-state-280000000.json, step 280000000
[2024-02-27T11:19:05Z INFO  kimchi_optimism::mips::witness] processing step=290000000 pc=0000141c30 insn=0080370000 ips=113948.92 pages=5894 mem=23.0 MiB name=internal/poll.(*fdMutex).rwlock
[2024-02-27T11:19:07Z INFO  kimchi_optimism::mips::witness] Snapshot state in snapshot-state-290000000.json, step 290000000
[2024-02-27T11:20:33Z INFO  kimchi_optimism::mips::witness] processing step=300000000 pc=00005811ac insn=0024210001 ips=113938.47 pages=5922 mem=23.1 MiB name=github.com/ethereum/go-ethereum/trie.(*nodeIterator).peek
[2024-02-27T11:20:35Z INFO  kimchi_optimism::mips::witness] Snapshot state in snapshot-state-300000000.json, step 300000000
[2024-02-27T11:22:02Z INFO  kimchi_optimism::mips::witness] processing step=310000000 pc=000022cf90 insn=0000000825 ips=113886.85 pages=5952 mem=23.2 MiB name=github.com/ethereum/go-ethereum/rlp.(*Stream).readByte
[2024-02-27T11:22:04Z INFO  kimchi_optimism::mips::witness] Snapshot state in snapshot-state-310000000.json, step 310000000
[2024-02-27T11:23:30Z INFO  kimchi_optimism::mips::witness] processing step=320000000 pc=00000164d8 insn=001060fffa ips=113879.00 pages=5990 mem=23.4 MiB name=runtime/internal/atomic.spinLock
[2024-02-27T11:23:32Z INFO  kimchi_optimism::mips::witness] Snapshot state in snapshot-state-320000000.json, step 320000000

Perf:
[2024-02-27T11:17:39Z INFO  kimchi_optimism::mips::witness] Snapshot state in snapshot-state-280000000.json, step 280000000
[2024-02-27T11:19:05Z INFO  kimchi_optimism::mips::witness] processing step=290000000 pc=0000141c30 insn=0080370000 ips=113948.92 pages=5894 mem=23.0 MiB name=internal/poll.(*fdMutex).rwlock
[2024-02-27T11:19:07Z INFO  kimchi_optimism::mips::witness] Snapshot state in snapshot-state-290000000.json, step 290000000
[2024-02-27T11:20:33Z INFO  kimchi_optimism::mips::witness] processing step=300000000 pc=00005811ac insn=0024210001 ips=113938.47 pages=5922 mem=23.1 MiB name=github.com/ethereum/go-ethereum/trie.(*nodeIterator).peek
[2024-02-27T11:20:35Z INFO  kimchi_optimism::mips::witness] Snapshot state in snapshot-state-300000000.json, step 300000000
[2024-02-27T11:22:02Z INFO  kimchi_optimism::mips::witness] processing step=310000000 pc=000022cf90 insn=0000000825 ips=113886.85 pages=5952 mem=23.2 MiB name=github.com/ethereum/go-ethereum/rlp.(*Stream).readByte
[2024-02-27T11:22:04Z INFO  kimchi_optimism::mips::witness] Snapshot state in snapshot-state-310000000.json, step 310000000
[2024-02-27T11:23:30Z INFO  kimchi_optimism::mips::witness] processing step=320000000 pc=00000164d8 insn=001060fffa ips=113879.00 pages=5990 mem=23.4 MiB name=runtime/internal/atomic.spinLock
[2024-02-27T11:23:32Z INFO  kimchi_optimism::mips::witness] Snapshot state in snapshot-state-320000000.json, step 320000000

Looks like on master I'm getting the same. I remembered I had around 80kHz some weeks ago. LGTM then.


Investigate window width / scaling on high-res screens

If I fullscreen the browser, the catalog content only takes a third of the available width at 100% zoom. Minor thing obviously, but this much information becomes pretty dense that way.

This was never reproduced and neither re-reported. Closing until such an issue is brought up again.


restored fdroid build flavor

restored fdroid build flavor in gradle files and copied greatfire assets to fdroid folder.  also updated version name/code to reflect actual version.

looks ok to me, didn't get a chance to run it though


Info Popup does not auto-close

The above popup displays whenever deployment is triggered. What's annoying is it covers part of the editor area and it doesn't hide itself unless I manually click the close button. I wonder if there is a way to auto-hide the popup. If not possible, it would be OK if I can disable the popup in some settings.

@jwang-navarik
VSCode wants that you have to close popups manually. It does not support auto close.
If you do not want to show a popup after each deployment, you can set showPopupOnSuccess setting.


Sequence labelling tutorial

Thank you for the great job and the tensorflow. But I don't think the tutorial on LSTM is clearly enough. In addition, I suggest adding sequence labelling tutorial on LSTM tutorial for better explanation.

I'm also interested in this.

This request is from last here. Is it still open?

Automatically closing due to lack of activity.


Adding index on task_roles task_id foreign key

Index on foreign key for performance improvement.

Closing thus PR as it is a draft PR and raised long time back


Cannot connect to Oracle database since version 3.6.5

I updated DBeaver to version 3.6.5. After restarting my computer, I cannot connect to some Oracle databases. I got the following error: ORA-01017: invalid username/password.
Reversed my version back to 3.6.3 and I can connect without a problem.
Can you guys look into this :)
Thanks in advance.
Laurent

Can't reproduce.
Do you use SSH tunnel or any proxies? If so check  #373.
Otherwise - is there anything special in your environment? Maybe something is in error/debug logs? (https://github.com/serge-rider/dbeaver/wiki/Log-files)

I attached the error log. error.txt
I reinstalled 3.6.5 and no error this time, maybe a corrupt install. Thanks for replying this.


"Click into imported Resources" is linked to a wrong project

This issue happened in PyCharm Community Edition 4.5.4 with IntelliBot 0.6.7 and robot files are .txt
I have two projects with the same subfolders named resources/common.txt let's say
A/test.txt
A/resources/common.txt
B/test.txt
B/resources/common.txt

with test.txt on both A and B contain
*** Settings ***
Resource  resources/common.txt

After switching from project A to project B/test.txt CTRL+click resources/common.txt is linked to A/resources/common.txt instead of B's folder. clearing PYTHONPATH, restarting PyCharm, restarting PC do not cure the problem.
After I renamed folder A to something else the problem disappeared and cannot reproduce again.
This might be a bug, so I log this for further investigation.

+1

+1

+1
With AOL acquisition of Millenial Media and later by Verizon, Intellibot seems abandoned, last commit was almost 6 months ago...
That's sad because still is the best Robot plugin for IntelliJ.
I should start considering moving to VS Code instead in the near future.


Sample scene errors

Unity version:  Mac silicon 2022.3.4f1
I downloaded the project as a zip file and started the project without doing any other things

NullReferenceException: Object reference not set to an instance of an object
LlamAcademy.Guns.GunScriptableObject.GetRaycastOrigin () (at Assets/Scripts/Guns/GunScriptableObject.cs:309)
LlamAcademy.Guns.Demo.PlayerAction.UpdateCrosshair () (at Assets/Scripts/Guns/Demo/PlayerAction.cs:46)
LlamAcademy.Guns.Demo.PlayerAction.Update () (at Assets/Scripts/Guns/Demo/PlayerAction.cs:41)

NullReferenceException: Object reference not set to an instance of an object
LlamAcademy.Guns.GunScriptableObject.TryToShoot () (at Assets/Scripts/Guns/GunScriptableObject.cs:190)
LlamAcademy.Guns.GunScriptableObject.Tick (System.Boolean WantsToShoot) (at Assets/Scripts/Guns/GunScriptableObject.cs:125)
LlamAcademy.Guns.Demo.PlayerAction.Update () (at Assets/Scripts/Guns/Demo/PlayerAction.cs:26)

NullReferenceException: Object reference not set to an instance of an object
LlamAcademy.Guns.Demo.Editors.PlayerActionEditor.OnSceneGUI () (at Assets/Scripts/Guns/Demo/Editor/PlayerActionEditor.cs:28)
(wrapper dynamic-method) System.Object.lambda_method(System.Runtime.CompilerServices.Closure,UnityEditor.Editor)
UnityEditor.SceneView.CallOnSceneGUI () (at /Users/bokken/build/output/unity/unity/Editor/Mono/SceneView/SceneView.cs:3782)
UnityEditor.SceneView.HandleSelectionAndOnSceneGUI () (at /Users/bokken/build/output/unity/unity/Editor/Mono/SceneView/SceneView.cs:2817)
UnityEditor.SceneView.DoOnGUI () (at /Users/bokken/build/output/unity/unity/Editor/Mono/SceneView/SceneView.cs:2485)
UnityEditor.SceneView.OnSceneGUI () (at /Users/bokken/build/output/unity/unity/Editor/Mono/SceneView/SceneView.cs:2325)
UnityEngine.UIElements.IMGUIContainer.DoOnGUI (UnityEngine.Event evt, UnityEngine.Matrix4x4 parentTransform, UnityEngine.Rect clippingRect, System.Boolean isComputingLayout, UnityEngine.Rect layoutSize, System.Action onGUIHandler, System.Boolean canAffectFocus) (at /Users/bokken/build/output/unity/unity/ModuleOverrides/com.unity.ui/Core/IMGUIContainer.cs:355)
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr, Boolean&) (at /Users/bokken/build/output/unity/unity/Modules/IMGUI/GUIUtility.cs:190)
There are other errors there, just they are not so critical.

I’m confused Sent from my iPhoneOn Mar 11, 2024, at 12:09 PM, Kosty @.***> wrote:
Unity version:  Mac silicon 2022.3.4f1
I downloaded the project as a zip file and started the project without doing any other things
NullReferenceException: Object reference not set to an instance of an object
LlamAcademy.Guns.GunScriptableObject.GetRaycastOrigin () (at Assets/Scripts/Guns/GunScriptableObject.cs:309)
LlamAcademy.Guns.Demo.PlayerAction.UpdateCrosshair () (at Assets/Scripts/Guns/Demo/PlayerAction.cs:46)
LlamAcademy.Guns.Demo.PlayerAction.Update () (at Assets/Scripts/Guns/Demo/PlayerAction.cs:41)
NullReferenceException: Object reference not set to an instance of an object
LlamAcademy.Guns.GunScriptableObject.TryToShoot () (at Assets/Scripts/Guns/GunScriptableObject.cs:190)
LlamAcademy.Guns.GunScriptableObject.Tick (System.Boolean WantsToShoot) (at Assets/Scripts/Guns/GunScriptableObject.cs:125)
LlamAcademy.Guns.Demo.PlayerAction.Update () (at Assets/Scripts/Guns/Demo/PlayerAction.cs:26)
NullReferenceException: Object reference not set to an instance of an object
LlamAcademy.Guns.Demo.Editors.PlayerActionEditor.OnSceneGUI () (at Assets/Scripts/Guns/Demo/Editor/PlayerActionEditor.cs:28)
(wrapper dynamic-method) System.Object.lambda_method(System.Runtime.CompilerServices.Closure,UnityEditor.Editor)
UnityEditor.SceneView.CallOnSceneGUI () (at /Users/bokken/build/output/unity/unity/Editor/Mono/SceneView/SceneView.cs:3782)
UnityEditor.SceneView.HandleSelectionAndOnSceneGUI () (at /Users/bokken/build/output/unity/unity/Editor/Mono/SceneView/SceneView.cs:2817)
UnityEditor.SceneView.DoOnGUI () (at /Users/bokken/build/output/unity/unity/Editor/Mono/SceneView/SceneView.cs:2485)
UnityEditor.SceneView.OnSceneGUI () (at /Users/bokken/build/output/unity/unity/Editor/Mono/SceneView/SceneView.cs:2325)
UnityEngine.UIElements.IMGUIContainer.DoOnGUI (UnityEngine.Event evt, UnityEngine.Matrix4x4 parentTransform, UnityEngine.Rect clippingRect, System.Boolean isComputingLayout, UnityEngine.Rect layoutSize, System.Action onGUIHandler, System.Boolean canAffectFocus) (at /Users/bokken/build/output/unity/unity/ModuleOverrides/com.unity.ui/Core/IMGUIContainer.cs:355)
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr, Boolean&) (at /Users/bokken/build/output/unity/unity/Modules/IMGUI/GUIUtility.cs:190)
There are other errors there, just they are not so critical.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

My bad I didn’t mean to make this public. I’ll put the working version on GitHub soon Sent from my iPhoneOn Mar 11, 2024, at 12:09 PM, Kosty @.***> wrote:
Unity version:  Mac silicon 2022.3.4f1
I downloaded the project as a zip file and started the project without doing any other things
NullReferenceException: Object reference not set to an instance of an object
LlamAcademy.Guns.GunScriptableObject.GetRaycastOrigin () (at Assets/Scripts/Guns/GunScriptableObject.cs:309)
LlamAcademy.Guns.Demo.PlayerAction.UpdateCrosshair () (at Assets/Scripts/Guns/Demo/PlayerAction.cs:46)
LlamAcademy.Guns.Demo.PlayerAction.Update () (at Assets/Scripts/Guns/Demo/PlayerAction.cs:41)
NullReferenceException: Object reference not set to an instance of an object
LlamAcademy.Guns.GunScriptableObject.TryToShoot () (at Assets/Scripts/Guns/GunScriptableObject.cs:190)
LlamAcademy.Guns.GunScriptableObject.Tick (System.Boolean WantsToShoot) (at Assets/Scripts/Guns/GunScriptableObject.cs:125)
LlamAcademy.Guns.Demo.PlayerAction.Update () (at Assets/Scripts/Guns/Demo/PlayerAction.cs:26)
NullReferenceException: Object reference not set to an instance of an object
LlamAcademy.Guns.Demo.Editors.PlayerActionEditor.OnSceneGUI () (at Assets/Scripts/Guns/Demo/Editor/PlayerActionEditor.cs:28)
(wrapper dynamic-method) System.Object.lambda_method(System.Runtime.CompilerServices.Closure,UnityEditor.Editor)
UnityEditor.SceneView.CallOnSceneGUI () (at /Users/bokken/build/output/unity/unity/Editor/Mono/SceneView/SceneView.cs:3782)
UnityEditor.SceneView.HandleSelectionAndOnSceneGUI () (at /Users/bokken/build/output/unity/unity/Editor/Mono/SceneView/SceneView.cs:2817)
UnityEditor.SceneView.DoOnGUI () (at /Users/bokken/build/output/unity/unity/Editor/Mono/SceneView/SceneView.cs:2485)
UnityEditor.SceneView.OnSceneGUI () (at /Users/bokken/build/output/unity/unity/Editor/Mono/SceneView/SceneView.cs:2325)
UnityEngine.UIElements.IMGUIContainer.DoOnGUI (UnityEngine.Event evt, UnityEngine.Matrix4x4 parentTransform, UnityEngine.Rect clippingRect, System.Boolean isComputingLayout, UnityEngine.Rect layoutSize, System.Action onGUIHandler, System.Boolean canAffectFocus) (at /Users/bokken/build/output/unity/unity/ModuleOverrides/com.unity.ui/Core/IMGUIContainer.cs:355)
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr, Boolean&) (at /Users/bokken/build/output/unity/unity/Modules/IMGUI/GUIUtility.cs:190)
There are other errors there, just they are not so critical.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

Hi! Have you followed the setup instructions to ensure you have the Unity Particle Pack installed and have a Particle System at the tip of your gun model? That error is commonly reported when the Unity Particle Pack has not been imported and a Gun model does not have a Particle System as a child.
The rest of your errors all seem to also point to a null reference related to the particle system.
Check the Quickstart on the readme and let me know if you still encounter the issue from there!

Since this has not received any updates I will close this. Please reopen a new issue if the above did not help you resolve the issue.


fix: KeyEventResult issue

Exception caught by services library
The following TypeErrorImpl was thrown while processing the key message handler: Expected a value of type 'KeyEventResult', but got one of type 'bool'

Thanks! @crawlinknetworks Could you please review it and then merge if look good to you?


Issue with snackbar dismiss

FloatingToolbar.Behavior needs to account these use cases:

FAB click while there's a snackbar being shown
Snackbar dismissal

Fixed in commit https://github.com/rubensousa/FloatingToolbar/commit/f66de184f31af73262b69d4d013fe03770dbc6c2