Console refactoring / improvements / bug fixes

❓ Questions and Help
Following some work from @shanaxel42 in #784 to better manage our console resources it is worth raising some additional questions around the expected behavior of the console in interactive environments like when napari is launched from IPython or Jupyter.
Right now if launched from IPython the button to open the console is there but pressing it does nothing and the console is not created. I think we should remove the button and stick with the principle that if launching from IPython you do not need a built-in console in napari. How does @jni feel about this?
Right now if launched from Jupyter we actually create a console and sync it with the jupyter notebook, but there is a weird issue where you now have to press shift-enter in the console to get things to execute. This is very confusing I think, and I'm not sure that we should provide a console in this case either. I propose we remove both the console and the console button when launching from jupyter.
Finally when launching from a script that contains two napari.Viewer() like calls strange things happen where both consoles get connected to the same kernal, typing in either though only makes output appear in the 2nd created one and the viewer variable gets overwritten so only the 2nd create Viewer is referred too. This is very strange, but I'm not sure how to improve it and very open to suggestions
In general there also might be an opportunity for @shanaxel42 to remove some unneeded code from QtConsole depending on how we resolve these questions, and potentially come up with a cleaner fix for #618 at the same time

for what it's worth, I agree with all these things.  no need for icon when launching from ipython or jupyter (or at the very least no shift-enter if possible when launching from jupyter... remove altogether if not possible).  and don't clober viewer variable, but do something like create a second variable name if viewer is in globals() and emit a warning...

@jni proposed the following - drop console + button from IPython but keep for Jupyter and try and fix shift-enter.
We should allow a remote console to be attached in all cases including #801, at which point for IPython the button should appear

We should also fix #687 which is related to our console

Console loading has now been make lazy, but we still need to improve the default passing of the viewer to the console, and the clobbering that happens with multiple viewers - see https://github.com/napari/napari/pull/1055#discussion_r395432200

The remaining work to address this is now captured by #1530 and #1531 and this can be closed


[feature] Custom palette/swatch system to share same colors among elements

It is rather common to reuse the same color for many different objects of a diagram. When wanting to replace this color with another one in draw.io, it is necessary to select all objects using this color (without missing any), and to change their color all at the same time. However, this can become very tricky in complex diagrams with dozens of elements, especially when the color to change is used in different kinds of objects (text, shapes, arrow, etc.).
Inkscape solved this solution through the "palette/swatch" system, which enables the creation of labeled colors, which can then be applied to different objects. This is very powerful, because changing the color of a labeled color will change the color of all objects using this label.
Example, with two labeled colors mycolor2 and mycolor3:

And if we change mycolor2, the color of both rectangle changes automatically:

Could we imagine something similar in draw.io in the future?

I think this feature is very useful. Will drawio support this?

I think the initial feature request is still valid.
Could you please re-open it?


decimal string

It works with this format
$buy = $bfx->new_order('BTCUSD',  0.01 , 1.0, 'bitfinex', 'buy', 'exchange market');
but it doesn't work with this
$x = 0.01 ;
$buy = $bfx->new_order('BTCUSD',  $x , 1.0, 'bitfinex', 'buy', 'exchange market');
returning error
Array ( [message] => Key amount should be a decimal string. [error] => 1 )
how can I fix it?

You have to send the amount ($x) as a string:
$x = '0.01';
$buy = $bfx->new_order('BTCUSD', $x , 1.0, 'bitfinex', 'buy', 'exchange market');

P.S. sorry for the late reply I totally missed this issue.


Divergence in classification between digits and GRE

When running a digits exported model into the GRE, I am getting some divergence in the values of the classification when compared to Digits. In some cases, it is flipping the classification completely to the opposite classification with regards to a binary classifier. When using the Digits REST (non GRE), I am getting identical classifications to Digits so I am wondering where the divergence lies.
I altered the first line in the deploy.prototext of my exported model from
input: "data"
input_shape {
dim: 1
dim: 3
dim: 227
dim: 227
}
To
name: "AlexNet"
layer {
name: "data"
type: "Input"
top: "data"
input_param { shape: { dim: 1 dim: 3 dim: 227 dim: 227 } }
}
I mimicked this based off the deploy.prototext in the GRE model folder and altered the shape to coincide with my model.
I saw in a previous issue that there was a problem with varying confidence and am wondering if it is related. https://github.com/NVIDIA/gpu-rest-engine/issues/3

Are you using the caffe version of GRE?

I am using the caffe version of GRE.

The issue you mentioned had varying confidence results for the same image. Is this the case?

It was actually the 2nd post of that thread that piqued my interest regarding a difference in confidence results.

Ah yes. It might be the preprocessing that's the culprit. Unfortunately the current code is not very generic to this regard.

I'm hoping we can identify the cause of the divergence as the GRE would work perfectly for what I need it to do.

Unfortunately, it's challenging to debug since I don't have access to your model. Do you know what DIGITS is using in the pre-processing steps? Crops? resizes? Augmentation?

For the Model portion; Subtract Mean, no crop option. On the Database, the default it had was squash image size 256x256, color image encoding png. On the deploy.prototxt, the dimensions are as above for the exported model.

Any thoughts on this? I'm hoping Griffin and I can work out a solution with you.

If you pre-resize the images to the right size (227x227?) before sending them to GRE, does it change anything?

Just toss them into photoshop, do a bilinear resize to 227x227 for all the images prior to GRE? In the classification code you have, do you use opencv for that part? I wonder if I can do it there before its classified.

Yes I use OpenCV for that.
For now I'm just trying to find out if it's the resize that's the culprit. So take this 227x227 image and send it to both GRE and DIGITS.

I can test it out.

Image blilinear resize to 227x227
Digits classify.py:
0: 21.5443
1: 78.4557%
GRE:
0: 0.5819
1: 0.4181
GRE unaltered image:
0: 0.553
1: 0.447

I will try a different approach; I'm thinking since I've worked a lot with the classify.py file from digits, I can try to modify that to see if I can reproduce the same value that the GRE produces.

Ok. And it should be pretty straightforward the compare the pre-processing steps from DIGITS and gpu-rest-engine. It's just a few lines of C++ code.
I think the resize is the only piece that might differ (since you can have multiple interpolation algorithms). The other steps should produce the same results, but since we still have a divergence, it's probably that it's not the same operations that are used.

Griffin has mentioned something about mapping values or weights being done differently with regards to DeepDetect. I'm wondering if something similar is happening here. I'll have to check back with him on this idea of what was happening.

Originally I thought there may an issue with something like dimension ordering (e.g. NKHW) but it seems unlikely that this is the culprit here.  I agree with @flx42 that something like resizing may be the problem - something like a different interpolation being used.
Let me see if I can reproduce this divergence with a simple example.

I think we ruled out the resize now, since @rperdon tried with images that are already at 227x227. So the culprit is probably something else: mean subtraction, cropping, etc.

I have played further with some other images which were flipping the classifications.
I resized in photoshop, baseline compression, (previous was png, no compression), bilinear to 227x227.
Digits:
0 95.79
1 4.21
GRE
0 0.501
1 0.499
Unaltered: (1024x1024)
Digits:
0 92.2
1 7.8
GRE
0 0.042
1 0.958
I made a mistake in the Digits classify.py compared to Digits REST comparisons I was using: The default is squash and subtract pixel for classify.py while in Digits I was working with subtract image. The model I am working as with the desired results is running from Digits REST, not the classify.py so I will have to do some work to ensure that classify.py is modified to represent what Digits is doing before being able to properly compare resizing code changes.

https://github.com/NVIDIA/DIGITS/issues/169
I think this may provide key insight to adapting GRE to work like Digits. I'm just wrapping my head around the implications and hopefully we can sort this out.

There has been other back and forth on email about this issue, I will close this.


Align the latest changes of DALI pkg

@TE-TakuyaNarihira
Please push the merge button
https://github.com/sony/nnabla-examples/pull/87


Fixed imports in undoaction.py

undoaction.py was missing an import and so it was broken as soon as you tried to use it. This is just a quick fix for that.

In this case, it is the specific import! Note that it's import binaryninja not from binaryninja import *
This is because line 54 calls binaryninja._initialize_plugins() which you is an underscored name, so the module name has to be imported. I based this PR on other API files that invoke that particular method.

Yup, it's totally fine, I missed that originally. Peter was giving me a hard time about it too. ;-) LGTM.


Possible to add quilt and diff to the list?

I'm honestly shocked there isn't a portable implementation of quilt (which uses diff) to use on windows suggest windows people to use, so as you're already implementing a lot of the ecosystem i thought 'why not ask?'.

Hi! I'm just a casual contributor, so don't take this comment for official position of the project.
Judging by #696, this repo is only concerned with re-implementing GNU coreutils. Since quilt isn't part of coreutils and isn't even part of the GNU project, it most likely won't be re-implemented here.
However it looks like if you (or someone else) starts rewriting quilt, this repo's owners will be happy to reference your project to get it more visibility.

Yeah, @Minoru pretty much nailed it.  I'm open to potentially starting other projects under uutils (like maybe binutils, if someone wanted to do that), but I'd rather not pollute this project with random utilities.  I'm totally cool with helping other projects get more visibility as well.


Incorrent migration file generation for string property

