Git / version control plan for Qiskit Patterns

This is part of https://github.com/Qiskit/documentation/issues/524. This issue is to centralize our plan for how to make changes in Git without them going live to production.
Goals

Avoid changes from going live until they everything is ready. They must be launched together.
Make it easy to preview patterns for the content team.
Keep it simple to maintain the work in progress.

Avoid duplicating pages when possible because it's hard for us to keep the duplicates in sync.
Reduce merge conflicts.
Ensure the WIP docs are using all of our new improvements and tooling.

Proposal: un-synced folders on main
The closed source app explicitly lists which folders to sync. For example, it does not sync over our folder translations/ja. That allows us to develop experimental content in open source without it going live to production.
{
  "commit": "2d76e799751d8fa6b763171f75875f20d6f5ddca",
  "files": [
    "docs/support.mdx"
  ],
  "folders": [
    "docs/api/qiskit",
    "docs/api/qiskit-ibm-provider",
    "docs/api/qiskit-ibm-runtime",
    "docs/api/migration-guides",
    "docs/build",
    "docs/run",
    "docs/start",
    "docs/transpile",
    "docs/verify",
    "public/images/api",
    "public/images/build",
    "public/images/migration",
    "public/images/optimize",
    "public/images/qiskit-ibm-runtime",
    "public/images/run",
    "public/images/start",
    "public/images/transpile",
    "public/images/verify",
    "public/videos",
    "public/api"
  ]
}

So, we can safely create new folders like docs/analyze and docs/start-new. Only once everything is ready to go live, we'll rename every folder to its final location and update the sync config to copy over the files.
Don't copy over files without changes till the end
For certain folders like start that will stay the same, we should avoid copying over files to start-new that will still the same. That would make it confusing to have duplicated content we need to keep in sync. Instead, start-new should only have the new content.
If we are concerned that start-new's _toc.json wouldn't show the final state, then we can simply have start-new/_toc.json include entries from start/ in it.
Edge case: optimize folder
This folder is only going to have the folder/files renamed from transpile. We don't want to duplicate the files.
We could create the new optimize folder with just _toc.json and have it point the URLs in the existing transpile folder. Then, once we're ready to go live, we remove the transpile folder to only have optimize.
Infrastructure changes: feature flags
We also need to change some of the infrastructure like the top nav bar and the home page. Those can only go live at the same time that we sync over the new content. Likewise, we need redirects.
We want to review and ideally review that code in advance, though, to avoid a last-minute crunch.
So, we can use feature flags. In the docs app, we will have code for both the old header and new one, and we control which to use through something like an env var or a boolean in the code. In testing, we can enable the new UI, but prod will use the original. Then, once we deploy, we flip the switch.
We don't remove the old code until a few days after the launch, in case we need to easily revert back to the old UI.
We used this approach for qiskit.org redirects and it worked great.
We'll also use a PR that is open indefinitely that uses the feature flag so that people can easily see what the new UI looks like.

We're going to try this out for now and will adjust if it's not working well.


About annotation files

Hello, did you use coco2017_train.pkl or coco2017_train.json in the experiment? I found that using gen_noisy_cococ.py is to generate coco2017_train.pkl
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x80 in position 0: invalid start byte
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0
The above problem occurred when I used coco2017_train.pkl as the training annotation file

We did use coco2017_train.pkl to train clean-FasterRCNN and clean-RetinaNet, and this file works fine.
We have not encountered this error before. Perhaps you were training the model in Windows system? If so, you may refer to this issue.

We did use coco2017_train.pkl to train clean-FasterRCNN and clean-RetinaNet, and this file works fine.
We have not encountered this error before. Perhaps you were training the model in Windows system? If so, you may refer to this issue.

I am running on Linux system, can you tell me the version of pycocotools used? I guess it has something to do with writing a function. Also please tell me the version of pytorch python if it is convenient.


Wwise plugins support - add the possibility to build a Wwise Effect plugin

Wwise Source plugins allow synthesis, analysis of the audio input, etc., but can't affect any audio signal coming from Wwise; Wwise Effect plugins are designed for this and it would be nice to be able to generate both types of Wwise plugins with Heavy.
Heavy-Wwise Effect plugins would support:

[reveive]
[receive~]
[send]: this feature would work as the Wwise Meter and send the parameters to RTPCs
[send~]

@martinbussyparis Wwise Effect Plugins are coming, we've got it working just need to roll it out on the site. The way it would work is

[receive @hv_param 0 1 0.5] would be your RTPC input to the plugin
[adc~] audio rate input
[dac~] audio rate output

It should be possible to send messages out of the plugin but I haven't integrated that into the Wwise wrapper yet.

Yes, [adc~] and [dac~] of course... sorry, this is what I meant.
Good to know it is almost there :)


Added ReadMe in English version

A work in process on ReadMe in English version.

Will be up again when it is ready.


MLDBFB-636 fix join dataset rowhash to work like rowName and fix orde…

…r by a rowhash of a sub dataset

Minor things.  +1 once they are addressed.


[vituum + twig] project build directory

When building the project, the files are collected in the public folder, while the build.outDir parameter is ignored.
vite.config.js:
import { defineConfig } from 'vituum';
import twig from '@vituum/twig';

export default defineConfig({
  integrations: [twig()],
  templates: {
    format: 'twig',
  },
  build: {
    outDir: '../dist',
  },
});

The build.rollupOptions.output option also doesn't work.
vite.config.js:
// ...
export default defineConfig({
  // ...
  build: {
    // ...
    rollupOptions: {
      output: {
        entryFileNames: `assets/js/[name].js`,
        chunkFileNames: `assets/js/[name].js`,
        assetFileNames: `assets/[name].[ext]`
      }
    }
  }
});

The test was carried out both on a local Linux machine and in a stackblitz environment.

You should use output option https://vituum.dev/config/main-options.html#output
And Vite options should go into vite option. But I agree this might be confusing and I might consider change this in the future that vituum options would go under vituum and vite options would go directly as config.

By default public dir is same as output dir in Vituum, as for the build options, you should add these in to vite option. Same goes to publicDir option.
See https://vituum.dev/config/

I probably misunderstood the purpose of vituum...
I wanted to use it as a tool for layout of sites, with subsequent integration of layout into the CMS, but as I see it, this will not work without rewriting the logic of this tool. It is also not clear how the javascript code is split into chunks, I have two library imports, and everything is placed in one chunk with the name of the last import, and when splitting into chunks is disabled, the assembly breaks.
Thank you for your work!

Yeah I have a similar goal. I want to move away from Gulp.
Vituum seemed like a good choice since it support Twig template which we use.
Currently, we use Gulp to generate a static version of a website using Twig template, before it's integrated in the chosen CMS templating language (Wordpress, Typo3, Drupal, etc...).
I'm getting close to what I want to do, but I'm facing one last issue. When using preserveModules: true, Vituum seems to generate .js file for my css and twig template and I don't know why ! Here is my config:
export default defineConfig({
    build: {
        manifest: true,
        modulePreload: false,
        rollupOptions: {
            input: [
                './src/templates/emails/*.{json,latte,twig,liquid,njk,hbs,pug,html}',
                './src/templates/pages/**/*.{json,latte,twig,liquid,njk,hbs,pug,html}',
                '!./src/templates/pages/**/*.{latte,twig,liquid,njk,hbs,pug,html}.json'
            ],
            preserveEntrySignatures: "exports-only",
            output: {
                dir: 'build',
                preserveModules: true,
                entryFileNames: `[name].js`,
                chunkFileNames: `[name].js`,
                assetFileNames: `[name].[ext]`
            },
        }
    },
    plugins: [
        vituum({
            pages: {
                normalizeBasePath: true,
                dir: './src/templates/pages',
            },
            imports: {
                filenamePattern: {
                    '+.css': [],
                    '+.scss': 'src/styles'
                }
            },
        }),
        twig({
            root: './src/templates',
            dir: './src/templates/pages',
            base: './src/templates',
            functions: customFunctions,
        }),
    ]
});

And here are the 2 files that I don't understand:

I was able to fix the app.scss.js, weird issue.
My app.scss was empty while testing.
When adding css in it, it render app.css correctly.
Now, I only need to find why I get a templates/pages/index.twig.html2.js


Feature Request: URI friendly Web API handlers

On the server side, I think Marlin is good at implementing Web APIs.
When adding SiteHandlers, maybe we can take some resource patterns into consideration.
Something like this:
    SiteHandler *handler = new UserListHandler();
    site->SetHandler(HTTPCommand::http_get, CString("user"), handler);
    handler = new UserAddHandler();
    site->SetHandler(HTTPCommand::http_post, CString("user"), handler);
    handler = new UserEditHandler();
    site->SetHandler(HTTPCommand::http_put, CString("user/{id}"), handler);

When handling the request, we pick the corresponding handler via METHOD and resource pattern.
I think that means a lot of work to do, what do you think?

That is a VERY good idea. (And not a lot of work as well). In fact in my production servers
(in the commercial applications that use the Marlin framework) I have implemented just that.
The handlers there (e.g. the GET) decode a combination of "//" and then accordingly find a session object that was registrated for that user.
But the handler could decode the absolute path element of the URI (here "user" and "user" / id) before calling the general "HandleMessage"
The URI can be parsed to e.g. a vector of strings.
Question:
Are you not implying that there could be multiple handlers per METHOD per uri, as in:
   SiteHandler *handler1 = new UserListHandler();
   SiteHandler* handler2 = new GroupListHandler();
   site->SetHandler(HTTPCommand::http_get, CString("user"), handler1);
   site->SetHandler(HTTPCommand::http_get, CString("group"), handler2);

??

No, a pair of METHOD and resource prefix defines one handler only.
I want to implement API using the generic handler in the beginning,
but it'll be an elegant, instinctive way if Marlin has some builtin mechanism to support Web API development.
Maybe we can call it the Marlin's routing engine:-)

let every SiteHandler provides a regexp?
site can use it to pick handler,
and handler itself use it to parse request.

OK, Here is a design idea:

