Active Learning not working

I followed the steps mentioned here but Active Learning doesn't seem to work. I posted it on Stackoverflow: https://stackoverflow.com/questions/54827888/active-learning-suggestions-in-qnamaker to get some techical. I have mentioned the details in the Stackoverflow thread. Is there any help I can get here?

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

ID: e302f635-c2b9-e505-7b9e-d947dab75514
Version Independent ID: 2c1e8216-c9d5-68bd-cbca-6c80dcb2f2b3
Content: Improve knowledge base - QnA Maker - Azure Cognitive Services
Content Source: articles/cognitive-services/QnAMaker/How-To/improve-knowledge-base.md
Service: cognitive-services
GitHub Login: @diberry
Microsoft Alias: diberry

@Souvik04 Thanks for the feedback. We are investigating into the issue and will update you shortly.

@Souvik04 There could be two reasons why the suggestions are not shown:

Active Learning is supported for run time version of 4.4.0 and above. Please check the application settings of your app service to check if the correct version is used for the service in azure portal.

There could not be enough answers returned by QnA Maker for a given query. If the score differences lie within a small range, then the query is considered a possible suggestion for each of the possible answers.

@RohitMungi-MSFT Thanks for the response.

I checked the app settings of the QnAMaker service in the Azure portal. The only relevant key-value I found is:

QNAMAKER_EXTENSION_VERSION:latest

Is there a way I can test this with the chit-chat knowledge base (which comes as an option while creating the KB) of the QnAMaker?

@Souvik04 the run time version can be checked on the App Services' application settings page as seen in this screen grab.

You can test with the chit-chat KB using the Test option in qnamaker.ai for your KB, A screen shot of the same is also attached for a test KB I created.

I hope this helps you in testing your KB and it provides enough data for Active Learning to provide suggestions as it is integrated with your application.

@RohitMungi-MSFT I checked the version. It is v 4.7 same as yours. I have some confusion understanding how the Active Learning works. Could you please take the chit-chat (professional) and explain it through some example questions? So that I can try the same here and see if those gets added as suggestion.
I tried some questions (similar to few existing question) in the Test window of QnAMaker.

The data in my Chit-chat KB is like this where I was expecting the new question suggestions to get added.

I kept hitting with same similar set of questions and waited for about 20 minutes but I didn't see the suggestions getting added. Please let me know if I can provide some more details.

@Souvik04 Here is a github repo which simulates active learning for a test bot. I was able to follow the steps and get some suggestions for sample queries, screen shot below.

Please ensure the qnamaker-activelearning.bot is correctly updated with your endpoint key, hostname, KBid and subscription key.
You can find the endpoint key, hostname and KBid in the settings page of your KB in qnamaker.ai page.

The KB id would be the id in the POST request you see in the sample deployment details of the setting page.
Subscription key is available in azure portal from the keys tab of your QnA service

Please ensure you setup the pre-requisites correctly and login to your azure account where the QnA service is created from the emulator so that the QnA emulator can connect to your endpoint when queries are entered.
I have also tested by adding some editorial content in my KB from qnamaker.ai portal, Screen shot below:

Then tested from the desktop emulator with the following questions.

aws
what is gcp
what is windows azure

For the last question the bot checked what I actually meant by providing suggestions since "windows azure" is not added in my editorial content. Screen shot below.

However, I'm still not able to view suggestions in qnamaker.ai portal. I will pass my findings to the product team to understand when the qnamaker.ai portal will display the suggestions and update this issue with their response.
I hope the above steps will help you in getting started with QnAMaker!!

Related issue #24709. Suggestions are not showing up in the QnaMaker portal.

@Souvik04, follow the link to the Active Learning sample bot in the BotFramework-Samples repo for a example of how you can query the QnA service from your bot with active learning enabled.

After conversing with the QnA team (Rohit is included in the conversation), here's a little more light regarding when you would actually see the suggestions inside the portal at qnamaker.ai.
When there is a low confidence score difference between the top answers, we collect weighted implicit and explicit feedback to cluster suggestions for any QnA ID.
=> When enough feedback is collected for any given suggestion, it will show in the KB.
More specifically, we cluster similar user queries to generate suggestions. When minimum required feedback is collected, only then will the suggestions show in the KB.
The QnA team wants to avoid publicly divulging the exact logic of what exactly is the "minimum required feedback" and how often suggestions are generated (besides, the team is working on improving and optimizing the logic behind active learning as well)
--however to see suggestions appear in the qnamaker.ai portal:

not only ensure that you've given the bot enough feedback
but also give the back end "some time" to allow for the suggestions to appear in the portal.

Again, feedback is collected when your user types in a query that returns answers from QnA that have confidence scores that are close together.
It is also good to note that feedback is not collected in the Test panel in the qnamaker.ai portal as of now. You will need to chat with your bot via emulator or a channel to provide feedback to your bot that it can use for active learning.

@Zerryth Thanks for updating the issue. @Souvik04 and @iMicknl Please use the active learning sample bot and train your KB with the steps I mentioned before. If the "Did you mean" dialog shows up in the emulator provide the relevant option as feedback and this will ensure the system captures them, When there are enough such similar feedback they are shown as suggestions in the KB on the portal. If you accept the suggestions please ensure to publish the KB again and check if it shows the correct answer without additional prompts. Here are the screen shots of the above question i asked and it shows them as suggestions.
Suggestion in Portal:

Response from bot for before and after accepting & publishing the KB:

I will now go ahead and close this issue. We will be updating the document as new features are rolled out and please continue to let us know your feedback. If there are further questions regarding this matter, please tag @RohitMungi-MSFT in your reply. We will gladly continue the discussion and we will reopen the issue.

@RohitMungi-MSFT and @Zerryth Thanks for explaining it and the samples. I had previously looked into the Bot Builder samples github link. I was able to get the suggestions in the portal. The sample some amount of code which basically generated the Hero cards and takes the feedback. I was under the impression that I can simply query the knowledge base with QnAMaker library, specifying the top and score options and I will get the Hero cards out of the box, which can be used as feedbacks. The Active Learning document links doesn't explain this at all, at least not in a way a user can pick up easily in my opinion.

@Souvik04 We are working on updating the document with the findings of this issue and suggestions from another issue 24709. These updates will provide clarity on these steps and future service updates.

@RohitMungi-MSFT Thank you for the update.

Seeing the same problems too. It used to work on 4 endpoints and now all 4 have stopped serving suggestions.

@paulio - please submit a user voice issue from the LUIS portal "General Inquiry through user voice" and include your LUIS app id.

We have a support call logged I just wanted to comment because it seems like identical symptoms

hello, I am having the same problem, I have raised the Active Learning example https://github.com/microsoft/BotBuilder-Samples/tree/master/experimental/qnamaker-activelearning/csharp_dotnetcore and followed the steps for its implementation to discard any problem in my code, but I have not managed to work, I can not get the bot appear sugenrencias for example to "Surface Pro 4", not in my KB appear suggestions, some help? I have tried different options and I have not succeeded.
Thank you very much for your help!

hello, I am having the same problem, I have raised the Active Learning example https://github.com/microsoft/BotBuilder-Samples/tree/master/experimental/qnamaker-activelearning/csharp_dotnetcore and followed the steps for its implementation to discard any problem in my code, but I have not managed to work, I can not get the bot appear sugenrencias for example to "Surface Pro 4", not in my KB appear suggestions, some help? I have tried different options and I have not succeeded.
Thank you very much for your help!

hello @RohitMungi-MSFT any update for the issue?
@paulio they got some advance with the support call
Thank you


Did wasmlink Move Again?

Hey there!
I'm looking for wherever https://github.com/peterhuene/wasmlink went, if it still exists.
I'm trying to link a WASM module compiled from C with a WASM module compiled from Rust, and be able to call the C functions from Rust.

wasmlink was removed in https://github.com/bytecodealliance/wit-bindgen/pull/240 as it's being replaced by the component model. It was historically used for different purposes of possibly polyfilling the component model but a different route is being taken for that now.

@zicklag wasm-compose will offer some of wasmlink's functionality, namely being able to compose a component from other components. The remaining functionality of generating adapter code has been supplanted with support for the component model proposal in Wasmtime itself.
Currently the wasm-compose example is all Rust, but it shouldn't be that difficult to use current wit-bindgen to replace one of the components with an implementation in C.
I'm currently working on improving wasm-compose for general use.

Awesome, thank you both for your links and quick responses!


Timing of bicep upgrade message

When I run az deployment group create I often get a message like A new Bicep release is available: v0.5.6. Upgrade now by running "az bicep upgrade".
It always annoys me because I cannot run the suggested upgrade while the terminal is blocked waiting for the deployment. I haven't tried to open another terminal to run the upgrade in parallel, because I am not sure if it is safe to do so.
When the deployment finishes, I have forgotten about the update, and the upgrade message gets scrolled out of view by the deployment success/error message. So I forget to upgrade and receive the same upgrade warning message again and again.
I would suggest to move the message so it appears on screen below the deployment success/error message and appears just when the command completes.

Need to investigate if this is even possible, but we agree with the frustration :)

@alex-frankel I think there's a couple of issues with this, take converting bicep to json as part of the build (for whatever reason, but say, because Azure SDK does not support deploying bicep):

