Updated Notes section

Added: "* Synonyms can only be added to a project that has at least one question and answer pair."

@jboback – Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"

@ShannonLeavitt : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Add Deployments tab to the Component Details page

Fixes
Fixes HAC-4697
Description
Adds a Deployments tab to the component details page
Type of change

[x] Feature

Screen shots / Gifs for design review

/cc @karthikjeeyar

/retest

/retest

/retest

/lgtm
/approve


Add delay node

Adds a simple delay node.

Ok never mind. This is (almost) the same as Timer 😄. Thanks to AcinexGames | Xandrios from Discord for pointing out.


Support arrays of hierarchical settings

The $config system supports setting Arrays and Structs (php associative arrays), but for some config keys, an Array of Structs is required.
The most prominent example is perhaps apps_paths:
'apps_paths' => [
        [
                'path'=> '/var/www/nextcloud/apps',
                'url' => '/apps',
                'writable' => true,
        ],
],

I expected to be able to use this from hiera yaml as:
nextcloud::config:
  app_paths:
    - path: /opt/nextcloud/apps
      url: /apps
      writable: false
    - path: /opt/nextcloud-user-apps/
      url: /user-apps/
      writable: true

This fails with:

parameter 'value' expects a value of type Boolean, Integer, or String, got Struct
parameter 'verify_value' expects a value of type Boolean, Integer, or String, got Struct

Currently, this can be worked around by putting the array indices into the key names:
nextcloud::config:
  app_paths 0:
      path: /opt/nextcloud/apps
      url: /apps
      writable: false
  app_paths 1:
      path: /opt/nextcloud-user-apps/
      url: /user-apps/
      writable: true

Perhaps the code from nextcloud::config can be extracted into a recursive function to properly support this?

@oxc Could you provide an example how to set these values using the occ config:system:set commands?

@oxc Could you provide an example how to set these values using the occ config:system:set commands?

Of course. Since the described workaround works, the commands must be:
occ config:system:set apps_paths 0 path  /opt/nextcloud/apps
occ config:system:set apps_paths 0 url /apps

etc.
This also matches the puppet run output.

Of course. Since the described workaround works, the commands must be:
occ config:system:set apps_paths 0 path  /opt/nextcloud/apps
occ config:system:set apps_paths 0 url /apps

etc.
This also matches the puppet run output.

We had a similar problem with dovecot configs, where multiple levels of nesting are possible. The branches for Array and Hash just collect the key hierarchy and recursively delegate to the function itself:
https://github.com/oxc/puppet-dovecot/blob/master/functions/create_config_resources.pp

We had a similar problem with dovecot configs, where multiple levels of nesting are possible. The branches for Array and Hash just collect the key hierarchy and recursively delegate to the function itself:
https://github.com/oxc/puppet-dovecot/blob/master/functions/create_config_resources.pp

Implemented in https://github.com/markt-de/puppet-nextcloud/commit/a09fb94def04e61aafc2345a860bd73f70047a7e. Please give it a try and report back.


fstar-mode error message attributes error to exernal module

When reporting an error (in a newly opened split buffer, but this is probably a separate issue), fstar-mode includes the name of the most recently open'd module like this:
Example code, which causes an error due to the missing 't' at the end of "nat":
module Test

open FStar.Seq

type x = na

/home/kk/repositories/everest/FStar/ulib/FStar.Seq.Properties.fst(716,0-723,49): (Error) Identifier not found: [na](Also see: (5,9-5,11))] (FIXME)

Without opening FStar.Seq, the error is just

F*: subprocess exited.

(without openeing a split buffer). While running fstar.exe Test.fst from the command line just outputs:

./Test.fst(5,9-5,11) : (Error) Identifier not found: [na]

in both cases
Or maybe this is due to a configuration error on my part?

When using hints, the error message becomes:

Warning (emacs): F* checker reported issues in other files: [/home/kk/repositories/everest/FStar/ulib/FStar.Seq.Properties.fst(716,0-723,49): (Error) Identifier not found: [na](Also see: <
input>(5,9-5,11))] (FIXME)

This seems to be an issue with F* itself, not fstar-mode.el
$ cat transcript
#push 1 0
module M
open FStar.Seq
let x = x
#end #done-ok #done-nok

$ cat transcript | bin/fstar.exe --in M.fst
desugar_partial_module curmod=None
[...]/FStar.Seq.Properties.fst(716,0-723,49): (Error) Identifier not found: [x](Also see: <input>(3,8-3,9))
#done-nok

Closing this since we're tracking it in FStar.


Update Developer Guide

Changes
This PR amends the following in the Developer Guide:

Standardise Use Case line breaks
Include tag find subsystem (proposed feature)
Manual testing for Tabbed views (team's comments needed to see if I need to add on to anything)

A live demo of the changes can be found here
Related Issues
None.

Codecov Report

Merging #337 (a569390) into master (40f21dc) will increase coverage by 0.30%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #337      +/-   ##
============================================
+ Coverage     72.20%   72.50%   +0.30%
- Complexity     1261     1280      +19
============================================
  Files           160      160
  Lines          3889     3914      +25
  Branches        458      460       +2
============================================
+ Hits           2808     2838      +30
+ Misses         1002     1001       -1
+ Partials         79       75       -4

Impacted Files
Coverage Δ

src/main/java/seedu/contax/model/ModelManager.java
96.85% <0.00%> (-1.42%)
:arrow_down:

...java/seedu/contax/logic/commands/BatchCommand.java
86.30% <0.00%> (+0.19%)
:arrow_up:

src/main/java/seedu/contax/model/AddressBook.java
98.70% <0.00%> (+0.23%)
:arrow_up:

...ain/java/seedu/contax/model/tag/UniqueTagList.java
100.00% <0.00%> (+2.27%)
:arrow_up:

...va/seedu/contax/model/person/UniquePersonList.java
100.00% <0.00%> (+2.43%)
:arrow_up:

src/main/java/seedu/contax/model/person/Email.java
100.00% <0.00%> (+10.00%)
:arrow_up:

src/main/java/seedu/contax/model/person/Phone.java
100.00% <0.00%> (+10.00%)
:arrow_up:

...c/main/java/seedu/contax/model/person/Address.java
100.00% <0.00%> (+10.00%)
:arrow_up:

src/main/java/seedu/contax/model/tag/Tag.java
100.00% <0.00%> (+11.11%)
:arrow_up:

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 40f21dc...a569390. Read the comment docs.


Add a general DEFAULT_WAIT constant for all the wait_times in bdd tests

there were lots of different wait_times all over the place... I figured one (overridable) constant might make things neater / easier to do a global increase or decrease?

Absolutely.  Makes sense to me!


MM-19614 - Added local_only parameter to GetMarketplacePlugins

Summary

Added local_only parameter to GetMarketplacePlugins to support displaying local plugins in the webApp when the remote marketplace server is unavailable.
Ticket Link
MM-19614
Related PRs
WebApp: WIP
Server PR: https://github.com/mattermost/mattermost-server/pull/13250

Merging to feature branch


iOS build failed

I have got this error message via iOS building
  2 warnings generated.
    /Users/builder/programs/flutter/.pub-cache/hosted/pub.dartlang.org/receive_sharing_intent-1.3.0/ios/Classes/SwiftReceiveSharingIntentPlugin.swift:69:84: error: 'UIApplicationOpenURLOptionsKey' has been renamed to 'UIApplication.OpenURLOptionsKey'
        public func application(_ application: UIApplication, open url: URL, options: [UIApplicationOpenURLOptionsKey : Any] = [:]) -> Bool {
                                                                                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                                                                UIApplication.OpenURLOptionsKey
    UIKit.UIApplicationOpenURLOptionsKey:2:18: note: 'UIApplicationOpenURLOptionsKey' was obsoleted in Swift 4.2
    public typealias UIApplicationOpenURLOptionsKey = UIApplication.OpenURLOptionsKey
                     ^
    /Users/builder/programs/flutter/.pub-cache/hosted/pub.dartlang.org/receive_sharing_intent-1.3.0/ios/Classes/SwiftReceiveSharingIntentPlugin.swift:69:17: warning: instance method 'application(_:open:options:)' nearly matches optional requirement 'application(_:open:options:)' of protocol 'FlutterPlugin'
        public func application(_ application: UIApplication, open url: URL, options: [UIApplicationOpenURLOptionsKey : Any] = [:]) -> Bool {
                    ^
    /Users/builder/programs/flutter/.pub-cache/hosted/pub.dartlang.org/receive_sharing_intent-1.3.0/ios/Classes/SwiftReceiveSharingIntentPlugin.swift:69:17: note: move 'application(_:open:options:)' to an extension to silence this warning
        public func application(_ application: UIApplication, open url: URL, options: [UIApplicationOpenURLOptionsKey : Any] = [:]) -> Bool {
                    ^
    Flutter.FlutterPlugin:43:26: note: requirement 'application(_:open:options:)' declared here
        optional public func application(_ application: UIApplication, open url: URL, options: [UIApplication.OpenURLOptionsKey : Any] = [:]) -> Bool
                             ^
    /Users/builder/programs/flutter/.pub-cache/hosted/pub.dartlang.org/receive_sharing_intent-1.3.0/ios/Classes/SwiftReceiveSharingIntentPlugin.swift:55:36: error: 'UIApplicationLaunchOptionsKey' has been renamed to 'UIApplication.LaunchOptionsKey'
            if let url = launchOptions[UIApplicationLaunchOptionsKey.url] as? URL {
                                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                       UIApplication.LaunchOptionsKey
    UIKit.UIApplicationLaunchOptionsKey:2:18: note: 'UIApplicationLaunchOptionsKey' was obsoleted in Swift 4.2
    public typealias UIApplicationLaunchOptionsKey = UIApplication.LaunchOptionsKey
                     ^
    /Users/builder/programs/flutter/.pub-cache/hosted/pub.dartlang.org/receive_sharing_intent-1.3.0/ios/Classes/SwiftReceiveSharingIntentPlugin.swift:57:58: error: 'UIApplicationLaunchOptionsKey' has been renamed to 'UIApplication.LaunchOptionsKey'
            } else if let activityDictionary = launchOptions[UIApplicationLaunchOptionsKey.userActivityDictionary] as? [AnyHashable: Any] { //Universal link
                                                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                                             UIApplication.LaunchOptionsKey
    UIKit.UIApplicationLaunchOptionsKey:2:18: note: 'UIApplicationLaunchOptionsKey' was obsoleted in Swift 4.2
    public typealias UIApplicationLaunchOptionsKey = UIApplication.LaunchOptionsKey

@KasemJaffer What do you mean? I have set Swift language version in Share Extension to 4.2..  Building with iosSDK 12.2. Xcode 10.2

I have the same issue.
I am using SWIFT_VERSION 5
Seems just a matter of updating those obsolete references

Fixed in the latest version v1.3.3


Add missing MTR targets.

https://issues.redhat.com/browse/MTA-1268
Requires: https://github.com/konveyor/rulesets/pull/21

Done.


added support for gym env kwargs

@kouroshHakha Good addition to the package! I added it to my fork. I think this should be merged as it does not conflict with the current codebase.


Make unique indexes unique

Description of problem
Updated indexes in DNN 9.6.2 should be marked as "UNIQUE" to avoid SQL Server to add an unnecessary ID
Description of solution
CREATE UNIQUE NonClustered INDEX
Description of alternatives considered
none
Additional context
https://blog.sqlauthority.com/2019/07/24/sql-server-performance-impact-of-unique-index/
Affected browser

[x] any

I am going to submit a PR

you simply ROCK man!

still an issue, will submit a PR

still an issue, will submit a PR

still on my plate

this is still on my plate


Remove Flask debug toolbar

What is the context of this PR?
This removes Flask-DebugToolbar dependency from Runner, script in setup.py and all associated tests & environmental variables. Any mentions in README file has been removed as well. Pipfile.lock has been updated and regenerated.
How to review
Run unit and integration tests, check if debug toolbar is not present in Runner.

 Remove Flask Debug Toolbar (s)

A bit confused why we had EQ_DEV_MODE when it just seems a hurdle to get over to use flask-degubtoolbar, but I have checked the code and can't see it being used anywhere else

I did some searching with different keywords and, as you said, it looks like it's only being used as a step before actual flask toolbar script.


fix(book): fix two broken links

Great spot, thanks @not-my-profile

Credit goes to @richb-hanover for spotting it :)


Add tags and attributes as case insensitive

Fixed in https://github.com/hughbe/html-generator/commit/6255516ab11033488cedef3c6cedec2525ed7c8c


Error OSError: No such file as /dataset/ukdale.h5

run saed_model_evaluation
Started training for  SAED
Joint training for  SAED
............... Loading Data for training ...................
Loading data for  ukdale  dataset
Traceback (most recent call last):
File "experiment.py", line 46, in
template.run_template(EXPERIMENT_NAME,
File "/home/user/miniconda/lib/python3.8/site-packages/deep_nilmtk/utils/templates/experiment_template.py", line 71, in run_template
_ = setup(self.experiment, experiment_name=experiment_name,
File "/home/user/miniconda/lib/python3.8/site-packages/deep_nilmtk/utils/experiment_starter.py", line 36, in setup
api_res = API(experiment)
File "/home/user/miniconda/lib/python3.8/site-packages/nilmtk/api.py", line 46, in init
self.experiment()
File "/home/user/miniconda/lib/python3.8/site-packages/nilmtk/api.py", line 91, in experiment
self.train_jointly(clf,d)
File "/home/user/miniconda/lib/python3.8/site-packages/nilmtk/api.py", line 208, in train_jointly
train=DataSet(d[dataset]['path'])
File "/home/user/miniconda/lib/python3.8/site-packages/nilmtk/dataset.py", line 45, in init
self.import_metadata(get_datastore(filename, format))
File "/home/user/miniconda/lib/python3.8/site-packages/nilmtk/utils.py", line 323, in get_datastore
return HDFDataStore(filename, mode)
File "/home/user/miniconda/lib/python3.8/site-packages/nilmtk/docinherit.py", line 53, in f
return self.mthd(obj, *args, **kwargs)
File "/home/user/miniconda/lib/python3.8/site-packages/nilmtk/datastore/hdfdatastore.py", line 18, in init
raise IOError("No such file as " + filename)
OSError: No such file as /dataset/ukdale.h5
Closing remaining open files:/tmp/nilmtk-guod8xa5.h5...done
I already copied the ukdale dataset in dataset folder but still getting the error. Please help.

I believe that you copied the dataset after building the docker image. The following command may solve your problem without re-building the image:
docker run --gpus 'all' --name=exp_exec experiment_setup -v ~/dataset:/dataset


Sort line graph labels based on team id, not lexigraphically

If we have over 10 teams, we sort the teams based on a string comparison, so that our ordering of teams goes:
team1
team10
team11
team12
team2
team3
We need to update this so that it goes team1, team2, team3....team10, team11, team12

There appears to be a bug with chartjs, we are passing in our dictionary sorted (1-10), but it insists on sorting it 1,10, 2-9

Going to investigate highcharts and see if that fixes things?


Improve type hints and docs in timm/models

Part of #1989.
Improves type hints and docs in the following files:

timm/models/mobilenetv3.py
timm/models/resnet.py

Fixes minor typo in CONTRIBUTING.md.
@rwightman I would like to know your thoughts on the changes below, and what you do and don't prefer. Will make sure to follow them for subsequent PRs.
I would also like to say that your implementations here have helped me in understanding a lot about going from paper to code as mentioned in the issue. It is really well written and intuitively understandable in terms of code. Thanks for this!

Also noticed that the formatting command black --skip-string-normalization --line-length 120 <path-to-file> (as mentioned in the contributing guide) has not been run on many files. When executed on the above two files, a long list of unrelated indent changes occur, and so I've not committed that so far.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@a-r-r-o-w thanks! looking overall good so far. Lettting the CI run, annotations shouldn't impact eager but can impact torchscript tests.
A few things:
Black settings were put in there by another Hugging Face dev, it's a common setup for HF repos but I'm opinionated, black is very opinionated and I have some issues so have not made the leap to turn that on. So let's not run it. Still looking for the right balance for formatting rules, black, autopep8, yapf...
The layer typing as you've noticed isn't super consistent, some models support just nn.Module types, some strings, some both, etc. Looks like you're capturing that, but the 'type' of a non-instance of nn.Module is not nn.Module, it's type[nn.Module] (or typing.Type[nn.Module] for pre python 3.9, which we need to support still, officially 3.7 but can bump up to 3.8 now I feel).
Some of the MobileNetV3 and EfficientNet types are very specific to just those two model classes (BlockArgs especially). Should probably constrain those types to _efficientnet_blocks.py as any other model won't use them. Having the typing module for other common types makes sense but should contain types that are broadly common. Also might want to consider moving that to timm/layers/typing.py as may find some of them propagate to layers with an eventual goal to cover them as well.
Similarly, some other model instances might have some non-trivial types specific to them, unless you see it being common to other models, should probably define at the top of the model module.

Makes sense about the formatter part so will put that on hold.
I've added a few changes based on my interpretation about the typing part, and it makes sense for when I open subsequent PRs for more files. Is this what you meant? If not, could you add suggestions to the relevant parts while reviewing?
Also, don't know how I confused the nn.Module as class instances instead of class objects initially :sweat_smile: There are some instances in resnet.py where I thought it was better to put Type[nn.Module] instead of LayerType as the latter is more general (as in Union of types that is then processed by some internal function to yield a nn.Module) and only applies to ResNet (where some preprocessing on the activations is performed) and not BasicBlock (where activations are only instantiated). If this is too convoluted and since typing is not really strict in python, we could just stick to everything being LayerType maybe?

@rwightman Any updates on this?

@a-r-r-o-w I've had hands full pushing some releases out, handling some other things, on quick look it was looking good but will aim to do a more detailed check and merge in a few days.

Sure, take your time as there's no hurry at all. Eventually it should get faster once I understand the format you expect changes in.

@rwightman Added changes from your suggestions. Let me know if this is ready to go.


Fixes OSIO-1500 and adds static wit groups test

Adds functional tests for work item type groups. Functional tests checks if links are present and click-able.
https://openshift.io/openshiftio/openshiftio/plan/detail/1500

@nimishamukherjee fabric8/fabric8-ui:SNAPSHOT-PR-2132-1 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2132-fabric8-planner.badger.fabric8.io


[tcgc] map discriminator string value type to enum value type

fix: https://github.com/Azure/autorest.go/issues/1206

All changed packages have been documented.

:white_check_mark: @azure-tools/typespec-client-generator-core

Show changes
@azure-tools/typespec-client-generator-core - fix ✏

map discriminator string value type to enum value type

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/834/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/834/


Refactoring for layouts + loading indicator

Sorry for the big PR, maybe try to review one commit at a time :)
The refactoring supports current method, so no breaking changes. Show a notice to users.
Loading page looks like this:

The spinner in the middle only show if the delay is > 250ms, which avoid flash spinner (which sucks).
Note the top bar (logarithmic effect: start quick, and the more it loads, the less its fast). Better UX imo compared to most loading topbar which quickly load around 20% then go right to the 100%.
Boilerplate 404 is now like docs

I will add more components and layouts in default boilerplate after that (have local wip, but I found a bug and want to write test for it).
Will closes #182 and #357.

Awesome UX with topbar and loading spinner


Remove init method from batch-loader plugin

Motivation
According this comment https://github.com/backstage/backstage/issues/12254#issuecomment-1217793906 backstage has refresh_state table with entity_ref column, which we can use to query multiple entity refs at the same time
Approach
Remove migrations, init method and refactor get entities query to use join between two tables

The following preview packages were published:

Name
Version
Install Command

@frontside/backstage-plugin-batch-loader
0.1.2-dl-remove-migrations.0
yarn add @frontside/backstage-plugin-batch-loader@dl-remove-migrations

Generated by @thefrontside/actions

@taras It's in 10-20 ms slower. But it's not big deal in comparison with catalog api


Getting mentions from words that end with highlight

I only have ian in my list.

Ruko_ hitting me with that nasty dab in that screenshot


dimension test

After installing MIMIC-III in a local Postgres database, create a test to compare dimensions of local database with online database.

Nice suggestion, thanks Maria. This is something that we have talked about in the past. We'll try to get onto it! Pull requests welcome in the meantime :)

Sort of added with https://github.com/MIT-LCP/mimic-code/blob/fac79ba4ab3a3ca7fc37b24603c0f5021fcc4106/buildmimic/postgres/postgres_checks.sql


Check that version of dataset is set in info

Close #215

Going to use https://github.com/ga4gh/compliance/pull/241 instead


Bottom two leaves of Fig. 1 in Split-reduc paper are wrong (thanks to Andreas Soteriou for pointing this out)

Also, that Figure isn't the best example, since all 3 final functions have minimum = 1, meaning we don't get to show the working of min(a,b,c) for final three functions a,b,c. Also one of a,b,c is still cubic!

Also, that Figure isn't the best example, since all 3 final functions have minimum = 1, meaning we don't get to show the working of min(a,b,c) for final three functions a,b,c. Also one of a,b,c is still cubic!


Feat/reminder interval

Hi @TwiN
Summary
Regarding this issue #599.
Checklist

[x] Tested and/or added tests to validate that the changes work as intended, if applicable.
[X] Updated documentation in README.md, if applicable.

I want this!

Wow I'm really sorry for forgetting about this for so long!
@bugrakocabay any chance you can resolve the conflict and ping me when it's done? I'll get this merged and released ASAP


Housing

Could you please share .stl files for the housing like you did for the receiver?

Sorry for the delay. They are now available.

Thank you. Will post images of my project when it's done ;)


[Bug]: whois does not accept non-Discord IDs

Describe the bug
The /whois command can take either a Discord or HTB ID and retrieve the corresponding linked account. Because the paramater is typed as a Member though, the client does not allow entering numbers such as a HTB ID
https://github.com/hackthebox/Hackster/blob/main/src/cmds/core/user.py#L191
So only the Discord -> HTB lookup works
To reproduce

Type /whois
Fill the user field with a HTB ID (83743)

Expected behavior
I expected that I'd see my Discord account linked
Instead, I saw this error

Relevant log output
No response
Additional context
No response

I don't like having an argument user that can take both a Member and an HTB user_id, so I'm gonna convert the user to optional and will add a second argument called htb_id. What do you think about it?


[#198] Fixed homepage link when running the app on subpath

Resolves #198
@bindeali Thanks for reporting the bug!

@FilipKopecky The Netlify issue is a big one and the solution is not clear. For now let's just assume that the Netlify preview does not work, but at least we can be sure that the app builds when the PR checks complete. I'm merging this to fix the dev instance.


input-next

New input to replace the old one.
Should solve

Cursor position not moving unexpectedly
Abstract all events to own class (called Action)
With upgrades in smoothly add partialFormatting, so that types with separators (price, duration etc.) will behave as expected and not fully format until blur.

So far this is only implemented for price

Bug
Don't allow more then 2 decimals (or whatever the limit is) while writting.

Bug
Spamming Enter on input-next does something weird like tries to add new rows.

price bug

Cursor can be where ever on input-next.
On old input they jumped to relevant places, this shpuld be fixed

replaced by:

https://github.com/utily/smoothly/pull/908

replaced by:

https://github.com/utily/smoothly/pull/908


install specific versions, not master.

this package should install specific versions, so that people can require + depend on a specific go-ipfs version.

done in e859dc5

You should probably change README to indicate that this package's version corresponds with the installed IPFS's version.


Professional Edition - Managed Package

Hi
This looks like a great tool. I have PE edition and am not a registered ISV develpoer. Do you have a managed package of this that could be installed in PE?
We are looking to use the delete with Process Build Tool
Keep up the amazing work!
Dan

I haven't done that.


Is there a way to decompose components of a glyph in a TTF?

I am attempting to decompose components in glyphs within a TTF, but having trouble.
Is there a way to do this?
It seems that there is nothing direct, and that I may need to use pens. I am pretty new to pens, and not sure I understand them completely.
I think I would need to find a way to record points of each component to a pen object, and then draw those points into the ttGlyph with the .drawPoints() method. However, it doesn’t seem that the decomposingRecordingPen works with TTFont objects. I also can’t figure out whether I am using the ttGlyphPen correctly (can it even record points, or is it just a way to store points to later draw?).
def decomposeGlyph(font, glyphName):

    components = (font['glyf'][glyphName].getComponentNames(font['glyf']))

    ## BASIC IDEA (not yet working)
    # for component in glyph
        # record tt points of component base
        # draw points to glyph

    glyphset = font.getGlyphSet()

    for comp in components:
        glyph = glyphset[comp]
        pen = ttGlyphPen.TTGlyphPen(glyph)
        # font['glyf'][glyphName].drawPoints(pen, font['glyf'])
        font['glyf'][glyphName].draw(pen, font['glyf'])

It may or may not be a related goal, but I am also hoping to remove overlap in the glyphs in a TTF. If this is easy to point out, I would love to get a head start on it, but if it’s a totally separate thing, feel free to skip this part and I will look into it separately.
Thank you so much for any tips or insights!

This snippet works:
from fontTools.ttLib import TTFont
from fontTools.pens.recordingPen import DecomposingRecordingPen
from fontTools.pens.ttGlyphPen import TTGlyphPen

src = "font.ttf"
dst = "font-decomposed.ttf"

f = TTFont(src)

glyfTable = f["glyf"]
gs = f.getGlyphSet()
for glyphName in gs.keys():
    if not glyfTable[glyphName].isComposite():
        continue
    dcPen = DecomposingRecordingPen(gs)
    gs[glyphName].draw(dcPen)
    ttPen = TTGlyphPen(None)
    dcPen.replay(ttPen)
    glyfTable[glyphName] = ttPen.glyph()

f.save(dst)

Regarding remove overlap: booleanOperations does not (yet) support quadratics, but I think skia-pathops does.

It should be easy to make a DecomposingRecordingPointPen. Would you like to give it a shot?
Yes, skia pathops supports quadratics. You can see how ufo2ft uses it in its removeOverlaps filter

Just's snippet now also removes overlaps using skia-pathops, see  https://github.com/fonttools/fonttools/commit/3ad201cbe5865e3eda63631ff56b759bdb4162f0

I now realize that a decomposing filter pen would be nice to have, as it eliminates the need to record.

Thank you so much for the snippet, @justvanrossum! This is so helpful. Thanks also for adding it to the snippets of the project.
Huh, being new to pens, I'm still wrapping my head around quite how this works. Probably once I use it, it will start to make more sense. I will try to blog about my pen learnings at some point, to pay it forward.
Thanks, @anthrotype, for the additional insight! I will check out ufo2ft for a reference.

I will check out ufo2ft for a reference.

The way ufo2ft uses pathops.union() function will not work with a TTGlyph object as these don't have getPen() method (unlike defcon or ufoLib2 Glyph). I have already updated Just's script to remove overlaps using skia-pathops. Just use that.

Awesome, thank you for the help on overlap removal! That is amazing.


Placeholder changes selected order in multiselect

When I create my select2 item with:
 <select id="from_lang" name="native" class="select2-multi" multiple="multiple" style="width:100%;">

<script>
 sel = $(".select2-multi")
sel.select2({
                placeholder:"test"
            });
</script>

The selected items are appended at the front instead of the back.
I cannot imagine this is desired behaviour.
A quick fix I found was adding data-placeholder="test"  to the select2 tag

Please, send us a JSFiddle example code simulating this scenario (with runnable code and steps described there, to allow us to follow this steps and check the situation). It is much more easy to understand and help to provide some solution or workaround. You are free to reopen when the JSFiddle are done.


Test for https://github.com/apache/camel-k/issues/1870

see https://github.com/apache/camel-k/issues/1870

@ppalaga @jamesnetherton I don0't have much time to check why this PR is failing, if anyone has some spare time I think it would be nice to have it, otherwise I think we can close it

@lburgazzoli I can try to take this work forward for https://github.com/apache/camel-quarkus/issues/2713.

Closing this one and I'll try to redo the PR on the latest main soon.


Remove sudo from all images

According to dockerfilelint:

Use of sudo is not allowed in a Dockerfile.  From the official document Best practices for writing Dockerfiles:

You should avoid installing or using sudo since it has unpredictable TTY and signal-forwarding behavior that can cause more problems than it solves.
If you absolutely need functionality similar to sudo (e.g., initializing the daemon as root but running it as non-root), you may be able to use gosu.

As Linuxbrew does not need root access this should not be a problem.

I think the idea behind having sudo is that IF those who use these images need sudo for whatever reason they could use it to immediately solve their problem(s). Otherwise, they would have to report it here, wait for the PR to be merged, etc... And even then, their problem might be unrelated to Linuxbrew and if we don't accept their changes, they would have to solve their problem some other way. So, I'm not sure what would be the right decision here (it all depends on what we think these images should be used for).

sudo is needed so that the user of the image can run (for example)
sudo apt-get update && sudo apt-get install vim

I haven't looked into using gosu as a replacement, but sudo has worked fine for me. We're not using it to run a daemon, but to run apt-get, which works fine. I'd appreciate a PR to configure dockerfilelint to disable this particular warning.
Closing without merging, but thanks for your contribution all the same, @VonUniGE .

doesn't apt-get update and apt-get install work fine without sudo on docker?

@AlJohri Not when using the USER instruction, such as:
USER linuxbrew

It seems like the USER command is not used in this Dockerfile: https://github.com/Homebrew/brew/blob/master/Dockerfile
I found that changing the USER to linuxbrew created weird issues where I could no longer copy files to my usual /app folder in my Dockerfile.
Why is this USER linuxbrew necessary? Can it be switched back to USER root at the end if is necessary?

It used to be that Homebrew would refuse to run as root. It still does outside of Docker, but it now allows running as root within a Docker container. So, these Dockefiles could all be updated to match https://github.com/Homebrew/brew/blob/master/Dockerfile where it creates the linuxbrew user but doesn't USER linuxbrew.

Great! Would you like me to submit a PR?

Yes, please. Thanks, Al!


Automation of Unified Service Desk 4.1

Unified Service Desk 3.3 was running in IE browser and pywinauto was helpful to test automation.  However after the upgrade of USD to 4.1 version it is running in Chrome browser and the pywinauto scripts were not working.
Is there any good test automation API's for USD 4.1

@timheuer @jonstoneman @cmaneu Help please

Following this

I found an article titled "Automating Hybrid Apps" from Appium docs. The link is given below. Maybe this could be helpful for you.
http://appium.io/docs/en/writing-running-appium/web/hybrid/

I've checked in detail and I'm afraid this is not supported as mentioned in the issue linked below:
Is it possible to switch from a windows desktop application, to an internet explorer webview?

Yes, I went thru many automation API's and articles related to that, but I do not see that there is a right tool for the automation of Unified Service Desk, since that contains Win32 and Browser.
There was  a software called Rapise, which claims to be the tool for USD automation but  I see that is not really doing as expected

What about tools designed for GUI Web testing like Selenium? Can't they
handle your scenarios?
pt., 18 paź 2019 o 14:30 jjbright<EMAIL_ADDRESS>napisał(a):

Yes, I went thru many automation API's and articles related to that, but I
do not see that there is a right tool for the automation of Unified Service
Desk, since that contains Win32 and Browser.
There was a software called Rapise, which claims to be the tool for USD
automation but I see that is not really doing as expected
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/WinAppDriver/issues/916?email_source=notifications&email_token=AK6K7GZOBZEPFQUVV7VIB2TQPGUADA5CNFSM4JBKLGK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBUHIRA#issuecomment-543716420,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK6K7GYL7QPSTQZE57XVCKTQPGUADANCNFSM4JBKLGKQ
.

We tried, but clicking the USD components are not working

How about mixing WinAppDriver and Selenium WebDriver? Win32 components will be manipulated by WAD; web elements with usage WebDriver?

In our Desktop app we'are using devex controls, which're hosting web browser - I'll check if its possible to do it that way.

We would like to use Python / Java to develop the testing code, so if you have any realtime proof of concept on clicking windows native components(menu items) and also interact with web view (browser rendered) components, let know...

The solution for your problems is described here:
https://docs.microsoft.com/en-us/dynamics365/unified-service-desk/chrome-process?view=dynamics-usd-4.1
You have to turn on accessibility support:
"When you use Chrome Process to host web application in Unified Service Desk, to enable accessibility support for the webpages, the ChromeAccessibilityRenderer is introduced. By default, the accessibility support for chrome process is disabled. Add the UII option and set the value as True to enable and ensure that screen reader applications read the webpage."
After doing that, all web controls will be visible for automation tools.

Closing issue as it's not relevant to WinAppDriver.

Thank you @pm90pl ! Your solution works great!

Did somebody have success with switching to USD webview and working with it as romote sesssion?
I enabled chrome remote debuggin in USD, then
connected to chrome webview session but remote page is empty.

Is it possible to configure USD to use Edge instead of Chrome? If yes then maybe WinAppDriver could help.

@naeemakram
What do you mean use WinAppDriver?
I already use WinAppDriver , but for most WebView controls only text are available and need to use xpath that very very slow.
I want to use Selenium for web part that will be faster.

Using Selenium with a WebView is a different thing. I believe WinAppDriver an work with Edge for automating WebView.
Secondly, found this on Appium's website and thought maybe it will be helpful for you:
http://appium.io/docs/en/writing-running-appium/web/hybrid/

@naeemakram Thank you for help.

Using Selenium with a WebView is a different thing. I believe WinAppDriver an work with Edge for automating WebView.
Using chromium is mandatory for our project, also I cannot find ability to enable remote mode for Edge inside USD.
Secondly, found this on Appium's website and thought maybe it will be helpful for you:
http://appium.io/docs/en/writing-running-appium/web/hybrid/
It is applicable only for mobile, not supported by WinAppDriver


Return workflow failure detail

Also need to test the current behavior

https://github.com/indeedeng/iwf/pull/155


Latex env only completes for \begin{ not for \end{

Hi, I loaded this plugin and found it doesn't work for \end{ completion inside $$. Is it a bug?

Hello @usualwitch,
There is no end in the completions, it's missing. I'm going to review the latex provider, probably using some library instead of listing them all. Do you have any suggestions ?

I usually write tex with vim, and my solution to entering begin/end pairs is just reading from a snippet file and then inserting on a new line.
I don’t know js, so I plan to port my simple vimscript solution to obsidian vimrc. Is it possible for the autocomplete plugin to insert these begin/end in pairs?

Hello @usualwitch,
In the next release I will add the \end tag and in the following releases I will try to improve vim suggestions.
The begin/end pairs will be added with the snippets feature soon, right now I am refactoring the core.
Thanks for your patience,
have a nice day!

I've added \end tags in release 0.8.0. In the future I will improve latex mode handling.


Don't use revoked keys for keyring shares

We should never use keys that are revoked when adding to a keyring:

find the non-revoked version of a keypair for a user when adding them to a keyring, and use that
teach the worklog to re-encode keyring membership with a non-revoked version, when a revoked one is found. This should be auto-resolvable.

This will be hard to test without having revoked keys, so:

add torus keypairs generate --forceto force a revocation/recreation of a set of keypairs.

We also need to ensure we select the right key for encryption: https://github.com/manifoldco/torus-cli/blob/master/daemon/logic/utils.go#L378
We probably want to introduce a Claimtree (all keys in org) and Keypair (users keys) struct that we can attach some methods too for finding a targeted key (and returning an error if the key is revoked or doesn't exist), selecting a valid key from the tree for use for a targeted user, etc.
Then we could introduce another method that takes the Claimtree for the arigato org and the target org to navigate down the entire tree to validate the full authenticity of the key signature chain.


Compatibility with IDEA 2021.2.x

Hi,
Can you make it compatible with the latest IDEA version, please?
Do you need any help? Is this project still active?
Thanks.

If someone is in a hurry to try this plugin with latest JetBrains editors, you can checkout the related PR and then run:
./gradlew
./gradlew build

and the plugin should appear under folder build/distributions/

I'm working on my own version; I'll release it in the next couple of weeks

This is my version, It isn't a fork itself, but I used a lot of code from the codebase https://github.com/MarioAriasC/zig-support
In a couple of days, it'll be ready for download on the plugins website.


Configuration in ASP.NET Core 6

I'm upgrading an ASP.NET Core project from v5 to v6. I'm using this library.
The new docs say I should register Swashbuckle like so:
builder.Services.AddEndpointsApiExplorer();    // what is this?
builder.Services.AddSwaggerGen();

That AddEndpointsApiExplorer is new to v6. Everything works as expected whether I add it or not.
Given that method's name, I'm assuming it is related / similar to this library. How is it related? Should I use it or skip it?

Cross-post on StackOverflow.
PS, does someone know how to create a tag on SO for this library? E.g. asp.net-api-versioning.

AddEndpointsApiExplorer is for Minimal APIs. If you aren't using Swashbuckle with Minimal APIs, then it is not necessary. The setup you want to follow is the more traditional OpenAPI Example.
AddApiExplorer is added via AddControllers, both of which require, at least, MVC Core. AddEndpointsApiExplorer creates API Explorer descriptions for basic Endpoint defines that come from Minimal APIs.

The tag  aspnet-api-versioning already exists. I come through there fairly regularly. ;)

Lol, didn't see it... the autocomplete failed me!

Wow there are full examples here - never saw them before! Should be linked in docs, so very helpful. Thanks for that link!

Those examples are what I need, thanks... closing! :+1:

Great! I'm glad that worked out. The wiki and other places should have links, but they problem go back to ms (formerly master). The project and repo are undergoing a big change. Since you're using the new packages, I presume you've read about it. I'm still held up on code signing. As soon as that is done, then I'll make the main branch the default branch. That will help people find the correct examples. It's on my backlog to create a better documentation site than just the wiki. That should help too.
You're assumptions are correct. Minimal APIs are a newer way to create very simple and quick APIs, but that in no way invalidates the existing model with controllers. Technically, you can mix the two approaches and it should all still work. I haven't tried it myself nor can I think of why someone would want to do that - but they can.

PS we found a way forward on that SO question. Seems like the official MS docs some attention. Thanks for putting it in perspective.

Yes, I've read about the imminent changes for this library, I wish you the best of luck! :smiley:


Would it make sense to use Diesel for building the SQL queries?

http://diesel.rs/

Probably, yes.
I'm going to stick with my current approach while I build out the initial POC because it is a style I am familiar with and can move fast (in the limited amount of weekend/evening time I can commit to this project at the moment).
If I do move to diesel to represent relational plans then I do have some requirements ... I don't know if diesel supports them or not currently ...

UDT support
Partition-aware relations
Ability to serialize relational plans to JSON
Ability to represent lambda/closure style operations (eventually)

I'll definitely revisit this when I'm a little further along.

I think Diesel is a query builder that generates SQL, whereas DataFusion needs to parse SQL so I actually don't think this is a good fit as this time.


(#191) - avoid negative values in _changes limit param

I restarted the build earlier, still failing. The problem seems legit:
https://travis-ci.org/pouchdb/pouchdb-server/jobs/203278304

That's really weird. In that case, we'll have to debug and investigate.

I don't believe it's related to this PR. Also failing in master: https://travis-ci.org/pouchdb/pouchdb-server/builds/203312544

In that case I suggest we merge this in and create a separate issue for the failing build?

Good idea; we might as well just merge now and fix the build later. :+1:


RegexQuery performance

Hi,
I'm trying to implement prefix matching. My idea was to use a RegexQuery for
this: given a user input of "foo", build an automaton that matches "foo.+" and
use that to search the fst.
(Aside: is there a way already to do this in tantivy and I've missed it?)
While implementing this, I stumbled upon a limitation of the current
RegexQuery: the RegexQuery struct owns a String with the given regex pattern,
and builds an automaton whenever it's needed in the specialized_weight method.
This is a problem as building the automaton is an expensive operation, and I have
to search for the same prefix over N fields.
What I did to address on a branch of mine:

change RegexQuery to own a fully built Regex instead of a string pattern

change Regex and the inner DFA to derive Clone, as the specialized_weight
method needs to return an owned Regex. This is assuming that a) making Regex
and DFA derive Clone is OK, and b) that cloning a Regex is cheap compared to
recompiling from a pattern

What do you think of this? Are my assumptions correct?
If there isn't a simpler way to do prefix matching that I have completely
missed, and if you agree on the approach, I'd be happy to make PRs to
tantivy-fst and to tantivy for this.

Everything you said is correct. No other way to compute a prefix query today.
and it would be better to compile the Regex once at the Query level.
Rather than Clone you could wrap the Dfa into a Arc and clone that. You can then have
struct ArcDfa(Arc<Dfa>) implement the Automaton trait.
The part where I am not sure here, is whether the extra indirection will confuse the compiler into writing inefficient code.

Rather than Clone you could wrap the Dfa into a Arc and clone that

Good point yes, I see the value in not changing the fst crate too much.

The part where I am not sure here, is whether the extra indirection will confuse the compiler into writing inefficient code.

With the disclaimer that I am by no means an expert in this, when I look at the generated ASM for https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=f19e6c2522b7465a3a4a7429bcd4c3ec
it looks like the extra levels of indirection are indeed optimized out (when compiling in Release mode). An interesting bit is that the optimization seems to happen already in the rust compiler, while I would have thought that this would be left to LLVM (i.e.: the LLVM IR is already showing no trace of the useless bits of indirection in Release mode).
So it seems like we should be able to add the wrapper (with #[inline] as this has to cross the crate boundary) with a reasonable expectation of the resulting code being efficient.
Anyway, it for sure beats having to rebuild the automaton every time :-)
I just submitted #640, is that what you had in mind?

With the disclaimer that I am by no means an expert in this, when I look at the generated ASM for

That's a false equivalency. The important difference with the Arc is that you need to follow a pointer.
I modified my comment afterwards.
Either can you try to have an Arc on the Query (no need to wrap it), and pass the reference of the automaton in the AutomatonWeight ?
It  seems we already have impl<'a, T: Automaton> Automaton for &'a T.

Thanks for the suggestion! I just pushed a few commits to the PR, could you have a look? I managed to reference the Regex directly in the AutomatonWeight, but to do so I had to relax a few lifetime bounds that required 'static.
In particular, I changed the weight method of the Query trait to:
 fn weight<'a, 'b>(&'a self, searcher: &'b Searcher, scoring_enabled: bool) -> Result<Box<dyn Weight + 'a>>;

as (TIL 😃) by default Box<dyn T> has a 'static lifetime bound. Now I tied it to the lifetime of the Query instead.
My reasoning was (correct me if I'm wrong), this should not be a problem for other users of the library, as any implementation of Query was required to produce a 'static Weight, while now the bound is just 'a.
(the code could probably be cleaned up a bit to avoid repeating Box<dyn Weight + 'a> all over the place, I'll look into that if you think this approach can work)

Oops. I think we want to keep the static lifetime in weight. Can you just
pass a clone of the Arc and deref it in the call to .scorer.
Sorry my indications are super vague and inaccurate.
If you don't have time, I'll fix your PR
tomorrow
Le mer. 21 août 2019 à 21:28, Alberto Piai<EMAIL_ADDRESS>a
écrit :

Thanks for the suggestion! I just pushed a few commits to the PR, could
you have a look? I managed to reference the Regex directly in the
AutomatonWeight, but to do so I had to relax a few lifetime bounds that
required 'static.
In particular, I changed the weight method of the Query trait to:
fn weight<'a, 'b>(&'a self, searcher: &'b Searcher, scoring_enabled: bool) -> Result<Box<dyn Weight + 'a>>;
as (TIL 😃) by default Box has a 'static lifetime bound. Now I
tied it to the lifetime of the Query instead.
My reasoning was (correct me if I'm wrong), this should not be a problem
for other users of the library, as any implementation of Query was
required to produce a 'static Weight, while now the bound is just 'a.
(the code could probably be cleaned up a bit to avoid repeating Box<dyn
Weight + 'a> all over the place, I'll look into that if you think this
approach can work)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tantivy-search/tantivy/issues/639?email_source=notifications&email_token=AAHZMQWAGVZL6DB7LABL4HDQFUYE7A5CNFSM4INQSHNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4ZPRUQ#issuecomment-523434194,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHZMQS7GJNGOOQHLGLAV7DQFUYE7ANCNFSM4INQSHNA
.


Run Django tests in GitHub Actions

@zkan I would like to give it a try

@selamanse Sure, thanks!

@zkan sorry took me a while, but I have now a proposal for you. #7


[wip] Fixing memory leaks

RP to fix https://github.com/apple/swift-distributed-actors/issues/1168

The failure was https://github.com/apple/swift-distributed-actors/issues/1089 again.
We can merge with it failing as we need to investigate that one separately

We can disable the test until then as well

Still missing the latest changes probably and requiring 6.0?


without-a-bundler-no-modules example does not build

Describe the Bug
The without-a-bundler-no-modules example does not build.
Steps to Reproduce

git clone https://github.com/rustwasm/wasm-bindgen
cd wasm-bindgen/examples/without-a-bundler-no-modules/
wasm-pack build --target no-modules
See the error below.

Expected Behavior
Build finishes without errors.
Actual Behavior
Build fails leaving an error (using rustc 1.45.0-nightly (a74d1862d 2020-05-14)):
thread 'main' panicked at 'internal error: entered unreachable code', crates/cli-support/src/decode.rs:157:1
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Error: Running the wasm-bindgen CLI
Caused by: failed to execute `wasm-bindgen`: exited with exit code: 101

Additional Context
It seems like a regression after merging https://github.com/rustwasm/wasm-bindgen/pull/2105
The same build works before the merge.

hey there. We ran into the same thing yesterday here. Turns out you have to copy the example's folder (in your case without-a-bundler-no-modules) somewhere out of the wasm-bindgen repo

Indeed yes I believe this is the same issue, closing in favor of that.

Confirmed! Thanks for pointing it out @wattux


Bug on trying to run the project

Hi! I ollowed the tutorial on the read me but as i got to the get schema phase it required the data/query_outputs_decomposed_tests to be ready, i think there is a error on the order of the bash calls

Please follow the instructions for installing CodeQL (https://github.com/Intelligent-CAT-Lab/AlphaTrans?tab=readme-ov-file#codeql-based-static-analysis) and running the queries to create that directory properly before creating schemas.

Hi, I believe I followed the instructions correctly, but I'm still a bit confused.
In the simplify_script.sh query linked below, the output path is not explicitly defined. This alone caused an error, but I’m unsure if I should set it to data/query_outputs or data/query_outputs_decomposed_tests.
Additionally, I tried running the script this week, but I found the Bash calls somewhat confusing. I’m not sure if this is due to gaps in the README or if I’m misunderstanding the code itself. For instance, as I understand it, program_transformation only instantiates method decomposition, and constructor decomposition would be overlooked. Is this correct?
Could you clarify this behavior and provide guidance on the proper setup for the output path?
link: https://github.com/Intelligent-CAT-Lab/AlphaTrans/blob/main/queries/simplify_script.sh
Thanks in advance!

When io try to run bash run.sh i ran into the following error:
[Errno 2] No such file or directory: 'vscode-codeql-starter/codeql-custom-queries-java'
/home/gabriel.pinheiro/trabalhos/traducao_de_codigo/12-Aplicar-AlphaTrans/AlphaTrans/vscode-codeql-starter/codeql-custom-queries-java
run.sh: line 13: $'\nbash simplify_script.sh commons-cli commons-cli query_outputs\nbash simplify_script.sh commons-codec commons-codec query_outputs\nbash simplify_script.sh commons-csv commons-csv query_outputs\nbash simplify_script.sh commons-validator commons-validator query_outputs\nbash simplify_script.sh commons-fileupload commons-fileupload query_outputs\nbash simplify_script.sh commons-pool commons-pool query_outputs\nbash simplify_script.sh commons-graph commons-graph query_outputs\nbash simplify_script.sh commons-exec commons-exec query_outputs\nbash simplify_script.sh jansi jansi query_outputs\nbash simplify_script.sh JavaFastPFOR JavaFastPFOR query_outputs\nbash simplify_script.sh JavaFastPFOR JavaFastPFOR query_outputs\nbash simplify_script.sh ChallengingProject ChallengingProject query_outputs': command not found
usage: execute_queries.py [-h] --project_name PROJECT_NAME --query_name
QUERY_NAME --database_name DATABASE_NAME
--output_path OUTPUT_PATH
execute_queries.py: error: the following arguments are required: --output_path
usage: execute_queries.py [-h] --project_name PROJECT_NAME --query_name
QUERY_NAME --database_name DATABASE_NAME
--output_path OUTPUT_PATH
execute_queries.py: error: the following arguments are required: --output_path
usage: execute_queries.py [-h] --project_name PROJECT_NAME --query_name
QUERY_NAME --database_name DATABASE_NAME
--output_path OUTPUT_PATH
execute_queries.py: error: the following arguments are required: --output_path
usage: execute_queries.py [-h] --project_name PROJECT_NAME --query_name
QUERY_NAME --database_name DATABASE_NAME
--output_path OUTPUT_PATH
execute_queries.py: error: the following arguments are required: --output_path
usage: execute_queries.py [-h] --project_name PROJECT_NAME --query_name
QUERY_NAME --database_name DATABASE_NAME
--output_path OUTPUT_PATH
I dont know if it was implicit to use a certainm output path that i messed out on

@FreitasVarejo please clone https://github.com/github/vscode-codeql-starter into the root of the repository, as described on our README.


update functionality for the generator

read .yo-rc.json contents, so it works similar to reapeating yo m --skip-prompts in the same directory. Be careful since .yo-rc.json contents get overwritten without any warning. This might lead to complications when version controlling cordova platforms and plugins via .yo-rc.json

usual procedure:

always document changes to generator files (readme is a good place)

gulp tasks, gitignore, ...

create new folder
setup new project (same name, bundle ids, ..)
cp .git/ folder to new project
copy custom files and folders if you have any (artifacts, doc, ...), that definitely won't have conflicts
remove modules in app/ and test/karma and test/protractor
copy your own modules into the respective folders (spare .eslintrc files and such)
git add, check for unwanted overwrites before
replace content of root files (config.xml, readme, ...)
git add, check for unwanted overwrites before
copy files from app/, res/ (omit bower_components)

possible issues:

overwrite forgotten updates to generator files (bower.json, package.json, .eslintrc, ...)
app code might break due to breaking changes of bower components
incompatible config.xml settings, platform/plugin versions

test:

gulp watch
gulp karma
gulp protractor
gulp build
CI build

gulp update - creates new project (with current and new generator version) based on .yo-rc.json, diff only generator project

first draft: unfortunately some weird issues with using npm programatically. Upon calling install of the second generator version npm throws this error, no matter what I do: timeout, loading npm again, ...
npm ERR! install trying to install 1.7.0 to /Users/jonathan/Projects/generator-m-ionic-demo/node_modules/generator-m-ionic
npm ERR! install but already installed versions [ '1.6.0' ]

npm i --save-dev yeoman-test in project
gulp/update.js
'use strict';
// gulp
var gulp = require('gulp');
var options = gulp.options;
// node core
var npm = require('npm');
var fs = require('fs');
var path = require('path');
// other
var chalk = require('chalk');
var answers = require('../.yo-rc.json')['generator-m-ionic'].answers;

var help = {
  loadNpm: function (cb) {
    npm.load({}, function (err) {
      if (err) {
        console.log(chalk.red('error loading npm\n'), err);
      }
      else {
        console.log(chalk.green('npm loaded successfully\n'));
      }
      cb();
    });
  },

  npmGen: function (command, packageName, cb) {
    npm.commands[command]([packageName], function (er) {
      if (er) {
        console.log(chalk.red('error running: ') + command + ' ' + packageName, er);
      }
      else {
        console.log(chalk.green('successfullly ran: ') + command + ' ' + packageName);
      }
      cb();
    });
  },

  runGeneratorVersion: function (version, cb) {
    this.loadNpm(function () {
      this.npmGen('install', 'generator-m-ionic@' + version, function () {
        process.chdir('../');
        fs.mkdirSync(help.projectDirVersion(version));
        process.chdir(help.projectDirVersion(version));

        var yeomanTest = require('yeoman-test');
        var generatorBase = '../' + help.projectDir + '/node_modules/generator-m-ionic/generators/';
        var ctx = yeomanTest.run(path.resolve(generatorBase + '/app'))
          .withGenerators(help.getDirectoryDirs(generatorBase))
          .withOptions({ // execute with options
            'skip-install': true, // don't need to install deps
            'skip-sdk': true // for some reason won't install cordova properly, so just leave it
          })
          .withPrompts(answers)  // answer prompts
          .on('end', function () {
            this.npmGen('uninstall', 'generator-m-ionic', cb);
          }.bind(this));
        ctx.settings.tmpdir = false; // don't run in tempdir
      }.bind(this));
    }.bind(this));
  },

  projectDir: (function () {
    var projectDir = process.cwd().split('/');
    projectDir = projectDir[projectDir.length - 1];

    return projectDir;
  })(),

  projectDirVersion: function (version) {
    return this.projectDir + '@' + version;
  },

  getDirectoryDirs: function (dirPath) {
    return fs.readdirSync(dirPath)
    .filter(function (file) {
      return fs.statSync(path.join(dirPath, file)).isDirectory();
    })
    .map(function (dir) {
      return path.resolve(dirPath, dir);
    });
  }
};

// UPDATE
gulp.task('update', [], function (done) {
  if (!options.from || !options.to) {
    console.log(chalk.red('error: ') + 'supply proper generator versions with --from and --to');
    return;
  }
  help.runGeneratorVersion(options.from, function () {
    help.runGeneratorVersion(options.to, function () {
      done();
    });
  });
});

Some related research:
http://stackoverflow.com/questions/20686244/install-programmatically-a-npm-package-providing-its-version
http://stackoverflow.com/questions/15957529/can-i-install-a-npm-package-from-javascript-running-in-node-js
http://stackoverflow.com/questions/1968512/getting-the-difference-between-two-repositories

use at the end:
git diff master remotes/update/master > update-diff
git diff master remotes/project/master > project-diff
diff update-diff project-diff

version 3: applies update on-top of current repo
'use strict';
// gulp
var gulp = require('gulp');
var options = gulp.options;
// node core
var fs = require('fs');
var path = require('path');
var exec = require('child_process').exec;
// other
var yeomanTest = require('yeoman-test');
var chalk = require('chalk');
var answers = require('../.yo-rc.json')['generator-m-ionic'].answers;

var help = {
  projectDir: (function () {
    var projectDir = process.cwd().split('/');
    projectDir = projectDir[projectDir.length - 1];

    return projectDir;
  })(),

  projectDirVersion: function (version) {
    return this.projectDir + '@' + version;
  },

  getDirectoryDirs: function (dirPath) {
    return fs.readdirSync(dirPath)
    .filter(function (file) {
      return fs.statSync(path.join(dirPath, file)).isDirectory();
    })
    .map(function (dir) {
      return path.resolve(dirPath, dir);
    });
  },

  updateWithYo: function (version, cb) {
    this.install(version, function () {
      // execute generator
      var generatorBase = '../' + this.projectDir + '/node_modules/generator-m-ionic';
      var generatorGenerators = generatorBase + '/generators';
      console.log(chalk.green('running: ') + require('../' + generatorBase + '/package.json').version);

      var ctx = yeomanTest.run(path.resolve(generatorGenerators + '/app'));
      ctx.settings.tmpdir = false; // don't run in tempdir
      ctx
      .withGenerators(this.getDirectoryDirs(generatorGenerators))
      .withOptions({ // execute with options
        'skip-install': true, // don't need to install deps
        'skip-sdk': true, // for some reason won't install cordova properly, so just leave it
        'force': true
      })
      .withPrompts(answers) // answer prompts
      .on('end', function () {
        // uninstall
        this.uninstall(version, cb);
      }.bind(this));
    }.bind(this));
  },

  uninstall: function (version, cb) {
    process.chdir('../' + this.projectDir);
    console.log(chalk.green('uninstalling: ') + version + ' ' + process.cwd());
    exec('npm uninstall generator-m-ionic', function (error) {
      if (error) {
        console.log(chalk.red('error: ') + 'uninstalling ' + version + '\n', error);
      }
      cb();
    });
  },

  install: function (version, cb) {
    console.log(chalk.green('installing: ') + version + ' to ' + process.cwd());
    exec('npm i generator-m-ionic@' + version, function (error) {
      if (error) {
        console.log(chalk.red('error: ') + 'installing ' + version + '\n', error);
      }
      cb();
    });
  }
};

gulp.task('update', [], function (done) {
  if (!options.to) {
    console.log(chalk.red('error: ') + 'supply proper generator version with --to');
    return;
  }

  help.updateWithYo(options.to, function () {
    done();
  });

});

@MathiasTim, @lordgreg, @DrMabuse23 and team. I invested some (quite a lot) time into this and found out (as expected) that this is a very delicate and complex task.
However I have an initial proposal that I marked as experimental. Please let me know what you think and if this is similar to what you had in mind, originally.
Before this goes live tomorrow you can find a guide here:
https://github.com/mwaylabs/generator-m-ionic/blob/dev/docs/guides/generator_update.md

@gruppjo I will probably have to wait for upcoming update (1.8.0). What I've did is:

cloned a project with generator-m 1.3.3.
installed yeoman-test with npm i yeoman-test --save-dev
created new branch git checkout -b update
copied update.js from gulp directory of dev branch of generator-m and pasted it into my current project folder.
ran eperimental-update task with version 1.7.0 since 1.8.0 isn't available yet.

Here's what I get:
Gregor@HomePC /c/Development/genm-upgrade (update)
$ gulp experimental-update --to=1.7.0
[20:00:17] Using gulpfile C:\Development\genm-upgrade\gulpfile.js
[20:00:17] Starting 'experimental-update'...
installing: 1.7.0 to C:\Development\genm-upgrade
module.js:341
    throw err;
    ^

Error: Cannot find module '../../C:\Development\genm-upgrade/node_modules/generator-m-ionic/package.json'
    at Function.Module._resolveFilename (module.js:339:15)
    at Function.Module._load (module.js:290:25)
    at Module.require (module.js:367:17)
    at require (internal/module.js:16:19)
    at Object.<anonymous> (C:\Development\genm-upgrade\gulp\update.js:41:46)
    at C:\Development\genm-upgrade\gulp\update.js:77:7
    at ChildProcess.exithandler (child_process.js:193:7)
    at emitTwo (events.js:100:13)
    at ChildProcess.emit (events.js:185:7)
    at maybeClose (internal/child_process.js:850:16)
    at Socket.<anonymous> (internal/child_process.js:323:11)
    at emitOne (events.js:90:13)
    at Socket.emit (events.js:182:7)
    at Pipe._onclose (net.js:477:12)

That was, however, after waiting approximately 30 seconds in the terminal without knowing if there's any progress or not.
$ npm --version
3.7.3

Gregor@HomePC /c/Development/genm-upgrade (update)
$ node --version
v5.9.1

So what about migrations? Maybe with a diff From your git  repo from Tag you can find out which files are habe to be changed. And maybe we can add a migration json Form Every Version to Version its simple but strong
Sent from my Cyanogen phone
Am 28.04.2016 8:06 nachm. schrieb Gregor<EMAIL_ADDRESS>@gruppjo I will probably have to wait for upcoming update (1.8.0). What I've did is:
cloned a project with generator-m 1.3.3.installed yeoman-test with npm i yeoman-test --save-dev created new branch git checkout -b update copied update.js from gulp directory of dev branch of generator-m and pasted it into my current project folder. ran eperimental-update task with version 1.7.0 since 1.8.0 isn't available yet.
Here's what I get:
Gregor@HomePC /c/Development/genm-upgrade (update) $ gulp experimental-update --to=1.7.0 [20:00:17] Using gulpfile C:\Development\genm-upgrade\gulpfile.js [20:00:17] Starting 'experimental-update'... installing: 1.7.0 to C:\Development\genm-upgrade module.js:341 throw err; ^ Error: Cannot find module '../../C:\Development\genm-upgrade/node_modules/generator-m-ionic/package.json' at Function.Module._resolveFilename (module.js:339:15) at Function.Module._load (module.js:290:25) at Module.require (module.js:367:17) at require (internal/module.js:16:19) at Object. (C:\Development\genm-upgrade\gulp\update.js:41:46) at C:\Development\genm-upgrade\gulp\update.js:77:7 at ChildProcess.exithandler (child_process.js:193:7) at emitTwo (events.js:100:13) at ChildProcess.emit (events.js:185:7) at maybeClose (internal/child_process.js:850:16) at Socket. (internal/child_process.js:323:11) at emitOne (events.js:90:13) at Socket.emit (events.js:182:7) at Pipe._onclose (net.js:477:12)
That was, however, after waiting approximately 30 seconds in the terminal without knowing if there's any progress or not.
$ npm --version 3.7.3 Gregor@HomePC /c/Development/genm-upgrade (update) $ node --version v5.9.1
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

@lordgreg, please use at least npm 3.8.3 or higher. This version fixed some of the performance issues of npm.

@lordgreg, it seems like the paths are not windows-safe. Wops. I'll try to fix that. The wait is normal, since the installation takes a while and I don't output the buffer, so you don't see the usual installation progress. Which I'd like to have in a future version but I'm not familiar with buffers and streams in JS so much and I really want to release this version soon.
@DrMabuse23, I like the idea of providing a diff. We could do that in every release for the previous version and mention in the doc that you can create the diffs between the versions you need yourself. I'm not sure what you mean by migration json?

@lordgreg, please try the new version of update.js. It should now be windows safe!

@gruppjo will do roger wilco (aka today evening). Will report my result afterwards. 👍

I will leave this issue open so we can improve on this in future versions together.

So, here's how my update just went. No errors:
Gregor@HomePC /c/Development/genm-upgrade (update)
$ gulp experimental-update --to=1.8.0
[19:13:14] Using gulpfile C:\Development\genm-upgrade\gulpfile.js
[19:13:14] Starting 'experimental-update'...
installing: 1.8.0 to C:\Development\genm-upgrade
running: 1.8.0

Gregor@HomePC /c/Development/genm-upgrade (update)
$ git status
On branch update
Untracked files:
  (use "git add <file>..." to include in what will be committed)

        gulp/update.js
        jsconfig.json

nothing added to commit but untracked files present (use "git add" to track)

Gregor@HomePC /c/Development/genm-upgrade (update)
$ git log
commit e7d3ef57c07514bc6fa7be375962c97b82fa7661
Author: Gregor<EMAIL_ADDRESS>Date:   Thu Apr 28 19:54:58 2016 +0200

    initial commit

Checking README.md, there's still version 1.3.3 listed. diff shows no other things were updated, which leads me thinking... did it worked? :8ball:

@lordgreg, it most likely did not. The output is not complete. This is how it should look:
$ gulp experimental-update --to=1.8.0
[15:45:30] Using gulpfile ~/Projects/generator-m-ionic-demo/gulpfile.js
[15:45:30] Starting 'experimental-update'...
installing: 1.8.0 to /Users/jonathan/Projects/generator-m-ionic-demo
running: 1.8.0
uninstalling: 1.8.0 /Users/jonathan/Projects/generator-m-ionic-demo
[15:45:59] Finished 'experimental-update' after 29 s

You're missing the second half.
I'm a little confused. It seems like the task is failing but silently. It's very inconvenient that I cannot test this on windows myself. You could insert some console.logs to see where it fails. It appears to fail somewhere along these lines. https://github.com/mwaylabs/generator-m-ionic/blob/master/generators/app/templates/gulp/update.js#L32-L50

Addy Osmani over at Yeoman, mentions the "emit" feature that has been coming out of the CLI ecosystems. https://github.com/yeoman/yeoman/issues/1265#issuecomment-285806051

Removed
As part of #481, we're removing this feature from the master.
Alternatives
1 Manually upgrade
You can manually upgrade the generator by creating a new project and moving your old files to the new project. A good step by step guide I have written down here for your consideration: https://github.com/mwaylabs/generator-m-ionic/issues/158#issuecomment-168699072
2 Run yo m-ionic
When running yo m-ionic again in your project's directory and it will run the generator again, using your old answers (except for a few ones that you will prompted to answer again). Then use yeoman's diff feature to look for changes.
3 CLI and "emit"
Maybe the future of generators is a CLI/Generator hybrid that can emit the config upon request as mentioned above.

Addy Osmani over at Yeoman, mentions the "emit" feature that has been coming out of the CLI ecosystems. https://github.com/yeoman/yeoman/issues/1265#issuecomment-285806051

The current implementation for reference
'use strict';
// gulp
var gulp = require('gulp');
var options = gulp.options;
// node core
var fs = require('fs');
var path = require('path');
var exec = require('child_process').exec;
// other
var yeomanTest = require('yeoman-test');
var chalk = require('chalk');
var answers = require('../.yo-rc.json')['generator-m-ionic'].answers;

var help = {
  getDirectoryDirs: function (dirPath) {
    return fs.readdirSync(dirPath)
    .filter(function (file) {
      return fs.statSync(path.join(dirPath, file)).isDirectory();
    })
    .map(function (dir) {
      return path.resolve(dirPath, dir);
    });
  },

  updateWithYo: function (version, cb) {
    this.install(version, function () {
      // execute generator
      var generatorBase = path.resolve('node_modules/generator-m-ionic');
      var generatorGenerators = path.join(generatorBase, 'generators');
      console.log(chalk.green('running: ') + require(path.join(generatorBase, '/package.json')).version);

      var ctx = yeomanTest.run(path.join(generatorGenerators, '/app'));
      ctx.settings.tmpdir = false; // don't run in tempdir
      ctx
      .withGenerators(this.getDirectoryDirs(generatorGenerators))
      .withOptions({ // execute with options
        'skip-install': true, // don't need to install deps
        'skip-sdk': true, // for some reason won't install cordova properly, so just leave it
        'force': true
      })
      .withPrompts(answers) // answer prompts
      .on('end', function () {
        // uninstall
        this.uninstall(version, cb);
      }.bind(this));
    }.bind(this));
  },

  uninstall: function (version, cb) {
    console.log(chalk.green('uninstalling: ') + version + ' ' + process.cwd());
    exec('npm uninstall generator-m-ionic', function (error) {
      if (error) {
        console.log(chalk.red('error: ') + 'uninstalling ' + version + '\n', error);
      }
      cb();
    });
  },

  install: function (version, cb) {
    console.log(chalk.green('installing: ') + version + ' to ' + process.cwd());
    exec('npm i generator-m-ionic@' + version, function (error) {
      if (error) {
        console.log(chalk.red('error: ') + 'installing ' + version + '\n', error);
      }
      cb();
    });
  }
};

gulp.task('experimental-update', [], function (done) {
  if (!options.to) {
    console.log(chalk.red('error: ') + 'supply proper generator version with --to');
    return;
  }

  help.updateWithYo(options.to, function () {
    done();
  });
});

The current docs:
Generator Update (experimental)

This feature is highly experimental!
We're exploring ways that can make upgrading your project to a new version of the generator easier. We're very happy to get feedback from you, but please be careful with what you are doing. The full discussion with many different ideas on how to achieve an update can be found in this issue.

Considerations
When you are trying to upgrade your project there's many things to consider that stem from a complex technology stack, how you work on your project, whether and how you use git and many other factors. So depending on these topics there might be better ways of how to perform an update. Please feel free to explore them and share them with us.
Additional considerations that play a role on how easy an update is:

What version are you upgrading from (that's the version of the generator you used when you called yo m-ionic for your project the first time)?
Which version are you upgrading to (most likely the latest one)
What changes were introduced to the generator in between those two versions?

only gulp file update, new features and subgenerators?
or possibly larger, less isolated changes that affect many aspects of the generator?

Did you change any of the generator's core files after project setup or just add new stuff?

gulp tasks, config files like .eslintrc and the index.html would be core files for instance. Basically everything that is generated on the first run of yo m-ionic

Additional/alternative measures

Consulting the Changelog and the changes between the two version in the generator repository and especially the demo repository can be enormously helpful.
You might want to simply create a new project with yo m-ionic and piece by piece copy your stuff

Versions <1.8.0
If you're upgrading from an version of the generator that is smaller than 1.8.0. You'll need to do two things:
# install yeoman-test as a devDependency (in your project folder)
npm i yeoman-test --save-dev

And save the following gulp file as gulp/update.js in your project.
Perform experimental update
First of all we recommend switching to a new git branch (just to be sure).
git checkout -b update

Before you perform the next step.
Read this carefully. This will do the following:

temporarily install the new generator version locally
take the contents of your .yo-rc.json (the choices you made when running yo m-ionic to set up your project)
pass these answers into the new generator version
run the generator on top of your current project
overwrite all files that the generator generated in the first place with the new ones
thus very very likely break your code

=> It's up to you to make sense of these changes and differentiate between changes we made and possible changes you made, and reapply them when necessary.
This will not:

delete any files that became obsolete with the new version (doesn't happen often)
take into consideration any of the changes you made to the core files (how would we know?). They will be overwritten.
manipulate any other files, and magically make your code work
re-run any sub-generators you used

There's still some things that could go wrong:

your .yo-rc.json is missing some answers (because they weren't available when you set up the project). To remedy this, just add the desired values for those answers.

If you are lucky and additionally:

the version you're updating form is not too old
you didn't drastically change any of the generator's core files
you're smart, used git properly, know your project and all that
then ...

... updating might be very simple.
No promises!
Then, after careful consideration of all of this, run:
# update to version 1.8.0
gulp experimental-update --to=1.8.0
# wait
git diff

Good luck and please give some feedback in this issue!


Update jquery-ui.min.js to fix minified syntax bug

In two places in this file there is the syntax + ++ which throws an interpreter error and causes all javascripts in the same file (such as when combining this file with other files) to fail to load. This problem can easily be solved by wrapping the ++n and ++t with parentheses. which is what I've done in this pull request.
This bug is discussed at:
https://bugs.jqueryui.com/ticket/5576
and
http://www.crockford.com/javascript/jsmin.html
I'm not sure why nobody else has ever encountered this issue. Till now I've been manually editing this one file after any framework updates.

I'm not sure why nobody else has ever encountered this issue.

Indeed - I can only imagine the issue isn't actually that prevalent and because of that I'm not sure this will get merged. 3.x is security fixes only now so this is unlikely to make it in.
Are you using jquery-ui for something custom or is it used in the CMS out of the box? What browser do you experience this error in? Does it happen in a plain install with no other changes?
I'd expect the workaround we'd recommend is include jquery-ui in your own code and then use requirements to override the file that's imported

Yep all of what you said makes perfect sense.
Using jQuery-ui on the front end so don't really need to be using the framework version and will do what you recommend.
Been occurring for a long time so been through lots of browsers, but error is currently experienced in Firefox 61.0.1 (64-bit) and Chrome Version 67.0.3396.99 (Official Build) (64-bit).

@AlphaCactus thanks for your understanding


Update ExecutorClassLoader.scala

Error when starting spark shell in windows:
Caused by: java.net.URISyntaxException: Illegal character in path at index 32: spark://LAPTOP-XXX:XXX/X:\classes
at java.net.URI$Parser.fail(URI.java:2848)
at java.net.URI$Parser.checkChars(URI.java:3021)
at java.net.URI$Parser.parseHierarchical(URI.java:3105)
at java.net.URI$Parser.parse(URI.java:3053)
at java.net.URI.(URI.java:588)
at org.apache.spark.repl.ExecutorClassLoader.(ExecutorClassLoader.scala:57)

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Can one of the admins verify this patch?

@guolianwei Thank you for your contribution. Could you enable GitHub Actions to run test.

On top of @wangyum 's comment, please file an Apache Spark JIRA issue . You can see more contributor's guide here.

https://spark.apache.org/contributing.html


Windows 11 context menu

Windows Terminal version
No response
Windows build number
No response
Other Software
No response
Steps to reproduce
Duplicate
Expected Behavior
No response
Actual Behavior
wanting to stop of it show up under show more options and only show in windows 11 default context menu only

I don't think we can do anything about that. We just tell Explorer that we're a context menu entry and it does what it does (which is to add us to both the new and old context menu).

/dup #6113

is there a way to disable the old one need an example


upgraded ASM to 7.1, added Java 13 to CompilerConfiguration

+1

Any objection to including this on the 2.5 line?

Works for me. Let's wait a day or two and see if anyone else has an opinion.

Merged into the 2_5_X branch.
See also: https://issues.apache.org/jira/browse/GROOVY-9020


Formatting/prettier

Description
Since there are a bunch of people working on this project we want to make sure that the code is formatted correctly.
I have added in pretter and connected it with ESlint to allow auto formatting so editors that have autoformatting on should auto format to the settings in the .prettierrc file.
I have also added it to husky so when someone does a commit it should auto format.
If for some reason it skips husky, the eslint action should fail so the editor would have to run
npm run format

which would then format all the files to fix it.
NOTE
This PR will conflict with any active PR's so this either needs to be added in next or when the the repo is clean.
Related Issue
https://github.com/SauveJeanLuc/coffee-tasks/issues/86
Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works

Screenshots (if appropriate):

I think you'll have to resolve conflicts.
and I'll merge this before any other active PR.

@SauveJeanLuc all up to date now.


When upgrading from Squirrel.Windows to Velopack, do not recreate shortcuts if the user had previously deleted them

When applying updates, Squirrel.Windows expressly did not attempt to recreate desktop or Start menu shortcuts if the user had previously deleted them. When Squirrel.Windows applies a Velopack package and begins the migration process, Velopack should match this behavior and not recreate the shortcuts as if it were a fresh installation.

I'm don't think I can fix this. Velopack doesn't know what shortcuts were configured to be installed in Squirrel. Imagine that your Squirrel install only added a StartMenuRoot - but now in Velopack you would like to have shortcuts in Desktop as well. We can't tell the difference between "squirrel installed desktop shortcut but user deleted" and "there was never a desktop shortcut".


Github check never completes

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

App Id
d1r644r8c33agt
AWS Region
us-east-1
Amplify Hosting feature
Web previews
Describe the bug
I have the same issue as https://github.com/aws-amplify/amplify-hosting/issues/2598.
Amplify Github check never reports complete despite the build building successfully. Instead, we see that the build is in "In Progress" after several hours despite the build finishing successfully in several minutes.
Expected behavior
Amplify Github check should always report the actual status of the build or have some mechanism to restart the check besides pushing another commit.
Reproduction steps

Create a PR against a branch you have previews enabled for

Build Settings
version: 1
frontend:
  phases:
    preBuild:
      commands:
        - rvm install 2.7.1
        - ruby -v
        - gem install bundler:2.3.11
        - bundle install
    # IMPORTANT - Please verify your build commands
    build:
      commands:
        - if [ "${AWS_BRANCH}" = "main" ]; then JEKYLL_ENV=production bundle exec jekyll build; fi
        - if [ "${AWS_BRANCH}" != "main" ]; then JEKYLL_ENV=development bundle exec jekyll build; fi
  artifacts:
    # IMPORTANT - Please verify your build output directory
    baseDirectory: _site
    files:
      - '**/*'
  cache:
    paths: []

Log output

"Build successfully completed"
"No tests have been configured for this app, check our walkthrough to get started!"
2023-06-09T02:06:33 [INFO]: Beginning deployment for application d1r644r8c33agt, branch:pr-242, buildId<PHONE_NUMBER>
2023-06-09T02:06:34 [INFO]: Got archive: 36631051 bytes
2023-06-09T02:06:36 [INFO]: Deployment complete

Additional information
No response

It seems to have been a temporary issue.  A subsequent commit to the PR triggered another build.   GitHub reports "All checks have passed".


zfsUnstable: 2.0.4 -> 2.1.0-rc3

Motivation for this change
New release candidate https://github.com/openzfs/zfs/releases/tag/zfs-2.1.0-rc3
Please read the commit message.
Replaces #118616
Things done

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

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

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

Result of nixpkgs-review pr 119848 1

4 packages marked as broken and skipped:

bareos
linuxPackages-libre.zfsUnstable
linuxPackages_latest-libre.zfsUnstable
samba4Full

2 packages failed to build:

libvmi
rubyPackages_3_0.ruby-libvirt

56 packages built:

ceph
ceph-client
collectd
collectd-data
docker-machine-kvm
docker-machine-kvm2
easysnap
gnome3.gnome-boxes
libceph
libguestfs
libguestfs-with-appliance
libvirt
libvirt-glib
linuxPackages.zfsUnstable
linuxPackages_4_14.zfsUnstable
linuxPackages_4_19.zfsUnstable
linuxPackages_4_4.zfsUnstable
linuxPackages_4_9.zfsUnstable
linuxPackages_5_11.zfsUnstable
linuxPackages_5_4.zfsUnstable
linuxPackages_hardened.zfsUnstable
linuxPackages_latest_hardened.zfsUnstable
linuxPackages_latest_xen_dom0.zfsUnstable
linuxPackages_latest_xen_dom0_hardened.zfsUnstable
linuxPackages_lqx.zfsUnstable
linuxPackages_testing_bcachefs.zfsUnstable
linuxPackages_xanmod.zfsUnstable
linuxPackages_xen_dom0.zfsUnstable
linuxPackages_xen_dom0_hardened.zfsUnstable
linuxPackages_zen.zfsUnstable
minikube
minishift
nixopsUnstable
ocamlPackages.ocaml_libvirt
partition-manager
perl530Packages.SysVirt
perl532Packages.SysVirt
python38Packages.guestfs
python38Packages.libvirt
python39Packages.guestfs
python39Packages.libvirt
qemu_full
rubyPackages.ruby-libvirt
rubyPackages_2_7.ruby-libvirt
sanoid
terraform-full
terraform-provider-libvirt (terraform-providers.libvirt)
vagrant
virtmanager (virt-manager)
virtmanager-qt (virt-manager-qt)
virt-top
virtviewer (virt-viewer)
virtlyst
zfs
zfsUnstable
zfstools

/marvin opt-in
/status needs_reviewer

@ofborg test zfs.unstable

@jonringer I think this is looking good now. If you agree, please merge this and close #118616.


word is incorrectly filtered out

The answer was QUALM, and my guesses were:
I R A T E
X X ^ X X

C L A S H
X ? ^ X X

...

When I entered that in, "QUALM" wasn't in the words list, but it should have been.

Oh, I see. QUALM is just super down in the list! It's word number 19002, with 34744 instances.
Looking at the larger list (the full count_1w.txt), it looks like qualms is much more common: 221,869 instances, compared to 34,744 for the singular.
I should amend my script to not just remove 4-letters that turn into 5-letters when you add an s, but also add to the list when there's a 5-letter that turns into a 6-letter when you add an s. And maybe even 7-letter with es.


hexf: add a shebang

When trying out the new hexf tool from 2.6.0, I noticed it was missing a shebang.

Thanks!
Did you get an error?

Thanks for merging so fast!
Yeah, if running it in a non-sh shell (like fish) it'll fail to execute with:
Failed to execute process '/usr/local/bin/hexf'. Reason:
exec: Exec format error
The file '/usr/local/bin/hexf' is marked as an executable but could not be run by the operating system.


refactor: add sampleRate and stayId to output data

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


Created issue templates and pull request template

Added to guide users to create new issues and pull requests, according to Keyfactor Community templates.

Merged, thanks @KarolinHem.


How do I enable fonts in emacs?

So I use emacs but I don't know how to enable the fonticons for emacs-in-terminal ? I added:
(require 'icons-in-terminal) (insert (icons-in-terminal 'oct_flame)) ; C-h f icons-in-terminal[RET] for more info
but it did nothing? Can someone show me how their init.el looks?

Try https://github.com/seagle0128/icons-in-terminal.el.


deprecate igorio as is is not maintained

It looks as though igor has been forked - https://github.com/NeuralEnsemble/python-neo/pull/1268 - so I think we can just replace "igor" with "igor2" as a dependency.

Good to know. Nice to see the project is still kept alive.
I will update the PR accordingly.


[Feature] Set different bar gaps on same category

What problem does this feature solve?
Need an option to set different gaps for each bar

as you can see each 2 bars are close each other with zero padding
What does the proposed API look like?
set property for single bar "isSingleBar" then the "barGap" will come relevant only for him

Sorry I cannot find the exact problem if there is no demo. My best guess to solve this is to try adjusting series.barGap.


Remove unnecessary dependency

asdf-unit-schemas was never necessary for asdf, it only existed so that asdf-format/asdf-standard#342 could be merged without issue.
Since that entire line of support has been abandoned at this point, this dependency should be dropped.

as user API is not changed, and users will not need to update their environments, is a change log entry necessary? I guess it doesn't hurt to still have one

as user API is not changed, and users will not need to update their environments, is a change log entry necessary? I guess it doesn't hurt to still have one

Its useful to have in the changelog simply because the dropping of the dependency does change the Conda configuration slightly.

I'm all for cleaning up this if indeed the splitting up of the standard has been abandoned. However until @perrygreenfield @nden @eslavich and other stake holders can weight in I don't think we should undo the work that has been done.

Looking at my notes for ASDF 3.0, the question of keeping/removing this dependency is moot. It is my understanding that as part of ASDF 3.0 (which is the main development path for ASDF now), we would stop packaging schemas which are no longer part of the ASDF-standard.
Originally, the asdf-transforms-schemas where added as a dependency to ASDF for backwards compatiblity reasons. That is since they were totally removed from ASDF-standard, users would have to add them separately as a dependency in order to continue using them. At around the same time, ASDF-standard was formally removed from the ASDF package itself and repackaged into its own independent python package. Thus ASDF then required both asdf-standard and asdf-transform-schemas as dependencies to facilitate backwards compatiblity for ASDF 2.x versions. Part of the plan for ASDF 3.0 was to totally remove asdf-transform-schemas (asdf-standard is still required as the ASDF package is fundamentally tied to it) since we were breaking backwards compatiblity in several other ways.
The original proposal for breaking ASDF-standard apart into more sub-packages, including asdf-unit-schemas, was inspired by the process followed by asdf-transform-schemas. As such, the new packages were supposed to be added to ASDF as dependencies to preserve backwards compatiblity in the same way. However, only asdf-unit-schemas was ever actually made a dependency of ASDF because the split of ASDF-standard was forgotten about and/or abandoned. Thus just as asdf-transform-schemas ought to be removed, asdf-unit-schemas should be removed for ASDF 3.0.
I have not included the removal of asdf-transform-schemas in this PR as this topic clearly needs more discussion.

However, only asdf-unit-schemas was ever actually made a dependency of ASDF because the split of ASDF-standard was forgotten about and/or abandoned.

By this I mean that ASDF-standard has made significant changes to its schemas which have not being properly propagated into the packages it was intended to be split into. As such, the work initially carried out to facilitate this, including asdf-format/asdf-standard#342, needs to almost entirely be re-done in order to ensure that no loss of changes will occur and that the git history can be properly followed, if the ASDF developers wish to continue with the proposed "split".


Fixed german locales

Fixed german translation for "ein Monat" (to "einen Monat") in all three german translations. Can you approve this, @lluchs?

This is tricky. You're probably thinking of something like "einen Monat später". However, if you're just formatting standalone durations, it's just "ein Monat". Basically, we currently have Nominativ and Dativ for durations, and you'd like to replace Nominativ with Akkusativ.
This is a general momentjs issue: only two cases are supported while some languages have more than that.
Looking at Wiktionary, your change for de-at isn't correct in any case (pun not intended) as it's "das Monat" there (though you'd better ask a native speaker for that...).

I'm afraid but there is no such thing as "ein Monat". As far as I know, when using nominative, "ein" can only be used when it's followed by a neuter noun (ein Pferd, ein Computer, ein Schloss), "einen" is used when a masculine noun follows (einen Tag, einen Brief, einen Weg, einen Monat).

No, the indefinite articles are ein/eine/ein, see e.g. Wikipedia.

Yeah, that's what I realized when I was taking today's morning shower, that you are speaking about the indefinite article and not about the number. It's indeed tricky to apply the api docs saying "passing true as the second parameter returns value without the suffix" to German. Thanks for this discussion.


fix smtpd TLS connection regex

Hey, this little change fixes parsing lines like this:
Anonymous TLS connection established from host.example.com[<IP_ADDRESS>]: TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256

Thanks for the PR!


Update Pages “home”

Automatically generated by Netlify CMS

👷 Deploy Preview for parsimony-dev processing.
🔨 Explore the source changes: 966c907f76a67ef166d23caf018d0488b779d6d5
🔍 Inspect the deploy log: https://app.netlify.com/sites/parsimony-dev/deploys/6189800200ee4e0007285255


Checkboxes and Radio-buttons are inspected as Buttons. Is there a way WinAppDriver can get the state of these elements?

Using inspect.exe on our AUT, shows checkboxes and radio-buttons as Buttons. Is there a way for WinAppDriver to get the on/off state of these elements?

you can use   bool checkboxStatus = session.FindElementByName(name).Selected;

you can use bool checkboxStatus = session.FindElementByName(name).Selected;

That property for the checkbox (recognized as a button in my case) is always returning True, even when the checkbox is not selected.
I am using python with Selenium by the way.

These same elements are identified correctly (as checkboxes/radio-buttons) in UFT (using just the built-in windows add-in and no ui-automation) but not inspect.exe. Can't understand why.

Went with pywinauto library in python to work with these elements (checkboxes, radio-buttons) that are identified as "Buttons". pywinauto had a "ButtonWrapper" that can be used to workaround this. So, for now moving forward with pywinauto instead of WinAppDriver.


Enforce Ternary expression conditions to be enclosed in parentheses

Preferred:
let rowHeight = contentHeight + ((hasHeader == true) ? 50 : 20)

Not Preferred:
let rowHeight = contentHeight + (hasHeader == true ? 50 : 20)

This is currently infeasible because whitespace determines how operators are parsed in Swift.


core: store NEP5 transfers and balances as big.Int

closes #1133

We'll need to profile it, but it can be done post-preview2, at the moment correctness is more important, so overall it's good.


issue in courses page that is text is overflowing through section

here is issu that text is over flowing throug the section and when we switch to dark mode some part of text disappear

this issue was resolved, so i am closing this issue.