Organizer-Ruby_Capstone-project: All changes!

This is the final PR for the Ruby capstone project Catalog of my things
This project was done by 2 team members:

@Profsain
@jahongiry
@Sahar-SE
Here is a descriptive Video for the project:

https://user-images.githubusercontent.com/91022355/190725177-5a6c4ded-f207-4fe4-8b71-0c174a03fb5f.mp4
The following requirements are done in this PR:
Group tasks

Create Item class in a separate .rb file.
All Item class properties visible in the diagram should be defined and set up in the constructor method. Exception: properties for the 1-to-many relationships should NOT be set in the constructor method. Instead, they should have a custom setter method created.
Add all methods visible in the diagram.
Implement methods:
can_be_archived?() in the Item class
should return true if published_date is older than 10 years
otherwise, it should return false
move_to_archive() in the Item class
should reuse can_be_archived?() method
should change the archived property to true if the result of the can_be_archived?() method is true
should do nothing if the result of the can_be_archived?() method is false
Create a main.rb file that will serve as your console app entry-point.
Implement startup actions:
Present the user with a list of options to perform.
Let users choose an option.
If needed, ask for parameters for the option.
Have a way to quit the app.
@jahongiry
Create a Book class in a separate .rb file.
Create a Label class with an association to the Item class (in a separate .rb file).
All Book class properties visible in the diagram should be defined and set up in the constructor method.
All Label class properties visible in the diagram should be defined and set up in the constructor method.
Implement required methods
Add unit tests for all implemented methods.
The following options should be available:
List all books
List all labels (e.g. 'Gift', 'New')
Add a book
All data should be preserved by saving collections in .json files.
Create a schema.sql file with tables that will be analogical to the structure of the classes that you created:
books table (add all properties and associations from the parent Item class as table columns)
labels table

@Sahar-SE

Create MusicAlbum class in a separate .rb file.
Create Genre class with an association to the Item class (in a separate .rb file).
All MusicAlbum class properties visible in the diagram should be defined and set up in the constructor method.
All Genre class properties visible in the diagram should be defined and set up in the constructor method.
Implement required methods
Add unit tests for all implemented methods.
The following options should be available:
List all music albums
List all genres (e.g 'Comedy', 'Thriller')
Add a music album
All data should be preserved by saving collections in .json files.
Create a schema.sql file with tables that will be analogical to the structure of the classes that you created:
music_albums table (add all properties and associations from the parent Item class as table columns)
genres table

@Profsain

Create a Game class in a separate .rb file.
Create an Author class with an association to the Item class (in a separate .rb file).
All Game class properties visible in the diagram should be defined and set up in the constructor method.
All Author class properties visible in the diagram should be defined and set up in the constructor method.
Implement required methods
Add unit tests for all implemented methods.
The following options should be available:
List of games
List all authors (e.g. 'Stephen King')
Add a game
All data should be preserved by saving collections in .json files.
Create a schema.sql file with tables that will be analogical to the structure of the classes that you created:
games table (add all properties and associations from the parent Item class as table columns)
authors table

❌ ❌ ❌ STATUS: INVALID  ❌ ❌ ❌

Hello @jahongiry @Profsain @Sahar-SE  ✋🏼
You have submitted a project, that is not working. Kindly fix the error and submit it for review again.

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding! 💻 👏

Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
Invalid Code Review Request does not count into the code reviews limit.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.
GitHub | LinkedIn | Twitter
Reviewed by Ahmet Bozacı

Hello @ahmetbozaci, kindly could you run ruby app.rb to run the project as we mentioned in our README.rb

@ahmetbozaci, We would love to receive your review before we request another review

Hello @ahmetbozaci, kindly could you run ruby app.rb to run the project as we mentioned in our README.rb

Sorry ı didn't see it. Usually ı run the main.rb file. The next code reviewer will check. Have a great weekend.

Thanks for your response, I change to main.rb to avoid this kind of confusions 😊


InfluxDB 0.9 compatibility

Hi,
I'm wondering if this tool is compatible with InfluxDB 0.9+ ?
Thanks.

No.  I have this branch to support 0.9.x, but use at your risk: https://github.com/schemacs/munin-influxdb/tree/v0.9 . I will create a pr when my code is ready.


(REACT NATIVE) How to focus an Input in dispatch actions

Hello guys.
I ask you guys, how to focus an Input in dispatch actions (react native) ?
I try to use this function inside my action but it doesn't work for me :
import { focus } from 'redux-form';

export const initSignup = () => (dispatch) => {
  dispatch(focus('myForm', 'email'));
};

Hi,
This works for me, at least it set the field state to 'active'. Then in the render function of the 'CustomTextInput', I added:
{       formStates.filter((state) => meta[state]).map((state) => {                     if(state === 'active'){                         this.refs.email.focus()                     }            })     }

Thanks @seppestaes i wanted to focus it by actions creator but it doesn't works, so i did it with a simple reference then it worked.


[Dotcom shell]: React VO does not announce location to user after using Skip to content button

Description
When the user selects the Skip to content button from the masthead, the screen reader does not announce the new location. User must tab to get the VO to announce anything, skipping directly to the first TOC item,  and misses the "You are in a list" announcement.
https://user-images.githubusercontent.com/45692622/150417561-321e43d6-ee8c-478d-aa37-203172e095e1.mov
Present in Production
Component(s) impacted
Masthead (assuming it affects masthead)
Dotcom shell
Browser
Chrome
Carbon for IBM.com version
1.30.0 rc 0
Severity
Severity 2 = Aspects of design is broken, and impedes users in a significant way, but there is a way to complete their tasks. Affects major functionality, has a workaround.
Application/website
carbon for IBM.com
Package
@carbon/ibmdotcom-react
CodeSandbox example
https://carbon-design-system.github.io/carbon-for-ibm-dotcom/next/react/?path=/story/components-dotcom-shell--default
Steps to reproduce the issue (if applicable)
No response
Release date (if applicable)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate issues

Need to ask accessibility team for guidance on this.

Try testing on codesandbox...if still exists there, assign severity and pipeline. @oliviaflory

Retested in both React production and React 1.31.0 rc1
React v1.30
When I select the Skip to main content button the VO does not announce where I am within the main content.
https://user-images.githubusercontent.com/45692622/154540146-a9229a23-7a84-4dd4-80c2-c23bce7eaae4.mov
React v1.31.0 rc 1
https://user-images.githubusercontent.com/45692622/154540101-c61df891-cfc8-45e0-98a5-3ef64846cb5d.mov
Web components v1.16.0 rc1
Web components does not have Skip to main content button enabled
I believe all navigation needs a Skip to main content option if the user does not wish to go through the main navigation.
https://user-images.githubusercontent.com/45692622/154540215-1bd3960a-b931-4b6d-94cc-5887fd837e96.mov

@oliviaflory I'm not sure it needs to announce the main content area, based on the accessibility documentation. https://www.carbondesignsystem.com/components/UI-shell-header/usage/#interactions https://www.w3.org/TR/2016/NOTE-WCAG20-TECHS-20161007/G1
Since the link itself already announces it is for skipping to the main content, it seems this is behaving correctly?

@kennylam hmm ok, I guess I assumed you'd want to hear what you landed on, but to does sound like we might be ok to close.
I swear there was another example that announced "you are currently on a list" when you land on the TOC, but I don't hear it in that example


Discussion: Is parsing conceptually part of running, or not?

Background: The Parser converts textual source code into atoms, which are then interpreted.  The interpreter runs within a runner, which has a runtime state.  The question at hand is whether the parser should be considered part of the run process, and thus be able to read and modify the runtime state or whether it is a pre-process that happens before running.
The reason it matters is because some tokens in the source text make reference to objects in the runtime state.  The simplest example being &self.
In the current implementation, the Tokenizer (invoked by the Parser) replaces the text &self with a grounded atom that refers to the running Space.  However, different modules have different interpretations of &self.  For now, we can work around this because different modules also have different Tokenizers, but this design choice has major conceptual implications for the MeTTa language.
If the answer is “Yes”, ie that parsing is part of running, then the parser should replace the token with a value derived from the actively running context.  Which means parsing needs to occur within an active runner, not as a pre-process.
If the answer is “No”, ie that parsing is a pre-process and MeTTa’s native program format is strictly Atom-based, then the parser does not have enough information to resolve the &self token all the way to a specific grounded Space atom at parse time.  Therefore it should turn&self into some abstract atom, or leave it as a symbol atom.  And then finally the interpreter would substitute the abstract atom or symbol with the specific grounded atom, possibly based on an equality expression from the running module's Space.
My personal feeling is that “No” makes more sense.  It is more similar to other languages where the running program can't reconfigure the parse logic as the program runs.  Most languages I know about require macros, etc. to be applied upstream, either in a pre-pass (like C) or from a dependency that loads in advance (like Rust proc macros), and I feel this will be less error-prone.
This does, however, mean we need to rethink the bind! operator, perhaps to allow a grounded atom to be substituted for a symbol using an equality defined in the space, so bind! would not be needed.

For me, what matters is the behavior of metta-scripts rather than whether parsing should or should not be conceptually part of running. i'm ok with an option when the needed behavior is achieved by another mechanism. We may want to get rid of special tokens at all, although parsing strings or numbers is difficult to avoid. And the idea of having a fixed parser, which will incorporate all such special cases, doesn't look appealing to me. If we do have some tokens, why not to allow defining them by modules and extensions? Consequently, as @vsbogd pointed out, the set of tokens can be different for different modules.  Maybe, we are overusing this mechanism of extending the set of tokens. I'm in favour of not using bind! if we have a better mechanism. [Unfortunately?] we don't support equalities for symbol atoms (e.g. (= &one 1)), but we could still have, say, (= (self) %unrepresentable-grounded-atom-space%) put into the space of the module, which equates (self) and the corresponding grounded atom instead of having &self token, which is replaced by the same grounded atom during parsing. OTOH, what is the benefit of having (= (self) ...) inserted into the space? I'm not sure. We insert something, which cannot be directly represented in metta scripts. A possibility to introduce direct syntactic representation of grounded atoms like &self looks natural. Once again, I'm not advocating for using bind!. Recently, I prefer to place equalities in the space instead, e.g. ! (let $space (new-space) (add-atom &self (= (my-space) $space)) (which could be made more convenient, though) rather than !(bind! &my-space (new-space)). I'd prefer also a version of import!, which returns a space rather than binds it to a new token. But even if we get rid of token bindings in import! and reduce the use of bind!, because it is kind of misused due to absence of a convenient token-less alternative, the possibility to introduce new tokens by models and extensions is very handy. We cannot avoid this by inserting equalities or other runtime binding mechanisms, if we want to be able to extend the language by families of tokens like numbers, strings, etc. For example, we wanted to introduce regex atoms. We need a token family for this like r"[^"]*" or such. Of course, we can have constructors, which convert strings to the needed object, e.g. (regex "^Hello[[\.|!]]?$") instead of r"^Hello[[\.|!]]?$". It has its own inconveniences, e.g. (= (intent (regex "^Hello[[\.|!]]?$")) Hello) will not work for (intent "Hello"), because (regex "^Hello[[\.|!]]?$") will not be executed by the pattern matcher. It can be overcome by different means, but it does not work for now.
Basically, introduction of new tokens can be replaced by other means. Sometimes it should be replaced, indeed. In other cases, it is somewhat questionable if it is worthy. Answering the question "Is parsing conceptually part of running, or not?" I'd say that "Yes" answer is more amusing, and I'd stay with "Yes" unless there are indeed deep technical troubles implied by this choice. I'd prefer to reduce the use of custom tokens and encourage a token-less approach, when possible. But I wouldn't forcefully remove runtime tokenization. I understand that the question may arise from the work on modules and imports, and it could be an important choice right now. But if this is really the case, we may need discussing what are the problems with runtime parsing for the module system.


Add values schema

Add a values schema, like eg. #57 tried to do it.
Needed for eg. a OpenShift Helm chart certification (https://redhat-connect.gitbook.io/partner-guide-for-red-hat-openshift-and-container/helm-chart-certification/helm-chart-requirements)
We should find a way to enforce the proper adaption of the schema in future PRs.

I did a bit of research on this and this is certainly not a painless topic. There are unmaintained projects such as https://github.com/karuppiah7890/helm-schema-gen where you can generate a schema file from values, but that will require manual editing of the resulting schema to make things work. And doing things manually completely makes that a moot point for when you have to update values.
Then there's going from the other way around with something like https://github.com/giantswarm/helm-values-gen, but that completely breaks the current documentation setup.
I found some things where both the values and schema would be generated from Yet Another Format.
So, this is currently not actionable without a bigger research task and discussion around how to manage this.

I stumbled across this recently: https://github.com/SocialGouv/helm-schema
Probably worth a proof of concept for a few values.

There is https://github.com/dadav/helm-schema, too


core.counter: Add double-buffering

This is primarily intended to avoid a performance problem discovered on PR #558 where frequently accessing many public counter objects is expensive because their addresses are page-aligned and this clashes with the set-associative L1/L2 cache on the Xeon.
Double buffering is a handy technique though and it could become useful in the future for publishing structured shared memory objects and supporting atomic reads.

Further work on this feature is being done on #568.


Show if the user is "pending"

A relatively minor feature but if the user is "pending", indicate this in the initial messages posted to the top of a modmail thread. Something like "This user has not completed Member Verification", assuming that the verification functionality has been enabled in a guild. I haven't yet been able to find anything about whether this is part of Modmail already ^_^

As far I know, Discord's API does not cover this.

The bot doesn't currently support this, but it would be a fairly easy addition! Out of interest, what kind of a use case were you thinking for this? For example, I could see an option that rejects modmail threads from users that haven't completed verification as potentially useful to cut down on accidental replies to greeting messages.

The bot doesn't currently support this, but it would be a fairly easy addition! Out of interest, what kind of a use case were you thinking for this? For example, I could see an option that rejects modmail threads from users that haven't completed verification as potentially useful to cut down on accidental replies to greeting messages.

Hiya. So mainly, a use case could be whether a user is clearly a troll abusing modmail and not a legitimate user.
I don't recall if Member Verification blocks access to DM's? I remember reading somewhere that DM's are not possible with a user who is pending, so it might be this feature isn't needed.

predo ter 2880 narain

For servers that don't have Member Verification, an option that makes messaging the bot require a role might also be nice, for the exact same purpose.


Changing RelativePath reflects only after 2 page loads

VERSION:
Swashbuckle.AspNetCore v6.1.5 in an Asp.Net Core 5 project.
STEPS TO REPRODUCE:
Add this Route Filter
// This filter has been simplified for debugging purposes.
public class SwaggerRouteFilter : IDocumentFilter
{
    public void Apply(OpenApiDocument swaggerDoc, DocumentFilterContext context)
    {
        // This will update the first action endpoint path found.
        foreach (var apiDescription in context.ApiDescriptions)
        {
            apiDescription.RelativePath = "overwritten/path";
            break;
        }
    }
}

Add this to your startup.cs
services.AddSwaggerGen(c =>
{
   c.DocumentFilter<SwaggerRouteFilter>();
}

Test procedure

Load your swagger UI or Json file. Notice that your first endpoint's relative-path has not changed. Its still the same.
Refresh your browser. Notice that only now the relative-path has changed.

EXPECTED RESULT:
On the first page load, the relative path of the first endpoint should change.
ACTUAL RESULT:
The relative path only changes on the second loading of the page.
ADDITIONAL DETAILS
A similar issue https://github.com/domaindrivendev/Swashbuckle.WebApi/issues/361 was reported in 2015 but instead of using a IDocumentFilter it was using IOperationFilter.

@krugertech as was mentioned on the earlier issue you found, the ApiDescription is just passed for context - what you need to be modifying in an IDocumentFilter is the OpenApiDocument parameter. In the above-described case you could do something like:
public class SwaggerRouteFilter : IDocumentFilter
{
    public void Apply(OpenApiDocument swaggerDoc, DocumentFilterContext context)
    {
        var firstPath = string.Empty;
        var firstEndpoint = null;
        foreach (var path in swaggerDoc.Paths)
        {
            firstPath = path.Key; // the path
            firstEndpoint = path.Value; // the OpenApiPathItem representing the operations/parameters/etc
            break;
        }

        swaggerDoc.Paths.Remove(firstPath);
        swaggerDoc.Paths.Add("overwritten/path", firstEndpoint);
    }
}

It seems like root cause of the behaviour you saw is that the ApiDescription gets cached internally, so when you modify it during one page load, the change happens to be reflected on subsequent page loads.
I'm assuming by now (a year+ after raising the issue) you have either found a solution or moved on - but hopefully this can help others who run into the same thing, like I did 😁


Forwarding requests to multiple ports as json.  Second one arrives empty.

Trying to forward requests to multiple ports as application/json, but the second request arrives empty.  This doesn't happen when sent as text/plain.

<?php

require_once 'vendor/autoload.php';

$loop = \React\EventLoop\Factory::create();

$browser = new \React\Http\Browser($loop);

$clients = ["http://<IP_ADDRESS>:8001", "http://<IP_ADDRESS>:8002"];

$server = new \React\Http\Server($loop, function (\Psr\Http\Message\ServerRequestInterface $request) use ($browser, $clients){

    # forward requests to multiple ports as json
    foreach($clients as $client){
        $browser->post(
            $client,
            ['Content-Type' => 'application/json'],
            $request->getBody()
        )
    }
    # this works and is received on both 8001, 8002
    foreach($clients as $client){
        $browser->post(
            $client,
            ['Content-Type' => 'text/plain'],
            "hello"
        )
    }
});

$socket = new \React\Socket\Server('<IP_ADDRESS>:8000', $loop);

$server->listen($socket);

$loop->run();

The request uses a RingCentral\Psr7\BufferStream as return value of $request->getBody(). By calling $request->getBody() this buffer will be emptied and is not available for our future requests.
What you may want is to call $request->getBody()->getContents() and save the value to a variable for reusage.
I hope this helps 👍

@legionth works! thank you!! :)

@eitelkrauss Thanks for bringing this up!
I agree that only being able to read the request body once (and only for some requests) is somewhat inconsistent, so I've just filed #395 to make sure you can always read the raw request body. Note that the getContents() method will only read the remainder of the buffer as per the PSR-7 definition. Reading will advance this previous position, so a second call will return an empty string unless you rewind() or seek() the request body.
In your specific use case, assigning this to a string once and reusing this sounds like a reasonable alternative :+1:


UPnP improvements

This ensures that the port is mapped/removed succesfully or at least attempts to ensure.

Ok, so I have two routers with UPnP.  Both are NETGEAR routers.
This fix is for my primary router.
My primary router won't map the port on the first attempt, but on the second attempt it maps properly.
My secondary router WILL map the port on the first attempt, but it will throw an exception calling _device.GetAllMappings() prompting the error box to come up when starting the listen server.
Are there any other libraries we could use, or should we just leave it as is?  If one of my routers is having trouble connecting with UPnP on the first attempt to map the port, I wouldn't be surprised if any other users would have the same problem.

Ignore the above.  I've found the best way to attempt to ensure the router processes the requests.  Just loop through it a few times... Won't hurt anything either!


Test that the runner process is using the github user environment

Change-type: patch

I self-certify!


deduplicated urls in backlink plugin results #65

Coverage increased (+1.06%) to 87.044% when pulling 8f68a563290e3daa5d68731e6a9f45ac947c572c on valentinlehuger:master into 28ca0c1b938dde52bf4f59a835c98dd5ab22cad6 on commonsearch:master.

Coverage increased (+1.1%) to 87.097% when pulling b0da1d5b44cbf825b5e5ffcaaf9b65e015ee7ff2 on valentinlehuger:master into 28ca0c1b938dde52bf4f59a835c98dd5ab22cad6 on commonsearch:master.


Missing param force / replace

moved from former sysctl module (not supported any more) to augeasproviders_sysctl.
existing target files in /etc/sysctl.d cannot be overwritten:
Failed to save Augeas tree to file.
Dependency Sysctl[] has failures.
Type "file" has force / replace params these I am missing here.
May be there is a simple other solution?
Thanks in advance
Anton

Which former sysctl module are you referring to?

Hi Raphaël,
I moved from thias/sysctl to augeasproviders getting errors with /etc/sysctl.d/ with same names:
Error: /Stage[main]/Commons::Sysctl_changes/Sysctl[net.ipv4.tcp_keepalive_time]: Could not evaluate: Failed to save Augeas tree to file. See debug logs for details.
augeasproviders_sysctl has no option to overwrite ... thats all
Cheers

@real-aga 	can you run Puppet with --debug and send the debug log? Also, can you provide the current sysctl file and the Puppet code you're trying to apply?


admission controller types

AdmissionReview / AdmissionRequest / AdmissionResponse do not have generated types in k8s-openapi.
This is possibly (speculation) for good reasons; the way they work is very different from a normal api call: https://kubernetes.io/blog/2019/03/21/a-guide-to-kubernetes-admission-controllers/
The types however are present in kuberenetes/api/admission, and don't see how people can do admission controllers without them.
I imagine we could provide a module that hand-rolled those structs, make them generic over the Resource trait, and it might not be that bad?
Have not had much time to look at this, but it could be a nice clean feature for someone to try, and am happy to review PRs. Marking as help wanted, but it's probably not easy.
Testing wise, a web server would pull in a lot of deps, but maybe we could do something light with warp or hyper as an example.

This is now officially released in 0.53. Though it did receive a number of minor tweaks in master from #488, #498, #499, #507, #509.
Thanks again!


Reinstate query-state command to client binary

CHANGELOG:

Added the alias query-state to the query-global-state command in the client binary to maintain backwards compatibility

bors r+


Convert http errors to json

What problem does the pull request solve?
When the Notify service throws a 'NotifyClientException' Exception, the returned string is unparsable due to inconsistent json formatting.
Examples below:

{"Status code 400. Error: {"errors":[{"error":"BadRequestError","message":"Can\u2019t send to this recipient using a team-only API key"}],"status_code":400}\n, Exception: Status code 400. The following errors occured [\r\n  {\r\n    "error": "BadRequestError",\r\n    "message": "Can’t send to this recipient using a team-only API key"\r\n  }\r\n]"}

{"Status code 400. Error: {"errors":[{"error":"ValidationError","message":"phone_number Not enough digits"}],"status_code":400}\n, Exception: Status code 400. The following errors occured [\r\n  {\r\n    "error": "ValidationError",\r\n    "message": "phone_number Not enough digits"\r\n  }\r\n]"}

Change converts output from BaseClient.HandleHTTPErrors to return a standard JSON object of type 'NotifyHTTPErrorResponse' to allow Json Deserialisation.

{"status_code":"400", "errors": [{"error":"ValidationError","message":"phone_number Too many digits"}],"exception":null}

{"status_code":"400", "errors":[{"error":"ValidationError","message":"phone_number Not enough digits"}],"exception":null}

Related issue thread: https://github.com/alphagov/notifications-net-client/issues/161
Checklist

[x] I’ve used the pull request template
[x] I’ve written unit tests for these changes
[x] I’ve update the documentation (in DOCUMENATION.md and CHANGELOG.md)
[x] I’ve bumped the version number (in src/GovukNotify/GovukNotify.csproj)

Any way we can get this merged to master?

Any way we can get this merged to master?

I'm not 100% sure if I missed a step in requesting the merge,
@samuelhwilliams can you advise (Apologies if wrong person, I've seen you are quite active on this repo)

Any way we can get this merged to master?

I'm not 100% sure if I missed a step in requesting the merge, @samuelhwilliams can you advise (Apologies if wrong person, I've seen you are quite active on this repo)

Thank you for responding. Just realised, does this change mean the documentation will have to change to?
https://docs.notifications.service.gov.uk/net.html


[KOGITO-3910] Add option to run tests with cluster wide operator

Signed-off-by: Karel Suta<EMAIL_ADDRESS>https://issues.redhat.com/browse/KOGITO-3910
Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[x] You have read the contributors' guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains a link to the JIRA issue
[x] Pull Request contains a description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[x] Your feature/bug fix has a unit test that verifies it
[x] You've tested the new feature/bug fix in an actual OpenShift cluster

BDD tests can be started once https://github.com/kiegroup/kogito-cloud-operator/pull/675 is merged

/jenkins test


PowerToys Run can show meanings and synonyms

📝 Description of the feature
PowerToys Run can act as a dictionary and search for meanings. A prefix before the word like - would enable dictionary mode and display meaning and synonym.

Duplicate of #6678


Logo proposal

Hello! I would like to collaborate with this project by providing a free new logo that suits with the project! Let me know if interested so we can get in touch.

Hello! I am very happy to know this, you can send the logo file to my mailbox<EMAIL_ADDRESS>

@LuigiBaute thank you, you can contact me with telegram  laobie

greetings @laobie this is what I came up with, I hope you like it!

If you like any of these, I'll upload it in a pull request!

The green one. Thanks.


Some notes on Windows don't display anything when opened

I've got a note on Windows that can't be opened properly. When opening it, the window contains neither the note title nor the note text. When trying to click the delete button, I immediately get the message about data corruption. Pressing abort leads to the following trace:
TApplication.HandleException Access violation
  Stack trace:
  $00000001002017E8
  $0000000100200089
  $000000010012F7A8
  $000000010015023A
  $000000010015096F
  $000000010013C4A1
  $0000000100150102
  $000000010000EDA5
  $000000010012E7B1
  $0000000100122ADC
  $00000001001A7527
  $00000001001052B9
  $0000000100105A7C
  $00000001001EEA0E
  $00007FF800ED6CC1
  $00007FF800ED62CE
  $00007FF800ED6058
WARNING: TBitBtn.Destroy with LCLRefCount>0. Hint: Maybe the component is processing an event?

I deleted the debug log before triggering the problem. It doesn't contain anything else other than the trace.

I am afraid the stack trace does not really help. I am assuming you are using 0.16 of tomboy-ng ?  Would it be possible for you to send me this note if it has nothing private in it ?
Otherwise, trick might be for me to send you a version of tomboy-ng binary with full debug enabled. Its pretty wordy and slow but might tell us whats happening.
Windows is not the easiest platform to debug from with its no console issues but I really would like to track this down if at all possible. I thought I had all bad xml issues suitably handled and reported on.
Ah, I wonder if you know the note title ?   Perhaps not. I think a special debug version might be best approach. I could add a --debug-index option for this purpose.....
Davo

This problem was caused by a particular cunning xml problem. A note with a title in XML but not in content (a rather strange aspect of the Tomboy schema).  It can now. I have released some new binaries (v0.16a) and will do remainder in a few days.


sync snowfall with k2 change

This PR should be merged when https://github.com/k2-fsa/k2/pull/587 get merged as there's some API changes in k2 (actually only get_tot_scores)
Wondering what python formatter you guys use, seems there's a lot of automatic style changes at my side...

Isn't it possible to disable automatic formatting of lines that weren't touched?

Isn't it possible to disable automatic formatting of lines that weren't touched?

:NoAutoFormatBuffer can disable auto format on save.
:AutoFormatBuffer enables it.
:FormatLine is able to format a selected range when auto format is disabled.
Friday, 15 January 2021, 19:47 +0800 from<EMAIL_ADDRESS><EMAIL_ADDRESS>
Isn't it possible to disable automatic formatting of lines that weren't touched?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly,  view it on GitHub , or  unsubscribe .

:NoAutoFormatBuffer can disable auto format on save.
:AutoFormatBuffer enables it.
:FormatLine is able to format a selected range when auto format is disabled.
Friday, 15 January 2021, 19:47 +0800 from<EMAIL_ADDRESS><EMAIL_ADDRESS>
Isn't it possible to disable automatic formatting of lines that weren't touched?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly,  view it on GitHub , or  unsubscribe .

OK, style changes reverted.

OK, style changes reverted.

+2
ready to merge

+2
ready to merge


Jumps when using tooltips in Chrome

In Czech:

e. V prohlížeči Chrome při zobrazování tooltipů stránka různě skáče.

Cannot confiirm, do you have any idea, @istvan-satmari , @KataBoku or @josef-janousek ?

Confirmed
page: /index.html#/createnewtask
resolution: 1920x1080, 100% zoom (i.e. default appearance)
tooltip: "statistical data" label
This is expected since the corresponding tooltip is too long and takes more of the page (it cannot fit), thus vertical scrollbar appears (but only for the time of existence of the tooltip).
While I successfully pushed the footer of the page further below (so the vertical scrollbar is ALWAYS present), I still think the tooltip is too long.
Please, consider reducing its length. (If not, simply put this issue among closed ones, since visually it is OK, I think.)

Confirmed
page: /index.html#/createnewtask
resolution: 1920x1080, 100% zoom (i.e. default appearance)
tooltip: "statistical data" label

This is expected since the corresponding tooltip is too long and takes more of the page (it cannot fit), thus vertical scrollbar appears (but only for the time of existence of the tooltip).

While I successfully pushed the footer of the page further below (so the vertical scrollbar is ALWAYS present), I still think the tooltip is too long.
Please, consider reducing its length. (If not, simply put this issue among closed ones, since visually it is OK, I think.)

Fair enough, I'll look into it.

I have shortened the label in https://github.com/odalic/odalic-ui/commit/2b6dbce3f379ff190937e9a110ebcfe9830f7635 . i would revert the moved footer change, it creates unnecessary empty space


Trim optional white space in CombinedHttpHeaders values

Motivation:
The updated HTTP/1.x RFC allows for header values to be CSV and separated by OWS [1]. CombinedHttpHeaders should remove this OWS on insertion.
[1] https://tools.ietf.org/html/rfc7230#section-7
Modification:
CombinedHttpHeaders doesn't account for the OWS and returns it back to the user as part of the value.
Result:
Fixes #6452

@normanmaurer @Scottmitch @fenik17 Sorry for the delay in addressing the comments. I've done a few basic changes and invited @fenik17 to submit a follow-up with the changes he has in mind.

@ddossot sorry for the delay :( Just merged into 4.1 (9c1a1916962354f1e69fe43107219c25a86e6cef) . @ddossot would you mind port this over to 4.0 and open a new PR for it as well ?

Just for closure here, after discussing with @normanmaurer in meatspace, we will not port to 4.0 as the CombinedHttpHeaders feature is lacking altogether there.

👍

Am 22.03.2017 um 21:05 schrieb David Dossot<EMAIL_ADDRESS>Just for closure here, after discussing with @normanmaurer in meatspace, we will not port to 4.0 as the CombinedHttpHeaders feature is lacking altogether there.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Would this work with Office 2019 (with adapted script)?

Do you think it is possible to make Office 2019 (non-subscription version) using a fork of this script? Or would the attempt be futile?

That wouldn't work because the script emulates a Windows 7 environment while Office 2019 and higher require at least Windows 10 (see #18 for why emulating newer Windows versions hasn't worked so far).


Contributing : Add Anirban Singha's profile

Followed the contribution protocol to add my profile in the repo

Followed the contribution protocol to add my profile in the repo

follow all the guidelines stepwise after that your pr accepted


Update the user entity - data model in the wiki page

Describe the change
What part of the documentation should change
Update the user entity in the data model in the wiki page to reflect the changes made in user model.
Why is this happening
Briefly mention why the change is happening if not inherently obvious
Two more fields are added:

ResetToken
CanReset.

Both of which are for checking if the user is authorized to reset password.

Closing issue as completed  data model

Thanks Hong :)


Failed to install on OS X 10.10.5 with Atom 1.0.13.

The following build error message was thrown.
><EMAIL_ADDRESS>install /private/var/folders/4v/x63j8szs4ygbh5_sv3_rrpyr0000gs/T/apm-install-dir-115818-69749-12abaso/node_modules/Hydrogen/node_modules/zmq
> node-gyp rebuild

  CXX(target) Release/obj.target/zmq/binding.o
  SOLINK_MODULE(target) Release/zmq.node

ld: warning: directory not found for option '-L/opt/local/lib'
ld: library not found for -lgcc_s.10.5
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [Release/zmq.node] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/Applications/Atom.app/Contents/Resources/app/apm/node_modules/npm/node_modules/node-gyp/lib/build.js:269:23)
gyp ERR! stack     at ChildProcess.emit (events.js:98:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:820:12)
gyp ERR! System Darwin 14.5.0
gyp ERR! command "node" "/Applications/Atom.app/Contents/Resources/app/apm/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /private/var/folders/4v/x63j8szs4ygbh5_sv3_rrpyr0000gs/T/apm-install-dir-115818-69749-12abaso/node_modules/Hydrogen/node_modules/zmq
gyp ERR! node -v v0.10.40
gyp ERR! node-gyp -v v2.0.2
gyp ERR! not ok
npm ERR! Darwin 14.5.0
npm ERR! argv "/Applications/Atom.app/Contents/Resources/app/apm/bin/node" "/Applications/Atom.app/Contents/Resources/app/apm/node_modules/npm/bin/npm-cli.js" "--globalconfig" "/Users/malar/.atom/.apm/.apmrc" "--userconfig" "/Users/malar/.atom/.apmrc" "install" "/private/var/folders/4v/x63j8szs4ygbh5_sv3_rrpyr0000gs/T/d-115818-69749-1ao0jif/package.tgz" "--target=0.30.6" "--arch=x64"
npm ERR! node v0.10.40
npm ERR! npm  v2.13.3
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the zmq package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls zmq
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /private/var/folders/4v/x63j8szs4ygbh5_sv3_rrpyr0000gs/T/apm-install-dir-115818-69749-12abaso/npm-debug.log

I'm getting the same build error after upgrading from atom 1.0.11 to 1.0.13 or 1.0.14.
Hydrogen worked fine under atom 1.0.11.

Looks like this was probably the Atom bug you mentioned, which is now closed. Can anyone confirm that this is resolved?

I upgraded Xcode to 7.0 and it is resolved.

Requesting reopen:
I am having the same described here. I installed Xcode 7 but it did not fix the issue
><EMAIL_ADDRESS>install /private/var/folders/nj/vg3121g97fsd5ym8__cfhjlr0000gn/T/apm-install-dir-115911-41649-qgmk3p/node_modules/Hydrogen/node_modules/zmq
> node-gyp rebuild

/bin/sh: pkg-config: command not found
gyp: Call to 'pkg-config libzmq --libs' returned exit status 127. while trying to load binding.gyp
gyp ERR! configure error
gyp ERR! stack Error: `gyp` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (/Applications/Atom.app/Contents/Resources/app/apm/node_modules/npm/node_modules/node-gyp/lib/configure.js:357:16)
gyp ERR! stack     at ChildProcess.emit (events.js:98:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:820:12)
gyp ERR! System Darwin 14.5.0
gyp ERR! command "node" "/Applications/Atom.app/Contents/Resources/app/apm/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /private/var/folders/nj/vg3121g97fsd5ym8__cfhjlr0000gn/T/apm-install-dir-115911-41649-qgmk3p/node_modules/Hydrogen/node_modules/zmq
gyp ERR! node -v v0.10.40
gyp ERR! node-gyp -v v2.0.2
gyp ERR! not ok
npm ERR! Darwin 14.5.0
npm ERR! argv "/Applications/Atom.app/Contents/Resources/app/apm/bin/node" "/Applications/Atom.app/Contents/Resources/app/apm/node_modules/npm/bin/npm-cli.js" "--globalconfig" "/Users/tom/.atom/.apm/.apmrc" "--userconfig" "/Users/tom/.atom/.apmrc" "install" "/private/var/folders/nj/vg3121g97fsd5ym8__cfhjlr0000gn/T/d-115911-41649-155stgi/package.tgz" "--target=0.30.7" "--arch=x64"
npm ERR! node v0.10.40
npm ERR! npm  v2.13.3
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the zmq package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls zmq
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /private/var/folders/nj/vg3121g97fsd5ym8__cfhjlr0000gn/T/apm-install-dir-115911-41649-qgmk3p/npm-debug.log

