Access control support

This PR adds the access control template cherry picked from @valeriano-manassero's chart.

Rebased & squashed commits, should be good to merge!


Division by zero when calculating AUC

Hello,
The following code results in app crash:
import numpy as np
import xgboost as xgb
y = np.random.randint(0,2,size=(100,1))
y[50:] = 0
dmat = xgb.DMatrix(np.random.rand(100,2), label=y)
dtrain = dmat.slice(np.arange(50))
dtest = dmat.slice(np.arange(50, 100))
xgb.train({'objective':'binary:logistic','eval_metric':'auc'}, dtrain, 100, [(dtrain,'train'),(dtest,'test')])
The reason is the division by zero at the line 134 of src/metric/rank_metric.cc
Thanks,
Sergey

Bug related to an old version, closing.


Review of glycosaminoglycan metabolic process (GO:0030203) and proteoglycan metabolic process (GO:0006029) branches

There are several issues with these 2 branches. Several terms have their definitions and Xrefs that need to be updated, other terms need to be made obsolete and some new terms are needed.
I will add the requests to this thread gradually as some will depend on the decision made.
Background
Glycosaminoglycans (GAG) are polysaccharides composed of repeating disaccharides with variable sulfation levels. They are classified according to the repeating disaccharide subunit.

chondroitin sulfate (CS) (glucuronic acid and N-acetylgalactosamine)
dermatan sulfate (DS) (iduronic acid or glucuronic acid and N-acetylgalactosamine)
heparan sulfate/heparin (HS) (iduronic acid or glucuronic acid and N-acetylglucosamine)
keratan sulfate (KS) (galactose and N-acetylglucosamine)
hyaluronan (HA) (glucuronic acid and N-acetylglucosamine)
peptidoglycan (N-acetylglucosamine and N-acetylmuramic acid) made by Bacteria

Apart from hyaluronan and bacterial peptidoglycan, GAG are covalently bound through a linkage region to Ser, Thr or Asp to core proteins to produce proteoglycans (PGs)
For chondroitin sulfate (CS), dermatan sulfate (DS), heparan sulfate/heparin (HS) and keratan sulfate (KS), the sugar units are added sequentially and directly to the protein.
The synthesis involved 2 mains steps:

the formation of the linear disaccharide repeats chain
the post-elongation modification, including epimerisation, sulfation
Hyaluronan is not sulfated.

CS, DS and HS starts with the synthesis of a common tetrasacchardide linker. Reactome, KEGG and MetaCyc has separate pathway for this.
heparan sulfate and heparin differs by their levels of sulfation. It's seems that heparin is made mainly by mammals (not sure).
For a review: Book chapter
Below is an example of the synthesis of chondroitin sulfate proteoglycan in Drosophila to give an idea.

Useful reviews:
PMID: 29495527 Glycosaminoglycans and Proteoglycans (2018)
Proteoglycans and Sulfated Glycosaminoglycans (2009)
PMID: 38500384 A biological guide to glycosaminoglycans: current perspectives and pending questions (2024)
Having searched extensively, I concur with @rozaru - all but one (hyaluronan) of the common GAGs are assembled on a protein core and exist as proteoglycans.
Thus, for chondroitin sulfate, dermatan sulfate, keratan sulfate, heparan sulfate and heparin, it doesn't make sense to have separate metabolic/biosynthetic/catabolic process BP terms for the GAG alone and for the corresponding proteoglycan.
I think part of the confusion here has been that the published papers/reviews and metabolic databases often refer to these process as 'GAG metabolism/synthesis/degradation' (rather than 'proteoglycan metabolism/synthesis/degradation') and thus don't make any reference to the protein to which the GAG is attached, even though the protein is present.
Further, I don't think there's a need to try to split the specific proteoglycan biosynthetic processes into sub-processes, such as 'polysaccharide chain biosynthetic process' and 'enzymatic modification'. External databases (MetaCyc, Reactome, KEGG) represent these processes in single pathways, and sometimes the enzymatic modification is a single-step process.


id < this->size():Error:Assert failed:in file unicharset.cpp, line 300

Tesseract 4.0alpha from github
Windows 10 - WSL
Reading symbols from dawg2wordlist...done.
(gdb) run
Starting program: /usr/local/bin/dawg2wordlist Devanagari.lstm-unicharset Devanagari.lstm-word-dawg Devanagari.words
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
Loading word list from Devanagari.lstm-word-dawg
Reading squished dawg
Word list loaded.
id < this->size():Error:Assert failed:in file unicharset.cpp, line 300

Program received signal SIGSEGV, Segmentation fault.
ERRCODE::error (this=this@entry=0x665558 <ASSERT_FAILED>, caller=caller@entry=0x4217f6 "id < this->size()",
    action=action@entry=ABORT, format=format@entry=0x420abf "in file %s, line %d") at errcode.cpp:86
86            if (!*p)
(gdb) backtrace
#0  ERRCODE::error (this=this@entry=0x665558 <ASSERT_FAILED>, caller=caller@entry=0x4217f6 "id < this->size()",
    action=action@entry=ABORT, format=format@entry=0x420abf "in file %s, line %d") at errcode.cpp:86
#1  0x000000000040b667 in UNICHARSET::id_to_unichar_ext (this=0x7ffffffde340, id=216) at unicharset.cpp:300
#2  0x0000000000405ac9 in WERD_CHOICE::string_and_lengths (this=this@entry=0x7ffffffdde50,
    word_str=word_str@entry=0x7ffffffddde8, word_lengths_str=word_lengths_str@entry=0x0) at ratngs.cpp:435
#3  0x0000000000402925 in tesseract::CallWithUTF8 (cb=0x687870, wc=0x7ffffffdde50) at dawg.cpp:113
#4  0x0000000000403abb in Run (a1=0x7ffffffdde50, this=0x687850) at ../ccutil/tesscallback.h:2145
#5  tesseract::Dawg::iterate_words_rec (this=this@entry=0x687930, word_so_far=..., to_explore=<optimized out>,
    cb=cb@entry=0x687850) at dawg.cpp:135
#6  0x0000000000403b73 in tesseract::Dawg::iterate_words_rec (this=this@entry=0x687930, word_so_far=...,
    to_explore=<optimized out>, cb=cb@entry=0x687850) at dawg.cpp:139
#7  0x0000000000403b73 in tesseract::Dawg::iterate_words_rec (this=this@entry=0x687930, word_so_far=...,
    to_explore=<optimized out>, cb=cb@entry=0x687850) at dawg.cpp:139
#8  0x0000000000403b73 in tesseract::Dawg::iterate_words_rec (this=this@entry=0x687930, word_so_far=...,
    to_explore=<optimized out>, cb=cb@entry=0x687850) at dawg.cpp:139
#9  0x0000000000403b73 in tesseract::Dawg::iterate_words_rec (this=this@entry=0x687930, word_so_far=...,
    to_explore=to_explore@entry=0, cb=cb@entry=0x687850) at dawg.cpp:139
#10 0x0000000000403dba in tesseract::Dawg::iterate_words (this=this@entry=0x687930, unicharset=..., cb=cb@entry=0x687870)
    at dawg.cpp:122
#11 0x0000000000402838 in WriteDawgAsWordlist (unicharset=..., dawg=dawg@entry=0x687930,
    outfile_name=outfile_name@entry=0x7ffffffde70a "Devanagari.words") at dawg2wordlist.cpp:69
#12 0x000000000040215a in main (argc=<optimized out>, argv=<optimized out>) at dawg2wordlist.cpp:94
(gdb)

Works fine with Latin, eng, san and hin traineddata files.

Regenerated the Devanagari.* files using 'combine_tessdata -u'. Works now.
Possibly the earlier unicharset was corrupted.


Include Cargo.lock in the repository

I agree this should be added, not entirely sure what happened to the first one :/


it will be nice to support musli encode decode option

in some case like version compat,  "musli encode decode" is better than "Serde bincode"

Can you provide a better justification? I am using bincode2, which is a reasonably mature library for creating a binary exportable representation of the Value object. Musli is a new and unstable crate. What should be the benefits?


Fixed bug where blacklisted players could join password protected groups

Prior to this commit, blacklisting players from password protected groups would have no effect as the JoinGroup command didn't check if said player was blacklisted or not.

Please base against the civ14 branch

Also you have to do the same in the GUI, not only the command

Thanks for contributing


Add YARA Windows Binaries

YARA is basically the only tool I've found myself needing to add to flare-vm.
Windows installation instructions here: https://yara.readthedocs.io/en/v3.6.0/gettingstarted.html#installing-on-windows
I can take a stab at this, success not guaranteed.

I'm working on a project similar to flare-vm, and in the process of building the appropriate choco package. Would love to collaborate on getting some of these packages fixed up and included.

@cryps1s that's great to hear. I started to look at the choco documentation on building a new package, but it isn't something I had experience with. If you finish that please link it to this issue - I'd appreciate it.

Can confirm that you can get the python-yara library pretty easily:

choco install vcbuildtools -y
choco install yara-python -source python -y

This will install:

DotNet 4.5.1
VC Build Tools 2015
Yara-Python Library

Haven't done testing in-depth, but it seems to function okay.


Allow chaining of connections

