Make CMSSW 12 geometry default + Cleanup

This PR makes the CMSSW 12 geometry the default and removes the compilation flag. Similarly, the pT>0.8 GeV configuration becomes the default and the relevant compilation flag is removed. Some cleanup is also done.
Comparison plots (names+git tags should be self-explanatory):
https://uaf-10.t2.ucsd.edu/~evourlio/SDL/moveToCMSSW12Geometry/
Timing before all changes (phi3):
Total Timing Summary
   Evt    Hits       MD       LS      T3       T5       pLS       pT5      pT3      TC       TCE      Event      Short           Rate
   avg      6.3      3.8      5.0      4.4      3.8      1.4      2.5      2.1      4.9      6.5      40.7      33.0+/-  4.6      42.8   explicit_cache[s=1]
   avg     11.1      5.1      4.6      6.8      6.7      1.9      4.6      3.4      8.0      8.0      60.2      47.2+/-  7.5      32.7   explicit_cache[s=2]
   avg     21.1      7.9      6.8     12.6     13.5      3.3     10.2      6.5     16.9     12.3     111.1      86.6+/- 14.1      29.9   explicit_cache[s=4]
   avg     31.2     10.7      9.9     18.0     20.2      4.5     16.6      9.3     24.2     16.1     160.7     125.0+/- 17.5      28.3   explicit_cache[s=6]
   avg     49.9     12.3     11.4     25.1     27.7      6.0     24.4     12.5     31.1     19.2     219.7     163.7+/- 26.9      29.1   explicit_cache[s=8]

Timing after all changes (phi3):
Total Timing Summary
   Evt    Hits       MD       LS      T3       T5       pLS       pT5      pT3      TC       TCE      Event      Short           Rate
   avg      6.4      4.1      4.2      4.6      3.9      1.3      2.2      1.9      4.4      6.4      39.3      31.6+/-  5.8      41.2   explicit_cache[s=1]
   avg     10.8      5.7      4.6      7.1      6.6      1.7      4.3      3.1      7.5      8.0      59.3      46.8+/-  8.5      32.2   explicit_cache[s=2]
   avg     21.5      7.8      6.3     12.3     13.3      3.3      9.5      5.9     15.3     11.0     106.3      81.5+/- 12.5      28.2   explicit_cache[s=4]
   avg     33.9     10.9      8.9     20.2     21.2      4.5     16.5     10.1     23.2     14.6     163.9     125.5+/- 19.2      28.9   explicit_cache[s=6]
   avg     49.2     12.5     11.3     25.5     27.1      5.3     20.4     12.1     32.6     20.1     216.0     161.5+/- 25.7      28.4   explicit_cache[s=8]

How many events do we have with the 12X? Is it the same 500 as in 104X/old?

This is the timing I got on lnx7188. Closer to the previous times for cmssw 11:

This uses the same input file as before correct? If so, there is still a bit of an efficiency drop compared to old. Otherwise, This passes the usual checks.

The stacked plots seem to be missing a plot though.

This uses the same input file as before correct? If so, there is still a bit of an efficiency drop compared to old. Otherwise, This passes the usual checks.

When you say "old", do you refer to the configuration before removing things? If yes, then the input is indeed the same and same amount of events. This PR is purely technical, so I don't see where variation may come from, except from the order things run on the GPU. Do you think there's another source?

The stacked plots seem to be missing a plot though. The efficiency verse pt

You are right, I didn't notice that. I will rerun the plots to fix this.

The stacked plots seem to be missing a plot though. The efficiency verse pt

Turns out this is a long standing typo in the html file. I am fixing it with an additional commit to this PR. Thanks for catching it.

If you are talking about ttbar, then no, the statistics is not the same. The 12X sample has 175 events. If we want more, I had volunteered to produce some but, if I recall correctly, Philip suggested that this production may be a good exercise for Matt. I don't know how it evolved from there but we can revisit this.

Doesn't Gavin have more data already available? (was it 6K events?)

Well my concern isn't so much the this PR's cleanup vs the previous master but rather the difference between the cmssw 12 results vs cmssw 11. There is, and has been, a decrease in the efficiency in using cmssw 12 which i thought was still being understood and addressed. Going from 90% efficiency to 85%. I'm a bit concerned about making it the default now when this drop in efficiency remains. We can't really say that we have comparable efficiency to current patatrack+CFK.

Doesn't Gavin have more data already available? (was it 6K events?)

I think he has but I don't know the details of where this is stored. We can ask and make it available to all.

Well my concern isn't so much the this PR's cleanup vs the previous master but rather the difference between the cmssw 12 results vs cmssw 11. There is, and has been, a decrease in the efficiency in using cmssw 12 which i thought was still being understood and addressed. Going from 90% efficiency to 85%. I'm a bit concerned about making it the default now when this drop in efficiency remains. We can't really say that we have comparable efficiency to current patatrack+CFK.

The difference has been there since a long time with no obvious way to fix it, at least to my knowledge.
In any case, we got "complaints" by the TRK people for using such an old geometry for the tracker when using it for comparisons, so I don't think that sticking with the old geometry makes sense, just for saying that we are comparable with the current patatrack+CKF. But let's talk about it at the meeting if there are other proposals.

I know this difference has always existed, but I was under the impression that it was being addressed and cmssw 12 would become the default once it was. The issue with this becoming the default now is also that we lose the cmssw 11 geometry option. Which means it would be harder to go back to make comparisons. To me it seems like we would be accepting this drop in efficiency and I don't think that is what we should do.

i mean, maybe we can regain some efficiency by returning the cuts since they were all set for cmssw 11 geo, but i think the differences went down to which hits we were seeing right?

Also, just looking at the cmssw 12 efficiency plots alone, it seems clear there is some kind of issue..., the turn on has a noticeable drop around 1 GeV that seems out of place. The plateau starts to drop for high pt tracks (>10 GeV). These plots just don't look correct at the moment. I think this should be addressed before we make this the default that we share outside of our group....

Doesn't Gavin have more data already available? (was it 6K events?)

I think he has but I don't know the details of where this is stored. We can ask and make it available to all.

The 6k event sample is located here:
/mnt/data1/gsn27/samples/CMSSW_12_5_0_pre3/src/ttbar_CMS12_9000

Can we aim to conclude on merging this PR some time in the Tuesday meeting or soon after in this first week of October?

For documentation and for reference, here are the validation plots after the PR was merged:
https://www.classe.cornell.edu/~evourlio/www/SDL_GPU/afterMovingToCMSSW12Geometry/eff_plots__GPU_explicit_cache_236179b_PU200_n200/
And here is the timing on lnx7188 (close to the timings of CMSSW 10 geometry):
Total Timing Summary
   Evt    Hits       MD       LS      T3       T5       pLS       pT5      pT3      TC       TCE      Event      Short           Rate
   avg      4.8      3.4      3.6      3.5      3.1      1.1      1.8      1.6      4.0      5.4      32.5      26.5+/-  4.1      35.0   explicit_cache[s=1]
   avg      7.3      5.1      4.7      6.1      5.5      1.7      3.7      2.6      6.7      6.6      50.0      40.9+/-  8.5      31.0   explicit_cache[s=2]
   avg     16.4      7.4      6.5     11.2     10.8      2.8      8.2      5.4     13.9      9.9      92.5      73.4+/- 10.7      25.1   explicit_cache[s=4]
   avg     25.5     10.1      8.2     17.2     17.1      3.7     13.6      8.5     20.2     12.7     136.9     107.7+/- 17.8      24.6   explicit_cache[s=6]
   avg     36.1     11.6     10.5     21.3     20.6      4.8     19.2     11.3     26.0     16.6     178.0     137.1+/- 21.4      24.1   explicit_cache[s=8]

An extensive set of comparisons is linked in (the now closed) issue #191.


duplicated indicators

All indicators from RRI appear twice in here: https://landportal.info/book/dataset/rri-ftd
Drupal information seems correct and not duplicated.

fixed


Enable autoscaling, tune cluster parameters, add docs

Closes https://github.com/opentargets/issues/issues/3142.
Now that https://github.com/opentargets/issues/issues/3145 is addressed, enabling autoscaling is easy ;-)
I have verified that it works using LD index as an example. I didn't run it to completion, but it has a very characteristic CPU load profile where we can immediately see the difference. On a single 96 core VM, graph width = 6 hours:

And on an a autoscaled cluster of 20 VMs (plot width = same 6 hours):