Why  HasMaxLength(number) is Needed` if I already  specified when I declared the type (HasColumnType("nvarchar(50)")). Where is used  maxlength?
Migration file

Entity

Configuration by Fluent API

@armanpetrosyan23 What are you trying to achieve?

I want to ask, where is it used? Maxlength()

@armanpetrosyan23 Max length is in the migration because it was specified in the configuration. What is is "used" for depends on what you mean by "used". There's lots of things I could write about this, but without knowing more about what you specifically want to know I'm not going to randomly spew out bits of information in the hope that I cover what you are actually asking about. Please be specific about the question you are asking.

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


Message not received when broadcasted via go-libp2p-pubsub

Some nodes are implemented in golang using (go-libp2p-pubsub), but they never arrive using this js library. Is there any known issue ?

They will be able to receive the messages, but it will depend on how your overlay network is defined.
You will need to be connected to a set of libp2p nodes sharing the same pubsub topic interests, in order to receive the message. How are you establishing your node connectivity and peer discover, and how will they discover go nodes? Moreover, are you using this on Node.js or browser node?

I connect them directly to each other using peerstore and dial and they still don't receive messages. I am using electron which is executed in context of nodejs.

Can you provide us a test case for this?
I locally tried go node with js node using gossipsub (directly connected), and after subscribing I could receive the messages as expected.

I have this same issue, I cannot get the data to be received in a simple test
import Gossipsub from 'libp2p-gossipsub';
import Libp2p from 'libp2p';

test('exact reporoduction of gossip', async (done)=>{
    const Gossipsub = require('libp2p-gossipsub');
    const options = {emitSelf: true, gossipIncoming: true};
    const node1= await createNodeExact();
    const node2 = await createNodeExact();
    // await node1.start();
    // await node2.start();

    const gsub1 = new Gossipsub(node1, options);
    const gsub2 = new Gossipsub(node2, options);

    await gsub1.start();
    await gsub2.start();

    gsub1.on('fruit', (data) => {
      console.log(data)
      done();
    });
    gsub1.subscribe('fruit');

    gsub2.publish('fruit', new TextEncoder().encode('banana'))
  }, 20e3);

  async function createNodeExact() {
    const transportKey = TCP.prototype[Symbol.toStringTag];
    const defaultConfig = {
      addresses: {
        listen: ['/ip4/<IP_ADDRESS>/tcp/0']
      },
      modules: {
        transport: [TCP],
        streamMuxer: [MPLEX],
        connEncryption: [NOISE],
        peerDiscovery: [MulticastDNS],
        pubsub: Gossipsub
      },
      config: {
        pubsub: {
          emitSelf: true,
          enabled: false
        },
        peerDiscovery: {
          autoDial: true,
          [MulticastDNS.tag]: {
            interval: 1000,
            enabled: true
          }
        },
        transport: {
          [transportKey]: {
            filter: filters.all
          }
        }
      }
    };

    return Libp2p.create(defaultConfig);
  }

Hey @jerradpatch
For your nodes to be able to properly exchange messages they need to be connected to a set of common peers (you can look into the spec: https://github.com/libp2p/specs/blob/master/pubsub/gossipsub/gossipsub-v1.0.md). If the peers are directly connected to each other, they will be able to to exchange pubsub messages without any problems, but if they are not they will need to have the minimum number of common connected peers to be able to exchange pubsub messages reliably. You can see the tests in https://github.com/ChainSafe/js-libp2p-gossipsub/blob/master/test/go-gossipsub.js for tests that create these connectivity scenarios, denseConnected and sparseConnect


Expose statistics and cluster API's

Any plans to expose endpoints such as /v2/stats/* and /v2/members in this client?

We don't have that plan so far.
The client pkg https://github.com/coreos/etcd/tree/master/client has
implemented /v2/members. You could give it a try.
On Wed, Mar 4, 2015 at 1:00 PM, Brian Akins<EMAIL_ADDRESS>wrote:

Any plans to expose endpoints such as /v2/stats/* and /v2/members in this
client?
—
Reply to this email directly or view it on GitHub
https://github.com/coreos/go-etcd/issues/191.

Thanks. I will take a look at that. It's somewhat confusing that there are two clients, however.

It is in experimental stage, and we are pushing forward step by step.
On Mon, Mar 9, 2015 at 11:28 AM, Brian Akins<EMAIL_ADDRESS>wrote:

Thanks. I will take a look at that. It's somewhat confusing that there are
two clients, however.
—
Reply to this email directly or view it on GitHub
https://github.com/coreos/go-etcd/issues/191#issuecomment-77913419.

Is this https://github.com/coreos/etcd/tree/master/client going to replace go-etcd at some point?

@victorcoder Yes, the intention is to replace go-etcd with the github.com/coreos/etcd/client package.


Ensure to draw unused markers in the range of scroll bars

Fix #985.

@vasily-kirichenko Can you check this first? Thanks.

I've checked it carefully but forget to merge :( Sorry.


W10M Template 10 HamburgerMenu DetailPage disapears after resume

Visual Studio 2015 sp3.
Template 10 version 1.1.12

Select new project template 10 (hamburger).
build project to mobile device or emulator.
Stop debug.
Start the app from device.
The text box label "Parameter to Pass" value Gas. 2. is showed.
Click windows button (home) on device.
Select "all apps ->" and Click on icon "Template10App" agin.
Now i only see the hamburger menu in the top left corner. label "Parameter to Pass" value Gas. 2 is gone.

//Joacim Wall

I will investigate.

All is fine.
Just change ExistingContent.Exclude to ExistingContent.Include for the following line in public override UIElement CreateRootElement(IActivatedEventArgs e) {App.xaml.cs).
public override UIElement CreateRootElement(IActivatedEventArgs e)
{
    ...

    var navService = NavigationServiceFactory(BackButton.Attach, ExistingContent.Include);

    ...
}

Thanks now it works prefect.


🐛 Bug Report: addTemplateGlobals does not properly support callables

📜 Description
The addTemplateGlobals function appears to be bugged.
Static values work just fine, but callables are not executed properly. They are either reduced to empty strings or result in broken strings.
#13704 Added the functionality to configure globals and also added test for both constants and callables.
👍 Expected behavior
Given the following module:
import { createBackendModule } from '@backstage/backend-plugin-api';
import { scaffolderTemplatingExtensionPoint } from '@backstage/plugin-scaffolder-node/alpha';
import { randomUUID } from 'crypto';

export const scaffolderModuleGuidGenerator = createBackendModule({
  pluginId: 'scaffolder',
  moduleId: 'guid-generator',
  register(reg) {
    reg.registerInit({
      deps: {
        scaffolder: scaffolderTemplatingExtensionPoint,
      },
      async init({ scaffolder }) {
        scaffolder.addTemplateGlobals({
          guid: () => 'randomUUID()',
          guidT: (typeof( () => randomUUID())),
          guidNonReserved: () => randomUUID(),
        });
      },
    });
  },
});

and this template:
apiVersion: scaffolder.backstage.io/v1beta3
kind: Template
metadata:
  name: test
  title: Test
  description: Test
spec:
  owner: user:guest
  type: service

  parameters: []

  steps:
    - id: fetch-base
      name: Fetch Base
      action: fetch:template
      input:
        url: ./src
        targetPath: .
        values:
          g1: ${{ guid }}
          g2: ${{ guidT }}
          g3: ${{ guidNonReserved }}

with a file in ./src containing:
${{ g1 }}
${{ g2 }}
${{ g3 }}
${{ guid }}
${{ guidT }}
${{ guidNonReserved }}

There should be (callable) globals accessible to the templater which return
randomUUID()
function
<some random UUID>
So the scaffolder should pass
{
   "url": "./src",
   "targetPath": ".",
   "values": {
     "g1": "randomUUID()",
     "g2": "function",
     "g3": "<some random UUID>",
   }
 }

to the fetch:template action.
The templated file should contain
randomUUID()
function
<some random UUID>
randomUUID()
function
<some other random UUID>

👎 Actual Behavior with Screenshots
{
   "url": "./src",
   "targetPath": ".",
   "values": {
     "g2": "function"
   }
 }

Is passed to the fetch:template action.
The templated file contains

function

(...args) => JSON.parse(callGlobal(name, args))
function
(...args) => JSON.parse(callGlobal(name, args))

scaffolder.addTemplateFilters({
  guid: () => randomUUID(),
});

works just fine with ${{ "" | guid }} but does not declare intend and results in hideous templates.
👟 Reproduction steps
Refer to expected behavior.

Add a module with the given code.
Add a template with the given step.
Add a file to the src subdir of the template with the given content.
Use the Template Editor to load and execute the template.

📃 Provide the context for the Bug.
I wanted to extend the scaffolder with a guid generator to fill in some guids in the service template. They are referenced multiple times and need to therefore be assigned to a value.
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
No, but I'm happy to collaborate on a PR with someone else

I was able to call the function in the template:
  steps:
    - id: fetch-base
      name: Fetch Base
      action: fetch:template
      input:
        url: ./src
        targetPath: .
        values:
          g1: ${{ guid() }} # 👈🏻
          g2: ${{ guidT }}
          g3: ${{ guidNonReserved() }} # 👈🏻

But as you said, it resolves to an empty string when the function is not called before being passed as a values entry:
  steps:
    - id: fetch-base
      name: Fetch Base
      action: fetch:template
      input:
        url: ./src
        targetPath: .
        values:
-          g1: ${{ guid }} # 👈🏻
+          g1: ${{ guid() }} # 👈🏻
          g2: ${{ guidT }}
-          g3: ${{ guidNonReserved() }} # 👈🏻
+          g3: ${{ guidNonReserved }} # 👈🏻

I couldn't find documentation for this, so I wanted to check with @kissmikijr and @backstage/scaffolder-maintainers if it is expected that global functions can only be called in template files.

It was quite some time ago I think I tried to copy the additionalTemplateFilters behaviour could be that I missed some steps and the fetch:template does not get the config for its nunjucks

On the last SIG we discussed this request and that's not what we are going to support.
Templates work with serialized objects, and serializing functions can go messy really quick.

@camilaibs That's a bit embarrassing, but would explain the signatures in the expansion. I swear my journey for documentation did not specify function calls anywhere, but I found it here.
It would be nice if additionalTemplateGlobals would get some love in the documentation. Even, if they will not be supported in fetch:template anytime soon.

@VeithBuergerhoff would you like to contribute some of the documentation changes that you think would be helpful for others :pray:

@benjdlambert Gladly! Would around here be a good place to mention additionalTemplateGlobals?

@VeithBuergerhoff yes, seems sane!

I was busy with exams, work, and moving, but I finally got around to updating the docs.

Closing as this has been fixed :tada:


Unable to sign in or to recover a password after creating a user

I have created a new user, I have received the confirmation email and have clicked the link.

When I click "sign in" an enter my details the following message appears:
"Authentication failed! Please check your credentials and try again."
When I click "Forgot password?" and enter my email, it says the following message:
"E-Mail address isn't registered! Please check and try again."
When I try to create a user using the same email, says the following message:
"Registration failed! Please try again later."

It looks like if the user has not been created but the email has been saved in the database.

I have checked the status of the DB in the Prod environment.
There are three records for Miquel Bonet but none is activated, so something must have gone wrong during activation.

I registered using three different emails. Each time, I received the activation email but when I tried to sign in it didn't recognise my credentials.

Tested in prodaction and it works properly.


Use WinRPM for Windows binaries

I put libgit2 up on WinRPM. It's at 0.21.3 right now. This is easier to update in the future using the github-like interface on the openSUSE build service here: https://build.opensuse.org/package/show/windows:mingw:win64/mingw64-libgit2 and https://build.opensuse.org/package/show/windows:mingw:win32/mingw32-libgit2
WIP until AppVeyor runs on it.

Coverage remained the same when pulling 1cf56bb6f8b857aa487604e946b46357e441818d on tkelman:patch-2 into 238915dc9da563a6f4d1d974fb3b3b2be4ca821e on jakebolewski:master.

This win32 error (which was present before): ERROR: test failed: (Signature("test1","test1@test.com","2015-01-08 12:21:45 Coordinated Universal Time") == Signature("test1","test1@test.com","2015-01-08 12:21:45 Coordinated Universal Time"))
probably indicates a 32 bit integer issue somewhere.

No longer relevant since 0.3 is not supported any more. And I don't think anyone has been using this package on 0.4 either afaik.


Astar demo doesn't run

May simply be issue with current Godot master, but putting here just in case.
Editor gives error at line 114 of pathfind_aster.gd:

Parser Error: Function signature doesn't match the parent. Parent signature is: 'NodePath get_path()'

Replacing occurrences of get_path() with get_path_thing() resolves error and project runs fine.

Godot 3.1 isn't out yet. Yes it does work in the stable release

Replacing occurrences of get_path() with get_path_thing() resolves error and project runs fine.

I've done this and run into a new error.

Parser Error: Identifier 'IDLE' is not defined in the current scope.

The demos are for Godot 3.0, not Godot 3.1. If you want to port it to Godot 3.1, PRs are welcome.
To fix this error you've got to access the enums' members explicitly:
enum EnumName {IDLE, WALK, ...}

EnumName.IDLE```


