sportstracker: init at 8.0.1

Description of changes
Closes https://github.com/NixOS/nixpkgs/issues/298184
This PR adds 1 package: sportstracker
I went with a capitalized executable name, because the releases also capitalize them.
This is a JavaFX application. The package wants to fetch javafx for itself, however we can provide javafx through nixpkgs by overriding the jdk. This way we won't have to patch any .so files inside the fetched files and there will be no platform-dependent mvnHash differences.
I haven't added the patch to pin the default maven plugin versions yet, but should be trivial.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@Thatoo could you test if this works for you?

How do I test? It doesn't appear on search.nixos.org ?

How do I test? It doesn't appear on search.nixos.org ?

It won't appear there unless this PR gets merged.
Until then you can either clone the repository and git checkout to my PR branch and run
nix-shell -I nixpkgs=. -p sportstracker

or
if you don't want to clone manually you can just run
nix-shell -I nixpkgs=https://github.com/NixOS/nixpkgs/archive/68e5e1732a328c9878a02cdece0a2b3403bfbec8.tar.gz -p sportstracker

Running this/these will temporarily make SportsTracker availiable as an executable in your current shell session.

Well, it launch but I can't say it works nicely.
When I try to open a file here is what I get

In terminal here is what it says :

[nix-shell:~]$ SportsTracker
2024-03-30 12:26:08 INFOS de.saring.sportstracker.gui.STDocumentImpl loadOptions Loading application options...
2024-03-30 12:26:09 INFOS de.saring.sportstracker.gui.STControllerImpl$LoadTask call Loading application data...
2024-03-30 12:26:09 INFOS de.saring.sportstracker.gui.STDocumentImpl readApplicationData Reading application data
2024-03-30 12:26:09 INFOS de.saring.sportstracker.storage.db.DbStorage openDatabase Opening database jdbc:sqlite:/home/thatoo/.sportstracker/sportstracker.sqlite
SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
2024-03-30 12:26:09 INFOS de.saring.sportstracker.storage.db.DbStorage createSchema Creating database schema
2024-03-30 12:26:09 INFOS de.saring.sportstracker.gui.STDocumentImpl readApplicationData Opened SQLite database in 513 msec
2024-03-30 12:26:09 INFOS de.saring.sportstracker.storage.db.AbstractRepository readAll Reading all SportType entries
2024-03-30 12:26:09 INFOS de.saring.sportstracker.storage.db.ExerciseRepository readAll Reading all Exercises
2024-03-30 12:26:09 INFOS de.saring.sportstracker.storage.db.AbstractRepository readAll Reading all Note entries
2024-03-30 12:26:09 INFOS de.saring.sportstracker.storage.db.AbstractRepository readAll Reading all Weight entries
2024-03-30 12:26:09 INFOS de.saring.sportstracker.gui.STDocumentImpl readApplicationData Loaded all data in 6 msec
2024-03-30 12:26:09 INFOS de.saring.sportstracker.gui.STDocumentImpl importApplicationDataFromXml Importing application data from XML
2024-03-30 12:26:10 INFOS de.saring.sportstracker.storage.db.DbStorage importExistingApplicationData Importing existing application data to database
2024-03-30 12:26:10 INFOS de.saring.sportstracker.storage.db.DbStorage commitChanges Committing database changes
2024-03-30 12:26:10 INFOS de.saring.sportstracker.storage.db.AbstractRepository readAll Reading all SportType entries
2024-03-30 12:26:10 INFOS de.saring.sportstracker.storage.db.ExerciseRepository readAll Reading all Exercises
2024-03-30 12:26:10 INFOS de.saring.sportstracker.storage.db.AbstractRepository readAll Reading all Note entries
2024-03-30 12:26:10 INFOS de.saring.sportstracker.storage.db.AbstractRepository readAll Reading all Weight entries
2024-03-30 12:26:20 INFOS de.saring.sportstracker.gui.STControllerImpl onOpenHrmFile Opening HRM file '/home/Sport/Garmin/ACTIVITY/87KJ3909.FIT' in ExerciseViewer...
2024-03-30 12:26:39 INFOS de.saring.sportstracker.gui.update.STUpdateChecker checkForUpdates Checking for SportsTracker updates...
Killed

Well, it launch but I can't say it works nicely. When I try to open a file here is what I get

Could you tell me what system are you running on? (NixOS/MacOS/other)

 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.82, NixOS, 23.11 (Tapir), 23.11.5742.219951b495fc`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - channels(root): `"nixos-23.11, nixos-unstable"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Okay, I pushed a patch for

It indeed open files now.
However, the tab "Chemin" (I guess "Path" or "Track" in english) don't display anything so we can't see the track on map :

Hm, I'll look i to it.
I don't have a .fit file to test with, what do you recommend I do?

Here is a .fit file (link available for one download only) : https://lufi.leprette.fr/r/wTpvHsfbwh#lmr8dkpNnlBjd5jvCME8DNy4TRAb29hY+Z40kNNI46A=

It looks like libjfxwebkit.so isn't currently supported in the latest JDKs.
https://github.com/NixOS/nixpkgs/issues/162064
Not sure what I can do, but I'll try some things

@Thatoo it looks like the program works correctly now, and displays the map as expected. I don't really recommend trying to build the package now because it takes about ~1 hour to compile everything.
I recommend waiting until the other PR reaches nixos-unstable-small (https://nixpk.gs/pr-tracker.html?pr=300573)

Is it ok now?

Is it ok now?

Should be, though I haven't built it in a while.

if it builds on unstable, can you also backport it to 24.05?

This still isn't merged, if that's what you're asking.
I'll backport it to stable once it does get merged.

Just checked, still works with rebased on latest unstable.

Could it be backported to 24.11?


Fix matching different guest products with a single Unlimited Virtualization subscription

See the Fix: Allow matching any guest products for Unlimited Virtualization subscriptions commit for more details. I also wrote new test scenarios (42 and 43) testing this fix.
Moreover, the PR contains additional (small) fixes:

fixed some older test data (mostly the README.md files)
moved SwapMove classes to correct packages
fixed duplicate log lines when running tests
logging test scenario number when running tests

Review commit-by-commit please, commit messages are descriptive.

@mcalmer I'd like to hear your opinion on the idea of my fix. The essence of the fix is:
An unlimited virtualization assigned to a host now covers all products installed on the guests that are "coverable" by the subscription. So, if a guest 1 has SLE 11 and guest 2 has SLE 12, they'll be both covered.
The only catch I could think of would be, if the subscription covered 2 products, that could be installed on the guest simultaneously, e.g.:
       HOST
        |
      GUEST1
      - SLE 11
      - some other product coverable by the same subscription

With my patch, both of these products would be covered by a single UV subscription. I'm not sure if this can happen.

@hustodemon yes this happens with all the modules you need for SLES, but counting happens based on the "host" - VMs are "unlimited" and this means every VM with one or more of the listed products in the subscription are "OK".
Important is, that the CPU counting happens based on the host CPUs. The VM is not important here.

After some further thinking I think there are 2 ways to improve the clarity of the output.json:

do not report the unconfirmed matches in the output: in many cases, their cents value is incorrect; in some cases, multiple matches, that look the same (but are not, because they have different cent group) are reported, which is confusing
include the cent groups in the output.json too: each reported match would contain a centGroupId field; we also might to report additional array of cent groups in the output

Just to recap, a cent group is a mechanism that I've added, so that multiple confirmed matches can consume N cents (the motivation was 1-2 subscriptions: confirming either 1 or 2 matches of a 1-2 subscription always consumes 100 cents). More details on this are in our deprecated gitlab repo (merge request 29). Ping me for any further info.
@mcalmer, @moio : I'd be happy, if we could talk about this next week or later.

Are there downsides you can see in evicting unconfirmed matches from the output?
Do we use them anywhere or could we treat them as an internal implementation detail of the matcher?
Would testing/debugging become significantly more complex by removing them from output.json?

Are there downsides you can see in evicting unconfirmed matches from the output?
You guessed one of the downsides in your 3rd point: we're losing some information, that might be useful during debugging. At the same time, having them in the output can bring some confusion, as in some cases (as I mentioned in my latest comments), the cents field contains gibberish. We could either omit the cents field from unconfirmed matches, or we could make the cent groups part of the output, as described above.

Do we use them anywhere or could we treat them as an internal implementation detail of the matcher?
Afaik, they are not used by the matcher output consumers (SUMA UI).

You guessed one of the downsides in your 3rd point: we're losing some information, that might be useful during debugging.

Would it be an option to have this information that's only relevant to debugging in logs, thereby cleaning up output.json for its intended audience? (maybe as simple as dumping the json there)

I find this to be a nice idea! Maybe we could expose the loglevel control
and this dump could be controlled via either a cli option or by debug
field in /etc/rhn.conf).
Silvio Moioli<EMAIL_ADDRESS>schrieb am Fr., 14. Aug. 2020,
17:14:

You guessed one of the downsides in your 3rd point: we're losing some
information, that might be useful during debugging.
Would it be an option to have this information that's only relevant to
debugging in logs, thereby cleaning up output.json for its intended
audience? (maybe as simple as dumping the json there)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openSUSE/subscription-matcher/pull/18#issuecomment-674123286,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKYZLBYM4F6AV4EFPAQIBTSAVIGPANCNFSM4N7MERFA
.

I find this to be a nice idea! Maybe we could expose the loglevel control and this dump could be controlled via either a cli option or by debug field in /etc/rhn.conf).

We do something like this already for the Gatherer. So it should be easy to have something like this for Matcher too.
What about a mumble session to discuss this issue?

If you need my input just ping in chat

@moio @mcalmer I improved the logging (passing log level from the CLI, logging all facts in the assignment as JSON). I played around with sorting facts in the Construction Heuristics with mixed results - I still don't know, if this is the right direction. I think the best way here would be to employ the Optaplanner benchmark tool to find the "right" parameters for big setups.
Originally, the user reported 2 problems:

an UNLIMITED virtualization didn't cover various products installed on guests (fixed by Fix: Allow matching any guest products for Unlimited Virtualization subscriptions)
the SUSE Manager subscription was unmatched (fixed by Increase the unimproved count limit )