Request reopen on this. I'm having the same issue described here. I installed xcode 7 but it did not help
OS X 10.10.5, xcode 7.0.1
Atom: 1.0.9
Previously installed hydrogen: 0.62
><EMAIL_ADDRESS>install /private/var/folders/nj/vg3121g97fsd5ym8__cfhjlr0000gn/T/apm-install-dir-115911-41854-adh1uy/node_modules/Hydrogen/node_modules/zmq
> node-gyp rebuild

  CXX(target) Release/obj.target/zmq/binding.o
  SOLINK_MODULE(target) Release/zmq.node

ld: warning: directory not found for option '-L/opt/local/lib'
ld: library not found for -lgcc_s.10.5
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [Release/zmq.node] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/Applications/Atom.app/Contents/Resources/app/apm/node_modules/npm/node_modules/node-gyp/lib/build.js:269:23)
gyp ERR! stack     at ChildProcess.emit (events.js:98:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:820:12)
gyp ERR! System Darwin 14.5.0
gyp ERR! command "node" "/Applications/Atom.app/Contents/Resources/app/apm/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /private/var/folders/nj/vg3121g97fsd5ym8__cfhjlr0000gn/T/apm-install-dir-115911-41854-adh1uy/node_modules/Hydrogen/node_modules/zmq
gyp ERR! node -v v0.10.40
gyp ERR! node-gyp -v v2.0.2
gyp ERR! not ok
npm ERR! Darwin 14.5.0
npm ERR! argv "/Applications/Atom.app/Contents/Resources/app/apm/bin/node" "/Applications/Atom.app/Contents/Resources/app/apm/node_modules/npm/bin/npm-cli.js" "--globalconfig" "/Users/tom/.atom/.apm/.apmrc" "--userconfig" "/Users/tom/.atom/.apmrc" "install" "/private/var/folders/nj/vg3121g97fsd5ym8__cfhjlr0000gn/T/d-115911-41854-h5hz2p/package.tgz" "--target=0.30.7" "--arch=x64"
npm ERR! node v0.10.40
npm ERR! npm  v2.13.3
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the zmq package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls zmq
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /private/var/folders/nj/vg3121g97fsd5ym8__cfhjlr0000gn/T/apm-install-dir-115911-41854-adh1uy/npm-debug.log

Holy cow, this is a pernicious bug that is not hydrogen's fault (as it would seem). I have a fix that is gross, but workable. I'm leaving it here to save others with similar issues a huge ammount of time.
The solution comes from discussion in:  https://github.com/nodejs/node/issues/2933
It looks like it hits lots the npm ecosystem on os x if the user's xcode is not fully up to date. It looks like it can be resolved by manually linking in the library that node-gyp is struggling to find:
cd /usr/local/lib sudo ln -s ../../lib/libSystem.B.dylib libgcc_s.10.5.dylib

This is the hackiest of hacky ways to fix this. Since the link is only needed at build time, users can (and probably should) reverse the link after hydrogen installs by:
rm /usr/local/lib/libgcc_s.10.5.dylib

No need to re-open this issue as previously requested as this is not a hydrogen issue.

Wow, this is sketch. Thanks for reporting back!


go 1.12 windows/amd64 with go-oci8 crashes

Installation of go-oci8 seems to be ok, followed the instructions accurately, but
go test -v github.com/mattn/go-oci8 -tags noPkgConfig -args -disableDatabase=false -hostValid replace_with_your_db_host -username scott -password tiger
fails with
exit status<PHONE_NUMBER>
Any ideas how to resolve that? I've tried a lot already

Installation of go-oci8 seems to be ok, followed the instructions accurately, but
go test -v github.com/mattn/go-oci8 -tags noPkgConfig -args -disableDatabase=false -hostValid replace_with_your_db_host -username scott -password tiger
fails with
exit status<PHONE_NUMBER>
Any ideas how to resolve that? I've tried a lot already

The exit status<PHONE_NUMBER> usually means an Oracle setup issue. It maybe also mean a C++ compiler setup issue, but that is probably not the case.
Please double check all the steps in the readme installation. Note that you either need to use pkgConfig or the CGO_CFLAGS and CGO_LDFLAGS environmental variables.
Note I install Go 1.12 on Linux and tested, worked fine.

Ok, I will also try on Linux.
Note that the build was successful. I've checked with dumpbin and all dll's are available. Just simple code with import of go-oci8 fails to run.

Will most likely work with Go 1.12 on Windows as well, just saying that I did test it on Linux. I am running Go 1.11 on my Windows box.

Confirmed that on ubuntu it works fine.


The commit 9600954 introduced a bug when using the numpad keys

Is there an existing issue for this?

[X] I have searched the existing issues.

Is this issue related to iced?

[X] My hardware is compatible and my graphics drivers are up-to-date.

What happened?
After the commit 9600954, when pressing the numpad keys, instead of producing the specific character for that key it results in the keycode as if that key was pressed with Shift modifier. For example, on my keyboard the numpad key "1" can also be used as "End" when pressed with shift. But when I press it on Iced it is always getting that key event as "End" instead of "1", no matter if I'm pressing shift or not.
This means that I can't use the numpad to write numbers on a textinput.
What is the expected behavior?
Before the mentioned commit that doesn't happen, it produces a Character("1") when pressed normally and a Named(End) when pressed with shift, as expected.
Version
master
Operating System
Windows
Do you have any log output?
No response

Yes, the behavior was changed on purpose.
You should use the text field of the KeyPressed variant for any kind of user input.

Oh, I see you mean the TextInput widget is broken!

But it's not me using anything. Its a problem on the TextInput. When I press the keys on Numpad it doesn't do anything!

Oh, I see you mean the TextInput widget is broken!

Yes. I didn't check if the TextEditor is as well or not...

Thanks for the report! #2278 should fix this.

Thanks for the report! #2278 should fix this.

Well that was quick! I can confirm that does fix it for me!
I tried to look in to the code to see if I could fix it, but since this was such a recent change I thought you probably could figure it out much faster, as it was! Thanks again!


[Feature] Docker image documentation

Hey, you have written in your readme However, there is also a Docker image available for Whisper-TikTok which can be used to run the program in a containerized environment. so i am trying to set it up using docker, but there is no documentation and no docker image. It´s ok for me, i can build the image myself, but some documentation what paths, variables, etc are needed would be highly useful, otherwise i have to figure this out myself.

Hi,
Docker image is available at https://github.com/MatteoFasulo/Whisper-TikTok/pkgs/container/whisper-tiktok (a cool feature of GitHub which allows to also upload docker images on repository). This image spins up the streamlit dashboard but due to the fact that the entire project requires Whisper model (and a lot of GPU related packages) it is a bit too heavy to run but still working.

Hey, thanks for the quick reply. What about the documentation for the docker image? Could you maybe provide a quick docker run command?

Hey, thanks for the quick reply. What about the documentation for the docker image? Could you maybe provide a quick docker run command?

Hi,
you can find a quick guid on how to use the image here: https://matteofasulo.github.io/Whisper-TikTok/3-docker/

Did you manage to solve the issue @Joly0 ?

Yep, works. Thx


[docs] Preview pane docs outdated

Provide a description of requested docs changes
The docs for the preview pane (https://github.com/microsoft/PowerToys/blob/master/src/modules/previewpane/README.md) are outdated.
I would love to have them up to date next week, because then I could easy implement monaco, because I have many time next week.
We should also move this document into the doc  folder.

@aaron-junker, been a long time since we last looked at these.  what makes these out of date?
I don't think the settings aspect for Monaco will be easy / the installer aspect.  https://github.com/microsoft/PowerToys/issues/10126 still needs to happen for file associate

For example Navigate to the powerpreview project and edit the powerpreview.h file. Add the following Settings Object instance to m_previewHandlers settings objects array in the constructor initialization:  There is no way to add a settings object instance to m_previewHandlers Because there is no m_previewHandlers

Is this in progress now? I'm not sure what is wrong/missing integrating preview.

Is this still an issue?

These are dev docs so there is a chance.

<rel>#761, #17951</rel>

This seems to be completed.


intelligence-SpeechToText bot always gives timeout

SDK Platform
node sdk
URL of Code Sample
https://github.com/Microsoft/BotBuilder-Samples/tree/master/CSharp/intelligence-SpeechToText
Problem Description
I've added my key, started the sample. each time I upload the wav the bot waits, then there is a timeout when posting the wav to Bing.
The access token is received correctly by the bot, but the post request fails with timeout.

I had a similar problem with the Skype calling bot sample. There was a line of code that was commented out in the sample:
binaryContent.Headers.TryAddWithoutValidation("content-type", "audio/wav; codec="audio/pcm"; samplerate=16000");
After uncommenting that, I was able to upload the audio.
I don't know why that line was commented out in the original sample. (line 45 - https://github.com/Microsoft/BotBuilder-Samples/blob/master/CSharp/skype-CallingBot/Services/MSCognitiveSpeechService.cs)

The V3 intelligence-SpeechToText sample is fixed with this PR: https://github.com/Microsoft/BotBuilder-Samples/pull/1226 It is now using Cognitive Services Speech, instead of the deprecated Bing Speech.


BUG: Fix loading of color volumes into existing volume

Volume type check was incorrect (because a vector volume is a scalar volume, too).
Also added resetting of VoxelVectorType (RGB/RGBA) for vector output volume.
Fixes #215

Thank you Andras 👍

@lassoan I am getting this error with a dataset that has a gap in the file sequence (it is really not an imagestack so the gap shouldn't matter).
I am loading the sequence using the select file option, which shouldn't do any parsing, but use the selected files (to give flexibility for situations like this).

Loading failed: Replacement index 6 out of range for positional args tuple
Traceback (most recent call last):
  File "C:/Users/murat/AppData/Local/NA-MIC/Slicer 5.0.2/NA-MIC/Extensions-30822/SlicerMorph/lib/Slicer-5.0/qt-scripted-modules/ImageStacks.py", line 395, in onLoadButton
    outputNode = self.logic.loadVolume(self.currentNode())
  File "C:/Users/murat/AppData/Local/NA-MIC/Slicer 5.0.2/NA-MIC/Extensions-30822/SlicerMorph/lib/Slicer-5.0/qt-scripted-modules/ImageStacks.py", line 666, in loadVolume
    logging.debug("After downsampling, {} size is {} x {}\n\n{} size is {} x {} ({} scalar components)".format(
IndexError: Replacement index 6 out of range for positional args tuple

There was a syntax error in the message formatting - I've pushed a fix: https://github.com/SlicerMorph/SlicerMorph/pull/217
That error message is displayed if the frames in the image stack are inconsistent (different size, different number of scalar components, etc.). The filename pattern must select a consistent set of frames. If there is no such pattern then you might need to move subset of frames into separate subfolders.


dTest framework update

@david-yz-liu I've made the changes requested to get the sub menu tab for Test Framework to appear again.
Also gotten rid of some merge conflict markers I left behind.
The Test Framework view doesn't currently load because of a number of runtime errors - I tried for a couple of hours to work through these but haven't had much luck. Maybe @WilsonChiang can take a look?
Do we want this entire view to be good to go and loading before we merge this PR?

@david-yz-liu @WilsonChiang

@jeremygoh @WilsonChiang There was a problem in how objects were being passed to the partial.
I fixed the problem here @1663641 (and commented out another problem), but this form really needs a lot of work. Focus on allowing admins to upload at least one test result file, and making sure the updates are actually saved. For now, I don't care if you can only upload one file at a time.

Sorry, just realized I didn't reply to this. Thanks! @WilsonChiang and I will look at that functionality this week.


refactor: 💡 rename input variables to use hyphen-case

BREAKING CHANGE: 🧨 angular_version is now angular-version, file_path is now file-path
PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?
API input variables are in snake_case (not standard)
Issue Number: N/A
What is the new behavior?
API input variables are in hyphen-case (standard)
Does this PR introduce a breaking change?
[x] Yes
[ ] No

Input variables have been renamed.

angular_version is now angular-version
file_path is now file-path

Other information

@allcontributors please add @timdeschryver for Documentation.


Switching Region and Profile

If ~/.oci/config has multiple profiles, the extension doesn't seem to allow switching unless going into the settings and changing the "default profile name" to what is needed.  Changing either the region or profile from VS Code and refreshing the Resource Manager tree seems to load only the stacks from the default profile that was in the extension config.

Thanks for reporting the issue @agregory999. We have fixed the above issue in the version 1.0.3. Please give it a try and let us know if you have any concerns.

Closing this issue as this issue is fixed in the version 1.0.3. Please feel free to open in case of any concerns.

Closing this issue as this issue is fixed in the version 1.0.3. Please feel free to open in case of any concerns.


Extend rename_on_delete behaviour on compaction and view files

This change makes compaction files to stay in .deleted directory in case config parameter rename_on_delete is set and renames them in fashion, that makes it easier to restore them if necessary.
Also rename_on_delete behaviour extended on the views, with renaming ddoc directories of the deleted databases, rather than removing them.

@rnewson @iilyak - the comments addressed.
Regarding rename-then-delete approach, just to move the comment here out of the outdated diff:
As described here https://issues.apache.org/jira/browse/COUCHDB-1003 , this is done because deletion of large files on block-oriented filesystems can take time, so the file first moved into .deleted directory that got cleaned up on couch start.
@rnewson if you think this is not applicable here, I'll remove the rename part, I was just reproducing behaviour of nuke_dir/2 as it was before this PR.

@eiri I'm still confused. The way Cloudant addresses the problem of deletes taking a long time is that we do not call file:delete on it. Instead, we truncate it a few meg at a time until it's 0 length then delete it. Your code does a rename then a full delete, it doesn't achieve the goal.

@rnewson Just curious: what the idea stands behind "truncate by few MiBs toward 0 and then delete" and why it's better than "rename and full delete"? The latter sounds to be use less disk I/O operations and be better.

@rnewson My goal was not change the delete process, but to preserve existing behaviour and the code does exactly what've been done here previously. My understanding from your comment in linked jira is that it was done this way, so if Couch goes down during a long delete the file still be purged on a restart.

I'm +1 with the change. However I want a +1 from @rnewson on this one before we merge.

@kxepal: because the kernel/filesystem will devote itself to freeing the blocks, it causes all other reads/writes to suffer hugely. It is more expensive to repeatedly truncate vs simple delete, but it's spaced out over time so affects other operations considerably less.

@eiri the first PR comment says 'This change makes compaction files to stay in .deleted directory in case config parameter rename_on_delete is set and renames them in fashion, that makes it easier to restore them if necessary. ' but what you've done is rename-then-delete which does not achieve the goal.
I think rename_on_delete might be a misleading name. rename_instead_of_delete is the meaning. Wherever we delete a file, whether that's a database or view compaction that completes or a user deletes a database, we want one of two behaviours: 1) we delete the file immediately (yes, we'll mv it to the .delete directory first so that we can clean up easily in the event of a crash, but that's normal couchdb behaviour) or 2) we rename the file (appending a unique timestamp or similar) and do not delete it, so that it can be recovered. Anyone that sets this config flag is then required to find and delete these files later if they don't intend to restore them (so a useful pattern like *.deleted.couch / *.deleted.view is warranted).
Have I misunderstood the scope of this pull request?

@rnewson Interesting. What would you recommend to read about? (sorry for offtopic)

@kxepal it was an observation from production but I'm sure you can trivially reproduce it. make a 200gb file on ext4, delete it while running any kind of benchmark.

@kxepal We actually stole the truncate-to-zero approach from a talk at one of the devops conferences some years ago. Its purely based on the observation that @rnewson made, when you delete a file, the kernel will consume a large amount of disk resources trying to remove that file. Seeing it in the talk made us realize why we ended up having such a performance impact when a compaction finished. Here's a couple quick links that describe similar solutions:
http://www.awasu.com/weblog/deleting-large-files-from-an-ext3-file-system/
http://serverfault.com/questions/480526/deleting-very-large-file-without-webserver-freezing
The ionice suggestion in the second link I'm pretty sure depends on having a kernel io scheduler configured and we've seen that have a performance impact with anything other than the noop scheduler.

I should've made the point that ionice doesn't work if you're using noop io scheduling.

@rnewson @davisp Thanks a lot!

Hm...reading the links, references and quick googling shows that this problem is extX specific and other FS may handle delete right without IOpain. Feels like this algorithm better to make configurable and notify admins about such problem and the option, but I would like to make some benchmarks for FS I uses for CouchDB: XFS and ZFS. Though for XFS this shouldn't be a problem already.

@rnewson View compaction files handled in couch_file:deleted/3 function which does what the first PR comment describes and you are commenting on couch_file:delete_dir/2 which is called on database's views directories when database deleted while parameter rename_on_delete set to false [ref]
The scope of PR is to extend exiting rename_on_delete behaviour on compaction and view files. I'm doing this by making views directories to be renamed on deletion of a database in the same manner the database file renamed right now. For compaction files I'm making them not to be deleted in .deleted directory after renaming, the same way as Cloudant does. I'm renaming them differently, though, so they'd be more easily identified if necessary.
The renaming configuration parameters or radically diverging from how file preservation works right now is not in scope of this PR.

(noting that ionice doesn't apply to delete operations anyway)

So thinking through this I think we may be conflating two different features that while similar are serving different purposes.
Specifically, there's one feature where we rename things into the .delete directory so that we can get rid of them quickly as well as manage cleanup when the server crashes during deletions. This feature has been in CouchDB for a long time dating back years.
The second feature was a small thing that Cloudant added to help manage recovering from accidental deletions. This feature was implemented by just changing the delete to rename a .couch file to a version that includes the deletion timestamp. This allowed operators to recover a "deleted" database for clients.
This PR appears to attempt to flatten both of these features by renaming files into the .delete directory with useful names and makes the actual deletion a bit subtler in that the various renames are munged and so its not 100% obvious which feature we're handling in the various functions named delete/nuke_dir/delete_file/etc.
As a follow on, the renaming here also tries to move away from using a UUID in the .delete directory and instead uses a URL encoded path of the deleted file (relative to the root database directory). On first glance this seemed like a good idea, but researching filename length limits it'd be very easy to break this as a user only needs to create a database with > 255 characters on extX filesystems (and shorter on others).
What I think we should consider is that all of our deletions move the file to the .delete directory mirroring the original filesystem hierarchy. Then our existing "rename_on_delete" feature is relabeled as "delete_after_rename" with a default of true. This allows us to enable a sysadmin approach to recovering deleted databases as well as enables sysadmins to institute their own policies for actual deletion.
As a last note, I'd also like to see all of the file system/deletion logic moved to couch_file. Its quite awkward to have it split between couch_server and couch_file. I'd work on structuring the commits to have one that moves the existing logic to couch_file and then a subsequent commits to do the filesystem hierarchy mirroring.

@davisp I thought there are performance penalty on deleting a lot of files in deep filesystem hierarchy, I remember @rnewson mentioned this while we've been discussing how rename_on_delete should be done before I started this implementation. That's why I did urlencode trick to keep the file structure flat.
Do I understand correctly that you are proposing instead: 1) to change rename_on_delete config parameter for delete_after_rename 2) keep doing what we are doing with renaming files in uuids, moving them in flat fashion in .delete and removing them if delete_after_rename set to true. 3) mirroring data directory hierarchy in .delete directory, moving there after-compaction, deleted database and view files and disabling init_delete_dir/1 if delete_after_rename set to false
If this is the case, do you think we should add random suffix to moved after-compaction files to keep them distinguished? On one hand this could lead to accumulation of a lot of files in .delete, on a flip hand rapid compaction without suffixing would override previous compactions files that might have some value.
Should we change config parameter to rename_instead_of_delete as @rnewson proposed, since there are two different ways to rename files and strictly speaking Couch admin don't necessary need to know how we are deleting files when we are deleting them for real.
Switching rename_instead_of_delete (or delete_after_rename) during Couch life could lead to an interesting state in .delete directory - do you think this is not a concern?

Companion PR apache/couchdb-couch-mrview#44

Great work @eiri. All tests pass locally.

@davisp I've redone this "database recovery" thing in a fashion we discussed, can you please take a look when you'll have some time? thank you.

Awesome work. +1


Platform specific unwinder support

This needs change in RyuJit/Runtime/OW.

CC @smosier

Fixed via https://github.com/dotnet/corert/commit/72c3eecde5c089b1e21b4f6d38042c2048446bd2


[MM-27231]: cypress test for MM-T1840

Summary

Automate MM-T1840: Description allows for special character

Ticket Link

Part of https://mattermost.atlassian.net/browse/MM-27231

/update-branch


Missing @RabbitHandler method triggers an infinite retry loop

Back port issue of: https://github.com/spring-projects/spring-amqp/issues/2437

Fixed via: https://github.com/spring-projects/spring-amqp/commit/d00be7185bfc72452a21eba524df33f645104cd6


Fix docs

Description

Update example notebook
Fix docs
Add example notebook to docs

Status

[x] Ready to go

@hannahbaumann I'd just sqash merge this one in since the history is a bit all over the place.


Factorize page description for all components

To fix all accessibility issues on component pages
To have same structure for all

To test, all component pages


Get run shortcuts to work properly on Windows

Currently, when you have a multiline input ctrl-enter doesn't execute anymore but just adds lines (you have to press esc before to make it run).
I know this is outlined in the FAQ, but it's really annoying :disappointed: - can it be solved?

Does alt-enter work instead?  Meta is usually the same as Alt, not as Control.
On Unix, Control-Enter and Enter are exactly the same, so it's impossible to know the difference.
But on Windows, we do know the difference, so technically it's possible to bind some behaviour to Control-Enter.

While in multiline mode nothing makes the code to run until I press Esc
(I've tried all combinations with alt, ctrl, shift and enter).
Thanks,
-- Ionel Cristian Mărieș, blog.ionelmc.ro
On Mon, Jan 26, 2015 at 2:57 PM, Jonathan Slenders<EMAIL_ADDRESS>
wrote:

Does alt-enter work instead? Meta is usually the same as Alt, not as
Control.
On Unix, Control-Enter and Enter are exactly the same, so it's impossible
to know the difference.
But on Windows, we do know the difference, so technically it's possible to
bind some behaviour to Control-Enter.
—
Reply to this email directly or view it on GitHub
https://github.com/jonathanslenders/ptpython/issues/14#issuecomment-71456781
.

Hmmmm, I'm using http://mridgers.github.io/clink - could that be a problem?
Thanks,
-- Ionel Cristian Mărieș, blog.ionelmc.ro
On Mon, Jan 26, 2015 at 2:59 PM, Ionel Cristian Mărieș<EMAIL_ADDRESS>wrote:

While in multiline mode nothing makes the code to run until I press Esc
(I've tried all combinations with alt, ctrl, shift and enter).
Thanks,
-- Ionel Cristian Mărieș, blog.ionelmc.ro
On Mon, Jan 26, 2015 at 2:57 PM, Jonathan Slenders <
<EMAIL_ADDRESS>wrote:

Does alt-enter work instead? Meta is usually the same as Alt, not as
Control.
On Unix, Control-Enter and Enter are exactly the same, so it's impossible
to know the difference.
But on Windows, we do know the difference, so technically it's possible
to bind some behaviour to Control-Enter.
—
Reply to this email directly or view it on GitHub
https://github.com/jonathanslenders/ptpython/issues/14#issuecomment-71456781
.


Got Play store note

In my flutter app I'm getting this error in Play Store even though I haven't added this plugin in pubspec.yaml
The SDK provider of com.google.android.recaptcha:recaptcha has added a note for recaptcha:18.1.2. Here's what the SDK provider told us:
A critical security vulnerability was discovered in reCAPTCHA Enterprise for Mobile. The vulnerability has been patched in the latest SDK release. Customers will need to update their Android application with the reCAPTCHA Enterprise for Mobile SDK, version 18.4.0 or above. We strongly recommend you update to the latest version as soon as possible.
How can I fix this.

Are you using firebase auth perhaps?

yes

We will update this here: https://github.com/firebase/firebase-android-sdk/issues/5638


run demo: docker/scripts/release_start.sh but  blocked (运行阻塞了)

i use vmware, ubuntu14.04.
I have successfully install docker. then follow the docs in the demo_guide directory.
when run docker/scripts/release_start.sh
It can successfully download many files.  then run to tell me type 'y' to agree licence. when i type 'y' , it block（ 阻塞在那里了）
then I quit by CTRL_C  and run   "docker/scripts/release_start.sh but  blocked"   again.
AS follow:
root@ubuntu:/opt/APOLLO/apollo-master# bash docker/scripts/release_start.sh
APOLLO_ROOT_DIR=/opt/APOLLO/apollo-master
Error response from daemon: Get https://registry-1.docker.io/v2/: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)_
apollo_release
apollo_release
/opt/APOLLO/apollo-master/data/core/core_%e.%p
[WARNING] Failed to find device with pattern "ttyUSB*" ...
[ OK ] Found device: /dev/ttyS31.
[ OK ] Found device: /dev/ttyS30.
[ OK ] Found device: /dev/ttyS29.
why?

Error response from daemon: Get https://registry-1.docker.io/v2/: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)
how to resolve?

@wanglei828 can you help double check the "BLOCK" issue?
@luckdsfy your second problem looks like a network connection problem.

This problem has been fixed. please pull the latest code.

@wanglei828   I down the latest code and It runs ok now.
but:
when i run:
rosbag play docs/demo_guide/demo.bag --loop
fail msg:  bash: rosbag: command not found
how to resolve?

Are you running the rosbag command in docker or not? If in docker, the rosbag command should work fine. I just tried in my machine.

If you are not in docker, you can try source the ros/setup.bash by
source bazel-apollo/external/ros/setup.bash

And see if command rosbag works or not.

@wanglei828   I follow　the readme in   apollo-master\apollo-master\docs\demo_guide  step by step?
how to  running the rosbag command in docker ?
any tip in the readme?
why rosbag command not founded?

You can follow these steps:

./docker/scripts/dev_start.sh
./docker/scripts/dev_into.sh
rosbag play docs/demo_guide/demo.bag --loop

I run the demo, follow the readme in apollo-master\apollo-master\docs\demo_guide:
./docker/scripts/release_start.sh
./docker/scripts/release_into.sh
rosbag play docs/demo_guide/demo.bag --loop

@wanglei828
I try
./docker/scripts/dev_start.sh
./docker/scripts/dev_into.sh
rosbag play docs/demo_guide/demo.bag --loop
the same fail message:  bash: rosbag: command not found
Is  something i have not install ?
how to install rosbag command ?

@luckdsfy  In dev docker, If you have never built apollo repo,  ros package will not be downloaded into your environment.
Please follow this way:

./docker/scripts/dev_start.sh
./docker/scripts/dev_into.sh
./apollo.sh build
source scripts/apollo_base.sh

^ This works. Thanks


Doesn't show data

Dev Home version
0.601.297.0
Windows build number
10.0.22621.2506
Other software
OS Build Version: 10.0.22621.2506.amd64fre.ni_release.220506-1250
.NET Version: .NET 6.0.23
Steps to reproduce the bug
After adding a widget, it doesn't display data, only default widget view.
Expected result
Displays widget data.
Actual result
Displays default widget view.

Included System Information
Physical Memory: 31.73GB (14.04GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHomeGitHubExtension_0.600.283.0_x64__8wekyb3d8bbwe
Microsoft.Windows.DevHome_0.601.297.0_x64__8wekyb3d8bbwe

Duplicate of #1248


Upstart Service Provider doesn't handle certain status output

Back in May I mentioned in CHEF-5265 that the "status" output isn't well-parsed for some Upstart jobs; now that I've been
Some Upstart services, including "network-interface", take parameters.  Some of them also have hyphens in their names.  In both cases, the UPSTART_STATE_FORMAT regex in Chef::Provider::Service::Upstart fails to match the output of "status ".
The status(8) man page and Ubuntu Upstart docs give the following possible status output:
job start/pre-start, process 902

job start/post-start, process 1234
      post-start process 1357

job start/post-start, (post-start) process 1357

job (tty1) start/post-start, process 1234
      post-start process 1357

In addition, I see this output from "status network-interface INTERFACE=eth0":
network-interface (eth0) start/running

The current regex is :
UPSTART_STATE_FORMAT = /\w+ (?(\w+))?/ /
The "\w" doesn't match hyphenated job names, and the regex doesn't work when there's an instance (like "eth0" in the status output.  I think that this:
UPSTART_STATE_FORMAT = /\S+ (?:\(\S+\) )?([\w-]+)\/([\w-]+)/

would work.  Anyone have counterexamples where this won't work, or shall I go ahead and submit a PR?  (CLA already signed.)

PR would be useful to get it fixed.  In general this bites us all the time and regexps should err on the side of being permissive.  I think your regexp looks fine, but I've also got a headcold so I might be missing something.

@tarrall hey, we would very much like the PR if you'd still like to submit one.

This was fixed by #4160


Temporarily remove satisfied clauses from watchlists

This is an idea I have for optimizing unit propagation. It tries to avoid work in the following scenario:

A clause C = x, y1, ..., yn is satisfied by a single literal x which was assigned on a fairly low decision level
It's other literals y1, ..., yn get repeatedly assigned false and unassigned again on deeper decision levels

I need to benchmark how often this happens, but I figure it should be common.
The standard algorithm would repeatedly encounter the clause C while scanning the watchlists for y1, ..., yn. Blocking literals (already implemented) make sure that the second time this happens, we don't have to access the clause itself, as the watch will contain x, which is true, as blocking literal.
I think it's possible to further improve this by temporarily moving the watch to a inactive-watches list associated with x. During backtracking when x is unassigned the inactive-watches list is then emptied and the watches are moved back.
I can also imagine this to be especially useful for solving with assumptions, where we essentially never unassign some literals.
This might have synergies with #20.

While this does happen frequently (seems to be 10% to 50% of watches which have a true blocking literal for the instances I looked at) it doesn't happen often enough in a row to outweigh the overhead of moving the watch somewhere else before moving it back.


Refator(LSP): refactor lsp frame.(#297)

remove tower lsp and use lsp_server instead
add vfs in lsp to detect file changes in client
refactor lsp, the new version is based on event polling,  perpare for incrementalized compilation
move find_ref module to LSP crate

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2106

Relevant Lines:
2331

💛  - Coveralls


Bug fix/fix spin locks

Scope & Purpose
Add missing rollback of state changes when an attempt-based write operation on a spin lock fails.

[x] Bug-Fix for devel-branch (i.e. no need for backports?)
[x] The behavior change can be verified via automatic tests

Testing & Verification
This change is already covered by existing tests, such as all.
http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/10990/

Not ready to merge yet: I had added an assertion, but this does not work in the arangodbtests unit tests context.

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/11005/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/11007/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/11013/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/11017/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/11024/


[OpenAI] Using OpenAI > 1 fails due to import errors

The main readme states that you should be able to use OpenAI > 1 in conjunction with DSPy but there's dirct access to .errors which no longer exists:
) /usr/bin/env /Users/mikeldking/.virtualenvs/phoenix/bin/python /Users/mikeldking/.vscode
/extensions/ms-python.python-2023.22.1/pythonFiles/lib/python/debugpy/adapter/../../debugp
y/launcher 56834 -- /Users/mikeldking/work/openinference/python/instrumentation/openinfere
nce-instrumentation-dspy/examples/dspy_rag.py
Traceback (most recent call last):
  File "/Users/mikeldking/.pyenv/versions/3.10.3/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/Users/mikeldking/.pyenv/versions/3.10.3/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/Users/mikeldking/.vscode/extensions/ms-python.python-2023.22.1/pythonFiles/lib/python/debugpy/adapter/../../debugpy/launcher/../../debugpy/__main__.py", line 39, in <module>
    cli.main()
  File "/Users/mikeldking/.vscode/extensions/ms-python.python-2023.22.1/pythonFiles/lib/python/debugpy/adapter/../../debugpy/launcher/../../debugpy/../debugpy/server/cli.py", line 430, in main
    run()
  File "/Users/mikeldking/.vscode/extensions/ms-python.python-2023.22.1/pythonFiles/lib/python/debugpy/adapter/../../debugpy/launcher/../../debugpy/../debugpy/server/cli.py", line 284, in run_file
    runpy.run_path(target, run_name="__main__")
  File "/Users/mikeldking/.vscode/extensions/ms-python.python-2023.22.1/pythonFiles/lib/python/debugpy/_vendored/pydevd/_pydevd_bundle/pydevd_runpy.py", line 321, in run_path
    return _run_module_code(code, init_globals, run_name,
  File "/Users/mikeldking/.vscode/extensions/ms-python.python-2023.22.1/pythonFiles/lib/python/debugpy/_vendored/pydevd/_pydevd_bundle/pydevd_runpy.py", line 135, in _run_module_code
    _run_code(code, mod_globals, init_globals,
  File "/Users/mikeldking/.vscode/extensions/ms-python.python-2023.22.1/pythonFiles/lib/python/debugpy/_vendored/pydevd/_pydevd_bundle/pydevd_runpy.py", line 124, in _run_code
    exec(code, run_globals)
  File "/Users/mikeldking/work/openinference/python/instrumentation/openinference-instrumentation-dspy/examples/dspy_rag.py", line 1, in <module>
    import dspy
  File "/Users/mikeldking/.virtualenvs/phoenix/lib/python3.10/site-packages/dspy/__init__.py", line 1, in <module>
    from dsp.modules.hf_client import ChatModuleClient
  File "/Users/mikeldking/.virtualenvs/phoenix/lib/python3.10/site-packages/dsp/__init__.py", line 1, in <module>
    from .modules import *
  File "/Users/mikeldking/.virtualenvs/phoenix/lib/python3.10/site-packages/dsp/modules/__init__.py", line 2, in <module>
    from .gpt3 import *
  File "/Users/mikeldking/.virtualenvs/phoenix/lib/python3.10/site-packages/dsp/modules/gpt3.py", line 7, in <module>
    import openai.error
ModuleNotFoundError: No module named 'openai.error'

Happy to open a PR to remove this error.

Hmm are you perhaps using an old version of dspy?

Hmm are you perhaps using an old version of dspy?

I'll try to repo in a clean environment. Will PR with a possible fix if I figure it out!

Basically that line is not there on main now at least. I think it's also not pypi latest

I assume this was an old version of DSPy but happy to reopen if not


NSPDK descriptors are different for the same objects at different script runs

Configuration:

EDeN version: main, from master
Python version: Python 3.7.3
Operating system: MacOS 10.14.3
Are you using conda? Yes, I do

Description:
I have a training and test sets with different graphs. I created Vectorizer object and did transform on the training set. Then saved Vectorizer object to pickle (by myself or by save-method). Then I load this pickle (by myself or by load-method) with Vectorizer object and use transform-method on my test data.
But for the same data in different runs I have different descriptors.
Here is the code to reproduce:
First script for training data:
>>>import networkx as nx
>>>from eden.graph import Vectorizer
>>>import pickle
>>>graph = nx.Graph()
>>>for i in range(3):
>>>    graph.add_node(i, label="C")
>>>    if i != 2:
>>>        graph.add_edge(i, i+1, label='1')
>>>vec = Vectorizer(complexity=3, nbits=8, normalization=False, inner_normalization=True)
>>>res = vec.transform([graph]).todense()
>>>vec.save(model_name=save_path)
(or
>>>with save_path.open("wb") as f:
>>>    pickle.dump(vec, f)
)

Second script for test data (for simplicity in the example I will use the same graph):
>>>import networkx as nx
>>>from eden.graph import Vectorizer
>>>import pickle
>>>graph = nx.Graph()
>>>for i in range(3):
>>>    graph.add_node(i, label="C")
>>>    if i != 2:
>>>        graph.add_edge(i, i+1, label='1')
>>>vec = Vectorizer(complexity=3, nbits=8, normalization=False, inner_normalization=True)
>>>vec.load(load_path)
(or
>>>with load_path.open("rb") as f:
>>>    vec = pickle.load(f)
)
>>>print(vec.transform([graph]).todense())

And if I run this script multiple times, I have different descriptors:
First run:
[0.         0.         0.         0.         0.         0.
0.         0.40824829  0.         0.         0.31622777 0.
....
Second run:
[0.         0.         0.         0.         0.         0.81649658
0.31622777  0.         0.         0.         0.         0.
...
And ect.
Please help, what's wrong?

All problems are from python3. Will be no problem if you will use python2.
Because now everyone use python3, I have created Docker with python2 for the convenience of generating NSPDK features.
This docker gets txt- or smi- file with molecules SMILES and save generated nspdk-features to txt file.
Docker image name: nspdk-features
Repo on GitHub: https://github.com/aigulkhkmv/nspdk-features
Docker on DockerHub: https://hub.docker.com/repository/docker/aigul/nspdk-features

All problems are from python3. Will be no problem if you will use python2.
Because now everyone use python3, I have created Docker with python2 for the convenience of generating NSPDK features.
This docker gets txt- or smi- file with molecules SMILES and save generated nspdk-features to txt file.
Docker image name: nspdk-features
Repo on GitHub: https://github.com/aigulkhkmv/nspdk-features
Docker on DockerHub: https://hub.docker.com/repository/docker/aigul/nspdk-features

All problems are from python3. Will be no problem if you will use python2.
Because now everyone use python3, I have created Docker with python2 for the convenience of generating NSPDK features.
This docker gets txt- or smi- file with molecules SMILES and save generated nspdk-features to txt file.
Docker image name: nspdk-features
Repo on GitHub: https://github.com/aigulkhkmv/nspdk-features
Docker on DockerHub: https://hub.docker.com/repository/docker/aigul/nspdk-features

Problem solved


Refactor: func Save in package pkg/cmd/image shouldn't create file from options.Output

pkg side should assume a StdOut is given (i.e., not create a file from options.Output)

Signed-off-by: Yakun Jing<EMAIL_ADDRESS>

What I'm a bit unsure about is whether we need to move the following code into the main package of command?
https://github.com/containerd/nerdctl/blob/bba23d57744dbdd14c7674efab3e743600a3ba1f/pkg/cmd/image/save.go#L35-L44

What I'm a bit unsure about is whether we need to move the following code into the main package of command?

Yes this should be part of the cmd side (i.e., cmd/nerdctl/image_save.go). pkg side should assume a StdOut is given (i.e., not create a file from options.Output)

close #1940 .


Impoving react-router-config

I have 2 features to propose:

Do not force users to use renderRoutes inside components. Instead of that renderRoutes can be something like this:

<route.component {...props} {...extraProps} route={route}>
    {renderRoutes(route.routes)}
</route.component>

Minus one dependency in component.
2. Render routes by itself if no component was passed
{route.component && (
    <route.component {...props} {...extraProps} route={route}>
        {renderRoutes(route.routes)}
    </route.component>
)}
{!route.component && (
     {renderRoutes(route.routes)}
)}

That will help if I configure router not in one file.
const routes = [{
    path: '/',
    component: App,
    routes: [{
        path: '/bar',
        routes: require('./bar)
      }, {
        path: '/baz',
        routes: require('./baz')
    }]
}];

Files bar.js and baz.js export some routes structure.

I'm not sure what you're asking there, but I believe you're looking for matchRoutes or something like that. I'd like to refactor renderRoutes to make use of matchRoutes, so this will likely be covered by that change.

Is basically the same as #5429. We should back out of the subtree when traversing the routes.

No. That I'm asking is not using reactRouter it route component. Example:

const routes = [{
    component: Root,
    routes: [{
        path: '/child1',
        component: Child1
    }, {
        path: '/child2',
        component: Child2
    }]
}];

const Root = ({route}) => (

Root

    {renderRoutes(route.routes)}

);

const Child1 = () => (

Child1

);

const Child2 = () => (

Child2

);

Instead of that I what something like this:
const routes = [{
    component: Root,
    routes: [{
        path: '/child1',
        component: Child1
    }, {
        path: '/child2',
        component: Child2
    }]
}];

const Root = ({children}) => (

Root

    {children}

);

const Child1 = () => (

Child1

);

const Child2 = () => (

Child

);

How's that connected?

https://github.com/ReactTraining/react-router/issues/5429 describes route without path, but with component. I want oposite of that.

In your example, there's nothing connecting a plain array to the rendering of a component. There's no way for us to get involved there.

It's related because it talks about backing out of a subtree to match on another subtree. To fix that, we need to be able to back out of the recursion. Once we can do that, we can back all the way out to the root and render a "404" component when nothing matches.

Of course you need call renderRoutes once in root of your app like this.

ReactDOM.render((
  <BrowserRouter>
    {renderRoutes(routes)}
  </BrowserRouter>
), document.getElementById('root'))

But why do I need to use it in other components? Why can't renderRoute pass render it by itself and pass rendered children to my component?

Ah, I see what you are saying. The main issue is related to the refactor I mentioned. We're basically just wrapping a <Switch> at the moment and I would like to refactor to wrap matchRoutes instead, allowing us to traverse the entire tree and back out of subtrees as well. This is all related to a big refactor of renderRoutes.


use container started events instead of pod updates

This should eliminate the need of keeping internal state by reacting
on container started events for pods instead.

Great work @0robustus1! And thanks @maxlaverse for the review.
Small fix (not urgent): https://github.com/xing/kubernetes-oom-event-generator/pull/5


Projects are empty

Hi,
First of all: I love the theme. Really wonderful work. Only have one issue with it: if I switch a project, the sidebar is completely empty. No folders are shown anymore. If I drag a project folder onto the Sublime Text icon, it is opened correctly. But as soon as I use CTRL+CMD+P to switch the project, the sidebar is empty as shown in the screenshot. Any idea what this can be?

Hmm. That's probably not a problem with Predawn. It looks like you might have renamed the directory your project is in or something? Maybe try Project > Refresh Folders and see what happens?

Hey Jamie,
I did this already. I am not sure if this is a problem of Predawn, but it never happened before the installation. And it is not only one project, but all I try to switch to. If I quit Seublime text and restart it, all folders and files are there again.

Darn. Well, what happens if you open a .sublime-project file without using ctrl-shift-p?

Found what it was! It wasn't your theme indeed. I somehowmust have switched on "Show Open Files" just for one project. If that is set differently in ST3, then it causes this bug. Found the solution here in the post by "sentience" a bit down in the thread. But thanks for responding, Jamie.

No problemo!


Update SWELL for R2D2 Version 2.  Three components to this update:

API calls.
Config restructure (no longer a r2d2 config yaml).
SWELL must work in conjunction with a MySQL server.  We can install this "locally" (on Discover) or leverage the AWS cloud server for this.

Some, but not necessarily all, issues:

Do we need to account for fc_date_rendering (e.g. store_background.py)?  TBD.
window_length for observations (e.g. save_obs_diags.py and obs_process_setup.py?
model not specified in old API calls (e.g. obs_process_setup.py bias correction), just use 'geos' here?
Shall we preregister metadata in the database (e.g. experiments, models, etc.)?  Or should SWELL code register them on the fly?

Every API call must be successfully tested before considering a merge into develop.

Thanks for starting these changes @tariqhamzeyssai. I'm on PTO next week so we'll have to discuss the week afterwards. I suspect we can neglect fc_date_rendering. We'll need window_length I think. I understood that R2D2 will eventually prepare a file with all the observations for that window length. Why is 'model' needed when getting observations? I wouldn't imagine that would depend on the model. Swell should be able to add outputs to the database. But we should pre-register all the inputs. At the moment they are all in /discover/nobackup/drholdaw/R2D2DataStore. Could we start copying files from there to the database to start testing the new code?

If you want to run the coding norms locally you can issue python pycodestyle_run.py from the root directory of the swell repo.

Why is 'model' needed when getting observations? I wouldn't imagine that would depend on the model.

We're getting bias correction. Is that not model-specific?
EWOK scripts were specifying model for R2D2V1 calls, as it is for V2.  I can't log into Discover at the moment to view your R2D2 build, but I think the old 'bc.yaml' file validated model with no default.  It is not immediately clear how SWELL bc fetches were passing without model.


push not working (no errors on dovecot side)

I installed the software according to documentation (dovecot module and xapsd-daemon, extracted certificate from OS X server / working connection to apple push server).
I see no errors in the log, everything seems to be working. But no push is received by the iOS device.
This is the log from the xaps-daemon:
2016/04/18 17:48:49 [DEBUG] Accepted a connection
2016/04/18 17:48:49 [DEBUG] Received request: NOTIFY dovecot-username="mhostinsky"      dovecot-mailbox="INBOX"
2016/04/18 17:48:49 [DEBUG] Sending notification to 023C540F-6BA7-4D15-AF27-6FAE230EEF8D / C7E48933EFC239A633FB56846C148B9260C79D9925732A683C0272F4C9436D16
2016/04/18 17:48:49 [DEBUG] Sending notification to E2666B84-A0EF-4883-A745-7AE01BB23C8E / C7E48933EFC239A633FB56846C148B9260C79D9925732A683C0272F4C9436D16
2016/04/18 17:48:49 [DEBUG] Sending notification to 79D8C83C-70A6-4253-AAE0-6A9E73921E9A / CCDB9BA57CB6C5818ED5972DAE03CC03809D93E1849A192BEBFF26DDBE488444
2016/04/18 17:48:49 [DEBUG] Returning success:
I have several iOS devices configured, but it didn't work either with one device.
Is this software still functional?

Did you remove the mail accounts and reinstalled them ? Because I had to do it to make sure they register properly.
Also check the registry json db to ensure your devices appear there.

First to your last question: Yes, the plugin still works fine (at least on my server).
Looks to me like this is one of this cases where you depend on Apples Push Service to get it right. This usually solves itself after some hours/days. I had the same strange behavior when I first set this up.
Your log output indicates that you are delivering notifications for 3 devices to Apple. So from there on it depends on their push service.
Voodoo Tips that might improve the situation or might do nothing at all:

Reboot the receiving devices (But as they have registered themselves to your server they seem to be aware that the mailserver knows Push)
Add your mail domains to the OS X Server instance that generated the cert (my theory is that the OS X Mailserver is passing information about "known domains" to Apple. I never verified this.)

I solved it. It seems that there is an issue with Dovecot (2.2.10) shipped with Centos7. I upgraded Dovecot to 2.2.19 (from GF Repo) and it works!
This issue someone mentioned in some issue of xaps-plugin.
So Dovecot 2.2.15 and later should work.
BTW, xaps is totally awesome. Finally I found solution for push email using native iOS client.


Unhandled exception when column header name contains a colon (:) character

The code apparently assumes a certain naming convention for column header names where a data type can be specified using the notation "What is your age?:int".  However, the logic throws a TypeLoadException during a call to DelimitedRecordReader.ReadColumnHeaders() if the column header name contains a colon (:) that does not specify a valid type name. i.e. "What is your age : as of 31 Dec?". If the implied type name throws a TypeLoadException, it is probably safest to assume the type should be a string.

You can change the type separator character by setting the property ColumnHeaderTypeSeparator. Would that solve your issue?

I still think there should be a try-catch block surrounding the call to try to load an assembly based on an assumed type request from the header.  If an exception is thrown, default to string. I think this increases robustness.  I just disabled the feature for now to keep moving forward. I'll circle back at some point. The vendor I get data from has an API available so I'm looking to migrate away from exporting CSV files completely.

Or as in my case cause the file to fail to load and render the file useless. I don't control the headers. My feature is to allow users to map fields dynamically so they don't have to adhere to a specific file format, column order, header names, etc.


Enable authentication for firebase RTDB

Generates an access token using Google's REST authentication endpoint (email authentication)
Attaches the access token to every request thereafter using the "auth" url parameter
See https://firebase.google.com/docs/reference/rest/database/
See https://firebase.google.com/docs/reference/rest/auth

ctx = ussl.SSLContext()
SSLContext is not supported by ussl (at least on micropython)
How can i download a pycopy bin for my esp32 ?

I will finish the readme in the next few days

Alright cool, thanks for the merge/overhaul.