Daily update

Automated changes by create-pull-request GitHub action

Build failed because of:
no headquarter addresses for ratio-d20a at https://www.crunchbase.com/organization/ratio-d20a
Empty twitter for CSIRO/DATA 61: https://twitter.com/CSIROnews
Empty twitter for TYNYBAY Inc: https://twitter.com/tynybayinc
Empty twitter for Truepic, Inc.: https://twitter.com/truepicinc
Empty twitter for Tectonic Labs Ltd.: https://twitter.com/cloud_ridge
Empty twitter for eBaoTech International Pte Ltd: https://twitter.com/Ebaotech
Empty twitter for China Telecommunications Corporation: https://twitter.com/chinatelecom1
Ratio either has no crunchbase entry or it is invalid

Build failed because of:
no headquarter addresses for ratio-d20a at https://www.crunchbase.com/organization/ratio-d20a
Empty twitter for CSIRO/DATA 61: https://twitter.com/CSIROnews
Empty twitter for TYNYBAY Inc: https://twitter.com/tynybayinc
Empty twitter for Tectonic Labs Ltd.: https://twitter.com/cloud_ridge
Empty twitter for Truepic, Inc.: https://twitter.com/truepicinc
Empty twitter for eBaoTech International Pte Ltd: https://twitter.com/Ebaotech
Empty twitter for China Telecommunications Corporation: https://twitter.com/chinatelecom1
Ratio either has no crunchbase entry or it is invalid


Upgrade to mesos-utils-0.21.0-1

See also #791.

any update on how this could be achieved as of current versions ?
Thanks


Custom URLProtocol doesn't work with firebase calls

Xcode version: 8.3.3 and 9.0 beta 3
Firebase SDK version: 4.0.0
Firebase Product: auth, database

I use a custom URLProtocol class in my tests. If I register my URLProtocol class and use URLSession to load a URL, the custom class is used, but if I make a call to any firebase API, the custom class doesn't get used.
Steps to reproduce:

Write a URLProtocol subclass, implement canInit(with request:), set a breakpoint here
In the setUp() method of your test class, add URLProtocol.registerClass(CustomURLProtocol.self)
Write a test and load any URL using URLSession.shared.dataTask(with:)
Run the test and notice the breakpoint in canInit gets hit
Change the test to call Auth.auth().createUser or Database.database().reference().child(path).setValue
Run the test and notice the breakpoint in canInit does not get hit

Sorry, I think I misunderstood how registerClass would work. It works for the shared instance of URLSession, but not one with a set configuration. I'll look for another way to test.


Pull-in C++/WG21 biblio references directly from their source

Currently C++/WG21 references are pulled-in through Specref.
They're now available in a Specref-like format directly from https://wg21.link/specref.json.
Handling these in Specref frequently causes the auto-update process to run out of allocated memory and be terminated by Heroku. In turn, this blocks updates from W3C, WHATWG, WICG and IETF sources, sometimes for weeks.
There obviously are technical solutions to this issue, but given the only usecase for having them in Specref to begin with is so they can be used in Bikeshed, it makes sense to avoid having them in Specref altogether.
Copying @jfbastien, @jyasskin, and @m-ou-se so they can help adapt the format if necessary.

I'm happy to consume it directly if you can keep it in SpecRef's GitHub.
Having them present in SpecRef is definitely the correct thing eventually, tho.

Or I guess wg21 is the one exposing it, so that's fine.

Haven't had time to pay attention due to family things, but from our IRC convo a few days ago that sounds like the right thing. @m-ou-se maintains wg21.link fwiw. On thing missing would be https 😊

@jfbastien you'd still get the Specref references, just Specref would host yours.

Okay, I'm getting it from wg21.link now. You can remove it from SpecRef, @tobie.


CRAYSAT-1837: Improve python-csm-api-client builds

IM: CRAYSAT-1837
Reviewer: Ryan
python-csm-api-client builds were taking approximately 35 mins to complete the build.
Will improvise it to take less amount of time by adding poetry.lock file that resolves the dependencies
Summary and Scope
python-csm-api-client builds are taking 35 minutes
To improvise the build to take less amount of time, we added the poetry.lock file which is an auto generated script from Poetry 1.5.1
poetry.lock file already has its transitive dependencies.
As we added poetry.lock file Jenkins does not need to resolve its dependencies, which takes a minimal amount of time.
Issues and Related PRs
Resolves CRAYSAT-1837.
Testing
List the environments in which these changes were tested.
Test description:
Will observe the Jenkins build pipeline.
It takes only 1m 36s to complete the build pipeline after adding poetry.lock file
Risks and Mitigations
Minimal
Pull Request Checklist

[ ] Version number(s) incremented, if applicable
[ ] Copyrights updated
[ ] License file intact
[ ] Target branch correct
[ ] CHANGELOG.md updated
[ ] Testing is appropriate and complete, if applicable
[ ] HPC Product Announcement prepared, if applicable

To write lock file for the first time it taken 3649.0s
(myenv) cray:~/python-csm-api-client # poetry lock
Updating dependencies
Resolving dependencies... Downloading https://files.pythonhosted.org/packages/96/b6/e1221bd711c54a2fc511a4d081fa1216163951863685b
Resolving dependencies... Downloading https://files.pythonhosted.org/packages/96/b6/e1221bd711c54a2fc511a4d081fa1216163951863685b
Resolving dependencies... Downloading https://files.pythonhosted.org/packages/96/b6/e1221bd711c54a2fc511a4d081fa1216163951863685b
Resolving dependencies... (3649.0s)

Writing lock file

(myenv) cray:~/python-csm-api-client # ls
.git     .gitignore    Jenkinsfile.github  README.md      coverage.cfg    poetry.lock       pyproject.toml  tools
.github  CHANGELOG.md  LICENSE             build_scripts  csm_api_client  pycodestyle.conf  tests           unittest.cfg


Crash when trying to release another player character

Exception: Object reference not set to an instance of an object. (System.NullReferenceException) Target site: Boolean IsClientOwner(Barotrauma.Networking.Client) Stack trace:     at Barotrauma.Character.IsClientOwner(Client client) in <DEV>\Barotrauma\BarotraumaServer\ServerSource\Characters\CharacterNetworking.cs:line 48    at System.Collections.Generic.List1.Find(Predicate1 match)    at Barotrauma.Networking.GameServer.Update_Patch2(GameServer this, Single deltaTime)    at Barotrauma.GameMain.Run() in <DEV>\Barotrauma\BarotraumaServer\ServerSource\GameMain.cs:line 352    at Barotrauma.Program.Main(String[] args) in <DEV>\Barotrauma\BarotraumaServer\ServerSource\Program.cs:line 89