The FindHTTPSite() from the server finds the longest matching HTTPSite or subsite from an URI. This is done via the 'longest-match' algorithm by scanning backwards through the URL
Routing is only handy if the site handler receives by some means a parameter from the part NOT in the matched URI. E.g. if the "user" site receives the "{id}" so that it does not have to parse it for itself
Routing information could be stored by the "FindHTTPSite()" method and discarded on a non-match.
Routing information can be stored in the HTTPMessage and retreived afterwards in parts as in a "GetRoute(int index)" method. It should be read only of course.
Routing information should be propagated to at least the SOAPMessage and maby to the JSONMessage in order to support those frameworks.
Handlers for sites and sub-sites alike would receive the overflow of the URI in the from of routing parameters, which the can re-use for their own purpose.
Parsing route information in this way is time linear. No regular expression matching needed (which is non-linear)
Routing than can be enhanced later to register multiple handlers on the same site or subsite. Multiple handlers are now executed on a stack like manner. Routing could decide (non-stack-like) which handler to pick. This is not in conflict with handlers on sub-sites under the site.

Would this be sufficient in your eyes?

Will implement this right after the service framework is done.

Nice.
Thanks.
can't wait to use it.


Diag index changes

Fixing some bugs, and everything appears to work now!  Just need to do some more rigorous testing


Add diff id to PostbuildSummary

Add diff id to Differential
Report diff id in post build summary
Correct unit test for revision id

It would be nice to provide differential id in Differential class. The
diff id is extracted from raw conduit json in the same way as revision
id, although it is also available from Jenkins params DIFF_ID.
Mean while, the differential test is updated to distinguish diff id
from revision id.

@jjx @sectioneight would you help me merge the pr? Github think I am not authorized to do so.


Fix/wait for nav element

I may have a isolated experience but the script wasn't working for me and I quickly realised the <nav /> element was missing.
This fix adds an Observer waiting for the sub element to appear.
Update version to 0.2.1

Thanks for your PR! When I tested the code it didn't work though, which made me look harder. After some experimentation I got to version 0.2.1 I just pushed in https://github.com/mdenburger/userscript-identify-aws-console/commit/5c854fd5352d1a232505660b7067ab8643ed3434. That version also cleans up the mutation observers nicely.


Settings for readme in nupkg

Fixes #158
I added the needed settings in the project file

@DiDoHH your changes have been merged, thanks for your contribution :+1:

@all-contributors please add @DiDoHH for doc


sixth pharyngeal arch hypoplasia - amend def and placement

sixth pharyngeal arch hypoplasia MP:0013271
CURRENT DEF: underdevelopment or reduced size of the fourth pharyngeal arch, usually due to reduced cell number
AMENDED DEF: underdevelopment or reduced size of the sixth pharyngeal arch, usually due to reduced cell number
NOTE: sixth pharyngeal arch hypoplasia (MP:0013271) should be made a child of pharyngeal arch hypoplasia (MP:0003873)
Anna

Amended def and placement.
Also added:
small third pharyngeal arch MP:0030133
small fourth pharyngeal arch MP:0030134
small sixth pharyngeal arch MP:0030135


Line numbers not lining up with code block

I am using Slackware Linux to develop a website.  With shebang code snippets, I am seeing the line numbers column appear as it should, but the numbers are vertically misaligned with the code on the right, being spaced closer together than the displayed code.  I have viewed the site in both Firefox and Chromium with the same results. See the attached screenshot for an example; this 41 line block of code has the line numbers ending about an inch higher than code on the screen.  I am not sure where the problem is, but another theme I've tested (Flex) does not have this issue, so I suspect it is now pygments that is the cause.  Please test to see if this is reproducible

.  Thanks!


WebUI 2.0.0 Status

Milestone
2.0.0
I want to get base level parity with 1.0.0 before officially deprecating it and pulling this out of beta
Working

Section
Functionality

Log
View real-time logsSearch in the logs (by task, item, keyword, ...)

History
View latest accepted historySearch by task

Pending List
Approve and Reject entries in a listCreate a new Pending List

Miscellaneous
Shutdown FlexgetReload config

Features/Plugins

