Execution time

Hi,
Everytime I execute the function, I get:
Fatal error: Maximum execution time of 30 seconds exceeded in ..../Wrep/Notificato/Apns/Gateway.php on line 173

The push notification is sent, but the server code never resumes. It just keeps running until our server maximum settings.

Thanks for using Notificato!
Can you show me what function you execute in what way? Some code would be really useful!
Some other questions that could help me see what's going on:

Are the examples in the Readme working for you or not?
When you (temporary) raise the time limit, does it return or give an error then?
Is the push notification received by the iOS device?

Hi @mac-cain13 ,

I'm using the The full blown example, as specified on push.md. The only thing I did was to change variables to reflect my device token.
If I raise the limit, issue still persists.
Yes, it is received.

I believe is some kind of error on Apple servers.
Thank you,

Any news on that @mac-cain13 ?

I stopped using the library, it doesn't work for me.

Sad to hear and sorry to see your go! I can't reproduce the issue over here, want to look further into it, but I'm short on time at this moment.
Hope you can find a good working solutions for your pushnotifications!

Solved!

What solved it for you? Another lib or something fixed in this lib?

A crazy long execution time (5 minutes).

Alright, thanks for letting me know!


Support ADPCM sounds

(Formerly part of #198, but not necessary for a merge)
We need to be able to decode and play ADPCM WAV sounds, since they're quite common and no browser supports them natively.

Fixed see #494

@htmlgames Have you submitted a PR?

Hi, slightly off topic - I've made many fixes to my local copy of phosphorus, some of which are work in progress (a simple diff of the phosphorus.js on the website I link to will show this) so I still need to update my github fork (it's currently just a copy of master at the moment) for the wav pcm stuff...
Also, I believe Nathan is aware of this fix and maybe looking into other methods (from comments on other threads) - the fun issue here of course is all the different PCM formats that can be held in a wav container! ...
btw: recording all sounds in Scratch itself (rather than a third party tool) helps a lot here.


Changed "rest" to "res" (typo)

"Rest" always returns 25 results as it's not a valid query.

Please let me know if my change to def url type is unnecessary, this is my first pull request.

Awesome! thanks @Lamarcke


As a user I want to be able to filter news based on type [because]

Acceptance criteria

Types need to be defined
[Criteria 2]

Needs clairification
One type that is needed but not defined on the newsroom page is Crude Oil Assays

Covered by other stories


local html file can't load js?

Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
...

Expected results:
Actual results:

Logs

this is my local HTML file, it works fine in android Webview,

window.mathwalletchains was inited in 'index.js' file, but now print(window.mathwalletchains) is undefined.
so I don't know why it can't work?
unsupported?
Thank you!

Hi @guangxingmao
This platform is not meant for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


Convert_commit_data_to_string throws error while navigating in a new file

When working on a file which is in .gitignore or when working on a new file an error has thrown.

Fixed with #3


[HxGrid] ContextMenuCssClass parameter

We would like to be able to set css classes on ContextMenu created using ContextMenu RenderFragment parameter the same way we are able to set css classes on HxGridColumn using ItemCssClass parameter.
Right now for our ContextMenu we have to use a workaround where we define ContextMenu as HxGridColumn
with parameter Order = int.MaxValue - 1.

Will probably go with HxGrid.ContextMenuSettings to be able to pass additional settings to the HxContextMenu encapsulated inside HxGrid.

released


Enable IE 11 compability of Angular + pdf.js

Do you want to request a feature or report a bug?
feature
What is the current behavior?
Currently pdf.js defines document.currentScript but not link.origin nor link.protocol. If angular starts it checks if it is allowed to bootstrap automatically it checks for currentScript and asume that this will be enough to filter IE. This check will not work in combination with pdf.js
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via https://plnkr.co or similar (template: http://plnkr.co/edit/tpl:yBpEi4).
To demonstrate this behaviour there is a plnkr with a minimal environment, that shows the problem: http://plnkr.co/edit/YFCQM2Px0QU0KnGzsAlM?p=preview
What is the expected behavior?
Angular should check also for link.origin and link.protocol if they are implemented, to make sure that it works correctly with e.g. pdf.js
What is the motivation / use case for changing the behavior?
pdf.js is widely used to render pdfs and angular is spreaded for web-apps. They will occur on the same page more often than probably expected, because they work in similiar domains, so they shouldn't interfier each other.
Which versions of AngularJS, and which browser / OS are affected by this issue? Did this work in previous versions of AngularJS? Please also test with the latest stable and snapshot (https://code.angularjs.org/snapshot/) versions.
The whole angular 1.6.x tree is affected. The problem only occurs on IE 11 on each Windows.
Other information (e.g. stacktraces, related issues, suggestions how to fix)
Probably there can be a additional check for link.origin like:
if (link.origin !== undefined && document.location.origin === link.origin) { …

And there can be a case for the undefined protocol:
switch (link.protocol) {
  case 'http:':
  case 'https:':
  case 'ftp:':
  case 'blob:':
  case 'file:':
  case 'data:':
  case undefined:
    return true;
  default:
    return false;
}

Can you post a demo where the actual pdf.js and AngularJS files are used? I cannot reproduce the issue. To clarify, what you mean is that AngularJS autoboostrap is not working in IE11 when pdf.js is included?

The workaround for this is simple. If you are using PDF.js on IE then you just manually bootstrap AngularJS. So instead of relying upon <body ng-app="app"> you use:
angular.bootstrap(document.body, ['app']);

It is unlikely that we will fix this on our side. It is PDF.js that is polyfilling document without accounting for what other libraries might be expecting.

I'd add that libraries shouldn't modify properties of objects they don't own as it causes issues like that (& many others). If they need a particular API that is missing in some supported environments they may wrap its usage in a function that will use the shimmed version if the native one is not available.

Thanks for the discussion the argument are valid and good. So I'll close this issue.

Thanks for closing.

Faced with this issue on Firefox and angular 1.6.x
anybody solve it? how?


TestMempoolReactorMaxActiveOutboundConnections is flaky

TestMempoolReactorMaxActiveOutboundConnections is flaky
Setup
ci
Config
ci
What happened?
tests failed when running 40x
What did you expect to happen?
tests to pass
How to reproduce it
use this branch to run ci 40x
Logs
https://github.com/cometbft/cometbft/actions/runs/7410713559/job/20163676330?pr=1954

More proof thank god for ci or it would end up in the circular file:
https://github.com/cometbft/cometbft/actions/runs/7436559313/job/20233157052?pr=1954


Soccer is wrong

Football is the correct name ;-)

Thanks!

Thanks!

You do know that was a joke, right? In British  English we usually say football, whereas American English employs soccer. This is despite the  term soccer having originated in the UK.

You do know that was a joke, right? In British  English we usually say football, whereas American English employs soccer. This is despite the  term soccer having originated in the UK.

I didn't know the final part, but I just decided to join in the joke ;-)
This is similar to Portuguese, where there are differences between Portuguese-BR and Portuguese-PT: "baliza" = "gol", for example

I didn't know the final part, but I just decided to join in the joke ;-)
This is similar to Portuguese, where there are differences between Portuguese-BR and Portuguese-PT: "baliza" = "gol", for example

For historical book keeping : Why americans call it soccer?

For historical book keeping : Why americans call it soccer?


HDDS-6815: EC: getFileCheckSum should return null EC files until ECFileChecksum implemented.

What changes were proposed in this pull request?
GetFileChecksum is implemented only for Ratis based file layout. For EC files layout, we have no implementation ready yet. This JIRA is to return null for EC files until we have the implementation ready. Otherwise, we will get the wrong CRC unknowingly for EC files.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6815
How was this patch tested?
Added a UT.

Merged. Thanks @umamaheswararao !

Thanks @jojochuang for the review!


PR 2

Code Review Template
Base goals

[x]  Clear and descriptive commit messages
[x] Effectively uses Markdown in notebooks
[ ]  Passwords,credentials,  etc. are not in repo
[ ]  Functions have comments
[x]  Code is written by the student
[x]  Code is organized and easy to read
[x]  Code demonstrates understanding of underlying data/statistics concepts
[x]  MVP Complete

Stretch goals

[ ]  Exceeds accuracy and efficiency expectations of the assignment
[ ]  At least 1 stretch completed

Notes


How to make "AR@k evaluation does not count those proposals on the 'seen' classes into the budget (k)" ?

Hi, thank you for your excellent work. I used your model to eval on the coco_val2017_unseen_classes under your framework, and it works well.
However, when I convert the result into a format that can be eval by detectron2 and test it on the coco_val2017_unseen_classes  under the detectron2 framework , the performance becomes poor.
So, i'd love to know how to make what you said "AR@k evaluation does not count those proposals on the 'seen' classes into the budget (k)" ？
Looking forward to your reply.

Oh, I found mmdet/datasets/coco_split.py and know the answer.
So, if I have separated the annotations json of seen and unseen categories, I don’t need to use this ‘CocoSplitDataset’, but just change mmdet/datasets/coco.py directly, right?

I found that I made a mistake in the order of the images corresponding to the bbox and score in the result, it should be the coco annotation ['images'] sequence. After I modified it, the performance under detectron2 is the same as mmdet.


[9.x] Allow factories to recycle models with for method

This PR addresses https://github.com/laravel/framework/pull/44107#issuecomment-1249158498 where using the Factory for method instead of specifying a 'model_id' => Model::factory() definition does not use recycled models. I think this makes sense, although typically you'd call for from your test where you could just pass the model directly, so I think it's more of an edge case.
I considered adding support for the has method as well, however, I don't think that one makes as much sense. If the recycled model belongs to another model, then I don't think it's parent relationship should be re-assigned, especially if it's part of a has many relationship where more than one model might be needed. I could make it attach the model in a many-to-many relationship, but it would probably need to be conditional on whether one or many models were requested. In any case, I can't think of a scenario where this would be a papercut so I'm not inclined to look into it further right now.

I wasn't too keen on making the recycle property public as it doesn't match the other properties. But I thought it might also be overkill to create a public method to access it.


Version are sorted by string rather than number ? Version 9.2 is larger than 11.6 !

TL;DR
The following project https://github.com/mljs/spectra-processing is current at version 11.6.0.
We are using a 'release-please-action' in this github action:
https://github.com/mljs/spectra-processing/blob/master/.github/workflows/release.yml
that uses the following template:
https://github.com/zakodium/workflows/blob/main/.github/workflows/release.yml
In the latest PR rather than adding the changes since 11.6.0, release-please added all the changes since 9.2.0 and has proposed an lower version number
https://github.com/mljs/spectra-processing/pull/153/files#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519
Expected behavior
We expect that only changes from 11.6.0 appears in the change log and the new version is 11.7.0
Observed behavior
All the changes since 9.2.0 are in the change log and the proposed version is 10.0.0
Action YAML
name: Release

on:
  push:
    branches:
      - master

jobs:
  release:
    uses: zakodium/workflows/.github/workflows/release.yml@release-v1
    with:
      npm: true
    secrets:
      github-token: ${{ secrets.BOT_TOKEN }}
      npm-token: ${{ secrets.NPM_BOT_TOKEN }}

---------------------------------------------------

name: Release a Node.js package

on:
  workflow_call:
    inputs:
      npm:
        description: >
          Whether the package will be published to npm. You must pass the
          npm-token secret in that case.
        type: boolean
        default: false
      node-version:
        description: Version of Node.js used to run the npm publish steps.
        type: string
        default: 16.x
      npm-setup-command:
        description: Command used to setup the package before publishing to npm.
        type: string
        default: npm install
    secrets:
      github-token:
        required: true
      npm-token:
        required: false

jobs:
  release-please:
    runs-on: ubuntu-latest
    outputs:
      release_created: ${{ steps.release.outputs.release_created }}
    steps:
      - name: Get package name
        run: echo "PACKAGE_NAME=$(jq .name package.json | tr -d '"')" >> $GITHUB_ENV
      - uses: google-github-actions/release-please-action@v3
        id: release
        with:
          token: ${{ secrets.github-token }}
          release-type: node
          package-name: ${{ env.PACKAGE_NAME }}
          bump-minor-pre-major: true
  npm-publish:
    needs: release-please
    if: ${{ inputs.npm && needs.release-please.outputs.release_created }}
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v3
      - uses: actions/setup-node@v3
        with:
          node-version: ${{ inputs.node-version }}
          registry-url: 'https://registry.npmjs.org'
      - run: ${{ inputs.npm-setup-command }}
      - run: npm publish --access public
        env:
          NODE_AUTH_TOKEN: ${{ secrets.npm-token }}

Log output
No response
Additional information
No response

This was fixed in https://github.com/googleapis/release-please/pull/1451, thanks !


support "jsx":"preserve"

All I needed to do was pass it as if it was JavaScript, and babel had no problem with it.
For some reason, my bundle output has some extra cruft-- don't know what that is all about.

many thanks


Add.Column.ByCurve does not work

When using the AddColumn.ByCurve nothing happens in Revit. The component gives a different output compared with the component AddBeam.ByCurve. (See image below)
Revit 2019 - <IP_ADDRESS>5 20180216_1515(x64)
Rhino - (7.0.19183.10495, 07/02/2019)

Could you please test it again with the last version?

Perfect, now it works!
Thanks for your help.


Handle Cassandra Statement queries in statement tag

Currently we get errors like this when you use a Statement class rather than a string for the query and you have tracing enabled:
TypeError: <SimpleStatement query="SELECT * FROM table_name", consistency=Not Set> is not JSON serializable

👓  @spladug

:haircut_man: addressed feedback and cleaned up the commits.

:haircut_man:

:haircut_man:

To test this, I used this version of baseplate to run the gateway locally.  Without these changes, I got the expected error when trying to flush a span.  With these changes, I did not get this error.  I did get an error  when trying to flush spans, but it was about not being able to reach the trace endpoint which is expected since the endpoint specified in my dev config does not exist. :)


