BUG: Identical Player/Team Ids

PlayerId is supposed to be unique but this is causing some bugs where the playerId is returning the same id value as the team id. It becomes impossible to differentiate the players since the id is no longer unique. With both IGN's and player id no longer being unique to a player, it becomes impossible to be able to identify a player with accuracy.
cUrl: https://api.dc01.gamelockerapp.com/shards/na/matches?page[offset]=0&page[limit]=50&sort=-createdAt&filter[playerNames]=LaidToRest
{
            "type": "player",
            "id": "498d582c-f0a9-11e5-b465-06fc87f1dd11",
            "attributes": {
                "name": "LaidToRest",
                "shardId": "na",
                "stats": {
                    "elo_earned_season_4": 2405.8808509438663,
                    "elo_earned_season_5": 2386.2975294201347,
                    "elo_earned_season_6": 2624.162678557534,
                    "elo_earned_season_7": 2623.5272050435183,
                    "karmaLevel": 2,
                    "level": 30,
                    "lifetimeGold": 12943,
                    "lossStreak": 0,
                    "played": 4293,
                    "played_ranked": 2241,
                    "skillTier": 28,
                    "winStreak": 0,
                    "wins": 1577,
                    "xp": 175450
                },
                "titleId": "semc-vainglory"
            },
            "relationships": {
                "assets": {
                    "data": []
                }
            }
        }

{
            "type": "team",
            "id": "498d582c-f0a9-11e5-b465-06fc87f1dd11",
            "attributes": {
                "name": "NOVA",
                "shardId": "na",
                "stats": null,
                "titleId": ""
            },
            "relationships": {
                "assets": {
                    "data": []
                }
            }
        },

According to the JSON API spec, this is valid.

In a single document, you can think of the type and id as a composite key that uniquely references resource objects in another part of the document.

This cannot be "fixed" without rewriting (or duplicating) either team or player objects, which breaks current and future applications that cache that data. :-1:

@schneefux if it was a feature like that would it not be true for every account? For the majority 99% of accounts, this is not the case. However, a few seem to have a bug where their team id and player id are identical. If it was intended to be like this then it would have been true for all accounts.
For example:
cUrl: https://api.dc01.gamelockerapp.com/shards/na/matches?page[offset]=0&page[limit]=50&sort=-createdAt&filter[playerNames]=Levatikyn
This will give you a player and team id that is unique. For the majority of users, the id is unique as they should be but for some rare circumstances, it's not.

Yeah, this is definitely weird.  You're right, each object should have a globally unique guid, period.  It should identify a single object and no duplication should exist across types.
We'll find out if this is a problem with the service, or the game.  Strange.
@schneefux can you provide a specific matchID where this is an issue?  I notice that got omitted.
FWIW - ShardID and TitleID shouldn't be blank either.

I'm not sure that this is actually a bug, according to the JSON-API spec objects are uniquely identified by (id, type), id only has to be unique during creation of a type (that is, no players have the same id).
This seems like more of a failing on the developer side.


Separate relay process

Problem: if for any reason bosun is down, the data relay breaks for OpenTSDB. When bosun is back up, many alerts go unknown before the data can be restored. It is also not scalable, since all OpenTSDB data must go through a single host. The sole purpose of the relay is to index the metric and tag data for tag search.
Proposed solution:
Have a completely separate process that handles the relay to OpenTSDB and collects all needed data for tag search. Bosun gets a new API that is exposed to the web for pushing tag search data. The relay writes the tag search data to this API at some interval. This will allow the relay to continue while bosun is restarting or crashed, and still maintain the search database.
The existing relay code in bosun can be kept for those who do not want to run a separate relay process. This relay will be changed to write to the new search API listed above.

How will we support multiple relays? I assume the push apis will aggregate data as it comes in?

Each relay can push tag search data to the API on its own. The API is in charge of aggregation, not the relays.