[ ] Pending List (#3) (PRs: #19, #44)
[ ] Scheduler (#4)
[ ] Movies (#5)
[ ] Seen (#6)
[ ] Task (#9)
[ ] Series (#10) [WIP]
[ ] Database Operations (#11)
[ ] Config Editor v1
[x] Log
[x] History

Bug Fixes/Smaller issues

[x] Base URL (#24) (Fixed in #43 & Flexget/Flexget#2518 )
[ ] UI Polish

Awesome work! In order to benefit casual and lazy (like me) users the end game should be to generate and update configuration file from UI. Check my old mockups from: https://flexget.mybalsamiq.com/projects/webui/Series - Set up

@paranoidi Yeah absolutely agreed.  That's the follow up to the stuff in this milestone along with how to properly support custom web plugins and configurable views and etc as well.


v2: enable / disable swipe open the side menu

moving the PR from ionic2 repo to ionic 2.0 branch:

https://github.com/driftyco/ionic2/pull/784
https://github.com/driftyco/ionic2/pull/785

PR here: https://github.com/driftyco/ionic/pull/4990
this new PR implementation is simplier and allows overriding the swipe open option with API too

merged


Show array and object settings in configuration UI as unsupported instead of hiding them entirely

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2477
Originally created by @DouweM on 2020-12-08 23:53:34

We can do better than https://gitlab.com/meltano/meltano/-/commit/644b79a95fa4649b2737cdcaf82db6c55f691f49.

View 1 previous comment from the original issue on GitLab


Converting assert statements to separate unit tests?

grep -r src/ -e assert gives:
src/poliastro/frames/fixed.py:            assert type(equatorial_frame) == HCRS
src/poliastro/frames/fixed.py:            assert fixed_coo.body == equatorial_frame.body
src/poliastro/frames/fixed.py:            assert type(equatorial_coo) == HCRS
src/poliastro/frames/fixed.py:            assert equatorial_coo.body == fixed_frame.body
src/poliastro/twobody/propagation.py:    assert np.abs(f * gdot - fdot * g - 1) < 1e-5  # Fixed tolerance
src/poliastro/twobody/orbit.py:        assert np.any(r.value), "Position vector must be non zero"
src/poliastro/core/propagation/farnocchia.py:    assert abs(x) < 1
src/poliastro/core/propagation/farnocchia.py:    assert abs(x) < 1
src/poliastro/core/propagation/farnocchia.py:    assert abs(x) < 1
src/poliastro/core/propagation/farnocchia.py:    assert abs(x) < 1
src/poliastro/core/propagation/farnocchia.py:    assert abs(x) < 1
src/poliastro/core/propagation/farnocchia.py:    assert -np.pi <= nu < np.pi
src/poliastro/core/iod.py:    assert tof > 0
src/poliastro/core/iod.py:    assert k > 0
src/poliastro/core/iod.py:    assert abs(ll) < 1
src/poliastro/core/iod.py:    assert T > 0  # Mistake on original paper

Would it be a good choice to remove the above assert statements and instead convert them into their own unit tests?
For example, I had tried to perform a lambert maneuver where the final orbit's epoch was before the initial orbit's epoch where:
src/poliastro/core/iod.py:    assert tof > 0

was violated. I got an AssertionError with no error message. So maybe in such situations, it would be good to explicitly throw an error message? Also, would there be any reason to keep assert statements in src sometimes?
Thanks!

Yeah, good point @Yash-10. assert in the code is a lazy error handling mechanism, and a bad one. If the assert is False, an error without message is raised.
At the same time, some of those assertions could just go away, since the code is supposed to give that result. Therefore, we would need some judgement to understand which assert statements to remove, and which ones to transform into a try ... except block.
One idea:

Replace all assert statements in poliastro.core with assert cond, msg (so at least the assertion gives a message if it's false)
Replace all remaining assert statements by an appropriate try ... except block

What do you think?


PHP Fatal error:  Uncaught Gitlab\Exception\RuntimeException: 404 Not Found

problem
When I build a merge request use the mr api, it give me some errors.

some warning

PHP Warning:  Yaf_Loader::autoload(): Failed opening script /data/web/mysite.com/releases/origin/hlq/newapp/src/application/library/Http/Client/Common/HttpMethodsClientInterface.php: No such file or directory in /data/web/mysite.com/releases/origin/hlq/newapp/vendor/m4tthumphrey/php-gitlab-api/lib/Gitlab/HttpClient/Plugin/HistoryTrait.php on line 10
PHP Warning:  Yaf_Loader::autoload(): Failed opening script /data/web/mysite.com/releases/origin/hlq/newapp/src/application/library/Symfony/Component/HttpClient/HttplugClient.php: No such file or directory in /data/web/mysite.com/releases/origin/hlq/newapp/vendor/php-http/discovery/src/ClassDiscovery.php on line 231
PHP Warning:  Yaf_Loader::autoload(): Failed opening script /data/web/mysite.com/releases/origin/hlq/newapp/src/application/library/Nyholm/Psr7/Factory/HttplugFactory.php: No such file or directory in /data/web/mysite.com/releases/origin/hlq/newapp/vendor/php-http/discovery/src/ClassDiscovery.php on line 231
PHP Warning:  Yaf_Loader::autoload(): Failed opening script /data/web/mysite.com/releases/origin/hlq/newapp/src/application/library/Nyholm/Psr7/Factory/HttplugFactory.php: No such file or directory in /data/web/mysite.com/releases/origin/hlq/newapp/vendor/php-http/discovery/src/ClassDiscovery.php on line 231
PHP Warning:  Yaf_Loader::autoload(): Failed opening script /data/web/mysite.com/releases/origin/hlq/newapp/src/application/library/Nyholm/Psr7/Factory/HttplugFactory.php: No such file or directory in /data/web/mysite.com/releases/origin/hlq/newapp/vendor/php-http/discovery/src/ClassDiscovery.php on line 231

A fatal error

PHP Fatal error:  Uncaught Gitlab\Exception\RuntimeException: 404 Not Found in /data/web/mysite.com/releases/origin/hlq/newapp/vendor/m4tthumphrey/php-gitlab-api/lib/Gitlab/HttpClient/Plugin/GitlabExceptionThrower.php:50
Stack trace:
#0 /data/web/mysite.com/releases/origin/hlq/newapp/vendor/php-http/httplug/src/Promise/HttpFulfilledPromise.php(34): Gitlab\HttpClient\Plugin\GitlabExceptionThrower->Gitlab\HttpClient\Plugin\{closure}(Object(GuzzleHttp\Psr7\Response))
#1 /data/web/mysite.com/releases/origin/hlq/newapp/vendor/m4tthumphrey/php-gitlab-api/lib/Gitlab/HttpClient/Plugin/GitlabExceptionThrower.php(54): Http\Client\Promise\HttpFulfilledPromise->then(Object(Closure))
#2 /data/web/mysite.com/releases/origin/hlq/newapp/vendor/php-http/client-common/src/Plugin/VersionBridgePlugin.php(19): Gitlab\HttpClient\Plugin\GitlabExceptionThrower->doHandleRequest(Object(GuzzleHttp\Psr7\Request), Object(Closure), Object(Closure))
#3 /data/web/pre2-api-growth.qkto in /data/web/mysite.com/releases/origin/hlq/newapp/vendor/m4tthumphrey/php-gitlab-api/lib/Gitlab/HttpClient/Plugin/GitlabExceptionThrower.php on line 50

What I did
I install the dependency library as the doc guide says:
composer require m4tthumphrey/php-gitlab-api php-http/guzzle6-adapter:^1.0

more try
I have try to install earlier version like below, but not work:
composer require m4tthumphrey/php-gitlab-api:9.11 php-http/guzzle6-adapter:^1.0

Yet I found the anwser:
In version 9.11 it lost some required dependency library, symfony/http-client and nyholm/psr7.
If you got the same problem, and installed this libaray yet,you need remove it first and then execute  command:
composer require m4tthumphrey/php-gitlab-api:9.11 php-http/guzzle6-adapter:^1.0 nyholm/psr7 symfony/http-client


Add -Wno-stringop-truncation to build flags on Linux.

Works around https://github.com/tensorflow/tensorflow/issues/39467 for gcc 10+ builds.

Thanks


1.2.1 Can't checkbox disable

checkbox disable dos't work
Consequently Ctrl+F2  dos't work

................
win10x64 21H2 | i5-4670K 4.5GHz | 16Гб DDR3 | MSI GTX1080ti GAMING X 11G || Oculus Quest2

rolled back to 1.2 and all works fine!

This is likely the issue caused by XRNeckSafer that other people have experienced.
You need to go to your registry at HKEY_LOCAL_MACHINE\Software\Khronos\OpenXR\1\ApiLayers\Implict then delete the Default value key.

Yes! I use XRNeckSafer !
Thanks a lot! I try it now!

Yes!  it works!!!

Fixed in 03c40a1b02d7877f64f791b325a9b8588da6b7e6.


switch to using NVTX annotate instead of range

Purpose
Switches to NVTX.@annotate, which applies at the definition, and makes sure we don't miss any precomputed quantities calls.
To-do

Content

[x] I have read and checked the items on the review checklist.

A bit clearer what is going on now:

bors r+


FreakyEntryHandler : PlatformView cannot be null

The Native Entry handler raises an issue with FreakyEntry, more information in the .Net Maui bug here:
https://github.com/dotnet/maui/issues/17165

fix for this is out with 0.4.5-pre.

fixed with the latest version v0.4.10


Seperate communication channels for controller and monitor

The game currently works by sending EVERYTHING through the same room. The player's device figures out if a signal was meant for him or not.
This could be more efficient, but it needs a good system. I could open one channel for the devices merely displaying the game, and one separate channel for the devicies merely controlling the game.
Problem 1: it needs some extra work to create two rooms per game.
Problem 2: the rooms need to be connected. Somehow, a signal in "AAAA-Monitor" needs to be able to trigger a change or signal in "AAAA-Controller".

So far, the speed has been blazing fast (even on a free server), so I don't know if two channels are actually needed ... It might just complicate things without leading to a clear benefit.

At third glance, YES we want this. Not only is it not as difficult as I thought (as I said, save the roomcode in the client, and just add "controller" or "monitor" to the end, dependnig on the case), it will also save a lot of bandwidth and create a more robust "game engine" in the long run. (Because I want to use this test project to build a proper, unique online game later on.)


504 error

$ rhc app create gistreveal nodejs-0.10 --from-code=http://github.com/ryanj/gis
t-reveal.it
Application Options
-------------------
Domain:                ttvvoo
Cartridges:            nodejs-0.10
Source Code:           http://github.com/ryanj/gist-reveal.it
Gear Size:             default
Scaling:               no

Creating application 'gistreveal' ... Server returned an unexpected error code: 504

FYI: The one-click button got 504 too.

the repo name was shortened to "gist-reveal" since the "." char isn't an allowed character for openshift V3 app names.
I think I fixed the link urls and README examples a while ago. They should all be up to date.
Please reopen if you find any other outdated urls.


Add generated docre documentation

I recreated PR #17 but from this repo instead of from my fork. This way @wokalski should be able to add things easier.
TODOs:

[ ] type handled, unhandled, t don't have a corresponding doc comment.
[ ] docre can't handle module type (cc @jaredly) but we can safely remove module type from our rei
[ ] There's no documentation for the Result module.
[ ] Poor wording and outdated information in the readme file.

This closes #16


Quest Gui not working 1.14.4

What is your Quests version / build number (do not say "latest")?
3.7.7
Describe the problem and how to replicate it. Any console errors to post using Pastebin.com?
Hi im new to this plugin and im trying to setup a quest it kinda goes like this, talk to npc with gui pop up. click the quest. kill 35 zombies and 5 creepers, talk to npc to finish. but the problem I have is when you go to the gui to begin its like an infinite item haul you can keep closing and opening the gui to take from im on 1.14.4 how can I fix this
A way to replicate this is by creating a quest such like the one described above and then adding a gui to an npc to start it.

Thanks for your report. I'm unable to replicate this issue. In my testing, taking the item closes the GUI and does not allow it to be re-opened.
Please try temporarily removing non-essential plugins from your server to make sure nothing is conflicting with Quests. If you still experience this issue, post your quests.yml via Pastebin/Hastebin.

It keeps happening

Again, please try temporarily removing non-essential plugins from your server to make sure nothing is conflicting with Quests. If you still experience this issue, post your quests.yml via Pastebin/Hastebin.
Sent with GitHawk

I was able to recreate the problem. My first attempt i left out extra mobs to kill and made everything extreamly short but somehow it worked... After that my entire plugin flatlined and i had to reinstall. This morning i recreated the quest ive had issues with.
This is what my Quest.yml looks like currently
https://pastebin.com/n1NqJtLF
This is what my Config looks like
https://pastebin.com/DEW6YK4P
I want to remove some plugins but most plugins are tied with my essentials for my server I'm not receiving any error codes either.

I managed to get an error code to appear
erver thread/ERROR Could not pass event InventoryClickEvent to Quests v3.7.7-b12
01.08 13:37:45 [Server] INFO org.bukkit.event.EventException: null
01.08 13:37:45 [Server] INFO at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:320) ~[spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:70) ~[spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:529) ~[spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:514) ~[spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO at net.minecraft.server.v1_14_R1.PlayerConnection.a(PlayerConnection.java:2133) ~[spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO at net.minecraft.server.v1_14_R1.PacketPlayInWindowClick.a(SourceFile:33) ~[spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO at net.minecraft.server.v1_14_R1.PacketPlayInWindowClick.a(SourceFile:10) ~[spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO at net.minecraft.server.v1_14_R1.PlayerConnectionUtils.lambda$0(PlayerConnectionUtils.java:19) ~[spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO at net.minecraft.server.v1_14_R1.TickTask.run(SourceFile:18) [spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO at net.minecraft.server.v1_14_R1.IAsyncTaskHandler.executeTask(SourceFile:144) [spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO at net.minecraft.server.v1_14_R1.IAsyncTaskHandlerReentrant.executeTask(SourceFile:23) [spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO at net.minecraft.server.v1_14_R1.IAsyncTaskHandler.executeNext(SourceFile:118) [spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO at net.minecraft.server.v1_14_R1.MinecraftServer.aX(MinecraftServer.java:908) [spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO at net.minecraft.server.v1_14_R1.MinecraftServer.executeNext(MinecraftServer.java:901) [spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO at net.minecraft.server.v1_14_R1.IAsyncTaskHandler.awaitTasks(SourceFile:127) [spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO at net.minecraft.server.v1_14_R1.MinecraftServer.sleepForTick(MinecraftServer.java:885) [spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO at net.minecraft.server.v1_14_R1.MinecraftServer.run(MinecraftServer.java:818) [spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO at java.lang.Thread.run(Thread.java:748) [?:1.8.0_201]
01.08 13:37:45 [Server] INFO Caused by: java.lang.NoSuchMethodError: com.sk89q.worldguard.protection.managers.RegionManager.getApplicableRegions(Lorg/bukkit/Location;)Lcom/sk89q/worldguard/protection/ApplicableRegionSet;
01.08 13:37:45 [Server] INFO at me.blackvein.quests.listeners.PlayerListener.onInventoryClickEvent(PlayerListener.java:147) ~[?:?]
01.08 13:37:45 [Server] INFO at sun.reflect.GeneratedMethodAccessor158.invoke(Unknown Source) ~[?:?]
01.08 13:37:45 [Server] INFO at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:1.8.0_201]
01.08 13:37:45 [Server] INFO at java.lang.reflect.Method.invoke(Method.java:498) ~[?:1.8.0_201]
01.08 13:37:45 [Server] INFO at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:316) ~[spigot_1.14.4.jar:git-Spigot-1981d55-31adfa7]
01.08 13:37:45 [Server] INFO ... 17 more

@Ratthew Thanks. What is your WorldGuard version?

7.0.0

Beta or release?
For example, there's 7.0.0 and 7.0.0-beta3 on BukkitDev

I think its release 7.0.0, I see nothing with beta

@Ratthew Use https://ci.codemc.org/job/PikaMug/job/Quests/18/
Warning: The build linked above is experimental. Make sure to back up your Quests folder before attempting to use it. Alternatively, wait until the final release on Spigot.


E2E Coverage for the E2E protractor cases.

We have huge E2E test cases with Protractor for our Angular App. Can anyone suggest how we can calculate the Coverage for these test cases?
P.S: They are separated from the code and are more into E2E and Selenium-based.
Please pardon if noob question

Hi there!
Your question is better suited for StackOverflow or Gitter. Please ask a question there with the 'protractor' tag or post in the Gitter Channel to get help.
From the the getting help section of the README:

Please ask usage and debugging questions on StackOverflow
(use the "protractor" tag) or in the Angular discussion group. (Please do not ask support questions here on Github.)

Thanks!


MAINT: Remove hack modifying a readonly attr

coeffs is clearly intended to be readonly. The previous code causes #7181.
This new version will work before and after that numpy change

This will fail travis until https://github.com/numpy/numpy/pull/8788 is merged

thanks, lgtm


feat: configure FTPS state

Checklist

[x] I've updated both the azurerm_linux_function_app and azurerm_windows_function_app resources.

Test passed 🎆
But i have a question for @equinor/terraform-baseline-maintainers, according to this FTPS we can just disable (as i have it by default) both FTP and FTPS if we are not using FTP deployment. But if we want enchanced security, we should allow FTP over TLS/SSL. Do you guys have any thoughs about this ?

Ran new test and it passed 👍🏼

Terraform test passed 👍🏼


Allow users to programmatically override default i18n strings

Implements the API set out in #47 to allow botogram users to override i18n messages without the need to recompile the .po files. This change may conflict with #85, but the PR looks pretty dead. This also contributes towards the completion of the #46 roadmap.
When the FrozenBot._ method is called it will first look up the msgid in the i18n_override map, before falling back to _lang_inst.gettext. This actually allows users to define custom messages stored in the bot instance and is independent from the language the bot is using.
While not strictly related to #47, .po files have been rebuilt from sources as they were missing some new messages and used outdated references.

6bbc17e should address the changes you requested. I was thinking of opening another PR to work on the narrative documentation since it fits with other changes proposed in #33. I was also hesitant about writing anything at all about i18n since #85 is still open.

Thank you for this pull request, I merged it!
About the narrative documentation, #85 isn't a blocker for that: the language_code thing can be added later when it's implemented. And thanks for considering writing it, more documentation is badly needed.

I think #47 should be closed and #46 updated to reflect the changes that have been merged.

Done.


Design patterns

I cleaned up the content of the add-dp branch into this new branch. I will delete the add-dp branch after this one is merged.
Now, before to merge, we need to decide what to do with the autogenerated names and definitions. Do we want to keep the existing ones or do we want to use the autogenerated ones instead.
I do not have strong opinion about that although I think that for the stress traits, we should go for the autogenerated ones.
I can change the yaml file according to what we decide
@cmungall @austinmeier @cooperl09

For TO I think we keep the existing ones for now, but we should do a comparison.
I do vote for removing the embedded IDs in the text defs
I agree about OOPS

I was not talking about oops but about the 'responsivity traits' (biotic and abiotic)
And I agree for the IDs in the text defs.

As we discussed today:

the labels and definitions are not created by the yaml file for the existing TO terms (at least, not for now)
added the more TOs to the TSV files
as discussed with @austinmeier. A dedicated pattern for the 'number of' would need to be created. The pattern would be something like that:

'has number of' and 'inheres in' some (_small part_ and 'part of' some _big part_)


feat: Setup PR check and post-merge build&push

Signed-off-by: Radim Hopp<EMAIL_ADDRESS>

Example of the PR check run is in my fork here: https://github.com/rhopp/e2e-tests/pull/4/checks
I wasn't able to verify the build&push phase, as it requires credentials to quay.io. We have them in here, but I don't want to add them to my fork to verify it. If it will not be working I'll provide followup PR.


Incorrect function folding

VSCode Version: Code 1.7.2 (7ba55c5860b152d999dda59393ca3ebeb1b5c85f, 2016-11-21T22:01:10.756Z)
OS Version: Darwin x64 16.1.0
Extensions:

Extension
Author
Version

vscode-eslint
dbaeumer
1.1.0

python
donjayamanne
0.3.21

debugger-for-chrome
msjsdiag
0.5.1

vscode-react-native
vsmobile
0.2.2

Steps to Reproduce:
http://ideone.com/WaSwWC

When you fold the first sample the code of second sample will get folded as well
If you make indent the same this issue will go away

@dimal97 I cannot access the linked document. Note that folding is indentation-based.

Thanks, than if folding is indentation-based there is no bug


fix(SelectMenu): multiple count error on page load

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Would you mind sharing some details of what you're trying to solve here?

I don't think this fix is correct but rather in the selected computed introduced in https://github.com/nuxt/ui/pull/1349 not taking the by key into account.

@benjamincanac While this does fix the problem in the linked issue, it doesn't resolve the underlying problem, which is that the selected computed property doesn't take the by prop into account.
I would approach it like this: https://github.com/nuxt/ui/pull/2228

@benjamincanac  When the page loads and there are selected values, the selected computed property doesn't correctly reflect those values, leading to an incorrect length.
However, I noticed that props.modelValue.length always returns the correct length of the selected items.

The filter condition isn't working on page load, and the selected computed property is empty.

The result:
const options = [
  { key: 'id', label: 'ID' },
  { key: 'title', label: 'Title' },
  { key: 'status', label: 'Status' }
]

const selected = ref([
  { key: 'id', label: 'ID' },
  { key: 'title', label: 'Title' }
])

@NasrALdaya I've already fixed it in my PR


show/hide new field based on other field value

Hello,
How can I (hide/show/destroy/create)  field2 based on the value of field1?
        {
          key: 'spoon',
          type: 'switch',
          templateOptions: {
            label: 'are you real?'
          }
        },
        /**--- show only if spoon is true ---**/
        {
          key: 'reason',
          type: 'textarea',
          templateOptions: {
            label: 'how come you are not real ?',
          }
        }
       /**--- show only if spoon is true ---**/

appreciate any help.

Just updated the docs with some examples: https://matt-sanders.gitbooks.io/vue-formly/content/v/2.0/how_to_use/conditional-rendering.html

thank you @matt-sanders 💯


Reimplement sort system: step 1

Support piDom checks.

From

to

I think I did my best. Idk why all lines in Meta.java are marked as partial hits.

bors r+


Kanban Board Setup

Kanban Board Setup 👍
Here we:

Created our Kanban board
Our team has 2 members:

Christian Ani
Leehaney George

Hi Team,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights 💯

Descriptive info ✔
Tasks distributed equally ✔

Optional suggestions
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


How to achieve full text search using entity framework 7

Hello,
Previously I was using EntityFramework 6 with MVC5. Now I am planning to migrate application to MVC6 with EntityFramework 7. Our application has full text search feature of SQL Server. Application need to fire full text search query to take advantage of full text search like link below:
https://msdn.microsoft.com/en-us/library/ms142583.aspx
With EntityFramework 6 we were managing generate full text search query using IDbCommandInterceptor. (Modifying query before it goes to SQL server) see below link:
http://www.entityframework.info/Home/FullTextSearch
Now, in EntityFramework 7, there is no more IDbCommandInterceptor exist, can anybody suggest me that how I can achieve full text search using EntityFramework 7?
Thank you,
Ashish

You can use FromSQL with a TVF that use the Fulltext search functions, as demonstrated here: https://github.com/rowanmiller/Demo-Temp/blob/master/FromSql/Program.cs


provide example configuration for a system-wide phpmyadmin container

A lot of users are familiar with using phpmyadmin to administer/examine their local databases.  There is a phpmyadmin docker image (https://hub.docker.com/r/phpmyadmin/phpmyadmin/) that we could base off.
Looks like there is a way to start it without specifying a server to connect to, but specify a connection via the UI - it might require a bit of docco, but would be good to provide guidance.

If you can provide a POC command to do the job I can make this work. Alternatively make it work and send me the output of docker container inspect amazeeio-phpmyadmin (or the container name you give it).
In the meantime, the following works but doesn't connect to a server, so this definitely needs work:
services:
  amazeeio-phpmyadmin:
    Name: amazeeio-phpmyadmin
    Disabled: false
    Config:
      Image: phpmyadmin/phpmyadmin
    HostConfig:
      PortBindings:
        80/tcp:
          -
            HostPort: 8080

networks:
  amazeeio-network:
    - amazeeio-haproxy
    - amazeeio-phpmyadmin

so - what I got to work was providing the env PMA_ARBITRARY=1 to phpmyadmin - how do we specify that in the config above?
on the login screen you then get the option to specify a server - you can use the bridge gateway IP address, and the port of the MySQL/mariadb image you need to connect to.

I've just updated my example to include:
      Env:
        - "PMA_ARBITRARY=1"

Testing shows it works as expected, let me know :)

perfect - It's awesome being able to do this!

Note that networks have changed given #88:
Networks:
  amazeeio-network:
    Config:
      IPAM:
        Config:
        - Gateway: <IP_ADDRESS>
          Subnet: <IP_ADDRESS>/24
    Containers:
    - amazeeio-haproxy
    - amazeeio-phpmyadmin
    Name: amazeeio-network


Documentation on how to run tests results in error

When following the instructions in ./docs/edgex-taf.md, at step 4, when running python3 -m TUC --exclude Skipped --include deploy-base-service -u deploy.robot -p default, the result is the following error:
INFO:root:Run testing for profile 'default'
Traceback (most recent call last):
File "/usr/lib/python3.6/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/usr/lib/python3.6/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/jim/.local/lib/python3.6/site-packages/TUC/main.py", line 10, in
trigger.start()
File "/home/jim/.local/lib/python3.6/site-packages/TUC/trigger.py", line 140, in start
run_robot(args)
File "/home/jim/.local/lib/python3.6/site-packages/TUC/trigger.py", line 153, in run_robot
setup_config(args)
File "/home/jim/.local/lib/python3.6/site-packages/TUC/trigger.py", line 102, in setup_config
constant = import("TAF.config.global_variables", fromlist=['global_variables'])
ModuleNotFoundError: No module named 'TAF'
For clarity sake, here is the env setup
jim@jim-ubuntu:~/forks/edgex-taf/TAF/utils/scripts/docker$ env |grep ARCH
ARCH=x86_64
jim@jim-ubuntu:~/forks/edgex-taf/TAF/utils/scripts/docker$ env |grep SECURITY
SECURITY_SERVICE_NEEDED=false
jim@jim-ubuntu:~/forks/edgex-taf/TAF/utils/scripts/docker$ env |grep USE_DB
USE_DB=redis
jim@jim-ubuntu:~/forks/edgex-taf/TAF/utils/scripts/docker$

Should run the command in the directory of edgex-taf.

thanks @jinlinGuan


[REQUEST] add games that are not automatically paired in the catalog

an interesting idea is to add games that are not automatically paired in the catalog. for example, put them in the "suggestions" and let the volunteers add links to repacks with their hands for free (links must be verified by the community)
-----------------next, just my thoughts, the main idea has already been expressed-----------------
You can become a volunteer via chat upon request. you can make a small button "suggest \ add a game" there to add the name of the game, a description from the store (steam or other sources) and a link to the repack. the volunteer checks the link and description, if the link is malicious (by the decision of the volunteer or by the decision of the community), then either refuse to add the game, or the volunteer himself is looking for a more reliable source. It is better to mark such games that they are added by the community.
or you can do without volunteers, and for example, once a week to collect all the offers, highlight them in a separate tab and people will vote for the game and the link to it.
As the solution to the problem of suggestion-mess, there may be included a simple basic account system for volunteers for stats and votes

all download links are the responsibility of the community.


Move geostlyer data (internalDataDef) to GeoStylerContext

:tada: This issue has been resolved in version 13.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


integrity checksum failed when using sha512

Please provide all the information requested. Issues that do not follow this format are likely to stall.
i am trying to install react native 0.62.2. this error is coming
Description
Please provide a clear and concise description of what the bug is. Include screenshots if needed.
Please test using the latest React Native release to make sure your issue has not already been fixed: https://reactnative.dev/docs/upgrading.html
React Native version:
Run react-native info in your terminal and copy the results here.
version--0.62.2
node 12.10.0
npm 6.10.0

Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.

npm install<EMAIL_ADDRESS>

Expected Results
Describe what you expected to happen.
Snack, code example, screenshot, or link to a repository:
Please provide a Snack (https://snack.expo.io/), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
You may provide a screenshot of the application if you think it is relevant to your bug report.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve

https://github.com/nodejs/node/issues/13635

None of the above given worked


Setup GitHub sponsoring?

At work we sponsor all our dependencies solely through GitHub bulk sponsoring.
Is it possible to setup GitHub sponsoring for caniuse?
I know this can be connected somehow to open collective, maybe a similar setup is also possible for patreon?

Sounds good, I hadn't heard of this! Will look into it, thanks for the suggestion :)

GitHub sponsoring has now been added, if there's some follow-up connection necessary let me know!

Everything seems setup perfectly from our end!
Thank you for this!


Support "scoped" middlewares

Feature Request
In practices, we often want enable some middlewares on the subtree of the routes.
Detailed Description

For example, we have URI /v1/:user_id/posts/:post_id, and we need to check whether the post belongs to that user. Or we have an OAuth2Middleware, but the only subset of our APIs need to be protected.
Currently, we need to intrusive modify the middleware if we want to archive this purpose. We have the maintain the router but also the middleware implementation.
Context

root.at("/:user_id/posts").nest(|parent| {
    parent.middleware(access_guard); // <----
    parent.post(crate::service::post::new_post);
    parent.at("/:post_id")
        .get(crate::service::post::get_post)
        .put(crate::service::post::mod_post)
        .delete(crate::service::post::del_post);
});

Possible Implementation

Add associated middlewares information to the NFA states of route-recognizer.(Global middlewares associated with the initial state??)
When perform the matching, we need to trace the path from the initial state to the terminal state. After the matching process, we can collect the associated middlewares on the path and invoke them in order.

Maybe need an API for clearing the parent's middleware. parent.clear_middleware()?

It was supported before, but removed during revamp. Original commit: https://github.com/rustasync/tide/commit/dfe72abe238ad7afd642c4b546596393bd1bdade

Is there any plan to bring this feature back?
Also, the article linked in the README is misleading, as it shows an example of the common use-case scenario of protecting a subset of routes: https://rustasync.github.io/team/2018/11/27/tide-middleware-evolution.html
Maybe a comment about it not being currently supported should be added.

@kamyuentse @cquintana-verbio I once implemented router nesting and per-endpoint middleware, and I definitely want to re-implement and get this back! I just need a clear design that can be documented or written in code.
A simple solution of per-endpoint middleware would be an endpoint wrapper. It would wrap an existing endpoint and apply middleware when invoked. Router nesting is rather tough; it should deal with partial matching and collecting match params.


Imported passwords does not sync with other devices

Feel free to merge with issue #11.
I successfully imported passwords from a csv I exported from Chrome into Firefox on my MacOS device. Unfortunately the logins does not sync with my iPhone. The only login credentials that I have access to via sync are those saved using Firefox.
Reconnecting my Firefox account on my iPhone does not help. This issue seems to be affecting lockwise as well as noted in #25.

I managed to get the passwords in sync.
what I did:

reinstall firefox (without setting up the firefox sync)
find out where my firefox profile directory is located
run:
ffpass import --from 'Chrome Passwords.csv' -d /home/wesley/.mozilla/firefox/pth337gw.default-release
sign in with my firefox sync account

Then everything started to appear on Lockwise as well.
good luck buddies! :pray:

Huzzah! That did the trick, thank you.

no need to reinstall or clear your profile or set up a new profile

i simply disconnected SYNC from firefox sync options,
imported the passwords,
changed one imported password (i don't know if this is needed)
reconnected SYNC
forced a "sync now"

and all imported passwords synced to other devices.
remember to force "sync now" on other devices to see them ASAP.

Even fewer steps. You just need to disconnect and reconnect the account, basically. Here's what I did

Import passwords
Disconnect sync account (about:preferences#sync)
Restart browser (maybe not needed)
Connect sync account


Add new api version 2024-06-01-preview for Microsoft.Cdn

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[x] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been created in adherence to OpenAPI specs PR creation guidance).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with ARM review (PR workflow diagram Step 2), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

The changes to the REST API definition are correcting the definition to accurately describe the service behavior, so bypass the review process by adding BreakingChange-Approved-BugFix.
1023 - TypeFormatChanged is introdued when the reviewer ask adding "format": "arm-id" in https://github.com/Azure/azure-rest-api-specs/pull/29369
1025 - RequiredStatusChange and 1034 - AddedRequiredProperty are reuired change for correcting the definition to accurately describe the service behavior in https://github.com/Azure/azure-rest-api-specs/pull/29369. The SecretVersion alway needed.
1026 - TypeChanged is to fix the definitions missing the "object" type definition.

@ZianWang02  Can you please merge your branch with the main. There seems to be some merge conflicts.

Can you please take a look at the swagger lintdiff errrors and fix those as well :
https://github.com/Azure/azure-rest-api-specs/pull/29588/checks?check_run_id=27496185108

Most of these errors are from prev swagger errors, for ex, {put-responses-codes, del-response-codes, errorcontent, name patterns} are from before this change and exists on following the existing service api pattern, which if not followed will fail to adhere to existing service behaviour or lead to over-hauling of the entire service apis.
The Post-response-errors are from changes not related to http-connect. I will let @ZianWang02  comment on those.

Can you please take a look at the swagger lintdiff errrors and fix those as well :
https://github.com/Azure/azure-rest-api-specs/pull/29588/checks?check_run_id=27496185108

Can you please take a look at the swagger lintdiff errrors and fix those as well :
https://github.com/Azure/azure-rest-api-specs/pull/29588/checks?check_run_id=27496185108

Hi @raosuhas, because we are following our previous swagger definition and copied from old versions, can we request for exceptions for PostResponseCodes, PutResponseCodes, DeleteResponseCodes, LroErrorContent Lintdiff errors?
@barathvg For ResourceNameRestriction for profileName, can we use the already defined parameter "#/parameters/profileNameParameter" so that the pattern will automatically apply to it?
@barathvg For ResourceNameRestriction for targetGroupName and tunnelPolicyName, if we don't want to add patterns to them, can you apply for an exception with @raosuhas?

Can you please take a look at the swagger lintdiff errrors and fix those as well :
https://github.com/Azure/azure-rest-api-specs/pull/29588/checks?check_run_id=27496185108

Hi @raosuhas, because we are following our previous swagger definition and copied from old versions, can we request for exceptions for PostResponseCodes, PutResponseCodes, DeleteResponseCodes, LroErrorContent Lintdiff errors? @barathvg For ResourceNameRestriction for profileName, targetGroupName and tunnelPolicyName, if we don't want to add patterns to them, can you apply for an exception with @raosuhas? Or can we use the already defined parameter "#/parameters/profileNameParameter" so that the pattern will automatically apply to it?

@ZianWang02 Thanks for pointing out the profileNameParameter that is already defined, I have fixed all the ResourceNameRestriction errors in my commit, please merge and proceed with exceptions. Thanks.

Can you please take a look at the swagger lintdiff errrors and fix those as well :
https://github.com/Azure/azure-rest-api-specs/pull/29588/checks?check_run_id=27496185108

Hi @raosuhas, because we are following our previous swagger definition and copied from old versions, can we request for exceptions for PostResponseCodes, PutResponseCodes, DeleteResponseCodes, LroErrorContent Lintdiff errors? @barathvg For ResourceNameRestriction for profileName, targetGroupName and tunnelPolicyName, if we don't want to add patterns to them, can you apply for an exception with @raosuhas? Or can we use the already defined parameter "#/parameters/profileNameParameter" so that the pattern will automatically apply to it?

@ZianWang02 Thanks for pointing out the profileNameParameter that is already defined, I have fixed all the ResourceNameRestriction errors in my commit, please merge and proceed with remaining exceptions as requested above. Thanks.

Thank you @barathvg, I have merged the PR. @raosuhas Could you please help take another look at this PR? Thanks!

/azp run

"DeploymentStages": {

Add also a description for this schema

Refers to: specification/cdn/resource-manager/Microsoft.Cdn/preview/2024-06-01-preview/afdx.json:7387 in 7091be1. [](commit_id = 7091be1d1ab485d190e31917632c7c123f2cf734, deletion_comment = False)

"DeploymentStageStatus": {

Add also a description for this schema

Refers to: specification/cdn/resource-manager/Microsoft.Cdn/preview/2024-06-01-preview/afdx.json:7402 in 7091be1. [](commit_id = 7091be1d1ab485d190e31917632c7c123f2cf734, deletion_comment = False)

      "default": 80,

The new capacityConsciousThreshold and distributionType properties will be part of the AFDOriginGroupUpdateParameters for a PATCH opeartion. You should only have patchable properties

Refers to: specification/cdn/resource-manager/Microsoft.Cdn/preview/2024-06-01-preview/afdx.json:6761 in 7091be1. [](commit_id = 7091be1d1ab485d190e31917632c7c123f2cf734, deletion_comment = False)

      "default": 80,

The new capacityConsciousThreshold and distributionType properties will be part of the AFDOriginGroupUpdateParameters for a PATCH opeartion. You should only have patchable properties
Refers to: specification/cdn/resource-manager/Microsoft.Cdn/preview/2024-06-01-preview/afdx.json:6761 in 7091be1. [](commit_id = 7091be1, deletion_comment = False)

Hi @razvanbadea-msft , I don't quite understand why we can't add capacityConsciousThreshold or distributionType properties.
These two properties will be added in all API requests of origin group (including LIST, GET, POST, PATCH, etc.). May I know what is the patchable properties? + @anupamsr because these properties are added by him.

Because both of them has default values; you can read more about this rule here https://github.com/Azure/azure-openapi-validator/blob/main/docs/patch-body-parameters-schema.md#why-the-rule-is-important

In reply to:<PHONE_NUMBER>

Because both of them has default values; you can read more about this rule here https://github.com/Azure/azure-openapi-validator/blob/main/docs/patch-body-parameters-schema.md#why-the-rule-is-important
In reply to:<PHONE_NUMBER>

Fixed, thank you!

For other swagger lintdiff errors check the suppression guide

I have added Approved-LintDiff tag according to the guide, because all the Lintdiff (PostResponseCodes, PutResponseCodes, DeleteResponseCodes, LroErrorContent, BodyTopLevelProperties) are introduced by previous version:

/RequestMerge

/pr RequestMerge

/pr RequestMerge


Revise console output, resolves #70

This PR revises the command line output, per #70.

:+1:

:+1: :smoking: :dragon_face:


[Bug]: After a day, the script hides the video description and the only way to get it back is to uninstall the script

Describe the bug
Title
Screenshots
No response
What browser are you using?
Microsoft Edge v116 on Windows 11 23H2 v23531
What version of AdashimaaTube are you using?
Latest, as I got it yesterday from userstyles.world

Edit: it gets working back randomly, then stops working again...

can you provide a screenshot?

Can you try to disable "Hide game section" option?

Same issue. I reset to default and tested all options, it appears to be "old watch page" causing the issue. It's only occurring for me in once channel, other channels under the same account work fine. Very frustrating.

Same here, the description is hidden if the script is on.
With AdashimaaTube:

Without:

@WolfRider3000 @minou2020 Please let me know if this works or not.

Install Violentmonkey or Tampermonkey extension.
Install revert-ui-changes.user.js

It seems to be working. Are you planning to make these changes part of AdashimaaTube or it's only possible with Tampermonkey?

It seems to be working. Are you planning to make these changes part of AdashimaaTube or it's only possible with Tampermonkey?

It's only possible with Tampermonkey


Create Response class/structure

Response class/struct contains response data, response type, and status code.

Done!


[BrokenLinksH2] fixing broken link

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

File
Status
Preview URL
Details

dynamics-nav/Deploying-Microsoft-Dynamics-NAV-Using-ClickOnce.md
:bulb:Suggestion

Details

dynamics-nav/Deploying-Microsoft-Dynamics-NAV-Using-ClickOnce.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@ShawnJackson odd error message here, not sure what to do with it.

@StephenBrentPeters - Closing and reopening the PR might clear it up.

Closing and reopening as suggested by Shawn

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

File
Status
Preview URL
Details

dynamics-nav/Deploying-Microsoft-Dynamics-NAV-Using-ClickOnce.md
:bulb:Suggestion

Details

dynamics-nav/Deploying-Microsoft-Dynamics-NAV-Using-ClickOnce.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Global effort to fix broken links
The Content & Learning team is fixing broken links on docs.microsoft.com for the rest of H2. This effort will eliminate potential accessibility, security, and usability issues. This PR includes only link fixes and does not change other content. Please review within five business days and merge, or comment in the PR with any changes you'd like to see. Thanks!


Using ABAP with the SAP API Business Hub

Tutorials: https://developers.sap.com/tutorials/hcp-abh-abap.html
The screenshots in the tutorial shows that you should download the api.github.com certificate. However, you need the https://sandbox.api.sap.com/ certificate instead.
Perhaps the screenshots of the tutorial should also be updated to the current API Business Hub UI.

This tutorial has been archived.


⚠ CrackingItaly has degraded performance

In 0acab85, CrackingItaly (https://crackingitaly.to) experienced degraded performance:

HTTP code: 200
Response time: 18604 ms

Resolved: CrackingItaly performance has improved in a6bb4b1.


New package: FiloSottile.mkcert version 1.4.4

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@wingetbot run

Please see #80613 (review)

It this really required? If so, shouldn't winget validate detect it instead of letting reviewer providing feedback? winget validate passes and successfully installs the package - this means winget actually has code to read .yml files.

It this really required?

Yes - it's required for the WinGet package repository.
WinGet will able to validate .YML files but the repository (as well as the Validation Pipelines) now focuses on .YAML files to ensure consistency between file types (as well as the capitalization of the file types).
winget-create / YamlCreate.ps1 will allow you to generate manifest files based on what the repository accepts.

Files renamed.
This requirement should be incorporated into the regular validation to save contributors/reviewers' time.

Close with reason: To be replaced with a new PR;

This requirement should be incorporated into the regular validation to save contributors/reviewers' time.

It's usually displayed in https://github.com/microsoft/winget-pkgs/pull/80613/checks?check_run_id=8684095056 whenever there's an issue with your pull request.
Before winget-create & YamlCreate.ps1 existed people were creating manifests by hand which created a lot of inconsistencies such as spelling mistakes, capitalization issues, etc.
Now that winget-create & YamlCreate.ps1 exists people should take advantage of one of the tools mentioned so you're less likely to have manifest submission issues as you're less prone to make mistakes.


Benchmark and optimize analyze call

During manual testing I realized that analyze can be a bit slow, depending on the contract. This might be due to a heavy ParsedWasm::parse or maybe even due to loading the whole Wasm from disk at once.
In https://github.com/CosmWasm/wasmd/issues/1813 we start exploring first steps towasds mass migrations. In order to ensure MsgMigrateContract is performant (dozens, maybe hundreds of migrations in a block), we also need to ensure that the Analyze call is fast.

[ ] Create benchmark for analyze for different contracts
[ ] Evaluate and implement performance optimizations

After adding some more of the testdata contracts to the benchmark harness and running it through cargo-flamegraph, it seems like it spends most if its time inside wasmparser::validator::func::FuncValidator.
In there it steps through the function body byte-by-byte and validates that it's well-formed. Makes sense that we have that step. Since that is part of Bytecode Alliance's wasmparser, I'm not sure how much we could actually optimize this on our side.

Some bench examples when removing the validation of the function body:
Cache/analyze_cyberpunk.wasm
                        time:   [168.58 µs 168.66 µs 168.80 µs]
                        change: [-83.658% -83.618% -83.580%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 1 outliers among 12 measurements (8.33%)
  1 (8.33%) high mild
Cache/analyze_cyberpunk_rust170.wasm
                        time:   [157.20 µs 157.61 µs 158.03 µs]
                        change: [-79.398% -79.357% -79.303%] (p = 0.00 < 0.05)
                        Performance has improved.
Cache/analyze_floaty.wasm
                        time:   [117.18 µs 117.26 µs 117.37 µs]
                        change: [-82.821% -82.791% -82.763%] (p = 0.00 < 0.05)
                        Performance has improved.
Cache/analyze_hackatom.wasm
                        time:   [124.91 µs 125.00 µs 125.11 µs]
                        change: [-83.251% -83.200% -83.159%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 1 outliers among 12 measurements (8.33%)
  1 (8.33%) high mild

As apparent from the benches, we see an ~80% improvement in performance across the board and are comfortably entering the low three-digit microsecond territory.

In there it steps through the function body byte-by-byte and validates that it's well-formed. Makes sense that we have that step. Since that is part of Bytecode Alliance's wasmparser, I'm not sure how much we could actually optimize this on our side.

Something we can do to improve this is pull out the function validation into a separate step. Then

In fn check_wasm we stop the execution on error (if any) early and only perform the function validation if everything else is ok
In fn analyze we don't need to do the function validation at all since the Wasm file was checked before and we don't use that extra data

Makes sense, will look into that.
I also managed to get some performance improvements (averaging at ~22%) by parallelising the function body validation (without reusing the allocations from the validator since that would add locking overhead)


[Feature request] add keyboard shortcut for exiting

Is your feature request related to a problem? Please describe.
On linux, APPs often use Control+Q or Control+W or ESC for exiting/closing.
I'm using a window manager i3wm.
Describe the solution you'd like
Add keyboard shortcuts configuration.
Describe alternatives you've considered
No response
Additional context
No response

Thanks for the issue.
Adding a separate setting will complicate the UI. So Control+Q will be the default on Linux.
Implemented in 9cd3286335a97e7f09ab202c1d7fcbf5debd8d61


Add OMIS pricing

This:

adds pricing fields to the order model
updates those values when an order or an assignee is saved
updates the order endpoints to return those values
updates the search to include those fields. Only the total_cost is indexed as it will be used for searching in the near future
updates the content of the quote to include the pricing values

Codecov Report

Merging #469 into develop will increase coverage by 0.07%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           develop    #469      +/-   ##
==========================================
+ Coverage    96.33%   96.4%   +0.07%
==========================================
  Files          108     110       +2
  Lines         3327    3394      +67
  Branches       301     311      +10
==========================================
+ Hits          3205    3272      +67
  Misses          84      84
  Partials        38      38

Impacted Files
Coverage Δ

datahub/omis/order/serializers.py
100% <ø> (ø)
:arrow_up:

datahub/omis/order/signal_receivers.py
100% <100%> (ø)

datahub/omis/quote/utils.py
100% <100%> (ø)
:arrow_up:

datahub/omis/order/pricing.py
100% <100%> (ø)

Continue to review full report at Codecov.

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


Feature/es7

I have this issue https://github.com/yarax/swagger-to-graphql/issues/12
after the refactor

Hey!
Thank you very much for a great job! Sorry for response delay
After building the fork I got the error:
/home/roman/sandbox/swagger-to-graphql/lib/index.js:111
  var _ref2 = _asyncToGenerator(regeneratorRuntime.mark(function _callee2(swaggerPath) {
                                ^

ReferenceError: regeneratorRuntime is not defined
    at /home/roman/sandbox/swagger-to-graphql/lib/index.js:111:33

we need to use babel-polyfill to make async await works
import 'babel-polyfill';

Also sadly that backward compatibility for src/index is broken now (it exported default object) but seems like it must to be done

feel free to improve this branch in another branch or in this one as well

Created https://github.com/yarax/swagger-to-graphql/tree/release-1.1 branch based on your fork which will also require last issues


add support for Node 13

Node 13 came out last month, with a NODE_MODULE_VERSION of 79 based on the releases list:

cherry-picked to master


Fix the resource types saved into the violations table.

Adding cla: yes label, as the PR author is a googler at the time of this PR was created.


[BUG] "when this sprite clicked" block does not work outside of stage size when using camera controls while in fullscreen

Mystic Island proto 7.zip

hastily made bug showcase.zip
this one is better

It is a known bug.
but there are two simple alternatives to a "solution".

... which are?

The first solution is based on the coordinates or distance of the mouse and then verify if the mouse is pressed.
The second is based on an object that follows the mouse and it is verified if that object is being touched and the mouse has been clicked.

here the alternatives.
I hope it helps something.
Proyecto.zip


Dedupe based on sha1 as well as on event ID

We need sha1 deduplication for jobs as the ID is the signature of an individual event and it's unique even for events that are copies of each other. So if the ID match it's guaranteed that sha1 match, so we don't need to compare ID if sha1 exists and matches.
cc @wikimedia/services

Coverage decreased (-0.8%) to 88.807% when pulling 0f97e5389fd0fa56645c578cdf81f23606ad5da0 on Pchelolo:dedupe_sha1 into e20fb17ef83d75460968d208f5f40f47ec4abfae on wikimedia:master.


Update prometheus-operator from new place

prometheus operator has moved to https://github.com/prometheus-community/helm-charts

Closing in favour of https://github.com/kinvolk/lokomotive/issues/745#issuecomment-690975542.


DataTree.identical should consider checking that coordinates are defined at the same level

What is your issue?
Consider the following two data trees:
one = DataTree.from_dict(
    {
        "/": xr.Dataset(coords={"a": 1}),
        "/b": xr.Dataset(coords={"a": 1}),
    }
)
two = DataTree.from_dict(
    {
        "/": xr.Dataset(coords={"a": 1}),
        "/b": xr.Dataset(),
    }
)

Currently, DataTree.identical() considers one and two to be identical, but the coordinate a is defined redundantly in the first tree.
For most purposes, these DataTree objects are identical, but are edge cases where it could be different, e.g., when wrting these trees to netCDF or Zarr.
It would be helpful if the .identical() checked for such discrepancies, e.g., to facilitate writing unit tests as in https://github.com/pydata/xarray/pull/9214.

Can this be closed after the changes in https://github.com/pydata/xarray/pull/9555? That's what I understood from this comment.

Yes, we can close. These trees are neither equal nor identical now.
On Thu, Oct 10, 2024 at 1:38 AM Tom Nicholas @.***>
wrote:

Can this be closed after the changes in #9555
https://github.com/pydata/xarray/pull/9555? That's what I understood
from this comment
https://github.com/pydata/xarray/pull/9588/files#r1791110146.
—
Reply to this email directly, view it on GitHub
https://github.com/pydata/xarray/issues/9215#issuecomment-2402806006,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJJFVQOMIX5S7UWTTVIY7TZ2VLXJAVCNFSM6AAAAABKPW2EHKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBSHAYDMMBQGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Browsers will not open http://localhost:8888/

UPDATE:
I trashed the previous installation and re-installed. When I did I got the error message

Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the canvas package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs canvas
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls canvas
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
/Users/bob/Applications/audiogram/npm-debug.log

I believe I saw this error after the last install, as well.  just downloaded node.js and npm, so that's not the problem.
I following the directions to the letter, as far as I can tell.
Thanks for your time and attention!
I'm sure there's an answer to this somewhere, but I'm not finding it—apologies if I'm just being daft. I get as far as:

info: Listening on 8888

but when I try to load http://localhost:8888/ into any browser, I get a blank page in FF, and in Chrome I get

This site can't be reached
The webpage at http://localhost:8888/ might be temporarily down or it may have moved permanently to a new address.
ERR_SOCKET_NOT_CONNECTED

In Safari, I get:

Safari Can't Open the Page
Safari can't open the page "http://localhost:8888/" because the server unexpectedly dropped the connection. This sometimes occurs when the server is busy. Wait for a few minutes, and then try again.

I feel like I'm sooooo close, but the grapes are JUST beyond my grasp. Help? Please?
Thanks!
Jason

I'm not sure about the latter error, but it looks like canvas is failing to install. Did you successfully install the dependencies first (https://github.com/nypublicradio/audiogram/blob/master/INSTALL.md#mac-os-x-installation)? If so, do you have a more complete error log from the installation?  I would open an issue here: https://github.com/Automattic/node-canvas/issues

THANKS for the reply @veltman !  Like that's ever really useful, right!?

Glad it's resolved!


cilium-docker doesn't respect --debug flag

When starting cilium-docker the --debug flag is not respected, it seems some messages that should be printed are not being printed.

Merged #2019


Rewrite of code that checks for inventories.

Inserters now work as follows:

if target-inventory can receive item, it will receive it.
source will only be pulled when target-inventory accepts it
Inserters disable when a redstone-signal is applied #8

Some more changes.


[Solver PR 2] Move solvers from Core into SDK

Overview
Moves the solvers from Core into the SDK layer. This seems like a more natural fit for them, as they're MonoBehaviours and are components that build off of the Core layer.

We need to keep the solvers in Core

Why do we need to keep solvers in Core?

Because the Lines depend on them, and they also should stay in Core, as Pointers depend on Lines.

Which lines depend on solvers?

Which lines depend on solvers?

All of them. I suppose the real question is why do you want to move it into the SDK?

All of them.

Can you show me the lines of code where they depend on Solvers? I did not have to update any code, and the demo scene and pointers all still work.

I suppose the real question is why do you want to move it into the SDK?

They make more sense in the SDK. These are MonoBehaviour scripts that developers will need to put into their scene in order to use. That's basically the definition of the SDK layer, not the internal/core MRTK layer. If the Core is distributed as a DLL, is it possible to consume MonoBehaviours that have been included?

If the Core is distributed as a DLL, is it even possible to consume MonoBehaviours that have been included?

Yes, but it's not ideal.

Can you show me the lines of code where they depend on Solvers?

I'm surprised you didn't get compile errors after moving them

Oh wait... maybe it was distorters I was thinking of.

While, yes these are generally going to be MonoBehaviours, I think it makes sense to keep them in core as they're used in important physics calculations but I suppose they could potentially be made into an interface if we want to abstract it out more.

Yes, but it's not ideal.

Then it sounds like we shouldn't include any MonoBehaviours in the Core that developers need to add to the scene. We need to make the Toolkit as easy as possible to be consumed.

I think it makes sense to keep them in core as they're used in important physics calculations

If we want to bring out certain physics calculations into the core, that sounds like a great future task to talk about. As-is, nothing in the core depends on or uses Solvers, and I'm not sure it makes sense for them to.

We need to make the Toolkit as easy as possible to be consumed.

Agreed That's def my mantra.

If we want to pull some primitive physics pieces back into the Core.

But it's already in the core. I have a feeling this is a bad idea. Can we at least keep the physics namespace?


NDS-570: Add swagger spec documentation

Added docs.sh to generate Swagger html (manual)
Added swagger-spec/index.html
Added /specs API endpoint to render HTML page

Closing this PR in lieu of #144


Whitespace issues with editing namestorage

It seems blank lines are condensed when saving namestorage in the web UI.
This may be related to a bug I experienced where the generated patch data referenced the wrong lines.  My resulting file then had some of the content shifted a few lines from where it should have been added.

fixed by #36


Clean ÖbEngine for v0.4

A lot of stuff is not used anymore due to the recent reworks made on ÖbEngine, here is a list of stuff we should clean before release :

Remove engine/Sprites folder
Remove engine/Obe.vili
Remake engine/Mount.vili
Remake engine/Lib/Internal/Debug.lua
Clean engine/Lib/Internal/Environment.lua
Clean engine/Lib/Internal/LuaCore.lua ?
Remove engine/Lib/Internal/Network.lua
Clean engine/Lib/Internal/ObjectInit.lua
Remake engine/Lib/Stdlib/Class.lua
Remove engine/Lib/Stdlib/ConsoleColor.lua
Remove engine/Lib/Stdlib/Constant.lua
Clean engine/Lib/Stdlib/Contains.lua ?
Remove engine/Lib/Stdlib/Copy.lua ?
Remove engine/Lib/Stdlib/IsArray.lua ?
Remove engine/Lib/Stdlib/Overload.lua ?
Remove engine/Lib/Stdlib/String.lua ?
Remove engine/Lib/Stdlib/Table.lua ?
Remove engine/Lib/Toolkit ?
Clean Fonts add an engine default font
Remove engine/Data/GUI
Clean engine/Data/config.cfg.vili
Remove extlibs/filesystem

engine/Lib/StdLib/Class.lua is still used in engine/Lib/Internal/Canvas.lua but is easily removable
engine/Lib/Internal/Debug.lua should be reworked in a separate issue (https://github.com/Sygmei/ObEngine/issues/280)
engine/Lib/StdLib/ConsoleColor.lua still exists since the Toolkit uses it (completely unusable in the new architecture but I'm leaving it as is for the moment)
engine/Lib/StdLib/contains.lua is still used by the toolkit, we should probably use PenLight or some other general library instead but I'm leaving it as is for now
engine/Lib/StdLib/Copy.lua is still used by the toolkit, we should probably use PenLight or some other general library instead but I'm leaving it as is for now
engine/Lib/StdLib/String.lua is still used by the toolkit, we should probably use PenLight or some other general library instead but I'm leaving it as is for now
engine/Lib/StdLib/Table.lua is still used by the toolkit, we should probably use PenLight or some other general library instead but I'm leaving it as is for now
The entire Toolkit will be kept in the source even if it is dead code currently since there is no UI to display it

I chose NotoSans as the engine default library, it might change later


Aggregated comments

Combines comments and screenshots, retains existing fail/pass/skip priority logic.
Test shows what the comment would look like:

This is superseded by the inclusion of Mocawesome Report graphics


Example plugin can not build

Hello, I am new to rust audio so I just want to learn from the examples. I cloned this repository but when I try to run with cargo xtask bundle gain --release it failed to compile. I am using the lastest stable Rust on windows 11 and the error message look like this:
error[E0369]: binary operation `==` cannot be applied to type `LitStr`
  --> nih_plug_derive\src\params.rs:65:55
   |
65 |                         if params.iter().any(|p| p.id == s) {
   |                                                  ---- ^^ - LitStr
   |                                                  |
   |                                                  LitStr

error[E0369]: binary operation `==` cannot be applied to type `LitStr`
   --> nih_plug_derive\src\params.rs:106:67
    |
106 |                         if persistent_fields.iter().any(|p| p.key == s) {
    |                                                             ----- ^^ - LitStr
    |                                                             |
    |                                                             LitStr

   Compiling assert_no_alloc v1.1.2 (https://github.com/robbert-vdh/rust-assert-no-alloc.git?branch=nih-plug#95120d99)
error[E0277]: `LitStr` doesn't implement `Debug`
   --> nih_plug_derive\src\params.rs:475:5
    |
470 | #[derive(Debug)]
    |          ----- in this derive macro expansion
...
475 |     id: syn::LitStr,
    |     ^^^^^^^^^^^^^^^ `LitStr` cannot be formatted using `{:?}` because it doesn't implement `Debug`
    |
    = help: the trait `Debug` is not implemented for `LitStr`
    = note: this error originates in the derive macro `Debug` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0277]: `LitStr` doesn't implement `Debug`
   --> nih_plug_derive\src\params.rs:484:5
    |
479 | #[derive(Debug)]
    |          ----- in this derive macro expansion
...
484 |     key: syn::LitStr,
    |     ^^^^^^^^^^^^^^^^ `LitStr` cannot be formatted using `{:?}` because it doesn't implement `Debug`
    |
    = help: the trait `Debug` is not implemented for `LitStr`
    = note: this error originates in the derive macro `Debug` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0277]: `LitStr` doesn't implement `Debug`
   --> nih_plug_derive\src\params.rs:494:9
    |
489 | #[derive(Debug)]
    |          ----- in this derive macro expansion
...
494 |         group: Option<syn::LitStr>,
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^ `LitStr` cannot be formatted using `{:?}` because it doesn't implement `Debug`
    |
    = help: the trait `Debug` is not implemented for `LitStr`
    = help: the trait `Debug` is implemented for `Option<T>`
    = note: this error originates in the derive macro `Debug` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0277]: `LitStr` doesn't implement `Debug`
   --> nih_plug_derive\src\params.rs:500:9
    |
489 | #[derive(Debug)]
    |          ----- in this derive macro expansion
...
500 |         id_prefix: syn::LitStr,
    |         ^^^^^^^^^^^^^^^^^^^^^^ `LitStr` cannot be formatted using `{:?}` because it doesn't implement `Debug`
    |
    = help: the trait `Debug` is not implemented for `LitStr`
    = note: this error originates in the derive macro `Debug` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0277]: `LitStr` doesn't implement `Debug`
   --> nih_plug_derive\src\params.rs:508:9
    |
489 | #[derive(Debug)]
    |          ----- in this derive macro expansion
...
508 |         group: Option<syn::LitStr>,
    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^ `LitStr` cannot be formatted using `{:?}` because it doesn't implement `Debug`
    |
    = help: the trait `Debug` is not implemented for `LitStr`
    = help: the trait `Debug` is implemented for `Option<T>`
    = note: this error originates in the derive macro `Debug` (in Nightly builds, run with -Z macro-backtrace for more info)

I also tried several gain_gui also got errors. I would appreciate it if someone could tell me how run examples.

Fixed in 3f007c8a683417fc633cf5447c92b6cef9432809. I assumed the Eq trait would always be derived for these structs but I guess not. Weird how it compiled just fine on the CI.


Openai mod validator

Summary by CodeRabbit

New Features

Introduced a new content moderation feature using OpenAI's moderation model to ensure message compliance.

Documentation

Updated tutorials with new kernel specifications, Python versions, and minor content edits.
Added a hyperlink to Pydantic documentation for better user guidance.

Bug Fixes

Removed duplicate content in the knowledge graphs tutorial for clarity.

Refactor

Streamlined import statements in the validation module for better code organization.

Can add a doc page likecontent/moderation.md

do you mean under 'examples'? or should I add a new directory?


Is possible to change folder icon?

Is possible to change the folder icon?
Now is standard folder like a standard folder bookmark.

Unfortunately, the WebExtension API does not let us do this.

For now, you could at least append an emoji to the start of the name, to differentiate them from normal folders. Maybe one of those: 🔶🆚📶🔁↗🔔

change the icon folders of the bookmarks toolbar !
in the userChrome.css 👍
#bookmarks-view treechildren::-moz-tree-image(container),
#PlacesToolbarItems toolbarbutton[container="true"],
#PlacesToolbarItems menu[container="true"] .menu-iconic-left,
#BMB_bookmarksPopup menu[container="true"] .menu-iconic-icon,
#bookmarksMenu menu[container="true"] .menu-iconic-icon,
.bookmark-item[container] {
list-style-image: url("folder5.svg") !important; /* place svg file in the same folder of the css */
}

replace folder5.svg by this 👍

change the icon folders of the bookmarks toolbar !
in the userChrome.css +1
#bookmarks-view treechildren::-moz-tree-image(container), #PlacesToolbarItems toolbarbutton[container="true"], #PlacesToolbarItems menu[container="true"] .menu-iconic-left, #BMB_bookmarksPopup menu[container="true"] .menu-iconic-icon, #bookmarksMenu menu[container="true"] .menu-iconic-icon, .bookmark-item[container] { list-style-image: url("folder5.svg") !important; /* place svg file in the same folder of the css */ }

Just so it's clear - this changes every folder icon not just for livemarks folders, correct?

well yes, that's impossible to change the icon of the webext folders , you can eventually change the other folders by naming them one by one :
.bookmark-item[container="true"] [label="Label of the folder"] {
list-style-image: url("folder_icon.svg") !important;
}

well yes, that's impossible to change the icon of the webext folders , you can eventually change the other folders by naming them one by one :

.bookmark-item[container="true"] [label="Label of the folder"] {
	list-style-image: url("folder_icon.svg") !important;
}

That actually works for me, thank you!

well yes, that's impossible to change the icon of the webext folders , you can eventually change the other folders by naming them one by one :

.bookmark-item[container="true"] [label="Label of the folder"] {
	list-style-image: url("folder_icon.svg") !important;
}

That actually works for me, thank you!

glad to have help you a bit

A way to automatize is using prefixes, instead of listing (unread = 1 space, read = 2 spaces, + 1 space added between prefix and URL):
.bookmark-item[container][label^="  "], /* ADDED */
.bookmark-item[container][livemark] {
  list-style-image: url("./../../image/livemark-folder.png") !important;
  -moz-image-region: auto !important;
}
/* ADDED */
.bookmark-item[container][label^="  "] > .toolbarbutton-text {
  margin-left: -6px !important;
}
.bookmark-item[container][label^="   "] > .toolbarbutton-text {
  margin-left: -9px !important;
}

(Set image URL and size of spaces, 3px for me)
https://github.com/Aris-t2/CustomCSSforFx/issues/211#issuecomment-520222395

Duplicate of #60


Fix cart

Because I was confused by what you wanted.  Fixed in five minutes.
On Sat, May 11, 2019 at 5:56 PM Robert French<EMAIL_ADDRESS>wrote:

@rfrenchseti requested changes on this pull request.
In log_analyzer/opus/session_info.py
https://github.com/SETI/pds-opus/pull/676#discussion_r283115673:

@@ -239,7 +239,13 @@ def __download_cart_metadata_csv(self, _query: Dict[str, str], _match: Match[str
def __create_archive(self, query: Dict[str, str], _match: Match[str]) -> SESSION_INFO:
self.performed_download()
has_url = query.get('urlonly') not in [None, '0']

   text = f'Download {"URL" if has_url else "Data"} Archive for Cart'

   if self._previous_product_info_type is not None:

I don't understand why you are using _previous_product_info_type here. The
"create archive" URLs have their own "types=XXX,YYY" field, which is what's
important here, not what happened to be given for some previous status or
view API call.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SETI/pds-opus/pull/676#pullrequestreview-236397178,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABC2LIJ3TSP5Z4EYKFQ4AGLPU5TLTANCNFSM4HMHEC7Q
.


Add version: TTKN.CNKIExpress version 0.0.5

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.0 schema?

Microsoft Reviewers: Open in CodeFlow

/azp run


Using Api

How to do the same using c# code

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d90bbf62-44f9-30a6-d6ea-bbf776f6efc5
Version Independent ID: b3961eac-6c70-b345-f48c-e4c0ec67059c
Content: Azure App Service IP Restrictions
Content Source: articles/app-service/app-service-ip-restrictions.md
Service: app-service-web
GitHub Login: @btardif
Microsoft Alias: byvinyal

@NambiVenkatesh good question. I have passed this on to the content owner to review and provide an answer. Please stay tuned.

thanks for your interest in this area, we are currently working on a new version of this API and will release new UX and instructions to match this in the coming weeks.

FYI @panchagnula

Thank you

Any updates on this, when can we expect the new api release

I am also interested by this. Using the interface is very time consuming and hard to automate.

We released the UI experience using Azure portal for App services today. Please check it out.

i see a few things here - the  part about your API getting Disabled messages - depends on the kind  of IP restrictions added - you can add an implicit Allow (Any - <IP_ADDRESS>/0) & check if this allows your app to work correctly.
Regarding the IP restrictions Blade itself failing, can you send me an email sisirap@microsoft with the details, do a CTRL+ALT+D after launching this blade & send this in the email, also if you open your Devtool bar console window of your browser & add a snap shot of the errors you see here. Thank you.

I'm interested in this from PowerShell.
Including values for name and priority in the objects added to ipSecurityRestrictions property of the Properties property returned form Get-AzureRmResource  -ResourceType 'Microsoft.Web/sites/config' still leads to, on refresh, the Portal's IP list still saying "No Names or Priorities" set (and all IPs being blocked until that point.

It looks like this issue has been addressed, so I am closing this. Thanks!


Scilab Integration?

Any chance of integrating Scilab with SoS. Scilab is a solid open source alternative to MATLAB. Scilab includes Xcos which is similar to Simulink. (see https://www.scilab.org/).

Understood... I'll take a look and see what I can do.

We are making progress by learning scilab along the way. We will likely release a beta version of the scilab module this week.

Just checking back in... anything I can do to help? Testing? Regards...

Data frame does not work since we could not find an equivalent datatype in scilab. Other than that we believe the module is working ok, although we are doing more testing and documentation before we make an initial release.
Anyway it should be safe for you to install the module from source and start testing.


ResourceGroupLocation in example is incorrect format with spacing

ResourceGroupLocation: "Australia East" errors with
ERROR: unrecognized arguments: East
Correct reference
ResourceGroupLocation: "AustraliaEast"

Thanks for the observation @thomast1906 . PR raised for the solution. https://github.com/Azure/arm-deploy/pull/45

Thanks @thomast1906 for noticing. As PR for this has been merged so closing this issue.