The user reported additional problem, but it seems this is unrelated to these 2 (and it's very likely not a matcher bug (the CPU count for some systems in his setup is misreported, thus leading to wasted subs)).
This PR is already big and fixes 2 problems, so I would recommend merging it after review. We can address possible additional issues separately. Please re-read the PR description and review.

@moio @hustodemon In the early days we had also a list of systems not getting a subscription. This was removed long ago.
So I think the UI calculate it. The UI send a list of systems and get the matched systems back. Every system which is not in matched, miss a subscription and is an "error".
Does the UI show it? If yes, I am fine.

Fair point. We display a list of unmatched products. I need to take a look
how these are computed in SUMA. Thanks for bringing this up!
Michael Calmer<EMAIL_ADDRESS>schrieb am Mi., 26. Aug. 2020,
17:20:

@moio https://github.com/moio @hustodemon
https://github.com/hustodemon In the early days we had also a list of
systems not getting a subscription. This was removed long ago.
So I think the UI calculate it. The UI send a list of systems and get the
matched systems back. Every system which is not in matched, miss a
subscription and is an "error".
Does the UI show it? If yes, I am fine.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openSUSE/subscription-matcher/pull/18#issuecomment-680947499,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKYZLA5HA2V3LXZS4VGT3TSCUR5HANCNFSM4N7MERFA
.

@mcalmer we are safe with regards to your last comment - unmatched products are computed in the way you described (we don't rely on "unconfirmed" matches in the output, so we're good to go!


Feature 6108 remove cartodb logo

Close #6108
Pending for ruby changes by @juanignaciosl
reviewer @xavijam
cc @mbforr

@xavijam , @acanimal , take a look at to check whether this fixes your needs, please. After talking to @acanimal and seen the overlay and map creation workflow I think the best I can do here is don't adding the logo at the default visualization initialization if visualization user has that feature flag.
Let me know if this is what you actually need, please, and then I'll ask for CR to other backender.

That was my first idea, so fine by me :+1:.

Ok. @gfiorav , CR this, please. I can't follow Hound conventions about count instead of  select .. count because overlays is not an Array but a Virtus model whose count can't receive a block.

Apart from the comments, :+1:

@gfiorav ok, I just fixed the indentation. Frontend code is @acanimal 's and maybe he's still testing this.

ok @juanignaciosl !
@acanimal Sorry about the uncalled-for CR, lol!

@juanignaciosl @gfiorav @xavijam let me test it all together locally before merge.

@acanimal yep, and I still have to fix the tests. They're failing because of wrong mocking and so on, not because the new production code.

Tests passed, let me know if you need anything else from me :-).

reviewer @xavijam
I have simplified a lot the code necessary on client side thanks to the work of @juanignaciosl and comments from @gfiorav
I tested all locally. When a map is created the logo overlay is stored depending on the desabled_cartodb_logo feature flag.

Note: if you create a map without the flag activated the logo is stored although later you activate the FF.

Apart from my comment, ready to go :), great job. :+1:
cc @mbforr

good to go!


Disable testCyclicRecordViaFields temporarily

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.
This is just to make other PRs stable

Test case tracked by https://github.com/ballerina-platform/ballerina-lang/issues/32617
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Disabling the test case as many PR builds are blocked on this. We are immediately starting #32617 to fix the underlying issue.


This cannot be installed via the Extensions Manager?

Is it true that this extension cannot be installed via the Extensions Manager and can only be installed via your npm installation extension?

Yes

Thanks for the quick response. I actually got it all working via your instructions. While it's odd to have to install two of your extensions to get one, I can see no reasonable basis to complain when I get working software configured quickly, for free, following basic instructions. And your typescript plugin does exactly what I was looking for. So, bravo!

Thanks.
Long story:

brackets-typescript requires to download stuff from npm when installed
extensions manager can't do it
i built brackets-npm-registry so people can install complex extensions like brackets-typescript
i even put up a PR (https://github.com/adobe/brackets/pull/10602) so people didn't have to use brackets-npm-registry but it still hasn't been approved (PR created 15 Feb 2015)


Merge all Coinbase and non-Coinbase functions into one

I saw @keefertaylor 's TODO note and figured I'd just try to tackle it myself :D
Definitely worth a second set of eyeballs as the duplicate functions started getting heady toward the end lol

Great stuff! I'm OOO for the weekend but will look at this an associated PRs next week.
Thanks for cleaning up my poor code :)

Thanks man! I like the product so figured the best way to get more familiar was by getting the hands dirty, great work!
And don't thank me just yet ;)

Also if these changes aren't reflected in other associated libs in places I've missed feel free to point them out and I'll update. Not too too familiar with the structure just yet.


images are not being included

// JSON.stringify(this.state.editorState.getCurrentContent())
{
  "entityMap": {},
  "blocks": [
    {
      "key": "cv5ve",
      "text": "test",
      "type": "unstyled",
      "depth": 0,
      "inlineStyleRanges": [],
      "entityRanges": [],
      "data": {}
    },
    {
      "key": "bup2p",
      "text": "",
      "type": "atomic",
      "depth": 0,
      "inlineStyleRanges": [],
      "entityRanges": [],
      "data": {
        "src": "http://i.imgur.com/ARlToD4.jpg",
        "type": "image",
        "display": "medium",
        "caption": "caption",
        "rightsHolder": "rights"
      }
    }
  ]
}

// stateToHTML(this.state.editorState.getCurrentContent())

test

I have just had this same issue, trying to export to html while using medium draft. It seems that draft-js-export expects images to be an Entity, rather than a Block type. I am trying to fix this issue myself, so will let you know how I go.

I'm also having this issue.  Was anyone able to figure out a fix?

Also having the same problem, anyone able to patch locally or anything?

I have worked out how to do this, and I now have no problem using draft-js-export-html to export images to html. As long as you have an atomic block which accepts the src as data all should be fine. This is how I did it:
export function editorStateFromHtml (html, decorator = defaultDecorator) {
  if (html === null) {
    return EditorState.createEmpty(decorator)
  }

  const contentState = convertFromHTML({
    htmlToBlock: (nodeName, node) => {
      if (nodeName === 'img') {
        return {
          type: 'atomic',
          data: { src: node.src, type: 'image' }
        }
      }

      if (nodeName === 'figure') {
        if (!node.children.length) { return null }

        let caption = '', src = '', blockType = 'image'
        let captionNode = node.children[1]
        if (captionNode !== undefined) { caption = captionNode.innerHTML }
        let blockNode = node.children[0]
        if (blockNode !== undefined) { src = blockNode.src }

        let type = blockNode.tagName.toLowerCase()
        if (type === 'iframe') { blockType = 'video' }

        return {
          type: 'atomic',
          data: { src: src, type: blockType, caption: caption }
        }
      }
    }
  })(html)

  return EditorState.createWithContent(contentState, decorator)
}

Note that the decorator can be removed if you are not using one for links.
In my example this is exporting to html as follows
A more detailed example is in my convert util in my last draft editor
Note that I was not able to get the import-html to work due to the following issue so I used draft-convert for the import state from html

cooel

Facing the same issue. I tried using StevenIseki's solution, but no luck. It still does not include images. Has anyone found an alternative solution?

I've had success using the, as of now, undocumented option: entityStyleFn. I'll write a simple example and submit a PR for this one.

I am facing the same issue , has anyone found the fix?

I was able to fix the issue. I used this line of code instead:
   console.log(stateToHTML(this.state.content.getCurrentContent()));

I'm not sure how can I fix this trouble yet..

I use draft-js-video-plugin to import video into draft editor but when try to export video, I have same issue above.
// stateToHTML(this.state.editorState.getCurrentContent())

I use draft-js-video-plugin to import video into draft editor but when try to export video, I have same issue above.
// stateToHTML(this.state.editorState.getCurrentContent())

Was you able to fix this?

@haibui2207 was you able to fix this?


Cuisine demonyms

This is a feature suggestion which seems feasible to implement. I was impressed by the number of possible languages to select in the translation menu - in continuing with the idea of allowing for a "complete" list of options, the cuisine menu could include every national demonym with the correct format on the list. Tags like Pakistani, Bosnian, Senegalese, and so on; this is not the only way cuisines can be tagged of course but it covers a lot of ground. This could help people be more specific when they might not consider that an option.
There are also some nationalities where the correct format may not be obvious to people that could use some standardization. Not everyone knows that Somali and not Somalian would be correct for Somali food, and there are some more hard to guess ones out there like Burkina Faso = Burkinabe, United Arab Emirates = Emirati, Kiribati = I-Kiribati, and so on.

Are cuisine suggestion also coming from iD presets?
If yes, then adding some missing ones like in https://github.com/openstreetmap/id-tagging-schema/pull/252 would be a proper fix.

Agreed, this is for the iD tagging schema. See the faq for links.

I'll bring it up there, it seems like they have some ideas about how to do this


Create simple HTML page which redirects to google home page

I have created an HTML page that redirects to the google home page. would you please accept me as a contributor?
Thank you!

Great Job !! Follow me on GitHub

Thank you!


Add support for ANSI escape codes for colors

ANSI escape codes are used by many console applications to specify the color of the output. Currently, WhackWhackTerminal doesn't support those codes, and outputs the escape sequences instead.
Expected result in the standard Powershell console (conhost):

Actual result in WhackWhackTerminal:

(this is the output produced by simple-targets-csx)

I have never seen this before in my testing. I should be handling escape code sequences properly. That was part of the entire purpose of this terminal. Can you create a minimal sample that reproduce this?

As an example, if i type echo -e "\e[36msimple\e[39;49m" into my bash prompt i get a properly colored output.

So I was able to reproduce this by running build.cmd in WhackWhackTerminal, but the strange thing is I got the same behavior when running in powershell and cmd. They didn't seem to be respecting the escape sequences. The output was similar to simple-targets: Running "default"... (dry run)

I'm not sure... build.cmd uses csi.exe to execute a .csx script. I don't think it does any explicit escaping.
The code for producing the escape code is here

So I tested the scenario with the following code:
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace ConsoleApp1
{
    class Program
    {
        private static string Default(bool color) => color ? "\x1b[0m" : "";
        private static string Green(bool color) => color ? "\x1b[32m" : "";

        static void Main(string[] args)
        {
            Console.WriteLine(GetColor());
        }

        private static string GetColor()
        {
            return $"{Green(true)}simple{Default(true)}";
        }
    }
}