This looks like and issue with the base server, although maybe they made it so you can't release a client from the character?
Write down concrete steps to reproduce.


using default emoji config

Most of the methods use these utils. so by default they should use TWITTER_TEXT_DEFAULT_CONFIG
Problem
By default com.twitter.twittertext.Validator#isValidTweet must use emoji parsing enabled config
so that emoji char count matches in both java nad js
Solution
as TWITTER_TEXT_DEFAULT_CONFIG is already set, we must use that in default method to avoid confusion
Result
this will greatly avoid confusion and developer need not modify any code to use it

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

We've made this change internally and will be distributing it back here to GitHub next time we push an update. Hoping to get to that push this week.


Manager/installer specification document

Create a specification document standardizing how to download and install packages.
There is a lot to align on such as:

Calling the Registry API
Filtering packages by type and relevance
supporting image/audio preview files
choosing which binary to download
downloading and extracting files
creating folders and copying files into correct locations
scanning for local packages

Work in progress:
https://docs.google.com/document/d/1Txsx98QPjp8UXvbOE3bW5tFJ3cIzlBRtCPmClYcfepU/edit

will the reviews, comments, user submitted audio clips, etc. be collected on each app / cli / website in different repos or will they all be aggregated and sent upstream and downstream

Currently reviews/comments are out-of-scope of these documents:

Registry plugin database
Manager plugin download/installer

I think the Registry specification could be extended to support reviews, but would need some thought to it.


jquery.datetimepicker.full.js is missing from the package

This was a surprise to me, too. Can we revert this line? https://github.com/xdan/datetimepicker/commit/9de4639abf855d2aaec6cf53ffbeee61618c99e8#diff-eacf331f0ffc35d4b482f1d15a887d3bL81
This is not a patch level change since it is not backwards compatible.


WIP JSON frontend to Kriti

Might be nice having a Kriti frontend that is also valid JSON. This is a spike to experiment with that idea.

I'm interesting in exploring this idea further but it seems like it is on the back burner for now. I'm going to close the PR. We can reopen it when we decide to revisit the idea.


openssl error for lan devices

What issue do you have? Please be as thorough and explicit as possible.
plugin is failing to load, error:
[12/12/2022, 9:53:30 AM] [Govee] [HTTP] disabling client as Could not find openssl on your system on this path: openssl at /homebridge/node_modules/homebridge-govee/node_modules/@bwp91/pem/webpack:/pem/lib/openssl.js:257:1.
Details of your setup.

Do you use (1) Homebridge UI-X (2) Homebridge CLI or (3) HOOBS?

Homebridge UI-X

Which version of Homebridge/HOOBS do you have?

Homebridge v1.6.0

Which platform do you run Homebridge/HOOBS on (e.g. Raspberry Pi/Windows/HOOBS Box)? Please also mention your version of Node.js/NPM if known.
Docker

Which version of this plugin (homebridge-govee) do you have? Has the issue started since upgrading from a previous version?
7.5.1 - It was working, but not sure of the previous version when it was.

Which Govee devices do you have that are causing issues? Please include product models if applicable.
Govee WiFi Thermometer Hygrometer H5179 - 3 of them

Please paste any relevant logs below.

Nevermind, simple fix.   Installed openssl via apk add openssl.  Working again.


chore: remove vitest globals types from base tsconfig

Summary

The tsconfig.json file includes types for vitest/globals. Those types are already included in tsconfig.test.json which makes them available in test files. If we include the types in tsconfig.json we make them available in all files.
This is, for example, what happens in packages/solid/src/index.ts:

After this change a TypeScript error is shown instead:

How did you test this change?

I run pnpm run build && pnpm run test && pnpm run types.
This is my first PR to this project so please let me know if I missed something.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.37%

Totals

Change from base Build<PHONE_NUMBER>0:
0.0%

Covered Lines:
4193

Relevant Lines:
4427

💛  - Coveralls

Thanks


Substrate contracts tester

Signed-off-by: Sean Young<EMAIL_ADDRESS>

Changes are in https://github.com/paritytech/pallet-contracts-waterfall/ instead.


[DOCS] Add a security policy

Description
Add a security policy to be used in all repositories which don't have one.
Specific repositories can override the security police to provide more details.
Screenshots
No response
Additional information
What do you think about this? Thanks. 🙂

We can add a SECURITY.md file to your repository's root, docs.


Fix billing API bootstrap

Currently, if you use the script for bootstrapping vumi go, the billing API will fail to start. This is because it's still trying to use an old django admin command, instead of twistd. The plan is to remove the old command, and to change the supervisor command to use twistd. Also, we will need to update the readme to have postgres setup steps, as the billing api does not support sqlite.

Ready for review.

One small question -- otherwise looks good!

:+1:


csi-manila: roll-back storage resources on failure

What this PR does / why we need it:
csi-manila currently leaks Manila resources if an error occurs during share/snapshot provisioning, they're simply left on the backend without any sort of clean up.
This PR first determines the type of the error that has occurred using the Status field of the share/snapshot and User messages API (better error reporting, easier troubleshooting for the users!) and then attempts to clean up the failed resource.
User messages API requires minimum Manila microversion 2.37 (Queens)
Release note:

CSI Manila now requires Manila version >=2.37

/ok-to-test

/lgtm

/approve


Update .travis.yml: add newer node versions

remove 0.11 in favor of 0.12, add 4.2 (LTS) and 5.6 (latest)

Tests are failling due to the new Set, Map and Intl objects on the global scope, could you be able to fix the tests to accept them? Also update the dependencies versions would be cool :-)

Looks like the leaks are from an outdated version of lab, so updating the dependencies should resolve that.
I'll get on it :-)

addressed in #20, thanks for the contribution @PixnBits \o/

Rats, pushed my changes to the wrong branch (see #15).
However, I see from Travis that @bcoe also committed some dep updates.
Heh, that's what I get for committing while tired :-P

And I'm 2min late to comment, ha ha ha
Good on ya, @bcoe!


Use of crates that are written with smol with other runtimes

I planning to migrate a small library project that I have to smol. Before I start the migration I wanted to see how crates written with smol can be used in projects written with tokio. So I wrote the following example:
use futures::prelude::*;
use smol::{Async, Task};
use std::net::TcpStream;

async fn foo() {
    Task::spawn(async {
        let mut buf = [0u8; 1024];
        let mut s = Async::<TcpStream>::connect("<IP_ADDRESS>:4444").await.unwrap();

        s.write_all(b"write something\n").await.unwrap();
        let len = s.read(&mut buf).await.unwrap();

        println!("{}", std::str::from_utf8(&buf[..len]).unwrap());
    })
    .await
}

#[tokio::main]
async fn main() {
    // start smol runtime in a thread
    std::thread::spawn(|| smol::run(future::pending::<()>()));

    foo().await;
}

I notice that without std::thread::spawn(|| smol::run(future::pending::<()>())) my example doesn't work.
My question now is: Should I require from users of my crate to have at least one instance of smol::run? What if smol spawns a new thread automatically if is no smol::run instance exists?

Here's how you can make smol and tokio work together...
Solution 1: set up tokio context inside smol:
use std::thread;

use futures::prelude::*;
use tokio::runtime;

#[tokio::main]
async fn main() {
    let handle = runtime::Handle::current();
    thread::spawn(move || handle.enter(|| smol::run(future::pending::<()>())));

    // ...
}

Solution 2: enable the "tokio02" feature in smol, which creates a tokio context inside smol for you:
fn main() {
    smol::run(async {
        tokio::task::spawn(async {
            // ...
        });

        // ...
    })
}

I think the decision on what to do should be left to the user. After all, the user is the one that is implementing the main function.

Thanks! I forgot about handle::enter.


Adjusting timing of ranking, payout, and announcement schedules

Ongoing discussion on adjusting to improve accuracy and creating some buffer against errors. Relates to announcement schedules and timing.
@hstove @GinaAbrams @cuevasm @pstan26 please contribute more thoughts.

@GinaAbrams is leading this and the doc is currently in progress

Updated the FAQ.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/24405

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/26062

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/27710

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/27710

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/30996

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/31009

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/31054

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/31443

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/32232

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/33407

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/36160

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/36782

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/37606

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/42640

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/88543

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/88793

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/90691


massive dupe glitch with workbench in cyclic

Minecraft version & Mod Version:
1.12.2 1.12.2-1.10.5
Single player or Server:
single player
Describe problem (what you were doing / what happened):
using the workbench from cyclic to craft items with a full inventory causes items to duplicate enmasse and is easily repeatable
Crash log (if any):
no crash log but bad dupe glitch
(If this is a suggestion and not a bug report, delete all this and just write your suggestion!)

I just tried crafting with left click and with right click and shift click, I dont see this happen yet.  Can you explain in a bit more detail please? Was it a specific vanilla or modded item?  Maybe with a screenshot or gif?   http://www.screentogif.com/
This is a gif of me crafting stairs trying to make it glitch

i crafted a hopper and paper it duped the 3 sugarcane into like 12 stacks of sugarcane and the 1 hopper into 2 basically and ill try to screenshot if i can

also it was shift clicking the items into the table and shift clicking the output not normal clicking

Thanks for the reply ill keep testing.

Yep i somehow made a stack of 84. (not going to list exact steps to reduce exploits). Fix out soon

I got a fix into latest version, let me know if it works for you https://minecraft.curseforge.com/projects/cyclic/files/2511574

umm its still there in the latest version thats on curseforge


Add character:  to font:

Character :
Font :

@codewithnick
Forgot to ask to for assignment prior to PR, sorry.

@komal7141, I am giving this to @jonbrohauge as he has already done the PR. @jonbrohauge request for assignment before initiating a PR for any further contributions.


Support vLLM as the backend

/assign @Jerry-Kon

Solved by https://github.com/InftyAI/llmlite/pull/42

We should support batch messages.


Fix IL panic: concurrent read/write from map

Adds RWLocks around the map that was being concurrently accessed @ozevren
Add a test case to induce this failure.  @mandarjog

Release note:

NONE

This change is

Fixes #1282

@mandarjog: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-presubmit.sh
fddf02c003212fec1bb9d29e207432249650185c
link
/test mixer-presubmit

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.

/lgtm

/approve

@mandarjog: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
a615b993b00953aef24394e6a3aa274c654c8101
link
/test mixer-e2e-smoketest

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.

/test mixer-e2e-smoketest

/retest

@mandarjog: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
a615b993b00953aef24394e6a3aa274c654c8101
link
/test mixer-e2e-smoketest

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.

/retest

@mandarjog: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
a615b993b00953aef24394e6a3aa274c654c8101
link
/test mixer-e2e-smoketest

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.

/lgtm


Typesetting Markdown

Added link to blog post about a way to typeset Markdown documents. Minor change to punctuation. (Probably a matter of taste; I tend to avoid using full-colons within lists that were introduced with a full-colon.)

Thanks!


date masking is allowing user to enter alphabets

Hi,
Date masking is allowing user to enter alphabets and moreover it is accepting too many characters.
Please help me how can i restrict not to accept alphabets when user enters (developing app for iPads)....Thanks
below is Code and package json details:
Package Json:
"react": "16.3.1",
"react-native": "0.55.2",
"react-native-masked-text": "^1.6.5",
"react-native-material-kit": "^0.5.1",
export default class App extends Component {
render() {
return (

    <View style={{flex:1}}>
    <TextInputMask
			ref={'myDateText'}
    type={'datetime'}
    customTextInput={TextfieldWithFloatingLabel}
			options={{
				format: 'DD-MM-YYYY'
			}}
		/>
    </View>
    <View style={{flex:1,backgroundColor:'green'}}>
    <Text>hi</Text>
    </View>

  </View>

)
}
}

Hi, this can be because this ISSUE.
I will close it and after I resolve that issue it occurs again, I will reopen.
Thanks for reporting


Fixes a bug in SimulraHSB2

The HSI.hsi2rgb() function expects the hue value in the range [0,6) rather than radians. Therefore, the result of atan2() needs to be adjusted to fit within this range.

Ketaro, thank you for bringing this up!
I cannot merge your change, but it looks sensible to me.
Did you check whether other HSI based maps are affected?


Roles, submethods, constructors, destructors.

In this ticket I'd like to gather scattered pieces of one big problem on role composition and submethod handling related to this. Related topics are rakudo/rakudo#2241, rakudo/rakudo#2250, rakudo/rakudo#2677, rakudo/rakudo#2678.
In general, the problem is about:

How submethods in roles are composed into the consuming classes and if they should be composed at all?
How we call submethods.
How constructors/destructors are treated and how to use those defined in roles.

I'd like to propose a set of rules on treating roles and their composition in to classes.
A submethod belongs to where it is defined.
Contrary to methods, submethods from roles must not be copied over to the consuming class. The following:
role R {
    submethod foo { }
}
class C does R {
}
C.new.foo

must be a method not found error. The right call to foo would be C.new.R::foo.
Calling a submethod results in a single call.
In rakudo/rakudo#2677 it is proposed by @jnthn and backed by @TimToady that if a submethod is declared on both class and and role then calling it on a class instance would result in both submethods being called:
role R { submethod foo { say "R" } }
class C does R { submethod foo { say "C" } }
C.new.foo; # R\nC\n

My point of view is that there is no reason to break the existing semantics of one call - one (sub)method. For cases where one needs a batch-call there're already .+ and .* operators. But I'll get back to them in the following section.
Another point is what implicit batch-call must return? How would it handle flattening returns?
Also, what would be the way for a user to call submethods individually when it is necessary? Wouldn't implicit batch-call over-complicate this rather simple task?
Batch-call of submethods must be possible in reverse MRO.
This is quite natural semantics for submethods following how constructors/destructors are being used. But it shouldn't be the only possible way as implicit batch-calling implies. My point of view is that in addition to the mention above .+ and .* we could provide .- which does the same as .+ but in reverse MRO. With the operator it's easier to provide DWIM semantics for a user.
Unfortunately, I cannot find appropriate analog for reverse version of .*.
Attribute flattening
Consider the example:
role R1 { has $.a }
role R2 does R1 { }
role R3 does R1 { }
class C does R2 does R3 { }

This code currently results in Attribute '$!a' conflicts in role composition error. This happens because R1 is flattened into R2 and R3 causing the conflict when both are flattened into C.
What must happen is $.a must only be flattened at C composition time directly from R1.

MRO and mro method
By default mro method doesn't report back roles. But if called with :roles named parameter it will include them into the result. I call this RMRO or Rolified MRO. Possible output can be found in this comment. Method mro_unhidden in turn respects the is hidden  trait and only returns non-hidden classes and roles with :roles.
Constructors/destructors
Basically, they follow common submethod semantics. The interesting point here though is classes inheritance when child consumes same role one of its parents already does. The answer to this is again lies in the mentioned above comment. The sample RMRO there looks like:

C1 R1 R0 C0 R3 R1 R2 R0

Reversed it provides the order of calling constructors/destructors meaning that if R0 and R1 define any they'd be executed twice.
Cross-version compatibility
Roles created with 6.e language version are not to be consumed by 6.d- classes. Though it should in general be possible for a 6.e class to consume 6.d- role.
NOTE Metamodel::ClassHOW already does Metamodel::LanguageRevision role. ParametricRoleHOW is a subject to change yet.

This is a draft I propose for consideration. It may also be extended to cover rakudo/rakudo#2663.

Additional considerations on role composition
Despite roles are expected to flatten down on their consuming class, actual implementation does it more like a tree walk. This results in problems similar to the mentioned above attribute conflict. Besides, it also has its performance cost in cases when roles consume another roles. This results in unnecessary extra copying over of attributes and methods.
I think flattening must only take place at class composition time. The whole process would be akin to creating an alloy: throw components into a pot and melt it down, mixing not stirring.
Technically this would mean that RoleToRoleApplier would do only few things if anything at all (I'll be looking closer at it). Whereas RoleToClassApplier would deal with a flat linear list of roles, free of duplicates. Thus, any conflicts possible at the time of composition would be only real ones. Besides, it might have positive performance impact reducing the number of method/attribute copying.

More on batch-call operators
It was proposed more as a joke but suddenly it might make sense to introduce .?+ and .?- operators. It would make a complete matrix of method call operators:

. for simple calls, fails if no method.
'.?` ignores missing method
.+ batch-calls methods on MRO, fails
.- batch-calls method on MRO in reverse order, fails
.?+ and .?- do the same as .+ and .- but instead of failing they put Failure wrapped around X::Method::NotFound where methods are not found on a type object.

The above set is sufficient. Questions arise though:

.* becomes redundant. Deprecate?

It makes sense executing submethods on all MRO type objects, including roles. What about just methods? Do we also find them on roles and call? But this would break backward compatibility with the current behavior of .+. Besides, methods, contrary to submethods, are not copied over from roles to classes. Thus they are not likely to suffer from double-calls caused by batch-processing.
So, I'd rather proclaim that for classes the above batch-call ops would pick up any of methods or submethods; whereas only submethods will be taken from roles.

Closing with merge of rakudo/rakudo#3199 and perl6/roast#581


Base AST visitor for SQL throws type warning, if sql function argument has the field name

Base AST visitor for SQL throws warning 'Failed to infer type for Identifier(name='CompanyName', namespace=())'
for endswith(CompanyName,'Futterkiste'). This is harmless warning, but needs to be fixed as this is a valid expression as per OData spec

my_odata_query2 = "endswith(CompanyName,'Futterkiste')".strip() lexer = ODataLexer() parser = ODataParser() ast = parser.parse(lexer.tokenize(my_odata_query2)) bqVisitor = AstToSqlVisitor()  print("*******************************************") print(f"SQL query: {bqVisitor.visit(ast)}")

Fair point. I've turned it into a debug log for now, so it's easier to turn off but users can at least know the cause of type-related failures.
The reason we need type inference is to know which functions we'd need to call, e.g. endswith(x, y) is valid for both string and list types and we need to pick the correct implementation based on the types of the arguments.
odata-query cannot infer this type for every Node. The main failure case is for Identifier nodes, for which we'd have to know the entire schema of the database up front.


ModelTesting-functions tutorial fails with error

Describe the bug
The fifth code cell on the ModelTesting-functions fails when run.
To Reproduce
Launch the ModelTesting-functions and execute the cells in the notebook. Cell 5 gives:
results.add_estimates(results2)
results.add_estimates(results3)

pygsti.report.construct_standard_report(
    results, title="Model Test Example Report", verbosity=1
).write_html("../tutorial_files/modeltest_report", auto_open=True, verbosity=1)

Running idle tomography
Computing switchable properties
---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
Input In [5], in <cell line: 4>()
      1 results.add_estimates(results2)
      2 results.add_estimates(results3)
----> 4 pygsti.report.construct_standard_report(
      5     results, title="Model Test Example Report", verbosity=1
      6 ).write_html("../tutorial_files/modeltest_report", auto_open=True, verbosity=1)

File ~/repos/pyGSTi/pygsti/report/factory.py:1263, in construct_standard_report(results, title, confidence_level, comm, ws, advanced_options, verbosity)
   1259         _warnings.warn("Not all data sets are comparable - no comparisions will be made.")
   1261 printer.log("Computing switchable properties")
   1262 switchBd, dataset_labels, est_labels, gauge_opt_labels, Ls, swLs = \
-> 1263     _create_master_switchboard(ws, results, confidence_level,
   1264                                nmthreshold, printer, None,
   1265                                combine_robust, idt_results, embed_figures)
   1267 if len(Ls) > 0 and Ls[0] == 0:
   1268     _warnings.warn(("Setting the first 'max-length' to zero, e.g. using"
   1269                     " [0,1,2,4] instead of [1,2,4], is deprecated and"
   1270                     " may cause 'no data to plot' errors when creating"
   1271                     " this report.  Please remove this leading zero."))

File ~/repos/pyGSTi/pygsti/report/factory.py:330, in _create_master_switchboard(ws, results_dict, confidence_level, nmthreshold, printer, fmt, combine_robust, idt_results_dict, embed_figures)
    327 switchBd.clifford_compilation[d, i] = est.parameters.get("clifford compilation", 'auto')
    328 if switchBd.clifford_compilation[d, i] == 'auto':
    329     switchBd.clifford_compilation[d, i] = find_std_clifford_compilation(
--> 330         est.models['target'], printer)
    332 switchBd.profiler[d, i] = est_modvi.parameters.get('profiler', None)
    333 switchBd.meta_stdout[d, i] = est_modvi.meta.get('stdout', [('LOG', 1, "No standard output recorded")])

KeyError: 'target'

Expected behavior
I would expect a report to be generated and open.
Environment (please complete the following information):

pyGSTi version [<IP_ADDRESS>]
python version [3.10.4]
OS [MacOS 12.3.1]

@coreyostrove , thanks for the follow-up. This isn't critical for me right now, but I think that this example should be removed at the very least. Have the pygsti devs considered running the notebooks in CI? I can't think of another way to keep regressions like this from happening in the future.
I've set up things like this before, so I'm happy to have a discussion about it if you're interested.

Yes we have considered running the Jupyter notebooks in CI, it just hasn't been a high priority. I'll revisit that the next time I have a few extra cycles. :)

A patch for this issue with the ModelTest not getting passed in a target model exists on the branch feature-circuitlistsdesign-reporting included in commit 7c1e01b. Once this branch gets merged in we should be good to go.
P.S. We have automated notebook regression testing up and running on the latest version of develop, so we should be better equipped to catch these problems going forward.

Closed as this is now in 0.9.11 release


fix(spawn-ant): use a valid default ttlSeconds

:tada: This PR is included in version 2.7.0-alpha.6 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


.NET 6: Switch Expression Produces Redundant Code in IL

Description
Switch Expression Function that returns an Action, compiles with a redundant if (1 == 0) { } in IL:

Thought it might've been my use of the Array check, but it still persisted after a rewrite:

Also tried putting it back into its main function, but it doesn't go away:

Configuration

.NET 6 Console App.
Visual Studio Community 2022 17.7.6
ILSpy 2022 v<IP_ADDRESS>56

Regression?
Unknown. First time examining a Switch Expression in IL.
Other information
N.A. Thank you.

Update: Tried rewriting it as a traditional switch. The redundant portion was not generated:

This implies that it is the compilation of a switch expression that's causing the issue.
Also, to cover all bases, I made it 7 cases to compile as a switch statement in IL. The redundant portion was also not generated here:

Hope this helps.

As far as I can tell, the redundant IL is only generated in Debug mode, so I don't think it's important to remove it.


schema["type"] should allow str or unicode, and the error now prints the type

A fix for a frustrating bug I encountered. I'm surprised the unit tests even work.

@MasterEric You broke the tests, needs to be fixed and the linting of the changed code is not good. Both needs to be fixed before i could accept this PR.

Apologies for the delay in fixing your issues.
I corrected the linting on my code changes, and discovered that modifying one of the error messages to be more verbose broke an assertion in one of the unit tests, which I fixed. All unit tests now pass on my machine.

Coverage remained the same at 87.2% when pulling 9443f006e695c5658bbdb5e97d50d0876af22fe3 on MasterEric:unstable into e7fac6a5f5d27a472b206b896f684ee83650b5be on Grokzen:unstable.

Coverage remained the same at 87.2% when pulling 9443f006e695c5658bbdb5e97d50d0876af22fe3 on MasterEric:unstable into e7fac6a5f5d27a472b206b896f684ee83650b5be on Grokzen:unstable.

@Grokzen Looking at the Travis page, it says that 2 checks failed, however there are no logs for the checks that failed. All tests where there are logs show all checks have passed. Can you please fix this issue?

@Grokzen This issue appears to have been closed without comment, can you please clarify?

@MasterEric I accidentally closed all open MR:s when i removed the destination branch. I am currently manually merging all of them including this one.

@MasterEric Manually merged the commits here 24adf3d1a1d74ccc260b0eca1a87e1ea48a3785b..dfb45cb6450e00f80dcfe0f05dad2e1b7cd9e9f0


Update title of DataFrame extension docs

This changes the page title of dataframe-extend.rst from "Subclass DataFrames" to "Extending DataFrames" as the documentation covers topics other than subclassing (e.g. registering extension arrays). Also, I updated "Extend Dispatched Methods" to be a subsection of the "Subclass DataFrames" section. I think this is correct, but am not 100% sure
cc @TomAugspurger

[ ] Tests added / passed
[ ] Passes black dask / flake8 dask

Thanks @jrbourbeau


Can't install on ember 2.7.0

Hi !
I have an issue with this addon when i try to install directly i get:
> ember install ember-collection
DEPRECATION: Overriding init without calling this._super is deprecated. Please call `this._super.init && this._super.init.apply(this, arguments);` addon: `ember-cli-htmlbars`
    at Function.Addon.lookup (/Users/hugoruscitti/proyectos/huayra/huayra-stopmotion/node_modules/ember-cli/lib/models/addon.js:1009:27)
Installed packages for tooling via npm.

but installing alpha.6 seems to work, it's normal to specify this version for new projects? or there are any other workaround?
> ember install<EMAIL_ADDRESS>

@hugoruscitti seems like the first one successfully installed. I'm updating the issue title to make it clearer.


Add User Authentication

User authentication that can create a user for the database with the following input [Name, Email, Password]. But need to add sessions to the authentication.

[x] User can registrate.
[x] Users password is encrypted with bcrypt
[x] User can authenticate.
[ ] Sessions
[ ] User can logout.
[ ] User can't see login page while logged in.
[ ] User can't see registration page while logged in.

Passport might be an idea later on for some part of the code, but for now this works quite fine.

Du commiter stadig kode der allerede er commitede. :D @Aavild

1eb7a31 er en merge og ikke en commit af kode. Heriblandt var der nogle smart merges. Fjernede kun den forkerte smart merge i 20a7c41


💡[FEATURE]:  Create the logo fro ScrapQuest

I want to create the logo for Scrap Quest .
Please assign this me under gssoc i would like to work on this.

@namandude Hey, I thought you would add it onto the website. If not then I will close this issue.

This is what i m adding
that just i shared so u letting know this.

@namandude I am closing this issue. You can raise a PR for footer issue.


I cant complete the challenge, even after pass the github and heroku urls

Challenge Timestamp Microservice has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:50.0) Gecko/20100101 Firefox/50.0.
Please describe how to reproduce this issue, and include links to screenshots if possible.