reaching out for external resources during a build phase (this is against MS's own internal practices).
it is reported as an error (returns with -1).
the suggested solution to opt-out (az config set bicep.check_version=False) is the opposite of good default, as no one would know about this until it breaks the build.
the alternative - auto-upgrades as part of a pipeline would mean unreproducible builds.

I think we should strive towards hermetic/reproducible tooling configuration by default and the behaviour should at very least change to warning (0 return code), but ideally also require opt-in to checks, instead of opt-out of them.

Not dismissing your points, but I am seeing this as a warning, not an error:

Are you consistently able to repro this as an error? Is it possible it is related to your CI setup?

I was fiddling with this half a day today, the problem was with the msbuild Exec task that I wanted to run as pre-build target. I thought it was the error exit code, cause build was reporting it as error, but it turned out due to how the task was interpreting the outputs/streams. If anyone needs to know, this is what works: add IgnoreStandardErrorWarningFormat="true" ContinueOnError="WarnAndContinue" to avoid having it break the build.


Add capability to compute counterfactuals per point

Description
To be able to compute counterfactuals from RAI dashboard we need the backend code in counterfactual_manager to be able to generate counterfactuals and feature importance per point.  This PR adds that functionality and related unit tests.
Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #1561 (0c4bca0) into main (22ede53) will decrease coverage by 9.54%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1561      +/-   ##
==========================================
- Coverage   85.12%   75.58%   -9.55%
==========================================
  Files          57       10      -47
  Lines        2877      258    -2619
==========================================
- Hits         2449      195    -2254
+ Misses        428       63     -365

Flag
Coverage Δ

unittests
75.58% <ø> (-9.55%)
:arrow_down:

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

Impacted Files
Coverage Δ

raiwidgets/raiwidgets/dashboard.py

...iwidgets/raiwidgets/explanation_dashboard_input.py

erroranalysis/erroranalysis/_internal/utils.py

raiwidgets/raiwidgets/fairness_dashboard.py

raiutils/raiutils/data_processing/__init__.py

raiutils/raiutils/webservice/webservice.py

.../raiutils/data_processing/data_processing_utils.py

...ranalysis/erroranalysis/_internal/cohort_filter.py

raiwidgets/raiwidgets/error_handling.py

raiwidgets/raiwidgets/cohort.py

... and 37 more

Continue to review full report at Codecov.

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


fix: update and cache archlinux-keyring

Relates to #73
Hoping that caching this in the Docker image will reduce subsequent build times.

closing since issue got fixed in base arch-linux docker image


Create a logo for DevOps community

Submit logos for the review.
Whichever, Community loves the most will be set as official logo for DevOps Community.
I want to open-source everything from community side, This is your chance to contribute.
Note: Create logo from scratch, Don't use copyright or template images.

Submit the PR for this @cosmicqbit

Shall I include original files of the logo in PR?

Yes, Please.

Alright, here is the download link.

Thank you, Issue closed


Visual Novel Hooker does not attach.

I have tried a few Visual Novels and none of them seem to hook into the game. After selecting the process no hooks are found even after advancing several lines of text. Adding custom hook codes does not populate the list of hooks. It seems it is not attaching to the game at all. Is there a hotkey to start the injection process? Textractor and copy to clipboard work fine but I need the Game2Text hooker for ease of making anki cards.
I have tried:
Replacing the textractor cli with the latest version
Using three different browsers
Older versions of Game2Text
Multiple Visual Novels that do work with Textractor GUI
Set the path inside Game2Text of the Textractor cli

Try the debug version and let us know of any errors or warnings in the console.


openai: keep text consistent with UI

Update the UI text in articles/ai-services/openai/includes/python.md to keep the text consistent with UI.

@mrbullwinkle - can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Embed an SVG in another

Is it possible to add a Fragment inside of another one ?
Let's say I've got two SVGs, one of a landscape, the other one is a tree. I'd like to put my tree inside of my landscape, at a specific place, defined by a rect.
I tried several solutions, none of them seem to work. I tried turning my tree SVG into a pattern, then using it to fill my rect, but it seems like I cannot turn a Fragment into a pattern.

    const element = Snap('#painting')

    Snap.load(landscapeUrl, function (landscape) {
      if (element) {
        element.add(landscape);
        Snap.load(treeUrl, function (tree) {
          if (element) {
            const rect = landscape.select('rect').getBBox();
            var s = Snap(0, 0)
            let image = tree.select('image')
            var image = s.paper.image(design, 0, 0, rect.width, rect.height);
            image = image.pattern(0, 0, rect.width, rect.height);
            landscape.select('rect').attr({fill: image})
          }
        })
      }
    })

Is there a way to achieve this ?

Should be possible without a pattern, as long as element is a container object, like group,svg etc. Maybe if you put up links to the images or whatever it may help.

@ibrierley I edited the snippet with the URLs. When you say "element is a container object", I assume you're talking about the rect in which I'd like to display the 2nd SVG ?

Hi, yes, I guess you are trying to use a pattern because a rect can't have a child element, which is one approach, but not sure why you need a pattern...why not just place the image on top of the other image ?

My first SVG includes several layers, some of which are supposed to go over the second one.
I managed to accomplish what I wanted, although I stumbled upon some weird things.
First, the difference between a Fragment and an Element is quite blurry.
Second, I had to use a weird workaround to access my SVG. If I load an SVG, I cannot access it directly.
loadedFragment.select('svg') // returns null
loadedFragment.select([any first-depth element in my SVG]).parent() // returns my SVG

in the end, this was what I had to use in order to get a proper display, scale after my rect

	            const rect = baseProduct.select('rect').getBBox();
	            var s = Snap(0, 0)
	            let test = Snap(rect.width, rect.height);
	            let svg = baseProduct.select('rect').parent()
	            baseProduct.select('rect').attr({fill: 'transparent'})

	            var spot = Snap.parse("<g id='designGroup'></g>");
	            svg.add(spot, rect.x, rect.y, rect.width, rect.height)
	            design.select('g').parent().attr({
	              y: rect.y,
	              x: rect.x,
	              height: rect.height,
	              width: rect.width,
	            })
	            baseProduct.select('#designGroup').add(design)
	            element.add(baseProduct);


Remove double MaterialApp wrap from api samples

The widget under test already contains a MaterialApp, so there's no need to wrap it again with one in the test. In fact, the additional MaterialApp could hide problems in the widget under test.

@bleroux @TahaTesser Can I ask you to watch out for this pattern in reviews of API sample tests? If the widget under test already includes a MaterialApp we don't want the test to double-wrap it in another MaterialApp in the test. This can inadvertently hide problems in the API sample code (as an example see https://github.com/flutter/flutter/pull/150018).

@bleroux @TahaTesser Can I ask you to watch out for this pattern in reviews of API sample tests? If the widget under test already includes a MaterialApp we don't want the test to double-wrap it in another MaterialApp in the test. This can inadvertently hide problems in the API sample code (as an example see #150018).

That's for the reminder. We'll watch out for this.


[W-11392825] - sa11y testing for NPSP - Donation History + Giving Summary

Technical Document
Technical Design Review - W-11392825 - NPSP
Changes
Adding sa11y jest test for components and ensure the tests are passing and the accessibility of the components is standards compliant.
Work Item
W-11392825

It seem like givingSummary component is not passing sa11y accessibility test. Can you also address that?

@andrewyu-salesforce Thank you, Andrew. Since sa11y was passing locally in Jest, I had not noticed the github action was failing. I have fixed the aria-role-parenting issue and can see the tests for both Donation History and Giving Summary are passing.
Please let me know if you notice anything else :)


phpfpm: allow old config format as well

Motivation for this change
The restructuration of the config broke everyone using phpfpm. This commit allows the old config format to be used.
Things done

[x] Tested using sandboxing
(nix.useChroot on NixOS,
or option build-use-chroot in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Thanks for the fix! Sorry I missed that.


Houndify keeps saying "I didn't get that"

I'm not sure if this is an issue at all or is just a limitation of the technology, but there's some questions Houndify just immediately gets, and some of them that are less westernized, like for example, I asked "When did Malaysia change it's name to Malaysia", no matter how many times, it's always "I didn't get that", I've also asked who a twitch streamer was, same response sadly.

That is a Houndify issue. You may be able to get more functional questions by enabling more "domains" in your Houndify dashboard.


Table view router

Description
AttributeSelector component:
Added a handler to respond to an attribute button click and update the Router URL accordingly.
Initially this has been completed for a table view, but is planned to improve flexibility and handle any attribute which has a button associated with it.
buttonClickHandler: path => dispatch(push(/gui/${path[0]}/table)
Testing instructions
Add a set up instructions describing how the reviewer should test the code

Review code
Check Travis build
Review changes to test coverage
{more steps here}

Agile board tracking
connect to #155

Ok – I just removed it and it still runs ok.
It was something to do with the test framework I think – but obviously just an aberration on my part!
From: ray-millward-tessella<EMAIL_ADDRESS>Sent: 26 June 2018 15:20
To: dls-controls/malcolmjs<EMAIL_ADDRESS>Cc: Gillingham, Ian (DLSLtd,RAL,TEC<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [dls-controls/malcolmjs] Table view router (#190)
@ray-millward-tessella commented on this pull request.

In package.json https://github.com/dls-controls/malcolmjs/pull/190#discussion_r198163110 :

@@ -15,6 +15,7 @@
"enzyme": "^3.3.0",
"enzyme-adapter-react-16": "^1.1.1",
"express": "^4.16.3",

"fs": "0.0.1-security",

do you know why you need this fs package?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/dls-controls/malcolmjs/pull/190#pullrequestreview-132042569 , or mute the thread https://github.com/notifications/unsubscribe-auth/AIFa8cpUA4E0D6y8EjIuWiC-fUUxLvmkks5uAkMqgaJpZM4U4Aot .  https://github.com/notifications/beacon/AIFa8S2gQ59sPUOxySdglvRDszq8wWhLks5uAkMqgaJpZM4U4Aot.gif

Oh yes – forgot! I’ll do it now and push back.
From: ray-millward-tessella<EMAIL_ADDRESS>Sent: 26 June 2018 15:24
To: dls-controls/malcolmjs<EMAIL_ADDRESS>Cc: Gillingham, Ian (DLSLtd,RAL,TEC<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [dls-controls/malcolmjs] Table view router (#190)
@ray-millward-tessella commented on this pull request.

In src/malcolmWidgets/attributeDetails/attributeSelector/attributeSelector.component.js https://github.com/dls-controls/malcolmjs/pull/190#discussion_r198164717 :

@@ -71,7 +73,12 @@ export const selectorFunction = (
} else if (widgetTag === 'widget:flowgraph') {
return <ButtonAction text="View" clickAction={() => {}} />;
} else if (widgetTag === 'widget:table') {

return <ButtonAction text="View" clickAction={() => {}} />;

/* TODO: IG - push table route to redux router and display table in centre pane

I think this TODO is done now, does the comment need deleting?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/dls-controls/malcolmjs/pull/190#pullrequestreview-132044546 , or mute the thread https://github.com/notifications/unsubscribe-auth/AIFa8WKa_j21Ad52OzcLZtF1aR2v8BJ2ks5uAkQKgaJpZM4U4Aot .  https://github.com/notifications/beacon/AIFa8SPLNGei5fSvWV2q5dwNPwJWkfrVks5uAkQKgaJpZM4U4Aot.gif

Looks good, approved, you've just got one merge conflict so you'll want to pull version1 in before you can merge your branch

Codecov Report

Merging #190 into version1 will decrease coverage by 0.41%.
The diff coverage is 71.42%.

@@             Coverage Diff              @@
##           version1     #190      +/-   ##
============================================
- Coverage     86.08%   85.67%   -0.42%
============================================
  Files            57       55       -2
  Lines          1186     1110      -76
  Branches        184      167      -17
============================================
- Hits           1021      951      -70
+ Misses          145      140       -5
+ Partials         20       19       -1

Impacted Files
Coverage Δ

...s/attributeSelector/attributeSelector.component.js
86.48% <71.42%> (+0.37%)
:arrow_up:

src/blockDetails/blockDetails.component.js
77.08% <0%> (-22.92%)
:arrow_down:

src/malcolm/reducer/layout.reducer.js
76.92% <0%> (-5.77%)
:arrow_down:

src/drawerContainer/drawerContainer.component.js
85.71% <0%> (-0.96%)
:arrow_down:

src/malcolm/reducer/malcolmReducer.js
83.63% <0%> (-0.44%)
:arrow_down:

src/malcolm/actions/method.actions.js
100% <0%> (ø)
:arrow_up:

src/malcolm/malcolmActionCreators.js
100% <0%> (ø)
:arrow_up:

src/malcolm/malcolmHandlers/attributeHandler.js
97.05% <0%> (ø)
:arrow_up:

...colmWidgets/buttonAction/buttonAction.component.js
100% <0%> (ø)
:arrow_up:

... and 7 more

Continue to review full report at Codecov.

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

Looks like the snapshot tests failed because some styling was added, the changes look fine so I think you just need to run npm test -- -u to update them.


Since there was a bug fix that does not specify a mailbox at delete_all

I was using the delete_all but errors are out "(Error in IMAP command received by server.) Net :: IMAP :: NoResponseError", was added because there was no process to select the mailbox.

Fixed by 4d1319696f48d9a2a010f81cb18ef741b1d89dae


serve metrics in clusternet-agent

What type of PR is this?
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Codecov Report

Merging #608 (327fbe6) into main (e2ffa5f) will decrease coverage by 0.41%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #608      +/-   ##
==========================================
- Coverage   13.60%   13.19%   -0.41%
==========================================
  Files          66       71       +5
  Lines        7492     7792     +300
==========================================
+ Hits         1019     1028       +9
- Misses       6391     6681     +290
- Partials       82       83       +1

Impacted Files
Coverage Δ

...ernet/clusternet/pkg/hub/apiserver/shadow/utils.go
89.28% <0.00%> (-10.72%)
:arrow_down:

...m/clusternet/clusternet/pkg/scheduler/scheduler.go
3.82% <0.00%> (-0.12%)
:arrow_down:

...ithub.com/clusternet/clusternet/pkg/agent/agent.go
1.42% <0.00%> (-0.07%)
:arrow_down:

...et/clusternet/pkg/registry/shadow/template/rest.go
2.87% <0.00%> (-0.01%)
:arrow_down:

...hub.com/clusternet/clusternet/pkg/utils/options.go
0.00% <0.00%> (ø)

.../clusternet/clusternet/pkg/agent/status_manager.go
0.00% <0.00%> (ø)

...t/clusternet/pkg/registry/shadow/template/scale.go
0.00% <0.00%> (ø)

...github.com/clusternet/clusternet/pkg/utils/flag.go

...thub.com/clusternet/clusternet/pkg/agent/option.go

...ternet/clusternet/pkg/scheduler/apis/validation.go
42.85% <0.00%> (ø)

... and 6 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Fine Tuning the Model

I want to fine-tune ProGen2-small on my own dataset.
See this google colab notebook for an annotated version of the code and the error:
https://colab.research.google.com/drive/1_R0xgf6Kw0K88PYF7-ZOCIh9WRSmXN8C?usp=sharing
First I load the model like this:
import torch
from tokenizers import Tokenizer
from progen.progen2.models.progen.modeling_progen import ProGenForCausalLM

model = ProGenForCausalLM.from_pretrained('/content/drive/MyDrive/progen2-small', torch_dtype=torch.float16, low_cpu_mem_usage=True).to(device)

I am using the huggingface Trainer to fine-tune the model with the DataCollatorForLanguageModeling. I load the tokenizer like this:
def create_tokenizer_custom(file):
    with open(file, 'r') as f:
        return Tokenizer.from_str(f.read())

tokenizer = create_tokenizer_custom(file='/content/progen/progen2/tokenizer.json')

And then convert it to a PreTrainedTokenizerFast as suggested by: https://github.com/huggingface/tokenizers/issues/325
from tokenizers import Tokenizer
from transformers import PreTrainedTokenizerFast

tokenizer.save("my-tokenizer.json")
fast_tokenizer = PreTrainedTokenizerFast(tokenizer_file="my-tokenizer.json")

During fine-tuning, the training loss becomes 0.0000. After training, I attempt to produce new samples:
with torch.no_grad():
  input_ids = torch.tensor(fast_tokenizer.encode("1GRGL")).view([1, -1]).to(device)
  tokens_batch = model.generate(input_ids, do_sample=True, temperature=0.7, max_length=50, top_p=10, num_return_sequences=1, pad_token_id=0)
  as_lists = lambda batch: [batch[i, ...].detach().cpu().numpy().tolist() for i in range(batch.shape[0])]
  print(tokenizer.decode_batch(as_lists(tokens_batch))[0])

However, I get this error: RuntimeError: probability tensor contains either inf, nan or element < 0
Please see the above google colab notebook for the entire code.

top_p=10 might be <1?

top_p=10 might be <1?

Unfortunately no, I still get the error after setting top_p to <1. But thank you!

top_p=10 might be <1?

Unfortunately I still get the error after setting top_p to a value less than one. Thank you though!

I am getting a warning and an error which are as follows:
Warning: You're using a PreTrainedTokenizerFast tokenizer. Please note that with a fast tokenizer, using the __call__ method is faster than using a method to encode the text followed by a call to the pad method to get a padded encoding.
Error:  RuntimeError: "LayerNormKernelImpl" not implemented for 'Half'.

@cv277 where you able to resolve the issue?

To fix this, you should use torch_dtype=torch.float32 instead.

我想知道数据集的样式是什么样的，能否提供呢

I've switched to torch_dtype=torch.float32 but am still getting this issue for progen-base and larger models, but not for progen-small when I'm calling
model = ProGenForCausalLM.from_pretrained('/content/drive/MyDrive/progen2-small', torch_dtype=torch.float32, low_cpu_mem_usage=True).to(device)
Has anyone experienced similar issues or is there somewhere else I need to change the dtype?

@oliverfleetwood that works for me, I tried loading the progen2-large model and it loads fine - what error are you encountering?


DM: support PCRE regexp syntax for routing

Is your feature request related to a problem?
I'd like to consolidate many schema by a simple route config.
Describe the feature you'd like
if there are databases for source like

mercari_a
mercari_b
mercari_c
mercari_d
mercari_e

following config do not work
  route-a:
    schema-pattern: "mercari_a"
    target-schema: "mercari"
  route-others:
    schema-pattern: "mercari_(?!a).*"
    target-schema: "mercari-other"

Describe alternatives you've considered
No response
Teachability, Documentation, Adoption, Migration Strategy
No response

I understood Golang regxp support RE2 syntax, not PCRE

I understand there isn't lightweight safe proper library for Golang...

You can use mercari_[^a] in RE2 syntax. I think it's confusing to have two RE flavors and they can be converted to the other in most case.

the actual wildcard syntax is mercari_[!a]. you need to prefix with ~ to use the regexp syntax. but the doc still seems to only recommend the wildcards * and ? at the end
schema-pattern: 'mercari_[!a]'
# or
schema-pattern: '~^mercari_[^a]'

sorry My example was a bit wrong and I think I can't cope with existing RE2 syntax

mercari_sample
mercari_sample2
mercari_sample_hoge
mercari_asample
mercari_bsample
...

following config do not work
route-a:
schema-pattern: "mercari_sample"
target-schema: "mercari"
route-others:
schema-pattern: "^mercari_(?!sample$).*$"

It won't work because assertions (?!x) is not "regular" and thus not part of RE2 syntax. There is no plan to support regex flavors other than Golang's built-in one.
Route-rules do not allow overlapping matches[^1], but if the "other" schemas are a finite set you could always do
routes:
  route_a:
    schema-pattern: mercari_sample
    target-schema: mercari
  route_others:
    schema-pattern: '~^mercari_(sample2|sample_hoge|asample|bsample|...etc...)$'
    target-schema: mercari_others

[^1]: I think this can be relaxed, given that route-rules being an array with order, we can easily break the tie by favor the first matching rule.


map aspect: figure out what's wrong with awesome markers

The awesome markers were introduced in e4b7db8274553758f3e6d7d5d1ef27d87dcfb7ae , in order to display different icons depending on the underlying node (type). Unfortunately, they seem to be interfering with the rest of the web UI.  For example, tiles fail to load (no error, but they just never appear), and elements from the header (nav bar, drop down button, aspect names) disappear as the mouse cursor hovers over them.
I reverted to the basic leaflet markers in 49b97b224484b8b9d7099ce6923dcb01ff07d70b until we figure out what's wrong with the awesome markers.

additional info: I'm seeing the same glitches with the basic leaflet markers, when there's enough of them.
100 seems to be ok, while 1000 isn't.

a potential workaround: https://github.com/Leaflet/Leaflet.markercluster

https://camlistore-review.googlesource.com/10746

I don't see the glitches on firefox (49.0). everything is just a tiny bit slow with 1000 markers, but no problem otherwise.
With chrome (59.0) it's another story.

On a limb I've just tried with a different window manager (gnome) than my "archaic" rio (same chrome) and of course I couldn't see any of the glitches anymore. So I'm guessing it's either because chrome is started with specific options on gnome (launched the clicky way), or more likely because rio does not conform to some X11 rule and is doing something that chrome does not like.
In any case, I'm going to assume I'm the only one affected, as most users will run a "sane" environment where these glitches do not show up on chrome.


Add support for planetary gears

It would be helpful if one could design a set of planetary gears using BOSL.

As I understand it, only the ring gear is maybe confusing to make, and that can be done by making a spur_gear() with interior=true then differencing it away from a cylinder.

There should be an example showing that this is possible.   Is there any way to discover this in the manual?

include <BOSL2/std.scad>
include <BOSL2/gears.scad>
$fa=1; $fs=1;
diff()
cyl(d=100,l=10)
    tag("remove")
        spur_gear(
            pitch=8, teeth=32, thickness=11,
            pressure_angle=20, interior=true);

color("green")
back(27)
spur_gear(
    pitch=8, teeth=11, thickness=11,
    pressure_angle=20);

Shouldn't the teeth on the ring gear look the same as the teeth on the spur gear?

Let me see if I can whip up a proper planetary gear assembly with animation.

That might be overkill.  The question isn't whether the above thing will work ok in an animation, but whether it's actually the right way to make a ring gear.

No, the interior teeth of a ring gear do NOT look like inward pointing spur gear teeth.  The animation I just make makes that fairly clear.
https://github.com/BelfrySCAD/BOSL2/assets/350001/106b6ae8-d36f-40d3-ae55-d71610092959

Oh, the code for the above animation:
include <BOSL2/std.scad>
include <BOSL2/gears.scad>
$fa=1; $fs=1;
diff()
cyl(d=100,l=10)
    tag("remove")
        spur_gear(
            pitch=8, teeth=32, thickness=11,
            pressure_angle=20, interior=true);

color("green")
back(27)
spur_gear(
    pitch=8, teeth=11, thickness=11,
    pressure_angle=20);

There is an outstanding issue about how we need actual docs for the gears stuff.  It would definitely help.  I mean, we need a multi-page gears intro section that explains all the terminology and gear types and gear parts with pictures.  But for now I made a few tweaks to the docs that should help a bit.  I added an example of non-parallal-shaft helical gears----but I couldn't figure out how to reliably get them to mesh in general.  Do you know the answer?
I added a reference to internal ring gears to the spur_gear module.  It appears that technically helical gears are not a subtype of spur gears, so the name isn't ideal, though I guess it's a tradition in OpenSCAD to do that.
I wonder about how to make herringbone or double helical gears.  The one gear example I printed using herringbone gears.  It should be an easy development from what we have, but just stacking to gears, they weren't aligned properly at the joint.   I'm not sure if a separate module makes sense?   Or a herringbone=true or double=true option?   I guess doing these really well would mean rounding the teeth where they meet in the center of the gear, which would be much harder than just sticking together two gears from the existing code.
In the case of bevel gears, the same question arises, actually.  The other thing with bevel gears is that there is an interior option.  But I couldn't figure out what an internal bevel gear is.   The interior option really should be internal, but maybe too late to change?  I don't know.   And the question also arises of whether internal should have a default_tag of "remove".
Also it appears that you can't make the worm curve to match the worm gear for the "double enveloping" assembly.
So here are some questions:

how do you determine meshing for non-parallel shaft helical gears
how do you make herringbone aka double helical gears?  Or the analgous spiral bevel gear construction?
What is an interior bevel gear?
Should "interior" change to "internal"?
Should interior gears have a default tag of "remove"
Double enveloping worm gear assembly?
Any idea about zerol bevel gears?

spur_gear(
       pitch=pitch, teeth=n, thickness=10,
       shaft_diam=5, helical=0, slices=12,
       $fa=1, $fs=1
   )
  position(RIGHT)
   spur_gear(
       pitch=pitch, teeth=n, thickness=10,
       shaft_diam=5, helical=0, slices=12,
       $fa=1, $fs=1,anchor=LEFT
   )
;

Above code makes two mated gears (I think) but it's impossible to rotate the gears with the above code.  There are two problems.  One is that rotating an object rotates in children and the other is that spin= rotates relative to the anchor instead of object center (a design decision that has always seemed wacky to me).   Is there a way to assemble gears using positioning, or is that just an idea that doesn't work?   Do gears need to have an option to offset their teeth so they can be aligned?

I've been playing around with the code, and from a user's perspective,
what is wanted is to specify the OD and the mechanical advantage and
perhaps the diameter of the central gear.  I guess the question is how
much effort do you want to put into making this simple for a library user.
Jon
On 6/17/2023 11:52 PM, Revar Desmera wrote:

Oh, the code for the above animation:
include  <BOSL2/std.scad>
include  <BOSL2/gears.scad>
$fa=1;$fs=1;
diff()
cyl(d=100,l=10)
tag("remove")
spur_gear(
pitch=8, teeth=32, thickness=11,
pressure_angle=20, interior=true);
color("green")
back(27)
spur_gear(
pitch=8, teeth=11, thickness=11,
pressure_angle=20);
—
Reply to this email directly, view it on GitHub
https://github.com/BelfrySCAD/BOSL2/issues/1188#issuecomment-1595942221,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AASP3SCGR6WIGYNBTFMRKELXLZ3Q7ANCNFSM6AAAAAAZKJZUDY.
You are receiving this because you authored the thread.Message ID:
@.***>

--------------sQLPdxueG0cOljOBhkQQDILs
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: 8bit

I've been playing around with the code, and from a user's
      perspective, what is wanted is to specify the OD and the
      mechanical advantage and perhaps the diameter of the central
      gear.  I guess the question is how much effort do you want to put
      into making this simple for a library user.
Jon

On 6/17/2023 11:52 PM, Revar Desmera
      wrote:

Oh, the code for the above animation:

include <BOSL2/std.scad>

include <BOSL2/gears.scad>
$fa=1; $fs=1;
diff()
cyl(d=100,l=10)
tag("remove")
spur_gear(
pitch=8, teeth=32, thickness=11,
pressure_angle=20, interior=true);
color("green")
back(27)
spur_gear(
pitch=8, teeth=11, thickness=11,
pressure_angle=20);

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message
ID<EMAIL_ADDRESS>

--------------sQLPdxueG0cOljOBhkQQDILs--

Not sure what you mean about specifying "the diameter of the central gear".   Are you thinking of a whole assembly of gears?  We don't have any code in the library for making gear assemblies, just individual gears.

Yes.  Sorry.  The diameter of the assembly.  It will quite likely have
to fit into something else.
Worm gears and worms seem like an assembly.
The planetary gear set seems like an assembly.
Put differently, if I were to design the central, planetary, and
outer/ring gears separately, how would I know they would make a
successful assembly absent a single central module() which created them
as a set?
Your library, your rules.  Just my thoughts.
Jon
On 6/18/2023 10:55 AM, adrianVmariano wrote:

Not sure what you mean about specifying "the diameter of the central
gear". Are you thinking of a whole assembly of gears? We don't have
any code in the library for making gear assemblies, just individual gears.
—
Reply to this email directly, view it on GitHub
https://github.com/BelfrySCAD/BOSL2/issues/1188#issuecomment-1596175643,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AASP3SF6KKCWDFLFIMZ6HP3XL4JGJANCNFSM6AAAAAAZKJZUDY.
You are receiving this because you authored the thread.Message ID:
@.***>

--------------7lD0AvUgE3gip9uiiXMckvJ0
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: 8bit

Yes.  Sorry.  The diameter of the assembly.  It will quite likely
      have to fit into something else.
Worm gears and worms seem like an assembly.
The planetary gear set seems like an assembly.
Put differently, if I were to design the central, planetary, and
      outer/ring gears separately, how would I know they would make a
      successful assembly absent a single central module() which created
      them as a set?
Your library, your rules.  Just my thoughts.
Jon

On 6/18/2023 10:55 AM, adrianVmariano
      wrote:

Not sure what you mean about specifying "the
    diameter of the central gear". Are you thinking of a whole
    assembly of gears? We don't have any code in the library for
    making gear assemblies, just individual gears.

—

    Reply to this email directly, view it on GitHub, or unsubscribe.

    You are receiving this because you authored the thread.Message
      ID: <BelfrySCAD/BOSL2/issues/1188/1596175643@github.com>


FormData needs additional constructor

Additional constructor that takes in HTMLFormElement as parameter as here:
https://developer.mozilla.org/en-US/docs/DOM/XMLHttpRequest/FormData
declare var FormData: {
prototype: FormData;
new (): FormData;
new (form: HTMLFormElement): FormData;
};
This is copied from https://typescript.codeplex.com/workitem/720, the fix isn't incorporated into TS v 1.4.
It's possible to write the constructor in the optional form as well:
new (form?: HTMLFormElement): FormData;
Thanks in advance!

Duplicate of #1074


Make AI-Based Enhancements and Modify Data Processing at the Dataset Construction Script

This Pull Request comes with some important changes on the script for building a dataset that incorporates more inputs from AI and better processing of data.
Firstly, new adjective vertically humanized advanced scripting for dataset creation has been inserted to an up-to-date script. A new method, namely, build_advanced_dataset has been also added in order to take advantage of the AI-generated information concerning parameters of crafting a particular dataset. This method has been designed on the basis of a dummy function called ‘analyze_data_quality’, which gives an impression of how an AI could actually perform the data quality analysis and what improvements could be suggested. This strategy’s goal is to optimize the dataset characteristics by using AI approach, which results in the datasets that are more fit for their tasks.
The analyze_data_quality function in the current skeleton is the first step to integrating AI to the creation of the dataset. This function developed through the concept of using machine learning models in an attempt to assess the quality of the data and recommend the most favourable parameters that would enable one create a dataset. Through this use of AI analysis, the script forms grounds for enhancing the development of other elaborate approaches for building a dataset in the future.
Apart from theses AI-enabled options, bugs in the script have been fixed couple with enhancements repeatedly. Proxy code that was not used earlier or the lines of code that were commented have been omitted for the better maneuverability. Variable naming has been enhanced regarding each variable as the script was not functioning well in some instances as intended and proper error control has been enhanced. Such updates again help in building the dataset and also in sorting out whether the script runs well or not.
The commit also have changes in existing methods like, enhanced methods of filtering datasets and handling queries. Now, query results when compared to a script are managed in a more flexible way in terms of the creation of new datasets, and the dataset construction based on images with descriptions includes some advanced features.
All in all, these improvements add value in the script by presenting AI made improvements and better data management practices. It has now been possible to incorporate AI insights into building datasets.

This pull request describes substantial changes to the building script of the dataset connecting to the presence of AI-oriented features and improving data handling.
Key Updates:

AI-Driven Dataset Building:

Extended the algorithm with the method for creating an advanced form of a dataset based on AI-driven suggestions for editing the parameters of the dataset. This new functionality shows that application of AI can be in the creation of a dataset where the creation is made based on the data quality of the existing datasets.
Added the analyze_data_quality function which would be replaced with AI based data quality analysis in future versions. This function will in the later future use natural language processing and other AI models for assessing and providing the best dataset parameters.

Bug Fixes and Code Improvements:Bug Fixes and Code Improvements:

Cut unnecessary and commented out statements for the better purpose of understanding the scripts and the added bonus of improved script performance.
Some of the improvements that have been made include refining variable naming to avoid conflicts and fixing up of errors in order to make the dataset creation more reliable.

Enhanced Data Management:

New algorithms for each steps to improve the quality of the datasets and each queries.
Better handling of datasets derived from image with text.

Impact:
The computerization of these updates permits creating a basic structure for incorporating the AI-improvements into the dataset construction and guarantees the dynamic adjustments on the behalf of AI. When focusing on code clarity and functionality, there is a development of a more capable and optimized script as well.


fix(headers): Ensures that Markdown headers are preceded by a space

Changes subParser for headers to require a space or tab character before a header, so that
  # Header

Is a valid header, whereas:
  #Header

Is not rendered as a header.
(This seems to be the standard from what I can tell by testing across the internets, but if we'd prefer to make this an option so that it's not a breaking change I'm happy to do that too!)

According to the spec, a space is not required between # and the rest of the text.
For this reason (and because it's a breaking change) making this an option is preferable.

The spec at http://spec.commonmark.org/0.26/#atx-headings would seem to
indicate that the space is required.
Every implementation I can find does it this way also - but I understand
the desire to avoid a breaking change. Which Markdown spec does showdown
follow?
On Wed, 20 Jul 2016 at 14:20, Estevão Soares dos Santos <
<EMAIL_ADDRESS>wrote:

According to the spec, a space is not required between # and the rest of
the text.
For this reason (and because it's a breaking change) making this an option
is preferable.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/showdownjs/showdown/pull/277#issuecomment-234086984,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAZSvu_YGC5fLAARML_1oeVckai7dN-ks5qXpE0gaJpZM4JRL88
.

Showdown follows the original markdown spec. https://daringfireball.net/projects/markdown/syntax
In markdown.pl, created by the original author, spaces are explicitely optional.
commonmark is a variant of the markdown spec

OK, cool - so would you be open to a patch that added a requireSpacesBeforeHeadings option? This is how GFM works, and we're trying to maintain render compatibility between our Ruby and Javascript libs.

Yes, that would be the best option. I ask you, however, to make the MPR against develop branch instead of master

Since there's no update on this, I'm gonna implement this as a feature myself.


[ENG-1451] Fix editable field.

Ticket: ENG-1451
Feature flag: n/a

Purpose
Fix overview page editable field weirdness.
Summary of Changes
Somehow selectedLicense is an alias to node.license. Remove that alias.
Side Effects
None.
QA Notes
First go to overview page of a registration. Change the license and save. Then change the license again but instead of clicking Save, click Cancel. The change should not be persisted and the license field shows the correct license.

@fabmiz I believe onCancel is actually called. Also, the @selected=this.selectedLicense doesn't really do anything right now. If you go to validated-input/power-select you'll see that @selected={{or (get this.modelOrChangeset this.valuePath) null}}, which will always set the selected to whatever is currently in the valuePath of the model, so it actually works as expected.

@adlius you're right it seems onCancel is used on the cancel button. We need also have the editable-field component use some version onCancel that checks for unsaved changes then resets if any, for the x button on the dialog.


Use new conda rapids env meta-pkgs

I noticed that the dependencies removed from this PR don't map to the three new conda recipes 1:1, but everything built okay for me so I'm assuming that's fine.

Yes these also include build dependencies that we didn’t include previously but are needed for testing or a part of the conda build requirements across the repos. Also for consistency some repo wide dependencies that were in the docker container below these were rolled up so we change them in one environment across the repo.
This “base” layers from the gpuci Docker hub repo also need updating to remove some of the redundant installs here. That is a separate effort but I wanted to get this out first.


simplify service helpers

related to https://github.com/chef/chef/issues/3835

:+1: so much better

Is it possible that people are using the service resource to ensure the state of non-service unit files?  For instance
service "sshd.socket" do
  enable [:enable, :start]
end

Perhaps we can check for either "#{cfg_base}/systemd/system/#{svc_name}.service" or "#{cfg_base}/systemd/system/#{svc_name}"
As you mentioned this will miss loaded units that aren't backed by files.  From what I've seen most dynamically generated units are going to be device, mount, automount, or swap units.  Perhaps it is OK that we miss those?

is it ok to use service resource define socket? we can explicitly mention this, that the service resource only controls *.service units

is it ok to use service resource define socket?

The code I posted works currently with Chef.  I haven't seen much usage like that to know if we'd be breaking a large or small number of cookbooks.

hmm.. then we have to check for both i guess. but its kinda not idiomatic, is it acceptable if we add a deprecation warnings for non-service units, cause systemd supports a wide array of unit types, we wont be able to address the resulting corner cases within a single service provider. we can recommend folks to use the systemd cookbook for non service units

a socket is not a service, and if anyone's using service resources to manage non-service units, it's their own fault if it breaks; if it works at present, it's only because chef isn't doing what it ought to and specifying the unit type when shelling out to systemd for the service resource (i.e. systemctl start #{service_name} instead of systemctl start #{service_name}.service.

fix for systemd service provider overloading proposed in #3838

ugh, and i realize that https://github.com/chef/chef/pull/3836#issuecomment-135852297 sounded way crankier than i meant it to... i meant "own fault" in the "shouldn't be surprised" sense, more than the "finger wagging" sense... apologies.

From what I've seen most dynamically generated units are going to be device, mount, automount, or swap units.

Not sure this is a safe assumption to make. While we as a large systemd user don't do this currently, we have talked about it before. Others might as well. You can for instance have a single<EMAIL_ADDRESS>unit file but instantiate dozens of redis instances using<EMAIL_ADDRESS>as the unit name without making new files (I think...=). The 1234 can then be subbed into the unit file at runtime by systemd to use as the port number for that redis instance.

I like this approach because it doesn't slam dbus, but this would break for us do to the aforementioned  service "foo.socket" We use heaps of sockets. Thats ok in this patch tho if you include .socket files  (for us at least).
There has to be a way to ask systmed 'is this a unit even if its not loaded' without invoking systemctl --list-all-units. I was trying to figure that out last night.
I personally wouldn't mind being able to tell chef, hey we are a systemd shop, don't go through these shenanigans to try  and be smart. Just run systemctl status on this string I am passing you. I don't get why the platform detection is concerned about a specific service anyhow. thats the providers job isn't it?
FWIW  chef 11.16 works great with ~20k systemd services.

@joemiller interesting. using the<EMAIL_ADDRESS>model wouldn't actually work for the systemctl list-unit-files method of detecting either, unless you stripped everything after the @ off the resource name (doable i suppose, but eww). we could try to change the has_systemd_service_unit? method here to check for the unit file after having stripped the service name of anything after the @, but i dunno... feels like a more targeted resource would be more appropriate at that point :/.
@spheromak have you looked at using the systemd_socket resource in the systemd cookbook?

@nathwill A socket is a service in that if you use socket activation to run services it is a service :). Pedantically sockets aren't the same type to systemd, but from a chef perspective a socket should be considered a service. You still run the same lifecycle commands on them.

@natewalck no we don't use the systemd cookbook, but sounds interesting.

@spheromak we'll have to agree to disagree there ;) it works only at a very shallow level, if you try to do anything more involved, the illusion falls apart quickly (e.g. wtf does "reload" do on a socket?).

@nathwill as long as theres a path for managing socket units I am not bothered. Just that the current patch would break on the existing chef code I have inherited. Guess thats better than the current master that times out dbus :)

i think my main objection here is that management of non-service units via the service resource has only worked in the past/present via exploitation of what is clearly a bug in Chef (no-one intended to support random unit types via the systemd service resource/provider).
i understand some (many?) folks may have been exploiting that bug in the absence of am 'official' way to manage misc units, but continued support for doing so (by not closing the bug) will only muddy the waters if/when chef later adds proper support for other unit-based resources (which i assume is inevitable), and it inherently sanctions continued misuse of the service resource.
to my mind at least, it's better to add a changelog warning and enforce the systemd-service provider's area of responsibility, and then add true support for other units if they're needed/wanted.

@nathwill hey just tried this on one of our machines with 10k units; it doesn't work with .automount units, generates an error about no providers responding to supports, then kind of looks like it hangs. I only had a brief opportunity to test this then had to roll it back. I'll keep messing with it and see if I can get what you have to work.
[2015-08-29T01:35:53+00:00] INFO: Processing service[srv-bindings-55fc383fa9e041bdafc5d0567eb87b2b-files.automount] action start (endpoint::_appserver_bindings line 415)
[2015-08-29T01:35:53+00:00] DEBUG: Providers for generic service resource enabled on node include: [Chef::Provider::Service::Redhat, Chef::Provider::Service::Systemd, Chef::Provider::Service::Init]
[2015-08-29T01:35:53+00:00] DEBUG: No providers responded true to `supports?` for action start on resource service[srv-bindings-55fc383fa9e041bdafc5d0567eb87b2b-files.automount], falling back to enabled handlers so we can return something anyway.
[2015-08-29T01:35:53+00:00] DEBUG: Provider for action start on resource service[srv-bindings-55fc383fa9e041bdafc5d0567eb87b2b-files.automount] is Chef::Provider::Service::Systemd

@nathwill I'd guess that is probably something to do with the previously discussed non-services-as-service-resources. I've scheduled some time to look at your lwrps, which are super promising, not sure how I'd missed them so far :)

@joemiller so the redis@6380 type service should work now as well. that look alright?

@fujin yeah, definitely related, sorry for the hassle. this seems to me like an opportune time to clean this up, but maybe the maintainers will want to deprecate it over a couple releases? :8ball:

I definitely think we need to continue to support cookbooks doing "server 'sshd.scoket'" until some deprecation period is over.  Breaking straightforward cookbooks in a minor release is a no-go.

@stevendanna alright, this should support non-service-unit "services" now as well.

LGTM, everyone :+1: on shipping?

:+1:

i'm obviously :+1: on it as well :)

alright, straightened out some merge conflicts and got the specs working again. still interested in merging?


Server summary

Return resource counts for all available resources on a server

Counts of all resources supported by a servers REST API via
summary = server.summary()


Add a toJson method for generated Table class.

Is it possible to add a toJson method to the generated table class? Because I sometimes need to serialize the table class into json, thank you very much!

I create a PR #3 to fix this issue, please check it.


Create: 0127-word-ladder.dart

File(s) Modified: 0127-word-ladder.dart
Language(s) Used: dart
Submission URL: https://leetcode.com/problems/redundant-connection/submissions/1275550818/

Important
Please make sure the file name is lowercase and a duplicate file does not already exist before merging.

@matthewkeck Thanks for your contribution 👍


Repeating Events

One of my big roadblocks for this plug-in is the lack of options for repeating events. As far as I can tell, you can only set it up to repeat events either daily or every week. It would be a big help to be able to set up events to repeat every x number of weeks, every x number of months and even every x number of years. It would also be helpful to have an option to customize repeats. These items are available on many calendars. Thank you for your consideration.

Thanks for submitting! this is a duplicate of #22. This is definitely on the roadmap


Can't import creation functions from 'rxjs'

Looks like the package.json points to Rx.js while I'd need index.js
RxJS version:
6.0.0-alpha.3
Code to reproduce:
import { of, from } from 'rxjs';
Expected behavior:
should import those functions
Actual behavior:
functions not found
Additional information:
Repo at https://github.com/evertbouw/redux-observable/tree/rxjs6

Fixed by https://github.com/ReactiveX/rxjs/commit/c2b00f459d4c8e280a0c5b05506464c985d3d4f6


Added support for multiple delivery attempts for UPS tracking

Added some further enhancements on failed delivery attempts with support for UPS (to complement previous changes for USPS)

This is a nice feature! I'm a little worried about that test though. Maybe worth just testing with unit tests where you can mock the returns from ups and make sure it's parsed correctly?
I think putting in a test that we know is likely to start failing in some number of months might be a bad move.

Hey, @cwise, just took a look at your changes - I think it's great that you added support for attempted delivery attempts. That said, I'm not as on board with modifying the existing TrackingResponse class to have multiple delivery_attempt attributes. I think just one was enough, especially since other carriers only ever try to deliver once.
That said, I think recording the first delivery attempt for UPS is still worth doing.
What are your thoughts, @kmcphillips @mdking ?

I agree with @jonathankwok that I would like to avoid adding the multiple delivery_attempt attributes. It is a larger change in the code but what is everyone's thoughts on adding a attempted_delivery_dates attribute which is the array of delivery_attempt dates?
There would be a bit of work to make sure it's backwards compatible with the existing attempted_delivery_date attribute.

@cwise I'm going to close this out for now. If you rebase, get test covering the change, get the build green, and get supporting docs. Feel free to re-open this PR.


Inference environment

Hi,
Thanks for the great work and for sharing the code.

Is the reported performance on model compiled to TRT or naive Pytorch?
Do. you have comparison of latency (batch size of 1) rather than throughput that you dominate?

Pytorch.
The comparisons with batch size=1 are very consistent with those with batch size=128, so we did not report them in the paper.


Fix some typos in [en] docs.

This pr is fix some tiny typos.
likes
1 Config Maps --> ConfigMaps
2 liveleness --> liveness

In general, we don't fix blogs that are older than one year.
In this case, it is a typo fix, which is fine.
/lgtm


fix: lighthouse a11y

Hi Joe,
Noticed two quick fix for the accessibility of your website according the lighthouse  results.
Cheers
David

Thanks for kicking this off. I bumped some contrast in the header and footer and added the HTML doctype. Perfect 100s now!

Awesome!!! Well played @joemasilotti 🤙


Translate database values

I have a list of countries coming from the database, to be shown in a dropdown for when users enter their address.
Using ng2-translate, is there a way to translate them?
I'd like to be able to do something like the following:
<select formControlName="country">
  <option *ngFor="let country of codesService.countries$ | async" value="{{country.code}}">{{ 'CODES.COUNTRIES.{country.code}' | translate }}</option>
</select>

With translations in a json file like:
"CODES": {
    "COUNTRIES": {
      "AF": "Afghanistan",
      "AX": "Aland Islands",
      "AL": "Albania",
      "DZ": "Algeria",
      "AS": "American Samoa",
      ...
    }
  }

But of course it doesn't work :(
Or should I have the translations in the database too and use currentLang to retrieve the entries in the desired language?

it will be difficult with the pipe, but you can do it programmatically with the service in your component

Not sure how...
What would be your approach for such values coming from the database?

You can do something like:
codesService.countries$.subscribe(country => {
     let codes = translate.instant('CODES.COUNTRIES')
     this.country = codes[country];
});

I'm using instant here, but you can use get as well if you're not sure that the translations have been loaded

I have my components do this:
<select>
  <option *ngFor="let country of countries" value="{{country.code}}">
    {{ 'CODES.COUNTRIES.' + country.code | translate }}
  </option>
</select>

and it works like a charm

Thanks @DethAriel, like a charm indeed :)


System Error when running Ganache 2.5.4 on win32

<Hi - While installing Ganache I got the message from checkpoint antivirus running on my system mentioning that Ransomware has been detected. After that, checkpoint shows another message that it has been taken care of. Now when I press Quickstart Ethereum option, it gives me an error with below mentioned details. >
PLATFORM: win32
GANACHE VERSION: 2.5.4
EXCEPTION:
Error: EPERM: operation not permitted, rename 'C:\Users\shash\AppData\Roaming\Ganache\workspaces\Quickstart\Settings.2036283228' -> 'C:\Users\shash\AppData\Roaming\Ganache\workspaces\Quickstart\Settings'
    at Object.renameSync (fs.js:643:3)
    at writeFileSync (C:\Program Files\WindowsApps\GanacheUI_<IP_ADDRESS>_x64__5dg5pnz03psnj\app\resources\app.asar\node_modules\write-file-atomic\index.js:124:8)
    at Proxy.setItem (C:\Program Files\WindowsApps\GanacheUI_<IP_ADDRESS>_x64__5dg5pnz03psnj\app\resources\app.asar\node_modules\node-localstorage\LocalStorage.js:217:7)
    at JsonStorage_JsonStorage.setToStorage (C:\Program Files\WindowsApps\GanacheUI_<IP_ADDRESS>_x64__5dg5pnz03psnj\app\resources\app.asar\webpack:\src\main\types\json\JsonStorage.js:48:18)
    at JsonStorage_JsonStorage.set (C:\Program Files\WindowsApps\GanacheUI_<IP_ADDRESS>_x64__5dg5pnz03psnj\app\resources\app.asar\webpack:\src\main\types\json\JsonStorage.js:53:10)
    at WorkspaceSettings_WorkspaceSettings.set (C:\Program Files\WindowsApps\GanacheUI_<IP_ADDRESS>_x64__5dg5pnz03psnj\app\resources\app.asar\webpack:\src\main\types\settings\Settings.js:59:19)
    at Workspace_Workspace.saveAs (C:\Program Files\WindowsApps\GanacheUI_<IP_ADDRESS>_x64__5dg5pnz03psnj\app\resources\app.asar\webpack:\src\main\types\workspaces\Workspace.js:105:19)
    at IpcMainImpl.<anonymous> (C:\Program Files\WindowsApps\GanacheUI_<IP_ADDRESS>_x64__5dg5pnz03psnj\app\resources\app.asar\webpack:\src\main\index.js:442:17)

Anti virus issue.


UI for deletion of Test Cases, Inputs, and Outputs in the Test Bench Panel

This PR enhances the functionality for deleting test cases, inputs, and outputs in the test bench panel of CircuitVerse's main simulator. While the underlying functionality was already present, the UI implementation was inadequate, resulting in the absence of delete buttons.
With this update, minus (-) buttons have been added to the UI, allowing users to remove mistakenly added test cases or inputs/outputs, which can disrupt the test setup. This improvement enables users to clean up their test bench more efficiently.
Screenshots of the changes -
Before

After

Summary by CodeRabbit

New Features

Enhanced user interface for the test creator with updated button styles for improved consistency.

Style

Standardized class names for delete buttons across various functions to align with the overall design strategy.
Minor adjustments to HTML structure for better readability and consistency.

@tanmoysrt
If possible, could this PR be approved under 'hacktoberfest-accepted' label.

@tachyons
Would it be possible to have this PR labeled as hacktoberfest-accepted since October is coming to an end? If you're busy, we can work on merging it later, but I'd really appreciate it if you could label it now.

@gr455
Could this PR please be labelled as hacktoberfest-accepted. It would really count towards my contributions.

@gr455 Could this PR please be labelled as hacktoberfest-accepted. It would really count towards my contributions.

Yep. Verry sorry for the delay

https://github.com/CircuitVerse/CircuitVerse/pull/5068 adds an empty array which is why the CI lint was failing and the conversation isn't resolved.


mir: Pull patch to fix evdev device misses

Description of changes
Will wait until this is merged upstream or 23.05 is close to finalise. A fix is now merged upstream :slightly_smiling_face:. Having a well-working Mir(iway) on 23.05 release would be nice.
Should fix the Miriway test on aarch64, and improve usage on some aarch64 hardware.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg test miriway

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2246


pretty sure there are edge cases that are not considered io read/write

System Design Considerations and Future Improvements
When it comes to the current process of reading and writing data, there are various complexities and invariants that must be maintained for the system to function correctly. It is crucial to address these potential pitfalls to prevent errors caused by overlooking these intricate invariants that can be challenging to debug.
One specific area of concern is the use of a thread-local encoder for both broadcast and local packets. This dual usage introduces the risk of errors, especially when handling serialization and removal of local packets post-serialization. The intricacies of this logic pose a significant opportunity for mistakes to occur.
To mitigate these issues, there is ongoing work on the broadcast branch aimed at enhancing the system's robustness. The improvements implemented in this branch are expected to address some of the existing challenges and enhance the overall reliability of the data reading and writing processes.
https://github.com/andrewgazelka/hyperion/tree/feat-broadcast/broadcast/

This has been largely resolved.


hide description and subject fields from search results display

Closes #907.
Closes #1099.

@tampakis We talked about this with Joyce yesterday, and updated #1099 with a much smaller list of fields that should be displayed in the search results.  Can you update this PR?

Coverage decreased (-0.05%) to 99.951% when pulling 1d27fd337ebc4b6761d7b126128bff8185595abd on index-view-fields into 1c4efdd0d9b57b9631a1a46d9f40985f5af01f3d on master.

Coverage decreased (-0.02%) to 99.976% when pulling 6c3a8ca6ff1280ab6405888b886ff708b3130737 on index-view-fields into 67d50ca1272bf52a91591ebad77c38a020493bb1 on master.

Coverage remained the same at 100.0% when pulling a26b0102740b0400a9e40ad62df92540e930d5d4 on index-view-fields into 67d50ca1272bf52a91591ebad77c38a020493bb1 on master.


Move all database work to either SQLite or PostgreSQL

We currently have one (and maybe soon 2) calls to a postgres database. Since we're already using SQLite to store and pass around environmental data we should go ahead and add the BBS data to that database as well and extract it from there.

Oh, I was all excited to use PostGIS for faster buffer cutting with the NLCD data now that I've basically gotten all of this nonsense working.  But running raster::extract overnight is probably just as good, so I have no objections.

Yeah, it's a tradeoff with ease of reproducibility. I think we should pick either Postgres or SQLite. SQLite will be easier for others since it doesn't require installing anything extra, but I'm OK with having an opinionated dependency here as well.
Also to consider when we're making this decision: @sdtaylor has just finished creating a more efficient version of extract for use with buffers that is apparently quite a bit faster.

Whoa, nice @sdtaylor

I've updated the title to reflect a broader conversation about this. Thoughts on which DBMS we'd like to use throughout?
Major tradeoff: more power/speed with Postgres, but requires users to install and get PostgreSQL working in a way that the code can easily access the database (which as @davharris has experienced can be a bit of work).

My extract with buffer improvement is only  faster when you have many rasters of the same data. Like prism or bioclim. For a single raster it shouldn't be any better than just extract(). NLCD has 3 or 4 version though, so it would help some there.

If we don’t need PostGIS for this project and the data sets don’t get any bigger, I don’t see any upside to using psql.

On Apr 22, 2016, at 11:01 AM, Ethan White<EMAIL_ADDRESS>wrote:
I've updated the title to reflect a broader conversation about this. Thoughts on which DBMS we'd like to use throughout?
Major tradeoff: more power/speed with Postgres, but requires users to install and get PostgreSQL working in a way that the code can easily access the database (which as @davharris https://github.com/davharris has experienced can be a bit of work).
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub https://github.com/weecology/bbs-forecasting/issues/38#issuecomment-213464743

My extract with buffer improvement is only faster when you have many rasters of the same data. Like prism or bioclim. For a single raster it shouldn't be any better than just extract(). NLCD has 3 or 4 version though, so it would help some there.

Ah, got it. Thanks for the clarification @sdtaylor.

If we don’t need PostGIS for this project and the data sets don’t get any bigger, I don’t see any upside to using psql.

OK, if @sdtaylor is on board with this then let's proceed with SQLite and swap it out in the future if we end up needing more power.

@ethanwhite have you done anything with this? I have to play around with this part of the code anyway in #61 so I thought I would switch things to sqlite while I'm at it.

Go for it. Thanks.

I think this could be really helpful—especially because we might be able to do a lot of the joins and stuff before the data even hits R.
Thanks for looking into this!

On Aug 15, 2016, at 11:18 AM, Ethan White<EMAIL_ADDRESS>wrote:
Go for it. Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/weecology/bbs-forecasting/issues/38#issuecomment-239830087, or mute the thread https://github.com/notifications/unsubscribe-auth/AAzdCUTxUrDBzleligVoywyo1N5nz9-Pks5qgINKgaJpZM4INoYE.

#66 done, and working without issue for a while.


Traceback errors

I'm using Windows 10, tried both Python 3.8 and 3.9
Ran these commands:
git clone https://github.com/r0oth3x49/udemy-dl.git
cd udemy-dl
pip install -r requirements.txt
python3 udemy-dl.py https://www.udemy.com/course/master-the-concepts-of-digital-circuit-design/ -k cookies.txt
did it in CMD, already bought the course and found the access_token for my account
get these errors:
Traceback (most recent call last):   File "udemy-dl.py", line 31, in <module>     import udemy   File "C:\Users\Prince\udemy-dl\udemy\__init__.py", line 32, in <module>     from udemy.udemy import course, fetch_enrolled_courses   File "C:\Users\Prince\udemy-dl\udemy\udemy.py", line 27, in <module>     from udemy.internal import InternUdemyCourse as Udemy   File "C:\Users\Prince\udemy-dl\udemy\internal.py", line 26, in <module>     from udemy.compat import time, sys   File "C:\Users\Prince\udemy-dl\udemy\compat.py", line 33, in <module>     import m3u8 ModuleNotFoundError: No module named 'm3u8'
and i doubled checked with pip install -r requirements.txt and got:
Requirement already satisfied: six in c:\users\prince\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages (from -r requirements.txt (line 1)) (1.15.0) Requirement already satisfied: m3u8 in c:\users\prince\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages (from -r requirements.txt (line 2)) (0.7.1) Requirement already satisfied: colorama in c:\users\prince\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages (from -r requirements.txt (line 3)) (0.4.4) Requirement already satisfied: requests in c:\users\prince\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages (from -r requirements.txt (line 4)) (2.24.0) Requirement already satisfied: unidecode in c:\users\prince\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages (from -r requirements.txt (line 5)) (1.1.1) Requirement already satisfied: pyOpenSSL in c:\users\prince\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages (from -r requirements.txt (line 6)) (19.1.0) Requirement already satisfied: cloudscraper in c:\users\prince\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages (from -r requirements.txt (line 7)) (1.2.48) Requirement already satisfied: iso8601 in c:\users\prince\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages (from m3u8->-r requirements.txt (line 2)) (0.1.13) Requirement already satisfied: certifi>=2017.4.17 in c:\users\prince\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages (from requests->-r requirements.txt (line 4)) (2020.6.20) Requirement already satisfied: urllib3!=1.25.0,!=1.25.1,<1.26,>=1.21.1 in c:\users\prince\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages (from requests->-r requirements.txt (line 4)) (1.25.11) Requirement already satisfied: chardet<4,>=3.0.2 in c:\users\prince\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages (from requests->-r requirements.txt (line 4)) (3.0.4) Requirement already satisfied: idna<3,>=2.5 in c:\users\prince\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages (from requests->-r requirements.txt (line 4)) (2.10) Requirement already satisfied: cryptography>=2.8 in c:\users\prince\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages (from pyOpenSSL->-r requirements.txt (line 6)) (3.2.1) Requirement already satisfied: pyparsing>=2.4.7 in c:\users\prince\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages (from cloudscraper->-r requirements.txt (line 7)) (2.4.7) Requirement already satisfied: requests-toolbelt>=0.9.1 in c:\users\prince\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages (from cloudscraper->-r requirements.txt (line 7)) (0.9.1) Requirement already satisfied: cffi!=1.11.3,>=1.8 in c:\users\prince\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages (from cryptography>=2.8->pyOpenSSL->-r requirements.txt (line 6)) (1.14.3) Requirement already satisfied: pycparser in c:\users\prince\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages (from cffi!=1.11.3,>=1.8->cryptography>=2.8->pyOpenSSL->-r requirements.txt (line 6)) (2.20)

@TCVinNYC it's more related to module and not udemy-dl, try installing that module using pip and then let me know if the error still there.
I tried with python3.8 and python3.9 both can't reproduce the issue.
Install module using pip

python -m pip install m3u8 --upgrade or
pip install m3u8 --upgrade

remember module should be installed for the version of python you are using to run udemy-dl.
if you have multiple installation in your system and one of that is added to ENV let say python2 then pip install will install the module for python2 instead if actual python version, So double check when trying to install the modules.

Ohh that makes a lot more sense, thank you haha :)


fix(errors): Do not check for Google Cloud SDK if using an s3 bucket_url

Before:
>> shaka-streamer -i input.yml -p pipeline.yml -c s3://local-media/out/

Fatal error:
  Google Cloud SDK not found! Please install version 212.0.0 or higher of Google Cloud SDK.

After:
No error, it runs.

If tests are not running, could it be because your PR is in draft mode?  I think first-time contributors might need approval to run tests in GitHub Actions, but I don't see a button for that, presumably because it's just a draft PR.
Feel free to convert it to "ready", even if it's not completely.  :-)

@joeyparrish I just re-enabled the build_and_test.yaml Github action. Maybe it'll now show you something to approve?

It's working now I think. Not yet "ready" really, will comment again when it is.

Sound good.  I saw the approval button, and I clicked it.

@joeyparrish could you approve that Github action again? For some reason it's asking for approve upon every new commit..

Workflow triggered.
I'm a little confused about your change, though.  The cloud upload node is still using gsutil to upload to s3:// URLs, right?  So why would you not want to check for the Google Cloud SDK?

Ah, I didn't know that was even a feature of gsutil :) Will close this then.

No worries.  I see you had done a lot of the work needed to re-enable the test workflow.  Would you be willing to put up a PR that focuses on that?


Custom_fields isnt able to configure multiple fields

I'm not able to configure multiple custom_fields when creating a new Contract.
Expected Behavior
A new contract with the user-defined fields rufnummer, kundennummer, kundenkonto, rahmenvertrag, vertragspartner, rechnungsempfanger and tarif is to be filled in and created with the corresponding information.
Current Behavior
Writes error and doesn't create a new contract.
Invoke-WebRequest : The remote server has returned an error: (400) Invalid request.
In C:\Program Files\WindowsPowerShell\Modules\FreshservicePS\0.1.6\Private\Invoke-FreshworksRestMethod.ps1:191 Zeichen:28
+                 $results = Invoke-WebRequest @restParams
+                            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (System.Net.HttpWebRequest:HttpWebRequest) [Invoke-WebRequest], WebException
    + FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShell.Commands.InvokeWebRequestCommand

Possible Solution

Steps to Reproduce (for bugs)

Create multipe custom fields in a contract type
Fill in the corresponding variables with your own (approver_id, vendor_id, user-defined field names)

New-FreshServiceContract -approver_id XXXXXXXXXXX -contract_number CNTR-8 -contract_type_id 101 -cost 1 -end_date 2025-08-15T00:00:00 -name 0151/55063578 -start_date 2021-08-16T00:00:00 -vendor_id XXXXXXXXXXX -custom_fields @{"cf_rufnummer"= "4915112345678"; "cf_kundennummer"= "12345678"; "cf_kundenkonto"= "12345678"; "cf_rahmenvertrag"= "XX123"; "cf_vertragspartner"= "Cortonso AG"; "cf_rechnungsempfanger_1805356751"= "CORTONSO AG"; "cf_tarif"= "Business Mobil L mit Handy 3. Gen"}

Context
If I configure only one custom field as in the example below, it works, but I want to fill in every custom field at once.
New-FreshServiceContract -approver_id XXXXXXXXXXX -contract_number CNTR-8 -contract_type_id 101 -cost 1 -end_date 2025-08-15T00:00:00 -name 0151/55063578 -start_date 2021-08-16T00:00:00 -vendor_id XXXXXXXXXXX -custom_fields @{"cf_rufnummer"= "4915112345678"}

My goal is to create multiple contracts that will be imported from a CSV file. The contract type will be created and the custom fields will be configured with that contract type.
Your Environment

Module version used: 0.1.6
Operating System and PowerShell version: Windows 11 Pro and PowerShell 5.1.22621.2506

With your example and your recommendation, I was able to create a contract with customised fields.
Thank you very much!

With your example and your recommendation, I was able to create a contract with customised fields.
Thank you very much!


Update Matter SDK to show Authorisation dialog only for account login flow.

[Problem]
Authorisation dialogue should only be shown to user if user enabled it from the secure settings. matter_show_authorisation_dialog show the user selection .
[Solution]
Check the value of matter_show_authorisation_dialog before showing the dialogue. If it's 1 only then we show the dialogue.
[Test]
Tested-by: Lazar Kovacic<EMAIL_ADDRESS>

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.osamabb seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@osama10  could you sign the CLA? Robot says "osamabb" did not sign.


Drag and drop is broken... again

Such a fickle set of APIs. I've gotten around to creating a sample app to try and track down the issues, which are many:

[x] When dragging the very first item down, the recyclerview scrolls past the other items at mach speed, which is clearly not helpful
[x] When dragging any other item down, behavior is still kinda janky (but for the most part does work)
[x] Attempting to drag any item up will result in it being moved one space before it is forcibly "dropped"; again, clearly not helpful

So, we have a very interesting head-scratcher here. It seems that the following code from RealmBasedRecyclerViewAdapter.java was causing issues:
// Try to be smarter here and detect cases where an item has simply moved.
if (deltas.size() == 2 && areDeltasFromMove(deltas.get(0), deltas.get(1))) {
    if (deltas.get(0).getType() == Delta.TYPE.DELETE) {
        notifyItemMoved(deltas.get(0).getOriginal().getPosition(),
            deltas.get(1).getRevised().getPosition());
    } else {
        notifyItemMoved(deltas.get(1).getOriginal().getPosition(),
            deltas.get(0).getRevised().getPosition());
    }
}

But why is this so interesting? Well, once upon a time (when I released version 1.0 of this library, to be precise), this code block was the core of what made drag and drop work, but it is now the sole reason why it doesn't work...and I am at a loss for why that is suddenly the case.
I have done multiple tests, each using a different combination of the support libraries from version 23.1.1 through 23.3.0 and Android versions 21-23 (which were what I tested version 1.0 on), as well as testing each of the release versions of this library from 1.0 through 1.6.0 on the same Android versions to see if it was an issue with my code, the support libraries' code, some Android version weirdness, or a combination of one or more of those things.
Yet, amazingly, it is suddenly broken in /every single/ case, and (inexplicably) the only fix which works is to remove the above code.
Why "magical"? Because this code is the only place which calls notifyItemMoved() when the items get dragged past each other, which (as I understand it at this time) should be a necessity,  hence my confusion. I expected that commenting out the code would just break things in a different way, but instead I found that doing so fixed all of the issues I was having (see initial description).
So what actually happens now is that, when items get dragged, the RecyclerView somehow just knows that that has happened, and it perfectly performs all of the correct animations all of its own volition, without any of the notify*() methods being called. (I verified this by applying breakpoints to every one of the notify*() methods; none of them get hit as I dragged the items around. Ever.)
So, since this issue is technically "solved", I will close it.
However, I'm not exactly inclined to trust this seemingly magic "fix" to continue since I cannot figure out why code which worked perfectly at one point randomly broke in all cases (I suppose it's possible that its working in the first place could have been a repeated fluke, but I'll never be able to prove/disprove such a thing), and similarly cannot comprehend why the fix for the original fix breaking is to remove said fix.
With that in mind, I'm keeping the code block there, just commented out, in the event that it decides to revert to the other behavior at some future point in time. I will also leave a comment linking here for an explanation as to why it exists there.


Requested features

A basic instruction manual
Add miles
Include custom database fields. I have added a number of extra fields.
Allow locations that have been defined by name in GSAK. Such as Home.
What can I do with the exported flow sequence? Can I use it to create a macro or search filter?
Auto arrange the flow chart when adding a new block in the middle.
Allow selecting several blocks so they can be moved as a unit. This is useful for realigning the blocks after adding one in the middle.
I suspect that while several of the requests seem simple to me the programming will be lengthy.

I lot of issues stated in one. Maybe the next time add them seperatly so we can track the status. Otherwise items in this list might get forgotten.
A basic instruction manual

A wiki has been started on this website (github) and there is a video. This is just a start and I agree that it should be extended. But I will first focus on functionalitey and more videos.

Include custom database fields. I have added a number of extra fields.

They are included.

Allow locations that have been defined by name in GSAK. Such as Home.

Good point, will be added

What can I do with the exported flow sequence? Can I use it to create a macro or search filter?

Yes. I will create a video to show you how. However, version V<IP_ADDRESS> has an error that copies the file to a protected folder. This will be fixed in V<IP_ADDRESS>

Auto arrange the flow chart when adding a new block in the middle.
Allow selecting several blocks so they can be moved as a unit. This is useful for realigning the blocks after adding one in the middle

Tricky, because that would take some programming. But it is on the list todo (just nog a high priority)

I can resubmit let me know
On Nov 5, 2015 10:39 AM, "Robert Peters"<EMAIL_ADDRESS>wrote:

I lot of issues stated in one. Maybe the next time add them seperatly so
we can track the status. Otherwise items in this list might get forgotten.
A basic instruction manual
A wiki has been started on this website (github) and there is a video.
This is just a start and I agree that it should be extended. But I will
first focus on functionalitey and more videos.
Include custom database fields. I have added a number of extra fields.
They are included.
Allow locations that have been defined by name in GSAK. Such as Home.
Good point, will be added
What can I do with the exported flow sequence? Can I use it to create a
macro or search filter?
Yes. I will create a video to show you how. However, version V<IP_ADDRESS> has
an error that copies the file to a protected folder. This will be fixed in
V<IP_ADDRESS>
Auto arrange the flow chart when adding a new block in the middle.
Allow selecting several blocks so they can be moved as a unit. This is
useful for realigning the blocks after adding one in the middle
Tricky, because that would take some programming. But it is on the list
todo (just nog a high priority)
—
Reply to this email directly or view it on GitHub
https://github.com/GlobalcachingEU/GSAKWrapper/issues/6#issuecomment-154116087
.

No, its ok.
Just for future reference ;-)

thanks for the GSAKwrapper  it looking good

What about a display box to show the answers in  so you do not have to open GSAK up

or
2 Add a drop down for Views in GSAk
or  both would be great

taybee, please submit new issues and assign the correct label (request in this case).
What about a display box to show the answers in so you do not have to open GSAK up

What do you mean with Displaybox? You do not have to open GSAK in order to use the wrapper. More actions will be added for daily tasks such as export to GPX.

Add a drop down for Views in GSAk

What do you mean?

But please open new issues for these, otherwise these remarks wmight get forgotten.


DO NOT MERGE TESTS: Hvolschenk fix/text field custom validation

tests for https://github.com/material-components/material-components-web-react/pull/430

Codecov Report

Merging #436 into rc7.0 will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##            rc7.0     #436      +/-   ##
==========================================
+ Coverage   96.14%   96.16%   +0.01%
==========================================
  Files          55       55
  Lines        1841     1849       +8
  Branches      214      217       +3
==========================================
+ Hits         1770     1778       +8
  Misses         71       71

Impacted Files
Coverage Δ

packages/text-field/Input.js
95.78% <100%> (+0.38%)
:arrow_up:

Continue to review full report at Codecov.

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


audacity: 3.4.2 -> 3.5.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/audacity/audacity/releases.
meta.description for audacity is: Sound editor with graphical UI
meta.homepage for audacity is: https://www.audacityteam.org
meta.changelog for audacity is: https://github.com/audacity/audacity/releases
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.5.0 with grep in /nix/store/anksfs2c0w97scivz2yj7c2wasxrfay5-audacity-3.5.0
found 3.5.0 in filename of file in /nix/store/anksfs2c0w97scivz2yj7c2wasxrfay5-audacity-3.5.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

audacity

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/anksfs2c0w97scivz2yj7c2wasxrfay5-audacity-3.5.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A audacity https://github.com/r-ryantm/nixpkgs/archive/9260d293d04a969b45cc4736068865ecb42a2a22.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/9260d293d04a969b45cc4736068865ecb42a2a22#audacity

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/anksfs2c0w97scivz2yj7c2wasxrfay5-audacity-3.5.0
ls -la /nix/store/anksfs2c0w97scivz2yj7c2wasxrfay5-audacity-3.5.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

audacity

Maintainer pings
cc @lheckemann @veprbl @wegank for testing.

Result of nixpkgs-review pr 306032 run on x86_64-linux 1

1 package built:

audacity