The text prints fine in powershell, but not in cmd. Looking into the issue further is appears the problem is actually a C# limitation in how you are supposed to print to the console. I searched and found this link. Which led me to this page on the microsoft docs. It would seem that you are not supposed to print the ansi sequences directly, and while it may work its not guaranteed to.

For example, i tested your build.cmd script on another windows 10 machine I had and colored output worked in WhackWhackTerminal and Powershell, but not CMD.

Weird... I just tried again in WWT, and now it works. I have no idea what could be causing this.

I think its some weird way that the underlying win32 APIs are detecting ANSI support and turning it on. For now I think your best course of action if you want this 100% consistent is to use the Console.ForegroundColor properties.

OK, thanks for investigating. Hopefully it won't happen too often...


Add top-level vendor dir to TO-Go and TM unit test containers

What does this PR (Pull Request) do?
The TO-Go and TM unit test containers are failing to find the vendored
influxdb library after moving it to the top-level vendor dir. Fix the
containers to include the top-level vendor dir in the build.

[x] This PR fixes is not related to any issue

Which Traffic Control components are affected by this PR?

Traffic Monitor (tests only)
Traffic Ops (tests only)

What is the best way to verify this PR?
Make sure both of these containers build successfully and the tests pass.
docker-compose -f traffic_monitor/tests/docker-compose.yml up --build
docker-compose -f traffic_ops/app/bin/tests/docker-compose.yml up --build unit_golang

The following criteria are ALL met by this PR

[x] This PR includes tests OR I have explained why tests are unnecessary
[x] This PR fixes test container builds, no docs necessary
[x] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[x] This PR includes any and all required license headers
[x] This PR ensures that database migration sequence is correct OR this PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4602/


transfer ownership?

From this doc, I think the process is:

I transfer this repo to you
you publish a new (patch) version

Sounds right? If so, let me know if you're still interested :)
@Alhadis

Yep, sounds good. :)

OK, I started the process on my end - you should have an email from github.

Got it, thanks!


unknown this.editor referencing.

it seems this.editor is undefined and being reference here again with

Is there i away to fix this one?
Is not causing any functionality issues so far, but i just want to make the console cleaner.

Should be fixed with 13.0.5. Can you confirm?
(You are using Angular 13?)

yes! its fixed man, thanks @MaaxGr .


Using translate in controllers

Hello,
First of all - thanks for great work, angular-translate is very useful piece of code :)
I'm using this in quite typical way - application bases on Angular 1.3.x currently, I've got languages files and simply call {{'KEY' | translate }} in my HTML templates, however I have to provide translation directly in controller - there's $notify service calling from controller and displaying various messages to the user, for example:
FilterService.setResults(false);
  $ionicLoading.hide();
  notify({
    message: 'Something goes wrong :/',    // <--- provide message in selected language here
    position: 'right',
    classes: 'notification  notification-warning',
    duration: 3000
});
$timeout(function () {
  notify.closeAll();
}, 3000);

I don't want to modify original HTML template of $notify and one more reason is that currently I am not able to predict if there won't be any other situations where this particular option (translate in controller) will have to be used.
First thing that came to my mind was to simply add some method(s) to provide this functionality directly in angular-translate code - but first of all, I don't want to modify its code (as I update all additional components via Bower and first update will destroy my code :) )
and second - I think this option might be useful not only for me, so it would be nice, if provided code will be well tested and follow best practices.

Easily use the service itself:
$translate('yourkey').then(function (translation) {
	FilterService.setResults(false);
	  $ionicLoading.hide();
	  notify({
	    message: translation,    // <--- provide message in selected language here
	    position: 'right',
	    classes: 'notification  notification-warning',
	    duration: 3000
	});
	$timeout(function () {
	  notify.closeAll();
	}, 3000);
})

Or another level of wrapping depending on the need of ui state.

If you have one specific language only (in the app, no live changes of the translation at this point), you could use
      notify({
        message: $translation.instant('yourkey'),    // <--- provide message in selected language here
        position: 'right',
        classes: 'notification  notification-warning',
        duration: 3000
    });

@knalli :
Thank you for such quick response, I've just found this in doc (I don't know how I missed it earlier, shame on me :P )
$translate('HEADLINE').then(function (headline) {
    $scope.headline = headline;
  });

but I'd rather do something like this:
  message: $translate.translate('ERROR')

But I see as $translate service returns a promise - it's the only option right now, am I right?

Also, it seems that with notify you can create your own templates. So, probably, you can translate your messages directly in the template.

@knalli

no live changes of the translation at this point

Unfortunately I have to handle more than one language and have to provide language switch option.

Then go with the first example I've given you. What is the problem with that? Promises will not hurt. ;)

@knalli
Of course they won't :) Just looking as simple way to do this as possible :) And it seems this migth be the solution I've been looking for.
@DWand
This is also good solution, however I'm trying to do this directly in controller as I do not want to create/modify anything but this one single line of code (where I should provide translated message).
Thanks guys for your help.

@knalli
Yep, solution with $translate('KEY').then( /* show notify stuff */) works perfect.
Once again thank you.


Fix hunger damage not being applied properly

About the PR

it was being applied on threshold change instead of continuously at low health.
tbqh idk how i missed this when i added it back.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

no cl no fun

thank you crunchy


TE 3.3 hierarchical_weight_resolution_test.go

Hi,
Vlan id should start from 1 as per yang file.
I have updated config to start vlan id from 1.
Thanks,
Prabha

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1181

Relevant Lines:
2064

💛  - Coveralls

/gcbrun

/gcbrun


Add apk instructions to README

Currently at the second-to-last version; bump to latest package is around the corner.


[ktx] onNeverAskAgain not called

Overview
onNeverAskAgain callback is not called when "Don't ask again" is selected.
Expected
onNeverAskAgain callback is called, so I can show the appropriate UI.
Actual
onNeverAskAgain callback is not called.
Environment

Running ktx-sample on API 29 and API 30, using ktx v1.1.3

Investigation
After debugging it, I can see that onNeverAskAgain: WeakReference<Fun>? is null in PermissionRequestViewModel.invoke().
After reading through the closed issues, I came across https://github.com/permissions-dispatcher/PermissionsDispatcher/issues/735. Wondering if the weakreference could be the issue? If I downgrade the ktx library to 1.0.4, onNeverAskAgain is successfully called.
Reproducible steps
https://user-images.githubusercontent.com/35034828/158357946-129980bf-6547-4228-82a4-a03399f88269.mp4

Agree, I have the same issue WeakReference is null. It is null for all three arguments:

It can be reproduced on Android < 10. For example, in my case, I have used an Android 7.1.1 build on Emulator.

I can also reproduce it on Android > 10. The references are null and so going back and forth the callbacks are not invoked.

I'm able to reproduce the problem using ktx:1.1.4, I've found a workaround keeping a reference to the onNeverAskAgain lambda from the Fragment that is constructing the permission request using constructPermissionsRequest; see sample code:
class MyFragment : Fragment() {
    private val locationPermissionNeverAskAgain : () -> Unit = { /* never ask again code here */ }
    private val locationPermissionRequest = constructPermissionsRequest(
        /* other settings here */
        onNeverAskAgain = locationPermissionNeverAskAgain
    ) { /* permission granted code here */ }
}

Probably library should declare onNeverAskAgain as a val on PermissionsRequesterImpl.


Doc: Secret Management Design v2

What would you like to be added?
Upgrade Secret Management Design, v1 version: Hosting and Retrieving Secrets
Why is this needed?
We have refreshed the AppConfig model, and the v1 version cannot used in the real prod environment.
P.S. Codes in kusion do not need to be updated

Already written a design doc, published on yuque.


Allow user to request default version using envvar

I'll close this as we seem to be moving to jll anyway


tweaked note about unified audit logging requirement

Docs Build status updates of commit d11ecf4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

office-365-management-api/get-started-with-office-365-management-apis.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[enhancement] Support special (xf86...) keycodes

Figured out that keyd -m does not track keycodes like xf86audiodown and similar. Would be nice to have, as some keyboards provide these codes =)

Do you mean XF86AudioLowerVolume? Those are X events but they usually
correspond to 'volumedown' (which my keyboard generates). What board are
you using? If you could send the output of xev when pressing the key in
question that might also help.
On Sat, Aug 14, 2021 at 8:10 PM Alexey Ismailov @.***>
wrote:

Figured out that keyd -m does not track keycodes like xf86audiodown and
similar. Would be nice to have, as some keyboards provide these codes =)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rvaiya/keyd/issues/14, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOP7GJFSVJISO4KCE4E3BDT44AWXANCNFSM5CFSOEAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Tested on both Corne and Vortex Tab90M (that one hasn't volume ctrl, but has CALC key instead, xev inputs below. keyd -m doesn't register those
So vol down:
MappingNotify event, serial 33, synthetic NO, window 0x0,
    request MappingKeyboard, first_keycode 8, count 248

FocusOut event, serial 33, synthetic NO, window 0x6200001,
    mode NotifyGrab, detail NotifyAncestor

FocusIn event, serial 34, synthetic NO, window 0x6200001,
    mode NotifyUngrab, detail NotifyAncestor

KeymapNotify event, serial 34, synthetic NO, window 0x0,
    keys:  0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
           0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0

MappingNotify event, serial 34, synthetic NO, window 0x0,
    request MappingKeyboard, first_keycode 8, count 248

volup:
MappingNotify event, serial 34, synthetic NO, window 0x0,
    request MappingKeyboard, first_keycode 8, count 248

FocusOut event, serial 34, synthetic NO, window 0x6200001,
    mode NotifyGrab, detail NotifyAncestor

FocusOut event, serial 35, synthetic NO, window 0x6200001,
    mode NotifyUngrab, detail NotifyPointer

FocusIn event, serial 35, synthetic NO, window 0x6200001,
    mode NotifyUngrab, detail NotifyAncestor

KeymapNotify event, serial 35, synthetic NO, window 0x0,
    keys:  2   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0
           0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0

MappingNotify event, serial 35, synthetic NO, window 0x0,
    request MappingKeyboard, first_keycode 8, count 248

Calculator (omg this so useless isn't in? :sweat_smile: ):
MappingNotify event, serial 32, synthetic NO, window 0x0,
    request MappingKeyboard, first_keycode 8, count 248

KeyPress event, serial 32, synthetic NO, window 0x6200001,
    root 0x6c2, subw 0x0, time 375052301, (411,745), root:(2332,760),
    state 0x10, keycode 148 (keysym 0x1008ff1d, XF86Calculator), same_screen YES,
    XLookupString gives 0 bytes:
    XmbLookupString gives 0 bytes:
    XFilterEvent returns: False

KeyRelease event, serial 33, synthetic NO, window 0x6200001,
    root 0x6c2, subw 0x0, time 375052373, (411,745), root:(2332,760),
    state 0x10, keycode 148 (keysym 0x1008ff1d, XF86Calculator), same_screen YES,
    XLookupString gives 0 bytes:
    XFilterEvent returns: False

MappingNotify event, serial 33, synthetic NO, window 0x0,
    request MappingKeyboard, first_keycode 8, count 248

This is btw strange that xev registers XF86Audio[...] as unnamed keycodes, but, it looks like it always been like this ¯_(ツ)_/¯

This is btw strange that xev registers XF86Audio[...] as unnamed
keycodes, but, it looks like it always been like this ¯_(ツ)_/¯

These show up fine on my machine. Either your keyboard is not generating
the appropriate keycode or they are being
grabbed by some X program. If you are running xbindkeys or have them bound
in your window manager you will have
to unbind them before they appear. The calc key should work though. Can you
run keyd -m while keyd is not running
and see if you get any output? What version are you running (keyd -v)?
On Sun, Aug 15, 2021 at 5:28 AM Alexey Ismailov @.***>
wrote:

Tested on both Corne and Vortex Tab90M (that one hasn't volume ctrl, but
has CALC key instead, xev inputs below. keyd -m doesn't register those
So vol down:
MappingNotify event, serial 33, synthetic NO, window 0x0,
request MappingKeyboard, first_keycode 8, count 248

FocusOut event, serial 33, synthetic NO, window 0x6200001,
mode NotifyGrab, detail NotifyAncestor

FocusIn event, serial 34, synthetic NO, window 0x6200001,
mode NotifyUngrab, detail NotifyAncestor

KeymapNotify event, serial 34, synthetic NO, window 0x0,
keys:  0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0

       0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0

MappingNotify event, serial 34, synthetic NO, window 0x0,
request MappingKeyboard, first_keycode 8, count 248

volup:
MappingNotify event, serial 34, synthetic NO, window 0x0,
request MappingKeyboard, first_keycode 8, count 248

FocusOut event, serial 34, synthetic NO, window 0x6200001,
mode NotifyGrab, detail NotifyAncestor

FocusOut event, serial 35, synthetic NO, window 0x6200001,
mode NotifyUngrab, detail NotifyPointer

FocusIn event, serial 35, synthetic NO, window 0x6200001,
mode NotifyUngrab, detail NotifyAncestor

KeymapNotify event, serial 35, synthetic NO, window 0x0,
keys:  2   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0

       0   0   0   0   0   0   0   0   0   0   0   0   0   0   0   0

MappingNotify event, serial 35, synthetic NO, window 0x0,
request MappingKeyboard, first_keycode 8, count 248

Calculator (omg this so useless isn't in? 😅 ):
MappingNotify event, serial 32, synthetic NO, window 0x0,
request MappingKeyboard, first_keycode 8, count 248

KeyPress event, serial 32, synthetic NO, window 0x6200001,
root 0x6c2, subw 0x0, time 375052301, (411,745), root:(2332,760),

state 0x10, keycode 148 (keysym 0x1008ff1d, XF86Calculator), same_screen YES,

XLookupString gives 0 bytes:

XmbLookupString gives 0 bytes:

XFilterEvent returns: False

KeyRelease event, serial 33, synthetic NO, window 0x6200001,
root 0x6c2, subw 0x0, time 375052373, (411,745), root:(2332,760),

state 0x10, keycode 148 (keysym 0x1008ff1d, XF86Calculator), same_screen YES,

XLookupString gives 0 bytes:

XFilterEvent returns: False

MappingNotify event, serial 33, synthetic NO, window 0x0,
request MappingKeyboard, first_keycode 8, count 248

This is btw strange that xev registers XF86Audio[...] as unnamed keycodes,
but, it looks like it always been like this ¯_(ツ)_/¯
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rvaiya/keyd/issues/14#issuecomment-899022424, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOP7GOKJLK6SF6NL2YIZH3T46CEJANCNFSM5CFSOEAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Any update?

Sorry was busy, will try to check tomorrow, as I really have xbindkeys =)

No worries. I just release 0.1.0 which makes some major changes, though I haven't changed any of the key detection logic. It shouldn't touch anything related to your problem but it is better to upgrade to avoid any confounding variables. See the changelog for details.

Did you ever get this working?

I just fixed a bug which pertains to high numbered keycodes. It shouldn't have affected your issue since the codes you mentioned were within range, but it is worth giving the latest commit a try.


「Restful api 」only create vertex index and fail when create edge index

Describe the bug
Related api: POST http://{host}:{port}/db/{graph_name}/index/{label}/{field}
It only creates 「vertex index」, when try to create 「edge index」 it failed.
To Reproduce
Steps to reproduce the behavior:
1.Post a request which create edge index with edge label.
2.It will then return 400 error.
Expected behavior
According to the description of this method, all index creation includes 「vertex index」、「edge index」.
So this api should apply to create edge index.
Screenshots

Environment:

OS: Ubuntu 20.04 LTS (GNU/Linux 5.15.30-2-pve x86_64)
CPU: Intel(R) Xeon(R) Gold 5218R CPU @ 2.10GHz
Compiler: GCC 5.4.0

Additional context
Related Code:

The bug here only deal with AddVertexIndex method, miss AddEdgeIndex senareo.
So fix code is to add a branch here.
I wonder if I can create a pr to fix this bug or just waiting for the next exciting version.🤔

Thank you for report. We suggest to use cypher to create edge/vertex index, restful api based on cpprestsdk will be removed in the future and use brpc's rest interface.
Refer to https://github.com/TuGraph-db/tugraph-db/blob/master/doc/zh-CN/7.community/3.roadmap.md


I'd like to join this project's community

Your public repo
https://github.com/adgitate1/gov4git-identity-public.git
Your public branch
main

I donated to the book project and received an email telling me I could join the ledger.


Using a ChartEntryModelProducer breaks Compose preview

Describe the bug
When using a ChartEntryModelProducer charts do not render at all in the Compose preview (they do take up space though). The chart renders fine on a device.
It also works fine when providing an entry model instead.
Module

[x] compose
[ ] views
[ ] compose-m2
[ ] compose-m3
[ ] core (select if none of the above apply)

To reproduce
Steps to reproduce the behavior.

Create a ChartEntryModelProducer instantiating it with the preview data
Create a chart using this ChartEntryModelProducer
Create a compose preview for the above
Chart does not render in the preview

Expected behavior
A clear and concise description of what you expected to happen.
Chart preview should render in the preview.
Screenshots
If applicable, add screenshots to help explain your problem.
Android versions
Versions of Android where the bug occurs.
Android Studio Giraffe | 2022.3.1
Additional context
Add any other context about the problem here.

Hi, I think you can create preview with Chart and ChartEntryModelProducer. I'm using Vico 1.9.2 and the following code works for me:
@Composable
private fun Chart(
    chartEntryModelProducer: ChartEntryModelProducer
) {
    ProvideChartStyle(rememberChartStyle()) {
        Chart(
            chart = lineChart(),
            chartModelProducer = chartEntryModelProducer,
            startAxis = rememberStartAxis(),
            bottomAxis = rememberBottomAxis(guideline = null),
            runInitialAnimation = false
        )
    }
}

@Composable
@Preview
private fun Preview() {
    Surface {
        Chart(
            ChartEntryModelProducer(
                listOf(
                    FloatEntry(0.0f, 0.0f),
                    FloatEntry(1.0f, 2.0f),
                    FloatEntry(2.0f, 3.0f),
                    FloatEntry(3.0f, 4.0f),
                    FloatEntry(4.0f, 5.0f),
                )
            )
        )
    }
}

I'm able to see the following preview:

@Leedwon Thanks, I couldn't find this as I was on an earlier version of the library. Appreciate the help!
In case it helps anyone you can use
runInitialAnimation = !LocalInspectionMode.current
to only disable the animation for Compose previews - as per the documentation.

Hello! Thank you for your input, @Leedwon. I’m glad a solution has been found. We’ll be looking into whether we can make it the default for initial animations to be turned off in previews. It’d be preferable for this not to affect interactive previews, so the question is whether we can detect what kind of preview is being used.

Starting with Vico 1.12.0, initial animations are no longer run in previews. Thanks for contributing, @Leedwon!


I am your father

Also a few fixes to yaml files

Do you mind adding some simple docs for this one, explaining what it is and when to use it? I checked out SentimentIntensityAnalyzer, but couldn't immediately understand how I'd use it.
Thanks :D

How do you want the docs?

How do you want the docs?

Here's some examples: https://github.com/frikky/Shuffle-apps/tree/master/Docs
We'll further relate these to the apps themselves, and publish them as part of app usage when exploring apps in the future.

We'll further relate these to the apps themselves, and publish them as part of app usage when exploring apps in the future. Just write a reference to it here somewhere and we'll deal with the whole linking thing later. No need to overcomplicate it right now.

I've been thinking about whether it would make sense to add a MY_APP/VERSION/doc directory with markdown.  That way it would be in place with and updated along with the app.  At some future point, that could lend itself to a UI interface to pull up the docs for a specific app.  Thoughts?

We'll further relate these to the apps themselves, and publish them as part of app usage when exploring apps in the future. Just write a reference to it here somewhere and we'll deal with the whole linking thing later. No need to overcomplicate it right now.

I've been thinking about whether it would make sense to add a MY_APP/VERSION/doc directory with markdown. That way it would be in place with and updated along with the app. At some future point, that could lend itself to a UI feature to pull up the docs for a specific app. Thoughts?

Yep this works as well, and you're right that it's probably a better structure. Do you think we should do something like this for OpenAPI as well? Put it under APPNAME/VERSION/doc? It would probably be beneficial long-term, however we don't follow versioning for those yet, and we'll for sure get more of those than python.

Do you think we should do something like this for OpenAPI as well? Put it under APPNAME/VERSION/doc?

Without a doubt, yes.  My mindset is to keep the docs close to the code.  Reduce the number of repos people have to track, manage, and maintain.  I imagine that will help reduce (but never eliminate) lag between the docs and the code to a certain extent.

I agree with the directory change, however I would modify one thing and make sure the docs are localized

Check it out now, see if this is good

Looks good now! Let's migrate all the apps soonish to follow the same structure :)


Run CC process treating stdout / err as ByteString

With readCreateProcessWithExitCode() as we previously used it, we got an error while compiling test/builtin_auto/builtin_functions_test.act:
...
actonc: fd:7: hGetContents: invalid argument (invalid byte sequence)
The problem was noticed in CI for a new branch, which does not modify these parts of the compiler, yet fails on an existing test. It is reproducible on a Linux machine when LANG is not set (CI does not set LANG), so the system defaults to LANG=C (I think). The hGetContents is not called directly from our code (we enforce encoding where we do), but it must be called from within readCreateProcessWithExitCode() from the standard System.Process library. We have no way of modifying it. It's puzzling because the output of the commands run do not appear to contain anything but US ASCII, so it's unclear how we can get an invalid byte sequence. And why is this only happening in a branch with seemingly unrelated changes?
I refactored the code to use createProcess and async reading of stdout and stderr (to avoid blocking), which allows more direct control over the stdout & stderr pipes. We now treat stdout and stderr as ByteStrings, so we should transparently pass whatever is written and effectively, the encoding is no longer any of our problem. This should be fine and safe because we only pass through the entire output. If we for example would start slicing it, we would need to honour UTF-8 or whatever encoding is being used, but since we are not, this is OK.
Running CC is also moved to a separate function that can be called from the two places where we run CC. Less code!

This has passed 2 CI runs now without a hitch. Retrying a 3rd time and setting auto-merge.


RESEARCH other alternative metrics

Is there good literatur for interesting metrics to make carbon emission better understandable?

Hot topics to elaborate:

climate migration (number of people)
climate deaths (number of people)
shortened lifespan from bad life quality
food security
water shortage (evaporation)

The IPCC has a VERY HIGH CONFIDENCE for the following impacts of global warming:

Climate-related illnesses
premature deaths
malnutrition in all its forms
threats to mental health and well-being are increasing
(very high confidence1 )

Additionally, there is a HIGH CONFIDENCE that global warming increases
5. Migration
6. Conflicts
while at the same time, actions taken have a very high confidence to reduce or avoid negative effects for human health an well being

Premature Deaths
Directly addressed in 2019, around every one thousand ton of Co2e one premature death occurs. Therefore, a direct link between the estimated carbon emissions and the impact on society can be made.
Journal Ranking & Pradatory Risk Assessment:
Source: https://www.frontiersin.org/journals/psychology/articles/10.3389/fpsyg.2019.02323/full?fbclid

Migration per ton of carbon:
Difficult Metric, with no direct paper addressing this. It is possible to make (VERY) rough assumptions based on the following data:
Gigaton of Co2e Budget for either the 1,5 or 2 degrees goal. Source
You then take the budget and break it down to estimated global migration numbers linked to global warming.
Question: is this a sufficient evidence based metric then? I mean any number is better then none at the moment, but do we sacrifice legitimacy and or trust for providing such an oversimplified calculation?


Please ship the typings with npm

Hey there,
could you please either ship the .d.ts with npm or register it ?

It is.

uuups. Wrong version number in package.json. Sorry. Have a nice day.


Validate vSphere CSI StorageClass Name and Policy

A current failure mode in the cluster runs as follows:

user deletes default vSphere CSI StorageClass name
user creates new vSphere CSI StorageClass with a storagePolicyName different from the default defined by EKSA.
user runs a cluster upgrade
cluster upgrade fails when it tries to apply the default storage class because storagePolicyName is an immutable field

To protect against this, at the start of cluster upgrade we should validate that if a StorageClass with default EKSA storage class name exists, it has the same storagePolicyName as EKSA default.

We removed vsphere csi driver, closing.


Cannot reproduce the results on s3dis area 5, including pretrained model

Thanks for releasing the pretrained model on S3DIS. However, the mean IoU reported by test_any_model.py is only around 57 on area 5 (pretrained model), which is far from 65 reported by the paper.

In addition, we trained the model on s3dis area 5 with the default parameter setting, however, the mean IoU can only achieve around 59.

Problem is solved. It is because the dataset we used is aligned version of S3DIS. In S3DIS.py, all rooms in same area are merged together for further processing. Since the rooms in aligned version are rotated by some angles, there will be overlapped points when merge them, which will bring negative effects to train and test.


Fix dirty state not matching save button state

Undo after save can make things go negative. This is fine as redo will push it back towards zero

Actually seems there's another issue. Updating a cell with new output and undoing doesn't work.

Codecov Report

:exclamation: No coverage uploaded for pull request base (ds/custom_editor@3e8598c). Click here to learn what that means.
The diff coverage is 100%.

@@                 Coverage Diff                 @@
##             ds/custom_editor    #9993   +/-   ##
===================================================
  Coverage                    ?   60.81%
===================================================
  Files                       ?      564
  Lines                       ?    29876
  Branches                    ?     4490
===================================================
  Hits                        ?    18169
  Misses                      ?    10714
  Partials                    ?      993

Impacted Files
Coverage Δ

...tascience/interactive-ipynb/nativeEditorStorage.ts
55.35% <100%> (ø)

Continue to review full report at Codecov.

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

Seems like VS code is doing weird stuff here too. Now at least my internal state matches theirs.
If you do this:

Edit file
Save file
Undo once - File tab is dirty at this point
Apply ANY random edit - File tab goes clean.

I think they need to change the API to allow us to set the undo stack and the dirty state.


absolute in transform fails

https://github.com/anxpara/getActualClientRect/issues/5
already fixed and tested in gACR, publishing soon, then will add tests in projectrix and publish

https://github.com/anxpara/projectrix/commit/d64e02fd6d1d2e2ed92a1b77d095aad402c8570a


Can I use this module on a React Native project ?

Everything is in the question title. I was wondering if I could use this module in a React Native project.
I installed the module the following way :
npm i sanitize-html
And I entered this in my component code :
import { sanitizeHtml } from "sanitize-html";

...

const htmlContent = sanitizeHtml(this.props.html, {
       allowedTags: ['img', 'a', 'p', 'span', 'br', 'ul', 'ol', 'li', 'h1', 'h2', 'h3', 'h4'],
       allowedAttributes: {
         'a': ['href'],
         'img': ['src']
       }
});

Am I importing the module the wrong way ?

I forgot to say, I have this error on Android simulator :
Requiring unknown module "undefined".If you are sure the module is there, try restarting Metro Bundler.  The rest is up to you - I have no expertise with react native but it shouldn't matter if you are using a (mumble mumble) packaging system such as webpack or browserify that can cope.
However, keep in mind that trusting a client app to sanitize anything is a fundamental security fail. You should be using this on the server end of your app. The exceptions are very few and have to do with previewing, not sanitizing. Also keep in mind this module would pull in a lot of code; no big deal on the server, but an overhead on the browser.

I won't use it on my app then :)
Thank you so much for the explanation !


How to set multiple error messages?

An input box has multiple validation rules, such as required and email, so how can I distinguish between setting prompt characters for them?
<input type="text" class="form-control" v-model="form.email" v-validate.disable="'required|email'" data-vv-as="email address" name="email" placeholder="enter email">
I hope that the error prompt of required is "mailbox must be filled", and the email is "the mailbox format you entered is not correct".

Every rule that you add to your input has it's own error message.
In your example
<input type="text" name="email" v-valdidate.disable="'required|email'" ....>

you set two rules and each rule has it's own error message.
Once a rule on that input fails, the message for that particular rule will be added to errors field.
To find out what particular rule failed, you could to this:
const err = errors.first(email);
if(err) {
  const failedRule = err.rule;
}

Does this somehow answer your question?

Thank you very much. I may not be familiar with vee-validator yet. I would like to implement the following functions in the simplest way as follows: the fields of jquery.validator.js correspond to error prompts one by one
 $("#form").validate({     rules: {       email: {         required: true,         email: true       },       email2: {         email: true       },       phone: {         required: true,         customPhoneRule: true       }     },     messages: {       email: {         required: "Please fill in your email",         email: "The mailbox format is incorrect"       },       email2: {         email: "What you entered is not a legitimate mail"       },       phone: {         required: "Cell phone Numbers should not be empty",         customPhoneRule: "The phone number format is incorrect"       }     }   });
In the code above, I used two "required", and the error messages corresponding to the two "required" were completely different, and the error messages corresponding to the two "email" were also different.
However, when I used vee-validator to implement the above functions, it was difficult
<input type="text" class="form-control" v-model="form.username" v-validate.disable="'required|checkAccount'" data-vv-as="Mobile phone or email" name="username">   <input type="password" class="form-control" v-model="form.password" v-validate.disable="'required|length:6'" data-vv-as="login password" name="password">
The first "required" error prompt is: "Please enter Mobile phone or email".
The second "required" error prompt is: "please enter login password".
"Length :6" The error message is: "The password value is not valid."
What can I do to implement something like using jquery.validator.js above?

Ah ok, now I understand 💡
You want different messages for the same rule depending on the field.
From my perspective, this is currently not possible.

Well, thank you!However, this requirement is needed in many places. I hope you can consider adding it in the future!

I just went through the code... it's not impossible to implement this, I could already think of a possible solution.
Let's ask @logaretm what he thinks about a possible feature request.

Okay

I made a quick change to my local vee-validate repo, where you can use a data-* attribute to set a message for any rule used inside the template.
You could either pass a static string or an arrow function that takes field name, rule parameters and data as arguments.
// Proposed syntax
data-vv-{ruleName}-message

// static message for rule required
data-vv-required-message="Hey man, this is required!"

// message using an arrow function and additional parameters
:data-vv-required-message="(fieldName, params, data) => `Yo, ${fieldName} is required...!`"

@logaretm What do you think? I could shoot you a PR, if this feature is any useful 🚀

If you use "data-vv-{ruleName}-message" to set error messages, I hope you can support "v-bind:data-vv-{ruleName}-message", which is useful when doing internationalization.
I tried "v-bind:data-vv-as", but I ended up with "[object Object]".
I think you can configure the error message using "v-bind:data-vv-messages=messages", which is an object, as follows:
{
  data(){
    return {
      messages: {
        required: "Please enter your email address",
        email: "The email format is incorrect"
      }
    }
  }
}

Have a look the docs for data-* attributes
There is no attribute where you can specify any messages.

Sorry, was away for a few days.
Thanks for the suggestions and the proposals, I believe this was requested before and I was against it since I don't like adding too many attrs to the template, it can get ugly really quick.
Instead, use the custom field in the dictionary to set your custom messages, which is very similar to the jquery validator.
Validator.localize({
  en: {
    custom: {
      email: {
        required: 'Please enter your email address',
        email: 'Your email does not look right'
      }
    }
  }
})

Here is an example of it:
https://codesandbox.io/s/rlvk13ln7n

yes, it does. Thank you very much.But there is also a situation where I have an email input box on multiple pages, and in this way the value of the "name" attribute in my input box has to be different, otherwise the error message will not be correct
//page1：
<input type="text" name="email" v-validate="'required|email'">
//page2:
<input type="text" name="email2" v-validate="'required|email'">
//main.js:
Validator.localize({
  en: {
    custom: {
      email: {
        required: "Please enter your email address",
        email: "Your email does not look right"
      },
      email2: {
        required: "Please enter your email address222222",
        email: "Your email does not look right22222"
      }
    }
  }
});

If I have 10 pages with email input fields, and they all have their own validation rules and corresponding error messages, then I'm not going to define 10 names.So what I'm trying to say is can you define error messages in the page, not in mian.js, because in mian.js the definition can affect the whole world

Agreed with 941477276, a global dictionary object is unwieldy when most of your custom error messages are one-offs for specific inputs that may share names across different forms.
There needs to be a way to override an error message for one specific Vue instance.

Any updates?


Add badges

[x] Code coverage
[x] Code quality
[x] Licence
[x] contributors
[x] commits
[x] CI/CD

have added most of them but code coverage is still pending!


Support query string options for mssql.

This is a PR to add support to the mssql dialect to parse query string parameters when a connection string is provided to knex. It's needed for Azure SQL, where encryption is turned on and the mssql library knex uses requires encrypt: true.
The implementation just uses the existing url.parse function's second parameter to parse query strings into an object. Then it assigns a clone of this object to the options property accepted by the library.
Example connection string: mssql://user:password@example.com/ExampleDB?encrypt=true
I also added the Babel plugin to implement a transform for object rest spread. It is supported in the latest Node 8 LTS so I think it should be supported here.
There are some issues with this PR I'd like opinion on:

Boolean arguments. The mssql library checks for encrypt: true through type coercion. So if encrypt=false is passed, then it's still coerced to a boolean true. Without parsing the boolean string 'false' or changing the mssql library this is potentially confusing for some users.
Not sure if any additional tests are needed. I think this is already covered fairly well by the existing integration tests, which support dynamic configuration of the connection parameters/string.
I'm aware there has been discussion around this in the past and #1528 suggests a bigger overhaul may be necessary. If this is not suitable please let me know.

Thanks!
EDIT: I do not know why the Travis CI builds have failed. I cannot view the logs.

Travis did fail because:
  Failed Tests: There was 1 failure
    parses mssql connections, aliasing host to server
      ✖ should be equivalent
  total:     257
  passing:   256
  failing:   1
  duration:  3.3s

I'm not sure about this PR... it does fix parsing a bit, but it still doesn't support any stranger mssql connection string formats and I'm not sure if this is correct implementation (I know some other dialects also parses query parameters and passes them to driver...). Some more generic way for this standard type of query string would be good and then additional parsers for weirder stuff like mssql other connection string types.
Also test would be needed to check that new code path is executed correctly.
I don't have time in next couple of weeks to investigate further is this correct way to so hopefully someone else will.

 Any progress on this one?
Am working on deploying a dockerized node app in Azure at the moment and would love to be able to use this along with the Connection String setting in Azure.

@mattyweb If you are up for resolving conflicts, writing a test and testing it to work properly, this PR definitely could be accepted :)

@khell Are you still interested in finishing this PR?

This seems to be redundant with https://github.com/knex/knex/pull/3547


[Meson] dependency to tizen pkgs

Code clean, define dependencies to tizen packages.

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #303. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


Added decode support for RawType.uuid

looks good :)
could you please also bump the version (0.4.1) and reference this PR in the changelog (([#13](...) by [steffenhaak](https://github.com/steffenhaak)). thank you!


Feature request: auto-unpack tar files

Hi,
Would it be possible to add an option or plugin to auto-unpack tar files when staged? This would be handy when staging directories with test data, to be used within nf-test
Thanks
M

In the meantime we could just do it outselves? https://stackoverflow.com/questions/7128171/how-to-compress-decompress-tar-gz-files-in-java


Can you install this Lib as a dev dependency

Is your feature request related to a problem? Please describe.
If I use PrivateAssets="all" my app crashes.
It is looking for the lib at runtime even though I only use it for generating assets at build time
Describe the solution you'd like
Ability to use Lib as a dev dependency in order to reduce app payload

We are specifying <PrivateAssets>All</PrivateAssets> for the NuGet package AspNetCore.SassCompiler in our .csproj files and it works for us at runtime. Therefore it is generally possible.
How is your setup? Are you specifying any options in appsettings.json? Or are you using sasscompiler.json as we are? Are you maybe using services.AddSassCompiler(); outside of DEBUG environment? What is the error you get at runtime?

We are using sasscompiler.json
No services.AddSassCompiler();
Error is here
https://github.com/MudBlazor/MudBlazor/pull/7933#issuecomment-1863130585
Currently the PR I have removed the All in order to get it to work

Its because of this https://github.com/koenvzeijl/AspNetCore.SassCompiler/blob/2d7ff3808660c9a557a00d4f943055ac365460f7/AspNetCore.SassCompiler/build/AspNetCore.SassCompiler.props#L29
So event If I set PrivateAssets="all" all it still modifies the assemblyinfo with an unknown namespace

Hi @mikes-gh ,
It indeed looks like the watcher is determined to be enabled and thus will generate the assembly attribute. It is only enabled in Debug configuration, but I noticed that you're using it in a RazorClassLibrary so then I think it should indeed also be disabled in Debug mode.
One way you can force it to disable the watcher is to add <SassCompilerEnableWatcher>false</SassCompilerEnableWatcher> inside of a <PropertyGroup>, that way it also won't generate the assembly attribute anymore.
But I think this should be disabled by default for a RazorClassLibrary. When we added support for those projects we also added a check that disables the watcher if the project it was used in is not an executable. However, that seemed to have caused to also disable the watcher in projects where it shouldn't (see #77 and how we fixed it)
I don't have enough information why that caused an issue and broke it for some people, one thing that comes to mind is maybe we should do a case-insensitive check so that if OutputType was lowercase exe it would still enable the watcher. But I don't know if that really was the case at the time, or that there is another issue with this check, which makes me hesitant to add it back.

Hi @mikes-gh ,
It indeed looks like the watcher is determined to be enabled and thus will generate the assembly attribute. It is only enabled in Debug configuration, but I noticed that you're using it in a RazorClassLibrary so then I think it should indeed also be disabled in Debug mode.
One way you can force it to disable the watcher is to add <SassCompilerEnableWatcher>false</SassCompilerEnableWatcher> inside of a <PropertyGroup>, that way it also won't generate the assembly attribute anymore.
But I think this should be disabled by default for a RazorClassLibrary. When we added support for those projects we also added a check that disables the watcher if the project it was used in is not an executable. However, that seemed to have caused to also disable the watcher in projects where it shouldn't (see #77 and how we fixed it)
I don't have enough information why that caused an issue and broke it for some people, one thing that comes to mind is maybe we should do a case-insensitive check so that if OutputType was lowercase exe it would still enable the watcher. But I don't know if that really was the case at the time, or that there is another issue with this check, which makes me hesitant to add it back.

Its actually a blazor class library.  Not sure if that matters.  Thanks for the info it was very helpful

@sleeuwen

One way you can force it to disable the watcher is to add false inside of a , that way it also won't generate the assembly attribute anymore.

That solved it no more runtime dependencies :-)

Somehow this issue is a very strange one. For me, it is unclear why I even need to set SassCompilerEnableWatcher in an Razor Component Library Project because it should be set to false automatically as implemented in .props file, see
https://github.com/koenvzeijl/AspNetCore.SassCompiler/blob/6e063ee468b5429a6a066d31c0b59ae0dab6577b/AspNetCore.SassCompiler/build/AspNetCore.SassCompiler.props#L30.
I tried to debug why the condition is not true. It seems like $(OutputType) is empty when the .props file is imported, but when I log the property via

in the RCL project, the output is Library.
Why on earth is $(OutputType) empty when the .props file is imported?


Weird mismatch between client and SSR

Version
v2.4.0
Reproduction link
https://codesandbox.io/s/rlmxx1p0m4
Steps to reproduce
Let the sandbox load
Click the "refresh" button within the sandbox preview
What is expected ?
Both loading spinners should be rendered properly, with a radial gradient, on both page loads.
What is actually happening?
The spinners load properly the first time, but then every subsequent load they are broken.
Additional comments?
So this is a weird one. I created 2 spinners to demonstrate the issue in 2 ways. The first spinner uses _uid to make a unique reference to the component. The second spinner uses Math.random() to generate random numbers in the data property.
If you inspect element when loading spinner appears broken (no radial gradient), you can see that the 2 places in each component the id is interpolated, the numbers don't match up. It appears that part of the component (the linearGradient element) is rendered AGAIN after page load on the SSR and then spit back out. It is insanely bizarre.
You can see in the browser console log and in the console on the server (in the sandbox) that the numbers output are different from each other, even within the same component.
This screenshot (which is kind of a mess) shows the console log on the server vs the browser and how they are output in the HTML: https://d.pr/i/dL7qtV

This bug report is available on Nuxt community (#c8607)

That's indeed strange 🤔
Not sure whether it's a Vue or a Nuxt issue.

@joelworsham Not necessarily. The "issue" appears because of hydration it seems. This can only happen after SSR but could be a bug in the vue-server-renderer instead of Nuxt itself.

Interesting. Well, I don't know enough about under the hood to figure out where it's coming from. The only reason I came here is because our project is using Nuxt, and I couldn't seem to reproduce the bug without Nuxt.
I'm really either hoping to A) Discover a bug that can be fixed or B) Have a workaround pointed out to me that we can use.
This one has me banging my head against the wall

This caught me right now on a client project because transitions will trigger on hydration when using dynamic components :see_no_evil:

bump :)

This is an upstream Vue SSR bug; closing it here.


Adding font scaling based on system font size for accessibility on desktop

Use case
According to https://docs.flutter.dev/ui/accessibility-and-internationalization/accessibility#large-fonts, font scaling based on Android and iOS system-wide font settings works.
I tested this with GNOME on Linux, and changing the interface font size did not adjust the font size in Flutter. I suspect that the same applies to Windows and macOS devices.
Font scaling on desktop would be wonderful for high pixel-density screens, and for the sake of accessibility.
Proposal
It would be great if font scaling based on system settings could also work for desktop. It seems better to have a single API for font scaling in Flutter apps than to depend on a combination of behavior for phone vs desktop platforms.
Thanks! 😀

looks like we can use https://developer.apple.com/documentation/uikit/uifont/1619042-systemfontofsize?language=objc to look up scaling for a certain font in macOS
not sure about windows and linux. cc @cbracken do you know if we have this API ?

Oops looks like I read your question backwards. On Linux @robert-ancell would know. For windows you want this setting You might be able to capture the setting change by watching for WM_SETTINGCHANGE messages, which is what would fire for e.g. high contrast mode. I'm not sure what API call can be used to fetch the updated scaled text size though.

The Linux embedder sends the font scaling factor on the flutter/settings channel. It is got from GSettings (gsettings get org.gnome.desktop.interface text-scaling-factor). Is this not working for your system?

I'm not sure off-hand what the correct Windows API is. From a quick search, this stackoverflow post recommends this WinRT API. I'm not sure if win32 has an equivalent API.


Behaviour of waitForReady and delay

What is the behaviour of creevey if I set the following story parameters:
export default {
  parameters: {
    creevey: {
      waitForReady: true,
      delay: 2000,
    },
  },
}

Will creevey wait for the call of window.__CREEVEY_SET_READY_FOR_CAPTURE__.() and wait then 2000 ms to make the screenshot or will creevey make the screenshot immediately when window.__CREEVEY_SET_READY_FOR_CAPTURE__.() get's called?

@pernsteinerc hi, the first option, creevey will wait for __CREEVEY_SET_READY_FOR_CAPTURE__ call and then wait 2000 ms. I guess the name or global function is confusing, because of how creevey works. It sends to a browser a request to switch the story or re-mount it and waits for a response. In a browser Creevey waits for when the story will be rendered in that moment story might want to update itself or load something asynchronous, that's where waitForReady plays the role. When the story gets stable, a browser sends a response to Creevey, where it waits for extra 2 seconds before capturing the screenshot.


add one more condition for user name 1st chr must be a chr

"Closes: #1"
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[x] Tests pass
[ ] Appropriate changes to README are included in PR

thx for this class it is very good


[kernel] Updating vboxmanage to fixing parse in Virtual Box 5.0

This PR closes #481.
In Virtual Box 5.0, there is a slight change in the ouput of the command VBoxManage showvminfo azk-vm-dev.azk.io --machinereadable:
# Virtual Box < v5.0:
VideoMode="720,400,0"@0,0
# Virtual Box v5.0:
VideoMode="720,400,0"@0,0 1

Updating gullitmiranda/node-vboxmanage, this isse was solved.

LGTM

Guys, its serious issue. Please pull this PR.

You're right @t3hk0d3.
We are doing the testing and final adjustments to launch a release with a new version of azk (v0.15.0).

@t3hk0d3: The fix for this issue is available in azk v0.14.6.
May you check if this worked for you?
Thank you for your support!


Check tests for old use cases and inconsistencies

It seems that we have old use cases that are no longer accurate and are misleading to users (and even the team).
One example is this , the market never sells the datatokens. https://github.com/oceanprotocol/ocean.js/blob/727a55046c44c1841360f6aa2a4f94befe157860/test/integration/Marketplaceflow.test.ts#L359
We need to check them

Will be made obsolete by new ocean.js which also start fresh with its tests


Update readme for v1 api

[X] I updated the README.md docs to reflect this change.
[X] This is either not a breaking API change, or I incremented the API version.

:+1: :+1:


Windows graphics changes

Note:
I am not using the previously used variable ssections because I could not find a reliable way to map any of the displays to the values output. I can go back in and add pixelDepth and main with the value from the main display in ssections as the default for all the monitors.
Description
I am opening this pull request because after #853 was submitted I ended up looking at the commands being used to pull the information on windows and noticed that there was some room for improvement. I fixed an issue where in some situations the output would exclude displays. I have also added the fields displayId and productionYear which was previously only available on MacOS.
Notable changes
The number of displays is now based on the variable tsections. I then match the InstanceName property to the Instance name property in msections and isections. For dsections I am finding the correct display by matching tsections.InstanceName to dsections.PNPDeviceID
After all of the objects associated with the display have been found the app will then pull variables in this order:
tsection
dsection
msection
and finally isection
Since the output of dsection is the most complete I pulled what I could from that object first and then went down the list from there. If a variable was not pulled from dsection or the correlated object could not be found I would then fill in blank values or pull backup variables from the other objects.
Also:

Added the fields: displayId and productionYear

Examples
In this situation I am using a RDP connection to a different desktop
Existing module
    "displays": [
      {
        "vendor": "",
        "model": "",
        "deviceName": "\\\\.\\DISPLAY17",
        "main": true,
        "builtin": false,
        "connection": "DVI",
        "resolutionX": 2560,
        "resolutionY": 1440,
        "sizeX": 53,
        "sizeY": 30,
        "pixelDepth": 32,
        "currentResX": 2560,
        "currentResY": 1440,
        "positionX": 0,
        "positionY": 0,
        "currentRefreshRate": 84
      }
    ]

Modified module
    "displays": [
      {
        "vendor": "Dell Inc.",
        "deviceName": "Dell E2414H (Digital - DVI)",
        "model": "Dell E2414H",
        "productionYear": 2014,
        "serial": "VMT444ASU",
        "displayId": "DISPLAY\\DEL4091\\5&21dbb09b&0&UID24832_0",
        "main": "",
        "builtin": false,
        "connection": "DVI",
        "sizeX": 53,
        "sizeY": 30,
        "pixelDepth": 0,
        "resolutionX": 2560,
        "resolutionY": 1440,
        "currentResX": 2560,
        "currentResY": 1440,
        "positionX": 0,
        "positionY": 0,
        "currentRefreshRate": 0
      },
      {
        "vendor": "D E L",
        "deviceName": "DELL U719D",
        "model": "DELL U719D",
        "productionYear": 2018,
        "serial": "GLQW7R",
        "displayId": "DISPLAY\\DEL415F\\5&21dbb09b&0&UID24833_0",
        "main": "",
        "builtin": false,
        "connection": "HDMI",
        "sizeX": 60,
        "sizeY": 34,
        "pixelDepth": 0,
        "resolutionX": 0,
        "resolutionY": 0,
        "currentResX": 0,
        "currentResY": 0,
        "positionX": 0,
        "positionY": 0,
        "currentRefreshRate": 0
      },
      {
        "vendor": "L E X",
        "deviceName": "A409U",
        "model": "A409U",
        "productionYear": 2015,
        "serial": "16843009",
        "displayId": "DISPLAY\\LEX0000\\5&21dbb09b&0&UID24836_0",
        "main": "",
        "builtin": false,
        "connection": "DP",
        "sizeX": 128,
        "sizeY": 72,
        "pixelDepth": 0,
        "resolutionX": 0,
        "resolutionY": 0,
        "currentResX": 0,
        "currentResY": 0,
        "positionX": 0,
        "positionY": 0,
        "currentRefreshRate": 0
      },
      {
        "vendor": "H W",
        "deviceName": "H V41",
        "model": "H V41",
        "productionYear": 2014,
        "serial": "3CQ4113RS",
        "displayId": "DISPLAY\\HWP311F\\5&21dbb09b&0&UID24839_0",
        "main": "",
        "builtin": false,
        "connection": "DVI",
        "sizeX": 52,
        "sizeY": 29,
        "pixelDepth": 0,
        "resolutionX": 0,
        "resolutionY": 0,
        "currentResX": 0,
        "currentResY": 0,
        "positionX": 0,
        "positionY": 0,
        "currentRefreshRate": 0
      },
      {
        "vendor": "(Standard monitor types)", //Current display 17
        "deviceName": "Generic Non-PnP Monitor",
        "model": "Generic Non-PnP Monitor",
        "productionYear": 0,
        "displayId": "DISPLAY\\Default_Monitor\\1&31c5ecd4&0&UID256_0",
        "main": "",
        "builtin": true,
        "connection": "RDP",
        "sizeX": 0,
        "sizeY": 0,
        "pixelDepth": 0,
        "resolutionX": 0,
        "resolutionY": 0,
        "currentResX": 0,
        "currentResY": 0,
        "positionX": 0,
        "positionY": 0,
        "currentRefreshRate": 0
      }
    ]

Without RDP original code:
    "displays": [
      {
        "vendor": "Dell Inc.",
        "model": "Dell E2414H (Digital - DVI)",
        "deviceName": "\\\\.\\DISPLAY1",
        "main": false,
        "builtin": false,
        "connection": "DVI",
        "resolutionX": 2560,
        "resolutionY": 1440,
        "sizeX": 53,
        "sizeY": 30,
        "pixelDepth": "32",
        "currentResX": 2560,
        "currentResY": 1440,
        "positionX": -2560,
        "positionY": 740,
        "currentRefreshRate": 84
      },
      {
        "vendor": "",
        "model": "",
        "deviceName": "\\\\.\\DISPLAY2",
        "main": true,
        "builtin": false,
        "connection": "HDMI",
        "resolutionX": 3840,
        "resolutionY": 2160,
        "sizeX": 60,
        "sizeY": 34,
        "pixelDepth": "32",
        "currentResX": 3840,
        "currentResY": 2160,
        "positionX": 0,
        "positionY": 0,
        "currentRefreshRate": 84
      },
      {
        "vendor": "",
        "model": "",
        "deviceName": "\\\\.\\DISPLAY3",
        "main": false,
        "builtin": false,
        "connection": "DP",
        "resolutionX": 1920,
        "resolutionY": 1080,
        "sizeX": 128,
        "sizeY": 72,
        "pixelDepth": "32",
        "currentResX": 1920,
        "currentResY": 1080,
        "positionX": 7,
        "positionY": -1080,
        "currentRefreshRate": 84
      },
      {
        "vendor": "",
        "model": "",
        "deviceName": "\\\\.\\DISPLAY4",
        "main": false,
        "builtin": false,
        "connection": "DVI",
        "resolutionX": 1920,
        "resolutionY": 1080,
        "sizeX": 52,
        "sizeY": 29,
        "pixelDepth": "32",
        "currentResX": 1920,
        "currentResY": 1080,
        "positionX": 1927,
        "positionY": -1080,
        "currentRefreshRate": 84
      }
    ]

Without RDP modified code:
"displays": [
      {
        "vendor": "Dell Inc.",
        "deviceName": "Dell E2414H (Digital - DVI)",
        "model": "Dell E2414H",
        "productionYear": 2014,
        "serial": "VMT444ASU",
        "displayId": "DISPLAY\\DEL4091\\5&21dbb09b&0&UID24832_0",
        "main": "",
        "builtin": false,
        "connection": "DVI",
        "sizeX": 53,
        "sizeY": 30,
        "pixelDepth": 0,
        "resolutionX": 0,
        "resolutionY": 0,
        "currentResX": 0,
        "currentResY": 0,
        "positionX": 0,
        "positionY": 0,
        "currentRefreshRate": 0
      },
      {
        "vendor": "D E L",
        "deviceName": "DELL U719D",
        "model": "DELL U719D",
        "productionYear": 2018,
        "serial": "GLQW7R",
        "displayId": "DISPLAY\\DEL415F\\5&21dbb09b&0&UID24833_0",
        "main": "",
        "builtin": false,
        "connection": "HDMI",
        "sizeX": 60,
        "sizeY": 34,
        "pixelDepth": 0,
        "resolutionX": 0,
        "resolutionY": 0,
        "currentResX": 0,
        "currentResY": 0,
        "positionX": 0,
        "positionY": 0,
        "currentRefreshRate": 0
      },
      {
        "vendor": "L E X",
        "deviceName": "A409U",
        "model": "A409U",
        "productionYear": 2015,
        "serial": "16843009",
        "displayId": "DISPLAY\\LEX0000\\5&21dbb09b&0&UID24836_0",
        "main": "",
        "builtin": false,
        "connection": "DP",
        "sizeX": 128,
        "sizeY": 72,
        "pixelDepth": 0,
        "resolutionX": 0,
        "resolutionY": 0,
        "currentResX": 0,
        "currentResY": 0,
        "positionX": 0,
        "positionY": 0,
        "currentRefreshRate": 0
      },
      {
        "vendor": "H W",
        "deviceName": "H V41",
        "model": "H V41",
        "productionYear": 2014,
        "serial": "3CQ4113RS",
        "displayId": "DISPLAY\\HWP311F\\5&21dbb09b&0&UID24839_0",
        "main": "",
        "builtin": false,
        "connection": "DVI",
        "sizeX": 52,
        "sizeY": 29,
        "pixelDepth": 0,
        "resolutionX": 0,
        "resolutionY": 0,
        "currentResX": 0,
        "currentResY": 0,
        "positionX": 0,
        "positionY": 0,
        "currentRefreshRate": 0
      }
    ]

@shoustech wow, thank you! I will check it the upcoming days and if is also works here on my side, I will crate a new release.

@shoustech i am really interested in the code to find out if bitlocker is set up. i could not find it in your fork, where could i find the code?


Turn on Edit session badge updates when there are no working changes

Testing #161872

Ensure edit sessions are off on the desktop
Open any remote repository and do not make any changes, ex: https://insiders.vscode.dev/github/microsoft/vscode
Continue working on desktop
Open the cloned repository and see that badge appears to turn on edit session

I still see the badge

Sorry, I missed verification steps. To verify this issue specifically (which I just checked works as expected in latest insiders):

(clearing bad insiders-only local state) if you already have the badge on desktop, turn edit sessions on and then off
Open a remote repository and have no working changes
Continue working on desktop
Open the cloned repository and there should be no accounts badge

Am I supposed to have edit sessions enabled or disabled on the web? WIth them enabled, but no changes, I ran the steps and have a badge in my new cloned repo in insiders


fix #65 focus does not scroll if cell is already active

The spreadsheet will not scroll to the target cell if the cell is already active. This solution patches the issue by selecting a different cell before selecting the target cell.

Thanks for the fix @corburn! I tested this locally and noticed that there's a display artifact where the active cell will change to another cell and then back to the intended target. I considered implementing this hack earlier when I first noticed the bug, but decided to wait on Google to fix the bug upstream (they appeared to have noticed it because it was categorized as a bug). I'd prefer to wait for a fix rather than have the active cell jump back and forth (it's also a bit slower because an additional call to setActiveSelection is necessary, and that API is really slow for some reason).

@jairideout  thanks for testing the fix and letting me know about the display artifact. I did not see it and would have probably otherwise used this on different project.

I think the display artifact may not show up in certain cases, depending on how Google is batching its spreadsheet operations behind the scenes. Maybe that was why you didn't see it on your end. For me it was happening quite a bit unfortunately.
If you have any other ideas for a fix I'd love to hear them! This is a pretty annoying bug :(


Do not delete on focus but on start of typing

Great script, thank you for this.
Because of auto focus on start, its important for me not to delete placeholder before starting with typing. Is it possible to rewrite it to delete placeholder on type instead focus ?
Thanks again,
great script.

Was @mlms13 fork ever merged into this library?

Was anything similar to @mlms13 fork added to this library? Why was it closed and not merged?

@gudmundurmar I don't think it was ever merged. When they changed the default branch from master to gh-pages all of the PRs against the master branch were automatically closed.
The PR was open for about a year, but it was kind of a lot of work to keep it up-to-date with master, so eventually I stopped maintaining my fork. The PR is here if anyone else wants to tackle it: https://github.com/mathiasbynens/jquery-placeholder/pull/209

+1

+1


Inner ScrollView does not scroll the parent SingleChildScrollView

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Just run my code.
Expected results
I want to scroll parent SingleChildScrollView after the inner scroll view (with a specific height) is fully scrolled. Please note I have specific requirements constraint the height of the inner ListView.
Actual results
I'm not able to scroll when the inner Listview is completely scrolled but I can scroll if the inner Listview has padding from left or right and when I try to scroll from that padded area I can scroll parent ScrollView in my case It's SingleChildScrollView.
Code sample
Code sample
SingleChildScrollView( child: Column(children: [

      SizedBox(
        height: 700,
        child:ListView.builder(
            itemCount:
          20, itemBuilder: (context, index) {
            return const ListTile(leading: Icon(Icons.icecream,
              color: Colors.amber,), title: Text("Ice Cream"),);
          },),
        ),

      SizedBox(
        height: 300,
        child: ListView.builder(
          itemCount: 20, itemBuilder: (context, index) {
          return const ListTile(
            leading: Icon(Icons.cake, color: Colors.red,),
            title: Text("Cake"),);
        },),
      ),

    ],),)

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.10.4, on macOS 13.4 22F66 darwin-x64, locale en-GB)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.1)
[✓] IntelliJ IDEA Community Edition (version 2022.2.3)
[✓] VS Code (version 1.79.1)
[✓] Connected device (3 available)
[✓] Network resources

• No issues found!

@ianloic Please have a look at this issue.

Add NeverScrollableScrollPhysics and shrinkWrap on all listviews
Example :
ListView(
          physics: NeverScrollableScrollPhysics(),
          shrinkWrap: true,
          ...

Add NeverScrollableScrollPhysics and shrinkWrap on all listviews
Example :
ListView(
          physics: NeverScrollableScrollPhysics(),
          shrinkWrap: true,
          ...

If I do this I won't be able to scroll my inner ListViews as they have a constraint height.

remove height 300,700 and put this code inside all listview
 physics: NeverScrollableScrollPhysics(),           shrinkWrap: true,

@mehroze-zaidi It seems @HassanBalousha's solution can solve your issue. Please try it to see if it's applicable to your case.

@huycozy @HassanBalousha Hi! I hope you both guys are doing well. Perhaps I wasn't able to explain my question enough so here is my issue: I have SingleChildScrollView as a parent and in that, I have two listviews each list view is wrapped with SizedBox with a specific height (like 700), what I want is, when I scroll up all the views that are in the first list, the first Listview should scroll up and then I'll be able to scroll next Listview, Please have a look into the code for more good understanding. Your help means a lot to me. Thank you in advance.
Note1: Setting fixed height is required behavior in my case, I can't use the technique you mentioned because my client wants this behavior that I mentioned.
Note2: I'm getting this required behavior in Chrome but not on a mobile device
For better understanding, you guys can also visit Stackoverflow as I asked the same question there.
Link: https://stackoverflow.com/questions/73166712/i-want-to-scroll-up-after-the-all-childs-of-listview-have-been-scrolled-up-pare

@anaisbetts

I have nothing to do with this bug and have no idea why I am being mentioned

@mehroze-zaidi Since you need this on Web, I re-check this on Chrome and see it works as you expected:
https://github.com/flutter/flutter/assets/104349824/100406a5-67c1-4c0b-94a9-22903d157682
If it's still not your expectation, can you please provide a demo visual (video/gif..) for that?

@mehroze-zaidi No. Please do not CC me on bugs. I do not work on Flutter and I am not employed by Google.


Fix the typo in VM status

Short description:
There is a typo in VM statuses: the word "stopped" should begin with a
capital letter S

Please make the pr description more clear

Please make the pr description more clear

done

!verified

!cherry-pick v4.12

!cherry-pick v4.11

!cherry-pick v4.10

!cherry-pick v4.9