Issue installing this on a hybrid OpenShift 4.11 cluster with Windows Server 2022 workers

I'm working with a customer who is trying to install this on a hybrid cluster running OpenShift 4.11 with windows server 2022 on worker nodes. Documentation says its supported on OpenShift but doesnt really say which version also doesn't mention version of Windows Server supported on worker nodes.
Instructions for windows deployment is missing some config required for scheduler to schedule the daemonset on windows nodes
as described in this document here https://docs.openshift.com/container-platform/4.11/windows_containers/scheduling-windows-workloads.html
after adding this in values.yaml
- key: "os"
      operator: "Equal"
      value: "windows"
      effect: "NoSchedule"

we can see the workload getting scheduled on windows nodes but container doesn't start up because of the windows version mismatch between the container and host worker machine
Just logging this here as an enhancement request to support windows server 2022 and also update missing information in documentation. I'm willing to work on this but wanted to reach out here first to see if this is already in the works or not

Thank you for your report!
Did you mean to report this issue on https://github.com/signalfx/splunk-otel-collector-chart ? This seems related to our helm chart as you mention a values.yaml file.

Not sure. What needs to happen is who ever maintains this image https://quay.io/repository/signalfx/splunk-otel-collector-windows needs to release a new version that supports windows server 2022. Currently this is the image that is being used and we can see in errors the about the version mismatch

Agree doc update that needs to be made to add the above config in values yaml needs to go to the chart repo but the main issue around support for windows server 2022 still would be here IMO

OK I understand better. Thank you for clarifying.

Perhaps I should have created two separate issues :) Sorry if I confused

We have identified this issue as an enhancement and will look at supporting a Windows 2022 image. This is also reported in https://github.com/signalfx/splunk-otel-collector-chart/issues/551

Windows 2022 docker images are now supported. Closing this issue and will follow up with Openshift issues, if any, in the chart repository.


Export Research to document (PDF, DOCX)

I see that within the  page on https://storm.genie.stanford.edu/ there is an option to show an article as a PDF.
Will that capability become part of the source, which can be run locally, either as PDF or DOCX?

That would be a great feature to include in the demo_light! Our team right now are prioritizing effort on tasks described in road map section of readme. We are happy to review and merge effort on improving demo light.

waiting to pdf export as in product demo page..


Win32 Input modules not integrated

Up until now the win32 input modules are missing.

[x] raw_input
[x] xinput

Modules still need to be tested in the suites


GMU-27: Logging improvement

Purpose
https://folio-org.atlassian.net/browse/GMU-27
Logging improvement

Approach

TODOS and Open Questions

Learning

Pre-Merge Checklist:
Before merging this PR, please go through the following list and take appropriate actions.

Does this PR meet or exceed the expected quality standards?

[x] Code coverage on new code is 80% or greater
[x] Duplications on new code is 3% or less
[ ] Check logging
[x] There are no major code smells or security issues

@DmytroBykov1 , why did you change error log level to warn?

Why do we need to log rules/rule in large number of places?

Why do we need to log rules/rule in large number of places?

@khandramai
As far as I see, the Rule-object is the business logic key element.

Please, fix all the comments. Pay attention of needenes of logging rules/rule and logging object hash code's - I assume for the large number of objects hash code will be in logs instead of values.

@khandramai
done

Why do we need to log rules/rule in large number of places?

@khandramai As far as I see, the Rule-object is the business logic key element.

@DmytroBykov1 is it changed during whole the business flow to log it on so large number of steps?

@DmytroBykov1 , 1) Lombok toString() seems contradict approach used on project. Please, make logging of original objects in json format and values in unchanged form.
2) Please, make logging for values/jsons that are critical or changed during business flow. For example, if rule isn't changed during flow we don't need to save its unchanged value in different places.


Update ros2.md turtlesim section

Great quick start - here a couple small suggestions from a fresh set of eyes!
Also, the turtlesim image link is broken. I can add this image
to fix it - where in the repo should such an asset be stored?

Thanks for helping to improve it!
Since you uploaded the image in this pr you should have a gitub url that you can paste into the markdown directly.


Check availability mail-directory at startup

Hi,
I am running the mail-server on AWS using ECS on an instance with CoreOS operating system, with the mail-directory on a separate persistent EBS volume. CoreOS is configured to automatically reboot the instance when a new version of the operating system is downloaded.
During the last weeks it happened twice that the mail-server seemed to be working properly (could login into dovecot, read mails, send mails, etc.) only I did not receive any new mails. Checking the logs I found that upon receiving new mails postfix was not able to deliver them to the mailbox because it had no permission to. So these mails ended up in a "black hole". Each time this happened this was after a new version of CoreOS was installed and the instance was automatically rebooted. Apparently after a reboot and subsequent automatic restart of all docker containers the mail-server container gets restarted before the EBS volume is available.
Each time this problem occurred a simple manual restart of the mail-server container was sufficient to solve the issue, but each time 2 - 3 days worth of new mails got lost because it took me a few days to realise something was wrong.
I am no expert in bash scripting, but going through setup.sh I don't think there is a check in there if the mail-directory is available and if the system has permission to write to it.
Would it be possible to add such a check to setup.sh? Preferably in a loop that checks this every xx number of seconds and does not continue until the check is positive. This would ensure the restart of the mail-server is delayed until the directory is indeed available and writable, and in case that "never" happens there would at least be a clear indication to the user that something is wrong.
Best regards, Ton

I think the real question is why your ebs is not mounted yet?
I also run on aws but Ubuntu but almost every running service depends on the mounts in fstab.
Do you have log of the mailserver when it goes wrong?

Hi Johan,
There is no fstab in CoreOS I am afraid. The EBS volume is mounted through the cloud-config file, along with some other basic services.
But even so, there can be numerous reasons why the mail-directory would not be available at startup, but isn't it the responsibility of the docker container to check if all resources that it needs are indeed available at startup?
It can also be other containers it depends on, or a database.
In the docker images I develop myself I consider it good practice to check availability of all needed resources in the startup script.
Certainly in clusters where container orchestration systems like ECS (which I am using) or Kubernetes are being used it can always happen that nodes fail, are being restarted, and containers are restarted. In these environments I think docker containers must be robust, check availability of external resources and warn the user if something is wrong.
Unfortunately the log-files of the faulty containers disappeared when I “restarted” (restart is delete and recreate here...) them. I could probably recreate them by manually rebooting the AWS instance, but now is not a good time for that.
Best regards, Ton

A docker container in my perspective is the same as running a application/service on your machine. This container misuse the docker idea of one process per container but just for argument we take postfix as an example.
The service file that starts postfix has requirements, one of those requirements is a filesystem. During boot (in our case system.d) makes sure the fs is mounted before starting postfix because it is a dependency.
With docker it is the same, when you start a docker container dependencies should be met before starting. How can the program inside the docker know that they are missing? The container can only check for certain files, but those files are also missing when doing a clean start with no emails send.
Also you really depend on the type of program to detect failure. A normal postfix service would do the same if the directory is missing. Because it can start without any problems but when it will try to deliver email it rans into problems, this will not stop the postfix service but just log a warning. The same as inside the docker. I assume emails will be saved in the spool dir which is not on a permanent state dir (it also failed to mount). I understand what you mean but how to catch these exceptions for all services in the docker I would like to hear how this could be done.
I also don't know if you start the docker container that keeps checking if a directory is available will see the volume when the host mounts it. Does this mount get detected? I guess not because email will go in a blackhole. So a strategy would be to fail startup because dependencies where not met.
Can you share the code that you use to verify the availability of the resources for at least startup?

Naturally one cannot check all resources for availability at startup. Pure “internal” resources must be considered to be available assuming sufficient memory and CPU is allocated to the container. However to make the container stateless certain directories may be linked to directories on the host, so that when a container fails a replacement container with the same host directories linked will take it’s place without any additional intervention needed.
I think it is the possibility to link external host-directories that makes running a docker container different from running just any application/service on your machine. It makes the environment in which the application within the docker container runs less controlled. Therefore I think it is good practice to check if directories or files that have a high likelihood of being linked to directories or files on the host are available at startup. Certainly those directories and files that are crucial for operating the system and errors due to their unavailability would remain unnoticed.
I do not know postfix that well, but I suspect that in my case the file postfix-accounts.cf is not yet available at startup of the container. Postfix apparently is okay with that and starts up, but it doesn’t know about usernames and passwords, so it cannot deliver any new mails to the mailboxes. Hence the messages I see in the log that postfix has no permission to deliver the mails.
So I think at least a check on that one is required. The availability of the directory /var/mail is also very important, but if postfix already checks that herself, it is not necessary to repeat that.
Again, I am no expert in bash scripting, but something simple like:
# Make sure postfix-accounts.cf is available
echo "Checking if postfix-accounts.cf is available..."
while ! [ -e /tmp/docker-mailserver/postfix-accounts.cf ]; do
     echo “File postfix-accounts.cf not yet available.”
     sleep 0.1 # wait for 1/10 of a second before checking again
done

at the beginning of the startup script should do the trick. It will delay starting up the container until postfix-accounts.cf is available, and if that never happens it will just keep on looping. Since in that case the mail-server will not start it will be clear to the user that there is something wrong.
No doubt there will be more sophisticated solutions though… :-)

Keep in mind that for ldap enabled server this file does not need to be present.
I would prefer to exit the container with a error code. The container restart will take over but other error logging will be available instead of a non working container that is running waiting for the directory.
Could you create a PR for this?

I would certainly like to contribute where I can, but I am not a software developer, so I do not know what a PR is, or how to create one…
But always eager to learn, so if you could explain in a few words what a PR is and what I should do to create one, or point me to some documentation, I could certainly give it a try.

PR means pull request.
You can read here more about it: https://help.github.com/articles/creating-a-pull-request-from-a-fork/


feat(extension): add a readme [EE-3085]

closes EE-3085
Changes:

Adds a readme with build instructions to the extensions directory

closing as this did not make it into the release


Synchronize the list of docker images between Submit Notebook and Submit Pipeline

Submit Notebook:

Submit pipeline:

Some notes on our current implementation:
The submit dropdown is hard-coded in one string on line 184 in SubmitNotebook.ts but the value (framework) is never used and instead the docker image is hardcoded in generateNotebookPipeline() in utils.ts to tensorflow/tensorflow:1.12.3-py3
Note that the pipeline editor docker image list is "hard-coded" in properties.json

Not sure if this came up in the scrum, but should these be hard-coded at all?  I'm wondering if these should be a function of the metadata instance since I would imagine the image to use is a function of the runtime metadata.

it did come up during the scrum, suggestion from luciano was to have a react component of a sorts, to provide a the same list of images to both items

But where does that component get the image names from?  Are these "factory" images or will users be able to "BYO"?

Based on a conversation with @lresende last year when I was writing the pipeline properties code we will have a set of images that we support that we will update from time to time. Thus a "hard-coded" list. I believe what we want to address with this issue is twofold:

Fix the fact that notebook-scheduler not only has a vague list of images, but also doesn't actually use the user's choice anyway.
Create a util to handle the image list, ie some sort of component that both packages "ask" for the list they are to use.

The first one is a bug and could be fixed relatively easily, but the second is not as simple.
We would most likely still have the list hard-coded in the utility component, but it would instead be in the application package.
Given the obvious bug here in notebook-scheduler I would either suggest a quick hard-coded bug fix for this sprint with a followup issue to add the util component, or for this to be a top priority enhancement and make creating the util part of this sprint. Either way I can handle this, but if we take the second path forward I will most likely need to push a few of my issues into next sprint.
@lresende  what's your opinion?

I'm just trying to understand.  If the hard-coded list is "as-designed", that seems fine provided we don't intend to let others bring their own images, etc.  What we need to avoid is having to cut a new release just to add/remove an image from the list - irrespective of if the set if "factory only" or not.
I agree that the two consumers need to get the set of images from the same source.
Since it appears that both consumers are referencing runtime metadata, I'm a little curious why these weren't part of that, especially considering that facility is already user-extensible (by design).

I think I see where you're coming from. Let's discuss this in detail at scrum Monday and I'll start working on it Monday based on whatever decisions we make there.

Fix the fact that notebook-scheduler not only has a vague list of images, but also doesn't actually use the user's choice anyway.

Yes, fix the bug...

Create a util to handle the image list, ie some sort of component that both packages "ask" for the list they are to use.

I would love to see better reutilization of UI components, as well as backend services components, and this seems to be what we are suggesting here:

A common widget to display a list of available TensorFlow images
If we want to enable users to add their own set of images, we then could expose the supported list of images from the metadata services (e.g. ,metadata/images)

Thanks for the response @lresende.  Is the reason not to add an images field (list-valued) to the runtime metadata?  Just thinking that these would be closely tied to the platform (kfp, airflow), but if images are "platform-agnostic", then I suppose we'd need to look into adding another namespace type (which  triggers lots of repetitive things so we need to be sure that's what we want to do).


add support for setting commit status on a repo

It's for external validation systems that work per-commit/pull-request, like continous integration systems.
See https://developer.github.com/v3/repos/statuses/.

Hey @eins78! Update on this? Would be great to merge this soon! Thanks! :smile:

Closed in error; reopening.

Closing this PR as there has been no activity for more than one year.


HuggingFaceEndpoint incorrectly assumes that the endpoint returns the prompt prepended to the response

Description: HuggingFaceEndpoint truncates the text because it assumes the endpoint returns the prompt together with generated text (but that's not the case, so the the code wrongly truncates the answer)
Issue: Fixes #7353
Dependencies: None
Tag maintainer: @baskaryan
Twitter handle: @juanan

This issue has also been discussed here:
https://huggingface.co/tiiuae/falcon-40b-instruct/discussions/51

#7386


setx COGNITIVE_SERVICE_KEY your-key

setx COGNITIVE_SERVICE_KEY your-key

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

ID: 7e866571-0651-bddb-f012-bf3bfe16f179
Version Independent ID: 27429467-3c37-27da-b839-065268c6af23
Content: Create a multi-service resource for Azure AI services - Azure AI services
Content Source: articles/ai-services/multi-service-resource.md
Service: azure-ai-services
GitHub Login: @eric-urban
Microsoft Alias: eur

@yueyuecompany23
Thanks for posting this. Based on that shared information we are not able to understand your query/feedback completely.
Could you please elaborate the query for better understanding. So, we can assist you further.


Adding custom window size and position

Hello, I am not a Native Android application developer. But I think custom window size and position can be added. How can I do this?
Best regards.

Thank you for bring it up,  i wish developer could continue this project but usually they find good developer and buy them or just make the busy with other thing,
No new update will be available , unfortunately.


Omnisharp not detecting already started server

I try to use OmniSharp for asp.net vnext stuff, so I do not have a solution file. I start the server separately using
mono OmniSharp.exe -p 2000 -s "~/path/to/my/project"
Unfortunately when I enter vim, I get an error that the server couldn't be started due to missing solution file. It looks like it doesn't detect my already started server. When starting the server, I get this error message:
Monodoc documentation tree could not be loaded.System.IO.FileNotFoundException: Could not find file "/usr/lib/monodoc/monodoc.xml".
File name: '/usr/lib/monodoc/monodoc.xml'
  at System.IO.FileStream..ctor (System.String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, Boolean anonymous, FileOptions options) [0x00000] in <filename unknown>:0
  at System.IO.FileStream..ctor (System.String path, FileMode mode, FileAccess access, FileShare share) [0x00000] in <filename unknown>:0
  at (wrapper remoting-invoke-with-check) System.IO.FileStream:.ctor (string,System.IO.FileMode,System.IO.FileAccess,System.IO.FileShare)
  at System.Xml.XmlUrlResolver.GetEntity (System.Uri absoluteUri, System.String role, System.Type ofObjectToReturn) [0x00000] in <filename unknown>:0
  at Mono.Xml2.XmlTextReader.GetStreamFromUrl (System.String url, System.String& absoluteUriString) [0x00000] in <filename unknown>:0
  at Mono.Xml2.XmlTextReader.ReadTextReader (Int32 remained) [0x00000] in <filename unknown>:0
  at Mono.Xml2.XmlTextReader.PeekChar () [0x00000] in <filename unknown>:0
  at Mono.Xml2.XmlTextReader.ReadContent () [0x00000] in <filename unknown>:0
  at Mono.Xml2.XmlTextReader.Read () [0x00000] in <filename unknown>:0
  at System.Xml.XmlTextReader.Read () [0x00000] in <filename unknown>:0
  at Mono.Xml.EntityResolvingXmlReader.Read () [0x00000] in <filename unknown>:0
  at Mono.Xml.DTDValidatingReader.ReadContent () [0x00000] in <filename unknown>:0
  at Mono.Xml.DTDValidatingReader.Read () [0x00000] in <filename unknown>:0
  at Mono.Xml.Schema.XsdValidatingReader.Read () [0x00000] in <filename unknown>:0
  at System.Xml.XmlValidatingReader.Read () [0x00000] in <filename unknown>:0
  at System.Xml.XmlDocument.ReadNodeCore (System.Xml.XmlReader reader) [0x00000] in <filename unknown>:0
  at System.Xml.XmlDocument.ReadNode (System.Xml.XmlReader reader) [0x00000] in <filename unknown>:0
  at System.Xml.XmlDocument.Load (System.Xml.XmlReader reader) [0x00000] in <filename unknown>:0
  at System.Xml.XmlDocument.Load (System.String filename) [0x00000] in <filename unknown>:0
  at Monodoc.RootTree.LoadTree (System.String basedir, Boolean includeExternal) [0x00000] in <filename unknown>:0
  at MonoDevelop.Projects.HelpService.InitializeHelpTree () [0x00000] in <filename unknown>:0

But then the server still prints
OmniSharp server is listening
Does anyone have an idea on how to fix this? Btw., I saw that there is OmnisharpRoslyn on its way, is that already usable?

The Monodoc error is safe to ignore.
Yeah, good progress is being made with omnisharp-rolsyn https://github.com/OmniSharp/omnisharp-roslyn/issues/29.
It's currently not compatible though because omnisharp-vim sends requests using form url-encoded, whereas omnisharp-roslyn currently only accepts json requests in the form body https://github.com/OmniSharp/omnisharp-roslyn/issues/27 .
Apart from that minor detail, it should work. I've since defected to the emacs camp though, so I don't have a great deal of motivation to look at this. Fancy taking a look at it?

Since I'm still quite a novice, it's probably gonna remain at a look :D. But maybe I can just get the normal server somehow working for me.

You can definitely get the original server working this way. Looks like you already have.
Don't expect to get completions for aspnet5 stuff though. It won't work.

 When I have a solution file, everything works fine, but when I start the server manually, it still tries to start the server on startup of vim, but fails due to missing solution file.
When i use
let g:Omnisharp_start_server = 0
I get a single error on startup:
Unable to find csc.exe
And this error only happens when I started the server manually, otherwise nothing happens. How could I solve that error?

OmniSharp doesn't reference csc.exe anywhere. What other plugins do you have installed?

I know this is old and you probably figured it out already but if you have vim-csharp plugin installed that will reference csc.exe for its C# compiler.

@markwoodhall Thanks for that tip. There is an [open issue])(https://github.com/OrangeT/vim-csharp/issues/31) in the vim-csharp plugin repo.


Consensus network

Description of the Change
Network interface for consensus
Benefits
Allows to start actual implementation and sync component

Please, add method PeerInfo getPeerInfo() const to PeerClient


Preserve options from @types/ws - missing origin

Would it be possible to add the same options as @types/ws, that way it's a simple "drop-in" for ws with the extra features of reconnecting.
import WebSocket from 'reconnecting-websocket';

const origin = "https://example.com";
const ws = new WebSocket("wss://<SERVER>", [], {origin});

Client Option types from @types/ws
    interface ClientOptions {
        protocol?: string;
        handshakeTimeout?: number;
        perMessageDeflate?: boolean | PerMessageDeflateOptions;
        localAddress?: string;
        protocolVersion?: number;
        headers?: { [key: string]: string };
        origin?: string;
        agent?: http.Agent;
        host?: string;
        family?: number;
        checkServerIdentity?(servername: string, cert: CertMeta): boolean;
        rejectUnauthorized?: boolean;
        passphrase?: string;
        ciphers?: string;
        cert?: CertMeta;
        key?: CertMeta;
        pfx?: string | Buffer;
        ca?: CertMeta;
        maxPayload?: number;
    }

This library is meant to be a "drop-in" replacement for WebSocket libraries that implement the WebSocket API Interface.
This library pretends to be compatible with WebSocket from browser, react native and any API compatible library. Many of the client options defined for ws are not aplicable.
If you want to use full ws capabilities, you can, simply create a new Class extending it:
import WS from `ws`;
import RWS from `reconnecting-websocket`;

...

class MyWebSocket extends WS {
    constructor(url, protocol) {
        super(/*pass here your ws options*/)
    }
}

// Then you can use reconnecting-websocket

const rws = new RWS("url", [], {WebSocket: MyWebSocket, etc});


[build performance] check if we can use jaotc or GraalVM to "AOT compile" r8.jar

Example: https://devblogs.microsoft.com/java/aot-compilation-in-hotspot-introduction/
We might have more success with this when we are on JDK 21.

Ok, this feature was removed in JDK 17:

https://docs.oracle.com/en/java/javase/17/migrate/removed-tools-and-components.html#GUID-D7936F0D-08A9-411E-AD2F-E14A38DA56A7


Restore the zoom to fit behaviour

#623 broke the zoom to fit all behavior with an empty document (creating a new finite document no longer centered the document in the user's view)

Good eye.


deis releases:destroy

Hello guys,
I deployed a medium-size DEIS environment, containing 7 CoreOS nodes with about 40 apps. I have 120GB x 7 ~ 1TB of storage and it's just not enough. So I decided to delete the old releases to free space (each of the apps has some), but there isn't such option.
I would like to have something like:
$ deis releases:destroy --app=myapp --release=v1
I have to say it seems to me rather weird to have my images 7 times around the cluster (replica in each node). If you think about large clusters (lets say 100 nodes) it is such a waste. what would happen if I share /var/lib/docker via nfs between all nodes in the cluster ? will it work?
Thanks,
Ido

I recommend looking at the techniques listed in #3675, which will help cleanup unused images.
I think a better cleanup solution is something that would be nice to have, because I agree you're unlikely to want to roll back to really old releases. Another proposed solution is to only save the last few releases, with older releases being deleted. Something that has been hampering this is that the docker image registry didn't have a delete option, so it's hard to implement a delete option in deis.
It's not nearly as wasteful in large clusters as you might think though. The image registry stores 3 replicas of the images. Then, the app is only downloaded onto containers that are running it, which in a large cluster is a small subset of machines.

Thank you for the comment, I'll try using the docker cleanup service mentioned there.
How about using a centralized storage and mounting /var/lib/docker of all nodes via nfs (same directory for all)? Do you have an experience with such architectures?

closing as a duplicate of #3698.


apptestai-test 0.1.0

New Pull Request Checklist

[x] I will not move an already shared step version's tag to another commit
[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[x] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x] I read and accept the Abandoned Step policy

Thank you for the new step @forcemax !
Happy Building! 🚀

@trapacska please merge.


Update course, fix typos and remove whitespace at end of lines

Update the SPARK course to take into account latest updates:

foward goto is supported
all access types are supported (including access-to-constant)
Ada.Text_IO spec is now in SPARK

 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.


Fixed test file name

Fixed test file name. PHPUnit tests should end with Test verbatim. The class name already did, so the file name was updated to match. This fixes an issue where targeting the tests directory, e.g. vendor/bin/phpunit tests did not find any tests to run.
Removed test bootstrap because it didn't do anything.
Added final to indicate inheritance of test case is not intended.
Changed visibility of member variable to private because class is final.
Changed visibility of overridden setUp method to match parent.
Changed Travis to use Composer invocation of test suite, since such a script is defined for this purpose already but not currently used. Doesn't work on PHP < 5.6.
Fixed code coverage generation.

This is just a first pass to fix the major test issues. More to follow if this PR is accepted.

Looks good!


doc-gen: migrate scalar functions (string) documentation 3/4

Which issue does this PR close?

Closes #13671 .
Rationale for this change

What changes are included in this PR?

As discussed in #13671  , this PR will migrate the parts of builtin string functions documentation that currently support migration.
Are these changes tested?

Are there any user-facing changes?

I merged it, and maybe other related PRs can automatically run CI , which was previously blocked by  “first-time contributor”.


Configure App Service to return a usable access token

I beleive there is a type on this section it states: to navigate to
In the left browser, click subscriptions > <your_subscription> > resourceGroups > myAuthResourceGroup > providers > Microsoft.Web > sites > <back_end_app_name> > config > authsettings.
but I think this should be
In the left browser, click subscriptions > <your_subscription> > resourceGroups > myAuthResourceGroup > providers > Microsoft.Web > sites > <front_end_app_name> > config > authsettings.
My example would not work until I made this adjustment.

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

ID: 8ff46628-b5dc-8b02-02b6-8f510d8059b0
Version Independent ID: 24d09b70-d144-d152-ac5a-c49219a5d8e8
Content: Authenticate and authorize users end-to-end in Azure App Service
Content Source: articles/app-service/app-service-web-tutorial-auth-aad.md
Service: app-service-web
GitHub Login: @cephalin
Microsoft Alias: cephalin

@lorindavis Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@lorindavis Thanks for reporting. The update is merged and will be published at next 10am PST. #please-close


[r] Timestamp write and read of data frame and array objects

Issue and/or context:
This PR adds the ability to write, and read, at given time stamps covering the key SOMA object data frame as well as sparse and dense arrays.
There is a corresponding follow-up PR #2925 which exposes time points (as before) as well as range (begin and end) as taken by the C++ API.
Changes:
Code changes as well as a new test file.  It also contains two additional smaller PRs (#2866, #2871).
Notes for Reviewer:
SC 52516
The branch had been open for two-plus weeks and has been rebased a few times so the final commit timestamps are not from the initial commits.

Thanks for #2929 and the follow-up comments here.


Update Podfile

npx pod-install fails because of the edited line in cocoapods.
This seems to have fixed it -> https://stackoverflow.com/questions/60880105/cocoapods-could-not-find-compatible-versions-for-pod-reactcommon-jscallinvoker

Closes #

Did #5 fixed this issue?


Reactstrap is completely unusable with Preact

Issue description

reactstrap version #6.2.0
import method es

What is happening?
On first import of Reactstrap it calls Polyfill from react-lifecycles-compat on TabContent, even if your code doesn't use TabContent.
In it's turn Polyfill obviously fails to work with Preact.Component which leads to termination of rendering and whole application is now one big white screen.
What should be happening?
Application should render normally using Preact.
Steps to reproduce issue

Create project with preact, preact-compat and reactstrap
Try to render it

Error message in console
Uncaught Error: Can only polyfill class components
    at polyfill (react-lifecycles-compat.es.js:54)
    at Object../node_modules/reactstrap/dist/reactstrap.es.js (reactstrap.es.js:5487)

Code
Separate demo project showcasing this error: https://github.com/dotterian/preactstrap-error
Possible solution
Rewrite TabContent using current React lifecycle, so there won't be any need in react-lifecycle-compat
Temporary solution
Check if TabContent is React Component before applying polyfill.
var prototype = TabContent.prototype;

if (prototype && prototype.isReactComponent) {
  polyfill(TabContent);
}

react-lifecycle-compat was added before of #1044. It allows reactstrap to better maintain backwards compatibility with react 16.
reactstrap doesn't really try to be compatible with preact and reactstrap doesn't have any tests to ensure it works with preact. That said, we do accept PRs to improve preact compatibility as long as it doesn't hinder usage with react.

Reactstrap seems to work fine on Preact X for me as long as I alias preact/compat mentioned in the docs: https://preactjs.com/guide/v10/switching-to-preact

I'm closing this since the the issue is no longer active and it appears it was answered above. Please comment and it will be opened if you feel the issue has not been resolved / the question has not been answered.

I wrote a script that can patch it in place for compatibility


Total number of PublishAsync & Respond from ApplicationMessageProcessed does not tally

Hi,
I'm using ManagedClient 2.8.5 and I found something weird.
FYI, I need to monitor every PublishAsync call to make sure all messages are delivered to the server. So I use the HasSucceeded or HasFailed property of ApplicationMessageProcessedEventArgs in the ApplicationMessageProcessed event to determine the status.
However, I found the total number of respond received (HasSuceeded and HasFailed) from ApplicationMessagedProcessed event will always more than the total number of PublishAsync call. Is it due to the logic behind of PublishAsync call will auto publish again when connection is disconnected or publish is failed? If the answer is yes, then I should able to get the total of HasSuceeded count same with the total of PublishAsync count. But when I monitor the HasSuceeded flag only, my system could not proceed due to web socket or client is disconnected while waiting for the HasSuceeded respond. May I know how do i ensure the connection can persist until last HasSuceeded respond to return?

Hi,
yes this is due to several retries. The client will stop trying after disconnect was detected but it can still happen that the event is raised a couple of times for a single message.
Your idea sounds good but requires some architecture changes in the managed client.
Best regards
Christian

Hi chkr1011,
Thanks for your reply.
Seems like this is a big change to managed client. Is there any plan for this change in the next release?

Maybe not next release but I can add it on the list.

Hi chkr1011,
No problem. Really hope to see this feature in the near future...
Thanks.

Is this already possible, @chkr1011?

Hi @SeppPenner
have you solve your problem? I tested the ApplicationMessageProcessedHandlerMethod, the HasSucceededis's value is also be true when msg published fail.
Thanks.

I'm not sure if this is already fixed in the latest version of the library or not @liaoliaoda

Hi @SeppPenner ,
It is working when the CleanSession value is false.

Please try with the latest version and reopen this ticket if the problem still occurs.


Add exhaust or more intense ventilation in the small boxes

After session is finished, I clearly smell the box. I fear there might be a buildup of odorant as the session progresses.
Further way of assessing it is using the PID and trigger the odor valves and equivalent amount of times while the door is closed #70

I think until we describe the problem better with the PID, I would close this issue. I will add a point for the olfactory calibration to perform a 1h calibration with continue odor presentation to see if there any substantial building or decrease in the box (this can also help us see if there is a decrease in headspace odor concentration)


homebrew macvim build failed on macOS 10.14 with Xcode 10 beta

Last 15 lines from /Users/kaito/Library/Logs/Homebrew/macvim/02.make:
	from /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/2.3.0/universal-darwin18/rbconfig.rb:214:in `'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/2.3.0/universal-darwin18/rbconfig.rb:7:in `'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/2.3.0/rubygems.rb:9:in `require'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/2.3.0/rubygems.rb:9:in `'
	from :4:in `require'
	from :4:in `'

** BUILD FAILED **

The following build commands failed:
	CompileC /tmp/macvim-20180611-46855-pbvqpw/src/MacVim/build/MacVim.build/Release/MacVim.build/Objects-normal/x86_64/MMPreferenceController.o /tmp/macvim-20180611-46855-pbvqpw/src/MacVim/MMPreferenceController.m normal x86_64 objective-c com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)
make[1]: *** [macvim] Error 65
make: *** [first] Error 2

Please, tell if official vim is compatible with Ruby 2.3 and can be compiled on macOS 10.14 with XCode 10 beta. If not, please report to the upstream.

Looks like vim can be compiled.

==> Upgrading vim
==> Downloading https://github.com/vim/vim/archive/v8.1.0050.tar.gz
==> Downloading from https://codeload.github.com/vim/vim/tar.gz/v8.1.0050
######################################################################## 100.0%
==> ./configure --prefix=/usr/local --mandir=/usr/local/Cellar/vim/8.1.0050/shar
==> make
==> make install prefix=/usr/local/Cellar/vim/8.1.0050 STRIP=/usr/bin/true
==> Not running post_install as we're building a bottle
You can run it manually using `brew postinstall vim`
🍺  /usr/local/Cellar/vim/8.1.0050: 1,432 files, 23.3MB, built in 11 minutes 33 seconds


update the column name in pipeline page

What is version of KubeSphere DevOps has the issue?
v3.3.2
How did you install the Kubernetes? Or what is the Kubernetes distribution?
No response
Describe this feature

[任务状态] -> [流水线配置]
Additional information
No response

/close


Bug: Getting PHP 8.4.0 on windows

Describe the bug
We're seeing Windows CI jobs only get PHP 8.4.0, even though 8.4.1 is available (and was 8.4.0 actually a release?!). Even when forcing php-version: 8.4.1 I'm seeing 8.4.0:

Version

[X] v2
[ ] v1

Runners
GitHub Hosted
Operating systems
Windows-latest
PHP versions
PHP 8.4 dev
To Reproduce
      - name: Setup PHP
        uses: shivammathur/setup-php@v2
        with:
          php-version: '8.4.1'
        env:
          debug: true

Expected behavior
PHP 8.4.1 is installed
Screenshots/Logs
No response
Additional context
No response
Are you willing to submit a PR?
No

Fixed in https://github.com/shivammathur/php-builder-windows/commit/3452b4e1d93773b6fc4dc20d04f53095f38347f1

Seems to be working. Thank you!

Thanks @shivammathur !


[WIP]  Add socket read timeout implementation

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, releated PRs, TODO items or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #15261 into master will increase coverage by 11.8%.
The diff coverage is 79.1%.

@@             Coverage Diff              @@
##             master   #15261      +/-   ##
============================================
+ Coverage     23.24%   35.05%   +11.8%
+ Complexity     5065      176    -4889
============================================
  Files          1342       66    -1276
  Lines         67457     2077   -65380
  Branches       9193      261    -8932
============================================
- Hits          15680      728   -14952
+ Misses        50039     1296   -48743
+ Partials       1738       53    -1685

Impacted Files
Coverage Δ
Complexity Δ

...nalang/stdlib/socket/endpoint/tcp/client/Read.java
100% <100%> (ø)
7 <0> (ø)
:arrow_down:

...ng/stdlib/socket/endpoint/tcp/server/Register.java
100% <100%> (ø)
4 <0> (ø)
:arrow_down:

...stdlib/socket/endpoint/udp/client/ReceiveFrom.java
77.77% <100%> (+4.44%)
5 <0> (ø)
:arrow_down:

...nalang/stdlib/socket/tcp/ReadPendingSocketMap.java
100% <100%> (ø)
6 <2> (ø)
:arrow_down:

...ballerinalang/stdlib/socket/tcp/SocketService.java
76.92% <100%> (+4.19%)
4 <1> (+1)
:arrow_up:

...llerinalang/stdlib/socket/tcp/SelectorManager.java
76.76% <100%> (+0.49%)
50 <0> (ø)
:arrow_down:

...g/ballerinalang/stdlib/socket/tcp/SocketUtils.java
76.36% <50%> (-2.07%)
10 <1> (+1)

.../stdlib/socket/endpoint/tcp/server/InitServer.java
62.96% <60%> (-0.68%)
3 <0> (ø)

...tdlib/socket/endpoint/udp/client/InitEndpoint.java
54.34% <60%> (-0.42%)
6 <0> (ø)

...tdlib/socket/endpoint/tcp/client/InitEndpoint.java
64.51% <60%> (-2.16%)
3 <0> (ø)

... and 1277 more

Continue to review full report at Codecov.

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


File API uses 32 bit integers

Godot version:
3.2 Beta 2
OS/device including version:
Issue description:
Godot.File methods such as get_len() and get_buffer(int) return/take 32 bit signed integers, meaning over ~2GB files cannot be read
Steps to reproduce:
Minimal reproduction project:
Related to #288

There's PR to address this #27247

What kind of game asset is over 2GB?

What kind of game asset is over 2GB?

That's not really relevant to the issue, but anyway:
$ du -sh projects/tps-demo/level/geometry/demolevel.dae
322M   projects/tps-demo/level/geometry/demolevel.da

That's just for the very small TPS demo level. Not 2GB, but not that far from this order of magnitude.
And more importantly, .pck packages can definitely be over 2GB: #27168.

Well, for your tetris you don't need this, I'm sure. But "not all games need this" banner is a bit rugged now, don't you think?
In general why set limits when you don't have to?
Also limits like this hit in unusual cases, when, for example you use Godot as a tool to prepare your game assets (you have a city in blend file, you export it and then generate all lods/annotation/paths, split it into chunks for loading in Godot as part of import process. This way you don't need to patch engine too much to implement tooling required to do your job. If you need to patch engine for everything, then what remains?
But I wonder that even this small limitaton relaxation has its resistance, imagine how hard it s to "sell" new features to people who don't want anything.

Closing as superseded by https://github.com/godotengine/godot-proposals/issues/400 to keep everything in one place.


Unable to page throught Docs API V2 with resume mode NEXT_ROW

Working on the paging of the sub-documents in the Docs API V2, seems that paging state returned by a row can not be used to continue paging from that state.
So far I was able to investigate a bit about the paging states that are being sent. That's with C*4.
This is the state that we sent back to the bridge, which is not working (note that WWFLIFErlKIQNqPq is document id):
value: "\020WWFLIFErlKIQNqPq,\377UUUUUUU\000\bproducts\004food\006Orange\004name\377UUUUUUUU\360\177\377\377\377\360\177\377\377\377"

When doing the same query with the page size and page with the result set paging state I get back following states. They do work correctly:
value: "\000+\377UUUUUUU\000\bproducts\004food\005Apple\004name\377UUUUUUUU\360\177\377\377\376\360\177\377\377\376"
value: "\000,\377UUUUUUU\000\bproducts\004food\005Apple\005price\377UUUUUUUU\360\177\377\377\375\360\177\377\377\375"
value: "\000*\377UUUUUUU\000\bproducts\004food\005Apple\003sku\377UUUUUUUU\360\177\377\377\374\360\177\377\377\374"
value: "\0002\377UUUUUUT\000\bproducts\004food\006Orange\004info\005price\377UUUUUUUU\360\177\377\377\373\360\177\377\377\373"
value: "\000,\377UUUUUUU\000\bproducts\004food\006Orange\004name\377UUUUUUUU\360\177\377\377\372\360\177\377\377\372"
value: "\000*\377UUUUUUU\000\bproducts\004food\004Pear\004name\377UUUUUUUU\360\177\377\377\371\360\177\377\377\371"

@mpenick does something rings a bell here?

False alarm, my bad.


Create basket with transparent holes.

I was not able to makes holes transparent in the SVG.
Therefore, I used Gimp to modify the existing PNG.
Resolves #1.

I will take another shot at also doing this for the svg as some devices need really high resolution icons and it would be great to have this as svg as well.


feat(ProjectInfo): fire function for 'edit project'

This PR makes a small update to the ProjectInfo section: "Projekt bearbeiten" now fires a function instead of linking to another route.

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


Add ability to truncate raw partition after compaction

Is your feature request related to a problem? Please describe.
When using an event stream as a "table", Create/Update/Delete options are modelled as posting new key/value pairs.  The stream needs an infinite retention policy, because some keys may never get new values.
Pulsar already has a Compaction feature, which converts a raw topic into a compacted topic, containing only the latest value for each key.  Clients can choose to read the compacted stream.  However the raw topic remains indefinitely on disk.
This means that the raw topic can grow without bounds, even for a limited size table, as the entire history of updates is kept indefinitely.
Describe the solution you'd like
Add the ability to truncate the raw topic data after compaction.  This might mean rotating the compacted ledger into the place of the original raw ledger.
Since compaction is done on demand, this rotation can be done on demand too.  It could be an attribute of the compaction request.
(Aside: I don't fully understand what happens when compaction is run on an already-compacted topic; and I don't know what happens if a client asks to read the compacted version of a topic which has not yet been compacted)
Describe alternatives you've considered
As far as I can see, the only other option is to spool all the data out of the compacted topic into a fresh topic.
Additional context
This would bring feature parity to Kafka's log compaction.
Possibly related to #2736

Closed as stale and no one seems to work on it. Please create a new issue if it's still relevant to the maintained versions.


Require issue template

This will require all issues to be created using one of the templates.  Currently there are two issue templates (bug for problems with an existing test and enhancement which is a request to add to a test or create a new test)
This only applies to issues, not pull requests

Pull Request Functional Test Report for #2036 / dada3901e00b95b187bf6a331e831e1de5a3a423
No tests identified for validation.
Help


Redirect Logging to ELK or FileSystem etc.,

Hi @StrongMonkey,
How to redirect the logs of the application to ElasticSearch/Filesystem or any other option that fluentd supports.How to customize it ?
Thanks & regards,
Hari

@mkharibalaji At this point Rio doesn't really manage logging for you. So you might have to look at documentation on how to setup ElasticSearch/Fluentd in Kubernetes.

Thanks a lot @StrongMonkey


[FR] Click to add node in web UI

Was unsure if this was existing expected behaviour or not, but after https://github.com/breadboard-ai/breadboard/pull/1085 I've confirmed. It would be very nice to be able to just "click to add" on the items as well as being able to drag them in.
Also reminded me that https://github.com/breadboard-ai/breadboard/issues/897 is now resolved. But it would be very nice to also be able to pin the toolbox open.

I made it so you can double click. It currently adds it at [0, 0] in the graph because I can't think of a better location for a new node 😄 – hope that helps!


chore(pkglint): Add build:test and build:test:package scripts

Adds a build:test and build:test:package script to all packages so that they can be easily be built and tested (and optionally packaged) in topological order using a single command (lerna run build:test or lerna run build:test:package).
This allows failing faster, as packages not passing unit tests will not even be built.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.

Here's bikeshedding to the max, but can we call it build+test? For some reason that appeals to me.
I don't particularly like the : symbol, it seems to imply a scope for me.

I'm not against that. And it's easy enough to change. Let me run a quick test to verify that it doesn't somehow break NpmPrettyMuch or the shell or whatnot, and if not, then that's cool by me!


Running the playbook on the dev branch fails at Create NDSs step with minimal group_vars config

Possibly having incorrect variable definitions, but I'm not sure where the error is based on following the README.
I'm running the playbook off of the dev branch and reading the README file.  Under the install instructions, step 2, I'm going with the minimal vars file....

I have a 3 node cluster

master
worker1
worker2

There's only 2 disks on workers, there's no disks on the masters
I have created this group_vars/all file:
# cat group_vars/all
scale_storage:
  - filesystem: gpfs01
    overwriteNSDs: true
    disks:
      - device: /dev/vdb
        servers: autogen-centos76-x-worker1,autogen-centos76-x-worker2
      - device: /dev/vdc
        servers: autogen-centos76-x-worker1,autogen-centos76-x-worker2

And running the playbook, it fails at "create NSDs"
TASK [core/cluster : storage | Create new NSDs] *************************************************************
failed: [autogen-centos76-x-master] (item={u'changed': True, u'uid': 0, u'dest': u'/var/tmp/StanzaFile.new.gpfs01', u'owner': u'root', 'diff': [], u'size': 426, u'src': u'/root/.ansible/tmp/ansible-tmp-1584925465.15-242076789828031/source', 'ansible_loop_var': u'item', u'group': u'root', 'item': u'gpfs01', u'checksum': u'4aa55cdfd89693a224d1f0218519f4e5f3c6d91d', u'md5sum': u'1efddd5f6b9317038f51ae5af5ecfc10', 'failed': False, u'state': u'file', u'gid': 0, u'mode': u'0644', u'invocation': {u'module_args': {u'directory_mode': None, u'force': True, u'remote_src': None, u'dest': u'/var/tmp/StanzaFile.new.gpfs01', u'selevel': None, u'_original_basename': u'StanzaFile.j2', u'delimiter': None, u'regexp': None, u'owner': None, u'follow': False, u'validate': None, u'local_follow': None, u'src': u'/root/.ansible/tmp/ansible-tmp-1584925465.15-242076789828031/source', u'group': None, u'unsafe_writes': None, u'checksum': u'4aa55cdfd89693a224d1f0218519f4e5f3c6d91d', u'seuser': None, u'serole': None, u'content': None, u'setype': None, u'mode': None, u'attributes': None, u'backup': False}}}) => {"ansible_loop_var": "item", "changed": true, "cmd": ["/usr/lpp/mmfs/bin/mmcrnsd", "-F", "/var/tmp/StanzaFile.new.gpfs01", "-v", "no"], "delta": "0:00:00.656419", "end": "2020-03-22 18:04:29.807511", "item": {"ansible_loop_var": "item", "changed": true, "checksum": "4aa55cdfd89693a224d1f0218519f4e5f3c6d91d", "dest": "/var/tmp/StanzaFile.new.gpfs01", "diff": [], "failed": false, "gid": 0, "group": "root", "invocation": {"module_args": {"_original_basename": "StanzaFile.j2", "attributes": null, "backup": false, "checksum": "4aa55cdfd89693a224d1f0218519f4e5f3c6d91d", "content": null, "delimiter": null, "dest": "/var/tmp/StanzaFile.new.gpfs01", "directory_mode": null, "follow": false, "force": true, "group": null, "local_follow": null, "mode": null, "owner": null, "regexp": null, "remote_src": null, "selevel": null, "serole": null, "setype": null, "seuser": null, "src": "/root/.ansible/tmp/ansible-tmp-1584925465.15-242076789828031/source", "unsafe_writes": null, "validate": null}}, "item": "gpfs01", "md5sum": "1efddd5f6b9317038f51ae5af5ecfc10", "mode": "0644", "owner": "root", "size": 426, "src": "/root/.ansible/tmp/ansible-tmp-1584925465.15-242076789828031/source", "state": "file", "uid": 0}, "msg": "non-zero return code", "rc": 1, "start": "2020-03-22 18:04:29.151092", "stderr": "mmcrnsd: Name \"nsd_autogen_centos76_x_worker1,autogen_centos76_x_worker2_vdb\" is not allowed.\nIt contains the following invalid special character:  ,\nmmcrnsd: Error found while processing stanza\n    %nsd:\n      device=/dev/vdb\n      nsd=nsd_autogen_centos76_x_worker1,autogen_centos76_x_worker2_vdb\n      servers=autogen-centos76-x-worker1,autogen-centos76-x-worker2\n      usage=dataAndMetadata\n      failureGroup=-1\n      pool=system\nmmcrnsd: Name \"nsd_autogen_centos76_x_worker1,autogen_centos76_x_worker2_sdc\" is not allowed.\nIt contains the following invalid special character:  ,\nmmcrnsd: Error found while processing stanza\n    %nsd:\n      device=/dev/sdc\n      nsd=nsd_autogen_centos76_x_worker1,autogen_centos76_x_worker2_sdc\n      servers=autogen-centos76-x-worker1,autogen-centos76-x-worker2\n      usage=dataAndMetadata\n      failureGroup=-1\n      pool=system\nmmcrnsd: File /var/tmp/StanzaFile.new.gpfs01 does not contain any NSD descriptors or stanzas.\nmmcrnsd: Command failed. Examine previous error messages to determine cause.", "stderr_lines": ["mmcrnsd: Name \"nsd_autogen_centos76_x_worker1,autogen_centos76_x_worker2_vdb\" is not allowed.", "It contains the following invalid special character:  ,", "mmcrnsd: Error found while processing stanza", "    %nsd:", "      device=/dev/vdb", "      nsd=nsd_autogen_centos76_x_worker1,autogen_centos76_x_worker2_vdb", "      servers=autogen-centos76-x-worker1,autogen-centos76-x-worker2", "      usage=dataAndMetadata", "      failureGroup=-1", "      pool=system", "mmcrnsd: Name \"nsd_autogen_centos76_x_worker1,autogen_centos76_x_worker2_sdc\" is not allowed.", "It contains the following invalid special character:  ,", "mmcrnsd: Error found while processing stanza", "    %nsd:", "      device=/dev/sdc", "      nsd=nsd_autogen_centos76_x_worker1,autogen_centos76_x_worker2_sdc", "      servers=autogen-centos76-x-worker1,autogen-centos76-x-worker2", "      usage=dataAndMetadata", "      failureGroup=-1", "      pool=system", "mmcrnsd: File /var/tmp/StanzaFile.new.gpfs01 does not contain any NSD descriptors or stanzas.", "mmcrnsd: Command failed. Examine previous error messages to determine cause."], "stdout": "", "stdout_lines": []}

For better readability, I ran the command:
# /usr/lpp/mmfs/bin/mmcrnsd -F /var/tmp/StanzaFile.new.gpfs01 -v no
mmcrnsd: Name "nsd_autogen_centos76_x_worker1,autogen_centos76_x_worker2_vdb" is not allowed.
It contains the following invalid special character:  ,
mmcrnsd: Error found while processing stanza
    %nsd:
      device=/dev/vdb
      nsd=nsd_autogen_centos76_x_worker1,autogen_centos76_x_worker2_vdb
      servers=autogen-centos76-x-worker1,autogen-centos76-x-worker2
      usage=dataAndMetadata
      failureGroup=-1
      pool=system
mmcrnsd: Name "nsd_autogen_centos76_x_worker1,autogen_centos76_x_worker2_vdc" is not allowed.
It contains the following invalid special character:  ,
mmcrnsd: Error found while processing stanza
    %nsd:
      device=/dev/vdc
      nsd=nsd_autogen_centos76_x_worker1,autogen_centos76_x_worker2_vdc
      servers=autogen-centos76-x-worker1,autogen-centos76-x-worker2
      usage=dataAndMetadata
      failureGroup=-1
      pool=system

It's complaining about the comma and looks like the NSD generated names is not quite right....
nsd=nsd_autogen_centos76_x_worker1,autogen_centos76_x_worker2_sdc

Any thoughts here?

@whowutwut Looks like this is an defect with default nsd names  if nsd disks is serving more than one nsd servers . i was thinking to write a logic in ansible to generate auto increament nsd names ( nsd_1 , nsd_2 etc) just like we have in the toolkit but In Ansible its little difficult to add this logic . I will try to fix this issue.

@rajan-mis  is my group_vars file content correct?

@whowutwut Yes your group_vars file content is correct.

@whowutwut Can we close this issue.


Add

@11830069 The website of the university now seems to be broken and impossible to browse. I'll get back to this request as soon as the website is operational. I cannot review the eligibility of the request without browsing the website. Thank you.

@philipto The school’s official website address is https://www.mtu.ac.kr/
Email service provided by Google, use @mtu.kr

@philipto The school’s official website address is https://www.mtu.ac.kr/
Email service provided by Google, use @mtu.kr or @mtu.ac.kr

@11830069 Thank you for the clarification. I am sorry, the screenshot you have attached does not look clear enough for me.
Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain mtu.kr that you are submitting as an official email domain for the students.

You can provide us with any of the following as the proof:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.


update pod

Hi, could update Cocoapods v3.2?

We tried this go around but got a cryptic error.   We'll try with 3.2.1 as well to see if it has changed.

Going to see if this helps http://codeslingers.co.uk/2014/05/16/why-your-podspec-is-failing/

OK. Thanks Aure

If anyone has any clues for me, the full output of the pod spec lint is here:
https://gist.github.com/aure/49e4f8d5c4b13252608c26d60308d74d
Everything goes well and then at the end..
** BUILD SUCCEEDED **
-> AudioKit (3.2)
- ERROR | [OSX] xcodebuild: Returned an unsuccessful exit code.

Hi, I have tried pod lib lint to testing it. Get this error below:
-> AudioKit (3.2)
- ERROR | file patterns: The source_files pattern did not match any file.

that error I got past by specifying source_files, but still no.

We're very close to fixing this I believe.

Could early of Sep release AudioKit v3.2.1? I noticed v3.2 that's not very stable.

Ferrick, AudioKit is free and open-source written and maintained by volunteers.  I was willing to keep this issue open for you until resolution, but not now that you are suggesting a schedule for us and making statements on stability.

@aure  I'm so sorry about that, just willing to know because I got project deadline. Else I can start to try another way to archive my project. Hope u can understand my situation.

I am back from the conference and will be working on AudioKit a lot this week so hopefully everything gets done soon.  Thanks!

Updated cocoapods to 3.3 on iOS and tvOS.  Something still weird on macOS with Cocoapods.


Controller as support?

Angular agility is sad when I try to use Controller as http://plnkr.co/edit/6ikYLTrV5wqgkBaqCGK5?p=preview
`

<input type="email" aa-field-group="person.email" required/>

    <button aa-submit-form="save()" class="btn btn-default">Save</button>

`
And the error
Error: [ng:areq] Argument 'main' is not a function, got undefined
The offending line is
line 1136: return $delegate.apply(this, arguments);

@JasonMore
Shouldn't it be

instead of your

?

Yeah, he's got it reversed

I also can't get the module to understand my form name attribute pointing to my controller with the "controller as" syntax (e.g. <form name="$ctrl.form"> instead of just <form name="form">) so that I can access the FormController as form on my controller directly rather than hacking around it e.g. by having to inject $scope and use $scope.form or use the $postLink lifecycle hook on my component to assign ctrl.form = $scope.form.

Yeah this bit was written before controller as was even a thing. Would be happy to accept a PR if someone can fix it!


Add a startup policy API to indicate that the workers are created only after the leader is ready

What would you like to be added:
Startup policy API to delay creating the workers sts until the leader is ready
Why is this needed:
some frameworks require the leader to start (like ray head) before the workers are started so they can connect to it.

I'd like to have a try. Here are some thoughts about this feature.

First, let's define this startup policy as WaitForLeaderReady.
lws controller will set worker sts replicas=0 before all leader sts pod are ready.
Pod controller will watch the leader sts status(status.readyReplicas == status.replicas (or spec.replicas)) and scale worker sts replicas to actual value after all leader sts pod are ready.

Make sense.

@nayihz I think it worth a simple KEP since we're adding a new API which also helps to organize the logics in advance.

@nayihz I think it worth a simple KEP since we're adding a new API which also helps to organize the logics in advance.

OK.
/assign

this is nice! thanks @ahg-g


don't re-install dependencies after cache restore

I am doing dependency caching the manual way and just found out about this package.
it seems to be a time saver <3. thanks for this great package.
I did some initial tests and it seems to work like expected. I noticed though, that the composer installation process is still triggered on a cache hit.
I learned today in a different github action, that invoking composer install after a successfull restore is unnecessary.
wdyt.. could we save this call and the few seconds this requires when running the action on a cache hit?

This action doesn't cache the vendor/ directory but, rather, the downloaded packages. composer install is still necessary to unpack the cached packages into the project's vendor/ directory and create the autoloader.
If you'll note here: https://github.com/shivammathur/setup-php#cache-composer-dependencies

Please do not cache vendor directory using action/cache as that will have side effects.

This action follows that same advice, since it uses action/cache internally.

Does anyone know what those side-effects are? I'm caching the vendor folder on CircleCI for years without issues. Caching the vendor folder would remove ~10 seconds from my build which is now spent unpacking archives although nothing changes.

Perhaps @shivammathur can offer some advice on this?

Caching vendor is not recommended as per this discussion https://github.com/travis-ci/travis-ci/issues/4579.
My reasons for not recommending caching vendor in GitHub Actions are these.

Not all projects commit composer.lock (mostly libraries). If a project only has composer.json and uses its hash in the key, then caching vendor can result in it being stuck on old dependencies.

If composer.lock is committed and it changes, the restore-keys still might restore an old cache as those are fallback keys. If restore keys input is removed, the whole cache is lost as soon as one dependency is updated. So, while some seconds are saved caching vendor when there is a cache hit, the frequency of cache hits overall will decrease.

So, I prefer caching composer internal directory with both keys and restore keys (as it is done currently), and let composer figure out if it wants to use the restored cache for each dependency. This might take some more seconds to extract archives, but the frequency of cache hit in jobs would be better.


PR: Fix failing test due to a new version of nbconvert

Hello @ccordoba12! Thanks for submitting the PR.

There are no PEP8 issues in the file spyder/app/tests/test_mainwindow.py !

Thanks @ccordoba12 !


Add many Helpers.

Current coverage is 15.05% (diff: 0.00%)

Merging #48 into master will decrease coverage by 1.82%

@@             master        #48   diff @@
==========================================
  Files            37         43     +6
  Lines           960       1063   +103
  Methods         914       1015   +101
  Messages          0          0
  Branches         46         48     +2
==========================================
- Hits            162        160     -2
- Misses          798        903   +105
  Partials          0          0

Powered by Codecov. Last update 1c3398e...3411c03


chore: Release 0.7.0

:robot: I have created a release beep boop
0.7.0 (2023-07-08)
⚠ BREAKING CHANGES

bberg: add secp256r1 builtin to barretenberg (#223)

Features

bberg: add secp256r1 builtin to barretenberg (#223) (ceb4770)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/noir-lang/acvm-backend-barretenberg/releases/tag/v0.7.0 :sunflower:


Support BAM as input

Would be nice if Graphmap would support read input from BAM files, with the option to retain existing tags like insertion and deletion qualities for example. Also, the new SMRT Portal / Pacbio versions will produce BAM as native sequencing output apparently.

The latest release (v0.3.0) supports reading sequences from SAM and GFA files. Hope this will do for now? :-)
Ivan

Where's the thumbs up icon?


Null passed as arguments to controller method parameters when no InputFormatter matches

if the request's content-type is missing or not matching any InputFormatter then controller methods are passed null as arguments where data need to be deserialized. In the example below 'employee' is then null when the method is called. Instead the code 415 should be returned and the controller method should NOT be called.
public IActionResult UpdateEmployee(uint employeerId, [FromBody] Employee employee)
P.S:
A workaround is to create a middleware the code below to force the error it is not very pretty and breaks if other content types than json is added because I am not aware how to check properly any of the registered InputFormatter's match rather then hardcoding it.
       async Task Invoke(HttpContext context)
        {
          string method = context.Request.Method;
          if (method == "PUT" || method == "POST" || method == "PATCH")
          {
                RequestHeaders typedHeaders = context.Request.GetTypedHeaders();
                MediaTypeHeaderValue contentType = typedHeaders.ContentType;
                if (contentType==null || contentType.SubType!=@"json")
                {
                    context.Response.StatusCode = Status415UnsupportedMediaType;
                    return;
                }
          }

          await next.Invoke(context);
        }

If you want this behavior add [Consumes("application/json", "text/json")]. You can add this to the list of  global filters if you want to configure it application-wide.
services.AddMvc(options => options.Filters.Add(new ConsumesAttribute(...)));

@rynowak I can not confirm this. I tried adding the consumes attribute to the base controller for all my controllers and it does not change the behavior. I still get null passed to the employee argument if content-type is missing for my PUT request to the update method UpdateEmployee.

@bluemmc - this is definitely at odds with how the feature was intended to work - https://github.com/aspnet/Mvc/issues/1782
It looks like this code has remained mostly unchanged since it was added aspnet/Mvc@60fa4a6f
I'll dig in and see what I can figure out.

This case, and the surrounding comment just don't really fit the intent of the feature. I think we've had this bug for a while because no one was paying attention to what happens when you don't have a content type!
https://github.com/aspnet/Mvc/commit/60fa4a6f#diff-69de547dc02178bc5eb004e33185ce7cR78
I'd suggest the following changes to the handling when there's no ContentType specified in the body:

If another action exists without consumes, call it
If no other action exists (your case) - this should be a 415
If multiple other actions exist (overloaded on [Consumes(...)] - this should be a 415

Thanks for looking at this. I think behavior should depend on missing/wrong content-type AND if there are arguments that needs serialization or not. If there is no content content-type is not  required. if There is content-type should be required.

Based on ghost's comment:

Thanks for looking at this. I think behavior should depend on missing/wrong content-type AND if there are arguments that needs serialization or not. If there is no content content-type is not required. if There is content-type should be required.

This behaviour is not currently honoured in Mvc v2.0.2.
calling a POST method with a null body and null Content-Type is also returning 415 status code. It sounds like the intended behaviour is to allow null input with no content-type specified. The parameter in the controller action should then be null.

The behavior is to return 415 when you have an argument that would need to be deserialized, Independent of whether you sent content type or not, and to let requests without content type go through when they aren’t deserializing arguments from the body.

So in my example of:
Controller action has a [FromBody] argument of type object, and [HttpPost]
annotation
POSTing to that action's route with an empty body, and no Content-Type
header.
Status code 415 is or is not expected in this case? I would argue that we
should expect the action to be hit with an argument of default(T). What we
actually observe is 415 returned at model binding time (i.e. the action is
not hit).
On Wed, Feb 21, 2018 at 7:29 AM, Javier Calvarro Nelson <
<EMAIL_ADDRESS>wrote:

The behavior is to return 415 when you have an argument that would need to
be deserialized, Independent of whether you sent content type or not, and
to let requests without content type go through when they aren’t
deserializing arguments from the body.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aspnet/Mvc/issues/3874#issuecomment-367363713, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACfI6KMaMRJyJtqtGTB8aXJ8N1FXgHhMks5tXDZsgaJpZM4G_h0I
.

as per HTTP/1.1 spec: https://www.w3.org/Protocols/rfc2616/rfc2616-sec7.html#sec7.2.1

Any HTTP/1.1 message containing an entity-body SHOULD include a Content-Type header field defining the media type of that body. If and only if the media type is not given by a Content-Type field, the recipient MAY attempt to guess the media type via inspection of its content and/or the name extension(s) of the URI used to identify the resource. If the media type remains unknown, the recipient SHOULD treat it as type "application/octet-stream".

[emphasis mine]
Based on this deinition, ASP.NET Core is not going against a "must"/"must not" portion of the spec. However the bolded section indicates that there should be a default value.
So in the request received pipeline - empty content-type should be coerced into "application/octet-stream" and then model binding should be performed on the null data, with the null object passed into the controller action method. The action should handle null data input.

If the media type remains unknown, the recipient SHOULD treat it as type "application/octet-stream"
Old Web API used to have that behavior and it was a frequent source of issues for customers. The current behavior, while not "fully compliant" is less surprising to our customers and results in a better experience.

The result in the general case would be the same as we don't have any formatter that handles application/octect-stream and there is no useful way the framework can provide a mapping between application/octect-stream and a given type.
If you want to have this behavior you can just write your own input formatter and plug it in.

If you want to have this behavior you can just write your own input formatter and plug it in.

This is what I'd already ended up doing via a ResourceFilter: https://gist.github.com/fiksen99/61cef3d1e9af8830e12933e70a2437af
However it still feels like this behaviour is broken. Why should the existence of Content-Type matter when the body of a request is null? Regardless of Content-Type, we should be able to map this to a null/Default(T), and have my controller handle the request, instead of failing at model biding time.


RHODS-13084 update links for new Pantheon titles

Update to links to match new Pantheon titles, using attributes where possible

lgtm


Loosen dependencies versions to allow for ‘MINOR’ updates

This eases installation in apps and avoids users having to set overrides.
Fixes jesseshieh/master_proxy#5
@jesseshieh, I updated other dependencies' version specifications as well as plug_cowboy's. Please check you're happy with this.

Thanks!

Published in v0.1.3


CognitoUser.prototype.setUserMfaPreference is missing from typescript index

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Method is missing.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. Your bug will get fixed much faster if we can run your code and it doesn't have dependencies other than AWS Amplify.
What is the expected behavior?
The function should be exposed
Which versions of Amplify, and which browser / OS are affected by this issue? Did this work in previous versions?
2.0.3
You can turn on the debug mode to provide more info for us by setting window.LOG_LEVEL = 'DEBUG'; in your app.

Hi @HopScotch47, I have sent a PR about this fix to Amplify team. They will review it and merge it soon! Thanks.

Hello, I have seen that this was merged - thank you.
When will a new version be available on npm?

After testing with version 2.0.5 pushed to NPM yesterday
the function setUserMfaPreference still expects string[] parameters while it should take parameters of type any like that described in the documentation :
totpMfaSettings = {
PreferredMfa : true,
Enabled : true
};

@powerful23 will merge in the fix very soon.

Is there any updates to a fix for this - I see that in the 2.0.14 unstable releases still expect a string[]

@whiskerch sorry about that, will fix it soon.


Fixing #133

How should I send PRs for other issues fixed?
Should I create a separate branch for them?

No need to create a specific branch for a small fix. Just send a PR from the dev branch.
BTW do one different PR for each fix.

Sorry to bother, but that means at a time I can send a single PR for small fixes right?

One PR for each fix yes.


DOTA_COMBATLOG_PICKUP_RUNE never firing

When running several demo's trough the combatLogger it never gets the DOTA_COMBATLOG_PICKUP_RUNE. Is this not implemented yet?

DOTA_COMBATLOG_PICKUP_RUNE is pretty old, so if it is not firing, there are no such events in the replay. I don't know if older replays maybe do contain the data, but I supect that would be of no help, anyway.


Automated tests in GitHub Actions

Adds GitHub Actions job to run module tests.

Just one question why are we not using our php image with official nginx images?


Too many requests, sleeping for %d seconds 1m0s

I've noticed this error sometimes even when running my application using a single account and looking up a single profile but this error typically occurs during the insta.Login() process of the app. I know this isn't an error on my end because of how it's logged. Curious if you know what this could be from

I would need a bit more details. There are a few requests made during login that are expected to generate this response, also in the Android app. I would need to know whether this response is generated by one of those endpoints or not

instagram_1            |   "body": "{\"message\":\"Please wait a few minutes before you try again.\",\"status\":\"fail\"}",
instagram_1            |   "endpoint": "fbsearch/nullstate_dynamic_sections/",
instagram_1            |   "status": 429
instagram_1            | }
instagram_1            | Too many requests, sleeping for %d seconds 1m0s

Here is the error with the error following after. Would using proxies help mitigate this issue ?

Is it actually erroring out? It should get ignored, this endpoint is not of importance "Non fatal error while setting search null state"

Well the issue isn't that , the problem is the Too many requests, sleeping for x amount of seconds causes the app to stop sending additional request so if I am want to send a following request shortly there after nothing happens.


Design/information bar

The pr introduces the design for the InformationBar component and the LightningNode component which are going to be displayed in the federation admin screen.
Pictorial representations below with dummy data:

InformationBar Component :

LightningNode Component

link to design

Corrections made @wbobeirne please review

@IgboPharaoh, it'd be great to introduce the information bar UX specifically as part of implementing #63
Looking at the changes proposed here,

you have the UX for message bars, so there's two more tasks relevant to the specific guardian experience
the lightning node component can be introduced later, when working on #64

@IgboPharaoh, it'd be great to introduce the information bar UX specifically as part of implementing #63
Looking at the changes proposed here,

you have the UX for message bars, so there's two more tasks relevant to the specific guardian experience
the lightning node component can be introduced later, when working on Guardian UI: Show information about connected lightning nodes / gateway #64

I believe the best course of action is to start working on #63 and pull the necessary changes from this PR

Closed with #63


[all hosts] (Publishing) Guidance around add-ins in government cloud environments

For developers looking to publish in GCC and other Soverign Cloud environments, there are special considerations to make. This document outlines them.

Learn Build status updates of commit 600c862:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/publish/government-cloud-guidance.md
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

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

Learn Build status updates of commit 65febb5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/publish/government-cloud-guidance.md
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

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

Learn Build status updates of commit ea44271:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/publish/government-cloud-guidance.md
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

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

Learn Build status updates of commit 672dcde:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/publish/government-cloud-guidance.md
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

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


Sound null safety disabled in flutter web release build

It appears that sound null safety is disabled during a web release build. This was unexpected and caused a bug in my application when jsonDecode was used (which produces a dynamic) and a key didn't exist in the returned dynamic. A null value was assigned to a non-nullable type, which caused a downstream bug. I was unable to produce this failure during debugging, which is when I realized that sound null safety appears to be switched off in release but not debug.
If there is a performance reason for this, I would request that a commandline option allow sound null safety to remain enabled.
Steps to Reproduce

Create any sample flutter app.
In a widget's build method, cast a Map to a dynamic and access a non-existent key.
Store the value of the non-existent key in a non-null String variable.
Display the value of the non-null String variable to the screen.
Run the app in web debug mode, observe behavior.
Run the app in web release mode, observe behavior.

Expected results:
Debug build:

Release build:

Actual results:
Debug build:

Release build:

Code sample

  @override
  Widget build(BuildContext context) {
   String thing = ({} as dynamic)["thing"];
   return Text("thing: $thing");
  }

Logs

Launching lib\main.dart on Chrome in debug mode...
This app is linked to the debug service: ws://<IP_ADDRESS>:65461/bPBQlViR65o=/ws
Debug service listening on ws://<IP_ADDRESS>:65461/bPBQlViR65o=/ws
 Running with sound null safety
Connecting to VM Service at ws://<IP_ADDRESS>:65461/bPBQlViR65o=/ws
Flutter Web Bootstrap: Programmatic
══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
The following TypeErrorImpl was thrown building MyHomePage(dirty, state: _MyHomePageState#a05ad):
Expected a value of type 'String', but got one of type 'Null'
The relevant error-causing widget was:
MyHomePage
When the exception was thrown, this was the stack:
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 266:49      throw_
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 99:3        castError
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 452:10  cast
C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/_internal/js_dev_runtime/patch/core_patch.dart 705:17                as
packages/null_sound_safety_bug/main.dart 68:12                                                                                build
packages/flutter/src/widgets/framework.dart 4992:27                                                                            build
...

[√] Flutter (Channel stable, 3.3.4, on Microsoft Windows [Version 10.0.22621.674], locale en-US)
    • Flutter version 3.3.4 on channel stable at C:\Users\**********\Documents\Work\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision eb6d86ee27 (2 weeks ago), 2022-10-04 22:31:45 -0700
    • Engine revision c08d7d5efc
    • Dart version 2.18.2
    • DevTools version 2.15.0

[√] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at C:\Users\**********\AppData\Local\Android\Sdk
    • Platform android-32, build-tools 32.1.0-rc1
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.3.4)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.3.32901.215
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2021.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)

[√] VS Code (version 1.72.2)
    • VS Code at C:\Users\**********\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.50.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22621.674]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 106.0.5249.119
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 105.0.1343.53

[√] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

In release builds Flutter tooling invokes dart2js with -O4 optimization level which discards all implicit downcasts.
 -O3
       Enables optimizations that respect the language semantics only on
       programs that do not ever throw any subtype of `Error`.  These
       optimizations improve the generated code, but they may cause programs to
       behave unexpectedly if this assumption is not met.  To use this
       option, we recommend that you properly test your application first
       without it, and ensure that no subtype of `Error` (such as `TypeError`)
       is ever thrown.

       Equivalent to calling dart2js with these extra flags:
         -O2
         --omit-implicit-checks

 -O4
       Enables more aggressive optimizations than -O3, but with the same
       assumptions. These optimizations are on a separate group because they
       are more susceptible to variations in input data. To use this option we
       recommend to pay special attention to test edge cases in user input.

       Equivalent to calling dart2js with these extra flags:
         -O3
         --trust-primitives

  --omit-implicit-checks
    Omit implicit runtime checks, such as parameter checks and implicit
    downcasts. These checks are included by default in Dart 2.0. By
    using this flag the checks are removed, however the compiler will assume
    that all such checks were valid and may use this information for
    optimizations. Use this option only if you have enough testing to ensure
    that your program works with the checks.

You could use --dart-define=Dart2jsOptimization=O2 to switch to a lower optimization level though performance would suffer.
A better solution would be to rewrite your code to use explicit casts rather than implicit. dart2js will not remove explicit casts, which means the code below will actually throw as you expect:
  final thing = ({} as dynamic)["thing"] as String;

To find all the places where you rely on implicit casts from dynamic you can enable strict-casts in the analyzer as described here, by adding the following to your analysis_options.yaml:
analyzer:
  language:
    strict-casts: true

I like the explicit casting solution a lot. Thanks for explaining @mraleph and also for the workarounds.


kak-lsp: fix build on Darwin

Description of changes
Added required CoreServices dependency on Darwin (aarch64 at least). I'm not sure why it is now required and wasn't a few weeks ago -- presumably something about the Rust toolchain? Howerver, it does seems to be a legitamate dependency.
Without this, build fails with:
error: linking with `/nix/store/ybr7a0lvp4dzr14f1l3882x5qb68vkls-clang-wrapper-11.1.0/bin/cc` failed: exit status: 1
  |
  = note: LC_ALL="C" PATH="/nix/store/s71dj54b3q0594sis3vp4hl2mdbnn6p3-rustc-1.72.0/lib/rustlib/aarch64-apple-darwin/bin:/nix/store/8mdcavvgk4jf5blq6mspls97iki29l38-cargo-1.72.0/bin:/nix/store/8057g3ixa1i7bxbs43xpgydydl4avsgp-cargo-auditable-0.6.1/bin:/nix/store/qnfrjk7bpxanl6s60y2197q0r782h9q5-auditable-cargo-1.72.0/bin:/nix/store/8mdcavvgk4jf5blq6mspls97iki29l38-cargo-1.72.0/bin:/nix/store/s71dj54b3q0594sis3vp4hl2mdbnn6p3-rustc-1.72.0/bin:/nix/store/ybr7a0lvp4dzr14f1l3882x5qb68vkls-clang-wrapper-11.1.0/bin:/nix/store/6m1ncyha74vyyksinykcsic5gc1b29i7-clang-11.1.0/bin:/nix/store/g9pvwpwpgkhzlq0kv4bbllclm18kv0gj-coreutils-9.3/bin:/nix/store/7yafaqqhz1i9s4jmsj51g28c22yhq3kq-cctools-binutils-darwin-wrapper-11.1.0-973.0.1/bin:/nix/store/5v4ycnqpfammvlwpadyc26by7hy604cz-cctools-binutils-darwin-11.1.0-973.0.1/bin:/nix/store/g9pvwpwpgkhzlq0kv4bbllclm18kv0gj-coreutils-9.3/bin:/nix/store/cnjxsppdzfbwvmkdrkpps6j75a2xqgks-findutils-4.9.0/bin:/nix/store/fmb75iiyrc7bhr0al97izqch5pzrh5dy-diffutils-3.10/bin:/nix/store/c9606i14wkrcjavx5kvhaghxfpgy204k-gnused-4.9/bin:/nix/store/xrqvjyn24z6ljnh9583c13iarks1faa3-gnugrep-3.11/bin:/nix/store/gg71r49xcnn2l565cwz11ws3q38liwdw-gawk-5.2.2/bin:/nix/store/h1y472wdyk5jn7qsgwhcblaj9j42xrx9-gnutar-1.35/bin:/nix/store/5m5mvssdqh2liqz0d6bhjp7sql1bsi62-gzip-1.13/bin:/nix/store/p2i1j2pj8wjmy445xjlcfzq64r7gad50-bzip2-1.0.8-bin/bin:/nix/store/221p2yp3dlb6dxr93iidq4gpx9i25n81-gnumake-4.4.1/bin:/nix/store/zqx1fik7mcyg1037s015kfbw2zq0qhqn-bash-5.2-p15/bin:/nix/store/9ahpbnv7xxyp72f6a08qm0pyrzg85l7x-patch-2.7.6/bin:/nix/store/nws79v3asz8a9cyyn8f76kwykjr6lhkb-xz-5.4.4-bin/bin:/nix/store/f2zhjzawah3klb72plmq79kldymn93gc-file-5.45/bin" VSLANG="1033" ZERO_AR_DATE="1" "/nix/store/ybr7a0lvp4dzr14f1l3882x5qb68vkls-clang-wrapper-11.1.0/bin/cc" "-arch" "arm64" "/private/tmp/nix-build-kak-lsp-14.2.0.drv-0/rustcoRixb3/symbols.o" "/private/tmp/nix-build-kak-lsp-14.2.0.drv-0/source/target/aarch64-apple-darwin/release/deps/kak_lsp-e6562ae60fd40469.kak_lsp.f56ce52223958bad-cgu.04.rcgu.o" "-L" "/private/tmp/nix-build-kak-lsp-14.2.0.drv-0/source/target/aarch64-apple-darwin/release/deps" "-L" "/private/tmp/nix-build-kak-lsp-14.2.0.drv-0/source/target/release/deps" "-L" "/nix/store/s71dj54b3q0594sis3vp4hl2mdbnn6p3-rustc-1.72.0/lib/rustlib/aarch64-apple-darwin/lib" "/nix/store/s71dj54b3q0594sis3vp4hl2mdbnn6p3-rustc-1.72.0/lib/rustlib/aarch64-apple-darwin/lib/libcompiler_builtins-9a5dde18cc747729.rlib" "-framework" "CoreFoundation" "-framework" "SystemConfiguration" "-framework" "CoreFoundation" "-framework" "CoreServices" "-liconv" "-lSystem" "-lc" "-lm" "-L" "/nix/store/s71dj54b3q0594sis3vp4hl2mdbnn6p3-rustc-1.72.0/lib/rustlib/aarch64-apple-darwin/lib" "-o" "/private/tmp/nix-build-kak-lsp-14.2.0.drv-0/source/target/aarch64-apple-darwin/release/deps/kak_lsp-e6562ae60fd40469" "-Wl,-dead_strip" "-nodefaultlibs" "/private/tmp/nix-build-kak-lsp-14.2.0.drv-0/source/target/aarch64-apple-darwin/release/deps/kak_lsp_audit_data.o" "-Wl,-u,_AUDITABLE_VERSION_INFO"
  = note: ld: framework not found CoreServices
          clang-11: error: linker command failed with exit code 1 (use -v to see invocation)

error: could not compile `kak-lsp` (bin "kak-lsp") due to previous error

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release 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

[x] Fits CONTRIBUTING.md.

Note: Built with sandbox=true, even though not in nix.conf.
Without this, recent master started failing to build on Darwin with an error that CoreServices is not available.
With this, the build completes successfully again.

Result of nixpkgs-review pr 261866 run on aarch64-darwin 1

1 package built:

kak-lsp (kakounePlugins.kak-lsp)


Godot release 4.3 update.

Hi,
I just want to ask if you are ok with it, will you be bumping this up for 4.3 (the release seems to be unless more issues surface sometimes in the next week probably).
Secondly, since the last update was 8 months ago, did you drop this project, please?
(No pressure, it's a really nice thing so I'm just curious about the future of your project? :)).
Thank you very much!

The project is not dropped, but development will be slow for a while because I'm currently busy with preparing the release of Days of Defiance, which is going to take all the time I have available.
I have big plans for this project and will return to it as soon as I'm not as busy anymore.

Oh that's a pretty cool project you have there :).
So not even bumping it up for the 4.3 I take it? Thanks and good luck with your game!

Bumping is going to happen, but only after 4.3 is actually released.


bug: convertLabelSelectorsToMap is broken three ways

Describe the bug
pkg/ocm/ocm.go:convertLabelSelectorsToMap is broken in three ways.

The return type can not represent the OR functionality that is the semantics in https://github.com/kubestellar/kubestellar/blob/ad2f86d60cf5b27a88a16839d419d152b346ff5a/api/edge/v1alpha1/types.go#L26-L27
https://github.com/kubestellar/kubestellar/blob/ad2f86d60cf5b27a88a16839d419d152b346ff5a/pkg/ocm/ocm.go#L190 will fail for a selector that is not simply an AND of key=value pairs. There are indeed more complex label selectors. See https://v1-28.docs.kubernetes.io/docs/concepts/overview/working-with-objects/labels/#label-selectors .

Steps To Reproduce
This bug report comes from code inspection.
Expected Behavior
Any valid user input can be handled.
Additional Context
No response

/cc @pdettori
/cc @vMaroon

@MikeSpreitzer you mentioned pkg/ocm/ocm.go:convertLabelSelectorsToMap is broken in two ways, but I only see two listed.  Please let me know if I am missing anything.


Retry failed requests reported via AcaPy Webhook calls

Currently, we receive many errors via AcaPy Webhook calls. Maybe, just retrying them would allow us to still process them and therefore make the system more reliable and resiliant.

In order to retry a connection issuance after an error has been reported via the webhooks, would require the controller to store a list of all pending credential issuances to be able to tricker them again in case of a failiure.


verasaw website address corrected

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address: 5UmAAf8hHJe81DwSi1n2ZhfkmAqg9ABdK36oKKgh7iBL
Token Name: Verasaw
Token Symbol: VRSW
Logo: https://raw.githubusercontent.com/solana-labs/token-- - list/main/assets/5UmAAf8hHJe81DwSi1n2ZhfkmAqg9ABdK36oKKgh7iBL/logo.png
Link to the official homepage of token: https://verasaw.io
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

Logo: https://raw.githubusercontent.com/solana-labs/token-list/main/assets/mainnet/5UmAAf8hHJe81DwSi1n2ZhfkmAqg9ABdK36oKKgh7iBL/logo.png

https://github.com/solana-labs/token-list/commit/a0fe7cfc9fcde071365a12da19aaf36292a4966d

would strongly recommend you squash your changes in the future