Add pipeline to $lookup stage

Q
A

New Feature
yes

RFC
yes/no

BC Break
yes/no

Mongo 3.6 add a pipeline lookup functionality that helps a lot in the futures developments, do you think that this feature can be added in a future veresion?
Thanks

$lookup stage is already available in 1.2, what seems to be missing?

This is a new nice feature
New in version 3.6.
To perform uncorrelated subqueries between two collections as well as allow other join conditions besides a single equality match, the $lookup stage has the following syntax:
copycopied
{
$lookup:
{
from: ,
let: { <var_1>: , …, <var_n>:  },
pipeline: [  ],
as:
}
}

All right, thanks for clarification. I've put this into 2.x milestone. If you have some time it'd be great to see a PR!

Strongly needed!

@bluedogmilan sending us a PR with implemented feature will greatly speed things up

@bluedogmilan furthermore, we’re holding all further features back to 2.1 to avoid introducing more complexity in 2.0. 2.1 will mainly focus on adding aggregation pipeline features from previous mongodb releases.

@bluedogmilan sending us a PR with implemented feature will greatly speed things up

@malarzm Sorry for the newb question but I really do not know what PR stands for...

I've taken a little time on it, since I'll probably need to use this functionality myself. Those two ways of handling $lookup are however really different and do not really mix & match.
Using one practically should disable another.
Is it acceptable to treat them as two separate classes with common ancestor? Or simply use some kind of flag to indicate which type of lookup is being used?
Should pipeline accept an instance of Stage / Builder like Facet does, or should it also allow to pass an array?
I've started by adding:
public function let(array $let) : self
{
    $this->let = $let;
    return $this;
}

public function pipeline($pipeline) : self
{
    if (! is_array($pipeline) && ! $pipeline instanceof Expr) {
        throw new \InvalidArgumentException(__METHOD__ . ' expects either an aggregation builder or an aggregation stage.');
    }

    $this->pipeline = $pipeline;
    return $this;
}

methods and considering
public function getExpression() : array
{
    if (isset($this->pipeline)) {
        $pipeline = $this->pipeline instanceof Expr ? $this->pipeline->getExpression() : $this->pipeline;
        return [
            '$lookup' => [
                'from' => $this->from,
                'let' => $this->let,
                'pipeline' => $pipeline,
                'as' => $this->as,
            ],
        ];
    } else {
        return [
            '$lookup' => [
                'from' => $this->from,
                'localField' => $this->localField,
                'foreignField' => $this->foreignField,
                'as' => $this->as,
            ],
        ];
    }
}

First of all, I made some changes on my fork here: https://github.com/emulienfou/mongodb-odm
It's a work in progress to add some tests.

However, I was able to make it work using an array in the pipeline key, such has:
$builder->lookup('OnlineFestival')
    ->let(['festival' => '$fk_online_festivals'])
    ->pipeline(
        [
            [
                '$match' => [
                    '$expr' => [
                        '$and' => [
                            ['$in' => ['$_id', ['$ifNull' => ['$$festival', []]]]],
                            ['$eq' => ['$status', 'published']],
                            ['$gte' => ['$endAt', new UTCDateTime()]]
                        ]
                    ]
                ]
            ]
        ]
    )
    ->alias('online_festivals')

But using the query builder directly, seems to not working. I tested this kind of code:
$builder->lookup('OnlineFestival')
    ->let(['festival' => '$fk_online_festivals'])
    ->pipeline([
        $builder->match()->addAnd(
            $builder->matchExpr()->in(['$_id', $builder->expr()->ifNull('$$festival', [])]),
            $builder->matchExpr()->equals(['$status', 'published']),
            $builder->matchExpr()->gte(['$endAt', new UTCDateTime()])
        )->getExpression()
    ])
    ->alias('online_festivals')

This code seems to put the $match inside the lookup pipeline, but duplicate it in the root just after the lookup code

@emulienfou

This code seems to put the $match inside the lookup pipeline, but duplicate it in the root just after the lookup code

This usually happens if you use the wrong expression variable.
First it seems to work, as you get the correct result as return, but the root builder will also append the same code.
Try using $builder->expr()->addAnd(...)

@buffcode seems not not working, I have the next error message: Unrecognized pipeline stage name: '$and'
If someone wants to work on this feature, feel free to open a PR my master branch is updated with this new feature.

How about this? It would be great to use pipeline on lookup stages

How about this? It would be great to use pipeline on lookup stages

Hi everyone, is there any progress on this issue?

Simple working example:
class PipelineLookup extends \Doctrine\ODM\MongoDB\Aggregation\Stage\Lookup
{
    private array $pipeline = [];

    public function pipeline(array $pipeline): self
    {
        $this->pipeline = $pipeline;

        return $this;
    }

    public function getExpression(): array
    {
        $expression = parent::getExpression();

        if (false === empty($this->pipeline)) {
            $expression['$lookup']['pipeline'] = $this->pipeline;
        }

        return $expression;
    }
}

and in your code:
$builder = $this->createAggregationBuilder();

$lookup = new PipelineLookup($builder, 'events', $this->dm, $this->class);
$lookup->pipeline([
    [
        '$match' => [
            '$expr' => [
                '$and' => [
                    ['$in' => ['$_id', ['$ifNull' => ['$$festival', []]]]],
                    ['$eq' => ['$status', 'published']],
                    ['$gte' => ['$endAt', new UTCDateTime()]]
                ]
            ]
        ]
    ]
]);

$builder->addStage($lookup);

#2459 was merged manually, we can close this one 🎉


Replace deprecated functions removed in OTP 24

Bump OTP to 22.1+ for the new crypto-related functions (I could try to maintain compatibility but I don't think it's useful)
Handle ECPrivateKey extra element

Bump OTP to 22.1+ for the new crypto-related functions (I could try to maintain compatibility but I don't think it's useful)

It would be nice to keep it though, as it's just a check on the OTP version.
Otherwise you might as well bump the Elixir required version to 1.10.3 then, since it's the oldest version supporting OTP > 22.

It would be nice to keep it though

I added it in #29

Closing in favor of #29


Discuss and analyze webassembly security

WebAssembly does not protect against problems inside the sandbox. If you have C++ code that executed undefined behaviour, it can do any possible thing it likes inside that sandbox, poking the interface to the host in any possible way, but the full C++ UB experience does not transfer to the embedder. In that way it can be thought of like a mini-process in your process, or as a container for a library.

https://www.oreilly.com/library/view/webassembly-the-definitive/9781492089834/ch04.html
https://petermalmgren.com/serverside-wasm-data/
https://youtu.be/glL__xjviro
https://news.ycombinator.com/item?id=24216764
https://mvolkmann.github.io/blog/webassembly/?v=1.0.19
https://stackoverflow.com/questions/69789570/webassembly-thread-safety-and-c-c-local-variables
https://stackoverflow.com/questions/64763007/why-is-webassembly-safe-and-what-is-linear-memory-model

We won't use wasm hints


Where to add translation?

I have noticed a few untranslated items in the app but can't seem anywhere to translate them (to Danish). For example 'Mail drop box' looks like it was translated in the Transifex project for iD Editor at least two years ago. I can't find anywhere else to translate it?

I see that Danish translation for amenity/post_box (Mail drop box) was published on 12 Jul 2022 in id-tagging-schema 3.4.0 so that one should indeed be in, as last everydoor release (4.1) happened on 6. Nov 2023, so I would expect it includes changes published before that.
And in EveryDoor 4.1 it indeed seems to include that translation, but I do see an interesting problem when it tries to use it:
When I manually switch language to Dansk in the app, try to add POI by searching for Mail drop box, I see 3 results: one is (correctly) translated to Postkasse, but there are two more instances that are not.
All three tag only amenity=post_box (checked in raw tags editor), so those are not untranslated other sub-items. I'm puzzled why that happens; I would expect just one result...
Also, when clicking on existing amenity=post_box (e.g. the one just created), it always shows English Mail drop box in that blue title, instead of Danish Postkasse (which I probably why @ERYpTION thought the translation didn't made it in). I'd guess this is connected with that duplication above.


Fixing missing export types/Actions in the final bundle

#151

@matheusml  Can you merge this to the master branch?


Continue On Codespace: Asked to sign-in twice

Testing #167433
I was using a new user data dir and Codespaces was the first extension to require sign-in. The sign-ins happened shortly after another. I had a change in the local workspace that was cloned from a private GitHub repo.

Which were the extensions requesting sign in? I would guess they request different scopes. cc @TylerLeonhardt

The first sign-in is right after I choose 'Create New Codespace' from the 'Continue Working On...' QuickPick. That doesn't tell me which extension triggers it. The second sign-in right after the first announces that GitHub Codespaces is requesting the sign-in. I don't see which additional permissions it needs. Signing out and going through the Continue On UI again shows the same behavior.

Sounds like maybe Edit Sessions and the Codespaces extension that ask for different sets of scopes.
I'm not sure we can do anything about this given the Codespaces use of the scope codespace. We'll never require that for edit sessions I think.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Documentation on custom templates?

Hi,
I am interested in using this for some internal forms at my company, but we typically use material components for the web over bootstrap.  In the documentation, "Specify custom templates for rendering different object types" is listed as a feature but I do not see any documentation on how this should be accomplished.  Can you provide any guidance?
Thanks

If you install the templates locally into your project, the local copies
take priority over the ones loaded from assembly.
You can therefore update the copied templates to reflect the html required
for your framework.
On Thu, 9 Apr 2020, 21:06 Alex Griffith<EMAIL_ADDRESS>wrote:

Hi,
I am interested in using this for some internal forms at my company, but
we typically use material components for the web over bootstrap. In the
documentation, "Specify custom templates for rendering different object
types" is listed as a feature but I do not see any documentation on how
this should be accomplished. Can you provide any guidance?
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mcintyre321/FormFactory/issues/96, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACDJ6UR2LZXPP3GK2E5NZ3RLYTEHANCNFSM4ME7Y4HQ
.

Cool, thanks for the clarification! I'll try to play around with it some this weekend.


Reference to non-existing setting, "Experimental View Manager"

This page mentions a setting for Azure Logic Apps Standard regarding "Experimental View Manager". There is no setting of that name to be found in my edition of VSCode and the Azure Logic Apps Standard extension, both of which are at the latest version.
Azure Logic Apps (Standard), 2.0.16
Visual Studio Code, Version: 1.77.0 (user setup)

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

ID: cbdd9c30-1bc1-7af6-158d-f65ad23f45b7
Version Independent ID: c8828014-4369-ead5-2d05-8c3e823fc93e
Content: Create Standard workflows in single-tenant Azure Logic Apps with Visual Studio Code - Azure Logic Apps
Content Source: articles/logic-apps/create-single-tenant-workflows-visual-studio-code.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

@sandokanfirst
Thanks for your feedback! We will investigate and update as appropriate.

@ecfan I didn't find this setting in both VS code extension as well as package.json .
Please review.

@sandokanfirst & @mumurug-MSFT: Thanks, we'll update the doc.

@sandokanfirst & @mumurug-MSFT: The doc update will go live after 4 PM Pacific Time.


Setup the project structure

[x] GitHub Repo
[x] Maven project for a web and connect it with github
[x] Add necessary dependencies
[x] Setup logging for the project
[x] Setup a workspace in postman

Close issue #1


Adding refresh token

Is your feature request related to a problem? Please describe.
Adding a JWT refresh token, In the current situation there is no refresh token for the users, they have to continously authenticate them selves after 20min of expiration of the token

@Harshdev098 I can work on this.. Assign this to me


Cle/4958 receive full elr

This PR refactors Sender into an abstract base Sender class with concrete subclasses FullELRSender and CovidSender. The former is the first step into the FHIR pipeline, the latter will be used for the existing covid pipeline. This PR does not include any functionality to actually route the submissions from a full ELR sender, and using a sender configured as such will cause a 500 error. This is expected, the work to route these messages and add tests is in follow-on tickets (#5051, for example).
Test Steps:

Run existing tests, verify covid messages for senders configured with topic covid-19 work properly.

Changes

Create abstract Sender class that is base for FullELRSender and CovidSender
Implement json sub typing for proper deserialization into child classes
Remove settings metadata as it is no longer needed and is in the table itself
Remove SenderAPI in favor of Sender since SenderAPI is not needed and causes deserialization typing issues
Create Topic enum within the Sender class to more strongly track our sender topics

Checklist
Testing

[x] Tested locally?
[x] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?

Pull reviewers stats
Stats for the last 30 days:

User
Total reviews
Time to review
Total comments

cwinters-usds
18
5h 3m
12

TomNUSDS
15
1h 27m
30

carlosfelix2
13
5h 39m
8

jimduff-usds
9
1h 49m
12

MauriceReeves-usds
9
17h 34m
0

jbiskie
8
1h 52m
0

TurkeyFried
7
11h 39m
2

kevinhaube
7
17m
0

brick-green-agile6
7
18h 19m
2

sethdarragile6
7
59m
0

ahay-agile6
5
13m
0

clediggins-usds
5
4h 2m
4

snesm
5
3h 26m
3

oslynn
4
11m
0

jeremy-page
4
12m
0

jh765
4
1h 45m
7

JosiahSiegel
3
12m
0

RickHawesUSDS
3
3d 8h 47m
2

bgantick
2
16m
1

whytheplatypus
1
13m
2

acoushawk
1
4m
0

jorg3lopez
1
2d 2h 20m
0

rhood23699
1
15m
0

Wrong git account


e.instancePath is undefined

const validate = ajv.compile(jsonSchema)
validate(value)
validate[0].instancePath

There is however a schemaPath and a dataPath

The TS shows an instancePath and schemaPath but no dataPath
export interface ErrorObject<K extends string = string, P = Record<string, any>, S = unknown> {
    keyword: K;
    instancePath: string;
    schemaPath: string;

What version of Ajv are you using? Does the issue happen if you use the latest version?
"ajv": "^8.12.0",

Ajv options object

What results did you expect?
TS types to match the actual data
Are you going to resolve the issue?
Happy to but wanted to see if I'm doing something wrong

it looks like you have data from a previous ajv version - impossible to say without sample code to reproduce.

Closing as requested example code not provided.


Validation of HTML strings and HTML-like labels, or lack thereof

Introduction
Currently, pydot does not validate HTML(-like) strings or HTML(-like) labels, as defined by The DOT Language. I am opening this issue not expecting an immediate change in that, but rather to have a central place for discussion and tracking related issues, more specifically:

To document the lack of validation and the reasons behind it,
To link to specific issues that were caused or aggravated by the lack of validation,
To discuss mitigations, workarounds, documentation and partial or, who knows one day, complete implementations of validation,
To share other findings or opinions related to these subjects.

Definitions
To aid in discussion, we should try to use the same terms. Here is a first suggestion, feel free to suggest improvements:

HTML string or HTML-like string: As defined by the DOT language. Note that there it says that HTML strings must be legal XML, so "HTML string" may be somewhat of a misnomer, but it is used throughout the DOT and Graphviz documentation, so I will follow that.
HTML label or HTML-like label: An HTML string used as a label attribute in certain contexts, as defined by the DOT language and the document Node Shapes: HTML-Like Labels. Besides being legal XML, they must also follow the syntax defined in that last document. Again, "HTML label" is a misnomer, because the syntax is described as "modeled on HTML", but "not really HTML". For example, it does not support all of HTML (e.g.
 is not supported), and adds its own attributes (without the data- prefix that would allow for that in HTML5), such as
.
True HTML or real HTML: The normal (outside the DOT/Graphviz world) definition of HTML. Use adjectives like "true" or "real" to clearly distinguish it from the DOT-specific terms defined above.

Examples
Here are three minimal working examples, followed by a summary and a link to two real world examples.
Used versions: python 2.7.16 and 3.7.3, pydot 1.4.1, chardet 3.0.4, pip 19.2.3, pyparsing 2.4.2, setuptools 41.2.0, wheel 0.33.6, Debian 10.1 (buster), Graphviz 2.40.1-6.
Example 1: Invalid XML in an HTML string
import pydot

G = pydot.Dot()
G.add_node(pydot.Node('N', **{'<<test test>': '""'}))
print(G.to_string())
G.write_dot('broken_html_ID.dot', prog=['dot'])

Output of G.to_string():
digraph G {
N [<<test test>=""];
}

Output of G.write_dot():
"dot" with args ['-Tdot', '/tmp/tmpUiJgrg'] returned code: 1

stdout, stderr:

Error: /tmp/tmpUiJgrg: syntax error in line 4 scanning a HTML string (missing '>'? bad nesting? longer than 16384?)
String starting:<<test test>=""];
}

Traceback (most recent call last):
  File "broken_html_string_ID.py", line 7, in <module>
    G.write_dot('broken_html_ID.dot', prog=['dot'])
  File "pydot.py", line 1734, in new_method
    encoding=encoding)
  File "pydot.py", line 1817, in write
    s = self.create(prog, format, encoding=encoding)
  File "pydot.py", line 1947, in create
    assert process.returncode == 0, process.returncode
AssertionError: 1

Example 2: Invalid XML in an HTML label
import pydot

G = pydot.Dot()
G.add_node(pydot.Node('N', **{'label': '<test>'}))
print(G.to_string())
G.write_dot('broken_html_label_closing_tag.dot', prog=['dot'])

Output of G.to_string():
digraph G {
N [label=<test>];
}

Output of G.write_dot():
"dot" with args ['-Tdot', '/tmp/tmpWXjQWa'] returned code: 1

stdout, stderr:
 digraph G {
	graph [bb="0,0,54,36"];
	node [label="\N"];
	N	 [height=0.5,
		label=<test>,
		pos="27,18",
		width=0.75];
}

Error: mismatched tag in line 1
... test ...
in label of node N

Traceback (most recent call last):
[...]
  File "pydot.py", line 1947, in create
    assert process.returncode == 0, process.returncode
AssertionError: 1

Example 3: Valid true HTML, valid XML, but invalid DOT HTML label
The paragraph tag (
 or
) is not included in the DOT definition of HTML-like labels.
import pydot

G = pydot.Dot()
G.add_node(pydot.Node('N', **{'label': '<
test
>'}))
print(G.to_string())
G.write_dot('broken_html_label_p_tag.dot', prog=['dot'])

Output of G.to_string():
digraph G {
N [label=<
test
>];
}

Output of G.write_dot():
"dot" with args ['-Tdot', '/tmp/tmp3Y6fk4'] returned code: 1

stdout, stderr:
 digraph G {
	graph [bb="0,0,54,36"];
	node [label="\N"];
	N	 [height=0.5,
		label=<
test
>,
		pos="27,18",
		width=0.75];
}

Error: Unknown HTML element
 on line 1
in label of node N

Traceback (most recent call last):
[...]
  File "pydot.py", line 1947, in create
    assert process.returncode == 0, process.returncode
AssertionError: 1

Summary of examples 1, 2 and 3
All three examples show that:

pydot allows adding invalid HTML strings and HTML labels to a node.
pydot will write DOT strings containing invalid HTML strings and HTML labels, which I believe makes those DOT strings invalid in their entirety as well.
Graphviz then complains about the broken HTML by emitting an error message and returning a non-zero exit code (here: 1).
After receiving the non-zero exit code from Graphviz, pydot raises an AssertionError.

Examples 2 and 3 also show that:

Graphviz is somewhat tolerant of some invalid HTML strings or labels. In that case, it will still print an error message and return a non-zero exit code, but return data on stdout as well. In case of DOT output, it will contain the invalid HTML again. (Not shown in the examples, but in case image output was requested, Graphviz would write a readable image to stdout.)
However, pydot ignores the Graphviz output on stdout in these cases, because Graphviz returned a non-zero exit code.

Real world examples
The lack of validation by pydot means that only at the very end, when receiving back the results for the entire graph from Graphviz, an error can be reported. Since many Graphviz errors share the same exit code (1), pydot cannot do much more than pass on the Graphviz error message. But in practice, with large graphs containing many HTML strings, the Graphviz error message will not be as clear as the ones shown in the minimal examples above. With such a late and unclear error message, it is difficult for the user to trace back what went wrong exactly. Two examples of this, from two different users, can be found in issue pydot/pydot#203 (Problem with graph.write.png...AssertionError: 1).

Other findings

For comparison, I had a look at the Graphviz C libraries that come with Graphviz itself. Unless I am missing something, it looks like they also do not do any validation of HTML during graph construction, neither at the time an HTML-like string is added to storage using agstrdup_html(), nor later when the label-attribute of a node is set through agsafeset(). Only at very end, when layout of the entire graph is requested by calling gvLayout(), will errors be returned. This is comparable to the current behavior of pydot (1.4.1). See attachment graphviz-html-test.c for example code.

I also checked if Graphviz provides a specialized validation component to possibly use:

No: The Graphviz manual page (man 7 graphviz) lists all the executables with short descriptions. gvpr came close, because it has a function ishtml(). However, the outcome of that function is merely determined by the so-called HTML bit, which seems to be set if the string was wrapped in angle brackets (ishtml: true) instead of double quotes (ishtml: false). I did not see any sign of actual validation. gvpr does fail with the exactly the same error as other Graphviz executables (e.g. dot) on a label with a seriously broken tag like label=<<test test>, but that is during the parsing of the DOT file, not while the gvpr program is traversing it itself. Also, just as other Graphviz executables, it requires an entire DOT file as input, not just an HTML string. See attachments for examples: is-node-html.gvpr, is-node-html-1.dot, is-node-html-2.dot.
No: In the Graphviz C libraries (even though pydot currently does not use those), I found the aghtmlstr() function, which according to the cgraph manual page can be used to "query if a string is an ordinary string or an HTML-like string". However, after reading the source code and testing it a bit, I believe that, similar to the gvpr function ishtml(), the return value of aghtmlstr() just depends on if the string was claimed to be an HTML-like string at the moment it was added. There does not seem to be any actual XML or (DOT) HTML validation involved. The Graphviz C library example code mentioned above contains examples (graphviz-html-test.c).
Possibly: In the Graphviz source code, in lib/common there are some files starting with html* that seem to do the validation and parsing. They use expat (C library for parsing XML) and Bison (a parser generator). Unfortunately, I could not immediately find a mention of them in the header files of the compiled Graphviz libraries, so they may not be part of the API. At first glance, htmlparse.y seems to be too much integrated with the parsing process for just validation, but htmllex.c could be a possible candidate, although there would clearly still be a long way to go then.

The table below shows the combinations of Graphviz stdout/stderr/exit code that I have seen so far, sorted from most serious error to normal behavior. It is probably still incomplete.

stderr
exit code (0=success)
output on stdout?
example

error(+warning)
1
no
Example 1 in my earlier comment.

error(+warning)
1
yes
Examples 2 and 3 in my earlier comment.

warning only
0
yes
An edge to a node:port, but the node not having that port. This example is not related to HTML strings or labels, though.

nothing
0
yes
Normal behavior in case of no problems.


fix(nav): invalid justify-content property value

@kotzendekrabbe Thanks! Makes sense to me. I dropped the property in the newest commit. 👍


How to plot the loss terrain along with the path taken by the optimizer

Hello,
Can you help me understand how did you plot the loss terrain in 2D along with the path taken by the optimizer? There are millions of parameters involved and I want to understand how to bring them to 2D

Great question; we project every point in weight space (dimension n, which can be millions as you said) into a plane defined by two orthogonal vectors, both of dimension n.
We choose these vectors by looking at the first, middle, and final weights of the inner optimization algorithm.
Vector 1 = middle weight - first weight
Vector 2 = last weight - first weight - (amount explained by Vector 1)
where the amount explained by Vector 1 is analogous to what is done in the Gram-Schmidt process


Flow GUI and params database inconsistency

Steps:

Insert a print statement to print the retrieved value in this line:
https://github.com/NCAR/VAPOR/blob/dc53d18f80860753c33dddc0809be22b6cea9a8c/lib/render/FlowRenderer.cpp#L884
Import one step of Katrina
Create a 2D flow renderer with vars x:U10 y:V10
Set the seed distribution to random
Enable the renderer, tt will work as expected
Change to random with bias. Notice that in the GUI, the bias variable is CANWAT. However, the print out message says it's P, hence the inconsistency.

@sgpearse it appears that VComboBox does not handle the case when a value outside the set of options is in the params database.

Understood, but I'm not getting to this any time soon.  My plate is stacked.   If anyone can debug this other than me, then it will move us closer to an earlier release.  I'm swamped right now.

@stasj I took a quick look at this and I think the VComboBox is doing the right thing; that is just displaying either the 2D or 3D variables.  However we are not setting a new value when we switch between 2D and 3D variables.  Would you agree that the PVariableWidget should be setting a default 2D/3D variable when this switch is made?

Understood, but I'm not getting to this any time soon. My plate is stacked. If anyone can debug this other than me, then it will move us closer to an earlier release. I'm swamped right now.

@sgpearse I've opened a PR that fixes the GUI issue. VComboBox will now display the currently selected option, even if it is not a valid one.

@StasJ I took a quick look at this and I think the VComboBox is doing the right thing; that is just displaying either the 2D or 3D variables. However we are not setting a new value when we switch between 2D and 3D variables. Would you agree that the PVariableWidget should be setting a default 2D/3D variable when this switch is made?

I don't believe it should be the GUI's job to fix the params database.

Thanks Stas, I really appreciate it.
I'm not sure if this fixes the problem though.  If we follow the use case that Sam wrote, the user is now allowed to use the 3D variable P as a bias variable when performing 2D flow, which will break.  I think you know this though as you stated above.

I don't believe it should be the GUI's job to fix the params database.

Do you think that FlowParams should be setting its own defaults then?  I can't remember if it has a notion of whether it's operating on 2D vs 3D variables, but if it does, maybe this is what's needed.


[JENKINS-70869] New option “clone” option to SCM-based retrievers to avoid workspace/xxx@libs/

JENKINS-70869
With this new option you can avoid locking and updating the workspace/xxx@libs/ subdirectory at the job level; instead, the SCM is checked out directly into the build directory. This is particularly practical in combination with Git shallow clones. Reduces disk usage somewhat; may avoid contention in heavily concurrent jobs (since no lock is needed during SCM operations); and is generally simpler, bypassing issues such as that solved by https://github.com/jenkinsci/workflow-cps-global-lib-plugin/pull/115.
Tested interactively with a large private library used by @cloudbees with >1600 commits and >200 files, using some combinations of both plain Git SCM source (ambient authn) and GitHub SCM source (PAT), with and without the new clone mode, with and without shallow clone option when using the new mode, and with and without caching. With clone mode on and using a shallow clone with GH PAT, could retrieve the lib for a build in a couple seconds in some cases (variable depending on GH response time), whereas a deep clone would typically take at least ten seconds.

[X] basic impl
[X] GUI config
[X] tests
[x] pack files into a JAR to reduce inode count? #57
[x] backport some improvements from #57 so this is independently mergeable

Could be merged independently but I did a bunch of follow-on changes relating to this in #57 and it seems more straightforward to just merge them together (pending review).

an incremental build of this

https://repo.jenkins-ci.org/incrementals/io/jenkins/plugins/pipeline-groovy-lib/649.v4a_8971b_1a_d8b_/ is getting published normally. I am not clear on what https://github.com/MarkEWaite/docker-lfs/commit/a7754553612cad5d44a961024eb6ec4274219875 was picking up, as https://github.com/jglick/pipeline-groovy-lib-plugin/commit/c722f7acf8c2 does not exist.

an incremental build of this

https://repo.jenkins-ci.org/incrementals/io/jenkins/plugins/pipeline-groovy-lib/649.v4a_8971b_1a_d8b_/ is getting published normally. I am not clear on what MarkEWaite/docker-lfs@a775455 was picking up, as jglick@c722f7acf8c2 does not exist.

As far as I could tell from the jobs, 652 was in build 6 but not published as an incremental. I downloaded it from the build 6 artifacts and used it directly because it was not available as an incremental.  I wanted to be sure that I was testing the most recent build of the pull request.
Then I fell back to use 649 in plugins.txt because it was available as an incremental.  That's what I'm running now, on the assumption that differences between 649 and 652 are not critical to the function of the pull request.

not published as an incremental

Probably predated disabling of PR-merge mode.

In case someone is looking for how to use this ( like me )
in jenkins system config, global shared lib config:

config-as-code


[Question] Finetuning of BERT and GPT-2

Is it possible to finetune BERT and GPT-2 on my own corpus? Thanks!

BERT, you could fine-tune by setting trainable True, API Document
GP2 can't finetune for now, maybe will add finetune feature later.

For BERT, is it possible to do unsupervised language model fine-tuning? For example, pre-training BERT on medical documents

For BERT, is it possible to do unsupervised language model fine-tuning? For example, pre-training BERT on medical documents

If you want to train BERT, just use the keras-bert. After training language model with keras-bert, load pre-trained language model in kashgari for training classification/labeling model.
For medical documents, here is a pre-trained model called BioBERT, maybe you can use this.

BioBERT: a pre-trained biomedical language representation model for biomedical text mining


redmine 3.0 support

there is mass-assign problem
eq: WARNING: Can't mass-assign protected attributes for ReleaseLogQueue: name, title_template, email_notification_recipients, group_by_issue_type
I think it's related to #2

Same issue with Redmine 3.0.1
Started POST "/release_log_configurations" for <masked> at 2015-04-05 23:47:21 +0200
Processing by ReleaseLogConfigurationsController#create as HTML
  Parameters: {"utf8"=>"?", "authenticity_token"=>"<masked>", "release_log_configuration"=>{"project_id"=>"3", "enabled"=>"1", "release_log_queue_id"=>"", "email_notification_recipients"=>"<masked>"}, "commit"=>"Create"}
  Current user: <masked> (id=8)
WARNING: Can't mass-assign protected attributes for ReleaseLogConfiguration: project_id, enabled, release_log_queue_id, email_notification_recipients

  Rendered plugins/release_logs/app/views/release_log_configurations/_form.html.erb (3.5ms)
  Rendered plugins/release_logs/app/views/release_log_configurations/new.html.erb within layouts/base (3.9ms)
  Rendered plugins/redmine_code_review/app/views/code_review/_html_header.html.erb (0.2ms)
  Rendered plugins/redmine_pretend/app/views/user/_unpretend.html.erb (0.2ms)
  Rendered plugins/clipboard_image_paste/app/views/clipboard_image_paste/_headers.html.erb (0.5ms)
  Rendered plugins/sidebar_hide/app/views/sidebar/_hideButton_partial.html.erb (0.4ms)
  Rendered plugins/redmine_code_review/app/views/code_review/_body_bottom.html.erb (0.1ms)
  Rendered plugins/clipboard_image_paste/app/views/clipboard_image_paste/_add_form.html.erb (3.2ms)
Completed 200 OK in 59ms (Views: 46.6ms | ActiveRecord: 1.8ms)

I tested the version 0.1.1 on Redmine 3.1.0 and got it work, but need to apply some changes because of new rails strong_params.

Thank you @phoebusnetto, I will proceed with what it needs to be done soon.

Support for 3.0 added.


Missing release-schema.json

Hello!
I noticed your project does not include a release-schema.json file. This is an important metadata file needed to make use of the schema in automated tools, please do create it. :+1:
Kind regards

This extension does not consider the addition of values within additionalProcurementCategories array. I am not sure how we have to build this file. Could you explain me?

@antonioherrera puedes agregar un release-schema.json vacío

This can be closed – though the release-schema.json file can simply be omitted if empty.


Turn off snippets 5000

Fixes #22393 . If you want to "comment it out", this would probably work. If you want to remove it completely, then we should just remove the 4 files added in https://github.com/dotnet/AspNetCore.Docs/pull/21396.

@gewarren even after merge to live still getting failures
see https://github.com/dotnet/AspNetCore.Docs/pull/22391


Is the detail loss normal？

Hello, I'm trying to write my own code to reproduce your paper. I output the sum of boundary losses（boundery_bce_loss + boundery_dice_loss） as shown in the figure below. It seems that its value will not change soon. Is this normal?

Hello, I'm trying to write my own code to reproduce your paper. I output the sum of boundary losses（boundery_bce_loss + boundery_dice_loss） as shown in the figure below. It seems that its value will not change soon. Is this normal?

我看了下日志，整体走势跟你的差不多，应该是正常的

非常感谢！！！


My i use this and codechickencore in a modpack im making

Pm on curse with answer if you could

Taken from Chicken Bones post on MinecraftForum.net:
" ModPacks:
You are welcome to include any of my mods in modpacks as long as you provide credit. My name and a link to this page is fine "

okey dokey

Please close this again..


sw_engine Raster: Improving image quality in image scale down

An average value sampled based on adjacent pixels of the target pixel is used.
[Before / After / ubuntu image viewer ]

[Before / After ]

@JSUYA still draft?

I think image raster needs 3 scenarios.

Arbitrary Tranfsformed (Required texture mapping based algorithm)
Just Scaled Up/Down => Simple Bilinear (In that case do we really need up/down separate methods?...)
Original => Immediate Drawing.

I'm expecting scenario 2 which is the general use-case we can focus on in this enhancement...

You want to apply bilinear interpolation to both cases "Up/Down scale",
but I can't figure out how to properly apply it in Down scale.
so.. In down scale, adjacent pixels are lost according to the scale ratio, so I averaged them.
I hope this approach is appropriate.
Please review again

@mgrudzinska It needs more beholders, Could you please review this also?

Of course, I'll review this patch

if it's not very urgent, I'd like to read a bit about scaling before I finish my rev

I've made some tests:

scaling << 1 - one can see a big improvement after this patch:

1 > scaling > 0.5 - not much changes when compared to the current version of TVG, results from the viewer are better
we could consider some other algorithm in such cases

I'd use bilinear interpolation for the cases when scaling is in the range 0.5-1.
but I don't have args from literature for that. It seams to be reasonable though - we use 4 pixels in both algorithms. the only difference is, that in the one for the downsampling used in this patch, the weight for these 4 pixels are equal, and in the bilinear interpolation they are not.
@JSUYA what do you think of such an approach?

I'd use bilinear interpolation for the cases when scaling is in the range 0.5-1.
but I don't have args from literature for that. It seams to be reasonable though - we use 4 pixels in both algorithms. the only difference is, that in the one for the downsampling used in this patch, the weight for these 4 pixels are equal, and in the bilinear interpolation they are not.
@JSUYA what do you think of such an approach?

wow thanks for your good idea.  As you said, when 0.5 - 1 down scaling, sampling algorithms doesn't make much sense.
I will improve to apply bilinear interpolation when the scale is 0.5 - 1.
It may take a little more time to change the code :)

I did some tests. And as a result, I modified to use bilinear interpolation when scale is down(0.25) ~ up, and to use average pixel only in 0.25 or less.
70%

30%

10%

@mgrudzinska Thank you for check.
I thought that the effect was small because the value of the halfScaling(average2Nx2NPixel()) referenced when the downscale value was 0.25f ~ 0.5f is 1. (at least in my resource)
I found the sample image you used (google image search) and tested it.
I changed condition to 0.5f.
If you have any good ideas to improve this, please share to me :)

for me it's ready to merge, unless @hermet wants to take a look


Databases with no label don't appear in sidebar

If a database is created without a label, it will not appear in the sidebar at all. It would be nice if it would use the ID if there is no label.

fixed in prod


Docs: Quarkus 2.x migration before/after examples

Converted from note, added for issue-tracking

Closed by linked issues


Added two new paragraphs fixed conflict

Added your first flood and storing water for a rainy day [#2]

I have done the final pull request, thanks for the help.
On Wed, Jun 5, 2024 at 2:15 PM Abner Bogan @.***> wrote:

Merged #2
https://github.com/CUAHSI-GitFlows-Learners/learning-gitflows-jackhazard00/pull/2
into main.
—
Reply to this email directly, view it on GitHub
https://github.com/CUAHSI-GitFlows-Learners/learning-gitflows-jackhazard00/pull/2#event-13056315662,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BABM2LSHVVTC4XLMTIOQE2DZF5PW3AVCNFSM6AAAAABIZDUYNWVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGA2TMMZRGU3DMMQ
.
You are receiving this because you authored the thread.Message ID:
<CUAHSI-GitFlows-Learners/learning-gitflows-jackhazard00/pull/2/issue_event/13056315662
@github.com>


Add contributing guide

Codecov Report

Merging #4 (7df3fd5) into main (d5e809e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #4   +/-   ##
=======================================
  Coverage   89.22%   89.22%
=======================================
  Files          18       18
  Lines         464      464
  Branches       29       29
=======================================
  Hits          414      414
  Misses         34       34
  Partials       16       16

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 d5e809e...7df3fd5. Read the comment docs.


Docs tab is not working in Storybook with Vite.js

Describe the bug
I have set up the Storybook in my vite.js project as per the instructions provided in this official article. Storybook started perfectly in 6006 port. But the problem is the Docs tab throws me this error when I switch to it.

To Reproduce
You can use this to reproduce that issue.
https://github.com/yushanwebdev/storybook-with-vite/tree/storybook-issue
Steps to Reproduce

Clone the repo.
Switch to the storybook-issue branch.
Run yarn
Run yarn run storybook
Select the Primary story in the Button
Select the Docs tab

System
Environment Info:
  System:
    OS: macOS 12.1
    CPU: (8) arm64 Apple M1
  Binaries:
    Node: 17.2.0 - /opt/homebrew/bin/node
    Yarn: 1.22.17 - /opt/homebrew/bin/yarn
    npm: 8.1.4 - /opt/homebrew/bin/npm
  Browsers:
    Chrome: 97.0.4692.99
    Firefox: 96.0.2
    Safari: 15.2
  npmPackages:
    @storybook/addon-actions: ^6.4.15 => 6.4.15
    @storybook/addon-essentials: ^6.4.15 => 6.4.15
    @storybook/addon-links: ^6.4.15 => 6.4.15
    @storybook/react: ^6.4.15 => 6.4.15

Additional context
I have tried the solutions mentioned in https://github.com/storybookjs/storybook/issues/10982 also. But didn't work out.

I'm not sure of the exact underlying issue but the ESM version of react-popper-tooltip is not being used in your reproduction which is causing the error. The ESM version is being used in this repo and everywhere else I use the builder. Without fixing the underlying issue the easiest way to fix this is to add viteFinal to your main.js e.g.:
  async viteFinal(config){
    config.optimizeDeps.include = [...config.optimizeDeps.include, "react-popper-tooltip"];

    return config;
  }


allow for object groups.

Object groups make it easy to duplicate complex objects consisting of multiple objects. Object groups in SVG terms are just a lot of rects with a wrapper around them

@fabulousduck groups work for me after #423 Consider this svg: groups.svg.txt
Levels compiles correctly. Game runs the game as expected. What exactly do you expect from groups?

it seems that this is only a bug for rects that have not had their names changed. This happens for me only when i init a new rect, group it with another.

@fabulousduck could you please provide the bugged svg?

@fabulousduck your problems might be caused by the transformations applied on the rects. Search for transform="scale(-1,1)" in the svg text. Inkscape likes to insert them when you manipulate several objects at once. Unfortunately we don't support that. I'd recommend to avoid selecting several objects and trying to do transformations on them for now.

It would be nice to have a tool that sort of "normalizes" svg by applying all of the transformation leaving rects with absolute coordinates, but I don't know any of such tools.

@fabulousduck you may find this useful https://stackoverflow.com/questions/13329125/removing-transforms-in-svg-files Maybe we should even document that.

I could probably whip up a small regex that does that next weekend when i have time for it. doesn't sound like the hardest problem in the world

It would be nice to have a tool that sort of "normalizes" svg by applying all of the transformation leaving rects with absolute coordinates, but I don't know any of such tools.

If I don't remember wrong there exists a few JavaScript based ones. Svgo is the one that comes to mind, but I think there might be others as well.

Superseded by #592


RTView: extended notifications

Corresponds to previous issue CAD-2482.
Currently, only email notifications are supported. We should add another ways as well: for example, messengers.

Closing this. If this is still relevant please reopen.


Blade Runner RPG Official Sheet Release

Submission Checklist
Required

[X] The pull request title clearly contains the name of the sheet I am editing.
[X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[X] The pull request makes changes to files in only one sub-folder.
[X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is:  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is:  (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is:  (i.e. Dungeons & Dragons, FATE)

[X] I have followed the Character Sheets Standards when building this sheet.

[X] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Initial release of character sheet including:

Handling for pushing rolls
Compendium drag and drop integration
A Vehicle sheet
Shift and day tracker
Downtime tracking

Character Sheet Info Roll20 Internal Use only.


a dialog which do not show the code

System information:

Operating system (distribution) and version :macOS 11.0.1
DBeaver version:<IP_ADDRESS>011291229
Java version:JDK15
Additional extensions

Connection specification:

Database name and version
Driver name
Do you use tunnels or proxies (SSH, SOCKS, etc)?

Describe the problem you're observing:
Steps to reproduce, if exist:
Include any warning/errors/backtraces from the logs

Hi @beiyinghu ,
can you provide some steps to reproduce it?
Which database do you use?

Fixed in the scope of #10535. The fix is already available in the recent EA build.


Support reading tiles across projection bounds

I think an example of this would be creating a 4326 tile that goes up to 90 Latitude from a web mercator raster.

blocked by https://github.com/DanielJDufour/geowarp/issues/27


DCOS-15921 (3 of 4): Implement ComponentExample component for design system

This PR adds the ComponentExample component, which is made up of the CodeExampleHeader and CodeExampleFooter components. It is the primary component for the code example component.
This PR depends on: #2312, #2313
To Test:

See branch tommy/design-system/component-example-ace-editor for the complete implementation.
In the UI: Design System --> Components --> Checkboxes --> Code Tab
React code is shown on one tab and HTML on the other
Clipboard function works for both tabs
Collapse/Expand of code

Optional properties:
The optional properties are:

only: accepts one of preview, react and html
maxLines: max number of lines the code block can show

Test optional properties:

preview hides the code block
react shows only react code
html shows only html code
maxLines should result in truncation if the number of lines in the code block exceeds it

Checklist

[x] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

As discussed in today's meeting, there needs to be some changes in the design of the component example. I will be closing this PR and creating a new one for the changes.


how to access a remote shared folder with credential in asp.net core ?

i  want to access a remote shared folder with credential,neither my app server nor remote file sever is in domain.
i notice that asp.net core use  under code to specify a local path:
app.UseStaticFiles(new StaticFileOptions()
{
    FileProvider = new PhysicalFileProvider("C:\\Files"),
    RequestPath = "/files"
});

then i can browser localhost:5000/files/test.jgp to view a file.
but now i need to aceess a  remote server file with credential,.
i instead of the '"C:\Files' to '\\<IP_ADDRESS>\files',unfortually,it thow an exception: The directory name \\<IP_ADDRESS>\files is invalid.
detail error shows under:
System.ArgumentException: The directory name \\<IP_ADDRESS>\files is invalid.
Parameter name: path
   at System.IO.FileSystemWatcher..ctor(String path, String filter)
   at System.IO.FileSystemWatcher..ctor(String path)
   at Microsoft.Extensions.FileProviders.PhysicalFileProvider.CreateFileWatcher(String root)
   at WU.OnlineReport.Web.Startup.Startup.Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory loggerFactory) in D:\work\Report.Web\Startup\Startup.cs:line 80
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.AspNetCore.Hosting.ConventionBasedStartup.Configure(IApplicationBuilder app)
   at Microsoft.AspNetCore.Server.IISIntegration.IISSetupFilter.<>c__DisplayClass3_0.<Configure>b__0(IApplicationBuilder app)
   at Microsoft.AspNetCore.Hosting.Internal.AutoRequestServicesStartupFilter.<>c__DisplayClass0_0.<Configure>b__0(IApplicationBuilder builder)
   at Microsoft.AspNetCore.Hosting.Internal.WebHost.BuildApplication()

How can I resolve this problem ?

For IIS hosted apps: https://stackoverflow.com/questions/1496068/asp-net-read-files-from-a-password-protected-network-share

@Tratcher  thanks! I'm at home now and have no dev enviroment.I'll try it next monday.

@tangname running your web app as admin is very dangerous. Find a more restricted account you can use.


Enhancements to plonky2 verifier: range check and circuit size optimization

Problem

There were soundness issues due to the lack of range checks.
The circuit size was large, requiring a huge amount of RAM.

Accomplishments
Implemented ArithmeticChip
I implemented an ArithmeticChip that imposes the constraint for computing modulo the order p of the goldilocks field: a*b + c = p*q + r. This includes an 80-bit range check for the quotient q and a 64-bit range check for the remainder r. Although there is no constraint for r < p, I believe this will not lead to any soundness issues. This chip also imposes the constraint of a*b + c = p*q + r on extension field.
Implemented PoseidonBn254Chip
The computational bottleneck in the plonky2 verifier is the calculation of the poseidon hash used in FRI and challenger, which requires about 3000 poseidon permutations. The original poseidon hash is based on the goldilocks field, which, being a non-native field, results in massive and inefficient modulo calculations. Therefore, I implemented Bn254PoseidonGoldilocksConfig for plonky2, which uses poseidon hash on bn254 for FRI and challenger. I also reduced the number of constraints for the poseidon hash on bn254 by making its partial and full round into a single custom gate in halo2.
Added tests for plonky2 gate chip implementations
I added tests in src/snark/chip/plonk/gates.
Migration from snark-verifier to halo2-solidity-verifier
I migrated the on-chain verifier for halo2 from snark-verifier to halo2-solidity-verifier. This resolved issues with contract size limits and made it possible to introduce ArithmeticChip and PoseidonBn254Chip.

I think we should update readme now. I will create docs folder to keep the technical docs classified by the applied changes. Since this PR brings huge change, we now need new docs for the work!


Caching swift templates

@kzaher if you have time, it would be better for you to review it since you are the author :)

          1 Error

:no_entry_sign:
Any changes to library code need a summary in the Changelog.

          1 Warning

:warning:
PR is classed as Work in Progress

  Generated by :no_entry_sign: danger

@kzaher @krzysztofzablocki any ideas how to make swiftc link frameworks? I've tried -F and -framework but it feels like they have no effect. Maybe I miss some other flags to make it work, but I could not find any inspecting Xcode build commands.
I was able to build template with SPM swift build. But this requires not a framework binary but module build artefacts.

swiftc worked when linked agains a framework that I set up from scratch with SPM. It was originally missing modules artefacts 🎉

@krzysztofzablocki @kzaher can you please test my latest changes locally? I think it should work much faster now 🎉

Kudos to @kylef for this helpful tip 🙇

@krzysztofzablocki have some troubles here making SPM build work again. First it made me to move all the files to the single folder... Then I added a framework to targets in Package.swift, like this:
targets: [
        Target(name: "sourcery", dependencies: ["SourceryFramework"]),
        Target(name: "SourceryFramework"),
    ]

And this gives error, descriptive as any Apple error:
error: these referenced modules could not be found: SourceryFramework
fix: reference only valid modules

Hi @ilyapuchka ,
I believe for you to be able to write something like
targets: [
        Target(name: "sourcery", dependencies: ["SourceryFramework"]),
        Target(name: "SourceryFramework"),
    ]

You need to have SourceryFramework named directory under Sources that contains all of *.swift files in a flat structure, no subfolders, no other files.
Now I see this

I also get some other compiler errors while trying to run the tests on 172ec963f20660aee1d680f72d0b05af1bb1071b.

That's exactly what I did, tried both with simlynk (like for sourcery module) and just copying all the sources directly. I also have fixed compiler errors after merge. I've push all of that, if you have a chance to look at it deeper would be cool

@ilyapuchka now I see

Doesn't look like this will compile 😀 Perhaps symlink gone wrong 😀 ¯\_(ツ)_/¯
I can probably hack a script that rebuilds this hierarchy if you like. We do this for RxSwift. Not the prettiest solution, but works :)

That was actually an issue, with proper symlink it works 🎉

Ha, interesting. I think those subfolders might be treated as submodules. Probably documented somewhere ¯\_(ツ)_/¯
Just one more thing :)

Yes, if there is a folder in Sources with no main.swift it's treated as a framework module.
Regarding the error that you get you probably need to update yaml (I guess you use it as you don't provide any cli params, right?) file as the paths now are relative to its location

I was trying to get it to print help page so I can figure out the arguments :)

./.build/debug/sourcery --help will do that.
There is in fact configuration file checked in already and by default we look at the current path and if there is config file we ignore cli arguments. Maybe we should change that and always use provided cli arguments. I've updated this config any way, so now it should work with just ./.build/debug/sourcery after you remove ejs template ofc.
Can you take another look at the changes when you have time? I'd like to work on some other features for swift templates like #275 or #243 but it would be better if they will be done on top of that to avoid more conflicts.

@krzysztofzablocki can we merge this? 🙏🏻

go ahead, just squash it 👍


completed-docs: methods and properties on interfaces are checked.

PR checklist

[x] Addresses an existing issue: #3534
[x] New feature, bugfix, or enhancement

[x] Includes tests

[x] Documentation update (no changes required)

Overview of change:
Properties and methods in interfaces were not being analyzed by the completed-docs rule. Now they are.
Is there anything you'd like reviewers to focus on?
Nope.
CHANGELOG.md entry:
[enhancement] completed-docs: check properties and methods in interfaces

bump
Any chance of this getting reviewed/merged/released anytime soon? The rule is quite useless without this fix.

When will the next version of tslint be released with this fix? It's been quite a long time since the last release.

FYI to anyone that finds this, version 5.12.0 includes this change.


Deprecated dependency: request

As of Feb 11th 2020, request is fully deprecated.
— npm: request

Alternative libraries have been suggested: request/request#3143

PRs accepted. :)

Looks like this was closed in #32?

Closed in d15b0c5fbb96877c2652a6d65be4b8f707da8f02. :)


electron-eval error: Electron process exited with code 2

UNEXPECTED ERROR: If this is a bug in WebTorrent, report it!
OPEN AN ISSUE: https://github.com/feross/webtorrent/issues

DEBUG INFO: webtorrent-cli 1.9.0, webtorrent 0.98.10, node v7.7.1, linux x64, exit 1
events.js:163
      throw er; // Unhandled 'error' event
      ^

Error: electron-eval error: Electron process exited with code 2
    at ChildProcess.<anonymous> (/home/example/.nvm/versions/node/v7.7.1/lib/node_modules/webtorrent-hybrid/node_modules/electron-eval/lib/index.js:174:22)
    at emitTwo (events.js:106:13)
    at ChildProcess.emit (events.js:194:7)
    at maybeClose (internal/child_process.js:899:16)
    at Socket.<anonymous> (internal/child_process.js:342:11)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:191:7)
    at Pipe._handle.close [as _onclose] (net.js:504:12)

This happens when I seed more than one file. One usually seems to work ok, but if i have 2-3 it crashes frequently. Probably a memory/resource issues -- its on a $5 vps.

If you have to seed many files, I recommend using http web seeds instead of webtorrent-hybrid. They will be much more reliable.

I am confused. isn't webtorrent a cli torrent client? It can only seed one file at a time?

WebTorrent isn't a CLI torrent client. It's a JavaScript torrent client that has many different frontends. It can be used via an official CLI, a desktop app, or a <script> tag in a browser.
The CLI only supports one torrent at a time. Use the desktop app if you want to seed multiple torrents.

gotcha. i thought cli was a full blown torrent client?

Yes, sorry. If someone else wants to build a full-featured CLI torrent client on top of the webtorrent engine, they should feel free. It's not something I want to do. Since most users want a GUI, I'd rather put my effort there.

Makes sense. Dont' mean to bombard you with all these questions -- but this project looks very promising and I'm sure others will have same questions, so at least there is a record.


Weather skill throws TypeError

This doesn't seem like a parser mistake.
Full conversation:
Almond: Hello! How can I help you?
User: what is the weather
Almond: Sorry, I had an error processing your command: Invalid parameter types Enum(raining,cloudy,sunny,snowy,sleety,drizzling,windy), Enum(raining,cloudy,sunny,snowy,sleety,drizzling,windy,foggy) for ==.

Full log:
2/10/2021, 3:58:16 PM: Initializing dialogue policy for devices:
2/10/2021, 3:58:16 PM: --- DEPTH 0
2/10/2021, 3:58:16 PM: depth 0 took 0.00 seconds
2/10/2021, 3:58:16 PM: --- DEPTH 1
2/10/2021, 3:58:16 PM: depth 1 took 0.00 seconds
2/10/2021, 3:58:16 PM: --- DEPTH 2
2/10/2021, 3:58:16 PM: depth 2 took 0.00 seconds
2/10/2021, 3:58:16 PM: --- DEPTH 3
2/10/2021, 3:58:16 PM: depth 3 took 0.00 seconds
2/10/2021, 3:58:16 PM: --- DEPTH 4
2/10/2021, 3:58:16 PM: depth 4 took 0.00 seconds
2/10/2021, 3:58:16 PM: --- DEPTH 5
2/10/2021, 3:58:16 PM: depth 5 took 0.00 seconds
2/10/2021, 3:58:16 PM: --- DEPTH 6
2/10/2021, 3:58:16 PM: depth 6 took 0.00 seconds
2/10/2021, 3:58:16 PM: --- DEPTH 7
2/10/2021, 3:58:16 PM: depth 7 took 0.00 seconds
2/10/2021, 3:58:31 PM: Analyzed message into @org.thingpedia.weather . current ( ) ;
2/10/2021, 3:58:31 PM: Batched dataset request for org.thingpedia.weather
2/10/2021, 3:58:31 PM: TypeError: Invalid parameter types Enum(raining,cloudy,sunny,snowy,sleety,drizzling,windy), Enum(raining,cloudy,sunny,snowy,sleety,drizzling,windy,foggy) for ==
2/10/2021, 3:58:31 PM: at TypeChecker._resolveOverload (/opt/thingengine/node_modules/thingtalk/dist/typecheck.js:325:15)
2/10/2021, 3:58:31 PM: at process._tickCallback (internal/process/next_tick.js:68:7)