While implementing this I ran into some problems. Currently our search service records the most recent 2 data points for each metric+tag combination, for use with opserver (http://bosun.org/api.html#apihost). Relays make this pretty complicated. I'm thinking about instead using a model where the relay has no idea about the data, it just listens on a port and sends data to as many locations as specified.


Fix the delete WorkflowTeam permission

Purpose
When a ProgramAdmin removes a member of their program, the member has to be removed from the program, in other words, the WorkflowTeam has to be deleted.
Related issue: TolaDataV2/#536

Coverage increased (+0.03%) to 52.979% when pulling 0937d42497de4623e91f70754832ec8caf895fed on jefmoura:fix-delete-wfteam-permission into b577dd78b117f056832efe04efcedcc8deb97b9e on toladata:dev-v2.


ContributionArea not correct

The Get-MVPContributionArea script return the following output:
PS C:\Users\raffaeler\documents\mvp> Get-MVPContributionArea

AwardName                                  ContributionArea
---------                                  ----------------
Visual Studio and Development Technologies {@{Id=aec301bb-189a-e411-93f2-9cb65495d3c4; Name=.NET; AwardName=Visual S...

I understood (but may be wrong) that I should set "Visual Studio and Development Technologies" as Contribution. But this is refused by the New-MVPContribution script.
This is what I see on the website:

In other words, the Visual Studio and Development Technologies is a top level category that has several children behind (which are the only ones admitted in the New-MVPContribution script.
It would be nice to have a different script retrieving the values under Visual Studio and Development Technologies (or the other categories that has a similar structure.
... and thanks for the great work!

Hi @raffaeler !
Yeah you need to specify the technology, not the category.
If you use New-MVPContribution, you'll see that it lists all the technologies. Here is the logic in the code: https://github.com/lazywinadmin/MVP/blob/master/MVP/Public/New-MVPContribution.ps1#L104

As you mentioned, by default Get-MVPContributionArea will retrieve your category, but it does not show the technologies directly. To do so, you can do something like:
# Retrieve the technologies of your category
(Get-MVPContributionArea).contributionarea
#or
Get-MVPContributionArea | Select -expand contributionarea

If you want to find all the technologies of all the categories, you can do something like this:
# Retrieve all technologies of all categories
Get-MVPContributionArea -all

New-MVPContribution is retrieving the technology object (with its ID, Name, etc), this is the way the API is working. See here: https://github.com/lazywinadmin/MVP/blob/master/MVP/Public/New-MVPContribution.ps1#L156-L160
Hope this help, let me know

Yep I'll be there 😄  Hope to see you too!


Implemented all the sub blocks in "set __ effect to __" block from Scratch.

Impressive work that goes quite deep into the inner workings of PyStage. I have to test it before I merge, though. One thing that comes to my mind from reading the code: is it correct to apply the effects after the rotozoom? What is Scratch doing. For something like a whirl, for example, I would expect that the effect is applied first so that the whirl rotates with the image.
Another question is, if the effects are applied too often. Is there a chance to cache the image with effects applied. It also looks like all effects are applied all the time. Again, need some tests, but maybe you can clarify some of my thoughts already.

I have already tested those functions, and you can test them too, some of effect doesn't behave same with Scratch, like changing color. The color is generated from value 0-200. I really don't know the process it generated in Scratch, and I didn't find the document for it.
And for whirl and fisheye and others, I just try my best to make it more Scratch-like, but not 100%.

And for repetetive processing the images, I thought about this before, but don't know how to solve it elegantly.
One of way I think out is to add a dict called last_effect_map to save the number of effects, and compare last_effect_map with the new values before calling process_image. If they are  same, then return last_image(pretend it is created).

I know the problem is there, what I thought is implement the feature first and optimize it later before think out a better solution.


Possibility to navigate different drives on Windows

Please describe the problem you're trying to solve
I'd like to switch drives from yazi itself, without tricks like zoxide or jumping back to the shell, changing directory, and then opening yazi
Would you be willing to contribute this feature?

[ ] Yes, I'll give it a shot

Describe the solution you'd like
When you go to the root folder, it should be possible to switch to a different drive letter
Additional context
No response

You can either:

Click g<Space> to open the interactive cd, then type D:\ to navigate
Bind a key to navigate like this:{ on = [ "g", "d" ], run = "cd 'D:\\'" }

With the nightly build, you can use D: instead of D:\ / D:\\ for convenience. Closing as already supported.


#5614 Deprecate ArrayDocumentType & SetDocumentType

Signed-off-by: Nigel Jones<EMAIL_ADDRESS>

Description

Deprecates ArrayDocumentType & SetDocumentType
Migrates type patches to a 3.1 archive, and provides clean base for 3.2

The deprecations were intended to occur alongside ArraySchemaType (see 838847cf8f68730aaa52c946b748691168084ccd )
See #5614
How Has This Been Tested?
Build/FVT
Check definition of ArrayDocumentType & ArraySchemaType using postman (types/by-name)
Check REX browsing works both for non-deprecated, and including deprecated types
Any additional notes for reviewers?
The first two commits (deprecations) are also added to 3.1 release.

I've not changed the top level link so I think this should be good...


Release v0.1.6

Coverage remained the same at 85.579% when pulling f2f93bcfa3c07bb0bb415441e1cd3544ec21e084 on ajkannan:release-0.1.6 into 59c3215b5a0f7b7fb8e35392073dbc7ad5292bc6 on GoogleCloudPlatform:master.


NeoBundleUpdate from shell command

I use a command to update bundles without starting vim: vim --cmd "set t_ti= t_te=" +NeoBundleUpdate +q, but since commit 41b2c06 the behavior seems to have changed, and the one prior 9e5c19a still seems to be working.
Before, I would see something like this:
[neobundle] Update started: (2016/01/03 16:41:42)
[neobundle] ( 1/29) [                    ] vim-rspec
[neobundle] ( 2/29) [=                   ] vim-phpqa
[neobundle] ( 3/29) [==                  ] python-mode
(and so on)

without the actual vim interface rendering, but now it does that. I found it more convenient the other way.
Was that intended?

Fixed.

Sweet, thank you!


FMCS fix for Windows DLLs

Fixed a bug which prevented FMCS to work correctly on Windows when linking to DLLs
The fix is along the lines of the one done in https://github.com/rdkit/rdkit/pull/960. I have moved the switch into FMCS.cpp such that the correct functor address is put into MCSParameters if dynamic linking is used under Windows. This wasn't the case when the switch was into rdFMCS.cpp, which caused the FMCS Python test to fail.
At first I hadn't noticed this failure as so far I have only built DLLs under MinGW, where a number of Python tests failed due to the dreaded boost 1.60 bug. During the weekend I found the time to patch the MinGW boost packages, and only this failure was left, so I could nail it down.

Travis/Python 2.7 failed downloading AvalonToolkit_1.2.0.source.tar, otherwise it would be OK (it is for me).

Apologies for being thick, but I don't see why this should be necessary. Though the atom and bond comparison functions (incorrectly called functors in the header file) are declared in the header file, they are only defined in FMCS.cpp, so it seems like there shouldn't be a problem across linkage units.
What error are you seeing on Windows (and how do you trigger it)?

I am going to show you what happens in a few minutes.

can do that on slack if it makes things easier

I added the following debugging lines at the beginning of setMCSFunctionPtrs():
  std::cerr << "libFMCS.dll setMCSFunctionPtrs() MCSAtomCompareAny = " << (void *)MCSAtomCompareAny << std::endl;
  std::cerr << "libFMCS.dll setMCSFunctionPtrs() MCSAtomCompareIsotopes = " << (void *)MCSAtomCompareIsotopes << std::endl;
  std::cerr << "libFMCS.dll setMCSFunctionPtrs() MCSAtomCompareElements = " << (void *)MCSAtomCompareElements << std::endl;

and the following at the beginning of FindMCS():
  std::cerr << "libFMCS.dll findMCS() MCSAtomCompareAny = " << (void *)MCSAtomCompareAny << std::endl;
  std::cerr << "libFMCS.dll findMCS() MCSAtomCompareIsotopes = " << (void *)MCSAtomCompareIsotopes << std::endl;
  std::cerr << "libFMCS.dll findMCS() MCSAtomCompareElements = " << (void *)MCSAtomCompareElements << std::endl;

Then I compiled and ran pyFMCS:
ctest -I 81,81 -V
UpdateCTestConfiguration  from :C:/build/rdkit/build_mingw/DartConfiguration.tcl
UpdateCTestConfiguration  from :C:/build/rdkit/build_mingw/DartConfiguration.tcl
Test project C:/build/rdkit/build_mingw
Constructing a list of tests
Done constructing a list of tests
Checking test dependency graph...
Checking test dependency graph end
test 81
    Start 81: pyFMCS

81: Test command: C:\msys64\mingw64\bin\python.exe "C:/build/rdkit/Code/GraphMol/FMCS/Wrap/testFMCS.py"
81: Test timeout computed to be: 9.99988e+006
81: C:/msys64/mingw64/lib/python2.7/site-packages/numpy/core/__init__.py:14: Warning: Numpy built with MINGW-W64 on Windows 64 bits is experimental, and only available for
81: testing. You are advised not to use it for production.
81:
81: CRASHES ARE TO BE EXPECTED - PLEASE REPORT THEM TO NUMPY DEVELOPERS
81:   from . import multiarray
81: libFMCS.dll setMCSFunctionPtrs() MCSAtomCompareAny = 0x6e6c1990
81: libFMCS.dll setMCSFunctionPtrs() MCSAtomCompareIsotopes = 0x6e6c1a00
81: libFMCS.dll setMCSFunctionPtrs() MCSAtomCompareElements = 0x6e6c15b0
81: libFMCS.dll findMCS() MCSAtomCompareAny = 0x6e6c1990
81: libFMCS.dll findMCS() MCSAtomCompareIsotopes = 0x6e6c1a00
81: libFMCS.dll findMCS() MCSAtomCompareElements = 0x6e6c15b0
[...]
81: .
81: ----------------------------------------------------------------------
81: Ran 7 tests in 1.194s
81:
81: OK
1/1 Test #81: pyFMCS ...........................   Passed    4.21 sec

100% tests passed, 0 tests failed out of 1

Total Test time (real) =   4.37 sec

All good. This is what happens when I move setMCSFunctionPtrs() into rdFMCS.cpp (where it originally was) and I update the debugging info to reflect the new location:
  std::cerr << "rdFMCS.pyd setMCSFunctionPtrs() MCSAtomCompareAny = " << (void *)MCSAtomCompareAny << std::endl;
  std::cerr << "rdFMCS.pyd setMCSFunctionPtrs() MCSAtomCompareIsotopes = " << (void *)MCSAtomCompareIsotopes << std::endl;
  std::cerr << "rdFMCS.pyd setMCSFunctionPtrs() MCSAtomCompareElements = " << (void *)MCSAtomCompareElements << std::endl;

Then I re-compiled and ran pyFMCS:
ctest -I 81,81 -V
UpdateCTestConfiguration  from :C:/build/rdkit/build_mingw/DartConfiguration.tcl
UpdateCTestConfiguration  from :C:/build/rdkit/build_mingw/DartConfiguration.tcl
Test project C:/build/rdkit/build_mingw
Constructing a list of tests
Done constructing a list of tests
Checking test dependency graph...
Checking test dependency graph end
test 81
    Start 81: pyFMCS

81: Test command: C:\msys64\mingw64\bin\python.exe "C:/build/rdkit/Code/GraphMol/FMCS/Wrap/testFMCS.py"
81: Test timeout computed to be: 9.99988e+006
81: C:/msys64/mingw64/lib/python2.7/site-packages/numpy/core/__init__.py:14: Warning: Numpy built with MINGW-W64 on Windows 64 bits is experimental, and only available for
81: testing. You are advised not to use it for production.
81:
81: CRASHES ARE TO BE EXPECTED - PLEASE REPORT THEM TO NUMPY DEVELOPERS
81:   from . import multiarray
81: rdFMCS.pyd setMCSFunctionPtrs() MCSAtomCompareAny = 0x681866f0
81: rdFMCS.pyd setMCSFunctionPtrs() MCSAtomCompareIsotopes = 0x681866d0
81: rdFMCS.pyd setMCSFunctionPtrs() MCSAtomCompareElements = 0x681866d8
81: libFMCS.dll findMCS() MCSAtomCompareAny = 0x6e6c1990
81: libFMCS.dll findMCS() MCSAtomCompareIsotopes = 0x6e6c1a00
81: libFMCS.dll findMCS() MCSAtomCompareElements = 0x6e6c15b0
[...]
81: .
81: ======================================================================
81: FAIL: test3IsotopeMatch (__main__.TestCase)
81: ----------------------------------------------------------------------
81: Traceback (most recent call last):
81:   File "C:/build/rdkit/Code/GraphMol/FMCS/Wrap/testFMCS.py", line 104, in test3IsotopeMatch
81:     self.failIf(Chem.MolFromSmiles('CC[13CH3]').HasSubstructMatch(qm))
81: AssertionError: True is not false
81:
81: ----------------------------------------------------------------------
81: Ran 7 tests in 0.831s
81:
81: FAILED (failures=1)
1/1 Test #81: pyFMCS ...........................***Failed    2.45 sec

0% tests passed, 1 tests failed out of 1

Total Test time (real) =   2.59 sec

The following tests FAILED:
         81 - pyFMCS (Failed)
Errors while running CTest

This shows that rdFMCS.pyd doesn't know the correct entry points for the functions, so it cannot properly fill in those values into the MCSParameters struct, while moving setMCSFunctionPtrs() in the same DLL that will actually call those functions solves the problem.
I hope this makes it clear which the problem is, which is basically the same that I had faced in #960 . Please let me know if this is still a bit obscure.

I think I misunderstood something in an earlier comment: this is a mingw build?

Regardless, unless I've really badly misunderstood something (certainly always possible), if everything is dynamically linked, you really should only have one copy of MCSAtomCompareAny (as well as the others). Those function pointers really should all be the same.
Before introducing this change, which seems to add un-necessary code, I'd like to understand what is happening.
Can you use dependency walker or something similar to see where the extra definition(s) is/are coming from?

Yes, it is a MinGW build. But I think the same would happen with MSVC if I built DLLs rather than static libraries.
I am quoting Brian here from #960 :

If you want to see the details of what is going on, take a deep breath and read this:
https://www.akkadia.org/drepper/dsohowto.pdf
If you like beer, have a lot of it ready :)

In that case I used a workaround to fix the problem. Here the simple solution is to set the function addresses in the same DLL which hosts the functions, so it is guaranteed that when libFMCS.dll is loaded at run time, MCSParameters is filled with the correct addresses.
I think the problem is that the memory address of those functions is defined only when the libFMCS,dll is loaded at run time. When rdFMCS.pyd is first loaded, libFMCS,dll hasn't been loaded yet, so the address that rdFMCS.pyd returns is bogus. This is, at least, my understanding of what is going on. I should go deeper into that document to fully understand, but I don't have the time.

That document seems to be focused on linux, so I'm not sure how much help it is.
I will do some exploration with MSVC builds this evening/tomorrow morning to see if I can figure out what's going on and find a fix that feels like less of a hack. I'll probably fail, but I need to at least try. :-S

I don't think it's a hack. It is just a way to force the libFMCS.dll DLL to be loaded before filling in the function pointer values into MCSParameters. The libFMCS.dll DLL is loaded anyway by rdFMCS.cpp just a few lines of code onwards, when FindMCS() is called.

Well, if you could just forget all the rant I made above... I have now removed the hack and submitted the proper fix. What can I say? Thanks for resisting to the hack, thus prompting a search for a proper fix :-)

oh, I had completely forgotten about the joy that is __declspec. (It is fun to say though)

There's one thing I don't particularly like: there is no way to define BOOST_ALL_DYN_LINK from the cmake command line. What about adding a -DBOOST_DYNAMIC_LINK  cmake ON/OFF switch which allows to add such a definition? Currently the only way is to add a definition into CMakeLists.txt.
If you are positive I'll make a separate PR for that.

Thanks for this.
Let's investigate the BOOST_ALL_DYN_LINK thing separately.


Reflexive cascading delete for shared+plural association (same model, many to many, association table record not deleted after model destroy)

Sails version: 0.12.1
Node version: v4.3.1
NPM version: 2.14.12
Operating system: ubuntu 14.04.2 (docker)

Model:
module.exports = {

  attributes: {
    name: {
      type: 'string',
      required: true
    },

    fans: {
      collection: 'user',
      via: 'followedUsers'
    },

    followedUsers: {
      collection: 'user',
      via: 'fans'
    }
  }
};

step1: create Bob
sails> User.create({name:'Bob'}).exec(console.log);
undefined
sails> null { name: 'Bob',
  createdAt: '2016-12-15T07:07:08.957Z',
  updatedAt: '2016-12-15T07:07:08.957Z',
  id: 1 }

step2: create Alice
sails> User.create({name:'Alice', followedUsers:1}).exec(console.log);
undefined
sails> null { name: 'Alice',
  createdAt: '2016-12-15T07:07:16.000Z',
  updatedAt: '2016-12-15T07:07:16.000Z',
  id: 2 }

step3: query association table
mysql> select * from user_followedusers__user_fans;
+----+-----------+--------------------+
| id | user_fans | user_followedUsers |
+----+-----------+--------------------+
|  1 |         1 |                  2 |
+----+-----------+--------------------+

step4: destroy Bob
sails> User.destroy({id:1}).exec(console.log);
undefined
sails> null [ RowDataPacket {
    name: 'Bob',
    id: 1,
    createdAt: Thu Dec 15 2016 07:07:08 GMT+0000 (UTC),
    updatedAt: Thu Dec 15 2016 07:07:08 GMT+0000 (UTC) } ]

step5: query association table
mysql> select * from user_followedusers__user_fans;
+----+-----------+--------------------+
| id | user_fans | user_followedUsers |
+----+-----------+--------------------+
|  1 |         1 |                  2 |
+----+-----------+--------------------+

the record still exists

ok, fixed!

ok, fixed!

@kumakichi thanks for the report!  @particlebanana  and I are looking into it.

For future reference, see https://github.com/balderdashy/waterline/blob/8e69cf2a6ada4dc2c4302690f94642f9cfc2c2da/lib/waterline/query/dql/destroy.js#L63-L136

@kumakichi Update: we confirmed this locally and are working on a fix.

@kumakichi I just pushed up a change if you could give it a test in your app and let me know what the outcome is. I published it under the 0.11-pre tag on NPM to make sure I figured out all the cases before a publish.
npm install<EMAIL_ADDRESS>

Hey @kumakichi, have you had a chance to try that out yet in your app?
@sgress454's PR here may also have been related https://github.com/balderdashy/waterline-schema/pull/65  (not sure if that needs a backport)
@particlebanana Just checking: did you get your fix onto v1.0?

Hey @kumakichi -- we were able to verify that @particlebanana's patch works, and this is now fixed in a new prerelease on npm (v0.12.12-1).  To try that out, run npm install<EMAIL_ADDRESS>in your app.
We'll publish that patch for everyone on NPM tomorrow.  But in the mean time, would you let me know if the prerelease solves your issue?  Thanks!

Sorry for late response, i'm on holiday the past days
I just tested 0.12-pre,it works,thank you

@kumakichi no problem, glad to hear it worked.  Hope you had a good holiday!


Added checkstyle analysis

PR for #80

Coverage remained the same at 78.313% when pulling b1de1fb2661c31a96568b9613622b47b07975a6e on 80 into d19c4fcc3787532470a47d7336afb321394e2232 on master.

@rultor merge

@rultor merge

@amihaiemil OK, I'll try to merge now. You can check the progress of the merge here

Coverage remained the same at 78.313% when pulling b1de1fb2661c31a96568b9613622b47b07975a6e on 80 into d19c4fcc3787532470a47d7336afb321394e2232 on master.

@rultor merge

@amihaiemil Done! FYI, the full log is here (took me 4min)


Unable to get managed stack trace on Crashlytics

Hi,
I created a sample project on VS4M, using one of the available templates (Multiplatform App-> Native (iOS, Android)).
When using Crashlytics on the iOS project and simulating a crash, all I can see on Firebase's dashboard is something along these lines
FirebaseTest.iOS
registrar.m - Line 62
native_to_managed_trampoline_2(objc_object*, objc_selector*, _MonoMethod**, unsigned int)

On the Build Output I can see that the dSYM is being correctly uploaded
Target Build:
Target _FirebaseCrashlyticsUploadDSymToFirebaseOnMac:
  Task "Message"
    Uploading dSYM to Firebase using Fabric upload-symbols script located at /Users/gaston/Library/Caches/XamarinBuildDownload/Fbrc-1.7.13
  Done executing task "Message".
  Using "Exec" task from assembly "/Library/Frameworks/Mono.framework/External/xbuild/Xamarin/iOS/Xamarin.iOS.Tasks.dll".
  Task "Exec"
    '/Users/gaston/Library/Caches/XamarinBuildDownload/Fbrc-1.7.13/upload-symbols' -gsp '/Users/gaston/Projects/FirebaseTest/iOS/GoogleService-Info.plist' -p ios 'bin/iPhone/Debug/device-builds/iphone10.2-11.4.1/FirebaseTest.iOS.app.dSYM'
    2018-12-03 13:00:18.116 upload-symbols[24784:5138325] Successfully submitted symbols for architecture arm64 with UUID 332a2db2af6e3a3f8034f7ad9dafcdb2 in dSYM: /Users/gaston/Projects/FirebaseTest/iOS/bin/iPhone/Debug/device-builds/iphone10.2-11.4.1/FirebaseTest.iOS.app.dSYM
  Done executing task "Exec".
Target _FirebaseCrashlyticsUploadDSymToFirebaseMasterTarget:

Done building project "FirebaseTest.iOS.csproj"

I've also manually generated a .zip file with all the dSYM found on bin/iPhone/Debug/device-builds/iphone10.2-11.4.1 and uploaded through the dashboard, to no avail.
Am I missing something?

@gnola14 If Firebase Console does not ask you for a certain dSYM, it means that it was successfully uploaded. Can you share the full stack error of what you see on Crashlytics Dashboard on Firebase Console? Thanks in advance.

Sure @SotoiGhost, I've attached the full crash log. The Dashboard does mention some "Optional" dSYM's that are missing, but none of the UUID's match with the one of my app
crash_log.txt

@gnola14 Is the crash a C# exception or it's happening on Objective-C side? Also, I tested our sample and it seems to be working correctly. Can you provide a test case, please?

@SotoiGhost it's an unhandled C# exception, being generated on the shared project (I hard-coded throwing an ArgumentException on one of the screens). I'll give it another shot with a different sample project to see if I still have the same issue. I'll get back to you with a test case.

@gnola14 I think that Firebase does not work properly with C# exceptions as the tool was thought to symbolicate iOS side exceptions, not .Net exceptions. I'm not a dSYM expert, to be honest, so, yeah, let's give a try to see if we can achieve that Firebase can symbolicate C# exceptions. I'll wait for a test case. Thanks in advance!

@SotoiGhost Sorry if I didn't come back with a test case. I believe you're right, in that Crashlytics won't be able to provide more information about a C# exception. However, I found the method recordCustomExceptionName. Far from a perfect solution, but it might do the trick. Thanks for your help!

@gnola14 That's a good way to go, not the best due to the performance but, according to the docs, that's the way to go. Closing this issue then. If you face something else related to this, please, open a new issue.
Happy coding!
Best!


14.8 support? 🙏 (bluetoothd attached)

This tweak is exactly what I was looking for but I see 14.8 isn’t currently supported :(
I'm attaching my bluetoothd.. I had to add file extension to get it to upload but just remove it
bluetoothd.md
Thank you!!!

bluetoothd.zip
Here’s the zip if you prefer

Done, and thanks for binary.

Any idea when this will get released to or by bigboss? Or should I try compiling it myself again.. I wasn’t able to compile without errors on my previous attempts

Could be in a week.

I was able to compile the tweak now and install it, my airpods are showing now as connected (and appears in accessibility) but the popup and volume control aren't working. I think the code is treating my device like it's rootless but it's actually rootful? When I compiled it gave some error about not finding rootful directory so not sure what that means.

I was able to compile the tweak now and install it, my airpods are showing up (and appears in accessibility) but the popup and volume control aren't working.
Also when I close the case the battery widget switches to showing generic headphones.

For volume control you would have to install rocketbootstrap and respring, not really sure why it wasn't working as it works in my device. But it's going to be reimplemented, either in the correct way (may require extra binaries so it's kind of hard) or just by a socket, as IPC doesn't work for iOS 15.
The popup was implemented in a buggy way as well, hopefully it would be reimplemented soon, if possible.

Thanks I'll check it out soon! Fyi, I noticed that tweak isn't getting injected into sharingd for some reason so maybe had something to do with volume issue? I killed the daemon but it restarts without the module loaded in

After installing new tweak and restarting phone the popup works perfectly! Except volume control still doesn't and then after re-pairing the airpods it no longer shows up as airpods icons and doesn't appear in accessibility menu haha.. seems popup was fixed but everything else stopped working. I'll try messing with it some more.

What's your iOS version, is it exactly iOS 14.8?
I think it could be caused by the strict version check in my code.

After going back to old version forgetting, changing name of airpods re-pairing etc., and re-installing newest version of tweak all the old stuff works again, plus the pop-up, but still no volume changing. I think the issue above was caused by accessory model somehow getting changed to "accessory_something" instead of "airpod Pros".. but now it's back to normal.
My version just says 14.8 (18H17)

I do get this error several times when I compile is this any problem?

That's fine, ignore it.

After going back to old version forgetting, changing name of airpods re-pairing etc., and re-installing newest version of tweak all the old stuff works again, plus the pop-up, but still no volume changing. I think the issue above was caused by accessory model somehow getting changed to "accessory_something" instead of "airpod Pros" when I paired on the new version, but now it's back to normal. However it might only be fixed because I paired it while on older version.
My version just says 14.8 (18H17)
fyi spatial tracking also doesn't seem to track anywhere just stays virtually in one spot

Have you tried that one in the Releases?

Have you tried that one in the Releases?
No I didn't see that I'll try it now

Unfortunately no difference
Here’s log of me swiping airpod volume gesture one time (down)

Also seeing it from mediaserver

I'm not an expert but as far as I can tell, bluetoothd instructs mediaserverd to change volume (YES).. mediaserver rejects the request, bluetoothd then switches sendvolume to NO

No it's fine, the problem is on the kBTAudioMsgPropertySupportRemoteVolumeChange=NO property being sent right when the earbuds connect.

Yeah I see that, fyi the new latest edits don't seem to fix it in case you didn't already know. Possible to directly override mediaserver from ignoring the data? I guess you're not sure why it's being set to No value.

It's kind of an indirect way, and requires hooking on extra binaries.
Would try that if no other workable way found.

Yeah direct wasn't the right word I realize that's a bit of a hacky method I just meant it as a last resort I guess thanks for response.

I found the cause, probably:
I forgot to initialize the variable len!
char exec_path[512];
uint32_t len;
_NSGetExecutablePath(exec_path, &len);

Which end up with a zero-length and make the path comparison fails occasionally

Have a try on the latest one.

It works!! It only works while music/media is playing which I assume is normal? But this is awesome, thanks so much!
I'm not sure that optimized charging function is working as far as I can tell, but I'll have to use it more.

It works!! It only works while music/media is playing which I assume is normal? But this is awesome, thanks so much!
I'm not sure that optimized charging function is working as far as I can tell, but I'll have to use it more.

Volume works for me regardless of if any media is playing.

hm, yeah it doesn't work for me unless I have something playing.. that's why it didn't seem to be working at first. I'll restart phone though

hm, yeah it doesn't work for me unless I have something playing.. that's why it didn't seem to be working at first. I'll restart phone though

Well, that's better than it not working at all in calls like it does for me lol.

Well, that's better than it not working at all in calls like it does for me lol.

Oh I thought you were LNSSPsd oops, but yeah at least it works with music. Does optimized charging work for you? I'm pretty sure mine isn't because it seems to always goto 100% charge which will degrade battery quicker so that sucks

Well, that's better than it not working at all in calls like it does for me lol.

Oh I thought you were LNSSPsd oops, but yeah at least it works with music. Does optimized charging work for you? I'm pretty sure mine isn't because it seems to always goto 100% charge which will degrade battery quicker so that sucks

I won't even care about mine's battery lol

Well, that's better than it not working at all in calls like it does for me lol.

Oh I thought you were LNSSPsd oops, but yeah at least it works with music. Does optimized charging work for you? I'm pretty sure mine isn't because it seems to always goto 100% charge which will degrade battery quicker so that sucks

Haha all good. I can't tell if optimized charging works and I honestly don't even know what to look for.

Well, that's better than it not working at all in calls like it does for me lol.

Oh I thought you were LNSSPsd oops, but yeah at least it works with music. Does optimized charging work for you? I'm pretty sure mine isn't because it seems to always goto 100% charge which will degrade battery quicker so that sucks

I won't even care about mine's battery lol

AppleCare+ for AirPods is a super good deal so I got it for that exact reason. I'll just get a replacement in about a year if I'm having issues. I had my entire AirPods Pro 1st gen replaced right before the warranty was up. I was having an issue with the charging case not charging the AirPods and every time I went they kept telling me I had to replace the AirPods not the case so they replaced the buds about 3-5 times before they gave me a new case lol. So in the end I had brand new AirPods and a brand new case 2 years later.

I can't tell if optimized charging works and I honestly don't even know what to look for.

It learns how you use the airpods and waits to charge fully to avoid battery wear. On my 1st gen pros it will typically goto 80% and the popup display will show that it's in optimized charging mode and presents a button to make it charge fully if you want.
Usually don't need it fully charged it helps give it a longer lifespan. Airpod batteries aren't replaceable. I see the option in settings and have it enabled but don't think it's working on 2nd gen.

I can't tell if optimized charging works and I honestly don't even know what to look for.

It learns how you use the airpods and waits to charge fully to avoid battery wear. On my 1st gen pros it will typically goto 80% and the popup display will show that it's in optimized charging mode and presents a button to make it charge fully if you want.
Usually don't need it fully charged it helps give it a longer lifespan. Airpod batteries aren't replaceable. I see the option in settings and have it enabled but don't think it's working on 2nd gen.

Oh yea I'd want that disabled either way. My phone does that and it's always when I'm like 20% away from 100% and I'm trying to get it to 100 before I go to sleep and the disable button only works like 1/2 the time. But I'll let you know when I plug them in tonight if I get a pop up like that.

I restarted phone and still doesn't change volume unless media is playing 🤷♂

I guess I mainly would use it to change volume while media is playing anyway so it's not a big deal, I'll see if it works during a call though


[Feature]: Keycard support

Is your request related to a problem?
As a Keycard user I can't connect to the web3 applications this library is used.
Feature Description
Implement support for the Keycard wallet.
Alternative Solutions
No response
Anything else?
I was initially redirected by the Gnosis team: https://github.com/safe-global/web-core/issues/1315

cc @bitgamma @gravityblast @guylouis for visibility.

@willianpaixao thanks for the feature request! Most of our wallet support packages are community driven at this point. If you or a member of the Keycard team would like to create a PR to extend support we would be happy to provide full assistance in the process. If this isn't an option we can create a card in the backlog but it could be a while before we get to it. Much faster route would be through PR.
P.S. Dig the layout of your GH profile page 🔥

@guylouis thank you for chiming in! Keycard looks super interesting and innovative! When your team is ready to integrate with Web3-Onboard please let us know and we can assist. Thank you! Closing issue for now.


Ask for nearby if photo has location in Exif, and possible nearby photo

Summary:
This issue is request of one of our user, Luistxo Fernandez:

When picture location is in the Exif, prompt dialogue: 'Do you wish to add this image to some nearby wikidata item'? showing a list...

This can be very beneficial indeed. Ie. a user doesn't know how to use nearby feature, but uploading a photo from nearby list. We can directly ask to edit wikidata item too.

Not a list, but we now suggest one Wikidata item if it is close enough. 🙂


fix - GetSolutionFolderProjects fail to detect recursive solution folders

Sub solution folder's Kind is not EnvDTE80.ProjectKinds.vsProjectKindSolutionFolder, so GetSolutionFolderProjects  fail to detect files in it

@pkradioman fantastic! thanks!


[SDK] test: add unit test for get_job_logs method of the training_client

What this PR does / why we need it:
New unit test for get_job_logs function
To verify run:
pytest -v sdk/python/kubeflow/training/api/training_client_test.py -k test_get_job_logs

Which issue(s) this PR fixes (optional, in Fixes #<issue number>, #<issue number>, ... format, will close the issue(s) when PR gets merged):
Ref: #2161
Checklist:

[ ] Docs included if any changes are user facing

Hi @andreyvelich, please help review it when you are available. Thank you!

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

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

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
66

Relevant Lines:
66

💛  - Coveralls

@seanlaii Also, please rebase your PR so we can merge it.

@seanlaii Also, please rebase your PR so we can merge it.

Rebased. Thanks for reviewing the PR.


Debian Package Issues

Hello Graham,
It seems like Debian Jessie is a bit outdated now and I can't apt-get any packages with your docker image because it requires me to do an apt-get update and they don't have some of the packages for Jessie hosted anymore. Is there a way I can work around that? I am needing to unzip client certificates to be used with apache and unzip is apparently not included in the debian version you used for these docker containers.

This project is effectively dead, just not officially. Suggest you read an comment on https://github.com/GrahamDumpleton/mod_wsgi-docker/issues/34


BOCM-215 exchange of valid and incomplete reasons

Describe your changes

Issue ticket number and link
BOCM-215
Type of change 🧩

[ √] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ √] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ √] Documentation update
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[√ ] I have performed a self-review of my own code
[ √] I have double checked this work does not include any hardcoded secrets or passwords
[√ ] I have made corresponding changes to the documentation
[√ ] I have provided details on how I have tested my code
[ √] I have referenced the ticket number above
[ v] I have provided a description of the ticket
[√ ] I have included unit tests to cover any testable code changes

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


sqlc Forgets the Column Name When Functions Applied on Both Sides

Correct Behavior
The following query:
-- name: selectUserByUsername :one
SELECT
	*
FROM
	users
WHERE
	LOWER(username) = $1 AND age = $2
LIMIT 1;

Generates this Go code:
type selectUserByUsernameParams struct {
	Username string
	Age      sql.NullInt32
}

Unexpected Behavior
Note how here I applied the LOWER() function on both sides of the comparison:
-- name: selectUserByUsername :one
SELECT
	*
FROM
	users
WHERE
	LOWER(username) = LOWER($1) AND age = $2
LIMIT 1;

The generated code ends up having Lower as the parameter name rather than Username, which is the column name:
type selectUserByUsernameParams struct {
	Lower string
	Age   sql.NullInt32
}

The AND age = $2 is just there to force the generation of the struct. Eliminating the AND age = $2 will still name the argument/parameter lower rather than username. I think the struct makes it more obvious to catch.
Playground
https://play.sqlc.dev/p/3d035e555d980f57e8a93672f9a1aec4a25fbcce02edff8946a528f3fc989b92

Dupe of https://github.com/kyleconroy/sqlc/issues/229, I think.

Kinda but not exactly. It's expected if it were part of the selected columns, because SQL engines end up naming the column with that. It's unexpected when it's one of the comparison parameters of WHERE. It generates the correct field name if I omit the use of LOWER on the right side.

Not sure if this is the same issue but:
-- name: GetUserByEmailAndPassword :one
SELECT
  "email",
  "updated_at",
  "created_at",
  "id"
FROM
  "users"
WHERE
  "email" = $1
  AND "password" = crypt($2, "password")
LIMIT 1;

results in:
type GetUserByEmailAndPasswordParams struct {
	Email string `json:"email"`
	Crypt string `json:"crypt"`
}

but perhaps should be:
type GetUserByEmailAndPasswordParams struct {
	Email string `json:"email"`
	Password string `json:"password"`
}

This has been fixed
https://play.sqlc.dev/p/8b270e64462166a51326f9c26712b83cc39b5e69c67617b6d14185604ab33e07

This has been fixed
https://play.sqlc.dev/p/8b270e64462166a51326f9c26712b83cc39b5e69c67617b6d14185604ab33e07

Is the field name becoming Lower expected behavior?
https://play.sqlc.dev/p/3d035e555d980f57e8a93672f9a1aec4a25fbcce02edff8946a528f3fc989b92

Whoops, I checked the first case instead of the second case. Reopened

This happens when LOWER is used on the values side of an INSERT statement as well. The InsertUserParams struct name is "Lower" instead of "Username".
https://play.sqlc.dev/p/97df49dbc642f1d1c1a3e91c7594ea7da9e5ded863855db3af48b55d554b2e73

I was able to get around with by using sqlc's named parameters. My problem was when I had a query like
-- name: GetUserByFirstAndLastName :many SELECT * FROM users WHERE LOWER(first_name) LIKE LOWER($1) AND LOWER(last_name) LIKE LOWER($2);
the generated result was
type GetUserByFirstAndLastNameParams struct { 	Lower   string 	Lower_2 string }
To get around this, I used named parameters:
-- name: GetUserByFirstAndLastName :many SELECT * FROM users WHERE LOWER(first_name) LIKE LOWER(@first_name::text) AND LOWER(last_name) LIKE LOWER(@last_name::text);
which resulted in
type GetUserByFirstAndLastNameParams struct { 	FirstName string 	LastName  string }
Hope this helps.


feat: Rill cloud E2E setup

[x] Get a working docker compose for rill cloud
[x] Figure out auth (adding a container that has barebones oauth)
[ ] Figure out github. We can perhaps skip app installation steps and start a local git server.
[x] Figure out a good way to distribute .env (Will only contain keys for the dockerised components)
[ ] Add a github action to build UI and start cloud

We will be using dev env for running e2e


Fails to build for target s390x-unknown-linux-musl

[361]error[E0277]: the trait bound `u64: From<i32>` is not satisfied
[362](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L362)  --> /home/buildozer/.cargo/registry/src/github.com-eae4ba8cbf2ce1c7/terminal_size-0.1.17/src/unix.rs:31:23
[363](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L363)   |
[364](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L364)31 |     if unsafe { ioctl(fd, TIOCGWINSZ.into(), &mut winsize) } == -1 {
[365](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L365)   |                 ----- ^^ the trait `From<i32>` is not implemented for `u64`
[366](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L366)   |                 |
[367](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L367)   |                 required by a bound introduced by this call
[368](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L368)   |
[369](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L369)   = help: the following other types implement trait `From<T>`:
[370](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L370)             <f32 as From<i16>>
[371](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L371)             <f32 as From<i8>>
[372](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L372)             <f32 as From<u16>>
[373](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L373)             <f32 as From<u8>>
[374](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L374)             <f64 as From<f32>>
[375](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L375)             <f64 as From<i16>>
[376](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L376)             <f64 as From<i32>>
[377](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L377)             <f64 as From<i8>>
[378](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L378)           and 67 others
[379](https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L379)   = note: required because of the requirements on the impl of `Into<u64>` for `i32`

https://gitlab.alpinelinux.org/otlabs/aports/-/jobs/802217#L361

Thanks for the report.  Unfortunately I'm having a hard time reproducing this.  My rustup doesn't access to a s390x-unknown-linux-musl target; I only have s390x-unknown-linux-gnu target.   I do have some musl non-s390x targets available, and I tried a few of them, and none had this problem.
If you have easy access to a s390x-unknown-linux-musl target, can you try to build terminal-size v0.2.1?  That new version uses rustix, not libc, which changes the particular line that is failing.  If that works, then I can try to update rav1e and its deptree to use v0.2

I would quite appreciate if you can try to update rav1e and its deptree to use v0.2.1.

In order to update rav1e to terminal-size v0.2.1, I think all we need is a new release of clap.  (The latest clap release uses v0.1.17, but the master branch is using v0.2.1).  Since rav1e doesn't have a Cargo.toml, they already use the latest version of clap by default.

https://github.com/rust-lang/libc/pull/2615 should fix the
wrong type for ioctl on musl.
@OTLabs could you try again with newest libc crate version (e.g. 0.2.139)?

Sure! Thank you! Giving it a try at https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/44836 but get a new error...

The build fails because cargo cannot download arbitrary package.
In your config, cargo fetch received --target flag but cargo build
didn't. Maybe that's what the problem is.

The s390x binary was successfully build! Thank you!

Thank you very much @tesuji and @faptc for your input here.  I'm going to close this issue as "solved" now


MSOutput: support custodial/tape data placement

Fixes #9842
Status
ready
Description
This PR makes MSOutput able to also perform the custodial (tape) data placement. Features in place are:

tape dataset placement supports PhEDEx and Rucio;
dataset placement is skipped if the tier is blacklisted in the MicroService configuration;
dataset placement is skipped for tiers blacklisted in the Unified configuration;
transfer request / rule creation is always made against a single RSE/node (with a single copy)

the same RSE endpoint gets used by the other datasets within the same workflow

a workflow is only completely done once all the Disk + Tape placement has succeeded
dry-run mode supported as well;
service summary report unified (data structures were the same anyways);
update the local caches for both MSOutput threads (Consumer and Producer);
removed "&cms_type=real&rse_type=DISK" from the RelVal RSE expression. We always set the specific RSE name anyways, so that expression won't make any difference;
FIXME: tape destination placeholder created (and hardcoded to FNAL). To be fixed in #9841

Is it backward compatible (if not, which system it affects?)
no, new feature
Related PRs
none
External dependencies / deployment changes
none

Basic tests done, including some real call against DDM and PhEDEx (rucio account needs to be sorted). So, it's good enough to go.


Use results to compute sessions status

Close this issue because we rework from the ground up the GUI and how data are displayed. Feel free to open a new issue to https://github.com/aneoconsulting/ArmoniK.Admin.GUI if needed.


Incorrect Send payment section - Amount window words entering values.

I am not sure what the error is here, could you explain more ?

@obany perhaps he means the error message should change depending on the selected unit?


Add dynamic VRS option that generates VRS density map with a shader

This PR adds a VRS mode that builds the density map fully in our shader. No data needs to be calculated on the CPU and transferred to the GPU. This removes a lot of overhead and will allow us to more easily adjust the density map based on performance metrics.
Ideally we deprecate VRS_XR in favour of this once its completed.
Still todo:

[ ] Detect changes and skip drawing if nothing changed from previous update.
[ ] Add settings that allow control by the developer (ranges for mix/max radii).
[ ] Figure out a way to vary the radii based on current performance metrics.

Note: Build on top of #99551

Still working on this but @DarioSamo @dsnopek , could love some early feedback.


Send a response body in defaultFailureHandler

Would it be possible to change the defaultFailureHandler so it includes a basic response body? Something like:
func defaultFailureHandler(w http.ResponseWriter, r *http.Request) {
	http.Error(w, http.StatusText(FailureCode), FailureCode)
}

Instead of the current http.Error(w, "", FailureCode).
Using http.StatusText() wouldn't leak any extra useful information to an attacker, but it would be a nicer and less confusing user (and developer) experience than just seeing a blank page on failure.
I'm very happy to send a PR for this, if you like.

Great idea. A PR would be appreciated.

Did not close automatically after merging #42 for some reason. Closing.


For charts that request more than one query, schedule report send as csv not working properly.

For charts that request more than one query, the schedule report send as csv not working.
It is attaching a file with .csv extension in the email that is corrupted.
How to reproduce the bug

Create a chart with multiple queries.
Schedule a report.

Expected results
It should attach a zip file containing all related csv files.
Actual results
It is attaching a file with .csv extension in the email that is corrupted.

Hi there, and sorry this has gone quiet for so long. We're no longer supporting Superset 2.x or prior, and since it's been a while since this thread saw any activity, I'll close this as stale. If it is still an issue in Superset 3.x or newer, we can re-open this, or feel free to open a new issue with updated context. Thanks!


Enable an option to switch b/w NHCX and Open HCX

[ ] Add an environment variable to decide the flavor of HCX used.
[ ] Add a utility class for each flavor of HCX which should handle encrypting and decrypting data, generating login tokens, and other setup-related functions.

Instead of switching b/w the HCX flavor, we should be enabling the usage of multiple flavors (Open HCX and NHCX) at the same time. It should be decided based on the payer ID.


Sentinel lab Module 4 - exercise 2

Is your feature request related to a problem? Please describe.
At point 5 of exercise 2, it is a bit unclear what is meant there.
https://github.com/Azure/Azure-Sentinel/blob/master/Solutions/Training/Azure-Sentinel-Training-Lab/Modules/Module-4-Incident-Management.md#exercise-2-handling-incident-sign-ins-from-ips-that-attempt-sign-ins-to-disabled-accounts
Describe the solution you'd like
You are supposed to click on "view playbooks" on the right hand side, then run the playbook "Get-GeoFromIpAndTagIncident" and then after a +/- 1 minute the tags will appear under the "view full details" window of the incident.
screenshot 1

screenshot 2

screenshot 3

Oh nevermind, i didn't realize the screenshots were actually there, but came later! They were actually presented after Point 7 of the exercise!


Change cursor while handwriting

While handwriting, the large (+) shaped cursor is quite distracting, it should change into a small circle while handwriting, the circle should probably be as large as the stroke it will create

Coming from using a lot of different inking apps over the years, I'll agree with this.   My first time using excalidraw I was like "that cross is very distracting while trying to write)

I found the solution and recored it as a tutorial. Have a great day. Link: https://youtu.be/INLcE7SuBKg


Cardinal neighbour quadtree

Implement CNQuadtree, cardinal neighbour quadtree (cf paper Cardinal Neighbor Quadtree: a New Quadtree-based Structure for Constant-Time Neighbor Finding)

Coverage increased (+0.4%) to 97.694% when pulling ce2052d22e33ee401625b6ba0ac2d67021c7695d on cardinal-neighbour-quadtree into d01ff65e6647dbc0cc3e1816d3a4edf640401dcb on master.


Add hook for extracting protocol from URLs

In order to sanitize potentially dangerous URLs that contain executable
behavior (e.g. “javascript:” URLs), we need to determine the protocol.
Unfortunately, URL parsing is notoriously error-prone, so we want to use
the host environment’s native functionality such that the protocol we
report is the same as what it will act upon.
In this case, we expose a protocolForURL hook that uses a generated
<a> element to set its href and check the resulting protocol. A
Node.js implementation could fall back to using the url package that
is included in the standard library.

:+1:

Will release shortly.


New version: Flow-Launcher.Flow-Launcher version 1.17.0

Pull request has been created with WinGet Releaser v2 :rocket:
Microsoft Reviewers: Open in CodeFlow

/AzurePipelines run

Issue identified with this release version. Closing.


Fix content body response on 304 code for #18199 issue

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

Fixed issues
#18199

Fix content body response on 304 http status code, according to rfc7232

Thank you!

Thank you!

It's a pleasure


Load users in Symfony resource provider

Allows the token to contain not just the username but the actual user
object, using a user provider
Also the token will have roles from the user

Also fixes https://github.com/authbucket/oauth2-symfony-bundle/issues/8


Causal sites

tstrait currently chooses causal sites at random among all the sites in the tree sequence data. We can change the Python API and provide user control of which sites could be causal. For example, the users can select the minimum and maximum allele frequency of causal sites, such that tstrait doesn't choose singletons.
stdpopsim has some functionalities that could be used for this.

Is this a duplicate of #53

Yes, it is. Thank you for noticing it.


error when starting the application

it happens just run the application, and update the repository, I need something new?

OS: Windows 10
Git Commit: (run 'git log -n 1 --pretty=format:"%H"' and paste it here)
Python Version: (run 'python -V' and paste it here)

#2768

@k4n30 Sorry, I do not understand your comment xD

You need to get a copy of encrypt.dll . You can compile one yourself or you can see if someone on slack has one they're willing to share.


raw query with dictionary causes crash

Hi,
if I want to build a custom query which takes a dictionary for parameters the debug toolbar crashes.
Person.object.raw('select %(id)s as id', {'id': 23})
The error message is
Exception Type:	TypeError
Exception Value:	format requires a mapping

There seems to be a regression: inspecting a fully working RawQuery with django-debug-toolbar 1.5 results in an exception that int where expected, whereas str has been received.
Internal Server Error: /__debug__/sql_select/

Traceback (most recent call last):
  File "django/core/handlers/exception.py", line 39, in inner
    response = get_response(request)
  File "django/core/handlers/base.py", line 249, in _legacy_get_response
    response = self._get_response(request)
  File "django/core/handlers/base.py", line 187, in _get_response
    response = self.process_exception_by_middleware(e, request)
  File "django/core/handlers/base.py", line 185, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "django/views/decorators/csrf.py", line 58, in wrapped_view
    return view_func(*args, **kwargs)
  File "debug_toolbar/panels/sql/views.py", line 19, in sql_select
    cursor.execute(sql, params)
  File "django/db/backends/utils.py", line 79, in execute
    return super(CursorDebugWrapper, self).execute(sql, params)
  File "django/db/backends/utils.py", line 64, in execute
    return self.cursor.execute(sql, params)

TypeError: list indices must be integers, not str


(PUP-4330) Adopt plist into Puppet from Facter

We no longer use the plist utility in Facter, but several Puppet
providers use it. Adopt the code into puppet/vendor for use in the
providers.

Why not vendor it like the other vendored libs, as described at https://github.com/puppetlabs/puppet/blob/6464fa3e1c629c5624944742aa8a4effa4664bb2/lib/puppet/vendor.rb#L2-L23?
Also, I'm tempted to target this at stable, i.e. for 4.0.0, since its low-risk and since we're already claiming to support facter 3 in puppet 4.

CLA signed by all contributors.

I'll retarget at stable and follow the vendoring instructions (missed they were there).

Closing in favor of https://github.com/puppetlabs/puppet/pull/3767.


Question

Can I we change the syntax color ?
Cuz mine is almost similar to the normal text.
Not like those pictures u have upload

@Pindicator If you aren't seeing the same thing then it's likely an issue with another extension or your VSCode configuration. Can you post a screenshot?
See also: https://github.com/Arcensoth/language-mcfunction/issues/40


unable to deploy in windows

unable to deploy in windows by running this command
ngh --no-silent false --name="nileshpatel17"<EMAIL_ADDRESS>Below is the log
Cloning https://github.com/NileshPatel17/ngrx-example-app.git into C:\Users\admin\AppData\Roaming\npm\node_mod
ules\angular-cli-ghpages\node_modules\gh-pages\.cache

Cleaning

Fetching origin

Checking out origin/gh-pages

Removing files

Copying files

Adding all

Committing

Pushing

An error occurred!
 { ProcessError: bash: /dev/tty: No such device or address
error: failed to execute prompt script (exit code 1)
fatal: could not read Username for 'https://github.com': Invalid argument

    at ChildProcess.<anonymous> (C:\Users\admin\AppData\Roaming\npm\node_modules\angular-cli-ghpages\node_modules\gh-pages\lib\git.js:47:23)
    at emitTwo (events.js:106:13)
    at ChildProcess.emit (events.js:191:7)
    at maybeClose (internal/child_process.js:877:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)
  code: 128,
  message: 'bash: /dev/tty: No such device or address\nerror: failed to execute prompt script (exit code 1)\nfatal: could not read Username for \'https://github.com\':
Invalid argument\n',
  name: 'ProcessError' }

window version: Windows 7 Ultimate SP1

Please scroll down in the README or read the wiki. You have to provide credentials for GitHub.

Please scroll down in the README or read the wiki. You have to provide credentials for GitHub.

Unfortunately i could not find. but anyway, i got it done by using SSH. I suggest you to please check README.
By the way, awesome package.
Thank You!

I welcome every effort to improve the documentation. Would you be so kind to send a PR with your suggestions?!  :+1:


Completed the required changes

Checklist
Link for the hosted website: https://preetshah67.github.io/multisearch/
Required changes:

[x] Center the description (Line 17 in HTML file) using a single change in CSS
[x] Fix JS so that when the search is executed, the search engine is not present in query
[x] Add Google Drive, Google Slides and Google Sheets as a provider

Optional changes:

[x] Add other search providers (Amazon, bing, GitHub, Spotify, etc)
[x] Additional CSS styling (as you like)
[x] Added a google link in the search button

Want to see hosted version

I have added the link to the hosted version


Pass exp_og by value in QPDF::readObjectAtOffset

Prevent calls to reconstruct_xref from accidentally changing an exp_og passed from the xref table.

Probably worthwhile reviewing other interfaces as well. I suspect most ought to be changed.

Included in #835


Update Mobile with Video Asset

The Mobile Asset is currently an Image.
This should be replaced with a video so we can see the animations on it.

Blocked Pending Results of #89


initiate reboot / check connection

Hi,
I've build two ESP8266 - Arduino pro mini devices.  Both identical, running the same code.
One runs just fine.  The other runs just fine, but then stops responding after a day or two, until i restart it.
Is there a way for the arduino to check the connection to the wifi / connection to the ESP... and if it is faulty restart the ESP, then the arudino...  ?

You can add something in the loop and if the wifi isn't up or you don't get a topic message in a specific timeframe then reboot.

yes, you can call boolean ESP::ready() for test connection between esp8266 and arduino. if timeout, you can reset by CH pin

Thanks I will try that.

Hello,
When I upload software to esp8266 I get an error that is below. Please help me !!!.
pi@raspberrypi /usr/share/arduino/libraries/espduino $ sudo esp8266/tools/esptool.py -p /dev/ttyUSB0 write_flash 0x00000 esp8266/release/0x00000.bin 0x40000 esp8266/release/0x40000.bin
Could not find platform dependent libraries
Consider setting $PYTHONHOME to [:]
Traceback (most recent call last):
File "esp8266/tools/esptool.py", line 22, in
import serial
ImportError: No module named serial


use yaml alias for platforms and add linux/riscv64 if possible

also add linux/s390x for agent/cli release

blocked by https://github.com/techknowlogick/xgo/issues/168


ubuntu 22 系统未安装 kernel unattended-upgrades 提示 /etc/apt/apt.conf.d/50unattended-upgrades does not exist

kubean:v0.17.5
ubuntu 22
Describe the bug
系统未安装 unattended-upgrades，提示缺少文件  /etc/apt/apt.conf.d/50unattended-upgrades does not exist

How To Reproduce
Steps to reproduce the issue:
touch /etc/apt/apt.conf.d/50unattended-upgrades

By normal default, The ubuntu system will have unattended-upgrades installed by default, and kubean will also have this tool installed by default. Suppose this tool is not installed on your Ubuntu. In that case, it may indicate that some customization of the operating system environment has been performed in the early stage, in which case you will need to manually install it.

https://github.com/kubernetes-sigs/kubespray/pull/11479


Travis CI badge stuck on "failing" even though builds are passing

Describe the bug
Our Travis CI status badge is stuck on failing despite successful builds. https://github.com/travis-ci/travis-ci/issues/4024 Unfortunately there doesn't seem to be any clear guidance on how this is being handled.
To Reproduce
Run any CI build in master.
Expected behaviour
Our Travis CI badge is green and passing.

And as if by magic, merging #52 fix it!


Adding dict, revising Rakefile to use 32-bit Perl

Lion no longer ships with a 32-bit-only Perl. I think, then, that our only option here is to set the Prefer 32-Bit Perl option.
I've also added dict, a MacRuby script to look up a word in the dictionary. I don't think it's possible to do this with RubyCocoa, so this one script will require MacRuby to be installed.

Now that I have the 32bitSafePerl solution in master, do you think you could pull, revert the changes to Rakefile (just the "Prefer 32-bit Perl" option), and re-submit the pull request? Sincerest apologies this has taken 6-months!


Can the following cron expressions be supported?

Can the following cron expressions be supported?
0 0 0 23W * ?
0 0 7 ? * 2#4

@wyc194 These features are not supported by cron-validator, but this alternate library supports them https://github.com/Airfooox/cron-validate The AWS preset should do from those examples.


Approve an allowance for an account

Problem
The Mirror Node Explorer currently does not support approving an allowance for tokens.
User Stories
As a developer or retail user, I want to approve a contract or account to spend hbars on behalf of my account.
As a developer or retail user, I want to approve a contract or account to spend fungible on behalf of my account.
As a developer or retail user, I want to approve a contract or account to spend non-fungible tokens on behalf of my account.
As a developer or retail user, I want to see what contracts can spend tokens on behalf of my account.
As a developer or retail user, I want to remove a contract or account allowance.
Solution
Add support for approving allowances.
Flow

A user wants to allow a contract or account to spend tokens on their behalf
The application provides a way for the user to enter the transaction details to approve the token allowance for a contract

The user specifies their account ID as the owner of the tokens
The user specifies the account ID or contract ID they want to grant the token approval for
The user specifies the ID of the token they want to grant the contract approval for

The token can be hbars, fungible token, an individual non-fungible token, or the entire class of non-fungible tokens

The method that needs to be called will depend on the token type the user specifies to grant the allowance for

If the user wants to grant an allowance for a fungible token, the corresponding method must be used
If the user wants to grant an allowance for a single NFT, the corresponding method must be used
If the user wants to grant an allowance for all NFTs in a given class, the corresponding method must be used
If the user wants to grant an allowance for hbars, the corresponding method must be used

The user specifies the number of tokens if it is hbar or a fungible token is specified

The transaction details will be sent to the supported wallet
The user can sign the transaction using a supported wallet
The user can view the contracts or accounts that were provided a spending allowance on behalf of the user account

Alternatives
No response

The mirror node REST API does not yet return the NFTs an account has approved a spender for.
https://github.com/hashgraph/hedera-mirror-node/issues/4348

Closing this  and opening #520 to track the missing piece of functionality which depends upon REST API enhancement.

@svienot could this be enabled for mainnet soon as many dApps have likely been affected by a change in v0.36.0 which requires accounts to approve token allowances.
https://github.com/hashgraph/hedera-services/issues/6253

Hi @mshakeg
Approving  allowance is already supported by HashScan. You have to connect your wallet (on the Staking page) for the functionality to be visible:

Hi @svienot thanks.
I did notice an issue with allowances for NFTs not showing, for example on account 0.0.1242.
I've approved an NFT allowance for:

ALL NFTs for token 0.0.3974685 in this tx
for NFT with serial_id 14 for the same token in this tx

Also, I'd like to recommend allowing for a user to select an infinite allowance for fungible or HBAR allowances. On Ethereum if an allowance is equal to type(uint256).max then that is understood to be infinite allowance. Not sure if this is even possible on Hedera but if it is I'd imagine type(int64).max to be understood as an infinite allowance.


Running in Real-time

Could you let me know if this application can perform the tasks in Real-time?
What should the device require to run the model in real-time?

I also care about this issue. I test SAM, it seems one second to segment things from an image, it is unacceptable in real time system. How could it use in VR or ohter real time fields?

So badly, the encode part costs too much in SAM, so it cannot run in real-time.

Can we expect SAM to run in real time with the lightest encoder model ?


More improvements (see description)

RibbonControl is now just Ribbon
RibbonTabGroups now use Items instead of Content, meaning no more boilerplate panels in each one - File menu is less ugly now:
RibbonWindow is now an actual window
Raster icons are no longer used, except by the buttons in the sample program, they're all vector now
Switching tabs via scroll wheel no longer wraps around after reaching either end of the tab strip
Probably something else I've forgotten about

So you can use raster or vector icons or you are forced to use vector icons?

So you can use raster or vector icons or you are forced to use vector icons? I do like the idea of being able to use Vector icons when those are available. Love all the changes!

At the time of creating this PR, I was only referring to the icons baked into the Ribbon itself (e.g. for the Ribbon groups' popout buttons, the RibbonWindow's Caption Buttons, etc), but since it seems GitHub has automatically added my subsequent commits to the PR, I can now say that for buttons defined by the application, vector icons can now also be used, in addition to raster icons.
I haven't swapped out the icons used in the sample app as of just yet, mind you, but I'll be getting to that fairly soon. I may leave a few of them still using bitmaps though, just to demonstrate the possibility.
Also I feel it may be worth noting that a side benefit of swapping the caption buttons's raster icons on the RibbonWindow for vector icons is that now they'll be much, MUCH easier to re-style, should any developer using this Ribbon desire to put their own spin on the overall look and feel (which I, admittedly, probably would - I respect your wish for a Microsoft Office-like look, but in all honesty, I'm not a huge fan of all this minimalism myself)

Well I really like the office ribbon but we shouldn't stop ourselves from imagining new functionality. Office ribbon just a "starting kit". Nonetheless, minimalism - if it is synonym of user-friendly, well designed GUI - I'm found of it :)


1.1-18-1 update

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

Using toolchain directly in this manner is deprecated.  Consider using the compilers outlined here.

r-nlme win build is still running, hence the missing dependency.


[GHSA-6hvf-xvwm-vrw4] XMLTooling Library Incorrectly Handles Some Exceptions

Updates

Affected products

Comments
This is affects the cpp version not the java version. I am not sure why the correct ecosystem should be in this case.
See also https://github.com/github/advisory-database/pull/2509

@twn apologies for that. We don't have a category for C++ stuff at the moments so, our bad and I'll get this advisory pulled.


New package: BioGenerics v0.1.0

Registering package: BioGenerics
Repository: https://github.com/BioJulia/BioGenerics.jl
Created by: @BenJWard
Version: v0.1.0
Commit: 43fe762e9c992796cbb7b1abbf091395f21a0c84
Reviewed by: @BenJWard
Reference: https://github.com/BioJulia/BioGenerics.jl/commit/43fe762e9c992796cbb7b1abbf091395f21a0c84#commitcomment-39398469

Please close this

This package is moving over from the BioJuliaRegistry (more are to follow).

Is this already in the BioJulia registry? If you want to keep the same version across registries I think we can merge this and fix the compat bound later

Yeah it's already in the BioJulia registry

FormatSpecimens #15253 is also the same as in the BioJulia registry

@DilumAluthge Can you check this is ok and merge? If I can get this and FormatSpecimens in then BioSequences and a lot of the other packages become much more simple: I can move packages that exist in both registries at different versions using registrator, and then bulk migrate over the brand new packages in one PR by coping over their directories.


[Bug Report] Play song automatic close

Bug Description
[17:08:02] [ERROR] [SignatureCipherManager]: Problematic YouTube player script /s/player/fb725ac8/player_ias.vflset/fr_FR/base.js detected (issue detected with script: no n function match). Dumped to C:\Users\Antoine\AppData\Local\Temp\lavaplayer-yt-player-script9784303936185792342.js
[17:08:03] [ERROR] [AudioHandler]: Track on9TXY8kYyk has failed to play
com.sedmelluq.discord.lavaplayer.tools.FriendlyException: Please sign in
at dev.lavalink.youtube.clients.skeleton.Client.getPlayabilityStatus(Client.java:74)
at dev.lavalink.youtube.clients.skeleton.NonMusicClient.loadTrackInfoFromInnertube(NonMusicClient.java:98)
at dev.lavalink.youtube.clients.skeleton.StreamingNonMusicClient.loadFormats(StreamingNonMusicClient.java:35)
at dev.lavalink.youtube.track.YoutubeAudioTrack.loadBestFormatWithUrl(YoutubeAudioTrack.java:183)
at dev.lavalink.youtube.track.YoutubeAudioTrack.processWithClient(YoutubeAudioTrack.java:109)
at dev.lavalink.youtube.track.YoutubeAudioTrack.process(YoutubeAudioTrack.java:71)
at com.sedmelluq.discord.lavaplayer.track.playback.LocalAudioTrackExecutor.execute(LocalAudioTrackExecutor.java:109)
at com.sedmelluq.discord.lavaplayer.player.DefaultAudioPlayerManager.lambda$executeTrack$2(DefaultAudioPlayerManager.java:339)
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at java.base/java.lang.Thread.run(Unknown Source)
Steps to Reproduce
idk
Expected Result
i play songI place the order, he enters the living room and leaves
Debug Output
System Properties:
  java.version = 11.0.25
  java.vm.name = OpenJDK 64-Bit Server VM
  java.vm.specification.version = 11
  java.runtime.name = OpenJDK Runtime Environment
  java.runtime.version = 11.0.25+9
  java.specification.version = 11
  os.arch = amd64
  os.name = Windows 10

JMusicBot Information:
  Version = 0.4.3
  Owner =<PHONE_NUMBER>160449085
  Prefix = !
  AltPrefix = null
  MaxSeconds = 0
  NPImages = false
  SongInStatus = true
  StayInChannel = false
  UseEval = false
  UpdateAlerts = true

Dependency Information:
  JDA Version = 4.4.1_353
  JDA-Utilities Version = 3.0.5
  Lavaplayer Version = 2.2.1

Runtime Information:
  Total Memory = 152
  Used Memory = 110

Discord Information:
  ID =<PHONE_NUMBER>829053521
  Guilds = 1
  Users = 2

Additional Info
No response
Checklist

[X] I have looked for information about this within the documentation
[X] I have searched for similar issues on the issues page
[X] I am using an official release from the releases page

JMusicBot Version
0.43

Duplicate of #1588


Refactor creation of vehicles from API responses

Proposed change

This is an internal change with no impact on the usage of the library or in Home Assistant (except when patching get_vehicles() directly, e.g. for testing purposes).
During development of the tests for https://github.com/home-assistant/core/pull/76613, I had to copy most of the code from get_vehicles() to the HA test case due to different testing mechanisms.
In the library tests, we rely on capturing the HTTP requests and providing the correct answers. However in HA, this would be a lot of doubled development effort, so I just patched get_vehicles(), which triggers all requests (including login).
With this PR, one only has to patch get_vehicles(), load the vehicle and state responses and then call add_vehicles(), without worrying about additional needed keys or of the vehicle already exists.
Type of change

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality to this library)
[ ] Breaking change (fix/feature causing existing functionality to break)
[x] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:

Checklist

[x] The code change is tested and works locally.
[x] Tests have been added to verify that the new code works.

Codecov Report

Merging #480 (be81777) into master (bb781d2) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #480   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           16        16
  Lines         1213      1221    +8
=========================================
+ Hits          1213      1221    +8

Flag
Coverage Δ

3.10.x
100.00% <100.00%> (ø)

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

Impacted Files
Coverage Δ

bimmer_connected/account.py
100.00% <100.00%> (ø)

bimmer_connected/vehicle/vehicle.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Update images.mdx

Description (required)
Showcases the use of the relative image path without leading ./ in MDX file(s).

I am proposing this change in Astro docs to show how Astro users can use the particular functionality.

Yup, I've made change in only the english docs.
Related issues & labels (optional)
The related PR is https://github.com/withastro/astro/pull/10754

Looks like this was released, so good to merge now! :partying_face:


Microsoft Visual Studio Code template

Changes

Add Microsoft Visual Studio Code template

Boot will took a few minutes for installing packages and dependency. VSCode can be accessed from
https://fqdndomain.com/?tkn=tokenfromenvironment

this is exactly like https://coolify.io/docs/services/code-server except
it's from Microsoft. so you get to use Microsoft ecosystem like installing
extension from https://marketplace.visualstudio.com/vscode, syncing, or
using github copilot, which is not available on normal code server.
On Sat, Nov 9, 2024, 1:34 AM 🏔 Peak @.***> wrote:

What would this be useful for - is this not just and ubuntu container with
VScode installed?
—
Reply to this email directly, view it on GitHub
https://github.com/coollabsio/coolify/pull/4155#issuecomment-2465396825,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKPO4HKZ2J3WNG2WTEZ2UTZ7TYZBAVCNFSM6AAAAABRJ7GVZSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRVGM4TMOBSGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hey @yokowasis,
Unfortunately, we won't add this service to the official list, as it involves installing third-party components into a simple Docker image.
We will soon define the exact criteria for what can be added to the service list.
If you use the Docker Compose buildpack and copy-paste this compose file, it will still work as is.
Sorry about that. I am closing this PR.


Win11 - Internal Error with Vite and TypeScript - "Unable to follow symbol for 'Promise'" Description

Describe the bug
Note: Linux/Debian works (Docker tested) only Win11 has a problem.
I encountered an internal error while using Vite with TypeScript. The error message is as follows:
typescript - terminal message
[vite:dts] Internal Error: Unable to follow symbol for "Promise"
This issue occurs when attempting to initialize a promise within a function. The relevant code snippet is:
typescript - code
1
exampleFunction(input: Input, config: Config): Promise<ExampleClass> {

    let initConfig = extend(DEFAULTCONFIG, config);

    return new Promise<ExampleClass>((resolve, reject): void => {

        // some logic here
    });
}

2
exampleFunction(input: Input, config: Config): Promise<any> {

    let initConfig = extend(DEFAULTCONFIG, config);

    return new Promise<any>((resolve, reject): void => {

        // some logic here
    });
}

The error seems to originate from Vite's handling of TypeScript type declarations, particularly when dealing with the Promise type.
Reproduction
https://stackblitz.com/edit/stackblitz-starters-bmmh1n?file=api.ts
Steps to reproduce
Steps to Reproduce
Create a TypeScript project with Vite.
Implement a method that returns a Promise as shown in the code snippet above.
Run the Vite build or development server.

Expected Behavior
Vite should correctly process the TypeScript types without throwing an internal error related to the Promise symbol.
Actual Behavior
Vite throws an internal error, stating it is unable to follow the symbol for Promise.
System Info
## System:
    OS: Windows 11 10.0.22631
    CPU: (16) x64 AMD Ryzen 7 3700X 8-Core Processor
    Memory: 29.95 GB / 63.92 GB
  Binaries:
    Node: 20.9.0 - C:\Program Files\nodejs\node.EXE
    npm: 10.8.2 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Chromium (127.0.2651.74)
    Internet Explorer: 11.0.22621.3527

## Environment:

    Vite Version: 5.2.12
    TypeScript Version: 5.4.5
    Operating System: Win11
    Node Version: 20.9.0

Validations

[X] Read the FAQ.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

Appears to be related to rollupTypes: true. Without it, it does work.

Extra info: we found that this was only an issue on Windows, not Mac or Linux


i cant log in in my account

please help me to open my account GLOBAL MMM . because I enter the wrong code . and I never booked or using this Authenticator .

I need Additional Authorization removed in my MMM account. How can I do that?


Docs: update location of icons in documentation pages

Description
Adjust the location of icons as referenced throughout the documentation
Testing in sage-lib
Copy changes only
Testing in kajabi-products
No external impact. Doc changes only.

Is this to adjust the text used? However, if it's supposed to reference a new location, then I have an issue because  I don't see anything pointing to the new Icon documentation in Pine.

Is this to adjust the text used? However, if it's supposed to reference a new location, then I have an issue because I don't see anything pointing to the new Icon documentation in Pine.

The updates here were meant to adjust the language to reference “Icons” in the Foundations page. I could clarify that in all areas by saying “See Foundations / Icons”. I also see the need to add information directing users to Pine as well. What do you think?

It was the title of the PR that threw me off.


Optimize disk usage during install

Installation is noticably slow. There's a lot of redundant disk work during installation, associated with the transaction machinery. A lot can be optimized here.

I don't specifically know what is inefficient here, but somebody could do some stracing on Linux and see if there's anything obvious.

stat appears to get called 3 times for each file in  set_file_perms:

One to get the file permissions in needs_x https://github.com/rust-lang/rustup.rs/blob/67d740fca643f07b38618f42a28757fa988b5ba7/src/rustup-dist/src/component/package.rs#L131
One inside utils::is_directory https://github.com/rust-lang/rustup.rs/blob/67d740fca643f07b38618f42a28757fa988b5ba7/src/rustup-dist/src/component/package.rs#L141
One to get the file permissions (again) before setting the mode https://github.com/rust-lang/rustup.rs/blob/67d740fca643f07b38618f42a28757fa988b5ba7/src/rustup-dist/src/component/package.rs#L151

This could probably be reduced to just one stat call.

Given that the majority of filesystem operations which Rustup performs are reading and writing large (10s of MBs) datasets, I think the stat() inefficiency is by no means the most important thing to consider for speed.
I do, however, think that we should look at consider if there's anything we can do to improve install performance on Windows.

Installation on Linux filesystems is fast even on HDD so measuring performance there won't help much for Windows but still it'd be nice to make it even faster.
I don't know which one is more limiting NTFS or AV software but shipping less files would be the way to go here.
Nightly on my Linux machine consists of 17957 files, 16617 of them docs. It'd be significant improvement for Windows if they could be fused together into few bigger files.

If I understand correctly, the set_file_perms function walks the entire tree of files after they have been extracted to ensure their permissions are correct. This is especially costly in the docs, since there are a tremendous number of files there as @mati865 points out. I would expect tens of thousands of fewer system calls in an installation operation to be noticeable, especially e.g. on NFS.

I've done some basic analysis on  Windows 10 Surface; since OS specific code paths can make Linux traces (while convenient to grab) misleading :).
The high level view is that for docs we spend:
2m20s extracting ~16K files
30 seconds copying from staging to final dir
26 seconds deleting from staging dir
sidebar: We are quite chatty in the OS syscalls we make - many extra opens than needed, but looking at the timing, that should be considered a microoptimisation, not a first-pass target. See the trace excerpts below for examples.
What should we be able to expect without getting into hairy levels of OS specific integration?
A trivial Python program can write a similar sized data set in 60seconds, and the archive should be in hot cache, and is tiny compared to RAM these days so - that should be a decent target.

Python benchmark

import os
import shutil
import time

now = time.time()

os.mkdir("root")
os.chdir("root")
written = 0
for dirname in range(80):
    os.mkdir("%s" % dirname)
    for fname in range(200):
        with open("%s/%s"%(dirname, fname), "wt") as f:
            written += f.write(("this is a small file %s\n" % fname)*512)

finish = time.time()
os.chdir("..")
shutil.rmtree("root")

print("created 16000 files in 80 directories in %0.2fs with %s bytes" % (finish-now, written))

Total time right now is 3m20s, of which 60 seconds is entirely dealing with making an unused copy of the content: the staging process is:

unpack the archive to a temp directory
copy from the temp directory into the final directory
delete the temp directory contents

So we can shave a minute off if we unpack directly into the final directory. Would that be safe? If the archive has been validated during download, so we know its contents are intact, then there are no additional IO errors that can occur unpacking directly:

the archive, just like the temp storage, is a file on local disk
the output directories are being made and validated as we go
because we are making copies, we can already run out of disk space mid-operation

Alternatively, we could unpack to a staging directory but then rename into the final directory - that would require double-touching each file, but ensure we could fit all the files on disk before placing any of them into the target location (if thats important - I don't think it is :)).
That would still leave 2m20s to unpack vs 60s for a trivial driver; some of that may be decompression overhead, IO from the source archive etc- I don't have a breakdown for that yet.

Current performance quick metrics:
With defender on:
my perf script above: 1m8s
rustup master release build: 3m26
stat-per-dir rather than stat-per-file: 3m14s
8M IO buffer on the tar: 3m14s
less stats + buffer - 3m11
With a defender exclusion on ~/.rustup:
my perf script: 24s
rustup master release build: 1m6s
stat-per-dir: 1m3
8M IO buffer on the tar: 50s
less stats + buffer - 36s
So there is still a lot of fat we should be able to cut; I have some experiments (see above) but nothing that is as definitive as halving the IO as my first patch did :). The benchmarks are pretty noisy too :/.
A variable I have to isolate yet is the windows search index; that is also possibly double / triple working (once on initial write, once on the rename, and then removing from the index at the rename step as well). Setting the indexing service to pause reduces load while testing, but I haven't run a set of benchmarks with it enabled/off : and when it is enabled it dynamically throttles, so enables is not always enabled, adding complexity in generating robust numbers.
tl;dr: there is hope for more improvements; a lot of syscall noise still remains though changes deeper in e.g. tar and possibly rust itself will be needed to fix that (e.g. setting the mode on a file in windows should be done while it is open - chmod is sugar for fchmod).
Users wanting things to be as fast as possible today should do the following IMO:

set an exclusion on their .rustup directory, or exclude rustup.exe; the former is safer than excluding the binary image name, as that would provide a trivial malware vector. Excluding just the toolchains dir is not sufficient; we write to random paths in .rustup/tmp.
disable windows indexing service while running rustup; let the index service batch process the new files after they are installed rather than contending. Possibly there are APIs we can call to hint this to the OS; I have not investigated this yet.

I think this ticket should be closed: we're doing create, write, close now for each file, no extra copies. We do do the move-into-place thing but as docs is moved as a top level dir, its a single syscall.

I concur, thank you @rbtcollins


Build Unbonding Form View

What's the problem? (required)
I am unable to unbond partially within the explorer.
What's the current behavior? (required)
Clicking on unbond gives a metamask popup prompting you to unbond all of your stake.
Do you have any personal experiences dealing with this problem? (optional)
Yes, I have found that you can change the gas price in MetaMask but that's about it. An unbonding form should be built that is similar to the current bonding form. The difference would be how it operates.
Describe a possible solution you've considered (optional)
When a user click unbond and unbonding modal containing an form should popup that guides the user to unbond partially or fully unbond. Hence, an unbonding form view is needed to give the user the ability to interact with the unbonding form mentioned in issue #294.
Additional context (optional)
The view should look similar and operate similar to the current bonding form in the explorer with the difference being that it has an option to unbond partially or unbond fully. Here is the bonding view's  code, which is example code of how the unbonding form view code may look and work.
Bonding Form

Unbonding Form Example

Note: The styling of the unbonding form component should be the same/similar to the bonding form.
Please see partial unbonding and fully unbonding for a walk through of the mock ups and user flow of how partial/fully unbonding should operate.

Note: when you click unbond all LPT, the amount should be autofilled. When you click unbond partial LPT, the amount should be editable.

@Randy1Burrell  please note I have adjusted the text for the unbonding popup, and this should be consistent everywhere:
Unbonding Definition. Unbonding allows you to withdraw your token by releasing the token from the deposit with its Transcoder.
Unbonding Period. Once you unbond, it takes 7 days (the unbonding period) in order to be able to withdraw your token.
Amount. You may unbond ALL or a PARTIAL amount of LPT. Enter the amount of LPT you wish to unbond in the box below.
View Unbonding Transactions. View unbonding transactions on the Staking page. Go to Account -> Staking and scroll to "Pending LPT."
Multiple Unbonding Transactions. Each time you Unbond a partial amount of token, a transaction is created and must be withdrawn or rebonded separately.
Change your mind? If you have initiated the Unbonding process but wish to rebond your LPT, you do not need to wait for the period to end. Go to the Staking page on your account and click "Rebond."