Time to update the code?

Dear all, it is possible to change your code?
Change from here:

https://github.com/go-gomail/gomail
https://github.com/go-gomail/gomail/issues/188
https://github.com/go-mail/mail
https://github.com/Shopify/gomail
https://github.com/Shopify/gomail/issues/59
https://github.com/ory/mail

It has been unmaintained and maybe one or more vulnerabilities :/
There are:

https://github.com/xhit/go-simple-mail from @xhit
https://github.com/wneessen/go-mail from @wneessen

Linked to:

https://github.com/go-gomail/gomail/issues/182

Thank you, replacing this package is a long term goal for us but not something we are actively working on. A known vulnerability could make this a priority, but following through the links I can't find one. Do you have a link to a CVE / Vulnerability?

@cstockton: No link, the author has not published, please read publications but it must be changed:

https://github.com/go-gomail/gomail/issues/188
https://github.com/Shopify/gomail/issues/59


Why spacebar on checkbox and Enter key on select dropdown ?

Bug, feature request, or proposal:
Proposal (having the same shortcut ?)
What is the expected behavior?
In a form i can easily navigate with keyboard :

Input : focus > write > tab
Select : arrows > enter > tab
Checkbox : enter > tab
...

What is the current behavior?

Input : focus > write > tab
Select : arrows > enter > tab
Checkbox : enter > ... > enter > ... > ??? > roll on keyboard > hit space > OMG > tab
...

What is the use-case or motivation for changing an existing behavior?
Having Enter key to select a value from dropdown and the Spacebar to check a checkbox, in the same form it's very disturbing ! In the begining i was thinking the checkbox haven't keyboard controls...
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Latest
Is there anything else we should know?
What is the reason behind this behavior ?

My bet is:

consistency with standard controls

&&

Enter on checkbox would trigger form submit (in many cases) while Enter on select does not as it is "one level up" - it just selects option and closes list.

If you insist on Enter, I believe you are free to use (keypress) to catch Enter, cancel the event and toggle checked flag on checkbox. It would be confusing for many users though, IMHO.

It seems to be a standard : https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/ARIA_Techniques/Using_the_checkbox_role

The expected keyboard shortcut for activating a checkbox is the Space key.

@WizardPC you could close the issue then, probably

@PeS82 yep

You could still implement your own checkbox and override the keydown behavior :)

Sure :) But from what i know now, it's seem to be the standard behavior, so i will stay with it.


Disconnected containers

[Enter feedback here]
There is nothing on this page about how I get a container in ACR to a disconnected Azure Stack Hub running K8s, in my case running on a ship which only comes into port every three to six months.  We are trying to help the customer swap out vmware and move to microservices at sea, however there must be other scenarios where this is needed.

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

ID: d3914390-367c-471b-b653-7672f967f472
Version Independent ID: 08ecfe0f-a12d-9d24-8d25-a23b1aa1401a
Content: Azure disconnected deployment decisions for Azure Stack Hub integrated systems - Azure Stack Hub
Content Source: azure-stack/operator/azure-stack-disconnected-deployment.md
Service: azure-stack
Sub-service: azure-stack-hub
GitHub Login: @BryanLa
Microsoft Alias: bryanla

Hi @Deepfat , thanks for reaching out. We'll review this and update the document as appropriate

@Deepfat - thanks very much for your feedback on this. We've added your suggestion to the "Features that are impaired or unavailable" table, which points to a new entry in the ACR FAQ that provides additional how-to details.
#please-close


Generic transport layer is required

We need a generic transport layer, thus:
Terra <--> Transport layer <--> Networking library
So that we do not directly hook up to a networking library from Terra. We are currently using Aeron networking library, but end users might wish to migrate to a different networking library - for any number of reasons - let's not make it unnecessarily difficult for them by hard coding things into Terra.

We are using Venom as transport layer


New attribute disappears on binary_sensor after state change

What is your question
When I create a attribute on a binary_sensor that triggers the creation of the attribute, the attribute disappears after another state change of the binary sensor.
Is this by design? I would like the attribute to be permanent.
YAML configuration
---
alias: Oprit Hue
id: oprit_hue

trigger:
  - platform: state
    entity_id: binary_sensor.hue_reserve_motion_lumi
    from: "off"
    to: "on"

action:
  - service: python_script.hass_entities
    data:
      action: set_attributes
      entity_id: binary_sensor.hue_reserve_motion_lumi
      attributes:
        - start: "{{ as_timestamp(now()) |float(0) }}"

Screenshots
Animated gif!

Version
v1.1.0
Additional information
Logfile info:
2022-05-04 10:44:50 INFO (SyncWorker_2) [homeassistant.components.python_script] Executing hass_entities.py: {'log_enabled': True, 'action': 'set_attributes', 'entity_id': 'binary_sensor.hue_reserve_motion_lumi', 'attributes': [{'start':<PHONE_NUMBER>.053345}]}
2022-05-04 10:44:50 DEBUG (SyncWorker_2) [homeassistant.components.python_script.hass_entities.py] Python Script: hass_entities.py -> START of action: set_attributes
2022-05-04 10:44:50 DEBUG (SyncWorker_2) [homeassistant.components.python_script.hass_entities.py]   Entity: 'binary_sensor.hue_reserve_motion_lumi' -> New attribute 'start': '1651653890.053345'
2022-05-04 10:44:50 DEBUG (SyncWorker_2) [homeassistant.components.python_script.hass_entities.py]   DONE -> Attributes set for entity 'binary_sensor.hue_reserve_motion_lumi'
2022-05-04 10:44:50 DEBUG (SyncWorker_2) [homeassistant.components.python_script.hass_entities.py] Python Script: hass_entities.py -> END of action: set_attributes