Hi @zr0n! Thank you for reporting this issue.
I'm sorry to hear this isn't working for you. Can you please give us more details?
If you reload the page and try again, does it work? How about a different browser?
Are you seeing any errors? Can you describe what happens after you click the submit button, or send a screencapture of what is going on?

I've tried in chrome and firefox. Even after logout and relogin it is not working :C
If someone of the main staff read this, my user is zr0n (as the github)

@zr0n can you paste your urls here?

@zr0n Are you still having issues submitting this challenge?

@dhcodes  Urls:
http://timestampmservice.herokuapp.com/January 1, 2016
https://github.com/zr0n/timestampservice
@Bouncey  Yes, still having issues

@zr0n
So, just to confirm you aren't able to submit the above URLs to your repo and the app on the challenge page?
Can you take some screenshots of whats happening?
Can you also take screenshots of the console (Developer Tools) for your browser and let us know if there are any errors?

@zr0n do you have any updates? We could use your help in reproducing this.


i18n(zh-cn): Add guides/internationalization.mdx in Simplified Chinese language

Description (required)
I have completed the translation of the 'Internationalization' section from English to Simplified Chinese. Please review.

Related issues & labels (optional)

Closes #
Suggested label: i18n

感谢你的翻译，非常遗憾的告诉你，目前此页面被标记为 i18nReady: false，表示本页还没有准备好翻译，你可以先保留你的分支，等准备好了再提交 PR。

Thanks for your reply, I'll preserve my branch until the translation is needed,


Continued?

Hi community!
Just a short question - is this programm sill in progess, or is it abandoned?
Greetings form germany - xoxo!