I cannot reproduce this on my account. Is that the latest version of the weather skill (version 9)?


Notification header text v1.12.1

Gitea version (or commit ref): 1.12.1
Git version: 2.18.4
Operating system: centos 8
Database (use [x]):

[X] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[X] Not relevant

Log gist:

Description
After updating from 1.11.5 to 1.12.1 I get the following for notifications:
Screenshots

Are you using custom header template?

oh right! Forgot about that! :) Thats it! :)


Verify that ConfigOptions only use supported primitives

After removing Kryo, Titan does no longer support serializing arbitrary objects. All such instances have been removed inside the core graph module, but there might be cases where the configuration system relies on non-supported classes.
For instance, BerkeleyJE used some ConfigOption instances that made references to BerkeleyJE specific classes (e.g. LockMode, IsolationLevel). Those need to be replaced by String variants (or other suitable data types). See BerkeleyJEStoreManager#ISOLATION_LEVEL as an example and utilize ConfigOption.getEnumValue() has a helper method.
Please go through all ConfigOption and ensure that they only use objects explicitly supported by the serializer.

@mbroecheler StandardSerializer maps some enum types to a simple serializer that avoids the need for ConfigOption.getEnumValue:
        ...
        registerClassInternal(45,Order.class, new EnumSerializer<>(Order.class));
        registerClassInternal(46,Multiplicity.class, new EnumSerializer<>(Multiplicity.class));
        registerClassInternal(47,Cardinality.class, new EnumSerializer<>(Cardinality.class));
        registerClassInternal(48,Direction.class, new EnumSerializer<>(Direction.class));
        registerClassInternal(49,ElementCategory.class, new EnumSerializer<>(ElementCategory.class));
        registerClassInternal(50,ConsistencyModifier.class, new EnumSerializer<>(ConsistencyModifier.class));
        registerClassInternal(51,SchemaStatus.class, new EnumSerializer<>(SchemaStatus.class));
        registerClassInternal(52,LogTxStatus.class, new EnumSerializer<>(LogTxStatus.class));
        registerClassInternal(53,MgmtLogType.class, new EnumSerializer<>(MgmtLogType.class));
        registerClassInternal(54,Timestamps.class, new EnumSerializer<>(Timestamps.class));
        registerClassInternal(55,TimeUnit.class, new EnumSerializer<>(TimeUnit.class));
        registerClassInternal(56,Mapping.class, new EnumSerializer<>(Mapping.class));
        ....

Most of those are Titan or TP3 enums, though at least one (TimeUnit) is part of Java.
Why do you want to move to ConfigOption.getEnumValue(string, enumclass) instead of just adding additional type mappings to StandardSerializer for the rest of the enums required by Titan, and leaving getEnumValue for third-party extensions?

I want to avoid cyclic dependencies. Hence, anything that is not titan-core should not use enums for serialization. That was the problem with BerkeleyJEStoreManager#ISOLATION_LEVEL - to add that to the serializer would have introduced a cyclic dependency (or some ugly reflection business).
So, within titan-core we should register the enums. Outside, we should use the helper method to go through string.


Running docker in systemd-nspawn

I was trying to start the container in a systemd-nspawn container where /dev/kvm is bind mounted inside.
Inside the jail i run kvm-ok:
root@debian:/mnt/safe/docker/compose/windows$ kvm-ok
INFO: /dev/kvm exists
KVM acceleration can be used

The Jail is the docker host with the following config:
version: "3"
services:
  windows:
    image: dockurr/windows
    container_name: windows
    devices:
      - /dev/kvm
    cap_add:
      - NET_ADMIN
    ports:
      - 8006:8006
      - 3389:3389/tcp
      - 3389:3389/udp
    stop_grace_period: 2m
    restart: on-failure
    volumes:
      - /mnt/tank/all/kvm/win:/storage

But when i start the container the VM just shuts down:
1933312K ........ ........ ........ ........ 41% 65.1M 37s

1966080K ........ ........ ........ ........ 42% 69.9M 36s

1998848K ........ ........ ........ ........ 42% 33.9M 36s

...

4718592K ........ ........ ........ ........ 99% 69.3M 0s

4751360K .....                              100% 38.2M=62s

❯ Extracting Windows 11 bootdisk...

❯ Extracting Windows 11 environment...

❯ Extracting Windows 11 setup...

❯ Extracting Windows 11 image...

❯ Adding XML file for automatic installation...

❯ Building Windows 11 image...

❯ Creating a 64G growable disk image in raw format...

❯ Booting Windows using QEMU emulator version 8.2.1 ...

3h3h3hBdsDxe: failed to load Boot0002 "UEFI QEMU QEMU HARDDISK " from PciRoot(0x0)/Pci(0xA,0x0)/Scsi(0x0,0x0): Not Found

BdsDxe: loading Boot0001 "UEFI QEMU QEMU CD-ROM " from PciRoot(0x0)/Pci(0x5,0x0)/Scsi(0x0,0x0)

BdsDxe: starting Boot0001 "UEFI QEMU QEMU CD-ROM " from PciRoot(0x0)/Pci(0x5,0x0)/Scsi(0x0,0x0)

❯ Shutdown completed!

How can i get more logs or what do i miss? Is it not possible to run with bind mounted /dev/kvm or nspawn-jail?
I also installed qemu-kvm inside the jail:
sudo apt -y install qemu-kvm libvirt-daemon  bridge-utils virtinst libvirt-daemon-system

System: TrueNAS-Scale; Ryzen 1600x, SVM enabled
JailOS: Debian Bookworm

So it did some digging and found out that the the container also does need the device /dev/vhost-net.
I found this while looking at a different docker-kvm project in the troubleshoot section:
https://github.com/BBVA/kvm?tab=readme-ov-file#notes--troubleshooting
So what i did was running bind mounting  /dev/vhost-net into the nspawn-container and then qemu was able to boot up.
The weird thing is when i enable the console for more debug like @kroese mentioned:
environment:
  CONSOLE: "Y"

The VM would not start and only the following logs appeared when CONSOLE: "Y"  :
❯ Starting Windows for Docker v2.05...

❯ For support visit https://github.com/dockur/windows

❯ Booting Windows using QEMU emulator version 8.2.1 ...

char device redirected to /dev/pts/0 (label serial0)

... and when i remove it, it works but the logs are kind of empty:
No log line matching the '' filter

I guess there is some kind of bug with that environment variable.
I don't know if this issue is only on my part with my special setup or this should be a separate issue.

You are right @kroese - it could not use mknot to create the device because the nspawn container needs the rights to do so.
I removed the bind mount of the device and gave it the rights to rwm of /dev/vhost-net like in: systemd-nspawn container with '--property=DeviceAllow=/dev/vhost-net rwm' and the windows VM boots up without issues!

Ok so I tried to reproduce the problem with 2.05 and tested with winxp and I could not reproduce it.
I tried to install win11  again with the --property='DeviceAllow=/dev/vhost-net rwm' but still does not work - mhm, maybe it is only an installation issue or previously I only tried with winxp because it is faster to test - sadly I don't remember anymore.
So it is at least a win11 issue.
The statement that it works with --property='DeviceAllow=/dev/vhost-net rwm' and win11 is false!
Next test was instead --property='DeviceAllow=/dev/vhost-net rwm', bind mounting --bind=/dev/vhost-net, but sadly same issue.
Next was --capability=all without  --property='DeviceAllow=/dev/vhost-net rwm' or --bind=/dev/vhost-net -> not working.
Next was --capability=all, --property='DeviceAllow=/dev/vhost-net rwm' and  --bind=/dev/vhost-net -> not working.
I also did modprobe vhost_vsock on the jail-host because I remember doing that, but no.
Then I tested all bind mounts listed in this + vhost-net:
--bind=/dev/kvm --capability=all --bind=/dev/vhost-net --bind=/dev/fuse --bind=/dev/vsock --bind=/dev/vhost-vsock

-> not working.
I tried with winxp with same settings to make sure this at least still works, and it did.
Then I also tried to bind vhost-net in the dockerfile with win11 again like:
devices:
  - /dev/kvm
  - /dev/vhost-net

And still does not work. So I am not sure anymore what all I did in my testing but I can't get it running anymore with 2.05 with win11, and I do not remember exactly if it ever run with with11 - so there is that.
Then I tired with all the previous mentioned settings win10 and it did not work.
With win8 it did work! So something beginning with win10 is the issue.
I am sorry that I tested wrong (changing more than one setting(winxp)).
Then i did a docker compose pull and restarting the container with win8
First boot did not work after recreation - but then i restarted the container again and it did boot. I don't know, maybe a fluke?
❯ Starting Windows for Docker v2.07...

❯ For support visit https://github.com/dockur/windows

❯ Booting Windows using QEMU emulator version 8.2.1 ...

3h3h3hBdsDxe: loading Boot0003 "Windows Boot Manager" from HD(1,GPT,3C9F9213-A787-474C-8772-B5A6D80C1E6B,0x800,0x40000)/\EFI\Microsoft\Boot\bootmgfw.efi

BdsDxe: starting Boot0003 "Windows Boot Manager" from HD(1,GPT,3C9F9213-A787-474C-8772-B5A6D80C1E6B,0x800,0x40000)/\EFI\Microsoft\Boot\bootmgfw.efi

❯ Shutdown completed!

❯ Starting Windows for Docker v2.07...

❯ For support visit https://github.com/dockur/windows

❯ Booting Windows using QEMU emulator version 8.2.1 ...

3h3h3hBdsDxe: loading Boot0003 "Windows Boot Manager" from HD(1,GPT,3C9F9213-A787-474C-8772-B5A6D80C1E6B,0x800,0x40000)/\EFI\Microsoft\Boot\bootmgfw.efi

BdsDxe: starting Boot0003 "Windows Boot Manager" from HD(1,GPT,3C9F9213-A787-474C-8772-B5A6D80C1E6B,0x800,0x40000)/\EFI\Microsoft\Boot\bootmgfw.efi

Then i tried win10 again with no bind-mount and so on, but is not working with v2.07.  It also does not work with all the bind mounts and privileged: true and - /dev/vhost-net in docker compose file.
In summary all the settings did nothing. Only thing i found out is that the it only works with winxp and win8 (win7, vista, ... I did not test) and it does not work with win10 and win11
So I am fairly certain that the devices or bind mounts are not the problem - so sadly we are back to square one.


False prompt to reload window

After todays update I got a prompt to reload because of modified extensions

You were correct that there will probably be a dup -- #39765


tinirc: Wait for epair on start, set address selection policy

This applies to pots without an rc script.
Closes #201

I guess it could make sense to move the address selection policy in a different PR,
to cover all the cases: no tinirc and all stack selection (ipv4, ipv6, dual) support

@pizzamig
I removed the ip6policy changes from the pull request, so this can land.
I'll open a second one with this change afterwards:
if sysctl -n kern.features.inet6 >/dev/null 2>&1; then
	${IP6ADDRCTL_CMD} flush >/dev/null 2>&1
	cat <<EOT | ${IP6ADDRCTL_CMD} install /dev/stdin
	::1/128          50      0
	::/0             40      1
	::ffff:0:0/96   100      4
	2002::/16        30      2
	2001::/32         5      5
	fc00::/7          3     13
	::/96             1      3
	fec0::/10         1     11
	3ffe::/16         1     12
EOT
fi

Thanks!


Fix email send response handling

The send(email:) call always throws, even in the success case. This is because the API returns 202 accepted, not 200 ok in the success case.
This is why I've added a check against .accepted. To not break anything else, I've kept the check against .ok, but from my understanding it could be removed.

Is this project still maintained?


feat: add description to setting entry

Added a new optional field description that will allow to add a small explanation to the setting entry.
Sometimes the setting value is very clear but sometimes it's a little bit tricky and having an explanation close to it would facilitate its use.
Examples:

EMAIL_WHITELIST, comma separated list with the allowed email domains.
WEEK_STARTS_ON, allowed values: 0 (Monday), 1 (Tuesday), ... (6 or 7) Sunday.

Thank you for this useful PR.

@obdulia-losantos you can upgrade to 0.2.0 version.

Thank you!


internal error: entered unreachable code

thread 'api_types::export_bindings_apitypests' panicked at 'internal error: entered unreachable code', .cargo/registry/src/github.com-1ecc6299db9ec823/ts-rs-6.1.2/src/lib.rs:367:9
stack backtrace:
   0: rust_begin_unwind
             at /rustc/9ad5d82f822b3cb67637f11be2e65c5662b66ec0/library/std/src/panicking.rs:577:5
   1: core::panicking::panic_fmt
             at /rustc/9ad5d82f822b3cb67637f11be2e65c5662b66ec0/library/core/src/panicking.rs:110:14
   2: core::panicking::panic
             at /rustc/9ad5d82f822b3cb67637f11be2e65c5662b66ec0/library/core/src/panicking.rs:48:5
   3: <core::option::Option<T> as ts_rs::TS>::name
             at .cargo/registry/src/github.com-1ecc6299db9ec823/ts-rs-6.1.2/src/lib.rs:367:9
   4: <timetrackrs::api_types::ApiTypesTS as ts_rs::TS>::inline
             at ./src/api_types.rs:49:18
   5: <timetrackrs::api_types::ApiTypesTS as ts_rs::TS>::decl
             at ./src/api_types.rs:49:18
   6: ts_rs::export::generate_decl
             at .cargo/registry/src/github.com-1ecc6299db9ec823/ts-rs-6.1.2/src/export.rs:63:19
   7: ts_rs::export::export_type
             at .cargo/registry/src/github.com-1ecc6299db9ec823/ts-rs-6.1.2/src/export.rs:34:5
   8: ts_rs::TS::export
             at .cargo/registry/src/github.com-1ecc6299db9ec823/ts-rs-6.1.2/src/lib.rs:262:9
   9: timetrackrs::api_types::export_bindings_apitypests
             at ./src/api_types.rs:49:18
  10: timetrackrs::api_types::export_bindings_apitypests::{{closure}}
             at ./src/api_types.rs:49:18
  11: core::ops::function::FnOnce::call_once

here: https://github.com/Aleph-Alpha/ts-rs/blob/703cd15cde8ea9a27d555c5eb49f0bdb02ff6004/ts-rs/src/lib.rs#L367
when serializing this type:
    #[ts(export)]
    #[serde(tag = "type")]
    pub enum ApiTypesTS {
        time_range {
            request: TimeRangeRequest,
            response: Vec<SingleExtractedChunk>,
        },
        single_event {
            request: SingleEventRequest,
            response: Option<SingleExtractedEventWithRaw>,
        },
        rule_groups {
            request: (),
            response: Vec<TagRuleGroup>,
        },
        invalidate_extractions {
            request: InvalidateRangeRequest,
            response: (),
        },
        update_rule_groups {
            request: Vec<TagRuleGroup>,
            response: (),
        },
        get_known_tags {
            request: (),
            response: Vec<String>,
        },
        timestamp_search {
            request: TimestampSearchRequest,
            response: Option<Timestamptz>,
        },
    }

The macro-expanded code there has stuff like this:
let res = ::alloc::fmt::format(::core::fmt::Arguments::new_v1(
    &["", "", ": ", ","],
    &match (
        &"response",
        &"",
        &<Option<SingleExtractedEventWithRaw> as ts_rs::TS>::name(),
    ) {
        _args => [
            ::core::fmt::ArgumentV1::new(_args.0, ::core::fmt::Display::fmt),
            ::core::fmt::ArgumentV1::new(_args.1, ::core::fmt::Display::fmt),
            ::core::fmt::ArgumentV1::new(_args.2, ::core::fmt::Display::fmt),
        ],
    },
));

which calls name() on Option which I'd guess is where it throws.

I am failing to reproduce this. Could you provide a minimal reproducible example?

I'm trying to convert this code from using typescript_definitions to ts-rs. Let me know if you see something obvious in there, otherwise I'll try to create a reproducable minimal example.

    #[derive(Serialize, TS)]
    #[ts(export)]
    #[serde(tag = "type")]
    pub enum ApiTypesTS {
        time_range {
            request: i32,
            response: Vec<i32>,
        },
        single_event {
            request: i32,
            response: Option<i32>,
        },
        rule_groups {
            request: (),
            response: Vec<i32>,
        },
        invalidate_extractions {
            request: i32,
            response: (),
        },
        update_rule_groups {
            request: Vec<i32>,
            response: (),
        },
        get_known_tags {
            request: (),
            response: Vec<i32>,
        },
        timestamp_search {
            request: i32,
            response: Option<i32>,
        },
    }

generates
type ApiTypesTS = { type: \"time_range\", request: number, response: Array<number>, } | { type: \"single_event\", request: number, response: number | null, } | { type: \"rule_groups\", request: null, response: Array<number>, } | { type: \"invalidate_extractions\", request: number, response: null, } | { type: \"update_rule_groups\", request: Array<number>, response: null, } | { type: \"get_known_tags\", request: null, response: Array<number>, } | { type: \"timestamp_search\", request: number, response: number | null, };

Here's a minimal example:
macro_rules! make {
    (pub enum $name:ident {
        $($r:ident { response: $resp:ty }),+
    }) => (
        #[derive(TS)]
        #[ts(export)]
        pub enum $name {
            $( $r { response: $resp }),*
        }
    )
}

make! {
    pub enum Foobar {
        single_event {
            response: Option<i32>
        }
    }
}

It seems to only happen when the enum is defined with a macro - even though that macro is not really doing anything?

I ran into this issue today, and after some testing, I can say that this is not a bug in ts-rs. I tested a struct with named fields and encountered the same error. Here is my testing code:
macro_rules! unit {
    (
        $(#[$meta:meta])*
		$struct_vis:vis struct $struct_name:ident {
			$(
				$(#[$field_meta:meta])*
				$field_vis:vis $field_name:ident : $field_type:ty
			),*$(,)+
		}
    ) => {
        $(#[$meta])*
        $struct_vis struct $struct_name {
            $(
				$(#[$field_meta])*
				$field_vis $field_name: $field_type,
			)*
        }
    };
}

unit! {
    #[derive(Debug, ts_rs::TS)]
    #[ts(export)]
    struct Test {
        pub field: Vec<String>,
        pub fieldTwo: Option<String>,
    }
}

After looking into ts-rs' source, I found the culprit to be the extract_type_args function in types/generics.rs in the macros crate. Essentially the problem is that when the declarative macro gets expanded, the types get parsed as Type::Group instead of Type::Path. This leads the extract_type_args to try and find generics on String and fail and falls back to the TS blanket impl on Option<T> where T: TS which has name as unreachable. This is either a bug in the syn crate or with how rustc expands declarative macros. For now this can be worked around by using fully qualified paths for types with generics in macro declarations. For example, the code above will work as:
macro_rules! unit {
    (
        $(#[$meta:meta])*
		$struct_vis:vis struct $struct_name:ident {
			$(
				$(#[$field_meta:meta])*
				$field_vis:vis $field_name:ident : $field_type:ty
			),*$(,)+
		}
    ) => {
        $(#[$meta])*
        $struct_vis struct $struct_name {
            $(
				$(#[$field_meta])*
				$field_vis $field_name: $field_type,
			)*
        }
    };
}

unit! {
    #[derive(Debug, ts_rs::TS)]
    #[ts(export)]
    struct Test {
        pub field: std::vec::Vec<String>,
        pub fieldTwo: std::option::Option<String>,
    }
}

I don't see any reasonable way for this to be worked around in ts-rs, so for now this will have to do. Hope this helps someone who runs into this is the future 😁 .

Closing this since @tsar-boomba presented a fix, there is no way for us to fix this since it's a syn bug and this issue has been inactive for two years