Hi, I would say it depends on how the sensor is internally handled by the HUE integration.
It looks like the sensor it's recreated each time, instead of updated.
Consider that the script only set the attribute value, creating the attribute if not exists.
It doesn't delete the attribute, unless you don't use the delete_attribute action (which is not the case from what I'm seeing).
I'm afraid you need to adapt the trigger of your automation and set the attribute value for each sensor state change.

Oké, thanks for your explanation. Then I'll have to work my way around this behaviour. Nice script though!

👍


Taiwan is not a contury!

I found that in the country page, Taiwan is actually displayed as a country, and Taiwan is not independent. Taiwan is an inseparable part of Chinese territory!
This is a political issue. I hope it can be amended as soon as possible!!!!
我发现在国家页面中台湾居然显示为一个国家，而台湾并非是独立的，台湾是台湾是中国领土不可分割的一部分！
这是一个关乎政治的问题，希望尽快修改！！！！

Taiwan is not a contury Taiwan is an inseparable part of Chinese territory! I hope it can be amended as soon as possible!!!!

See #1019 and #1298


Headers are sanitized to lower case against Fetch spec

I raised similar issue in https://github.com/github/fetch/issues/503.
This should be fixed in this project too.

Looks like the spec changes behavior this year...
https://github.com/whatwg/fetch/pull/476

But wait:

The change shouldn't affect any JavaScript API behavior. They will still expose header names as lowercase even though the case given by the developer is used on the network.

Looks like the way it's going is to:

Store header name as-is (no normalization)
Compare header using case-insensitive search (so comparison works either way)
But when using accessors to get the headers, it will always returns in lowercase (for compatibility).

https://github.com/whatwg/fetch/issues/304

It was a shame as HTTP2 actually enforce lowercase, just that header normalization are breaking some old servers...

totally agree with this, I read the spec the same way, and I find the implementation forcing the lower-case to be a nuisance when working with inflexible middleware like prentender.js or a poorly-behaved server.

It was a shame as HTTP2 actually enforce lowercase header

Couldn't agree more 😞

Hey all... soooo @bitinn I had taken a look into this on Sunday and I agree it seemed pretty simple on the surface, and wasn't too bad getting implementing the header class to behave as per the updated spec. But I couldn't get my TDD~ish tests to pass.. and then I realized headers were still being lower-cased over the wire:
https://github.com/nodejs/node/blob/master/lib/_http_outgoing.js#L519,L525
So basically we could:

Monkey patch http's http.OutgoingMessage.prototype.setHeader - this seems like not a great idea, because we can't even "properly" do it by using a decorator or before/after pattern, since the functionality we need to skip is right in the middle of the function and I can't think of a good way to do it other than just re-implementing all of setHeader and validateHeader
Ditch http and drop down to net - this seems like an even worse idea because recreating the wheel
Try to get some movement upstream

The applicable open issue is: nodejs/node#3591, and it's been sort of dead. I have started crafting up a response but I haven't decided whether to post there or open a new issue that hasn't been derailed 20 times over 45 messages.
Looking for some input to how you'd like to move forward.. or maybe there's a solution I haven't come up with, thanks :)

Ah man, I was hoping it wasn't something in the internal of http, in this case I say we put this issue on hold. Agree? @TimothyGu

+1 from me.

But thx @jkantr for digging into this issue!

So - fun fact - I'm running into an issue where the header's name.toLowerCase fails because the name is undefined. It tanks headers.get. Is this something that would come under this ticket?

I'll close out this issue. For the Node.js issue, there isn't really anything actionable from our side. https://github.com/bitinn/node-fetch/commit/1592ca1148e657110fd6f64dc8b29a576cd1da1c should help nevertheless.

Information in this thread about Node.js http lowercaing headerse are not longer accurate, i tried it just now and was able to send headers with uppercase.


net: lldp: Fix timeout triggering if multiple workers

The code was not working properly if there was multiple timers
that were triggered in different times.
Signed-off-by: Jukka Rissanen<EMAIL_ADDRESS>

Codecov Report

Merging #9503 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #9503   +/-   ##
=======================================
  Coverage   52.22%   52.22%
=======================================
  Files         212      212
  Lines       25948    25948
  Branches     5577     5577
=======================================
  Hits        13551    13551
  Misses      10140    10140
  Partials     2257     2257

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 47889cd...249f80a. Read the comment docs.


多卡推理报错

多卡推理报错，提示中间计算部分变量不在同一个 device 上

Thanks for the feedback!
We will open source the sample code for multiGPUinference today.
If you have more questions or need more help for your bug, you can provide your runtime environment and all the code!


Small tweaks to make teximg interchangeable with text

It would be nice to make teximg and text more similar. This way one can start using text as a first approach and then later change to teximg if needed so.
For comparison
f = text(0,0, text = L"E = mc^2", align=(:left, :bottom))
teximg!(L"E = mc^2", position = (0,0) , align=(:left, :bottom))
teximg!(L"E = mc^2", position = (0,0) , align=(:right, :bottom), scale=14/12)
teximg!(L"E = mc^2", position = (0,0) , align=(:right, :top), scale=14/12)
f

produces

Note the following issues:

The vertical alignment is reversed.
For Makie, the default font size seems to be 14pt, whereas MakieTeX compiles the LaTeX string with 12pt. Can this be fixed easily?
Could the arguments of both methods be more similar? Could we have teximg(0,0, text = L"E = mc^2", align=(:left, :bottom)) instead of teximg(L"E = mc^2", position = (0,0) , align=(:left, :bottom))
Could the baseline of the image be inferred?

That sounds like a great idea, thanks for proposing it!  I don't currently have the time to solve this but will hopefully be a bit more free in December.
The alignment issue is a bit strange, we should definitely correct that.  The default font size is from an earlier version of Makie, that can also be changed.
Inferring the baseline of the image is unfortunately not really possible, unless TeX can pass us some info through the command line somehow?

That sounds great! If I can help somehow, I would try to do it so in my spare time.
For the record, I found this discussion about the baseline and other related issues such as the weird "clipping" MakieTeX.jl also experiences (for the moment), https://mactextoolbox.sourceforge.net/articles/baseline.html.
See the clipping occuring this time from below.

It seems that it could be solved with something like
\geometry{margin=1pt}


Add snapshots as YAML files to track name/spec

This also ensures that only snapshots which are defined can be used by a
feature.
Snapshots are still not published, and this does not affect the built
JSON at all.

Maybe a single yml file as the snapshots allow-list would be enough, but this works, too.

I feel the pull of this argument, and went back and forth, but ultimately favored similarity of structure between features, groups and snapshots. Out of the three, snapshots look the silliest as separate YAML files. If we end up not adding anything more to the schema, perhaps we should squash it into a single file again.
But I'm glad you think this works, because then I can merge :)


Use romans.py

Can you use new file in plugin?
from .romans import *

in the sublime one?

thanks.

are you going to sync today by the way?
On Thu, May 17, 2018 at 9:46 PM Alexey<EMAIL_ADDRESS>wrote:

thanks.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GCTsalamagkakis/Romanize/issues/2#issuecomment-389969208,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkyHKCB0Kwn2WVbP-ubBNh1Tghlhvn8kks5tzcV2gaJpZM4UDhCR
.

I will sync after you apply pull-request, is it ok?

I have synced py file with you- https://github.com/CudaText-addons/cuda_romanize

Alright, im really new to git so it takes me some time to figure out whats
going on etc, sorry
On Thu, May 17, 2018 at 10:41 PM Alexey<EMAIL_ADDRESS>wrote:

I have synced py file with you-
https://github.com/CudaText-addons/cuda_romanize
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GCTsalamagkakis/Romanize/issues/2#issuecomment-389985331,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkyHKBYphUICF3FzhKXv6JT-3GzTjf5eks5tzdJhgaJpZM4UDhCR
.


Can't import ipcRenderer

Not being able to import ipcRenderer in any of the components. I think it's a webpack  issue, but I'm not sure

@codedepon Can you elaborate more about the  error you getting?
If you trying to access node specific stuff in renderer process, try the config mentioned here https://github.com/codesbiome/electron-react-webpack-typescript-2022/issues/18#issuecomment-926057413

That is considered unsafe, I've found a workaround, but didn't have time to propose it, if you have time, I think this should be included in the boilerplate. Here's the solution https://github.com/electron/electron/issues/9920#issuecomment-947170941
Also it's outside of this topic, but it would be nice if there was a support for css modules, and I've already opened a discussion about it and described how they would be enabled.

That is considered unsafe, I've found a workaround, but didn't have time to propose it, if you have time, I think this should be included in the boilerplate. Here's the solution https://github.com/electron/electron/issues/9920#issuecomment-947170941

The safe approach of contextBridge.exposeInMainWorld() is already being used for custom window events in misc/window folder.

Also it's outside of this topic, but it would be nice if there was a support for css modules, and I've already opened a discussion about it and described how they could be enabled.

For the CSS modules approach, we will be testing the css-loader config you advised in discussion.


Cannot automatically create expert or master difficulty worlds

Currently providing a difficulty value of 1 or 2 in the config does not cause expert or master mode worlds to be generated.
This is because the difficulty parameter is not a command line parameter and is only available in the config file
I will have to figure out a way to generate a config file and have it be loaded by the server

Verified this works


Update to Docusaurus v3

All is in the title.
I think we can update the documentation with the release candidate (no futur breaking changes normally).
https://github.com/facebook/docusaurus/discussions/9429
https://twitter.com/sebastienlorber/status/1715309311767831033

Seem to be getting this error when navigating to the API spec.

Yeah, I think it is necessary to wait for a redocusaurus patch or suggest one (I'll take a look).

@triphora I updated the dependencies and it works now.


Add exponential backoff to get_response

If people like this I can add it to get_embeddings and get_transcription. To test turn off your internet run the following then turn your internet back on:
import asyncio
from spice import Spice, print_stream

client = Spice(retry_count=10)
asyncio.run(
    client.get_response(
        [{"role": "user", "content": "Please write a poem!"}],
        model="gpt-3.5-turbo",
        streaming_callback=print_stream,
    )
)

Looks like there aren't actually any apiconnectionerrors.  I think you're right we don't need this


Adds tutorial for using Psi4's SADGuess class

Description
I added Self-Consistent-Field/RHF_libSADGuess.py, which uses Psi4's SADGuess class to produce a SAD Guess.  This has been doable since my last PR to Psi4, which included the necessary python bindings.  The file includes a toggle which allows one to compare the SAD and CORE guesses.
What are your new additions? Please provide a brief list.

New Features

SAD Guesses

Changes

Adds Self-Consistent-Field/RHF_libSADGuess.py

Any questions for the community?

This is not a tutorial on how to implement a SAD Guess, merely how to use a Psi4 object. Is this desirable to Psi4numpy?

Status

[x] Click when ready for review-and-merge

Codecov Report

Merging #36 into master will decrease coverage by 0.14%.
The diff coverage is 86.81%.

@@            Coverage Diff             @@
##           master      #36      +/-   ##
==========================================
- Coverage   94.81%   94.66%   -0.15%
==========================================
  Files          36       37       +1
  Lines        4357     4445      +88
==========================================
+ Hits         4131     4208      +77
- Misses        226      237      +11

Impacted Files
Coverage Δ

Self-Consistent-Field/RHF_libSADGuess.py
86.81% <86.81%> (ø)

Configuration-Interaction/CI_DL.py
98.36% <0%> (-0.02%)
:arrow_down:

Self-Consistent-Field/ROHF_libJK.py
98.01% <0%> (+0.93%)
: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 4e1d82d...d2b690b. Read the comment docs.

Hmm, I think you will at least need to write out a SAD guess to compare it to. This should very simply however, build a NMO x NMO matrix, put the densities on the diagonal, and your done. You can use scf_e, scf_wfn = psi4.energy("SCF", molecule=atom_mol) to get the densities out of Psi4.

@schiebermc Would really love to get this in. Could you make the minor fixes required to do so?

@dgasmith would you like this comparison to be in a separate file from Self-Consistent-Field/RHF_libSADGuess.py? If not how would you prefer me to organize this?

I think it should be the same file, but instead of doing iterations/normal guesses I would just use the python based SAD machinery to do a comparison. The bulk of the code however, should be towards computing the SAD guess itself.
Basically what you want to do is the very simple:
D = np.zeros((nso, nso))
start = 0
for atom in molecule:
    start
    scf_e, scf_wfn = psi4.energy("SCF", molecule=atom)
   sl = slice(start, start + scf_wfn.nso())
   D[sl, sl]] = scf_wfn.Da()

Cocc = partial_cholesky(D)
...

The compare your density and Cocc vs the C++ SAD density. Thats all pseudocode, but should display the basic idea. The idea here is that we show how to compute SAD rather than just how to use the Psi4 python bindings.


MH-13286 Remove unused functionality from MetadataField

@KatrinIhler willl this prevent the temporal field from being accessible?

@karendolan This PR only affects the TemporalDate- and TemporalDurationMetadatafields as they are not used by any opencast code as far as I could tell. The TemporalStartDate, TemporalStartTime- and TemporalMetadataField are unaffected by this.


Write Part - 3 Step - 6 & unit test

Implement Step06Controller and write all unit tests. Verify the display name of the test conforms to the pattern of "Part X Step Y". Follow the standardize naming, comments, etc. to be consistent.

Very close to Part 01 Step 15 (#20)


Yoga: add option to call measure callback on all nodes

Summary

When using Yoga in combination with external layout systems (brown field scenarios) Yoga leaf nodes can be subtrees w.r.t. the external layout system.
During the measure phase, these subtrees need to be measured by the external layout system - even if the Yoga node's mode is YGMeasureModeExactly.
This change adds a preprocessor conditional that can be set by out-of-tree platforms to always call measure for all nodes that have the measure callback.
Changelog

[General] [Added] - Yoga: add preprocessor option to always call measure callback.
Test Plan

Verified that this behaves as intended.

For some more context, this change was needed for the Win32 version of react-native. @acoates-ms has some more context as well.

@sagokhal To merge the change into Yoga you need to merge that into Yoga repository. After it merged, it will be automatically sync-ed with React Native repo.
Could you please open another PR here: https://github.com/facebook/yoga/ ?


Added a prime number list up to 200

Added a prime number list up to 200, plus two useful links to other resources. Happy Hacktoberfest!

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding. ✈


Make The NavBar

The navbar that will allow people to switch pages with ease

done

Looks good, but I'm not sure what the instructions mean by having it "dynamically displays other pages in the app"


Fix/27 concurrent modification

As described in #27 any call to subscribe or unsubscribe from a newState would create a ConcurrentModificationException because the list of subscriptions was modified (by un/subscribe) while iterating over it (from the setter of _state).
By using a CopyOnWriteArrayList the iteration over the original list finishes (without modification) while any future access of subscription reflects the mutation.
Fixes #27

LGTM

I have doubts over performance because CopyOnWriteArrayList would create a copy everytime subscribe/unsubscribe is called of a list which carries all active subscriptions. Also since by definition it is thread safe, it would be slower than general arraylist.

While the performance issue is a concern in my mind too, the only other way out seems like a synchronized list which has a worse performance characteristic in case of more read operations, which would be the case I'm guessing, with the CopyOverWriteArrayList(COWAL) the read operations will remain more performant but the writes will suffer.
Should be fine for lower number of concurrent writes, but will slow down significantly for larger number because of the way JVM handles the COWAL

I agree that synchronization incurs performance overhead.
If we don't rely on COWAL we have to put in explicit locking whenever we access subscriptions - which also has performance overhead and makes the code harder to understand & reason about. The performance overhead might be less though, because we don't need to copy if we properly guard against concurrent access.

@kmmraj Waiting for your inputs on this, to confirm, merge and close the issue.

@NemoOudeis The README.md update to mention the existence of COWAL and comment above the COWAL declaration to ensure concurrent reads should be always greater than writes should be good enough to warn library users and avoid OOM.

I will update the README.md to mention the issue.

@NemoOudeis Can you take a fresh pull, to ensure the build is successful, some issues on Travis Side for Oracle JDK, I have fixed them in my latest merged PR


including lightning talks?

Do you want slides from lightning talks too?

I say yes! Especially because the lightning talks don't currently leave an imprint outside of YouTube.

Closed based on tweet! https://twitter.com/dillonniederhut/status/1017911567482945538


Update Readme

The first sentence should something like:
"stacks-bitcoin-api is a web library to easily verify BTC transactions on Stacks"
In Background:

Remove or clarify This project allow smart contract developers to verify transactions which occured on the Bitcoin blockchain.

In How it works:

Replace Once a proof is assembled, the entirety of the proof is sent to a Clarity smart contract endpoint. The smart contract is responsible for forwarding the proof to the clarity-bitcoin.clar contract for confirmation of its validity. with something like Once a proof is assembled, the entirety of the proof can be sent to a Clarity smart contract endpoint. This can be the clarity-bitcoin.clar contract or a custom contract the delegates to clartiy-bitcoin.clar.`

In Getting Started:

Add example code or remove Example code coming soon.
Add code snippets from tests to show how to use stacks-bitcoin-api

In Limitations:

The section about limitations should be updated.

In Coming soon:

Update the section or remove it or add github issues

In Acknowledgements:

Fix typo Freiger -> Friedger

Fix typo Freiger -> Friedger
Apologies. 😊

The section about limitations should be updated.
As part of the updates I added a test for coinbase transactions and they can be verified. However, the coinbase message isn't available.


Update the kitchen

Use current Chef omnibus package version
Update CentOS 6 box to the latest available

@opscode-cookbooks/ociv


tag id contains dots

Hi,
Great extension. I am using it in a project where the element ids contain dots and jQuery does not like them.
so I escape them using the standard way:
jQuery("#user\\.attributes\\.country").countrySelect();
in order to initiate the countrySelect. Although this fails with the _code field.
I suggest a change where by one can pass in an additional id to define where to store the code for the country. Also to remain backwards compatible. Perhaps you could also escape dots ->
when you are looking for the element in the code ->
		this.countryCodeInput = $("#"+this.countryInput.attr("id")+"_code");

What do you think ?

That seems like a perfectly reasonable solution to the problem if naming the field to not have dots is not an option for your project (which is good practice in general).
So something like this to avoid the problem with periods in the ID string?
var inputId = this.countryInput.attr("id");
this.countryCodeInput = $('input[id='+inputId+"_code");

If that resolves your issue, then fair enough, and we can get that packaged up and merged in.


allow any borders to be skipped

Allow object as borderSkipped and borderWidth.
Examples:
borderWidth: 2,
borderSkipped: {top: true, left: true}

borderWidth: { left: 0, top: 0, right: 2, bottom: 2},

Pen (https://github.com/chartjs/Chart.js/pull/6047/commits/9c993a6f6cb1171b16a552494788dd85304dc8b8)
Fixes: #5565
Closes: #5709, #4681
Related: #3293 (fixed in master already)

Also related is https://github.com/chartjs/Chart.js/pull/5262 which implements the 'none' option as well by checking if borderSkipped === null

Could we use an array for specifying multiple borders instead? So instead of 'left right' we could do ['left', 'right']. It looks like right now it would accept 'leftrighttop' as a valid value, but I think it might be nice to be slightly stricter

Updated per comments

Do we still need to make borderSkipped an object if we're also making borderWidth an object? It seems like it'd be simpler to only make the latter an object

Do we still need to make borderSkipped an object if we're also making borderWidth an object? It seems like it'd be simpler to only make the latter an object

I think we should probably just deprecate borderSkipped, as everything can be done with only borderWidth. But didn't want to remove it just yet.

One thing that might arise from this, is the fact that top/bottom for vertical and left/right for horizontal are actually start/end.
Changing that would be a breaking change - so probably another thing that should be corrected in the next major.

I think it's confusing to be able to set borderSkipped as an object, but not be able to set multiple borders skipped. I agree we should deprecate, borderSkipped, but I'm still not sure why we need to expand it with new functionality in the meantime. We can document in the borderSkipped documentation that you should use borderWidth instead as it's more flexible and allows for skipping multiple borders

refactored quite heavily to make it simpler (imo), better or not?

Added some image based tests

After talking a bit more with @kurkle, it seems that the following makes sense (at least for v2):

do not deprecate borderSkipped but keep supporting only one side - inverted
add support for borderSkipped: false to display all rectangle borders
add support for borderWidth as object - not inverted (top is top, always)

Thoughts?

Here are some use cases:

Users who want to hide border at origin should keep using borderSkipped: 'bottom' (default)
Users who want to show only the "end" border (e.g. #5071) would use :

borderSkipped: 'bottom',          // e.q. origin
borderWidth: {top: 2, bottom: 2}  // left/right implicitly 0

Users who want to show all borders but with different style (not inverted for negative value:

borderSkipped: false,
borderWidth: {top: 4, bottom: 2}

Users who want to show a border differently based on the current value:

borderSkipped: false,
borderWidth: function(ctx) {
    var neg = ctx.dataset.data[ctx.dataIndex] < 0;
    return {top: neg ? 1 : 4, bottom: neg ? 1 : 4};
}

The last use case (4) may not be common, so I don't think it's an issue to ask for a scriptable option. Also, the condition (< 0) is really up to the user.

That all sounds reasonable to me

Updated to comply https://github.com/chartjs/Chart.js/pull/6047#issuecomment-462924957
@benmccann can you come up with something better to docs? 😄

Are these glitches due to antialiasing?
test/fixtures/controller.bar/borderWidth/indexable-object.png

test/fixtures/controller.bar/horizontal-borders.js

Maybe we can fake the border using a path if the border widths are uneven.
ctx.beginPath();
ctx.rect(/* outer rect */);
ctx.rect(/* inner rect */);
ctx.fill('evenodd');

Good catch, @nagix! I did not notice those.
Filling between rectangles does not work for skipped border (in chrome). But I'll try outer rectangle +manual path and fill. So lets not merge this yet.

Issue found by @nagix was not that simple to fix. With no skipped border, rectangle approach works like a charm and is a really compact solution. It does not work with skipped borders however.
I tried using a rectangle + path, but that same issue lets the fill leak in some cases.
So here is a new solution. While at it, added borderRadius as it was a natural fit.
This got way bigger than my intention originally was, but at least I have explored some options :)
(description updated with a new pen)

@kurkle What happens with the rectangle approach with skipped borders? Does zero-width border still remain visible?

@kurkle What happens with the rectangle approach with skipped borders? Does zero-width border still remain visible?

Firefox is fine, Chrome draws a faint line there.

I see a problem when borderRadius is greater than half of the bar width. Also, the inner shape is a rectangle even if borderRadius is greater than borderWidth.

If we don't take the arc approach, another option would be to use a clip in the same way as the arc element (link). If you still see glitches on the edge, you can shift that section of the border a little bit outward so that it is completely outside the clip area.
ctx.save();
ctx.beginPath();
ctx.rect(/* outer rect */);
ctx.clip();
ctx.beginPath();
ctx.rect(/* outer rect minus half of the thickest border width but shift outward based on the width of each section*/);
ctx.lineWidth = /* thickest border width */;
ctx.stroke();
ctx.restore();

If we don't take the arc approach, another option would be to use a clip in the same way as the arc element (link). If you still see a faint line on the edge, you can shift that section of the border a little bit outward so that it is completely outside the clip area.

I'll do another PR using that approach later.

Closing in favor of #6077

@nagix about the clipping approach, according the MDN:

The CanvasRenderingContext2D.clip() method of the Canvas 2D API turns the current or given path into the current clipping region. It replaces any previous clipping region.

We are already clipping the canvas to the chart area, so it may not work in this case.

CanvasRenderingContext2D.save() and CanvasRenderingContext2D.restore() save/restore the current clipping region, so I think it should be no problem.

The outer clip was introduced in https://github.com/chartjs/Chart.js/pull/3658 to clip overshooting bars / lines. So this inner clipping will ignore that. I'll see if there is a easy workaround.

Ah, right, the problem is the solution in this PR will ignore the outer clip...


Implement admin UI

Currently there is a basic admin UI present in the frontend. This should be implemented fully. It should feature:

A simple way to create an archive and get a sharable link to send to someone
Control over existing archives (remove them etc.)
Possibility to search for files and archives
Ability to download and decrypt a file

It would be nice with the possibility to download an entire archive at once, this of course would induce a high memory load as it is unlikely that the files may be streamed, decrypted and added to a zip file in a streaming manner.


Support for Firefox and IE

Will taiko support firefox and IE as seamlessly as chrome?

Gauge will not support Internet Explorer because it's been deprecated by Microsoft.
Please follow this issue https://github.com/getgauge/taiko/issues/112 for Microsoft Edge support
and this issue for updates on Firefox support https://github.com/getgauge/taiko/issues/301


Create a Switch Component

PatternFly-ng currently does not have a switch component (as seen PatternFly Core and PatternFly-React). As such, one should be created so that it can be used by those that consume PatternFly-ng.

Examples of switches using the HTML checkbox and CSS:

CSS Switch with Labels
CSS Toggles without Labels

The switch component in Patternfly core is based on Bootstrap and jQuery. However, it appears that the Patternfly-react component also uses the same Bootstrap selectors (e.g., bootstrap-switch-label).
I'm wondering if it would be better to follow Patternfly-React and base this on the existing Bootstrap implementation or use your new CSS?

I think it just depends on if we want to rely on an external library or just rely on something directly inside of PatternFly-ng. I'm fine with Bootstrap's, but it still has to be styled a bit differently to match the PatternFly styling.

The Bootstrap styling overrides should already exist in Patternfly?

As long as it uses the same classes, then yes, it will inherit what is already there.

Closing due to inactivity.


[PBD-361] Copay - Add ability for Particl paper wallet import

Allow the paper wallet to sweep Particl or Bitcoin paper wallets

@dasource  tagging on Henk's request :)


Extend compiletime.testing.typechecks with certain transform phases

Adds transform phases to the scala.compiletime.testing.typechecks and typeCheckErrors methods.
At first I attempted adding all of the pre-erasure transform phases (as suggested here), but some caused issues (notably CrossVersionChecks, which in this configuration with the loaded symbols from current compilation could crash the compiler). Ultimately it's easier to add a few that are necessary, than filter out all of the unwanted ones.
So I ended up with:

InlineVals - adding checks that while not necessary for the issue, I believe could still be useful
ElimRepeated - necessary for RefChecks
RefChecks - needed by the issue fixed here

I made it so this can be easily extended with more MiniPhases/MegaPhases, if that is necessary.
Fixes #18150

Actually, I would have loved the option to use this feature for my compiler plugin.

there would not be many users of such a feature (but of course I may be mistaken here)

Indeed those are strictly compiler plugin writers. Their indirect usage could be big, so a good way to test is an advantage. That said, there are workarounds like publishing the library within the CI and running the remaining tests as external program calls.

@odersky Not many people touched those compiletime.testing.typechecks methods before, so I asked you for the review. With that said, do feel free to reassign (even back to me, and I'll try to find someone else).

Rebased to be able to properly run open community build (no changes in "Files changed")

I might have run the open community build a little too early, but nevertheless, here it is: https://virtuslab.github.io/community-build3/2024/11/26/3.6.4-rc1-20241125-08f407d-snapshot.html
It shows 16 regressions since 3.6.4-RC1-bin-20241121-5d1d274-NIGHTLY. With that said, none of those look related to this PR. I set the compile and run tests option when running the snapshot, which I don't believe is usually run, which might have contributed to that number (no errors pointed to typechecks or typecheckErrors, none were crashes)


FHIR Resource Type Filter

Requested Feature
Modify the FHIR exporters to only export specific resources, similar to how the CSV exporter can include/exclude specific files:
https://github.com/synthetichealth/synthea/blob/c266ab656670fb7b7506b892245060944e4d9576/src/main/resources/synthea.properties#L36-L43

Any update on this?
I have found that I can hack a solution for this by commenting out certain parts in org.mitre.synthea.export.FhirR4.java. In my case the lines for Claim and ExplanationOfBenefit.
https://github.com/synthetichealth/synthea/blob/5d90e086985aecf69e8a1140d9793ad8f104fe73/src/main/java/org/mitre/synthea/export/FhirR4.java#L267-L337
Would you recommend any other way? Ideally the resources I want to omit should not be generated at all, but I guess there are some dependencies between these.

I don't believe anyone is actively working on this yet. If you are motivated enough to issue a Pull Request, we'd happily consider it.
The hack you are using is fine for a quick local change, but you are correct that there are dependencies... or at least, references between the resources. Ideally each time a reference is created, it would instead be conditionally created based on what resources are being included/exported. This would need some pretty good test cases though to validate the results (broken references would be invalid).

Implemented in #1178


Handle binding correctly in join and other plans

The default logic for binding the parameters of a stored procedure to plans is here:
https://github.com/cmu-db/peloton/blob/master/src/planner/abstract_plan.cpp#L65
Basically it just calls the binding function of all its children.
If there're certain fields in the plan that need to be bound, such as predicate expressions, we have to customize the binding function. We already have some overrode functions for InsertScanPlan, UpdatePlan, etc. However there're still some plans that need to be fixed. For example if there're some parameters in the join condition, then we cannot handle it (assuming we can generate a join plan).

Fixed by @hzxa21 and @chenboy


Is there a way to attach different ID's to different segments of one same chart?

Hi there, first of all, great library!
I've been trying to attach js event handlers to different segments of one donut chart. It's obviously easily possible for the whole chart but is there a way to do it for separate parts?
Basically if this wasn't bad practice and would work, I'd be happy:
$( ".ct-series-a" ).on( "click", function() {
       console.log( "Segment a was clicked!" );
});

Or if there was an option like this for the series hash:
idName: "myId"

Couldn't hack it to work with the className option either.

Hi there! Maybe this jsbin helps you out? http://jsbin.com/jedapu/edit?js,output

Forgot to push at work, gonna try this out tomorrow! Thanks man!

Sure, the thing you always need to remember is that chartist re-creates DOM, sometimes very rapidly (on updates, resizes etc.). In order to create sustainable listeners use delegation:
$(document).on('click', '.ct-series-a', function() {
       console.log('Segment a was clicked!');
});

Cool, the jsbin is working for a pie chart. Let's see if I can make it work with the animated donut! Thanks for your help!

@NicoSa what parts would you give an ID and for what purpose? Can you give an example?

I thought it would make it easier to attach event handlers to the elements but it would still be the same way you suggested, which did the job!


Revert "release: 2023 01 11 (#29)"

This reverts commit eec50f818e0ff91ff9715cfcfdb81fdcd9209a72.

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


[natives][loader] New API for fully annotated MoveTypeLayout

Added API that generates a MoveTypeLayout from a loaded Type, but adds all struct tag information
Exposed new loader API in the native context
Exposed the function in the session for completeness

Motivation

I have some native function code for tests that fully walks a MoveValue, but to do so, I really need the type information for the struct. To get this, I need a MoveTypeLayout that uses MoveStructLayout::WithTypes instead of Runtime
Happy to put this somewhere other than the loader (the caching of the results is nice but not super important). Maybe @vgao1996 will have some ideas

Test Plan

Using it on another branch, and will be used extensively (so we will effectively have end-to-end tests for this function).
But we should probably have tests for this... just a bit hard at the moment as none of these functions have their own test suite

Sounds like a reasonable approach to the problem you're trying to solve... I could imagine us offering some kind of traversal API in the future so we can say goodbye to all of these layouts, but we can work on that later.
I'll take a closer look tomorrow at some of the details

The MoveStructLayout is becoming more and more of an abomination. Let's think about alternative solutions that can address its use cases...

Agreed. I'm wondering if we basically want to split it into a runtime layout version, used internally by the VM, and an annotated/decorated version, which is probably more useful for external tools that don't have a full view of the world to rely on (like the VM sort of does internally)
We've been using MoveTypeLayout::WithTypes a lot on Sui for more or less annotating the layout of the struct and then syncing that with some sort of Rust type (with the same layout). Worth considering this sort of paradigm when thinking about a more externally facing API vs one used by the VM


Problem with peerDependencies while installing

Hi there.
I was trying to install this module but I've received this errors:

I don't know what it is or how to solve it. I hope you guys can help me.
Thanks.

So, my knowledge of NPM package management is a bit insufficient. Can anybody advise what the best strategy is here? Since RN updates on a weekly/biweekly cycle, whats the best option here? Update our release every week/s or change this to be very accomodating for future versions?

gb-native-router requires react-native >= 0.12.0 in its peer dependencies. And that works well for all versions except rcs, that do not fall into that >=. See how react-native-vector-icons deals with that.

So, there is not a definitive solution, right??
And I'm not be able to change that by myself, I have to wait for the module's creator to do it, is that right?
Thanks.

I patched that up in the Pull request referenced above. Just copy that code into your node_modules folder.. We have to wait for @MikaelCarpenter to merge and make a new release.

@sethx although I copy that code into my node_modules folder, it doesn't work anyway.
When I do npm install gb-native-router or npm install gb-native-router --save it overwrites my package.json

@jvlobo: Why not stick to 0.15.0 ? Is there anything you absolutely need from a RC version?

I don't think so, I've just updated React Native and this is the latest version, nothing else...

It's a prerelease version, you're better off using 0.15.0 or you will most likely run into even more bugs than usual :) just downgrade whatever is depending on 0.16.0rc

Thanks a lot buddy, with that version it works perfect :)

Welcome!

The module is correctly installed but there are a bunch of errors by downgrading React Native to 0.15.0 :/

Such as?

http://puu.sh/lD1LF/5b536e3a0f.png

That's a whole different issue altogether.
I'd say your node_modules folder is borked. Delete it and redo npm install.

To workaround it manually, upgrade to npm3 that ignores that requirement.

I'm still having this problem... I was waiting for @MikaelCarpenter to see if he accepts @sethx pull requests but it seems like he's not over here.
So, I updated package.json inside gb-native-router module folder and when I go to run my project on Xcode I get this:

I don't know if you guys could help me out with this because I'm unable to develop this project because of those errors.
Thanks :)

@sethx so I'm merging the PR to update the dependencies for now, but is this really best practice for a module?
Would it be better to require the most recent versions of RN or allow a large range of versions?


Add Victoria College scraper

https://www.victoriacollege.edu/courseinstructorinformation
This one is fairly complicated.  There may be a way to reduce the number of requests, but at least with the approach I took, it's five callbacks.  There may be a few kinks to work out still.
This should be ready for review, but I'm leaving the dirty commit history in for reference.

Looks like you could make a scraper for LSCO that's pretty similar to this one.

@HarrisonGregg I originally tried to follow your advice of getting "syllabus urls directly from the search results page", but after attempting that a bit in https://github.com/opensyllabus/osp-scraper/pull/52/commits/8f8704df0aa1a1fa04ff394989292f67b0a69ef8, it seems like its hard to select both the syllabus link and its appropriate anchor (especially cause sometimes the syllabus link won't exist).  I could probably nail it if I spent more time working on it, but I think it might just be easier to stick to the old method, even if it is one more request.


FileNotFoundError: [Errno 2] No such file or directory: '/logs/exp_0013/config.yml'

I am successfully able to run all the experiments on the docker container. Now i am trying to reproduce on my local ubuntu environment, but i am getting the error while running this code. Please looking for your kind consideration and needful help.
How can I resolve this error or how can I set the path of logs and weights files? I am also attaching the screenshot of my environment where I am actually running the jupyter notebook.

I think exp_log_dir is set to None in default.
You have to set this correctly.
For examples, If your config.yml is located at "/home/irshad/configs/exp_0013/config.yml",
exp_log_dir should be set to "/home/irshad/configs/".

I think exp_log_dir is set to None in default.
You have to set this correctly.
For examples, If your config.yml is located at "/home/irshad/configs/exp_0013/config.yml",
exp_log_dir should be set to "/home/irshad/configs/".


Counter-strike executable not found

Describe the bug
Trying to watch a cs2 demo, which i've imported and analyzed, but when I click watch it pops up with

Demo links
No response
Steps to reproduce
Pretty hard to reproduce on your end I'm sure, but my CS is installed on my F: drive

Analyze demo
Click watch
Observe error

Expected behavior
The demo to load up in CS
How often does this bug happen?
Every time
Log file
Sun, 03 Dec 2023 10:38:56 GMT | LOG | server | WS:: message with name is-cs-running and uuid f8e8d4fe-8e7a-470d-9acd-744e2a7255e0 received from renderer process
Sun, 03 Dec 2023 10:38:57 GMT | LOG | server | WS:: message with name watch-demo and uuid 664bcbe6-ae9c-40ed-b71b-5305049dc913 received from renderer process
Sun, 03 Dec 2023 10:39:01 GMT | LOG | server | WS:: message with name add-demos-to-analyses and uuid cf4218ea-db48-408b-888d-4d05008d8eee received from renderer process
Sun, 03 Dec 2023 10:39:01 GMT | LOG | renderer | WS:: message with name "demos-added-to-analyses" received from server
Sun, 03 Dec 2023 10:39:01 GMT | LOG | server | starting demo analyzer with command "C:\Users\aria\AppData\Local\Programs\cs-demo-manager\resources\static\csda.exe" -demo-path="C:\Users\aria\Desktop\1-d1dd5c13-5711-42e0-9d9b-efb9e74424ab-1-1.dem" -output="C:\Users\aria\AppData\Local\Temp\cs-demo-manager" -format="csdm" -source="faceit"
Sun, 03 Dec 2023 10:39:01 GMT | LOG | renderer | WS:: message with name "analysis-status-changed" received from server
Sun, 03 Dec 2023 10:39:05 GMT | LOG | server | {Player team swap game-event occurred but player is nil 2}
{Player team swap game-event occurred but player is nil 2}

Sun, 03 Dec 2023 10:39:05 GMT | LOG | renderer | WS:: message with name "analysis-status-changed" received from server
Sun, 03 Dec 2023 10:39:05 GMT | LOG | server | demo analyzer exited with code 0
Sun, 03 Dec 2023 10:39:05 GMT | LOG | renderer | WS:: message with name "analysis-status-changed" received from server
Sun, 03 Dec 2023 10:39:05 GMT | LOG | renderer | WS:: message with name "analysis-status-changed" received from server
Sun, 03 Dec 2023 10:39:07 GMT | LOG | renderer | WS:: message with name "analysis-status-changed" received from server
Sun, 03 Dec 2023 10:39:07 GMT | LOG | renderer | WS:: message with name "match-inserted" received from server
Sun, 03 Dec 2023 10:39:09 GMT | LOG | server | WS:: message with name fetch-match-by-checksum and uuid 881b28ac-9dfb-4c4c-a972-b23c65e97a6e received from renderer process
Sun, 03 Dec 2023 10:39:10 GMT | LOG | server | WS:: message with name is-cs-running and uuid 67a84c91-0656-42dd-bf7d-e4b70962d8c2 received from renderer process
Sun, 03 Dec 2023 10:39:10 GMT | LOG | server | WS:: message with name watch-demo and uuid a3d67a7d-be07-49c1-9a9e-509be5c4260c received from renderer process
Sun, 03 Dec 2023 10:39:34 GMT | LOG | server | WS:: renderer process socket disconnected 1001 { type: 'Buffer', data: [] }
Sun, 03 Dec 2023 10:39:36 GMT | LOG | server | WS:: renderer process socket connected
Sun, 03 Dec 2023 10:39:36 GMT | LOG | server | WS:: message with name connect-database and uuid 8b2d3af0-c5b2-4b65-8b53-bbf821513704 received from renderer process
Sun, 03 Dec 2023 10:39:36 GMT | LOG | renderer | WS:: connecting to server
Sun, 03 Dec 2023 10:39:36 GMT | LOG | renderer | WS:: connected
Sun, 03 Dec 2023 10:39:36 GMT | LOG | server | WS:: message with name init-application and uuid 09001e50-8dd0-47c2-ade7-f0cae2cba24d received from renderer process
Sun, 03 Dec 2023 10:39:36 GMT | LOG | server | WS:: message with name fetch-matches-table and uuid 21c6fa91-5b9c-43e7-ae8b-22d7661d7336 received from renderer process
Sun, 03 Dec 2023 10:39:38 GMT | LOG | server | WS:: message with name fetch-match-by-checksum and uuid 87c9fe05-40c4-4c5b-90f3-a81b8b24721d received from renderer process
Sun, 03 Dec 2023 10:39:38 GMT | LOG | server | WS:: message with name is-cs-running and uuid 7cc0bf58-a819-4acc-8a25-7173c1f91643 received from renderer process
Sun, 03 Dec 2023 10:39:39 GMT | LOG | server | WS:: message with name watch-demo and uuid 6e33e810-0e66-4d96-96bf-a14f663be241 received from renderer process

Environment information
Version: 3.0.1
OS: win32 x64 10.0.22621
Electron: 27.1.3
Chrome: 118.0.5993.159
Last database migrations:
v1 - 2023-12-03T18:27:11.739Z
Additional context
No response

CS is installed on my F: drive

It's probably related to this - I just tried with CS2 installed on another drive without problems.
Could you send the content of the file C:\Program Files (x86)\Steam\steamapps\libraryfolders.vdf please?

"libraryfolders"
{
	"0"
	{
		"path"		"C:\\Program Files (x86)\\Steam"
		"label"		""
		"contentid"		"1139811302078677694"
		"totalsize"		"0"
		"update_clean_bytes_tally"		"51222270077"
		"time_last_update_corruption"		"0"
		"apps"
		{
			"228980"		"447984047"
			"286160"		"2969565005"
			"374320"		"21819457695"
			"570940"		"7366427395"
			"975370"		"273900934"
			"1245620"		"51659755776"
			"1384160"		"26118323374"
		}
	}
	"1"
	{
		"path"		"G:\\SteamLibrary"
		"label"		""
		"contentid"		"2523325894995635671"
		"totalsize"		"4000768323584"
		"update_clean_bytes_tally"		"124966009706"
		"time_last_update_corruption"		"0"
		"apps"
		{
			"1224290"		"1930636391"
		}
	}
	"2"
	{
		"path"		"D:\\SteamLibrary"
		"label"		""
		"contentid"		"4323154027193209704"
		"totalsize"		"120031539200"
		"update_clean_bytes_tally"		"39030788768"
		"time_last_update_corruption"		"0"
		"apps"
		{
			"607260"		"52973609"
			"980610"		"221614581"
			"1172470"		"74278342237"
		}
	}
	"3"
	{
		"path"		"F:\\SteamLibrary"
		"label"		""
		"contentid"		"851154846393983372"
		"totalsize"		"1000202039296"
		"update_clean_bytes_tally"		"18701864070"
		"time_last_update_corruption"		"0"
		"apps"
		{
			"105600"		"687925078"
			"294100"		"495497372"
			"438100"		"1120729460"
			"505040"		"108880236"
			"527440"		"9127456"
			"632360"		"2254733415"
			"824270"		"6589785072"
		}
	}
	"4"
	{
		"path"		"H:\\SteamLibrary"
		"label"		""
		"contentid"		"2867030957398390537"
		"totalsize"		"1000186310656"
		"update_clean_bytes_tally"		"200696342775"
		"time_last_update_corruption"		"0"
		"apps"
		{
			"240"		"4628887753"
			"730"		"69675769102"
			"221100"		"20662774205"
			"223850"		"4607824420"
			"231350"		"3347244"
			"238960"		"38148833502"
			"251570"		"17048154297"
			"252490"		"24755588908"
			"427520"		"2002798904"
			"594650"		"53098436819"
			"671860"		"3108610724"
			"714010"		"14625972077"
			"739630"		"23664700229"
			"1063730"		"93419136949"
			"1549660"		"2660195336"
			"1604030"		"8872096186"
			"1789480"		"8897417136"
			"1818450"		"20125414477"
			"1966720"		"885156037"
		}
	}
}

Thanks, few questions:

What is the exact location of CS2? You can find it from Steam -> Right-click on CS2 -> Manage -> Browse local file
Could you check if any folder steamapps\\common\\Counter-Strike Global Offensive is in one of your drives (even C:)?
For example, G:\\SteamLibrary\\steamapps\\common\\Counter-Strike Global Offensive. It could be a conflict.

Having 2 steamapps\common\Counter-Strike Global Offensive folders was the issue.
About the game crash, I think it's a CS2 issue - sometimes the game crashes when loading a demo or during playback.
Please read this doc to check if it's a CS2 issue.

 the demo loads fine manually

nvm, seems like i mixed up two demos. the one that was crashing is a cs2 issue, not a cs-demo-manager issue. will open a new issue if i see it happening again and can verify it's not cs2


Recommendation

Hi, thanks so much for your helper tool.
BTW, I think maybe a new function called auto-buy-low could be added. If you are interested in letting me join you, please contact me!

That’s essentially the same as the autobuyer proof of concept (still viewable in a separate branch).
Autobuying is not something I want to provide with these addons because the general community frowns upon it.
If you want to help in any way, feel free to browse the issue list and do a Pull Request. Any help is appreciated!
In case you meant something different, feel free to reopen the issue and clarify.


DatabaseBackup Performance Issue large database

Description of the issue
The call to sys.dm_db_file_space_usage can be expensive for large databases.  It looks like this is only needed when @ChangeBackupType='Y'. This statement could be moved down so that it executes within the IF @ChangeBackupType='Y' block below.  :
IF DATABASEPROPERTYEX(@CurrentDatabaseName,'Status') = 'ONLINE' AND ((@Version >= 12.06024 AND @Version < 13) OR @Version >= 13.05026) AND (@CurrentAvailabilityGroupRole = 'PRIMARY' OR @CurrentAvailabilityGroupRole IS NULL)
BEGIN
  SET @CurrentCommand07 = 'SELECT @ParamAllocatedExtentPageCount = SUM(allocated_extent_page_count), @ParamModifiedExtentPageCount = SUM(modified_extent_page_count) FROM ' + QUOTENAME(@CurrentDatabaseName) + '.sys.dm_db_file_space_usage'

  EXECUTE sp_executesql @statement = @CurrentCommand07, @params = N'@ParamAllocatedExtentPageCount bigint OUTPUT, @ParamModifiedExtentPageCount bigint OUTPUT', @ParamAllocatedExtentPageCount = @CurrentAllocatedExtentPageCount OUTPUT, @ParamModifiedExtentPageCount = @CurrentModifiedExtentPageCount OUTPUT
END

SQL Server version and edition
Microsoft SQL Server 2016 (SP2-CU4) (KB4464106) - 13.0.5233.0 (X64)   Nov  3 2018 00:01:54   Copyright (c) Microsoft Corporation  Enterprise Edition: Core-based Licensing (64-bit) on Windows Server 2012 Standard 6.2  (Build 9200: )
Version of the script
2019-04-28 16:40:00
What command are you executing?
dbo.DatabaseBackup
What output are you getting?
Should not be waiting minutes for this command to run when it is not needed unless @ChangeBackupType='Y'

How large is the database in your case?

I was looking at the code. It is used for smart differential backup.
That is if you specify @ChangeBackupType = 'Y' and @ModificationLevel.

The db is around 150TB. I was able to fix it by moving the code block to execute within the If @ChangBackupType='Y' block below.  I tried to do this as a pull request but hit a problem with permissions so either I can't do that or I'm not doing it right.

That's great - Thank you!  You've done a great job with this - thank you for the time you have put into it and for sharing it with the community.


Migrate from JCenter

Hi
Could you put this package on a different gradle repository?  JCenter's being shut down :-(
Ta muchly :-)

bump

bump

Try my library that is still maintained and got jetpack compose support:
https://github.com/warting/android-signaturepad

JCenter has been shutdown today :(


Add PHP 7.3 to travis

The release of PHP 7.3 is near: https://wiki.php.net/todo/php73.

Sounds like "Winter is coming" to me :)
Thanks


Json rpc server config

This is an example PR illustrating how config for json_rpc server and other mod will be refactored.

@hmoniz @adlrocha Just a final update for this PR before closing. For the tests, I shifted the variables around within the test mode and settle down to something like the following:
        let config_str = formatdoc!(
            r#"
            [server]
            ...

            [subnets]
            ...
        "#
        );

        let config = Config::from_str(config_str.as_str()).unwrap();

        check_server_config(&config.server);
        check_subnets_config(&config.subnets);

Should strike the balance mentioned above.


Unable to add skills or traits

When I open the sheet and go to Abilities, wises, skills, and traits, there is only an add item under wises. I dont seem to be able to add skills or traits.
Thank you!

The skills and traits need to be dragged to the sheet from the compeniums

That got it. Thank you.


mockRNCNetInfo from @react-native-community/netinfo/jest/netinfo-mock.js

how can I mock react-native-community/netinfo in typescript file

Review the README there is a section to mock the library but it doesn't work for typescript, how can I mock it in Typescript file?
thanks

there is nothing special about netinfo for mocking that I know of, this is just a jest question I think.
I do this

mike@bistromath:~/work/Kullki/ksocialscore/packages/public-app/src/mocks (p2p-features) % cat @react-native-community/netinfo.ts

jest.mock('@react-native-community/netinfo', () => {
  const mockNetinfo = jest.requireActual('@react-native-community/netinfo');
  mockNetinfo.NetInfo.fetch = () =>
    Promise.resolve({
      type: 'cellular',
      isConnected: true,
      isInternetReachable: true,
      details: {
        isConnectionExpensive: true,
        cellularGeneration: '3g',
      },
    });
  mockNetinfo.NetInfo.addEventListener = () => () => {
    /* do nothing */
  };

  return mockNetinfo;
});

if that works well for you please post a PR to the docs to help others


license file

It seems like the license file belongs to a different package, judging by the name and description.. ;) maybe ubdate it?

Hmmm, I just copied it without giving it too much attention and forgot about it ;)
Thanks for the reminder


Fix graylog sis-updater stats not working

Is logging broken again? investigate

logging was okay, stats now fixed in grafana


Can't get Entire Video description

When using
youtube-dl.exe --write-info-json https://www.youtube.com/watch?v=xxxxxxxx
or
youtube-dl.exe --write-description https://www.youtube.com/watch?v=xxxxxxxx
I'm getting partial description, each time a part of description is cutted with "..." at 161 characters.
I can't get why ?! real bug or do i something wrong ? i'm the only one ?

The last two videos I downloaded (one on 7/13 and one on 7/16) are like that—description gets cut off at exactly 161 characters (same with JSON info). Descriptions from 7/11 and earlier seem to be fine. I didn't change the version of youtube-dl I was using between those dates, so maybe something changes server-side.

Duplicate of #25937

Note: The last (and, therefore, "current") version is 2<IP_ADDRESS>, which is over a month old.
Is there any indication when a new version will be out, incorporating this fix (the problem of the description being truncated)?
Given that we used to get new versions like every few days or so, this seems long overdue.

Replying because I got CC'd, but I'm not particularly knowledgable here:
Sergey - who has been handling releases since as long as I have been using youtube-dl - has been inactive for 10+ days now. I don't otherwise know what the dev/mod hierarchy is for this project.  But given the current pandemic, I can only hope that he and his loved ones are well and in good health.


[BUG] example project fails to generate code

Describe the bug
With a fresh checkout of this project (d27237607d3b865c29379ae2cb1cdba08e459f7b), the example fails to generate code.
To Reproduce
cd to the example subdirectory and run:
flutter pub get
flutter pub run build_runner build --delete-conflicting-outputs
Expected behavior
Classes to be generated for the swagger petstore example.
Library version used:
d27237607d3b865c29379ae2cb1cdba08e459f7b
Additional context
The code that's being generated has paths in the export
Output:
[SEVERE] swagger_dart_code_generator:swagger_dart_code_generator on input_folder/swagger_dart_code_generator_temp.dart (cached):

Could not format because the source could not be parsed:

line 2, column 61 of .: Expected to find ';'.
  ╷
2 │ export 'input_folder\pet_service_swagger.swagger.dart' show InputFolder\petServiceSwagger;

@vipw  woah it's nice catch. could you share your swagger file name?

It's using the default swagger files from the example.
input_urls:
             - "https://raw.githubusercontent.com/epam-cross-platform-lab/swagger-dart-code-generator/master/example/input_folder/pet_service_json.json"
             - https://raw.githubusercontent.com/epam-cross-platform-lab/swagger-dart-code-generator/master/example/input_folder/pet_service_yaml.yaml

I think the files are downloaded to input_folder which the generator then uses.

@vipw  Looks like problem in slashes. I am working on macOS machine. Will check it on my Windows laptop

Experiencing issues with slashes on windows, everything above version 2.3.13 is not working.

@ByronHolt  yes yes, the main priority for us -  to fix Windows for now

Hi @ByronHolt I've fixed your issue, but lost phone and cannot sign in with two factor auth at weekend. Today evening I will push changes and close this issue. Thanks for waiting.


[BUG] Dropdown widget: Issues with Required JS

Steps to replicate:
1.Hit https://app.appsmith.com/
2. Under an organization, click on 'Create New' in order to create a new application
3. Edit Application
4. From the Form Widgets, pick Dropdown widget
Scenario 1 : In Selection Type, choose 'Single Select',  Required JS : disabled
Issue : Once a value is selected, user doesn't have an option to not have any value in the widget. (Even though Required JS is disabled)
Scenario 2: In Selection Type, choose 'Multi Select',  Required JS : Enabled
Issue: User can keep no value in the widget. (Even though Required JS is enabled)
Attachments : https://drive.google.com/drive/folders/1aXYRXU8pvhw09yPqr-eq-ZvFyer3WN8j?usp=sharing

@Saket2 if no default value is specified, it will have no option selected.
Also required option is only for disabling the form button when the dropdown is in the form


Update glue_transform.py

Fix issue when crawler name is not specified by assigning name attribute
Feature or Bugfix

Bugfix

Detail

When Glue crawler name is not specified, the StepFunction will attempt to use the crawler_name, but it is never set so the StepFunction is using a null crawler_name, causing a CloudFormation failure. The fix assigns a crawler_name attribute when crawler name is not specified, fixing the StepFunction related issue

Relates

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

Closing to resolve in #167


NPM-ification. Rangy 1.3

Hi everybody,
couple of things come with this pull request and I'm aware of the fact that certain things are very much opinionated. Just take a look at it and cherry-pick whatever you consider fitting your masterplan.
NPM-ification
When I installed the upfront-editable package from npm, I noticed that there are quite a few things missing that would allow seamless installation and use of editable.js in an npm/browserify/webpack environment. In order to have better support for these use cases, I added/changed the following:

Added jQuery and rangy (1.3, more on that later) to the dependencies.
Added require-statements to all source files that are depending on either rangy or jQuery or both. I think with this we finally got rid of all the globals.
Added build scripts to npm, that use browserify-shim to inject rangy and jQuery globals in order to not break the current behavior of the lib but still make it possible for npm users to install and compile seamlessly. I really tried to make this work with grunt but gave up after an hour or so. I wasn't the first one as I found out later.
Added core.js as entry file and now you can:

var Editable = require('upfront-editable');

Rangy 1.3
When I first installed rangy with npm, I didn't realize that your repo still works with 1.2.3.
The only thing that was broken was the range-save-restore.spec.js because rangy handles the conversion of a range to HTML differently than it used to. All the other specs passed. If you consider your test coverage adequate, you should be fine with 1.3.
That should be it. What do you think on that?

:heart_eyes: thx. will check it out.

I'm using the alias option for the shim in grunt in the livingdocs-framework:
browserify:
      options:
        browserifyOptions:
          extensions: ['.coffee']
        transform: ['coffeeify']
        alias: ['./src/modules/jquery_browserify.coffee:jquery']

and the shim in ./src/modules/jquery_browserify.coffee
module.exports = $

Regarding rangy 1.3 I just didn't had the time to give it a thorough testing so far. There are quite a few cases that are not tested in the unit tests because controling cursor, selection and focus programatically is often not the same as if a user had done the respective actions in the browser. So manual testing is necessary.
But I do not expect many problems if at all.

Shims are in line with your way of things now.
When I'm running karma:build I get a couple of errors though, all because of rangy's getNativeRange returning undefined. I haven't seen this while clicking through the example et al.

The errors I had were due to a bug in the alias section of the browserify config in Gruntfile.js.
Updated and tested all dependencies as well.

@peyerluk btw aliases in browserify are deprecated.

What is the status on this one? Would love to be able to install/require this via npm :)

It's not tested yet and some dependencies should get updated. But that should be about it.


Github Service Failing on Pull

Certain github pull data is causing a fatal error, here is the incoming data of one that fails:

git+https://github.com/jisaac89/recoil.git url
[nodezoo_github - 5297]: { _id: 'raganodejs',
  _rev: '1-c66ea346282e36b98b2fb84b5f50d5a6',
  name: 'raganodejs',
  description: 'Raga is learning NodeJS.',
  'dist-tags': { latest: '0.0.1' },
  versions:
   { '0.0.1':
      { name: 'raganodejs',
        version: '0.0.1',
        description: 'Raga is learning NodeJS.',
        main: 'main.js',
        scripts: [Object],
        author: [Object],
        license: 'MIT',
        _id<EMAIL_ADDRESS>        _shasum: 'c1ffd6e45b613f03bab22e6fe8c021128915a0f5',
        _from: '.',
        _npmVersion: '2.15.1',
        _nodeVersion: '4.4.3',
        _npmUser: [Object],
        dist: [Object],
        maintainers: [Object],
        _npmOperationalInternal: [Object],
        directories: {} } },
  readme: 'ERROR: No README data found!',
  maintainers:
   [ { name: 'ragara01',
       email<EMAIL_ADDRESS>} ],
  time:
   { modified: '2016-04-19T21:05:20.742Z',
     created: '2016-04-19T21:05:20.742Z',
     '0.0.1': '2016-04-19T21:05:20.742Z' },
  author: { name: 'Ragavendran Ramadoss' },
  license: 'MIT',
  readmeFilename: '',
  _attachments: {} }

and the error (null ref)

[nodezoo_github - 5297]: /Users/gege/nearform/nodezoo-github/lib/github.js:58
        user: matches[1] || null,
                     ^

TypeError: Cannot read property '1' of null

This is an assignment error, we need to clean this code up so there is a check to see if matches is not null or an array.

Good test case here as well

@Georgette This is a problem with a url not always being in npm, it is coded to repository.url and if it isn't in there then it cant extract the user and repo from it, mine doesn't crash though it just says it doesn't exist on npm, which clearly isn't true


Port to core2.1

Updated PL #6 by @Gimly:

Upgraded to .NetCore 2.1
Updated HttpClient, yes HttpFactory pattern
Added serviceCollection for easier installation on .net core projects.
Added Makefile to build projects
Updated dependencies

Hi @cristipufu, just wondering if you have a target date to publish to NuGet. Thanks

Hello @cristipufu, sorry to ask again. Do you still plan to publish to NuGet? Thanks


Bump ZHA modules versions.

Bump bellows-homeassistant version.
Bump zigpy-homeassistant version.
Bump zigpy-xbee-homeassistant version.
Description:
Install new zigpy modules
Checklist:

[X] The code change is tested and works locally.
[X] Local tests pass with tox. Your PR cannot be merged unless tests pass
[X] There is no commented out code in this PR.

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly (example).
[x] New dependencies have been added to requirements in the manifest (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

Good to merge when build passes


Aug31 Testnet Build Failure

https://bitsharestalk.org/index.php/topic,17962.msg233179.html#msg233179

See my patch #285

Sorry about that, I goofed -- didn't push fc.  It should work now.


[TableHeader]Chaning the font and color of table header row text

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

I want the color and font of the text in the header row of the table to be editable in material-ui 0.19.4.
Current Behavior

I found the headerStyle property to change the default style but it doesn't seem to work.Can anyone suggest the properties in headerStyle that has to be overridden to make this requirement work. Sample codes would help..my sample code: https://stackblitz.com/edit/react-emfcf1?file=TableComponent.js
Steps to Reproduce (for bugs)

Context

Your Environment

Tech
Version

Material-UI
0.19.4

React
15.6.1

browser

etc

We're not adding new features to 0.19, If you have a question about how to implement something, please use gitter or StackOverflow.


Simple Implementation of NGram Tokenizer

It does not yet support edges
It could probably be better in many "rusty" ways
But the test is passing, so I'll call this a good stopping point for
the day.

[x] support edgegrams
[x] how should a user configure the min/max gram stuffs
[x] should it be in the default token manager (I think no)
[ ] is this following tantivy best practices
[x] how should we support white space - Lucene doesn't address it. It uses other filters

Coverage increased (+0.02%) to 91.091% when pulling 288dffb7148fec3e59cb5a5c9f2a9f69e18bd9d5 on drusellers:ngram into 5637657c2f076d7fc74adc317d9e093b2ac3fcfc on tantivy-search:master.

@fulmicoton i'm ready for any feedback you might have on this PR

@drusellers  Cool.  Will have a look at that this weekend then.

I need more information here. Can you explain the use case? Especially do we want a Tokenizer or a TokenFilter ?

Thank you! I'll square this away this weekend!

I need more information here. Can you explain the use case? Especially do we want a Tokenizer or a TokenFilter ?

Looks like Lucene has both. I just started here, but can add the Filter too.
I originally desired this functionality so I could implement EdgeGramTokens so I could do type ahead search for my project. Also, it looked like a relatively easy Tokenizer to port from Lucene for Tantivy.

Good job. I can merge it as soon as you rename the edges_only and location stuff.

I think we are good to go. Ideally location would have been renamed but this is private anway.
I added you as a member of tantivy. In the future you can push your work as feature branch in tantivy-search/tantivy.

Oh, I thought I did. :( re: location
I'll square that away.
And thank you. 🎉


OSRSLog/ScapeCloud — Fix Item Container metadata information

Closing this PR as it is inactive. (If you wish to reopen this PR, please resolve the merge conflicts it has)


Platform miwifi does not generate unique IDs

Logger: homeassistant.components.device_tracker
Source: helpers/entity_platform.py:566
Integration: Rastreador de dispositivo (documentation, issues)
First occurred: 21:20:42 (1 occurrences)
Last logged: 21:20:42
Platform miwifi does not generate unique IDs. ID device_tracker.tira_rgb_1 is already used by device_tracker.tira_rgb_1 - ignoring device_tracker.tira_rgb_1
How do I solve this issue?

Rename one of the devices on the router. in version 2.0 names will be generated based on mac addresses

I did this but the problem continues. The name based on mac addresses I think is the best option and resolve the problem. Thanks.

I have the same messages - it appears when you connect same laptop sometimes to wired port of router, and sometimes via wifi.
I can propose to include not only mac, but also 'LAN_' or 'WiFi_' into device name. Yes, one physical device will have 2 names in HA and you have to double the scripts and configuration for it, but at least one of them will not be ignored by HA.

I have the same messages - it appears when you connect same laptop sometimes to wired port of router, and sometimes via wifi. I can propose to include not only mac, but also 'LAN_' or 'WiFi_' into device name. Yes, one physical device will have 2 names in HA and you have to double the scripts and configuration for it, but at least one of them will not be ignored by HA.

I don't see this as a problem, wifi and lan are different devices, and each adapter has its own mac. Therefore, it will be unnecessary to make a prefix.

Fixed in release: https://github.com/dmamontov/hass-miwifi/releases/tag/v2.0.0b


export Activity Map context data variables through data feeds

Issue in ./help/analyze/activity-map/activitymap-faq.md
Hello. In this doc it says that activity map context variables cannot be exported in data feed, but in the Data column reference it shows the clickmap columns are available: https://experienceleague.adobe.com/docs/analytics/export/analytics-data-feed/data-feed-contents/datafeeds-reference.html?lang=en

Good find! I've pushed a fix for the Activity Map FAQ. It will be live shortly.


Connecting to web application

Hi, i have been playing around the PerchIOS app and it works fine on the mobile platform.
Now i have built a rather simple Peerjs web application and i wish to use the IOS app to connect to the web application. However, there is an error on connecting.
On the web application, i need to create a new user, for example, MrX, and joins the room. When i  press connect on the mobile app, the connecting displaying message is (Connecting to Peers) rather than "connecting to web application user name"... Also, the mobile user's name becomes visible on the "call-peer" list, for example, MrY, on the web application, but when i press the connect button on the web application, it says connection failure.
What i am wondering is that, is there something wrong with the default peer choice in the Mobile application in this case? Because on the mobile itself, we don't have the option of selecting which peer to connect to, but rather, we connect to the default peer we see in the room?
How do i fix that?
Btw, the Peerjs Demo is from the XirSys SDK
https://github.com/xirdev/xsdk
Thanks very much
Regards

Alright, i think the problem doesn't lie in the peer selection part, but rather the mobile to web browser connectivity part.

@dreamfm I haven't tried the Peerjs demo from XirSys but I don't think the application level signaling used is the same as PerchRTC. Both clients need to send the same types of messages, with an agreed upon payload.

@ceaglest
Hi, Thanks for the prompt reply.. Can you point me to a direction on how i may solve this? Or is it going to be too complicated?
Thanks

@dreamfm In this example the message keys are defined in: https://github.com/perchco/perchrtc/blob/master/PerchRTC/XirSys/XSMessage.h
Despite the fact that both sample apps use the XirSys signaling server the clients still need to speak in terms of the same message types.


Storage stress tests explode after hitting fluid:telemetry:SummaryManager:SummariesDisabled

I can't figure out quickly why this event is generated in the middle of the stress run.
But once this event fires (and it reliably fires in the run), the run explodes as we immediately hit 10K ops limit.
Would appreciate some help here, as that might block ODSP scalability tests (on the path to GA).

You would need https://github.com/microsoft/FluidFramework/pull/6172 to get through.
And possibly https://github.com/microsoft/FluidFramework/pull/6168

Fixed by above mentioned PR


coordinates of a skewed image

Hi,I have given a skewed image as input to Tesseract-ocr.
The plain text output generated de-skews the image and is giving the correct output.
But in hocr output, the bounding box coordinates of the words are with respect to the original skewed image.
So,Is there any way to get the coordinates of the words of image which is deskewed by the tesseract or can we get hold of deskewed image itself?
Thank you.

Please use tesseract user forum for asking questions.


Capture low volume metrics in telemetry

Telemetry already exposes min, max and avg for some of the metrics. The avg field can be used to calculate the volume on the fly but for certain metrics (eg query count), there could be just one or two queries in an hour and when calculating avg it gets rounded down to 0 for the whole hour. This can either be fixed by using floats (at a given precision) for avg or by sending the totals calculated for the hour as a separate field (this option is preferred).

These ones are a little weird because they're not gauges like memory usage or CPU. They're meant to represent rates (i.e. throughput).
These are a little strange because what we're trying to capture is the rate information in 1m intervals. However, we send a report every hour and we don't want to send all 60 recorded intervals in. The tricky bit is that the hour isn't exact, so a total isn't useful unless you have a start time and end time for the total reporting interval (so you can calculate a rate), or you have the last report with some total and time that you can compare to this report with a total and its own time.
So what would be useful here? We'd like to know if there are big spikes or dips in throughput over the course of the reporting interval (1 hour) and we'd like to know the average throughput in seconds over the course of the hour.
Since many cases have fewer than 1 query (or write) per second, if we express the rate at that level, we'd want to capture it as a float.
Closing this out, maybe we want 3 rates expressed for each hour long reporting interval:

min_count_minute (the count of queries/writes of the minute over the reporting interval with the fewest)
max_count_minute (the count of queries/writes of the minute over the reporting interval with the most)
total (the total number over the reporting interval)
rate_seconds (the rate over the reporting interval, should be a float. If the interval is exactly 1 hour, the total / 3600 will be this number)

this makes sense to me @pauldix, maybe I'm misunderstanding but what you described as min_count_minute and max_count_minute - is that not what the _min_1m and _max_1m metrics are already capturing? this request came after I was exploring the data and noticed fields like QUERY_REQUESTS_AVG_1M were mostly 0's, because it's a rounded whole number of the per/minute values. So, if the user is running a few queries every few minutes, it could easily average out to zero

@mona-influx yes, the existing _min_1m and _max_1m capture the counts I was talking about.

@pauldix - I think you'll be able to compare the average throughput spike/dips as long as it's done at least 60 of them in a minute. If it's done anything below it'll come up as 0. If we need to compare low "volume" (<60 reads/writes per min) then we can add the fields as you mentioned.


Fix Celery/ServiceBus

Codecov Report

Merging #103 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #103      +/-   ##
==========================================
+ Coverage   91.25%   91.26%   +0.01%
==========================================
  Files          20       20
  Lines         823      824       +1
==========================================
+ Hits          751      752       +1
  Misses         72       72

Impacted Files
Coverage Δ

opwen_email_server/config.py
95% <100%> (+0.26%)
: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 eb35deb...044f338. Read the comment docs.


Edabit solution repository, Create more issue for problem task

Before submitting a PR please Check some details.

[ ] fileName formet should be in camelcase
example- returnTheNextNumber.py

[ ]  In file there should be problem statement and link to probelm
example -

/* this program will add two numbers
    * Problem : https://edabit.com/challenge/SFzHtm63XT6EYNHWY
*/
int main() {
    int bla;
    return 0;
}

This repository is vaild for HacktoberFest2020
Good Luck,

can i work on this?

ya sure @subramanian0 assigned, Take care of all suggestions before making PR

@subramanian0
Problem Task - You have to make issues in repo using issue template.
find issue on https:://edabit.com

I want to contribute

may i move ahead for this ?


Performance Questions

Performance for showing the completion menu is VERY slow (takes over 3 seconds to appear for a 150-line Python script even for primitive types like str). Confirmed that this happens on two different machines: gen 10 i7s with 16GB and 64GB of RAM respectively so hardware isn't the bottleneck.

Does anyone else see slowdowns to this degree?
Are there any settings I can disable to improve performance?
Is there a way to profile/trace to see where in the stack the slowdown is happening?

Versions
Fedora 35
Sublime v 4126
Python v3.10.4
LSP v1.16.2
LSP-pylsp v2.3.3
Configs
LSP-pylsp.sublime-settings
{
    "settings": {
        "pylsp.plugins.pyflakes.enabled": false,
        "pylsp.plugins.pylsp_mypy.enabled": false,
        "pylsp.plugins.mccabe.enabled": false,
        "pylsp.plugins.preload.enabled": false,

        "pylsp.plugins.pycodestyle.enabled": false,
        "pylsp.plugins.pycodestyle.ignore": [
            "E221",
            "E241",
            "E501",
            "W504",
        ],
    }
}

LSP.sublime-settings
{
    "document_highlight_style": "fill",
}

Finally, thanks for your work. This thing does almost everything I need it to, just unbearably slowly...

Try LSP-pyright instead.

Try LSP-pyright instead.

I would were it not for the NodeJS requirement. :)

Think I found the culprit. Looks like the rope plugin bogs things down considerably.
Was able to get the completions list appearance time from ~4s to <1s with this in LSP-pylsp.sublime-settings:
{
    "settings": {
        "pylsp.plugins.rope_completion.enabled": false,
    },
}

Should probably disable it by default then.
Some relevant issue from another LSP: https://github.com/emacs-lsp/lsp-mode/issues/923

Rope completions indeed seem to be disabled by default in pylsp: https://github.com/python-lsp/python-lsp-server/blob/4bffc39ca6923a007b084d13f67a8f2ea10613ad/pylsp/plugins/rope_completion.py#L16
Though their documentation is out of sync and says that it's enabled by default: https://github.com/python-lsp/python-lsp-server/blob/develop/CONFIGURATION.md

@dbazile would you like to make a PR to disable rope by default?

Will do as I'm updating the package already.

Released in https://github.com/sublimelsp/LSP-pylsp/releases/tag/2.4.0


Error mismatch between get_version and publicKey

Bug steps:

Connect ledger and wait until the screen shows the message vires in numeris...
Call get_version() => error_message = No errors
Call publicKey(HDPATH) => error_message = Unknown error code

@fedekunze Do we still need changes here after our chat?

No, this can be handled client side. I created an issue on voyager for it https://github.com/cosmos/voyager/issues/2015


BuddyStories component

No carousel yet
No mobile view yet
Graphics from the original site (including favicon.ico)

Thanks a ton @smabuk!  I've been busy on other things so getting that front page closer to done is huge!


Template for MwS missing?

I'm missing a template for MwS or have I overlooked it?

It's in the develop branch, not yet fully tested. Will be merged into master once it's ready


Not importing Angular Router

In the "Navigation and Routing Basics: Placing the Views" section of the course we implement routerLink and router-outlet in our app.component.ts, but we aren't importing any routing services from Angular, so the page throws errors and doesn't display.

Non-issue. I must have been asleep during the Routing Configuration video. Those services, as well as the routing path definitions, are contained in that video.

Glad you resolved it!


datagrid with pagination disappears  within tab-content

Describe the bug
I'm using clr-tabs, clr-tab, clr-datagrid, clr-dg-pagination to create a datagird inside a clr-tab. When the clr-datagrid is not the active tab on load, on activation doesn't not render.
However if it's on first tab it works fine
How to reproduce
Not working Tab2 datagrid Not working datagrid. Just press Tab2->Tab1->Tab2 again
Working Tab2 datagrid as active tab Working Datagrid. Just press Tab1->Tab2 again (You might need to refresh the page)
Expected behavior
On Tab change datagrid and pagination should work as expected
Versions
Clarity project:

[ 5.4.0] Clarity Core
[ 5.4.0] Clarity Angular/UI

Clarity version:

[ ] v3.x
[ ] v4.x
[ X] v5.x

Framework:

[X ] Angular
[ ] React
[ ] Vue
[ ] Other:

Framework version:
ie: Angular 12
Device:

Type: [HP i7]
OS: [Windows]
Browser [chrome]
Version [ 91.0.4472.164 ]

This can be accomplished by using the de-sugared syntax of the *clrIfActive structural directive.
  <clr-tab>
    <button type="button" clrTabLink>Tab2</button>
    <ng-template [(clrIfActive)]="showDatagrid">
      <clr-tab-content>
        Content Tab2
        <clr-datagrid>
          <clr-dg-column>User ID</clr-dg-column>
          <clr-dg-column>First Name</clr-dg-column>
          <clr-dg-column>Last Name</clr-dg-column>
          <clr-dg-column>Email</clr-dg-column>
          <clr-dg-row *clrDgItems="let user of users">
            <clr-dg-cell>{{user.id}}</clr-dg-cell>
            <clr-dg-cell>{{user.first_name}}</clr-dg-cell>
            <clr-dg-cell>{{user.last_name}}</clr-dg-cell>
            <clr-dg-cell>{{user.email}}</clr-dg-cell>
          </clr-dg-row>
          <clr-dg-footer>
            {{users.length}} users
            <clr-dg-pagination></clr-dg-pagination>
          </clr-dg-footer>
        </clr-datagrid>
      </clr-tab-content>
    </ng-template>
  </clr-tab>

Here is an example.
I am going to convert this into a discussion, it isn't an issue with Clarity and it may be helpful to others with the same question.


Rule: Disallow HTML comments

I can't think of a good use case for using <!-- HTML comments --> in templates since {{!-- handlebars comments --}} serve the purpose of commenting the template, but are not included in the compiled output and eventual DOM. But I'm so used to HTML comments that I always forget and use them when I don't mean to! I'd love a lint rule to catch this.
I see that the linter uses HTML comments for its instructions. Is this because brocolli-template-lint doesn't expose handlebars comments or something? Or could it instead support instructions in handlebars comments? I believe this rule could be made to work either way (obviously ignoring HTML comments that are instructions to the linter).
I'd be happy to take a crack at a PR for this if it sounds like a good idea outside of my head as well as inside of it :smile:

We use HTML comments for inline enable/disable of rules (we then strip them from the emitted HTML). A rule forbidding all other HTML comments would be welcome...


[Update] Remove custom sheet extension

Description
This PR removes the custom sheet extension from the Sample Viewer as part of removing iOS 15 support. Uses of the sheet were replaced with a popover, which displays better on iPad and Mac Catalyst
A couple things to note:

One case, the Create symbol styles from web styles sample, required non-tirval updates since the list would not update correctly after the sheet was replaced with a popup.
On Slack, we decided to keep the "Done" buttons on the popovers. Adding buttons to sheets missing them and updating the screenshots will happen in a follow-up PR.

Linked Issue(s)

swift/issues/5145

How To Test
The following code allows you to easily test all of samples modified by this PR. To use it, replaced the favoriteSamples property in FavoritesView.swift with the code. All the samples affected by this PR will show up under the "Favorites" category without the view structure changing at all.
Please test on iPhone, iPad, and Mac Catalyst.
Testing Code
private var favoriteSamples: [Sample] {
    let sampleViews = [
        AddDynamicEntityLayerView(),
        AddFeaturesWithContingentValuesView(),
        Animate3DGraphicView(),
        ChangeMapViewBackgroundView(),
        ConfigureClustersView(),
        CreateMobileGeodatabaseView(),
        CreateSymbolStylesFromWebStylesView(),
        DensifyAndGeneralizeGeometryView(),
        DisplayContentOfUtilityNetworkContainerView(),
        DownloadPreplannedMapAreaView(),
        FindRouteView(),
        FindRouteAroundBarriersView(),
        GroupLayersTogetherView(),
        ManageBookmarksView(),
        ManageOperationalLayersView(),
        OrbitCameraAroundObjectView(),
        SetVisibilityOfSubtypeSublayerView(),
        ShowViewshedFromPointInSceneView(),
        SnapGeometryEditsView(),
        StyleSymbolsFromMobileStyleFileView(),
        ValidateUtilityNetworkTopologyView()
    ] as [Any]

    return sampleViews.compactMap { view in
        let viewName = String(describing: view).components(separatedBy: "(").first
        return SamplesApp.samples.first { sample in
            sample.nameInUpperCamelCase.appending("View") == viewName
        }
    }
}

To Discuss

Before implementing this, Ting and I decided having a dedicated frame for Mac Catalyst unnecessarily clutters the code. As a result, the frames are sized for iPad as follows. Let me know what you think about this approach:

Width: 320
Height: Sized to the content on iPad, with a max of 380.

Similarly to this feedback, I removed all the conveniences I didn't think brought value (mostly properties for buttons). Let me know if you think I should be more aggressive and remove more.

Looks good but we are reusing these two view modifiers a lot (and the same parameters):
.presentationDetents([.fraction(0.5)])
.frame(idealWidth: 320, idealHeight: 380)

So should we create a custom view modifier that we can reuse? Something like this (not sure if the naming is the best):
struct PopoverContentFormat: ViewModifier {
    func body(content: Content) -> some View {
        content
            .presentationDetents([.fraction(0.5)])
            .frame(idealWidth: 320, idealHeight: 380)
    }
}
extension View {
    func defaultPopoverContentSize() -> some View {
        modifier(PopoverContentFormat())
    }
}

So should we create a custom view modifier that we can reuse? Something like this (not sure if the naming is the best):

I'm not sure that would be best the way to go. Part of the motivation behind this PR is that we are trying to move away from individual samples having any dependencies that make it so they can't be easily copy and pasted into another project. See https://github.com/Esri/arcgis-maps-sdk-swift-samples/pull/372#discussion_r1567718805

Thank you for taking time to review this!


import_step: do not assign labels to Compound.for_construction

This is a proposed fix for #662.
After importing a STEP file, the for_construction attribute of Compounds contain an OCP.TDF.TDF_Label. This will make Shape.__deepcopy__ fail because the label is not pickleable.
I removed the assignment of TDF_Labels to for_construction in the STEP import, and pass the label as a function argument to build_assembly instead.
I tried importing a few STEP files and it appears to work. WDYT?

Thanks!  I had originally expected to do more with the TDF_Label stored temporarily in the for_construction attribute but as you've shown it isn't really required.


PMID:20838651

normal protein targeting to endosome
increased ubiquitin-specific protease activity
decreased ubiquitin-specific protease activity
elongated vegetative cell with central constriction

abnormal ubiquitin-specific protease activity FYPO:0005111
decreased ubiquitin-specific protease activity FYPO:0005112
increased ubiquitin-specific protease activity FYPO:0005113
normal protein localization to endosome FYPO:0005114
elongated vegetative cell with central constriction FYPO:0005115
Note: FYPO:0005114 has "targeting" in a related synonym. I did it that way because GO only has a "protein localization" term, not a "protein targeting" term, for endosome, and I don't want to go back to the worm-can of distinguishing targeting from other localization.

edit file: 02e53975e4138596f952e50fe8675346a0276112
release: bd7cb989ef38a30349babc1206d0fdb124313237


Remove previous git tag for dev releases

This fixes the issue that GitHub releases for dev were associated with a stale git tag.

I tested this in my fork, so merging before tests complete (which wouldn't do anything useful anyway, because the CI skips the job that contains all the changes here).


Use Chrome's network stack for installing extensions

VSCode Version: 1.8.0
OS Version: Ubuntu 16.04.1 LTS

With 1.7 extensions worked fine.  Now on 1.8 it is ignoring the environment variables.  Doing a packet capture you can see it going direct.  I tried adding the setting for proxy in user settings still not working. It also says that https_proxy is not a setting.  My setting for the proxy are with authentication.
Steps to Reproduce:

Environment Variable set for http_proxy and https_proxy.
Extension tries going direct instead of proxy.
Doing 'Check for Updates' goes through proxy.

Hi
It seems that the issue is related to https://github.com/electron/electron/pull/7577
So now when searching for extensions vscode uses chromium's network stack however installation of those extensions seems to use node's net stack.

I think that what's best is to always use chromium's net stack since it's more proxy-friendly

I will hijack this issue to move all network code to the Chrome's network stack.
Related to #17786

Removing important, as it's crucial we decide on the right approach, which at the moment is not clear: #18103

Done.

Great job @joaomoreno