Hi. First readme lines:
USE [OpenRGB](https://gitlab.com/CalcProgrammer1/OpenRGB) INSTEAD
This project is not actively maintained. No support for additional boards will be added.


Change version for helm charts in README

Update versions for charts in README files.

We should automatically bump this version as well as part of #3342
Also there is a tool to automatically generate docs for helm charts.

We would need to generate the doc if we want to make helm as official installation method

This is fine to fix the current documentation. I created https://github.com/kserve/kserve/issues/3356 to track the docs generation separately.
/lgtm

/approve


Lens blocks the Internet on my device

Describe the bug
After a certain amount of time Lens has been running, the Internet is blocked. This problem appears quite rarely and usually after intensive interaction with several K8S instances.
When this problem occurs, I cannot connect to any resource through the browser, all messengers stop working, reconnecting to other Wifi access points does not solve the problem. Sometimes I even forget that I need to turn off Lens for this problem to go away.
I can’t say which version of Lens this problem started with, but I’ve had it for at least 1 year, but I was hoping it would be fixed without my issue.
To Reproduce
Steps to reproduce the behavior:

Open Lens
Working with multiple k8s instances
Waiting for a problem with network

Environment:

Lens Version: 2024.1.300751-latest
OS: MacOS Ventura 13.6.2

Additional context
It seems to me that this problem is caused by the Lens proxy process. It freezes frequently.

Thanks for reporting this, we have just responded to a duplicate of this issue here: https://github.com/lensapp/lens/issues/6063


security/acme-client: add support for Google Public CA

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[X] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[X] When the request is meant for an existing plugin, I've added its name to the title.

Is your feature request related to a problem? Please describe.
In release 3.0.3 acme.sh added support for Google's Certificate Authority Service.
Describe the solution you'd like
The plugin should support all (useful) CAs that are supported by acme.sh.
Describe alternatives you've considered
None.
Additional context
https://github.com/acmesh-official/acme.sh/wiki/Google-Public-CA
https://github.com/acmesh-official/acme.sh/releases/tag/3.0.3

superseded by https://github.com/opnsense/plugins/pull/3029


Release 0.5.7

Fixes CRITICAL bug that prevented questionnaire data to be submitted (arrays could not properly be processed) - 5ecffe6
Updates dependencies

Which version was this introduced in?

As far as I can tell, it was introduced in version 0.5.0. It's immediately apparent when trying to upload questionnaire data, since no such data will be accepted (500 error).

It was introduces quite long ago: 13 Jul 2020. Older versions will have to be deprecated.


feat: add vm and connect to it

Create a VM with preview environement and connect to it with SSH

Will close it because it failed


Fix code scanning alert - Incomplete string escaping or encoding

Tracking issue for:

[x] https://github.com/tugcantopaloglu/juice-shop/security/code-scanning/60

Issue fixed on df7240c345b0797b2edd391eaad12aea42b99fcc


Does Doc comment support all features of static styleguide

Hello
I'm wondering if Doc comment support all features of static styleguide ? For example, can we use tabs and code snippet with Doc comment ?
Best regards

Whatever https://github.com/reactjs/react-docgen supports, we support since it's what we use internally.

Thank for this precision !
Great job


Added PATCH, PUT, DELETE

Thanks dude

<EMAIL_ADDRESS>released on https://www.npmjs.com/package/onlinelabs

:+1:


REPL output speed is much slower that in other products

Try ls("package:base")

mine returns the responses (1210 lines) in < 1 sec...  what are you seeing?

Maybe a Debug/Release thing? Or slows down when running RTVS under debugger?

It is less than a second, but RS is instant. I'll check on release.

In Debug, it is very slow, and lines show one by one.
In Release, there is a noticeable 1-2 sec delay where nothing is output, then all the output comes in at once.

OK.

I suspect it might have something to do with lines being sent one by one, so each line is basically its own packet. It might be possible to improve it if we start buffering writes.


API for capturing & drawing videos

As a developer building group chat application for iOS/Android upon phonertc, API to draw container's video content onto the html canvas would be great, so that we can render videos in an arbitrary layout/design in javascript.

Hi, is the recording feature already implemented ?


migration to add uniqueness constraint on cdn domain name

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/817/
Test PASSed.


[gui error report] Error: Initializing daemon...

👉 Please describe what you were doing when this error happened.
Specifications

OS: win32 10.0.19043
IPFS Desktop Version: 0.24.0
Electron Version: 19.0.9
Chrome Version: 102.0.5005.167

Error
Error: Initializing daemon...
Kubo version: 0.15.0
Repo version: 12
System version: amd64/windows
Golang version: go1.18.5

Error: remove C:\Users\liuji\.ipfs\datastore\002694.log: The process cannot access the file because it is being used by another process.

    at errorTemplate (C:\Program Files\IPFS Desktop\resources\app.asar\src\daemon\migration-prompt.js:78:49)
    at Object.loadWindow (C:\Program Files\IPFS Desktop\resources\app.asar\src\daemon\migration-prompt.js:129:26)
    at startIpfsWithLogs (C:\Program Files\IPFS Desktop\resources\app.asar\src\daemon\daemon.js:191:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async startDaemon (C:\Program Files\IPFS Desktop\resources\app.asar\src\daemon\daemon.js:216:27)
    at async startIpfs (C:\Program Files\IPFS Desktop\resources\app.asar\src\daemon\index.js:48:17)
    at async setupDaemon (C:\Program Files\IPFS Desktop\resources\app.asar\src\daemon\index.js:116:3)
    at async run (C:\Program Files\IPFS Desktop\resources\app.asar\src\index.js:85:5)

Thanks for submitting this issue @jiajia4451
It looks like there is a long running process that is writing to that particular log file, there could be multiple reasons for that, I'm moving this issue to the kubo team to better answer this.


Trigger Controllers Error Handling

Is this a BUG REPORT or FEATURE REQUEST?:
Feature request
It would be nice to be able to hook into the error handling instead of simply aborting.
Currently we log and dequque the message:
https://github.com/kubeless/kubeless/blob/master/pkg/controller/kinesis_trigger_controller.go#L168
https://github.com/kubeless/kubeless/blob/master/pkg/controller/kafka_trigger_controller.go#L168
https://github.com/kubeless/kubeless/blob/master/pkg/controller/cronjob_trigger_controller.go#L177
But there are use cases where allowing the failed messages to be used is valueable -- e..g publishing messages that don't pass into an error or retry queue that can be reprocessed later, or even triaged by a human.

But there are use cases where allowing the failed messages to be used is valueable

Could you please elobrate this?
Basically in processNextItem controller is acting on the events recived from the Kubernetes API server. If there is an error, there is a retry before eventially giving up.

@murali-reddy Conceptually they are reprocessing queues / dead letter queues
e.g. https://eng.uber.com/reliable-reprocessing/

I am not sure how much flexibility we can give there but it's true that we could offer at least a "at least once" strategy for sending messages instead of the current "at most once". Also note that the retries you point in the code are related to retries when creating a message consumer (goroutine that listen for messages in a queue). Function calls are not retried:
https://github.com/kubeless/kubeless/blob/master/pkg/event-consumers/kafka/kafka-consumer.go#L102

Also note that the retries you point in the code are related to retries when creating a message consumer (goroutine that listen for messages in a queue). Function calls are not retried:

ahh. Thanks for the clarification. Yup I totally missed that :D
I guess I can implement it in user-land as a shared abstraction.


Adds full path to default config for linux

This adds the full path to the default config directory for linux in which config.toml is located by default.

I approved the changes via the web UI. Feel free to merge.
Let me know if you need anything else from my end. all best,


Re-enable LUIS orchestration tests

Once UK South is updated to support the latest version of LUIS, re-enable tests attributed to this tracking issue.

Deprecated


[QA-1882] rework run-workflow ui test

Rework flaky function launchWorkflowAndWaitForSuccess():

Handles unexpected scenarios.
pRetry code fails to handle Failed status.

Conflicts are due to #3065 moving the launchWorkflowAndWaitForSuccess function to integration-tests/utils/workflow-utils.js.

Conflicts are due to #3065 moving the launchWorkflowAndWaitForSuccess function to integration-tests/utils/workflow-utils.js.

@nawatts Hi Nick, rebased just now. PTAL.


ChangeDetection is not reenabled on active tab when returning to Tabs page, and ionViewDidEnter is also not fired on active tab page.

Short description of the problem:
If you push a page in tabs.ts, and later return - the changedetection in the active tab page is not reenabled, and lifecycle events such as ionViewDidEnter is also not fired.
Steps to reproduce:
Run my plunker link, open console to see log events from ionViewDidEnter.
Push the tab called "Push" and close it again and notice the counter is no longer updating.
Which Ionic Version? 1.x or 2.x
2.beta.10
Plunker that shows an example of your issue
http://plnkr.co/edit/4gOLUPs3Doyf3Twz0qjA?p=preview

Note for self and team: http://plnkr.co/edit/Yrvj6SL9gQTsRb2TIHqn?p=preview As this plunker shows, pushing a page in a tabs app is working fine, so it seems to be isolated to the use case of pushing a page by clicking on a tab.

I just wanted to add that if you change your goToPage1 method in your plunker to use rootNav, the lifecycle events will not fire when returning, but changedetection does seem to work in this case.
goToPage1() {
  this.nav.rootNav.push(Page1);
}

Hello @andersfa87 Is this still an issue with beta.11? Thanks!

Hey @jgw96 Yeah, both problems are still there.
Any workaround I can use to manually start tracking changes again? Stuff like pull to refresh is quite broken when change detection is not running :)

@andersfa87 Sorry for the delay on this. Is their any way that you could push your page from a button instead of from clicking on a tab? Their is a separate issue open about doing other events besides navigating to a tab by clicking on a tab, but at this time it is recommended to only use tabs to navigate to a different tab (as they are in our tabs starter) and not to push a page or do other actions.

Hey @jgw96
I work with andersfa87, so I can give you some more info about the problem we experience.
The problem is not only when you click on the tabs.
Every tab has an active view/component. If you on this active view (let us call it view1) use a normal button and push another view (view2) to the rootNav (thereby replacing tabs with view2), then ionViewWillEnter and ionViewDidEnter are not fired on view1 when returning (click back).
ionViewWillEnter and ionViewDidEnter are only fired on the tabs page/component.
From a technical aspect it makes sense that when returning to the tab, the events are fired in tabs. But the user actually returns to the active view in the tab and therefore we need the possibility to update the content of the view.

up! need too, no sense to not fired ionViewWillEnter when change tabs active page

Hey @jgw96, was this fixed? I'm still having the exact issue described by @rasmusl7 above.

Same, getting this issue here trying to get animations to activate based on tab changes.

Still getting this. I don't think it's fixed.

A temporary fix that resolved it for our team was to call this.appRef.tick() in ionViewWillLeave.
The only issue we're facing is that the change detection is not being triggered if you go between tabs which you've already visited before. Any new page or tab is fine. All ionView... callbacks are being triggered properly, just not the model changes.

Adding this.appRef.tick() worked for us, thanks @emiljohansson

in src/components/tabs/tab.ts there is no emit for view events on selected in L364, maybe, we can submit a patch for that and raise view event ?
  /**
   * @hidden
   */
  setSelected(isSelected: boolean) {
    this.isSelected = isSelected;

    this.setElementClass('show-tab', isSelected);
    this.setElementAttribute('aria-hidden', (!isSelected).toString());

    if (isSelected) {
      // this is the selected tab, detect changes
      this._cd.reattach();
      this.viewWillEnter.emit();
    } else {
      // this tab is not selected, do not detect changes
      this._cd.detach();
      this.viewWillLeave.emit();
    }
  }

Same issue here

Got the issue too, when pushing a view to rootNav from tabs, and getting back to it with back button, only the tab component hooks are fired, not the view selected, a little bit annoying I need to refresh datas (in active view in tabs component) and I don't want to bypass it with custom events...


Create a process for detecting network connection - middlware/backend

In order for the student to be able to do their work asynchronously and only sync with the DAP when connected to the internet, we will need to know when they are connected to the internet and detect changes in their connection so we can trigger an action that will sync their data with the database when they have network connection.
Your job for this ticket will be to create the class, functions, services, and processes that determine network connectivity.
For example, this might utilize an open source library we can integrate with Android, or we can create our own. We will need the following:

Whether the app is currently connected to internet or not
A process that updates the connection status as needed
Whether or not the current status has changed from a previous one (ie, whether we have newly acquired or lost connection)

This will likely be helpful for upcoming tickets: https://developer.android.com/guide/components/services
We should bind this network checking service to certain events, such as when we want to write to the DAP


[Feat] Add task caching mechanism to improve the running speed of repetitive tasks

Brief Summary of The Change
close: #49
Pull Request checklist
I confirm that the following checklist has been completed.

[x] Add/Change test cases for the changes.
[x] Add/Change the related documentation, should also change docs/source/config.rst when you change file default_config.yaml.
[ ] (Optional) Add your change to UPDATING.md when it is an incompatible change.

Do not merge until https://github.com/apache/dolphinscheduler/pull/13194 is merged

Codecov Report

Merging #50 (7ff8e0b) into main (f82e07d) will increase coverage by 0.07%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##             main      #50      +/-   ##
==========================================
+ Coverage   90.71%   90.79%   +0.07%
==========================================
  Files          59       60       +1
  Lines        2090     2119      +29
==========================================
+ Hits         1896     1924      +28
- Misses        194      195       +1

Flag
Coverage Δ

unittests
90.79% <88.88%> (+0.07%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/pydolphinscheduler/core/task.py
93.25% <83.33%> (-0.35%)
:arrow_down:

src/pydolphinscheduler/constants.py
100.00% <100.00%> (ø)

src/pydolphinscheduler/utils/versions.py
100.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

dependence on dolphinscheduler 3.2.0

Do not merge until apache/dolphinscheduler#13194 is merged

may releasing in version apache/dolphinscheduler:3.2.0


protection main view - update view to show open positions + claim in the same view

remove the claim tab on the main table and include that section below the main open positions table.
https://www.figma.com/file/3yED7o7TgoLwoEjV2EkRLL/Bancor-V2?node-id=736%3A2295
https://www.figma.com/file/3yED7o7TgoLwoEjV2EkRLL/Bancor-V2?node-id=736%3A1604
https://www.figma.com/file/3yED7o7TgoLwoEjV2EkRLL/Bancor-V2?node-id=736%3A2024
https://www.figma.com/file/3yED7o7TgoLwoEjV2EkRLL/Bancor-V2?node-id=736%3A1238

this was done already.
duplicate.


Copying tags for volumes when DeleteOnTermination is false

While using the script for an on-demand to spot conversion I noticed the tags on original volume doesn't get copied to the new volume created.
In my case the DeleteOnTermination was originally false so there was an extra diff for volume,

Impact:
I generally keep a backup schedule based on tags. Since the new volume won't have tag there's a requirement to manually add it, else the backup schedule will use the old volume.

No biggie. But wanted to know your view

v0.11.0 released!


Default axios' maxRedirects to 0 (fixes #89)

Please do not merge yet; discussing a new finding at #89.


cacheAsBitmap / generateTexture offset sprite positions

This is an underlying PIXI issue i believe. However its good to notify in the docs that those methods cacheAsBitmap / generateTexture are offsetting sprite positions. The reason is that transparent pixels around graphics are cut automatically, and therefore the sprite positions get shifted. This happens since 2.3 up to 2.8.1
So the docs should give a warning i believe.

I've noticed this with graphics.generateTexture().
It happens with group.cacheAsBitmap = true also?

I havent tried with a group but it happens with sprites that have children.
I have seen bug reports of this happening with groups.
https://github.com/photonstorm/phaser/issues/1925


openning index.php slowly

In local network, when I try to open the page of index.php using firefox ,progress is very slow,and developemnt tools show "mutating the [[Prototype]] of an object will cause your code to run very slowly; instead create the object with the correct initial [[Prototype]] value using Object.create"
libs.js:13:21273

Are you running the latest version? I have 1.14.0 and the page is not slow, plus I do not see the warning you are referring to


Unable to use tag_names in site24x7_website_monitor

Hi,
We are trying to create a monitor and apply tags by specifying tag_names, but it's not working.
Only the tag_ids approach is working.
My tags are using the following values :
# result from the API call https://www.site24x7.com/api/tags
    {
      "tag_value": "nginx-prod",
      "tag_name": "service",
      "tag_type": 1,
      "tag_color": "#E55445",
      "tag_id": "29977100xxxxxxxxxx"
    },
    {
      "tag_value": "nginx-dev",
      "tag_name": "service",
      "tag_type": 1,
      "tag_color": "#B7DA9E",
      "tag_id": "29977100xxxxxxxxxx"
    },
    {
      "tag_value": "nginx-stage",
      "tag_name": "service",
      "tag_type": 1,
      "tag_color": "#FCEA8B",
      "tag_id": "29977100xxxxxxxxxx"
    },

So I'm trying to use the following syntax in Terraform module, but it's not working
  tag_names = [
    "service:nginx-prod"
  ]

What is the expected syntax ?
Thanks
Thomas

It doesn't look like the tag_names are even used in website monitoring module: https://github.com/site24x7/terraform-provider-site24x7/blob/b1d27a459bdbb282f16c55e6274da6c5601b5946/site24x7/monitors/website.go#L542-L545, only tag_ids.

Unfortunately one can't use tag ids either as id is not exposed for the tag resource. Here's an attempt to address this: https://github.com/site24x7/terraform-provider-site24x7/pull/98

Hi Thomas,
Please try giving the value present in the "tag_name"(in the API response) eg:
tag_names = [
"service"
]
Please bear with us till we support data sources for fetching tag_ids.
Thanks & Regards,
Jim

It doesn't look like the tag_names are even used in website monitoring module:
https://github.com/site24x7/terraform-provider-site24x7/blob/b1d27a459bdbb282f16c55e6274da6c5601b5946/site24x7/monitors/website.go#L542-L545
, only tag_ids.

Please refer https://github.com/site24x7/terraform-provider-site24x7/blob/b1d27a459bdbb282f16c55e6274da6c5601b5946/site24x7/monitors/website.go#L619-L624

@jim-billy my apologies, haven't noticed where the tags are applied.
Unfortunately the mentioned block doesn't take into account tag values. Having the following tags defined service:foo, service:bar and service:quz and using the suggested TF block when for the monitor definition:
tag_names = [
"service"
]

Would result in a monitor that would be tagged with 3 tags listed above.
@jim-billy Could you please confirm that it's the case? And if so, is it expected behavour? What I'm trying to understand what is the primary use-case of the tags in the site24x7 definition of it.

@velimir Our apologies for not considering tag values. Will fix the issue and update here.

@Th0masL @velimir
To fetch the tags, please use the site24x7_tag data source available in our latest version 1.0.24 - https://registry.terraform.io/providers/site24x7/site24x7/latest/docs/data-sources/tag


feat(apollo-compiler): Add description getters

Exposes the description fields

@aschaeffer: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/

Thank you so much for this @aschaeffer! I'll merge this in if you can sign the CLA agreement - you only have to do it once on your first PR to any of the repos in apollographl org.

Done :smiley: