[RFC] Qiskit Pulse: Relative timing for Schedules

Add an RFC proposing an implementation for relatively-timed instructions in pulse Schedules within Qiskit Pulse

 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.

 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.

I realized we are using "relative time/timing" in the two meanings: "timing by channels (by-channels)" and "no t=0", i.e. "absolute time/timing" means "timing across channels (across-channels)" and "common t=0".
For example,
`Instruction`s will not be played at an explicit time; rather, they will begin immediately after the last is done.

is talking about "timing by channels", while
explicit timing of instructions will become confusing with the introduction of control-flow to quantum programs as it would become difficult to program experiments containing loops that are not unrollable.

is talking about "no t=0".
Regarding "by-/across-channels",
I think we have two options:

Build and save with "time counting by channels" (as of the current RFC)
Build with "across-channels" and save with "by-channels"

I guess we are heading to "by-channels" because real devices do not support communication among channels. That makes "single time counting across channels" too much for real device.
So I don't disagree with heading "timing by channels" for Schedule (as a data class to be passed to backend) if communication among channels would never be supported, but I think we need to be careful when thinking about schedule builder (as a class for users to build pulse schedule). At the time building schedules, we often want to do insert(time, instruction). So we may need to have intermediate (mutable) class to build schedule supporting insert(time, instruction).
IMO, currently Schedule class is doing two things, building and storing, as like QuantumCircuit. While QuantumCircuit perform well because it's mutable, it is difficult for Schedule to perform well because it's immutable. I thought that's why we're going to introduce "builder class" for Schedule as like "StringBuilder" and "String" class in Java. Am I misunderstanding schedule builder?
Regarding "no/common t=0" considering control-flow,
we can think of another option (I don't say better option).
What if we introduce a kind of "(control-flow) block" and "no t=0" among blocks but having "common t=0" within a block?

@itoko I don't understand your comment; I don't understand what you mean by "cross channels"

I'm not totally clear on this justification. Can you clarify the complexity of union with absolute and relative datastructures? I don't think it's different with the currently proposed structure, but I could be wrong.

It also encourages inefficient operations such as taking the union of two schedules. At first glance this operation should be efficient since all one needs to do is specifying the starting time of the instructions, but this requires verification that no instructions overlap in time in the two schedules resulting in expensive checks. The requirement for such a check to merge two schedules is obvious within the relatively schedules model, as one must look to make sure a delay exists in the first schedule for all instructions of the second schedule to merge.

Can we list the types of operations that we do in schedules (append, insert, delete/replace, union, find channel idle times) and the complexity of those in a relative vs. absolute data structure? I think for the basic blocks we should choose what makes compiler manipulation faster. I agree they can be converted to each other, but their manipulation is not the same.

On Apr 2, 2020, at 5:22 PM, Lauren Capelluto<EMAIL_ADDRESS>wrote:
@lcapelluto commented on this pull request.
In 0000-qiskit-pulse-schedule-relative-timing.md:

+{

DriveChannel(0):

   ('instructions'=[Delay(duration=10, ...),

                    Play(duration=100, ...),

                    Play(duration=20, ...)],

    'duration'=130),

DriveChannel(1):

   ('instructions'=[Delay(duration=110, ...),

                    Play(duration=20, ...)],

    'duration'=130),

...
+}
I think Thomas' new RFC touches on this

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

@ajavadia I agree. I'll do that 👍

Ok so it based on this, append and shift are faster in relative, insert faster in absolute?
If I compare this with #18 it seems the findings are different in the case of insert. Can we come to an agreement on this @lcapelluto @itoko? Because I think the timed circuit and pulse case should be pretty identical in terms of datastructure and manipulation runtime.
In any case I just wanted to make sure we are basing this on some concrete runtime analysis, which will also help in the implementation. I think it is clear that relative wins in terms of the most common operation which is append.


MST/MTC/MSX colour breaks

After sending long messages, the colour breaks and reverts to default.

I would also like this to be fixed, because I wasn't able to come up with a fix myself.

There is no easy solution to this problem. I'm going to have to write a full LFS String parser so that it can properly split them over multiple lines.

I'm looking at finally fixing this. I have an idea in mind that makes use of strrchr

Issues are now resolved as of <IP_ADDRESS>

@LFSPovo Your issue is still pending a good way to do that. I suppose I could check if the string starts with / take upto the first space and prepend that to every message. I will look at best options when I have free time on Wednesday.


Integrate some kind of functionality to view and edit secrets in place as strings

What would you like to be added:
When working with secrets using kubectl there are scenarios where you would like to view and edit the value of the secret in place as a string. Kubectl already includes a way to create secrets conveniently from files and other sources. It would be really convenient if it also included subcommands to view and edit secrets in place without having to do multiple extra steps with base64 encoding and decoding.
I don't have a very specific proposal for the design, but I think https://github.com/rajatjindal/kubectl-modify-secret is a fairly good implementation of this idea. If something similar could be integrated into kubectl then that would be useful.
Why is this needed:
Currently viewing and especially editing secrets is fairly inconvenient and error-prone, requiring piping together multiple commands that may or may not include subtle errors that break on things such as newlines in the secret value. This is especially problematic if you need to edit one part of the secret while keeping others intact (for example if the secret value contains JSON or another structured format). I can't say for sure how common such scenarios are in general, but it comes up somewhat often with the work that I do.
Having a well supported way to do this using kubectl itself without external tools would be preferable to external plugins given the sensitive nature of secrets. I don't think the kubectl-modify-secret plugin I linked beforehand is malicious, but I still feel a little uncomfortable giving this responsibility to a fairly unknown third-party plugin.

/close
I am closing this because there is a plugin that solves this issue. We would potentially consider a PR for this if someone were to create one.

@mpuckett159 would something like kubectl edit secrets/foo --decode be acceptable as a MR?


Add check to only create native enum once

This will fix #3657

Can you also add test for this? I think we had integration tests for enums already, should be easy to expand.

@mogzol Do you need any help with this?

Oh sorry I forgot about this actually. I don't really have much free time right now if someone else want's to take this over, or wait until closer to the end of the year and I should have a chance to fix it.

Updated with some tests and a slightly different approach, now the result of the type call in getColumnType in the column compiler is cached so it's only run once (per column), which should fix this issue for any types that have side effects, instead of just fixing the specific postgres enu type.

Updated with some tests and a slightly different approach, now the result of the type call in getColumnType in the column compiler is cached so it's only run once (per column), which should fix this issue for any types that have side effects, instead of just fixing the specific postgres enu type.

@mogzol Could you also please add an integration test to test/integration2?

@mogzol Could you also please add an integration test to test/integration2?

@kibertoad Yes, added some integration tests. I also found that altering native enum columns still failed because postgres doesn't allow casting directly between enum types, they must be converted to text first, so I added a check in the postgres column compiler for that.

@kibertoad Yes, added some integration tests. I also found that altering native enum columns still failed because postgres doesn't allow casting directly between enum types, they must be converted to text first, so I added a check in the postgres column compiler for that.

Thank you!

Thank you!


Doctrine version

Your bundle looks great, but i can't use it as the doctrine/orm version requirement is <2.4-dev ; any reason to that?

Long time not used it. Merged, thx.


Credentials not escaping command arguments.

If any of the arguments passed on the command line to the groovy script has a apostrophe in the content the command will fail.
I ran into this with a password with an apostrophe. I think issue may extend to other functionality.

The new jenkins_user type [also] handles this correctly. https://github.com/jenkinsci/puppet-jenkins/blob/master/NATIVE_TYPES_AND_PROVIDERS.md#jenkins_user

Oh, that's way better.

It would be great if you could give it a try and see if it works for you.  This is a legit bug and we do need to either fix jenkins::user or turn it into a wrapper around the native type.

I'm using the jenkins_credentials type.
I'm running into a load error for the puppet_x requires.  I'm running puppet 3.8.x  on my agent and hosts.  There's a workaround I've seen used in another plugin, that plugin references # JJM WORK_AROUND #14073 and #7788
I'm essentially having to do this:
begin
  require 'puppet_x/jenkins/type/cli'
rescue LoadError => detail
  require 'pathname' # JJM WORK_AROUND #14073 and #7788
  require Pathname.new(__FILE__).dirname + "../../" + "puppet_x/jenkins/type/cli"
end

I'm not fully tested yet, once I do i'll submit a PR.

This is the upstream bug https://tickets.puppetlabs.com/browse/SERVER-973, I believe I tested the workaround with both puppetserver 1.x and 2.x.  I don't use puppetserver in production at this point so it is possible the situation has changed.

Yeah, i read that.  Not very helpful.  I got the all the imports happy, however i'm running into
Error: Could not find a suitable provider for jenkins_credentials

That likely means the cli jar isn't present or your missing the retries gem.  You could try manually installing the retries gem in the agent's env.

Was the retires gem.

All good now.

I'd like to leave this open since this issue is still present in jenkins::user

We are running in to the same problem. I can confirm the workarount "# JJM WORK_AROUND #14073 and #7788" works. I can also confirm that I needed to install the "retries" gem.
Questions:

Why is the retries gem not installed by Jenkins Module as part of enabeling the provider ?
How is this incompatibility between jenkins module and puppetserver addressed ? (Jenkins module implementing the workaround ? Moving your code to lib ? )

Regards


PC console paper status export file name should show 'filtered'

it looks like the exported csv file name does not have (Filtered) when the paper status tab data is filtered
now the exported file names are the name no matter the paper status collection is filtered or not

looks like this is caused by typo of variable name


Fix sentence spacing

Probably the most pedantic PR you'll ever see, but two spaces after periods is both not recommended in most current styleguides and not displayed in HTML content anyways.

Maybe we're not keeping you busy enough . . .

Maybe we're not keeping you busy enough . . .


Collection Management - Add centroid type in configuration

In gitlab by @sylvaingaudan on Apr 28, 2017, 13:36

In gitlab by @sylvaingaudan on Apr 28, 2017, 13:36
changed title from Collection Management - Add ce{-e-}ntroid type in configuration to Collection Management - Add centroid type in configuration

In gitlab by @sylvaingaudan on Apr 28, 2017, 13:36
added ~1918909 label

In gitlab by @sfalquier on May 9, 2017, 09:13
added ~1918367 and removed ~1918909 labels

In gitlab by @sylvaingaudan on Apr 28, 2017, 13:46
changed milestone to %4

In gitlab by @sfalquier on May 9, 2017, 14:11
added ~1918368 and removed ~1918367 labels

In gitlab by @sfalquier on May 9, 2017, 14:36
added ~1918367 and removed ~1918368 labels

In gitlab by @sfalquier on May 9, 2017, 14:12
assigned to @sfalquier

In gitlab by @sfalquier on May 10, 2017, 13:47
added ~1918368 and removed ~1918367 labels

In gitlab by @sfalquier on May 10, 2017, 16:45
mentioned in merge request !46

In gitlab by @sfalquier on May 11, 2017, 17:32
closed

In gitlab by @sfalquier on May 11, 2017, 17:32
removed ~1918368 label


Add supported OS'es to README

Please add what OS's are supported to your README.
Windows?
Seem like an awesome project! :)

added (one year later)


Not working for russian language 'ru'

From the README: "It comes with a limited set of words and phrases in Dutch, English, and Finnish.".
Would you like to contribute Russian labels?


Trouble defining a Log Directory

Hi Brenden,
I am having trouble setting a log directory for it to save the results to.  I’ve set it in my config.json Similarly to the config_common.json file you have in your repository.  I have a directory named ./log but for some reason it keeps giving me the same error WARNING: logdir not provided results will not be saved to file.  Could you specify exactly where the logdir needs to be set so I can check to see where the issue is?  For reference I am running DSO from a python script (I.e. similarly to the scipy interface).

Hi @Sean-Reilly, if you are running with the Python interface, you are likely creating a DeepSymbolicOptimizer object, correct? You instantiate that object by passing it a path to a config file, e.g.
model = DeepSymbolicOptimizer("path/to/my_config.json")

Inside your config JSON file, in the "experiment" top-level key, you specify the path to your log directory via the "logdir" parameter, e.g.:
{
    "experiment" : {
        "logdir" : "/path/to/log/directory"
    }
}

It should then save to file when running model.train().
Relative paths are supported, but I've found absolute paths to be more stable so it doesn't depend on where you run your Python script from.
Another note is that the default value for "logdir" is "./log", so it could be that you overwrote the parameter at some point? If the above doesn't help, if you could write a minimum working example that would be helpful. Thanks!

I was using the DeepSymbolicRegressor function, which I’ve discovered seems to have it’s log output disabled in the __init__ file.  I’ll use the DeepSymbolicOptimizer from now on to avoid this issue.


SPFX WebPart exposed as teams app not visible in New MS Teams version

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Developer environment
None
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[ ] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[X] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

SPFx version 1.16.1
MS Teams manifest version schema v1.16
MS Teams Windows Desktop version 23231.412.2343.5284
App is loaded via context "personalTab" and https://{teamSiteDomain}/_layouts/15/TeamsLogon.aspx?SPFX=true&dest=/_layouts/15/teamshostedapp.aspx%3Fteams%26personal%26componentId=....id...%26forceLocale={locale}%26theme={theme}"

Describe the bug / error
The webpart is loaded without any issues in the normale Teams version, but not in the "New" version. The loading indicator is shown for 1 second and after that a blank page.
Current manifest.json validdomains:
"validDomains": [
"{teamsitedomain}",
"cdn.XXXXXX.com", (our own cdn)
".login.microsoftonline.com",
"statica.akamai.odsp.cdn.office.net",
".sharepoint.com",
"*.office.com",
"spoprod-a.akamaihd.net",
"resourceseng.blob.core.windows.net"
],
Steps to reproduce

Expected behavior
Load the webpart from the sharepoint hosted webpage.

Thank you for filing a report. As this is definitely a potential bug, I've filed a ticket for the Product Group to take a further look at this.

Can you provide specific repro steps and/or a link to a repro repository?

Currently not

Does this occur with a web part generated with yeoman and no other changes?
Additionally, do you have a custom Teams manifest? And if so, what is listed in webApplicationInfo?

{
  "$schema": "https://developer.microsoft.com/en-us/json-schemas/teams/v1.16/MicrosoftTeams.schema.json",
  "manifestVersion": "1.16",
  "packageName": "XX",
  "id": "XXX",
  "version": "2023.2.1",
  "developer": {
    "name": "XXX",
    "websiteUrl": "XXX",
    "privacyUrl": "XXXX",
    "termsOfUseUrl": "XXXX"
  },
  "name": {
    "short": "XXXX",
    "full": "XXXXX"
  },
  "description": {
    "short": "XXX",
    "full": "XXXX"
  },
  "icons": {
    "outline": "65a87007-ac17-41b6-ace1-b1755acd42f3_outline.png",
    "color": "65a87007-ac17-41b6-ace1-b1755acd42f3_color.png"
  },
  "accentColor": "#004578",
  "showLoadingIndicator": false,
  "isFullScreen": true,
  "staticTabs": [
    {
      "entityId": "XXX",
      "name": "XXX",
      "contentUrl": "https://{teamSiteDomain}/_layouts/15/TeamsLogon.aspx?SPFX=true&dest=/_layouts/15/teamshostedapp.aspx%3Fteams%26personal%26componentId=XXXXXX%26forceLocale={locale}%26theme={theme}",
      "scopes": [
        "personal",
        "team"
      ],
      "context": [
        "personalTab",
        "channelTab"
      ]
    }
  ],
  "configurableTabs": [
    {
      "configurationUrl": "https://{teamSiteDomain}/_layouts/15/TeamsLogon.aspx?SPFX=true&dest=/_layouts/15/teamshostedapp.aspx%3FopenPropertyPane=true%26teams%26componentId=XXXXX%26forceLocale={locale}%26theme={theme}",
      "canUpdateConfiguration": true,
      "scopes": [
        "team"
      ],
      "context": [
        "channelTab",
        "personalTab"
      ],
      "supportedSharePointHosts": [
        "sharePointFullPage",
        "sharePointWebPart"
      ]
    }
  ],
  "defaultInstallScope": "team",
  "validDomains": [
    "{teamsitedomain}",
    "cdn.XXXXX.com",
    "*.login.microsoftonline.com",
    "spoppe-a.akamaihd.net",
    "spoprod-a.akamaihd.net",
    "resourceseng.blob.core.windows.net",
    "msft.spoppe.com"
  ],
  "webApplicationInfo": {
    "resource": "https://{teamSiteDomain}",
    "id": "00000003-0000-0ff1-ce00-000000000000"
  }
}

A fix has been included in the latest SPFx release. Please upgrade to SPFx 1.18 and redeploy the solution.

@mdissel - are you using context.app.host.name in your code and throwing an error if it's unknown host (similarly to how it's done in the default scaffolded solution)?
If yes - Teams introduced new host name 'TeamsModern' which is valid one and specifies the app is running in "new" Teams.
Could you please modify your code to take it into consideration and check if that fixes the issue?
Thanks!

Thanks! We will upgrade to 1.18 and let you know the results.

If yes - Teams introduced new host name 'TeamsModern'

We're are doing some other stuff based in context.app.host.name so we will add 'TeamsModern' as an alternative name for MS Teams.

Thanks! The fullscreen app in the modern Teams app is working after upgrading SPFX to 1.18

Any update on this bug?
We have the same issue (mentioned in the dupe that is closed) and are on spfx 1.14 and cannot upgrade so easily because some of react component dependencies.

Hi
Any update on this bug?
We have the same issue and are on SPFx version 1.15.2.

Hi
Any update on this bug?
We have the same issue and are on SPFx version 1.15.2.

maybe this helps someone: It seems that somewhere around spfx 1.15 or 1.16 they introduced a default/scaffolded function called _getEnvironmentMessage(). This function holds a switch statement wich throws "unkown host" when in new Teams because only "Teams" but not "TeamsModern" is checked. in my case i just added a additional TeamsModern-Case-Statement an it worked in New Teams.
But additionally an "old" Bug presists, you can't change the webparts settings once it is added as Teams Tab.

We are experiencing the same problem in the Microsoft Outlook  365 MSO (Version 2401 Build 16.0.17231.20194) 64-bit
The first time we dock an app to the left panel, the app is loaded without any problem. But when you close and open Outlook and click on the docked app, the {teamSiteDomain} is not replaced, resulting in a white/empty browser page.

I can confirm the behavior described by @mdissel,  using Microsoft Outlook Microsoft 365 MSO (Version 2401 Build 16.0.17231.20236) 64 Bit. But the same is true when using the app as "M365 App" it works when using the SPFx-App directly in the browser but crashes wen using the desktop "Microsoft 365 (office)"-App. Unable to test it in new outlook as the apps are not listed there.
Cleary there is something wrong with the URL replacement
I also noticed that now we can change webpart settings in new teams - but it seems that everytime you edit the webpart settings (of a existing tab) technically a new tab will be generated (because the tab-url is not available anymore).

maybe this helps someone: It seems that somewhere around spfx 1.15 or 1.16 they introduced a default/scaffolded function called _getEnvironmentMessage(). This function holds a switch statement wich throws "unkown host" when in new Teams because only "Teams" but not "TeamsModern" is checked. in my case i just added a additional TeamsModern-Case-Statement and it worked in New Teams.
But additionally an "old" Bug presists, you can't change the webparts settings once it is added as Teams Tab.

Can you show me what files were edited and how please? I'm getting this issue even when updating to 1.18

you show me what files were edited and how please? I'm getting this issue even when updating to 1.18

in your *WebPart.ts in src\webparts\{webpartname} you should find a method called _getEnvironmentMessage this method contains a switch statement, where the last clause is case 'Teams', in default-clause the unkown host exception is thrown. Now simply add case 'TeamsModern' just below case 'Teams' like so:
private _getEnvironmentMessage(): Promise<string> {
    if (!!this.context.sdks.microsoftTeams) { // running in Teams, office.com or Outlook
      return this.context.sdks.microsoftTeams.teamsJs.app.getContext()
        .then(context => {
          let environmentMessage: string = '';
          switch (context.app.host.name) {
            case 'Office': // running in Office
              environmentMessage = this.context.isServedFromLocalhost ? strings.AppLocalEnvironmentOffice : strings.AppOfficeEnvironment;
              break;
            case 'Outlook': // running in Outlook
              environmentMessage = this.context.isServedFromLocalhost ? strings.AppLocalEnvironmentOutlook : strings.AppOutlookEnvironment;
              break;
            case 'Teams': // running in Teams
            case 'TeamsModern':
              environmentMessage = this.context.isServedFromLocalhost ? strings.AppLocalEnvironmentTeams : strings.AppTeamsTabEnvironment;
              break;
            default:
              environmentMessage = strings.UnknownEnvironment;
          }

          return environmentMessage;
        });
    }

Please note, in this example I have removed the exception - instead only a string will be returned.
If you do not need this info about the current environment as string, i think you can remove the entire method.

I also noticed that spfx webparts suddenly started working (again) in outlook

@nick-pape We are experiencing the same problem in the Microsoft Outlook 365 MSO (Version 2401 Build 16.0.17231.20194) 64-bit It does work correctly in Outlook Web Access.
The first time we dock an app to the left panel, the app is loaded without any problem. But when you close and open Outlook and click on the docked app, the {teamSiteDomain} is not replaced, resulting in a white/empty browser page.
Using spfx version 18.0

I can confirm the behavior described by @mdissel, using Microsoft Outlook Microsoft 365 MSO (Version 2401 Build 16.0.17231.20236) 64 Bit. But the same is true when using the app as "M365 App" it works when using the SPFx-App directly in the browser but crashes when using the desktop "Microsoft 365 (office)"-App. Unable to test it in new outlook as the apps are not listed there.
Clearly there is something wrong with the URL replacement
I also noticed that now we can change webpart settings in new teams - but it seems that everytime you edit the webpart settings (of a existing tab) technically a new tab will be generated (because the tab-url is not available anymore).
I'm using spfx 1.18.1

I just sent this through to SP admin and they tested on new Teams and they report it has worked. Thank you to Undefined-none.


When to use NuGet Client Sdk over NuGet API

"Not to be confused with the NuGet Web API" is not enough for me to understand the differences between each other, and when to use one over another?

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

The NuGet "Client" SDK is the set of libraries that expose nuget client (nuget.exe) functionality.
For example - when installing a package using nuget.exe, it calls the NuGet.org v3 endpoint to get the package, and then installs it into the target project.
What is your scenario?

@stg609, does this change make it a little clearer? Please let us know if you have any suggestions! https://github.com/NuGet/docs.microsoft.com-nuget/pull/1409

@stg609 please reopen the issue if you feel the updates don't resolve your concern.

Sorry for the late feedback. I was busy on doing other projects these days.
Our scenerio is to fetch package from local nuget gallery, and unzip the package. So, does it mean I should use NuGet "Web" API directly? Currently, I was doing the same thing using NuGet Client SDK.
BTW, to make use of the NuGet client SDK, does it require the client machine to have the nuget.exe? Thanks.
@loic-sharma

No worries! The NuGet client SDK should be a good fit for your scenario, you shouldn't need to move to the NuGet "Web" API.
You do need to install nuget.exe on your machine to use the NuGet client SDK.

Thanks very much!


Take advantage of having interface{} chanotify keys

Signed-off-by: Burcu Dogan<EMAIL_ADDRESS>

/cc @LK4D4

LGTM


Event.currentTarget is readOnly

https://developer.mozilla.org/ru/docs/Web/API/CustomEvent
https://www.w3.org/TR/DOM-Level-2-Events/events.html#Events-Event-currentTarget
RobustWebSocket.prototype.dispatchEvent = function (event) {
  if (!(event.type in this.listeners)) {
    return;
  }
  event.currentTarget = this;

A contradiction?

Thanks. I have even been asserting that currentTarget is actually the WebSocket instance this whole time because of that, lol.


Create templates

Build out templates for reading and generating annotations.

[ ] Add in the old transformLong code from the previous repository here: https://github.com/ktoddbrown/SoilDRaH/blob/f70b80b6199b157aefd44ad9e15f48d9f47c312f/R/readTemplate.R#L48

@ursapillay I went through and suggested a little more 'why' focus on the vignettes/readTemplate.Rmd Could you go through and build this out a little more. I think we are close!

@ktoddbrown I just revised vignettes/readTemplate.Rmd a bit


RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 64 but got size 40 for tensor number 2 in the list.

python gradio_app.py
text_config_dict is provided which will be used to initialize CLIPTextConfig. The value text_config["id2label"] will be overriden.
/media/47/Project_andy/envconda/mmyolo/lib/python3.8/site-packages/transformers-4.27.4-py3.8.egg/transformers/models/clip/feature_extraction_clip.py:28: FutureWarning: The class CLIPFeatureExtractor is deprecated and will be removed in version 5 of Transformers. Please use CLIPImageProcessor instead.
warnings.warn(
0%|                                                                                | 0/50 [00:00<?, ?it/s]
Traceback (most recent call last):
File "/media/47/Project_andy/envconda/mmyolo/lib/python3.8/site-packages/gradio/routes.py", line 393, in run_predict
output = await app.get_blocks().process_api(
File "/media/47/Project_andy/envconda/mmyolo/lib/python3.8/site-packages/gradio/blocks.py", line 1108, in process_api
result = await self.call_function(
File "/media/47/Project_andy/envconda/mmyolo/lib/python3.8/site-packages/gradio/blocks.py", line 915, in call_function
prediction = await anyio.to_thread.run_sync(
File "/media/47/Project_andy/envconda/mmyolo/lib/python3.8/site-packages/anyio/to_thread.py", line 31, in run_sync
return await get_asynclib().run_sync_in_worker_thread(
File "/media/47/Project_andy/envconda/mmyolo/lib/python3.8/site-packages/anyio/_backends/_asyncio.py", line 937, in run_sync_in_worker_thread
return await future
File "/media/47/Project_andy/envconda/mmyolo/lib/python3.8/site-packages/anyio/_backends/_asyncio.py", line 867, in run
result = context.run(func, *args)
File "gradio_app.py", line 254, in run_grounded_sam
image = pipe(prompt=inpaint_prompt, image=image_pil, mask_image=mask_pil).images[0]
File "/media/47/Project_andy/envconda/mmyolo/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 28, in decorate_context
return func(*args, **kwargs)
File "/media/47/Project_andy/envconda/mmyolo/lib/python3.8/site-packages/diffusers/pipelines/stable_diffusion/pipeline_stable_diffusion_inpaint.py", line 854, in call
latent_model_input = torch.cat([latent_model_input, mask, masked_image_latents], dim=1)
RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 64 but got size 40 for tensor number 2 in the list.
running in inpainting mode ，some errors occur ，what is the reason

i got same error about
Sizes of tensors must match except in dimension 1. Expected size 64 but got size 112 for tensor number 2 in the list.
how to fix it


CustomClauseQueries don't work on ent fields

CustomClauseQuery doesn't expect to have source ents because it's original use was only on the root resolver. Following #1768 it was added to Ents, but the source was never accounted for. The source ent is used as a key into the map of edges (so if you have two ents fetching edges it knows which edges set goes with which ent). The offending line is here: https://github.com/lolopinto/ent/blob/main/ts/src/core/query/custom_clause_query.ts#L186 which just hardcodes 1 as the source ID.

The tough part here is that BaseEdgeQuery has an abstract method sourceEnt(id: ID): Promise<Ent | null>. CustomClauseQuery just always returns null, but you need to call the static method on the source instance's class. What if we added a new property to Ent that was just a reference to its own class?
class UserBase implements Ent {
  public entClass = UserBase;
}

Then CustomClauseQuery (or whatever) could just do:
sourceEnt(id: ID): Promise<Ent | null> {
  return this.source?.entClass.load(this.source.viewer, this.source.id);
}

A more detailed explanation of the problem:

Give an ent Horse, there exists this method on it:

@gqlField({
  class: 'Horse',
  name: 'offspring',
  type: gqlConnection('Horse'),
})
offspring(): CustomClauseQuery<Horse> {
  return new CustomClauseQuery(this.viewer, {
    loadEntOptions: Horse.loaderOptions(),
    clause: query.Or(
      query.Eq('sire_id', this.id),
      query.Eq('dam_id', this.id),
    ),
    name: 'getOffspring',
  });
}

Codegen generated this in the GraphQL horse_type.ts:

export const HorseType = new GraphQLObjectType({
  name: 'Horse',
  fields: (): GraphQLFieldConfigMap<Horse, RequestContext> => ({
    offspring: {
      type: new GraphQLNonNull(HorseToOffspringConnectionType()),
      resolve: (
        obj: Horse,
        args: any,
        context: RequestContext,
      ) => {
        return new GraphQLEdgeConnection(
          obj.viewer,
          obj, // <-- Notice that the Horse object is being set as the "source"
          (v, obj: Horse) => obj.offspring(),
          args,
        );
      },
    },
  }),
  interfaces: () => [GraphQLNodeInterface],
  isTypeOf(obj) {
    return obj instanceof Horse;
  },
});

Inside GraphQLEdgeConnection, when a source is provided, it uses the source ID to get the correct page info for graphQL connections/pagination

edge_connection.ts#L139:
async queryPageInfo(): Promise<PaginationInfo> {
  await this.queryData();
  const paginationInfo = (await this.query).paginationInfo(); // *A
  if (this.source !== undefined) {
    return paginationInfo.get(this.source.id) || this.defaultPageInfo(); // *B
  }
  if (paginationInfo.size > 1) { // *C
    throw new Error(`Query mas more than one item yet no source was given`);
  }
  for (const [_, value] of paginationInfo) { // *D
    return value;
  }
  return this.defaultPageInfo();
}

Some things to note above:

A) It's getting the paginationInfo from the query (this is a BaseEdgeQuery instance – so that includes CustomClauseQuery as well as generated CustomEdgeQueryBase-extended instances).
B) Because this.source was set to the Horse object from the HorseType graphQL object, it will try to get the pagination for this specific horse edge. It will never find it (I'll show why later), so it always falls back to the default which won't include cursor information and makes it impossible to paginate.
C) This is where it checks that if you didn't pass in a source, there is only one possible pagination object (this is something that is hard coded into CustomClauseQuery)
D) And this is where it gets that one pagination object and returns it.

Normally, for generated edge queries on ents, it generates something like this, extending CustomEdgeQueryBase (instead of the CustomClauseQuery I returned from the offspring method above):

export class HorseToSireOfQueryBase<
  TEnt extends Horse = Horse,
> extends CustomEdgeQueryBase<TEnt, Horse> {
  constructor(
    viewer: Viewer,
    src: TEnt | ID,
    sortColumn?: string | OrderBy,
  ) {
    super(viewer, {
      src: src, // Notice here that it's passing the source
      groupCol: 'sire_id',
      loadEntOptions: Horse.loaderOptions(),
      name: 'HorseToSireOfQuery',
      sortColumn: typeof sortColumn === 'string' ? sortColumn : undefined,
      orderby: typeof sortColumn === 'string' ? undefined : sortColumn,
    });
  }

  async sourceEnt(id: ID) {
    return Horse.load(this.viewer, id);
  }
}

And so finally, when you call loadRawData from inside these queries (this method that does the actual fetching), CustomEdgeQueryBase instances correctly cache the results with the source's ID value as the map key: custom_query.ts#L291 whereas CustomClauseQuery has no reference to the source object and so defaults the cache to a map with just a single 1 key: custom_clause_query.ts#L186. This is why step 3B above always fails. The this.source object is always set in the GraphQLEdgeConnection object, but never in the underlying CustomClauseQuery.

what do you think of https://github.com/lolopinto/ent/pull/1837?


Adds documentation on how to boot from a RHCOS live ISO and interact with it

Adds some basic steps on how to boot from a live ISO in bare metal servers.
It includes also how to interact with the live ISO so that it can be configured to access using SSH instead of the console.

cc/ @donpenney @browsell @obochan-rh

/uncc @rcarrillocruz

/LGTM

/retest

/retest


Fix wrong invocation of toggleTodo which should be editTodo

Thanks for the fix!

Would also appreciate any other comments you have on the overall content of the new pages so far, as well.

So far great reads! I would love to see info about orchestrating nested resources. For example if I have a book and an author, and I can only know the author's name after I fetch the book, how can I make sure my component will have both book and author available?
How can I do this while showing loading indicators?
How to structure it with indexed lists?

Hmm.  Most of that really doesn't have much to do with reducer structuring, really.

I agree

Suppose you have an entity named security alert which belongs to an organization.
How do you structure your reducer(s) in order to handle:

Fetching all organizations
Fetching all security alerts by an organization
Fetching a security alert before knowing the organization itself (i.e - flat router)

How do you maintain the state for loader indicating in each item?
Using your system from Normalizing State Shape:
{
    entities: {
        securityAlerts : { byId : {}, allIds : [], isFetching: false },
        organizations : { byId : {}, allIds : [], isFetching: false },
        securityAlertsByOrganization : {
            isFetching: false,
            fetchingIds: [],
            byId : {
                1 : {},
                2 : {}
            },
            allIds : [1, 2]
        }
    }
}

Or perhaps something like that. But maintaining that state is pretty difficult with multiple reducers (Some perhaps only getting a slice of the original state)

Dynamic fetching and loading is admittedly something that I haven't had to deal with yet, so I don't have too much help to offer on that topic.


Fmt.kstrf doesn't like partial application

The following program gives an assert failure:
let parse fmt = Fmt.kstrf (fun x -> x) fmt

let check f = assert (f "X" = f "X")

let () = check (parse "%s")

This is fixed if you replace Fmt.kstrf by Format.ksprintf. This is fixed if you eta-expand (parse "%s") into (fun x -> parse "%s" x). Not sure exactly what is happening there ...

A Buffer.reset is missing on this line to protect whenever the internal buffer ends up being reused.

Here aswell.


LINK-1383: updated external user checking

Description :sparkles:
This PR fixes to check for external user flag
Issues :bug:
LINK-1383
Testing :alembic:
Tested locally

Codecov Report

Merging #204 (2a08c0d) into develop (9c5c0c7) will decrease coverage by 0.03%.
Report is 13 commits behind head on develop.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #204      +/-   ##
===========================================
- Coverage    99.88%   99.86%   -0.03%
===========================================
  Files          507      507
  Lines         9456     9465       +9
  Branches      2075     2080       +5
===========================================
+ Hits          9445     9452       +7
- Misses          11       13       +2

Files Changed
Coverage Δ

src/domain/organization/utils.ts
100.00% <100.00%> (ø)

... and 7 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Why React is listed in package.json, but not used?

It's a peer dependency of literallycanvas package. See: here.


Save the api-fetched tokens into browser cookie

@remo5000 iirc you mentioned this is why it prompts for login every refresh

Found the cause. role was not stored, causing a redirect to login  at the function redirectToAcademy(). localStorage was working as expected

Right, my bad actually I didn't change how the stuff was being saved when I added the check that role should not be undefined for user to access academy.


Start separating generated files in unique folder instead of inside the src/example tree

Problem
ZAP generated files are in-tree and this has undesired sideffects:

code review is harder as every change updates different distinct path. If zap updates are mixed with code updates, reviewers have to jump through files to filter out genreated vs non-generated files
relative paths are used extensively and include odities (e.g. data_model generated has same file names as example generated making include order important) go easier unnoticed

Change overview

created a 'zzz_generated' folder (for alpha-sorting to go at the end) for all example generated files. Explicitly named folders as 'zap-generated' instead of 'gen' to keep track of them easier in the future (a gen/include is used for build time include generation, would like zap to be explicitly different)
added 'output folder' support in the regen all script (make all examples go to the zzz folder)
Migrated examples and other builds to the new structure
Used <....> includes instead of  "..."

Testing
Verified that things build (except pump/pump-controller which does not build in ToT yet), CI will also validate things.
Checked that zap-regen-all does not change any output.

What do you think about also moving the .zap file of all examples and the controllers.
Instead of being located in the "xxx-example/xxx-common" that is now almost empty we could maybe regroup them. It would also make easier to go through them in zap tool gui to enable some clusters to all for example.

What do you think about also moving the .zap file of all examples and the controllers.
Instead of being located in the "xxx-example/xxx-common" that is now almost empty we could maybe regroup them. It would also make easier to go through them in zap tool gui to enable some clusters to all for example.

All for it. The PR is already quite large and some updates still need to be done. The src/controller/data_model also needs moving, java and python need moving and the regrouping of examples should happen as well.
Expect to gradually update things. Hoping only this starter PR was painful

@chrisdecenzo @jelderton @LuDuda @mspang @saurabhst ?


WRQ-6188: Added FloatingPopup prototype

Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
There is a requirement for a floating popup capable of free positioning.
This should also be able to receive content such as images and text.
This is a prototype implementation of the requirements.
Resolution
The component provides a container so that the app can pass contents inside the popup.
The component allows you to specify the desired position.
In the default sampler, we guide you on how to use it when passing images and text in the sampler.
Additional Considerations
Coordinate conversion may be required as the screen changes.
RTL may be required.
may need to trigger click events on components that are not hidden by the container.
Links
WRQ-6188
Comments
Enact-DCO-1.0-Signed-off-by: Hyelyn Kim<EMAIL_ADDRESS>

I have some questions about the props and component naming.

Popup component already renders in a FloatingLayer. FloatingPopup sounds confusing to me. What do you think?
Don't we need a prop for supporting on/off RTL positioning? When I change the locale to the ar-SA, icon and text flips which makes sense but the position remains. Some cases may remain but some cases may change the position to the right-based align. What do you think?

@seunghoh

I have some questions about the props and component naming.

Popup component already renders in a FloatingLayer. FloatingPopup sounds confusing to me. What do you think?
Don't we need a prop for supporting on/off RTL positioning? When I change the locale to the ar-SA, icon and text flips which makes sense but the position remains. Some cases may remain but some cases may change the position to the right-based align. What do you think?

I agree. So I changed naming to PopupContainer from FloatingPopup. This means that it is a container with the function of creating a popup.

Good. I added a prop for supporting on/off RTL position. But I should check whether it is needed under the normal circumstances of the app through the guide. If necessary, i should check whether the app wants to change only the location or the content direction as well.

LGTM. We don't have a fixed requirement yet, so I will keep this as a draft PR.


Labeling Files For Rag Inclusion

Describe the enhancement
For Rag.  What I Do At The Moment Is To Put At The Top Of Every File "Project::<project-name. # " And I Expect Everyone Does Something Similar.  I Hope You Will Implement Some Way To Select Files Based On  Frontmatter Or Those Parameters.   If there is a particular restriction on how you would select files it would not be compatable with that would be great to know.

Noted, great idea. Lots of improvements coming soon to automatic context including/easier context management in general.


Reconnection feature

Some of cloud providers are terminating long running connections (eg for MS Azure it's 4min default). So in this case a socket (which is created by the main Session object) may be closed and client will get this error https://www.dropbox.com/s/t2obll61wdzrxdq/Screen Shot 2017-01-03 at 03.23.57.png?dl=0
So, this issue may be prevented if the connection is reopened in some interval.
Looking forward to merging this PR, thank you.

I have merged it into staging branch. Will release new version in 1 week or 2.
Thank you :)

@tifayuki Nice one, thank you! 👍


Pressing "X" on a line with a single char doesn't do anything

If you have something like:
{
    ...
}

and try to press X (shift-x, that is) on the }, nothing happens.

This plugin has a lot of edge cases that I'm slowly fixing; thanks for bringing my attention to this one.
I won't have time to work on this plugin for a while, but I'll make sure to address this issue when I do.

Sounds good

Just wondering if there is any update on this issue?

Sorry, I don't have any updates.
I do recognize it as a bug, and will leave this issue open until it gets fixed. But that may be a long time from now, or never, if I'm being honest.
I haven't run into this issue myself during regular use, and I also haven't been using Kakoune much recently, so I haven't been motivated to fix it.

This has been resolved in the latest fix (https://github.com/evanrelf/byline.kak/commit/b1800206761b7b70041c1418a3e6f108cc0280d6).


⚠ OS ERROR 1332 - Windows PowershellCore Installation

I have installed LSD using release tag → 0.17.0 → (lsd-0.17.0-x86_64-pc-windows-msvc.zip)
and using scoop later as mentioned in #389.
Expected behavior
I have installed the latest release on Ubuntu WSL, output looks:
(ls and l are aliases as they are mentioned in the docs)

Actual behavior
But on Windows, this happens as mentioned in #356

I tried including lsd.exe path in Environment Variables
Second time I installed using scoop

Please tell me what do I do to make it work on Powershell Core 7.0.3

@ABSphreak
Hey bro, how you install it on «Windows» tried 100 times but nothing? ʅ_(ツ)_ʃ
I have WSL > Ubuntu 20.04

@rediffusion You want to install it on Windows (Powershell) or WSL?

@ABSphreak
WSL 2 > Windows Terminal > Ubuntu-20.04

@rediffusion I just downloaded lsd-musl_0.17.0_amd64.deb from Releases - 0.17.0 Page using wget and installed using dpkg.
# Downloads the release as .deb package
wget https://github.com/Peltoche/lsd/releases/download/0.17.0/lsd-musl_0.17.0_amd64.deb

# Installs it using dpkg
sudo dpkg -i lsd-musl_0.17.0_amd64.deb

@ABSphreak
OMG so easy. Bro, you are RAMBO! 😵
📍 Find another way too, check it:

export RUSTUP_IO_THREADS=1
curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh
cargo install lsd

Have a good day! 👍

Awesome! 🦄

@Peltoche can you point me to the solution of this issue?

@ABSphreak
If you want links here:
1, 2.


Impossible package out of default client-side

I noticed a problem that is easier explained than made a repository for.
Client-side, loading a dynamic file is done after XHR through FuseBox.dynamic. The third argument is the options argument that allows to specify a package.
    public static dynamic(path: string, str: string, opts?: {
        /** The name of the package */
        pkg: string
    }) {
        this.pkg(opts && opts.pkg || "default",

This function is never (beside in tests) called with options set, Following is the call from xmlhttp.onreadystatechange
FuseBox.dynamic(normalized, content);

I checked and saw a file pkg/file.name dynamically loaded is indeed registered in package default with pkg/file.name as its path.

So this part doesn't make any difference?

well, as opts is never given, these two lines miss each another: (350-351)
https://github.com/fuse-box/fuse-box/blob/master/src/loader/LoaderAPI.ts#L350
FuseBox.dynamic register the path pkg/file.js in package default
FuseBox.import looks for the path file.js in package pkg

@eddow because you are not setting it
require("pkg/target.js") /// or
FuseBox.import("pkg/target.js")

It's all simple.

I was getting to this conclusion in parallel indeed - I guess then the user has to load the packages "manually" if he wants to do it with ajax, execute it and then import it.

FuseBox v4 Major Upgrade
As of today, the latest major version of FuseBox v4 is available in fuse-box@next. FuseBox has undergone a full re-write from scratch, learning from its mistakes in the previous versions.
There are a lot of improvements, speed and config wise.
Get started here
We are still working on the documentation and polishing before the major release. We are getting very close. And we need help testing.
npm install fuse-box@next

Most of the issues that I am closing have been resolved, however, if you find that your issue is still relevant in v4 don't hesitate to re-open or (better) create a new one with v4 configuration.


Example project polish

Fixed up a few minor things with the example project to make it easier for newbies such as myself to get it running correctly.

Could you please update this pull request to reflect the latest changes to the master branch? Let me know and then I can pull it in.

I'll reopen it based against the master branch.


embed json-ld in html pages

This embeds the JSON-LD markup for sets, guides, and sources within the HTML.  It is intended to boost SEO.  The JSON-LD partials already exist.
This addresses #8187.

this is 👍


Fix issue 10777 - multiSort should return a SortedRange

As I now bumped for the second time into the problem of having a chainable multiSort (=SortedRange), I quickly wrote my own solution. Afterwards I realized that #4061 is already open, there are a couple of minor differences, here

a inline, static function instead of MultiPred is used.
in debug mode the symmetry is checked with an assert
the actual return code of opCmp is returned

As always, I am very happy about feedback.
@SzMK - let's join efforts to get this into Phobos?

thanks for your input @9il  - addressed ;-)

Another example of the new Travis linting at work :)
There was also a small issue with autotester, that should be fixed now.

"What would be other use cases for MultiPred" (from #4061)

Please not that the difference to the stalled #4061 is that we just use a static, pred function which is passed to assumeSorted, so this is hidden from the user and if wanted we can still extend add a MultiPred struct in the future if wanted (?).
That being said 1) I would like to see MultiPred too and 2) this should be ready to go. The change for the user is that instead of void a SortedRange will be returned. If so, it will be added to the changelog once #4228 is there.
Ping @MartinNowak.

Same comment as on #4061 applies, converting a multi-sort predicate to a sort predicate should be seen as an intermediate step towards a SortedRange w/ multi-sort support, b/c it doesn't make use of the performance benefits (e.g. lowerBound makes M * log(N) comparisions for M predicates).

Changelog entry missing.

Changelog entry missing.

Added (could you please have a look at #4228)?

b/c it doesn't make use of the performance benefits (e.g. lowerBound makes M * log(N) comparisions for M predicates).

Have you looked at the implementation of predFun? It will break as soon as either lessFun(a,b) or lessFun(b, a) is true. Is there really a better way to do this?
All other comments were addressed - and I added a small fix (multiSort didn't work for a single predicate ...)

@MartinNowak I am too lazy to rebase all the time, so I removed the changelog entry again. Here is the message:
   $(LI $(XREF algorithm, sorting, multiSort) returns now a
       $(XREF range, SortedRange) instead of `void`)

edit: btw I thought that if it's linked to an issue, you can generate the log entry automatically?

Yes, you'll get a Bugzilla fixed entry for references in a commit message. You can recognize that by the check mark in the dlang-bot comment.
https://github.com/dlang/phobos/pull/4235#issuecomment-214419022

Have you looked at the implementation of predFun? It will break as soon as either lessFun(a,b) or lessFun(b, a) is true. Is there really a better way to do this?

Ah great, that's as good as it can get.

Auto-merge toggled on

This pull request introduced a regression:
https://issues.dlang.org/show_bug.cgi?id=16179


Improve response message when a connection fails to respond within a timeout.

Duplicate of https://github.com/KumoCorp/kumomta/issues/196


As a grid operator I want to be able to change the power limit details of a metering point

Benefit hypothesis
As a grid company
We want to be able to update the power limit details of a metering point
So that we can ensure that the data is always up to date
Description
It must be possible for a grid company to update metering point master data after a metering point has been created. So once a metering point has been created it should be possible immediately after to update metering point master data. The things in scope for this story is the request from the grid company, validation of the message and accept/reject the request. The information to current and future balance supplier is treated in a separate story.
This story enables the grid company to update the power limit details for a metering point
Acceptance criteria

[ ] It must be possible for a grid company to send in a request for update metering point master data with business reason update metering point master data

[ ] If the request is rejected an reject message is generated and sent to the grid company.

[ ] If the request is approved an accept message is generated and sent to the grid company.

DoD

[ ] All tasks have been completed
[ ] All acceptance criteria have been completed
[ ] Unit test performed
[ ] Integration test performed
[ ] Tested
[ ] Accepted by business
[ ] Solution is deployed to test environment

Is solved in #299.


README: fix simple typos

A few simple typos in the README that could be fixed.
Thanks for this great repo !

@petervandenabeele thanks, and thanks for the sponsor contribution


Integrate kube deployment?

Our openwhisk-kube repo https://github.com/apache/incubator-openwhisk-deploy-kube currently is where our docs and deployment for kube are located.
But as kube is becoming increasingly the way our users want to deploy and use openwhisk, we should consider making it easier by integrating the kube deployment here instead. This will also better signal our commitment to kube which is something a lot of the other faas platforms already do.
I’m opening this issue to start the discussion and I’ll follow this up with a note to the Apache dev list.

+1
I dont see hard blockers to be Travis/Jenkins integrated

👍  any thoughts on Apache Mesos ?

my 2 cents.  Once a deployment has stabilized (isn't being rapidly revved & has basic Travis CI testing), then it seems to me to make sense to pull it back into the core repo to make it easier to maintain.

FWIW, there should be some community driven criteria on what constitutes a deployment being "ready" for inclusion. That is, we should not be encouraging inclusion of deployments that could not be built on every PR with an acceptable level of BVT.  Basically, what is the standard where the dev. community is not perpetually claiming depl. X is broken...  In addition, we have to make sure anything in main repo. is ready for the apache release process being developed.  Clearly, Kube and Mesos are on that path; what is the consistent test/criteria we apply to any other that comes along?

As a short term improvement, should we improve the docs to list all the deployment options and point to the external repos?

How is it going?

Consensus was to keep kube separate in its own repo.


Email notifier not working

Email notifier appears to be broken; I'm also wondering what these
fields SMTP email masking and SMTP Sender Masking are for; maybe you can
provide a sample data for it? Please see the image below. Thank you.

Please provide the error message you found.
The email and sender masking are used for  the alias of sender. But yet I’m not sure this alias will be worked on common mailserver.
You don’t need to set the email notifier when your app hosted by the server that been support for PHP mailer.

In the logs and debugbar, there are no errors visible.


[PE-D][Tester B] Overly long domain name is accepted

Should have either a warning or a blocking of such possible domains.

Labels: type.FeatureFlaw severity.Low
original: Saezenn/ped#3

Update email input constraints to accept long domain names in UG.

Overly long domain name is accepted ⇒ tbh why not? but sure warning works


variants: beagleconnect_freedom: Remove unnecessary nodes

ADC and PWM is now enabled in the base devicetree of beagleconnect freedom 0. Hence no need to enable it here.

I have fixed the CI here: https://github.com/zephyrproject-rtos/gsoc-2022-arduino-core/pull/125

@Ayush1325 try rebasing on -next now?

done


normalizeColor is copy-pasted in several React projects

normalizeColor is used in several places within react-native / react-native-web, and is also present in animated. To avoid redundancy I wonder if we could publish it as another separate package?

@necolas good call. I just published normalize-css-color under the react-community org. I will put up a PR for RN to depend on it, as well as react-primitives, and you are free to do so as well.
https://github.com/react-community/normalize-css-color
npm i normalize-css-color


feat(Data/Matrix/Basic): add missing theorem mulVec_sub

Adds the following missing theorem
theorem mulVec_sub [Fintype n] (A : Matrix m n α) (x y : n → α) :
    A *ᵥ (x - y) = A *ᵥ x - A *ᵥ y

Currently there only is mulVec_sub. I asked about it here on zulip.

done

bors merge


Apparently it doesn't work on windows 10 if you need to test locally.

I had to modify certain scripts:

@tosios/client package.json line 49 : "build": "react-scripts build" // because GENERATE_SOURCEMAP=false crash on terminal.
package.json at line 31: "build:move": cp is only works on Unix, I try to change to copy to universal, but fail, also with robocopy, so I just remove, and copy manually, and also remove the last part on && yarn build:move.
With that I can run the project.

also with the code .tsx inside packages client ...
in order to fix just put on : .prettierrc
"endOfLine":"auto"

also with the code .tsx inside packages client ...
in order to fix just put on : .prettierrc
"endOfLine":"auto"

Hi @lucks,
Thanks for bringing these issues up! I think I can find a better way of handling this that would be cross-platform. I should do a PR quite soon.

Hi @lucks,
Thanks for bringing these issues up! I think I can find a better way of handling this that would be cross-platform. I should do a PR quite soon.

I know this is quite a while ago, @lucks , but for anyone else who stumbles upon this - I am using the Windows Subsystem for Linux on Windows 10, installed the 20.04 ubuntu distribution, and via linux terminal was able to build and run this without any issues at all.


any suggestions trying troubleshoot an issue with mitmdump

We're migrating a set of jobs that run integration tests for one of our applications from one server to another.  There is a pair of jobs -- one that runs mitmdump to write to a file capturing the http traffic from our tests and then one that runs the same test suite using mitmdump replaying the responses.
This process is working fine our current server, which runs on OSX using v 0.10 of mitmdump.  However, we are experiencing one issue on our server we are moving to, which is redhat linux running v0.11.4 of mitmpdump.
One our tests performs and post to a webservice and expects it to succeed with an HTTP 201 status code.  It then repeats the same web service call, only this time it expects the call to fail -- because its testing that the process is non-repeatable.
Our test job that uses the actual backend web service succeeds -- the first response passes, 2nd fails.
However, our test job that replays the captured conversation from the above job fails.  Both web services calls result in a 201 status, where we expect the 2nd call to fail.
Was there a change in behavior between 0.10 and 0.11.4 that might have introduced this behavior difference?
Is there some switch or configuration option I might be able to tweak to adjust the behavior for sequential requests?  I looked at the doc but didn't see anything obvious.
In the meantime, it looks like the capture file is just pure text.  I'm going to try to see if I can see anything obvious in them that might provide some clues.
Thanks.
Doug

Turns out possibly to no surprise my issue had nothing to do mitmproxy.   I was grasping at strays mainly because I don't know implementation details on how it works.   The issue was on my end where my script was not setting a much needed env variable which was crucial to ensuring that the tests would repeatedly send the same requests.
Thanks again for a great tool.


Dotnet Test selecting the wrong Test Execution Command Line Tool Version

https://github.com/microsoft/vstest/issues/2271
I'm unable to reopen the above issue.
Same issue, I have the most recent SDK(.NET Core SDK (3.1.100)) and it is still using 16.3, as opposed to 16.4.
If I run from the CLI, using "dotnet test", 16.3 is used. However, if I use the developer CLI from Visual Studio, and use "vstest.console.exe", 16.4 is used as expected.
I've uninstalled all SDK's, then reinstalled 3.1.100, same issue. I also found that the version of "vstest.console.dll" that is being placed in the dotnet directory(Program Files\dotnet\sdk\3.1.100) is version 16.3.

Reproduced for me also. How I can use latest released 1.6.5 version of Microsoft.Net.Test.Sdk when start tests by dotnet test command ?

Test platform is inserted into Visual Studio and dotnet toolset and will be resolved to those versions that are built-in, for 3.1.100 that is 16.3. You can override the path to the vstest console using  -p:VSTestConsolePath="<path>", the appropriate assembly is shipped in microsoft.testplatform.portable nuget package.
Relevant code
dotnet test -p:VSTestConsolePath="$(Get-Item ~\.nuget\packages\microsoft.testplatform.portable\16.5.0\tools\netcoreapp2.1\vstest.console.dll)"

Next release of SDK will reference 16.5.0 test sdk.
More info about the insertions here.

NET Core SDK 3.1.102 still using version 16.3 of vstest

@goblinmaks yes, that is how the shipping works, the version of vstest.console is baked into dotnet test and Visual Studio. The latest version should be in 3.1.200

What is the fix that you are looking forward to use? Because if you update the desktop SDK version the new test host will be used from that nuget package.

@nohwnd it should be expected behavior to use SDK version from nuget for my point of view. I'm still wait https://github.com/microsoft/vstest/issues/862#issuecomment-585427338 this improvement.
Thanks,

@nohwnd we know anything about release date of 3.1.200 sdk ?

@goblinmaks

we know anything about release date of 3.1.200 sdk

I don't think there was anything publicly announced. And I can't share internal info 🙂

it should be expected behavior to use SDK version from nuget for my point of view.

Are you talking about test.sdk here? Because it does. Test SDK does not carry the console, just the targets that make your project a test project  and give it a default entry point. And then it depends on test host nuget.
The version of console is selected by the runner, not by the project.  Or VS uses the one that is built in into it. Or dotnet test uses the one the is built into the sdk.  You can also override it using the VSTestConsolePath msbuild param shown above:
C:\Projects\temp\UnitTestProject1 [master +12 ~9 -0 !]> dotnet --version
3.1.101
C:\Projects\temp\UnitTestProject1 [master +12 ~9 -0 !]> dotnet test
Test run for C:\Projects\temp\UnitTestProject1\UnitTestProject1\bin\Debug\netcoreapp3.1\UnitTestProject1.dll(.NETCoreApp,Version=v3.1)
Microsoft (R) Test Execution Command Line Tool Version 16.3.0
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...

A total of 1 test files matched the specified pattern.

Test Run Successful.
Total tests: 1
     Passed: 1
 Total time: 1,9922 Seconds
C:\Projects\temp\UnitTestProject1 [master +10 ~9 -0 !]> dotnet test -p:VSTestConsolePath="C:\Users\jajares\Downloads\microsoft.testplatform.portable.16.6.0-preview-20200309-01\tools\netcoreapp2.1\vstest.console.dll"
Test run for C:\Projects\temp\UnitTestProject1\UnitTestProject1\bin\Debug\netcoreapp3.1\UnitTestProject1.dll(.NETCoreApp,Version=v3.1)
Microsoft (R) Test Execution Command Line Tool Version 16.6.0-preview-20200309-01
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...

A total of 1 test files matched the specified pattern.

Test Run Successful.
Total tests: 1
     Passed: 1
 Total time: 1,9813 Seconds

That said I think it would be nice if we were publishing a package that will add the executables to the output folder, like testhost does, and that vs and dotnet test would look for those executables first. This would help to both simplify the delivery of updates, because we could fix the version without updating VS, and might work nicely for self-contained scenarios. I will try to bring it up with our team, but don't expect it any time soon, we still work hard on stabilization, and this would be a new feature.

@nohwnd Is there somewhere a documentation (or an easy way to find out) on which VsTest versions are bundled with .NET SDK versions? E.g. for .NET Core 2.1

Not that I am aware of, but we should start putting them into release notes I think. I will be working some more on our release pipelines and shipping strategy in the upcoming sprint, what I would like to see are better release notes that sum up our efforts and also point out community contributions. Linking the version to the sdk in which it was released would be a nice addition to that.
Just btw if you was .NET Core 2.1 which exact version do you mean? It might change with every servicing release.  🙂

@nohwnd Thx! The release note of 16.5 that summarized all the issues of the pre-releases (since 16.4) was very useful (so that people did not have to cumulate the issues of the pre-releases), so please keep that practice as well!

Great to hear that, I was lazy and just wrote a script to write it for me, so there is no added work to just specify the previous major version instead of previous preview version when I am releasing :)


Cursor position always changes by 2

I noticed that when splitting text, the chunks have a cursor position which changes by length of 2. I had assumed this was meant to point to the character position for the start of the text?
In this block of code:
https://github.com/kawakamimoeki/baran/blob/main/lib/baran/text_splitter.rb#L21C7-L26C10
Is the intention to do cursor += chunk[:text].length rather than cursor += chunk.length ?

This was a bug. I fixed it #17 . Thank you!

I see that this commit was created before the 0.1.11 tag/release, and when I install the gem from rubygems, it installs v0.1.11, however the code doesn't appear to have this issue fixed:

Any idea why?

@moracca Sorry, I made a mistake pushing. I have released a new version, please try it.

@kawakamimoeki thank you, 0.1.12 is working for me.  Should 0.1.12 be set as the default version on rubygems?


IsActive Mixin undefined

I`m trying to upgrade to rc1 from beta-3. In docs i see new mixin - IsActive:
import { Link, IsActive } from 'react-router';
but it returns undefined.

I noticed that IsActive mixin not exported from index.js...
Stable release on NPM? Seriously?

Is the plan for people to use the History mixin instead, @ryanflorence ?

woops, missed that in the upgrade guide, just added it, check out the table at the end: https://github.com/rackt/react-router/blob/master/UPGRADE_GUIDE.md#state-mixin


Flake8: Fix leftover of W504 and E402

During multiple merges we missed some Flake8 rules

Hi @grendel513 / @mtesauro / @devGregA / @Maffooch / @cneill / @blakeaowens. I would like to ask for a review.
It is a blocker for #9755 and #9754  and it would be nice to have them in the following release.

I may have been responsible for these while fixing conflicts. If so, I apologize!


Move stream helper templates to base classes try2

Move stream helper templates to base classes

add a protected virtual Read/Write function to facilitate the template functions existing in the base classes without compilation errors.
Update ClmFile and MapReader to use new template.
Update some MapReader types to be fixed width
Add std namespace prefix to some size_t calls in MapReader.cpp
Add some fixed size variables to MapWriter.cpp
Refactor ranged based loops to use auto and const & where appropriate
Remove some range based loops when writing data

This resolves issue #38

Here's a possible implementation for a templated Write method:
	template<typename T, typename A> inline void Write(const std::vector<T, A>& vector) {
		WriteImplementation(vector.data(), vector.size() * sizeof(T));
	}

The one downside, is you'd need to include <vector> for that method to be valid. Not sure if it's worth putting in the main Stream header, or as an external helper method elsewhere.

This is a good idea. Why didn't I think of it. :)
Perhaps the correct thing to do is add one for vector, string, and array. These are the common ones we use in stream operations. Then of course you are including all three headers. I don't think this is a big deal in our case though. We tend to include all three or at least 2 of the container headers in any of our code that requires a stream anyways. I think the gains are probably worth including the extra headers. Otherwise we are constanly having to upkeep what we are taking the sizeof in the read/write operations.
These templates would be used in VolFile, ClmFile, MapReader, and MapWriter (basically all of our stream code). Since all areas we use streams would benefit from them, I think that is a pretty strong reason to include them in the stream library.
If we go the separate Stream header route, all four files would have the added include. Might as well not require adding the extra include everywhere we are using streams.
I don't see using this stream library in an environment where array, vector, and string are not being included. If the user is avoiding these headers, they are probably going to go for a 'lower level' solution than this stream library, such as just using ifstream and memcpy directly?
Open to other suggestions though.
Thanks,
Brett

Good point about making it available for vector, string, and array.
I think std::array might already be covered by the template methods, since &arrayVar and sizeof(arrayVar) do the expected things.
For string we would need to decide if the null terminator gets written or not. There are many cases where you don't want the null terminator included, though also many cases where you do want it included. We could possible provide alternate methods to do either, though as the type has not changed, this is not a simple overload. You would need different method names to support both operations. Or just leave out the specializations, and force the programmer to be explicit about it.
For vector, this does seem common enough that default support is desirable. My concern is that importing <vector> might bring in many other large headers, or be slow to compile. I wonder if there is a way to do it generically, so it doesn't require importing the <vector> header in the stream library. In particular, if calling code needs to read/write vectors, the calling code will have imported the vector library itself.
It's tempting to try and use the contiguous container property, though it seems contiguous contains are not marked in the type system, so there is no type trait to query. The standard simply references these properties as an editorial note. It may be possible to provide your own template specializations, though I think that requires importing the vector header.
Related, there should be a safety mechanism so the template is not used for certain types where it might be dangerous to use it. In particular, using the template method to write a vector will not do what's expected. Perhaps we could add a restriction on what types the template is usable for, perhaps checking type traits for a POD datatype.
This is starting to sound like a separate issue.


Try installing ROS Indigo from Debian packages

In PabloGN/Docker-raspbian-ros-indigo, ROS Indigo (ros-indigo-ros-base) is installed via apt-get.
See if we can do the same using our base image, replacing the time-consuming source installation.
Notes

Question on ROS Answers: http://answers.ros.org/question/242119
See http://wiki.ros.org/ROSberryPi
See http://wiki.ros.org/indigo/Installation/UbuntuARM ("RPi3 with Rasbian Jessie")

Successfully installed ros-indigo-ros-base from Debian packages ✅


Android Compose Add a Scrollable List

Name of the Codelab or Codelab URL
Describe the problem
There is no "headlineSmall" stylein typography for card text
In which lesson and step of the codelab can this issue be found?
Lesson3. Add a list to your app
step 9
How to reproduce?
What are the exact steps to reproduce the problem?
Versions

What version of Android Studio are you using?
What API level are you targeting?

Additional information
Add any other context about the problem here.
codelab: basic-android-compose-training-add-scrollable-list

Hi @jsmecha
Thank you for writing in with your issue.
As you did not provide any code or screenshots it is hard to know exactly what you typed and tried.
I can confirm that using MaterialTheme.typography.headlineSmall does work as intended while working through the codelab.
If you would like, please make sure you have the latest release version of Android Studio and the latest code from the repo and try working through the lesson again.
If you still have an issue, create a new issue and provide as much details, code, screenshots to fully demonstrate the problem so we can help you further.
Hope you are enjoying the course!
Regards.


[Accessibility] Footer: update p with h3 heading tag for theme sites title

No known ticket, fix requested in chat after testing with Wave evaluation tool plugin
https://62713c57bcbdf4004a3d99b8-pngiaidlsk.chromatic.com/?path=/story/layout-footer--default

@eea-jenkins build storybook


fix(wallet): Outputing TXIDs after wallet optimization

:tada: This PR is included in version 3.4.2 :tada:
Your semantic-release bot :package::rocket:


[Question]: solution for it ?

Describe your problem
[ERROR]Model(jina/jina-embeddings-v2-base-de@Ollama) not authorized

Did you delete this model in 'Settings > Model providers'?


Many - Many doesn't work

Hi @kiaking ,
I am encountering the issue Cannot read property 'hasPivotFields' of undefined when trying to setup Many - Many relationships.
I am using Laravel as Backend so my data structure is

Member model is:
static fields() {
    return {
      id: this.attr(null),
      books: this.belongsToMany(Book, MemberBook, 'memberId', 'bookId'),
    }
}

Book Model:
static fields() {
    return {
      id: this.attr(null),
      members: this.belongsToMany(Member, MemberBook, 'bookId', 'memberId''),
    }
}

MemberBookModel
static fields() {
    return {
      memberId: this.attr(null),
      bookId: this.attr(null)
    }
}

Thank you in advance!

Hi! Thank you for the report! The error sounds like it's not available to find the model. Could you give me the entity name for the models? Or maybe you should do this...?
store.dispatch('entities/members/insert', {
  data: json.member
})

@kiaking
Hi mate,
The entity names are member, book and memberBook respectively. I don't know if the issue of coming from my pivot key?
Is it possible for you to show me the data response structure you used in the example?
Thanks heaps!

Hmm... strange. OK, so the following test is passing.
it('can create a data with belongs to many relation', async () => {
  class User extends Model {
    static entity = "users"

    static fields() {
      return {
        id: this.attr(null),
        permissions: this.belongsToMany(Role, RoleUser, 'user_id', 'role_id')
      }
    }
  }

  class Role extends Model {
    static entity = 'roles'

    static fields() {
      return {
        id: this.attr(null)
      }
    }
  }

  class RoleUser extends Model {
    static entity = 'roleUser'

    static primaryKey = ['role_id', 'user_id']

    static fields() {
      return {
        role_id: this.attr(null),
        user_id: this.attr(null)
      }
    }
  }

  const store = createStore([{ model: User }, { model: Role }, { model: RoleUser }])

  await store.dispatch('entities/users/create', {
    data: {
      id: 1,
      permissions: [{ id: 1 }, { id: 2 }]
    }
  })

  const expected = createState('entities', {
    users: {
      '1': { $id: 1, id: 1, permissions: [1, 2] }
    },
    roles: {
      '1': { $id: 1, id: 1 },
      '2': { $id: 2, id: 2 }
    },
    roleUser: {
      '1_1': { $id: '1_1', role_id: 1, user_id: 1 },
      '2_1': { $id: '2_1', role_id: 2, user_id: 1 }
    }
  })

  expect(store.state.entities).toEqual(expected)
})

Now, the key name pivot will be ignored by Vuex ORM because it doesn't show up on your model definition. But it shouldn't throw any error... It looks like it should work.
Is it possible for you to provide me the full code or create reproducible fiddle? You could send me an email if the code is too private to show to the public<EMAIL_ADDRESS>

@huy-tran Hi! Do you have any update on this? I'm kinda stack without some reproducible code :/

@kiaking Oh sorry mate, I just forgot this entirely. We can close the issue for now. However, I will send email to you if I can asap. I am currently finalizing some projects. Thanks for your great support again!
Cheers!

OK! Thank you for the confirmation! I'll close the issue for now.
Please feel free to reopen this issue if you encounter this again, or open another issue in case you have any other troubles : )
Good luck for your project! 👍

I too had this error: Cannot read property 'hasPivotFields' of undefined
In my case it was caused by attempting to use a model before registering it correctly.  That is, I forgot to include this line: database.register(AwsSync, aws_sync)
It might be nice to have a slightly more useful error message in this case?
Cheers

@josh-m-sharpe Thank you for the report! Yes, we should have better error messages :/


return double quotes on timeout exception

Do we need to do the same for InterruptedException?

I'm not sure. But, I think it'd be safer to make it specific. The current problem we are solving is the timeout issue which happens when the user pulls the app into the foreground.
Do you know what triggers InterruptedException here?

In theory, it's if something interrupts the sleeping thread while we're waiting for the future to complete. It's unlikely but possible.
My thought is, if we send something on failure, we should always send the same thing on failure.

NICE SUGGESTION! I'm now much happier with it! Thanks!!!

I love this! If you're happy with your tests, please merge it to v4.5.4dev


Add autopkgtest support by default

This is a patch by Martín Ferrari<EMAIL_ADDRESS>which I'm forwarding. I've already applied it in the latest upload of dh-make-golang for Debian.

Thanks. In the future, it might be prudent to get patches merged before including them in uploads to Debian.

Am 03.07.2017 um 10:54 schrieb Michael Stapelberg:

Thanks. In the future, it might be prudent to get patches merged before
including them in uploads to Debian.

You're right, sorry about that.

Hey all, I did not even remember this repo existed.. Are we not using alioth as the canonical source?

As far as I know, this here on github is the canonical source. However, it would make sense for me to use alioth instead. Michael, what do you think? Time to switch?

Let’s postpone the switch until alioth was upgraded to software which allows for pull requests and continuous integration (the lack of which, plus networking effects, is the reason why this and other repositories are on GitHub in the first place).


Timestamp & Save flag

What kind of change does this PR introduce?
PR introduces timestamps on builds and a save option in watch mode to allow a new build to refresh regardless of change. #1499 & #1992
Did you add tests for your changes?
No
Does this PR introduce a breaking change?
Probably not

Would be nice to introduce this now, so people aren't 100% shocked about changes in the new CLI repo. This PR could be discarded but it seems reasonable to "backport" this from our repo.

@bebraw What's the status on this? I could close it if it's not on the agenda

Looks good to me.

We definitely need to add tests to this once binCases are finished.

@TheLarkInn where's that?


Hydroglycerin, a disease that fills you with Potassium

About the PR
Decided to play around with the disease system and made this. Maybe it's good enough to go in-game, who knows?
Obtained only via artifacts (and admemes but whatever)
Features of disease include:

Spaca resistance of 0.15 (really high and i should probably lower it, what do you think?)
 (Signify that thirst is draining rapidly)

disease-hydrophobia = You panic at the thought of water. (AKA Hydrophobia, there is probably a better sentence to use here)

Cured by drinking water -- but it only really works if the water doesn't react with the potassium in your stomach...
Super unbalanced! Yay!

Why: i dunno, felt like it
Screenshots

Changelog

Needs a way to be acquired in game

Needs a way to be acquired in game

eh, fine, i'll do that in a bit
would letting artifacts give it to you work?

This pull request has conflicts, please resolve those before we can evaluate the pull request.

:\

i kinda like this in concept

The potassium bombing incidents all over again :mistaken:

Closing because tbh this is a low effort PR and I don't want to work on it, revive if you want


Remove most python2 packages

Motivation for this change
TL;DR: Just playing around, commented out all python2 packages that do not cause any issues. I believe only 16 of them are left untouched.
Now the question is: What's the deprecation policy of nixpkgs :thinking:
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

@ofborg build nixops

error: undefined variable 'rpkg'

       at /home/siraben/.cache/nixpkgs-review/pr-131585/nixpkgs/pkgs/development/tools/fedpkg/default.nix:29:29:

           28|   patches = [ ./fix-paths.patch ];
           29|   propagatedBuildInputs = [ rpkg offtrac urlgrabber fedora_cert ];
             |                             ^
           30|

       … while evaluating 'chooseDevOutputs'

       at /home/siraben/.cache/nixpkgs-review/pr-131585/nixpkgs/lib/attrsets.nix:497:22:

          496|   /* Pick the outputs of packages to place in buildInputs */
          497|   chooseDevOutputs = drvs: builtins.map getDev drvs;
             |                      ^
          498|

       … from call site

       … while evaluating the attribute 'propagatedBuildInputs' of the derivation 'fedpkg-1.29'

       at /home/siraben/.cache/nixpkgs-review/pr-131585/nixpkgs/pkgs/stdenv/generic/make-derivation.nix:203:11:

          202|         // (lib.optionalAttrs (attrs ? name || (attrs ? pname && attrs ? version)) {
          203|           name =
             |           ^
          204|             let

       … while evaluating the attribute 'out.outPath'

       at /home/siraben/.cache/nixpkgs-review/pr-131585/nixpkgs/lib/customisation.nix:156:13:

          155|             drvPath = assert condition; drv.${outputName}.drvPath;
          156|             outPath = assert condition; drv.${outputName}.outPath;
             |             ^
          157|           };

       … while querying the derivation named 'fedpkg-1.29'


Bash on Windows

Great project, icloud for Windows is garbage.  Before I jump in do you know if it's compatible with Bash on Windows (Ubuntu) or do I need to use Docker on Win10?  Thanks

Hi @brentratliff, I think it should work fine on Windows, even from the Windows command prompt. I'm using os.path.join so the file paths are cross-platform. However, I haven't tested on Windows, so I'd be interested to hear if there's any issues.

Thanks. I use Windows for various reasons but I'm really more comfortable with the Linux command line.  The Linux subsystem for Windows is one of the main reasons I moved from MacOS after all it's recent bloat.  I'll give it a shot and report back.

Hi @brentratliff, did you ever manage to get it working on Windows? Closing this for now, but would be great to get some feedback if you ran into any problems (or if you didn't!)

Not yet, I got side tracked by other projects.  When I get to it, I'll be sure to report back.  Thanks.


Entering the page shows that ReferenceError is not caught: defineComponent is not defined

Version
3.0.7
Reproduction link
https://jsfiddle.net/7z8dr6fq/

Steps to reproduce
npm run dev
What is expected?
The page is displayed normally
What is actually happening?
Page crash

楼主怎么解决的，我今天也碰到了。 突然间就坏了。

I fixed it!. FYI if someone later met this issue

Terminal: npx vite --force
Chrome dev tools:  Network => Check Disable cache.
Hard refresh page

This occurs when I refactor my project to use the script setup sugar.

又出现了这个问题，我们同事都炸了

Please open a new issue and provide a valid minimum repro, otherwise it will still be closed


Use Spatial Indexes in Geospark SQL?

Expected behavior
I see Geospark provides Quad-Tree and R-Tree indexes.. Can we let SparkSQL choose which index?
Actual behavior
Steps to reproduce the problem
Settings
GeoSpark version = ?
Apache Spark version = ?
JRE version = 1.8?
API type = Scala or Java?

@paulzwu Yes, you can do it. Please read [1] to set the spatial index used in GeoSparkSQL join query.
[1] Set GeoSparkSQL parameter: http://datasystemslab.github.io/GeoSpark/api/sql/GeoSparkSQL-Parameter/


People do not save when the first line of the notes is a URL.

Not sure when but this appears to be fixed.


Output file with final resting positions of nodes

A self created file at the end of the run containing all node positions would save you losing all the data each time you close python. This can possibly be triggered using a --save flag?

Currently crudely implemented using code below but can easily be done in a neater fashion.
   names = [i.split(' ')[0] for i in  tuple(open(args.input_graph))[2:2+g.num_vertices()]]
    f = open('tsnet.out','w')
    for i in g.vertices():
        f.write('%s,%s,%s\n'%(names[int(i)],pos[i][0],pos[i][1]))
    f.close()

Thanks for the suggestion! That does seem very useful, and trivial to implement. I'll add this in a few days.


Support named parameters as form fields

Like Datasette, except they will apply to all of the queries on the page.

People won't be able to use sql as a parameter, I think that's OK.

Documentation: https://www.psycopg.org/docs/usage.html#passing-parameters-to-sql-queries
I'm just going to extract %(date)s style parameters and turn those into form fields - I won't support %s positional parameters.

In [1]: class CaptureDict:
   ...:     def __init__(self):
   ...:         self.accessed = set()
   ...:     def __getitem__(self, key):
   ...:         self.accessed.add(key)
   ...:         return ''
   ...:
In [2]: d = CaptureDict()
In [3]: 'foo %(name)s and %(name)s and %(bar)s' % d
Out[3]: 'foo  and  and '
In [5]: d.accessed
Out[5]: {'bar', 'name'}

Surprising bug: In this block of code: https://github.com/simonw/django-sql-dashboard/blob/87fc304754d73f2eb6a3610a1296394f36e07746/django_sql_dashboard/views.py#L48-L55
Switching to using cursor.execute(sql, params) causes a new exception I haven't seen before:
    def _execute(self, sql, params, *ignored_wrapper_args):
        self.db.validate_no_broken_transaction()
        with self.db.wrap_database_errors:
            if params is None:
                # params default might be backend specific.
>               return self.cursor.execute(sql)
E               django.db.utils.InternalError: current transaction is aborted, commands ignored until end of transaction block
../../../.local/share/virtualenvs/django-sql-dashboard-ZqMBSvf9/lib/python3.9/site-packages/django/db/backends/utils.py:82: InternalError

The query being executed at this point which raises the error is:
-> return self.cursor.execute(sql, params)
(Pdb) sql
'RELEASE SAVEPOINT "s4610936256_x20"'

So I think this confusion is caused because I've ended up running a savepoint, when I really wanted to be manipulating a standard transaction.

Since I'm PostgreSQL only maybe I can work around this by digging down through Django's wrappers to get a raw psycopg2 cursor?

How about using a separate database alias which sets that READ ONLY thing on every new connection? Something like this:
import psycopg2.extensions

DATABASES = {
    # ...
    'OPTIONS': {
        'isolation_level': psycopg2.extensions.ISOLATION_LEVEL_SERIALIZABLE,
    },
}


Tenancy Database Connection Error

Thank you for this package. I  set up a laravel project with Hyn package was able to create my 2 tenants.
Now, I followed the doc here [https://hyn.readme.io/docs/eloquent-facade-tenant-system-model] Created a TenantModel and have one of the of the tenants' model extends it.
I expected the model to pull data from the tenant database but I get this error.
InvalidArgumentException in DatabaseManager.php line 239: Database [tenant] not configured.
Do i need to configure connection parameters for each tenant? Thank you for your anticipated help.
in DatabaseManager.php line 239
at DatabaseManager->getConfig('tenant') in DatabaseManager.php line 158
at DatabaseManager->makeConnection('tenant') in DatabaseManager.php line 68
at DatabaseManager->connection('tenant') in Model.php line 3351
at Model::resolveConnection('tenant') in Model.php line 3317
at Model->getConnection() in Model.php line 1861
at Model->newBaseQueryBuilder() in Model.php line 1834
at Model->newQueryWithoutScopes() in Model.php line 1804
at Model->newQuery() in Model.php line 3561
at Model->__call('orderBy', array('created_at', 'desc')) in Model.php line 3573
at Model::__callStatic('orderBy', array('created_at', 'desc')) in CourseTrait.php line 24
at CourseController->getAllWithPagination() in CourseTrait.php line 31
at CourseController->index()
at call_user_func_array(array(object(CourseController), 'index'), array()) in Controller.php line 55
at Controller->callAction('index', array()) in ControllerDispatcher.php line 44
at ControllerDispatcher->dispatch(object(Route), object(CourseController), 'index') in Route.php line 189
at Route->runController() in Route.php line 144
at Route->run(object(Request)) in Router.php line 653
at Router->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in EntrustRole.php line 42
at EntrustRole->handle(object(Request), object(Closure), 'admin|system|tutor') in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in SubstituteBindings.php line 41
at SubstituteBindings->handle(object(Request), object(Closure)) in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Authenticate.php line 43
at Authenticate->handle(object(Request), object(Closure)) in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in VerifyCsrfToken.php line 65
at VerifyCsrfToken->handle(object(Request), object(Closure)) in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in ShareErrorsFromSession.php line 49
at ShareErrorsFromSession->handle(object(Request), object(Closure)) in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in StartSession.php line 64
at StartSession->handle(object(Request), object(Closure)) in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in AddQueuedCookiesToResponse.php line 37
at AddQueuedCookiesToResponse->handle(object(Request), object(Closure)) in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in EncryptCookies.php line 59
at EncryptCookies->handle(object(Request), object(Closure)) in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 104
at Pipeline->then(object(Closure)) in Router.php line 655
at Router->runRouteWithinStack(object(Route), object(Request)) in Router.php line 629
at Router->dispatchToRoute(object(Request)) in Router.php line 607
at Router->dispatch(object(Request)) in Kernel.php line 268
at Kernel->Illuminate\Foundation\Http\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in CheckForMaintenanceMode.php line 46
at CheckForMaintenanceMode->handle(object(Request), object(Closure)) in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in HostnameMiddleware.php line 17
at HostnameMiddleware->handle(object(Request), object(Closure)) in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 104
at Pipeline->then(object(Closure)) in Kernel.php line 150
at Kernel->sendRequestThroughRouter(object(Request)) in Kernel.php line 117
at Kernel->handle(object(Request)) in index.php line 53

What version of hyn/multi-tenant are you on?

@Luceos
Am using version 2.0 with laravel 5.3

Is the service provider registered?

Yes is registered

and alias

Service provider is registered as stated on the documentation page

I changed it to this Hyn\Tenancy\TenancyServiceProvider it didn't work either

Yes it should be the FrameworkServiceProvider. I'll take another look.

A few things I notice:

HostnameMiddleware runs
Connection requested is tenant which is great, because it actually tries to use the tenant connection.
Tenant connection configuration is not set. This blocks code execution.

Possible cause is not having configured a website to the hostname, related code.

Exactly the point. I  think the package creates the connection parameters automatically. I can see new usernames created in mysql user table. Am I suppose to hard code tenant's connection parameters again?

Here is the table for website

and that of hostname

No the package will automatically set up the database connection when it finds a tenant. Perhaps the hostname is not matched. Let me check real quick how it resolves it.

So the code is not matching your custom port:
https://github.com/hyn/multi-tenant/blob/2.x/src/Tenancy/Helpers/TenancyRequestHelper.php

Oh great. That was the cause. I changed my port back to 80 and is working fine now. I noticed something when I change my connection name to 'tenant' in .env file. All tenant sites come up fine but the main site does not. What i did was to rename the parent's site connection parameters from hyn to 'tenant' in database.php. So when you are on parent's hostname it picks up the parameters and while on tenant's hostname resolves to default way of connecting to the database
 ```

'tenant' => [
'driver'    => 'mysql',
'host' => env('DB_HOST', 'localhost'),
'port' => env('DB_PORT', '3306'),
'database' => env('DB_DATABASE', 'forge'),
'username' => env('DB_USERNAME', 'forge'),
'password' => env('DB_PASSWORD', ''),
'charset'   => 'utf8',
'collation' => 'utf8_unicode_ci',
'prefix'    => '',
'strict'    => false,
'engine' => 'InnoDB ROW_FORMAT=DYNAMIC',
],
Is this recommended? Or there is another way of getting around this?

That is a great solution. You can do this yes, the drawback of this is that the settings will be overridden (temporarily during runtime) whenever a tenant was identified.
It's okay like this. I assume this issue is then resolved?


randomize unga select widget (q2)

Ideally, this should be implemented in the master enketo/enketo-core and then merged here before major customization work restarts (to avoid 2 merge projects). It should work in all select and select_multiple widgets.
See details here: https://github.com/kobotoolbox/enketo-express/issues/363
The special unga-select widget should inherit this randomization option.

done