Investigate while(true)

@irskep While I think of it, what's the deal with this. I only noticed as I'm looking into some linting and it was not happy with this line.
https://github.com/literallycanvas/literallycanvas-core/blob/e8c2e0415c8f0e4387a17cccec3cec3fa3f7c17e/src/TextRenderer.js#L39
I couldn't actually manage to get the code to run through this path yet.
How can this possibly work like this? I don't see anything async in there, so it can't be an event loop. Surely this would just instantly block everything?

It's probably an automatic translation from CoffeeScript to ES6. It has a few returns in it.


Make fields "requires" and "provides" optional for task specs

When a task spec has the field "requires" empty ([]), it would be good if it can be omitted. The same for "provides".
When any of them are not present, it would be assumed empty array.

Done in branch master. Will be available in next release.


ISSUE: Logo is not displayed in the sidebar (top left)

Hello.
I can 't figure out how to fix the logo display error in the left sidebar (where the menu is ).
HTML markup is available

        Limbozone

, but it does not appear in the sidebar itself.

Did you change it in one of teh views or did you set it inside the configuration file?

Did you change it in one of teh views

I don 't quite understand. You mean did I extend @extends('adminlte:: page')? If Yes, I'm in the /resources/view/admin/dashboard.blade.php  and added an example from guides
@extends('adminlte::page')

@section('title', __('adminlte::dashboard.title'))

@section('meta_tags')
    <meta name="description" content="{{ __('adminlte::dashboard.description') }}">
    <meta name="keywords" content="{{ __('adminlte::dashboard.keywords') }}">
@endsection

@section('content_header')

{{ __('adminlte::dashboard.title' }}

@stop

@section('content')
    // custom content
@stop

What i meant was; did you publish the assets, and hardcoded it in the master.blade file, or did you set the url inside config/adminlte.php

config/adminlte.php — almost default. in the file I changed only the paths in logo:
'logo' => 'Adminlte',
    'logo_img' => '/images/circle-logotype.png',
    'logo_img_class' => 'brand-image img-circle elevation-3',
    'logo_img_xl' => null,
    'logo_img_xl_class' => 'brand-image-xs',
    'logo_img_alt' => 'logotype',

and configured menu.

...and hardcoded it in the master.blade file
No, file master.blade.php didn't make any change

It turns out that there is html markup. Paths are displayed correctly ( the image is loaded). But the block itself picture/ image papdis not outputting in top-left  sidebar

Hmm, Interesting. I cannot replicate this issue myself...

I noticed an oddity. HTML markup that does not display the logo:

            Limbozone

but if the delete link .brand-link - the logo starts to appear(crooked, but visible). Where else is it used class="brand-link".
maybe it's because of him...

You are using the latest version?

I am on the latest version, Laravel 6 and Adminlte 3

strange, performed a clean install of version 6 ( I had 7). The same thing...

hmm, strange. It shouldn't be the problem, but what browser are you on?

I'm very surprised myself. I do everything exactly according to the instructions and the problem is only in this moment:/
I use chrome Version 80.0.3987.149 and firefox 74.0 (64-bit). OS: ubuntu 18.04.4
problem in all browsers ...

The problem was solved accidentally (when transferring from a local server to hosting).
Google Chrome extension: AdGuard blocked the output of this block (there was no warning).
So consider this point :)


Brad proctor<PHONE_NUMBER>

Registration Form