This is a nice to have, but could help with boilerplate in definitions.
I want to create a chain of items (LED's) that all have power and ground requirements.  Currently I have to define it like this:
power = new Power
led1 = new LED
led2 = new LED

led1.power ~ power
led2.power ~ power

It would be nice to use syntax like this instead
led1.power ~ led2.power ~ power

This'll be great when we get it in, and improve readability if we do it right!
Some of the reason we haven't yet is because we also want to support chaining serial connections and it needs to be super obvious whether you're doing a serial or parallel chain.
For series we've had two independent suggestions for: a ~> b ~> c (where b has an "in" and an "out" / is a dipole)
I quite like that, but it's very similar to the parallel chaining you've proposed.
For parallel what about something like a ~ (b, c) instead? That'd yield links a ~ b and a ~ c?

Sorry - hit the wrong button! Definitely an open issue!

@m5p3nc3r Possible in v0.3 with a fab module! I'll leave if open though, because I believe it should also be possible via ato


[$1000] Web – DOB – Arrow button shows weird behavior when used with future dates.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open your profile.
Click on "Settings".
Click on "Personal Details".
Click on "Date of Birth".
Select the year 2018 and click on the right arrow to proceed to the next month after selecting the 31st of the current month.
Notice that initially, it will jump to the next month. However, when you go back and select the date again, a "not allowed" sign appears over the arrow.

Expected Result:
If the selected date is in the future, the "Not Allowed" sign should remain visible in the previous month.
Actual Result:
The "Not Allowed" sign does not appear even if the selected date is in the future.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.32-5
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/db8afe18-6f21-4032-b1eb-2b80b7e3444f
https://github.com/Expensify/App/assets/43996225/050ca16f-acb3-4b5d-9539-c9fde82093cc
Expensify/Expensify Issue URL:
Issue reported by: @Usmantariq96
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1687253920451309
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01a703d7a35964ac71
Upwork Job ID:<PHONE_NUMBER>738194944
Last Price Increase: 2023-07-03

Looks like we already have one proposal up for review @0xmiroslav

Proposal
Please re-state the problem that we are trying to solve in this issue.
Web – DOB – Arrow button shows weird behavior when used with future dates.
What is the root cause of that problem?

The solution
Here, we should change moment(this.props.maxDate).endOf('month').startOf('day') to moment(this.props.maxDate).endOf('day') as here the only purpose of this hasAvailableDatesNextMonth fxn seems to be to check whether the adding 1 month to the current date won't exceed the current date.
Result after change
Testing on June 27, If I select May 27 I am able to move forward one month, but selecting May 28,29, etc wont allow me to click on the next month button.
https://github.com/Expensify/App/assets/47215240/f4798853-f7fe-4bfc-bf0a-087efa05a0e5
What alternative solutions did you explore? (Optional)
NA

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01e0f36fff0755c57b

@0xmiroslav could you review the proposals here?

@0xmiroslav @lschurr any updates?

@0xmiroslav If I understood the problem correctly, the 'next month' button should be disabled if clicking on it (i.e adding 1 month to the selected date) results in a 'future' date. If that is the case, then this is still an issue as seen in the below clip.
https://github.com/Expensify/App/assets/47215240/b2d556ca-c430-458f-a76a-b4d5a6fe01ab
I tested this on the main branch on July 4. If on the calendar, I select a date after June 4, the button is not disabled, and clicking on it results in the warning text "Date should be before 2018-07-04". But the expected result says that the button itself should be disabled in such a case. Please clarify if my understanding is correct.

@0xmiroslav - do you have thoughts on @CallMeAmartya's question above?

Hmm I feel like just paginating through the months doesn't/shouldn't actually select a date. So we shouldn't show any kind of error until the user actually selects a date inside of the month.

I'm not so sure I agree. I would bring this up in the open source room for larger discussion though. cc @mountiny I feel like we chatted about something similar recently?

I agree with Shawn, there was a discussion in Slack but I cant find the link right now unfortunately which concluded that we should only select the date once user click on the specific date. Thats how majority of other datepickers work.
I would actually put this on hold for the datepicker refactor so we dont duplicate work https://github.com/Expensify/App/issues/20630

Still on hold I believe.

Has this one been fixed @fedirjh?

@usmantariq96 could you apply to the job here? https://www.upwork.com/jobs/~01a703d7a35964ac71

Thanks! Offer sent @usmantariq96

Accepted with thanks

Paid and closing!


chore(merge): merge feat/carbon-for-ibm-dotcom-v2 into main

Description
Merge feat/carbon-for-ibm-dotcom-v2 feature branch into main.
Changelog
New

{{new thing}}

Changed

{{changed thing}}

Removed

{{removed thing}}

This PR has been squashed, causing all contribution history to be lost. 😕
(same with https://github.com/carbon-design-system/carbon-for-ibm-dotcom/pull/11217)
A year's work? has been reduced to a one commit (https://github.com/carbon-design-system/carbon-for-ibm-dotcom/commit/f7fd7e7d4af8ca78d5ce86658095ca39f6d28a33) in the name of @kennylam:

4,373 changed files with 106,075 additions and 421,213 deletions.

Not just for me, but for all the other contributors.
@kennylam @jeffchew can we get this back? Thank you! 🙏

Thanks for pointing out @abdonrd , I think also should be for all team members that touched the feature branch.
@kennylam , any way to easily merge this in as a merge commit? I know you spent a lot of time resolving the conflicts....

Fixed, thanks @kennylam!


Audience Clarification - Main Guidance and Guidelines for Optimising existing Protection Profiles

If the Guidelines for Optimising existing Protection Profiles becomes part of the Guidance for Cloud Evaluations Document, then the audience for the latter will need to be expanded.
Given that they serve two different audiences (Guidance for Cloud lists "evaluation sponsors and manufacturers, but it is also related to evaluators, certifiers and end users of this type of products."; Guidelines for Optimising is focused on iTCs/PP authors) they may need to stay separate documents.

Leave open for discussion.

The main guidance document has included all audiences without need to spin off additional docs. Closing per discussion.


Support for more column types

gorm doesn't support postgis geometry column types yet.
What's the best way to go about implementing support for geometry or geography column types?

I created a small module out of this, just in case someone else need it https://github.com/nferruzzi/gormGIS

@calvinchengx how did you implement it incorrectly? I am having the same issue

I found that this did not work correctly with MySQL, FYI, because in order for MySQL to parse a coordinate from a string it needs to be wrapped in the "GeomFromText" function.
Unfortunately, it seems like you have to drop down to raw SQL to create or update these rows with MySQL.

I will second the comment from @benguild something changed in MySQL versions is my suspicion with Point() data.  This solution doesn't work, I will probably drop down to storing as float columns so I can continue using GORM.

Hi, i'm implementing a GeoPoint field, @calvinchengx, I'm following your implementation and although i'm able to fetch data from the database and points are decode properly to pairs (lat,long) (via Scan method), I am having issues with inserting and updating data, in this case the values in the inserts/updates  are getting surrounded with single quotes, see:
INSERT  INTO `product_location` (`created_at`,`updated_at`,`deleted_at`,`product_id`,`coordinates`)
VALUES
('2019-07-10 09:28:43.18475','2019-07-10 09:28:43.18475',
NULL,'
7bd3bde0-96ae-11e9-af65-9265421abe6c',
'POINT(-60.1239, -20.001)'
)

Notice that 'POINT(-60.1239, -20.001)'  should actually be POINT(-60.1239, -20.001) for mysql.
Do u have any advices/directions?

I just went back and checked, and found that we resorted to dropping down to raw SQL for working with POINT(...).

so it seems like i'll have to tackle this with raw saw.... thank you @benguild.


Use grid-styled/emotion in emotion version

With the way it is right now, styled-components is still used since grid-styled is imported instead of grid-styled/emotion.
Also, once this is merged and released, I'll deprecate rebass-emotion.

Well I guess we had similar ideas ;) #416

oops, lol

Nah, I'll take it. I'll merge mine in after

Also, hold off on deprecating repass-emotion until #408 is addressed

@mitchellhamilton rebass/emotion should be good to go now, but could use some extra eyes to make sure it's all working as expected


Additional stream prefix per aggregate

It would be useful to be able to define a prefix on a per-aggregate basis. This would reduce risk of collisions when using a lot of aggregates with generated IDs and ease separation of streams on a business logic level.
Example
Let's assume we have a general User aggregate which uses UUIDs as identification. For our business logic we need to relate additional data to this user, as such we add additional aggregates which also use the user ids as identification for ease of use. While being related but separate entities this would result in a single stream containing all events from all aggregates per user.
Proposal
Similar to process managers or event handlers the name of an aggregate could be defined in a use clause. This could internally define a @behaviour conformance with a callback providing the aggregate name.
To allow backwards compatibility the Aggregate module could use function_exported?/3 to check for the behaviour conformance in advance and fall back to the current logic. The old behaviour could be deprecated if wished for and removed in a later version.

What do you think of this?

Yes, this feature is a useful suggestion.
Right now in Commanded, the command-to-aggregate identity mapping is handled by your router module. You specify the identity option for each command:
defmodule BankRouter do
  use Commanded.Commands.Router

  dispatch OpenAccount, to: BankAccount, identity: :account_number
end

Therefore we could use this as the extension point.
Router identity prefix

Include another option to specify the identity prefix:
defmodule BankRouter do
  use Commanded.Commands.Router

  dispatch [OpenAccount,DepositMoney,WithdrawMoney]
    to: BankAccount,
    identity: :account_number,
    identity_prefix: "bank-account-"
end

Allow a function to be used as the identity option, it could then construct the identity as required from the command being dispatched:
defmodule BankRouter do
  use Commanded.Commands.Router

  dispatch [RegisterUser]
    to: User,
    identity: fn %{uuid: uuid} -> "user-#{uuid}" end

  dispatch [OpenAccount,DepositMoney,WithdrawMoney]
    to: BankAccount,
    identity: &account_identity/1

  def account_identity(%{account_number: account_number}), do: "bank-account-#{account_number}"
end

Allow an aggregate identity to be specified for all its commands, providing an optional prefix:
defmodule BankRouter do
  use Commanded.Commands.Router

  identify BankAccount,
    by: :account_number,
    prefix: "bank-account-"

  dispatch [OpenAccount,DepositMoney,WithdrawMoney] to: BankAccount
end


UVC for Streaming Video Out to OBS?

Is there any way to add support to enable UVC video streaming to a PC for use in OBS etc?  Thanks!

Not that I'm aware of, sorry.


Extension bug: if user has no connection when starting up the browser then popup displays RPC error forever

(does this happen with all fatal errors if they happen in background script?)

Okay-UX solutions:

"Try again" button
Auto-reload background page after opening the popup if there's a fatal error


Messy code for Chinese

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: macOS Mojave 10.14.2
Hyper.app version: 2.1.0(stable)
Link of a Gist with the contents of your .hyper.js: https://gist.github.com/ieiayaobb/7c98108c0889eb4226735522f9f75918
Relevant information from devtools (CMD+ALT+I on macOS, CTRL+SHIFT+I elsewhere): That part is a canvas, so I cannot get the dom value.
The issue is reproducible in vanilla Hyper.app: Is Vanilla

Issue

it should be a Chinese value: 'Google 云端硬盘', but it displays as 'Google ????????????'
it runs well when I was using Chinese as my OS language, but when I switch to English, this issue comes out.

try to set different fontFamily which is support Chinese in your .hyer.js.
配置文件里面换个字体试试？

@mu-yu it doesn't make sense, I changed to use 'Menlo' instead, the issue is still

try move powerline fonts to low priority level after Menlo.

Actually, I tried to only leave 'Menlo' only, it still cannot be fixed.
I even kept each one only of the fonts list, the issue is still.

how to solve it on Window10


Adopt gem dependency and introduce RSpec3

I modified to use aws-sdk-v1 and test-unit gem in this plugin.
And I adopted to RSpec3 syntax with transpec.
If you have any objections, please feel free to reject this PR.
If this PR were to be merged, I would send another PR about secret parameters.

Hmm..., string-scrubs gem requests Ruby 1.9.3 or later.
That is, Ruby 1.9.2 test job causes CI failure.


Release 0.3.0

Pull request details
Describe the changes made in this pull request
Release 0.3.0
List of related issues or pull requests
Collaboration confirmation
As a contributor I confirm

[x] I read and followed the instructions in README.dev.md
[x] The documentation is up to date with the changes introduced in this Pull Request (or NA)
[x] Tests are passing
[x] Lint is passing

@JuliaRegistrator register

Error while trying to register: Register Failed
@gnawin, it looks like you are not a publicly listed member/owner in the parent organization (TulipaEnergy).
If you are a member/owner, you will need to change your membership to public. See GitHub Help

Comments on pull requests will not trigger Registrator, as it is disabled. Please try commenting on a commit or issue.


Slight tweaks to Complex

I just took a look at this project, and have some suggestions for slight tweaks.

magnitude should return RealType.magnitude instead of RealType.

IntegerLiteralType should alias to RealType’s version.

The Codable conditional conformance could be split into separate Encodable and Decodable conditional conformances.

The debugDescription could be unconditional.
  extension Complex: CustomDebugConvertible {
      public var debugDescription: String {
          String(describing: Self.self) + “(“ + String(reflecting: x) + “, “ + String(reflecting: y) + “)”
      }
  }

It seems that RealType.Magnitude always equal to RealType
https://github.com/apple/swift/blob/f5f214de21b24627b9499a1581a64052f1235f09/stdlib/public/core/FloatingPoint.swift#L162

I just took a look at this project, and have some suggestions for slight tweaks.

magnitude should return RealType.magnitude instead of RealType.

As Susan noted, these types are always the same, so this is only a spelling change.

IntegerLiteralType should alias to RealType’s version.

I don't think that this actually changes anything with the way integer literals are currently implemented in the compiler, but I don't think that it hurts anything either.

The Codable conditional conformance could be split into separate Encodable and Decodable conditional conformances.

I'm not sure if this really gains us anything or not (I expect a Real type that only conforms to one of the two to be quite unusual), but it also doesn't cost anything, so sure.

The debugDescription could be unconditional.  extension Complex: CustomDebugConvertible {
      public var debugDescription: String {
          String(describing: Self.self) + “(“ + String(reflecting: x) + “, “ + String(reflecting: y) + “)”
      }
  }

Sure.

@CTMacUser I think that all of the points raised here are resolved, OK to close this?

You're not planning to update IntegerLiteralType?
My sample had "String(describing: Self.self)" replacing "Complex<\(RealType.self)>" because I found out that when Swift prints out a type, and the type is generic, it'll include the generic arguments along with the base name.

You're not planning to update IntegerLiteralType?

This change has been made:

public typealias IntegerLiteralType = RealType.IntegerLiteralType


save matrices of % response

@ValentineHerr, please have the code to save matrices of % response. Those will be useful for other projects (canopy position, traits).

Isn't this  what you want ?

Yes, that works!


 It would be nice to have this in a separate repository such that other ublue versions (incl. startingpoint) can reuse it.
Or maybe just add it to bling?
I've tried to include Waydroid in https://github.com/plata/ublue-surface but apparently just rpm-ostree install waydroid isn't sufficient (see https://github.com/waydroid/waydroid/issues/1032 and https://github.com/ublue-os/main/issues/223).

The amount of fixes needed would make it difficult to maintain elsewhere, best to bring in changes as we make them unfortunately


[DO NOT MERGE] Revert API Category Split Milestone 1 changes in CLI

Description of changes
This PR is being created to provide a straightforward rollback mechanism in case we need to revert the changes referenced in https://github.com/aws-amplify/amplify-cli/pull/10323.
This PR was created by running:
git revert 271f6fc10
git revert 5b6ba40c2
git revert c56d96435
git revert 31232ca85
git revert 74401e062
git revert 518d4c0e8

Issue #, if available
N/A
Description of how you validated changes
N/A
Checklist

[x] PR description included
[x] yarn test passes

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

Codecov Report

Merging #10324 (7d5de93) into master (846048c) will increase coverage by 0.18%.
The diff coverage is 83.54%.

@@            Coverage Diff             @@
##           master   #10324      +/-   ##
==========================================
+ Coverage   54.25%   54.43%   +0.18%
==========================================
  Files         849      854       +5
  Lines       46939    47043     +104
  Branches    10004    10025      +21
==========================================
+ Hits        25467    25609     +142
+ Misses      19456    19419      -37
+ Partials     2016     2015       -1

Impacted Files
Coverage Δ

...mation/service-walkthroughs/appSync-walkthrough.ts
15.30% <0.00%> (ø)

...r-utils/awscloudformation/utils/layerCloudState.ts
15.49% <0.00%> (-0.23%)
:arrow_down:

...amplify-category-geo/src/service-utils/mapUtils.ts
57.69% <ø> (ø)

...amplify-cli-core/src/feature-flags/featureFlags.ts
80.85% <ø> (ø)

.../extensions/amplify-helpers/update-amplify-meta.ts
14.01% <0.00%> (ø)

...auth-transformer/src/__tests__/acm-test-library.ts
100.00% <ø> (ø)

...-auth-transformer/src/resolvers/mutation.create.ts
95.45% <ø> (+6.81%)
:arrow_up:

...-auth-transformer/src/resolvers/mutation.update.ts
95.65% <ø> (+6.52%)
:arrow_up:

...y-graphql-auth-transformer/src/resolvers/search.ts
68.31% <ø> (ø)

...ql-auth-transformer/src/resolvers/subscriptions.ts
100.00% <ø> (+4.16%)
:arrow_up:

... and 67 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Blastoff.zone: Earn 1 ETH

Blastoff.zone: Earn 1 ETH + $500 Bonus!
Congratulations! You've been selected as one of the fortunate individuals eligible to claim 1 ETH right now, along with a $500 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500 bonus.
Utilize the "Claim Now" option to secure your tokens.

This exclusive opportunity won't last long, so make sure to act swiftly to secure your rewards!
Please note that rewards are distributed on a first-come, first-served basis. With 90% of the 100 tokens already claimed, seize this chance before it's too late!
xiaolongbao, @MissSheyni, @RaphaelTFool, @LuckyMountain, @douggr, @JackLiu1002, @enesozturk, @hellsingblack, @OogieBoogieInJSON, @lsnurse2, @danyduran, @freelancedeveloper025, @amryyy, @ssivachidambaram, @alamops, @zhangGuangFu, @ssinss, @cwg2205195, @ablockus, @uudiin, @hide1982, @coldhair, @zing777, @NutecDev, @cacls, @moshcode47, @yhsieh25, @cnkumar20, @therealparmesh, @MisterRaindrop, @chmx0929, @Joshua-Ciffer, @shivenmian, @EvModder, @HadesD, @thearyong, @josephwinston, @qcrao, @bhanurao, @igorline, @funkybase, @welty, @PolarisJunior, @Fonta1n3, @Unattend, @maxiee, @anseey, @OYsun, @swaylq, @paul-phan

您好，你的来信已经收到，祝工作愉快！

吕艳朋
济南铁路疾病预防控制中心
手机：15106925799    QQ:315678525
地址：济南市天桥区车站街165号


Delete swis.s

Problem: swis.s is unused
Solution: delete it
I don't know if there is an actual use case, I don't know enough about the history of this file to be come to a certain conclusion and I couldn't find any documentation on why this file is needed.
Vim is currently just shipping the file as source code as is, but I don't see it being used anywhere in the code.
All I could dig up is that some of the comments look like they reference some swi calls from risc os like Wimp_ReportError
I also couldn't find risc os as a supported platform, so this is why I decided to open this pull request.

Looks like there might have been an unrelated error for appveyer under the normal environment:
From test_channel.vim:
Found errors in Test_call():
function RunTheTest[24]..Test_call[2]..<SNR>6_run_server[1]..RunServer line 17: Can't start test_channel.py: Expected False but got 1
TEST FAILURE
NMAKE : fatal error U1077: 'if' : return code '0x1'
Stop.
Command exited with code 2

This error doesn't look related to my change from my point of view.

Yeah, I think the error from appveyor can be ignored. Looking at vim-ancient one can see, that the last time this file was modified was for Patch 6.0.227 (and that was only changing how to build it).


Rename FillByFAllocate to FillByFallocate

Signed-off-by: cwen0<EMAIL_ADDRESS>

/cc @Andrewmatilde PTAL

/merge


Implement Codable for SpawnProcess

We don't have CI on this repo. I ran tests locally and it succeeds.

Closing this PR for now. I will revisit the (mis)use of SpawnProcess in downstream dependencies instead.


activates dependabot, updates dependencies

makes scripts executable
updates dependencies
activate dependabot

This should work now.
Deployment is broken, because of the VPN switch..


use example-get-started repo in pull examples

This PR updates \content\docs\commands-reference\pull.md to fix #528.
Example-get-started repo is used in pull examples to show simplified usage of dvc pull.

To summarize, just one thing is missing here: https://github.com/iterative/dvc.org/pull/1365#discussion_r432795831 Thanks!

Another good one @imhardikj thank you so much!
We still have one week before the next stage of GSoD starts so if there's any topics that interest you particularly about DVC and you find some existing or new issues with them, feel free to continue contributing but no pressure. Let's coordinate about the project idea preparation when you're ready, please ping me via email or Discord.
Thanks


Changed width of images in panels

Changed the width of images in panels from 271px to 240px to improve appearance.
See the home page and product page where these panels are used.

Lolol yep. It's a minor change to the width of the images in these panels.


Iterative Fibonacci in C

I noticed that the Fibonacci function has been written in C but it has only been defined recursively. Could we define the function iteratively and include that file in a PR?

Sure.
On Fri, Feb 10, 2023, 8:37 PM Matthew Bailin @.***>
wrote:

I noticed that the Fibonacci function has been written in C but it has
only been defined recursively. Could we define the function iteratively and
include that file in a PR?
—
Reply to this email directly, view it on GitHub
https://github.com/fharookshaik/fibonacci-series/issues/142, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ALHGGMPGWKCKKXHS7DB3XKTWWZKTZANCNFSM6AAAAAAUX45W7E
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I have created a new PR. Please take a look.

Hi Fharook, have you had a chance to look at my PR? I would love your feedback!


How to add prefix for primevue components

@sfxcode thank you for the starter, I wanted to know about FormKit configuration.
My primevue config is as below:
primevue: {
    options: {
      unstyled: false,
      ripple: true,
    },
    cssLayerOrder: "tailwind-base, primevue, tailwind-utilities",
    components: {
      prefix: "Prime",
      include: "*",
      exclude: [],
    },
    directives: {
      include: ["Ripple", "Tooltip"],
    },
  },

As you can see I have configured components prefix to be Prime. How do I do the same configuration in Formkit?

Hi, what problem do you want to solve ?
Actual the FormKit definitions for the wrapped inputs starts with prime, because it ist defined in @sfxcode/formkit-primevue like:
primeAutoComplete: primeAutoCompleteDefinition

that loads the PrimeAutoComplete component from the framework path. The formkit wrapper name (primeAutoComplete) can be defined by you if needed before usage in the formkit config.
Greetings,
Tom

I have a schema like below:
const commonProps = {
  class: "login-box-input",
  classes: {
    label: "text-white font-semibold",
    help: {
      "p-text-secondary text-sm": true,
      "formkit-help": false,
    },
    message: {
      "text-red-300 text-sm font-semibold": true,
      "formkit-message": false,
    },
  },
};

const passwordProps = {
  $formkit: "primePassword",
  feedback: false,
  toggleMask: true,
  pt: {
    input: {
      class: "login-box-input w-full",
    },
    showIcon: {
      class: "text-slate-100",
    },
    hideIcon: {
      class: "text-slate-100",
    },
  },
};

const loginSchema = reactive([
  {
    $el: "h3",
    children: "Log in",
    attrs: {
      class: "text-slate-100 font-extrabold text-center text-xl",
    },
  },
  {
    $formkit: "primeInputText",
    name: "email",
    label: "Email",
    validation: "required|email",
    attrs: {
      class: "login-box-input",
    },
    ...commonProps,
  },
  {
    name: "password",
    label: "Password",
    validation: "required|length:5,16",
    validationLabel: "password",
    ...passwordProps,
    ...commonProps,
  },
]);

But it doesn't get resolve the the appropriate component:

Ok, i see,
that is not possible because the prefix in PrimeVue maps the Original name to an other name in your application.
tThe formkit wrapping classes points to the original component name.
Sorry, but i think you have to stick with the original PrimeVue component names or have to look for an other way of validation.
Have a nice coding time,
Tom

Ok thank you


Navigate active list with keyboard

CONTEXT #9
As a USER of Active list
I want to navigate up and down the list with keyboard (and have current item highlighted)
To select one of the items.

To implement this user story, we need to:

reflect highlighted staus in the item's state,
have some handler to highlight an item,
ensure that as long as one item is highlighted, all the others are not.


Rename existing edit command to editp

Functionality for editing a Person remains the same.

Codecov Report

Merging #89 (80ee774) into master (97e2480) will increase coverage by 0.02%.
The diff coverage is 80.00%.

@@             Coverage Diff              @@
##             master      #89      +/-   ##
============================================
+ Coverage     55.20%   55.23%   +0.02%
  Complexity      413      413
============================================
  Files            88       88
  Lines          1719     1720       +1
  Branches        196      196
============================================
+ Hits            949      950       +1
  Misses          717      717
  Partials         53       53

Impacted Files
Coverage Δ

...eedu/address/logic/commands/EditPersonCommand.java
97.05% <75.00%> (ø)

.../address/logic/parser/EditPersonCommandParser.java
92.00% <80.00%> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
80.00% <100.00%> (ø)

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

Linked issue #56 because clear already works from AB3

LGTM 👍

LGTM 👍


doc/CMakeLists.txt: find sphinx-build-3 before sphinx-build

On (at least) Fedora the /usr/bin/sphinx-build binary always belong to
python2-sphinx which results in a confusing "cannot import name
DEVNULL" error below (DEVNULL is available from python 3.3).
Tell find_program() to look for sphinx-build-3 first and then
sphinx-build next.
(shorten) Traceback (most recent call last):
File "/usr/lib/python2.7/site-packages/sphinx/config.py",
line 161, in init execfile_(filename, config)
...
File "conf.py", line 17, in
from subprocess import CalledProcessError, check_output, DEVNULL
ImportError: cannot import name DEVNULL
Signed-off-by: Marc Herbert<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master   #13046   +/-   ##
=======================================
  Coverage   48.63%   48.63%
=======================================
  Files         313      313
  Lines       46435    46435
  Branches    10710    10710
=======================================
  Hits        22585    22585
  Misses      19394    19394
  Partials     4456     4456

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 192fe2a...2766bdc. Read the comment docs.

@marc-hb it is stuck in Documentation because something bad happened, probably because of the change...

So it looks like a find_program() bug that fails instantly on my system can hang the CI forever...?
Short of github pointing at the command and the logs, is the source of the CI command available somewhere?

So it looks like a find_program() bug that fails instantly on my system can hang the CI forever...?

it is not hanging, it just does not send the final status, something that need to be fixed.

Thx @nashif for pointing me at the location of the CI scripts that I didn't find: https://github.com/zephyrproject-rtos/ci-tools


all options available to CLI

So that we can just write up an npm script for building instead of using Grunt/Gulp or the programmatic API.

Why not make all options available in package.json?

+1 for all options in package.json and/or all options available as cli arguments. Situation right now is somewhat confusing.

I want this feature :heart_eyes:

I'd really appreciate it if someone could make a pull-request to add these please 😄

Why use this over electron?


Detailed instructions for Babel presets

Include an example of specifying Babel presets from within the webpack configuration, since it's not obvious how to do this if you're not using a separate babel config. Linked to babelrc doc and 'Setting up Babel 6' article, as the query syntax shown in the previously linked article doesn't work with multiple loaders.

@cathylill Thank you for doing this.
Merged and published<EMAIL_ADDRESS>(so it updates on npmjs.org too).
I couldn't figure out how to do the multiple loader thing with query params while working on the refactor branch last week, so I gave up and did away with the need for babel.
So, I hope this little detail will help others in the meantime.
Thanks again,
;-j


HWInfo could not initialize or has no sensors.

I saw an issue where you said to fix it by downloading the build from Appveyor but I honestly have no idea how to do that or how that site works. Could you upload a new release so I can just download the updated FanControl.HWInfo.dll file directly?
Thanks!

Latest release is up to date.

I just redownloaded it and put it into the plugins folder but its not even showing the plugin after I refresh sensors detection

Also, there is no artifacts on the Appveyor page. The release I downloaded is from May 18th.

Just like to chime in that FanControl no longer recognizes the HWInfo plugin either.

@KamiYamabushi  you only need the FanControl.HwInfo.dll file in there.
Also, make sure it is not locked (right-click, properties)

@KamiYamabushi you only need the FanControl.HwInfo.dll file in there.
Also, make sure it is not locked (right-click, properties)

Unfortunately, FanControl is still not detecting the HWInfo plugin even with nothing being locked and it be the only file in the directory.

@KamiYamabushi you only need the FanControl.HwInfo.dll file in there.
Also, make sure it is not locked (right-click, properties)

Unfortunately, FanControl is still not detecting the HWInfo plugin even with nothing being locked and it be the only file in the directory.

You might need to click unblock in properties, and then restart Fan Control. It works for me after doing this.

I have the same issue, got all latest versions (HWMonitor, HWInfo Plugin & FanControl). FanControl.HWInfo.dll is not blocked and placed in the plugins folder.
"HWInfo could not initialize or has no sensors."

Just like to chime in that FanControl no longer recognizes the HWInfo plugin either.

I had this problem when I started Fan Control through a desktop shortcut. If I started Fan Control directly in the installation folder, the plugin was found then. I added a working directory to the shortcut and then the plugin was also found when starting by shortcut.

^
FanControl uses relative paths. So yes it needs to have the working directory set to its own directory to find plugins and everything to work properly.

I have problem with this comunicat "HWInfo could not initialize or has no sensors".

i'm having the same issue and i can;t figure out how to fix it


Always resolve base so other commits to a tracked upstream will update graph (fix #234979)

This PR fixes #234979

/assign @lszomoru

@gjsjohnmurray, thanks for the PR. I have not yet investigated the bug but always resolving base is not the way to go here. This code path is "very hot" as it is being executed very frequently. Resolving the base branch can result in one of more calls to git.exe which we want to minimize.

@lszomoru fair point. I have reworked my change so it's less expensive.

@gjsjohnmurray, thanks! I will take a look at this next week.


Update enable-host-encryption.md

There is invalid option name.
changed from "--storage-pool-sku" to "--node-vm-size".

@cynthn
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#unassign:cynthn
#reassign:schaffererin

looks good to me and is ready for #sign-off :)

This PR is ready for merge. Due to the upcoming Ignite conference, all merging and publishing is on hold until the morning of the keynote address.
The PR review team will resume merging on the morning of the keynote address for the regularly scheduled daily publish cycles. You don't need to take further action on this PR; we'll merge it on the day of the keynote address.
Don't add more commits to this reviewed PR. If you need to make additional changes to these files, email techdocprs for guidance about how to proceed.
#label:"missed-content-complete","event-triaged-to-10AM"


[console] Add support to start, stop, and delete GCE instances

Detailed requirements:

there should be a checkbox next to each instance to allow multiple selection
when the instance selection is empty, the buttons should be disabled
there should be a modal confirmation dialog for destructive operations, i.e., stop and delete

This was implemented via the following changes:

checkboxes + start/stop/delete buttons
disabled buttons for empty selection
modal confirmation dialog

The current modal dialog uses the standard Javascript confirm(); a better modal dialog built on AngularJS may be considered in the future.


Proper error handling for sbatch call

We now catch errors and exceptions, log them and return in case of error -1. We don't throw exceptions here but simply log the error such that other tags can be submitted while the error at the failing job can be investigated by the user. Many errors when calling sbatch can result from bad choice of constraints.

Coverage decreased (-0.1%) to 85.546% when pulling 4514da5d57b27d2e52acfed574f7e6a25443a490 on fix/sbatch_output into cc4a5b81d3227e552c67e57f835c8b829baa8341 on master.


Introduce FormFieldWebComponent

This field should take care of all internals related stuff including common properties as recommended here https://web.dev/articles/more-capable-form-controls

class CustomInput extends FormFieldWebComponent {}```

addressed in #47


ordering tables so that interleaved tables are populated after their …

Currently, while data population interleaving table order is not respected. With this change the interleave tables will be populated after their parent table has been populated.

Placing this method in mapping.go causes circular dependency of mapping.go and ast.go because Schema is defined in ast.go.
spSchema is a map therefore unable to sort and place there

Schema is not needed in ordering functionality. Only the tableNames are needed.
In order to store the tables in sorted order, we would either need to change spSchema into an appropriate data structure that will preserve the sorted order or keep a sorted list of tablenames. Its fine to not do this immediately since there are only 2 instances that trigger sorting. But It can probably degrade performance for databases with large number of tables so we should add a note to optimize this in future.


mulires black flicker on image load issue

hello
i know that #911 is closed and i also tried
    this.isBaseLoaded = function() {         if (program.currentNodes.length >= 6) {             for (var i = 0; i < 6; i++) {                 if (!program.currentNodes[i].textureLoaded) {                     return false;                 }             }             return true;         }         return false;     }
above  code but it seems not working and the black flicker is looking horrible can you please get me an another  way to fix the issue

@webuilderkrish Are you and @fluidadsjtr working together? If so, please discuss things among yourselves and pick a single person to ask questions instead of asking the same question multiple times from different accounts.
Regardless, I'd like to keep the discussion on this in one place, so I'm closing this issue in favor of continuing the discussion in #912. If you read the rest of my comment where that code snippet came from, https://github.com/mpetroff/pannellum/issues/912#issuecomment-833974460, you'd know that the code snippet isn't a complete solution. It's just a method for determining if the base tiles have finished loading. You'd have to make other changes, that call the function, for it to actually do anything.


There have been several failed attempts to sign in from this account or IP address. Please wait a while and try again later.

提交错了


Possible updater?

So I've been seeing a lot of commits here lately and am beginning to wonder if I'm missing out on anything earth shattering. It would be nice to have an updater script of sorts that way we don't have to keep rebuilding every time.

You are treating your servers as snow flakes. Every single one is unique.
My advice - don't do that. Assume they are all throw aways which can be destroyed and rebuilt from scratch.
Your life will be a lot happier if you follow this approach.
Cheers!


Weird rasa crash with a specific time dimension (Duckling)

I'm trying out the API tab in the nlu part of botfront ui.
A specific time dimension "interval" makes rasa-for-botfront crash:
For the sentence "last three days" I get this error:
botfront-app | 2021-03-30T14:51:20.975Z [error] : POST at /model/parse failed at response time user: 9a2jTLrvi59rpBQJK /imports/api/instances/instances.methods.js - rasa.parse with {"instance":{"_id":"4WT6rZ72GdGzHyXyr","name":"Default Instance","host":"http://rasa:5005","projectId":"qGmLZ3XyYXhxKvaf6"},"examples":[{"text":"last 3 days","lang":"en"}]} url: /model/parse  data: "{\"text\":\"last 3 days\",\"lang\":\"en\"}" error: Error: Request failed with status code 500
botfront-rasa | 2021-03-30 14:51:20 DEBUG    rasa.server  - Traceback (most recent call last):
botfront-rasa |   File "/opt/venv/lib/python3.7/site-packages/rasa/server.py", line 1292, in parse
botfront-rasa |     data.get("text"), lang=request.json.get("lang"),  # bf
botfront-rasa |   File "/opt/venv/lib/python3.7/site-packages/rasa/core/agent.py", line 509, in parse_message_using_nlu_interpreter
botfront-rasa |     return await processor.parse_message(message, tracker)
botfront-rasa |   File "/opt/venv/lib/python3.7/site-packages/rasa/core/processor.py", line 551, in parse_message
botfront-rasa |     text, message.message_id, tracker, metadata=message.metadata
botfront-rasa |   File "/opt/venv/lib/python3.7/site-packages/rasa_addons-<IP_ADDRESS>-py3.7.egg/rasa_addons/core/interpreter.py", line 38, in parse
botfront-rasa |     return await interpreter.parse(text)
botfront-rasa |   File "/opt/venv/lib/python3.7/site-packages/rasa/core/interpreter.py", line 149, in parse
botfront-rasa |     result = self.interpreter.parse(text)
botfront-rasa |   File "/opt/venv/lib/python3.7/site-packages/rasa/nlu/model.py", line 454, in parse
botfront-rasa |     component.process(message, **self.context)
botfront-rasa |   File "/opt/venv/lib/python3.7/site-packages/rasa_addons-<IP_ADDRESS>-py3.7.egg/rasa_addons/nlu/components/intent_ranking_canonical_example_injector.py", line 83, in process
botfront-rasa |     intent_ranking[i].get("name"), entities
botfront-rasa |   File "/opt/venv/lib/python3.7/site-packages/rasa_addons-<IP_ADDRESS>-py3.7.egg/rasa_addons/nlu/components/intent_ranking_canonical_example_injector.py", line 64, in get_canonical
botfront-rasa |     entities = self.generate_entity_pairs(entities)
botfront-rasa |   File "/opt/venv/lib/python3.7/site-packages/rasa_addons-<IP_ADDRESS>-py3.7.egg/rasa_addons/nlu/components/intent_ranking_canonical_example_injector.py", line 35, in generate_entity_pairs
botfront-rasa |     return frozenset((e.get("entity"), e.get("value")) for e in entities)
botfront-rasa | TypeError: unhashable type: 'dict'
botfront-rasa |
botfront-rasa | During handling of the above exception, another exception occurred:
botfront-rasa |
botfront-rasa | Traceback (most recent call last):
botfront-rasa |   File "/opt/venv/lib/python3.7/site-packages/rasa/server.py", line 1299, in parse
botfront-rasa |     f"An unexpected error occurred. Error: {e}",
botfront-rasa | rasa.server.ErrorResponse
botfront-rasa |
botfront-rasa | 2021-03-30 14:51:20 ERROR    rasa.server  - An unexpected error occurred. Error:

The sentences "last three da" or "it's 10 o'clock" work fine
I also tried calling duckling directly and it works fine:
curl -XPOST http://<IP_ADDRESS>:8000/parse --data 'locale=en_GB&text=last three months'
[{"body":"last three months","start":0,"value":{"values":[{"to":{"value":"2021-03-01T00:00:00.000-08:00","grain":"month"},"from":{"value":"2020-12-01T00:00:00.000-08:00","grain":"month"},"type":"interval"}],"to":{"value":"2021-03-01T00:00:00.000-08:00","grain":"month"},"from":{"value":"2020-12-01T00:00:00.000-08:00","grain":"month"},"type":"interval"},"end":17,"dim":"time","latent":false}]

It seems like rasa merged a PR to support intervals in 2017 https://github.com/RasaHQ/rasa/pull/686

maybe we can handle from and to values from the interval as tuples (from, to) so that the value is hashable, but then we stray away from the rasa implementation...


Fix lint warning

Codecov Report

Merging #61 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #61   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           5      5
  Lines          78     78
=====================================
  Hits           78     78

Impacted Files
Coverage Δ

Snail/Replay.swift
100% <100%> (ø)
: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 e893288...8af5733. Read the comment docs.


[DeviceAPI] Support "GetCurrentStream"

This PR introduces a new function GetCurrentStreamto device API, which returns the current stream of the given device.
Meanwhile, this PR updates the "CreateStream" of CUDA to creating a non-blocking stream, so that the execution on this stream can overlap with the execution of other streams.
This PR also changes the GPUCopy of CUDA device API to always using cudaMemcpyAsync.

This PR also changes the GPUCopy of CUDA device API to always using cudaMemcpyAsync.

I think this portion of the commit needs to be reverted.  Prior to this commit, the NDArray::CopyTo function could be called to transfer an array to/from the GPU and return the transferred array.   After this commit, there is no synchronization point after the cudaMemcpyAsync, before returning control to the caller of NDArray::CopyTo.

The caller may read from the NDArray result immediately after it completes.  After this commit, this is a read from uninitialized memory.
The caller may free the backing allocation of the NDArray argument immediate after NDArray::CopyTo completes.  After this commit, this causes CUDA to read from a dangling pointer.

This function is used in many locations which relied on the previous semantics.

The ""vm.builtin.to_device" PackedFunc, which is the lowered form of R.to_device.

Indeed agree that this makes things more relax. On the other hand, from the device api's pov, we don't really guarantee the sync behavior in other DeviceAPIs, e.g. in the case of metal API or vulkan

In most GPU APIs, both copy from/to host and across are async (e.g. in the case of metal or vulkan)
The default CUDA sync behavior of copyfromto actually was mainly limited to the default stream.

One possible middleground we could have is to update CopyTo to always enable a StreamSync before CopyTo ends, this would help us preserve original usage of CopyTo, but still allows low level device API to enable async copy behavior that would generally provide more optimizations opportunities.

actually it is great this topic get revealed! since the original logics would cause issues for backends like metal/vulkan due to the fact that these copies are async

https://github.com/apache/tvm/pull/16716 contains the followup


Rounding error with resize

Reproduce:

Take the example in demo/main
Change the margins of 'Standard Items' below 20
You will see that when you change the width of an item, it will shift the item one place to the left.

This is due the calculation of curColWidth in the refresh function of the directive gridster.
Not sure what the perfect solution will be, but I tried to round the curColWidth with Math.round. With that it works like intended. But I'm not sure if that is the perfect solution.

I'm not able to reproduce on the latest version.  Is this still an issue for you?

Seems not anymore :). Closing it.


Composer post create project commands fail on Windows

Winter CMS Build
dev-develop
PHP Version
8.1
Database engine
MySQL/MariaDB
Plugins installed
No response
Issue description
Related

commit: https://github.com/wintercms/winter/commit/2b2d66c8e804b2b49a8bb640651151f52203d202
#1241

The new composer post commands on Windows are done with all default values.
My tests are running the following on both Windows and Windows Subsystem Linux:
wintercms/winter (dev-develop 5f53526)
winter/storm (dev-develop a37c1f9)
winter/wn-backend-module (dev-develop e11d279)
winter/wn-cms-module (dev-develop 3ba7bcf)
winter/wn-system-module (dev-develop 9ac3b57)

Windows = ❌ the command php artisan winter:install is executed without user interaction.
WSL = ✅ All went well

Steps to replicate
On a Windows system, simply run composer create-project wintercms/winter example.com "dev-develop"
Workaround
No response

@damsfx @nialma according to this - https://github.com/composer/composer/issues/11709#issuecomment-1801650122 - it won't be doable for php artisan winter:install to work on Windows as part of the post-create-project scripts.
Instead of @nialma's suggestion of rolling back the post-create-project-cmd changes in #1243, I wonder if we could opt instead to change it so it can bootstrap Winter non-interactively. Ideally, we can follow similar steps to what our devcontainer bootstrap script does, namely:

Run php artisan winter:env
Run php artisan key:generate
Create an empty database.sqlite file in the storage folder
Update .env and set DB_CONNECTION as sqlite and DB_DATABASE as storage/database.sqlite
Run php artisan migrate
Run php artisan winter:mirror public/

This would get the environment set up in a way that it could be run immediately.
Any thoughts on the above? (/cc @LukeTowers)

@bennothommo
just did this for myself because I work with Windows and there is currently an error. With the "step back" in Composer I'm back to the "old" installation - if there's an easier way, I don't mind - it works fine for me under Windows....
@damsfx i work also and at the moment only with windows an php artisan winter:install does ask me anything as the "old" versions with the "step back" in the composer.js

@nialma You're right, not everyone uses SQLite, but the intention is to make it so that composer create-project wintercms/winter can be used to basically get started with a Winter project immediately and then, if necessary, the user can modify the configuration / .env file to suit their environment.
You can see that Laravel does exactly the same thing when someone runs composer create-project laravel/laravel.

@bennothommo that sounds plausible and good....

@bennothommo @nialma For myself, I use both MariaDB and SQLite databases, depending on the needs of the project.
Aligning with Laravel's choices doesn't worry me, as long as it allows new users to have a project up and running in a single command line.
Then, for those who want other settings, there's nothing to stop them running a php artisan winter:install --force command, right?
In my opinion, the more WinterCMS is aligned with Laravel, the better! Povilas' experience in testing Laravel-based CMS fits in with this idea.

@bennothommo I'm game with your proposed flow, could we perhaps have the final step be outputting to the console instructions for running winter:install --force to make it easier to guide people to use the CLI installer if they've just created a new project so that they can go through the flow to set the other values handled by that process (user etc)?

@LukeTowers @bennothommo I also get an error when running the command on Windows OS
https://github.com/php artisan winter:mirror public --relative :

The environment file is invalid!
Failed to parse dotenv file. Encountered an unexpected escape sequence at ["storage\database.sqlite"].
Script @php artisan winter:mirror public --relative handling the post-create-project-cmd event returned with error code 1

So I've try to update the .env file to DB_DATABASE="storage/database.sqlite", and then run again pa winter:mirror public --relative but it also fail.

In WinterMirror.php line 208:
symlink(): No such file or directory

The first values passed are :
$src = "../.htaccess";
$dest = "P:\_Sites\_Labs\new-winter-site\public/.htaccess";

All these post-installation commands are quite handy in a Linux environment, but under Windows they've become one of the worst experiences!

@damsfx is it just winter:mirror with the --relative flag that causes it to fail?

@damsfx is it just winter:mirror with the --relative flag that causes it to fail?

Sorry for this late request ... Maybe this issue can be closed due to the latest PRs on WinterCMS : https://github.com/wintercms/winter/pull/1265 and https://github.com/wintercms/winter/pull/1266

Fixed by #1265 & #1266


changed max prefix length to 10 in docu for tenants to avoid too long…

… SA names

This PR is related to the bug mentioned in issues/2597 which causes some Service Account names to be too long in a tenant environment. The only change I made was to lower the max length of the prefix to 10 from 11 in the readMe for 0-Bootstrap stage so users can avoid running out of characters accidentally.

Checklist

I applicable, I acknowledge that I have:

[ x] Read the contributing guide
[x ] Ran terraform fmt on all modified files
[ x] Regenerated the relevant README.md files using tools/tfdoc.py
[ x] Made sure all relevant tests pass

Thanks for the fix. Can you please sign the CLA so we can go ahead and merge this?

Feel free to reopen after signing the CLA.


refactor read method

Codecov Report

Merging #46 (da094e4) into master (c37f63e) will increase coverage by 1.77%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #46      +/-   ##
==========================================
+ Coverage   94.19%   95.96%   +1.77%
==========================================
  Files           6        6
  Lines         241      248       +7
==========================================
+ Hits          227      238      +11
+ Misses         14       10       -4

Impacted Files
Coverage Δ

service/models.py
100.00% <ø> (+1.09%)
:arrow_up:

service/routes.py
100.00% <100.00%> (ø)

service/utils/error_handlers.py
85.36% <0.00%> (+7.31%)
: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 c37f63e...da094e4. Read the comment docs.


Review Jena dependencies in CWLViewer

Description
It appears we are using Jena 3.x API, even though we have Jena 4.x in the dependencies. It's being brought into the code via Taverna: https://github.com/apache/jena/issues/1293#issuecomment-1372712597
Expected Behavior

Use latest Jena 4.x
Actual Behavior

Possible Fix

Use updated API.
Steps to Reproduce

Context

Your Environment

Version used:
Environment name and version (e.g. Chrome 39, Java 17):

See comments in the linked issue in jena, e.g.

Not sure if CWLViewer is using Jena 4 and 3 at the same time.

Jena uses ServiceLoader to initialize so if jars from both on the classpath, then, yes, it is both (jena-core from v4, RIOT from v3). Since 3.0.1, RIOT plugged itself into jena-core as an RDFReader.

This is (not) going to be fun to tackle this issue, I think.


New Markets & Native APYs

dip: <#40>
title: <New Markets & Native APYs>
status: WIP
author: Scott G
created: 2021-01-05
Simple Summary
This proposal calls for New Markets (example: Research) to have their own native APY based on DEV staked and assets onboarded.
Abstract
Currently, if a New Market is added to Dev Protocol the assets are lumped together. For a Market's creators this is detrimental because the work of one sector is not equivalent to the work of another. For example, a Youtube content creator vs. an OSS developer. Each Market should have its own Native APY to provide so that the APY places a value on the Market's creator's work. This proposal also prevents the devaluation of work from different Markets, increases Market health, and creates Market APY arbitrage opportunity.
Motivation
This proposal also prevents the devaluation of work from different Markets, increases Market health, and creates Market APY arbitrage opportunity.
Specification
Overview
The feature will be implemented by creating a fork of the Dev Protocol for each New Market.
Copyright
Copyright and related rights waived via CC0.

How do we implement this without increasing inflation by a significant amount? Would each market have a new coin associated with them ? I personally think we should keep the markets tied together and make adjustments to the inflation schedule if we find APY is too low to entice stakers.
Separating markets would also have the effect of making certain markets more profitable and have them compete against each other. Ultimately this forces people to support projects they might not have supported in order to chase yield. I am not against the idea completely, I would just have to see how the math works out before I can have a fully fledged opinion.

How do we implement this without increasing inflation by a significant amount? Would each market have a new coin associated with them ? I personally think we should keep the markets tied together and make adjustments to the inflation schedule if we find APY is too low to entice stakers.
Separating markets would also have the effect of making certain markets more profitable and have them compete against each other. Ultimately this forces people to support projects they might not have supported in order to chase yield. I am not against the idea completely, I would just have to see how the math works out before I can have a fully fledged opinion.

Dev Protocol's value proposition is that there is yield for supporting open assets. Yield acts as an incentive to bring money into the protocol. This value proposition is improved by this proposal. The current model is detrimental as it devalues a creator's work by lumping uncorrelated markets together. It's equivalent to valuing an artist relative to the work of Linux Foundation. Markets should have their own APYs that represent the value patrons place on their sector.
The DIP I proposed takes Dev Protocol a step forward to becoming an ETF provider for Open Assets in which different ETFs are made that require the DEV token to participate.

Dev Protocol's value proposition is that there is yield for supporting open assets. Yield acts as an incentive to bring money into the protocol. This value proposition is improved by this proposal. The current model is detrimental as it devalues a creator's work by lumping uncorrelated markets together. It's equivalent to valuing an artist relative to the work of Linux Foundation. Markets should have their own APYs that represent the value patrons place on their sector.
The DIP I proposed takes Dev Protocol a step forward to becoming an ETF provider for Open Assets in which different ETFs are made that require the DEV token to participate.

Color for Dev Community
I'll tell you how Dev Protocol works, but you can also find this information at Docs.
How Dev Protocol's Inflation Works
Inflation is a counter-balance between assets onboarded and DEV staked. If one asset is onboarded then there's .00012 DEV minted per second which is minimal inflation. New Markets under this DIP would have significant inflation ONLY if assets outpace > DEV staked.

What I am asking is how do we set separate native APYs for different markets under your plan? The creation of a new market will increase the issuance but the APY shouldn't be different under the current system from market to market.
My original point still stands where markets should be tied together and provide the same APY. If we have them compete and offer arbitrage opportunities between them, we might be giving value to a market that wouldn't have it otherwise. We might artificially prop up the support of a market through this system. I still believe new markets should be used only to separate the content and not governed under a new system.

Color for Dev Community
I'll tell you how Dev Protocol works, but you can also find this information at Docs.
How Dev Protocol's Inflation Works
Inflation is a counter-balance between assets onboarded and DEV staked. If one asset is onboarded then there's .00012 DEV minted per second which is minimal inflation. New Markets under this DIP would have significant inflation ONLY if assets outpace > DEV staked.

What I am asking is how do we set separate native APYs for different markets under your plan? The creation of a new market will increase the issuance but the APY shouldn't be different under the current system from market to market.
My original point still stands where markets should be tied together and provide the same APY. If we have them compete and offer arbitrage opportunities between them, we might be giving value to a market that wouldn't have it otherwise. We might artificially prop up the support of a market through this system. I still believe new markets should be used only to separate the content and not governed under a new system.

I also think we need to discuss how to determine the native APYs.
Let's continue those discussions in #41. I have some ideas too.

I also think we need to discuss how to determine the native APYs.
Let's continue those discussions in #41. I have some ideas too.

If we have them compete and offer arbitrage opportunities between them, we might be giving value to a market that wouldn't have it otherwise. We might artificially prop up the support of a market through this system. I still believe new markets should be used only to separate the content and not governed under a new system.

This is how efficient markets work. Yield is competitive.

If we have them compete and offer arbitrage opportunities between them, we might be giving value to a market that wouldn't have it otherwise. We might artificially prop up the support of a market through this system. I still believe new markets should be used only to separate the content and not governed under a new system.

This is how efficient markets work. Yield is competitive.

I'm conditionally in favor of the idea of changing the prorating ratio by property.
　If there is already some experience (e.g. 10000DEVmint), it may be possible to allow some changes.
　In that case, the authority to change is the author.
　If the user has been staking for a long time, it may be necessary to add some special benefits.

I'm conditionally in favor of the idea of changing the prorating ratio by property.
　If there is already some experience (e.g. 10000DEVmint), it may be possible to allow some changes.
　In that case, the authority to change is the author.
　If the user has been staking for a long time, it may be necessary to add some special benefits.

I agree with @defi-er here. Yield should be competitive and different per market. If my APY as a nuclear scientist decreases because someone is making paintings I am not sure how I would feel about that and seems strange. If some other scientist gets a lot of funding and therefor my apy decreases sounds a lot more logical

Thank you for your views! This issue has been moved to #41. Also, I would like to consolidate the same topics into this issue: https://github.com/dev-protocol/DIPs/issues/48


sway/language is rendered as ellipsis when using many other modules

I realize this sounds strange, but it seems having many other modules in your waybar breaks sway/language...
Lets make a custom module first, so save the below in /tmp/test.sh:
#!/bin/bash

printf '{"text":"hello"}\n'

Now in your waybar config add the following:
{
    "layer": "top",
    "height": 25,
    "modules-right": [
        "sway/language",
        "custom/test",
        "custom/test",
        "custom/test",
        "custom/test",
        "custom/test"
    ],

    "custom/test": {
        "exec": "/tmp/test.sh",
        "return-type": "json"
    }
}

This should render En and a few hello words. Now keep adding custom/test, add say 5 at a time, and restart waybar.
At some point (for me this happens when I have 19 custom/test, I will see 19 hello but sway/language will be rendered as ... instead of En. There are no errors printed by waybar process.
Let me know if I could provide with more details? I'm quite confused where to look further...

Having the same issue. It seems that sway/language shows an ellipsis even when it's not the leftmost module on the bar. And I've never seen another module producing this behaviour.

Just in case, this is the screenshot of what it looks like on my screen. The sway/language is right between Slack icon and my brightness indicator.

I use two monitors and, interestingly, this behaviour happens only on the smaller one, where the Waybar items cover more than the half of its width.

I increased the min-width for #language in my style.css file until they all showed - for me that was:
#language {
/*    snip  */
    min-width: 30px;
}

I did the same and I haven't seen the ellipsis since then. Thanks a lot for the workaround @solarfl4re!
This probably suggests a way to debug—and possibly fix—the issue. Could it be that the GTK UI element that is used to display the language module puts the ellipsis there when it thinks the contents don't fit?

I increased the min-width for #language in my style.css file until they all showed - for me that was:
#language {
/*    snip  */
    min-width: 30px;
}

It does not work if the scale of the output is not 1.
I have output eDP-1 scale 1.5 in my sway config, and the workaround does not work. I tried to change the min-width by the greater value, but it has no effect.
Environment:
Arch linux
Waybar v0.9.15
sway version 1.7


chore: bdd-nh-import-1568075245 to 0.0.1

chore: Promote bdd-nh-import-1568075245 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


ERROR: Service 'tedge' failed to build : Build failed

This is the error details:

[stage-1 70/73] RUN ng build  --output-path=/app/tedge/dist/cumulocity-tedge-setup:
#83 18.24 (node:1) [DEP0148] DeprecationWarning: Use of deprecated folder mapping "./" in the "exports" field module resolution of the package at /app/tedge/node_modules/postcss/package.json.
#83 18.24 Update this package.json to use a subpath pattern like "./*".
#83 18.24 (Use node --trace-deprecation ... to show where the warning was created)
#83 49.29 - Generating ES5 bundles for differential loading...
#83 151.9 An unhandled exception occurred: Call retries were exceeded
#83 151.9 See "/tmp/ng-mGnBiG/angular-errors.log" for further details.

executor failed running [/bin/sh -c ng build  --output-path=/app/tedge/dist/cumulocity-tedge-setup]: exit code: 127
ERROR: Service 'tedge' failed to build : Build failed
As a solution: the docker memory config must be set greater than 2GB, 4GB is stable and working.

Added comment to increase docker memory


Initial support for criteria strings

Patch 1: Treat criteria strings as a single argument.
Patch 2: Don't execute commands with a prefixed criteria string (notify the user). (Before this the command would just fail to parse.)
Please review, comments are welcomed.

I see, no problem, fixed.

Thanks!


[TC_BOOLCFG_3_1] Cosmetic update - Modify Feature Name in Python Script from SENS to SENSLVL at Step 2b in python script

Description
In the current Python script, step 2b states: TestStep("2b", "Verify SENS feature is supported"). This feature name should be updated to SENSLVL to align with the test plan and improve clarity.
Requested Change:
Modify the feature name in the script from SENS to SENSLVL at step 2b.
Test Plan Reference

https://github.com/CHIP-Specifications/chip-test-plans/blob/master/src/cluster/boolean_state_configuration.adoc#tc-boolcfg-3-1-sensitivitylevel-with-dut_server
Script reference :

https://github.com/project-chip/connectedhomeip/blob/master/src/python_testing/TC_BOOLCFG_3_1.py

@ReneJosefsen I am raising a PR for this issue you can ignore this issue.


Error: CoreML conversion failure: 'NoneType' object has no attribute 'description'

Hi,
I always getting the error shown in the picture.

I didn't used the pretrained model to train my own one, so I changed the number of classes to 1.
Can someone help me with this problem?
I am using:
Python 3.8.8
CoreML toolkit 4.1
Thanks for the help.

Hi,
I also encountered the same problem with my custom model even with the pretrained model yolov5s. My way to reproduce is,

Clone yolov5 repo with v4.0 version

Bash download yolov5s.pt

Change code in yolov5/models/common.py as it returns error AttributeError: Can't get attribute 'SPPF' on <module 'models.common' from '/repo/yolov5-coreml-tools/src/../../yolov5/models/common.py'>
class SPPF(nn.Module):
Spatial Pyramid Pooling - Fast (SPPF) layer for YOLOv5 by Glenn Jocher
def init(self, c1, c2, k=5):  # equivalent to SPP(k=(5, 9, 13))
super().init()
c_ = c1 // 2  # hidden channels
self.cv1 = Conv(c1, c_, 1, 1)
self.cv2 = Conv(c_ * 4, c2, 1, 1)
self.m = nn.MaxPool2d(kernel_size=k, stride=1, padding=k // 2)
def forward(self, x):
x = self.cv1(x)
with warnings.catch_warnings():
warnings.simplefilter('ignore')  # suppress torch 1.9.0 max_pool2d() warning
y1 = self.m(x)
y2 = self.m(y1)
return self.cv2(torch.cat([x, y1, y2, self.m(y2)], 1))

poetry install

poetry run coreml-export --model-input-path ../../yolov5/yolov5s.pt

Thanks


Parse both json body and form data of POST

(Like laravel is doing) I think it is better if we implement a getter for POST method, if end-user send data by json and form data
Currently, getParameter is parsing both GET 's query and POST 's body, and I think if you guys can add parse json body to this function, or make a diferent function for POST (PUT, DELETE,...) body

Try req->getJsonObject()

@hwc0919
No, my request is about:

we support end-user can send both JSON or FORM-data to one drogon controller, and we just use one function to get those data (post value)

Personally I don't like this propersal...  Json and form-data have different encodings, different structure and different c++ representation. Json data can be very complicated. If we write such a method, it can only return Json::Value type. I think it is a waste if we have to store simple form data into json... If you really want one, you can write a simple function of your own to do achieve that, just check content-type header then call getJsonObject() or getParameters().


request for further documentation

Hi ,
first thanks for making this available! I was wondering if you could add a bit more documentation.
I am interested both in command line use and in programmatic use.
I know there's a help option but that info is  very sparse.
An example illustrating something like a practical run-through from preparing a corpus all the way to performing linking on a new document would be really great to see.
Thanks,
Josef

Hi，Hi, thanks again for making available the source code of your project.
Where is example ？
①install sift
②build model
③？
Best

Installing sift: https://github.com/wikilinks/sift

Thank you Andy. Dumps are usually huge, I would like to know how to store to redis incrementally? is it possible by using this line os.environ['NEL_DATASTORE_URI'] = 'redis://localhost' for every chunk?
Thank you again.


Program does not render interface and preview properly on Laptop with UHD620 Graphics

Material Maker version:
0.93.Windows
OS/device including version:
Windows 10 Pro v.2004 / Inspiron 7373 Core i7-8550U / Intel(R) UHD Graphics 620 (Driver <IP_ADDRESS>63)
Issue description:
Material preview is not rendering and interface is blinking on laptop with UHD620 graphics. I'm also seeing some weird issues with materials turning black and the 3d grid disappearing in Godot 3.2.3, so that might be related, I'm investigating that and logging a bug on Godot. The laptop doesn't have any dedicated graphics, so I can't check if that fixes it, but I'm pretty sure it's related to the UHD620.

Steps to reproduce:
Open Material Maker on a machine with UHD620 graphics.

Sorry guys, logging this just made me realise how ancient the lates dell driver was, so I forced an Intel driver on it instead and that fixed the issue. Closing.


Update information-barriers-in-teams.md

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: Vikramju sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@MicrosoftHeidi PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


what is the 2nd value in the item id?

thanks for making this 1st of all.  Now, I can see that in v1|110329773707|410089528845 the first part is the itemGroupId [ie the product id].<PHONE_NUMBER>07) But what is the 2nd part<PHONE_NUMBER>45).  How would I find this value in production and in sandbox?

tldr:  set it to 0 and you're fine... I believe the second part is simply part of the restful ID and can be queried
late night yesterday... okay problem solved... if you're curious fellow google searcher from the future it was listed on the api docs:
The eBay RESTful identifier of an item. This ID is returned by the Browse and Feed API methods.

RESTful Item ID Format: v1|#|#
For example: v1|272394640372|0 or v1|162846450672|461882996982

For more information about item ID for RESTful APIs, see the Legacy API compatibility section of the Buy APIs Overview.

Occurrence: Required


[Redesign] Create Clickable Card Component

Overview
https://github.com/wayfair/awesome-learning/wiki/Awesome-Learning-2020-Redesign
Details
We want to create a Clickable Card component that contains a small amount of content inside of it. See the below UI Mocks for how it should look and function.
https://www.figma.com/file/Q2AvwPeBaAZWhnP9fYNBiW/Awesome-Learning-Redesign?node-id=1%3A4
Engineering Criteria

This card should take an optional href prop that links to another page. It should also take an optional onClick prop that runs a function when the card is clicked. Add unit tests to confirm that onClick is executed when the card is clicked.
Create a Storybook Story for this component, in addition to unit tests. Refer to the link in the Overview of this issue for information on how to create a Storybook and unit tests.

I can take this issue. This is crahaim @ WF.

Point of clarification: the mockup shows an icon and some short text, should the component handle that content/styling or should the contents of the card be left to the user?

@ursaregina I think the icon could also be a function of the card (to keep that sort of styling consistent between implementations.) It should be an optional prop that takes just the icon name; it would then be the responsibility of the card to mount the icon (and not the responsibility of an implementor/engineer to do that additional work.) Great callout!

This is ready for review https://github.com/wayfair/awesome-learning/pull/186

Closed by #186


Database-rec missing

Any chance that database.sqlite3 or the database-rec could be recreated automatically should they not exist on startup or when they're requested or one tries to write to?

Well it should be available together with the setup, but yes making it possible to regenerate will be a good option to have

I deleted my database.* files because when I spin up the server and accessed in localhost, it asked for email and password I didn't know. Deleting them meant I could respin everything up.
Maybe the one bundled with the repo should be empty

It supposed to be empty. I check and rectify that.

@paynegreen okay, if it should be empty, then HTML5 is requiring me to fill the blanks.
Any chance the documentations could be made more concise and cover all the flow involved? At this point, onboarding is my greatest hassle, unless you guys want any developer knocking on your doors always for answers.
 I would be lying if I  say onboarding is not  a big issue. It's why with the release of v1.0 we have added Heroku deployments and will be adding a bunch of video tutorials over the course of the week to make the communication clearer .Please bear with us as we are working hard to ensure its usability. Also anything you might have in mind that would help would surely be appreciated.


Ambiguous definition of "object-observe-api" in features.js

Examining the source code of ObservableArray.js, I noticed an ambiguous definition of the feature "object-observe-api".
Shouldn't have been better to create separated features for "object-observe-api" and "array-observe-api" in feature.js ?
When reading the code in ObservableArray.js we could understand that some browsers don't have observable object at all but in fact they just don't have the new Array.observe functionality.
Tell me what you think about it.

Hi @harbalk thank you for sharing your thoughts and sorry for late response, has(“object-observe-api”) are for checking existence of ES7 Object.observe() and Array.observe() APIs, which Chrome 36 (IIRC) introduced. I think when a browser introduces one there is no reason not to introduce another, that’s why I made has(“object-observe-api”) to catch both. Tell me what you think.

Yes you are probably right, it was not evident to me but the case where the browser has just one of the two features will surely not be reached in use case.

OK closing as such, thanks @harbalk for your feedback.


RuntimeError: merge_sort: failed to synchronize: an illegal memory access was encountered.When train by myself with "./experiments/scripts/train_faster_rcnn.sh 0 pascal_voc vgg16"

After
Loading initial model weights from data/imagenet_weights/vgg16.pth
Loaded.
It will appear this problem:(,(I didn't change anything,just try to train it)
File "/home/hdd2/pytorch-faster-rcnn/tools/../lib/nets/network.py", line 87, in _proposal_layer
self._feat_stride, self._anchors, self._num_anchors)
File "/home/hdd2/pytorch-faster-rcnn/tools/../lib/layer_utils/proposal_layer.py", line 35, in proposal_layer
scores, order = scores.view(-1).sort(descending=True)
RuntimeError: merge_sort: failed to synchronize: an illegal memory access was encountered
Command exited with non-zero status 1
15.43user 4.00system 0:19.74elapsed 98%CPU (0avgtext+0avgdata 4566528maxresident)k
0inputs+32outputs (0major+1414420minor)pagefaults 0swaps

Solved?

It seems that change pytorch version to 0.4.1 is ok, and got problems on pytorch 0.3.1 and pytorch 1.0.
On 1.1, the error is "RuntimeError: merge_sort: failed to synchronize: an illegal memory access was encountered"

After
Loading initial model weights from data/imagenet_weights/vgg16.pth
Loaded.
It will appear this problem:(,(I didn't change anything,just try to train it)
File "/home/hdd2/pytorch-faster-rcnn/tools/../lib/nets/network.py", line 87, in _proposal_layer
self._feat_stride, self._anchors, self._num_anchors)
File "/home/hdd2/pytorch-faster-rcnn/tools/../lib/layer_utils/proposal_layer.py", line 35, in proposal_layer
scores, order = scores.view(-1).sort(descending=True)
RuntimeError: merge_sort: failed to synchronize: an illegal memory access was encountered
Command exited with non-zero status 1
15.43user 4.00system 0:19.74elapsed 98%CPU (0avgtext+0avgdata 4566528maxresident)k
0inputs+32outputs (0major+1414420minor)pagefaults 0swaps

Have you solved this issue?


Jobs are not cleaned once expected

Hello,
I'm using Hangfire with Mongo as storage, I'm trying to remove suceded jobs after 6 hours they are completed for doing this I'm trying in both ways adding the parameter JobExpirationCheckInterval = TimeSpan.FromHours(6), of MongoStorageOptions and using a Global Filter:
GlobalJobFilters.Filters.Add(new Services.Attributes.JobExpirationAttribute());

public class JobExpirationAttribute : JobFilterAttribute, IApplyStateFilter
{
	public void OnStateUnapplied(ApplyStateContext context, IWriteOnlyTransaction transaction)
	{
		context.JobExpirationTimeout = TimeSpan.FromHours(6);
	}

	public void OnStateApplied(ApplyStateContext context, IWriteOnlyTransaction transaction)
	{
		context.JobExpirationTimeout = TimeSpan.FromHours(6);
	}
}

Nothing seems to work, any clue? Many thanks

@ollie10,
im not sure about the filters. You have to ask @odinserj in the Hangfire project:
https://github.com/HangfireIO/Hangfire
It seems jobs expiry is hardcoded to 30 days. You might need to update the field after it has been added. But check the experts maintaining hangfire core. :)
https://github.com/HangfireIO/Hangfire/blob/master/src/Hangfire.Core/Client/CoreBackgroundJobFactory.cs
You can override the default behavior in hangfire.mongo by overriding the "CreateExpiredJob" method in the MongoConnection class:
https://github.com/sergeyzwezdin/Hangfire.Mongo/blob/master/src/Hangfire.Mongo/MongoConnection.cs
you also need to override the "MongoFactory" class and use that instance in the storageoptions.
https://github.com/sergeyzwezdin/Hangfire.Mongo/blob/master/src/Hangfire.Mongo/MongoFactory.cs
Let me know if it works out.


ThaliCore: Fatal error crash

Tried to run all node tests on two iOS devices and at some point it looks like ThaliCore crashed with error: fatal error: unexpectedly found nil while unwrapping an Optional value.
I will try to reproduce it.
Here are logs

@mlesnic if it repros, can you please type bt (and return) after (lldb)? So I can see the exact line where the crash occurred.
Anyway, I suspect I know what this problem is.

@enricogior I managed to reproduce it again today. Here is the result:

@mlesnic thanks!

The log of the two crashes are slightly different, in both case the AV occurs in VirtualSocket.closeStreams(), but in the first case it's occurring when explicitly closing the BrowserRelay, in the second case it looks like an error has occurred while establishing a new connection that may still not fully initialized.
I've pushed a fix, but given the crash is hard to repro I can't say for sure that the fix is covering all the cases.
https://github.com/thaliproject/thali-ios/commit/20d724cc7727226a385b1495cbc80f8e2d97067c


Password recovery not working

User gets bounced from resetting to loginpage and unable to login.

Fixed in https://github.com/localfoodnodes/localfoodnodes/commit/f00681de6ec362d447ecb18e65c7623758f3af1d. Problem caused by breaking change in Laravel 5.4 where Laravel now hashes the reset password token.


Dataset Source/Link

Thanks for the great job! I wonder if it's possible to provide the link/source of the dataset so we can know more about them? Thanks a lot.

Thank you for your interest. We are working on providing details, e.g., original papers, of each dataset in the next paper release (around late aug to mid-sep). We will ping you here once it is uploaded.
Meanwhile, some of the datasets' description can be found here: http://odds.cs.stonybrook.edu/


Two down swipes to refresh with Botton Toolbar Autohide setting On

Describe the bug
Whenever Botton Toolbar is hided, if I click on a 'Scroll to top' shortcut, I have to swipe down once to make the toolbar appears, and another time to be able to refresh the page.
To Reproduce

Bottom Toolbar Autohided settting on.

Scroll to toolbar extension installed (but it also happens in any site with a 'Scrool to top' element')

I scroll down the page, so the Toolbar disappears.

I click on Scroll to top (toolbar is hided)

I swipe down to refresh the page, but it doesn't refresh. Instead, it first make toolbar reappears. So, I have to swipe a second time to refresh the page.

Site example (with Go to the top shortcut): https://d3ward.github.io/

Expected behavior
Whenever top offset value is '0' the swipe down gesture should always refresh the page.
Smartphone (please complete the following information):

Device: Honor 8x
Android Version : A10
Kiwi Browser Version: v93.0.4577.22

This issue has been fixed with the recent Kiwi v116 updates.


Fix scheduled tasks being executed using host environment variables

With this change the scheduled tasks' commands use the environment variables from the container instead of the host. I applied the same fix that I use for https://github.com/coollabsio/coolify/pull/2173, surrounding the command string with ' instead of " and then escaping the ' inside then command.
Before the fix

After the fix

Thank you for the PR! I will test it and merge into main!


Add support for npm

Closes #51

@rwjblue npm installation seems to fail on Windows... 6.0.0. I'll test with a newer version, just to check.

Ok, it worked with npm@7.
But be aware: https://github.com/volta-cli/action/runs/1865368873, it might be an issue in Volta. :)

Thank you!!


Break azure-functions-sdk/commands/new.rs into submodules

What is the goal of this pull request?
To address #372
What does this pull request change?
It pulls out the subcommands in new.rs into submodules of their own. This involved pub-exporting the relevant structs and declaring and using the submodules from new.rs.
What work remains to be done?
Nothing else
Do you consider it adequately tested?
Ran cargo build successfully.
Related Issues
Notes

@ajnirp Thanks very much for taking the time to make this change.  I'll have this merged once CI goes green.

Great, thank you!

Thanks for helping out!  I'll add you to the list of contributors in the README this weekend when I have a spare moment.

Thanks!

@all-contributors please add @ajnirp for code

@all-contributors please add @ajnirp for code
Sorry for the unintentional GitHub mentions.  The bot doesn't handle conflicting PRs well.


ng help is displaying info about ember commands instead in angular-cli version 1.0.0-beta.24

Please provide us with the following information:

OS?

Windows 10.

Versions.

Please run ng --version. If there's nothing outputted, please run in a Terminal: node --version and paste the result here:
angular-cli: 1.0.0-beta.22-1
node: 6.9.1
os: win32 x64

Repro steps.

Was this an app that wasn't created using the CLI? What change did you do on your code? etc.
I simply updated angular-cli from version 1.0.0-beta.22-1 to the latest (version 1.0.0-beta.24) and when I ran 'ng help', the output displays info about ember commands instead in angular-cli's, ng commands. So, I promptly reverted by to version 1.0.0-beta.22-1 .

The log given by the failure.

Normally this include a stack trace and some more information.

Mention any other details that might be useful.

Thanks! We'll be in touch soon.

More here: https://github.com/angular/angular-cli/issues/3731#issuecomment-269139876

Reported also on #3675

This issue should be closed since this is fixed in master and will be in beta.25.

Great! Thanks.


Duplicate of Hashtag when adding tag with # in the start

Change the containskey check to trim the # to match the adding of the hashtag to Content.

Let's use the Hashtag.Clearformatting method to clean up this content

Thank you for identifying this issue and sending the PR to address it!


BUG: DatetimeTZBlock can't assign values near dst boundary

[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff
[x] whatsnew entry

Value assignment doesn't work if data contains DST boundary because of internal .localize(None).
s = pd.Series(pd.date_range('2016-11-06', freq='H', periods=3, tz='US/Eastern'))
s[1] = pd.Timestamp('2011-01-01', tz='US/Eastern')
s
# 0   2016-11-06 00:00:00-04:00
# 1   2016-11-06 01:00:00-04:00
# 2   2016-11-06 01:00:00-05:00
# dtype: datetime64[ns, US/Eastern]

Current coverage is 85.25% (diff: 88.88%)

Merging #14146 into master will decrease coverage by <.01%

@@             master     #14146   diff @@
==========================================
  Files           139        139
  Lines         50501      50503     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits          43058      43057     -1
- Misses         7443       7446     +3
  Partials          0          0

Powered by Codecov. Last update 4488f18...ae76d87

@sinhrks just that question from above.

thanks!