Updating CMake Debugging Docs to have CMake Debugger

Adding CMake Debugger information and refreshing top-level screenshots

@TylerMSFT
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"
@MicrosoftDocs/public-repo-pr-review-team

@TylerMSFT It should be good to review now, thank you for your help!

#sign-off

Merging now.


Script requires pandoc at most version 2.19.2

I had installed the latest pandoc through Homebrew, which is at version 3.x at the moment. When running the export script, it complained and errored over pandoc. I downgraded to version 2.19.2 (the last 2.x version) to have the export script running without errors.
Downgrade:

Uninstall pandoc from brew (brew uninstall pandoc)
Manual install pandoc 2.19.2 downloaded from https://pandoc.org/releases.html#pandoc-2.19.2-2022-08-22

I think this is due to the upstream things library, so nothing to be done atm. But this could at least be mentioned in the project README?

Pandoc is not used in the upstream library but only in this script.


[Questions]: Playwright version and Chromium versions

Your question
Though we see a particular version of playwright-java is certified with a particular version of Chromium,
we observed that Playwright Chromium is upgrading to latest version automatically sometimes, without changing playwright-java version!

Could you explain when this happens?
In future if we just upgrade Chromium manually and have PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD=1 in the env, can we expect it to work?
Can we expect Playwright to work if we download latest Chromium (not playwright chromium) and place in the appropriate location with PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD=1 in the env, without changing playwright-java version?

Closing as part of the triage process since it seemed stale or did not get enough upvotes in the past. Please create a new issue with a detailed reproducible or feature request if you still encounter issues.


added LiveScript support for migrations correctly #903

Previously, I had directly modified files in lib/ because I didn't realize they were generated from files in src/.  I have since reverted those changes and changed the right files, and went through the build process to generate the files that needed to be generated.
Apologies to @rhys-vdw and thanks to @MadcapJake for noticing my mistake before it was too late.

Thankyou. :-)


Table prints go unlogged

The new studio output window features that print({ value = 1 }) is visualised, instead of being outputted as table: hex ID. However, run-in-roblox does not detect this at all, acting as if nothing was printed.
This also applies to print(Instance.new("TextLabel"))

Ah, that's rough. Does this happen with the in-game dev console as well?
If you look at [the code that run-in-roblox uses to catch logs]https://github.com/rojo-rbx/run-in-roblox/blob/40fb70c0f76d0e47aa497d32669c30732d40f551/src/plugin_main_template.lua), you can see that we use LogService. I don't know if there's another way to capture logs that would pick this up. It sounds like a Roblox bug! :(

The in-game dev console (F9) just prints the old table: hex ID. It's specific behaviour of print in studio.

Addressed by Roblox Release Notes for 531. Messages longer than 2048 characters will be truncated in the event handler.


remove the IFrames state on the player controller.

on the player, iframes can't be a finite state, because the player will still expect to have all their other affordances, like running and jumping.
Remove the IFrames state and make if a boolean or a simple timer check.

moved IFrames node out of state machine.


Opencensus Span Failure

Affecting:
Server
Version:
c49512d
Describe the bug
When executing execute_delete_by_query on a scan result, a key error is triggered in self._attach_shard_span_attrs. I suspect this likely has to do with ES6 vs ES7 (I am currently using es 7.6.2). I'm not even running opencensus sensors at the moment so this code path should definitely not be causing internal server errors.
To Reproduce
Steps to reproduce the behavior:

Setup to use an elastic 7.x cluster
Collect some scans
Login as an admin
Visit a scan result page and attempt to delete the scan using the admin interface.
Scan is deleted, but causes an error afterwards