This branch has merge conflicts and needs to be resubmitted =(
https://github.com/dallaspythondojo/python/pull/137
Also, please do not use .zip files on github


Locales no longer persist, roles are fine with the current implementation of them, perhaps to change at a later date.
Closed.


Various readme failures

Various failures:

$ref to original location could not be found in the new refs collection

Error: $ref to original location 'https://github.com/Azure/azure-rest-api-specs/blob/d9cf7c7ed3d674ebd482836e82b274014245ae67/specification/cognitiveservices/data-plane/Common/Parameters.json#/components/parameters/ImageUrl' is not found in the new refs collection

#245: ~FATAL: Error: Not able to process media type JSON at this moment.~
Cannot read property

FATAL: Failed validating: 'https://github.com/Azure/azure-rest-api-specs/blob/d9cf7c7ed3d674ebd482836e82b274014245ae67/specification/cost-management/resource-manager/Microsoft.CostManagement/stable/2019-11-01/costmanagement.json', error encountered: TypeError: Cannot read property 'properties' of undefined

[Exception] No input files provided.
#246: ~FATAL: Error: Enum types of 'object' and format 'undefined' are not supported. Correct your input (DatasetCompressionLevel).~
FATAL: System.InvalidOperationException: Unable to find object property with serialized name value in schema StatusesDefault
Error: Cannot resolve pipeline stage 'openapi-document/identity'.
#155: ~FATAL: Error: Not able to process media type / at this moment.~
Fixed: ~Bad indentation~

Readmes affected:

https://github.com/Azure/azure-rest-api-specs/blob/d9cf7c7ed3d674ebd482836e82b274014245ae67/specification/cognitiveservices/data-plane/Face/readme.md
https://github.com/Azure/azure-rest-api-specs/blob/d9cf7c7ed3d674ebd482836e82b274014245ae67/specification/cognitiveservices/data-plane/LUIS/Authoring/readme.md
https://github.com/Azure/azure-rest-api-specs/blob/d9cf7c7ed3d674ebd482836e82b274014245ae67/specification/cost-management/resource-manager/readme.md
https://github.com/Azure/azure-rest-api-specs/blob/d9cf7c7ed3d674ebd482836e82b274014245ae67/specification/datafactory/resource-manager/readme.md
https://github.com/Azure/azure-rest-api-specs/blob/d9cf7c7ed3d674ebd482836e82b274014245ae67/specification/datalake-analytics/data-plane/readme.md
https://github.com/Azure/azure-rest-api-specs/blob/d9cf7c7ed3d674ebd482836e82b274014245ae67/specification/intune/resource-manager/readme.md
https://github.com/Azure/azure-rest-api-specs/blob/d9cf7c7ed3d674ebd482836e82b274014245ae67/specification/iotcentral/data-plane/readme.md
https://github.com/Azure/azure-rest-api-specs/blob/d9cf7c7ed3d674ebd482836e82b274014245ae67/specification/machinelearning/resource-manager/readme.md
https://github.com/Azure/azure-rest-api-specs/blob/d9cf7c7ed3d674ebd482836e82b274014245ae67/specification/marketplace/resource-manager/readme.md
https://github.com/Azure/azure-rest-api-specs/blob/d9cf7c7ed3d674ebd482836e82b274014245ae67/specification/mixedreality/resource-manager/readme.md
https://github.com/Azure/azure-rest-api-specs/blob/d9cf7c7ed3d674ebd482836e82b274014245ae67/specification/resources/resource-manager/readme.md
https://github.com/Azure/azure-rest-api-specs/blob/d9cf7c7ed3d674ebd482836e82b274014245ae67/specification/service-map/resource-manager/readme.md
https://github.com/Azure/azure-rest-api-specs/blob/d9cf7c7ed3d674ebd482836e82b274014245ae67/specification/web/resource-manager/readme.md

~Crossed out~ some items that are already filed as issues and linked to them.


Unintentionally small area for clicking the "New Tab" plus button

System Details

OS: Ubuntu 22.04.3 LTS x86_64
Thorium Version 117.0.5938.157 (Official Build) stable, built on Linux (64-bit)

Problem
When trying to open a new tab by clicking on the plus icon next to an already opened tab, I often times miss it because the "hitbox" for the said button is smaller then it is shown when hovered over/when using thorium on windows. The hover preview and clickability is only active when hovering almost on the plus sign, which is a rather small area. This is easier to see with a gif.
Screenshots

Additional Notes
This is not that big of a bug but it does make using Thorium on my Ubuntu laptop slower and annoying as opposed to using it on Windows 10.

This issue is fixed with Version 121.0.6167.204 (Official Build) stable, built on Linux (64-bit), which I am now using so I'll be closing this issue.


Feature request : backport to all open feature branches

It would be nice to be able to backport to all feature branches currently open named eg 'feature/*'.
The configuration would be something like:

from: main
to: feature/*
for example.

This would create a PR on each feature/* branch.

Hi @RupertBarrow 👋 That's a great request!
✅ You can use the target_branches input for this, since v1.3.0.
Note that the target_branches input requires space-delimited branches, so finding these requires an extra step in your workflow.
name: Backport to all feature branches
on:
  pull_request_target:
    types: [closed]
permissions:
  contents: write # so it can comment
  pull-requests: write # so it can create pull requests
jobs:
  backport:
    name: Open backport pull requests
    runs-on: ubuntu-latest
    # Don't run on closed unmerged pull requests
    if: github.event.pull_request.merged
    steps:
      - uses: actions/checkout@v3
        with:
          # Fetch all branches
          fetch-depth: 0

      - id: branches
        shell: bash
        # list all banches, filtering for 'feature/*', and cutting off remotes/origin/
        # then finally substitute newlines for spaces
        run: |
          branches=$(git branch --list --all | grep 'origin/feature/' | cut -c 18- )
          space_delimited=${branches//$'\n'/ }
          echo "BRANCHES=${space_delimited}" >> $GITHUB_OUTPUT

      - uses: korthout/backport-action@v1
        with:
          # now you can use the branches output in the backport-action
          target_branches: ${{ steps.branches.outputs.BRANCHES }}

Tested with an example workflow

Example run, using workflow:
name: Space delimit branches
on:
  workflow_dispatch: {}
jobs:
  branches:
    name: Space delimited branches
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v3
        with:
          fetch-depth: 0
      - id: branches
        shell: bash
        run: |
          branches=$(git branch --list --all | grep 'origin/stable/' | cut -c 18- )
          space_delimited=${branches//$'\n'/ }
          echo "BRANCHES=${space_delimited}" >> $GITHUB_OUTPUT
      - run: echo "${{ steps.branches.outputs.BRANCHES }}"

Wow, thanks for the quick turnaround and the detailed response !
It would be great if "target_branches" accepted a regex. If that is difficult, you could create a specific "target_branches_regex" parameter just for that.
Thanks for the solution : I will try it out and tell you how it goes !

Hi @korthout ,
Unfortunately the backport did not go as well as expected : feature branches are found correctly but I have 2 issues:

backport is trying to merge into the feature branch which triggered the packport Github Action : we need to find a way of excluding that branch from the target_branches
more severe :
the cherry picking is failing for an unknown reason, the same on each target branch (see attached log)

backport.log.zip

Hi @RupertBarrow Thanks for sharing the logs, it helps a lot in troubleshooting this.
🏗 Let's see if we can get it to work as you expect.

🔧 There are a few ways you could achieve this:

make sure that the branch is deleted before the workflow executes

It might be enough to let GitHub delete branches as soon as a Pull Request is merged (I suggest you try this first).
use a GitHub Action (e.g. SvanBoxel/delete-merged-branch) to delete the branch before running the checkout step

remove the branch using inverted grep | grep -v ${merged_branch_name}, which will require you to figure out the name of the merged branch. This may be available under the pull_request_target's payload pull_request, e.g. github.event.pull_request.head.ref or github.event.pull_request.head.label, but you may have to strip parts of the ref away.

❌ The actions failed backporting to:

the merged branch, because the cherry-pick results in an empty commit which is not supported by this action (yet).
the other branches, because there is a merge conflict. There is only so much automation can achieve, the action cannot decide how to resolve conflicts. In that case, it comments on the pull request with instructions how to attempt the cherry-pick locally, so a human can attempt to resolve the conflicts. Try to backport pull requests with changes that won't conflict on the other branches.

✅ The workflow succeeded because the error is reported as comments on the pull request. If you wish to change this behavior you can use the action's outputs to fail the workflow ❌

Thanks, again, for your quick answers.

of course, best answer. But I don't always (yet) want to automatically deleted merged branches, often because I could make fixes on the same branch after QA (not best practice but ... well ...)
so your Github Action attempts the merge itself, immediately ? I thought it would be more respectful of developers working on those branches to propose them a PR : up to them to merge the PR when they prefer, and when they are ready. Can your workflow not generate a PR, rather than merging itself immediately ?
I understand your point of view.

I'm happy to try and get this working for you 👍 I'm convinced we can.

As a counter argument, when GitHub deletes the branch it keeps a hidden reference so you can always recover it (there's even a button in the bottom of the pull request for this after deleting the branch). In addition, since git is decentralised the branch still exists on your local machine and you can easily push it to GitHub again with your new commits.
If it really is no option to delete the branch, then please follow the instructions I gave on removing the branch name from the target_branches input. This will require some additional bash instructions in the branches step of the workflow above. If you need help with this, please let me know.

The action creates a pull request (see step 3 in How it works). It does not automatically merge the created pull request. However, in step 2 it cherry-picks the commits from the original pull request onto the target branch. There can be conflicts on cherry-picking the changes, which is the case in the logs you've shared. If you want the action to succeed, you need to backport changes that don't conflict with the branches they are backported to.

Hope this helps 🙇

I thought a bit more about the problem you're facing, and I see two feature requests:

exclude original pull request's branch name (head) from the target branches (can also be considered a bug on target_branches input
allow glob patterns in target_branches input, with a documentation note that this requires a deep checkout using fetch-depth: 0

I'll re-open this issue to track this.

exclude original pull request's branch name (head) from the target branches (can also be considered a bug on target_branches input
allow glob patterns in target_branches input, with a documentation note that this requires a deep checkout using fetch-depth: 0

@RupertBarrow I've resolved the first of these two with #344, which is now available on versions v1.3.1, v1.3 and v1. If you have the chance, please give it another try.
To support glob patterns in target_branches input, I'll open a separate issue.


Database empty after subscription

Lemon Squeezy for Laravel Version
^1.6
Laravel Version
^11.9
PHP Version
8.3.3
Description
When i subscribed to a product (product with 2 variants, one for month and other for year subscription).
Wen I sub, all good. But my database is full empty.
Steps To Reproduce

Subscribe to a plan
See the database empty

Are you running the webhooks?

Hey, yes. But Im working with Windows currently

I don't understand. Why is that a problem? This should all work fine really. Could you please share a detailed step-by-step guide to reproduce the issue?

Feel free to respond once you can.

I deleted the repository. Maybe I will retry it after


apply 3d displacement on  boundary nodes of 2d mesh

Dear MFEM users
I input .msh format model generated by gmsh into the EX2 of MFEM and try to solve it, I want to apply xyz displacement on boundary node, but it seems that the EX2 of MEFM only solves the problems of 2D plane model and 3D solid model. i check sol.gf,  but the node displacement is only in the X and Y directions, which is obviously not what I want.
If possible, please let me know what I need to know, I would be very grateful!

Hi @hedashua,
I am not sure I understand the problem -- are you trying to model a thin 2D plate in 3D? If that's the case, you need shell finite element which we don't currently have in MFEM.
Hope this helps,
Tzanio


chore(CHANGELOG): update to v4.4.0

Updates the CHANGELOG to v4.4.0

Updated the Changelog, added the library upgrades as well as templates -> element templates.


A small update about zh-CN

As you see,here's just a small update.It's a little embrassing that I forgot how to change my old Pull Request，and I'm failed to understand it once more,so this is the reason......Well,never mind about it.
By the way,how's the update going?Will we see the new version this year?

By the way, how's the update going? Will we see the new version this year?

Hello InkSsskeleton!
Thank you for your help in translating and for the Chinese Wikipedia, which you write so carefully )
I sometimes look at it )
The mod is currently being ported to version 1.18. However, in parallel with this, I am rewriting many mechanics almost from scratch. Therefore, the process is not moving as fast as we would like.
Also, it makes no sense to follow the GitHub, since all changes occur in a closed repository.
However, you can follow the progress on the mod's official discord.
Also, in light of current events, I've taken a short break from development. I hope to be able to return to work soon.
And here is a quote from the discord news channel:

Not just the port:

Terrain generator - completely rewritten (no more generation lags! The edges of the cliffs now look as they were originally intended);
Fog & clouds renderer - completely rewritten (now it works entirely on its own shaders. No more translucent planes! Volumetric shadows and something else interesting 😉 );
The growing trees system - completely rewritten (now the generation and growth of trees can be controlled from the same places. This means that it will be easier for me to create many new tree-like vegetation)...
... the work continues ...

By the way, how's the update going? Will we see the new version this year?

Hello InkSsskeleton! Thank you for your help in translating and for the Chinese Wikipedia, which you write so carefully ) I sometimes look at it ) The mod is currently being ported to version 1.18. However, in parallel with this, I am rewriting many mechanics almost from scratch. Therefore, the process is not moving as fast as we would like. Also, it makes no sense to follow the GitHub, since all changes occur in a closed repository. However, you can follow the progress on the mod's official discord.
Also, in light of current events, I've taken a short break from development. I hope to be able to return to work soon.
And here is a quote from the discord news channel:

Not just the port:

Terrain generator - completely rewritten (no more generation lags! The edges of the cliffs now look as they were originally intended);
Fog & clouds renderer - completely rewritten (now it works entirely on its own shaders. No more translucent planes! Volumetric shadows and something else interesting wink );
The growing trees system - completely rewritten (now the generation and growth of trees can be controlled from the same places. This means that it will be easier for me to create many new tree-like vegetation)...
... the work continues ...

I though I didn't succeed posting the previous file before. That's why I said it twice.

I though I didn't succeed posting the previous file before. That's why I said it twice.

Dame you're just right here!Really embarrassing.

Hello again!
I'm sorry that I have to send such massive words here instead of better websites,but for some reason,I'm unable to use Discord and many other foreign websites these days.Somehow,Github is accessable,so please forgive.
It's been 1 year at least since my last contribution to the Misty World mod,for I was busy studing in the high school.And now,since i've already graduated from high school,I guess it's time to deal with those,well,"issues left over from history".
In China ,there's a well-known standard of translation("信,达,雅",which is means "faithfulness, expressiveness and taste").Though we can just take some characters with similar pronunciation to the target word as a translation,and we actually did so,it's likely to be regarded as a low-quality translation and embrassing.So if someone need to translate a word into Chinese,it is quite important to know its meaning, its (word) roots,its animal prototype.Many mob names in the mod make me confusing and cannot figure out what they are(in both names and outlooks).So,could you please try to explain their names in your convenience?I'll list all the mob names needed to explain in the end.
If some of those listed name was just a flash in your mind,as long as you can remember,just tell,even if it's ridiculous.There's a mob called Andid in The Betweenland mod,which is the variant of "an idea".The mod's Chinese translaters at last translate it to "念鲤","念"has the meaning of "ideas",and"鲤" means carp,from its outlook.Even those experienced translaters need such informations,for someone not good at English and know almost nothing about foregin cultures,it only gets more important.
By the way,I also got some little questions to ask:
Why players cannot use lingering potion while wearing an unopen respirator?is it a bug or a setting on purpose?
Will the Taming skill be more easy to be realized in the next version?There's only 1 kind of Ⅰ level tamable mob,you know,it really needed to be optimized.
Could you please add some prefix like "foggy" to blocks that has the same name with something in vanilla Minecraft?It's quite weired and sometimes inconvenient.
I would be appreciated it if you could have a early and detailed reply.
The list:
Barvog
Wulder
Horb
Monk(mainly its animal prototype)
Galaga
Hulter
Barchiodon(mainly its root)
Momo(is it something like tapir?)

Now, on to other issues.

The inability to use exploding potions is a bug. I will fix it.
The situation with taming is not as terrible as it seems. Yes, it may not be obvious at all, but at the beginning you can tame not only animals with level I, but also with level II. It is true that you can only get a tamed animal with the appropriate level. But you can also “try” to tame an animal 1 level higher than yours.  Not as fast as taming the right animal, but sooner or later, your level will rise and the process will go easier.
This can be done at any level of taming.
I'll think about it.

Momo is, rather, a hybrid of a mini-hippopotamus with a platypus )

......sorry but,WTF?


MRet Instruction

The MRet instruction differs between RTL and Sail in two ways:

The RTL should clear the mstatus MPRV if the new privilege level is not machine.
The RTL currently throws an illegal instruction exception if the PCC cannot access system registers, but it should throw a CHERI exception instead according to the Sail.

#1: @marnovandermaas, maybe you can take a look? I think it might be an ibex thing.
#2: I will try to fix - possibly merge with the fix for CSR access permission violation.

I don't think 1 should affect CHERIoT Ibex, since we have no modes other than M and therefore can't transition to them, though it may affect non-CHERIoT Ibex.

MRET will set the new privilege level to mstatus.mpp, which is initialised as PRIV_LVL_U, which unless I am missing something means one can easily enter unprivileged mode. I can find a full trace if you like.

1. The RTL should clear the mstatus MPRV if the new privilege level is not machine.

This is a difference between version 1.11 and 1.12 of the privileged specification. Upstream this has already been fixed: https://github.com/lowRISC/ibex/commit/423264ce5fd6b3ca560cae1749d86dbcfdd794a7
I suggest that we stay with version 1.11 for CHERIoT Ibex unless we want to rebase on upstream.

RTL commit 4a739b4 changed the behavior of MRET to throw a CHERI exception (mcause = 0x1c, mtval =0x418) for MRET without ASR permission.

Fix is correct, thanks.

Just realised the first issue hasn't been fixed yet.

@mndstrmr , I have made an RTL update (876a46a to include the mprv fix. Please verify.

This proves, thanks.


Faster ShardedFragmentSampler

Problem: fragment sampler downloads fragments in a blocking fashion. This is a bottleneck if your batch size is larger than the number of rows in a fragment.
Solution: read-ahead with threads
Rough implementation
class ReadAheadShardedFragmentSampler(ShardedFragmentSampler):
    def __init__(
        self,
        rank: int,
        world_size: int,
        randomize: bool = False,
        seed: int = 0,
        read_ahead: int = 16,
    ):
        super().__init__(rank, world_size, randomize, seed)
        self._read_ahead = read_ahead

    def iter_fragments(self, dataset: lance.LanceDataset, fragment_queue: Queue):
        fragments = dataset.get_fragments()
        if self._randomize:
            random.seed(self._seed)
            random.shuffle(fragments)
        for offset, idx in enumerate(range(self._rank, len(fragments), self._world_size)):
            fragment_queue.put((offset, fragments[idx]))
        fragment_queue.put((None, None))

    def iter_batches(self, fragment_queue: Queue, batch_queue: Queue, **kwargs):
        while True:
            offset, fragment = fragment_queue.get(block=True)
            if offset is None:
                batch_queue.put((None, None))
                fragment_queue.task_done()
                return

            for idx, batch in enumerate(fragment.to_batches(**kwargs)):
                batch_queue.put((idx * offset + idx, batch))
            fragment_queue.task_done()

    def __call__(
        self,
        dataset: lance.LanceDataset,
        *args,
        batch_size: int = 128,
        columns: list[str] | dict[str, str] | None = None,
        filter: str | None = None,
        batch_readahead: int = 16,
        with_row_id: bool = False,
        **kwargs,
    ) -> Generator[pa.RecordBatch, None, None]:
        fragment_queue = Queue(maxsize=self._read_ahead)
        batch_queue = Queue()

        fragment_thread = threading.Thread(
            target=self.iter_fragments,
            args=(dataset, fragment_queue),
            daemon=True,
        )
        batch_threads = [
            threading.Thread(
                target=self.iter_batches,
                args=(fragment_queue, batch_queue),
                kwargs=dict(
                    batch_size=batch_size,
                    columns=columns,
                    filter=filter,
                    with_row_id=with_row_id,
                    **kwargs,
                ),
                daemon=True,
            )
            for _ in range(batch_readahead)
        ]

        fragment_thread.start()
        for thread in batch_threads:
            thread.start()

        current_idx = 0
        buffer = {}
        while True:
            try:
                idx, batch = batch_queue.get(timeout=0.01)
                if idx is None:
                    batch_queue.task_done()
                    raise StopIteration
                buffer[idx] = batch
            except Empty:
                continue

            if current_idx in buffer:
                yield buffer[current_idx]
                del buffer[current_idx]
                current_idx += 1

@eddyxu Any thoughts on something like this?

@westonpace has done a great amount of work to improve scan throughput in v2 format. I will let him chime in


The minecraft server cannot connect to the detection server

The minecraft server cannot connect to the detection server

sorry i found problem


Right View of Binary Tree using queue in C++

EXPLANATION
Our task is to print the right view of a Binary Tree with the help of queue.
MOTIVATION
I have read the contributing guidelines.

/assign


Example for Append()?

I'm trying to Append() to an embedded array like this:
row, err := r.Table("things").
    Get(thingId).
    Field("array").
    Append(map[string]string{"tag": "value"}).
    RunWrite(dbSession)

"array": [ ]    ->    "array": [ {"tag": "value"} ]

... it does not work; the array remains empty. What am I doing wrong?

I tried the same thing but for some reason Changes().Run doesn't get fired. Any ideas?

Hi, could you please post the code you are trying to run?

Sure, thanks for the quick reply :)

Schema
Append item to array
Delete item from array
Watcher

I forgot to add:
fmt.Printf("row: %v", row)

gives
row: {0 0 0 0 1 0 0 0 0 0 0 0 0 0 []  [] []}

Which equals to Row.Replaced = 1

The update term looks like:
	if row, err := m.Table.Get(set).Update(func(d r.Term) interface{} {
		ret := map[string]interface{}{
			"keys": d.Field("keys").Append([]interface{}{keys}...),
		}
		fmt.Printf("Updating: %v", ret) //
map[keys:var_0.Field("keys").Append([{Algorithm="", Use="", Key={N={}, E=65537}, KeyID="public"}])]
		return ret
	}).RunWrite(m.Session); err != nil {
              // ...

I can't see anything that looks obviously wrong but I will have a proper look when I get home from work this evening. If you create a changes query in the RethinkDB admin UI does it show the updates?

unfortunately I'm spinning up a docker rethink db image for automated testing and I need some work to do for this to be done manually. The insert command fires a change as expected. Is there maybe something like an index I need to create in order for array changes to be processed?

I refactored the code to not use embedded arrays but rather regular documents and encountered the same issue. Missing error handlers caused the error to be ignored and not printed. In general, it is very hard to track down the error as there's some magic going on (LastErr is overriden several times?).
I believe that the error is caused by a MarshalJSON method receiver on DefaultPolicy.Conditions. The referred map was nil.

The error I was able to recover by injecting some good old log.Printf("%s", err) inside of Cursor.nextLocked which got me something like: Cannot marshal type x into type z. However, Cursor.Err() resulted in an EOF error.

Scrap my last comments (I deleted them). Still investigating.

I can't see anything that looks obviously wrong but I will have a proper look when I get home from work this evening. If you create a changes query in the RethinkDB admin UI does it show the updates?

This will take some time until I have it up and running in a non-automated test environment.

I tried creating a very simple example based on your code and it appears to work so my guess is that either your application is creating data in a way that is incompatible with GoRethink or your changes query is setup slightly wrong. Hopefully if you can setup a non-automated test environment we should be able to get a better idea about what is going on.
Here is the code I am running, let me know if you think I have replicated something incorrectly.
package main

import (
	"encoding/json"
	"fmt"
	"github.com/davecgh/go-spew/spew"
	"time"

	r "github.com/dancannon/gorethink"
)

type rethinkSchema struct {
	KID string          `gorethink:"id"`
	Set string          `gorethink:"set"`
	Key json.RawMessage `gorethink:"key"`
}

func main() {
	sess, err := r.Connect(r.ConnectOpts{
		Addresses: []string{
			"localhost:28015",
		},
	})
	if err != nil {
		fmt.Printf("Connection Failed [ERR: %s]", err.Error())
	}

	r.DB("test").TableDrop("test58").Exec(sess)
	r.DB("test").TableCreate("test58").Exec(sess)

	connections, err := r.DB("test").Table("test58").Changes().Run(sess)
	if err != nil {
		panic(err.Error())
	}

	go func() {
		for {
			var update map[string]*rethinkSchema
			for connections.Next(&update) {
				spew.Dump(update)
			}

			connections.Close()
			if connections.Err() != nil {
				panic(connections.Err().Error())
			}

			connections, err = r.DB("test").Table("test58").Changes().Run(sess)
			if err != nil {
				panic(err.Error())
			}
		}
	}()

	resp, err := r.DB("test").Table("test58").Insert(&rethinkSchema{
		KID: "abc",
		Set: "def",
	}).RunWrite(sess)

	spew.Dump(resp, err)

	resp, err = r.DB("test").Table("test58").Insert(&rethinkSchema{
		KID: "abc",
		Set: "def",
	}).RunWrite(sess)

	spew.Dump(resp, err)

	time.Sleep(5 * time.Second)
}

Hey @dancannon sorry, I accidentally didn't link the commit but rather the branch head. I resolved the issue by removing the embedded array. Sorry for the inconvenience
Here are the original code lines:

Schema
Append item to array
Delete item from array
Watcher


npm install is not enough to use npm run test

Hi, thanks very much for adding this new exercise. In all the other JS exercises, I have been able to run the tests after typing the following commands into terminal:

npm install
npm run test

However with this exercise, I have not been able to run the tests. This is what I have tried:
First error: jest: command not found

npm install
npm run test
Error received: sh: jest: command not found

To fix this, I typed in npm install --save-dev jest
This fixed the above error. However now a new error appears and I don't know how to fix this.
New error: Cannot find module '@babel/preset-env'
I can see that there is a file called babel.config.js in the folder. Inside the file, I can see the preset configuration.
However I don't know how to fix the error.
I tried typing: npm install --save-dev @babel/core @babel/cli but that did not fix the error.
I have not needed to do anything like this to run tests for the other exercises, so I thought I would raise the issue here.
Apologies if this is really simple, but I'm learning and tried to Google, but still couldn't find an answer I understood.
Below is the debug log.
Thanks for your help.
0 info it worked if it ends with ok
1 verbose cli [ '/usr/local/bin/node', '/usr/local/bin/npm', 'run', 'test' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'pretest', 'test', 'posttest' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>7 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
8 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/node-gyp-bin:/Users/ai-lyntang/Exercism/javascript/resistor-color/node_modules/.bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin
9 verbose lifecycle<EMAIL_ADDRESS>CWD: /Users/ai-lyntang/Exercism/javascript/resistor-color
10 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'jest --no-cache ./*' ]
11 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
12 info lifecycle<EMAIL_ADDRESS>Failed to exec test script
13 verbose stack Error<EMAIL_ADDRESS>test: `jest --no-cache ./*`
13 verbose stack Exit status 1
13 verbose stack     at EventEmitter.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/index.js:301:16)
13 verbose stack     at EventEmitter.emit (events.js:182:13)
13 verbose stack     at ChildProcess.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/lib/spawn.js:55:14)
13 verbose stack     at ChildProcess.emit (events.js:182:13)
13 verbose stack     at maybeClose (internal/child_process.js:962:16)
13 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:251:5)
14 verbose pkgid<EMAIL_ADDRESS>15 verbose cwd /Users/ai-lyntang/Exercism/javascript/resistor-color
16 verbose Darwin 16.7.0
17 verbose argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "test"
18 verbose node v10.15.0
19 verbose npm  v6.4.1
20 error code ELIFECYCLE
21 error errno 1
22 error<EMAIL_ADDRESS>test: `jest --no-cache ./*`
22 error Exit status 1
23 error Failed at the<EMAIL_ADDRESS>test script.
23 error This is probably not a problem with npm. There is likely additional logging output above.
24 verbose exit [ 1, true ]

From your message I can't figure out which exercise; from the log i think it's resistor-color.
If we look at its package.json, we can see jest is already present.
I've also gone ahead an download the exercise again and run the appropriate commands
cd resistor-color
npm i
npm test

No errors 😓 .

Thanks so much for looking into this @SleeplessByte . My apologies for not listing the exercise - I thought I had navigated to the correct exercise on GitHub before posting, but I must not have!
You were right, it was indeed resistor-color.
And as is always the case, now that you've looked at it, I can no longer replicate the problem either. I think perhaps that npm install didn't work properly the first time I did it.
Thanks again, and very sorry to waste your time. I'll close this issue now.

@ailyntang I'm happy it got resolved! Have fun on exercism.

@SleeplessByte Thanks for looking into this!


use this template

I don't see it anymore ?

That's a github UI bug. If you zoom out, or make the window wider it will show up.

@lishid nice one. They are modifying a lot the UI these days, which is a good thing. If you know how to get rid of this blue dot BTW, I take it

If you click on it, it disappears.

no, no way...


Feature/merge support

This allows you to specify -m to merge and internalize all output assemblies into the first output assembly.
This also allows you to use the same name for -o if you do not want to rename anything in the output path (typically for useful for the first output path).
This also adds -k <keyFile> to resign the first output dll.
dotnet AssemblyRewriter.dll
   -i "c:\X\Elastic.BenchmarkDotNetExporter.dll" -o "c:\X\Elastic.BenchmarkDotNetExporter.dll"
   -i "C:\X\Nest.dll" -o "c:\X\Elastic.Internal.Nest.dll"
   -i "c:\X\Elasticsearch.Net.dll" -o "c:\X\Elastic.Internal.Elasticsearch.Net.dll"
   -m

I've updated the PR to remove the ILRepack support @russcam


Fixed wrong parameters in Readme

After trying to use the Readme parameters to downloader and scanner, saw that both now use -i and not -a
 Basem Vaseghi<EMAIL_ADDRESS>Daimler TSS GmbH, Impressum

Thanks for this! There's just a minor typo in the commit message: "boh" should be "Both".

The tests seem to fail since the rebase after changing the commit message. I just saw that that was fixed in another commit in master and hopefully this update will do so as well.

Thanks again. Just a small hint for the next contribution: We use imperative mood in the commit messages. That is, instead of writing "fixed documented parameters" we prefer "Fix documented parameters".

I apologize, seems I left the commit message untouched. This should comply.

@mnonnenmacher please merge despite the CI failure.


[Broken] Youtube: Incomplete data received error when downloading from a community page

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running yt-dlp version 2<IP_ADDRESS>
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

I:\archive scripts\batch scripts\member_script>yt-dlp -v --cookies newcookiefile_2.txt --download-archive "I:\archive scripts\Hololive\Gen 3\rushia_member_archive.txt" -o "E:\Hololive\Gen 3\Rushia Ch. 潤羽るしあ\【Member Video】\%(title)s\%(upload_date)s - %(title)s" -i --match-filter "!is_live" --abort-on-unavailable-fragment -f "bestvideo+bestaudio/best/best" --write-thumbnail "https://www.youtube.com/channel/UCl_gCybOJRIgOXw6Qb4qJzQ/community"
[debug] Command-line config: ['-v', '--cookies', 'newcookiefile_2.txt', '--download-archive', 'I:\\archive scripts\\Hololive\\Gen 3\\rushia_member_archive.txt', '-o', 'E:\\Hololive\\Gen 3\\Rushia Ch. 潤羽るしあ\\【Member Video】\\%(title)s\\%(upload_date)s - %(title)s', '-i', '--match-filter', '!is_live', '--abort-on-unavailable-fragment', '-f', 'bestvideo+bestaudio/best/best', '--write-thumbnail', 'https://www.youtube.com/channel/UCl_gCybOJRIgOXw6Qb4qJzQ/community']
[debug] Loading archive file 'I:\\archive scripts\\Hololive\\Gen 3\\rushia_member_archive.txt'
[debug] Encodings: locale cp65001, fs utf-8, out utf-8, pref cp65001
[debug] yt-dlp version 2<IP_ADDRESS>
[debug] Python version 3.9.1 (CPython 64bit) - Windows-10-10.0.19041-SP0
[debug] exe versions: ffmpeg 4.3.1-2020-11-19-essentials_build-www.gyan.dev, ffprobe 4.3.1-2020-11-19-essentials_build-www.gyan.dev
[debug] Proxy map: {}
[youtube:tab] UCl_gCybOJRIgOXw6Qb4qJzQ:
[download] Downloading playlist: Rushia Ch. 潤羽るしあ - Community
[youtube:tab] UCl_gCybOJRIgOXw6Qb4qJzQ page 1: Downloading API JSON
WARNING: [youtube:tab] Incomplete data recieved. Retrying ...
[youtube:tab] UCl_gCybOJRIgOXw6Qb4qJzQ page 1: Downloading API JSON (retry #1)
WARNING: [youtube:tab] Incomplete data recieved. Retrying ...
[youtube:tab] UCl_gCybOJRIgOXw6Qb4qJzQ page 1: Downloading API JSON (retry #2)
WARNING: [youtube:tab] Incomplete data recieved. Retrying ...
[youtube:tab] UCl_gCybOJRIgOXw6Qb4qJzQ page 1: Downloading API JSON (retry #3)
ERROR: Incomplete data recieved
  File "c:\users\samph\appdata\local\programs\python\python39\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "c:\users\samph\appdata\local\programs\python\python39\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\samph\AppData\Local\Programs\Python\Python39\Scripts\yt-dlp.exe\__main__.py", line 7, in <module>
    sys.exit(main())
  File "c:\users\samph\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\__init__.py", line 606, in main
    _real_main(argv)
  File "c:\users\samph\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\__init__.py", line 596, in _real_main
    retcode = ydl.download(all_urls)
  File "c:\users\samph\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\YoutubeDL.py", line 2488, in download
    res = self.extract_info(
  File "c:\users\samph\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\YoutubeDL.py", line 1017, in extract_info
    return self.__extract_info(url, ie, download, extra_info, process, info_dict)
  File "c:\users\samph\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\YoutubeDL.py", line 1024, in wrapper
    return func(self, *args, **kwargs)
  File "c:\users\samph\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\YoutubeDL.py", line 1061, in __extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "c:\users\samph\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\YoutubeDL.py", line 1145, in process_ie_result
    return self.__process_playlist(ie_result, download)
  File "c:\users\samph\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\YoutubeDL.py", line 1287, in __process_playlist
    entries = list(itertools.islice(
  File "c:\users\samph\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\extractor\youtube.py", line 2810, in _entries
    self._downloader.report_error(last_error)
  File "c:\users\samph\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\YoutubeDL.py", line 743, in report_error
    self.trouble(error_message, tb)
  File "c:\users\samph\appdata\local\programs\python\python39\lib\site-packages\yt_dlp\YoutubeDL.py", line 705, in trouble
    tb_data = traceback.format_list(traceback.extract_stack())

[youtube:tab] playlist Rushia Ch. 潤羽るしあ - Community: Downloading 0 videos
[download] Finished downloading playlist: Rushia Ch. 潤羽るしあ - Community

Description

I've been trying to download all the videos(member and non-member using --cookies) from a channel's community page, but I keep getting the incomplete data received error. The problem is that it's stuck on page 1 and I guess the API JSON download fails, preventing subsequent pages from being found. I've tried using youtube-dl but it gives the same result, though without the "Downloading API JSON" message. If there are any videos on page 1 then yt-dlp will download it, but it won't proceed to the next page to download more. I can download the videos from later pages if I manually get the video URL.
I tried just a basic command --list-formats on something like https://www.youtube.com/channel/UCl_gCybOJRIgOXw6Qb4qJzQ/community
and only the videos on the first page would be displayed(if there are any).

Can reproduce - appears the community continuation pages return a slightly different JSON response.

@colethedj I have already fixed it on my local version. Just haven't pushed it yet


adding code of conduct

per the ustwo OSS purposed guidelines we'll want to have a code of conduct for this. https://github.com/ustwo/development-at-ustwo/issues/23

:+1:

:+1:

Current coverage is 100.00%

Merging #15 into master will not affect coverage as of 7df7b63

@@            master     #15   diff @@
======================================
  Files            6       6
  Stmts           87      87
  Branches         0       0
  Methods          0       0
======================================
  Hit             87      87
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 7df7b63

Powered by Codecov. Updated on successful CI builds.


[question] does thumpcord transmit chat from a single player world to a discord channel?

by that I mean most discord mods I have seen use a bot linked to the mc server that relays the chat, im looking for a mod that will relay chat from a single player world as well.

Yes, you just need to install Thump and Thumpcord locally then it should work from your single player world.

Yep - You'll still need to create a bot in Discord's developer site as normal to attach to, but Thumpcord works in single-player just fine; set it up as you would on the server and it should be fine. If not, let us know!


When publication tags edited from reader view, tag state in library is completely overriden

library view: open publication X
reader view: detach, place the 2 windows side by side
library view: add "test-library" tag to publication X (close dialog, scroll down to confirm tag exists)
reader view: (note that the "test-library" tag is not shown, this is issue https://github.com/readium/readium-desktop/issues/450 ) add "test-reader" tag
library view: notice how "test-library" has disappeared, replaced with tag state from reader view ("test-reader" tag)
reader view: add/remove tags and observe state update at the bottom of the library view

Related issues:
https://github.com/readium/readium-desktop/issues/450
https://github.com/readium/readium-desktop/issues/451

Fixed in PR #466


Change the flipYs value in the pickFontColor function in colors.ts to 0.4

The current flipYs value is 0.342 but this causes too many course blocks to have black text (which can look unattractive). 0.342 was chosen for accessibility, but changing it to 0.4 still maintains accessibility and allows for more white text. (the last two images show the difference between 0.342 and 0.4)

With 0.342:

With 0.4:

@IsaDavRod It's harder to read; this is making it less accessible. Let's stick with how it works right now.

@Razboy20 It is not any harder to read. 0.4 still allows for bright backgrounds to have black text. Sriram and Sam also agreed with this change.

Sam believes that left looks better to me, BUT accessibility is important so probably should stay the same? conflicted...

I mention Google Calendar because Google prides itself on accessibility and is more lenient than us on this issue. We should follow their steps 😈

@IsaDavRod can you provide a source of GCal using 0.4?

Keep in mind that Google uses different background colors so that is another variable to account for. I too would be interested in seeing what value GCal uses if at all. I do think that accessibility is a priority and out current system follows that nicely. Should we decided to pursue this change, we need further analysis.
Closing issue for now due to it being stale and non-conclusive results.


No support to pass a Type to Async Insert methods

It would be great if the Type parameter could also be passed to the asynchronous Insert... methods.
I've done this myself, but I don't know how to push it to source.
In SQLIte.cs, add:
///
    /// Inserts all specified objects.
    /// </summary>

    /// <param name="objects">
    /// An <see cref="IEnumerable"/> of the objects to insert.
    /// </param>
    /// <returns>
    /// The number of rows added to the table.
    /// </returns>
    public int InsertAll(System.Collections.IEnumerable objects, Type type, bool beginTransaction = true)
    {
        if (beginTransaction)
        {
            BeginTransaction();
        }
        var c = 0;
        foreach (var r in objects)
        {
            c += Insert(r, type);
        }
        if (beginTransaction)
        {
            Commit();
        }
        return c;
    }

In SQLIteAsync.cs, add:
    public Task<int> InsertAllAsync(IEnumerable items, Type type)
    {
        return Task.Factory.StartNew(() =>
        {
            var conn = GetConnection();
            using (conn.Lock())
            {
                return conn.InsertAll(items, type);
            }
        });
    }

This has been added!


minor(flag): Valid default for auth-file

Use the basename of the configuration file we use by default in the Docker image.
This avoids this non-helpful error message when running without arguments:
[debian]$ ./fortigate-exporter.linux.amd64
2021/07/10 21:45:43 FortigateExporter 1.5.1-1-g9994480 ( 9994480 )
2021/07/10 21:45:43 Failed to read API authentication map file: open : no such file or directory

Also this commit type should be feat

I don't agree, but I will change it. My argument is that this doesn't change anything anyone could reasonably depend on (the default value right now is invalid and only causes crashes).

I agree with you, if somebody could depend on it, it should be a breaking change.
But minor will not generate a release and as I understand you that is your intend.

Ah, right. You are correct :-)

What about fix(flag) instead? It feels a bit too much to bump to 1.8.0 just because of this.

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


[FLINK-12254][table] Update table source and sinks to new type system

What is the purpose of the change
This PR updates the source and sink interfaces to the new type system. It also updates related interfaces. Backwards compatibility is ensured by existing tests.
Brief change log
See commit messages.
Verifying this change
This change is already covered by existing tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): yes
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? JavaDocs

Thanks @dawidwys. Merging...


How it work for linux user ?

Error: The reference assemblies for .NETFramework,Version=v4.7.1 were not found. To resolve this, install the Developer Pack (SDK/Targeting Pack) for this framework version or retarget your application. You can download .NET Framework Developer Packs at https://aka.ms/msbuild/developerpacks
This is the error i face in Vs code  in Ubuntu. I could not download .NETFramework v4.7.1 . only avaialble for window.

Full detail :
Starting OmniSharp server at 5/7/2020, 7:12:33 PM
Target: /home/prakash/Documents/Unity_Projects/test18422/test18422.sln
OmniSharp server started.
Path: /home/prakash/.vscode/extensions/ms-dotnettools.csharp-1.21.18/.omnisharp/1.35.1/run
PID: 29575
    Starting OmniSharp on ubuntu 19.10 (x64)

    DotNetPath set to dotnet

    Located 1 MSBuild instance(s)
        1: StandAlone 16.4 - "/home/prakash/.vscode/extensions/ms-dotnettools.csharp-1.21.18/.omnisharp/1.35.1/omnisharp/.msbuild/Current/Bin"

    MSBUILD_EXE_PATH environment variable set to '/home/prakash/.vscode/extensions/ms-dotnettools.csharp-1.21.18/.omnisharp/1.35.1/omnisharp/.msbuild/Current/Bin/MSBuild.dll'

    Registered MSBuild instance: StandAlone 16.4 - "/home/prakash/.vscode/extensions/ms-dotnettools.csharp-1.21.18/.omnisharp/1.35.1/omnisharp/.msbuild/Current/Bin"
        CscToolExe = csc.exe
        MSBuildToolsPath = /home/prakash/.vscode/extensions/ms-dotnettools.csharp-1.21.18/.omnisharp/1.35.1/omnisharp/.msbuild/Current/Bin
        CscToolPath = /home/prakash/.vscode/extensions/ms-dotnettools.csharp-1.21.18/.omnisharp/1.35.1/omnisharp/.msbuild/Current/Bin/Roslyn
        BypassFrameworkInstallChecks = true
        MSBuildExtensionsPath = /home/prakash/.vscode/extensions/ms-dotnettools.csharp-1.21.18/.omnisharp/1.35.1/omnisharp/.msbuild

    Detecting Cake files in '/home/prakash/Documents/Unity_Projects/test18422'.

    Could not find any Cake files

    Detecting projects in '/home/prakash/Documents/Unity_Projects/test18422/test18422.sln'.

    Queue project update for '/home/prakash/Documents/Unity_Projects/test18422/Assembly-CSharp.csproj'

    Detecting CSX files in '/home/prakash/Documents/Unity_Projects/test18422'.

    Could not find any CSX files

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpFormattingWorkspaceOptionsProvider, Order: 0

    Loading project: /home/prakash/Documents/Unity_Projects/test18422/Assembly-CSharp.csproj

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.RenameWorkspaceOptionsProvider, Order: 100

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.ImplementTypeWorkspaceOptionsProvider, Order: 110

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.BlockStructureWorkspaceOptionsProvider, Order: 140

    Configuration finished.

    Omnisharp server running using Stdio at location '/home/prakash/Documents/Unity_Projects/test18422' on host 29421.

    The reference assemblies for .NETFramework,Version=v4.7.1 were not found. To resolve this, install the Developer Pack (SDK/Targeting Pack) for this framework version or retarget your application. You can download .NET Framework Developer Packs at https://aka.ms/msbuild/developerpacks

    Failed to load project file '/home/prakash/Documents/Unity_Projects/test18422/Assembly-CSharp.csproj'.

/home/prakash/Documents/Unity_Projects/test18422/Assembly-CSharp.csproj
/home/prakash/.vscode/extensions/ms-dotnettools.csharp-1.21.18/.omnisharp/1.35.1/omnisharp/.msbuild/Current/Bin/Microsoft.Common.CurrentVersion.targets(1193,5): Error: The reference assemblies for .NETFramework,Version=v4.7.1 were not found. To resolve this, install the Developer Pack (SDK/Targeting Pack) for this framework version or retarget your application. You can download .NET Framework Developer Packs at https://aka.ms/msbuild/developerpacks
    Attempted to update project that is not loaded: /home/prakash/Documents/Unity_Projects/test18422/Assembly-CSharp.csproj

@PrakTech You make a good point. This isn't a helpful error message when on MacOS or Linux. For these platforms you can install the stable release of Mono (6.8.0 from https://www.mono-project.com/download/stable/#download-lin) to install the .NET 4.7.1 reference assemblies that Unity projects require.
@cartermp Can we update this error or the linked website to include instructions for non-Windows platforms?

@JoeRobich  you are right . Linux needs mono install along with dotnet core sdk to work out . Thanks for solution.

@JoeRobich Thanks!

This issue is not yet fixed – the message on Linux is still unhelpful.
My VS Code version:
% snap list code
Name  Version   Rev  Tracking       Publisher  Notes
code  ea3859d4  52   latest/stable  vscode✓    classic

This issue is not yet fixed – the message on Linux is still unhelpful.
My VS Code version:
% snap list code
Name  Version   Rev  Tracking       Publisher  Notes
code  ea3859d4  52   latest/stable  vscode✓    classic

Installing mono <IP_ADDRESS>-1 on Manjaro did not fix my issue, so I really don't know what to do.

Think i found a solution,

Install latest stable mono from above.
In the bottom right of VS Code you should see a little c# symbol (The language setting VS Code detects for your project). Click that and open the "Configure C# Language based settings".
Add the following line to your settings "omnisharp.useGlobalMono": "always".
4: restart your omni server. (There should be a little pop up in bottom right with a button that says restart omni server)

This caused the error to go away and for intellisense to kick in.
Hope it helps

Think i found a solution,
1. Install latest stable mono from above.

2. In the bottom right of VS Code you should see a little c# symbol (The language setting VS Code detects for your project). Click that and open the "Configure C# Language based settings".

3. Add the following line to your settings `"omnisharp.useGlobalMono": "always"`.
   4: restart your omni server. (There should be a little pop up in bottom right with a button that says restart omni server)

This caused the error to go away and for intellisense to kick in.
Hope it helps

Thanks a lot, I search all over the internet and found only your solution working.

wow! Thank you very much, @Morganb816! I was trying to find the solution for days!  this is definitely the only thing that worked

This solution seems so janky because literally every C# dev in Linux will encounter this.  Can omnisharp.useGlobalMono be set to always by default on Linux oslt?

Think i found a solution,

Install latest stable mono from above.
In the bottom right of VS Code you should see a little c# symbol (The language setting VS Code detects for your project). Click that and open the "Configure C# Language based settings".
Add the following line to your settings "omnisharp.useGlobalMono": "always".
4: restart your omni server. (There should be a little pop up in bottom right with a button that says restart omni server)

This caused the error to go away and for intellisense to kick in. Hope it helps

Wow! That just perfectly solved my problem! Thanks.


Geolocation - add warning about null return value

Fixes https://github.com/dotnet/maui/issues/10562

GetLocationAsync can return null in some scenarios. This indicates that the underlying platform is unable to obtain the current location.

That is not very helpful. For a robust app, it must be clear why the platform returns null and how to assure it returns real values again.

@janseris Google's Android docs and Apple's iOS docs don't make clear in which circumstances getting the current location returns null.

@janseris Google's Android docs and Apple's iOS docs don't make clear in which circumstances getting the current location returns null.

Thanks. If it is like this it would be very nice if the info that it's a direct call to Android API and thus it behaves like Android API and the programmer should be looking for issues/questions on Android API for more details.


Adding tests for the error.go resource

This is my first commit. adding tests for "nephio\controllers\pkg\resource\error.go"

/ok-to-test

/lgtm

/approve

/lgtm
/approve


Literals with underscores not recognized by print and returns

https://is.gd/GRNBmY

Its the combination of _ and f that doesn't work. Perhaps they are not meant to be used together.

Good catch for you both.
The source of this bug is that I feed the string to C's function that converts a string to a float.  C does not accept underscores, so when it finds one, it stops and converts the number up to that.
This should not be that difficult to fix, but it will require building a new string from the old one before sending the number to C.

Fixed.


chore: drop Python 3.5 & 3.6 support

This is work in progress. I created a Draft PR in order to be able to verify progress with CI.

I'm still polishing various things... Need to clean up the tests and undo some foul import hacks.


Fix KeyCombination throwing when duplicates are fed in

This class may require some test coverage. Just pushing out for immediate visibility.

Added test coverage, fixed bug in implementation (https://github.com/ppy/osu-framework/pull/6130/commits/e2f325f04e571ac07396d2f8237185d5f29188c3)

fixed bug in implementation

🤕


Glassfish 4 JaxRS support

Hello James,
My name is Pete Sfiridis Ive been working in the Health Care industry a number of years. Ive used HAPI HL7 many times as well as other commercial HL7 parsers.
I'm starting to use HAPI FHR JaxRS Server for a project, but I need to use Glassfish 4. Basically because of how the constructor is coded Glassfish 4 doesn't start up my Patient resource provider (extends AbstractJaxRsResourceProvider). Here's a short summary of why. Btw I do have a fix, see comments below. If you'd like me to contribute I would need access to the Git repo. I cant create a new Pull Request and not sure why.
When deploying to Glassfish4 a NullPointerException occurs in the
public  JaxRsMethodBindings(AbstractJaxRsProvider theProvider, Class theProviderClass) {
        for (final Method m : ReflectionUtil.getDeclaredMethods(theProviderClass)) {
            final BaseMethodBinding foundMethodBinding = BaseMethodBinding.bindMethod(m, theProvider.getFhirContext(), theProvider);
stacktrace
Severe:   Exception while loading the app
Severe:   Exception while loading the app : EJB Container initialization error
java.lang.NullPointerException
at org.cdsframework.iis.integration.rs.service.EJB31_Generated__PatientProvider__Intf____Bean.getFhirContext(Unknown Source)
at ca.uhn.fhir.jaxrs.server.util.JaxRsMethodBindings.(JaxRsMethodBindings.java:58)
at ca.uhn.fhir.jaxrs.server.util.JaxRsMethodBindings.getMethodBindings(JaxRsMethodBindings.java:140)
at org.cdsframework.iis.integration.rs.service.AbstractRsResourceProvider.(AbstractRsResourceProvider.java:91)
Its primarily due to the timing of when the Object in Glassfish has access to the FhirContext.
The fix involved adding a postConstructor annotation to AbstractJaxRsResourceProvider and moving the
JaxRsMethodBindings.getMethodBindings call to the post postConstructor. I also had to make some other minor changes as well as change two of the test classes.
Let me know how to proceed with communicating the changeset. I tried creating a New Pull Request but couldn't. I could fork it, and I can add my change to the fork.
At this point let me know how I should proceed, I've cloned the Master repo and created a local branch but am unable to push it up.

Hey pxsfir, wow, seeing this myself here (GF 4.1.1).  I can see your workaround.  I'm wondering if it would be a decent permanent 'fix'?  I'm technically running Payara, but samething.

This sounds like a logical fix to me. I'd definitely be interested in a pull request to this effect.

Does seem like a container lifecycle issue and it is timing sensitive I think (i.e, on a large war/ear, it might behave inconsistently).  Curious if there is another way to do this...  Looking at options that end with the same result @pxsfir has come up.

The @PostConstruct annotation should work on any J2EE container therefore
moving the code under this annotation should not create any different
behavior.
For now I just extended AbstractJaxRsProvider and created my own version.
Ideally it would be nice to roll this change in the core project.
public abstract class AbstractRsResourceProvider
extends AbstractJaxRsProvider
On Thu, Jun 22, 2017 at 3:53 PM Gary Teichrow<EMAIL_ADDRESS>wrote:

Does seem like a container lifecycle issue and it is timing sensitive I
think (i.e, on a large war/ear, it might behave inconsistently). Curious if
there is another way to do this... Looking at options that end with the
same result @pxsfir https://github.com/pxsfir has come up.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jamesagnew/hapi-fhir/issues/606#issuecomment-310484760,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKM1252xuMsTf23KWb7yNHlc27Y8CPiMks5sGsZNgaJpZM4MlY7w
.

Sounds like a plan.  I'll take your approach short term and wait for it to show up.  Cheers.

@pxsfir, not sure if you are using AbstractJaxRsPageProvider, but do you think it needs the same approach with a PostContruct() method?

Im not using as of yet. The issue I had was with this.
/**
 * This post constructor is necessary so a J2EE container (e.g.,

Glassfish) can initialize the bindings at the appropriate time)
*/
@PostConstruct
protected void setUpPostConstruct() {
if (theProviderClass == null) {
theProviderClass = getClass();
}
theBindings = JaxRsMethodBindings.getMethodBindings(this,
theProviderClass);
}
Basically theBindings werent initialized in the timing that they need to be
initialized in Glassfish so moving it to a @PostConstruct took care of the
issue.
On Thu, Jun 22, 2017 at 6:35 PM Gary Teichrow<EMAIL_ADDRESS>wrote:

@pxsfir https://github.com/pxsfir, not sure if you are using
AbstractJaxRsPageProvider, but do you think it needs the same approach with
a PostContruct() method?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jamesagnew/hapi-fhir/issues/606#issuecomment-310521431,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKM121gD9DGxWq8xDBoRlwC7Jv3dtexxks5sGuw-gaJpZM4MlY7w
.

@pxsfir, It does seem to be an issue with AbstractJaxRsPageProvider having the same symptom and I took a similar approach to what you've described adding a PostConstruct aspect and removing that code from the two constructors into that new method:
        @PostConstruct
        public void postContruct() {
	    try {
	        myBinding = new PageMethodBinding(getFhirContext(), PageProvider.class.getMethod("getPage"));
	    } catch (Exception e) {
	        throw new ca.uhn.fhir.context.ConfigurationException(e);
	    }
        }

If you'd like to make this change and submit with your pull, go for it.  If you'd like me to, I'll do that.
Cheers.


Add pylint checking to GitHub workflow

See https://github.com/Senzing/stream-loader/issues/303

Pull requests:

https://github.com/Senzing/g2explorer/pull/52


Request to host package - PureMVC

This issue was originally filed by<EMAIL_ADDRESS>
Hi there,
I've updated the PureMVC library to deal with all the recent changes to Dart. PureMVC is already hosted on pub.dartlang.org, it just needs to be updated.
Currently version 2.0.0 is hosted:
http://pub.dartlang.org/packages/puremvc
The repository is here:
git://github.com/PureMVC/puremvc-dart-multicore-framework.git
Please use tag 2.0.1
Cheers,
-=Cliff>

This issue has been moved to dart-lang/pub#267.


ERROR No connection available for failover

Whenever I am using the library and try to connect the client with an ENPS server using the client code given in examples folder of repository I am getting the error ERROR No connection available for failover. However I am able to receive the heartbeat message from the server.
Please assist.

MOS-connection supports connecting both to a primary and a secondary ("buddy") ENPS server (to use for fallback in case the primary connection fails).
The error message No connection available for failover shows up when there is no secondary server configured and the primary link fails for some reason (usually network-related).
If you're sure that the connection to the primary server is good (pings on both the "upper" & "lower" port), there might be a bug in the code for failover. In that case, could you activate the debug-flag in the connection-config-options and post any relevant logs?

Hi, I have kept the debug True and ran client and got the following stack:
registerOutgoingConnection 0
registerOutgoingConnection 1
registerOutgoingConnection 2
registerOutgoingConnection 3
registerOutgoingConnection 4
registerOutgoingConnection 5
Connect client 0 on lower on host X.X.X.X
2020-02-14T07:18:32.870Z 'Socket lower attempting connection'
port 10540 host X.X.X.X
Connect client 1 on upper on host X.X.X.X
2020-02-14T07:18:32.872Z 'Socket upper attempting connection'
port 10541 host X.X.X.X
Connect client 2 on query on host X.X.X.X
2020-02-14T07:18:32.872Z 'Socket query attempting connection'
port 10542 host X.X.X.X
Connect client 3 on lower on host
2020-02-14T07:18:32.873Z 'Socket lower attempting connection'
port 10540 host
Connect client 4 on upper on host
2020-02-14T07:18:32.875Z 'Socket upper attempting connection'
port 10541 host
Connect client 5 on query on host
2020-02-14T07:18:32.875Z 'Socket query attempting connection'
port 10542 host
new mosDevice:  mosID_primaryID null
exec command ReqMachInfo { port: 'lower' }
switching connection
Heartbeat on lower: Error: Can't prepare message: ncsID missing
Heartbeat on upper: Error: Can't prepare message: ncsID missing
It lands here
ERROR No connection available for failover
rawMessage incoming_6 newConnection From ::ffff:<IP_ADDRESS>:53748
Added socket:  6
rawMessage incoming_7 newConnection From ::ffff:<IP_ADDRESS>:49580
Added socket:  7
rawMessage incoming_8 newConnection From ::ffff:<IP_ADDRESS>:59158
Added socket:  8
Socket event error: connect ECONNREFUSED X.X.X.X:10542
Socket closed with error
Socket should reconnect
MOS command sent from lower :
primaryID
mosID
2

2020-02-14T07:18:32,872

bytes sent: 336
rawMessage primary sent
primaryID
mosID
2

2020-02-14T07:18:32,872

MOS command sent from upper :
primaryID
mosID
3

2020-02-14T07:18:32,873

bytes sent: 336
rawMessage primary sent
primaryID
mosID
3

2020-02-14T07:18:32,873

rawMessage primary recieved
mosID
primaryID
2

2020-02-14T07:18:33

lower Received:
mosID
primaryID
2

2020-02-14T07:18:33

Heartbeat on lower received.
rawMessage primary recieved
mosID
primaryID
3

2020-02-14T07:18:33

upper Received:
mosID
primaryID
3

2020-02-14T07:18:33

Heartbeat on upper received.
Heartbeat on lower: Error: Can't prepare message: ncsID missing
Heartbeat on upper: Error: Can't prepare message: ncsID missing
2020-02-14T07:18:35.874Z 'Socket query attempting connection'
port 10542 host X.X.X.X
Socket event error: connect ECONNREFUSED X.X.X.X:10542
Socket closed with error
Socket should reconnect
Heartbeat on lower: Error: Can't prepare message: ncsID missing
Heartbeat on upper: Error: Can't prepare message: ncsID missing
2020-02-14T07:18:38.875Z 'Socket query attempting connection'
port 10542 host X.X.X.X
Socket event error: connect ECONNREFUSED X.X.X.X:10542
Socket closed with error
Socket should reconnect
here XXXX is host IP address, mosID is the MOS ID of registered MOS and primaryID is primary ENPS server ID.

I have resolved the issue related to ncsID and ECONNREFUSED by switching the port to 10543 as 10542 was not open at the server. @nytamin , I have confirmed that upper and lower heart beat is getting received from the server which means connection with the server is good. Please suggest what could be the issue which is still causing No connection available for failover?

If the query-port (10542) isn't open on the server you could set the dontUseQueryPort in the connection options, to suppress the ECONNREFUSED error messages related to that port.
Please try that, and if it doesn't work, please post your new logs here (and also the connectionOptions you send into yourMosConnection.connect(theseConnectionOptions).

@nytamin thanks, it worked but still I am not able to figured out that why it is not establishing connection with primary server; However I am getting the heartbeat.
Log is given below:
registerOutgoingConnection 0
registerOutgoingConnection 1
registerOutgoingConnection 2
Connect client 0 on lower on host XX.XX.XX.XX
2020-02-18T10:33:44.570Z 'Socket lower attempting connection'
port 10540 host XX.XX.XX.XX
Connect client 1 on upper on host XX.XX.XX.XX
2020-02-18T10:33:44.573Z 'Socket upper attempting connection'
port 10541 host XX.XX.XX.XX
Connect client 2 on query on host XX.XX.XX.XX
2020-02-18T10:33:44.573Z 'Socket query attempting connection'
port 10543 host XX.XX.XX.XX
exec command ReqMachInfo { port: 'lower' }
**No connection available for failover**
MOS command sent from lower : <mos>
  <ncsID>XXXXXXXXXXXXX</ncsID>
  <mosID>test.enps.mos</mosID>
  <messageID>2</messageID>
  <heartbeat>
    <time>2020-02-18T10:33:44,573</time>
  </heartbeat>
</mos>
bytes sent: 336
MOS command sent from upper : <mos>
  <ncsID>XXXXXXXXXXXXX</ncsID>
  <mosID>test.enps.mos</mosID>
  <messageID>3</messageID>
  <heartbeat>
    <time>2020-02-18T10:33:44,574</time>
  </heartbeat>
</mos>
bytes sent: 336
lower Received: <mos>
<mosID>test.enps.mos</mosID>
<ncsID>XXXXXXXXXXXXX</ncsID>
<messageID>2</messageID>
<heartbeat>
<time>2020-02-18T10:33:45</time>
</heartbeat>
</mos>
Heartbeat on lower received.
upper Received: <mos>
<mosID>test.enps.mos</mosID>
<ncsID>XXXXXXXXXXXXX</ncsID>
<messageID>3</messageID>
<heartbeat>
<time>2020-02-18T10:33:45</time>
</heartbeat>
</mos>
Heartbeat on upper received.

where XX.XX.XX.XX is the IP address of the host and XXXXXXXXXXXXX (alphanumeric) is the NCS  ID of the ENPS Server.
connection is mentioned as:
let mos = new MosConnection(new ConnectionConfig({
	mosID: 'test.enps.mos',
	acceptsConnections: true,
	profiles: {
		'0': true,
		'1': true,
	},
	openRelay: true,
	debug: true
}))
mos.init()
.then((listening) => {
	let mosdev = mos.connect({
		primary: {
			id: 'XXXXXXXXXXXXX', //alphanumeric NCS ID
			host: ' XX.XX.XX.XX', //IP of the host address
			timeout: 100000,
                        dontUseQueryPort:true
		}
    })
})

MOS id is registered with ENPS Server and MOS version selected is 2.8.4

I'm curious about the exec command ReqMachInfo { port: 'lower' } part in the logs above.
Could it be so that you're calling device.getMachineInfo() before the connection has been established (determined by mos.onConnection(),  mosDevice.onConnectionChange() or mosDevice.getConnectionStatus())?
If you've used the example code and there is something wrong with it, feel free to send in a PR with a working example.

@nytamin I have commented LOC 1023, 1024 and 1025 in MOSDevice.ts file, and it worked for me. I am able to fetch the machine info, RO lists and stories.
private executeCommand (message: MosMessage, resend?: boolean): Promise<any> { 		if (this._currentConnection) { 			if (this._debug) console.log('exec command', message) 			// if (!this._currentConnection.connected) { 			// 	return this.switchConnections(message) 			// } 			return this._currentConnection.executeCommand(message).then((res) => { 				if (res.mos.roAck && res.mos.roAck.roStatus === 'Buddy server cannot respond because main server is available') { 					return Promise.reject('Buddy server cannot respond because main server is available') 				} 				return res
However, this is just a patch because this._currentConnection.connected  returns false although the connection is fine with ENPS Server.

Thanks @ripundeep , that helps me with closing in on the issue.
I've pushed a debug-branch debug/issue-49, could you try that out and post your findings in the logs?
Make sure that you have debug: true set.
If you also could add a call to console.log(mosDevice.getConnectionStatus()) right after you've called connect(), that might help.
Another thing to try: does setting openRelay: false make any difference?

Hi @nytamin
keeping debug: true and openRelay: true|false, I have got the log given below along with the ERROR No connection available for failover.
registerOutgoingConnection 0
registerOutgoingConnection 1
registerOutgoingConnection 2
Connect client 0 on lower on host HOSTIPADDR
2020-03-07T09:47:32.801Z Socket lower attempting connection
port 10540 host HOSTIPADDR
Connect client 1 on upper on host HOSTIPADDR
2020-03-07T09:47:32.804Z Socket upper attempting connection
port 10541 host HOSTIPADDR
Connect client 2 on query on host HOSTIPADDR
2020-03-07T09:47:32.804Z Socket query attempting connection
port 10543 host HOSTIPADDR
new mosDevice:  connector2_ENPSSRVRID null

**{
  PrimaryConnected: false,
  PrimaryStatus: '0 (lower): useHeartbeats: true, connected: false, 1 (upper): useHeartbeats: true, connected: false, 2 (query): useHeartbeats: false, connected: false',
  SecondaryConnected: false,
  SecondaryStatus: 'No secondary connection'
}**

exec command ReqMachInfo { port: 'lower' }
0 (lower): useHeartbeats: true, connected: false, 1 (upper): useHeartbeats: true, connected: false, 2 (query): useHeartbeats: false, connected: false
ERROR No connection available for failover
MOS command sent from lower : <mos>
  <ncsID>ENPSSRVRID</ncsID>
  <mosID>connector2</mosID>
  <messageID>2</messageID>
  <heartbeat>
    <time>2020-03-07T09:47:32,804</time>
  </heartbeat>
</mos>
bytes sent: 336
MOS command sent from upper : <mos>
  <ncsID>ENPSSRVRID</ncsID>
  <mosID>connector2</mosID>
  <messageID>3</messageID>
  <heartbeat>
    <time>2020-03-07T09:47:32,805</time>
  </heartbeat>
</mos>
bytes sent: 336
lower Received: <mos>
<mosID>connector2</mosID>
<ncsID>ENPSSRVRID</ncsID>
<messageID>2</messageID>
<heartbeat>
<time>2020-03-07T09:47:33</time>
</heartbeat>
</mos>
Heartbeat on lower received.
upper Received: <mos>
<mosID>connector2</mosID>
<ncsID>ENPSSRVRID</ncsID>
<messageID>3</messageID>
<heartbeat>
<time>2020-03-07T09:47:33</time>
</heartbeat>
</mos>
Heartbeat on upper received.

Thank you, @ripundeep .
It looks like you're calling mosDevice.getMachineInfo() before the connection has been initialized.
Please make sure that you've gotten the initial connection by using
mosDevice.onConnectionChange() or mosDevice.getConnectionStatus() before starting to send commands.
When the connection is down, it is normal for the methods to throw errors.
If there still is a problem, could you post the code you're using to try to connect?

closing this due to inactivity, feel free to reopen if this is still an issue.


Assignment 1 for python, working.

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Added functionality to anagram_checker function.
What did you learn from the changes you have made?
How to write function logic in python.
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
No others considered.
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
Getting to know how Jupyter Notebooks work.
How were these changes tested?
By running the file and observing the outputs.
A reference to a related issue in your repository (if applicable)
N/A
Checklist

[x] I can confirm that my changes are working as intended

You did a great job here!
Please go ahead and close this pull request now. Thanks!


Settings should be accesible from Assesment Screen

This helps to Change the Set easy

Also add the posibility to Push message If Assesment ist due


bug: travis CI broken in MacOS

travis CI broken in MacOS:
t/plugin/limit-count-redis.t ........... 10/?

#   Failed test 'TEST 4: up the limit - status code ok'

#   at /Users/travis/build/iresty/apisix/test-nginx/lib/Test/Nginx/Socket.pm line 935.

#          got: '500'

#     expected: '200'

#   Failed test 'TEST 4: up the limit - pattern "[error]" should not match any line in error.log but matches line "2019/10/09 08:19:42 [error] 49562#100938: *196 [lua] limit-count.lua:114: phase_fun(): failed to limit req: connection refused, client: <IP_ADDRESS>, server: localhost, request: \"GET /hello HTTP/1.1\", host: \"localhost\"" (req 0)

# '

#   at /Users/travis/build/iresty/apisix/test-nginx/lib/Test/Nginx/Socket.pm line 1280.

#   Failed test 'TEST 4: up the limit - status code ok'

#   at /Users/travis/build/iresty/apisix/test-nginx/lib/Test/Nginx/Socket.pm line 935.

#          got: ''

#     expected: '200'

t/plugin/limit-count-redis.t ........... 13/?

#   Failed test 'TEST 4: up the limit - status code ok'

#   at /Users/travis/build/iresty/apisix/test-nginx/lib/Test/Nginx/Socket.pm line 935.

#          got: ''

#     expected: '503'

fixed: https://github.com/iresty/apisix/pull/651


feat: use nested bundles in sequencer

In ideal world

user posts some kind of id to the sequencer and sequencer assigns a sequence for this id and keeps it for some time (e.g. 1 minute). The generated sequence (the 'sortkey') is returned to the user
User creates a data-item with the sort-key assigned by the sequencer, the original id (that was sent to sequencer in point 1) and all the usual interaction data, signs it and sends back to the Sequencer.
The sequencer validates the data (i.e. whether the sort-key set in the data-item is the same sort-key that sequencer generated for the id used in point 1) and if the sequence has not expired - posts the data-item to Bundlr (as a nested bundle) and indexes it in Warp Gateway.
Note: without this "handshake" in points 1 and 2 - the generated sort-key had to be set in the nested-bundle tags.

This way we could achieve 2 things:

blind sequencing (i.e. the sequencer does not know any content of the transaction that it is assigning the sequence to - so it cannot manipulate the sequence based on the transaction data)
the sort-key is stored directly in the original data-item - this will greatly simplify interactions retrieval directly from the arweave.net gateway (e.g. if someone will decide to not use the Warp Gateway)

Regarding point 1:
I'd recommend using a transaction signature as the magic Id submitted by the user. It could guarantee:
a) Verification of who sends the data for ordering -> it will allow us to charge a small fee for getting a sequence - otherwise the service could be spammed or just used strategically to permanently keep a space in the transactions queue
b) Maintaining the integrity of the transaction. The transaction accepted for processing in point 3 should be the same as the one submitted for ordering. Otherwise, the user can substitute the content or even sell a place in the queue for anyone else interested in front-running and getting a priority spot without going through step 1

You cannot use the transactions signature, because the user cannot sign the transaction before obtaining the sort key from the sequencer. Remember that we need to put the sortKey into the transaction/data-item's tags.


[WIP] Move logstash filters into PaaS Tech Docs

Hi. We've added instructions for using Logit.io to the Tech Docs for the GOV.UK PaaS (which is built on Cloud Foundry.)
We can now refer people to our Tech Docs for these instructions, rather than using the very brief explanation currently provided in the GDS Way. This PR removes the previous instructions and instead links to our expanded version.

We want to discuss internally where to link to, so I've retitled this to a WIP and please don't merge this for now.

Thanks for this! 👍 in principle and I look forward to the PR when it's ready.

Our docs have now been merged and the link has stayed as-is so this PR is good to go.
I cannot remove the WIP from the title since @46bit raised from his personal account 😭

@chrisfarms huh. I managed to remove WIP from the title 😄

@chrisfarms Sorry about that--I wasn't sure about making a paas-gds-way repository and couldn't think of a better option.


Document non-default tmux key bindings in README.md

Document this line in README.md because a user new to tmux will likely be following guides that use the default Ctrl-b instead of our Ctrl-a.
Also document the key bindings for screen splitting. We should probably not unbind the defaults.

The bindings we use are common bindings people use because the defaults are retarded, but I agree with you nonetheless.

@anassinator Could we close this already?


consider adding a section or episode on data formats

similar to https://enccs.github.io/hpda-python/scientific-data/
now there's a discussion of different packages for efficient IO in the motivation (JLD2, Feather), and it would be nice to show how they work

you can always and that link in the "see also" section, but you can also reuse whatever is most relevant


doesn't make sense.

should not MainPage in the line MessagingCenter.Subscribe<MainPage,...

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

ID: 4d80a771-2198-328b-b9fa-fe25f16261cd
Version Independent ID: be0408fe-d00b-baf6-8a9c-e24659029c4b
Content: Xamarin.Forms MessagingCenter - Xamarin
Content Source: docs/xamarin-forms/app-fundamentals/messaging-center.md
Product: xamarin
GitHub Login: @davidbritch
Microsoft Alias: dabritch

@kalexanderman can you explain your question further?

It's ok, I would have preferred to delete the comment entirely.
Regards,
Kevin
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Wednesday, October 10, 2018 6:35 PM, Craig Dunn<EMAIL_ADDRESS>wrote:

@kalexanderman can you explain your question further?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks for letting me know.


integreate file2store behaviour

lets import file2store behaviour somehow so we can write github actions that program HNS rules and use Get-HnsDiag to validate all rules and parse the json for exact conformance to the windows specification for the service proxy
this can be our initial CI / GH Action that we run for pre-merge, no k8s cluster required.

/help

this is a good first i9ssue - grab f2s.go and add it here so
windows networking developers could test their windows-service-proxy code
w/o actually making a full blown k8s cluister..... uist making a file


Wrong row is returned from sub select

Take the following two programs, the first in go, the second in perl:
package main

import (
	"database/sql"
	"fmt"
	_ "github.com/mattn/go-sqlite3"
)

func main() {
	db, _ := sql.Open("sqlite3", ":memory:")

	db.Exec("CREATE TABLE foo (name TEXT, value INTEGER)")

	db.Exec("INSERT INTO foo VALUES ('pass', 1), ('fail', 0)");

	var name string

	db.QueryRow("SELECT name FROM (SELECT name, MAX(value) FROM foo)").Scan(&name)

	fmt.Printf("%s\n", name)
}

use feature 'say';
use strict;
use warnings;

use DBI;

my $dbh = DBI->connect('dbi:SQLite:dbname=:memory:');

$dbh->do('CREATE TABLE foo (name TEXT, value INTEGER)');

$dbh->do('INSERT INTO foo VALUES ("pass", 1), ("fail", 0)');

say $dbh->selectrow_array('SELECT name FROM (SELECT name, MAX(value) FROM foo)');

And when ran you get the following:
$ go run sqlite.go
fail
$ perl sqlite.pl
pass

In my testing the go version only seems to break if you use a sub select and you select less than the total columns from the sub select.

because your query returns two rows. and the order of result is not defined. Try this.
package main

import (
	"database/sql"
	"fmt"
	_ "github.com/mattn/go-sqlite3"
)

func main() {
	db, _ := sql.Open("sqlite3", ":memory:")

	db.Exec("CREATE TABLE foo (name TEXT, value INTEGER)")

	db.Exec("INSERT INTO foo VALUES ('pass', 1), ('fail', 0)")

	var name string

	db.QueryRow("SELECT name FROM (SELECT name, MAX(value) V FROM foo order by V desc)").Scan(&name)

	fmt.Println(name)
}


Delete cert manager helm

Description
Delete Cert Manager Helm chart.
Safety checklist

[ ] This change is backwards compatible and safe to apply by existing users
[ ] This change will NOT lead to data loss
[ ] This change will NOT lead to downtime who already has an env/service setup

How has this change been tested, beside unit tests?
Tested Manually on AWS with full stack example and hello opta
CI test here

@juandiegopalomino can you take a look at this PR

Duplicates  #823 and #795


Biomass transport

add option to split biomass potential to single nodes instead of global EU node
transport of solid biomass between nodes (costs based on JRC-EU-Times Bioenergy report and depend on country and distance)
add costs for upgrading biogas to gas
smaller changes:
-> drop nan values when removing industry electricity demand to avoid warning (runtime error)
-> add color for 'industry electricity' to color map in config.yaml
-> move input data for transport to Snakefile

[ ] add tabula-py to environment (from conda-forge)


Update sbill

Add --range= option for specifying min and max of Service; to further filter jobs.

There is also a new hidden feature in --histogram for plotting histogram of CPUusage and GPUusage, but expected to be rarely use in practice.


JSON Parser Golang

Added changes to the json-parser challenges file. Thank you!

Sorry, I'm not sure I understand. I added only a single line

Please check the diff, you've also reformatted the code.

Closed and manually added.


in/2 doesn't come up in Explorer.Query documentation

Explorer.Dataframe.filter(a in ^series) seems to ~work~ at least not raise, but not be documented in Explorer.Query. I was about to add it to binary_delegates in the module, but got confused because I would imagine that to also affect how things work not just documentation.

Hmm, I think I misunderstood it not raising with it working 😅


Exception after adding Policy

After merging #7132, when I try to add Policy, I'm getting exception:
[----] F, [2016-04-07T23:15:53.520714 #9399:2b2902896038] FATAL -- : Error caught: [NoMethodError] undefined method `description' for nil:NilClass
/home/rblanco/devel/manageiq/gems/pending/util/extensions/miq-object.rb:9:in `deep_send'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:449:in `block (2 levels) in count_only_or_objects'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:449:in `collect'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:449:in `block in count_only_or_objects'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:449:in `each'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:449:in `sort_by'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:449:in `count_only_or_objects'
/home/rblanco/devel/manageiq/app/presenters/tree_builder_policy.rb:86:in `x_get_tree_po_kids'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:388:in `x_get_tree_objects'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:417:in `x_build_node'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:418:in `block in x_build_node'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:417:in `map'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:417:in `x_build_node'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:418:in `block in x_build_node'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:417:in `map'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:417:in `x_build_node'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:435:in `x_build_node_dynatree'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:313:in `block in x_build_dynatree'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:308:in `map'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:308:in `x_build_dynatree'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:245:in `build_tree'
/home/rblanco/devel/manageiq/app/presenters/tree_builder.rb:173:in `initialize'
/home/rblanco/devel/manageiq/app/controllers/application_controller/feature.rb:28:in `new'
/home/rblanco/devel/manageiq/app/controllers/application_controller/feature.rb:28:in `build_tree'
/home/rblanco/devel/manageiq/app/controllers/miq_policy_controller.rb:528:in `block in replace_right_cell'
/home/rblanco/devel/manageiq/app/controllers/miq_policy_controller.rb:522:in `each'
/home/rblanco/devel/manageiq/app/controllers/miq_policy_controller.rb:522:in `replace_right_cell'
/home/rblanco/devel/manageiq/app/controllers/miq_policy_controller/policies.rb:77:in `policy_edit'

Screenshots:

@martinpovolny @moolitayer related to #7617

cc @gtanzillo

@romanblanco what am I missing to reproduce this? I'm creating a container image compliance policy with context and notes but I cannot reproduce

@moolitayer I've found out that the problem was on my side. Everything works fine. Thanks :blush:


[Internal] improve token connection error message

Description
Improve error message when user hit authentication error when using meid_token connection
local:

cloud:

All Promptflow Contribution checklist:

[ ] The pull request does not introduce [breaking changes].
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.
[ ] Create an issue and link to the pull request to get dedicated review from promptflow team. Learn more: suggested workflow.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

@melionel - When the error message is outputted, there is a \n appended to the url link (from the call stack somewhere, as it's not in https://github.com/microsoft/promptflow/blob/main/src/promptflow-tools/promptflow/tools/exception.py#L64), which is resulting in a 404:
...You can refer to https://learn.microsoft.com/en-us/azure/ai-services/openai/how-to/role-based-access-control\n'}}]

cc - @chjinche @chenslucky


turn off default allow waypoint by default

This means out of mesh traffic like Prometheus can never reach Waypointed workloads. A service captured one must also be workload captured to make prom traffic go through the Waypoint.

@stevenctl: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
33746263bba52d603ffbc9d5461655b1ee776f87
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
33746263bba52d603ffbc9d5461655b1ee776f87
link
true
/test lint

gencheck_istio
33746263bba52d603ffbc9d5461655b1ee776f87
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
33746263bba52d603ffbc9d5461655b1ee776f87
link
true
/test lint

gencheck_istio
33746263bba52d603ffbc9d5461655b1ee776f87
link
true
/test gencheck

integ-ambient_istio
33746263bba52d603ffbc9d5461655b1ee776f87
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio
b0bf4fd7cc358ddd0e0c41254763d4bad8e780ce
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio
b0bf4fd7cc358ddd0e0c41254763d4bad8e780ce
link
true
/test integ-ambient

integ-security-multicluster_istio
b0bf4fd7cc358ddd0e0c41254763d4bad8e780ce
link
true
/test integ-security-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
b0bf4fd7cc358ddd0e0c41254763d4bad8e780ce
link
true
/test integ-security-multicluster

integ-ambient_istio
d2aa3acf19675fe004e4f74415bd8f1a7194341a
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio
c8a3970eacc963332a960ffa844649836450da48
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

In response to a cherrypick label: new pull request created: #50862

Related - https://github.com/istio/istio/issues/50870


Do an experiment where you turn off the weighted random sampler and/or visualize sampling prob distribution.

In DT/BC training I use a weighted random sampler. I'm kinda suspicious of it and want to see what happens if I turn it off. I'd also potentially like to visualize the distribution of sampling probabilities

[ ] Add a plotly graph upload of a histogram of the sampling probabilities
[ ] See if basic DT's on Dynamics Obstacles learn different when it's off/on. Also try memory env.

I've looked into this and decided it really doesn't seem like it could affect the results very much since the sampling reweighting is very minor (range is 1x to 5x but very few samples are 5x as long as median length. We can already see this in the reward/traj length graph.