(@ireneisdoomed Sorry again for the clash, I didn't immediately realise you were running the step at the same time as I did — will be more vigilant going forward & change the output paths, etc.)
The cluster was very stable at 2 primary + 20 secondary workers (I even tested randomly killing a few secondary workers — it recovers beautifully), so I increased the maximum number of secondary VMs to 50 in the autoscaling policy. I'll monitor performance and adjust as needed going forward.
I also added some basic documentation on the high level organisation of the pipeline, including which DAGs we have, and how autoscaling works. (Feel free to suggest alternative places where this piece of documentation can be moved if you have some ideas)

Codecov Report

Merging #227 (dbb2f29) into tskir-3145-fix-hydra-instantiation (dbb2f29) will not change coverage.
The diff coverage is n/a.

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

@@                         Coverage Diff                         @@
##           tskir-3145-fix-hydra-instantiation     #227   +/-   ##
===================================================================
  Coverage                               85.35%   85.35%
===================================================================
  Files                                      82       82
  Lines                                    1850     1850
===================================================================
  Hits                                     1579     1579
  Misses                                    271      271

@tskir try to merge when you can


Update Plane.js

Mapped UVs for rectangular textures, I'm not sure if nonrectangular textures ever happen? if so, then this method will not suffice.

weird? the test fails on a piece of code that I haven't touched?   somebody else did a faulty commit maybe?

Hmm that failing build check is weird, the PR looks good though!


[devops:bot] update deps in nix/sources.json (from: rhino-core@ba22dad

update nix refernces for rhino-core and|or nixpkgs to match https://github.com/rhinofi/rhino-core/blob/ba22dad6518643ccdcc3f412790b8ec501e97bac/nix/sources.json (branch: dev)

superseded by #108


Always use utf8 encoding (should fix #335)

Thanks @hamishmack - looks like multiple people addressed this one. Fixed via #343


toss @polkadot/dev after extracting out relevant configs and deps from it

so we don't have extraneous dependencies and don't run into hoisting issues in future

Initially I wanted to try react-scripts, to see if it's compatible with our TS projects, so that the overhead of bumping packages in polkadot-js/dev(-react), and making sure each bump doesn't break anything, is done and tested by facebook, not by us.
So basically:

@polkadot/dev = react-scripts + typescript + some babel plugins + some other packages for internal maintenance
@polkadot/dev-react = @polkadot/dev + styled-components + enzyme

Inside @polkadot/dev, all the polkadot-dev-* scripts will stay the same, but use package versions (jest, webpack, babel) dictated by react-scripts.

Packages like api, common or tools will obviously not use react-scripts build|start|test, but polkadot-dev-* scripts directly.
Packages like apps or light-apps here will use react-scripts build|start|test included in @polkadot/dev.

Above is an ideal future (imo), but today we're not there, and having both is apparently a headache. We have 2 solutions:

drop @polkadot/dev, and follow react-scripts.

we could create a dev/ package inside this repo, which will be react-scripts + typescript + the minimal amount of packages to make this whole thing work. And if the above long-term scenario sounds good to you @jacogr @yjkimjunior, then we could move this dev/ to @polkadot/dev
downside: there'll be 2 dev/s for a while.

drop react-scripts, and do the same as @polkadot/apps

no config, everything will work (we rely on Jaco bumping the packages)
any project using @polkadot/dev + CRA will have a headache, we stray away from CRA

I prefer 1, but I'm open to 2 too for consistency between Parity projects, and revisit when react-scripts becomes more suited for TS.

+1 for 1


Cannot launch scrcpy on Debian 10

I installed scrcpy with apt-get install scrcpy on linux Debian 10. (It's an new installation)
Everything worked but when Im trying to start scrcpy it just says this:

INFO: scrcpy 1.14 https://github.com/Genymobile/scrcpy
stat: No such file or directory
ERROR: '/usr/local/share/scrcpy/scrcpy-server' does not exist or is not a regular file

The folder share does NOT contain a scrcpy/scrcpy-server folder/file. I reinstalled scrcpy but that did not help.
Thx for your help!

which scrcpy
echo $SCRCPY_SERVER_PATH


Add option to useNativeDriver; defaults to false

Fixes #339

@mastermoo it would be so awesome if we could get this merged!

Thanks @fohletex , following your repo in the meantime :)

Thanks @fohletex , following your repo in the meantime :)

ping @mastermoo, would you be able to merge this and cut a new release?

@fohletex thank you for your PR i merged internally my library.

I did test this and the problem is that and background color on the button will be lost when setting useNativeDriver: true
This is the case even though the latest versions of React Native support animating backgroundColor.

If anyone can get this working I'd love to know, and I'd be pretty impressed. So far nothing I've tried fixes the button losing its background color if using useNativeDriver: true


[REEF-520] Enforce JavadocType and TodoComment checkstyle as errors in reef-annotations

This enforces JavadocType and TodoComment checks in reef-annotations subproject
to avoid regressions during further changes.
JIRA:
REEF-520

Thanks! I will merge.


Update jsoniter-scala-core, ... to 2.13.11

Updates

com.github.plokhotnyuk.jsoniter-scala:jsoniter-scala-core
com.github.plokhotnyuk.jsoniter-scala:jsoniter-scala-macros

from 2.13.10 to 2.13.11.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.plokhotnyuk.jsoniter-scala" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #402.


sortable animation before dropping the item

Thx, I have a problem with your sortable example: http://codef0rmer.github.io/angular-dragdrop/#!/sortable.
I want to see the sortable reorder animation when I drag the item on other items. It's just like the reorder when I drop the item finally. Could I set any options to make it ? Such as the jqueryui sortable demo here: http://api.jqueryui.com/sortable/

@nianiaJR Yeah, Unfortunately there is no way to integrate Sortable in angular-dragdrop. I think if you go ahead with creating your own, might be complex but give it a shot if you want.

Thank you prompt reply


break the build

Testing Jenkins integrations. Please ignore.

Ok: CI reports failures as expected, yet doesn't close the PR.


Compiling and running c/C++ code

I am new to vs code and have been working quite comfortably before the july update.
I tried a very basic Hello World Example and configured my Task.json and launch.json file.
I am able to build my code successfully but i am not able to view my output.
In my debug console, I see few warning messages too. I could not get that.
Help me resolve this issue.

VSCode Version:1.14.1
OS Version: Windows 7

Steps to Reproduce:

Build tasks.json by Ctrl+Shift+B
Run the code by F5.

C.zip

Reproduces without extensions: Yes

Please file this agains the C++ extension here https://github.com/Microsoft/vscode-cpptools


⚠ Bofrost.fr has degraded performance

In 60055f6, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1054 ms

Resolved: Bofrost.fr performance has improved in 501c13f after 8 minutes.


Latest docker image fails with permission denied

Running from your latest umbrel app settings: https://github.com/Impa10r/umbrel-apps/blob/master/peerswap/docker-compose.yml
App fails to start:
2024/03/23 20:14:24 open /home/peerswap/.peerswap/psweb.log: permission denied
2024/03/23 20:14:28 open /home/peerswap/.peerswap/log: permission denied

That's because your .peerswap is owned by root and all files in it. See this to fix.

Looks like somehow this node was in the following state:
/home/umbrel/umbrel/app-data/peerswap/data was owned by root and was empty
/home/umbrel/.peerswap was owned by umbrel and contained all the config files.
resolved by running:
sudo chown umbrel:umbrel /home/umbrel/umbrel/app-data/peerswap/data
cp ~/.peerswap/* /home/umbrel/umbrel/app-data/peerswap/data/
sudo /home/umbrel/scripts/app restart peerswap


How to get user profile data inside views?

I want to access user profile information inside my views. for example I have a field called UserAvatar So I want to access this property this way:
User.Identity.GetUserAvatar();
Any idea?

I found the solution:
First of all we should create this extension method to get out custom property:
public static class IdentityExtention
    {
        public static string GetAvatar(this System.Security.Principal.IIdentity user)
        {
            var service = SmObjectFactory.Container.GetInstance<IApplicationUserManager>();
            return service.FindByIdAsync(int.Parse(user.GetUserId())).Result.Avatar;
        }
    }

Then we can use it inside our view this way:
@User.Identity.GetAvatar()

If it's for 1 user, that's OK (even it violates the rule of don't query from the view). But if it you are using this method in a loop, it will produce n+1 selects which could be solved by issuing just one query.


Inverse Kinematics feature "Pole Target"

Would be great if IK Node supported "Pole Target" Spatial node that tells IK Solver in which direction it should bend.
Blender IK Constraint can be used for reference.

First of all thank you for your report and sorry for the delay.
We released Godot 3.0 in January 2018 after 18 months of work, fixing many old issues either directly, or by obsoleting/replacing the features they were referring to.
We still have hundreds of issues whose relevance/reproducibility needs to be checked against the current stable version, and that's where you can help us.
Could you check if the issue that you described initially is still relevant/reproducible in Godot 3.0 or any newer version, and comment about it here?
Our Bugsquad will review this issue more in-depth in 15 days, and potentially close it if its relevance could not be confirmed.
Thanks in advance.
Note: This message is being copy-pasted to many "stale" issues (90+ days without activity). It might happen that it is not meaningful for this specific issue or appears oblivious of the issue's context, if so please comment to notify the Bugsquad about it.

Yes :)

The IK module was dropped in 3.0, so this is no longer applicable.


Get-IntuneManagedDevice always returns 0 for physicalMemoryInBytes

I was trying to use Get-IntuneManagedDevice to retrieve physicalMemoryInBytes but it returns "0" for all devices. This data is in InTune. I found this issue posted about the Graph API that points out you need to query each individual device and select that attribute specifically. I tried using -Select physicalMemoryInBytes but got an error that the argument doesn't belong to the set. Same results when specifying a managedDeviceID as well.

I'm having the same problem. Even querying 1 device directly and selecting only the physicalMemoryInBytes attribute still returns 0.

Hello,
Seems this issue is still ongoing even a year later. I have tried to switch to beta profile as well and is still not working.
Any idea if there is an ongoing fix for this or some workaround to make it work?
Cheers

Still waiting for an update on this? @microsoftopensource  @nsoy

+1

physicalMemoryInBytes listed under Device Hardware in the Intune GUI is accurate.  Get-IntuneManagedDevice shows physicalMemoryInBytes of 0.  Please prioritize and update as this has been broken for years.

Hi.  I have just been looking at this too and am experiencing the same issue.  As I am looking at reviewing the current Windows estate for upgrade accurate information would be very useful for this parameter.


Any plan for Value support?

Std version serde-json has serde_json::Value type so that we can deal with any JSON even if we don't know its struct.
But It is not found in serde_json core.

For anyone else seeing this issue, this exists: https://github.com/CosmWasm/serde-cw-value


TF GRID Demo : change chatfliow into "Go to Manual"

as a user i dont understand what a 'chatfliow' is.

change chatfliow into "Go to How-to Manual"

Commit_ID f7e8cf778a66de2a280feda51cdb06c41761a594 Branch development


Lingua is too naive when parsing the Accept-Language header

Firstly, thanks for providing lingua - apart from this issue it's just what I need!
Regarding the issue, take the following Accept-Language header value: 'en-gb;q=0.8,en-us;q=0.5,en;q=0.3'. Currently lingua extracts 'en-gb' as the locale. This is fine if I have a lingua resource for the 'en-gb' locale but imagine I only have a lingua resource with 'en-us', or more likely simply 'en'. In the latter two cases the lingua will currently use the default locale which I consider a bug. Instead I think lingua should be make a locale choice based on the full header value. This means ordering the language tags by qvalue in case more than one tag matches a resource (see http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.4).
In addition it would be useful if lingua could also extract the primary subtags (i.e. the 'en' in 'en-gb') and match them against a resource (see http://www.w3.org/Protocols/rfc2616/rfc2616-sec3.html#sec3.10).
I'll attempt to provide a patch for this functionality.

Now available in release 0.2.0


Add platform to the name of binary file

Just a suggestion, the current prisma-fmt binary name is of the format
prisma-fmt-<hash> (add .exe for windows)
This led to the issue when we inadvertently had a mac binary shipped and the name exactly matched the binary name on Linux https://github.com/prisma/vscode/issues/248
Proposal: we can download the binary and add "platform" name to it. Where the platform is nodejs's os.platform: https://nodejs.org/api/os.html#os_os_platform
prisma-fmt-<platform>-<hash> (add .exe for windows)
Related issue(s):
https://github.com/prisma/vscode/issues/253

This is obsolete as we do not have platform dependent binaries in the extension any more, but a Wasm module since release 3.6.0: https://github.com/prisma/prisma/releases/tag/3.6.0 🥳


Composite HID Keyboard + HID Custom don't work

I am using STM32F1 device and the composite devices don't work
In particular I need to allow two HID interfaces on one device

1x keyboard
1x custom

The code create from your source won't start and don't enumerate the device
Do you know this matter?
Do you have any suggest?

You can add composite USB support for F4 without needing this plugin.
Just create each device one at a time using Cube MX, save all the added code.
Then create the next device...
Then add all the code back to set-up all devices.
You can get support ST on ols.st.com


raycast intersecting with panel, set colour of all connected cables

when not intersecting with a panel, all cables in world are white. when intersecting with a panel, all cables connected with that panel get coloured. maybe use a multicolour, one colour per each cable?

4.x abandoned. archived in mischmaschWeb


my data seems to be changing on editor load?

The schema I'm using to initialize the editor is examples/meta_schema.json  (which I plan to modify in the future, but I'm just using the stock json for now), and the data I'm loading in is a JSON schema a user created using this editor+meta_schema.
It is structured somewhat like this (significant edits made for brevity):
{   "type": "object",   "properties": {     "dataSource": {       "type": "object",       "properties": {         "userName": {           "type": "string"         },         "queries": {           "type": "array",           "items": {             "type": "object",             "properties": { (stuff)             },             "required": [               (stuff)             ]           },           "minItems": 1         }       }     } ....
However, the editor will not include any of the items underneath "queries":
{   "type": "object",   "properties": {     "dataSource": {       "type": "object",       "properties": {         "userName": {           "type": "string"         },         "queries": {}     } ....
Clicking on 'Edit JSON' to view the JSON, or using the getValue() method on the editor will reveal the same change. I have tried using both the startVal parameter when initializing the editor, and I've tried not using startVal and instead using setValue() right afterwards.
Moreover, once I try pasting the correct JSON into the 'Edit JSON' box and hit 'save', I get the same issue. Sometimes I will see "queries": {"type":"object"} or "queries": {"type":"string"} (where is it getting string from?!) instead of "queries": {}. The only way I have been able to get the data to display properly is by selecting 'Array' on the dropdown in the editor (see pic), and then pasting the correct data back into the 'Edit JSON' box and click save.

I went through the editor's initialization code in a debugger, and at no point did I see the data change. It seems like this only happens once the editor is actually spit out onto the webpage.
What can I do?

@danwf

have you seen this issue before when using a meta-schema?

Maybe it was, but right now, it is not an issue for me.
Set the keep_oneof_values option to TRUE and try again. And set the schema and the data when creating the editor-instance (new JSONEditor(el, {schema: schema, startval: data, keep_oneof_values: true})), this might do the trick. Though, editor.setValue() is also working for me, it's used for Undo/Redo.

@oleics Thanks. Unfortunately, no luck.

@danwf
You could create a gist containing the full schema and post the url to it here (https://gist.github.com/) ;-)

@oleics https://gist.github.com/danwf/51618e26b1f272bbad28c457bd8121c4
The exact same as examples/meta_schema.json except I commented out 'title' from 'defaultProperties'

@danwf
Sorry for not being enough specific, but what I meant was the schema created by the meta-schema. Add all three (meta-schema, created schema and the mentioned data made by the created schema) as .json-files to the gist and I'll have a look at this.

@oleics Unfortunately, I'm not able to share the full created schema; the most I can give is the problem area I've posted in the OP. And by "data" I just meant that I was trying to load the created schema into the editor.

Yeah, I would like to help you, but for that I really need to reproduce the
problem.
On Thu, Aug 25, 2016, 17:36 danwf<EMAIL_ADDRESS>wrote:

@oleics https://github.com/oleics Unfortunately, I'm not able to share
the full created schema; the most I can give is the problem area I've
posted in the OP. And by "data" I just meant that I was trying to load the
created schema into the editor.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jdorn/json-editor/issues/661#issuecomment-242432638,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAtGA6QE9pyTP-pHCVqosG24H3hySiyFks5qjbaSgaJpZM4JrJUm
.

@oleics Sorry, was away for a while.
I updated that gist again. To reproduce it, you can:

initialize a JSONEditor with 'schema' set to JsonSchemaMetaSchema (from my metaschema.txt) and 'startVal' set to the schema in "createdschema.txt."
load it on a page. You should see that what's displayed corresponds not to "createdschema.txt" but to "loadedschema.txt"
Contrast this with another JSONEditor with 'schema' set to what I have in 'blankSchema.txt'. This behavior doesn't happen.

Ok, I think I've fixed it actually. I played around with it on the sample editor and saw that there were some validation errors.
The first error was related to 'additionalProperties', so I set that to 'true' in 'schemaBase'. Then, I got some additional errors saying that it validated against 2 schemas when it expects exactly one. I took out 'No Type' from the 'oneOf' element and it finally validated. Working
This is sufficient for me because my users don't need 'No Type' as an option. If that seems like it's a common thing, maybe we should update the meta-schema that's in this repo? I have posted my working meta-schema in that same gist.

@danwf  thanks your work.
I have a problem with your working demo.  when select different type, "string", "integer" or other types.  Sometime the "type" property didn't output in the right JSON Output box.
Sometime output like this below
{
  "type": "object",
  "properties": {
    "aaa": {
      "properties": {}
    }
  }
}

It should be
{
  "type": "object",
  "properties": {
    "aaa": {
      "type": "integer",
      "properties": {}
    }
  }
}

@jinwyp same here. Maybe it will be better open a new issue addressing that.


Converted to use netstandard and dotnetcore supporting mongodb driver

Various changes to update AspNetCore.Identity.MongoDB to use the new MongoDb C# driver (pre).
Allows AspNetCore.Identity.MongoDB to run in asp.net core

Nice, I will have a look! Thanks a lot!

@mattwhetton sorry for the delay getting back 😞
This is pretty cool that you also used .NET Core version of the library. However, 2.3.0-beta1 has dependency problems which have been fixed here. I think we need to include the dependencies manually for now.
I asked the team whether they plan to push the fix to NuGet.

@mattwhetton can you add the dependencies manually?
I will try to get build working on travis CI (which I did on https://github.com/tugberkugurlu/AspNetCore.Identity.MongoDB/tree/rc2, I assume it will be probably getting it from there).

I'll look at getting the dependencies mentioned sorted and get back to you

I just added the collections dependency - are all the others required?

@mattwhetton thanks a lot for the changes!
It build locally nicely for me, I suppose that was only needed thing. I opened up #7 to make the build work which is on top of this. When that is merged, your changes will be on master and you will officially be the first contrubutor this project 🎉 🎉 👯 👯

It's in now and I pushed a version to NuGet http://www.nuget.org/packages/AspNetCore.Identity.MongoDB/1.0.0-rc2

No problem, glad to help.


.tile() is not masking region outside of image when nodata used

I'm not sure if this is a usage issue on my part or a bug, but I'm noticing that the .tile() method sometimes doesn't mask the output image when the tile extends beyond the extent of the source dataset.
All looks good for tile x=82, y=198, z=9 when not specifying nodata:
from rio_tiler.io import Reader

class PNG(bytes):
    def _repr_png_(self):
        return self

reader = Reader('https://github.com/giswqs/data/raw/main/raster/landsat7.tif')
img = reader.tile(82, 198, 9, nodata=0)
PNG(img.render(img_format="png"))

But then when I specify nodata it seems the returned image is no longer masked outside the bounds:
img = reader.tile(82, 198, 9, nodata=-1)
PNG(img.render(img_format="png"))

Originally posted in https://github.com/banesullivan/localtileserver/issues/197

@banesullivan interesting this is related to #669 where we removed WarpedVRT for nodata handling.
We know use simple rasterio.read method
https://github.com/cogeotiff/rio-tiler/blob/874f1410e04c3e46bd50f6a272340787ef39fb55/rio_tiler/reader.py#L229-L241
There might be a bug in rasterio when using boundless=True and fill_value=nodata
https://github.com/rasterio/rasterio/blob/main/rasterio/_io.pyx#L680-L735

import rasterio

with rasterio.open("tests/fixtures/cog.tif") as src_dst:
    arr = src_dst.read(
        masked=True,
        fill_value=-1,
        window=((-100,200), (-100,200)),
        boundless=True,
)

print(arr.data)

>> [[[0 0 0 ... 0 0 0]
  [0 0 0 ... 0 0 0]
  [0 0 0 ... 0 0 0]
  ...
  [0 0 0 ... 1 1 1]
  [0 0 0 ... 1 1 1]
  [0 0 0 ... 1 1 1]]]

I would expect the outside to be -1 instead of 0


Hotfix for key access on datatable when table is empty

Hotfix for key access on datatable when table is empty. Introduced in PR #2662.
Changes

Modified cfgov/jinja2/v1/_includes/organisms/table.html to fix issue rendering tables when previewing. The  to_python method is not called when previewing, so the has_data property doesn't exist.

Modified cfgov/v1/atomic_elements/organisms.py to add safer method for accessing dictionary keys.

Testing

Visit http://beta.consumerfinance.gov/policy-compliance/rulemaking/small-business-review-panels/ and observe the error.
Visit http://localhost:8000/policy-compliance/rulemaking/small-business-review-panels/ and observe the table renders as expected.

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[ ] Reviewers requested with the Assignee tool :arrow_right:

How about adding a regression unit test that covers this case?

Yep, working on that now.

@chosak, I added a regression test in https://github.com/cfpb/cfgov-refresh/pull/2691/commits/c36428d2ab4667578e90505acc8fa6943f55fee1


M3-2996 Settings option not showing in Linode Action dropdown unless backups are enabled

Description
'Settings' action was being removed if backups weren't enabled. You should now always see 'Settings' in the dropdown (and only Enable or View Backups depending on if you have backups enabled)
Type of Change

Bug fix ('fix')

Note to Reviewers
Please check linodes with and without enabled backups.

Another optional thing: there's a helper function out there somewhere for unit testing these action menus, so that you can assert that the right set of actions is present under the right conditions. If you're interested in messing with it, see VolumesActionMenu.test.tsx for examples.

agreed with @Jskobos. We should avoid magic numbers at all times.

I updated per your feedback, @Jskobos ! I agree you solution is a lot cleaner and more future-proofed than before, thank you!! I'm still working through tests, I commented out for now but want to get them working before I merge this. Could maybe also address them as a follow up.


Tone down the red to improve contrast

Pure red is too bright to be used as a background color.

Thanks!


Contour: spacing shows inf and cannot recover

To reproduce:

Load Duku-3_0_0_alpha0.nc, create a contour renderer, enable it.
In the Appearance subtab, set "count" to 1.
Spacing field shows "inf", and cannot recover...

Works for me

It's not working on at least Linux. I've updated the procedures to reproduce, and attached a screenshot.

Since @sgpearse  is working on refining the isoline selector, it'd make more sense for Scott to look at this as well. So I'm re-assigning.


Generated documentation file should contain version of k8s

Hello,
first, thanks for creating the project and also sharing the files.
Could it be possible to have the version of k8s the documentation applies on. Perhaps for last 5 supported versions a directory named based on the version (1.17, 1.16, ...) and put each file inside ?
Thanks

Thanks for the first issue :)
I will create folders and generate files for 1.18 soon

The ebooks for 1.17 and 1.18 are now put in separate folders https://github.com/tha2015/kubernetes-doc-epub


Django 1.9 support for undocumented addition of contains_aggregate

In Django commit afe0bb7 and check for a contains_aggregate is made which assumes that attribute is present.
This causes an exception when ExtraJoinRestriction is checked:
File ".../django/db/models/sql/where.py", line 157, in _contains_aggregate
    return obj.contains_aggregate
AttributeError: 'ExtraJoinRestriction' object has no attribute 'contains_aggregate'

This causes three errors in the taggit tests.
This PR sets the attribute to False in ExtraJoinRestriction which, other than preventing the error, I think is benign for Django < 1.9 and >= 1.9.

Closing in favor of https://github.com/alex/django-taggit/pull/352

Just released 0.17.2 that has Django 1.9 fixes
https://pypi.python.org/pypi/django-taggit/0.17.2


🐛 High priority: Firefox text rendering issues (next)

https://github.com/flutter/flutter/issues/71846
Makes it break very easily:

DOM Canvas has issues, no issues in canvaskit. It's being investiagted by the Flutter team.

Will be fixed by Flutter soon(ish)


How to disable the auto update check

Thanks for this awesome workflow!
But how can I disable the auto update check? Since the auto update check always occupys the first enty, and I am not planning to upgrade Alfred 5 yet.

Thanks

Please manual download Release v0.3.0 · cage1016/alfred-opencc.
set CHECK_FOR_UPDATE as 0 to disable auto check for update

Great thx!!!


Create event framework

As a followup on #15, we need to setup a structure for the content of events (see the second PR in https://github.com/RSE-leaders/SORSE20/issues/15#issuecomment-642940625).
We need to add events to the front page, which should show closest the upcoming events, and we need to add a page that shows all events.
@vsoch suggested to use a combination of fullcalendar.io and a collection, where each contribution get's one markdown page (see https://github.com/RSE-leaders/SORSE20/issues/37#issuecomment-640239723)). As no one opposed this idea and it also matches what the people talked about today during the programme team call, I think we are good to go @vsoch :smiley:
shall I come up with some draft PR that contains an events collection with some dummy events and you setup the calendar thing? We are not really in a hurry with this as we do not have submissions, yet

Yes that would be great!

alright. I won't have time before Thursday, though

maybe wednesday

@Chilipp you are unbelievably hard working on this site! Please take whatever time you need, I will be around when you are ready!

Thank you :blush:


Another one exception with init

INFO: SteamworksPy loaded for Windows
INFO: Steam is running
ERROR: Steamworks failed to initialize!
Setting breakpad minidump AppID = 480
Steam_SetMinidumpSteamID:  Caching Steam ID:  765*** [API loaded no]
code:
from steamworks import *
Steam.Init()
i'm added steam_appid.txt to main folder and its starts, but got new exception with it
I tried any Pre compiled versionos of dll, and self compiled with c++ building tools -
cl /D_USRDLL /D_WINDLL SteamworksPy.cpp steam_api64.lib /link /DLL /OUT:SteamworksPy64.dll
steel have problem

Make sure you have your steam running, by running it has to be fully loaded, Steamworks failed to initialize! most of the time will be when you forget to fully open Steam.
Just to test:
1. Open steam go to your library/store
2. Then run your application and check what you get.

@labanau is probably right.  Sounds like an issue with your Steam client.  Everything else looks OK though.

Make sure you have your steam running, by running it has to be fully loaded, Steamworks failed to initialize! most of the time will be when you forget to fully open Steam.
Just to test:
1. Open steam go to your library/store
2. Then run your application and check what you get.

Steam client program stay running.
I'm worked with facepunch.steamworks library with c# and i do not got problems with initialize.

Well, no.  The steam_appid.txt is required for games that aren't run -through- the Steam client itself because Steam doesn't know what game is being run otherwise.  When it is run through Steam, then it knows what game it is.  Hence why it crashes without it.
If you have the client fully running, the steam_appid.txt file in place, and using the pre-compiled DLL there, I'm not sure what the issue is as I cannot duplicate it.  What OS are you using?

Windows 8.1 x64

Hmm, well, I updated the steamworks.py file to give additional initialization failure messages like in my GodotSteam module so see what that tells you.
Additionally I tested things on Linux and the master branch gave me the same response you are getting:  ERROR: Steamworks failed to initialize!
That being said, the library still works fine so that error message is erroneous.  The experimental library works correctly but not sure where the difference lies that creates that false message.
Try to use any of the functionality, despite that message, and see what happens then report back.  Also see if you get a different error message, just in case.


Snippets: placeholder with both default value and transform don't work

Issue Type: Bug
	"foo": {
		"prefix": "foo",
		"body": "${1:placeholder/(.*)/${1:/downcase}/}",
		"description": "foo"
	}

When I type foo and press tab, I get placeholder/(.*)/placeholder/(.*)//downcase// right in the editor.
VS Code version: Code - Insiders 1.27.0-insider (fd610b4b6ab8d4735a4d84a2b59206ad3a9c8099, 2018-08-17T05:18:51.696Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 x 2808)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.65GB (4.11GB free)

Process Argv
C:\Programs\Microsoft VS Code Insiders\Code - Insiders.exe

Screen Reader
no

VM
0%

Extensions (11)

Extension
Author (truncated)
Version

Bookmarks
ale
9.0.3

ng-template
Ang
0.1.9

bracket-pair-colorizer
Coe
1.0.59

vscode-eslint
dba
1.4.12

EditorConfig
Edi
0.12.4

tslint
eg2
1.0.35

prettier-vscode
esb
1.6.1

angular2-switcher
inf
0.1.9

csharp
ms-
1.15.2

debugger-for-chrome
msj
4.8.2

code-spell-checker
str
1.6.10

As per this comment, seems like it's a wontfix. 😢

Is there an issue about it already? Or should this one stay open?
Why not simply swap over the parts of the syntax?
${1:placeholder/(.*)/${1:/downcase}/} → ${1/(.*)/${1:/downcase}/:placeholder}

This actually might work. I should avoid the regression described in #52931. Maybe @jrieken can comment on it, if he would accept the grammar. I'am currently on holiday, maybe i find some time to elaborate this topic.

Not a fan because we want to stay to the 'real' Textmate syntax as close as possible.


Window title still says 3.6.1 in latest version

this is 3.6.9

Whoops, I usually have things get the version number from git at compile time so I didn't think to update the hardcoded number in this. I've bumped it in 039e365f0ca44ed4f0c31b65fa79cce0210d26d6 preemptively for the next version but I'll leave this open until the next release.

Fixed with v3.6.10


SDK Version Bump

This got closed from the release last week, but needs to be merged so I can go to 0.9.96


ZEPPELIN-3278. Avoid duplicated interpreter setting

What is this PR for?
Straightforward bugfix for ZEPPELIN-3278. Just don't create interpreter setting instance when registering from template.
What type of PR is it?
[Bug Fix]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-3278

How should this be tested?

CI pass

Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@weand  Could you help try this PR ? Thanks

@weand I have committed it for 0.8.0 release, feel free to open the ticket if it doesn't resolve your issue

I can confirm that it works - tested on branch-0.8


docs: re-generate changelogs

this generated changelog was incorrect and was regenerated.
Tag is required for generation.

How did this happen? Did I give the wrong command?

we needs git tags!!!!

Ah~, got it. thanks 🙏


iOS - Lockscreen issue

Steps to reproduce:

While the app is in the foreground and playing some audio, lock the screen.

Turn on the screen again. Click play/pause on the lockscreen.

Notice that you have to click twice in order for the app to respond.

If the app is in the background before locking the screen, there is no such issue. Very weird.

I've got an app in the iOS app store that does not have this issue. Can you post some of your code for the plugin? What version are you using?

This Plugin: 1.4.1
Background Audio Plugin: Here
Ionic: 3.6.0
Ionic Native: 4.1.0
Cordova: 7.0.1
Cordova iOS: 4.4.0
Here is the code:
platform.ready().then(src => {

      this.mediaControls.subscribe().subscribe(action => {
           console.log(action);
      });

});

I have to click the play/pause button twice in order to see the console log once. The issue only happens when I lock the screen while the app is in the FOREGROUND and the audio is playing.
Thanks!

@shawnlan I think 2.1.3 may have solved some of these issues. can you try it out? Thanks!

you also shouldn't really need that background audio plugin, just enable the background audio capability through xcode, it just takes a click and you're good to go.

Good to know! I'll give it a try. Thank you guys.

I have a similar issue.
The progress bar also does not update when using (updateElapsed)

I fix it using the cordova-plugin-background-mode


[QUESTION] I want to convert a cmake to autotools, is this possible?

[QUESTION] I want to convert a cmake to autotools, is this possible?

No, sorry. This works the other way around, to convert an autotools project to cmake.


[BUG] <In the threshold setting, the notification template environment variable display error>

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
在阈值设置中的 通知模版环境变量 显示错误

Expected Behavior
No response
Steps To Reproduce
No response
Environment
HertzBeat version(s):v1.3.2

Debug logs
No response
Anything else?
No response

hi 你选择了什么阈值指标出现的这种情况

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

Hi, what threshold indicator did you choose for this situation?

hi 你选择了什么阈值指标出现的这种情况

多谢回复，就目前我这边测试到的情况而言，随便选择一个指标都会出现该情况，只要不是选择“监控可用性”即可

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

hi What threshold indicator did you choose for this situation?

Thank you for your reply. As far as I am currently testing, this situation will occur if you choose any indicator, as long as you do not choose "monitoring availability".

收到🫡 谢谢反馈，我们处理下

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

Received 🫡 Thank you for your feedback, we will deal with it


ThemeService requires hundreds of listeners to implement theming

Running integration tests out of sources:
"[284] potential listener LEAK detected, having 551 listeners already. MOST frequent listener (25):", source: file:///Users/bpasero/Development/Microsoft/monaco/out/vs/base/common/event.js (366)
[93944:0216/145956.860378:INFO:CONSOLE(367)] "    at new SuggestWidget (file:///Users/bpasero/Development/Microsoft/monaco/out/vs/editor/contrib/suggest/suggestWidget.js:167:49)
    at InstantiationService._createInstance (file:///Users/bpasero/Development/Microsoft/monaco/out/vs/platform/instantiation/common/instantiationService.js:89:20)
    at InstantiationService.createInstance (file:///Users/bpasero/Development/Microsoft/monaco/out/vs/platform/instantiation/common/instantiationService.js:60:31)
    at file:///Users/bpasero/Development/Microsoft/monaco/out/vs/editor/contrib/suggest/suggestController.js:83:59
    at IdleValue._executor (file:///Users/bpasero/Development/Microsoft/monaco/out/vs/base/common/async.js:673:35)
    at file:///Users/bpasero/Development/Microsoft/monaco/out/vs/base/common/async.js:682:59", source: file:///Users/bpasero/Development/Microsoft/monaco/out/vs/base/common/event.js (367)

Not a leak. This is the theme service which seems to require zillions of listeners so that theming works. Assigning back to you and @aeschli to consider a solution that doesn't need hundreds of listeners

The limit is already at 400 https://github.com/microsoft/vscode/blob/684f61b4567ade085e320d9c686ae3d25c60209a/src/vs/workbench/services/themes/browser/workbenchThemeService.ts#L123

Every UI component that depends on a themable color also needs to listen to theme changes. We have lots of UI part so > 400 listeners is feasible. I don't think there's a problem with that and refactoring the code just to keep that number down is not a must.
The number of listeners alone can not be the reason alone for a LEAK warning. Does it also check that the number always rises, over a long period of time?
Perfect would be if we can detect that the listener is the reference that prevents an object from being collected.

Does it also check that the number always rises, over a long period of time?

It is not a leak, the number doesn't rise only over the months as we add more features

I don't think there's a problem with that and refactoring the code just to keep that number down is not a must.

I disagree. There is obviously no right number but if you have 400 listeners and each listener takes 1ms to do its thing then you have a severe perf problem. Today (didn't count listener) it takes ~100ms on my machine.
I would say that 99% of the theme listeners adjust a color or a border-style, so this my random idea: the theme service could support css variables for which it knows what theme color defines the respective values. When a theme changes it can simply update those variable values, like a better supported variant of things like this

Yes, css-variables could help in some cases to reduce the number of listeners and are a good idea.
If you go through our theming code you will see that many places logic is needed: If a certain color is set, use color, if not set border of that some width and color. An example for that is in the code that you linked to.
I don't know how much is possible there with css variables.
Also I don't know that from a performance standpoint, if it is faster to use CSS variables or directly set colors/borders on dom nodes..
Also the evaluation of CSS variables can not be free for the renderer and might have to happen on every stylesheet change, vs the current solution where we only do work when when the user changes a theme or customizes a color.


Why Zemax CSS

https://main--zemax--hlxsites.hlx.page/pages/why-zemax

Fix awkward curve to look like the curve that is displayed to look like the original design. (see image below)

Note: the curve is very important for our brand

Add in additional spacing to separate the separate the sections (possibly by 5-10px)
The font for the description is too big and needs to be slightly smaller. Also the content needs to be moved up so that the title is aligned with the top of the image.
The "learn more" link doesnt link to it's respective page (https://www.zemax.com/pages/the-optical-design-industry-is-moving-forward)
Add in additional spacing to separate the separate the sections (possibly by 5-10px)

Repeat the steps above for all the respective sections underneath (Products, Multi-physical, Careers, Connect)
Also, the"learn more" links for each respective section is missing so I will include the proper links for each section below:

Products (learn more): https://www.zemax.com/resources/eguides/designing-for-manufacturability
Multi-physics (learn more): https://www.ansys.com/blog/ansys-helps-innoviz-technologies-build-a-better-automotive-lidar
Careers (learn more): https://www.zemax.com/pages/careers
Connect (contact us): https://www.zemax.com/pages/contact-us

Be sure that each logo is equally horizontally aligned across all 3 boxes
Be sure that the body content is equally horizontally aligned across all 3 boxes
Be sure that the CTA buttons are equally horizontally aligned across all 3 boxes

[ Mobile ] https://www.zemax.com/pages/why-zemax

Add padding to ensure the title is not stuck to the left side
Slide feature does not work

Switch the positioning of the image and the text so that the text appears first.
Add padding to both sides of the image and text so they are not pressed up against the sides (make this change for all similar components on the page)

Change from button, to general hyperlink.


Update alpine:3.20 Docker digest to e72ad07

This PR contains the following updates:

Package
Type
Update
Change

alpine
final
digest
beefdbd -> e72ad07

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Branch automerge failure
This PR was configured for branch automerge. However, this is not possible, so it has been raised as a PR instead.

Branch has one or more failed status checks


Clicking on a category crashes the app

When looking at an image file and clicking on one of its categories, the app crashes and shuts down.

@picsi Does this happen on latest master or on one of the market builds?

I'm using the latest from F-Droid.

Can you share logs from your device?

Not entirely sure how to do that. I'm on Lineage 15.1 on mido, if that helps.

Just click on Settings > Send logs from within the app. It will open an email client and with the log files attached.

Am closing this issue as its not reproducible. Feel free to reopen it if the issue persists. :)


How To Correctly Implement React GoogleOAuth

I am trying to implement the React GoogleOAuth. I currently have installed the @React-oauth/google library. When trying to import the GoogleLogin component from the library I am getting an parsing error from the console.

Can you share with me your package.json deps

"dependencies": {
"@azure/msal-browser": "^2.18.0",
"@emotion/core": "^10.0.14",
"@emotion/styled": "^10.0.14",
"@sentry/react": "^6.19.6",
"@sentry/tracing": "^6.19.6",
"autoprefixer": "^7.1.1",
"babel-core": "^6.24.1",
"babel-eslint": "^7.1.1",
"babel-jest": "^15.0.0",
"babel-loader": "^7.1.1",
"babel-plugin-syntax-dynamic-import": "^6.18.0",
"babel-plugin-transform-class-properties": "^6.24.1",
"babel-plugin-transform-object-rest-spread": "^6.8.0",
"babel-polyfill": "^6.23.0",
"babel-preset-env": "^1.5.1",
"babel-preset-react": "^6.11.1",
"cancan": "^3.0.0",
"chunk-manifest-webpack-plugin": "^1.1.2",
"classnames": "^2.2.5",
"compression-webpack-plugin": "^1.0.0",
"copy-webpack-plugin": "^4.0.1",
"css-loader": "^0.28.2",
"eslint": "3.19.0",
"eslint-config-airbnb": "^14.1.0",
"eslint-import-resolver-webpack": "^0.8.3",
"eslint-plugin-import": "^2.2.0",
"eslint-plugin-jsx-a11y": "^4.0.0",
"eslint-plugin-react": "^6.10.0",
"expose-loader": "^0.7.3",
"extract-text-webpack-plugin": "^3.0.0",
"file-loader": "^0.11.1",
"flexbox-react": "^4.4.0",
"immutability-helper": "~2.0.0",
"intro.js-react": "^0.3.0",
"isomorphic-fetch": "^2.2.1",
"jest": "^15.1.1",
"js-yaml": "^3.8.4",
"jsuri": "^1.3.1",
"jwt-autorefresh": "^0.2.4",
"jwt-decode": "^2.0.1",
"lodash": "^4.13.1",
"moment": "^2.22.2",
"moment-business-days": "^1.0.6",
"moment-range": "2.2.0",
"mousetrap": "^1.6.1",
"nock": "^8.0.0",
"normalize.css": "^4.1.1",
"normalizr": "^2.2.1",
"path-complete-extname": "^0.1.0",
"postcss": "^6.0.9",
"postcss-cssnext": "^3.0.2",
"postcss-loader": "^1.3.3",
"postcss-smart-import": "^0.7.2",
"precss": "^1.4.0",
"prop-types": "^15.6.2",
"qs": "^6.3.0",
"rails-erb-loader": "^5.1.0",
"react": "^16.0.0",
"react-addons-css-transition-group": "^15.4.2",
"react-addons-perf": "^15.4.2",
"react-another-signature-pad": "^0.9.2",
"react-confirm": "^0.1.14",
"react-css-themr": "^2.1.0",
"react-data-export": "^0.5.0",
"react-dnd": "^2.1.4",
"react-dnd-html5-backend": "^2.1.2",
"react-dom": "^16.0.0",
"react-dropzone": "^3.6.0",
"react-google-login": "^5.0.0",
"react-helmet": "^5.1.3",
"react-hot-loader": "next",
"react-infinite": "^0.12.1",
"react-joyride": "^2.3.0",
"react-redux": "^5.0.2",
"react-responsive": "^1.1.3",
"react-router": "^3.0.5",
"react-router-redux": "^4.0.5",
"react-router-scroll": "^0.4.1",
"react-sanfona": "^0.2.3",
"react-select": "^1.0.0-rc",
"react-signature-canvas": "^1.0.1",
"react-signature-pad": "^0.0.6",
"react-tagsinput": "^3.13.6",
"react-tap-event-plugin": "^1.0.0",
"react-test-renderer": "^15.3.2",
"react-toast-notifications": "^2.4.4",
"react-toolbox": "^2.0.0-beta.12",
"react-tooltip": "^3.2.1",
"redbox-react": "^1.2.9",
"redux": "^3.5.2",
"redux-actions": "^0.12.0",
"redux-auth-wrapper": "^2.0.2",
"redux-devtools": "^3.3.2",
"redux-thunk": "^2.1.0",
"redux-ui": "^0.1.1",
"remote-redux-devtools": "^0.5.16",
"reselect": "^2.5.3",
"resolve-url-loader": "^2.0.2",
"rsvp": "^3.5.0",
"sass": "^1.43.4",
"sass-loader": "^7.2.0",
"signature_pad": "^2.2.1",
"sinon": "^1.17.4",
"style-loader": "^0.18.1",
"tableau-api-js": "^2.8.1",
"tap-spec": "^4.1.1",
"tape": "^4.6.0",
"toolbox-loader": "0.0.3",
"webpack": "^3.10.0",
"webpack-manifest-plugin": "^1.1.0",
"webpack-merge": "^4.1.0",
"worker-loader": "^0.8.1",
"xlsx": "^0.14.0"
}

Upgrade you react and react scripts versions to support hooks at least react 16.8

We are currently using class based comps and their are a lot of method we are currently using that are now deprecated so upgrading would be a lot more extensive. Is there anything else that you would recommend?

You can copy paste the implementation from the source code, and build your own component


Service Type Compatibility Changes

$subject.

Fixed with the following PRs.

https://github.com/ballerina-platform/module-ballerina-email/pull/41
https://github.com/ballerina-platform/module-ballerina-email/pull/43


[Hub Generated] Review request for Microsoft.ContainerInstance to add version stable/2020-10-19

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
Contribution checklist:

[x] I commit to follow the Breaking Change Policy of “no breaking changes
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

/azp run automation - sdk

While adding new version the recommendation is to copy the existing version into new directory structure for first commit and then push new changes for easy review. Please create new branch with this recommendation for faster review.

No activity for a long time. please re-add ARM feedback label if you need a review from ARM. removing to reduce noise.

If you want to merge this PR please open another one since this PR has no recent activity.


NamedArgs for obfuscated custom attribute

Named arg for custom attribute includes field/property name in their encoding ECMA-335, II.23.3, p.269.
So, when/if named argument field/property name obfuscated, encoding of custom attributes using it should be updated.
Here is test case:
using System;
using System.Reflection;

namespace ConsoleApp1
{
    [AttributeUsage(AttributeTargets.Field)]
    public class TestAttribute : System.Attribute
    {
        public bool testField = false;
    }

    [Obfuscation(Exclude = true, ApplyToMembers = true)]
    class Program
    {
        [Test(testField = true)]
        public static string FieldToReproduceProblem;

        static void Main(string[] args)
        {
            var item = (TestAttribute)Attribute.GetCustomAttribute(typeof(Program).GetField(nameof(FieldToReproduceProblem)),
                typeof(TestAttribute));
            Console.WriteLine(item.testField);
        }
    }
}

<?xml version='1.0' encoding="utf-8"?>
<Obfuscator>
  <Var name="InPath" value=".\" />
  <Var name="OutPath" value=".\Output\" />
  <Var name="HideStrings" value="false" />
  <Var name="KeepPublicApi" value="false" />
  <Var name="OptimizeMethods" value="false" />
  <Var name="SuppressIldasm" value="false" />

 <Module file="$(InPath)\ConsoleApp2.exe" />
 </Obfuscator>

In provided app call to Attribute.GetCustomAttribute will fail due incorrect attribute encoding. testField was renamed, but FieldToReproduceProblem still looks:
.field public static string FieldToReproduceProblem
.custom instance void A.A::.ctor() = ( 01 00 01 00 53 02 09 74 65 73 74 46 69 65 6C 64   // ....S..testField
                                       01 )

Thanks. As Mono Cecil does not automatically associate the two parts, renaming the attribute class (and its field) would lead to broken pieces like you reported.
To fix that, either Mono.Cecil would be improved or Obfuscar should add some post-processing. Would require some efforts, while I am not quite available recently.
Welcome pull requests if anyone is willing to try.


requirements.txt should include more

When I use Charts.ipynb, it requires cufflinks and tablib. (pip -r requirements.txt was not enough)
I'll send pull request for this issue later.

cufflinks is currently broken, and the old version is not available on pypi.
Please use the Dockerfile provided, as it contains the workaround.


Use http.JsonResponse when the response is JSON

Warning this is untested. I have splinter errors when running tox.
http.HttpResponse with manual json.dumps call is, well, kind of acceptable. Unless you forget to set content_type to „application/json”. But then, everything still is fine, tests pass and stuff. But then, after some time, you run the code in the production, on, say, nginx+uwsgi. This combo will attach HTML tags to your json responses, head, body, the html tag itself, thus rendering json responses as unreadable to the client.
TL;DR: Use http.JsonResponse when the response is JSON

(I actually created this PR in hope that TravisCI will handle the testing for me xfingersx xcrossedx)

Works for me ;)
Go ahead and merge when you're ready

@jpic do you want me to merge this myself, did I understand correctly?

@mpasternak yes, when you're ready ;)


Map Clustener

Will add the 'map clustener'?
https://github.com/googlemaps/js-marker-clusterer

+1, would like this as well

I should actually do more than that: Update codebase, fix issues and add some plugins like I did with leaf-let. :+1:
Give me some time guys. Apologies for the delay.

https://github.com/2amigos/yii2-google-maps-library/issues/52#issuecomment-228899879


Added commit status fetching functionality

Basically I needed this functionality (and a few other things I'll be adding), but figured I might as well try do it "properly". Let me know what needs changing :smile:

Hmmm, a Swift spec is failing. As far as I can tell that must have been an existing issue.
/Users/travis/build/octokit/octokit.objc/OctoKitTests/SwiftSpec.swift:11:8: error: module file's minimum deployment target is ios9.0 v9.0: /Users/travis/Library/Developer/Xcode/DerivedData/OctoKit-cjajgpsfwpostubkfrhudkncaxub/Build/Products/Debug-iphonesimulator/Quick.framework/Modules/Quick.swiftmodule/i386.swiftmodule
import Quick
       ^
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━���━━━━
      ✗ Compile Swift source files (2466 ms)

Thanks @Benjamin-Dobell! Looks like CI is passing now. Just one question.

Regarding the points above suggesting the use of enums over strings; we definitely want some string constants, but should avoid the use of enums. I was actually going to bring-up the problematic use of enums in another pull request that I'll be submitting very shortly (webhook support).

Good points in the other PR :+1: String constants sound good.

Good points in the other PR :+1: String constants sound good.

Thanks, not all those points apply here - because we wouldn't be using a bitfield here, just a regular enum. Nonetheless, it does still have limitations if a new state is introduced: we'd either choke during transformation on the unknown state, or present inaccurate information to API consumers.
I'll definitely add some string constants. Thanks for your time/feedback!

Good points

Sorry for the delay @Benjamin-Dobell! I didn't realize the PR had been updated. Thanks for your work!


Stack: Support version numbers followed by a space

And provide a few real-world version string examples.
Signed-off-by: Sebastian Schuberth<EMAIL_ADDRESS>

/rebase


[BUG] Expected a tensordict with shape==env.shape, got torch.Size([1]) and torch.Size([])

Describe the bug
What is the env.shape ? Does not appear to be defined anywhere
To Reproduce
import torchrl
env = GymEnv("CartPole-v1",  device=device)
tdact = TensorDict({"action":torch.tensor([1])}, batch_size=(1,))
env.step(tdact)

Traceback (most recent call last):
  File ...
*** RuntimeError: Expected a tensordict with shape==env.shape, got torch.Size([1]) and torch.Size([])

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
No stacktrace
System info
Describe the characteristic of your environment:

Describe how the library was installed (pip, source, ...)
Python version
Versions of any other relevant libraries

import torchrl, numpy, sys
print(torchrl.__version__, numpy.__version__, sys.version, sys.platform)

0.3.0 1.26.1 3.10.13 (main, Sep 11 2023, 13:44:35) [GCC 11.2.0] linux

Additional context
Add any other context about the problem here.
Reason and Possible fixes
If you know or suspect the reason for this bug, paste the code lines and suggest modifications.
Checklist

[x ] I have checked that there is no similar issue in the repo (required)
[x ] I have read the documentation (required)
[ x] I have provided a minimal working example to reproduce the bug (required)

The error comes from the fact that the tensordict should have an empty batch size like the env. Shape and batch size are synonymous but I agree the error is misleading

@vmoens , thanks for the tip. With  tdact = TensorDict({"action":torch.tensor([1])}, batch_size=torch.Size([])) , the code snippet did not throw that  error. Agree that  the "error" message is confusing, it being not even an error. What is the reason for that check to be there ?

Because we want to enforce that the leading dims of the tensors match the environment batch size (and therefore comply with the specs).
If your tensordict has a batch size of [1] it means that all its content has a leading dimension of [1] and hence that you're running an environment that has a batch size (even if it's just 1).
Once you start executing envs in parallel, we build circular buffers to pass the data across the workers, using the env specs as indicators of the shape of the buffers. If you have data that is not properly shaped using some conventions it's easy to break things as the data won't fit in the buffers.


Mosaic failed to compile TPU kernel

Description
I am trying to fine-tune Gemma 2 on TPU and got the following error:
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/compiler.py", line 266, in backend_compile
    return backend.compile(built_c, compile_options=options)
jaxlib.xla_extension.XlaRuntimeError: INTERNAL: Mosaic failed to compile TPU kernel: Unsupported input data type in matrix multiplication.

at location: loc("/dot_general"(callsite("_splash_attention"("/usr/local/lib/python3.10/dist-packages/jax/experimental/pallas/ops/tpu/splash_attention/splash_attention_kernel.py":2277:0) at callsite("__call__"("/usr/local/lib/python3.10/dist-packages/jax/experimental/pallas/ops/tpu/splash_attention/splash_attention_kernel.py":2312:0) at callsite("wrap_flash_attention"("/home/cantonese-gemma2/maxtext/MaxText/layers/attentions.py":352:0) at callsite("tpu_flash_attention"("/home/cantonese-gemma2/maxtext/MaxText/layers/attentions.py":358:0) at callsite("_call_wrapped_method"("/usr/local/lib/python3.10/dist-packages/flax/linen/module.py":1211:0) at callsite("wrapped_module_method"("/usr/local/lib/python3.10/dist-packages/flax/linen/module.py":694:0) at callsite("apply_attention"("/home/cantonese-gemma2/maxtext/MaxText/layers/attentions.py":234:0) at callsite("_call_wrapped_method"("/usr/local/lib/python3.10/dist-packages/flax/linen/module.py":1211:0) at callsite("wrapped_module_method"("/usr/local/lib/python3.10/dist-packages/flax/linen/module.py":694:0) at "__call__"("/home/cantonese-gemma2/maxtext/MaxText/layers/attentions.py":977:0))))))))))))

The MLIR operation involved:
  %4183 = "tpu.matmul"(%4179, %4181, %4182) <{transpose_lhs = false, transpose_rhs = true}> : (vector<512x128xbf16>, vector<128x128xbf16>, vector<512x128xf32>) -> vector<512x128xf32>
... additional diagnostics were skipped.

System info (python version, jaxlib version, accelerator, etc.)
Python 3.10.12
jaxlib 0.4.33
TPU v3-8

These are the subsequent errors:
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/compiler.py", line 266, in backend_compile
    return backend.compile(built_c, compile_options=options)
jaxlib.xla_extension.XlaRuntimeError: INTERNAL: Mosaic failed to compile TPU kernel: Unsupported input data type in matrix multiplication.

at location: loc("/dot_general"(callsite("_splash_attention"("/usr/local/lib/python3.10/dist-packages/jax/experimental/pallas/ops/tpu/splash_attention/splash_attention_kernel.py":2277:0) at callsite("__call__"("/usr/local/lib/python3.10/dist-packages/jax/experimental/pallas/ops/tpu/splash_attention/splash_attention_kernel.py":2312:0) at callsite("wrap_flash_attention"("/home/cantonese-gemma2/maxtext/MaxText/layers/attentions.py":352:0) at callsite("tpu_flash_attention"("/home/cantonese-gemma2/maxtext/MaxText/layers/attentions.py":358:0) at callsite("_call_wrapped_method"("/usr/local/lib/python3.10/dist-packages/flax/linen/module.py":1211:0) at callsite("wrapped_module_method"("/usr/local/lib/python3.10/dist-packages/flax/linen/module.py":694:0) at callsite("apply_attention"("/home/cantonese-gemma2/maxtext/MaxText/layers/attentions.py":234:0) at callsite("_call_wrapped_method"("/usr/local/lib/python3.10/dist-packages/flax/linen/module.py":1211:0) at callsite("wrapped_module_method"("/usr/local/lib/python3.10/dist-packages/flax/linen/module.py":694:0) at "__call__"("/home/cantonese-gemma2/maxtext/MaxText/layers/attentions.py":977:0))))))))))))

The MLIR operation involved:
  %4473 = "tpu.matmul"(%4469, %4471, %4472) <{transpose_lhs = false, transpose_rhs = true}> : (vector<512x128xbf16>, vector<128x128xbf16>, vector<512x128xf32>) -> vector<512x128xf32>
... additional diagnostics were skipped.

Please report a bug at: https://github.com/google/jax/issues/new?assignees=apaszke

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/cantonese-gemma2/maxtext/MaxText/train.py", line 774, in <module>
    app.run(main)
  File "/usr/local/lib/python3.10/dist-packages/absl/app.py", line 308, in run
    _run_main(main, args)
  File "/usr/local/lib/python3.10/dist-packages/absl/app.py", line 254, in _run_main
    sys.exit(main(argv))
  File "/home/cantonese-gemma2/maxtext/MaxText/train.py", line 770, in main
    train_loop(config)
  File "/home/cantonese-gemma2/maxtext/MaxText/train.py", line 665, in train_loop
    state, metrics = p_train_step(state, example_batch, nextrng)
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/traceback_util.py", line 180, in reraise_with_filtered_traceback
    return fun(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/pjit.py", line 332, in cache_miss
    outs, out_flat, out_tree, args_flat, jaxpr, attrs_tracked = _python_pjit_helper(
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/pjit.py", line 190, in _python_pjit_helper
    out_flat = pjit_p.bind(*args_flat, **p.params)
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/core.py", line 2782, in bind
    return self.bind_with_trace(top_trace, args, params)
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/core.py", line 443, in bind_with_trace
    out = trace.process_primitive(self, map(trace.full_raise, args), params)
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/core.py", line 949, in process_primitive
    return primitive.impl(*tracers, **params)
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/pjit.py", line 1739, in _pjit_call_impl
    return xc._xla.pjit(
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/pjit.py", line 1721, in call_impl_cache_miss
    out_flat, compiled = _pjit_call_impl_python(
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/pjit.py", line 1651, in _pjit_call_impl_python
    ).compile(compile_options)
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/interpreters/pxla.py", line 2313, in compile
    executable = UnloadedMeshExecutable.from_hlo(
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/interpreters/pxla.py", line 2827, in from_hlo
    xla_executable = _cached_compilation(
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/interpreters/pxla.py", line 2639, in _cached_compilation
    xla_executable = compiler.compile_or_get_cached(
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/compiler.py", line 426, in compile_or_get_cached
    return _compile_and_write_cache(
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/compiler.py", line 654, in _compile_and_write_cache
    executable = backend_compile(
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/profiler.py", line 333, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/jax/_src/compiler.py", line 271, in backend_compile
    raise handler_result from e
  File "/home/cantonese-gemma2/maxtext/MaxText/layers/attentions.py", line 977, in __call__
    prefill_unnormalized_output, prefill_exponentials_max, prefill_exponentials_sum = self.apply_attention(
  File "/home/cantonese-gemma2/maxtext/MaxText/layers/attentions.py", line 234, in apply_attention
    return self.tpu_flash_attention(query, key, value, decoder_segment_ids, self.attn_logits_soft_cap), None, None
  File "/home/cantonese-gemma2/maxtext/MaxText/layers/attentions.py", line 358, in tpu_flash_attention
    x = wrap_flash_attention(query, key, value, decoder_segment_ids)
  File "/home/cantonese-gemma2/maxtext/MaxText/layers/attentions.py", line 352, in wrap_flash_attention
    return jax.vmap(splash_kernel)(query, key, value, segment_ids=decoder_segment_ids)
  File "/usr/local/lib/python3.10/dist-packages/jax/experimental/pallas/ops/tpu/splash_attention/splash_attention_kernel.py", line 2312, in __call__
    return _splash_attention(
  File "/usr/local/lib/python3.10/dist-packages/jax/experimental/pallas/ops/tpu/splash_attention/splash_attention_kernel.py", line 2277, in _splash_attention
    return _splash_attention_custom(
jax._src.pallas.mosaic.error_handling.MosaicError: INTERNAL: Mosaic failed to compile TPU kernel: Unsupported input data type in matrix multiplication.

The MLIR operation involved:
  %4473 = "tpu.matmul"(%4469, %4471, %4472) <{transpose_lhs = false, transpose_rhs = true}> : (vector<512x128xbf16>, vector<128x128xbf16>, vector<512x128xf32>) -> vector<512x128xf32>
... additional diagnostics were skipped.

While Pallas demonstrates strong potential, certain limitations arise when deploying its operations on TPUv3 hardware. Initial observations suggest potential incompatibilities and accuracy discrepancies that require further investigation.
Firstly, some Pallas operations may not yet be fully supported on TPUv3, leading to execution failures. Identifying these specific operations is crucial for either seeking alternative implementations or advocating for broader TPUv3 support.
Secondly, the use of bfloat16 precision for QKV computations, while potentially efficient, might contribute to the observed numerical inaccuracies. Exploring the impact of switching to float32 precision is recommended, although it may not completely resolve the discrepancies.

This matmul is not supported on TPU V3 (bf16 X bf16 with float32 accumulation). You should instead try to cast the inputs up the float32 before the matmul.
That being said, we're working on improving the error reporting in these cases since it will be much more clear if the error clearly stated that the operation was not supported on a specific hardware generation.


Cleanup

Sorry. Did not intend to submit an upstream PR! I've been working too long today apparently. :(

😂 Thank would have been one hell of a PR! Take it easy 😊


Use hardware based device id

Use a device id which is based on hardware (e.g. wifi mac address) to allow the same firmware to be deployed to multiple devices.

Resolved by https://github.com/thin-edge/freertos-esp32-client/pull/10


[2pt] Add unit tests for all implemented methods - Task3 - [TM1]

Add unit tests for all implemented methods.

#35


[2.2.x backport] Add the proxy

This is a somewhat messy backport because the testing code changed out from under us.  I think I cherry-picked the right PR to make everything work though.

Going to try this again.


broken

invalid import at Line 22
crashes at Line 80 when changing the invalid module to SET_ACTIVITY from INVITE_BROWSER

Thanks for porting the fix over, I had talked with e-boi about it already but hadn't gotten to it


NPM issue installing @microsoft/generator-sharepoint

Hi,
While installing @microsoft/generator-sharepoint using npm I'm getting following error:
C:\>Npm install -g @microsoft/generator-sharepoint
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.
npm WARN deprecated<EMAIL_ADDRESS>connect 2.x series is deprecated
npm WARN deprecated<EMAIL_ADDRESS>not actively maintained anymore
npm WARN deprecated<EMAIL_ADDRESS>Use uuid module instead
npm WARN deprecated<EMAIL_ADDRESS>ReDoS vulnerability parsing Set-Cookie https://nodesecurity.io/advisories/130
npm ERR! tar.unpack untar error C:\Users\useraccount\AppData\Roaming\npm-cache\node-forge\0.7.1\package.tgz
C:\Users\useraccount\AppData\Roaming\npm
`-- (empty)
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules@microsoft\generator-sharepoint\node_modules\chokidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm ERR! Windows_NT 10.0.10586
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Users\useraccount\AppData\Roaming\npm\node_modules\npm\bin\npm-cli.js" "install" "-g" "@microsoft/generator-sharepoint"
npm ERR! node v6.11.5
npm ERR! npm  v3.10.10
npm ERR! path C:\Users\useraccount\AppData\Roaming\npm\node_modules.staging\node-forge-29ac5de4\flash\swf\SocketPool.swf
npm ERR! code ENOENT
npm ERR! errno -4058
npm ERR! syscall lstat
npm ERR! enoent ENOENT: no such file or directory, lstat 'C:\Users\useraccount\AppData\Roaming\npm\node_modules.staging\node-forge-29ac5de4\flash\swf\SocketPool.swf'
npm ERR! enoent ENOENT: no such file or directory, lstat 'C:\Users\useraccount\AppData\Roaming\npm\node_modules.staging\node-forge-29ac5de4\flash\swf\SocketPool.swf'
npm ERR! enoent This is most likely not a problem with npm itself
npm ERR! enoent and is related to npm not being able to find a file.
npm ERR! enoent
npm ERR! Please include the following file with any support request:
npm ERR!     C:\npm-debug.log
npm ERR! code 1
Here are my node & NPM version details
C:\>node -v
v6.11.5
C:\>npm -v
3.10.10

Please let me know what could be the issue here
Thanks,
Sanju

Have you tried installing the generator again to verify this wasn't a one-time issue?

Hi Waldek,
I have tried installing @microsoft/generator-sharepoint multiple times on my system but I got the same error.
So I took "node_modules" including @Microsoft folder from my colleague's system and copied the same to my project "HelloWorld" local folder and it started working.

I have reopened this ticket, as above solution is just a work-around. I need to know why @microsoft/generator-sharepoint module is not getting installed on my system and how do I check the same?

Have you tried running the command prompt as admin?

Yes ..I'm running the command prompt as admin.
Another observation, after copying node_module to local folder of my sharepoint web part project. I'm able to do "gulp serve" and it opens the SharePoint workbench.
But if I move the same node module contents to "C:\Users\user-name\AppData\Roaming\npm" then runs gulp server, it says
Local gulp not found in [current_folder_path]
Try running: npm install gulp
Why its looking for local gulp when its there in global path?

That's specific to gulp and how it works: it always require you to be installed both globally and locally.

On another note: you should not move your local node_modules folder to the location where globally installed packages are. The only package you have to have installed globally is (at)microsoft/generator-sharepoint. In your project however, you have installed all dependencies required for your project to run.

Now I installed gulp locally, npm install gulp. It successfully got installed
if I run gulp serve, it throws error "Error: Cannot find module '@microsoft/sp-build-web'

But @microsoft/sp-build-web is already there in global folder. This means, for some reason it always looking for all the modules in local folder only. What could be the issue?

are you still having the node_modules folder in C:\Users\user-name\AppData\Roaming\npm?

Hey Waldek,
Just stumbled upon this post. Apparently I am also facing the same issue. There is a small difference, I am getting this when I create SPFx project. The strange thing is, out of 10 attempts, it works at times.
Had tried all the combinations.

Trying npm cache clear  (Running as an admin).

These did not help either. Any idea what might be stopping this from creating the project.
Help is appreciated

@arjunumenon what error are you getting exactly?

Hey @waldekmastykarz ,
During the execution, the first error which I am getting is,
npm ERR! tar.unpack untar error C:\Users\username\AppData\Roaming\npm-cache\node-forge\0.7.1\package.tgz
And towards the end of execution, following are the errors which are there at the bottom.
npm ERR! Windows_NT 10.0.10586
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Users\username\AppData\Roaming\npm\node_modules\npm\bin\npm-cli.js" "install" "--cache-min" "86400"
npm ERR! node v6.12.0
npm ERR! npm  v3.10.10
npm ERR! path D:\blah\blah\SampleWP\RnD\node_modules.staging\node-forge-dc10c1f7\flash\swf\SocketPool.swf
npm ERR! code ENOENT
npm ERR! errno -4058
npm ERR! syscall lstat
npm ERR! enoent ENOENT: no such file or directory, lstat 'D:\blah\SpFx\SampleWP\RnD\node_modules.staging\node-forge-dc10c1f7\flash\swf\SocketPool.swf'
npm ERR! enoent ENOENT: no such file or directory, lstat 'D:\blah\SpFx\SampleWP\RnD\node_modules.staging\node-forge-dc10c1f7\flash\swf\SocketPool.swf'
npm ERR! enoent This is most likely not a problem with npm itself
npm ERR! enoent and is related to npm not being able to find a file.
Let me know if you need any more information on the same please.

#1115 I Have created another issue in Git. We can converse in that if you are fine with that.

Closing this issue as #1115 was fixed.


Failure to detect next line beginning white space for Korean language

Environment
tesseract 4.00 alpha recent released traineddata.
32&64 bit.
Current Behavior:
Tesseract is unable to detect next line white spaces.
For example:
가나다라
마바사아
자차카타
파하
If you run text recognition on above example and add the strings then it shows as,
가나다라마바사아자차카타파하
instead of
가나다라 마바사아자차카타파하
Expected Behavior:
Correct OCR should be 가나다라 마바사아자차카타파하
from the above example.

Please provide the input image you used. Without it, we won't able to help you.
.

Please see

https://stackoverflow.com/questions/41381162/tesseract-ocr-line-breaks-on-command-line
This could be related to the difference in line endings for Unix vs those
for Windows.
On Fri 10 Aug, 2018, 12:29 AM Amit D<EMAIL_ADDRESS>wrote:
Please provide the input image you used. Without it, we won't able to help
you.
.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tesseract-ocr/tesseract/issues/1829#issuecomment-411861861,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE2_o2pEV2cUQGagni26BBUYwBVdxdr7ks5uPIaDgaJpZM4V1Z1l
.
On Fri 10 Aug, 2018, 12:29 AM Amit D<EMAIL_ADDRESS>wrote:
Please provide the input image you used. Without it, we won't able to help
you.
.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tesseract-ocr/tesseract/issues/1829#issuecomment-411861861,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE2_o2pEV2cUQGagni26BBUYwBVdxdr7ks5uPIaDgaJpZM4V1Z1l
.


Name is too long

Should consider renaming from "OrganizationSnapshots.jl" to "Snapshots.jl"

Done.


text-field: allow to disable the Chrome autocomplete

Reference: https://developer.mozilla.org/en-US/docs/Web/Security/Securing_your_site/Turning_off_form_autocompletion

never mind, it works with [autocomplete]="null".

https://bugs.chromium.org/p/chromium/issues/detail?id=468153#c164

Upstream team working on resolving.
reference issue and PR:
https://github.com/material-components/material-components-web/issues/4447
https://github.com/material-components/material-components-web/pull/4469
Just a heads up, Chrome update may have resolved this as well.

Abhinay and I just re-tested and we're actually unable to repro the original bug right now, so we're going to investigate if Chrome in fact changed this behavior. Abhinay discovered that it seems to be firing an input even now upon autofill.

Closing.


Added Dockerfile, added build and run instructions.

This pull request adds the instructions and Dockerfile to run drivesync in a container using Docker or Podman.
The Dockerfile is meant to build an image based on the official ruby:2.6.6 image with the drivesync source code.
I added a short instruction to the README.md that describes building and starting drive sync.
Below the output of the drivesync image build:
$ docker build -t drivesync .
STEP 1: FROM ruby:2.6.6
Getting image source signatures
Copying blob 8d134d749716 done
Copying blob 5a63a0a859d8 done
Copying blob 0ed5a9824906 done
Copying blob 496548a8c952 done
Copying blob 376057ac6fa1 done
Copying blob 2adae3950d4d done
Copying blob 174d855079b8 done
Copying blob a749489e9ce3 done
Copying config 107c48f680 done
Writing manifest to image destination
Storing signatures
STEP 2: RUN git clone https://github.com/MStadlmeier/drivesync.git /drivesync
Cloning into '/drivesync'...
remote: Enumerating objects: 3, done.
remote: Counting objects: 100% (3/3), done.
remote: Compressing objects: 100% (3/3), done.
remote: Total 245 (delta 0), reused 2 (delta 0), pack-reused 242
Receiving objects: 100% (245/245), 53.05 KiB | 448.00 KiB/s, done.
Resolving deltas: 100% (153/153), done.
--> febb80e1ba4
STEP 3: RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y locales
Get:1 http://security.debian.org/debian-security buster/updates InRelease [65.4 kB]
Get:2 http://deb.debian.org/debian buster InRelease [121 kB]
Get:3 http://deb.debian.org/debian buster-updates InRelease [49.3 kB]
Get:4 http://security.debian.org/debian-security buster/updates/main amd64 Packages [201 kB]
Get:5 http://deb.debian.org/debian buster/main amd64 Packages [7905 kB]
Get:6 http://deb.debian.org/debian buster-updates/main amd64 Packages [7380 B]
Fetched 8350 kB in 1s (5703 kB/s)
Reading package lists... Done
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following additional packages will be installed:
  libc-l10n
The following NEW packages will be installed:
  libc-l10n locales
0 upgraded, 2 newly installed, 0 to remove and 1 not upgraded.
Need to get 4907 kB of archives.
After this operation, 20.8 MB of additional disk space will be used.
Get:1 http://deb.debian.org/debian buster/main amd64 libc-l10n all 2.28-10 [847 kB]
Get:2 http://deb.debian.org/debian buster/main amd64 locales all 2.28-10 [4060 kB]
Fetched 4907 kB in 0s (21.7 MB/s)
debconf: delaying package configuration, since apt-utils is not installed
Selecting previously unselected package libc-l10n.
(Reading database ... 23979 files and directories currently installed.)
Preparing to unpack .../libc-l10n_2.28-10_all.deb ...
Unpacking libc-l10n (2.28-10) ...
Selecting previously unselected package locales.
Preparing to unpack .../locales_2.28-10_all.deb ...
Preparing to unpack .../locales_2.28-10_all.deb ...
Unpacking locales (2.28-10) ...
Setting up libc-l10n (2.28-10) ...
Setting up locales (2.28-10) ...
Generating locales (this might take a while)...
Generation complete.
--> 893d3e8b1c5
STEP 4: RUN sed -i -e 's/# en_US.UTF-8 UTF-8/en_US.UTF-8 UTF-8/' /etc/locale.gen &&     dpkg-reconfigure --frontend=noninteractive locales &&     update-locale LANG=en_US.UTF-8
Generating locales (this might take a while)...
  en_US.UTF-8... done
Generation complete.
--> 10ce88f4cd8
STEP 5: ENV LANG en_US.UTF-8
--> 9cddf37aebe
STEP 6: ENV LC_ALL en_US.UTF-8
--> 54de2287c04
STEP 7: WORKDIR /drivesync
--> 5df7c1c7d20
STEP 8: RUN bundle update --bundler     && bundle install
Fetching gem metadata from https://rubygems.org/........

Using public_suffix 3.0.2
Using addressable 2.5.2
Using bundler 1.17.2
Using declarative 0.0.10
Using declarative-option 0.1.0
Using multipart-post 2.0.0
Using faraday 0.15.2
Using jwt 2.1.0
Using little-plugger 1.1.4
Using multi_json 1.13.1
Using logging 2.2.2
Using memoist 0.16.0
Using os 0.9.6
Using signet 0.8.1
Using googleauth 0.6.2
Using httpclient 2.8.3
Using mime-types-data 3.2016.0521
Using mime-types 3.1
Using uber 0.1.0
Using representable 3.0.4
Using retriable 3.1.2
Using google-api-client 0.23.4
Bundle complete! 1 Gemfile dependency, 22 gems now installed.
Use `bundle info [gemname]` to see where a bundled gem is installed.
STEP 9: COMMIT drivesync
--> a924a460204
a924a460204b398faefd3ad7457eb05a1e1bed689d37271103a21e2f6807f0ed

The ruby:2.6.6 base image and the drivesync image:
$ docker images
REPOSITORY                   TAG      IMAGE ID       CREATED          SIZE
localhost/drivesync          latest   a924a460204b   19 minutes ago   994 MB
docker.io/library/ruby       2.6.6    107c48f680c0   12 days ago      862 MB

The run:
$ docker run -it -v /etc/localtime:/etc/localtime:ro \
                 -v /root/Documents/drive:/root/Documents/drive:z \
                 -v /root/.credentials:/root/.credentials:z \
                 -v /root/.drivesync:/root/.drivesync:z \
                 drivesync \
                 ruby drivesync.rb
Local folder is 3 files behind and 0 files ahead of remote
Starting sync at 2020-05-28 23:02:54 +0200
Downloading file files/spot02.mp3 ...
Downloading file files/photo3736453.jpeg ...
Downloading file files/list.xml ...
Sync complete.

@tedsluis Thanks for the contribution! I will cherry-pick this into the development branch and include it in the next release.

Merged into develop


Rename download buttons

I have some suggestions about the download buttons:

The ODK Data could be renamed to Submissions and the internal links:

Submissions as CSV
Submissions as JSON
Attached Files

The OSM Data could be Geospatial data or Map data and the internal links:

All data as OSM
Filtered data as OSM
All data as GeoJSON
Filtered data as GeoJSON
All data as CSV
Filtered data as CSV

See https://github.com/hotosm/OpenMapKitServer/issues/56, about the CSV files.
What do you think, @smit1678 @mojodna?

I agree with your suggestion, @willemarcel - that would make a lot more sense to users that are new to OMKServer or are not as familiar with the data types.

Does "filtered data" as such also apply to the ODK data/submissions? Or does that not have filtering?

@PaulUithol yes, it applies.


Wrap lists + remove nested divs

Not sure on how we might want to remove nested divs. Was just messing around with this and it seems to work.
I also tried 'bubbling' them up the tree until they were no longer 'nested', but that felt worse than just unwrapping them.

Maybe something to come back to after undo is merged

Can we (or I) add "strip cite from links" to this, and maybe prioritize this? That bug in particular is pretty urgent

Not really sure on the stability of these, nested divs in particular was pretty fiddly. Mainly did it to test out the feasibility of extending this code with new rules. Maybe just open a new PR that strips citations and see if we can merge that?

sounds good

List stuff seems to work pretty well.
Div stuff definitely works but we possibly want to scope it better to certain situations/do something nicer there. The alternative approach would be to separate them out rather than condense them into their parent. In my testing they both look kind of weird from a user standpoint.


fail gracefully if user doesn't have bitcoind with scanblocks enabled

they need bitcoind >=v24 (i think) and blockfilterindex=1 enabled
They also need the index to be finished building
fallback to esplora?
need to fix esplora in that case https://github.com/bitcoindevkit/bdk/issues/1120#issuecomment-1834676960

Hey, I tried this and got error:
$ cl smaug add "wpkh([56003681/84h/0h/0h]tpubDDrecdRDGxYiu2eduJiPyojMQQJcCnQpyLpA18BkEFtr8S9jgAGAhZ5TKgpArzcnu8qYcVtad2KGXhWsxRgjJbLLwMDH3SW4YcaHbScwLs1/0/*)#840qygs5" "wpkh([56003681/84h/0h/0h]tpubDDrecdRDGxYiu2eduJiPyojMQQJcCnQpyLpA18BkEFtr8S9jgAGAhZ5TKgpArzcnu8qYcVtad2KGXhWsxRgjJbLLwMDH3SW4YcaHbScwLs1/1/*)#kp2peaqv"
"JSON-RPC error: RPC error response: RpcError { code: -1, message: \"Index is not enabled for filtertype basic\", data: None }"

If the block filter index is currently syncing, the add command runs and waits for it to finish, we should an error if the block filter index is not done being built.


closing #6 formating of duration for table output.

Added two functions on the Report struct which format the duration according to the discussion in the issue. Both functions return the result of formatDuration which does the actual formatting. Tests of this function are included.
+-----------------+--------------------+--------------+
| CURRENT PROJECT | WORKED SINCE START | WORKED TODAY |
+-----------------+--------------------+--------------+
| timetest        | 0h 0min 5sec       | 1h 6min      |
+-----------------+--------------------+--------------+

And over one minute
+-----------------+--------------------+--------------+
| CURRENT PROJECT | WORKED SINCE START | WORKED TODAY |
+-----------------+--------------------+--------------+
| timetest        | 0h 1min            | 1h 7min      |
+-----------------+--------------------+--------------+

Thanks for the feedback - I appreciate it 😃! Let's open an Issue for tests? As I am getting more familiar with the code base I would be happy to write some more tests.

Yes, we could even open an issue per package and create PRs package-wise.