Expected behavior
The scan should be deleted and the application should not throw a 500.
Logs
<IP_ADDRESS> - - [26/Jun/2020 08:50:28] "POST /admin/scans/delete/60ab2a6a40250d8c6c13e9f6d1a39e84 HTTP/1.1" 500 -
Traceback (most recent call last):
  File "/.venv/lib/python3.8/site-packages/flask/app.py", line 2464, in __call__
    return self.wsgi_app(environ, start_response)
  File "/.venv/lib/python3.8/site-packages/flask/app.py", line 2450, in wsgi_app
    response = self.handle_exception(e)
  File "/.venv/lib/python3.8/site-packages/flask/app.py", line 1867, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/.venv/lib/python3.8/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/.venv/lib/python3.8/site-packages/flask/app.py", line 2447, in wsgi_app
    response = self.full_dispatch_request()
  File "/.venv/lib/python3.8/site-packages/flask/app.py", line 1952, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/.venv/lib/python3.8/site-packages/flask/app.py", line 1821, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/.venv/lib/python3.8/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/.venv/lib/python3.8/site-packages/flask/app.py", line 1950, in full_dispatch_request
    rv = self.dispatch_request()
  File "/.venv/lib/python3.8/site-packages/flask/app.py", line 1936, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/.venv/lib/python3.8/site-packages/flask_login/utils.py", line 272, in decorated_view
    return func(*args, **kwargs)
  File "/opt/natlas/natlas-server/app/auth/wrappers.py", line 38, in decorated_function
    return f(*args, **kwargs)
  File "/opt/natlas/natlas-server/app/admin/routes.py", line 463, in delete_scan
    deleted = current_app.elastic.delete_scan(scan_id)
  File "/opt/natlas/natlas-server/app/elastic/interface.py", line 153, in delete_scan
    result = self.client.execute_delete_by_query(
  File "/opt/natlas/natlas-server/app/elastic/client.py", line 132, in execute_delete_by_query
    self._attach_shard_span_attrs(span, results)
  File "/opt/natlas/natlas-server/app/elastic/client.py", line 165, in _attach_shard_span_attrs
    span.add_attribute("es.shards.total", results["_shards"]["total"])
KeyError: '_shards'

The contents of results at the time of failure:
{'took': 78, 'timed_out': False, 'total': 1, 'deleted': 1, 'batches': 1, 'version_conflicts': 0, 'noops': 0, 'retries': {'bulk': 0, 'search': 0}, 'throttled_millis': 0, 'requests_per_second': -1.0, 'throttled_until_millis': 0, 'failures': []  }

The response body for Elasticsearch 6.x and 7.x do not contain any _shard stats. Thus, we should not attempt to add these attributes to OpenCensus.


Requesting new actions to move to previous/next peak on selected track

I'm requesting this on behalf of a bunch of blind folks who're using Reaper with screen reader software via an open source accessibility bridge called OSARA. I reckon these actions would also get considerable use from the wider Reaper userbase, but - in the grand scheme of things - the number of people in our community means it's tricky to make a splash in the feature requests forum of reaper.fm, so I'm hoping an SWS coder can help.
Currently, there are actions to identify and move to the peak of an item. However, when working with tracks containing hundreds of items (think editing longform podcast material etc), the process of identifying peaks item by item becomes a huge time suck, because there's currently no equivalent workflow for zooming out, mousing along the timeline and scanning visually for peaks.
I'm imagining two new actions called something like:
SWS: Move to previous peak on selected track
SWS: Move to next peak on selected track
It would be superb if these actions also worked on the master track when visible and selected.
I might be grossly oversimplifying, but it seems like these actions could feed off of existing code that analyzes multiple items when normalizing to common gain?
Ready and willing to provide any additional info that might be helpful, or to test builds etc.
Thanks for reading!
Scott

Hi there, the way I envisage this, is that there are two actions, not next and previous, but ones that correspond to SWS: Move cursor to item peak sample and SWS: Analyze and display item peak and RMS (entire item)
The difference is that they would consider all the items on the track or all the selected items in the same way that "SWS: Normalize items to overall peak RMS" considers all the selected items.
This would speed up my work flow considerably. Currently to find the peak sample on a track with many items, I need to use the true peak option from within the Loudness tool. This is fantastic, however time consuming and not as flexible as it would be to have the above two actions.
I want to be able to find and navigate to the single peak sample from a selection of items, rather than just the single item, and have the RMS, (and ideally the integrated loudness) of that selection of items displayed
Proposing something like:
SWS: Move cursor to peak sample of selected items and
SWS: Analyze and display item peak and RMS (selected items


Rust lang quick install link using curl ...

The easiest way to install rust is through the install page here
rust-lang.org/install
the command given is :
curl -s https://static.rust-lang.org/rustup.sh | sudo sh

Instead can we not have the link like this :
curl -s https://get.rust-lang.org | sudo sh

such a link is easier to remember at the top of the head and most people wont have to visit the site  just to get the install link.
Thanks.

With the new rustup.rs install instructions the link is pretty short. That being said, I don't know that the link being memorable matters that much.


Changed target to chrome 39.

Hi! Reasons for this change are:

aligning with targets used in SDK / CLI
impact is minimal (only difference is the typeof as you said, which is IE only anyway)
being specific about what we (want to) support. Currently the lowest official target is chrome v39 (used to be chrome 44). Using IE 11 as a target, suggests we “officially support” it

Fair enough, although personally I see significant difference between officially supporting platform and setting it as target for bundling. Some of platforms that we may want officially support in the future (especially embedded ones) are not included in babel presets so eventually I'd expect we would end up setting lowest possible preset (ie 11 in this case) and adding custom polyfills.


KeyError:'meta'

HI, I just pull out the master branch code and run the experiment without changing anything. But still has this problem:
2018-03-13 21-02-21 toxic >>> Training...
2018-03-13 21-02-21 toxic >>> step xy_train adapting inputs
Traceback (most recent call last):
File "/usr/local/lib/python3.6/dist-packages/deepsense/neptune/job_wrapper.py", line 113, in
execute()
File "/usr/local/lib/python3.6/dist-packages/deepsense/neptune/job_wrapper.py", line 109, in execute
execfile(job_filepath, job_globals)
File "/usr/local/lib/python3.6/dist-packages/past/builtins/misc.py", line 82, in execfile
exec_(code, myglobals, mylocals)
File "main.py", line 382, in
action()
File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 722, in call
return self.main(*args, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 697, in main
rv = self.invoke(ctx)
File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 1066, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 895, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 535, in invoke
return callback(*args, **kwargs)
File "main.py", line 255, in train_evaluate_predict_cv_pipeline
pipeline_name)
File "main.py", line 311, in _fold_fit_loop
_ = pipeline.fit_transform(data_train)
File "/neptune/steps/base.py", line 71, in fit_transform
step_inputs[input_step.name] = input_step.fit_transform(data)
File "/neptune/steps/base.py", line 71, in fit_transform
step_inputs[input_step.name] = input_step.fit_transform(data)
File "/neptune/steps/base.py", line 71, in fit_transform
step_inputs[input_step.name] = input_step.fit_transform(data)
File "/neptune/steps/base.py", line 74, in fit_transform
step_inputs = self.adapt(step_inputs)
File "/neptune/steps/base.py", line 153, in adapt
raw_inputs = [step_inputs[step_name][step_var] for step_name, step_var in step_mapping]
File "/neptune/steps/base.py", line 153, in
raw_inputs = [step_inputs[step_name][step_var] for step_name, step_var in step_mapping]
KeyError: 'meta'
I use pip3 and install all package in the requirement, so did I do it wrong? or could you please fix this?
Thank you

When training first level models with train_evaluate_predict_cv_piepeline you need to pass --model_level first .
It wasn't explained in the wiki as we were previously using train_evaluate_predict_pipeline.
Now the wiki is updated and I think it should work just fine.
Will test it tomorrow.


Migration fails with 'invalid cross-device link'

Before you start please confirm the following.

[X] Yes, I've searched similar issues on GitHub.
[X] Yes, I've checked whether this issue is covered in the Portainer documentation or knowledge base.

Problem Description
Portainer BE fails to start with the following error message:
portainer  | 2023/09/20 04:06AM INF main.go:444 > encryption key file not present | filename=portainer
portainer  | 2023/09/20 04:06AM INF main.go:478 > proceeding without encryption key |
portainer  | 2023/09/20 04:06AM INF github.com/portainer/portainer-ee/api/database/boltdb/db.go:125 > loading PortainerDB | filename=portainer.db
portainer  | 2023/09/20 04:06AM INF github.com/portainer/portainer-ee/api/datastore/backup.go:164 > creating DB backup |
portainer  | 2023/09/20 04:06AM INF github.com/portainer/portainer-ee/api/datastore/backup.go:74 > copying DB file | from=/data/portainer.db to=/data/backups/EE/portainer.db.<IP_ADDRESS>30920040654
portainer  | 2023/09/20 04:06AM INF github.com/portainer/portainer-ee/api/database/boltdb/db.go:125 > loading PortainerDB | filename=portainer.db
portainer  | 2023/09/20 04:06AM INF github.com/portainer/portainer-ee/api/datastore/migrate_data.go:90 > migrating database from version 2.18.4 to 2.19.1 |
portainer  | 2023/09/20 04:06AM INF github.com/portainer/portainer-ee/api/datastore/migrator/migrate_ce.go:51 > migrating data to 2.19.0 |
portainer  | 2023/09/20 04:06AM INF github.com/portainer/portainer-ee/api/datastore/migrator/migrate_dbversion100.go:133 > enable `community` addon on all MicroK8s master nodes |
portainer  | 2023/09/20 04:06AM INF github.com/portainer/portainer-ee/api/datastore/migrator/migrate_dbversion100.go:281 > Success to migrate private key seed to private key file |
portainer  | 2023/09/20 04:06AM INF github.com/portainer/portainer-ee/api/datastore/migrator/migrate_dbversion100.go:287 > update edge stack status to have deployment steps |
portainer  | 2023/09/20 04:06AM INF github.com/portainer/portainer-ee/api/datastore/migrator/refresh_rbac_roles.go:15 > refreshing RBAC roles |
portainer  | 2023/09/20 04:06AM WRN github.com/portainer/portainer-ee/api/datastore/migrate_data.go:57 > migration failed, restoring database to previous version | error="failed to migrate database: failed in Always migrations returned error: failed rebuilding stack file system with version: failed to copy stack 2 project folder: failed to backup source directory: rename /data/compose/2 /data/compose-backup: invalid cross-device link" stack_trace=[{"func":"(*Store).MigrateData","line":"55","source":"migrate_data.go"},{"func":"initDataStore","line":"183","source":"main.go"},{"func":"buildServer","line":"481","source":"main.go"},{"func":"main","line":"784","source":"main.go"},{"func":"main","line":"250","source":"proc.go"},{"func":"goexit","line":"1598","source":"asm_amd64.s"}]

Expected Behavior
for it to start :)
Actual Behavior
migration fails
Steps to Reproduce
Theoretically:

Install the previous version of portainer in docker
Update to the current version -> migration fails

Portainer logs or screenshots
No response
Portainer version
2.19.1
Portainer Edition
Business Edition (BE/EE) with 5NF / 3NF license
Platform and Version
Docker 24.0.6
OS and Architecture
Debian 12 x64
Browser
No response
What command did you use to deploy Portainer?
version: "2"

services:
  portainer:
    container_name: portainer
    image: portainer/portainer-ee:alpine
    command: -H unix:///var/run/docker.sock
    restart: always
    environment:
      - VIRTUAL_HOST=docker.xxxx.de
      - VIRTUAL_PORT=9000
    ports:
      - 9000:9000
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
      - portainer_data:/data
      - /opt/docker:/data/compose
    labels:
      portainer.setting.hide: "hide"
      com.centurylinklabs.watchtower.enable: "true"

volumes:
  portainer_data:

networks:
  default:
    name: proxy-net
    external: true

Additional Information
No response

As part of the 2.19 release we made some changes to the filesystem for the compose directory in the portainer_data volume to facilitate adding the new stack versioning functionality. When the upgrade is performed, we take a backup of the compose directory (as compose-backup) for safety.
Because, in your specific configuration, you have /data in one location (the portainer_data volume) and the /data/compose directory in another (a bind mount to /opt/compose), the backup is attempting to traverse file systems, which the code is not happy with doing.
I'm discussing this with the wider team, but it's worth noting that splitting /data and /data/compose is not a configuration we support, hence this error.

@jamescarppe because stack versioning is moving everything from /data/compose/<id> to /data/compose/<id>/v1, it breaks volumes mounted using relative paths:
e.g.
volumes:
      - ./etc:/etc/caddy

Is there a better practice to handle this? I don't see myself adding ./v1/ to all the paths, that would break local testing.

@alin23 This sounds like a separate issue to the one in this report - can you potentially open this as a separate report? That way we can ensure we track it properly.


[refs #42] Introduce our testing policy

Add documentation for our testing policy for browsers, screen
resolutions and assistive technologies. Including a checklist for pull
requests. Also remove redundant documentation sections.

@mcheung-nhs @davidhunter08 is there anything you’d like to add/remove to this?

@mcheung-nhs cheers Michael, is there anything you can add to the Accessibility section, maybe about testing for users that change colours of their screen/high contrast mode?

@Ross-Clark @sabi98 have you any feedback on this? Does it all make sense?

@AdamChrimes I would consider adding links in the software column of the "Assistive Technology Support" section, linking to the respective softwares official website. This might make it easier for users of this page to research these peices of software if they are not familiar with them.

Good suggestion @Ross-Clark do you want to see if you can make the change? On the feature/testing-criteria-docs branch


Method 'get_LockReleaseBehavior' in type does not have an implementation.

Running the following command generates the following error
dotnet ef dbcontext scaffold "Provider=Microsoft.ACE.OLEDB.12.0;Data Source={Removed};" EntityFrameworkCore.Jet
System.TypeLoadException: Method 'get_LockReleaseBehavior' in type 'EntityFrameworkCore.Jet.Migrations.Internal.JetHistoryRepository' from assembly 'EntityFrameworkCore.Jet, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60' does not have an implementation.
   at Microsoft.Extensions.DependencyInjection.JetServiceCollectionExtensions.AddEntityFrameworkJet(IServiceCollection serviceCollection)
   at EntityFrameworkCore.Jet.Design.Internal.JetDesignTimeServices.ConfigureDesignTimeServices(IServiceCollection serviceCollection)
   at Microsoft.EntityFrameworkCore.Design.Internal.DesignTimeServicesBuilder.ConfigureDesignTimeServices(Type designTimeServicesType, IServiceCollection services)
   at Microsoft.EntityFrameworkCore.Design.Internal.DesignTimeServicesBuilder.ConfigureProviderServices(String provider, IServiceCollection services, Boolean throwOnError)
   at Microsoft.EntityFrameworkCore.Design.Internal.DesignTimeServicesBuilder.CreateServiceCollection(String provider)
   at Microsoft.EntityFrameworkCore.Design.Internal.DesignTimeServicesBuilder.Build(String provider)
   at Microsoft.EntityFrameworkCore.Design.Internal.DatabaseOperations.ScaffoldContext(String provider, String connectionString, String outputDir, String outputContextDir, String dbContextClassName, IEnumerable`1 schemas, IEnumerable`1 tables, String modelNamespace, String contextNamespace, Boolean useDataAnnotations, Boolean overwriteFiles, Boolean useDatabaseNames, Boolean suppressOnConfiguring, Boolean noPluralize)
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.ScaffoldContextImpl(String provider, String connectionString, String outputDir, String outputDbContextDir, String dbContextClassName, IEnumerable`1 schemaFilters, IEnumerable`1 tableFilters, String modelNamespace, String contextNamespace, Boolean useDataAnnotations, Boolean overwriteFiles, Boolean useDatabaseNames, Boolean suppressOnConfiguring, Boolean noPluralize)
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.ScaffoldContext.<>c__DisplayClass0_0.<.ctor>b__0()
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.OperationBase.<>c__DisplayClass3_0`1.<Execute>b__0()
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.OperationBase.Execute(Action action)
Method 'get_LockReleaseBehavior' in type 'EntityFrameworkCore.Jet.Migrations.Internal.JetHistoryRepository' from assembly 'EntityFrameworkCore.Jet, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60' does not have an implementation.

It sounds like you are using alpha 1 from nuget.
That was built against the preview 7 of upstream EF Core. While for the most part it should work, there was a major change between the upstream RC1 and RC2 related to the History Repository (the part that deals with the __EFMigrationsHistory table).
This is the part that it is looking for and not finding.
I do have that fixed in the current daily builds (see #265) so the next release to nuget will have it.
In the meantime, you can try follow the instructions and link to a daily build.
I will keep this open for now until there is a version available in nuget that has the above fix in it

@roughiain There is a beta release that should now be on nuget that is compatible

@ChrisJollyAU will test and give feedback

Works! thank you.


Add basque language support

Thanks for your support, I think you may missing a case for Basque in i18nObjInLocale method, please refer to other PRs to add it, thanks very much

I added the case for Basque in i18nObjInLocale method just before default case. I was not sure if I need to order it alphabetically because apparently no order was used, so I put it in the last position.
Looking to others PRs I realized I did not edit README so I did. Let me know if something else is missing. Thanks.

Sorry, I can't see the 'eu' case in i18nObjInLocale in PR, however, I added it in this PR, so Basque has been merged ,thanks for your support

You were right, I forgot to commit the change. My fault. Thanks very much for all


Add -Coverflow-checks=on to the list of 'careful flags'

I believe this mostly matters for if the stdlib will have these on, since normally it would be on for non-release builds of user code. This perhaps indicates a bug in the stdlib more than in the user's code, but I am not sure if that applies to 100% of the cases this could catch.

Filed https://github.com/RalfJung/rustc-build-sysroot/issues/1 about the fact that this will not take effect until the next time the user updates their nightly.


Refactor BaseLabeler as parent class of label models

Description of proposed changes

Move predict_proba to BaseLabeler class as abtractmethod
Move predict, score, save, and load methods to BaseLabeler class
as shared methods in parent class
Update RandomVoter, MajorityClassVoter, MajorityLabelVoter, and
LabelModel to be subclasses of BaseLabeler.
Update docstrings to reflect abstract/parent class hierarchy.
NOTE: We re-define methods in the LabelModel(BaseLabeler) subclass for predict and score in order to explicitly show docstring examples

Test plan

Existing tests pass: Ran tox and tox -e complex.

Checklist
Need help on these? Just ask!

[x] I have read the CONTRIBUTING document.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] I have run tox -e complex and/or tox -e spark if appropriate.
[x] All new and existing tests passed.

The logic all looks good. The only thing that bugs me is having predict() and score() being strictly pass-through methods on LabelModel. The duplicated passing of arguments and documentation invites staleness. The documentation demonstrating usage should be covered by unit tests anyway. I'd vote we just let the LabelModel inherit those methods from its parent.

Discussed IRL. Decision to leave the methods on LabelModel to keep the Examples section for the documentation.


animation not working first time why?

when I open activity, animations not working and if I reopen that activity it works what the reason?

Hey @Rizq-App , can you please share the relevant part of your code so we can have a look?


Update ember-cli to 1.13 and the fixture adapter to work with Ember Data 2.0

Can this be merged?

+1

Need travis check success to be merged ?
This fix is needed.

@Swizz @jme783. I've been busy with client work and haven't had much time to clean up this pr. If one of you would like to take this branch and get the tests passing I would be happy to cut a release once the changes have been merged.

Any updates?

@demoneaux just a head up regarding the fixture adapter: it is kind of deprecated in favor of the much more mature ember-cli-mirage, ember-data-factory-guy and even https://github.com/pretenderjs/pretender. Depending on your use case, I would suggest you try one of those libraries.

What is a good way to embed static data into your models.  Data not located on the server.  I've traditionally used the fixture adapter, but seems to be out of style now.

@wdch-nseeley ember-cli-mirage can be used in production for example, see http://www.ember-cli-mirage.com/docs/v0.1.x/server-configuration/#enabled. If you don't want to use an additional addon, you can also store.push your fixture in the model() hook of your application route.

I agree with @wdch-nseeley.
I develop Ember-electron apps, and I commonly use Fixtures to give me a way to enpack statics datas with my app.
I tried Mirage for other projects but I keep in mind that Fixture is more easy to use in this case.
And I do not know if pushing all models separatly is more pleasant as writing a simple array of objects.

@Swizz ok, thanks for bringing up your use case. I can see now how there is still need for this. Please take my statement about the fixture adapter being kind of deprecated with a grain of salt.
I've opened a rebased version of this PR. Can you give this a try and report back if this allows you to use the fixture adapter with ember-data 2.x? The location is at pangratz/ember-data-fixture-adapter#rebased-2.0...

Thanks. I will give it a try.


Feature Request: PF 1st. Ed.??

 People have been asking for a downtime module for PF 1st. Ed. Is there any way you could just snip this out of 5th Ed. and make it a universal module? ..or make another one that is?
I see that you've made like 40 different revisions on this, so I'm guessing that you would be WELL aware if this would be a fairly simple task, or a living nightmare. I wouldn't even ask, but it does seem like something like this would be fairly easy to make in to a universal sort of module.
If it would be easier to plug it in to PF 1st. Ed. (which is basically DnD 3.5) for some reason, I'd be happy to collect data for integration and etc. I mean, I could even attempt the coding myself.. but it's been a few decades. ;)

Trust me, you're not the first nor probably the last to ask for other platforms. When I built this module I initially intended for it to be cross-platform, I just haven't gotten to implementing it in another system. In theory there are only a handful of sections that need changed, but the more interesting bit is trying to make it in a maintainable way so that I don't have 50 million if statements. It's definitely not in the ballpark of living nightmare, but there are definitely things that need to be tested. I'll look into it.

Progress update!

Just got to work on making sure rolls can be done, and that's the hard part.

Progress update!

Rolling for abilities, saves, and skills seem to be working. Just have to research what tools are available and test custom formulas. Will likely push a beta branch here shorty for you to test on.

Please check out the new beta branch for pf1 support. It seems to work on my end, but I want to see if y'all can break it. You'll need to download the files from the branch as there's no manifest for it. https://github.com/Ethck/Ethck-s-Downtime-Tracking/tree/feature/pf1e-support

Holy crap! I just found this entire thread in my email!! More than happy to
take a stab at it!
Geez, this is like accidentally opening a birthday present up early! :D
You are awesome! :D

James T. Long

There are 10 kinds of people in the world - those who understand binary,
and those who don't.
On Fri, Aug 27, 2021 at 9:51 AM Ethck @.***> wrote:

Closed with bd326f0
https://github.com/Ethck/Ethck-s-Downtime-Tracking/commit/bd326f07a3fe8849c57635188869a2d74fa03aaf
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Ethck/Ethck-s-Downtime-Tracking/issues/68#issuecomment-907300628,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFSKADX4NFQLUTV4JY3QGKLT66X7BANCNFSM5CRRSD5A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Slash direction

I've noticed that some slashed (strikethrough) icons have the slash in a different direction, sometimes necessarily. Any way to make this uniform or offer every icon with slashes in both directions?

All icons have one direction except phone-off.
For phone-off wrong direction is only one nice variant.


YesButton and NoButton text

Is It possible to change YesButton and NoButton text?

I assume that you are referring to the AlertBuilder class since AlertDialogBuilder is deprecated.
For AlertBuilder, the yesButton and noButton are nothing but simple extension methods on top of positiveButton and negativeButton:
inline fun AlertBuilder<*>.yesButton(noinline handler: (dialog: DialogInterface) -> Unit) =
    positiveButton(android.R.string.yes, handler)

inline fun AlertBuilder<*>.noButton(noinline handler: (dialog: DialogInterface) -> Unit) =
    negativeButton(android.R.string.no, handler)

so I think you can just call the positiveButton and negativeButton with the text you like.

I assume that you are referring to the AlertBuilder class since AlertDialogBuilder is deprecated.
For AlertBuilder, the yesButton and noButton are nothing but simple extension methods on top of positiveButton and negativeButton:
inline fun AlertBuilder<*>.yesButton(noinline handler: (dialog: DialogInterface) -> Unit) =
    positiveButton(android.R.string.yes, handler)

inline fun AlertBuilder<*>.noButton(noinline handler: (dialog: DialogInterface) -> Unit) =
    negativeButton(android.R.string.no, handler)

so I think you can just call the positiveButton and negativeButton with the text you like.

How can I call this in Anko dialogs?

Something like this:
alert {
    positiveButton("YES!!!", {})
    negativeButton("NOPE!!!, {})
    // something else ...
}


Constantly rebooting when running on qemu

It is constantly rebooting after this point when i tried to run the built image on qemu.

When running with -no-reboot qemu just quickly closes:
PS C:\Users\trist\Downloads\felix-main\felix-main> qemu-system-i386 -drive file=build/disk.img,index=0,media=disk,format=raw,if=ide -no-reboot

(qemu:14460): Gtk-WARNING **: 12:55:30.708: Could not load a pixbuf from icon theme.
This may indicate that pixbuf loaders or the mime database could not be found.
PS C:\Users\trist\Downloads\felix-main\felix-main>

Looks like it is the same error as this: https://stackoverflow.com/questions/57373848/qemu-triple-faulting-when-enabling-interrupts

It's due a unexpected behavior caused by static mutables.
For some reason the interrupt descriptor table gets overwritten random data, so when an interrupt cpu does not jump to the expected handler.
I'm working on a fix.
By the way, have you built the image using build.sh script or using docker?

Ok, so I tried a lot of things:

Building via docker: Doesn't run with qemu on neither windows nor WSL
Building directly on WSL: Doesn't run either
Running an image from the releases: Works on both windows and WSL
Building on a Kali Linux VM: Runs via qemu on Kali Linux

Looks like it is something Windows specific when building however i don't understand why it then won't run when built on WSL or docker. My WSL is running ubuntu.

Maybe different versions of the tools used in build script lead to different behaviors.
However it should be fixed now.
Can you try building again in those different environments and check if it works now?
It would be very helpful.

yes, works both when built directly with the script on WSL and with docker on windows

Thank you for testing!


Exposing reviewers property on accessReviewInstance model

Overview
The following is an overview of the end to end flow for access reviews scenario.

So far, we've exposed three properties for reviewers configuration(reviewersType,reviewers and backupReviewers) on the "accessReviewScheduleDefinition" object.
An accessReviewScheduleDefinition contains access review instances. Now every accessReviewScheduleDefinition contains multiple instances[one for every invocation of the review. For ex: if a schedule definition running weekly, there's one instance every week]. Every instance automatically inherits the list of reviewers and fallbackReviewers from an accessReviewScheduleDefinition when it's created
We now want to expose these three properties : reviewersType, reviewers and fallbackReviewers so that developers can change it for an actively running instance if they want.

Changes

Expose three properties: reviewersType, reviewers and fallbackReviewers on the accessReviewInstance object[same as how they exist on accessReviewScheduleDefinition object]
Expose route for PUT instance

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

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

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

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

Adding new API(s)
Adding a new API version
[ ] Ensure to copy the existing version into new directory structure for first commit (including refactoring) and then push new changes including version updates in separate commits. This is required to review the changes efficiently.
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

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

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

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Hi @shubhamguptacal as you are adding new version, it's good to fix the lintdiff warnings https://github.com/Azure/azure-rest-api-specs/pull/14560#issuecomment-849061367
and you can safely ignore the current cross version breaking change failure, as the single error add enum value to modeled as string enum is false alarm and will be corrected soon.

Hi @shubhamguptacal as you are adding new version, it's good to fix the lintdiff warnings #14560 (comment)
and you can safely ignore the current cross version breaking change failure, as the single error add enum value to modeled as string enum is false alarm and will be corrected soon.

Thanks @zhenglaizhang for reviewing this PR. I went through the warnings and fixed as many as possible. Can you please take a look again?

/azp run

/azp run

@ArcturusZhang  @msyyc Please help check the SDK breaking change.

@shubhamguptacal pls help fix the validation errors under https://github.com/Azure/azure-rest-api-specs/pull/14560#issuecomment-849061367

Probbaly wrong place but I'm trying to deploy an access review using arm following the content on https://docs.microsoft.com/nl-nl/azure/templates/microsoft.authorization/accessreviewscheduledefinitions?tabs=json
This always leads to something like: The request content was invalid and could not be deserialized: 'Could not find member 'backupReviewers' on object of type 'TemplateResource
tried different schema's but all with the same result. Same goes for "descriptionForReviewers", "descriptionForAdmins" etc depending on which one occurs first in the arm template


first-published-tag-for-merged-pr on issues

It’s not as straightforward because we’d need to fetch the commit I think, but what do you think about adding the banner on issues that end with “closed by PR”? It’d have to exclude PRs coming from other repos however

Maybe the cacheKey of first-published-tag-for-merged-pr could become pull request number so we don't need a separate cache for this.

I think this feature is becoming a little too big would need to extract the logic into a new file too.

Agreed, I think that a whole new feature can be added just for organization purposes.
first-published-tag-for-fixed-issue (so long, but needed for consistency)
Or:

tag-after-pr (rename)
tag-after-issue

The complexity in tag-after-issue comes from having to use the API to fetch the merge commit from the "fixed by" PR. In the PRs this piece of information is available on the page:
https://github.com/refined-github/refined-github/blob/a4a7f4839cc95b2e90a26171291b0993a1b25053/source/features/closing-remarks.tsx#L46


Integration of Vector Crypto into the Unprivileged spec.

Note that I've completed integration of the Vector Crypto spec into the Unprivileged spec.
Here is the PR.
And here's the PDF.
I expect the PR will be merged on Weds, March 20th or Thursday, March 21st.

Vector crypto is now integrated into the Unprivileged spec.  Note that I'll begin asking for Issue closure or transfer to the riscv-isa-manual repo when an issue is still valid.  Following that, this repository will be archived.


typos

README.md: "formely known as" -> "formerly known as"

Codecov Report

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

@@          Coverage Diff           @@
##           master    #816   +/-   ##
======================================
  Coverage    78.9%   78.9%
======================================
  Files          34      34
  Lines        6192    6192
======================================
  Hits         4886    4886
  Misses       1306    1306

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 71d597d...e5a18b3. Read the comment docs.

Thanks!


[Question] Does hyperkube in this repo also a pruned one?

I found that here are some explanation for hyperkube in k3s https://github.com/rancher/k3s#hyperkube , so does the hyperkube here a pruned one?
As I found that the binary of hyperkube was only 100+M, while the hyperkube from vanila k8s was 200+M.
And alslo the code here for k8s seems also pruned https://github.com/rancher/k3s/tree/master/vendor/k8s.io/kubernetes ?

The hyperkube is a compilation of the k8s source we use so it has all the same bits removed as k3s.
The source for it and the corresponding patches are referred to in vendor.conf in the root of this repo.
Currently the hyperkube by itself is not well tested. It should work by YMMV. Please file issues if something doesn't work. We will work on testing it more.

@ibuildthecloud can you please show some detail for how did you remove the useless code in https://github.com/rancher/k3s/tree/master/vendor/k8s.io/kubernetes ?

There is a lot of manual pruning involved, looking at the history of https://github.com/rancher/k3s/commits/k3s-v1.13.3 might help.

Thanks @erikwilson , this really helps!!


How can i decrease the width(distance) between two x axis labels in bar graph

Is there any way to decrease the distance between two labels(name of people) represented by the red arrow.Give me the solution.

There is no way to do this. It looks like your datasets have no data for the first label. Why not just remove the first label until there is data


Default Options (without AMD)

Description

[x] Source branch in your fork has meaningful name (not master)
This is an attempt to add the defaultOptions (see #1584).
It currently seems to work for the commonJS (non-AMD version), but doesn't have AMD support.
For that, there might be some bigger changes with the exports required.
Maybe @bitwiseman can give me a hint.

Fixes Issue:
Closes #1584
Before Merge Checklist
These items can be completed after PR is created.
(Check any items that are not applicable (NA) for this PR)

[x] JavaScript implementation
[x] Added Tests
[ ] README.md documents new feature/option(s)

@Hirse  Any chance you'll get to this?


Parinfer mangles pasted code

Pasting code into Nightcode mangles the grouping symbols. For example pasting (from http://www.braveclojure.com/do-things/)
(if true
"By Zeus's hammer!"
"By Aquaman's trident!")
results in
(if true)
"By Zeus's hammer!"
"By Aquaman's trident!"
This is true for both the REPL pane and the editor. Is it possible to disable parinfer, either all together or at least during pasting code?

This issue should now be fixed. Please let me know if you still see paste issues with the latest commit.

It now works great in the editor pane. The pane in the lower left (I called it the REPL pane but maybe it is called the pad?) still mangles the parentheses.

Are you sure? I tried pasting your if statement into the REPL pane and it looks like this:

Ah I see, thank you for testing it out. I'll go ahead with a new release.


gitlab authentication error

I am attempting to set the stable release with gitlab. I have gotten as far as getting it to redirect to an internal insance of gitlab and I am able to authorize it to connect with kanboard but then when I get back to the login "External authentication failed" the debug look like they are gettng what they need from gitlab but then the user is not created.

Did you have a look into Debug logging?
$ grep DEBUG config.default.php
define('DEBUG', false);
define('DEBUG_FILE', __DIR__.'/data/debug.log');

Yes. It was in those logs where I saw that kanboard was getting back what looks like appropriate info from gitlab. But then I'd get the above error and no kanboard user was created.

Do you think without a bit more info (for example the corresponding logs) anybody is able to help you? Telling others 'It doesn't work, please help!' might a bit unhelpful to help you with your problem.

Note that Gitlab OAuth is working only if you have enabled url rewriting for Kanboard because Gitlab doesn't support url with query string arguments.

What sort of rewriting needs to be done?

http://kanboard.net/documentation/gitlab-authentication#known-issues
http://kanboard.net/documentation/nice-urls

Ok I have that enabled, do i need to change the gitlab redirect uri?

Nevermind, I found the oath url in interrogations. Still is returning the error however.

Finally I was able to make gitlab auth working, but it was tricky. Here's what I did
1. Rewrite
First of all, you need to turn on url rewriting:

Enable mod_rewrite

$ sudo a2enmod rewrite

Comment out the ENABLE_URL_REWRITE in config.php (or set it to true):

// Enable/disable url rewrite
//define('ENABLE_URL_REWRITE', false);

2. HTACCESS
Second, somehow I was missing .htaccess in root directory, which prevented mod_rewrite from working.
3. External user
Create external user in kanboard and make sure you fill Gitlab Id. You can check your gitlab user id with api:
$ curl -k https://your.gitlab.com/api/v3/users?private_token=TOKEN

You can check your token value in your profile settings: https://your.gitlab.com/profile/account
4. Gitlab Redirect
After you enable url rewrite, you should update callback url in gitlab to be like:
https://your.kanban.com/oauth/gitlab

I  got similar issue, and solved by create remote user in admin panel. It seems the gitlab integration does not create new user if it does not exists.

Thank you @at15, your hint helped me to trace the problem and find a workaround, now I can successfully login using GitLab
modify the file: app/User/OAuthUserProvider.php
    public function isUserCreationAllowed()
    {
        return true; // change to true
    }

and
    public function getUsername()
    {
        return $this->user['username']; // return the username
    }

@fguillot Can you look into this? is it ok to use the above workaround ?

Gitlab auth have been moved to a plugin now. If you have any problem, open issues here: https://github.com/kanboard/plugin-gitlab-auth


[nrf fromlist] Bluetooth: Mesh: Fix friend node establishment and message

Upstream PR: zephyrproject-rtos/zephyr/pull/30260
Fix friend node unable decrypt friend_cred encrypt poll message
when friendship not establishment.
Fix friend node cache segment message for given lpn with same
sequence number, cause to lpn replay attack.
Fix friend KR phase to 1, no need to add friend_update.
However KR phase to 2, must add friend_update.
Signed-off-by: Lingao Meng<EMAIL_ADDRESS>

All checks passed.
checkpatch (informational only, not a failure)
-:180: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#180: FILE: drivers/ieee802154/ieee802154_nrf5.c:417:
+    !defined(CONFIG_NRF_802154_SER_HOST)$

-:427: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#427: FILE: include/net/socket.h:850:
+    char ifr_name[IFNAMSIZ]; /* Interface name */$

- total: 0 errors, 2 warnings, 639 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

Your patch has style problems, please review.

NOTE: Ignored message types: AVOID_EXTERNS BRACES C99_COMMENT_TOLERANCE CONFIG_EXPERIMENTAL CONST_STRUCT DATE_TIME FILE_PATH_CHANGES MINMAX NETWORKING_BLOCK_COMMENT_STYLE PREFER_KERNEL_TYPES PREFER_SECTION PRINTK_WITHOUT_KERN_LEVEL REPEATED_WORD SPDX_LICENSE_TAG SPLIT_STRING VOLATILE

NOTE: If any of the errors are false positives, please report
      them to the maintainers.

Tip: The bot edits this comment instead of posting a new one, so you can check the comment's history to see earlier messages.

Included in https://github.com/nrfconnect/sdk-zephyr/pull/402


use unicode escapes in config.cson

Thanks a lot for this plug-in. It save a lot work, avoiding the need of making new plug-ins for every external command integration.
I am trying to integrate the comand for this solution: http://stackoverflow.com/a/9395552/469294
More or less:
grep --color='auto' -P -n "[\x80-\xFF]" {FileName}
so I did try:
  "atom-shell-commands":
    commands: [
      {
        name: "grep_unicode"
        command: "grep"
        arguments: [
          "--color='auto'"
          "-P"
          "-n"
          "\"[\x80-\xFF]\""
          "{FileName}"
        ]
        options:
          cwd: "{FileDir}"
          keymap: "cmd-u"
      }
    ]

but it doesn't work:
grep","--color='auto'","-P","-n","\"[-ÿ]\"","myFile.java"]
Usage: grep [OPTION]... PATTERN [FILE]...
Search for PATTERN in each FILE or standard input.
.....

How am I supposed to specify the escape sequence for those unicode characters ?

Did you try escaping the backslashes? It looks like the command did what it was supposed to. In CSON, \x80 is replaced with � in the string content, so write \\x80, and the string content will end up as \x80.


encoding.py missing OGG_OPUS and SPEEX_WITH_HEADER_BYTE encodings

The new encodings  OGG_OPUS and SPEEX_WITH_HEADER_BYTE are missing.
https://github.com/GoogleCloudPlatform/google-cloud-python/blob/master/speech/google/cloud/speech/encoding.py

@lukesneeringer Are these in some pb2 module?

https://github.com/googleapis/api-client-staging/blob/master/generated/python/gapic-google-cloud-speech-v1/google/cloud/gapic/speech/v1/enums.py#L66-L67

So shouldn't we just use those values?

Just a note that we will be adding additional new encodings in Q2.  So +1
on Danny's suggestion: it's highly desirable to expose the autogenerated
values directly to reduce the manual changes needed as we add new features.
On Wed, Apr 12, 2017 at 9:55 AM, Danny Hermes<EMAIL_ADDRESS>wrote:

So shouldn't we just use those values?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-python/issues/3288#issuecomment-293641499,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AS_go6rSW2TxUh5xiTWKIBh1F6v16-ACks5rvQHkgaJpZM4M6jiV
.

@lukesneeringer Can we re-open this / file a new issue to discuss a plan of attack to dump the hand-crafted enums?

The latter, please.

Filed #3296


Remove "Redux itself is very simple"

Reflecting a few years later this was a bit of a silly thing to write in the docs.
Of course it's not simple to people learning it.

I agree!


[bug]:  Stacked 3.4.0. IndexTrackingViewModel PageTransitionSwitcher

Describe the bug
Not sure if this is related,,  earlier versions of stacked I used to create Tabs page with PageTransitionSwitcher,, this will animate transition between tabs from BottomNavigationBar,,, using the same approach here is not working,, not sure why,, maybe you can help
The source code is below below and  I am attaching a video of how it looks like
To reproduce
class HomeView extends StackedView<HomeViewModel> {
  const HomeView({Key? key}) : super(key: key);

  @override
  Widget builder(
    BuildContext context,
    HomeViewModel viewModel,
    Widget? child,
  ) {
    return Scaffold(
      bottomNavigationBar: BottomNavigationBar(
          type: BottomNavigationBarType.fixed,
          currentIndex: viewModel.currentIndex,
          selectedLabelStyle: const TextStyle(
            fontSize: 12,
          ),
          selectedItemColor: SystemColors.primary,
          unselectedItemColor: SystemColors.black,
          unselectedLabelStyle: const TextStyle(fontSize: 11),
          onTap: (index) async {
            viewModel.setIndex(index);
          },
          items: [
            ...viewModel.tabs.asMap().entries.map((entry) {
              int index = entry.key;

              return BottomNavigationBarItem(
                icon: viewModel.getTabIcon(index),
                label: viewModel.getTabName(index),
                activeIcon: viewModel.getTabActiveIcon(index),
              );
            }).toList()
          ]),
      body: PageTransitionSwitcher(
        key: key,
        duration: const Duration(milliseconds: 2000),
        reverse: viewModel.reverse,
        transitionBuilder: (Widget child, Animation<double> primaryAnimation,
            Animation<double> secondaryAnimation) {
          return SharedAxisTransition(
            animation: primaryAnimation,
            secondaryAnimation: secondaryAnimation,
            transitionType: SharedAxisTransitionType.vertical,
            fillColor: Colors.red,
            child: child,
          );
        },
        child: Container(
            decoration: const BoxDecoration(
                color: SystemColors.lightbg),
            child: viewModel.getTab(viewModel.currentIndex)),
      ),
    );
  }

  @override
  HomeViewModel viewModelBuilder(
    BuildContext context,
  ) =>
      HomeViewModel();
}

View model is  IndexTrackingViewModel
class HomeViewModel extends IndexTrackingViewModel {

  List<Map<String, dynamic>> get tabs => [
        {
          'tab': const DashboardView(),
          'name': LocaleKeys.tabs_dashboard,
          'icon': 'dashboard.svg'
        },
        {
          'tab': const EventsView(),
          'name': LocaleKeys.tabs_events,
          'icon': 'events.svg'
        },
        {
          'tab': const NewsView(),
          'name': LocaleKeys.tabs_news,
          'icon': 'news.svg'
        },
        {
          'tab': const MyProfileView(),
          'name': LocaleKeys.tabs_my_profile,
          'icon': 'profile.svg'
        },
      ];

  getTab(int index) {
    return tabs[index]['tab'];
  }

  getTabIcon(int index) {

    return Padding(
      padding: const EdgeInsets.all(3),
      child: SvgPicture.asset(
        'assets/icons/${tabs[index]['icon']}',
      ),
    );
  }

  getTabActiveIcon(int index) {
    return Container(
      color: SystemColors.primary[50],
      padding: const EdgeInsets.all(3),
      child: SvgPicture.asset(
        'assets/icons/${tabs[index]['icon']}',
      ),
    );
  }

  getTabName(int index) {
    return tr('${tabs[index]['name']}');
  }

  changeTab(int index) async {
    setIndex(index);
  }
}

Expected behavior
No response
Screenshots
https://github.com/Stacked-Org/stacked/assets/10797121/5321dbad-e3a9-4970-8690-0603862daa5d
Additional Context
No response

Hi @bdairy , how are you? What is the code below 👇 ? tr function 😲 ?
getTabName(int index) {
    return tr('${tabs[index]['name']}');
}

IMPORTANT
You are using UI (Widgets) inside your ViewModel, that is not the way to go, you are breaking the conventions.

Hi @bdairy , how are you? What is the code below 👇 ? tr function 😲 ?

getTabName(int index) {

    return tr('${tabs[index]['name']}');

}

IMPORTANT
You are using UI (Widgets) inside your ViewModel, that is not the way to go, you are breaking the conventions. All your UI code should stay in your View and all your presentation logic should be on your ViewModel.

@ferrarafer thanks for the reply,
tr function is a translation function provided by a package called EeasyLocalization,, basically getting the translated text based on your locale and a provided key.
For the point you mentioned I agree . I should've created a widget in the view , but this is not the final code yet. I just posted hoping to focus on the transition issue.
Thanks for the guidance

@bdairy you can do something like below to solve your problem.
PageTransition
class PageTransition extends StatelessWidget {
  const PageTransition({
    Key? key,
    required this.child,
    required this.reverse,
  }) : super(key: key);

  final bool reverse;
  final Widget child;

  @override
  Widget build(BuildContext context) {
    return PageTransitionSwitcher(
      reverse: reverse,
      duration: const Duration(milliseconds: 300),
      transitionBuilder: (
        Widget child,
        Animation<double> primaryAnimation,
        Animation<double> secondaryAnimation,
      ) {
        return SharedAxisTransition(
          animation: primaryAnimation,
          secondaryAnimation: secondaryAnimation,
          transitionType: SharedAxisTransitionType.vertical,
          fillColor: Colors.red,
          child: child,
        );
      },
      child: child,
    );
  }
}

HomeView
class HomeView extends StackedView<HomeViewModel> {
  final int startingIndex;
  const HomeView({Key? key, this.startingIndex = 0}) : super(key: key);

  static const List<Map<String, dynamic>> tabs = [
        {
          'tab': const DashboardView(),
          'name': LocaleKeys.tabs_dashboard,
          'icon': 'dashboard.svg'
        },
        {
          'tab': const EventsView(),
          'name': LocaleKeys.tabs_events,
          'icon': 'events.svg'
        },
        {
          'tab': const NewsView(),
          'name': LocaleKeys.tabs_news,
          'icon': 'news.svg'
        },
        {
          'tab': const MyProfileView(),
          'name': LocaleKeys.tabs_my_profile,
          'icon': 'profile.svg'
        },
      ];

  @override
  Widget builder(
    BuildContext context,
    HomeViewModel viewModel,
    Widget? child,
  ) {
    return Scaffold(
      body: PageTransition(
        reverse: viewModel.reverse,
        child: tabs[viewModel.currentIndex]['tab'],
      ),
      bottomNavigationBar: BottomNavigationBar(
          type: BottomNavigationBarType.fixed,
          currentIndex: viewModel.currentIndex,
          selectedLabelStyle: const TextStyle(fontSize: 12),
          selectedItemColor: SystemColors.primary,
          unselectedItemColor: SystemColors.black,
          unselectedLabelStyle: const TextStyle(fontSize: 11),
          onTap: viewModel.setIndex,
          items: [
            ...tabs.asMap().entries.map((entry) {
              int index = entry.key;

              return BottomNavigationBarItem(
                icon: viewModel.getTabIcon(index),
                label: viewModel.getTabName(index),
                activeIcon: viewModel.getTabActiveIcon(index),
              );
            }).toList()
          ]),
    );
  }

  @override
  HomeViewModel viewModelBuilder(
    BuildContext context,
  ) =>
      HomeViewModel();
}

@ferrarafer that worked like charm,,, So I guess because we are rebuilding the UI whenever we change the index,, the animation is lost,, MY BIG question,, what is the difference between the earlier versions of stacked where we use normal stateless widgets with reactive view models,, and now when we just use the StackedView???
Again thanks for the helo @ferrarafer

No problem, glad to hear it worked.
Under the hood is the same, we just changed some styling methods and names because we think it's more comfortable and readable. It ends up in a functions with less nesting (indentation).
As you can see on the documentation, you can still pass v1 to the create view command on the stacked_cli to use the ViewModelBuilder but we set StackedView as the default style because is the one we like more.

@ferrarafer awesome work but one problem I noticed is the millisecond white flicker or any background color behind the PageTransition or the fillColor shows before the transition execute, it's somehow hurts in the eye how to avoid that?

@Dionnie123 that is due to your surface color set in the theme.
You can change it to match the expected background or set it to transparent.


[sqlite3]: Shared library support for Linux

Basically #6122 but I have signed the CLA

Related issue: #6008

Hi @jumpinjackie, since the sqlite3 version has been updated, please resolve the file conflict.
Thanks.

Thanks for the PR!


Please add document about "CopyToClipboard"

I found out about the existence of CopyToClipboard through the issue below.
https://github.com/pynecone-io/pynecone/issues/372
I want to use the CopyToClipboard component, but nowhere in the official documentation does it cover how to use it.
Could you please add this component usage to the documentation?

Fortunately, I found the usage through the code.
pc.copy_to_clipboard(
    pc.icon(tag="copy",),
    text="This text will be copied!"
)

Maybe you can contribute to my documentation?
If I can contribute, do you have any documentation that I can refer to?

It's not builtin to the core library yet - but this would definitely be good to have. All the docs are in the pcweb repo - see here.

Since we have the event rx.copy_to_clipboard I believe we can close this.


(documentation) describe push API Methods

Good day.
Could you describe API methods https://github.com/HMS-Core/hms-cordova-plugin/tree/master/cordova-plugin-hms-push#3-cordova-sdk-api-methods--constant-definitions ?

Hello @expcapitaldev ;
You can find the API Reference from https://developer.huawei.com/consumer/en/doc/development/HMS-Plugin-References/functions-0000001050135733

thank you @aktug


Thanos v0.5 is not supported

What happened?
While deploying thanos v0.5.0 getting exception as
Error parsing commandline arguments: unknown long flag '--cluster.address'
thanos: error: unknown long flag '--cluster.address'
Did you expect to see some different?
Thanos sidecar should successfully injected and started
How to reproduce it (as minimally and precisely as possible):
Deploy using s3 bucket and thanos v0.5.0
Environment
EKS

Prometheus Operator version:
v0.30.1

Kubernetes version information:
1.12.7

Kubernetes cluster kind:
EKS

Manifests:

Spec:
          image:
            tag: v2.10.0
          thanos:
            baseImage: improbable/thanos
            version: v0.5.0
            peers: thanos-peers.monitoring.svc:10900
            objectStorageConfig:
              key: thanos.yaml
              name: thanos-objstore-config

thanos.yaml:
  type: s3
  config:
    bucket: niki-cortex
    region: ap-south-1

Prometheus Operator Logs:

ts=2019-06-10T04:22:57.880300257Z caller=main.go:169 msg="Starting Prometheus Operator version '0.30.0'."
 This should be fixed on master already.

I was having the same issue - it does appear to be fixed in the master image.
Is there a release planned or some regular release cycle? I don't mind checking back, but if I know about when to start looking it's always helpful.

we don't have a set schedule, but I think we've finished all the features we've wanted to for the next release so we should be good to go

I was just curious anyway - thanks for all the work you do on this.

+1 on creating a release for Thano 0.5.0 support

+1 on the new release, please. We are trying to fix several issues with our current deployment.

@brancz the latest release, 0.31, does it offer support for Thanos 0.5.0? I read the note about removing gossip but I didn't read anything about thanos 0.5.0 although gossip is definitely removed on 0.5.0.
Thank you.

Yes you can use v0.5.0 with it (in fact that's what the tests do). Gossip was already possible to not be used in 0.4.0 though.

This has now been released. Closing.


Setting host header causes a python casting error

I get a TypeError when trying to request a URL and passing a Host header to it. I tried with quotes and without, but nothing works.
See the following example where prefixing m. to the hostname causes an error:
$ http -v http://m.youtube.com Host:m.youtube.com

http: error: TypeError: Cannot mix str and non-str arguments

Compared to:
$ http -v http://m.youtube.com Host:youtube.com
GET / HTTP/1.1
Accept: */*
Accept-Encoding: gzip, deflate
Connection: keep-alive
Host: youtube.com
User-Agent: HTTPie/0.9.6

HTTP/1.1 301 Moved Permanently
Content-Length: 0
Content-Type: text/html
Date: Thu, 22 Sep 2016 12:53:56 GMT
Location: http://www.youtube.com/
Server: YouTubeFrontEnd
X-Frame-Options: SAMEORIGIN
X-XSS-Protection: 1; mode=block

The same request with curl works properly so definitely looks like something's off on httpie.

This may not be a bug in httpie, rather a bug in the underlying requests framework.
There appears to be an incorrect assumption made in requests.cookies.MockRequest, see here:
class MockRequest(object):
    ...
    def get_full_url(self):
        # Only return the response's URL if the user hadn't set the Host
        # header
        if not self._r.headers.get('Host'):
            return self._r.url
        # If they did set it, retrieve it and reconstruct the expected domain
        host = self._r.headers['Host']
        parsed = urlparse(self._r.url)
        # Reconstruct the URL as we expect it
        return urlunparse([
            parsed.scheme, host, parsed.path, parsed.params, parsed.query,
            parsed.fragment
        ])
    ...

In Python2 this works fine where str and unicode objects are allowed interchangably in urlunparse.
In Python3 this is an issue because host = self._r.headers['Host'] may return a bytestring and         parsed = urlparse(self._r.url)  returns a ParseResult with str objects. Combining these two causes urlunparse to throw a fit.

This issue has been resolved in the requests library.
https://github.com/kennethreitz/requests/issues/3597

Thanks @bbamsch !

http://andisatya.atlassian.net/www.youtube.com

andisatya.slack.com


XSD Schema Unbounded Causes All elements to be treated as a 1...many cardinality

I was doing some work around pulling data out of mediainfo using the xsd and noticed an issue.
The xsd for trackType allows unbounded occurrence.
<xsd:complexType name="trackType">     <xsd:sequence>         <xsd:choice maxOccurs="unbounded">
And then each element specifies it only occurs once like the below
<xsd:element name="FrameRate" minOccurs="0" maxOccurs="1" type="xsd:float"/>
The xsd b/c of the maxOccurs = unbounded, it results in each element being a list and allows something like the below to validate. Was this your intention or is this a bug in the schema?  Currentyl, 2 FrameRate elements will return valid.
<OverallBitRate_Mode>VBR</OverallBitRate_Mode>             <OverallBitRate>188473606</OverallBitRate>             <FrameRate>29.970</FrameRate>             <FrameRate>29.970</FrameRate>             <FrameCount>1110</FrameCount>             <StreamSize>1048580</StreamSize>

@jsciame please confirm that https://github.com/MediaArea/MediaAreaXml/blob/24c9b2e25ae1fee90ed18e960222c9bfb2ad95fa/mediainfo.xsd is as you expect.

@JeromeMartinez This looks better from our parsing library. All of the values are single values and the duplicate FrameRate field now fails validation.


Create source timestamp fields.

This is used to indicate when the source last changed, for latency
considerations and because CreateTime and UpdateTime are grafeas
specific.

Also added the proto for v1beta1 since this isn't a v1 only feature.


Issues with Bootstrap 3

I'm putting the pie chart in a Bootstrap panel. The pie chart that works ok outside, in the panel here is hiding the chart colors, while when hovering the colors will appear.

My configuration is the following:
var pie = new d3pie(id, {
            "header": {
                "title": {
                    "text": "Distribution By Genre",
                    "fontSize": 20,
                    "font": "open sans"
                },
                "subtitle": {
                    "text": "Artist distribution by genre type",
                    "color": "#999999",
                    "fontSize": 11,
                    "font": "open sans"
                },
                "titleSubtitlePadding": 9
            },
            "footer": {
                "color": "#999999",
                "fontSize": 10,
                "font": "open sans",
                "location": "bottom-left"
            },
            "size": {
                "canvasWidth": 600,
                "pieOuterRadius": "90%"
            },
            "data": {
                "sortOrder": "value-desc",
                "content": data
            },
            "labels": {
                "outer": {
                    "pieDistance": 32
                },
                "inner": {
                    "hideWhenLessThanPercentage": 3
                },
                "mainLabel": {
                    "fontSize": 11
                },
                "percentage": {
                    "color": "#ffffff",
                    "decimalPlaces": 0
                },
                "value": {
                    "color": "#adadad",
                    "fontSize": 11
                },
                "lines": {
                    "enabled": true
                },
                "truncation": {
                    "enabled": true
                }
            },
            "effects": {
                "pullOutSegmentOnClick": {
                    "effect": "linear",
                    "speed": 400,
                    "size": 8
                }
            },
            "misc": {
                "gradient": {
                    "enabled": true,
                    "percentage": 100
                }
            }
        });

What I see from the console is the properties fill, stroke on the arc are missing in the white blocks:

while it is correctly (I assume) present in the colored arcs

As soon as I hover that arc, the property will change and filled properly:

Closing since I think it was due to a override of the svg property
path {
 fill: none
}


Add option to nyaml2nxdl to add space to self-closing tags

Also add short options

Thank your for the changes, @PeterC-DLS. Flipping the logic looks good too. I'll leave the final review to @RubelMozumder and @sanbrock before the merge.
I was thinking that we could just not have such a flag and just add the space in self-closing tags as that seems to be the norm usually. But it really is a choice so I leave it up to the three of you to decide what is the best way to proceed.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 13 (100.0%)  changed or added relevant lines in 2 files are covered.
252 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.5%) to 49.518%

Files with Coverage Reduction
New Missed Lines
%

tests/dataconverter/test_writer.py
1
96.77%

pynxtools/dataconverter/readers/transmission/reader.py
2
97.67%

pynxtools/dataconverter/readers/xps/reader.py
13
91.75%

pynxtools/nexus/nexus.py
93
73.38%

pynxtools/dataconverter/readers/xps/reader_utils.py
143
58.85%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
5395

Relevant Lines:
10895

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 13 (100.0%)  changed or added relevant lines in 2 files are covered.
252 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.5%) to 49.518%

Files with Coverage Reduction
New Missed Lines
%

tests/dataconverter/test_writer.py
1
96.77%

pynxtools/dataconverter/readers/transmission/reader.py
2
97.67%

pynxtools/dataconverter/readers/xps/reader.py
13
91.75%

pynxtools/nexus/nexus.py
93
73.38%

pynxtools/dataconverter/readers/xps/reader_utils.py
143
58.85%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
5395

Relevant Lines:
10895

💛  - Coveralls

It is a matter of choice to have space before the self-closing tag. Now, I assume that with the '-s' option no space before the closing tag, otherwise a default. So, the choice is up to the users on how they want to see XML files.
I would say this makes sense, right?
So, my flag is ready to go to the master branch. Thanks to @sherjeelshabih for keeping me in conversation.

the code has already been migrated to a for of nexus_definitions (FAIRmat-NFDI:nyaml2nxdl-migration). Can you please migrate this PR there?


Added PeerManagement.Timeout config option

Which problem is this PR solving?
This PR adds a new config option called PeerManagement.Timeout which will be used when communicating with peer discovery systems implementing the Peers interface. This PR also add support for context.Context when first connecting to the Peers implementation.
Short description of the changes

NewPeers() now accepts a context.Context
Moved NewPeers() initialization after logging setup to take advantage of logs during any initial connect attempts.
Added //go:build all || race which is the new 1.18+ format for build tags (I can revert if not needed, my IDE auto fixed it 😄 )
Fixed minor memory leak in Router where the cancel() function is never called.

Note
I'm noting here that context.Context passed to NewPeer() is not as useful to redis because the redis dial function already has a retry and timeout loop. However, it will be useful for other concrete implementations of the Peers interface. If this PR is accepted, I wish to follow up with a PR that uses https://github.com/hashicorp/memberlist for peer discovery.

This will also need a rebase.

This will also need a rebase to resolve conflicts.

Done.

@Baliedge please could you update the log to use WithFields as @kentquirk suggestion too?

Also done.


Add CosmosDB Service isn't fully implemented

Version
<IP_ADDRESS>327
Describe the bug
Adding a cosmosdb resource must not be coded, because

it doesn't seem to think I have any cosmosdb resources
it has undefined in the text of the UI

To Reproduce
Steps to reproduce the behavior:

click on add cosmosdb provider

Expected behavior
To be able to enumerate and pick the cosmosdb resource

to select from a alphabetically sorted list of tenant names and have that selection be sticky
to select from a alphabetically sorted list of subscription names and have that selection be sticky
to select from a alphabetically sorted list of resource group names and have that selection be sticky
to select from a alphabetically sorted list of cosmosDB storage accounts

Screenshots

Additional context
Add any other context about the problem here.
[bug]

I'm not able to repro this. I believe the issue might be with multi-tenant accounts where the DBs reside on a tenant other than the default. The Emulator does not yet have support for multi-tenant accounts.


Add French translation

Hi, I have translated the latest version of hortusfox-web in French.
I tried to translate literally but I replaced the word "Upload" for the button to add new photos by "Ajouter", which means "Add". Download and upload are translated by the same French word "télécharger", but it almost always means download.
I also translated the confirmation dialogs by something along the lines of "Do you confirm the deletion of ...?" to keep it gender-neutral in French.
I noticed while translating that the key last_authored_plants appears twice in the array of the app.php file.

Hi, thank you for your PR. I'll review this asap and if everything is fine I'll include it in the next release!


Objective C & Swift Version with StoryBoard

Just updated MainViewController and DrawerTableViewController (both programmed in Swift) to objectiveC version (MainViewControllerObjc and Menu), this way will be easier to implement on Objective-c Projects, with storyboard support.

@danrosero Hi! Thanks! :+1: